[coverage] RUST_MIN_STACK

Motivation
Attempt to prevent tests from overflowing the stack under code coverage
Have you read the [Contributing Guidelines on pull requests]
y
Test Plan
CI
Related PRs
none

Codecov Report

Merging #4946 into master will increase coverage by 0.6%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #4946     +/-   ##
=========================================
+ Coverage    38.8%    39.4%   +0.6%
=========================================
  Files         852      852
  Lines      105030   105030
  Branches    49865    49865
=========================================
+ Hits        40822    41473    +651
+ Misses      24350    23081   -1269
- Partials    39858    40476    +618

Impacted Files
Coverage Δ

...guage/bytecode-verifier/src/instantiation_loops.rs
65.6% <0.0%> (-0.9%)
:arrow_down:

language/move-prover/spec-lang/src/ast.rs
57.0% <0.0%> (-0.5%)
:arrow_down:

mempool/src/shared_mempool/tasks.rs
29.5% <0.0%> (-0.4%)
:arrow_down:

json-rpc/src/tests/unit_tests.rs
23.2% <0.0%> (ø)

storage/libradb/src/state_store/mod.rs
53.0% <0.0%> (ø)

language/move-lang/src/expansion/aliases.rs
57.6% <0.0%> (ø)

language/compiler/ir-to-bytecode/src/compiler.rs
36.4% <0.0%> (+<0.1%)
:arrow_up:

language/move-lang/src/to_bytecode/translate.rs
48.3% <0.0%> (+0.1%)
:arrow_up:

language/move-lang/src/naming/ast.rs
18.7% <0.0%> (+0.3%)
:arrow_up:

types/src/proof/definition.rs
64.3% <0.0%> (+0.3%)
:arrow_up:

... and 39 more

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 2e4748f...e85e144. Read the comment docs.

Sweet, a clean coverage run:   https://app.circleci.com/pipelines/github/libra/libra/23057/workflows/458d6b7a-563c-4ea4-bf12-57cbcc84ca80/jobs/145434/steps

@bors-libra r+

:pushpin: Commit 5d01620 has been approved by rexhoffman

:hourglass: Testing commit 5d01620ce6fe88828be90dddcfd6f1bd12f548dc with merge ff9f6b53bfa009994e4c697d843e531df95620f1...

:bulb: This pull request was already approved, no need to approve it again.

This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

:pushpin: Commit 5d01620 has been approved by sausagee

:hourglass: Testing commit 5d01620ce6fe88828be90dddcfd6f1bd12f548dc with merge 5aafb23e715805051bef537561f6daae3b9c145a...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: sausagee
Pushing 5aafb23e715805051bef537561f6daae3b9c145a to master...


chore: update docs to represent zksync supporting necessary precompiles

Since its latest v24 upgrade, zkSync supports ecMul, ecAdd, and ecPairing.
The verifier compiles and deploys on zkSync Sepolia Testnet
Description
Problem*
Resolves
Summary*
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Thanks!


[SPARK-31107][CORE]Extend FairScheduler to support pool level resource isolation

What changes were proposed in this pull request?
Currently, spark only provided two types of scheduler: FIFO & FAIR, but in sql high-concurrency scenarios, a few of drawbacks are exposed.
FIFO: it can easily causing congestion when large sql query occupies all the resources
FAIR: the taskSets of one pool may occupies all the resource due to there are no hard limit on the maximum usage for each pool. this case may be frequently met under high workloads.
So we propose to add a maxShare argument for FairScheduler to control the maximum running tasks for each pool.
Why are the changes needed?
Improvements
Does this PR introduce any user-facing change?
No
How was this patch tested?
existing UT

Can one of the admins verify this patch?

Can one of the admins verify this patch?


issues with installation

on CentOS 6.7 npm install returns :
npm WARN EPEERINVALID<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Linux 2.6.32-573.12.1.el6.x86_64
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v5.4.1
npm ERR! npm  v3.3.12
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the bcrypt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls bcrypt
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /root/code/drywall/npm-debug.log

kerberos can be install separately :
npm install kerberos
bcrypt fails but can be replaced with bcryptjs.
<EMAIL_ADDRESS>
seems to work
code that needs changing :  schema/User.js
The require bcrypt should now point to bcryptjs
 userSchema.statics.encryptPassword = function(password, done) {
    var bcrypt = require('bcryptjs');
    bcrypt.genSalt(10, function(err, salt) {
      if (err) {
        return done(err);
      }

      bcrypt.hash(password, salt, function(err, hash) {
        done(err, hash);
      });
    });
  };
  userSchema.statics.validatePassword = function(password, hash, done) {
    var bcrypt = require('bcryptjs');
    bcrypt.compare(password, hash, function(err, res) {
      done(err, res);
    });
  };

I am quite new at node but these solutions helped me so I thought to share here.
Thanks

Thanks for creating an issue.
I have not tested with node v5.x and npm v3.x. kerberos may need to be added to package.json.
There is a wiki page for bcrypt issues that mentions bcryptjs:
https://github.com/jedireza/drywall/wiki/bcrypt-Installation-Trouble
I have limited time to test in this exact environment. Please update this thread with your findings or solutions you find.


specifying _pd_read_kwds for unemployment and zipcodes datasets

Addressing issue #16
This commit specifies the separator (tab) for the unemployment data and dtype for zip_code variable in zipcodes dataset.

Thanks!
Looking at the unemployment data, that should have worked automatically because it is a tsv file. Turns out there was a bug in tsv handling that I just pushed a fix & test for: #18 So you can remove the Unemployment class; it should work without it now.
Regarding zipcodes: is there a reason to prefer categories rather than strings in this case?

Great! I don't have a strong opinion on categorical vs. string. I just think that ZIP codes are categories in nature (maybe because I'm using it as a categorical variable in regression etc.). But I can see that ZIP codes are hierarchically organized and thus also makes sense to have it as a string.

I think we should switch to strings then... I think categories make sense when you have many instances within each category. Here there is only a single instance of each category, and so strings are slightly more efficient.

Looks great! Could you also add a test of the zipcode data type?

Merged master and added a minor fix to test_unemployment_tsv().

Thanks!


lib/customisation: add NoMkOverridable variants of callPackage

in some places we want to use callPackage but don't want to inject the makeOverridable attrs so far import with manual inherits has been used
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

IIUC, the purposes of callPackage are all of:

Using splicedPackages (as opposed to unspliced)
Adding overriding

specifically override
also mkDerivation overriding attrs, but only out of necessity, for multi layer overriding to be somewhat coherent.

