:memo: Fix: Bad import in docs

Description
Fixes https://github.com/evmts/tevm-monorepo/issues/968
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

#969  👈
main

This stack of pull requests is managed by Graphite.


Adding CLUU to the list of OSes

This PR adds CLUU into the OS list in the Showcase chapter.

Thanks for the comments!
I did an update, I hope it's sufficient. If not, please comment.

Fixed!


mosh-client interactions not completely hermetic

Describe the bug
When running the mosh program in a context where $PATH is not set up perfectly for nix usage, running mosh foo.example.com fails to run, saying it can not count colors.
To Reproduce
Steps to reproduce the behavior:

Run mosh in a "clean" env without any nix paths on it: env -i /run/current-system/sw/bin/mosh foo.example.com
observe that it's unable to exec mosh-client:

Can't exec "mosh-client": No such file or directory at /nix/store/ixa7f6icfr92wxi8p1qv220b1pazgcrz-mosh-1.3.2/bin/.mosh-wrapped line 305.
Can't count colors: No such file or directory

Expected behavior
An attempt to connect to foo.example.com
Additional context
I suspect this could easily be fixed by making the wrapper that mosh installs prefix the mosh bin to $PATH.
Notify maintainers
@viric
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-darwin"`
 - host os: `Darwin 19.6.0, macOS 10.15.6`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.7`
 - channels(asf): `"darwin, nixpkgs-20.09pre241765.7bc3a08d3a4"`
warning: Nix search path entry '/nix/var/nix/profiles/per-user/root/channels' does not exist, ignoring
 - nixpkgs: `/Users/asf/.nix-defexpr/channels/nixpkgs

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: nixpkgs.mosh

As a workaround, I currently run mosh like this in iTerm:
env PATH=/run/current-system/sw/bin/:/usr/local/bin:/bin:/usr/bin mosh $$HOST$$

That seems not ideal, but it works for now (:


feat(core): allow customizing the i18n directory path

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation
Requested by @slorber on Discord. In addition to adding the i18n.path option, this PR also made an internal refactor (namely, adding context.i18nDir) to make the future feature of per-locale path customization easier.
I did not include the per-locale path customization in this PR, to make it easier to review since it's almost like a refactor (getting rid of the I18N_DIR constant).
Test Plan
Updated all existing tests
Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs

many existing packages/sites seem to use our util: getPluginI18nPath

The API for that util has changed as well, so when they upgrade, they will know.

How do we avoid annoying the existing community and plugin authors?

It's not the first time this happened😅 It used to be a quite contrived and wonky API, and now with a special localizationDir it should be more stable into the future.

The API for that util has changed as well, so when they upgrade, they will know.

This will break sites (and potentially plugins too), possible with a hard to understand error message, or fail silently to load translations.
We want users to notice and simplify the upgrade path
I'm not against this API change at all, but maybe we could just keep the old one too for retrocompatibility, and emit a clear warning?
TS overload, or the new API could have a different name

I don't really want to think about backward compatibility in beta, particularly I don't really see any high-impact plugins in that search result. The very first result in that list, https://github.com/Computerization/computerization.github.io, is a site I maintain. Literally 90% of those results are from source code forked from us, and they are unlikely to upgrade anyway. I think the actual impact is quite low.

yes agree, after looking deeper it seems fine and unlikely to be found in popular third-party plugins 👍
Let's merge then
We'll figure out how to doc/tech this later as there are other options to add anyway


my own similarity metric

Hello,
If I want to include my own custom similarity metric to the module, how do I do it?
Could you provide an example?
Thanks!

The first thing to do is to go through the steps down to running 1-Similarities.py. Take a look at the output and try to understand it.
All you need to do is somehow generate your own files in the same folder as the output of 1-Similarities.py, and the remaining scripts will pick them up.
 In the meanwhile, let me know if you can't figure it out.

Hi,
I'm working on evaluate_similarity_method function in 1-Similarities.py.
So here, you convert SMILES files into mol files with get_rdkitmols. What I can't figure out is, within the same mol d, you compute pairwise similarity (and somehow length of d=5 always). I might be wrong I just thought we will compute pairwise similarity within the pairs of the whole data set?


(Feature) Register created contracts

This PR adds the basic feature of registering the crowdsale address in a Registry contract. It depends on oraclesorg/ico#7.
There are several things needed after this, but I want to verify that this approach works.
I added a .env.example file. For this to work, you need to copy that to a .env file. This (not committing the actual env file) is standard practice, although arguably not necessary for an app that has only front. Still, I preferred to err on the side of security.
This adds an environment file that maps network ids to the address of the deployed Registry contract. This means that, before starting the app, this contract needs to be deployed. To make that setup easier during development, the address in the example is the address of the first deployed contract when you use ganache-cli in deterministic mode. I added a NPM script that starts ganache in this mode and also uses a predefined network id. I also added another NPM script that deploys the Registry contract and starts the app.
tl;dr, do this:

cp .env.example .env
In one terminal, execute npm run ganache
In other terminal, execute npm run dev

Let me know if you think this approach works.

Oh, and @igorbarinov, if we go this route, we'll need to deploy the Registry to the testnets and the mainnet, and add the proper environment variable to Netlify. Since the Registry contract is not final, I would not deploy it to the mainnet yet, but we can begin to test it in the testnets. You can give me access to the Netlify dashboard if you want, otherwise I will paste here the environment variable that needs to be added (I'll deploy the Registry to Kovan and Rinkeby and paste the addresses here).

I deployed the Registry contract in Rinkeby and Kovan and I'm going to test the Netlify preview deploy, so I modifies .gitmodules to temporarily use my fork of ico. I'll change it back after performing the tests and when that PR is merged.

Tested the Netlify deploy preview. Worked fine in both Rinkeby and Kovan.

