cmd/go: go mod fails to find direct dependencies

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

$ go version
go version go1.11.2 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
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/deklerk/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/deklerk/workspace/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
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/lk/zs4m7sv12mq2vzk_wfn2tfvm00h16k/T/go-build259032920=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
git clone https://github.com/golang/oauth2.git
cd oauth2/google
go mod init
go mod tidy

What did you expect to see?
A go.mod file with,
module golang.org/x/oauth2/google

require (
	cloud.google.com/go/compute/metadata v0.0.0-20181107005212-dafb9c8d8707
	golang.org/x/oauth2 v0.0.0-20181106182150-f42d05182288
	google.golang.org/appengine v1.3.0
)

And a populated go.sum.
What did you see instead?
module golang.org/x/oauth2/google

And an empty go.sum.
Note: another weirdness is that when I run go.vendor, I get:
$ go mod vendor
go: finding google.golang.org/appengine/urlfetch latest
go: finding golang.org/x/oauth2 latest
go: finding golang.org/x/oauth2/internal latest
go: finding golang.org/x/oauth2/jws latest
go: finding golang.org/x/oauth2/jwt latest
go: finding cloud.google.com/go/compute/metadata latest
go: no dependencies to vendor
$ cat go.mod
module golang.org/x/oauth2/google

require (
	cloud.google.com/go/compute/metadata v0.0.0-20181107005212-dafb9c8d8707 // indirect
	golang.org/x/oauth2 v0.0.0-20181106182150-f42d05182288 // indirect
	google.golang.org/appengine v1.3.0 // indirect
)

It found several dependencies, but it labeled them indirect. That's odd, because golang.org/x/oauth2/google directly depends on all of these. Furthermore, go mod vendor then goes on to state no dependencies to vendor immediately after it just vendored a bunch of things.

Is there a reason you did the go mod init inside the google directory?
Doing it at the top level directory works just fine:
$ git clone https://github.com/golang/oauth2.git
Cloning into 'oauth2'...
$ cd oauth2
$ go mod init
go: creating new go.mod: module golang.org/x/oauth2
$ go mod tidy
go: finding google.golang.org/appengine/urlfetch latest
go: finding google.golang.org/appengine v1.3.0
go: finding golang.org/x/net/context/ctxhttp latest
go: finding cloud.google.com/go/compute/metadata latest
go: finding golang.org/x/net/context latest
go: finding golang.org/x/net latest
go: downloading golang.org/x/net v0.0.0-20181106065722-10aee1819953
go: downloading cloud.google.com/go/compute/metadata v0.0.0-20181107005212-dafb9c8d8707
go: downloading google.golang.org/appengine v1.3.0
go: finding golang.org/x/net v0.0.0-20180724234803-3673e40ba225
go: finding github.com/golang/protobuf v1.2.0
go: finding golang.org/x/text v0.3.0
go: downloading github.com/golang/protobuf v1.2.0
go: finding golang.org/x/sync/errgroup latest
go: finding golang.org/x/sync latest
go: downloading golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f
$ cat go.mod
module golang.org/x/oauth2

require (
        cloud.google.com/go/compute/metadata v0.0.0-20181107005212-dafb9c8d8707
        golang.org/x/net v0.0.0-20181106065722-10aee1819953
        golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f // indirect
        google.golang.org/appengine v1.3.0
)
$ go list -m all
golang.org/x/oauth2
cloud.google.com/go/compute/metadata v0.0.0-20181107005212-dafb9c8d8707
github.com/golang/protobuf v1.2.0
golang.org/x/net v0.0.0-20181106065722-10aee1819953
golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f
golang.org/x/text v0.3.0
google.golang.org/appengine v1.3.0

I suspect the issue here in the case you report is that:
go: finding golang.org/x/oauth2 latest

find a package golang.org/x/oauth2/google that in effect clashes with the main module, but that this doesn't get reported/handled in the right way.
Can you give details about what you were trying to achieve here?

@jadekler ah, I see. Yes, submodules (also known as multi-module repos, indeed submodules are a special instance of multi-module repos) are supported. Here's an example:
https://github.com/go-modules-by-example/index/blob/master/009_submodules/README.md
The sequencing of things is the slightly tough bit.
But, I will just draw your attention to https://github.com/golang/go/issues/27858 and https://github.com/golang/go/issues/28136. The last time the question of splitting x/tools into modules was discussed on the golang-tools fortnightly call the answer from @bcmills was that @rsc et al are thinking about how submodules could/should work and how to version x/* along with Go releases. It's a slowly evolving picture.
I'll leave this issue open however, because I think there is an outstanding issue that there should have been a more clear error in what you originally attempted.

Thank you very much! This also needs to happen for cloud.google.com/go, so thanks for the tutorial link. re:oauth2; sounds like I should punt on that for now and go sync up with y'all that are already thinking about this. Will check in with Hana/Brad about that.

You're welcome. The advice at the moment is definitely to only create submodules/multi-module repos when you really need to. One indicator of being in that position is:

your major versions are >= 1
a major version bump demanded by breaking changes in one subset of packages is not acceptable to other packages (because by definition the major version moves forward for all packages if the entire repo is a single module)

Would certainly be interested in hearing your feedback on submodules/multi-module repos here or over in #27858/#28136 so please keep us posted.

a package golang.org/x/oauth2/google that in effect clashes with the main module, but that this doesn't get reported/handled in the right way.

Indeed. I believe that makes this a duplicate of #27063.

module$ git clone https://github.com/golang/oauth2.git .
module$ cd google
module/google$ go mod init
go: creating new go.mod: module golang.org/x/oauth2/google

module/google$ go mod tidy
go: finding google.golang.org/appengine v1.3.0
go: finding google.golang.org/appengine/urlfetch latest
go: downloading google.golang.org/appengine v1.3.0
go: finding golang.org/x/oauth2 latest
go: finding golang.org/x/oauth2/jws latest
go: finding golang.org/x/oauth2/internal latest
go: finding golang.org/x/oauth2/jwt latest
go: extracting google.golang.org/appengine v1.3.0
go: finding cloud.google.com/go/compute/metadata latest
go: downloading golang.org/x/oauth2 v0.0.0-20181106182150-f42d05182288
go: extracting golang.org/x/oauth2 v0.0.0-20181106182150-f42d05182288
go: downloading cloud.google.com/go/compute/metadata v0.0.0-20181114215827-b51c7c77b284
go: extracting cloud.google.com/go/compute/metadata v0.0.0-20181114215827-b51c7c77b284
go: finding github.com/golang/protobuf v1.2.0
go: finding golang.org/x/net v0.0.0-20180724234803-3673e40ba225
go: finding golang.org/x/text v0.3.0

module/google$ go list
go: finding golang.org/x/oauth2 latest
go: finding golang.org/x/oauth2/jws latest
go: finding golang.org/x/oauth2/internal latest
go: finding golang.org/x/oauth2/jwt latest
go: finding cloud.google.com/go/compute/metadata latest
can't load package: package golang.org/x/oauth2/google: unknown import path "golang.org/x/oauth2/google": ambiguous import: found golang.org/x/oauth2/google in multiple modules:
        golang.org/x/oauth2/google (/tmp/tmp.hhDmashDWj/module/google)
        golang.org/x/oauth2 v0.0.0-20181106182150-f42d05182288<EMAIL_ADDRESS>


[Explicit Module Builds][Incremental Builds] Only re-build module dependencies which have changed or whose dependencies have changed.

Up until now, incremental explicit builds behaved on an all-or-nothing basis: if any of the dependencies have changed, or new dependencies were added, then the driver launches a full dependency scan, followed by re-building all dependency modules.
Now, instead, we take a fine-grained approach to rebuilding module dependencies. The overall incremental build flow w.r.t. explicit module dependencies is now as-follows:

If there is a prior inter-module dependency graph and the source import set has not changed, and all dependencies are up-to-date, skip the dependency scan and re-use the prior graph.
When computing the first compilation wave's mandatory pre-compile jobs, examine each module dependency in the graph (either re-used prior or newly-computed). If any of the module's inputs are newer than the output, or if the output does not exist, this module is invalidated and must be re-compiled. Necessarily, every module in the graph between the root and the invalidated module must also be invalidated and re-compiled.

(Note: that we do not need to concern ourselves with changes to the compile command of a given module changing from the prior build - if that happened, then the module output path will have changed to reflect the change in the context hash, and expected output will be missing at the new expected path.)
Resolves rdar://113638007

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test Windows platform


Provided example for loading GGUF model is not working [Bug]:

Your current environment

🐛 Describe the bug
Hi @Isotr0py @mgoin,
I ran the gguf inference example gguf_inference
and I was getting NotImplementedError error

Could you please help me for this?

We haven't supported gguf quantization on cpu backend yet. You can try to install vllm with GPU backend.

@Isotr0py -- I tried with GPU backend, the error persists, please advise:
OSError: It looks like the config file at '/mnt/.cache/huggingface/hub/models--TheBloke--TinyLlama-1.1B-Chat-v1.0-GGUF/snapshots/52e7645ba7c309695bec7ac98f4f005b139cf465/tinyllama-1.1b-chat-v1.0.Q4_0.gguf' is not a valid JSON file.

The released 0.5.4 version hasn't included the gguf supoport. You can build from source code or install the latest nightly wheel:
export VLLM_VERSION=0.5.4 # vLLM's main branch version is currently set to latest released tag
pip install https://vllm-wheels.s3.us-west-2.amazonaws.com/nightly/vllm-${VLLM_VERSION}-cp38-abi3-manylinux1_x86_64.whl
# You can also access a specific commit
# export VLLM_COMMIT=...
# pip install https://vllm-wheels.s3.us-west-2.amazonaws.com/${VLLM_COMMIT}/vllm-${VLLM_VERSION}-cp38-abi3-manylinux1_x86_64.whl

I'm having the same issue. I tried to install a nightly commit, as suggested by @Isotr0py. But even then,I get the same output as @sergeol no matter which gguf image I try to load in, be it from huggingface hub, or a local path.
I wiped my conda & pip cache just to make sure nothing was get cached and I somehow got on older version, but no. Even on a fresh install of Python, the GGUF functionality seems broken.

The solution provided by @Isotr0py works well for me.


min-chars=1 then backspace, shows entire list

Hello
Im using the auto complete like
<input auto-complete
  class="form-control"
  [(ngModel)]="selectedColumn"
  [source]="columnsNotInDv"
  [min-chars]="1"
  [max-num-list]="10"
  placeholder="enter column name"
  display-property-name="DisplayName">

If I type in something that matches, with one character everything shows fine. Then if I backspace the one character out, it shows the entire list again.

fixed, please verify the test

@allenhwkim, did you publish it on NPM already?

@crh225 I did

@allenhwkim, this is not completly fixed
repeat the steps, and you can see a little grey div that appears, but it is empty.


