Intel OpenVINO backend

Proposed changes:
resolves https://github.com/RasaHQ/rasa/issues/9849
This PR introduces Intel OpenVINO backend for HuggingFace networks optimization. Pip package openvino brings runtime for CPU/iGPU/VPU and other hardware targets.
Enabled models:

bert-base-chinese
bert-base-uncased
distilbert-base-uncased
gpt2
openai-gpt
rasa_LaBSE
roberta-base

Disabled for now: "xlnet"
Status (please check what you already did):

[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

Enable DoD

[ ] ML engineer review
[ ] talk to infra about CI changes

Hi, @TyDunn!
OpenVINO optimizes inference time (latency and throughput). So users might use this feature to improve efficiency of their pipelines.
To confirm the speed up, I wanted to ask for efficiency benchmarks which I could use.

Hi! Just wanted to ask if you have some comments here.

@dkurt Thanks for running that quick experiment.
Question: When you say TensorFlow backend, what is the config that you use? I am particularly interested in knowing what's the max length of the input in that case? I can see that you used 9 when use_openvino is True but can't see the value when its False (assuming thats when the TF backend kicks in)

@dakshvar22,
Sure, sorry for not complete description. TensorFlow backend means default backend for lm_featurizer models from HuggingFace (which if TensorFlow). So config is just
language: en

pipeline:
  - name: WhitespaceTokenizer
  - name: LanguageModelFeaturizer
  - name: DIETClassifier
    constrain_similarities: true
    epochs: 100
  - name: ResponseSelector
    epochs: 100
    constrain_similarities: trues

The options use_openvino: true and openvino_max_length: 9 are optional. use_openvino enables OpenVINO when value true.
openvino_max_length: 9 is a config specific for OpenVINO. There is currently a limitation which requires long internal reinitialization if model processes variable length inputs. This parameter is similar to tokenizer's max_length + pad_to_max_length when we pad input to fixed length and then just cut the output. I've named this value with openvino_ prefix to indicate that it's OpenVINO only.

Right, that's what I thought.. Can you set that parameter to the maximum sequence length you would get on the dataset with rasa init? That will ensure that both the backends are processing sequences of same length which will remove any differences in time occurring because of different size of sequence lengths.

@dkurt Let's try running these benchmarks on a larger dataset. I'd suggest cloning one of our public bots: Carbon bot. Once you have cloned, you can change the config.yml file to the above config that you have been using.
For benchmarking, what really matters is the time taken by LanguageModelFeaturizer inside its train and process method. It would be great if you can setup a way to track these times inside the respective functions (simple calls to time.time() at the start and end of the function would do?).
To record the time in train, you can run rasa train nlu and see the times reported by your custom setup.
To record the time in process, what you really want to do is measure how much time it takes to process one message. We'll have to write a bit of code for this and do somethings manually.
I think something like this would work:
from rasa.shared.nlu.training_data.loading import load_data
from rasa.nlu.featurizers.dense_featurizer.lm_featurizer import LanguageModelFeaturizer

data = load_data(<path to nlu.yml of carbon bot>)
featurizer = LanguageModelFeaturizer({"model_name": <>, "model_weights": <>, "cache_dir": <>})

for example in data.training_examples:
    featurizer.process(example)

We should record the average amount of time it takes for featurizer.process(example) to run.
(I know it would be great to have all of this automated and we are currently in the process of setting up that infrastructure but I don't want you to be blocked by it, so we would appreciate if you could try this out manually for now :) )

Hi, @dakshvar22. Thanks for help!
I used the following script to measure process efficiency:

Script
import time
import logging
import numpy as np
from rasa.shared.nlu.training_data.loading import load_data
from rasa.nlu.featurizers.dense_featurizer.lm_featurizer import LanguageModelFeaturizer
from rasa.engine.graph import (
    ExecutionContext,
    GraphSchema,
)
from rasa.nlu.tokenizers.whitespace_tokenizer import WhitespaceTokenizer

logging.basicConfig()
logger = logging.getLogger("rasa").setLevel(logging.INFO)

data = load_data('data/nlu.yml')

def create_whitespace_tokenizer(config=None):
    return WhitespaceTokenizer(
        {**WhitespaceTokenizer.get_default_config(), **(config if config else {}),}
    )

tk = create_whitespace_tokenizer()
tk.process_training_data(data)

def run(use_openvino):
    featurizer = LanguageModelFeaturizer({"model_name": "bert",
                                          "model_weights": "rasa/LaBSE",
                                          "cache_dir": None,
                                          "use_openvino": use_openvino,
                                          "openvino_max_length": 77,
                                          "alias": ""},
                                         execution_context=ExecutionContext(GraphSchema({}), "1"))

    total_start = time.time()
    times = []
    for example in data.training_examples:
        start = time.time()
        featurizer.process([example])
        times.append(time.time() - start)

    print(f"Use OpenVINO={use_openvino}")
    print(f"Total time: {time.time() - total_start} seconds ({len(times)} messages)")
    print(f"Median for one message: {np.median(times)} seconds")
    print()

run(use_openvino=False)
run(use_openvino=True)

Results:
Use OpenVINO=False
Total time:<PHONE_NUMBER>7746277 seconds (2517 messages)
Median for one message: 0.08835482597351074 seconds

Use OpenVINO=True
Total time:<PHONE_NUMBER>5444946 seconds (2517 messages)
Median for one message: 0.04765892028808594 seconds

CPU: Intel(R) Core(TM) i7-6700K
OpenVINO relies on manually specified openvino_max_length value to pad input tokens by zeros instead of internal model reinitialization (I believe there is coming feature in future OpenVINO releases which will allow to avoid such parameter). Maybe it make sense to rename it to something like openvino_fixed_length because of confusion with

@dkurt Those are some great results! Thanks for carrying out that experiment.
For sake of completeness, I would request you to carry out one more experiment -> We recently upgraded the tensorflow version in our dependencies and that brings in a slowdown in process of LanguageModelFeaturizer. There are some more details of the impact of upgrading tensorflow in this blog post.
To completely rule out its effect on the numbers we are seeing above, can you please create a new PR, but this time targetting 2.7.x branch of Rasa OSS, port the changes required for OpenVino there and run the same benchmark? You can leave this PR still open. If we decide to merge it, we'll still merge it to main but would be good to have numbers with a version of Rasa OSS which is not impacted by the performance degradation.
Thanks again for your patience on this contribution! 🙏

@dakshvar22, indeed there is a difference in TensorFlow numbers. Rebased the changes to 2.7.x branch: https://github.com/dkurt/rasa/tree/2.7.x_openvino

Adopted script
import time
import logging
import numpy as np
from rasa.shared.nlu.training_data.loading import load_data
from rasa.nlu.featurizers.dense_featurizer.lm_featurizer import LanguageModelFeaturizer
from rasa.nlu.tokenizers.whitespace_tokenizer import WhitespaceTokenizer

logging.basicConfig()
logger = logging.getLogger("rasa").setLevel(logging.INFO)

data = load_data('data/nlu.yml')

def create_whitespace_tokenizer(config=None):
    return WhitespaceTokenizer(
        {**WhitespaceTokenizer.defaults, **(config if config else {}),}
    )

tk = create_whitespace_tokenizer()
for example in data.training_examples:
    tk.process(example)

def run(use_openvino):
    featurizer = LanguageModelFeaturizer({"model_name": "bert",
                                          "model_weights": "rasa/LaBSE",
                                          "cache_dir": None,
                                          "use_openvino": use_openvino,
                                          "openvino_max_length": 77,
                                          "alias": ""})

    total_start = time.time()
    times = []
    for example in data.training_examples:
        start = time.time()
        featurizer.process(example)
        times.append(time.time() - start)

    print(f"Use OpenVINO={use_openvino}")
    print(f"Total time: {time.time() - total_start} seconds ({len(times)} messages)")
    print(f"Median for one message: {np.median(times)} seconds")
    print()

run(use_openvino=False)
run(use_openvino=True)

Use OpenVINO=False
Total time:<PHONE_NUMBER>6608887 seconds (2517 messages)
Median for one message: 0.07500743865966797 seconds

Use OpenVINO=True
Total time:<PHONE_NUMBER>4964905 seconds (2517 messages)
Median for one message: 0.04769301414489746 seconds

TensorFlow is 2.3.4

Thanks @dkurt !
Looking at the description of your PR, it seems like we need to maintain openvino "compatible" versions of model weights:

What does it take to convert any model weights of a specific architecture (like BERT, etc.) to OpenVino format?
Do you all plan to actively support the models hosted at https://huggingface.co/dkurt/openvino?

@dakshvar22, the models can be generated in runtime by the following method (already a part of PR):

Conversion code
    def _load_model(self, input_ids: np.ndarray, attention_mask: np.ndarray) -> None:
        # Serialize a Keras model
        @tf.function(
            input_signature=[
                {
                    "input_ids": tf.TensorSpec(
                        (None, None), tf.int32, name="input_ids"
                    ),
                    "attention_mask": tf.TensorSpec(
                        (None, None), tf.int32, name="attention_mask"
                    ),
                }
            ]
        )
        def serving(inputs: List[tf.TensorSpec]) -> tf.TensorSpec:
            output = self.model.call(inputs)
            return output[0]

        self.model.save("keras_model", signatures=serving)

        # Convert to OpenVINO IR
        proc = subprocess.Popen(
            [
                sys.executable,
                "-m",
                "mo",
                "--saved_model_dir=keras_model",
                "--model_name",
                self.model_name,
                "--input",
                "input_ids,attention_mask",
                "--input_shape",
                "{},{}".format(input_ids.shape, attention_mask.shape),
                "--disable_nhwc_to_nchw",
                "--data_type=FP16",
            ],
            stdout=subprocess.PIPE,
            stderr=subprocess.PIPE,
        )
        proc.communicate()

        # Load model into memory
        self.net = self.ie.read_network(self.model_name + ".xml")

However it's time consuming procedure so I uploaded models to the hub. Maybe it's better to modify the proposal to generate model once for users into the cache?

@dakshvar22, did the following changes:

Models now converted in runtime with no external reference to custom hub
Once converted, OpenVINO is saved in common cache
Skip CI tests despite distilbert-base-uncased - others consume too much memory and Actions crash. Works fine locally.
There are too deps: openvino (runtime, official package) and experimental-openvino-mo (TF->OV converter, unofficial). There are two questions from my side: 1) Should I make them optional/extras so OpenVINO will be optional 2) Is there a way to manage poetry's package sub-dependencies like pip install --no-deps? If so, I could replace an unofficial experimental-openvino-mo to official one after dependency conflict.

