Plain Java support

Hi,
this project looks really promising for my needs and I would like to thank you for your work.
I am working on a JSF web application and would like to embed this log viewer into it. The documentation on this topic is not written yet and the source code seems to be tightly coupled to Spring code. Am I missing something or is there no support for plain Java web projects right now?
Thanks in advance :)

You can use log viewer in a plain java application, you have to register com.logviewer.web.LogViewerServlet as a servlet and create a default spring context.
See https://github.com/sevdokimov/log-viewer/tree/master/demo/simple-java-web-application/ as a sample project.
There is MyLogViewerServlet.java class that extends com.logviewer.web.LogViewerServlet and creates a default spring context in init() method.

Hi, thanks for the answer. I already found this demo sample, but it seems to only work with a Spring context. The web application I'm working on is not a Spring application, but a JSF without Spring application.
Can this Spring context be used in a non-Spring application without any issues?


AES Decryption

Is there a way to decrypt NSData using only a key and IV  using . CTR and noPadding ? if so can you give me an example it would be so great, i have encrypted my data using only a key and iv key but i could find any example on the library to how to do it

Please see the FAQ: https://github.com/RNCryptor/RNCryptor/blob/master/README.md#faq
Q: Can I use RNCryptor to read and write my non-RNCryptor data format?


how to get billboard instance witch created by entity

Cesium 1.102.0 version
is there a way to get billboard instance witch created by entity?
PS:
at ScreenSpaceEventType.LEFT_CLICK event
use viewer.scene.pick() can get billboard instance

Hi @jinrilong! Cesium is a big project, so we use GitHub for feature requests and bug tracking exclusively. In the future, please take any questions to the Cesium Forum where there are members of the community and developers from the team who can help. Thanks!


Create cs.json

Add Czech translation

There is a Czech translation in the develop branch with some translation differences.

The one in develop branch is more accurate. Sorry. Didn't notice there is already a translation pending.


Make Kestrel TryAdd when adding it's IServer implementation

For ANCM in-proc to work with scenarios like:
WebHostBuilder.CreateDefaultWebHostBuilder() // which calls UseKestrel() and UseIISIntegration();
    .UseKestrel();

we need the call to UseIISIntegration to always add it's IServer implementation rather than Kestrel overriding it.
As we want out-of-proc and in-proc to work without code changes, we will need Kestrel to TryAdd it's server implementation.
cc/ @davidfowl @pakrym @Tratcher

Who's doing this?

You tell us 😁

Side note: Clone this bug for HttpSys.

Pushback: This pattern won't work for HttpSys and we'd like to use a consistent patern.

"Needs-design"?

What's happening for preview2 if anything?  @Tratcher @davidfowl @jkotalik do we need to get into a room and talk again?

@davidfowl @Tratcher implementing IDefaultServer feature right now would be a bit rushed in this timeline, for now we can make it when IISIntegration is called inproc, it can be called multiple times.

We decided not to do IDefaultServer or IServerSelector, just make UseIISIntegration override IServer every time.

@pakrym who's doing that?

https://github.com/aspnet/IISIntegration/issues/741 @pakrym ?


Allow column comments

I'm submitting a...

[ ] Bug report
[x] Feature request
[ ] Question

I have not found a way to support comments per column (working on MySQL). I might be missing something, but if not already supported, can it be added?

Comments are completely niche, nearly no one uses them, so it is not really high on prio. However if you want to submit a PR for it, you're welcome.

Implementing this, I should add an entry to both node-db-migrate's interfaces as well as an implementation in the mysql driver, right?
Assuming that a comment can be either a table or a column comment, adding comment would be something like:
exports.up = function (db, callback) {
  db.createTable('pets', {
    columns: {
      id: { type: 'int', primaryKey: true, autoIncrement: true, comment: 'primary key' },
      name: 'string'  // shorthand notation
    },
    comment: 'pets table stores all pets',
    ifNotExists: true
  }, callback);
}

No just the driver, since this is mostly MySQL exclusive stuff, most other dbs don't have that feature. But anyway, no there is no need to touch the interface, rather it would be an extension of alterColumn and createTable.
Your provided example however is ok. The shorthand notation could make sense, but I would put that outside of the scope of this addition and discuss that separately.
You will also need to move table up to the mysql driver directly
https://github.com/db-migrate/db-migrate-base/blob/7af3f80bad7fa5c64509a4435cce68891403c0d1/index.js#L172-L245
And after that you just need to handle comment accordingly.

No please keep operations atomic.

Either it is one create table or you must pass two statements although in db-migrate.

actually, postgres also has this functionality. But, since it's not SQL standard, their interface is very different than mySQL's => https://www.postgresql.org/docs/10/static/sql-comment.html

I will be going forth with the separate table function to the mysql driver. If another driver such as postgres needs this functionality I guess that would need another implementation, since the common interface supports more db's which do not have implementations (yet).

It seems that my table comment (comment associated with the table directly and not with a column) conflicts with the quick column definition (trying to create a column named "comment" of type whatever the comment text is).
In order to make the above notation work, comment has to become a reserved word in mysql and if a column named "comment" is needed the extended notation is required.

Currently low on, time will try to respond in the next days. But concerning the comment reserved keyword, this is a no go and not gonna happen. actually when creating with the columns notation this should however alreay work. See the logic for this here:
https://github.com/db-migrate/db-migrate-base/blob/7af3f80bad7fa5c64509a4435cce68891403c0d1/index.js#L177-L181

I have created the adjustment for column comments but table comments is still not agreed upon. Any input?
The current implementation works with changeColumn and any column spec since it simply enhances it.
I shall try to add some tests for that as well.


Loading in chunks - items are added continuously at top

Please check the samples: https://rintoj.github.io/ngx-virtual-scroller/samples

Go to LOADING IN CHUNKS
Check the Add items at top checkbox.
Also enable random heights.
Now scroll a bit down and then up (which should reach the scroll top 0).
Then the items are getting added continuously and the scroll doesn't stop at the first element in the viewport (i.e., when the items are prepended at scrollTop 0 then the scroll shouldn't move same as when we append the item at the bottom).

Need a quick solution for this or a workaround example in stackblitz would be helpful.

The same scenario doesn't work WITH SINGLE COLUMN example also.

Go to WITH SINGLE COLUMN
Enable random heights (i.e., Toggle Random Height).
Now prepend 10 items.
Then viewport should contain the same elements before prepending i.e, from index 0-4 but it scrolls to the prepended items. whereas it shouldn't change the viewport but the scroll to adjust depending on the prepended items height.

The viewport is changed when the items are prepended/appended which shouldn't.


Teams view task card highlighting

Summary
Adding a Task Card wrapper to enable highlighting with yellow or red border based on configurable Handle Time thresholds.
Show queue name on the Task Card instead of phone number
Checklist

[x] Tested changes end to end
[x] Any config changes added to all environment files
[x] Completed README template for new features in /docs/docs/feature-library/ and added feature row entry into the /docs/docs/feature-library/00_overview.md file

[x] Feature summary - high level summary of what the feature does
[x] Flex User Experience with animations - animations with descriptions if necessary of the user experience in Flex
[x] Setup and dependencies - description of any setup steps required for this feature
[x] How does it work? - description of plumbing supporting feature

[x] Added PR Label "ready-for-review"
[x] Requested one or more reviewers for the PR

Merging Task Card Highlighting


Add css-(un)escape, allow using escaped selectors

This adds:

An ability to escape CSS strings (css-escape, API akin to CSS.escape()) and parse escaped ones back to literal values (css-unescape, a naïve reverse of css-escape).
Simplified identifier parsing, as per the CSS spec, instead of unrelated and more restrictive HTML spec.
The extension of the clss-name syntax rule to allow backslash-escaped characters (should it be #x1F to #xFF, actually? I've included all of ASCII just in case...)
The intuitive parsing of those in the matchers.

All in all, it fixes #13 (if you use css-escape on the class name) and allows for a safer and more reliable element selection.
@Shinmera, I've tried to optimize the hell out of css-escape and css-unescape, but I may have missed some important nit. For instance, how do you specify the element type of a string -- it should be character, right? SBCL complains about it being unspecified :/

Should be all addressed now:

I've used do&cond in favor of loop.
css-unescape is now called when compiling selectors, not matching them.
escapable is a toplevel function now.

I've introduced some index-out-of-bounds mistakes while refactoring, they should be fixed now :P

Looks good (save for not upcasing t, but I'll let that slide for now.)

Ahhhh, right, I'm always forgetting about this stylistic convention of yours :sweat_smile:


Add SmartModule support w/ WASI workflow in docs

In This week in Fluvio #24 we walk through compiling Fluvio w/ WASI support.
This is a valuable workflow for a (very) technical user developing SmartModules, and it belongs somewhere in the docs so we don't lose it.
The current structure doesn't provide an obvious place for this to live, so this may need to wait for reorganization to occur first.

I think we'll need reorganize our documentation in the following areas:

Data Streams
Intelligent Streams
Connectors
Pipelines

That should give us to add "WASM" related content in "Intelligent Streams"
I suggest we open a PR on the content hierarchy to start tackling these issues.


How to add a new op which is similar with matmul

I want to add a new op which is similar with matmul, for example anothermatmul.

add REGISTER_OP("AnotherMatMul") to tensorflow/core/ops/math_ops.cc like MatMul
add anothermatmul_op.h anothermatmul_op.cc to tensorflow/core/kernels/
bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package
then I can see  bazel-bin/tensorflow/core/libmath_ops_op_lib.lo is update, I think the new op has been compiled successfully.
install

I find util/python/python_lib/tensorflow/python/ops/gen_math_ops.py is not updated, and cannot use AnotherMatMul in python.
how to update gen_math_ops.py?
or how to generate python wrap?

Please be a bit clearer about "how to use".
There are a few steps:

Make sure you add anothermatmul_op in the BUILD file
Make sure you follow the instructions if you want to run in-place (or install the pip)

If the gen_math_ops.py is not generated, then it means that you didn't rebuild properly. Try bazel build --explain=/tmp/blaze_debug.log --config=opt //tensorflow/python:math_ops
It should show that the file is updated.

Thanks. I add anothermatmul_op to BUILD,  bazel build and install. But the  gen_math_ops.py cannot be updated.

run bazel build tensorflow/python:math_ops_gen, and a new gen_math_ops.py is generated in bazel-genfiles/tensorflow/python/ops/, which can find the new op another_mat_mul.
cp bazel-genfiles/tensorflow/python/ops/gen_math_ops.py util/python/python_lib/tensorflow/python/ops/gen_math_ops.py
add a function def another_matmul: gen_math_ops.another_mat_mul to util/python/python_lib/tensorflow/python/ops/math_ops.py
when call tf.another_matmul, get error:tensorflow.python.framework.errors_impl.NotFoundError: Op type not registered 'AnotherMatMul'

however, I have add REGISTER_OP("AnotherMatMul") to tensorflow/core/ops/math_ops.cc
how to add a new op?
Thanks in advance.

OK, that's great. The build/install should have worked. I never had to do this before, but try to uninstall first before installing. And maybe remove /tmp/tensorflow_pkg before you run the pip building script.
The error you are seeing is because the _pywrap_tensorflow_internal.so has not been updated. If you figure out how to rebuild the PIP install then it should work.

Thanks! uninstall and install again, that solved!


Enable sbt plugin hack in Maven repo paths by default

@jvican Didn't you mention something related on the Gitter of coursier? I've been wanting to enable that by default.

Indeed, I did :). Good to see this change.
I finally did this change to my code as well as adding sbt repos by hand.


Only Afghanistan is visible after switching indicators

Description
Axis option can be changed by clicking Axis description(Bubble , Bubble_Map)
Actual result
SUMMARY:  Chart is disturbed by selecting “GDP per capita”
SEVERITY:  Critical
MODULE:  Bubble Map chart
Platform:

Windows 7 / Chrome 47.0
MAC / Safari 9.0

RELEASE: Stage
STEPS TO REPRODUCE:

Navigate to the URL : http://tools-stage.gapminderdev.org/tools/map
Bubble Map Chart will be loaded.
Click on Y axis description
Hover the Wealth option and Select “GDP per capita”
Note that Chart is disturbed by selecting “GDP per capita”

ACTUAL RESULTS:
Chart is disturbed by selecting “GDP per capita”
EXPECTED RESULTS:
Chart should not disturb in any case
Other information
Reported by: Abid Ali
Test case number: TC69
Test run: https://valorsoftware.testlodge.com/projects/15827/runs/189933?tab=2&run_section_id=162891&executed_case_id=9210263#executed_case_9210263

@k6Labz Can not reproduce. Please retest


Essential Oils Diffuser support

eWeLink Diffuser - details :
manufacturer SHINGHUNG
model YB-01
sw versson PSF-BTA-GL v2.9.0
cloud online
rssi -66
If you need anything from me to get this working just let me know. Currently I can only turn it on/off but it has a light and fan and also a water sensor. Thanks.

Please, share it to my account<EMAIL_ADDRESS>in eWeLink app.
This may not work, depending on your country.

I've shared it with you.

Got. Can I turn it on for a short time at any time?

Of course, feel free :)

