Crash while trying to manually set item variable.

Issue type:Crash

:bug: Bug

Short description:
Crashed to desktop while editing a script trying to manually set a variable to an item
Code : let step = {id_item:(id: 'minecraft:arrow')}

Steps to reproduce the problem:

Create a script with this code:
let step = {id_item:('minecraft:arrow')}
Copy step to variable card
3.Put in Display Pannel

Expected behaviour:
Error in system

Versions:

This mod:
Minecraft: 1.21.1
Mod loader version: 21.1.71

I know what it says here but I'm certain this is a problem with your mod
Log file:

https://pastebin.com/WHACb85Q

Thanks for reporting!

The crashlog seems unrelated to the problem you describe, as seems to be caused by AE2.
Could you try reproducing the problem without any other mods installed?


DOCS: add clarity of "OpenVPN" versus vagueness of possibly working with an…

What this PR does / why we need it:
add clarity of "OpenVPN" versus vagueness of possibly working with any VPN
Which issue (if any) this PR fixes:
Fixes #
Any other useful info:

:+1:


Remove ‘Account’ from above page title

Unnecessary, so shouldn't be there.
Removing highlighted:

:+1:


ClustersService does not list the available clusters

Describe the bug
I am trying to list the clusters for my organization using ClustersService.
The following code snippet does not retrieve any clusters despite I know I have deployed a couple
(the *client instance is valid since I am able to list my projects' names)
	projects, _, err := client.Projects.GetAllProjects(ctx, nil)
	if err != nil {
		log.Fatal(err)
	}
	for _, project := range projects.Results {
		client.Clusters.List(ctx, project.ID, nil)
	}

Expected behavior
Clusters should be listed.
Logs
If applicable, add logs to help explain your problem.
Desktop (please complete the following information):

OS: MacOS Ventura
Lib Version v0.36.0

Additional context
Add any other context about the problem here.

Hi @pantelis-karamolegkos currnetly I'm unable to reproduce
Full snipped
ackage main

import (
	"context"
	"fmt"
	"log"
	"os"

	"github.com/mongodb-forks/digest"
	"go.mongodb.org/atlas/mongodbatlas"
)

func main() {
	apiKey := os.Getenv("API_KEY")
	apiSecret := os.Getenv("API_SECRET")
	baseURL := os.Getenv("BASE_URL")
	t := digest.NewTransport(apiKey, apiSecret)

	tc, err := t.Client()
	if err != nil {
		log.Fatalf(err.Error())
	}

	client, _ := mongodbatlas.New(tc, mongodbatlas.SetBaseURL(baseURL))

	ctx := context.Background()
	projects, _, err := client.Projects.GetAllProjects(ctx, nil)
	if err != nil {
		log.Fatalf(err.Error())
	}
	// ppProjects, err := json.MarshalIndent(projects, "", "\t")
	// if err != nil {
	// 	log.Fatalf(err.Error())
	// }
	//fmt.Println(string(ppProjects))
	for _, p := range projects.Results {
		fmt.Println("projectID", p.ID)
		clusters, _, err := client.Clusters.List(ctx, p.ID, nil)
		if err != nil {
			log.Fatalf(err.Error())
		}
		for _, c := range clusters {
			fmt.Println("clusterName", c.Name)
		}
	}
}

Are you able to reproduce with the snipped above? if yes, can you share details like the project ID which you expect to have access to clusters? and if you are not able to share here then I would recommend to reach out to mongodb support to share those details?
Also a little reminder this project is now deprecated and probably won't receive further updates so we recommend to migrate to https://github.com/mongodb/atlas-sdk-go/

Hi @gssbzn apologies, this was a typo on my code! Sorry for not closing this promptly and waisted your time 🙏🏻


Some instruction cleanup

@ehrlinger,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


core API

Someone tick the boxes in #130 because I can't

May I ask, just for my understanding, what this change does?

It's an API cleanup, in preparation for publishing librespot as a library. See the completed items in https://github.com/librespot-org/librespot/issues/130


gmock_gtest_all not found

Hello, I was trying to build OpenSFM and I folled the documentation for building:
git clone --recursive https://github.com/mapillary/OpenSfM
pip install -r requirements
#Install some dependencies
python3 setup.py build

But I got the error that these files were not located:
gmock_gtest_all.cc
gmock_main.cc
testing_main.cc
If you have any advice I would appreciate it.

I encountered the same problem, did you solve it?

The most frequent errors encountered are:

pyyaml installation error (cython error with pyyaml version 5.4.1)
pybundle error
third_party/gtest error

By reviewing the commit history from August 9 (commit ef872b2399cf1cc036d91e950f210a7be33c2745) onward, you’ll notice that all workflows for subsequent commits, like "Docker CI / Build docker and run tests (push)" and "Docker CI / Build docker installing CeresSolver 2 and run tests (push)," fail.
Below is a step-by-step guide explaining how I resolved these issues.
The missing files should be taken from the release, but only those for the third_party folder (missing gtest files), as the release uses outdated dependencies and likely Ceres 2.0.
To begin, create two clones of the latest repository:

One will serve as the main repository.
The other will be used to retrieve the missing parts.

In the root of the main repo, run:
`git submodule update --init --recursive
This will load the removed submodules.
In the main repo, remove pyyaml==5.4.1 from requirements.txt and install it manually with the following command:
pip install "cython<3.0.0" && pip install --no-build-isolation pyyaml==5.1
In the second repo, go to the August 9 commit:
git checkout ef872b2399cf1cc036d91e950f210a7be33c2745
From this commit, copy the missing files (gmock_gtest_all.cc, gmock_main.cc, testing_main.cc, and any other required files) into the main repository.
Now you should be able to build the image from the modified dockerfile in which you should install ceres 2.2 insted ceres 2.0
RUN \     curl -L http://ceres-solver.org/ceres-solver-2.2.0.tar.gz | tar xz && \     mkdir ceres-bin && cd ceres-bin && \     cmake ../ceres-solver-2.2.0 && \     make -j3 && \     make install


Deafness renamed to Hearing Loss

See https://github.com/monarch-initiative/mondo/issues/551

thanks for your help with this @matentzn! I did all the remaining fixes manually.


[VS Code] Joining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.

<!--
For Visual Studio problems/feedback, please use the Reportjjjjjjjk

Another instance of the failure, will let @legomushroom  and @setaskin to debug this.

Hi @mrspouladgar , thanks for reporting this. Would you give us some more info for investigation. Did you think this was a copy paste failure, did you joined the session when you tried again ? and how the host send you the link? ( via e-mail, messenger ect.)

This issue should now be resolved. Please re-open it if you're still experiencing this problem. Thanks!


Handshake fails on iOS/Android

Handshake is failing due to mismatched auth.  It may be an endian issue.  Lost my output, but will attach some logs tomorrow.

Glad i found this issue, guys!
I am trying the echo example to ws://echo.websocket.org with sync client and failing with "Websocket Handshake failed: Invalid Sec-Websocket-Accept (expected AnDKkAsrkWcWifKgdqIeekL8lH0= got gnDKkIsrkWcWifKg9qIeesL8lH0=)"
I am on pure Lua 5.1, so trying to use

struct = lua-struct
bit = lua-bit-numberlua

Thanks to the @lipp example ('The quick brown fox jumps over the lazy dog'), i was able to figure that i have to use numberlua in LuaBitOp compatiubility mode, i.e. bit = require 'bit.numberlua'.bit (and not in the default or bit32 compatibility mode). I suspect that's what @BoarK meant by "to require it properly'. Now i'll follow down the band clue he dropped and see if that might be a reason for some mishap in frame.decode() ....

Update: turned out the issue was with the struct library doing some of the packing wrong. After fixing it websockets works like a charm!
So if someone else runs in this problem... check your struct functionality

@EnTerr :+1: what kind of struct problem did you have?

In my particular case,  bytes were not getting packed/unpacked right
https://github.com/iryont/lua-struct/issues/2


Server/routing

Fixed create and join trip to set current trip of users.
Fixed add expense.
For the body.user part. Shouldn't we might as well call it body.id since the user is going to have the entire document including his id?

Added test spec


Displaying Historical Information on a user view

This pull request addresses issue #300 where it aims to display the correct historical information such as games won and such to a view where any view can access it.

Update: I implemented the code such that there is statistics for matches played, matches won, tournaments played, tournaments won. I added the attributes in views.py in the users folder. I also went into userprofile_detail.html and added a link to the user's stats. The button works and goes to the user's stats. This can be accessed by any user if they go on the user's profile.

I implemented the requested changes, but my only comment is that a user can be in a match if assigned by an admin, and not be in a lobby. I think there should be some functionality in the future to require a lobby for a match just so a person can't have matches won, but no matches played due to it being tied to lobbies.

Added back remove friend functionality; got removed elsewhere in another PR.

Closing comment: In this PR, we created a view that can be accessed in the profile of a user that displays the stats of a user, so for example, how many matches a user has been in, how many matches they've won, how many tournaments they've participated in and how many have they won. This is visible to any user when looking at another user's profile. One thing is that match count is linked to lobby, and since an admin can manually add a user to a match without a lobby, there should be something in the future that mandates a lobby for a match such that the numbers stay consistent. Also in the future, there should be more stats, such as rank in Chigame website, and maybe win rate, and things like that.
Team contributions can be found in issue closing statement.

Issue Score: Excellent
Comments:
Great work using models from other teams to complete this PR!


Window sandbox: any way to not access the file system?

We package Servo within a UWP app, which does not allow access to the file system.
When using canvas.fillText() in Servo, it crashes: https://github.com/servo/servo/issues/26015
Enabling the UWP capability broadFileSystemAccess solves the issue, but this will make it harder to submit the app to the Windows Store.

Enabling the UWP capability broadFileSystemAccess solves the issue

Actually I'm not sure about that capability. Still experimenting.

I'm actually not sure it's a sandbox issue.
I see that fontkit is looking for these files for SansSerif fonts:
"C:\\WINDOWS\\FONTS\\ARIAL.TTF"
"C:\\WINDOWS\\FONTS\\ARIALI.TTF"
"C:\\WINDOWS\\FONTS\\ARIALBD.TTF"
"C:\\WINDOWS\\FONTS\\ARIALBI.TTF"
"C:\\WINDOWS\\FONTS\\ARIBLK.TTF"
"C:\\WINDOWS\\FONTS\\ARIAL.TTF"
"C:\\WINDOWS\\FONTS\\ARIALBD.TTF"
"C:\\WINDOWS\\FONTS\\ARIBLK.TTF"

… which do not exist.
I think font-kit should not unwrap here, and just push if the result is Ok.
Servo should also do something better if not fonts are found.


revert fails with variable

Prerequisites
Please put an X between the brackets as you perform the following steps:

[X] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[X] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[X] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
revert fails with metavariables when a variable statement is used, but succeeds if a proper argument is used or nonrec is added.
Context
https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/revert.20and.20variable
affects also extract_goal in mathlib
Steps to Reproduce
variable {P : Prop}
theorem bug --adding nonrec fixes the issue
  --{P : Prop} --uncommenting this line fixes the issue
  (S : P → Prop) :
  ∃ (hx : P), S hx := by
  apply Exists.intro
  · revert P -- tactic 'clear' failed, target depends on 'bug'
    sorry

Expected behavior:
revert succeeds, and behaves identically for variables and parameters
Actual behavior:
revert succeeds only without variable
Versions
Lean 4.10.0 (also earlier)
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

Preliminary investigation notes:
The revert code finds that bug (the aux decl usually not shown here) depends on P, and tries to clear it, but clear refuses to do so. Note that clear bug yields the same error message. Making P a normal parameter still makes clear bug print that error message, but revert P seems to not attempt clear bug.
So the next question would be: Why does bug depend on P when it’s a variable, but not when it’s a parameter? And if so, is that desirable behavior?

How severe is this issue in practice?

My only occurrence in practice was that it made extract_goal fail with an unclear error message; it may well be that the best solution is to just fix the error handling in this tactic

Ok, in that I'd downgrade this to low priority until we have more evidence of severity. And when we do maybe the fix will just be a more helpful error message, suggesting nonrec or instantiating goals in a different order.


Fix #513 - Add renew lock feature

Fix #513

Coverage increased (+0.04%) to 58.142% when pulling 34bac4db1ff51a678121589f5bb2934fa085f79a on lmazuel:fix_513 into c04736958ac5ad265e9fd2012bbf2279240cf201 on Azure:dev.

Hey @lmazuel
Question:
We are using this feature of renew_lock. But this does not update the broker_properties['LockedUntilUtc']. So we can we know when is the lock is going to be expired?
Thanks!

Hi @oak-tree
Unfortunately, if you look at the RestAPI, this properties is not returned when we renew :(
https://docs.microsoft.com/en-us/rest/api/servicebus/renew-lock-for-a-message
So I'm unsure how I'm supposed to get the updated value :/

@oak-tree I got an answer from the SB team. New lock has to be computed manually. When you use renew, you should now wait the "entity’s default lock duration" (quote from SB answer).
I can probably provide a wrapper or something to do that. I'm not a huge fan of updating LockedUntilUtc, since it's an actual value from the initial message. But I can probably provide a "when_expire" or something method that returns LockedUntilUtc or the computed time. Please open an explicit issue asking for this if you feel this is interesting.

@lmazuel Thanks for replying! How can I get the entity's default lock duration? If I understand correctly the api call is restarting the lock clock?
and yes, this can be help at the future. I'll open an issue for that

I'm not sure yet how to get the entity's default lock duration :/

@lmazuel getting the default lock duration can be done by
get_subscription(topicname,subscriptionname).lock_duration


Can you provide  log of train,thanks!!!

The early version of the checkpoint has already been provided, Please check the checkpoint in updated readme.

Seems that the link requires permission? Should we apply for the access permission? Thanks @songruiqi


MM-10877 - Updating edited behaviour IE11

Summary
MM-10877 - Updating edited behaviour IE11
Ticket Link
https://mattermost.atlassian.net/browse/MM-10877
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Ran make check-style to check for style errors (required for all pull requests)
[ ] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[x] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

@hmhealey I think that went in with the long post changes, so I just removed it altogether, we can think about adding it again, but that may require some refactoring.


docker fails

13 3.674 This copy of InterSystems IRIS has been licensed for use exclusively by:
13 3.674 Community License expired.
the related PR is ignored for > 3 months

Hi Mr. Robert,
Sorry for this late response.
Your PR was merge. Thank you for it!


Monitor gradient global norm

Context
Sometimes it is useful to use gradient clipping regulated by clip_thresh, which is used by tf.clip_by_global_norm as clip_norm parameter. If tf.global_norm(gradients) > clip_norm, gradients are shrinked by clip_norm / tf.global_norm(gradients).
https://www.tensorflow.org/api_docs/python/tf/clip_by_global_norm
Problem and proposed changes
It may not be so clear what is the appropriate value for clip_thresh to prevent NaN values. To help with that, I have added code to monitor current gradient global norm. Specifically, if clip_thresh is not None, gradient clipping is possible and the calculated self._grads_norm is reachable by self.grad_global_norm_monitor.
I'll post some screenshots to show that everything works okay. I have an ongoing training which compares this branch with the current master. I am tracking internal optimizer values with this setup:
optimizer = YFOptimizer(learning_rate=0.05, momentum=0.0, clip_thresh=10)
 tf.summary.scalar("learning_rate", optimizer._lr_var)
tf.summary.scalar("momentum", optimizer._mu_var)
tf.summary.scalar("global_gradient_norm", optimizer.grad_global_norm_monitor)

As promised, here are some screenshots showing that everything works okay. This commit just adds global norm monitoring. Like before, I am using Python 3 and TF 1.2r. Two GPUs for the current master and two for this branch.

I've updated this PR to include recent changes to the master branch.


Flimmer/Blurry text on vertical mouse movement in 1.29 (Windows, disappears with --disable-gpu)

VSCode Version: 1.29.0 (user setup); the problem was introduced a few major versions ago, don't remember exactly when.

OS Version:

Edition: Windows 10 Pro
Version: 1803
OS Build: 17134.345

Machine details:

CPUs
Intel(R) Core(TM) i7-3610QM CPU @ 2.30GHz (8 x 2295)

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

Memory (System)
15.95GB (6.22GB free)

Process Argv
--disable-extensions

Screen Reader
no

VM
0%

GPU
NVidia GeForce GTX 660M; NVIDIA GeForce Experience Version <IP_ADDRESS>

Steps to Reproduce:

Move the mouse vertically on the sidebar, the settings page, status bar, and sometimes in the text area for files.

Does this issue occur when all extensions are disabled?: Yes; and with an empty User/settings.json. Launching with code --disable-gpu solves the issue.
Video showing the bug (didn't work to convert to gif...): http://sendvid.com/5bwj2543

Can you download this build to check whether it's fixed with the next version of electron?
https://az764295.vo.msecnd.net/exploration/a19e0f85ac7167f27716b3cbbb07a6164f47684e/VSCode-win32-x64-1.29.0-exploration.zip

@roblourens
Thanks for the quick response. :)
I tested the binary with and without extensions + settings.json...
I cannot reproduce the bug with the binary provided in any scenario so the bug seems to be gone.

Alright, sounds like the fix is coming down the pipeline. There are some similar issues.

@roblourens I'll continue to use --disable-gpu in the interim which has worked thus far so there's no rush, but could you speak a bit about when that is estimated to be?.

I don't know. Not this month.

@roblourens Alright; Thanks!


[GURPS 1.6.0]

Changes / Comments

Rolltable update: Move "Ref" to the right of roll type to save space. Suggested by Mike W.
Grimoire tab: Added checkbox to highlight spells. Added a spell level modifier column. Suggested by Mike W.
Falling/Collision notes: Revised the default notes when the falling damage is rolled. However, these notes are only visible on new characters sheets, since the those notes can be edited on the options page.
Bug fix: Racial  Traits Self-control roll. Success/Fail text was missing, fixed.

Correction
For: Rolltable update: Move "Ref" to the right of roll type to save space. Suggested by Mike W.
This should say Roll Template update, not ### Rolltable.


can't evaluate field Defer in type interface {} - "github.com/HugoBlox/hugo-blox-builder/modules/blox-tailwind" is not compatible with this Hugo version

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/HugoBlox/hugo-blox-builder/issues?q=is%3Aissue
[X] This issue is not a question, feature request, or anything other than a bug report directly related to Hugo Blox Builder. Please post those things on Discord: https://discord.gg/z8wNYzb

Description
I tried to build with several versions of hugo-extended the landing-page template, and I keep getting the following error:
"github.com/HugoBlox/hugo-blox-builder/modules/blox-tailwind" is not compatible with this Hugo version
It seems to me that I cannot work with<EMAIL_ADDRESS>actually it looks to me that all errors lead to here:
partials/components/backlinks.html:4:11: executing "partials/components/backlinks.html" at :  can't evaluate field Defer in type interface {}
Can you tell me please (and perhaps update the documentation) of the verison requirements of the  landing-page. If this is a minor issue, than I think it should work with any Hugo-extended versions.
Reproduction Link
https://github.com/antaldaniel/quakers
Steps to Reproduce

Just build the site with any version of hugo-extended between current from 0.122.0

Expected Result
No error message.
Actual Result
"github.com/HugoBlox/hugo-blox-builder/modules/blox-tailwind" is not compatible with this Hugo version
What Hugo Module versions does your site use?
go 1.19
What operating system(s) are you seeing the problem on?
Windows, Linux
What browser(s) are you seeing the problem on?
Chrome
Which Hugo Blox template are you using?
landing-page
What version of Hugo are you using?
Hugo Extended 0.122.0 and latest

Please use Discord for asking questions on which Hugo version each template requires. You can find the compatible Hugo version in the netlify.toml file of each template.


Halfmoon to reduce the size of the sidebar

Is there any pre-built feature to reduce the size of the sidebar and text on the page without having to mess with your own css?

https://github.com/halfmoonui/halfmoon/issues/44


When a user changes their email on Discord, they cannot log into dashboard

This is something that should not be happening. I'll look into later if I remember.

Can you give me a video of how to reproduce this?

change your discord account email and login?

Basically yeah kinda obvious

What specifically happens? Does it error on the console, and what do you get back as a response?


Does not show rootPage under [locale] when rewrite is enabled

Issue Description:
We are testing this library and managed to almost make it work perfectly. Love that you have added an option to not show the language in the URL, since we work with domains and not different language-paths.
We also have a different basePath than the /-path. When we navigate to the root which is localhost:3000/medlem it wont use the files that are within app/[locale]/page but it uses app/page. Is there something that I'm doing wrong or does it not work? It should rewrite from /basePath to /basePath/[locale] but it doesn't at the moment.
Reproduction Steps:
Run the server and go to localhost:3000/medlem it shows the default next template code, but it should actually show the text This is the root page with locale turned on which is under app/[locale]/page.tsx. It only shows pages from [locale] when I'm navigating to level 2 which is /[locale]/under in my repro-repo linked:
https://github.com/aulonm/next-international-repro
Expected Behavior:
It should show pages that are under [locale] and not a page outside of the [locale] folder when navigating to the root of the website.
Actual Behavior:
Shows the page from the root of the app-dir
Next.js version: 13.4.13
next-international version: 0.8.1
Best regards,

I've tried locally and this looks like an issue with the matcher option of the middleware. It doesn't run at all when navigating on the website (you can add a console.log inside the middleware to test).
I'm not sure to understand exactly how matcher works when having a basePath, but matcher: ['/'] seems to work.

oooh, the matcher! That made more sense. I'll be testing it on our production build with different matchers but it looks like it works as it should in the repro repo I linked to with a second string in the matcher-config. Doing a test with ['/'] and it looks like it works as you said.
Thank you again, Tom!
I'll close it for the time being and I'll reopen it if I find something else that is relevant here


Cannot run demo

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
Output  of the command
RuntimeError: Unable to find target for this triple (no targets are registered)
free(): invalid pointer
Aborted (core dumped)
Reproduction

What command or script did you run?

 python demo/pcd_demo.py demo/data/kitti/kitti_000008.bin configs/second/hv_second_secfpn_6x8_80e_kitti-3d-car.py checkpoints/hv_second_secfpn_6x8_80e_kitti-3d-car_20200620_230238-393f000c.pth --show

Did you make any modifications on the code or config? Did you understand what you have modified?
No

What dataset did you use?

Environment
(/moyujian/moyujian/mmdetection3d/mmdetection3d_env) ps@ps-X10DRG:/moyujian/moyujian/mmdetection3d/mmdetection3d_code/mmdetection3d$ python mmdet3d/utils/collect_env.py
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
sys.platform: linux
Python: 3.7.12 | packaged by conda-forge | (default, Oct 26 2021, 06:08:21) [GCC 9.4.0]
CUDA available: True
GPU 0,1: NVIDIA GeForce RTX 3090
GPU 2,3,4,5: NVIDIA GeForce RTX 2080 Ti
CUDA_HOME: /usr/local/cuda/
NVCC: Build cuda_11.0_bu.TC445_37.28845127_0
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.10.1
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.4-Product Build 20210904 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.2.3 (Git Hash 7336ca9f055cf1bfa13efb658fe15dc9b41f0740)
OpenMP 201511 (a.k.a. OpenMP 4.5)
LAPACK is enabled (usually provided by MKL)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 11.3
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
CuDNN 8.2
Magma 2.5.2
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.3, CUDNN_VERSION=8.2.0, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.10.1, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.11.2
OpenCV: 4.5.5
MMCV: 1.4.0
MMCV Compiler: GCC 7.5
MMCV CUDA Compiler: 11.0
MMDetection: 2.21.0
MMSegmentation: 0.21.1
Error traceback
RuntimeError: Unable to find target for this triple (no targets are registered)
free(): invalid pointer
Aborted (core dumped)

It's strange, the error should be reported by open3d? and we have not tested MMDetection3D on Pytorch 1.10.0, maybe you can downgrade the pytorch version if possible.

I meet the same issue with pytorch1.10


Question about the gh action steps, also thank you

Thank you for posting this github action.
In regards to the app setup and pipeline coupling where the app is moved into the "review" stage. Is there any benefits to doing it in 2 stages? Would it be possible to combine the two steps using?
https://devcenter.heroku.com/articles/platform-api-reference#review-app-create

ah, upon further reading it seems the 2 stage approach uses endpoints which explicitly document the usage and expectation of the app.json being present in the source tar
where as the /review-apps endpoint makes no mention if it also looks for the app.json

Also, the review apps API is disabled by Heroku because of the security incident. I don't think they've enabled yet because none of their recent updates states that.

@pmbanugo Yes. Thanks!


metrics-mongodb.rb support for Mongo 3

metrics-mongodb.rb will need a version check added to stop it erroring on mongo 3
http://docs.mongodb.org/manual/reference/command/serverStatus/
Changed in version 3.0: The server status output no longer includes the workingSet, indexCounters, and recordStats sections.
Removing the 9 row "if" section which references indexCounters causes it to work on Mongo 3 so I suspect the quickest fix would be to wrap this in a "if serverStatus.version != 3" ?

I'm just writing a pull request for this....

Version comparison
http://stackoverflow.com/questions/2051229/how-to-compare-versions-in-ruby

The Gem::Version.new('x.y.z') method for comparing the current version to 3.0.0 seems the cleanest so I've used this but depends on Ruby 2 and above.  Let me know if you want me to change this to something 1.x compatible.

Changlog
The latest released version is 0.0.4 but the changelog has entries under 0.0.5 and 6.  I've put mine under "unreleased" which is where I assume these should be? (and then that becomes 0.0.5?).  I hope that's correct.

FYI, I've asked Mongo support about this and they directed me to .. https://jira.mongodb.org/browse/SERVER-16378
Basically they recommend monitoring page_faults instead of index counters so we should probably add this in to the reported stats?

Yes, sounds good, can you change your PR?

I'll roll up all changes into one release.

I was just about to add it and noticed it was already there...
server_metrics['mem.pageFaults'] = server_status['extra_info']['page_faults']

Fixed in 0.0.7. Thanks @jhsubscribe!


Gh 2240 federated view validator bug

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@142c1d4). Click here to learn what that means.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             develop    #2241   +/-   ##
==========================================
  Coverage           ?   64.92%
  Complexity         ?     2413
==========================================
  Files              ?      839
  Lines              ?    27155
  Branches           ?     3233
==========================================
  Hits               ?    17631
  Misses             ?     8140
  Partials           ?     1384

Impacted Files
Coverage Δ
Complexity Δ

.../federatedstore/schema/FederatedViewValidator.java
77.5% <81.81%> (ø)
0 <0> (?)

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 142c1d4...e9cad6d. Read the comment docs.


Upgrading nokogiri gem due to security vulnerability

Note that this upgrade changes minimum required ruby version from
1.9.3-p551 to 2.1.8.
$ bundle audit check
Name: nokogiri
Version: <IP_ADDRESS>
Advisory: CVE-2016-4658
Criticality: Unknown
URL: https://github.com/sparklemotion/nokogiri/issues/1615
Title: Nokogiri gem contains several vulnerabilities in libxml2 and libxslt
Solution: upgrade to >= 1.7.1

Name: nokogiri
Version: <IP_ADDRESS>
Advisory: CVE-2017-5029
Criticality: Unknown
URL: https://github.com/sparklemotion/nokogiri/issues/1634
Title: Nokogiri gem contains two upstream vulnerabilities in libxslt 1.1.29
Solution: upgrade to >= 1.7.2

Name: nokogiri
Version: <IP_ADDRESS>
Advisory: CVE-2016-4658
Criticality: Unknown
URL: https://github.com/sparklemotion/nokogiri/issues/1615
Title: Nokogiri gem contains several vulnerabilities in libxml2 and libxslt
Solution: upgrade to >= 1.7.1

Name: nokogiri
Version: <IP_ADDRESS>
Advisory: CVE-2017-5029
Criticality: Unknown
URL: https://github.com/sparklemotion/nokogiri/issues/1634
Title: Nokogiri gem contains two upstream vulnerabilities in libxslt 1.1.29
Solution: upgrade to >= 1.7.2

Vulnerabilities found!

Reviewers: @chase-childers @lap1817

@jnb this may impact Yelp's deployment? I'm not sure if you guys have upgraded to Ruby 2.1 yet?

We're still evaluating whether this is the proper solution.


QR code vs Verifiable QR Code

[ ] Content
[ ] Video Script
[ ] Video
[ ] Published on microsite

Video script under review. Dependency on Rajeesh

Video script under review. Dependency on Rajeesh

Content added, a new ticket to be created for video


lookAt returns a vector instead of a matrix

Hi,
Thanks for writing a JS library compatible with GLM!
Much appreciated to try and port my https://en.wikibooks.org/wiki/OpenGL_Programming#The_basics_arc tutorials :)
I found a bug: glm.lookAt returns a quat rather than a matrix.
Cf. http://glm.g-truc.net/0.9.8/glm-0.9.8.pdf §5.2.
At first glance, stripping "glm.quat" in the various implementation of 'lookAt' in source code fixes the issue.
Note: I wanted to test my changes but I cannot run 'make test' or 'make build' from nodejs (tested with v4.7.2 and 6.9.2), I get various errors:

looks at closure-compiler somewhere in /tmp
requires build/VA_ARGS.js which needs to be build manually (missing make dependency)
uses the non-standard 'load' keyword
errors related to "ES5 strict mode"
I'm not sure how I'm supposed to run all this :)

Hi Beuc,
Cool!  Porting those kinds of tutorials sounds like an interesting use case.
As a short term workaround you could try converting the return value to a matrix using var m = glm.toMat(glm.lookAt(...)));
And yeah... my hacked-together build system needs some TLC.  Fortunately, you can also bootstrap glm-js from source (ie: without any build system or minification); here's a quick snippet to demonstrate:

web page
the underlying gist

Hope that helps and I'll schedule a permanent fix for glm.quat with the next update.
Regards,
-Tim

Hi,
AFAICS this work-around is incomplete. When I try it, the tut05 rotating cube is not centered on the screen anymore. Inspecting the results shows that the last matrix vector (last line in toString()) is zero'd:
# GLM 31728cb4
glm.toMat4(glm.lookAt(glm.vec3(0.0, 2.0, 0.0), glm.vec3(0.0, 0.0, -4.0), glm.vec3(0.0, 1.0, 0.0))).toString()
"mat4x4(
	(1.000000, 0.000000, 0.000000, 0.000000),
	(0.000000, 0.894427, 0.447214, 0.000000),
	(0.000000, -0.447214, 0.894427, 0.000000),
	(0.000000, 0.000000, 0.000000, 1.000000)
)"

# patched GLM without the extra "glm.quat" in glm.lookAt
# (AFAIU "glm-js.min.js" is relying on GL-Matrix.)
glm.lookAt(glm.vec3(0.0, 2.0, 0.0), glm.vec3(0.0, 0.0, -4.0), glm.vec3(0.0, 1.0, 0.0)).toString()
"mat4x4(
	(1.000000, 0.000000, 0.000000, 0.000000),
	(0.000000, 0.894427, 0.447214, 0.000000),
	(0.000000, -0.447214, 0.894427, 0.000000),
	(0.000000, -1.788854, -0.894427, 1.000000)
)"

Thanks for sharing the non-minified setup, though what I aimed at was running the test suite against a patch so I could submit it here :)

Thanks for sharing the non-minified setup, though what I aimed at was running the test suite against a patch so I could submit it here :)

OK, will figure out the missing console dependencies so the command line test / build can be used by others. :)
Just to mention -- you can also run the entire test suite locally using a web browser -- like by opening file:///..../glm-js/test/index.html from the project folder.

ps: I see now that the glm.toMat4 workaround can't ever work here -- because the glm.quat only brings forward the rotation information, while the lookAt matrix also contains a translation.

Cf. #5 for a patch :)

Thanks! :)


fix windows ci build

Description
main fix involves working around a bug reported in cirruslabs/cirrus-ci-docs#423
the build originally broke because CIRRUS_WORKING_DIR was changed and introduced a bug where the default was not set prior to environment variable definition
this means that our environment variable, VISPY_GL_LIB, which was set to $CIRRUS_WORKING_DIR\opengl32.dll, came out equivalent to \opengl32.dll
since this environment variable was used by VisPy to determine where to load the OpenGL library from, it could not find it
thanks to @csweaver for her help with sleuthing around the possible causes :)

since cirruslabs/cirrus-ci-docs#423 was resolved, the build on master should be fixed
most of the other changes are related to cleanup/readability so I'm closing this and will include them in another PR that cleans up our implementation and fixes the OSX build

actually I forgot that conda installation was an issue too so this is still relevant


Test mk do fast

I think this is a good candidate for PR CI - it deploys the operator and exercises address creation round tripping against an existing cluster in a few scenarios.
>time make test-mk-do-fast
...
DEPLOY_OPERATOR=true USE_EXISTING_CLUSTER=true ENABLE_WEBHOOKS=false go test  ./controllers/... -test.timeout=40m -ginkgo.slow-spec-threshold=30s -ginkgo.fail-fast -ginkgo.v -ginkgo.label-filter="do && !slow"
ok  	github.com/artemiscloud/activemq-artemis-operator/controllers	100.609s

real	1m48.126s
user	0m23.600s
sys	0m4.259s

This is an impressive improvement!!!
My stats on minikube:
DEPLOY_OPERATOR=true USE_EXISTING_CLUSTER=true ENABLE_WEBHOOKS=false go test  ./controllers/... -test.timeout=40m -ginkgo.slow-spec-threshold=30s -ginkgo.fail-fast -ginkgo.v -ginkgo.label-filter="do && !slow"
ok      github.com/artemiscloud/activemq-artemis-operator/controllers   109.830s

real    1m57.028s
user    0m19.845s
sys     0m3.591s

My stats on crc:
DEPLOY_OPERATOR=true USE_EXISTING_CLUSTER=true ENABLE_WEBHOOKS=false go test  ./controllers/... -test.timeout=40m -ginkgo.slow-spec-threshold=30s -ginkgo.fail-fast -ginkgo.v -ginkgo.label-filter="do && !slow"
ok      github.com/artemiscloud/activemq-artemis-operator/controllers   117.956s

real    2m5.798s
user    0m20.973s
sys     0m3.881s

I converted this PR to draft because it depends on #400


Implement attribute input types

The other PR was accidentally soft-locked
Closes #4244.
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[x] Database migration files are up to date.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Here is the report for 4f8321283f487076bbabf9eabdf9501ceafb4bc4 (NyanKiyoshi/saleor @ attributes/types)
Base comparison is c4a8e66db69e3e960a95bee958b6c774b9599609.
No differences were found. (click me)

# api.benchmark checkout
  test name                           	left count 	right count
  ------------------------------------	-----------	-----------
  add billing address to checkout     	         41	         41
  add shipping to checkout            	          7	          7
  checkout payment charge             	         16	         16
  complete checkout                   	          6	          6
  create checkout                     	         45	         45

# api.benchmark homepage
  test name                           	left count 	right count
  ------------------------------------	-----------	-----------
  retrieve main menu                  	          5	          5
  retrieve product list               	          4	          4
  retrieve secondary menu             	          5	          5
  retrieve shop                       	          2	          2

# api.benchmark product
  test name                           	left count 	right count
  ------------------------------------	-----------	-----------
  product details                     	         16	         16

# api.benchmark variant
  test name                           	left count 	right count
  ------------------------------------	-----------	-----------
  retrieve variant list               	         13	         13


Presentation demo no longer existing

Hi,
just wanted to tell you that in the demo section the following presentations give "404 page error" :

"Gauge conditions and blackhole stability by Kartik Prabhu" gives a 404 error page. I think the author has removed it.
"Speeding Up Drupal by Evan Willhite @evanmwillhite"
"How to build WordPress theme and plugin by Bernard Latanowicz"

The followings have wrong link:

Rukun Iman | Jeniv by Jefry Nivella - A cube presentation.
Toward the development of a web literacy map by Doug Belshaw

I think you can remove all these links from the list of demos.
Cheers,
Alessandro

Also "Speeding Up Drupal by Evan Willhite @evanmwillhite". 404 error page.

Just tell me if you prefer a pull request to remove them.

I went ahead and fixed the Speeding up Drupal link http://www.electricthread.com/speed-drupal. Thanks for the heads-up @alebaffa

@alebaffa if you are taking about the links on wiki: https://github.com/impress/impress.js/wiki/Examples-and-demos - there is no need for pull request, it should be editable by anyone, so go ahead and remove links that are dead.
Thanks for checking it out!

ok cool! :+1:


Use taiki-e/install-action to install dev tools in CI

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (5bd4f71) to head (a89ee68).
Report is 1 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main      #6   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          1       1
  Lines        238     238
=====================================
  Misses       238     238

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


build errors

I was attempting to build the latest version of whitebox_tools (commit df41d62680d8bad2fb5047e827817a3caa1b2f1d) using the latest version of rust. I received the following errors. I'm not familiar with Rust. Is some other build option needed?
Thanks
--
rustup 1.19.0 (2af131cf9 2019-09-08)
rustc 1.37.0 (eae3437df 2019-08-13)
> cargo build --release

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_idw_interpolation.rs:686:79
    |
686 |                                                         frs.insert(p.x, p.y, ((255 << 24) | (clr.blue << 16) | (clr.green << 8) | clr.red) as f64);
    |                                                                               ^^^^^^^^^^^
    |
    = note: #[deny(exceeding_bitshifts)] on by default

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_idw_interpolation.rs:686:93
    |
686 |                                                         frs.insert(p.x, p.y, ((255 << 24) | (clr.blue << 16) | (clr.green << 8) | clr.red) as f64);
    |                                                                                ^^^^^^^^^^^^^^^^

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_idw_interpolation.rs:806:52
    |
806 |                                             val = ((255 << 24) | ((val_blue.round() as u16) << 16) | ((val_green.round() as u16) << 8) | (val_red.round() a
s u16)) as f64;
    |                                                    ^^^^^^^^^^^

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_idw_interpolation.rs:806:66
    |
806 |                                             val = ((255 << 24) | ((val_blue.round() as u16) << 16) | ((val_green.round() as u16) << 8) | (val_red.round() a
s u16)) as f64;
    |                                                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_idw_interpolation.rs:871:60
    |
871 |                                                     val = ((255 << 24) | ((val_blue.round() as u16) << 16) | ((val_green.round() as u16) << 8) | (val_red.r
ound() as u16)) as f64;
    |                                                            ^^^^^^^^^^^

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_idw_interpolation.rs:871:74
    |
871 |                                                     val = ((255 << 24) | ((val_blue.round() as u16) << 16) | ((val_green.round() as u16) << 8) | (val_red.r
ound() as u16)) as f64;
    |                                                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: attempt to shift left with overflow
   --> src/tools/lidar_analysis/lidar_nn_gridding.rs:665:79
    |
665 |                                                         frs.insert(p.x, p.y, ((255 << 24) | (clr.blue << 16) | (clr.green << 8) | clr.red) as f64);

@jterstriep I noticed this bug yesterday myself when I added a new tool and tried to build. It was the result of adding the ability to interpolate LiDAR RGB values. Interestingly, when I added that functionality, I perform cargo check, which did not raise the errors. It was only after a build that I encountered it. Anyhow, a fix has been pushed to the repo now and you should be able to build without issue. I'll close the issue now but if you continue to experience problems, please feel free to re-open. Thanks for posting this and it's always nice to know that people are paying attention ;-)


function evaluation unwrap error

function(config={})
{
  [if 1 == 1 then "item" + std.toString(x)]: x for x in std.range(0,2)
}

When evaluation above code, it panicked:
thread 'main' panicked at 'called Option::unwrap() on a None value', crates/jrsonnet-evaluator/src/lib.rs:128:17
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
I tracked the call stacks and found it may be causd by stdlib missing.
Is there any reason that not importing the stdlib in function scope? I tried with go-jsonnet and it's fine.

In jrsonnet all evaluation state is stored in TLS, which being filled when it needed
There was missing state enter on TLA evaluation, fixed in 4accdd0afdc96071a6bf819c1e1ddc35ece3e78c, thanks!


Improve the log message for view location cache miss

For example, its currently "View lookup cache miss for view 'About' in controller 'Home'"
Many people might not care about it but we could just make it clear for anyone interested to understand how the system works.

@kichalla what is the suggestion?

Logging can't teach people how the product works sadly, that's what documentation does

@kichalla any thoughts?

Maybe something like the following?

Unable to find location of view 'About' in cache for controller 'Home'
Found location of view 'About' in cache for controller 'Home'

I don't think that these suggestions are a significant improvement, that's just a different way of saying what the existing message says.
I recommend we close this unless we find a significant way to improve this /cc @Eilon


Report time spent within action/task

When executing a task, I would like Eliot to report execution time of a task. Use case: when using logstash/kibana I would like to see execution time of each request to my API.
I'm wrapping my API end points with a <start_action> context manager - it would be convenient if the context manager itself told me the wall time of the request.
I'm not attaching any pull requests because I'm not sure what the best way would be - would every context manager report time, would it be controlled by a parameter or should there be a <timed_start_task> - from my point of view time reported for every context would be great, but I'm not sure if it works for everyone.
Note that this can be almost achieved with manually creating non-finishing contexts and using <add_success_field> but lack of <add_error_field> or just <add_field> it's very inconvenient.
In order of preference I would like eliot to:

automagically report time elapsed in a context manager
allow me to add custom fields within non-finishing contexts ( I believe that might be worth it's own issue - for completeness, <add_success_field> should have it's matching handlers for errors? )

I think this is a little too complicated for me jump and start writing code, but let me know if I can help.

I know it's been a while, but I'm going to respond to let you decide/close/resolve the issue as appropriate.
Agreed - if I wanted to really dive into performance details, the end-of-action message would probably not contain enough information.
However, what I had in mind was a little less advanced - I didn't want something to tell me exactly what the problem is, but something that would indicate that there's something wrong, something that would indicate the problem without any pre/post processing, something that would work out of the box.
Another cool thing is that I can have as many nested actions as I want, right? So, if I was into sql debugging, I wouldn't use run_sql as an action type. Instead, within parent task, I would do: action_type "portal:login:user_login:run_sql:_get_user_by_id:id=123"` or similar, with as many details are required - in this way, I can control which piece of information I care about on each levels, I think?

Still have this in the back of my head; non-urgent Eliot work may take a while.
Custom action types for this use case seems like a hack, given you have ability to pass in additional fields.
Probably I'm overthinking this and as first pass just including action_elapsed would make logs that much more useful, yes.

Agreed and understood, I'm only upgrading to Eliot-0.6 now - no urgency required.
Yeah, it occurred to me too when I posted the previous message.
But, in practice, I never found myself looking at the additional fields first - I look at the action type and I expect it to tell me what's going on. That's why I would not call a task run_sql. A portal:login:run_sql maybe, but not a generic, plain name.
Without it, looking at the dashboard is too confusing, I think.
(that's another feature though, auto-appendable task names? :))
Agreed.

Got a few more requests for this.

eliot-tree will now display this for existing logs, even without support in Eliot  proper.


Command to automatically install / set up boilerplate

Create a command, such as cdev site:create which would display the following questions:

What CMS boilerplate would you like to install? [WordPress]/[Magento]/[Drupal]
What version of the CMS would you like to install? [...]
Would you like to run Composer? [Y]/[N]
Would you like to install an .env file with default varialbes for PHP variables? [Y]/[N]
Would you like to install Front-End Tools? [Y]/[N]

Breakdown:

Clones the boilerplate git repository for the chosen CMS
WordPress, Mage1/2 or Drupal 7/8
Runs composer install followed by composer update in relevant directories
Creates and populates an .env file within correct directory (such as default database details)
Clones the front-end theme tools directory within the appropriate theme directory, and runs pnpm install, npm install or yarn. Also replace vhost URL in gulpfile.js (cloned) with the URL set in cdev configure

It would be good if at the end of running cdev configure you are then given the option to run this command.

https://github.com/garethmidwood/cdev/pull/24

Pull request #24 has been merged with master. This still requires work so will remain open for now.

Will create a new ticket to address the creation of environment variables and close this ticket.


Be clear which lint rules affect health

https://pub.dartlang.org/experimental/help#health

How can you improve your health score?
Use strong-mode checks and some linter rules, and make sure you fix all the warnings and errors before publishing."

It's not clear from this description which lint rules contribute towards the health score. People will naturally want to make sure they're following all the rules that will result in higher listings, but turning them all on is probably overkill (and some are rather subjective).

Also - that page has the URL of the lint rules page on it in the sample file, but no clickable-link!

The lints come from here: https://github.com/dart-lang/pana/blob/master/lib/src/analysis_options.dart
we should improve the link to this, agreed!

Aha; I did see mention of pana though it's in the maintenance score section but not the health section.

Can we simplify this make it more concrete; for example:
How can you improve your health score?
For Flutter packages, run flutter analyze and resolve any errors or warnings shown.
For Dart packages, run tbd and ....


Add TrueGalaxyCash (TGCH)

Currency name: TrueGalaxyCash
Symbol: TGCH
Website: https://galaxycash.info
Theme color: #434247
Logo.svg.zip

We need an SVG version of the logo. You have only supplied a bitmap EPS and PNG.

Uploaded svg.

There are still bitmap images embedded in the SVG. Open the source file and look for data:image/png;base64.
I've cleaned up the SVG file for you: tgch-logo.svg.zip


LongParameterList no longer recognizes ignored annotations in 1.19.0

Expected Behavior

Prior to 1.19.0, this configuration worked to prevent LongParameterList warnings on constructors with a few "known" injection annotations from dagger.
LongParameterList:
  active: true
  functionThreshold: 7
  constructorThreshold: 7
  ignoreDefaultParameters: false
  ignoreDataClasses: true
  ignoreAnnotated: ['javax.inject.Inject', 'dagger.Provides', 'dagger.assisted.AssistedInject']

Observed Behavior

In 1.19.0, these seem to be ignored now
Steps to Reproduce

Here are some snippets that repro for us
class ActivityFeedPresenter @Inject constructor(
  private val activityApi: ActivityApi,
  private val userRepository: UserRepository,
  private val botsDataProvider: BotsDataProvider,
  private val mpdmDisplayNameHelper: MpdmDisplayNameHelper,
  private val prefsManager: PrefsManager,
  private val conversationRepo: ConversationRepository,
  private val lockedThreadTransformer: ActivityFeedLockedThreadTransformer,
  private val mentionItemFactoryLazy: Lazy<MentionItemFactory>,
) : ActivityFeedContract.Presenter, LoadingStateProvider {

class AppDialogFragment @AssistedInject constructor(
  private val appDialogsRepository: AppDialogsRepository,
  private val presenter: AppDialogPresenter?,
  private val keyboardHelper: KeyboardHelper,
  private val snackbarHelper: SnackbarHelper,
  private val appBuildConfig: AppBuildConfig,
  private val appDialogMenuBinder: AppDialogMenuBinder,
  private val appDialogMenuViewFactory: AppDialogMenuView.Factory,
  private val appDialogSelectViewFactory: AppDialogSelectView.Factory,
  private val appDialogTextViewFactory: AppDialogTextView.Factory,
  private val platformLogger: PlatformLogger,
  @ForScope(UserScope::class) private val fragmentNavRegistrar: FragmentNavRegistrar
) : ViewBindingFragment(),
  AppDialogContract.View,
  MenuSelectionListener,
  BackPressedListener {

class AppActionDelegateImpl @Inject constructor(
  private val appActionsRepository: Lazy<AppActionsRepository>,
  private val attachmentActionHelperLazy: Lazy<AttachmentActionHelper>,
  private val attachmentRepositoryLazy: Lazy<AttachmentRepository>,
  private val blockKitActionDelegateLazy: Lazy<BlockKitActionDelegate>,
  private val blockKitDialogHelperLazy: Lazy<BlockKitDialogHelper>,
  private val customTabHelperLazy: Lazy<CustomTabHelper>,
  private val formattedTextBinderLazy: Lazy<FormattedTextBinder>,
  private val messageActionsHelperLazy: Lazy<MessageActionsHelper>,
  private val commandRepository: Lazy<CommandRepository>,
  private val platformEventHandler: Lazy<PlatformEventHandler>,
  private val toasterLazy: Lazy<Toaster>,
  private val typefaceSubstitutionHelperLazy: Lazy<TypefaceSubstitutionHelper>,
  private val networkInfoManagerLazy: Lazy<NetworkInfoManager>,
  private val platformAppsManagerLazy: Lazy<PlatformAppsManager>,
  private val messageDaoLazy: Lazy<MessageDao>,
  private val messageEventBroadcasterLazy: Lazy<MessageEventBroadcaster>,
  private val slackActionDelegateLazy: Lazy<SlackActionDelegate>,
  private val snackbarDelegate: SnackbarDelegate,
  private val blockKitStateProvider: BlockKitStateProvider,
  private val triggerResolver: TriggerResolver,
  @ForLegacyFeature(MOBILE_INPUT_BLOCKS_IN_MESSAGES) private val isMobileInputBlocksInMessagesEnabled: Boolean
) : AppActionDelegate, SnackbarDelegate by snackbarDelegate {

Context

It is a regression
Your Environment

Version of detekt used: 1.19.0
Version of Gradle used (if applicable): N/A
Gradle scan link (add --scan option when running the gradle task): private repo
Operating System and version: macOS 11
Link to your project (if it's a public repository): N/A

I assume that you are not using type solving. I need to check how the old implementation was getting the full qualified names...
As a "workaround" you can use the name of the annotation instead of the full qualified name. Or enable type solving.

Yeah type solving is a nonstarter for us as we run this via CLI in a commit hook


Introduce Enhanced Consent plugin

Introduces an enhanced consent plugin for tracking consent events and CMP load time events.
Details of the API can be found here:
https://www.notion.so/JS-tracker-enhanced-consent-API-7aa67bb253e14c6cab01090f120556be?d=390079798b994ae0a8374c9bd9f7b621

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Under the flex layout, reducing the width of the parent container results in incomplete truncated content

Describe the bug
Under the flex layout, reducing the width of the parent container results in incomplete truncated content.
To Reproduce
https://codesandbox.io/s/ccti1?file=/index.js
Reproduce Step

Open the https://codesandbox.io/s/ccti1?file=/index.js
Use the mouse to reduce the width of the parent container.
Then use the mouse to restore the width of the parent container


8 nodes no leader

The demo works charm for me! Great!
But, when I set the numbers of nodes to 8, and kill each leader one by one;  When the number of nodes is equal
and less then 4 nodes, there will be no Leader but only candidates and followers remain.
Any idea?
Take a look!
Screencast!
https://1drv.ms/v/s!AsLam7iob4O7m2KJTwURRylR6Dsj?e=cfv7nz

This is the correct behavior.
RAFT algorithm can tolerate failures equal to the cluster size minus the quorum.
(* quorum = cluster size / 2 + 1)
This means that more than half of the nodes in the cluster are required to reach consensus.
In your example, the cluster size is 8 and the quorum is 5.
Therefore, at least 5 nodes are required to reach consensus.
screenshot of README.md

This is the correct behavior.
RAFT algorithm can tolerate failures equal to the cluster size minus the quorum. (* quorum = cluster size / 2 + 1) This means that more than half of the nodes in the cluster are required to reach consensus.
In your example, the cluster size is 8 and the quorum is 5. Therefore, at least 5 nodes are required to reach consensus.
screenshot of README.md

Thanks for the explanation!
I will check it!


OIDC Authentication not working with Keycloak.org

Describe the bug
Keycloak, an OIDC provider does not work with datahub.
To Reproduce
Steps to reproduce the behavior:

Configure OIDC against out of the box keycloak well known config target
attempt to visit datahub, redirect to OIDC provider takes place properly
successful login into provider
500 error returned from datahub

Expected behavior
Expect Datahub to work with OIDC providers.
Screenshots
2021-08-11 20:21:53
00:21:53 [application-akka.actor.default-dispatcher-371] WARN  o.p.o.profile.creator.TokenValidator - Preferred JWS algorithm: null not available. Using all metadata algorithms: [PS384, ES384, RS384, HS256, HS512, ES256, RS256, HS384, ES512, PS256, PS512, RS512]
2021-08-11 20:21:53
00:21:53 [application-akka.actor.default-dispatcher-371] WARN  o.p.o.profile.creator.TokenValidator - Preferred JWS algorithm: null not available. Using all metadata algorithms: [PS384, ES384, RS384, HS256, HS512, ES256, RS256, HS384, ES512, PS256, PS512, RS512]
2021-08-11 20:21:53
00:21:53 [application-akka.actor.default-dispatcher-371] ERROR react.auth.AuthModule$2 - Unable to renew the session. The session store may not support this feature
2021-08-11 20:21:53
00:21:53 [application-akka.actor.default-dispatcher-371] ERROR react.auth.AuthModule$2 - Unable to renew the session. The session store may not support this feature
2021-08-11 20:21:53
00:21:53 [application-akka.actor.default-dispatcher-371] ERROR akka.actor.ActorSystemImpl - Internal server error, sending 500 response
2021-08-11 20:21:53
00:21:53 [application-akka.actor.default-dispatcher-371] ERROR akka.actor.ActorSystemImpl - Internal server error, sending 500 response

Hi @mhensley-cls. Thanks for reporting!
Can you provide the OIDC configurations that are shown at your .well-known config URL? It seems there is some issue with validation of the token due to an unsupported algo.
Would love to see what KeyCloak is supporting by default!

@jjoyce0510 - can't share the exact endpoint, but here's a redacted version...
{
"issuer": "https://REDACTED.com/auth/realms/REDACTED",
"authorization_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/auth",
"token_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/token",
"introspection_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/token/introspect",
"userinfo_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/userinfo",
"end_session_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/logout",
"jwks_uri": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/certs",
"check_session_iframe": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/login-status-iframe.html",
"grant_types_supported": [
"authorization_code",
"implicit",
"refresh_token",
"password",
"client_credentials",
"urn:ietf:params:oauth:grant-type:device_code",
"urn:openid:params:grant-type:ciba"
],
"response_types_supported": [
"code",
"none",
"id_token",
"token",
"id_token token",
"code id_token",
"code token",
"code id_token token"
],
"subject_types_supported": [
"public",
"pairwise"
],
"id_token_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512"
],
"id_token_encryption_alg_values_supported": [
"RSA-OAEP",
"RSA-OAEP-256",
"RSA1_5"
],
"id_token_encryption_enc_values_supported": [
"A256GCM",
"A192GCM",
"A128GCM",
"A128CBC-HS256",
"A192CBC-HS384",
"A256CBC-HS512"
],
"userinfo_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512",
"none"
],
"request_object_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512",
"none"
],
"request_object_encryption_alg_values_supported": [
"RSA-OAEP",
"RSA-OAEP-256",
"RSA1_5"
],
"request_object_encryption_enc_values_supported": [
"A256GCM",
"A192GCM",
"A128GCM",
"A128CBC-HS256",
"A192CBC-HS384",
"A256CBC-HS512"
],
"response_modes_supported": [
"query",
"fragment",
"form_post",
"query.jwt",
"fragment.jwt",
"form_post.jwt",
"jwt"
],
"registration_endpoint": "https://REDACTED.com/auth/realms/REDACTED/clients-registrations/openid-connect",
"token_endpoint_auth_methods_supported": [
"private_key_jwt",
"client_secret_basic",
"client_secret_post",
"tls_client_auth",
"client_secret_jwt"
],
"token_endpoint_auth_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512"
],
"introspection_endpoint_auth_methods_supported": [
"private_key_jwt",
"client_secret_basic",
"client_secret_post",
"tls_client_auth",
"client_secret_jwt"
],
"introspection_endpoint_auth_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512"
],
"authorization_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512"
],
"authorization_encryption_alg_values_supported": [
"RSA-OAEP",
"RSA-OAEP-256",
"RSA1_5"
],
"authorization_encryption_enc_values_supported": [
"A256GCM",
"A192GCM",
"A128GCM",
"A128CBC-HS256",
"A192CBC-HS384",
"A256CBC-HS512"
],
"claims_supported": [
"aud",
"sub",
"iss",
"auth_time",
"name",
"given_name",
"family_name",
"preferred_username",
"email",
"acr"
],
"claim_types_supported": [
"normal"
],
"claims_parameter_supported": true,
"scopes_supported": [
"openid",
"email",
"phone",
"microprofile-jwt",
"groups",
"offline_access",
"profile",
"roles",
"web-origins",
"address"
],
"request_parameter_supported": true,
"request_uri_parameter_supported": true,
"require_request_uri_registration": true,
"code_challenge_methods_supported": [
"plain",
"S256"
],
"tls_client_certificate_bound_access_tokens": true,
"revocation_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/revoke",
"revocation_endpoint_auth_methods_supported": [
"private_key_jwt",
"client_secret_basic",
"client_secret_post",
"tls_client_auth",
"client_secret_jwt"
],
"revocation_endpoint_auth_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"HS256",
"HS512",
"ES256",
"RS256",
"HS384",
"ES512",
"PS256",
"PS512",
"RS512"
],
"backchannel_logout_supported": true,
"backchannel_logout_session_supported": true,
"device_authorization_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/auth/device",
"backchannel_token_delivery_modes_supported": [
"poll",
"ping"
],
"backchannel_authentication_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/ext/ciba/auth",
"backchannel_authentication_request_signing_alg_values_supported": [
"PS384",
"ES384",
"RS384",
"ES256",
"RS256",
"ES512",
"PS256",
"PS512",
"RS512"
],
"require_pushed_authorization_requests": false,
"pushed_authorization_request_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/ext/par/request",
"mtls_endpoint_aliases": {
"token_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/token",
"revocation_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/revoke",
"introspection_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/token/introspect",
"device_authorization_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/auth/device",
"registration_endpoint": "https://REDACTED.com/auth/realms/REDACTED/clients-registrations/openid-connect",
"userinfo_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/userinfo",
"pushed_authorization_request_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/ext/par/request",
"backchannel_authentication_endpoint": "https://REDACTED.com/auth/realms/REDACTED/protocol/openid-connect/ext/ciba/auth"
}
}