@dkurt Summarizing all the findings here:

This PR attempts to speed up the inference time inside LanguageModelFeaturizer on CPU environments.
Internal benchmarking suggests a speedup of 1.37x when using the openvino backend for inference on CPU.
All models need to be converted to openvino format before they can be used with the openvino backend. This can be done offline and hosted on the hf model hub or on the fly as a one time activity when the model is loaded.
Users need to necessarily specify a maximum input sequence length if using the openvino backend.

Does this sum up the important points you would like us to know about the contribution?

@dakshvar22, yes, completely! Do you think I need to put this summary to changelog/9826.feature.md or docs/docs/openvino-backend.mdx?

@dkurt Thanks for confirming! I'd hold off making any more changes to the PR. I've communicated the intended effects of this PR to product management internally and we'll be back soon with the next steps. Thanks for your patience 🙏

Hi! Do you think it might reduce maintenance effort and help with integration decision if we wrap OpenVINO related code into a separate package? In example, like Hugging Face's Optimum extension there is https://github.com/dkurt/optimum-openvino (work in progress).

Hi @dkurt,
Thanks for the PR and sorry for making you wait so long.
We're currently trying to decide whether we should keep this in the codebase or make it into a separate package.
From what I understand, if we go with a separate package you'll be mainly responsible for maintaining it. If this is correct and if you're willing do to it, then what I'd like to know is 1. how do you imagine supporting it i.e. what kind of maintenance do you intent to do? and 2. will you need any assistance from us?

Hi, @jupyterjazz! No worries :) I have opened a PR which demonstrates how we can move OpenVINO related code to a separate package: https://github.com/dkurt/rasa/pull/5.

how do you imagine supporting it i.e. what kind of maintenance do you intent to do? and

The package will be regularly tested on the same version of Transformers library which RASA uses. For now it's under my account but if that approach is more comfortable, we will initiate transfer under https://github.com/openvinotoolkit/ org.

will you need any assistance from us?

Thanks! Just a review, I will be in touch how integration goes and can help with users experience.

@dakshvar22, @jupyterjazz, Good news! Reproduced the same experiment as https://github.com/RasaHQ/rasa/pull/9826#issuecomment-966051532 but with coming OpenVINO release (2022.1). It will propose a dynamism feature which allows process variable input sizes. So parameter openvino_max_length could be avoided.
Thus, the performance is:
OpenVINO 2022.1 openvino_max_length 77
Total time:<PHONE_NUMBER>8589172 seconds (2517 messages)
Median for one message: 0.05508852005004883 seconds

OpenVINO 2022.1 dynamic shapes (no openvino_max_length)
Total time: 69.31450295448303 seconds (2517 messages)
Median for one message: 0.027573347091674805 seconds

Hi! I have updated this PR to use a separate package for OpenVINO related logic. I'm going maintain and develop it. I've also initiated a transfer procedure to https://github.com/openvinotoolkit/.

Hi! Glad to inform that we transferred the package to OpenVINO org (here). Updated this pull request correspondingly.

@jupyterjazz, @dakshvar22, I have updated PR to use the latest OpenVINO version 2022.1. Also, removed max_length option.


Works-About_Us-Form-Sections

Added works sections
Added about us sections
Added form section
Added social media buttons at bottom

Finally! Thanks very much for the review.


fix(tree): Sorted tree problems.

Coverage remained the same at 100.0% when pulling 06e1107055f48e664c4b5f9b19e9840b34f89bcb on sort-tree-problems into f1d141423f5d74a4c453fcb2305a52bc971d9ae8 on master.

Coverage remained the same at 100.0% when pulling 06e1107055f48e664c4b5f9b19e9840b34f89bcb on sort-tree-problems into f1d141423f5d74a4c453fcb2305a52bc971d9ae8 on master.


Capture 405 in access logs

Describe the bug
If a URL request is made that does not match a valid endpoint, we are not seemingly logging the 405 in the general access log.
VinylDNS Version
Latest
To Reproduce
Steps to reproduce the behavior:

Make any request against the API that is not valid, for example GET /foo
The API returns a 405, which is correct
Look at the log output, no log is made on the invalid request

Expected behavior
A log entry should be made indicating the PATH that was requested
Screenshots
Stack trace or error log output
Additional context
Add any other context about the problem here.

addressed in https://github.com/vinyldns/vinyldns/pull/508


CodeChecker store problem with file paths (Windows Client to Linux Server)

Hello,
I am trying to use CodeChecker store to send the Reports to an server but that does not work as expected:
I do the analysis on a Windows 10 PC with clang-tidy. The clang tidy report is converted with the converter tool (report-converter)  of this project into plist files.
The plist files contain references to the source files with the whole path, e.g.:
	<array>
		<string>H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\rc_lib\Ctr_lib\inc\ctr_bas.h</string>
		<string>H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\soft.xxx\CTR\APPL\projectdefines.h</string>
		<string>H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\soft.xxx\CTR\System\ce.c</string>
	</array>

Then I upload the reports to a Ubuntu machine running a CodeChecker server, by using the command CodeChecker store. The process looks good on the client side, no errors are reported. On the server a run is created, but it is basically empty, containing no information.
In the server log I find a lot of these Errors:
[ERROR][2022-04-20 13:25:04] {server} [25289] <140079552255808> - mass_store_run.py:409 __store_source_files() - File ID for /home/sev/.codechecker/tmptwoeu6ad/root/H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\soft.xxx\CTR\System\vis.c is not found in the DB with content hash 8f1ee6035bffbd31504fff06ae7b62f8b937deddb4fca53f54b26851c1ec4a85. Missing from ZIP?

and later many of these errors appear:
[WARNING][2022-04-20 13:25:05] {server} [25289] <140079552255808> - mass_store_run.py:866 __process_report_file() - Failed to get database id for file path '/home/sev/.codechecker/tmptwoeu6ad/reports/362d7e1b01f2458f4dba9c951e5979f0/H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\soft.xxx\Sim\doktabl.cpp /home/sev/.codechecker/tmptwoeu6ad/reports/362d7e1b01f2458f4dba9c951e5979f0/H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\rc_lib\Ctr_lib\inc\ctr_bas.h'! Skip adding report: /home/sev/.codechecker/tmptwoeu6ad/reports/362d7e1b01f2458f4dba9c951e5979f0/H:\repos\projects\_BUS\CPU_WinPC\Software\Neutralproject\soft.xxx\Sim\doktabl.cpp:587:3 [clang-analyzer-security.insecureAPI.strcpy]

I suspect there is something wrong with the file path prefixes, because the full path of the file from the Windows system appears in the logs of the linux systems.
I tried using the --trim-path-prefix option for the store command with different variants of the path prefixes. However, it seems that this changes nothing on the server side, the same errors appear every time.
Is the --trim-path-prefix even working? Am I doing something wrong?
Version infos:
CodeChecker Version on Client: Git tag information  | 6.19.1
CodeChecker Version on Server: Git tag information  | 6.19.1

I have the same problem - the reason seems to be that the encoded filenames are not harmonized in anyway. Both where they are added to the zip file and to the JSON data. I am looking into maybe adding a function for harmonizing the filenames for all platforms. I think it makes sense that the path's are all using UNIX style /.
I poke at it a bit and try to get it to work.

I have just been bitten by this issue too, cf https://github.com/Ericsson/codechecker/issues/3814
If any of you have an idea of something I could try, I'd be happy to !

It seems spot-rewriting all the paths to be POSIX-y when assembling the ZIP (and also changing the contents of the plists) might be a viable solution. It is bound to get ugly in presentation, however: c:/foo/bar
If we want to get extra pedantic here, we could do what Wine and Lutris are doing and rewrite C:\ as drive_c, and only then normalise the paths replacing \ with /. Extra care needs to be taken for path components that contain spaces or special characters as they need to be escaped on the POSIX side of things.
~/Games $ ls -alh ./dosdevices
c:  ->  ../drive_c
d:  ->  ../drive_d
z:  ->  /

The only big problem site I can envision with this is the "diff a server's contents against a local report directory"...

that would be great. note that you need to replace the c: part of the path with something like drive_c, because otherwise ZipFile just removes the drive prefix.

We have encountered this issue as our last main blocker to have CodeChecker running on Windows. Is there a suggested fix for this at this stage?


Update config-settings.rst

For https://github.com/mattermost/mattermost-server/pull/11372

@aaronrothschild Gentle reminder to review this PR


Experimental bundling of Java primitive function implementations with their metadata

One way to cut down on the complexity of adding primitive function implementations in Java. We need at a minimum, for each primitive: 1) an implementation, 2) the name and a convenient way of constructing a term from the name, and 3) the type signature. (1) and (2) are illustrated here. (3) is not yet.

This is really cool! Makes for a more elegant mental model and will probably make it easier to decode from DSL :)


chore(STONEINTG-888): added CODEOWNERS

Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@1fad89e). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #823   +/-   ##
=======================================
  Coverage        ?   71.82%
=======================================
  Files           ?       43
  Lines           ?     3986
  Branches        ?        0
=======================================
  Hits            ?     2863
  Misses          ?      844
  Partials        ?      279

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


yaml: remove | from BBMASK

BBMASK is space separated list of regular expressions. So it's possible to use |, but space is more proper separator. Also spaces increase readability.

Signed-off-by: Ruslan Shymkevych<EMAIL_ADDRESS>Acked-by: Volodymyr Babchuk<EMAIL_ADDRESS>

Also this PR is fixing:
WARNING: BBMASK contains regular expression beginning with '|', fixing: |meta-rcar-gen3-adas/recipes-bsp/optee


DNSARecord, DNSSRVRecord issues

I'm having issues creating DNS records with nested domains and wildcards, examples below:
apiVersion: network.azure.upbound.io/v1beta1
kind: DNSARecord
metadata:
  annotations:
    crossplane.io/external-name: '*.some-subdomain'
  name: some-a-record
spec:
  forProvider:
    records:
    - <IP_ADDRESS>
    resourceGroupName: some-resource-group
    ttl: 300
    zoneName: some.dns.zone
  providerConfigRef:
    name: azure-upjet
  providerRef:
    name: azure-upjet
---
apiVersion: network.azure.upbound.io/v1beta1
kind: DNSSRVRecord
metadata:
  annotations:
    crossplane.io/external-name: some.multi.level.domain
  name: some-srv-record
spec:
  forProvider:
    record:
    - port: 443
      priority: 0
      target: some.target.domain
      weight: 0
    recordType: SRV
    resourceGroupName: some-resource-group
    ttl: 300
    zoneName: some.dns.zone
  providerConfigRef:
    name: azure-upjet
  providerRef:
    name: azure-upjet

