Error when create BrowserDriverBuilder with chrome

When I create a new instance off BrowserDriverBuilder with type chrome a receive an error.
Example:
$driver = (new BrowserDriverBuilder('http://localhost:4444/wd/hub'))
                 ->withType('chrome');

Error:
TypeError: Argument 1 passed to Facebook\WebDriver\Chrome\ChromeOptions::addArguments() must be of the type array,
null given, called in ./vendor/athena-oss/php-fluent-webdriver-client/src/Browser/BrowserDriverBuilder.php on line 239

One solution I found was that I can pass chrome option like:
$driver = (new BrowserDriverBuilder('http://localhost:4444/wd/hub'))
            ->withType('chrome')->withChromeOptionsArguments([]);

But I think the best solution would be to add a default value to $chromeOptions var in the BrowserDriverBuilder.php file
Tested in version: v0.7.2

@antfig-fixeads I think we could use your idea of having a default value to $chromeOptions. Can you please create a PR with your change?

@caiomoritz yes I can do that, no problem

@caiomoritz you can close the issue I guess :)
You will create some new tag? because I have my composer with ^0.7 and don't update the code with latests changes

@antfig-fixeads please check release v0.7.3


Crop improvements and  toolbar overlapped fix

Added circle crop type
Added crop overlay color configuration. Previously, the default one was white and barely visible (specially on top of light background photos)
Added grid overlay configuration
Fixed toolbar overlapped by image. Previously, for ratio less than 1 the image was overlapping the toolbar and its buttons

Note:
This is my first experience using Stevia, so I believe there are more convenient ways to layout YPCropView subviews rather than my implementation.

Has this been merged?  When you crop do you now get an outline of the crop area (again very hard to use with standard white background)

Has this been merged? When you crop do you now get an outline of the crop area (again very hard to use with standard white background)

@justdan0227
The pull request is still open (not merged).
Yeah, now you can see the crop area specially for ratio less than 1. In addition you can customize the overlay color to match with you app design and needs using the property below.
/// Defines the crop overlay color. Default is label color with alpha of 0.4
public var cropOverlayColor: UIColor = .ypLabel.withAlphaComponent(0.4)
In the example below the showsCrop property was set to .rectangle(ratio: 0.4)

Thanks until this is merged I can't find cropOverlayColor to try out your suggestion above?

Try to checkout my fork: https://github.com/tsahi-deri/YPImagePicker

Bingo!  Thanks!

Take a look with this crop functional already merged https://github.com/Yummypets/YPImagePicker/commit/6c5460a08a52f389f8ce93bb839cc34b8e5a9380

@NikKovIos My implementation was indeed based on that pull request but I also added a couple necessary fixes on top of it. Please take a look in PR description.

@tsahi-deri Sorry, I didn't find your PR first. Could you please pull from YPImagePicker master and test your implementation. If it is ok, I will merge it too.

@NikKovIos Done, please check my PR. Thanks

@NikKovIos Will you create a new release soon with all latest changes ?

@tsahi-deri think so. Now @s2mr gonna check spm and cocapods dependency and then we will ready to release.

@s2mr Hey, any update about the next release ?

Why is it that cropOverlayColor is not found in master?  Has this not been merged?

@justdan0227 The branch is fully merged into master but a new version is not released yet.
So you probably checked out version 4.5.0 without this feature.
@s2mr  any update about the next release ?

Sorry, I don't know this topic.
Did you make a mistake mention?

@s2mr Please see NikKovIos comment above regarding the a new release with all latest changes.

Check 5.0.0 release please.


docs/Dockerfile.build: use sphinx version 1.7.9

This may fix #576

For safety might also want to version sphinxcontrib-googleanalytics

@SteveOss I just updated it to version sphinxcontrib-googleanalytics @ 0.1, but there hasn't been an update to that pip package since it's initial release in 2010, so seems unlikely that would break.

@SteveOss can you re-review? Thanks


AttributeError: 'NoneType' object has no attribute 'find_all'

Hi,
if i want to monitor any Interface (throughput option) i get following error:
THROUGHPUT UNKNOWN: AttributeError: 'NoneType' object has no attribute 'find_all'
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/runtime.py", line 43, in wrapper
return func(*args, **kwds)
File "/usr/local/lib/python2.7/dist-packages/check_pa/check_paloalto.py", line 22, in main
check.main(verbose=args.verbose, timeout=args.timeout)
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/check.py", line 120, in main
runtime.execute(self, verbose, timeout)
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/runtime.py", line 128, in execute
with_timeout(self.timeout, self.run, check)
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/platform/posix.py", line 19, in with_timeout
func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/runtime.py", line 117, in run
check()
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/check.py", line 105, in call
self._evaluate_resource(resource)
File "/usr/local/lib/python2.7/dist-packages/nagiosplugin/check.py", line 72, in _evaluate_resource
metrics = resource.probe()
File "/usr/local/lib/python2.7/dist-packages/check_pa/throughput.py", line 72, in probe
for item in ifnet.find_all('entry'):
AttributeError: 'NoneType' object has no attribute 'find_all'
OS is Debian Jessie, latest patches.

Hi,
there is a problem finding the "ifnet" entry in the returning XML. Maybe you are querying the wrong interface? To get all available names of your interfaces, please have a look at https://www.paloaltonetworks.com/documentation/61/pan-os/pan-os/getting-started/configure-interfaces-and-zones.html
In my environment the XML-format for throughput looks like this:
<?xml version="1.0" encoding="utf-8"?>
<response status="success">
    <result>
        <ifnet>
            <entry>
                 ...
            </entry>
        </ifnet>
    </result>
</response>

To find the exact issue, please open a shell and insert the following command (please change HOST, TOKEN and INTERFACENAME with your own).
check_paloalto -H HOST -T TOKEN throughput -i INTERFACENAME -v -v -v
You should see something like this:
...
Reading XML from http://PA-IP/api/...cmd=<show><counter><interface>INTERFACENAME</interface></counter></show>
...

Please have a look at this URL and paste the XML output here.

Hi,
i tried and my result is:

But your hint regarding interfaces was pointing me into the right direction. its case sensitive and Palo is naming their Interfaces in small letters...other than Cisco :)
Now as i'm using the correct writing it seems to work!
Thank you for the quick support!!!

seems to be resolved. thx again!


showing a thumbnail image in admin add page causes the error

In order to show a thumbnail image, add following lines to admin.py;
class PhotoAdmin(admin.ModelAdmin):
admin_thumbnail = AdminThumbnail(image_field='thumbnail')
list_display = ('caption, 'admin_thumbnail')
fields = ['admin_thumbnail', 'original_image', 'caption']
The thumbnail is shown in the list and the detail page.
But when adding a new photo, the error occurs;
File "/usr/lib/python2.6/site-packages/django/template/debug.py" in render_node
73.             result = node.render(context)
File "/usr/lib/python2.6/site-packages/django/template/debug.py" in render
90.             output = self.filter_expression.resolve(context)
File "/usr/lib/python2.6/site-packages/django/template/base.py" in resolve
510.                 obj = self.var.resolve(context)
File "/usr/lib/python2.6/site-packages/django/template/base.py" in resolve
653.             value = self._resolve_lookup(context)
File "/usr/lib/python2.6/site-packages/django/template/base.py" in _resolve_lookup
698.                             current = current()
File "/usr/lib/python2.6/site-packages/django/contrib/admin/helpers.py" in contents
171.             f, attr, value = lookup_field(field, obj, model_admin)
File "/usr/lib/python2.6/site-packages/django/contrib/admin/util.py" in lookup_field
214.             value = attr(obj)
File "/usr/lib/python2.6/site-packages/imagekit/admin.py" in call
28.                 obj, self.image_field))
Exception Type: TemplateSyntaxError at /admin/lupines/photo/add/
Exception Value: Caught Exception while rendering: The property  - None is not defined on thumbnail.
And local values are;
self 	imagekit.admin.AdminThumbnail object at 0xa6409ac
obj 	Photo:  - None
thumbnail 	ImageSpecFile: None

Ah. This is happening because empty files are falsy as of 8147cb85df162b1731490970a156e07ab90366cc.
Also, the order of the arguments in the string error interpolation are backwards! Would make a better error message if it made sense.

3022eb037d7bb18b0f1b44a0bd38dc8e0dd21382 Should fix this. Thanks for the report!


Build failure: samba with pkgs.sambaFull

Steps To Reproduce
Steps to reproduce the behavior:

enable samba with configuration option  services.samba.package = pkgs.sambaFull;

run nixos-rebuild switch

Build log
error: builder for '/nix/store/87n50bz3hzyhkgn4qkfm5nn6jdlv3h1z-python3.10-flask-sqlalchemy-3.1.1.drv' failed with exit code 1;
       last 10 log lines:
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
       >   File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
       >   File "<frozen importlib._bootstrap_external>", line 883, in exec_module
       >   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
       >   File "/nix/store/45kqk9na9lqb7bf8dighylafbx724zwi-python3.10-flask-sqlalchemy-3.1.1/lib/python3.10/site-packages/flask_sqlalchemy/__init__.py", line 5, in <module>
       >     from .extension import SQLAlchemy
       >   File "/nix/store/45kqk9na9lqb7bf8dighylafbx724zwi-python3.10-flask-sqlalchemy-3.1.1/lib/python3.10/site-packages/flask_sqlalchemy/extension.py", line 40, in <module>
       >     t.Type[sa_orm.DeclarativeBase],
       > AttributeError: module 'sqlalchemy.orm' has no attribute 'DeclarativeBase'. Did you mean: 'declarative_base'?
       For full logs, run 'nix log /nix/store/87n50bz3hzyhkgn4qkfm5nn6jdlv3h1z-python3.10-flask-sqlalchemy-3.1.1.drv'.