Please send link to device. Can't google it.

https://www.amazon.co.uk/dp/B07WF7MQ17?ref=ppx_pop_mob_ap_share

Check https://github.com/AlexxIT/SonoffLAN/releases/tag/v2.2.1

Working perfectly. Thank you for getting this to work :)


Units not restarting after a VM reboot

This happened to me after I reboot my VM's, my state is this
UNIT                MACHINE             LOAD    ACTIVE      SUB
deis-builder.service        f41053a5.../10.240...   loaded  activating  start-post
deis-cache.service      4565cb86.../10.240...       loaded  active      running
deis-controller.service     4565cb86.../10.240...   loaded  inactive    dead
deis-database.service       f41053a5.../10.240...   loaded  active      running
deis-logger.service     f41053a5.../10.240...   loaded  inactive    dead
deis-logspout.service       2bace2bf.../10.240...   loaded  activating  auto-restart
deis-logspout.service       4565cb86.../10.240...       loaded  activating  auto-restart
deis-logspout.service       f41053a5.../10.240...   loaded  active      running
deis-publisher.service      2bace2bf.../10.240...   loaded  active      running
deis-publisher.service      4565cb86.../10.240...       loaded  active      running
deis-publisher.service      f41053a5.../10.240...   loaded  active      running
deis-registry.service       4565cb86.../10.240...       loaded  active      running
<EMAIL_ADDRESS>     4565cb86.../10.240...       loaded  active      running
<EMAIL_ADDRESS>     2bace2bf.../10.240...   loaded  active      running
<EMAIL_ADDRESS>     f41053a5.../10.240...   loaded  active      running
deis-store-daemon.service   2bace2bf.../10.240...   loaded  active      running
deis-store-daemon.service   4565cb86.../10.240...   loaded  active      running
deis-store-daemon.service   f41053a5.../10.240...   loaded  active      running
deis-store-gateway.service  4565cb86.../10.240...   loaded  active      running
deis-store-metadata.service 2bace2bf.../10.240...   loaded  inactive    dead
deis-store-metadata.service 4565cb86.../10.240...       loaded  inactive    dead
deis-store-metadata.service f41053a5.../10.240...   loaded  inactive    dead
deis-store-monitor.service  2bace2bf.../10.240...   loaded  active      running
deis-store-monitor.service  4565cb86.../10.240...   loaded  active      running
deis-store-monitor.service  f41053a5.../10.240...   loaded  active      running
deis-store-volume.service   2bace2bf.../10.240...   loaded  activating  start-pre
deis-store-volume.service   4565cb86.../10.240...   loaded  activating  start-pre
deis-store-volume.service   f41053a5.../10.240...   loaded  activating  start-pre

I have tryied running a deisctl start platform but it hangs after this.
● ▴ ■
■ ● ▴ Starting Deis...
▴ ■ ●

Storage subsystem...
deis-store-monitor.service: active/running
deis-store-daemon.service: active/running

My initial state was this:
UNIT                MACHINE             ACTIVE  SUB
deis-builder.service        2bace2bf.../10.240...   active  running
deis-cache.service      f41053a5.../10.240...   active  running
deis-controller.service     4565cb86.../10.240...       active  running
deis-database.service       f41053a5.../10.240...   active  running
deis-logger.service     4565cb86.../10.240...       active  running
deis-logspout.service       2bace2bf.../10.240...   active  running
deis-logspout.service       4565cb86.../10.240...       active  running
deis-logspout.service       f41053a5.../10.240...   active  running
deis-publisher.service      2bace2bf.../10.240...   active  running
deis-publisher.service      4565cb86.../10.240...       active  running
deis-publisher.service      f41053a5.../10.240...   active  running
deis-registry.service       2bace2bf.../10.240...   active  running
<EMAIL_ADDRESS>     4565cb86.../10.240...       active  running
<EMAIL_ADDRESS>     2bace2bf.../10.240...   active  running
<EMAIL_ADDRESS>     f41053a5.../10.240...   active  running
deis-store-daemon.service   2bace2bf.../10.240...   active  running
deis-store-daemon.service   4565cb86.../10.240...       active  running
deis-store-daemon.service   f41053a5.../10.240...   active  running
deis-store-gateway.service  2bace2bf.../10.240...   active  running
deis-store-metadata.service 2bace2bf.../10.240...   active  running
deis-store-metadata.service 4565cb86.../10.240...   active  running
deis-store-metadata.service f41053a5.../10.240...   active  running
deis-store-monitor.service  2bace2bf.../10.240...   active  running
deis-store-monitor.service  4565cb86.../10.240...   active  running
deis-store-monitor.service  f41053a5.../10.240...   active  running
deis-store-volume.service   2bace2bf.../10.240...   active  running
deis-store-volume.service   4565cb86.../10.240...       active  running
deis-store-volume.service   f41053a5.../10.240...   active  running

Im following this guide: http://docs.deis.io/en/latest/installing_deis/gce/, and Im running deis version Deis CLI 0.10.0
Starting the units one by one with deisctl start <service>, works ok

Hi,
I'm having the same problem. I have a fresh installation on 3 bare metal(VivoPC with 8GB RAM). I'm using this user-data (IP and discovery URL has changed): https://github.com/deis/deis/blob/master/contrib/coreos/user-data.example
Just using deis register command. Same problem occurred after nodes reboot.
CoreOS info:
core@coreos-1 ~ $ cat /etc/os-release
NAME=CoreOS
ID=coreos
VERSION=607.0.0
VERSION_ID=607.0.0
BUILD_ID=
PRETTY_NAME="CoreOS 607.0.0"
ANSI_COLOR="1;32"
HOME_URL="https://coreos.com/"
BUG_REPORT_URL="https://github.com/coreos/bugs/issues"

Before reboot:
core@coreos-1 ~ $ deisctl list
UNIT                            MACHINE                         LOAD    ACTIVE          SUB
deis-builder.service            b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-cache.service              9383b249.../<IP_ADDRESS>       loaded  active          running
deis-controller.service         b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-database.service           b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-logger.service             9383b249.../<IP_ADDRESS>       loaded  active          running
deis-logspout.service           9383b249.../<IP_ADDRESS>       loaded  active          running
deis-logspout.service           9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-logspout.service           b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          9383b249.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          b767ce63.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>       9383b249.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         b767ce63.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         9e67d237.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       b767ce63.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>  9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       b767ce63.../<IP_ADDRESS>       loaded  active          running

After reboot:
core@coreos-1 ~ $ deisctl list
UNIT                            MACHINE                         LOAD    ACTIVE          SUB
deis-builder.service            b767ce63.../<IP_ADDRESS>       loaded  activating      start-post
deis-cache.service              9383b249.../<IP_ADDRESS>       loaded  active          running
deis-controller.service         b767ce63.../<IP_ADDRESS>       loaded  inactive        dead
deis-database.service           b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-logger.service             9383b249.../<IP_ADDRESS>       loaded  inactive        dead
deis-logspout.service           9383b249.../<IP_ADDRESS>       loaded  active          autp-restart
deis-logspout.service           9e67d237.../<IP_ADDRESS>       loaded  activating      auto-restart
deis-logspout.service           b767ce63.../<IP_ADDRESS>       loaded  activating      auto-restart
deis-publisher.service          9383b249.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-publisher.service          b767ce63.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>       9383b249.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         b767ce63.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         9e67d237.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>         9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-daemon.service       b767ce63.../<IP_ADDRESS>       loaded  active          running
<EMAIL_ADDRESS>  9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-metadata.service     b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-monitor.service      b767ce63.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       9383b249.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       9e67d237.../<IP_ADDRESS>       loaded  active          running
deis-store-volume.service       b767ce63.../<IP_ADDRESS>       loaded  active          running

