Wait 60s (instead of 30s) for RabbitMQ to come up

Admin server gives a minute for app to start. Starting RabbitMQ is part of this process.
I have at least one case when 30 seconds were not enough for RabbitMQ to start: https://ocd.appscale.com:8080/job/Blobstore/2681 . When 30s were exceeded, controller restarted RabbitMQ, so start process was repeated. But meanwhile AdminServer's timeout was exceeded.

We may also want to configure the monit start time since by default it only waits 30 seconds for processes to start.

https://ocd.appscale.com:8080/job/Daily Build/4198


Instantiate Neural Network

Create a clean neural network

This may or may not be the next step after #110 but some research will tell


cod.27.1-2coastN missing refpts and recreational catches

This stock has an MP with other refpts than msy (Fmgt, SSB_lowerbound), which are missing from the graphs. The catch graph is also missing recreational catches.

@jofall thank you for spotting this! We are working on some upgrades and hopefully, these will fix these issues.


Remove restriction on subgroups in version catalogs for plugins

This removes a restriction on subgroups in version catalogs for plugins. Fix for libraries was already applied before but for plugins this did not work.
This means, that:
plugins {
    alias(libs.plugins.greeter)
    alias(libs.plugins.greeter.second)
}

now works.
Note that I had to move ProviderConvertible<T> to public API.
Fixes #18142

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.


envoy: removing srds from E-M

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest


Memory leak and bash block on Ubuntu

Ubuntu 15.10, installed with the script. However, after reboot, bash wouldn't load (terminal was blank) and thefuck process was cloning itself causing memory leaks (would eat up all 8gb of RAM, cause machine to freeze).

Can you post your .bashrc?

Attached below (renamed to added extension to upload).
.bashrc.txt

It's strange, can you post output of thefuck --alias?

It's unavailable. Outputs nothing and then loads up my system within seconds :(

Same problem here, also Ubuntu 15.10
Actually eval $(thefuck --alias) produces a endless recursion of process spawning in order thefuck --alias, sh /bin/zsh -ic alias, zsh -ic thefuck --alias

I couldn't reproduce ot. This is what I got:

@JaffParker: what's your bash version?
@dekuenstle: you seem to be using ZSH instead. Correct?

@scorphus my bash version is  version 4.3.42(1)-release (x86_64-pc-linux-gnu)

@JaffParker: The screenshot above is from a just-installed Ubuntu 15.10 (aka Wily Werewolf). TheFuck was installed using [install script(https://github.com/nvbn/thefuck#installation-experimental), the same procedure you used. So there seems to be something a lot different in your setup. Let's try and find out what that is. Please post the output for each of the following:
$ which thefuck

$ which python

$ pip freeze

$ python -c 'import sys; print(sys.version)'

$ python -c 'import psutil; print(psutil); print(psutil.__version__);'

$ python -c 'import six; print(six); print(six.__version__); print(six.PY2)'

$ ls -lha $(dirname $(dirname $(which python)))/lib/py*/site-packages/

Thanks!


Add task per executor slot

Allow tuning the number of dynamically allocated executors to reduce executor allocation overhead resource waste

@polarBear64 use the internal branches criteo- instead of the master branch.


Packages installed in wrong directory

I have noticed that the Temurin11 package is now installed in a different directory than the expected. By skimming through the install helper I expected the installation directory to be the following (for x64):
C:\Program FIles\Temurin
Instead packages are installed to:
C:\Program Files\Eclipse Adoptium
The latter is mentioned as default in the Eclipse Adoptium MSI installer documentation.
I also seem to recall this to have been caused by a recent upgrade of Temurin11. Could it perhaps be related to changes on the MSI installer behavior?

Hi Nick,
Indeed, the default directory is set by the MSI installer. I think it was Temurin quite some time ago, but they already changed it more than a year ago. Also see https://github.com/johanjanssen/TemurinChocolateyPackages/issues/19 where we changed the documentation for the Chocolatey package.
If you look on https://community.chocolatey.org/packages/Temurin11 it mentions:
** Note: You must use INSTALLDIR with FeatureMain. INSTALLDIR Default is $env:Programfiles\Eclipse Adoptium **
So I don't think this is a bug? Maybe the install helper could be changed a bit to make it more clear, but I guess most people don't even look at the code :).

Oh sorry about that! I thought it was intended to be installed to Temurin, that's why I opened an issue! I somehow managed to miss that its documented! Thanks for your reply!

No problem, I also had some problems with the different names (Temurin/Eclipse Adoptium) in the past, it's quite confusing :).


extract codegen & introduce optparse-applicative

introduce optparse-applicative to parse CLI input
extract codegen to simplify program

With optparse, user can get CLI helps
$ witc-exe -h instance
Usage: witc-exe instance COMMAND

  Generate code for instance (wasm)

Available commands:
  import                   Generate import code for instance (wasm)
  export                   Generate export code for instance (wasm)

@dm4  fixed

Experiment: I will say the magic words to summon the code review bot. Please excuse the construction!
flows summarize

Here is another bot that reviews changed code files in a PR. The magic words to summon it are:
flows review

Hello, I am a serverless review bot on flows.network. Here are my reviews of changed source code files in this PR.

app/Main.hs
There are no significant problems with this source code snippet.
However, I would suggest a couple of small changes:

It would be better to separate versionOption and programOptions out of the main function as separate top-level bindings.
Since there is no need to execute anything when check is called, it makes more sense for its return type to be Parser () rather than Parser (IO ()).

Here's the updated code:
module Main where

import Control.Monad
import Data.Functor
import Data.List (isSuffixOf)
import Options.Applicative
import Prettyprinter
import Prettyprinter.Render.Terminal
import System.Directory
import Wit

versionOption :: Parser (a -> a)
versionOption = infoOption "0.2.0" (long "version" <> help "Show version")

programOptions :: Parser (IO ())
programOptions =
  subparser
    ( command
        "check"
        ( info
            (check <$> optional (strArgument (metavar "FILE" <> help "Name of the thing to create")))
            (progDesc "Validate wit file")
        )
        <> command
          "instance"
          ...
      )

check :: Maybe FilePath -> IO ()
check (Just file) = checkFileWithDoneHint file
check Nothing = do
  dir <- getCurrentDirectory
  witFileList <- filter (".wit" `isSuffixOf`) <$> listDirectory dir
  mapM_ checkFileWithDoneHint witFileList

checkFileWithDoneHint :: FilePath -> IO ()
checkFileWithDoneHint file = do
  checkFile file $> ()
  putDoc $ pretty file <+> annotate (color Green) (pretty "OK") <+> line

main :: IO ()
main = do
  join $
    execParser
      ( info
          (helper <*> versionOption <*> programOptions)
          ( fullDesc
              <> progDesc "compiler for wit"
              <> header
                "witc - compiler for wit, a language for describing wasm interface types"
          )
      )

codegen :: Direction -> Side -> FilePath -> String -> IO ()
codegen d s file importName =
  parseFile file
    >>= eitherIO check0
    >>= eitherIO (putDoc . prettyFile Config {language = Rust, direction = d, side = s} importName)

The patch modifies the Main module by refactoring the command-line interface using Options.Applicative. It replaces getArgs with execParser to parse command-line arguments. The handle function is replaced with a check function that takes a Maybe FilePath argument and validates the given file or all the .wit files in the current directory in case of Nothing. Additionally, the codegen function is added to generate code for runtime or instance according to the command-line inputs.
bindings/rust/invoke-witc/src/lib.rs
Overall, the code looks good and there are no obvious security issues from the given source code snippet. However, there are some minor suggestions for improvement:

The unreachable!() macro is used twice in this code. It may be a better practice to instead return a Result type to deal with unexpected or invalid input.
Instead of hard-coding the expected version number of witc-exe as "0.2.0", it might be better to retrieve this version from a configuration file, package manifest, or other source which may make it easy and flexible to maintain and update.
Currently, the code is handling panics with a panic!() macro which simply aborts the program. This might be okay for simple utilities like this code, but for larger systems, panics may not be desirable or even catastrophic. It might be a better practice to use Rust's Result type and return an error message rather than panic.
There are no comments in the code, it may be helpful to add comments or clear function names to provide additional documentation to make it easier for other developers to understand the code.

The patch updates the arguments to the Command::new method in the check_version function. It changes the argument from version to --version in order to correctly pass the version command to the witc-exe application.
package.yaml
This is a valid package.yaml file defining metadata for a Haskell package, so there are no glaring issues with the file. However, I can provide some general comments:

It is good to see a clear definition of the package's dependencies and extensions it uses.
It is also helpful to have specified ghc-options that are reasonably strict, especially Wall and compat.
It is good practice to add tests to the package, which is done in this file. hspec is a good testing library for Haskell.
We can also see that optparse-applicative is added to the dependencies list, which suggests that the package might accept command-line arguments.

The patch added optparse-applicative to the dependencies list and modified ghc-options by adding -Wall. The -Wall option turns on a number of warnings for the compiler. The addition of optparse-applicative suggests that the package may now accept command-line arguments.
witc.cabal
This code snippet provides a cabal file for building a Haskell application witc. The dependency optparse-applicative is introduced for command line parsing.
I cannot find any potential problems with this code snippet. However, it is important to note that reviewing only the cabal file is not enough to ensure the code's correctness, security, or maintainability. The actual implementation of the dependencies and the application itself must be reviewed carefully.
The patch introduces a new dependency, optparse-applicative, and adds it to the library, executable, and test-suite sections of the cabal file.


Update CONTRIBUTORS.md [rakhiazfa]

Hey there,
Thanks for your PR & contribution :wave:!
Don't forget to join follow me, join our organization, & follow our organization to stay updated with the latest developments 💻
Thank you, and have a good day ahead 😊


Without PC, but android device only through termux

This is just my stupid question, is it possible to dump keys without ADB from PC, but use the android device itself with termux?

I don't think so. You must use a Frida client so a computer is required


Broken link update to PNG and SVG icons

Updating the links from the Iconography page.
The error was caused by the change in Google Drive a few months ago when all the reference links were modified automatically .

great — thanks @leolohmann — looks like that passed, looking forward to getting the rest of the updates so we can close out #1011, #1014, and #1023!


374 txa segment

add support for TXA segments

@jgrant22  check slack.

Merging to move forward.  Several TODOs remain.


bug fix in e2e.go unit test

just realize I made a mistake in unit test.

Has Travis been failing? Is Travis even running this test?

Nope... The test condition was reversed...

LGTM


Improve computation of maxTick

Addressing the y-axis issue: https://trello.com/c/82IW2Lvm/323-poor-y-axis-range-when-graph-barely-enters-critical-range
If the highest y coordinate of the dataset surpassed upper limit (in this case 20%) of the High level, the maxTick would be set to that Y, rather than 1.5 x the upper limit.  This is why maxTick went from 35% to 21% as soon as the data passed 20%.

@pnavarrc Can you verify this when you get back?  Thanks!

 Poor y-axis range when graph barely enters critical range.


[Feature] Implement ensemble reduce

Description
This implements a reduction module for use with ensembles.
Motivation and Context
This relies on #1359 to make progress on https://github.com/pytorch/rl/issues/1344.

[x] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes
What types of changes does your code introduce? Remove all that do not apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of examples)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[x] I have read the CONTRIBUTION guide (required)
[x] My change requires a change to the documentation.
[x] I have updated the tests accordingly (required for a bug fix or a new feature).
[x] I have updated the documentation accordingly.