Both of these end up producing the same error in the underlying terraform provider.
A name must start with a letter or underscore and may contain only letters, digits, underscores, and dashes.
It seems to me that this is a bug because currently it's not possible to create DNS records for nested domains.

We are using the external-name annotation in a patch for the actual record containing characters that are not allowed.
toFieldPath: metadata.annotations[crossplane.io/external-name]


Allow multitenant wallet creation through camel route

Supported like this ...
                from("direct:admin")
                    .to("hyperledger-aries:admin?service=/multitenancy/wallet");

Done


add admin3 boundary layer and config; COUNTRY=mozambique

Description
Adds admin-level 3 boundary and tests adding non-administrative boundary file
How to test the feature:

[ ]
[ ]

Checklist - did you ...

Test your changes with

[ ] REACT_APP_COUNTRY=rbd yarn start
[ ] REACT_APP_COUNTRY=cambodia yarn start
[ ] REACT_APP_COUNTRY=mozambique yarn start
[ ] Add / update necessary tests?
[ ] Add / update outdated documentation?

Screenshot/video of feature:

Note that as discussed for Zimbabwe, this is currently breaking AA


[ZEPPELIN-23] Set version of default spark interpreter build profile from 1.1 to 1.3

I have activated spark-1.3 profile by default instead of change default spark version from 1.1.1 to 1.3.1. Because of, to make spark 1.3.1 work, not only spark.version but also all related library version should be taken care.
Also unitest are updated to make work with spark 1.3.x

@ilganeli Thanks for the feedback! I have updated source code's indentation, also followed your suggestion that check version number in numerical value.

Changed to return SparkInterpreterTest.getSparkVersionNumber() >= 13

LGTM

Thanks!


scrollTo does not work for initialised value

When an lt-body is initialised with a value for @scrollTo, the table does not automatically scroll to this position.
Instead scrollTo only seems to work if a value is provided after the initial render.
I'm willing to take a look at fixing this, but I'm finding it hard to find the code where the scroll is actually registered for a scrollTo, so any pointers are welcome :)

Ok, I think I've figured out the issue...
Because scrollTo is triggered when via didReceiveAttributes, the @scrollTo arg is processed before the table renders.
Additionally, because the implementation relies on a private _scrollTo property - where if this value matches the @scrollTo arg, the implication is that scroll was successful and should not be re-triggered. But the issue here is that this can be set before the table has even rendered, so the ember-scrollable was not able to perform the scroll.
It seems that generally a more robust system is needed to properly handle scroll, but for now I'm going to try and patch something so that scrollTo also works for first render


refactor(minifier): unify ValueType

#6545  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Get-DbaXESession not parsing Linux paths well

Verified issue does not already exist?
Yes
What error did you receive?
if ($files) {
                    foreach ($file in $files) {
                        if ($file -notmatch ':\\' -and $file -notmatch '\\\\') {
                            $directory = $server.ErrorLogPath.TrimEnd("\")
                            $file = (Join-DbaPath $directory $file)
                        }
                        $filecollection += $file
                        $remotefile += Join-AdminUnc -servername $server.ComputerName -filepath $file
                    }
                }

Needs a Test-LinuxOS or whatever that command is
Steps to Reproduce
PS /mnt/c/github/psmodulecache> Get-DbaXESession | select targetfile

TargetFile
----------
{/var/opt/mssql/log\15_second_io_error}
{}
{/var/opt/mssql/log\AlwaysOn_health.xel}
{/var/opt/mssql/log\/tmp/.trc\dbatoolsci-session.xel}
{/var/opt/mssql/log\Login Tracker}
{}
{}
{}
{}
{/var/opt/mssql/log\system_health.xel}
{}

Are you running the latest release?
Yes
Other details or mentions
No response
What PowerShell host was used when producing this error
PowerShell Core (pwsh.exe)
PowerShell Host Version
na
SQL Server Edition and Build number
na

fixed


Figure out how to make npm run version work correctly

Related to https://github.com/npm/npm/issues/7906 .
Still missing some bit here.

There's a hook now. Just need to wait for a new stable version of NPM to go out.
https://github.com/npm/npm/commit/b07f7c7c1e5021730b3c320f1b3a46e70f8a21ff


issue

could not opening

We are pleased to inform you that the issue has been resolved. You can now continue to use our application as usual. Thank you for bringing this to our attention


Smart execution providers

closes #30

The purpose of this PR is:

Best effort to use the gpu over cpu if not fallbacks to wasm
Use onnxruntime-node in case is installed as dep in any node app using transformers.js (5x faster than WASM). If not installed it will falback to WASM provider.
Adds the hack for this issue setting numThreads to one
Use onnx executorProviders fallback chain to improve faster inference in browser if the model supports it.
Wraps multiple onnxruntime-web  requirement dependancy  under one file

@xenova This should do the trick. I need to test the web and check that the fallback is proceeding fine.

@xenova Im not totally happy with the PR unless we remove all the backends and allow the user to install node bindings, which are much faster than wasm.
Also the fallback is not working if the model do not support a layer 🤦
I think that we should provide a way to expose the desired  executor.
So I would change the PR to do this.
let ONNX;
let executionProviders = [ 'wasm' ];

try {
    ONNX = require('onnxruntime-node');
    executionProviders = [ 'cuda', 'cpu' ];
} catch (err) {
    ONNX = require('onnxruntime-web');
    if(typeof process === 'object') {
        // https://github.com/microsoft/onnxruntime/issues/10311
        ONNX.env.wasm.numThreads = 1;
    }
}

With the code below we have at least fixed a rough edge and allow the user to use node bindings if desired.
What do you think?

Yeah I agree 👍 Once WebGPU releases, I'll be more focused on getting GPU support working (both for node and browser).
I'll merge this main into this PR, make some edits, then merge the PR back into main (hopefully soon haha). Thanks again for your contributions!

I ran some tests to see what kind of speedup these changes make, and it's amazing!

Task
Speedup

Text classification
1400%

Question answering
600%

Image-to-text
400%

Text-to-text generation
350%

Code generation
350%

Embeddings
325%

Masked language modelling
300%

Translation
225%

Summarization
200%

Zero-shot image classification
200%

Image classification
175%

Text generation
150%

🎉
I'm doing some final merging and will hopefully get it published soon :)

@DavidGOrtega Can you grant me write access to your fork? I would like to push the changes without having to create a new fork and make a PR. (I think you can just add me as a collaborator?)

Got it working, PR merged! 🎉 Thanks again for your contributions!


Remove conda from travis-ci.

We don't use any conda specific packages (anymore).
Conda installs a series of packages which can interfere with the versions we want to install.
e.g. currently this is certifi
This is even faster to test as (by ~1 min) as we don't install conda and create a new environment.
Travis will still use a python virtualenv to set everything up in.

@DanielAndreasen You can merge this if you are happy with removing conda.
I think this will Fix #115.


Menu selection not rendered correctly on iPad

iPad, I can repro both on Chrome and Safari - however only on an iPad
Notice how in the picture the click path is not properly selected and rendered in blue.
In the picture below "View" and "Editor Layout" should be in blue. "Command Palette" should not be in blue.
If I use the mouse on an iPad the bug does not reproduce.
So I would guess that you are clearing selection on Hover. Can you also make sure to update selection on Click (which is actually touch)

@sbatten can you provide some pointers for this one. I would like to understand it better and potentially fix in June.

This code (for submenus) lives here: https://github.com/microsoft/vscode/blob/e1f0f8f51390dea5df9096718fb6b647ed5a9534/src/vs/base/browser/ui/menu/menu.ts#L692


Missing info - Azure Firewall is not mentioned, and NAT Gateway can't be used from peered VNets

Regarding this doc:

https://learn.microsoft.com/en-us/azure/load-balancer/load-balancer-outbound-connections

Please add Azure Firewall as a SNAT alternative.

Please add known, important limitations to the various products compared here.

Like the fact that NAT Gatewat can't be used from peered VNets / hub-spoke scenario.

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

ID: a96406ce-2ffc-cdb3-1234-5154ccd5cc76
Version Independent ID: f840a092-e1bf-16e2-1916-4270cdfbbcdd
Content: Source Network Address Translation (SNAT) for outbound connections - Azure Load Balancer
Content Source: articles/load-balancer/load-balancer-outbound-connections.md
Service: load-balancer
GitHub Login: @mbender-ms
Microsoft Alias: mbender

@o-l-a-v
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@o-l-a-v thanks for the feedback on this doc. The intention of this doc is to provide information on Azure's most commonly used methods for outbound connectivity rather that to list all available outbound connectivity methods. I have made edits to the doc to reflect this intention.
In relation to your other points of feedback, we are actively working on updating this doc overall to provide more clarity on use cases and will publish updates in the new year.


Allow custom classes for PerfectScrollbar container and elements

I've been recently trying to use PerfectScrollbar with a Chrome Extension that renders some content in Gmail.
What happens, however, is that Gmail already defines their own .ps class, which interferes with the styling for the custom scrollbars:

For this case in particular, we were able to work around the issue by manually setting padding and margin to 0 !important directly in the style attribute of our elements, but it's extremely convoluted and requires us to be very verbose with containers.
I think a more approachable solution would be to allow developers to define their own set of classes to be applied to the scrollbar container and its elements, something like this:
const ps = new PerfectScrollbar('#container', {
    classes: {
        main: 'custom-ps',
        element: {
            thumb: x => `custom__thumb-${x}`,
        },
    },
});

It would then be at the developers' liberty to rename the classes in a custom style file.

Make sure these boxes are checked before submitting your issue -- thank you!

[x] Check FAQ and Caveats
[x] Search if there's already one reported in Issues
[ ] Prepare a JSFiddle reproducing the issue

perfect-scrollbar JSFiddle

[ ] Provide a page or source code where the issue can be checked

@mdbootstrap I've opened #886 to partially solve this.
I'm happy to make any alterations you think necessary in order to get this live.


[TEST] proper credit_wallet function from services for test helpers

this is required for bolt11 library merge PR https://github.com/lnbits/lnbits/pull/1817
the issue was, creditwallet used an empty payment_hash

Codecov Report

Merging #1862 (b1f7387) into main (03acf5e) will decrease coverage by 6.96%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1862      +/-   ##
==========================================
- Coverage   56.09%   49.14%   -6.96%
==========================================
  Files          48       48
  Lines        7328     7328
==========================================
- Hits         4111     3601     -510
- Misses       3217     3727     +510

see 17 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


NotImplementedError when executing Pong example

Hey guys,
first of all thanks a lot for this project. It might become handy during my studies :)
I ran into an error while executing the example. I downloaded the pretrained model, but
python3 -m baselines.deepq.experiments.atari.enjoy --model-dir /tmp/models/model-atari-duel-pong-1 --env Pong --dueling
raised the following error:
[2018-01-31 12:24:50,221] Making new env: PongNoFrameskip-v4
Traceback (most recent call last):
  File "/usr/lib/python3.4/runpy.py", line 170, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.4/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/username/keras-tf-p3/baselines/baselines/deepq/experiments/atari/enjoy.py", line 70, in <module>
    play(env, act, args.stochastic, args.video)
  File "/home/username/keras-tf-p3/baselines/baselines/deepq/experiments/atari/enjoy.py", line 43, in play
    obs = env.reset()
  File "/home/username/keras-tf-p3/baselines/baselines/common/atari_wrappers.py", line 167, in reset
    ob = self.env.reset()
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 104, in reset
    return self._reset()
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 283, in _reset
    return self.env.reset()
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 104, in reset
    return self._reset()
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 310, in _reset
    observation = self.env.reset()
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 104, in reset
    return self._reset()
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 311, in _reset
    return self._observation(observation)
  File "/home/username/keras-tf-p3/lib/python3.4/site-packages/gym/core.py", line 321, in _observation
    raise NotImplementedError