core@coreos-1 ~ $ etcd --version
etcd version 0.4.7
core@coreos-1 ~ $ fleet --version
fleetd version 0.9.1
core@coreos-1 ~ $ deisctl --version
1.4.1

When i reboot just one node, there is no problem. But when i reboot them all(3 nodes) same time, the error occurs. For example if the power grid fails, Deis can't boot properly.
Some of the problematic component (deise-builder, deise-logspout) of logs is as follows.
$ journalctl -fu deis-logspout

Mar 24 12:45:07 coreos-2 systemd[1]: Started deis-logspout.
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 attach: 2a6fb54d3fad deis-logspout success
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 attach: d420d9003071 deis-store-metadata success
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 attach: 6b0c4264583c deis-router success
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 attach: c3947f0e5091 deis-store-daemon success
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 attach: 01d2045c12ef deis-store-monitor success
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 attach: b072b573c1c5 deis-publisher success
Mar 24 12:45:08 coreos-2 sh[18803]: 2015/03/24 12:45:08 etcd: http://<IP_ADDRESS>:4001
Mar 24 12:45:09 coreos-2 sh[18803]: 2015/03/24 12:45:09 url: 100: Key not found (/deis/logs/host) [525703]
Mar 24 12:45:09 coreos-2 systemd[1]: deis-logspout.service: main process exited, code=exited, status=1/FAILURE
Mar 24 12:45:09 coreos-2 docker[18845]: Error response from daemon: No such container: deis-logspout
Mar 24 12:45:09 coreos-2 docker[18845]: time="2015-03-24T12:45:09Z" level="fatal" msg="Error: failed to remove one or more containers"
Mar 24 12:45:09 coreos-2 systemd[1]: Unit deis-logspout.service entered failed state.
Mar 24 12:45:09 coreos-2 systemd[1]: deis-logspout.service failed.

$ deisctl journal deis-builder

Mar 24 12:46:50 coreos-3 sh[1532]: builder: waiting for confd to write initial templates...
Mar 24 12:46:55 coreos-3 sh[1532]: 2015-03-24T12:46:55Z c1f498dd8212 confd[3649]: ERROR Config check failed: exit status 1
Mar 24 12:46:55 coreos-3 sh[1532]: 2015-03-24T12:46:55Z c1f498dd8212 confd[3649]: ERROR Config check failed: exit status 1
Mar 24 12:46:55 coreos-3 sh[1532]: builder: waiting for confd to write initial templates...
Mar 24 12:47:00 coreos-3 sh[1532]: 2015-03-24T12:47:00Z c1f498dd8212 confd[3664]: ERROR Config check failed: exit status 1
Mar 24 12:47:00 coreos-3 sh[1532]: 2015-03-24T12:47:00Z c1f498dd8212 confd[3664]: ERROR Config check failed: exit status 1
Mar 24 12:47:00 coreos-3 sh[1532]: builder: waiting for confd to write initial templates...

Since you rebooted all three nodes at once, you've lost quorum for etcd. etcd has a fault tolerance of < half of the total nodes: https://coreos.com/docs/cluster-management/scaling/etcd-optimal-cluster-size/#odd-active-cluster-size
https://github.com/deis/deis/issues/2410#issuecomment-62041804

Thanks for the reply.
A small note for those faced with the same problem:
If nodes shuts down due to any reason and you want to start the Deis, all nodes are required to give these commands(deisctl stop platform && deisctl start platform) after nodes started.


DoS attack caused by segmentation fault

Description
Denial-of-Service (DoS) Vulnerability of taurusxin ncmdump v1.3.2 allow a remote attacker to break ncmdump online service via the crafted .ncm files.
Attack vectors provided in poc will trigger segmentation fault caused by illegal read access. The crash point is at ncmdump/src/ncmcrypt.cpp:251, a function call to NeteaseCrypt::FixMetadata(). Coredump file is provided in ./coredumps.
See also https://github.com/Helson-S/FuzzyTesting/blob/master/ncmdump/dos_FixMetadata/dos_FixMetadata.md
Reproduction
Run the following command:
./bin/ncmdump ./poc/I1DWE0~U; cp ./poc/origin/I1DWE0~U ./poc/I1DWE0~U

Postscript: To successfully reproduce it, one need to run the command for several times.
Screen-shot
Crash:

Debug coredump:

Any codes to fix it?
Fixing metadata need taglib, seems the bug is caused by taglib.


[Feature Request] Add highlights for Urls

It would be nice if urlview.nvim could add highlights for the URLs if it is possible. I would like to have URLs underlined (& maybe have blue fg) so that they're readily visible.

Hi, this is most likely out of scope for this plugin, since writing a URL parser would most likely be inefficient (especially given the current Lua string parsing design) unless nvim-treesitter is used (only checking URLs in strings or comments).

I came across a plugin that uses a cursed regex to do this, but I don't want to use that for performance reasons. I've not looked into the code of this plugin, but are the URLs of buffer pre-computer, or does it search for the next URL when the command is called? I was imagining that we could scan the buffer for URLs and then add highlights?
I don't think it is in scope for treesitter, because it just parses comments as comments, and doesn't look into them.

Oh yes that regex parser indeed looks very scary. This plugin finds URL buffers when the user calls the command. Applying highlights to the entire buffer each time doesn't seem like a very efficient way of doing so, which is why I was considering using treesitter to limit the scope and only change affected nodes.

I don't think it is in scope for treesitter, because it just parses comments as comments, and doesn't look into them.