The same thing that lazyFunction does (#194514)

So by removing overriding, it seems that you could simplify all this by leaving callPackage untouched and instead implementing this variation with lazyFunction and splicedPackages instead. callPackage is already rather complicated, so that would be much nicer.


Rendering at the beginning may have a frozen video

https://twitter.com/aceCourtBot/status/1386958989557125123?s=19

https://github.com/c0decracker/video-splitter/issues/9


Repeater's schema method does not bring the current record

Package
filament/filament
Package Version
3.2.115
Laravel Version
11.9
Livewire Version
3.0
PHP Version
8.2.16
Problem description
It is not possible to access the current item within the schema method in the Repeater, and the entire Repeater state of all items is returned.
->itemLabel(function (array $state) {
                        return new HtmlString(
                            view('filament.components.service-repeater-item')
                                ->with('state', $state)
                                ->render()
                        );
                    })

If you print the $state inside the itemLabel method, you will see something like:
array:3 [▼ // app/Livewire/Opportunity/OpportunityQuote.php:39
  "icon" => "heroicon-m-archive-box"
  "name" => "Move"
  "description" => "Sea, Air, Road, Storage"
]

Expected behavior
When using:
->schema(function (array $state) {
        dd($state);

The idea was that only the current item would be returned so that it could be used in dynamic components, or that at least the uuid of the current item would be included, so that it could be accessed.
If you print $state, you will see something like:
array:1 [▼ // app/Livewire/Opportunity/OpportunityQuote.php:46
  "efa34400-a95d-4410-a695-e293184ecd3b" => array:3 [▼
    "icon" => "fas-paw"
    "name" => "Pet Transportation"
    "description" => "Dog, Cat, etc..."
  ]
]

So, I need to somehow do the following action:
->schema(function (array $state) {
                        return [
                            // Livewire::make(OpportunityQuoteItem::class, [
                            //     'item' => $state
                            // ]),
                        ];
                    })

Or, do we have some other way to send the data to a custom Livewire component within Repeater?
In the documentation https://filamentphp.com/docs/3.x/forms/advanced#injecting-the-current-state-of-a-field it says that the array $state would give me access to the item current , not the entire state.
Steps to reproduce
Crie um Repeater assim:
Repeater::make('quote_services_repeater')
                    ->label('Serviços Adicionados')
                    ->itemLabel(function (array $state) {
                        dump($state);
                        return new HtmlString(
                            view('filament.components.service-repeater-item')
                                ->with('state', $state)
                                ->render()
                        );
                    })
                    ->schema(function (array $state) {
                        dump($state);

                        return [

                            // Livewire::make(OpportunityQuoteItem::class, [
                            //     'item' => $state
                            // ]),
                        ];
                    })

Tente acessar a última linha adicionada dentro do método schema da forma correta, para que funcione durante a renderização etc
Reproduction repository (issue will be closed if this is not valid)
https://github.com/stuartcusackie/filament-wizard-header-scroll-issue
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

The schema() method belongs to the Repeater object and not an Item object. In face, the Item object does not even exist for this behaviour. For itemLabel() we deliberately only pass the current state.
Changing this would break behaviour, and disallow people from accessing data in other items.
I see you're trying to pass dynamic data into the LW component, in this case instead of injecting $state on the schema, pass a function to the second arg of Livewire::make(), where you can inject $get which is relative to the current repeater item.


Failed running 'az aks enable-addons'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks enable-addons
Errors:
argument of type 'NoneType' is not iterable
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 608, in execute
    raise ex
cli/core/commands/__init__.py, ln 666, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 659, in _run_job
    six.reraise(*sys.exc_info())
lib/python3.8/site-packages/six.py, ln 703, in reraise
    raise value
...
azure/cli/core/__init__.py, ln 493, in default_command_handler
    return op(**command_args)
cli/command_modules/acs/custom.py, ln 1846, in aks_enable_addons
    _ensure_container_insights_for_monitoring(cmd, instance.addon_profiles['omsagent'])
cli/command_modules/acs/custom.py, ln 2496, in _ensure_container_insights_for_monitoring
    if 'loganalyticsworkspaceresourceid' in addon.config:
TypeError: argument of type 'NoneType' is not iterable

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks enable-addons --addons {} --resource-group {} --name {}

Expected Behavior
Environment Summary
macOS-10.15.3-x86_64-i386-64bit
Python 3.8.1
Shell: bash

azure-cli 2.0.81

Additional Context

thanks for reporting the issue, we'll look at it.

We believe a fix has been made, please reference this issue for the latest tracking. I am going to close this to consolidate info to one thread. If you still hit issues after the S166 milestone release, feel free to recomment.
https://github.com/Azure/azure-cli/issues/11456


Demo User: UI is frozen when Demo user/pass is passed to the URL

<EMAIL_ADDRESS>

Not reproducible


Build for the Mac?

Comment:
Is there a reason this only supports Linux?
Or just that it needs someone to actually do the work to set it up?
Windows also -- but I'm betting that's harder.

I jsut tried qluickly building f rom source on my Mac -- and it barfed out -- so it will take a bit of work. Though hopefully only a little, the configure script does seem to attempt to support the Mac.

Thank you for the suggestion.
The main reason why the OSX build is not enabled is that I have no client to test it.
I've pushed the changes to enable it and you should be able to install it soon.
Could you please update the issue with a report if it's working?
I'll take look if I can enable Windows, but this should be another issue.

It's working for me - minimally tested, but at least a coupole commands seem to work.
Thanks!

Note: I submitted a PR for building for ARM -- it's fialing, but it looks like it's failing on building the man pages -- so hopefully not a huge deal to fix -- though I can't say I know what else to try.
maybe ping the conda-forge team?


Somalaya/release/17.2.1

Hi @somalaya
Could you please check with the team about this pull request and include the fix in the following release?
https://github.com/AzureAD/microsoft-authentication-library-common-for-android/pull/2274
I created this pull request and I'm waiting for review from the team.
Thanks for the answer!


Detect and show Thrust build version

Companion PR of 03f4964 from #2584. Starting CUB & Thrust 1.9.9 they are version-locked with each other (in theory, explained below), so this could help detect/debug if there's any incompatibility issue.

Thrust and CUB have a circular dependency. However, if we use CUB alone, there's a good chance it's decoupled from Thrust, unless iterators are used (see https://github.com/NVlabs/cub/issues/182#issuecomment-631272979). I tested with CuPy that the latest CUB 1.9.10 works with an old Thrust from CUDA 10.0, so I don't think there's any concern here, and this is just for richer config output.

pfnCI, test this please.

Jenkins CI test (for commit 0fa2e54433bb42b6e3596209c8897121f1893922, target branch master) failed with status FAILURE.

Would you rebase this on the current master as we need #3469 to pass CI to follow the latest SciPy release?

pfnCI, test this please.

Jenkins CI test (for commit 0fa2e54433bb42b6e3596209c8897121f1893922, target branch master) succeeded!

LGTM!

Thanks, @takagi @kmaehashi!


feat!: add filtering function in StargateMsg of wasm

Description

Add filtering function in StargateMsg of wasm
Current deny list is lbm.fswap.v1.* and lbm.fbridge.v1.*.
Motivation and context

How has this been tested?

Screenshots (if appropriate):
Checklist:

[x] I followed the contributing guidelines and code of conduct.
[x] I have added a relevant changelog to CHANGELOG.md
[x] I have added tests to cover my changes.
[ ] I have updated the documentation accordingly.

Would you add a comment or test pattern explaining why CustomMsg should not be checked?
https://github.com/Finschia/wasmvm/blob/v1.1.1-0.12.0/types/msg.go#L101


Update esp-idf-sys to 0.35.0

Updated esp-idf-sys to 0.35.0

Thank you. Have you tried this? I currently don't have the time to test this out on hardware.

I have pulled esp-ota locally and also changed the esp-idf-sys version to 0.35.0 and it built fine.
My MWE code for BT OTA updating seemingly also worked fine with it.
So if you find the time to update it upstream that would be great!
Thanks and kind regards,
Riesi

Sorry, I didn't see this PR before adding one myself. Confirmed 0.35 works on an Arduino Nano esp32s3. I added a couple other minor fixes in my PR as well. Thanks!


Update the troubleshooting gude regarding the toolchain version for LTS

Error reported here; replacing the toolchain may or may not help, but more information is needed.
1.68.2-aarch64-apple-darwin with rustc 1.68.2 (9eb3afe9e 2023-03-27) seems to be the stable toolchain.
All stable toolchains should be working with an LTS branch, but a code update is needed.

Closing, since it was fixed in https://github.com/hyperledger/iroha/pull/3404 according to @Arjentix. I propose a discussion of the toolchain-related updates as a method of troubleshooting, because this is probably not the last similar issue.


Get rid of gravmag forward modeling Cython modules

Replace the Cython gravmag forward modeling modules by their numpy counterparts. Benchmarks show that the numpy code is almost as fast. The speed gain is not enough to justify the burden of compiling code and all the hell that comes with that on Windows.
Also, the sphere.gz numpy version can be made ~4.5x faster than the Cython code by switching the r3 = (dx**2 + dy**2 + dz**2)**1.5 by r = np.sqrt(dx**2 + dy**2 + dz**2); r3 = r*r*r (square-root and multiplying is faster than a single pow). Goes to show that the Cython code for this is surely premature optimization.
Move things to a gravmag.forward package. Will have all kernels there and functions to calculate for general models with different objects. Kernel functions calculate on 1 element only. This is useful for
inversion and eliminates the loop over a single element in Jacobian matrix building.
Will need better tests now that I don't have two implementations to compare (good thing).
Checklist:

[ ] Make tests for new code (at least 80% coverage)
[ ] Create/update docstrings
[ ] Include relevant equations and citations in docstrings
[ ] Docstrings follow the style conventions
[ ] Code follows PEP8 style conventions
[ ] Code and docs have been spellchecked
[ ] Include new dependencies in doc/install.rst, requirements.txt, environment.yml, and README.rst
[ ] Documentation builds properly (run make docs locally)
[ ] Changelog entry (leave for last)

Benchmarking code used to test the new implementation.
Prism
Code used for the master branch Cython implementation:
# Get processor information
tmp = !cat /proc/cpuinfo | grep "model name"
processor = tmp[0].split(':')[1].strip()
print(processor)
# Make a model for testing
from __future__ import division, print_function
import numpy as np
from fatiando import gridder, utils
from fatiando.mesher import PrismMesh
model = PrismMesh((-200, 200, -200, 200, 0, 400), (10, 10, 10))
model.addprop('density', np.ones(model.size))
model.addprop('magnetization', utils.ang2vec(2*np.ones(model.size), 25, -10))
inc, dec = -30, 50
x, y, z = gridder.regular((-500, 500, -500, 500), (50, 50), z=-1)
print('Model size: {}'.format(model.size))
print('Grid size: {}'.format(x.size))
# Time the forward modeling of gravity, gradients and mag
from fatiando.gravmag import prism
print('Times:')
print('   gz: ', end='')
%timeit prism.gz(x, y, z, model)
print('   gzz: ', end='')
%timeit prism.gzz(x, y, z, model)
print('   tf: ', end='')
%timeit prism.tf(x, y, z, model, inc, dec)

Output from master branch:
Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz
Model size: 1000
Grid size: 2500
Times:
   gz: 1 loop, best of 3: 3.76 s per loop
   gzz: 1 loop, best of 3: 1.71 s per loop
   tf: 1 loop, best of 3: 6.58 s per loop

Sphere
Code used for the master branch Cython implementation:
# Get processor information
tmp = !cat /proc/cpuinfo | grep "model name"
processor = tmp[0].split(':')[1].strip()
print(processor)
# Make a model for testing
from __future__ import division, print_function
import numpy as np
from fatiando import gridder, utils
from fatiando.mesher import PointGrid
model = PointGrid((-200, 200, -200, 200), 100, (50, 50))
model.addprop('density', np.ones(model.size))
model.addprop('magnetization', utils.ang2vec(2*np.ones(model.size), 25, -10))
inc, dec = -30, 50
x, y, z = gridder.regular((-500, 500, -500, 500), (100, 100), z=-1)
print('Model size: {}'.format(model.size))
print('Grid size: {}'.format(x.size))
# Time the forward modeling of gravity, gradients and mag
from fatiando.gravmag import sphere
print('Times:')
print('   gz: ', end='')
%timeit sphere.gz(x, y, z, model)
print('   gzz: ', end='')
%timeit sphere.gzz(x, y, z, model)
print('   tf: ', end='')
%timeit sphere.tf(x, y, z, model, inc, dec)

Output from master branch:
Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz
Model size: 2500
Grid size: 10000
Times:
   gz: 1 loop, best of 3: 2.18 s per loop
   gzz: 1 loop, best of 3: 2.19 s per loop
   tf: 1 loop, best of 3: 2.8 s per loop

Polyprism
Code used for the master branch Cython implementation:
# Get processor information
tmp = !cat /proc/cpuinfo | grep "model name"
processor = tmp[0].split(':')[1].strip()
print(processor)
# Make a model for testing
from __future__ import division, print_function
import numpy as np
from fatiando import gridder, utils
from fatiando.mesher import PolygonalPrism
vertices = utils.circular_points([-300, 300, -300, 300], 1000)[::-1]
props = {'density': 1000, 'magnetization': utils.ang2vec(2, 25, -10)}
model = [PolygonalPrism(vertices, 0, 2000, props)]
inc, dec = -30, 50
x, y, z = gridder.regular((-500, 500, -500, 500), (50, 50), z=-1)
print('Model size: {}'.format(len(vertices)))
print('Grid size: {}'.format(x.size))
# Time the forward modeling of gravity, gradients and mag
from fatiando.gravmag import polyprism
print('Times:')
print('   gz: ', end='')
%timeit polyprism.gz(x, y, z, model)
print('   gzz: ', end='')
%timeit polyprism.gzz(x, y, z, model)
print('   tf: ', end='')
%timeit polyprism.tf(x, y, z, model, inc, dec)

Output from master branch:
Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz
Model size: 1000
Grid size: 2500
Times:
   gz: 1 loop, best of 3: 1.12 s per loop
   gzz: 1 loop, best of 3: 478 ms per loop
   tf: 1 loop, best of 3: 4.22 s per loop

just following your efforts with .... nice!! :+1:


task.setvariable runs despite not supposed to be executed (e.g. when wrapped in if loop)

Troubleshooting
Exporting a variable appear to run despite not supposed to run:
First Task: Azure CLI - Version 1:
if [[ "yes" = *"no"* ]]; then
  echo "##vso[task.setvariable variable=VAL1]foo"
  echo "VAL1 exported"
else
  echo "##vso[task.setvariable variable=VAL2]bar"
  echo "VAL2 exported"
fi

Output (as expected, only else code block outputted to stdout):

snipped
2018-06-20T20:16:13.8688825Z VAL2 exported
snipped

Subsequent/next task: Azure CLI - Version 1:
set

Output:

snipped
2018-06-20T20:16:21.7604765Z VAL1='foo"'
2018-06-20T20:16:21.7618478Z VAL2=bar
snipped

^^ wtf...
Environment

Server - VSTS or TFS on-premises? VSTS, Account/Team/Build/BuildNumber are sensitive.

Agent - Hosted or Private: Hosted Linux Agent queue

Issue Description

This will happen only in the case of inlineScript, the agent runs the specific commands by parsing the logs. The parameters are printed to the agent logs (which you can see when you enable debugging logs) everytime a task is executed, the commands which you gave in your inline script parameter would probably be executed even before the shell script is executed.
The solution to avoid this is, to either check in your logic into a file and use that shell script file, or change the if-else logic to something like
if [[ "yes" = *"no"* ]]; then
  result=foo
  echo "result set to foo"
else
  result=bar
  echo "result set to bar"
fi
echo "##vso[task.setvariable variable=result]$result"

The variable result would be overriden the second time the command is executed.

@ds-ms reopening for you to consider not printing inline scripts if they contain ##vso. For example that is what the cmdline task does.

@ericsciple The task doesn't explicitly print the parameters. I meant the parameters which get printed to the output stream when system.debug is enabled.

@ds-ms ##vso commands aren't processed for debug lines.
Is the task lib not escaping the newlines?

Looks like it.
2018-07-07T02:23:58.5657106Z ##[debug]inlineScript=if [[ "yes" = *"no"* ]];
2018-07-07T02:23:58.5671396Z then
2018-07-07T02:23:58.5713036Z ##[debug]Processed:   echo "##vso[task.setvariable variable=VAL1]foo"
2018-07-07T02:23:58.5726389Z   echo "VAL1 exported"
2018-07-07T02:23:58.5738593Z else
2018-07-07T02:23:58.5768905Z ##[debug]Processed:   echo "##vso[task.setvariable variable=VAL2]bar"
2018-07-07T02:23:58.5784195Z   echo "VAL2 exported"
2018-07-07T02:23:58.5796840Z fi

Should the version upgrade of tasklib fix it?

@ds-ms nope, I have a repro with the bash task now which I think is a recent task lib version. I'll investigate more...

@stephenmichaelf if you have time to look

Looking now.

Any updates on this?

@iyerusad Can you try to run this again? I have been unable to recreate the issue using Bash. This task (AzureCLI) was updated 6 days ago so that may have resolved issues you were having before.
If you can recreate, could you send me the logs stfrance at microsoft.com
Thanks!

@stephenmichaelf diagnosed this down to the task lib. Escaping newlines in the debug command was added in a newer version. Coincidentally this task updated to the newer version last sprint. This will be fixed when the current deployment finishes rolling out to all scale units (next couple weeks).
Also, I was wrong about having a repro in the bash task. I got confused when testing.

@ds-ms Can you confirm this is working?

@stephenmichaelf yes, looks like the fix worked. It's not processing the debug statements now.

I am still facing this issue, what fixed this for you @ds-ms ?


CosmosDB: Add Patch/Increment method

It looks like CosmosDB has a distributed counter through its patch method: https://docs.microsoft.com/en-us/java/api/com.azure.cosmos.models.cosmospatchoperations.increment?view=azure-java-stable
I'm hoping this SDK has it or plan on adding as it's a common feature in most databases I work with such as Google Cloud Firestore with Go.
Thanks!

Support for Patch (including Increment) is part of the future deliverables.

Support for Patch (including Increment) is part of the future deliverables.

Is there an ETA for this?

@volmedo No ETA. There is currently a PR open that adds these APIs in progress

oh, I saw the PR is being worked on just right now! 🎉


Update the project setup guide

Motivation:

The current project setup guide is split into three pages but each of
them are not very long and thus could be merged into one page.
We do not distribute -shade JARs anymore.

Modifications:

Merged setup-*.rst into setup.rst.
Added a table that lists the major modules and their descriptions.

Result:

Better-looking documentation
Up-to-date information in setup.rst

Screenshot: https://screenshots.firefox.com/1KMrkDnOMKLQrbiq/null


Add screen navigation methods for Speculos backend.

Codecov Report
Base: 97.01% // Head: 85.98% // Decreases project coverage by -11.02% :warning:

Coverage data is based on head (ed05a5a) compared to base (4eab0fa).
Patch coverage: 23.94% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop      #31       +/-   ##
============================================
- Coverage    97.01%   85.98%   -11.03%
============================================
  Files           15       15
  Lines          402      471       +69
============================================
+ Hits           390      405       +15
- Misses          12       66       +54

Impacted Files
Coverage Δ

src/ragger/backend/speculos.py
60.00% <15.25%> (-40.00%)
:arrow_down:

src/ragger/backend/ledgercomm.py
96.77% <60.00%> (-3.23%)
:arrow_down:

src/ragger/backend/ledgerwallet.py
92.53% <60.00%> (-2.63%)
:arrow_down:

src/ragger/backend/interface.py
100.00% <100.00%> (ø)

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

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


tests: remove fixtures, use pytest-servers

Codecov Report

Merging #89 (eb476cb) into main (010cdfb) will decrease coverage by 2.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   34.32%   32.02%   -2.30%
==========================================
  Files          33       33
  Lines        1681     1611      -70
  Branches      302      229      -73
==========================================
- Hits          577      516      -61
+ Misses       1091     1090       -1
+ Partials       13        5       -8

Impacted Files
Coverage Δ

tests/conftest.py
100.00% <100.00%> (+12.32%)
:arrow_up:

src/dvc_data/hashfile/hash.py
58.82% <0.00%> (-0.40%)
:arrow_down:

src/dvc_data/stage.py
0.00% <0.00%> (ø)

src/dvc_data/db/index.py
0.00% <0.00%> (ø)

src/dvc_data/hashfile/state.py
31.13% <0.00%> (ø)

tests/hashfile/test_hash_stream.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

@dtrifiro Looks like s3 fixture is still randomly failing. Could you take a look, please?

Only 3.11 is failing right now. It seems that before 3.11, doing something like :
path = UPath("s3://container")
path.mkdir()

would raise ImportError: Install s3fs to access S3 when s3fs is not installed.
On 3.11 it does not. I'm currently investigating

Seems universal pathlib is broken on 3.11, see https://github.com/fsspec/universal_pathlib/issues/67

Waiting on #98

@dtrifiro Please rebase.


Stop setting deprecated fullversion param

A new shortcode (added in PR https://github.com/kubernetes/website/pull/40692) looks up the correct patch version from data maintained by the release team.
/area web-development
/kind cleanup

Relevant to https://github.com/kubernetes/website/pull/42462/files#r1288090326

/lgtm

/approve


Extend Arlington model for PDF/A-3 Associated Files feature (PDF 1.7 extension)

Impacts AFRelationship key and AF everywhere. Use ISO_19005_3 to mirror other ISO extensions that all use the ISO document number:
"SinceVersion" = fn:Eval(fn:Extension(ISO_19005_3,1.7) || 2.0)

But not for the excluded annots in PDF/A-3: 3D, Sound, Screen, Movie (and, of course, RichMedia which was only introduced in PDF 2.0).
And for those that were only introduced by the Adobe extensions to PDF 1.7 also need fn:Eval((fn:Extension(ADBE_ExtnXXX,1.7) && fn:Extension(ISO_19005_3,1.7)) || 2.0)

Yes - sorry. I have an ANTLR grammar that is not quite ready for prime-time to permanently address these kinds of issues. I'll see if I can do a temporary patch in the meantime.


Feature/unset active

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/datorama/akita/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?

If you've set strict null checks it is currently not possible to reset the active state to its original status.
What is the new behavior?
Now you can leave the argument of setActive() empty to reset the active entity to null.
Additionally it is possible to check if there is an active entity by a new query method (hasActive()).
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
If you appreciate the changes I would add tests.

I also thought about an explicit unsetActive method - what's your opinion about it?

The hasActive() sounds great. I'm not sure about the setActive() method. I think it's not understood from the call that you set it to null and the unsetActive method doesn't help too much. What's wrong with setActive(null)?

setActive(null) would also be a good option from my point of view.
Then I would like to adjust the function signature to setActive(id: ID | null) {...}, because currently it is not possible to pass null in a project with strict null checks on (like mine).

👍

Then I will open a new PR to get a clean history without my old changes, ok?

sure, you can also add yourself to the contributors' list.


Make Promise.Sequence report progress.

Promise.Sequence was not reporting progress. Now it will report an overall progress for the whole sequence going from 0 to 1, plus all the intermediate values inbetween.
This is, if you have 4 promises reporting their own progress (from 0 to 1) in a sequence, the progress that the Promise.Sequence call will report is:
Promise1: [0, 0.25]
Promise2: [0.25, 0.5]
Promise3: [0.5, 0.75]
Promise4: [0.75, 1]

Looks good! I just added one comment for something that I think is an issue with your code, but once that's resolved I'll be happy to merge.


how to pass data from node.js to html file for printing pdf

I have created https://github.com/bapatel1/node-html-pdf-sample  with your lib and using angular.js in html. Now my problem is , I have to send filtered data to display from node.js to html file to generate pdf from that. (Kind of like html is template and pass data from node.js )
Any idea how may i achieve that, really appreciate your help in this matter as have release in near future.
Thank you

@marcbachmann
just for an example -
var data = {
ticketNum : 12121212,
dateIssued: '2014-04-02',
personName: 'John Doe',
notes:' Lorem Ipsum Lorem ipsum dolor sit amet, consectetur adipiscing'
};
//TODO: pass this data object json to html to print this data.

pdf.create(html,report_options).toStream(function(err, stream){
    console.log(stream);
    stream.pipe(res);
});

Ah, that's not the job of this library. You can use some template language to generate your html.
All template languages in node give you a .render method that output the html rendered with the locals.
load template -> render using data -> html-pdf

Something like this:
var nunjucks = require('nunjucks')
nunjucks.configure(__dirname+'/views')
html = nunjucks.render('template.html', foo)
htmlPdf.render(html, options).toStream(function...)

@marcbachmann  Oh I see now. I guess i am still trying to understand node better and I appreciate your help in this query. I will give it a try now and will post here.
thank you,

you're welcome

@marcbachmann You saved me .. nunjucks worked for me like a charm and pdf is also generating using your lib. I have updated my sample code with some more node.js clustering support.
https://github.com/bapatel1/node-html-pdf-sample
Thank you once again.

Closing issue with solution which @marcbachmann  suggested as above. Using template engine called "nunjucks" worked for me.

I prefer Jade. Works the same, but is more “stylish”.


RFC: Adjust project scope to formatter / linter

The web community really needs a great formatter and linter, which is what Biome close to being today.
If Biome can pull just that off, instead of being spread to thin with compilers, test runners and bundlers, in direct competition with project like Bun, I believe it has a much higher change of a very bright future.

The current focus is clearly on formatter and linter. No need to create debates.

@Conaclos , great point :)


Malformed ip pool name causes crash

Seen on Calico 2.6, if you have an IP pool in etcd that has an unexpected name format then we fail to parse it with a panic.
Example key: /calico/v1/ipam/v4/pool/577559 (taken from an old OpenStack scale testing script).
Expected Behavior

Warning log and drop.
Current Behavior

panic: invalid CIDR address: 577559

goroutine 71 [running]:
github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/model.KeyFromDefaultPath(0xc420321940, 0x1e, 0xc420627b30, 0x1)
	/go/src/github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/model/keys.go:209 +0x139c
github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/etcd.(*etcdSyncer).sendUpdate(0xc4201a3b60, 0xc420321940, 0x1e, 0xc420042640, 0x3e, 0x124, 0x1)
	/go/src/github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/etcd/syncer.go:525 +0xf7
github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/etcd.(*etcdSyncer).mergeUpdates(0xc4201a3b60, 0xc42004f0e0, 0xc42004f080, 0xc42004f140)
	/go/src/github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/etcd/syncer.go:456 +0x15d1
created by github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/etcd.(*etcdSyncer).Start
	/go/src/github.com/projectcalico/felix/vendor/github.com/projectcalico/libcalico-go/lib/backend/etcd/syncer.go:158 +0x192

Possible Solution

Handle the error, log and drop instead of panicking.  Audit other key parsing functions for same.
Steps to Reproduce (for bugs)

Manually insert an IP pool with above key
Start felix (probably other components affected)

Context

Your Environment

libcalico-go version: Something around edcbcf64348004bfeb0dc3882cd2790743ec7728 (using felix from the calico-2.6 PPA)
Operating System and version: Ubuntu 14.04
Link to your project (optional):

Correct.  It was written with an etcdctl set in this case.  In the real world, the user could modify etcd directly for purposes unknown or we could hit, for example, a memory corruption on write.
While both of those are rare (at least at 100-node scale?), the impact is that every felix cyclicly restarts.  In the old python felix, we tried very hard to avoid any kind of crash caused by invalid keys or data since they cause cluster-wide outages.  I think "don't panic" should be our motto in the datastore interaction/parsing code.


Map Properties-Map Size-Unable to display the entire map.

When the Map Properties setting Map Size is very large,
1>On the page to edit the map，there is no scroll bar on the right and bottom of the page to drag.
2>On the page browsing the map, there is no bottom scroll bar to drag, and the full width of the map cannot be displayed.
For example: Map Size 4000*2000.

Is this a feature request or a bug.  I've not used wm before.  So, the question may sound odd.

I have the same issue, even if on my side a have the vertical scroll, but not the horizontal one.
So I'd like to say it's a bug !

I think it should be regarded as a bug. There are also requests for such high-pixel maps.
Draw high pixels, use a lower resolution display, and cannot browse the entire map without scroll bars.
There is no horizontal and vertical scroll bar to drag the page on the edit map page.

Previous versions did not have this problem.

There is no horizontal scroll bar at the bottom of the map display page.

In addition, when using high pixels, such as 4000*2000。you cannot add node in the box area. I think it is necessary to draw a larger map in many cases.

Let me check this one out.  Should be simple CSS.  Have to override Cacti CSS.

Okay, this is fixed in my personal repo until Beta-3 is out or the release.

Sorry not close yet, this issue it still present when you edit a map.
For the display on the Weathermap tab it's doing great.

You will have to be much more specific.

Sorry, There was 2 points on the start of this thread.
The point 1 is not resolve:
When the Map Properties setting Map Size is very large, 1>On the page to edit the map，there is no scroll bar on the right and bottom of the page to drag.
You solve the point 2 on the weathermap tab page, now we have scroll bar on the side and on the bottom, but not on the console-> weathermap->'edition of a large map'

In the previously WeatherMap releases was there a scroll bar, or did the image just get resized to the page size?  And how did you know the other point since you did not have a scroll bar before?  Seems almost like a new issue.  Well it is a new issue.  So, can you please open a new case?

Well in version 0.98a I have an vertical scroll bar
But the image was resize on the horizontal size.
As for the other point (point 2) I know it's solve since now I have both scroll bar. any way i can open a new case no problem.

Cool thanks.  The scroll is still there, but for some reason, the horizontal scroll bar on the editor is missing.  You can use the scroll wheel on the mouse combined with the shift key to still scroll.  Have to figure out why the scrollbar is not visible though.


Update README.md

Description

A clear and concise summary of the change and which issue (if any) it fixes. Should also include relevant motivation and context.
Resolved or fixed issue:
Affirmation

[ ] My code follows the CONTRIBUTING.md guidelines

Checkmarx One – Scan Summary & Details – 25590f3c-0dae-4949-b221-bcf98b7b0c35
No New Or Fixed Issues Found


fix namespace organization

Coverage remained the same at 100.0% when pulling 29331fc52fea3702ccf7ff8f5aa126bbe725e392 on namespaces into 6e651f52bd71c6d3fe080f56488b49d9184ab14c on master.


Publish a snowchange container

It would be good to publish a public image with snowchange as the entrypoint in the Dockerfile. It would be fairly easy, and I'm happy to work on it, but I would need access to the Docker Hub (I'm assuming) registry where you want to publish it.

Hey there @stewartbryson, sorry for the slow response. We're currently working to get a PyPI project setup so that snowchange can be installed via pip, which will make the whole process of building an image even easier. And we could definitely add a Dockerfile to the project to make it easier for folks to build an image.
For those that want to use a container in a CI/CD pipeline you're right that one will likely need to be published to Docker Hub. I don't currently have bandwidth to get that setup or to maintain it, but if you're willing to help that would be awesome! If not you could always just setup your own Docker Hub project for your needs right now.
Thanks for reaching out!

Hey @stewartbryson, just wanted to close the loop here that we have added a Docker file to the project to make it easy for folks to get started with snowchange in a container. Hope that helps!

Hey @stewartbryson, just wanted to close the loop here that we have added a Docker file to the project to make it easy for folks to get started with snowchange in a container. Hope that helps!

Thanks for adding the container. Sorry I didn't respond to your comment on Nov 10.


undefined: blackfriday.Run

New install:
vendor/github.com/matcornic/hermes/hermes.go:76:23: undefined: blackfriday.Run
Seems blackfriday API has changed:
// ToHTML converts Markdown to HTML
    func (c Markdown) ToHTML() template.HTML {
            return template.HTML(blackfriday.Run([]byte(string(c))))
    }

Which Hermes version do you use (tag or git hash) ?

is this related?
# ./app/vendor/github.com/matcornic/hermes
vendor/github.com/matcornic/hermes/hermes.go:76:23: undefined: blackfriday.MarkdownCommon
(error exit: exit status 2)

Find out that vgo for some reason installs v1.1.0 version default and build fails with this error. Vgo can't find v1.1.1, but pointing to specific commit or master branch solves a problem.

@xdefrag probably because I mixed formats in the tag names. I renamed the 1.1.1 tag to v1.1.1
Could you try again ?

@matcornic Yep, it's worked. Thanks.


Version checking and forcing refresh

Deprecate config xhAppVersionCheckEnabled in favor of object based xhAppVersionCheck.
Add support for shouldRequireRefresh flag

See corresponding hoist-react PR: https://github.com/xh/hoist-react/pull/3407

tom and I agreed on the single config and moving the interval in.
I think if we were going to so such aggressive renaming restructure (I am
sympathetic), I might prefer just a mode like
mode: 'forceReload/promptReload/silent'
Re: "silent" --  it seems like we should encourage that this timer to never
be turned off (or at least not advertise it with an explicit flag) as it
usefully updates the client environment service as to the new server
version, and having that always accurate could be useful for apps and hoist
alike
On Fri, Jun 30, 2023 at 2:09 PM Anselm McClain @.***>
wrote:

@.**** commented on this pull request.
In grails-app/init/io/xh/hoist/BootStrap.groovy
https://github.com/xh/hoist-core/pull/299#discussion_r1248147076:

       xhAppVersionCheckEnabled: [

           valueType: 'bool',

           defaultValue: true,

           groupName: 'xh.io',

           note: 'True to show an update prompt banner when the server reports to the client that a new version is available. Can be set to false to temporarily avoid an upgrade prompt (e.g. while validating a deploy). Use xhAppVersionCheckSecs to completely disable this feature.'

       ],

       xhAppVersionCheckSecs: [

           valueType: 'int',

           defaultValue: 30,

       xhAppVersionCheck: [

           valueType: 'json',

           defaultValue: [

               interval: configService.getInt('xhAppVersionCheckSecs', 30),

               shouldUpdate: configService.getBool('xhAppVersionCheckEnabled', true),

               shouldRequireRefresh: false

           ],

I think it's a great goal to land on a consolidated config for this
feature. The server-side configs are not sensitive, so mostly a matter of
potential confusion where some keys are read on server and relayed to
client, with others read by client directly. But having a single overall
config to control the service seems like enough of a win to make it worth
it.
What if we went with something like:
[
// Read on client - will completely disable polling on client if false (i.e. really disable the feature, client would need to reload if changed).
enabled: true,
// Read on client - used to setup timer, but only if generally enabled.
intervalSecs: 30,
// Read on server, relayed to client with each check response.
promptForReload: true,
// Read on server, relayed to client with each check response.
forceReload: false
]
Maybe we could introduce a level of nesting to pull out the two more
dynamic configs, something like;
[
enabled: true,
intervalSecs: 30,
dynamic: [
promptForReload: true,
forceReload: false
]
]
Where dynamic is some key name that someone can come up with that best
expresses this idea.
—
Reply to this email directly, view it on GitHub
https://github.com/xh/hoist-core/pull/299#pullrequestreview-1507662671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARTL2KJSEMSKWWUYAPB4XLXN4I6ZANCNFSM6AAAAAAZZUS2MI
.
You are receiving this because your review was requested.Message ID:
@.***>

Yes I think the mode makes a lot of sense to eliminate possibility of contradictory options (prompt:false/force:true)
I was thinking we might run into a case where a client didn't want the polling (e.g. to avoid it acting as a session keepalive or something like that) - alert banners are the other example of constant background polling Hoist will do, and that does have such a flag.  But we could definitely cross that bridge if/when we ever come to it.

if we ever came to that bridge, we could use -1 to get across it! 😀
On Fri, Jun 30, 2023 at 2:26 PM Anselm McClain @.***>
wrote:

Yes I think the mode makes a lot of sense to eliminate possibility of
contradictory options (prompt:false/force:true)
I was thinking we might run into a case where a client didn't want the
polling (e.g. to avoid it acting as a session keepalive or something like
that) - alert banners are the other example of constant background polling
Hoist will do, and that does have such a flag. But we could definitely
cross that bridge if/when we ever come to it.
—
Reply to this email directly, view it on GitHub
https://github.com/xh/hoist-core/pull/299#issuecomment-1615035355, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARTL2NMUJD2YVF32RHMEILXN4K7HANCNFSM6AAAAAAZZUS2MI
.
You are receiving this because your review was requested.Message ID:
@.***>


search doesn't work on multi levels grouping

in my case i built a multi levels grouping  as the following example :
options: [			 		{ label:'Option Group 1' , 				options: [ 						{ label:'Option Group 1-1' , value:'4'}, 						{ label:'Option Group 1-2' , value:'238'}, 						] 		}, 		{ label:'Option Group 2' ,  				options: [ 								{ label:'Option Group 2-1' , 										options: [ 										{ label:'Option Group 2-1-1' , value:'9'}, 										{ label:'Option Group 2-1-2' , value:'185'}, 										{ label:'Option Group 2-1-3' ,  												options: [ 												{ label:'Option Group 2-1-3-1' , value:'200'}, 										]}, 										] 								}, 							] 		}, 	]
and i pass it to the init function:
<script type=\"text/javascript\">   VirtualSelect.init({   ele: '#categories',   options: [".$options."],   dropboxWrapper: 'self',   multiple: true,   selectedValue: [".$Value."],   optionsCount: 13,   optionHeight: '14x',   textDirection: 'rtl',   showValueAsTags: true,   disableSelectAll: true,   search: true,   zIndex: 1500, });	</script>
search is working fine just for the second level, when i have more i got a JavaScript error :
Uncaught TypeError: can't access property 22, o is undefined

@masdar80 as of now, this plugin would be working fine for 1 level of options group only.
I will provide support for multi level options group in the future release.
Thanks for letting me know it.

@masdar80 as of now, this plugin would be working fine for 1 level of options group only.
I will provide support for multi level options group in the future release.
Thanks for letting me know it.

Thank you it will be a good feature to add
for now i am using
onServerSearch: onSampleSelectServerSearch,
instead
but i lost the ability to select (lost the checkbox) for main groups, when i remove the onServerSearch property i get the checkbox back for the main groups.
it is important for me because user can check all children categories by choosing the father category
any idea please?

Is there any progress on this topic?


zoomtype: Inner problem

this is my wrapper
<img class="col-md-12 noPadding prodDetImg img-responsive" id="zoomImg" ng-src="{{currentImage}}" ez-zoom ez-zoom-image="{{currentImage}}" ez-zoom-config="{'zoomType': 'inner', 'cursor': 'crosshair', 'easing' : true}""/>
My inner zoom works fine but once zoomed in the image remains zoomed in. On mousleave the zoomcontainer still remains.
If we changed the image, each time a new zoom container is created which is also zoomed in. why is it so?

fixed issue.

How did you fixed?
Can you post your controller, please? I am confused with images and catalog.
Thanks


Boost 1.74 compatibility + Conan updates

Hi,
this PR includes two changes: The first is rather small, I added BOOST_ASIO_USE_TS_EXECUTOR_AS_DEFAULT (see yandex/ozo#266) to the defines of the generated CMake package helper for the time being, so consumers of the library don't have to do it on their end. The async/coro.cpp example fails under 1.74 without this, for example.
Other than that, I updated the conan package, so it can be consumed with the cmake_find_package generator. Conan does not allow any find helper CMake files into conan-center, so in order to stay comaptible with that, the defines/dependencies using target_link_libraries and target_compile_definitions need to be replicated in the conanfile. I updated the test package to check for compatibility accordingly.
This should not change anything for users of the current conanfile, as the generated CMake helpers are still included in here and will work with the conan_paths generator.
Cheers

Thanks!

Thanks!


Unable to compile vkuTestBasicCube on Linux

Trying to compile the vkuTestBasicCube using the build.sh script results in the following error:
basic_cube.c: In function ‘main’:
basic_cube.c:83:5: error: too few arguments to function ‘vkuDestroyVertexBuffer’
   83 |     vkuDestroyVertexBuffer(stagingBuffer, context->memoryManager);
      |     ^~~~~~~~~~~~~~~~~~~~~~
In file included from basic_cube.c:2:
../src/vkutils.h:145:6: note: declared here
  145 | void vkuDestroyVertexBuffer(VkuBuffer buffer, VkuMemoryManager manager, VkBool32 syncronize);
      |      ^~~~~~~~~~~~~~~~~~~~~~
basic_cube.c:200:5: error: too few arguments to function ‘vkuDestroyVertexBuffer’
  200 |     vkuDestroyVertexBuffer(vertexBuffer, context->memoryManager);
      |     ^~~~~~~~~~~~~~~~~~~~~~
../src/vkutils.h:145:6: note: declared here
  145 | void vkuDestroyVertexBuffer(VkuBuffer buffer, VkuMemoryManager manager, VkBool32 syncronize);
      |      ^~~~~~~~~~~~~~~~~~~~~~

Hey there, thank you for addressing the issue and making the changes. This is actually my first pull request ever. Happy holidays!

Hey there, thank you for addressing the issue and making the changes. This is actually my first pull request ever. Happy holidays!


issue with google-auth.js

After running the node google-auth.js ,it requires me to paste my code.I do paste my code from the web,but it doesn't work,and then it returns this:
/home/viking/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/components/auth.js:73
      if (error) throw new Error('Error getting tokens:', error);
                 ^

Error: Error getting tokens:
    at oauthClient.getToken (/home/viking/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/components/auth.js:73:24)
    at /home/viking/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/node_modules/google-auth-library/lib/auth/oauth2client.js:154:5
    at Request._callback (/home/viking/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/node_modules/google-auth-library/lib/transporters.js:70:30)
    at self.callback (/home/viking/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:186:22)
    at Request.emit (events.js:180:13)
    at Request.onRequestError (/home/viking/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:878:8)
    at ClientRequest.emit (events.js:180:13)
    at TLSSocket.socketErrorListener (_http_client.js:395:9)
    at TLSSocket.emit (events.js:180:13)
    at emitErrorNT (internal/streams/destroy.js:64:8)

@V1Kin9
Without fully posting the commands and and the input you gave, there is little I can do.
Please re-open when you have the info.

I have the same error. One thing to note, however I doubt it has much to do with anything, is that the browser it opens closes immediately after I put in an email.
The following is my input/output at the terminal. I covered some of the key with "*"s but they are only letters. The rest of the license is untouched; The forward slash and underscores are part of the code I was given.
pi@raspberrypi:~/MagicMirror/modules/MMM-Assistant/scripts $ node google-auth.js
Opening OAuth URL. Return here with your code.
Paste your code: 4/AAA******xzggf0AMt1v1QYmcGPA7RYlxiUw*****************
/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/components/auth.js:75
if (error) throw new Error('Error getting tokens:', error);
^
Error: Error getting tokens:
at oauthClient.getToken (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/components/auth.js:75:24)
at /home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/node_modules/google-auth-library/lib/auth/oauth2client.js:154:5
at Request._callback (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/node_modules/google-auth-library/lib/transporters.js:106:7)
at Request.self.callback (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:185:22)
at Request.emit (events.js:180:13)
at Request. (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:1157:10)
at Request.emit (events.js:180:13)
at IncomingMessage. (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:1079:12)
at Object.onceWrapper (events.js:272:13)
at IncomingMessage.emit (events.js:185:15)

So I reinstalled and double checked that everything is up to date:
pi@raspberrypi:~/MagicMirror/modules/MMM-Assistant/scripts $ node -v && npm -v
v9.11.1
6.1.0

Then I went through and the dependencies and everything is 'newest', no need for updates :+1:
I went ahead and dropped my secret.json in the MMM-Assistant folder and a google-client-secret.json in the assets folder (both are the same file).
Then I proceeded to run node google-auth.js with a new key/code from google and got the same error
Digging through it the error happens when a request is being made with the transporter. I don't know js but based on the number of if(err)'s there are, I would assume the script would catch us if Oauth key is missing before we continued to make requests.
The throw happens here:
const processTokens = (oauthCode) => {
    if (!oauthCode) process.exit(-1);

    // get our tokens to save
    oauthClient.getToken(oauthCode, (error, tkns) => {
      // if we didn't have an error, save the tokens
      if (error) throw new Error('Error getting tokens:', error);

      tokens = tkns;
      saveTokens();
    });
  };

And the error is returned and given somewhere here:
this.transporter.request({
    method: 'POST',
    uri: uri,
    form: values,
    json: true
  }, function(err, tokens, response) {
    if (!err && tokens && tokens.expires_in) {
      tokens.expiry_date = ((new Date()).getTime() + (tokens.expires_in * 1000));
      delete tokens.expires_in;
    }
    var done = opt_callback || noop;
    done(err, tokens, response);
  });

I'm planning this project as a gift and really wanna do some cool things like the google assistant. Hope we can find a solution soon :)

So I just downgraded node and npm in case the versions were the problem and am still hitting this error:
(again the '*'s are just letters/numbers)
pi@raspberrypi:~/MagicMirror/modules/MMM-Assistant/scripts $ node google-auth.jsOpening OAuth URL. Return here with your code.
Paste your code: 4/AA*****************************************************
/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/components/auth.js:75
      if (error) throw new Error('Error getting tokens:', error);
                 ^

Error: Error getting tokens:
    at oauthClient.getToken (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/components/auth.js:75:24)
    at /home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/node_modules/google-auth-library/lib/auth/oauth2client.js:154:5
    at Request._callback (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/google-assistant/node_modules/google-auth-library/lib/transporters.js:106:7)
    at Request.self.callback (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:185:22)
    at Request.emit (events.js:180:13)
    at Request.<anonymous> (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:1157:10)
    at Request.emit (events.js:180:13)
    at IncomingMessage.<anonymous> (/home/pi/MagicMirror/modules/MMM-Assistant/node_modules/request/request.js:1079:12)
    at Object.onceWrapper (events.js:272:13)
    at IncomingMessage.emit (events.js:185:15)
pi@raspberrypi:~/MagicMirror/modules/MMM-Assistant/scripts $ node -v && npm -v
v9.10.1
5.8.0


Colour graph by napari track colur

Fixes https://github.com/lowe-lab-ucl/arboretum/issues/9.

sorry based on this morning's discussion I need to work out how to run this before I carry on with my review

Since this has an approval and the linux tests pass, I'm going to merge. https://github.com/lowe-lab-ucl/arboretum/pull/35 needs to be merged to fix the windows tests.


feat[eventsourced-nats]: inactive-threshold for stream consumer

Closing, because not solving the issue.


Replace 'inline' with 'static inline'

This change enables compilers to correctly scope the symbol and apply inlining optimizations. For example, clang emits functions and calls in the code, which is suboptimal. By assisting the compiler, we ensure that this function, not used elsewhere, does not need global scope. Without this modification, the project cannot be built on macOS, where Clang is the default compiler.

Merged. Thanks for your contribution!


Using wget or HTTrack with youtube-dl?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2<IP_ADDRESS>. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>

Before submitting an issue make sure you have:

[ x] At least skimmed through README and most notably FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ x] Question
[ ] Other

Hello, I was wondering if there was any easy way to connect youtube-dl to something like wget or HTTrack when downloading things like soundcloud songs or youtube pages. I would like to use something like wget to copy websites and when playing the song / video from the downloaded html page, I would like it to pull from whereever youtube-dl put the song (not pulling from online but from the download location).
If there isnt a way to do this, could I do this with something like bash or would I have to program something something entirely to do this?

It's technically impossible since download URLs are dynamic and involve real communication with remote party not a mirrored copy.

Hi just wondering if you made any progress on this? I am interested in getting HTTrack to use youtube-dl to download videos.


WIP: Possible fix for #1252: Using @partial-block twice in a template not possible

I've added a possible fix for this bug, but it includes some refactorings of the @partial-block-code. After thinking about it a lot (this is kind of a brain-twister), I've come to the conclusions that partial-blocks should be treated like function-closures:
With the example from one of the tests:
Template
<template>{{#> outer}}{{value}}{{/outer}}</template>

outer
<outer>{{#> nested}}<outer-block>{{> @partial-block}}</outer-block>{{/nested}}</outer>

inner
<nested>{{> @partial-block}}</nested>

Explanation
When the partial-block of the outer partial  (<outer-block>{{> @partial-block}}</outer-block>) is executed, the data-variable @partial-block of outer must be called. The fix does not achieve this by popping the partial-block once its been used, but by storing the partial-block in the local closure-context of the invokePartial-function. The partial-block for that partial is than wrapped with another function that loads the previous partial-block from the local closure-context.
I can add more explanation, once my laptop-battery has recharged. In any way, I would like to have a review of the changes.
@lawnsea I changed some of the code from your fix for #1185, so could you review if my changes are valid in your eyes? The changes are in the branch "tmp/issue-1252".
After review, I would probably make some style changes for better readability.

I am not sure the language should allow multiple @partial-blocks at a given nesting level. I'd like some feedback from the other maintainers and @wycats in particular.

You also need this patch, if you want to use the #each-helper with partial-blocks

Template: <template>{{#> list value}}value = {{.}}{{/list}}</template>
Partial "list": <list>{{#each .}}<item>{{> @partial-block}}</item>{{/each}}</list>
Input: { value: [ 'a', 'b', 'c' ] }

@wycats The question we are having here is:
Should this be possible: <partial>{{@partial-block}} - {{@partial-block}}</partial>
and this: <partial>{{#each list}}{{@partial-block}}{{/each}}</partial>
It was possible in version 4.0.3 and 4.0.4, but there were other problems (#1185). Versions 4.0.5 and 4.0.6 do not allow it.
You said you don't want new features to be implemented until we have a spec. Do you consider the above examples to be a new feature or a fix?
My opinion is that it is a fix. Since a @partial-block is basically the equivalent of the options.fn()-callback of a block-helper, it should work like options.fn(), even if it is called multiple times.
@lawnsea I have added multiple test-cases for your scenarios and a few more (e.g. a partial-block called from within an #each block. Those tests pass.

@nknapp I've come around on this, particularly since it used to work and doesn't anymore, thanks to me. I still think <partial>{{@partial-block}} - {{@partial-block}}</partial> is weird and not very compelling, but <partial>{{#each list}}{{@partial-block}}{{/each}}</partial> is pretty clearly useful.
Thanks for adding the test cases. I'll take a look.

Let's give @wycats a couple more days to weigh in. If we don't hear from him, I think we should land this and cut a 4.0.7 release.

This has been waiting for almost a month now. I have not had the time to do it, but I will merge it now.

I have condensed the branch into 2 commits

The fix with a test for the fix
Additional testcases for partial-blocks


Extremely unhelpful error messages on asset load failure

Bevy version
0.5
Operating system & version
Windows 10
What you did
asset_server.load("pieces/cburnett/bK100.png")
What you expected to happen
The image was loaded into memory and the texture displayed on the relevant sprite
What actually happened
The image failed to load and this extremely unhelpful error message was emitted:
WARN bevy_asset::asset_server: encountered an error while loading an asset: Error reading image file pieces/cburnett/bK100.png: failed to load an image, this is an error in `bevy_render`.
Additional information
The file bK100.png was created by opening bK.png (500px, also exported from GIMP) in GIMP, downscaling it to 100px and exporting as PNG. As far as I can tell the image properties (colour depth etc.) are all identical except the dimensions.
The original 500px image works correctly with asset_server.load(...).
Both images are viewable in image viewing and editing software.

I tried loading your image and got the following error with main:
May 17 21:21:28.753  WARN bevy_asset::asset_server: encountered an error while loading an asset: Error reading image file bk100.png: failed to load an image: Format error decoding Png: CRC error, this is an error in `bevy_render`.

So... good news is error message has already been improved, bad news is your image has a CRC error. Most software nowadays just ignore those errors, but we can't do much on Bevy side, we're dependent on https://github.com/image-rs/image for that
Re-exporting the image, it loads fine
your bK100.png:

I see, I was running with --release, perhaps that removes information from the error?

@mockersf was this also fixed by the solution to #2257 and if so are we ok to close this issue?

I see, I was running with --release, perhaps that removes information from the error?

No, the error message was improved in current git main, you'll have it in next version 🙂
Were you able to fix your image?

Opening the image in Paint 3D (because regular MS Paint doesn't support transparency) and re-exporting seemed to fix whatever CRC issue was breaking them earlier.


Optionally resist fingerprinting

Tor Browser instances that use StemNS are vulnerable to fingerprinting, because a website can probe for StemNS-based eTLD's via subresource loading.  This is arguably not a huge deal, since each supported naming system only adds 1 bit of fingerprintability.  It is also arguably not a huge deal because we intend to make Namecoin support the default.  However, IMO it's still a problem that we should fix, especially during the transitional period where Namecoin support isn't ubiquitous.
Tor Browser exposes the first-party eTLD+1 via the SOCKS username.  We can thus resist fingerprinting like this:

If the SOCKS username eTLD matches the destination eTLD, resolve via Prop279 like we do now.
If the SOCKS username eTLD does not match the destination eTLD, leak the resolution to the exit relay (as though StemNS were not active).

This will prevent websites from fingerprinting StemNS-based eTLD's via subresource loading.  It will not prevent websites from fingerprinting StemNS-based eTLD's via first-party redirects, but that's outside of Tor Browser's threat model anyway.  It will cause some security degradation, in that websites with a DNS-based first-party domain who access subresources on Namecoin will be vulnerable to hijacking of those subresources by malicious exit relays.
Note that we do not simply return a resolution error in case 2, because that would be fingerprintable based on the delay taken for an error to happen while loading the subresource.
It should be noted that the .bit.onion eTLD will not have any security degradation from this, because Tor will see it as an invalid .onion domain, and not leak it to an exit relay.  It should also be noted that .bit domains with TLS enabled will not be hijackable unless the exit relay is colluding with a compromised TLS CA, and ncp11 is not installed in Tor Browser.  .bit domains that resolve via an IP address without TLS are vulnerable to hijacking either way, so that's not really a problem.  The major source of security degradation is .bit domains that resolve via an onion service without TLS.  I tend to think that the "right" way to handle this is to encourage usage of TLS with .bit domains, even when onion services are used (or else make it clear that .bit domains that resolve via non-TLS onion services should not be used as subresources of first-party non-.bit domains).
This behavior should be optional.  Some users may prefer to never leak resolution to an exit relay, even if it makes fingerprinting easier.

I think it seems gross to leak it to the exit relay; I'd rather we first try and get a patch done in upstream that makes it so that it will never try to resolve .bit domains by default, as if it were a suTLD. (The further upstream we can get this, the better it would be for the goals of the project.)

I think it seems gross to leak it to the exit relay; I'd rather we first try and get a patch done in upstream that makes it so that it will never try to resolve .bit domains by default, as if it were a suTLD. (The further upstream we can get this, the better it would be for the goals of the project.)

@yanmaani I think this is orthogonal.  If I were implementing such a patch upstream, I would make Tor automatically close any streams for .bit domains as soon as they are attached to a circuit.  And the way StemNS would resist fingerprinting would be by attaching .bit streams that pose a fingerprinting hazard to a circuit.  So, the behavior by StemNS would be unchanged regardless of whether Tor has such a patch.


max_batch_size limit doesn't work well at queue.next_batch()

https://github.com/huggingface/text-generation-inference/blob/4dfdb481fb1f9cf31561c056061d693f38ba4168/router/src/infer/v3/queue.rs#L362
When max_size is 0, but the batch_requests.len() > 0，max_batch_size limit fails.

Hi @AndersWXJY 👋
Could you be a bit more specific? Does this cause a crash or do you mean that the max_batch_size isn't enforced even if it should?
Also if you have an example of how to reproduce the bug/unwanted behavior, that would help a lot 👍

Yeah I mean that the max_batch_size isn't enforced even if it should.
I am working on customization  for a new device. As a backend of text-generation-server, there is a limit of max batch size such as 32, even if the max_batch_prefill_tokens and max_batch_total_tokens are under limit. So, the batching task could send a new batch but the server backend couldn't process anymore.
So i insert a code segment as below at the beginning of function State.next_bactch()to make it work well.
// Check if we have enough batch slots
if let Some(max_size) = max_size {
    if max_size <= 0 {
        tracing::debug!("Not enough batch slots for new requests!");
        return None;
    }
}


audit rules mistake init/finit

Describe the Issue
V-238295 / ubtu_20_010179 doesn't pass SCAP benchmarking because it's missing the init portion.
Rule text:  "The Ubuntu operating system must generate audit records for successful/unsuccessful uses of the init_module and finit_module syscalls."
Lockdown template entries:
-a always,exit -F arch=b32 -S finit_module -F auid>=1000 -F auid!=4294967295 -k module_chng
-a always,exit -F arch=b64 -S finit_module -F auid>=1000 -F auid!=4294967295 -k module_chng

STIG check passing value:
-a always,exit -F arch=b32 -S init_module,finit_module -F auid>=1000 -F auid!=4294967295 -k module_chng
-a always,exit -F arch=b64 -S init_module,finit_module -F auid>=1000 -F auid!=4294967295 -k module_chng

Expected Behavior
Check should pass after enabling variable for check item
Actual Behavior
template just missing init_module
Control(s) Affected
V-238295 / ubtu_20_010179
Environment (please complete the following information):

branch being used: [e.g. devel]
Ansible Version: [e.g. 2.10]
Host Python Version: [e.g. Python 3.7.6]
Ansible Server Python Version: [e.g. Python 3.7.6]
Additional Details:

Additional Notes
Anything additional goes here
Possible Solution
Enter a suggested fix here

hi @kfiresmith
Thank you for raising this issue, I am happy to say that the suggested fix has been added and merged to both devel and main branches.
Unless you have any issue, i believe this could now be closed?
many thanks
uk-bolly


change to Google docstrings style. remove TODO comment

Just some minor comments but the it looks good :+1:

Thanks. Corrected and pushed again

thanks!


Add acceptance test ci mode

Context
Implementing part of the baseline deployment into the runner.
Tested in https://app.circleci.com/pipelines/github/ministryofjustice/fb-runner-node/2042/workflows/3050a35d-6a14-44cd-a134-bbf057cc6bb8/jobs/3710

This means that we can merge this PR on acceptance tests https://github.com/ministryofjustice/fb-acceptance-tests/pull/109


AQCU-1185

Coverage increased (+0.0002%) to 98.919% when pulling bdf4d2ce43e0be3af63c62a832dbe7e3dd5fb413 on kmschoep-usgs:AQCU-1185-take-2 into 85a9a2ea23256a2562c8e16c0a979f691dc921c0 on USGS-R:master.

Coverage increased (+0.0002%) to 98.919% when pulling a3fe8ef5f9494edb7dd8042da1f0cd664b867ef9 on kmschoep-usgs:AQCU-1185-take-2 into 85a9a2ea23256a2562c8e16c0a979f691dc921c0 on USGS-R:master.

Coverage increased (+0.0002%) to 98.919% when pulling a3fe8ef5f9494edb7dd8042da1f0cd664b867ef9 on kmschoep-usgs:AQCU-1185-take-2 into 85a9a2ea23256a2562c8e16c0a979f691dc921c0 on USGS-R:master.


Bancor TVL issue

Bancor team brought up that there is a difference in TVL compared to our subgraph
Etherscan of V3 vault: https://etherscan.io/tokenholdings?a=0x649765821D9f64198c905eC0B2B037a4a52Bc373
I suspect the issue is that we are measuring their deposits (they are single sided) and our subgraph doesn't reflect the current deficits in the pools (this was noted here in the original QA: https://docs.google.com/spreadsheets/d/1Yg4JvsYiCccmQhhVwUKvfWffXSB165GyqaVbvGRc82o/edit#gid=0). Will respond to them and raise issues if needed.

@steegecs so we talked to Bancor. the current pool and TVL is based on the staked amount. we should change to amount in the pools. the staked amount is different from the pool amount.
https://etherscan.io/tokenholdings?a=0x649765821D9f64198c905eC0B2B037a4a52Bc373
would it be possible to change this? i can do a quick QA after to make sure it is good. not a huge deal, we had documented this issue before and it seems the bancor team recommends doing amounts in pool vs staked amount.

@ishraq8 Bumping this. Is it still relevant?

@bye43 Yeah, this got backlogged. Bancor asked us to calculate their pool TVL and amount by how much is there and not how much was deposited. They are facing a deficit and surplus in certain pools. May want to assign someone who is free.


Working in the mobile version

Thank you so much for the plugin!
Is it possible for it to work on mobile devices? I often create my daily notes from my mobile even before I get to my laptop. It would be great if meta information about the weather was put down at this moment...

The only reason I have it disabled is because it uses the statusbar to display the current weather which the mobile version does not support. I don't actually have the mobile version installed but I will install it to test this.
You may be able to get to work on mobile by editing the manifest.json and setting "isDesktopOnly": true to "isDesktopOnly": false then disabling the statusbar in OpenWeather plugin settings. I will try this when I have the time to install the mobile version.

Just tested this and it works fine. Just make the edit to the manifest and it works perfectly, although without the statusbar. I will add the change on the next update.


adding FUNDING.yml

As an Open Source project, we @unlock-protocol believe Rimble should have a FUNDING.yml file which lets people make donations if they use the software.
This pull requests adds donations thru an Unlock lock. Locks are smart contract which let people become members by purchasing a key. The key itself is a non fungible token: 🗝 .
You can make a donation (and purchase a key!) on this page (linked inside of the FUNDING.yml file)
This specific lock is at 0xE7dD173114d9be8Cc52ea693277d8c2F3Af416F1. The curent key price is 1 DAI. I currently own the lock but I'll happily transfer its ownership to you if you can send me an address ;) After that, you will be able to update the price, the name and withdraw the funds from the Unlock dashboard.
Keys are valid for 1 month and there can be an infinite number of keys.
PS: if you want to create your own FUNDING.yml link you can easily with this application.

One quick addition, when/if you merge this you also need to enable donations in the settings (see https://github.com/ConsenSys/rimble-ui/settings) :

Hello @gesquinca ! I'd love to get your feedback on that one. We blogged about the motivations behind that PR.
One of the next steps that we're thinking of is to create some kind of "shared" lock between many Ethereum OSS projects. The idea is that people can then chose to support a single project, like Ethlint, or the whole ethereum OSS ecosystem. This shared lock would belong to some kind of shared address (or even a DAO) to organize the funds and maybe distribute them to specific projects.
Anyway, my point is that I'd love to kick off a discussion here :)

@julien51 Thanks for the update. I'll check out your writings. I do really like what you're all doing.
Update on our end. We still haven't gotten a solid answer on who should custodian any possible assets from this, so we'll keep on deferring this PR until then.


A–List Issue #5 News and Events

News
ECMA and TC39
Members of the Node.js project have begun engaging with TC39 helping better define the needs of the ECMAScript spec and the JavaScript language as it evolves. Much of ECMAScript's history and evolution has been defined by the needs of browser vendors. Today's full stack world has evolved and Node.js is directly hosting or indirectly executing today's JavaScript tooling. The need for collaboration is greater than ever.
Standards and Governance for Node.js and the Web Platform
TC39 - ECMAScript - Get involved

JUL 24-26: TC39 65th meeting - Redmond, CA
SEP 25-27: TC39 66th meeting - New York, NY
NOV 27-29: TC39 67th meeting - Cupertino, CA

Events
2018

JUL 10: Always Forward with dshaw: Dev Related on Modules - with @tzmanics
JUL 23-25: Node Summit - San Francisco, CA
JUL 26-27: Node Summit Training Days - San Francisco, CA - Organize by dshaw 🌈🦄
AUG 21-23: JSConf US 2018 – Carlsbad, California
OCT 10-12: JS Interactive - Vancouver, BC, Canada
OCT 12-13:  Node.js Collaborator Summit – Vancouver, BC, Canada.
NOV 15-8: Cascadia JS – Seattle, WA.

2019

FEB: JSConf Hawaiʻi - Honolulu, HI

@codeekage Let's add:

https://www.lfasiallc.com/events/kubecon-cloudnativecon-china-2018/
https://events.linuxfoundation.org/events/kubecon-cloudnativecon-north-america-2018/

SEP 9: NodeConf NG 2018 - Lagos, Nigeria


No Corresponding FORM - If "SELECT UP TO 1 ROWS" is inside the FORM

Hello,
I don't know if this is a issue, but I have an old report in which a SELECT is executed with the addition "UP TO 1 ROWS". This happens inside a FORM.  When I want to run the ABAP CLeaner, I get the following error message:

If I comment out the addition "UP TO 1 ROWS", it works again.

Hi JoachimEck,
this sound a bit like ABAP cleaner (wrongly) expects an ENDSELECT and therefore doesn't expect the FORM block to be closed yet when the ENDFORM is parsed. Unfortunately, it is not trivial to determine whether or not ENDSELECT is required – could you post the SELECT statement used here? (Exact table and field names don't matter, but the structure of the SELECT).
Kind regards,
Jörg-Michael

Hi Jörg-Michael,
here's the SELECT-Statement:
SELECT MAX( col ) UP TO 1 ROWS
FROM table
INTO variable
WHERE something EQ else
AND something1 EQ else1 .
I have not been an ABAP developer for very long. I would have said that only one value comes back with the SELECT, so I could leave out the addition, but I have read that it can apparently also be that several values come back. The report is already old and I don't want to change the logic.
Kind regards,
Joachim

Hi Joachim,
ah, very good, now I could reproduce the issue. What confuses ABAP cleaner here is that "UP TO 1 ROWS" is in an unusual position – you'd usually have it after the "INTO" clause (see ABAP documentation for SELECT):
SELECT mainquery_clauses
  [UNION|INTERSECT|EXCEPT ...]
  INTO|APPENDING target
  [UP TO ...] [OFFSET ...]
  [abap_options].
  ...
[ENDSELECT].

But nevertheless, since the ABAP compiler accepts the syntax from your example, ABAP cleaner should cope with it, too, so I shall try to fix this! In the meantime, if you move "UP TO 1 ROWS" below "INTO variable", ABAP cleaner should be fine with it, too.
Kind regards,
Jörg-Michael

Thanks a lot :)
Kind regards,
Joachim

Hi Joachim,
this issue will be fixed in the next release!
Kind regards,
Jörg-Michael

Hi Joachim,
thanks again for this finding – this should now be fixed in version 1.8.0, which was just released!
Kind regards,
Jörg-Michael


Add mysql-client

brew reference:  https://github.com/Homebrew/homebrew-core/blob/47fe0f1860f73ffe393074de1b749ac8a0e0ee10/Formula/m/mysql-client.rb
docker reference:  https://github.com/Logiqx/mysql-client/blob/master/Dockerfile

This is a tricky one... The homebrew formula uses a tarball which is linked from nowhere (or my google-fu is bad).
The mysql homepage does not list specific client packages.
I would have to build from the source repo and see how to use the result.
Let me meditate on this 🧘

This is not going anywhere. Looking at the alpine package mysql-client this is way to complicated.


Search does not work for map key

Hi,
search does not seem to get any results when searching for the key of the map. This works in BeatDrop.
I guess this is because according to https://github.com/Alaanor/beatlist/blob/master/src/lib/BeatSaverAPI.ts#L17 the search/text endpoint is used but BeatDrop uses search/text/all (https://github.com/StarGazer1258/BeatDrop/blob/master/src/actions/searchActions.js#L51)
Also while looking at that piece of code I noticed that URL encoding might be missing in all functions in https://github.com/Alaanor/beatlist/blob/master/src/lib/BeatSaverAPI.ts .

Nice catch ! You're valid on both points. I'll fix that for the next version. (See rewrite branch)

I was going to work on this on the rewrite branch and I noticed that the /all endpoint does not exist but doesn't fail either. Here's the source for the API endpoint on beatsaver side. It silently ignores as there's a page keyword available here /search/[page]?q=[search] if it's not a number he just uses the default value: 0.
What beatdrop do however (from what I can read on the code) is another request if he detects the search to be a key.
I'm not sure how I will do on my side yet, but I will implement a way to search by its key. 👌


compareValues only works for string/string or number/number

We wouldn't want to be calling this function on an arbitrary T.
Noticed this during review of #19452.

Mind if I just pull this into #19452?

Sure.


Changing configuration parameter to sensor level

Having an idea about the configuration parameters, that I would like to discuss. We now have configuration parameters as lists and dictionaries. In each list we have to define a MAC address, like this:
sensor:
  - platform: mitemp_bt
    encryptors:
      'A4:C1:38:2F:86:6C': '217C568CF5D22808DA20181502D84C1B'
    sensor_fahrenheit:
      - 'C4:7C:8D:6B:4F:F3'
    sensor_names:
      'A4:C1:38:2F:86:6C': 'Livingroom'
    whitelist:
     - 'B4:7C:8D:6B:4F:D3'

I think it would make more sense to define this per sensor, something like this.
sensor:
  - platform: mitemp_bt
     automatic_add: False
     entities:
     - mac: 'A4:C1:38:2F:86:6C'
       encryption_key: '217C568CF5D22808DA20181502D84C1B'
       name: 'Livingroom'
     - mac: 'C4:7C:8D:6B:4F:F3'
       unit_of_measurement: fahrenheit
     - mac: 'B4:7C:8D:6B:4F:D3'

I understand that this is a significant breaking change, but it has the advantage that we can expand this in a second step to define other parameters on sensor level, like rounding, median, etc.
I’m not saying that we have to make this change, but I see some advantages, despite the breaking change (and quite some work to make this change). Let’s first discuss @Magalex2x14 what you think of this idea and see where we end up.

When I tested the changes with the whitelist today, I had the thought that something needs to be done with these dicts/lists, but I never got to the point of trying to formulate what exactly to do with them)
Looks great to me. We prepare lists for the parser only once at the start of the system, so the complication of this stage will not affect the overall performance of the component. Yes, this is a breaking update, but I think users will support it as the new format is much clearer.
The only thing is, instead of entities, I would write sensors (or devices?). And I also want to change automatic_add to something more sonorous ) If I understand correctly, this is a reference to whitelist: true. Something like autodiscovery, maybe )
In addition, if my memory serves me, there was a request from users to individually configure the averaging and rounding parameters for each device...

I will have a look tomorrow what the usual parameter naming is. I’ve seen sensors, as well as entities, I thought. I used automatic_add, as it was a parameter for rfxtrx component. autodiscovery sound better, I agree. I will check the developer docs of home assistant.
For the rounding option, etc, we can make a parameter on component level, which can be overruled on sensor level (or perhaps even on temperature sensor/ humidity sensor level).
Lets start with the dictionaries and lists first. I will start looking into this and will make a separate branch for this.

I did a study on the most common parameters in the code.

device (73x)
devices (70x)
sensors (44x)
entities (20x)

and

discovery (18x, but some are doublures)
whitelist (6x)
automatic_add (12x, but 10 of them in rfxtrx)
discover (1x)
auto_discovery (0x)
autodiscovery (0x)

I noticed that the following parameters are in homeassistant/const.py

CONF_ENTITIES = "entities"

CONF_DEVICE = "device"

CONF_DEVICES = "devices"

CONF_SENSORS = "sensors"

CONF_DISCOVERY = "discovery"

CONF_WHITELIST = "whitelist"

I think, based on the above, I would choose discovery and devices.
sensor:
  - platform: mitemp_bt
     discovery: False
     devices:
     - mac: 'A4:C1:38:2F:86:6C'
       encryption_key: '217C568CF5D22808DA20181502D84C1B'
       name: 'Livingroom'
     - mac: 'C4:7C:8D:6B:4F:F3'
       unit_of_measurement: fahrenheit
     - mac: 'B4:7C:8D:6B:4F:D3'

What do you think?

Wow) Serious approach)
I like it, yes.

I've changed unit_of_measurement to temperature_unit. See config validation
sensor:
  - platform: mitemp_bt
     discovery: False
     devices:
     - mac: 'A4:C1:38:2F:86:6C'
       encryption_key: '217C568CF5D22808DA20181502D84C1B'
       name: 'Livingroom'
     - mac: 'C4:7C:8D:6B:4F:F3'
       temperature_unit: F
     - mac: 'B4:7C:8D:6B:4F:D3'

Finished with 0.8.1 release.


Can You provide any example how to use this API?

When I tried to use with pods it's giving me error saying DLnotificationscheduler initializer is inaccessible due to internal protection

it would help me lot in my current app in which i am going to trigger notification on geo-fenced area.

Hello,
There was some errors that I fixed and it should work now. Re-install the pod and make sure it updates to release 0.04. The documentation for the location function can be found here


Compiling on Windows 64 bit and Python 3

Hi @thesamovar
have you ever succeeded in compiling KK2 on Windows 64 bit and Python 3? I tried with no success..

I haven't tried because I run Python 2.x only. What happened when you tried?

The classic errors on windows: unable to find vcvarsall, or some bug in cygwincompiler.py, etc etc. I tried many things with the Visual Studio compilers. Using mingw64 is not possible apparently.
At some point we should resolve this on a dedicated build machine and ship binstar binaries so that no one else needs to do the compilation!

Oh there's a fix for this that we're using in Brian 2 but I don't remember what it is exactly. @mstimberg? I think it's just something like you have to import setuptools before distutils or something.

Yes, we compile C extensions for Python 3 on Windows 64 Bit with MSCV 10/SDK 7.1 on appveyor. I think it also works with the "C++ for Python" compiler, but I'd have to check that again (I'm sure it works for Python 2, but I did less testing for Python 3). The trick is indeed to import setuptools (a fairly recent version, but everything >=6 should work, I think) in the beginning, it will monkey-patch distutils and then things work.

@mstimberg have you got this working with MSVC 2010 Express Edition? Or do I need to get hold of the full edition?

I think all that is needed is the SDK 7.1, I don't have any full MSVC compiler on my (virtual) Windows machine. The tricky bit is that you have to set up the environment in the right way. We do this in a script on appveyor, it is a copy from this repository: https://github.com/ogrisel/python-appveyor-demo The relevant bits are here: https://github.com/ogrisel/python-appveyor-demo/blob/master/appveyor/run_with_env.cmd#L35
The necessary steps are also described here: https://github.com/cython/cython/wiki/CythonExtensionsOnWindows#using-windows-sdk-cc-compiler-older-method

I'm actually not that sure anymore that the monkeypatching by setuptools is necessary here, maybe it was only necessary for the C++ for Python 2.7 compiler.

I've managed to get it to build (python 3.4, 64-bit) by installing VS2010 (full edition, express only has 32-bit). Now I'm trying to build a binstar package but I guess something to do with the monkey-patching is failing...
  File "C:\Users\Max\Miniconda\lib\subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['C:\\Users\\Max\\Miniconda\\Scripts\\pa
tch.exe', '-p0', '-i', 'c:\\users\\max\\appdata\\local\\temp\\tmpqu71m7conda_ske
leton_klustakwik2-0.2.2.tar.bz2\\pypi-distutils.patch']' returned non-zero exit
status 2

We'll get there... slowly!
@mstimberg have you had any experience with binstar packages?

would it be simpler to use wheels and pip?

hey I've solved this previous issue - turns out there was an issue with conda's version of patch. I built it from source.
Then tried to file an issue and discovered it was already filed - a year ago :(
https://github.com/conda/conda-build/issues/119

boom - now it's all done https://binstar.org/kwikteam/klustakwik2 (for win64 python 2.7 / python 3.3 / python 3.4 only - other platforms coming at some point)

Nice work! Will look in more detail soon.
On 2 July 2015 23:49:40 BST, nippoo<EMAIL_ADDRESS>wrote:

boom - now it's all done https://binstar.org/kwikteam/klustakwik2 (for
win64 python 2.7 / python 3.3 / python 3.4 only - other platforms
coming at some point)

Reply to this email directly or view it on GitHub:
https://github.com/kwikteam/klustakwik2/issues/63#issuecomment-118187425

@nippoo could you tell me how you built the conda packages? I'd like to update newer releases on anaconda cloud (Python 3.5 etc). I now have a Windows setup that can build Python packages on Py 2.7 and Py 3.5 (I have Visual C++ 2015 installed for the latter).
I also uploaded Windows wheels on PyPI so I think pip install klustakwik2 might just work on Windows?


Cannot build usql

I'm getting an error while trying to build usql 0.9.0
$ go get github.com/xo/usql
go get<EMAIL_ADDRESS>updating to
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>reading github.com/myesui/uuid/go.mod at revision v1.0.0: unknown revision v1.0.0
can you fix it please?
Thanks.

Looks like a duplicate of #250

Well I did remove the go.mod, go.sum, but after that you need to do go mod init xxxx. Even though I'm still getting:
$ go build go<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>reading github.com/myesui/uuid/go.mod at revision v1.0.0: unknown revision v1.0.0

I found similar issues in other repositories where people report that github.com/myesui/uuid has been removed. The most similar one is this: https://github.com/ProtonMail/proton-bridge/issues/163 We should report this issue to https://github.com/sql-machine-learning/gomaxcompute so they can fix their dependencies.

When you call go build, you could add -tags 'no_maxcompute'.

Hmm it's weird. I can seen no maxcompute.go this tag:
// +build all,!no_maxcompute most,!no_maxcompute maxcompute,!no_maxcompute
it's generated code... however is is correct? Anyway it didn't help:
$ go build -tags no_maxcompute
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>reading github.com/myesui/uuid/go.mod at revision v1.0.0: unknown revision v1.0.0

@kenshaw I don't think we should close this one. github.com/myesui/uuid is gone but it's still being cached in proxy.golang.org. I don't know if it has a TTL, but it will become a problem for all users at some point.
This is reproducible if you set export GOPROXY=direct and remove that package from ~/go/pkg/mod.
For the time being, can we make maxcompute optional?

I also just noticed that there's no LICENSE in https://github.com/sql-machine-learning/gomaxcompute. I'm not sure if we care, but I wouldn't use such module.


Constantly crashing on startup

First time happening to me and needing some help on what is the issue

Which steps have you taken to try and fix it? Have you tried uninstalling and reinstalling XivAlex?

yeah, repeatedly done but the same issue comes up

Hello. I am having the same issue. Any fix?