HIVE-23741: Store CacheTags in the file cache level

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HIVE-XXXXX: Fix a typo in YYY)
For more details, please see https://cwiki.apache.org/confluence/display/Hive/HowToContribute

+1 pending tests


Optimized StateMachine.sol.

Refactored so that the transitions are stored in a mapping, instead of an array. Now all operations are O(1)

No dude, you should tell me that coverage is not high enough, the rules are the same for all.
Anyway, I did a major change in StateMachine.sol which requires review again. I've removed createState.
Now a state machine is defined just by declaring the transitions between states.
This implementation is simpler and more efficient, with the same functionality.


feat: add Dovetail connector

Checklist

[x] Ran Linter
[x] Create PR from non-main branch

Notes
It supports pagination
Testing
GET
URL: http://localhost:4444/api/v1/notes

GET notes by id
URL: http://localhost:4444/api/v1/notes/1IxNQrhqWFPzyZ45jFXkM1

POST
URL: http://localhost:4444/api/v1/notes

PATCH
URL: http://localhost:4444/api/v1/notes/5mntv5PjL3CLHddiBSnNNx

DELETE
URL:http://localhost:4444/api/v1/notes/5mntv5PjL3CLHddiBSnNNx

PAGINATION
Requesting a paginated list of notes using page[limit] in the param
URL:http://localhost:4444/api/v1/notes?page[limit]=5

Requesting a paginated list of notes using page[start_cursor] in the param will give the results of the next page
URL:http://localhost:4444/api/v1/notes?page[start_cursor]=WyJjcmVhdGVkX2F0X2Rlc2MiLFsiMjAyNC0wOS0xNlQxNDowNjoyMy44NDQzNzUrMDA6MDAiLCJjYmM1YTk3Mi1lMDdlLTRhMzQtYWEyNS1mNzk2MjRjMzAyZGQiXV0

I have updated the  base URL in Dovetail


MM-44973_Add prompt to start trial in Custom Groups menu entry

Summary

Show the Custom Groups entry in the Product Switcher disabled when in Starter plan with the key icon that will open the modal to start the trial.
Show the Custom Groups entry with the high voltage during the trial.

Ticket Link
https://mattermost.atlassian.net/browse/MM-44973
Screenshots
Cloud free no trial
https://user-images.githubusercontent.com/79058848/173142466-8e459d98-dc02-4881-8035-c7eea20974ce.mp4
Cloud free in Trial
https://user-images.githubusercontent.com/79058848/173142563-1e932d2e-f27e-49b8-bad0-9ccf0e525bc2.mp4
Cloud free after trial
https://user-images.githubusercontent.com/79058848/173142656-cbde8ce2-993a-4a6a-b81a-98b7e321c3e4.mp4
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/206018

I'm not sure if there is a discrepancy in spacing, but just to be safe, the title should be 24px away from the body


Provide more agents examples

Currently, everyone is talking about agents. However, most only offer examples like "search today's temperature". While this is not entirely useless, it is not overly useful either. Some more general and advanced agent examples would be more convincing.

@MonolithFoundation, that's a good point!
I was precisely searching for some examples: do you have some ideas of stuff that would have a real "wow effect" ?

Yes. If individuals possess passion and time, the following suggestions come to mind. These suggestions are not only entertaining but also useful:

Compose image and text-rich articles based on a given title or several provided materials. This implies that agents need to comprehend and analyze the title, utilize search engines to gather materials, and assemble an article interspersed with images and text that is ready for publication. (Initially, there is no need to understand images, but one must know how to insert images in the correct positions within the article. Additionally, the article should not appear to be generated by AI.)
Computer control Agents (high level and need multi-modal abilities);

The primary objective of this proposal is not to confine AI agents to revealing text information but to expand to a more diverse range and make it possible for the output to be publishable and easily digested by users.

Could you provide an example of multi-agent orchestration? It would be helpful to see how to coordinate multiple agents within the framework.

Please provide examples of multi-agents orchestration.

I just added an example of multi-agent orchestration @MonolithFoundation !
Tell me what you think!

Please add article writter

I think it's good as is for now! I'll come back to it if there's need to demonstrate a specific technical difficulty.


Move the rejected promises part of no-throw to new rule no-reject

This bascially moves #116 to a new rule called no-reject. See this comment for the background.
Fixes #115.

Released in 5.3.0.


K8SPG-597 Add all images from 2.3.1

K8SPG-597 Add all images from 2.3.1

sorting it like it is in the other operators, eg. 12.19, 12.17, 13.15, 13.13..., adding latest recommended versions for 12.x, 13.x, 14.x and 15.x

adding pgbackrest, pgbouncer and postgis images for each database as well and also not have just one recommended, but the same as in databases

Test name
Status

api-tests
passed


Not printing

Hello I have printing problem.
I have raspberry pi 3 (Linux raspberrypi 4.9.80)
and Seiko RP B10 RS232s interface printer, with comport serial adapter cable.
other information:
lsusb:
Bus 001 Device 010: ID 067b:2303 Prolific Technology, Inc. PL2303 Serial Port
lsusb -v -d 067b:2303
Endpoint Descriptor:
bLength                 7
bDescriptorType         5
bEndpointAddress     0x02  EP 2 OUT
bmAttributes            2
Transfer Type            Bulk
Synch Type               None
Usage Type               Data
wMaxPacketSize     0x0040  1x 64 bytes
bInterval               0
Endpoint Descriptor:
bLength                 7
bDescriptorType         5
bEndpointAddress     0x83  EP 3 IN
bmAttributes            2
Transfer Type            Bulk
Synch Type               None
Usage Type               Data
wMaxPacketSize     0x0040  1x 64 bytes
bInterval               0
ls /dev/tty*
/dev/ttyUSB0
pi@raspberrypi:~ $ pip freeze | grep pos
python-escpos==3.0a3
script:

from escpos import *
Epson = printer.Usb(0x067b,0x2303,0,0x83,0x02)
Epson.text("Hello\n")
Epson.cut()

There is no error but nothing is happening.
After checking ls /dev/tty*,  /dev/ttyUSB0_is disappeared..

Hi @arborin .
You are connecting to the printer via a USB to Serial adapter that creates a virtual serial device in your system. So I think you should use the python-escpos serial printer initialization:
Epson = printer.Serial("/dev/ttyUSB0")

Hello, thank you for your answer.
I checked it, but it is not working...
pi@raspberrypi:~ $ ls /dev/ttyUSB0
/dev/ttyUSB0
There is not an error but printer doesn't make anything.
pi@raspberrypi:~ $ sudo python3
Python 3.5.3 (default, Jan 19 2017, 14:11:04)
[GCC 6.3.0 20170124] on linux
Type "help", "copyright", "credits" or "license" for more information.

from escpos import *
Epson = printer.Serial("/dev/ttyUSB0")
Serial printer enabled
Epson.text("Hello\n")
Epson.cut()

@arborin - This looks like a transport issue. Have you configured the serial port speed, parity, etc to match something that the printer understands? Correct serial port settings should be listed in the vendor manual.
Once you use stty to apply the correct settings, you should be able to send text:
stty ...
echo "Hello World" >> /dev/ttyUSB0

See "What is the easiest way to configure serial port on Linux?" on Unix StackExchange for usage. Once it works on the command-line, you should be able to do the same thing using the available Printer.serial() from python.

Hi,
i'm using the lib version 3.0a9.dev26+g84a97e5-py3.6.egg. And i have the same kind of issue using a PL2303 USB to Serial port Adapter.
In my case even if the is_online() methoid return correctly True, when i use another method like text to send a string. Nothing happens and no error is return.
Do you have any clue to solve my issue ?
I'm connection using this command :
from escpos import *
p = printer.Usb(0x067b,0x2303,None,0,0x83,0x02)
p.text("anefoanefan")


[feature request] title (mouse over) on bookmarks

first, this is great, thank you and all the contributors for making this  (where's the buy me coffee link?)
FR:  could you add another field for bookmarks that would allow you to have mouse-over text appear when you hover over them?
Kind of like the app description field.
my use case is simple.. the link url is http://printer.mylan  but I want to put the IP address in a mouse over field so i can get the IP without having to do an nslookup or log in to my dns server.

+1 to this.  Description on links would be great for our use case as well.


ENH: Add single-sided p-values to remaining spearmanr and linregress

Reference issue

Part of work being done for gh-12506.
What does this implement/fix?

This adds an 'alternative' parameter to the spearmanr and linregress tests.
Additional information

Implementation (at the time of writing) is the same as that of gh-12597. The reasoning for combining work on spearmanr and linregress is that they are the two remaining functions on the table at gh-12506 that use the T distribution to calculate their p-values.

@DominicChm I think I resolved the merge conflicts for you. Please respond to @WarrenWeckesser's comments when you have a chance. Since it's easy, let's prioritize this.

Thanks @mdhaber!
@WarrenWeckesser Sorry for the delay, I've been focusing on finals for the past week. I've switched spearmanr to _ttest_finish. At some point, once these are all pushed and done, I'll probably go back and rename it and its uses as you suggested. For now I think it's better leave it as-is to avoid conflicts as much as possible.

@DominicChm @tirthasheshpatel how does this look to you?

@tirthasheshpatel Yes, linregress should have had some sort of external validation, so I added it. I also refactored the other tests a bit; I think they are a bit easier to follow now.  Let me know if this still has your approval.
@DominicChm Here's another one that I wanted to get merged - do the changes look good to you? Do you agree that they are improvements? If so, I'll merge.

Yes, linregress should have had some sort of external validation, so I added it. I also refactored the other tests a bit; I think they are a bit easier to follow now. Let me know if this still has your approval.

Yes, all the new changes look good!

This all looks good to me!

@tupui you said feel free to ping you, so here's one : )

@tupui you said feel free to ping you, so here's one : )

haha sure ;)

Merging; thanks everyone!


Remove removed files from autoconf Makefile.am files

A recent cleanup removed / moved the AUTHORS, etc. to an upper level. These should be removed from the autoconf Makefile.am files in the appropriate places

This seems to have been addressed.


Create estin.txt for the estin.dz domain

This is a state/public school founded a couple of years ago.

@MounirHamani Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


bug: minio doesn't return X-Amz-Bucket-Region if no location configured

minio doesn't return X-Amz-Bucket-Region if no location configured which will make our s3 region detect failed. We should better handle those:
If head bucket returns 403 but no X-Amz-Bucket-Region is given, we can use us-east-1 as region.

Hi  @Xuanwo ,i want to try this ;

Hi @Xuanwo ,i want to try this ;

Thanks a lot, have fun!


Class resilience cleanup

Follow-up to https://github.com/apple/swift/pull/18893 with additional cleanups and code review feedback suggested by @rjmccall. No functionality change intended.

@swift-ci Please test

@swift-ci Please test source compatibility

@swift-ci Please test


Scroll to a specific time in the day view