I thought it was possible to get the node content using ts query (especially using @spell defined by highlight.scm

Update: I've managed to apply highlights to comments and strings using the @spell annotation in #52. Will need to have a look later at attaching the parser to a buffer and using LanguageTree:register_cbs with on_changedtree to update changes more efficiently, like folke/todo-comments.nvim or p00f/nvim-ts-rainbow.

Amazing. I'll start testing this then. Also, you might want to refer to the fork of nvim-ts-rainbow: https://github.com/HiPhish/nvim-ts-rainbow2.

This is working perfectly for me:
return {
  "axieax/urlview.nvim",
  branch = "url-parser",
  init = function()
    vim.api.nvim_create_autocmd("BufEnter", {
      desc = "HighlightUrls",
      callback = function() require("urlview.parser").parse() end,
    })
  end,
  event = "BufEnter",
  opts = {},
}

Glad to hear that it's applying the right highlights! However, the main issue with only parsing once is that changes to nodes don't reapply the highlights, causing incorrect highlighting, as can be seen in the clip below:
https://github.com/axieax/urlview.nvim/assets/62098008/e8ed5606-da26-44f3-ab05-cfd532bd6575

Not able to see the view due to "No video with supported format and MIME type found". Try sharing it on asciinema or as a GIF?
Also, I understand what you're saying. If I add new URLs, then they won't get highlighted which is fine. I just want existing URLs in the file to be visible, I'm unlikely to add any myself. AFAI can tell, existing highlights don't get messed up by edits.
You could also add a parser_event option which will be used, which can also be set to nil to disable the parser.

Yeah, edge edits were an issue. But as I said, I rarely add/modify URLs, so I'm fine with the tradeoff. With a config option, the user can choose something like BufWritePost.

Cool, in that case, please continue using this branch for this feature. I'll merge the PR when the updating functionality is completed.

Seems like the treesitter parser for comments have recently added native support for highlighting uris as well. Would this be enough for your use case? If so, I will close the PR.

Seems like the treesitter parser for comments have recently added native support for highlighting uris as well. Would this be enough for your use case? If so, I will close the PR.

I'm not sure how to enable that.

Can you try running :TSInstall comment or :TSUpdate comment? You need the comment treesitter parser which automatically highlights all uris.

Can you try running :TSInstall comment or :TSUpdate comment? You need the comment treesitter parser which automatically highlights all uris.

Works. Thanks!


unpin eccodes again

[x] Closes #4521
[x] Passes isort . && black . && mypy . && flake8

That was fast - eccodes already fixed the issue.

yup, thats #4516

Thanks @mathause


CAMEL-20543 - Camel-AWS-Bedrock-Agent: Support more operations on the producer side - ListIngestionJobs Operation

Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test aws


Ice

Core Section

[X] I have searched for this issue on this repository
[X] This bug persist and I need help

Environment
Singleplayer/Local | NO LAN
Modpack Version
Ironberry Expert 0.8b
Description
I can't seem to find a way to make ice.  I would imagine that the Entropy Manipulator ought to be able to make ice but it doesn't seem to do it.
Procedure
I have charged up the Entropy Manipulator, and from what I can tell from various Youtube videos, when you shift right click a water source, it ought freeze the water, at which point you can break the ice into shards, then combine the shards to make ice.  However this does not seem to be working.  Is there an alternate way of making Ice that I'm missing?
Additional things
N/A

Unfortunately I can't find a way to make normal ice, other than platforming out like crazy and hoping to find a chilly biome where water naturally freezes.

Unfortunately I can't find a way to make normal ice, other than platforming out like crazy and hoping to find a chilly biome where water naturally freezes.

mh.. strange , maybe was disabled by other parts of temp-scripts that i didn't know , i will investigate

uploaded a 0.8d where all recipes are comeback but age 5 still not completed


[FLINK-26737] Add CRD management in development doc

Add CRD management in development doc

@mbalassi Thanks for your comments, I have fixed it. Please take a look again.


Rename Suite.010 and Suite.011 in CDR Test Documentation

Description
CDR Test Documentation
Related Work Items
In source file rename Suites 10 and 11 as follows:

Suite.010 Energy Standing Data, Usage and DER
Suite.011 Energy Standing Data, Usage and DER SR

Renaming these suites conforms to the naming convention in place for other Suites and aligns with the changes to the standards in v1.16.1 where spec were created for the SR endpoints and were appended with SR

Suites were renamed before the documentation was published


z-index option support

I have some other modal elements which I'm trying to use with prelodr and it becomes a problem when some modal elements have a higher z-index then prelodr does.
It looks like you've hard coded the z-index into the main css file, it would be good if you provided an option to adjust the z-index for prelodr.
Thanks.

Is not sufficient? https://github.com/joseluisq/prelodr/blob/master/scss/prelodr.scss#L12
It appears as you want to set different z-index values for each preloader. In this case, I will consider as a new option to implement in near future. :+1:


Add enable_orientation SDF element to imu

Signed-off-by: Ian Chen<EMAIL_ADDRESS>🦟 Bug fix
Summary
Add enable_orientation element to SDF spec that allows users to configure whether an IMU sensor will generate and output orientation data
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

the code looks fine; we just have to be careful when adding fields to the spec, both when merging forward, and in terms of back porting to libsdformat6 and libsdformat9
do you have a plan for forward and back-porting?

do you have a plan for forward and back-porting?

We currently don't need this in earlier versions but I can backport it if we want to keep the spec consistent?
What's the process for merging forward? Should I create separate PRs to add the field to the 1.8 (sdf11) and 1.9 (sdf12) specs?

do you have a plan for forward and back-porting?

We currently don't need this in earlier versions but I can backport it if we want to keep the spec consistent?

this would be easier if we kept the spec and parser in separate repositories. we'd probably need to add a patch number to the spec to handle new fields (i.e. 1.6.1). Until then, let's try to keep them consistent

What's the process for merging forward? Should I create separate PRs to add the field to the 1.8 (sdf11) and 1.9 (sdf12) specs?

you can copy them over to the new specs in the merge-forward pull request

sounds good, I merge this first and then start porting to other branches.


Feat: Create CRDs for Diago Workers

In order to decouple configurations of our workers from our storage layer such that configuration can stick around even when Diago dies, we decided to try it moving all configuration to a CRD.
This lets us codify Worker configurations separately and a developer can manually specify their K8s configurations for their workers (and eventually tests). All Diago has to do is to fetch the worker configuration every time it needs to run a test.

Codecov Report

Merging #16 (13c0062) into dev (e674f7e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #16   +/-   ##
=======================================
  Coverage   46.94%   46.94%
=======================================
  Files           4        4
  Lines         262      262
=======================================
  Hits          123      123
  Misses        136      136
  Partials        3        3

Continue to review full report at Codecov.

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

Codecov Report

Merging #16 (13c0062) into dev (e674f7e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #16   +/-   ##
=======================================
  Coverage   46.94%   46.94%
=======================================
  Files           4        4
  Lines         262      262
=======================================
  Hits          123      123
  Misses        136      136
  Partials        3        3

Continue to review full report at Codecov.

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


[YUNIKORN-2705] Fix DOAP syntax

What is this PR for?
I'm having some trouble finding an authoritative reference, but it looks like a release (which is a property) may not contain multiple Versions (which is a node) in RDF/XML. The python rdflib at least doesn't support it, and whatever generates https://projects.apache.org/json/projects/yunikorn.json and https://projects.apache.org/project.html?yunikorn also doesn't seem to process it correctly.
What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[x] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?

Open an issue on Jira https://issues.apache.org/jira/browse/YUNIKORN/
Put link here, and add [YUNIKORN-Jira number] in PR title, eg. [YUNIKORN-2] Gang scheduling interface parameters

How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Hi @raboof,
Could you create a Jira and change this PR title to the corresponding Jira name?
We're using Jira the track every changes in YuniKorn.

https://yunikorn.apache.org/community/how_to_contribute

Hi @raboof,
Could you create a Jira and change this PR title to the corresponding Jira name? We're using Jira to track every changes in YuniKorn.
* https://yunikorn.apache.org/community/how_to_contribute

done!


Add route for retrieving a version

@cmh2166 the reason I did this PR was because @aaron-collier said on #373:

The reason for this is that swagger doesn't support optional path parameters or duplicate paths. So we can't specify GET /resource/{ID} AND GET /resource/{ID}/{version}.

So I thought this would fix that problem.  If it was intentionally designed for it to be a query parameter, then I'm fine with taking that approach too.


spam report again

url: https://github.com/kubeflow/examples/issues/1158
@thesuperzapper

I have closed them.
Also, when I talked to GitHub, they say that @ghost is not a real account and is what happens when someone deletes their account, so that's annoying.
/close


add translation file for arabic language support

Thanks for this translation, @SafwanLjd! I'll merge this in now and it will be included in the next theme release.


Remove id param from logout endpoint

Rationale for why this is necessary
The logout endpoint takes the userId in the url param. Meaning that you can pass any known ID to log out a user. Even though we have the id in an encoded form, we should remove the id param.
Work:
Use /logout without the id param. Find alternative to retrieve the current user and the issue to logout api call.

The nextjs express server should be able to retrieve necessary information from the current session.
It could pass an auth token as header to the backend. The backend should be able to use hydra introspection to retrieve the belonging Reacton user in the same manner other graphql calls are validated and then process the meteor logout.

resolved via #637


Incompatibility With RecipeBook

When Using This mod and the Recipe Book mod at the same time, the recipe book menu cannot be brought up.
Log of session where I attempted to open recipe book:
factorio-current.log

Actually, this might be incorrect.
turns out this was a different mod. Ignore this.

Nevermind, it does seem to be this mod.

To reproduce the issue:

Load a world with Recipe book and this mod enabled.
open the RPG menu at least once
attempt to open the recipe book.

This is now fixed in 1.0.16.
Do note that opening RPG settings (cogwheel)

Still closes the RecipeBook frame, since RPG utilizes player.opened and so does RecipeBook.
Thank you! 👍


Dual control plane upgrade 1.7 -> 1.8 test failure

https://prow.istio.io/job-history/gs/istio-prow/logs/istio-dual-control-plane-upgrade-kind-1.7-1.8
https://prow.istio.io/view/gs/istio-prow/logs/istio-dual-control-plane-upgrade-kind-1.7-1.8/1339057702050992128
https://prow.istio.io/view/gs/istio-prow/logs/istio-dual-control-plane-upgrade-kind-1.7-1.8/1339420092051492864
This seems like an issue with the underlying test infrastructure
Creating cluster "istio-upgrade-testing" ...
 • Ensuring node image (kindest/node:v1.19.1) 🖼  ...
 ✓ Ensuring node image (kindest/node:v1.19.1) 🖼
 • Preparing nodes 📦   ...
 ✗ Preparing nodes 📦
ERROR: failed to create cluster: docker run error: command "docker run --hostname istio-upgrade-testing-control-plane --name istio-upgrade-testing-control-plane --label io.x-k8s.kind.role=control-plane --privileged --security-opt seccomp=unconfined --security-opt apparmor=unconfined --tmpfs /tmp --tmpfs /run --volume /var --volume /lib/modules:/lib/modules:ro --detach --tty --label io.x-k8s.kind.cluster=istio-upgrade-testing --net kind --restart=on-failure:1 --publish=<IP_ADDRESS>:36201:6443/TCP kindest/node:v1.19.1@sha256:98cf5288864662e37115e362b23e4369c8c4a408f99cbc06e58ac30ddc721600" failed with error: exit status 125

Command Output: docker: Error response from daemon: error creating aufs mount to /var/lib/docker/aufs/mnt
/8af43ca302fb33afc5d5bbd5d0930fac45e1aae4e64864096c4348835d330986-init: mount target=/var/lib/docker/aufs/mnt
/8af43ca302fb33afc5d5bbd5d0930fac45e1aae4e64864096c4348835d330986-init data=br:/var/lib/docker/aufs/diff
/8af43ca302fb33afc5d5bbd5d0930fac45e1aae4e64864096c4348835d330986-init=rw:/var/lib/docker/aufs/diff
/c627ce7f79b5d6d07cd6c4982003316ce199ae819fa838a9bd3df0dbc388674e=ro+wh:/var/lib/docker/aufs/diff
/5a8f909a198eb533a11e1047a977bea9cc4ad273a4535e8eca3e3bdc4c4f0ef1=ro+wh:/var/lib/docker/aufs/diff
/43aba0ffbeca112d3c0b52e51078d7b815fc315c734528a8f8e6450617bcad49=ro+wh:/var/lib/docker/aufs/diff
/a4868b1b2628213e6503d31a75f531e0ffbfc97acffcfe7c45e0fbc529b1b02f=ro+wh:/var/lib/docker/aufs/diff
/e7979157e253953bad2fed9356a9b94f1dd5ff300006c10f3b0e1d31c2c78a34=ro+wh:/var/lib/docker/aufs/diff
/f55f6d5a5466cbe41ff3fb037c4d58e99200a6b6778669d1486d74087f639551=ro+wh:/var/lib/docker/aufs/diff
/d75e1e078993a2d7301a575f6a6af898fe2db1635c1bc31626f5c5c266ba6147=ro+wh,dio,xino=/dev/shm/aufs.xino: invalid
argument.

(Added newlines for readability)
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[X] Developer Infrastructure
[X] Upgrade

It is strange that other tests are not failing (at least not due to this reason if they do). It seems like there is a new job added in this PR  - https://github.com/istio/test-infra/pull/3052/files . @Monkeyanator - could you please take a look?

Yes, taking a look now. Seems like the test has been failing since merge


[Docker Open Source] Deprecate Docker Toolbox

Tell us about your request
Docker Toolbox https://github.com/docker/toolbox was an early solution for users to work with Docker on Windows and Mac providing the bundling of a selection of tools to enable this. This was superseded by Docker Desktop, but initially lacked support for Windows Home. With WSL 2's release on Windows Home Docker Desktop is now supported on Windows Home - our main user of Docker Toolbox.
We have not been maintaining this actively for many years, and would like to formally deprecate it.
If you are an active user of Docker Toolbox please let is know here what your use case is.
Which service(s) is this request for?
Docker Toolbox
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Make it clearer which open source tools we are developing actively, focus on tools that are being used, have active communities and reduce duplication of tools.
Are you currently working around the issue?
We are not maintaining this actively, but have not made it clear what the status and roadmap is.
Additional context
https://www.docker.com/blog/docker-desktop-for-windows-home-is-here/

I have several college students that used Docker Toolbox this past semester because they were on Home edition and not the latest version and it was their only option. Seeing that WSL2 is available for all with the latest version, it makes sense to deprecate.
I'd recommend not pulling releases (making it disappear), as there are always going to be folks on older versions of Windows that can't upgrade for various reasons. But, I think it's fine to run deprecated software in those cases.

AMD hackintosh users are relying Docker Toolbox on Mac OS to spin up docker containers. It would be great to see Docker Desktop can support AMD processors on Mac OS one day.
Sample workflow for now.
https://github.com/sergeycherepanov/homebrew-docker-virtualbox

We have been using Virtualbox to deliver virtual machines to distance learning students for several years, and are looking to move to Docker from October 2020. Our students are required to provide their own computers to a minimum specification, but the bar is set low: computers that are several years old and running what we might class as "legacy operating systems". I would expect that a not insignificant proportion of out students will be running old and basic versions of Windows without WSL 2 support. For those students, running the Docker Toolbox is the only solution available to many of them.

@psychemedia I would love to have a chat about your students, if you would be interested drop me an email on<EMAIL_ADDRESS>:)

