@azure/storage-blob - containerClient.listBlobsFlat() metadata property is empty

Package Name: containerClient.listBlobsFlat()
Package Version: 12.3.0
Operating system: windows 10 internal
[ ] nodejs

version:  v10.15.3

[ ] browser

name/version: Chrome Version 86.0.4240.198 (Official Build) (64-bit)

[ ] typescript

version:3.7.5

Is the bug related to documentation in

[ ] README.md
[ x] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Metadata set on blob isn't returned as part of blobInfo in ListBlobsFlat. The Metadata property is returned, it just doesn't have the data.
To Reproduce
Steps to reproduce the behavior:

Create blob in FrontEnd React (create-react-app) with blobClient.uploadBrowserData(file, options);
Read returned results for url property which is the url for the blob
Set metadata on blob with url - blobClient.setMetadata({ urlAtCreation: blobClient.url });
Verify blob has metadata set with local Azure Storage Explorer
List blobs in container with containerClient.listBlobsFlat() - metadata property returned but is empty

Expected behavior
I expect if I set the metadata for a blob, and I can see the metadata in the Storage Explorer for that blob, then it is the same metadata that is returned with listBlobsFlat. Not sure why metadata is a string when I expect an object.
Screenshots
2 screenshots from react app debugging issue where metadata is returned as a string. 1 screenshot of storage explorer with metadata property I expect to see in code.

Additional context
I didn't add any additional security that would indicate I don't want metadata returned in the list.

You need to set the options.includeMetadata to true for List Blob to return metadata.

@ljian3377 When I set the metadata with a property name with a capitalization, why is the property name lowercased when returned via containerclient.listBlobsFlat? I can't find the docs that indicate this casing change.
`
// set metadata
const setMetadataResults = await blobClient.setMetadata({urlAtCreation:'test'});
// get blobs - urlAtCreation is empty
for await (const blob of container.containerClient.listBlobsFlat(options)) {
const simplifiedBlob: AppBlobInfo = {
name: blob.name,
createdOn: blob.properties.createdOn,
lastModified: blob.properties.lastModified,
etag: blob.properties.etag,
contentLength: blob.properties.contentLength,
contentType: blob.properties.contentType,
blobType: blob.properties.blobType,
url: blob.metadata['urlAtCreation']
};
  returnedBlobUrls.push(simplifiedBlob);
}

`

When I set the metadata with a property name with a capitalization, why is the property name lowercased

This is unfortunate because the metadata is passed to the service via http header keys, and our http client node-fetch forces all lower cases for header key names. See related issues #4966, #8117.
We have some warnings in getProperties() and should probably add similar notes to the doc of setMetadata().
https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/storage/storage-blob/src/Clients.ts#L1481

@jeremymeng Could there be a more general statement made such as..."All modern (track 2) libraries which use node-fetch and sends data via header key names, such as Storage Blob metadata, will have lowercased property names due to a node-fetch restriction."
Or just all libraries that depend on node-fetch can assume property names of JSON will be lowercased?

@diberry  What is your suggested follow-up for this issue?

Any storage methods that set or get headers that are lowercased need to have additional text - @jeremymeng text is fine.
All modern (track 2) libraries which use node-fetch and sends data via header key names, such as Storage Blob metadata, will have lowercased property names due to a node-fetch restriction.


feature for source navigation

is it possible to do source navigation in vscode, i.e. track a function's topology(caller, callee relationship etc), something like a built-in source navigator(https://github.com/rogerz/sourcenavigator)? that will be really powerful to study and debug source code. the current peek/goto of definition/declaration are wonderful, adding API relationships will be great.

Duplicate of #70231


SW-1388 Refactor SearchService to expose subquery

To calculate accession stats based on search filters, we will need to construct a
"get the summary stats" query that includes the underlying search query. Rework
the query construction in SearchService so that it exposes a method that returns
the NestedQueryBuilder.
No change in behavior here (tests continue to pass as-is).

Current dependencies on/for this PR:

main

PR #558

PR #559   👈

PR #560

This comment was auto-generated by Graphite.


add environment variation TLS_INSECURE_SKIP_VERIFY

add environment variation TLS_INSECURE_SKIP_VERIFY to support skip https certificate verify
close #1452

Codecov Report

Merging #1453 (ee27a49) into main (85385e1) will decrease coverage by 0.07%.
The diff coverage is 78.78%.

@@            Coverage Diff             @@
##             main    #1453      +/-   ##
==========================================
- Coverage   59.63%   59.56%   -0.08%
==========================================
  Files         129      129
  Lines       21791    21795       +4
==========================================
- Hits        12995    12982      -13
- Misses       7251     7260       +9
- Partials     1545     1553       +8

Impacted Files
Coverage Δ

pkg/object/restful.go
21.37% <78.78%> (+0.90%)
:arrow_up:

pkg/chunk/cached_store.go
73.18% <0.00%> (-1.39%)
:arrow_down:

pkg/chunk/disk_cache.go
79.31% <0.00%> (-0.50%)
:arrow_down:

pkg/meta/sql.go
68.39% <0.00%> (-0.26%)
:arrow_down:

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 a82242b...ee27a49. Read the comment docs.


Access to pending transactions in offchain-worker?

Hello. Can I access pending transactions in the offchain-worker code?

Hey, no that is not supported. Anything that you want to achieve?


npx create-divi-extension not working

Hi, i was trying to create a Divi extension from this tutorial https://www.elegantthemes.com/documentation/developers/create-divi-extension/ but showing the following errors https://take.ms/z2AN0C
I'm using MAMP on mac. Other versions are
npm 8.1.4
yarn 3.1.1
node 17.2.0

@devatiq you would have to downgrade the node's version because the maximum that it supports is 14.x.x and with that, it should work fine.


cluster fails with consul

Hey,
Consul registration seems fine with 0.6  👍
$ docker exec 2a9dcdbf4da1 sh -c "dig rabbitmq.service.consul"
;; QUESTION SECTION:
;rabbitmq.service.consul.   IN  A

;; ANSWER SECTION:
rabbitmq.service.consul. 0  IN  A   <IP_ADDRESS>
rabbitmq.service.consul. 0  IN  A   <IP_ADDRESS>

both nodes can tcp each other:
$ docker exec 2a9dcdbf4da1 telnet <IP_ADDRESS> 5672
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.

