Slow suggestions

Using Volar, auto completion become unusable because it can takes seconds to display the suggestion, when, most of the time you already have finished typing. I've seen #455 but i don't know if it's linked, i tested with every extension disabled other than volar 0.27.21

Please try to provide repro case, see https://github.com/johnsoncodehk/volar/issues/507#issuecomment-922251114.

there is no need of repro case, i just tested with default volar-starter and it's behaving the same way, suggestion is slow to display/load

Have you try disable other extensions? And please also make sure volar.lowPowerMode is disabled.
Please provide your VSCode info by Help: About command, and profiling record: https://github.com/johnsoncodehk/volar/discussions/413

i tested with every extension disabled other than volar 0.27.21

VSCode Info :
Version: 1.60.1 (system setup)
Commit: 83bd43bc519d15e50c4272c6cf5c1479df196a4d
Date: 2021-09-10T17:07:10.714Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043

volar.lowPowerMode is set to false
little example of what i'm trying to say, and btw, the part before displaying Loading... is also about 0.5 sec long

Can you compare the auto-complete speed of plain ts and SFC's

here is the chrome profile
Profile.zip

Can you compare the auto-complete speed of plain ts and SFC's

You need to compare it by same code with same auto-complete trigger offset, because inside setup(), auto-complete is much slower due to TS overhead.

like this ?

maybe it'll be easier like this, here is my current repo https://github.com/Zenthae/seer/tree/v2
i invited you so you should be able to clone it to test

It seem typescript-language-features has additional completion cache logic, will try to implement, but it's not easy.

i think this problem just only appear in windows

Now it does respond so slowly that I don’t want to use it

I think it can be compared with vetur. Now the prompt speed of volar seriously affects the user experience. And I think volar is now much slower than before

Please try to downgrade volar and let me know which version occur the problem beginning.

slower than vanilla TS but still a lot better than before.

@Zenthae what version did you switch to? I'm also having performance issues with Volar, albeit my i7-7500U is pretty old now so that might also slow it down.

for my latest gif i used the latest version available which is v0.27.24

I think it can be compared with vetur. Now the prompt speed of volar seriously affects the user experience. And I think volar is now much slower than before

Please try to downgrade volar and let me know which version occur the problem beginning.

I tried to use the old version v0.22.23 of volar, and use the import module statement in the setup script, which is significantly faster than the latest version of volar
The following is the information output of the 2 versions。
I only enter "im"  and wait for the prompt "import from" statement
v0.22.23 of volar：
Received response 'textDocument/completion - (15)' in 1164ms.

v0.27.24 of volar：
Received response 'textDocument/completion - (9)' in 1796ms.

@CrystalWindSnake could you run debug mode to check is the problem occur begin in this commit https://github.com/johnsoncodehk/volar/commit/15e744f335fcbdb4db2e42a2778970d597b92ae0 ?

I think it can be compared with vetur. Now the prompt speed of volar seriously affects the user experience. And I think volar is now much slower than before

Please try to downgrade volar and let me know which version occur the problem beginning.

I tried to use the old version v0.22.23 of volar, and use the import module statement in the setup script, which is significantly faster than the latest version of volar
The following is the information output of the 2 versions。
I only enter "im" and wait for the prompt "import from" statement
v0.22.23 of volar：
Received response 'textDocument/completion - (15)' in 1164ms.

v0.27.24 of volar：
Received response 'textDocument/completion - (9)' in 1796ms.

I found that the new version of volar will repeatedly send completion messages.
When I enter "im", he will receive "Received response 'textDocument/completion" messages twice

@CrystalWindSnake could you run debug mode to check is the problem occur begin in this commit 15e744f ?
Please also provide:

VSCode info by Help: About command
TS version if you are using workspace TS module

VSCode info by Help: About ：
版本: 1.60.2 (user setup)
提交: 7f6ab5485bbc008386c4386d08766667e155244e
日期: 2021-09-22T12:00:31.514Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19042

TS version(This is the prompt version of volar in the lower right corner of the interface):
TS 4.4.3

I found that the new version of volar will repeatedly send completion messages.
When I enter "im", he will receive "Received response 'textDocument/completion" messages twice，and the old version is only sent once.
Is this the cause of the problem?

@CrystalWindSnake I think you received 2 responses because you get a incomplete completion result when you typed i. When you typed m, TS calculate the complete result.
0.27.5 has related changes, please let me know if it's effect to you.

@CrystalWindSnake I think you received 2 responses because you get a incomplete completion result when you typed i. When you typed m, TS calculate the complete result.
0.27.5 has related changes, please let me know if it's effect to you.

Thank you, hope volar will get better and better

遇到同样的问题，mac系统，启用插件的时候，智能提示太慢了。

遇到同样的问题，mac系统，启用插件的时候，智能提示太慢了。

同样的问题，有解决办法吗

遇到同样的问题，mac系统，启用插件的时候，智能提示太慢了。

同样的问题，有解决办法吗

暂时没有。。研究用 webstorm 了。。

@johnsoncodehk    我在windows上，也遇到了同样的问题，应该如何解决

@chivesssss, @zhoujingfu see #455.
@jahnli please provide repro case otherwise I can't check.

I think it can be compared with vetur. Now the prompt speed of volar seriously affects the user experience. And I think volar is now much slower than before

Please try to downgrade volar and let me know which version occur the problem beginning.

Maybe 0.29.0

@johnsoncodehk  I don't know how to reproduce. At present, Windows will be slow if it only installs a volar code prompt. After uninstallation, it will be restored to normal

Please track #808.


[FEAT] A pytoil bug command

Is there an existing issue for this?

[X] I have searched the existing issues

How would this feature be useful?
Not essential but go has go bug which helps you file an issue about the language. I like the idea, might help tease out some bugs from people (if any) who are using it.
Describe the solution you'd like
A pytoil bug command that just opens up pytoils issues. Super simple!
Describe alternatives you've considered
No response
Anything else?
No response

Closed in #577


Updated restrictions in types and functions

Previously, we were relying on a Julia wild card constructor for composite types with the restrictions we used in the composite types. This wild card constructor only worked for standard composite types, but not parametric types, resulting in slightly differing descriptions.
Hence, we concluded to have the same restrictions for all declared types.

This PR should only be merged, once we have a first release and the package registered.

Using Vector{<:Data} leads to breaking changes, as it would not allow to pass an empty Array and transform it then to a Vector{Data} array. This is circumvented by leaving it at Vector{Data}. The only difference is RefStorage{T} where it was already required as it is a parametric type.


How can I use the Node3DGizmo class?

Godot version
mono c# 4.1.1
System information
Deepin os, opengl
Issue description
How can I use the Node3DGizmo class? Where is the source code of this class? I could not use this feature for some reason.
Steps to reproduce
my code:
var gizmo = new Node3DGizmo();
error:
'Node3DGizmo' does not contain a constructor that takes 0 arguments
Minimal reproduction project
using Godot;
using System;

public partial class Gizmo : Node3D
{
	// Called when the node enters the scene tree for the first time.
	public override void _Ready()
	{
		var gizmo = new Node3DGizmo();
		//AddGizmo(gizmo);
	}

	// Called every frame. 'delta' is the elapsed time since the previous frame.
	public override void _Process(double delta)
	{
	}
}

Please note that this platform is for reporting bugs, not questions on how to use Godot or its API. Please refer to other community platforms for user support: https://godotengine.org/community/
Still, some docs were missing for this class so I'm adding them in #82681. What you should look at is EditorNode3DGizmo, which has ample documentation.

Actually, what I was trying to explain was that the Node3DGizmo class could not be created, I thought it was a bug. I think it is necessary to use the EditorNode3DGizmo class, but I don't understand why this Node3DGizmo was created, if it cannot be used?

@unicornlox It's an abstract class, we have a few in our API. You are not supposed to use them directly.
In this case, the reason it exists is purely internal, so updating the docs to mention that users shouldn't bother with the class should be sufficient.


Running Validator without ignoring warnings will fail silently