Please deploy also on Core (https://core.poa.network) and Sokol (https://sokol.poa.network)
For Sokol, you can get free coins here https://faucet-sokol.herokuapp.com/
send me your wallet and I'll send you some test coins

I deployed the registry to Sokol and updated the .env.example and Netlify configuration. I also tested the deploy preview with Sokol and it worked.
I created an account in Core, the address is 0x8220D966abA3C328eBf32858CB3A5fe868e02167, if you send me some coins I'll deploy the registry there and test it.

@fvictorio thanks, I sent. Please check balance

Deployed in Core, worked fine, although Netlify's cache gave me some headaches.
Next step is to merge https://github.com/oraclesorg/ico/pull/7 and then update the .gitmodules in this PR.

This had several changes since it was reviewed, so please test it again and take a look at the last commits.
This should be functional now. The main thing pending is filling the columns in the list of crowdsales (right now it only shows the addresses, it should show the token name, ticker, and creation date). But I think we can show the button in production anyway, what do you think?

@fvictorio
I'm facing with this issue on the main page (Ropsten network):

It seems Metamask isn't injected into the page properly. The error happened when I tried to refresh the page. I tried in Chrome incognito mode too.

But I think we can show the button in production anyway, what do you think?

I agree with button and only addresses in production as a first step.

@vbaranov  That doesn't happen to me when I connect to Ropsten. The Registry wasn't added in that chain (I just added it), but that wouldn't cause that error. Any hints on how to reproduce it?

In my tests this happened not systematically: not every refresh caused this error.

I’ll let you know, if I’ll find the conditions of appearance of this error


sync/atomic: 1.8beta2 ARM / panic: runtime error: invalid memory address or nil pointer dereference

What version of Go are you using (go version)?
go version go1.8beta2 linux/arm (installed from official pre-compiled distribution)
What operating system and processor architecture are you using (go env)?
Raspberry Pi 3 with latest Archlinux ARM
GOARCH="arm"
GOBIN=""
GOEXE=""
GOHOSTARCH="arm"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/pierre/go"
GORACE=""
GOROOT="/home/pierre/.gimme/versions/go1.8beta2.linux.arm"
GOTOOLDIR="/home/pierre/.gimme/versions/go1.8beta2.linux.arm/pkg/tool/linux_arm"
GCCGO="gccgo"
GOARM="7"
CC="gcc"
GOGCCFLAGS="-fPIC -marm -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build204219494=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?
go test github.com/pierrre/imageserver/cache/groupcache
What did you expect to see?
Test should pass
What did you see instead?
--- FAIL: TestServer (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x4 pc=0x11c0c]

goroutine 3 [running]:
testing.tRunner.func1(0x10768400)
	/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/testing/testing.go:621 +0x238
panic(0x261e10, 0x3df9a0)
	/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/runtime/panic.go:489 +0x288
sync/atomic.addUint64(0x107a60ac, 0x1, 0x0, 0x1, 0x107a6018)
	/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/sync/atomic/64bit_arm.go:31 +0x4c
github.com/golang/groupcache.(*AtomicInt).Add(0x107a60ac, 0x1, 0x0)
	/home/pierre/go/src/github.com/golang/groupcache/groupcache.go:470 +0x2c
github.com/golang/groupcache.(*Group).Get(0x107a6000, 0x242e70, 0x1076e218, 0x295a1e, 0x4, 0x3bde38, 0x10772880, 0x8, 0x3bc101)
	/home/pierre/go/src/github.com/golang/groupcache/groupcache.go:207 +0x58
github.com/pierrre/imageserver/cache/groupcache.(*Server).Get(0x10780150, 0x10772840, 0x107277b0, 0x1079e5e8, 0x0)
	/home/pierre/go/src/github.com/pierrre/imageserver/cache/groupcache/groupcache.go:36 +0xf8
github.com/pierrre/imageserver/cache/groupcache.TestServer(0x10768400)
	/home/pierre/go/src/github.com/pierrre/imageserver/cache/groupcache/groupcache_test.go:33 +0xfc
testing.tRunner(0x10768400, 0x2a7ae0)
	/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/testing/testing.go:656 +0x88
created by testing.(*T).Run
	/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/testing/testing.go:693 +0x244
FAIL	github.com/pierrre/imageserver/cache/groupcache	0.049s

I works perfectly on my Linux amd64 laptop.
I think it's not an issue in my code.
Maybe in groupcache, but I'm not sure...
Most probably in atomic/sync.

Check issue 599
On Fri, 16 Dec 2016, 09:26 Pierre Durand<EMAIL_ADDRESS>wrote:

What version of Go are you using (go version)?
go version go1.8beta2 linux/arm (installed from official pre-compiled
distribution)
What operating system and processor architecture are you using (go env)?
Raspberry Pi 3 with latest Archlinux ARM
GOARCH="arm"
GOBIN=""
GOEXE=""
GOHOSTARCH="arm"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/pierre/go"
GORACE=""
GOROOT="/home/pierre/.gimme/versions/go1.8beta2.linux.arm"
GOTOOLDIR="/home/pierre/.gimme/versions/go1.8beta2.linux.arm/pkg/tool/linux_arm"
GCCGO="gccgo"
GOARM="7"
CC="gcc"
GOGCCFLAGS="-fPIC -marm -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build204219494=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
go test github.com/pierrre/imageserver/cache/groupcache
What did you expect to see?
Test should pass
What did you see instead?
--- FAIL: TestServer (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x4 pc=0x11c0c]
goroutine 3 [running]:
testing.tRunner.func1(0x10768400)
/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/testing/testing.go:621 +0x238
panic(0x261e10, 0x3df9a0)
/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/runtime/panic.go:489 +0x288
sync/atomic.addUint64(0x107a60ac, 0x1, 0x0, 0x1, 0x107a6018)
/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/sync/atomic/64bit_arm.go:31 +0x4cgithub.com/golang/groupcache.(*AtomicInt).Add(0x107a60ac, 0x1, 0x0)
/home/pierre/go/src/github.com/golang/groupcache/groupcache.go:470 +0x2cgithub.com/golang/groupcache.(*Group).Get(0x107a6000, 0x242e70, 0x1076e218, 0x295a1e, 0x4, 0x3bde38, 0x10772880, 0x8, 0x3bc101)
/home/pierre/go/src/github.com/golang/groupcache/groupcache.go:207 +0x58github.com/pierrre/imageserver/cache/groupcache.(*Server).Get(0x10780150, 0x10772840, 0x107277b0, 0x1079e5e8, 0x0)
/home/pierre/go/src/github.com/pierrre/imageserver/cache/groupcache/groupcache.go:36 +0xf8github.com/pierrre/imageserver/cache/groupcache.TestServer(0x10768400)
/home/pierre/go/src/github.com/pierrre/imageserver/cache/groupcache/groupcache_test.go:33 +0xfc
testing.tRunner(0x10768400, 0x2a7ae0)
/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/testing/testing.go:656 +0x88
created by testing.(*T).Run
/home/pierre/.gimme/versions/go1.8beta2.linux.arm/src/testing/testing.go:693 +0x244
FAIL	github.com/pierrre/imageserver/cache/groupcache	0.049s
I works perfectly on my Linux amd64 laptop.
I think it's not an issue in my code.
Maybe in groupcache, but I'm not sure...
Most probably in atomic/sync.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/18334, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcAyHfuBHl6iwrq6kiGsIjiXA5-yJjks5rIb6WgaJpZM4LOpB0
.

Indeed, this looks like #599.
0x107a60ac is not 8-byte aligned.
Go 1.8 is stricter about this than Go 1.7, even thought it was always a bug. We just let you get lucky sometimes in the past.
Looks like this is a bug in groupcache. Group.Stats is not aligned on 32-bit.
Fix coming.

@pierrre, update your groupcache. Should now be fixed by https://github.com/golang/groupcache/commit/d092608c065831f1984ac9d332ca5b2c95a11f57
I didn't verify on ARM, though.

@bradfitz thank you, the problem is fixed ! :smile:
Is "alignment" something I should take care of, as a "normal" developer ?

Yes. At this time if you use atomic you must manually ensure alignment.

OK thank you


How many responses have the schema defined and associated?

Let's understand how well schema are applied. Optional breakdown by response code; optional breakdown by local vs $ref

See initial report at https://github.com/postman-open-technologies/knowledge-base/blob/main/R/oas_paths_operations.md


Beta Designation

Is this code base still in Beta, as per the documentation:
NOTE: terraform-provider-fivetran is in [BETA](https://en.wikipedia.org/wiki/Software_release_life_cycle#Beta) development stage. Future versions may introduce minor changes and bug-fixes.

since the V1.0 release back in August.

Hey @marcusinthesky!
Looks like we just forget to cleanup readme file.
V1.0.0 means that we have passed beta stage and we won't introduce further breaking changes for existing resources.


Add support for textField cursorColor

Wow, nice! Are you planning any other changes? Should I hold off with publishing the next version?

Thanks!  You can publish it, nothing planned right now.  I've been working on changes for an app and have been using this widget all week; I like it a lot.  Better than other solutions I could find e.g., NumberPicker.  These pr's have been changes for things I noticed while working with SpinBox.  I'll link you the app when changes are done so you can see how I'm using it.  There are other properties for the TextField cursor (height, radius, width) that would also be easy to add but I didn't need them right now.  I wasn't sure if e.g., the cursorHeight might cause layout problems for SpinBox.  Probably not but I didn't look into it much.  I'll add tests for the caching changes when I can.

Alright, thank you for taking the time to contribute!


Increase timeout for wait for window driver response delay

Response timeout for iframes - 5 sec (leave it as it is).
Response timeout for windows - 20 sec.

@testcafe-build-bot retest


[REQUEST] A major feature is missing at OB2

While OB2 comes with some great new features, it has a major feature missing.
Missing feature: Custom line input to start from isn't available.
Example: Suppose I have a text file of 1000 lines, and I want to start it from line number 501. But OB2 is missing this feature while OB1 has the feature available.
I guess you got my point. Thanks to consider my request as a high priority :)

This is already present, it's called Skip now. You can configure it in the job options.


Replace with new service url

The headless service page url has moved.

This change is

CLA is now signed.


fixed aggregations ordering issues

@joneit @Dwaynekj

@stevewirts It might be a little cleaner to use process.ENV to set a path to our local dev hyper-analytics module or any module


output labels in CLI are not visible in PowerShell

When running a command like "relay whoami", the labels are in blue color. We heard from a customer running in PowerShell with a default blue background, and the labels were very difficult to see. Their request is to use a more compatible color or no color override.

FORCE_COLOR=0 as an env variable can be used at present.


Node labels only refreshed after Swarm Manager restart

If a node's labels are updated or changed, the changes aren't reflected in Swarm manager( even after restarting the engine). The only way to reflect these changes is to restart the Swarm Manager itself.
Swarm Version: swarm/0.4.0
Node engine version: 1.8.2
Steps to Reproduce:

Update the labels on a node ( e.g add DOCKER_OPTS= --label=environment=production to /etc/default/docker)
Restart the node's engine
Check the labels of that node on Swarm manager using 'docker info'
Restart the Swarm manager container itself.
Repeat Step 3 >> now you'll see the new labels

Perhaps we should call refreshSpecs in the engine refresh loop.
Thoughts? /cc @docker/swarm-maintainers

I think it is reasonable to update refreshLoop. Since docker 1.10 supports labels configuration reload, swarm should have the ability to get new labels of all docker engines.


Added cookie consent warning to site ( expiration set to 1 day )

PR Checklist

[ ] No broken links found using link-checker.

Linkchecker
Use the following command to check the broken links.
docker run -it --rm --network=host linkchecker/linkchecker --check-extern http://<IP_ADDRESS>:4000/

Before:

After:

Please resolve conflicts

Please resolve conflicts

Done


Cannot import name 'xxx' from 'shapely'

What am I doing wrong here at the install here?
from shapely import minimum_bounding_circle

ImportError                               Traceback (most recent call last)
----> 1 from shapely import minimum_bounding_circle
ImportError: cannot import name 'minimum_bounding_circle' from 'shapely' (...)

Shapely installed fine, and I have GEOS > v8.0.0.
$ brew info geos
==> geos: stable 3.11.1 (bottled)
...

$pip show shapely
Name: Shapely
Version: 2.0.0
...

I have the geometries, but many of the algorithms presumably exported form init.py are missing. I even checked to submodules incase the docs were wrong:
import shapely
'minimum_bounding_circle' in dir(shapely)
# False

import importlib
modules = ['affinity', 'algorithms', 'coords', 'ctypes_declarations', 'errors', 'geometry', 'geos', 'impl', 'linref', 'predicates', 'speedups', 'topology']
any(['minimum_bounding_circle' in dir(importlib.import_module("shapely."+mod_str)) for mod_str in modules])
# False

I also tried down-dating to the 1.8.x version but the result was the same. Am I missing something very basic here?
Operating system
Mac OS X 10.15.7

Can you show the result of running import shapely; shapely.__version__ ?
First guess is that you are actually using 1.8 for some reason.

Also, the modules you try to import to check if those have "minimum_bounding_circle" are the modules from 1.8. If you run that code with shapely 2.0, that should raise an error about some of those modules not being found.

Ahh after digging in for about 25 minutes I found out my geos-config was not pointing things to the correct location(s). After re-linking a few things and forcibly re-installing shapely 2.0.0, it is now working...
Sorry, I'm not sure why it wasn't working before after several re-install attempts... thanks for the response though!

Good to hear it is working now!


Implement Adaptive RTO

Spectral's current retransmission model uses a fixed timeout of 1 second, and adjusts the timer to RTT*3 when missing acknowledgment ranges are detected. While acknowledgments are deliberately delayed, this approach fails to account for network variations and the inherent delays in acknowledgment processing, resulting in suboptimal retransmission behavior.

Implemented in https://github.com/cooldogedev/spectral/commit/30630bf76889603be4b41e00ae21e5f7445b0bd6


Add a workaround for Hypothesis adder menu.

This is a temporary fix to a side effect caused by the body position: relative  change that was introduced to make the ResizeSensor work again.

Reviewed, as discussed during concall. Let's merge, I will re-run my tests. This should fix https://github.com/readium/readium-js-viewer/issues/652


Mongo errors

Catch the errors (globally) and generate user friendly message.  Should distinguish user errors (4xx such as duplicate keys) from system errors (5xx).

There doesn't seem to be a global MongoDB error hook, so we need to .catch the error.  Luckily all errors have their own class.  We can change sendError #39 to test for this.
  mongo
    .updateAsync(...)
    .then(result => ,.,)
    .catch(res.sendError);


Fix grammatical errors and enhance the contact links on the 'Contact Us' page

Description

Fix Grammatical Errors:

It should be 'Got "an" Idea' instead of 'Got "a" Idea'.
It should be 'Chat "With" Us' instead of 'Chat "On" Us'.

Enhance the contact links:

Clicking on the phone number and email id should redirect to 'calling' and 'mailto'.

Screenshots

Additional information
No response
What browser are you seeing this bug on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I have read the Code Of Conduct
[X] The changes don't break the code

Go ahead!


Sync docs to readthedocs.io

PR was merged and turbiniadocs are now at https://turbinia.readthedocs.io/en/latest/user/install.html.


Regex pattern not working

Hi Antonio,
Thanks for your library. I tried it our and it's amazing. I found one issue. When I tried with the regex pattern it's not working.
phone=/^58/. When I tried with the regex like this it didn't give me the results. It's giving empty list rather providing the phone numbers starting from 58. I tried to debug and find the issue. But no luck. Could you please help me to resolved this issue.
Thanks !

Hi Manula,
Thank you for reporting the issue. I will fix the issue.
Thanks.

Hi Antonio,
Thank you very much. Waiting for the fix. Appreciate your work.
Thanks !

Hi Manula,
I have released a new version 1.0.1 to fix the use of regex pattern.
Let me know your comments.
Thank you for reporting the issue.

Hi Antonio,
I tested the new version 1.0.1. The bug resolved and now I'm able to search from regex. Many thanks for the fix and I'm highly appreciate your work.
Thanks !


colab demo is not valid

when I execute the second code block, it has error

in the mid, also have error


ci: refine artifact publishing

Summary
Releases should only include packages for which the artifact is available.
i.e included in the CI build result.

Test Plan

Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

!canary

!canary


bug fix for R Markdown

Rewrite the markdown syntax regex expr
Add a test file test/rmarkdown.md

I can't run the tests, it said cant find module loadash, but a have this module installed.

Have you tried npm install --save-dev in your local language-markdown repository?

I've looked through the latest commit, and I'm a little confused. The R Markdown spec is about the pattern {r ...}</code>, but you've chosen to allow <code>{javascript, etc.} as well. What's your reasoning behind that? R Markdown has specifications, and that gives users expectations of how their syntax will be highlighted. Widening what's accepted as valid syntax, also increases the chance of false-positives which is something I try to avoid. There are some smaller remarks I have as well, but I'd like to tackle this issue first. I applaud your enthusiasm, but this fix was primarily intended to get rid of the trailing space issue.

this doc describes the shorthand of the engine directive.

It not said its a shorthand of the engine directive, but I think they are equivalent.

I think you are right. But since it only accepts seven engines, you can use that in the regex instead of allowing any value for the engine. And since there's also only seven, that makes it acceptable to generate code to recognize them.
If you could adapt the PR so it gets rid of the trailing space issue, and only allow those seven languages (+r), and keep the rest of the changes as limited as possible so the tests don't break, then I think that's best for now. I can merge those changes and release them. If you like to keep working on even better support after that, you are very welcome. Any luck with the tests yet?

I think the R markdown should not only accepts seven engines as shorthand. It seems there is no other markdown specifications use the pattern like ```{foobar}, so it's not a problem. I can add two regex for both condition, so if one day this package would support more than one markdown specifications that use the same pattern, fix would be very easy.

The test suit itself runes well, since the test case goes down... I'm fixing that

Since the R Markdown spec specifies these seven engines, we'll follow that spec. I don't intend to create a new unsupported feature. That's the whole reason why there's no consensus on a single Markdown spec.
Again, this PR only needs to fix the trailing space, and accept the selection of supported languages, and then it's good for now. I don't want to limit your contribution or dampen your spirit, I appreciate your effort. Let's do this fix, and then we can discuss additional changes via a different PR.
Simple PR's make it easy for me to see what's changed, what it has fixed, and make for quick merges. If we overcomplicate this, there's a good chance we'll have a difference of opinion at some point, and all the work would be wasted. Agreed?

OK

Hey @4Oranges,
Are you still interested in picking this up?

Yeah. I'sorry I met some trouble in my company and still not get it done,
and as a result I don't have much time to deal with this issue. I'll try to
fix it in one or two weeks. I'm sorry for such an inconvenience.
On Tuesday, October 18, 2016, David van Gemeren<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Hey @4Oranges https://github.com/4Oranges,
Are you still interested in picking this up?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/burodepeper/language-markdown/pull/139#issuecomment-254475805,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHg8IpsY5wto0f8USaPoXmR9nxTcPteDks5q1KfigaJpZM4KSBYm
.

No pressure, absolutely not! Was just curious. Let me know if you can't get around to it, then I'll look at it myself.

I'm writing and testing the regex on regex101.  I'm not sure which part went wrong. Please review this commit to resolve this.

Thanks! My instinct says that we could omit the three failing tests 4b, 5d and 9a. I've taken a quick look, and they're all annotated because of some very specific reason. I'll take a closer look myself later this week. :+1:

And again, I'm really happy with your contributions!

No, I had test the changes in atom, just leave the 'grammars/language-markdown.json' behind the commit.
The '[ ]' and '\n' are all aims to the situation that the input string contains newlines. They're not necessary if the parser works as it wished to be. I'll fix this later
I'm not sure the r markdown spec allows extra spaces, but I think we should support it, for users who has lots of rmarkdown not well formeted ,such as the extra spaces,  could switch to this package and need few changes with their old code.
 I'll migrate the cases on regex101 into the test suit soon or later.  Before that I have to spend some time to get familiar with the test suit in atom.

I'm not sure the r markdown spec allows extra spaces, but I think we should support it, for users who has lots of rmarkdown not well formeted

If their syntax is not correctly formatted, it should not be highlighted as valid. Syntax highlighting is a tool to help find mistakes in your syntax.
@pimentel You might be able to help us here. Do you know, or can you verify for us, if an R Markdown code-block that starts with something like this ( ```{js nolabel= "nolabel", engine="first"  ,   engine ="second"   , engine="third"} ) is valid? We are particularly interested in the extra spaces between arguments, and before and after the =. The content is of the dummy variety of course, but I think you get the idea.
@4Oranges Have you installed the language-ass package? That should give you syntax-highlighting for .ass files, and might clear things up a little. I'll explain it in more detail of course if you require so.