error: 1 dependencies of derivation '/nix/store/0jsm7x6m4ny8i3h8pg8h0zsf3cdxr72l-python3.10-factory-boy-3.3.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/a62a0lcd34lnicn651kjrmrpiqfca83r-python3.10-pytest-randomly-3.13.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/ldiwm8d0jl22cjh45fszsh68s16ridc5-python3.10-httplib2-0.21.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/v5w7j1jq2n3g300wlnciidiy2ypahncj-python3.10-oauth2client-4.1.3.drv' failed to build
error: 1 dependencies of derivation '/nix/store/wpkagz3jp9avmwa7nfwlpdvhpxmc5jdg-python3.10-google-auth-2.21.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/9qkdp10i79xwlna3c728008vh49m1zhx-python3-3.10.13-env.drv' failed to build
error: 1 dependencies of derivation '/nix/store/3mimc8673720pz8cp2xvkc9vcz76nvd4-ceph-18.2.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/qaspqpvr9r23fy1i5bc4jfcp96pxabqh-samba-4.19.1.drv' failed to build
error: 1 dependencies of derivation '/nix/store/cjzkbxnzdms5frd8slwwmz31caymwcf5-man-paths.drv' failed to build
error: 1 dependencies of derivation '/nix/store/wp4h3jpdm3n42q5ngddlnl1jz4hga6pq-samba-4.19.1_fish-completions.drv' failed to build
error: 1 dependencies of derivation '/nix/store/ylibh561ri5291r26fq1myncgfvccqpx-system-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/nngk1drzmr0d4kfkrzzv19zlxvps5ykn-unit-samba-nmbd.service.drv' failed to build
error: 1 dependencies of derivation '/nix/store/7ywpvi9rkk7jcaz9nmn883k3bgwzn18a-unit-samba-smbd.service.drv' failed to build
error: 1 dependencies of derivation '/nix/store/2czkvhd63022bss58c12acf9r034dy7b-unit-samba-winbindd.service.drv' failed to build
error: 1 dependencies of derivation '/nix/store/gchz1wh1lr8msbn6xv24rg1k0vp6ybm7-nixos-system-tyan-23.11.20231024.8efd5d1.drv' failed to build

Additional context
nixpkgs rev 8efd5d1e283604f75a808a20e6cde0ef313d07d4
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.8, NixOS, 23.11 (Tapir), 23.11.20231024.8efd5d1`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.1`
 - nixpkgs: `/nix/store/z1nvpjx9vd4151vx2krxzmx2p1a36pf9-source`

https://github.com/NixOS/nixpkgs/pull/263439


[Bug] TE hybrid parser doesn't work with latest Python version (3.9+)

This issue below happens during importing onnx model in tvm with relay, from_onnx more specifically. The stack trace is as follows:
Actual behavior
"/home/yj/tvm/python/tvm/te/hybrid/parser.py", line 466, in <listcomp>
    args = [self.visit(i) for i in node.args]
  File "/home/yj/anaconda3/lib/python3.9/ast.py", line 407, in visit
    return visitor(node)
  File "/home/yj/tvm/python/tvm/te/hybrid/parser.py", line 379, in visit_Subscript
    for i in args:
TypeError: 'IntImm' object is not iterable

Expected behavior
This model should be imported without an error under Python 3.9
Environment
TVM commit a94558686834c4ff7e4d90e4bc1fde5451ce445d
Python 3.9
Steps to reproduce
import sys
from os import path as osp
import onnx
import numpy as np
import tvm
from tvm import relay
from tvm.meta_schedule.tune import tune_relay
from tvm.target.target import Target

if __name__ == "__main__":
    target = Target("llvm --num-cores=8")

    dev = tvm.cpu() if str(target.kind) == "llvm" else tvm.cuda()
    onnx_model = onnx.load(f"/home/yj/models/ssd.onnx")
    shape_dict = {}
    datas = {}
    input_shape = [1, 3, 1200, 1200]
    input_dtype = "float32"
    input_name = "image"
    shape_dict[input_name] = input_shape
    datas[input_name] = tvm.nd.array(np.random.randn(*input_shape).astype(input_dtype), dev)
    mod, params = relay.frontend.from_onnx(onnx_model, shape_dict, freeze_params=True)

The model is ssd.onnx downloaded from OctoML's aquarium.
cc: @areusch

#8577 is tracking this problem.

closing in favor of #8577


Problem with promises and onjs

In trying to get the second example here to work in a static page, I'm encountering a problem.
I want to make an interactive figure using WGLMakie, but whenever I include a promise in a function I give to onjs the figure ends up not being rendered, and I get errors in the browser console (in both Firefox and Chrome):

The promises seem to work fine in on_document_load, so I wonder if this has something to do with things not being loaded yet, but I expect such a problem would be apparent when making the linked example too.
Example code I'm having problem with (simplified from the linked example)
using Bonito, WGLMakie
Page(exportable=true, offline=true)

App() do session::Session
    s1 = Slider(1:100)
    slider_val = DOM.p(s1[])
    fig, ax, splot = scatter(1:4)

    on_document_load(session, js"""
        $(splot).then(plots=>{
            const scatter_plot = plots[0]
            console.log(scatter_plot)
        })
    """)

    onjs(session, s1.value, js"""function on_update(new_value) {
        $(splot).then(plots=>{
            const scatter_plot = plots[0]
            scatter_plot.geometry.attributes.pos.array[0] = (new_value/100) * 4
            scatter_plot.geometry.attributes.pos.array[1] = (new_value/100) * 4
            scatter_plot.geometry.attributes.pos.needsUpdate = true
        })
    }
    """)
    return DOM.div(s1, fig)
end

In the above, if I remove the onjs function, everything works as expected.

Should be fixed by #224.
Btw, I'd rather use:
export_static("index.html", app)
instead of Page(exportable=true, offline=true), which is a bit fragile.

Thank you for the quick response!
As for the export_static advice, my impression is that using this would give me HTML files themselves, whereas I'd like to embed these plots in other things. Similar to the WGLMakie documentation but using a different tool. So is there a right way to use export_static I'm not aware of, or would Page be fine in this case?


Migrate release provider to new provider infrastructure

Resolves se-passau/VaRA#602

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 4

Complexity increasing per file
==============================
- varats/data/provider/release/release_provider.py  10

Clones removed
==============
+ varats/projects/c_projects/gzip.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- varats/data/provider/release/release_provider.py  10

Clones removed
==============
+ varats/projects/c_projects/gzip.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- varats/data/provider/release/release_provider.py  10

Clones removed
==============
+ varats/projects/c_projects/gzip.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity increasing per file
==============================
- varats/data/provider/release/release_provider.py  10

Clones removed
==============
+ varats/projects/c_projects/gzip.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity increasing per file
==============================
- varats/data/provider/release/release_provider.py  10

Clones removed
==============
+ varats/projects/c_projects/gzip.py  -1

See the complete overview on Codacy


Reset selected Closed Caption language to fix crash if it is not available for playing video.

App was getting crash if selected language in not available for playing video . I reset selected Closed Caption language to fix crash if it is not available for playing video.
JIRA : https://openedx.atlassian.net/browse/MOB-1347
@aleffert  @nasthagiri  Please review.

This will reset the language, but that means if I switch between videos where one has translations and one doesn't, my preferences will keep getting lost. Instead we should fall back to what's available if there are no translations, but not actually change the user's preferences, so that when they switch videos their preference is preserved.

Removed code which was responsible for resetting persisted language.
@aleffert  Please review .


feat: added UI for creating charges

Related Issue(s)

resolves #300

Quick overview of the change

https://user-images.githubusercontent.com/11084692/197110376-3971330d-b9fc-4675-ac94-502d40b8e17f.mov

Added UI for creating charges
Added UI elements for AE after charges have been created

Quick testing guide

Also want to ask about the user flow, but would the associate show this screen to the user for them to enter the tip amount? or is it something they would add in after they pay on their own time?

@Wuon This would depend on what the client prefers, I sorta picked a middle of the road option for now but we can change it depending on how they feel


Deselecting item when disabled

Description
If the selection is disabled (isDisabled={true}), the selected option can still be deselected by clicking on the cross.

chakra-react-select Version
2.0.0
Link to Reproduction
No response
TypeScript?

[ ] Yes I use TypeScript

Steps to reproduce

Select some items
Disable select
Deselect them by clicking on the cross

Operating System

[ ] macOS
[ ] Windows
[X] Linux
[ ] iOS/iPadOS
[ ] Android

Additional Information
No response

Thanks for the bug report! I have figured out what is wrong and will push a new version with a fix shortly.

This is now fixed in v4.1.5!  Check out this demo to see it in action:
https://codesandbox.io/s/chakra-react-select-isdisabled-fix-vn24dg?file=/example.js


No such host error handling

Description of Bug
Need better handling of the following error Get "https://REDACTED.tld/api/route": dial tcp: lookup REDACTED.tld on 10.10.0.X:53: no such host. It currently returns the error for every single endpoint; instead it should return the error once and try additional hosts if any are known.

Fixed in 2b83f36


Allow form to submit with validation errors

Simply warn users about validation errors, but proceed to generate the cards regardless.

Validation should be removed altogether, there's no use case for it, as the forms are not submitted.


Performance analysis for different values of cache_block_seq_len

Motivation
@lvhan028 @lzhangzz @grimoire We tested the performance for different values of cache_block_seq_len parameter and got the following results:

cache_block_seq_len
RPS
FTL (min)
FTL (max)
FTL (avg)
PTL (50%)
PTL (75%)
PTL (95%)
PTL (99%)

1
8.759
0.058
4.065
0.437
0.041
0.059
0.116
0.182

8
8.728
0.06
4.072
0.438
0.041
0.059
0.116
0.188

16
8.718
0.068
4.218
0.458
0.042
0.061
0.114
0.181

32
7.218
0.069
4.26
0.449
0.051
0.065
0.129
0.178

64
7.337
0.07
4.265
0.468
0.049
0.064
0.13
0.194

128
7.245
0.061
4.102
0.571
0.049
0.063
0.131
0.189

