Dispose stream after creating icon?

FYI @punker76  @Lakritzator
I've had this PR from a fellow developer dangling around for quite a while:

I didn't dispose that stream because the Icon now owns it, which should take care about actually cleaning it up. I obviously can't guarantee that, but by closing it myself right after creating the Icon instance, I might mess things up for the Icon.
OTOH, I was reported potential memory leaks when using the NotifyIcon to do animations. What's your 0.02$ on this?

For bitmap it's bad practice to dispose the stream, I'm sure it's the same for icons.
The workaround for Bitmap is do something like:
using (stream)
using (var bitmap= new Bitmap(stream)) {
 return new Bitmap(bitmap);
}

I'm quite sure it's similar for icons!
Also using icon with a stream will AFAIK not support some of the newer features.
I'm sure I have code for that, will report back.

@Lakritzator But using does the same, it calls the Dispose of the stream. Isn't it?

Btw:
This actually also touches the following PR, it seems someone invested a lot of time :grin:
https://bitbucket.org/hardcodet/notifyicon-wpf/pull-requests/11/added-different-ways-to-set-the-icon/diff
Which came from: https://bitbucket.org/hardcodet/notifyicon-wpf/pull-requests/9/changed-imagesource-to-bitmapsource/diff
It's an overlap, as the stream "issue" is in both.

@Lakritzator But using does the same, it calls the Dispose of the stream. Isn't it?
Yes, it's not about the dispose, the point in the example code is that I create a new bitmap of the bitmap which was created from the stream, stupid hack but only than one can dispose the stream.

@Lakritzator The snippet there looks dangerous. Despite the wrapping, you return a new Bitmap that will operate on a disposed object (the wrapped Bitmap). If that works, it still looks like a timebomb that rather coincidentally works... Thoughts?

I believe that this is not a simple question, which it somehow never is. :cry:
In this example every time this is called, a stream is opened for something which is already there. When using this from animations, I could imagine caching it somehow. Also when using icons there is this size issue, as the render size should depend on the DPI settings. I believe there is room for improvements when we look at the icon handling.
Maybe we should focus on this "dispose" issue, see if we can solve this and than create something where we discuss what functionality we want to provide for the icons... like animations, resizing and different formats?

Btw:
This actually also touches the following PR, it seems someone invested a lot of time 😁
https://bitbucket.org/hardcodet/notifyicon-wpf/pull-requests/11/added-different-ways-to-set-the-icon/diff

cough :)
Yes, let's revisit this one.

In this example every time this is called, a stream is opened for something which is already there. When using this from animations, I could imagine caching it somehow.

Yup, I was thinking about that, too. I guess we can assume that once an Icon is set, it's safe for us to close the stream we obtained for the previous one.

@Lakritzator The snippet there looks dangerous. Despite the wrapping, you return a new Bitmap that will operate on a disposed object (the wrapped Bitmap). If that works, it still looks like a timebomb that rather coincidentally works... Thoughts?

AFAIK it's not, it internally really clones the first bitmap, Win32 behaviour... But we don't need this solution, let me think about it.

Yes, let's revisit this one.
Now with @punker76 having way more WPF experience than I had at that time, and even a gazillion times more experience with icons, I'm guessing we can now design a much better solution.

For now I will close this, because of adding the changes in #115. If we got issues from there then we should look again on this.


Request to update babel-runtime library to latest version

Hi Team,
Can you please update all your depedency packages to latest version. Particularly babel-runtime library. We are getting Security vulnerability error for core-js which is dependency package of babel-runtime library. Thanks.

@chandruapk latest version of babel-runtime is updated in v1.2.8

after this update, i got an error when building my project, it seem to be a babel-runtime error
✖ Errors:
resolve 'babel-runtime/helpers/extends' in 'xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>Parsed request is a module
using description file: xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>(relative path: ./dist)
Field 'browser' doesn't contain a valid alias configuration
resolve as module
xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>doesn't exist or is not a directory
xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>doesn't exist or is not a directory
xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>doesn't exist or is not a directory
looking for modules in xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>xxx/node_modules/.pnpm/react-lottie<EMAIL_ADDRESS>doesn't exist
looking for modules in xxx/node_modules/.pnpm/node_modules
xxx/node_modules/.pnpm/node_modules/babel-runtime doesn't exist
xxx/node_modules/node_modules doesn't exist or is not a directory
looking for modules in xxx/node_modules
xxx/node_modules/babel-runtime doesn't exist
/home/gitlab-runner/builds/Sfx2kxgV/1/fcq/fcbip/data-analysis/node_modules doesn't exist or is not a directory
/home/gitlab-runner/builds/Sfx2kxgV/1/fcq/fcbip/node_modules doesn't exist or is not a directory
/home/gitlab-runner/builds/Sfx2kxgV/1/fcq/node_modules doesn't exist or is not a directory
/home/gitlab-runner/builds/Sfx2kxgV/1/node_modules doesn't exist or is not a directory
/home/gitlab-runner/builds/Sfx2kxgV/node_modules doesn't exist or is not a directory
/home/gitlab-runner/builds/node_modules doesn't exist or is not a directory
/home/gitlab-runner/node_modules doesn't exist or is not a directory
/home/node_modules doesn't exist or is not a directory
/node_modules doesn't exist or is not a directory

@chandruapk @babel/runtime is removed in v1.2.9

@chenqingspring version v1.2.8 and on-wards is broken without babel-runtime

Hi  Chen Qing,
Thanks for the update. Appreciate your support.
Regards,
Chandru P
On Fri, 22 Nov 2024, 12:51 Ilyas Bakirov (aka eRIC), <
@.***> wrote:

@chenqingspring https://github.com/chenqingspring version v1.2.8 and
on-wards is broken without babel-runtime
—
Reply to this email directly, view it on GitHub
https://github.com/chenqingspring/react-lottie/issues/166#issuecomment-2493041182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BB4QUZODDUZHRCYH4SESXWL2B3LPHAVCNFSM6AAAAABRTGPYISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJTGA2DCMJYGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Play mode not starting, highlighting MixedRealityToolkit gameobject instead

Describe the bug
It sometimes happens that I want to go into play mode, which I can't enter (exits immediately) and MixedRealityToolkit gets a ping.
I dug into where that comes from and it seems that the Active Profile is null then. I have to restart Unity for this to be fixed. Since this is a SerializedField with a proper value in it, I'm puzzled how this can happen.
To reproduce
I do not have steps to reproduce it yet
Expected behavior
The Active Profile never goes null
Screenshots

Your Setup (please complete the following information)

Unity Version 2018.3.13f1
MRTK Version v2.0

Target Platform (please complete the following information)

Editor

I was able to repro this once but I'm not quite sure how I did it.

I dug into where that comes from and it seems that the Active Profile is null then.

Yeah this seems to be the immediate culprit:
if (playModeState == UnityEditor.PlayModeStateChange.ExitingEditMode && activeProfile == null)
{
    UnityEditor.EditorApplication.isPlaying = false;
    UnityEditor.Selection.activeObject = Instance;
    UnityEditor.EditorGUIUtility.PingObject(Instance);
}

I have a suspicion ResetConfiguration is getting called by a dead / dying instance of the toolkit.

I'm able to reproduce this consistently when using git to switch between branches while Unity is open.

Create a new scene
Use git to switch to / from  prerelease/2019_rc1 / mrtk_development/other major branch
Let Unity re-import assets and recompile
Add a MixedRealityToolkit instance using MixedRealityToolkit/Add to Scene and Configure
Select a configuration profile
Try to enter playmode

The bug happens regardless of which branch I'm in.
Restarting the project or triggering recompilation makes the bug stop.

Just a headsup, this issue is still relevant today.
I'm using Unity 2019.1.6f and MRTK RC1 Refresh.
I reproduced it like this:

Add scene with MRTK integration (Playspace + Toolkit)
Add new scene without MRTK integration (No camera).
Start the new scene without MRTK integration.
Go to scene with the MRTK camera implemented
Try to enter playmode

(Also at step 2, this will trigger an aggressive pop-up telling you to add the MRTK Camera, switching to any other scene using the project manager will reopen this annoying window too. "Don't show this again" does nothing.)

@Railboy is this fully fixed now? Can this be closed?


Fix typo

Description
Please explain the changes you made here.
Fixed a typo where Parcello was written as Parcel.
Checklist

[x] Code compiles correctly
[x] Created tests which fail without the change (if possible)
[x] All tests passing
[x] Extended the README/documentation, if necessary
[ ] Added myself / the copyright holder to the AUTHORS file

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master      #3   +/-   ##
======================================
  Coverage    96.8%   96.8%
======================================
  Files           7       7
  Lines         313     313
======================================
  Hits          303     303
  Misses          5       5
  Partials        5       5

Continue to review full report at Codecov.

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


added a node to scale option , default worker

/rerun all

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

etcd-perf
run_etcd_tests_fromgit.sh
PASS
00:02:33

scale-perf
run_scale_fromgit.sh
PASS
00:01:35

network-perf
smoke_test.sh
PASS
00:09:13

kube-burner
run_clusterdensity_test_fromgit.sh
PASS
00:04:45

kube-burner
run_maxnamespaces_test_fromgit.sh
PASS
00:03:32

kube-burner
run_maxservices_test_fromgit.sh
PASS
00:02:57

kube-burner
run_nodedensity_test_fromgit.sh
FAIL
00:00:04

kube-burner
run_nodedensity-heavy_test_fromgit.sh
FAIL
00:00:04

storage-perf
run_storage_tests_fromgit.sh
PASS
00:03:34

upgrade-perf
run_upgrade_fromgit.sh
PASS
00:00:21

router-perf
run_router_test.sh
FAIL
00:00:20


Add better support / warnings for wp-config.php quote variants

The scripts provided here parses out config settings like DB_NAME from wp-config.php.
See:

https://github.com/mandrasch/ddev-pull-wp-scripts/blob/main/.ddev/providers/ssh.yaml#L60
https://github.com/mandrasch/ddev-pull-wp-scripts/blob/main/.ddev/providers/backup.yaml#L35

Settings for wp-config.php can be written in 3 styles:

// single quotes
define('DB_NAME', 'my-db');

// double quotes
define("DB_NAME", "my-db");

// mixed quotes
define('DB_NAME', "my-db");

Currently only single quotes are supported, which causes problems for users, e.g. https://github.com/mandrasch/ddev-pull-wp-scripts/issues/2.
A better regex could get values from all three variants.
If anyone is a pro, I'm happy to receive PRs for this challenge 👍

