Cant install pycocotools

Hi i tried installing this on automatic1111, but lavis could not be installed due to failure in pycocotools installation. error is as follows.
Building wheel for pycocotools (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for pycocotools (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [23 lines of output]
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-cpython-310
creating build\lib.win-amd64-cpython-310\pycocotools
copying pycocotools\coco.py -> build\lib.win-amd64-cpython-310\pycocotools
copying pycocotools\cocoeval.py -> build\lib.win-amd64-cpython-310\pycocotools
copying pycocotools\mask.py -> build\lib.win-amd64-cpython-310\pycocotools
copying pycocotools_init_.py -> build\lib.win-amd64-cpython-310\pycocotools
running build_ext
cythoning pycocotools/_mask.pyx to pycocotools_mask.c
C:\Users\Office PC\AppData\Local\Temp\pip-build-env-33xdxvhq\overlay\Lib\site-packages\Cython\Compiler\Main.py:369: FutureWarning: Cython directive 'language_level' not set, using 2 for now (Py2). This will change in a later release! File: C:\Users\Office PC\AppData\Local\Temp\pip-install-b89_0me7\pycocotools_8a34d91df0aa4dddb120d85857272ba3\pycocotools_mask.pyx
tree = Parsing.p_module(s, pxd, full_module_name)
building 'pycocotools._mask' extension
creating build\temp.win-amd64-cpython-310
creating build\temp.win-amd64-cpython-310\Release
creating build\temp.win-amd64-cpython-310\Release\common
creating build\temp.win-amd64-cpython-310\Release\pycocotools
"C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.35.32215\bin\HostX86\x64\cl.exe" /c /nologo /O2 /W3 /GL /DNDEBUG /MD "-IC:\Users\Office PC\AppData\Local\Temp\pip-build-env-33xdxvhq\overlay\Lib\site-packages\numpy\core\include" -I./common -ID:\SD1\stable-diffusion-webui\venv\include -ID:\Users\OfficePC1\AppData\Local\Programs\Python\Python310\include -ID:\Users\OfficePC1\AppData\Local\Programs\Python\Python310\Include "-IC:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.35.32215\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.35.32215\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Auxiliary\VS\include" /Tc./common/maskApi.c /Fobuild\temp.win-amd64-cpython-310\Release./common/maskApi.obj
maskApi.c
./common/maskApi.c(8): fatal error C1083: Cannot open include file: 'math.h': No such file or directory
error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.35.32215\bin\HostX86\x64\cl.exe' failed with exit code 2
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pycocotools
Failed to build pycocotools
ERROR: Could not build wheels for pycocotools, which is required to install pyproject.toml-based projects
i installed buildtools again but still error persists.

Does this directory exist?
C:\Program Files (x86)\Windows Kits\10\Include\<YOURVIRSION>\ucrt

Nope in Windows Kits\10  folder there is no folder named include

Okay-
Plase open Visual Studio Installer >  Modify > Check Desktop development with C++ and install

Thanks i will check it out again

There shouldn't be.
If you can't, try download from https://developer.microsoft.com/ja-jp/windows/downloads/windows-sdk/

Windows 11 SDK was required. Now it has installed. Thanks.


[Feature Request] Make start_update/execute_update work with vscode/pyright and add pyright to CI

Describe the solution you'd like
Right now vscode/pylance/pyright fails with:

I think we probably just need to remove the typing for multi-param update (use Sequence[Any] like signal/query). Then we need to add https://github.com/microsoft/pyright to CI alongside MyPy if it's not too difficult.

Also note that the return type is not properly inferred from the method in Pyright like it is in MyPy

Another example:
samples-python(main) .venv/bin/pyright --pythonpath .venv/bin/python hello/hello_activity.py
/Users/dan/src/temporalio/samples-python/hello/hello_activity.py
  /Users/dan/src/temporalio/samples-python/hello/hello_activity.py:58:24 - error: No overloads for "execute_workflow" match the provided arguments (reportCallIssue)
  /Users/dan/src/temporalio/samples-python/hello/hello_activity.py:59:13 - error: Argument of type "(self: GreetingWorkflow, name: str) -> Coroutine[Any, Any, str]" cannot be assigned to parameter "workflow" of type "str" in function "execute_workflow"
    "function" is incompatible with "str" (reportArgumentType)
2 errors, 0 warnings, 0 informations

samples-python(main) .venv/bin/pyright --version
pyright 1.1.350
samples-python(main) .venv/bin/python --version
Python 3.10.13

So #471 probably fixed this error, but leaving this open because we still want pyright in CI


Test: Sandbox test cases

Signed-off-by: tarukumar<EMAIL_ADDRESS>

@tarukumar do we still need this one? could we close it?


process has died [version `GLIBC_2.33' not found]

Hello,
Thank you so much for this repo. While running roscore, i get the follow error:
/opt/ros/noetic/lib/rosout/rosout: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /home/dragon/ros-humble-ros1-bridge/install/ros1_bridge/lib/libapr-1.so.0)
[rosout-1] process has died [pid 2098, exit code 1, cmd /opt/ros/noetic/lib/rosout/rosout __name:=rosout __log:=/home/dragon/.ros/log/da3a8dd0-f89e-11ee-81b3-d4e98a419308/rosout-1.log].
log file: /home/dragon/.ros/log/da3a8dd0-f89e-11ee-81b3-d4e98a419308/rosout-1*.log

i have tried updating my system: sudo apt-get update && sudo apt-get upgrade
and also tried to install glibc:  sudo apt install glibc  but error remain same.
System: Ubuntu 22.04
Any assistance in resolving this issue would be greatly appreciated. Thank you.

hmm.  I see you are running roscore in a ROS Noetic system.   I assume this ROS Noetic system is running on a Ubuntu 20.04 or older?   Anyways, it shouldn't interfere with what you have installed on your Ubuntu 22.04.    Right?
If you have both ROS humble and ROS Noetic installed on Ubuntu 22.04.  You are going to get conflict like this.   You are better off running ROS Noetic on a Docker container or another separated machine.

hmm. I see you are running roscore in a ROS Noetic system. I assume this ROS Noetic system is running on a Ubuntu 20.04 or older? Anyways, it shouldn't interfere with what you have installed on your Ubuntu 22.04. Right?
If you have both ROS humble and ROS Noetic installed on Ubuntu 22.04. You are going to get conflict like this. You are better off running ROS Noetic on a Docker container or another separated machine.

I don't have 20.4 in my system. I'm running roscore in gui terminal (using rocker command in readme file)

OK. I see the issue..  rocker is starting a container but sharing your Ubuntu 22.04's home directory (due to the --home option).  As a result, your ~/.bashrc is executed inside the container.  And it most likely you sourced the bridge's local_setup.bash in your ~/.bashrc.   The solution is to remove the --home option from the rocker command:
rocker --x11 --user --privileged \          --volume /dev/shm /dev/shm --network=host -- osrf/ros:noetic-desktop \          'bash -c "sudo apt update; sudo apt install -y tilix; tilix"'
I have updated the README.md file.  Thank you for reporting this issue.

OK. I see the issue.. rocker is starting a container but sharing your Ubuntu 22.04's home directory (due to the --home option). As a result, your ~/.bashrc is executed inside the container. And it most likely you sourced the bridge's local_setup.bash in your ~/.bashrc. The solution is to remove the --home option from the rocker command:
rocker --x11 --user --privileged --volume /dev/shm /dev/shm --network=host -- osrf/ros:noetic-desktop 'bash -c "sudo apt update; sudo apt install -y tilix; tilix"'
I have updated the README.md file. Thank you for reporting this issue.

Thank you. it works now


the system cannot find the path specified

would be brilliant, thx
yet, I get: "the system cannot find the path specified" when writing the txt files, txt files do not get written?
x.sctemp files get written in the storage directory, yet they are empty AFAIK
Windows with python according to specs
this seems to be a ffmpeg issue (cf. https://stackoverflow.com/questions/75644012/the-system-cannot-find-the-file-specified-error-when-trying-to-execute-ffmpeg-co)
I tried running the file with admin rights too/on drive c ...
ffmpeg version 6.1-essentials_build-www.gyan.dev Copyright (c) 2000-2023 the FFmpeg developers
built with gcc 12.2.0 (Rev10, Built by MSYS2 project)
configuration: --enable-gpl --enable-version3 --enable-static --pkg-config=pkgconf --disable-w32threads --disable-autodetect --enable-fontconfig --enable-iconv --enable-gnutls --enable-libxml2 --enable-gmp --enable-bzlib --enable-lzma --enable-zlib --enable-libsrt --enable-libssh --enable-libzmq --enable-avisynth --enable-sdl2 --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-libaom --enable-libopenjpeg --enable-libvpx --enable-mediafoundation --enable-libass --enable-libfreetype --enable-libfribidi --enable-libharfbuzz --enable-libvidstab --enable-libvmaf --enable-libzimg --enable-amf --enable-cuda-llvm --enable-cuvid --enable-ffnvcodec --enable-nvdec --enable-nvenc --enable-dxva2 --enable-d3d11va --enable-libvpl --enable-libgme --enable-libopenmpt --enable-libopencore-amrwb --enable-libmp3lame --enable-libtheora --enable-libvo-amrwbenc --enable-libgsm --enable-libopencore-amrnb --enable-libopus --enable-libspeex --enable-libvorbis --enable-librubberband
libavutil      58. 29.100 / 58. 29.100
libavcodec     60. 31.102 / 60. 31.102
libavformat    60. 16.100 / 60. 16.100
libavdevice    60.  3.100 / 60.  3.100
libavfilter     9. 12.100 /  9. 12.100
libswscale      7.  5.100 /  7.  5.100
libswresample   4. 12.100 /  4. 12.100
libpostproc    57.  3.100 / 57.  3.100
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'G:\My Drive\ABSA_Tweets-master\01HAPNHB8BPAXHQ1KM28DYR3SD-1592574611\bennett#TSP048 - Tom Bennett\1.m4a':
Metadata:
major_brand     : isom
minor_version   : 512
compatible_brands: isomiso2mp41
title           : #TSP048 - Tom Bennett
artist          : The Two Shot Podcast
date            : 2018
encoder         : Lavf58.39.101
comment         : https://www.youtube.com/watch?v=qkk0CFuc3Ew
Duration: 00:48:02.99, start: 0.000000, bitrate: 126 kb/s
Stream #0:00x1: Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 125 kb/s (default)
Metadata:
handler_name    : SoundHandler
vendor_id       : [0][0][0][0]
[out#0/mp3 @ 000001f772f600c0] No explicit maps, mapping streams automatically...
[aost#0:0/libmp3lame @ 000001f7714c0300] Created audio stream from input stream 0:0
Stream mapping:
Stream #0:0 -> #0:0 (aac (native) -> mp3 (libmp3lame))
Press [q] to stop, [?] for help
[graph_0_in_0_0 @ 000001f771531540] tb:1/44100 samplefmt:fltp samplerate:44100 chlayout:stereo
Output #0, mp3, to 'Scribe\storage\1.m4a.mp3':
Metadata:
major_brand     : isom
minor_version   : 512
compatible_brands: isomiso2mp41
TIT2            : #TSP048 - Tom Bennett
TPE1            : The Two Shot Podcast
TDRC            : 2018
comment         : https://www.youtube.com/watch?v=qkk0CFuc3Ew
TSSE            : Lavf60.16.100
Stream #0:0(und): Audio: mp3, 44100 Hz, stereo, fltp, delay 1105, 128 kb/s (default)
Metadata:
handler_name    : SoundHandler
vendor_id       : [0][0][0][0]
encoder         : Lavc60.31.102 libmp3lame
[in#0/mov,mp4,m4a,3gp,3g2,mj2 @ 000001f7714bb2c0] EOF while reading input
[in#0/mov,mp4,m4a,3gp,3g2,mj2 @ 000001f7714bb2c0] Terminating demuxer thread
[aist#0:0/aac @ 000001f772fe6540] Decoder thread received EOF packet
[aist#0:0/aac @ 000001f772fe6540] Decoder returned EOF, finishing
[aist#0:0/aac @ 000001f772fe6540] Terminating decoder thread
No more output streams to write to, finishing.
[out#0/mp3 @ 000001f772f600c0] All streams finished
[out#0/mp3 @ 000001f772f600c0] Terminating muxer thread
[AVIOContext @ 000001f771525ec0] Statistics: 46129605 bytes written, 2 seeks, 177 writeouts
[out#0/mp3 @ 000001f772f600c0] Output file #0 (Scribe\storage\1.m4a.mp3):
[out#0/mp3 @ 000001f772f600c0]   Output stream #0:0 (audio): 110365 frames encoded (127139840 samples); 110366 packets muxed (46128483 bytes);
[out#0/mp3 @ 000001f772f600c0]   Total: 110366 packets (46128483 bytes) muxed
[out#0/mp3 @ 000001f772f600c0] video:0kB audio:45047kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.001528%
size=   45048kB time=00:48:02.97 bitrate= 128.0kbits/s speed=70.5x
[in#0/mov,mp4,m4a,3gp,3g2,mj2 @ 000001f7714bb2c0] Input file #0 (G:\My Drive\ABSA_Tweets-master\01HAPNHB8BPAXHQ1KM28DYR3SD-1592574611\bennett#TSP048 - Tom Bennett\1.m4a):
[in#0/mov,mp4,m4a,3gp,3g2,mj2 @ 000001f7714bb2c0]   Input stream #0:0 (audio): 124160 packets read (45258713 bytes); 124160 frames decoded; 0 decode errors (127139840 samples);
[in#0/mov,mp4,m4a,3gp,3g2,mj2 @ 000001f7714bb2c0]   Total: 124160 packets (45258713 bytes) demuxed
[AVIOContext @ 000001f7714c4f80] Statistics: 45816060 bytes read, 2 seeks
The system cannot find the path specified.
Could Not Find C:\Users\gnmarten\Downloads\Scribe\storage\1.m4a.txt
Press any key to continue . . .

It seems that whisper is not starting correctly, did you install python and the dependencies correctly? Were there any errors while you were starting Scribe for the first time?

It gave me a similar error. The cause of the error was that the load_audio function uses the ffmpeg command to load and decode the audio file. It was solved by installing ffmpeg and configuring it in the system PATH.

FFmpeg should automatically be configured in the PATH when you first install Scribe, maybe it had to do with administrator privilege's, if you are still having problems check the latest issue post I made. I'll make it clear in the README that FFmpeg needs to be installed.

I've added some information on FFmpeg in the README.

Thanks for the clarification. In my case, I did not have ffmpeg installed or the path configured previously and, for some reason, probably due to the administration permissions issue, the path was not configured. By the way, I didn't say it in the previous message, thanks for the application, I'm trying it out, very interesting and useful.

Glad you like it! If you have anymore questions please feel free to ask. Some day I need to revamp it because it's quite old and there are a lot of things I'd like to improve upon.

Did you have the issue on v<IP_ADDRESS> or v<IP_ADDRESS>?

Excellent, I'll be attentive. I had the numpy and ffmpeg issue with version 1.0.3

I ended up reworking the whole program already. I fixed pretty much all the bugs related to FFmpeg + a bunch of other stuff, I listed the fixes in the changelog. Let me know how it works whenever you get around to trying it!


Missing Description

-ExtensionProvider
{{Fill ExtensionProvider Description}}

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

ID: 3690f5b7-2873-5969-73ab-ec2478681747
Version Independent ID: a2f8ae1a-f299-c15f-d0b2-76afc1f5c0e8
Content: New-SPODataConnectionSetting (sharepoint-online)
Content Source: sharepoint/sharepoint-ps/sharepoint-online/New-SPODataConnectionSetting.md
Service: sharepoint-powershell
GitHub Login: @trent-green
Microsoft Alias: trgreen

@officedocsbot assign @yogkumgit

Hello @hwkwicre thanks for reporting

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @hwkwicre Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[Relay] Added Merge Composite pass

This adds a pass, MergeComposite, which merges patterns (expressed as Relay expressions). The detected patterns are wrapped in a function call which is marked with a 'Composite' attribute that names the pattern.
This is primarily for use with the external codegen infrastructure in the case where a combination of Relay ops map to a single external codegen op. It is not the same as fusion.
Further discussion on this PR can be found at https://discuss.tvm.ai/t/rfc-external-codegen-defining-composite-relay-operators/5470.

I've updated the PR to account for the priority ordering of patterns. You no longer specify a pattern map, but a list of tuples (pattern_name, pattern) where the order or the tuples corresponds to the order of the matching. To get this to work, I had to pass strings between Python/C++ requiring this change: https://github.com/apache/incubator-tvm/pull/4806.

I've done a big clean up + simplified the code a bit and added some comments.

@zhiics Regarding the add-sub/sub-add case, yes this would require two patterns with the order or merging controlled by their priority. I can't think of any general way to express both these cases as a single pattern, but if you have any thoughts I'd be glad to hear them. There is potentially an issue with requiring lots of patterns and if we can come up with some concrete examples where that may be the case then I can try and reason about how to improve the pattern matching.

@mbarrett97 Considering that this is a relatively standalone pass and it helps fusion with external functions, I think it is okay to take it in.
For the pattern matching cases, one possible way I am thinking of is automatically generating some possible pattern given some metadata.
@comaniac and I will start an RFC to discuss how we can add the whitelist based annotation back. It probably will leverage the pattern matching here.
@masahi @anijain2305 PTAL and share your comments if there is any. Thanks.

My only request is to add more test cases.  But, I am ok with the scope of this PR and delaying aggressive testing to later PR. I think it helps in quick flush of e2e pileline.

Oh just saw. The tests are already added. Thanks, its good from my side :)

@zhiics @comaniac It is worth discussing if we can use composite and partitioning passes to remove the annotation pass, as mentioned by @mbarrett97

Thanks @mbarrett97 @comaniac @zhiics @anijain2305 this is merged.

@masahi @comaniac @zhiics Thanks for the reviews. An RFC on alternative annotation mechanisms would be great.


Parse error on async IIFE with nightly build (javascript file)

TypeScript Version:  2.4.0-dev.20170606
Code
I am unsure of how to get a proper reproduction for this problem. It doesn't seem to happen consistently either, and just writing an async IIFE anywhere is not enough to cause the error.

Expected behavior:
Code should parse.
Actual behavior:
Parse error wanting an = in a place that doesn't make sense.

Any idea which nightly you were using prior?

I was running the latest 2.3 stable. Can try to bisect in a bit.

Can confirm that it parses on 20170601; will check the days in between.

This seems to only reproduce with 20170606, but only if --modules esnext is enabled.

@yuit can you take a look.

I can't repo this just from the code in the image. Would you be able to share more code in the same file?

I can't seem to find any way to reproduce this in less code than the entire 2800 line file so far.
The error happens on a .js file with // @ts-check, and only in this particular async IIFE. Converting it into a non-arrow expression changes the error to "identifier expected". I thought details such as being inside an async method inside an object, and having self-references could be a problem, but doing that in another file still doesn't reproduce it.
The only clue so far is that removing // @ts-check from this file makes it not have a parse error anymore.
The object structure is similar to:
const message = {
  /** @param {JQuery} inputField */
  async method (inputField) {
    // ...
    const receiverId = inputField.data('receiverId')
    if (receiverId) {
      (async () => {
        try {
          // ...
        } finally {
          // ...
        }
      )()
    }
  }
}

Would it be possible to share the file over email?

@Kovensky if you could send it over the email, that will be great. I also have a branch up (https://github.com/Microsoft/TypeScript/tree/master-fixIncrementalParsingWithDynamicImport) if you would like to give it a try !

@Kovensky  The PR is checked into master today. If you would like to give typescript@nightly a shot...Closing it for house keeping sake but feel free to reopen if the issue persist

It does seem to still be a parse error if I write it as an IIFE.
The workaround of saving the function in a variable before calling it is working for us, though.
I will see what I can do about getting a reproducible version of the file, but I'm not sure if I'll be able to do that soon 😕

passed with ; in begining
import * as puppeteer from 'puppeteer'

const url: string = 'https://www.socks-proxy.net'
;(async () => {
  const browser = await puppeteer.launch()
  const page = await browser.newPage()
  await page.goto(url)
  debugger

  await browser.close()
})().catch(console.error)


TTL values of the async persist request are not being utilized on master.

Describe the bug
This is discovered after fixing https://github.com/Alluxio/alluxio/issues/8404
TTL values of the async persist request are not being utilized on master.
This issue is to make this situation visible and decide a solution to guard against possible issues.

Still valid and will be considered for async write redesign


Show delete menu action for certain cells?

New issue checklist

[x] I have read the README, documentation, and FAQ.
[x] Contributing guidelines confirmation: ____
[x] I have searched existing issues and this is not a duplicate.

General information

JSQMessagesViewController version: 7.3.4
iOS version: 10.0
Devices/Simulators: iPhone 7

I haven't been able to find an example of this, but how do you allow the delete menu action item to appear for certain cells. In my case, I only want message cells that have been sent by the current user to show the delete action. I guess this question applies to all native and custom menu item actions.
Thank you :)

Implement something like:
override func collectionView(_ collectionView: UICollectionView, canPerformAction action: Selector, forItemAt indexPath: IndexPath, withSender sender: Any?) -> Bool {
    if action == #selector(delete(_:)) {
        return self.messages[indexPath.item].senderDisplayName() == self.senderDisplayName
    }
    return super.collectionView(collectionView, canPerformAction: action, forItemAt: indexPath, withSender: sender)
}

@acjh Ty so much!


Updating to newest standing controller

This feature updates the gains for run_osc_standing_controller and adds in the options to command different center of mass positions and pelvis orientations using the radio controller.

This change is

There's still an issue where the robot will shake even when it isn't at the edges of the support polygon. Could postpone updating until we fix this.
Might also be worth waiting until we have new gains for the updated model.


Remove the request.user requirement

I realized that requiring the Request object to have a .user attribute
is a product of 5 years of my legacy integrating Mixpanel. It is not
really required.
Turns out, by killing this one assumption, the code cleaned up considerably!

We probably want to review/polish/merge https://github.com/niteoweb/pyramid_mixpanel/pull/7 first ... I'm just putting this one out there for comments.


:wip: Adding image upload to the draft-editor

Because, #18 .

this is what's currently up, @nshntarora

As of now, we have 2 TODOs pending, which we'd take care of later. Lets keep it simple as of now.


[BUG]  Enhancement of UI of Signup Page

Description
The signup page UI has to be enhanced.
Steps to Reproduce
1.Go to the signup option in the navbar
2.On clicking on signup you can see the UI is not good.
Expected Behavior
The UI has to be improved.
Actual Behavior

Additional Information
Please assign me this issue
Browser
Safari
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach screenshots and videos of the development work once done

Can you please assign me level 2 on this issue

i worked on this issue  and this pull request i already raised


Isn't it supported "except field" of ipBlock of NetworkPolicy with VPC CNI ?

What happened:
As far as I know, AWS announced supporting NetworkPolicy with VPC CNI couple of month ago.
https://aws.amazon.com/blogs/containers/amazon-vpc-cni-now-supports-kubernetes-network-policies/
I've been testing this one to control Pod network traffic. And I tried to deny access instance metadata(<IP_ADDRESS>/32).
Unfortunately, this one isn't working "except filed" of ipBlock.
`apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
name: default-netpol
namespace: default
spec:
podSelector:
matchLabels:
app: test
policyTypes:

Egress
egress:
to:

ipBlock:
cidr: <IP_ADDRESS>/0
except:

<IP_ADDRESS>/32`

Refer to the below repository, you guys don't provide except filed hands-on.
https://github.com/aws-samples/eks-network-policy-examples/tree/main/advanced
I was wondering if VPC CNI don't support this feature.
Please let me know.

Environment:

Kubernetes version (use kubectl version): 1.28
CNI Version: v1.15.0-eksbuild.2
OS (e.g: cat /etc/os-release): cpe:2.3:o:amazon:amazon_linux:2
Kernel (e.g. uname -a): Linux ip-10-210-86-134.ap-northeast-2.compute.internal 5.10.192-183.736.amzn2.x86_64 aws/amazon-vpc-cni-k8s#1 SMP Wed Sep 6 21:15:41 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Please refer to this #58 . Issue should be addressed in the latest release (i.e.,) v1.15.1.

@achevuru, it doesn't work properly.
The NetworkPolicy "default-netpol" is invalid: spec.egress[0].to[0].ipBlock.except[0]: Invalid value: "<IP_ADDRESS>/32": must be a strict subset of cidr
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: default-netpol
  namespace: default
spec:
  podSelector:
    matchLabels:
      app: test
  policyTypes:
  - Egress
  egress:
  - to:
    - ipBlock:
        cidr: <IP_ADDRESS>/1
        except:
        - <IP_ADDRESS>/32

Your  /1 should cover the /32 you are using. But, with the new VPC CNI version you don't need the work around. You can upgrade and try your original policy.

@achevuru, I upgraded the VPC CNI version to 1.15.1 and checked it properly. (In version 1.15.0, setting for <IP_ADDRESS>/32 is not possible.)


Migrate v1 API

Hi, @tmtk75 !
I've tried to use brand new v1 API for Output Plugin.
Could you take a look?
If these changes are acceptable for you, please release new version as v1.0.0 instead of v0.2.0.
This PR breaks backward compatibility because brand new v1 API does not have backward compatibility.

Thanks!

HI @cosmo0920,
I really appreciate your maintenance.
I've released as you said. Please check it.
https://rubygems.org/gems/fluent-plugin-sqlite3
By the way, I think I'm able to add you as owner of fluent-plugin-sqlite3 if you'd like to frequently update the gem. Please let me know If you want.

By the way, I think I'm able to add you as owner of fluent-plugin-sqlite3 if you'd like to frequently update the gem. Please let me know If you want.

Sounds great!

OK, then are you using twitter?
I'm https://twitter.com/tmtk75 there. I will add you if you give me your email address via DM.

Yep, I'm in https://twitter.com/cosmo__.
But my rubygems account email address is same as commit email address in https://github.com/tmtk75/fluent-plugin-sqlite3/commit/7d94b3219221d01610690d8ec27d97795d71c635.patch.

Or my rubygems email address is displayed in public.
https://rubygems.org/profiles/cosmo0920

Hi, I think I added you probably. Can you take a look?
$ gem owner fluent-plugin-sqlite3 -a<EMAIL_ADDRESS>Owner added successfully.
Owners for gem: fluent-plugin-sqlite3
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>


Error in DSA/TDE System: Replace folder.data.name with folder.name

Describe the bug
There are console errors in the DSA/TDE system for Foundry VTT related to the use of folder.data.name, which should be updated to folder.name due to updates in Foundry's API. This may cause issues with folder functionalities within the system.
I encountered this bug when I tried to initialize the module Interactive Aventuria Map version 1.0.6.
To Reproduce
Steps to reproduce the behavior:

Install and enable the Interactive Aventuria Map module version 1.0.6.
Launch Foundry VTT with the DSA/TDE system enabled.
Attempt to initialize the module.
Observe the console during initialization.
Notice errors pointing to folder.data.name being undefined.

Expected behavior
The system should initialize without console errors, and all folder operations should function correctly by using the updated property folder.name.
Screenshots
/
Versions:

Foundry VTT Version: 12 Build 331
DSA/TDE Version: 6.1.4
Related Modules Version: Interactive Aventuria Map Version: 1.0.6

Additional context
In the initializer.js file, lines 217 to 240, the code currently accesses folder.data.name, which is deprecated in newer versions of Foundry VTT. The correct property should be folder.name. Here is the code and the specific changes:
    if (foldersToCreate) {
      let head = await this.getFolderForType('JournalEntry');
      let headReplace = json.folders[0].name;
      if (head) {
        this.folders[head.data.name] = head;
        json.folders.shift();
      }
      let createdFolders = await Folder.create(foldersToCreate);
      if (!Array.isArray(createdFolders)) createdFolders = [createdFolders];
      for (let folder of createdFolders) this.folders[folder.data.name] = folder;

      const updates = [];
      for (let folder in this.folders) {
        const flag = this.folders[folder].getFlag('dsa5', 'parent');
        let parent = flag == headReplace ? game.i18n.localize(`${this.moduleScope}.name`) : flag;
        if (parent) {
          updates.push({
            _id: this.folders[folder].id,
            parent: this.folders[parent].id,
          });
        }
      }
      await Folder.updateDocuments(updates);
    }

// Current code
this.folders[head.data.name] = head;

// Should be updated to
this.folders[head.name] = head;

// Current code
for (let folder of createdFolders) this.folders[folder.data.name] = folder;

// Should be updated to
for (let folder of createdFolders) this.folders[folder.name] = folder;

I have not checked if this issue occurs in other parts of the codebase, so there may be additional instances where folder.data.name needs to be replaced with folder.name.
Could you please update the code accordingly and release a new version?
All of the above was discovered by Kristov from the League of Extraordinary FoundryVTT Developers Discord while trying to help me get the module up and running.

Thx for the update.
When I initialize the module all journal entries are placed on the first level without subfolder. It would be nice if the journal entries are placed in a subfolder like "DSA5 Interaktive Aventurienkarte".
BTW: Is there an option to change the permission for multiple entries/folders?

Diese Fragen müssen an den Ersteller des Moduls gehen. Hier sind sie falsch.

Ich dachte es wäre ein Folgefehler von oben und läge in eurer Hand. Dann geht es an den Ersteller des Moduls.


Windows binary operator error

I'm getting the following error on Windows when I try to use this package:
.git/hooks/pre-commit: line 11: [: /c/Program: binary operator expected pre-commit:
This is line 11 from the referenced file:
if [ -z $HAS_NODE ] && [ -f ~/.bash_profile ]; then
Can you provide any help?
Package version 1.0.10
Node version: 0.10.31
NPM version 2.13.2
Windows 7

Having this issue as well.  I'm thinking that in this line "if [ -z $HAS_NODE ] && [ -f ~/.bash_profile ]; then"  it seems to be evaluating to C:/Program Files but since Program Files has a space then it throws the issue.

could you try adding ""so it becomes -z "$HAS_NODE"


java.lang.NullPointerException: validateProjectPath(...) must not be null

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Java: JetBrains s.r.o. 17.0.7
OS: Windows 10 amd64
IDE: WebStorm 2023.1.4
Version: 2.0.2

Additional context
Add any other context about the problem here.
Stacktrace
null

Duplicate of #246


Update to latest spec chages

In particular, there were a couple of changes, the ones I remember and that we did in FreeType, Skia and FontTools:

Paint*AroundCenter for scale, skew and rotate
Change of precision in a number of values, usually made smaller, in particular in PaintScale, for example https://github.com/googlefonts/colr-gradients-spec/commit/f6263236c9c855f6980655301f6039d4fe068989
No more retrieving of bounding box from glyf but either own computation of required glyph allocation size, or using the new ClipList and ClipBox
ColorIndex was removed in spec, but that might be a no-op in the implementation - inlining of alpha.
Affine2x3 matrix is now located at an offset, no longer inline https://github.com/googlefonts/colr-gradients-spec/commit/af58fa7a43dfc40feac6a5cf063db99ffe34122c

CC @anthrotype

I can't work on this earlier than september.

Didn't mean to rush, I just wanted to leave a note of the info I had on the main changes. This helped me to go through them when I was updating FreeType and Skia and thought this info might be useful here.

Yes, thanks for the note, that's useful indeed! I didn't mean to make you sound like you're rushing anyone, and just wanted to note that we haven't abandoned the project.


Is it possible to turn off the human reply dots (…)

When testing the form on others, someone commented that it looked like someone else was typing while they were typing. I think I agree – there's no reason to visually indicate that the user are typing, and it might be a distraction.

@joakimc it's a very valid point. We're considering whether this should be removed for the 1.0.0 release. Until then, you can remove the typing by cf-chat-reponse thinking{display: none !important;}

Thanks! I removed it with cf-chat-response text[thinking] {display: none !important;} to hide the bubble is as well while typing.


Move Combine logic into AuthorizeFilter

Fixes https://github.com/aspnet/Mvc/issues/4757

:watch:

@blowdart Can you help review?

Updated to call thru to the provider on each call to OnAuthorizization when a Policy is not explicitly set.  Added a test for this new behavior.

Any other feedback will push later today?

:watch: for the last two questions.

:shipit:

:shipit:

c7a46e4caf0d1f25dd5f8bcd385af50b34b0f5dd


Create a sequence diagram for helm-service

With Keptn 0.8.x new cloud-events have been introduced (e.g., deployment, release), which lead to a huge refactoring of helm-service.
While the code is now better structured, it's now not so easy to figure out what each step of helm-service should do, and we need to document this using a sequence diagram.
IMHO we need a sequence diagram for every single cloud-event that helm-service can process, with the following participants:

api/nats/distributor (entry point for the cloud-event)
helm-service
helm
kubernetes cluster (k8s)
configuration-service (git)

The following cloud-events come to my mind:

onboard service / create service
delete service
deployment:promote (for both, blue-green and direct deployments)
deployment:rollback (for both, blue-green and direct deployments)
release (for both, blue-green and direct deployments)
remediation action (e.g., scaling)

Using the sequence diagram, one needs to be able to figure out the following things easily:

When is helm install/upgrade called
When something is read from the git repo
When is something updated in the git repo
When is traffic shifted and to which service
When is values.yaml file updated
When a cloud-event is sent back to the api/nats/distributor

Definition of Done

[ ] The aforementioned sequence diagram and any necessary documentation for it is placed in a docs folder in helm-service.

Sequence diagram for handling service.created events (i.e. onboarding a service):

Sequence diagram for handling service.deleted events:

Sequence diagram for handling deployment.triggered events:

Sequence diagram for handling release.triggered events for a direct deployment:

Sequence diagram for handling release.triggered events for a b/g deployment which is promoted:

Sequence diagram for handling release.triggered events for a b/g deployment which is rolled back:

Sequence diagram for handling action.triggered events (i.e. scaling the deployment):

I used https://sequencediagram.org/ for the following sequence diagrams. Furthermore, I have attached the sources.
helm-service_action-triggered.txt
helm-service_deployement-triggered.txt
helm-service_release-triggered-bg-promote.txt
helm-service_release-triggered-bg-rollback.txt
helm-service_release-triggered-direct.txt
helm-service_service-created.txt
helm-service_service-deleted.txt

Thanks. We will need this in the repo in the helm/docs folder!
@warber you have been working a lot with helm-service lately, can you also please take a look at the sequence diagrams and let us know if you believe that they are representing the current implementation?


Upload asset antrea-windows-containerd.yml

Upload new asset antrea-windows-containerd.yml to support Windows with containerd.
Signed-off-by: Lan Luo<EMAIL_ADDRESS>

/skip-all


Can not load native addon module in production

Describe the issue / bug.
#Can not load native addon module in production
How can I reproduce this problem?
# write a native addon and build for production
Tell me about your development environment.

Node version: 8.10
NPM version: 3.10.10
vue-cli version: (if necessary)
Operating System: Windows 10 64-bit

Hi,
I want to use a custom native addon module in my electron-vue app. The module is located in
C:\my-application-root-dir\node_modules\myaddon\build\Release\myaddon.node

and in src\main\index.js I require my module with relative path:
var myaddon = require('../../node_modules/myaddon/build/Release/myaddon')

in the package.json, I put my module in dependencies:
  "dependencies": {
    "myaddon":"^1.0.0",
  },

this config is working in the develop environment.
When I build my app with electron-builder for production, build was successfully and a setup file is generated, and I find the myaddon in unpacked directory under build
C:\my-application-root-dir\build\win-ia32-unpacked\resources\app.asar.unpacked\node_modules\myaddon\build\Release\myaddon.node

However after I install my application in another PC, A Javascript error occurred in the main process:
Uncaught Exception:
Error:
Cannot open C:\my-application-root-dir\node_modules\myaddon\build\Release\myaddon.node: Error: Cannot find module .

It seems try to find the module with an absolute path which only exist in my development machine.
After I search for answers for hours I found something in the electron-builder document:
If you have native addons of your own that are part of the application (not as a dependency), set nodeGypRebuild to true.

does this mean, in my case, myaddon is not as a dependency? Should I put my myaddon inside /node_modules or in /src ?
And if I set nodeGypRebuild to true,  the build process will fail, did I make something wrong?
Or the way I require my custom addon is wrong?
Any help would be appreciated.

I found a workaround here
https://github.com/electron-userland/electron-packager/issues/217
if (process.env.NODE_ENV === 'development') {
  var myaddon = require('../../node_modules/myaddon/build/Release/myaddon')
} else {
  const path = require('path')
  const module = require('module')
  module.paths.push(path.resolve('node_modules'));
  module.paths.push(path.resolve('../node_modules'));
  module.paths.push(path.resolve(__dirname, '..', '..', '..', '..', 'resources', 'app', 'node_modules'));
  module.paths.push(path.resolve(__dirname, '..', '..', '..', '..', 'resources', 'app.asar', 'node_modules'));
  module.paths.push(path.resolve(__dirname, '..', '..', '..', '..', 'resources', 'app.asar.unpacked', 'node_modules'));
  var myaddon = require('myaddon/build/Release/myaddon')
}

issue solve by replace node-loader with native-ext-loader, I also test node-addon-loader, but it's not working.
webpack.main.config.js:
      {
        test: /\.node$/,
        loader: 'native-ext-loader',
        //use: 'node-addon-loader',
        //use: 'node-loader',
        options: {
          //rewritePath: path.resolve(__dirname, 'dist')
        },
      }

for Electron applications, the rewritePath needs to remain undefined.
then in \src\main\index.js, require the native module with relative path:
var myaddon = require("../../node_modules/myaddon/build/Release/myaddon");

The error "Cannot find module" can happen if the module is not compiled for your electron version. The error is misleading, because the file exists but is not recognized as a module for the specific version.


Default kubernetes version is not secure (CVE-2018-1002105)

Let's update hetzner-kube to the latest version so we are not impacted by the breach !
https://nvd.nist.gov/vuln/detail/CVE-2018-1002105

#240

pr #240 should do the trick

@pushthat can you move back to 1.9.11 (or the last patch version of the same minor)

Yes
Do you think we could add a option to select the version we want to deploy ?

@mavimo I didn't double check earlier but unfortunatly 1.9.11 is not fixing CVE-2018-1002105.

Is this still a concern? It's obviously a show-stopper for using hetzner-kube.

I recently installed a new Kubernetes cluster with hetzner-kube and got version v1.16.3, so this issue should be closed.

@tholu thx for your feedback!


Disable parallel execution for test_gettext

This works fine on Windows, but not so on macOS/Linux. Apparently, Mutex implementation in Mono doesn't work as expected. As a workaround, I run tests with -frameworks notcoreapp2.1,netcoreapp3.1 and then Mono separately.
I don't know how to make it work under Mono. I am making this pull request anyway since it is better than nothing. @slozier, @jameslan, any ideas?

Mutex doesn't work at all, or in combination with the .NET core mutex? If I had to guess, Mono and .NET Core are using incompatible Mutex implementations. More work for us, but we could probably accomplish the same thing via lock files.

Ah, mutex isn't cross-process on Mono non-Windows. See https://aakinshin.net/posts/namedmutex-on-mono/. It looks like there's a link to an alternative solution using file streams.

Thanks @slozier for investigating it. The blog post you found really explains a lot. So lock files it is. I suppose this can be implemented in a separate PR.

@BCSharp Created a separate issue for the Mutex thing: https://github.com/IronLanguages/ironpython3/issues/946


lease compatible

The related issue: https://github.com/open-cluster-management/backlog/issues/13338
Signed-off-by: Yang Le<EMAIL_ADDRESS>

/assign @qiujian16

/hold Don't merge until the fix is verified with latest version of acm

/hold cancel

/approve
/lgtm


[KYUUBI #2120] Optimize RenewDelegationToken logs in Spark engine

Why are the changes needed?
Now Spark supports RenewDelegationToken, but there is no log on the driver side when it is actually updated.
https://github.com/apache/incubator-kyuubi/issues/2120
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #2121 (5c651d1) into master (f8efcb7) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2121      +/-   ##
============================================
- Coverage     61.19%   61.18%   -0.01%
  Complexity      108      108
============================================
  Files           317      317
  Lines         15441    15444       +3
  Branches       1988     1988
============================================
+ Hits           9449     9450       +1
  Misses         5186     5186
- Partials        806      808       +2

Impacted Files
Coverage Δ

...ubi/engine/spark/SparkTBinaryFrontendService.scala
82.43% <100.00%> (+0.74%)
:arrow_up:

...ache/kyuubi/operation/KyuubiOperationManager.scala
92.50% <0.00%> (-2.50%)
:arrow_down:

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
96.40% <0.00%> (-0.14%)
:arrow_down:

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 f8efcb7...5c651d1. Read the comment docs.

22/03/14 00:10:28,514 [SparkThriftBinaryFrontendServiceHandler-Pool: Thread-118] INFO SparkThriftBinaryFrontendService: Update delegation tokens.The number of tokens sent by the server: 1.The actual number of updated tokens: 1.

thanks, merging to master


Rename File: Allow New Parent Directory

When renaming a file, check if the new path contains a parent directory that does not exist yet and create it.
Eg: file.x => dir/file.x : create dir/ directory first.
Currently this results in an error.

That is actually what the move command does. If you have the default mappings, just use m.

Is there any reason for splitting the functionality between move and rename?
Seems like they should just have the same behaviour.

It's not really a split at all, I am just emulating what is normally found at the command line. What you requested is actually a mv command, not a rename command.
It just so happens that mv can also be used to rename. Personally, when at the command line, I always use mv even when I am renameing because it is shorter to type and more versatile for a single file or directory.

I also had the same question before, now I only use 'move' for renaming. While it's strange(?) that:

It just so happens that mv can also be used to rename. [...]

I think this should imply that we actually don't need the rename functionality?

I think this should imply that we actually don't need the rename functionality?

I can certainly see why you would say that, but I disagree. To me, having different commands for different actions helps reduce accidents. I would never intentionally move a file when I meant to rename, so if it did happen it would be a typo and I would want it to fail.
Also, if I did remove the "rename" command as redundant, I would still leave "move" as "move". I would never rename "move" to "rename" because that would not be an accurate description of what the command does.
Of course, I am just talking about the default mappings, which you don't have to use at all. Feel free to map r to "move" if you want!

I would never intentionally move a file when I meant to rename, so if it did happen it would be a typo and I would want it to fail.

This is very reasonable to me, thanks for your clarification!


after 'loading pretrained weights from vgg_caffe.pth', appear the error 'segmentation fault'

please post the entire stack trace

Same issue.
Indeed, there are no other warning or info or something else to help me solve this.
My env as follow:
Python = 3.7
Pytorch = 1.0
gcc = 5.2
cuda = 9.0

Wow, I have solved this issue. Indeed, the problem comes from the wrong version of python. If anyone face this issue I think you can try to rebuild a env.
I use the branch with pytorch-1.0. So, you should sure your pytorch version should be 1.0.0, and the Python should be 3.6. Of course, you should sure your cuda version is match to your pytorch. Last, you maybe need to updata(if your version < 5.2) your gcc to 5.2.0. And if you use anaconda, you can smplily find the gcc-5-5.2.0 on the Anaconda Clould. Install the gcc follow the command, after that the gcc version will change to 5.2.0. It is more easy and handy than install/updata your gcc version by yourself I think.


Add notification snackbar

Notification snackbar added when some event arrives from websocket:

Why are we replacing this?

I couldn't see any notification on screen when events arrived even with the displayNotifications function. Perhaps some misconfiguration in my environment?

Have you enabled permissions?

Notification permissions on Traccar server? Yes.

Not sure why it doesn't work then. You should probably investigate it instead of re-implementing it.

Right. I'll do that and if I don't find anything I'll close this PR.


[Enhancement] Option to NOT automatically target a token when dropping damage

@InexplicableVic
Hello, all. In GURPS Game Aid, after I drag and drop damage from the chat window onto a target and apply the damage, the target is "sticky" and won't release unless I go to the "Select Targets" tool and then click on the target to get rid of it. Any way to make that easier for the GM?

It now remembers what was targeted, saves it, targets the token to drop damage on it, and then resets the targets back to the original set (or none, if nothing was targeted).


Option to specify other name for req.user

Currently, successful authorization puts the JSON payload into req.user. This is confusing for those who are not associating a user per say with the payload but maybe some other model naming convention of theirs. It would be a nice option to have an option that would specify the name in req object other than user, like account or signer, etc.

https://github.com/jaredhanson/passport/blob/master/lib/authenticator.js#L117
This is a configuration option of passport and not up to individual strategies to implement.  See the document for passportjs.


Message box for Input variable opens on wrong monitor

Expected behavior
In a setup with two monitors:
When typing a shortcut for text with an input variable, the message box for that input variable should open above the window in which I typed the shortcut.
Actual behavior
The message box opens on the monitor, in which the Beeftext window itself is displayed, not on the monitor in which the txt was typed. I have to search for that message box.
Steps to reproduce the behavior
Start Beeftext on a system with two monitors. Display Beeftext on one monitor, a text editor on the other monitor. Type a combo that will show a message box.
Operating system
Windows 10
BTW: The new input variable is great. Thank you for adding it.

Hi @blauebank
This is strange, because I get a different behaviour. I have a three monitor setup, and the input dialog does not pop on the monitor where Beeftext is, but on the monitor where the mouse cursor is located.

What happens if the Beeftext window is closed?
Do you use any extra window or monitor management tool, like DisplayFusion?

I tried again this morning. It seems like the
first input dialog after I start Beeftext will
always pop up on my main monitor. The input dialogs afterwards are correct.
Here is a short screen capture:
https://www.screencast.com/t/3XYvApaaoHIX

First I type the shortcut in the text editor on
the right monitor. The Beeftext pop up  appears on the left monitor.
Then I click randomly for  5 second, please ignore.
Then I type again in the text editor on the
right monitor. This time the Beeftext pop  up  appears on the right monitor.
If I close Beeftext and restart it, I can repeat this behavior.

I don't use any extra window management tool.
At 21:14 26.03.2019 -0700, you wrote:

Hi https://github.com/blauebank@blauebank
This is strange, because I get a different
behaviour. I have a three monitor setup, and the
input dialog does not pop on the monitor where
Beeftext is, but on the monitor where the mouse cursor is located.

What happens if the Beeftext window is closed?
Do you use any extra window or monitor
management tool, like DisplayFusion?

You are receiving this because you were mentioned.
Reply to this email directly,
https://github.com/xmichelo/Beeftext/issues/141#issuecomment-476965681view
it on GitHub, or
https://github.com/notifications/unsubscribe-auth/AurwEew9MuX28oddNGlWbKlKpXHz_cerks5vavAOgaJpZM4cLTMSmute
the thread.

Hi.
From what I see in your screencast, the dialog pops-up on the monitor where the mouse pointer is located, which is the expected behavior.

So the dialog does not care about the cursor, but pops up on the window of the mouse pointer?
Ahh...
I didn't get that the dialog followed the mouse pointer. I was expecting it the pop up on the window of the cursor.


chore: refactor share splitting and merging

Description
This PR refactors splitting and merging. It incorporates the changes found in https://github.com/celestiaorg/celestia-core/pull/820, please see the second commit that addresses the remaining feedback from that PR. The rest of the changes involves splitting up the existing code into different files and adding backwards compatibility with blocks that use and don't use wrapped txs for non-interactive defaults (pointed out below).
part of: #382
closes: #632

this refactor is a herculean effort 💪 great work! @evan-forbes

Codecov Report

Merging #637 (fb6b9d7) into main (24bbf82) will increase coverage by 6.13%.
The diff coverage is 58.14%.

@@            Coverage Diff             @@
##             main     #637      +/-   ##
==========================================
+ Coverage   29.78%   35.92%   +6.13%
==========================================
  Files          16       24       +8
  Lines        2055     2628     +573
==========================================
+ Hits          612      944     +332
- Misses       1382     1600     +218
- Partials       61       84      +23

Impacted Files
Coverage Δ

pkg/shares/utils.go
0.00% <0.00%> (ø)

pkg/shares/share_splitting.go
13.25% <13.25%> (ø)

pkg/shares/shares.go
50.00% <50.00%> (ø)

x/payment/types/payfordata.go
75.00% <62.50%> (-0.36%)
:arrow_down:

pkg/shares/split_message_shares.go
69.51% <69.51%> (ø)

pkg/shares/split_contiguous_shares.go
70.40% <70.40%> (ø)

pkg/shares/share_merging.go
70.78% <70.78%> (ø)

pkg/shares/merge_contiguous_shares.go
77.77% <77.77%> (ø)

pkg/shares/merge_message_shares.go
88.60% <88.60%> (ø)

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

after thinking about this a bit more, unfortunately I think we'll have to this, along with the non-interactive default logic, to core. The idea of keeping all app specific logic is ideal, but as discussed in https://github.com/celestiaorg/celestia-core/issues/842 I don't think we'll be to avoid have at least a copy there. It would be one thing to only have a copy of the contiguous share splitting logic, but we will also have to have a copy of the non-interactive default logic.
The refactor and move to ./pkg here is still worthwhile, so I think we should merge this first and then we can move the relevant portions of the non-interactive defaults over to the core/pkg directory as well.

merging now, even tho we're planning on moving this to its own repo as per this comment


AccountLocks: validate_account_locks

Problem

We want account lock validation should be possible for any SVMMessage:

Code currently lives in SanitizedTransaction

Summary of Changes

Add account_locks::validate_account_locks that takes AccountKeys and limit
Add custom has_duplicates:

for >= 32 accounts, use a pre-allocated AHashSet
for <  32 accounts, use brute-force O(n^2) algorithm

Fixes #

See the final benchmark results (modified) below.
Instead of normal bench-suite I ran for batch size of 1 for various numbers of accounts.
I did this to confirm that 32 accounts was the sweet spot for switching over to using the hash set instead of O(n^2)

Benchmark Results
bench_lock_accounts/batch_size_1_locks_count_2
                        time:   [249.89 µs 250.13 µs 250.36 µs]
                        thrpt:  [4.0901 Melem/s 4.0939 Melem/s 4.0978 Melem/s]
                 change:
                        time:   [+0.8693% +1.0815% +1.2943%] (p = 0.00 < 0.05)
                        thrpt:  [-1.2777% -1.0699% -0.8618%]
                        Change within noise threshold.
Found 5 outliers among 100 measurements (5.00%)
  2 (2.00%) low mild
  3 (3.00%) high mild
bench_lock_accounts/batch_size_1_locks_count_4
                        time:   [342.27 µs 342.59 µs 342.90 µs]
                        thrpt:  [2.9863 Melem/s 2.9890 Melem/s 2.9918 Melem/s]
                 change:
                        time:   [-1.9893% -1.7873% -1.5844%] (p = 0.00 < 0.05)
                        thrpt:  [+1.6099% +1.8198% +2.0297%]
                        Performance has improved.
Found 2 outliers among 100 measurements (2.00%)
  1 (1.00%) high mild
  1 (1.00%) high severe
bench_lock_accounts/batch_size_1_locks_count_8
                        time:   [546.80 µs 547.51 µs 548.25 µs]
                        thrpt:  [1.8677 Melem/s 1.8703 Melem/s 1.8727 Melem/s]
                 change:
                        time:   [-0.4993% -0.2809% -0.0710%] (p = 0.01 < 0.05)
                        thrpt:  [+0.0711% +0.2817% +0.5018%]
                        Change within noise threshold.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) low mild
  1 (1.00%) high mild
bench_lock_accounts/batch_size_1_locks_count_16
                        time:   [1.0024 ms 1.0032 ms 1.0040 ms]
                        thrpt:  [1.0199 Melem/s 1.0208 Melem/s 1.0216 Melem/s]
                 change:
                        time:   [-0.9296% -0.7228% -0.5230%] (p = 0.00 < 0.05)
                        thrpt:  [+0.5258% +0.7281% +0.9383%]
                        Change within noise threshold.
Found 5 outliers among 100 measurements (5.00%)
  4 (4.00%) low mild
  1 (1.00%) high mild
bench_lock_accounts/batch_size_1_locks_count_32
                        time:   [1.8623 ms 1.8636 ms 1.8649 ms]
                        thrpt:  [549.09 Kelem/s 549.46 Kelem/s 549.84 Kelem/s]
                 change:
                        time:   [-7.1087% -6.9359% -6.7658%] (p = 0.00 < 0.05)
                        thrpt:  [+7.2568% +7.4528% +7.6527%]
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe
bench_lock_accounts/batch_size_1_locks_count_48
                        time:   [2.6437 ms 2.6461 ms 2.6484 ms]
                        thrpt:  [386.65 Kelem/s 386.99 Kelem/s 387.33 Kelem/s]
                 change:
                        time:   [-18.093% -17.911% -17.729%] (p = 0.00 < 0.05)
                        thrpt:  [+21.550% +21.819% +22.090%]
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild
bench_lock_accounts/batch_size_1_locks_count_64
                        time:   [3.4304 ms 3.4333 ms 3.4364 ms]
                        thrpt:  [297.99 Kelem/s 298.25 Kelem/s 298.51 Kelem/s]
                 change:
                        time:   [-26.656% -26.489% -26.319%] (p = 0.00 < 0.05)
                        thrpt:  [+35.720% +36.034% +36.345%]
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild


App and pod validation errors for missing network name MARATHON-7398

Throws validation exception instead of just NormalizationException when default network name is not specified

Hey @jdef,
when your back after US holidays, could you have a look, this would be awesome!
Thanks =)

Thanks for the review! I'll fix those comments over the weekend :-)

@jdef @unterstein can you re-review please? I fixed all your comments
PS: Sorry for the delay - holidays :-)

@jdef @unterstein anything I can do to get this merged? :-)

@jdef good point - fixed

Thanks @alenkacz !
    .  o ..
     o . o o.o
          ...oo
            __[]__          Shippin'
         __|_o_o_o\__         it!
         \""""""""""/
          \. ..  . /
     ^^^^^^^^^^^^^^^^^^^^


restore_events_polling_thread

-> should fix https://github.com/vector-im/vector-android/issues/133
The polling thread does not seem to be used when GCM is disabled
-> the events polling thread is restored when GCM is disabled
-> make GA and GCM use configurable (gradle)
-> update build script to build a FDroid version (without GCM) and a google play version (with GCM)

GcmRegistrationManager.java:
Why new field is named sUseGCM and not mUseGCM? even if it is a static field should not it ne named mUseGCM
useGCM(): missing code in catch() section, at least a Log msg..
setUseGA(): missing code in catch() section, at least a Log msg..
method naming updateListenerNotification(): since this method sets the service in foreground or remove it from foreground , why it is "only" called updateListener. The important point here is that this method modifies the priority of process the service belongs to.

Apart these remarks LGTM!


Fix focus interop bugs

Wrong focus was when we have:

a popup and a SwingPanel. Fix is in SkiaBaseOwner - we pass different parent focus manager for popups.
SwingPanel inside ComposeWindow. Fix is in ComposeLayer and ComposeWindowDelegate.

The feature PR

merging into jb-main


Add ZPE Propulsion System from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of GagaX, to add ZPE Propulsion System to CKAN.
Please direct questions about this pull request to GagaX.

Closing in favour of #2981


Add support for watch mode in dev server

Currently, you have to run vercel dev each time a change is made to the code.
It would be super nice if the server can watch for changes in filesystem and restart

This is definitely working for me here (see screenshot for vc version). Feel free to reopen if the issue still exists.


add support for tentative letters

As a solver, I sometimes want to guess at an answer and enter it into the grid, while making it clear that I haven't fully solved the clue and this is therefore not a "firm" entry. It would be useful for exolve to support this, perhaps with some sort of cell background (using lowercase letters is common but that only works for alphabets that have lowercase).

I have updated the code to deal with this better.
Please note that Exolve somehow would need to know how to segment a sequence of characters in the placeholder slot into letters. I have taken the same approach for this as in the grid specification: you can separate letters by spaces.
See this documentation that I've now added to the README file:
You may also want to provide placeholder blanks for languages such as Hindi that have compound letters. Exolve recognizes multi-character letters separated by spaces within placeholders. So, if you have a 3-letter clue, and the placeholder text contains "उ स्ता द" then copying this into a 3-letter entry will place "उ" in the first cell, "स्ता" in the second cell, and "द" in the third cell.

BTW: there are many times when it would be nice to get weigh-ins from more users, and I find that rarely happening on GitHub Issue/Discussion area. So I've now created a Google Group. Do consider joining: https://groups.google.com/g/exolve-exet-etc

Uh. I just realized, that my previous comment should have gone to a different issue altogether :-)


DialogFlow detectIntent not returning control

I'm trying to detect an intent from a text message sent via Messenger. If I call the method detectIntent, the code is somehow "stuck" and I never get the control back:
<?php

$sessionsClient = new SessionsClient();

try {
    $logger->info('try');
    $formattedSession = $sessionsClient->sessionName(getenv('GCP_PROJECT_ID'), '123456');

    $textInput = new TextInput();
    $textInput->setText($message);
    $textInput->setLanguageCode('it-IT');

    $queryInput  = new QueryInput();
    $queryInput->setText($textInput);
    $dfResponse = $sessionsClient->detectIntent($formattedSession, $queryInput);
} catch (Exception $e) {
    $logger->info('catch');
    $logger->info($e->getMessage());
} finally {
    $logger->info('finally');
    $sessionsClient->close();
}

$logger->info('after');

This code logs:
try
finally

and never print after. What am I doing wrong? Is this related with GuzzleHttp promises library?
I'm on PHP 7.4.10 and library version 0.17.3.

Do you have the gRPC extension installed and enabled? If not, have you tried this code with gRPC? Have you tried in different environments (i.e. locally and on AppEngine or a web server)?

I've not enable gRPC extension, the documentation says that is necessary only for streaming detection (audio), should I try it anyway? I'm making the request in a local docker environment, any hint to a possible cause that I can investigate? Thanks.

If it isn't too much trouble to test using gRPC, it would be quite helpful in isolating a cause, yes. You should also see performance improvements once the problem is ironed out.

Enabling gRPC extension seems to fix the problem:
FROM php:7-fpm
RUN apt-get update -q -q && apt-get install -y zlib1g-dev \
    && pecl install grpc \
    && docker-php-ext-enable grpc

I think that the documentation should more clear about that. Thanks for your time.

Glad that you are unblocked! We'll look into the issue with the REST transport.

@giampa-cz I've tried reproducing this and haven't run into the same issue, although in my use case I swapped logger calls with some basic print statements. I'm going to close this out, but if you're still running in to any issues please feel free to re-open and please provide some more details about the environment you're running your code in.


Fix help used as command returns errors when required flags are missing

What type of PR is this?

bug

What this PR does / why we need it:
Using app help or app h without providing required flags causes app.Run() to return an errRequiredFlags error.
This does not happend if using -help or -h.
Added a function to check if any of the arguments matches HelpFlag.Names() to stop early and show the app help.
Which issue(s) this PR fixes:
Fixes https://github.com/urfave/cli/issues/1247
Special notes for your reviewer:
I'm new to the code base so perhaps there is an easier way to check if the command matches help or h  before calling checkRequiredFlags. However, context.Command.Name is empty just before checkRequiredFlags so I couldn't just check that first.
Testing
Added test Test_Help_Required_Flags_Does_Not_Error to help_test.go
Release Notes
(REQUIRED)

Fixed https://github.com/urfave/cli/issues/1247

will be great to see this landed as it is surprising behavior without a fix. @jaimem88 let me know if you ran out of steam!

I think this also hits when help is default. Ex. if your app is named "foo", foo and foo help will trigger the unnecessarily arg validation

Issue not present in latest releases. Closing.

No movement for this for a long time and fix isn't sufficient. Closing for now

Closing. Will revisit when new v3 parser is introduced.


Volume roll current contracts could flip back and forth

In get_contract_center of the VolumeRollFinder, we only look at a single date when trying to determine what the current contract is. This means that when deciding between the given front and back contracts we just choose the one with the higher volume. Since this method has no knowledge of any other dates, it can potentially flip flop on the current contract from day to day. This means that getting the active contract on a single date needs knowledge of the rolls of surrounding dates.
This symptom of seeing flip-flopping rolls could also be seen in history calls due to appending the current contract to the last day here: https://github.com/quantopian/zipline/blob/master/zipline/data/data_portal.py#L811
The one-off changes to the dates in the test are a result of https://github.com/quantopian/zipline/pull/1863, which allows contracts to be on active on their auto close date.

Coverage increased (+0.02%) to 87.244% when pulling ba7ce55390dcaa7fe68d35bbf4a52facc686cb56 on spot-contract-broken into e28b704d039b6b8b995e701f7a1f970e6d61a25f on master.

Coverage increased (+0.02%) to 87.246% when pulling 62f58196e3aebd419321a8de076e7b00b9ae5c17 on spot-contract-broken into e28b704d039b6b8b995e701f7a1f970e6d61a25f on master.


[WIP]xgboost4j-spark

it is a WIP PR for xgboost4j-spark

@tqchen , one thing we need to confirm as how the format of DMatrix binary file looks like,
In Spark/Hadoop, the output looks like
dmatrix1-partition0

dmatrix1-partition1

dmatrix1-partition2

Can we assume each partition forms a complete DMatrix?

@CodingCat

I think we do not need to save DMatrix binary file, instead always load data from spark's native data structure
Please remove tracker.py from repo as it is created by copy from dmlc-core/tracker/dmlc_tracker. It would be great if we have a pom that does this copy in build time.
Since we are already under package(namespace spark). I think we are safe to name the class XGBoost as opposed to XGBoostSpark

in another words, in the spark and other dataflow interface, the input data is RDD, DataSet and DMatrix is like developer API that is invisible in the higher level

please do a squash of commits, so the file creation and deletion won't get into history

The first step goal is to make a training API that load data distributedly like DistTrain to validate the concept.
Then we want to take RDD[LabeledPoint] in sparkMLLib as input, and convert that into DMatrix

hmmm....
Let me describe with an example
case 1:
we have a training dataset which is a larger version of demo/data/agaricus.txt.train, we save it in HDFS and load as RDD[String]; for each partition, it is a Iterator[String].
case 2: dense matrix,
the RDD loaded from HDFS is essentailly RDD[Float], for each partition, it is Iterator[Float],
case 3: sparse matrix (CSR)
In this case, we just need to ensure that we partition the data in a perfect way, so that each partition of RDD is a parse-able CSR format, the partition of the RDD loaded from HDFS is Iterator[CSRPartition]
In the above case, we need to ensure that when we save file, each partition of the file is parse-able, especially for the third case, right?


environment.yml does not have azure-quantum package in it

https://github.com/microsoft/qdk-python/blob/main/azure-quantum/environment.yml does not list azure-quantum package.

This is on purpose. The environment.yml is what we use to setup the local environment to develop azure quantum. After setting up your conda environment you still need to do pip install -e . in the environment, to install azure quantum in development mode.
For more detailed instructions on how to setup your environment, please take a look at: https://github.com/microsoft/qdk-python/blob/main/CONTRIBUTING.md


Go back to code formatting for file names in docs

Italics is too hard to read. Dumb change. Change them all back.

Accidentally checked into master 🤦♂


Multi data in each row in funnel chart

What problem does this feature solve?
Similar to https://i.ibb.co/0yRPtLT/Funnel.png
Having multiple data each row in the will help show multiple data for the same element. Say "visits" and "engagement" for a particular URL. The funnel percentage can be calculated by the sum of the values used in it.
What does the proposed API look like?
See the attached image: https://i.ibb.co/0yRPtLT/Funnel.png
series: [
        {
            name:'漏斗图',
            type:'funnel',
            left: '10%',
            top: 60,
            bottom: 60,
            width: '80%',
            min: 0,
            max: 100,
            minSize: '0%',
            maxSize: '100%',
            sort: 'descending',
            gap: 2,
            label: {
                show: true,
                position: 'inside'
            },
            labelLine: {
                length: 10,
                lineStyle: {
                    width: 1,
                    type: 'solid'
                }
            },
            itemStyle: {
                borderColor: '#fff',
                borderWidth: 1
            },
            emphasis: {
                label: {
                    fontSize: 20
                }
            },
            data: [
                {name: 'baidu', values: [
                    {
                        data: 50,
                        label: 'Visits'
                    },
                    {
                        data: 30,
                        label: 'Engagement'
                    }
                ]},
                {name: 'google.com', values: [
                    {
                        data: 30,
                        label: 'Visits'
                    },
                    {
                        data: 60,
                        label: 'Engagement'
                    }
                ]},
                {name: 'weChat', values: [
                    {
                        data: 80,
                        label: 'Visits'
                    },
                    {
                        data: 90,
                        label: 'Engagement'
                    }
                ]}
            ]
        }
    ]

Hi, I think it's a feature we won't plan currently. You can try to draw it by using custom series.


Self Evaluation 🧐 Courtroom ⚖

A task where three instances of a language model interact within a court-room ⚖ setting
and a fourth model evaluates the other models. The lawyer language models (the prosecutor and the Defense Attorney)
have to take sides on a wide range of issues some of which do not have a single answer (like the Trolley problem).
The judge language model 👩⚖ has to understand the arguments made on both sides, ask relevant questions to inform the debate
and resolve the dispute by finally announcing a judgement as unbiased as possible.
The fourth language model is a bystander who looks at each of the participants and evaluates their
respective capabilities 🧐.

@kaustubhdhole Thank you for submitting your task! We are in the process of pairing submissions with task reviewers and will be in touch soon.

@kaustubhdhole We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

@a-dai are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@TalSchuster are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

Hi. This is interesting and super creative! I really like the idea of letting LMs debate and argue with each other (Maybe add IBM's debater project as related work?) I have one technical question and one question about the evaluation setting:

I tried to load it with the colab notebook but couldn't find the task in the drop list. Do you know how to resolve this?
How do you justify the self-evaluation? If I understand correctly, the LM here is evaluating itself as the same model is used for generation and for scoring. What prevents the LM from just evaluating itself high no matter what it actually did? If an LM made a bad argument, why should we expect the same LM to identify that this is a bad argument? I feel like we might need some objective way to control the evaluation. One direction I was thinking of is to intentionally add some noise for the generation model, and to see if the evaluating model scores it lower than the noise-free generation. But it might also be problematic as the scorer might just score based on level of noise... What do you think?

Hi Tal,
Thank you for your review and your questions:
If I understand correctly, the LM here is evaluating itself as the same model is used for generation and for scoring. What prevents the LM from just evaluating itself high no matter what it actually did?
Since both are taking alternative turns, the third language model has to pick up one to make a decision and choose one. Also, the model is being passed a speaker argument (which changes from one speaker to another), so ideally - the model isn't rating itself. Also, I feel the judge choosing one model instance over another would imply that one model performs poorly and one doesn't.
One direction I was thinking of is to intentionally add some noise for the generation model, and to see if the evaluating model scores it lower than the noise-free generation. But it might also be problematic as the scorer might just score based on level of noise... What do you think?
This is an interesting direction! The noise can definitely be a perturbed version of the current context or some negated/noised version which invalidates the arguments made in the context!

Hi @kaustubhdhole,
Thanks for your answer.
If I understand correctly, it's not different LMs but the same LM wearing different hats (i.e. queried with different prompts). It is analogous to a student at school being asked to play two parts in a scene and then asked to evaluate her own performance. I think the setting is very creative and interesting but I'm just trying to understand if it can provide an objective benchmark since how can we provide any promise that the self-evaluation is reliable?

The amount of tasks we received this round is tremendous.  With so many tasks to review we have decided to extend the review period to Tuesday, June 29.
Reviewers: if the submitter has made revisions please be sure to notify me with a formal ”accept” or ”does not meet criteria” by Tuesday, June 29  (if you haven't already done so).
this is an automated message

Hi @TalSchuster, I'm sorry I noticed I missed this question of yours. I'm not fully sure of the answer to that question but I'm hopeful that if the models are prompted in the chat style and if they have gone through many conversations, they might be able to make that judgement eventually.

@TalSchuster @a-dai given the time constraints, could you please mark this PR with a formal accept if you think it is promising. I would be happy to discuss more sophisticated versions with @TalSchuster separately too :)

@kaustubhdhole  Hello! I'm the meta-reviewer for this task.
I really like this task.  I've added a couple of nits in the task.py, and have some other small suggestions that I think should be included before merging.  Full review below:
Correctness: Everything is mostly formatted correctly, modulo the presence of 2 possibly superfluous model interactions and one typo.  I've added a comment in the task.py about this.
Formatting: Looks great! (but please do check the things mentioned in "Correctness")
Specificity: The scope of the task is suitably narrow.
Thoroughness: While the number of examples is somewhat low, I think the extended nature of the interaction is more than enough to capture the spirit of the task.  I think that relying only on the tasks themselves to score the outputs is insufficient as a long term diagnostic, though--could you add BLEU or ROUGE scoring to the outputs in the score dict, just as a coarse measure of output quality?
Difficulty: The task is extremely difficult, but also difficult to score properly.  This will probably require human evaluation to get a "real" score, but another proxy could suffice in the mean time.
Not solvable by memorizing the Internet: This task is not really solvable by memorizing, by its nature.
Novelty: This task is similar in spirit to a handful of other tasks mentioned by the authors, but is different enough that I think it warrants inclusion.
Justification: The authors do a good job justifying the importance of this task in the readme.
Data sourcing: Could the authors please put somewhere in the readme where these examples came from?  If they were all from the author, that's fine, we just want to document data sources.

@cdfreeman-google thank you very much for your suggestions! Please take a look!

Almost there! Looks like there's a failure when the dummy model evaluates your task.  Could you take a look at the failing test?

@cdfreeman-google, fixed the test

added one missing keyword--looks great, merging shortly!


fix: updated regExptext instead of stringmatch and few anti pattern

Issue #105 📐

I abi beniyal have worked for #105

Guidelines 🔐
I accept the fact that I have followed the guidelines and have not copied the codes from around the internet

[x] Contribution Guidelines
[x] Code of Conduct

Issue to be closed 🛅

My pull request closes #105

Screenshots 📷
Here are the pictures of changes that I have made 🔽

yeah sure,
I Have used regExptest() instead of stringmatch() in 1 file

I have used strict equals instead of loose equals in 2 files
I Have used regular expression literal instead of the 'RegExp' constructor In 1 file

I have used optional chain expression instead of logical and in 1 file

I have changed local variable named which shadowed variables in outer scope in 4 files

Good, merging it to main

thank you, will continue to contribute. can you please add the hacktoberfest-accepted label too?

Yes, I completely forgot it. Was preping for an interview, very busy.


Regenerate Implementation Layer with Interface Scoping Changes

See #17368 for instructions.

This does not apply to Event Hubs as it does not use RestProxy.


Fix a11y issue with Filter button (#603)

Fix the issue of the Filter button not being read by screen readers on small screens

closes #608


Request to share dataset

Thanks for the great work!!
Could you please share your dateset?
Thanks,
Harsh

Hello,  the dataset we used is from a private research project, so I can't share it with you currently. You can run the code on other open-source datasets, just like this.


#3527 - Investigate errors in SFAS Bridge File load and remediate as required.

Investigated the issue with the mentioned lines
Reproduced the issue in the local dev environment
Modified log improvement to make the messages clear
Fixed column numbers for the substring method for affected fields
Added three fields to class SFASIndividualRecord to better reflect the data for INDIVIDUAL DATA RECORDS in SFAS to SIMS Bridge file. The fields are not saved to the database.

pdStatusDate
supplierNumber
siteNumber

Added E2E tests to ensure the data import has the expected behaviors.

Successful processing locally of data import for file "SFAS-TO-SIMS-2024JUL09-110701.TXT" in the story
{
  "jobData": {},
  "returnValue": [
    {
      "summary": [
        "Processing file SFAS-Receive/SFAS-TO-SIMS-2024JUL09-110701.TXT.",
        "File contains 48077 records."
      ],
      "success": true
    },
    {
      "summary": [
        "Updating student ids for SFAS individuals.",
        "Student ids updated.",
        "Updating and inserting new disbursement overaward balances from sfas to disbursement overawards table.",
        "New disbursement overaward balances inserted to disbursement overawards table.",
        "Inserting student restrictions from SFAS restrictions data.",
        "Inserted student restrictions from SFAS restrictions data."
      ],
      "success": true
    }
  ]
}

Screenshots of some possible errors during data import

Screenshot of the added E2E tests

Good job @lewischen-aot with the investigation and the fix. Thanks for adding the E2E test as well.


Added the Scroll to Top Button

I've added the scroll to top button on the bottom-right corner of the page.

Resolves the issues - #31

Image for reference:

Thank you for accepting the contribution. I will try to add more features in the project.
Can you please add the GSSoC'23 label for this PR?


Runway should respect getRouteKey from Eloquent model

Runway should respect the getRouteKey method on Eloquent models which defines the model key used in URLs. This is helpful in large applications where you don't want to expose incrementing IDs to the end user.
How to reproduce
Add this method to an Eloquent model and try to visit a Runway edit publish form. It uses the record ID. (Note to self: see on pod project)
public function getRouteKey()
{
    return 'uuid';
}

What I expected
I expected that the URL would use the UUID value, instead of the incrementing id.
Environment
Statamic 3.0.34 Pro
Laravel 8.18.1
PHP 7.4.3
aryehraber/statamic-captcha 1.1.0
doublethreedigital/runway 1.0.1

Will be supported in the next release.


Fixes #1431. Add the ability to add auxiliary comments in UMD wrapper.

Fixes #1431. Add the ability to add auxiliary comments in UMD wrapper.
Pulls in aditional property from output named auxiliaryComment which can either be type object or string.
If string, will append the same comment above each import style.
If object, you can set string values for commonjs, commonjs2, amd, and root. If any keys are omitted, the comment will subsequently be ignored for that wrapper.

@sokra I updated the logic to be inline this time instead, so far passes tests locally we'll see once it gets through checks.

@sokra do you know why appveyor would be failling here?

@sokra should I worry about that coverage failure?

[Current coverage][cc-pull] is 83.65%

Merging [#2113][cc-pull] into [master][cc-base-branch] will decrease coverage by 7.73%

@@             master   #2113   diff @@
=======================================
  Files           231     228      -3
  Lines          9151    8812    -339
  Methods        1665       0   -1665
  Branches       1830    1707    -123
=======================================
- Hits           8363    7372    -991
- Misses          788     894    +106
- Partials          0     546    +546

Powered by Codecov. Last updated by [36e4fd2...a12d129][cc-compare]
[cc-base-branch]: https://codecov.io/gh/webpack/webpack/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/webpack/webpack/compare/36e4fd2348381d578c43abcbd15ef39a78b5bc07...a12d12994d4d27fc70e6efd484abd3486de967ef
[cc-pull]: https://codecov.io/gh/webpack/webpack/pull/2113?src=pr

Thanks

NP 😏


Translate the docs to english

Is your feature request related to a problem? Please describe.
translate the docs to english
Describe the solution you'd like

open json file in to ~/BootstrapBlazor.Shared/Locales
update the docs under ~/BootstrapBlazor.Shared/Pages folder content

Additional context

[x] Alerts
[X] Anchors
[x] AutoCompletes
[x] Avatars
[x] Badges
[ ] BarcodeReaders
[ ] Breadcrumbs
[ ] Buttons
[ ] Calendars
[ ] Cameras
[ ] Capthcas
[ ] Cards
[ ] Carousels
[ ] CheckboxLists
[ ] Checkboxs
[ ] Circles
[ ] Collapses
[ ] Coms
[ ] Consoles

@pomeara
hi, could you join this work?

Hi, this can be perfect, I try to implement the library but it's too complicated with no English documentation. I was looking for a blazor bootstrap 4 library and as far I can found this is the most complete.

Hi, this can be perfect, I try to implement the library but it's too complicated with no English documentation. I was looking for a blazor bootstrap 4 library and as far I can found this is the most complete.

@georgeemr hi，thank you for replay. could you join us for the translate work?

@georgeemr hi，thank you for replay. could you join us for the translate work?

Hi @ArgoZhang , sure, I will fork the project and do it in my free time. English is not my primary language but I will try it.

Hi @ArgoZhang , sure, I will fork the project and do it in my free time. English is not my primary language but I will try it.

@georgeemr thank you for reply. translate work is a huge work. maybe we should redesign the docs system. do you have any suggestion?

@georgeemr thank you for reply. translate work is a huge work. maybe we should redesign the docs system. do you have any suggestion?

@ArgoZhang hey let me know when you need some English translation assistance, I am happy to help out as usual

@ArgoZhang hey let me know when you need some English translation assistance, I am happy to help out as usual

@pomeara thanks. the docs need follow this

open json file in to ~/BootstrapBlazor.Shared/Locales
update the docs under ~/BootstrapBlazor.Shared/Pages folder content

@pomeara thanks. the docs need follow this

open json file in to ~/BootstrapBlazor.Shared/Locales
update the docs under ~/BootstrapBlazor.Shared/Pages folder content

@ArgoZhang are we using .resx files for the translations, this appears to be the best practice from MS for .net apps

@ArgoZhang are we using .resx files for the translations, this appears to be the best practice from MS for .net apps

@pomeara i'm very sorry for that this project use json resource file. but it support .resx format. cause by my environment is on mac. so can't edit .resx format file. json is better

@pomeara i'm very sorry for that this project use json resource file. but it support .resx format. cause by my environment is on mac. so can't edit .resx format file. json is better

i am coming

@densen2014 great stuff, @ArgoZhang  I start a mini break for two weeks from tomorrow so I will have some time to help out then

@pomeara Nice to meet you, iam chinease, vivo en madrid , ;->

@pomeara great! @densen2014 thanks for you contribution

I can help to translate to Turkish language if you like but i need an English version for that.

@ahmetkocadogan that's great! the English version resource is ready. you can check it out from https://github.com/dotnetcore/BootstrapBlazor/blob/main/src/BootstrapBlazor/Locales/en.json

@ahmetkocadogan that's great! the English version resource is ready. you can check it out from https://github.com/dotnetcore/BootstrapBlazor/blob/main/src/BootstrapBlazor/Locales/en.json

Is that the correct link? And what is that for:
https://github.com/dotnetcore/BootstrapBlazor/blob/main/src/BootstrapBlazor.Shared/Locales/en.json
Do you need any help with translating at the moment?

@ArgoZhang Also, is there a reason why the complete code documentation is in chinese and not in english?

Is it planned to change this to english in the future?

@ArgoZhang Would be nice to know :)

@ArgoZhang Also, is there a reason why the complete code documentation is in chinese and not in english?
Is it planned to change this to english in the future?

sorry for the delayed reply. this is need help too.

@ArgoZhang  pull translations from here https://github.com/pomeara/BootstrapBlazor/commit/c287e5a3eec7ea0ec899bfee9dacb5256478a359

@ArgoZhang please get many commits from here all for the translations

@pomeara it's cool. great works. thank you very much. I merged it into my repo. later will release

@ArgoZhang please grab some more translations from the same branch

@pomeara Great! thank you very much.
BTW: delete the branch after I merged and recreating a branch is better

@ArgoZhang more translations (in a new branch)
https://github.com/pomeara/BootstrapBlazor/tree/new-translations

@pomeara great job thank you very very much
BTW: better steps

delete the branch after merged
sync repo from https://github.com/dotnetcore/BootstrapBlazor
create a new branch from main branch

@ArgoZhang please grab this branch also https://github.com/pomeara/BootstrapBlazor/tree/translations-2
i did the 3 steps you mentioned also

@pomeara Thank you very much for your work. I think maybe my expression is wrong. the branch translations-2 can't merged into main because too many conflict.
I notice that you merge the Main branch into your translations-2. it's a bad idea.
https://github.com/pomeara/BootstrapBlazor/commit/76706d78e29bc2751e66654c0b61de7c07da4790
the better step is

delete branch translatons-2
create a new branch from main (sync bootstrapblazor main branch) named 'doc-trans'
update the docs

@ArgoZhang I added you to the repo please work your magic would love to see all the translations make it into the master

@pomeara done! thank you for doing this work again. it's very cool.


Investigate npy vs csv file types

npy files are many times faster than csv and text files to load in. Speeds up the process for large data https://towardsdatascience.com/what-is-npy-files-and-why-you-should-use-them-603373c78883


f3d2a07 introduces a syntax error?

It has been a while since my last update of this theme.  It looks like f3d2a0755ef421b22b8d5f9a8db6d66e2f774ef9 introduces a regression:
powerlevel9k.zsh-theme:65: parse error near `|'

This causes the powerline-style markup of the segments to not colorize or show special characters. This issue is present in the current version as well:
powerlevel9k.zsh-theme:232: parse error near `|'

I'll follow up if I find the issue or a fix.

My guess is that the 'extension' refers to the SI prefixes and they are supposed to be strings, but are being interpreted as functions or some such thing. This patch should fix it:
diff --git a/powerlevel9k.zsh-theme b/powerlevel9k.zsh-theme
index c2ee262..f532cd9 100755
--- a/powerlevel9k.zsh-theme
+++ b/powerlevel9k.zsh-theme
@@ -229,7 +229,7 @@ function print_icon() {
 printSizeHumanReadable() {
   local size=$1
   local extension
-  extension=(B K M G T P E Z Y)
+  extension=('B' 'K' 'M' 'G' 'T' 'P' 'E' 'Z' 'Y')
   local index=1

   # if the base is not Bytes

If you are using omzsh common aliases, G is aliased to | grep and being expanded.
Moving the units to strings fixed this for me, thanks!


feat: typescript typings

Hi @ricardogobbosouza! This PR will add typescript support :)
I used types from vue-analytics
I'm new to typescript, so might be something wrong. But I tested and seems to be working!

Thanks @JoaoPedroAS51 :heart:


admin.intializeApp returns Raw server response

Operating System version: Nodejs (Alpine based) docker container, node:8.7.0-alpine
Firebase SDK version: 5.4.3
Library version: 0.8.2
Firebase Product: Auth

I am trying to initialize the connection to my Firebase application using admin.intializeApp with the credentials that were provided when downloading the private key in the service accounts tab of the Firebase console. The result of calling that function is Error: An internal error has occurred. Raw server response: "{}".
The application that utilizes the Firebase SDK is deployed to a Kubernetes cluster inside of GKE. It is behind an ClusterIP Service object which is reachable via an Ingress Object for external requests.
Our implementation of the function is as follows:
admin.initializeApp({
  credential: admin.credential.cert({
    projectId: FIREBASE_PROJECT_ID,
    clientEmail: FIREBASE_CLIENT_EMAIL,
    privateKey: FIREBASE_PRIVATE_KEY
  })
});

Where the constants are set as environment variables.

admin.initializeApp() is a local operation, and doesn't make any network calls. You will have to debug your app, and see what the underlying error is. It is likely to do with your use of environment variables. Can you show the code where you instantiate those constants?

@hiranya911 figured it out. I put quotes around the private key for the yaml file that shouldn't have been there.


run example for tvm complie tensorflow model， It's very slow

WARNING:strategy:For x86 target, NCHW layout is recommended for conv2d.
WARNING:strategy:For x86 target, NCHW layout is recommended for conv2d.
WARNING:strategy:For x86 target, NCHW layout is recommended for conv2d.
WARNING:strategy:For x86 target, NCHW layout is recommended for conv2d.
WARNING:strategy:For x86 target, NCHW layout is recommended for conv2d.
WARNING:strategy:For x86 target, NCHW layout is recommended for conv2d.
tvm:  2.159142017364502
tvm:  2.1827878952026367
tvm:  2.186274528503418
tvm:  1.9802303314208984
tvm:  2.016192674636841
tvm:  2.0197951793670654
tvm:  1.995375156402588
tvm:  2.2087066173553467
tvm:  2.593811511993408
tvm:  2.403001070022583
African elephant, Loxodonta africana (score = 0.58335)
tusker (score = 0.33901)
Indian elephant, Elephas maximus (score = 0.02391)

Please use https://discuss.tvm.apache.org/ for related topics


Profiling...

Has this profiling yielded any interesting result?

I only did a quick stack analysis when using sbt-coursier from a project with plenty of modules and dependencies. I don't remember the exact numbers, just that way too many of the now cached things from ToSbt.scala were created. The first commit addresses that.
The second commit can't hurt, it should avoid some IO. That one isn't motivated by profiling though.
Overall, that seemed to address some issues that @vil1 was seeing on a big project of his.


Move vars/main.yml to defaults/main.yml

The vars defined in the vars directory has a higher precedence than the likes of group_vars, making it difficult or impossible to work with this role. Move them to defaults so users can configure this role as necessary.

Complete. Thanks for the PR!


Relnote header fix

Deploy preview for docsdocker ready!
Built with commit a2a14a73d27652cb5f6fad8251454041f420fb16
https://deploy-preview-9869--docsdocker.netlify.com

Deploy preview for docsdocker ready!
Built with commit 3e62e982c6ccfad9e0220191c54522dfc77a9390
https://deploy-preview-9869--docsdocker.netlify.com


Introduce a flag/constructor argument to raise warning as errors instead of printing them

Currently there is no proper way to detect warnings programmatically from MorphIO except parsing stderr. Would be good to raise errors or somehow wrap those warning as Python warnings. I have no preference. Anything to catch them programmatically on Python side.

I don't think we can wrap these as python warnings. The warnings are just printed strings and we do not throw / return anything.
We can maybe add a flag to throw instead but this is a bit weird. I need to think about it

We can maybe add a flag to throw instead but this is a bit weird.

This will work for me

The warnings are just printed strings

A library writing output seems weird to me; it should be programatically handle-able.
There are only a few files that are writing to cout/cerr:
src/properties.cpp
include/morphio/section_base.tpp
src/errorMessages.cpp
src/vasc/section.cpp

Those should be cleaned up.
Note that morphio.set_maximum_warnings(int) allows one to set the errors printed; and iirc, there is a way to get a log of the errors, but I don't see it in a quick pass of the doc.


VS Code crashing

VSCode Version: 1.20.1 (1.20.1)
OS Version: Mac OS X 10.12.6 (16G29)

Steps to Reproduce:
No idea. Hard to find reproducible pattern that causes crash.
Potentially related issue

Does this issue occur when all extensions are disabled?: Yes/No
Yes
 Pastebin of error log

Please add the log to the issue that you are referring to.

@tsalinger I added the log in the pastebin link, didn't want to clutter up the github issue


menu: Warning dialog box for Reset App Settings.

Add a warning dialog for Reset App Setting in menu similar to the warning in Settings for Clear App Data.

Merged, thanks.


Available Version is empty

Available Version is empty

Please make sure you have followed these Guidelines related #23