here my env vars (both nodes are 100% identical docker-compose scale rabbitmq=2
                "CONSUL_HOST=consul.service.consul",
                "CONSUL_ACL_TOKEN=",
                "RABBITMQ_ERLANG_COOKIE=secrect",
                "CONSUL_SVC_ADDR_NIC=eth0",
                "AUTOCLUSTER_TYPE=consul",
                "RABBITMQ_DEFAULT_PASS=guest",
                "RABBITMQ_DEFAULT_USER=guest",
                "CONSUL_PORT=8500",
                "RABBITMQ_LOGS=-",
                "RABBITMQ_SASL_LOGS=-",
                "RABBITMQ_VERSION=3.6.2",
                "RABBITMQ_DEBIAN_VERSION=3.6.2-1",
                "HOME=/var/lib/rabbitmq"

but still the cluster is not made up...the relevant log gave me this:
=INFO REPORT==== 4-Jun-2016::18:36:07 ===
autocluster: Delaying startup for 214ms.

=INFO REPORT==== 4-Jun-2016::18:36:07 ===
autocluster: Starting consul registration.

=INFO REPORT==== 4-Jun-2016::18:36:07 ===
autocluster: Registering node with consul.

=ERROR REPORT==== 4-Jun-2016::18:36:07 ===
** System NOT running to use fully qualified hostnames **
** Hostname <IP_ADDRESS> is illegal **

=ERROR REPORT==== 4-Jun-2016::18:36:07 ===
** System NOT running to use fully qualified hostnames **
** Hostname <IP_ADDRESS> is illegal **

=ERROR REPORT==== 4-Jun-2016::18:36:07 ===
autocluster: Can not communicate with cluster nodes.

setting a longname wont work in docker containers :/
and the node name is not! the ip, its:
=INFO REPORT==== 4-Jun-2016::18:47:58 ===
node           : rabbit@2a9dcdbf4da1

i guess its because the real node-name isnt stored in the consul service data and therefore i think you assume something wrong during clustering?!

Hmm no, it's more that erlang doesn't want to use the IP address for the node name  Check out https://www.rabbitmq.com/networking.html#dns
Do I understand what you want to do? On the same host/VM/etc:

Use a single Consul server/agent running in a container
Run two RabbitMQ containers that autocluster

If so, I'll work through the pattern and let you know what I come up with.

Thats exactly my (only my? I dont think so) use case. Throw some rabbitmq
nodes onto a docker swarm cluster and let them autocluster...on Production
each swarm peer has a consul, but all them in a consul cluster.
On Development i have 1 consul and x rabbitmq nodes...I think its not
different to a swarm cluster in the aspect of rabbitmq+consul clustering.
I would love to help on that issue, but im not into erlang. I can test and
Report Bugs :D
Gavin M. Roy<EMAIL_ADDRESS>schrieb am Sa., 4. Juni 2016, 22:47:

Hmm no, it's more that erlang doesn't want to use the IP address for the
node name Check out https://www.rabbitmq.com/networking.html#dns
Do I understand what you want to do? On the same host/VM/etc:

Use a single Consul server/agent running in a container
Run two RabbitMQ containers that autocluster

If so, I'll work through the pattern and let you know what I come up with.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aweber/rabbitmq-autocluster/issues/66#issuecomment-223777456,
or mute the thread
https://github.com/notifications/unsubscribe/AAR612qQHe5J98cwRa_pVhjQ79cHx0o2ks5qIeRggaJpZM4IuMJz
.

"Every node in a cluster must be able to resolve every other node's
hostname, short or fully-qualified. By default RabbitMQ will use short
hostnames."

Robert Schoenthal<EMAIL_ADDRESS>schrieb am Sa., 4. Juni 2016, 23:24:

Thats exactly my (only my? I dont think so) use case. Throw some rabbitmq
nodes onto a docker swarm cluster and let them autocluster...on Production
each swarm peer has a consul, but all them in a consul cluster.
On Development i have 1 consul and x rabbitmq nodes...I think its not
different to a swarm cluster in the aspect of rabbitmq+consul clustering.
I would love to help on that issue, but im not into erlang. I can test and
Report Bugs :D
Gavin M. Roy<EMAIL_ADDRESS>schrieb am Sa., 4. Juni 2016,
22:47:

Hmm no, it's more that erlang doesn't want to use the IP address for the
node name Check out https://www.rabbitmq.com/networking.html#dns
Do I understand what you want to do? On the same host/VM/etc:

Use a single Consul server/agent running in a container
Run two RabbitMQ containers that autocluster

If so, I'll work through the pattern and let you know what I come up
with.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aweber/rabbitmq-autocluster/issues/66#issuecomment-223777456,
or mute the thread
https://github.com/notifications/unsubscribe/AAR612qQHe5J98cwRa_pVhjQ79cHx0o2ks5qIeRggaJpZM4IuMJz
.

Just for completeness i link the old issue here https://github.com/aweber/rabbitmq-autocluster/issues/60 so any reader gets why we need the ips for communication instand of ids here

I am hitting this same issue, except inside rancher instead of docker swarm. My rabbitmqs are registering in Consul with an IP, but the cluster is not forming.

Same here with consul and 2 Docker - Hosts, is there any working example to get this running?

I have example working on top of k8s with etcd backend - https://github.com/binarin/rabbit-on-k8s-standalone
But I don't see why you can't make it work with consul and plain docker (except for #83)
Also there is a detailed explanation at this setup at https://www.mirantis.com/blog/clustered-rabbitmq-kubernetes/

I have trouble to connect the nodes. We have two different Docker Hosts not using an Multi-Host Network, so i would love to use the "Host-ips" to connect the cluster. This only works when i add each host to the hosts file on the container.. but thats not a "automatic" solution.. further i am unsure why i need Consul for such a setup?
docker run --privileged -h <IP_ADDRESS> \   -e AUTOCLUSTER_TYPE=consul \   -e AUTOCLUSTER_DELAY=50 \   -e CONSUL_HOST=<IP_ADDRESS> \   -e CONSUL_SVC_ADDR=<IP_ADDRESS> \   -e RABBITMQ_NODENAME=rabbit1 \   -e AUTOCLUSTER_CLEANUP=true \   -e RABBITMQ_USE_LONGNAME=true \   -p 4369:4369 \   -p 5672:5672 \   -p 15672:15672 \   -p 25672:25672 \   gavinmroy/alpine-rabbitmq-autocluster
and on the other node:
   docker run --privileged -h <IP_ADDRESS> \     -e AUTOCLUSTER_TYPE=consul \     -e AUTOCLUSTER_DELAY=50 \     -e CONSUL_HOST=<IP_ADDRESS> \     -e CONSUL_SVC_ADDR=<IP_ADDRESS> \     -e RABBITMQ_NODENAME=rabbit2 \     -e AUTOCLUSTER_CLEANUP=true \     -e RABBITMQ_USE_LONGNAME=true \     -p 4369:4369 \     -p 5672:5672 \     -p 15672:15672 \     -p 25672:25672 \     gavinmroy/alpine-rabbitmq-autocluster
both running consul and registering correct with:
service:rabbitmq:<IP_ADDRESS> / service:rabbitmq:<IP_ADDRESS>
if you have any tipps.. this would made my day ;)

RABBITMQ_NODENAME=rabbit2 - this doesn't look correct. Correct version should be<EMAIL_ADDRESS>etc. And this is also mentioned in a blog post that I've referenced.

Thanks for your support, i have trouble with:

Even before trying to start any rabbits, you need to be sure that containers can reach each other using selected naming scheme – for example, a ping should work with the part that comes after the @

I thought consul would handle this, if i have to ping the other container on a different docker host i need DNS or modified hosts eh?

No, autocluster/consul is used ONLY as a discovery mechanism. It just stores the hostname part of every RABBITMQ_NODENAME  - so any starting node can fetch this hostname and reconstruct remote RABBITMQ_NODENAME. Everything else happens using the regular erlang distribution mechanism - so this name needs to be resolvable (unless the IP address is used), and at least ports 4369 and 25672 should be available there for every other machine in the cluster.

You already have almost everything needed in place for successful clustering - by exposing ports 4369 and 25672 on the host. From the erlang distribution standpoint that makes a host equivalent to the rabbitmq container running there (with the only downside that you can run only one such container per host). So for a hostname part of RABBITMQ_NODENAME you can use either host IP or host DNS (if any).

The problem is that you are using CONSUL_SVC_ADDR_NIC=eth0 and the plugin registers the node on consul as:
service:rabbitmq:<IP_ADDRESS>   ( because it uses  inet:getifaddrs() )
when the plugin tries to join the nodes you got the error Hostname <IP_ADDRESS> is illegal
This is more and Erlang problem than an RabbitMQ problem, for example:
node1:
➜  ~ erl -name<EMAIL_ADDRESS>Erlang/OTP 19 [erts-8.1] [source-77fb4f8] [64-bit] [smp:4:4] [async-threads:10] [kernel-poll:false]

Eshell V8.1  (abort with ^G)
(node1@this.node.consul)1>

node2:
➜  ~ erl -sname node2
Erlang/OTP 19 [erts-8.1] [source-77fb4f8] [64-bit] [smp:4:4] [async-threads:10] [kernel-poll:false]

Eshell V8.1  (abort with ^G)
(node2@mac)1> net_kernel:connect('node1@this.node.consul').
false

=ERROR REPORT==== 20-Mar-2017::17:43:01 ===
** System NOT running to use fully qualified hostnames **
** Hostname this.node.consul is illegal **
(node@mac)2>

So you should not use "eth0" or try to use long name.
Here is an working example using consul


InstallGuide: move uninstall ArcGIS Data Store before uninstall ArcGIS Server

...in Uninstall Software section

Fixed in v.next. Closing issue.


Ensure RelativeTime component's text representation is client side computed

When relative time is rendered in an statically generated route, the relative value is no longer correct as it does not change based on the time of the request.
Solution:
Relative time to be a client component, and representable relative time text to be computed at client side

Hey @rithviknishad ,
I'd like to contribute to this issue for Hacktoberfest . Could you please assign it to me?

Hey @rithviknishad,
I'm encountering an issue with my project where the website looks different on the live version compared to my localhost setup.
Live Version:

Localhost Version:

As you can see, there are discrepancies in the appearance. Could you help me identify the root cause and figure out a solution to make both versions consistent?

You mean the data present in both is different? Or the layout?

You mean the data present in both is different? Or the layout?

Both things are different in live version and in localhost

data being different is likely because your local data-repo is outdated.
layout, I don't see any difference.


Addendum to #832 for CUDA/HIP devices

Hi Aksel,
While checking what we did in #832, I realised we overlooked a fix for one of the device properties for the HIP backend.
maxThreadsDim[x] seems to be the same for all x, at least on the MI200s, so the cast is actually the most important bit here... since the multiplication is otherwise performed in 32 bits 🥶.
Cheers,
-N

Thanks! Would we not also need to do the same change for CUDA? I think it has the same code:
https://github.com/OpenSYCL/OpenSYCL/blob/442311b26fcc3669c7acc346361c6bb273ea2b0d/src/runtime/cuda/cuda_hardware_manager.cpp#L222

Unnecessary, the multiplication doesn't overflow there... unless you want the code to be literally the same...
Oh, wow, but according to the docs both operands are actually of int type though... How is it working? Can you give it a try?

Thanks! It might depend on what these queries actually return, i.e. the target device, and potentially driver settings or similar.
I think the CUDA docs were indeed the right ones, not the LLVM docs, since this is a query for the CUDA runtime library. The LLVM nvptx backend should not be involved here :-)


Fix/improve group join

fix broken redirect on "home" page
make join-group available for everybody. for logged-in users, show only groups that they are not yet part of
align join button to the right
add padding

Can be merged instantly hopefully ;)

Codecov Report

Merging #565 into master will increase coverage by 0.16%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #565      +/-   ##
==========================================
+ Coverage   98.41%   98.58%   +0.16%
==========================================
  Files         239      239
  Lines        3734     3739       +5
==========================================
+ Hits         3675     3686      +11
+ Misses         59       53       -6

Impacted Files
Coverage Δ

...ient/app/components/joinGroupList/joinGroupList.js
66.66% <ø> (-8.34%)
:arrow_down:

client/app/components/home/home.controller.js
100% <100%> (ø)
:arrow_up:

...mponents/joinGroupList/joinGroupList.controller.js
100% <100%> (+60%)
:arrow_up:

client/app/components/home/home.spec.js
100% <100%> (ø)
:arrow_up:

...app/components/joinGroupList/joinGroupList.spec.js
100% <100%> (ø)
:arrow_up:

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 5277155...c388d47. Read the comment docs.


Vimnav improvements

Here’s my humble attempt at filling the gaps in the vim navigation layer 🙂
Next/previous word – nwd/pwd

C-rght → [T]
C-lft → [B]

Vim style word-by-word navigation in every text area! This one is definitely biased towards Ergo‑L as it maps perfectly with its w and b, so the position is the same as in vim.
Find – fnd

C-f → [G]

Still quite biased towards Ergo‑L, but in this case we’re not so far from Qwerty’s f.
Redo – rdo

C-y → [/]

Position symmetrical to Arsenik’s existing ndo.

Since I started using HRMs this week, I'm not so fond of the C-f mapping anymore (it is basically useless now).
What about replacing it with A-tab? It would be right next to S-tab and tab.

In fact, with HRM, nearly all C- shorcuts are useless in Ergo-L as Ergo-L keep shortcut keys location …
I like C-right and C-left mappings (require to press only two keys instead of three).
Redo make sens with the rest of the shortcuts, but i am not convinced with the proposed location, ihmo on the left hand makes more sens.
No opinion on A-tab (and no effect with my setup).
I think a good improvement to make the layer more suitable with HRM would be to have :

tab and S-tab not under HRM Ctrl, to be able to change tabs in navigators with only one hand
tab not under HRM Alt for easy A-tab

Redo make sens with the rest of the shortcuts, but i am not convinced with the proposed location, ihmo on the left hand makes more sens.

Definitely agree with you, but I didn’t want to change the existing mappings, thus introducing a breaking change. If the maintainers of Arsenik express a wish to improve the full layer, maybe we could rethink some existing placements.

No opinion on A-tab (and no effect with my setup).

Do you mean it doesn’t work on your machine or simply that you already have a better way to Alt-Tab?

I think a good improvement to make the layer more suitable with HRM would be to have :

tab and S-tab not under HRM Ctrl, to be able to change tabs in navigators with only one hand
tab not under HRM Alt for easy A-tab

