dask read_parquet changes disable parquet file statistics reading in default case

Changes introduced in this dask PR https://github.com/dask/dask/pull/8992, especially here and here break/change the default case logic in nvtabular for reading parquet file statistics (to effectively disable it).
This introduced a significant slow down (10-15 min) at start up (and possibly other problems) in our use case with KerasSequenceLoader and the large criteo dataset, as the entire dataset was being read in to determine the number of rows.
Work around is to pin dask version or explicitly set gather_statistics=True (or calculate_divisions=True to avoid dask deprecation warning) as argument to Dataset constructor, but would be good to have default behavior remain as before.

Thoughts on how to address this, @rjzamora? The internals of merlin.io.Dataset are outside my wheelhouse 😅

Thanks for raising this @eordentlich (and sorry for letting this regression slip by).  Dataset is in need of a general update. I should be able to work on it this week (and optimized len after read_parquet will be a top priority).


Aligment via cli issue

Its seems like after last update aligment does not working, if using via CLI. Few days before all was great

It should be fixed in 1fb30099642c15912bbd43affdfdfc8af630703b.

Great, thanks


Update Cloudinary Attachment Links to Trigger Direct Downloads

Currently Cloudinary returns URL link to attachment, which opens in a new browser tab upon clicking.  Requirement is for the attachment to be downloaded directly upon clicking the link.

Pull request #125


Build error in Xcode when creating simple AWSService instance

Describe the bug
I am getting the following error when trying to create a simple S3 AWService instance: Cannot use instance member 'awsClient' within property initializer; property initializers run before 'self' is available
To Reproduce
Steps to reproduce the behavior:

Try and create an S3 AWSService instance using the following syntax.

import UIKit
import SotoS3

class S3ViewController: UIViewController {

    let bucket = "my-bucket-abc123"

    var awsClient = AWSClient(
        credentialProvider: .static(accessKeyId: "ACCESS_KEY", secretAccessKey: "SECRET_ACCESS_KEY"),
        httpClientProvider: .createNew
    )
    let s3 = S3(client: awsClient, region: .useast1) <-- ERROR HERE

}

Expected behavior
No errors or build failures.
Setup (please complete the following information):

OS: MacOS Big Spur 11.1
Xcode version: 12.3
Version of soto: 5.0.0
Authentication mechanism: hard coded creditials

Additional context
I am very new to Swift development and might be missing something quite small. Please delete this issue if my setup has nothing to do with using sots.

That's an issue with your Swift code. You can't have one member variable dependent on another if they are initialised like that. At the point you are initialising s3 self doesn't exist so self.awsClient doesn't exist.
Unfortunately you cannot override UIViewController.init easily. Instead you could do the following
class S3ViewController: UIViewController {
    var awsClient: AWSClient!
    var s3: S3!

    override func viewDidLoad() {
        super.viewDidLoad()
        self.awsClient = AWSClient(
            credentialProvider: .static(accessKeyId: "ACCESS_KEY", secretAccessKey: "SECRET_ACCESS_KEY"),
            httpClientProvider: .createNew
        )
        self.s3 = S3(client: awsClient, region: .useast1) <-- ERROR HERE
}

I'm not overly keen on this though. I would prefer that AWSClient is separate from your S3ViewController. It should probably be in some global state. In the future you might change how you acquire your AWS credentials and this might involve network access and you don't want to be asking for credentials every time you display your S3ViewController. Also you might end up using another AWS service and want to use the AWSClient elsewhere.

That's an issue with your Swift code. You can't have one member variable dependent on another if they are initialised like that. At the point you are initialising s3 self doesn't exist so self.awsClient doesn't exist.
Unfortunately you cannot override UIViewController.init easily. Instead you could do the following
class S3ViewController: UIViewController {
    var awsClient: AWSClient!
    var s3: S3!