256
6.944
0.064
5.451
1.844
0.045
0.059
0.13
0.19

512
6.586
0.168
8.909
4.98
0.038
0.046
0.109
0.159

1024
5.838
0.108
14.542
10.414
0.029
0.033
0.074
0.124

(The FTL is the first token latency and the PTL is the per-token latency)
Experiment Settings

backend engine: turbomind
model: llama2_13b_chat
dataset: ShareGPT_V3
batch_size (concurrency): 128
num_prompts: 1000

Findings

The throughput (RPS) is decreasing when we increase the cache_block_seq_len.
We get the largest RPS when cache_block_seq_len = 1.
When cache_block_seq_len >= 256:

The FTL increased a lot, that means the prefill phase is slower.
The PTL decreased a lot, that means the decode phase is faster.

Possible Explanation

The prefill phase is computationally intensive and the decode phase is memory access intensive. Larger block size will reduce the times of memory access, but will reduce the GPU memory utilization which will cause smaller batch size and smaller parallelism.

Conclusion
The cache_block_seq_len parameter have significant influence to the system performance. Maybe we need to discuss how to set the default value of the cache_block_seq_len appropriately and tell users this parameter can be tuned.
Related resources
No response
Additional context
Reproduction Procedure

Modify the cache_block_seq_len in ${triton_model_path}/config.ini
Run test command:

python benchmark/profile_throughput.py /workdir/ShareGPT_V3_unfiltered_cleaned_split.json ./workspace --concurrency 128 --num-prompts 1000

Environment
sys.platform: linux
Python: 3.11.5 (main, Sep 11 2023, 13:54:46) [GCC 11.2.0]
CUDA available: True
MUSA available: False
numpy_random_seed:<PHONE_NUMBER>
GPU 0: NVIDIA A100-SXM4-80GB
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.8, V11.8.89
GCC: gcc (GCC) 10.2.1 20210130 (Red Hat 10.2.1-11)
PyTorch: 2.2.0+cu118
PyTorch compiling details: PyTorch built with:
  - GCC 9.3
  - C++ Version: 201703
  - Intel(R) oneAPI Math Kernel Library Version 2022.2-Product Build 20220804 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v3.3.2 (Git Hash 2dc95a2ad0841e29db8b22fbccaf3e5da7992b01)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 11.8
  - NVCC architecture flags: -gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_90,code=sm_90
  - CuDNN 8.7
  - Magma 2.6.1
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.8, CUDNN_VERSION=8.7.0, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -D_GLIBCXX_USE_CXX11_ABI=0 -fabi-version=11 -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-stringop-overflow -Wsuggest-override -Wno-psabi -Wno-error=pedantic -Wno-error=old-style-cast -Wno-missing-braces -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=2.2.0, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=1, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF, USE_ROCM_KERNEL_ASSERT=OFF,

LMDeploy: 0.2.4+a270a8d
transformers: 4.37.2
gradio: 3.50.2
fastapi: 0.109.0
pydantic: 2.6.0

Hi @lvhan028 @lzhangzz @grimoire
Perhaps we may consider updating the default value here: https://github.com/InternLM/lmdeploy/blob/be9c15a9f1f360f1d06941a8dd6989af464022e8/lmdeploy/turbomind/deploy/target_model/base.py#L59
Additionally, it might be beneficial to include logging and documentation to guide users on adjusting this parameter for various scenarios.

And when cache_block_seq_len == 0, it essentially equals LightLLM's TokenAttention.

When cache_block_seq_len >= 256:

The FTL increased a lot, that means the prefill phase is slower.
The PTL decreased a lot, that means the decode phase is faster.

This happen when you run out of cache blocks. FTL increase because new requests are pending for cache blocks. PTL decrease because the actual batch size become smaller.
Also, benchmark results for batch size 128 with only 1000 prompts can be unstable because of severe tail effect (when the response for the last few requests are long).

When cache_block_seq_len >= 256:

The FTL increased a lot, that means the prefill phase is slower.
The PTL decreased a lot, that means the decode phase is faster.

This happen when you run out of cache blocks. FTL increase because new requests are pending for cache blocks. PTL decrease because the actual batch size become smaller.
Also, benchmark results for batch size 128 with only 1000 prompts can be unstable because of severe tail effect (when the response for the last few requests are long).

Exactly, as the cache_block_seq_len increases, the actual number of block count decreases, which will result in longer waiting time for new requests.
And the latency of decoding decreases, because the increase of cache_block_seq_len results in a reduction in the number of memory accesses to kv cache because more tokens can be accessed at once. The actual batch size you mentioned becoming smaller also has an impact.
Is there anything wrong with my understanding?

Also, benchmark results for batch size 128 with only 1000 prompts can be unstable because of severe tail effect (when the response for the last few requests are long).

@lzhangzz Here is the test result for 5000 prompts. The RPS is much higher when cache_block_seq_len <= 16.

cache_block_seq_len
RPS
FTL (min)
FTL (max)
FTL (avg)
PTL (50%)
PTL (75%)
PTL (95%)
PTL (99%)

1
10.081
0.057
4.134
0.245
0.044
0.061
0.122
0.187

8
10.097
0.061
4.163
0.257
0.045
0.064
0.122
0.186

16
10.078
0.056
4.222
0.261
0.045
0.064
0.122
0.187

32
7.971
0.067
4.998
0.307
0.055
0.069
0.141
0.193

64
8.042
0.067
6.189
0.441
0.053
0.069
0.142
0.195

128
7.847
0.06
7.229
0.963
0.051
0.068
0.143
0.205

256
7.649
0.062
7.595
2.687
0.047
0.059
0.134
0.188

512
7.258
0.101
11.746,
6.098
0.039
0.046
0.113
0.155

1024
6.299
0.192
17.091
11.746
0.03
0.033
0.076
0.128

Also, benchmark results for batch size 128 with only 1000 prompts can be unstable because of severe tail effect (when the response for the last few requests are long).

@lzhangzz Here is the test result for 5000 prompts. The RPS is much higher when cache_block_seq_len <= 16.
cache_block_seq_len	RPS	FTL (min)	FTL (max)	FTL (avg)	PTL (50%)	PTL (75%)	PTL (95%)	PTL (99%)
1	10.081	0.057	4.134	0.245	0.044	0.061	0.122	0.187
8	10.097	0.061	4.163	0.257	0.045	0.064	0.122	0.186
16	10.078	0.056	4.222	0.261	0.045	0.064	0.122	0.187
32	7.971	0.067	4.998	0.307	0.055	0.069	0.141	0.193
64	8.042	0.067	6.189	0.441	0.053	0.069	0.142	0.195
128	7.847	0.06	7.229	0.963	0.051	0.068	0.143	0.205
256	7.649	0.062	7.595	2.687	0.047	0.059	0.134	0.188
512	7.258	0.101	11.746,	6.098	0.039	0.046	0.113	0.155
1024	6.299	0.192	17.091	11.746	0.03	0.033	0.076	0.128

Hi @lvhan028 @lzhangzz @grimoire
Maybe we could change the default value of cache_block_seq_len to 16 based on the benchmark results showing significant advantages for RPS, FTL, and PTL. Do you have any suggestions? Thanks.

@ispobock @zhyncs Thank you so much for such comprehensive experiments.
Let me check if it can pass the evaluation test for smaller cache_block_seq_len.

Here are my test results:

When cache_block_seq_len=16, the inference result is in chaos. It can be reproduced by `lmdeploy chat turbomind <turbomind_model_path>
When cache_block_seq_len=32, the model evaluation results shown in the below table seems fine. The test methods is presented in this guide.

I agree with exposing `cache_block_seq_len` in the API.  For other numbers, the results are almost all in chaos. We need to figure out why it happens.

=sm80, tile size 32
sm75,70, tile size 64

@lvhan028 Got it. I just try to change the tile size to 16, and the result for cache_block_seq_len = 16 is fine. I also tested the throughput and found the the RPS dropped from 10.078 to 7.734.

cache_block_seq_len
RPS
FTL (min)
FTL (max)
FTL (avg)
PTL (50%)
PTL (75%)
PTL (95%)
PTL (99%)

16
7.734
0.062
4.958
0.276
0.057
0.072
0.143
0.195

So using smaller cache_block_seq_len will not achieve performance gain.

Merged in PR #1218


Refactor taskflowbackend base test classes

These base classes are convoluted, contain a lot of repetition and don't work perfectly for all cases.
Tasks

[x] Move TaskflowAPI.cleanup() to tests
[x] Move duplicated setUp() and tearDown() to TaskflowTestMixin
[x] Rename API project/assignment creation helpers for clarity
[x] Refactor base classes and usage for relevant use cases
[x] Do #1718

Done, to be merged.


Setup of Reposense for org repo, for better tracking of code contributions.

Coverage increased (+0.3%) to 66.733% when pulling d4d3af80fdf7bd257f5ecd51555b1852e28b30a5 on m-aslam-mj2:reposense into 8ad1d76a2252f3e561ee1b4ba3eb460646900fc0 on CS2113-AY1819S1-T16-1:master.


fixed darkmode to lightmode

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!
Providing the issue number/url helps us identify the problem being solved and keep the issues tab clean and tidy

This is not a fix. This is a request to add login and signup pages to the site.
Summarize what this PR fixes:-
Fixes darkmode to lightmode https://github.com/zero-to-mastery/Keiko-Corp/issues/514
Checklist
(Add an x between the brackets to check the items)

[x] I have summarize what this PR fixes above
[x] I have provided the issue ID/URL above
[x] I have not added Bootstrap or jQuery and used correct styling guide.
[x] This PR fixes only the issue mentioned above (create separate PRs for separate issues)
[x] I was the first one to claim the issue or I created the issue - I created the issue

There is a conflict with your dark-mode and this is no longer how the light/dark mode is implemented. Closing this for now.


Release Google.Cloud.Dataplex.V1 version 2.6.0

Changes in this release:
Bug fixes

Remove unused annotation in results_table (commit 78ebeac)

Created release for Google.Cloud.Dataplex.V1-2.6.0


Does this model is capable for doing a object detection on a image

I want a one-shot object detector to detect an object using a template (like cv2.template matching), where I can give a template outside from the target image and the model find the template in the target image if it is present.
So does by modifying the internal functions can I achieve this, and if not then why?


project: ##[error]Guid should contain 32 digits with 4 dashes (xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx).

I put your project name, 'OS' in the project param, but it wants the GUID for projectId.
##[error]Guid should contain 32 digits with 4 dashes (xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx).

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

ID: b03d67d0-8738-72e6-dbe7-324c1784d4fc
Version Independent ID: bbb4d774-cd98-3ecb-7ccf-5f49e6f2b425
Content: Download Pipeline Artifact task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/download-pipeline-artifact.md
Product: devops
Technology: devops-cicd
GitHub Login: @willsmythe
Microsoft Alias: wismyth

@randyram-at-ms Thanks for keeping up on this page. I went ahead and updated the description to read "The project GUID from which to download the pipeline artifacts." I think this clarifies. @willsmythe feel free to close this issue if you agree with the fix.

In the docs, it says that the 'project' argument takes the project's guid. Where can I find my project's guid?

This is what I used to get my project's guid: https://docs.microsoft.com/en-us/rest/api/azure/devops/core/projects/list?view=azure-devops-rest-5.0.

Unfortunately all of the examples still reference a textual project name.
Also, it would be useful if there was something that explained how to get the pipeline id and project guids as they don't seem to be exposed through the UI

Sorry but this page is really not helpful.  The arguments in no way match the YAML snippet.  As noted by others, the "project" value is not the friendly name but a GUID (which by the way, making users dig for a GUID is really not intuitive).  Is it pipeline or pipelineId?  Is it Artifact or ArtifactName?  In Azure DevOps when I add this, it has a value called "definition" which appears to be pipeline(?).  Again, what does "Definition: 30" or "Pipeline: 12" even mean?
To download a pipeline artifact, you should need the name of the pipeline which generated the artifact (not a GUID), artifact name from the publish, build version (latest or specific), etc.  Please make this friendlier for users, not archaic and cryptic.

Agree with what all the above say, and to be clear, it's not just about this page, it's also about the ADO product.
This guid we seek -- there is absolutely nowhere in the ADO UI I can find one.  Also, the specific task argument is "project", not "pipeline" (btw, the example shows 'FabrikamFiber' as a possible value, which isn't a GUID).  The hierarchy at the top of my browser when looking at a build's results is like so:
 /  / Pipelines / Builds /  / #
See that project?  Not the project you're looking for, and no guid there.
I've been fighting with trying to find a way to download an pipeline artifact that the PREVIOUS build uploaded. Can't seem to get there.
So far, I haven't needed to use the REST API.  I need to for this?  If so, could you document it?  This place wold be handy for things like that.

Arrrg, the above comment had angle-brackets around those items that were in-between the "/ /" text above.  Removed to avoid html injection, apparently.  Should have been:
/ MY-ORG / Pipelines / Builds / MY-PIPELINE-NAME / #BUILD-DATE-AND-NUMBER

I tried using the SYSTEM_DEFINITIONID of the 'source' pipeline as written in this page but got this error.
I also did not understand where I should find that Guid... I'll try to copy anything that looks like a Guid from the source pipeline build variables...
Btw as it is this page looks wrong and misleading 😅. What's that 12?

I had it working, in case it helps someone else:
- task: DownloadPipelineArtifact@2   inputs: # artifact is not specified, will download all of them     source: 'specific'     project: '{project guid, 'project' as ADO root pj}' # GET https://dev.azure.com/{org}/_apis/projects?api-version=5.0      pipeline: 7693  #to get this, run the source build and check SYSTEM_DEFINITIONID variable      runVersion: 'latestFromBranch' # take the latest from      runBranch: 'refs/heads/master' # master

You can also get the pipelineId from the URL when viewing the pipeline.
Be careful: if you are viewing a specific run you will see the runId, not the pipelineId. View the pipeline list of runs and that will be the pipelineId.
In keeping with the random names approach it's called 'definitionId' in the URL.

@vsalvino - That is more helpful than not mentioning it at all (I'm looking at you MS). But it assumes the Project Id and Pipeline Id you want to use are the current ones. I have separate build and deployment pipelines so, the ProjectId I can use but the Pipeline Id I still have to take a different tack.
I use a REST API call to get the RunId and I am trying to change it to get the Pipeline Id too. Here's the RunId if anyone can use it (how you set the variables for useLatestInternal, buildBranch, and BaseVersionNumber are up to you):

powershell: |
Need to convert ${{parameters.useLatest}} from 'true' to a Powershell boolean.
$parametersUseLatest = [System.Convert]::ToBoolean( '$(useLatestInternal)' )
if ( $parametersUseLatest -eq $true) {
$runUrl = 'https://dev.azure.com/{organization}/{project}/_apis/build/builds?api-version=5.0&branchName=$(buildBranch)&$top=1&queryOrder=finishTimeDescending'
} else {
$runUrl = "https://dev.azure.com/{organization}/{project}/_apis/build/builds?api-version=5.0&branchName=$(buildBranch)&buildNumber=$(BaseVersionNumber)"
}
Write-Host "URL: $runUrl"
$runList = Invoke-RestMethod -Uri $runUrl -Headers @{ Authorization = "Bearer $(System.AccessToken)" }
if ($runList.Count -ne 1) {
Write-Host "runList is not one. There's something wrong."
Write-Host "$runList"
Write-Host "##vso[task.logissue type=error]Version $(BaseVersionNumber) has $($runList.Count) runs. The runCount must be 1."
Write-Host "##vso[task.complete result=Failed]"
}
$runId = $runList.value.Id
Write-Host "RunId = $runId"
This is the way to set variables to be used in the pipeline
Write-Host "##vso[task.setvariable variable=runId]$runId"
displayName: Getting runId

@henry-padilla - that looks useful, but I think if you want to limit this based on a specific Git repository, you'd need to specify repositoryId which is a Guid that requires looking up (in the ADO UI).
This lookup would really need to be done dynamically in order to be useful in a YAML build/release pipeline, and I currently cannot see an easy way to do this.
MS needs to sort this out and allow the friendly and the unique names of repos, pipelines, runs etc. for a lot of these YAML tasks that under the hood rely on Guids.
More info on the Build API and its parameters can be found here:
https://docs.microsoft.com/en-us/rest/api/azure/devops/build/builds/list?view=azure-devops-rest-5.1

OK, looks like $(Build.Repository.ID) can be used...! Which works for the current repo.

Maybe a good idead to change the examples as well on this...


Show display names in search form

Display_names are displayed in the search field, if they are specified, otherwise field names are displayed.
Before:

After:

Fork deleted, so unable to merge this.  Reimplemented in #4284.


Feature: add Node information into AppendEntriesRequest request

Add another field leader_node: Node to AppendEntriesRequest.
This way the follower or learner can perceive the node information even when the
leader is not in the membership config.
https://github.com/datafuselabs/openraft/blob/732304ba080dd22e778749d35e71914bc35ea048/openraft/src/raft/message/append_entries.rs#L14-L27
Why
Openraft allows a leader to be not in the membership.
When a leader commits a new membership config that does not contain itself,
the leader will continue to run for a while. During this period, the leader is
not in the cluster(not a voter or learner).
When a Follower receives write-log request, it just returns an error indicating
the caller to forward the request to the leader. But the follower can not find
out Node information in its local memberhship config.
Thus we must embed the leader Node into the AppendEntriesRequest so that
when a heartbeat or append entry request is received by the follower, The
follower will know the leader's information.

Close it.
Transmitting leader node information is application level requirement, and it should not be in the core of a raft implementation.


Replace ShardSchema & ShardTable expression

Micro optimization: IndexParts is now only created once instead of
twice, and this get's rid of two classes.

Codecov Report

Merging #6252 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #6252      +/-   ##
============================================
- Coverage     83.29%   83.26%   -0.03%
+ Complexity    12572    12561      -11
============================================
  Files          1454     1452       -2
  Lines         47540    47533       -7
  Branches       5827     5827
============================================
- Hits          39598    39578      -20
- Misses         5233     5242       +9
- Partials       2709     2713       +4

Impacted Files
Coverage Δ
Complexity Δ

...o/crate/metadata/shard/ShardReferenceResolver.java
88.23% <100%> (+0.73%)
6 <0> (ø)
:arrow_down:

...arch/indices/recovery/BlobStartPrefixResponse.java
66.66% <0%> (-33.34%)
3% <0%> (-2%)

...a/io/crate/executor/transport/ActionListeners.java
33.33% <0%> (-22.23%)
2% <0%> (-1%)

...ticsearch/indices/recovery/BlobRecoveryStatus.java
63.63% <0%> (-9.1%)
3% <0%> (-1%)

...ate/operation/merge/RamAccountingPageIterator.java
94.44% <0%> (-5.56%)
8% <0%> (-1%)

...va/io/crate/data/join/NestedLoopBatchIterator.java
95.45% <0%> (-4.55%)
21% <0%> (-1%)

.../projectors/BatchIteratorBackpressureExecutor.java
92.53% <0%> (-4.48%)
17% <0%> (-1%)

...io/crate/protocols/postgres/types/VarCharType.java
91.66% <0%> (-4.17%)
10% <0%> (ø)

...otocols/postgres/RetryOnFailureResultReceiver.java
88.46% <0%> (-3.85%)
10% <0%> (-1%)

...te/blob/pending_transfer/PutHeadChunkRunnable.java
76.71% <0%> (-2.74%)
12% <0%> (-1%)

... and 14 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 b1837ba...d0dcff2. Read the comment docs.


Failed to open users list file! (rfid_users_list.db)

Hello
Thanks for your great esphome-door-look.
Everything is working fine, except that error of opening users list file and I receive the below error.
Where should I put this file.
[0;35m[C][switch.gpio:011]: Setting up GPIO Switch 'Door lock'...
[D][switch:017]: 'Door lock' Turning OFF.
[D][switch:037]: 'Door lock': Sending state OFF
[D][switch:017]: 'Door lock' Turning OFF.
[D][rfid:062]: Loading users list from file...
[E][rfid:066]: Failed to open users list file!
Thanks
Ashraf

Did you send the users list with Home Assistant service Upload Users?

Yes, and I got error
[0;36m[D][rfid:100]: Saving users list to file...
[1;31m[E][rfid:104]: Failed to open users list file!

Sure, I use this structure. I call the service from HA Service "ESPHome: door_lock_upload_users_list"
I am using HA ESPhome to compile and install the code on ESP8266 MCU.
I uploaded the file "rfid_users_list.db" to HA under "/config/esphome" folder, is this the correct location for this file?
Also I tried to upload it directly to esp8266 using arduino IDE DATA Upload tool but no clue.
Thanks

Sorry, I was missing that the SPIFFS.open("/rfid_users_list.db", "w") function will create the file when calling the service "ESPHome: door_lock_upload_users_list".
So, May  there is a problem with esp8266 mcu itself.
Thanks

It is Working now. I used ESP32 MCU and changed the file system from SPIFFS to LittleFS.
Thanks
Ashraf


build: undo some protobuf hacks, put some new ones in.

Use protobuf from HEAD to ensure we have the
https://github.com/google/protobuf/pull/3327 fix for CI and local
builds. Users can opt to use a specific release tag with the
ENVOY_PROTOBUF_COMMIT env var.

Remove the workaround for https://github.com/google/protobuf/pull/3327
in do_ci.sh.

Remove the multiple protobuf versions that existed because
https://github.com/google/protobuf/pull/2508 wasn't merged.

Add some evil symlink stuff to get @protobuf_bzl inferred from
wherever WORKSPACE points the envoy_dependencies path at.

As a bonus, enabled more verbose build of external deps so we don't sit
around for minutes on initial checkout with no activity indicator. This
can be done safely now as everyone should be at Bazel 0.5.2.

@mattklein123 @rshriram @khgandhi

We probably need to split this into 2 commits?

Yeah, I thought we could try and sneak past that for a few commits, but it's probably best. We actually need 3 commits:

Switch protobuf version, rebuild Docker image.
Update build system etc.
Remove dual build of thirdparty in Docker image.

Closing out to split the commits.


Margin trades examples

Hello,
I was going through manual, as well as several old posted issues, but was not able to find a relevant example of margin trading. For instance, Kraken and Bitfinex. I'm using Python, btw.
This issue (https://github.com/ccxt-dev/ccxt/issues/204) has an example for Kraken posted by you:

import ccxt
kraken = ccxt.kraken({'apiKey': 'YOURKEY', 'secret': 'YOURSECRET'})
order = kraken.create_market_sell_order('BTC/USD', 1, {'leverage': 2})
print(order) # should have an id
It's a bid off, it should be {'leverage-sell': 2}, but anyways either way it doesn't work.

Here is my code for Kraken:

order = exch.create_order('BTC/USD', 'market', 'sell', 0.01, {'leverage_sell': 3})

It actually created an order and did a trade. But! It was a regular market order, not margin: https://cl.ly/2C46331U0X1s
Furthermore, this is the code I have for bitfinex (according to their API documentation):

order = exch.create_order('BTC/USD', 'market', 'buy', 0.01, {'type': 'margin'})
And here is the error:
File "/usr/local/lib/python3.6/site-packages/ccxt/exchanges.py", line 2585, in create_order
raise NotSupported(self.id + ' createOrder not implemented yet')
ccxt.errors.NotSupported: bitfinex2 createOrder not implemented yet
"not implemented yet"? o_O

So I guess, what I'm trying to ask is, could you please update the documentation with few more relevant examples of margin trading? To highlight few usecases of using {params}. Thank you,

Hi, @Fcl69!
For Kraken:
import ccxt

kraken = ccxt.kraken ({
    # 'apiKey': 'YOUR_API_KEY',
    # 'secret': 'YOUR_SECRET',
})

kraken.load_markets ()

for symbol in kraken.symbols:
    print(symbol,
        'Available leverage levels',
        'Buy:', kraken.markets[symbol]['info']['leverage_buy'],
        'Sell:', kraken.markets[symbol]['info']['leverage_sell'])

# it should be "leverage", not "leverage_sell" nor "leverage-sell"
kraken.create_order('BTC/USD', 'market', 'sell', 0.01, {'leverage': 3})

For bitfinex2 – this is their experimental Bitfinex API v2 (it's raw and they don't guarantee that it works), and v2 does not offer an endpoint for creating orders at all. This is not because we can't implement it, it's just because their v2 does not offer that yet. So, for trading with Bitfinex, use v1 (the stable version of it).
Does this answer your question?

I edited the above example for Kraken a little, let me know if you still have questions with it.

It did work with Kraken, thank you!
And thank you for explaining about bitfinex2. H/e could you please send an example for bitfinex margin order as well? I'm a bit confused at this stage... I'd like to go with:
order = exch.create_market_buy_order(symbol, trade_amount, {'type': 'margin'})
But https://docs.bitfinex.com/v1/reference#rest-auth-new-order says:

Either “market” / “limit” / “stop” / “trailing-stop” / “fill-or-kill” / “exchange market” / “exchange
limit” / “exchange stop” / “exchange trailing-stop” / “exchange fill-or-kill”. (type starting by
“exchange ” are exchange orders, others are margin trading orders)
"others are margin trading orders". So how should I deal with a default "market" to makge it margin?

By default, Bitfinex v1 will create an order of type exchange market (if you are calling create_market_buy_order()). You will have to redefine the type for market`, as you wrote it. So, you can go ahead with this line (it should work with v1):
exch.create_market_buy_order(symbol, trade_amount, {'type': 'market'})

