Model Inferencing

Hi, I'm using clone detection in GraphCodeBERT and I'm trying to do model inference from the saved checkpoint. I'm trying to load the saved model from a folder called "roberta" with all the pre-requisits that I got from hugging-face transformers. While Inferencing using the code given to inference "!python run.py --output_dir=saved_models --config_name=roberta --model_name_or_path=roberta --tokenizer_name=roberta --do_eval --train_data_file=dataset/train.txt --eval_data_file=dataset/valid.txt --test_data_file=dataset/test.txt --epoch 1 --code_length 128 --eval_batch_size 4 --learning_rate 2e-5 --max_grad_norm 1.0 --evaluate_during_training" So while running this command that I'm trying to load from the folder "roberta". I'm facing an error as "ValueError: The state dictionary of the model you are training to load is corrupted. Are you sure it was properly saved?". I load the model separately and try to print it. The model is in OrderedDict instead of StateDict. I tried rerunning and fine tune the model once again to save the model correctly and I see it doing the same things and saving in an OrderedDict format. And maybe this is causing the error.

PS:
What I'm trying to do is, Given two pieces of code in the dataset.json and the test.txt when all the indexes and actual predictions are mentioned I need to to inference on the saved model to give me the prediction whether it is similar or not. Either 0 or 1. How do I go about this, If possible give me a brief on how to do the process only for the above mentioned problem.

Do you change saved path? "roberta" folder is a pre-trained model and you can't save the model to this folder when you fine-tune the model.
please follow this https://github.com/microsoft/CodeBERT/tree/master/GraphCodeBERT/clonedetection and results will be printed to saved_models/predictions.txt (https://github.com/microsoft/CodeBERT/blob/edc05c303747c1e86a6ed886279248973e923ab8/GraphCodeBERT/clonedetection/run.py#L510)

Do you change saved path? "roberta" folder is a pre-trained model and you can't save the model to this folder when you fine-tune the model.
please follow this https://github.com/microsoft/CodeBERT/tree/master/GraphCodeBERT/clonedetection and results will be printed to saved_models/predictions.txt
https://github.com/microsoft/CodeBERT/blob/edc05c303747c1e86a6ed886279248973e923ab8/GraphCodeBERT/clonedetection/run.py#L510

So after fine tuning the model, I downloaded the fine tuned model and added it to the roberta folder and renamed it  to "pytorch_model.bin". Now after that I'm trying to inference the model to do eval from the folder "roberta"

You can't replace "roberta/pytorch_model.bin" by fine-tuned model, and you need to keep roberta folder unchanged. Here will reload fine-tuned model
https://github.com/microsoft/CodeBERT/blob/edc05c303747c1e86a6ed886279248973e923ab8/GraphCodeBERT/clonedetection/run.py#L614-L616

You can't replace "roberta/pytorch_model.bin" by fine-tuned model, and you need to keep roberta folder unchanged. Here will reload fine-tuned model
https://github.com/microsoft/CodeBERT/blob/edc05c303747c1e86a6ed886279248973e923ab8/GraphCodeBERT/clonedetection/run.py#L614-L616

So, I was able to load and run the fine-tuned model for inferencing for evaluation only, using the following command "!python run.py --output_dir=saved_models --config_name=microsoft/graphcodebert-base --model_name_or_path=microsoft/graphcodebert-base --tokenizer_name=microsoft/graphcodebert-base --do_eval --train_data_file=dataset/train.txt --eval_data_file=dataset/valid.txt --test_data_file=dataset/test.txt --epoch 1 --code_length 128 --eval_batch_size 4 --learning_rate 2e-5 --max_grad_norm 1.0 --evaluate_during_training". I'm running this on the dataset provided in the github itself. So I have created my own dataset with func and idx and created valid.txt also. So how do I evalute this on the model. and generate predictions.txt on clone detection of graphcodebert

You can use the same format to save your dataset. And then replace our dataset by your own dataset.

I need to create the dataset in python and I'm facing few errors when done and being tested on the model. json.decoder.JSONDecodeError: Expecting ',' delimiter: line 1 column 414 (char 413 This one and an nother jsondecodeerror so. Can you please let us know. Or provide a simple code here lets say for adding two numbers and subracting two numbers in python. maybe if you provide this simple input it will be a lot helpful for me to create the custom data and test it on the model
ps. i tried following the same instructions and created the dataset
find below an example of what i have created
{"func": "    public class MinimumValueExample \n {\n   \n static int findMin(int inputArr[], int index, int size)\n {\n     \n     if(size == 1)\n     {\n         return inputArr[index];\n     }\n     \n     return Math.min( inputArr[index], findMin(inputArr, index + 1, size - 1));\n }\n public static void main(String argvs[]) \n {\n     int numArr[] = { 27, 56, 90, 12, 120, 263 }; \n     System.out.println("The input Array is : " + Arrays.toString(numArr));\n     int length = numArr.length; \n     int minVal = findMin(numArr, 0, length); \n     \n     System.out.print("Minimum element of the input array is: " + minVal + " \n ");\n 	}\n }", "idx": "1212121"}

Your data is not correct json format. Please use json.dumps to dump your each func. Here is an example:
>>>import json
>>>js = {"func": """public class MinimumValueExample \n {\n \n static int findMin(int inputArr[], int index, int size)\n""", "idx": "123"}
>>>f = open("valid.txt","w")
>>>f.write(json.dumps(js)+"\n")

So I dumped as you mentioned above and got rid of the json error. Now I'm getting a Value error

I have only 3 values in my text.txt file

and i have successfuly entered the code that i wanna test into the data.jsonl file too

after all this when i run the comman !python run.py --output_dir=saved_models --config_name=microsoft/graphcodebert-base --model_name_or_path=microsoft/graphcodebert-base --tokenizer_name=microsoft/graphcodebert-base --do_test --train_data_file=dataset/train.txt --test_data_file=dataset/test.txt --epoch 1 --code_length 256 --data_flow_length 64 --learning_rate 2e-5 --max_grad_norm 1.0 --seed 123456 2>&1| tee custom_saved_models/test.log
I'm getting this valueerror. how shall i solve it

For test.txt file, please use Tab, i.e \t, to separate them instead of space key.

So, I saved the finetuned model and I am trying to load and use it and I'm getting the following error
ValueError: The state dictionary of the model you are training to load is corrupted. Are you sure it was properly saved?

What to do to load the saved model and use it while inferencing

I suggest you carefully read README and you will find the answer. Don't change any code you show. If you want to reload fine-tuned model from other path, just change here:
https://github.com/microsoft/CodeBERT/blob/3e585728c9b7391495a98fe7b044e7be4818713f/GraphCodeBERT/clonedetection/run.py#L614-L615


[Enhancement] Remove project naming and examples

When we run the command there are some names that are related to some of our projects.
What's the name of your package? Ex: (service he hst results api)

We must remove that and add a more open-source example. This it should work for non Guidesmiths projects 👍
What's the name of your package? Ex: (My service api)

https://github.com/infinitaslearning/infinitas-hygen-template-generators/commit/b858724a2c48414d980a8184ee3bc3b91012d1f1


Revert cryptography version bump

To avoid further disruption in our rollouts and to take us back to a consistent deployment state, we have decided to revert our Python cryptography package version bump. That bump was not consistent and left us w/ old versions of the package on many physical machines (at least 350) and any Ubuntu 16.04 virtual machine we create.

Release Note Category

[x] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
We undid the most visible part of the dotnet/arcade#13186 to address rollout issues between now and when we believe we will have a complete solution to the underlying problem. In about two weeks, we hope to be using a current cryptography package on all physical machines and any VM we create.

!32777 fix has rolled out. #418 covers restoring the cryptography version bump as soon as we can — after #377 and #379 work is done.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
commit add 3 lines to make username cannot start with '.' and '_'
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Thank you!


Add a way to check Track's peak amplitude per channel

Oh, we make sure there's only one class/struct per file. Please move TrackAmplitudes to a new file or nest it inside Track


What is the schema of barcodes_<BAM_FILE>.bam.tsv file?

We've generated a BAM file from our DropSeq analysis following the instructions here, tweaking dropest parameters to fix any issues that came up on the way, as described here.
The issue we are experiencing just now, and we can't fix, has to do with generating a file with the corrected barcodes, which in our case is named "barcodes_result.bam.tsv".
We attempt to generate the file using the command velocyto tools dropest-bc-correct result.bam dropEst_out.rds
The command finishes successfully, but the "barcodes_result.bam.tsv" has no content. The log produced is:
velocyto tools dropest-bc-correct result.bam dropEst_out.rds
2018-12-11 06:24:44,795 - INFO - Loading `merge_targets` from /home/picrin/programming/cancer_velocyto/dropEst_out.rds using R / rpy2
2018-12-11 06:24:45,465 - INFO - Generating /home/picrin/programming/cancer_velocyto/barcodes_result.bam.tsv
2018-12-11 06:25:18,116 - INFO - Done

We would like to generate this file ourselves, but we would have to know what is the correct schema of such file. Could someone share this information?

