Better Record repr to show ClassMap when it is stored internally

🚀 Feature
Is your feature request related to a problem? Please describe.
More informative to show class_map when a Record object is storing it
Take for example a record loaded from the main repo:
from icevision.all import *

data_dir = Path("~/icevision/samples/")
class_map = icedata.coco.class_map()
parser = parsers.COCOMaskParser(annotations_filepath=data_dir/'annotations.json', img_dir=data_dir/'images')
records = parser.parse(data_splitter=SingleSplitSplitter())[0]
record = records[0]

print(record)

## Output:

BaseRecord

common:
	- Filepath: /Users/rahulsomani/git/icevision-orig/samples/images/000000343934.jpg
	- Image: None
	- Image size ImgSize(width=640, height=480)
	- Image ID: 0
detection:
	- Masks: <EncodedRLEs with 1 objects>
	- Labels: [4]
	- Areas: [43522.80595]
	- BBoxes: [<BBox (xmin:175.14, ymin:175.68, xmax:496.21999999999997, ymax:415.68)>]
	- Is Crowds: [0]

This record internally has access to the class_map via record.detection.class_map, which is great, but not known when you print the record. Additionally, if you print record.components, you get:
{<icevision.core.record_components.AreasRecordComponent at 0x7fbb5b54a4d0>,
 <icevision.core.record_components.BBoxesRecordComponent at 0x7fbb5b54acd0>,
 <icevision.core.record_components.FilepathRecordComponent at 0x7fbb5b54a690>,
 <icevision.core.record_components.InstancesLabelsRecordComponent at 0x7fbb5b54a7d0>,
 <icevision.core.record_components.IsCrowdsRecordComponent at 0x7fbb5b54ad90>,
 <icevision.core.record_components.MasksRecordComponent at 0x7fbb5b54a150>,
 <icevision.core.record_components.RecordIDRecordComponent at 0x7fbb5b54a9d0>,
 <icevision.core.record_components.SizeRecordComponent at 0x7fbb5b54a810>}

I'd have expected ClassMapRecordComponent to be in there as well?

I'd have expected ClassMapRecordComponent to be in there as well?

Hmm, yeah.. So currently what happens is that LabelsRecordComponent inherits from ClassMapRecordComponent. Do you think it's a good idea separating them? A drawback is that we would need to pass ClassMapRecordComponent when creating BaseRecord

To printing class_map that would be a good idea, and very easy to implement, we just need to add a _repr to ClassMapRecordComponent

Do you think it's a good idea separating them?

I think it is a good idea indeed - having clarity on inspecting .components is great because it reduces the expectation of a dev to know what's happening behind the scenes as they can just see what components are being stored.

A drawback is that we would need to pass ClassMapRecordComponent when creating BaseRecord

Right. I think that's not too bad. Alternatively, we could inclue ClassMapRecordComponent in BaseRecord.base_components? Can't think of any common scenario where you don't need a ClassMap (audio, video, images included)

Alternatively, we could include ClassMapRecordComponent in BaseRecord.base_components? Can't think of any common scenario where you don't need a ClassMap (audio, video, images included)

That would not work because each ClassMap is specific to a task, this is also why it's not so straight forward to separate labels and class_map into two different components. We would need to do:
BaseRecord(
        (
            FilepathRecordComponent(),
            ClassMapRecordComponent(task=tasks.detection),
            InstancesLabelsRecordComponent(),
            BBoxesRecordComponent(),
        )
    )

And if we have classification we would need a new class map component for each.

Let's us first address the issue of the class map not being displayed (which is a simple solution) and we can then come back to this (maybe with another more specific issue)


this issue want to reopen:  https://github.com/mui/material-ui/issues/39613

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://mui.com/material-ui/react-accordion/
Double-click an item highlight the all text and all unexpended other accordion items,
Steps:

expend item 1,  double click the item text and make sure all texts are highlighted
expend item 2, you will find the second item texts are also highlighted

Current behavior 😯
https://github.com/mui/material-ui/issues/39613, response as closed and fixed. Actually the tester only highlighted one word. If you double-click to ALL text, you will find the issue
Expected behavior 🤔
the events occurs one item, should not affect the other items
Context 🔦
I want to reopen the issue https://github.com/mui/material-ui/issues/39613
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

@binli120 Thanks for following up ~ I've re-opened the original issue, and updated the repro steps from here!

Duplicate of https://github.com/mui/material-ui/issues/39613


Run a block sync in CI with 2 large checkpoints

Motivation
In #1185, I add a 2000 block sync test, which downloads the second checkpoint.
We want to activate that test in CI.
We can't run it by default, because it can take up to 2 minutes to complete. (Or fail on slow or unreliable networks.)
Solution
Add cargo test commands to CI which runs the ignored zebrad tests.
Related Issues
Based on PR #1185.
Closes #1188.

@dconnolly do you think it's also worth running these sync tests in CD?

@dconnolly do you think it's also worth running these sync tests in CD?

Given that these take ~2 minutes I'm fine with merging these as part of CI, 👍

I need to add a stop_at_height config option, which makes the state service shut down the process when a particular height is reached

I saw a sled recovery (or maybe sled corruption) while I was testing this feature. The process would just hang loading the database.
That's why I added the sync and the extra tests.
Also, without the sync, the genesis block sometimes doesn't get written to disk.

Flushing the sled state is a good idea, but it seems like the other code in this PR is more complicated than I'd expect (or maybe I just don't understand the complexity).  For instance, having to track the caller with StopCheckContext seems like a sign that the behavior is actually different, not common behavior that should be factored into a single function.
Couldn't we just check whether the block was committed successfully, call flush, and then exit the process (in the new block case), or check whether the chain tip is bigger than the stop height (in the constructor)?


Switch subgraph metadata to relational storage

Move subgraph metadata from JSONB storage to relational storage. One nice effect of this is that the one entity we change all the time, SubgraphDeployment, now lives in its own table rather than being intermingled with all the other metadata. Since we have ~ 700 deployments, but between 600k and 900k metadata entries, we are now making changes in a much smaller table.
The migration in this PR takes about 5 minutes against a production dump from yesterday.
I tested this by deploying new subgraphs locally (mostly Uniswap), and by running against a migrated production dump and indexing (from their state in the dump, not from the start) the subgraphs protofire/makerdao-governance, graphprotocol/livepeer, graphprotocol/uniswap, protofire/token-registry, melonproject/melon, and daostack/alchemy
This fixes https://github.com/graphprotocol/graph-node/issues/1394

I don't mean to pile up additional changes, but the templates field within data sources is long deprecated, do you think this PR would be an opportunistic moment to drop support for it by not including it in the relational schemas?

I don't mean to pile up additional changes, but the templates field within data sources is long deprecated, do you think this PR would be an opportunistic moment to drop support for it by not including it in the relational schemas?

The change to drop that data is pretty easy: delete it form the GraphQL schema and regenerate the various generated bits of SQL. Are we sure that removing the templates attribute won't lead to trouble anywhere else? If so, I can add this when I brush up this PR.

@lutter It would also need to be removed from the dynamic data sources query and from the corresponding data structures in data/schema/subgraphs.rs.

@leoyvens We still have some subgraphs that have templates associated with data sources; they are all fairly old and are aragonblack/aragonpm, iherger/melon, zeframlou/betoken, and zeframlou/betoken-mainnet-test. I assume they won't break if I remove the templates attribute from data sources.

Actually, code like this makes me think there is more going on than just removing an unused data field. I now think it's safer to do that removal as a separate issue/PR. When we do that, the main work we'd save doing the change now is dropping the templates column from the two data source tables we'll have - that won't be a big deal since there are only ~ 150k data sources, so not a huge migration.

Rebased to latest master

@lutter this would break those subgraphs, melon and betoken have their official subgraphs under a different org, the only one I'm unsure about is in use is aragonblack/aragonpm.
That code actually looks wrong in that it's not taking top level templates in consideration.
It's fine if you'd rather merge this and leave that to a future migration.

Rebased to latest master

Addressed Leo's comments

Rebase to latest master; remove commit that duplicated #1526

I just reran this against the most recent production dump, and the migration took 10 minutes; before we deploy this to production, I will remove unsued subgraph deployments, which will help make this faster since it reduces the amount of metadata we have. The fact that the production DB has faster IO than my test instance should also help in speeding this up.


adds configurable option to run nginx master process as non-root user

Signed-off-by: Collin McNeese<EMAIL_ADDRESS>Description
Adds a boolean configurable option to the chef-server.rb file of nginx['nginx_no_root'].  This setting will update the runsv process that launches nginx to not use the root user on a system, even for the master nginx process.

Adds attribute default['private_chef']['nginx']['nginx_no_root'] = false
Updates docs page for optional chef-server.rb settings with details of the function and considerations
Updates the fix_permissions recipe to include nginx-related directories if ['nginx_no_root'] is enabled.
Updates the nginx recipe configuration to use proper permissions for resources rather than hard-coded to root:root
Updates the nginx.conf template to not specify a user if ['nginx_no_root'] is enabled as this will generate an error when launching nginx daemon if this is populated and the master process is already set to the worker user.

Acceptance Criteria:

[ ] Omnibus Build deploy and upgrade can support these changes
[ ] Automate hab builds do not break with the current changes.

Issues Resolved
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[X] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/2847#99ef1728-4120-4169-ba9d-b3c1ee279e8a

Testing Report -

Fix permissions should be moved to nginx recipe as those directories not exist.https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/3076#1186b387-f85f-4d90-90ce-df5f951503ed
Tested the changes working as expected in the development environment. Branch - https://github.com/chef/chef-server/tree/shahid/nginx-non-root
Still need to verify the changes in the automate environment for final confirmation.
@PrajaktaPurohit @collinmcneese @tas50

Testing update in Automate environment.
There are some issues with habitat dependencies while deploying chef-server-nginx package in automate environment.I will be continuing the testing after resolving these issues by habitat refresh team.
@PrajaktaPurohit

Automate - uses its own nginx.conf - we will be ok to not test these changes in Automate. Is there a place we could highlight these differences?

@collinmcneese Could you please rebase with main for merging.
cc - @PrajaktaPurohit


[fr] Localize search