Hi @mhensley-cls @jjoyce0510 hope below might help - I was able to successfully configure Datahub v0.8.12 OIDC Authentication with Keycloak default installation (as far as I can tell). Here is my .well-known config:
{     "issuer": "https://******/auth/realms/*****",     "authorization_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/auth",     "token_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/token",     "introspection_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/token/introspect",     "userinfo_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/userinfo",     "end_session_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/logout",     "jwks_uri": "https://******/auth/realms/*****/protocol/openid-connect/certs",     "check_session_iframe": "https://******/auth/realms/*****/protocol/openid-connect/login-status-iframe.html",     "grant_types_supported": [         "authorization_code",         "implicit",         "refresh_token",         "password",         "client_credentials",         "urn:ietf:params:oauth:grant-type:device_code",         "urn:openid:params:grant-type:ciba"     ],     "response_types_supported": [         "code",         "none",         "id_token",         "token",         "id_token token",         "code id_token",         "code token",         "code id_token token"     ],     "subject_types_supported": [         "public",         "pairwise"     ],     "id_token_signing_alg_values_supported": [         "PS384",         "ES384",         "RS384",         "HS256",         "HS512",         "ES256",         "RS256",         "HS384",         "ES512",         "PS256",         "PS512",         "RS512"     ],     "id_token_encryption_alg_values_supported": [         "RSA-OAEP",         "RSA-OAEP-256",         "RSA1_5"     ],     "id_token_encryption_enc_values_supported": [         "A256GCM",         "A192GCM",         "A128GCM",         "A128CBC-HS256",         "A192CBC-HS384",         "A256CBC-HS512"     ],     "userinfo_signing_alg_values_supported": [         "PS384",         "ES384",         "RS384",         "HS256",         "HS512",         "ES256",         "RS256",         "HS384",         "ES512",         "PS256",         "PS512",         "RS512",         "none"     ],     "request_object_signing_alg_values_supported": [         "PS384",         "ES384",         "RS384",         "HS256",         "HS512",         "ES256",         "RS256",         "HS384",         "ES512",         "PS256",         "PS512",         "RS512",         "none"     ],     "response_modes_supported": [         "query",         "fragment",         "form_post"     ],     "registration_endpoint": "https://******/auth/realms/*****/clients-registrations/openid-connect",     "token_endpoint_auth_methods_supported": [         "private_key_jwt",         "client_secret_basic",         "client_secret_post",         "tls_client_auth",         "client_secret_jwt"     ],     "token_endpoint_auth_signing_alg_values_supported": [         "PS384",         "ES384",         "RS384",         "HS256",         "HS512",         "ES256",         "RS256",         "HS384",         "ES512",         "PS256",         "PS512",         "RS512"     ],     "introspection_endpoint_auth_methods_supported": [         "private_key_jwt",         "client_secret_basic",         "client_secret_post",         "tls_client_auth",         "client_secret_jwt"     ],     "introspection_endpoint_auth_signing_alg_values_supported": [         "PS384",         "ES384",         "RS384",         "HS256",         "HS512",         "ES256",         "RS256",         "HS384",         "ES512",         "PS256",         "PS512",         "RS512"     ],     "claims_supported": [         "aud",         "sub",         "iss",         "auth_time",         "name",         "given_name",         "family_name",         "preferred_username",         "email",         "acr"     ],     "claim_types_supported": [         "normal"     ],     "claims_parameter_supported": true,     "scopes_supported": [         "openid",         "profile",         "groups",         "email",         "phone",         "offline_access",         "web-origins",         "roles",         "address",         "microprofile-jwt"     ],     "request_parameter_supported": true,     "request_uri_parameter_supported": true,     "require_request_uri_registration": true,     "code_challenge_methods_supported": [         "plain",         "S256"     ],     "tls_client_certificate_bound_access_tokens": true,     "revocation_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/revoke",     "revocation_endpoint_auth_methods_supported": [         "private_key_jwt",         "client_secret_basic",         "client_secret_post",         "tls_client_auth",         "client_secret_jwt"     ],     "revocation_endpoint_auth_signing_alg_values_supported": [         "PS384",         "ES384",         "RS384",         "HS256",         "HS512",         "ES256",         "RS256",         "HS384",         "ES512",         "PS256",         "PS512",         "RS512"     ],     "backchannel_logout_supported": true,     "backchannel_logout_session_supported": true,     "device_authorization_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/auth/device",     "backchannel_token_delivery_modes_supported": [         "poll"     ],     "backchannel_authentication_endpoint": "https://******/auth/realms/*****/protocol/openid-connect/ext/ciba/auth" }
and custom properties in Helm deployment values file
datahub-frontend:   extraEnvs:     - name: AUTH_OIDC_ENABLED       value: "true"     - name: AUTH_OIDC_CLIENT_ID       value: "datahub"     - name: AUTH_OIDC_CLIENT_SECRET       valueFrom:         secretKeyRef:           name: "datahub-keycloack-client"           key: "client-secret"     - name: AUTH_OIDC_DISCOVERY_URI       value: "https://***************/auth/realms/******/.well-known/openid-configuration"     - name: AUTH_OIDC_BASE_URL       value: "http://****************"     - name: AUTH_OIDC_SCOPE       value: "openid profile email groups"     - name: AUTH_OIDC_JIT_PROVISIONING_ENABLED       value: "true"     - name: AUTH_OIDC_PRE_PROVISIONING_REQUIRED       value: "false"     - name: AUTH_OIDC_EXTRACT_GROUPS_ENABLED       value: "true"     - name: AUTH_OIDC_GROUPS_CLAIM       value: "groups"     - name: AUTH_JAAS_ENABLED       value: "false"
  @mhensley-cls maybe there are some client properties you need to adjust? I myself used all the default for "Fine Grain OpenID Connect Configuration" and etc.