it worked! thank you so much for the prompt replies!

Thanks for reporting back! I'm closing this issue if you don't mind. Feel free to ask your questions, if you have any left.

Hello.
How should I redefint the type for market in order to use the margin trade at bitfinex via API?

Hi, @udvarid !

To create a market order of default type (which is exchange market):

A. exch.create_market_buy_order(symbol, trade_amount) ←↓ this is an exchange order
B. exch.create_market_buy_order(symbol, trade_amount, {'type': 'exchange market'})
Options A and B are equivalent for exchange orders.

To create a market margin order of type market:
exch.create_market_buy_order(symbol, trade_amount, {'type': 'market'}) ← this is a margin order

Thank you!
And any idea how to get my open margin positions with here? The fetch_balance() is only give my balances (exchange and margin) but it would be nice if I could see the open margin positions as well.
Thank you in advance!

@udvarid sure, see here: https://github.com/ccxt/ccxt/issues/318. Hope it helps (it should).

thank you very much!
On 2017. Nov 14., at 22:27, Igor Kroitor <notifications<EMAIL_ADDRESS>wrote:
@udvaridhttps://github.com/udvarid sure, see here: #318https://github.com/ccxt/ccxt/issues/318. Hope it helps (it should).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ccxt/ccxt/issues/255#issuecomment-344404511, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AgGod9YVYJL8GiyUIpuiYac3GHTXqDvrks5s2gXcgaJpZM4PpzjO.

Hello! How can I get the trade balance? I'm using private_post_trade_balance() but I get this error:
raise ExchangeError(self.id + ' ' + self.json(response))
ccxt.base.errors.ExchangeError: kraken {"error":["EGeneral:Internal error"]}

@fedecaccia, please, post a minimal, but a complete snippet of code that I could copypaste into my python to reproduce it (without the keys), without seeing the code it's hard to tell.

Hi, thanks for your responde. While I was writng the code, I recognize that I was using type="maker" instead of type="market". Thanks!

@fedecaccia thx for reporting back!

@kroitor how can I get my bitfinex margin position information?
I try to do it by exch.fetch_my_trades(symbol=symbol,params=params)
params = {'type': 'trading'}
but it result is all my history order information ,is not my real margin position information.

@ChacoLv https://github.com/ccxt/ccxt/search?q=bitfinex+position&type=Issues

@kroitor OK,Get it ,Thank you


Get it working with 0.12.0-dev / latest nightly Zig

This PR gets ZigAndroidTemplate working again with the latest Zig version.
This requires > current nightly Zig, and will not work with Zig 0.11, as std.c actually needs to be patched to support Android bionic libc: https://github.com/ziglang/zig/pull/17176
Context: I did this work as I am using this as a reference for Mach's own Android support. At this point, I will probably diverge from this repository and may not update this PR. So if anything here is not useful to you feel free to discard it - this achieved what I wanted to achieve, so I am just sharing the results with you. Feel free to integrate (or not integrate) any parts of this, I don't mind what you do with it.
I was able to confirm on an Apple Silicon host machine (any host OS should work) that:
zig build -Dexample=egl install push && sleep 1 && zig build run

works as expected, displays a Zig logo and when volume is turned all the way up, tapping the screen plays a quiet 'beeeep' sound with AAudio.
see individual commit messages for details, I explained the changes in there.
Helps https://github.com/hexops/mach/issues/17

At this point, I will probably diverge from this repository and may not update this PR.

Can you explain that? I really don't like to see that we have two implementations of the Android stuff, as it means we both have to make either the same mistake twice or constantly monitor the other repository to backport things. What exactly would you change, and why? And can we make those changes as optional features of this repository?

Not sure if i can merge it like that, especially as you unnecessarily changed several versions of SDK/NDK/Android version

No worries, as I mentioned this was less of a 'please merge this' but rather more of a 'here's what I did, just sharing the results, hope any of this is useful to you' PR. Not asking for this to be merged as-is :) I'll close the PR, I just wanted you to be aware the code is here if you want to cherry-pick anything from it.
Changing the NDK version was for good reason, see the commit message for details.

