feat(avm): hashing to simulator

Please read contributing guidelines and remove this line.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

master

PR #4457

PR #4526

PR #4527   👈

This stack of pull requests is managed by Graphite.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 295cd556 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,700
180,580
720,100

l1_rollup_calldata_gas
222,844
870,796
3,466,108

l1_rollup_execution_gas
316,035
989,244
3,686,486

l2_block_processing_time_in_ms
1,152 (-1%)
4,371 (-6%)
17,263

note_successful_decrypting_time_in_ms
320 (-1%)
1,011 (-3%)
3,681 (-1%)

note_trial_decrypting_time_in_ms
31.2 (-7%)
46.8 (+11%)
147 (+2%)

l2_block_building_time_in_ms
17,818
70,414
282,848

l2_block_rollup_simulation_time_in_ms
13,094 (-1%)
51,739
208,272

l2_block_public_tx_process_time_in_ms
4,697
18,598
74,322

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,773
26,650 (+1%)

note_history_successful_decrypting_time_in_ms
2,401 (+2%)
4,753 (+2%)

note_history_trial_decrypting_time_in_ms
84.3 (-17%)
204 (-11%)

node_database_size_in_bytes
17,346,640
32,956,496

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
307
44,528
25,825

private-kernel-ordering
194
44,145
16,225

base-rollup
1,405
128,654
933

root-rollup
82.6 (-3%)
4,192
729

private-kernel-inner
437
71,331
25,825

public-kernel-private-input
241
32,287
25,825

public-kernel-non-first-iteration
237
32,329
25,825

merge-rollup
7.87 (-1%)
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
2 leaves
8 leaves
16 leaves
32 leaves
128 leaves
64 leaves
512 leaves
1024 leaves
2048 leaves
8192 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.2
11.2 (+6%)
12.8 (+1%)
17.1 (+1%)
22.8
63.9
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.9
17.5
23.0
31.6
47.0
143
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.593
0.626 (+6%)
0.547 (+2%)
0.529 (+1%)
0.477
0.440
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
81.9 (+10%)
47.6
238 (-1%)
461
900
3,555

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
N/A
N/A
N/A
159
96.0
543
1,055
2,079
8,223

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.506 (+10%)
0.488
0.435
0.431
0.428
0.427

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
107 (+4%)
56.8
350 (-1%)
691 (+1%)
1,373
5,426

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
N/A
N/A
N/A
197
104
691
1,363
2,707
10,771

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.501 (+2%)
0.497
0.474
0.474 (+1%)
0.477 (+1%)
0.471

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
56.5
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
94.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
0.577
N/A
N/A
N/A
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
16,859
40,167

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
555
1,306

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.502 (-4%)
576


CustomArgumentParser interface should provide the CommandSender caller

Please describe your suggestion
CustomArgumentParser should provide the CommandSender so that CustomArgument implementations can use the sender to determine if there is an error.
Example:
Say I define a custom argument implementation where I want to allow all offline players but not the player making the call:
fun offlinePlayerNotSelfArgument(nodeName: String?): Argument =
    CustomArgument(nodeName) { name ->
        Bukkit.getOfflinePlayerIfCached(name)
            ?: Bukkit.getPlayer(name)
            ?: throw CustomArgumentException(MessageBuilder("That player does not exist"))
    }.overrideSuggestions { sender ->
        (Bukkit.getOnlinePlayers().map { it.name }.toSet()
                + Bukkit.getOfflinePlayers().map { it.name }.toSet())
            .filter { it != sender.name }
            .toTypedArray()
    }

The autocomplete list does not include the current player, but you can type in the current player and it completes. Would like to receive the sender in the CustomArgument with the node value so that I can throw an exception if the player selected is the sender.

In experimenting with this, I have confirmed that using the PlayerArgument doesn't work for offline players. I had to create custom arguments to handle offline players, and offline players excluding self.

In experimenting with this, I have confirmed that using the PlayerArgument doesn't work for offline players. I had to create custom arguments to handle offline players, and offline players excluding self.

The PlayerArgument has been rewritten in 6.0.0 (Commit 33421b59e0ea3b49b368960b299930f6ffffb1f2) to support OfflinePlayer objects instead of Player objects

Implemented in 6.0.0. OfflinePlayers can be handled using the new OfflinePlayerArgument and I've added an overloaded method (temporarily, I'll redesign the custom argument at some point wayyyy in the future) which lets you use a command sender.

nice work! I'll try it out once Paper bugs are fixed and I can run CommandAPI again!


Make TIDSID related code work with French games

Fix addresses of FUN_LoadAdrsAdrs and FUN_CompAdrsAdrs for French and other european languages in TIDSID editor
Add support for French in TIDSID randomizer: add a parameter for the rng call address to adapt to other languages than English

I tested both codes on my French Diamond version

I added japanese rev5 to the tidsid editor thanks to your dumps. I didn't add japanese rev6 because it is missing base_address and I am not sure yet how to get it.
For the randomizer I added the other languages as well and used shifts and masks to convert the rng call address. For this one I only got the French value from searching patterns in the memory editor. Is there a better way to get it? Otherwise I'll do the same for the other languages when I have some time.


Can i know the training environment?

I wanna train and get weight file of rfb mobilenet ssd model end-to-end by myself.
Should I need to load pretrained feature extractors weights? Then how can i get that weight file?
Can i know the training environment like epoch, learning rate, scheduler and learning decaying parameters?

Hi @seongkyun,

Should I need to load pretrained feature extractors weights?  Then how can i get that weight file?

You can either train from scratch or with pre-trained weights.

Then how can i get that weight file?

The weights are located on the main page of the repository, need to click on the accuracy (in blue).

Can i know the training environment like epoch, learning rate, scheduler and learning decaying parameters?

You can specify all this parameters in the cfg file.
Cheers!

Thank you, @lupotto .
I have another question.
Then, the training specifications in cfg file is for end-to-end(from scratch) training or with pre-trained backbone network training?
I mean, if i train my net from scratch with only cfg training specifications, then can I get same accuracy on main page results?

Then, the training specifications in cfg file is for end-to-end(from scratch) training or with pre-trained backbone network training?

What I know is that the weights that they provide are end-to-end. If they trained with pre-trained weights or not I don't know, you have to ask them.

I mean, if i train net by myself from scratch with only cfg training specifications, then can I get same accuracy on main page results?

I am not 100% sure because I trained from scratch with my own dataset so I didn't check if the results correspond to the numbers that the repository claim.


Question：how to render a epub file with DRM?

A company want to hire us to build a web reading system.
They'd like to host their books with DRM in their server and let us build a front-end reader to access these books. What are the solutions we have by using ReadiumJS?
Thanks.
I can not find any information about this topics, please help.

I can say that Readium LCP libraries have been developed for mobile and desktop implementations, but we don't intend to implement DRM on a pure web client implementation, as we don't see how such implementation can be secure.

@llemeurfr  Thank you for your reply.
What about server-side processing DRM and front-end only rendering books? Is this approach possible?

Sure it is possible. And then authentication + HTTPS protect content during its move from the server to the client, in a "streaming" mode. The Readium-2 architecture, with its Golang "streamer" module, is made just for that.

@llemeurfr
Great, could you please tell me where I can find information about how to do it. I didn't see any document for this.
Thanks

If you want to look at this path, you'll find info in -> https://github.com/readium/readium-2 (architecture, doc) and https://github.com/readium/r2-streamer-go (golang streamer). You'll have to browse the project to learn how we work. We'd like help to developer a new JS "navigator", i.e. an update of Readium JS.


Outdated Documentation

I'm one of the TAs of Programming Language this semester. I would like to assign a homework for practicing cmake and promote nana GUI library at the same time .
However, I found that the documentation is a bit outdated. for example, the following code cannot been compiled by VS2017.
msgbox(fm, L"Form closing").icon(msgbox::icon_information)<<L"The form is closing now"
I should change L"" string literal to u8"" string literal for it to be compile successfully. I am afraid that my student will encounter some problem

Thank for reporting !
Where did you find this?
Yes, it is old. Just eliminate the L

I did not find a page for msgbox here:
https://github.com/cnjinhao/nana-refman/tree/master/en-us/content/utilities
It will be great if a page in the documentation could include a link to the github file people could edit directly to contribute

Thanks for your feedback! I will create new doc for it.

fixed.


UI Defect: There's a gap between white bar and top of screen on mobile view.

Gap only occurs up to 640 width. I believe this is messing things up:
@media only screen and (max-width: 640px)
.sqs-layout .sqs-row .sqs-block:first-child {
padding-top: 14px !important;
}
Can we make it 0px or try to remove this?

Should be fixed with PR 87

Will this be fixed with the move to gh-pages?


Warn if -days without -x509

A source of confusion on the mailing list, apparently.

Merged, thanks.

I a little bit worry that if we provide a warning on this particular nonsensical combination we are implicitly committing to provide a warning on all such combinations.  The latter seems perhaps harder than we want to sign up for...

I think we should do that as we find out about them.

Regarding this particular nonsensical combination, I've seen it suggested in a handful of guides and howtos on the web, which for a while confused me much as it did the user on the mailing list. I think it's reasonable to warn about this one even if we don't rigorously warn about every one.


multiple nodes

Hello, I would like to ask if you can use multiple machines and cards for training. That is, using multiple nodes, because it looks like you can only support single multi-card

It is currently not possible and I don't plan to add it. However, it requires only minor changes since we use the torch.distributed primitives.


mailcap: 2.1.53 -> 2.1.54

Description of changes
Changelog: https://pagure.io/mailcap/blob/master/f/NEWS
Diff: https://pagure.io/mailcap/c/9699055a1b4dfb90f7594ee2e8dda705fa56d3b8?branch=master
Notably, this updates the mime type of .js files from application/javascript to text/javascript.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/updating-mime-types-in-nginx/36382/2