I actually have 2 questions about the day view.
The first one being is it possible to scroll to a specific time inside the day view?
The second one:
Is it possible to change the font/color/size of the time indicator on the left hand of the view?

I have same question, pls


Capitalize displayName for ECK 1.3.0 in community-operators

This is a follow up from https://github.com/operator-framework/community-operators/pull/2654, where the display name was not capitalized.
Please let me know if any other information is needed.

@david-kow We are moving to github actions. Can you please rebase to latest master? Thanks

/ok-to-test

/lgtm


running example Error: Type error: This kind of expression is always truthy.

hello
while running the exmaple (docker compose build) I'm constanatly facing this error Type error: This kind of expression is always truthy.
Can anyone help with this?
Screenshot for ref.

Issue was solved by changing from this code:
switch (condition) {     case "required":         return
This field is required.
;     case "minLength" || "maxLength":         return (
                 Your password must be between 8 and 64 characters long.
         ); }
to this: switch (condition) {     case "required":         return
This field is required.
;     case "minLength":     case "maxLength":         return (
                 Your password must be between 8 and 64 characters long.
         ); }
in the indicated files. There are 3 or 4 to be corrected

For people who have that in the codebase and are asking themself what happened, it was a hidden bug that was revealed by Typescript 5.6 update. Here is the PR https://github.com/microsoft/TypeScript/pull/59217


Skd path is different from what the demo shows .

Can't find package paths like this?
com.amazon.identity.auth.device.authorization.api.;
My aws-android-sdk-core jar has directory structure like this:
com.amazonaws.auth.;

Which sample are you referring to? Generally, the package belongs to Login with Amazon. You can get the library here http://login.amazon.com/android.


"Calligraphy" word in the surah Settings is mis-leading

Removing "Calligraphy" (رسم الخط) word in the surah Settings can help finding translations.
It's very mis-leading at first sight what do these two options at the bottom do

Do you have any recommendations?


API: should the RangeIndex constructor accept start=range_index?

xref #11892 (see changes and discussion)
RangeIndex, a new Int64Index subclass that will be used as the new default index (currently) has the following signature: RangeIndex(start=None, stop=None, step=None, name=None...)
For converting an xrange/range object into a RangeIndex, we have the constructor RangeIndex.from_range.
Although it isn't documented, the current constructor supports accepting a RangeIndex instances as the start argument, in which case a copy of the original index is produced, e.g., RangeIndex(RangeIndex(0, 10, 1)) -> RangeIndex(0, 10, 1).
I think this is a mistake: it is reusing the start parameter for a different type of argument, at odds with the documented behavior and predictable function signatures. If a user wants to create another RangeIndex from a RangeIndex, they can simply use the Index construtor, e.g., Index(RangeIndex(...)).
@jreback argues that not accepting a RangeIndex as the first argument to RangeIndex would be an API break, because it would create an Index constructor that is not idempotent. It would be "a major break with the current Index design".
Thoughts from anyone else?

