Access Harbor Api occur Unauthorized

Harbor Version v0.4.5
Unexpected code Response{protocol=http/1.1, code=401, message=Unauthorized, url=http://<IP_ADDRESS>/api/users}
Is there any login api?

what's your request? in code or using swagger?
you can setting basic auth for your request.
or login on UI and then using the curl visit API.

@yhua123
I request Harbor Api in code!
How can I set basic auth for my request in code?

@yhua123  Thanks a lot  ^_^
I figured out !
	OkHttpClient client = new OkHttpClient();
	String credential = Credentials.basic("admin", "Harbor12345");
	Request request = new Request.Builder()
			.url("http://<IP_ADDRESS>/api/users")
			.header("Authorization", credential)
			.build();
	Response response = client.newCall(request).execute();

our API UT also did that;
src/ui/api/harborapi_test.go

Thanks guys


42702 order by "Id" is ambiguous

Version: Npgsql.EntityFrameworkCore.PostgreSQL: 2.1.2
Having 1..n relationship and fetching all parent entities along with child entities generates wrong query.
Npgsql.PostgresException (0x80004005): 42702: ORDER BY "Id" is ambiguous

Failed executing DbCommand (7ms)
SELECT x."EntityId" AS "Id", x."Id"
FROM "Table" AS x
WHERE x."Code" = @__ValueOrDefault_0
ORDER BY x."FromUtc", x."ToUtc", "Id", x."Id"
LIMIT @__p_2 OFFSET @__p_1 42702: ORDER BY "Id" is ambiguous

Can you post the code that generated this query?

I'm going to close this for inactivity. Feel free to keep posting if you have additional questions or information for us.
With only limited knowledge of Automapper, this looks related to some ambiguity between foreign key conventions in EF Core and Automapper.


error in RSEM: can't find resource file

Running RENEE from command line:
./renee run --input /data/sevillas2/v2.5.11/.tests/*.gz --output /data/sevillas2/output_2.5.11/ --genome hg38_30 --mode slurm

Ran into the following error:
Cannot open /data/CCBR_Pipeliner/Pipelines/RENEE/resources/hg38/30/rsemref/hg38.grp! It may not exist.
[Fri Jan  5 19:11:57 2024]
Error in rule rsem:
    jobid: 0
    input: /data/sevillas2/output_2.5.11/bams/KO_S4.p2.Aligned.toTranscriptome.out.bam, /data/sevillas2/output_2.5.11/RSeQC/KO_S4.strand.info
    output: /data/sevillas2/output_2.5.11/DEG_ALL/KO_S4.RSEM.genes.results, /data/sevillas2/output_2.5.11/DEG_ALL/KO_S4.RSEM.isoforms.results
    shell:

    # Setups temporary directory for
    # intermediate files with built-in
    # mechanism for deletion on exit
    if [ ! -d "/lscratch/$SLURM_JOBID/" ]; then mkdir -p "/lscratch/$SLURM_JOBID/"; fi
    tmp=$(mktemp -d -p "/lscratch/$SLURM_JOBID/")
    trap 'rm -rf "${tmp}"' EXIT

    # Get strandedness to calculate Forward Probability
    fp=$(tail -n1 /data/sevillas2/output_2.5.11/RSeQC/KO_S4.strand.info | awk '{if($NF > 0.75) print "0.0"; else if ($NF<0.25) print "1.0"; else print "0.5";}')

    echo "Forward Probability Passed to RSEM: $fp"
    rsem-calculate-expression --no-bam-output --calc-ci --seed 12345          --bam --paired-end -p 16  /data/sevillas2/output_2.5.11/bams/KO_S4.p2.Aligned.toTranscriptome.out.bam /data/CCBR_Pipeliner/Pipelines/RENEE/resources/hg38/30/rsemref/hg38 /data/sevillas2/output_2.5.11/DEG_ALL/KO_S4.RSEM --time         --temporary-folder ${tmp} --keep-intermediate-files --forward-prob=${fp} --estimate-rspd

        (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

Checking the file:
/data/CCBR_Pipeliner/Pipelines/RENEE/resources/hg38/30/rsemref/hg38.grp

This file does not exist. Instead, it's:
/data/CCBR_Pipeliner/Pipelines/RENEE/resources/hg38/30/rsemref/hg38_30.grp

Unsure why this file is labeled differently than expected. Looking at the other files, they are all labeled the same so this is a really odd error.

I wonder why it worked for me with v2.5.10 last week, I don't think we moved any resource files since then?

@slsevilla are you running on FRCE or Biowulf?
It seems to me that you are running on Biowulf and are ending up reading the FRCE JSON ... which is weirder!
Biowulf JSON has it correct as "hg38_30"
But FRCE JSON has it incorrect as "hg38" which you cannot find!
Can you retry by providing full path to the h38_30 JSON .. not just "hg38_30"?

Oh weird!!! I am on Biowulf... I'll test by adding the full json path.

Running two tests
Run pipeline from command line with genome information
./renee run --input /data/sevillas2/v2.5.11/.tests/*.gz --output /data/sevillas2/renee_2.5.11_genome/ --genome hg38_30 --mode slurm

Submission output as expected
RENEE (v2.5.3)
Thank you for running RENEE on BIOWULF!

Generating config file in '/data/sevillas2/renee_2.5.11_genome/config.json'... Done!
/data/sevillas2/renee_2.5.11_genome/resources/runner slurm -j pl:renee -b /gpfs/gsfs10/users/CCBR_Pipeliner,/data/CCBR_Pipeliner,/gpfs/gsfs12/users/sevillas2/v2.5.11/.tests,/data/sevillas2/renee_2.5.11_genome,/lscratch -o /data/sevillas2/renee_2.5.11_genome -c /data/sevillas2/renee_2.5.11_genome/.singularity -t /lscratch/$SLURM_JOBID/ -n biowulf
Successfully submitted master job: TRIGGEROPTIONS:--rerun-triggers mtime
16693264

Run pipeline from command line with json information
./renee run --input /data/sevillas2/v2.5.11/.tests/*.gz --output /data/sevillas2/renee_2.5.11_json/ --genome /data/sevillas2/v2.5.11/config/genomes/biowulf/hg38_30.json --mode slurm

Submission output as expected
Thank you for running RENEE on BIOWULF!

Generating config file in '/data/sevillas2/renee_2.5.11_json/config.json'... Done!
/data/sevillas2/renee_2.5.11_json/resources/runner slurm -j pl:renee -b /data/CCBR_Pipeliner,/gpfs/gsfs10/users/CCBR_Pipeliner,/gpfs/gsfs12/users/sevillas2/v2.5.11/.tests,/data/sevillas2/renee_2.5.11_json,/lscratch -o /data/sevillas2/renee_2.5.11_json -c /data/sevillas2/renee_2.5.11_json/.singularity -t /lscratch/$SLURM_JOBID/ -n biowulf
Successfully submitted master job: TRIGGEROPTIONS:--rerun-triggers mtime
16693269

Both submissions failed with the same error:
Activating singularity image /gpfs/gsfs12/users/sevillas2/renee_2.5.11_genome/.snakemake/singularity/c06d00499469a82e355b3fb78cc49e84.simg
Cannot open /data/CCBR_Pipeliner/Pipelines/RENEE/resources/hg38/30/rsemref/hg38.grp! It may not exist.

I think I figured it out!
The renee script is using a resources dir outside the git repo (see #54) and in that dir is a json file with the incorrect rsemref path: /data/CCBR_Pipeliner/Pipelines/RENEE/resources/hg38/30/config/genomes/biowulf/hg38_30.json.
We need to not use this shared resources dir except for large files that cannot be version controlled (e.g. the kraken db).

Attempted to run from @kelly-sovacool's dev folder but ran into this error
[sevillas2@biowulf renee-dev-sovacool]$ ./renee run --input /data/sevillas2/v2.5.11/.tests/*.gz --output /data/sevillas2/renee_2.5.11_kelly/ --genome hg38_30 --mode slurm
RENEE (v2.5.10)
Thank you for running RENEE on BIOWULF!

Generating config file in '/data/sevillas2/renee_2.5.11_kelly/config.json'... Done!
/data/sevillas2/renee_2.5.11_kelly/resources/runner slurm -j pl:renee -b /gpfs/gsfs10/users/CCBR_Pipeliner,/data/CCBR_Pipeliner,/gpfs/gsfs12/users/sevillas2/v2.5.11/.tests,/data/sevillas2/renee_2.5.11_kelly,/lscratch -o /data/sevillas2/renee_2.5.11_kelly -c /data/sevillas2/renee_2.5.11_kelly/.singularity -t /lscratch/$SLURM_JOBID/ -n biowulf
Traceback (most recent call last):
  File "/gpfs/gsfs10/users/CCBR_Pipeliner/Pipelines/RENEE/renee-dev-sovacool/./renee", line 2475, in <module>
    main()
  File "/gpfs/gsfs10/users/CCBR_Pipeliner/Pipelines/RENEE/renee-dev-sovacool/./renee", line 2471, in main
    args.func(args)
  File "/gpfs/gsfs10/users/CCBR_Pipeliner/Pipelines/RENEE/renee-dev-sovacool/./renee", line 1157, in run
    open(os.path.join(sub_args.output, "logfiles", "mjobid.log")).read().strip()
FileNotFoundError: [Errno 2] No such file or directory: '/data/sevillas2/renee_2.5.11_kelly/logfiles/mjobid.log'

Weird, does that file really not exist? Wonder if it's a latency error?

Good guess - ran again and this time it submitted the job.


revert status colour CSS tweaks

As noted by highscore2:

Might I ask for a better seperation in colors? Red is now very washed out and not really red. Don’t know where this happened but it has something to do with AES extension as I have not seen that color before I installed AES yesterday.

source: highscore2 n the AES forum topic
perhaps these can live behind a feature flag for now

This has been fixed in #58. It’ll be included in the v0.6.9 release.


COSBETA-555

Description
Add functionality to allow user to update their nano-material information.
Checklist:

[ ] Automated checks are passing locally.
[ ] CHANGELOG updated if change is worth telling users about.

General testing

[ ] Test without javascript
[ ] Test on small screen

Accessibility testing

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable css - does content make sense and appear in a logical order?
[ ] Passes automated checker (automated in build or manual)

Pull Request Test Coverage Report for Build 6294

-5 of 13 (61.54%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 95.17%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cosmetics-web/app/models/nano_element.rb
1
2
50.0%

cosmetics-web/app/models/nano_material.rb
1
2
50.0%

cosmetics-web/app/controllers/additional_information_controller.rb
2
5
40.0%

Totals

Change from base Build 6289:
-0.04%

Covered Lines:
10600

Relevant Lines:
11138

💛  - Coveralls


How can make dropdown as optional

After i choose an option from dropdown i want to unselect this option and don't select another option

Hii
I am also facing this issue.
Please let me know if you have any update.


iOS can't find Windows though Windows can find iOS and transmit.

Describe the bug
iOS can't find Windows though Windows can find iOS and transmit.
To reproduce
Just install the software. Basic function.
Expected behavior
Issue continue bothering me after I tried:

Close the firewall on Windows entirely;
Use troubleshoot;
Give Localsend all ports access;
Setting other pairs of ports for two devices;
Reinstall the software;
and the problem remains the same. My router works just fine.
I can only use browser to receive files from iOS.

Screenshots

Desktop (please complete the following information)
Windows 11 22H2 Build 42
Smartphone (please complete the following information)
iOS 17.3.1 Build 43
Additional context
Wired.

Portable bug, don’t t use portable edition.

not a portable bug, cr with the regular install

Can confirm this bug currently happening as well. My Android to Windows both discovers each other fine. My Android to  iPhone 14(running 17.4.1) again fine but my iPhone 14 to windows is where the issue lies.i can see my iphone in nearby devices from my windows machine but iphone doesn't show my windows laptop. New to ios if you can guide me on how to take logs will be more than happy to help

Can confirm this bug currently happening as well. My Android to Windows both discovers each other fine. My Android to iPhone 14(running 17.4.1) again fine but my iPhone 14 to windows is where the issue lies.i can see my iphone in nearby devices from my windows machine but iphone doesn't show my windows laptop. New to ios if you can guide me on how to take logs will be more than happy to help

just don‘t use portable version on windows

just don‘t use portable version on windows

I am not. It's an installed version. But will reinstall it


update g20210505 to g20210523 make does not do anything

Hi.
I have my photoprism g20210505 totally compiled and I wanted to update it to g20210523.
In my photoprism-freebsd-port folder, I did a git pull to update the repo. Thus, several packages were updated. Then, I did a make configure to be sure everything was fine. Then, I typed make, but it did not do absolutely anything. It did not download the new g20210523 version or compile it.
I had to git clone https://github.com/huo-ju/photoprism-freebsd-port again to a new folder. But, doing this, it also downloaded again tensorflow 1.15.2 and began again its compilation...
Could be possible just to download and compile the photoprism itself, since the tensorflow version has not changed this time?
Thanks!

Yes, I agree it should be improved, but it's a little complicated because the original build script must be rewritten. I'm working on it, but need more time to finish.

Part of the problem is that the ports tree isn't really designed for building multiple things like this. make clean; make package should rebuild it properly but at the moment it's going to rebuild tensorflow.


Bug: Unable to use single character strings in contracts

Environment information
Latest ape from main
  ens         0.1.0b2
  infura      0.1.0b3.dev1+g97c2be4
  solidity    0.1.0b3.dev11+g925e687
  ledger      0.1.0b2
  vyper       0.1.0b2.dev2+g0d310ec
  gitlab      0.1.dev2+g09d2e6c.d20220130)
  jules       0.1.dev19+ga7eb802)
  keyring     0.1.0b1)

Python Version: 3.9.7
OS: macOS

What went wrong?
Was trying to call a method I was able to call before (safeMint()) but now am getting an encoding error from eth-abi.
Here is a simple repro:

create a simple smart contract:

// SPDX-License-Identifier: MIT
pragma solidity ^0.8.2;

contract MyContract {
    function safeMint(address to, string memory uri) public {
    }
}

make sure it compiles
try to call safeMint() in a script:

owner = accounts.test_accounts[0]
receiver = accounts.test_accounts[1]
my_contract = owner.deploy(project.MyContract)
my_contract.safeMint(receiver, "0")   #  <--- Errors
my_contract.safeMint(receiver, "0.json")   #  <--- Does not error

You will get this error:
---> 88     raise exc(
     89         "Value `{rep}` of type {typ} cannot be encoded by {cls}{msg}".format(
     90             rep=abbr(value),
     91             typ=type(value),
     92             cls=cls.__name__,
     93             msg="" if msg is None else (": " + msg),
     94         )
     95     )

EncodingTypeError: Value `HexBytes('0x00')` of type <class 'hexbytes.main.HexBytes'> cannot be encoded by TextStringEncoder

How can it be fixed?
I am looking into it.
This seems bad!

NOTE: Figure out this has to do with the str being a single character.
Like "0" and "1" cause failures, but "0.json" does not cause failures

Note: still seeing this issue after upgrading eth-abi

contract string_contract {

    function test_string(string memory my_test_string) external {}

}

In [1]: contract = accounts.test_accounts[0].deploy(project.string_contract)
SUCCESS: Contract 'string_contract' deployed to: 0x274b028b03A250cA03644E6c578D81f019eE1323

In [3]: contract.test_string("a", sender = accounts.test_accounts[0])
Out[3]: <Receipt 0xcefb6ce1acb66f37d2a28a9583d35b9a60473b143ec9a20c030f5eaf90871b8c>

In [4]: contract.test_string("0", sender = accounts.test_accounts[0])
Out[4]: <Receipt 0xccb5f8d581c020877d0b32fa6da3365a56a29be59c939342a397082d57bed5ee>

this no longer seems to be an issue.


682 data types

Checked syntax. Not spec tests.
Some choices are worth discussion.

Good to see this :)

So @ghoneycutt from my pov the PR is basically ready.
Some choices have been done which need review:

How to manage numbers.
Current approach accepts digits both passed as strings (but still digits) or Numeric.
This is needed to pass some of the existing tests, which would fail otherwise.
Current behaviour is somehow more permissive and this is better, IMHO, at is limits the risk to avoid breaking existing setups.
On the same spirit, In various cases the exact accepted type of parameters was not clean (to me), generally I've tried to follow the less restrictive approach. (Any)
Verification of the error message on some input validation checks have been simplified/generalized to avoid the need to refactor whole spec classes.

Needless to say that whatever the tests may cover, with such an invasive commit we may break existing setups using some edge case not tested.

@ghoneycutt  I squashed this one in a single commit.
I'll leave the Sensitive part the the relevant PR, to merge once this one is merged.
Can eventually fix later some details like the ones based on pending decisions, or the ones pointed out by @Phil-Friderici  and @jeffmccune
I'd prefer to merge and consolidate asap a common code base based on puppet 4 types.

Released in v2.26.0


Fix OnyxCheckboxGroup check-all with disabled options

When checking all clickable options, the check all box should show "checked", instead it shows "indeterminate".

The problem is, that the "loading" option also counts. Proposal: we should remove the loading checkbox from this example in storybook, as it is only leading to confusion

https://github.com/SchwarzIT/onyx/pull/999


[Question] LLaVA-Next (Stronger) checkpoint 404 error

Question
Hi, great work! I want to download the llava-next stronger checkpoint, but the website (https://huggingface.co/collections/lmms-lab/llava-next-6623288e2d61edba3ddbf5ff) meets 404 Error.

It seems that u can find all models here: https://huggingface.co/lmms-lab  (github: https://github.com/LLaVA-VL/LLaVA-NeXT)


Fix firmware update status on errored updates

Minor refactoring to state reset code, plus fixing a bug where errors did not lead to resetting of update states.
Note:

I have not built this yet -- and I have limited JS capabilities :)
I couldn't find a style guide, but I'm guessing "_" prefix for private methods
I hope that this (in beginFirmwareUpdate()) is the same this (in _resetUpdateStatus()) when calling other methods in the same type..

Fixes #3966

@zwave-js-bot fix lint

Thanks! This is a prime candidate for another state machine, but thats a problem for another day.

I couldn't find a style guide, but I'm guessing "_" prefix for private methods

I tend to only use _ for private variables.

I hope that this (in beginFirmwareUpdate()) is the same this (in _resetFirmwareUpdateStatus()) when calling other methods in the same type..

Yep, otherwise TypeScript would scream at you.


container-launcher: get image info from WLM

What does this PR do?
NOTE: this is a PR against the branch for #12440.
Using dockerutil to get container information assumes that it is a
docker container, which is not always the case.  WLM can provide the
information directly and more simply.
Motivation
Fix errors like
2022-07-01 20:17:50 UTC | CORE | WARN | (pkg/logs/internal/launchers/container/tailerfactory/defaults.go:86 in defaultSourceAndServiceInner) | Could not inspect container b229ed1b1167bb47da9d8bc9143b87b712fb99fae70e3b8a28f3eea646387505: "docker container b229ed1b1167bb47da9d8bc9143b87b712fb99fae70e3b8a28f3eea646387505" not found

and resulting lack of default source/service set for the log source.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Run the agent in a containerd-based k8s environment, with container_collect_all enabled, and start a pod.  Check that the logs for that pod have the appropriate source and service (either with agent stream-logs or agent status).
Repeat in a docker-based k8s environment.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

Good point about Docker-based k8s.  I don't know if there are k8s environments based on podman.

(I've set skip-qa here because I'm not sure it will generate a card since it's a merge to another PR -- I'll copy the QA instructions on that PR)


Should accept pipelines and transformers

Describe the bug
I was passing a pipeline to a table vectorizer via something like this:
TableVectorizer(datetime_transformer=time_pipe)

But this breaks.
    raise ValueError(
ValueError: 'transformer' must be an instance of sklearn.base.TransformerMixin, 'remainder' or 'passthrough'. Got transformer=Pipeline(steps=[('selectcols', SelectCols(cols='date')),
                ('functiontransformer',
                 FunctionTransformer(func=<function datetime_feats at 0x1377ea0c0>)),
                ('splinetransformer',
                 SplineTransformer(extrapolation='periodic', n_knots=12))])

Steps/Code to Reproduce
from sklearn.preprocessing import FunctionTransformer
from sklearn.pipeline import make_pipeline
from skrub import TableVectorizer

TableVectorizer(datetime_transformer=make_pipeline(FunctionTransformer()))

Now try and fit some data against it.
Expected Results
No error is thrown.
Actual Results
Error is mentioned earlier
Versions
skrub version 0.1.0

thanks! this is a duplicate of #886
it will be fixed by #902


add smoke tests for ionide extension

ref http://vscode-docs.readthedocs.io/en/stable/extensions/testing-extensions/
ref https://code.visualstudio.com/docs/extensions/testing-extensions
to add basic smoke test for os, in travis/appveyor

@enricosada There's Fable.Template.Jest: https://github.com/jgrund/fable-template-jest

As the vast majority of logic nowadays is in FSAC and Ionide is a rather thin client, this is no longer needed


CSX test runner

Is there a recommended approach to writing tests for CSX scripts? Both powershell (Pester) and golang (built in) have easy ways to run and report on tests for single files. It would be nice to have a way to have a script file foo.csx, write tests for it in foo.test.csx and be able to run dotnet script test to execute the *.test.csx files in the current directory.
This is something that should be possible currently by just running the foo.test.csx file but that would require a ton of boilerplate code every time (eg. using XUnit AssemblyRunner). Is a streamlined approach to this something that has been considered or does something similar already exist?

Hi @plaisted. I've created ScriptUnit just for this purpose. Check it out here. https://github.com/seesharper/ScriptUnit

This looks like what I was looking for. Appreciate the work @seesharper


Periphmraa

The error were generated by some implicit casts that the compiler did not like. The question remains as to why this is a warning when building mraa, but an error when trying to build periphmraa.

Merged. Probably should merge periphmraa now.


Does not compile

I get this:
cc -Wall  -std=c99 -pedantic -g -DDEBUG -o obj/xbright.o -c xbright.c
xbright.c: In Funktion »brightUp«:
xbright.c:76:50: Fehler: expected expression before »)« token
     unsigned int newValue = (current+1 < MAXVALUE) ? current+1 : MAXVALUE;
                                                  ^
xbright.c: In Funktion »brightDown«:
xbright.c:90:27: Fehler: expected expression before »)« token
     if (current > MAXVALUE) {
                           ^
xbright.c: In Funktion »brightSet«:
xbright.c:107:47: Fehler: expected expression before »)« token
     if ((setValue < 0) || (setValue > MAXVALUE)) {
                                               ^
xbright.c: In Funktion »usage«:
xbright.c:146:53: Fehler: expected expression before »)« token
     printf("usage: %s [+-=[0-%d]]\n", NAME, MAXVALUE);
                                                     ^
make: *** [Makefile:50: obj/xbright.o] Fehler 1

Using ArchLinux on gcc 6.2.1

The problem is that the Makefile cannot find the brightness data in /sys. ArchLinux is using bleeding edge Linux kernels, so I reckon you may need to change the find command in the Makefile to make it work.
Makefile:
Autoconfiguration
BRIGHTNESSFILE=find /sys/devices/|grep "video0/brightness" | tail -1
MAXVALUE=find /sys/devices/ | grep "video0/max_brightness" | tail -1 | xargs tail -1
Please search for the brightness files (eg. find /sys/devices | grep -i brightness) and update the Makefile.
It's been a while since I used the program last time. I'll test when I have a chance and update the repo.

Ah, I see. You should look in
/sys/class/backlight/*/

nowadays. That's the supported way of today

Cool. Thanks for the info. Does it work once the path is changed?

I haven't checked yet, haven't found the time

FYI: I've checked on Debian and the old command worked. In any case I've updated the Makefile to use /sys/class/backlight.

wasn't really a problem - rather misconfiguration.

Sorry I had a lot to learn for uni in the last days.
Anyway, /sys/class/backlight is the supported way and works for me.
Sometimes the backlight is not attached to the display device or video output which is why this is a better way


This server requires you to connect with Velocity.

I use Velocity and followed the setup process of BungeeGuard+, but the problem is that my server is kicking me out with the message 'This server requires you to connect with Velocity.' So, I tried switching to the modern forwarding function, and NLogin is automatically installing BungeeGuard. As a result, my server shuts down every time I start it. Could you help me?

Hi, @Wojpik
If you are using the modern forwarding, you can safely remove BungeeGuard from the backend servers.

Yes I tried it but when nlogin lunch it download and install bungeeguard.

Yes I tried it but when nlogin lunch it download and install bungeeguard.

The installation only happens in first run. You can delete it after.

On my server it installs every time. I think it is because i don't setup nlogin on proxy and lobby server. I can't set it up because i can't join

On my server it installs every time. I think it is because i don't setup nlogin on proxy and lobby server. I can't set it up because i can't join

Try downloading the latest version of nLogin:
https://www.nickuc.com

I downloaded it yesterday and used it.

I fixed it. I needed to set in paper_global.yml velocity to false.


RunInTerminalResponse given from vscode does not have a body.

VSCode Version: 1.22.2
OS Version: Windows 10

The debug protocol:
https://raw.githubusercontent.com/Microsoft/vscode-debugadapter-node/master/debugProtocol.json specifies that RunInTerminalResponse has a mandatory body, yet, the result from vscode when asked to run something in the integrated terminal does not have a body.
i.e.: The output I have is {'request_seq': 2, 'command': 'runInTerminal', 'type': 'response', 'success': True, 'seq': 5}
So, even if the body is an empty object, the actual body should be there (or the specification should be updated to make it optional).

Since making a mandatory element optional would be a breaking DAP change, I've made sure that VS Code returns a 'body' element in the response.

Yeah, that was a simple, low-risk fix... ;-)