Submitted a PR to support multiple quote types: https://github.com/mandrasch/ddev-pull-wp-scripts/pull/4
Tested with single, double, and mixed successfully for me.
Thanks!

Thanks very much!


Added helper functions for malloc and realloc and replaced all calls to those functions

I thought they may work better as functions than macros. Of course, it's your call.
I like this project, and I'd like to continue contributing. I'm thinking of adding doc comments, and after that, unit testing. What do you think?

Hello @awmc000 it would be better if you used the die function to exit the program like die("malloc") or die("realloc") that i wrote in ptext.c

Will do!
On Fri, Feb 2, 2024 at 11:00 AM Hoorad Farrokh @.***>
wrote:

Hello @awmc000 https://github.com/awmc000 it would be better if you
used the die function to exit the program like die("malloc") or
die("realloc") that i wrote in ptext.c
—
Reply to this email directly, view it on GitHub
https://github.com/proh14/ptext/pull/19#issuecomment-1924503080, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXTFQKPBKSG72IKNPE6ZEUDYRUZWPAVCNFSM6AAAAABCXCUNAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUGUYDGMBYGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you!


OpenSSL 1.1.1 all password bytes are ignored after first newline

Hey, I was looking around openssl code, I was trying to port decrypting of openssl encrypted files. I've run into problem when password used for encoding these files had new lines in it.
https://github.com/openssl/openssl/blob/7c69495712e3dc9aa8db38271f0c3faeb2037165/apps/apps.c#L439
Thanks to that openssl is ignoring all following password bytes after first \n occurrence.
I'm not quite sure if it is a bug or a feature, but It would be good to at least mention about it there  https://www.openssl.org/docs/man1.1.0/apps/openssl.html#Pass-Phrase-Options

It is documented for file:pathname, but that should probably be moved to a paragraph of its own that refers back to the pass phrase options where this applies (all the different file ones).

The docs should also mention that OpenSSL will not strip \r for the line and will feed it to the password reader, thus password files need to have Unix LS on any POSIX-like system.

@t8m Here my suggestion #24878 for an improved documentation that should resolve this issue.


MIC-582 /update-decision screen

MIC-582

Codecov Report
Attention: Patch coverage is 63.15789% with 7 lines in your changes missing coverage. Please review.

Project coverage is 92.33%. Comparing base (9c08837) to head (b5edc2e).

Files with missing lines
Patch %
Lines

server/routes/csip-records/controller.ts
50.00%
4 Missing :warning:

...rver/routes/journeys/update-decision/controller.ts
66.66%
1 Missing :warning:

...routes/journeys/update-investigation/controller.ts
66.66%
1 Missing :warning:

...rver/routes/journeys/update-referral/controller.ts
80.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #243      +/-   ##
==========================================
+ Coverage   92.03%   92.33%   +0.30%
==========================================
  Files         262      262
  Lines        3906     3890      -16
  Branches      612      602      -10
==========================================
- Hits         3595     3592       -3
+ Misses        310      297      -13
  Partials        1        1

Flag
Coverage Δ

