python27Packages.boltons: 20.0.0 -> 20.1.0

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python2.7-boltons/versions.
meta.description for python27Packages.boltons is: "220+ constructs, recipes, and snippets extending (and relying on nothing but) the Python standard library"
meta.homepage for python27Packages.boltons is: "https://github.com/mahmoud/boltons"
Updates performed:

Version update
Quoted meta.homepage for RFC 45

Compare changes on GitHub

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 20.1.0 with grep in /nix/store/cmrqyh8ayfv67ydpplimdnsqph14wafc-python2.7-boltons-20.1.0

directory tree listing: https://gist.github.com/0fa84d0861ece66728ff86cbdbbfd18a

du listing: https://gist.github.com/ed86c3bbe647d47d577a00002324c0bc

Rebuild report (if merged into master) (click to expand)

36 total rebuild path(s)
9 package rebuild(s)
9 x86_64-linux rebuild(s)
9 i686-linux rebuild(s)
9 x86_64-darwin rebuild(s)
9 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python27Packages.boltons
python27Packages.face
python27Packages.glom
python37Packages.boltons
python37Packages.face
python37Packages.glom
python38Packages.boltons
python38Packages.face
python38Packages.glom

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cmrqyh8ayfv67ydpplimdnsqph14wafc-python2.7-boltons-20.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python27Packages.boltons https://github.com/r-ryantm/nixpkgs/archive/f3f15a591b59e62114ee7f73673f8265555e274d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cmrqyh8ayfv67ydpplimdnsqph14wafc-python2.7-boltons-20.1.0
ls -la /nix/store/cmrqyh8ayfv67ydpplimdnsqph14wafc-python2.7-boltons-20.1.0/bin

cc @twey for testing.

@ofborg eval


Error when building mxnet with USE_DIST_KVSTORE

Description
(A clear and concise description of what the bug is.)
I am building MxNet from source code, following the steps:

git clone --recursive https://github.com/ChaokunChang/incubator-mxnet byteps-backend
cd byteps-backend; mkdir build; cd build;
cmake -DUSE_CUDA=1 -DUSE_MKLDNN=1 -DCMAKE_EXPORT_COMPILE_COMMANDS=1 -DBUILD_CYTHON_MODULES=1 -DUSE_DIST_KVSTORE=1 ..
DEBUG=1 make -j48

Then I met an error like this:
Error Message
[  1%] Built target libomp-needed-headers                                                                                [5/1878]
[  2%] Built target dmlc
[  1%] Built target sample_lib
[  3%] Performing build step for 'ZMQ'
[  4%] Built target subgraph_lib
make[3]: *** No targets specified and no makefile found.  Stop.
3rdparty/ps-lite/CMakeFiles/ZMQ.dir/build.make:111: recipe for target 'external/ZMQ-prefix/src/ZMQ-stamp/ZMQ-build' failed
make[2]: *** [external/ZMQ-prefix/src/ZMQ-stamp/ZMQ-build] Error 2
Scanning dependencies of target gtest_main
CMakeFiles/Makefile2:2196: recipe for target '3rdparty/ps-lite/CMakeFiles/ZMQ.dir/all' failed
make[1]: *** [3rdparty/ps-lite/CMakeFiles/ZMQ.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  9%] Built target dnnl_common
[ 10%] Building CXX object 3rdparty/googletest/googletest/CMakeFiles/gtest_main.dir/src/gtest_main.cc.o
[ 14%] Built target omp
In file included from /home/ubuntu/byteps-backend/build/3rdparty/ps-lite/src/meta.pb.cc:4:0:
/home/ubuntu/byteps-backend/build/3rdparty/ps-lite/src/meta.pb.h:10:10: fatal error: google/protobuf/port_def.inc: No such file or directory
#include <google/protobuf/port_def.inc>
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
3rdparty/ps-lite/CMakeFiles/pslite.dir/build.make:113: recipe for target '3rdparty/ps-lite/CMakeFiles/pslite.dir/src/meta.pb.cc.o' failed
make[2]: *** [3rdparty/ps-lite/CMakeFiles/pslite.dir/src/meta.pb.cc.o] Error 1
make[2]: *** Waiting for unfinished jobs....
[ 32%] Built target dnnl_cpu
In file included from /home/ubuntu/byteps-backend/3rdparty/ps-lite/src/van.cc:14:0:
/home/ubuntu/byteps-backend/build/3rdparty/ps-lite/src/./meta.pb.h:10:10: fatal error: google/protobuf/port_def.inc: No such file or directory
#include <google/protobuf/port_def.inc>
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
3rdparty/ps-lite/CMakeFiles/pslite.dir/build.make:100: recipe for target '3rdparty/ps-lite/CMakeFiles/pslite.dir/src/van.cc.o' failed
make[2]: *** [3rdparty/ps-lite/CMakeFiles/pslite.dir/src/van.cc.o] Error 1
CMakeFiles/Makefile2:2159: recipe for target '3rdparty/ps-lite/CMakeFiles/pslite.dir/all' failed
make[1]: *** [3rdparty/ps-lite/CMakeFiles/pslite.dir/all] Error 2
[ 32%] Linking CXX static library ../../../lib/libgtest_main.a
[ 32%] Built target gtest_main
[ 95%] Built target mxnet_static
Makefile:140: recipe for target 'all' failed

To Reproduce
The code is from https://github.com/ChaokunChang/incubator-mxnet; branck byteps-backend. The code change doesn't include settings related to compile.
Steps to reproduce
What have you tried to solve it?

I have tried with ninja (add _GNinja when cmake), but with the same error.
Everything will be ok if I remove the -DUSE_DIST_KVSTORE in cmake-step.

Environment
We recommend using our script for collecting the diagnositc information. Run the following command and paste the outputs below:
curl --retry 10 -s https://raw.githubusercontent.com/dmlc/gluon-nlp/master/tools/diagnose.py | python

# paste outputs here
----------Python Info----------
Version      : 3.6.6
Compiler     : GCC 7.2.0
Build        : ('default', 'Jun 28 2018 17:14:51')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 19.3.1
Directory    : /home/ubuntu/anaconda3/lib/python3.6/site-packages/pip
----------MXNet Info-----------
No MXNet installed.
----------System Info----------
Platform     : Linux-4.15.0-1054-aws-x86_64-with-debian-buster-sid
system       : Linux
node         : ip-172-31-39-16
release      : 4.15.0-1054-aws
version      : #56-Ubuntu SMP Thu Nov 7 16:15:59 UTC 2019
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              8
On-line CPU(s) list: 0-7
Thread(s) per core:  2
Core(s) per socket:  4
Socket(s):           1
NUMA node(s):        1
Vendor ID:           GenuineIntel
CPU family:          6
Model:               79
Model name:          Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz
Stepping:            1
CPU MHz:             2724.723
CPU max MHz:         3000.0000
CPU min MHz:         1200.0000
BogoMIPS:            4600.12
Hypervisor vendor:   Xen
Virtualization type: full
L1d cache:           32K
L1i cache:           32K
L2 cache:            256K
L3 cache:            46080K
NUMA node0 CPU(s):   0-7
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch cpuid_fault invpcid_single pti fsgsbase bmi1 hle avx2 smep bmi2 erms invpcid rtm rdseed adx xsaveopt
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0028 sec, LOAD: 0.3791 sec.
Timing for GluonNLP GitHub: https://github.com/dmlc/gluon-nlp, DNS: 0.0005 sec, LOAD: 0.3934 sec.
Timing for GluonNLP: http://gluon-nlp.mxnet.io, DNS: 0.0008 sec, LOAD: 0.0229 sec.
Timing for D2L: http://d2l.ai, DNS: 0.0005 sec, LOAD: 0.0032 sec.
Timing for D2L (zh-cn): http://zh.d2l.ai, DNS: 0.0002 sec, LOAD: 0.1704 sec.
Timing for FashionMNIST: https://repo.mxnet.io/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0003 sec, LOAD: 0.0171 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0005 sec, LOAD: 0.0531 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0002 sec, LOAD: 0.0235 sec.

Please post the output of cmake -DUSE_CUDA=1 -DUSE_MKLDNN=1 -DCMAKE_EXPORT_COMPILE_COMMANDS=1 -DBUILD_CYTHON_MODULES=1 -DUSE_DIST_KVSTORE=1 ...
It seems protobuf is not found correctly on your system. If this is the case and if protobuf is required, we can change the CMakeLists.txt to error out in case of not finding protobuf.

Besides protobuf, zmq is also a required dependency. However, cmake currently treats it as optional, leading to build error
../3rdparty/ps-lite/src/./zmq_van.h:8:10: fatal error: zmq.h: No such file or directory  #include <zmq.h>
if -DUSE_DIST_KVSTORE=1 is used.

I verified this error is due to DLAMI shipping protoc version 3.8.0 but headers from version 3.0.0.
google/protobuf/port_def.inc is included in headers generated by protoc version >= 3.7.0, but of course if your system comes only with headers from 3.0.0 compilation will fail.

I'm opening a PR to ps-lite so that Protobuf_ROOT='~/anaconda3/' cmake [...] can be used to workaround the DLAMI bugs.
I expect the DLAMI team can fix the bugs in their release in early April 2020

You can workaround the DLAMI bugs by running with
cmake -DProtobuf_PROTOC_EXECUTABLE=/usr/bin/protoc [...]

Reported an issue upstream requesting if cmake may have any strategy to handle broken systems such as DLAMI https://gitlab.kitware.com/cmake/cmake/issues/20403

With respect to the Makefile build @apeforest, the reason is that Makefile based build doesn't use system protobuf, but downloads and compiles protobuf-cpp-3.5.1.tar.gz as part of the build.

https://github.com/dmlc/ps-lite/pull/170


ng-animate stops working correctly when there is an ng-if element looking at the same property

Look at this plunkr:
http://plnkr.co/edit/iv4qFtSShUWR7glbAhWA
You can see the behavior I'm talking about if you use chrome dev tools and break on attribute modification on the div with the ng-class property.
Basically, ng-animate stops using the classes 'red-add' or 'red-remove' if there is an ng-if element inside looking at the same property. The data-ng-animate-status attribute is stuck to 1 and doesn't advance to 2, I believe this is an unintended bug, but it took me a LOT of time to find out the reason why it wasn't working. Even if it's not immediately fixable, a footnote or a warning in the docs would at least give people something to look out for.

In the example, the classes are never added, because there are no transitions / animations on the element. If you add transitions, the classes are added and animations start to work: http://plnkr.co/edit/2XiyN5HaIlkSCWfMmIc4?p=preview