caskroom.io is down (expired?)

http://caskroom.io/ opens to parking page:

Pinging @phinze in case that wasn’t on purpose.
But that domain has been abandoned for a while. It continued to work but only redirected to https://caskroom.github.io. You’ll notice there’s no mention of http://caskroom.io in the documentation.
Relevant discussion: https://github.com/caskroom/caskroom.github.io/issues/46

I agree with discussion and will change link in my project README to point to https://caskroom.github.io.
Thanks!

I can confirm that letting go of that domain was on purpose - I'm guessing it should become available soon if anybody wants to pick it up in the future! 👍


[cli] Update Teaclave Command Line Tool so it can output cmac

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change (select or add applied and delete the others)

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] API change with a documentation update
[ ] Additional test coverage
[ ] Code cleanup or just sync with upstream third-party crates

How has this been tested?
Checklist

[ ] Fork the repo and create your branch from master.
[ ] If you've added code that should be tested, add tests.
[ ] If you've changed APIs, update the documentation.
[ ] Ensure the tests pass (see CI results).
[ ] Make sure your code lints/format.

Merged. You can safely delete this branch. Thanks!


Group headings appear in a mixture of languages

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
It should display a list of tasks grouped by dates in either German or English.
Current behaviour
Displays tasks in German and English date grouping.

