How to set the key's index and iterator?

My key was timestamps ,and i want use the timestamps key iterator,Hao to do it?
however the  Index can only be set the key of values ?

Indexes can only be assigned to values. You cannot add an index to a key. The key is always ordered by it's binary value.
You will need to format your timestamp so it's sortable.
For example:
 2017-09-11T10:35:21.9082
 2017-09-11T10:35:28.8293
 2017-09-12T08:05:13.0012

Or if you want to make the best use of space you can store the nanoseconds representation an int64 or time.Time.
The following serializes a time.Time object into an 8 byte key with nanosecond precision.
key := make([]byte, 8)
binary.LittleEndian.PutUint64(key, timestamp.UnixNano())

Then add the key/value to the database.
tx.Set(string(key), "some value", nil)

@tidwall  i want use  Multi Value Index with json,like this ‘{"product":"RPM","level":"2"}’ ,i want the equal this pattern ,how todo it?

db.CreateIndex("index_name", "*", buntdb.IndexJSON("product"), buntdb.IndexJSON("level"))

please see https://github.com/tidwall/buntdb#multi-value-index

@tidwall  i see set Multi Value Index,but how can ascend equal this two conditions？

Please see #18 for an example of how to ascend on a multi value index.

I'm very sorry ，I have a lot of questions，Thank you for your answer patiently。
i have this json data
{     "product": "RPM",     "level": "2",     "modular": "DP",     "event": "1",     "origValue": "",     "currentValue": "测试",     "operatorKey": "9527",     "operatorAt":1502676106,     "operatorName": "周星星",     "operatorIP": "<IP_ADDRESS>",     "operatorClient": "1",     "sqlSentence": "select * from dual",     "sqlParameter": "[1,2,3]" },{     "product": "RPM",     "level": "5",     "modular": "DP",     "event": "1",     "origValue": "",     "currentValue": "测试",     "operatorKey": "9527",     "operatorAt":1502676106,     "operatorName": "周星星",     "operatorIP": "<IP_ADDRESS>",     "operatorClient": "1",     "sqlSentence": "select * from dual",     "sqlParameter": "[1,2,3]" }{     "product": "RPM",     "level": "6",     "modular": "DP",     "event": "1",     "origValue": "",     "currentValue": "测试",     "operatorKey": "9527",     "operatorAt":1502676105,     "operatorName": "周星星",     "operatorIP": "<IP_ADDRESS>",     "operatorClient": "1",     "sqlSentence": "select * from dual",     "sqlParameter": "[1,2,3]" },{     "product": "RPM",     "level": "3",     "modular": "DP",     "event": "1",     "origValue": "",     "currentValue": "测试",     "operatorKey": "9527",     "operatorAt":1502676104,     "operatorName": "周星星",     "operatorIP": "<IP_ADDRESS>",     "operatorClient": "1",     "sqlSentence": "select * from dual",     "sqlParameter": "[1,2,3]" }{     "product": "RPM",     "level": "1",     "modular": "DP",     "event": "1",     "origValue": "",     "currentValue": "测试",     "operatorKey": "9527",     "operatorAt":1502676102,     "operatorName": "周星星",     "operatorIP": "<IP_ADDRESS>",     "operatorClient": "1",     "sqlSentence": "select * from dual",     "sqlParameter": "[1,2,3]" },{     "product": "RPM",     "level": "4",     "modular": "DP",     "event": "1",     "origValue": "",     "currentValue": "测试",     "operatorKey": "9527",     "operatorAt":1502676102,     "operatorName": "周星星",     "operatorIP": "<IP_ADDRESS>",     "operatorClient": "1",     "sqlSentence": "select * from dual",     "sqlParameter": "[1,2,3]" }
i want to query use product&level&modular&event fields  and user operatorAt field sorting,how to do this?


Add "Expected price impact" or similar when removing liquidity

A lot of removing liquidity transactions are failing because we have a minimumReceived.
During spikes, that minimumReceived is changing a lot, so it makes sense to have a slippage selector here as well

The UI we have on the Trade tab, which should be the same then on the Remove Liquidity screen within Pool tab:

We have two use cases here simple and advanced and we will have to handle them separately.

Simple
In this case we will probably need to rethink the design/usage because currently we specify how many shares we want to remove and based on that we show the user how many ocean that means. Unfortunately that was misunderstood. ocean.pool.removeOceanLiquidity requires a number of ocean tokens that you want to remove (not shares) and then you can calculate how many shares you need for that amount of ocean and then you add the slippage to that pool share amount.
The steps would be:

users inputs amount of ocean he wants to remove oceanAmount
we calculate how many shares are needed for that : poolSharesNeeded = getPoolSharesRequiredToRemoveOcean(poolAddress, oceanAmount)
we calculate the slippage based on the value the user selected in the slippage control (see above image from @kremalicious ) :
 maxShares = poolSharesNeeded  * (100 + slippage)
we remove the ocean: removeOceanLiquidity( accountId, poolAddress, oceanAmount, maxShares)

Advanced


leaflet-measure with angular-material

Whenever I npm install the dependency, it also removes the angular-material files that I have. Do these packages not work with each other?

I got leaflet-measure to work without Angular Material (which is fine, albeit an inconvenience). However, the images icons do not show on the map.

@maihannijat Have you ever had a problem with L.Control.Measure is not a constructor?
I'm wondering if I'm bringing it into my component wrong. Currently bringing it in like this:
declare let L: any;
import 'leaflet-measure';

@maihannijat Have you ever had a problem with L.Control.Measure is not a constructor?
I'm wondering if I'm bringing it into my component wrong. Currently bringing it in like this:
declare let L: any;
import 'leaflet-measure';

I got the same issue. Ended up defining my own typings based on a post in another thread here. My project sometimes decide not to complain about it anymore.

Have you ever had a problem with L.Control.Measure is not a Property 'Measure' does not exist on type 'typeof control'

@satas12825 i have the same issue, did you find the solution?


Create trigger/stored procedure to add a new subdomain

💡 Summary
Create a function that will create a subdomain and deal with possible edge cases.
Motivation and context
Creating a subdomain can be tricky especially when the root domain doesn't exist, we want to create a function (either trigger or stored procedure) that will streamline this process. it will create the subdomain if the root exists, if the root doesn't exist it will create the root and then create the subdomain
Implementation notes
Create SQL function/ stored procedure that can quickly add a subdomain to the database and create a root domain if necessary

Created initial stored procedure to add new sub_domains. However when trying to implement it into our workflow discovered it may be necessary to convert it to a postgresql function instead

Changed this to a postgres function that adds a subdomain, and if the root doesn't exist yet, it will create that as well


Evergreen bundle missing in dist

Hi, thanks for the great idea to add an evergreen bundle! However, it is stated that an "zone-evergreen.js" bundle can be find in the dist directory, but I couldn't find it : is this an error on my side, or is it really not here? Thanks a lot for your help.

@benjamincombes, this changes have not been released, after next version 0.9.0 released, it will under dist.

Hi @JiaLiPassion! As it was mentioned on the main page of the project, I thought it was already available, thanks for your answer.


fail to convert [KNIFT]

Issue Type
Bug
OS
Ubuntu
OS architecture
x86_64
Programming Language
Python
Framework
TensorFlow, TensorFlowLite
Download URL for tflite file
https://github.com/google/mediapipe/blob/master/docs/solutions/models.md#knift
Convert Script
tar="KNIFT"
option="--flatc_path ../flatc --schema_path ../schema.fbs"

./../flatc -t --strict-json --defaults-json -o ./${tar}/ ../schema.fbs -- ${tar}/knift_float.tflite

tflite2tensorflow --model_path ${tar}/knift_float.tflite ${option} --output_pb