NotImplementedError

The cartpole example however works fine (python -m baselines.deepq.experiments.train_cartpole)
I found this issue in gym (https://github.com/openai/gym/issues/775) with the same error message, but mine seems to occur in reset() and not in render(). Downgrading to pyglet version 1.2.4 as suggested did not change anything.
Does someone know a solution to this?

I found the solution: The NotImplementedError was due to an old gym version. Updating to latest gym version leads to cannot import name spaces error which can be solved by https://github.com/openai/gym/pull/851

simontudo : how you update gym? Which version of gym you are using?

Depends on how you installed it. If you just installed it via pip then execute pip install gym --upgrade
If you cloned the git repository first, you need to cd to the repo, execute git pull and then pip install -e .

Simontudo Thanks I have installed it from github repo Also I have updated it by command
However, my problem remains the same.
I saw the same error for another code I am using gym!
Your problem is solved by just doing update nothing else?

So you have the latest versions of baselines and gym and still getting the NotImplementedError?
When I updated gym, I got cannot import name spaces and solved it with https://github.com/openai/gym/pull/858

Yes, I have this problem(NotImplementedError)when I donwloaded from https://github.com/openai/gym
I saw today the following comments in github for gym it has written: (Did you make these changes? If so, could you explain what we should do clearly? Sorry for misunderstanding)
What's new
2018-01-25: Made some aesthetic improvements and removed unmaintained parts of gym. This may seem like a downgrade in functionality, but it is actually a long-needed cleanup in preparation for some great new things that will be released in the next month.
Now your Env and Wrapper subclasses should define step, reset, render, close, seed rather than underscored method names.
Removed the board_game, debugging, safety, parameter_tuning environments since they're not being maintained by us at OpenAI. We encourage authors and users to create new repositories for these environments.
Changed MultiDiscrete action space to range from [0, ..., n-1] rather than [a, ..., b-1].
No more render(close=True), use env-specific methods to close the rendering.
Removed scoreboard directory, since site doesn't exist anymore.
Moved gym/monitoring to gym/wrappers/monitoring
Add dtype to Space.
Not using python's built-in module anymore, using gym.logger
2018-01-24: All continuous control environments now use mujoco_py >= 1.50. Versions have been updated accordingly to -v2, e.g. HalfCheetah-v2. Performance should be similar (see https://github.com/openai/gym/pull/834) but there are likely some differences due to changes in MuJoCo.

Hello, I am facing the same issue as described above, do you know if there has been any solution? Upgrading gym did nothing for me!

Hello, I am facing the same issue as described above, do you know if there has been any solution? Upgrading gym did nothing for me!

Have you solved your problem?  I fix the issue by downgrade the version of gym.

I have the same problem as well

It may be a version-conflict problem. Upgrading the gym or Downgrading can solve it.

In my case, downgrading from 0.18 to 0.10 solved the problem.

It may be a version-conflict problem. Upgrading the gym or Downgrading can solve it.

In my case, downgrading from 0.18 to 0.10 solved the problem.

I am having this issue too


⚠ SkipTheTrailers has degraded performance

In aae4afc, SkipTheTrailers ($STT) experienced degraded performance:

HTTP code: 200
Response time: 1297 ms

Resolved: SkipTheTrailers performance has improved in e8dca41.


failed in building the recipe

I added the recipe without modification (except the uri of the package), I got the following error message
| DEBUG: Executing shell function do_compile
| NOTE: make -j 4
| Makefile:10: Makedefs: No such file or directory
| make: *** No rule to make target 'Makedefs'.  Stop.
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
It seemed ./configure should be run to create the Makedefs file. So I added "autotools" at the end the "inherit" statement to run the configure . Different error message appeared
| DEBUG: Python function autotools_aclocals finished
| DEBUG: Executing shell function do_compile
| NOTE: make -j 4
| make: *** No targets specified and no makefile found.  Stop.
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
I am new to Yocto，anytging will help.

OK, so the autotools stuff actually will break PAPPL - configure needs to be run for the build but not aclocal/automake/autoconf/etc. I will look into this but this was working for Kirkstone and other older Yocto releases...

OK, try the current repo - should be fixed now...

Thanks , Michael.
I have tried the new pappl recipe with the original cups recipe in poky layer.I got the following error:
ERROR: pappl-1.3.2-r0 do_package_qa: QA Issue: File /usr/lib/libpappl.so.1 in package pappl doesn't have GNU_HASH (didn't pass LDFLAGS?) [ldflags]
ERROR: pappl-1.3.2-r0 do_package_qa: Fatal QA errors were found, failing task.
ERROR: Logfile of failure stored in: /home/dean/var-fsl-yocto/build_xwayland/tmp/work/armv8a-fslc-linux/pappl/1.3.2-r0/temp/log.do_package_qa.58812
ERROR: Task (/home/dean/var-fsl-yocto/sources/meta-titan/recipes-printing/pappl/pappl_1.3.2.bb:do_package_qa) failed with exit code '1'
I was trying to fix it with the solution from Yocto  by adding  TARGET_CC_ARCH += "${LDFLAGS}" .Unfortunately, it was still not working.
I tried your cups recipe , but got the error ERROR: Nothing RPROVIDES 'cups-lib' which was required .

Thanks , Michael.
I have tried the new pappl recipe with the original cups recipe in poky layer.I got the following error: ERROR: pappl-1.3.2-r0 do_package_qa: QA Issue: File /usr/lib/libpappl.so.1 in package pappl doesn't have GNU_HASH (didn't pass LDFLAGS?) [ldflags] ERROR: pappl-1.3.2-r0 do_package_qa: Fatal QA errors were found, failing task. ERROR: Logfile of failure stored in: /home/dean/var-fsl-yocto/build_xwayland/tmp/work/armv8a-fslc-linux/pappl/1.3.2-r0/temp/log.do_package_qa.58812 ERROR: Task (/home/dean/var-fsl-yocto/sources/meta-titan/recipes-printing/pappl/pappl_1.3.2.bb:do_package_qa) failed with exit code '1'
I was trying to fix it with the solution from Yocto by adding TARGET_CC_ARCH += "${LDFLAGS}" .Unfortunately, it was still not working.
I tried your cups recipe , but got the error ERROR: Nothing RPROVIDES 'cups-lib' which was required .

I still stucked here , tried a lot even chat-gpt.

After several attemps, I got following error:
/home/dean/var-fsl-yocto_my/build-xwayland/tmp/work/armv8a-fslc-linux/pappl/1.3.2-r0/recipe-sysroot-native/usr/bin/aarch64-fslc-linux/../../libexec/aarch64-fslc-linux/gcc/aarch64-fslc-linux/11.3.0/ld: ../pappl/libpappl.a(system-accessors.o): in function _papplSystemSetHostNameNoLock': | /usr/src/debug/pappl/1.3.2-r0/pappl-1.3.2/pappl/system-accessors.c:1815: undefined reference to cupsSetServerCredentials'
| collect2: error: ld returned 1 exit status
| make[1]: *** [Makefile:73: testmainloop] Error 1
| make[1]: *** Waiting for unfinished jobs....
| make: *** [Makefile:28: all] Error 1
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
"cupsSetServerCredentials" is defined in cups.h , I think the cups-lib is not in the correct location so pappl cannot find it when building.


Progress of download file

Please add a progress for download file.
When I send file (upload file), I can see the progress for uploading file.

But, when I download file, I can't see the progress.
Also, please notify me when the downloading finished.
Thank you.

@JinZhuXing we have an internal ticket for this already: https://mattermost.atlassian.net/browse/MM-22239
Hoping to work on it soon :)


Ability to specify units for an expression.

Is your feature request related to a problem?
Here is the e-mail from Olivier Cessenat:
Dear VisIt developers,
When setting an expression, either interactively with the VisIt expression menu, or in the database using Silo's DBPutDefvars, I would like to be able to change the default unit.
For the moment VisIt guesses the unit from the first met variable in the expression.
Typically I would like to define the Poynting vector as "cross(E,H)" and specify it's unit as a power per square distance.
Thanks a lot - and I wish you a happy new year !
Olivier Cessenat.
Impact

[X] Medium - User productivity partially improved

So, we support units in ordinary Silo variables and those get handled in https://github.com/visit-dav/visit/blob/46fce7ae5e82cd2393bf9fc0d737f9c9f389b2e9/src/avt/DBAtts/MetaData/avtVarMetaData.xml#L9
as an opaque (e.g. non-interpreted) string. We could add the ability for a user to specify a units string for an expression in the expression GUI. On the Silo side, for DBPutDefvars we'd need to support string units there too.
I do wish we had more formal support for units akin to what we did in SAF. It would sure make for more sane treatment of and automated derivation of units


Feature/#37 singlemulti select component

Related issue
Closes #37
Description

Screenshots (before and after the change):

I will review this.

Please resolve conflicts with upstream branch.


Generated CLI should use generated interface

Current implementation has the CLI generating its own handlers.

Already done for the brownie


Is it possible to use wildcards in ConfigureRefresh?

Guys, is it possible to use wildcards (say *) when configuring the refresher in order to update all the settings values under a given parent? I want to update any setting value inside MySeetings when they change.
class MySeetings
{
     public string Setting1 { get; set; }
     public bool Setting2 { get; set; }
     ...
}