92.33% <63.15%> (+0.30%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Shreejichandra

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Add one row to a binary tree with the given value at the given depth.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Issue #2563

Closing due to inactivity.


build: use lifecycle common instead of extensions

Modifications were cherry-picked from @marukami #351 pull request.


chore: move to rfc folder

This was an old RFC that ended up being placed in the wrong place. Now I moved it in the folder with the rest of the RFCs.

oops I forgot to check that in my review 🤦


Update chat message api to return cached result and add get unsent message api

connect #94

@rickmak I am going to merge this pull request


feat: more robust request recording

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[opencolorio-tools] Separate feature application to avoid circular dependency

The features of openimageio(opencolorio) and opencolorio(application) are dependent on each other.
According to the discussion, reorganize the build list, and separate opencolorio's feature applications into separate ports.
Build list should be :

install opencolorio
install openimageio[opencolorio]
install opencolorio-tools

Fix build errror when building applications.
Fixes #9851.

opencolorio's feature 'applications' --> new port opencolorio-tools
Currently dependency list:
openimageio[opencolorio]  --> Build-Depends: opencolorio
opencolorio --> Build-Depends: glew[core], freeglut[core], lcms[core], yaml-cpp[core], tinyxml[core]
opencolorio-tools  -->  Build-Depends: openimageio[opencolorio]


[OpenEXR] Update to v2.3.0

Upgrade the whole package:

[x] move to github
[x] update to v2.3.0
[x] improve the FindOpenEXR.cmake, maintaining backwards compatibility (there's an official one distributed with the package but it is completely different, for now it is avoided)
[x] remove the ilmbase port because now it is part of OpenEXR
[x] enable installation of OpenEXR tools

@cenit
Shouldn't OpenEXR be disabled for UWP? The now removed port for ilmbase didn't support UWP, and now that it is embedded into OpenEXR doesn't the same restriction apply?

if you see any failure yes sure. Maybe I didn't check all the ports with all the triplets, CI would be great to be exposed ;)
Shall I remove UWP build from OpenEXR itself alone or also in all dependent ports? (I never really appreciated the fact that sometimes it happens to build all the dependencies just to tell you "ok no way to build this port with this triplet"...)

Great. I made a few changes so that OpenEXR can be built on Linux and MacOS, for now those changes are only on my branch.
I also pulled your changes to openvdb (PR #5265) as the removal of ilmbase would cause regressions otherwise, the CI build takes a while to finish, but hopefully this time it is successful.

closed in favor of #5417


Unexpected error pops up when Python remote

Environment

Steps to Reproduce
1.Launch VS and create new python application
2.Right the default environment and install package 'debugpy'
3.Typing code in python file and set a breakpoint

4.Run this with CTRL+F5 then attach to tcp://localhost:5678 and it stops at the breakpoint
Expected behavior
No error pops up and hits the breakpoint
Actual behavior
Unexpected error pops up

Even though I restarted VS, the squiggly line under debugpy still exists
Additional context and screenshots

I can't repro this in Microsoft Visual Studio Enterprise 2022 (64-bit) - Int Preview Version 17.3.0 Preview 1.0 [32411.352.main]


[Bug]: Cannot use FlashAttention-2 backend because the vllm_flash_attn package is not found. But I have installed vllm-flash-attn.

Your current environment
Collecting environment information...
/app/apps/anaconda3/envs/vllm_040p1/lib/python3.9/site-packages/requests/__init__.py:102: RequestsDependencyWarning: urllib3 (1.26.16) or char
det (5.2.0)/charset_normalizer (2.0.4) doesn't match a supported version!
  warnings.warn("urllib3 ({}) or chardet ({})/charset_normalizer ({}) doesn't match a supported "
WARNING 08-03 23:03:20 _custom_ops.py:15] Failed to import from vllm._C with ImportError('libcudart.so.12: cannot open shared object file: No
such file or directory')
PyTorch version: 2.4.0+cu118
Is debug build: False
CUDA used to build PyTorch: 11.8
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.31

Python version: 3.9.12 (main, Apr  5 2022, 06:56:58)  [GCC 7.5.0] (64-bit runtime)
Python platform: Linux-5.4.0-148-generic-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: 11.8.89
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA A100-SXM4-80GB
Nvidia driver version: 470.161.03
cuDNN version: Probably one of the following:
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn.so.8.2.4
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn_adv_infer.so.8.2.4
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn_adv_train.so.8.2.4
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn_cnn_infer.so.8.2.4
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn_cnn_train.so.8.2.4
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn_ops_infer.so.8.2.4
/usr/local/cuda-11.4/targets/x86_64-linux/lib/libcudnn_ops_train.so.8.2.4
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Byte Order:                      Little Endian
Address sizes:                   46 bits physical, 48 bits virtual
CPU(s):                          16
On-line CPU(s) list:             0-15
Thread(s) per core:              2
Core(s) per socket:              8
Socket(s):                       1
NUMA node(s):                    1
Vendor ID:                       GenuineIntel
CPU family:                      6
Model:                           106
Model name:                      Intel(R) Xeon(R) Platinum 8369B CPU @ 2.90GHz
Stepping:                        6
CPU MHz:                         2899.998
BogoMIPS:                        5799.99
Hypervisor vendor:               KVM
Virtualization type:             full
L1d cache:                       384 KiB
L1i cache:                       256 KiB
L2 cache:                        10 MiB
L3 cache:                        48 MiB
NUMA node0 CPU(s):               0-15
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Mmio stale data:   Vulnerable: Clear CPU buffers attempted, no microcode; SMT Host state unknown
Vulnerability Retbleed:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Enhanced IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht sysca
ll nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc cpuid tsc_known_freq pni pclmulqdq monitor ssse3 fma cx16 pdcm pcid s
se4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch cpuid_fault invpcid_single ssbd ibrs ibpb stib
p ibrs_enhanced fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb avx512cd sha_
ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves wbnoinvd arat avx512vbmi avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg avx51
2_vpopcntdq rdpid arch_capabilities

Versions of relevant libraries:
[pip3] flake8==3.8.2
[pip3] flake8-bugbear==22.9.23
[pip3] flake8-comprehensions==3.10.0
[pip3] flake8-executable==2.1.2
[pip3] flake8-pyi==20.5.0
[pip3] mypy-extensions==0.4.3
[pip3] numpy==1.21.5
[pip3] numpydoc==1.2
[pip3] nvidia-nccl-cu11==2.20.5
[pip3] nvidia-nccl-cu12==2.18.1
[pip3] pytorch-crf==0.7.2
[pip3] sentence-transformers==2.2.2
[pip3] torch==2.4.0+cu118
[pip3] torchaudio==0.12.1+cu116
[pip3] torchnet==0.0.4
[pip3] torchstat==0.0.7
[pip3] torchsummary==1.5.1
[pip3] torchvision==0.18.1+cu118
[pip3] transformers==4.42.4
[pip3] transformers-stream-generator==0.0.4
[pip3] triton==3.0.0
[conda] blas                      1.0                         mkl
[conda] mkl                       2021.4.0           h06a4308_640
[conda] mkl-service               2.4.0            py39h7f8727e_0
[conda] mkl_fft                   1.3.1            py39hd3c417c_0
[conda] mkl_random                1.2.2            py39h51133e4_0
[conda] numpy                     1.21.5           py39he7a7128_1
[conda] numpy-base                1.21.5           py39hf524024_1
[conda] numpydoc                  1.2                pyhd3eb1b0_0
[conda] nvidia-nccl-cu11          2.20.5                   pypi_0    pypi
[conda] nvidia-nccl-cu12          2.18.1                   pypi_0    pypi
[conda] pytorch-crf               0.7.2                    pypi_0    pypi
[conda] sentence-transformers     2.2.2                    pypi_0    pypi
[conda] torch                     2.4.0+cu118              pypi_0    pypi
[conda] torchaudio                0.12.1+cu116             pypi_0    pypi
[conda] torchnet                  0.0.4                    pypi_0    pypi
[conda] torchstat                 0.0.7                    pypi_0    pypi
[conda] torchsummary              1.5.1                    pypi_0    pypi
[conda] torchvision               0.18.1+cu118             pypi_0    pypi
[conda] transformers              4.42.4                   pypi_0    pypi
[conda] transformers-stream-generator 0.0.4                    pypi_0    pypi
[conda] triton                    3.0.0                    pypi_0    pypi
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.3.post1
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	CPU Affinity	NUMA Affinity
GPU0	 X 	0-15		N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

🐛 Describe the bug
INFO 08-03 22:48:53 selector.py:189] Cannot use FlashAttention-2 backend because the vllm_flash_attn package is not found. `pip install vllm-flash-attn` for better performance.
INFO 08-03 22:48:53 selector.py:54] Using XFormers backend.

But I have installed vllm-flash-attn：
vllm-flash-attn    2.6.1

@xyfZzz, you can try searching for libcudart.so.12 using the command find / -name libcudart.so.12.
This may reveal a path similar to path/to/site-packages/nvidia/cuda_runtime/lib/libcudart.so.12.
Execute export LD_LIBRARY_PATH=path/to/site-packages/nvidia/cuda_runtime/lib:$LD_LIBRARY_PATH in your terminal.
Alternatively, you can add this line to your ~/.bashrc file and then runsource ~/.bashrc to apply the changes.
Try import vllm_flash_attn to verify if this work.

libcudart.so.11.0 doesnot work?
but how to use libcudart.so.12 using cuda118

@xyfZzz, you can try searching for libcudart.so.12 using the command find / -name libcudart.so.12. This may reveal a path similar to path/to/site-packages/nvidia/cuda_runtime/lib/libcudart.so.12. Execute export LD_LIBRARY_PATH=path/to/site-packages/nvidia/cuda_runtime/lib:$LD_LIBRARY_PATH in your terminal. Alternatively, you can add this line to your ~/.bashrc file and then runsource ~/.bashrc to apply the changes. Try import vllm_flash_attn to verify if this work.

My environment is cuda12. For cuda118, maybe you should take a look at https://github.com/vllm-project/vllm/issues/5232 .

It seems that you are not using the cu118 version of vllm-flash-attn.
Could you try switching to that version? @xyfZzz

@duanshengliu @ShuaibinLi @fyuan1316  Thank you very much. After I upgraded the cuda version to 12.4, this warning no longer appeared.

It seems that you are not using the cu118 version of vllm-flash-attn. Could you try switching to that version? @xyfZzz

Could you please tell me where to find vllm-flash-atten 2.6.2+cu118?


https API_URL fix

HTTP not work at this time
HTTPS work fine

Thank you! Just merged this PR in and we'll publish the new version soon


Add a new command to check migration status

Description
So far the migrate-repo command would queue up a migration and wait for it to finish. In order to make repo migrations parallel, we need to queue up all repo migrations and wait for them one by one to finish. For that we need another command to check for the migration status of all repos.
How it works
wait-for-migration command will query all migrations for an org and report the number of in-progress and queued ones, it exits when there is no more migration left in progress. It can also optionally take a migration_id as an argument, the only difference is that it will exit once the specified migration is either failed or succeeded but in each report cycle, will report the number of all in-progress or queued migrations just like the no arg variation.

if it takes a migration id, and that migration fails, it should return a non-zero exit code.

if it takes a migration id, and that migration fails, it should return a non-zero exit code.

@dylan-smith my understanding is that the no arg variation should also return a non-zero code in case of any failed migration, no?

no, I think we need a way to indicate that a migration failed. if there's no migrationid arg then it's irrelevant, but if we pass an id and it fails, we should return an exit code IMO


App refuses to display output when there are warnings from stderr

I really love this new jbofihe port but here's an issue that I've spotted, hope it gets fixed.
Sometimes jbofihe reports warnings in stderr and still gives output, e.g. when a sentence contains only a sumti:
ma gleki .i la vrude
            ^

Warning: Sentence may be missing selbri at line 1 column 13?

The current jboski seems to return an error as long as there is anything in stderr.

It would be nice if the output could still be shown when there are warnings.

@rlpowell @lagleki please fix this the website has been broken for months

I wouldn't call that "broken", like, at all, since all normal sentences work, but I'll take a look.

I wouldn't call that "broken", like, at all, since all normal sentences work, but I'll take a look.

No, some normal sentences do not work. Answers to ma-questions, for example, normally do not contain a selbri, and do not work. Sentences with jai used for abstraction also do not work and are also normal Lojban.

Yeah, that's still not what "the website has been broken for months" means.

Functionality it used to have has been gone for months. I'd call that broken.

OK, well, as someone who professionally maintains websites for a living, "broken" is "the page doesn't load", and it offends my professional sensibilities that you'd describe this problem that way, implying that I've been letting the site be totally non-functional for months on end.
But if you'd prefer I not investigate, by all means, keep arguing.

OK, well, as someone who professionally maintains websites for a living, "broken" is "the page doesn't load", and it offends my professional sensibilities that you'd describe this problem that way, implying that I've been letting the site be totally non-functional for months on end.
But if you'd prefer I not investigate, by all means, keep arguing.

I've closed my other issue about this. While the page has loaded so broken isn't the right word, you have allowed grammatical Lojban to fail for months. Looking at the python code involved in generating the output, it looks like it'd be a simple change of one or two lines but I don't know how to test changes myself.

@samflir As a volunteer who first became aware of this issue within the last two hours, I find your continued insistence that I suck to be profoundly rude.  I considered simply not fixing this issue just because of that, but that's not fair to the rest of the community.  Please try not to treat volunteers in this way in the future.
@DGCK81LNN You're quite correct, it was unconditionally failing when anything was in stderr.
Fixed in https://github.com/lojban/jboski/commit/55d39080b86662f4c9e467d9303d31197f64f70f , and also in production.

sorry I got impatient and thank you for the fix

Thanks for the fix.


Feat/helmet

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


Config for 11GB GPU

Hi. This is one of my most anticipated project recently and thanks for finally opensourcing it. It seems like the current training config is made  for 16GB GPUs so I encountered OOM on 11GB ones. Could you please suggest a config that is compatible for smaller GPUs (11GB in my case)? Thanks a lot!

FYI, I am using a 1024 image database as the default FastGAN also used 1024.

You have three options:

train on smaller resolution (duh)
set e.g. --batch-gpu=2 or the smallest value that still works for you
reduce the size of the FastGAN generator, e.g., set ngf=64

reduce the size of the FastGAN generator, e.g., set ngf=64

Is it also possible to reduce the number of feature maps of the discriminator?

you could, but it is not the limiting factor at all. The actual trainable parameters of the discriminator are less than 1M.

you could, but it is not the limiting factor at all. The actual trainable parameters of the discriminator are less than 1M.

Yes, I know. I am actually looking at reducing the discriminator in order to match a much weaker generator, not in order to reduce the memory usage. Just piggy-backing of this issue.

What happens if you use the PG discriminator with your weaker Generator? Does the discriminator overfit quickly?

@justanhduc do you still run into memory problems? Actually, the default configuration should work just fine with an 11GB GPU. This is the first tick with the default config:
tick 0     kimg 0.1      time 55s          sec/tick 6.3     sec/kimg 98.19   maintenance 48.5   cpumem 3.55   gpumem 6.33   reserved 9.86

So, the needed  GPU memory is below 11 GB.

@xl-sr in many instances, the discriminator would quickly get a loss of around 0 with the loss of G constantly growing or capping at a high value. Periodically G would briefly outsmart D in which D receives a very high loss for a single iteration until it goes back to 0. Reducing the feature maps by 2 seemed to help, but even better was removing the inner-most projection instead.
I am trying to train an image-to-image GAN so the use-case is not very typical. These generators seem to be much weaker when conditioned on images instead of noise that is sampled or injected akin to the StyleGAN Generator.

Cool, I want to see how well it works in Im2Im and what the challenges are.
The next thing you should try is setting D_kwargs.backbone_kwargs.patch = True. This might work better you.

@zimonitrome I am going to close this since it is about memory size originally. Please keep me updated on your progress, via mail or a new issue, I am also interested in the results :)


dictdiffer.merge.Merger no longer supports ignore=

When I run the tests with the latest dictdiffer, there are lots of
TypeError: Merger.__init__() got an unexpected keyword argument 'ignore'
https://github.com/inveniosoftware/dictdiffer/blob/master/dictdiffer/merge.py#L38 doesnt support ignore=.

Ah, that is implemented in https://github.com/mafrosis/dictdiffer/tree/merge-ignore


v2 - Error on update, if a default value gets retrieved from the database (but not on insert!)

Hello!
I run into another strange issue.
In my model, I have a field with a default value, which gets its value from the database (via a function).
It looks like:
let Item = Class.create({
  name: 'Item',
  collection: Collections.Items,
  fields: {
    title: {
      type: String,
      default: function() {
        return Settings.get('itemDefaultTitle');
      }
    }
  }
});

It works perfectly. The value gets displayed in the UI when I create a new item and saved to the database.
But, if I update the given item, I get an infinity loop, starting with a ServerError 500.
In the console, I get:
Exception while invoking method '/Astronomy/update' Error: Can't call yield in a noYieldsAllowed block!
What is causing the issue? Why does the default values getting called on update?
Why is it working perfectly on insert, but not on update?
Did I do something wrong or do we have a bug here?
Cheers,
Dom

Without reproduction it's hard to tell

Yes, it is hard to describe. I took your Astronomy v2 test and modified it to reproduce the error.
This is not the way, how I do it in my App (for example, I use React).
You can find the repo here: https://github.com/peXed/astronomy-test
Simply add a new user (this works). Then edit the user and you will get the error-loop.

To be honest I have no idea why it's not working. I'm gonna have to ask in the Meteor GitHub repository.

Thanks for your help @jagi
It is indeed a really strange issue and the callstack you get is not really helpful.

@peXed I've asked on Meteor GitHub. I just want to know what these errors mean. In deed stack trace is not helpful in this situation. I've tried to prepare reproduction that would be very similar to what happens in Astronomy but without effect. We will see what MDG will say about that

I still haven't managed to resolve this issue, so I will release 2.0 with this bug. However I'm gonna fix it in 2.0.1. I've said that I will release 2.0 before the end of this month and I will do so, later tonight :)

Any news on this bug? Can I help you somehow, to resolve it?

@peXed I think that I've resolved a problem however I don't know if it's gonna work in all cases so I would like you to test my fix. Please clone the v2 branch of Astronomy GitHub repository to your local packages directory and try running your code.

Hello! Your fix works like a charm!
Thank you very much.


get appointment startdate always give "JAN" month

@w9jds @CarlosMendonca @saldana  @JimDaly @msftgits
i solve this issue just add below code in Utils.java
SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss'Z'");                     simpleDateFormat.setTimeZone(TimeZone.getTimeZone("GMT"));
Instead of
SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyy-MM-DD'T'HH:mm:ss'Z'");

This has already been addressed inside of the open pull request. A sizable update is coming to the SDK and this fix will be included.

till get wrong time. how to solve this issue?
date issue solve by above code but time got wrong.
@w9jds

@patelanil2007 you need to use:
SimpleDateFormat simpleDateFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss");


[version update] Updated versions of Python Modules #55

Updated requirements.txt so that it uses the latest versions of python modules. Also, added the badge of django-ipam from requires.io. I'll squash the commits if everything is fine.
GCI task: https://codein.withgoogle.com/dashboard/task-instances/6728753267867648/
Closes #55

Been constantly trying to fix that travis build... I'll work on it later today.

Pull Request Test Coverage Report for Build 152

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 144:
0.0%

Covered Lines:
453

Relevant Lines:
453

💛  - Coveralls

Update: I don't know if I did it right, but it works!

PS: do also what @atb00ker wrote about the commit message please, remember to use git commit --amend

Oh okay, I'll do that better from now on.


Add drinking water data

including 1) measure of problem eg # violations, 2) measure of systems, eg count and/or consolidation, and 3) measure of source, eg what % relies on GW. Consider option to aggregate CWS data by GSP area, using Vanderbilt CWS shapefiles (https://lab.vanderbilt.edu/dwjlab/research/united-states-community-water-system-cws-estimated-service-area-boundary-esab-geospatial-database/) and GSP_submitted shapefiles (on Box under data_spatial_raw or downloadable at https://sgma.water.ca.gov/webgis/?jsonfile=https%3A%2F%2Fsgma.water.ca.gov%2Fportal%2Fresources%2Fjs%2Fmapconfigs%2FGspSubmittalsConfig.js&_dc=0.23941161367525954)