Add example of series of image usage with visualization

For now, we don't push you to follow any predefined schema of issue, but ensure you've already read our contribution guide: https://open-metric-learning.readthedocs.io/en/latest/from_readme/contributing.html.

cc @AlekseySh


Why is my CPU Performance of Float64 tf.matmul in TensorFlow2 significantly lower than the NumPy matmul, even in the graph mode?

I'm comparing the single thread performance of the matrix-matrix products in TensorFlow 2 and NumPy. I compare separately for single precision (float32) and double precision (float64). I find that the NumPy performance is almost equivalent to the Intel MKL C++ implementation (used as a benchmark for matrix multiplication) for both single and double precision (DGEMM and SGEMM). But in TensorFlow, only the single precision (float32) performance is equivalent to the MKL, and the double precision (float64) performance is significantly slower. Why is Tensorflow slower when used with double precision data?
Sample Scripts:
I consider the following instance to reproduce my observation. Consider the matrix multiplication:

C = AB where A and B are of size 3000x3000

The TensorFlow2 and NumPy code are given below:
Tensorflow2 code
import tensorflow as tf
import os
import time

#Check if MKL is enabled
import tensorflow.python.framework as tff
print("MKL Enabled : ", tff.test_util.IsMklEnabled())

#Set threads
tf.config.threading.set_inter_op_parallelism_threads(1)
tf.config.threading.set_intra_op_parallelism_threads(1)

#Problem size
N = 3000
REPS = 20
DTYPE = tf.float64
#DTYPE = tf.float32

@tf.function
def gemm_implicit_noup(A, B):
    #C = A @ B
    start = tf.timestamp()
    with tf.control_dependencies([start]):
        C = tf.matmul(A,B)
    with tf.control_dependencies([C]):
        end = tf.timestamp()
    tf.print(end-start)
    return C

tf.config.run_functions_eagerly(False)

A = tf.random.normal([N, N], dtype=DTYPE)
B = tf.random.normal([N, N], dtype=DTYPE)

#Building Trace
C = gemm_implicit_noup(A,B)

for i in range(REPS):
   C = gemm_implicit_noup(A,B)

Numpy code
import os
os.environ["OMP_NUM_THREADS"] = "1"
import numpy as np
import time

N = 3000
REPS = 20
DTYPE = np.float64
#DTYPE = np.float32

def gemm_implicit_noup(A, B):
    #C = A @ B
    C = np.matmul(A,B)
    return C

A = np.random.randn(N,N).astype(DTYPE)
B = np.random.randn(N,N).astype(DTYPE)

for i in range(REPS):
   start = time.perf_counter()
   C = gemm_implicit_noup(A,B)
   end = time.perf_counter()
   print(end-start)

System and Installation settings:
The performance was compared on Intel Xeon Skylake 2.1 GHz with CentOS 7 and also on MacBook Pro 2018 with BigSur. The performance was compared on both Tensorflow 2.7 and 2.8, which were built with Intel MKL. Python 3.9.7 and 3.7.4 were checked. I compare the single thread performance so that the results can be reliably reproduced. I observe similar performance numbers in all the settings:
Single precision performance is as expected:

Intel MKL C++ SGEMM ~ 0.5s
NumPy float32 ~ 0.5s
TensorFlow float32 ~ 0.5s

But Double precision performance:

Intel MKL C++ DGEMM ~ 0.9s
NumPy float64 ~ 1s
TensorFlow float64 > 2.5s (Much Slower!!)

Hi @Saduf2019 ! Could you please look at this issue. Attaching gist for reference.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Updating dist/index.js

This amends @josecorella's fix #20 to download Dafny built for x64 on versions < 3.10.0 (the only arch we were building for) and Dafny built for os.arch() otherwise

What is this supposed to do? I can't judge from an empty description.
How have these changed been tested?

Description updated. The matrix in the "Validate action" workflow covers Dafny versions greater than and less than 3.10 on a both x64 and arm64 runners (across all OSs), so the current testing should be sufficient.


Add SPM config support

This PR adds support for Swift Packages in the config. This allows Sourcery to determine the module name of the scanned sources.
It depends on https://github.com/apple/swift-package-manager which is quite a big dependency (binary size also almost doubled after adding it). It is however the easiest way to read the contents of the package. Other method could be use the JSON output from swift package dump-package and parsing that ourselves. The upside to this is that we're not dependent on this big dependency and we're less attached to local Xcode version vs shipped SPM version. Downside is we have to parse stuff ourselves.
I'd love to hear your thoughts on that.

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

  Generated by :no_entry_sign: Danger

This is great, could you add some docs for the new config?

I've added some docs in Usage.md and added a changelog entry.

thanks for contributing 🙇

@krzysztofzablocki  @art-divin I'm sorry to bother you but do you have any plans for a new release in the near future? Having these changes would help me massively.

Thanks for asking, @jimmya 👋
I am all for the new release, @krzysztofzablocki what do you say? How to proceed?

@art-divin there is rake release workflow which you can use to make a new release


Dropdown Popover: Non-text id leads to "is not a valid selector"