There is no WSL support for Windows 8.1 which is still supported by Microsoft  until Jan 2023.
Will there be any supported way to run Docker on Windows 8.1 in 2021?

My company makes active use of Docker Toolbox in Mac desktops, we have modern desktops but most of us, me included, prefer to use the toolbox as it provides minimal tooling (e.g. docker-machine, docker). I don't make use of the UI and prefer not to download an additional application. I would love there was some way to install only the runtime without downloading an app.

My company uses VirtualBox for development and testing, and it appears there's still no good way to have both VirtualBox and Docker Desktop:

Enabling Hyper-V breaks old versions of VirtualBox, and reduces performance and stability of new versions, so is not a reasonable option at this time
It looks like there's a "Docker for Windows Home" advertised (unclear if it would work on Windows 10 Pro w/o Hyper-V?), but I can't actually find it, the docs link to https://hub.docker.com/editions/community/docker-ce-desktop-windows/ which says "Requires Microsoft Windows 10 Professional or Enterprise 64-bit. For previous versions get Docker Toolbox." and just errors with "requires Hyper-V enabled" when ran.

That being said, the docs also say it requires "Windows version 2004" which we have not yet rolled out, and will not for quite some time, the Docker docs claim "Docker supports Docker Desktop on Windows based on Microsoft’s support lifecycle for Windows 10 operating system" and the lifecycle for Windows 10 lists Windows version 1909 as valid until May 2021 (or 2022 for Enterprise) - I realize it's not claiming all Docker tools follow the Windows 10 lifecycle, but it would be nice to have a supported option.

@Jimbly sorry we haven't updated the Hub page yet, the docs are correct. And with MSFTs recent backport of WSL 2 Docker Desktop is supported through WSL2 on Windows home 1903 and up.
https://docs.docker.com/docker-for-windows/install-windows-home/
Thanks!

So here we use VirtualBox, and sometimes other virtualization solutions, but not Hyper-V, as it is less supported and VMs are slower under it. Until now Docker Toolbox integrated nicely with our workflow. If it is deprecated, we will just drop running Docker on Windows, and use a Linux VM in VirtualBox.
Sad to hear that maintaining integration with VirtualBox in addition to Hyper-V is such big effort, that it must be avoided. I wonder what is in it that requires specific work, except of course for calling VirtualBox to create a VM and expose a port.
Could you at least provide Windows binaries for docker client only?

@whut yes we are looking at what we do for Windows binaries and will follow up with something in this space.
Closing as we have now marked the repo as deprecated. This does not stop people using what we had before but is to be clear that we will no longer be fixing/providing updates to Docker Toolbox.

Hi nebuk89,
thank you for docker toolbox that I am currently still using on Windows 10 Home. I'm considering installing Docker desktop, however I'm not sure if I necessarily need to uninstall docker toolbox before doing so.
Hence my quick questions:

Can docker desktop be installed alongside docker toolbox?
Does docker toolbox need to be uninstalled first?

Thank you very much in advance for your time and advice!

@Gettlar
You do not need to uninstall Docker Toolbox to install Docker Desktop.  However, at the time of deprecating Docker Toolbox, the version of VirtualBox that it used only worked with Hyper-V disabled, and Docker Desktop only works with Hyper-V enabled, so you needed to enable/disable that and restart Windows to switch between them.  AFAIK, the newer versions of VirtualBox now work with Hyper-V enabled, so you could theoretically switch between Desktop and Toolbox (assuming it works with that VirtualBox versions).  It might be useful to keep both installed for troubleshooting, as not all Docker images work / work the same on Desktop as they did on Toolbox.

@Jimbly
Thank you for your quick response! I will try it out. !
Have a nice day and stay well !

Just as a new user of Docker and a Linux user, how can I alternatively use Linux kernel on Windows image or Windows-based image alternatively to Docker Toolbox? I would like to test my containers on different OS. I believe that was one of the use cases of Docker Toolbox. Thanks in advanced.


Using v-for on an object with key/value, the key is typed as number

Vue version
3.3.4
Link to minimal reproduction
https://play.vuejs.org/#eNp9UmFLwzAQ/StHELZB3XD7VraBygT94ET9ZoTV7la7pUlI0jop/e9eUrdVkPVDyb17effetTW71npYlchiNrWpybUDi67UIBKZzThzlrM5l6mS1oFD65YfW0xdDDW87WKwzuQyez8coIEZ1FwC7PD7KoZelYgSx73oFxqfIC4bLrvCd6VMXa4kKfQTk9HtVjMCquhiWw1gNm8nGLJpJKwuas9uYgiHcbMKytNRm4a8U+Gw0CJxSBXAdJ1XUF1ulKF8/WAn8uYGkMtORM4gJpQ49A5LAHrq+o/ZPvUiCBoDaGguyY9In9jTUWcqi2iTlHWTZ8OtVZLWHUJwlqpC5wLNUns92jatth3FWSKE+noImDPk8oCnn5ju/sG3du8xzp4MWjQVcnbsOVoOUijfXrw84p7Ox2ah1qUg9pnmM1olSu+xpd2Uck22O7zg9r7Qyjj6UK92sXco7SGUN+qZTeBzRv/c7ZnoJ7uT4STco8/Kmh/qs+TD
Steps to reproduce
Use v-for on an object with value and key as iterators, like in https://vuejs.org/guide/essentials/list.html#v-for-with-an-object
Then check the type of the key variable (number)
What is expected?
The key type should be string
What is actually happening?
The key type is number
System Info
No response
Any additional comments?
My guess is that it is typed as a number because v-for is usually used on an array, and in that case the second argument is the index. But v-for also supports iterating on an object and in that case it will be a string.

Use Record<string, string>.

Thanks for your input, it seems like the types for v-for are declared here:
https://github.com/vuejs/core/blob/3.2/packages/runtime-core/src/helpers/renderList.ts#L41
And we can see that the second argument key gets the type K which is K extends keyof T (with T the type of the v-for source).
So in my case keyof { [key: string]: string } evaluates to string | number.
This was introduced with Typescript 2.9: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-9.html#support-number-and-symbol-named-properties-with-keyof-and-mapped-types
The use of keyof seems logical, however in practice using v-for on an object will always provide the key as a string, so maybe Vue should directly type the key as string?


Use new XML parsing library: lxml instead of libxml2

replace libxml2 with lxml: more up-to-date, less problems to install across different platforms
add tests using mocks
use secure environment variabels for travis-ci

:+1:


Improve loading in MSGraphv2

see comments here "#134" on msgraph-sdk-powershell's issues.
Need to load in only specific modules.

Fixed by Todd


Roaring Serialization broken for containers greater than Short.MAX_VALUE

ContainerMeta's bit_count field is a short. This means the cast here can result in negative values if the container is over half full. This should likely just be an int, keeping the (short) cast here for serializing ContainerMeta.

Thanks! I haven't run a benchmark yet, but this fix will likely improve the performance of the java client with roaring imports, since the overflow caused it to choose the wrong container type to serialize.


feature(sinsp-example): print info about all threads

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area libsinsp
/area tests

What this PR does / why we need it:
A flag has been added to allow sinsp-example to print information about all threads in process and not just the main one. This may cause too much data to be print for a regular user, but it is useful when creating automated tests for it. In particular the event-generator uses threads for triggering events, so filtering by main thread only makes both these binaries incompatible with each other.
sinsp-example now also handles SIGTERM, this is done to simplify stopping it when running it in a container.
The PR is somewhat related to #271 and this PoC repo
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

Hi @Molter73. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Molter73
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I marked this as a feature just because I don't really know what else I could've mark it as 😝

/ok-to-test

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: dc181a6484d826020caa1169d1ca26bcbb830d5b


Atlassian Sydney reportedly all WFH enforced

.. and we’re closed with immediate effect in Sydney.  Literally a “please leave now”.

Here's an update for the same.
https://twitter.com/Atlassian/status/1237996563953324032


[729] Get new DRLS branch working with Logica Health - Connectathon

Change scripts for startup.

Looks good. Was able to test functionality locally by doing npm run start. Running npm run startDev confusingly starts up with the proxy configured for the logicahealth.

Pat, was "proxy configured for the logicahealth" one of the changes you needed to make?

Looks good. Was able to test functionality locally by doing npm run start. Running npm run startDev confusingly starts up with the proxy configured for the logicahealth.

Pat, was "proxy configured for the logicahealth" one of the changes you needed to make?

Hmm, startDev is assuming that the VERSION environment variable is set, I'll fix that.
I had to configure some things to work with logicahealth, I was trying to streamline the configuration to make it less confusing.


Updating ssh unit test to really check for SSH in the payload

Can one of the admins verify this patch?


invite_users: Remove "#" in private streams and add globe icon in Web-public streams

Fixes: #23496

Screenshots:
Before -

After -

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@SameepAher Please make sure to end all lines / paragraphs in your commit messages with a period (.). Then it should detect the commits as being linked to the issue mentioned in their last line too

@SameepAher Please make sure to end all lines / paragraphs in your commit messages with a period (.). Then it should detect the commits as being linked to the issue mentioned in their last line too

@N-Shar-ma Thanks for clarifying. I have updated the commit messages as per the conventions.

@alya this PR is ready for your review.

@ganpa3 Is this PR ready from your perspective? I'd love to get this fixed.

@ganpa3 Is this PR ready from your perspective? I'd love to get this fixed.

LGTM.

Looks good to me in manual testing!