Can you explain that? I really don't like to see that we have two implementations of the Android stuff

Yeah, I get that.. happy to explain:

I'm comfortable requiring AAudio and Vulkan, and dropping older APIs like OpenSL ES, GLES, etc. entirely
Mach is already on Zig 0.12.0-dev, and will continue following (pinned, specific) nightly Zig versions. I understand you intend to stick with older Zig 0.11 (fair!) but I can't/won't do that so it introduces a divergence between our projects.
I will possibly use native_app_glue instead of the main ZigAndroidTemplate provides, not sure yet though.
ZigAndroidTemplate feels a bit like a kitchen sink with Sdk.zig, AAudio, OpenSL, JNI, and various other Android APIs wrapped all in the same project. I think this is by design, but it doesn't mesh well with Mach which tries to solve each of these as independent projects:

sysaudio would have an AAudio backend
I feel uneasy about Sdk.zig and how many levers it necessarily has to pull here, if Zig does in fact lose custom build steps the approach taken in this project would be very very hard to adapt to that I suspect. There are other ways to structure this to avoid this potential outcome, application packaging in general is a more broad topic than just Android, but I suspect you wouldn't be happy with the way I might approach this.

Like you, I also don't have a ton of time. Hopefully one day, but for today my bandwidth is very very limited so I have to be very careful where I spend my time to have productive outcomes.

I am grateful for ZigAndroidTemplate/you pioneering the way here, and giving me something I can reference/look at. In short I think that Mach has different constraints/goals than you w.r.t. Android support and that's not straightforward to unify with little time.

Thanks for the clarification!


Change error exit codes to help identify docker run errors from conta…

…iner errors
Signed-off-by: Sally O'Malley<EMAIL_ADDRESS>

I think this will eventually satisfy  https://bugzilla.redhat.com/show_bug.cgi?id=1097344 and https://bugzilla.redhat.com/show_bug.cgi?id=1097334

Please submit upstream.
Cut and paste Regan's name to your pull request also. So you both get credit.

will do.

(Here are the test conditions I ran)
#/bin/bash
Test changes to api/client/run.go error codes
echo 'docker run contained process exits 1, expect 1:'
docker run fedora /bin/false; echo $?
echo
RESULT:    1
echo 'docker run non-existing image, expect 126:'
docker run foo; echo $?
echo
RESULT:    Unable to find image 'foo:latest' locally
                   Pulling repository foo
                   126

echo 'docker run image ok, cmd does not exist, expect 126:'
docker run fedora foo; echo $?
echo
RESULT:    exec: "foo": executable file not found in $PATH
                  126

echo 'docker run contained cmd could not execute, expect 127:'
docker run fedora /bin/foo; echo $?
echo
RESULT:    exec: "/bin/foo": stat /bin/foo: no such file or directory
                  127

echo 'docker run no errs, contained no errs, expect 0:'
docker run fedora /bin/true; echo $?
echo
RESULT:    0

I've replaced this pr with https://github.com/rhatdan/docker/pull/77


React tutorial follows interface naming lint rules that require 'I' prefix

The lint rules expect I to prefix interfaces.

Hi @flatiron32  - I think overall we're fine without these changes, it'd have to be consistent all over the docs and (personally) I don't think it adds much in a structural type system where almost everything is an interface.  So I'm going to close this PR, but thanks for helping to improve out docs.


ci: Added sse2 scrypt option to Win64 experimental build

bench_dogecoin on Windows 10 (Intel i7 w/AVX2 SHAs)
c:\dogecoin>avx2\bin\bench_dogecoin.exe
#Benchmark,count,min,max,average,min_cycles,max_cycles,average_cycles
...
SHA1,768,0.001311466097832,0.001372124999762,0.001338852879902,2766443,2906564,2828688
SHA256,384,0.002561874687672,0.002745762467384,0.002618161961436,5426619,5756365,5530148
SHA256_32b,6,0.189843535423279,0.195170998573303,0.192108511924744,401013412,412416121,405702898
SHA512,640,0.001647468656301,0.001767069101334,0.001697367057204,3499345,3715532,3583560
Scrypt,4608,0.000224933493882,0.000230658333749,0.000226830753187,473877,485739,479228
SipHash_32b,48,0.020995736122131,0.022479534149170,0.021513854463895,44609869,48099245,45437085
Sleep100ms,10,0.100115895271301,0.100720047950745,0.100514984130859,211440027,212718901,212244756
Trig,27262976,0.000000035440735,0.000000040950454,0.000000038092909,76,84,80

bench_dogecoin on Windows 10 (Intel i7 w/AVX2 SHAs and SSE2 Scrypt)
c:\dogecoin>avx2_sse_scrypt\bin\bench_dogecoin.exe
#Benchmark,count,min,max,average,min_cycles,max_cycles,average_cycles
...
SHA1,768,0.001281008124352,0.001342564821243,0.001314044309159,2742723,2839084,2774571
SHA256,384,0.002562001347542,0.002837650477886,0.002622580155730,5418986,5976710,5536936
SHA256_32b,6,0.189343571662903,0.206030011177063,0.200109998385111,399318005,435111878,422554670
SHA512,640,0.001646999269724,0.001733690500259,0.001676921918988,3494553,3679962,3544248
Scrypt,5632,0.000175375025719,0.000182999297976,0.000178022013808,371993,387787,375935
SipHash_32b,48,0.020983994007111,0.024076461791992,0.021306956807772,44129880,50467834,44999032
Sleep100ms,10,0.100460052490234,0.100820541381836,0.100678014755249,212059922,212909643,212461765
Trig,27262976,0.000000035372068,0.000000041885414,0.000000038290505,74,89,80

c:\dogecoin>

@edtubbs same with this one, just make sure to unmark WIP so i can see this one forward because i <3 the SIMD and intrinsics work.

@edtubbs same with this one, just make sure to unmark WIP so i can see this one forward because i <3 the SIMD and intrinsics work.

@michilumin ok cool!  this is ready to review, and merge after https://github.com/dogecoin/dogecoin/pull/3071 is merged


Implement @smeragoel's data table designs

Preview URL Pandas example

Cross reference: https://github.com/Quansight-Labs/czi-scientific-python-mgmt/issues/50

The preview build has been checked by @smeragoel. This PR is now ready for review.

Had a quick nosey and I have a couple questions

Non-data tables will be styled in a separate PR correct?
Since the tables in widgets seem to inherit/use the same style was it intentional that the div/tabbed component in which these are embedded remain light grey in dark mode? Or is this an unintended thing?

Replies to above questions:

I think that's a good plan to keep this PR moving along.
I think you're referring to how when the Pandas data table is put inside the example tabbed ipywidget. We get a set of layers that goes (from back to front) black > gray > white > black, as shown in the following screenshot:
.

I admit it doesn't look great but it is intentional. I applied the cell-output-background mixin to the ipywidget container (.widget-subarea) for the same reason it is applied to other notebook outputs—namely, neither those outputs nor ipywidgets have been styled for dark themes.
If you don't apply the cell-output-background mixin with the light gray background, then you're not going to see the black text "Hello" because ipywidgets isn't styled for dark mode—it assumes a light background.
There's really no way around this. If we want to follow the existing dark/light mode conventions in the stylesheets and we want to apply these new styles to Pandas data tables that appear inside of ipywidgets, then we are going to end up with this dark-light-dark layering in dark mode. There are only three ways I see around it:

Don't apply the data tables mixin to ipywidgets, which would mean our new styles would not get applied to Pandas data tables inside of ipywidgets
Write two versions of the mixin, one light, one dark, rather than relying on CSS vars that vary on mode. This is possible I think but it goes against the pattern of how dark mode is implemented in other places in our stylesheet code.
Try to add our own styles to ipywidgets so they look good in dark mode. My guess is we could write a set of rules that work in most but not all cases. This would be a band-aid solution until ipywidgets supports dark mode. If ipywidgets supported dark mode, we wouldn't have this dark-light-dark layering issue at all.

Failed checks unrelated, see #1772

Not sure about the failing codecov/project check...

I think you're referring to when the Pandas data table is put inside the example tabbed ipywidget.

Yes, that is what I was referring to. While this does not look as nice or polished as the other tables, reading the alternatives for aligning the style makes them seem more hacky/problematic than the current style mismatch, so we can leave them as they are for now (they are already much better than they were before, anyway).

Try to add our own styles to ipywidgets so they look good in dark mode. My guess is we could write a set of rules that work in most but not all cases. This would be a band-aid solution https://github.com/jupyter-widgets/ipywidgets/issues/3675. If ipywidgets supported dark mode, we wouldn't have this dark-light-dark layering issue at all.

This would be perhaps the best solution. The linked issue does not have any follow-ups or responses, so perhaps we should discuss it internally and see if we can offer some time/capacity to make this move forward WDYT @gabalafou?


Added scaffold-eth-2-noir

This adds scaffold-eth-2-noir to the list.
Unfortunately it still has a pending issue on ecdsa signature verification. But I'm keeping my eyes on it and will update once it's ready.

Thanks for submitting the PR!


Differentiate created templates on vCenter

Author Name: Adrian Menendez (Adrian Menendez)
Original Redmine Issue: 5053, https://dev.opennebula.org/issues/5053
Original Date: 2017-03-09

When create a template making a vm persistent, add an attribute to know if has alredy been created on vCenter (vm has been deleted) or is still only in OpenNebula. Another option is to add an attribute while is waiting and delete it when it's created, so all of the created templates have the same attributes and the only different ones are those who has been created making a vm as persistent.

Original Redmine Comment
Author Name: Jaime Melis (@jmelis)
Original Date: 2017-06-08T09:42:05Z

Can you please elaborate? We are having difficulties understanding this issue.

Original Redmine Comment
Author Name: Adrian Menendez (Adrian Menendez)
Original Date: 2017-06-09T10:21:46Z