A large number of performance changes (related to #156)

Add ability for Executors to run setup() on thread pool start rather than construction
Boost Beast-based async HTTP endpoint (Redis-related waiting is synchronous, async redis api exists but is not used yet in this PR as it should be bypassed by most HTTP-related requests)
Redis bypass for fully local calls and calls explicitly setting directresulthost
"Soft shutdown" for executors to free up some resources immediately on threadpool exit
Executors ignore tasks if their timeout has already passed by the time they are in the front of the queue
Optional scheduler monitoring file that provides regularly-updated counters of tasks in the system

This is a fairly straightforward cherrypicked set of changes from my fork of faabric, I'm open to discussing api changes/tests/etc., otherwise it is ready to merge if you'd like.

This is very out of date, I might still extract out individual bits of my changes into smaller PRs and submit them later.


Update README.md

@VikashPR Pulled the main branch and posted a PR again. Do look into it

@VikashPR Pulled the main branch and posted a PR again. Do look into it

@ktk04 good job 👍🏽


Keyspace Event For Expired Values

Hello @JohnSully ,
I am very happy to have found this fork which support member expiration. I've tested it and it works very well.
However there is a feature I'd like to see: have the possibility to know/get the deleted value on the "notify-keyspace-events" notification system. Currently, we can only get the key of the set by parsing the channel.
To be honest, I've just started digging into Redis/KeyDB and I don't know much about it, so I don't even know if it's possible to add this feature without rewriting the whole project.
So, do you think that's possible? And if yes, would you agree to implement it? (Or maybe it's already possible?)
Thanks in advance.
Originally posted by @vfrz in https://github.com/JohnSully/KeyDB/issues/51#issuecomment-535694943

Hi, any progress on this ? Trying to build a simple service discovery on keydb, based on watch hash.

I am also interested in this feature, @JohnSully is there any progress on it?

bump


cddl_gen: make -t really supports multiple values

nargs='+' seems to apply operation on variable, however, the type
is a list, so it's a bit strange to me. However, using action='append'
make the type being a list and all the -t foo -t bar land to this
list ['foo', 'bar'], instead of ['bar'] for nargs.

I tend to agree with @MarcFinetRtone here that optional args are traditionally given -t foo -t bar. Both click and docopt work this way with repeated options as well, IIRC.

That being sid, I think that I went too quickly and didn't read the nargs documentation correctly, and even if I think that the current behavior is tricky, I can now adapt my makefile to "correctly" use the existing implementation (i.e. -t foo bar).

Are you planning on just working around this for now then?

@kyberdin

Are you planning on just working around this for now then?

I have this patch in my branch, but having upstream branch is better.
@oyvindronningstad

It may be possible to allow both.

It would be great (at least repeated -t as I'm used to :wink:), but requires a different patch than mine. Anyhow, if -t foo bar ought to be supported (for compatibility or any other reasons), this PR should be closed.

@MarcFinetRtone I'd like to keep the current way. It should be possible to support both, so feel free to reopen with that change.


Add fwupgrade_hal stub code

fwupgrade_hal is required by rdk-fwupgrade-manager.

https://jenkins.code.rdkcentral.com/job/github-triggered/10282/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/35831/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/5088/ started

Copyright scan failure
Commit: 109d1290cb2085b57a5db7a4265046958a12b8e6
Report detail: https://gist.github.com/a2141d4320a6e6e3d03c110bfde3dabf

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkb-turris-hal/111/rdkcentral/rdkb-turris-hal

Commit: 109d1290cb2085b57a5db7a4265046958a12b8e6

Report detail: gist

Hi Simon,
As discussed, please credit Technicolor in NOTICE:
Copyright [2017] [Technicolor, Inc.]
Licensed under the Apache License, Version 2.0

https://jenkins.code.rdkcentral.com/job/github-triggered/10283/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/35834/ started.

Copyright scan failure
Commit: 78f28375dbb808f9af8350771bcfba6a898ba2e9
Report detail: https://gist.github.com/26661699e3439c439ef3fce2e8b82aa8

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkb-turris-hal/111/rdkcentral/rdkb-turris-hal

Commit: 78f28375dbb808f9af8350771bcfba6a898ba2e9

Report detail: gist

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/5089/ started

Smoke tests for ['build-tur-bb'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/90567/


Status Update to Job workflow

I want to update the status of a PR right away when detected, but my job takes a long time to run. As a result, I am stacking a "detection" job in front of my long-running job so that I can see on the PR that Concourse is about to run the job:
resource_types:
- name: pull-request
  type: docker-image
  source:
    repository: jtarchie/pr

resources:
  - name: git-pr
    type: pull-request
    source:
      access_token: {{github-token}}
      repo: emcniece/concourse-pr-test

jobs:
- name: detect-pr
  plan:
  - get: git-pr
    trigger: true
    version: every
  - put: git-pr
    params:
      path: git-pr
      status: pending

- name: docker
  plan:
  - get: git-pr
    passed: [detect-pr]
    trigger: true
  - task: tag
    config:
      image_resource:
        type: docker-image
        source:
          repository: alpine
      platform: linux
      outputs:
        - name: tag
      run:
        path: sh
        args: ["-c", "echo test > tag/name"]
  - put: git-pr
    params:
      path: git-pr
      status: success

This looks like:

So the first job sets a "pending" status on the PR, then after the docker job is done the PR is set to "success". The docker job could also use the on_failure and on_success conditions to set the PR status as well and I am doing this in other pipelines.
Is this the best way to add a "pending" status to a PR? Is there some sort of on_start condition for jobs, or is there a better pattern for setting pending status while a job executes?

That is the pattern I would follow. You could just make it a step in your docker job, so it is all one job.
resource_types:
- name: pull-request
  type: docker-image
  source:
    repository: jtarchie/pr

resources:
  - name: git-pr
    type: pull-request
    source:
      access_token: {{github-token}}
      repo: emcniece/concourse-pr-test

jobs:
- name: docker
  plan:
  - get: git-pr
    trigger: true
    version: every
  - put: git-pr
    params:
      path: git-pr
      status: pending
  - task: tag
    config:
      image_resource:
        type: docker-image
        source:
          repository: alpine
      platform: linux
      outputs:
        - name: tag
      run:
        path: sh
        args: ["-c", "echo test > tag/name"]
  - put: git-pr
    params:
      path: git-pr
      status: success

I really don't understand what the issue is, though? Do you not like this pattern, is something slow, etc?

I was just curious as to whether this was the best way to proceed. Your recommendation for adding a task step is the kind of adjustment I was looking for ;)
Thank you!


KAFKA-3817 Follow-up: Avoid forwarding old value if it is null in KTableRepartition

Also handle Null value in SmokeTestUtil.

Ping @granders @mjsax @dguy for reviews.

Jenkins failure are un-related and is tracked in a separate JIRA.

LGTM

Merged to trunk and 0.10.0.

@guozhangwang, the agreed process is that we should add a JUnit test whenever a system test uncovers a bug that was not caught by a JUnit test. Is there a reason why that wasn't done here? Can we file a JIRA for that please?

Thanks for fixing the system test failure by the way. :)

@ijuma Yes I'm working on that right now, while I'm making a pass of all the streams unit tests. I merged this fix just to unblock system test Jenkins build.

Awesome, thanks.


Develop layout analysis R&D pipeline

Write a method for going from document text to predicted document segment classes. This should include:

a method for extracting text (embedded text or OCR)
a model

Developed a pipeline in the pdf-parsing-evaluation repo. This pipeline integrates:

classes for a document image, bounding box labels for an image, and a bounding box label dataset (multiple pages)
multiple possible pre-processing and post-processing steps (although none implemented so far)
the ability to add a model, and functions to easily add one from Layout Parser
the ability to map labels names using a dictionary mapping, e.g. when comparing models trained on datasets with different label schemes
model evaluation: AP and AR at a fixed IOU or a range of IOUs, at a certain confidence threshold.


Terminal Tab list is not scrollable, and height is not adaptive

Terminal Tab list is not scrollable, and height is not adaptive

@bogalex93 Do you know how to fix this ? I have the same problem with terminal list.


Draw referenced ways and nodes

For relations. Extract way drawing to a separate function.

That would be so useful. Would like to help, but my PHP skills are not that great :(

For that not php, but javascript skills are needed :)


Bun build support

This PR stems from attempting to build the engine from source as a submodule within a bun-based project. Ideally, the engine should be able to be built in bun successfully as a submodule without conflicting with the parent project node module types.
Overview

Adds bun lockfile https://bun.sh/docs/install/lockfile
Fixes type searching to be limited to local node modules (stops looking in folders above if engine is submodule)

Can you add some context about the motivation for this PR please?

So what's the Bun lockfile? How and when does it get updated?

So what's the Bun lockfile? How and when does it get updated?

So the bun lockfile works the same as package-lock.json but instead of being a json object its a binary.

So what's the Bun lockfile? How and when does it get updated?

So the bun lockfile works the same as package-lock.json but instead of being a json object its a binary.

I expressed some concern to ChatGPT for its take and it chimed with what I was thinking:

Your inclination not to commit a bun.lockb file to the repository is valid, especially given the PlayCanvas Engine's primary reliance on npm as its package manager. Including a lock file for a package manager that the repository doesn't officially support can introduce confusion, increase maintenance overhead, and potentially give the impression that bun is officially supported when it is not.

If you're using a submodule, can't you use a local override or generate a file into your submodule with a pre-build step or something?

So what's the Bun lockfile? How and when does it get updated?

So the bun lockfile works the same as package-lock.json but instead of being a json object its a binary.

I expressed some concern to ChatGPT for its take and it chimed with what I was thinking:

Your inclination not to commit a bun.lockb file to the repository is valid, especially given the PlayCanvas Engine's primary reliance on npm as its package manager. Including a lock file for a package manager that the repository doesn't officially support can introduce confusion, increase maintenance overhead, and potentially give the impression that bun is officially supported when it is not.

If you're using a submodule, can't you use a local override or generate a file into your submodule with a pre-build step or something?

So I've tested building the engine with bun and that works fine too with the same package lock. Yes you do not need to have bun lock file but for installation performance it's advised to have it by Bun themselves.
As for the using the engine as a sub module the types scoping is required regardless as types of the sub module will conflict is you are using a different runtime in the engine to your parent project.

Can we compromise on the types update then and back out the bun lockfile for now?

Can we compromise on the types update then and back out the bun lockfile for now?

Yep that works for me


RIOT-rs Contributing Guide

I was checking out how to contribute to this project, and could not find any documentation on it. Therefore the suggestion to make a RIOT-rs Contributing Guide.
Why

Making it easy for new contributors to contribute to the project
Increasing the number of (future) contributors

How
Creating a Contributing Guide
What
Template:
https://contribute.cncf.io/maintainers/templates/contributing/
Good example:

Kubernetes contributing guide
Contributing to the Atom Editor

I intend to pick up this one in November.