Sorry for the problem. I thought that with the latest version of dropest that command is not needed anymore... The schema is the same as for 10X barcodes list file. It is just one barcode per line, barcodes should be the same  as they appear in the bam cellbarcode tag. (hint: you coukd try to have a look at the source of the command if you have doubts on what is going wrong)BestGioele-------- Original Message --------Subject: [velocyto-team/velocyto.py] What is the schema of barcodes_.bam.tsv file? (#162)From: picrin To: "velocyto-team/velocyto.py" CC: Subscribed We've generated a BAM file from our DropSeq analysis following the instructions here, tweaking dropest parameters to fix any issues that came up on the way, as described here.
The issue we are experiencing just now, and we can't fix, has to do with generating a file with the corrected barcodes, which in our case is named "barcodes_result.bam.tsv".
We attempt to generate the file using the command velocyto tools dropest-bc-correct result.bam dropEst_out.rds
The command finishes successfully, but the "barcodes_result.bam.tsv" has no content. The log produced is:
velocyto tools dropest-bc-correct result.bam dropEst_out.rds
2018-12-11 06:24:44,795 - INFO - Loading merge_targets from /home/picrin/programming/cancer_velocyto/dropEst_out.rds using R / rpy2
2018-12-11 06:24:45,465 - INFO - Generating /home/picrin/programming/cancer_velocyto/barcodes_result.bam.tsv
2018-12-11 06:25:18,116 - INFO - Done
We would like to generate this file ourselves, but we would have to know what is the correct schema of such file. Could someone share this information?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

Thank you @gioelelm. I'm still confused about the "barcodes_result.bam.tsv". We are using the latest version of dropest (downloaded from the github repository), and we generate our bam file with:
-g <our-gtf-file>.gtf -o dropEst_out -L eiEIBA -c drop_seq_velocyto.xml <our-DropSeq-BAM-file>.bam
According to your information, the resulting ".tagged.bam" file should be directly usable by the velocyto run-dropest. However, when we try this, we get an error that "barcodes_result.bam.tsv" couldn't be found.
Maybe the velocyto run-dropest pipeline needs to be modified to accept input without enforcing the barcodes file?
For the immediate solution, does the barcode file need to contain a set of unique barcodes, or a list of barcodes corresponding to reads from the BAM file?

I assumed a set of unique barcodes, and was able to get past the error regarding the tsv file.
However, a new error now appears, which is due to argument parsing. Running velocyto run-dropest correct_result.bam yang38.gtf gives:
2018-12-11 08:00:05,221 - INFO - Attempting to find automatically the valid barcode list file /home/picrin/programming/cancer_velocyto/barcodes_correct.tsv
2018-12-11 08:00:05,221 - INFO - /home/picrin/programming/cancer_velocyto/barcodes_correct.tsv found
Traceback (most recent call last):
  File "/home/picrin/programming/Ruibin_scNGS/venv/bin/velocyto", line 11, in <module>
    sys.exit(cli())
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/velocyto/commands/run_dropest.py", line 104, in run_dropest
    samtools_memory=samtools_memory, dump=dump, verbose=verbose, additional_ca=additional_ca)
  File "/home/picrin/programming/Ruibin_scNGS/venv/lib/python3.6/site-packages/velocyto/commands/_run.py", line 54, in _run
    raise IOError(f"Something went wrong in the argument parsing. You passed as bamfile: {bamfile}")
OSError: Something went wrong in the argument parsing. You passed as bamfile: /home/picrin/programming/cancer_velocyto/correct_result.bam

Is there a known fix for the problem?

I can't spot the problem without a deeper check. Maybe somebody from @pkharchenko can spot the problem.
My suggestion for now is just using the normal run command.

It turns out the problem manifests itself in line 49 in _run.py
https://github.com/velocyto-team/velocyto.py/blob/20eee627e78b62444c2656e041668f48ac49dcb8/velocyto/commands/_run.py#L49
Due to a suspected bug earlier in the pipeline the variable bamfile is of type string as opposed to of type tuple. A hotfix is to comment out all the lines 49 -- 54 and replace them with
bamfile = [bamfile]
multi = False

However, there are further issues in the pipeline, in line 233:
FileNotFoundError: [Errno 2] could not open alignment file `/home/picrin/programming/cancer_velocyto/cellsorted_correct_result.bam`: No such file or directory

The problem was with samtools version, the samtools version I used did not support the -t switch.


Picsure 301 audit log nullpointer exception bug

picsure 301 audit log nullpointer exception bug fix

Not big changes here, already deployed and used for a 10 days, need this change in master, will merge it by myself