@timabbott This PR has been reviewed by @ganpa3 and myself -- please take a look.
Note that the commit message will need to be modified so that https://github.com/zulip/zulip/issues/23496 remains open, as only the first point in the issue description has been addressed.

Here's the czo thread link for in-depth discussion on second point of the issue.

I squashed the last two commits (since the middle commit introduced a bug fixed by the third commit) and then adding a final commit to complete the issue, since it was simple and seemed unrelated to the actual discussion in the chat.zulip.org thread.
There's probably more cleanup we can do here, and I encourage continued discussion on that thread, but I didn't want this bug fix to be blocked indefinitely.


Various gas fixes

Various fixes related to how we were managing the gas consumption:

inexplicit_panic and panic_from_far_call count as instructions and we should be decreasing the gas base cost for running them as well.
When pushing a new far_call frame if the bytecode was for the evm interpreter, we were returning back not only the left gas but also the stipend one.

I have some questions regarding these fixes. Are we fixing some tests with these changes? If not, then how are we sure they are really necessary?

The reasons seem quite clear. When comparing the gas_left field during the execution of the interpreter tests, it was clear that the ret operation was adding a lot more more gas to the new frame due to the stipend. Even though this issue might not cause any test failures right now, given that the issue is pretty evident, I would not wait until for a potential future test failure, which might never occur.

A good way of testing this would be using the benchmark-analyzer of the compiler tester with the interpreter tests
The idea would be the following:
Run:
cargo run --verbose --features lambda_vm --release --bin compiler-tester -- --path tests/solidity/complex/interpreter/test.json --mode 'Y+M3B3 0.8.25' --benchmark lambda_vm.json
Then run:
cargo run --verbose --release --bin compiler-tester -- --path tests/solidity/complex/interpreter/test.json --mode 'Y+M3B3 0.8.25' --benchmark vm1.json
This will generate 2 json files with data for each interpreter test, we care about ergs, running:
cargo run --release --bin benchmark-analyzer -- --reference vm1.json --candidate lambda_vm.json
Will output a table like this
╔═╡ Size (-%) ╞═════╡ EVMInterpreter M3B3 ╞═╗
║               Mean                                                   0.000 ║
║               Best                                                     0.000 ║
║              Worst                                                   0.000 ║
║                Total                                                  0.000 ║
╠═╡ Cycles (-%) ╞═══╡ EVMInterpreter M3B3 ╞═╣
║                Mean                                                4.834 ║
║                Best                                                   7.022 ║
║               Worst                                                 -1.611 ║
║                Total                                                  5.573 ║
╠═╡ Ergs (-%) ╞═════╡ EVMInterpreter M3B3 ╞═╣
║               Mean                                                  -3.166 ║
║                Best                                                    0.000 ║
║               Worst                                                 -6.275 ║
║               Total                                                   -3.573 ║
╠═╡ Ergs/gas ╞══════╡ EVMInterpreter M3B3 ╞═╣
║              ADD                                                     67.667 ║
║               MUL                                                   39.400 ║
║              SUB                                                     67.667 ║
.
.
.
║            SWAP16                                            66.000 ║
║            RETURN                                                    inf ║
║             REVERT                                                   inf ║
╠═╡ Ergs/gas (-%) ╞═╡ EVMInterpreter M3B3 ╞═╣
║                 ADD                                              29.021 ║
║                 MUL                                               31.119 ║
║                  SUB                                              29.021 ║
.
.
.
║                 SWAP16                                       38.700 ║
╚════════════════════════════════╝
We care about ergs/gas which indicates the relation between the ergs spent by the eravm and the gas spent by the interpreter, since the gas should be the same in both, this will tell us if the ergs spent is correct, if all opcodes have ergs/gas equal 0 then we are calculating ergs correctly
Have in mind that we may have to change some things in compiler tester to let it know the ergs we spent


Can't set breakpoints when deploying on HoloLens

Overview
When I set a breakpoint in the code and I remote deploy on the HoloLens, the breakpoints get disabled. This worked some time ago, maybe it broke after I updated to Unity 2017.2 or to Windows SDK 10.0.16299.0.
Build is set to Debug and x86 and Windows SDK 10.0.16299.0.
Maybe related to #1398
Expected Behavior
Breakpoints should work. The code should stop and be debugabble.
Actual Behavior
Breakpoints don't get hit. They are disabled when debugging.
Unity Editor Version
2017.2.0.p1-MRTP4
Mixed Reality Toolkit Release Version
Probably around 22ade4fa7a1ab4082ab1dd7f5e01b55dca83a5a7

@StephenHodgson could you reproduce this bug?

No I cannot

It's weird. I can't reproduce it myself now...

I Have actually the same problem.
I was debugging a Visual Studio project built from Unity, deployed on HoloLens.
Now, with Unity 2017.2, SDK 10.0.16299, and Visual Studio 2017, breakpoints does not work anymore at all.
I completely reinstalled a fresh Windows 10 pro with version 1709 and it doesn't change, breakpoints just won't work, there is then clearly something wrong either from Unity or Visual or SDK which prevent the app from breaking correctly in Visual Studio with this configuration at the moment (symbols are loaded for Assembly-CSharp).
I didn't find out anything relevant on internet nor myself about why it fails to break but if it doesn't fail at all to break for you guys i would love to know how!

I'm having similar issues but the behavior is inconsistent.
Breakpoints that are set on Unity functions like Start and Awake are not hit and when running the breakpoint is deleted. Breakpoints on my functions are hit (though the breakpoint is moved to the opening line of the function), but only in the first scene in a multi-scene project.

We got the same problem. We added the last version of the Mixed Reality Toolkit (2017.2.0.p1-MRTP4) to our project and for some reason we were not able to debug anymore.
the last day i have been trying out different things and found out the following:
Tools
Visual Studio 2017 version 15.4.5
Unity 2017.2 or Unity 2017.3 beta
.NET SDK 10.0.166299.0
Windows 10 with minimum FCU
Created an Unity project with gameobject, script and some simple code
Build VS solution and did debug (debug, x85, remote machine) to HoloLens
No problems. All breakpoints work
Added the Mixed Reality Toolkit as package to Unity solution
Build VS solution and did debug (debug, x85, remote machine) to HoloLens
None of the breakpoints worked anymore.
Removed from the assets folder both folders regarding Sharing (Sharing and SharingWithUNET)
Build VS solution and did debug (debug, x85, remote machine) to HoloLens
No problems. All breakpoints work
Adding the folders back will cause again problems with breakpoints. I expect that there is something in the folders causing problems with the PDB files generated and code debugging in VS.
Hopefully this could help investigate further the solution


bug: ios crash on ionic deploy update in airplane mode

Type: bug
Platform: ios 7 webview
xcode dies on line 61 of ionicDeploy.m
scenario:
I check for an update on app resume. I added a try/catch but this goes beyond the js level. This works fine in android, only iOS crashes out. I'd submit a fix but I don't know where ionicDeploy.m is coming from :/

Should we be submitting issues for deploy here? As a workaround you could use the Cordova network status plugin to check if you have connection before calling the update method

Greetings @jacksonkr!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


build(deps): bump sentry-cli from 2.31.2 to 2.32.1

This PR resolves []
Description
This PR updates sentry CLI from 2.31.2 to 2.32.1

PR Checklist

[X] I have tested my changes on iOS and Android.
[X] I hid my changes behind a feature flag, or they don't need one.
[X] I have included screenshots or videos, or I have not changed the UI.
[X] I have added tests, or my changes don't require any.
[X] I added an app state migration, or my changes do not require one.
[X] I have documented any follow-up work that this PR will require, or it does not require any.
[X] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

bump sentry-cli from 2.31.2 to 2.32.1 - mrsltun

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (bump sentry-cli from 2.31.2 to 2.32.1 - mrsltun)

  Generated by :no_entry_sign: dangerJS against 40afc8a4e94e8ccfdb16adf2c486defbe3a06ee1


Block Websites only in in incognito mode

Hi,
Would it possible to block certain websites only in the incognito mode?

I think this would be great for my use case. On the other hand, this might be unnecessary work for you and may not be useful for other people.
Perhaps allowing the context to include the time in which it is blocked would allow you to shoot two birds with one stone.

It woudn't be an issue to add this feature.

I'd like to suggest an alternative method of handling this, rule blocks.
Rule blocks are blocks of block rule text(tongue twister?) which you can set rule context on and are applied in cascade.
rule contexts could be:

allow only in incognito(I might want to keep my history clean and be sure it's impossible to make a mistake)
don't allow in incognito
Schedule enable and disable(enable daily from 14 to 19/enable now for 2 hours)
Counter enabler(enable rule if sites from the rule set are opened X times)
Allow time enabler(Allow access for 2 hours daily)
And more. It's possible to overlap(cascade) rules for more complex effects (rule set is enabled from 18 to 23 but you can only access for two hours)

blocks are blocks of block rule text

haha, really a tongue twister. It's tricky because, do we allow blocking or we block allowing? 😄
Thank you for the suggestions.
Having one question. Is there a case where we would like to allow page in incognito, but block in normal window? I'd say, we always block everywhere, and optionally allow in incognito. But honestly, if I block youtube, but can open it in incognito, doesn't it go against the purpose?
Limited time access (like 2 hours daily): We would have to track every tab when activated, check if there is page open that we plan to block, and start/continue tracking spent time. This is not something I'd plan to do.
Limited opened count (can open 10x a day let's say), or blocking during some days/hours, sounds ok.

As far as I can see the original request is still not possible. It's okay if it is not planned, but just wanted to point that out.


Implement Cardano chain service and balance service

#3862

Please check issues in https://www.notion.so/subwallet/Integration-Cardano-ecosystem-f545819c79ea4c7c8f7f18f0f20bf94e?pvs=4


music.163.com doesn't work

I played different tracks but the extension simply doesn't recognize it :(

What means "the extension simply doesn't recognize it"? Is a question mark icon appears at the extension toolbar?

It works fine now, thank you.

Closing as solved.


AccessibilityService in nativescript not working in Android

I've used android docs and a blog post as a reference to parse the USSD response. I've added following permissions in the manifest file on top,
<uses-permission android:name="android.permission.CALL_PHONE" />
<uses-permission android:name="android.permission.BIND_ACCESSIBILITY_SERVICE" />
<uses-permission android:name="android.permission.RETRIEVE_WINDOW_CONTENT" />

And  defined a service at the bottom,
<service android:name="com.tns.notifications.USSDService" android:enabled="true" android:exported="false" android:permission="android.permission.BIND_ACCESSIBILITY_SERVICE">
    <intent-filter>
        <action android:name="android.accessibilityservice.AccessibilityService" />
    </intent-filter>
</service>

As per the nativescript docs here, write a service in Javascript,
// app/notifications/USSDService.js
var USSD =  android.accessibilityservice.AccessibilityService.extend("com.tns.notifications.USSDService", {
  onAccessibilityEvent: function(AccessibilityEvent) {
    console.log(AccessibilityEvent.getEventType());
  },

  onServiceConnected: function() {
    this.super.onServiceConnected();
    console.log('hi')
  }
});
new USSD();

Did you verify this is a real problem by searching [Stack Overflow]
Yeah. Found http://stackoverflow.com/a/36178368/415057 a little helpful.
Tell us about the problem
With the all the setup above, I installed the app on the real android phone using livesync command and enabled the accessibility feature for the app from Android Settings -> Accessibility as well.
So when I dial the USSD code using the application I'm building with *111#, I get the accessibility popup showing bunch of options, however, the onServiceConnected or onAccessibilityEvent defined above did not trigger.
var intent = new android.content.Intent(android.content.Intent.ACTION_CALL);
intent.setData(android.net.Uri.parse("tel:" + '*111*3*1*1' + android.net.Uri.encode('#')));
application.android.foregroundActivity.startActivity(intent);

Please provide the following version numbers that your issue occurs with:

CLI: 2.3.0
Cross-platform modules: 2.3.0
Runtime(s): {
"version": "2.3.0"
}
Plugin(s): 2.3.0

Did the error happen while the app was being constructed? (buildtime error)
No build time or compile time error while running new USSD() above which means the code is correct.
Did the error happen while the app was executing? (runtime error)
No suspicious error in the logs with __enableVerboseLogging() enabled.
Please tell us how to recreate the issue in as much detail as possible.
Take a grocery app, update the AndroidManifest.xml file, use the above USSD service, and dial the USSD code on button click. I'm hoping to see onServiceConnected or onAccessibilityEvent  events invoked.

@Pip3r4o It seems that tns build android has resolved the issue. The onServiceConnected function is called as soon as the Accessibility is turned on from Android Settings.

@codef0rmer We advise against using livesync when writing extending classes/interfaces in NativeScript, as code needs to be rebuilt to have bindings generated for your updated code. Glad to hear that you figured it out.
Closing the issue as resolved. Ping me in Slack if there is anything else I can help you with.

@Pip3r4o It stopped working again. Last time when I informed, It usually logging the message as soon as accessibility is turned on for the app, but not now.
// app/notifications/USSDService.js
var USSD =  android.accessibilityservice.AccessibilityService.extend("com.tns.notifications.USSDService", {
  onAccessibilityEvent: function(AccessibilityEvent) {
    console.log(AccessibilityEvent.getEventType());
  },

  onServiceConnected: function() {
    this.super.onServiceConnected();
    console.log('hi')
  }
});
new USSD();

Might be useful for the lost soul. Updating Nativescript to the latest version and tns full build resolved the issue for me.
android.accessibilityservice.AccessibilityService.extend("com.tns.notifications.USSDService", {
  onAccessibilityEvent: function(event) {
    console.log(event.getText().toString());
  },

  onServiceConnected: function() {
    this.super.onServiceConnected();
    var AccessibilityServiceInfo = android.accessibilityservice.AccessibilityServiceInfo;
    var info = new AccessibilityServiceInfo();
    info.packageNames = ["org.nativescript.MyApp"];
    info.flags = AccessibilityServiceInfo.DEFAULT;
    info.eventTypes = android.view.accessibility.AccessibilityEvent.TYPE_WINDOW_STATE_CHANGED;
    info.feedbackType = AccessibilityServiceInfo.FEEDBACK_GENERIC;
    this.setServiceInfo(info);
  }
});

Thanks @Pip3r4o for your time and effect.

@Pip3r4o After updating tns-android to 2.4.1, the service again stopped registering. I no longer see the log added above.

@Pip3r4o Damn, I do not know why it's happening but cloning the repo in a different directory and reinstalling all over again resolved the problem. 🙏 :nativescript:

Has this issue been resolved? I have same problem.
I want to use nativescript...
An AccessibilityService is required.

@chohs928 the original problem was a user error, so technically dealing with a service should work, when dealt with properly.
Should you fail implementing the accessibility service in JavaScript, you can always fallback to Java.

@Pip3r4o Does replacing with java mean to use extend-existing-android-app?


Menu doesn't respect weigth parameter

Another thing I came across is: The menu points at the top of the page now do not represent the weighted order of the sections. Which seems a bit odd to me. I would expect they appear from top to bottom like they appear from left to right in the menu bar.
Originally posted by @timpauli in https://github.com/victoriadotdev/hugo-theme-introduction/pull/143#issuecomment-481880667

@snow2k9 Are you interested in fixing this?

Yep see #150


Cell separator # %% with comments after indentation does not render cell

Environment data

VS Code version: 1.54.3
Extension version (available under the Extensions sidebar): v2021.3.680753044
OS and version: MacOS 10.14.6
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.5 with Anaconda
Value of the python.languageServer setting: PyLance

Expected behaviour
Using # %% to create cells under a function should also render the cell when it is followed by additional comments.
Actual behaviour
When adding additional comments followed by the cell separator, the cell is not rendered.
Steps to reproduce
# Cell separator

# %%

# %% Has cell

def test_func():
    # %%
    print("This has cell")

    # %% This has no cell
    print("This has no cell")

Screen shot below:

Thanks for the bug. Yes it seems comments on cell delimiters is not supported inside of functions.

Thanks for the feedback! However, we don't have plans on adding this functionality at this time.


Added highlighting to string interpolation

Can you provide some screen captures, to show what this will look like to a user? I'm happy with how string interpolation is highlighted currently, I'd like to see what this looks like first. thanks!

Well it was not working in my case (ruby and elixir)
here is a screenshot:
before

after

it does look nice, interesting though that it still highlights on a syntax error.

I'll need to pull this down and test first, I'll report back after I have a chance to use it for it. If anyone else has any thoughts or feedback before I merge, please feel free to add comments

I think highlighting on syntax error is the right behavior.
Possible improvement is to render just the hello word in white such as variables, but did not get the chance to do it so far.

@AndreaCatalucci some changes have been made since this pr. I'm going to close for now, but I'd be happy to re-open if this is still desired and not working.


feat!: simplify identity hint determination

Since OpenSSL apparently only passes null-terminated strings as the identity hint to PSK callbacks, this PR simplifies the process of obtaining the identity hint and introduces a slight (but breaking) API change to the PskCredentialsCallback type definition.

Codecov Report
Base: 82.12% // Head: 82.72% // Increases project coverage by +0.60% :tada:

Coverage data is based on head (cd5e4c6) compared to base (e107dd8).
Patch coverage: 66.66% of modified lines in pull request are covered.

:exclamation: Current head cd5e4c6 differs from pull request most recent head e03e466. Consider uploading reports for the commit e03e466 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   82.12%   82.72%   +0.60%
==========================================
  Files          10       10
  Lines         660      654       -6
==========================================
- Hits          542      541       -1
+ Misses        118      113       -5

Impacted Files
Coverage Δ

lib/src/psk_credentials.dart
100.00% <ø> (ø)

lib/src/dtls_client.dart
77.66% <66.66%> (+1.80%)
:arrow_up:

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

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


PowerShell scripts doesn't work

Using PowerShell 7.1.0 the provided script doesn't work correctly.
I created a version which works for me. It can be found here: https://gist.github.com/eBerdnA/09bd45f21c7ce575679dad2d8113f842

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

ID: 9d7d5f36-11fc-9ac5-778b-c03823bbb072
Version Independent ID: 814e151a-2018-285a-697f-f929a73957d4
Content: Download tools from NuGet (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/customerengagement/on-premises/developer/download-tools-nuget.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

Related PR with code changes: #2132

v7 includes support outside of .NET. But all the tools that get downloaded are for .NET today. They should work for the version of PS that is available on Windows 10. If we optimize for v7, we are excluding the majority of people who will use the tools today.
All our tools use .NET Framework today, but this won't be true forever.
I think we need to optimize for the most common situation today. If there is an expectation that these should work with PS 7, we should add a note that this works for PS 5.1 since that is the most common case.
The script below works on my (desktop) environment using Windows PowerShell edition but writes a message when using PowerShell v7  edition.
if($PSVersionTable.PSEdition -ne 'Desktop')
{
    Write-Host "This script requires the Desktop edition of Windows PowerShell." -ForegroundColor Red
}
else
{

[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
$sourceNugetExe = "https://dist.nuget.org/win-x86-commandline/latest/nuget.exe"
$targetNugetExe = ".\nuget.exe"
Remove-Item .\Tools -Force -Recurse -ErrorAction Ignore
Invoke-WebRequest $sourceNugetExe -OutFile $targetNugetExe
Set-Alias nuget $targetNugetExe -Scope Global -Verbose

##
##Download Plugin Registration Tool
##
./nuget install Microsoft.CrmSdk.XrmTooling.PluginRegistrationTool -O .\Tools
md .\Tools\PluginRegistration
$prtFolder = Get-ChildItem ./Tools | Where-Object {$_.Name -match 'Microsoft.CrmSdk.XrmTooling.PluginRegistrationTool.'}
move .\Tools\$prtFolder\tools\*.* .\Tools\PluginRegistration
Remove-Item .\Tools\$prtFolder -Force -Recurse

##
##Download CoreTools
##
./nuget install  Microsoft.CrmSdk.CoreTools -O .\Tools
md .\Tools\CoreTools
$coreToolsFolder = Get-ChildItem ./Tools | Where-Object {$_.Name -match 'Microsoft.CrmSdk.CoreTools.'}
move .\Tools\$coreToolsFolder\content\bin\coretools\*.* .\Tools\CoreTools
Remove-Item .\Tools\$coreToolsFolder -Force -Recurse

##
##Download Configuration Migration
##
./nuget install  Microsoft.CrmSdk.XrmTooling.ConfigurationMigration.Wpf -O .\Tools
md .\Tools\ConfigurationMigration
$configMigFolder = Get-ChildItem ./Tools | Where-Object {$_.Name -match 'Microsoft.CrmSdk.XrmTooling.ConfigurationMigration.Wpf.'}
move .\Tools\$configMigFolder\tools\*.* .\Tools\ConfigurationMigration
Remove-Item .\Tools\$configMigFolder -Force -Recurse

##
##Download Package Deployer
##
./nuget install  Microsoft.CrmSdk.XrmTooling.PackageDeployment.WPF -O .\Tools
md .\Tools\PackageDeployment
$pdFolder = Get-ChildItem ./Tools | Where-Object {$_.Name -match 'Microsoft.CrmSdk.XrmTooling.PackageDeployment.Wpf.'}
move .\Tools\$pdFolder\tools\*.* .\Tools\PackageDeployment
Remove-Item .\Tools\$pdFolder -Force -Recurse

##
##Remove NuGet.exe
##
Remove-Item nuget.exe
}

By the time we have .NET Core support, I hope we have a better way for people to access tools than this script.

Couldn't this be solved by making a Note (similar to what can be seen on eg. https://docs.microsoft.com/en-us/powerapps/maker/portals/admin/migrate-portal-configuration) that users should make sure to use Windows Powershell, not Powershell? There's only a single mention of Windows Powershell anywhere on the page, the rest of the mentions are just Powershell. I suspect more and more people are installing v7.
JimDaly's script worked like a charm by the way; by creating a tools.ps1 file the output became that much nicer. See the gist: https://gist.github.com/alrekr/60015bcc79905eea962628330de8192e#file-tools-ps1

@eBerdnA  I added 'Windows' PowerShell in a few places for clarification. Thanks


(v1.0.1) Allow OpenJCEPlus test execution with Java 11 (#5164)

cherry-pick https://github.com/adoptium/aqa-tests/pull/5164
Java 11 now includes the OpenJCEPlus module and should be tested accordingly similar to other Java releases.

Will hold on to this PR until tonight's run passes.


[ENH] conditional_join improve

PR Description
Please describe the changes proposed in the pull request:

minor performance improvements for conditional_join

This PR improves conditional_join.

PR Checklist

Please ensure that you have done the following:

[ ] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

[ ] If you're not on the contributors list, add yourself to AUTHORS.rst.

[ ] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl

🚀 Deployed on https://deploy-preview-1038--pyjanitor.netlify.app

I think we're good to merge!


[Bug]: Fix login and get started button

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
As you click on the 'Get started' button the user is directed to create your account page.
Also when the user clicks on the 'login' button, it redirects to Sign in page, and in the same page if the user is already registered it redirects to sign-in page.
So there are two sign up pages

It would be better if we interlink the pages properly.
Also, it would avoid confusion among the users.
I would link this signup page when user clicks on get started and delete the old login page.

Add Screenshots
Uploading AmbuFlow - Google Chrome 2024-10-06 22-25-27.mp4…
What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

yes, signup page blue one, not greenone

yes, signup page blue one, not green one

This issue #288 has already been assigned to me, and it appears to be a duplicate.


Check pixel density

There’s no reason to render canvases at 2x on machines that are not doubly dense: https://developer.mozilla.org/en-US/docs/Web/API/Window/devicePixelRatio

Check the PIXEL_DENSITY constant in the Visualizer component.


Fix server authentication tab - haml - duplicate validate button on save

Just negated the condition for the disabled version of the button, introduced in 6f735742
https://bugzilla.redhat.com/show_bug.cgi?id=1203168

@dclarizio verified fix.


Error: This repository is over its data quota.

Not sure how to remedy this error

This repository is over its data quota.
Account responsible for LFS bandwidth should purchase more data packs to restore access.

Any idea what this means other than what it says? Any workaround?
$ make
make: [Makefile:21: clean] Error 1 (ignored)
STEP 1/9: FROM docker.io/fedora:35
STEP 2/9: RUN dnf update -y &&     dnf install -y unzip g++ npm python make gcc git rpm-build libxcrypt-compat patch
--> Using cache f0632ab100ef8ba3c8c93370971584011f56379f0fba967011d0e1ee2ebc441c
--> f0632ab100e
STEP 3/9: RUN curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.rpm.sh | bash &&     dnf install -y git-lfs &&     git lfs install
--> Using cache 34f28e6ca497fd470a8003a55b86dbd1f2101d9739224f782fa240fd8cec76a3
--> 34f28e6ca49
STEP 4/9: ARG SIGNAL_VERSION
--> Using cache fbad44b28de75dee03ba1e8ad21e855f7ca34768ebcfd0eab2d8c315fc9db3b3
--> fbad44b28de
STEP 5/9: ENV NVM_DIR /root/.nvm
--> Using cache ead8d1fb2d24b8203d90b57551c2e5a096d84079389ee12cc73d5fadec825715
--> ead8d1fb2d2
STEP 6/9: RUN npm install --global yarn &&     curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh | bash &&     source /root/.nvm/nvm.sh --no-use &&     nvm install $(curl -o- https://raw.githubusercontent.com/signalapp/Signal-Desktop/v${SIGNAL_VERSION}/.nvmrc)
--> Using cache 6180d60ccfc16c6c9777d86b02f04b7cbd674dd7b49374ad3c05b8c417811c76
--> 6180d60ccfc
STEP 7/9: COPY package.json.patch /root/package.json.patch
--> Using cache 6bd22283c8b2dbd1a30f08f7f0512dcda9071208613dc79ad40d3ab85191c6f7
--> 6bd22283c8b
STEP 8/9: RUN cd /root &&     git clone -b v${SIGNAL_VERSION} --depth 1 https://github.com/signalapp/Signal-Desktop.git &&     patch /root/Signal-Desktop/package.json /root/package.json.patch
--> Using cache 37054d4bd478e060a6b55438e1ea8a5e05507678f623f86e4cf3623de7eac551
--> 37054d4bd47
STEP 9/9: RUN cd /root/Signal-Desktop &&     source /root/.nvm/nvm.sh --no-use &&     nvm use &&     yarn install --frozen-lockfileyarn &&     yarn generate &&     yarn build-release
Found '/root/Signal-Desktop/.nvmrc' with version <16.13.2>
Now using node v16.13.2 (npm v8.1.2)
yarn install v1.22.19
[1/6] Validating package.json...
[2/6] Resolving packages...
[3/6] Fetching packages...
error Command failed.
Exit code: 128
Command: git
Arguments: clone https://github.com/signalapp/better-sqlite3 /usr/local/share/.cache/yarn/v6/.tmp/6b9ea74d442c0d03a1b4c2a1b8696978
Directory: /root/Signal-Desktop
Output:
Cloning into '/usr/local/share/.cache/yarn/v6/.tmp/6b9ea74d442c0d03a1b4c2a1b8696978'...
Downloading deps/sqlcipher.tar.gz (28 MB)
Error downloading object: deps/sqlcipher.tar.gz (a23d694): Smudge error: Error downloading deps/sqlcipher.tar.gz (a23d6941a63307a5aec6a62d260d44406c378462b1d2e16b867f37817db20132): batch response: This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.

Errors logged to '/usr/local/share/.cache/yarn/v6/.tmp/6b9ea74d442c0d03a1b4c2a1b8696978/.git/lfs/logs/20220709T173019.190087361.log'.
Use `git lfs logs last` to view the log.
error: external filter 'git-lfs filter-process' failed
fatal: deps/sqlcipher.tar.gz: smudge filter lfs failed
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

It means what it says unfortunately, Signal repo https://github.com/signalapp/better-sqlite3/ contains a "big" LFS file that everybody is trying to download while cloning, so this repo exceed max LFS bandwidth for free github accounts
I openend an issue yesterday to ask Signal to remove this LFS file (or to pay for more bandwidth...) : https://github.com/signalapp/better-sqlite3/issues/6
I will try to find a workaround in the mean time

I successfully exclude LFS file deps/sqlcipher.tar.gz by adding
RUN git config --global lfs.fetchexclude deps/sqlcipher.tar.gz

but the Signal-Desktop build process does need this file (or it fails) to build with Signal-Desktop with sqlcipher (which is the only reason Signal forked this repo, they added sqlcipher support to better-sqlite3)
So I don't think we can find a workaround, we have to wait for Signal answer here https://github.com/signalapp/better-sqlite3/issues/6 (we just have to wait for them to face the same problem while building their own project^^)

Thank you for trying to find a workaround.
Does sqlcipher.tar.gz exist somewhere else so it can be downloaded in a paralell process which does not fail?
In the very long thread in how to get it to compile on Fedora we agree on that it would be so much easier if the project had their own repo(s) for all the major distributions (Fedora, OpenSUSE, Archlinux/Manjaro) and not just Debian/Ubuntu.
Especially true with Ubuntu 22.04, some machines that were supported in Ubuntu 20.04 are not supported longer, forcing those machines to Manjaro. All of a sudden, the graphic cards were no longer supported by Ubuntu 22.04. Everything worked in 20.04, why not just add them in 22.04. Of course there must be a reason for it, but the reason is unknown. Nobody knows why some graphic cards and autdio drivers are no longer there in 22.04, but that is the reality. Ubuntu used to be the support-them-all-distro but now Manjaro seems to do that instead. And some machines will only accept Fedora because the built in PCI graphic card is broken so the legacy graphic card from the motherboard, usually some Intel chip, can be used instead. Ubuntu does not discover a faulty graphic card and use the legacy automatically, and with every update you have to miss with grub causing you to have a black screen pretty much random. Those funnies on semi working hardware...
Otherwise, the easy way is to go for Debian-based distribution which is supported by the project, if the machine can run it.

Fixed with https://github.com/BarbossHack/Signal-Desktop-Fedora/commit/b1164243571c37ac10e1e0452a4d85710501b649

In the very long thread about how to get it to compile on Fedora we agree on that it would be so much easier if the project had their own repo(s) for all the major distributions (Fedora, OpenSUSE, Archlinux/Manjaro) and not just Debian/Ubuntu.

Totaly agree, but as long as Signal doesn't have any official repo for fedora I will continue to build my own rpm (as I don't want to go for Debian-based distribution...)


UI issue with distribution functions on contact details

line width does not change
legend is empty

I've removed the legend option as this does not make sense if there is a single curve anyway.