config.AddAzureAppConfiguration(options =>
{
       options.Connect(settings["ConnectionStrings:AppConfig"])
              .ConfigureRefresh(refresh => { refresh.Register("MySettings:*"); });
});

When I do that I get a 404 error. Is there another way to do that?

Only individual key-values can be configured for refresh. The closest way to get a similar behavior to what you are talking about is to use a "sentinel" key-value in the app configuration, and trigger a reload all of your configuration when that value changes.
options.Connect(settings["ConnectionStrings:AppConfig"])
              .ConfigureRefresh(refresh => { refresh.Register(key: "MySettings:Sentinel", refreshAll: true); });

It's a slightly different workflow because you would need to update the sentinel whenever you want the app to pick up some changes that you made in the app configuration.

There's a chance to have a feature like that in the future? I mean, given I have my configuration in Azure I don't want to update my code every time I add a new setting (in this example within  MySettings class) that way I can take advantage of the dynamic configuration avoiding to register a new setting to be refreshed.
Also, the scenario I have is an array inside of MySettings, so I need to refresh my configuration when I add an item into the array, so that array isn't fixed.

@vany0114 right now there is no plan for that.
Does the approach using refreshAll: true that I mentioned not work for your scenario? It does not require any code changes in your application, so I'm unsure what you meant by having to update code every time you add a new setting.
The flow would be like this

Deploy app with refresh.Register(key: "MySettings:Sentinel", refreshAll: true);
Perform updates to the array in app configuration
Update MySettings:Sentinel in app configuration
Observe app now uses the latest array values.

Sorry, I misunderstood you, I was left wondering about this:

It's a slightly different workflow because you would need to update the sentinel whenever you want the app to pick up some changes that you made in the app configuration.

I haven't tried it out yet but should work as you said. So the Sentinel might be of whatever type, right? it triggers the refresh even if the Sentinel value is the same right? as long as I update that guy.

Right, sentinel could be anything. Just touching that value in app configuration will have the desired affect. The app will see that it got updated and doesn't need to worry about the actual value.
Although, it will get the new value for the sentinel in case you use one who's value you do care about.

Got it, thanks to reply so fast!

@vany0114 right now there is no plan for that.
Does the approach using refreshAll: true that I mentioned not work for your scenario? It does not require any code changes in your application, so I'm unsure what you meant by having to update code every time you add a new setting.
The flow would be like this
1. Deploy app with `refresh.Register(key: "MySettings:Sentinel", refreshAll: true);`

2. Perform updates to the array in app configuration

3. Update `MySettings:Sentinel` in app configuration

4. Observe app now uses the latest array values.

This definitely works, but seems excessively onerous for consumers. Explaining how to set this up and use it to every single one of the teams I support isn't nearly as straightforward as a bespoke option.


Adding output modes to the field_colour

This is non-breaking, the default behavior is unchanged. I've added two extra modes to the color picker for returning strings and return indices instead of hex values.

Tested on chibi?

Yep, works in Chibi


Pull-from-upstream could provide the pkgdiff report within the generated PR