When a vm is deployed as persistent, OpenNebula creates a template too. Meanwhile the vm is up and running, the template is only on OpenNebula, just a registry, not a real vCenter template. It would be interesting to be able to differentiate which of them really exists on vCenter.

Check if VM Template in OpenNebula is locked until the vCenter VM is converted to VM Template. If not, lock it.
With this functionality, locked VM Templates are the ones not present in vCenter.

[ONE]

https://github.com/OpenNebula/one/pull/4394

[DOC]

https://github.com/OpenNebula/docs/pull/1064


Faster upload speed for ESP8266

Change the upload baud rate to 921600 (as CH340G can handle it) and save quite a lot of time actually
Before
Wrote 389152 bytes (280698 compressed) at 0x00000000 in 24.8 seconds (effective 125.7 kbit/s)...
After
Wrote 389152 bytes (280698 compressed) at 0x00000000 in 4.1 seconds (effective 760.6 kbit/s)...

Yes, that works fine with CH340, CP210x and most USB bridges.
But... what I'm worried about is that some Windows drivers (Linux is fine) for the FT2232H and CP2102 may not support high-speed communication.
(This comment is not an objection because everything worked for me!)

CP2102 is a part of CP210x family. Can confirm this works well with CP2102 on both NodeMCU ESP32 and Wemos D1 Mini (modification with CP2102). FTDI chips should also work well, but don't have any to test.

CP2102 is a part of CP210x family. Can confirm this works well with CP2102 on both NodeMCU ESP32 and Wemos D1 Mini (modification with CP2102). FTDI chips should also work well, but don't have any to test.

I'm always uploading at 921600 with cp2102 ch340, so I agree to increase the baudrate if there are no further issues.

But... what I'm worried about is that some Windows drivers (Linux is fine) for the FT2232H and CP2102 may not support high-speed communication. (This comment is not an objection because everything worked for me!)

The CP2102 is a beast as it can do 3000000 (3mil) baud no problem (I tried it once on my only esp32 with cp2102)

should probably set it for esp32 too

the esp01 part is broken

the esp01 part is broken

Yup I saw that


12 ydbqueryrunner.loadtables

Ydb.QueryRunnerloadTables and dependences

Let's use commit style of typeorm look here
And there is too much changes for one PR, let's limit PR size by 200 lines


Publish Chrome extension

Make the chrome extension publicly available. Right now you have to load it as an unpacked extension to use it. Publishing should happen automatically via GitHub actions.

GitHub OAuth seems to be broken in the Chrome extension iframe. Need to investigate that before publishing


Links in E2EE chats are leaked plaintext by URL preview API.

An example request a client (Riot) makes to the server (synapse) when receiving a link in an E2EE chat:
GET /_matrix/media/r0/preview_url?url=https://google.com

That's why clients like riot-web disable URL Previews by default for E2EE rooms (https://github.com/vector-im/riot-web/issues/2084)
A solution to this is possible, client can generate their own url previews, at the cost of their privacy from the target server mentioned in the url: https://github.com/vector-im/riot-web/issues/2953

In practice I believe the spec just needs to recommend that clients not URL preview things in encrypted rooms, or that they do their own previews. Ultimately it's not a spec problem that the clients decide to leak this information, but the spec can recommend them to not do that.


add html and css

My Portfolio
Using Html and CCS to build the first three section

Alright thank you . Will amend the changes
On Wed, 24 Nov 2021, 7:37 pm ELYOR, @.***> wrote:

@.**** requested changes on this pull request.
Hi @Ginohmk https://github.com/Ginohmk ,
Good job so far!
To highlight 👏

The container of the Toolbar section is a header tag
The hamburger button is on the right and has a nav tag
The contents are aligned with flexbox
The sections have the margins (top, bottom, left, right) defined in
the design
The sections use the font and colors (background and font) defined
for the template

There are some issues that you still need to work on to go to the next
project but you are almost there!
[image: almost-there-mario]
https://user-images.githubusercontent.com/67087664/143291509-b952693c-73e7-4467-b774-9b5b8feddf54.gif
Required Changes ♻

Please make sure you stick to the design as much as possible (e.g.
background-images, margins) using the templates in Figma
https://www.figma.com/file/l7SqJ3ZfkAKih9sFxvWSR4/Microverse-Student-Project-1?node-id=0%3A1
see 👇
Original template
[image: original]
https://user-images.githubusercontent.com/67087664/143295098-e2f3c6ae-f0a4-4d79-95c5-fca8f0fb43ef.png
Your version
[image: portfolio-mobile]
https://user-images.githubusercontent.com/67087664/143295176-ed6db6c2-207e-4aef-89f5-4ece34008046.png

Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In index.html
https://github.com/Ginohmk/My-portfolio/pull/1#discussion_r756322877:

@@ -0,0 +1,52 @@

+
+

Kanu Mike C

My Logo

Make sure the logo has a link  tag

In index.html
https://github.com/Ginohmk/My-portfolio/pull/1#discussion_r756328695:

My Logo

Please kindly use the appropriate tags for each element. For
example, by using  instead of  tag for the container you
follow HTML, CSS best practices
https://github.com/microverseinc/curriculum-html-css/blob/main/articles/html_css_best_practices.md
.

In index.html
https://github.com/Ginohmk/My-portfolio/pull/1#discussion_r756334446:

It would be better if you use  tags instead of  for each
icon inside s, so one can click to reach your social media profiles

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Ginohmk/My-portfolio/pull/1#pullrequestreview-815245158,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOAMQM6FP55W4ZOUVAI2FLDUNUWEZANCNFSM5IWQ7OUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


adding composer autoload support

Ref #52
Polyfills will be always loaded and TestRunner when needed with this change.
Parser bootstrap was also for autoloading only and should also be fine

@NielsLeenheer it fails here of course, because it's not in the seperate repo.
https://travis-ci.org/WhichBrowser/WhichBrowser/jobs/94584147
I will wait for your answer on #52


Store not updating - SwiftUI iOS 14 TabView

Describe the bug
For some reason store updates are not propagating if a view is inside a TabView
To Reproduce
// Works
WindowGroup {
	ResultsView(store: store)
}

// Computer says no
WindowGroup {
	TabView {
		ResultsView(store: store)
		ContentView()
	} .tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))
}

Expected behavior
Store updates should work
Environment

Xcode: Version 12.0 beta 2 (12A6163b)
Swift: 5.3
OS: iOS 14

Additional context
I have tried passing store and using WithViewStore and also having keeping a reference to view store via @ObservedObject

@mbrandonw @stephencelis
Interesting... Adding .id(UUID()) makes it work
@main
struct MyApp: App {
	let store: Store<AppState, AppAction>
	@ObservedObject var viewStore: ViewStore<AppState, AppAction>
	init() {
		self.store = Store(
			initialState: AppState(),
			reducer: appReducer,
			environment: AppEnvironment.live
		)
		self.viewStore = ViewStore(store)
	}
	var body: some Scene {
		WindowGroup {
			TabView {
				ResultsView(viewStore: viewStore)
				ContentView()
			}
			.id(UUID())
			.tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))
			.onAppear {
				viewStore.send(.didFinishLaunching)
			}
		}
	}
}

Hey @asos-danieltavares, thanks for the report!
Is there a reason you are passing the viewStore to ResultsView instead of just the store? I think typically you do not want to do this because you want each view to set up its own observation of the state (via WithViewStore or @ObservedObject). Perhaps we should even include some documentation on this.
Beyond that, are you able to reproduce this bug in a full code snippet that we can run locally? I had trouble reconstructing the problem.

@mbrandonw I've rejjigged come code around and it started working. Must of been my setup. I think a documentation around the viewStore/store will be great. I have been using it for a while and it's the one thing that I usually miss :). Will keep you informed if I see any more bugs (or my dodgy programming). Cheers

I've managed to replicate it 100% now. I've attached the project.
Some notes:
The culprit seems to be .tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))
As soon as you add that, everything breaks.
// Keeping an id for the viewstore seems the only way to update things and get things working correctly
.id(viewStore.id)
.tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))

// This combo works, but any further updates to the store in later tabs will re-draw everything.
.id(UUID())
.tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))

// Adding the below breaks it
.tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))

TabViewBug.zip

Hey @asos-danieltavares thanks for reopening with a repro! We'll try to take a look at this soon.

Looks like this is another SwiftUI bug unrelated to TCA. A vanilla repro: https://gist.github.com/stephencelis/ee80fec9be7c7626012d3d005c8f712b
Wrapping your NumberView in a VStack seems to fix things:
TabView {
  VStack { NumbersView(store: store) }
  Text("View 2")
  Text("View 3")
}

Gonna close this out since it's another thing out of our control. Hopefully some of these bugs get fixed before the beta cycle finishes!


Dark mode does not apply to simple editor

I am running on Android (14), and can reproduce this in both chrome and Firefox for android.
The background/UI is dark, but the editor textbox is light when set to simple.

That did it, thank you :)


Inconsistent Formatting for while condition

Description:
Consider the following capture,
https://user-images.githubusercontent.com/1329674/150897253-6403002d-124e-41bc-a853-36ee5b0aff3f.mp4
The let expression of the while loop is broken down to two lines even though it should be in the same line
Steps to reproduce:
Affected Versions:
SwanLake-beta6 at least

Closing since it is a duplicate of https://github.com/ballerina-platform/ballerina-lang/issues/31950


Table Header & Table body content not properly Viewed

I'm using react 15.5.4 and bootstrap 4 alpha 6

Cleared By adding react bootstrap css manually


[Fix](inverted index) fix use after free when duplicate key in index dir when index file writer open index #42207

cherry pick from #42207

run buildall

run buildall


update deps

Signed-off-by: Sherin Varughese<EMAIL_ADDRESS>For https://github.com/open-cluster-management/backlog/issues/15469

/retest

/retest

Merging as all tests except e2e passed.


Limit praise to the chapel

I'm pretty sure this should work, just need to add chapel_channel to the config file.