Reproduction
https://formkit.link/0b54327f8f8a6497e3578bdc77af924d
Describe the bug
When using the popover option in combination with element ids that do not contain only text (e.g. 0test or 0.test`) for a dropdown, the dropdown is not shown and an error is shown in the console:
Uncaught DOMException: Failed to execute 'querySelector' on 'Document': '#0test_popover[popover]' is not a valid selector.

Environment
• OS: macOS
• Browser: Chrome

As https://stackoverflow.com/a/46399063/3920200 suggests, maybe escaping the id when using the querySelector would already help?

Have a fix for this. Instead of accessing by id directly:
document.querySelector(`#${node.context.id}_popover[popover]`)

We can access by property selector:
document.querySelector(`[id="${node.context.id}_popover"][popover]`)


Normalized counts or raw counts as input?

Hello, I am trying to normalized rna-seq exp data from the ICGC and I am having issues trying to figure out if the expr.tsv input has to be normalized counts or raw counts.
¿What is the most suitable count to use?
Thank you.

Hi @epolanco03
If you are not sure if your data are raw counts or normalized counts it's best to check the source - ICGC. Usually the documentation is quite explicit about the type of data they provide. If no such info is given, consider asking. However, as a rule of thumb: raw counts are integers, while TPM / FPKM / CPM values are floats.
Which data to use totally depends on your use case.


Always HEAD blobs at least once during pushes

Signed-off-by: Jon Johnson<EMAIL_ADDRESS>- What I did
Change push behavior to always HEAD each blob before trying to mount. The docker client used to do this, but that changed in #26564. This causes a lot of unnecessary blob uploads if the blobs already exist in the target repository.
- How I did it
Set checkOtherRepositories to true for the first layerAlreadyExists check.
- How to verify it
Push an image to a registry, there should be a HEAD request for each blob before trying to mount.
- Description for the changelog
Always check to see if blobs exists before pushing.

Hi @jonjohnsonjr - I'm having some trouble understanding how the change prevents unnecessary blob uploads. You're changing the checkOtherRepositories parameter to true, but this first call is intended to check the target repository (as indicated in the comment), so we aren't mounting a different repository if the blob exists in the target repository to begin with. If the blob isn't found in that repository, there's another call to layerAlreadyExists below that checks other repositories. Can you outline a repro case for a scenario where unnecessary blob uploads happen? If there's a problem with unnecessary uploads happening, I suspect a different fix is required.

Hey @aaronlehmann, thanks for the response!
The first call to layerAlreadyExists will only check the target repository if we have metadata where the SourceRepository is the same as the target repository (I believe this means we pulled from the target repository or have pushed to it already, but I'm not super familiar with the v2MetadataService code). I think checkOtherRepositories is misleading here, because "other" seems to refer to other source repositories in the metadata, not other target repositories.
The subsequent call to layerAlreadyExists will only check the target repository for blobs larger than 10MB (based on getMaxMountAndExistenceCheckAttempts).
So, if you pull an image from one registry and push to another, the SourceRepository won't match the target repository (so the first layerAlreadyExists will not HEAD), the blobs won't be mounted because cross-registry mounting isn't supported, and existence checks will only be performed for large blobs. If any  blobs under 10MB are already in the target repository (e.g. if they were pushed by another client, so we have no corresponding metadata) they will be unnecessarily re-uploaded.

Thanks for the explanation. This makes sense to me now. If I understand correctly, the old code would only do an initial HEAD request on the target repository if it found metadata directly associated with that repository (i.e. if we had pushed the blob to that repository or pulled it from that repository before). Otherwise, it would skip this and move on to trying cross-repository mounts. If all those attempts failed, it would try HEAD requests based on digests associated with other repositories, but only for large blobs.
The change in that PR allows that initial HEAD request to proceed with a digest found in another repository's metadata. If there was a digest associated with the target repository, there should be no change, because sortV2MetadataByLikenessAndAge should favor that one anyway.
This seems sane to me, but I'm surprised there's a benefit. I would expect that if the blob was known to be in another repository, the cross-repository mount attempt would generally succeed. Are you using a registry that doesn't support cross-repository mounts, by any chance? Or is authentication done in such a way that it's often impossible to access repositories that were previously used for pushes and pulls?
In any case, these heuristics were invented without any serious data collection, and we're open to tweaking them based on real-world experience.
@dmcgowan @miminar: Can you please give this a sanity check?

If I understand correctly...

Yep, exactly.

This seems sane to me, but I'm surprised there's a benefit. I would expect that if the blob was known to be in another repository, the cross-repository mount attempt would generally succeed.

It is currently impossible to do a cross-repository mount between registries. For example, if I pull from DockerHub and push to a private registry, the client will not even attempt to mount because the domains differ.
From getRepositoryMountCandidates:
if err != nil || reference.Domain(repoInfo) != reference.Domain(sourceRepo) {
	continue
}

FWIW, there was some discussion around cross-registry support (see here) but the current implementation does not allow for it.

LGTM. The change will benefit at least us and our customers. I don't have the numbers though.

LGTM
ping @dmcgowan

LGTM


Strange API behavior on add server via UI

requires an interface, although it is specified. Observed in the master version
{
    "data": {
        "alerts": [
            {
                "text": "a server must have at least one interface",
                "level": "error"
            }
        ]
    },
    "status": 400,
    "config": {
        "method": "POST",
        "transformRequest": [
            null
        ],
        "transformResponse": [
            null
        ],
        "jsonpCallbackParam": "callback",
        "url": "/api/3.0/servers",
        "data": {
            "updPending": false,
            "tcpPort": 80,
            "httpsPort": 443,
            "ipIsService": true,
            "ip6IsService": false,
            "statusId": 1,
            "hostName": "AMS-2120463",
            "domainName": "am1.myawesomehost.com",
            "cdnId": 2,
            "profileId": 10,
            "cachegroupId": 1,
            "typeId": 12,
            "interfaceName": "eth0",
            "ipAddress": "<IP_ADDRESS>",
            "ipNetmask": "<IP_ADDRESS>",
            "ipGateway": "<IP_ADDRESS>",
            "interfaceMtu": 1500,
            "physLocationId": 1
        },
        "headers": {
            "Accept": "application/json, text/plain, */*",
            "Content-Type": "application/json;charset=utf-8"
        },
        "cached": false,
        "withCredentials": true
    },
    "statusText": "Bad Request",
    "xhrStatus": "complete"
}

downgrade to 4.1 fix issue

This is fixed by #4747


Fix log gradient test

I guess there was still a (small) probability randn + 1 could still be up against 0.

Nice catch, thanks for the fix

Hmm maybe we should just ensure it's not zero? Perhaps .abs().add(1)

That would probably be a better way of solving it honestly. Though it appears there is some other build error now :(


Can't disable rotation in ViewController

I want to lock the application in portrait mode, but I can't do it for the view controller that implements that slacktextviewcontroller. Everyone in my application is locked into portrait, but nothing I do disables it in this view controller. What other information should I provide?

For some reason, now that worked. I don't know why it wasn't working before. Thank you!!


fix: [web] Don't apply SvgTouchableMixin on every component

Hi 👋
Do you remember this bug? Well, it happens again, for a different reason. 😄
By default, every component of this lib implements the SvgTouchableMixin, even if no Touchable properties are used, making every component accessible by default again.
Summary

hasTouchableProperty is now a function, SvgTouchableMixin is applied only if it returns true.

Test Plan
What's required for testing (prerequisites)?

Init a project with create-react-app and react-native-web
Add react-native-svg dependency and render a random SVG using it

What are the steps to reproduce (after prerequisites)?

SVG elements should not be accessible and focusable if Touchable props are not present.
SVG elements should be accessible and focusable if Touchable props are present.

Checklist

[X] I have tested this on a browser
[ ] I added documentation in README.md
[ ] I updated the typed files (typescript)
[ ] I added a test for the API in the __tests__ folder

Oh interesting, I suspect this won't handle cases where at first there's no touchable property, but in a later re-render there is. Wonder what makes it accessible to begin with, have you dug deeper into this?

@msand Yes! By applying TouchableMixin this exact function will return true, making every component of this lib keyboard focusable.

Hmm, I can't seem to figure out why that return value would depend on using TouchableMixin, can't find anything inside it that should effect it. Guess I'd have to run the actual code to figure it out.

@msand Instead of using a TouchableMixin, we might conditionally wrap the rendered element with a TouchableWithoutFeedback, no?

I think that probably wont work well together with gesture responders. Not sure though.

This seems to work:
const isWeb = Platform.OS === 'web';
const keys = Object.keys(SvgTouchableMixin);
const touchKeys = isWeb ? keys.filter(key => key !== 'componentDidMount') : keys;

Seems related to this._isTouchableKeyboardActive in Touchable Mixin

Actually, deciding factor seems to be:
this._touchableNode.addEventListener('blur', this._touchableBlurListener);

Not sure how the presence / absence of a blur handler decides if a svg element can become the active element or not.

I see
https://svgwg.org/svg2-draft/interact.html#Focus

User agents may treat other elements as focusable, particularly if keyboard interaction is the only or primary means of user input. In particular, user agents may support using keyboard focus to reveal ‘title’ element text as tooltips, and may allow focus to reach elements which have been assigned listeners for mouse, pointer, or focus events. Authors should not rely on this behavior; all interactive elements should directly support keyboard interaction.

@necolas Is this behaviour intentional? Is the blur event handler partially there to make svg some elements focusable? What are the cases in which they should they be focusable and not?
I'd assume if there's no focus/press/gesture related handlers, they shouldn't be focusable, while if that changes, it should adapt correctly.
Seems like disabling the blur event listener / skipping componentDidMount is not a good solution.
Could perhaps attach/detach as a side-effect based on the presence of event handlers. Wdyt?

@necolas Sorry to ping you twice like this. Feel free to instruct me not to ping you and I won't cause any further interruptions. Was suspecting it went past your radar / forgot about this. Would you mind having a quick look and giving a bit of guidance with regards to the touchable api in react-native-web? I'm considering I should write a react-native-svg specific handler, that de/attaches the blur event listener on demand. Do you see any problems with this approach?

@zoontek What do you think of this implementation?

@msand This should work, I can try it tomorrow to be sure if you want. But this (kind of) seems fragile. Maybe react-interactions will help at some point? For now the implementation target React DOM, but I think it's their solution to get rid of Touchable Mixins at some point.

Yeah, this certainly isn't very clean, but the closest api to the react-native implementation and most react-native-web compatible I can think of at the moment. I'd love to see a cleaner implementation of this and certainly willing to merge it.

I think the mixin should be run on the prototype once rather than on each instance object as well. Only the event listeners need to be attached, and the state initialized, per instance.

I don't have enough context to understand what's going on here. You're trying to mixin Touchable on an SVG element? Why is that part of an svg library?

It's used in the react-native implementation to get support for touch/gesture events, so quite a few components are implemented based on that api of react-native-svg, and thus a compatibility layer for the web components has been requested.

@necolas Perhaps I should elaborate, the Touchable mixin has been part of the (Shape) api since v4.1.0 or 9 Jun 2016
https://github.com/react-native-community/react-native-svg/commit/73077157124c84cfa87c00055b96cfb697288d9d#diff-8a2f6ca181e875c2bffa1bebff2800b2
Many of the use cases of this library aren't static content, but interactive visualisations and complex ui elements with various touchable filled and/or stroked areas. Many of them are simple press handlers, others have turing complete dependence on the history of gesture events / other inputs.
In the case of this pr, as we've made the api work the same way on the web as it already does in native (i.e. support touch and gesture handler directly on the components), causes the dom elements to be focusable, as the componentDidMount handler from Touchable in react-native-web attaches a blur event handler to the dom elements, causing them to be considered accessible elements, and thus when you tab thru the page, the elements receive focus, even if they don't have any press / gesture event handlers actively attached to the react-native-svg elements.
This pr currently detaches the blur event handler after componentDidMount, in case there are no touchable props given, and reattaches if at a later point there is. Ideally we would have had two sets of elements, one with touchable support, and one without, but this api was grandfathered in before I made my first contributions.
@zoontek Did you have time to test this?

@msand Yes, I use the last version on two projects and it works correctly now.
Otherwise if I understand the necessity of interactive SVG, why can't we just wrap elements in TouchableWithoutFeedback? (Ex: to make a Path pressable). It's added complexity and possible side effects for not so much.

@zoontek You mean the last released version? last commit from develop branch? or from this branch/pr?
That would be ideal, but I'm not sure I could sustain the amount of opened issues that kind of breaking change to the api would cause. I would likely stop supporting this library, given the history of types of issues opened over the past few years. So, perhaps in case you would commit to handling all issues of that type I could consider it.

My bad, I was thinking about the other issue I had (height & width at 100% by default). I forgot to try this one.
I installed it on my project (using this branch, up-to-date) and all the elements are still focusables.

Hey, is there any plan to merge this? I'm encountering the same problem (all SVG elements being tabbable) that this is discussing.

Hi! Is there any progress on this?
For now, react-native-svg is not usable on an RNW project with accessibility considerations.
I'm also using react-native-svg on a RN project and wasn't aware that elements can be touchable.
I'm surely miss a point. What is preventing to use the library like this:
<Svg>
  <Touchable>
    <Rect />
  </Touchable>
</Svg>

instead of having onPress/* directly on svg elements ?

IMHO, the cleanest way to do this should be dropping RNW 0.13 support and relying on usePressEvents. Exactly like the TouchableWithoutFeedback implementation.
I can easily make the change. Even if I also think that the best change should be to drop support for press events il the whole codebase (like @Freddy03h said), and rely to Touchable* and Pressable instead.

Huge 🆙
I rewrote the web implementation completely without the TouchableMixin. Instead, I apply TouchableWithoutFeedback conditionally. Accessibility is fixed, performances are superior (even if the way the style is resolved might be improved), codebase is a bit more up to date.
I also bumped some type definitions / some dependencies versions in order to fix the build (Bob build / type generation failed with the version set), added an /example project to help future contributors.
I also switched to yarn. Sorry about this one, but npm delete the example project when your run the reinstall script with it - for some reason). As I found a bit silly to have a yarn.lock in example and a package-lock.json at root. I can revert it.

I have this problem too, is it possible to merge these commits into main branch?
Thx

I think it's time to merge this. The problem is really serious

already test the implementation and so far so good ty @zoontek

Since https://github.com/react-native-svg/react-native-svg/pull/1585 has been merged, and I think this PR takes quite another approach to the problem, could you update this PR to only concern the ReactNativeSVG.web.ts file if it is possible? As for the other changes, the example app is already in the repo. As a side note, mixing multiple things in one PR makes it really hard to review and test easily 😅  Or maybe https://github.com/react-native-svg/react-native-svg/pull/1585 is enough for most use cases and we can close this? I believe the problem occurs when re-rendering the component with touchable props added. Am I right? cc @zoontek

I will rebase against master, remove the exemple project.

I added a working web example with RNW + kept my implementation (using TouchableWithoutFeedback) because the current implementation did not worked when I tried it.
There's still a small issue with ref forwarding, but it could be fixed in another PR.
I also deleted the PanReponder example (it just does not work, either on web or mobile)

My recommandation: Cleanup this repository to the max, update dependencies, and schedule some breaking changes.
Delete all press handling props / methods (for the web and native).
It just add a lot of complexity for nothing. Developers can wrap Svg elements with Pressable, View + PressResponder, Animated.View… it's fine!

@WoLewicki I had several errors in the console. I can create a new PR to fix the example app (and maybe delete TestsExample? I think this directory is useless)

TestsExample will be used for providing tests examples for new PRs, which can be then tested to check if there are no regressions e.g. before a release. Example app is to be treated as a showcase of how to do things with the library, you can see the same structure in react-native-screens. PR with the changes you mentioned would be very welcome for sure :tada:

@WoLewicki The PR is here: https://github.com/react-native-svg/react-native-svg/pull/1717
Like I said here, the onPress* are currently broken on the web, the latest fix removed the focus on all SVG elements, but also broke pressability. And the PanResponder example doesn't work.
But IMHO it's a good start to implement breaking changes. I can help creating a roadmap for v13.

I added a review of #1717 and it seems to work just fine, thanks for that 😄  Are you sure #1585 broke the pressability? I commented the if checking for pressable options and on web the elements still do not react to being pressed, but I may be doing something wrong.

@WoLewicki Indeed, it might was super broken with focus on all elements and pressability broken too, I didn't checked 😄

Could you merge the current main to resolve conflicts?

@WoLewicki Done. However, I recommend using that as a temporary solution and remove press handling in next version.
It sures fixes the onPress related events, but:

svg are not focusables
some press responders errors are printed in the console when onPress* is set. It's working tho (but it should be removed, it's easier to wrap the whole Svg with a Pressable)
PanResponder does not work when set directly on the Svg (but who does that? just wrap them with a View, they accept GestureHandlers)

Sorry for the late response. So do you think this PR provides better solution than the current implementation? About wrapping the whole Svg with a Pressable, do you think it won't add new problems, as the clickable area?

It's better than the current implementation for sure. But not perfect.
Wrapping SVG with Pressable / Animated.View / View with PanResponder will allow deletion of a lot of code / the suppression of a lot of issues 🙂

I think quite a few people depend on the press related functionality to only respond when interacting with filled / stroked parts of them canvas. I'm quite sure that wrapping with Pressable / PanResponder won't respect that part of the api. Or?

@msand IMHO, if a part of the SVG should be animated / move using PanResponder but not the rest, that's not an SVG, but two.
And even with that, components are implemented using basic UIView / View, it shouldn't be an issue.

Well, in practise, if the pressability would change to be based on bounding boxes rather than fill / stroke, a bunch of use cases would no longer be possible or would be buggy. The ability to have events based on this comes from the svg spec, so I'd say it is in scope of what should be supported by react-native-svg for single svg components.

This is a blocker for our A11y audits. I've moved our implementation to module alias react-native-svg to react-native-svg-web in the meantime.


Support lists as font-family attribute

The font-family attribute in the SVG standard is a prioritized list of font family names and/or generic family names. When exporting svgs with referenced text, sometimes fallback fonts are added, which breaks Jovial.
E.g. given that the font NexaRegularItalic and the font Nexa are listet as fonts in pubspec.yaml, font-family="NexaRegularItalic, Nexa" in an SVG will not use Nexa but the default font instead.

... will not use Nexa but the default font instead.

Not surprising, but it might not be too hard to add more extensive support for text fonts in this way.
Please provide an asset that can be used to reproduce the issue.  I just make it a policy to always ask the submitter for that; it removes a source of ambiguity, which can be important with a gigantic hodgepodge like the W3C SVG specifications.

Sure, here it is!

@zathras That was fast, thank you:)

Released in 1.1.21


Better Arg Typechecking / Validation

A common error is for a --model_arg value that is meant to be a string to be cast/passed as an int, or for an arg that should be an int or float to stay a string.
This requires manual casting + handling of various model kwargs for edge cases. It would be ideal if we could more extensibly handle all these cases so that a user doesn't have to worry about model kwargs being the expected types. Not hugely high priority but would be quite nice to have.
#2167 is one instance of this.

how do you feel about pydantic? obviously much less flexible, but should be a way to partially validate some fields

I've not worked with Pydantic before, embarassingly. (I know Inspect AI uses it pretty heavily if I'm recalling correctly?) Will check it out and follow up


Fix the problem in printing feature in c++ API examples : feature_extract

Fixed a run-time error caused by mismatch of output dimensions.
Description
The dimensions output with origin code and model is (2, 1024, 1, 1), which may lead to the following errors:
terminate called after throwing an instance of 'dmlc::Error'
  what():  [10:34:56] /mnt/feature_extract/3rdparty/mxnet-cpp-package/include/mxnet-cpp/ndarray.hpp:379: Check failed: shape.size() == 2 (4 vs. 2) : The NDArray needs to be 2 dimensional.

so we should reshape the output array to (2, 1024)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Feature1, tests, (and when applicable, API doc)
[x] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Thanks for your contributions @somone23412 .
@leleamol Can you review this PR as well?
@mxnet-label-bot add [pr-awaiting-review, C++]

@leleamol Gentle Ping ..

Merged. Thanks for your contribution!


read me for this project

Hi, thank you for building this project
It seems the readme is for JoeyNMT instead of your actual sigmorphon project. May I know how to get hands on your model (inference, training. etc)? Thank you

Hello, thank you for your interest and sorry for the incredibly long time in replying. I'll update the readme tomorrow, after running a few tests to make sure things really work the way I think they do. The gist of using it is that everything is specified in config files, of which I've included several (I hope still) working examples in configs/. Training a multilingual g2p model is accomplished by running
python -m joeynmt train configs/task1/multi/rnn/multi-128-512-2-2-langall.yaml
The only thing you should need to change about this config is the path to the data -- you can replace those lines with your own. If memory serves, you can manually specify all of the paths of the files, or use glob syntax (as in the provided example). I think the code will parse the language code from the name of the file, so it's important not to rename any of the files.
At test time, you can run
python -m joeynmt test configs/task1/multi/rnn/multi-128-512-2-2-langall.yaml


Config map not applying following Update and pod restart

Hello.
We are using the MQ v<IP_ADDRESS> image with OpenShift deployment.  We have seen behavior, of MQ in a pod following a configmap update and pod restart.  The pod restarts and goes to a "Ready" status, but , in the OpenShift MQ log, the QMgr reaches the point where the channel initiator starts, but the configmap is not applied.  We think that connecting applications are causing this.  Is there a way around having to stop the connecting application (WAS liberty) before restarting the pod?

I assume this was resolved offline.  Please feel free to re-open if not.


Suggestion: un-pin the highly-specific requirements

The tightly-pinned requirements file make this not play nice in projects. In my case, pydantic is the problem, because other libraries are using other versions.
Is there a specific reason pydantic is pinned to 1.10.17 rather than <2? And with click, tabulate, kubernetes, and opentelemetry pinned to very specific versions any project with a minute difference has a conflict.
For Pydantic, version 2.x was released a full year ago, so any library that has moved to 2.x is not compatible with gretel-python-client. Pydantic 2 ships with a copy of v1, just needs an import change to use. So you could switch the pin to pydantic<3, update some imports, and it would work and be more compatible.
I think this library will be better served by ranges. Otherwise it has to be partitioned off from all other work and, ultimately, will be used less. It's just not installable in larger projects.

I pushed https://github.com/gretelai/gretel-python-client/commit/3158848a7282a1958a72fd387c68677ac25eecf7 through, I can leave this issue open until the pydantic upgrade lands though, up to you

We merged a change to bump to pydantic v2 a bit ago, so I'm going to close this


[BUG] Jobs using length, delay, and period at the job level and metric level results in inconsistent start time and end time for GetMetricData queries

Is there an existing issue for this?

[X] I have searched the existing issues

YACE version
No response
Config file
No response
Current Behavior
The way YACE handles length, period, and rounding period when a job has metrics with different configured values is really weird.
Length, period, and rounding period all factor in to calculating the start + endtime for the GetMetricData API call. Each GetMetricData API call has 500 MetricDataQuery's. Each MetricDataQuery represent a single metric returned from ListMetrics which has a configured length, and period. This creates a mismatch in parameters the start + endtime needs to depend on values from the 500 different metrics included in the query. Instead of partitioning the requests around the potential difference in start + end time from differing length + period combinations YACE tries to make a best guess at what is a good enough value for what it is doing,

Calculate rounding period based on the smallest period in a BATCH if the job does not have rounding period configured

This can easily result it different batches having different rounding periods
The end result will be varying start + endtimes for different batches depending on which metrics were included in the batch

The length is set to the largest configured length of all metric configs in the job

A longer length for a job which does not require it results in more individual data points being returned
Extra individual data points are thrown away resulting in wasted IO time and unclear impact on API response times (querying for more data likely takes longer)

Delay is only respected at the job level even though it's configurable at the metric level

Expected Behavior
GetMetricData requests should contain queries for metrics which all have the same start and end time determined by the configured metric value for rounding period, period, length, and delay. These should be done in batches of 500.
Steps To Reproduce
No response
Anything else?
No response

Just adding a note I realized,

Delay is only respected at the job level even though it's configurable at the metric level

Is especially bad because it will have very unintended consequences when fixed. Metric validation adds a default 5 minute delay to all metrics if they do not have a defined delay and the job does not have a defined delay. This default delay has largely been ignored because all example configs use no delay at the job level. As soon as this is fixed that delay will take effect potentially breaking the expected behavior for everyone. I'm still trying to think through how to approach this in a safe manner but felt it was necessary to write down.

Hi, so I need help to understand what delay is being depreciated means in https://github.com/nerdswords/yet-another-cloudwatch-exporter/blob/master/CHANGELOG.md#0590
At present I can only consistently get metrics if I have a configuration along the lines of the below. Otherwise I more often than not get NaN values due to the metric not being present in CloudWatch at scrape time.
  - type: AWS/NATGateway
    regions:
      - <region-1>
      - <region-2>
      - .....
    roles:
      - roleArn: arn:aws:iam::<accountid-1>:role/monitoring-role
      - roleArn: arn:aws:iam::<accountid-2>:role/monitoring-role
      - roleArn: arn:aws:iam::<...........>:role/monitoring-role
    period: 60
    length: 60
    delay: 120
    addCloudwatchTimestamp: true
    metrics:
      - name: ActiveConnectionCount
        statistics: [Maximum, Sum]
      - name: BytesInFromDestination
        statistics: [Maximum, Sum]
      - name: BytesInFromSource
        statistics: [Maximum, Sum]
      - name: BytesOutToDestination
        statistics: [Maximum, Sum]
      - name: BytesOutToSource
        statistics: [Maximum, Sum]
      - name: ConnectionAttemptCount
        statistics: [Maximum, Sum]
      - name: ConnectionEstablishedCount
        statistics: [Maximum, Sum]
      - name: ErrorPortAllocation
        statistics: [Maximum, Sum]
      - name: IdleTimeoutCount
        statistics: [Maximum, Sum]
      - name: PacketsDropCount
        statistics: [Maximum, Sum]
      - name: PacketsInFromDestination
        statistics: [Maximum, Sum]
      - name: PacketsInFromSource
        statistics: [Maximum, Sum]
      - name: PacketsOutToDestination
        statistics: [Maximum, Sum]
      - name: PacketsOutToSource
        statistics: [Maximum, Sum]

Are you able to help with my understanding ? I understand my period and length are short, but if by default there is a 5 minute delay, why would i need to set the delay option
Any help is appreciated

Hey @rhys-evans, your usage of delay won't be impacted as you're setting it at the job level. The deprecation is at the metrics level where it's valid to set the value but it was never being used.

Ah , thank you


Name

How about calling the package (and repo) dfspy? Then the scope is more apparent - DFS file IO.
There is only one existing repo with that name and that is impossible to mix up with ours is and, most importantly, it is not on PyPI.

I think it's a good idea to remove DHI from the name

How about mikeio ?
Dfs is a bit overloaded term (e.g. Distributed File System)

how about mikecore ... that's what the corresponding C# NuGet package is called

@FranzThomsen1089 good call with using same names for same things! However, mikecore sounds to me like, well, the core of MIKE (models), which this is not. In fact, just a help for pre/post-processing of files. But name preferences are obviously very subjective...
@ecomodeller mikeio is also more like a component of a model (the I/O part). But this choice of name could be justified by the fact that this library, in a way, interfaces with exactly that component.
Since you put so much work into the code, @ecomodeller, I think you should just decide.

I agree with you @j08lue, I don't think mikecore is the best term, since there is already something else called that (and this is not an official MIKE package). mikeio is cute and nice because you can pronounce it but still I would prefer something which is contains dfs (eventhough the term is a bit overloaded) since that is exactly what is does: handle input/output to dfs files. For me MIKE is more the models. I would prefer pydfs, dfspy or similar, but also think that @ecomodeller and @marcridler  should make the decision.
Greetings from Melbourne :-)

renamed to mikeio


running the demo pipeline

Hi there,
I wanted to run the demo, to check if everything is running well on our servers.
I got to the last point, however I could not find the sample meta file:
--m-sample-metadata-file path-to-sample-metadata.tsv
is it possible to upload it for download.
Thanks in advance.
Mustafa

I would highly appreciate if someone could forward me the sample meta data, since I would love to perform qemistree.

@MuBulut, that file doesn't exist. I understand the tutorial is a bit misleading. The specific command where this file is mentioned is the q2-empress command. If you want to  get more information about empress I suggest reading the tutorial here:
https://github.com/biocore/empress
The "sample-metadata-file" files follow QIIME2's metadata guidelines:
https://docs.qiime2.org/2022.2/tutorials/metadata/
Here's an example:
https://data.qiime2.org/2022.2/tutorials/moving-pictures/sample_metadata.tsv
Hope this helps, if not feel free to reopen the issue. Thanks so much for your patience.


Modernize plugin: Enable incremental builds to support parallel development of components

What feature do you want to see added?
follow instruction
https://www.jenkins.io/doc/developer/tutorial-improve/enable-incrementals/
Upstream changes
No response

done in 3663aba: Incrementalified.
therefore closed


[move prover] add sequential option to prover task runner

Motivation
This PR adds an option --sequential to the Move Prover so that when it is turned on, the Boogie instances will run sequentially instead of simultaneously, meaning that the second Boogie instance will only get to run if the first instance failed. This is implemented using a semaphore.
In addition, this PR also sets the number of Boogie instances to be 2 and turns --sequential on for testing. For command line, the default number of instances is 1 and sequential off. Verification is turned on for a few move functions which previously timed out to see how they behave with the new feature on in MIRAI.
Have you read the Contributing Guidelines on pull requests?
Yes

@bors-libra land


Drovio release v3.4.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi,
Seems like the Installer Validation is taking a huge time to be performed. Is this normal?


Add golang specific props into proto file

Merged into https://github.com/th2-net/th2-grpc-common/pull/56


"TypeError: Cannot interpret feed_dict key as Tensor: The name 'tf_new_X:0' refers to a Tensor which does not exist. The operation, 'tf_new_X', does not exist in the graph."This error I get while I was deploying my model on the server. Can anyone please help ?

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

Nagging Assignee @tatianashp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Closing due to lack of activity. Please reopen and provide additional information if this is still an issue and you think it is a bug. Please direct questions to StackOverflow.


fix: remove unused modules which lead to invalid imports

This should also fix https://github.com/adfinis-sygroup/ember-validated-form/issues/147 as embroider strictly checks imports which failed at this point.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for Modern Fortran extension

Fixes #IssueNumber
Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Adds support for the Modern Fortran extension by recognizing its added languages.

Can you please provide links to the extensions (in marketplace) that are using those language ids?

https://marketplace.visualstudio.com/items?itemName=krvajalm.linter-gfortran


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/23 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-source-version-target.git

Docs Build status updates of commit 432b013:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


feat(SFINT-5119): GEN AI source citations component created for quantic

SFINT-5119
In this PR:
Added source citations that were used to create the generated answer returned by the Coveo Gen AI models
These:

Display the citations which will be received from the Gen AI answer component
Are covered by unit tests
Act as links to the sources citations and open in a new tab
Call the function citationClickHandler which will be received from the Gen AI answer component

When many citations wrapping:

DEMO:
https://github.com/coveo/ui-kit/assets/73316533/87cd6137-2216-48df-961a-8bd8fc247ad9

added a screenshot with many citations

added a screenshot with many citations

Thanks!

You may wanna confirm with UX, but I think the `learn more` text needs to be always aligned with the first line instead of being vertically centered.

Yeah ok, I checked atomic, that is indeed how it behaves, will remove the vertically centered and align to top instead thx!

You may wanna confirm with UX, but I think the `learn more` text needs to be always aligned with the first line instead of being vertically centered.

Yeah ok, I checked atomic, that is indeed how it behaves, will remove the vertically centered and align to top instead thx!

Fixed it, it now reflects the same as atomic:

I think there is a bug when only one citation is given:

Fixed as of latest commit it looks like:

I think we have a styling issue when a link of a source citation is focused:

hold up what?! 😂 I dont remember it doing that.... Let me take a look

I think we have a styling issue when a link of a source citation is focused:

hold up what?! 😂 I dont remember it doing that.... Let me take a look

here, found the culprit, fixed it:
https://github.com/coveo/ui-kit/assets/73316533/5f7ef958-b220-4f27-8265-b4f53c6d4544


Revert "Merge pull request #9532 from IntelLabs/issue9529"

This reverts commit 069ad4324013050a3eb8ca3ec015081c463bd4df, reversing changes made to 0a9a2cceaf9691e2266c1ee9a0d3e461586236c5.

using https://github.com/numba/numba/pull/9617 instead


Is Grch38 with chr prefix supported

Thx for a great software!
I am processing a vcf with peddy version 0.4.3 with the following contigs for grch38:
##contig=<ID=chr1,length=248956422>
##contig=<ID=chr2,length=242193529>
##contig=<ID=chr3,length=198295559>
##contig=<ID=chr4,length=190214555>
##contig=<ID=chr5,length=181538259>
##contig=<ID=chr6,length=170805979>
##contig=<ID=chr7,length=159345973>
##contig=<ID=chr8,length=145138636>
##contig=<ID=chr9,length=138394717>
##contig=<ID=chr10,length=133797422>
##contig=<ID=chr11,length=135086622>
##contig=<ID=chr12,length=133275309>
##contig=<ID=chr13,length=114364328>
##contig=<ID=chr14,length=107043718>
##contig=<ID=chr15,length=101991189>
##contig=<ID=chr16,length=90338345>
##contig=<ID=chr17,length=83257441>
##contig=<ID=chr18,length=80373285>
##contig=<ID=chr19,length=58617616>
##contig=<ID=chr20,length=64444167>
##contig=<ID=chr21,length=46709983>
##contig=<ID=chr22,length=50818468>
##contig=<ID=chrX,length=156040895>
##contig=<ID=chrY,length=57227415>
##contig=<ID=chrM,length=16569>

Peddy has the option --sites hg38, but that means that peddy will use the git repo shipped file with no chromosome prefix.
Will peddy work with the contigs in my vcf which has the chromosome prefix together with this hg38 file? It seems to, but it is a bit unclear in the documentation and with this issue #15
The sex-check, also gives this output:
INFO sex_check
no intervals found for livingox.vcf.gz at X:2781480

but also seems to predict the sex anyway.
Thx!

yes, as you've found, it should work for hg38 with or without the 'chr' prefix. the warning just means that no samples had coverage at that particular position. this can be ignored.


Closes #95

I have a question concerning the digest email: AFAIU interleaving always happens daily. What about users who chose only weekly digest emails? What content will be shown to them?

At the moment they will get the exact same content as those who get daily mail, except that the daily mail contains top 3 each day for the current day. While weekly contains top 3 each day for the entire week by date, which becomes 15 articles like this.
Monday

1
2
3
Tuesday
1
2
3
...

I see. I added a sentence on this.

@oyvi96 Can you please look at this so that I can roll out the changes?


Feature/mouse support

Added support for mouse:
I kept this
with detailed left, right and middle button but we could have done like this.

These changes need to be rebased on master. They currently seem to include the modifier key stuff that has already been merged to master.
When it has been rebased, I'd like to take a look at sending separate movement events vs click events. I have a feeling they should be combined to support drag 'n drop. Right now it seems a pointer move event indicates a mouse button was released, which may not be the case.

#151

device_kind must be set to kFlutterPointerDeviceKindMouse, otherwise the button will seen by the flutter framework will always be the primary one.
Doc

LGTM
BUT
It doesn't work, the PointerDownEvent is always equals to kPrimaryMouseButton
$ flutter --version
Flutter 1.6.3 • channel beta • http://github.com/flutter/flutter
Framework • revision bc7bc94083 (3 weeks ago) • 2019-05-23 10:29:07 -0700
Engine • revision 8dc3a4cde2
Tools • Dart 2.3.2 (build 2.3.2-dev.0.0 e3edfd36b2)

 Dart testing code
// This example makes a [Container] react to being entered by a mouse
// pointer, showing a count of the number of entries and exits.

import 'package:flutter/material.dart';

import 'package:flutter/gestures.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Code Sample for widgets.Listener',
      theme: ThemeData(
        // If the host is missing some fonts, it can cause the
        // text to not be rendered or worse the app might crash.
        fontFamily: 'Roboto',
        primarySwatch: Colors.blue,
      ),
      home: MyStatefulWidget(),
    );
  }
}

class MyStatefulWidget extends StatefulWidget {
  MyStatefulWidget({Key key}) : super(key: key);

  @override
  _MyStatefulWidgetState createState() => _MyStatefulWidgetState();
}

class _MyStatefulWidgetState extends State<MyStatefulWidget> {
  int _enterCounter = 0;
  int _exitCounter = 0;
  double x = 0.0;
  double y = 0.0;

  void _incrementCounter(PointerEnterEvent details) {
    setState(() {
      _enterCounter++;
    });
  }

  void _decrementCounter(PointerExitEvent details) {
    setState(() {
      _exitCounter++;
    });
  }

  void _updateLocation(PointerHoverEvent details) {
    setState(() {
      x = details.position.dx;
      y = details.position.dy;
    });
  }

  void _pointerDown(PointerDownEvent details) {
    print(details);
    if (details.buttons == kPrimaryMouseButton) {
      print("Primary");
    }
    if (details.buttons == kSecondaryMouseButton) {
      print("Secondary");
    }
    if (details.buttons == kMiddleMouseButton) {
      print("MiddleMouse");
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Hover Example'),
      ),
      body: Center(
        child: ConstrainedBox(
          constraints: BoxConstraints.tight(Size(500.0, 600.0)),
          child: Listener(
            onPointerDown: _pointerDown,
            onPointerEnter: _incrementCounter,
            onPointerHover: _updateLocation,
            onPointerExit: _decrementCounter,
            child: Container(
              color: Colors.lightBlueAccent,
              child: Column(
                mainAxisAlignment: MainAxisAlignment.center,
                children: <Widget>[
                  Text('You have pointed at this box this many times:'),
                  Text(
                    '$_enterCounter Entries\n$_exitCounter Exits',
                    style: Theme.of(context).textTheme.display1,
                  ),
                  Text(
                    'The cursor is here: (${x.toStringAsFixed(2)}, ${y.toStringAsFixed(2)})',
                  ),
                  TextField(
                    decoration: InputDecoration(hintText: 'TextField 1'),
                    onSubmitted: (value) {
                      setState(() {
                        print(value);
                      });
                    },
                  ),
                  TextField(
                    decoration: InputDecoration(hintText: 'TextField 2'),
                    maxLines: 2,
                    textCapitalization: TextCapitalization.characters,
                    onSubmitted: (value) {
                      setState(() {
                        print(value);
                      });
                    },
                  ),
                ],
              ),
            ),
          ),
        ),
      ),
    );
  }
}

It's working, I've check event are correctly send to flutter BUT, when you hold your mouse button and move the pointer, it does not know buttons status.

I would say the problem is on flutter side.

@zephylac, Once you get the multi-button click figured out, info, this PR is good to be merged.

Sorry for the late review. I've just merged. I've also just added the test code that @Drakirus shared to the examples repo as a "mousebuttons" example. https://github.com/go-flutter-desktop/examples/tree/master/mousebuttons


Update deploy translatable strings script to push commit hash

Summary
this is to pull the correct commit that is not main (which is how translations repo pull in qiskit repo for docs build) or to not wait till the actual release
Details and comments

@Mergifyio backport stable/0.25

Oh wait, I hadn't even realised that you'd made this PR directly to stable/0.25.  We actually need it on main and stable/0.45 as well, so I'll just forward port it to those.
@Mergifyio backport stable/0.45 main

@Mergifyio backport stable/0.45 main

Ah, I didn't realize that I was on stable/0.25 when I committed the changes. Sorry

No worries, it's a good excuse for me to learn if the bot will "backport" between completely arbitrary branches!


az.cmd can't handle arguments with a @()

Describe the bug
the az.cmd batch script that wraps python.exe and passes in the correct arguments to start azure.cli seemingly cannot handle arguments with a @() in them
my example is with az webapp config connection-string set because that is where I found it, but it most likely impacts every command
example:

PS C:\Users\mgrandi> # verify we have no existing connection strings in our web app
PS C:\Users\mgrandi> az webapp config connection-string list --resource-group "mgrandi-rg" --name "mgrandi-webapp"
[]

PS C:\Users\mgrandi> # try and set a connection string with a `@()` in it as the value
PS C:\Users\mgrandi> az webapp config connection-string set --connection-string-type custom --name mgrandi-webapp --settings "TestKey=@Microsoft.KeyVault(SecretUri=https://html5zombo.com)" --resource-group mgrandi-rg
--resource-group was unexpected at this time.
C:\Users\mgrandi>  "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin\\..\python.exe" -IBm azure.cli webapp config connection-string set --connection-string-type custom --name mgrandi-webapp --settings TestKey=@Microsoft.KeyVault(SecretUri=https://html5zombo.com) --resource-group mgrandi-rg
PS C:\Users\mgrandi> # note, the above is the entire output, i did not type that python.exe command

PS C:\Users\mgrandi> # verify still no connection strings
PS C:\Users\mgrandi> az webapp config connection-string list --resource-group "mgrandi-rg" --name "mgrandi-webapp"
[]

PS C:\Users\mgrandi> # try and set it with calling python.exe manually
PS C:\Users\mgrandi> & 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe' -IBm azure.cli webapp config connection-string set --connection-string-type custom --name mgrandi-webapp --settings "TestKey=@Microsoft.KeyVault(SecretUri=https://html5zombo.com)" --resource-group mgrandi-rg
  "TestKey": {
    "type": "Custom",
    "value": "@Microsoft.KeyVault(SecretUri=https://html5zombo.com)"
  }
}

PS C:\Users\mgrandi> # verify we have one connection string now
PS C:\Users\mgrandi> az webapp config connection-string list --resource-group "mgrandi-rg" --name "mgrandi-webapp"
[
  {
    "name": "TestKey",
    "slotSetting": false,
    "type": "Custom",
    "value": "@Microsoft.KeyVault(SecretUri=https://html5zombo.com)"
  }
]

PS C:\Users\mgrandi> # if we get rid of the ending ')' , it works
PS C:\Users\mgrandi> az webapp config connection-string set --connection-string-type custom --name mgrandi-webapp --settings "TestKey2=@Microsoft.KeyVault(SecretUri=https://html5zombo.com" --resource-group mgrandi-rg
{
  "TestKey": {
    "type": "Custom",
    "value": "@Microsoft.KeyVault(SecretUri=https://html5zombo.com)"
  },
  "TestKey2": {
    "type": "Custom",
    "value": "@Microsoft.KeyVault(SecretUri=https://html5zombo.com"
  }
}

PS C:\Users\mgrandi> # should have 2 connection strings now
PS C:\Users\mgrandi> az webapp config connection-string list --resource-group "mgrandi-rg" --name "mgrandi-webapp"
[
  {
    "name": "TestKey2",
    "slotSetting": false,
    "type": "Custom",
    "value": "@Microsoft.KeyVault(SecretUri=https://html5zombo.com"
  },
  {
    "name": "TestKey",
    "slotSetting": false,
    "type": "Custom",
    "value": "@Microsoft.KeyVault(SecretUri=https://html5zombo.com)"
  }
]

Command Name
az webapp config connection-string set
Errors:
after running, you get this output to the powershell terminal
--resource-group was unexpected at this time.
C:\Users\mgrandi>  "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin\\..\python.exe" -IBm azure.cli webapp config connection-string set --connection-string-type custom --name mgrandi-webapp --settings TestKey=@Microsoft.KeyVault(SecretUri=https://html5zombo.com) --resource-group mgrandi-rg

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

run a command with a @() in it

Expected Behavior
it should pass the arguments to the python process and either work or not, depending on the arguments
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6
Installer: MSI

azure-cli 2.9.1

Additional Context

Hi @mgrandi, this is powershell issue. If you use suuch command in cmd, it should work as expect.
In powershell, to resolve it please add --% after az, e.g.
az --% webapp config connection-string set --connection-string-type custom --name mgrandi-webapp --settings "TestKey=@Microsoft.KeyVault(SecretUri=https://html5zombo.com)" --resource-group mgrandi-rg

If it's powershell issue, then how come it works when I call python.exe directly?


Exit Delay doesn't match the configured time

Alarmo Version
v1.8.6
HA Version
2022.3.5
Bug description
Starting in the last month or so, the configured exit delay in Alarmo doesn't match the count down timer given on the Alarmo card.  For Arm Away, it is configured for 30 seconds exit and 30 seconds entry.
For example, when leaving home, I press 'Arm Away' it is showing a 4 second countdown timer in the alarmo card.   Returning home and entering the house yield a short timeout as well.
I have noticed this is happening on the wall panel tablet (Fire Tablet) w/ Fully Browser but not on my phone running the native app.
I reported this in the discussion thread a week ago or so and it was suggested to raise a ticket.
Today, I spent a little more time looking at the issue.   It appears the countdown is very short, but the alarm itself doesn't actually arm until the appropriate time.   It is starting the countdown at 4 seconds, gets to 0, and then just sits there until the 30 second configured time is reached and then it shows armed as expected.  So it may be more related to the countdown on the card being wrong, but Alarmo itself using the right value.
What information would be helpful in debugging?
Steps to reproduce
For example, when leaving home, I press 'Arm Away' it is showing a 4 second countdown timer in the alarmo card.   Returning home and entering the house yield a short timeout as well.
Relevant log output
No response

If I understand correctly, this is an issue with Alarmo-card, not Alarmo itself?

That would be my best guess with the realization that it only seems to impact the tablet that controls the whole house and doesn't seem to impact my phone or web interface

i am having the opposite issue.  I set alarmo to delay exit and it immediately triggers  on both arm away and arm home


[BUG] Update all Track 1 packages to point to new versions

When you search for any package on mvnrepository.com you see this:
https://mvnrepository.com/search?q=eventhubs&sort=popular

We need to go through all old packages and update the description to point to the new versions.

Use the description tag and not relocation tag with this message:
Update the package links accordingly in the sample below.
This library has been replaced by the following new Azure SDKs. You can read about the new Azure SDKs at https://aka.ms/azsdkvalueprop.
The latest libraries to interact with the Azure KeyVault service are:
* https://search.maven.org/artifact/com.azure/azure-security-keyvault-keys
* https://search.maven.org/artifact/com.azure/azure-security-keyvault-secrets
* https://search.maven.org/artifact/com.azure/azure-security-keyvault-certificates
It is recommended that you move to the new package.

Duplicate of https://github.com/Azure/azure-sdk-for-java/issues/7968
/cc @conniey


add support for CREATE EXTERNAL TABLE

currently only temporary external tables are supported, it will be nice to remove that limitation please

From an implementation side, what you are asking for is straightforward. In fact, it was already discussed a couple of times internally. Unfortunately, there is more than just implementation to this feature. In particular, usability and security pose challenges.
The main issues currently are:

If you move a .hyper file around, how does Hyper locate the external files? Through paths relative to the Hyper file? Absolute file paths?
Should there be a way to package external files together with the .hyper file? E.g., when uploading it to Tableau Cloud.
If you send a Hyper file via email, and some other person opens it, should Hyper read whichever external files are specified in the .hyper file? What if someone maliciously added an external table which reads /etc/passwd as an external CSV or some other sensitive data?
What if you upload a Hyper file to Tableau Cloud? Should that file be allowed to instruct Hyper to read /etc/password and display it as part of some visualization?

While the answer for /etc/password is clearly a "no, this should not be allowed", it's hard to draw the line here

1- absolute Path
2- no, that fail the purpose of an external table, the data has to be in a shared storage
3- that's not Hyper fault if someone store sensitive data without encryption, moreover only the user can see it anyway, but I am not a security expert
4- an option in tableau cloud to block reading from internal data
my use case is reading parquet files from remote storage, which I think is a very common pattern those days with lakehouse and stuff :)
thanks a lot for your reply.

Agree with @djouallah, permanent external table (and views) is missing in Hyper.
To answer your questions :
1 : both ( relative and absolute)
2 : external mean external so no packing external data into the hyper file
3 - limit external file extensions to csv, and parquet
4 - limit external file extensions should manage the problem. Limit the number of files in globs to 1000. Limit may be also on directories ( no /etc no /usr/,no /opt, no c:\windows, c:\program files....)
Do this security concerned cannot be blocked also by à security tool (edr or antivirus) on the Tableau cloud clusters ?
Object storage is great but there can be also fast parallel remote filesystems like pnfs or lustre that also provide excellent performance to access remote data....

recently DuckDB added an option to turn off reading from a local filesystem, I guess you guys can do the same for Tableau cloud, turn it off by default for security reason.


AST graphviz printer for visualization

Use the --ast cmdline flag to produce an AST graphviz output instead of interpreting.

Also fixed #21


ArchLinux Error with git

Error

Repo Details

System Details

From the AUR package as suggested by the docs
my log id: 8be6f05a31fb589d6c303f1c

I'm a little confused about what the error is.  Your log message said that git push failed, but you didn't show that error above.  Is it the same /usr/share/keybase error?  That message doesn't appear to be generated by keybase; at least I don't see it in the logs.  I wonder if that's something related to an interesting variable you have set in your environment, like $LD_LIBRARY_PATH or some $XDG variable.
Or is it just the --set-upstream issue?

Well my $LD_LIBRARY_PATH is unchanged, it's simply :/usr/local/lib.
The --set-upstream is not really the problem, since:

It might be a quirk of the AUR installation method but then I am confused as to why it has not been reported before...

I'm sorry, I don't have any ideas.  Maybecheck your .git/config to see if there's anything related to /usr/share/keybase in there?  I'm not an Arch expert and I don't have an environment available for testing; wish there was more I can do.

We have lots of folks using this install and this is the first such complaint so I would guess it’s specific to your config.

If that helps...

Try which keybase then ls it? Maybe there is a deal symlink somewhere in your path.

ls -ls /usr/bin/keybase

Oh, I don't know what sl is.  Sorry, i don't know

@HaoZeke This is pretty odd. Could you try making a fresh git repo and pushing and pulling from that?
If that doesn't work either, could you give the output of
$ pacman -Qkk keybase-bin
$ \ls -ls /usr/bin/keybase
$ env | grep XDG

I am not sure what the right permissions are though...

XDG_DATA_HOME not writable by you? That seems like a serious misconfig.

(One that has nothing to do with our software by the way)

Yup, that seems like the issue. Check if you're setting $XDG_DATA_HOME anywhere in your shell rc files. A good default is $HOME/.local/share. Better yet, just unset it if you don't know what it is/don't want to customize it.

Yikes, that was the issue. I am so sorry for bugging everyone here. Thanks a ton everyone!


[Zero Trust] Update index.md

@nmldiegues I adapted some content you wrote and added it to the config docs. wdyt? cc @abelinkinbio

Semi orthogonal to this PR but I think we can provide a script/gist to do this or add a bash script to this README. Thoughts @nmldiegues ?

Semi orthogonal to this PR but I think we can provide a script/gist to do this or add a bash script to this README. Thoughts @nmldiegues ?

I wouldn't want to get into that rabbit hole since that depends on how cloudflared is running. As a command? As a service? Which OS? As as container in docker? As a pod in k8s? etc...


lualine sometimes disappears under cmdline

Describe the bug
lualine sometimes disappears under cmdline, and I use the default configuration.
Now the trigger condition is to enter multiple letters
Which version of Neovim are you using?
Version: Nightly
Screen

Can't reproduce. Please provide a https://github.com/folke/noice.nvim/wiki/Minimal-`init.lua`-to-Reproduce-an-Issue

https://gist.github.com/jinzhongjia/9474de837f6a04676f1555ca0fa3fd4b I have stored minimal.lua on the gist, and this file can restore the situation, I suspect it is caused by my lualine configuration

And my system is Arch Linux, and mu neovim(nightly) is from repo called archlinuxcn

I know what change is causing this, but strangely enough I don;t have it with my config. So I'm looking into what is different in my config, so I can properly fix it


Adds isbn-verifier exercise

Hi, this PR is to add a new exercise for isbn-verifier, here is a link to the problem-specifications
I'm still learning Dart, so feel free to point out anything that needs to be change or improved. I'm also happy to change the config.json to better fit your layout for the track.
Thanks! :)

@Stargator Updated.


Query: include on groupby key doesn't get applied when the key is entity

query using gears of war model:
ctx.Gears.GroupBy(g => g.CityOfBirth)
    .OrderBy(g => g.Key.Name)
    .Select(g => g.Key)
    .Include(c => c.BornGears).ThenInclude(g => g.Weapons)

the includes should be applied, but they are not at the moment

Putting this on the backlog for now, but we should consider it along with #12238, which may mean that it never gets implemented.

Duplicate of #16752?


python312Packages.numpy-groupies: remove broken updateScript

The python package has an updateScript by default, and in most simple cases, we don't need to add it.
reported in https://github.com/NixOS/nixpkgs/pull/332158#discussion_r1713102734
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks.


Include XML documentation files in NuGet packages

Fixes #1031 Update NuGet packages to contain XML documentation
Also found a bad merge in Microsoft.Build.csproj

@dotnet-bot test OSX Build for CoreCLR please


Show a banner for our treasury proposal

closes #306
I went for just when there's something displayed in the home, and not when there's nothing to be found.

Great idea!
We need a bit of padding between it and the pure header component.

ah, good one, I didn't check with a pure :pray: fixed


[WIP] Move completion module to its own crate

This is a WIP PR to move the completion module to a separate crate. I'm relatively new to rust, so please let me know if I'm approaching this in the wrong way.

[x] Move check/tests/completion.rs to new crate. (Currently uses check/tests/support module so can't be moved trivially)
[x] Refactor completion crate's public API (Make internal types/functions private) Looking at the module again, it is already hiding non-public functions and interfaces. I think it makes sense to save any further refactoring for another PR.
[ ] The issue mentions that the completion is used in the language server. Find out where this is, I only saw the completion code used in repl/, check/tests/completion.rs, and tests/vm.rs

Resolves #332

Once the gluon_completion crate is published I'll update the language server with the new dependencies.

Once the gluon_completion crate is published I'll update the language server with the new dependencies.

Great! It should be a simple change. Only annoyance is that you will need to use a git dependency for testing it. After I publish 0.6 of gluon (which is overdue by now) I will change the language server's version to point to 0.6 again.


No internet sync in daemon.sh

On WittyPi3 there was a call in the daemon.sh file to a sync time script - https://github.com/uugear/Witty-Pi-3/blob/49763e0487e941ecbc06430368355002f3d3463b/Software/wittypi/daemon.sh#L108 This sync time script had a call to sync the pi with network time if there was internet - https://github.com/uugear/Witty-Pi-3/blob/49763e0487e941ecbc06430368355002f3d3463b/Software/wittypi/syncTime.sh#L40
In the WittyPi4 daemon there is no call to sync the time with the internet. This means a user has to manually sync the time when the board has been re-attached.

This change is made on purpose. The accuracy of RTC is good enough and there is no need to sync the time during every boot. On the other hand, we have seen many cases that the time was incorrectly synchronised due to a network issue, and hence causes problem on subsequent schedule script execution.

Ok thanks that is useful to know. I'll triple check whether that works for me because my board was 4 hrs out when plugged in this morning. I'll try and get to the bottom of that


Which kind of BN layer do you use?

Which kind of BN layer do you use in your implementation?
Fix BN parameters of pretrained backbone net?
or do you use the synchronized BN across GPU?

We choose the synchronized BN across GPU.


is tauri-plugin-oauth compatible with tauri v2?

I'm not really sure what I'm doing wrong, but for whatever reason my app doesn't work anymore after installing the plugin.
Here's the error I got:
><EMAIL_ADDRESS>tauri /Users/ahmed/code/env-man
> tauri "dev"

    Running BeforeDevCommand (`pnpm dev`)

><EMAIL_ADDRESS>dev /Users/ahmed/code/env-man
> vite

♻  Generating routes...
✅ Processed routes in 132ms

  VITE v5.4.8  ready in 608 ms

  ➜  Local:   http://localhost:1420/
    Info Watching /Users/ahmed/code/env-man/src-tauri for changes...
    Updating crates.io index
       Fetch [===========>                     ] 8 complete; 1 pending
       Fetch [==============>                  ] 9 complete; 69 pending
       Fetch [==============>                  ] 76 complete; 2 pending
       Fetch [=================>               ] 78 complete; 152 pending
       Fetch [=================>               ] 88 complete; 142 pending
       Fetch [=================>               ] 174 complete; 56 pending
       Fetch [=================>               ] 221 complete; 9 pending
       Fetch [=================>               ] 222 complete; 8 pending
       Fetch [=================>               ] 229 complete; 1 pending
error: failed to select a version for `webkit2gtk-sys`.
    ... required by package `wry v0.18.3`
    ... which satisfies dependency `wry = "^0.18.3"` of package `tauri-runtime-wry v0.9.0`
    ... which satisfies dependency `tauri-runtime-wry = "^0.9.0"` of package `tauri v1.0.0`
    ... which satisfies dependency `tauri = "^1"` of package `tauri-plugin-oauth v0.0.0-alpha.0`
    ... which satisfies dependency `tauri-plugin-oauth = "^0.0.0-alpha.0"` of package `env-man v0.1.0 (/Users/ahmed/code/env-man/src-tauri)`
versions that meet the requirements `^0.18` are: 0.18.0

the package `webkit2gtk-sys` links to the native library `web_kit2`, but it conflicts with a previous package which links to `web_kit2` as well:
package `webkit2gtk-sys v2.0.1`
    ... which satisfies dependency `ffi = "^2.0.1"` of package `webkit2gtk v2.0.1`
    ... which satisfies dependency `webkit2gtk = "=2.0.1"` of package `tauri v2.0.0`
    ... which satisfies dependency `tauri = "^2.0.0"` of package `env-man v0.1.0 (/Users/ahmed/code/env-man/src-tauri)`
Only one package in the dependency graph may specify the same links value. This helps ensure that only one copy of a native library is linked in the final binary. Try to adjust your dependencies so that only one package uses the `links = "web_kit2"` value. For more information, see https://doc.rust-lang.org/cargo/reference/resolver.html#links.

failed to select a version for `webkit2gtk-sys` which could resolve this conflict
 ELIFECYCLE  Command failed with exit code 101.

You need to use this repos v2 branch not the crates.io release.

Thank you so much!


Fix accessing trait collection outside main queue

Summary
With iOS 15 devices I am getting a UI access outside of main thread crash:
Main Thread Checker: UI API called on a background thread: -[UIView AS_traitCollection]
PID: 70497, TID: 20879363, Thread name: (none), Queue name: com.apple.root.user-initiated-qos, QoS: 25
Backtrace:
4   FormPresenter-Unit-SnapshotTests    0x000000010d30f738 -[UIView(DynamicTypeSnapshotting) AS_traitCollection] + 52
5   UIKitCore                           0x0000000184a891ac +[UIKeyboardCache enabled] + 40
6   UIKitCore                           0x0000000184a894f4 -[UIKeyboardCache cachedImageForKey:fromLayout:] + 52
7   UIKitCore                           0x0000000184a4f900 __37-[UIKBKeyplaneView prepareForDisplay]_block_invoke + 224
8   libdispatch.dylib                   0x0000000180108db0 _dispatch_call_block_and_release + 24
9   libdispatch.dylib                   0x000000018010a5a4 _dispatch_client_callout + 16
10  libdispatch.dylib                   0x000000018011b4ac _dispatch_root_queue_drain + 736
11  libdispatch.dylib                   0x000000018011bd28 _dispatch_worker_thread2 + 160
12  libsystem_pthread.dylib             0x00000001c99ccb04 _pthread_wqthread + 224
13  libsystem_pthread.dylib             0x00000001c99cb904 start_wqthread + 8
2022-05-24 11:12:14.345711-0400 AppHost-FormPresenter-Unit-Tests[70497:20879363] [reports] Main Thread Checker: UI API called on a background thread: -[UIView AS_traitCollection]
PID: 70497, TID: 20879363, Thread name: (none), Queue name: com.apple.root.user-initiated-qos, QoS: 25
Backtrace:
4   FormPresenter-Unit-SnapshotTests    0x000000010d30f738 -[UIView(DynamicTypeSnapshotting) AS_traitCollection] + 52
5   UIKitCore                           0x0000000184a891ac +[UIKeyboardCache enabled] + 40
6   UIKitCore                           0x0000000184a894f4 -[UIKeyboardCache cachedImageForKey:fromLayout:] + 52
7   UIKitCore                           0x0000000184a4f900 __37-[UIKBKeyplaneView prepareForDisplay]_block_invoke + 224
8   libdispatch.dylib                   0x0000000180108db0 _dispatch_call_block_and_release + 24
9   libdispatch.dylib                   0x000000018010a5a4 _dispatch_client_callout + 16
10  libdispatch.dylib                   0x000000018011b4ac _dispatch_root_queue_drain + 736
11  libdispatch.dylib                   0x000000018011bd28 _dispatch_worker_thread2 + 160
12  libsystem_pthread.dylib             0x00000001c99ccb04 _pthread_wqthread + 224
13  libsystem_pthread.dylib             0x00000001c99cb904 start_wqthread + 8

This PR corrects this crash by ensuring we access AS_traitCollection via the main queue.
This is a workaround while we figure out why this is now crashing on iOS 15+ simulator devices. From our investigation we found that UIKit is calling traitCollection from a background queue and thus causing this to crash. Without swizzling this crash does not occur.

Two thoughts:

Should we limit this behavior to iOS 15+, since it doesn't seem to affect older versions?
Lets add a comment in the code explaining why we're changing threading behavior here.

Two thoughts:

Should we limit this behavior to iOS 15+, since it doesn't seem to affect older versions?
Let's add a comment in the code explaining why we're changing threading behavior here.

Good call on #1, added and availability check!
For #2 I added a comment!


Setup Rspec as the unit testing framework

We have chosen to use Rspec as the testing framework rather than Minitest (the default).

👍


Bug: Setting computed value

Is it possible in VTU 2 to set initial computed data from a setup function? We can do it in Options API:
{
  computed: {
    hospitalId () {
      return this.aSampleDataKey
    }
  }
}

Then we can set the value of it manually in tests like
wrapper = mount({
  computed: { hospitalId: () => 69 },
})

This doesn't work in Composition API
const hospitalId = computed(() => aSampleDataKey.value)

return { hospitalId }

Is it possible to set computed value like this one?
wrapper = mount({
  computed: { hospitalId: () => 69 },
})

It is not possible to set the value of a computed with the composition API. As you can see in the mount API Reference (https://test-utils.vuejs.org/api/#mount) there is not computed option.
Depending in your case I would:

Test the real behavior

or if not possible/too complicated to test

Use a composable file and mock it

In case you need some help with a specific case, you can open a discussion