Current set-up available at https://pad.riot-os.org/contributing-guide#


Vector benchmarks

Perhaps there should be a serialisation/deserialisation benchmark for most/all of the types we have instances for. I can add them if desired, what do you think @fumieval

I didn't have enough type to cover all types. It would be definitely nice


Add pre-commit autoupdate workflow

This adds a GitHub actions workflow that will run the pre-commit autoupdate command to update pre-commit and its dependencies. The workflow is set to run weekly on Wednesdays.

Checklist

[x] Testing comment in the PR documents testing used to verify the changes

@xylar Can you tell me if you see anything obvious that should be changed about this workflow?

Very nice work, by the way!  I'm looking forward to seeing this in action!


missing line of code in gif

int c = a + b; is missing in the gif

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

ID: f783f152-b26c-0ea4-8017-3929c7465368
Version Independent ID: 8777c084-6578-8a7f-89db-80b3f9d547a7
Content: Tutorial: Create a simple C# console app - Visual Studio
Content Source: docs/get-started/csharp/tutorial-console.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@pb-balt Thank you for the feedback!
I think the purpose of the GIF is to facilitate an understanding on how auto-complete works and not necessarily follow the code snippet verbatim, though I will include the author here for future consideration.
FYI @tglee

My apologies for opening a ticket an issue along the same lines, I missed this before reporting. However, the purpose of the GIF is very obviously to show how to implement the example in all its glory. The GIF is simply incorrect and should be fixed.

@msebolt This does not need to be reopened; I will address it with #4000 - Thanks.


Profile information isset() error

On the profile settings page, if you delete the bio content and save, go back to the main profile page, the heading of Summary will still be present. This is supposed to be being checked by isset($data['bio']), but it's currently not blocking it from showing up.
When you use @if($data['bio'] != '') , the content is hidden if bio does not have a value present, but it breaks the unit test for the profile page.

Fixed with commit: c1155d52177f4bbecc0e8938a49e9f03c1d14e87


panic: could not match regexp with string

I was trying to transpile cmark on Mac OS.
What I did:

go get -u github.com/elliotchance/c2go
git clone https://github.com/commonmark/cmark.git
cd cmark
make : this is needed to generate buid/src/config.h etc.
c2go transpile -clang-flag -Ibuild/src src/blocks.c

I get this error:
$ c2go transpile -clang-flag -Ibuild/src src/blocks.c
panic: could not match regexp with string
^(?P<address>[0-9a-fx]+) <(?P<position>.*)>(?P<default> Default)?[\s]*$
0x7fee0d905c68 </Users/kjk/src/go/src/github.com/kjk/cmark/src/cmark.h:467:16, col:36> Inherited Default

goroutine 70 [running]:
github.com/elliotchance/c2go/ast.groupsFromRegex(0xc420346410, 0x47, 0xc4206451b3, 0x68, 0x1)
	/Users/kjk/src/go/src/github.com/elliotchance/c2go/ast/ast.go:270 +0x379
github.com/elliotchance/c2go/ast.parseVisibilityAttr(0xc4206451b3, 0x68, 0x12796db)
	/Users/kjk/src/go/src/github.com/elliotchance/c2go/ast/visibility_attr.go:12 +0x62
github.com/elliotchance/c2go/ast.Parse(0xc4206451a4, 0x77, 0x1274be1, 0x5)
	/Users/kjk/src/go/src/github.com/elliotchance/c2go/ast/ast.go:244 +0x2a96
main.convertLinesToNodes(0xc42020b190, 0x272, 0x15fd, 0x0, 0x0, 0x0)
	/Users/kjk/src/go/src/github.com/elliotchance/c2go/main.go:89 +0x198
main.convertLinesToNodesParallel.func1.2(0xc4200882a0, 0xc42024c000, 0xc42020b190, 0x272, 0x15fd, 0x0)
	/Users/kjk/src/go/src/github.com/elliotchance/c2go/main.go:121 +0x59
created by main.convertLinesToNodesParallel.func1
	/Users/kjk/src/go/src/github.com/elliotchance/c2go/main.go:119 +0x19d

Not sure which clang is used as I have llvm installed with brew, but probably the one that comes with XCode:
$ clang --version
Apple LLVM version 9.0.0 (clang-9<IP_ADDRESS>)
Target: x86_64-apple-darwin17.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Hi @kjk, I've put in some fixes with #645 . Try again with the latest version :)

The error is gone, thanks!


[Bug] distinct with windows function，the result contains duplicate values.

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.0.2-rc05
What's Wrong?
SQL:
select  distinct max(o_orderdate)over(partition by o_orderdate order by o_orderkey) from orders
the result contains duplicate values.
What You Expected?
Result shoud be distinct.
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@m719030572  Could you please provide the table schema and some sample data that could reproduce the bug?

orders is a tpch table

@yiguolei orders is a tpch table，use tpch-tools to build
   ./bin/build-tpch-dbgen.sh     ./bin/gen-tpch-data.sh -s 1     ./bin/create-tpch-tables.sh -s 1     ./bin/load-tpch-data.sh

if you execute sql:
explain analyzed plan select  distinct o_orderdate from orders
this distinct will be convert to agg，however, if there is a window function in the project, this conversion will not occur, only the property distinct = true will be set in the project, and this familiarity will be lost after cascade optimize


[MSI] - Have the upgrade experience restarts explorer once

Currently upgrading restarts explorer twice.  I'm 99% sure that is due to the uninstall script doing this along with the install script.
Is it possible to have some type of flag for when we are in an update process that doesn't restart for the uninstall since we are in an upgrade loop?

Closing this in favor of https://github.com/microsoft/PowerToys/issues/2730


[Merge Conflict Fix] port 'Use mina toolchain instead of ci-toolchain-base' to compatible

This is autogenerated Pull Request for porting #15792 to compatible
@dkijania

!ci-build-me

!ci-build-me

!ci-build-me


Import Dat Flow

This was brought up by user feedback. When importing a new dat we should be able to select which files to download.
In other download clients this is usually done through a dialog that asks some details, with a default location & all files being downloaded by default.
Related: https://github.com/datproject/dat-desktop/issues/335

@yoshuawuyts Cool, good feedback. I'll work on it!
If you had any concrete clients in mind that solve this well, which ones are those?
If not I'll start with that research.

uTorrent does this

Great job documenting!  I agree that it is a pretty crucial feature

As per discussion with @yoshuawuyts, we're splitting off a separate issue for selecting download location: https://github.com/datproject/dat-desktop/issues/335

This will be a multi-step flow:

Click "Import Dat" button
Paste Dat Link
( we now start looking for metadata)
select download location

optional if metadata is available:
3) give a name to the dat (if you're the owner)
4) select which files to download

@mafintosh and @joehand made something that is similar to the selecting which files to download https://github.com/mafintosh/directory-index-html
What's the status on this?

@karissa we have some sketches on paper; think we discussed that @Kriesse is going to sketch them out next week ish, and I'll implement them after I'm back from Italy. This relies on SLEEP to land though.

Paper sketch:

@juliangruber as discussed:

progress:

Mockups, also covering #260:
Download Dat View

Detail Dat View (#260)

@juliangruber Updated mockups above. I started the styling in add/download-screen
Can you help to finish?

[ ] debug: when entering a dat link, the download view flashes, then disappears. entering the link again fixes this.
[ ] debug: the "fetching" state never shows up
[ ] merge this with your work on the file list

debug: when entering a dat link, the download view flashes, then disappears. entering the link again fixes this.

by "disappears" do you mean you get back to the dats list or that there is a blank view?

@juliangruber I think this is almost there, except the flash of empty page when importing a dat. This is what I currently  see:

Enter dat link
(2. Only sometimes: download view flashes for a moment)
Dark empty screen
Download view says "N/A" in metadata fields
Download view shows actual metadata

This is what it should be:

Enter dat link
Download view in "Fetching Metadata" state
Download view with actual metadata loaded

ok, yeah then you see the same as I do. Should be fixed later today!

nice!!

not fixed yet, but @yoshuawuyts and me are working on it!

this has landed :)


Task 5: Add docstrings to every file.

Adds Docstrings to relevant functions.
No docstrings for setup.py or .gitignore.

⭐


Implements sameAs to use Rotonde from multiple machines

As described in #147 sameAs is an extension to portal.json.
It adds an array sameAs that contains entries to other portals. If those portals in turn have your portal in their sameAs arrays, you are considered remotes of each other.
The entire point is to decouple your Rotonde portal from a single machine. Using this feature you can now:

see the same feed as your remotes
others following your portal will see the posts you make from remotes
mentions to either the source portal or its remotes will be visible in the mentions tab

Start using it:
If your main portal is dat://7f2ee...0c5/ and your work computer has a portal at dat://3ba71...829/

Visit your main portal at dat://7f2ee...0c5/ and run mirror dat://3ba71...829/
Visit your work portal at dat://3ba71...829/ and run mirror dat://7f2ee...0c5/
You're done! You can now Rotonde from any of the machines with ease.

Remove remotes using the unmirror command:
unmirror dat://7f2ee...0c5/ this severs the link between remotes.
Thanks to @0x0ade for some back and forth across twitter, rotonde and github! :~

Madness

Wow, GitHub just automatically mirrors our comments here!
Also, 🎉 great work!

Has Paul seen this?

Who wants to have the honour of pressing merge?

I'm imagining you two staring with a dead-serious glaze at the screen, cursor over the merge button, finger over the mouse button... 🍿

Just read the high level description, looks like a solid solution.

I got Paul's blessing, I'm merging.


Add the GSA caller's email as a metric dimension.

We should consider having the Ingress add ce-actor to the events it admits as well.


Change description for lifecycle

Original PR: https://github.com/apple/swift-driver/pull/1065
Cherry-picking to release/5.7.

@swift-ci please test


Palash dev

Functionality - Allow superAdmin to add, update and delete the reserved configuration such as roles, action_groups and tenants. Till now, the reserved were loaded from the default configuration   present in yml files and now superAdmin has the privileges to do the same.
Testing - Tested on local to create, update and delete config.
Add the Reserved configuration -
curl -X PUT https://localhost:9200/_opendistro/_security/api/roles/palash_role -k -H 'Content-Type: application/json' -d '

{
"reserved": true,
"description": "This is Palash default role",
"cluster_permissions": [],
"index_permissions" : [
{
"index_patterns" : [],
"fls" : [],
"masked_fields" : [ ],
"allowed_actions" : [
"crud","read"
]
}
],
"tenant_permissions": []
}'
{"status":"CREATED","message":"'palash_role' created."}