In general constructors in pandas are "too smart", but we are stuck in a lot of cases.
I don't know that Index constructor idempotency is an API guarantee we necessarily need to make. It was part of the original effort to be similar to np.array. Further, are we expecting people to actually use pd.RangeIndex in many direct ways? I only ever thought of RangeIndex as being an Int64Index-like performance optimization. You can see my original patch here
https://github.com/pydata/pandas/commit/ef878f050938e9497b4228d72b1b297feff22306
For internal pandas development, we should probably start biasing toward more precise internal APIs where possible, but when user convenience is concerned (where I've typically leaned toward more "polymorphic" / intuitive constructors at the expense of purity) there's a decent argument.
So I know it is a significant improvement and lowers memory usage, but is RangeIndex something we even wish to advertise heavily, since for most users it will just "be there" incidentally through use of read_csv or DataFrame or other constructors that create default indexes?

RangeIndex was/is really an internal performance substitute for Int64Index. However we DO have an (maybe implicit) guarantee on idempotency for a Index objects.
So happy to discuss:

I think its much easier in a new issue, as I doubt that long twisting conversation (about multiple topics) was easy to follow
as discussed this is mostly internal, though we are exposing it as a range-like object, hence the default constructor
I agree with a bit too much magic in places, but I find this convenient and unambiguous, as I indicated to @shoyer , I don't find any good reason not to do this.
contrary to @shoyer 's statement, RangeIndex(RangeIndex(...)) merely returns the original object (you can of course get a copy via copy=True, but it is False by default).
many-many times internally we have to guarantee that something is an index, usually via _ensure_index, but this goes thru Index, not always what you want. I don't think this is actually used very much (e.g. almost always self._shallow_copy() is called), but I don't see any good reason to break from the: self._constructor(self) possibility. Then you have special casing all around. Something which I feel is SO important at the size of the library. Special cases just causes bugs.

@shoyer made two points: is this user-facing or a performance enhancement.
It is both, hence the intertanglement of the two.

@shoyer range does the same (10 in range(10) is/means stop=10).  Do you think adequate documentation would be a good enough compromise (current error message of RangeIndex constructor says start must be integers)?

In [1]: Index(xrange(0,10))
Out[1]: RangeIndex(start=0, stop=10, step=1)

In [5]: Index(pd.RangeIndex(0, 10))
Out[5]: RangeIndex(start=0, stop=10, step=1)

In [2]: pd.RangeIndex(pd.RangeIndex(0,10))
Out[2]: RangeIndex(start=0, stop=10, step=1)

In [3]: pd.RangeIndex.from_range(xrange(0,10))
Out[3]: RangeIndex(start=0, stop=10, step=1)

to satisfy @shoyer concern about magic and my own about idempotency/consistency
I would suggest we do this:
# [1] (though impl changes slightly),, but API the same
In [1]: Index(xrange(0,10))
Out[1]: RangeIndex(start=0, stop=10, step=1)

In [5]: Index(pd.RangeIndex(xrange(0, 10)))
Out[5]: RangeIndex(start=0, stop=10, step=1)

# [2] stays the same
In [2]: pd.RangeIndex(pd.RangeIndex(0,10))
Out[2]: RangeIndex(start=0, stop=10, step=1)

# but also accepts range (so this covers [3] from above)
In [2]: pd.RangeIndex(xrange(0,10))
Out[2]: RangeIndex(start=0, stop=10, step=1)

# make the 'range like' constructor something else (from_steps)
In [3]: pd.RangeIndex.from_steps(0, 10)
Out[3]: RangeIndex(start=0, stop=10, step=1)

so this makes RangeIndex constructor very much like the other Index constructor

def __new__(cls, data=None, dtype=None, copy=False, name=None, fastpath=False, **kwargs):

allows idempotency, and no possibility of confusion/magic
makes de-facto equal actions yield equal results (e.g. xrange/RangeIndex) have a single argument constructor from a 'range' object. These are quite similar, but very different from passing 3 integers.

if consensus, this is a pretty easy change.

@jreback the only downside of your new proposal is that the repr now no longer matches the constructor. I suppose we could make it RangeIndex(range(0, 10, 1)) but that does look a little strange to me.

In [20]: df.index
Out[20]: RangeIndex(start=0, stop=5, step=1)

hmm, right.
could do
RangeIndex.from_steps(0, 10, 1)

bug agreed that is a bit more verbose
of course could accept start,stop,step as **kwargs and correctly handle that
(though we are then back to the original constructor that I had)


How to report System.Exception using iOS Crashlytics?

There is Crashlytics.SharedInstance.RecordCustomExceptionName() and Crashlytics.SharedInstance.RecordError() but none of them is accepting System.Exception. What if I want to manually report a System.Exception, with the message, stacktrace etc.? Is it possible or not?
I did not find any docs for the Xamarin.iOS Crashlytics Nuget, so I'm asking here.

@SotoiGhost could you please update the link to the example? Or is there any other info how to use System.Exception with RecordError?

@qmark Here's the updated link to Crashlytics doc. Hope it helps you. Best!
https://github.com/xamarin/GoogleApisForiOSComponents/blob/master/docs/Firebase/Crashlytics/GettingStarted.md#log-non-fatal-exceptions


parent host marked as down stills shows as up in the metrics / host status

I have a test setup using ATS 8.0.3 configured with 5 parents using the following configuration:
remap.config:
map / http://ats_httpbin_1:80

parents.config:
dest_domain=. parent="ats_httpbin_1:80,ats_httpbin_2:80,ats_httpbin_3:80,ats_httpbin_4:80,ats_httpbin_5:80" parent_is_proxy=false round_robin=strict

I shutdown one of the parents (ats_httpbin_5) forcing ATS to mark that parent as down after some failed requests:
[Aug 21 10:28:35.488] {0x2b0cbab2b700} NOTE: Parent initially marked as down ats_httpbin_5:80
[Aug 21 10:30:00.924] {0x2b0cbaa29700} NOTE: Failure threshold met failcount:10 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down
[Aug 21 10:30:03.022] {0x2b0cbac2d700} NOTE: Failure threshold met failcount:11 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down

but traffic_ctl still shows the host as up:
root@ae6b460a4f72:/# traffic_ctl metric match host_status
host_status.ats_httpbin_1 1
host_status.ats_httpbin_2 1
host_status.ats_httpbin_3 1
host_status.ats_httpbin_4 1
host_status.ats_httpbin_5 1
root@ae6b460a4f72:/# traffic_ctl host status ats_httpbin_5
host_status.ats_httpbin_5 1

I've been able to reproduce the issue with master as well:
[Aug 21 10:36:33.793] [ET_NET 1] NOTE: Parent initially marked as down ats_httpbin_5:80
[Aug 21 10:37:51.912] [ET_NET 1] NOTE: Failure threshold met failcount:10 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down
[Aug 21 10:37:52.427] [ET_NET 0] NOTE: Failure threshold met failcount:11 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down

root@25ee1fccf746:/# traffic_ctl metric match host_status
proxy.process.host_status.ats_httpbin_1 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_2 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_3 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_4 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_5 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
root@25ee1fccf746:/# traffic_ctl host status ats_httpbin_5
proxy.process.host_status.ats_httpbin_5 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0```

same happens when the option parent_retry=unavailable_server_retry is used and a parent server is marked as down cause it's returning 503s. The log shows the host marked as down:
[Aug 22 05:18:38.923] [ET_NET 0] NOTE: Parent initially marked as down ats_go-http-server_1:80

but metrics and host status don't reflect this change

That log message “Parent initially marked as down ats_go-http-server_1:80” the host was marked down and that the internal fail count was incremented.  When the internal fail count reaches the configured, proxy.config.http.parent_proxy.fail_threshold (default == 10),  value then the host is made unavailable for use and you’ll then see the message “Failure threshold met failcount:%d >= threshold:%d, http parent proxy ats_go-http-server1:80”
The host will remain unavailable until the retry window elapses, proxy.config.http.parent_proxy.retry_time (default == 300 seconds). Once that window has expired, the host will be retried on the next transaction that picks it.  If the retry succeeds, the host is automatically marked up.
Now these fail counts and status are per entry in the parent.config, so think of it as per remap.  That host is only taken down for that particular entry in the parent.config.  If that host appears in another parent.config line, that entry is unaffected and remains available.  This is useful as it limits parents getting marked down in the case where a slow origin for a particular remap is timing out which may cause a parent to get marked down and made unavailable.
The Host status metrics however are global.  If you use traffic_ctl to mark down a parent or origin, you’ll see that it’s reflected in the metrics and that host you marked down is marked down across all parent.config entries and will be completely unavailable until you mark it back up with traffic_ctl.
John
<EMAIL_ADDRESS>From: Valentín Gutiérrez<EMAIL_ADDRESS>Reply-To: apache/trafficserver<EMAIL_ADDRESS>Date: Thursday, August 22, 2019 at 1:15 AM
To: apache/trafficserver<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [apache/trafficserver] parent host marked as down stills shows as up in the metrics / host status (#5854)
same happens when the option parent_retry=unavailable_server_retry is used and a parent server is marked as down cause it's returning 503s. The log shows the host marked as down:
[Aug 22 05:18:38.923] [ET_NET 0] NOTE: Parent initially marked as down ats_go-http-server_1:80
but metrics and host status don't reflect this change
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/apache/trafficserver/issues/5854?email_source=notifications&email_token=ABCXE3UKKSTGZ33KM7C2GKLQFY4INA5CNFSM4IOFCATKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD44EG2Q#issuecomment-523780970, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABCXE3WQNRMWL6VV27R3B7TQFY4INANCNFSM4IOFCATA.

Valentin,
This is known behavior.  There are two statuses, a global host status and a per parent.config health check status.  There are health check status created by parent.config line for each host.  Host foo.com on line 1 is treated independently of host foo.com on line 2 as far as health status is concerned.  His is because a parent trying to reach a slow origin might get marked down by the child cache due to a transaction timeout.  If this happens for foo.com for the entry on line 1 and foo.com is totally marked down, the entry for foo.com on line 2 for a different origin remains unaffected by health check timeouts.  If you use the wildcard destination in your parent.config, then all remaps would see that foo.com is unavailable.
The host status is the global status for foo.com.  It’s not changed by passive transaction health checks.  It may only be changed by using traffic_ctl to mark foo.com down.  When you do so, foo.com is marked down globally for all remaps and parent.config entries.
John
<EMAIL_ADDRESS>From: Valentín Gutiérrez<EMAIL_ADDRESS>Reply-To: apache/trafficserver<EMAIL_ADDRESS>Date: Wednesday, August 21, 2019 at 4:39 AM
To: apache/trafficserver<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [apache/trafficserver] parent host marked as down stills shows as up in the metrics / host status (#5854)
I have a test setup using ATS 8.0.3 configured with 5 parents using the following configuration:
remap.config:
map / http://ats_httpbin_1:80
parents.config:
dest_domain=. parent="ats_httpbin_1:80,ats_httpbin_2:80,ats_httpbin_3:80,ats_httpbin_4:80,ats_httpbin_5:80" parent_is_proxy=false round_robin=strict
I shutdown one of the parents (ats_httpbin_5) forcing ATS to mark that parent as down after some failed requests:
[Aug 21 10:28:35.488] {0x2b0cbab2b700} NOTE: Parent initially marked as down ats_httpbin_5:80
[Aug 21 10:30:00.924] {0x2b0cbaa29700} NOTE: Failure threshold met failcount:10 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down
[Aug 21 10:30:03.022] {0x2b0cbac2d700} NOTE: Failure threshold met failcount:11 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down
but traffic_ctl still shows the host as up:
root@ae6b460a4f72:/# traffic_ctl metric match host_status
host_status.ats_httpbin_1 1
host_status.ats_httpbin_2 1
host_status.ats_httpbin_3 1
host_status.ats_httpbin_4 1
host_status.ats_httpbin_5 1
root@ae6b460a4f72:/# traffic_ctl host status ats_httpbin_5
host_status.ats_httpbin_5 1
I've been able to reproduce the issue with master as well:
[Aug 21 10:36:33.793] [ET_NET 1] NOTE: Parent initially marked as down ats_httpbin_5:80
[Aug 21 10:37:51.912] [ET_NET 1] NOTE: Failure threshold met failcount:10 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down
[Aug 21 10:37:52.427] [ET_NET 0] NOTE: Failure threshold met failcount:11 >= threshold:10, http parent proxy ats_httpbin_5:80 marked down
root@25ee1fccf746:/# traffic_ctl metric match host_status
proxy.process.host_status.ats_httpbin_1 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_2 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_3 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_4 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
proxy.process.host_status.ats_httpbin_5 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0
root@25ee1fccf746:/# traffic_ctl host status ats_httpbin_5
proxy.process.host_status.ats_httpbin_5 HOST_STATUS_UP,ACTIVE:UP:0:0,LOCAL:UP:0:0,MANUAL:UP:0:0,SELF_DETECT:UP:0```
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/apache/trafficserver/issues/5854?email_source=notifications&email_token=ABCXE3R6SEAKBFCKVHNCBGLQFULPRA5CNFSM4IOFCATKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HGPG7TA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABCXE3UTBD2SFQKCHYP6KSLQFULPRANCNFSM4IOFCATA.

@jvgutierrez can you close this issue?

@jrushford Thanks for the explanation, is there any way to monitor hosts that are being marked as down only for one parent entry besides parsing the log?

the log line doesn't seem enough because it doesn't reflect the affected parent entry


TS Server language server does not apply compilerOptions to excluded files (such as tests), making development less ergonomic

Bug Report

🔎 Search Terms

tsserver
tsconfig
exclude
include
tsserver 2307
esm
compilerOptions

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about tsconfig, tsserver, language service

⏯ Playground Link
Not possible to reproduce in playground
💻 Code
This repository reproduces the issue reliably, with a full explanation and 3 possible workarounds.
https://github.com/isaacs/tsserver-2307-exports-issue/tree/main
Also tried:

Having a separate tsconfig.test.json that included the test files and excluded the source, in the hopes that tsserver would pick that up and use it instead.
Setting nvim's tsserver.tsconfigPath (this is a bit inconvenient, since it has to be done with each project, and it means that newcomers to the project will need to change their editors around before they can easily contribute)

🙁 Actual behavior
Receive a tsserver 2307 error stating that types could not be found, because compilerOptions were not applied when TS Server is processing the excluded test file, meaning that it does not know that it is supposed to load the module using import and Node16 semantics.
🙂 Expected behavior
Excluded test files should be processed by tsserver in the same manner as project source files. Ie, include and exclude settings in the tsconfig.json file should be ignored.

This would result in a massive amount of wrong errors in some of my projects. I've excluded files because they're incompatible with the settings of that specific tsconfig file and should not be evaluated using it.

@MartinJohns Hmm, that's a really good point.
Another path forward might be if tsserver did something like ts-node, so you could have "tsserver": { "include": ["**/*.*"] } to override the main tsconfig settings. (And I guess that's  how platforms end up with config proliferation issues 😅)

I have a hard time seeing the proposed behavior as anything except the source of a constant stream of "What do you mean exclude doesn't exclude things?" bugs. The suggested alternative configuration (I wouldn't even call it a workaround; it's just how things should be set up) is much clearer.

I have a hard time seeing the proposed behavior as anything except the source of a constant stream of "What do you mean exclude doesn't exclude things?" bugs.

As it is, the current behavior leads to "this module doesn't properly export types" bugs posted to maintainers of esm-only projects. It's very surprising that a node project with "type":"module" is managed in the editor as if it was building for es5 and commonjs. I'm not the smartest developer out there, but I've been building hybrid modules for a while now and getting very into the weeds with tsconfig edge cases, and I was completely stumped by this until I took a day to actually reproduce it and figure out what was going on.
Perhaps there are more sensible defaults that could be used by tsserver for excluded files in such a scenario? Or at least a better error message than "can't load the types", indicating that the current file is excluded from the tsconfig?
A tsserver override in tsconfig.json would also be helpful, as it would provide both a more convenient workaround which package authors could suggest in those issues, and something to likely find in a google search. I use that for ts-node in a few places for similar reasons.

@RyanCavanaugh Would you be open to / recommend an enhancement request or discussion regarding better ways for tsserver to communicate that the file being processed is excluded from the config it's using, and/or ways to provide alternative configuration overrides to tsserver?  I see that excluding the file is "Working as Intended", but I find it hard to believe that the resulting confusion is intended 😅
Some suggestions, any or all of which would improve the ergonomics imo, but also have their own trade-offs to explore:

attach a message to the tsserver error string if the file is excluded. So instead of Cannot find module 'some-dep' or its corresponding type declarations. (tsserver 2307) in this case, it'd be something like Cannot find module 'some-dep' or its corresponding type declarations. (tsserver 2307) (This file is excluded by the configuration in tsconfig.json.)
support a tsserver: { ... } section in tsconfig.json, akin to how ts-node does.
support loading tsconfig.tsserver.json along with (or instead of) tsconfig.json. This could be similar to (2), or could use first config file that includes the file (or something).

Probably other ideas out there that might be better.


set CLIENT_METADATA_REQUEST_USE_CONNECTION_CTX=true

When database and schema is null, getColumns() searches all columns of all databases.
This causes a bug like below.
When mode is INSERT, create temporary table first and its schema is calculated by DatabaseMetaData getColumns().
When the table of same name exists across some databases, temporary table schema consists of all of the tables' columns.

1. embulk-output-jdbc call getColumns() to fetch table columns [code]
2. In this case, table.getDatabase() and table.getSchema() are null.
3. when databse and schema are null, snowflake-jdbc searches all columns in account [code]
4. but,CLIENT_METADATA_REQUEST_USE_CONNECTION_CTX is true, session context's database/schema is used to search columns. [code]

@giwa @civitaspo please review my code (and my poor English)


[previewer/builtin] turn utils.has_ts_parser call into protected call

I've experienced that recent changes causes the following error if TreeSitter is not activated.
I get this error even on require('fzf-lua').files call:
Error executing vim.schedule lua callback: ...arm64/share/nvim/runtime/lua/vim/treesitter/language.lua:84: lang: expected string, got nil
stack traceback:
        [C]: in function 'error'
        vim/shared.lua: in function 'validate'
        ...arm64/share/nvim/runtime/lua/vim/treesitter/language.lua:84: in function 'has_ts_parser'
        ...e/pack/_/start/fzf-lua/lua/fzf-lua/previewer/builtin.lua:758: in function 'ts_attach'
        ...e/pack/_/start/fzf-lua/lua/fzf-lua/previewer/builtin.lua:825: in function <...e/pack/_/start/fzf-lua/lua/fzf-lua/previewer/builtin.lua:803>
        ...e/pack/_/start/fzf-lua/lua/fzf-lua/previewer/builtin.lua:833: in function 'do_syntax'
        ...e/pack/_/start/fzf-lua/lua/fzf-lua/previewer/builtin.lua:954: in function 'preview_buf_post'
        ...e/pack/_/start/fzf-lua/lua/fzf-lua/previewer/builtin.lua:716: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

$ nvim --version
NVIM v0.11.0-dev-836+g64847fbdc
Build type: RelWithDebInfo
LuaJIT 2.1.1724512491
Run "nvim -V1 -v" for more info

$ fzf --version
0.55 (devel)

This PR turns the related call (following stacking trace) into protected call.
I've tested it with my own config .config/nvim/init.lua on macOS.

This was just changed in #1464 by @RayGuo-ergou.

Are you using the latest nightly?
How can I reproduce the error? Will disabling treesitter be enough?

I just tried reproducing this on NVIM v0.11.0-dev-881+g865ba42e0 without treesitter enabled with no success.
Is it possible NVIM v0.11.0-dev-836+g64847fbdc doesn’t contain the aforementioned upstream treesitter changes? Can you try with the latest nightly?

Hi @balazs4 try to upgrade your neovim to recent nightly. Should fix your issue.
The reason for my PR was to adopt new breaking change in neovim and yes it will break for some nightly users but I think the point of "nightly" is to expect breaking changes with most advanced features. So I feel this should not be reverted.

Had a second look into upstream change, it seems the add function was only return void(nil in lua?). Which means loaded is always falsely so treesitter never work in preview window.

https://github.com/ibhagwan/fzf-lua/commit/ebb89e4e4065e31b029eee8f618e1ca660f41f35
Looking at the error it seems that lang is returned nil and then the call to vim.treesitter.start will fail, I simplified the PR by  validating lang before calling utils.has_ts_parser, merged manually

Ah that's right sorry did not pay much attention to the error message.

@ibhagwan @RayGuo-ergou thanks for quick response here! 🙏
i've just updated both nvim-nightly and fzf-lua (with the merged commit);
it works ✅
ps: Thanks for this extension, btw. 💯


Docker pull error

Hi,
Please help, while downloading the docker image on client machine it gives an error.
Command: nvidia-docker pull hub.baidubce.com/paddlepaddle/deep_speech_fluid:latest-gpu
Error: Error response from daemon: error unmarshalling content: invalid character '<' looking for beginning of value
Please help to fix the issue.

try nvidia-docker pull hub.baidubce.com/paddlepaddle/deep_speech_fluid:latest-gpu again, then please check this error for nvidia-docker.


Allow the instantation of NoState

Until now, since the init was internal, it was not possible to do:
let state: Storage<State> = .init(.init())
typealias State = NoState

Now it is.

@Aymenworks
That's true!
Thank you for your contribution!


Switch from ias to dcap

This PR tries to fix the issue #1240 , use dcap attestation in CI

ignoring the flaky M6 test because it ran through and only trivial changes followed


fix: lookup('url', ..) causes known issue on MacOS controllers

This PR addresses a known issue running lookup('url', ..) on MacOS.
The writeup in the issue indicates that there are 2 possible solutions:

uninstall requests and urllib3
use the uri module instead of the lookup

This ticket pursues the latter option, as the former is unrealistic for many users.
Additionally, this work includes downloading the node_exporter archive into a temporary directory, which is cleaned up after copying the extracted binary to the target.

Hello @gardar.  Thanks for the quick feedback!

the tempfile solution, rather than using the hardcoded /tmp path

I see issue that you are getting at.  My primary interest was to ensure that there was no trace left upon completion of the role.  Perhaps there is still a way to do this (handler?), but for now i will revert that commit as admittedly it was out of scope for the problem i was trying to address.

I see issue that you are getting at. My interest here was to ensure that there was no trace of the downloaded file left upon completion of the role. Perhaps there is still a way to do this (handler?), but for now i will revert that commit as admittedly it was out of scope for the problem i was trying to address.

Agreed, I'm not a fan of leaving temp files around after the run but there are still cases where someone might want to keep the downloaded binary to avoid downloading it in the next run. The download destination could perhaps be moved to a variable, so that you could choose where it would reside and then also have a optional cleanup task.
But for now you can always just do the cleanup in your playbook.

facepalm Well that is embarrassing. Of course you are right. These should (+will) be delegated to the controller for exactly the reasons you've suggested, and because running lookup on the a Mac controller was my issue in the first place.

Does the uri module work when delegated to localhost on mac? One would think that both the url lookup and the uri module delegated to localhost would behave in a similar way.

One of my most closely held software design principles is that it should be sufficiently robust such that it helps its users not have to care about the nuances or deficiencies of their system, especially when those are dependencies of the software itself. This certainly doesn't mean that the software should try to cover every possible circumstance. Rather, it should be willing to take on reasonable burdens if they absolve the user, how ever many of them there may be and at whatever level of skill, from having to search for and create workarounds to use that software.

I do agree with that statement.
While the proposed change is fairly minimal I am still not fully convinced that it is within the scope of the collection to be making workarounds for different controller hosts, installing dependencies on the controller hosts or configuring them in any way.
It's a known fact that ansible is not fully cross platform and different controller hosts require different setup, the workaround for this issue and also other controller host issues are listed in the official ansible documentation: https://docs.ansible.com/ansible/latest/reference_appendices/faq.html#running-on-macos
It looks to me like this should be a part of the setup for everyone that's running ansible on mac os.
If we were to add controller host specific code to the collection, then we would also need to be able to test it in the ci.
With that being said, the readme for the roles already lists dependencies that are required on the controller hosts.
https://github.com/prometheus-community/ansible/blob/f6e6ac041e22ec7c9cefaa14f8782e7f0236ddf7/roles/node_exporter/README.md?plain=1#L12-L13
Perhaps it would be a good idea to add the information about OBJC_DISABLE_INITIALIZE_FORK_SAFETY there?
(I hope that I don't sound overly negative :cold_sweat: )

Does the uri module work when delegated to localhost on mac? One would think that both the url lookup and the uri module delegated to localhost would behave in a similar way.

I've confirmed that the uri module does not suffer the same issue as lookup('url', ..).


SQLAlchemy: UUID value on PostgreSQL

I had some issue with UUID column on PostgreSQL because in models we mix String column with UUID.
It worked with mysql because of this where UUID was "casted" as chars.
But it's not the same for any other database server.

Sorry another question, why are the log entries removed when removing the workflow/bibworkflowobject?   Aren't they useful even after an object has been removed? Some of these logs should be stored in an audit log instead, but we don't have that yet.

Are you talking about the cascade deletion introduced here https://github.com/inveniosoftware/invenio/commit/ac2bd1fa67d73f1c0d0ff0d649966b850e878a00 ? :)

@lnielsen Thats a good point. Indeed we should probably amend that to not delete the logs.

@hachreak I just noticed the commit message:
* Defines a delete cascade action for BibWorkflowObject and
  BibWorkflowObjectLog if you delete a Workflow.

The cascade for BibWorkflowObjectLog was introduced by https://github.com/inveniosoftware/invenio/commit/ac2bd1fa67d73f1c0d0ff0d649966b850e878a00
Sorry, the commit message is not completely correct.


fix(docs): punctuation adjustment

close #1849

Could the title be more specific about the issue it is going to fix? @Jocs

Could the title be more specific about the issue it is going to fix? @Jocs

cool, done.


iPhone 7

Describe the bug
A clear and concise description of what the bug is and what you expected to happen.
Open unc0ver and click jailbreak from a brand new iOS 12.4 install. Gets to step 2 and errors out. The message on the screen says: “Error. Unable to find kernels credentials in kernel memory”
To Reproduce
Steps to reproduce the behavior:

Go to 'unc0ver '
Click on 'jailbreak'
See error

Screenshots
If applicable, add screenshots to help explain your problem.

Device (please complete the following information):

iOS Version: 12.4
iOS Device: iPhone 7 (iPhone 9,3)
unc0ver Version: 3.5.2

Place an "x" between the brackets if true:

[ ] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[x] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Logs
If applicable, add logs or error messages here.

Try doing the following:

Disable Find My iPhone
Disable your password and touch ID to unlock
Delete the unc0ver application completely
Reboot your device
Reinstall unc0ver via Cydia Impactor
Jailbreak as you would normally

I fixed it by reinstalling unc0ver 3.5.2. Not sure what the first problem was but that fixed it. It should have messed around with it more before submitting a bug message. Oh well


EXC_BAD_ACCESS KERN_INVALID_ADDRESS

Crashed: chillisource.flutter_uploader.queue (QOS: UNSPECIFIED)
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x0000000000000000
0  Runner                         0x103049d0c specialized Dictionary.subscript.getter +<PHONE_NUMBER> (<compiler-generated>:4297743628)
1  Runner                         0x103045f1c SwiftFlutterUploaderPlugin.urlSession(_:task:didSendBodyData:totalBytesSent:totalBytesExpectedToSend:) +<PHONE_NUMBER> (<compiler-generated>:4297727772)
2  Runner                         0x103046730 @objc SwiftFlutterUploaderPlugin.urlSession(_:task:didSendBodyData:totalBytesSent:totalBytesExpectedToSend:) +<PHONE_NUMBER> (<compiler-generated>:4297729840)
3  Foundation                     0x1baf709d4 <redacted> + 24
4  Foundation                     0x1bae724a8 <redacted> + 104
5  Foundation                     0x1baf72d30 <redacted> + 24
6  Foundation                     0x1bae72124 <redacted> + 740
7  Foundation                     0x1baf7375c <redacted> + 24
8  Foundation                     0x1baf7321c <redacted> + 184
9  libdispatch.dylib              0x1ba85239c <redacted> + 144
10 libdispatch.dylib              0x1ba845fd8 <redacted> + 20
11 libdispatch.dylib              0x1ba848758 <redacted> + 408
12 libdispatch.dylib              0x1ba847eb8 <redacted> + 596
13 libdispatch.dylib              0x1ba854bd4 <redacted> + 348
14 libdispatch.dylib              0x1ba855384 <redacted> + 120
15 libsystem_pthread.dylib        0x1ba8ab690 _pthread_wqthread + 216
16 libsystem_pthread.dylib        0x1ba8b19e8 start_wqthread + 8

Device
Model: iPhone XR
Orientation: Portrait
RAM free: 93.88 MB
Disk free: 18.26 GB
Operating System
Version: 13.3.1 (17D50)
Orientation: Portrait
Jailbroken: No

@onikiri2007 @BlueChilli would you please guys take a look and release a new version?
it would be also great to include fixes for #39

@onikiri2007 any updates on this issue? and for crash referred in #39

@ragnor-rs @ahujamanish how to reproduce the crash?

If you initiate multiple (binary) upload tasks, I get a crash in the didSendBodyData delegate method.

@ened it does seem to appear randomly. more likely with multiple tasks as @ahujamanish said

@ragnor-rs ok. I will take a look over the weekend. We need some refactoring / cleanup on the iOS side anyway.

@ened @ragnor-rs i was able to fix the issue. I’ll do a few test runs and I’ll raise a PR.

any updates?

A fixed version is now available: 2.0.0-beta.5. Looking forward to your feedback.


Compiling Komposer into App with Rollup throws "Does not Export Compose"

I was trying to bundle React-Komposer into my react project but when I try to use CommonJS & Node Resolver to import it, I get thrown the following error:
[Error: Module C:\Users\*******.******\Desktop\*******\JSApps\node_modules\react-komposer\index.js does not export compose (imported by C:\Users\*****.******\Desktop\*******\JSApps\containers\******UI.js)]

How are your importing the react-composer?

import {compose} from 'react-komposer';


Learn how to RunAs authority\system

Figure out how to get RunAs = "nt authority\system" working as described on https://learn.microsoft.com/en-us/windows/win32/com/runas . Relevant quote:

Can use "nt authority\system" with a COM server that is already started or that has an entry in the class table. However, you cannot use "nt authority\system" with a COM server that is not already started.

Untested startup command:
c:\Dev\tools\PSTools\PsExec.exe -i -s c:\Dev\ComSamples\x64\Debug\MyServerCpp.exe
Sample code: AppID SetRunAsPassword
Related links
Relevant:

https://learn.microsoft.com/en-us/windows/win32/com/localservice
https://learn.microsoft.com/en-us/windows/win32/com/installing-as-a-service-application
https://github.com/microsoft/Windows-classic-samples/tree/main/Samples/Win7Samples/com/fundamentals/dcom/dcomperm

Bacground:

https://stackoverflow.com/questions/70533985/cannot-create-object-as-regular-user-client-from-server-running-as-service with reproducer on https://github.com/willcoxson/CppService/tree/master/CppService
https://stackoverflow.com/questions/39796093/run-a-com-service-as-system

Task moved to https://github.com/forderud/RunInSandbox/issues/11


SurfaceOrientation typo in jni wrapper

Describe the bug
When using SurfaceOrientation.Builder().build() in Android platform, the no implementation exception is thrown out.
The JNI method name is mBuilderTriangleCount instead of nBuilderTriangleCount.
No implementation found for void com.google.android.filament.SurfaceOrientation.nBuilderTriangleCount(long, int) (tried Java_com_google_android_filament_SurfaceOrientation_nBuilderTriangleCount and Java_com_google_android_filament_SurfaceOrientation_nBuilderTriangleCount__JI)
To Reproduce
Steps to reproduce the behavior:
Try the code in android:
SurfaceOrientation surfaceOrientation = new SurfaceOrientation.Builder()
.vertexCount(vertexCount)
.triangleCount(triangleCount)
.normals(normals)
.build();
Expected behavior
Without exceptions.
Screenshots
N/A
Smartphone (please complete the following information):

Device: Pixel 2
OS: Android 10

Additional context
Fixed in PR: #2853

Thanks!


fix: webredirect info in Problem Report V2

Signed-off-by: sudesh.shetty<EMAIL_ADDRESS>

Covered as part of pull request bby @Moopli


Removed unused str() methods in tests models.

Follow up to 6461583b6cc257d25880ef9a9fd7e2125ac53ce1.

buildbot, test on selenium.

For what it's worth, I found myself adding back a __str__() method to debug failures like:
======================================================================
FAIL: test_orders_nulls_first_on_filtered_subquery (ordering.tests.OrderingTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tim/code/django/tests/ordering/tests.py", line 199, in test_orders_nulls_first_on_filtered_subquery
    self.assertQuerysetEqualReversible(
  File "/home/tim/code/django/tests/ordering/tests.py", line 129, in assertQuerysetEqualReversible
    self.assertSequenceEqual(queryset.reverse(), list(reversed(sequence)))
AssertionError: Sequences differ: <QuerySet [<Author: Author object (763856[103 chars]9)>]> != [<Author: Author object<PHONE_NUMBER>765191[92 chars]21)>]

First differing element 0:
<Author: Author object<PHONE_NUMBER>55102721)>
<Author: Author object<PHONE_NUMBER>76519169)>

- <QuerySet [<Author: Author object<PHONE_NUMBER>55102721)>, <Author: Author object<PHONE_NUMBER>69572353)>, <Author: Author object<PHONE_NUMBER>76519169)>]>
+ [<Author: Author object<PHONE_NUMBER>76519169)>,
+  <Author: Author object<PHONE_NUMBER>69572353)>,
+  <Author: Author object<PHONE_NUMBER>55102721)>]