HOW CAN I INSTALL TORCH WITH PY37 ON AGX XAVIER

it seems that no available pytorch package was given with python3.7 on jetson right now

https://forums.developer.nvidia.com/t/pytorch-for-jetson-version-1-9-0-now-available/72048

I still feel confused about should I build from source to install pytorch? or I can use the py36 wheel to build  it?
thank you!


Pilot logins but Transporter fails (-20283) wrong password

So I have this weird issue that I suspect (after reading some posts here) might be connected with special characters in my password, that is "!!" one after another.  Deliver on the other hand seems to work fine.
Issue Description
DEBUG [2017-09-01 19:15:26.31]: [Transporter]: DBG-X:   parameter Version = 1.9.3
DEBUG [2017-09-01 19:15:26.31]: [Transporter]: DBG-X:   parameter iTMSTransporterMode = upload
DEBUG [2017-09-01 19:15:26.31]: [Transporter]: INFO: id =<PHONE_NUMBER>1526-965
DEBUG [2017-09-01 19:15:26.57]: [Transporter]: DBG-X: Apple's web service operation return value:
DEBUG [2017-09-01 19:15:26.57]: [Transporter]: DBG-X:   parameter ErrorMessage = Your Apple ID or password was entered incorrectly. (-20283)
DEBUG [2017-09-01 19:15:26.57]: [Transporter]: DBG-X:   parameter Errors = [Your Apple ID or password was entered incorrectly. (-20283)]
DEBUG [2017-09-01 19:15:26.58]: [Transporter]: DBG-X:   parameter ErrorCode = -20283
DEBUG [2017-09-01 19:15:26.58]: [Transporter]: DBG-X:   parameter Success = false
ERROR [2017-09-01 19:15:26.58]: [Transporter Error Output]: Your Apple ID or password was entered incorrectly. (-20283)
Environment
🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.12.6

Ruby
2.3.1

Bundler?
false

Git
git version 2.12.2

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.3.1_2/lib

OpenSSL Version
OpenSSL 1.0.2j  26 Sep 2016

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.3.3

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
fastlane_require 'dotenv'
Dotenv.load
CredentialsManager::AppfileConfig.try_fetch_value(:app_identifier)

import "../MasterFastfile"