I do not know, what I have done differently with the task descriptions.
Steps to reproduce
I used this query in one file.
short mode
not done
(scheduled on 2023-07-20) OR (scheduled before 2023-07-20) OR (due on 2023-07-20) OR (due before 2023-07-20)
group by scheduled
sort by status.name

This Task in another file.
- [ ] Termine einen Augenarzt Check-up für April 2024. ⏳ 2023-07-17

This Task in another file.
- [/] Erstelle eine Übersicht über den State of the Art. ⏳ 2023-07-17

Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[X] Linux
[ ] macOS
[X] Windows

Obsidian Version
1.3.5
Tasks Plugin Version
4.1.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
I am sorry, but I am clueless.
Best
TH

path does not include template
path does not include templates

These are my (only) global queries.

Thanks for the new picture and the extra info.

Multiple same mondays on the other hand are annoying.

I totally agree.

Could it have something to do with different devices, that are modifying tasks?

If it is, it's nothing to do with Tasks. Tasks only ever reads the notes on the local machine. There is no communication between Tasks on your different machines.
What is your Language setting in the Obsidian Settings -> About?

I doubt very much that I am going to be able to do anything about this, other than to make suggestions for things that you can experiment with.
Have you updated Tasks recently and not restarted Obsidian? I doubt that would cause this, but I know that Tasks updates require a restart anyway.
At least now you can control the formatting of dates in group headings, with the new Custom Grouping feature,.
These group by function examples are for Due date, but they will all work for Scheduled too:
https://publish.obsidian.md/tasks/Queries/Grouping#Due+Date
So I suggest you look at the Tokens column in the table in the following page, and try setting up a custom group by function line that gives you consistent behaviour.
https://momentjs.com/docs/#/displaying/format/
Please do report back if you find something that works for you.

What is your Language setting in the Obsidian Settings -> About?

Language is set to english.

Have you updated Tasks recently and not restarted Obsidian? I doubt that would cause this, but I know that Tasks updates require a restart anyway.

Updated all and restarted. The issue persists.
I tried removing the written out day of the week.
group by function task.scheduled.format("YYYY-MM-DD")

That seems to work. I'll test, if it consistently fixes the behavior and report back. Although this only is a workaround. The source of different language interpretation by the plugin should still be investigated.

I understand that this is a genuine annoyance for you, but to the best of my knowledge, nobody else has ever reported this problem in Tasks. (Of course, others may have experienced it, but I don't think it's been reported.)

So as there is a workaround via user control over the group names now, I am inclined to resolve this as 'Not planned' - it's not something that will ever be important enough, or likely to be able to find a solution to it...
Thank you for reporting it though - much appreciated. It's always helpful to understand how it works in non-English-language systems.


Possibility to add support for 2gis maps

Hi is it possible to add support in echarts library for this maps? Thanks in advance.
https://docs.2gis.com/en/mapgl/overview#nav-lvl2--Full_code_example

@myrulezzz I have never worked with 2gis, and I don't remember seeing a plugin for ECharts to support it.
It may make sense to create a separate plugin for 2gis, it should be easy to do and allow to add of markers from data sources.

Do you know any link that could help me create the connector

@myrulezzz I would look at the Google, Gaode, and Baidu Maps extensions for clues and do something similar. The extension interface is described in the Apache ECharts documentation.
Also, I would ask the Apache ECharts team if they plan to add it or hear about it in their GitHub repository.
This issue is out of the scope of the Grafana Panel. If you find an extension, we will happily support it in future releases.


Request: add a DialogflowApp.getLang() method

To write internationalized apps, it is useful to know the language of the request. If the request comes from Google Actions, we have a user in the originalRequest field (originalRequest.data.user) and a corresponding locale field (originalRequest.data.user.locale). We can easily get the locale of the user with the method AssistantApp.getUserLocale().
However, if the request comes directly from the DialogFlow console (using the try it now field) there is no user from where we can get the locale. It is still possible to get the language of the request using the lang field of the request. Indeed, I can access the lang field using app.body_.lang but I think that it would be "cleaner" to access this field through a dedicated getLang() method.
note: I would rather call this method lang(), but to stay consistent with the other methods of DialogflowApp (getIntent, getArgument, ...) I suggest to call this method getLang

Can you please assign me this issue...Also please explain in brief what to do?

@Canain the original issue is a bit outdated, but the spirit of the issue applies.
If I have an intent handler:
app.intent('Default Welcome Intent', conv => {
  conv.ask(`User language is ${conv.user.locale}`)
})

The user's locale is passed through when using the AoG simulator.

{
  ...
  "user": {
    "locale": "en-US",
    "userVerificationStatus": "VERIFIED"
  },
}

However, if I use the Dialogflow simulator instead, the JSON request is from a different pathway and is not passed to conv.user.locale.

{
  "responseId": "70e4b740-589e-4ff7-9ddc-b268e78c6bba-2a4c0c5e",
  "queryResult": {
    "queryText": "hello",
    "action": "input.welcome",
    "parameters": {},
    "allRequiredParamsPresent": true,
    "fulfillmentText": "Good day! What can I do for you today?",
    "fulfillmentMessages": [
      {
        "text": {
          "text": [
            "Good day! What can I do for you today?"
          ]
        }
      }
    ],
    "intent": {
      "name": "projects/user-lang/agent/intents/a31bf099-88ea-4167-8858-9a04549a553a",
      "displayName": "Default Welcome Intent"
    },
    "intentDetectionConfidence": 1,
    "languageCode": "en"
  },
  "originalDetectIntentRequest": {
    "payload": {}
  },
  "session": "projects/user-lang/agent/sessions/218f7fc6-b9da-7d2c-63f8-75ebea3e8835"
}