Simplify kubeflow

Simplify KubeFlow for data scientists. All we need is is a pip install kubeflow

I think this PR is a bit incomplete. Can you specify where exactly this instruction is missing?

I think this PR is a bit incomplete. Can you specify where exactly this instruction is missing?

Kubeflow is not possible to install with pip, it is a Kubernetes platform.
/close

+1
We're doing a lot of work to make it just juju deploy Kubeflow or microk8s enable kubeflow but it's a lot of underlying K8s things going on.


[REF] Improve efficiency of empirical Fisher

This PR addresses an efficiency problem in the empirical Fisher, similar to that described in #83 for the MC Fisher.
Remove for-loop over data and use GGN-vector products of a pseudo-loss instead which parallelizes the loop.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

14 of 14 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 89.063%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
969

Relevant Lines:
1088

💛  - Coveralls


Accept any AsRef<[T]> as input

Since Vec implements AsRef<[T]>, this API change should be backward-compatible.
Looking at how the input is consumed in the trait implementations, I think we could easily go even farther with flexibility of the input here, accepting a slice of impl IntoIterator<Item = T, IntoIter = impl ExactSizeIterator>, which might be useful for e.g. inputs from VecDeque<T> for example, with the tradeoff of having a complex method signature.

Thanks for the PR! This seems like a nice improvement. I'll take a proper look asap, hopefully tomorrow.