Description
I tried to convert mediapipe KNIFT model but it failed due to channel size mismatch between input and filter.
Is there a mistake in my procedure? If no, could you fix the bug?
Relevant Log Output
Traceback (most recent call last):
  File "/usr/local/bin/tflite2tensorflow", line 6614, in <module>
    main()
  File "/usr/local/bin/tflite2tensorflow", line 5881, in main
    TFLite_Detection_PostProcess_flg = make_graph(
  File "/usr/local/bin/tflite2tensorflow", line 720, in make_graph
    output_tensor = tf.nn.depthwise_conv2d(
  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/util/traceback_utils.py", line 153, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/framework/ops.py", line 1963, in _create_c_op
    raise ValueError(e.message)
ValueError: Dimensions must be equal, but are 1 and 16 for '{{node depthwise}} = DepthwiseConv2dNative[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], explicit_paddings=[], padding="SAME", strides=[1, 1, 1, 1]](rgb_to_grayscale_1, depthwise/filter_in)' with input shapes: [200,32,32,1], [3,3,16,1].

Source code for simple inference testing code
No response

Thank you for your reply.
I can convert KNIFT model using the procedure you proposed.


bugreport: invalid handling of negation of aromaticity when parsing SMARTS

>>> from rdkit import Chem
>>> Chem.MolToSmarts(Chem.MolFromSmarts("[#6&!a]"))
'c'

expected output would be "C", as the semantics of [#6&!a], according to Daylight spec is "carbon and non-aromatic". Of course a chemist probably wouldn't write "[#6&!a]" instead of "C", however such SMARTS are sometimes generated by various automatic chemoinformatics scripts, which then get confused. Same thing happens with explicit aliphatic specifications:

>>> Chem.MolToSmarts(Chem.MolFromSmarts("[#6&!A]"))
'C'

Verified on the current git repo.

Confirmed.
Thanks for reporting it.

This fix is relatively straight forward, but I might not understand the ramifications.  However, it looks like you really keep the original smarts forms in these cases of negation, if this is the case ( and I think it is ) then I think the is low risk.


Add line continuation for the WASM demo terminal

This is an old branch from a few months ago (I just forgot to make a PR), but it still works and there are no merge conflicts!

My javascript-fu is weak, but this seems ok to me.


No local costmap built

I was investigating the teaching branch in order to support multiple robots and I noticed that, when using the package fake_localization no local costmap is built. Probably something is wrong in the yaml files.
I will check again the main branch to see if it's working there with Thorvald (it's working for Vogui).

This problem is fixed in #39 which will also be merged into the master branch. I will leave the issue open until the merge.


Refactor Data Tests to Prevent Copy Paste Coding

The test cases for SQLite, in Test.KronoMata.Data.SQLite, were cut and paste from the tests written in Test.KronoMata.Data.Mock. Additional test cases were added for SQLite but not for the Mock implementation.
The IDataStoreProvider and supporting IDataStoreXxx interfaces define the expected behavior of implementations and those are what should be tested against.
Refactor the tests in a manner that allows for testing implementations of IDataStoreProvider with minimal code duplication. Use the SQLite tests as the template and then refactor Test.KronoMata.Data.SQLite and Test.KronoMata.Data.Mock to use the common tests.
Take the following test case that looks identical in both sets of tests:
public void Can_Create()
{
	var now = DateTime.Now;

	var scheduledJob = new ScheduledJob()
	{
		PluginMetaDataId = 1,
		HostId = 1,
		Name = "Name",
		Description = "Description",
		Frequency = ScheduleFrequency.Week,
		Interval = 2,
		StartTime = now,
		EndTime = now,
		IsEnabled = true,
		InsertDate = now,
		UpdateDate = now
	};

	_provider.ScheduledJobDataStore.Create(scheduledJob);

	Assert.That(scheduledJob.Id, Is.EqualTo(1));
}

There should be an elegant way to just pass the _provider to a shared template like the following:
public void Can_Create()
{
	var scheduledJob = SomeCommonScheduledJobCan_CreateMethod(_provider);
	Assert.That(scheduledJob.Id, Is.EqualTo(1));
}

Verified all tests pass after refactoring.


ngmodel missing

Error
Uncaught (in promise): Error: Template parse errors:
There is no directive with "exportAs" set to "ngModel"
-->[(ngModel)]="model.username"  not work
Please help

sorry my mistake..


hash and deploy-url for media

🚀 Feature request
Command (mark with an x)

- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
 A clear and concise description of the problem or missing capability...
now only scripts and media referenced by stylesheets can be hashed.
media in component html like <img src="..."> can't be hashed, both it in or not in assets folder.
deploy-url not affect them too.
which is harder to read and may have trouble to deal with dynamic
Describe the solution you'd like
 If you have a solution in mind, please describe it.
media referenced by html need to be hashed, just like them referenced by css.
in my deployment only index.html served from main domain and all other static files are from CDN. so hash and deploy-url support is important.
for backward compatibility, it could behind a build config.
Describe alternatives you've considered
 Have you considered any alternative solutions or workarounds?

Duplicate of https://github.com/angular/angular-cli/issues/3415


Logging and Tracing

Knowing what went wrong when errors occur is somewhat difficult right now. Ideally, we would have logging and tracing to be able to easily correlate logs.

See https://docs.rs/tracing/latest/tracing/


Migrate plugin to Taverna 3

Updated 52North library to latest version with Apache license
Created GisClient interface and implementation
Added GisClientFactory
Implement GisActivityFactory
Improve GisActivity configuration

@edikaradumi, what could be missing? We got build errors..
(Thanks @redmitry for testing!)
COMPILATION ERROR :
-------------------------------------------------------------
org/apache/taverna/gis/GisActivityHealthChecker.java:[57,60]
incompatible types: org.apache.taverna.gis.GisActivityConfigurationBean
cannot be converted to com.fasterxml.jackson.databind.JsonNode
org/apache/taverna/gis/GisActivityFactory.java:[52,46] cannot find symbol
   symbol:   variable ACTIVITY_TYPE
   location: class org.apache.taverna.gis.GisActivity
org/apache/taverna/gis/GisActivityFactory.java:[72,62] cannot find symbol
   symbol:   variable IN_FIRST_INPUT
   location: class org.apache.taverna.gis.GisActivityFactory
org/apache/taverna/gis/GisActivityFactory.java:[77,70] cannot find symbol
   symbol:   variable IN_EXTRA_DATA
   location: class org.apache.taverna.gis.GisActivityFactory
org/apache/taverna/gis/GisActivityFactory.java:[91,72] cannot find symbol
   symbol:   variable OUT_REPORT
   location: class org.apache.taverna.gis.GisActivityFactory
org/apache/taverna/gis/GisActivityFactory.java:[95,64] cannot find symbol
   symbol:   variable OUT_SIMPLE_OUTPUT
   location: class org.apache.taverna.gis.GisActivityFactory
org/apache/taverna/gis/GisActivityFactory.java:[97,64] cannot find symbol
   symbol:   variable OUT_MORE_OUTPUTS
   location: class org.apache.taverna.gis.GisActivityFactory
7 errors

Hi Stian,
I'll have a look later today. It has been a while since I worked on it


"stack haddock" fails when using system GHC

See this StackOverflow question. In short, the following fails with an error about base not being in the build plan on a fresh 64-bit Windows 7/8 MinGHC 2015-08-13 installation:

stack new stack-test new-template
cd stack-test
stack haddock

Changing "stack haddock" to "stack haddock --install-ghc --no-system-ghc" makes it work.

I just tested, and I'm unable to reproduce the problem. Can you capture the full output of the following commands and put them in a paste:
stack path
stack exec which ghc
stack haddock --verbose
ghc-pkg describe base --expand-pkgroot
stack exec -- ghc-pkg describe base --expand-pkgroot

Interesting, it only happens when I install MinGHC in a directory in C:\Program Files. If I let the installer use the default install location in AppData\Local\Programs, the issue doesn't occur. However, if I install in Program Files, the issue occurs even if I run the commands with admin privileges.
Output from the commands you listed: https://gist.github.com/arotenberg/030e00254737a98496b5

That certainly narrows it down, sounds like a bug in stack's Haddock code not knowing how to handle quoted filepaths. We'll get that fixed for the next release (pinging @borsboom), but it looks like you at least have a decent workaround until then.

OK, I've pushed a commit to stack that addresses this problem. If you build from master, stack should work with your install of MinGHC.


Provide option to hide PlotControls

First, thanks for the great project!  We're working hard to incorporate this into a new app for biologists running from within the Cytoscape platform.  Once option that would be really helpful is to provide the ability to hide the plot options panel, sort of similar to the way that the plotly chart studio does it.

I'm glad you're finding this useful!
Are you using <PlotlyEditor>? If so, there is a hideControls prop already you can use, but it doesn't render any UI widget that turns it on or off, mostly because it's hard to decide where such a widget should live. In other projects we've positioned it absolutely in the bottom left hand corner but people haven't been uniformly thrilled with it there.

Thanks for the suggestions!  We wound up doing pretty much the same thing.  We are using , but we wanted users to be able to show and hide the controls interactively, so we just added a new button with absolute positioning.  It works pretty well for now.

For what it's worth, I put a button in the mode bar to toggle visibility of the controls in my application. When the controls are open, there is also a more visible <Button> in a <SingleSidebarItem> to hide the controls. That seems to work well for my use case.

I'm struggling with how to get that to work in react. It seems that when I call the function assigned to click, it cannot see that state of the component. I'm new to Javascript so would appreciate any assistance.


Added zh-mo.js

Coverage decreased (-0.07%) to 88.336% when pulling b57cd99ff42cf6b8c41ac09506f2edf4ca788b52 on inpovin:develop into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Coverage decreased (-0.07%) to 88.336% when pulling b57cd99ff42cf6b8c41ac09506f2edf4ca788b52 on inpovin:develop into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Coverage decreased (-0.07%) to 88.336% when pulling b57cd99ff42cf6b8c41ac09506f2edf4ca788b52 on inpovin:develop into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

@inpovin your code needs to be put under src/locale/* and tests under src/test/locale/*.


Migrations aren't propagated to existing connections

EdgeDB Version:1.0-alpha.8+dev.5386.g6933b454f
Steps to Reproduce:

Start edgedb (repl) and do a query:

edgedb> SELECT Counter;
error: object type or alias 'default::Counter' does not exist
  ┌─ query:1:8
  │
1 │ SELECT Counter;
  │        ^^^^^^^ error

Then in another shell do edgedb migrate to create the type
Then return back to this shell and confirm that error still there (while it shouldn't be):

edgedb> SELECT Counter;
error: object type or alias 'default::Counter' does not exist
  ┌─ query:1:8
  │
1 │ SELECT Counter;
  │        ^^^^^^^ error

Also try and confirm that it works in a new connection:

edgedb> SELECT Counter;
{}

Note: this doesn't reproduce with CREATE TYPE only with migrations.

This would be fixed by the rework in the iocomp branch.

This is fixed.


Collapse options into a single accordion

https://github.com/kylebarron/all-transit/commit/959b78ef0b62febeec1564daa3d4977cad43c1e5


Sync k3s and rke2 images

Sync rke2 and k3s images synced from docker hub to the prime registry
All versions supported by rancher v2.7 and newer.
Make sure to sync rke2 images and images in the rke2-images-all.* artifacts

kdm already syncs images using regsync


Integration Library with ASP.NET Core?

Do you have plans to create an ASP.NET Core integration library for this? If you want to create the project I could submit a PR

it's built-in in core.

In what Core? IdentityModel or ASP.NET Core? I'm thinking of the use case when using net451 in a new core project. Similar to this guy: https://github.com/IdentityServer/IdentityServer3.AccessTokenValidation.Integration.AspNetCore

In ASP.NET Core this is built-in.


I will dig deep into zipcode level data. Let's start with cumberland county.

1-3 tiers are slow
4-5 tiers are fast

divide the tier1-5 into zipcode boundary first. save them to each zipcode geograpy folder
do the same thing to the census data


Investigate the flake in the E2E tests

This run has failed on main, that's unacceptable:
https://github.com/aabounegm/cast/runs/6322279300?check_suite_focus=true

For now, I side-stepped the issue by adding up to two retries to that flaky test. Couldn't really figure out what was causing it to fail


Kafka topic is not created when provisioning the Kafka Cluster

I try to run the example to provision a Kafka cluster in the Confluent Cloud and create a topic.
The environment and cluster are successfully created but the topic is not created. It looks like the Kafka broker is not yet ready to create the topic.
I'm using:

Terraform v0.13.4
provider registry.terraform.io/mongey/confluentcloud v0.0.5
provider registry.terraform.io/mongey/kafka v0.2.10

terraform apply output:
confluentcloud_environment.environment: Creating...
confluentcloud_service_account.test: Creating...
confluentcloud_environment.environment: Creation complete after 1s [id=env-9pk7v]
confluentcloud_kafka_cluster.test: Creating...
confluentcloud_kafka_cluster.test: Creation complete after 0s [id=lkc-9g3ky]
confluentcloud_api_key.provider_test: Creating...
confluentcloud_api_key.provider_test: Creation complete after 1s [id=145974]
kafka_topic.syslog: Creating...

Error: Provider produced inconsistent result after apply

When applying changes to confluentcloud_service_account.test, provider
"registry.terraform.io/mongey/confluentcloud" produced an unexpected new
value: Root resource was present, but now absent.

This is a bug in the provider, which should be reported in the provider's own
issue tracker.

Error: kafka: client has run out of available brokers to talk to (Is your cluster reachable?)

  on main.tf line 54, in resource "kafka_topic" "syslog":
  54: resource "kafka_topic" "syslog" {

When I run terraform apply for the second time the topic will be created.
Is there a way to wait until the Kafka cluster is ready to create a topic?

This is a know issue ... I have started to look at it in #37

Thanks @Mongey

Hey I'm getting the same error when trying to create a an API key with a service account

@ryandutton Yeah I believe there's a similar issue with API Keys.... they aren't ready to be used immediately after creation. I'm experimenting with having the provider wait until the key is active -- by having the provider repeatedly attempt to connect to the cluster. That's going on in #37, and if successful there, I can port that to API Key creation.

It might be a different issue but I was getting similar results where I would create an api key using the confluentcloud provider and then use that to authenticate the Kafka provider. What I found was that there was insufficient permissions on the service account key to allow the creation of a topic (or an act to allow topic creation). Once I manually assigned cluster and topic create permissions in the UI, I could successfully run the terraform to create the topics.

Not sure if possible in the confluentcloud provider but if you can create acl for cluster and topic creation and read then that will allow the Kafka provider to have sufficient permissions to create topic.

Not sure if possible in the confluentcloud provider but if you can create acl for cluster and topic creation and read then that will allow the Kafka provider to have sufficient permissions to create topic.

I'm also looking for this to be integrated in the terraform plugin

Does anybody know if this is resolved after a year since has post?  I still get the same error.
confluent kafka topic produce topic-filing-alert
Error: failed to obtain topics from client: API key may not be provisioned yet

Edit: I asked my network admin to open port :9092 then I could connect and did not get the “API key may not be provisioned yet” error.


Fix accordion and tab targets with special meaning characters

This fixes the issue of special meaning characters in id's on togglable tabs, and collapsible panels without the need for escaping the id's.
Refs: https://github.com/twbs/bootstrap/issues/14285, https://github.com/twbs/bootstrap/pull/13832

Another weird case is IDs with a colon. For example, Hugo will generate section anchors with IDs like #SOME_TITLE:13f5f3bf.
While this is a valid "href" attribute for an anchor, passing it to jQuery directly will not find the element. However, $("#SOME_TITLE\\:13f5f3bf") will work.


Load Tensorflow Model from HDFS

dependent on the outcome of this issue:
https://github.com/tensorflow/serving/issues/359

merged


Component: Hero Slider

Hero Slider
This component will include the image, slider functionality and the three dots at the bottom. It does not need to include the caption or the button. Those components will be added to the molecular Hero component.
Desktop:

Mobile:

Closed by #169


Enable warning support explicitly

When enabling RSpecs warning mode, it should include the keyword argument deprecations from Ruby.
See #2809 for the background for this.
(Fixes #2809)

@eregon are you ok with this change?


src/components/tx-events: getTicker returns unique abbreviation that may not match actual token symbol

Context: https://github.com/hirosystems/explorer/blob/397eaaa3ca858e64dfd38af0d39e58b6176ab5fa/src/components/tx-events.tsx#L20
Hello. 👋  We've been in the process of integrating Wrapped Bitcoin (xBTC) with a large crypto exchange, and there was some confusion between our teams because in some places in the explorer, Wrapped Bitcoin is referenced as WBI (the symbol defined in our contract is xBTC); they thought there might be two different tokens intermingling with one another. I also noticed in another case that a user in one of the communities was asking why the symbol for Wrapped Bitcoin is WBI. We were concerned there might have been an issue with the initialization of the contract / symbol, however, it seems the above reference may be the reason.
So just wanted to open this issue to see what people's thoughts on a practical solution might be, and if it's possible to give precedence to the symbol defined in the deployed contract, as opposed to creating a unique ticker that might not match.
Additional info
Contract:
SP3DX3H4FEYZJZ586MFBS25ZW3HZDMEW92260R2PR.Wrapped-Bitcoin

Related to https://github.com/hirosystems/explorer/issues/1088

This should be fixed since we are using lunar crush now


migrate from ecdsa and secp256k1 to coincurve

i think we should just migrate to coincurve - it is well maintained (last secp256k1 release was in 2021) - and even CLN uses coincurve and not secp256k1 for their python plugins
it would be great to also drop the ecdsa dependency and use only coincurve

I've never been a fan of this library coming with a signature library by default. How about switching to coincurve and making it optional?

I want to reduce the number of libraries used by lnbits, not to increase it.

Done in https://github.com/lnbits/bolt11/pull/46 and released via 2.1.0


The Coin Change Problem in C++

Related Issue
Solved issue number: #392
Describe the changes you've made
Adding Source Code for The Coin Change Problem in C++
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C++
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 documented the code.
[x] New and existing unit tests pass locally with my changes.

Please review the code ;-)


Test if microversion value not nil and not empty

This PR fixes https://github.com/fog/fog-openstack/issues/359

@shoudusse, LGTM besides the operator change. Thanks

@shoudusse, Please do not merge your own code. I know it's tempting and we have all been there (including me!).
Although sometimes there are exceptions such as emergencies (broken CI, etc) and cases where it doesn't impact the code directly such as documentation or tests.
Also we don't enforce 2 peers review, it's a good practice especially about complex issues.
I believe the practice is to always have someone else merge your patch to guarantee a peer review.
Thanks


Add support for specific users/roles to grant specific badges

I think that a granular permission system for granting specific badges would be helpful, I'm envisioning adding 2 parameters to the badge model in contentful, one for allowed users and one for allowed roles, this would just be a list of discord ids that the bot would check against
Places this would come in handy:

robot rumble badge (I want anton and noah to be able to award this to participants)
some kind of badge for mentors to give out to students?

My code doesn't work

Pretty sure it was contentfuls fault, my code does work after all! What a rollercoaster


error generating manifest: unresolved nodes

Hi all,
I'm trying to create my cf-deployment.yml to deploy CF via bosh. When i launch the command :
./generate_deployment_manifest openstack ../cf-stub.yml > cf-deployment.yml
I have the following error :
2015/04/29 05:31:53 error generating manifest: unresolved nodes:
(( merge ))	in ./templates/cf-properties.yml	properties.uaa.clients.gorouter.secret	(properties.uaa.clients.gorouter.secret)
whereas i dont't find any mention of the properties.uaa.clients.gorouter.secret attribute neither in my ../cf-stub.yml, nor in my cf-deployment.yml.
Thanks in advance for your support.
Sylvain

Hi, @MyGoul,
The spiff output is telling you that the cf-properties.yml template has an unresolved node, namely this one. The generate_deployment_manifest script merges several templates from cf-release together with the stub you provide to generate a complete deployment manifest for CF, so you'll need to supply a value for that node in your own cf-stub.yml file.
Thanks,
Eric, for the CF Runtime Diego team

Sounds simple and it works :)
Thank you for your support.

Glad to hear it! Closing this out.
Thanks again,
Eric


How can I speed up listing videos on a channel?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Using yt-dlp --print "%(id)s;%(title)s" "URL" > file.csv, I can list all the videos in a channel. But, the process is very slow. I was wondering why it's super slow (since presumably, it shouldn't actually be downloading the video?), and how can I speed it up?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
N/A -- there's no problem, just a question.

--flat-playlist


"hashport" = named addressable port

Closing in favor of tracking at flowbased/flowbased.org#20 since this is a question that should be handled on the cross-runtime standardization level


AWS EC2 long ids support

AWS EC2 is migrating to long ids this year - from 8 chars to 17 chars - https://aws.amazon.com/ec2/faqs/#longer-ids
https://aws.amazon.com/blogs/aws/heads-up-longer-ec2-ebs-resource-ids-coming-in-2016/
If enabling long id support in EC2 for instances, chef-apply fails with
    NoMethodError
    -------------
    undefined method `id' for nil:NilClass

Full stack trace:
NoMethodError: machine[test-machine] ((chef-apply cookbook)::(chef-apply recipe) line 89) had an error: NoMethodError: undefined method `id' for nil:NilClass
~/.chefdk/gem/ruby/2.1.0/gems/chef-provisioning-aws-1.7.0/lib/chef/provisioning/aws_driver/driver.rb:1273:in `converge_ec2_tags'
~/.chefdk/gem/ruby/2.1.0/gems/chef-provisioning-aws-1.7.0/lib/chef/provisioning/aws_driver/driver.rb:631:in `ready_machine'
~/.chefdk/gem/ruby/2.1.0/gems/chef-provisioning-1.5.1/lib/chef/provider/machine.rb:39:in `block in <class:Machine>'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/provider.rb:362:in `action_ready'
~/.chefdk/gem/ruby/2.1.0/gems/chef-provisioning-1.5.1/lib/chef/provider/machine.rb:56:in `block in <class:Machine>'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/provider.rb:362:in `action_converge'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/provider.rb:144:in `run_action'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource.rb:585:in `run_action'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/runner.rb:49:in `run_action'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/runner.rb:81:in `block (2 levels) in converge'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/runner.rb:81:in `each'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/runner.rb:81:in `block in converge'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/resource_list.rb:83:in `block in execute_each_resource'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/stepable_iterator.rb:116:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/stepable_iterator.rb:116:in `call_iterator_block'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/stepable_iterator.rb:85:in `step'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/stepable_iterator.rb:104:in `iterate'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/stepable_iterator.rb:55:in `each_with_index'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/resource_collection/resource_list.rb:81:in `execute_each_resource'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/runner.rb:80:in `converge'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/application/apply.rb:189:in `run_chef_recipe'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/application/apply.rb:198:in `run_application'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/lib/chef/application/apply.rb:211:in `run'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.5.1/bin/chef-apply:25:in `<top (required)>'
/opt/chefdk/bin/chef-apply:54:in `load'
/opt/chefdk/bin/chef-apply:54:in `<main>'

This looks like it only needs changing in the aws_instance resource. Changing that line to this should fix this:
name =~ /^i-(?:[a-f0-9]{8}|[a-f0-9]{17})$/ ? name : nil

Can this fix be merged? It works, updateing this regexp....
Thanks in advance

Bump.   Just merged PR #474 into my long-lived multi-patch branch that I'm using to support production to "resolve" this -
@jkeiser @tyler-ball this is pretty critical since new accounts (ie, the one I created last night) now default to long IDs.
without this patch, multiple runs will create duplicate instances (since metal thinks the instance doesnt exist) - and destroying a node will not destroy it for the same reason.   This will be massively problematic to new users to this toolkit.....
I just confirmed correct functionality on my brand new account - instances are created once, and then destroyed.   And since the old ID format is still in the regex, this should be 100% safe for short IDs as well.

Please merge, this is hurting us in new AWS accounts with the long id

#474 is merged, so this can probably close now. :)

There hasn't been a release since, though.

fair enough.  I've never used a released version, haha. :)
On Mon, Apr 11, 2016 at 2:09 PM, Eli Young<EMAIL_ADDRESS>wrote:

There hasn't been a release since, though.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/chef/chef-provisioning-aws/issues/431#issuecomment-208481723

We typically close issues as we resolve them, so I'm going to do that here and I added a release task to my TODO list

Thanks!


add infraredTelescope experiment from Asteroid Day

Set infraredTelescope to be transmissible and scale data value to be similar to other experiments in similar tech nodes.  Solves https://github.com/ShotgunNinja/Kerbalism/issues/138

Also, what is with the lack of proper line endings in these files?  I had to :set binary noeol to keep it that way.  It violates POSIX and probably other specs.

Closing since it's merged into Kerbalism-Continued.


feat: add rateType argument in query market function

It's ready to be merged after fixing the typo


Unable to init project with vbom.ml/util import

vbom.ml/util is hosted at https://github.com/fvbommel/util
tried:

dep init
creating Gopkg.toml and then dep ensure
creating Gopkg.toml and then dep  ensure -v -add vbom.ml/util:github.com/fvbommel/util.git

Either way I receive an error:
The following issues were found in Gopkg.toml:

  ✗ unable to deduce repository and source type for "vbom.ml/util": unable to read metadata: unable to fetch raw metadata: failed HTTP request to URL "http://vbom.ml/util?go-get=1": Get http://vbom
.ml/util?go-get=1: dial tcp <IP_ADDRESS>:80: getsockopt: connection refused

ProjectRoot name validation failed

Gopkg.toml I am trying to use:
[[constraint]]
  name = "vbom.ml/util"
  source = "https://github.com/fvbommel/util.git"

@redbaron hi, I tried adding vbom.ml/util using the same ensure command and it seems to work perfectly. I get the dependency added to my vendor/vbom.ml/.
Can you reproduce it? Maybe a connection issue?

what does it resolve to?
 curl -v  -L https://vbom.ml/util?go-get=1
*   Trying <IP_ADDRESS>...

@darkowlzz could it be that you have it in your GOPATH/pkg/dep/sources already?

OMG, our internal DNS hijacks .ml domain for whatever reason.


Added more fields to Akismet

Added more Akismet options in addition, the user-agent and comment_type will be sent to akismet.
akismet:
  type: comment
  author: name #field
  content: message #field
  email: email #field
  url: url #field

Note I replaced the "enabled" option with "type" for comment_type.

Just an update. Before this commit, akismet caught about 20/50 spam comments in the span of three days. After this update, akismet caught about 50/50 spam comments in the same span.

This looks great, thanks!
To be honest, I don't have a lot of experience using Akismet and its implementation on Staticman has been a bit of an experimentation, so I really appreciate your help on this. At the moment, the public instance is using my Akismet key, which isn't sustainable in the long-term because it comes with usage limits.
Ideally, we'd allow site admins to supply their own Akismet keys, which is a bit tricky for sites in open repositories, because that information would be open to the public eye. Do you have any thoughts on this?
(Sorry I haven't replied to the e-mail you sent me a few days ago. I was planning on replying when I had the time to properly look into the whole Akismet implementation.)

Yeah I can't think of a good way to do it. One way is to get people to encrypt their information with a public key you generate for your instance of Staticman. But that would introduce a whole layer of complexity and people can't easily switch between Staticman hosts.

Merging. Thanks! 🎉


Known High Severity Security Vulnerability in 'marked'

A GitHub security scan identified the following issue:
We found a potential security vulnerability in a repository for which you have been granted security alert access.
hyperledger/indy-sdk
Known high severity security vulnerability detected in marked <0.3.9 defined in: https://github.com/hyperledger/indy-sdk/blob/master/vcx/wrappers/node/package-lock.json
package-lock.json update suggested: marked ~> 0.3.9.
These issues were also found:
Known moderate severity security vulnerability detected in marked < 0.3.9 defined in package-lock.json.
Known moderate severity security vulnerability detected in marked < 0.3.7 defined in package-lock.json.

We are using Jira to track issues now, please use this Jira IS-972 ticket for any further discussion.


End of Overture color corruption.

LordUrQuan Today at 11:08 PM
ok... that was unexpected
got distracted so the overture ran through to the end.  final graphic has psychedlic color corruption
https://media.discordapp.net/attachments/552838120895283210/580593042021679125/overture_complete.jpg?width=1241&height=703

I confirmed it is happening.

I fixed the overture. Maki changed the decompress function a while back and everywhere that uses it needs to skip the first 4 bytes of buffer and I saw a spot that isn't skipping the bytes.

Fixed another bug nearby. If you look at the zoomed in top left you will see 4 pixels. this shifted everything over 4. This was solved by skipping 8 bytes of data from the start of the buffer.


Provide Bicep real values examples including Command

Could you please :

Confirm the Command would override the Dockerfile CMD & ENTRYPOINT ?

Add in this doc pointers to the Azure Monitor Agent config + provide sample about how to embed its agent inside the Docker file and  in the ACA Bicep code ?

https://learn.microsoft.com/en-us/azure/azure-monitor/app/java-in-process-agent

https://learn.microsoft.com/en-us/azure/azure-monitor/app/java-spring-boot#spring-boot-via-docker-entry-point

https://learn.microsoft.com/en-us/azure/azure-monitor/app/java-in-process-agent#set-the-application-insights-connection-string

https://techcommunity.microsoft.com/t5/apps-on-azure-blog/observability-with-azure-container-apps/ba-p/3627909

Provide Bicep real values examples including Command like below :

Below looks correct, could you please confirm quickly ?
          command: [
            'java, -javaagent:"/tmp/app/applicationinsights-agent-3.4.1.jar", org.springframework.boot.loader.JarLauncher, --server.port=9090, --spring.profiles.active=docker,mysql'
          ]

Below looks not correct : no need of square brackets [] , quotes never EXECPT for the javaagent
          command: [
            '["java", "-javaagent:/tmp/app/applicationinsights-agent-3.4.1.jar", "org.springframework.boot.loader.JarLauncher", "--server.port=8080", "--spring.profiles.active=docker,mysql"]'
          ]

And add sample showing how to install the agent in the Dockerfile ?
See my sample at https://github.com/ezYakaEagle442/aca-java-petclinic-mic-srv/blob/main/docker/petclinic-admin-server/Dockerfile#L71
ENV APPI_VERSION="3.4.1"
ENV APP_INSIGHTS_AGENT_JAR_FILE_PATH="/tmp/app/applicationinsights-agent-${APPI_VERSION}.jar"
RUN curl -SL --output ${APP_INSIGHTS_AGENT_JAR_FILE_PATH} https://github.com/microsoft/ApplicationInsights-Java/releases/download/${APPI_VERSION}/applicationinsights-agent-${APPI_VERSION}.jar

@vturecek @torosent
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 61f4086e-c368-eb50-6d14-97882deec44f
Version Independent ID: 86fe5cc7-446f-4406-2c3d-1795be52662a
Content: Containers in Azure Container Apps
Content Source: articles/container-apps/containers.md
Service: container-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@ezYakaEagle442
Thanks for your feedback! We will investigate and update as appropriate.

Correction, it looks like it should be an Array having each item wrapped by single quote , EXCEPT for the AppInsight Agent file path wrapped by double quotes"". Could you please confirm quickly ? And please add the correct syntax in the docs.
          command: [
            'java', '-javaagent:"/tmp/app/applicationinsights-agent-3.4.1.jar"', 'org.springframework.boot.loader.JarLauncher', '--server.port=9090', '--spring.profiles.active=docker,mysql'
          ]

Correction again, no need to rewrap the file path with double quotes. This runs fine :
@description('The applicationinsights-agent-3.x.x.jar file is downloaded in each Dockerfile. See https://docs.microsoft.com/en-us/azure/azure-monitor/app/java-spring-boot#spring-boot-via-docker-entry-point')
param applicationInsightsAgentJarFilePath string = '/tmp/app/applicationinsights-agent-3.4.1.jar'

          command: [
            'java', '-javaagent:${applicationInsightsAgentJarFilePath}', 'org.springframework.boot.loader.JarLauncher', '--server.port=8888'
          ]

@ezYakaEagle442
Could you please confirm whether your query is resolved?

@ManoharLakkoju-MSFT the above command seems to be the correct one, but this issue 99961 is not resolved as real values should be clearly added in the docs

@craigshoemaker
Can you please check and add your comments on this doc update request as applicable.

Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.
#please-close


Changing the JSON metadata format for the website.

@DanTup wanted a slightly different format for the package metadata, so this makes it so:
Instead of nested maps, he wanted a list of maps sorted by date (I put most recent first).
Added tests to make sure we keep the entries unique when it comes to hash/channel pairs (the original reason for the nested maps).

FYI, I'll push the updated versions of the existing metadata when I merge this PR (unless you want them sooner).

@gspencergoog No rush from me (though it's not clear whether you mean today/tomorrow/next week). Not sure if @mit-mit is hoping to get the page out soon though; I'll let him shout if so.

Oh, I see it's merged already anyway :-)

Yes, all merged, and the metadata files are also updated to the new format, so your staging server is likely to not work until you update your code to match.

Perfect, thanks!


Isuue  #63

Files and folder structure updated

Kindly solve merge conflict

solved merge conflicts

kindly solve the merge conflict

isuue 63 solved

kindly use proper Issue number at the PR time

means make your PR correaspomding that issue

issue #63 #81 files and folder structure


Port to futures 0.2 or 0.3

Would be nice if this would be ported to a current version of futures...

Would be nice if this would be ported to a current version of futures...

Hyper is using futures 0.1 so I don't think this can be done right now.


Update hive logo

Previous rev had some white (ish) pixels around the border which showed up against a dark background. Fix.
Co-Authored-By: @SafiraandOCs

/assign @abutcher

This is awesome, thank you so much @SafiraandOCs!
/lgtm


refactor(gtfs-reader): replace javacsv with opencsv

This PR exchanges the library net.sourceforge.javacsv:javacsv with com.opencsv:opencsv which is used to parse CSV files from GTFS.
The motivation for this change is, that javacsv was released 16 years ago, its source code cannot be found anymore, and it was distributed under LGPL, whereas opencsv is Apache 2.0.
For the start I just wrote a wrapper to preserve the old API of the CsvReader from javacsv, and then used the new CSVReader from opencsv within that wrapper. There's also code for writing CSV files, which is not used anywhere though.

I double checked for several GTFS feeds, such as VBB, London, Munich, Japan, and Cairo. The only difference I found was in the number of feed errors being raised, due to a behavioral change in the get method (see other comment).
I adjusted the NOTICE.md.

I furthermore removed the unused code for writing GTFS feeds.

@karussell I noticed that opencsv comes with several transient dependencies from Apache Commons: commons-beanutils, commons-text, commons-collections4, and commons-lang3.
However, the CSV parsing requires only commons-lang3, the others are just for mapping CSV rows directly on Java objects, which we do not need here. Therefore, I would exclude commons-beanutils, commons-text, and commons-collections4 via pom.xml.


[WIP] AnyCable and Stimulus Reflex

I'm currently trying to get Stimulus Reflex working using AnyCable as a drop-in replacement for ActionCable. I'm running into some nasty worms. Ideas welcomed, especially from @palkan :nerd_face:
Worm 1: ApplicationCable::Connection.env returns nil
We merge connection.env with our own constructed hash, resulting in us getting an undefined method 'merge' for nil:NilClass error.
I tried to cheat and just pass our constructed env in to ActionDispatch::Request.new() and it came back with Missing rack.input when we tell ActionDispatch to process the request.
I temporarily reverted the sample project to use standard ActionCable and normally rack.input contains an instance of Puma::NullIO. Unfortunately, this is precisely where my troubleshooting capacity tapers off dramatically.
Worm 2: session.id returns undefined method `fetch' for nil:NilClass
Long story short, if I hard-code a session ID from the database for testing purposes, it seems to work fine. So, we don't have access to the session.id when AnyCable is in the picture.
I don't have a lot of experience with other session stores, but I wonder if it's possible to fix this by switching to using Redis instead of cookies?
I did go through the AnyCable Troubleshooting page and reviewed "From Action to Any" but I could find no reference to these particular issues.
So, not quite drop-in yet but I am optimistic that Vladimir will know what to do!

Vladimir will know what to do!

I hope so) Thanks for pinging me.

Worm 1: ApplicationCable::Connection.env returns nil

Interesting:

We set @env here: https://github.com/anycable/anycable-rails/blob/114c7484966de509809d26b8a5a44d0def6f84b0/lib/anycable/rails/actioncable/connection.rb#L40
Action Cable defines attr reader here: https://github.com/rails/rails/blob/080265dedbffda29d1babb3cb141238569cf2e30/actioncable/lib/action_cable/connection/base.rb#L50

Need to do some debugging.

Worm 2: session.id...

Yep, session is not supported out-of-the-box.
There is a way to do that like this (from this comment):
env['rack.session'] = cookies.encrypted[Rails.application.config.session_options[:key]]

That's just an example, a universal solution would be more complicated.

I wonder if it's possible to fix this by switching to using Redis instead of cookies?

Take a look at: https://github.com/anycable/anycable-rails/issues/95#issuecomment-502458973

This may prove helpful. https://www.driftingruby.com/episodes/plugging-in-anycable

That video is great! Unfortunately, it doesn't address either of the two issues I outlined, I don't think.
I was able to get AnyCable running in the general sense after some trial and error; in other words, it functions well as a drop-in enhancement to ActionCable. However, in the context of StimulusReflex, we need a workaround for what I describe as "Worm 1"... consider it a "10/10 flames" category blocker.
We might be able to get around the session (aka "Worm 2") but we do rely on it heavily for most of our apps. I hope to be able to dig in deeper on this later in the month.
However, nothing can really move forward until I get some help on the 1st concern.

We're currently working on the out-of-the-box session support. That should eliminate the 🐛 №2 (if we work it out).
As for the first one, I figured out why it's nil: we only provided env for connect and disconnect actions but not for channel interactions. Just pushed a quick fix to master, please, check it.

Great news, @palkan! Whatever you did seemed to work. I have SR + AnyCable up and running with local source versions of whatever was current last night.
I am currently attempting to figure out how to access ActionController.cookies outside of a controller, if anyone has any magic scrolls handy. Rails source is clear as mud.

So far, I've been able to get to:
ActionDispatch::Request.new(Rails.application.env_config.merge(connection.env)).cookie_jar.encrypted[Rails.application.config.session_options[:key]]
This returns nil, which is an improvement on all of the other things I've tried. Help! :grinning:

You can try this branch: https://github.com/anycable/anycable-rails/pull/111
It makes using sessions transparent (it's currently under review and will be merged into v1.0 development branch soon).
It would be great if you give it a try and come back with the feedback 🙂

@palkan hey again!
Thanks for the PR. I checked it out and so far I’m not seeing what we’re looking for. Is there something I could be missing, like obvious in hindsight gotchas? It’s just returning nil.
There’s a second issue as well: in the code it’s clear you have to call session.fetch(“value”, nil) instead of session.value.
This would still be a successful outcome: we could explain to people that if they want to use session, they have to convert their code to use fetch. However, is not a “drop-in”.  Is it possible to also support the shorthand syntax?

@palkan any plans to release new version of AnyCable with anycable/anycable-rails#111 seems like this will be important fix for those of us running into the same issue and would make AnyCable as a drop-in replacement for SR

Hey everyone!
I've release the first preview version of the upcoming AnyCable v1.0, which includes the PR mentioned above and other changes which improve StimulusReflex compatibility.
You can give it a try by doing the following:

Add gem "anycable-rails", "1.0.0.preview1" to the Gemfile.
(required) Install anycable-go v1.0.0.preview (binaries available here, Docker images are also available).
If you use session in your reflexes to keep some state, add persistent_session_enabled: true to anycable.yml (or run our brand new installation wizard rails g anycable:setup which should do that for you).

This version successfully runs all StimulusReflexExpo demos but one ("snake" is hardly possible to implement with short-lived connections, since it spawns tons of threads; I don't think it makes a lot of sense to spend a lot of time to make it compatible with AnyCable, 'cause I hope you do not do stuff like that in production 🙂).
And feedback is appreciated!
/cc @daya @leastbad

:snake: :snake: :snake:
You don't like snakes!?
Thank you so much for getting this out the door and your advice along the way, Vladimir. You rock.

@palkan a quick follow-up question: you mentioned that snake won't work because it spawns tons of threads. What is the specific reason that snake doesn't work? Is it because it's spawning threads or some other factor? eg. Can I spawn one thread? Can I spawn n threads slowly?
Obviously the goal isn't to fix :snake: but I do need to document any caveats for people using the libraries together.

What is the specific reason that snake doesn't work?

It relies on the session object: every thread uses the same session as a state store. The problem is that AnyCable "free" all the connection data after every command (i.e., there is not long-lived objects). Thus, when we trigger a turn, the session from the background thread is not getting updated—these are two different objects.
So, it's OK to use session in threads if you're not going to update it (the example from the documentation would work fine).
Another option is to reload the session before reading it from the thread (you can do that by calling session.send(:load!)). That would only work with sessions stored in a cache store (e.g., Redis). Actually, I made snake work by adding reload to the reflex, but due to the high update frequency there still were lags.

@palkan perfect answer. Thank you again!

Hi @palkan I am testing something with Stimulus Reflex, and for me the session.send(:load!) doesn't work, I get undefined method load!' for #<AnyCable::Rails::SessionProxy`. What am I missing? Thanks!

I'm getting a similar error using AnyCable:
StimulusReflex::Channel Failed to re-render http://localhost:3000/ undefined method `load!' for {}:Hash

Which is happening here, in reflex.rb:37:
req.tap { |r| r.session.send :load! }

stimulus_reflex 3.1.4

It seems that Rails' .delegate_missing_to doesn't allow calling private methods.
Fixed in anycable-rails (branch 1.0-dev). See the commit link above.

Hi,
Could you help me to setup anycable with stimulus_reflex?
When performing an action my reflex is being executed, but I get an error:
cco-web-anycable-rpc    | [AnyCable sid=Pwu_q_n3fYm_u5RwA1Jrl] StimulusReflex::Channel#receive({"target"=>"PartnerArea::ExampleReflex#example_action", "args"=>[], "url"=>"http://web.cco.local/partner_area", "attrs"=>{"class"=>"btn btn-link", "data-action"=>"click->partner-area--example#example_stimulus", "data-controller"=>"partner-area--example", "checked"=>false, "selected"=>false, "tag_name"=>"A"}, "dataset"=>{"data-action"=>"click->partner-area--example#example_stimulus", "data-controller"=>"partner-area--example"}, "selectors"=>[], "permanent_attribute_name"=>"data-reflex-permanent", "reflexId"=>"658747d0-3298-4e67-9f56-6eaab9a964cf", "params"=>{}})
cco-web-anycable-rpc    | example_action REFLEX action running
cco-web-anycable-rpc    | current_user.id=a6cca6b6-6a22-4d62-9539-2a8eebc805c3
cco-web-anycable-rpc    | [AnyCable sid=Pwu_q_n3fYm_u5RwA1Jrl] StimulusReflex::Channel Failed to re-render http://web.cco.local/partner_area Devise could not find the `Warden::Proxy` instance on your request environment.
cco-web-anycable-rpc    | Make sure that your application is loading Devise and Warden as expected and that the `Warden::Manager` middleware is present in your middleware stack.
cco-web-anycable-rpc    | If you are seeing this on one of your tests, ensure that your tests are either executing the Rails middleware stack or that your tests are using the `Devise::Test::ControllerHelpers` module to inject the `request.env['warden']` object for you. /usr/local/bundle/ruby/2.6.0/gems/devise-4.7.1/lib/devise/controllers/helpers.rb:143:in `warden'
cco-anycable-go         | D 2020-06-12T15:45:41.500Z context=pubsub Incoming pubsub message from Redis: {"stream":"StimulusReflex::Channel:a6cca6b6-6a22-4d62-9539-2a8eebc805c3","data":"{\"cableReady\":true,\"operations\":{\"dispatchEvent\":[{\"name\":\"stimulus-reflex:server-message\",\"detail\":{\"stimulusReflex\":{\"target\":\"PartnerArea::ExampleReflex#example_action\",\"args\":[],\"url\":\"http://web.cco.local/partner_area\",\"attrs\":{\"class\":\"btn btn-link\",\"data-action\":\"click-\\u003epartner-area--example#example_stimulus\",\"data-controller\":\"partner-area--example\",\"checked\":false,\"selected\":false,\"tagName\":\"A\"},\"dataset\":{\"data-action\":\"click-\\u003epartner-area--example#example_stimulus\",\"data-controller\":\"partner-area--example\"},\"selectors\":[\"body\"],\"permanentAttributeName\":\"data-reflex-permanent\",\"reflexId\":\"658747d0-3298-4e67-9f56-6eaab9a964cf\",\"params\":{},\"serverMessage\":{\"subject\":\"error\",\"body\":\"StimulusReflex::Channel Failed to re-render http://web.cco.local/partner_area Devise could not find the `Warden::Proxy` instance on your request environment.\\nMake sure that your application is loading Devise and Warden as expected and that the `Warden::Manager` middleware is present in your middleware stack.\\nIf you are seeing this on one of your tests, ensure that your tests are either executing the Rails middleware stack or that your tests are using the `Devise::Test::ControllerHelpers` module to inject the `request.env['warden']` object for you. /usr/local/bundle/ruby/2.6.0/gems/devise-4.7.1/lib/devise/controllers/helpers.rb:143:in `warden'\"}}}}]}}"}

any idea how to solve this?
#app/channels/application_cable/connection.rb
module ApplicationCable
  class Connection < ActionCable::Connection::Base
    identified_by :current_user

    def connect
      self.current_user = find_verified_user

    end

    private

    def find_verified_user
      ckey = Rails.application.config.session_options[:key] || raise("No cookies key in config")
      redis_key =  "#{ENV['APP_SESSION_REDIS_PREFIX']}#{cookies[ckey]}"
      env["rack.session"] = JSON.parse(REDIS_SESSIONS.get(redis_key))
      user = Warden::SessionSerializer.new(env).fetch(:user)
      reject_unauthorized_connection unless user
      User.where(id: user.id).first
    end

  end
end

# my example reflex
class PartnerArea::ExampleReflex < ApplicationReflex
  delegate :current_user, to: :connection

  def example_action
    puts "example_action REFLEX action running".green
    puts "current_user.id=#{current_user.id}"
  end
end

Thank you for your help!

Hi,
Could you help me to setup anycable with stimulus_reflex?
When performing an action my reflex is being executed, but I get an error:
cco-web-anycable-rpc    | [AnyCable sid=Pwu_q_n3fYm_u5RwA1Jrl] StimulusReflex::Channel#receive({"target"=>"PartnerArea::ExampleReflex#example_action", "args"=>[], "url"=>"http://web.cco.local/partner_area", "attrs"=>{"class"=>"btn btn-link", "data-action"=>"click->partner-area--example#example_stimulus", "data-controller"=>"partner-area--example", "checked"=>false, "selected"=>false, "tag_name"=>"A"}, "dataset"=>{"data-action"=>"click->partner-area--example#example_stimulus", "data-controller"=>"partner-area--example"}, "selectors"=>[], "permanent_attribute_name"=>"data-reflex-permanent", "reflexId"=>"658747d0-3298-4e67-9f56-6eaab9a964cf", "params"=>{}})
cco-web-anycable-rpc    | example_action REFLEX action running
cco-web-anycable-rpc    | current_user.id=a6cca6b6-6a22-4d62-9539-2a8eebc805c3
cco-web-anycable-rpc    | [AnyCable sid=Pwu_q_n3fYm_u5RwA1Jrl] StimulusReflex::Channel Failed to re-render http://web.cco.local/partner_area Devise could not find the `Warden::Proxy` instance on your request environment.
cco-web-anycable-rpc    | Make sure that your application is loading Devise and Warden as expected and that the `Warden::Manager` middleware is present in your middleware stack.
cco-web-anycable-rpc    | If you are seeing this on one of your tests, ensure that your tests are either executing the Rails middleware stack or that your tests are using the `Devise::Test::ControllerHelpers` module to inject the `request.env['warden']` object for you. /usr/local/bundle/ruby/2.6.0/gems/devise-4.7.1/lib/devise/controllers/helpers.rb:143:in `warden'
cco-anycable-go         | D 2020-06-12T15:45:41.500Z context=pubsub Incoming pubsub message from Redis: {"stream":"StimulusReflex::Channel:a6cca6b6-6a22-4d62-9539-2a8eebc805c3","data":"{\"cableReady\":true,\"operations\":{\"dispatchEvent\":[{\"name\":\"stimulus-reflex:server-message\",\"detail\":{\"stimulusReflex\":{\"target\":\"PartnerArea::ExampleReflex#example_action\",\"args\":[],\"url\":\"http://web.cco.local/partner_area\",\"attrs\":{\"class\":\"btn btn-link\",\"data-action\":\"click-\\u003epartner-area--example#example_stimulus\",\"data-controller\":\"partner-area--example\",\"checked\":false,\"selected\":false,\"tagName\":\"A\"},\"dataset\":{\"data-action\":\"click-\\u003epartner-area--example#example_stimulus\",\"data-controller\":\"partner-area--example\"},\"selectors\":[\"body\"],\"permanentAttributeName\":\"data-reflex-permanent\",\"reflexId\":\"658747d0-3298-4e67-9f56-6eaab9a964cf\",\"params\":{},\"serverMessage\":{\"subject\":\"error\",\"body\":\"StimulusReflex::Channel Failed to re-render http://web.cco.local/partner_area Devise could not find the `Warden::Proxy` instance on your request environment.\\nMake sure that your application is loading Devise and Warden as expected and that the `Warden::Manager` middleware is present in your middleware stack.\\nIf you are seeing this on one of your tests, ensure that your tests are either executing the Rails middleware stack or that your tests are using the `Devise::Test::ControllerHelpers` module to inject the `request.env['warden']` object for you. /usr/local/bundle/ruby/2.6.0/gems/devise-4.7.1/lib/devise/controllers/helpers.rb:143:in `warden'\"}}}}]}}"}

any idea how to solve this?
#app/channels/application_cable/connection.rb
module ApplicationCable
  class Connection < ActionCable::Connection::Base
    identified_by :current_user

    def connect
      self.current_user = find_verified_user

    end

    private

    def find_verified_user
      ckey = Rails.application.config.session_options[:key] || raise("No cookies key in config")
      redis_key =  "#{ENV['APP_SESSION_REDIS_PREFIX']}#{cookies[ckey]}"
      env["rack.session"] = JSON.parse(REDIS_SESSIONS.get(redis_key))
      user = Warden::SessionSerializer.new(env).fetch(:user)
      reject_unauthorized_connection unless user
      User.where(id: user.id).first
    end

  end
end

# my example reflex
class PartnerArea::ExampleReflex < ApplicationReflex
  delegate :current_user, to: :connection

  def example_action
    puts "example_action REFLEX action running".green
    puts "current_user.id=#{current_user.id}"
  end
end

Thank you for your help!

Hey guys,
I finally managed to make it work. It was missing to configure the middleware:
#config/environment/development.rb

# https://github.com/anycable/anycable-rails/issues/127#issue-624598894
  AnyCable::Rails::Rack.middleware.use Warden::Manager do |config|
    Devise.warden_config = config
  end

Thanks.

Sorry you had trouble, but thanks for documenting this (however unintentionally). I've borrowed the link to the AnyCable issue for the next release of the SR docs.

@leastbad You can also add a link to the corresponding AnyCable documentation article 😉: https://docs.anycable.io/v1/#/ruby/authentication
Also, please, check out the doc on Stimulus Reflex (it’s a very simple early version I’ve just pushed yesterday), maybe, you have something to add: https://docs.anycable.io/v1/#/ruby/stimulus_reflex

@palkan thanks so much for writing that up! I had no idea you'd done that. I have done my best to work in the details to https://docs.stimulusreflex.com/deployment#anycable as well.
I'm a bit fuzzy on persistent_session_enabled: true - did this become unnecessary as of v3 of SR?
Truth be told, I don't have any specific feedback re: the instructions you gave beyond that you should feel welcome to borrow anything on our page for yours. I'm not concerned if they are basically mirror images of each other... both serve to advertise the other regardless of how people find them.
Finally, give us a heads-up when you're pushing a new version! You're always welcome on Discord but even a note here helps us keep things from falling through the cracks.

did this become unnecessary as of v3 of SR?

TBH, I haven't check v3 yet. Add added this feature while making stimulus_reflex_expo work with AnyCable a while ago. From what I understand, Stimulus Reflex v3+ commits session after each reflex interaction, so the option should be unnecessary (and even could lead to unexpected results: https://github.com/anycable/anycable-rails/issues/129).

https://docs.stimulusreflex.com/deployment#anycable

Awesome!

I'm not concerned if they are basically mirror images of each other... both serve to advertise the other regardless of how people find them.

Agree. I think, cross-linking them would be a good idea.

@palkan please let me know ASAP if you ever find out that the option is causing harm.

Here is the Expo running on AnyCable: https://github.com/anycable/stimulus_reflex_expo
I found one bug (so, 1.0.0.rc1 could not work correctly, anycable-rails#master has a fix), checked with both persistent_session_enabled: true and persistent_session_enabled: false — all demos work. Hence, this option is not needed for Reflex v3.

@palkan yikes!
I took out the reference to persistent_session_enabled (thanks for investigating!) but this news about rc1 not working is troubling. I don't mind telling people to draw off master, but does it make sense to cut an rc2? Also... is the bug in anycable-rails or anycable-go?

does it make sense to cut an rc2?

Agree, it makes sense. Released.

is the bug in anycable-rails or anycable-go?

Only anycable-rails. Other libraries are still rc1.

First time a Reflex is invoked i'm getting this error Perform error: Application error: undefined method rescue_with_handler' for nil:NilClassand second time No route matches`.

@matedemorphy Could you please provide a bit more context? Do you use Rails 6.1 (and rescue_from in a connection class)?

@palkan i'm usinig rails 6.0.3. But this is related to that is a multi-tenant app, and i'm using the Apartment gem, there are some additional configurations that should be done, but i couldn't figure out yet, like make it work with dynamic subdomains, among others.


Anaconda package

This issue has as a goal to investigate how and if an anaconda package for GHEtool would be an added benefit.

This is work in progress. Please check: https://github.com/conda-forge/staged-recipes/pull/23240 for the conda version of GHEtool v2.1.2

This will be fixed in v2.2.1, since the will be removed from this repo and moved to another one.

Waiting for approval to be merged into main ...


Automate form interactions standard

Split off from #11.
See https://github.com/cucumber-ltd/bbc-a11y/blob/master/standards/19_form-interactions.md

There's a mix of easy and very-hard in this one, I think.
Checking for a submit button is trivial. Testing the effect of triggering change, focus or blur events might be possible, but sounds tough to do in a fool-proof way.

Checking for the submit button is a high value test.
I don't think there is sufficient value to automate testing visible changes that may lead to false positives.

We are checking for submit buttons.


Version stuff1

Added some rudimentary version support for haproxy-api in order to get this into a proper build pipeline. @relistan @bparli @mihaitodor Inspired from this post https://blog.cloudflare.com/setting-go-variables-at-compile-time/
The health check now spits out the version that was set at compile time when running and even when healthcheck fails.
This also supports a docker in docker build for the mac or Jenkins.
$PACKAGE_DIR/docker/build.sh build builds binary in local os for linux we may want to pass the GOOS=osx in this case.
$PACKAGE_DIR/docker/build.sh buildindocker uses a golang 1.7 alpine container which does the same build but in a consistent controlled environment ensuring proper godeps and gopath is used.  Hopefully preventing unicorn builds.
Also started work on a release function which uses github-release so we can publish the binaries on their own on github.
I was also looking briefly at drone.io and https://getgb.io/ it might be time to start looking at a better build tool.  I prefer one that supports docker.   I kind of like the docker in docker with build.sh but its pretty basic.
Also we need tests but that is out of scope for this.

Since this is an open source project, I was planning for this (and our other OSS projects) to use https://docs.travis-ci.com/user/languages/go for builds. Combining that with https://docs.travis-ci.com/user/deployment/releases/ should do most of this for us and be something outside contributors can easily integrate with.

Good call @relistan didnt realize this was OSS I'll revise the changes.


Remove unpatching functionality from instrumentation package

Unpatching in many cases is a "lie" because to be able to do that we need indirection between the patched function and the caller. For functions we cannot assure that for, we either have to

add a toggle into the instrumentation that dynamically decides whether to short-circuit into the original function or not based on whether the instrumentation is enabled during the call and/or(if the patched function is short-lived = less than the duration of the span)
patch the function that returns the function we actually need to patch.

Even this:
const { readFileSync } = require('fs');
// now there's no way to unpatch but to add a toggle inside the instrumentation.

... renders unpatching useless. Requiring with de-structuring is an increasingly common pattern. What makes it even worse is that TS compiler currently hides that by referencing objects dynamically in the compiled asset:
// typescript:
import { readFileSync } from 'node:fs';
readFileSync(...);
// compiles into something like
const _temp_fs = require('node:fs');
_temp_fs.readFileSync(...);

Other languages don't have functionality to dynamically "unpatch", in many of them it's even unthinkable. The only use-case I know of are the tests if one has to run all tests in one process, but that can be mitigated by emptying the require cache between the tests and calling require again.
I propose we move towards removing it in the future(with a proper deprecation process, etc). WDYT?
Additional questions:

Which are the use cases unpatching is required for?
Could we live without any toggling mechanism(instrumentation enable/disable) and say that needs to be decided on boot and is then immutable. Note that it doesn't necessarily mean that whether the application is exporting telemetry data or not is immutable, but it just needs to be done in a different part of the pipeline.

[ ] Deprecate unpatching,
[ ] Adding a test utility that cleans the require cache to contrib,
[ ] Discourage implementing unpatching functionality,
[ ] Decide if we need disable and how it should be implemented across instrumentations,
[ ] Removing unpatching altogether

There were discussions about this in the past, see #618 and #732.
As you can read there I was never a fan of unpath for the same reasons you write here so fine for me to remove it.
As instrumentation and instrumentations are still experimental no deprecation cylce is needed usually. But we might want some replacement for tests.
I think two flags per instrumentation make sense:

instrument if true instrument if false don't instrument. Clearly changing this flag for a running application makes no sense
active if true create spans,... if false pass through everything unmodified, act like if you were not there. changeable at runtime at any time.


"Didn't find custom op for name 'Convolution2DTransposeBias' with version 1 Registration failed."

Thank for your guide!   :)
But when completed, I got an error "cannot find operation Convolution2DTransposeBias". It seems "CustomOperations" not participating in the compilation.
Could you please share your files which modified compared to tensorflow's source, include "CustomOperations" files and "BUILD" files ?

The error is
"Didn't find custom op for name 'Convolution2DTransposeBias' with version 1
Registration failed."

I have solved this problem, and I will share my method in this issue soon.

Great, just came across your issue.
In case anybody needs it - here's the full git diff and the BUILD file in the custom_ops directory
https://gist.github.com/wolterlw/767a467d6f75b3533cd49482ba25ae7e

I have solved this problem.
My method is more complex.
The official source code has some "offical custom ops" in tensorflow/lite/kernels, showing how to add custom_ops.
So I used the following steps:

According to tensorflow/lite/kernels/relu1.cc , revise the new custom_ops codes ( *.cc files );
Put new custom_ops *.cc files into tensorflow/lite/kernels/;
Modify tensorflow/lite/kernels/register.cc to register new custom_ops;
Modify tensorflow/lite/kernels/BUILD, Bazel will compile custom_ops *.cc files into library builtin_op_kernels.
Recompile the whole project.

Official guide maybe useful: https://www.tensorflow.org/lite/guide/ops_custom

Could you please share your files which modified compared to tensorflow's source?  I have the same problem

I have solved this problem.
My method is more complex.
The official source code has some "offical custom ops" in tensorflow/lite/kernels, showing how to add custom_ops.
So I used the following steps:

According to tensorflow/lite/kernels/relu1.cc , revise the new custom_ops codes ( *.cc files );
Put new custom_ops *.cc files into tensorflow/lite/kernels/;
Modify tensorflow/lite/kernels/register.cc to register new custom_ops;
Modify tensorflow/lite/kernels/BUILD, Bazel will compile custom_ops *.cc files into library builtin_op_kernels.
Recompile the whole project.

Official guide maybe useful: https://www.tensorflow.org/lite/guide/ops_custom

Could you please share your files which modified compared to tensorflow's source? I have the same problem

oh, my method is too complex, you can use @wolterlw 's method.

@wolterlw
Thanks for sharing.
I followed these changes : https://gist.github.com/wolterlw/767a467d6f75b3533cd49482ba25ae7e. Additionally I had to change the include path in each *.cc. I am stuck with this error.
tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.cc: In static member function 'static tflite::interpreter_wrapper::InterpreterWrapper* tflite::interpreter_wrapper::InterpreterWrapper::CreateInterpreterWrapper(std::unique_ptr<tflite::FlatBufferModel>, std::unique_ptr<tflite::interpreter_wrapper::PythonErrorReporter>, std::__cxx11::string*)': tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.cc:207:13: error: 'tflite_operations' has not been declared              tflite_operations::RegisterMaxPoolingWithArgmax2D());              ^ tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.cc:209:13: error: 'tflite_operations' has not been declared              tflite_operations::RegisterMaxUnpooling2D());              ^ tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.cc:211:13: error: 'tflite_operations' has not been declared              tflite_operations::RegisterConvolution2DTransposeBias());

I figured out the namespace mediapipe has to be appended in interpreter_wrapper.cc

mediapipe

What changes did you do in interpreter_wrapper.cc?

I figured out the namespace mediapipe has to be appended in interpreter_wrapper.cc
tflite_operations::RegisterMaxUnpooling2D() should be mediapipe::tflite_operations::RegisterMaxUnpooling2D()


help add this repo to vcpkg

https://github.com/Microsoft/vcpkg
https://github.com/Microsoft/vcpkg/issues/6077

@bitnick10 We would love to see contributions here if you've the resources. Thanks.

I just gave a try, it's extremely hard. By default vcpkg builds protobuf but protobuf isn't designed to work well in that way.

Completed

Ort can consume packages from vcpkg, but itself is not in vcpkg yet.

Cool


Unnecessary for/id attributes in Basic HTML and HTML5

Describe your problem and how to reproduce it:
Form challenges in the Basic HTML and HTML5 course suggest using for attributes on form labels:

It is considered best practice to set a for attribute on the label element, with a value that matches the value of the id attribute of the input element.

However, this is only true when inputs are not nested within labels. When using nesting, this relationship is created implicitly - see https://developer.mozilla.org/en-US/docs/Web/HTML/Element/label
Therefore, the following example in the link below:

  <input id="indoor" type="radio" name="indoor-outdoor">Indoor

should either be removed in favour of keeping only the first example (which features a nested label without explicit association), or replaced with a non-nested version (often preferred in real-world scenarios due to more styling control):
<input id="indoor" type="radio" name="indoor-outdoor">
Indoor

Why I feel this is important:
For a beginner learning HTML, explicit relationships using attributes introduce additional cognitive load and are unnecessary due to the implicit relationship created by nesting. They could also create confusion when the name and value attributes are introduced later - in the context of forms, name or value often share the same value with id, so they could be mixed up easily by a beginner.
Additional notes:
I have tested with VoiceOver on macOS and NVDA on Windows, across Chrome and Firefox (both systems) and Safari (on macOS), and confirmed that input labels are read aloud correctly when using implicit association.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/responsive-web-design/basic-html-and-html5/create-a-set-of-radio-buttons
(and subsequent form-based challenges)
Tell us about your browser and operating system:
Screen reader support for nested labels tested across the following browsers:
Browser Name: Google Chrome
Browser Version: 89.0.4389.90 (Official Build) (arm64)
Operating System: macOS Big Sur 11.3 Beta (20E5217a)

Browser Name: Firefox
Browser Version: 87.0 (64-bit)
Operating System: macOS Big Sur 11.3 Beta (20E5217a)

Browser Name: Safari
Browser Version: 14.1 (166<IP_ADDRESS>.12)
Operating System: macOS Big Sur 11.3 Beta (20E5217a)

Browser Name: Google Chrome
Browser Version: 89.0.4389.90 (Official Build) (64-bit)
Operating System: Windows 10 Pro 20H2 19042.906

Browser Name: Firefox
Browser Version: 87.0 (64-bit)
Operating System: Windows 10 Pro 20H2 19042.906

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
See code examples above

Thanks for the excellent report @davidbailey00

Hey @freeCodeCamp/moderators what do you think?

I think it's a good issue to reduce complexity :+1:

Agreed with having a non-nested version.
<input id="indoor" type="radio" name="indoor-outdoor">
Indoor

I could go either way - leave as is, or change to what has been mentioned. 👍
I do want to point out: https://html.spec.whatwg.org/multipage/forms.html#the-label-element

If the for attribute is not specified, but the label element has a labelable element descendant, then the first such descendant in tree order is the label element's labeled control.

I can imagine a case where there is more than one labelable element descendant within a label element (perhaps, not semantically correct), and the for attribute would be required, if the desired associative element is not first.
Probably just me (maybe, even freeCodeCamp's fault), but I always give my labels an associated for attribute.

I agree that the example shown above by @hassaanp should replace the existing example, but I also don't think it would hurt to put the existing example after it.  There is nothing that says you can not do what our example shows (that I am aware of).


Helpfu looking l link texts do not go to the referenced resources

Clicking this link takes you to the Bootstrapper Packages to Redistribute Components website to check for updates.
No, it takes you to Microsoft.com/net  - This seems to be a common problem - links bouncing up to the "home" page throughout the new visual studio documentation. I've been trying to solve a Bootstapper package issue (Windows Installer 4.5 - on a newly installed Visual Studio 2017/Windows 10 development machine) working on a project built on a Win 7/VS 2015 machine. I had high hopes for that link.

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

ID: f615d057-7cc1-797a-1f9d-dad140a310a8
Version Independent ID: 93a0adaf-43cd-faf1-640d-2c355e3d5bfd
Content: Prerequisites Dialog Box - Visual Studio
Content Source: docs/ide/reference/prerequisites-dialog-box.md
Service: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

Hi Genevieve,
Thank you for this! I’ve been battling this stuff all morning – and its nice to hear my feedback may help me or someone else too someday!
Regards,
Thom
Thom C. Blackwell,  VP Technical Services
Boston Software Systems, Inc.
Phone:<PHONE_NUMBER> x807
Mobile:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
www.bostonsoftwaresystems.comhttp://www.bostonsoftwaresystems.com/
The information contained in this message may be confidential and legally protected under applicable law. The message is intended solely for the addressee(s). If you are not the intended recipient, you are hereby notified that any use, forwarding, dissemination, or reproduction of this message is strictly prohibited and may be unlawful. If you are not the intended recipient, please contact the sender by return e-mail and destroy all copies of the original message.
From: Genevieve Warren<EMAIL_ADDRESS>Sent: Wednesday, May 16, 2018 3:05 PM
To: MicrosoftDocs/visualstudio-docs<EMAIL_ADDRESS>Cc: Thom Blackwell<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/visualstudio-docs] Helpfu looking l link texts do not go to the referenced resources (#962)
@thomb1https://github.com/thomb1 Thank you for the feedback Thom. It seems that the button doesn't even exist in the Prerequisites dialog box anymore either ("Check Microsoft Update for more redistributable components"). I will update the topic and apologize that the link didn't help you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/visualstudio-docs/issues/962#issuecomment-389630957, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGV4zyfb1GmvjtlsnuD2k_qVXGSHsbrpks5tzHhUgaJpZM4UBpZ1.


Multiple issues with the decoder

Unfortunatly there are issues with the decode algorithm used in the original Python code.
For example :
&-&-- is decoded as && instead of &&-
I have use Proptest crate (a property testing framework) to spot them.

As far as I can see these are all edge case, this library still usefull and very usable but for the sake of correctness this need to be fixed. It may be not that simple with the decode algorithm used in the Pyton code.


How work with big dataset?

Hello.
I try to work with the ml-latest dataset and got error "Memory error".
How to work with a large dataset, I should split this data, but how I can recommend something based on half of the data?

Hello!
I think this question is not relevant to this GitHub repository as the repo contains code to do things quickly with the moderate sized dataset. Still, there are various ways to do that -

Try on optimizing your import in pandas, the default for numeric is float64, you can reduce it to float8,16.32 based on datatype (Check this kernel - https://www.kaggle.com/shrutimechlearn/large-data-loading-trick-with-ms-malware-data)
Try algorithms which work on batch datasets, active learning or stochastic gradient descent method
Try running your code on a bigger machine which can accommodate data in RAM
Try distributed computing on Spark and use Spark MLlib

Hope these help!


Bug:  contribution guide page showing 404 error

Describe the bug
contribution page is showing 404 error.
Link is broken
Steps to reproduce
Visit any of the repos and click the "contributing guide" link
Affected services
opensauced.pizza, admin.opensauced.pizza, docs.opensauced.pizza
Platforms
Desktop
Browsers
Firefox
Environment
Production
Additional context
No response
Code of Conduct

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

thanks for reporting this, good catch!

.take


Add ComparableAssert#isBetween and #isStrictlyBetween?

I just noticed that #isBetween and #isStrictlyBetween are missing from ComparableAssert while they are present for IntegerAssert, LongAssert etc. I guess it makes sense to add them.
What do you think?

That would be a nice addition indeed.


fix:[CORE-1752] Change asset type for aws port policies from EC2 to SG

Description

we are changing the way we check ports with public access for EC2. Going forward, we are going to check only the open ports for security groups
previously we used to check if the subnet or vpc the EC2 is in, is open to public access using route tables, If yes, the SG for that EC2 is checked for open ports
Now since we are changing the asset types of these policies from EC2 to SG, the policy will check only for ports that are publicly accessible on SG
These are the new policies added to SG to check the open ports that are not required required for EC2: "Deny Public Access to NETBIOS Port 138", "Deny Public Access to HTTP Port 80", "Deny Public Access to HTTP Port 8080".

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)

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

[ ] create an SG with open ports(80, 8080, 138) in the saasdev aws account for test purpose
[ ] run aws collector, shipper and the port policies for SG
[ ] verify violations created for that SG on UI

Checklist:

[ ] My code follows the style guidelines of this project
[ ] My commit message/PR follows the contribution guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Other information:
List any documentation updates that are needed for the Wiki

created a security group for testing

After running collector, shipper and policy rules


Personal Access Token doesn't work username/password does

Hi,
Great plugin!
I want to use a Personal Access Token instead of username/password to authenticate when I do a gradlew tagVersion but I get an error:
org.eclipse.jgit.api.errors.TransportException: https://github.com/ThreeAxis/CPNamespace.git: https://github.com/JohnDoe/SomeRepository.git/info/refs?service=git-receive-pack not found: Not Found
Like I said, when I use username/password it all seems to work.
Would you have an idea how to solve this? What access does the plugin need through the token that I need to set when generating it?

Sorry, I fixed it, you need to provide full repo-access to the Personal Access Token. Maybe something you would like to add to the README.MD.

Sorry, I fixed it, you need to provide full repo-access to the Personal Access Token. Maybe something you would like to add to the README.MD.


On wallet opening all transactions are sending/ receiving

Opening a wallet using the '0.1.5' code I saw every transaction as sending and receiving and had to do a repair wallet.
This is most likely something to do with the mempool as that is where the transaction confidences are stored.

This turned out to be a new option in Context called 'EventHorizon'.
Transaction confidences over a certain depth have some details elided and this caused the tx to be seen as Sending/ receiving.
There is a Context constructor where you can set the event horizon. I've set it to Integer.MAX to effectively switch off this behaviour.
Awaiting review and closing
(Try doing a repair on a wallet with tx older than a couple of months).

Verified working. Closing.


Possible incorrect JSON representation of log.level for Pino log format

Using v1.3.0 of ecs-pino-format, I get the following output for a message:
{
  "log.level": "info",
  "@timestamp": "2021-08-26T14:45:06.489Z",
  "labels": {
    "env": "prod",
    "app": "test"
  },
  "service": {
    "name": "test"
  },
  "tags": [
    "audit"
  ],
  "ecs": {
    "version": "1.6.0"
  },
  "message": "Hello world"
}

But, it seems like log.level should be an Object instead of a string key, like:
{
  "log": {
    "level": "info"
  },
  "@timestamp": "2021-08-26T14:46:10.475Z",
  "labels": {
    "env": "prod",
    "app": "test"
  },
  "service": {
    "name": "test"
  },
  "tags": [
    "audit"
  ],
  "ecs": {
    "version": "1.6.0"
  },
  "message": "Hello world"
}

Below is the example program in TypeScript I was using to test:
import ecsFormat from "@elastic/ecs-pino-format";
import pino from "pino";

const log = pino({
  base: {
    labels: {
      env: "prod",
      app: "test",
    },
    service: {
      name: "test",
    },
    tags: ["audit"],
  },
  ...ecsFormat(),
});

log.info("Hello world");

See also https://github.com/elastic/ecs-logging-java/issues/51#issuecomment-908104672 on how to use the dot_expander processor in an Elasticsearch ingest node pipeline.


I tried to use the new range over func loops. Here is the code:
https://go.dev/play/p/rYiGIM4RaVe
This is a pared down version of a much longer program.  The code here does no longer make much sense, but has no external dependencies and still shows the problem.
What did you see happen?
There are several problems with this code:

On the playground, at https://go.dev/play/p/rYiGIM4RaVe, the code crashes with the message fatal error: concurrent map read and map write.  Note that there is no explicit concurrency in this code.  The crash goes away if the print statement on line 80 is enabled.
On my own system (an M2 MacBook Air), the same code crashes with the following error message:unexpected fault address 0xf48d0c3a929e0
fatal error: fault
[signal SIGSEGV: segmentation violation code=0x2 addr=0xf48d0c3a929e0 pc=0x104d58a50]

I attach the full stacktrace, in case it is useful: stacktrace.txt.  When I enable the print statement on line 80 the problem goes away.
While trying to debug the problem, similar code (which I didn't keep, sorry!) sometimes triggered the panic on line 95 with impossible key values for the map bestM.

What did you expect to see?
The code should not crash, and the absence/presence of the print statement should not change the behaviour of the program.

Related Issues and Documentation

sync: concurrent map iteration and map write on Map error #24112 (closed)
cmd/compile: ICE compiling rangefunc for < 1.23 in combination with Cgo #67497 (closed)
runtime?: "concurrent map read and write" crashes no longer put the 2 offending goroutines at the top of the output #63172 (closed)
maps: segmentation violation in maps.Clone #69110 (closed)
cmd/compile: Go1.22RC1 compiler crash when using GOEXPERIMENT=rangefunc #64930 (closed)
cmd/compile: seg fault with generics, channels and closures  #51355 (closed)
golang/io: `io.Copy` will return an error that contain unexpected fault address when passing `io.MultiWriter` on darwin #60378 (closed)
fmt: stack overflow with self-referenced map #53237 (closed)
cmd/compile: SIGSEGV with 'go build' #50968 (closed)
Heap corruption crash on darwin/arm64 #49865 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Thanks for the report and reproducer!
This looks like some sort of memory corruption of the map. My initial thought is to wonder if the map is stack allocated and the coroutine handling is corrupting the stack/use-after-free.
cc @golang/runtime @dr2chase

Possibly related to #69434

This looks like an escape analysis bug? m := make(map[byte][]string) is stack allocated (confirmed with compiler's -m flag), whereas it is assigned to bestM which is a captured variable in the outer frame.

@gopherbot Please open a backport to 1.23.
This appears to be a compiler bug related to the range-over-function code.  The compiler is generating bad code--a variable that escapes is stack-allocated.  There is no workaround (other than avoiding range-over-func).


Add AtmosphereAutopilot from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of BorisBarboris, to add AtmosphereAutopilot to CKAN.
Please direct questions about this pull request to BorisBarboris.

Closing in favour of #2985


Flink

Apache Flink is quickly gaining momentum as an alternative to Spark Streaming, Storm, etc.

Apache Flink is an open source platform for distributed stream and batch data processing. Flink’s core is a streaming dataflow engine that provides data distribution, communication, and fault tolerance for distributed computations over data streams.

What are your thoughts on developing a plugin for Flink Streaming in StreamFlow? The rationale is that Flink provides a Storm compatible API:

Flink provides a Storm compatible API (org.apache.flink.storm.api) that offers replacements for the following classes:

TopologyBuilder replaced by FlinkTopologyBuilder
StormSubmitter replaced by FlinkSubmitter
NimbusClient and Client replaced by FlinkClient
LocalCluster replaced by FlinkLocalCluster

In order to submit a Storm topology to Flink, it is sufficient to replace the used Storm classes with their Flink replacements in the Storm client code that assembles the topology. The actual runtime code, ie, Spouts and Bolts, can be uses unmodified. If a topology is executed in a remote cluster, parameters nimbus.host and nimbus.thrift.port are used as jobmanger.rpc.address and jobmanger.rpc.port, respectively. If a parameter is not specified, the value is taken from flink-conf.yaml.

StreamFlow now uses an external process for deploying a StreamFlow topology to a Storm cluster. It should be relatively straight forward to implement an alternate submitter. More changes will likely be required to provide the hooks for topology status and metrics.
What's the primary motivation for Flink integration?  Performance?
I saw the word-count performance comparison, but it was comparing a Storm topology to a Flink DSL approach.  Are there any performance comparisons of running an unmodified Storm topology in Flink vs natively in a Storm cluster?


Switch hangs for significant amount of time when using RAND_write_file API with openssl version 1.1.1h and above.

My guess would be that OpenSSL is waiting for the system randomness source to properly seed.  This was an intentional change.  Without it security will likely be lost.

Answered, closing.


Use CloudFormation for the ECS example

https://github.com/weaveworks/integrations/issues/59

Probably should also remove the ecs-cli as it's used in an odd way.


Update virtual-machines-managed-disks-description-customer-managed-ke…

…ys.md
Updated documentation to include expected VM behavior during key expiry / disable / delete / rotate scenarios.

@roygara

Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Add missing anchor link in the README

This links to the new docs for the shim-links-with-button-role.js
cc. @robinwhittleton.

Oops, good catch.