@smorad Friendly ping about this :)


Add A Xcode Image plugin

Please accept my request.

Can you please provide a better description for the new link?

victorgama, I added more information. thanks

Thank you!


/usr/bin/env: ‘bash\r’: No such file or directory

i am trying to run
docker-compose -f docker-compose-CeleryExecutor.yml up
docker-compose -f docker-compose-LocalExecutor.yml up
in both case i am getting following errror and container are restarting for 1. worker 2. scheduler 3. flower 4.webserver
any solution to solve this and make it run without issues

I had the same problem and solved it using the bash command dos2unix to convert the entrypoint.sh file to unix and it worked
dos2unix myscript.sh
Hope it helps


Need 'good' tertiary added to Makefile

@adamnovak @cmarkello Assuming a bam exists from the secondary steps in the Makefile can you cook up a 'good' (could be GATK...) to generate a vcf? The test file is ecoli just to run fast but will switch over to the chr20 that trevor pointed to so if you want to start with that bam on your way to a vcf that works. I can add in the details on reference download if we need it patterned after the treehouse makefile.

I'm going to try and paste in a cheap and cheerful freebayes snippet to call variants as a 'sub-good' start.

I have mapping to hg38 and variant calling ceremony hooked up and running, but, after 20 hours freebayes is still running on the test sample. @adamnovak @cmarkello @rlorigro or anyone else have any knowledge of using freebayes on long reads like this? Or suggestions on an alternative variant caller? Freebayes is single threaded but should be faster then this - at least that is my albeit very limited experience using it on short reads (DNA and RNA).

With the latest tooling I've been running freebayes for ~20+ hours o this chr20 sample. Tailing the VCF being generated its up to 266124 (assuming its going sequentially as the bam is sorted).  That's very very roughly 266124/64444167 or .4% of the way through so something must be seriously wrong. The single core is at 100% and using 1.9G memory and the VCF is up to 28M at this point so I strongly suspect user error. Anyone else have experience with freebayes on long reads mapped with minimap against HG38?

For convenience here is the invocation:
Calling variants against hg38...
docker run -it --rm --cpus="64" -v `realpath data/test`:/data \
        -v `realpath data/references`:/references \
        --user=`id -u`:`id -g` \
        quay.io/ucsc_cgl/freebayes@sha256:b467edda4f92f22f0dc21e54e69e18bfd6dcc5cbe3292e108429e2d86034e6e5 \
        --fasta-reference /references/hg38.fa \
        --vcf /data/test.hg38.vcf \
        /data/test.minimap2_hg38_sorted.bam

Clairvoyante seems to be running well and fast. Closing until we all talk in person about what other options that might involve VG, or options around benchmarking Clairvoyante and focusing on finding things that the short read exome sequencing @ UCLA might have missed that nanopore/longer reads might elucidate.


Apex Pro Tkl Wireless

i tried to use your software on
TK Pro Wireless
with
Zorin ( Ubuntu fork )
kadir@kadir-GE76-Raider-11UH:~/repo/apex-tux$ sudo target/release/apex-tux
Error: No supported SteelSeries device found!
kadir@kadir-GE76-Raider-11UH:~/repo/apex-tux$ cat /etc/udev/rules.d/97-steelseries.rules
SUBSYSTEM=="input", GROUP="input", MODE="0666"
SUBSYSTEM=="usb", ATTRS{idVendor}=="1038", ATTRS{idProduct}=="1630", MODE="0666", GROUP="plugdev"
KERNEL=="hidraw*", ATTRS{idVendor}=="1038", ATTRS{idProduct}=="1630", MODE="0666", GROUP="plugdev"
kadir@kadir-GE76-Raider-11UH:~/repo/apex-tux$ lsusb
Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 003 Device 005: ID 1038:1122 SteelSeries ApS SteelSeries KLC
Bus 003 Device 004: ID 1038:1630 SteelSeries ApS Apex Pro TKL Wireless
Bus 003 Device 011: ID 1e0e:9001 Qualcomm / Option
Bus 003 Device 013: ID 1038:172b SteelSeries ApS SteelSeries Rival 650 Wireless
Bus 003 Device 008: ID 8087:0032 Intel Corp.
Bus 003 Device 007: ID 5986:1160 Acer, Inc Integrated Camera
Bus 003 Device 006: ID 1038:1134 SteelSeries ApS SteelSeries ALC
Bus 003 Device 002: ID 1038:1726 SteelSeries ApS SteelSeries Rival 650 Wireless
Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Whats wrong ?

Hi, this is "normal", as we do not have every steelseries' keyboard, there are some that are not recognized by the the software, the list of recognized keyboard is stored in apex-hardware/src/usb.rs, currently, only 5 should work, I will do a pull request adding the apex pro tkl wireless (but won't be added now, I don't know when not-jan will be available)

If you want to try it out, you can download my pull request here and compile it again, it should "just work", but since I'm not seeing any other wireless keyboard in the code, I can't promise anything

okay btw , when i connect cable i see 2 device
kadir@kadir-GE76-Raider-11UH:~/repo$ lsusb | grep -E Wireless
Bus 003 Device 004: ID 1038:1630 SteelSeries ApS Apex Pro TKL Wireless
Bus 003 Device 014: ID 1038:1632 SteelSeries ApS Apex Pro TKL Wireless
one is cable connected  other one is wireless.
I will check it out now with new

okay btw , when i connect cable i see 2 device
oh, I don't know how it's going to handle being plugged, I hope it just works, but since it makes another interface I'm not sure

kadir@kadir-GE76-Raider-11UH:~/repo/apex-tux$ sudo target/release/apex-tux
Error: hidapi error: hid_error is not implemented yet

not i got this error

can you try running it as root? it seems like a udev issue

yeah i am running it as root.
i m not sure my udev is true
i have tried this:
kadir@kadir-GE76-Raider-11UH:~/repo/apex-tux$ sudo cat /etc/udev/rules.d/97-steelseries.rules
SUBSYSTEM=="input", GROUP="input", MODE="0666"
SUBSYSTEM=="usb", ATTRS{idVendor}=="1038", ATTRS{idProduct}=="1630", MODE="0666", GROUP="plugdev"
KERNEL=="hidraw*", ATTRS{idVendor}=="1038", ATTRS{idProduct}=="1630", MODE="0666", GROUP="plugdev"
SUBSYSTEM=="usb", ATTRS{idVendor}=="1038", ATTRS{idProduct}=="1632", MODE="0666", GROUP="plugdev"
KERNEL=="hidraw*", ATTRS{idVendor}=="1038", ATTRS{idProduct}=="1632", MODE="0666", GROUP="plugdev"

if you're already running as root, udev rules don't matter, I'm perplexed

by chance, are you running openrgb?

i have sim7600 module connected to my laptop also
i have msi ge76 raider 11th intel gen with rtx 3080

thank you for your clarification, i'm running an AMD system, but I don't think there should be any difference in how usb is handled by linux, that main difference that i can think of is the wireless part.
can you try to edit the line in apex-hardware/src/usb.rs#21 from
    ApexProTKLWireless = 0x1630,

to
    ApexProTKLWireless = 0x1632,

then recompile, and try it in root when your keyboard is plugged in (to see if, when wired, apex-tux can recognize it)

kadir@kadir-GE76-Raider-11UH:~/repo/apex-tux$ sudo target/release/apex-tux
Error: hidapi error: hid_error is not implemented yet