Looking at how the input is consumed in the trait implementations, I think we could easily go even farther with flexibility of the input here, accepting a slice of impl IntoIterator<Item = T, IntoIter = impl ExactSizeIterator>, which might be useful for e.g. inputs from VecDeque<T> for example, with the tradeoff of having a complex method signature.

Yes this seems like a very good next step! Just one thought then, do we stop and rest here for a bit, or go straight for the next?

IMO the VecDeque case isn't all that big of a deal, since one can use the impl From<VecDeque<T>> for Vec<T> implementation which is guaranteed to only maybe have to re-order the elements, but will not reallocate. The slice-as-input case is the one I personally have a reason to want to fix: the sample data is accessed through a pointer received through FFI, and there's no way to construct a Vec<T> from that without allocating and copying. Other users may have some more exotic cases, but I'd argue that maybe we should wait for such users to file issues before trying to fix a hypothetical issue.

Though, I suppose accepting Iterators would allow people to supply potentially interleaved data without copying / reallocating.

Though, I suppose accepting Iterators would allow people to supply potentially interleaved data without copying / reallocating.

Yes it could certainly be useful for data in some interleaved binary format. One could then create an iterator that both converts the bytes to floats and de-interleaves. But you are right, nobody has asked for it.
I'm perfectly fine with making a new release with the AsRef solution, and save the iterator idea for later.

Only note here is that this change means the Resampler<T> trait is no longer object safe. So after this patch, this used to work but doesn't any more:
let b: Box<dyn Resampler<f64>> = todo!();

203 |     fn process<V: AsRef<[T]>>(&mut self, wave_in: &[V]) -> ResampleResult<Vec<Vec<T>>>;
    |        ^^^^^^^ the trait cannot be made into an object because method `process` has generic type parameters

I personally think this is fine - but I wanted to make sure it's understood that this does break some compatibility. A workaround for anyone who needs the trait to be object safe is to define their own proxy trait with a blanket impl that has concrete parameters:

Oh, I didn't think of this. Thanks for pointing it out!
I actually use it exactly like that myself and this makes it much less convenient.
Is there some other way to avoids requiring Vecs, that can still keep Resampler<T> object safe?

Not to my knowledge, but anyone can define an object safe proxy trait like this if they need it (see below).
The reason I took note of this PR is that I've been working on a set of traits for audio buffers and that rubato was one of the trial libraries. I am currently holding off for GATs because they're needed to make the abstractions ergonomical and to avoid runtime costs. But the consequence here is that the fork of the Resampler trait I was working with is not object safe either all though I used the trait proxy trick to provide something compatible in DynResampler<T>. This would allow using different kinds of audio buffers like @jessa0 mentioned.
Object safety would be difficult to come by anyway if you ever intend to make the resampler abstract over the shape of the audio buffers. Which is why I personally don't mind dropping it. For cases where you need multiple different kinds of resamplers at runtime the trait proxy trick can be used or one can simply opt to use an enum with the variants that are needed.
Object safe proxy trait
use rubato::{ResampleResult, Resampler};

pub trait ObjectSafeResampler<T> {
    fn process(&mut self, wave_in: &[Vec<T>]) -> ResampleResult<Vec<Vec<T>>>;
    fn nbr_frames_needed(&self) -> usize;
    fn set_resample_ratio(&mut self, new_ratio: f64) -> ResampleResult<()>;
    fn set_resample_ratio_relative(&mut self, rel_ratio: f64) -> ResampleResult<()>;
}

impl<T, U> ObjectSafeResampler<T> for U where U: Resampler<T> {
    fn process(&mut self, wave_in: &[Vec<T>]) -> ResampleResult<Vec<Vec<T>>> {
        Resampler::process(self, wave_in)
    }

    fn nbr_frames_needed(&self) -> usize {
        Resampler::nbr_frames_needed(self)
    }

    fn set_resample_ratio(&mut self, new_ratio: f64) -> ResampleResult<()> {
        Resampler::set_resample_ratio(self, new_ratio)
    }

    fn set_resample_ratio_relative(&mut self, rel_ratio: f64) -> ResampleResult<()> {
        Resampler::set_resample_ratio_relative(self, rel_ratio)
    }
}

fn main() {
    let b: Box<dyn ObjectSafeResampler<f64>> = todo!();
}