The goal of this issue is to have at the very least Dialogflow's language passed through to conv.user.locale if it is not defined already.


WebRTC code has moved from svn to git

New code repo is here: https://chromium.googlesource.com/external/webrtc/+/master
These build scripts are using the old SVN repo which no longer receives changes.

@csdodd This should be resolved in latest scripts

https://github.com/pristineio/webrtc-build-scripts/blob/master/android/build.sh#L29


Please release for ReSharper 10

LOVE this plug in but we just moved to VS2015 so i have to put in RS10 :(

@pcKris don't worry - in version 10 postfix is embedded in ReSharper itself. :)

I know, saw that right after I posted and felt like a complete newb :(
On Jan 12, 2016 12:05 PM, "derigel"<EMAIL_ADDRESS>wrote:

@pcKris https://github.com/pcKris don't worry - in version 10 postfix
is embedded in ReSharper itself. :)
—
Reply to this email directly or view it on GitHub
https://github.com/controlflow/resharper-postfix/issues/61#issuecomment-170976091
.

@derigel23 looks like they dropped the .format :'( one of my favorite ones.
guess i will have to stop being so lazy


Add streaming push

New function - push_blob_stream lets users pass a stream to the blob upload function in order for it to iterate.
The signature is similar to the one of the pull so those could be chained.
Builds upon #109

I'm not sure about this PR because when I tested it it took infinite time to run opposed to the "one off" functions. I'll finish the re-auth first.

@aviramha can you rebase this PR against main?
Also, @thomastaylor312 and I would like to tag a new release of this crate once this PR gets merged :)

I think it'll take me some time to re-visit, while I tried to use this flow I couldn't manage to get it to work so I fear there needs to be more thinking on the design.

@aviramha no worries, I don't think we're on a hurry to push out a new version of the crate


🔀 ci/typescript-upgrade + other upgrades

[x] 🙈  pnpm content-addressable store is ... localized?
[x]  ⬆<EMAIL_ADDRESS>[x] 🔧 next.config
[x] ➖  next-compose-plugins 🚚 build-info.json
[x] ➖  @types/github-slugger
[x] ➖  emoji-api, node-emoji

This pull request has been deployed to Vercel.

Latest commit:
87ae226

✅ Preview:
https://jeromefitzgerald-kbjentsr4-jerome5.vercel.app

🔍 Inspect:
https://vercel.com/jerome/jeromefitzgerald/B4A6vedzKRXeUm1UP1a3r17kV9RT

View Workflow Logs


[fix][broker] Fix wrong double-checked locking for readOnActiveConsumerTask in dispatcher

Motivation
The access on readOnActiveConsumerTask is not thread safe.
if (readOnActiveConsumerTask != null) {
    return;
}
readOnActiveConsumerTask = topic.getBrokerService().executor().schedule(() -> {/* ... */});

There is a case that:

Steps
Thread 1
Thread 2

1
Read readOnActiveConsumerTask: null

2
call schedule

3

Read readOnActiveConsumerTask: null

4

call schedule

5
Write readOnActiveConsumerTask to the result of schedule

6

Write readOnActiveConsumerTask to the result of schedule

Then schedule() will be called twice and only one result (it could be either step 5 or 6) will be assigned to readOnActiveConsumerTask.
Modifications
Follow the double-checked locking when readOnActiveConsumerTask is null to ensure readOnActiveConsumerTask cannot be called concurrently.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@lhotari scheduleReadOnActiveConsumer is thread safe. The steps of scheduleReadOnActiveConsumer are:

Call cancelPendingRead(), which might set havePendingRead with false. (The concurrent call of cancelPendingRead() does not matter because it does not set havePendingRead with true so that following logic will be skipped.)
Skip the following logic if havePendingRead is true, which could only be caused by readMoreEntries.
Call readMoreEntries() immediately or lazily according to the subscription type and the config.

Yes, there is a chance that readMoreEntries is called concurrently.

Steps
Thread 1
Thread 2

1
Read havePendingRead: false

2

Write havePendingRead: true

3
Call readMoreEntries

However, it's still safe because in readMoreEntries, before reading messages through cursor, it checks havePendingRead again in the synchronized block. https://github.com/apache/pulsar/blob/442595ea26c8c0699807a0fef2b7e2e27c677c08/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherSingleActiveConsumer.java#L343-L349

if the same bug pattern is in other locations in the Pulsar code base?

BTW, yes, the same bug pattern is use in many places.

@lhotari This PR is merged too quickly without a 2nd review. After revisiting the code, I found the double-checked locking is unnecessary.
scheduleReadOnActiveConsumer() is always called in pickAndScheduleActiveConsumer(), which is always called in addConsumer() and removeConsumer(). However, both of them are synchronized methods. So scheduleReadOnActiveConsumer could never be called concurrently.
The original motivation is the IDE warning:

It's true that it's a non-atomic operation on volatile field 'readOnActiveConsumerTask'. It's a commonly seen error to access a volatile field so IDE gives such warnings. However, we don't need to make scheduleReadOnActiveConsumer() thread safe. Unfortunately, there is no comment that notes this point. The lack of such comments makes code hard to maintain. Generally, developers assume all methods must be thread safe so the code might become unnecessarily complicated.
I opened another PR (https://github.com/apache/pulsar/pull/22285) that improves the code quality and reduces the unnecessary atomicity. That PR also reverts the changes of this PR. PTAL again.

@lhotari This PR is merged too quickly without a 2nd review. After revisiting the code, I found the double-checked locking is unnecessary.

@BewareMyPower We don't have a policy that there should be a 2nd review. This PR didn't contain any risky changes and wasn't harmful at all and that's why I merged. It's better to keep on moving instead of optimizing for perfect PRs. The resolution that you have done is a good one: opening another PR with follow-up changes.

Generally, developers assume all methods must be thread safe so the code might become unnecessarily complicated.

@BewareMyPower I agree that generalizing thread safety to a certain style in the Pulsar code base won't be very useful. However there are real problems where thread safety has been ignored and fixing the issues isn't prioritized. For example the topic level policies contain real problems, #21303.


InputMethodManager( 5324): showSoftInput() view=io.flutter.embedding.android.FlutterView{88bd8f0 VFE...... .F...... 0,0-1080,2340 #1 aid=1073741824} flags=0 reason=SHOW_SOFT_INPUT

I'm facing this issue during type. It becomes prolonged.

Closing this as a duplicate of https://github.com/flutter/flutter/issues/117995.


Duplicate Sinks.ApplicationInsights namespace?

Tried implementing a new telemetry converter and noticed that ITelemetryConverter and its implementations are in the namespace
Serilog.Sinks.ApplicationInsights.Sinks.ApplicationInsights.TelemetryConverters
Should Sinks.ApplicationInsights. really occur twice in that path?

If you look at this line in src the namespace seems to be correct:
https://github.com/serilog/serilog-sinks-applicationinsights/blob/master/src/Serilog.Sinks.ApplicationInsights/TelemetryConverter.cs#L15
using Serilog.Sinks.ApplicationInsights.Sinks.ApplicationInsights.TelemetryConverters;

I see I was a bit unclear. I wasn't wondering whether it was actually the case. I was wondering whether it is an intended namespace or a bug, and if perhaps the duplicate part of the namespace should be removed.

Fixed in 4.0.0-dev 👍


test(ticdc): revert canal json basic test changes

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/4661
What is changed and how it works?

revert canal json basic test changes https://github.com/pingcap/tiflow/pull/4558 Fixed by https://github.com/pingcap/tidb/pull/32301
fix incorrect require test value order. The second parameter should be the expected value.

Check List
Tests

Unit test

Code changes
None
Side effects
None
Related changes
None
Release note
None

/run-all-tests

/merge

Codecov Report

Merging #4662 (05814a4) into master (9607554) will decrease coverage by 0.5847%.
The diff coverage is 40.4020%.

Flag
Coverage Δ

cdc
59.5296% <44.2307%> (-0.3927%)
:arrow_down:

dm
51.3090% <36.2105%> (-0.7199%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4662        +/-   ##
================================================
- Coverage   55.6402%   55.0554%   -0.5848%
================================================
  Files           494        513        +19
  Lines         61283      63723      +2440
================================================
+ Hits          34098      35083       +985
- Misses        23750      25153      +1403
- Partials       3435       3487        +52

/run-all-tests

/run-all-tests

/run-leak-test


feat(react-email): support dir option

fix #248

Hey @m-shaka, can you fix the conflicts?

fixed!

Awesome stuff @m-shaka!
Do we need to add this option for the export command as well?
Here's a PR to add some docs for this already: https://github.com/resendlabs/react-email/pull/401

Oh yeah, we'll need this for export as well:
https://github.com/resendlabs/react-email/blob/main/packages/react-email/source/commands/export.ts#L51
@m-shaka can you update the PR?


Add bower.json

This is the first step towards making unexpected more accessible for bower users.
Before merging this branch a new publishing flow should be put in place which adds unexpected.js to version control when preparing a release.
Currently there is no easy way for consumers to get a built unexpected.js-file without installing unexpected npm dependencies and having a *nix environment available to run the make command. This is a huge blocker and should be remedied

Coverage remained the same when pulling 774d0762cc5ee711bbf467610f4cba2b64ea7ae3 on Munter:bower into f1060e552e62059be7d0d8541b0776a7b211e512 on sunesimonsen:master.

Coverage remained the same at 95.06% when pulling b887fa15640216f0725f49d00368930475fe9570 on Munter:bower into f1060e552e62059be7d0d8541b0776a7b211e512 on sunesimonsen:master.

Thanks for the pr, looks great. I'll merge this pr as your are right - a lot of people like bower so we should probably support that. But I've like to merge it as part of reworking the makefile to be the main interaction point for releasing and publishing. So unless you really need this now, I'll wait a bit with this change. I'll just leave the pr for later when I'm ready to merge it.

I've checked in a built version for my immediate purposes, so no rush

Great - thanks. I'll get it done soon, just not right now.

Fixed by<EMAIL_ADDRESS>


fixed linting and added validation

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


HaxePunk autocompletion

when trying to use autocompletion in simple HaxePunk project, there is no completion. There is only error shown on statusbar, saying that BitmapData class was not found. But compilation is ok though.
HaxePunk engine uses flash.display.BitmapData, so i think haxe completion server somehow misses it.
Is it possible to switch plugin to compiler completion and not server one? It seems to me that it may be helpful, because in FlashDevelop completion works and it uses compiler completion.
UPD: tested in FD with server completion - everything is fine.

so. magically issue solved by me. today i tried heaps lib and it reqired 3.2 haxe version, so i've replaced my installed one with this. now i've tried using autocompletion with haxepunk on this version and got some other error. something about vectors or whatever. but compilation was success. then i decided to revert to 3.1.3. and.. everything is ok. i really don't know what was it. may be my haxe version was too old or whatever.
and thank you for great lib)

The next day it returned. Have not changed much. Just was testing my project and noticed that autocompletion fails again.

Can you share your build file (project.xml)?

yes. shure. but it is nearly default.

<app file="Main" main="Main" />

<window fps="60" background="0x333333" />
<window width="640" height="480" resizable="false" unless="mobile" />
<window fullscreen="true" if="mobile" />
<window orientation="landscape" vsync="true" antialiasing="0" if="cpp" />

<source path="src" />

<haxelib name="format" />
<set name="legacy" />
<haxelib name="openfl" />
<haxelib name="openfl-bitfive" if="html5" />
<haxelib name="HaxePunk" />

<assets path="assets/graphics" rename="graphics" include="*.png|*.jpg" />
<assets path="assets/audio" rename="audio" include="*.mp3" if="flash" />
<assets path="assets/audio" rename="audio" include="*.wav|*.ogg" unless="flash" />
<assets path="assets/font" rename="font" include="*.ttf" />
<assets path="assets/levels" rename="levels" include="*.xml" />
<assets path="assets/shaders" rename="shaders" include="*.vert|*.frag" />

<icon path="assets/HaxePunk.svg" />

Please test.

wow. thank you! it's alive! +1


Opensearch integration

Does the snow owl work with Opensearch or only with ElasticSearch ?

Thank you for the feedback @cmark !


Implement manual polling mechanism in the runtime for ARM long-running operations

Description
Refactor the runtime to allow manual polling using our state machine.  The Polling state should be easily serializable, and polls should be asynchronous and non-blocking.  This allows for polling in workflow scenarios where the state may need to be serialized and re-hydrated.
Optionally refactor generated code to use the manual polling mechanism.
Cost: 8

Please consider a fix for this one as well https://github.com/Azure/azure-sdk-for-net/issues/2479

This work has been deprioritized. We will for sure add this feature sometime in next Feb-March 2019 time frame.

@pakrym is this something thats already solved in Azure Core + new codegen?

Yes, closing.


Debug limit rate and LPF

Description

Debug limit_rate to correctly be passed to franka::Robot::control
Change low pass filter to use libfranka's native implementation

Fixes issue where the error controller_torque_discontinuity pops up occasionally.
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[ ] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After
If this PR introduces a change or fixes a bug, please add before and after screenshots (if this is causes a UX change) or before and after examples(this could be plain text, videos etc ) to demonstrate the change.
If this PR introduces a new feature, please also add examples or screenshots demonstrating the feature.
Testing
Please describe the tests that you ran to verify your changes and how we can reproduce.
Checklist:

[ ] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I ran on hardware (1) all scripts in tests/scripts, (2) asv benchmarks.

Codecov Report

Merging #1027 (69ec69f) into main (2c9c1d3) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1027      +/-   ##
==========================================
- Coverage   33.30%   33.16%   -0.15%
==========================================
  Files         347      346       -1
  Lines       34289    34034     -255
==========================================
- Hits        11419    11286     -133
+ Misses      22870    22748     -122

Impacted Files
Coverage Δ

interpreter/robot/tests/world.py
74.45% <0.00%> (-0.73%)
:arrow_down:

agents/droidlet_agent.py


Fixing African easterly wave density plots in TC analysis

Description
Karthik (@karthik-balaguru) noted the odd Y shapes in easterly wave density plots in recent TC analysis results (issue 1). And @paullric also noticed that the ERA5 results are also off, which always have a base value of ~5 (issue 2).

Closes #<ISSUE_NUMBER_HERE>

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

If applicable:

[ ] New and existing unit tests pass with my changes (locally and CI/CD build)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have noted that this is a breaking change for a major release (fix or feature that would cause existing functionality to not work as expected)

Not sure the root cause of issue 2, but I think plotting libraries changes caused the second issue over versions. By reading in files without subsetting the region, it fixed the issue.
https://github.com/E3SM-Project/e3sm_diags/pull/851/commits/3fefb76575115d74106fb019c53dc50e9ac51126

Issue 1 remains, I will keep investigating.

Comparing with apply tempest-extremes (TE) on native ne120pg2 vs data regrided to 720x1440. The Y shapes only appear with native grid data.  Below are the AEW density plots.

The AEW density plots uses three TE functions:
VariableProcessor to calculate relative vorticity
DetectNodes and StitchNodes to detect and stitch candidates
Finally HistogramNodes to generate histogram for the detection.
Detailed steps can be found here
@paullric I'm not sure if TE is applied properly in this case, any advice for further trouble shooting is appreciated!

Also to add that:
TC count increases if the storms are tracked on the regridded output. ~73 (regridded data on 720x1440) vs ~67 (native ne120) for one year of data.

Also to add that: TC count increases if the storms are tracked on the regridded output. ~73 (regridded data on 720x1440) vs ~67 (native ne120) for one year of data.

Following suggestion from @wlin7, when regridding, with a bilinear mapping file (to replace the aave one I used), the TC count (70) get closer to the results tracked with native ne120 data.

This PR will also address: empty input TC files.
from comment: https://github.com/E3SM-Project/zppy/pull/633#issuecomment-2423038746

This PR will also address: empty input TC files. from comment: E3SM-Project/zppy#633 (comment)

This is addressed by https://github.com/E3SM-Project/e3sm_diags/pull/866/commits/0cb36e971c32db4caa3002bc184fff2e1df895db  by @tomvothecoder in cdat-migration-branch. I will copy the code over here, so that current main branch also has this fix.

@tomvothecoder this PR is ready to review, it partially addresses the plotting problem (Y shapes remains); also cross year TCs are removed; I copied over the error capture for cyclones stitches files as well.


Feature/mkdocs

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


FreeSwitchClient make port option optional

The default port is 8021, so it shouldn't be required to pass it to the constructor.

Thank you @greenbea !


error with permissions

What is your Quests version / build number (do not say "latest")?
3.9.4-
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
players without permission continue to be able to execute Player Commands
this happens in case the quests journal permission is given to them
If applicable, can you provide an example quest from quests.yml using Pastebin.com?
just give quests.journal permission, and he will have all other player permissions!

All non-admin permissions are given by default. You must negate/subtract the permission to disable functionality.


Remove ssh from tls_authentication_over_grpc_test

We don't require SSH support and test of whether authentication works "over grpc" should stick to testing over grpc

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.219%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

The README.md should be updated to reflect this change in the plan as well.  It sounds like this test validates the following:

TODO: Set the user and password using AAA CLI on the device, configured to authenticate using TACACS+ with a fallback of the local database.
Configure local user on the DUT using GNMI
Ensure GNMI Get/Set requests succeed using the local credentials.
Ensure GNMI Get/Set requests fail with invalid credentials.

The test plan is removing:

Console credential verification
SSH credential verification

My opinion is also that SSH and console access are still relevant tests - when a network device breaks, SSH and console access are still going be relevant for troubleshooting for quite a while.  I think they should be left and marked at least as TODO items.


✨ Feature: Integrate Konamimojisplosion

https://www.npmjs.com/package/@codecademy/konamimojisplosion

I'd like to write a docusaurus plugin for it. 😁


remove sparkmonitor

Remove SparkMonitor based on feedback from the demo.

Do you want to make this a toggle vs a straight removal? In the future users might be able to choose whether it's enabled or not if so

I think it's fine to remove it for now. If needed, we can easily reintroduce it later.


Error setting up entry for panasonic_cc

After a few weeks without any trouble, this is what happened today. This beast seems to never wanna be tamed.
Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/config_entries.py", line 313, in async_setup     result = await component.async_setup_entry(hass, self)  # type: ignore   File "/config/custom_components/panasonic_cc/__init__.py", line 79, in async_setup_entry     devices = await hass.async_add_executor_job(api.get_devices)   File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/local/lib/python3.9/site-packages/pcomfortcloud/session.py", line 172, in get_devices     self.login()   File "/usr/local/lib/python3.9/site-packages/pcomfortcloud/session.py", line 99, in login     self._create_token()   File "/usr/local/lib/python3.9/site-packages/pcomfortcloud/session.py", line 132, in _create_token     raise ResponseError(response.status_code, response.text) pcomfortcloud.session.ResponseError: Invalid response, status code: 503 - Data: {"message":" ","code":5300}

I have the very same issue today. with core version : 2021.12.2


Add support for Groovy annotation

Need to be fixed in the scripting-groovy package.

Since Scijava kernel will be run from Fiji/ImageJ, the need for annotations such as Grab is less important. I will downgrade the priority of this issue.

I actually think some dependency bootstrapping mechanism is super important. It would actually be nice if it were agnostic to the language, but the Groovy-specific @Grab is better than nothing.
Requiring users to install ImageJ or Fiji in order to use SciJava stuff is, IMHO, bad. One of the beautiful things about the notebook to me is that you can just open a .ipynb file and go—no need to install any other crap. (That's one thing I'll miss about classic Beaker, BTW: the ease of installation. Installing Jupyter is still a PITA by comparison.)

There are multiple points here :

A language agnostic bootstrapping mechanism would be really great indeed. I guess we could re-use the Scijava annotation mechanism with something like that :

// @GrabResolver name='imagej', root='http://maven.imagej.net/content/groups/public/'
// @Grab 'net.imagej:imagej:2.0.0-rc-58'

or similar. With this, the support for Groovy annotation is definitely a low priority.

I get your point on the kernel being independent on Fiji and it really makes sense to me now you say it. For now the kernel is installed from within Fiji and so is setup to use the Java used to execute Fiji (and the associated classpath, see here for code).

But once we have a language agnostic bootstrapping mechanism, we could definitely get rid of the need of Fiji. And even create a conda package of the scijava kernel that depend on the Java conda package. Then will only have to install the Anaconda distribution, then execute conda install notebook scijava-jupyter-kernel and will be able to run any notebook file made for a scijava kernel. That would be awesome !

Jupyter is easy to install if you use the Anaconda distribution : https://www.continuum.io/downloads :-)

Jupyter is easy to install if you use the Anaconda distribution

$ brew cask install anaconda
==> Caveats
To use anaconda, you may need to add the ~/anaconda3/bin directory
to your PATH environment variable, eg (for bash shell):

  export PATH=~/anaconda3/bin:"$PATH"

==> Satisfying dependencies
complete
==> Downloading https://repo.continuum.io/archive/Anaconda3-4.3.1-MacOSX-x86_64.sh
######################################################################## 100.0%
==> Verifying checksum for Cask anaconda
==> Running installer script Anaconda3-4.3.1-MacOSX-x86_64.sh
==> PREFIX=/Users/curtis/anaconda3
==> installing: python-3.6.0-0 ...
...
==> installing: jupyter-1.0.0-py36_3 ...
==> installing: jupyter_client-4.4.0-py36_0 ...
==> installing: jupyter_console-5.0.0-py36_0 ...
==> installing: jupyter_core-4.2.1-py36_0 ...
...
==> installation finished.
🍺  anaconda was successfully installed!
$ which jupyter
/usr/local/bin/jupyter
$ ls -la /usr/local/bin/jupyter
-rwxr-xr-x  1 curtis  admin  241 Aug  8  2016 /usr/local/bin/jupyter*
$ jupyter --version
4.1.0
$ anaconda3/bin/jupyter --version
4.2.1

So I guess I have two versions of Jupyter now... and I have no idea what I even did last August to install the 4.1.0 version... sigh

which jupyter should tell you where the jupyter file is and then you should be able to remove the associated anaconda folder.

Oh I didn't see the which jupyter command you've executed.
I guess you used pip install jupyter for the first version you have. Just uninstall it with pip and everything should be fine.

Heh, I guess I had yet a third jupyter installed with pip:
$ pip uninstall jupyter
Uninstalling jupyter-1.0.0:
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/DESCRIPTION.rst
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/INSTALLER
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/METADATA
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/RECORD
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/WHEEL
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/metadata.json
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/pbr.json
  /usr/local/lib/python2.7/site-packages/jupyter-1.0.0.dist-info/top_level.txt
  /usr/local/lib/python2.7/site-packages/jupyter.py
  /usr/local/lib/python2.7/site-packages/jupyter.pyc
Proceed (y/n)? y
  Successfully uninstalled jupyter-1.0.0
$ which jupyter
/usr/local/bin/jupyter

Thanks for the idea, though. I won't worry about it for now, since it probably won't hurt anything.

ahah ! A classic. In the future only use conda for Python libs installation and everything will be fine.
To get latest update you can also enable conda-forge : conda config --add channels conda-forge && conda update --all

I am closing here in favor of #28.


Tox & Conda instructions for running tests

There have historically been issues between tox and conda playing nicely together. We should warn contributors who use conda about this.

Closed by #117

Link to documentation


When the next lychee version will come?

We're using lychee action v1.3.2 which uses the latest lychee v0.8.2. And faced a weird issue related to hubcaps.
From this PR #502 it's clear that lychee already replaced hubcaps with octocrab and it's a great decision. But the latest lychee version still uses hubcaps. So, we desired to know when the next lychee release will happen with the changes of replacing hubcaps?

Hi, I can create a new release and bump lychee-action afterwards.
What issue did you face with hubcaps?

That's the error. And we've faced it on several workflow runs. But it also fixed automatically without any config changes.
And the time we were facing the issue all the links were valid but it's giving this error for many urls.
2022-03-21T16:16:42.4021715Z ✗ https://github.com/allegroai/clearml/blob/master/examples/reporting/matplotlib_manual_reporting.py: Network error when trying to connect to an endpoint via hubcaps

Huh, then it wasn't a hubcaps issue. The local test worked on my machine (see above), so it's either a problem with lychee-action or with the pipeline. Is the Github action workflow file public so that I can have a look? And we should probably move that discussion to https://github.com/lycheeverse/lychee-action instead. Could you create a new issue there? 😉

Some checks:

Do you see other errors or is that the only one? Specifically, any 429 errors?
Do you have any other Github links which work? If not, can you add one to check?
Did you set a Github token as described in the lychee-action docs? https://github.com/lycheeverse/lychee-action#usage
(This is only necessary if you get a 429 somewhere.)

Here some Github links that are working fine,
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/click [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/hydra [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/kerastuner [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/scikit-learn [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/tensorboardx [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/autokeras [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/services/monitoring [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/fastai [200 OK]
✔ https://github.com/allegroai/clearml/tree/master/examples/frameworks/keras [200 OK]

Here's the issue I've created on lyche-action lycheeverse/lychee-action/issues/97


[WIP]new rfc: authn and authz on chaos dashboard

Rendered markdown: link

PTAL  @dcalvin @WangXiangUSTC @YangKeao @cwen0 @zhouqiang-cl

What's the meaning of authn and authz?


Add rs-dnn-vino sample

rs-dnn-vino
This example demonstrates OpenVINO™ toolkit integration with object detection, using
basic depth information to approximate distance.

The same exact neural network (MobileNet-SSD) is used here as in the OpenCV DNN sample (rs-dnn), for
comparison.
Detected objects are labeled.
Multiple models, if supplied, can be loaded at once and switched at runtime, for comparison.

I'm working on renaming the openvino_helpers::face_detection -> object_detection


Introduce named options to replace array of strings usage

Instead of functions passing and giving array of strings (we refer to them as "strv") that represent options, use an array of typed values. The downside of the strv approach in practice meant that we had to discard work done by parsing for splitting the key/value (or more complex work done by JSON conffile parser).

@barbieri could you take a look at this?

Replaced by #428.


IPAM fix cleanup + tests

Coverage increased (+0.7%) to 51.127% when pulling 1728f6477d161034b0e5083fcaa747dd53f7ff0d on fgschwan:IPAM-fix into 83504cb83b2c28548c69d3c339aae032804e6426 on contiv:master.


Trim Trailing Whitespace Breaks Markdown

When this is set to true
"files.trimTrailingWhitespace": true

trailing spaces get removed at the end of the line in md/markdown files. By default, two spaces in markdown is the same as a <br>. When saving the file with that setting, those two spaces get removed and it changes the output of the markdown.

Well I think we need language specific setting overrides which we have a feature request already for.

@bpasero could you link to the other feature request to make it easy to find? Not sure whether this issue has been fixed!

I think you can use the editor-config extension to configure this now (https://marketplace.visualstudio.com/items?itemName=EditorConfig.EditorConfig)

Successfully disabled files.trimTrailingWhitespace for Markdown using language specific configuration.
Snippet from my settings.json:
    "files.trimTrailingWhitespace": true,
    "[markdown]": {
        "files.trimTrailingWhitespace": false
    },


Unknown Contract Address error

File "/Users/ASUS/opt/anaconda3/lib/python3.8/site-packages/ypricemagic/magic.py", line 51, in get_price
price = yearn.get_price(token, block=block)
File "/Users/ASUS/opt/anaconda3/lib/python3.8/site-packages/ypricemagic/yearn.py", line 24, in get_price
share_price, underlying = fetch_multicall(
File "/Users/ASUS/opt/anaconda3/lib/python3.8/site-packages/ypricemagic/utils/multicall2.py", line 56, in fetch_multicall
raise ValueError(response)
ValueError: {'jsonrpc': '2.0', 'error': {'code': -32602, 'message': 'Invalid parameters: Expected from 1 to 2 parameters.', 'data': '"Got: 3"'}, 'id': 486}
Not entirely sure how to get around this issue.

What asset and block number were you querying?

@PatrickCroke can you try updating brownie to its most recent version and lmk if you still get this error? I'm having trouble reproducing on my end

This is fixed in #8, thanks


Update metadata address class to use truncated hashes

Summary
Some metadata addresses (records, record specifications) when encoded to bech32 format will exceed the maximum allows length of 90 chars.
Problem Definition
Currently the metadata module uses address with the full 32 byte sha256 hash value when incorporating string names.  This can result in bech32 addresses that exceed the 90 char limit (record1q2xcpvj6czy5g354dews3nlruxjelpkssxyyclt9ngh74gx9ttgptgalfudjkzuz9ng46mq4krcq5zqsttnk0).
These addresses would not be parsed correctly by libraries that enforce the specification limits.
Proposal
The full 32 bytes of the sha256 value for the component of these addresses is not required to guarantee uniqueness when they are already combined with a full UUID.  In light of this we should truncate the sha256 hash values to 16 bytes.  This will significantly reduce the amount of data stored in the key space will still maintaining sufficient amounts of uniqueness for the Records within Scopes and ContractSpecifications.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

This needs to be done after the big pr from @dwedul-figure is merged but before we cut 0.2.0 or we will have conflicts with IDs for any stored data.


maybe LlamaModel？

 Thank you for spotting this error and we will get to fix it soon!

Hi @gongel Could you please fix it with a PR? We welcome contributions from the community. Thanks.

Ok. And about this issue ？#3415

Ok. And about this issue ？#3415

Replied.


[draft] Deadline

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Add ENFORCE_FINISH_DEADLINE when a dag is submitted
Add ENFORCE_START_DEADLINE dag action when a job is submitted
create a trigger for ENFORCE_FINISH_DEADLINE in DagManagementTaskStreamImpl
create a trigger for ENFORCE_START_DEADLINE in DagManagementTaskStreamImpl
implement EnforceFinishDeadlineDagProc
implement EnforceStartDeadlineDagProc
delete trigger and delete ENFORCE_FINISH_DEADLINE when dag finishes
delete trigger and delete ENFORCE_START_DEADLINE when job starts

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

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

Project coverage is 50.19%. Comparing base (1ca4944) to head (4092206).
Report is 29 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3950      +/-   ##
============================================
+ Coverage     46.63%   50.19%   +3.56%
+ Complexity    11247     5904    -5343
============================================
  Files          2248     1075    -1173
  Lines         88520    41155   -47365
  Branches       9701     4611    -5090
============================================
- Hits          41278    20657   -20621
+ Misses        43510    18699   -24811
+ Partials       3732     1799    -1933

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

One concern I have is to make sure we use the idea for "reminder" in the DagActionReminderScheduler properly with the MALA. For MALA a reminder means that there's an existing entry in lease table (however for deadlines there was no existing lease attempt so it will complete as if it was a new event).

all actions in the DagActionReminderScheduler are for reminder events -> u added a field to DagAction to know its a reminder
we attempt lease when the reminder trigger fires in the DagActionReminderScheduler
need to pass the reminder flag to the MALA so it handles lease arbitration correctly


Proposed legal & governance improvements for the OpenSSF

There have been some formalizations with governance for projects in the OpenSSF.
I think these are all for the better, but if you have a concern, PLEASE let me know soon! I plan to wait for 1 month (2024-03-12) before starting down this path.
Here is the full documents

LLC Series. LF Legal would like this project to become a "series" within "LF Projects, LLC". This creates a legal identity for the project & provides various legal protections. We hope to not need legal protections, but it's best to have them. To do that, the TSCs to be listed on the project page on who is in the TSC.

The TSC then needs to vote on the
Technical Charter Draft
Make sure the mission is accurate.
Choose a delegate to sign the
Contribution Agreement Draft
Then the LF will submit the
Series Agreement Draft
Please let me know if you have any questions or concerns.

sorry for the delays on this. https://github.com/ossf/osv-schema/pull/228 is approved.
re the contribution agreement draft, I'm not sure any of us understand the legal implications of this as none of us are lawyers.
I'm also not sure there is any owner who can claim the "OSV-Schema" trademark to begin with, given that we started this project/repo in the OSSF -- which I would've assumed meant that it belonged to the OSSF to begin with?


moveToViewController not working in viewDidLoad

preCurrentIndex is overwrite by currentIndex (which is 0) in updateContent(), so i comment it out temporary

set animated true
self.moveToViewController(at:  1 , animated: true)


Automated cherry pick of #2272: .travis.yml: 改正配置

Cherry pick of #2272 on release/2.11.
#2272: .travis.yml: 改正配置

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@9176381). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #2274   +/-   ##
==============================================
  Coverage                ?    7.1%
==============================================
  Files                   ?     452
  Lines                   ?   71240
  Branches                ?       0
==============================================
  Hits                    ?    5060
  Misses                  ?   65764
  Partials                ?     416

Flag
Coverage Δ

#aFlag
7.1% <ø> (?)

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 9176381...6006726. Read the comment docs.

/lgtm

/approve


Add README w/ examples

The Project Home page at https://code.google.com/p/closure-stylesheets/ had a good summary of Closure Stylesheets' features, but github has no README and doesn't point to any equivalent documentation that I can find. Any reason not to convert the Project Home from code.google.com to a README.md?

Porting all of the closure-stylesheets documentation here is one of the things on my TODO list. Since you brought it up, I'll try to get to it this Thursday.

Thanks to your prompting, I did this even sooner that I said. You'll find a new README.md on the front page. I also ported the wiki pages from Google Code. I've got a request in with the team to start redirecting the old site here.


iree-codegen-cpu-materialize-host-encoding fails with invalid reference to GlobalOp

What happened?
I have input MLIR that is exported from the test in this PR.
It's compilation with assertions on fails with
iree-compile: repo/compiler/src/iree/compiler/Dialect/HAL/Analysis/DeviceAnalysis.cpp:180: auto mlir::iree_compiler::IREE::HAL::DeviceAnalysis::gatherDeviceTargets(mlir::Attribute, mlir::Operation *, SetVector<IREE::HAL::DeviceTargetAttr> &)::(anonymous class)::operator()(auto) const [symRefAttr:auto = mlir::SymbolRefAttr]: Assertion `globalOp && "global reference must be valid"' failed.

Full error message is in the reproducer zip.
Steps to reproduce your issue

Download the reproducer sharded-toy-llama.zip.
Run compile.sh.

What component(s) does this issue relate to?
Compiler
Version information
5a2dd5609d41df71e7ad77fb706e1cb971492f7b
Additional context
No response

that pass is fundamentally incompatible with multi-device - you need to disable data tiling

@benvanik How do you disable data tiling? I tried with
--iree-opt-data-tiling=false
--iree-llvmcpu-disable-arm-sme-tiling

But I get the same error.

@hanhanW do you know how to disable it?

Adding --iree-opt-data-tiling=false should disable the data-tiling. I can't tell much without looking at logs.

@hanhanW, I forgot to mention in the description that the full error message is in the reproducer archive.

@benvanik, aside from the problem that this pass should not be called at all, should flow.tensor.transfer ops that reference a nonexistent device be considered illegal? The verifier did not complain. We have
%2 = flow.tensor.transfer %1 : tensor<?x?x?x?xf32>{%c3, %c26, %c2, %c14} to #hal.device.affinity<@__device_1>

but there is no corresponding
util.global private @__device_1 ...

flow.tensor.transfer does not have the op interface SymbolUserOpInterface and does not verify the referenced symbol.

I'm guessing you went in and deleted ops manually?

I have not hand-edited. The input IR references the symbol through promises. E.g.
%56 = flow.tensor.transfer %cast_2 : tensor<?x?xi32>{%dim_4, %dim_6} to #hal.device.promise<@__device_1>

The attribute is what is referencing the symbol, not the op.

One could make the same argument for function call ops. The attribute contains the symbol, but the op still verifies the symbol and it can not be any other way as the attribute itself is not bound the graph (does not have this context).

There should not be a case where this happens without hand-editing (if there is that's what we need to track down).

@benvanik I will track where the origin of the problem is. But shouldn't we add this to the flow.tensor.transfer verification? It will be a lot easier to track this problem with such a verification.

Here is an mlir-print-ir-after-all dump compiling for hip giving the same issue with following iree compile command:
./build/tools/iree-compile ../sharktank/program.mlir --iree-input-type=torch --iree-hal-target-device=hip[0] --iree-hal-target-device=hip[1] --mlir-print-ir-after-all --iree-opt-data-tiling=false --iree-gl
obal-opt-enable-early-materialization=false -o program.vmfb &> hip.txt

https://sharkpublic.blob.core.windows.net/sharkpublic/ian/hip.txt

There is a bug in getFuncExecutableTargetAttrs on data-tiling path. The bug is likely in HAL device analysis and I can take a look.
There is another issue about disabling data-tiling. I thought that --iree-opt-data-tiling=false should work, but it does not work on my machine. It still runs the set_encoding passes even with the flag. I'll take a look.

The input for pass iree-auto-input-conversion lacks information about @__device_1.
It only has about the default local device.
module attributes {hal.device.targets = [#hal.device.target<"local", [#hal.executable.target<"llvm-cpu", "embedded-elf-x86_64", {cpu = "znver4", cpu_features = "+prfchw,-cldemote,+avx,+aes,+sahf,+pclmul,-xop,+crc32,+xsaves,-avx512fp16,-usermsr,-sm4,-egpr,+sse4.1,+avx512ifma,+xsave,+sse4.2,-tsxldtrk,-sm3,-ptwrite,-widekl,+invpcid,+64bit,+xsavec,-avx10.1-512,+avx512vpopcntdq,+cmov,-avx512vp2intersect,+avx512cd,+movbe,-avxvnniint8,-ccmp,-amx-int8,-kl,-avx10.1-256,+evex512,-avxvnni,-rtm,+adx,+avx2,-hreset,-movdiri,-serialize,-sha512,+vpclmulqdq,+avx512vl,-uintr,-cf,+clflushopt,-raoint,-cmpccxadd,+bmi,-amx-tile,+sse,-avx10.2-256,+gfni,-avxvnniint16,-amx-fp16,-zu,-ndd,+xsaveopt,+rdrnd,+avx512f,-amx-bf16,+avx512bf16,+avx512vnni,-push2pop2,+cx8,+avx512bw,+sse3,+pku,-nf,+fsgsbase,+clzero,+mwaitx,-lwp,+lzcnt,+sha,-movdir64b,-ppx,+wbnoinvd,-enqcmd,-avx10.2-512,-avxneconvert,-tbm,-pconfig,-amx-complex,+ssse3,+cx16,+bmi2,+fma,+popcnt,-avxifma,+f16c,+avx512bitalg,+rdpru,+clwb,+mmx,+sse2,+rdseed,+avx512vbmi2,-prefetchi,+rdpid,-fma4,+avx512vbmi,+shstk,+vaes,-waitpkg,-sgx,+fxsr,+avx512dq,+sse4a", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-i128:128-f80:128-n8:16:32:64-S128", native_vector_size = 64 : i64, target_triple = "x86_64-unknown-unknown-eabi-elf"}>]> : !hal.device], iree.consteval} {

Later at iree-hal-materialize-target-devices pass this becomes
util.global private @__device_0 = #hal.device.target<"local", [#hal.executable.target<"llvm-cpu", "embedded-elf-x86_64", {cpu = "znver4", cpu_features = "+prfchw,-cldemote,+avx,+aes,+sahf,+pclmul,-xop,+crc32,+xsaves,-avx512fp16,-usermsr,-sm4,-egpr,+sse4.1,+avx512ifma,+xsave,+sse4.2,-tsxldtrk,-sm3,-ptwrite,-widekl,+invpcid,+64bit,+xsavec,-avx10.1-512,+avx512vpopcntdq,+cmov,-avx512vp2intersect,+avx512cd,+movbe,-avxvnniint8,-ccmp,-amx-int8,-kl,-avx10.1-256,+evex512,-avxvnni,-rtm,+adx,+avx2,-hreset,-movdiri,-serialize,-sha512,+vpclmulqdq,+avx512vl,-uintr,-cf,+clflushopt,-raoint,-cmpccxadd,+bmi,-amx-tile,+sse,-avx10.2-256,+gfni,-avxvnniint16,-amx-fp16,-zu,-ndd,+xsaveopt,+rdrnd,+avx512f,-amx-bf16,+avx512bf16,+avx512vnni,-push2pop2,+cx8,+avx512bw,+sse3,+pku,-nf,+fsgsbase,+clzero,+mwaitx,-lwp,+lzcnt,+sha,-movdir64b,-ppx,+wbnoinvd,-enqcmd,-avx10.2-512,-avxneconvert,-tbm,-pconfig,-amx-complex,+ssse3,+cx16,+bmi2,+fma,+popcnt,-avxifma,+f16c,+avx512bitalg,+rdpru,+clwb,+mmx,+sse2,+rdseed,+avx512vbmi2,-prefetchi,+rdpid,-fma4,+avx512vbmi,+shstk,+vaes,-waitpkg,-sgx,+fxsr,+avx512dq,+sse4a", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-i128:128-f80:128-n8:16:32:64-S128", native_vector_size = 64 : i64, target_triple = "x86_64-unknown-unknown-eabi-elf"}>]> : !hal.device

Whatever is generating the jit IR for iree-auto-input-conversion is dropping the device definitions that are present befor iree-consteval-jit-globals.
#device_target_local_0_ = #hal.device.target<"local", {ordinal = 0 : index}, [#executable_target_embedded_elf_x86_64_]> : !hal.device
#device_target_local_1_ = #hal.device.target<"local", {ordinal = 1 : index}, [#executable_target_embedded_elf_x86_64_]> : !hal.device
module @module attributes {stream.affinity.default = #hal.device.affinity<@__device_0>} {
  util.global private @__device_0 = #device_target_local_0_
  util.global private @__device_1 = #device_target_local_1_

It makes sense that iree-consteval-jit-globals would ignore device placement while evaluating to constants and use only the default 'local' device.
Maybe for each declared device it should set it to point to the single CPU device that is available at time of compilation.
E.g.
#device_target_local_0_ = #hal.device.target<"local", {ordinal = 0 : index}, [#executable_target_embedded_elf_x86_64_]> : !hal.device
module @module attributes {stream.affinity.default = #hal.device.affinity<@__device_0>} {
  util.global private @__device_0 = #device_target_local_0_
  util.global private @__device_1 = #device_target_local_0_

Sorry that I was busy on other stuff today, so no more updates from my side. I'll prioritize this issue tomorrow.

I am working on a PR for iree-consteval-jit-globals to define all the required devices to point to the single default/local device in the JITed module.

Hold up - that's not the correct approach.

Two issues: data tiling as it is plumbed through codegen today is fundamentally incompatible with jit eval, and jit eval today should be stripping all transfer ops and device references.
Data tiling needs to separate itself from executable target and only use that as a way to seed default information. Data tiling must always be overridable independently of executable target once it gets to codegen. That is, we must be able to codegen data tiling behavior for one executable target on a different executable target. Today it always assumes it can use the executable target to derive the information it needs. Things like jit eval and hoisting do not work if we can't do that - and we need jit eval and hoisting - so someone really needs to get on that (I've been saying this for years :).
In order for that to work jit eval needs to ensure that information is present in the IR separate from the target used for jitting (the host). This means that the split of executable target and tiling information needs to happen before jit eval such that what device is running any particular op is independent from what data tiling is being performed. After that point no device information from the original program is required in the jit eval program, and the only device that should exist is the host default device. All device affinities and transfer ops should be stripped.
So don't alias devices - remove them all. There should be no transfer ops in a jit eval function, and no device affinities specified.

OK, I will add stripping of flow.tensor.transfer ops to the iree-consteval-jit-globals pass.

Here is a PR #18663 that strips the affinities.


OAuth redirect points to localhost dev instead of ngosi.io

Email link
https://rhiklmqymyrelhzwdgbq.supabase.co/auth/v1/verify?token=YdsVkWFaq9gsfYv0VJp6Jw&type=signup&redirect_to=https://localhost:3000
Error redirect link
https://localhost:3000/#error_code=404&error_description=Confirmation+Token+not+found

@iAmWillShepherd  Would this issue be solved if we change the redirect URI from localhost dev to ngosi.io? I wanted to go ahead and try this but I don't know which platform we used to register ngosi for OAuth.

I think it's best to control this from an environment variable  this should work in your local dev machine in adding to production.


Synthesize indirect dispatch workload calculations for data-dependent shapes

When we aren't able to fully parameterize shapes based on input shapes (such as when a shape is sliced out of an arbitrary tensor) we'll need to synthesize a dispatch that computes the workload for indirect dispatch. We'll want to add bounds checking and compatibility checks and some way of communicating data loss errors (such as when the computed workload is not possible to perform), though we could possibly work around this by enqueuing multiple indirect dispatches and using as many as we need to satisfy the request.

This is just an idea; will file targeted issues when we get around to it.


ESConfiguration - 	 CREATE_FAILED

CloudFormation did not receive a response from your Custom Resource. Please check your logs for requestId [14855573-d52a-4c49-86c0-253981190145]. If you are using the Python cfn-response module, you may need to update your Lambda function code so that CloudFormation can attach the updated version.

This error in the lambda execution:
[ERROR] Runtime.ImportModuleError: Unable to import module 'lambda_function': No module named 'requests_aws4auth'

Thanks for submiting, can you test with updated runtime? Thanks!


Fix for compatibility with Java version 12+

This commit fixes issues #1342, #1335, #1301, #1257, and #1249
Prior to this change, static analysis of code containing a break statement would fail when running on Java version 12+, with the following error:
BugPattern: Finally
Stack Trace:
java.lang.NoSuchFieldError: com/sun/tools/javac/tree/JCTree$JCBreak.label
...

This was due to a change in the class com.sun.tools.javac.tree.JCTree$JCBreak. In Java 12+, label is no longer a field, and is instead a calculated value. This change modifies com.google.errorprone.bugpatterns.Finally to call JCBreak.getLabel() instead of JCBreak.label.
This change maintains compatibility with prior versions of Java.
On branch 1342-compatibility-java-12
Changes to be committed:
modified:   core/src/main/java/com/google/errorprone/bugpatterns/Finally.java

Fixed in https://github.com/google/error-prone/commit/a3bd193f0597d28aa382e911dfae8c9b57951ac0.


Entrypoint in index.html not rewritten if nested or multiple entrypoints defined

In case of nested entrypoints, e.g. /_dist_/ui/something.js, baseFile gets only filename 'something' and not 'ui/something' to be replaced properly.
In case of multiple entrypoints, the /_dist_/ prefix is omitted from unhashedEntrypoints and isEntrypoint check always fails.

@cbxp i actually plan on ripping this feature out. It's brittle and error prone. There's a lot of edge cases and I think it's easier to just tell the user to read from the generated manifest.
#24 addresses it, I just haven't gotten around to making it happen.


Screen for Learn more about Donating

Describe information about donation, best practices and cautions

Assign me this issue with GSSOC extd 2024, hacktoberfest and level tag.

assign level label after reviewing PR

I see the latest commits were related to hiding the "firebase_options.dart". I do understand the need to hide such information but for the ease of development, can u update the repo with the file.

Assign me this issue with GSSOC extd 2024, hacktoberfest and level tag.

@Reboot2004 Staus?

@prajapatihet  Under Progress

@may-tas you can processed!!!

Thanks, will do.


Changed Stream tests to use httpbin.org instead of google.com

Stream tests were failing for non-US developers because the page at
google.com looks different, depending on the user’s location and
language. The expected string “Search the world's information” seems to
appear only on the US version of google.com.

Current coverage is 74.19% (diff: 88.88%)

No coverage report found for master at 11281ab.

Powered by Codecov. Last update 11281ab...fb0ef79

Thanks :)


Graph generation

I am attempting to use this to analyze chess games represented as graphs. Is it possible to modify this model for graph generation?

Hi! Unfortunately I don't have an experience with graph generation. GPS and other GNNs/GTs could be used for the graph representation, but honestly I would expect CNNs or Transformers with tokenization similar to ViT to be a better fit given the regular structure and small size of the chess board grid.
Best, Ladislav

Oh okay gotcha. Just wondering, what is the input/prediction of your model? Graph transformer novice here 😁

No problem. The input is a graph as a collection of (attributed) nodes and (attributed) edges between them. The output is task-dependent; it can be a global graph property (1 graph -> 1 output), individual node properties (1 graph with N nodes -> N outputs), link prediction (for two nodes in a graph, what is the probability there should be an edge between them?), and other. Typically for the GPS model, we assume so call inductive regime, when we learn on one (large) set of graphs and test on another set of graphs. But there is also a whole world of transductive learning or 1-graph regime, where the input is one (large) graph and we need to generalize from a labeled set of nodes to an unlabeled set of nodes in this single graph. I hope that helps!


SARIF Explorer does not handle multiple issues with the same line and column number well

I have a SARIF file with multiple findings on a single line at the same column. When I double click the first time on an issue on line x, column, then the SARIF explorer updates. However, when I double click on another issue on the same line and column, then the explorer does not update.
VSCode Version: 1.21.1
OS Version: MacOS 10.14
Steps to Reproduce:
Have a SARIF file with several issues with the same line and column number
I have a video with the issue demonstrated, but it won't let me attach. Happy to share video and SARIF file via another method.
SARIF file was exported from CodeSonar.
Does this issue occur when all extensions are disabled?: Irrelevant

Thanks for the bug, I think this is due to a limitation of how we're detecting when an issue is selected. We use CodeActions that gets triggered when a location gets focus that we have a diagnostic at. But we don't have a way to determine which result if there are two at the same location. I don't think there is a way to get the currently selected diagnostic in the problems view. I'll open an issue against vscode to see if they'll be willing to add something to their api to allow us to get this information.
I'll also look into trying to implement something so the ResultsList in the Sarif Explorer is able to handle this scenario.


[ENHANCEMENT] Navigate from python file robot file

When I use "Find All References" from a function implemented in a python file, the usages in robot files are not found. Is this feature implemented? If so, is there something to configure? If not, could this feature be implemented?
Note: I'm able to navigate from robot file to python file with "Go to Definition"

VS Code: 1.92.2
RobotCode: v0.87.0
OS: Linux
Python: 3.11.9
RobotFramework: 6.1.1

No this is not possible, because the RobotCode language server only knows robot files and if you import a library robotframework only get line numbers from python.
The trick here is, instead going to the python and try to find all references from here, you can also use "Find All References" from a keyword call in a robot/resource file.

Thx for the quick reply. I know that I can still workaround this by using search tool, or going to a usage in a robot file and using "Find All References", but this is still a bit frustrating. Is there any chance to see this feature implemented? Is it conceivable that the language server listens to python files only for this purpose?

Searching for Python files is easy, but finding the exact position of a declaration can be tricky since the RobotFramework Libdoc tool, which RobotCode uses internally, only provides the start line numbers of the methods/functions in the Python file. Additionally, there are dynamic libraries like remote libraries that do not provide a line number and so on.
I have changed this issue to an enhancement so that we don’t forget about it and can take a closer look at it later.


Use localstorage to save options

This small change just incorporates the usage of the browser's local storage to remember and restore a handful of options for a minor increase to the overall quality of life.
Namely it saves and restores:

System memory size
Video memory size
The networking proxy that's configured (saving this will be rather nice if testing a local one)
The boot order
If audio is enabled
If ACPI is enabled

Worst case, this doesn't save info in a way that someone finds helpful and they have to change the value as they would have normally. Best case, people re-use the config for a bunch of different testing and this change saves them a bit of clicks / keystrokes.

Good idea, thanks!

Cheers!


Added a new pattern

Added C++ pattern of a hollow triangle inscribed in a rectangle

@Red-0111 I'm closing the PR as it is not fulfilling the requirement...you can send a PR again after maintaining all the steps and eligibility criteria for a valid PR...


AvcConfig Usage

Hi there!
I am trying to combine a .png image and .m4a audio file and am not sure what is required for pic_param_set/seq_param_set. Could you possibly provide a pointer please?
PS - I assume the image/audio data is added to the resulting mp4 via the mp4_writer.write_sample function?
Thanks!

Hey @EVLRBT-01, check out the mp4copy example for an idea of setting the sps/pps in AvcConfig by using the bytes extracted from the track data.

Closing. Feel free to respond with any questions.


Divide into multiple packages and monorepolize

Is your feature request related to a problem? Please describe.

In developing #551, there is a problem that the amount of types exposed from a single package increases too much.
When updating packages in @ts-graphviz, it takes time to fix between repositories and there is a lag.

Describe the solution you'd like

[ ] Divide the ts-graphviz package by layers.
[ ] In addition, we will start operation with monorepo.

Challenged this issue, but the configuration became more complex and did not improve maintainability.
Close due to policy change.


Update limit for neighbours in a given state depending on group size

Still need clarification from Peter and Chris for the best way round this but currently with a default group size of 5, the condition for number of neighbours being > 5 is never going to be met.
Current suggestion is: for a pupil in passive state:

If pupil has 5 or more neighbours, change to red/green if 5 or more neighbours are red/green
If pupil has < 5 neighbours, change to red/green if all neighbours are red/green

Working on this on the logic-updates branch.