Describe your changes
Localize search in French
Related issue number or link (ex: resolves #issue-number)
#2204
Checklist before opening this PR (put x in the checkboxes)

[X] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[X] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

LGTM 😁 🚀


Crawl behavior

Even with stayOnDomain="true" and maxDocuments=10 when the crawler finds a sitemap (https://www.flickr.com/sitemap/sitemap-photos-index-00000.xml) on external site it goes through all the links. Had to cancel crawl.
Am I wrong to thing stayOnDomain="true" would prevent crawling external site?

Yes it should stick to the domain(s) of the URL(s) you provide.  If flickr was not part of your start URLs, it should not be picked up.  Unless your sitemap itself contain an external link to flicker?  Just guessing.  Can you attach your config?

Start is url not sitemap and only one domain. After adding referenceFilters that includes this domain it works as expected.

Great, but it should work without you having to do this.  It would be nice if you can share a config that reproduces the issue.

Thanks to the offline material you sent me, I was able to find the cause and fix it.  The latest snapshot has the fix.
Please confirm.

@OkkeKlein: Have you had a chance to confirm this fix?  Can we close?

This fix was released in 2.5.0.


Matrix vector multiplication

Hello,
I want to compute a Matrix vector multiplication based on your code .... what should I change?
Can the hls code be adapted to any matrix sizes?

Hi,
instead of sending two serialized matrices, you would have to send a matrix and a vector. You have to modify the kernel_mmult function only, I guess.
The size of this approach is limited by the available resources. If you wanna perform larger matrix-matrix multiplications, you would have to use a different approach, i.e. use a systolic array and block matrix multiplication. Have a look at the Vitis gemm examples.


Command line tool for conformation prediction

Hi, this seems like a very nice project. A colleague and I are interested in developing a command-line tool based on a pre-trained model to predict 3D conformation (e.g. from a SMILES file). Is this something you already thought about, or maybe even already started work on?
ping @anuprulez

Hi @simonbray
We haven't thought about this tool! Would be curious to see what comes out of your work on this tool!

Thanks for the reply @mansimov, will keep you informed!


refactor(system-tests): change cluster provisioning

Adjust the system-tests driver configuration to use dcos-launch to
provision clusters.

run system tests

Tell me more about the mesosphere-ci/github integration. I can't find a wiki page about it

@brandonc those should be all our jobs https://jenkins.mesosphere.com/service/jenkins/view/DCOS UI/
We're mostly running pipelines for 1.11 and 1.12 but for older releases we still use simple jenkins jobs.

run system tests

run system tests

soon to be dropped


[train_task_classifyapp.py] Cannot sample 1500 from 0 files found in task/classifyapp/ir_train/1

I called python train_task_classifyapp.py and got this output:
Evaluating ClassifyappInst2Vec ...
Getting file names for 104 classes from folders:
task/classifyapp/ir_train
task/classifyapp/ir_val
task/classifyapp/ir_test
	training  : Read file names from folder  task/classifyapp/ir_train/1
 Traceback (most recent call last):
  File "train_task_classifyapp.py", line 478, in <module>
    app.run(main)
  File "/home/selp/.local/lib/python3.6/site-packages/absl/app.py", line 300, in run
    _run_main(main, args)
  File "/home/selp/.local/lib/python3.6/site-packages/absl/app.py", line 251, in _run_main
    sys.exit(main(argv))
  File "train_task_classifyapp.py", line 470, in main
    dense_layer_size, print_summary, num_epochs, batch_size)
  File "train_task_classifyapp.py", line 289, in evaluate
    len(seq_files)) + " files found in " + folder
AssertionError: Cannot sample 1500 from 0 files found in task/classifyapp/ir_train/1

As far as I understand, this line
classifyapp_accuracy = evaluate(NCC_classifyapp(), embeddings, folder_data,
    train_samples, folder_results, dense_layer_size,
    print_summary, num_epochs, batch_size)

should be replaced with this line
classifyapp_accuracy = evaluate(NCC_classifyapp(), embeddings, "task/classifyapp/seq",
    train_samples, folder_results, dense_layer_size,
    print_summary, num_epochs, batch_size)

Am I right?

Hi @GostD,
Did you extract the dataset in the correct folder? It can be found at: https://polybox.ethz.ch/index.php/s/JOBjrfmAjOeWCyl


[3.0.1-rc1][BUG] QEMU Guest Agent is always enabled

Hi.
According to the documentation by default agent should be set to 0 (disabled).
QEMU Guest Agent is always enabled. Changing to 0 does not work.
Proxmox version: 8.1.4

I can confirm this.
If you have it enabled in your template then it cannot be disabled (agent = 0 is ignored).
If you have it disabled in your template then it can stay disabled by omitting the agent parameter or setting agent = 0 (simply because it is ignored) or can be enabled with agent = 1.
This could be an issue with the api where the value 0 is treated as an invalid value.
if config.Agent != 0 {
		params["agent"] = config.Agent
}

There are two different places where the same code appears:
https://github.com/Telmate/proxmox-api-go/blob/b043e37def62aad374c848259dd1832ee4a85fd4/proxmox/config_qemu.go#L305C2-L307C3
https://github.com/Telmate/proxmox-api-go/blob/b043e37def62aad374c848259dd1832ee4a85fd4/proxmox/config_qemu.go#L1053C2-L1055C3

I can confirm this, thanks for the additional info @babblerz :slightly_smiling_face:
To be clear, when the qemu agent enabled in the template:

Omitting the agent line from the terraform does not disable the agent in Proxmox and
Explicitly setting agent=0 also does not disable the agent in Proxmox

Sorry for not responding. Will you be able to fix this issue? It doesn't allow me to destroy Talos Linux VM's.
Thanks!

@dawidtomczyk An issue was created in the library that needs to be changed to fix this bug. So a fix is in the queue.
As a workaround until this is fixed, you should be able to disable the agent in the template VM that you are cloning and it will be disabled in the VMs based on your template.


[SwiftCompilerSources][build] Require recent Swift compiler in HOSTTOOLS mode

Some code in SwiftCompilerSources, especially the parts that use C++ interop, triggers bugs in older versions of the Swift compiler. To make sure that HOSTTOOLS builds don't produce obscure compiler errors or crashes, let's require a relatively new Swift compiler for HOSTTOOLS build.
Since the CI currently uses Xcode 13.0 beta, let's require Swift 5.5 for now to avoid breaking the CI.

@swift-ci please smoke test


Publish to AWS gives file already open

Describe the bug
Whenever I Publish to AWS from Visual Studio 2019 16.5.1 I get a message dialog that says:
The document '%userprofile%\documents.aws\temp\b38ca48f-610e-486c-95fa-d6596f9af50b.hostedControl' is already open. Do you want to close it?
This showed up after upgrading Visual Studio from 16.4

Yes version 16.5.0 of Visual Studio 2019 has a regression. We have been working with Microsoft to get the issue resolved. We have a fix queued up in the toolkit which is going out very soon. We are tracking the issue using the following issue https://github.com/aws/aws-toolkit-visual-studio/issues/65


Translate DecodeLine tests to parametrized test

Follow up #299.
I try to convert DecodeLine tests to parametrized test style.

Add JUnitParams dependency and use it for Parametrized test

Question:

This parametrized tests makes slightly longer test execution time. Is this execution time acceptable?

@cosmo0920 Did you have chance to try and evaluate testng?
I would like to have feedback to compare testng vs. junit + JUnitParams.

I see. Comparing testng and JUnitParmas is reasonable.
Translate tests with testng in another PR or issue?
But, sadly, I have not enough time until weekend.
Could you wait until weekend?

Could you wait until weekend?

Yes, of course.
Current testing structure has following problems:

Too slow: slow test is bad. this comes from 2 causes:
** Guice and JRuby initialization: initialization of JRuby is taking the longest time. Unless setting use_global_ruby_runtime system config, Guice initialization creates a new JRuby instance. And EmbulkTestRuntime.apply creates a new Guice Injector for each test case. It means that JRuby initialization runs every time before running a test method. This is too slow. I want to solve this somehow.
** parallel execution: running tests in parallel makes it faster, of course.
lack of standarized way to write parameterized tests: because embulk is a tool to process data, we often want to iterate tests for different set of data.
plugin instantiation: to create a new instance of a plugin, we need to get Guice Injector from EmbulkTestRuntime, and get PluginManager from it, and call newPlugin method. And to test transaction method, create a dummy Controller instance to hook TaskSource, then interrupt commit if necessary.

I would like to solve those problems about testing.
TestNG has parallel execution buit-in and some integration with Guice in addition to parametrized test.

I'm implementing testNG style parametrized tests in this branch: https://github.com/embulk/embulk/compare/master...cosmo0920:use-testng
disclaimer: This branch is working in progress. TestNG tests do not work properly.... :<

Or, another plain is using gradle's parallel execution and JUnit experimental Enclosed Runner feature: https://github.com/embulk/embulk/compare/master...cosmo0920:translate-decodeline-tests-to-parametrized-test-with-parallel-fork
This features makes shorten execution test time.

Now, we can choose three strategies implementing parametrized tests