https://hackaday.io/project/181693-usbshareexe/log/197965-trying-rust

The thing i want is , i also want to watch battery status what i do on windows also.
Well i cant even pair now i dont know what to do

Depending on how the wireless is implemented this project may or may not work. I wouldn't hold my breath. Battery status for example is not implemented at all (and likely won't be unless one of the devs gets their hands on one of those keyboards).

do you know how to sniff pcie data on windows ?
if yes , i can do it on other pc and get the required communication data on usb.

Hi, I'm hitting the Error: hidapi error: hid_error is not implemented yet issue with my Apex Pro TKL Wireless. I tried all combinations of wired or wireless mode, a regular user or root, 0x1630 or 0x1632.
Anything I can do to help out? I'm up for a live debugging session over zoom (or any other conferencing option) if that would help...

I tried to troubleshoot this myself and i've found out that the problem is at
usb.rs:50:                 let handle = device.open_device(&api)?;
hidapi::lib.rs:498:   hidapi.open_path(self.path.as_c_str()) // note: self.path here is 3-1:1.1
hidapi::lib.rs:259:   let device = unsafe { ffi::hid_open_path(device_path.as_ptr()) };
hidapi::hid.c:1025: res = libusb_open(usb_dev, &dev->device_handle); // note: this returns -3

tldr: the libusb_open(...) call returns this error:
      /** Access denied (insufficient permissions) */
        LIBUSB_ERROR_ACCESS = -3,

I've tried chmod 666 /dev/hidraw*, and i've tried disabling selinux (i'm using Fedora 39), but none of that helped and neither did running apex-tux as root.

Hi, sorry for responding late, I haven't touched apex-tux for quite some time.
It seems weird, are sure no other software is using the keyboard? (openrgb, for example)
I'm wondering if the wireless keyboard uses an alternate interface?
could you try lsusb -d 1038: -v?
my apex 5 gives me :
Bus 001 Device 003: ID 1038:161c SteelSeries ApS SteelSeries Apex 5
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass            0 [unknown]
  bDeviceSubClass         0 [unknown]
  bDeviceProtocol         0
  bMaxPacketSize0        64
  idVendor           0x1038 SteelSeries ApS
  idProduct          0x161c SteelSeries Apex 5
  bcdDevice            2.01
  iManufacturer           1 SteelSeries
  iProduct                2 SteelSeries Apex 5
  iSerial                 0
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength       0x0086
    bNumInterfaces          5
    bConfigurationValue     1
    iConfiguration          0
    bmAttributes         0xa0
      (Bus Powered)
      Remote Wakeup
    MaxPower              500mA
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      65
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0008  1x 8 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      37
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        2
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      43
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x83  EP 3 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0020  1x 32 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        3
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength     106
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x84  EP 4 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0006  1x 6 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        4
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      21
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x85  EP 5 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
Device Status:     0x0000
  (Bus Powered)

 lsusb -d 1038: -v

Bus 001 Device 007: ID 1038:1630 SteelSeries ApS [unknown]
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass            0 [unknown]
  bDeviceSubClass         0 [unknown]
  bDeviceProtocol         0
  bMaxPacketSize0        64
  idVendor           0x1038 SteelSeries ApS
  idProduct          0x1630 [unknown]
  bcdDevice            3.d4
  iManufacturer           1 SteelSeries
  iProduct                2 Apex Pro TKL Wireless
  iSerial                 0
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength       0x009f
    bNumInterfaces          6
    bConfigurationValue     1
    iConfiguration          3 SteelSeries Config
    bmAttributes         0xa0
      (Bus Powered)
      Remote Wakeup
    MaxPower              100mA
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      2 Mouse
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      98
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      65
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        2
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      25
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x83  EP 3 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        3
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      37
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x84  EP 4 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        4
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      0 [unknown]
      bInterfaceProtocol      0
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      21
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x85  EP 5 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        5
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.11
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      43
          Report Descriptors:
            ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x86  EP 6 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
Device Status:     0x0000
  (Bus Powered)

Well this is weird, the second interface is a keyboard while the first is a keyboard, I guess they share the same dongle for both keyboard and mouse?
if that's the case, it will be way more complicated as it probably mean a new protocol that we don't know and that apex-tux doesn't support, to know we'd need packet capture (with a virtual machine), but I don't know how to do one

One thing you can try (but I wouldn't even place a bet on it), is change the line 44~ in usb.rs
device.interface_number() == 1

and change the 1 to values between 2 and 5

if you want I can try to come on zoom or another site but I'm french so english is not my main language

https://broadcom.zoom.us/j/93010799463?pwd=N0FrdllWdDcvcUJXL29XQzRjcGZ6UT09

Quick recap :

changing the interface does nothing
darkprokoba will try to get a usb capture and upload it

Hi, everyone. Thanks for the effort :) I think the easiest way would be for @darkprokoba to use the official SteelSeries software on Windows or Mac and use something like Wireshark to capture the USB traffic. Please include a step-by-step of what you did in the SteelSeries app so we can make sense of the data more easily. Please also include any images of GIFs so we can find them in the data stream.

Hi, @not-jan
Here are the steps i took.

Booted a win10 kvm domain, with steelseries gg running.
Started the wireshark capture.
Configured passthru for 0x1038, 0x1630 (a.k.a. usb redirection)
Waited for 15 seconds, this caused my keyboard lighting to change.
Unconfigured passhtru
Stopped wireshark

I have a color profile for all-white static backlight. I will attach the image i'm testing with.

And here's the capture: https://prokoba.com/apex_pro_tkl_wireless.pcapng
One thing bothers me though, apex-tux fails to open the hid raw device. I.e. it never gets to speaking to the device. Perhaps this is the first thing we need to address before looking for any protocol changes...

BTW if you can recommend a set of steps to perform in the official windows software, I can provide another capture.


Test count bars should line up with the middle of the line chart points

Right now they are anchored to the beginning of the bars

We discussed this and it seems the solutions are:

put space between the y-axis and the start of the x-axis. This keeps the y-axis of the main chart and the count chart aligned. Or
offset the bars to line up with the points. This makes the y-axis of the count chart not line up with the y-axis of the main chart.

I think both are less visually appealing than having the test count bars not centered with the points in line charts. If I had to choose, I'd go with option 1.


Add a command to allow mods to turn off/on alerts for mods/broadcaster

Is your feature request related to a problem? Please describe.
As has become apparent, there have been a lot of follow bot attempts in recent times. This issue is to discuss the feasability of adding a chat command which would suspend or enable showing the overlays.
Describe the solution you'd like
Something like submitting !overlaysoff of safemode on/safemode off, which would hold internally a boolean property that if false, would not continue processing the sending of events to display or play the alert/associated sound effects. In the case of being bot attacked, this would take the pressure of the streamer turning overlays on/off in OBS and giving moderators the ability to do that, as they will potentially also be the ones setting the channel into slow/follower only modes

I really like this. I had planned to do something similar in past bots. 😄
The alerts only show if we emit a message of the event. So I'm thinking this logic should be in the hub logic. Keeping a boolean (or booleans) that we update based on events.
Another thing to consider is that we don't necessarily want to suppress everything. We may want to only disable follow notifications. Or perhaps follows & chat messages. I don't imagine we'll want to suppress donations/subs/cheers.
So maybe !follow on / off and !chat on / off? or a multi-word command like !mod follow on/off or !mod chat on/off.
Another thing to consider is that in the future we will be recording events in Fauna so we can build the credits at the end of the stream. We'll probably want to pause recording those events as well.
Thoughts?

What about having a timeout similar to what you implemented for the Twitch Themer? I can see mods/broadcaster forgetting to turn alerts back on after turning them off. If we know how long the alert sfx is and the number of bot follows there were, we can reasonably estimate how long the alerts will occur and timeout the sfx accordingly. Just a thought... 🤔


Stale errors in watch mode

Hello everyone!
I'm not quite sure that this is really a bug or if I am missing a simple configuration option but I googled for 1.5 hours and came up with nothing:
TypeScript Version:  3.0.0-dev.20180619
Search Terms:  Stale error, watch mode, saved file, closed file
Code
File a.ts:
import test from './b';

test(4, 5);

File b.ts:
function test(x: number, y: number) {
  return x + y / 5;
}

export default test;

tsconfig.json:
{
  "compilerOptions": {
    "module": "commonjs",
    "noEmit": true,
    "strict": true,
    "watch": true
  }
}

Expected behavior:
When running tsc on console, everything is fine. When one of the parameter types in b.ts is changed to string, the running compiler (in watch mode) should now show an error, becuase test is called with 2 numbers.
Actual behavior:
Errors only show up when compiler is restarted or when a.ts is saved.
Playground Link: https://github.com/andreasgruenh/typescript-watch
Related Issues: #21325 sounds similar but was fixed according to comments.

I am not seeing this on typescript@next. do you get the stale errors consistently? what version are you using?

I'm using the version that was installed with @next this morning (as stated in the initial post). Don't know if this matters, but I'm on Windows 10 and it doesn't matter what program I use to save files (vscode/np++/notepad)
When I omit the noEmit flag, the files are getting updated, but the error is not.

I think there is something else going on.. here is what i am seeing...

do you have other @types in your node_modules?

This is really weird... This is what I'm seeing:

Note that the timestamps in the watch output get updated.
You can also see, that the project is empty.
One addition: I addded ./node_modules/.bin to my path, so the tsc call from console starts the cli from inside node_modules

Oh wow, update: When updating the second parameter, it is working. When updating the first, it's not.


WIP: Make using pagination for GitLab API requests automated

This should fix https://github.com/egnyte/gitlabform/issues/37.

Closing as #37 has been closed.


Dami/hex no std

The protocol-ink poseidon contract could not compile successfully due to std feature issues. This is a slight change to enable the poseidon contract compile successfully.

Tests and CI need to be updated/fixed @dharjeezy

Closing this because it has old commits. New PR here https://github.com/webb-tools/arkworks-gadgets/pull/216


Initial Nest.js commit

Added Nest.js starter project respecting current functionality.
This will help to build a complex backend while respecting beautiful code standards and design patterns.

Thank you but I'm not currently looking to extend the existing application.

Thank you but I'm not currently looking to extend the existing application.


Add toString, hashCode and equals to plugin annotation

Generate toString, hashcode and equals for classes of the annotation plugin.

Would be great if we could optimise toString to only print out activated properties.


Introduce a workflow for running E2E tests against podman nightly

Install necessary dependencies for running playwright podman desktop e2e tests, checkouts repository and build podman desktop. Downloads and installs podman nightly, initialize and start it in a rootless mode. Finally the playwright e2e tests are being run.

@adrianriobo works on my branch, although it is encounting https://github.com/containers/podman-desktop-e2e/issues/32.


Allow configuration of Sentry log level

Implements the feature of #304
Because Logger has explicit permitted macro levels, I’ve put in some validation.
We’re currently experiencing periodic spike protection and the Sentry logging is also causing some spiking on our log service, too (and because of the ?\n in the log, each Sentry send failure is taking up three log lines). I’d like to handle such logging through after_send_event.

Hopefully we can get a release soon that includes this so that I can start silencing some of our noisier errors on a spike and reporting the errors we care about (we’re getting a number of JSON encoding errors because of tuples in our data that we can’t quite localize).

@halostatue just published 7.0.6 to Hex 🙂


Search issues

I'm noticing a few issues with search:
First, it looks like the Gruntfile is indexing both draft and published content. Ideally it would only index content if draft:false.

Secondly, it's not generating URLs properly:
https://nychealth.github.io/eh-dataportal/data-explorer/economic conditions
Two issues here:

It's generating eh-dataportal when it needs EH-dataportal (and I'm not seeing where in the code this case issue is coming up).
And, per the example above, it's generating the URL with a space in it instead of a hyphen.