Description
Some packages need to be checked for API/ABI compatibility between releases, which could be done using the pkgdiff utility. It is called with 2 arguments: pkgdiff <old_rpm> <new_rpm> (so it needs to be done for every RPM generated by the package.
The report from this tool is stored in very nicely human-readable HTML pages, which could be stored in some location, and the links would be provided within the PR itself.
Benefit
Speeds up the rebase process for package maintainers, as they need to do it manually now.
Importance
Medium. It's not a problem to do it ourselves, but in that case, we need to generate the scratch-build, pkgdiff, and PR ourselves and then it doesn't make very sense to use Packit in that type of package.
Workaround

[X] There is an existing workaround that can be used until this feature is implemented.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

hi @ljavorsk ! I can definitely see that users could benefit from this, we have also similar issue #2195.
We would ideally first like to do research (packit/packit-service#2299) on how we could approach the validation more generally (we are also discussing the priorities for next quarter and this is one of the epics we are considering) and let you know once we come up with a plan.


(Low priority) Code docs 1

First part of a probable series of code documentation work that gets done as it annoys me and isn't otherwise part of actual work on the code.
Some of the changes seem odd from this bulk change view, so be sure to dig into the commit messages: lots of explanations in there.
BTW: this is a LOW PRIORITY PR.  Can even wait till after the start of the new year.  It was just time for me to put it in a PR.

Incorporating into PR99.


The terminal has disappeared and cannot leave it.

There were no associated mods with TerminalAPI.

Incorrect Template for bug report.


Hybrid mode causes unrecoverable system crash

please complete the following information:

OS version: Ubuntu 20.04
Editor: PyCharm
Editor version: 2022.2.3, Build #PY-222.4345.23
Programming language: Python 3
TabNine extension version: 0.7.26
Engine version: 4.4.158-60

Issue Details:
Recently installed TabNine, every attempt to use it in Hybrid mode leads to unrecoverable system crashes - No signal to monitor, all buttons/keys unresponsive including system power button. Power has to be manually cut at the PSU to shut down and restart.
Functionality appears normal in Cloud mode.
gz#18851

Hi @JTrentSmart
When you work in hybrid mode, we run the AI model locally.
That means TabNine will require a significant amount of cpu and RAM to operate.
On some systems, this may cause halts. It might be because there is a hardware issue that manifests when the system is busy, or because the system overheats.
If you can, please switch to hybrid mode, and monitor the running processes.
You would probably see a process named TabNine-deep-local consuming a lot of cpu.
You can also try to use the sensors utility to monitor cpu temperature.
Assuming that's the case, you can try to limit the amount of cpu that TabNine-deep-local process consumes.
You can use the cpulimit utility to limit cpu usage of TabNine-deep-local.
cpulimit --exe=TabNine-deep-local -l 200

I would really appreciate if you can report back to us - and let us know if it helped.


detox.init() ios hangs

Description
Running detox.init() using lerna (so runs in several packages) hangs about 50% of the time, causes mocha to reach timeout
Detox, Node, Device, Xcode and macOS Versions

Detox: 8.2.3
React Native: 0.51.0
Node: v8.9.4
Device: iPhone 7
Xcode: 9.4
macOS: HighSierra

Device and verbose Detox logs
[15:45:06][Step 7/7] before detox.init
[15:45:06][Step 7/7] detox info 15:45:06: server listening on localhost:50076...
[15:45:08][Step 7/7] detox info 2: Searching for device matching iPhone 7...
[15:45:11][Step 7/7] detox info 5: Booting device 044F9BD6-73D3-4496-A5CD-2932B4A6C4BC
[15:48:29][Step 7/7] detox ERR! 7: running "/usr/bin/xcrun simctl io 044F9BD6-73D3-4496-A5CD-2932B4A6C4BC screenshot "/dev/null"" returned 2
[15:48:29][Step 7/7] detox ERR! 7: stderr: An error was encountered processing the command (domain=SimulatorKit.SimDisplayScreenshotWriter.ScreenshotError, code=2):
[15:48:29][Step 7/7] Error creating the image
[15:48:29][Step 7/7]
[15:48:32][Step 7/7] detox info 8: Uninstalling com.asurion.anywhereexpert...
[15:48:34][Step 7/7] detox info 8: com.asurion.anywhereexpert uninstalled
[15:48:34][Step 7/7] detox info 9: Installing /Users/Soluto/BuildAgent/work/cdcd1423766ba686/mobile/ios/build/Build/Products/Release-iphonesimulator/Anywhere Expert.app...
[15:48:43][Step 7/7] detox info 9: /Users/Soluto/BuildAgent/work/cdcd1423766ba686/mobile/ios/build/Build/Products/Release-iphonesimulator/Anywhere Expert.app installed
[15:48:43][Step 7/7] detox info 10: Terminating com.asurion.anywhereexpert...
[15:48:44][Step 7/7] detox info 10: com.asurion.anywhereexpert terminated
[15:48:44][Step 7/7] detox info 11: Launching com.asurion.anywhereexpert...
[15:48:50][Step 7/7] detox info 11: com.asurion.anywhereexpert launched. The stdout and stderr logs were recreated, you can watch them with:
[15:48:50][Step 7/7]         tail -F /Users/Soluto/Library/Developer/CoreSimulator/Devices/044F9BD6-73D3-4496-A5CD-2932B4A6C4BC/data/tmp/detox.last_launch_app_log.{out,err}
[15:50:06][Step 7/7] Timeout of 300000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
[15:50:06][Step 7/7] Error: Timeout of 300000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.
[15:50:07][Step 7/7] child_process.js:644
[15:50:07][Step 7/7]     throw err;
[15:50:07][Step 7/7]     ^
[15:50:07][Step 7/7]
[15:50:07][Step 7/7] Error: Command failed: node_modules/.bin/mocha __tests__/detox/build --opts __tests__/detox/mocha.teamcity.opts --configuration ios.release     --grep :android: --invert     --artifacts-location "artifacts/ios.release.2018-08-30 12:44:54Z"
[15:50:07][Step 7/7]     at checkExecSyncError (child_process.js:601:13)
[15:50:07][Step 7/7]     at Object.execSync (child_process.js:641:13)
[15:50:07][Step 7/7]     at runMocha (/Users/Soluto/BuildAgent/work/cdcd1423766ba686/mobile/node_modules/detox/local-cli/detox-test.js:123:6)
[15:50:07][Step 7/7]     at run (/Users/Soluto/BuildAgent/work/cdcd1423766ba686/mobile/node_modules/detox/local-cli/detox-test.js:78:7)
[15:50:07][Step 7/7]     at Object.<anonymous> (/Users/Soluto/BuildAgent/work/cdcd1423766ba686/mobile/node_modules/detox/local-cli/detox-test.js:191:1)
[15:50:07][Step 7/7]     at Module._compile (module.js:652:30)
[15:50:07][Step 7/7]     at Object.Module._extensions..js (module.js:663:10)
[15:50:07][Step 7/7]     at Module.load (module.js:565:32)
[15:50:07][Step 7/7]     at tryModuleLoad (module.js:505:12)
[15:50:07][Step 7/7]     at Function.Module._load (module.js:497:3)
[15:50:07][Step 7/7] error Command failed with exit code 1.

I don't believe that these logs are from Detox 8.2.3, it is 7.x

are you sure?
Could it be prefering the global npm detox over the local one?

Not likely. Why do you install detox globally?

Ok in the yarn.lock I see this:
<EMAIL_ADDRESS>  version "7.0.0"
  resolved "https://registry.yarnpkg.com/detox-server/-/detox-server-7.0.0.tgz#82e571577aa5f867c5ed1fd5b8785e1a9e033ef0"
  dependencies:
    lodash "^4.13.1"
    npmlog "^4.0.2"
    ws "^1.1.0"

<EMAIL_ADDRESS>  version "8.0.0"
  resolved "https://registry.yarnpkg.com/detox/-/detox-8.0.0.tgz#6322d0ea9c27cbe8fd9f22524b8fb004ba2a8a95"
  dependencies:
    child-process-promise "^2.2.0"
    commander "^2.15.1"
    detox-server "^7.0.0"
    fs-extra "^4.0.2"
    get-port "^2.1.0"
    ini "^1.3.4"
    lodash "^4.14.1"
    minimist "^1.2.0"
    npmlog "^4.0.2"
    proper-lockfile "^3.0.2"
    shell-utils "^1.0.9"
    tail "^1.2.3"
    telnet-client "0.15.3"
    tempfile "^2.0.0"
    ws "^1.1.1"

whats detox-server?

detox-server is not in use now.

ok, so the logs are from detox 8.0.0 from what it seems
should I upgrade to 8.2.3?

I don't think it is going to solve your problem, but anyway it won't hurt.

Run your test with --debug-synchronization 1000 and paste the log output here. Thanks


bibata-cursors: 2.0.5 -> 2.0.5

Description of changes
https://github.com/ful1e5/Bibata_Cursor/compare/v2.0.3...v2.0.5
Things done

Built on platform(s)

[x] 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.05 Release Notes (or backporting 23.05 and 23.11 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.

#261532


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-06-30)
Bug Fixes

when k closes the window it behaves itself (#4) (fbb0fc7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/benlubas/wrapping-paper.nvim/releases/tag/v1.0.0 :sunflower:


Restricted job creation to Portal and above

What does this PR (Pull Request) do?

[x] This PR fixes #3116

Raises the required permissions level to submit new jobs to "Portal", regardless of tenancy and/or DS-to-user assignment(s).
Which Traffic Control components are affected by this PR?

Documentation
Traffic Ops

What is the best way to verify this PR?
Build TO, try to create a job as a user with "read-only" Role, both with and without DS assigned to you, both with and without the proper tenant, and verify that the operation is forbidden. Then repeat with Portal Role and verify that under the right conditions, the operation is allowed.
If this is a bug fix, what versions of Traffic Ops are affected?

master (@ 95366ccaf333332990f1b4fe9d10ffd6f603d4ed)
3.0.1

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3997/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4037/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4040/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4043/
Test PASSed.


Unable to Abandon First Drilled Layer

In my current Rimworld save, I have a single Deeprim layer that I am seemingly unable to delete. When selecting the 10m layer via the "change target" button, I get this error: https://gist.github.com/HugsLibRecordkeeper2/f5502e21c4f100644a5aa1bcc7c80dcb#file-output_log-txt-L2519-L2531
And when trying to delete the selected layer, I get another error: https://gist.github.com/HugsLibRecordkeeper2/f5502e21c4f100644a5aa1bcc7c80dcb#file-output_log-txt-L2533-L2544
Problem is, I am unable to reproduce this error with a fresh colony, modded or not... here are some steps I've tried:

Start a new colony with only deeprim loaded, build a drill, generate a later, delete the layer.
Generate a layer, wait a bit, save and reload, delete the layer.
Load the game with all the mods in the above log, perform the same 2 previous steps.

Note that in my problem colony, I am able to generate new DeepRim layers and subsequently select and delete them without any issue.

As a temporary workaround, I can destroy the 10m layer with the "Destroy Map" debug command. It still shows up as a menu entry in mining shafts, but the map is seemingly unloaded from the game.


heap: BinaryHeap benchmarks

Codecov Report

Merging #65 into master will decrease coverage by 0.88%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   72.82%   71.94%   -0.89%
==========================================
  Files          21       21
  Lines         898      909      +11
==========================================
  Hits          654      654
- Misses        220      231      +11
  Partials       24       24

Impacted Files
Coverage Δ

heap/binaryheap.go
78.43% <0%> (-21.57%)
:arrow_down:

Continue to review full report at Codecov.

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


Issue #SB-15180 added constants and template for forgot password

Codecov Report

Merging #771 into release-2.6.0 will increase coverage by 1.56%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##             release-2.6.0     #771      +/-   ##
===================================================
+ Coverage            37.03%   38.59%   +1.56%
- Complexity             849      979     +130
===================================================
  Files                  124      149      +25
  Lines                 9322    10074     +752
  Branches              1060     1118      +58
===================================================
+ Hits                  3452     3888     +436
- Misses                5509     5777     +268
- Partials               361      409      +48

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/sunbird/common/models/util/JsonKey.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...va/org/sunbird/notification/utils/FCMResponse.java
0% <0%> (ø)
0% <0%> (?)

...notification/fcm/provider/NotificationFactory.java
46.15% <0%> (ø)
3% <0%> (?)

...a/org/sunbird/notification/beans/EmailRequest.java
43.75% <0%> (ø)
6% <0%> (?)

...ava/org/sunbird/notification/beans/OTPRequest.java
46.87% <0%> (ø)
8% <0%> (?)

...java/org/sunbird/notification/beans/Constants.java
0% <0%> (ø)
0% <0%> (?)

...ation/email/service/impl/SmtpEMailServiceImpl.java
28.57% <0%> (ø)
3% <0%> (?)

...sunbird/notification/email/GMailAuthenticator.java
100% <0%> (ø)
2% <0%> (?)

...nbird/notification/utils/NotificationConstant.java
0% <0%> (ø)
0% <0%> (?)

...ication/sms/providerimpl/Msg91SmsProviderImpl.java
66.04% <0%> (ø)
39% <0%> (?)

... and 16 more

Continue to review full report at Codecov.

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


inconsistencies between consistent-tree versions about column contents

Different CTrees versions output different columns (after column 34 from my read). Might require a more generic approach matching the fields to the values.

Probably best to switch to using parse_ctree.sh to avoid any mis-reading of columns


[Emojis] :shrug: ¯_(ツ)_/¯

The ingame :shrug: emoji is missing an arm.

log on to the server
type :shrug:
look at output
missing an arm:

Expectation:
¯\(ツ)/¯
All servers

Should be fixed for next restart. Thanks for the report!


Extract BIDS bold task names as scan ids in fMRIQC

Suggested to address #21

Would this give the same label to multiple runs?
On Jan 20, 2016 8:25 PM, "William Triplett"<EMAIL_ADDRESS>wrote:

Suggested to address #21 https://github.com/poldracklab/mriqc/issues/21
You can view, comment on, or merge this pull request online at:
https://github.com/poldracklab/mriqc/pull/22
Commit Summary

Extract BIDS bold task names as scan ids in fMRIQC

File Changes

M mriqc/utils.py
https://github.com/poldracklab/mriqc/pull/22/files#diff-0 (11)

Patch Links:

https://github.com/poldracklab/mriqc/pull/22.patch
https://github.com/poldracklab/mriqc/pull/22.diff

—
Reply to this email directly or view it on GitHub
https://github.com/poldracklab/mriqc/pull/22.

@chrisfilo: I don't think so, the code extracts the run number portion of the filename if its present, but if not it just grabs the task name.

I see. So 'taskinfo' is matching run number (plus 'run-' prefix) not the
task name as the variable name suggest?
On Jan 20, 2016 8:54 PM, "William Triplett"<EMAIL_ADDRESS>wrote:

@chrisfilo https://github.com/chrisfilo: I don't think so, the code
extracts the run number portion of the filename if its present, but if not
it just grabs the task name.
—
Reply to this email directly or view it on GitHub
https://github.com/poldracklab/mriqc/pull/22#issuecomment-173450711.

@chrisfilo good point. Ideally if its a BIDS dataset we could break out the other qualifiers (acq-???, rec-???) into separate columns for completeness. The lesser used qualifiers would be empty more often than not, but at some point this is going to come up. Maybe #21 could be merged into issue #8 regarding multi-session datasets in favor of more comprehensive BIDS support in general?
In the shorter term having task name included in the CSV file and plots is critical for datasets with multiple tasks.
@oesteban I'm happy to keep working on this and issue #8 -- let me know what you'd prefer.

Once tests pass, I'll merge it to master. Good job, thanks!


all: use github.com/moov-io/base

github.com/moov-io/base/http (and others) are shared libraries intended to be used by all apps. Let's switch over paygate to use them.

Codecov Report

Merging #38 into master will decrease coverage by 0.68%.
The diff coverage is 17.27%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage    38.7%   38.01%   -0.69%
==========================================
  Files          15       14       -1
  Lines        1739     1710      -29
==========================================
- Hits          673      650      -23
+ Misses        908      904       -4
+ Partials      158      156       -2

Impacted Files
Coverage Δ

depositories.go
40.81% <0%> (ø)
:arrow_up:

main.go
0% <0%> (ø)
:arrow_up:

customers.go
25.41% <0%> (ø)
:arrow_up:

gateways.go
38.23% <11.11%> (ø)
:arrow_up:

events.go
41.75% <16.66%> (ø)
:arrow_up:

transfers.go
34.01% <25%> (ø)
:arrow_up:

microDeposits.go
48.91% <33.33%> (ø)
:arrow_up:

originators.go
35.71% <7.69%> (ø)
:arrow_up:

http.go
78.43% <87.5%> (+0.17%)
:arrow_up:

Continue to review full report at Codecov.

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


Update android bindings

Android bindings have been updated to mostly use 4.33 across the board, except in a few cases where 4.28 is used as we don’t have all the required dependencies for newer versions.

@Redth, I am curious how you avoided the compile error for AccountKit:
OnSpinnerEventsListener is not public in CountryCodeSpinner; cannot be accessed from outside package
I tried making a sample accountkit application using the bindings from this PR and I get the above compile error. Perhaps my implementation is incorrect, am wondering if you ran into this issue with these bindings?


Rename old service name

Visual Studio Team Services -> Azure DevOps

@changeworld : Thanks for your contribution! The author, @serinakaye, has been notified to review your proposed change.


3541 - Change validation for quarterly projections in WP

Description
Changes validation for quarterly projection "number" fields in the WP so that users can enter whole numbers OR percentages.
Related ticket(s)

CMDCT-3541

How to test
**Test link: **https://d3ozpalbymoth5.cloudfront.net/
User with plans<EMAIL_ADDRESS>/ PR / Puerto Rico
Work plan:

Create a work plan and go the the State or Territory-Specific Initiatives section.
Add a new initiative and in the Evaluation Plan section, click "Add objective."
In the Add Objective modal, select Yes for "Does the performance measure include quantitative targets?"

The quarterly objectives fields should allow user to enter any text, not just numbers. (e.g., "50%" should be a valid input)
The validation for required should still work.
There should be no masking of input.
The styling of the input should look the same as other number fields (slightly less wide than normal text inputs).

SAR:

Create a SAR associated with the previous work plan
In the State or Territory-Specific Initiatives section, open a specific initiative dashboard, and select Edit for Objective Progress (be sure to pick one where quarterly projections were entered in the WP).
Click "Report objective progress" to open the modal.

The "Actual value" input fields should follow the same validation rules (styled like number inputs, but validated like text)
The "Target value" fields should have the values formatted the same way as they were when entered in the WP.

Regression test:
Other numerical inputs should behave the same as expected:

Funding source input fields are still masked as currency and only allow number inputs
Transition Benchmark projections should only allow whole integers or "N/A"

Pre-review checklist

[x] I have added thorough tests, if necessary
[ ] I have updated relevant documentation, if necessary
[x] I have performed a self-review of my code
[x] I have manually tested this PR in the deployed cloud environment

Pre-merge checklist

Review

[ ] Design: This work has been reviewed and approved by design, if necessary
[ ] Product: This work has been reviewed and approved by product owner, if necessary

Security
If either of the following are true, notify the team's ISSO (Information System Security Officer).

[ ] These changes are significant enough to require an update to the SIA.
[ ] These changes are significant enough to require a penetration test.

convert to a different template: test → val | val → prod

Have the same worry as Ben. Would definitely prefer to see a new validation type with its own tests that check for compliance for this rather than moving this to having a text validation that would allow a lot more than just whole numbers or percentages.

This was a specific request from product to move to a text validation for this field, in order to avoid the extra effort to think through specific validations at this time and still meet the request from our stakeholders. Is there a concern with the technical aspects of having this field be a simple text field? Are there some dependencies with the API that might cause problems?

(Written by Ben) For posterity: We three just had a conversation about this ticket, and Nick and I approved the PR as-is (effectively removing validation from these fields). Ideally we will be able to follow up after launch to suss out more precise business requirements and implement them, and hopefully we will be able to do so before too much crazy data finds its way into the DB :slightly_smiling_face:


I can't build/compile and unable to play the Engine.

Alright, I've been struggling with engine build/compile for about two months.
I've tried every version of psych engine and always ended up in the same situation expect the error logs are always different.
I have tried and installed whatever the source code and the github page mentioned and even used the "setup-windows.bat" and "setup-msvc-win.bat".
I'll put the screenshots from the latest "Psych Engine 0.7.3" after running it, and inside the game.
Second Image translated to English, shortly: "Psych Engine is not compatible for Windows or includes an error. Try reinstalling the application or contact to the developer."

Command Prompt/Terminal logs (if existing)
BUILDING GAME
C:/HaxeToolkit/haxe/lib/flixel-ui/2,5,0/flixel/addons/ui/FlxUICursor.hx:564: characters 9-11 : Warning : Potential typo detected (expected similar values are flixel.addons.ui.SortMethod.ID)
C:\Users\HP\Desktop\FNF-JohnEngine-v0.8.0\export\32bit\windows\obj>setlocal enabledelayedexpansion
Warning: Could not find environment variables for Visual Studio
Missing HXCPP_VARS
Error: Could not automatically setup MSVC

done.
Press any key to continue . . .

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Download normal MSVC

Tried that 5 times, still same error.


Search File Pattern Fail

Issue Type: Bug
If I search for "onclick" without the quote and without anything in the "files to include" or "files to exclude" I am seeing a result in some of my src/ folder inside some of my .tsx file.
However, if I add in the "files to includes" any of these possibilities, I have no result:
src/**/*.tsx
/src/**/*.tsx
**/*.tsx
*.tsx

I am expecting to see all the tsx file with "onclick"
VS Code version: Code - Insiders 1.24.0-insider (a521c01d56ec7f07bea2471880f28a77f37983af, 2018-05-21T05:11:15.297Z)
OS version: Darwin x64 16.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2900)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (0.02GB free)

Process Argv
/Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron -psn_0_327760

Screen Reader
no

VM
0%

Extensions (17)

Extension
Author (truncated)
Version

line-counter
dol
0.5.2

gitlens
eam
8.3.2

EditorConfig
Edi
0.12.2

tslint
eg2
1.0.29

vscode-codemetrics
kis
1.16.0

python
ms-
2018.4.0

vscode-jest
Ort
2.8.0

vscode-subword-navigation
ow
1.2.0

vscode-docker
Pet
0.0.27

java
red
0.25.0

vscode-java-debug
vsc
0.9.0

vscode-java-pack
vsc
0.3.0

vscode-java-test
vsc
0.6.1

vscode-maven
vsc
0.8.0

vscode-wakatime
Wak
1.2.2

vscode-todo-highlight
way
0.5.12

vscode-import-cost
wix
2.6.2

(1 theme extensions excluded)

I fixed a couple bugs involving include patterns in Insiders. Can you try again with tomorrow's build? #50175 and #50311.

It's working now. Thank you very much for the fast response.


fix: send OK message for ephemeral events

Hi, according to last changes in NIP01, the OK message MUST be sent in response to every EVENT message:  https://github.com/nostr-protocol/nips/blob/4b9f13d983245e4dd166f102308afc28b8bb1603/01.md?plain=1#L153

Thanks for noticing and fixing this!  Merged.


Bit and byte reverse

three functions:
bit_swap swap group of bits
byte_reverse reverse  byte order
bit reverse reverse bit order
bit_reverse is useful for fft and can use byte_reverse and bit_swap

Are you sure it's not byte_swap? Is it little - big endian thing for bytes?

Please then in the comment/ doc say byte_swap for searching ( i didn't look - maybe you already do)

The doc says:
//!   @note eve::byte_reverse is the functional equivalent to std::byte_swap.
//!   However, we decided to name it byte_reverse in order to keep a proper naming
//!   scheme aligned with the eve::bit_swap/eve::bit_reverse functions.


KALM [BUG] AcceleratorKit Bugs

DataElement and ContactDataElements fix.
Added dynamique library pick based on the profile
TODO: how to pick the right context. How to switch to the right library for DataElements. Added tests

Description : PR to fix acceleratorKit bugs. CQL files generation if mostly fixed. We need to define the proper factor to pick the context and the library.
DataElement and ContactDataElements fix.
Added dynamique library pick based on the profile
TODO: how to pick the right context. How to switch to the right library for DataElements. Added tests

Github Issue:  https://github.com/alphora/KALM/issues/62 https://github.com/alphora/KALM/issues/64 https://github.com/alphora/KALM/issues/21 https://github.com/alphora/KALM/issues/24
[ x] I've read the contribution guidelines
[ x] Code compiles without errors
[ x] Tests are created / updated
[ ] Documentation is created / updated

By creating this PR you acknowledge that your contribution will be licensed under Apache 2.0

@rob-reynolds - This pattern is called "try-with-resources" in Java:
https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html


[ttx] Could the sbix table support ?

I'm not trying to export the images from sbix font, but want to use .ttx file as a source file to build sbix font - You know that for build CBDT font, this is ok, but since <extfileimagedata> is currently not available for sbix table, this is not feasible.
Are there any plans to make it supportable, or any hacks?

thanks, it'd be nice to have that, but there's no plans at the moment. If you like to give it a go you're welcome.
you may also want to try nanoemoji which has a --color_format=sbix to build a sbix color font from a set of .svg input files.
It also comes with a separate maximum_color script which you can use to add a sbix table to an existing vector color font (either COLRv1 or OT-SVG).


unable to select exact image module version for deployment (on SS 2.1.7)

Doesn't apply anymore.
Works on the latest version.


'pool_default_pg_num' is undefined for 'client' role

ceph-ansible: 0f3ae4c
ansible: <IP_ADDRESS>
If client role is assigned to a host with default group vars, this error pops up:
TASK [ceph-client : create pools] **********************************************
task path: /root/tfs/OpenStack2Configs/roles/ceph-client/tasks/create_users_keys.yml:2^[[0m
fatal: [n1]: FAILED! => {"failed": true, "msg": "[{u'name': u'test', u'pgs': u'{{ pool_default_pg_
num }}'}, {u'name': u'test2', u'pgs': u'{{ pool_default_pg_num }}'}]: 'pool_default_pg_num' is undefined"}
fatal: [n2]: FAILED! => {"failed": true, "msg": "[{u'name': u'test', u'pgs': u'{{ pool_default_pg_
num }}'}, {u'name': u'test2', u'pgs': u'{{ pool_default_pg_num }}'}]: 'pool_default_pg_num' is undefined"}
fatal: [n3]: FAILED! => {"failed": true, "msg": "[{u'name': u'test', u'pgs': u'{{ pool_default_pg_
num }}'}, {u'name': u'test2', u'pgs': u'{{ pool_default_pg_num }}'}]: 'pool_default_pg_num' is undefined"}
fatal: [n4]: FAILED! => {"failed": true, "msg": "[{u'name': u'test', u'pgs': u'{{ pool_default_pg_
num }}'}, {u'name': u'test2', u'pgs': u'{{ pool_default_pg_num }}'}]: 'pool_default_pg_num' is undefined"}

pool_default_pg_num is defined in mon role variables only. For my case, this can be workarounded by:
diff --git a/roles/ceph-client/tasks/main.yml b/roles/ceph-client/tasks/main.yml
index c2b363f..32f5743 100644
--- a/roles/ceph-client/tasks/main.yml
+++ b/roles/ceph-client/tasks/main.yml
@@ -2,3 +2,4 @@
 - include: pre_requisite.yml
 - include: create_users_keys.yml
   when: user_config
+  static: False

But, of course, the main issue is not solved and might annoy newcomers if they would try to add test pools as well. Should we move this pool_default_pg_num to all.yml or calculate pgs for test/test2 pools using http://ceph.com/pgcalc/ formula in order to avoid HEALTH_WARN messages (since these pools are not for production purposes anyway)?

This is actually a bit annoying to manage this value now since we removed a lot from the template.
We need to change this logic and use something like this condition:   when: ceph_conf_overrides.global.osd_pool_default_pg_num is defined. I'll work on a patch today.

@eduardegorov please have a look at https://github.com/ceph/ceph-ansible/pull/1146 and test :)

ceph-ansible: 6043b53
ansible: <IP_ADDRESS>
I initially had the same issue as @eduardegorov - after testing pull request #1146, I'm getting two new errors, both apparently because I have a default all.yml:
# ceph_conf_overrides: {}
First Issue
TASK [ceph-mon : set_fact] *****************************************************
fatal: [mon0]: FAILED! => {"failed": true, "msg": "The conditional check 'ceph_conf_overrides.global.osd_pool_default_pg_num is not defined' failed. The error was: error while evaluating conditional (ceph_conf_overrides.global.osd_pool_default_pg_num is not defined): 'dict object' has no attribute 'global'\n\nThe error appears to have been in '/etc/vagrant/ceph-ansible/roles/ceph-mon/tasks/ceph_keys.yml': line 34, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- set_fact:\n  ^ here\n"}

Line 38 in roles/ceph-mon/tasks/ceph_keys.yml was:
- ceph_conf_overrides.global.osd_pool_default_pg_num is not defined
Changing line 38 to the below let me bypass the problem:
- ceph_conf_overrides.global is not defined or ceph_conf_overrides.global.osd_pool_default_pg_num is not defined
Second Issue
TASK [ceph-client : create pools] **********************************************
fatal: [client0]: FAILED! => {"failed": true, "msg": "[{u'name': u'test', u'pgs': u'{{ ceph_conf_overrides.global.osd_pool_default_pg_num }}'}, {u'name': u'test2', u'pgs': u'{{ ceph_conf_overrides.global.osd_pool_default_pg_num }}'}]: 'dict object' has no attribute 'global'"}

I can't figure out why roles/ceph-client/tasks/create_users_keys.yml runs when ceph_conf_overrides.global.osd_pool_default_pg_num is not defined (seems contrary to the when check in roles/ceph-client/tasks/main.yml).
However, when defining ceph_conf_overrides.global.osd_pool_default_pg_num in group_vars/all, this PR runs without errors:
ceph_conf_overrides:
  global:
    osd_pool_default_pg_num: 8

@keepwatch yes I was overconfident with #1149, I need to rework this.

I'm still seeing the first issue, even with the commit in #1146, when running master.

@verdurin use master and declare ceph_conf_overrides.global.osd_pool_default_pg_num, everything should work.


Implemented traits on data structures do not get imported unless you import everything from the module using the glob operator

Summary
If you take a look at the test in this pull request you'll notice that I must import identity::* instead of identity::Identity.
The reason for this is that the trait does not exist unless you import everything.
This has happened in multiple other instances before.
That is far from desirable and correct behaviour.
Resolution: When importing a type such as Identity all of the traits that are implemented for it should be imported as well.
cc @FuelLabs/sway-compiler

Related: https://github.com/FuelLabs/sway/issues/1387

We do that for structs though 🤔
I have a quick resolution here: https://github.com/FuelLabs/sway/pull/2034

To follow the rust spec, you must import a trait in order for its implementations to be in scope. Just importing a type doesn't import every known trait implementation for it. So, there should be two ways to import a trait implementation:

Import the trait itself
Star-import the file which contains the trait

I think we only test 2, so we should introduce a test for 1 and make sure it works.

That seems like a design flaw.
Why would I have to import everything from a file or have to know every trait implementation for the type that I want?

Edit 2: With https://github.com/FuelLabs/sway/pull/2034, @Braqzen's test builds with use std::identity::Identity. @sezna are you saying this is now against the spec? What do you recommend we do?


[release/3.1][release/5.0] Ilasm and Ildasm not in correct location on Alpine Linux build

Issue
Various steps in the build process looks for ilasm in ildasm in $builddir/src/dotnet-3.1/Tools/source-built/coreclr-tools/x64 while they are  built to $builddir/src/dotnet-3.1/Tools/source-built/coreclr-tools on Alpine Linux 3.15.
To reproduce
Build on Alpine Linux 3.15 using APKBUILD available at: https://gitlab.alpinelinux.org/ayakael/aports/-/tree/testing/dotnet/testing/dotnet31 without build_coreclr-tools-path.patch
To fix
I constructed the following patch:
From<PHONE_NUMBER>b01b60f1c1f2d1c2a77e07ae814647 Mon Sep 17 00:00:00 2001
From: "build@apk-groulx" <EMAIL_ADDRESS>Date: Sun, 5 Dec 2021 15:46:08 +0000
Subject: [PATCH 1/1] Ildasm and ilasm paths changed to where they actually
 apper

---
 Directory.Build.props       | 4 ++--
 repos/coreclr-portable.proj | 2 +-
 repos/coreclr.common.props  | 2 +-
 repos/coreclr.proj          | 4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Directory.Build.props b/Directory.Build.props
index f6a6f54..8247c3e 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -133,8 +133,8 @@
     <AspNetRazorBuildServerLogDir>$(BaseOutputPath)aspnet-debug</AspNetRazorBuildServerLogDir>
     <AspNetRazorBuildServerLogFile>$(AspNetRazorBuildServerLogDir)razor-build-server.log</AspNetRazorBuildServerLogFile>
     <IlasmPath Condition="'$(OfflineBuild)' != 'true'">invalid: ILAsm is not expected to be needed in the online build</IlasmPath>
-    <IlasmPath Condition="'$(OfflineBuild)' == 'true'">$(PrebuiltSourceBuiltPackagesPath)coreclr-tools/$(BuildArchitecture)/ilasm</IlasmPath>
-    <IldasmPath Condition="'$(OfflineBuild)' != 'true'">$(ToolPackageExtractDir)coreclr-tools/$(BuildArchitecture)/ildasm</IldasmPath>
+    <IlasmPath Condition="'$(OfflineBuild)' == 'true'">$(PrebuiltSourceBuiltPackagesPath)coreclr-tools/ilasm</IlasmPath>
+    <IldasmPath Condition="'$(OfflineBuild)' != 'true'">$(ToolPackageExtractDir)coreclr-tools/ildasm</IldasmPath>
     <IldasmPath Condition="'$(OfflineBuild)' == 'true'">invalid: ILDasm is not expected to be needed in the offline build</IldasmPath>
     <!-- Dir where git info is generated during online builds. -->
     <GitInfoOutputDir>$(BaseOutputPath)git-info/</GitInfoOutputDir>
diff --git a/repos/coreclr-portable.proj b/repos/coreclr-portable.proj
index 78adc14..378547e 100644
--- a/repos/coreclr-portable.proj
+++ b/repos/coreclr-portable.proj
@@ -19,7 +19,7 @@
           Inputs="$(MSBuildProjectFullPath)"
           Outputs="$(RepoCompletedSemaphorePath)CopyTools.complete">
     <PropertyGroup>
-      <CoreClrDestinationDir>$(ToolPackageExtractDir)coreclr-tools/$(BuildArchitecture)</CoreClrDestinationDir>
+      <CoreClrDestinationDir>$(ToolPackageExtractDir)coreclr-tools</CoreClrDestinationDir>
       <AssemblyExtension Condition="'$(TargetOS)' == 'Linux'">so</AssemblyExtension>
       <AssemblyExtension Condition="'$(TargetOS)' == 'FreeBSD'">so</AssemblyExtension>
       <AssemblyExtension Condition="'$(TargetOS)' == 'OSX'">dylib</AssemblyExtension>
diff --git a/repos/coreclr.proj b/repos/coreclr.proj
index 7e67b28..037c68a 100644
--- a/repos/coreclr.proj
+++ b/repos/coreclr.proj
@@ -15,11 +15,11 @@
   <!-- OSX and portable only build this and not coreclr-portable, so we need to copy its tools here -->
   <Target Name="CopyTools"
           AfterTargets="Package"
-          Condition="'$(TargetOS)' == 'OSX' or '$(PortableBuild)' == 'true'"
+          Condition="'$(TargetOS)' == 'OSX' or '$(PortableBuild)' == 'true' or '$(TargetRid)' == 'linux-musl-x64'"
           Inputs="$(MSBuildProjectFullPath)"
           Outputs="$(RepoCompletedSemaphorePath)CopyTools.complete">
     <PropertyGroup>
-      <CoreClrDestinationDir>$(ToolPackageExtractDir)coreclr-tools/$(BuildArchitecture)</CoreClrDestinationDir>
+      <CoreClrDestinationDir>$(ToolPackageExtractDir)coreclr-tools</CoreClrDestinationDir>
       <AssemblyExtension Condition="'$(TargetOS)' == 'Linux'">so</AssemblyExtension>
       <AssemblyExtension Condition="'$(TargetOS)' == 'FreeBSD'">so</AssemblyExtension>
       <AssemblyExtension Condition="'$(TargetOS)' == 'OSX'">dylib</AssemblyExtension>
--
2.34.0

This would not work for other platforms, as I assume other platforms do actually look for ilasm and ildasm in currect location.
Note
I would've opened this issue in dotnet/coreclr, but unfortunately that space has been closed. Should I send this to dotnet/runtime?
Made as part of Alpine Linux dotnet31 / dotnet5 packaging project, see dotnet/source-build#2695

@omajid I see that ya'll encountered this bug in dotnet31 on Fedora. Any idea what causes this issue?

I am almost positive I know what this is, but I can find any references/details in my notes or here on github.
Here's what I remember:

At some point source-build 3.1 broke going from a N-1 build of 3.1 to the N build of 3.1. The N-1 build had coreclr-tools placed in one location, and the N build was looking at it in another location
In source-build, this was "fixed" by updating the artifacts for N-1 build in-place manually
For Fedora, I patched the build to look for coreclr-tools in the old location

Now that I am thinking about it, I am not sure this is needed anymore:

For recent builds they have a consistent location for N-1 and N coreclr-tools
The recent builds put coreclr-tools in both coreclr-tools/$(BuildArchitecture)" and coreclr-tools/

I wonder if this patch is really needed anymore...

I remember an important detail: ilasm was present in both locations, except that the one in  coreclr-tools/$(BuildArchitecture) gave me a segmentation fault, no doubt because it's pulled from linux-x64 prebuilt Private.SourceBuilt.Artifacts. The version available in coreclr-tools/ worked with linux-musl-x64, which makes sense given the source-build "fix" that updates the artifacts in-place. No doubt this becomes a non-issue with local Private.SourceBuilt.Artifacts, which means my patch is only necessary when building with dotnet31-stage0-artifacts package. I'll test, see if some patches cease to be necessary when building with dotnet31-bootstrap-artifacts (bootstrap being the musl built version)

Confirming that with proper artifacts dotnet-3.1 does not need this patch anymore, and dotnet 5.0 has no need for it at all.