fix: error on logged-out view

viewPeriod is not available if we're not logged in

i'm merging this without waiting for approval to fix the existing sites.


openssl x64 linux can not install correctly on Ubuntu 18.04 and all package depend on it can not install as well.

vcpkg install openssl:x64-linux
The following packages will be built and installed:
openssl[core]:x64-linux
Starting package 1/1: openssl:x64-linux
Building package openssl[core]:x64-linux...
-- CURRENT_INSTALLED_DIR=/root/vcpkg/installed/x64-linux
-- DOWNLOADS=/root/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=/root/vcpkg/packages/openssl_x64-linux
-- CURRENT_BUILDTREES_DIR=/root/vcpkg/buildtrees/openssl
-- CURRENT_PORT_DIR=/root/vcpkg/ports/openssl
-- Using cached /root/vcpkg/downloads/openssl-1.0.2o.tar.gz
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Applying patch /root/vcpkg/ports/openssl/ConfigureIncludeQuotesFix.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch /root/vcpkg/ports/openssl/ConfigureIncludeQuotesFix.patch done
-- Applying patch /root/vcpkg/ports/openssl/STRINGIFYPatch.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch /root/vcpkg/ports/openssl/STRINGIFYPatch.patch done
-- Applying patch /root/vcpkg/ports/openssl/EmbedSymbolsInStaticLibsZ7.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch /root/vcpkg/ports/openssl/EmbedSymbolsInStaticLibsZ7.patch done
CMake Error at ports/openssl/portfile-nonwindows.cmake:8 (message):
Could not find make.  Please install it through your package manager.
Call Stack (most recent call first):
ports/openssl/portfile.cmake:30 (include)
scripts/ports.cmake:72 (include)
Error: Building package openssl:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: openssl:x64-linux
Vcpkg version: 0.0.113-unknownhash
Additionally, attach any relevant sections from the log files above.

we do not want to set any extra PATH for this.


Continuing work on filename template

Here are my additions to the work you started. I really appreciate your attention to detail!

Give instructions about what template keywords are available in the tooltip/description

Move the location of this new field from Advanced Options to Operation

Bump AI Render version

Put .png (or real filetype) after the field

Ah crap, I didn't mean to submit this on your main branch. I'm going to close and submit to the right branch.


python310Packages.ruamel-yaml-clib: 0.2.0 -> 0.2.4 (fix python3.10 build)

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I just pushed some cleanups to the PR. I will probably do more in another one later.


allowClickAway prop didnt work properly

This is my component, so the problem is when I click on the WhatsApp icon after applying to allowClickAway, I can't even type a text message in the input text placeholder, it's automatically closing even when I click inside
<FloatingWhatsApp
phoneNumber="123456789"
accountName="my
allowEsc
notification
chatMessage="Hello there! 🤝 \nHow can we help?"
avatar=""
allowClickAway
className="animate-fade animate-once animate-duration-[3000ms] animate-delay-1000" />

Your code piece is working perfectly for me. Try removing entire className and then try to debug.


An error occurred and looking forward to your help

Hello, your work is very innovative and I am very interested. But when attempting to run your publicly available code, an error occurred. I hope to receive your help. thank you.
error location：
When in the second stage of training (Train the registration model): python train.py -- config modelnet40.yaml, an error occurs: FileNotFoundError: [Errno 2] No such file or directory: 'ckpt/modelnet40_overap.noise. py'
Attempt to solve：
When I tried to change the overlap retrieval kpt: ckpt/modelnet40_overlap noise.py in DBDNet_paper/config/modelnet40.yaml to DBDNet_paper/exp/modelnet40_overlap noise/2024-06-26-22:41:27/checkpoints/epoch_100.pth, I still reported an error. The error message is: loss ["loss"]. backward()  RuntimeError: Function 'LinalgSvdBackward0' returned nan values in its 0th output.
Looking forward to your early reply.

Hello, your work is very innovative and I am very interested. But when attempting to run your publicly available code, an error occurred. I hope to receive your help. thank you.
error location： When in the second stage of training (Train the registration model): python train.py -- config modelnet40.yaml, an error occurs: FileNotFoundError: [Errno 2] No such file or directory: 'ckpt/modelnet40_overap.noise. py'
Attempt to solve： When I tried to change the overlap retrieval kpt: ckpt/modelnet40_overlap noise.py in DBDNet_paper/config/modelnet40.yaml to DBDNet_paper/exp/modelnet40_overlap noise/2024-06-26-22:41:27/checkpoints/epoch_100.pth, I still reported an error. The error message is: loss ["loss"]. backward() RuntimeError: Function 'LinalgSvdBackward0' returned nan values in its 0th output.
Looking forward to your early reply.

Hi @WYQ0374
Thanks for your interests. There is a typo about the extension of checkpoint path, it should be .pth rather than .py. I have fixed the issue in a new commit, thank you for pointing out it. And the 2nd stage training is indeed a little unstable, maybe you can slightly adjust the learning rate in your experiment, I think it will not bring a large difference.

Thank you for your prompt reply. I would greatly appreciate it.
My remaining question is that the first stage of training did not generate the file (ckpt/modelnet40-overlap-noise. pth), making it impossible to proceed with the second stage of training. So, how to generate this file? Thank you for your help.

Thank you for your prompt reply. I would greatly appreciate it. My remaining question is that the first stage of training did not generate the file (ckpt/modelnet40-overlap-noise. pth), making it impossible to proceed with the second stage of training. So, how to generate this file? Thank you for your help.

Actually the 1st stage did not generate the file, I just rename the best checkpoint from epoch-xx.pth to modelnet40-overlap-noise.pth for better management during the release. So your initial attempt about change the config file is correct.