Fixed in PRs from hotfix-search-issues branch.

Thanks!
It looks like you hardcoded the site root, so we'd have to change that on deployment to another environment.
Anything else we should keep track of for later?

I can probably figure out a way parameterize it using the path node module.

@mmontesanonyc Used that same Hugo pattern of setting in the template and using in the JS. All we'll have to do is change the baseURL in the default config.toml.

Oh nice, thanks.
Was the problem with case-sensitivity what you thought it was - git indexing, requiring deleting and restoring some of the files?

Nope. We were explicitly converting the href to lower case, which worked because the old repo name was lower case. So I had to refactor the conversion. The economic conditions issue was because that folder had a space in it.

From: M. Montesano @.>
Sent: Wednesday, September 7, 2022 8:12 AM
To: nychealth/EH-dataportal @.>
Cc: Chris Gettings @.>; State change @.>
Subject: [EXTERNAL] Re: [nychealth/EH-dataportal] Search issues (Issue #77)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.  Forward suspect email to<EMAIL_ADDRESS>as an attachment (Click the More button, then forward as attachment).
Oh nice, thanks.
Was the problem with case-sensitivity what you thought it was - git indexing, requiring deleting and restoring some of the files?
—
Reply to this email directly, view it on GitHubhttps://github.com/nychealth/EH-dataportal/issues/77#issuecomment-1239308256, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADAZMQG44JHO6BSNKBVZK4DV5CBCTANCNFSM57S6AOCA.
You are receiving this because you modified the open/close state.Message ID: @.***>
Sent from the New York City Department of Health & Mental Hygiene. This email and any files transmitted with it may contain confidential information and are intended solely for the use of the individual or entity to whom they are addressed. This footnote also confirms that this email message has been swept for the presence of computer viruses.

Re-opening: build to staging assembled search results with /EH-dataportal path instead of /IndicatorPublic/beta

@cgettings , good to close?


feat: wrap up part 1

improve code organization
add actions and tests
install pytest-watch to run tests continuously
test quit event
make test quit event pass
test no key match does nothing
pass test no key match does nothing
test up arrow moves up
pass test up arrow moves up
refactor previous movement tests to a parametrized test
add parameter to test down movement
pass test down movement
add rest of movements and tests
rename test and parameters to use as a generic action tester
pass escape action test
handle movement in actual game

Codecov Report

Merging #22 (a3d7b02) into main (ee7ffd6) will decrease coverage by 35.08%.
The diff coverage is 62.26%.

@@             Coverage Diff              @@
##              main      #22       +/-   ##
============================================
- Coverage   100.00%   64.91%   -35.09%
============================================
  Files            1        3        +2
  Lines            4       57       +53
============================================
+ Hits             4       37       +33
- Misses           0       20       +20

Impacted Files
Coverage Δ

...eliketcodtutorial2021/rogueliketcodtutorial2021.py
28.57% <16.66%> (-71.43%)
:arrow_down:

rogueliketcodtutorial2021/actions.py
100.00% <100.00%> (ø)

rogueliketcodtutorial2021/input_handlers.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Can't run step definition.js file after new update

I honestly feel really about doing this,  but the problem in the new release does not appear to be resolved

I updated the dependency with the new release.  I opened cypress runner and and ran the test file

Expected Result
E2E test with BDD

However this is what I got, watch this video
Screen shot  of  testing

Versions

Cypress version: 7.4.0
Preprocessor version:  Mac Big Sur
Node version: 14.17.1

Try putting the shoppingCart.js file into integration/BDD/shoppingCart/shoppingCart.js (there is supposed to be a folder matching the feature file next to the feature file)

Hi thanks for getting back to me so quickly.  I really appreciate it.  All my files are in the same folder.

yeah that's not correct, try to do what I said please. ./integration/BDD/shoppingCart.feature ./integration/BDD/shoppingCart/shoppingCart.js . if you want to have the files in the same folder make nonGlobalStepDefinitions: false  , that will have the side-effect of having all steps available to all feature files

let me try it and let me get back to you

watch the video.  I have  misunderstood something let me know and break down step by step how I should rectify it.

@fypnlp I had to use regex instead. Did you try /^I am on the Ecommerce page$/ ? Sad that I can't use it with {string} as it shows in the documentation either.

@stevenlafl     I was able to FINALLY fix it

adding ANOTHER folder in the under BDD called shoppingCart
In the package.json I added the following

"cypress-cucumber-preprocessor": {
    "nonGlobalStepDefinitions": false,
    "stepDefinitions": "cypress/integration/BDD/"

Notice, I was forced to change the nonGlobalStepDefinitions to false
It was able to read the Feature file and start running the test .
I now have a data issues.  But this problem appears to be resolved!

Hello, can someone please help me? I am experiencing the same issue as reported above "Step Implementation Missing " I have  included the

The following in my package.json

"cypress-cucumber-preprocessor": {
"nonGlobalStepDefinitions": false,
"stepDefinitions": "cypress/e2e/BDD/"

I have added the .feature file directly into "BDD folder"
I have added a folder "repLogin" to the "BDD folder" and added step file to it

but I still get the error

@Thetestport, assuming your using @badeball/cypress-cucumber-preprocessor, you need to configure stepDefinitions with a pattern matching files, EG. cypress/e2e/BDD/*.js.

@badeball thanks for the response. i have done that and still get "step implementation missing

Then you need to provide a minimal, reproducible example that illustrates the issue.

1. Fixture file
{
"email"<EMAIL_ADDRESS>"password": "tentancles3."
}
2. PageObject
import testData from '../../../../fixtures/loginCredentials.json';
export class FaceBookLoginPage {
elements ={
emailField: ()=> cy.get('#email'),
passwordField: ()=> cy.get('#pass'),
loginButton: ()=>cy.get('#u_0_0_fV'),
}
Login(){
this.elements.emailField().type(testData.email);
this.elements.passwordField().type(testData.password);
this.elements.loginButton().click();
}
}
export const faceBookLogin = new FaceBookLoginPage();
3. Feature file
Feature: Login Functionality
User logs in to facebook
Scenario: Face Book Login
Given The User opens the url
And He Verifies the fields
When He specifies his valid credentials
Then He should be able to login successfully

4. StepDefinition
import { Given, When, Then, And } from 'cypress-cucumber-preprocessor/steps';
import { faceBookLogin } from '../../pageObjects/facebookLoginPage';
Given('The User opens the url', () => {
cy.visit(Cypress.env('externalConsoleUrl'));
});
And('He Verifies the fields', ()=>{
// Check if the register link is present
faceBookLogin.elements.emailField().should('be.visible');
faceBookLogin.elements.passwordField().should('be.visible');
faceBookLogin.elements.loginButton().should('be.visible');
})
When('He specifies his valid credentials', ()=>{
faceBookLogin.repLogin();
})
Then('He should be able to login successfully', ()=>{
// Assert URL
cy.url().should('include', 'web.facebook.com/');
})
5. cypress.config
const { defineConfig } = require("cypress");
const {addCucumberPreprocessorPlugin,} = require("@badeball/cypress-cucumber-preprocessor");
const {preprocessor,} = require("@badeball/cypress-cucumber-preprocessor/browserify");
async function setupNodeEvents(on, config) {
// This is required for the preprocessor to be able to generate JSON reports after each run, and more,
await addCucumberPreprocessorPlugin(on, config);
on("file:preprocessor", preprocessor(config));
// Make sure to return the config object as it might have been modified by the plugin.
return config;
}
module.exports = defineConfig({
viewportHeight: 800,
viewportWidth: 1280,
video: true,
LANG: 'en_US.UTF-8', // Set the desired encoding
env:{
url: "https://web.facebook.com/",
},
projectId: "",
retries:{
runMode: 1 //this will rerun failed test 1 more time
},
e2e: {
specPattern: 'cypress/e2e/**/*.feature',
setupNodeEvents,

},
});
6. package.json
{
"name": "errorreproduction",
"version": "1.0.0",
"description": "Error reproduction",
"main": "index.js",
"scripts": {
"test": "node_modules\.bin\cypress run"
},
"cypress-cucumber-preprocessor": {
"nonGlobalStepDefinitions": false,
"stepDefinitions": "cypress/e2e/BDD/*.js"
},
"keywords": [],
"author": "",
"license": "ISC",
"dependencies": {
"@badeball/cypress-cucumber-preprocessor": "^20.0.4",
"@cypress/browserify-preprocessor": "^3.0.2",
"chai": "^5.1.0",
"cypress": "^13.6.6"
},
"devDependencies": {
"@types/cypress": "^1.1.3",
"cypress-xpath": "^2.0.1"
}
}