`./fastlane/Appfile`
app_identifier "com.****" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address
team_id "********"```

### fastlane gems

| Gem      | Version | Update-Status |
| -------- | ------- | ------------- |
| fastlane | 2.55.0  | ✅ Up-To-Date  |

### Loaded fastlane plugins:

**No plugins Loaded**

Loaded gems

| Gem                       | Version      |
| ------------------------- | ------------ |
| did_you_mean              | 1.0.0        |
| slack-notifier            | 1.5.1        |
| CFPropertyList            | 2.3.4        |
| claide                    | 1.0.2        |
| colored2                  | 3.1.2        |
| nanaimo                   | 0.2.3        |
| xcodeproj                 | 1.5.1        |
| rouge                     | 1.11.1       |
| xcpretty                  | 0.2.4        |
| terminal-notifier         | 1.7.1        |
| terminal-table            | 1.4.5        |
| plist                     | 3.2.0        |
| multipart-post            | 2.0.0        |
| word_wrap                 | 1.0.0        |
| public_suffix             | 2.0.5        |
| tty-screen                | 0.5.0        |
| babosa                    | 1.0.2        |
| colored                   | 1.2          |
| highline                  | 1.7.8        |
| commander-fastlane        | 4.4.5        |
| excon                     | 0.54.0       |
| faraday                   | 0.10.0       |
| unf_ext                   | <IP_ADDRESS>      |
| unf                       | 0.1.4        |
| domain_name               | 0.5.20161021 |
| http-cookie               | 1.0.3        |
| faraday-cookie_jar        | 0.0.6        |
| fastimage                 | 2.1.0        |
| gh_inspector              | 1.0.2        |
| json                      | 1.8.3        |
| mini_magick               | 4.5.1        |
| multi_json                | 1.12.1       |
| multi_xml                 | 0.5.5        |
| security                  | 0.1.3        |
| xcpretty-travis-formatter | 0.0.4        |
| dotenv                    | 2.1.1        |
| bundler                   | 1.13.7       |
| faraday_middleware        | 0.10.1       |
| uber                      | 0.0.15       |
| declarative               | 0.0.9        |
| declarative-option        | 0.1.0        |
| representable             | 3.0.4        |
| retriable                 | 2.1.0        |
| addressable               | 2.5.1        |
| mime-types-data           | 3.2016.0521  |
| mime-types                | 3.1          |
| little-plugger            | 1.1.4        |
| logging                   | 2.1.0        |
| jwt                       | 1.5.6        |
| memoist                   | 0.15.0       |
| os                        | 0.9.6        |
| signet                    | 0.7.3        |
| googleauth                | 0.5.1        |
| httpclient                | <IP_ADDRESS>      |
| google-api-client         | 0.13.1       |
| io-console                | 0.4.5        |
| rubyzip                   | 1.2.0        |

*generated on:* **2017-09-01**

Solved it! Turns out it was not related to special characters.
When I've modified the method  load_password_for_transporter to only use the third method (standard iTC password) then it worked flawlessly!
So this error seems to be related to FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD which I have setup for a different team (and different apple_id). It would be great to be able to indicate that certain user may not want to use FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD.
Also a bit disconcerting is the fact that prior to the mentioned solution, after I've inserted the right account details through console it failed as well.

That's super weird, how can we fix this?

@KrauseFx I think that we'd need to determine whether or not it is a two-factor-authentification group first. Because right now the logic goes right this: 1. try two-factor auth 2. try app-specific keychain auth 3. try the regular keychain auth.
I'm also amazed how the solutions to one problem got so many solutions... I'd propose using only keychain for everything including two-factor auth with team_id as a prefix in key name.
The current way in which two-factor auth works is not really elastic enough (and it's also kinda cumbersome to make env variables in bash) to work with many teams.
Also I don't really understand (and never used) the app-specific keychain auth. Wouldn't it make more sense to do team-specific auth in keychain in general?

I think I am facing a similar problem where I had 2FA enabled for my account and specified FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD in the environment variable. Fastlane complains that I need to specify FASTLANE_PASSWORD as to continue to login. I have given app specific password as FASTLANE_PASSWORD as well and login failed. I think if FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD is already, set we should not authenticate using Keychain or FASTLANE_PASSWORD.

I think I am facing a similar issue.
load_password_for_transporter tries to load TWO_FACTOR_ENV_VARIABLE (FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD) before attempting any other method of login. I am not quite sure why I was prompted to enter Apple ID password though  FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD is already setup.
In addition, it also requires FASTLANE_PASSWORD to attempt login (if not present in keychain). It should ignore everything else if two factor environment variables are available.

Thanks everyone for chiming in here. We have seen issues with the transporter a few times. Is there anyone that would want to submit a PR for this and beat us to the punch? 🚀


All TITLES should alaign left

@valmirmustafa That should already be the case. On which page(s) do you see titles that are not aligned correctly ? Thanks.

On registration, but maybe they are left aligned.
When I try different screen sizes, it starts to minimize the fonts, which to some of them stays justified, then goes to left align.


Spell Quickslots selection hidden.

Describe the bug
A clear and concise description of what the bug is.
you cant select any characters spell slots anymore (they are just not there/dont exist) that should have them they suddenly dont and if i select fighter and then select mage, then mage inherit fighter ability and quickslot tabs.
i also cant make my cantrips to be set as used at all times (rightclick) does nothing otherwise it ues skills normaly.
To Reproduce
Steps to reproduce the behavior:
updated 1.1.2 to 1.3.6 table top tweaks
updated 1.3.7 to  1.3.14 toybox (not the reason why its crashing already tested all other mods i have)
Expected behavior
A clear and concise description of what you expected to happen.
to assign a spell from spell quickslot bag to a skil bar for a quick use
.

Can you upload a save file where this is taking place? As far as I can reproduce this is no longer an issue

This sounds exactly like what was happening to me here: https://github.com/Vek17/WrathMods-TabletopTweaks/issues/145
Followed this and it worked.

And you are 100% on 1.3.6? Because this was fixed in 1.3.6


Add incompatible_remove_native_http_archive option to build

Problem
I'm building a TensorFlow binary wheel for my specific hardware. When I run the process I get the following exception:
ERROR: error loading package '': Encountered error while reading extension file 'closure/defs.bzl': no such package '@io_bazel_rules_closure//closure': The native http_archive rule is deprecated. load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive") for a drop-in replacement.
Use --incompatible_remove_native_http_archive=false to temporarily continue using the native rule.

Approach
I updated build.sh to include the suggested bazel flag
Tested Environment
I ran into the issue and tested out a solution on a VM with the following CPU specs:
processor	: 7
vendor_id	: GenuineIntel
cpu family	: 15
model		: 6
model name	: Common KVM processor
stepping	: 1
microcode	: 0x1
cpu MHz		: 2399.996
cache size	: 4096 KB
physical id	: 7
siblings	: 1
core id		: 0
cpu cores	: 1
apicid		: 7
initial apicid	: 7
fpu		: yes
fpu_exception	: yes
cpuid level	: 5
wp		: yes
flags		: fpu de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 syscall nx lm constant_tsc nopl pni cx16 hypervisor kaiser
bugs		: cpu_meltdown spectre_v1 spectre_v2 spec_store_bypass l1tf
bogomips	: 4799.99
clflush size	: 64
cache_alignment	: 128
address sizes	: 46 bits physical, 48 bits virtual
power management:

I've run into this with Tensorflow version 1.9 as well as 1.12.
I've only used the no GPU configuration so I don't know if it occurs there.
I've tried it with Python 3.5 and 3.6

Thank you @hadim - this is such a useful process. I don't really understand why my build generates the exception when others haven't seen it.

Could you also add the flag to the CentOS build and also respect the indentation?

respect the indentation

The script uses both tabs and spaces for indentation, it looks fine if your tab width is 2. I think it's better to use either tabs or spaces exclusively. I personally prefer spaces.

Pretty sure this is caused by a Bazel version that is too recent (source: https://github.com/tensorflow/tensorflow/issues/24385).
This project now requires users to set Bazel version explicitly: https://github.com/hadim/docker-tensorflow-builder/commit/b4979fd307b08b424b8262bac42daab2a13f5770, making this issue a matter of setting the correct version.
I suggest closing this PR.


Prometheus 2.0: remote storage being full caused prometheus behavior-ed incorrectly

What did you do?
We are doing prometheus monitoring using grafana, consul and remote storage. While cratedb disk has been full and after we cleaned cratedb disk, we found that prometheus behavior-ed incorrectly.
grafana <- prometheus-> cratedb adapter -> cratedb , and  prometheus uses consul service discovery.
What did you expect to see?
We think that , while  cratedb disk has been full and  after we cleaned cratedb disk,

prometheus/consul should not be affected
prometheus remote write/read should recover
prometheus local storage should not be affected
grafana should be not affected

What did you see instead? Under which circumstances?
Instead, we saw the following,

prometheus/consul happened the following error(please seeing logs in details) :
... msg="Error refreshing service" ...
prometheus remote write/read did not recover
prometheus local storage has been affected
grafana has been affected and can not display data

Environment

System information:
  Linux 2.6.32-279.el6.x86_64 x86_64

Prometheus version:

Version: 2.0.0
Revision: 0a74f98628a0463dddc90528220c94de5032d1a0
Branch: HEAD
BuildUser: root@615b82cb36b6
BuildDate: 20171108-07:11:59
GoVersion: go1.9.2

Alertmanager version:

version: 0.12.0

Prometheus configuration file:

global:
scrape_interval: 15s
scrape_timeout: 10s
evaluation_interval: 15s
external_labels:
prometheusName: NJXZ-P1
alerting:
alertmanagers:

static_configs:

targets:

xx.yy.zz.kk:9093
scheme: http
timeout: 10s
rule_files:

/opt/prometheus-2.0.0.linux-amd64/rules.yml
scrape_configs:
job_name: consul_sd_configs
scrape_interval: 15s
scrape_timeout: 10s
metrics_path: /metrics
scheme: http
consul_sd_configs:

server:  xx.yy.zz.kk:9996
tag_separator: ','
scheme: http
services:

promether-exporter
relabel_configs:

source_labels: [__meta_consul_service]
separator: ;
regex: (.*)
target_label: job
replacement: $1
action: replace
source_labels: [__meta_consul_node]
separator: ;
regex: (.*)
target_label: instance
replacement: $1
action: replace
source_labels: [__meta_consul_tags]
separator: ;
regex: ',(.),(.),(.),(.),(.*),'
target_label: appId
replacement: $1
action: replace
source_labels: [__meta_consul_tags]
separator: ;
regex: ',(.),(.),(.),(.),(.*),'
target_label: ldc
replacement: $2
action: replace
source_labels: [__meta_consul_tags]
separator: ;
regex: ',(.),(.),(.),(.),(.*),'
target_label: env
replacement: $3
action: replace
source_labels: [__meta_consul_tags]
separator: ;
regex: ',(.),(.),(.),(.),(.*),'
target_label: ip
replacement: $4
action: replace
source_labels: [__meta_consul_tags]
separator: ;
regex: ',(.),(.),(.),(.),(.*),'
target_label: software
replacement: $5
action: replace

job_name: CTDSA_NJXZ_SIT_ xx.yy.zz.kk_JBOSSSERVER
scrape_interval: 15s
scrape_timeout: 10s
metrics_path: /metrics
scheme: http
static_configs:

targets:

xx.yy.zz.kk:9100
labels:
appId: CTDSA
env: SIT
ip:  xx.yy.zz.kk
ldc: NJXZ
software: JBOSSSERVER

targets: []
labels:
appId: CTDSA
env: SIT
ip:  xx.yy.zz.kk
ldc: NJXZ
software: JBOSSSERVER

job_name: CTDSA_NJXZ_SIT_ xx.yy.zz.kk_pushgateway
scrape_interval: 15s
scrape_timeout: 10s
metrics_path: /metrics
scheme: http
static_configs:

targets:

xx.yy.zz.kk:9091
labels:
appId: CTDSA
env: SIT
ip:  xx.yy.zz.kk
ldc: NJXZ
software: pushgateway

targets: []
labels:
appId: CTDSA
env: SIT
ip:  xx.yy.zz.kk
ldc: NJXZ
software: pushgateway

Alertmanager configuration file:

I felt alertmanager is not relevant to the issue, so please allow me ignore it.

Logs:

level=info ts=2018-02-05T02:09:14.673576977Z caller=queue_manager.go:338 component=remote msg="Currently resharding, skipping."
level=warn ts=2018-02-05T02:09:15.828729838Z caller=queue_manager.go:225 component=remote msg="Remote storage queue full, discarding sample. Multiple subsequent messages of this kind may be suppressed."
level=info ts=2018-02-05T02:09:22.132293894Z caller=queue_manager.go:253 component=remote msg="Stopping remote storage..."
level=error ts=2018-02-05T02:09:35.918461623Z caller=consul.go:283 component="target manager" discovery=consul msg="Error refreshing service" service=promether-exporter err="Get http://<IP_ADDRESS>:9996/v1/catalog/service/promether-exporter?index=116543&wait=30000ms: net/http: request canceled (Client.Timeout exceeded while awaiting headers)"

Hi could you update to the latest prometheus and see if that fixes the issue?

This should be fixed by the new WAL based remote_write code in 2.8.


Data source 'Blah.Blah' cannot be found in the test configuration settings

I have an App.config (it already contains some configuration information, but I've tried it empty also). Using VS2015 RC and latest Nuget package. I also tried the examples from your repo, but attempting to open one of the source files for the tests crashed VC2015 RC.
Cannot work out what is happening.

Okay, got the example tests to run and they pass. Cannot work out what is different in my tests that it doesn't work.

Since it clearly works for the test scenarios, I'll close this issue. You don't need to be debugging my mistakes. If I find something concrete, I will return. :smile:

OK, turns out I hadn't tried an empty app.config. I deleted all the sections out of my configuration and it started passing. I think that this may be creating an invalid app.config by assuming the config is empty. Mine has custom sections defined via configSections, startup, and runtime in there to support some of the test scenarios.

As an experiment I ran without any custom config sections in the app.config (i.e. config sections based on types defined in our codebase), and it worked. So, perhaps it has something to do with how it loads the config and not being able to resolve the custom sections?

Based off this answer StackOverflow, I think it's trying to resolve the custom type from DLLs in the folder of the test runner, not the DLL under test and so it is failing to load the app.config.
I don't know how to work around this, other than to manipulate the app.config without the ConfigurationManager - perhaps via LINQ-to-XML instead?

I am seeing the same issue. Installed with Nuget, I already have an app.config with a whole bunch of settings and it fails saying, "Data source <> cannot be found in the test configuration settings"

I am seeing the same issue. Installed with Nuget, I already have an app.config with a whole bunch of settings and it fails saying, "Data source <> cannot be found in the test configuration settings"


test: Add Windows e2e test runs to PRs and daily schedule

What type of PR is this?
/kind test

What this PR does / why we need it:
Adds Windows end-to-end test runs to Mandalay PRs and daily schedules.
Which issue(s) this PR fixes:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
179

Relevant Lines:
419

💛  - Coveralls


Remove Auto-Connection

Issue Type: Bug
Remove the auto-start of the panel from the right that asks for a connection. Make it a selection in settings.
Azure Data Studio version: Azure Data Studio 1.2.4 (1ebfd0235690ad044ed2874a02e65188f5fea01c, 2018-11-02T21:36:55.143Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz (8 x 2500)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (1.73GB free)

Process Argv
/Applications/Azure Data Studio.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (22)

Extension
Author (truncated)
Version

vscode-sqlite
ale
0.3.2

docs-markdown
doc
0.2.0

firstresponderkit
dre
0.3.0

gitlens
eam
9.0.2

vscode-reveal
evi
2.0.3

docker-explorer
for
0.1.6

gc-excelviewer
Gra
2.1.26

live-html-previewer
hdg
0.3.0

r
Iku
1.0.0

kubernetes-snippets
ipe
0.1.9

copy-markdown-as-html
jer
1.1.0

agent
Mic
0.35.0

import
Mic
0.4.0

profiler
Mic
0.3.0

server-report
Mic
0.1.3

sql-vnext
Mic
0.8.0

whoisactive
Mic
0.1.1

python
ms-
2018.11.0

sql-search
Red
0.2.1

markdown-preview-enhanced
shd
0.3.11

open-in-browser
tec
2.0.0

vscode-todo-highlight
way
1.0.4

(1 theme extensions excluded)

The Connection Dialog will only appear on startup until a user has at least 1 saved connection in the Server viewlet.  The idea is to help the user get started using the tool, since most functionality doesn't work without a connection.
@BuckWoody are you using the Azure Data Studio in a way where you don't use saved connections?


Remove Node Cache in favour of Stale While Revalidate Hooks

Caching on the frontend is hard, so let's leave some of that to GraphQL further down the line. Let's move in favour to a stale-while-revalidate hooks approach that wraps the catalogApi
Feature Suggestion

Remove node-cache
Implement something that won't cause flicking when clicking around the sidebar filters for the Catalog Table page.

Possible Implementation
Something like this:
https://www.toptal.com/react-hooks/stale-while-revalidate

Fixed in #1260


XBox One controller does not work on Raspbian using this remapping.

Hello!
I'm working on a program for Raspbian on a Raspberry Pi i4B and was hoping to use this file with SDL2 to resolve the remap needs.
But I just tested with my own controller and it does not work the mappings are still not working.
It does work if I use the SDL/test/controllermap program and the guid+remap file I get from that.
My guid was present in the SDL_GameControllerDB under 'Linux' but the remap codes were different from the ones I generated myself.
Which was a shame as I was hoping to use this repo to cover the large amount of controllers that you have here.
Any ideas of why this is happening?
Cheers
Fred

Hi Fred
Please post the map you got so we can compare with what's in the DB

Hi this is the one I generated and that works with my SDL2 program,
030000005e040000dd02000003020000,Xbox One Controller,platform:Linux,a:b0,b:b1,x:b2,y:b3,back:b8,guide:b10,start:b9,leftstick:b11,rightstick:b12,leftshoulder:b4,rightshoulder:b5,dpup:h0.1,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,leftx:a0,lefty:a1,rightx:a2,righty:a3,lefttrigger:b6,righttrigger:b7,
I'm using SDL pulled from Github, the latest in progress from a few days ago.

Here's another controller from a user I helped with this issue,
030000005e040000dd02000003020000,Xbox One Controller,platform:Linux,a:b0,b:b1,x:b2,y:b3,back:b8,guide:b10,start:b9,leftstick:b11,rightstick:b12,leftshoulder:b4,rightshoulder:b5,dpup:h0.1,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,leftx:a0,lefty:a1,rightx:a2,righty:a3,lefttrigger:b6,righttrigger:b7,
You can follow our discussion here,
https://github.com/Fredrum/chiaki/issues/14

The mapping comes from upstream, you'll have to take that up with the SDL project:
https://github.com/libsdl-org/SDL/blob/de6ba40d9e0dd829ab4db0567251005df1c1cbd8/src/joystick/SDL_gamecontrollerdb.h#L768

I have had a ticket there too since a few days ago but no reply at all so far.
If you know someone there maybe you would be able to get some attention?
https://github.com/libsdl-org/SDL/issues/4655

I am running without that and so is the other user.
I don't really want to add another piece of driver/software into the mix but thanks for the suggestion.
I also noticed this from June 30:
https://store.steampowered.com/news/app/312200/view/2988683441443402781
"Chasm uses SDL for low level functions like input handling, audio, etc. SDL's gamepad mapping file has entries for nearly every controller out there, so if you start the game up there's a good chance it will work right away. Problems began though when the community started adding new mappings for the PS4 controller and others that were incorrect. This caused the d-pad and other buttons to stop working or be mapped incorrectly. An update has just been released to remove the duplicate invalid mappings."

The problem you quoted had to do with a specific driver problem, and it's been fixed. It shouldn't overlap with whatever you're seeing here. If you're not willing to test with the software that may help us identify the problem as asked I'm not sure how you'd like me to help you.

I just installed xboxdrv using apt on the Raspberry.
The version I get is:
Package: xboxdrv
Version: 0.8.8-1
It didn't make any difference at all to the observed behaviours.

OK, thanks for clarifying Xboxdrv isn’t the issue. The next steps to resolve are:

Remove the map from the db altogether locally and see if SDL handles the device fine without a mapping.
Install Linux on a capable PC locally, and test the your device seeing if the map is correct in that context. If it is not, check the upstream commits with git blame and contact the person who submitted it to work out the problem.
Investigate what driver disparities and rules differences exist between Raspbian and a full featured desktop Linux that might be responsible for the difference.
Continue to pursue the issue upstream (SDL’s repo, forum, and or Discord) since we largely defer to upstream, and this mapping matches the rest.

Hi thanks a lot for the helping steps!
I'ts definitely not working without the mappings either as that's how I started. I have in my notes that xbox controllers were working fine before. It has been about 6 months since I tested that. And now I'm unsure if that's correct I think it is but of course I could be wrong.
It's a bummer as I just messed up Ubunto on my dual boot PC and now I just get Kernel panic! :D  I guess this is a motivation to fix it.
If I don't install any other custom xbox drivers what is the Linux driver that handles it?

I believe Xbox One controllers (depending on connectivity method) may just work with udev without any drivers or etc. Something weird about the GUID you reported is that the PID is 0x028e, which is clearly an Xbox 360 device.

Iv'e spent just a little time on this in the evening and spotted that my remapping string and his remapping string (both above) have the same button and axis bindings.
a:b0,b:b1,x:b2,y:b3,back:b8,guide:b10,start:b9,leftstick:b11,rightstick:b12,leftshoulder:b4,rightshoulder:b5,dpup:h0.1,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,leftx:a0,lefty:a1,rightx:a2,righty:a3,lefttrigger:b6,righttrigger:b7,
a:b0,b:b1,x:b2,y:b3,back:b8,guide:b10,start:b9,leftstick:b11,rightstick:b12,leftshoulder:b4,rightshoulder:b5,dpup:h0.1,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,leftx:a0,lefty:a1,rightx:a2,righty:a3,lefttrigger:b6,righttrigger:b7,
So I ran 'jstest' on my pad again to check and it's the same results as those above mappings.
This must mean something but I don't know what!?

I just re-flashed a microsd card with a 10 month old Raspbian OS image (official) and installed my software on that.
My Xbox One controller is working perfectly on that setup. So something has changed since then. I'm still looking for the precise change though.
These are some of the versions,
Setting up libsdl2-dev (2.0.9+dfsg1-1+rpt1) ...
-- Checking for module 'libudev'
--   Found libudev, version 241
5.4.51-v7l+
Iv'e noticed messages about the kernel driver 'xpad'. Not exactly for my symptoms though.

I believe the problem is to do with me using a Retropie image as my OS.
I couldn't reproduce the problem on official Raspberry OS image installs.

Xpad is a very likely culprit. Overzealous amateur Linux driver developers often ship broken or unfinished work that takes multiple kernel releases to stabilize. Very often the eventual tweaks to make sure SDL (generally the most common use case for games on Linux derivatives) handles the device differently in a “fixed” state via the driver are not tested and thus not functional for variants like Pi and Android.
If you want to solve the problem you’re seeing at the client software level you’ll likely need a condition to explicitly target the version of Retropie it’s impacting, change the mapping accordingly, and monitor future releases in case it’s reverted.
The underlying issue will obviously mean more thorough review of the changes in Retropie.

RetroPie has a custom xpad patch that switches the triggers from axes to buttons and breaks SDL gamepad mappings. I ran into this issue while testing Moonlight Embedded on RetroPie and filed https://github.com/RetroPie/RetroPie-Setup/issues/3379

thanks as always @cgutman :-]

Amazing!! :D
I can confirm that once I used the Retropie packages 'dialog' gui to remove xpad my Xbox One controller worked perfectly with my software once again, no re-mapping files needed at all. Just as I remembered it.
I'm super happy to have the solution!
Thanks a lot @offalynne and @cgutman !!
Cheers!


Crash on startup (probably)

Idk if it is the same thing as previous issue with syntra connector, but it also doesn't work on fabric (minecraft closes before starting). Here is the mod list:

derelict-1.1.0
fabric-api-0.92.2
fabric-language-kotlin-1.11.0+kotlin.2.0.0
owo-lib-0.11.2

All was done on fabric loader 0.15.11-1.20.1 (tested on others, same stuff happens).
And latest log mentioning derelict somewhere:
latest.log

Thank you as always! - I was doing some stuff wrong with my build process and the mixin refmap didn't get generated properly. Should be fixed in the now released 1.1.1 version: https://modrinth.com/mod/derelict/version/1.1.1


Migrate to null-safety

Any planning to migrate to null-safety?

Migrated to null safety. Please create a pull request if there is any issue.


Check Strapi for CRUD pages

1: Modal to create new content type (only set name field)

2: Input with label and description

3: advanced RadioGroup component

4: Checkbox with description

5: Tabs With Title and subtitle

6: Invalid state for input

7: Invalid state for tab

8: Tooltip

9: Fullscreen Loading

Content Manager
Empty State table

Sortable tables

Card

Autocomplete

Required label

Password field

Switch

Key Value pair informations

Custom Select with divs

Confirmation Dialog

Alerts

Block sized Status

Block buttons

Grid System

Disabled button

Checkbox inside dropdown

Indeterminate state for checkbox


.add() does not order the routes in a stable way

Hi,
According to this comment in the .add() method:
https://github.com/troch/route-node/blob/master/modules/RouteNode.js#L111-L142
the order of the routes should be preserved if they are equivalent under all the conditions (same length of the last segment) and so on. However, it uses the sort method of an array, which does not ensure the sort is stable:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort
We found a weird issue where the order of the routes swapped two routes each time a new route was added to the router.
Please, let's add extra conditions to avoid returning 0 from the sort callback!
Thanks!

Perhaps replace sort with a stable sort like the one implemented at
https://stackoverflow.com/questions/1427608/fast-stable-sorting-algorithm-implementation-in-javascript
?

Is your issue with more than 10 child routes in Chrome (V8)?

Yes, I have more than 10 routes and V8 (but on NodeJS).

Sort is unstable on V8 for arrays with more than 10 items. Can you make a PR to not return 0 and instead return the index difference?


Refused to execute script ("taffy.js" on Chrome)

I see this error on Google Chrome:
Refused to execute script from
'https://github.com/typicaljoe/taffydb/raw/master/taffy.js'
because its MIME type ('text/plain') is not executable, and strict MIME type checking
is enabled.

Uncaught ReferenceError: TAFFY is not defined

using this import:
<script src="https://github.com/typicaljoe/taffydb/raw/master/taffy.js"></script>

https://github.com/rognoni/adaptable/blob/master/Front-end/HTML5-Bootstrap4/index.html

A possible simple solution :-)
https://rawgit.com

This looks like a browser setting.  Did you try this?
<script type="text/javascript" src="...">

Yes, probably the content-type checks is made from the resource headers:
  <!-- Libraries -->
  <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js"></script>
  <script src="https://cdn.rawgit.com/twbs/bootstrap/v4-dev/dist/js/bootstrap.js"></script>
  <script type="text/javascript" src="https://github.com/typicaljoe/taffydb/raw/master/taffy.js"></script>


Fix hover issue with actions column on modules list

The following changes are implemented
TODO: Summary
Changes in the user interface:
TODO: Add screenshots, recordings or remove this section
Checklist when submitting a final (!draft) PR

[x] Commits are tidied up, squashed if needed and follow guidelines in CONTRIBUTING.md
[x] Code builds
[x] All existing tests pass
[x] All new critical code is covered by tests
[x] PR is linked to the relevant issue(s)
[x] Rebased with the target branch

My bad. LGTM


solve the "three-eyes" problem

What
Find a solution to the "three-eyes" problem we currently have that requires a 3rd developer to approve PRs in the vast majority of circumstances.
Why
Because it is slowing pairing developers down having to ask for approval, and may be causing us to take less care in what we are approving ("I'll swap you an approval")
Context
We currently have to require 2x approvers on PRs ... this is a workaround where if set to 1x approver, the possibility that a PR by dev1 can be commandeered by dev2 who contributes malicious code and then approves their own changes to get code out.
Ideas:

Have a slack bot you can ask to raise PRs for you, letting us approve our own PRs
Have a git bot that automagically raises PRs for every branch that matches a special pattern (ie a prefix like feature-), letting us approve our own PRs
count comments like "LGTM" by trusted devs as approvals that also match the same commit
...?

If we don't do this

Pairs will be unable to get code out on their own
Out-of-hours support changes would require waking up a third person!
More things will pile up in review
Approvals may get considered a chore and not a serious part of code review

@paroxp made this comment on the original trello card:  "If the idea is indeed for us to share the repositories and project across RE, then perhaps having "three" eyes, for pure knowledge sharing is exactly what we want?
Also, probably won't be that big of an issue where the team has only two people."

We've solved this problem for this repo with this story. Given all the reasons you outlined above, IA agreed to let us go back down to 1 approval for alphagov/gsp. The mitigations we have now are that the last commit on a PR has to be GPG signed by the person who opened the PR.
To enforce this, our Concourse GitHub resource trusts GitHub to give us all the information about a commit, its PR, its signatures and its reviewers, and checks that the commit signature matches the person who authored the PR. This means that another of the pair could commit things still, but the last commit has to be the person who started it, and so they'll be aware of the code that's been pushed before.
To ease the burden on reviewers to check this themselves, we've also made a Concourse CI pipeline to flag this up before it gets merged to master and breaks the release pipeline for everyone.
All of Verify's repos (including cluster-config ones) stay as 2 approvals as that's their IA rules.
Given the above, I'm closing this. :raised_hands:


5.0.0 changes

All changes for the alert categorization task

I'll merge this PR and other 2 PRs of alert categorization (for configuration UI and upgrade steps) few hours before the release.


No such file or directory: 'Vanilla_Resource_Pack/textures/blocks/grindstone_side.png

no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for chest found
no rotation for chest found
no rotation for barrel found
no rotation for barrel found
no rotation for chest found
no rotation for fence_gate found
no rotation for chest found
no rotation for fence_gate found
no rotation for barrel found
no rotation for barrel found
no rotation for chest found
no rotation for chest found
no rotation for barrel found
no rotation for barrel found
no rotation for chest found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for barrel found
no rotation for wall_sign found
no rotation for wall_sign found
no rotation for wall_sign found
no rotation for carved_pumpkin found
no rotation for pumpkin found
no rotation for wall_sign found
no rotation for wall_sign found
no rotation for pumpkin found
no rotation for carved_pumpkin found
no rotation for wall_sign found
no rotation for wall_sign found
no rotation for wall_sign found
no rotation for barrel found
no rotation for barrel found
no rotation for wall_banner found
no rotation for barrel found
Exception in Tkinter callback
Traceback (most recent call last):
File "structura.py", line 96, in generate_pack
File "armor_stand_class.py", line 74, in make_block
File "armor_stand_class.py", line 337, in block_name_to_uv
File "armor_stand_class.py", line 302, in extend_uv_image
File "PIL\Image.py", line 2891, in open
FileNotFoundError: [Errno 2] No such file or directory: 'Vanilla_Resource_Pack/textures/blocks/grindstone_side.png'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tkinter_init_.py", line 1883, in call
File "structura.py", line 161, in runFromGui
File "structura.py", line 99, in generate_pack
File "armor_stand_class.py", line 74, in make_block
File "armor_stand_class.py", line 337, in block_name_to_uv
File "armor_stand_class.py", line 302, in extend_uv_image
File "PIL\Image.py", line 2891, in open
FileNotFoundError: [Errno 2] No such file or directory: 'Vanilla_Resource_Pack/textures/blocks/grindstone_side.png'

you are using an old version of structura. this is updated in the latest


CX TruffleHog_HighEntropy_Strings @ packages/node/base/package.json [master]

TruffleHog_HighEntropy_Strings issue exists @ packages/node/base/package.json in branch master
The application uses the hard-coded password e4a13cfd135ec766dc9148ba4fe4d3ac76d94137 for authentication purposes, either using it to verify users' identities, or to access another remote system. This password at line 159 of packages\node\base\package.json appears in the code, implying it is accessible to anyone with source code access, and cannot be changed without rebuilding the application.Similarity ID: -857633280
Severity: Low
CWE:798
Checkmarx
Training
Recommended Fix
Lines: 159

Code (Line #159):
"@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#e4a13cfd135ec766dc9148ba4fe4d3ac76d94137",

Issue still exists.

Issue still exists.

Issue still exists.

Issue still exists.

Issue still exists.


Incorrect render for multiple terms of glossary

Describe the bug
Glossary supports display multiple terms but furo renders it incorrectly.
Checked on 2020.11.19b18 version.
To Reproduce
Steps to reproduce the behavior:

Add glossary with multiple term:

.. glossary::

    RC
    release candidate
        The environment separated and isolated from
        production intended for developing as well as manual and
        automated testing.

Render html by furo theme.
Incorrect result:

Expected behavior
Something like this:

PS: furo is grate theme! Thank for work!

Duplicate of #35

Released the fix as part of beta 19. Let me know if that's not good-enough. :)

Fantastic, thank!

Yes! thats fixed


Outdated documentation (and code?) regarding Stateful RNNs

The documentation for the base RNN layer contains the following explanation, which is outdated:

Note on using statefulness in RNNs:
You can set RNN layers to be 'stateful', which means that the states computed for the samples in one batch will be reused as initial states for the samples in the next batch. This assumes a one-to-one mapping between samples in different successive batches.
To enable statefulness:
Specify stateful=True in the layer constructor.
Specify a fixed batch size for your model, by passing If sequential model: batch_input_shape=(...) to the first layer in your model. Else for functional model with 1 or more Input layers: batch_shape=(...) to all the first layers in your model. This is the expected shape of your inputs including the batch size. It should be a tuple of integers, e.g. (32, 10, 100).
Specify shuffle=False when calling fit().
To reset the states of your model, call .reset_states() on either a specific layer, or on your entire model.
Note on specifying the initial state of RNNs:
You can specify the initial state of RNN layers symbolically by calling them with the keyword argument initial_state. The value of initial_state should be a tensor or list of tensors representing the initial state of the RNN layer.
You can specify the initial state of RNN layers numerically by calling reset_states with the keyword argument states. The value of states should be a numpy array or list of numpy arrays representing the initial state of the RNN layer.

In particular:

the batch_input_shape argument no longer exists, I believe we should instead add an Input layer and specify its batch_size argument, if I'm not mistaken.
the Sequential class no longer has a reset_states() method. Instead, I suppose we must iterate through the layers and if they have a reset_state() method (no s at the end), then we call it.

However, I was unable to build a stateful RNN, I'm getting the following exception:
Epoch 1/10
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[<ipython-input-100-aaee251b8e39>](https://localhost:8080/#) in <cell line: 3>()
      1 model.compile(loss="sparse_categorical_crossentropy", optimizer="nadam",
      2               metrics=["accuracy"])
----> 3 history = model.fit(stateful_train_set, validation_data=stateful_valid_set,
      4                     epochs=10, callbacks=[ResetStatesCallback(), model_ckpt])

1 frames
[/usr/local/lib/python3.10/dist-packages/keras/src/utils/traceback_utils.py](https://localhost:8080/#) in error_handler(*args, **kwargs)
    120             # To get the full stack trace, call:
    121             # `keras.config.disable_traceback_filtering()`
--> 122             raise e.with_traceback(filtered_tb) from None
    123         finally:
    124             del filtered_tb

[/usr/local/lib/python3.10/dist-packages/keras/src/utils/traceback_utils.py](https://localhost:8080/#) in error_handler(*args, **kwargs)
    120             # To get the full stack trace, call:
    121             # `keras.config.disable_traceback_filtering()`
--> 122             raise e.with_traceback(filtered_tb) from None
    123         finally:
    124             del filtered_tb

ValueError: Exception encountered when calling GRU.call().

Input tensor `sequential_10_1/gru_6_1/ReadVariableOp:0` enters the loop with shape (1, 128), but has shape (None, 128) after one iteration. To allow the shape to vary across iterations, use the `shape_invariants` argument of tf.while_loop to specify a less-specific shape.

Arguments received by GRU.call():
  • sequences=tf.Tensor(shape=(None, None, 16), dtype=float32)
  • initial_state=None
  • mask=None
  • training=True

I'm not sure whether this is a bug or whether I'm not implementing a stateful RNN correctly using Keras 3. If someone can please explain how to build one, I'm happy to update the documentation.

Thanks for the report. Do you have a Colab or code snippet to reproduce the error?
Best I can tell the docs look outdated. The two corrections you made are accurate.

Thanks @fchollet , here's a little code snippet to reproduce the error:
import keras
import numpy as np

model = keras.Sequential([
    keras.layers.Input(batch_shape=[1, 10, 3]),
    keras.layers.LSTM(10, return_sequences=True, stateful=True),
    keras.layers.LSTM(10, return_sequences=True, stateful=True),
    keras.layers.Dense(5)
])

model.compile(loss="mse", optimizer="sgd")

X_train = np.random.rand(100, 10, 3)
y_train = np.random.rand(100, 10, 5)
model.fit(X_train, y_train, epochs=1)

I'm getting the same exception as above.

Epoch 1/5
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[<ipython-input-3-b43b82db1597>](https://localhost:8080/#) in <cell line: 15>()
     13 X_train = np.random.rand(100, 10, 3)
     14 y_train = np.random.rand(100, 10, 5)
---> 15 model.fit(X_train, y_train, epochs=1)

1 frames
[/usr/local/lib/python3.10/dist-packages/keras/src/utils/traceback_utils.py](https://localhost:8080/#) in error_handler(*args, **kwargs)
    120             # To get the full stack trace, call:
    121             # `keras.config.disable_traceback_filtering()`
--> 122             raise e.with_traceback(filtered_tb) from None
    123         finally:
    124             del filtered_tb

[/usr/local/lib/python3.10/dist-packages/keras/src/utils/traceback_utils.py](https://localhost:8080/#) in error_handler(*args, **kwargs)
    120             # To get the full stack trace, call:
    121             # `keras.config.disable_traceback_filtering()`
--> 122             raise e.with_traceback(filtered_tb) from None
    123         finally:
    124             del filtered_tb

ValueError: Exception encountered when calling LSTM.call().

Input tensor `sequential_2_1/lstm_4_1/ReadVariableOp:0` enters the loop with shape (1, 10), but has shape (None, 10) after one iteration. To allow the shape to vary across iterations, use the `shape_invariants` argument of tf.while_loop to specify a less-specific shape.

Arguments received by LSTM.call():
  • sequences=tf.Tensor(shape=(None, 10, 3), dtype=float32)
  • initial_state=None
  • mask=None
  • training=True

Here's a gist notebook with the code above.

Thanks for the code. The origin of the issue is a discrepancy between the batch size specified in Input and the batch size effectively received by the model (if you pass raw numpy data to fit(), it gets chunked into batches, configured by the batch_size argument).
You can just pass batch_size=1 in fit() to fix it (or otherwise use a generator-like or tf.data.Dataset-like data source)
We should have a check somewhere to prevent against such a mistmatch.

Ah got it, thanks François. 👍


Cannot get Ports module

I'm fairly new to elm but I followed the README.md and I'm getting the following error:
Cannot find variable `Ports.LocalStorage.storageGetItemResponse`.

109|     Ports.LocalStorage.storageGetItemResponse RecIdFromStorage
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
No module called `Ports.LocalStorage` has been imported.

Any ideas?

Ah nm I imported Ports.LocalStorage and it went away


Guideline for training and testing

Hi,
Is there any guideline on training and testing available, or maybe a tutorial that one can follow to train/ test with custom data?
Thanks,

Hope this commit can help.


On follow, deliver last few posts

When we get a follow, should we deliver the last few posts from the followee to the follower? Seems like it'd be nice UX. Those posts' published/updated dates will be intact, so they should merge into the appropriate places in the follower's feed and not all show up at the top.

Hubzilla grabs the last 10 posts when you follow someone (if their platform supports it). It is a really nice feature. I think it should be adopted, where technically feasible.
It would also be nice if you can detect threaded conversations, and get the whole conversation. (Only platforms that have threaded conversations would support this.)

This probably used to work on ActivityPub, but now doesn't according to https://github.com/snarfed/bridgy-fed/issues/1248#issuecomment-2289615188.
On the Bluesky side, something seems to be amiss to some extent too, as atproto-browser "🚨 Failed to fetch collections."

Right, some fediverse servers backfill from outbox, which I disabled in #1248, but not all. I was thinking about proactively delivering historical posts for the servers that don't backfill.
"Failed to fetch collections" is probably because our com.atproto.repo.describeRepo implementation is incomplete right now: https://github.com/snarfed/arroba/issues/33 . Worth fixing, but probably unrelated here.


Refactor Z3SolverContext cellCache

[ ] ~Tests added for any new code~
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] ~Documentation added for any new functionality~
[ ] ~Entry added to UNRELEASED.md for any new functionality~

This PR refactors updates to cellCache to not happen during iterations. Closes #1595.

Codecov Report

Merging #1703 (99be08f) into unstable (7823b14) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 99be08f differs from pull request most recent head d06a4c7. Consider uploading reports for the commit d06a4c7 to get more accurate results

@@            Coverage Diff            @@
##           unstable    #1703   +/-   ##
=========================================
  Coverage     75.31%   75.32%
=========================================
  Files           357      357
  Lines         11541    11541
  Branches        576      595   +19
=========================================
+ Hits           8692     8693    +1
+ Misses         2849     2848    -1

Impacted Files
Coverage Δ

...rsyte/apalache/tla/bmcmt/smt/Z3SolverContext.scala
81.18% <100.00%> (ø)

...he/io/annotations/parser/CommentPreprocessor.scala
96.70% <0.00%> (+1.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Update whats-new.md

@BaherAbdullah : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@BaherAbdullah : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Use enum for typing NOTHING

Summary
This will allow those extending attrs to type NOTHING as Literal[NOTHING].
This is the recommended way of doing this, at least until PEP-661 lands.
I also just realized that Attribute types default as Optional[_T], when it should really be Union[_T, Literal[NOTHING]]. This would be a prerequisite for that.
Pull Request Check List

[x] Added tests for changed code.
Our CI fails if coverage is not 100%.
[x] New features have been added to our Hypothesis testing strategy.
[x] Changes or additions to public APIs are reflected in our type stubs (files ending in .pyi).

[x] ...and used in the stub test file tests/typing_example.py.
[x] If they've been added to attr/__init__.pyi, they've also been re-imported in attrs/__init__.pyi.

[x] Updated documentation for changed code.

[x] New functions/classes have to be added to docs/api.rst by hand.
[x] Changes to the signature of @attr.s() have to be added by hand too.
[x] Changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.
Find the appropriate next version in our __init__.py file.

[x] Documentation in .rst files is written using semantic newlines.
[x] Changes (and possible deprecations) have news fragments in changelog.d.
[x] Consider granting push permissions to the PR branch, so maintainers can fix minor issues themselves without pestering you.

Hm, I don't hate it.

Forgot to mention-- this is the recommended way of doing this, at least until PEP-661 lands.

I'm not really firm on these topics, so I'm assigning this to Tin. I hope the test_mypy failures are just caused by this? 😅

Don't worry, it's not failing anymore. That was just mypy improving attrs support and subtly changing an error message.

NOTHING is now the single variant of an Enum, keeping __repr__ and __bool__'s function.
However, since a type stub file overrides any typing from the python module, the enum has to be (mostly) repeated in the stub. If we're fine with it, that's okay. If not, we can work around this by putting it in its own module, and reimporting it in _make.py and __init__.pyi.
I'm looking into how the enum affects the docs automation right now.
One last question: should I update

@hynek I think this is ready to go. We we want a changelog entry?

yeah, because it probably breaks someone too
ideally with a short summary of the upsides please because i keep forgetting myself

The only upside is that it allows the use of Literal[NOTHING] in type signatures. That's useful for those extending attrs.
A project I'm working on does some of its own initialization logic, and inspecting Attribute properties to see if they're NOTHING is necessary. Using it as a return type annotation helps us make sure it's handled properly down the line.
(We have a compatibility shim that just re-exports NOTHING, but type stubs it as an enum)

Ah, I forgot to revisit this one part:
Some of the current typings are technically incorrect. For example Attribute.default is typed as Optional[_T]. For example:
@frozen
class Foo:
    bar: int = field()
    baz: int = field(default=2)

Someone inspecting the Attribute[int] for bar will have default typed as Optional[int], and will be surprised to find a NOTHING! Similarly, inspecting Attribute[int] for baz will have someone handling the None case, even though it can't be None.
I'll have to see if there are other examples. If you want, this could be handled as part of this PR, or another. Like you said, it might break someone's code (but I think it'll mainly just change up the type checking they do). Doing it in this PR would mean they handle it all in one fell swoop. Thoughts?

Where are you seeing this Optional[_T] thing?

In attrs.Attribute's type stub}

What are the next steps here?

We add a short changelog entry and merge this.

@hynek I've added a small changelog entry, merge at will.

Thanks @KevinMGranger!

I'm seeing really weird behavior. This works perfectly fine within attrs, but when importing it outside of attrs, mypy gives a weird complaint:
# foo.py
from typing import Literal
from attrs import NOTHING

def foo(val: str | Literal[NOTHING]):
    pass

$ mypy foo.py
foo.py:4: error: Function is missing a return type annotation
foo.py:4: error: Parameter 1 of Literal[...] is invalid
foo.py:4: error: Variable "attr.NOTHING" is not valid as a type
foo.py:4: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#variables-vs-type-aliases
Found 3 errors in 1 file (checked 1 source file)

Anyone else seeing this? I'm trying to debug if this is a mypy issue.

pyright doesn't complain, while mypy does. It looks like I misread GVR's suggestion, and missed:

(Alas, I haven't found a way to alias A = AA.A and be able to write Literal[A].)

There are two (and a half) options here:

Revert this change
Rename the enum class Nothing, use Nothing for type annotations, and NOTHING for the value. If that's too confusing for users, 1 is the better option.
Contribute to mypy so this works (kidding. maybe.)

I think we can leave it and submit a bug report to Mypy. Unfortunate.

Does it help if you annotate the alias with Final?

Re: MyPy option: how broken is this? Is this gonna make me fix all my projects like the AttrsInstance thing in 22.1 or is this something that's only broken if you're trying to use the new feature it's supposed to implement?

Yeah, I think it's only broken if you try to Literal it, and hopefully Mypy fixes that.


No Implementation Found For Method initPath

When I try to use the first command: final exif = await Exif.fromPath(picture!.path);
I get the following error:
E/flutter (10499): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: MissingPluginException(No implementation found for method initPath on channel native_exif)
I have tried to run flutter clean and flutter pub get to no avail.

I'm facing the same issue.

It sounds like a project misconfiguration.
You may want to check out https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects#full-flutter-app-migration (most importantly the GeneratedPluginRegistrant parts) or compare your iOS AppDelegate.[m/swift] and android MainActivity.[java/kt] with the example app of this project.
Also you may want to run flutter clean and restart your app.

我也有同样的问题在android13上，因为29之后无法读取gps信息https://developer.android.com/reference/android/provider/MediaStore.Images.ImageColumns#LATITUDE，所以我需要读取图片的exif

It sounds like a project misconfiguration.
You may want to check out https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects#full-flutter-app-migration (most importantly the GeneratedPluginRegistrant parts) or compare your iOS AppDelegate.[m/swift] and android MainActivity.[java/kt] with the example app of this plugin.
Also you may want to run flutter clean and restart your app.

flutter clean之后也没有用
`package io.flutter.plugins;
import androidx.annotation.Keep;
import androidx.annotation.NonNull;
import io.flutter.Log;
import io.flutter.embedding.engine.FlutterEngine;
/**

Generated file. Do not edit.
This file is generated by the Flutter tool based on the
plugins that support the Android platform.
*/
@Keep
public final class GeneratedPluginRegistrant {
private static final String TAG = "GeneratedPluginRegistrant";
public static void registerWith(@NonNull FlutterEngine flutterEngine) {
try {
flutterEngine.getPlugins().add(new me.yohom.amap_core_fluttify.AmapCoreFluttifyPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin amap_core_fluttify, me.yohom.amap_core_fluttify.AmapCoreFluttifyPlugin", e);
}
try {
flutterEngine.getPlugins().add(new me.yohom.amap_location_fluttify.AmapLocationFluttifyPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin amap_location_fluttify, me.yohom.amap_location_fluttify.AmapLocationFluttifyPlugin", e);
}
try {
flutterEngine.getPlugins().add(new me.yohom.amap_map_fluttify.AmapMapFluttifyPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin amap_map_fluttify, me.yohom.amap_map_fluttify.AmapMapFluttifyPlugin", e);
}
try {
flutterEngine.getPlugins().add(new me.yohom.amap_search_fluttify.AmapSearchFluttifyPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin amap_search_fluttify, me.yohom.amap_search_fluttify.AmapSearchFluttifyPlugin", e);
}
try {
flutterEngine.getPlugins().add(new me.yohom.core_location_fluttify.CoreLocationFluttifyPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin core_location_fluttify, me.yohom.core_location_fluttify.CoreLocationFluttifyPlugin", e);
}
try {
flutterEngine.getPlugins().add(new me.yohom.foundation_fluttify.FoundationFluttifyPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin foundation_fluttify, me.yohom.foundation_fluttify.FoundationFluttifyPlugin", e);
}
try {
flutterEngine.getPlugins().add(new com.cloudacy.native_exif.NativeExifPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin native_exif, com.cloudacy.native_exif.NativeExifPlugin", e);
}
try {
flutterEngine.getPlugins().add(new com.baseflow.permissionhandler.PermissionHandlerPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin permission_handler, com.baseflow.permissionhandler.PermissionHandlerPlugin", e);
}
try {
flutterEngine.getPlugins().add(new com.fluttercandies.photo_manager.PhotoManagerPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin photo_manager, com.fluttercandies.photo_manager.PhotoManagerPlugin", e);
}
try {
flutterEngine.getPlugins().add(new io.flutter.plugins.sharedpreferences.SharedPreferencesPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin shared_preferences_android, io.flutter.plugins.sharedpreferences.SharedPreferencesPlugin", e);
}
try {
flutterEngine.getPlugins().add(new io.flutter.plugins.urllauncher.UrlLauncherPlugin());
} catch(Exception e) {
Log.e(TAG, "Error registering plugin url_launcher_android, io.flutter.plugins.urllauncher.UrlLauncherPlugin", e);
}
}
}
`自动生成的文件中也看着没有问题

This problem is caused by writing to a read-only file, you should read the file into the app sandbox directory and then manipulate the file before saving the temporary file to an album or somewhere else.The following code works just fine.
var imgData = imgByteData.buffer.asUint8List();
File? tempFile = await Utils.saveFileToTemp(imgData, "tmp_image.jpg");
if (tempFile == null) {
return;
}
var tempFilePath = tempFile.path;
try {
final exif = await Exif.fromPath(tempFilePath);
await exif.writeAttribute("UserComment",
"123456");
final userComment = await exif.getAttribute("UserComment");
LogUtil.v("userComment: $userComment", tag: 'saveToLocal');
await exif.close();
} catch (e) {
LogUtil.e("err: $e", tag: 'saveToLocal');
}
static Future<File?> saveFileToTemp(Uint8List bytes, String fileName) async {
try {
final String tempPath = (await getTemporaryDirectory()).path;
final String filePath = tempPath + '/' + fileName;
final File targetFile = File(filePath);
if (await targetFile.exists()) {
await targetFile.delete();
}
return await targetFile.writeAsBytes(bytes);
} catch (e) {
print('e:$e');
}
}


Call composite switches between vertical and horizontal gallery depending on aspect ratio

What

Switch to vertical overflow gallery in VideoGallery in call composite when MediaGallery aspect ratio is > 16:9
Why

https://skype.visualstudio.com/SPOOL/_workitems/edit/3157102
How Tested

Tested in local calling sample
Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

The mobile snapshots look a bit off. They do not maintain their mobile aspect ratio, is this expected?


Fix ExceptionInInitializerError in reified GenericFutureListener

The issue was mentioned here and here.
The root of the problem:
Here we have a try-catch block that compiles to a separate class Deferred$fn__** to be lately instanciated (and invoked) in Deferred.success. The class contains a static variable with RT.readString of the ns declaration, I assume that comes from a clojure.tools.logging toolchain. The problem is that our first attempt to use this class happens on a Netty's thread (globalEventExecutor-** created by this factory to be more specific) and... there's no appropriate class loader there. RT.baseLoader function here falls back to a current thread context loader (because of the default value of *use-context-classloader* set to true) which is set to NULL on Netty's threads.
The solution:
The simplest thing we can do here is to carry class loader in a closure (either current loader from a thread local Clojure frame or newly created). I'm also going to submit a PR to manifold to set an appropriate class loader to each new thread created by this factory. It will reduce the number of calls to RT.makeClassLoader as we need to do this only once per thread.
I've been also thinking if it's worth the effort to try to submit an issue to Clojure JIRA with the problem definition but now I doubt if it's necessary. Being pessimistically careful here would introduce some performance penalties in a lot of places when in fact the code that was invoked from a non-Clojure thread is not that wide-spread pattern. And it's hard for the compiler to distinguish such cases in advanced. There're still a lot of improvements might be done at least to eliminate unnecessary calls to a reader.. but that's for sure a completely separate topic.
P.S. Submitting this a separate PR not to mess up with tons of changes from #406.

Curious if this issue has re-surfaced since the 2019 reporting (and I presume fixing)?
Moments ago I was testing a very simple Ring server [aleph "0.4.6"] with pretty much the exact code @kwladyka has in his sample code above, and got this familiar exception block:
May 24, 2020 7:07:37 PM io.netty.util.concurrent.DefaultPromise notifyListener0
WARNING: An exception was thrown by aleph.netty$wrap_future$reify__17414.operationComplete()
java.lang.ExceptionInInitializerError
        at manifold.deferred.Deferred.success(deferred.clj:398)
        at manifold.deferred$success_BANG_.invokeStatic(deferred.clj:243)
        at manifold.deferred$success_BANG_.invoke(deferred.clj:240)
        at aleph.netty$wrap_future$reify__17414.operationComplete(netty.clj:199)
        at io.netty.util.concurrent.DefaultPromise.notifyListener0(DefaultPromise.java:511)
        at io.netty.util.concurrent.DefaultPromise.notifyListenersNow(DefaultPromise.java:485)
        at io.netty.util.concurrent.DefaultPromise.notifyListeners(DefaultPromise.java:424)
        at io.netty.util.concurrent.DefaultPromise.setSuccess(DefaultPromise.java:94)
        at io.netty.util.concurrent.MultithreadEventExecutorGroup$1.operationComplete(MultithreadEventExecutorGroup.java:117)
        at io.netty.util.concurrent.DefaultPromise.notifyListener0(DefaultPromise.java:511)
        at io.netty.util.concurrent.DefaultPromise.notifyListenersNow(DefaultPromise.java:485)
        at io.netty.util.concurrent.DefaultPromise.access$000(DefaultPromise.java:33)
        at io.netty.util.concurrent.DefaultPromise$1.run(DefaultPromise.java:435)
        at io.netty.util.concurrent.GlobalEventExecutor$TaskRunner.run(GlobalEventExecutor.java:248)
        at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
        at java.base/java.lang.Thread.run(Thread.java:830)
Caused by: java.lang.ClassNotFoundException: clojure/lang/PersistentArrayMap
        at java.base/java.lang.Class.forName0(Native Method)
        at java.base/java.lang.Class.forName(Class.java:416)
        at clojure.lang.RT.classForName(RT.java:2211)
        at clojure.lang.RT.classForName(RT.java:2220)
        at clojure.lang.Reflector.invokeStaticMethod(Reflector.java:324)
        at clojure.lang.LispReader$EvalReader.invoke(LispReader.java:1322)
        at clojure.lang.LispReader$DispatchReader.invoke(LispReader.java:853)
        at clojure.lang.LispReader.read(LispReader.java:285)
        at clojure.lang.LispReader.read(LispReader.java:220)
        at clojure.lang.LispReader.access$200(LispReader.java:41)
        at clojure.lang.LispReader$MetaReader.invoke(LispReader.java:954)
        at clojure.lang.LispReader.read(LispReader.java:285)
        at clojure.lang.LispReader.readDelimitedList(LispReader.java:1398)
        at clojure.lang.LispReader$ListReader.invoke(LispReader.java:1243)
        at clojure.lang.LispReader.read(LispReader.java:285)
        at clojure.lang.LispReader.read(LispReader.java:220)
        at clojure.lang.LispReader.access$200(LispReader.java:41)
        at clojure.lang.LispReader$EvalReader.invoke(LispReader.java:1301)
        at clojure.lang.LispReader$DispatchReader.invoke(LispReader.java:853)
        at clojure.lang.LispReader.read(LispReader.java:285)
        at clojure.lang.LispReader.read(LispReader.java:216)
        at clojure.lang.LispReader.read(LispReader.java:205)
        at clojure.lang.RT.readString(RT.java:1878)
        at clojure.lang.RT.readString(RT.java:1873)
        at manifold.deferred.Deferred$fn__3712.<clinit>(deferred.clj:398)
        ... 16 more

A Google hop led me to this post, which towards the end says "... this issue appear only when you didn’t open website even once."
So, I added a (Thread/sleep 30000) before my (.close server), hit the handler once via the browser, waited, and got a clean shutdown with no exception.
Obviously there's going to be times when a system starts up, discovers an issue, and needs to tear itself down cleanly again before anyone connects.

@wls it looks like the fix for this was added in 0.4.7-alpha5, not 0.4.6