Thanks! I think just including that proxy trait in the library would be a good compromise.
I'll take it as is, and just rename it VecResampler or something, and document it along the lines of:
pub trait VecResampler<T> {
    /// Resample a chunk of audio.
    ///
    /// This is a helper trait that can be used when a [Resampler] must be made into an object.
    /// It differs from [Resampler] by fixing the types of the input of `process()` to &[`Vec<T>]`.
    /// This allows it to be made into a Trait object like this:
    /// `let boxed_resampler: Box<dyn VecResampler<f64>> = todo!();`
    //// Use this implementation as a template if you need to fix the input type to something else.

This solves the problem in many cases, and shows how to do it in others.

Whoops! That makes me think it'd be good idea to have unit tests ensuring any traits intended to be object-safe remain so in the future.

Good idea to add a test to make sure the object safe trait stays that way!
I implemented a new VecResampler with such a test in https://github.com/HEnquist/rubato/pull/21


Change the "Collect Data" icon to match the TDP icon on docs.newrelic.com

consistency is key for usability across the product. we'll need a slightly different treatment on it, as our icons have no colors. But we should change the "Collect Data" icon to match the new TDP icon on docs.newrelic.com.

moving to the current iteration as this will align with the NR marketing site well.

Here's the recolorized version of that icon for use on the developer site.
https://drive.google.com/file/d/14S-VMqARrBwYl8Ryo69j8eFT1qkmf8PR/view?usp=sharing

@djsauble I can make this change if you want to either upload the image directly to this issue, or give me access to the location on google drive. Thanks!

I'll leave this for the team to resolve. I thought it be quick to change, but it's not as simple as I thought. :)

Collect_data_icon.svg.zip

Heads up, this icon does not look super great at a small size. Here is a screenshot of the nav if we kept the icon the same size as the others:

Here's a more focused screenshot:

As you can see, its very difficult to make out what the icon is.
@djsauble we may need to rethink the sizing of the icons in the sidebar a bit if we want them to match, or we may need to rethink how we can best make this icon look alright in the sidebar. I really like the icon itself, but it doesn't look like its meant to render at small sizes.

We can probably increase the icon sizes a little to get more clarity... They do look really small on most screens.

Hrm, yeah, that doesn't look great. Let me play with the stroke width a li'l bit.

I doubled the stroke width on it, let's see if that helps. https://drive.google.com/file/d/14S-VMqARrBwYl8Ryo69j8eFT1qkmf8PR/view?usp=sharing

We can also change the width height on these svg icons to 2em

The icon is a touch better, but at this size, its still just a bit hard to fully understand what the icon is. Should we consider a larger size like @jpvajda 's suggestion?

Bumping the icon size to 2em could look cool. @djsauble what do you think?


clean up icon path confusion

With https://github.com/Microsoft/vscode/commit/269ae60a4b7f0afb4d779433d88d9c1ef2190bc6 the concept of an iconPath has been added. This competes with iconClass and having both doesn't make much sense. While I understand the reasoning for us having it iconPath is only honoured in a single place and every other user will fail to make it work. At least the MenuItemActionItem should know what to do, ideally no competing concepts exist.

Yes a bit ugly. One issue is that an extension today can contribute an icon as a path for an command and we expand that a) into an iconClass and b) into an iconPath. I think we could possible go away from iconClass and keep iconPath, though then we have to somehow keep the semantic of different icons per theme and switch accordingly. The beauty of CSS is that you do not have to care about our 3 base themes.
Alternatively, it would be relatively easy to support iconPath in MenuItemActionItem alone by just setting the background-image if we do not have a CSS class defined otherwise.

One issue is that an extension today can contribute an icon as a path for an command and we expand that a) into an iconClass and b) into an iconPath.

Well, the original design was only has iconClass which abstracts away the fact that extensions can provide one or two icons for the dark and light theme. And "yes" this also makes theme switching nice. Now, the bad iconPath has appeared which breaks the "information-hiding" and it also prefers the dark-variant of an icon. So, I don't think the iconPath can be supported in MenuItemActionItem with ease. What's needed is to carry along the full icon information and delay the CSS creation


Usage without module / exports / RequireJS/AMD

It would be nice if you could add a fallback for a regular JavaScript usage with <script src="/js/vue-progressbar.js"></script> - meaning without any AMD loading / gulp/grunt preprocessing or ES6 features (like for IE10, Mobile Safari on iOS 9/10) ? Seems like its dependent on the module.export

I second that.

Thirded

Generally speaking I'd agree. That said there are situations, uncommon or sub-optimal as they maybe, where it would be helpful (i.e. loading from a CDN like unpkg or jsdelivr).
I've submitted a PR if you're amicable to the change.


Add a toggle method

So when the user disable the touch, can collapse, expand the slide panel programatically

Given that there are multiple panel states, toggle is not a very clear action. Thank you for your contribution, but I'm not gonna merge this.


How to install it using Cocoapods ?

Hi ,
I noticed in your example you are using Cocoapods
but in ReadME file you don't mention how to install it using Cocoapods !!

The example isn't using cocoapods; it's using Carthage.
Cocoapods support is something I've thought about adding for others, but dependancies in my personal projects are managed with Carthage.
If you're familiar with Cocoapods, feel free to submit a PR that adds Cocoapods support.

After I install it using Carthage
I got this error :
 Module compiled with Swift 3.0.2 cannot be imported in Swift 3.1

Ah, yes. Swift moves pretty fast. I just created a swift-3.1 branch. Try this in your Carthage file to use the Swift 3.1 branch:
github "jyunderwood/WaveformView-iOS" "swift-3.1"

I got same error =(

After you've updated your Cartfile with the above, you need to update your builds. Also, to make sure you are compiling from source, pass --no-use-binaries.
carthage update --platform iOS --no-use-binaries

Also, make sure your command line Swift is set to the 3.1 beta: xcrun swift -version.
I'm going to close this issue because it doesn't seem to be an issue with the project, but with your build chain. If you're still running in to issues, you might want to look on StackOverflow on help using Carthage with a beta version of Xcode.


refactor: upgrade go version to 1.18.4

This PR updates the go to 1.18.4
related to: https://github.com/jenkins-x/jx/issues/8283

[x] Change the version in go.mod file.
[x] Run go mod tidy and ensure that it does not error out.
[x] Change GO_VERSION in makefile to the version you are upgrading.
[x] Check if there are any references to an older version and replace those.
[x] Verify that

[x] The build works by running make build.
[x] Tests are working, by running make test.
[x] Linting checks are working by running make lint.

/cc ankitm123

WARN [runner] The linter 'interfacer' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.
WARN [linters context] bodyclose is disabled because of go1.18. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649.


outreachy tasks

hello @pgleeson sir
As you suggested, I've now consolidated all of my tasks into a single pull request. If you can check this and let me know if it's okay and provide feedback, I'll close my other pull requests.
Thank you.

hello @pgleeson sir
Before committing, I pulled all of the changes from the branch and created this pull request. Could you please elaborate on what is expected of me as I am a little confused?

@Anushka-GA Have a look here: https://github.com/Anushka-GA/MDF/network. The latest changes from the development branch of mdf haven't been merged into your initial task branch. Please merge these in, and test your code with the latest mdf, and resolve any conflicts, e.g. CONTRIBUTING.md.
You also have 3 other PRs open. Close these if they're not relevant.
It's important for me to be able to run your code/notebooks to evaluate your contributions, so I need to be sure yoiu have the latest version of the main mdf code. Also I don't want to spend time in a PR if it's no longer relevant.
I did see in one of your notebooks, you are loading data from google drive. That wouldn't be possible for me to test locally in that case...


Why?

It's all in the linked Wikipedia article.

I see. Thanks for the explanation. IMHO it's kinda weird to have a package with a single-line function.

Easier than having to remember it in your head.

@endel The alternative is remembering how this work, which is insane as the check makes no sense when looking at it without having read the ECMAScript spec carefully. Copy pasting is an alternative, but that's not very DRY and will require you to include a comment anyways on what it does. Think of it as lego blocks. We make ready made blocks so you and other don't have to care about how it works internally and what it's supposed to do. I know this is a big shift in thinking for many coming from the front-end or other languages, but try it out, it's absolutely amazing.


Allow for users to omit certain data values from packet captures

Currently sensitive information is likely to be captured in the packets. Usually the exact values of this data isn't relevant to debugging a networking problem, instead it's the presence or absence of the data. It'd be good to provide a way for users to "clean" the packet capture of the secret values without compromising on being able to load them into Wireshark.

Resource for how to decrypt TLS packets manually:
https://lowleveldesign.org/2016/03/09/manually-decrypting-https-request/
Using scapy might be a good idea here: https://scapy.net/

Some things that may need to be scrubbed:

IP addresses
Domain names in DNS
Hosts in HTTP
Hosts in SNI
Hosts in certificates
Authorization HTTP header

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @bodd1593, @esneko, @lovelly, @angristan, @taco-c, @cartime99, @liyh12, @Ksuksu3, @tongcan, @geanyinabottle69, @DanteLorenzo, @lpsm-dev, @fbraz3, @samsheff, @THtianhao, @seekaddo, @nj-steve, @BlackJack0, @DrPep, @hugmatj


feat: expose email and sms APIs as services to SDK

Deployed at: https://door.chromium.link
admin 123456
Now, programs like Casnode can use Casdoor to send Emails and short messages.
You can send a request in postman like this to test the API:
POST: https://door.chromium.link/api/send-email?clientId=cdf67a8571cd13a62bcf&clientSecret=98e21c4e25153c556696dd21ca2ff9a23c642751
POST body:
{
    "title":"Email Test",
    "content":"Email Test Content",
    "receivers":["shiftregister233<EMAIL_ADDRESS>    "sender":"Email Test Sender"
}

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


Fix Splash Page font on Windows

Current splash page logo on Windows looks hideous since it defaults to some Serif font. We should switch the splash to an image for consistency, or make sure that we always have a somewhat nice sans serif to fall back to

Closed by #409


ci build test

ci test please

ci build test

ci build please

ci build please

ci build please

ci build please


Add licensing statement to title page

Copy licensing statement from README.md and add it to title page of book. Inspired by a comment by Stefan Münnich at Verovio tutorial at Music Encoding Conference 2021. Related to pull requests #18 and #19.

Merged, thanks!


interactive hyperdrive: add ability to adjust block mining timeframe

in the setup config we want to be able to set the time-between-blocks between "None" meaning mine on every transaction to "n" seconds. We will adjust this after we write out a plan.

Added in https://github.com/delvtech/agent0/pull/1242


Can't find extension on gitpod

Somehow I can't find this extension when searching for it in the extensions tab in VSCode for https://gitpod.io
Is there a way to work around this? By maybe getting a snippet file of all the snippets in this extension while we hopefully wait for a variation of the extension that works for gitpod.io ?

I do not know the tool but I have zero control and experience on gitpod.io


Validation tool allows incorrect R group for in-line SMILES

The HELM string:
PEPTIDE1{A.[[*:4]C(=O)[C@H](C)N([*:1])C].A}$$$$V2.0
Is invalid since the connection points are not consecutively numbered. However, the validation tool states that it is fine. The validation tool should not allow non-consecutively numbered connection points.

Update, the validation tool doesn't check that connection points are numbered at all. The following HELM strings should be invalid as the R groups have no numbers.
PEPTIDE1{[C(=O)[C@H](C)N(*)C]}$$$$
PEPTIDE1{[C(=O)[C@H](C)N([*])C]}$$$$
but the webservices accept them.


Grafana Integration

To implement nuanced authorization controls for Grafana, we need the id_token or access_token to contain information about the user that we can use to determine the user's access. Within Grafana, we have three roles: Admin, Editor, and Viewer. Grafana accepts a JMESPath expression to interrogate the ID Token returned by VEDA Auth Central and to determine which role (if any) the user will be granted.  This pattern makes for great flexibility in adapting to the format of VEDA Auth Central (ie we can format the ID token in any way we see fit).
There's a larger question around how we want to manage permissions associated with users within VEDA Auth Central. My first thought is that we approach this via Scopes -> Roles -> Groups -> Users.
Regarding scopes, I recommend that we use the following three client scopes:

grafana:viewer
grafana:editor
grafana:admin

For roles, we should follow a similar pattern:

GrafanaViewer (has grafana:viewer scope)
GrafanaEditor (has grafana:editor scope)
GrafanaAdmin (has grafana:admin scope)

For groups, this could be a bit more general:

SystemAdmin contains GrafanaAdmin
SystemOperator contains GrafanaEditor
SystemViewer contains GrafanaViewer

We would then use the following JMES path (playground):
contains(scopes, 'grafana:admin') && 'Admin' || contains(scopes, 'grafana:editor') && 'Editor' || contains(scopes, 'grafana:viewer') && 'Viewer'

We will set the role_attribute_strict flag to true in order to deny access from any users without membership to one of these groups:

Set to true to deny user login if the Grafana org role cannot be extracted using role_attribute_path or org_mapping. For more information on user role mapping, refer to Configure role mapping.]

Closed in favor of #65


no members matching ‘Eigen::TriangularView<const Eigen::SparseMatrix<double, 0, int>,

when i use the ospqeigen in ros,to solve qp problem,it occured error
       Eigen::SparseMatrix<double> hh;     hh.resize(6,6);     hh.insert(0,0) = 1;     hh.insert(1,0) = 1; ....     hh.insert(1,5) = 1;     hh.insert(2,5) = 1;     hh.insert(3,5) = 1;     hh.insert(4,5) = 1;     hh.insert(5,5) = 1;     solver.data()->setHessianMatrix(hh);     solver.data()->setGradient(gradient);     solver.data()->setLinearConstraintsMatrix(linearMatrix);     solver.data()->setLowerBound(lowerBound);     solver.data()->setUpperBound(upperBound);
when i compile it ,it will take a error such as:
[ 96%] Built target vehicle_info_msg_generate_messages In file included from /usr/local/include/eigen3/Eigen/Core:328,                  from /usr/local/include/eigen3/Eigen/Dense:1,                  from /home/skelon/demo/MPC_ros/src/mpc_fid/src/mpc_ros.cpp:7: /usr/local/include/eigen3/Eigen/src/Core/TriangularMatrix.h: In instantiation of ‘class Eigen::TriangularView<const Eigen::SparseMatrix<double, 0, int>, 2>’: /usr/local/include/OsqpEigen/Data.tpp:38:13:   required from ‘bool OsqpEigen::Data::setHessianMatrix(const Eigen::SparseCompressedBase<Derived>&) [with Derived = Eigen::SparseMatrix<double, 0, int>]’ /home/skelon/demo/MPC_ros/src/mpc_fid/src/mpc_ros.cpp:608:39:   required from here /usr/local/include/eigen3/Eigen/src/Core/TriangularMatrix.h:295:17: error: no members matching ‘Eigen::TriangularView<const Eigen::SparseMatrix<double, 0, int>, 2>::Base {aka Eigen::TriangularViewImpl<const Eigen::SparseMatrix<double, 0, int>, 2, Eigen::Sparse>}::solve’ in ‘Eigen::TriangularView<const Eigen::SparseMatrix<double, 0, int>, 2>::Base’ {aka ‘class Eigen::TriangularViewImpl<const Eigen::SparseMatrix<double, 0, int>, 2, Eigen::Sparse>’}   295 |     using Base::solve;
when i commented out the code
solver.data()->setHessianMatrix(hh);
it whill compile ok.

Hi @SkelonChan, did you solved the problem? If that is the case, it may be convenient if you share your solution, to avoid situations such as https://xkcd.com/979/ . :D

the reason is  Header file conflict,too difficult to find it

Ok, thanks anyway!

the reason is Header file conflict,too difficult to find it

I'm also having the same issue, where exactly is the conflict happening?

the reason is Header file conflict,too difficult to find it

I'm also having the same issue, where exactly is the conflict happening?

@Bzdss6 I suggest to open a new issue with a clear description of your setup. Commenting a closed issue is not great for visibility.


Add extension tutorial

This PR adds a tutorial of how to create your own trainer extension.
Related to https://github.com/chainer/chainer/issues/2576
TODO:

[x] Finish the chapter of @make_extension decorator
[x] Write a chapter for a class inherited from training.Extension

@bkvogel Thank you for the review. I resolved all points you pointed out, so please check this PR again and merge! :)