#11

#11


Documentation empty in certain places?

e.g. https://tomoki-yamashita.github.io/CrySPY_doc/tutorial/bo/

I haven't finished writing the documents yet. I'll write when I have time.

Ok, thanks!

@Tomoki-YAMASHITA closed as completed?


allow tensor inputs to not equals

follows the same logic as ge to accept tensor inputs

I think you need to update the version as well


Add Source: National Geographic's Photo of day

CLI option to select source while setting the wallpaper.
Add National Geographic's Photo of the day :framed_picture: source.

Hi @guptarohit I would like to work on this.

I have been working on this issue for a couple of hours. So far I have made a command line selector that reads from a file "sources.txt" but have not yet added support for other sources.

@erhan sure go ahead  :+1:

@erhan are you still working on this?

Yes, @houfu created a nice cli interface with click library (#6) :sparkles:


Bug: Occurrences Highlight is broken when set to off

What are these gray highlights/boxes that appear around my code?
It happens randomly, and wont go away until I type something or close VSCode.
Notice my cursor is not selecting anything, yet the "65A522" is randomly highlighted.

Update: I've managed to consistently replicate this behavior:

Ctrl + F to open search.
Search for something in the file.
Hit ESC to close the search.
The highlights linger on whatever I've searched for.

I'll add that this still randomly happens when I'm not actively searching for anything -- I just happened to notice that the search consistently triggers this behavior.
I don't know if this is related to some newer VSCode setting I'm unfamiliar with, as I just updated my VSCode. I had been running on a much older version prior to this. I'll add that I disabled all extensions, and it still happens.

I asked a colleague to test the same, and they report seeing the same bug when setting editor.occurrencesHighlight to off -- this is also using version 1.87.0.

Closing this issue, as its been identified and resolved in 1.87.1 -- https://github.com/microsoft/vscode/pull/206557


Extension causes high cpu load

Issue Type: Performance
Extension Name: code-spell-checker
Extension Version: 1.7.9
OS Version: Linux x64 4.15.0-38-generic
VSCode version: 1.33.1

streetsidesoftware.code-spell-checker-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Thank for including the attached file.
I need more information:

What kind (text, cpp, python, etc) of files are you editing?
Are there lots of files in your project?
Do you have a workspace plus folders?
How large are the files?

Thank you for attaching the file. That makes a big difference.
The strange issue is that it seems to be stuck in the VS Code extension loader. But the extension is NOT loaded. It is acting like VS Code is not able to find the extension.
What happens if you restart VS Code and re-install the extension?

I am hoping this is fixed with 1.7.10. I have bundled the extension as described by
Explain: extension causes high cpu load · code-loading


when will you  release  code ?

Hello, I'm interested in your research.
when will you  release  code ?

@han1222 Hi, we just re-submitted the paper, and we plan to release the code before mid-April. I will let you know once the code is ready, thx :)

谢谢作者

Amazing work!! When is the code going to be released?

@han1222 @12num @Cram3r95
Hi, the code of this project is released. If you have any questions, please feel free to post issues.


Replace "Arena" with "MemArena"

This replaces Arena with MemArena, the latter is the libswoc replacement for the former.

@cmcfarlen is going do take over this PR

Closed.  See #10979


CLI 4.21.4 generates queries for keys when queries:null is set.

Describe the bug
Prior to CLI v. 4.21.4, queries for @key directives were not generated automatically. As of 4.21.4, every @key directive seems to auto generate a query for the key, even if queries: null is set for the @model.
Not only does this result in potentially unwanted queries in the schema, it breaks the schema if queries: null is set. Without the generation of the default queries, input types necessary for the key queries aren't auto-generated.
Amplify CLI Version
4.21.4
To Reproduce

Add a model type to your schema with queries: null and at least one (named?) key.

type TestType
  @model(
    queries: null
    mutations: null
    subscriptions: null
  )
  @auth(rules: [{allow: private, provider: iam}])
  @key(name: "testKey", fields: ["string", "int"])
{
  id: ID
  string: String
  int: Int
  date: AWSDateTime
  email: AWSEmail}

$amplify push

View generated schema.graphql, you should see a query for the key, even though queries: null is set.

(I can't actually reproduce right now since 4.21.4 has been pulled from npm)
Expected behavior
I would expect no queries to be generated if queries: null is set on a model. If that's not the case, and queries on the model itself only applies to read/list and not the key queries, then I would expect there to be a way to disable key query generation as well.
Desktop (please complete the following information):

Node Version. v12.4.0

I was halfway through writing this up when 4.21.4 was pulled, so hopefully it's already being addressed. Wanted to file anyway in case this isn't the primary reason it was pulled back.

This is related to a feature which was included in v4.21.4, though we've noticed some issues with this. We are currently investigating this and will update the the following thread.
Closing this issue in favor of #4612

Reopening as this is a different issue, related to the @key changes. No a bug per se, the queries: null was never disabled the custom @key generation, and it is a change in behavior we'll have to decide about how to handle, since @key queries are now auto generated. But if auto generation will not be enabled by default then it is a non-issue.

@jhockett .. this is along the lines of the same issue. However, the difference here is that I would expect that if queryField is added to the key that it would generate the query (which it currently does) but it doesn't so generate all of the dependent types (specifically ModelFilterInput) which then throws an  error.
#5722


Could you push a tag, please?

Heya -- Thanks for providing this library.
From the last (and first 😄) release 1.0.0, quite a few changes have been merged.
When libraries depend on siphash's functionality from "after 1.0.0", they can use some dependency management tool to pin a git ref. However, when the siphash package is pulled in as a transitive dependency via dep, that utility will pick up the latest semver-ish looking tag. In this case, this would be 1.0.0 and can lead to some hard-to-find bugs that could be avoided 😉
So, for users of dep, it would be super convenient if there was a more recent tag.
Would you be willing to push one? 😃
Cheers

Done (v1.1.0). Thank you for noticing!


better issue template config (because i say that)

i love pull requesting

Please stop making pull request

why


docs: update contributing guide

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[x] 📝 Documentation
[x] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #
Closes #

Checklist

[ ] Code follows style guide (ruff)
[ ] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.

thanks @mfonekpo !


Indexed list comprehension

Problem

TBD

https://github.com/pen-lang/pen/blob/9addb4f761ee5e1d7e1778abfcf0296a5a3cfadb/examples/snake/entity/snake.pen#L52
https://github.com/raviqqe/perf-track/blob/93da37b4f50cfa23742cacc0c5a498aaa77e2960/database.pen#L79
https://github.com/raviqqe/perf-track/blob/93da37b4f50cfa23742cacc0c5a498aaa77e2960/database.pen#L87
Solution

Implement indexed list comprehension for list types.

Examples
[number x + " = " + Number'String(i) in x, i for xs]

Is parallel list comprehension enough?
Probably, yes.
[number x + " = " + Number'String(i) in x, i for xs, Sequence(xs)]

References

#1659

Closed in preference to #1659.


Default Value support for SimpleSelect

Hello,
is defaultValue supported for SimpleSelect? because the attribute 'value' makes it impossible to change the value.
<SimpleSelect defaultValue={...} >

Good point about defaultValue. (will look into it)
the value attribute works like that of React.DOM.Select, so you need you update the value you provide when it changes, like this:
...
render: function(){
    var self = this;
    return <SimpleSelect
        value = {this.state.selectedValue}
        onValueChange = {function(newValue, callback){
            self.setState({selectedValue: newValue}, callback);
        }}
    />
}

getInitialState: function(){

    // this is your initial value
    return {
        selectedValue: {
            label: "a",
            value: "a"
        }
    }

}
...

added defaultValue prop for SimpleSelect & defaultValues prop for MultiSelect, in release 0.3.1

:) :+1:

your last push did induce erros, search is not working anymore

yea version 3.0.x only works with React 0.14.0

Is it possible to just supply the value in the defaultValue instead of supplying the entire option label, value?

No.
This is the intended behaviour, just supplying the value to the default-prop makes it difficult to reason about the component (especially when working with custom Items), introduces extra type checks & makes the component more error prone.
The type of the defaultValue prop is Itemwhich means you have to supply the complete object.
by default, Item is a type alias for {label :: String, value :: a} where value can be of any type.
This is consistent throughout the component, all the props and event listeners share the same type.
defaultValue :: Item, value :: Item, onValueChange :: Item -> (() -> Void) etc.

The proposed solution didn't play nicely for me when used together with createFromSearch. Since the new option is not one of the standard available options, entering the name of the new option caused the selector to become empty. I came up with this alternative (still, I would welcome a defaultValue attribute):
export default class MySelector extends React.Component {
constructor(props) {
  super(props);
  this.state = {
    selectorOption: null
  };
}

_getDefaultOption = () => {
  return this.state.selectorOption !== null
    ? this.state.selectorOption
    : this._options().find(
        o => o.value = this.props.defaultValue
      );
}

_options = () => {
  return [
    {label: "one", value: 1}
    {label: "two", value: 2}
    {label: "three", value: 3}
  ]
}

render() {
  return <SimpleSelect
    value={this._getDefaultOption()}
    onValueChange = {option => {
      this.setState({
        selectorOption: option
      });
    }}
    options={this._options()}
  >
  </SimpleSelect>
}

}


Check eligibility

This branch adds a method to check a patron's eligibility by placing a hold on a fake item. CheckEligibility returns a standard message if the user is eligible, and otherwise fetches patron information and returns a list of potential problems with the patron's account. So far this is limited to checking fines, blocks, and expired cards.

Closing this because PatronElligibilityService is its own component now! https://github.com/NYPL-discovery/patron-eligibility-service


How to GIT pull latest BERT code

What is the correct commands for Git pull the latest BERT code and using it?  I have run into an issue with do_predict that I've seen others suggesting pulling the latest code base to potentially resolve such issues.
I'm using Colab and have the Git pull command figured out but not sure what I should do next.
https://github.com/google-research/bert.git

Figured it out.