Thank you for your help, and I wish you all the best in your research.
---- Replied Message ----
FromShiqi @.>Date06/30/2024 16:32 @.> @.>,
@.>SubjectRe: [Shi-Qi-Li/DBDNet] An error occurred and looking forward to your help (Issue #1)
Thank you for your prompt reply. I would greatly appreciate it. My remaining question is that the first stage of training did not generate the file (ckpt/modelnet40-overlap-noise. pth), making it impossible to proceed with the second stage of training. So, how to generate this file? Thank you for your help.
Actually the 1st stage did not generate the file, I just rename the best checkpoint from epoch-xx.pth to modelnet40-overlap-noise.pth for better management during the release. So your initial attempt about change the config file is correct.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

hello, I have a question that I would like to ask you.
During the 2nd stage of training, the following errors always occur:【 File "/home/wangyongqiang/miniconda3/envs/dbd2/lib/python3.9/site-packages/torch/autograd/init.py", line 173, in backward Variable._execution_engine.run_backward( # Calls into the C++ engine to run the backward pass RuntimeError: Function 'LinalgSvdBackward0' returned nan values in its 0th output.】
Attempting a solution: When commenting out the following code (train.py torch. autorad. set_detect-anomaly (True)) This error will occur again: U, S, V=torch. svd (H, some=False) torch._C._LinAlgError: cusolver error: CUSOLVER_STATUS_EXECUTION_FAILED, when calling cusolverDnSgesvd( handle, jobu, jobvt, m, n, A, lda, S, U, ldu, VT, ldvt, work, lwork, rwork, info). This error may appear if the input matrix contains NaN.】
Requesting your help. thank you

I think it might be helpful to first check whether the tensor corresponding to the source and target point clouds after overlap selection contains nan.


main-dev tests main?

I am looking at this line:
https://github.com/ashvardanian/SimSIMD/blob/0aea96c6048a58f63eed5797e028708ae9b5e15d/.github/workflows/prerelease.yml#L72
And it looks like the workflow for main-dev checks out the main branch instead to test.

If it is wrong, I can fix it with some other tweaks I am working on.

Also, you use checkout@3 and checkout@4. Any reason for the different versions?

Both are copy-paste issues and should be fixed, thank you, @sroussey!


Looks good to me


coprocessor: fix panic when Selection/TopN is the ancestor of Aggregation

Signed-off-by: Lonng<EMAIL_ADDRESS>
What have you changed? (mandatory)
Fix panic when Selection/TopN executor is the ancestor of Aggregation
What are the type of the changes? (mandatory)

Bug fix (change which fixes an issue)

How has this PR been tested? (mandatory)
unit tests

TiDB will never push down a Selection/TopN executor which is the ancestor of HashAgg, so close this PR.


link to Microsoft account on a github instead of adgroc account for Cognitive Service Text Analytics

link to Microsoft account on a github instead of adgroc account. Also added links to non-windows sdks

@SychevIgor,
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

@SychevIgor : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@azdocprs I hope that changes will be reviewed...

@kriscrider - Please review the proposed changes and add an appropriate comment so that automation can take appropriate action.  Thanks

#Sign-off


A paver (Python build and configuration utility) plugin

This add a plugin for Paver: http://paver.github.io/paver/ it has been tested on OSX, but not on Linux (however it should work also smoothly there).

+1 :+1:

Well, #2922 has already been merged in, but looks like this is more than ready.
@robbyrussell please merge.

/cc @robbyrussell


modular basins

An issue to start organizing how to potentially add basin modules to ucvm.
The cvmh repo may be a better place to share files ?
https://github.com/SCECcode/cvmh

Concept figure  by example of LA basin:

Are these basins going to be accessed through cvmh ? or are they going to
be separate standalone models?

From: Andreas Plesch @.>
Sent: Monday, October 4, 2021 9:48 AM
To: SCECcode/ucvm @.>
Cc: Subscribed @.***>
Subject: [SCECcode/ucvm] modular basins (#3)
An issue to start organizing how to potentially add basin modules to ucvm.
The cvmh repo may be a better place to share files ?
https://github.com/SCECcode/cvmhhttps://urldefense.com/v3/__https://github.com/SCECcode/cvmh__;!!LIr3w8kk_Xxm!_fOauK75dlZzUFnKvBMZJ7vHe0NhvrhpSoiKHGN9xYGrms4DIsFgKYox3aE$
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/SCECcode/ucvm/issues/3__;!!LIr3w8kk_Xxm!_fOauK75dlZzUFnKvBMZJ7vHe0NhvrhpSoiKHGN9xYGrms4DIsFgXE997R8$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ACE7UF6NBKGESHUT7KXCUUTUFHLEDANCNFSM5FJZUO5A__;!!LIr3w8kk_Xxm!_fOauK75dlZzUFnKvBMZJ7vHe0NhvrhpSoiKHGN9xYGrms4DIsFg-4eve6w$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!LIr3w8kk_Xxm!_fOauK75dlZzUFnKvBMZJ7vHe0NhvrhpSoiKHGN9xYGrms4DIsFgtKAGmQ4$ or Androidhttps://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!LIr3w8kk_Xxm!_fOauK75dlZzUFnKvBMZJ7vHe0NhvrhpSoiKHGN9xYGrms4DIsFgVmL1YDk$.

Hi there. Let's discuss this. I think the idea is that they are separate, so they can be developed further independently.

An example grid:

An example voxet:
CVMH15-1-Los-Angeles-Basin.zip

od can dump voxet files:
$ od -v -tf4 -w4 --endian=big  CVMHB-Los-Angeles-Basin/CVMHB-Los-Angeles-Basin_vp63_basin@@  | more

It has now for a while a --endian option.

Depth slice at -1900 with additional test point (circled) at:
421000.000000  3712000.000000  -1906

The test point is outside the HR area and close to nodata.
The HR area also has nodata points.

https://github.com/SCECcode/cvmhlabn


Missing requried property isInMultiAction

I'm getting the following error:
Error: error while validating payload: must have required property 'isInMultiAction' payload: {"event":"didReceiveSettings","action":"fun.shiro.counter.action","context":"sd-A00DA431142W9.Default.Keypad.2.0","device":"sd-A00DA431142W9","payload":{"settings":{"count":30,"steps":1},"coordinates":{"row":0,"column":2}}}

It may be related to me using opendeck, but for now I've got a local patch applied that makes isInMultiAction property optional.

Based on the official streamdeck sdk the property should not be optional.
I don't know anything about opendeck, but if you're only getting the error using that, I'm assuming, that the property has to be added there.

Alright, that makes sense. I'll close this issue


Error message

Hello! when i double click LedDashboard.exe a get an error message: "Error initializing Razer Chroma controller. Is Razer Synapse installed? Status = 6023" i press ok and then nothing. I have synapse 3 installed and chroma studio works fine.

Ok! made it work it was chroma connect problem! thenks for the great work. Plz make some more ADC champs. thnaks

Hi, i have the same error code, how you fix that?

Hi, i have the same error code, how you fix that?

I have the same issue, how did you solve yours?


Cannot upgrade hack using remote install

Just decided to share my experience, it might be useful for somebody. Also, there might be a bug with perms.
Goal: make some changes in hack sources (specifically, apply the change to adjust the volume via config from dev branch), build and upgrade some cam.
I've made some changes, then bumped version in wyze_hack/hack_ver.inc, then run build.sh script. Everything worked smooth, I've got release/<MY_NEW_VER>.
I tried the way with remote install, it worked pretty neatly for me for the initial install. So, I've run remote_install.sh in a new folder with a new version, run the process, it worked fine. But it didn't upgrade anything on camera.
So, I've tried the way with telnet install. I've copied the folder into NFS, and from camera run telnet_install.sh from that NFS folder. Then I saw this error:
/tmp/Upgrade/upgraderun.sh: line 8: /tmp/Upgrade/wyze_hack.sh: Permission denied
So, I went to telnet_install.sh and added one line:
chmod +x /tmp/Upgrade/wyze_hack.sh
/tmp/Upgrade/upgraderun.sh

Then I run it again and it worked this time. The camera rebooted and I've got updated stuff in /tmp/run/wyze_hack/hack_ver.inc and all other files.

thanks for reporting this. Yes, i confirmed this is a bug in my build script. It wasn't reproducing in my environment due to my own special setup. I've made a fix.


Code 1013 is reserved and may not be used

🛑 𝙎𝙏𝙊𝙋
When the server closes by sending a Close control frame, and there is no body, I received the following exception:
java.net.ProtocolException: Code 1013 is reserved and may not be used.
at okhttp3.internal.ws.WebSocketReader.readControlFrame(WebSocketReader.java:200)
at okhttp3.internal.ws.WebSocketReader.processNextFrame(WebSocketReader.java:103)
at okhttp3.internal.ws.RealWebSocket.loopReader(RealWebSocket.java:274)
at okhttp3.internal.ws.RealWebSocket$2.onResponse(RealWebSocket.java:214)
at okhttp3.RealCall$AsyncCall.execute(RealCall.java:206)
at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1152)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:627)
at java.lang.Thread.run(Thread.java:882)
I found this confusing because as stated in https://developer.mozilla.org/en-US/docs/Web/API/CloseEvent
it indices

the websocket server has session limit , why code 1013 is reserved and throw ProtocolException not invoking frameCallback.onReadClose(code, reason); and finaly
invoke listener.onClosing(this, code, reason);
is is bug ?
the okhttp version :

com.squareup.okhttp3
okhttp
3.12.0

The code mapping just needs updated to comply with https://www.iana.org/assignments/websocket/websocket.xml

@JakeWharton
static String closeCodeExceptionMessage(int code) {     if (code < 1000 || code >= 5000) {       return "Code must be in range [1000,5000): " + code;     } else if ((code >= 1004 && code <= 1006) || (code >= 1012 && code <= 2999)) {       return "Code " + code + " is reserved and may not be used.";     } else {       return null;     }   }
this source code is detect 1013  is reserved and may not be used. i dont't understand why  ?

Because that's what the original RFC mandates
On Mon, Jul 8, 2019 at 1:44 PM Change<EMAIL_ADDRESS>wrote:

@JakeWharton https://github.com/JakeWharton
static String closeCodeExceptionMessage(int code) { if (code < 1000 ||
code >= 5000) { return "Code must be in range [1000,5000): " + code; } else
if ((code >= 1004 && code <= 1006) || (code >= 1012 && code <= 2999)) {
return "Code " + code + " is reserved and may not be used."; } else {
return null; } }
this source code is detect 1013 is reserved and may not be used. i dont't
understand why ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/5275?email_source=notifications&email_token=AAAQIELZLINRIPWKGEEN4GTP6N4GFA5CNFSM4H656HT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZN2WHQ#issuecomment-509324062,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQIEM22ZQJG5EDVV2UQP3P6N4GFANCNFSM4H656HTQ
.


Error when run PVAnet_test

python dv.py --model ~/PVAnet_iter_70000.ckpt --net PVAnet_test
NotFoundError (see above for traceback): Key conv5_4/last_bn_scale/beta not found in checkpoint;
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/alpha not found in checkpoint
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
[[Node: save/RestoreV2_473 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_473/tensor_names, save/RestoreV2_473/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
[[Node: save/RestoreV2_473 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_473/tensor_names, save/RestoreV2_473/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
[[Node: save/RestoreV2_473 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_473/tensor_names, save/RestoreV2_473/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
[[Node: save/RestoreV2_473 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_473/tensor_names, save/RestoreV2_473/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
[[Node: save/RestoreV2_473 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_473/tensor_names, save/RestoreV2_473/shape_and_slices)]]
W tensorflow/core/framework/op_kernel.cc:993] Not found: Key conv5_4/last_bn_scale/beta not found in checkpoint
[[Node: save/RestoreV2_473 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_473/tensor_names, save/RestoreV2_473/shape_and_slices)]]

PVAnet is not ready yet. Test error is easy to fix. However, error rate is very high and not useable yet.

Was it solved?
There must be a typo in the test net.
I'll run some debug in my spare time

test error is easy to resolve, however PVAnet has very high error (mAP is below 0.1). It seems conv2_1/1/conv layer is different from the paper, can also be caused by issue in pva_negation_block/v2.

train is different from test, and both are different from Caffe version

Hi, there! I faced the same issue, but finally find the errors in PVAnet_test.py. Here is the lines I've changed to make the test work:
lines to comment:
85 --> .scale(c_in=384, name='conv5_4/last_bn_scale')
118 --> (self.feed('convf_rpn', 'convf_2')
119 -->   .concat(axis=3, name='convf'))
lines to change:
92 --> (self.feed('downsample', 'conv4_4', 'upsamle')   --->  (self.feed('downsample', 'conv4_4')
121 --> (self.feed('convf', 'rois')  ---> (self.feed('convf_2', 'rois')
122 -->  .roi_pool(7, 7, 1.0/16, name='roi_pooling')  ---> .roi_pool(6, 6, 1.0/16, name='roi_pooling')
Hope this will help!
BTW, as mentioned by @jwnsu, test error is quite high. I trained the model using VOC2012 data with 200000 iters, the mAP is about 0.03, which is totally unacceptable. I am gonna train the model with more iters, and meanwhile, check the PVAnet to see if it is the same with the caffe version mentioned by @autumnqin.
Will update if any news.

@baileyqbb Haven't you figured out the issue with low accuracy by any chance?

Hi guys, can anyone please share some tips on improving the mAP?

@baileyqbb @QueenJuliaZxx     do you run pva-faster-rcnn successfully?my pva.npy seem to be wrong.where do you download?what the script command do you run ?thanks for your help

@baileyqbb
thank you so much :+1:
it works very well :100: :1st_place_medal:


Implementation Status

The nnabla release of open-unmix is not feature complete with respect to our pytorch reference. The following issues would need to be solved

[X] Dataset parameters
[ ] Training parameters
[ ] Inference
[ ] Training/Inference on CPU (currently the nnabla LSTM does rely on cudnn)
[ ] Evaluation

How can I listen music on open unmix ?


SystemExit running units tests

I'm getting the same exception as https://github.com/Microsoft/vscode-python/issues/1529
Running on Ubuntu 18.04 python 3.7.2
I can reproduce with the trivial example below. This file resides in a subfolder in my project.
import unittest

class Test(unittest.TestCase):
    def test_(self):
        self.assertTrue(True)

if __name__ == '__main__':
    unittest.main()

launch.json is

    {
        "name": "Python: Current File (Integrated Terminal)",
        "type": "python",
        "request": "launch",
        "program": "${file}",
        "console": "integratedTerminal"
    },

I've also been unable to get 'Discover Unit Tests' to work, not sure if that's related. But I can run from the command line

python -m unittest tests

Works fine (I did have to add init.py and explicitly import tests from each file in .\tests by I guess that's normal)

Duplicate of #3201


Add schema validation

We know the graphql endpoint and we should be able to validate the query using the cached schema.

It would be great if you could validate queries in tests too, without having to make actual queries.

Now that we pipe queries into graphql-client, this is happening and can be closed, right?

True dat 👍


Upgrade to ahash v0.8

This is a resubmission of #1271, which was reverted in 6e46651, due to compilation errors with the wasm32 target. It builds on #1274 to fix those compilation errors.
Closes #1274.

@jorgecarleitao I believe this is ready to go. The clippy failures appear to be the result of new deprecations in v0.4.23 that are unrelated to this PR.

Thanks! has a conflict, unfortunately.

Thanks! has a conflict, unfortunately.

Boy, this PR is the gift that just won't stop giving. Fixed!


feat: TensorRT support

Description
TensorRT is also an important component of CUDA Toolkit.
It would be great if you could impl an func like install.cuda(tensorrt=7.2.3)

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Thanks for raising this!

/assign

We now have lib for you. https://github.com/tensorchord/envdlib/pull/10/files

Do you need to install binding with your DL framework?

How can I test if it works?
The error message from my setup(without envdlib) is
2022-11-01 11:46:15.238959: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvrtc.so.11.1: cannot open shared object file: No such file or directory
2022-11-01 11:46:15.239207: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvrtc.so.11.1: cannot open shared object file: No such file or directory

And I tried a build.envd like
envdlib = include("https://github.com/aseaday/envdlib")
def build():
    config.pip_index(url="https://mirror.sjtu.edu.cn/pypi/web/simple")                                                      base(os="ubuntu20.04", language="python3.8")
    envdlib.tensorrt(os="20.04", cuda="11.2.1", trt="7.2.3")
...

But envd build got Error: module has no .tensorrt field or method

@oocococo You can copy the function content directly to your file as a temporary test.
def tensorrt(os="20.04", cuda="11.6.2", trt="<IP_ADDRESS>"):
    """Install tensorrt

    Args:
        os (Optional[str]): os version
        cuda (Optional[str]): cuda version
        trt (Optional[str]): tensorrt version
    """
    run(
        [
            'CUDA_VERSION=%s && TRT_VERSION=%s && \
if [ "${CUDA_VERSION}" = "10.2" ] ; then \
    v="${TRT_VERSION%.*}-1+cuda${CUDA_VERSION}" &&\
    apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2004/x86_64/3bf863cc.pub &&\
    apt-get update &&\
    sudo apt-get install libnvinfer8=${v} libnvonnxparsers8=${v} libnvparsers8=${v} libnvinfer-plugin8=${v} \
        libnvinfer-dev=${v} libnvonnxparsers-dev=${v} libnvparsers-dev=${v} libnvinfer-plugin-dev=${v} \
        python3-libnvinfer=${v}; \
else \
    v="${TRT_VERSION%.*}-1+cuda${CUDA_VERSION%.*}" &&\
    apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2004/x86_64/3bf863cc.pub &&\
    apt-get update &&\
    sudo apt-get install libnvinfer8=${v} libnvonnxparsers8=${v} libnvparsers8=${v} libnvinfer-plugin8=${v} \
        libnvinfer-dev=${v} libnvonnxparsers-dev=${v} libnvparsers-dev=${v} libnvinfer-plugin-dev=${v} \
        python3-libnvinfer=${v}; \
fi'
            % (cuda, trt)
        ]
    )

@aseaday did you mean exec the run part manually?

It's just python function. You can copy that and call tensorrt(....) directly in your build function.

Author

Sorry for this inconvinience. Now you can use like this:
envdlib = include("https://github.com/tensorchord/envdlib")
def build():
    base(os="ubuntu20.04", language="python3")
    shell("zsh")
    install.cuda(version="11.6.2", cudnn="8")
    tensorrt()

Author

Sorry for this inconvinience. Now you can use like this:
envdlib = include("https://github.com/tensorchord/envdlib")
def build():
    base(os="ubuntu20.04", language="python3")
    shell("zsh")
    install.cuda(version="11.6.2", cudnn="8")
    envdlib.tensorrt()

I've update envd and use the default envdlib.tensorrt()，but still get error message in my the ipynb
2022-11-02 13:24:28.395692: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-11-02 13:24:28.521478: I tensorflow/core/util/util.cc:169] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2022-11-02 13:24:28.556504: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2022-11-02 13:24:29.198706: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory
2022-11-02 13:24:29.198832: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory
2022-11-02 13:24:29.198840: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.

I just run a example ipynb from sktime

Author

Sorry for this inconvinience. Now you can use like this:
envdlib = include("https://github.com/tensorchord/envdlib")
def build():
    base(os="ubuntu20.04", language="python3")
    shell("zsh")
    install.cuda(version="11.6.2", cudnn="8")
    envdlib.tensorrt()

I've update envd and use the default envdlib.tensorrt()，but still get error message in my the ipynb
2022-11-02 13:24:28.395692: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F AVX512_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-11-02 13:24:28.521478: I tensorflow/core/util/util.cc:169] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2022-11-02 13:24:28.556504: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2022-11-02 13:24:29.198706: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory
2022-11-02 13:24:29.198832: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory
2022-11-02 13:24:29.198840: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.

I just run a example ipynb from sktime

I will check it. It seems that the tensorflow binding with tensorrt.

Could you gimme your envd.build, because I found your tensorflow tried to found tensorrt7 but we had installed tensorrt7 only.

@aseaday sorry for the postpone
envdlib = include("https://github.com/tensorchord/envdlib")
def build():
    config.pip_index(url="https://mirror.sjtu.edu.cn/pypi/web/simple")
    base(os="ubuntu20.04", language="python3.8")
    config.apt_source(source="""
# See http://help.ubuntu.com/community/UpgradeNotes for how to upgrade to
# newer versions of the distribution.
    deb https://mirror.sjtu.edu.cn/ubuntu focal main restricted
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal main restricted
    deb https://mirror.sjtu.edu.cn/ubuntu focal-updates main restricted
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal-updates main restricted
    deb https://mirror.sjtu.edu.cn/ubuntu focal universe
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal universe
    deb https://mirror.sjtu.edu.cn/ubuntu focal-updates universe
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal-updates universe
    deb https://mirror.sjtu.edu.cn/ubuntu focal multiverse
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal multiverse
    deb https://mirror.sjtu.edu.cn/ubuntu focal-updates multiverse
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal-updates multiverse
    deb https://mirror.sjtu.edu.cn/ubuntu focal-backports main restricted universe multiverse
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal-backports main restricted universe multiverse
    deb http://archive.canonical.com/ubuntu focal partner
# deb-src http://archive.canonical.com/ubuntu focal partner
    deb https://mirror.sjtu.edu.cn/ubuntu focal-security main restricted universe multiverse
# deb-src https://mirror.sjtu.edu.cn/ubuntu focal-security main restricted universe multiverse
    """)
    install.python_packages(name = [
            "sktime[dl,all-extras]",
            "seaborn",
            "pyts",
            "ipykernel"
    ])
    install.vscode_extensions([
            "ms-python.python",
            "ms-toolsai.jupyter",
#            "GitHub.copilot"
    #        "ms-python.vscode-pylance"
    ])
    install.cuda(version="11.6.2", cudnn="8")
    envdlib.tensorrt()
    shell("zsh")
    install.apt_packages(name = [
            "linux-libc-dev",
            "htop",
            "fzf"
    ])

I found it was cause by the fact tensorflow used tensorrt 7. But we don't have a auto way to install tensorrt 7 because nvidia do not provide a apt package to install tensorrt 7.  Let me think of a way to do that.

@aseaday I make it work by installing TensorRT.tar manually with the official tutorial (https://docs.nvidia.com/deeplearning/tensorrt/install-guide/index.html#installing-tar). But during the installation process, things like ldconfig also makes me confused. Maybe envd can do all these things automatically?

@aseaday I make it work by installing TensorRT.tar manually with the official tutorial (https://docs.nvidia.com/deeplearning/tensorrt/install-guide/index.html#installing-tar). But during the installation process, things like ldconfig also makes me confused. Maybe envd can do all these things automatically?

Hi, I am discussing with TensorRT team about old version's pacakge management. I will help you to handle this problems this week. static link is really a hard thing for AI/ML engineers.


Hurricane matthew thumbnail broken

they are actually both broken right now still because we are doing CA & CO. This is still the work that is being finished for #32

So why is the issue closed now/

because #32 is still open and Hurricane-matthew isn't actually one of the thumbnails anymore

I guess I just closed it since this will close just because #32 closes, both can stay open!

gotcha, whoops. Closing


CRUD components - updating the chain and vuex

Create new Post
Update post
Delete post

@mihailmarcu @marinhoarthur Could you please re-review? I've tested using scaffold command and it all works for me - would like to merge.


add 42Angoulême

Thank you for these great tools you allow schools to use

@gpoblon First stage of verification passed, official domain is 42angouleme.fr as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


CvImage from cvMat with only one channel

Hi!
I'm working with a one channel buffer that contains only the luminance part of the image and would like to use FaceDetector.locateFaces.
The issue that I encountered is that CvImage expects only a cv.Mat with 3 channels:
const mat = new cv.Mat(buffer, height, width, cv.CV_8UC1);
const image = fr.CvImage(mat);
const detector = new fr.FaceDetector()
detector.locateFaces(image); // -> outputs no faces (there are 2)

const win = new fr.ImageWindow();
win.setImage(image); // -> will display a squashed version of the image, repeated 3 times on the horizontal axis.

As a workaround, one can duplicate the channel 3 times in order to fill the gap:
const fakeRGB = Buffer.alloc(buffer.length * 3);
for (let i = 0, j = 0; i < buffer.length; i++, j += 3) {
      fakeRGB[j + 0] = buffer[i];
      fakeRGB[j + 1] = buffer[i];
      fakeRGB[j + 2] = buffer[i];
}
const cvMat = new cv.Mat(fakeRGB, height, width, cv.CV_8UC3);
const cvImage = new fr.CvImage(cvMat);

detector.locateFaces(cvImage); // -> expected output
const win = new fr.ImageWindow();
win.setImage(image);  // -> normal image

Is there any other way around this that is less taxing because of the conversions?

Hi,
You should be able to simply convert your single channeled gray Mat to a three channel gray Mat with mat.cvtColor(cv.CV_GRAY2BGR).

Hi @justadudewhohacks. Thank you for the quick answer!
Substituting from the second example:
const cvMat = new cv.Mat(fakeRGB, height, width, cv.CV_8UC3);
const cvImage = new fr.CvImage(cvMat);

with
const cvMat = new cv.Mat(buffer, height, width, cv.CV_8UC1);
const cvImage = new fr.CvImage(cvMat.cvtColor(cv.COLOR_GRAY2RGB));

did the trick, and it seams to be less expensive as well.


Storages: enhance data sharing column cache

What problem does this PR solve?
Issue Number: close #6092
Problem Summary:
Using ColumnCache whose cache entry is a pack. ColumnSharingCache's cache entry is several packs.
What is changed and how it works?

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

Remove the duplicate code of getting data type on disk again: https://github.com/Lloyd-Pottiger/tiflash/pull/19


Typos in output: preffer -> prefer, preffered -> preferred

Code has following typos in its output:

preffer -> prefer
preffered -> preferred (note 'rr')

See git grep preffer (and e.g. Oxford language dictionary).
PS. while this bug is about user visible output, git grep Preffer shows also typoed variable name.

Fixed in db5fca7c07bf09dc0b7df51456b3403ebfacb2ba

Looks good, thanks!


feat(node): send cache requests to close peers

Why

Fixes #357
In testing.
On PUT cache requests tells peers that they should cache the data as well.
I think the reason for doing this is that we want the data to be replicated to other nodes
so that users closer to those peers can retrieve that same data.
My assumption is that this replication behavior should have a bound because
it wouldn't be too useful for nodes to tell peers that are far away (node is in San Francisco and peers are in Chicago? There seem to be connections this far apart at the moment) to cache some content. Can we assume that users in far way regions would want that same content most of the time?
Another idea I was toying with is using latency instead of location but latency is a bit volatile. Open to suggestions of course.
What

We still have a hash set of peers but in addition, we have a smaller set of peers that are closer to the node.
Graphsync is now using this subset. Everything else should be intact.
This functionality is not enabled by default so it doesnt break tests or break local testing.

Demo

Notes

Checklist

[ ] I have made corresponding changes to the tests
[ ] I have made corresponding changes to the documentation
[ ] I have run the app using my feature and ensured that no functionality is broken

@qti3e I know you said you were interested in regionizing the network. Please let me know your thoughts and if there are any ideas here that we could use. 🙏

Replaced by #427.


Unable to build cargo kit if package cache and flutter projects on different drives on Windows.

When gradle plugin runs run_build_tool.cmd it passes to script env variable %CARGOKIT_TOOL_TEMP_DIR%. Then sctipt make cd to dir that passed inside this variable. But if this variable points to directory that located on different drive than working directory for script, cd will not work. For it to work we need to pass /D parameter to cd. I made this change localy and it helps. Right now line 11 in my script looks like:
cd /D "%CARGOKIT_TOOL_TEMP_DIR%"

How i can contribute this changes to repo?

Hey! Can you make a PR for this here?
https://github.com/irondash/cargokit
Thanks!

Sure

Done.

Should be fixed in 0.8.3.


Add callback functions

Adding callback function capability will allow users to call a function when it finishes looping or when it returns to its original title/position. This will let it be better for the developers.
It should be a property like MarqueeTitle.callback and be set by evaluating MarqueeTitle.callback = function(){} || functionName;
There could be a parameter too for having variables like the amount of loops it has completed and more.

This didn't work out that great. And I don't see a use in it, but I though to make this feature for other developers that need it.


Fix trivial typos

Trivial typos

@elecharny Is this patch merged? Can we close this?

Hi,
the patch has been merged. This PR can be closed.

the patch has been merged. This PR can be closed.

Thanks!


[Bug]: Mouse drag lag.  Any other ways to avoid this?

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
My apologies.  I am aware of the mouse drag lag issue.  The old Vibrancy Extension had found a GREAT fix for bug; which I'm currently using.  However, after if I upgrade from VSCode 1.56.2 to a newer version, it doesn't work anymore.
Is there any hope for this extension without mouse drag lag?  This is the main reason why I stopped using Vibrancy.  Hopefully, you can find a new way to do this.
Expected Behavior
No mouse drag lag
Steps To Reproduce

Move VSCode window by dragging it with a mouse cursor.

Environment
- OS:
- VSCode:
- Extension:

Anything else?
No response

I haven’t been able to reproduce this issue on any of my devices, so I’ll need a bit more info to troubleshoot this.
What OS and version are you using? If it’s Windows 10/11, go to Settings → System → About to check.
Additionally, what is your display configuration? I’m curious about the amount of displays and refresh rates specifically.

@illixion thank you so much for responding so quickly!  This issue happens on both of my PC's (all major Windows 10 versions I've tried):
1903
1909
2004
20H2
21H1
22H2
My PC configurations:
PC1:           3 monitors:
Monitor1:  2560x1440 @60Hz
Monitor2:  1280x720 @60Hz
Monitor3:  1280x720 @60Hz
PC2:           2 monitors:
Monitor1:  1920x1080 @60Hz
Monitor2:  1920x1080 @60Hz
I also made a customized Vibrancy Extension:  MKANET Theme V2 which has no weird issues at all.. as long as I use VSCode Verson 1.56.2 or earlier.
Please let me know if there's anything else I can provide.

https://github.com/illixion/vscode-vibrancy-continued/assets/63966843/067ecba4-61f2-4c72-99e8-012751f21673
Windows 10 22H2 build 19045.3324
1920x1080 @144Hz

2023-09-04.20-35-18.mov
Windows 10 22H2 build 19045.3324 1920x1080 @144hz

Looks like you're also having the same issue.

Yeah

I'm still using VSCode 1.56.2 with the older version of Vibrancy Extension since that was the last version of VSCode that didn't have any issues at all.  It works flawlessly.

VSCode Version: 1.56.2
Extension:  MKANET Theme V2
https://github.com/illixion/vscode-vibrancy-continued/assets/811008/7e9fdb82-2f48-419f-a042-fd98c98d74a9

@illegitimate-egg I feel like I've seen this pattern of window movement before, specifically in cases where my GPU was struggling. Could you please clarify which kind of GPU you're using, and if it's integrated or dedicated?
I've also found this article that seems like it could help, especially since this seems to only occur in certain configurations: https://old.reddit.com/r/Windows11/comments/pj4fi8/fix_for_windows_dragging_lag_applies_to_windows/
There was a "Windows 10 fix" in Vibrancy that was disabled a while ago as it was causing issues with window snapping, I've compiled a VSIX with it restored, so you could try it and let me know if it resolves the issue for you: vscode-vibrancy-continued-1.1.20.vsix.zip
To install, simply go to the Extensions page and under the triple-dot menu, choose "Install from VSIX..." and select the decompressed file.

It's a dedicated GeForce GTX 1650, it shouldn't choke over moving a blurring window. After checking for IRQ overlap, the GPU overlaps with the PCIe Controller (which makes sense) and also the Motherboard's audio controller which is disabled, either way, I doubt the audio controller would start firing off IRQ 16 when I move a window.
The VSIX loads and installs, but transparency doesn't actually work (?) which is strange. VSCode looks exactly the same despite the plugin reload I did.

@illixion OMG Thank you!!!  I just tried vscode-vibrancy-continued-1.1.20.vsix.zip.  It actually works without any weird issues!  This is the first time I've seen the blur transparency effect on VSCode Versions: >1.56.2 without any weird issues.  I have had this issue since the original Vibrancy extension.  You finally fixed it!  Please update the official extension with this specific version of the extension.  It works very well from what I've tested so far.
There is no mouse lag or instability when dragging, moving, resizing!

I am also still experiencing this issue with my own setup:
Monitor 1: 144hz
Monitor 2: 60hz
GTX 1080Ti
VSCode: 1.85.2
Windows 10; Version 22H2; Build 19045.4046
I tried installing the vsix package of vibrancy above, but after reloading the extension and enabling the acrylic again, I still experience the issue with window movement lag. I'm not sure if its of any note, but I don't have any lag when resizing, only when moving the window around.


Homebrew formula

Is it possible to create a Homebrew formula to make installation on Mac easier?

Yes I would like this as well. Is there any way to install this on Mac?

Linux users will profit from this as well


ql-qlf: k6n10f: DSP: convert configuration ports to params

This PR changes the simulation modules, techmaps and passes in order to add configuration bits to MODE_BITS parameter instead of keeping dedicated configuration ports:

f_mode
output_select
saturate_enable
shift_right
round
register_inputs

@tpagarani, @rakeshm75 please take a look

This PR should be merged before https://github.com/chipsalliance/yosys-f4pga-plugins/pull/340.

@tpagarani Ok, so I will work on the second flavor


Is BERT a kind of cheating?

I know BERT has achieved SOTA in many NLP tasks, such as SQuAD and SWAG.
But note that the data (both training and test) of SQuAD is from Wikipedia, and that of SWAG is from the BookCorpus, and BERT is just pre-trained on this two corpora!
In other words, BERT has somehow "seen" the test data before test.
Is this a kind of cheating?

IMO it is more about learning language style than actual data. If language of new data set is like Wikipedia BERT works exactly as expected, but when data set is in different style like e.g. blogs BERT results are not good.

@chwang85  It's not cheating because when BERT was pre-trained on Wikipedia, it is trained to know how to predict missing words and classify if a pair of sentence follows from each other.
The task in SQUAD is to pick an answer from an input paragraph given a question. So, although the corpus is still Wikipedia, BERT was never trained on this particular task, so there is no cheating involved. Similar argument applies to SWAG.

I just want to express that BERT is over-estimated. An important reason for this over-estimation is that BERT significantly outperforms the previous SOTAs in SQuAD and SWAG, which use the same corpora as BERT does in its pretraining. If BERT selected corpora other than Wikipedia and BookCorpus to do its pretraining, and still achieved the same performance, that would be much more convincing.
As far as I know, in many tasks that are not based on Wikipedia and BookCorpus, BERT only brings very limited performance gain...

What do you think of using a pretrained BERT to further pretrained it on the target corpora before finetuning it to the target task like ULMFiT?

It will be OK as long as your"pretraining" and "further pretraining" do not involve any test data.

It's not any different from using a language model in machine translation to improve translation to that language. Or to use word2vec embeddings as features for a task on the same kind of data. What you are referring to is more like transfer learning. Transferring knowledge acquired from one domain or distribution and leveraging it for another. That said, I agree with you that BERT may not be the ImageNet of NLP as some have hoped (http://ruder.io/nlp-imagenet/). This remains to be shown...

You did not get my point.
SQuAD (both train and test) is completely from Wikipedia, and SWAG is from BookCorpus.
Now BERT is so popular mainly because it achieved SOTA on the two datasets.
But, why is BERT pretrained on Wikipedia and SWAG? Why is it not pretrained on other corpora?
Just using "transfer learning" to explain this is not convincing at all...

Are you assuming that the Squad questions are in the Wikipedia data used for training BERT? I don't think so, I could be wrong. If they are not, then I don't think training on wikipedia is anymore cheating than, for example, using a language model as a feature.

Wikipedia has no questions, questions in SQuAD are generated by human beings.
All passages in SQuAD (both train and test) are from Wikipedia.
The thing is, when a model has somehow "seen" the test data, it will be much easier for it to achieve good test result.
Suppose a student has seen all the materials (passages without questions is enough!) that might be used in a reading test, then he will definitely get a better mark than others.

By the way, BERT is also too intrusive to task models. You have to use the tokenization method given by BERT, but what if my text has already been tokenized in a different way?

I know BERT has achieved SOTA in many NLP tasks, such as SQuAD and SWAG.
But note that the data (both training and test) of SQuAD is from Wikipedia, and that of SWAG is from the BookCorpus, and BERT is just pre-trained on this two corpora!
In other words, BERT has somehow "seen" the test data before test.
Is this a kind of cheating?

A valuable question!

@erikchwang
The train and test data could come from the same type of data(wikipedia), but they are not the exact same passages, questions or answers.
The samples in the test data should be present in training data, but they should come from the similar data domain.
The idea that data should i.i.d explains it all.
This is my answer to your question. May be I am missing something very subtle from your question. Enlighten me.

Yes, you missed almost everything.
BERT's pre-training is on the FULL Wikipedia and FULL BookCorpus.
But for SQuAD, the fine-tuning of BERT is on a part of passages from Wikipedia, and the test is on another part of passages from Wikipedia.

@erikchwang
The train and test data could come from the same type of data(wikipedia), but they are not the exact same passages, questions or answers.
The samples in the test data should not be present in the training data, but they should come from the similar data domain.
The idea that data should be i.i.d explains it all.
This is my answer to your question. May be I am missing something very subtle from your question. Enlighten me.

Yes, you missed almost everything.
BERT's pre-training is on the FULL Wikipedia and FULL BookCorpus.
But for SQuAD, the fine-tuning of BERT is on a small part of passages from Wikipedia, and the test is on another small part of passages from Wikipedia.
So the pre-training data covers both the fine-tuning data and the test data, just in different formats (LM vs. MRC).

@chunduri11
Yes, you missed almost everything.
BERT's pre-training is on the FULL Wikipedia and FULL BookCorpus.
But for SQuAD, the fine-tuning of BERT is on a small part of passages from Wikipedia, and the test is on another small part of passages from Wikipedia.
So the pre-training data covers both the fine-tuning data and the test data, just in different formats (LM vs. MRC).


fix: repair useKeyboardJs hook

Description
Fixed the importing of the KeyboardJS library with backward compatibility. Updated how bindings are set to prevent KeyboardJS from calling handlers repeatedly while holding keys.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as before)

Checklist

[x] Read the Contributing Guide
[x] Perform a code self-review
[ ] Comment the code, particularly in hard-to-understand areas
[ ] Add documentation
[ ] Add hook's story at Storybook
[ ] Cover changes with tests
[x] Ensure the test suite passes (yarn test)
[ ] Provide 100% tests coverage
[x] Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
[x] Make sure types are fine (yarn lint:types).

I had a look at the tests, and it seems the existing code lacked tests because KeyboardJS is a peer dep. If you want me to add some let me know and I'll do so. I checked the storybook and the example works again.

Updated to resolve conflicts

@streamich Any chance we can merge this one?

:tada: This PR is included in version 15.2.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

No problem, when I do releases I'll check your hook periodically to make sure nothing breaks 🍻


Followbot not working

Fedoraware Version Date: Latest
Windows Version: Windows 10 Pro (The bot i'm trying to make it follow me also runs 10 Pro)
Configs used: (i dont have access to these atm, will update when im able to access these)

Description:
I have a VM that i set up as the bot that should follow me. No matter what i configure from the followbot tab, the bot doesn't do anyting and stays there

Steps to reproduce:
Make a VM with W10 and install TF2 on it
Be on a casual match with both accs and enable followbot on guest
Start to wonder why the hell does it not work

[x] With the steps below I am able to consistently reproduce this issue.

Steps to reproduce the behavior:

Additional info:

this issue looks really bad written im so sorry

We try not to support botting activities in general, if you could create a video of the issue occurring from the bots PoV, that would be great.

Frick- sorry, didnt know that one
Atm im not at my computer to send a video (will do it as soon as i reach my pc) but the thing is the followbot computer does absolutely nothing even if followbot is activated.


Skip can and upgrade gtest

Skip CAN tests if the interface is not available

Build status:

Linux
Windows

Build status:

Linux
Windows


[utils]: Fix use of va_list in FormatStringArgs

vsnprintf() will internally change the state of the passed in va_list. This results in incorrect logging as well as display output. Using va_copy() to create a temporary copy of the va_list fixes this issue.
This commit also changes the iteration variables for certain loops to match the correct size.

@KernelEquinox I noticed the logging functionality was just broken when it was saying a map had 210 rooms when the vector was only 14 elements long. This should fix the bug, though feel free to confirm yourself.


Sagemaker: Add update trial component

Hi,
Can I have more insight on the two test failed ? They don't seems to be related to anything I changed, so I'm little puzzled


Can't create a custom archetype when the root_id had an uppercase character.

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform:  v1.1.5
azure provider: 2.99.0
module: 1.1.4
Description
Describe the bug
When choosing a root_id with an uppercase character, one can't define custom archetypes that include the root_id like in the other built-in archetypes.
Steps to Reproduce

Set root_id to "Canary" or something with upper case.
Define a custom archetype file with the ID of "my_archetype"
Try to create a custom archetype by adding to the custom_landing_zone.

  custom_landing_zones = {
    "${var.root_id}-my-archetype" = {
      display_name               = "My Archetype"
      parent_management_group_id = "${var.root_id}-landing-zones"
      subscription_ids           = []
      archetype_config = {
        archetype_id   = "my_archetype"
        parameters     = {}
        access_control = {}
      }
    }
  }

Terraform fails to plan because the key to the map, in this case, "Canary-my-archetype" has an upper case character.

Screenshots

Additional context

I would expect the root_id to be more strict, requiring all lowercase characters.
Right now, I can create by using lower, but then it's inconsistent with the other archetypes.
  custom_landing_zones = {
    "${lower(var.root_id)}-my-archetype" = {
      display_name               = "My Archetype"
      parent_management_group_id = "${var.root_id}-landing-zones"
      subscription_ids           = []
      archetype_config = {
        archetype_id   = "my_archetype"
        parameters     = {}
        access_control = {}
      }
    }
  }

Thank you for raising this @fgauna12.
By design, we originally limited the inputs to lowercase, but in v0.1.0 we updated this. As you rightly pointed out, we appear to still have some inconsistencies in the validation rules and this should be an easy fix with no downstream customer impact.
I'll get this reviewed and added to our backlog. Please be aware that this fix will be applied to the latest version only, so you may need to upgrade.


crypto/perlasm/x86_64-xlate.pl: detect GNU as to deal with quirks

It turns out that GNU as and Solaris as don't have compatible ideas on
the .section syntax, so we need to check if we're using GNU as or
another assembler and adapt this .section syntax accordingly.
Fixes #11132

This fails in my solaris env due to the quotes around ".note.gnu.property" for the non-gnuas case. Without quotes works fine.

OK, I will fix

Looking better now?

Yep working now, thanks!

This pull request is ready to merge


Issues of evaluate code

Hi ,
when I run the evaluate.py , There is only " gt" result , no "prediction " result .What code should I modify to get the "prediction" result ! Thanks

I have the same issue, is anyone able to solve it?

@HanochZzhou @bertjiazheng Can you show your  gt" results screenshot.?
let me help you then.

I have the same issue.Everyone knows how to fix it?

I have the same issue.Anyone knows how to fix it?

@bertjiazheng Have you got the prediction result in your 'evaluate folder'? Did you fix your command line to run evaluate.py? Thanks.

@jcliu0428
Run with command line:
python evaluate.py --dataFolder=*** --numImages=760 --methods=2000000 --numOutputPlanes=10

@bertjiazheng Have you change anything in checkpoint.ckpt?
When I try your command line,it shows that:
InvalidArgumentError (see above for traceback): Unsuccessful TensorSliceReader constructor: Failed to get matching files on checkpoint/planenet_hybrid3_bl0_dl0_ll1_pb_pp_ps_sm0/checkpoint.ckpt: Not found: checkpoint/planenet_hybrid3_bl0_dl0_ll1_pb_pp_ps_sm0; No such file or directory

You don't need to modify anything. Make sure the methods=2000000, 0 will disable the corresponding method.

@bertjiazheng I modify ALL_METHODS,it also works.Thanks a lot!

Hello, I'm confused about the python evaluate.py output result

what does it mean?
Emm, where should I find the recall ?

@Qtodd you can find the pixel recall here and plane recall here.

@bertjiazheng sorry to bother you again
when I run the command line:
python evaluate.py --dataFolder=*** --numImages=760 --methods=2000000 --numOutputPlanes=10
it show that:
Traceback (most recent call last):
File "evaluate.py", line 1747, in
evaluatePlanes(args)
File "evaluate.py", line 538, in evaluatePlanes
plotResults(gt_dict, predictions, options)
File "evaluate.py", line 691, in plotResults
plotCurvesSplit(xs[metric_index], curves, filenames = [filename + '.png', filename + '_oracle.png'], xlabel=xlabels[metric_index], ylabel='Per-pixel recall', title=curve_titles[metric_index], labels=curve_labels)
File "/home/Qtodd/Plane/PlaneNet/utils.py", line 2337, in plotCurvesSplit
plt.plot(x, ys[order], figure=fig, label=final_labels[order], color=colors[order], marker=markers[order], linewidth=sizes[order])
IndexError: list index out of range

Hey, when I run the evaluate.py, I only got the prediction of PlaneNet.
What can I do to get the results of other existing methods ?? @jcliu0428 @bertjiazheng

Why is there no quantitative comparison output when I run evaluate.py?Looking forward to your reply. Thank you!@bertjiazheng

Hi, @DRAhmadFaraz, I have got the predictions of PlaneNet, but the per-pixel recall seems to be lower than that in the paper. Can you reproduce the evaluation in the paper?
Here are the results
Per-Pixel Recall: [0. , 0.11259292, 0.39307635, 0.50202066, 0.53868158, 0.55395703, 0.57877339, 0.57877339, 0.60665465, 0.60665465, 0.60665465, 0.62022231, 0.62022231]
Per-Plane Recall: [0. , 0.16, 0.32, 0.36, 0.38, 0.42, 0.46, 0.46, 0.48, 0.48, 0.48, 0.5 , 0.5 ]

Hello, when I run python evaluate.py, I do not know what the reason is and do not output quantitative results such as rel, rmse, etc. I hope you take the time to reply to me. Thank you!

Hello, when I run python evaluate.py, I do not know what the reason is and do not output quantitative results such as rel, rmse, etc. I hope you take the time to reply to me. Thank you!
@bertjiazheng

Hey, when I run the evaluate.py, I only got the prediction of PlaneNet.
What can I do to get the results of other existing methods ?? @jcliu0428 @bertjiazheng

Hello, why do I run the test code: python evaluate.py --dataFolder="./data" --numImages=760 --methods=2000000 --numOutputPlanes=10, there is no quantitative result output (such as rel, rmse, etc.)? Hope you take the time to reply! ! Thank you very much! ! !

@HanochZzhou @bertjiazheng Can you show your gt" results screenshot.?
let me help you then.
Hello, when I run python evaluate.py, I do not know what the reason is and do not output quantitative results such as rel, rmse, etc. I hope you take the time to reply to me. Thank you!

Hello, I'm confused about the python evaluate.py output result

what does this mean?
Emm, where should I find the recall ?
@bertjiazheng @art-programmer Thanks a lot~

Hello, you run the test code: python evaluate.py Is there quantitative output (such as rel, rmse, etc.)? Hope you take the time to reply! ! Thank you very much! ! !

Hi ,
when I run the evaluate.py , There is only " gt" result , no "prediction " result .What code should I modify to get the "prediction" result ! Thanks

Hello, you run the test code: python evaluate.py Is there quantitative output (such as rel, rmse, etc.)? Hope you take the time to reply! ! Thank you very much! ! !


Boilerplate v4.x app via Atlas

you might want to steal a little from parole or we should update atlas to use hooks and what not

I agree we need to update Atlas and perhaps move some components to their own repos. Still not sure about the reducer functions.

Yes, do we mono repo the components?
If you can find a pattern that you like better than reducers, I will be happy to check it out.


pass toolchains to maven_* rules

e.g. to pass a config setting for variabel substitution
string_flag(
    name = "version",
    build_setting_default = "0.0.0-dev",
    make_variable = "VERSION",
    visibility = ["//visibility:public"],
)

maven_bom(
    name = "bom",
    dependencies_maven_coordinates = "com.examplet:bom:$(VERSION)",
    java_exports = [
        # ...
    ],
    maven_coordinates = "com.examplet:bom:$(VERSION)",
    toolchains = [":version"],
)

Can you add a test that utilizes this code path? Something in your example that expands the templated variable with a defualt value will be good.

Hej Jin, apologies for the delayed response, somehow I'm not receiving notifications on GitHub.
I've just added a test with it in action and re-generated the docs, please let me know if there's anything missing! :)


silence build failures of futures-rs until https://github.com/rust-la…

…ng-nursery/futures-rs/pull/1445 is merged

Thanks!


metadata element is not clearly described

In particular it is unclear what entity it apples to. Consider the example in section 5.3.1. Where metadata contains a "client_id" that is not defined anywhere.  I assume it applies to the verifier, but that is not made clear anywhere.

The metadata area was left as a free form area for any Verifier-specific properties that may be expressed about a given input being requested. I am was only aware of one group using it, so one might argue for removal.

The metadata area was left as a free form area for any Verifier-specific properties that may be expressed about a given input being requested. I am was only aware of one group using it, so one might argue for removal.

We will add more text noting that it is a Verifier-set property with custom fields they can define for their own purposes.

We will add more text noting that it is a Verifier-set property with custom fields they can define for their own purposes.

Action: will pull this prop

PR #198 removes the metadata property.


fix: correct the function prototype defined in the stub file in simplest_c example

In the simplest_c example, the stub file defines the function square to take two arguments. However, it only takes one in the C-extension implementation. This PR corrects the stub file to make it consistent with the C-extension implementation.

Thanks!


[Bug][Master] send ACK event timeout

Purpose of the pull request

close #15345

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (ce11674) 37.81% compared to head (f5d5993) 37.81%.

:exclamation: Current head f5d5993 differs from pull request most recent head 083388c. Consider uploading reports for the commit 083388c to get more accurate results

Files
Patch %
Lines

...er/server/master/event/TaskResultEventHandler.java
0.00%
9 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15346      +/-   ##
============================================
- Coverage     37.81%   37.81%   -0.01%
  Complexity     4701     4701
============================================
  Files          1304     1304
  Lines         45217    45181      -36
  Branches       4859     4837      -22
============================================
- Hits          17099    17085      -14
+ Misses        26255    26234      -21
+ Partials       1863     1862       -1

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


no such table: django_netjsongraph_snapshot

this is on a freshly synced gentoo 32-bit system
Edit: missed this part:
I used the CLI to create the database:

./manage.py migrate
./manage.py createsuperuser

then I tried to delete it from the admin/web panel and I got the following:

django-netjsongraph (0.3.4)  - Reusable django app for collecting and visualizing network topology
INSTALLED: 0.3.4 (latest)
creating the database from the CLI

./manage.py migrate
and then trying to delete it from the admin web interface, I get:

Django Version: 1.11.2
Python Version: 3.4.6
Installed Applications:
['django.contrib.admin',
'django.contrib.auth',
'django.contrib.contenttypes',
'django.contrib.sessions',
'django.contrib.messages',
'django.contrib.staticfiles',
'rest_framework',
'openwisp_utils.admin_theme',
'django_netjsongraph']
Installed Middleware:
['django.middleware.security.SecurityMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware']

OperationalError at /admin/django_netjsongraph/topology/

no such table: django_netjsongraph_snapshot

@acoul what are you trying to delete?

updated the first post

@acoul I'm sorry, I still don't understand what object you are trying to delete and is giving you the error:

then I tried to delete it from the admin/web panel and I got the following:

Could you please clarify what does "it" refers to?

I tried to delete the whole database from the admin/web GUI
I suspect that I may have a mixed stable/snapshot django-netjsongraph installation on my gentoo system. I will further investigate this and do a follow-up.

Likely. If you find a bug that is possible to replicate let me know. I don't have any good suggestion with the current info. Fed

it was a mixed stable/snapshot django-netjsongraph installation. straighting this out produced a fully operational result. Sorry for the noise.


Pat binding

@pavel-v-chernykh @alexeykudinkin please review. This is needed in #109

:+1:

I will rebase my PR as soon as this one will be merged


fix(deps): update dependency @google-cloud/common to ^0.23.0

Move common to devDependencies. It is only used in
errors-transport.ts system tests.
Fixes: https://github.com/googleapis/nodejs-logging-winston/pull/151

Let me fix Node 6 CI.

It's possible we haven't shipped nodejs-common-grpc in a while

OK, passes with new logging. Ready for review


Unable to generate a URL for the named route  "" in Symfony 3.2

Hello,
When adding this field
            ->add('company', Select2EntityType::class, [
                'class' => 'ApiBundle:Company',
                'primary_key' => 'id',
                'multiple' => false,
                'text_property' => 'name',
                'minimum_input_length' => 2,
                'allow_clear' => true,
                'placeholder' => 'Select a Company',
                'required' => false,
                'empty_data' => null,
            ])

I get the following error:

Even if i remove all options from the field and do a simple
 ->add('company', Select2EntityType::class, []) i would get the error.
If I remove the field or put a regular entity field everything works perfectly.
The bundle is itself properly installed.
Any leads on where that could come from?
Thanks guys!

If you want to use Ajax you need to set the remote route in your parameters.
Le 15 mars 2017 6:55 PM, "Augustin Pasquini"<EMAIL_ADDRESS>a
écrit :

Hello,
When adding this field
        ->add('company', Select2EntityType::class, [
            'class' => 'ApiBundle:Company',
            'primary_key' => 'id',
            'multiple' => false,
            'text_property' => 'name',
            'minimum_input_length' => 2,
            'allow_clear' => true,
            'placeholder' => 'Select a Company',
            'required' => false,
            'empty_data' => null,
        ])

I get the following error:
[image: image]
https://cloud.githubusercontent.com/assets/1116475/23945169/c55684ac-0975-11e7-90ac-e97d9e2ab97c.png
Even if i remove all options from the field and do a simple
->add('company', Select2EntityType::class, []) i would get the error.
If I remove the field or put a regular entity field everything works
perfectly.
The bundle is itself properly installed.
"sensio/framework-extra-bundle": "^3.0", too.
Any leads on where that could come from?
Thanks guys!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tetranz/select2entity-bundle/issues/71, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABHrzyjLFAH0YJIxxS0kZ4hwO0EW-UnHks5rl8OpgaJpZM4MdyGa
.

Yeah I just realised that!
I don't want to use Ajax, though! That's the point!

I see there's a merge request currently for that feature https://github.com/tetranz/select2entity-bundle/pull/68 , it would be indeed nice to merge it! 😄

On that case there is an unfinished PR (still requires polishing) #68
Le 15 mars 2017 7:00 PM, "Augustin Pasquini"<EMAIL_ADDRESS>a
écrit :

Yeah I just realised that!
I don't want to use Ajax, though! That's the point!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tetranz/select2entity-bundle/issues/71#issuecomment-286709151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHrz0ejeyIAjlIHfShw0H3wc3hIa_gZks5rl8TbgaJpZM4MdyGa
.


Can't load plugin

Hi there, when following gcp-jupyter-sql/example/query-sql-pandas.py I receive the error:
NoSuchModuleError: Can't load plugin: sqlalchemy.dialects:postgresql.pymysql
I've done some Googling but no obvious solution. Any advice you can offer?
Cheers

My bad, I should have had postgresql+psycopg2


Extending dpump example

Greetings!
I'm working on a project that involves implementing a proprietary protocol between usb host and device using usbx, and since the hardware is not availble yet, the interim plan is to implement on linux. I was able to compile and run demo_usbx.c on linux, however I'm facing few issues

after tx_demo_thread_host_sim and tx_demo_thread_dev_sim threads exit, the app doesn't exit forever (ctrl-c terminates the app) -- is there an explanation to this behavior and a workaround on how to make the app exit gracefully?
threadx publishes regression tests on github, however, I couldn't find any for usbx; since the apis for host_sim and dev_sim are not documented, its unclear which programming apis to use from dev_sim or host_sim side to connect/disconnect, actiate/deactivate, exit gracefully etc

I would really appreciate if you could provide a hid-mouse example that exercises ux_hcd_sim_host and ux_dcd_sim_slave apis showing connect/disconnect, actiate/deactivate, graceful exit etc?
Thank you.

closing


User-assigned vs System-assigned identites

Is it possible to run this application using User-assigned Identities instead of the System-assigned ones? I know I can assign the user-assigned identities to the app services and grant the user-assigned identity the same roles that are being assigned to the system-assigned identities, but will the application code support it? Is there any further configuration needed to make it work?

Hi @KyleHammackRS ,
Thanks for raising the query. Please give us sometime and we will get back to you with proper response soon

@gsv022 This deployment currently deploys all resources with system-assigned identities and then assigns RBAC roles to those identities. I have need to grant a customer access to run the communicator deployment frequently who is not allowed to have access to Azure RBAC assignments. I can easily create a User-Assigned Identity, grant all RBAC roles to that User-Assigned Identity, and modify the ARM template resources to use that identity instead of the System-Assigned Identity.
My question for you is if I do that will the application still work as expected? Is there anything additional I need to do to make this work?

@KyleHammackRS Based on the existing design, using User-Assigned Identity would not affect the functionalities. However, as our support is limited with the default implementation and without updating the deployment template and thoroughly testing the functionality we cannot confirm on this to ensure that there are no impacts on the CC functionalities.

Not a problem, I already made template updates, just wanted confirmation that the code would support it, and if there were any app configs needed.

Please let us know if you have any queries with the standard version of Company Communicator otherwise, we can make it to closure

nope, good thanks


Enabling linter: wsl

What changed?

Enables the Whitespace Linter
Nothing was changed besides adding and removing lines

Why?
golangci-lint provides a variety of linter that can help us to keep code consistency in place, and minimize friction during reviews by helping to fix nits comments that happen quite often.
Here is the full list of linters: https://golangci-lint.run/usage/linters/
Here are the rules this linter will enforce: https://github.com/bombsimon/wsl/blob/master/doc/rules.md#anonymous-switch-statements-should-never-be-cuddled

How did you test it?

Release notes

Documentation Changes

@luizbafilho Is this rule enforceable within gofmt? I rely on my editor to format on save, and if these rules cannot be enforced by gofmt (or some other formatter we all run), then it will be a pretty major blocker for me.

You'd have to integrate golangci-lint in your editor: https://golangci-lint.run/usage/integrations/

I think you are confusing gofmt with linter, those are not the same thing, we already have golanglint-ci enable in the project, and it's already catching some stuff, you are compiling with it, I'm just adding an additional one initially. It's not just about whitespace.

If there are linting errors around whitespace, can I get golangci-lint to fix them for me?

I think you are confusing gofmt with linter, those are not the same thing, we already have golanglint-ci enable in the project, and it's already catching some stuff, you are compiling with it, I'm just adding an additional one initially. It's not just about whitespace.

If there are linting errors around whitespace, can I get golangci-lint to fix them for me?

nope, that linter specifically does not provide autofix, others do.


feat: cutover to use new BT-API-KEY header authentication

Description

Testing required outside of automated testing?

[ ] Not Applicable

Screenshots (if appropriate):

[ ] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


use with gulp

Hi!
I just started pixi.js and it seems marvelous!
But I can't make it totally work with Gulp and WebPack...
On Chrome (Mac), everything work but on Safari (iPhone) i get this error: fs.readFileSync is not a function from function FXAAFilter.
I get the same error on desktop when I use forceFXAA: true.
My gulp scripts task :
	gulp
		.src(config.scripts.src)
		.pipe(gulp_webpack({
			node: {
				fs: "empty"
			},
			module: {
				loaders: [
					{
						test: /\.js$/,
						exclude: /(nodes_modules)/,
						loader: "babel-loader",
						query: {
							presets: ["es2015"]
						}
					},
					{
						test: /\.json$/,
						loader: "json"
					}
				],
				postLoaders: [
					{
						include: path.resolve(__dirname, "nodes_modules/pixi.js"),
						loader: "transform?brfs"
					}
				]
			},
		}))
		.pipe(gulp_concat(config.scripts.out))
		.pipe(gulp.dest(config.scripts.dst));

Been posted a few times, may this can help?
https://github.com/pixijs/pixi.js/wiki/Using-Pixi.js-with-Webpack

Nope, none of this work... Still got the same error... none of the filters are "loadable/usable"...

brfs is the thing that resolved those into strings, if they aren't getting resolved then brfs isn't running properly.
Maybe post for help on the forums or from one of the webpack guys? I honestly don't know :(

v4 uses glslify instead of brfs to load and compile shaders into the build step. Fortunatly glslify has a webpack plug in too so you should be good to go :+1:


Docker v23 on linux causes devcontainer dockerfile to fail with eof

I have a pretty simple setup
.devcontainer/devcontainer.json
{
  "build": {
    "dockerfile": "Dockerfile"
  }
}

.devcontainer/Dockerfile
FROM mcr.microsoft.com/devcontainers/base:debian

When I execute:
devcontainer build --workspace-folder .

It fails with:
[8 ms] @devcontainers/cli 0.29.0. Node.js v18.12.1. linux 5.15.0-58-generic x64.
[131 ms] Start: Run: docker buildx build --load --build-arg BUILDKIT_INLINE_CACHE=1 -f /tmp/devcontainercli-nick/container-features/0.29.0-1675436665362/Dockerfile-with-features -t vsc-pdx-tools-c6e30ed9b67aad39ca1d11866a63d38e --target dev_containers_target_stage --build-arg _DEV_CONTAINERS_BASE_IMAGE=dev_container_auto_added_stage_label /home/nick/projects/pdx-tools/.devcontainer
[+] Building 0.1s (4/5)
 => [internal] load .dockerignore                                          0.0s
 => => transferring context: 2B                                            0.0s
 => [internal] load build definition from Dockerfile-with-features         0.1s
 => => transferring dockerfile: 433B                                       0.0s
 => [internal] load metadata for mcr.microsoft.com/devcontainers/base:deb  0.0s
 => CACHED [dev_container_auto_added_stage_label 1/1] FROM mcr.microsoft.  0.0s
 => preparing layers for inline cache                                      0.1s
ERROR: failed to receive status: rpc error: code = Unavailable desc = error reading from server: EOF
Error: Command failed: docker buildx build --load --build-arg BUILDKIT_INLINE_CACHE=1 -f /tmp/devcontainercli-nick/container-features/0.29.0-1675436665362/Dockerfile-with-features -t vsc-pdx-tools-c6e30ed9b67aad39ca1d11866a63d38e --target dev_containers_target_stage --build-arg _DEV_CONTAINERS_BASE_IMAGE=dev_container_auto_added_stage_label /home/nick/projects/pdx-tools/.devcontainer
    at pie (/home/nick/.volta/tools/image/packages/@devcontainers/cli/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:1916:1698)
    at async vF (/home/nick/.volta/tools/image/packages/@devcontainers/cli/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:1915:1972)
    at async uoe (/home/nick/.volta/tools/image/packages/@devcontainers/cli/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:2045:26894)
    at async aoe (/home/nick/.volta/tools/image/packages/@devcontainers/cli/lib/node_modules/@devcontainers/cli/dist/spec-node/devContainersSpecCLI.js:2045:25017)
{"outcome":"error","message":"Command failed: docker buildx build --load --build-arg BUILDKIT_INLINE_CACHE=1 -f /tmp/devcontainercli-nick/container-features/0.29.0-1675436665362/Dockerfile-with-features -t vsc-pdx-tools-c6e30ed9b67aad39ca1d11866a63d38e --target dev_containers_target_stage --build-arg _DEV_CONTAINERS_BASE_IMAGE=dev_container_auto_added_stage_label /home/nick/projects/pdx-tools/.devcontainer","description":"An error occurred building the image."}

I'm assuming this has something to do with Docker v23 (released a couple days ago) making buildkit the default on linux.
I'm happy to provide more info about my environment if that'll help -- or if there is a better repo to report this issue.

I posted in another issue, https://github.com/microsoft/vscode-remote-release/issues/6814#issuecomment-1416245446, but have a temporary solution

          We got an answer in docker buildx project, a fix is in the pipe: https://github.com/docker/buildx/issues/1325#issuecomment-1416444770

Originally posted by @jaudiger in https://github.com/microsoft/vscode-remote-release/issues/7958#issuecomment-1416495001


Tree is showing a trunk of unuseful folders

Issue
I tend to use detective in repos that have also API, backend and BFF implementations. That means that our front-end code resides in a deeper folder, e.g. my entry point is at /App/FrontEnd/src/main.ts; this means no relevant analyzable code is outside the folder /App/FrontEnd/src; I would like to see the tree pruned with these unuseful folders.
I came up with the idea to change the last few lines of inferFolders() to:
  let converted = toFolder(root);
  while (converted.folders.length === 1 && converted.folders[0].folders.length === 1) converted = converted.folders[0];
  return converted.folders;

This seems to work for me, but I realize this will also prune folders that do contain interesting stuff; looking  for a better idea.

Hi,
Thanks for pointing this out. Yes, this is something we should look into.
I'm thinking about adding a context menu to each tree entry. It could contain an option "focus". If an item has the focus, it becomes the root node of the shown tree. We could also store this setting in the detective.conf.json. And we need a reset button somewhere.
What do you think about this idea?
Best,
Manfred

I see what you mean, so I implemented it, see https://github.com/angular-architects/detective/pull/30


test(iroh): Disable DNS discovery altogether in tests

Description

Breaking Changes

Notes & open questions

Change checklist

[ ] Self-review.
[ ] Documentation updates following the style guide, if relevant.
[ ] Tests if relevant.
[ ] All breaking changes documented.

 But this is not the solution. Will need to think about this & talk to flub&others about this. We can reopen if we want.


Fix for File explorer not showing up and multiple notepads

Summary of the Pull Request
This PR fixes two issues, namely File Explorer not showing up and multiple notepads showing up due to multiple shortcut files in different locations.

PR Checklist

[x] Applies to #3223, #3180, #3599
[ ] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
Changes made in this PR are -

Previously we were checking if the LnkResolvedPath is null or empty, if so we were removing that program as a duplicate. However, this was true only in case of exe's and not lnk's. Hence, added an additional check to remove only the exe applications which do not have an lnkResolvedPath. This solves the issue of applications like File Explorer not showing up because they are lnk files without a resolvedPath.
The check that we previously had wherein we removed apps with the same LnkResolvedPath was not sufficient because shortcuts in different locations have different lnkResolved paths  (as mentioned in this comment - https://github.com/microsoft/PowerToys/issues/3180#issuecomment-631083296). Therefore, using the condition that if two applications have the same name, same executable name and same FullPath, they are qualified as the same item.

Validation Steps Performed

Created a copy of the notepad.lnk in multiple locations as mentioned https://github.com/microsoft/PowerToys/issues/3180#issuecomment-631083296 and validated that they don't show up twice.
Added tests for all the deduplication cases.

Probably unrelated to the PR, but I just noticed that start menu search doesn't actually find shortcuts on the desktop. I tried creating a shortcut to an unindexed file on desktop and Start menu doesn't find it but PT Run does.

What will happen if two shortcuts only difference in the start parameter for the linked app?
Is this then covered by different values in LnkResolvedPath?

What will happen if two shortcuts only difference in the start parameter for the linked app?
Is this then covered by different values in LnkResolvedPath?

Yes, the lnkResolved paths would be different then. The .lnk files contain information about the object that they are pointing to and also the arguments that must be passed to that object. Since they differ in their arguments, they would be different shortcuts as they do different things.  Hope this clarifies your question.
Ref - https://docs.microsoft.com/en-us/windows/win32/shell/links?redirectedfrom=MSDN#link-files


Remove script dependecies and install them instead

[ ] remove bootstrap script in public\index.html
[ ] install bootstrap via npm

package-json should show dependencies
dependencies should be installed on local machine by using the npm install command
(node_modules will stay on local machine leave part of .gitignore)

Hello @TimA314, I would like to solve this.

Hello @TimA314, I would like to solve this.

Hi unfortunately this was actually just solved. This project is still in its infancy and has so much work to do. If you know of something you would like to work on feel free to post an issue and assign yourself. I will continue to post more issues in the future!


Use CDIViews

Hey, I like your layout very much.
But I'm not able to use CDIViews for navigation.
Is it possible to use CDIViews?

Checkout Commit 83b649be89dd1d95cc1847fb86b6c0e669e7bf17

I have made added changes to the current api, please check if it works now, If it doesn't reopen this issue please


[Suggestion] Allow players to hide certain conduits when holding a conduit

You can hide conduits using the yeta wrench, which is a very useful feature, but that doesn’t work when you’re holding a conduit. Can you allow shift-scroll to work when holding conduits, not just the wrench?

Suggestions are tracked here: ideas.theideawall.com/EnderIO.
"Conduit Hiding" has already been suggested.  The conversation on that subject suggested allowing the Yeta Wrench to hide conduits from the off-hand.

I don’t want to use the yeta wrench from my left hand; I just want to hide conduits whilst holding conduits. That’s probably much easier to implement.

Because their is no voting system here, so I have no idea which suggestions have any kind of wider support in the player base and which ones are just made by loud people.

@HenryLoenwind i too hate it to create an additional login just to vote... is just found http://poll.gitrun.com/
it is open source and it features github authentication but you can only manage entrys for gits you have access too ... and it would only create new issues, i will see if i can work into the code and give it a function where it fetches the "Suggestion" label and makes an automatic list of all those issues