ok I made a script that grabs drinking water data  -- the end just creates a summary dataframe but doesn't save it or merge it, since I didn't want to overbuild something that didnt' fit into your current workflow. this can be played with a bit, right now it creates a count of PWS by GSA, health violation count (2014-2018) by GSA, and proportion of service connections reliant on groundwater by GSA. all adjustable. PWS boundary from state, other two files are downloaded to temp directories, not saved.


Add GPTProtocol (full)

Closes L2B-6387
not sure about EOA warn here since main contract is only upgradeable by polygon, but there exist plenty of routes to exploit as LocalAdmin

that vincenzos :D


Changing vss-api-netcore version to 0.5.103-private

Changing vss-api-netcore version to 0.5.103-private. This package has the dot file fix for Download Pipeline Artifact task.

@owenhuynMSFT @jahsu-MSFT  - Could you please review.

The code changes for bumping the version looks fine. As long as the CI tests pass. I'm ok with the changes.


added my  repo file

I have successfully registered for web3-conf. @aslams2020 please merge the PR now with suitable labels.

@Snehsk  please merge my PR.


BoundaryManager should be disabled on HoloLens

Overview
This code will fail when called on HoloLens. End result is the floor quad gets instantiated at (0,0,0) which is sub-optimal on a non-opaque device.
https://github.com/Microsoft/MixedRealityToolkit-Unity/blob/f2596ee39d740f2eab1209c869ea1e55d0478a49/Assets/HoloToolkit/Boundary/Scripts/BoundaryManager.cs#L87
Expected Behavior
When on HoloLens, boundary manager is disabled
Actual Behavior
When on HoloLens, boundary manager draws a floor quad at (0,0,0)
Steps to reproduce
add MixedRealityCameraParent to a new unity project and run on HoloLens.
Unity Editor Version
2017.2.0p2-MRTP5
Mixed Reality Toolkit Release Version
branch v2<IP_ADDRESS>-Stabilization
SHA f2596ee39d740f2eab1209c869ea1e55d0478a49

Updated issue as this occurs on both RS1 and RS4 based HoloLens

Looks like GitHub didn't automatically close these issues when merged into master. I'll close these manually and look into why.


Add Robomongo

Nice, can you add autoupdate information to it? See Autoupdate

I don't think it's possible, each build has a random number at the end of the name.
I.e.: https://download.robomongo.org/$version/windows/robomongo-$version-windows-x86_64-$random.zip
E.g.: https://download.robomongo.org/0.9.0/windows/robomongo-0.9.0-windows-x86_64-0786489.zip

You can use capture groups: robomongo-(?<version>[\d\w.-]+)-windows-x86_64-(?<random>[\d\w]+).zip
Now you have $version and $matchRandom available as variables. They work in url and extract_dir. (e.g. haxe.json)

I got the autoupdate to work, but it is detecting the first link in the page, which is a problem because that one is sometimes behind a paywall.

Take a look at https://robomongo.org/download

This one should do it: list-block--available.*?robomongo-(?<version>[\d\w.-]+)-windows-x86_64-(?<random>[\d\w]+).zip
https://regex101.com/r/XxYA2N/1

Done, thanks for the regex!

@r15ch13 What a great regex tool! I'll have to bookmark it


adding CI support

simple config for travisci.
you can see the build records at https://travis-ci.org/mwotton/HLearn/ now.

Hi Mike - any movement on this? Happy to put some work into getting a consistent CI system set up, but only if you're at a stage where it would actually be useful.

Hi Mark.  Thanks for following up!
Yes, I'm still interested in this.  But I don't think it quite makes sense for this repo yet.  There's a lot of changes still underway over the next month or two that are likely to entail some major restructuring still.
The subhask repo is probably at a place though where it would make sense.  I'd be interested in gettng the CI support integrated into the ghc7.10 branch.

I've finally gotten around to getting Travis working.  But it turns out they don't yet support GHC 7.10, so we can't use it.

Hvr has a method for doing it that lets you choose 7.10 - I'll have a look
in a week, travelling atm.
On Thu, 18 Jun 2015 5:16 am Mike Izbicki<EMAIL_ADDRESS>wrote:

I've finally gotten around to getting Travis working. But it turns out
they don't yet support GHC 7.10, so we can't use it.
—
Reply to this email directly or view it on GitHub
https://github.com/mikeizbicki/HLearn/pull/45#issuecomment-112967914.

I think I've figured it out.  I should be pushing out the changes in a bit.

I've finally got Travis working on the subhask repo.  (See https://travis-ci.org/mikeizbicki/subhask)  But there's two things I haven't figured out to my satisfaction:

GHC 7.10 requires llvm 3.5, which isn't available on Travis. My current solution is to download the source and compile it.  This isn't ideal because of the huge build time overhead.  But at least it works.

One of the errors I'm most scared of is performance regressions.  I'm not sure the best way to catch these errors.  Criterion gives us a good way to measure performance by hand, but not a good way to automate testing to see if performance has worsened.  On my local machine, what I've been doing is outputting all the criterion benchmarks to a csv file, then checking if any value in the csv file has changed by more than 20%.  I'm not sure how to do something similar on Travis, however, because I don't know what their architecture is like.

Once I've figured out if there's solutions to these problems I'll add travis to hlearn as well.

I think you'd want to use travis artifacts
http://blog.travis-ci.com/2012-12-18-travis-artifacts/
mark
On Mon, Jun 22, 2015 at 6:15 PM Mike Izbicki<EMAIL_ADDRESS>wrote:

I've finally got Travis working on the subhask repo. (See
https://travis-ci.org/mikeizbicki/subhask) But there's two things I
haven't figured out to my satisfaction:

GHC 7.10 requires llvm 3.5, which isn't available on Travis. My
current solution is to download the source and compile it. This isn't ideal
because of the huge build time overhead. But at least it works.
2.
One of the errors I'm most scared of is performance regressions. I'm
not sure the best way to catch these errors. Criterion gives us a good way
to measure performance by hand, but not a good way to automate testing to
see if performance has worsened. On my local machine, what I've been doing
is outputting all the criterion benchmarks to a csv file, then checking if
any value in the csv file has changed by more than 20%. I'm not sure how to
do something similar on Travis, however, because I don't know what their
architecture is like.
—
Reply to this email directly or view it on GitHub
https://github.com/mikeizbicki/HLearn/pull/45#issuecomment-114299426.

Criterion gives us a good way to measure performance by hand, but not a good way to automate testing to see if performance has worsened.

Assuming you're most worried mostly about gross degradations - O(n) -> O(n^2) say - you can always wrap the benchmark in a test and log an error on unacceptable performance.  One trick is to measure performance on n=100 and n=1000 say, then have the test guess the order.

I'm really interested inconstant factor performance.  I want to catch the scenario where a function that used to take 300ns goes up to 350ns on the same input.

I've completely redone the testing interface, so it's much improved.  Thanks for the push in this direction :)
Unfortunately, Travis builds sometimes fail due to GHC using too much memory.  I think the best way to get around this is to disable optimizations on the travis build, but I haven't figured out how to do this.  So basically everything is working, but I can't but the build-passing button on the README file yet due to this problem.

Great to hear it, looking forward to having the chance to play with HLearn a bit more seriously:)

(would cabal configure --disable-optimization be what you're looking for?)

I thought so too, but it doesn't seem to actually do anything for me.

Is this PR still relevant? It seems that master has a .travis.yml file already


feat: disable PurgeCSS by passing 'off' to options object

Fixes https://github.com/Halo-Lab/eleventy-plugin-styles/issues/4
For now I added support for disabling PurgeCSS only as it's the only thing I had personal interest in having. Disabling other plugins can be implemented in a similar fashion but I'm not sure if anyone else needs that. :)

Thank you for your contribution! Can you duplicate this PR to the new repo? Sorry for bothering you.
It is the great idea to add ability to disable plugins.


Clarify applicant requirement

This PR updates the applicant requirements now that it has been decided only project owners can submit applications.
This has been previously changed in the Getting Started section with this PR: https://github.com/badging/badging/pull/45

@germonprez @ElizabethN Could one of you please review? Thanks!🙏

@kaxada - could you merge this in?


wacs.exe doesn't work when I run it from CMD or PowerShell