Update the reserved configuration
curl -X PUT https://localhost:9200/_opendistro/_security/api/roles/palash_role -k -H 'Content-Type: application/json' -d '
{
"reserved": true,
"description": "This is Palash default role",
"cluster_permissions": [],
"index_permissions" : [
{
"index_patterns" : [],
"fls" : [],
"masked_fields" : [ ],
"allowed_actions" : [
"crud"
]
}
],
"tenant_permissions": []
}'
{"status":"OK","message":"'palash_role' updated."}

Delete the reserved configuration
curl -X DELETE https://localhost:9200/_opendistro/_security/api/roles/palash_role -k
{"status":"OK","message":"'palash_role' deleted."}

Good practice to title your PRs to reflect its functionality :-)


How to port the code for sensors other than temperature and humidity sensors

Hello, I am trying to port the IoT Hub data visualization sample to a flow sensor. There is no porting guide in the readme. I have made some changes in the index.js file in the public directory of the dashboard. I will attach screenshots of the file diff below.

After making only these changes, I am getting errors from the chart.js library. It is an uncaught type error which I do not quite understand. Screenshot below.

If I change the code back to temperature, it will start working again. Whenever I try to adapt the code to a different sensor is when I get this error.
Any help would be highly appreciated.
Please let me know if any further clarification or code is required.
Thank you.

I figured out how to do it. Everything is going fine but the ID names are case sensitive. Case sensitive id names are the real cause of this problem. Dataset IDs and yAxes ID names should match. I will attach screenshots below.

Everything in the file diff is still the same.
For the back end, JSON file should contain the new ID names. I will attach the screen capture of my JSON file down below.

Certain changes should be made on your chip to change the ID names of the JSON file.
Please add these recommendations to the Readme.
Happy Coding!!!


Fix Travis CI

This commit is supposed to fix the following error on Travis:
4: from /home/travis/.rvm/gems/ruby-2.6.3/bin/ruby_executable_hooks:24:in `<main>'
  3: from /home/travis/.rvm/gems/ruby-2.6.3/bin/ruby_executable_hooks:24:in `eval'
  2: from /home/travis/.rvm/rubies/ruby-2.6.3/bin/bundle:23:in `<main>'
  1: from /home/travis/.rvm/rubies/ruby-2.6.3/lib/ruby/site_ruby/2.6.0/rubygems.rb:294:in `activate_bin_path'