Totally! But just like redo this would mean messing with the existing mappings. I’m starting to think a new HRM optimized navigation layer (navigation_vim_hrm) would be a good solution.

Do you mean it doesn’t work on your machine or simply that you already have a better way to Alt-Tab?

I do not use Alt-Tab as I use a tiling windows manager.


Amendment's made to previous PR

Here as the amended changes to the original PR #76 ,
Please advise further if there is anything else that might need fixing,
Thanks =)

I think your branch is behind a bit, and some things have changed! Can you rebase off of main (or merge with main) to resolve conflicts, and then resubmit?
Alternatively, it could be good to break apart the "content" markdown changes from the CSS/code changes, into two separate PRs. It's easier to review and merge a markdown PR if there are no code changes.

Sorry @FredKSchott , Ill do that from now on


Fix delayed with dataclasses where init=False

[x] Closes #7654
[x] Tests added / passed
[x] Passes black dask / flake8 dask / isort dask

The one failing test failed in the conda solve, so I am going to go ahead and merge this.


Reorganize code base

Move all example into a sub folder.
Make code base , specially the root , cleaner.

too much stuff in root


[Issue 5725][localrun] Set connector's directory from system properties as well as environment parameter.

#5725
Motivation
I am developing a gradle plugin for pulsar.
The main task for the pulsar-gradle-plugin is to run sink-connectors, source-connectors, functions as localrun processes.
As outlined in https://pulsar.apache.org/docs/en/functions-debug/#debug-with-localrun-mode
This makes debugging of these components from withing IDE (idea) simpler.
Modifications
System.getenv("PULSAR_HOME")
cannot be changed programmatically from java and consequently from the gradle plugin (written in java).
This can be corrected by first checking for the "pulsar.home.path" system property.
System.getProperty("pulsar.home.path").
To allow more flexibility in the placement of the connectors the use of
System.getProperty("pulsar.connector.path") is also allowed.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

The public API:  is extended by this change. It is possible to set system properties which change the behavior.

Documentation
This will change the documentation for
https://pulsar.apache.org/docs/en/functions-debug/#debug-with-localrun-mode
Currently the documentation is missing a mention of the PULSAR_HOME environment variable.

Closed as fixed by #15693.


PowerPoint ShapeProperties does not show a:xfrm (Transform2D)

Powerpoint slide with shapes
Transform2D of shapes is generated when viewed from shapetree node (parent) but Transform2D values is not not shown when viewed from shapes node (child of shapetree).
Transform2D contains the Offset and Extents.

I haven't seen/noticed this issue when examining pptx files before.  I'll see if I can check it out tomorrow.
Fyi - this issue might belong in the Serialize.OpenXml.CodeGen project depending on where the issue is so don't be surprised if I move it there after I look into it.

I'm marking this as needs confirmation.  Meaning that I need to duplicate the issue on my side so I know how to fix.

Hi,
What values are you not seeing for the Transform2D element?  I just tried it and I am able to see the expected Offset and Extents properties just fine.  And off the top of my head, I don't recall any properties on the Transform2D element directly that are used often, if at all.  Please let me know.

Cosing this issue. I can't find the error now either. Must have look at the wrong shape when I reported this.


Added Score Keeper

Description
Added a Simple Ping Pong Score Keeper app.
Fixes #165

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system: Windows
Version: 10
Text-editors used: IntelliJ IDEA

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

remove this file @darkling10


Created DependencyInjection extensions module

Simple but useful extensions for DI.

Ok. I will make so. I made separate package only because I was worried about extra dependency. )


Quirk for Tuya PIR motion detection

Quirk for Tuya PIR motion detection. Same as Blitzwolf BW-IS3 -- just a Tuya's rebrand.
The cool down period is 15s.
Fixes: https://github.com/zigpy/zha-device-handlers/issues/445
Fixes: https://github.com/zigpy/zha-device-handlers/issues/319

Would it be possible to have this quick updated for the original device requested.  Looking at the quirk, the manufacturer code is not listed so it is not recognised in ZHA.
{
"node_descriptor": "NodeDescriptor(byte1=2, byte2=64, mac_capability_flags=128, manufacturer_code=0, maximum_buffer_size=82, maximum_incoming_transfer_size=255, server_mask=11264, maximum_outgoing_transfer_size=255, descriptor_capability_field=0)",
"endpoints": {
"1": {
"profile_id": 260,
"device_type": "0x0000",
"in_clusters": [
"0x0000",
"0x0003"
],
"out_clusters": [
"0x0003",
"0x0019"
]
}
},
"manufacturer": "_TYST11_7hfcudw5",
"model": "hfcudw5",
"class": "zigpy.device.Device"
}

Doh, fixed in https://github.com/zigpy/zha-device-handlers/pull/504


Topic2

Good. Regarding topic 2, it's fine but you should avoid commit not necesary files because mess up code review work.


Lost types when used in monorepo

Describe the bug
After installed package, import it in vite.config.ts, it can't found the type declarations. I think it maybe has a little different from other plugins which also build by unplugin

Reproduction
Not Need
System Info
System:
    OS: Linux 5.15 Ubuntu 20.04.4 LTS (Focal Fossa)
    CPU: (6) x64 Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz
    Memory: 2.86 GB / 15.59 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 16.15.0 - ~/.nvm/versions/node/v16.15.0/bin/node
    Yarn: 1.22.18 - ~/.nvm/versions/node/v16.15.0/bin/yarn
    npm: 8.5.5 - ~/.nvm/versions/node/v16.15.0/bin/npm

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Duplicate of #257


add fsspec support to SimpleDirectoryReader + default file readers

Description
Adds fsspec support to SimpleDirectoryReader.
Allows for out of the box support for loading a variety file-types from a variety of external file-systems supported by the fsspec protocol.
Some of the file-type loaders relied on 3rd party libraries that didn't allow for loading in of files from a file-like object. It had to be given a file-path, which would mean it was bound to reading from the local FS. For those, I am just logging a warning message if the fs was specified and letting it read from local FS for now.
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

awesome! LGTM. Should we add some 1-2 examples in our documentation for SimpleDirectoryReader that reads from a remote fs?

good idea, just added one! And ran into some bugs I had to fix in the process 😅 so very worthwhile


ufw issue with cronjob

doesn't work as cronjob , needed to replace all ufw commands with $UFW to get it working again with the latest update of UFW

It's better to setup paths inside "crontab -e", just like in the file /etc/crontab:

SHELL=/bin/sh
PATH=/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin

This way you can just use "ufw" inside the script


externalizing settings-properties for the plugin to course config file

I have certain worries about the capabilities of the A+ API we will be polling against, so the current thought is to externalize some settings to the course configuration file:
public class SubmissionStatusUpdater {
  ...
  // 10 seconds in milliseconds
  private static final long DEFAULT_INTERVAL = 10L * 1000;