Also, please let me know when you built a new image from it?! Pretty please!
From: YuAvillachLab<EMAIL_ADDRESS>Reply-To: hms-dbmi/pic-sure-auth-microapp<EMAIL_ADDRESS>Date: Monday, May 20, 2019 at 4:21 PM
To: hms-dbmi/pic-sure-auth-microapp<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [hms-dbmi/pic-sure-auth-microapp] Picsure 301 audit log nullpointer exception bug (#47)
Not big changes here, already deployed and used for a 10 days, need this change in master, will merge it by myself
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_hms-2Ddbmi_pic-2Dsure-2Dauth-2Dmicroapp_pull_47-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAEJHQSKAYCJ6OSAWGB42UY3PWMB4TA5CNFSM4HLWSTS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVZ6YAI-23issuecomment-2D494136321&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=pijzILWzyE5fbgehP7mrMWRDVS5IVXAsHSkDzMRm1qM&m=RczC4_6HAGNhgmhhBnxBXZqEODSxOCqml8m5suUlnK4&s=XLeaUDjm9QbktYPNP7eMyQrj-mVqL2nwDeLUcRUZmcg&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AEJHQSMFJTIU4X5CQN4BBRTPWMB4TANCNFSM4HLWSTSQ&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=pijzILWzyE5fbgehP7mrMWRDVS5IVXAsHSkDzMRm1qM&m=RczC4_6HAGNhgmhhBnxBXZqEODSxOCqml8m5suUlnK4&s=h7Ru5wSaG5OVikDDm5gdSeweEymWuJG5eiLSOUizkyg&e=.

pushed to dbmi/pic-sure-auth-services:master.4f2c799


added height style to chosen pane in DLS

Fixed an issue that changed the height of the chosen pane when adding / removing packages.

Codecov Report

Merging #232 (0070c6d) into master (afb34b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #232   +/-   ##
=======================================
  Coverage   30.67%   30.67%
=======================================
  Files          95       95
  Lines        1767     1767
  Branches      491      491
=======================================
  Hits          542      542
  Misses       1123     1123
  Partials      102      102

Impacted Files
Coverage Δ

src/components/form/Packages.js
41.17% <ø> (ø)

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 afb34b2...0070c6d. Read the comment docs.


Draft: Kc po9 switching to keyvalue

Jira ticket for this change
Summary
This change has the API remove escape characters and returns json as object as is
Setup to Run Your Code
TODO

@mergify-bot rebase

@mergifyio rebase


[openssl] replace the flaky strawberry perl distribution

Is your feature request related to a problem? Please describe.
I'm always frustrated when I try to install the openssl port and it fails due to vcpkg not being able to download the Strawberry Perl distribution (on Windows®).
Proposed solution

It would be neat if someone can get openssl to build with the perl that comes with git.

@BillyONeal https://github.com/microsoft/vcpkg/issues/28249#issuecomment-1344875851
Describe alternatives you've considered
The strawberry perl distribution (or the necessary subset) could be mirrored, but this is not considered feasible due to licensing issues.
Obviously, there are other Perl-distributions for Windows® out there which could (should?) be evaluated. However, I don't know any one which fits our criterias (e.g. ActiveState Perl (which is recommended by the OpenSSL wiki) requires an account to download).
Additional context
This is a follow-up on #28249

I took a look at their website issue tracker and it seems that similar [availability] issues have been reported there a few times within the last months:

https://github.com/StrawberryPerl/strawberryperl.com/issues/30
https://github.com/StrawberryPerl/strawberryperl.com/issues/29
https://github.com/StrawberryPerl/strawberryperl.com/issues/23

Solutions to this problem have previously been discussed here:

https://github.com/microsoft/vcpkg/issues/14280#issuecomment-757547441
https://github.com/StrawberryPerl/Perl-Dist-Strawberry/issues/21

Regarding the use of the Perl distribution bundled with git, @dg0yt noted that:

For a general replacement, the one which comes with msys2 would be nice, to avoid mixing msys2 runtimes in other ports. But IIRC openssl didn't behave well with msys2 perl when I tried to use it for mingw.
From other issues we also know that openssl needs one particular Perl module, IPC/Cmd.pm.

I am the original creator of Strawberry Perl, and although I'm no longer involved in it's maintenance I was just made aware of this issue and that it's distribution via a regular web hosting account has been causing problems for use cases like this.
With regards to licensing issues, I'm not sure what your problems are, but I'd be happy to help assist or give permission in any way I'm capable of that might help you fork/embed parts of it.
The build code that assembles the Strawberry Perl distribution is probably fairly easy to fork of sub-class, and I imagine it could be retooled to drop all the extra weight and just include the one module you need. There's an internal "Vanilla Perl" distro without all the extra bundled stuff that might serve as a suitable starting point.

Hi, thank you for the insights!

With regards to licensing issues, I'm not sure what your problems are, but I'd be happy to help assist or give permission in any way I'm capable of that might help you fork/embed parts of it.

This can only be answered by @BillyONeal who investigated this initially (and hasn't explicitly written down the exact issues anywhere AFAIK). Sadly he is a rather busy person, so I would expect an answer later rather than sooner 😅 @Cheney-W would you mind pinging him?

AFAIU the issue is fairly clear: Download. The official download servers  were unavailable more often than expected. vcpkg has only private asset caching, i.e. each user needs to download assets from official sources at least once.

AFAIU the issue is fairly clear: Download. The official download servers were unavailable more often than expected. vcpkg has only private asset caching, i.e. each user needs to download assets from official sources at least once.

Yes, that's the root issue. One proposed solution was to mirror the strawberry perl download with Microsoft / GitHub resources. This however was thought to be infeasible due to said licensing issues.

One proposed solution was to mirror the strawberry perl download with Microsoft / GitHub resources.

Strawberry Perl 5.38.0 and 5.36.1 have already been released/hosted on GitHub (64 bit only)
https://github.com/StrawberryPerl/Perl-Dist-Strawberry/releases
(32 bit request) is noted in https://github.com/StrawberryPerl/Perl-Dist-Strawberry/issues/96
The main site of https://strawberryperl.com/ hasn't been updated yet.
reference: https://github.com/StrawberryPerl/strawberryperl.com/issues/34#issuecomment-1642817144
@BillyONeal @Cheney-W Have a look through below links:
https://github.com/StrawberryPerl/strawberryperl.com/issues
https://github.com/StrawberryPerl/Perl-Dist-Strawberry/issues


Annotations zip file doesn't contain wider_val.txt

Steps to reproduce :

Downloaded the annotations zip file from dropbox :
https://www.dropbox.com/s/7j70r3eeepe4r2g/retinaface_gt_v1.1.zip?dl=0

Couldn't find val/wider_val.txt which is neccessary for running test_widerface.py

@yossibiton  Did you find wider_val.txt and label.txt?

@yossibiton you can generate by youself with following code
`from glob import glob
val_imgs_dir = '/workspace/Pytorch_Retinaface/data/widerface/val/images/'
txt_filename = '/workspace/Pytorch_Retinaface/data/widerface/val/wider_val.txt'
filenames = glob('/workspace/Pytorch_Retinaface/data/widerface/val/images//.jpg')
with open(txt_filename, 'a+', encoding='utf-8') as f:
for filename in filenames:
f.write(filename.replace(val_imgs_dir, '')+'\n')`


Python: Prevent divergence in type-hint analysis. (ODASA-8075)

This fixes an unfortunate (but highly specific) divergence in the points-to analysis.
The following piece of code is a (more or less) minimal example that exhibits the divergence:
class foo():
    pass

while True:
    foo = foo[foo]

The culprit is the support for subscripted types originally introduced in #1728. Essentially, given the above, foo (the variable) can point to the subscripted type foo[foo], but then also the subscripted type (foo[foo])[(foo[foo]) and so on. Thankfully, only two projects on LGTM.com exhibited the above divergence.
To fix this, we block the recursion by requiring that the two types involved in a type subscripting operation (A and B in A[B]) must not themselves be subscripted types.
As this fixes an (obscure) regression in the analysis, I don't think it needs a change note.
Performance comparison here: https://git.semmle.com/gist/taus/8c8f31f2bb72055e50cc97bcce070747
Looks like a ~30% increase in performance across the board.

As discussed, I've added a test case to show what happens when nested types are encountered. My main worry was that something like List[List[int]] would be ignored by the analysis, since it contains a nesting (and thus not be recognised as a subscripted List type). This is not the case, however, as the tests show. Instead it gets recognised as a subscripted List type where the subscript is unknown. I think this is fine -- the main point of the subscript handling is to ensure that List[int] gets analysed as having the class List, and that'll continue to be the case even with the changes in this PR.


typo: machinelearning/resource-manager/Microsoft.MachineLearning

capabilites -> capabilities
Definiton -> Definition
asets -> assets
Delere -> Delete
paramater -> parameter

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-js

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

@AutorestCI rebuild azure-sdk-for-node please

Encountered an unknown error
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 33, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/bot_framework.py", line 105, in manage_comment
    response = getattr(self.handler, orderstr)(webhook_data.issue, *split_text)
TypeError: rebuild() takes from 2 to 3 positional arguments but 4 were given

Encountered an unknown error
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 33, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/bot_framework.py", line 105, in manage_comment
    response = getattr(self.handler, orderstr)(webhook_data.issue, *split_text)
TypeError: rebuild() takes from 2 to 3 positional arguments but 4 were given

Encountered an unknown error
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 33, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/bot_framework.py", line 105, in manage_comment
    response = getattr(self.handler, orderstr)(webhook_data.issue, *split_text)
TypeError: rebuild() takes from 2 to 3 positional arguments but 4 were given

@AutorestCI regenerate azure-sdk-for-node

Encountered a Subprocess error
Command: ['/usr/local/bin/autorest', '/tmp/tmpes7ij3q6/rest/specification/machinelearning/resource-manager/readme.md', '--license-header=MICROSOFT_MIT_NO_VERSION', '--node-sdks-folder=/tmp/tmpes7ij3q6/azure-sdk-for-node', '--nodejs', '--use=<EMAIL_ADDRESS>Finished with return code 1
and output:
AutoRest code generation utility [version: 2.0.4283; node: v8.12.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
   Loading AutoRest core      '/root/.autorest/@microsoft.azure_autorest-core<EMAIL_ADDRESS>(2.0.4289)
   Loading AutoRest extension<EMAIL_ADDRESS>(2.1.119->2.1.119)
   Loading AutoRest extension<EMAIL_ADDRESS>(2.3.44->2.3.44)
[Exception] No input files provided.

Use --help to get help information.


Adding Open Collective contributors

Hi, I'm making updates for Open Collective. Either you or another core contributor signed this repository up for Open Collective. This pull request adds financial contributors from your Open Collective https://opencollective.com/expand ❤
What it does:

adds a badge to show the latest number of financial contributors
adds a banner displaying contributors to the project on GitHub
adds a banner displaying all individuals contributing financially on Open Collective
adds a section displaying all organizations contributing financially on Open Collective, with their logo and a link to their website

P.S: As with any pull request, feel free to comment or suggest changes.
Thank you for your great contribution to the Open Source community. You are awesome! 🙌
And welcome to the Open Collective community! 😊
Come chat with us in the #opensource channel on https://slack.opencollective.com - great place to ask questions and share best practices with other Open Source sustainers!

eXpand.lab release 19.1.402.16 includes commit that relate to this task:

[Adding Open Collective contributors (#472)

Adding Open Collective contributors](https://github.com/eXpandFramework/eXpand.lab/commit/aa59888cb8b3814e21003541b98f637301ada1c3)
Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
Thanks a lot for your contribution.


docs about admin's list_display short_description

Merged in 8ab8a8910c6f9382610cba5b06ac5c110d52b4b2, thanks! (Since there are already examples of using short_description I omitted the additional one.)


Added Butterfly Hexagon Button

Fixes Issue
Closes #1515

Changes proposed
Added Butterfly Hexgon Button

Screenshots
https://github.com/Spyware007/Animating-Buttons/assets/119161467/bfe957a4-9c77-4e48-80ad-b03aa4811b7e

Checklist

[x] You've linked this PR to the correct issue.
[x] You have checked that the code is working correctly.
[x] You ⭐ the repository!

Note to reviewers

@Souravvmishra please merge my pull request

@adity2911 please merge this pull req

@TANYA-CHAN  @adity2911  @Souravvmishra  @Spyware007 I added button in the website directly so please my pull request

https://github.com/Spyware007/Animating-Buttons/assets/119161467/a964e2df-8744-4549-a6f7-491c93031a8a


Can't format Date

When I try to add a date object to my excel rows I keep getting the following error:
undefined method `-' for "30-10-2015":String

I add rows to my workbook with the following code:
sheet.add_row ['Time:',i.invoice_date], :types => [:string, :date]

i.invoice_date is a datetime object, it sounds strange to me that this causes an error.
Thanks in advance!

Was there a solution for this?

@thomasvbeek what is the solution?

I could not reproduce this issue.
But, this will happen if i.invoice_date is actually String.
It may be good to check object.class, if you face this problem.


Fancy Zones: Zone Tabs for multiple docked windows

Description of the new feature / enhancement
Put a tab bar at the top of each zone which has multiple windows docked into it.  This is similar to the Sets feature that was tested and cancelled by MS, or Groupy by stardock, except it's tied to the Zone, not a window.
Scenario when this would be used?
Give more info at a glance what window is docked where.
Supporting information
No response

Thanks for your message. This is a Duplicate of an other issue. Additional comments may be added there. /dup #15921 or #12696 (and 14982)


doc: develop: tools: CLion: Add update note

CLion now offers a more native approach to working with Zephyr projects. We would like to inform users about this.

@olegznv will you be able to look into squashing the commits? I understand you've made the contribution through the GitHub web UI, so you'll have to work from a local clone to do this. Let me know if I can help

@kartben I'll try to do this. Thanks!

@kartben I've squashed commits. Please check


Generate image assets from layers in UXP plugins

Hello,
Is it possible to generate images from layers (or from folder that contains many layers) in UXP panels?
If yes - where can I find the documentation about this feature?

Via Kerri to Chuck: We have already have an example in the prerelease forums. Let's make it public :)

@html5maker What kind of output are you looking for? Also, is this you? (Timing seems similar.)

@html5maker What kind of output are you looking for?

Looking for PNG/JPEG output (not locally but prepare for sending to another service using api together with layer config).
Would be also great if it's possible to export SVG for vector paths.

Also, is this you? (Timing seems similar.)
Not me :)

Any updates on this?

Via Kerri to Chuck: We have already have an example in the prerelease forums. Let's make it public :)

Can you please at least provide some info in this thread?
We can't update our extension to UXP simply because the info about image generation is missing in your docs.

Via Kerri to Chuck: We have already have an example in the prerelease forums. Let's make it public :)

Can you please at least provide some info in this thread?
We can't update our extension to UXP simply because the info about image generation is missing in your docs.

@kerrishotts @ashryanbeats can you please provide some indo/documentation about exporting layers using UXP?

@kerrishotts @ashryanbeats can you please provide some indo/documentation about exporting layers using UXP?

Please ask on the forum here: https://forums.creativeclouddeveloper.com/c/photoshop/uxp-plugin-api/64; it's going to be a while before we get an official sample, but others are doing this and may be willing to share their code.

is UXP project dead?

It's not dead at all!

@html5maker Did you find any solutions?

@Valeryia-Hrabnitskaya unfortunately haven't found any info about this...

You should probably ask in our new UXP developer forums. Here's a thread that might be related.

@ErinFinnegan I created new isuue but I haven't got any answers yet

@ashryanbeats can you provide with docs about generate image from layers?

@ashryanbeats left Adobe a couple months ago... but @pklaschka is working with us this summer! I'm hoping he can help out with this.

Thanks.
@pklaschka can you provide with docs about generate image from layers? Or when this doc will be documented?

@ErinFinnegan @pklaschka @kerrishotts
Any update on this?
It was mentioned by @ashryanbeats that you already have an example in pre-release forum.

Hi @html5maker,
I can't find whatever @ashryanbeats was mentioning in the Prerelease forum.
The thing you need for this is APIs for pixel manipulation, specifically what we've been referring to internally as "layer renditions," which are a popular request. These APIs are on the way. We'll be updating the Photoshop extensibility roadmap soon. Keep an eye on the developer forums or our developer newsletter for updates.


Non-interactive login command.

Works only with 2FA disabled.

This would mean entering a password on the command-line, which would remain in shell history, etc., in plain text. Maybe that's fine on an otherwise secure environment, but that's not something we can assume.
There is "API token" support for this reason (see https://github.com/platformsh/platformsh-cli#customization).

Thanks! I guess this makes it PR useless.


[WEB-UI]Add the 'master' column to identify the type of resource management for the spark job, in the history server web ui.

What changes were proposed in this pull request?
Add the 'master' column to identify the type of resource management for the spark job, in the history server web ui.
such as:
      --master yarn, which means that the resource type of the blurs job is yarn.
      --master ip: 7077, that the spark resource management type is standalone.
      --master local, which indicates that the resource type of the blurs job is local.
Screenshots are as follows:

(Please fill in changes proposed in this fix)
How was this patch tested?
manual tests
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Why? It would be identical in almost all use cases.

Sorry, I did not understand what you meant.

What's the value of this column? In just about all cases it will have the same value in every row, like 'yarn'. Even if not, why is it useful?

1.It is more convenient for the observer to more easily know the resource scheduling mode of the current spark application.
2.For the 'Spark User' column. It would be identical in almost all use cases.But it still very important.So I think the 'master' column is equally important, This shows the content more clearly.

What does knowing the resource scheduler help? but my point is, it's already known in general.
A history server most certainly collects history from different users' apps, but not from different resource managers. It's possible to mix history logs from different ones but it's exceptional.

For example, I would like to view some of the applications of the resource schedule of yarn, I can sort the 'master' column.

This still isn't addressing the point: when would they not all be YARN?

Sorry，I do not understand what you mean.

When would the history server receive events from clusters with different resource managers?

When the application starts, spark will generate an app directory in the event log directory.

I don't know how else to ask the question. I'd close this if there's no answer to 'why' this is of use.

spark.mesos.dispatcher.historyServer.url
Set the URL of the history
server.  The dispatcher will then link each driver to its entry
in the history server.
spark.yarn.historyServer.address
The address of the Spark history server, e.g. host.com:18080. The address should not contain a scheme (http://). Defaults to not being set since the history server is an optional service. This address is given to the YARN ResourceManager when the Spark application finishes to link the application from the ResourceManager UI to the Spark history server UI.
For this property, YARN properties can be used as variables, and these are substituted by Spark at runtime. For example, if the Spark history server runs on the same node as the YARN ResourceManager, it can be set to ${hadoopconf-yarn.resourcemanager.hostname}:18080

I'll try to clarify @srowen issue for you @guoxiaolongzte
For most use-cases each Spark cluster has it's own history server and also uses one type of resource manager. Therefore for most history servers all the applications would use the same resource manager.
This doesn't mean one history server can't have apps with multiple resource mangers, but @srowen is arguing that it's a rare case. I would tend to agree with him. I think this feature might help cluster admins who collect lots of event logs to sift through on a history server, but they're not the average user.

Agree with Sean here, in the vast majority of cases the master of all applications will be the same, because it's very uncommon for the same cluster to use different cluster managers, and for different clusters to share the same SHS.
Even if that were the case, having this information in the application list isn't very useful by itself. If you're really looking for that, you can see that in the app's environment page.

For the 'Spark User' column. It would be identical in almost all use cases.

That is absolutely not true in a true multi-user environment.

Ok, Thank you for your comments.


feat: add timestamp to dependency if not specified

re-rev of #804

Reopening for release tracking for 1.0.22 legacy release


[BUG] Add  insufficient disk scheduling message

Describe the bug
we need to make an enhancement from the back-end side to report the volume info correct via pod instead leave it for the UI to grasp the events info and identify it manually, this should be a general k8s enhancement issue.
VM view:

pod view:

events:

To Reproduce
Steps to reproduce the behavior:

create VM with insufficient disk size e.g 100000Gi

close as duplicate of https://github.com/harvester/harvester/issues/1695


Local workers can stay alive if parent panics

I noticed a case where I wrote some bad code in tes-worker, which caused a panic. But the pool/slots had already started and when the parent died, the worker subprocesses stayed running. There should be a way to catch a panic at the top-level, run some cleanup code, and return a nice error message with the panic maybe.

The local worker is now in the same process as a goroutine


Fix handle jest v27 fake timers (issue #30)

The existing check for jest fake timers no longer works as jest v27 mutates the setTimeout global differently. This patch checks for both the old and new formats.

Thanks, do you think this works too? https://github.com/TheBrainFamily/wait-for-expect/pull/37
I like that it also upgrades the jest so the tests verify the change

Thanks, do you think this works too? #37
I like that it also upgrades the jest so the tests verify the change

👍 Looks like a better PR. You should merge that one.


launchy-2.4.3 does not work with generate_html()

require 'rbplotly'

x = [0, 1, 2, 3, 4]
trace0 = { x: x, y: [0, 2, 1, 4, 3], type: :scatter, mode: :lines }
trace1 = { x: x, y: [4, 1, 3, 0, 2], type: :scatter, mode: :'markers+lines' }
data = [trace0, trace1] # data must be an array

layout = { width: 500, height: 500 }

plot = Plotly::Plot.new(data: data, layout: layout)

plot.layout.height = 300 # You can assign plot's attributes.

plot.generate_html(path: './line_chart.html')

fails with
Traceback (most recent call last):
        5: from hoge.rb:14:in `<main>'
        4: from /usr/local/bundle/gems/rbplotly-0.1.2/lib/plotly/offline/exportable.rb:20:in `generate_html'
        3: from /usr/local/bundle/gems/launchy-2.4.3/lib/launchy.rb:29:in `open'
        2: from /usr/local/bundle/gems/launchy-2.4.3/lib/launchy/applications/browser.rb:78:in `open'
        1: from /usr/local/bundle/gems/launchy-2.4.3/lib/launchy/applications/browser.rb:67:in `cmd_and_args'
/usr/local/bundle/gems/launchy-2.4.3/lib/launchy/applications/browser.rb:63:in `browser_cmdline': Unable to find a browser command. If this is unexpected, Please rerun with environment variable LAUNCHY_DEBUG=true or the '-d' commandline option and file a bug at https://github.com/copiousfreetime/launchy/issues/new (Launchy::CommandNotFoundError)

~# ruby -v
ruby 2.6.1p33 (2019-01-30 revision 66950) [x86_64-linux]

The cause of error was due to running this in no web browser env.
I close this issue.


Error: The non-abstract class 'RTCVideoRenderer' ...

Hello,
Trying to launch the demo, on Windows 10, with Flutter 3.3 I get the following error message.
Error: The non-abstract class 'RTCVideoRenderer' is missing implementations for these members:
 -VideoRenderer.onFirstFrameRendered

I understand that the application is made in another version of Flutter.
Have you thought about upgrading the demo to 3.3?
Thank you

I am also facing same issue

I am also facing same issue

I am also facing same issue

After upgrading flutter_webrtc solved the problem.

Thanks,
You are right. No issues after upgrade.
But I dont know how to setup  the demo. After start the server in my machine
go run cmd\server\main.go
2022-09-19T11:17:37+02:00 INF Flutter WebRTC Server listening on: <IP_ADDRESS>:8086

Then start the demo width  flutter run -d windows , and setup the server localhost:8086 ... dont know how to continue.
Thanks

Thanks, You are right. No issues after upgrade. But I dont know how to setup the demo. After start the server in my machine
go run cmd\server\main.go
2022-09-19T11:17:37+02:00 INF Flutter WebRTC Server listening on: <IP_ADDRESS>:8086

Then start the demo width flutter run -d windows , and setup the server localhost:8086 ... dont know how to continue.
Thanks

Fill in <IP_ADDRESS> and fill in the ip address in another device.

Thanks.


WebUI always seems to re-subscribe

Could it be because of the token?

Looks more like a load issue so the blitz is dropping the connection all the time under high load.
Didn't experience this after I used my case on my Pi again which dropped the temp by 20 degree

I don't think I can do anything about this. The OS must leave enough power for the API to process its events. It could be that the ping events don't arrive as necessary.

Related to https://github.com/rootzoll/raspiblitz/issues/3255


Drop the connecting item from pool when dropping an idle connection

So when we hit the keep-alive timeout, we cannot open a new connection because we had a leftover item in the connecting map. This seems to fix the issue, but I'm not sure is this how kosher and did I break something else.

Seems good!


Removed superfluous interfaces and dead code

Motivation and Context
The projects have been bloated with a lot of unnecessary interfaces which are abstracting when abstracting is not needed or useful.
What has changed
The interfaces have been replaced with the concrete implementations.
A class and unit test which weren't being used anymore have been removed.
How to test?
Have run ITs and ATs to check there's no regression - passing OK.
Links
Trello: https://trello.com/c/CrZzmiKn/329-get-rid-of-all-the-interfaces-in-the-java-projects

the JAR isn't used by other projects so I don't know why we'd need javadocs - this isn't an API/library project


Equivalence prover

We want to bring back a KEQ like tool which enables proving that two programs in the same (or different) languages are equivalent.
We'll start with one that assumes the same programming language between the two executions.
Implementation ideas

Proof/Prover classes are here: https://github.com/runtimeverification/pyk/tree/master/src/pyk/proof
Make a new file equivalence.py, which has EquivalenceProof and EquivalenceProver
EquivalenceProof has data:

Two KCFG datastructures, for each execution.
List of pairs of node which are in "loose" equivalence relation.
List of pairs of nodes which are in "tight" equivalence relation.

EquivalenceProver class has data:

An EquivalenceProof
A heuristic (semantics specific, program independent): should_be_equivalent: Callable[[CTerm, CTerm], bool]
A predicate (user supplied program specific, or can supply default language specific program generic variant): are_equivalent: Callable[[Iterable[CTerm], Iterable[CTerm]], bool].

EquivalenceProof.advance_proof does:

Execute both KCFGs to completion using KCFGExplore.extend, similar to APRProver.advance_proof
Pairwise establish if should_be_equivalent holds between nodes in each KCFG.
For the nodes with the same LHS or same RHS between the pairs which passed the first predicate, establish that the tighter second predicate holds over their disjuncts.

An example we talked about:
$s = 3;
if (B) { $s = $s + 3 ; } else { $s = $s + 2 ; $s = $s + 1 };

ASTART => A1
A1 => A2
A2 => A3
A3 => BRANCH
BRANCH => END1
BRANCH => END2

$s = 3;
$s = $s + 3;

BSTART => END

Equivalence: ==P === $s ends the same.
Loose "should be same" predicate: (ASTART, BSTART), (END1, END), (END2, END)
Tight "is the same" predicate: ASTART ==P BSTART, END1 \/ END2 ==P END
Some reference literature from @daejunpark
The general approach then was:

Make a product semantics of the two (join their configurations).
Insert no-op "cut" instructions into both programs (the LLVM -> x86 code generator was instrumented directly to do this programatically, so these points were identified mechanically and automatically, but specific to LLVM/x86).
Execute one program (eg. LLVM program) on all paths until it hits the first set of "cut" instructions, call these  states "cut-points".
Execute the other program (eg. x86) on all paths until it hits the first set of "cut" instructions.
Establish which cut-point on teh LLVM corresponds to which on on x86.
Establish the tighter equivalence relation between the two programs at these points, even a user-supplied heuristic specific to the program needs.

This worked pretty well, and I think even scaled to some larger algorithms. The linked paper is an early version of the work, which did not make it to publication (other than in Daejun's thesis). But eventually they made it work between LLVM and x86, not just LLVM -> optimized LLVM. Unfortunately, I think the larger work never really made it to publication. Here is Daejun's thesis: https://www.ideals.illinois.edu/items/111906
The biggest challenge this approach faced was the "make a product semantics of the two". This is difficult for all sorts of technical reasons, mostly related to joining the syntax (overlapping productions) and semantics (conflicting rules) of the two semantics in a way that didn't change the behavior of each other. The facilities the K compiler provides for this are very limited.
Our approach will be different, as described. We'll instead just build the execution graph of each one separately, and then have the semantics-specific predicate saying "these two points should be in the simulation/bisimulation relation" run over all the nodes in the two graphs. Finally, after establishing that relationship, we'll check that the tighter simulation predicate (user-supplied, or we provide a default semantics-specific one) holds over those pairs of nodes.

Returning to Backlog until @PetarMax is available.


can not Sync Gitlab project in subgroups

today i  upgrade drone to last version but aslo cant sync project in subgroups 。
before upgrade ,the web page of " repo list " can show the project
but after upgrade,it not display in the list

see https://github.com/drone/drone/issues/2009#issuecomment-295686484


unable to connect to MS SQL server database.

I am unable to connect the DB server. please let me know what i am missing
Provider=SQL Server Native Client 11.0;Data Source=NFUMDEMO;Initial Catalog=PIP_TST_SOURCE;User ID=eeeeuser;Password=win@xxxxxation;

Remove the provider information in your connection string, it should resolve the issue.

After a quick check, I'd say that you've two options:

As previously suggested you remove the provider information, NBI will use the .NET Framework Data Provider for SQL Server (System.Data.SqlClient).
If you want to use the OleDb provider then you should use the valid nickname for "SQL Server Native Client 11.0": provider=SQLNCLI11.

If it responds to your question please close the issue.

Thanks Seddryck, this helped me to resolve this issue.
now the next question would be how to get the test results
I tried adding this, but not sure where to add this

pls let me know.

One issue, one question. Before asking read the documentation at www.nbi.io


MSID: 91605 Version: 1 DOI: 2<IP_ADDRESS>0843

"msas": "Neuroscience"
"msid": "91605"
"preprintDoi": "10.1101/2<IP_ADDRESS>0843"
"Reviewed Preprint posted": "2023-11-17"
"Sent for peer review": "2023-09-11"
"Posted to bioRxiv": "2023-10-06" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>0843v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91605)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/91605/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0843)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/91605/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/91605)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/91605

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/91605

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/91605/v1/91605-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/91605)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Publisher details and elocations missing from references.


feat: split thorchain savers fees into miner/protocol fees

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4649   👈

This comment was auto-generated by Graphite.


ClusterSpec: rename BaseImage to Repository

ref: https://github.com/coreos/etcd-operator/issues/1719

Currently this is a breaking change.
Opening up early to collect ideas and suggestions.

My thought is to deprecate baseImage in this release, but without breaking it.
Internally, baseImage will be converted to repository.

Fixed to automatically convert BaseImage to Repository.

LGTM

Manual tested:
Created EtcdCluster:
spec:
  size: 3
  baseImage: "quay.io/coreos/etcd"

Update from operator shows the change:
  spec:
    baseImage: quay.io/coreos/etcd
    repository: quay.io/coreos/etcd
    size: 3
    version: 3.2.10


fixes and adjustments

fsm transition returns if it worked or not
transform not set to dirty if translating my zero ( need ticket )
fix bounds math for transforms ( need ticket )
lock nineslice centers to 0 width and 0 height ( need ticket )
add can transition to FSM
add inside and surrounds to bounds ( need ticket )
add tests to bounds
prevent setting with and height for transforms < 0 ( need ticket )
add NewSpriteWithTransform to pass in a transform directly ( need ticket )

Codecov Report
Base: 3.98% // Head: 8.25% // Increases project coverage by +4.26% :tada:

Coverage data is based on head (b995f1f) compared to base (284a8c5).
Patch coverage: 38.46% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff            @@
##            main     #97      +/-   ##
========================================
+ Coverage   3.98%   8.25%   +4.26%
========================================
  Files          9       9
  Lines        678     703      +25
========================================
+ Hits          27      58      +31
+ Misses       648     642       -6
  Partials       3       3

Impacted Files
Coverage Δ

mathf/transform.go
0.00% <0.00%> (ø)

mathf/bounds.go
100.00% <100.00%> (+100.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Font Size Type Should Not Support String Text

Describe the bug
The plugin currently allows you to use string text as a value for in the Font Size token. The plugin should not allows users to input such a value.
Used-case:
Users can also reference this in the Font Size field of the typography token which they can save. Applying this does not give them the expected behaviour because the token does not have any numeric value defined nor does using any string text as a value work.
To Reproduce
Steps to reproduce the behavior:

Create a Font Size token
Use string text as a value
Save and apply the token on text
See error

Expected behavior
The plugin could display an error when users are inputting anything other than numbers as a value. Even for reference suggestions, it only displays tokens that have numbers as values.
Screenshots or Screencasts
https://github.com/tokens-studio/figma-plugin/assets/93133012/54a3a7c1-f735-4274-954a-1eecf3d0fe29

What do you mean with string text? Can you give a concrete example value?

@keeganedwin @six7 If I'm correct, the Font Size token should accept only a numeric value or an alias.
Current Behavior:
You can set the Font Size to any text (e.g., smallText), even if smallText is not an alias.

Figma File
Test File

This bug is no longer producible on Prod 1.38.9 or Beta V2-rc5


E0002 Unknown exception for rules E1010,  E1019,  E1029 on AWS::EC2::TransitGatewayPeeringAttachment

cfn-lint version: (cfn-lint --version) cfn-lint 0.72.3
Description of issue:   The latest release of cfn-lint (0.72.3) fails linting on AWS::EC2::TransitGatewayPeeringAttachment resource that worked with version 0.72.2 and earlier.  According to the AWS documentation, everything looks correct.
Example of the resourced defined in the template
  PeeringTransitGatewayAttachment:
    Type: AWS::EC2::TransitGatewayPeeringAttachment
    Properties:
      PeerAccountId:<PHONE_NUMBER>01
      PeerRegion: us-west-2
      PeerTransitGatewayId: aaa-0d57f575ea39ed5ab
      Tags:
        - Key: Name
          Value: transit-gateway-peering-attachment
      TransitGatewayId: !Ref TransitGateway

The following linting errors are thrown when running cfn-lint network.yml
E0002 Unknown exception while processing rule E1010: 'NoneType' object has no attribute 'get'
network.yml:1:1

E0002 Unknown exception while processing rule E1019: 'NoneType' object has no attribute 'get'
network.yml:1:1

E0002 Unknown exception while processing rule E1029: 'NoneType' object has no attribute 'get'
network.yml:1:1

Line 1 of template:  AWSTemplateFormatVersion: 2010-09-09
An example of one of the exception traces when debug (-D) is enabled:
E0002 Unknown exception while processing rule E1029: Traceback (most recent call last):
  File "C:\...\.venv\lib\site-packages\cfnlint\rules\__init__.py", line 320, in run_check
    return check(*args)
  File "C:\...\.venv\lib\site-packages\cfnlint\rules\__init__.py", line 44, in wrapper
    results = match_function(self, filename, cfn, *args, **kwargs)
  File "C:\...\.venv\lib\site-packages\cfnlint\rules\__init__.py", line 202, in matchall
    return self.match(cfn)  # pylint: disable=E1102
  File "C:\...\.venv\lib\site-packages\cfnlint\rules\functions\SubNeeded.py", line 78, in match
    getatts = cfn.get_valid_getatts()
  File "C:\...\.venv\lib\site-packages\cfnlint\template.py", line 278, in get_valid_getatts
    for item in build_output_string(
  File "C:\...\.venv\lib\site-packages\cfnlint\template.py", line 241, in build_output_string
    for k, v in prop.get("Properties", {}).items():
AttributeError: 'NoneType' object has no attribute 'get'

Fixed.  Release v0.72.4 is being put into pypi now.


Blitzar requires 12060

When I run the "Hello World" example from sxt-proof-of-sql with
cargo run --example hello_world --features="blitzar test"

I receive this output
Warming up GPU...cbindings/backend.cc:55 panic: Update GPU drivers. The maximum supported version is 12040 but blitzar requires 12060

I'm running a GeForce MX150 on my Ubuntu and I'm already running the latest proprietary and tested driver (550).

When I look for the latest Linux drivers on the NVIDIA website I see this

So, I could possibly download and manually install the latest driver version 550.120. However, before I do that I would like to know if this driver would meet the 12060 requirement as I'm not sure to what this number refers to exactly. Thank you for shining some light on this issue.

Those numbers are coming from the function cudaDriverGetVersion
https://docs.nvidia.com/cuda/cuda-runtime-api/group__CUDART____VERSION.html#group__CUDART____VERSION_1g8a06ee14a0551606b7c780084d5564ab
The 12040 is referring to the latest version of the cuda toolkit your drivers support:

Returns in *driverVersion the latest version of CUDA supported by the driver. The version is returned as (1000 major + 10 minor). For example, CUDA 9.2 would be represented by 9020. If no driver is installed, then 0 is returned as the driver version.

We're currently on cuda toolkit 12.6 update 1. If you look at the release notes, it will say what the minimum required driver is https://docs.nvidia.com/cuda/cuda-toolkit-release-notes/index.html

CUDA 12.6 Update 1   >=560.35.03 (linux).    >=560.94 (windows)

Alternatively, you could also fix the version to this older release https://github.com/spaceandtimelabs/blitzar/releases/tag/v1.80.3
which was built with cuda toolkit 12.4 and (I think) should work with the drivers you already have installed.

Thanks @rnburn , I understand. Indeed, running nvidia-smi shows CUDA Version: 12.4. So, I think I can only do testing with blitzar 1.80.3.


Installer functionality should be documented

I found out about the installer functionality via OpenProject and I'm using it on SIP Treadmill but it unfortunately has no documentation.

I know :( This is not as stable as I would like, but in the meantime I've pushed a README at https://github.com/pkgr/installer
Let me know if you have specific questions, I'll be happy to answer them.

Awesome, thanks @crohr.

Closing this for now (!). I'm not sure there is enough demand for it to consider making it official.


Training - Incorrect creation of Eval testset

All files on my machine.
I run an evaluate
./mmt evaluate -b b1_sprint3

While I was doing human evaluate I found this
ORIGINAL: 1192	en	Modem Advanced
HUMAN   : 1192	it	Modem Avanzate
MMT     : 1192	it	Modem avanzate
GT      : 1192	it	Modem avanzata

Unfortunately this is present in the train corpus just once, so I should not appear in the test set.
ubuntu@ip-172-30-0-162:~/mmt$ grep -i "Modem Advanced" ../benchmark-1.0/data/train/*.en
../benchmark-1.0/data/train/taus-undefined-milengo-10584.en:Modem Advanced

This is exactly the behavior we agreed to apply to the automatic splitting: if the training set contains duplicates, it is possible that you can have overlapping between training an test sets.

Removed "invalid label", I'm checking the data. Sorry for the label, I though the problem was different.

Ok, so in the folder benchmark-1.0/data/train/*.en you have all the original training corpus. The one the user provides.
The automatic splitting reads those files and selects a portion to use as dev and test set. So it's perfectly normal to find a test sentence in the original corpora; the corpora used during training is not exactly the original , but of course the one where MMT has removed the sentences used for test and training.
You will find all the test (and dev) sentences in the original corpora. But you won't find them, if they are unique in the training corpus, in the MMT models.


Feature Support split_by_char function

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

When there are multiple rows of data, the calculation cannot be performed correctly. I will fix it soon.

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.29 seconds
load time: 439 seconds
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221111235959_clickbench_pr_44676.html


Fixed issue with adding encodings or mark to chart object

Change

Fixed the issue with adding marks or encodings to the chart object using encode() or mark_{mark_type}
Unlike Altair, Bifrost doesn't store the initial encodings. Every time plot() gets called, it doesn't save the encodings and mark being used.

Altair stores the initial encodings. So, when you first give x, y, color, and mark and later give x and y, it still uses color and mark previously given in addition to the new x and y.
Bifrost gets reset every time plot() is called. It doesn't use any of the previously given encodings or mark because it provides users options to choose encoding and mark with Draco recommendation system when they are not given.

Agreed with the mark/encoding reset design choice. There's always the option of passing these as .plot parameters 👍


Make OSbuddy prices show in the "view  guide prices" under worn equipment tab.

Duplicate of #3927 and most likely no, as it is not useful there.


GridMap editor fixes and improvements

This change fixes a few outstanding issues and greatly improves the usability
of the GridMap editor through the following changes:

Copied mesh now gets displayed during pasting (also renamed the related
identifiers accordingly)
Duplication/paste indicator now gets rotated around the correct pivot point
(duplication worked properly before, but the indicator was shown misplaced
when rotated)
Selected mesh library item cursor is no longer shown during selection and
duplication/pasting
Back rotate X/Y/Z is now working during duplication/pasting
Added true cut operation thanks to now having a proper clipboard (clear
operation got remapped to the DEL key)
Got rid of some weird workarounds in the duplication code
Fill and clear operations now correctly make the selection marker inactive
as this was broken partly due to the workarounds mentioned above
(duplication continues to keep the selection marker active to allow
subsequent duplications)
Clear current selection on RMB, but treat selection as an action so previous
selection can be restored on undo
Separated selection and paste indicator data as it's prone to error and
confusion and it's anyway needed now that selection is treated as an action
Added support for cancelling paste, selection, and even unselect the
currently selected mesh library item with the ESC key (previously there
wasn't a way to unselect)
Changed the key binding of fill/clear/duplicate operations to use Ctrl as a
modifier
Changed erase to use RMB instead of Shift+RMB (free look is available
through Shift+F anyway, so no need to occupy RMB for it during gridmap
editing)
Removed unused area, external connector, and configure menu items (there's
also the non-functional clip mode menu items, but I'm not sure whether there
are any plans with that, I suppose it's meant to be an editor aid)
Renamed INPUT_COPY to INPUT_PICK to better reflect its purpose
Added support for using Shift+Q and Shift+E to select multiple floors/planes
without actually changing the current floor/plane as it happens when using
e.g. the mouse wheel

Fixes #25373 and #15883

Changes look pretty good to me. A rebase is needed, and ideally a review by someone more familiar with GridMap than I am.

Changes look pretty good to me. A rebase is needed, and ideally a review by someone more familiar with GridMap than I am.

Rebased.

Thanks a lot!

@aqnuep Such a great work btw, Gridmap definitely needed some love :)


Changing the width of the chat in landscape mode

Feature Request
Is your feature request related to a problem? Please describe.
We would like to be able to change the width of the chat to landscape mode as is possible on the "Xtra for Twitch" app.
Describe the solution you'd like
You should be able to choose the width of the chat as a percentage, as shown in the image

This is a screenshot taken from the Xtra for Twitch app to illustrate what we would like as a new feature.
The idea for adding this feature came from @xplod62

Closed due to new Scope of the project


Update hellojni to use c++11

We're trying to demo using c++11 w/studio -- is it possible to update hellojni to do this via:

adding the std=c++11 cppflag
using auto somewhere in the example

have a look at the code change when you get chance:
https://github.com/googlesamples/android-ndk/pull/162

this is done now, re-open the bug if it does not work for you


Allow access to the "object" value of documents

For image documents, this attribute contains a URL to a thumbnail. This
is handy!

Coverage decreased (-1.2%) to 93.299% when pulling 004d174f61eb468a2fa74b4e637036e8368f00be on djcp:master into 3bb2522e6905bf0279039e8f6d4e468f03148e12 on phereford:master.

Thanks @djcp. I am just checking to make sure the tests pass. Once those pass, Ill merge it in and issue a release.


Date validation failure. D

Your environment

Version of JQV: 1.19
Browser name and version: Google Chrome 107.0.5304.107 (Build oficial) (64 bits)

Current behavior
Validator allows any kind of dates even considerating day 30 in February
Expected behavior
Validator must return an error message.
Live demo
Try to validate 1983-02-30 and validator doesn't show anny error message

I believe that you used the Date method, which has been deprecated in favour of DateISO.  I get that from the comment in
https://github.com/jquery-validation/jquery-validation/issues/1845.

I believe that you used the Date method, which has been deprecated in favour of DateISO. I get that from the comment in #1845.

I'm using the DateISO method

I'm using the DateISO method

@lgnl272 Please provide a jsfiddle link

Duplicate bug: https://github.com/jquery-validation/jquery-validation/issues/276
DateISO only does structure checks.

I find it interesting that the 'date' validator is deprecated in favor for date ISO which is a standard notation/format.
There is no way to verify that the date is valid in time using ISO notation.
Probably need to implement  ISO week date which is part of ISO 8061 so the date can be checked for two(2) things:

valid ISO notation
valid date/time exists

DateISO should be named DateISO8061 or DateISOFormat, then de-depricate the 'date' validator.
Date validator correctly errors on  new Date('1983-02-30') as it should.

Reminds me of an old pull request:  https://github.com/jquery-validation/jquery-validation/pull/2032

return this.optional( element ) || ( !/Invalid|NaN/.test( new Date( value ) ) && /^\d{4}[\/\-](0?[1-9]|1[012])[\/\-](0?[1-9]|[12][0-9]|3[01])$/.test( value ) );
adding the !/Invalid|NaN/.test( new Date( value ) check fixes the current use case.
Basically copied the test case for date: (that is being deprecated) and logically AND it with dateISO format.
Screen shot of test cases:

new Date(dateString) is
new Date (dateString) format includes date-only forms:

YYYY
YYYY-MM
YYYY-MM-DD

YYYY-MM-DD follows exactly as a Date only ISO format.
Quote from MDN web docs "This is ISO-8601-compliant and will work reliably"

Disregard the above, I now see the problem with Date() and how different browsers interpret.
Bigger problem with JavaScript, not jquery-validate.

Here is a jsfiddle: https://jsfiddle.net/7sk4z9qt/13/
I have added a test case, grunt completes successful, all tests pass.
Questions:
I have created a new validator named 'isValidDate' using DateISO standards, should my new validator replace dateISO?
Should I submit PR as is, or wait for the above question answered?
I think this solution will close a lot of date issues.


Add task batching doc

Motivation

To add a documentation for BathingProcessor like task-compaction.adoc.

Related Issue: https://github.com/line/decaton/issues/128
Related PR: https://github.com/line/decaton/pull/139

@ryamagishi Hi, long time no see. Do you still have a plan to complete this PR?

Hi @ocadaruma, I apologize for the delay and for forgetting about this PR. If you would permit, I would like to complete it by the end of this month. I need some time to refresh my memory on the work, so I appreciate your understanding.

If you would permit, I would like to complete it by the end of this month

That's totally fine. Thank you!

@ocadaruma
I apologize for the late response.
I have corrected the parts you commented on.
If there is anything that needs to be corrected, I will deal with it right away so that it can be merged.
Please check it when you have time.

@ocadaruma
I have responded to your comments.
Please take a look.


ADR to improve 2FA login

Summary | Résumé
This ADR details the replacement of 2FA codes with "magic links", how they can improve and make the login experience more accessible for our users. It also details security concerns, potential mitigations, policy implications and potential AWS cost changes associated with the change.
Security Assessment Check | Contrôle de l'évaluation de la sécurité

Does this introduce any security concerns? | Est-ce que ça introduit des préoccupations liées à la sécurité?

Yes, these concerns were identified in a threat modelling exercise and potential mitigations have been outlined in the ADR.
TODO: Have the security team review mitigations and recommend a level of defensiveness to follow through with.
Reviewer checklist | Liste de vérification du réviseur
This is a suggested checklist of questions reviewers might ask during their
review | Voici une suggestion de liste de vérification comprenant des questions
que les réviseurs pourraient poser pendant leur examen :

[ ] Does this meet a user need? | Est-ce que ça répond à un besoin utilisateur?
[ ] Is it accessible? | Est-ce que c’est accessible?
[ ] Is it translated between both offical languages? | Est-ce dans les deux
langues officielles?
[ ] Is the code maintainable? | Est-ce que le code peut être maintenu?
[ ] Have you tested it? | L’avez-vous testé?
[ ] Are there automated tests? | Y a-t-il des tests automatisés?
[ ] Does this cause automated test coverage to drop? | Est-ce que ça entraîne
une baisse de la quantité de code couvert par les tests automatisés?
[ ] Does this break existing functionality? | Est-ce que ça brise une
fonctionnalité existante?
[ ] Should this be split into smaller PRs to decrease change risk? | Est-ce
que ça devrait être divisé en de plus petites demandes de tirage (« pull
requests ») afin de réduire le risque lié aux modifications?
[ ] Does this change the privacy policy? | Est-ce que ça entraîne une
modification de la politique de confidentialité?
[ ] Does this significantly alter performance? | Est-ce que ça modifie de
façon importante la performance?
[ ] What is the risk level of using added dependencies? | Quel est le degré de
risque d’utiliser des dépendances ajoutées?
[ ] Should any documentation be updated as a result of this? (i.e. README
setup, etc.) | Faudra-t-il mettre à jour la documentation à la suite de ce
changement (fichier README, etc.)?

This all makes sense to me - my only other comment is that if we implement magic links, I would also prefer if we keep the option to copy/paste the code... maybe make it a user preference if it's too confusing to have both by default? Magic links open new tabs and thus I lose my history.

This is doable I think. We'd need to do some design work to ensure that it does not become confusing. We could make it a user profile option / ask users when they sign up whether they prefer a code or a magic link for their 2FA. This way we can either let them know to check their email for a link, or provide them with the usual text box for the 2FA code to be manually input but not both at any given time.

Moving this to draft as we have decided not to go ahead with this for now


Errors in identifying extractor

I've run into a couple of cases where forum-dl can't identify the type of forum. What if we could specify an extractor as an option when running forum-dl?

If Forum-dl can't identify the type of forum, then it's unlikely to be able to extract it anyway. But it makes sense to give the user this option notwithstanding.


fixshould keep at least one slot materialized in agg node

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Issues at log in with a new created account in AMO-prod

Steps to reproduce:

Load AMO-prod page https://addons.mozilla.org/en-US/firefox
Click "Register" link and register as a new user
Go to your email and click the confirmation link
Try to sign in with the new created account

Expected results:
The message "Successfully verified" is listed in AMO page. After entering the new created email and password the user is logged in.
Actual result:
User cannot log in. An error message is displayed.
Notes/Issues:
Verified on FF42(Win 7). Issue is reproducing on AMO-prod.
Please see the screencast: http://screencast.com/t/nSBAQt540UJ

The MOC has been notified there's lots of sentry errors with:
OperationalError: (1290, 'The MySQL server is running with the --read-only option so it cannot execute this statement')

Moc bug is: https://bugzilla.mozilla.org/show_bug.cgi?id=1224527

After, some time I was able to log in with the created account.
Now:
-> if I try to delete the newly created account, I receive the message "Profile deleted" but I can log with the that account again.
-> if, being logged in, I access the "Account Settings" page I see that the "deleted" account became "Anonymous-xxxxxx".
Please see the screencast: http://screencast.com/t/N25th5Qr

So this sounds like a possible dupe of #913


Implement Main return code on WebAssembly

Currently, the generated WebAssembly method that calls into Main doesn't handle Main's return code and just returns a hard-coded value. The code that generates it is in WebAssemblyObjectWriter.EmitNativeMain. If Main returns a value, the native caller should alloca a spot for it to write the return value and pass that as the second parameter to Main and then return the value from that spot.

@gskachkov, have you had any luck with this (or any questions)? It would certainly help us bring up CI testing.

@morganbr Sorry for the delay fixing this task, I'm working on Mac and only recently I've managed to build solution, thanks for other contributors :-). But currently, we are releasing solution to prod on my primary job during next two weeks, so I have a small progress. But anyway I would be happy if you could wait at least for one, or two week, I'm still want to fix it, but if it block you from progress on other tasks or correct working CI feel free to take it from me, I'll take another one when have more time.

@gskachkov, no problem. As long as you're still interested over the next few weeks and we're not blocking you, we're in good shape.

@morganbr  Release done, so I start working on task :-) Sorry for delay with implementation

No need to apologize; I'm happy you're helping out 😄
In the meanwhile, I implemented return value checking in the test running script, so once you start returning real values, the test should fail if they're not 42. If you want, you can also change 42 to our more standard test return value of 100.
https://github.com/dotnet/corert/blob/ce9cd701b13774174765f731834fb2cb1cf8ebfc/tests/src/Simple/HelloWasm/HelloWasm.cmd#L6

Yeah, I look throw the code, so there is idea how to implement issue:

As now I can not find how take meta information when we emit main function in EmitNativeMain, so my idea store this information during create native function for main in
https://github.com/dotnet/corert/blob/eba7180dc25ff12d8989cc2eec2a0e7803071609/src/ILCompiler.WebAssembly/src/CodeGen/ILToWebAssemblyImporter_Statics.cs#L55
We create this function in LLVM with _universalSignature that is void (int, int).  I'm thinking about add new constructor that can create correct signature for ie int (string []) and take this info by GetReturnType in https://github.com/dotnet/corert/blob/master/src/ILCompiler.WebAssembly/src/CodeGen/WebAssemblyObjectWriter.cs#L238
end emit correct return type accordingly

If you don't have metadata, you might be able to make return code handling unconditional by just initializing the memory you allocate for returning to zero. If main has a void return, it just won't write to that location.
Introducing a new signature could be more difficult because it would require special-casing how argument reading and returning works in generating main.

@gskachkov, unfortunately, I needed to fix this to finish #5621. I appreciate you looking into it and we'd love to have you pick up another work item.


Convert BenchmarkSetup to use StatusOr.

Fixes https://github.com/googleapis/google-cloud-cpp/issues/2877
As we do not want to use exceptions, we create a "factory" function named MakeBenchmarkSetup that does all of the actual work and returns a StatusOr. As suggested by @coryan, I moved all of the fields from BenchmarkSetup into BenchmarkSetupData, which is accepted by the former's public constructor.

This change is

Why not just return setup_data?

That doesn't compile, even if I remove explicit. Did you have other changes in mind that would make this work?


Pointwise meta order

Pull Request Description
What issue does this change request address?
closes #854
What are the significant changes in functionality due to this change request?
The order of pointwise_meta is sorted.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

Job Mingw Test on 171cb54 : canceled by @joshua-cogliati-inl
just want to test other platforms

Job Test mac on 171cb54 : canceled by @joshua-cogliati-inl
just want to test other platforms

Job Test qsubs on 171cb54 : canceled by @joshua-cogliati-inl
just want to test other platforms

Job Test mac on bb03669 : canceled by @joshua-cogliati-inl
just want to test other platforms

Job Mingw Test on bb03669 : canceled by @joshua-cogliati-inl
just want to test other platforms

Job Test qsubs on bb03669 : canceled by @joshua-cogliati-inl
just want to test other platforms

Job Test mac on bb03669 : invalidated by @joshua-cogliati-inl

Job Mingw Test on bb03669 : invalidated by @joshua-cogliati-inl

Job Test qsubs on bb03669 : invalidated by @joshua-cogliati-inl

@alfoa  This can either be merged by itself, or as part of #865, your choice.  Note that this does change requirement tests.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code. Done
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts). n/a
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details. ok
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass. ok
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True. n/a
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync. requirement test changed. @alfoa (as chair of the CCB) approves this change
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done. #854
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added? n/a

requirement test changed. @alfoa (as chair of the CCB) approves this change

PR can be merged...
Checklist passed...


!setavatar problem

Says the url is unsupported img. what is supported?

Anything that the Discord client supports. E.g .png, .jpg.


Improvement of React Trainings, by highlighting changes

Currently work in Progress
Highlight Changes and add information to training materials

@robgietema finished, please review and merge

Awesome work!


Duplicate Nationality

The bug
In the Nationality Donut chart, you can see two Indian and India.
To Reproduce
https://www.covid19india.org/deepdive
look for patient nationality chart.
Expected behavior
Indian should be merged in India nationality.
Screenshots

Fixed.


doc: update list of builtin patterns

Great plugin! Thanks for all your work on it.
This PR updates the list of builtin patterns in the README to include path and hex

thanks!


API: Export Sorbent Trap and Sampling Train Data

As the ECMPS application, I need to be able to successfully export Sorbent Trap Data from the emissions module in order to support program goals
Given the export endpoint in both emission modules have been updated when sorbent trap data is exported then the begin_date of the data is within the year and quarter range the user provides
Sorbent Trap id of the sampling train record matches the ID of the sorbent trap record. Each emission evaluation record can have 0 or more sorbent trap records. Each sorbent trap record should have 0 or more sampling train records

[ ]  Add SorbentTrap data to the existing export endpoint in the emissions module
[ ]  This will also include sampling train data to the existing export endpoint in the emissions module controller
[ ]  Implement logic to export sampling train data and sorbent trap data

Adds sorbentTrapData and samplingTrainData to JSON under:
Root -> sorbentTrapData -> samplintRainData
The data should take this shape:
{
  "id": string,
  "monitoringLocationId": string,
  "reportingPeriodId": number,
  "beginDate": date,
  "beginHour": number,
  "endDate": date,
  "endHour": number,
  "monitoringSystemId": string,
  "pairedTrapAgreement": number or null,
  "absoluteDifferenceIndicator": number or null,
  "modcCode": string or null,
  "hgSystemConcentration": string or null,
  "calcPairedTrapAgreement": number or null,
  "calcModcCode": string or null,
  "calcHgConcentration": string or null,
  "userId": string or null,
  "addDate": date or null,
  "updateDate": date or null,
  "apsCode": string or null,
  "rataIndicator": number or null,
  "samplingTrainData": [
    {
      "id": string,
      "sorbentTrapId": string,
      "monitoringLocationId": string,
      "reportingPeriodId": number,
      "componentId": string or null,
      "sorbentTrapSn": string,
      "mainTrapHg": string or null,
      "btTrapHg": string or null,
      "spikeTrapHg": string or null,
      "spikeReferenceValue": string or null,
      "totalSampleVolumeDscm": number or null,
      "referenceSfsrRatio": number or null,
      "hgConcentration": string or null,
      "percentBreakthrough": number or null,
      "percentSpikeRecovery": number or null,
      "samplingRatioCheckResultCode": string or null,
      "postLeakCheckResultCode": string or null,
      "trainQaStatusCode": string or null,
      "sampleDamageExplanation": string or null,
      "calcHgConcentration": string or null,
      "calcPercentBreakthrough": number or null,
      "calcPercentSpikeRecovery": number or null,
      "userId": string or null,
      "addDate": date or null,
      "updateDate": date or null
    }
  ]
}

Sample data for non-workspace endpoint. Workspace and non-workspace endpoints for export share the same logic, tested with non-workspace due to data availability.
monitor plan id, quarter, year:
MDC-CA078A374DB04014AEB50EC406C538EA,2020,1
MDC-CA078A374DB04014AEB50EC406C538EA,2021,4
MDC-CA078A374DB04014AEB50EC406C538EA,2021,2
MDC-CA078A374DB04014AEB50EC406C538EA,2020,4
MDC-CA078A374DB04014AEB50EC406C538EA,2019,2
MDC-CA078A374DB04014AEB50EC406C538EA,2022,1
MDC-CA078A374DB04014AEB50EC406C538EA,2020,2
MDC-CA078A374DB04014AEB50EC406C538EA,2021,3
MDC-CA078A374DB04014AEB50EC406C538EA,2020,3
MDC-CA078A374DB04014AEB50EC406C538EA,2021,1
SQL used to find test data:
select distinct emission_evaluation.mon_plan_id, reportingPeriod.calendar_year, reportingPeriod.quarter
from camdecmps.sampling_train as samplingTrain
    left join camdecmps.sorbent_trap as sorbentTrap on samplingTrain.trap_id = sorbentTrap.trap_id
    left join camdecmps.monitor_location as monitorLocation on samplingTrain.mon_loc_id = monitorLocation.mon_loc_id
    left join camdecmps.monitor_plan_location as monitorPlanLocation on monitorLocation.mon_loc_id = monitorPlanLocation.mon_loc_id
    left join camdecmps.monitor_plan as monitorPlan on monitorPlan.mon_plan_id = monitorPlanLocation.mon_plan_id
    join camd.plant as plant on monitorPlan.fac_id = plant.fac_id
    left join camdecmps.emission_evaluation on monitorPlan.mon_plan_id = emission_evaluation.mon_plan_id
    join camdecmpsmd.reporting_period as reportingPeriod on emission_evaluation.rpt_period_id = reportingPeriod.rpt_period_id
where reportingPeriod.calendar_year >= 2019;


visualize features

Hello! May I ask if you could provide some sample visualization code? Thank you very much!

Hello! May I ask if you could provide some sample visualization code? Thank you very much!

You can refer to this repo for the visualization code: https://github.com/rentainhe/visualization


How to speedup tokenizer.encode?

I found in the pre-trained datasets, there are some docs has large amount chars, which cause a long time to encode them. For example, a doc has 15955671 chars, will cost 6.6 hours to encode it.
How do you speedup it? split the doc into many sub-docs? But I use the megatron to pre-train, has any idea?
Looking forward to hearing from you in your free time. Thank you very much.

Wrong repo? This is tinyLlama, not megatron LM, besides, for tokenizer.encode, Thats most likely an hf method, so you would have to look at the HF repos instead.

if possible, you could try tiktoken, which is supposably 3x faster

Thanks for your time.


Errors loading assets with spaces in their names

If I have spaces in my asset filenames, they fail to load at runtime (works fine without the spaces):

Asset can be seen in the tree on the left, pubspec in the top pane and the error in the bottom pane.

Deferring to @szakarias

Thanks, fix works fine!


Arshad561

The Readme file is currently lengthy and complex, making it challenging for newcomers to comprehend how to contribute effectively to the project.
To enhance accessibility and clarity, I propose relocating the Contributing section to the beginning of the Readme file and streamlining its content where feasible.
I kindly request assistance from anyone willing to dedicate time to address this issue and improve the project's accessibility for all contributors. Thank you!

Hello, this is Arshad. I have assigned this issue to myself. I will work on this issue and close this ASAP!
@caiton1 I will let you know if I need any help from you

Hi @caiton1 , I have raised a PR for this issue here https://github.com/NAU-OSS/jabref/pull/95. Please review my PR


Is this abandoned because it was fixed in SwiftUI itself?

It looks like abruptly stopped in 2022. Is it because retargeting was improved in SwiftUI?

Wave is largely stable and feature-complete right now, which is why there aren’t many changes. It’s definitely not abandoned.
If you run into an but or have a feature request, feel free to open another issue.


Old dll at tunderstore

@Cooleyy I think you uploaded the wrong dll file to thunderstore, as favourites items are in that version, but the input is not being blocked.

Not sure what happened here tested it on my end and was working but I have updated the files for the new update embedding the assetbundle and it should have fixed this issue. Let me know if its fixed.


Parsing environment: null byte is not allowed in input

Hi,
I am running into this error: Error: null byte is not allowed in input at line 1, column 4 at the setup step Parsing environment... of this action.
Here are the github action logs:
Run conda-incubator/setup-miniconda@v2
Gathering Inputs...
Creating bootstrap condarc file in /home/runner/.condarc...
Ensuring installer...
Setup environment variables...
Parsing environment...
Error: null byte is not allowed in input at line 1, column 4:

Here is my workflow file:
name: Main

on: [push]

jobs:
  test:
    runs-on: ubuntu-latest
    steps:
    - uses: actions/checkout@v1
    - uses: conda-incubator/setup-miniconda@v2
      with:
        activate-environment: anaconda-client-env
        environment-file: environment.yml
        python-version: 3.8
        auto-activate-base: false
    - run: |
        conda info
        conda list
    - run: ./test.sh

Here is my environment.yml at the root of my repository:
name: anaconda-client-env
channels:
  - defaults
dependencies:
  - numpy
  - python=3.8
  - tensorflow-gpu

It's worth to mention that I am developing on windows.
What should I do to fix this error?

I copied the content of the file in notepad and saved it, it worked.
Although this is more a trick than an actual fix, would it be possible to handle this case to improve the support of this action for windows developers?

However this is more a trick than an actual fix

Well the error Error: null byte is not allowed in input at line 1, column 4: is clear in that there is an invalid character on your file. This does not seem to be a windows related issue, but more of where you copied the original file from.

Ehm, I don't really know how to interpret some of the contents of the spec, but it kinda sounds like allowing null bytes would contravene this.
I really don't like doing anything to input files (look how much pain the python patching has bought us), but I extra don't like mucking about with encodings, etc. that say right on the tin how they are to be used.