Hi,
Whenever I execute the command ./wacs.exe or wacs.exe from PowerShell or CMD, it says the following:

Error while creating logger: The source was not found, but some or all event logs could not be searched.  Inaccessible logs: Security. -    at System.Diagnostics.EventLog.FindSourceRegistration(String source, String machineName, Boolean readOnly, Boolean wantToCreate)
at System.Diagnostics.EventLog.SourceExists(String source, String machineName, Boolean wantToCreate)
at System.Diagnostics.EventLog.SourceExists(String source, String machineName)
at Serilog.Sinks.EventLog.EventLogSink.ConfigureSource(EventLog log, String source)
at Serilog.Sinks.EventLog.EventLogSink..ctor(String source, String logName, ITextFormatter textFormatter, String machineName, Boolean manageEventSource, IEventIdProvider eventIdProvider)
at Serilog.Sinks.EventLog.EventLogSink..ctor(String source, String logName, ITextFormatter textFormatter, String machineName, Boolean manageEventSource)
at Serilog.LoggerConfigurationEventLogExtensions.EventLog(LoggerSinkConfiguration loggerConfiguration, String source, String logName, String machineName, Boolean manageEventSource, String outputTemplate, IFormatProvider formatProvider, LogEventLevel restrictedToMinimumLevel, IEventIdProvider eventIdProvider)
at PKISharp.WACS.Services.LogService..ctor()
I don't know what I've done wrong and I personally think it's a bug.

Have you tried running it as Administrator? It looks like a rights issue related to the event viewer.

The next version contains better handling for this scenario so that we won't crash anymore.


Fix lint issues

:scroll: Description
Lint will throw if there are warnings
#skip-changelog
:bulb: Motivation and Context

Line Length Violation: Line should be 120 characters or less: currently 124 characters (line_length)

:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report
Base: 90.45% // Head: 90.40% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (ac3b64a) compared to base (9c5aec6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1067      +/-   ##
==========================================
- Coverage   90.45%   90.40%   -0.06%
==========================================
  Files         114      105       -9
  Lines        3584     3387     -197
==========================================
- Hits         3242     3062     -180
+ Misses        342      325      -17

Impacted Files
Coverage Δ

dio/lib/src/sentry_transformer.dart

dio/lib/src/dio_event_processor.dart

logging/lib/src/logging_integration.dart

dio/lib/src/tracing_client_adapter.dart

dio/lib/src/breadcrumb_client_adapter.dart

logging/lib/src/extension.dart

dio/lib/src/sentry_dio_client_adapter.dart

dio/lib/src/failed_request_interceptor.dart

dio/lib/src/sentry_dio_extension.dart

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add github action to publish snapshots to maven

Description
Add github action to publish snapshots to maven
Issues Resolved
https://github.com/opensearch-project/cross-cluster-replication/issues/710
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #711 (0bb4576) into main (cafc069) will decrease coverage by 0.13%.
The diff coverage is 50.00%.

:exclamation: Current head 0bb4576 differs from pull request most recent head 1f88409. Consider uploading reports for the commit 1f88409 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #711      +/-   ##
============================================
- Coverage     73.34%   73.21%   -0.13%
+ Complexity      997      996       -1
============================================
  Files           141      141
  Lines          4637     4637
  Branches        519      519
============================================
- Hits           3401     3395       -6
- Misses          911      916       +5
- Partials        325      326       +1

Impacted Files
Coverage Δ

...h/replication/task/index/IndexReplicationParams.kt
77.77% <0.00%> (ø)

...h/replication/task/shard/ShardReplicationParams.kt
62.22% <100.00%> (ø)

...replication/task/shard/ShardReplicationExecutor.kt
71.42% <0.00%> (-3.58%)
:arrow_down:

.../action/status/TransportReplicationStatusAction.kt
96.73% <0.00%> (-1.09%)
:arrow_down:

...rch/replication/task/index/IndexReplicationTask.kt
67.86% <0.00%> (-0.85%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

BWC and security test failures are known issues on main

Build started failing due to a breaking change in OS: https://github.com/opensearch-project/cross-cluster-replication/pull/718/files
WIll raise a backport PR manually in 2.x for now.


Custom Fields not available

It would be nice if the provider make available the custom fields from the secrets.
Maybe iterating over the different types of the custom fields as there is no method to get all of them, and then joining those into a custom field as the record from core return.
Do you accept MR? we would be happy to help with this provider.

While I'm here, it is possible to modify the notes field using Terraform. The notes are available using the python SDK as well so you can add things you might want to the notes field e.g. as a yaml file and then read it back using python. I don't know if the other SDKs allow this but the notes are not available from the ksm command line utility which makes me sad.


Editor bug with ordered and unordered lists

Just a small glitch on editor, with both ordered and unordered lists.
Step to reproduce :

Enter a line of text, hit enter
Click on unordered list bullets (a bullet will appear)
Enter some text, hit enter

You'll end up with the following capture :

The line is marked as an unordered list line but no bullet is shown. If you click on unorderd list bullet it will remove the bullet from previous line.
There is definitively something strange in this behaviour...

Thanks! Yes, I know about this bug.
It will be fixed in the next version.


[FLINK-26612] Documentation build via GitHub actions

Based on zentol's work in https://github.com/apache/flink/pull/19103/commits/54678bbe907d6f0dc70da226b0209e38fa06c59e.

Merging to be able to continue work on this, as far as I understand rsynch needs to run from the repo.


ProcessHacker sometimes doesn't start hidden during log on

Brief description of your issue
ProcessHacker sometimes doesn't start hidden during log on.
Enabled Options: Start hidden, Enable start as admin, Start when I log on.
At first I thought that the Windows was running ProcessHacker twice during logon causing it to appear. I disabled the option "Allow only one instance" so that it will have 2 instances when it run twice and so when the ProcessHacker window appeared during log on, there is only one instance of ProcessHacker. Therefore, the Windows didn't run the ProcessHacker twice during log on.
I also tried to manually create a task in Task Scheduler to run ProcessHacker at log on with high privileges. Using -hide argument to hide with the options "Enable start as admin", "Start hidden", and "Start when I log on" are disabled. The result still the same, sometimes it's hidden and sometimes it's not.
I'm using HDD, so read-write access is slow compared to SSD.
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
Process Hacker x64 Nightly Build: 3.0.4661
Windows 10 Version 10.0.19044.1586

Do you have this option enabled in Windows settings > Accounts > Sign-in options > Scroll down to Privacy?

Do you have this option enabled in Windows settings > Accounts > Sign-in options > Scroll down to Privacy?

Yes, it is enabled. And I'm using local account.

For me, Process Hacker 3.0.4861 doesn't even load on startup, even though I don't have it selected to start hidden, and it has not loaded on startup for a long time.  Maybe it's a registry value that needs to be fixed in my case.

@dmex, Do you mean to uncheck the box, then restart, then check the box, then restart?  Or uncheck the box, restart Process Hacker, then check the box?  or just as you stated: uncheck then recheck the checkbox?  I've tried unchecking and rechecking and it does not fix the problem.

uncheck the box, restart Process Hacker, then check the box?

this

uncheck the box, restart Process Hacker, then check the box?

this

I did this and Process Hacker doesn't start on log on, still.  Could you help me to get it to work?

@dmex It's okay, it now loads on log on.


feat(mongodb): support v5

Which problem is this PR solving?

@opentelemetry/instrumentation-mongodb: Handle version 5 of MongoDB NodeJS Driver

Short description of the changes

feat(mongodb): Extend range of versions allowed for v4 patches to include v5
test(mongodb) Add test cases for v5

Duplicate of v4 with just callbacks removed (as it has been removed in v5)

chore(mongodb): Add command and configuration for v5 tests (package.json and .tav.yml)
chore(mongodb): Renamed package.json command test-new-versions to test-v4 (and adapted .tav.yml for it)

Notes

Unit tests of v4 are apparently running fine on MongoDB v5 when callbacks are adapted to Promises
Tested quickly on an app with jaeger, otel collector and signoz, it seems to work fine too
I'm not sure if there's any breaking change impacting the tracing in the V5. From what I understand, it's not the case

Feel free to comment/adapt/suggest things. I'm still learning on this kind of stuff 😉

Actually we tested a bit more today and noticed that the details of the queries are not present (e.g. the find parameters). I'll try to look at it soon.

I'm waiting for it.

For what I said before:

Actually we tested a bit more today and noticed that the details of the queries are not present (e.g. the find parameters). I'll try to look at it soon.

Nevermind, I thought it automatically included all the parameters of find(), but the enhancedDatabaseReporting option is actually used for that. By default, only the first level is included, like before.
I think the PR can be reviewed.

@osherv any update on this? v5 is working similar to v4 except for dropping callback support

I'm not an approver, but this looks good to me (there are currently merge conflicts though). We'd love to see this merged.

I quickly checked but it seems there are some tests failing now after fixing the conflicts.
I'll likely be able to fix these in the coming days.
Once done, I'll ping another maintainer as the (randomly?) assigned one doesn't seem to be active for the moment.

I've adapted the code based on the changes on main. The unit tests are all passing locally.
I had to use the same database name for all the tests, as right now the instrumentation only allows one poolName (and not one per connection). That's likely something to fix in another issue/PR.
@haddasbronfman Hello, sorry to ping you like that. I see you checked/worked on another PR for MongoDB. Would it possible to review this one when you have some time ? Thanks !

@legendecas or @dyladan could we get a review here?

@pichlermarc Thanks for the review and for maintaining the branch up-to-date with master 🙂
I've updated the PR based on your comments

Looks like there's a problem with our workflows - we'll have to take care of #1541 first to make CI pass. Sorry for the delay. :disappointed:


[Feature] Notification delivery throttling timeout

Introduction
Allow users to specify a timeout for notification deliveries that are hold off by some notification_trigger counting strategy.
Changes

Refactor code that gets generated by using BoomNotifier module by just defining wrapper functions of the notify_error related logic (now in BoomNotifier.Api) with a specific configuration.
Refactor code so that notification triggering is decided in the NotificationSender (ex NotifierSenderServer) GenServer. So we can handle the timeout in the same process.
Change tests that used self() to send messages back and forth test examples to named process.
Save the error info hash key in the ErrorInfo struct
Unify Faker Mailers

Great work 🙌🙌🙌

I'm gonna let you @germanbottosur do the approval since you've been working more closely on this 💖


What PATENTS mean?

What PATENTS mean? link: https://github.com/flutter/flutter/blob/master/PATENTS

Hi erlangp,
It means Google is letting you use any patents it might have related to flutter, when you use flutter.
If you sue us for patent infringement over flutter, we can stop letting you use flutter.
If you need more help understanding the language, please contact a local lawyer.
Best,
Max for Google OSPO

is Flutter BSD license? @sillsm

The Flutter framework (the Dart code) is licensed under a BSD variant.
The Flutter engine uses open source code with a variety of licenses, you can see the full details here:
https://github.com/flutter/engine/tree/master/travis/licenses_golden
I recommend contacting a local lawyer if you need advice on this matter. Thanks.


Fix link formatting in DOCX files

The hyperlinks in DOCX downloads of funder templates are all wonky with extra long underlining (see screenshot "Datacite organisation"). @briri suggested swapping the current gem for the one we use in DMPTool.

Duplicate of #592 , closing older issue

@briri PR submitted upstream as karnov/htmltoword#70 . Until they merge it in, we can put this in our gemfile to fix the situation:
gem 'htmltoword', :git => 'https://github.com/vyruss/htmltoword'

links still appear with white space in DOCX templates and plans

@stephaniesimms I updated the gemfile as @vyruss mentions above on roadmap-stg. Can you see if the issue is resolved? We will need to wait until the owner of the htmltoword gem accepts the fix before releasing this, but we can at least verify that its working as expected.

now things are looking good. links are formatted correctly and are functional. should i leave this issue open until the gem owner accepts the fix?

Looks good. No funny underscores around sides, but doesn't format as a URL either (i.e. blue)
@vyruss can you confirm re: the gem?

ready to test on roadmap stage when the PR comes through @vyruss @briri

@stephaniesimms @briri I can confirm that this has gone through, I just tested it and it works. The current Gemfile does not reference my fork therefore a bundle update htmltoword will just bring in htmltoword 1.0.0 which includes my fix.

links look lovely now. closing issue.


Invalid integer: ~fansub-patmud

I just tried to open an old notification.
App: Groups
Version: 5.2.0
Vere Version: undefined
Hash: 0v1j.a2lpu.el328.2oqjo.42k60.ki7t9.ij7l8.4sous.do23p.lagcp.pdek5
Synced From: ~sogryp-dister-dozzod-dozzod
Location: /apps/groups/groups/~nibset-napwyn/tlon/channels/chat/~nibset-napwyn/intros/message/~fansub-patmud/<IP_ADDRESS>6.479.821.389.625.399.695.201.884.176
Error: Invalid integer: ~fansub-patmud
  at Ce (https://fansub-patmud.tlon.network/apps/groups/assets/index-d551d3a3.js:1:24427)
  at ee (https://fansub-patmud.tlon.network/apps/groups/assets/index-d551d3a3.js:1:24832)
  at d (https://fansub-patmud.tlon.network/apps/groups/assets/index-d551d3a3.js:1:3483)
  at Jg (https://fansub-patmud.tlon.network/apps/groups/assets/index-d551d3a3.js:140:122546)
  at Mu (https://fansub-patmud.tlon.network/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:30:19470)
  at Bp (https://fansub-patmud.tlon.network/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:43828)
  at Tp (https://fansub-patmud.tlon.network/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:39610)
  at gh (https://fansub-patmud.tlon.network/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:39538)
  at So (https://fansub-patmud.tlon.network/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:39393)
  at ql (https://fansub-patmud.tlon.network/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:35786)

Recently encountered this same issue:
App: Groups
Version: 5.2.0
Vere Version: undefined
Hash: 0vb8p5e.u3nck.fm1uh.j6hhl.h2rcs.2fmlp.9iqh5.s08pn.q9oei.35pea
Synced From: ~sogryp-dister-dozzod-dozzod
Location: /apps/groups/groups/~hiddev-dannut/new-hooniverse/channels/chat/~hiddev-dannut/hooniversity--scheduled-classes-5052/message/170141184506570472258907461185014071296
Ce@https://urbit.youfoundron.com/apps/groups/assets/index-d551d3a3.js:1:23998
ee@https://urbit.youfoundron.com/apps/groups/assets/index-d551d3a3.js:1:24832
d@https://urbit.youfoundron.com/apps/groups/assets/index-d551d3a3.js:1:3483
Jg@https://urbit.youfoundron.com/apps/groups/assets/index-d551d3a3.js:140:125033
Mu@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:30:19470
Fl@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:3134
Bp@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:44601
Tp@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:39612
gh@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:39540
So@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:39393
ql@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:35786
Ac@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:36588
Jr@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:30:3258
Ae/<@https://urbit.youfoundron.com/apps/groups/assets/react-beautiful-dnd-5ffc626c.js:32:34123

looks like your drag-n-drop component is acting up


Improve: TypeScript annotations

Referenced

Generating types from JSDoc https://www.typescriptlang.org/docs/handbook/declaration-files/dts-from-js.html
JSDoc object types https://stackoverflow.com/a/31573441

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

@ashvardanian I made a mistake in the typings—all of the dimensionsOrConfigs object properties are required, when only dimensions needs to be required.  Do you know how to make them optional with JSDoc?
Other than that one quirk, the types work great 👍

@jlarmstrongiv, sadly not familiar with that.


wip: chore: Update sonar-java

#331
Currently, updating to any recent version of Sonar breaks our handling of NOSONAR and @SuppressWarnings. Not bothering with it at the moment, see https://github.com/SpoonLabs/sorald/issues/331#issuecomment-802678077
I will periodically update this PR to ensure that we stay somewhat up-to-date with sonar-java, but won't try to fix the issue filtering at this time.

accidentally closed

see also #331


Unfocus children in Group#clearChildren()

Group.removeActor(Actor child, boolean unfocus) gives the option to unfocus the removed child.
Group.removeActor(Actor child) unfocuses the removed child (calls removeActor(child, true)).
For some reason, Group.clearChildren() currently does not unfocus the children. This looks wrong.
This PR adds a clearChildren(boolean unfocus) method.
Now the question is: do you want clearChildren() (no arguments) to unfocus like removeActor(child) does, or do you want clearChildren() to not unfocus to avoid potentially breaking apps which may rely on the fact that clearChildren() never unfocused until now? I'm all for breaking apps, but 🤷.

After thinking about it some more, I believe not making clearChildren() (no arguments) unfocus its children is useless for me personally as I'll have to change all my clearChildren() calls to clearChildren(true), and most of the times I don't even call clearChildren() but clear().

Good find. I think clearChildren without specifying unfocus should match removeActor without specifying unfocus: it should unfocus. As @petoncle mentioned, this makes it a breaking change but the API consistency is likely worth it.
Some notes: Having removed actors retain focus can cause errors and surprises. We don't always remove focus though, because sometimes removal is temporary and keeping the focus is desired. Adding an unfocus boolean to a bunch of methods isn't great, but gives the flexibility needed and helps people consider the issue.

I wonder, should we have a Group clear(boolean unfocus)?

I wonder, should we have a Group clear(boolean unfocus)?

We might as well for consistency with the clearChildren and removeActor.

@NathanSweet I've added a Group#clear(boolean unfocus) method if you're interested. I think you'll have to re-open the PR to be able to merge it in.

No worries, I've made the change.


Make the quick draft appear above the collections banner & make the collections modal appear above everything

This PR fixes two points related to z-indices:

This PR makes the quick draft appear above the collections banner. To do so, I adjusted the z-index of the collections banner itself to z-10. I previously set to z-50, but in retrospect, that was too high for the banner. 😓  (We still want modals and slide-outs to show up on top, for example 🤞 ).

This PR makes the collections modal appear on top, by setting z-50. (Fixing the point mentioned by @chartgerink on https://github.com/libscie/ResearchEquals.com/issues/847#issuecomment-1288655909) Previously, the class name was z-999, but I think it was not a valid Tailwind class, and thus it was not applied.

(I did not separate the PRs because we identified these issues in #847. I'm open to separating if you think that's better, @chartgerink 👍 )
Fixes #847 ✅

@chartgerink Thanks!
I set the nav bar to z-10, then it was no longer covering the quick draft slide out.
But, this introduced a new problem where the nav bar is covering the manage authors slide out (z-index was not set there).

So, I ended up changing the z-index for the manage authors to z-10 as well.  (app/modules/components/ManageAuthors.tsx)

(It may be nice to refactor, and make a component for slide outs—if we plan using slide outs in other places in the future)
Would this work?

Makes sense! Agreed that we need a list/map of where the slide outs are used.
Merging this now ✅


take an editing pass at front-end content

review and edit topics used for display on the front end.
Currently topics created here:
#313
#308
Will continue to update this issue as new PRs are added.

FYI @edewit can you add your PRs to this list as you create new topics so we can track what needs a second editing pass?

Those 2 now listed come from the boosters, index.html we are going to have a process in place to merge these with the code and create PRs automatically.


Subclass distributed.Adaptive

This is still a work in progress. I still have a few pieces that were previously all in ._adapt that need to be added back in. This depends on https://github.com/dask/distributed/pull/1311

[ ] jobs with queued_active (I think move to should_scale_up)
[ ] len(s.workers) < len(self.cluster.workers)
[ ] Starting up busy number of clusters, instead of 2 * the current (from distributed.Adaptive.get_scale_up_kwargs

@mrocklin I'm not sure what kind of backwards compatibility guarantees we make here, but distributed.Adaptive takes (scheduler, cluster), while dask_drmaa.Adaptive takes (cluster, scheduler). I think we should align on distributed's signature.
https://github.com/dask/dask-drmaa/pull/35/commits/5fd79bcc9deeb004137dab84630e4658fb81f941 warns if the arguments are in the incorrect order.

Adaptive isn't used much.  I encourage a move-fast-and-break-things approach.


fix:remove redundant code

I think n=n-n/2 can be simplified as n=n/2
This PR will be imported into Gerrit with the title and first
comment (this text) used to generate the subject and body of
the Gerrit change.
Please ensure you adhere to every item in this list.
More info can be found at https://github.com/golang/go/wiki/CommitMessage

The PR title is formatted as follows: net/http: frob the quux before blarfing

The package name goes before the colon
The part after the colon uses the verb tense + phrase that completes the blank in,
"This change modifies Go to ___________"
Lowercase verb after the colon
No trailing period
Keep the title as short as possible. ideally under 76 characters or shorter

No Markdown
The first PR comment (this one) is wrapped at 76 characters, unless it's
really needed (ASCII art, table, or long link)
If there is a corresponding issue, add either Fixes #1234 or Updates #1234
(the latter if this is not a complete fix) to this comment
If referring to a repo other than golang/go you can use the
owner/repo#issue_number syntax: Fixes golang/tools#1234
We do not use Signed-off-by lines in Go. Please don't add them.
Our Gerrit server & GitHub bots enforce CLA compliance instead.
Delete these instructions once you have read and applied them

The result is not the same if n is odd. It's rounding up instead of down.
https://go.dev/play/p/sMb5HV9KuJ6


text input for Image Mixer

Dear Justin,
Thank you for your work in stable diffusion; it benefits me a lot.
Could you elaborate on how you train the 'Image Mixer' model? Why can it accept text input if the model is fine-tuned with only a CLIP image encoder? Or is the model fine-tuned with both the CLIP image and text encoder?
I appreciate your help.
Best wishes,
Zongze

+1


SDWebImageManager should have a function to just cancel a specific operation

Currently there's only one cancel function:

(void)cancelAll;
Which causes bug because it is possible to cancel all operations in the queue caused by quickly added operations(user tapped too fast), and actually should keep the last operation.
With this method, can cancel just the specific image downloading task, which can be hold as a property to get access to.

(void)cancelOperation:(SDWebImageCombinedOperation *)operation {
@synchronized (self.runningOperations) {
[operation cancel];
[self.runningOperations removeObject:operation];
}
}

There is no need for this method. If you have the instance of the operation, you can call cancel on it directly.

@bpoplauschi Thanks your reply.
I tried to call cancel, but the operation block is still actioning after cancel on certain condition: tapping very fast. App crashed because of Zombie action: action send to dealloced object.

@REWNing it will probably crash with the method you described as well. Can you take a look with Zombies and see which object is crashing and why? If you could create a PR, a number of people would benefit from that


dotnet new --uninstall not working for folder templates

Unable to uninstall folder-based templates
This could be an issue with the dotnet net --uninstall (or dotnet new -u) command, or just an issue with uninstall documentation, but it doesn't seem possible to uninstall a template via a folder template.
See also this Stack Overflow question.
General
Steps

Create template folder (.template.config folder with a template.json file)
I have confirmed it with this JSON config.
{
    "$schema": "http://json.schemastore.org/template",
    "author": "Some Author",
    "classifications": [ "Some", "Various", "Search Keywords" ],
    "identity": "Some.Identity",
    "shortName": "name-used-in-dotnetnew",
    "name": "Some Template Name"
}

Install template
dotnet new --install ./some-template-folder/

[optional] Use newly-installed template
The template "Some Template Name" was created successfully.

Attempt to uninstall template
dotnet new --uninstall ./some-template-folder/

Instead of uninstalling, I keep getting this error message.

Could not find something to uninstall called '.\some-template-folder\'.

This happened with .NET Core SDK v1.1.0 and v2.0.0.

[Migrated to dotnet/templating#1226 and closing this one, per tweets with @sayedihashimi]

dotnet new --debug:reinit


Permissions on Remote Server

Hey there!
I'm still trying to determine what permissions are needed on the remote server for using deployer?
The user I'm using is fo, and the user Nginx uses is the standard www-data. When everything deployed, checking the permissions it's fo:fo, but shouldn't it be fo:www-data? I've added the user fo to the www-data group. I just get a forbidden error when trying to access the site after deploying.
Any help is appreciated! Thanks!

Fixed. Now using setfacl.


Framework Version [Required]
com.github.getActivity:Toaster:12.6
Issue Description [Required]
Steps to Reproduce [Required]
home键后无法显示toast
Is the Issue Reproducible? [Required]
Yes
Project targetSdkVersion [Required]
30
Device Information [Required]
华为mate20 ，鸿蒙3.0系统
Android Version [Required]
华为mate20 ，鸿蒙3.0系统
Issue Source Channel [Required]
Encountered by myself
Is it specific to certain device models? [Required]
鸿蒙系统3.0
Does the latest version of the framework have this issue? [Required]
Yes
Is the issue mentioned in the framework documentation? [Required]
No
Did you consult the framework documentation but couldn't find a solution? [Required]
Yes
Has a similar issue been reported in the issue list? [Required]
No
Have you searched the issue list but couldn't find a solution? [Required]
Yes
Can the issue be reproduced with a demo project? [Required]
No
Provide Error Stack Trace
No response
Provide Screenshots or Videos
No response
Provide a Solution
No response

小伙子，你确定这个问题文档上面没有写？


why the anti aim not working?

i test tge anti aim alot of time and its diden't work
its need a update?
if some has fixed it, can tell me how please?

choke some packets

choke some packets

doesnt help.
make your own AA if it doesn't work.
i recommend installing this

i don't know how to make it.. is there any source working for osiris..

i had made a better anti-aim, ive since deleted the repo but if you can somehow download it from the pull requests on this repo then you can try it.

i did see it but u tell us its not working and u will fix it

nothing about it was broken.
I said I failed to sync the branch with master, you will be on an older version of Osiris if you choose to clone it so it's up to you to resync it.


Add trailing newline character in java home command

This PR fixes 965 bug.
Need to update config file loaded on sdk install

https://github.com/sdkman/sdkman-cli/pull/990


how to play ftp link video

my ftp video link:
ftp://<IP_ADDRESS>:8888/video/1.mp4
user: wxx
password: 1234
how to play?

_controller = VideoPlayerController.networkUrl(
Uri.parse('ftp://<IP_ADDRESS>:8888/video/1.mp4'),
closedCaptionFile: _loadCaptions(),
videoPlayerOptions: VideoPlayerOptions(mixWithOthers: true),
httpHeaders: {"ftp-user": "wxx", "ftp-pwd": "1234"}
this code cannot play video.

check logs
player =
       Player(configuration: PlayerConfiguration(logLevel: MPVLogLevel.debug));
   player.stream.log.listen((event) {
     print("log $event");
   });

and send it to have more idea of what is the problem


added hmatrix-vector-sized and hmatrix-backprop

hmatrix-vector-sized and hmatrix-backprop to stackage
Checklist:

[x] Meaningful commit message - please not Update build-constraints.yml

[x] Some time passed since Hackage upload

[x] On your own machine, in a new directory, you have successfully run the following set of commands (replace $package with the name of the package that is submitted, $version is the version of the package you want to get into Stackage):
stack unpack $package
cd $package-$version
stack init --resolver nightly
stack build --resolver nightly --haddock --test --bench --no-run-benchmarks

Double checked for typos this time :)
(https://github.com/fpco/stackage/commit/6e1695a239d6f750a0385697a3828c8a9ce65177)

Merged, thanks


Quauable Jobs should fully run through UI

I think that the quauable jobs should use a little bit of javascript to run through the job until it finishes or until it hits an error. Right now having to tell the job to run over, and over, and over again is cumbersome. As an example if I want to remove old page versions, and I have a few thousand pages on my site, i've got to kick off that job a thousand times by hand.

Nevermind I was looking at the wrong job class... Queuable jobs do seem to do exactly that.


[Monitor] Release Track2 MGMT SDK

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check for Azure.ResourceManager.Monitor
API review is created for Azure.ResourceManager.Monitor. You can review APIs here.


Pipeline fix may oom in pipeline load

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


T3 remove redundant new

In the T3 kernels, we have some functions called runTripletDefaultAlgoBBBB, and also the BBEE and EEEE regions. Those cuts are duplicating with the passPointingConstraintBBB ... functions. On the other hand, introducing beta iteration cuts to triplet objects are not physically meaningful. In this PR, we still keep the first beta cut as geometric requirement and move to passPointingConstraint function, but remove all the beta iteration functions.
Another improvement is that the triplet radius is calculated and saved for each of the T3s. In this case, we don't need to calculate the radius of triplet multiple times when building T5, pT3 and pT5s. This will also bring timing improvement.
Slides describing the influence of this PR is summarized here.
https://indico.cern.ch/event/1407246/contributions/5914814/attachments/2846057/4976175/check the T3 cuts EEEE 2.pdf

/run standalone
/run CMSSW

Merging this PR will be able to close Issue #214 and Issue #247 related to triplet building cuts

/run standalone

This is the timing summary running on cgpu-1 after the current change.

This is the timing summary on cgpu-1 of the master.

The timing reduction of T3 kernels is ~20%

This is the timing summary running on cgpu-1 after the current change.

please rerun to check that s=8 is just a fluctuation

This is the timing summary running on cgpu-1 after the current change.

please rerun to check that s=8 is just a fluctuation

Yes, I updated the previous comment by replacing the timing info

/run standalone
/run cmssw

/run standalone
/run CMSSW

Saw this warning while compiling, is this okay? Warning is maybe-uninitialized, not sure if it was discussed already.

Saw this warning while compiling, is this okay? Warning is maybe-uninitialized, not sure if it was discussed already.

Thank you for spotting this! I did a check and adds a printout just before line 203, and it can print meaningful values for the betaIn variable. And to be saved to the triplet collection, it has to pass through all the requirements, so for a good triplet the betaIn should be calculated and saved. To be safe, I can add initialization to betaIn as 0, as an extra protection for this.


use given-kotlin 0.1.0

The (binary only) given extension "jgiven-addons" is deprecated, so we should use https://github.com/toolisticon/jgiven-kotlin instead.

Turns out given-kotlin is not required to implement the testing extensions, but you can see the advantage of JGiven-kotlin in the https://github.com/holixon/axon-testing/blob/develop/examples/bankaccount-jgiven-junit5/src/test/kotlin/BankAccountAggregateJgivenKotlinTest.kt example.

Turns out given-kotlin is not required to implement the testing extensions, but you can see the advantage of JGiven-kotlin in the https://github.com/holixon/axon-testing/blob/develop/examples/bankaccount-jgiven-junit5/src/test/kotlin/BankAccountAggregateJgivenKotlinTest.kt example.


src/posix/Makefile-posix DAEMON=1 fails to build

Hello, I'm trying to build posix tools with DAEMON=1 using the latest commit on a freshly installed RPi4 and I get the following errors:
/home/pi/openthread/src/posix/../../src/posix/platform/netif.cpp:354:119: error: cast from ‘char*’ to ‘rtattr*’ increases required alignment of target type [-Werror=cast-align]
     rta           = reinterpret_cast<struct rtattr *>((reinterpret_cast<char *>(&req)) + NLMSG_ALIGN(req.nh.nlmsg_len));
                                                                                ^
/home/pi/openthread/src/posix/../../src/posix/platform/netif.cpp:366:105: error: cast from ‘char*’ to ‘rtattr*’ increases required alignment of target type [-Werror=cast-align]
         rta           = reinterpret_cast<struct rtattr *>((reinterpret_cast<char *>(rta)) + rta->rta_len);
                                                                                ^
/home/pi/openthread/src/posix/../../src/posix/platform/netif.cpp: In function ‘void processNetlinkEvent(otInstance*)’:
/home/pi/openthread/src/posix/../../src/posix/platform/netif.cpp:1041:75: error: cast from ‘char*’ to ‘nlmsghdr*’ increases required alignment of target type [-Werror=cast-align]
     for (struct nlmsghdr *msg = reinterpret_cast<struct nlmsghdr *>(buffer); NLMSG_OK(msg, static_cast<size_t>(length));
                                                                           ^
/home/pi/openthread/src/posix/../../src/posix/platform/netif.cpp: In function ‘void processMLDEvent(otInstance*)’:
/home/pi/openthread/src/posix/../../src/posix/platform/netif.cpp:1191:98: error: cast from ‘sockaddr*’ to ‘sockaddr_in6*’ increases required alignment of target type [-Werror=cast-align]
             struct sockaddr_in6 *addr6 = reinterpret_cast<struct sockaddr_in6 *>(ifAddr->ifa_addr);
                                                                                ^
  CXX      libopenthread_posix_a-virtual_time.o
/home/pi/openthread/src/posix/../../src/posix/platform/udp.cpp: In function ‘otError transmitPacket(int, uint8_t*, uint16_t, const otMessageInfo&)’:
/home/pi/openthread/src/posix/../../src/posix/platform/udp.cpp:131:86: error: cast from ‘const otIp6Address*’ to ‘const in6_addr*’ increases required alignment of target type [-Werror=cast-align]
     if (!IsMulticast(reinterpret_cast<const struct in6_addr &>(aMessageInfo.mSockAddr)) &&
                                                                                ^

Steps to reproduce:

get fresh sources
run bootstrap scripts
build using make -f src/posix/Makefile-posix DAEMON=1

Raspberry Pi OS version (fully updated): Linux raspberrypi 5.4.51-v7l+ #1333 SMP Mon Aug 10 16:51:40 BST 2020 armv7l GNU/Linux
Git commit: 690ddbf8817b62413a2444e0963c7d96496b0c98

You use -Wcast-align in configure.ac and code
rta           = reinterpret_cast<struct rtattr *>((reinterpret_cast<char *>(rta)) + rta->rta_len);
notify about cast-align warning on ARM platform.
So need rewrite code or -Wno-error=cast-align

Of course -Wno-error=cast-align remove errors.
But need correct implementation of
rta           = reinterpret_cast<struct rtattr *>((reinterpret_cast<char *>(&req)) + NLMSG_ALIGN(req.nh.nlmsg_len));
because you can have problem with alignement on ARM.
I hope G guru programmers resolve it.

@x893 I think the current implementation is a correct version. I found a similar discussion about lwIP. Throughts?

May be. But the compiler warns that the alignment can be violated during such an operation and a hardfault can occur. It would be perfectly correct to copy the bytes into the rtattr structure, but this takes more time. It is possible to change -Wcast-align (configure.ac) to -Wno-error=cast-align for compilation.

The implementation is correct, but the compiler isn't smart enough to check the alignment. Because the usage of rtnetlink is incompatible with the memcpy solution, I think the only solution is to ignore these errors. I've sent a PR to ignore cast-align at these locations.


How to create multiple passes from a single program ?

I want to create multiple .pkpass like the zip must contain 2 .pkpass files, How can I do that. Please help me out.

That is not possible to my knowledge.

Thanks again Ryan.... :)


Wu Anti-aliased lines demo, first draft.

I don't think this is ready to go into the graphics.h header file but I wanted to show what I've got so far.
I wrote four "reflections" of the basic Wu algorithm and a wrapper to call the right one.  (For me it's easier to do this than to figure out a single version that correctly handles the four cases, and then writing the single version is easier from the boilerplate.)
I embedded the line-drawing code in a simple demo.
Special-case code for horizontal, vertical and 45° degree lines is omitted for now.

I was able to test out the demo locally. Looks good! :)
I'm looking forward to being able to draw a 3D spinning cube soon hehehe.

I used your test code and realized that the implementation of Bresenham's algorithm I had included in include/graphics.h was completely broken, and found a bug in the compiler too :sweat_smile:
I just pushed a fix for comparison operators in ncc. It wasn't doing comparison of negative int values correctly. This may have affected your code. In general, need to be careful if comparing signed and unsigned values, e.g. int and u32. If you have a signed and an unsigned value, then an unsigned comparison operator will be used, which doesn't account for negative numbers.
If you want a signed comparison, you need to either use two int values, or cast to int first, e.g.
int a;
u32 b;
if (x < b) {} // uses an unsigned comparison
if (a < (int)b) {} // uses a signed comparison that accounts for negative values correctly

I was able to test out the demo locally. Looks good! :)

Ah, that's so cool!  I wanted to have something simple that obviously showed that lines in all four quadrants worked and looked okay, and that was at least a little interactive and visually interesting.  I'm not a graphics designer or demo scene writer, eh?

I'm looking forward to being able to draw a 3D spinning cube soon hehehe.

I remembered that I had uploaded my 3D demo to github years ago: https://github.com/calroc/Tkinter3D  (I don't recall whether I ever updated it to Py3.)  It's a really direct translation of the (C++?) code in "Physics For Game Developers" ( https://archive.org/details/physics-for-game-developers-2e )  https://github.com/calroc/Tkinter3D/blob/master/math3d.py
I understood it when I wrote it but now I'm a little fuzzy.  In any event, it's all there: 3D math, a simple scene graph, and the frustum transform for perspective.  Like I said, I got to the point of animating a rotating cube (dots on a Tk Canvas).  I always meant to use this (or at least what I learned from it) to implement a UI.  That's how long I've been waiting for UVM! :)  The code was old when I put it in github.
In any event you're welcome to it.  I'm probably going to spend at least a few days messing around with cheap fake 3D effects using integers and like tables of pre-computed sin values or something.  I don't actually need full 3D for what I want to do (affine transforms should cover most of it) but I too would like to see cool wireframe flythough demos and such.  :)  So if you don't get to it before me I'll eventually do it. ;)

I clicked the wrong button (I promise I won't mention it more than once a week, but I do not like GitHub's UI.)

You're welcome to it. I'm probably going to spend at least a few days messing around with cheap fake 3D effects using integers and like tables of pre-computed sin values or something. I don't actually need full 3D for what I want to do (affine transforms should cover most of it) but I too would like to see cool wireframe flythough demos and such. :) So if you don't get to it before me I'll eventually do it. ;)

I'm curious to see it if you implement it. Having fun 3D demos to show people would definitely make UVM more attractive.
For this PR, are you going to combine the 4 functions into one as you had hinted?

For this PR, are you going to combine the 4 functions into one as you had hinted?

I could, but is that desirable?  To my way of thinking it seems better to have these four fundamental "reflections" with a router or dispatcher function, because if you happen to know up front which one to use for your lines you can avoid the overhead of dispatching and just call them directly.  E.g. for a rotating cube most of the lines most of the frames will be in the same octant, one could keep the lines in separate per-octant lists (and either move them from one to the other as needed or something?)  The basic idea is to keep sorting separate from the pixel-pushing.

I'm curious to see it if you implement it. Having fun 3D demos to show people would definitely make UVM more attractive.

I'll get to it eventually if no one else does first, I'll have to at least skim that book again.

Hi @calroc ! Is this ready with all the edge cases handled and tested? :)

Hi!  Yeah, it all seems to work: horizontal, vertical, and 45° lines.  When the program starts the initial (x,y) location for the mouse results in a nice overlap between the two sets of diagonals and which shows off the anti-aliasing nicely.
It needs some editing for style and cleanup (s/carefree/unsafe/, etc.) still.  I should be able to get to that today.  (It's been raining all month here and I'm way behind on setting up my garden for Spring!)

Alright, almost done then :)


Upgrade from 0.4.5 did something wrong

Since IOS 10 update, all my Apple users complains of not being able to list and join rooms.
It's not only related to Safari browser but also with Chrome.
App accepts/rejects user's credentials then screen load in a loop.
All those users hadn't problems prior this update.

I was still using 0.4.5... I synced with master and now it's better but not good enough. Anytime a user try to join a password protected room, the server crash with this dump
events.js:154       throw er; // Unhandled 'error' event
Error: Illegal argument undefined     at module.exports (/var/local/logbook/node_modules/md5/md5.js:152:13)     at model.<anonymous> (/var/local/logbook/app/models/user.js:117:12)     at VirtualType.applyGetters (/var/local/logbook/node_modules/mongoose/lib/virtualtype.js:77:25)     at model.Document.get (/var/local/logbook/node_modules/mongoose/lib/document.js:904:18)     at applyGetters (/var/local/logbook/node_modules/mongoose/lib/document.js:2262:35)     at model.Document.$toObject (/var/local/logbook/node_modules/mongoose/lib/document.js:2044:5)     at model.Document.toObject (/var/local/logbook/node_modules/mongoose/lib/document.js:2197:15)     at clone (/var/local/logbook/node_modules/mongoose/lib/utils.js:252:16)     at cloneArray (/var/local/logbook/node_modules/mongoose/lib/utils.js:334:14)     at clone (/var/local/logbook/node_modules/mongoose/lib/utils.js:245:12)     at cloneObject (/var/local/logbook/node_modules/mongoose/lib/utils.js:315:13)     at clone (/var/local/logbook/node_modules/mongoose/lib/utils.js:258:16)     at model.Document.$toObject (/var/local/logbook/node_modules/mongoose/lib/document.js:2032:13)     at model.Document.toObject (/var/local/logbook/node_modules/mongoose/lib/document.js:2197:15)     at model.toJSON (/var/local/logbook/app/models/room.js:177:21)     at /var/local/logbook/app/controllers/rooms.js:277:31     at /var/local/logbook/app/core/rooms.js:55:13     at /var/local/logbook/app/models/room.js:163:13     at /var/local/logbook/node_modules/mongoose/lib/model.js:3340:16     at /var/local/logbook/node_modules/mongoose/lib/document.js:1944:18     at _combinedTickCallback (internal/process/next_tick.js:67:7)     at process._tickCallback (internal/process/next_tick.js:98:9)
When the service is restarted the same user can join directly, just before it crashed he was added to the participants room's list...
I did a npm update and I don't understand, please help

@mmeinert it should be fixed in master now, thank  you for bringing this up!

Thank you @hhaidar


Warnings when opening console

The one for Thoth.Json happens when the localStorage is empty? Not sure @MangelMaxime
About the "Cannot find type" ones, I think we talked about it once with @ncave but don't remember if we found the cause. What's inject.js? I don't see that one.