  // 3 minutes in milliseconds
  private static final long DEFAULT_TIME_LIMIT = 3L * 60 * 1000;

public class PluginSettings implements MainViewModelProvider {
  ...
  //  15 minutes in milliseconds
  public static final long MAIN_VIEW_MODEL_UPDATE_INTERVAL = 15L * 60L * 1000L;

and maybe even
public class PluginSettings implements MainViewModelProvider {
  ...
  //  15 seconds in milliseconds
  public static final long REASONABLE_DELAY_FOR_MODULE_INSTALLATION = 15L * 1000;

the benefit as I see is, we can quickly change the number of calls to the API (by reducing their frequency) wo having to wait "up to 2 (two) business days" (that can take JB to review the plugin) + risking some students never update the version... @nikke234 @OlliKiljunen
and maybe here, maybe definitely?
public class PluginSettings implements MainViewModelProvider {
  ...
  public static final String A_PLUS_API_BASE_URL = "https://minus.cs.aalto.fi/api/v2";

public class ScalaSdk extends IntelliJLibrary<PersistentLibraryKind<ScalaLibraryProperties>,
    ScalaLibraryPropertiesState> {

  private static final String URL = "https://scala-lang.org/files/archive/";

No.
Piece of data
Where to store it
My certainty on issue

1
A+ API URL
Course config file
Very certain

2
API polling schedule
Course config file
Could be, a very good proposal

3
Course config file polling schedule
Plugin itself
Very certain

4
Delay for module installation
Plugin itself
Certain

5
Scala SDK download URL
Plugin itself
Pretty certain

Numbers 3-5 should, however, be stored in a properties file in the plugin's resources.

is a must-have

must have is done and merged

after the Week 3 or so (performance)

Closing for now as we don't have issues.


[RFC] Component structure visualisation

Summary 💡
I'd like to gather feedback about a possible improvement to the documentation: visualization of a component structure.
It could (conceptually, don't mind the visual design) look like this: https://codesandbox.io/s/component-structure-visualisation-d31po?file=/demo.tsx (blue chips represent static classes, purple chips - conditional ones).
In my opinion this will have the most impact where component customization is required. Unstyled components come to mind first, but core components could also make use of it. It is important to know the structure of slots when customizing a component. Having a visual representation of CSS classes and where they are applied could also help.

I can definitely see some room to improve the customization documentation. Though the initial prototype feels a bit lost.
It should be clear what are classes, what are components, where are these located in the component etc. Right now the prototype isn't telling anything new because we already document in prose and name that MuiSwitch-Thumb is applied to the Thumb component of the Switch.


Add VT sequence: DECSCLM - Scrolling Mode

We currently only implement fast mode, but it might make sense to also implement smooth (slow) mode.
This mode only affects adding new lines at a slow rate, but - in modern age - we could correlate that feature with smooth scrolling (of new lines to be added), see #145.
One open question I am having is how long the linefeed should wait until resuming terminal I/O processing.
How to implement
One design idea I am having in my mind is, to actually suspend processing for N msecs on received linefeeds, if slow scrolling mode is enabled. This should effectively also suspend reading from the TTY to avoid flooding the internal buffers.
References

https://vt100.net/docs/vt510-rm/DECSCLM.html

One open question I am having is how long the linefeed should wait until resuming terminal I/O processing.

On the early DEC terminals (VT100, VT220, VT240, and VT340) the smooth scrolling speed was 6 lines per second. The VT340 apparently also had an option in the settings that could adjust the speed (choosing between 3, 6, and 12).
On level 5 terminals (like the VT510, VT520, and VT525) the default smooth scrolling speed was 9 lines per second, but there was also an escape sequence to adjust that to 18 (See DECSSCLS).
I think the choice was largely dictated by the hardware specs. For example, the VT100 had a 60Hz refresh rate with a cell height of 10px, so scrolling one pixel per refresh gives you 60/10 lines per second. For the VT510, you've got 72Hz and a 16px cell height, so scrolling two pixels per refresh gives you 72*2/16 lines per second.

What will happen if I have a stream that contains more lines that I want to have in the history and then enable slow scrolling? as an example find /


Cannot resolve class 'ControlValueAccessor' from Component

Hi, facing an issue that is blocking the development of a Web app I'm developing.
I'm using package:js interop to interface with a JS-only library, and up until recently, it was working perfectly.
But suddenly, I started seeing this in pub serve:
C:\Users\thosa\Commercial\texturist\texturist_web>pub serve
Loading source assets...
Loading angular and dart_to_js_script_rewriter transformers...
Serving texturist_web web on http://localhost:8080
[web] GET Served 4 cached assets.
[Warning from BuilderTransformer: Instance of 'LibraryBuilder' on material_components_web|primary]:
Could not resolve an OpaqueToken on a Provider!
[Warning from BuilderTransformer: Instance of 'LibraryBuilder' on material_components_web|primary]:
Could not resolve an OpaqueToken on a Provider!
[Warning from BuilderTransformer: Instance of 'LibraryBuilder' on material_components_web|primary]:
Could not resolve an OpaqueToken on a Provider!
[Warning from BuilderTransformer: Instance of 'LibraryBuilder' on texturist_web|primary]:
Could not resolve an OpaqueToken on a Provider!
[Warning from BuilderTransformer: Instance of 'LibraryBuilder' on texturist_web|primary]:

And saw this in the browser console:
'package:material_components_web/src/components/textfield/textfield.dart': malformed type: line 20 pos 16: cannot resolve class 'ControlValueAccessor' from 'MdcTextfieldComponent'
    implements ControlValueAccessor<String>, AfterViewInit, OnDestroy {
               ^: http://localhost:8080/main.dart

My class looks like:
@Component(
    selector: 'mdc-textfield',
    templateUrl: 'textfield.html',
    directives: const [
      COMMON_DIRECTIVES
    ],
    providers: const [
      const Provider(
        NG_VALUE_ACCESSOR,
        useExisting: MdcTextfieldComponent,
        multi: true,
      ),
    ])
class MdcTextfieldComponent
    implements ControlValueAccessor<String>, AfterViewInit, OnDestroy {
// ...
}

Thanks in advance.

ControlValueAccessor is part of forms. As of 4.0.0-alpha+3, there's a separate angular_forms package. So you'll need add an import/dependency on that.

Thanks!


Add simple test for external watchdog timer

Fixes #158

Codecov Report

Merging #159 (0c30188) into master (8304caf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   76.84%   76.84%
=======================================
  Files          17       17
  Lines         380      380
=======================================
  Hits          292      292
  Misses         88       88

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Photo Widget update: Interpretive Sites

ensure Reusable photo widget is implemented properly in the interpretive sites  Module.

Receive error when try to upload photo- A problem has occurred please check your internet connection. Once error received a number of field go blank and turn to red error state


Update top-level README tables & status for a few now-active proposals.

(oops, didn't double check the table itself before review... hence the last commit) 😖

Ah! I had actually updated that table but somehow committed without saving 🤦. Well, I am glad we are two pairs of eyes on that!


fix minor data races in both server and client crypto setups

These races are almost certainly non-critical, because GetSealerWithEncryptionLevel is only called when a packet packed with a certain encryption level needs to be retransmitted, i.e. long after the corresponding AEAD was created (which is the thing the mutex protects here).

Codecov Report

Merging #594 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #594      +/-   ##
========================================
+ Coverage   90.96%    91%   +0.04%
========================================
  Files          79     79
  Lines        5432   5436       +4
========================================
+ Hits         4941   4947       +6
+ Misses        315    314       -1
+ Partials      176    175       -1

Impacted Files
Coverage Δ

handshake/crypto_setup_server.go
87.5% <100%> (+0.11%)
:arrow_up:

handshake/crypto_setup_client.go
89.12% <100%> (+0.08%)
:arrow_up:

session.go
88.71% <0%> (+0.47%)
:arrow_up:

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 cc2dc2a...2014ced. Read the comment docs.


Grant job patch permission

Hi!
Here's a fix for an issue we found after upgrading from 0.8.0 to 0.9.0 we have an error stating that the operator cannot patch a k8s job:
2024-07-04 15:09:31,955 ERROR [ReconcilerExecutor-locusttestreconciler-480] com.locust.operator.controller.utils.resource.manage.ResourceCreationManager: Exception occurred during Job creation: Failure executing: PATCH at: https://<IP_ADDRESS>:443/apis/batch/v1/namespaces/locust/jobs/locust-email-reports-test-worker?fieldManager=fabric8. Message: jobs.batch "locust-email-reports-test-worker" is forbidden: User "system:serviceaccount:locust:locust-operator-locust-k8s-operator" cannot patch resource "jobs" in API group "batch" in the namespace "locust". Received status: Status(apiVersion=v1, code=403, details=StatusDetails(causes=[], group=batch, kind=jobs, name=locust-email-reports-test-worker, retryAfterSeconds=null, uid=null, additionalProperties={}), kind=Status, message=jobs.batch "locust-email-reports-test-worker" is forbidden: User "system:serviceaccount:locust:locust-operator-locust-k8s-operator" cannot patch resource "jobs" in API group "batch" in the namespace "locust", metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}), reason=Forbidden, status=Failure, additionalProperties={}).

Thank you @yuriipolishchuk first, for taking the time and second, for finding this bug and contributing back to the project 🚀
Fix looks good.

@yuriipolishchuk, version 0.9.1 has been released and can be directly used.


How to change border color of element?

Add this CSS class to the icon and replace #FF0000 with the colour of your choice.
.border-color {
    box-shadow: 0 0 0 4px #FF0000;
    border-radius: 50%;
}

Add this CSS class to the icon and replace #FF0000 with the colour of your choice.
.border-color {
    box-shadow: 0 0 0 4px #FF0000;
    border-radius: 50%;
}

having issues how to resize and align the box. Context box is not showing properly. How to fix their width and alignment?

You need to edit the boxShadow property in iconStyle. Change the #FFF white color to any color of your choice.
      <VerticalTimelineElement
        iconStyle={{
          boxShadow: "0 0 0 4px #FFF, inset 0 2px 0 rgba(0, 0, 0, .08), 0 3px 0 4px rgba(0, 0, 0, .05)"
        }}
      </VerticalTimelineElement>


Change OutboxService to be more generic

The OutboxService is now quite generic, accepting just byte[] as value. For existing client applications using protobuf for message serialization, the former OutboxService is now available as ProtobufOutboxService.
Because this is a binary incompatitble change, the major version is incremented.
@hnrkdmsk @danielrehmann This is valuable so that other projects not using protobuf can easier adapt this project. I'm well aware that for Tomorrow that means to change imports / rename things, but I think this is worth it. E.g. a GenericOutboxService and an OutboxService accepting protobuf messages would not be very clean IMHO.

LGTM 👍🏼 I like this approach 🤗

Great, let's see what @danielrehmann says 🙂

@magro I like this generic approach a lot and also do not see the issue of changing our existing projects for a major version upgrade. LGTM

Cool! I only just realized that the protobuf dependency then should be changed to optional, so that applications not using this for serialization don't get this dependency pulled in. I'll add a commit for that soonish...

The resulting poms look good, i.e. the protobuf dependency is marked as optional, see e.g.

https://s01.oss.sonatype.org/content/repositories/snapshots/one/tomorrow/transactional-outbox/commons/2.1.0-SNAPSHOT/commons-2.1.0-20231026.213920-1.pom
https://s01.oss.sonatype.org/content/repositories/snapshots/one/tomorrow/transactional-outbox/outbox-kafka-spring/3.0.0-SNAPSHOT/outbox-kafka-spring-3.0.0-20231026.213920-1.pom


Customize scroll axis

Hi,
thanks first of all for a very useful plugin!
I was wondering if you'd be open to add a way to customize the scrolling axis.
Example:
Given a horizontal embla carousel, I'd want to slide on vertical scroll events instead of horizontal scroll events.
I imagine something like
type WheelGesturesPluginOptions = {
  wheelDraggingClass: string,
  forceWheelAxis?: 'x' | 'y'  // defaults to undefined
}

and subsequently

    function handleWheel(state: WheelEventState) {
      const {
        axisDelta: [deltaX, deltaY],
      } = state
      const scrollAxis = foceWheelAxis ?? engine.options.axis;
      const primaryAxisDelta = scrollAxis === 'x' ? deltaX : deltaY
      const crossAxisDelta = scrollAxis === 'x' ? deltaY : deltaX

would be happy to submit a PR if you think that it'd be a fitting addition to your plugin.

Yes, I'm happy to take a PR to support this.

@lukasIO, feel free to add this option to the documentation website too. I would be happy to approve your PR.

@lukasIO I've added this to the documentation website so no PR needed.

thank you!


KeyError: 'LinkKeys'

I'm currently using MacOS Majave 10.14 beta.
Ross-MacBook-Pro:Desktop rossoknarun$ sudo ./BT-Linkkeysync.py
BT-Linkkeysync by DigitalBird

get Bluetooth Link Keys from macOS and store it to blued.plist
convert exported list from binary to xml
parse the converted plist
Convert the Link Keys and store them to btkeys.reg
Traceback (most recent call last):
File "./BT-Linkkeysync.py", line 45, in
print("  Bluetooth 2.0:    "+str(len(pl["LinkKeys"].keys()))+ " Links keys found")
KeyError: 'LinkKeys'

https://github.com/digitalbirdo/BT-LinkkeySync/blob/adb7985d3008a50fcf25ff60ae1f2d38938adbfe/BT-LinkkeySync.py#L15
set highSierraLoc = True please

@OrangeCat9  Unfortunately, this does not solve the problem

Traceback (most recent call last):
  File "./BT-Linkkeysync.py", line 45, in <module>
    print("  Bluetooth 2.0:    "+str(len(pl["LinkKeys"].keys()))+ " Links keys found")
KeyError: 'LinkKeys'

macOS High Sierra 10.13.6
iMac 21.5-inch Mid 2011

I had the same issue and resolved it in my branch. Please check if the fix works for you, and don't forget to comment in the pull request if it succeeded :)

Thanks, well done

I could not test it as I don't have a working hackintosh at the moment, but the change looks good. Thx @StefanLobbenmeier


Add help popup on hitting F1 or ?

For new users of igrep (like me!) it would be very useful to have a popup activated by the ? or F1 keys that would show all keybindings. It took returning to the GitHub repo and looking at the README for me to figure out the new context feature.

This is not a bad idea, thanks for suggestion :)


Wrap with options?

Can we wrap a function with options (like priority)? I saw this in the code, so wasn't sure if the options are always set to {} on a wrapped function.
Eternally grateful for this project. Thanks!
      wrap(fn) {
        return (...args) => {
          return this.schedule.apply({}, Array.prototype.concat(fn, args));
        };
      }

Unfortunately, not it is not possible since Bottleneck has no way to tell whether the options object was meant for the limiter or for the wrapped function!
If we wanted to allow wrap() to take options, then everytime the wrapped function is called an options object would have to be present.
It would look like this:
// You could still do this
const wrapped = limiter.wrap(myfunction)
wrapped('hi!')

// --------------------
// but you could also do this:

const wrapped2 = limiter.wrap(myfunction, { withOptions: true })

wrapped2({ priority: 1 }, 'abc')

// I would have to pass an empty object here even if I'm not specifying any options
wrapped2({}, 'xyz')

// This wouldn't work, since 'hello' would go to the limiter, not to `myfunction`
wrapped2('hi!')

Actually, I think the following looks even better:
var limiter = new Bottleneck({ maxConcurrent: 1, minTime: 1000 })

const wrapped = limiter.wrap(console.log)

wrapped('hi!')
wrapped('hello 1')
wrapped('hello 2')
wrapped.withOptions({priority: 1}, 'bye')

I'm working on this now. It won't tak very long.


fixcan not grant priv to __internal_schema

Proposed changes
Issue Number: close #xxx
mysql> grant SELECT_PRIV on `_internal_schema`.* to 'test'@'%'; ERROR 1102 (42000): errCode = 2, detailMessage = Incorrect database name '_internal_schema'

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Edit this page not working

Issue URL: http://wpveda.com/git/linux_commands.html
When we click on EDIT THIS PAGE its redirect on wrong location
http://wpveda.com/git/undefined/git/linux_commands.md
Also expose outdated nginx version.

Edit link fixed. :-)
Nginx version is outdated at CDN77.com level. There is nothing to worry about it as this is static site, directly hosted on CDN and all content is anyway open-source.


DataSource, Driver and ResultSet

This PR adds datasource, driver and result set metatada implementations
Please @juanpedromoreno could you review? Thanks

Comments addressed @juanpedromoreno

LGTM!


OGC_EDR examples

When the command pygeoapi openapi generate $PYGEOAPI_CONFIG --output-file $PYGEOAPI_OPENAPI is executed it gets the following error:
pygeoapi.provider.base.ProviderConnectionError: 'EntryPoints' object has no attribute 'get'
This is with the example-config-edr.yml file.

installation procedure updated for EDR, please check


fallback to apiserver if cannot connect to kubelet

fix https://github.com/juicedata/juicefs-csi-driver/issues/478

enable podInfoOnMount in csiDriver
set mount pod info in app pod
add pod manager to watch mount pod. list app pods by labels and run podDriver(mountpoint recover).

Codecov Report
Patch coverage: 6.71% and project coverage change: -1.32 :warning:

Comparison is base (c38b68d) 41.06% compared to head (0e82c80) 39.74%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #655      +/-   ##
==========================================
- Coverage   41.06%   39.74%   -1.32%
==========================================
  Files          31       32       +1
  Lines        3653     3784     +131
==========================================
+ Hits         1500     1504       +4
- Misses       2011     2134     +123
- Partials      142      146       +4

Impacted Files
Coverage Δ

pkg/config/config.go
0.00% <ø> (ø)

pkg/config/setting.go
75.22% <0.00%> (-5.96%)
:arrow_down:

pkg/controller/pod_controller.go
0.00% <0.00%> (ø)

pkg/controller/reconciler.go
0.00% <0.00%> (ø)

pkg/juicefs/mount/process_mount.go
40.70% <ø> (ø)

pkg/k8sclient/kubelet_client.go
0.00% <0.00%> (ø)

pkg/util/pod.go
35.45% <0.00%> (-3.94%)
:arrow_down:

pkg/juicefs/mount/pod_mount.go
35.18% <25.00%> (-1.23%)
:arrow_down:

pkg/juicefs/juicefs.go
50.28% <60.00%> (-0.10%)
:arrow_down:

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


Content editor: Allow tagging ancestors of selected nodes

At the moment, it is only possible to set tags for content nodes which have been added to the collection. This is usually reasonable - it is logical that one would only need to tag a node if it is included in the collection, and indeed the system makes some effort to remove tags for nodes which are not included.
But this gets tricky if we want to add a tag for a topic node. To do so, one must add that topic to the collection. Adding a topic means that all of its descendants are included, which is not always desired.
To solve this, we can do the following:

In CollectionEditorChannelPage, display ancestors of selected content nodes. Nodes which are included implicitly should not have Remove buttons, but they should have bulk select checkboxes and tag editors.
In ContentSelectionsTable, display content nodes in a hierarchy, replacing the existing breadcrumbs widget.
In the relevant data stores, make sure tags for implicitly included nodes are not pruned when exporting. (This will require some cleanups. Commit bb2feb56739130177bb82c8175a8a4b572c392b8 added some cleanups to the collectionBase module itself. In addition, #9 added a sanitize step on export).

This has a small amount of overlap with #11, although it's more complex. So, depending on how needed each one is, it may be better to deal with these two tasks in isolation.

so, depending on how needed each one is, it may be better to deal with the two tasks in isolation. (But in an ideal world I would try to solve both of them here).

That's a question for @hydrolet. Is arbitrary topics tagging a requirement? In the Explore plugin we won't be able to highlight channels in the Discovery page until #11 happens.

Okay let me try to understand the issue. Are both of these statements currently true?

If I want to tag an entire topic, I must include everything within that topic in the collection with the same tag.
If I want to include selected resources within a topic, or tag resources with different tags, I must tag those individually.

I like being able to tag a topic because then I don't have to take the time to tag the individual resources within that topic. This is helpful in the Career Girls channel, for example, where everything should be tagged "career."
But, there are likely many instances where I want to tag items within that topic with different tags. Like the Career Girls example, where it would be nice to also tag a few of the videos as "highlight."
Another example would be a TED-Ed topic where I want to tag all the resources in that topic (videos and quizzes) as "curious." But I only want one of the videos to be tagged "highlight." It sounds like I would have to do that manually from within the topic anyway. If that is the case, the ability to tag an entire topic is less of a priority, unless it is possible to tag an entire topic and then edit the tags within that topic (to add "highlight" to one or two of them, or remove a tag that doesn't fit an individual resource).

Thanks for the explanation @hydrolet . Yes then it sounds like it would be useful to tag a topic to consider all the resources inside as having the tag. Please note that if we go with this there won't be a way of tagging the topic itself without also tagging its resources.


sudo: ./svc.sh: command not found

Hi, I've just executed the command given by the VSTS deployment group page  to execute the agent on my Ubuntu 16.04.4 LTS server (via su root) and received the following error at the end. Is this a problem?
...
./run.sh
Must not run with sudo
sudo: ./svc.sh: command not found
sudo: ./svc.sh: command not found

The command given by VSTS on the https://tenant.visualstudio.com/project_name/_machinegroup?view=MachineGroupView&mgid=47&tab=Details URL was:
mkdir vstsagent;cd vstsagent;curl -fkSL -o vstsagent.tar.gz https://vstsagentpackage.azureedge.net/agent/2.129.1/vsts-agent-linux-x64-2.129.1.tar.gz;tar -zxvf vstsagent.tar.gz; if [ -x "$(command -v systemctl)" ]; then ./config.sh --deploymentgroup --deploymentgroupname "Staging" --acceptteeeula --agent $HOSTNAME --url https://<snipped_tenant>.visualstudio.com/ --work _work --projectname '<snipped_project>' --auth PAT --token <sniped_token>--runasservice; sudo ./svc.sh install; sudo ./svc.sh start; else ./config.sh --deploymentgroup --deploymentgroupname "Staging" --acceptteeeula --agent $HOSTNAME --url https://<snipped_tenant>.visualstudio.com/ --work _work --projectname '<snipped_project>' --auth PAT --token <snipped_token>; ./run.sh; fi

Looks like svc.sh  script doesn't exist in the agent.

Update: it didn't like root. I added my normal user to sudo group and it's happier, but still failing on sudo: ./svc.sh: command not found

Update: okay, after running the vstsagent/installdependencies.sh script and nuking the agent folder and re-running it worked.
So the commands given on visualstudio.com might need updating to include the dependencies script?

Did you run the commands here: https://github.com/Microsoft/vsts-agent/blob/master/docs/start/envubuntu.md ?
I initially missed these instructions and got the same errors as you. I rm -rf'd the vstsagent folder and redid the lengthy command and it succeeded.
I am however on 16.04.3 LTS instead of .4

No, because the user journey is you start from Azure Portal, run the command it tells you to, and that fails. That script needs fixing and then the pre-reqs here are irrelevant. You shouldn't have to know to come here and find the pre-reqs that you don't know you need.

Ah, okay. We're using TFS and the deployment groups page has this link: https://aka.ms/vstsagentubuntusystem I am unfamiliar with what VSTS has here.

 I didn't see that. I got too eager with the big command :)

looks like everyone is happy now, i am going to close the issue, please reopen if more help needed. :D

I commented out the following code in the config.sh file.This is helpful.
`# we want to snapshot the environment of the config user
if [ $user_id -eq 0 -a -z "$AGENT_ALLOW_RUNASROOT" ] && [ "${ALLOW_RUNASROOT:-default_value}" == "default_value" ]; then
echo "Must not run with sudo"
exit 1
#fi
`


Added all unit tests to LinuxMain

Forgot to add the new unit test files to LinuxMain.swift

Ok, this seems to be the same issue as in vapor/vapor#615. Not really sure what to do with it for now.

Ok, apparently these tests crash on Linux, not a good sign. I'll have a look fixing them.

Current coverage is 93.37% (diff: 100%)

Merging #22 into master will not change coverage

@@             master        #22   diff @@
==========================================
  Files            11         11
  Lines          1752       1752
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1636       1636
  Misses          116        116
  Partials          0          0

Powered by Codecov. Last update 5d39609...27db876


Overall feedback for app integration

Hi,
we want to support JXL in our photo app (iOS). It is a full featured format that brings unusual yet useful capabilities like float encoding, especially for HDR, but not only. We've built it and internally integrated it (v0.5), and we have this feedback so far. (If this is not the right place for this feedback, please let me know)

Memory footprint
Memory usage seems pretty steep when encoding a 12MP photo with JXL. I measured spikes > 800MB in Xcode when doing so, which cause crash on older iOS device lacking some memory. Is it expected? Will that stay that way in later release? Are there options that let you control that aspect?

Encoding duration
It looks like it takes over 10s to encode a 12MP on iPhone 6S, about 5s on iPhone XS (compiled with optimisations). Also, is that expected?

Data input API
Currently, we have to provide data as float RGB triplets, or RGBA quadruplets. It would be interesting to get an RGBX option (ignored alpha) if that makes sense in the input API, as rendering format very often is aligned on 4 floats per pixel. Currently it is required to re-layout data (consumes CPU / mem) prior to pass it to libjxl

Line-based encoder API for large images
In some situations, we generate very large picture that typically can't be stored in memory (too large). We generate them band by band, and we encode each band as they are made through libjpeg line-feeding. This approach does not seem possible at this time with libjxl. Is that something that will possible at some point in the future, or is it inherently impossible to achieve that because of the method used?

Metadata passing
We'd be very interested to pass along all the EXIF / TIFF / etc. metadata to the generated image file. Currently, this does not seems possible. Even orientation seems to be ignored.

Color space support
Could you consider adding linear ROMM / ProPhoto color space to the built-in API ? This color space makes a lot of sense for photo editing workflows.

Thank you.

Hi,
we want to support JXL in our photo app (iOS). It is a full featured format that brings unusual yet useful capabilities like float encoding, especially for HDR, but not only. We've built it and internally integrated it (v0.5), and we have this feedback so far. (If this is not the right place for this feedback, please let me know)

Hi! Thank you for the feedback, and for integrating JPEG XL! This is indeed the (a?) right place for it :)
I will answer to some of your questions, and let others deal with the rest.

Memory footprint
Memory usage seems pretty steep when encoding a 12MP photo with JXL. I measured spikes > 800MB in Xcode when doing so, which cause crash on older iOS device lacking some memory. Is it expected? Will that stay that way in later release? Are there options that let you control that aspect?
Encoding duration
It looks like it takes over 10s to encode a 12MP on iPhone 6S, about 5s on iPhone XS (compiled with optimisations). Also, is that expected?

For both of those, I suspect that there is a common cause - patch/dot detection. It should be possible to significantly reduce memory and time usage by reducing the encode effort to 6 instead of 7 - the effect should be minimal on most photographic content.

Line-based encoder API for large images
In some situations, we generate very large picture that typically can't be stored in memory (too large). We generate them band by band, and we encode each band as they are made through libjpeg line-feeding. This approach does not seem possible at this time with libjxl. Is that something that will possible at some point in the future, or is it inherently impossible to achieve that because of the method used?
This should in theory be possible, although the format features might require buffering some of that data (likely a few 8x8 downsampled fields, and the output image). I don't think this will be a problem in practice, but I'd like to hear from you about that.

Thank you.

Thank you again!

Regarding orientation: that can be set by setting the corresponding field in the JxlBasicInfo (https://github.com/libjxl/libjxl/blob/main/lib/include/jxl/codestream_header.h#L185). Like color space, we don't consider orientation to be 'merely' metadata, since it impacts image rendering.
Regarding metadata: There currently is no encode/decode api yet to add/retrieve 'real' metadata (Exif/XMP/JUMBF), I recently proposed something but we're still discussing how exactly to do it: see https://github.com/libjxl/libjxl/pull/501.
'Real' metadata is handled at the container (file format, 18181-2)  level, not at the codestream level (18181-1).
ProPhoto should be possible by using JXL_PRIMARIES_CUSTOM and JXL_WHITE_POINT_CUSTOM and setting the primaries and white point to the right values (they're listed e.g. here: https://en.wikipedia.org/wiki/ProPhoto_RGB_color_space), and using JXL_TRANSFER_FUNCTION_LINEAR.
Adding a convenience function for ProPhoto could in principle be done, but the same could be said about other 'popular' colorspaces like Display P3, Adobe RGB (1998), Rec.2100 PQ, etc. If we do that, I'd rather do it by defining some JxlColorEncoding constants that can be used with JxlEncoderSetColorEncoding() than to add a function like JxlColorEncodingSetToSRGB() for each of them.
Adding an RGBX pixel format option shouldn't be too hard I think. Any objections against adding that, @lvandeve / @deymo ?

For both of those, I suspect that there is a common cause - patch/dot detection. It should be possible to significantly reduce memory and time usage by reducing the encode effort to 6 instead of 7 - the effect should be minimal on most photographic content.

Great, I'll do some testing in that direction, and let you know how it goes.

This should in theory be possible, although the format features might require buffering some of that data (likely a few 8x8 downsampled fields, and the output image). I don't think this will be a problem in practice, but I'd like to hear from you about that.

OK. The way we typically generate the full size image band by band. Band height can be something like 256px or 512px, and typically cover the full width. We're talking of photos over 10k x 10k, which could be hard/impossible to represent in their entirety in RAM. Would be an extremely useful feature, as this is exactly why we had to use libjpeg instead of iOS's Image IO (which requires full-size image in RAM). That also would allow lower spec'd hardware to implement encoding, as a 12MP RGBA already takes ~200MB just by itself. Definitely opens the door to more outcomes.

Regarding orientation: that can be set by setting the corresponding field in the JxlBasicInfo (https://github.com/libjxl/libjxl/blob/main/lib/include/jxl/codestream_header.h#L185). Like color space, we don't consider orientation to be 'merely' metadata, since it impacts image rendering.

Yes, I did that (EXIF orientation 6 while encoding), but jxlinfo reports 1 on that image. Note that I used v0.5 for encoding, and v0.3.7 for jxlinfo.

Regarding metadata: There currently is no encode/decode api yet to add/retrieve 'real' metadata (Exif/XMP/JUMBF), I recently proposed something but we're still discussing how exactly to do it: see #501.
'Real' metadata is handled at the container (file format, 18181-2) level, not at the codestream level (18181-1).

Great. Looking forward to that. We'll then need to convert the CGImage (Image IO) ones to your API to get going.

ProPhoto should be possible by using JXL_PRIMARIES_CUSTOM and JXL_WHITE_POINT_CUSTOM and setting the primaries and white point to the right values (they're listed e.g. here: https://en.wikipedia.org/wiki/ProPhoto_RGB_color_space), and using JXL_TRANSFER_FUNCTION_LINEAR.

Definitely. I'll give it a try. For now, I'm using the (linear - no tone curve) sRGB with extended gamut (beyond 0-1), which in theory enables unlimited color representation as well [btw, is extended gamut fully supported?]. Because it's easier to setup that way in libjxl API.

Adding a convenience function for ProPhoto could in principle be done, but the same could be said about other 'popular' colorspaces like Display P3, Adobe RGB (1998), Rec.2100 PQ, etc. If we do that, I'd rather do it by defining some JxlColorEncoding constants that can be used with JxlEncoderSetColorEncoding() than to add a function like JxlColorEncodingSetToSRGB() for each of them.

Could be nice for easier specification. Possibly not required though as there is a workaround.

It could be that v0.3.7 of jxlinfo didn't properly report orientation, I remember fixing a bug in it. Could you check what the current jxlinfo reports? Also: does the image display OK? (i.e. when you decode, do you get the oriented pixels if you don't explicitly say you don't want the decoder to orient it for you?)
Extended gamut linear sRGB should be fully supported, yes (of course with the caveat that if you ask for the pixels in an integer format like uint16, they will be clamped to the sRGB gamut - but as long as you use floats, it's fine). It's the intermediate representation cjxl uses too.

For both of those, I suspect that there is a common cause - patch/dot detection. It should be possible to significantly reduce memory and time usage by reducing the encode effort to 6 instead of 7 - the effect should be minimal on most photographic content.

Great, I'll do some testing in that direction, and let you know how it goes.

Here's some results with 7 v. 6 effort (rough approximations from the gauges, did not run through profiler):
iPhone XS
Effort 7 - 12MP - duration=4s - RAM delta=700MB
Effort 6 - 12MP - duration=2.4s - RAM delta=300MB
iPhone 6s
Effort 7 - 12MP - duration=10.5s - RAM delta=700MB
Effort 6 - 12MP - duration=5.1s - RAM delta=300MB
Effort 6 should solve my crashing issue on older devices (available RAM exceeded). About file size, effort 6 is slightly larger (~5%), but I guess this depends on contents.

Effort 6 should solve my crashing issue on older devices (available RAM exceeded). About file size, effort 6 is slightly larger (~5%), but I guess this depends on contents.

That's a larger difference than I expected - I guess you could benefit from being able to just disable dots and patches, but IIRC that's not possible with the current API...
Anyway, good to know that the performance is better - I hope this is enough for practical usage!

It could be that v0.3.7 of jxlinfo didn't properly report orientation, I remember fixing a bug in it. Could you check what the current jxlinfo reports? Also: does the image display OK? (i.e. when you decode, do you get the oriented pixels if you don't explicitly say you don't want the decoder to orient it for you?)

Unless I made a mistake with input structure, I'm encoding (v0.5) with orientation 6, and jxlinfo (v0.5) still reports orientation 1 on generated data.

Extended gamut linear sRGB should be fully supported, yes (of course with the caveat that if you ask for the pixels in an integer format like uint16, they will be clamped to the sRGB gamut - but as long as you use floats, it's fine). It's the intermediate representation cjxl uses too.

Great, thank you. I'll do some additional testing at a later time to confirm there is no loss of information between sRGB extended range & linear ROMM / ProPhoto.

#535 and #630 should help to fix some of the issues reported here.
Thanks again for the feedback!
If you have further feedback, feel free to open more issues. Preferably different ones per issue, that makes it easier to track what is resolved and what is still open.
Closing this one now since I think everything here has been addressed; please reopen if I missed something.

Hi @rsebbe, I wonder can you please share how do you setup on ios?
I'd like the build the JXL decoder on ios app as well but I'm having a trouble with this library setup.
Thank you!


feat: support ssr target compilation

[x] ssr compiler
[ ] ssr compiler test (wip)

Now, vine can run on nuxt(ssr), but I still need to write test cases for ssr compilation.

What an excellent work! 💖 Thanks for @wzc520pyfm
This is a big step!

What an excellent work! 💖 Thanks for @wzc520pyfm
This is a big step!

Hi @wzc520pyfm
Would you please share your Wechat ID so that I could invite you to our  core members' chat group

What an excellent work! 💖 Thanks for @wzc520pyfm
This is a big step!

Hi @wzc520pyfm
Would you please share your Wechat ID so that I could invite you to our  core members' chat group

Of course! My Wechat ID is wzc520pyf.


<EMAIL_ADDRESS>Decompress error

Installing 'windows-terminal' (1.18.3181.0) [64bit] [extras]
WARN  By installing you accept following license: MIT (https://spdx.org/licenses/MIT.html)
Loading Microsoft.WindowsTerminal_1.18.3181.0_x64.zip from cache.
Extracting Microsoft.WindowsTerminal_1.18.3181.0_x64.zip ... ERROR Exception calling "Start" with "0" argument(s): "The directory name is invalid"
ERROR Failed to extract files from <REDACTED>\scoop\apps\windows-terminal\1.18.3181.0\Microsoft.WindowsTerminal_1.18.3181.0_x64.zip.g

\scoop\apps\windows-terminal\1.18.3181.0\Microsoft.WindowsTerminal_1.18.3181.0_x64.zip.g
did you modify the manifest?

Cannot reproduce.
> scoop install windows-terminal -a 64bit
Installing 'windows-terminal' (1.18.3181.0) [64bit] from extras bucket
Microsoft.WindowsTerminal_1.18.3181.0_x64.zip (10.8 MB) [=====================================================] 100%
Checking hash of Microsoft.WindowsTerminal_1.18.3181.0_x64.zip ... ok.
Extracting Microsoft.WindowsTerminal_1.18.3181.0_x64.zip ... done.
Running pre_install script...
WARN  Portable mode is enabled from version 1.17.11391.0, please migrate the settings manually:
WARN  Original settings dir: 'C:\Users\qwerty\AppData\Local\Packages\Microsoft.WindowsTerminal_8wekyb3d8bbwe\LocalState'
WARN  Portable settings dir: 'C:\Users\qwerty\scoop\apps\windows-terminal\current\settings'
Running installer script...
Linking ~\scoop\apps\windows-terminal\current => ~\scoop\apps\windows-terminal\1.18.3181.0
Creating shim for 'WindowsTerminal'.
Creating shim for 'wt'.
Creating shortcut for Windows Terminal (WindowsTerminal.exe)
Persisting .portable
Persisting settings
Running post_install script...
'windows-terminal' (1.18.3181.0) was installed successfully!
Notes
-----
Add Windows Terminal as a context menu option by running `reg import
"C:\Users\qwerty\scoop\apps\windows-terminal\current\install-context.reg"`
'windows-terminal' suggests installing 'extras/vcredist2022'.

Nope? However, IDK why, but now it works OK. Didn't even redownload the archive.


[boost-regex, boost-system] build failure on x64-windows, arm64-windows

Host Environment

OS: Windows
Compiler: VC17

To Reproduce
Steps to reproduce the behavior:
./vcpkg install boost-regex:x64-windows
./vcpkg install boost-regex:arm64-windows
I see the same behavior on other boost packages that require compilation, e.g. boost-system
Failure logs
Console:
C:\Git\openscad>vcpkg install boost-regex:arm64-windows
Computing installation plan...
The following packages will be built and installed:
    boost-regex[core]:arm64-windows -> 1.81.0
Detecting compiler hash for triplet arm64-windows...
Restored 0 package(s) from C:\Users\steph\AppData\Local\vcpkg\archives in 128.3 us. Use --debug to see more details.
Installing 1/1 boost-regex:arm64-windows...
Building boost-regex[core]:arm64-windows...
-- Using cached boostorg-regex-boost-1.81.0.tar.gz.
-- Cleaning sources at C:/Git/vcpkg/buildtrees/boost-regex/src/ost-1.81.0-5d02094f20.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Git/vcpkg/downloads/boostorg-regex-boost-1.81.0.tar.gz
-- Using source at C:/Git/vcpkg/buildtrees/boost-regex/src/ost-1.81.0-5d02094f20.clean
-- Including C:/Git/vcpkg/ports/boost-regex/b2-options.cmake
-- Found external ninja('1.11.0').
-- Getting CMake variables for arm64-windows
-- Configuring arm64-windows
-- Building arm64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:131 (message):
    Command failed: C:/Git/vcpkg/downloads/tools/cmake-3.25.0-windows/cmake-3.25.0-windows-i386/bin/cmake.exe --build . --config Debug --target install -- -v -j9
    Working Directory: C:/Git/vcpkg/buildtrees/boost-regex/arm64-windows-dbg
    See logs for more information:
      C:\Git\vcpkg\buildtrees\boost-regex\install-arm64-windows-dbg-out.log

Call Stack (most recent call first):
  installed/arm64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
  installed/arm64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
  installed/arm64-windows/share/boost-build/boost-modular-build.cmake:137 (vcpkg_cmake_install)
  ports/boost-regex/portfile.cmake:12 (boost_modular_build)
  scripts/ports.cmake:147 (include)

error: building boost-regex:arm64-windows failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
    https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+boost-regex
You can submit a new issue at:
    https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[boost-regex]+Build+error
Include '[boost-regex] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
    vcpkg-tool version: 2022-12-14-7ae0d8527fb488fde10a89c2813802dc9b03b6f9
    vcpkg-scripts version: f9bea5d58 2022-12-31 (2 days ago)

install-arm64-windows-dbg-out
[1/2] cmd.exe /C "cd /D C:\Git\vcpkg\buildtrees\boost-regex\src\ost-1.81.0-5d02094f20.clean\build && C:\Git\vcpkg\installed\arm64-windows\tools\boost-build\b2.exe toolset=msvc --user-config=C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg/user-config.jam --stagedir=C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg/stage --build-dir=C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg runtime-link=shared link=shared address-model=64 architecture=x86 target-os=windows threadapi=win32 variant=debug --disable-icu --disable-icu --with-atomic --with-random --with-date_time --with-filesystem --with-system --with-thread --with-chrono -j 9 -sBOOST_ROOT=C:/Git/vcpkg/installed/arm64-windows/tools/boost-build -sBOOST_BUILD_PATH=C:/Git/vcpkg/installed/arm64-windows/tools/boost-build --debug-configuration --debug-building --debug-generators --ignore-site-config --hash -q debug-symbols=on -d +2 threading=multi stage"
FAILED: CMakeFiles/boost C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg/CMakeFiles/boost
cmd.exe /C "cd /D C:\Git\vcpkg\buildtrees\boost-regex\src\ost-1.81.0-5d02094f20.clean\build && C:\Git\vcpkg\installed\arm64-windows\tools\boost-build\b2.exe toolset=msvc --user-config=C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg/user-config.jam --stagedir=C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg/stage --build-dir=C:/Git/vcpkg/buildtrees/boost-regex/x64-windows-dbg runtime-link=shared link=shared address-model=64 architecture=x86 target-os=windows threadapi=win32 variant=debug --disable-icu --disable-icu --with-atomic --with-random --with-date_time --with-filesystem --with-system --with-thread --with-chrono -j 9 -sBOOST_ROOT=C:/Git/vcpkg/installed/arm64-windows/tools/boost-build -sBOOST_BUILD_PATH=C:/Git/vcpkg/installed/arm64-windows/tools/boost-build --debug-configuration --debug-building --debug-generators --ignore-site-config --hash -q debug-symbols=on -d +2 threading=multi stage"
notice: loading B2 from C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/kernel/bootstrap.jam
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/util\numbers.jam:23: in check from module numbers
error: 64 in 64 1024 64
error: is not a number
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/util\numbers.jam:50: in numbers.range from module numbers
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/tools/features\cxx-template-depth-feature.jam:33: in load from module cxx-template-depth-feature
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/kernel\modules.jam:294: in import from module modules
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/tools/features\__init_features__.jam:22: in load from module __init_features__
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/kernel\modules.jam:294: in import from module modules
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/tools\builtin.jam:16: in load from module builtin
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/kernel\modules.jam:294: in import from module modules
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src\build-system.jam:12: in load from module build-system
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/kernel\modules.jam:294: in import from module modules
C:/Git/vcpkg/installed/arm64-windows/tools/boost-build/src/kernel/bootstrap.jam:135: in module scope from module

ninja: build stopped: subcommand failed.

Additional context
My guess is that there's a missing compiler setting in msvc.jam, but I don't understand this well enough to try fixing it myself.

Digging further, it looks like the issue itself is with Visual Studio 2022 17.5.0 Preview 2.0.
This version of msvc does not compile boost-build correctly, breaking jam file parsing (and boost-build's tests fail too). Seems to work fine with VS 2022 17.4.3.

Filed VS feedback: https://developercommunity.visualstudio.com/t/C-VS-175-Preview-2-compiles-boost-b/10243019?

@MonicaLiu0311 this is not a community triplet issue.

Close this issue because it is currently tracked by Developer Community.


package.json "rollup": "^4.21.3",

          "rollup": "^4.21.3",

Originally posted by @Ydoug36 in https://github.com/Shopify/shopify-app-js/pull/1495#discussion_r1794937599

      "rollup": "^4.21.3",


Improve flexibility for other triple extractor implementations

[x] Include API-like class for extracting Triples. Analyzer is now the API, of which CFGAnalyzer has been implemented
[x] Include Stanford OIE service as OIEAnalyzer
[x] Update logger format
[ ] Include many triples to one utterance:

[ ] To be discussed: In the Utterance class, do we make the triple property a list so it can contain many triples? Then , when creating capsules, do these get represented as different capsules?

@numblr could you please check the changes to the log? The configurations are now under a folder config. I am guessing these need to be added in the setup.py?


[v6.2] Item context menu bugs

Opening the item context menu only works occasionally in explorable areas, and not at all in outposts.
When it does work, it just keeps opening the context menu for the first item I right-clicked on, even if I try other items afterwards.
It also opens around the middle of my screen rather than under the item.

Oh so that's what that was for :o


Release tagging

Lighthouse already seems pretty stable to me, but many Linux distributions only allow packages with release tags.
Besides that, it is easy to distinguish between major and minor changes when you follow a common version scheme.
What do you think about it?
Jannis

SemVer is very common, the summary explains it very well.
We also need to keep this rule in mind for a starting point:

Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.

I would suggest 1.0.0 but you know the development best :)


Remove sudo, fix quoting issues

Resolves #1.
Adds quotes to handle case where ZFS filesystem name contains spaces
Specifies python2 for environment.

Thanks.


Scenario outline with  from Examples section in title.

Hello, guys.
I've found a possibility of using the  from Examples section in the Scenario Outline title in the Cucumber: https://github.com/cucumber/cucumber-js/issues/267
I've tried to use this in behat, but it does not work for me.
In PhpStorm when I use this parameter in title, it is highlighted and 'go to source' command moves the cursor to the same name column in Examples section.
When tests are run, I see the parameter is not substituted to its value, just stayed as it is,
e.g.
Scenario Outline: Login with <email>.
Seems like this feature is not implemented yet in behat, may be it will be a good idea to do it.

Is it just for title customizing or it also will be a step implementation?

As a step implementation this feature is already exist. So just for title customizing.

Well, I can't see how the Scenario Outline: Login with <email>. would have it's token replaced. From which Example line would it get its replacement?
For the pretty formatter the ExampleNode titles are not even printed, only the OutlineNode title.
On the other hand, I believe it makes sense in case any other formatter uses the ExampleNode title.
Anyway, this would be done on the Gherkin package (https://github.com/Behat/Gherkin/blob/master/src/Behat/Gherkin/Node/OutlineNode.php#L207), not on Behat core.
You should close this issue and open another one on https://github.com/Behat/Gherkin

For the pretty formatter the ExampleNode titles are not even printed, only the OutlineNode title.
Correction: ExampleNode titles are printed, and they're the corresponding line on the examples table

@bwowk once again - thanks for nudging in :)


docs(contributing): create contributing guidelines

Coverage remained the same at 89.925% when pulling 366c5739bfc653a5d364b0ec67c75550e2fe753b on wizzy25-patch-1 into 0518460a7ef96b979eb02818ddcbaeae1ec52ab4 on master.

Coverage remained the same at 89.925% when pulling 366c5739bfc653a5d364b0ec67c75550e2fe753b on wizzy25-patch-1 into 0518460a7ef96b979eb02818ddcbaeae1ec52ab4 on master.


Update sbt-native-packager to 1.8.1

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

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

labels: sbt-plugin-update, semver-minor

+1

[X] Do all automated continuous integration checks pass?
Yes
[X] Is the update a patch, minor, or major update?
Minor update, diff shows few changes, made of up documentation, automated relase process, and features that we don't use (e.g. docker, graal)
[X] Is the license still compatible with ASF License Policy?
Yes, still BSD-2. Note that we do not distribute this plugin with daffodil
[X] Have any changes been made to LICENSE/NOTICE files that need to be incorporated?
No changes to LICENSE. No NOTICE file
[X] Have any transitive dependencies been added or changed?
No changes to transitive dependencies


Built-in Parts

com.clarkparsia.sbol.editor has a file called cmyk_parts.rdf.  This might contain the "Built-in parts", which is a feature that should be reimplemented.

Local sbol files can now be added as registries.  Remaining task: get all iGem parts and include as default "built-in parts".

Built-in parts temporarily represented by GeneticToggle.sbol.

Let’s rename to BuiltInParts.xml
It can be same content for now though.
Chris

On Jul 1, 2016, at 12:49 PM, Michael Zhang<EMAIL_ADDRESS>wrote:
Built-in parts temporarily represented by GeneticToggle.sbol.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/SynBioDex/SBOLDesigner/issues/40#issuecomment-230021528, or mute the thread https://github.com/notifications/unsubscribe/ADWD99qvj5t-_7lv0XbZ4Ie-xxLG6Q7sks5qRWEzgaJpZM4I8L_B.

Done.

Swapped with previous BuiltInParts.  We should include a better dataset in the future.  Moving milestone to 2.1.

Replaced with other issue


How to display math vertical multiplication??

Hi, @kostub
Thanks for sharing this wonderful library,
I'm here to know about,
You can see the image below. Can i show this type of equations using this library ??

Thanks in advance.

Hey... @punitsilversky
do you know how to manage long latext content in automatically second line because i also use this Library and if Content is long then it is going out of frame. And Also not support Html taggs like img type .
please Help if you know anything

Hey @iOSharsh,
Sorry, I haven't manage long latex content
Thanks


cmd/go: vendor is ignored for packages inside $GOPATH

What version of Go are you using (go version)?
go version go1.9.1 linux/amd64
What did you do?
gopath.tar.gz
I have the following structure:
/tmp/go/src/app$ tree -L 3 ../
../
├── app
│   ├── main.go
│   └── vendor
│       └── github.com
└── shared
    └── toml
        └── toml.go

$ echo $GOPATH
/tmp/go
When building:
/tmp/go/src/app$ go build main.go
../shared/toml/toml.go:4:9: cannot find package "github.com/BurntSushi/toml" in any of:
/usr/lib/go/src/github.com/BurntSushi/toml (from $GOROOT)
/tmp/go/src/github.com/BurntSushi/toml (from $GOPATH)
vendor/ directory contains needed package.
main.go imports "shared/toml" package which in turn uses the needed vendored package.
The only way to build main.go is by copying the same "vendor" dependency inside shared/toml/ which seems overkill and duplication.
Does the compiler ignore the vendor directory? I.e. some sort of "lexical" scope since the parsed file at the moment (shared/toml) don't have a sub-directory vendor, but the initial compilation process starts from main.go.
If that is a current limitation and not my misuse, then that means if I want to have a complete insulation from "system" $GOPATH (and use my own) and keep all dependencies (normally collected with "go dep" / dep ensure) means I have to copy that vendor directory to all $GOPATH/src/modules
What did you expect to see?
Using the vendor path for building the app and imported packages inside $GOPATH
What did you see instead?
Missing package, already available in the vendor directory
/tmp/go/src/app$ go build main.go
../shared/toml/toml.go:4:9: cannot find package "github.com/BurntSushi/toml" in any of:
/usr/lib/go/src/github.com/BurntSushi/toml (from $GOROOT)
/tmp/go/src/github.com/BurntSushi/toml (from $GOPATH)

Thanks for providing a complete test case.
Quoting https://golang.org/cmd/go:

Code below a directory named "vendor" is importable only by code in the directory tree rooted at the parent of "vendor", and only using an import path that omits the prefix up to and including the vendor element.

In this case shared/toml/toml.go is trying to import github.com/BurntSushi/toml, but that directory is only available in app/vendor, not in shared/vendor.
Closing because this is working as designed.

My real case is:
company.com/protobuf-idl
site.org/app1
I have "shared" contract/interface package(s) and go applications that are using them.
Since I want to vendor my app1, I have a vendor directory inside site.org/app1, however to build app1 executable I also need to have vendor, which contains the same packages (github.com/golang/protobuf).
So, the expected behavior is to do "dep init" and "dep ensure" in each of the packages, right?

We use the issue tracker for bugs and proposals, not discussion.  Let's take questions to the mailing list, where you find better answers anyhow.  See https://golang.org/wiki/Questions .  Thanks.


TrRouting: Use the new v2/route endpoint of trRouting

It's ok, but we seem to do a lot of work to keep the V1 stuff around. Yes it makes for simpler commit, but there will be a lot of scafolding to remove afterward.

We have to keep it, the v1 is used for everything, including the simulation odTrip,s calculation, so it will be around for a while still... And i don't think there's that much scaffolding to remove later.
You're probably right about keeping v2 in backend, but we'll have to handle other types of responses from the backend as well I guess and provide a single API for errors. Not trivial, but not hard either, I'll open an issue for it, before doing the accessibility endpoint.

I really hope we don't have to keep V1 for a while. I really hope to kill it in TrRouting as soon as possible!

Depends on you or me implementing the v2/odTrips in trRouting... ;-)


[Button] Use deeper imports from unstyled, correct docs

Small corrections discussed in https://github.com/mui-org/material-ui/pull/27600#pullrequestreview-741612468
Preview https://deploy-preview-28074--material-ui.netlify.app/components/buttons/#unstyled

Could we wait for the package rename changes to go in before we merge this one?

Sure, no problem

I think that it fails because of https://github.com/mui-org/material-ui/blob/bbc5561f360985430fc6ef4f81eca0ce093eaf98/packages/material-ui/scripts/rollup.config.js#L85 while we have a .ts file not a .js one.

I guess we could test if the ts file is present, return it and default to the .js file if not.

Fixing in https://github.com/mui-org/material-ui/pull/28094

Looks great, thanks for the follow up.

@oliviertassinari can I have approval, then? 😉

Thanks :)


builder: fix go-licenses by installing later in the process

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jaxesn
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 [jaxesn]

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


Expectiminimax algorithm not generic for all stochastic games

The current implementation of Expectiminimax algorithm is limited to the stochastic games where rolling a pair of dice is the probabilistic event. It needs to be made generic for all stochastic games so that it can handle the any type of probabilistic event as defined by the game.

Sounds good!


[DOCFIX] Add API doc for getJobStatus

@yuzhu do you know the best way to test this?

Merged build finished. Test PASSed.

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

@yuzhu

@yuzhu

alluxio-bot, merge this please


MachXO3D initial

Initial work on MachXO3D fuzzing, all based on MachXO3 with few improvements.
Still require work on specifics for this architecture

This is now ready to be reviewed, and feature supported are same as on MachXO3


Code reorganization

Splitting code into multiple files

Acceptance Criteria:

There should be no commented code that is not otherwise needed.
Commonly accessed namespaces should be included into the file instead fo constantly accessed from the namespace itself.

Closed by #6


feat: add the ability to configure typescript to javascript file extension conversion

Reference #108
This adds the ability to configure the ts<>js extension mapping.
This is configured using the following:

Check the rule configuration typescriptExtensionMap
ESLint shared setting node.typescriptExtensionMap
Use .ts <=> .js, .cts <=> .cjs, .mts <=> .mjs, .tsx <=> .js

This is draft because I am not sure how to add the mapping to n/file-extension-in-import

@aladdin-add Do you want me to make the tsconfig detection mechanism PR now, or wait for you to review this PR? 👀

sorry, haven't gotten time to review. will try to do it in the week. :)

Do you want me to make the tsconfig detection mechanism PR now, or wait for you to review this PR? 👀

Yes, please feel free to. You can rebase this change when it's been merged.