Thanks for the response. I have reproduced my codes using a facebook login flow.

I have reproduced my codes and step missing error using a facebook login flow.

You haven't quote though. You omitted to provide file names and locations, which are crucial parts.
Given that you're importing And and doing so from cypress-cucumber-preprocessor/steps - I'm guessing you've been reading some outdated blog post. Stop doing that and instead look through any of the many examples, which all work. The package lives under a new name and no support is provided for cypress-cucumber-preprocessor. Everything you need to know about resolving "Step implementation missing" can be found in the documentation.

@badeball thanks a lot for the help. I am pretty much new to this and confused much, Nevertheless, I will go read up on the documentation you have provided

@badeball Thank you. I have read the documents in the links you provided and the error has been resolved

I have explained the steps here; please follow them, as I hope they will help. I faced the same issue and tried several other documentation sources without success. Eventually, I checked the npm packages and found that installing the related dependencies resolved the problem. Then i created medium post to help my cypress community.
npm i -D cypress cypress-cucumber-preprocessor
npm install --save-dev cypress-cucumber-preprocessor
https://medium.com/@imtejassingh/cypress-with-cucumber-in-just-5-simple-steps-8e0206f6da4f


terragrunt: 0.28.3 -> 0.28.4

Motivation for this change
Bump terragrunt to 0.28.4
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 112911 run on x86_64-linux 1

1 package built:

terragrunt

Result of nixpkgs-review pr 112911 at d9be8402 run on aarch64-linux 1

1 package built:

terragrunt

1 suggestion:

error: no-flags-array
buildFlagsArray is only intended to be used in Bash, not as a Nix attribute.
Near pkgs/applications/networking/cluster/terragrunt/default.nix:18:3:
   |