Sorry for the delay in approving. LGTM!

Should this PR be backported?


Use hostnames in inventory and variable files

This PR implements the support for using hostnames instead of IP addresses in the Ansible inventory file, leading to better scalability and portability.
I also added a couple of other changes:

Remove an extra space from Vagrant output.
Fix the ansible_ssh_connection_ip_v4_address variable assignment (we had extra quotes).
Move the Look up docker cgroup driver task up so we can use the corresponding fact earlier.
Add some more variables to the Print debug information task.

Close #60

@StefanoSalsano requested your review again, I fixed an issue with the build


custom fileds with fields permissions support

first of all, thank you so much for develop this useful plugin!
however, our work team force me to use fields permissions to behave our developers.
here is the screenshot of the problem.

Is there any possibilities to solve this problems?
Thank you again for reading this issue!

Hello @xuebowen09
Thank you for your kind message !
If the custom field has the "read only" parameter sets you can't modify this field by clicking on the "edit" button that's why you can't modify it using the plugin too. There's no way to dynamically edit a field you can't modify because of some administration settings (it will be the same on the specific edit page of the issue for example).
If you change the "read-only" permission attribute does it resolve your problem ? I've also implemented a configuration file where you can exclude some fields ( try looking at issue #41 ) maybe this feature could help you ?


Self update feature error on x86_64-pc-windows-gnu

Hi,
I was trying to build main with x86_64-pc-windows-gnu. This line gives the (summarized) error
error[E0432]: unresolved import `crate::command_config_symlinks::run_command_config_symlinks`

Self updating on Windows is only available with windowsstore feature.

Yeah, on Windows the only option that is really supported right now is msvc and then without any of the self update features, those just generally don't work on Windows (yet).

See https://github.com/JuliaLang/juliaup/pull/346.


Uptime monitoring

https://apex.sh/ping/ will always be a more premium service but we can provide the basics here.

[ ] add
[ ] document

What about integrating ping for up pro subscribers rather than implementing the same thing under up? Might save you some work and give ping some more visibility as it’s a great service.
IMO it would be pretty slick for up to create a check in ping for your production /ping route automatically.

They're using different billing systems at the moment so it'd be a bit tricky to implement, would be nice though if I had set things up differently haha

They're using different billing systems at the moment so it'd be a bit tricky to implement, would be nice though if I had set things up differently haha


substring search is case sensitive

substring search is case sensitive
This actually makes it harder to find groups than the "old" way

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

ID: 6864fd99-6f1f-a4db-931e-b5dfc915fec8
Version Independent ID: 76c2f17c-011d-c0a5-b163-7399cce97701
Content: Search and filter groups members and owners (preview) - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/groups-members-owners-search.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@grzegorzwierzbicki Thank you for your feedback . We will investigate and update this thread.

@curtand Could you please look into this issue and share your thoughts on this.

Hi @grzegorzwierzbicki
I updated the article to describe the behavior. Thanks for your contribution!
#please-close


NBCNEWS falls back to generic extractor

URL: http://www.nbcnews.com/business/autos/volkswagen-11-million-vehicles-could-have-suspect-software-emissions-scandal-n431456
C:>youtube-dl -v "http://www.nbcnews.com/business/autos/volkswagen-11-million-v
ehicles-could-have-suspect-software-emissions-scandal-n431456"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.nbcnews.com/business/autos/volks
wagen-11-million-vehicles-could-have-suspect-software-emissions-scandal-n431456'
]
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.09.09
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-71346-gdf4fca2
[debug] Proxy map: {}
[generic] volkswagen-11-million-vehicles-could-have-suspect-software-emissions-s
candal-n431456: Requesting header
WARNING: Falling back on generic information extractor.
[generic] volkswagen-11-million-vehicles-could-have-suspect-software-emissions-s
candal-n431456: Downloading webpage
[generic] volkswagen-11-million-vehicles-could-have-suspect-software-emissions-s
candal-n431456: Extracting information
ERROR: Unsupported URL: http://www.nbcnews.com/business/autos/volkswagen-11-mill
ion-vehicles-could-have-suspect-software-emissions-scandal-n431456
Traceback (most recent call last):
File "youtube_dl\extractor\generic.pyo", line 1222, in _real_extract
File "youtube_dl\utils.pyo", line 1656, in parse_xml
File "xml\etree\ElementTree.pyo", line 1300, in XML
File "xml\etree\ElementTree.pyo", line 1642, in feed
File "xml\etree\ElementTree.pyo", line 1506, in _raiseerror
ParseError: syntax error: line 1, column 0
Traceback (most recent call last):
File "youtube_dl\YoutubeDL.pyo", line 660, in extract_info
File "youtube_dl\extractor\common.pyo", line 287, in extract
File "youtube_dl\extractor\generic.pyo", line 1820, in _real_extract
UnsupportedError: Unsupported URL: http://www.nbcnews.com/business/autos/volkswa
gen-11-million-vehicles-could-have-suspect-software-emissions-scandal-n431456
Thanks
Ringo

Going through the NBCNewsIE class, the regex for valid url is:
(?x)https?://(?:www.)?nbcnews.com/(?:video/.+?/(?P\d+)|(?:watch|feature|nightly-news)/[^/]+/(?P.+))
If I am right, it only seems to accept pages from sections video, watch, feature and nightly-news.
On testing,
nightly-news -> is working
watch -> not sure if NBC News has a watch section now
feature -> I get a different error, "unable to extract bootstrap json"
video -> Looks like format of the url has changed. The id is now NOT separated by /.
For this issue, adding business to the matcher causes the program to use the correct extractor, but the problem of "unable to extract bootstrap json" comes up again.


[FEATURE] - Port spotInstance feature from furyctl-next branch

Since we need this feature on some clusters, it's better to release a new version now with the functionality.
Plus hotfix for vpc-and-vpn module, to add us-east-1 ami.

I will release the v0.7.0 version, thank you