Rasa version: 2.0.0a4
Rasa SDK version (if used & relevant): 2.0.0a4
Rasa X version (if used & relevant): n/a
Python version: 3.8.2
Operating system (windows, osx, ...): Linux-5.4.0-45-generic-x86_64-with-glibc2.29
Issue:
Running Validator.verify_nlu(ignore_warnings=False) (which I think needs to be renamed to verify_all() as I've mentioned here) will return False without providing logs (unless you enable debug logs.  It seems as though in the following functions:

verify_intents()
verify_intents_in_stories()
verify_utterances()
verify_utterances_in_stories()

logger.debug is used instead of logger.warning or rasa.shared.utils.io.raise_warning.  These are minor logs that are suppressed but nonetheless will still cause a silent failure, so I'm not sure if it's necessary to fail on these debug level issues.
Error (including full traceback):
N/A

Command or request that led to error:
rasa data validate (in my case I was using the Validator class directly)

Thanks for raising this issue, @melindaloubser1 will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗


Always unlock needs fixing

https://github.com/rednblkx/HomeKey-ESP32/blob/849957ecb18f225973a7f52afce429a2d8f5b55a/src/main.cpp#L1037C1-L1038C12
The closing curly brace '}' should move up before the 'else if'
Otherwise, lock always happens after auto unlock.

Not sure how that happened but thanks for the heads up, now fixed in dev, will be released shortly as a small update alongside other things.


How to use deliver with fastlane swift

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject

Deliver with fastlane swift

No

No
Question Description

When initializing fastlane swift, a DeliverFile is created in the FastlaneRunner project. But when initializing Deliver, a non-swift Deliverfile is created in the fastlane directory. When running lanes thta call deliver() this non-swift (I assume Ruby) file is used in place of the auto-generated one in the FastlaneRunner project.
Additionally, the instructions in the auot-generated Deliverfile.swift aren't exactly clear on how to create a new Deliverfile.swift that will be used.
//  This file is provided by fastlane and WILL be overwritten in future updates
//  If you want to add extra functionality to this project, create a new file in a
//  new group so that it won't be marked for upgrade

Following the above instructions will cause build errors in the FastlaneRunner Xcode project due to duplicate filenames.
What's the process for using deliver with fastlane swift?

✅ fastlane environment ✅
Stack

Key
Value

OS
10.14

Ruby
2.2.4

Bundler?
false

Git
git version 2.17.2 (Apple Git-113)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.14 (18A391)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

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

Xcode Version
10.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
`./fastlane/Fastfile.swift`
// This file contains the fastlane.tools configuration
// You can find the documentation at https://docs.fastlane.tools
//
// For a list of all available actions, check out
//
//     https://docs.fastlane.tools/actions
//

import Foundation

class Fastfile: LaneFile {

	var username: String {
		return environmentVariable(set: nil, get: "username", remove: nil)
	}

	func beforeAll() {
		configure()
	}

	func betaLane(withOptions options:[String: String]?) {
		desc("Push a new beta build to TestFlight")

		bumpVersion()

		buildApp(workspace: "Simple Booth.xcworkspace", scheme: options?["scheme"] ?? "Halo")

		uploadToTestflight(username: username)
	}

	func betaUploadLane(withOptions options:[String: String]?) {
		uploadToTestflight(username: options?["username"] ?? "", ipa: "Simple Booth Halo.ipa")
	}

	func uploadMetadataLane() {
		deliver()
	}

	func playgroundLane(withOptions options:[String: String]?) {
		desc("Doesn't 'do' anything. Just a place to run commands and see what they do and how they work.")

		let version = getVersionNumber(xcodeproj: nil, target: "Halo", configuration: nil)
		puts(message: "🎸🎸🎸 VERSION: \(version) 🎸🎸🎸")

		let build = getBuildNumber()
		puts(message: "🎸🎸🎸 BUILD: \(build) 🎸🎸🎸")

		let user = environmentVariable(set: nil, get: "username", remove: nil)
		puts(message: "🎸🎸🎸 USER: \(user) 🎸🎸🎸")

	}

	func cleanLane() {
		desc("Clean project and delete derived data")
		xcclean()
		clearDerivedData()
	}

	func dsymLane(withOptions options:[String: String]?) {

		let buildNumber = options?["buildNumber"] ?? getBuildNumber()

		let versionNumber = options?["versionNumber"] ?? getVersionNumber(xcodeproj: nil, target: "Halo", configuration: nil)

		downloadDsyms(username: username, appIdentifier: appIdentifier, version: versionNumber, buildNumber: buildNumber)

		let dsymPath = "com.simplebooth.halo-\(versionNumber)-\(buildNumber).dSYM.zip"

		uploadSymbolsToCrashlytics(dsymPath: dsymPath, apiToken: nil, gspPath: nil, binaryPath: nil, platform: "ios", dsymWorkerThreads: 1)
	}

	// MARK: - Helpers

	/// Bump the build number and automatically commit and push the change.
	func bumpVersion() {

		ensureGitStatusClean()

		let buildNumber = incrementBuildNumber(xcodeproj: "Simple Booth.xcodeproj")

		commitVersionBump(message: "Bumping version from fastlane", xcodeproj: "Simple Booth.xcodeproj", include: [])

		let version = getVersionNumber(xcodeproj: nil, target: "Halo", configuration: nil)

		addGitTag(tag: "\(version)-(\(buildNumber))", buildNumber: buildNumber)

		pushToGitRemote()
	}

}

No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.108.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
2.3.2

atomos
0.1.3

CFPropertyList
3.0.0

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.6

xcodeproj
1.6.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
1.8.0

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.4.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.5

tty-cursor
0.6.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.62.0

faraday
0.12.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.3

gh_inspector
1.1.3

json
1.8.1

mini_magick
4.5.1

multi_json
1.13.1

multi_xml
0.6.0

rubyzip
1.2.2

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.5.0

bundler
1.14.6

faraday_middleware
<IP_ADDRESS>

naturally
2.2.0

simctl
1.6.4

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
1.5.6

memoist
0.16.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.21.2

emoji_regex
0.1.1

libxml-ruby
3.0.0

psych
2.0.8

generated on: 2018-11-08

Can you maybe help out @taquitos here?

+1 for the explanation on how to use delver with swift runner.
PS I can add that Swift side of Fastlane is poorly documented. Is it still a beta feature?

Yes: https://docs.fastlane.tools/getting-started/ios/fastlane-swift/

@janpio yes to "Is it still a beta feature?" or to the lack of documentation?
I am well aware of the link you've shared, there is nothing there about how to call deliver or why does SwiftRunner doesn't read values from Deliver.swift and instead creates a "normal" Deliver file.

You asked a direct question, I replied with a link that mentions "Fastlane.swift (beta)" in the title. But I wouldn't disagree if you took my reply to an imaginary "Is fastlane.swift poorly documented?" question as well.
All the documentation is on GitHub, so if you have anything to add please open a PR and just add the missing information - that would be neat 💯
And for your deliver problem this issue exists and I tried to pull someone in who might know more, see above.


Fix Client Secret Desynchronization

Handle Client Secret through peer relation instead of StoredState

Pass to an Active State instead of Waiting if not a leader and stop observing leader_elected event.

Should fix these 2 bugs:
https://github.com/canonical/bundle-kubeflow/issues/354
https://github.com/canonical/bundle-kubeflow/issues/351

LGTM +1


weird error on compile

Describe your problem here.
this happens every time i compile my build, why does this happen?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
i edited the hscript hx and a few others just so i can get rid of the errors, mostly "kill" errors but this error that i have has nothing to do with the changes i did

there's no error attached, please send the error as an image or as a coodeblock
like this


[WIP] Add SwinV2

Fixes #6242

And I see an issue from the Microsoft SwinTransformer repo: https://github.com/microsoft/Swin-Transformer/issues/194
It thinks it's not necessary to divide the mask into 9 parts because 4 parts are already enough.
I kind of agree with that. Anyone has opinion on that?

According to the unittest logs, the state_dict has certain int values inside? I currently have no clue about this actually.
https://app.circleci.com/pipelines/github/pytorch/vision/18731/workflows/91dedbbe-264b-4665-a146-91117a6c8f63/jobs/1515479?invite=true#step-108-2975

It seems these mod operations are considered as nodes of the fx graph, and their output is float rather than torch.Tensor (actually why is it float rather than int?)

Concerning the linter, if you check the CI on tab Required lint modifications it will show you what's the problem:

Concerning your point the PatchMerging, I believe you are right. FX doesn't let you use the input of the tensor to control the flow of the program. This needs to move outside of the main function and be declared a non fx-traceable operator. I would patch this ASAP outside of this PR. cc @YosuaMichael

Hi @ain-soph, thanks a lot for the PR!
As of now, I am still reading SwinTransformerV2 paper and original code, and I will try to review afterwards.
Meanwhile, let me address some of the issue you raise:

On ufmt issue, can you make sure you install the following version: pip install ufmt==1.3.2 black==21.9b0 usort==0.6.4 (reference)
For the fx issue, I create a small patch: https://github.com/pytorch/vision/pull/6252

I'm currently using v2_logit_scale and use_v2 as 2 variables to check versions. Please comment if you have any better solution.

@ain-soph Thanks for you good PR. I have two advice:

we should fix the problem in  https://github.com/pytorch/vision/pull/6222
I think we should also add the pretrained_window_size on the SwinTransformer, I think it should be List[List[int]]

@xiaohu2015 I'll take a look at the problem you mentioned here, and add the pretrained_window_size as the argument (it's zero for all variants though). And I think it should be List[int]? (according to the original code in https://github.com/microsoft/Swin-Transformer/blob/b720b4191588c19222ccf129860e905fb02373a7/models/swin_transformer_v2.py#L526)

@xiaohu2015 I'll take a look at the problem you mentioned here, and add the pretrained_window_size as the argument (it's zero for all variants though). And I think it should be List[int]? (according to the original code in https://github.com/microsoft/Swin-Transformer/blob/b720b4191588c19222ccf129860e905fb02373a7/models/swin_transformer_v2.py#L526)

In torchvision, we define the window size as List[int], it means the window size of each dimension. The Tuple[int] in offical code means the window size of each block.

@xiaohu2015 I just read the window_size problem you mentioned and agree that the current implementation has some potential risk at some edge cases.
But I think it's a little out of scope and we should make another issue to get that solved.
I'm following the current V1 architecture.
If we plan to pass image_size as argument of Transformer model in the future to limit window size, we can definitely do that in a new PR.

The evaluation log of Swin_V2_T using pretrained weights from official repo:
torchrun --nproc_per_node=4 train.py --model swin_v2_t --test-only --weights Swin_V2_T_Weights.IMAGENET1K_V1
WARNING:torch.distributed.run:
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
| distributed init (rank 3): env://
| distributed init (rank 1): env://
| distributed init (rank 0): env://
| distributed init (rank 2): env://
Namespace(data_path='/data/user/data/image/imagenet', model='swin_v2_t', device='cuda', batch_size=32, epochs=90, workers=16, opt='sgd', lr=0.1, momentum=0.9, weight_decay=0.0001, norm_weight_decay=None, bias_weight_decay=None, transformer_embedding_decay=None, label_smoothing=0.0, mixup_alpha=0.0, cutmix_alpha=0.0, lr_scheduler='steplr', lr_warmup_epochs=0, lr_warmup_method='constant', lr_warmup_decay=0.01, lr_step_size=30, lr_gamma=0.1, lr_min=0.0, print_freq=10, output_dir='.', resume='', start_epoch=0, cache_dataset=False, sync_bn=False, test_only=True, auto_augment=None, random_erase=0.0, amp=False, world_size=4, dist_url='env://', model_ema=False, model_ema_steps=32, model_ema_decay=0.99998, use_deterministic_algorithms=False, interpolation='bilinear', val_resize_size=256, val_crop_size=224, train_crop_size=224, clip_grad_norm=None, ra_sampler=False, ra_reps=3, weights='Swin_V2_T_Weights.IMAGENET1K_V1', rank=0, gpu=0, distributed=True, dist_backend='nccl')
Loading data
Loading training data
Took 3.2938268184661865
Loading validation data
Creating data loaders
Creating model
/home/user/miniconda3/envs/dev/lib/python3.10/site-packages/torch/nn/parallel/scatter_gather.py:9: UserWarning: is_namedtuple is deprecated, please use the python checks instead
  warnings.warn("is_namedtuple is deprecated, please use the python checks instead")
/home/user/miniconda3/envs/dev/lib/python3.10/site-packages/torch/nn/parallel/scatter_gather.py:9: UserWarning: is_namedtuple is deprecated, please use the python checks instead
  warnings.warn("is_namedtuple is deprecated, please use the python checks instead")
/home/user/miniconda3/envs/dev/lib/python3.10/site-packages/torch/nn/parallel/scatter_gather.py:9: UserWarning: is_namedtuple is deprecated, please use the python checks instead
  warnings.warn("is_namedtuple is deprecated, please use the python checks instead")
/home/user/miniconda3/envs/dev/lib/python3.10/site-packages/torch/nn/parallel/scatter_gather.py:9: UserWarning: is_namedtuple is deprecated, please use the python checks instead
  warnings.warn("is_namedtuple is deprecated, please use the python checks instead")
Test:   [  0/391]  eta: 0:28:16  loss: 0.2255 (0.2255)  acc1: 96.8750 (96.8750)  acc5: 100.0000 (100.0000)  time: 4.3390  data: 2.0224  max mem: 817
Test:   [100/391]  eta: 0:01:28  loss: 0.6355 (0.6686)  acc1: 87.5000 (86.2314)  acc5: 96.8750 (97.4938)  time: 0.2590  data: 0.1852  max mem: 817
Test:   [200/391]  eta: 0:00:57  loss: 1.1148 (0.7348)  acc1: 71.8750 (84.2973)  acc5: 93.7500 (97.0771)  time: 0.2569  data: 0.1827  max mem: 817
Test:   [300/391]  eta: 0:00:27  loss: 0.9935 (0.8109)  acc1: 78.1250 (82.6100)  acc5: 93.7500 (96.2417)  time: 0.3584  data: 0.2839  max mem: 817
Test:  Total time: 0:02:00
Test:  Acc@1 81.644 Acc@5 95.858

Here is the transform:
        transforms=partial(
            ImageClassification, crop_size=256, resize_size=256, interpolation=InterpolationMode.BILINEAR
        ),

Here is the python script to transform official pretrained weights to the supported version of our implemented version:
from collections import OrderedDict
import torch

_dict: OrderedDict[str, torch.Tensor] = torch.load("swinv2_tiny_patch4_window8_256.pth")["model"]
new_dict = OrderedDict()

for key, value in _dict.items():
    k = key
    v = value
    if 'attn_mask' in k:
        continue
    if 'relative_position_index' in k:
        v = v.flatten()
    if k.startswith('patch_embed'):
        k = k.replace('patch_embed.proj', 'features.0.0')
        k = k.replace('patch_embed.norm', 'features.0.2')
    elif k.startswith('layers'):
        k = k.removeprefix('layers.')
        if 'blocks' in k:
            k = f'features.{2*int(k[0]) + 1:d}.'+k[2:].removeprefix('blocks.')
            k = k.replace('logit_scale', 'v2_logit_scale')
            if k.endswith('q_bias'):
                v_bias = _dict[key[:-6]+'v_bias']
                k = k.replace('q_bias', 'qkv.bias')
                assert value.numel() == v_bias.numel()
                v = torch.cat([value, torch.zeros_like(value), v_bias])
            elif k.endswith('v_bias'):
                continue
            if '.mlp.' in k:
                k = k.replace('fc1', '0')
                k = k.replace('fc2', '3')
        else:
            assert 'downsample' in k, k
            k = f'features.{2*(int(k[0]) + 1):d}.'+k[2:].removeprefix('downsample.')
    new_dict[k] = v
torch.save(new_dict, './test.pth')

I'm currently training a swin_v2_t using following command.
torchrun --nproc_per_node=4 train.py --model swin_v2_t --epochs 300 --batch-size 128 --opt adamw --lr 0.001 --weight-decay 0.05 --norm-weight-decay 0.0  --bias-weight-decay 0.0 --transformer-embedding-decay 0.0 --lr-scheduler cosineannealinglr --lr-min 0.00001 --lr-warmup-method linear  --lr-warmup-epochs 20 --lr-warmup-decay 0.01 --amp --label-smoothing 0.1 --mixup-alpha 0.8 --clip-grad-norm 5.0 --cutmix-alpha 1.0 --random-erase 0.25 --interpolation bicubic --auto-augment ta_wide --model-ema --ra-sampler --ra-reps 4  --val-resize-size 256 --val-crop-size 256 --train-crop-size 256

@xiaohu2015 could you elaborate more on the bug detail that you mention:

we should fix the problem in https://github.com/pytorch/vision/pull/6222 ( for input size=256 window size=8, it works well, but for input size=384 window size=12, it has a bug at the last block (it should be 6, but we do padding and use 8))

To be specific, can you explain for input_size=384 and window_size=12, what is the bug on the last block. And what do you mean by it should be 6, but we do padding and use 8 (what this number refer for).

@YosuaMichael and I are considering deprecating argument pretrained_window_size because it seems not used in standard variants. It's an argument about some fine-tuning related ablation studies in the original paper.
But this need some discussion. Any opinion on this? @xiaohu2015

Hi @ain-soph,  as discussed offline I am happy to take the review from here, and support you in the verification of the model weights that will be shipped.

This is my training result of swin_v2_t using the command from https://github.com/pytorch/vision/tree/main/references/classification#swintransformer
Epoch: [299] Total time: 0:20:24
Test:   [ 0/98]  eta: 0:06:43  loss: 1.2362 (1.2362)  acc1: 93.7500 (93.7500)  acc5: 98.4375 (98.4375)  time: 4.1221  data: 3.8947  max mem: 14880
Test:  Total time: 0:00:26
Test:  Acc@1 81.560 Acc@5 95.892
Test: EMA  [ 0/98]  eta: 0:06:01  loss: 1.2378 (1.2378)  acc1: 93.7500 (93.7500)  acc5: 98.4375 (98.4375)  time: 3.6888  data: 3.4649  max mem: 14880
Test: EMA Total time: 0:00:25
Test: EMA Acc@1 81.614 Acc@5 95.930
Training time 4 days, 13:34:44

Compared with the official result 81.2%, I think our result (81.56%) is satisfying.

@jdsgomes I've added the small and base variants in the previous commit. Please train the variants as well.
Btw, do you think the current tiny variant performance is acceptable?

Actually, maybe we can port the weights directly instead of training.
I've done such work and validate it at https://github.com/pytorch/vision/pull/6246#issuecomment-1179650923

I am going to launch the other training variants today and hopefully they will be ready by Monday. Regarding the difference in resolution size we can do a post training optimisation similar to what what we did in swin v1. There is no need for retraining we can just try to find the optimal val-resize-size

I am going to launch the other training variants today and hopefully they will be ready by Monday. Regarding the difference in resolution size we can do a post training optimisation similar to what what we did in swin v1. There is no need for retraining we can just try to find the optimal val-resize-size

As discussed offline my train runs failed to converge so we will work together with @ain-soph  to find the source of the problem  and update the PR once we managed to train all variants successfully.

I've launched another train based on the most up-to-date code for tiny architecture.
Currently I don't observe any strange trend showing failure of convergence.
Epoch: [142]  [2500/2502]  eta: 0:00:00  lr: 0.0006042917608127198  img/s:<PHONE_NUMBER>318139  loss: 3.6911 (3.6364)  acc1: 48.4375 (48.2548)  acc5: 73.4375 (71.9340)  time: 0.4798  data:
 0.0001  max mem: 14880
Epoch: [142] Total time: 0:20:10
Test:   [ 0/98]  eta: 0:06:25  loss: 1.5864 (1.5864)  acc1: 82.0312 (82.0312)  acc5: 96.0938 (96.0938)  time: 3.9314  data: 3.7073  max mem: 14880                                        Test:  Total time: 0:00:25
Test:  Acc@1 71.494 Acc@5 91.496                                                                                Test: EMA  [ 0/98]  eta: 0:05:43  loss: 1.4705 (1.4705)  acc1: 88.2812 (88.2812)  acc5: 97.6562 (97.6562)  time: 3.5061  data: 3.2585  max mem: 14880
Test: EMA Total time: 0:00:25                                                                                Test: EMA Acc@1 76.420 Acc@5 93.660

@ain-soph Could you please share the exact command you use to train it to ensure we are not missing anything from our side?

Ooops, there are also memory issues on GPU both on linux and windows. I know that a different test is actually failing but this can happen. Sometimes adding a bing new model leads to issues on other models because of failing to clear the memory properly. Usually the way around this is to reduce the memory footprint of the test by passing thought smaller sizes for input or disabling the particular test on the GPU.

torchrun --nproc_per_node=4 train.py --model swin_v2_t --epochs 300 --batch-size 128 --opt adamw --lr 0.001 --weight-decay 0.05 --norm-weight-decay 0.0  --bias-weight-decay 0.0 --transformer-embedding-decay 0.0 --lr-scheduler cosineannealinglr --lr-min 0.00001 --lr-warmup-method linear  --lr-warmup-epochs 20 --lr-warmup-decay 0.01 --amp --label-smoothing 0.1 --mixup-alpha 0.8 --clip-grad-norm 5.0 --cutmix-alpha 1.0 --random-erase 0.25 --interpolation bicubic --auto-augment ta_wide --model-ema --ra-sampler --ra-reps 4 --val-resize-size 256 --val-crop-size 256 --train-crop-size 256

Epoch: [221] Total time: 0:25:25
Test:   [ 0/98]  eta: 0:06:23  loss: 1.3831 (1.3831)  acc1: 91.4062 (91.4062)  acc5: 98.4375 (98.4375)  time: 3.9118 data: 3.6845  max mem: 14880
Test:  Total time: 0:00:25
Test:  Acc@1 77.738 Acc@5 94.460
Test: EMA  [ 0/98]  eta: 0:05:15  loss: 1.3172 (1.3172)  acc1: 90.6250 (90.6250)  acc5: 99.2188 (99.2188)  time: 3.2230 data: 2.9882  max mem: 14880
Test: EMA Total time: 0:00:25
Test: EMA Acc@1 80.012 Acc@5 95.286

torchrun --nproc_per_node=4 train.py --model swin_v2_t --epochs 300 --batch-size 128 --opt adamw --lr 0.001 --weight-decay 0.05 --norm-weight-decay 0.0  --bias-weight-decay 0.0 --transformer-embedding-decay 0.0 --lr-scheduler cosineannealinglr --lr-min 0.00001 --lr-warmup-method linear  --lr-warmup-epochs 20 --lr-warmup-decay 0.01 --amp --label-smoothing 0.1 --mixup-alpha 0.8 --clip-grad-norm 5.0 --cutmix-alpha 1.0 --random-erase 0.25 --interpolation bicubic --auto-augment ta_wide --model-ema --ra-sampler --ra-reps 4 --val-resize-size 256 --val-crop-size 256 --train-crop-size 256

Epoch: [221] Total time: 0:25:25
Test:   [ 0/98]  eta: 0:06:23  loss: 1.3831 (1.3831)  acc1: 91.4062 (91.4062)  acc5: 98.4375 (98.4375)  time: 3.9118 data: 3.6845  max mem: 14880
Test:  Total time: 0:00:25
Test:  Acc@1 77.738 Acc@5 94.460
Test: EMA  [ 0/98]  eta: 0:05:15  loss: 1.3172 (1.3172)  acc1: 90.6250 (90.6250)  acc5: 99.2188 (99.2188)  time: 3.2230 data: 2.9882  max mem: 14880
Test: EMA Total time: 0:00:25
Test: EMA Acc@1 80.012 Acc@5 95.286

Thank you for sharing the commands. Although you previously mention I missed that in v2 they used a different resolution size. I launched new jobs to train all the variants with the correct resolution size and seems are looking better now.

@ain-soph  thank you for your patience and great work! I managed to train all three variants and the results look good.
My plan is to update this PR in the next couple of ours with the model weights.

 Having SwinV2 in TorchVision is really awesome. Looking forward using them.

Btw, should we provide porting model weights from official repo as an alternative?

I think given we were able to reproduce the accuracy of the paper, there is no point offering both. What would be interesting on the future is to offer higher accuracy weights by using newer training recipes.


Added README assets and updated README

I believe the images should work once this is merged in, but I can hotfix directly on master if not since it'll only be changes to the README.


SSH key authentication does not work on Debian linux

I have tried both forms of ssh url (vs and devops). They both ask for a password for the site not the passphrase for the ssh key.
Where do I set the password for<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Here is the full text :
sudo git clone git@ssh.dev.azure.com:v3/{organization}/{project}/{repo} client
Cloning into 'client'...
The authenticity of host 'ssh.dev.azure.com (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is SHA256:ohD8VZEXGWo6Ez8GSEJQ9WpafgLFsOfLOtGGQCQo6Og.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added 'ssh.dev.azure.com' (RSA) to the list of known hosts.
<EMAIL_ADDRESS>password:
Permission denied, please try again.
<EMAIL_ADDRESS>password:
Permission denied, please try again.
<EMAIL_ADDRESS>password:
Permission denied (password,publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
I tried the passphrase and my password to the site but it did not work.

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

ID: 64e85ff9-9add-d15c-e92f-02d7dd05e0f2
Version Independent ID: 0cbd7427-6acd-3cfc-1a3b-8bf5ca60f3bd
Content: Connect to your Git repos with SSH - Azure Repos
Content Source: docs/repos/git/use-ssh-keys-to-authenticate.md
Product: devops
Technology: devops-code-git
GitHub Login: @apawast
Microsoft Alias: sdanie

@bill-thedbaguy -- Bill, thank you for your question. It looks like you're working through an issue with your scenario or implementation. You may find answers here:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

@apawast -- Apeksha, please look into this potential documentation issue.

I'm also asked to enter a password. Even if I test the connection:
Run: $ ssh -T<EMAIL_ADDRESS>I get<EMAIL_ADDRESS>password:
I tried my Azure DevOps password, but it does not work :) I did not use any password when creating ssh keys.

This might be the same issue. Could you please add a answer to Questions and troubleshooting section please? @WilliamAntonRohm @apawast

Sorry everyone for a late reply.  I was able to solve this problem by creating a config file with the following entries:
Host vs-ssh.visualstudio.com
User
IdentityFile ~/.ssh/id_rsa
IdentitiesOnly yes
Host ssh.dev.azure.com
User
IdentityFile ~/.ssh/id_rsa
IdentitiesOnly yes

It worked for me as well! Thank you @bill-thedbaguy
Would it be possible to add it to the Troubleshooting?

Hi, guys
I'm having this same problem
where did you create this file, @bill-thedbaguy ?
under what name?

It's this file: ~/.ssh/.config

thanks, @akselon !
I second the suggestion of adding it to the Questions and Troubleshooting section

This is in the troubleshooting questions, but the proposed config file is ugly. I'll clean it up.

its so frustrating wasting time on such bugs in the tool :(


Rat Count House 🐀 Problem

Problem Statement 🐀
The function accepts two positive integers ‘r’ and ‘unit’ and a positive integer array ‘arr’ of size ‘n’ as its argument ‘r’ represents the number of rats present in an area, ‘unit’ is the amount of food each rat consumes and each ith element of array ‘arr’ represents the amount of food present in ‘i+1’ house number, where 0 <= i
Note :

Return -1 if the array is null
Return 0 if the total amount of food from all houses is not sufficient for all the rats.
Computed values lie within the integer range.

Example:
Input:
r: 7
unit: 2
n: 8
arr: 2 8 3 5 7 4 1 2
Output:
4
Explanation:
Total amount of food required for all rats = r * unit
= 7 * 2 = 14.
The amount of food in 1st houses = 2+8+3+5 = 18. Since, amount of food in 1st 4 houses is sufficient for all the rats. Thus, output is 4.

@soumik2012 Can I take this issue ?? I will solve this in python.

Sure @swastikbaithalu you can take it

Can I solve this in C++ ?

Sure Go Ahead @Aniumbott

This issue is already solved @Aniumbott !!


"WT_Alternating_Lower/Upper" --> "Upper"

In the ontology "cluster6ontology_complete.rdf" click >Workout_Template >WT_Alternating_Lower/Upper.
Then in the Class Annotations Tab, at the SubClass Of Section, click the question mark for Workout_Template to get the explanation.
here comes the misbehaviour:
it says :
That's already stated in the ontology. Upper is workout template
but it is supposed to say :
That's already stated in the ontology. WT_Alternating_Lower/Upper is workout template
check out the screen shot

outdated/not relevant anymore


WinMergeU.exe not found, if installed to non-default directory

winMergeReport.LaunchApplication trows FileNotFoundException("WinMergeU.exe") if WinMergeU.exe is installed to non-default directory.
Windows already store the correct location in registry under HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\App Paths\WinMergeU.exe so your search (public string FindWinMerge()) is not required.

Good call out.  Thanks for letting me know.

Fixed:
https://github.com/GregFinzer/Compare-Net-Objects/commit/d1bb913c0980229c0e4e465868c7d0160ac5da8a


Food and Agriculture

          Food and Agriculture

@Anmol0904 @achalkamboj @psakash2003 @Surya2458 @saivaibhav7
Originally posted by @VikashPR in https://github.com/VikashPR/18CSE301J-Projects/issues/260#issuecomment-1487903335

@all-contributors please add @Anmol0904 for content, example.

@all-contributors please add @achalkamboj for content, example.

@all-contributors please add @psakash2003 for content, example.

@all-contributors please add @Surya2458 for content, example.

@all-contributors please add @saivaibhav7 for content, example.


Change time of efect in loop function

How I can change time of effect in loop function or add variable as time?
#include <Arduino.h>
#include<jled.h>
// blink internal LED every second; 1 second on, 0.5 second off.
auto led = JLed(LED_BUILTIN).Blink(100, 500).Forever();
void setup() {}
void loop()
{
led = JLed(LED_BUILTIN).Blink(1000, 500).Forever(); //this not work
led.Update();
}

This does not work because you are re-assigning the led object again and again in the loop() function. You need some kind of a trigger and re-assign only once, e.g. (not tested):
...
auto changed = false;
...
void loop() {
     static auto time_start = millis();
     if (mills() - time_start > 5000 && !changed) {
         changed = true;
         led = JLed(LED_BUILTIN).Blink(1000, 500).Forever();
    }
    led.Update();
}

I'm understand. Thanks for reply.


iOS "ImageMountFailed" is displayed

Connect and start, "ImageMountFailed" error will be displayed, android normal statistics.

Already solved, thanks open source


Embark not using account from blockchain.json

I'm trying to make Embark use ropsten account from blockchain.json but it keeps reading account list from geth as a result it unlocks account from keystore not the json. Even if I delete the file it just makes new account.
"testnet": {     "enabled": true,     "networkType": "testnet",     "networkId": 0,     "light": true,     "rpcHost": "localhost",     "rpcPort": 37505,     "rpcCorsDomain": "http://localhost:8000",     "account": {       "address": "0xec33cc4be8c301beb1d5ee13036030ad467c0130",       "init": false,       "password": "config/testnet/password"     }

@dstawinski did you ever figure this out?

A lot has changed, please see the docs for configuring accounts here


bug: proto3 default map key breaks deserialization

I have a proto3 map field with int32 keys:
message GameState {
    map<int32, PlayerData> player_data = 6;
}

This normally works, except when the map key is 0.
For example, this code is broken because from_bytes returns error REQUIRED_FIELDS:
var s = API.GameState.new()
s.add_player_data(0)
print(s.from_bytes(s.to_bytes()))  # Prints -9, aka "REQUIRED_FIELDS".

However, when I change the map key to something else (e.g. 1), deserialization works fine.
I assume this is due to improper handling of default values as map keys. I haven't tested it, but I assume the same issue is present for other default types (e.g. empty strings).

Bug fixed in v 0.5.0


Demo does not work

Dear Ttimot24,
thank you very much for working on the CMS, I would like to see the features you have build by following your links:
Frontend: https://horizontcms.herokuapp.com/
Backend: https://horizontcms.herokuapp.com/admin
Unfortunately that leads only to a installation page and I'm not aware of your MySQL settings,....
Thank you for having a look.

Hi Nowi5,
Thank you for your interest in my CMS. I fixed the wrong credentials, which was the root cause why you were redirected to the install page. Now you can try again.


Is not possible delete a product after uninstall it

After uninstall a product is not possible delete it
The procedure to reproduce the error is:

Create a product
Install it in a VM
Uninstall it
Delete the product

An error 500 is obtained with the exception violates foreign key constraint

Tracked in CLAUDIA-3769


Nested embeds

Updates HAL adapter to sideload _embedded records, even when deeply embedded.

@mixonic for you to review if you have a chance. After merge I will tag and release a new version

Bon!


Coral USB Accelerator: silicon shortage or end of production?

I'm trying to answer the question above, but searching on Google I got nothing. Everywhere I checked (e.g. all the links from coral.ai website) the stocks are zero. So, I'm wondering whether Google is killing the USB Accelerator...
Thanks in advance for any clues!

I hope not, unless they are coming up with a new version to support full tensorflow models.
The M2 modules are nice and priced much lower (< 1/2), but my initial experience with them has not been as smooth as with the USB version -- see issues #261 and #262  Not to mention that M2 slots are not common in IOT class machines at the moment and one that does (nanoPi) has issues here about using the modules.

This is a silicon shortage (the same as everyone else in the industry is dealing with right now). We are currently building as many as we can - we have recently completed and shipped a new batch  that should be available with our distribution partners shortly.

It looks like that batch sold out very quickly, which isn't at all surprising. @mbrooksx, may I ask if you know when the next batch might be shipping to distributors? Thank you again for taking the time to answer these questions, it's very much appreciated.

@mbrooksx, any update if we will be seeing more of these? I haven't seen any in stock for a long time now, and have never really had the chance to buy one being in Australia.

@tbgoose We have a small shipment of USB modules to some retailers such as OKDo and Pi3g this month. It's possible that there are already long waiting lists at the retailers.
We recommend to place a pre order with Mouser or other retailers and wait for delivery.
Thanks!

I placed an order with RS components in August last year. They've continued
to push delivery dates out as they approach and are now saying expected
stock end of July 2022. Fingers crossed...
On Wed, Apr 27, 2022 at 11:55 PM Hemanth-Jonnala @.***>
wrote:

@tbgoose https://github.com/tbgoose We have a small shipment of USB
modules to some retailers such as OKDo and Pi3g this month. It's possible
that there are already long waiting lists at the retailers.
We recommend to place a pre order with Mouser or other retailers and wait
for delivery.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/google-coral/edgetpu/issues/363#issuecomment-1111034629,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYXYKIODOWHIS7YII7WEBTVHFBOTANCNFSM43J64NYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I placed an order with Mouser in Oct 2021, its been pushed back to Aug 2022.
It's time to start considering Plan B.

Probably is, it's not even properly listed on mouser AU to be able to place
a pre-order there. What is plan b? Get a whole SoC board just for TPU edge
processing?
My use case was for frigate detection on my rpi4 home assistant setup. The
USB is just perfect for that.
On Fri, 29 Apr 2022, 2:42 am wb666greene, @.***> wrote:

I placed an order with Mouser in Oct 2021, its been pushed back to Aug
2022.
It's time to start considering Plan B.
—
Reply to this email directly, view it on GitHub
https://github.com/google-coral/edgetpu/issues/363#issuecomment-1112429727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYXYKIKFTIXR5PSAHGPN3DVHK5XJANCNFSM43J64NYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

arrow.com is apparently has a lot of stock and selling at 3x the price ... gouging...

The Movidius NCS2 is far inferior to the Coral TPU accelerators for my purposes.
@wb666greene, I know this is an old message, but I am curious and I would like to ask you why do you consider the NCS2 far inferior to the Coral TPU. I've been using an OAK-D Lite (same Movidius Myriad X as the NCS2, afaik) and I would say it's quite good, therefore I would expect a NCS2 connected to something like a RPI4 to be even better. Thanks!

My benchmark is frame rate using MobileNetSSD_V2 trained on the coco data set with USB3 TPU or NCS2 coprocessors.  The TPU uses Google's downloadable coefficients, the NCS2 uses the model converter to make the coefficients.  The OAK camera improves things quite a lot because the image data doesn't have to cross the USB3 bus.  But 1 co=processor per camera is not cost effective for my system.  Hence I round-robin sample the cameras and feed the images to the co-processor.


Use tailwind in (html/jsx) in markdown / mdx files

When creating pages for my site I want to (sometimes) use full html with tailwind so i can have fancy layouts or sections
Acceptance

[ ] can use html with tailwind in my content
[ ] documentation for this

Tasks

[ ] Change tailwind.config.js to track content/ folder - see below.
[ ] Fix the constant hot reloading of pages

Notes
If we have a setup (as we have) like:
tailwind.config.js
content/ <= content is in here

should be as simple as tweaking tailwind.config.js as follows:
module.exports = {
   content: ['./pages/**/*.{js,ts,jsx,tsx}', './components/**/*.{js,ts,jsx,tsx}',
       'content/**/*.{md,mdx}'],  // <= added
}

This does work ... however it also seems to trigger nextjs into hot reloading the page all the time even with no changes. Need to debug and fix this - at least for development mode.

@rufuspollock I've tested this and for me, it only seems to recompile when I'm switching pages in obsidian (which doesn't happen when I don't add /content folder to taiwlind config). But there is no infinite reload. It only recompiles once - still weird though.


ESM exports + updates

update dependencies
add esm exports, fixes issue #8

That is actually not neccessary if we omit type: "module" and thus, don't force node into a certain mode. Tested it again and it seems to work.

That is actually not neccessary if we omit type: "module" and thus, don't force node into a certain mode. Tested it again and it seems to work.

Then I don't think ESM will work :)
If you need dual support, one of them should be names cjs or mjs (depending on the base).
At least Vitest will default to cjs with this config, since importing exact file (not module) will throw

OK, I changed the exports to cjs and mjs so vitest and node may detect it properly; also increased the required node version to at least 14 to switch to a version that even allows these imports.

Thank you


Remove dependency on Java 9/a specific version of a specific project of Java 8

Pull Request Description
Remove dependency on Java 9/a specific version of a specific project of Java 8. sun.misc.IOUtils is a part of OpenJDK, but isn't standard in Java. Further, the method used here was only added about 1 year ago (see https://hg.openjdk.java.net/jdk8u/monojdk8u/rev/8994a5a78287). Strangely, it was added to the Java 8 repo and not the Java 9 repo, but is annotated with @since 1.9!

Future Release Comment
Remove dependency on Java 9/a specific version of a specific project of Java 8. sun.misc.IOUtils is a part of OpenJDK, but isn't standard in Java. Further, the method used here was only added about 1 year ago (see https://hg.openjdk.java.net/jdk8u/monojdk8u/rev/8994a5a78287). Strangely, it was added to the Java 8 repo and not the Java 9 repo, but is annotated with @since 1.9!
Fixes:

Remove dependency on Java 9/a specific version of a specific project of Java 8 (sun.misc.IOUtils)

Works great.
But style choice - I don't like assignments inside of a while loop's condition. Let's discuss

@AsafMah Let me know when you want to discuss. I think this is the standard Java way to read from an InputStream in a loop, because it's so convenient as a stopping condition (among others).
Here are the first 2 stackoverflow search results: https://stackoverflow.com/questions/309424/how-do-i-read-convert-an-inputstream-into-a-string-in-java and https://stackoverflow.com/questions/6160432/java-inputstream-reading-problem

I know it's a common pattern, I'm not a big fan of it usually, but here unlike in the examples it's not the only thing in the while, which makes it less readable in my opinion


openssl: bump to 1.0.2n

This bumps OpenSSL from 1.0.2k to 1.0.2n in 1.12/master.
The related PR for 1.11: https://github.com/dcos/dcos/pull/2634
The related PR for 1.10: #2548
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:
https://jira.mesosphere.com/browse/DCOS_OSS-1903
Checklist for all PRs

[x ] Read the DC/OS contributing guidelines
[x ] Followed relevant code rules Rules for Packages and Systemd

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ready for review


[Release] sdk/resourcemanager/compute/armcompute/0.1.0

…m spec commit: a5f8ef67c8170e4081527e400473c6deddcfabfd

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run prepare-pipelines

/azp run go - armcompute - ci

/azp run go - armcompute - ci


widens the cardinality bar and number a bit for 7-digit figures

yeah I think we should be moving to some nice-number formatting that accomplishes the same thing here.


Adding a new Enum with possible Gimmicks for Tasks

General

References to meitho177/Aquarium-Party#1

Type: Feature-Request
Author: @meitho177
Time-Estimate: 3h
Description

Create an additional Enum called Gimmick for the data.

enum class Gimmick {
    ADDITIONAL_PUNISHMENT,
    ADDITIONAL_QUESTION,
    ALTERNATIVE,
    BET,
    BONUS_QUEST,
    CHAMPION_REWARD,
    CONSEQUENCE,
    INVOLVING_SOMEONE,
    REVENGE,
    SECRET,
    SPLITTED,
    STAKE,
    SWAP_PLACES,
    TASK_BUDDY,
    WILDCARD,
}

Modify the Task-class to contain an attribute gimmickTexts: Map<Gimmick, String>. This map does not NEED to contain every Gimmick Element. If a certain Gimmick is not contained, that Gimmick MUST NOT be applied to the question

Delete all existing database entries, since they do not have gimmicks assigned (which should be handled as not having any gimmicks enabled).

Add Variable Key Condition.Target.Relationship.Single

Implemented in version 1.2.0.


Instead, we have released the latent codes of the multi-view normalized face images which are used for extracting texture UV-maps. Please refer to README.
You can generate face images by StyleGAN2 using the official pre-trained checkpoint.
Welcome to follow and discuss.

The script for generating face images from latent codes is available.
Please refer to README.


Use of langString

Why do we need rdf:langString?
If the reason is to have language-specific strings, why not to use rdf:PlainLiteral as the OWL spec recommends?
Then we wouldn't need to define rdf:langString as we currently do in: https://github.com/edmcouncil/fibo/blob/a3a23e6c69b4fd9dba93418c8922a550cf513389/FND/AgentsAndPeople/Agents.rdf#L56-L61

@mereolog You will see this in Commons as well.  We are not using it explicitly in FIBO, but this is needed for some applications. As it happens, rdf:PlainLiteral was defined in the OWL specification because there was a gap between what was in RDF 1.0 and the second edition of the XML Schema Datatypes. The OWL 2 working group added it to OWL as a stop gap, but no one has really implemented it.  Since the OWL 2 specification was revised in 2012, there is a new version of RDF, RDF 1.1, published in 2014.  In that version of RDF, they implemented the notion of a language tagged string, or rdf:langString, which is what most triple store and other vendors have implemented.  But it's a gap in OWL 2 and so we've included it in this union in the Text datatype for those that need it.  That approach, using such a union, was recommended by Evren Sirin at Stardog, in fact.


TST: linalg.blas: fix test concurrency, mark *ger as unsafe

This makes all tests except test_ger run in parallel when running the test suite with pytest-run-parallel. The good news is that all other functions appear to be robust with all BLAS libraries.
xref gh-21936 for the test_ger issue. We don't use *ger* routines anywhere for other higher-level functionality, so that's good news.

Note that this is trivial plumbing (moving to default_rng), no need for the reviewer to think about the *ger* routine here, that's what gh-21936 is for.

The Intel compiler CI failure was unrelated and fixed elsewhere.
merged, thanks Ralf


the 'pods-runner' target has transitive dependencies that include statically linked binaries

the 'pods-runner' target has transitive dependencies that include statically linked binaries: (xxxx/ios/Pods/AMapLocation/AMapLocationKit.framework)
flutter: 1.22.4
  amap_location_flutter_plugin:
    git:
      url: https://github.com/amap-demo/amap-location-flutter.git

iOS编译不通过，直接报错，注释use_frameworks!不可取，因为其他插件需要依赖

这位仁兄，请问问题有解决么。我遇到同样的问题

这位仁兄，请问问题有解决么。我遇到同样的问题

搞不定，已经弃用了

加一，搞不定～


Private game settings don't apply

How to reproduce :

open a private lobby
setup the match, edit some values like core meter multiplier or anything else
launch the game
settings aren't applied

I'll look into fixing this as I have time, I don't have a ton of free time this week, but it's more than last week
Not sure if Vanilla+ actually overrides anything for private matches though, will have to do a lot of checking to see if it does

I'll look into fixing this as I have time, I don't have a ton of free time this week, but it's more than last week
Not sure if Vanilla+ actually overrides anything for private matches though, will have to do a lot of checking to see if it does

my guess is that it's northstar if it's not V+ overriding values, but in any case this is preventing private matches from being tweaked

Note: Plugins v3 has broken the code used in Vanilla+ for plugins v2
2.3 is now going to be a hotfix for that, and for mods like Speech Bubbles
2.4 will (hopefully) be this

Note: I have not forgotten about this
Just had finishing week of a musical I was in the pit for finish this last Sunday, been crunching out assignments I didn't get to work on due to practicing that
Realistically, will do a deeper look at this tomorrow

Note: I have not forgotten about this Just had finishing week of a musical I was in the pit for finish this last Sunday, been crunching out assignments I didn't get to work on due to practicing that
Realistically, will do a deeper look at this tomorrow

no pressure

I'm not entirely sure how possible this is to fix;
Trying an older version of VanillaPlus that still used Northstar.Client, still was able to reproduce this issue
Trying Northstar with absolutely no core mods, I wasn't able to even get past the main menu, so I'm not entirely sure if this is specifically a VanillaPlus bug, or one that's just inside of Northstar

I was correct in my being unable to find this inside of Northstar.Client (the core mods VanillaPlus is based off of)
Running Northstar with no core mods and no VanillaPlus (in fact, just no mods period), this was still an issue. I'll try to look into the NorthstarDLL code to find out why, however this is unrelated to VanillaPlus


Added more debug information for CASSANDRA-12430

As discussed on the ticket, we need more debug information to understand why the test failed.

+1


Provide JSBin embeds JS, CSS, HTML, etc

I'm working on a documentation engine that I would like to show demo and example code in a JSBin for viewers of the generated documentation to play with.
Is there a way to create a JSBin embed where the html, js, and css source is read from the embedding page?  For example, if someone wrote:
@example
console.log('hello world');

I could generate:
JS Bin

If this is not possible, would you consider this feature?  I believe this is how Angular is able to do similar behavior with Plunkr example.
Thanks for your help and for creating an amazing tool.

Huh, sorry, I said the w/e, then I got slapped with Christmas! I'll pick this up this week!

...and I totally didn't.
Did you get anywhere with this (looking at the scoop links)?

We played around with it, yesterday, and found that while the functionality in the docs hasn't been successfully implemented in embed.js, it is possible to pass code to jsbin.com via url parameters.
e.g.
http://jsbin.com/?html,css,js,console,output&html=<!DOCTYPE html><meta charset%3D"utf-8"><title>JS Bin<%2Ftitle><%2Fhead>
Hello%2C DOM.<%2Fp><%2Fbody><%2Fhtml>&css=.pretty { color%3A pink%3B }&javascript=console.log('hello%2C console')%3B

Hey @remy, hope everything is going well!
Question: is it possible to embed an jsbin with dynamic content? e.g the link that Alfredo shared before, can we embed that?
In the example here http://jsbin.com/help/how-can-i-embed-jsbin, the href is set to a jsbin that already exists (http://jsbin.com/iwovaj/73/embed?js,html,output).

@remy, sorry for not being clear, I'm actually working in the documentation engine that Justin mentioned before, the idea is that you can wrote something like:
@example
   console.log('hello world')

then, I'd generate something like

    Hello world

to create a jsbin embed of the given code in the generated documentation.
I want to be able to pass it through the url, so when I change the code in the docs; lets say:
@example
   console.log('hello world updaded')

The resulting embed has the updated code. Makes sense?

@m-mujica does the code in the documentation change in real-time, or do you mean in some kind of build process?

@remy nope, it won't change in real-time (or by any build process).
Currently it seems that the only way that I can create an embed is with a saved jsbin, I want to know if it's possible to create an embed with code provided through URL params (or by any other mechanism); that's pretty much it.

@m-mujica oh, you want to only inject a portion of code, not the entire thing? i.e. into a pre-existing template bin...if you follow. Is that right?

Otherwise, what you've already got looks right to me...which is why I think I've misunderstood...!

@remy the problem is, the following snippet does not work

    Hello world

I tried it out in jsbin itself (how cool is that?), this is the bin link http://jsbin.com/jonoworefi/1/edit?html,console,output; if you open the browser's dev tool, there is an error in the console:
Refused to display 'http://jsbin.com/?console' in a frame because it set 'X-Frame-Options' to 'DENY'.
How can I workaround this? Thanks for your help.

Oooh, it's because it's hitting the edit page, which is blocking iframes, where as the embed page doesn't. Hmm. Yeah, okay, I see the problem.
I'm working on a PR that changes the embed view right now, so I'll see if I can safely get this working.

:+1: embedding dynamic content would be great to have! @remy any plans on this? Hitting @m-mujica X-Frame-Options Deny right now.
Would really love to use this in addition / alternative to tonicdev on https://doclets.io Thanks for making the awesome JSBin!

Hey @remy, +1 here, its would be great if jsbin allows bin creation with dynamically passed code.
You replied you are working on something related? How this is going ?


Failure when running apply-all for the first time within folder containing a terragrunt.hcl file

Hello,
I'm not sure if this is by design or if it is a bug, but I couldn't find documentation / related issues about this.
Here is our folder structure:
.
|-- project
|   `-- web
|       `-- review
|           |-- dynamic
|           |   |-- cluster
|           |   |   `-- terragrunt.hcl
|           |   |-- provider.tf
|           |   |-- services
|           |   |   `-- api
|           |   |       `-- terragrunt.hcl
|           |   `-- terragrunt.hcl
|           |-- static
|           |   |-- certificates
|           |   |   |-- api
|           |   |   |   `-- terragrunt.hcl
|           |   |   |-- front-end
|           |   |   |   `-- terragrunt.hcl
|           |   |   `-- websocket
|           |   |       `-- terragrunt.hcl
|           |   |-- network
|           |   |   |-- vpc
|           |   |   |   `-- terragrunt.hcl
|           |   |   `-- vpc_peering
|           |   |       `-- terragrunt.hcl
|           |   |-- storage
|           |   |   |-- cache
|           |   |   |   `-- terragrunt.hcl
|           |   |   `-- db
|           |   |       `-- terragrunt.hcl
|           |   `-- terragrunt.hcl

It seems to pull all the states of the relevant dependency when running terragrunt apply-all from within project/web/review/dynamic/services/api but ultimately it fails because it tried to use my mock_outputs values of the dependencies.  Obviously, my mock_outputs values aren't pointing to real resources.
Example of output: https://gist.github.com/cabouffard/9e21d3472d837708d8f5be414389fc3e
Ultimately, when running terragrunt apply-all from project/web/review, everything works as expected and it's able to pull the state of all my modules and apply (i.e. deploy) the ones that haven't been deployed.
I had assumed that running terragrunt apply-all from a specific folder, would apply-all the dependencies specific to that module.
The error I was seeing wasn't very straightforward and it took me a while to try to run it at the "root" folder which then worked.
I was wondering if perhaps this was a "bug" or it was a misunderstanding that I had about how this works. Perhaps, it's worth documenting this somewhere?
Thanks!
Have a great day!

Did you run terragrunt apply-all with --terragrunt-non-interactive? If so, the behavior of the non interactive flag is to ignore external dependencies. If you wish to include them, you need to use --terragrunt-include-external-dependencies in addition to the non interactive flag.
We could probably clarify that in the --terragrunt-non-interactive CLI arg docs. Would you be up for a quick PR to add that in?

Yes, I was using --terragrunt-non-interactive without --terragrunt-include-external-dependencies. I'll give it a shot and let you know if this works!
If it does, then I'll propose a PR with updated documentation.
Thanks!

Closing as stale. If this is still an issue, please create a new ticket with the updated context. Thanks!


chore: remove builder circular dep

What/why?
This aims to remove the following circular dependencies:

There are two big changes:

Moved types into it's own file to reduce the length of the control-serialization.ts file.
Moved modules that were recursively calling each other into the same file.

@chanceaclark, should removing all circular dependencies be a non-goal here? I feel like for certain structures, like mutual recursion as mentioned in the SO article you shared, it's natural to have circular dependencies. As long as the modules just have function declarations that depend on each other, there shouldn't be a problem. I think that the big issue that we had was that instead of having the serialization domain depend on each other, the dependencies were through control modules via index files that re-exported everything. This meant that things that we wouldn't want to have circular dependencies ended up having them. Think about serialization specifically, here's how I think it could work:

Types are leaves and while we might want to colocate type files with the control folders, we still want to import types directly (e.g., import { SomeType } from './path/to/module/type' instead of import { SomeType } from './path/to/module').

An alternative to colocation is to have a "types" module that is grouped by domain (e.g., import { SomeType } from './types' where the types folder has a file or folder for each control with the relevant types.

Serialization and deserialization rely on mutual recursion, so there will be circular dependencies. These circular dependencies, though, should be just between the serialization and deserialization functions and not the entire modules.

Just like types, we could use colocation where some-control/serialization.ts has the logic and we import directly from that file (not from some-control/index.ts) or have a "serialization" module where we have a file/folder per control

Serialization will depend on types, but since types are leaves, there's no circular dependencies
Serialization will depend on other serialization functions, which might in turn depend on the "root" serialization function, but that's ok, it's a small local cycle that is unavoidable due to the recursive nature of the function

We could put everything in a single file and avoid circular file dependencies, but this really doesn't buy us anything other than removing the circular dependency, but I think that should be a non-goal. I would rather have the files be well-organized rather than having huge files that grow as we add more controls and inevitably have to include legacy controls

Looking at the screenshot you shared above, that circular dependency doesn't seem problematic since it's "local" and simply there because files are split-up for organization purposes.
What are your thoughts on this?


overhaul Mili tools (plugin and makemili)

In meeting with Kevin Durrenberger, it is becoming clear that the Mili plugin and Makemili support tool are in need of an overhaul for several reasons mili data is not presented to Griz/Mili user's in a natural way e.g. 'sanded' data should be automatically handled by the plugin and not exposed in the GUI as a subsetting option using Mili in parallel VisIt causes rendering artifacts at domain boundaries due to lack of ghostzones (need ucd mesh ghostzone comm.) free nodes was a one-off feature not part of Mili 'standard' use but is nonetheless 'in your face' in the GUI in any Mili file We are not handling Mili 'materials' in the way that is most natural We are probably very innefficient in PopMD in using Mili library
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1806
Status: Pending
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: overhaul Mili tools (plugin and makemili)
Assigned to: Mark Miller
Category: -
Target version: -
Author: Mark Miller
Start: 04/15/2014
Due date:
% Done: 0%
Estimated time:
Created: 04/15/2014 04:43 pm
Updated: 07/24/2014 12:07 pm
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 3 - Medium
Expected Use: 3 - Occasional
OS: All
Support Group: Any
Description:
In meeting with Kevin Durrenberger, it is becoming clear that the Mili plugin and Makemili support tool are in need of an overhaul for several reasons mili data is not presented to Griz/Mili user's in a natural way e.g. 'sanded' data should be automatically handled by the plugin and not exposed in the GUI as a subsetting option using Mili in parallel VisIt causes rendering artifacts at domain boundaries due to lack of ghostzones (need ucd mesh ghostzone comm.) free nodes was a one-off feature not part of Mili 'standard' use but is nonetheless 'in your face' in the GUI in any Mili file We are not handling Mili 'materials' in the way that is most natural We are probably very innefficient in PopMD in using Mili library
Comments:

Most of this has been resolved in #3370. I don't believe we're handling ghost zones at processor boundaries, though. @markcmiller86 were there plans to add ghost information to the mili files?


Feature/xref preparse

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
New RefSeq and UniProt parsers using pre-parsed data from a database rather than the downloaded files
Also introduce new configuration for plant species
Use case
Pre downloaded and parsed data can speed up the parsing stage, especially when run for multiple species
Benefits
Parsing is quicker
Possible Drawbacks
Requires a central xref database
Testing
Have you added/modified unit tests to test the changes?
If so, do the tests pass/fail?
Have you run the entire test suite and no regression was detected?
Yes

Coverage decreased (-0.2%) to 82.107% when pulling 964ab1d57a2713728d204c315e8235a345c0b67a on feature/xref_preparse into 8517abcdf1b1255535b561418d3c1bc50dd00a98 on master.


Add option for span name prefix

Our application is build of multiple module and therefore we would like to have different prefixes in each module.
Could you please an option to search/fix for prefixes in span names?
A single prefix would be sufficent, so the lint command could be executed with a different prefix for each module folder.

Could you give an example of a full span name your application wants?

nvm I was just using it wrong, it gets a prefix if there is struct attached function.
I was using it with basic functions only.


Line thickness (cursor, indent guides) changes randomly when zoomed in/out

VSCode Version: 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331)
OS Version: macOS Sierra 10.12.3

Steps to Reproduce:

Zoom out (in the settings it should say "window.zoomLevel": -1 or similar).
Move the cursor around in a text file. The cursor will vary randomly between 1px and 2px in thickness (2 and 4 retina pixels).

The same thing happens for indent guides. Here's a recording on a retina screen, zooming out from zoom level 2 to zoom level -3. Sometimes the thickness is inconsistent, sometimes the lines disappear completely:

If you care about pixel-perfect rendering (i.e. are not projecting to a live audience), then I can only suggest to never use zooming. i.e. use editor.fontSize instead. See also other issues tagged with zoom.

Setting editor.fontSize is what I ended up doing. I do think it's an amazing feature to be able to just zoom the entire UI, if there only wasn't the line rendering issue.
All the zoom-tagged issues seem to be duplicates of basically the same problem -- it's all about various lines disappearing. Maybe some of them (including this one?) can be closed.


Fix for when there are no dashboards

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

If there are no dashboards (because the examples were deleted, or because the user has strict permissions) the loading page shows a permanent spinner.

TEST PLAN

Tested locally.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Codecov Report

Merging #8815 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8815      +/-   ##
==========================================
+ Coverage   65.84%   65.88%   +0.04%
==========================================
  Files         483      483
  Lines       24177    24177
  Branches     2777     2777
==========================================
+ Hits        15920    15930      +10
+ Misses       8079     8069      -10
  Partials      178      178

Impacted Files
Coverage Δ

superset/assets/src/welcome/DashboardTable.jsx
87.5% <100%> (ø)
:arrow_up:

superset/db_engine_specs/postgres.py
83.33% <0%> (+27.77%)
: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 ed54f6e...ad2e989. Read the comment docs.

👀 @khtruong


Identify multiline literals with single property correctly.

This PR addresses https://github.com/palantir/tslint/issues/856. It's only a one line fix, but quite a few test cases were added to cover the various flavors of multiline literals.

Looks good to me @pe8ter! Thanks for going through and adding those test cases as well.

Glad to help!


Field always required even if condition any.when(ref, { then: Joi.required() }) is not met

Context

hapi-swagger version: 7.7.0
any other relevant information:

joi version: 10.4.1

What are you trying to achieve or the steps to reproduce ?
I have a conditional query parameter which is required only when another query parameter has a specific value (comparable to the second code snippet in the joi docs):
{
    config: {
        auth: { mode: 'try' },
        description: 'FooBar',
        notes: 'Returns a single document',
        response: {
            modify: true,
            schema
        },
        tags: ['api'],
        validate: {
            params: {
            query: {
                collection: Joi
                    .number()
                    .integer()
                    .description('Collection ID')
                    .when('state', { is: 'foo', then: Joi.required() }), // only required if "state" is 'foo'
                state: Joi
                    .string()
                    .valid(['foo', 'bar', 'baz'])
                    .description('State')
            }
        }
    },
    handler: {
        async: controller.find
    },
    method: 'GET',
    path: `documents/{id}`
}

What result did you get?
Query parameter "collection" is required, even if "state" 'foo' is not selected.
What did you expect ?
Query parameter "collection" is only required if 'foo' is selected in the "state" query parameter.
I already tried to circumvent this behaviour using alternatives.when(), but any.when() is already converted correctly into this as far as I could find out.
Side note: There is a constraint that .when() has to be the last parameter in the definition (Joi.any().when(...).description('foo') does not work). Is this intended to be that way?

I see a similar behavior, combined with https://github.com/glennjones/hapi-swagger/issues/536


Leetcode Question Number 1952

Leetcode Question Number 1952 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Please assign me this issue


make metapath and path selections persistent

Makes it such that switching source/target nodes doesn't change the checked/highlighted metapaths or paths
Makes it such that checking/unchecking metapaths doesn't change the checked/highlighted paths

Please also resolve the conflicts between your branch and the master.


The questionnaire in Manitoba does not recommend the topics that are needed.  Case 1

I follow the steps mentioned belove:
Clear all data from the app (go to ? Help, Clear all Data).
Go to the questionnaire.
On Question 4, mark that you have all documents.
On Question 7, Select the answer "Intermediate".
On Question 8, mark "Fluent" and nothing else.
I expect to see the recommended topics that show up are:
1.1 Manitoba Start & Settlement agencies,
1.2 General information,
5.6 Register for language classes,
5.7 Continuing education for adults, and
6.1 Francophone settlement services
But instead, I see
Social Insurance number
opening bank account
credit and credit history
Canadian money and banking.
Acceptance Criteria

[ ]

Steps for QA:

Question: Social Insurance Number:
It has the same code for MB and BC. BC works but MB doesn't.
I expect it not to see the Social insurance number in MB when I click I have social insurance number, but it shows in the recommended topics.

I selected nothing in the Questionnaire. I expect to see the topics that are recommended for everyone.
I see

Social Insurance number
opening bank account
credit and credit history
Canadian money and banking.

[ ]  MB Social Insurance Number topic shows with questionnaire cleared, and also when the corresponding answer is selected.

Solving issue https://github.com/pg-irc/pathways-frontend/issues/1690 as part of this issue

I have done the above steps - now the topics I expect to see DO show up, but the problem is other topics show up before them as well.
I do see recommended topics:
1.1 Manitoba Start & Settlement agencies,
1.2 General information,
5.6 Register for language classes,
5.7 Continuing education for adults, and
6.1 Francophone settlement services
But before I see those, the "things to do right away" column shows:
opening bank account
Social Insurance number
I selected that I already had a SIN and I already had a bank account, so I should not be seeing those.

Hi! Sorry. Same as my comment from Nov 26 - I filled out the questionnaire as I should have, that I already have a SIN number and bank account, yet I am still seeing those topics. Does this have to do with the 'things to do right away' tag?

I think it has to do with "inverted" questions, i.e. the ones where selecting an answer causes topics to no longer be recommended, is just fundamentally not possible. I'll spend some time thinking through this

No longer meaningful after we removed the ability to invert questions


Functional specs gap analysis

User Story

As a [user role below]:
Functional analyst

I need to:
make sure every component/pattern has functional specs

so that I can:
provide proper documentation to my teammates.

Additional information

IBM.com Library

Just need to go through and see what could have been overlooked whether it doesn't exist, or we added a placeholder for the temporary interim. Then I can create functional spec stories for the functional analysts to pick up.
Acceptance criteria

[ ] New issues created for placeholder functional specs to be update
[ ] New issues created for non-existing functional specs to be created

Just some quick notes on what I've noticed so far. Will review with the other functional analysts on Monday hopefully.

Update Quote with 3rd line source
Image with caption Lightbox boolean and larger image?
Remove props from specs into separate document
Lightbox media viewer updates

image updates
ensure it’s all up to date

Add change log to functional spec stories, and version number to names
LinkWithIcon left icon? Is there a story for this?
Expressive Modal needs to be updated
Content layout needs to be updated
Utilities documented?

Markdown to Html

same height
altlangs
decodeString
escaperegexp
featureflag
geolocation
ipcinfoCookie
serialize
settings
smoothScroll
unique

Services documented?

Analytics
DDO
Locale
MarketingSearch
Profile
SearchTypehead
Translation
VideoPlayer
global

What’s happening with simple overview?
Logo grid duplicate in section pattern
What’s happening with simple benefits?

@photodow I was talking to the devs about the data model/props in the func specs as I was working on #2414 and got confirmation that they feel they should own this documentation and will maintain in the codebase and ensure that it's included in the readme in storybook. Also confirmed that Drupal would not need this in the func specs and are fine getting it from the readme. I think we can just slowly phase this out of the func specs as we update and/or create new ones.

@ellenchoi1 I had a similar discussion with @jeffchew which is why I put it on this list here.

Gap analysis is finished.  Closing this issue.  The individual issues to complete the work will be created by Roberta and reviewed by James.  The individual issues will be inserted into the pipeline as velocity permits.


'Fails provision request' Error when trying to add external collection to wix site

I have followed the instructions, including the SQL_CONFIG setup, but when I try to connect to the external database from wix I get the following error ...
"Unable to fetch result (Connector with namespace 'wixtest' and instanceId 'b6fa1942-76ef-4b08-b19d-9adc66ad4341' fails provision request)"

Did you set the SECRET_KEY env variable?

Yes, I did use the SECRET_KEY variable. I had the same problem when I tried the using the Firestore Adapter (for which I had also set the SECRET_KEY var)

I had the same issue. Try the following  style of connection string using the port:
{ "host":"myDBServerIP",
"port":3306,
"user":"myUser",
"password":"myPassword",
"database":"myDatabase"}

Hey guys, after a lot of searching I found out that you need to connect the cloud-run instance to the cloud SQL app using this guide.
After doing this simple connection - everything worked.
🥳

I had this problem before and it took me a day to figure it out ...
Make sure SQL_CONFIG is correct and in the right format. This is what I have   SQL_CONFIG: '{"host": "xxx", "user": "xxx", "password" : "xxx", "port": "xxx", "database" : "xxx"}' in one line
when connecting to the external database from wix, a provision request is sent, so you can also try sending the provision request in Postman directly to your endpoint, see if it succeeds.

Hi Guys,
Having followed all of the above and implemented changes as described, I am still having the same please help
I have tried both of the following and they do not work:
SQL_CONFIG        {"host":"xxx.xxx.xxx.xxx", "port":"xxxx",  "user":"xxxx", "password":"xxxxx", "database":"xxxx"}
and
SQL_CONFIG                      {"socketPath":"/cloudsql/ptwebdb",  "user":"xxxx", "password":"xxxxx", "database":"xxxx"}
Here are my remaining environment variables which I used:
SECRET_KEY                      "xxxxx"
ALLOWED_OPERATIONS  {"provision", "get", "find", "count", "update", "insert", "remove", "select", "delete", "describeDatabase", "finditem", "findSchemas", "listSchemas", "list", "convert", "load", "listbuilds"}
The container runs on Google Cloud Run, the MySQL database runs on Google SQL Cloud fully managed service, database is working fine and database user has all permissions.
Any help you can provide is much appreciated.
Regards,
Eugene Boulis

Hi Guys,
Having followed all of the above and implemented changes as described, I am still having the same please help
I have tried both of the following and they do not work:
SQL_CONFIG {"host":"xxx.xxx.xxx.xxx", "port":"xxxx", "user":"xxxx", "password":"xxxxx", "database":"xxxx"}
and
SQL_CONFIG {"socketPath":"/cloudsql/ptwebdb", "user":"xxxx", "password":"xxxxx", "database":"xxxx"}
Here are my remaining environment variables which I used:
SECRET_KEY "xxxxx"
ALLOWED_OPERATIONS {"provision", "get", "find", "count", "update", "insert", "remove", "select", "delete", "describeDatabase", "finditem", "findSchemas", "listSchemas", "list", "convert", "load", "listbuilds"}
The container runs on Google Cloud Run, the MySQL database runs on Google SQL Cloud fully managed service, database is working fine and database user has all permissions.
Any help you can provide is much appreciated.
Regards,
Eugene Boulis

Maybe temporary assign a hard coded sql config to sqlConfig to make sure mysql.createConnection(sqlConfig) succeeds first? If it does, at least you know the config is correct.
Next, you want to make sure JSON.parse(process.env.SQL_CONFIG) gives you the same config value as the one you just hard coded. You can do that by removing this line  https://github.com/wix/corvid-external-db-mysql-adapter/blob/6c4333c7d8e50e979f89a53913a3aae5a4bdaa80/client/database.js#L5

I am trying to setup a development environment to allow me to troubleshoot and experiment with your suggested settings.
I have cloned the adapter's GitHub repository as per "developing and extending the adapter" instructions here https://github.com/wix/corvid-external-db-mysql-adapter, however I am not sure I understand how this will work, could you please help explaing:

Why do we need to setup a local MySQL proxy since all we want is to create another version of the adapter which currently runs on Google Cloud Run while the database currently runs on Google MySQL Cloud fully managed, wouldn't the customized connector which I need to create also run on Google Run Cloud Run and connect to the current database?
How will the wix site connect to the local MySQL proxy, currently I obtain the MySQL adapter service point from Google Cloud Run service details page which shows the URL of the deployed MySQL adapter?
How to obtain the URL for the locally deployed adapter above?

Thanks for your help.

From: happyappledog<EMAIL_ADDRESS>Sent: August 21, 2020 2:12 AM
To: wix/corvid-external-db-mysql-adapter<EMAIL_ADDRESS>Cc: eboulis<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [wix/corvid-external-db-mysql-adapter] 'Fails provision request' Error when trying to add external collection to wix site (#17)
Hi Guys,
Having followed all of the above and implemented changes as described, I am still having the same please help
I have tried both of the following and they do not work:
SQL_CONFIG {"host":"xxx.xxx.xxx.xxx", "port":"xxxx", "user":"xxxx", "password":"xxxxx", "database":"xxxx"}
and
SQL_CONFIG {"socketPath":"/cloudsql/ptwebdb", "user":"xxxx", "password":"xxxxx", "database":"xxxx"}
Here are my remaining environment variables which I used:
SECRET_KEY "xxxxx"
ALLOWED_OPERATIONS {"provision", "get", "find", "count", "update", "insert", "remove", "select", "delete", "describeDatabase", "finditem", "findSchemas", "listSchemas", "list", "convert", "load", "listbuilds"}
The container runs on Google Cloud Run, the MySQL database runs on Google SQL Cloud fully managed service, database is working fine and database user has all permissions.
Any help you can provide is much appreciated.
Regards,
Eugene Boulis
Maybe temporary assign a hard coded sql config to sqlConfig to make sure mysql.createConnection(sqlConfig) succeeds first? If it does, at least you know the config is correct.
Next, you want to make sure JSON.parse(process.env.SQL_CONFIG) gives you the same config value as the one you just hard coded. You can do that by removing this line https://github.com/wix/corvid-external-db-mysql-adapter/blob/6c4333c7d8e50e979f89a53913a3aae5a4bdaa80/client/database.js#L5
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/wix/corvid-external-db-mysql-adapter/issues/17#issuecomment-678060766, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQV533ECXNSSBU6DK2TN5ETSBYF3DANCNFSM4LAQCU5Q.

I am not sure why local MySQL proxy is needed. I am actually using AWS RDS instead, and I can connect to it using its URL directly. Maybe a proxy is specifically required by Google MySQL in general?
The way I debug this is to use the database in the cloud, but host this connector locally. So nothing is changed except that now I am hosting the connector on my laptop instead of Google Cloud.
The local URL will be something like http://localhost:8080 . We can't connect to it from wix, it's just for debugging the codes locally.
Hope this help.

Thanks this is of great help it helps to clarify how the connector is setup for local development
Once you have tested it all works and ready to deploy to production, do you create a Github branch off the original repo then push your local changes to the Github branch creating a new Docker MySql adapter container URL which is then deployed to the Google Cloud Run service or can I just deploy straight from my local machine Github clone to Google Cloud Run?
Sent from my iPhone

On Aug 24, 2020, at 11:41 PM, happyappledog<EMAIL_ADDRESS>wrote:

I am not sure why local MySQL proxy is needed. I am actually using AWS RDS instead, and I can connect to it using its URL directly. Maybe a proxy is specifically required by Google MySQL in general?
The way I debug this is to use the database in the cloud, but host this connector locally. So nothing is changed except that now I am hosting the connector on my laptop instead of Google Cloud.
The local URL will be something like http://localhost:8080 . We can't connect to it from wix, it's just for debugging the codes locally.
Hope this help.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

If after debugging, you know you have the right SQL_CONFIG in the yml file, you don't have to change this adapter codes (unless there is something else you want to modify in the codes), which means you don't need to change the docker container.
If you do need to change the code, as long as you know how to create a docker container, I guess you can just deploy the new container to Cloud Run directly? I didn't try out Cloud Run here but App Engine. For App Engine, you can just deploy straight from your local folder.

Hi,
Thanks very much to the great help you provided to me, much appreciated.
I am a developer and have developed in J2EE, .Net as well as in C/C++, I have also developed on Windows, Linux and Solaris platforms, but I am totally new to Node.js. To enable me to get this MySQL Wix adapter to work, I have setup a Node.js development environment on my Windows desktop, I have installed NVM for Windows and the latest stable Node.js release.  I have also installed Ubuntu 18.04 running on top of VMware VM Player and I am using VS Code editor for Windows.
Before attempting a full clone on Ubuntu which I have to figure out how to connect to it from Wix (I may have to configure port forwarding to make it work), it would be much simpler if I can make temporary changes to the Node.js code and debug while still using the original docker container, or a clone of the container on a GCP folder, how would I be able to do that I mean make changes to the code in Github while still being able to deploy on Google Cloud Run using the same method ad before?
Again thanks a million for your great help which I am much grateful for.
Regards
Eugene Boulis

On Aug 26, 2020, at 4:30 AM, happyappledog<EMAIL_ADDRESS>wrote:

If after debugging, you know you have the right SQL_CONFIG in the yml file, you don't have to change this adapter codes (unless there is something else you want to modify in the codes), which means you don't need to change the docker container.
If you do need to change the code, as long as you know how to create a docker container, I guess you can just deploy the new container to Cloud Run directly? I didn't try out Cloud Run here but App Engine. For App Engine, you can just deploy straight from your local folder.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I am not sure why local MySQL proxy is needed. I am actually using AWS RDS instead, and I can connect to it using its URL directly. Maybe a proxy is specifically required by Google MySQL in general?
The way I debug this is to use the database in the cloud, but host this connector locally. So nothing is changed except that now I am hosting the connector on my laptop instead of Google Cloud.
The local URL will be something like http://localhost:8080 . We can't connect to it from wix, it's just for debugging the codes locally.
Hope this help.

@happyappledog I am also using an AWS RDS instance and hoping to connect it to my Wix site. Could you provide more details on how you got them connected? When you say you can connect to it directly using its URL do you mean you can literally add the URL to the Wix website builder with no additional development, or did you modify the Adapter template in this repo? If the latter, would love if you could publish your version of the Adapter for AWS RDS to GitHub if possible. Thank you!


Fix display since time round (#14226)

Backport #14226

Fix display since time round

Fix since time

Fix tests

@zeripath Thanks!!!

No problems