    override func viewDidLoad() {
        super.viewDidLoad()
        self.awsClient = AWSClient(
            credentialProvider: .static(accessKeyId: "ACCESS_KEY", secretAccessKey: "SECRET_ACCESS_KEY"),
            httpClientProvider: .createNew
        )
        self.s3 = S3(client: awsClient, region: .useast1) <-- ERROR HERE
}

I'm not overly keen on this though. I would prefer that AWSClient is separate from your S3ViewController. It should probably be in some global state. In the future you might change how you acquire your AWS credentials and this might involve network access and you don't want to be asking for credentials every time you display your S3ViewController. Also you might end up using another AWS service and want to use the AWSClient elsewhere.

Thanks, @adam-fowler for the quick response! This issue is my lack of knowledge for developing in swift so my applogize!  I planned on getting just a bare bone minimal code working before refactoring into what you described above. Let me give the above code a go and see if I can get it working properly. Feel free to close out this issue!

Thanks, @adam-fowler for the quick response! This issue is my lack of knowledge for developing in swift so my applogize!  I planned on getting just a bare bone minimal code working before refactoring into what you described above. Let me give the above code a go and see if I can get it working properly. Feel free to close out this issue!


Terra won't run on 1.16.5

im running a 1.16.5 paper server, but terra won't run on it. i tried all versions (6.1.2, 4.X.X, 5.X.X). I can;t know what to use

https://terra.polydev.org/install/versions.html
Clear the contents of the Terra folder before starting up your client / server and after upgrading / downgrading.


it seems that IE 6/7/8 also support documentFragment!

It seems that IE 6/7/8 also support documentFragment!

Note that that's the DOM Range entry you're looking at there which mentions DocumentFragment in its description, but doesn't refer to support to it.
documentFragment itself is indeed already supported everywhere and doesn't have its own entry.


project.cson is not working

The per project settings in project.cson are not loaded

Could you show how your project.cson file?

I have the same problem. I need differente tab lengths among projects but this doesn't work.
  {
    title: "Project A
    group: "Projects"
    paths: [
      "C:\\projects\\A"
    ]
    settings:
      'editor.tabLength': 2
  },
  {
    title: "Project B"
    group: "Projects"
    devMode: true
    paths: [
      "C:\\projects\\B"
    ]
    settings:
      'editor.tabLength': 4
  }
]```

I am wondering if Project Manager is not notified when atom is opened and the last project folder is automatically opened (as if atom opened the folder instead of user goes to Project Manager projects list and manually opened a project)

Hmmm, I gues there could be something overriding the settings. Are you using the editorconfig package or something similar?
The settings feature is a but bugged because of pretty crappy support for doing what it does, so it would probably need some more love and investigating what is possible.

Yes, I'm using the popular EditorConfig package among others.
I'll try to do some testing of whether disabling EditorConfig improves or fixes the project.cson behavior in Project Manager.
I wonder if others are having the same incompatibility?

Not worked for me too. I don't use EditorConfig but core per-language settings:
[
  {
    title: "Redux Actors"
    paths: [
      "/home/user/projectA"
    ]
    settings:
      'editor.tabLength': 4
      '.js.regexp.replacement.source.editor.tabLength': 4
      '.js.regexp.source.editor.tabLength': 4
      '.js.source.editor.tabLength': 4
  }
]


KYLIN-3699 SparkCubing failed when build with empty data

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 3959

24 of 34 (70.59%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.04%) to 25.847%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-job/src/main/java/org/apache/kylin/job/JoinedFlatTable.java
0
2
0.0%

source-hive/src/main/java/org/apache/kylin/source/hive/HiveInputBase.java
0
3
0.0%

source-hive/src/main/java/org/apache/kylin/source/hive/CreateFlatHiveTableStep.java
24
29
82.76%

Files with Coverage Reduction
New Missed Lines
%

source-hive/src/main/java/org/apache/kylin/source/hive/HiveInputBase.java
1
16.82%

source-hive/src/main/java/org/apache/kylin/source/hive/CreateFlatHiveTableStep.java
1
45.95%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

Totals

Change from base Build 3957:
0.04%

Covered Lines:
17857

Relevant Lines:
69087

💛  - Coveralls


add new tokens CITY / AFC / INTER

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

DONE : Create a folder under /tokens, the folder name is your token symbol in the lowercase format

DONE : Upload your logo in your folder

[ ] Filename in the uppercase format with suffix
[ ] Image format: png
[ ] Image size: 256 by 256 px

DONE : Upload a json file for additional info. File name is your token symbol in the uppercase format with suffix

asset must be your token's symbol
assetMap must be your token's symbol without suffix
market must be ALTS or FIAT. It determines which market the pair will be listed in when this token is the quote symbol.
display's default value is true, which means the information will be available in the explorer

The following contacts are supported in contact:

Twitter
Facebook
Reddit
Instagram
Medium
Steemit
Coinmarketcap
Binance Info
Youtube
Telegram
Discord
Linkedin
Github
Binance Research

Note: you must use the right key name

Example:
{
    "asset":"NOW-E68",
    "assetMap":"NOW",
    "market": "ALTS",
    "officialSiteUrl":"https://changenow.io/",
<EMAIL_ADDRESS>    "display":true,
    "contact":{
        "Telegram":"https://t.me/NOWtoken_Bounty",
        "Reddit":"https://www.reddit.com/user/ChangeNow_io",
        "Medium":"https://medium.com/",
        "Youtube":"https://www.youtube.com/channel/UCpnkFY4kom0q3RJnhHiuMnw",
        "Twitter":"https://twitter.com/ChangeNOW_io",
        "Facebook":"https://www.facebook.com/ChangeNOW.io/"
    }
}

BC is going to sunset: https://github.com/bnb-chain/BEPs/pull/333


Issue 258 paginated team list users

3 commits:

return user count as number on the teams endpoint,
return user count also on the get team endpoint,
paginated users from team.

@paolochiodi @p16 for end point I went with Action.ReadTeam rights to be accessible. It could have been Action.ListTeams

@mihaidma

I went with Action.ReadTeam

I think it's fine


Change color of specific Calliope mini blocks

#75c6c8

@haithembelhaj

@joernalraun change was deployed. Please check it out!


chezscheme 9.4 (new formula)

Chez Scheme is a compiler and run-time system for the language of the
Revised6 Report on Scheme (R6RS), with numerous extensions. The compiler
generates native code for each target processor, with support for x86,
x86_64, and 32-bit PowerPC architectures.

@VMatrix1900: I was just about to submit a PR for this but you beat me to it! How would you feel about adding "--threads" and "--installschemename=chez" to the configure and dropping the conflict with MIT Scheme?
I think it would also probably be better to make the X11 dependency optional (per https://github.com/cisco/ChezScheme/issues/9) and put this in the homebrew-core repository, but that would require some hacking on chez's build system.

Or if we wanted to hack it, the formula could comment out the include mentioned in that ticket (#define LIBX11 "/usr/X11R6/lib/libX11.dylib") in version.h if the X11 option is off.

@michaelballantyne : Thanks for your suggestion. Upstream may add a configure option to disable the X11 build dependency(cisco/ChezScheme#9). Let's wait for it.

@VMatrix1900 Thoughts on that? Want to get this merged or closed, I think.

@MikeMcQuaid  still waiting for the upstream update. But I think it's okay to merge.

@BrewTestBot test this please

Thanks for your contribution to Homebrew!  You rock!


State return values in documentation

Fixes #9

Done.
Consider making a "result" type for fancy printing and easy access? Think about it.


Remove runtests target

Description
Closes #2429
The first commit is the part that closes #2429. The second commit removes the SFML_ENABLE_COVERAGE option entirely because after the first commit, there was little remaining need to keep that option. Coverage can be enabled from outside of the build so there's no particular reason that needs to have its own designated option in the build. It's not as if any developers are routinely checking coverage statistics locally so no convenience is being lost.

SFMLBuildMaster: Build this please.

The new code coverage figure is down from before and I might know why. Previously we didn't do debug builds of the DRM backend which meant we never ran coverage on all the DRM-specific code. The new coverage job does include the DRM code. Because most of that code is untested, I think it's bringing down the percentage although that's good because it means the new coverage figure is more accurate.

I don't think DRM-specific code alone gives the explanation.
Looking at the details, we see less files and less lines are included in the coverage report.
@@            Coverage Diff             @@
##           master    #2430      +/-   ##
==========================================
- Coverage   23.18%   18.15%   -5.03%
==========================================
  Files         212      145      -67
  Lines       18063    12167    -5896

SFMLBuildMaster: Build this please.

https://app.codecov.io/gh/SFML/SFML/commit/b288af2cd4e303cdc674cf13015d21dbfc7ea9ee
The Windows coverage report contains no coverage information which is probably the real reason coverage stats are so messed up. I need to figure out what I'm doing wrong with OpenCppCoverage.

I'm tempted to remove OpenCppCoverage if I can't get it to work and just use Clang + llvm-cov to do coverage on all platforms. If we use the same coverage tool everywhere then we can stop maintaining as much platform-specific code. I'd love to get rid of another platform-specific tool.

Codecov is no longer reporting any errors in the coverage reports yet we're still seeing that -5% coverage figure.

SFMLBuildMaster: Build this please.

The first commit (https://github.com/SFML/SFML/pull/2430/commits/af4e95316f5ce6d9372d6ddd23c5a4f669763130) which does the bulk of the work refactoring code coverage only reduces coverage by 1.93%. It seems like I've fixed some of the issues by fixing that empty Windows coverage report.
What's also weird is how the second commit (https://github.com/SFML/SFML/pull/2430/commits/7fba87ae8714b1f359187c700e41bc8f3c740c34) which just removes a CMake option reduces coverage by another 1.42%.
I'm not sure how to improve this any more. Either Codecov is buggy or I'm forgetting to run coverage on a codepath that was previously being covered.

SFMLBuildMaster: Build this please.

I don't know why but adding a lot more coverage jobs in a89bca5 seemingly fixed the coverage gap.
SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

I now realize how the intent of this PR could largely be addressed by merely removing the ALL form the runtests target and I'm open to a PR that does just that (and adds a new testing CI job to ensure tests still get run). Removing coverage from the build scripts is a separate matter that can come later if we want.

I still want to do this but I'd rather wait until I can come up with an LLVM-based solution that will work the same on all platforms like clang-tidy does.


Convert middleware to TypeScript, plus some other things.

Convert middleware to TypeScript.
Remove unnecessary files from both NPM packages.
Upgrade to TypeScript 3, and tweak some configs.
Replace eslint with tslint and fix warnings.
Add CHANGELOG to middleware.

Not any more, it can be removed entirely.

Done.


Re-introdude CXXFlags for darwin

These flags aren't set by default on this OS so to avoid the need to pass them into bundler / gem config we can set it for users.
This should fix the issue raised in #177

Yes, this would be nice to get out-of-the-box builds for macOS again. Thanks for creating the PR for this.

I think https://github.com/brianmario/charlock_holmes/pull/179 is a better solution.

Closing in favour of #179


[WIP] updated inferentia build

Addresses 5520

Update setup to use inf1 with neuronx-dkms and aws-neuronx-tools
Update build process to use python venv instead of conda using [neuron build instructions(https://awsdocs-neuron.readthedocs-hosted.com/en/latest/frameworks/torch/torch-neuron/setup/prev-releases/neuron-2.5.0-pytorch-install.html?highlight=aws-neuronx-dkms)

Todo:

Update tutorial to use latest Deeplearning machine (pytorch neuron 20.04 Ubuntu).
Figure if we can update to latest version of neuron sdk
Figure if we should change to inf2, or add inf2 build to our process
Run tests manually and make sure they work

closing since https://github.com/triton-inference-server/python_backend/pull/226 solves the issue.


Expose inits for CoreMIDI packets and packet lists

MIDIKit contains various internal constructors for:

MIDIPacketList (MIDI 1.0, old API)
MIDIPacket (MIDI 1.0, old API)
MIDIEventList (MIDI 2.0, new API)
MIDIEventPacket (MIDI 2.0, new API)

Currently MIDIKit obfuscates them by having MIDIEvents be passed directly to an endpoint's send(event:) method and the packetlists/packets are subsequently created internally.
There are advanced use cases where a library consumer would benefit from having these methods be made public. Including methods where packets could be constructed from a MIDIEvent with a return type of one of the types mentioned above.
The goal of MIDIKit is to not export Core MIDI types and methods if possible in order to provide a cleaner namespace and not cloud/intermix autocomplete space with Core MIDI types.
To that end, it would be good to investigate whether this be done without exporting CoreMIDI.
Maybe @_implementationOnly import CoreMIDI - but will the methods become available in a context where the user imports CoreMIDI?

Done. Will be in next release (0.7.3).

As an addendum:

Maybe @_implementationOnly import CoreMIDI - but will the methods become available in a context where the user imports CoreMIDI?

This is exactly how it works which is kind of neat.

MIDIKitInternals is not vended by any MIDIKit product/library. It's only usable by a consumer if they explicitly import MIDIKitInternals. This is good because generally it's stuff consumers don't need and keeps internals out of sight.
MIDIKitInternals uses import CoreMIDI (not implementationOnly) exposing some public extensions on Core MIDI types
Any other MIDIKit targets will use @_implementationOnly import MIDIKitInternals so that they can use its methods without vending them to the consumer
If the consumer needs MIDIKitInternals, they can both import MIDIKitInternals and import CoreMIDI and then its Core MIDI extensions will become usable


Fix Cassandra read bug when user query has no where clause (fixes #24829)

It would be very useful to be able to pass a user-defined query to CassandraIO for purposes of column selection only.
However, the current method for query parsing assumes such query necessarily comes with a 'where' clause, which leads to an invalid resulting statement when this is not the case.
This behavior has been reported on issue #24829.
The hereby proposed change fixes #24829 by checking if the user given query contains a where clause, appending a "WHERE" to it when it doesn't, instead of a "AND".

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

fixes #24829

Thanks for the catch, would you mind adding a unit test for this (query has no where clause)?

@Abacn thank you for the quick feedback. A unit test has been added now.

I've just noticed there are currently two unreleased versions on the changelog: 2.44.0 and 2.45.0. Should I disconsider the latter and move my change to the former one?

I've just noticed there are currently two unreleased versions on the changelog: 2.44.0 and 2.45.0. Should I disconsider the latter and move my change to the former one?

It will be 2.45.0. 2.44.0 branch is already cut: https://github.com/apache/beam/tree/release-2.44.0

LGTM, thanks!

Thanks for the fix up suggestions!

Hey, @Abacn. How you doing?
Do we have any forecast for when 2.45.0 (or at least 2.44.0) is going to be realeased?
Thanks in advance

Hi @lpmarques should be in mid Feb. Beam releases every 6 weeks. 2.45.0 branch will be cut next week and usually it takes 4-6 weeks to release.


[Bug]: Olive installs the stable GPU version of ORT even though the nightly GPU version of ORT is already installed

What happened?
related to the issue here
description
It appears Olive installs the stable GPU version of ORT even though the nightly GPU version of ORT is already installed
steps to reproduce
On a v100s machine, I prepare my pyenv with :
jsoto@v100s:/$ git clone https://github.com/microsoft/Olive.git
jsoto@v100s:/Olive$ cd Olive/
jsoto@v100s:/Olive$ pyenv virtualenv 3.9 aceituna
jsoto@v100s:/Olive$ pyenv shell aceituna

When in aceituna pyenv, I install all the necessary packages with:
(aceituna) jsoto@v100s:/Olive$ cd examples/whisper
(aceituna) jsoto@v100s:/Olive/examples/whisper$ python -m pip install -r requirements.txt
(aceituna) jsoto@v100s:/Olive/examples/whisper$ python -m pip uninstall -y onnxruntime ort-nightly onnxruntime-gpu ort-nightly-gpu
(aceituna) jsoto@v100s:/Olive/examples/whisper$ python -m pip install ort-nightly --index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ORT-Nightly/pypi/simple/

When running the configuration preparation, I do:
(aceituna) jsoto@v100s:/Olive/examples/whisper$ python prepare_whisper_configs.py --model_name openai/whisper-tiny --multilingual
(aceituna) jsoto@v100s:/Olive/examples/whisper$ python -m olive.workflows.run --config whisper_gpu_fp32.json

I see that Olive does not use my ort-nightly-gpu and install and uses onnxruntime-gpu;
 [2023-09-12 20:27:44,600] [DEBUG] [resource_path.py:151:create_resource_path] Resource path code is inferred to be of type folder. [2023-09-12 20:27:44,602] [DEBUG] [resource_path.py:151:create_resource_path] Resource path code/user_script.py is inferred to be of type file. [2023-09-12 20:27:44,606] [WARNING] [engine.py:140:__init__] The following execution provider is not supported: AzureExecutionProvider. Please consider installing an onnxruntime build that contains the relevant execution providers.  [2023-09-12 20:27:44,606] [INFO] [run.py:112:dependency_setup] The following packages are required in the local environment: ['onnxruntime-gpu'] [2023-09-12 20:27:44,606] [INFO] [run.py:118:dependency_setup] Installing onnxruntime-gpu... Collecting onnxruntime-gpu Obtaining dependency information for onnxruntime-gpu from https://files.pythonhosted.org/packages/5c/0c/70575b46df32de18d99872c11ec17dc589a48095d6828892241e434d0adc/onnxruntime_gpu-1.15.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata Downloading onnxruntime_gpu-1.15.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (4.0 kB) Requirement already satisfied: coloredlogs in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from onnxruntime-gpu) (15.0.1) Requirement already satisfied: flatbuffers in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from onnxruntime-gpu) (23.5.26) Requirement already satisfied: numpy>=1.21.6 in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from onnxruntime-gpu) (1.24.3) Requirement already satisfied: packaging in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from onnxruntime-gpu) (23.1) Requirement already satisfied: protobuf in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from onnxruntime-gpu) (3.20.3) Requirement already satisfied: sympy in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from onnxruntime-gpu) (1.11.1) Requirement already satisfied: humanfriendly>=9.1 in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from coloredlogs->onnxruntime-gpu) (10.0) Requirement already satisfied: mpmath>=0.19 in /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages (from sympy->onnxruntime-gpu) (1.3.0) Downloading onnxruntime_gpu-1.15.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (121.6 MB) ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 121.6/121.6 MB 16.8 MB/s eta 0:00:00 Installing collected packages: onnxruntime-gpu Successfully installed onnxruntime-gpu-1.15.1

and then pip show onnxruntime-gpu gives me:
Name: onnxruntime-gpu Version: 1.15.1 Summary: ONNX Runtime is a runtime accelerator for Machine Learning models Home-page: https://onnxruntime.ai Author: Microsoft Corporation Author-email<EMAIL_ADDRESS>License: MIT License Location: /home/jsoto/miniconda3/envs/olive-repro/lib/python3.9/site-packages Requires: coloredlogs, flatbuffers, numpy, packaging, protobuf, sympy Required-by:

Version?
i am using this version from main

ok thanks. Let me try it

i have run the following commands:
python -m pip uninstall -y onnxruntime ort-nightly onnxruntime-gpu ort-nightly-gpu
python -m pip install ort-nightly-gpu --index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ORT-Nightly/pypi/simple/

I end up with the following packages installed :
pip freeze | grep onnx
onnx==1.14.0
onnxruntime-extensions==0.8.0

pip freeze | grep ort
importlib-resources==6.0.1
ort-nightly-gpu==1.16.0.dev20230912006

running python prepare_whisper_configs.py --model_name openai/whisper-tiny --multilingualworks fine
I DO NOT run the command python -m olive.workflows.run --config whisper_gpu_fp32.json --setup
when I run python -m olive.workflows.run --config whisper_gpu_fp32.json I get it seems to work
when I run python test_transcription.py --config whisper_gpu_fp32.json   I get the following error:
models/conversion-transformers_optimization-insert_beam_search-prepost/whisper_gpu_fp32_gpu-cuda_model.json
{'type': 'ONNXModel', 'config': {'model_path': '/home/jsoto/code/Olive/examples/whisper/models/conversion-transformers_optimization-insert_beam_search-prepost/whisper_gpu_fp32_gpu-cuda_model.onnx', 'onnx_file_name': None, 'inference_settings': None, 'use_ort_extensions': True, 'hf_config': {'model_name': 'openai/whisper-tiny', 'task': None, 'feature': None, 'model_class': 'WhisperForConditionalGeneration', 'components': [{'name': 'encoder_decoder_init', 'io_config': 'get_encdec_io_config', 'component_func': 'get_encoder_decoder_init', 'dummy_inputs_func': 'encoder_decoder_init_dummy_inputs'}, {'name': 'decoder', 'io_config': 'get_dec_io_config', 'component_func': 'get_decoder', 'dummy_inputs_func': 'decoder_dummy_inputs'}], 'dataset': None, 'model_loading_args': None}}}
[2023-09-13 20:57:16,389] [WARNING] [__init__.py:203:_is_valid_ep] Error: [ONNXRuntimeError] : 1 : FAIL : subgraph_whisper_encoder.cc:43 Validate expect 2 inputs, got:3Olive will ignore this CPUExecutionProvider.Please make sure the environment with CPUExecutionProvider has the required dependencies.
Traceback (most recent call last):
  File "/home/jsoto/code/Olive/examples/whisper/test_transcription.py", line 117, in <module>
    output_text = main()
  File "/home/jsoto/code/Olive/examples/whisper/test_transcription.py", line 102, in main
    session = olive_model.prepare_session(None, "cpu")
  File "/home/jsoto/.pyenv/versions/olive-ii/lib/python3.9/site-packages/olive/model/__init__.py", line 338, in prepare_session
    return get_ort_inference_session(self.model_path, inference_settings, self.use_ort_extensions)
  File "/home/jsoto/.pyenv/versions/olive-ii/lib/python3.9/site-packages/olive/common/ort_inference.py", line 64, in get_ort_inference_session
    sess = ort.InferenceSession(
  File "/home/jsoto/.pyenv/versions/olive-ii/lib/python3.9/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 419, in __init__
    self._create_inference_session(providers, provider_options, disabled_optimizers)
  File "/home/jsoto/.pyenv/versions/olive-ii/lib/python3.9/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 471, in _create_inference_session
    sess.initialize_session(providers, provider_options, disabled_optimizers)

Looks like the workflow used some previously cached run which does not match the current version of ort.
Can you run the workflow again after deleting the cache subdirectory?
Also, there is a bug in the latest ort-nightly. so please install this version
pip install ort-nightly-gpu==1.16.0.dev20230903001 --index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ORT-Nightly/pypi/simple/

yes that worked. The following procedure makes everything work for me:
cd examples/whisper
pip install -r requirements.txt
python -m pip uninstall -y onnxruntime ort-nightly onnxruntime-gpu ort-nightly-gpu
pip install ort-nightly-gpu==1.16.0.dev20230903001 --index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/ORT-Nightly/pypi/simple/

pip freeze | grep onnx
pip freeze | grep ort

onnx==1.14.0
onnxruntime-extensions==0.8.0
importlib-resources==6.0.1
ort-nightly-gpu==1.16.0.dev20230903001

python prepare_whisper_configs.py --model_name openai/whisper-tiny --multilingual
python -m olive.workflows.run --config whisper_gpu_fp32.json
python test_transcription.py --config whisper_gpu_fp32.json

Thanks for the help


Update sbt to 1.5.4

Updates org.scala-sbt:sbt from 1.5.2 to 1.5.4.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Codecov Report

Merging #162 (f2c953b) into master (8310fff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #162   +/-   ##
=======================================
  Coverage   84.04%   84.04%
=======================================
  Files          19       19
  Lines         282      282
  Branches        6        6
=======================================
  Hits          237      237
  Misses         45       45

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 8310fff...f2c953b. Read the comment docs.


Clean up manipulatory assignment operators

Something we noticed when trying to figure out what is going on in our Random Access experiments. Also, the code is nicer...

:yellow_circle: Regression Test (8-Queens)
'ssoelvsten/dd/assignment_ops' is a change in performance of -0.99% (stdev: 4.13%).

...
ssoelvsten/main
ssoelvsten/dd/assignment_ops

Mean
100.93
101.93

Standard Deviation
4.17
3.81

Number of samples: 15


Needs a license so people know if they can use this

Please add a license file of some kind
If you want other people to use your work, an MIT-style license is strongly encouraged

Thanks a lot for your reminder! we've already add MIT License~

I'm closing this issue now~

Thanks much


Invalid TinyDB request in find_one

TinyDB get() expects a query instead of a dictionary.  Also, when getting multiple matches, it will return a random item (https://tinydb.readthedocs.io/en/latest/usage.html#retrieving-data).
The solutions that I can see are either to:

Select all elements from the table, then get the first one from the list (this pull request).
Or, we can make a query that selects all, like Query()["_id"].exists().  I don't know if this would return the fist item every time, or a random one.

Actually, my solution does not work either if there are not documents yet -- all()[0] fails with an IndexError.
I wonder if get(eid=1) would work?  I am not that familiar with TinyDB, but EIDs should be assigned in order.

Thanks for the update.  I will do some testing on the merge as well.  Also if you are interested in helping directly with the project let me know..    The main goal of this project is to provide the ability to use code that requires mongo on any system testing or otherwise without really needing mongodb installed.

Thanks.  I am just starting to learn MongoDB and NoSQL, but I will be happy to send in pull requests for anything else that I find.


Issue 481 fix

This fix solves the #464 and #481 issues. The implementation of the fix is done entirely in the PropertyRule class.
IMPORTANT: It will break the so far implemented naming convention of types. Instead of deriving the typename from the name of the first using property, or file name, the type name is now always based on its name from the reference (for internal references) or the actual file name for external references. Please see also my comments on #481 for more information.
PLEASE SEE ALSO: _TODO_s in the implementation of the getPropertyTypeName function (see diff view below).
Regards,
Dimitrij

To make it better accessible, I copy the notes/questions from the mentioned TODOs here. In all cases they are related to the id of a schema, which yet is not supported.

The first is about the nested references to self? How to handle them: e.g: "$ref": "#/definitions/level/sublevel/actual_element" with type information available only on actual_element and level/sublevel used as namespaces to avoid type conflicts e.g. level could also contain an actual_element definition which differs from the sublevel/actual_element. Depending on whether a level/sublevel element defines a type or is just used for structuring purposes, it may be worth to generate either a java class with a nested static class or java subpackages for such cases. This rule however seems to be not applicable if our actual_element contains an id, which defines the namespace of it, thus can be mapped to a java package.
The second is about external references, e.g. "$ref": "my_type.json". The uri may by any valid URL (file, classpath, java, http, etc). The url may point to a schema with a type name already defined e.g. in the document referencing it, but with different contents. How to handle this? The usage of id can help to overcome this ambiguity.

What does the comunity think about it?

I really like the idea of the schema's URI being directly related to the type name.  Here are some of my thoughts on this:

It would definitely need to be something for v0.5.0 or later.
I would really like to see a one to one relationship between schema files and Java source files.
Nested schemas should be rendered as nested static classes, since this would eliminate namespace conflicts when generating the types.
I wonder if something could be done to provide a strategy for this.  This kind of change could really cause problems for people.

I've already prepared something which goes into this direction. The 3a1a96de7bf5dc6dcf2904badbb06e7cd5612e8d commit, referenced in the #481 issue includes, let's say the first drop for this. Since the JSON schema file can define multiple types, like an XML schema does, I personally see the one two one relationship between not a file, but a schema for a type definition and a Java source file. That means, that from my understanding, the id defines a namespace for a type, which corresponds to a package in Java.
Here a basic example. Let name this file job.json
{
    "id": "http://test.com/2016/job.json"
    "$schema": "http://json-schema.org/schema#",
    "description": "A Job definition",
    "type": "object",
    "properties": {
        "title": { "type": "string" }
    },
    "required": ["title"]
}

Without an id, the corresponding Java class is generated into the "default" package. With the id present, like in the example above, the class will be generated into the com.test._2016.job package. In the showed case the schema file contains just one type definition - of the job. Let me make the example more complex and add another type to the job.json file:
{
    "id": "http://test.com/2016/job.json"
    "$schema": "http://json-schema.org/schema#",
    "description": "A Job definition",

    "definitions": {
        "description": {
             "type": "object",
             "properties": {
                 "title": { "type": "string" }
            }
        }
    },

    "type": "object",
    "properties": {
        "title": { "$ref": "#/definitions/description" }
    },
    "required": ["title"]
}

Now the schema file defines two type, and thus two schemas, one for the job and one for description. Since the description schema does not define its own id, it belongs to namespace defined by the top level id. You say, a nested schema, like here, should be rendered as a nested static class. Why should it be nested? It has its own name description and having an (indirect) id it has its namespace. My understanding may be wrong, but as long as a type definition is not anonymous (defined inplace as a type of a property, thus does not have a name), it can be a standalone Java type, regardless whether the type defines an enum, or a class. I see it as a design error, if a namespace (defined by an id) defines multiple types (json schemas) with same name. If there is a need for same type name - just give it an id. You do it while defining an XML schema as well, right?
That also means, that not the location (folder structure) makes the types defined in the corresponding files unique, but the id given for each type. The aforementioned commit implements exactly this: As long as a type is referenced using "$ref", it is generated as a standalone Java type. I've tried to describe the behavior in the comments for the #481 issue.
I may miss important things, but to be honest, I encountered all those issues, I've linked to #481, and was looking for a fast solution, which would satisfy my needs. The current result is available with 3a1a96de7bf5dc6dcf2904badbb06e7cd5612e8d. I do not expect everybody to accept it is as is, but it could be a starting point, or even just a source of ideas.

Maybe it becomes more obvious, if a schema file does not define a top level type. Here is modified example from http://json-schema.org/latest/json-schema-core.html (I just added the "type": "object"):
"schema1": {
    "id": "schema.json",
    "type": "object"
},
"schema2": {
    "id": "otherschema.json",
    "type": "object"
    "nested": {
        "id": "#bar",
        "type": "object"
    },
    "alsonested": {
        "id": "t/inner.json#a",
        "type": "object"
    }
},
"schema3": {
    "id": "some://where.else/completely#",
    "type": "object"
}

schema1, schema2 and schema3 all define a standalone type. Each of them should correspond to a generated Java class. You see, here the type can't be derived from the file name. The nested and alsonested should be generated as nested static classes in the class generated for schema2.

The only reason I suggested inner static classes is that they work like a package, for name spacing purposes, and will prevent collisions at the file system level.  Again, this is just a suggestion.
This is what I would generate for the example you gave, assuming it is in schema.json:
public class Schema {
  ...
  public static class Schema1 {
    ...
    public static class Nested { ... }
    public static class AlsoNested { ... }
  }
  public static class Schema2 { ... }
  public static class Schema3 { ... }
}

That would prevent files from colliding, in the case where conflicting names are present in neighboring schemas.  Also, it seems to me that mimicking the schema structure in the java types would lead to the least amount of surprise for those using the types.

That's true. This is an other approach to prevent collisions. You last sentense is also very true. Going away from java types to packages would indeed lead to a lot of surprizes for people used to the behavior of the current implementation.
BTW. The schema example above is not supported by the current implementation. At least nothing was generated given the code I have in my schema_id_support branch. Maybe I have broken something, which was previously possible. Anyway I've recently enabled the generation of types from such schema files, which given the mentioned above id support would result in the following generated output:
package schema;
public class Schema1 { ... }

package otherschema;
public class Schema2 { ... }

package else.where.completely;
public class Schema3 { ... }

package bar;
public class Nested { ... }

package t.inner
public class AlsoNested { ... }

If however the above schema is changed to:
"schema1": {
    "id": "schema.json",
    "type": "object"
},
"schema2": {
    "id": "otherschema.json",
    "type": "object"
    "properties": {
        "nested": {
            "type": "object"
        },
        "alsonested": {
            "type": "object"
        }
    }
},
"schema3": {
    "id": "some://where.else/completely#",
    "type": "object"
}

follwing is generated:
package schema;
public class Schema1 { ... }

package otherschema;
public class Schema2 {
    public static class Nested { ... }
    public static class AlsoNested { ... }
}

package else.where.completely;
public class Schema3 { ... }

The bullet point 4 from your first comment is definitely very important. Both in sense of the description of the expected behaviour for the implementation, and also to inform everybody.
I've not pushed it yet, since all my work is need driven and the plugin is in my case a tool. Because of this I've not implemented/updated unit/integration tests. If you're interested in the details I can push it with given testability limitations.
An other thing I was looking for is the the oneOf support. I see a lot of discussions here (#391 is very prominent). Do you know whether somebody has already done something in this direction (I mean at least basic code)?
Regards,
Dimitrij

soory, I pushed the wrong button

I would take the oneOf discussion over to #392 and have a look at the network graph to see what commits may be out there.

Is there any chance to have this fix in master soon? And also have 0.4.19 released after that :)

@sveba from the comments and conversation so far, I don't believe this fix is complete and ready to merge. It's unlikely that this will appear in the next release.

Hi guys, i downloaded the verion 0.4.23 and this fix doesn't appear to be there.
How can I do to get only this fix without affect the rest of the code?
Is there a planned release soon?
Many thanks

Will it be merged anytime soon?

We too would like a fix for this

Any ETA on merging this?

Can we get this merged ? all seem happy with the fix, it's been over a year now.

This change isn't ready to merge.
It breaks compatibility for existing users of generated types in a major way and still has a number of unresolved issues that make it unsuitable for inclusion in the core.
Feel free to merge this into your own custom version.


dependencies versions should be upgraded to address security vulnerabilities

Is there any plans to release newer version of the zipkin-go-opentracing with upgraded dependencies?
The Veracode reports at least two vulnerabilities in the underlying dependencies see gomega and protobuf

Zipkin is a volunteering project, so you mind opening a PR with such
upgrades?
On Sat, 17 Jul 2021, 00:09 IF034Java, @.***> wrote:

Is there any plans to release newer version of the zipkin-go-opentracing
with upgraded dependencies?
The Veracode reports at least two vulnerabilities in the underlying
dependencies see gomega
https://sca.analysiscenter.veracode.com/vulnerability-database/libraries/github-com-onsi-gomega/GO/GO/lid-886825/versions/v1.4.3
and protobuf
https://sca.analysiscenter.veracode.com/vulnerability-database/libraries/github-com-gogo-protobuf/GO/GO/lid-885410/versions/v1.2.0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin-contrib/zipkin-go-opentracing/issues/158,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYATTQ3JYXA66OQI3L5DTYCUZXANCNFSM5AQK6ZOA
.

@jcchavezs
I was trying to push my local branch but received error
Permission to openzipkin-contrib/zipkin-go-opentracing.git denied to IF034Java
Do I need any specific permissions to contribute to the zipkin-go-opentracing project?

@IF034Java no you don't need special permissions to contribute, but you can't add branches directly to this repo. That indeed is reserved for maintainers only.
Make a fork of this repo on your local github account and then push your updates to a branch there. Then you can create the pull request from your fork to this repo.
Thanks for helping out!

@jcchavezs @basvanbeek
PR is created to address the protobuf issue
see here

Bump

new release has been cut with zipkin-go v0.4.1 and opentracing-go v1.2.0.


Too much space on Android between keyboard and keyboard-attach directive

Hello!
I am facing this issue below with too much space between keyboard and the input field with the keyboard-attach directive:

My device:
Nexus 4
Android 5.1.1

My code looks like:
<ion-view>
    <ion-nav-title>
    <ion-nav-content>
    <ion-footer-bar keyboard-attach>
        <button>
        <label>
            <input>
        <button>
</...>

The app runs in fullscreen mode and this is set as well in code:
ionic.Platform.isFullScreen = true

and in the AndroidManifest.xml:
<activity android:windowSoftInputMode="adjustResize">

BUT the issue still remains...

i've the same exact issue, did you found a work around about this?
i know that the problem is that app is not recieving a system event but a "local" event... and the full screen is activated when the system event ocurrs.
tried to find out how to get in ionic the height of the "hardware" buttons or even if the mobile has real hardware buttons or if they are on screen so i can force the body to go down some pixels.. without luck.

We had the cordova-plugin-statusbar installed to hide the statusbar and support fullscreen app. But luckily we wanted to fall back to a non-fullscreen app and since then the issue vanished...
Sorry, that i cannot give you a real solution. But it probably could give you a hint...

+1


Auto close Detour on win leave

I tried to make detour window automatically close when you leave it with:
      vim.keymap.set("n", "<c-w><enter>", function()
        require("detour").Detour() -- Open a detour popup
        local current_bufnr = vim.api.nvim_get_current_buf()
        local current_winid = vim.api.nvim_get_current_win()
        print(current_bufnr, current_winid)
        vim.bo.bufhidden = 'delete' -- close the terminal when window closes
        local test_au = vim.api.nvim_create_augroup("TimedayThemeAu", { clear = true })

        vim.api.nvim_create_autocmd({ "BufEnter" }, {
          -- pattern = "<buffer>",
          buffer = current_bufnr,
          callback = function()
            print "Blff leave"
            vim.api.nvim_win_close(current_winid, true)
          end,
          group = test_au,
          -- nested = true, -- dow notwork in this case
        })
      end)

But it does not seem to work. Maybe this could be added as option into detour itself...

As I understand, you want the popup to close when you switch to another window correct?
In that case, I think you want to use the "WinLeave" event for your autocmd instead of the "BufEnter" event. I changed it (and a couple other things) and it works :)
vim.keymap.set("n", "<c-w><enter>", function()
  require("detour").Detour() -- Open a detour popup
  local current_winid = vim.api.nvim_get_current_win()
  vim.bo.bufhidden = 'delete' -- close the terminal when window closes
  -- I am not entirely sure if there is any benefit to having this augroup
  local test_au = vim.api.nvim_create_augroup("TimedayThemeAu", { clear = true })

  vim.api.nvim_create_autocmd({ "WinLeave" }, {
    buffer = current_bufnr,
    callback = function()
      print "Blff leave"
      vim.api.nvim_win_close(current_winid, true)
      -- this autocmd only needs to return once so make sure you return true so it deletes itself after running.
      return true
    end,
    group = test_au,
    -- this nested attribute is actually needed at the moment or breaks the plugin's assumptions.
    -- I need to update the plugin so it doesn't depend on users remembering to setting the nested attribute.
    nested = true,
  })
end)

Leave this open until #3 is resolved

@carbon-steel  any ideas why above code stopped working , after latest update?

I'll take a look in a couple hours.

I tried reproducing your error with the following steps:

Open a new neovim instance with no user config/plugins except detour.nvim by running:
nvim --clean --cmd "set rtp+=/path/to/your/detour.nvim/"

To figure out where your detour.nvim is installed, you have to check where plugin manager keeps installed plugins.

As an example, here is where lazy.nvim stores plugins.

Alternatively, you could also git clone this repo to a path of your choosing and provide that path to this command. Make sure that you're using the main branch. Either way works.

Save the following code in a new lua file:

vim.keymap.set("n", "<c-w><enter>", function()
  require("detour").Detour() -- Open a detour popup
  local current_bufnr = vim.api.nvim_get_current_buf()
  local current_winid = vim.api.nvim_get_current_win()
  vim.bo.bufhidden = 'delete' -- close the terminal when window closes
  -- I am not entirely sure if there is any benefit to having this augroup
  local test_au = vim.api.nvim_create_augroup("TimedayThemeAu", { clear = true })

  vim.api.nvim_create_autocmd({ "WinLeave" }, {
    buffer = current_bufnr,
    callback = function()
      print "Blff leave"
      vim.api.nvim_win_close(current_winid, true)
      -- this autocmd only needs to return once so make sure you return true so it deletes itself after running.
      return true
    end,
    group = test_au,
    -- this nested attribute is actually needed at the moment or breaks the plugin's assumptions.
    -- I need to update the plugin so it doesn't depend on users remembering to setting the nested attribute.
    nested = true,
  })
end)

From within the buffer of the new lua file, run :luafile % to execute the code.
Run the :Detour command (a popup should appear)
Switch out of the popup with <C-w>w (the popup should disappear)

Here is what happens on my system when I do this (ignore the pink color of the popup. That's just the default colorscheme.):
autocmd2.webm
Could you try this a let me know if it works for you?
If it doesn't, could you provide the first three lines from: nvim --version?

Hey @JoseConseco, I've just introduced some changes to the plugin to add better autocmd support. Could you try running the plugin on the autocmds branch and see how it works for you? I'd like to get some feedback before merging it into main.

I just tried, and sometimes closing detour window  automatically,  works - for example when using using ctrl+w  + [jklw]  (to switch to different window). But then if I try to open detour in new split - eg. using ctrl+v  - then split will be done, but 'old' detour window  will remain open.
local function close_on_leave()
  local attached_bufnr = vim.api.nvim_get_current_buf()
  local attached_winid = vim.api.nvim_get_current_win()
  -- vim.bo.bufhidden = 'delete' -- close the terminal when window closes
  -- I am not entirely sure if there is any benefit to having this augroup
  local test_au = vim.api.nvim_create_augroup("DetourClose", { clear = true })

  vim.api.nvim_create_autocmd({ "WinEnter"}, {
    buffer = attached_bufnr,
    callback = function()
      print "Detour leave"
      -- check if current window is floating. If it is, skip closing detur window
      local winid = vim.api.nvim_get_current_win()
      if vim.api.nvim_win_get_config(winid).relative ~= "" or winid == attached_winid then
        return
      end
      vim.api.nvim_win_close(attached_winid, true)
      -- this autocmd only needs to return once so make sure you return true so it deletes itself after running.
      return true
    end,
    group = test_au,
    -- this nested attribute is actually needed at the moment or breaks the plugin's assumptions.
    -- I need to update the plugin so it doesn't depend on users remembering to setting the nested attribute.
    nested = true,
  })
end

vim.keymap.set("n", "<c-w><enter>", function()
  require("detour").Detour() -- Open a detour popup
  close_on_leave()
end)
vim.keymap.set("n", "gD", function()
  require("detour").DetourCurrentWindow() -- Open a detour popup
  close_on_leave()
  -- vim.bo.bufhidden = 'delete' -- delete this scratch buffer when we move out of it
  require('telescope.builtin').lsp_definitions({initial_mode = 'normal'})

end)

In the end, it would be cool if detour had build in option, where its floating window is closed automatically, when it is left.

It works now.  I must be bad at lua :/ .  Thanks, all is great now.  I can create split from detour, and  it will close popup ok now.

Done. All seems work ok, on dev too.

Great! thanks :)

I must be bad at lua :/.

No, autocmd's are genuinely tricky. I've been wrestling with them the whole time I've been working on this plugin.


New Search Icon at the right corner.

Hello All,
Can you please let me know on how to include new icons(search)  beside the phone and star icon in the right side
I have tried below snippet ,but I dont see the icon getting pulled from the path provided
++++++++++++++++++++++
// sample code below.
{
          name: 'search',
          badge:1,
          rightIconImage:require('./images/search.png'),
          isBadgeLeft:true,
          onPress: () => Alert.alert('search pressed!')
        },

+++++++++++++++
please let me know your thoughts.
Thanks,
Goutham KD

@GouthamKD Turns out, after seeing the code, the parameter used is not source, but image.
So try to change:
source : require('./images/search.png'),

into
image : require('./images/search.png'),

Hi guys, sorry for the delay.
You're right @subsub , it seems to be image in a rightIcons element. I'll update the docs right away

Updated in b0f99d3313c55a7dbc6245ebadf9aac597267b6a


adds port 4000

In the docker-compose file, the ports sections maps host port 4000 to container port 80, so if we want to see our app in a browser we should check on port 4000.
Proposed changes
I added port 4000 because that is the port mapped to the container's port 80. So if we want to view our app in the browser or with curl we need to include port 4000 in the url.
Related issues (optional)
The screenshot image should be updated, too.
Fixes #6904

Thanks for the fix. Let's ship it!

Thanks! :)


Security page open source software section

https://helium.nebra.com/security
The column for "description" here doesn't seem to make a lot of sense?

Fix description  https://github.com/NebraLtd/Helium-Guides/commit/6cd494e0cfb3e3d667f46d130608a6ec2e724ccf , https://github.com/NebraLtd/hm-gwmfr description is missing, I'll add that after checking with @ryanteck .

Gwmfr is for provisioning the ECC secure element chip, it's only used during manufacture essentially

Thanks @shawaj , added that too, now it's complete, please check and close 🙂👍

Looks good to me @salmanfarisvp


Add automatic issues for _Leanpub repo as well

Is your feature request related to a problem? Please describe.

There's not automatic issues for the Leanpub repository, so we should add that to help set up there too.

This has been done.


All values of a class setted to null

Trying to use the library in order to create random instances of unknown object ( which type is known at runtime ), all the values inside such a class are setted to null.
Might it be that these fields are private, and so they are not seen/changed by the library?
i.e. Trying to create a random instance of originalObject
PodamFactory factory = new PodamFactoryImpl();
Object x = factory.manufacturePojo(originalObject.getClass());

all fields for x are null.

You need to use the manufacturePojoWithFullData method.
Regards,
M.

On 12 Jul 2016, at 08:07, Giuseppe<EMAIL_ADDRESS>wrote:
Trying to use the library in order to create random instances of unknown object ( which type is known at runtime ), all the values inside such a class are setted to null.
Might it be that these fields are private, and so they are not seen/changed by the library?
i.e. Trying to create a random instance of originalObject
PodamFactory factory = new PodamFactoryImpl();
Object x = factory.manufacturePojo(originalObject.getClass());
all fields for x are null.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/devopsfolks/podam/issues/211, or mute the thread https://github.com/notifications/unsubscribe/AAWXINYSFE5Oa5Gcp5jVdb3FfXGflRMFks5qUz1GgaJpZM4JKGPr.

Might it be that these fields are private, and so they are not seen/changed by the library?
There could be million reasons for that. Could you please show the code of your POJO class?

I haven't the code for the class, since I am trying to generate random instances of objects which type is only known at runtime, through reflection.
I tried with ManufacturePojoWithFullData but doesn't seem to work as well.

I haven't the code for the class, since I am trying to generate random instances of objects which type is only known at runtime, through reflection.

Without knowing, which class isn't populated it's impossible to say anything about this issue. Name the problematic class or present a Podam log, which should contain enough information for fixing this issue.

I cannot share my POJO class since I am trying to randomize objects which type is only known at runtime. Thank you for your help.

Even you use random objects at runtime, they are still specific objects at each specific execution, so you can print class name into the log and Podam logs data regarding its instantiation as well.


uninitialized constant Twitter::REST

Working with twitter (5.13.0) and still happens. No twitter-text text present

@NARKOZ presumably you re-opened this for a reason. If you are still having an issue with this, please post a Gemfile (and potentially a link to your Gemfile.lock) as well.


Adding a reference causes build to create unwanted directory tree

I have the following file structure:
test/
    ts/
        test.ts
        tsconfig.json
    js/
        test.js <- compiler output

My tsconfig specifies “outDir”:”../js” so that ts files are compiled to js files in that directory.
But if I add a reference to a typescript file to test.ts, eg:
/// <reference path="../../lib/ts/File.ts"/>

Then on build, it creates an unwanted directory tree under ‘js’ and the compiled js ends up under js/test/ts/test.js. Eg:
test/
    ts/
        test.ts
        tsconfig.json
    js/
        test/
            ts/
                test.js <- compiler output

This happens on Windows 7 & 8.1.

Then on build, it creates an unwanted directory tree under ‘js’ and the compiled js ends up under js/test/ts/test.js. Eg:

This is by design. Output is based on a concept of commonRootDir. Search for rootDir and baseDir in Microsoft/TypeScript/issues for details.
Thanks ! :rose:


Fix LocalUfs.mkdir() for when parent dir is null

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8222/
Test PASSed.

alluxio-bot, merge this please


GraphQL Updates

What's new?
Renamed some schema types
Apologies for the annoying refactor, but we were talking about users having multiple galleries recently, and I figured this would be a good time to standardize in case we need a GalleryCollection in the future to represent a gallery in a collection.

GalleryNft -> CollectionNft
GalleryCollection -> Collection
GalleryCollectionLayout -> CollectionLayout
GalleryCollectionLayoutInput -> CollectionLayoutInput

New queries

nftById(id: DBID!)
collectionNftById(nftId: DBID!, collectionId: DBID!)

New media subtype
Added GltfMedia as a MediaSubtype. We'll use this media type when we see a .glb or .gltf extension.
Dataloader cache clearing
Any public API function that writes to the database will now clear all dataloader caches to ensure that we return the latest data to the caller.

seeing this late but this is a great idea! 👏

seeing this late but this is a great idea! 👏

On that topic: do you want me to keep tagging you as a reviewer on all this stuff? I left you out cause I figured you have enough stuff going on, but I'm happy to keep clicking your name when I put up PRs!

that'd be great if you don't mind! i enjoy seeing your PRs and now that I'm learning to use graphql on the frontend it's extra useful to see these changes (plus I have more context to understand them)

but definitely count me as an optional reviewer, don't want to block anything bc of timezone


[spi_device] Add remaining non-TPM DIFs

Signed-off-by: Alexander Williams<EMAIL_ADDRESS>

If this finishes off the SPI device DIFs (as the title suggests) could you update the checklist in a follow up?

If this finishes off the SPI device DIFs (as the title suggests) could you update the checklist in a follow up?

All the TPM DIFs still remain, so we're not there yet. Thanks for reviewing!


rust: lots of junk strings

from https://www.virustotal.com/gui/file/ef01c77275359e1399b9d2a7a1bc12e440d6cfc143e143237dc40918bab973fe
 ───────────────────────────
  FLOSS RUST STRINGS (9782)
 ───────────────────────────

82J@
82J@
@4J@
,vJ@
/rustc/2c8cc343237b8f7d5a3c3703e3a87f2eb2c54a74\library\std\src\io\mod.rs
failed to write whole buffer
C:\Users\user\.cargo\registry\src\github.com-1ecc6299db9ec823\aes-0.7.5\src\ni\utils.rs
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@
 GK@

I would like to work on this issue. Do u want to just ignore the junk strings and remove them?

yes. we'd like to understand why they are being identified and then update the algorithm so that they're no longer extracted.


Dump nydusd panic info and backtrace into logger and show nydusd git commit version

This RR helps debug and develop nydusd.

Nydusd is sometimes forked by nydus-snapshotter and has no stdout and stderr to print panic info. So dumping the panic info into logger is a good choice.
print nydusd git commit version by nydusctl CLI tool

This can address https://github.com/containerd/nydus-snapshotter/issues/104

The features is fantastic. Could we remove the dependency on "log-panics"?

The features is fantastic. Could we remove the dependency on "log-panics"?

I checked with log_panics code, it's surprisingly simple, so to avoid the crate deps, maybe we can have our own panic::set_hook(Box::new(move |info| { in apps/lib.rs
@changweige @jiangliu what do you think?

The features is fantastic. Could we remove the dependency on "log-panics"?

It is a little difficult. Even if we can get rid of crate log_panics, will still have to depend on crate backtrace. I prefer including the feature ASAP so we can move to other urgent tasks. If we have move bandwidth in the future, let's try to get rid of it.

We can get panic info in the log like
[2022-07-28 11:03:57.257415 +08:00] ERROR [/home/gechangwei/.cargo/registry/src/mirrors.sjtug.sjtu.edu.cn-4f7dbcce21e258a2/log-panics-2.1.0/src/lib.rs:130] thread 'fuse_server' panicked at 'assertion failed: self.offset + self.len == _offset': storage/src/device.rs:671
   0: backtrace::capture::Backtrace::create
   1: backtrace::capture::Backtrace::new

FAILED

The features is fantastic. Could we remove the dependency on "log-panics"?

I checked with log_panics code, it's surprisingly simple, so to avoid the crate deps, maybe we can have our own panic::set_hook(Box::new(move |info| { ... } in apps/lib.rs
@changweige @jiangliu what do you think?

Let's move on first, and simplify it later.


qt: Equalize notifySystray and notifyDBus interface

Make notifySystray and notifyDBus feature-equivalent by making notifySystray support custom icons. This uses the QSystemTrayIcon::showMessage overload introduced in Qt 5.9.
This is an initial step towards bitcoin-core/gui#573.

I guess the best way of testing this on Linux is to comment out the DBUS notificator code. And generate notifications somehow.
Also, it should be tested on Windows. DBUS is irrelevant on Windows, but the QTrayIcon code is used there. Removed the Linux label.

Closing, it looks like we're not using the custom icon functionality at all. Will remove.


ci: add weekly dependabot job

Do we get any for this repo? I may have missed it, thought we didn't


No requests are captured by the HUNT scanner

I have observed that no requests are captured by hunt scanner
I have performed passive scan through Target tab in burp v2020.4

same here, even with the remix version

Hi,
did you have specific targets in scope set within Burp?
Just targets in scope go through HUNT.
Cheers,
Simone

hey,
i have a (.*) regex in include scope and exclude scope contains some url.
is that a problem?
regards,
Trilok

hey,
i have a (.*) regex in include scope and exclude scope contains some url.
is that a problem?
regards,
Trilok

@bl4ckr0se-wq  No, it should not be a problem. HUNT Scanner uses Burp's callbacks.isInScope() API, if the scope is set correctly HUNT will process those requests.


not support 24-hour time by getValueFrom()

The value in cell B2 is 00:30:00, but the value parsed is 12:30:00.
cell := xlsx.GetCellValue("Sheet1", "B2")
I take a try to fix it as follows, but 12-hour time is not supported in this way.
func parseTime(i int, v string) string {
f, err := strconv.ParseFloat(v, 64)
if err != nil {
return v
}
val := timeFromExcelTime(f, false)
format := builtInNumFmt[i]  ===========>>h:mm:ss
    replacements := []struct{ xltime, gotime string }{
            {"yyyy", "2006"},
            {"yy", "06"},
            {"mmmm", "%%%%"},
            {"dddd", "&&&&"},
            {"dd", "02"},
            {"d", "2"},
            {"mmm", "Jan"},
            {"mmss", "0405"},
            {"ss", "05"},
            {"hh", "15"},
            {"h", "3"},  ===========>>{"h", "15"}
            {"mm:", "04:"},
            {":mm", ":04"},
            {"mm", "01"},
            {"am/pm", "pm"},
            {"m/", "1/"},
            {"%%%%", "January"},
            {"&&&&", "Monday"},
    }
    for _, repl := range replacements {
	format = strings.Replace(format, repl.xltime, repl.gotime, 1)
}
// If the hour is optional, strip it out, along with the possible dangling
// colon that would remain.
if val.Hour() < 1 {    ==============>>val.Hour() is 0
	format = strings.Replace(format, "]:", "]", 1)
	format = strings.Replace(format, "[3]", "", 1)
	format = strings.Replace(format, "[15]", "", 1)
} else {
	format = strings.Replace(format, "[3]", "3", 1)
	format = strings.Replace(format, "[15]", "15", 1)
}
    return val.Format(format)

}
The xlsx file is attached.   Thanks.
time.xlsx

@PineTreeWild Thanks for your issue. I have fixed it.


[Refactor] Make scripts of running SwiftLint and Manifest.lock optional

Purpose

Add Swiftlint.sh checking line to check swiftlint installed

if [ -f "${PODS_ROOT}/SwiftLint/swiftlint" ]; then

*Add [CP] Check Pods Manifest.lock checking line to check Manifest.lock existed
if [ -f "${PODS_ROOT}/Manifest.lock" ]; then

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

As per test from @jimchou-dev , there is some limitation for [CP] Check Pods Manifest.lock optimization


Add run-exports

@conda-forge-admin, please rerender

Travis stopped after 48 minutes without output. Let's try again…


SAK 30358

I've rebased these commits against the current master (12-SNAPSHOT).  I can send a separate pull request against the 11 release branch too, if that's useful?
From https://jira.sakaiproject.org/browse/SAK-30358:
I've incorporated some further feedback from Jeff Pasch and Kyle Blythe into the user dropdown menu and favorites drawer. This includes:

Improvements to the display of the favorites drawer on mobile

Changes to show gateway sites in the sites bar for users who aren't logged in

Changes to hide the "Sites" button from users who aren't logged in

Changes to the display of the user dropdown menu: make the avatar picture link to the user's profile, allow the menu to be toggled by clicking the "Sites" button a second time.

Really it is, thank you so much Mark !!

LGTM :+1:


Update maxLifetime re adding random 10 sec

Please update maxLifetime configuration comments with comments about Adding random 10 second for every hour in the maxlifetime.
https://github.com/mmuruganandam/HikariCP/commit/0b99b0655c3cc855f9d006746784ff0bca229926

You might also add to idleTimeout that it doe not renew the life of a connection within the minimumIdle connections. I see it being renewed when maxLifetime is reached.. I watched this with five connections in a pool of five, where I was using one connection and then stopped using it. It did not retire at 30 seconds and start the count over... and It was 150 seconds off from the other four connections.. when the other four got close to my 600 sec count that I set, they all started to randomly renewed along with the one lagging behind by 150 seconds.

The documentation says:

This property controls the maximum lifetime of a connection in the pool. An in-use connection will never be retired, only when it is closed will it then be removed.

The pool could choose to retire the connection at any time. The algorithm has changed over time, and may change again.  The behavior of the current version of HikariCP is subtly different than the version that you linked to.
Because of this, any further specificity in the documentation would be an implementation detail.  The contract of the configuration setting is correctly specified by the documentation.
I am not sure I understand the idleTimeout issue you are reporting.  Can you please post the configuration you were running with?  At this point in the HikariCP maturity curve, you probably have higher likelihood of being hit by a bus than finding a bug in HikariCP.

I'll try to be more clear. I am new to this and am just trying to provide some feedback on things that I came across as I was trying to understand it all and fix some issue I have been running into.
I am using 2.7.2
Issue 256 says that it added random 10 second for every hour in maxLifetime to manage the closing of connection in a span of seconds instead of all at once. It might be good to note that under maxLifetime.
I don't think it is a bug, but based on the behavior I observed, I do not think the idleTimeout applies to idle connections within the minimumIdle connections in the pool. The minimumIdle connections appear to be being managed by the maxLifetime or a cleanup cycle. It might be good to note that under idleTimeout if that is true.
The above two are just feedback, it they are off and not accurate then don't worry about them. But, I have been having debugging and have been trying to really understand what each of these settings are dong to work through the issue, that's where the feedback is coming from.
Thanks for the consideration on the above!

Can I ask this... what datasource or jdbcUrl I should be using with Percona, as I don't see it in the table? I am not sure if I should be leaning towards MySQL or MariaDB. Could/Should you add Percona to the table and identify if it can use a Datasource or needs to be jdbcUrl configuration? I was going the MySQL route, and once I came across this MariaDB Connector/J driver, I went this route and started having some success, but I am still not sure what specific change has brought that success, or if I should be going the direction of the MariaDB driver with Percona.
Here is what I have been going through. I would appreciate any comments you might have on any of this:
We have been running MariaDB local on CentOS7 with the com.mysql.jdbc.Driver, Hikari 2.4.1 a MaximumPoolSize=100 and minimumIdle default with no problems for a long time. The business is switching us to Percona. When switching to Percona we have run into two things, SSL warning, easy fix to the url, but the main issue is that we lose all connections around 10, 20, 80 or 90 minutes and they don't recover.
Switching to Percona I have been using the approach of MySQL from your table, and I have been using the Connector/J and jdbcUrl configuration with the com.mysql.cj.jdbc.Driver, Hikari 2.4.1 and tried 2.7.2. Is this the right approach or should I be taking another approach? I have been running into various issues where my pool blows up, and I get log errors like Failed to validate connection and Communications link failure. I am finding the documented solutions for these that are very clear, and making a lot of changes per all you have documented and other online resources, but the pool keeps blowing up at various times. I have a routine that gets a connection from the pool every minute and that is how I know that there are no connections around 10, 20, 80 or 90 minutes, which is the longest it has managed to keep connections. I was using all the default timeouts and a maximumPoolSize of 100 and minimumIdle 20. The database wait_timeout was the default 28800. I have verified time sync and my Google Cloud VM is running Crony.
The good news as of this morning... I seem to finally be able to run with the following configuration for 15 hours now, using one connection each minute, by doing the following, but I do not know what has resolved it just yet, I suspect either the larger VM, or taking maximumPoolSize from 100 to 20, or the MaxLifetime relative and the specific setting of 600 in the my.cnf file.
Current configuration with 15+ hours of up time now:
Increased my Google Cloud VM form micro to a small.
Hikari 2.7.2
Set MySQL default wait_timeout=600 and interactive_timeout=600
Using this Mariadb Connector/J driver
	HikariConfig config = new HikariConfig();
	config.setJdbcUrl(cfg.get("jdbc.url", "jdbc:mysql://123.123.123:3306/TestDB"));
	config.setUsername(cfg.get("jdbc.user", "xxx"));
	config.setPassword(cfg.get("jdbc.password", "xxx"));
	config.setConnectionTimeout(cfg.getInt("ConnectionTimeout", 10000));
	config.setIdleTimeout(cfg.getInt("IdleTimeout", 30000 ));
	config.setMaxLifetime(cfg.getInt("MaxLifetime", 570000  ));
	config.setConnectionTestQuery(cfg.get("ConnectionTestQuery", "SELECT 1"));
	config.setMinimumIdle(cfg.getInt("MinimumIdle", 5));
	config.setMaximumPoolSize(cfg.getInt("MaximumPoolSize", 20));
	config.setPoolName(this.getName());
	config.setRegisterMbeans(true);
	config.setCatalog(cfg.get("jdbc.catalog", ""));
	config.setDriverClassName(cfg.get("jdbc.driver", "com.mysql.jdbc.Driver"));
	config.addDataSourceProperty("cachePrepStmts", "true");
	config.addDataSourceProperty("prepStmtCacheSize", "250");
	config.addDataSourceProperty("prepStmtCacheSqlLimit", "2048");
	config.addDataSourceProperty("useServerPrepStmts", "true");

I know that I may not need some of these like ConnectionTestQuery and DriverClassName, but I have them there for assurance until I can narrow down exactly what is making my pool connections blow up.
Long, I know, but thanks for the help!

I have updated the documentation for idleTimeout and maxLifetime.
My understanding is that Percona is essentially MySQL.  I think you can use either the MariaDB driver or Connector/J.  Personally I favor the MariaDB driver as it carries less legacy baggage.  Because of previous issues with the DataSource implementations of both I would recommend using jdbcUrl based configuration.
Note properties such as cachePrepStmts are likely different from the Connector/J driver. And yes you do not need the connection test query.

@brettwooldridge do you happen to know if there is a compatibility guarantee of running the MariaDB driver with MySQL or it might break at some point in the future?

Of course there are no guarantees, but I doubt the MariaDB project wants to break wire compatibility with MySQL.

@rusher could I trouble you for your input too?

@johnou clearly, with an asterisk: no support for mysql protocol X.
Regarding the datasource problem, it has been corrected there must be 1 year and a half

@johnou The MariaDB blog states that the MariaDB JDBC driver is compatible with MySQL, but requires MySQL to be at least v5.5.3.

@rusher I am not sure which issue you think was corrected in the past 1.5 years, but the issue of MysqlDataSource not supporting login timeout still remains.
This means that a new connection creation that occurs during a network outage can hang indefinitely (8 hours on Linux by default).
It does appear that the MariaDB driver DataSource implementation corrected this issue.

I don't need a driver class name using jdbcURL, but I want to build in a config for it.  Can I set a default to Null or pass a bad driver name and Hikari will then behave like I did not set the driver class name?
config.setDriverClassName(cfg.get("jdbc.driver", NULL));
config.setDriverClassName(cfg.get("jdbc.driver", "Hikari.decides"));


Expand the options, move idmode into options, fix types

Refactored the SignedXml class to accept all options through a single options object, enhancing the flexibility and readability of the code. This change includes migrating the idMode argument into the options object for better consistency.
Also fixed some broken types, and updated README to be consistent with these changes.

@shunkica , I'm not sure why this is a draft PR, but it looks good so far. Thanks!

@shunkica , I'm not sure why this is a draft PR, but it looks good so far. Thanks!

I wanted to have another read through the readme before submitting.

@cjbarth go for it


Doesn't work with the Static Publisher module

This maybe should've been obvious to me before I tried it, but I was curious and went ahead anyway... We're using staticpublisher and presumably because of that, putting the site in Maintenance mode did absolutely nothing.
I'm guessing some kind of extra URL rewrite would be needed, but doing that as-needed would defeat the purpose of the one-click-ness of this module, and I'm not sure if the standard URL rewrites needed for staticpublisher are compatible with the way this module works.

Hi @xiehan, Thanks for bringing this to my attention.  I've never used the static publisher module, but I'll definitely look into it.

@xiehan This module now produces a static error page in the assets directory on dev/build
assets/error-503.html
There are 2 URL rewrite rules in the .htaccess file for 404 and 500 error pages:
ErrorDocument 404 /assets/error-404.html ErrorDocument 500 /assets/error-500.html
Presumably, you would have to add another for the error-503.html file:
ErrorDocument 503 /assets/error-503.html
Does this help?

@xiehan - After discussing this today with @sminnee, @dhensby and @zauberfisch at the 2015 SilverStripe European Conference (https://stripecon.eu), we concluded that you are, indeed, correct - this kind of module will not work with static publisher.  Additional rewrite rules would be required and possibly more development to make it possible.  However, this is outside of the scope of the use case for which this module was developed.  So, closing as a non-issue.


Add my name

I'm not sure about the formatting changes, but can take a look if it's a problem!

I think your IDE removed blank spaces, which is good 👍 . Thank you for the contribution 🤓


Command failed when i specified both video and audio format

Command failed anything i added both the video and audio format
this is the output of the error

(node:21565) UnhandledPromiseRejectionWarning: TypeError [ERR_INVALID_ARG_TYPE]: The "url" argument must be of type string. Received undefined
at validateString (internal/validators.js:118:11)
at Url.parse (url.js:159:3)
at Object.urlParse [as parse] (url.js:154:13)
at processData (/home/derrick/projects/youtube-cli-dowloader/node_modules/youtube-dl/lib/youtube-dl.js:42:17)
at getInfo (/home/derrick/projects/youtube-cli-dowloader/node_modules/youtube-dl/lib/youtube-dl.js:106:9)
at done (/home/derrick/projects/youtube-cli-dowloader/node_modules/youtube-dl/lib/youtube-dl.js:259:12)
at done (/home/derrick/projects/youtube-cli-dowloader/node_modules/youtube-dl/lib/youtube-dl.js:25:12)
at /home/derrick/projects/youtube-cli-dowloader/node_modules/universalify/index.js:23:46
at processTicksAndRejections (internal/process/task_queues.js:97:5)
(node:21565) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:21565) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

with following code
const videoDownloading = youtube('https://www.youtube.com/watch?v=j5-yKhDd64s',                 [-f 160+140],                 { cwd: path.join(getRootDir(),'.youtube-downloader')});

I think this is a duplicate of #221 .

okay


Can't connect peers on in-browser environment

Hi, I'm trying to build a simple test app with OrbitDB, relying only on the browser. This is my code:
import { createOrbitDB } from '@orbitdb/core';
import { createLibp2p } from 'libp2p';
import { createHelia } from 'helia';
import { identify } from '@libp2p/identify';
import { webSockets } from '@libp2p/websockets';
import { webRTC } from '@libp2p/webrtc';
import { all } from '@libp2p/websockets/filters';
import { noise } from '@chainsafe/libp2p-noise';
import { yamux } from '@chainsafe/libp2p-yamux';
import { gossipsub } from '@chainsafe/libp2p-gossipsub';
import { circuitRelayTransport } from '@libp2p/circuit-relay-v2';

const customConfig = {
    addresses: {
        listen: ['/webrtc']
    },
    transports: [
        webSockets({
            filter: all
        }),
        webRTC(),
        circuitRelayTransport({
            discoverRelays: 1
        })
    ],
    connectionEncryption: [noise()],
    streamMuxers: [yamux()],
    connectionGater: {
        denyDialMultiaddr: () => false
    },
    services: {
        identify: identify(),
        pubsub: gossipsub({
            allowPublishToZeroTopicPeers: true,
            emitSelf: true
        })
    }
};

export const initNewDb = async (name) => {

	const libp2p = await createLibp2p({ ...customConfig });
	const ipfs = await createHelia({ libp2p });
	const orbitdb = await createOrbitDB({ ipfs });
	let mydb = await orbitdb.open(name, { type: 'documents' });

        //return db address
	console.log(mydb.address);
        const addr =mydb.address;
        return addr;
}

export const openDb = async (address) => {
        console.log("opening existing db ", address)
        const libp2p = await createLibp2p({ ...customConfig });
	const ipfs = await createHelia({ libp2p });
        const orbitdb = await createOrbitDB({ ipfs });
        console.log("just before open");
        let mydb = await orbitdb.open(address);

        // OPEN DB AND READ DOCS
        console.log("opened mydb", mydb);
        for await (const record of mydb.iterator()) {
            console.log(record)
         }

}

Basically I use the initNewDB function to create a new db. In another piece of code I'm putting some documents in and everything works as expected.
In another browser instance I'm running the openDb function with the address of the first instance as parameter (e.g. /orbitdb/zdpuAkcwe7f3qrHSwmqKeUa55RJVC4pJxYv6YbYCSVypXUgGc). The function hangs for a while on
let mydb = await orbitdb.open(address);

and then in the browser console i get:
Uncaught (in promise) AggregateError: All promises were rejected

the error itself doesn't provide much information and I'm struggling to find some working examples for my case. Is there any configuration problem or is it a bug?
Thanks

If you are opening a remote db, the error indicates that the remote db cannot be reached.
In a browser environment, peers cannot find each other directly, they need a relay to help each browser peer find the other.
You can find more information about peer connections in our docs and also within Libp2p's own documentation.
The easiest way to deploy a relay node for OrbitDB is via this code base. Once you have a local copy, run:
npm i
npm run

You will see that one of the options runs a webrtc server.
npm run webrtc

A number of addresses will be printed out. Have your browser peers use one of these addresses to aid in the connection process. Again, the browser-to-browser connection documentation should help walk you through a basic process.
Unfortunately, the connection process for Libp2p and, by extension, OrbitDB is laborious but we hope to make this easier with projects such as orbitdb/quickstart and orbitdb/voyager. However, with limited time and budget, some of these features will take time to develop.

You can refer to this libp2p official WebRTC article, the browser needs to connect through the relay node, by referring to this article to improve your libp2p configuration, you can solve this problem.
https://docs.libp2p.io/guides/getting-started/webrtc/

Closing this due to inactivity but feel free to re-open if you are still having issues.


int2binary does not return array

From narray exercise
Bit conversion does not return an array unlike in numpy
Python:
# Author: Daniel T. McDonald

I = np.array([0, 1, 2, 3, 15, 16, 32, 64, 128], dtype=np.uint8)
bin = np.unpackbits(I[:, np.newaxis], axis=1)

print bin[0] returns
array([0, 0, 0, 0, 0, 0, 0, 0], dtype=uint8)
Ruby:

i = Numo::UInt8[0, 1, 2, 3, 15, 16, 32, 64, 128]
bin = Numo::Bit.from_binary(i.to_binary,[i.size,8]).reverse(1)

print bin[0] returns
0
It should be an array and not just an element at a particular index.

NArray is not always same as numpy.
If the number of arguments in NArray's [] method is one, it behaves as indexing on a flattened 1-d array.
For 2-d array, bin[0] in numpy corresponds to bin[0,true] in NArray.
> a = Numo::NArray[[1,2,3],[4,5,6],[7,8,9]]
=> Numo::Int32#shape=[3,3]
[[1, 2, 3],
 [4, 5, 6],
 [7, 8, 9]]

> 9.times{|i| p a[i]}
1
2
3
4
5
6
7
8
9

> 3.times{|i| p a[i,true]}
Numo::Int32(view)#shape=[3]
[1, 2, 3]
Numo::Int32(view)#shape=[3]
[4, 5, 6]
Numo::Int32(view)#shape=[3]
[7, 8, 9]

Hey
Its a bit tricky to extract elements. Suppose if I need the first column and do, a[true, 0], it would return
=> Numo::Int32(view)#shape=[3]
[1, 4, 7]

It changes it shape to a row.
This becomes difficult where if an array is initialized with zeros, and every column needs to be updated with new values.

How to correct this?

irb(main):183:0> @K = Numo::DFloat.zeros(4, 4)
=> Numo::DFloat#shape=[4,4]
[[0, 0, 0, 0],
 [0, 0, 0, 0],
 [0, 0, 0, 0],
 [0, 0, 0, 0]]
irb(main):184:0> @K[true,0] = Numo::NArray[[ 0.0769], [ 1.4713], [ 0.1293], [ 1.9335]]
=> Numo::DFloat#shape=[4,1]
[[0.0769],
 [1.4713],
 [0.1293],
 [1.9335]]
irb(main):185:0> @K
=> Numo::DFloat#shape=[4,4]
[[1.9335, 0, 0, 0],
 [1.9335, 0, 0, 0],
 [1.9335, 0, 0, 0],
 [1.9335, 0, 0, 0]]

Second issue I am facing is Transpose would not work in such slicing.

irb(main):203:0> a[0,true]
=> Numo::Int32(view)#shape=[3]
[1, 2, 3]
irb(main):204:0> a[0,true].transpose
=> Numo::Int32(view)#shape=[3]
[1, 2, 3]

EXPECTING =>
[[1],
[2],
[3]]

Yes, [] method possibly changes the array shape.
It trims axes where scalar index is specified.
Numpy also has the same feature.
@K = Numo::DFloat.zeros(4, 4)
=> Numo::DFloat#shape=[4,4]
[[0, 0, 0, 0],
 [0, 0, 0, 0],
 [0, 0, 0, 0],
 [0, 0, 0, 0]]

@K[true,0] = Numo::NArray[[ 0.0769], [ 1.4713], [ 0.1293], [ 1.9335]]
@K
=> Numo::DFloat#shape=[4,4]
[[1.9335, 0, 0, 0],
 [1.9335, 0, 0, 0],
 [1.9335, 0, 0, 0],
 [1.9335, 0, 0, 0]]

This behavior above is a bug. It should raise an exception. I will fix it.
Below is a right behavior.
@K[true,0] = Numo::NArray[0.0769, 1.4713, 0.1293, 1.9335]
@K
=> Numo::DFloat#shape=[4,4]
[[0.0769, 0, 0, 0],
 [1.4713, 0, 0, 0],
 [0.1293, 0, 0, 0],
 [1.9335, 0, 0, 0]]

If you want to keep axis, use range index and array index as below.
a = Numo::NArray[[1,2,3],[4,5,6],[7,8,9]]
=> Numo::Int32#shape=[3,3]
[[1, 2, 3],
 [4, 5, 6],
 [7, 8, 9]]

a[true,0]
=> Numo::Int32(view)#shape=[3]
[1, 4, 7]

a[true,[0]]
=> Numo::Int32(view)#shape=[3,1]
[[1],
 [4],
 [7]]

a[true,[0]].transpose
=> Numo::Int32(view)#shape=[1,3]
[[1, 4, 7]]

Hey
Is there a sampling from binomial distribution like so
np.random.binomial([np.ones((5,10))], 1-0.5)[0] * (1.0/(1-0.5))

array([[ 0.,  2.,  2.,  0.,  0.,  0.,  2.,  2.,  2.,  0.],
       [ 0.,  2.,  2.,  0.,  2.,  2.,  0.,  2.,  2.,  0.],
       [ 0.,  2.,  2.,  2.,  2.,  2.,  0.,  2.,  2.,  0.],
       [ 2.,  0.,  2.,  0.,  2.,  0.,  2.,  0.,  0.,  2.],
       [ 0.,  2.,  2.,  0.,  0.,  0.,  0.,  0.,  0.,  0.]])

In ruby the closest I got is
Numo::DFloat.new(Numo::DFloat.ones(5, 10).shape).rand(1-0.5) * (1.0/(1-0.5))

=> Numo::DFloat#shape=[5,10]
[[0.476774, 0.924088, 0.520622, 0.204563, 0.412933, 0.949026, 0.510621, ...],
 [0.551107, 0.772665, 0.759988, 0.707775, 0.545644, 0.729394, 0.793608, ...],
 [0.469024, 0.231555, 0.815419, 0.187351, 0.263405, 0.447322, 0.566199, ...],
 [0.270495, 0.0656589, 0.37876, 0.16591, 0.760717, 0.279128, 0.954446, ...],
 [0.633033, 0.557028, 0.873324, 0.742393, 0.17079, 0.250819, 0.609134, ...]]

Numo::GSL module has binomial method.
require 'numo/gsl'

rng = Numo::GSL::Rng::Mt19937.new

rng.binomial(1-0.5, 1, [5,10]) * (1.0/(1-0.5))
=> Numo::DFloat#shape=[5,10]
[[2, 0, 0, 2, 0, 0, 2, 2, 2, 2],
 [2, 2, 0, 2, 2, 0, 0, 0, 0, 0],
 [0, 0, 0, 2, 2, 2, 2, 0, 2, 0],
 [0, 0, 0, 2, 2, 2, 2, 0, 2, 0],
 [0, 2, 0, 0, 0, 0, 2, 0, 0, 2]]

Note that Numo::GSL is under development and gem is not provided yet.

Hey
Thanks for that heads up.
This morning I found this gem.
After defining an arr of dimension 5 and 10 filled with a random binomial number, I do this
irb(main):073:0> Numo::DFloat[arr].reshape(5,10) * (1.0/(1-0.5))
=> Numo::DFloat#shape=[5,10]
[[0, 0, 0, 2, 2, 2, 0, 2, 2, 0],
 [0, 2, 0, 2, 2, 2, 2, 2, 2, 0],
 [0, 2, 2, 2, 2, 2, 0, 0, 0, 2],
 [0, 0, 2, 0, 0, 0, 0, 2, 2, 0],
 [0, 2, 0, 0, 2, 2, 0, 0, 2, 0]]

Works for the moment.

I have to say I am really thank you for your awesome support since opening this ticket.

It's my pleasure.

I have a bit array (which were a result of s = (syn>0) and p = (prev>0))
irb(main):152:0> s
=> Numo::Bit#shape=[38,20]
[[0, 1, 1, 0, 1, 0, 1, 1, 1, 1, 1, 1, 0, 1, 0, 1, 1, 0, 1, 0],
 [1, 1, 0, 1, 0, 1, 0, 0, 0, 0, 0, 0, 1, 1, 1, 0, 0, 1, 0, 1],
 [0, 1, 0, 0, 1, 1, 1, 0, 0, 1, 1, 1, 0, 1, 0, 1, 1, 0, 1, 0],
 [0, 1, 1, 1, 0, 1, 1, 1, 1, 1, 0, 1, 1, 0, 0, 0, 0, 0, 0, 1],
 [1, 1, 0, 0, 0, 1, 1, 0, 0, 0, 1, 0, 0, 1, 1, 1, 1, 0, 0, 0],
 ...
irb(main):153:0>

irb(main):153:0> p
=> Numo::Bit#shape=[38,20]
[[0, 1, 1, 0, 1, 1, 1, 1, 1, 1, 1, 1, 0, 1, 0, 1, 1, 0, 1, 0],
 [1, 1, 0, 1, 0, 1, 0, 0, 0, 0, 0, 0, 1, 1, 1, 0, 0, 1, 0, 1],
 [0, 1, 0, 0, 1, 1, 1, 0, 0, 0, 1, 1, 0, 1, 0, 0, 1, 0, 1, 0],
 [0, 1, 1, 1, 0, 1, 1, 1, 1, 1, 0, 1, 1, 0, 0, 0, 0, 0, 0, 1],
 [1, 1, 0, 0, 0, 1, 1, 0, 0, 0, 1, 0, 0, 1, 0, 1, 1, 0, 0, 0],
 ...
irb(main):154:0>

I am trying to do this
irb(main):154:0> s - p
NoMethodError: undefined method `-' for #<Numo::Bit:0x8210e780>
	from (irb):154
	from /home/arjun/.rbenv/versions/2.2.3/bin/irb:11:in `<main>'
irb(main):155:0>

What is the correct minus operator here?

Bit array does not support arithmetic operation because its elements are either 0 or 1.
Bit array supports bit-wise operation instead:
s & p
s | p
s ^ p

or, You can cast it to integer array as
i = Numo::Int32.cast(s)


Add max_pool_connections to client config

This config value allows you to control the size of the
connection pools used in urllib3.  We were previously using the
hardcoded value of 10 (from requests).
I ended up not touching the pool_connections based on the
discussion from #766.
There ends up being quite a noticeable difference whenever
num_threads > max_pool_connection, so I think this will
help people running requests with a large number of threads.
As an aside, the implementation for this was much more complicated
than necessary.  I'm thinking on sending a few follow up PRs that
start to clean up some of this technical debt.
Closes #773
Closes #766
cc @kyleknap @JordonPhillips

Current coverage is 97.57% (diff: 100%)

Merging #1026 into develop will increase coverage by <.01%

@@            develop      #1026   diff @@
==========================================
  Files            44         44
  Lines          6884       6892     +8
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6717       6725     +8
  Misses          167        167
  Partials          0          0

Powered by Codecov. Last update 0917f55...b815df3

Looks good. Straightforward change. Just had a question about the existing name.

@kyleknap renamed.

Thanks. Looks good. :ship:

Close the loop... could you link to docs on this and how to use it from boto3?


[BUG] Login Page Issue

Describe the bug

Login page inputs have padding issues.

To Reproduce
Steps to reproduce the behavior:

Go to https://anshika-26.github.io/WordWise/login.html
See error

Expected behavior
Standard styling should be maintained.
Screenshots

Additional context
Please add hactoberfest and GSSoC24 tag. You can assign me for this issue.

wrong page again :) check it

this is our design

wrong page again :) check it . this is our design

Check this https://anshika-26.github.io/WordWise/login.html

as said in the signup issue , this is a stray file , we plan to remove it soon , its not linked to our main page , kindly navigate to https://anshika-26.github.io/WordWise/
click profile
click login

in footer sectiion. SIGN ME UP

okay , thanks for pointing out , this is a stray file and will be removed from this link soon !


Add findFirst utility method

Is your feature request related to a problem? Please describe.
When you want to find a value just by the key, but don't care about the structure, you most likely write a recursive method to do that for you. So I think it would be good to include this utility method into the project.
Describe the solution you'd like
Basically a recursive method for both AbstractObject and AbstractArray.
AbstractElement findFirst(String key)
The method should be on AbstractArray as well, because an AbstractArray could contain objects which contain the key I want to search for.

This could be implemented as an extension to the query method, maybe by adding a wildcard placeholder to it


docs(list.stories.tsx): add list with icon storybook

[x] I have followed the Your First Code Contribution section of the Contributing guide

add list with icon storybook
There are no breaking API changes.

Summary by CodeRabbit

New Features

Added a new story showcasing a list with icons in the List component documentation.

@SutuSebastian Hi Seba, pls check this one...

I think the best here would be to re-create the PR, since the main branch changed drastically and it has too many file changes, when actually it should have 1-2.

Closed by #1316.


Update install.md

Changes
Fixes typo in install.md
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x ] Includes docs (if user facing)

I signed it!

/ok-to-test

looks like it timed out, trying again.
/retest

/lgtm

/approve


feat(protocol): Send module information when smuggling errors.

BREAKING CHANGE: New protocol version.
Goes with https://github.com/Klafyvel/nvim-smuggler/pull/50.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 39.48%. Comparing base (7f24d17) to head (17714b3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   37.80%   39.48%   +1.67%
==========================================
  Files           7        7
  Lines         373      385      +12
==========================================
+ Hits          141      152      +11
- Misses        232      233       +1

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


Configure publishing for Android AAR via Maven

Summary
Configure publishing for Android AAR via Maven.

Updates gradle wrapper to 7.3
Allows Native Android SDK to be distributed via MavenCentral

How did you test your changes?
manually

#602

@dependabot rebase


fix broken link in docs

This fixes issue #5978

Can one of the admins verify this patch?

I signed it!


Fix people holding hands

Because of two errors, half of the "people holding hands" emoji sequences are broken:

The first commit addresss a bug in the code generating rtl sequences, the second adds aliases for the remaining emoji. The result is now:

Sigh... wrong email address used for commit