@donatas-jocas I got it working, we had an upstream issue where our LDAP provider flowing into keycloak made the usernames in keycloak "LAST_NAME, FIRST_NAME" ... the comma messed up some parsing.
Going to close this issue. @jjoyce0510 indicated in slack that they have improved logging in some recent versions (which helped us track this down)


ntp: add url to open action

Asana Task/Github Issue:
Description

Testing Steps

Checklist

Please tick all that apply:

[ ] I have tested this change locally
[ ] I have tested this change locally in all supported browsers
[ ] This change will be visible to users
[ ] I have added automated tests that cover this change
[ ] I have ensured the change is gated by config
[ ] This change was covered by a ship review
[ ] This change was covered by a tech design
[ ] Any dependent config has been merged

#1248  👈 (View in Graphite)
main

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


error on nuget reinstall command

It's a single dash, instead of a double dash, see reference in article.

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


feat(links.ts): custom identifiers for custom icons

{ I am pathologically incapable of rebasing, apparently -- #1653 }
This feature gives site authors more control over how links are displayed. You can define a regex that, on match, will auto-append either a string of your choice (so, an emoji) or a Hast element such as a vector path or an image to the link in the rendered HTML.
As an example, I have a substitution:
[
  /Qx!(.+)/, // Qx!https://quartz.jzhao.xyz
  Image("/static/external-quartz.png"), // I went into paint and magic wand deleted the background of the Quartz favicon
]

Which for [Quartz](Qx!https://quartz.jzhao.xyz) yields:

which links to the correct website as specified in the capturing group.
changes:

[x] switch from requiring manual hast to newtypes for the three most common use cases: Image, Path, or Emoji
[ ] discuss -- should this be as freeform as a regex, or should it be constrained to a prefix for better ergonomics and a lower barrier to entry?
[ ] documentation
[ ] favicon newtype - see the pond: transformers/AddFavicons

@saberzero1 thanks for reminding me, functional programmer with an automatic conception of let here haha

Yes, I want to refactor to make it easier and put all the raw hast under the hood. To let the author just pass the string they want and let it be picked up as the type of content that it is, I'm experimenting with tagged types as a newtype implementation but will probably have to settle for discriminating unions with type guards, same approach as in my RSS PR.

One half of the syntax for the feature is now more ergonomic.
I'm open to suggestions on whether how to specify what links you want to trigger the substitution should be simplified too. One option would be to force that it can only be a prefix, meaning the author passes a string rather than a regex and the plugin uses startsWith and slices to implement. Thoughts?


why is the last version not labelled as the stable version in the docs?

Describe the bug

Steps/Code to Reproduce
https://skore.probabl.ai/0.5/
Expected Behavior
no red "bandeau" at the top
Actual Behavior
there is a red "bandeau"
Environment
website

I think it's only a browser cache problem.


Abandoned?

Just wondering, is this project abandoned since there is no recent commits?

Yes, I am no longer working on in.


How to perform action on update?

If I want to perform an action before a resource is updated, how would I do that?
For a new resource I can use actions.new.before, but there doesnt seem to be something similar for update?

Actually, I had a read through the source and now understand how the actions work.
For future confused users, you would want to do an actions.edit.before hook, but only apply it when request.method === 'post'


Bow's enchantments issue

Minecraft version & Mod Version:
1.17.8
Single player or Server:
Single player
Describe problem (what you were doing / what happened):

Quickdraw
The quickdraw effect cannot be used to the bow in left hand.
Multi-Shot
The extra arrows always have the largest strength, whether the bow is full or not.

Good catch, i didnt test with left hand.
The extra arrow strength is weird i didnt expect that

https://minecraft.curseforge.com/projects/cyclic/files/2641400


Add timezone information to parser and formatter

Moment.js does not support parsing dates formatted according to ISO8601 (with timezone offset). Most browsers support that format natively, but IE9 doesn't.

Hi,
I get an error parsing dates of this format in IE8 - "2011-11-27T19:56:42+00:00" - using format("HH:mm:ss") I get NaN:NaN:NaN.  Works fine in Opera, Safari, Chrome and Firefox
Possibly a related issue.
Thanks,
Chris

I think I need to add timezone information to the parser and formatter. I'm changing the name of the issue to 'Add timezone information to parser and formatter' for easier reference.

This has been added, and will be released with 1.2.0

Brilliant - thanks.

Guys, I just tried parsing the ISO date string from the post above in IE8 and it does not work for me in Moment 1.4.0.
Check for yourself: http://jsfiddle.net/RN9vn/
Any suggestions?

Agreed - works in Chrome, broken in IE8 :(

Figured it out by passing a custom date format when parsing, thx.

Yeah, I am planning to add automatic parsing for iso as those are pretty common formats, but right now, all calls to moment(string) are essentially moment(new Date(string)), so if a browser doesn't support that particular format, the date isn't parsed correctly.


IR-385: Update incident reporting service data model and URLs

Depends on hmpps-incident-reporting-api#139

@karenmillermoj sorry, but i rebased this cos i had to change the JSON file after you approved. feel free to merge when you’re happy (cos i don’t have merge powers)


Null fields being written to cache for partial object (overwriting existing fields)

Describe the issue
A partial model provided to optimisticResult is overwriting the cache for the fields that were not provided with null values.
To Reproduce (MUST BE PROVIDED)
I provide this object to optimisticResult:
{
   id: 123
   name: kyle
}

There's already a user with id 123 in cache:
{
   id: 123
   name: bobby
   birthday: 2021-02-02
}

After the request is made, the cache will update user with id 123 to the following:
{
   id: 123
   name: kyle
   birthday: null
}

Expected behavior
The omitted fields will not overwrite the existing cache fields with null.
{
   id: 123
   name: kyle
   birthday: 2021-02-02
}

device / execution context
iOS
Full code
    final store = await HiveStore.open();
    final cache = GraphQLCache(
      store: store,
      possibleTypes: POSSIBLE_TYPES_MAP,
    );
    final token = getToken();
    final link = HttpLink(networkUrlManager.getGraphQLUrl(),
        defaultHeaders: {'Authorization': 'Bearer $token'});
    final client = GraphQLClient(
      cache: cache,
      link: link,
    );

    final variables = VariablesMutationCreateFriendRequest(
        input: InputFriendRequestInput(otherUserId: userId));

    QueryResult result = await client.mutate(MutationOptions(
        // ISSUE - this is setting the friendStats correctly, but setting all other fields on the User to null (the ones not provided)
        optimisticResult: <String, dynamic>{
          'data': {
            'createFriendRequest': {
              '__typename': 'User',
              'id': userId,
              'friendStatus': 'SENT_PENDING',
            },
            '__typename': 'Mutation'
          }
        },
        document: OptionsMutationCreateFriendRequest(variables: variables).document,
        variables: variables.toJson()));

I think this issue might be in the normalize_node library.
          final fieldData = normalizeNode(
            selectionSet: field.selectionSet,
            dataForNode: dataForNode[inputKey],
            existingNormalizedData: existingFieldData,
            config: config,
            write: write,
          );

dataForNode is null, because there's no value at inputKey.
existingNormalizeData will have a value, but be overwritten. And therefore fieldData will be null.
But I'd assume that it would first check if the partial (optimistic) object contains the key. If it does not, it should take the existing data. If it does contain the key, even if null, it'll overwrite.

Can you provide the mutation of your example?

This is a client error and I'm sorry for that.
We are missing this feature because we skip the cache at all https://github.com/zino-hofmann/graphql-flutter/blob/main/packages/graphql/lib/src/core/query_manager.dart#L75 I guess due to the complexity introduced by the optimistic result with the cache!
I will take this task, because I was already in mind refactoring this class a little bit, and maybe this issue is a good excuse.
The correct behavior is reported by the Appollo docs https://www.apollographql.com/docs/react/performance/optimistic-ui/#optimistic-mutation-lifecycle and in summary it is:

When the code above calls mutate, the Apollo Client cache stores a Comment object with the field values specified in optimisticResponse. However, it does not overwrite the existing cached Comment with the same cache identifier. Instead, it stores a separate, optimistic version of the object. This ensures that our cached data remains accurate if our optimisticResponse is wrong.
Apollo Client notifies all active queries that include the modified comment. Those queries automatically update, and their associated components re-render to reflect the optimistic data. Because this doesn't require any network requests, it's nearly instantaneous to the user.
Eventually, our server responds with the mutation's actual resulting Comment object.
The Apollo Client cache removes the optimistic version of the Comment object. It also overwrites the canonical cached version with values returned from the server.
Apollo Client notifies all affected queries again. The associated components re-render, but if the server's response matches our optimisticResponse, this is invisible to the user.

Thank you, @vincenzopalazzo, for the response! That was also my expectation for how optimistic query updates would work.

cache stores a Comment object with the field values specified in optimisticResponse.
Leads me to assume it just stores the provided fields (and doesn't store the omitted values with null).
associated components re-render to reflect the optimistic data
This is kind of how graphql-flutter is working now, it's just not merging the models correctly (as far as I can tell).

And sure, @budde377. Please let me know if you see I'm doing anything wrong.
mutation CreateFriendRequest($input: FriendRequestInput!) {
  createFriendRequest(input: $input) {
    ...UserWithFriendship
  }
}

fragment UserWithFriendship on User {
	...User
	friendStatus
}

fragment User on User {
	id
	firstName
	school {
		...School
	}
}

We don update nothing here the code https://github.com/zino-hofmann/graphql-flutter/blob/main/packages/graphql/lib/src/core/query_manager.dart#L75

Okay, so I think there're two things here. Arguably, we shouldn't be writing the optimistic result to the cache for the reasons mentioned in the apollo docs. I haven't looked into the details of how this is currently done, but I suspect it'll require a significant amount of work to get this feature to par.
The second problem is submitting partial data in the optimistic response. I can't make the examples in the issue fit with the mutation and optimistic response you've provided, so I assume that the fields that are being set to null are firstName and school on the User (rather than birthday). I would expect it to be a fair assumption that the optimistic response is a complete response to the operation you're performing. I.e., that it has all fields specified by the operation. Given this assumption, it is fair for the normalize library to assume that any omitted data is null (or actually throw a partial data exception). I.e., I wouldn't be surprised to find  firstName and school to be set to null.
Assuming that my understanding of your problem is correct, a work-around would be to use the cache access methods
to update the cache before the mutation. In your example, you could use the GraphQLClient.writeFragment to update the field:
fragment UpdateFriendStatusOnUser on User {
  friendStatus
}

we shouldn't be writing the optimistic result to the cache for the reasons mentioned in the apollo docs

Very much agree the Apollo implementation makes the most sense (since it supports undoing errored mutations).
Just to confirm, this is how the library works now, though, right? Via _getOptimisticQueryResult->attemptCacheWriteFromClient

I can't make the examples in the issue fit with the mutation and optimistic response you've provided

Oops that's my bad, I provided a fake example in my first message for simplicity. I just updated my original post and my response to more accurately represent what I'm seeing. But it's exactly as you said - I'm providing a partial object in the optimistic response and it's overwriting all omitted values in cache. In my updated example, name is being set to null.

I would expect it to be a fair assumption that the optimistic response is a complete response to the operation you're performing

I would have assumed the opposite (considering Apollo and this library support partial responses from network, therefore partial model merging is supported). But in doing some Googling, it looks like Apollo also doesn't support this, so I'll take it my assumption is just wrong. I'd still personally love it as a feature if it supported partials (since most of my mutations modify only a single field on a much larger object). But without that feature, I'm cool with finding a suitable workaround.

Assuming that my understanding of your problem is correct, a work-around would be to use the cache access methods
to update the cache before the mutation

So instead of using optimisticResult, you're suggesting I just do the cache update myself via writeFragment. So something like
final fragmentDoc = gql(
  r'''
  fragment UpdateFriendStatusOnUser on User {
    friendStatus
  }
  ''',
);

var fragmentRequest = FragmentRequest(
  fragment: Fragment(
    document: fragmentDoc,
  ),
  idFields: {'__typename': 'User', 'id': 123},
);

final data = client.readFragment(fragmentRequest);
// THE PART WHERE I DO THE OPTIMISTIC CHANGE
data.friendStatus = 'SEND_PENDING';
client.writeFragment(fragmentRequest, data);

Does this seem right?
The other alternative I could see would be reading the full fragment of my request from cache (similar to above, but with the full fragment I provided). But then instead of calling writeFragment directly, I provide data to the the optimisticResult of my mutation. So effectively doing the partial model merging myself.
The biggest negative to this is it's a lot of boiler plate and I have a lot of mutations and a lot of models. But I could always find a way to abstract it.

Just to confirm, this is how the library works now, though, right? Via _getOptimisticQueryResult->attemptCacheWriteFromClient

It is my understanding that the current implementation is writing the optimistic result to the cache. I might be wrong though.

The other alternative I could see would be reading the full fragment of my request from cache (similar to above, but with the full fragment I provided UserWithFriendship). But then instead of calling writeFragment directly, I provide data to the the optimisticResult of my mutation. So effectively doing the partial model merging myself.

This sounds like a good approach 🚀

Ops! Yes, we write to the cache the result, I was looking in the wrong place, sorry!
Also, I think that the approach suggested by @budde377 is the correct one, from the apollo spec I can understand that the store in the cache with a separate key the optimistic result, let's say "1234_optimistic", the query is run, and the result is compared with the optimistic one.
In this way, we can check if the optimistic result is correct.
On the other hand, I assume that from a library viewpoint we can accept a partial object given by the user, and merge this object with what we have in the cache after we run the optimistic process.
What do you think? sound good to you? maybe with some flag like "partialObj: true" to have the possibility to skip the emerging that can take some time!

@vincenzopalazzo, Without having looked too much into this, my thoughts are:

Separate libraries handle caching, so if this is not already supported (which it might be), it will require significant effort to implement.
There are some non-trivial questions around merging. What if the optimistic response is performed on an empty cache? How do we make sure the behavior is consistent? How do we do this without breaking serializers depending on output shape?
With these efforts taken into consideration, how big is this problem? I.e., how many users are affected, approximately? I.e., if apollo hasn't implemented it, maybe the current implementation is a fair assumption with the fewest exceptions. I'm worried that we spend too much effort complicating a design that might just need better documentation.


NuGet 6.7 Release Notes

Related to https://github.com/NuGet/Client.Engineering/issues/2371

@JonDouglas any comments?

Team Review: Re-run the automated release notes tool


Support ability for filepicker to be opened to select only directory or only files

Enable an admin to specify  a data attribute or option to indicate that the file picker is supposed to let the user select only a directory or only a file. Then when the file picker is opened:

If selecting a directory, display only directories (no files), and have the text of the button say "Select Directory" (or just "Select")
If selecting a file, display directories and files, but do not enable the select button unless the selected item is a file, and have the text of the button say "Select File" (or just "Select")

https://github.com/OSC/bc_js_filepicker/commit/0dea778136247e484c743462d26c4c81578f4714 addressed this


Securing etcd

After trying to setup a secured etcd2 cluster, I've run into an issue where it appears it's not actually secured.
I'm running etcd as a docker container and have created and configured all the tls keys.  The etcd startup log echos the key files being used (client and peer) with no errors, and my etcd nodes can talk to each other (cluster is healthy).
However from a remote client I can get values from etcd using etcdctl over port 2379 WITOUT specifying any keys...  obviously not good.
Any ideas what i'm missing?

@marclennox
Please check https://github.com/coreos/etcd/blob/master/Documentation/security.md#example-2-client-to-server-authentication-with-https-client-certificates for client cert verification using TLS.
Please check https://github.com/coreos/etcd/blob/master/Documentation/auth_api.md for client auth.

OK I see there's a new additional argument to turn on cert auth.  Thanks.


Buttons with Graphics aren't resizable properly

Hey wonderful JFoenix team,
Is it possible to resize a JFXButton with a Graphic on it to the size of the Graphic?
So it isn't a wide rectangle like this

But rather a square like in Chrome

Btw, that "bug"  is also the case for usual JavaFX buttons

I already tried resizing it in SceneBuilder, as well as in Code but it doesn't go smaller than in Image 1.
It seems to only be possible when I resize the SVGPath really small, which isn't what I want.
Thanks for any help! (I love jfoenix btw uwu)

Hello, can i take it ?

this issue is related to button's padding, will close this issue as it's not related to JFoenix development.
Regards,


Usage issue: RCTBottomSheetBehaviorAndroid.Commands

Hello,
In a new project, when trying the basic example in the readme, something like:
    return (
      <CoordinatorLayout style={styles.container}>
        <View style={{ flex: 1, backgroundColor: 'transparent' }}></View>
        <BottomSheetBehavior
            ref="bottomSheet"
            peekHeight={50}
            hideable={true}
            state={BottomSheetBehavior.STATE_COLLAPSED}>
            <View></View>
        </BottomSheetBehavior>
        <FloatingActionButton ref="fab" />
      </CoordinatorLayout>
    );

I get the following error:
undefined is not an object (evaluating '_reactNative.UIManager.RCTBottomSheetBehaviorAndroid.Commands')

I have no idea where the issue is coming from.

Hi, thanks for report, i will take a look.
which RN version are you using?
Did you tried the example?

Hi,
I tried using both<EMAIL_ADDRESS>and<EMAIL_ADDRESS>both don't work.
As for the example it runs well.

It seems that you didn't setup it correctly, did you follow the manually installation? did you made the changes on the MainApplication.java, android/app/build.gradle and android/settings.gradle?

Hello again,
Indeed, I went wrong somewhere during the installation. Edited android/build.gradle instead of android/app/build.gradle Thank you.

I also noticed that your module is incompatible with<EMAIL_ADDRESS>I was trying to use it in a Pepperoni boilerplate. Ultimately I had to manually upgrade my dependencies.
Again, thank you.

Yes, indeed, the recent break changes on react-native has been impossible to support old RN versions, i hope Pepperoni boilerplate update their react-native soon


chore(main): release 1.26.32

:robot: I have created a release beep boop
1.26.32 (2023-10-07)
Bug Fixes

print isunderscoop (beac6dc)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aidanaden/canvas-sync/releases/tag/v1.26.32 :sunflower:


Support to multilingual projects

Hello Lean,
As you, I'm not from a English speaking language country, and I'm developing a game that has support to English and Portuguese, to I decided to contribute to your project.
So, I made a lang file to support the many language as the developer want. It uses the browser default language as a default language, or if the user prefer, he can change the language at anytime.
I hope it was useful.

thanks @fabioars. Your work will be really useful


chore: replace deprecated ioutils module - align go mod version with actions and Dockerfile

This PR replaces the deprecated ioutil functions to please newwer versions of golangci-lint and it aligns the go version in go.mod with the versions uses in github actions and Dockerfile

Oops. I haven't seen this PR before I saw https://github.com/argoproj-labs/argocd-image-updater/pull/492 (which I merged) - sorry for that, and thanks for your contribution!
Would you be willing to rebase this PR and exclude the ioutil changes?
Thanks!


added check to provide SwId or DL in backend config and apps-frontend

this PR is currently inheriting the build failure from master


Right sidebar issue

Sometimes, the settings sidebar breaks its layout on IE, Chrome and FF latest versions

To reproduce the issue:

Select boxed layout
Select fixed layout
Unselect fixed layout

I will look into that. Thanks!

Fixed in master.