/home/travis/.rvm/rubies/ruby-2.6.3/lib/ruby/site_ruby/2.6.0/rubygems.rb:275:in `find_spec_for_exe':
  Could not find 'bundler' (1.17.3) required by your /home/travis/build/plataformatec/simple_form/Gemfile.lock.
    (Gem::GemNotFoundException)

Ref: https://github.com/bundler/bundler/issues/6882#issuecomment-469858087
Also, the CI can no longer build for jruby <IP_ADDRESS> due this error:
NoMethodError: undefined method `[]' for nil:NilClass
  /home/travis/.rvm/gems/jruby-<IP_ADDRESS>/gems/bundler-2.1.2/lib/bundler/source/git/git_proxy.rb:90:in `version'
  /home/travis/.rvm/gems/jruby-<IP_ADDRESS>/gems/bundler-2.1.2/lib/bundler/source/git/git_proxy.rb:147:in `block in copy_to'
  org/jruby/RubyDir.java:323:in `chdir'
  /home/travis/.rvm/gems/jruby-<IP_ADDRESS>/gems/bundler-2.1.2/lib/bundler/shared_helpers.rb:52:in `block in chdir'
  /home/travis/.rvm/rubies/jruby-<IP_ADDRESS>/lib/ruby/stdlib/monitor.rb:214:in `mon_synchronize'

I'm not sure what to do to fix that, maybe we could remove this from our build and keep only the last version of jruby?

I suppose the JRuby failure is due to incompatibility between newer versions of bundler and old versions of JRuby. Regardless, it seems like the tests are also failing in 9.2, so I think we can look into it separately later.


Intellicode for Java breaks Javadoc completion

Create a java file with
/**|  Javadoc completion no longer works here
public static void main(String[] args) {}

See https://github.com/redhat-developer/vscode-java/issues/811

@Eskibear can you take a look at this?

Sure, we didn't do lsp to code conversion for this snippet. I'll provide a fix.

We plan to publish a new version of the extension (1.1.5) on Monday with this fix. Thanks for reporting!


Update doxygen comments of the Cooperative Group API

lgtm.

@chrispaquot rebased the brand. Thanks for the notification.


[Backport] SQL: Avoid using timeseries with "having" clauses.

Backport of #5017 to 0.11.0.

Looks that CI failure relates to this patch. Would you check it?

Ah, the test used having-alias syntax not supported yet in 0.11.0. I changed the test slightly and it should run now.


Fixes issue where Rook operator fails to start

Adds the POD_NAMESPACE variable to the rook operator manifest
fixes #50

Thanks for the fix 👍

I cloned after this change was merged and encountered the same issue.

failed to run operator. Error starting agent daemonset: Error starting agent daemonset: cannot detect the pod name. Please provide it using the downward API in the manifest file

I'm new to k8s - Is there a way to recreate this Rook image?  I tried to bring up other pods and they all failed PersistentVolumeClaim is not bound because of this.  Should I just recreate those Pods (cockroachdb, wordpress, nginx ingress) or would they automatically connect to a new Rook?

FWIW -- Here is a diff of the example cluster from https://github.com/rook/rook/blob/master/cluster/examples/kubernetes/rook-operator.yaml and the one included in this repo:
--- rook-operator.yaml	2017-10-25 22:12:46.000000000 -0400
+++ rook-operator.example.yaml	2017-11-01 05:29:49.000000000 -0400
@@ -1,3 +1,8 @@
+apiVersion: v1
+kind: Namespace
+metadata:
+  name: rook-system
+---
 kind: ClusterRole
 apiVersion: rbac.authorization.k8s.io/v1beta1
 metadata:
@@ -12,6 +17,7 @@
   - pods
   - services
   - nodes
+  - nodes/proxy
   - configmaps
   - events
   - persistentvolumes
@@ -52,6 +58,8 @@
   resources:
   - clusterroles
   - clusterrolebindings
+  - roles
+  - rolebindings
   verbs:
   - get
   - list
@@ -79,13 +87,13 @@
 kind: ServiceAccount
 metadata:
   name: rook-operator
-  namespace: default
+  namespace: rook-system
 ---
 kind: ClusterRoleBinding
 apiVersion: rbac.authorization.k8s.io/v1beta1
 metadata:
   name: rook-operator
-  namespace: default
+  namespace: rook-system
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole
@@ -93,13 +101,13 @@
 subjects:
 - kind: ServiceAccount
   name: rook-operator
-  namespace: default
+  namespace: rook-system
 ---
 apiVersion: apps/v1beta1
 kind: Deployment
 metadata:
   name: rook-operator
-  namespace: default
+  namespace: rook-system
 spec:
   replicas: 1
   template:
@@ -122,6 +130,18 @@
         # current mon with a new mon (useful for compensating flapping network).
         - name: ROOK_MON_OUT_TIMEOUT
           value: "300s"
+        - name: NODE_NAME
+          valueFrom:
+            fieldRef:
+              fieldPath: spec.nodeName
+        - name: ROOK_OPERATOR_SERVICE_ACCOUNT
+          valueFrom:
+            fieldRef:
+              fieldPath: spec.serviceAccountName
+        - name: POD_NAME
+          valueFrom:
+            fieldRef:
+              fieldPath: metadata.name
         - name: POD_NAMESPACE
           valueFrom:
             fieldRef:

I removed the rook-system changes from that and the front-matter at the top of the file that defined rook-system, called this manifests/rook-operator.example.yaml, ran:
kubectl replace -f rook-operator.example.yaml

And now the rook-operator is running. I have Persistent Volumes.  Some of the other pods I created after the initial install have recovered.  I now have failing rook-agent pods - one for each node.

Hi @displague , thanks so much for investigating this issue! It sounds like this is indeed the issue. Either the kubernetes or rook API has changed abit such that it now requires some fixing. Unfortunately, I'm swamped with school now so I'm afraid I won't be able to look deeper into fixing this until mid December. If you wish, feel free to submit a PR until then :)
My hunch is that some flag needs to be added to the kubelet.service systemd file (which can be found at https://github.com/kahkhang/kube-linode/blob/master/manifests/container-linux-config.yaml and https://github.com/kahkhang/kube-linode/blob/master/manifests/container-linux-config-worker.yaml) to include --volume-plugin-dir=/etc/kubernetes/volumeplugins, and also probably the kubernetes version needs some bumping up as well to the latest one which supports flex volume plugins.
Thanks so much once again for highlighting the issue!


GeoPint is not implemented

Version info
Angular: 5.2.x
Firebase:  ^4.8.2
AngularFire: 5.0.0-rc.7
Description
AngularFire doesn't have GeoPoint interface
Expected behavior
import { AngularFirestore, AngularFirestoreCollection } from 'angularfire2/firestore';

...

constructor(private db: AngularFirestore) {
    let point = new db.firestore.GeoPoint(32.5522, 34.556656)
}

Actual behavior
TypeError: Cannot read property 'GeoPoint' of undefined
More info
Firebase JS reference:
https://firebase.google.com/docs/reference/js/firebase.firestore.GeoPoint

For the types, you should import from firebase.
import * as firebase from 'firebase/app'
...
let point = new firebase.firestore.GeoPoint(32.5522, 34.556656)

The jamesdaniels solution didn't work with me, any suggestions what might be the problem?
I'm getting error
TypeError: Cannot read property 'GeoPoint' of undefined

This is different in 5.0 now, you should do:
import { firestore } from 'firebase/firestore';
...
new firestore.GeoPoint ...

You are correct, GeoPint is not implemented. 🍻

I use this to make the GeoPoint work
import { firestore } from 'firebase';
let point = new firestore.GeoPoint(32.5522, 34.556656);

Hi what is the recommended way to import these classes?
There are different answers here.

Hi @jamesdaniels  I am confused.
What is the correct way to import?
import { firestore } from 'firebase/firestore';
...
new firestore.GeoPoint ...

does not work


Add support for outlook.live.com

Feature Description
Support outlook.live.com
Motivation and Context
Currently only office 365 is supported, however I am unable to log into it with my custom domain email - which is working on the outlook.live.com

I'm already working on it.

This repository is only for the Franz client. Please use this form for service requests or check out the guide to create your own service integration.


[5.1] Add UrlGenerator::clearForceRootUrl() to remove forcedRoot value.

It's useful is situation where I need to use different URL for asset() etc, and not anything else.
Signed-off-by: crynobone<EMAIL_ADDRESS>

Just curious, why not just write the asset URL by hand?

Just curious, why not just write the asset URL by hand?

It would take a slightly longer approach to do it as I need to considers:

multiple environment (local, staging, production).
multiple entry point (api via lumen, web via laravel). The image path is fetched from Eloquent accessor.
http vs https.

If only we can pass in $root when calling URL::asset(), I might have seen such PR before and was rejected. That would simplify many of this stuff.

Hi, You can build you own Helper function

So how are you using this in your app? Are you calling force, asset(), then clear? seems kinda cumbersome. :) just trying to understand <3

So how are you using this in your app? Are you calling force, asset(), then clear? seems kinda cumbersome. :) just trying to understand

Yeah it is. The easier non BC approach that I see right now is by this PR.
    /**
     * Get marker using accessor.
     *
     * @return string
     */
    public function getAvatarAttribute()
    {
        $path = config('upload.local');
        $url = app('url');
        $url->forceRootUrl(config('app.url'));
        $avatar = $this->pictures()->where('avatar', '=', 1)->first();
        $file = $this->getAvatarThumbnailPath($path, $avatar, 'marker') ?: "{$path}/default.marker.jpg";
        return $url->asset($file);
    }

But ideally I would love to see if we can do URL::asset($path, $secure = null, $root = null).
    /**
     * Generate a URL to an application asset.
     *
     * @param  string  $path
     * @param  bool|null  $secure
     * @param  string|null $root
     * @return string
     */
    public function asset($path, $secure = null, $root = null)
    {
        if ($this->isValidUrl($path)) {
            return $path;
        }
        // Once we get the root URL, we will check to see if it contains an index.php
        // file in the paths. If it does, we will remove it since it is not needed
        // for asset paths, but only for routes to endpoints in the application.
        $root = $this->getRootUrl($this->getScheme($secure), $root);
        return $this->removeIndex($root).'/'.trim($path, '/');
    }

Why not add a URL::assetFrom method which takes a custom root? :) More expressive than third argument.

Why not add a URL::assetFrom method which takes a custom root?

URL::assetFrom($root, $path, $secure = false)? Yeah that would be much better option.

Yeah. Wanna make a new PR or modify this one?

I'll work on a new PR. For both Laravel and Lumen.


fix: resolve broken link

When I was looking at your repo I found out that the HELP file is not present anymore. I think it should point to the docs/README.md as this is explaining some extra information around your bot.

:tada: This PR is included in version 1.6.6 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Fatal Exception on cells dequeue

Sometime i receive that error:

Fatal Exception: NSInternalInconsistencyException
unable to dequeue a cell with identifier MenuTableViewCell - must register a nib or a class for the identifier or connect a prototype cell in a storyboard

Can you help me? Thanks in advance

Looking into it @GZaccaroni

Hi @GZaccaroni, did you register the class in viewDidLoad?
self.tableView.registerClass(MenuTableViewCell.self,
            forCellReuseIdentifier: "MenuTableViewCell")

I have used the code of the example!! So yes! It happens only on iOS 8

Il giorno 09 giu 2016, alle ore 10:08, Adrian Mateoaea<EMAIL_ADDRESS>ha scritto:
Hi @GZaccaroni, did you register the class in viewDidLoad?
self.tableView.registerClass(MenuTableViewCell.self,
forCellReuseIdentifier: "MenuTableViewCell")
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

As per the move to Swift 3 and iOS 10 support, this is no longer an issue as we are not directly supporting iOS 8.


Unexpected :error_no_members

Hello, Seth!
I'm using pooler in my Elixir app to connect to my favorite music player service (MPD). I'm a bit confused with the behavior I'm seeing from pooler, though. With my config, I do see 2 ExMpd.Connection processes spawn to start. But when both are in use and I request a third, instead of creating a new one and ramping up toward 5 as I expected, I notice that the pooler:take_group_member(:mpd) call is returning :error_no_members.
Surely I'm just missing something simple, but I'm not sure what it is! The readme says that new workers should be spawned "on demand" ... but I would have thought calling take_group_member would qualify.
config :pooler, pools: [
  [
    name:       :mpd,
    group:      :mpd,
    init_count: 2,
    max_count:  5,
    start_mfa:  {ExMpd.Connection, :start_link, [mpd_host, mpd_port]}
  ]
]

Thanks in advance for any thoughts!
Adam

+1 I believe this issue is causing weird behavior with the riak elixir client, referenced above. It would be good to know if this behavior is intentional or not so libs can either deal with it or wait for a fix.

I would like to add, that I get this behavior only when trying to set it as a variable:

We've run into this issue as well and have tracked it down where it happens in the source w/in pooler.erl. It appears that this behavior is intentional, but the reasoning is still misunderstood.
Here's the code: https://github.com/seth/pooler/blob/a145451a2f12673b4f55c4657e1b572fc5dc4b8a/src/pooler.erl#L573-L585
If you're at the limit, you'll automatically get an error_no_members return while pooler asynchronously adds new workers.
@hansonkd - for what you're seeing, you've most likely set your init count to 1 and you're growing by 1 worker every time you get an error_no_members return. It just so happens that you're alternating calls that assign variables and then don't.

Any reason for not using take_member/2 with a timeout? A low figure timeout which would let the gen_server perform the batch add of new member before responding back to the caller.
Essentially you would be stating that the max allowed time for the caller to be stalled would be the timeout value. take_member/1 has 0 as timeout.
@seth: I'm curious about the design choice here. Any particular use case of take_member/1 with 0 timeout when culling free members is of interest, that is, having different init and max values?

First off, very sorry for the delayed responses. It's awesome to see pooler getting continued and on-going use!
In pooler's initial design, I avoided queuing and chose to have the pool return immediately with an error when no members are available. This met my needs at the time and kept the implementation simple. While errors aren't always nice, avoiding high latency variance sure is.
The dynamic pool re-sizing was added before the queuing feature (to my memory, didn't go look at git). So the choice to return an error during resize was made on purpose because it followed the original design and keeps things simple -- but not all work loads will be happy with this "fail first" approach.
With the queuing feature in place, no reason not to use it if that is the behavior you're looking for. I'd gladly take a patch to allow timeout to be specified for take_group_member.

I just added a pull request for this encase others run into it :P

As Seth mentioned, this behaviour is intentional. take_member(Pool) is equivalent to take_member(Pool, 0) (i.e. 0 timeout), so even though the new worker is spawned on background, the current requestor will get error_no_members.
And if someone wants to get a worker back no matter what, they need to use take_member with some reasonable timeout. As I understand, take_member/2 was not implemented, when this ticket was created, so, I'm closing it.


[scatter] Add bivariate choropleth view

enable scatter plot tool to view the data as a bivariate choropleth

add toggle for user to choose between scatter and bivariate choropleth
add code to generate the bivariate choropleth legend
data processing to use drawChoropleth method

move the code for drawing scatter plot into its own file so that chart.tsx calls either drawScatter or drawChoropleth depending on the chart type

dev updated (eg. https://dev.datacommons.org/tools/scatter#%26svx%3DMedian_Income_Person%26svy%3DCount_Person_AsianAlone%26pcy%3D1%26epd%3Dcountry%2FUSA%26epn%3DUnited States of America%26epts%3DCountry%26ept%3DCounty%26qd%3D0%26ld%3D0%26dd%3D0%26ct%3D1)

Thoughts from the demo page:

title: we may want to make A vs B clearer by emphasizing A and B, since they all could be long strings. Can we try make it 3 lines with "A", "vs", "B" in separate lines
Round the float value in bi-variate legend

If we make the title into 3 separate lines, the title will end up taking up more of the screen space and I think it might be better for the chart to have more space

Or, any other visual hint to make A and B more prominent? I found it hard to tell apart especially for two long names

How about this with the "vs" as a smaller and non bolded text?

this is better!


Unable to Hide Tab Bar

Overview
Given the available items under guioptions and toolbarhide, I expect to be
able to control the visibility of the tab bar but am un able to.
Is there another, canonical way to hide/show the tab bar?
Environment
:version
Pentadactyl 1.2pre (created 2015/05/15 11:35:37) running on:
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0

:help 'showtabline'


chore: add retry to pod exec

Copy of https://github.com/actions/runner-container-hooks/pull/125
This should get rid of the rare random CI failures because it will retry the executions 👍 , merging this, tagging you guys for visibility!

closes https://github.com/promaton/git-ops/issues/1202


WIP: runtime: fix runtime checks to work out of worker threads

As discovered on #7306 runtime checks don't work well outside of worker threads.
This is my first pass to fix this, which makes the integration tests happy but occurred to me is insufficient for a real fix, as downstream Envoy users might for example, use Envoy headers out of the worker thread, and need the runtime snapshot to not crash.
Uploading for conversation since I think we need to decide if TLS needs to work outside the worker thread, or just runtime.  I think for now it's just runtime, so if you look at the second part of the change (test-only) I think we could do a similar thing in real runtime, where we either give

const access to a shared runtime snapshot
non-const access to a shared (and mutex protected) snapshot
track thread ids and give non-const updates to snapshots
or
sort out the larger non-worker-thread-TLS issue

Without a second use case for 4) I'm inclined to do 3), and hand out shared pointer per-thread snapshots to non-worker threads, and snag the lock and update.  When/if we have a second use case we can look at a TLS refactor that works for both, since it's pretty minimal work to do #3
Want to discuss before I embark on anything major.
@mattklein123
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

@mattklein123 thoughts on the options above?

I think we can actually do one better, and make snapshot access actually const - looks like nothing but test code uses non-const and I can fix that up.  If so we can combine 2 and 3 for a fairly simple solution.
I'll go ahead and land that as a standone PR now, to see if it causes anyone trouble, then go fix this one up to use it.

Ok, I think this works better, but I want to see how CI is and probably do one more clean up pass before I remove the waiting tag.
Matt one question if you have a few, do you think this is large enough I should split the runtime change out from the HTTP feature port or leave them together?

Matt one question if you have a few, do you think this is large enough I should split the runtime change out from the HTTP feature port or leave them together?

Optimally I think it's worth splitting it out if you don't mind, but it's not a big deal to me either way.

I prefer smaller safer PRs so I'll close this off for a few and reopen once the runtime-only change has landed.


[ZIG-20]: Create comment and update TODOs

This pull request updates the .gitignore file and adds a TODO comment.
Changes:

Modified the .gitignore file to include .zig-cache/ instead of zig-cache/.
Added a TODO comment in main.zig to refactor and rewrite certain drawing functions to accept and render any game object, with a reference to a Linear ticket.

How to test:

Verify that .zig-cache/ is properly ignored.
Ensure that the new TODO comment is present in main.zig.

#22  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rafaelrene and the rest of your teammates on  Graphite

Merge activity

Jun 30, 1:44 PM EDT: @rafaelrene started a stack merge that includes this pull request via Graphite.


Xnero: Watch speedcarsrental.com

Xnero requests the watch of the watch_keyword speedcarsrental\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

speedcarsrental\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Lost changes when using --allow-unclean

Oops... it reset all of my changes

This may be expected behavior. https://github.com/L33T-KR3W/push-dir/blob/master/index.js#L75
Perhaps push-dir should prompt for or automatically stash changes if unclean?

This just bit me pretty good. A warning would have been pretty awesome.


Create code-coverage plugin

Hey, I just made a Pull Request!
How it works
Enabling
In order to use this plugin, you must set the
backstage.io/code-coverage annotation on your entity.
backstage.io/code-coverage: enabled

There's a feature to only include files that are in SCM in the coverage
report, this is helpful to not count generated files for example. To
enable this set the backstage.io/code-coverage annotation to
scm-only.
backstage.io/code-coverage: scm-only

Backend
The backend plugin provides API endpoints for submitting code-coverage
reports. Currently jacoco and cobertura are supported. These reports
are normalized to a json format that is stored in the database.
$ cat cobertura.xml | http POST "localhost:7000/api/code-coverage/report?entity=component:default/entity-name&coverageType=cobertura" "Content-Type:text/xml"
{
    "links": [
        {
            "href": "http://localhost:7000/api/code-coverage/report?entity=component:default/entity-name",
            "rel": "coverage"
        }
    ]
}

It also provides some additional API endpoints:

Viewing the latest report
Viewing a more condensed history of code coverage values
Retrieving file contents from source-control, used by the UI

Frontend
Provides a graph of code coverage change over time, as well as a file
view where you can see the highlighted lines.
Co-authored-by: nissayeva<EMAIL_ADDRESS>Signed-off-by: alde<EMAIL_ADDRESS>:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

there's a package-lock.json committed that needs to be removed! :)

there's a package-lock.json committed that needs to be removed! :)

actually there were two :-)

you'll see a conflict in plugins.ts - we just merged a thing that makes it unnecessary to register plugins through that file. So you can just revert that particular change entirely

One more larger thing. Your yarn.lock contains massive amounts of unrelated changes it seems. Was it regenerated from scratch perhaps? You will want to revert it to the version it was before your code changes, and then run yarn install again so you only get the delta that your code changes require

One more larger thing. Your yarn.lock contains massive amounts of unrelated changes it seems. Was it regenerated from scratch perhaps? You will want to revert it to the version it was before your code changes, and then run yarn install again so you only get the delta that your code changes require

I always check out yarn.lock from master and just run yarn afterwards, but I will try again

I always check out yarn.lock from master and just run yarn afterwards, but I will try again

Make sure that your code is rebased on that same master first, or get the lockfile from the branchpoint you are relative to :)

@freben redone all URLs now, so tackled that piece of feedback

@alde Note that I've rebased and kept working a little on top of your branch to see if we can get it in. So don't go off working on it locally :)

@alde Note that I've rebased and kept working a little on top of your branch to see if we can get it in. So don't go off working on it locally :)

all right! :-)


Add test for #208

This PR adds a simple test that checks for #208. Once we confirm failure on 1.9 but pass on 1.10 we can merge, add new version bounds and move on.
.... for simplicity I will rebase onto v0.6.x and then forward-port to main

TODO: fix test tolerance, version bounds.


Could not read script `.../ca.cutterslade.analyze' as it does not exist.