Use junit-parametrized simply (This PR)
Use junit-parametrized and Enclosed runner with gradle parallel execution feature (https://github.com/embulk/embulk/compare/master...cosmo0920:translate-decodeline-tests-to-parametrized-test-with-parallel-fork)
Use testng and add testng task which depends on check task (https://github.com/embulk/embulk/compare/master...cosmo0920:use-testng)

Very good feedback. Please let me take time to check through those code.

Yes, of course.

Sorry that this pull request has been left. It now has a large conflict, and things have been moved. (LineDecoder is now in embulk-util-text.)


[Bug] Tooltip should be "resolved thread" on resolved comments

Description
Tooltip currently says Resolve thread for comments that are marked Resolved. It should read Resolved thread
Steps to reproduce the behaviour:

Important Details

Version: Cloud
OS: Win10
Browser: Chrome
Environment: Release

Closing this, since this is being tracked on #7153


Error running the installer script

I tried running the installer script, but I got a strange error saying Repository coq-released is already set up and points to https://coq.github.io/opam-coq-archive/released-opam1.2.
agnishom@agnishomDuncan:~/soft$ cd coq-platform/
agnishom@agnishomDuncan:~/soft/coq-platform$ ls
charter.md                             notes
COPYING                                opam
coq_platform_make.sh                   README.md
coq_platform_make_windows.bat          shell_scripts
coq_platform_packages.sh               ToDo.md
coq_platform_switch_name.sh            variants.md
example_coq_platform_make.sh           windows
example_coq_platform_make_windows.bat
agnishom@agnishomDuncan:~/soft/coq-platform$ opam switch _coq-platform_.8.12.0+beta1
[ERROR] No switch _coq-platform_.8.12.0+beta1 is currently installed. Did you
        mean 'opam switch create _coq-platform_.8.12.0+beta1'?
        Installed switches are:
          - 4.10.0
          - 4.07.0
          - system

agnishom@agnishomDuncan:~/soft/coq-platform$ ./coq_platform_make.sh
shell_scripts/init_utilities.sh: line 192: [: __unset__: integer expression expected
shell_scripts/init_utilities.sh: line 192: [: __unset__: integer expression expected
================================ INTRODUCTION ================================
This script installs the Coq platform version 8.12.0+beta1, that is:

- the Coq compiler and Coq's standard library
- CoqIDE, a GTK3 based graphical user interface
- various widely used libraries and plugins

The script uses opam, the OCaml package manager, to do all the work.
In case opam is not yet installed, it will install opam.
A new opam switch named _coq-platform_.8.12.0+beta1 will be created.

The script compiles everything from sources, which might takes less than one
hour on a fast machine with lot's of RAM, or up to 6 hours with little RAM.

The script is tested on these platforms:
- Windows 10 with cygwin installed by coq_platform_make_windows.bat
- macOS Catalina 10.15.4 with XCode 11.0.3
- Ubuntu 18.04 LTS
In case you have issues, please report a bug at:
https://github.com/MSoegtropIMC/coq-platform/issues
================================ INTRODUCTION ================================
Enter y to continue with compiling/installing the Coq platform! (y/c=cancel) y

=============================== PARALLEL BUILD ===============================
The Coq platform opam build has two levels of parallelism:

- parallel build of (independent) opam packages
- parallel build inside the make of each opam package

Since a single coqc call can take more than 1 GB of RAM and since the two
above kinds of parallelism multiply, the total amount of memory can be large.
But it is not as bad as one might expect: test show that a full parallel
build takes less than 14GB of RAM with 15 parallel make jobs.

With 32 GB or RAM a parallel package build with 16 make jobs is recommended.
With 16 GB of RAM a parallel package build with 4 make jobs is recommended.
With 8 GB of RAM a sequential package build with 4 make jobs is recommended.
With 4 GB+1GB swap a sequential packahge build with 2 make jobs is recommended.

In order to remove packages, just edit this script at "PACKAGE SELECTION".

In case these recommendations don't work for you, please report an issue at:
https://github.com/MSoegtropIMC/coq-platform/issues
=============================== PARALLEL BUILD ===============================
Build opam packages parallel (p) or sequential (s)? (p/s/c=cancel) p

Number of parallel make jobs (number in 1..16) 15

================================== COMPCERT ==================================
The Coq platform installs the formally verified C compiler CompCert.

CompCert is *not* free / open source software, but may be used for research and
evaluation purposes. Please clarify the license at:

https://github.com/AbsInt/CompCert/blob/master/LICENSE

Parts of CompCert are required for the Princeton C verification tool VST.
Some parts of CompCert are open source and for exploring or learning VST
using the supplied example programs, this open source part is sufficient.
If you want to use VST with your own C code, you need the non open source
variant of CompCert. Before you install the full non-free version of CompCert,
please make sure that your intended usage conforms to the above license.

If you answer n=no below to skip CompCert, VST will also not be installed.

You can also change this later using opam commands.
================================== COMPCERT ==================================
Install full (f), open source (o) or no (n) CompCert? (f/o/n/c=cancel) f

======================= VERIFIED SOFTWARE TOOLCHAIN VST =======================
The Coq platform installs the Verified Software Toolchain VST.

Unfortunately VST takes a while to compile - on a fast machine with 32GB RAM it
may be just 20 minutes, but on a slow machine with 4GB RAM it is more likely
2 hours.

In case you do not plan to formally verify C code with VST, you might want
to select no (n) below. You can install VST any time later with:

  opam install coq-vst.2.6.
======================= VERIFIED SOFTWARE TOOLCHAIN VST =======================
Install VST (y) or do not install VST (n)? (y/n/c=cancel) y

===== CHECKING VERSION OF INSTALLED OPAM =====
Found opam 2.0.5 - good!
/usr/bin/opam
===== opam already initialized =====
===== CREATE OPAM SWITCH =====
[WARNING] No configured repositories by these names found:
          patch_coq-platform_.8.12.0+beta1
[patch_coq-platform_.8.12.0+beta1] Initialised
[ERROR] Repository coq-released is already set up and points to
        https://coq.github.io/opam-coq-archive/released-opam1.2. To change
        that, use 'opam repository set-url'.

Now, I tried deleting the repository with opam repository remove coq-resolved and retried:
agnishom@agnishomDuncan:/tmp/coq-platform$ sh ./coq_platform_make.sh
./coq_platform_make.sh: 14: source: not found
./coq_platform_make.sh: 15: source: not found
./coq_platform_make.sh: 16: source: not found
./coq_platform_make.sh: 20: source: not found
./coq_platform_make.sh: 27: function: not found
coq_platform_make.sh [options]

Create a new opam switch named $COQ_PLATFORM_SWITCH_NAME
and make and install the Coq platform in this switch.

If an option is not given, the option is explained and asked for interactively.
Except for expert users this is the recommended way to run this script.

OPTIONS:
  -h, -help    Print this help message
  -intro=n     Skip introduction message
  -parallel=p  Build several opam packages in parallel
  -parallel=s  Build opam packages sequentially
  -jobs=1..16  Number of make threads per package
  -compcert=f  Build full non-free version of CompCert
  -compcert=o  Build only open source part of CompCert
  -compcert=n  Do not build CompCert and VST
  -vst=y       Build Verified Software Toolchain (takes a while)
  -vst=n       Do not build Verified Software Toolchain
  -switch=k    In case the opam switch already exists, keep it
  -switch=d    In case the opam switch already exists, delete it
./coq_platform_make.sh: 51: Syntax error: "}" unexpected

@Agnishom : sorry for the very long response time - we have been busy.
The root cause for this is that you at some point in the past added a Coq repository for opam 1.2 to your opam 2.X. I guess you have been following some outdated instructions to install some package.
We treat this as an exceptional case - in case it happens to other users as well we will add a sanity check with a more user friendly error message.
The solution would be to call opam repo remove coq-released, but this might have some side effects.

Just wanting to report that I encountered a similar issue. I have had a working opam-based setup for a while, but wanted to switch to the platform instead, and ran into the problem that my coq-extra-dev repository was pointing to http://coq.inria.fr/opam/extra-dev instead of https://coq.inria.fr/opam/extra-dev (the difference is in the http vs https). It looks like an exceptional case indeed, but having some kind of informative error message would avoid having to discover this closed issue.

@MevenBertrand : OK, I re-opened this issue. I will add a sanity check for the default repo names.


java.lang.IllegalArgumentException: No enum constant com.github.messenger4j.receive.events.AttachmentMessageEvent.AttachmentType.FALLBACK

Greetings to all.
My application is receiving the following payload from the Messenger Platform:
{"object":"page","entry":[{"id":"999999999999999","time":1500492242330,"messaging":[{"sender":{"id":"9999999999999999"},"recipient":{"id":"9999999999999999"},"timestamp":1500488692699,"message":{"mid":"mid.$XXXXXXXXXXXXXXXXXXXXXXXXXX-HQ","seq":59099,"attachments":[{"title":"","url":"https://www.facebook.com/VixExplora/photos/a.205505322826303.50136.108929955817174/1490843614292461/?type=3","type":"fallback","payload":null}]}}]}]}
But is throwing the following exception:
java.lang.IllegalArgumentException: No enum constant com.github.messenger4j.receive.events.AttachmentMessageEvent.AttachmentType.FALLBACK
at java.lang.Enum.valueOf(Enum.java:238) ~[na:1.8.0_121]
at com.github.messenger4j.receive.events.AttachmentMessageEvent$AttachmentType.valueOf(AttachmentMessageEvent.java:177) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.events.AttachmentMessageEvent$Attachment.fromJson(AttachmentMessageEvent.java:120) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.events.AttachmentMessageEvent$Attachment.access$000(AttachmentMessageEvent.java:113) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.events.AttachmentMessageEvent.fromJson(AttachmentMessageEvent.java:67) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.callers.AttachmentMessageEventHandlerCaller.createEventFromJson(AttachmentMessageEventHandlerCaller.java:39) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.callers.AttachmentMessageEventHandlerCaller.createEventFromJson(AttachmentMessageEventHandlerCaller.java:23) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.callers.EventHandlerCaller.callHandlerIfResponsibleForEvent(EventHandlerCaller.java:60) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.MessengerReceiveClientImpl.processMessagingEvent(MessengerReceiveClientImpl.java:158) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.MessengerReceiveClientImpl.processEntry(MessengerReceiveClientImpl.java:151) ~[messenger4j-0.8.0.jar:na]
at com.github.messenger4j.receive.MessengerReceiveClientImpl.processCallbackPayload(MessengerReceiveClientImpl.java:144) ~[messenger4j-0.8.0.jar:na]
The platform documentaton (https://developers.facebook.com/docs/messenger-platform/webhook-reference/message), in "Message with fallback attachment from link scraping" says:
attachment object: type: audio, fallback, file, image, location or video.
Please review this issue. I will await your recommendations. Thank you.

Hi, thanks for reporting this.
This is a known bug and will be fixed with the next version.

Hi,
messenger4j 1.0.0-M1 is out. In this version your bug is now fixed.


feat: adds bin details and removes token associations

Description

Adds bin details enrichment
Removes token associations
Adds PATCH reactors and proxies methods

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 3.12.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Create a more attractive and responsive layout

Use Bootstrap to create a more friendly and responsive layout that looks good on both a desktop and a phone.

Improved in 03ccba35f30ad892a934125492abe0b5d2238df4.


Video watch page is broken?

Version
1.13.3
Platform
Windows 10
Steps to reproduce

Details
it was alright yesterday,suddenly shows this,no matter what video url I put in.
Checklist

[X] I have looked through existing issues to make sure that this bug has not been reported before
[X] I have provided a descriptive title for this issue
[X] I have made sure that this bug is reproducible on the latest version of the application
[X] I have provided all the information needed to reproduce this bug as efficiently as possible
[ ] I have sponsored this project
[ ] I have not read any of the above and just checked all the boxes to submit the issue

half an hour later,it repaired itself,i don't know why

Same problem here.. In my case, it doesn't repair at all ㅜ_ㅜ

Same problem here.. In my case, it doesn't repair at all ㅜ_ㅜ

I solved this problem by removing authentication (logout). It seems there is an issue about google account or something..?

half an hour later,it repaired itself,i don't know why

This happens sometimes. Certain transient errors are easier to outwait than to diagnose.


COLLECTIONS-850- Fix Flaky test testMultiValuedMapIterator

PR Overview
This PR propses a fix for the following tests:

org.apache.commons.collections4.multimap.ArrayListValuedHashMapTest.testMultiValuedMapIterator

org.apache.commons.collections4.multimap.TransformedMultiValuedMapTest.testMultiValuedMapIterator

JIRA Issue Link: https://issues.apache.org/jira/browse/COLLECTIONS-850
Test Overview
The test testMultiValuedMapIterator tests how the iteration of a multi-value map works. A MultiValuedMap is basically a map with a key value that can associate itself with either a single value or multiple values (Collections).  Here the MultiValueMap has a key that associates to an array of values.
You can reproduce the issue by running the following commands -
mvn install -pl . -am -DskipTests
mvn  -Dtest=org.apache.commons.collections4.multimap.ArrayListValuedHashMapTest#testMultiValuedMapIterator
mvn  edu.illinois:nondex-maven-plugin:2.1.1:nondex -Dtest=org.apache.commons.collections4.multimap.ArrayListValuedHashMapTest#testMultiValuedMapIterator

Problem
In this test, a multivalued map is defined and an iterator is defined on top of that. The test is flaky since the order inside the multivalued map is not maintained.
Since the order is not maintained, the iterator can have any one key of the three keys for this test ( one, two, three) in random fashion. This flakiness was identified by the nondex tool created by the researchers of UIUC.
The following code snippet fails because of the above reason
https://github.com/apache/commons-collections/blob/35e408717379eed0085cdb29e879209dbadc1ead/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java#L837-L857
Fix
To maintain order, we can implement a TreeMap or a LinkedHashMap but that will change the base code and affect all other entities related to this.
This PR proposes a fix inside the test while keeping the test logic intact.  Since the order is not maintained, we check for the iterator's next value to either be one of the keys and if it is, we check the original assert statements. We follow the same approach to assert all the cases.
https://github.com/apache/commons-collections/blob/02f85e2c9f6cba3a9c76ab30e51bd9d64070977d/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java#L837-L860
This PR provides a fix on the abstract class which fixes the classes that implement it, hence fixing the tests with classes implementing the abstract class

@anirudh711 See https://github.com/apache/commons-collections/pull/426#issuecomment-1902119084

Done differently and more thoroughly in git master. You are credited in changes.xml.


No longer auto-clicking

As of some time this week, the channel points bonuses are no longer being automatically clicked.  I am on Firefox 78.6.0esr.

Duplicate https://github.com/trmcnvn/twitch-channel-points/issues/14

Oh, sorry.  That issue sounded like it was talking about redeeming channel rewards, rather than the points bonuses.


[Bug]: Problem with @types/prosemirror-model after upgrading Angular from 13.3.0 to 13.3.5

What happened?
After upgrading Angular from 13.3.0 to 13.3.5, application is breaking at ng serve with two ocurrences of this error:
error TS2709: Cannot use namespace 'OrderedMap' as a type

Version
v12.4.0
Angular Version
v13.3.5
What browsers are you seeing the problem on?
No response
Link to reproduce
No response
Relevant log output
Error: node_modules/@types/prosemirror-model/index.d.ts:1147:40 - error TS2709: Cannot use namespace 'OrderedMap' as a type.

1147     nodes: { [name in N]: NodeSpec } | OrderedMap<NodeSpec>;
                                            ~~~~~~~~~~

Error: node_modules/@types/prosemirror-model/index.d.ts:1154:41 - error TS2709: Cannot use namespace 'OrderedMap' as a type.

1154     marks?: { [name in M]: MarkSpec } | OrderedMap<MarkSpec> | null | undefined;
                                             ~~~~~~~~~~

Willing to submit a PR?
No response

Can you try v12.4.1. I have pinned all the prosemirror definitely typed packages. Looks someone broke something on recent releases.

Hello, @sibiraj-s ! Thank you so much, it worked! But only after I ran ng new in a new folder, installed all my dependencies anew with npm (including ngx-editor) and and copied my src files to it.
Your package is fantastic and I was worried with having to replace it.

Glad that worked 🎉


The order of the "paths" groups does not respect the alphabetical order of their respective tags.

The way in which the “paths” groupings are being presented does not seem to respect the alphabetical order of the “tags”. Apparently the group titles are being sorted in alphabetical order by “path”. This makes it difficult to control which group should be listed before the other.
I'll try to illustrate the problem with some prints, and I'll also leave here the json of the api description for testing.
json sample
In this example we can see in the routes:
path = /Auth/Authenticate/Token , tags[0] = "1 - Autenticação"
and
path = /AcontaAiqfome/Auth/RefreshToken/{id_empresa}, tags[0] = "5 - Aiqfome"
that the route tagged "5 – Aiqfome" is listed in the left sidebar, before "1 - Autenticação".
I believe it is because the order used is being applied to the path and not the tag.
Is this right?

These examples might help:

https://rapidocweb.com/examples/arrange-by-tags.html#overview
https://rapidocweb.com/examples/collapsable.html

Thanks a lot for the tip, I hadn't seen that the tags were also present in the root element. I'm going to change my json document generator, to sort the tags in root.
Thanks again.

Ok @prassie , I did as shown in the examples you cited and everything worked as I expected. Thank you, I will close this issue.


Repository 'https://github.com/cloudwego/kitex-contrib/' not found

An error occurs while I following the quick-start on the step go run .
go<EMAIL_ADDRESS>invalid version: git fetch -f origin refs/heads/*:refs/heads/* refs/tags/*:refs/tags/* in /opt/go/pkg/mod/cache/vcs/d2a4a1998dd9cf86d2cb6627756503b46f2d0a457aa31fe3bdec9be7f0670bcd: exit status 128:
        remote: Repository not found.
        fatal: repository 'https://github.com/cloudwego/kitex-contrib/' not found

Since this is exactly what #2  is about, I close it as a dupe.


5981 missing event

Fixes:

it generates a 'Delete map' events in case a deleted dataset has dependent visualizations.

Improves:

check if provided user is valid before queuing a new event
add exception message to Rollbar call properties

@Kartones

:+1:


DXOS through cdn on a short lived webserver

Hi guys hope you're well
I'm trying a bit of a hack (using github-pages as psuedo, short-lived web server)
the idea to spin up a frontend repo on github-pages, and also the backend on github-pages on a private repo with keys to the cloud db (upstash) - might be a bit ridiculous haha - but will make both the front and the server hosting completely free
anyways i would love to finally check out dxos (been meaning to use this for ages)
i want to use dxos to make a server logs monitor for testing purposes
again the server lives on a live-per-request, and i cant install any deps
so i searched for examples but couldn't find anything working that does dxos coms through a CDN
the closest thing i've found is at: https://www.jsdelivr.com/package/npm/@dxos/client-services?tab=stats
(Can anyone verify this source? is it legit? security healthcheck seems not great
the code it recommends is:
<script type="module">
import dxosclientServices from 'https://cdn.jsdelivr.net/npm/@dxos/client-services/+esm'
</script>

but couldnt get it to work
so if you can please help:
 1. Can anyone show a server/client example of XDOS imported in vanilla js through a CDN url ?
 2. Can anyone refer to a working github-pages with DXOS ?
Posted on discord aswell.. https://discord.com/channels/837138313172353095/837138313172353098/1311218545346088981
Thanks a lot and all the best!

copying answer from discord:

sounds interesting! unfortunately an unbundled CDN version is not something we’ve really tried before and not something within the scope of what we’re able to support right now. We’re certainly not against it though and in principle it should be possible but its just hard to know how many issues there are that will need to be resolved. So if you (or anyone else) wanted to keep digging we can answer any questions you have to try to unblock you.


move EIA (plant part list) to FERC1 plant granular connection into PUDL

In partnership with RMI, we've developed a granular connection between FERC plant records (which are a combination of whole plants, units, generators, portions of plants based on utility ownership, etc.) to their appropriate collection of EIA generators/owners.
This currently happens in this module. I'd like to move this record linkage processing into PUDL's output/analysis layer.
A prerequisite for this issue is #952.

@cmgosnell @katie-lamb this is about to get merged in with #2224 right?


Adding a simple default DNS query resolver

OK, the PR is as simple as it could be. It adds a service that receives a string and returns the list of addresses. This works using something in /etc/hosts and deploying a headless service in Kubernetes. But this is to start a discussion.
When running in Kubernetes, we might have multiple replicas, and the service will return the addresses of all pods:
$ k get pods -n gingersnap
NAME                             READY   STATUS    RESTARTS   AGE
cache-manager-5869f7b9cb-542z5   1/1     Running   0          27s
cache-manager-5869f7b9cb-p2qcg   1/1     Running   0          27s
cache-manager-5869f7b9cb-z2jwv   1/1     Running   0          27s
db-syncer-6d6cf998d8-gm7cn       1/1     Running   0          27s
mysql-568447df7b-j4t6m           1/1     Running   0          27s

$ k logs db-syncer-6d6cf998d8-gm7cn -n gingersnap
INFO exec  java -Dquarkus.http.host=<IP_ADDRESS> -Djava.util.logging.manager=org.jboss.logmanager.LogManager -XX:+UseParallelGC -XX:MinHeapFreeRatio=10 -XX:MaxHeapFreeRatio=20 -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90 -XX:+ExitOnOutOfMemoryError -cp "." -jar /deployments/db-syncer.jar
2023-01-19 13:25:01,980 INFO  [io.gin.cdc.ManagedEngine] (main) Starting service
FOUND ADDR => cache-manager/<IP_ADDRESS>
FOUND ADDR => cache-manager/<IP_ADDRESS>
FOUND ADDR => cache-manager/<IP_ADDRESS>
2023-01-19 13:25:02,003 INFO  [io.gin.k8s.CacheRuleInformer] (main) Kubernetes client not found, not watching config map
2023-01-19 13:25:02,055 INFO  [io.quarkus] (main) db-syncer 0.1-SNAPSHOT on JVM (powered by Quarkus 2.14.3.Final) started in 0.685s. Listening on: http://<IP_ADDRESS>:8080
2023-01-19 13:25:02,055 INFO  [io.quarkus] (main) Profile prod activated.
2023-01-19 13:25:02,055 INFO  [io.quarkus] (main) Installed features: [cdi, kubernetes-client, micrometer, resteasy-reactive, smallrye-context-propagation, smallrye-health, vertex]

We now have multiple RemoteCacheManager, one for each pod. Keeping in mind that we want to add dynamic membership, some pods can join and leave at any time. We have an issue with how to maintain all these caches.
If pods can arrive at any time, we have to create a new Debezium engine to start consuming events from the very start, otherwise, the pod might miss some data. As a side-effect, each pod might have a different offset and, therefore, different contents in the cache.
The approach I have in mind is similar to how we handle multiple rules. Each pod can fail independently, meaning that backoff and error handling is independent, too. And each pod would be able to proceed at its own pace. No tying all pods from a rule together.
I would like to get feedback before starting a change like that. This is the setup I was running in Kubernetes.

The approach I have in mind is similar to how we handle multiple rules. Each pod can fail independently, meaning that backoff and error handling is independent, too. And each pod would be able to proceed at its own pace. No tying all pods from a rule together.

So if we have x pods and y rules, does that mean we'll have x*y engines? I don't have an idea of how heavyweight these processes are, what kind of overhead will this have in terms of db-syncer resources and load on the DB?

Yeah, that would cause an explosion in the number of engines :disappointed: and I am also not confident enough to say that that would scale well for any DB provider.
One thing, maybe riskier, is to join an engine into another once they reach the same offset. But I'll need to investigate if that is feasible. Once an engine joins onto another, we treat the multiple caches as a single block.

I'll update the PR to add Ryan's suggestion of polling using the DNS resolver to identify joiners/leavers.

Why would it have to be x*y? Would it not just be x for each pod? The rules already defined should be able to be lumped together. The problem would be rules coming up or down. But I think we can just stop the engine(s) and restart them with the new rules configured (the offsets should be maintained).

One thing to note about the DNS resolver here is we will want to have a configuration to disable this or only attempt the first address. For a cache-manager running as a headless k8s service we will want to connect to all services. But in the case of a bare metal clustered installation with a DNS record we don't want to connect to more than one.
In the log term we probably should check the client topology to see if it is clustered or not to determine if we should connect to all DNS records. But for the first pass we can just have a configuration option to enable or disable it.

@wburns Updated and stretched this a bit. Added the property gingersnap.dynamic-membership defaults to false. Added an interface that will deal with the dynamic member, the interface receives the URI to connect to. There is no implementation yet.

Integrated into main, thanks @jabolina !


Doesn't work with AWS ES service

With AWS's managed ES service (my target server), they don't allow the /_cluster/state operation so it errors out here.  Is that totally necessary?

Is it absolutely necessary? No.
It is necessary for alias support in the current implementation.
TODO: Investigate AWS compatible alias support call.

Great, thanks!


Allow margins for grid additional to position

What problem does this feature solve?
I am trying to auto calculate margins in a grid depending if a title, legend, dataZoom is added on top/bottom/left/right. The result is an amount of pixels I want to inset the grid from the position.
Currently this is only possible by positioning the grid with left/top. But this is not working if the grid is positioned relative, e.g. 50% from left. You cannot add an additional offset, e.g. "30px" when a dataZoom is added to the left yAxis.
The "containLabel" option is of no help here, as it doesn't cover dataZooms, titles, legends etc.
What does the proposed API look like?
Add following options to Grid option:
// Set margin to be 5 (px)
margin: 5
// Set the top and bottom margins to be 5 (px), and left and right margins to be 10 (px)
margin: [5, 10]
// Set each of the four margins seperately
margin: [
5,  // up
10, // right
5,  // down
10, // left
]

An even more powerful approach throughout the library would be to allow calculations widthin %/px attributes (e.g. left,right,top,bottom,width,height) similiar to calc() in CSS3.
E.g.
left: calc(50% + 18px)

An even more powerful approach throughout the library would be to allow calculations widthin %/px attributes (e.g. left,right,top,bottom,width,height) similiar to calc() in CSS3.
E.g.
left: calc(50% + 18px)

This would be very useful


Added seconds tag to stratux.appcache for page reloading if built cod…

…e has changed

Why wasn't stratuxBuild enough? It should change every time the web files change.


Add resetting of weeklyXP

Add resetting of weeklyXP at the start of a new week

If you'd like to work on this, let me know and I'll send more info


feature-ui-input-module

Basic Input Functionality, Prefabs, and Example Scene

We should not have any scenes in the build settings, currently two scenes from PinchUtility are there.

We should also not be using any layers / tags, since users will not be importing tag settings when they import module packages.

Looks like you have an empty Assets/Editor folder that is leaving a meta file around.

So this is looking all good for a merge, but I see that the most recent commit has the words 'first pass' in it, do you want me to hold off?

Ok everything looks good to me!  Merging in!


Allow ability to ignore specific NICs in linux guest agent

Resolves https://bugzilla.redhat.com/show_bug.cgi?id=1437145

Hi,
Thanks for the patch.
This repository is a mirror for gerrit.
Check these:
https://gerrit.ovirt.org/#/q/project:ovirt-guest-agent
https://www.ovirt.org/develop/dev-process/working-with-gerrit/
Thanks

@rollandf thanks, done.
https://gerrit.ovirt.org/#/c/74877/
You can close this if github PR's are not the defacto, I'm just so used to being in github 100% of the time

@myoung34 I don't have permissions to see the patch. Did you push your patch as draft ? I yes, publish it so that the maintainers of the project will be able to review. There is a "publish" button in the gerrit UI.


Crash when re-entering AR repeatedly

Mentioned by: @jacobwyn
After launching 5-10 times app (launch, put some 3d models, press back, repeat few times) I am getting next error

04-06 10:42:57.072 16549-17330 E/Adreno-GSL: <gsl_memory_alloc_pure:2202>: GSL MEM ERROR: kgsl_sharedmem_alloc ioctl failed.
04-06 10:42:57.328 16549-17330 W/Adreno-GSL: <sharedmem_gpuobj_alloc:2318>: sharedmem_gpumem_alloc: mmap failed errno 12 Out of memory

And app is stuck, can reproduce it even in ARRetail
I have a suspicion that it is somehow connected with bug 4 from here #69
My app https://github.com/jacobwyn/augmentedReality

Hi @jacobwyn I was unable to repro this with the ARRetail app on a Pixel 2 phone. I tried entering and exiting the Viro AR Activity about 20 times and the app didn't crash. Are you able to consistently repro this? What device are you using? Thanks!

We recently updated to ViroCore 1.6.0. Release Notes and upgrade instructions here -> https://virocore.viromedia.com/docs/releases
Were unable to repro, closing issue. Please reopen if you are still having issues and provide additional information and/or repro steps.


feature: Proguard configuration for turbo image

What feature or enhancement are you suggesting?
I have successfully integrated turbo image and it works properly and a perfect stand in replacement for fast image with new architecture support.
Can you add proguard support if there is one to the readme so i can use it for my project.
Thanks!
What Platforms whould this feature/enhancement affect?
Android
Alternatives/Workarounds

@Gautham495
According to coil's docs, Coil is compatible with R8 obfuscation, you don’t need to add other rules. If you need to obfuscate the code, you may need to add the corresponding obfuscation rules: Coroutines, OkHttp.

Then I saw okhttp docs

R8 / ProGuard
If you use OkHttp as a dependency in an Android project which uses R8 as a default compiler you
don't have to do anything. The specific rules are already bundled into the JAR which can be
interpreted by R8 automatically.
If you, however, don't use R8 you have to apply the rules from this file. You might
also need rules from Okio which is a dependency of this library.
The Coil's docs in English version does not mention Proguard.  I think we don't need Proguard configuration for Coil.

https://developer.android.com/build/shrink-code

When you build your project using Android Gradle plugin 3.4.0 or higher, the plugin no longer uses ProGuard to perform compile-time code optimization. Instead, the plugin works with the R8 compiler to handle the following compile-time tasks:

Closing it since no need to add ProGuard.


Fix block size

This branch is the result of me being bad at git and setting up my upstreams wrong, so that a new branch was pushed rather than a remote PR branch


Downloadable weights

Hi PeptideBERT team! This paper looks so exciting!
I would love to give it a whirl on some peptide sequences without going through the training process.
Would it be possible to share a link (e.g. google drive, or otherwise) to download pre-trained weights?
Thanks in advance!

You can access the model weights from the Releases section of the repository.


blueprint: Rename reference to stitch

The recent rename of stitch to blueprint forgot to rename a test file.

Thanks for fixing this! Sorry I missed it.


AVRO-2658: Fix JavaScript Linting

Jira

[x] My PR addresses AVRO-2658.
[x] My PR addresses that issue in its title.
[x] My PR adds jshint in the Dockerfile. jshint is licensed with the MIT license, which complies with the ASF 3rd Party License Policy.

Tests

[x] This PR fixes linting. It adds no run-time functionality, so does not require additional testing.

Commits

[x] My commits all reference the Jira issue in their subject lines.
[x] My commits follow the guidelines from "How to write a good git commit message"

Documentation

[x] This PR fixes linting. It adds no run-time functionality, so does not require additional documentation.

Duplicate of #651 and #705?

Whoops, closing as dupe.


register interest

as a health care organisation I want to easily register interest in healthsites so that I can be supported in sharing open data

we are using osm account now


F sharp templates

swtiched out some if.. then for pattern matches.  Changed 2 loops to a Seq.iter

Hi @jamessdixon, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@jamessdixon, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@sylvanc can you take a look?

LGTM :)

These changes are not necessary, though thanks for proposing them

while some F# programmers like to pattern match on single boolean true/false values, i'ts not particularly idiomatic to do that - indeed I suppose if/then/else would normally be seen as marginally more idiomatic,

For Seq.iter, much the same applies: some F# programmers eschew the use of "for". However there can also be good reasons to use it, notably the way it combines with computation expressions and comprehensions.

cheers and thanks
don

Who is this Don Syme and what does he know about F#?  Granted the changes aren't necessary, I have found that syntax like that does help when transitioning people from imperative constructs into a more functional style.  Since these templates are going into Visual Studio and a majority of VS Users are C# devs, it seemed like a easy point of education.

@jamessdixon :) I understand - yes, it's tricky balance between familiarity (more like C#) and teaching (use more F#-specific constructs).
However for these templates I think the emphasis shouldn't be on teaching F#, just on landing people with an Azure Function that does the job for them

I was going to add some more templates.  Can I use that style or do you want consistency in the code base?

@jamessdixon Consistency please (unless that prevents you adding the templates :) :) :) )

The empty template in Visual Studio right now has an unneeded semi colon:
let Run(log: TraceWriter ) =
log.Verbose("F# function executed.");
Want a pull request for that?

Visual studio is using older version F# templates, All empty templates have been removed from the repo. Also FSharpTemplates branch is no longer in use, It was created to temporarily hold some F# Templates. Please use send all future PR's against dev branch. :)

OK - will do
When will these templates be pushing into VS?  Should I ask the VS team?

VS team controls that timeline, I will ask them and let you know.

Thanks- I can also ask on the MVP thread if you don't get a fast answer

Summoning @lindydonna who has the answer on VS stuff.
FYI - our cli uses these templates when you run locally and it pulls from GitHub directly.

@jamessdixon We don't have a concrete timeline on when there will be a refresh of the Azure Functions Visual Studio tools. Currently it uses a snapshot of the templates feed from 0.6. We're changing up how the tooling works, which is why I don't have anything more concrete. :)

From the discussion above it looks like we might not be merging the changes, would it be okay if we close this PR?

Sure - though I think Don is making a mistake.  Let the records show that I shook my fist at him.  Thanks!

:)


Display HW usage information in a few graph views

As a Pipeline Builder/ HKube manager I want to be able to view HW usage information in a few graph views (one graph per information type), so that I can analyze HW overload and sizing over time.
User story implementation details:

The information to be displayed in the graph view will be detailed in other user stories.
The user will be able to filter the information displayed in all the graphs by time frame.
The information will be displayed per container using the container name.
The user will be able to filter the information displayed in all the graphs by choosing specific container .

TID-201


W-11732863-Create-v7-14-branch-CP

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

This PR seems to be no longer needed.
Closing this PR.


write rotation back onto entity

right now rotation is internal.   seem to be running into conversion issues when going from  world -> local -> world space and quaternion -> euler.

blocks https://github.com/caseyyee/aframe-vreditor-component/issues/5


removed unnecessary torch import on line 115

as per #1290

@sdbds You may not need to import torch there because torch is already imported at line 8.
If you do import torch inside a function, the Python interpreter regards torch as a local yet unbound variable until the import statement actually runs.

Or we do have another choice to add a global torch (or a nonlocal torch for this case) statement at the beginning of the function definition.

@kohya-ss thank you. since this PR has been merged, I will close #1290.


try osx arm 64 and linux aarch64

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

This is necessary for building QGIS in osx_arm64 and linux_aarch64
conda-forge/qgis-feedstock#289

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Aparently this is working. @gillins wanna give it a look and see if it's what we need for QGIS?

Looks good


Fix modal padding in authmodal.js

This PR has fixes for issue https://github.com/precision-sustainable-ag/dst-selector/issues/510
The modal after changing the padding from 4 to 2 looks as below:

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

this looks the same as before

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


Rider - Support later versions

Is there any reason why the Rider plugin doesn't support versions higher than 213? I tend to use EAP versions and I can't use this plugin with it as it has a maximum version defined.
Happy to guinea pig it and report back if that'd help.

It looks like rider requires you to opt in to supporting new IDE versions. This is the first new version of rider that has come out since the csharpier plugin was created so I wasn't aware of that until now.
I tested the plugin with the EAP and submitted a new version to jetbrains, it takes them a couple days to approve things. I'll throw the version up here too so you don't need to wait for it.

Here you go - https://github.com/belav/csharpier/releases/tag/rider-1.2.4

This is great thank you. I can see JetBrains have already approved it and I'm using it now. Will report back any findings. Thank you again for CSharpier, it's fantastic and becoming a key part of my workflow.


Add an option to dump the current effective config file

Would be great to be able to dump the effective envoy.json file at run time. When using xDS it can be tough to tell exactly what the configuration envoy thinks is being used. It would be handy for debugging that we could just take the config, tweak it, compare it, and maybe even run it outside of the xDS config environment.
something like /dump_config for the admin server would be great.

+1 -- would love to be able to gauge whether perf problems I'm working on would impact Istio and other applications that have raised performance issues.  Having a way to dump a config would hopefully make it easier to repro real world cases to ensure I'm optimizing for the right thing.

+1

+1
@htuch any chance someone could take a look at this. Would love to be able to get a dump back in ADS form (or some other stable API) that we could rely on when writing complex integration tests in Istio to detect when configuration has propagated.

I'll work on this - I'll try to prioritize it as there seems to be a lot of interest

@gsagula is interested in taking a look. @jsedgwick can you coordinate with @gsagula and decide how to approach this? Please ping me on Slack for more detail.

@htuch It makes more sense to assign it to @jsedgwick
I can work on some other issue. Thanks!

Cool, reassigned to @jsedgwick.

+1


rework newlib directory prioritization to fix include processing for non-Zephyr toolchains

This work closes #17564 and related C++ failures.
Zephyr builds with -ffreestanding meaning that __STDC_HOSTED__ is false.  Consequently the toolchain standard <stdint.h> is a minimal header that routes to <stdint-gcc.h> which defines integer types without defining the marker macro __int64_t_defined that signals newlib <inttypes.h> to provide support for PRIu64 (#14310).  In hosted builds <stdint.h> instead sets some flags then delegates to the host-provided (newlib) implementation of <stdint.h>.
In support of the desire to have support for integer types that aren't part of the freestanding system #14312 placed the newlib include headers before the system ones, specifically to make hosted features available in a non-hosted system.
This decision breaks building C++ applications that use standard C++ include files when using a non-Zephyr toolchain like gnumarmemb.  The C++ header <cstdlib> (referenced from <array> in #17564) wants to #include_next <stdlib.h>, but the include priority has placed the directory that provides it earlier in the search path, where it can no longer be found.
The workaround succeeded with the Zephyr toolchain only because the Zephyr SDK uses a cross-compiler infrastructure technique that leaves behind and references a second copy of the base include directory that is normally needed only when building the compiler.  <stdint.h> is then found at that second location.
This PR switches to a solution based on updating the include path to include an intercepting header that provides the required defines to satisfy the use case of #14310 without completely breaking the toolchain standard include processing.  This should work on GCC and clang.
Note that the test case requires a patch from #18243 to support ztest with C++.

Background information:
To understand the details you need to know what the system header search path is for a given compiler.  Get this with:
target-g++ -xc++ -v -E - < /dev/null

With ZEPHYR_TOOLCHAIN_VARIANT=zephyr the standard system header search paths are (e.g.):
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include/c++/8.3.0
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include/c++/8.3.0/arm-zephyr-eabi
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include/c++/8.3.0/backward
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/lib/gcc/arm-zephyr-eabi/8.3.0/include
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/lib/gcc/arm-zephyr-eabi/8.3.0/include-fixed
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/sys-include
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include

By adding -isystem ${LIBC_INCLUDE_DIR} this changes to:
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include/c++/8.3.0
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include/c++/8.3.0/arm-zephyr-eabi
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/include/c++/8.3.0/backward
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/lib/gcc/arm-zephyr-eabi/8.3.0/include
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/lib/gcc/arm-zephyr-eabi/8.3.0/include-fixed
/usr/local/zephyr-sdk-0.10.2/arm-zephyr-eabi/arm-zephyr-eabi/sys-include

The contents of the zephyr toolchain path ending in /sys-include is identical to that of /include except that it lacks the c++ subdirectory.  It is an artefact of a multi-stage toolchain bootstrap that allows libc to be built using a compiler that doesn't have a real libc.
With ZEPHYR_TOOLCHAIN_VARIANT=gnuarmemb the standard system header search paths are (e.g.):
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include/c++/8.3.1
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include/c++/8.3.1/arm-none-eabi
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include/c++/8.3.1/backward
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/lib/gcc/arm-none-eabi/8.3.1/include
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/lib/gcc/arm-none-eabi/8.3.1/include-fixed
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include

By adding -isystem ${LIBC_INCLUDE_DIR} this changes to:
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include/c++/8.3.1
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include/c++/8.3.1/arm-none-eabi
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/arm-none-eabi/include/c++/8.3.1/backward
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/lib/gcc/arm-none-eabi/8.3.1/include
/usr/local/gcc-arm-none-eabi-8-2019-q3-update/lib/gcc/arm-none-eabi/8.3.1/include-fixed

Note that this build does not have /sys-include because the ARM folks use a two-stage build where the final build doesn't include it.  Consequently #include_next for any file in the normal include path will be found in /sys-include for Zephyr toolchains, but will not be found in other toolchains that don't have duplicated standard include directories.

@pfalcon Please check this branch for the use case underlying #15627.  It essentially reverts a change from #14312 that you used to justify #15937, i.e. using -isystem for a libc header path in order to change its priority.
That change was incorrect for newlib, but is arguably appropriate for minimal libc.
I don't have a test case for #15627 but if your solution depended on the newlib use of -isystem this commit may have broken it again.  If so, I think the correct path is to add support for
a zephyr_early_include_directories() macro that uses the cmake BEFORE tag instead of the SYSTEM tag to ensure that some directory is placed earlier in the search path.  Or possibly use the technique demonstrated here to avoid name conflicts.

Please check this branch for the use case underlying #15627. It essentially reverts a change from

@pabigot: As a quick "first thing which comes to mind", based on wording above and "Commits 3 Files changed 11", I'd say that this is too late for 2.0.
I'll try however to look in detail as time permits.

@pfalcon Only one commit, and a change to two files, fixes this bug, which I thought was in scope for this stage of the release process.  The other two commits are (1) a test case to avoid regressions, and (2) additional cleanup(bug) fixes queued in #18243 necessary to do unit tests on C++ code.
However, I'm not adamant about merging this, as long as it's OK to leave C++ broken in 2.0.0 with non-Zephyr toolchains.

@pabigot

I don't have a test case for #15627 but if your solution depended on the newlib use of -isystem this commit may have broken it again.

So, testcases are includes in a PR that fixed that issue, https://github.com/zephyrproject-rtos/zephyr/pull/15937 , specifically, https://github.com/zephyrproject-rtos/zephyr/pull/15937/commits/a40cc0d18b07e410125517e3abb1116257b77433 . I.e., with my patch, POSIX-using apps no longer need to do target_include_directories(app PRIVATE $ENV{ZEPHYR_BASE}/include/) (which is kinda implementation detail of POSIX subsys, which should not leak into config of each and every user app).
As can be seen, CI is green for this PR, so hopefully teh handling above is not broken. Well, I'd still hope to try it manually myself to be double-sure (tomorrow, perhaps).
But otherwise, I'd leave it to @galak, the author of  https://github.com/zephyrproject-rtos/zephyr/pull/14312, to vote this PR up/down.

https://reviews.llvm.org/D32411 includes a commit that documents a design to support #include_next on MSVC, which doesn't have it, using nasty looking code and CMake.  However, the practice within newlib itself is to simply provide the missing content inline.
Are there non-clang/gnuc toolchains we need to worry about?

Ok, manually tested tests/posix/common, tests/posix/fs - all's ok (yeah, that should be covered by CI, but as we know there's a bit randomization and subsetting in it, so some tests aren't run all platforms, and given a trouble it took to lead POSIX-related changes into the mainline, I wanted to have an explicit checkpoint here).
So, I'd be happy to +1 this, just giving some time to @galak to review too.


Run google-cloud-python system tests against new runtime images.

Meta comment: It looks like you're running all of the tests inside a "docker build". It might be a little more straightforward to inject the tests into your docker image using a volume mount during a "docker run", and then run the tests that way.

@dlorenc how does that play with cloud build?

@dlorenc how does that play with cloud build?

It works fine with cloud build. You get full access to a docker daemon.

Please take another look.  I also addressed some things that Jon and I talked about over email.

Ok, can you take a final(?) look?


Template is missing source_ami_id in the variables section

The template reference {{ user `source_ami_id`}} but no such user variable is declared. The Makefile also passes this as a variable.

Hmm, this seems like a bug in packer...

Packer issue hashicorp/packer#6473

It works for me, with packer 1.2.4, which version of packer are you running?

Yeah, I realise it works, but it shouldn't, hence I filed hashicorp/packer#6473. When that is fixed this won't work.

Awesome, I'm not an expert on github, but I added the fix to my previous PR , see issue #2 for details.


Github damianszczepanik cucumber-reporting integration poc

From twitter:
https://twitter.com/bbrannic/status/1077892960426041344
The following repo has a good reporting solution which expects the test-run results in JSON format
https://github.com/damianszczepanik/cucumber-reporting
Zerocode already produces test-run results in JSON format.
Some subset of data from this is used for CSV reporting and some for HTML reporting.

Currently it uses extent-reporting for drawing aggregates, search/filter, fuzzy text matching etc

<dependency>
    <groupId>net.masterthought</groupId>
    <artifactId>cucumber-reporting</artifactId>
    <version>(check version above)</version>
</dependency>

TODO- (ACs)

Can anyone pick this as a POC to see if we have all the data available for drawing the cucumber-reporting HTML graph ?

Figure out if there are any optional and mandatory firlds in the cucumber-reporting drawing.

If we have all the necessary fields available in the raw JSON, then draw a sample report and attach it to this ticket or your own repo e.g. sample

Also verify the responsiveness of the drawn report for sanity only(optional)

Reference

Background section has this. Please visit this page.

The "Usage" section has code samples how to integrate.
https://github.com/damianszczepanik/cucumber-reporting#usage

Please reachout to the collaborators/authors @damianszczepanik for any low lavel details as and when needed.

I've worked on Cucumber before, I'd like to take this issue and work on the PoC. Can it be assigned to me please? @authorjapps

Do you need help with this one, @authorjapps?

Yes please, Thank you 👍
The below info might be helpful for proceeding with the ticket.
e.g.
When you run any test project using Zerocode, it generates the raw reports into the following folder

/target/zerocode-test-reports

You can simply clone the HelloWorld repo and do a mvn clean install.
You will find lot of these reports generated per test case wise into the above folder.
You can pick any .json file and see what all fields can be mapped to the cucumber report generation.
Then you can fork the Zerocode source code, and do the changes.
Happy to help further if you need us to!

Hi,
I have started to compare sample cucumber.json and zerocode json

we should explicitly mention scenario name, given, when, then like

Anyone working on this. I will like to contribute

is this issue Resolved?


Solidity/Vyper version selector interface design

[ ] follow python interpretor selector style for solidity version selector

after v2 merge ethcode will not use react panel anymore.


Traefik for Kubernetes in Azure (redirect to https for the frontend)

Do you want to request a feature or report a bug?
Bug
What did you do?
According to the official documentation, the redirect to https should be straightforward. For GKE (Google Kubernetes) it works right away indeed. For Azure Kubermetes (AKS) it worked once and then after Traefik redeployment it didn't get back to normal. Whatever I tried didn't work.
The config looks as the following:
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: traefik-conf
  namespace: kube-system
data:
  traefik.toml: |
    defaultEntryPoints = ["http","https"]

    [entryPoints]
      [entryPoints.http]
        address = ":80"
        [entryPoints.http.redirect]
          entryPoint = "https"
      [entryPoints.https]
        address = ":443"
        [entryPoints.https.tls]

    [frontends]
      [frontends.frontend2]
        backend = "backend1"
        passHostHeader = true
        # overrides default entry points
        entrypoints = ["http", "https"]

    [backends]
      [backends.backend1]
        [backends.backend1.servers.server1]
           url = "http://login.mynewsite.com"

What did you expect to see?
Expected behaviour: everything should work as advertised.
What did you see instead?
As I mentioned, whatever I do in the ConfigMap or at the Ingress level doesn't affect the actual behaviour. I have a sense it's been cached somewhere. Also, the annotations I implement are not reflected in the Traefik dashboard for the selected frontend (I expect to see color labels at least which indicate the rules applied).
Output of traefik version: (What version of Traefik are you using?)
1.7.9
What is your environment & configuration (arguments, toml, provider, platform, ...)?
A sample of Ingress for my application with annotations
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: identityserver-ingress
  annotations:
    kubernetes.io/ingress.class: traefik
    traefik.ingress.kubernetes.io/frontend-entry-points: https
    traefik.ingress.kubernetes.io/redirect-permanent: true

spec:
  rules:
  - host: login.mynewsite.com
    http:
      paths:
      - backend:
          serviceName: identityserver-svc
          servicePort: 80
  tls:
    - secretName: traefik-ui-tls-cert

One more weird thing the dashboard for Traefik didn't work for two days last week no matter what I've done. It just started working by itself yesterday. Kubernetes supposed to be the same everywhere. I use 1.11.8 in AKS as well as GKE, however, in GKE everything works like a charm in AKS does not.

Hi and thanks for reaching out!
Could you show us the logs involved so we can better understand what's going on?
Thank you.

Hi! How to gather the logs, please?

Hi! I'm Træfiker :robot: the bot in charge of tidying up the issues.
I have to close this one because of its lack of activity :disappointed:
Feel free to re-open it or join our Slack workspace for more community #support.


Allow to set an ERC20 token balance using json-rpc in anvil

Component
Anvil
Describe the feature you would like
I am currently forking anvil in Rust using alloy and I would like to set the token balance of my wallet to a specific amount
Additional context
I know that there is deal in forge when we are writing solidity tests, but is there a way to achieve the same behavior using the json-rpc API?

Thank you, I have found:
await network.provider.send("hardhat_setBalance", [
  "0x0d2026b3EE6eC71FC6746ADb6311F6d3Ba1C000B",
  "0x1000",
]);

Do you have an example of setting the balance of an ERC20 token?


[proposal] Enable default outside switch case

class PermissionStatus {
  static final granted = PermissionStatus._('granted');
  static final denied = PermissionStatus._('denied');
  static final default = PermissionStatus._('default');

  PermissionStatus._(this.value);

  final String value;
}

'default' can't be used as an identifier because it's a keyword.
Try renaming this to be an identifier that isn't a keyword.dart(expected_identifier_but_got_keyword)

This would likely mean making default a "built-in identifier", which is one step down the restriction list from "reserved word".
You can use built-in identifiers as variable names, but not as type names. That should be sufficient since that prevents default foo from being the start of a valid variable declaration.


Latest Revisions fail to compile under WP7

Revision 763cd269410e345ca9f1d373f8a2147701a0f948 (merge pull) broke WP7 builds due to unsatisfied reference to Type.FindInterfaces on WP7.

Hey, does not seems to be fixd yet? I downloaded the latest TinyIoC.cs today, included it in my Windows Phone 8 project, and it fails on line 3933, in the method "IsValidAssignment":
if (!registerImplementation.FindInterfaces((t, o) => t.Name == registerType.Name, null).Any())
return false;
The compile error is: "'System.Type' does not contain a definition for 'FindInterfaces' and no extension method 'FindInterfaces' accepting a first argument of type 'System.Type' could be found (are you missing a using directive or an assembly reference?)"
Do you have a quick fix to me so I can compile the project?

@rodsjo the latest version is PCL, which include WP8, so you shouldn't have issues with that - can you try adding it to a PCL library and linking that to your project? It might need the compiler to put the reflection shims in.

@grumpydev Thanks mr. Grumpy, it worked using #define PCL :-)

Phew :)


[build-script] Always clean 'SwiftTesting' and 'SwiftTestingMacros'

These products are built with CMake. In incremental build environments, these products don't detect compiler changes, so the artifacts aren't rebuilt unless the source code of these projects are changed. To workaround that, always clean them in build-script to ensure they are rebuilt.
rdar://135021207

@swift-ci Please smoke test


rxjs/webSocket SyntaxError: Unexpected token s in JSON at position 0

Bug Report
Current Behavior
using string urlConfigOrSource generates SyntaxError: Unexpected token s in JSON at position 0
with WebSocketSubjectConfig and defined deserializer function error does not appear.
Reproduction
    this.pubsubSubject = webSocket('ws://localhost:8080');
    this.pubsubSubject.subscribe(
      (msg) => console.log('message received: ' + msg),
      (err) => console.log(err),
      () => console.log('complete')
    );

throws error
SyntaxError: Unexpected token s in JSON at position 0
    at JSON.parse (<anonymous>)
    at deserializer (WebSocketSubject.js:12)
    at tryCatcher (tryCatch.js:6)
    at WebSocket.socket.onmessage [as __zone_symbol__ON_PROPERTYmessage] (WebSocketSubject.js:174)
    at WebSocket.wrapFn (zone.js:1188)
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:421)
    at Object.onInvokeTask (core.js:3811)
    at ZoneDelegate.push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:420)
    at Zone.push../node_modules/zone.js/dist/zone.js.Zone.runTask (zone.js:188)
    at ZoneTask.push../node_modules/zone.js/dist/zone.js.ZoneTask.invokeTask [as invoke] (zone.js:496)

to fix this we need use:
this.pubsubSubject = webSocket({
    url: 'ws://localhost:8080',
    deserializer: () => {}
});

after that this.pubsubSubjectsubscribe() doesn't throw error
Environment

Runtime: @angular 6.1.8 chrome 69.0.3497.100
RxJS version: 6.3.2
angular-cli 6.2.3,

You need to ensure that the JSON received is valid - I hit this when receiving an unquoted string for example.
You can test this by passing the data received to JSON.parse - this is what the underlying code will use

By default webSocketSubject is trying to deserialize the message by applying JSON.parse(). If you message is not an string ig going to fail (the error you are getting now). Now when you tell to webSocket to use this deserializer () => {}, you are telling it, just give me back an empty Object.
Here you have two options,

use JSON.stringify on the server or any other tool to make the message an string.
use a custom deserializer but not forget to get your data back. 😄

Example:
const WebSocketSubject = webSocket({
    url: 'ws://localhost:3200',
    // Here you can apply whatever transformation you want
    deserializer: msg => msg
});

What do you mean by

because it doesn't support async

Since you are sending (from server to client) chunks of data. You can make use perfectly of  deserializer property. Let's say you have,
Server.js
const src = fs.createReadStream(`${__dirname}/file.pdf`);
        src.on("data", chunk => {
            ws.send(chunk);
        });
        src.on("end", () => {
            ws.send('end');
        });

Now, in your client you can:
let pdf = new FileReader();
const WebSocketSubject = webSocket({
    url: "ws://localhost:3200",
    deserializer: msg => {
        const res = msg;
        if (res.data === "end") {
           // Here you have a complete pdf as Arraybuffer, you can do whatever you want
            console.log( pdf );
        } else {
            pdf.readAsArrayBuffer(msg.data)
        }
        return { type: "error" };
    }
});

@iamcco thanks for sharing your code, that makes clear your problem. I think your issue https://github.com/ReactiveX/rxjs/issues/4270 is pretty interesting and I support your request. But meanwhile you can use this as a workaround, let me know if it fills your requirements.
I tried to keep using your code .
const blobSubject = new Subject(), reader = new FileReader();
const blobMessages$ = blobSubject.asObservable();
let chunks = [];
reader.onloadend = event => {
    const response = new TextDecoder("utf-8").decode(event.target.result);
    const message = { type: "Blob", file: JSON.parse(response) };
    blobSubject.next(message);
};

const api = options => {
    const subject = webSocket(options);
    return {
        subscribe: (params: any) => {
            let observableA = subject.multiplex(
                () => params,
                () => ({ close: true }),
                (message: any) =>  {
                    return message.requestId === params.requestId
                }
            );
            observableA = concat(blobMessages$, observableA);
            return observableA
        }
    }
}
const ws = api({
    url: "ws://<IP_ADDRESS>:50055/ws/v2",
    protocol: 'json',
    binaryType: 'blob',
    deserializer: ({ data }) => {
        if (data === "end" && chunks.length > 0) {
            reader.readAsArrayBuffer(new Blob(chunks));
            chunks = [];
            return {type: "error"};
        } else {
            if (data && data.type === "") {
                chunks.push(data);
            } else {
                return JSON.parse(data);
            }
        }
        return {type: "error"};
    }
});

Notice I am using onloadend event instead of onload  since this event (onloadend) is triggered each time the reading operation is completed (either in success or failure). Then you can,
ws.subscribe({requestId: '1', data:{})
    .subscribe(data => {
        console.log(data);
    });

I just create a Branch here, you case you want play with!
https://github.com/luillyfe/rxjs-websockets/tree/features/blob-type-support

@luillyfe thanks for your help. i had check your branch, but i can't get it work.
it seems that this doesn't work.
this line:
https://github.com/luillyfe/rxjs-websockets/blob/features/blob-type-support/src/app.js#L89
binary will not subscribe before blobMessages$ is complete. so it will not send message to server.

@iamcco In your case it does not work because you need to replace this, (Here I am using .env variables to keep my API_KEY secret)
const dataSourceUrl = (topic = "Google", page = 1) => "https://newsapi.org/v2/everything?" +
    "q="+ topic +"&" +
    "from=2018-10-13&" +
    "sortBy=popularity&" +
    "page="+ page +"&" +
    "apiKey="+ process.env.API_KEY +"";

with your own API_KEY fro this site, https://newsapi.org/. Just request an API_KEY then it should works.

@luillyfe i have request an API_KEY and test it.

but the binary doesn't call:
https://github.com/luillyfe/rxjs-websockets/blob/features/blob-type-support/src/app.js#L70
the news receive from the server is through the chat subscribe and resolve by deserializer and emit by blobMessages$

@iamcco what is the error you are getting now, I mean what output do you have in the console ?

In any case I am about to update the branch, because there is not point at all to use concat and multiplexing however you can "mimic" this behaviour.
const news$ = blobSubject.asObservable().pipe(
    filter(message => message.type === "Blob"),
    map(message => {
        if (message.articles && message.articles.length > 0) {
            return message.articles;
        }
        return { message: "No data!" };
    })
);

Let me know how this works for you. 🤪

@luillyfe it works in this way, thanks.

Actually there is not point at all to create a new Subject. Just create in advance an Observer that will let you use the next method to send messages to the Observable.
const obs = {
    next(articles) {
        if (articles && articles.length >= 3) {
            const carusellActive = [...Array.from(document.querySelector(".carousel-item.active").children)];
            carusellActive.forEach((card, index) => {
                const {urlToImage, author, content, url} = articles[index];
                setCardInfo(card, {urlToImage, author, content, url});
            });
        } else {
            console.log("It looks like there is not more content related to this topic. We switched the topic.");
            setTopic();
        }
    },
    error(error) { console.log(error); },
    complete() { console.log("This is over!"); }
};

I already updated the Branch. https://github.com/luillyfe/rxjs-websockets/tree/features/blob-type-support. Any Improvements are welcome.

Event better, just use fromEvent to listen for changes on reader.
const news$ = fromEvent(reader, "loadend").pipe(
    map(event => {
        const response = new TextDecoder("utf-8").decode(event.target.result);
        return JSON.parse(response).articles;
    })
);

This seems like it was just a debugging session. Closing.


cors error

Hi,
I am using oidc-client version 1.10.1.  After 27th of January 2021 I keep getting this error after building and releasing my on-prem application via azure devops (.net core 3.1 web application with a SPA on top written in REACT):
Access to XMLHttpRequest at 'https://xxxxxxxxxx/adfs/.well-known/openid-configuration' from origin 'https://xxxxxxxxxx:8181' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
We are using ADFS 2016 to auth agains active directory.  Chrome and Egde is reporting the same error.
I have tried about almost everything to resolve this issue. Adding a proxy in frontend,, adding cors in the .net core pipeline at backend, adding allow cross origin with a wildcard directly in meta tag, intercepting the response header etc.   I even did a hard reset on my branche to redeploy last successfull build.  But with no luck.  Same error.
this is my config (which has not been changed in any way):
getConfiguration() {
Oidc.Log.logger = window.console;
Oidc.Log.level = Oidc.Log.DEBUG;
return {
  authority: REACT_APP_ADFS_IDENTITY_PROVIDER_URL,
  client_id: REACT_APP_ADFS_IDENTITY_PROVIDER_CLIENT_ID,
  redirect_uri: `${REACT_APP_HOST_URL}/oidc`,
    post_logout_redirect_uri: `${REACT_APP_HOST_URL}`,
  response_type: 'id_token token',
  loadUserInfo: false,
  scope: REACT_APP_ADFS_IDENTIY_SCOPE,
  revokeAccessTokenOnSignout: true,
  metadata: {
    issuer: REACT_APP_ADFS_IDENTITY_PROVIDER_URL,
    authorization_endpoint: `${REACT_APP_ADFS_IDENTITY_PROVIDER_URL}/oauth2/authorize/`,
    token_endpoint: `${REACT_APP_ADFS_IDENTITY_PROVIDER_URL}/oauth2/token/`,
    userinfo_endpoint: `${REACT_APP_ADFS_IDENTITY_PROVIDER_URL}/userinfo`,
    end_session_endpoint: `${REACT_APP_ADFS_IDENTITY_PROVIDER_URL}/oauth2/logout`,
    jwks_uri: REACT_APP_ADFS_IDENTIY_JKWS_URI
  }
};

}
Need help :) I do not know if this is a oidc-client issue, but the network tab in dev tools for chrome reports this:
openid-configuration	CORS error	xhr	oidc-client.min.js:758
Thanks!

I do not know if this is a oidc-client issue

It's not.

Access to XMLHttpRequest at 'https://xxxxxxxxxx/adfs/.well-known/openid-configuration' from origin 'https://xxxxxxxxxx:8181' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

You need to enable CORS on your IdP server to allow your SPA origin. If it doesn't allow that, then you're out of luck.

no worries.  We had react-scripts version 2.1.8. Upgraded to 4.0.2.  Cors error gone!


Text edit box resizing has weird acceleration.

Expected behavior
I expect it to follow my mouse cursor without much of a noticeable lag
Actual behavior
It lags behind. Like Android notification drawr in a low end device running Android 2.3.4. Github
How to reproduce
Resize the text edit box in either post or comment, pre or post posting.

Here is the rezising on steemit in glorious 60fps

Browser:  Firefox 60.0.1
Operating system: Windows 10

I mentioned it in https://github.com/busyorg/busy/issues/1930
but the main issue was fixed and I'm not sure if there is a task for that part of the code as well.
I'm not sure how much you care about that, to me it's the UI equivalent of nails on board. It behaves as if I'm pulling some heavy curtain, and it weird :)

It indeed seems to be an issue on FF. It's not something that happens because of our implementation, but antd behaves this way on FF. I will fill an issue.