18 |   buildFlagsArray = [
   |   ^


Add a Profile picture

When the user logs in and gets into the chat arena. Give an option in the navbar of the dashboard where the user can modify his/her profile details and upload a profile picture. If not in the navbar you can also create a private route with localhost:3000/dashboard so that later we can add a button somewhere which can be redirected to the dashboard page. In addition to that, if possible the profile picture to be shown in the App bar beside the name.

Feel free to ask if the issue is not clear. And do attach the screenshot when you send a Pull Request.

I'd like to take this issue.


fix: allow newer versions of httpx

I've seen this clash with some libraries that want the newest version of httpx. The changelog is here https://github.com/encode/httpx/blob/master/CHANGELOG.md. Would it be possible to allow newer versions of httpx together with this awesome apify client? 🙏

Hi, thanks for the PR! Unfortunately we can't upgrade HTTPX at this point, so I'll have to close it without merging.
We've actually upgraded to HTTPX 0.24.0 on Friday, but then we've had to quickly downgrade back to 0.23.0 because 0.24.0 had issues parsing IPv6 addresses in the NO_PROXY environment variable, which was breaking some of our workflows. The bug is already fixed in https://github.com/encode/httpx/pull/2659, so we'll upgrade as soon as a new version of HTTPX is released, which should be hopefully soon. 🙏
In the meantime, before we release a new version, if you think that the above mentioned bug won't affect your usecase, you can use the 1.1.0b3 Apify Client version, which depends on HTTPX 0.24.0, but is otherwise the same as the latest version.
By the way, if you don't mind me asking, what are the libraries that you want to use with this client that require HTTPX 0.24? Just curious.


install script fails miserably

I'm pretty new to lua and luarocks. Any pointers why the install script fails so miserably?
root@i7:/home/jaten/491/pkg/fblualib-build.7nIf4M/fblualib/fblualib/util# luarocks  make rockspec/fbutil-0.1-1.rockspec
        cmake -E make_directory build &&
        cd build &&
        cmake -DROCKS_PREFIX=/home/jaten/491/lib/luarocks/rocks/fbutil/0.1-1 \
              -DROCKS_LUADIR=/home/jaten/491/lib/luarocks/rocks/fbutil/0.1-1/lua \
              -DROCKS_LIBDIR=/home/jaten/491/lib/luarocks/rocks/fbutil/0.1-1/lib \
              .. &&
        make

-- Configuring done
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    ROCKS_LIBDIR
    ROCKS_PREFIX

-- Build files have been written to: /home/jaten/491/pkg/fblualib-build.7nIf4M/fblualib/fblualib/util
make: *** No targets specified and no makefile found.  Stop.

Error: Build error: Failed building.
root@i7:/home/jaten/491/pkg/fblualib-build.7nIf4M/fblualib/fblualib/util#

Guys I have encountered this problem, desperately looking for a solution.
Did you solve this issue?

@monabiyan There are two solutions posted above. Why don't you try them?

@glycerine
Thank you so much for your response.
I tried, your first solution, But maybe I quite did not understand:

there is only one 'build.sh' file in "fblualib-master/fblualib" folder
Is this the only one I should fix?

You have mentioned to add the following:
-DCMAKE_INSTALL_PREFIX:PATH=/home/me/path/to/my/installation

what is really "/home/me/path/to/my/installation" ?

Where exactly add this sentence?
is this the right place to add this sentence?

Just to let you know, I am in the following location :
/Users/mohsennabian/Desktop/fblualib-master/fblualib/python
and the I put in terminal:
$luarocks make rockspec/*
And here what I get :
Mohsens-MacBook-Pro:python mohsennabian$ luarocks make rockspec/*
        cmake -E make_directory build &&
        cd build &&
        cmake -DROCKS_PREFIX=/Users/mohsennabian/torch/install/lib/luarocks/rocks/fbpython/0.1-1 \
              -DROCKS_LUADIR=/Users/mohsennabian/torch/install/lib/luarocks/rocks/fbpython/0.1-1/lua \
              -DROCKS_LIBDIR=/Users/mohsennabian/torch/install/lib/luarocks/rocks/fbpython/0.1-1/lib \
              -DPYTHON_LIBRARIES=/Library/Frameworks/Python.framework/Versions/2.7/lib/libpython2.7.dylib \
              -DPYTHON_LIBRARY=$(python-config --prefix)/lib/libpython2.7.dylib -DPYTHON_INCLUDE_DIR=$(python-config --prefix)/include/python2.7 .\
              .. &&
        make

-- Found Torch7 in /Users/mohsennabian/torch/install
-- Boost version: 1.64.0
-- Found the following Boost libraries:
--   thread
--   chrono
--   system
--   date_time
--   atomic
-- Found PythonInterp: //usr/local/bin/python2 (found suitable version "2.7.13", minimum required is "2.7.13")
-- Found PythonInterp: //usr/local/bin/python2 (found version "2.7.13")
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/mohsennabian/Desktop/fblualib-master/fblualib/python
make: *** No targets specified and no makefile found.  Stop.

Error: Build error: Failed building.

I really appreciate if you make it clear for me . <3


Can't report issue on master

I get the traceback:
Traceback (most recent call last):
  File "/home/joseph/prog/spyder.git/spyderlib/spyder.py", line 2367, in report_issue
    url.addEncodedQueryItem("body", quote(issue_template))
AttributeError: 'QUrl' object has no attribute 'addEncodedQueryItem'

addEncodedQueryItem is obsolete: http://doc.qt.io/qt-5/qurl-obsolete.html. It should still be available though, I don't know why I don't have it.
This is with Qt and PyQt v5.4.1 and python 3.4.3 on arch linux.

@goanpeca, could you investigate this given that you looked at it last time?

Oh, I assigned you because I thought that you were the Qt5 guy.

Sure i think it should be easy to solve


Adjust for SVN r299888

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC @DougGregor @bob-wilson

@swift-ci please test and merge


Usability enhancements

Make game harder to guess [#151102389]

remedial score is any score that doesn't get a star (<33)
randomly placed target must be 30% width of target from previous target

Adjust sizes of WebViews for hints [#151150980], [#151152456], [#151152800]
Reset hint counter after showing hint [#151150991]
CODAP attribute name changes

"Challenge number" -> "Challenge" [#151148412]
"Start height above ground" -> "Height" [#151148155]

Hide feedback elements that correspond to hidden sliders [#151148376]

Height above ground
Distance up ramp

@kswenson I'm not sure moving only 30% the width of the target (at a minimum) will guarantee a big enough move. If you take a look at the story, it asks for a move of 30% of the length of the ramp. Unfortunately, I was not very clear as well. I meant the ramp runout length. We have a constant ramp runout length of 5 meters, so I probably should have requested a move of at least 5*0.30 meters as a minimum move.

@ddamelin Doh! I read the PT story for the minimum target move incorrectly. Will be fixed shortly.


it show  Error in mounted hook: "ReferenceError: $ is not defined" aways in the console

你好
在console里一直显示
[Vue warn]: Error in mounted hook: "ReferenceError: $ is not defined"
found in
--->  at node_modules/vue2-datatable-component/src/HeaderSettings/index.vue
 at node_modules/vue2-datatable-component/src/index.vue
 at src/components/ListeItem.vue
 at src/components/Service.vue
 at src/App.vue

请问这个是神码问题，谢谢

https://github.com/OneWayTech/vue2-datatable/issues/44


Troubleshooting hangups

What's this PR do?
Responds with an error and Blink suppress headers to Conversations when a legacy error occurs.
How should this be reviewed?

👀

Relevant tickets
Fixes #972
Checklist

[x] Tested on staging.

Codecov Report

Merging #973 into develop will decrease coverage by 0.18%.
The diff coverage is 9.09%.

@@             Coverage Diff             @@
##           develop     #973      +/-   ##
===========================================
- Coverage    58.88%   58.69%   -0.19%
===========================================
  Files           68       68
  Lines         1547     1552       +5
===========================================
  Hits           911      911
- Misses         636      641       +5

Impacted Files
Coverage Δ

lib/conversation/replies.js
28.04% <0%> (-1.59%)
:arrow_down:

lib/helpers.js
74.85% <25%> (-1.17%)
:arrow_down:

Continue to review full report at Codecov.

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


Re #543, add Scenario class to test framework, add test for scope

Add Scenario class which will carry methods for program level statistics
Add scope statistic to ProgramPage context
Add achieved-ratio getter for indicators in a Scenario
Add a test for the three overall on-scope stats (low, high, and on-scope)
Add multi-indicator scenario for testing stats
Add ONSCOPE_MARGIN constant to Indicator model
Put all remaining ScenarioBase tests into the IndicatorDetailMixin, in the hopes of providing better abstraction of the XBase classes.
Created IndicatorStatsMixin to hold program level statistics tests

Changed my mind, I think this will work for now.


Wrap-up forgetting pending tx

Issue Number

#836
Overview

[x] I have extended core-integration DSL
[x] I have added implementation of pendingTx in Cardano.Wallet using db methods added in previous PR
[x] I have added integration tests to transaction suite
[x] I have added cli support
[x] I have added extended cli integration test
[x] I have added corresponding cli unit test

Comments

Not addressed race condition
Not all scenarios ported in CLI from API

bors r+

bors r-

We should really separate CLI option parsing from CLI execution. Then the option handling can be tested in isolation and we wouldn't need to write 2× integration tests for every scenario...

Truth is that we would still need write the integration test scenario for the CLI too. Because there will still be some integration work to test between the CLI parser and the execution logic. Yet, I agree that all the parsing tests (and that's also true for the server and all the tests for wrong methods, headers and bad arguments) could be done at a unit level. We'll get there, eventually.

bors r+


Custom text for headings

This add's the ability to set custom text to be used for headings, either for all headings or per heading when using the array of objects in the headings array.
Also fixes issue #454

Is this project abandoned? :(


Automator: update istio/pkg@master dependency in istio/istio@master

Generated by Automator - 2021-07-02T14:07:12+00:00

@istio-testing: PR needs rebase.

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.


punsafeLetPure

Should be like plet, but should not bind anything if the bound variable isn't used. It's unsafe because the semantics of the program can change unexpectedly depending on whether you use the variable or not. You should only use this if you're 100% sure that the binding will never fail. This is e.g. true for indexing PDataRecord.

If the bound variable is only used once, it should be inlined, but it's not possible to know how many times it will be used, because it could be one appearance in e.g. a pmap.


chore: Update golangci configuration

Linters we have chosen to disable because we get too many false positives:

lll (line length)
gomnd (magic numbers)

Funlen was changed from 100 -> 140 lines 50 -> 70 statements, and excludes comments.
List of added linters:

bidichk
contextcheck
errorlint
exportloopref
goheader
gomoddirectives
makezero
nilerr
noctx
tenv
wsl
govet - fieldalignment

A reference to all available linters if we want to add more : https://golangci-lint.run/usage/linters/

I get this when I try to run this locally inside this branch:
❯ golangci-lint version
golangci-lint has version 1.44.0 built from 617470f on 2022-01-25T11:22:07Z

❯ golangci-lint run
WARN [runner] The linter 'golint' is deprecated (since v1.41.0) due to: The repository of the linter has been archived by the owner.  Replaced by revive.
WARN [runner] The linter 'maligned' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.  Replaced by govet 'fieldalignment'.
library/build.go:84:1: directive `// nolint:funlen // function length inflated due to number of env vars` is unused for linter "funlen" (nolintlint)
// nolint:funlen // function length inflated due to number of env vars
^
library/service.go:541:1: directive `// nolint: funlen // ignore function length due to comments and conditionals` is unused for linter "funlen" (nolintlint)
// nolint: funlen // ignore function length due to comments and conditionals
^
library/step.go:577:1: directive `// nolint: funlen // ignore function length due to comments and conditionals` is unused for linter "funlen" (nolintlint)
// nolint: funlen // ignore function length due to comments and conditionals
^

I think we want to take care of those deprecations and also remove the unneeded nolint declarations.
I see the build is passing, but I found that it might actually be reporting incorrectly. I saw this in the logs (https://github.com/go-vela/types/runs/5116600489?check_suite_focus=true#step:4:76):
   level=error msg="Can't read config: can't read viper config: While parsing config: yaml: line 34: found unknown escape character"

I forgot that i modified the yaml prior to running this with the following change:
     exclude:
-      - "^\/\/"
+      - "^\\/\\/"

I think that would still ignore comments, but I didn't test to be honest.

Would the plan be to update all repos with this change once it's in a good place?

@kneal correct

asciicheck : simple linter to check that your code does not contain non-ASCII identifiers

seems non-critical? leaning towards no

bidichk : checks for dangerous unicode character sequences

sure

contextcheck : check the function whether use a non-inherited context

ok, i think?

depguard : checks if package imports are in a list of acceptable packages

i don't think we have a deny list, leaning towards no

gci : Gci control golang package import order and make it always deterministic

i don't know if we want to use this in particular, but i think keeping import order consistent would be nice

gocritic : Provides diagnostics that check for bugs, performance and style issues.
  Extensible without recompilation through dynamic rules.
  Dynamic rules are written declaratively with AST patterns, filters, report message and optional suggestion.

i'm good to try it

godox : Tool for detection of FIXME, TODO and other comment keywords - with this one I noticed we leave a lot of

i don't mind them personally, however i don't want to keep them forever either, so this would just add noise at the moment, i think

goerr113 : Golang linter to check the errors handling expressions - this one doesn't like when we return new errors

not sure if we're ready for this yet, but i think the premise is fine?

goheader : Checks is file header matches to pattern

this would be nice to ensure the license header is in place

gomoddirectives : Manage the use of 'replace', 'retract', and 'excludes' directives in go.mod.

don't feel strongly on this one, but might be handy in the future?

ifshort : Checks that your code uses short syntax for if-statements whenever possible

it's a stylistic thing. i guess consistency is nice. i believe we use both styles currently, but i don't generally favor one over the other myself.

ireturn : Accept Interfaces, Return Concrete Types

i think it's fine

importas : Enforces consistent import aliases

no, i feel like sometimes we want the flexibility to name imports as whatever and managing this via allow/deny lists is too painful

makezero : Finds slice declarations with non-zero initial length

fine by me

nilerr : Finds the code that returns nil even if it checks that the error is not nil.

yup

nilnil : Checks that there is no simultaneous return of nil error and an invalid value.

wonder if this would trip us up somewhere.. i'll defer

noctx : noctx finds sending http request without context.Context

sure, let's try

prealloc : Finds slice declarations that could potentially be preallocated

sure

predeclared : find code that shadows one of Go's predeclared identifiers

seems non-critical? leaning towards no

wastedassign : wastedassign finds wasted assignment statements

doesn't ineffassign and deadcode do similar?

tenv : tenv is analyzer that detects using os.Setenv instead of t.Setenv since Go1.17

sounds good to me

thelper : thelper detects golang test helpers without t.Helper() call and checks the consistency of test helpers

sounds like there false positives on this, prefer no at this point

tparallel : tparallel detects inappropriate usage of t.Parallel() method in your Go test codes

sounds like there false positives on this, prefer no at this point

Taking this updated list and testing it against a few other repos to check for any major issues

@kaymckay @wass3r With the imminent release of Go 1.18 this month, are we upgrading in a right away and if/when we do, I'd love to add linting rules for the switch to interface{} to any.
I personally think, it makes a ton of sense to just align on that change sooner than later since that will be the recommend way to write it going forward past 1.18.

@kneal i have no issue with that, though sometimes it's better to wait for that sweet .1 release :P

Very true, honestly. So, maybe we put a pin it, with an issue?

@kneal an issue would be great! i can't promise i will remember otherwise myself :)

Got you fam https://github.com/go-vela/community/issues/501

removed a few that caused too much noise or was just a style preference that we didn't seem to follow.
as for fieldalignment , I have to remove govet from tests to remove the fieldalignment calling out misordered structs...
If we don't care about this micro performance enhancement and like the order of the structs as they are, I suggest removing the fieldalignment all together or else we will get a lot of false positives.

looks like with the current config it wants a space before the return here: https://github.com/go-vela/types/blob/850d8962915ad30182dc148525fc6f100427e87e/database/build.go#L83-L84

Thanks @kaymckay - i'd be interested to hear others' thoughts on fieldalignment (previously maligned). seems overkill for tests to me. outside of that, i'm a bit on the fence. i guess it was enabled before but i don't really recall it yelling at us too much.

@kaymckay @wass3r
My thoughts on fieldalignment are we should definitely consider ignoring this linter for test files.
As far as do we want to use this linter or not in general, I'm on the fence.
On the one hand, the linter does offer performance improvements from a memory consumption perspective:
https://pkg.go.dev/golang.org/x/tools/go/analysis/passes/fieldalignment
On the other hand, I seriously question to what extent we'll see improvements (how much?) from sorting structs.
If I had to choose, I guess I'd lean towards using fieldalignment under the guise that "every little bit helps"?
If someone can provide thoughts on why not to use it, I'm all ears because my argument to use it is weak 👍
cc: @kneal

fieldalignment can't say I have strong feelings about. Personally, I've always tried to list things alphabetically or logically to how the fields are being used for sanity. Honestly, I'm not sure I have a strong preference on it. I can't tell if this affects struct field ordering or just fields when you're calling a struct.
I know some struct field ordering we likely wouldn't want change, the YAML structs are an example because the way the struct is laid out plays some role in the auto-generation of a YAML file or maybe we don't care. I don't know.

i think we can summarize that we don't feel strongly about the gains, so let's table it and skip for now :)

waiting on approval to proceed with other repos


Ghost Marketplace: Dead Themes

I was just browsing through the Ghost Marketplace, and happened upon quite a lot of themes where the links (click through) simply don't work. This appears to apply to both free and paid categories.
http://marketplace.ghost.org/themes/paid/
This one in particular I thought would be worth a look:

So you click on the link hoping to learn more and you're bounced through to a "Suspended" (https://ghost.org/suspended) page, which is the best case, in many others you are taken away from the Ghost ecosystem and hosting to a third-party which also doesn't give you a theme.
This feels like a somewhat suboptimal user experience.
I'd propose that at a minimum we should consider:

[ ] Following all the links on Ghost Marketplace to look for HTTP 404.
[ ] Where the follow-on site is hosted on Ghost, make sure it isn't suspended.
[ ] Support reporting functionality ("This doesn't work!") with an element of human review.

If a link is found which doesn't seem to work any more, or if reports come in claiming that to be the case, there should be a means to retain the entry in the catalogue for a period of time (i.e. 90 days) but have it disappear from the Ghost Marketplace. Reinstatement would occur if the original issue was fixed and without needing to go through any submission process again.

@agh thanks for reporting this. The marketplace is a massive project which requires far more than just thew few things listed here. If you want to go through an compile all the dead links for some cleanup, by all means, but beyond that I'm afraid we're not going to track this in the ghost repo as we don't have resources to work on it at the moment.


Fix typo in error message.

I was just reading the source code and noticed this minor cut-and-paste error in the error message.

Thanks!


Support piranha-callers.xml from archive

The content of a piranha-callers.xml file can now be read from a property, but not from the actual file in an archive. This should be fixed. Proposed location is WEB-INF/piranha-callers.xml.

If we create a file with this name, I think we will need to create another file to do the JNDI bindings. Maybe we can use one file merge both configurations or use multiple files?

@Thihup That's a fair point to consider. This file here is largely like Tomcat's global tomcat-users.xml. Tomcat has the JNDI bindings in /META-INF/context.xml. GlassFish however has many of these things in a single file glassfish-web.xml.
This too is one if those things that hasn't really been decided yet.


A flattened Option masks parse errors inside the Option

I’ve noticed that wrapping a field type in Option, particularly when the field is a struct that contains other fields, masks genuine parse errors. For instance, the entire struct gets marked as None rather than an error being reported.
Is there a way to mark a field as optional and ignore it only as long as the field is nonexistent or the value is null, but error out if a value is specified and it isn’t valid?

Can you provide some runnable code that reproduces this?

Yes, I believe this is the same issue I'm currently having in some more complex code:

#[derive(Debug, Deserialize)]
struct A {
    c: u64,
}

#[derive(Debug, Deserialize)]
struct B {
    #[serde(flatten)]
    a: Option<A>,
}

#[tokio::main]
pub async fn main() {
    let s = serde_json::from_str::<B>("{\"c\": \"Hello\"}").unwrap();
    println!("{:?}", s.a);
    let s = serde_json::from_str::<B>("{\"c\": \"64\"}").unwrap();
    println!("{:?}", s.a);
    let s = serde_json::from_str::<B>("{\"c\": 64}").unwrap();
    println!("{:?}", s.a);
}```

Expected output:
```None
None
Some(A { c: 64 })```

Playground link:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=dbe5c4724844ad4a652ec334b7b4b3c7

What I'd prefer to have happen in this sort of situation is to have it raise an error that 'c' is the wrong type. Since the tag names match, this seems more likely to be an inadvertent error in the type specified (or, in more complex code, parsing the field value). That being said, I'm not sure how commonly people expect / rely on the opposite behavior so I acknowledge backwards compatibility is a concern.

In my case, I have an optional sub-struct sub-struct with several fields on it, some of which are optional, and some of the fields also include custom deserialize implementations. If the sub-struct key is present, then all the required fields should be present, though the optional fields can be absent. If any field name matches one in the sub-struct, but the field value is a different type or returns an error from a custom deserialize, then it's an error. Unfortunately, what's happening is that it's simply setting the entire sub-struct to None. This has caused a lot of confusion, because I'm transcoding between serialization formats and values have been disappearing without any errors being generated. The scale of records being processed (tens of thousands or more) makes it difficult to isolate which field is triggering the parse error.

I'm not sure how to work around this apart from implementing a custom deserialize for the sub-struct field (a). Maybe I just need to basically call A::deserialize? I haven't yet started digging in to see what's happening under the hood for serde.

I've just hit this. If a maintainer agrees that parse errors should not be masked by Option, I'd be happy to try fixing it sometime.

For me it makes sense to raise an error. The serialized B should have no c property, or the c property should be a number. If it has a c that is not a u64 number, it should raise an error.
It should also raise an error if c is null IMO, because c actually comes from A, and it's not optional there (but could be undefined, because a is optional in B). So {} and { c: 10 } are valid JSONs, but not { c: null } neither { c: '10' }. To accept { c: null } the struct A should be:
struct A {
    c: Option<u64>,
}

I think the relevant code locates in serde instead of serde-json
https://github.com/serde-rs/serde/blob/d5bc546ca53be0b31984a06a8ad587cbea4ca5ce/serde/src/de/impls.rs#L919-L925
where the method __private_visit_untagged_option is only ever invoked by FlatMapDeserializer
I think the main difficulty is that due to opaque Error, there is no way for the OptionVisitor to differentiate between missing_field and other types of error. Even for missing_field, it would not be able to locate where the error occurs for nested structure.
I imagine to make errors be reported, we need:

Introduce a custom error structenum FlattenError<E> {
    NotPresent,
    Other(E),
}

with serde::de::Error implemented, and only missing_field implemented to emit NotPresent.
Introduce a wrapper to FlatMapDeserializer (Say WrappedFlatMapDeserializer), which emits FlattenError<E> as Deserializer::Error, and replace deserialize_option in FlatMapDeserializerfn deserialize_option<V>(self, visitor: V) -> Result<V::Value, Self::Error>
where
    V: Visitor<'de>,
{
    match visitor.visit_some(WrappedFlatMapDeserializer(self)) {
        Ok(value) => Ok(value),
        Err(NotPresent) => Ok(None),
        Err(Other(e)) => Err(e),
    }
}

Now we can get rid of __private_visit_untagged_option.
Introduce wrappers for FlatStructAccess so that FlattenError::Other can correctly be emitted. Also patch WrappedFlatMapDeserializer to correctly funnel errors.

I will try a fork first to see whether these ideas work.


Server does not disconnect clients on close

Should boot them to login screen. FrontEnd.returnToLogin() may need to be made public for backend to handle.

Also will probably have issues on logout trying to send close message to server that doesn't exist. Close clients before server?

I think the server is sending QUIT messages to the clients, but I might have mistakenly put System.exit(0) before that. But maybe not.

It's not that the server isn't sending the QUIT messages, it's that the clients aren't handling them.... will fix.


icewm: rename strlcat/strlcpy to avoid conflicts, patch to fix w/musl

Mostly to fix things w/musl.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: icewm
Partial log (click to expand)
shrinking /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/bin/icewm-menu-fdo
shrinking /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/bin/icewmhint
shrinking /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/bin/icewm
shrinking /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/bin/icesh
gzipping man pages under /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/bin
patching script interpreter paths in /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2
/nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2/bin/icewm-set-gnomewm: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/sff28173y1vami6630vwkaklxy0y294g-icewm-1.4.2...

@GrahamcOfBorg test sddm slim

Success on aarch64-linux (full log)
Attempted: icewm
Partial log (click to expand)
shrinking /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/bin/icesh
shrinking /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/bin/icewm
shrinking /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/bin/icehelp
shrinking /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/bin/icewm-menu-fdo
gzipping man pages under /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/share/man/
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/bin
patching script interpreter paths in /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2
/nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2/bin/icewm-set-gnomewm: interpreter directive changed from "/bin/sh" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/xyk8j2bwbggzjvj914d6r395wqyknr3a-icewm-1.4.2...

Failure on x86_64-darwin (full log)
Attempted: icewm
Partial log (click to expand)
cannot build derivation '/nix/store/n35gwc3wrdhxh3whk5cmvggm9rk9vcsx-apple-framework-QuartzCore.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/grvi8pypvgba2b0silx6nlv5m0lddnl3-cmake-3.10.2.drv': 13 dependencies couldn't be built
cannot build derivation '/nix/store/98lnkfrm8gsvj2cr279rijf0ai8l9y05-apple-framework-Carbon.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/lii6flg3718n81zg2fis8lrayn52jla0-jasper-2.0.14.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/93b7wf4kb8wp82hf1kkcb3wbpgab2mvr-cairo-1.14.10.drv': 20 dependencies couldn't be built
cannot build derivation '/nix/store/rl3sx3qhmkv9fhza6sbcimqd4rc3dcq6-absolute_gir_path.patch.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/8mzl2c7sjdhlpjkb5ig3s6vv25w0rkrb-gobject-introspection-1.56.0.drv': 12 dependencies couldn't be built
cannot build derivation '/nix/store/hhxhys8i44lprfivmns1fpzrcyawpc1v-gdk-pixbuf-2.36.7.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/f3ra6yim9y3rnpmfmykr180qdfx2akdx-icewm-1.4.2.drv': 27 dependencies couldn't be built
error: build of '/nix/store/f3ra6yim9y3rnpmfmykr180qdfx2akdx-icewm-1.4.2.drv' failed

Failure on x86_64-linux (full log)
Attempted: tests.sddm, tests.slim
Partial log (click to expand)
cannot build derivation '/nix/store/04j7yh5j0fxb8311g19dyph6lg3c5xdv-unit-display-manager.service.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/naav6zql3q7sc7bv091fqq5ashg18nmr-system-units.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/yjn0b8y8sivibysm71yhx58pwnmfqxcr-etc.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/a7cfkll0c81fmfh4s7awdahb6vl8pzwk-nixos-system-machine-18.09.git.fcf3ce3.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/akprmvhs1rfilfazaljam9m6lgyiycm2-closure-info.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/q1lls2y4565j8jvdw0rhb8qxr696wn87-run-nixos-vm.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/hdf4ibiqgdghlb166asgd7nrb6bgcc0w-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/wm30y90w40s3sp5k9b5js3ja3qs7dg4h-nixos-test-driver-slim.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/wjjidl5fd5i7rxcn2sc9l4mpjkbzd9qy-vm-test-run-slim.drv': 1 dependencies couldn't be built
error: build of '/nix/store/wjjidl5fd5i7rxcn2sc9l4mpjkbzd9qy-vm-test-run-slim.drv' failed

Failure on aarch64-linux (full log)
Attempted: tests.sddm, tests.slim
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/3qgj84zdssif2xdy1afrmsdslgy356jk-nixos-test-driver
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/3qgj84zdssif2xdy1afrmsdslgy356jk-nixos-test-driver/lib  /nix/store/3qgj84zdssif2xdy1afrmsdslgy356jk-nixos-test-driver/bin
patching script interpreter paths in /nix/store/3qgj84zdssif2xdy1afrmsdslgy356jk-nixos-test-driver
/nix/store/3qgj84zdssif2xdy1afrmsdslgy356jk-nixos-test-driver/bin/.nixos-test-driver-wrapped: interpreter directive changed from " /somewhere/perl -w" to "/nix/store/3qhzffjvl8gbiabikiqdnz4jhh9v84nz-perl-5.24.3/bin/perl -w"
checking for references to /build in /nix/store/3qgj84zdssif2xdy1afrmsdslgy356jk-nixos-test-driver...
cannot build derivation '/nix/store/w709brx8shir20rclvfs6rnygzi5kl3i-nixos-test-driver-slim.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/4ybwxp5afivxx7z051cn39xnsdx88q6f-vm-test-run-slim.drv': 1 dependencies couldn't be built
error: build of '/nix/store/4ybwxp5afivxx7z051cn39xnsdx88q6f-vm-test-run-slim.drv' failed


Issue with root access

I cannot use root for composer, if I do, composer will throw an error.  I can fix this in two ways: change chown to a non-sudo user or give all the folders a 667 permissions.
What is the best way should I take that is acceptable for deployer?

Why do you run composer under root?

I am cleaning up on issues. Fill free to reopen if needed.
Also, we have a new section available: discussions.


batch qp solve with equality constraints.

Hi, I am recently want to try to use qpth to solve some of my own problems (and combine it into optNet later).
So my qp problem include only equality constraints, no inequality constraints, and I have encounter some problems and have some questions want to ask:

Does it necessary to have ineq? Originally I tried to give a
e = Variable(torch.Tensor())
as shown in the denoising example, but it raise some errros.
I found that in the code, there is
_, nineq, nz = G.size()
neq = A.size(1) if A.ndimension() > 0 else 0
assert(neq > 0 or nineq > 0)

which suggest so.
Is it because in the ref paper (Mattingley & Boyd, 2012), it introduce a slack variable on ineq?

I am a bit confuse on how to transform my qp problem into batch form.
For example there are 1000 unknown variables to solve, among them, I have equality constraints on 100 of them (which suggest only 900 variables needed to be solve).
If I divide them into batches, e.g. batchSize = 100..
How do I set up those eq constraints (because them seems global to the problem)..
thx a lot~

Hi, at some point, I had a version that didn't require inequalities, but now I've assumed in some places that there are always inequality constraints. I'll merge in a PR adding some conditionals to support no inequalities. You might also be able to set an inequality constraint that's always satisfied like x_0 <= 1e8, but I'm not sure if this will hurt stability.
Batch form is for solving multiple problems at the same time, not splitting up a problem.
I'll keep this issue open until we've added support back for no inequality constraints.

thx for the quick reply.
I'll first try to add a dummy ineq constraints that you described!

In terms of batch form, I am still a bit not sure how to do it.
In the original form, if I have a problem with around 3k variables, and I put that qp problem with eq constaints in cvxpy, and it is solved w/o problems.
My original first goal of using qpth and want to try if I can solve this problem w/o learning.
(and I will replace some part of the qp as unknown, and trying to learn them).
After my first try (sort of directly translate the thing I used in cvxpy set up to the qpth torch variables)
But it said it requires a huge chunk of memories.... so I think I must use it in the batch mode.
As the result, I am a bit confuse if it's solving multiple problems or splitting up my original problem??
thx

The batch mode is to take advantage of parallel batch operations when training on many examples at once. It's not for splitting up a problem and making it easier to solve as you're asking. You can see some more concrete examples in the optnet repo - https://github.com/locuslab/optnet
For that many variables, you can try using our sparse solver (though as a warning, it's still under development and you may have issues that we can work through)


Build failing when upgrading from 1.18 to 1.19

Error:Execution failed for task ':app:transformClassesWithJarMergingForDebug'.
> com.android.build.transform.api.TransformException: java.util.zip.ZipException: duplicate entry: com/orhanobut/hawk/AesCbcWithIntegrity$CipherTextIvMac.class

was able to fix by running a Clean


Add CPU tester that compares against nestest logs

I've been manually comparing logs of nestest generated by nintendulator, however it will soon become unmanageable. Currently the CPU emulator creates logs identical to nintendulator and those can be compared to determine correctness (assuming nintendulator is accurate which it is AFAIK).

Completed in 1ae33ba5031f00b2e8901b4eca07b049896c173d.


How to Disable/Off Video & audio by default at start ?

Please provide simple solution ?

Thanks for the question @bhaveshs12!
I went ahead and made a demo branch to show you how to. implement this feature. You can see the changes here: https://github.com/twilio/twilio-video-app-react/compare/demo-disable-tracks-on-startup
Note: While I was working on this I found a bug in the useIsTrackEnabled hook. I made a PR for that fix, but the fix is also included in the demo branch referenced above.

Closing this issue as it appears resolved. Please let me know if you have any other questions!

Hi @timmydoza , I am using the latest version of this repo and the example that you shared above did not help me. Therefore, Could you share how can I achieve the same functionality?

Hi @bisnurana - thanks for the question!
I have a diff here that disables audio and video on load.
Note: it still requests an audio track on load, but it immediately disables the track. It does this so that the track can be enabled again when the user clicks on the Unmute Audio button. Video tracks are handled differently (the Mute Video button publishes and unpublishes them instead of enabling/diabling them), so instead of disabling the video track on load, we simply don't request one (by passing {video: false} to createLocalTracks().

Hi @timmydoza  thanks for your help. The above solution seems to disable audio/video at the start but now it doesn't create local tracks anymore when i enable audio/video later, therefore, the remote users in the room are unable to see/talk with each other.

At starting the mic is disabled but after connecting to a room when I'm trying to unmute the audio is not getting published and remote user cannot hear anything.