Describe the bug
Could not read script `.../ca.cutterslade.analyze' as it does not exist.
To Reproduce
Steps to reproduce the behavior:

configure the plugin in the following way
root build.gradle.kts:

plugins {
...
  id("ca.cutterslade.analyze") version "1.9.0"
}
...
allprojects {
    apply("ca.cutterslade.analyze")
}
subprojects {
    apply("java")
}

It's a multi-module project

FAILURE: Build failed with an exception.

What went wrong:
Task 'analyzeClassesDependencies' not found in project.

I tried to add it to root, and each module build.gradle but still the same error.


为其添加连续对话功能

您好，请问怎么为此项目添加连续对话功能呢

同问

Binance Free Tokens Giveaway - Claim Yours Now! 🎉💰
Welcome to the exclusive Binance Free Tokens giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this fantastic opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Act swiftly and enjoy your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:

Spread the Excitement on Twitter:

Share the news about claiming your free tokens!

Tweet: Share this Airdrop

Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

Interact with the Contract:

Once your wallet is connected, go to the Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

Binance
Welcome to the exclusive Binance Free Tokens giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this fantastic opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Act swiftly and enjoy your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:

Spread the Excitement on Twitter:

Share the news about claiming your free tokens!

Tweet: Share this Airdrop

Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

Interact with the Contract:

Once your wallet is connected, go to the Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

Exciting Developer Opportunity at GitHub!
Congratulations! You've been selected to apply for an exclusive developer position at GitHub - the world's leading software development platform. Apply now
Position: Senior Developer
Salary:

Competitive salary package starting at $200,000 per year

Benefits:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Unlimited vacation days
Professional development opportunities

Responsibilities:

Lead the development of cutting-edge projects
Collaborate with talented teams to drive innovation
Contribute to open-source projects and communities

Requirements:

2+ years of experience in software development
Proficiency in multiple programming languages
Strong problem-solving skills
Excellent communication and teamwork abilities

If you're ready to take your career to the next level and join a dynamic team of developers, apply now!
Hurry! Applications close soon.
We look forward to welcoming you to the GitHub family!
Best regards,
GitHub Recruitment Team
q549130180, @mLisao, @yxnne, @cslply, @xiyouqishui, @eJayYoung, @simonkuang, @vagrant1991, @shusida, @jddj007-hydra, @shiinaao, @PimHwang, @Josephucas, @ChurchTao, @zhangxinlun, @dolymood, @huangzyan, @SiYFang, @zhangxiaolang, @DeyiXu


Print tx fee when using transaction build command

Resolves #3024

LGTM, but don't add more detail than this. Extra detail can go into cardano-cli transaction view. The only reason this info here fits ok is because it's only here that we really know which output is the change. We should not add more misc ad-hoc things to a single command that is for building, not viewing transactions.

I was thinking of adding a --verbose flag that defaults to not printing the fee.

bors r+

bors r+


pack command should not delete "dist" folder if specified another with "--distpath"

I specify the a folder named "pack" to save the executable with --distpath, and instead of deleting "pack" folder, it is deleting "dist", where was deleted lots of important files. Luckly The folder is from google drive, and I could recover the files.

We are working on the issue.

@FeodorFitsner I have created #2500 pull request to solve the issue... Kindly review the pr

@IagoBeuller The issue is now resolved. Kindly close the issue. Install pre-release (obviously in a virtual environment) or wait for the next release ;)


SafeAreaView should allow for selecting the safe area padding

Is your feature request related to a problem? Please describe.
When trying to respect the SafeArea, sometimes we don't want to respect all of it.. maybe we just want to respect the topAnchor.
Describe the solution you'd like
extension SafeAreaView {
    init(padding: [Padding], content: () -> UIView) {
        // ...
    }
}

Describe alternatives you've considered
One could embed a view and then manually update the constraints.
Additional context
N/A

//
//  ViewController.swift
//  ExamSIUIK
//
//  Created by Leif on 3/22/21.
//

import UIKit
import SwiftUIKit

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()

        view.embed {
            SafeAreaView {
                Image(color: .orange)
            }
            .ignore(
                leadingAnchorWithPadding: 0,
                trailingAnchorWithPadding: 0,
                bottomAnchorWithPadding: 0
            )
        }
    }
}


Allow breakpoint control

Occasionally, I have come across the need to change the breakpoint from 480 to x number, it would be nice to have an mj-container attribute or perhaps something in  that would allow this to be controlled.

Hi @dalefish !
We'll introduce a mj-mobile-breakpoint (or something similar) in mj-head for this in MJML 4 ( probably try to sneak it into last 3.3.X version too)

Is this still open? I didn't find any information in the docs about this. Would appreciate the feature as well :)

It's available in MJML4-alpha, couldn't make it happens in MJML 3

Hey..should this feature be working on MJML4-Alpha?
I updated to 4.0.0-alpha.5 and get the console output "No handler found for: mj-breakpoint, in mj-head, skipping it" when using it within the head..

Can you double check you're running the alpha ?
➜  test cat test.mjml
<mjml>
  <mj-head>
    <mj-breakpoint width="320px" />
</mj-head>
  <mj-body>
    <mj-section><mj-column><mj-text>
Hello world!
</mj-text></mj-column></mj-section>
  </mj-body>
</mjml>
➜  test npm-exec mjml
Too much input arguments received
➜  test
➜  test npm-exec mjml -V
mjml-core: 4.0.0-alpha.5
mjml-cli: 4.0.0-alpha.5

And it does change the media query breakpoint for me

seems like I was still using the 3.3.5 version..after really upgrading, the parameter seems to work, but I get the error "TypeError: Cannot read property 'handler' of null Input file(s) failed to render" when running "mjml src/emails/email.mjml"

You might need to open a separate issue with the MJML you're using, there's a lot of migration to do  on MJML4


Versioning darwin core / meta.xml

Hi,
I'm not sure if DWC standard enforce use of xml but when generating meta.xml (at IPT) no version of DWC is available at any place.
Can you place any validation/schema like eml?
xsi:schemaLocation="eml://ecoinformatics.org/eml-2.1.1

For developers trying to write indexers or custom validators you basically lost in which version of DWC the resource is made.
A similar issue is open at https://github.com/tdwg/dwc/issues/107

Thank you for reporting this issue. I'm glad you also reported it in the DwC project, because this is the appropriate place for it.
Please note that it is the IPT administrator's responsibility to keep their extensions up-to-date. This will guarantee that the IPT publishes using the latest DwC terms. Please refer to this section in the IPT User Manual for more information about how to update extensions.


Separate stable version list from language menu

Summary
The language menu is using a widget/theme element rst-versions which in
the original rtd theme (which is the grandparent of the qiskit sphinx
theme) is a pop up/dropdown menu that has a bunch of extra information
in it that can be used to list different versions of the documentation
as well as download links. In #1261 that functionality was used to add a
list of stable versions to the sidebar since it was the logical place
for it. However, this has apparently been a source of confusion for most
users as unless they have prior knowledge about the previous version
links being on the same menu as the language selection and the
availability of old versions of the documentation goes unnoticed. This
commit addresses this by splitting the previous versions list out of the
language selector drop down and just adding a list to the bottom of the
side bar. We can change the formatting and style of this as needed in
the future, but at least for right now it gives us a clear location for
links to old version of the documenation.
Details and comments
Fixes #1266

This is what the side bar looks like when I build this locally:

and with the language menu selected


added ability to select TLS version in TR

What does this PR (Pull Request) do?

[x] This PR fixes #3764

This PR allows the selection of TLS version in Traffic Router.  It allows the protocols field in the Connector setup to be read in and used instead of the default "all" value.  No tests or documentation have been updated as this is a minor bug fix and adds functionality back in that was previously supported
Which Traffic Control components are affected by this PR?

Traffic Router

What is the best way to verify this PR?

In server.xml, add a protocols field to the 443 Connectors like this:
protocols="TLS1.2"
In CDN in a Box docker, run and wait until TR has been successfully started.  Log into any of the containers (i used TO) and verify that only the specified version of TLS is available.  Here is how i did the verification, though there are other ways:
Download this zip locally: https://github.com/drwetter/testssl.sh/archive/3.0.zip
Copy it into the TO container: docker cp ~/Downloads/testssl.sh-3.0.zip cdn-in-a-box_trafficops_1:/testsshl.zip
Log into the TO container: docker exec -it cdn-in-a-box_trafficops_1 /bin/bash
Install unzip: yum install unzip
Unzip the file: unzip /testsshl.zip
Run the script on the test delivery service: testssl.sh-3.0/testssl.sh -p https://video.demo1.mycdn.ciab.test
Verify the results look like this:
Testing protocols via sockets except NPN+ALPN
SSLv2      not offered (OK)
SSLv3      not offered (OK)
TLS 1      not offered
TLS 1.1    not offered
TLS 1.2    offered (OK)
TLS 1.3    not offered and downgraded to a weaker protocol
NPN/SPDY   not offered
ALPN/HTTP2 not offered
Run the script against the API endpoint: testssl.sh-3.0/testssl.sh -p https://trafficrouter.infra.ciab.test:3443/crs/stats
Verify that this endpoint still allows all TLS versions with an output like this:
Testing protocols via sockets except NPN+ALPN
SSLv2      not offered (OK)
SSLv3      not offered (OK)
TLS 1      offered (deprecated)
TLS 1.1    offered (deprecated)
TLS 1.2    offered (OK)
TLS 1.3    not offered and downgraded to a weaker protocol
NPN/SPDY   not offered
ALPN/HTTP2 not offered
If this is a bug fix, what versions of Traffic Control are affected?

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Can one of the admins verify this patch?

ok to test

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4407/

I should have also mentioned that I backported Matt's changes to 3.1 and that is what we will be running.

@hbeatty there is currently an open vote on the 3.1 release candidate. If you've tested this out and are satisfied, feel free to merge it and -1 the current release candidate for us to cut another one with this in it.


Refactor CI environment variable checks.

The goal of this is to reduce existing code duplication. This will make it easier in the future to refactor all the CI environment variable checks to be done inside a static buildFinished context so that they are configuration cache compatible.
If the approach is okay, we will also need to do the same refactor for the CCUD Maven extension.

I understand it's an intermediate state where there's actually a bit more code: isXXX() + static isXXX() but eventually there should be only static isXXX() calls, right ?

I think I will continue working on this PR before reopening. Thanks for the feedback!


Change application name to "aws_elixir"

This is needed to avoid conflicts with aws-erlang's application name.
It solves https://github.com/aws-beam/aws-codegen/issues/73

LGTM. Do you think we should rename aws_erlang too or is it ok to keep it aws?

@robertoaloi I think it would be good to rename it too. Just for consistency :)


Pass required params to Session constructor. Allow undefined in AuthScopes

WHY are these changes introduced?
The parameters required by Session were previously not being passed to the constructor, and we were instantiating instances of Session with just their id. This caused some issues due to potentially missing required params.
An issue with scopes potentially being undefined and causing errors in the AuthScopes class is also fixed in this PR. This initially seemed like a Session issue but was able to be resolved in the AuthScope class itself by allowing undefined scopes in the constructor, which then translates to an empty array of scopes.
The change to AuthScopes should potentially fix #159 and similar issues.
WHAT is this pull request doing?

Moves all required Session params into the constructor.

this is a potentially breaking change, depending on how directly users interact with Session

Updates all testing where we call new Session to pass the required params.
Updates AuthScopes to allow for undefined in the constructor, and then check for the presence of scope.

important to note that a large portion of this PR is updates to test files
Type of change

[ ] Patch: Bug (non-breaking change which fixes an issue)
[ ] Minor: New feature (non-breaking change which adds functionality)
[x] Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above
[x] I have added/updated tests for this change
[ ] I have documented new APIs/updated the documentation for modified APIs (for public APIs)

This needs a good rebase and probably a bit of refactoring since we've merged #153. I'll get that taken care of so this should be in a good place soon.

Will this be merged in soon? Really critical


try to automatically set run-specfic params

If batch submission mode is enabled. Try to automatically set which params shall be run-specific.
Following cases might be considered:

[x] only one parameter: set it to run specifc
[x] multiple parameter containing File and non-File parameters: set the File parameters (this is often the case when same parameters should be applied to multiple samples)
[x] if no file parameters, set all parameters to run-specifc
[ ] exclude some patterns from the above rule, e.g.:

genome
reference

Point 1-3 done with #138


Move nodemailer types to dependencies.

Motivation
For resolving dependencies in package, npm requires to have them in dependencies instead of devDependencies. So we should move @types/nodemailer to dependencies.
Changes

Moved @types/nodemailer from devDependencies to dependencies.

Will create new one with tests, and package updates


Improve CSAR: Topology

*Add own Entities for WarFile and SampleDB
Add relationships:
*WarFile HOSTED_ON Websphere
*SampleDB HOSTED_ON DB2
*WarFile CONNECTS_TO DB2
Try to add cute images to the topology
Move scripts to new Entities

working on it:
http://<IP_ADDRESS>:8080/winery/servicetemplates/http%253A%252F%252Ftypes.opentosca.org/EnterpriseDemo/

Finally done by #7 :godmode:


adapt pydantic.v1.error_wrappers.ValidationError: 1 validation error for Prompt

I want to customize the adapt function,  change the language to Chinese
this is code：
cache_dir = ".cache"

adapt(
    metrics=evaluate_stk,
    language="Chinese",
    cache_dir=cache_dir,
    llm=llm
)

and this is the error
Traceback (most recent call last):
  File "/Users/Desktop/side2side-eval/langchain_idealab/ragas_evaluate.py", line 87, in <module>
    adapt(
  File "/Users/opt/anaconda3/envs/self-rag/lib/python3.9/site-packages/ragas/adaptation.py", line 36, in adapt
    metric.adapt(language, cache_dir=cache_dir)
  File "/Users/opt/anaconda3/envs/self-rag/lib/python3.9/site-packages/ragas/metrics/_answer_relevance.py", line 173, in adapt
    self.question_generation = self.question_generation.adapt(
  File "/Users/opt/anaconda3/envs/self-rag/lib/python3.9/site-packages/ragas/llms/prompt.py", line 181, in adapt
    return self._load(language, self.name, cache_dir)
  File "/Users/opt/anaconda3/envs/self-rag/lib/python3.9/site-packages/ragas/llms/prompt.py", line 281, in _load
    return cls(**json.load(open(path)))
  File "/Users/opt/anaconda3/envs/self-rag/lib/python3.9/site-packages/pydantic/v1/main.py", line 341, in __init__
    raise validation_error
pydantic.v1.error_wrappers.ValidationError: 1 validation error for Prompt
__root__
  output in example 1 is not in valid json format: Expecting value: line 1 column 1 (char 0) (type=value_error)

same issue

@Mervyn-1 @runwean sorry about this but do keep an eye on #774. will update that issue when we have a fix

I've encountered the same issue. It would be greatly appreciated if the author could look into it. Thank you so much!

It might not be an issue with the LLM API. I tried GPT-4o, GPT-4o-mini, and DeepSeek, and their translated cache JSON is almost the same, and I can't find any errors. Also, the first time you execute the adaptation and test generation, it works fine, so I guess something is wrong when it's trying to read the promt JSON saved as cache.

I guess something is wrong when it's trying to read the prompt JSON saved as cache.

Assuming this is correct, how would I clear / remove the JSON saved as cache?

我遇到了同样的问题。如果作者能研究一下，我们将不胜感激。非常感谢！

请问你解决了吗

This has been fixed with v0.2 - I know finally 😅 🎉
do checkout the docs here: https://docs.ragas.io/en/stable/howtos/customizations/metrics/_metrics_language_adaptation/
reference here: https://docs.ragas.io/en/stable/references/prompt/#ragas.prompt.PromptMixin
and if you're migrating from v0.1 check out the migration docs here: https://docs.ragas.io/en/stable/howtos/migrations/migrate_from_v01_to_v02


Plot to screen after writemime() deprecation

I'm trying to update Gaston for use in IJulia after the writemime() deprecation in Julia 0.5
In Julia 0.4, I had this function to generate a plot for use in an IJulia notebook:
function writemime(io::IO, ::MIME"image/png", x::Figure)
    # The plot is written to /tmp/gaston-ijula.png. Read the file and
    # write it to io.
    data = open(readbytes, "$(gnuplot_state.tmpdir)gaston-ijulia.png","r")
    write(io,data)
end

Following the discussion in https://github.com/JuliaLang/julia/issues/14052, I changed the function definition to
function show(::MIME"image/png", io::IO, x::Figure)

I also import Base.show instead of Base.writemime; I also tried reversing the order of MIME and io. However, when I try plotting in a notebook, I get the textual description of the plot, not the plot itself. How should I update my code to make it work again?
(I'm not sure if this is the right forum for this question, but nothing seems to be an obvious fit. Please let me know if I should post this somewhere else.)

Your arguments are in the wrong order.  Define Base.show(io::IO, ::MIME"image/png", x::Figure).

See also http://docs.julialang.org/en/latest/manual/types.html#Custom-pretty-printing-1

i.e. basically you just need to do a search-and-replace of writemime with show; the arguments are the same.

Thanks for your help. Defining Base.show() instead of show() did the trick.


The extension is not compatible with the new Visual Studio 2019 Preview

Hello and thank you for viewing the issue.
I have been using Visual Studio 2019 Preview for more than a month and I noticed some improvements in performance and startup time.I want to use  SourceTrail with it, but I can't install the extension since is not compatible with the IDE.
Can anybody help me solve this ? Can some modifications to the .vsix extension solve the problem or it is another solution ?
Thank you !

Would love to have this for VS 2019. Currently I'm only keeping VS 2017 installed in case I need to use this extension.

Implemented with merge of pull request #54


Hosting issues

I'm having problems hosting this on railway.app, I'm using a dockerfile which I have attached below. The app build and deploys successfully but on the website all it says is upstream connect error or disconnect/reset before headers. retried and the latest reset reason: protocol error and the deploy logs say
2022/12/16 02:08:58 error opening repository "/soft-serve/repos/config": file does not exist
2022/12/16 02:08:58 Starting SSH server on :23231

When I try to clone the config repo from the site it times out
Cloning into 'config'...
ssh: connect to host softserve-docker-production.up.railway.app port 22: Connection timed out
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

this is my dockerfile currently
FROM charmcli/soft-serve:latest

I appoligize if this is a dumb question but i didnt understand much of the readme


[Confluent] Add examples for using --ids

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Usage of --ids is global and trival. Do we need add examples for each command?

Usage of --ids is global and trival. Do we need add examples for each command?

Usage of --ids is global and trival. Do we need add examples for each command?

No requirements to do so for each command, but service team feels that this would help customers understand --ids better.

Usage of --ids is global and trival. Do we need add examples for each command?

No requirements to do so for each command, but service team feels that this would help customers understand --ids better.


ReFS - Data Deduplication correction for Spanish version

Spanish version should state 'Data deduplication' (Desduplicación de datos) available on Windows Server, since version 1709 and later. Thanks.

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

ID: eb95c569-f9ad-8e2d-3423-2c9629e65b72
Version Independent ID: e2a4c64a-97c0-b310-2e47-d0c348a8ead7
Content: Resilient File System (ReFS) overview
Content Source: WindowsServerDocs/storage/refs/refs-overview.md
Product: windows-server-threshold
GitHub Login: @gawatu
Microsoft Alias: gawatu

Please also correct it to 'Deduplicación de datos', currently there is a typo in the fist word. Thanks.

#please-close


Access & save desktop drives using fileopenpicker or storagefolder.

I  want to get all drive .i.e.(c:,d: etc.) of Desktop without fileopenpicker which is manuallly picks items .i.e.file/folder.
It will work like System.io.DriveInfo.GetDrives().Also.StorageFolder Gets access Denied Error.

UWP apps do not get unrestricted access to drives.


Config module very slow

Config module seems to have ground to a halt - very slow.  Is it computed values overloading?

Seems to be when a value is saved, there is a huge amount of processing going in the background to save the value and then process the received subscription update, in particular to merge the update with the stored graphql cache.

Delays occur in production build as well as dev.
Note it only seems to occur when talking/manipulating the dev SITL api params.

Confirmed it has something to do with dev SITL api.

Doing more testing on an old slow mobile device the pauses are apparent on all APIs, just much more pronounced on the dev SITL api.  I think it has to do with all the graphql parameter data being reactive and a non-keyed array of data.  Whenever the data comes in or changes, the frontend has to re-compute it into a keyed array so it the parameters can be accessed directly.
If it was possible to have the graphql data keyed, it would solve that problem.

Can you give an example of how that would be done in the API?

I was hoping there was some way that graphql could serve the equivalent of a hash, but looking at the schema it can't do that, it has to be an array of hashes.  That's OK, I'll refactor the handling of the frontend code.

Does the hash need to change when the data changes or be a unique hash that remains constant?

The hash/key really needs to be the id of the parameter, eg. ACCEL_Z_D, so we can easily access parameters - ie. params['ACCEL_Z_D'].  The alternatives are either to iterate through the array of hashes that come in through graphql and create a new keyed array (which is what is happening at the moment), or each time a value needs to be accessed the component has to scan through all params to access it.

Understood. I'll see what is possible re the graphql return structure to simplify things for you.

The trouble at the moment is that every time the subscription returns an updated parameter, it forces a recompute of the entire secondary keyed array.  I think it's this that's causing the pauses/slowdowns.

The trouble at the moment is that every time the subscription returns an updated parameter, it forces a recompute of the entire secondary keyed array.  I think it's this that's causing the pauses/slowdowns.

parameter subscription updates status parameters frequently, such as STAT_RUNTIME and STAT_FLTTIME.  This forces a recompile of the parameter data internally.  Adding code to exclude STAT_ parameter updates from recomputations.

Every parameter update (mutation) seems to get at least two or three updates back through the subscription.

Look at gql pagination - no need to retrieve and process the entire set of parameters at a time.

Refactored parameter query and subscription logic to not need rekeying into keyed object, instead having an array lookup method, and also filters out STAT_ messages.  Interface and updates and subscription feedback is reasonably fast now on Chrome and Firefox, but inexplicably hangs Safari on both the desktop and mobile/tablet.
Worth noting that on traditional GCS there is a long pause everytime the GCS connects to a firmware to download parameters which can take a long time.  In our model the API is permanently connected to the firmware and preloads all the parameters and then watches them, so any client that connects to the API gets a very fast download of parameters (<1sec on a decent desktop).  This is actually a big plus point for our model.

Safari only hangs when the top bar is active and in the config module, must be some kind of race condition somewhere.  Separate issue.

It looks like when GCS such as qgroundcontrol and probably mission planner connects, it requests a stream of all parameters.  So instead of a single 'hit' of all parameters in a single query like the API provides to the graphql client, when qgroundcontrol requests the parameters one by one, this is also sent via the API to the graphql client, one by one.
In addition, qgroundcontrol requests the streams at a much higher rate than the API is currently set to (1hz), and this then causes a very high rate of update to the graphql client which produces a very high CPU churn in the browser.  It is this that is probably the cause of hanging Safari, and also ramps up the CPU considerably in firefox and chrome.
In addition addition, using the vue-apollo library subscribeToMore mechanism (which adds a graphql subscription and updates a query by return) add significant cpu overhead to processing and causes blocking hangs in the browser, leading to unresponsiveness.  It turns out that using a more manual model of processing the graphql query and subscriptions provides a much more responsive browser experience.

So a couple of actions from the API end:

Stop the API sending a complete query/set of parameters after every subscription notice
Limit the number of subscription messages that the API can be sent out to a defineable rate - eg. a max of 2hz.  This is really the only way of not overloading the browser.

So interestingly the stateMessage seems to return at 1Hz and provides a full query and subscription every second.
vfrHudMessage is being provided at very high rate (5-10hz+), and does NOT provide a full query, just a subscription update.

Changing graphql client code to manual query+subscription model, like params component, has helped significantly.  So the automatic vue-apollo subscribeToMore definitely has performance problems.

Safari still hangs the UI but has less cpu usage and still does some updates, and if the display of the topnav data is turned off then safari is OK and responsive.  The subscription data is still processed at fast rate (10hz+) so it's the display of the data at that rate that Safari can't cope with.

Add a ticker mechanism to the top navbar which limits the frequency that the subscription updates can update the data containers, which in turns limits the visual component updates.  Set at 2hz, this now makes Safari useable if still laggy, and chrome quite useable.  In the planner module, it limits the cpu churn to very low amounts, in the config module the cpu usage is still quite high although useable.  This is certainly due to the background sending of the entire parameter set every subscription, which happens regularly due to STAT_, COMPASS_ etc updates.
This must be turned off at the API, nothing more we can do about this in the client.
However the ticker mechanism at least limits the damage that the API can do even if it's set at a high rate.  As the client has no control over the API, this is a good safety to build in anyway.

Added ticker mechanism to cockpit hud, limiting to 2hz drops cpu pegging down to 50% from 100%+

you think something like this would help?
https://github.com/convoyinc/apollo-cache-hermes

@cglusky v.interesting, afraid it's above my head how to integrate that or even diagnose where the actual bottleneck is in the low level front-end logic.  I'm just trying to get it not to crash at the moment for the proof of concept, we can look at optimising in the future.  Will keep this in mind!

understood.  was most intrigued by the write up in the wiki as to why they thought they needed it.  seemed to be along the lines of what you are experiencing re performance.

Refactored code now works much better even in Safari.  Config filter screen still lags a bit in Safari as the entire parameter set is pushed from the API and processed in the background constantly.  This still needs to be fixed at the API end:
https://github.com/goodrobots/maverick-api/issues/34


Handle some special characters as a part of TokenKeyword

Token Keyword may contain only letters and (if configured) underscore or number. How about other special characters that may occur in some strings, for example:
"name:some-value"
or even
"name:@value"
Assuming that those special characters are not a part of this specific grammar, it may be nice to add a way to automatically join them into Keyword. Maybe there is a way to do that right now, but I haven't found a solution. What I can suggest is to change:
parser.go file

func (p *parsing) parseKeyword() bool { ...

if unicode.IsLetter(r) ||
			(p.t.flags&fAllowKeywordUnderscore != 0 && p.curr == '_') ||
			(p.t.flags&fAllowNumberInKeyword != 0 && start != -1 && isNumberByte(p.curr))  ||
			p.t.<someKindOfListContains>(r) {

or something like that. Currently as a temporary solution to check I've added something like that and I am receiving keywords like "some-value" and "@value".

About custom delimiters in keywords - it's make sense.
But you have to know that any rules wont be work. Tokenizer doesn't supported any rules for token sequences because it just simple parser. For example rule "after colon may by only @ or keyword" won't be work.
Currently you need to write method for your complex keyword, like
colonTokenKey := TokenKey(1) // :
atTokenKey := TokenKey(2)    // @
tokenizer := New()
// ..
stream := tokenizer.ParseString(str)
for {
	keyword := []string{}
	if stream.CurrentToken().Is(TokenKeyword) {
		keyword = append(keyword, stream.CurrentToken().ValueString())
		if stream.IsNextSequence(colonTokenKey, atTokenKey, TokenKeyword) {
			stream.NextToken()
			stream.NextToken()
			keyword = append(keyword, ":", "@", stream.CurrentToken().String())
		} else if stream.IsNextSequence(colonTokenKey, TokenKeyword) {
			stream.NextToken()
			keyword = append(keyword, ":", stream.CurrentToken().String())
		}
		// ...
	}
}

(code not tested, it is just an example)

I am sorry, I think there is a little misunderstanding. I would like to skip additional logic to determine if something is a "@" or "-". In the example I have written, I wanted to use ":" as normal separator which will be tokenized and I have done this:
	parser := tokenizer.New()
	parser.DefineTokens(TColon, []string{":"})

The rest of characters like "@" and "-" - I want to treat as a part of keyword, no special meaning for them, thus I've written a simple, ugly piece of code to verify my idea, it looks like that:
if unicode.IsLetter(r) ||
			(p.t.flags&fAllowKeywordUnderscore != 0 && p.curr == '_') ||
			(p.t.flags&fAllowNumberInKeyword != 0 && start != -1 && isNumberByte(p.curr))  ||
			r == '@' || r == '-'

It seems to be working correctly, as I am receiving tokens:
{ TokenKeyword  :  name }
{ TColon  :  : }
{ TokenKeyword  :  some-value }
{ TokenKeyword  :  name }
{ TColon  :  : }
{ TokenKeyword  :  @value }

as intended. The "-" and "@" behaves in the same way as underscore character which is what I need.

I added AllowKeywordSymbols. Please try it and give me feedback.

No feedback