The provider.most/uwp/net5.0 pattern is awful

For SQLitePCLRaw.provider.e_sqlite3, there are four directories:
SQLitePCLRaw.provider.e_sqlite3
SQLitePCLRaw.provider.e_sqlite3.most
SQLitePCLRaw.provider.e_sqlite3.uwp
SQLitePCLRaw.provider.e_sqlite3.net5.0
The first one is just a dummy csproj with a nuspec to include the build output from the other 3.
Each of the other 3 has a Generated directory which contains the generated provider C# code.
The UWP version has a call to sqlite3_win32_set_directory8().  The call is omitted from other builds just in case its presence causes problems.  For example, if that call is around for an iOS build, the linker will fail because the underlying C call doesn't exist in the native library.
The net5.0 version is an unfinished attempt to use UnmanagedCallersOnly attributes instead of MonoPInvokeCallback attributes.  Work in progress.  It builds for net5.0 TFM.  See #423
The .most version covers, well, everything else.  It builds for the netstandard2.0 TFM.
Note that the UWP version does NOT build for a uap10.0 TFM, which is a problem.  It builds for netstandard2.0.  When I converted the codebase to require netstandard2.0 as a minimum, I neglected to update all the uap10.0 TFMs to uap10.0.16299.  So, for example, if I build a provider for uap10.0.16299, its reference to SQLitePCLRaw.core, which is built only for netstandard2.0, will fail.
Anyway, this is a mess.  SQLitePCLRaw.provider.e_sqlite3 wants to be one directory with multitargeting.  And when attempting to make that change, I tripped over this " uap10.0 isn't really netstandard2.0 " problem.

See also notes on #450

One could argue that the common case should have the call to sqlite3_win32_set_directory8() included, and if iOS has a problem with that, then iOS should be treated as the unusual case and have its own multi-targeted build.


add travis.yml

At #1 I've notice the CI is not running. This should fix that

Also, you need to activate repository here: https://travis-ci.org/DaveLiddament/phpsw


WFARQ-12 Allow extending ArquillianConfig service

Fine with me

@jamezp done

Alright 1.1.0.Beta2 has been released. It make take a few hours to hit maven central, but it's in the JBoss Nexus Repository.

Thanks a lot, really appreciate it!


Data processing details in pretraining

Hello, I have a question about data preprocessing during the pretraining phase. I know that OPT is similar to GPT training that groups text into a long chunk of text (length=2048). My question is whether there is special token between the grouped text. For example, we have:
sent1\n
sent2\n
sent3\n
sent4\n
in the training dataset. During pretraining, they will be grouped together into "sent1 sent2 sent3 sent4". My question is: are they concatenated with space? Or a special token like ? Or a line break token?
The pretraining details may be helpful for us to use the model properly.
Thanks

I have the same question too. And I also want to confirm if the same preprocessing is applied on conversational data (PushShift.io Reddit) ?
Thanks.


Document the deprecation of no-unused-variable

https://palantir.github.io/tslint/rules/no-unused-variable/
We end up explaining how this rule is deprecated in favor of the compiler options in a lot of GH threads. We should use the deprecationMessage in the rule metadata: https://github.com/palantir/tslint/blob/63644325d3e717d32c719d8ba0907ff245bf7669/src/rules/noUnusedVariableRule.ts#L34

Just want to point out that there is one important limitation when using the compiler options, versus TSLint:
The compiler options are all-or-nothing: I cannot turn off the check for specific use cases.
For example, we have a folder of generated files with an import used by only part of the generated files, however the compiler will now mark all the generated files not using the import with an error. With TSLint, we could just add a comment to turn off checking for these files.

@schmuli As I mentioned in another place:

The problem with using the flag in tsc is that it causes an error during build, while the tslint rules is more configurable and can be used to produce warnings only.
So these are two use cases in favor of using the tslint rule.

@schmuli As I mentioned in another place:

The problem with using the flag in tsc is that it causes an error during build, while the tslint rules is more configurable and can be used to produce warnings only.
So these are two use cases in favor of using the tslint rule.

The third argument would be that during a development phase you are very likely to have unused-imports. I would like to run tslint in CI in order to detect problems in a pull-request before the branch is merged. The compiler option doesn't allow you to have an intermediate "broken" state when you try things.
Please keep the rule (and undeprecate it).

Note that the compiler checks don't break your build or dev environment unless you set them up to do so either. tsc still transpiles JS that you can use in your project despite compilation errors.

The compiler checks for unused params are unusable for me right now due to how the tsc treats unneeded parameters, please see https://github.com/palantir/tslint/issues/1861#issuecomment-275050807
Where can I find reasoning behind depreciation of this rule, other than that TSC ca also do this?

@jsynowiec from that thread:

The approved fix is to prefix uninteresting parameters with an underscore, see Microsoft/TypeScript#9464 which breaks the variable-name TSLint rule.

There is a rule option exactly for this scenario called allow-leading-underscore. We can add it to the recommended config in the next release.
There is more discussion / explanation in https://github.com/palantir/tslint/issues/1481 and the linked issues. The rule implementation is buggy and the compiler does a better job with these checks, so the maintenance burden in the core ruleset doesn't feel worth it.

Can variable-name know about unused params? Most of us don't want to allow leading underscore elsewhere
const _varName = 1; // <-- this still needs to be a lint error
function foo(_: string, __: number, bar: boolean) {
  console.log('only interested in', bar);
}

I think that skipping just parameters with leading underscore is trivial fix(with new option) [works for case shown earlier].
However checking if it has leading underscore, but actually used (which should violate rule) - not so trivial and may be error prone.
I'll check it this weekend if nobody beats me.

#1481 no longer being deprecated


Add CLI tool and remove unused scripts

Fixes #22
Creates CLI tools grants_tagger with commands
grants_tagger COMMAND

Commands
* train
* pretrain
* preprocess (subcommands `bioasq-mesh` and `wellcome-science`)
* predict
* tag
* tune (subcommands `thresholds` and `params`)
* download
* explain
* evaluate

make tests fail on test_preprocess.py :(

i am aware, will fix today 👍

tests pass 👍

perfect, i will give them an opportunity to pass here as well and merge

Codecov Report

Merging #48 (fc59f13) into master (8a41b7e) will increase coverage by 6.61%.
The diff coverage is 32.14%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   42.27%   48.89%   +6.61%
==========================================
  Files          22       18       -4
  Lines        1204      947     -257
==========================================
- Hits          509      463      -46
+ Misses        695      484     -211

Impacted Files
Coverage Δ

grants_tagger/__main__.py
0.00% <0.00%> (ø)

grants_tagger/evaluate_human.py
0.00% <0.00%> (ø)

grants_tagger/evaluate_mesh_on_grants.py
0.00% <ø> (ø)

grants_tagger/evaluate_mti.py
0.00% <0.00%> (ø)

grants_tagger/evaluate_scispacy_meshtagger.py
0.00% <0.00%> (ø)

grants_tagger/optimise_params.py
0.00% <0.00%> (ø)

grants_tagger/pretrain.py
0.00% <ø> (ø)

grants_tagger/tune_threshold.py
74.35% <ø> (+8.06%)
:arrow_up:

grants_tagger/preprocess_mesh.py
71.42% <23.52%> (+8.46%)
:arrow_up:

grants_tagger/train.py
58.47% <35.00%> (+10.61%)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


Add precommit linters

Danger has a precommit hook for successful typescript compilation and tslint added by @kwonoj (https://github.com/danger/danger-js/commit/71ab9dbb28024e075508441de751169dcce51180#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R37) - emission should have this too

This should still happen. IT's really useful in danger.

These pre-commit linters are in.


Text editor destroy, _MOUSEUP event not removed from body

Destroy method leaves events attached on body, ui.js , line 1162, event should be removed from body, not doc.
event_reset(_MOUSEUP, body, do_resize_up);

Thanks. It works by move it to the body element but I decided to generalize it to the doc as sometimes body height can be less than the win height.


'$(...) is not supported" error showing in preview window under Fish shell

When opening a preview window using nvim under Fish, the following error shows at the top of the preview window:
fish: $(...) is not supported. In fish, please use '(file)'.
[[ "$(file --mime '<<FILE>>')" =~ binary ]] && echo <<FILE>>"
^

Where <<FILE>> is the currently selected file in the fzf list window.

Fixing this is simply a matter of setting the following in ~/.config/nvim/config.nvim:
let g:fzf_preview_if_binary_command = "string match 'binary' (file --mime {})"

I attempted to find a place in the plugin itself to add this, but do not see a hook for where to do so. Are there any suggestions about how best -- or even if -- to integrate this change?

I couldn't reproduce this problem using fish, but I was able to confirm it by setting the default shell to fish.
Changing fish to be the default shell can cause problems, so change it to run fish from bash.
You can run fish with bash as the default shell by writing exec fish in .bashrc or .bash_profile .

Try https://github.com/yuki-ycino/fzf-preview.vim/issues/96#issuecomment-607591685

Fix worked for me on ubuntu20 with fish shell and neovim


Implement Initial Sequence Support for MariaDB

Starting with version 10.3 MariaDB supports sequences. This PR introduce support through liquibase creating Sequences and default value for MariaDB. For example, following changelogs are supported through this PR:
<?xml version="1.0" encoding="utf-8" ?>
<databaseChangeLog
        xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-3.1.xsd">

    <changeSet id="1" author="bob">
        <comment>A sample change log</comment>
        <createSequence cycle="true"
                        incrementBy="2"
                        maxValue="1000"
                        minValue="5"
                        ordered="true"
                        sequenceName="seq_id"
                        startValue="5"/>
    </changeSet>

    <changeSet id="2" author="bob">
        <createTable tableName="person"
                     tablespace="A String">
            <column name="address" type="varchar(255)"/>
            <column name="id" type="int" />
        </createTable>
    </changeSet>

    <changeSet id="3" author="bob">
        <addDefaultValue
                         columnName="id"
                         defaultValueSequenceNext="seq_id"
                         tableName="person"/>
    </changeSet>

</databaseChangeLog>

@nvoxland, had you a change to look into this PR?

Had some early success playing with this feature. I encountered an exception when performing actions that generate snapshots. Maybe it's enough to add another case in SequenceSnapshotGenerator.getSelectSequenceSql(Schema, Database)?
Other than that, this is definitely a welcome addition!

Merged it in. Thanks!

Nice. :sunglasses:

Hi @schrieveslaach
Thank you for this Pull Request.
I’m putting together a core team of developers to help us with community requests. We’ll make Liquibase improvements faster together!
If you are willing to help, please reply to this comment and I’ll let you know the next steps.
Your friendly neighborhood developer advocate,
@molivasdat

@molivasdat, sorry for the delay, I was on vacation.
Thanks for the invitation to develop Liquibase further and I feel honored.
I still have to make sure that I have the time in order to fulfill the role as a core team member. What do you expect from a core team member and what do you think how much time a member would have to spent on a weekly basis?

Hi @schrieveslaach Please email me at<EMAIL_ADDRESS>and send me your email address and I send you the details.

@molivasdat, I wrote an email a couple days. Did you receive it?

Hi @schrieveslaach I did send an email.  Hopefully you received it.


Having a really hard time getting the font's working

I am trying to get font-awesome to work with vue pack.
I have added the following to my client/router/index.js:
require("font-awesome/css/font-awesome.css");

Now everything compiles fine and if I use the .fa class all the styles are applied the issue is the icons they are rendering really funny, at first I thought it may be an HMR issue so I tried just a plain build but this didn't seem to make a difference, is there something I am missing?

See #111. You have to remove (or comment out) the following section in webpack.base.js:
{
test: /.svg$/,
loader: 'raw-loader'
}

What the previous user failed to mention is that you also have to add svg to the file loader. In the same file, add svg| in test, like so:
{
    test: /\.(svg|ico|jpg|png|gif|eot|otf|webp|ttf|woff|woff2)(\?.*)?$/,   <--- here
    loader: 'file-loader',
    query: {
        name: 'static/media/[name].[hash:8].[ext]'
    }
}


Request Timeout

Does anyone know it's happening every time I try to recover a saved session, on timeout connection, and I can pull the qrcode normally, sometimes it works, and sometimes it doesn't.

Hello @Nelsonguiana.
Check that the cell phone has an internet connection and it is unlocked (see home screen) when reconnecting the session.

Hello @Nelsonguiana.
Check that the cell phone has an internet connection and it is unlocked (see home screen) when reconnecting the session.

There is some problem in my session, I deleted it and it came back thanks.


add a callback_plugin to have more human readable output

but don't enable it by default

Merged in https://github.com/liip/drifter/commit/82b861b088f36bf171866fce78ecdb6364d2aa02. Thanks! 🌞


Failing DDP race condition test on MI50

The following error messages are captured on MI50 server with the PyTorch build from IFU-master-2021-12-13 when running
HIP_VISIBLE_DEVICES=0,1 python -m torch.distributed.launch --nproc_per_node=2 DDP/ddp_race_condition_test.py
running DDP tests
/opt/conda/lib/python3.6/site-packages/torch/distributed/launch.py:186: FutureWarning: The module torch.distributed.launch is deprecated
and will be removed in future. Use torchrun.
Note that --use_env is set by default in torchrun.
If your script expects `--local_rank` argument to be set, please
change it to read from `os.environ['LOCAL_RANK']` instead. See
https://pytorch.org/docs/stable/distributed.html#launch-utility for
further instructions

  FutureWarning,
WARNING:torch.distributed.run:
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
i = 0
i = 0
model.a: grad.data_ptr() =<PHONE_NUMBER>58624, expected sum 16777216.0, got 16777216.0
model.a: grad.data_ptr() =<PHONE_NUMBER>71584, expected sum 16777216.0, got 16777216.0
model.b: grad.data_ptr() =<PHONE_NUMBER>77632, expected sum 8388608.0, got 8388608.0
model.b: grad.data_ptr() =<PHONE_NUMBER>87840, expected sum 8388608.0, got 8388608.0
rank 1 created group 0 with backend nccl
rank 1 created group 1 with backend nccl
rank 1 created group 2 with backend nccl
rank 0 created group 0 with backend nccl
rank 0 created group 1 with backend nccl
rank 0 created group 2 with backend nccl
Traceback (most recent call last):
  File "DDP/ddp_race_condition_test.py", line 52, in <module>
    loss.backward()
  File "/opt/conda/lib/python3.6/site-packages/torch/_tensor.py", line 352, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/opt/conda/lib/python3.6/site-packages/torch/autograd/__init__.py", line 175, in backward
    allow_unreachable=True, accumulate_grad=True)  # Calls into the C++ engine to run the backward pass
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 401, in allreduce_hook
    self.comm_ready_buckets(param)
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 535, in comm_ready_buckets
    self.allreduce_maybe_retain(self.buckets[bucket_idx], bucket_idx)
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 480, in allreduce_maybe_retain
    allreduced = self.allreduce_bucket(bucket, bucket_idx, force_default_stream)
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 450, in allreduce_bucket
    dist.all_reduce(tensor_to_allreduce, group=self.bucket_pgs[bucket_idx%self.num_allreduce_streams])
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 1314, in all_reduce
    work = group.allreduce([tensor], opts)
RuntimeError: NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1164, unhandled system error, NCCL version 2.8.4
ncclSystemError: System call (socket, malloc, munmap, etc) failed.
Traceback (most recent call last):
  File "DDP/ddp_race_condition_test.py", line 52, in <module>
    loss.backward()
  File "/opt/conda/lib/python3.6/site-packages/torch/_tensor.py", line 352, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/opt/conda/lib/python3.6/site-packages/torch/autograd/__init__.py", line 175, in backward
    allow_unreachable=True, accumulate_grad=True)  # Calls into the C++ engine to run the backward pass
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 401, in allreduce_hook
    self.comm_ready_buckets(param)
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 535, in comm_ready_buckets
    self.allreduce_maybe_retain(self.buckets[bucket_idx], bucket_idx)
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 480, in allreduce_maybe_retain
    allreduced = self.allreduce_bucket(bucket, bucket_idx, force_default_stream)
  File "/opt/conda/lib/python3.6/site-packages/apex-0.1-py3.6-linux-x86_64.egg/apex/parallel/distributed.py", line 450, in allreduce_bucket
    dist.all_reduce(tensor_to_allreduce, group=self.bucket_pgs[bucket_idx%self.num_allreduce_streams])
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 1314, in all_reduce
    work = group.allreduce([tensor], opts)
RuntimeError: NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1164, unhandled system error, NCCL version 2.8.4
ncclSystemError: System call (socket, malloc, munmap, etc) failed.
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 56622) of binary: /opt/conda/bin/python
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launch.py", line 193, in <module>
    main()
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launch.py", line 189, in main
    launch(args)
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launch.py", line 174, in launch
    run(args)
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/run.py", line 718, in run
    )(*cmd_args)
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launcher/api.py", line 131, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/opt/conda/lib/python3.6/site-packages/torch/distributed/launcher/api.py", line 247, in launch_agent
    failures=result.failures,
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
DDP/ddp_race_condition_test.py FAILED
------------------------------------------------------------
Failures:
[1]:
  time      : 2021-12-15_00:24:52
  host      : d740d63d6dfd
  rank      : 1 (local_rank: 1)
  exitcode  : 1 (pid: 56623)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
------------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2021-12-15_00:24:52
  host      : d740d63d6dfd
  rank      : 0 (local_rank: 0)
  exitcode  : 1 (pid: 56622)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
============================================================

Problem solved by using --shm-size=4g for docker run command on MI50 server.


Dataset retriever fails when trying to download gated dataset

Here is the output from my prompt_spec:
Instruction:
Task description: Identify a broad class given several examples from that class

Examples:
input=
Q: The similarity among DemandBase, InfusionSoft, and HotSchedules is that they are all
output=tech companies

input=
Q: The Architecture of Open Source Applications, Algorithms to Live By: The Computer Science of Human Decisions, and The Art of the Start: The Time-Tested, Battle-Hardened Guide for Anyone Starting Anything can be classified as
output=Computer Science books

input=
Q: Wrike, SEMrush, and Sprinklr are all
output=tech companies

Got the following error when trying to retrieve the dataset
FileNotFoundError: Couldn't find a dataset script at /projects/tir5/users/ssgandhi/prompt2model/bigbench/bigbench/BIG-bench/bigbench/imagenet-1k/imagenet-1k.py or any data file in the same directory. Couldn't find 'imagenet-1k' on the Hugging Face Hub either: FileNotFoundError: Dataset 'imagenet-1k' doesn't exist on the Hub. If the repo is private or gated, make sure to log in with `huggingface-cli login`.

@viswavi

Hey @saum7800, I took a look at this and if you read the error, it says that the dataset may be private or gated.
I looked at the specific dataset, and it seems that this is indeed the case: https://huggingface.co/datasets/imagenet-1k
There are two solutions to this:

Follow the instructions in the error message -- run the hugging face cli and get permission to use the data.
When you get this gated dataset error, gracefully proceed to using the next dataset.

"1." is a solution for this dataset, but you might always run into a new dataset that has problems, so I think "2." will need to be implemented. There are two ways that we could do this:

Simply write a for loop that steps over datasets (in the colab notebook and CLI?) and selects the next one any time the first one fails.
If there is a way to figure out if a dataset is gated through the hugging face API, we could indicate this in our metadata file.

Maybe we could just go with the first option for now.

Right, that makes sense!
In prompt2model_demo.py and .ipynb, user manually selects the dataset number/name. maybe it makes sense to inform the user that the dataset is gated and they should select another one from the retrieved datasets (assuming huggingface allows us to programatically know a dataset is gated). Does that sound right?

Yep. And in the worst case you could always catch the exception and programmatically parse the error message to see if it indicates that the model is gated.

This will be resolved once reranking PR is merged!


Can't find MvcMovie.csproj file

I'm trying to find MvcMovie.csproj file in solution explorer.

Duplicate of https://github.com/dotnet/AspNetCore.Docs/issues/26959.


z3: Add Java Bindings

Motivation for this change
This is my take on #91939
Things done
Introduce new arguments to build Java Bindings, and to disable building of Python bindings. Default behaviour is preserved.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build acl2 boogie cppcheck cuter dafny fstar isabelle petrinizer rsstail slither-analyzer solc tlaps vampire z3 z3-tptp zz


Some instances of LAS files written by adaptivefiltering are not readable

As reported by Michael.

Here is what happens: Before the "Finalize" button is clicked, the return object is a segmentation which conincidentally has a save method. Hitting the finalize button in ds.restrict() and then saving works fine.

Potential fix: create_segmentation returns a (potentially empty) dataset. If the user tries to save an empty dataset, he gets a telling error.


Doesn't start with Windows

On Windows 10 21H2, it looks like this app doesn't start after booting the computer and logging in. I have to manually launch it.
The option to start with Windows is checked inside the app, and Task Manager shows that this app is enabled for startup.
Version 1.3.0

I'm having the same issue on Windows 10 21H1. Let me know if there's anything I can do to help troubleshoot.

FYI, it also doesn't automatically update the wallpaper even though I have it set to do so hourly. I don't know if this is related or should be a new issue.

I was having the same issue (wallpapers not changing), I think you should go ahead and make that issue.


Remove duplicate suspend and resume operations.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[ ] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

There are duplicate entries for suspend and resume which causes AutoRest
to fail to parse the document as it's invalid.

Hi @jhendrixMSFT, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jhendrix). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@amarzavery FYI on removing duplicate entries.

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-ruby/pull/632

No modification for Python


[Bug] TEXT NOT VISIBLE IN DARK MODE

Is there an existing issue for this?

[X] I have searched the existing issues

Bug Description?
When the dark mode is apply the text in login and sinup page is not visible
Add Screenshots

What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue (Optional)
[X] I'm willing to provide further clarification or assistance if needed.

@KartikSaxena19 Aldready there was a issue and is solved now!

Not visible yet

@KartikSaxena19 Okay, its not visible for register function only. Do one thing make a new issue for this for register button only. Login one is fixed.

You need to fix this:

Make a issue i will assign and make PR quickly so that i can merge it.


add public and private threads to channel list

What would the FE look like right now if we enabled this? Screenshot possible?

just a long list with channels and threads mixed

just a long list with channels and threads mixed

No, this needs to be improved. We will need some data structure to pass down so we can display the heirarchy in UI.

@zberwaldt what kind of data would nested Select List require for this?

@zberwaldt what kind of data would nested Select List require for this?

This is data from a react app that has select in the way I think you are looking for:
We might have to roll our own select to handle this, I'm reading through the docks for react-select to see if we can handle this.
{
  "value": "834499078434979891",
  "label": "┌───  🤝Welcome  ───┐",
  "options": [
    {
      "value": "834504120307810344",
      "label": {
        "type": "div",
        "key": "834504120307810344",
        "ref": null,
        "props": {
          "className": "columns is-mobile is-vcentered",
          "children": [
            {
              "type": "div",
              "key": null,
              "ref": null,
              "props": {
                "className": "column",
                "children": {
                  "type": "p",
                  "key": null,
                  "ref": null,
                  "props": {
                    "className": "",
                    "children": [
                      {
                        "type": "span",
                        "key": "spany span",
                        "ref": null,
                        "props": {
                          "style": {
                            "color": "gray"
                          },
                          "className": "mr-1",
                          "children": {
                            "type": "i",
                            "key": null,
                            "ref": null,
                            "props": {
                              "className": "fas fa-hashtag"
                            },
                            "_owner": null
                          }
                        },
                        "_owner": null
                      },
                      "🔊announcements"
                    ]
                  },
                  "_owner": null
                }
              },
              "_owner": null
            },
            {
              "type": "div",
              "key": null,
              "ref": null,
              "props": {
                "children": [
                  {
                    "type": "div",
                    "key": null,
                    "ref": null,
                    "props": {
                      "className": "column is-pulled-right dropdown-disabled-text",
                      "children": "Invalid Write Permissions"
                    },
                    "_owner": null
                  },
                  {
                    "type": "div",
                    "key": null,
                    "ref": null,
                    "props": {
                      "className": "column is-narrow"
                    },
                    "_owner": null
                  }
                ]
              },
              "_owner": null
            }
          ]
        },
        "_owner": null
      },
      "channel_data": {
        "id": "834504120307810344",
        "name": "🔊announcements",
        "position": 0,
        "category_id": "834499078434979891",
        "category_name": "┌───  🤝Welcome  ───┐",
        "category_position": 3,
        "is_sesh_channel": false,
        "type": "Text",
        "has_write": false,
        "can_make_public_threads": false
      },
      "isDisabled": true
    },
   // other options...
  ]
}

@zberwaldt how about the API returns something like this, would that work?
        'channel_id<string>':
        {
           'channel_name': string,
            'threads':
               [
                  {
                     'thread_id': string,
                     'thread_name: string,
                     'thread_type: enum,
                   }
               ],
         },
}```

Seems like a good start; maybe can also add channel_type if we wanted to support Forum posts in the future?
Could also key the threads by thread_id same as channels


Something wrong with strings.TrimLeft when string start with '/v'

What version of Go are you using (go version)?

$ go version
go1.13.2

Does this issue reproduce with the latest release?
Not Sure
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

Usually we use strings.TrimLeft to trim some fixed prefix, however when exec strings.TrimLeft("./csv/visitor.csv", "./csv/"), the EXPECTED result is "vistor.csv", the REAL result is "istor.csv".
It's amazing.
https://play.golang.com/p/EeY2DKiWeRR
package main

import (
	"fmt"
	"strings"
)

func main() {
	fmt.Println("----------a-----------")
	a := "./csv/visitor.csv"
	fmt.Println(a)
	fff := strings.TrimLeft(a, "./csv/")
	fmt.Println(fff)

	fmt.Println("----------b-----------")
	b := "./csv/visitor.csv"
	fmt.Println(b)
	fff = strings.TrimLeft(b, "./csv")
	fmt.Println(fff)

	fmt.Println("----------c-----------")
	c := "./csv/visitor.csv"
	fmt.Println(c)
	fff = strings.TrimLeft(b, "./")
	fmt.Println(fff)
}

What did you expect to see?
----------a-----------
./csv/visitor.csv
visitor.csv
----------b-----------
./csv/visitor.csv
visitor.csv
----------c-----------
./csv/visitor.csv
csv/visitor.csv
What did you see instead?
----------a-----------
./csv/visitor.csv
isitor.csv
----------b-----------
./csv/visitor.csv
isitor.csv
----------c-----------
./csv/visitor.csv
csv/visitor.csv

I think you want https://godoc.org/strings#TrimPrefix instead.

Anyone could close this if you want.

Check the documentation: https://golang.org/pkg/strings/#TrimLeft:

TrimLeft returns a slice of the string s with all leading Unicode code points contained in cutset removed.
To remove a prefix, use TrimPrefix instead.

The TrimLeft function is removing every character from the cutest from the start of the string whereas what you’re looking is what TrimPrefix does, which is to remove a fixed prefix from the string.

Check the documentation: https://golang.org/pkg/strings/#TrimLeft:

TrimLeft returns a slice of the string s with all leading Unicode code points contained in cutset removed.
To remove a prefix, use TrimPrefix instead.

The TrimLeft function is removing every character from the cutest from the start of the string whereas what you’re looking is what TrimPrefix does, which is to remove a fixed prefix from the string.

yeah I have found this. stupid -_-
Thanks


Wait for lightning balances to be distributed

Otherwise we might try to open channels from nodes that can't spend any fund yet.
[EDIT] rebased on main after #257, ln on k8s is passing.

Excellent, thanks!


How do I show indent lines instead of eol?

On empty lines, I want to avoid showing the eol character as it breaks the guidelines. Can this be done?

They are not shown by default. Have you turned them on with :help g:indent_blankline_show_end_of_line?

Yeah, it was enabled. Thanks for pointing it out. It works as expected after disabling.


Docs: Timer.periodic could have better API documentation

This issue was originally filed by<EMAIL_ADDRESS>
URL (if available):
https://api.dartlang.org/apidocs/channels/dev/#dart-async.Timer@id_Timer-periodic
Issue/feedback:
According to the docs you can invoke a new Timer by using the named constructor "periodic" (Timer.periodic), which for its second argument takes in a "Function callback".
Problem: No where does it say that the given Function requires a Timer object as the parameter.
Meaning this wont work:
    new Timer.periodic(new Duration(seconds:1), () => print('timer'));
But this does:
    new Timer.periodic(new Duration(seconds:1), (Timer timer) => print('timer'));
I had to open up the source code to understand how the signature for the Function had to be. This could be clarified in the APIs.

The docs now show the type of the callback parameter in the signature.


php_unit_test_class_requires_covers should only affect methods, not classes

You can have coverage marked on every test method in your class, and this fixer will come along and mark the entire class as @coversNothing blowing away your coverage.
You also can't mark the class yourself since that will blow away per-test granularity.
This should only affect methods without @covers in classes without @covers
The PHP version you are using ($ php -v):
7.2.4-1
PHP CS Fixer version you are using ($ php-cs-fixer -V):
2.11.1 Grey Devil
The command you use to run PHP CS Fixer:
php-cs-fixer fix --rules="php_unit_test_class_requires_covers" t
The configuration file you are using, if any:
=> ....................................

If applicable, please provide minimum samples of PHP code (as plain text, not screenshots):

before running PHP CS Fixer (no changes):

<?php

class RandomTest extends PHPUnit_Framework_TestCase
{
    /**
     * @covers \RandomClass::randomMethod
     */
    public function testRandomMethod()
    {
        // ...
    }
}

with unexpected changes applied when running PHP CS Fixer:

<?php

/**
 * @coversNothing
 */
class RandomTest extends PHPUnit_Framework_TestCase
{
    /**
     * @covers \RandomClass::randomMethod
     */
    public function testRandomMethod()
    {
        // ...
    }
}

with the changes you expected instead:

<?php

class RandomTest extends PHPUnit_Framework_TestCase
{
    /**
     * @covers \RandomClass::randomMethod
     */
    public function testRandomMethod()
    {
        // ...
    }
}

Hi, thanks for report !
I do agree current situation may be unexpected, but I would say it's a bug on PHPUnit.
Consider this:
<?php

/**
 * @covers \RandomClass
 */
class RandomTest extends PHPUnit_Framework_TestCase
{
    /**
     * @coversNothing
     */
    public function testRandomMethod()
    {
        // ...
    }

    public function testBar() { /*...*/ }
}

testRandomMethod will not collect coverage, as it's annotated with coversNothing and annotation on method level, as it is considering more precise piece of tests, is taking as prio.
At the same time, testBar will collect coverage, as it will use less precise annotation of whole class.

in revert situation of
<?php

/**
 * @coversNothing
 */
class RandomTest extends PHPUnit_Framework_TestCase
{
    /**
     * @covers \RandomClass::randomMethod
     */
    public function testRandomMethod()
    {
        // ...
    }

    public function testBar() { /*...*/ }
}

testBar, as there is no method-level annotation, yet there is class-level annotation - shall follow class-level annotation and not collect coverage.
same time testRandomMethod shall treat it's own annotation as prio and collect coverage.
do you want to raise an issue at PHPUnit repo?

While I agree that PHPUnit is doing the wrong thing here, I'm pretty sure it's not going to change. PHPUnit has been doing it this way for a 'long long time' (tm) so I doubt they'd break BC on all coverage notations.
Their market share is also enough that this is essentially the 'default' behavior of coverage annotations.
Perhaps a config option is called for, but I think it should still default to affecting methods not classes

big :-1: for affecting all the methods, it gonna be super messy to add it to every single method of test class
Why assuming ahead that PHPUnit will ignore this issue? Simply raise it.
I would say it's kind of big edge cases nobody thoughts before, as covers annotations are designed to not be used together like that (only combination designed to be used together is @coversDefaultClass on class-level + @covers on method-level

I wouldn't bet to much on the long time thingy on PHPUnit, if you make a good case people there will adopt the new and better, we've some examples of that from the past :)
In the meantime, tweaking the fixer is still a nice improvement (IMHO)


how to deal with the volumes ?

How to deploy containers with volumes shares ? (volume / volumes-from)

Currently is a missing feature maybe It can be added easy.


List tasks as json

In order to list all available tasks with their dependencies, a custom task should be defined and registered via --tasks option. For instance, now WebStorm uses this code. Tasks are listed using this command:
grunt --gruntfile .../Gruntfile.js --tasks .../tasks _intellij_grunt_tasks_fetcher

It's not convenient to rely on internal Grunt API. Is it possible to add a new cli option that will output tasks with dependencies as json? Thanks.

Also, that would fix https://github.com/gruntjs/grunt/issues/1303.


Update download documentation

Update UG and DG for download command. Resolves #168

Codecov Report

Merging #196 (5a291fb) into master (dd37fe5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #196   +/-   ##
=========================================
  Coverage     70.68%   70.68%
  Complexity      629      629
=========================================
  Files            90       90
  Lines          1958     1958
  Branches        235      235
=========================================
  Hits           1384     1384
  Misses          482      482
  Partials         92       92

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 dd37fe5...5a291fb. Read the comment docs.


Saving in Linux (ubuntu) creates a new file instead of overwriting an existing file.

I have been working on windows where suppose for the first time if I save the drawing, with file-1.excalidraw then it saves in the destination folder say downloads.
🤔 What is desired?
Now, I am constantly making the changes and when I save the changes, it simply overwrites the existing file file-1.excalidraw. Which is desired.
👀 What is happening?
I am trying in ubuntu where if I save the changes, it creates a new file file-1(1).excalidraw and on the second save, it does the same file-1(2).excalidraw. (using firefox - haven't tested on chrome in Linux)
Now, I am not sure if this is the problem with linux or what.
Please fix, this is annoying.

@arnostpleskot @are since you are on linux could one of you confirm this ?

Hey, @ad1992 ! I've tried it, and it behaves in the described way when using Firefox. Chromium opens the native file select dialog.
Imho, this happens because Firefox has only partial support of the File System Access API that doesn't include saving files; therefore, it treats saved files the same way as regular downloads. This behavior should be the same on every OS when using a Firefox-based browser, but I can't confirm that atm.

@arnostpleskot yes thats expected since in firefox File System Access API isn't supported.
@AayushSameerShah I see you have updated the desc of the issue mentioning its happening on firefox so thats expected behaviour hence closing this issue.


Split get method

This change is

Coverage increased (+0.3%) to 69.756% when pulling 84c17607c834e30535b436a4ceb24808a38a5ea8 on oleksiyVeretiuk:659007975141383_refactor_listing_get into 1ba862ca60f56ff58378eeb0ca9a01085396a37a on openprocurement:ea_core_master.

Coverage increased (+0.3%) to 69.756% when pulling 84c17607c834e30535b436a4ceb24808a38a5ea8 on oleksiyVeretiuk:659007975141383_refactor_listing_get into 1ba862ca60f56ff58378eeb0ca9a01085396a37a on openprocurement:ea_core_master.


logy doesn't work for Curve and Path

%%opts Curve [logx=True logy=True]
holoviews.Curve((np.logspace(1, 3, 10), np.logspace(1, 3, 10)))

only results in an x-axis with log scale.

Right now you can only have one axis in logscale.
For me #554 fixes this.

Just merged #554, thanks for the fix!


Release version 1.0

@rultor release, tag is 1.0

@rultor release, tag is 1.0

@aliceice OK, I will release it now. Please check the progress here

@rultor release, tag is 1.0

@aliceice Done! FYI, the full log is here (took me 3min)


reactstrap

https://reactstrap.github.io/?path=/docs/components-accordion--flush @miyataken999

react flagment
https://qiita.com/bayarea-techblog/items/3c43c0fb545b570ef9a0

https://github.com/robinhuy/react-native-expo-examples


Interop project has documented Agent / Hub Support

In order for a Hub or Agent implementation to be supported, a document must be contributed to this repo describing how to leverage the Hub / Agent with at least 2 different DID Methods.
All code necessary to complete the linking should be included, such as protocol messages, public web service endpoints, authentication details, etc.

@codeglobally tagging you in for visibility into the community's request/needs

Specifically, we should focus on support for Write / Read.

Issuer Agent Creates Credential, Transfers it Subject Hub, Verifier can request Credential from Subject Hub.

@TelegramSam to add RFC / code links. May be a good opportunity to develop an "Aries implementers Guide".

Links, per our call yesterday:
The best collection of DID Comm related info is in the Reference Section of RFC 0005: https://github.com/hyperledger/aries-rfcs/blob/master/concepts/0005-didcomm/README.md#reference
In particular to our discussion, Message Types, Message Anatomy, and Agent Transports.
Code to connect a webserver to an Aries agent: https://github.com/hyperledger/aries-staticagent-python
The examples folder has relevant webserver examples.


Android touch support

Like the title suggested, this nice plugin runs smoothly in iOS but seems can not be touched and rotated on Android (tested on 2.2, they can rotate well though).
If possible, please kindly add the android support or simply expose some api (like "turn right") so user can use a button to rotate. I believe that will be a very interesting functionality and make more users love this little plugin.
Thanks.
Max

Max,
Android support is a must these days, I fully agree. It's just that developing webstuff for Android (especially Javascript) is still such a pain - emulators are rubbish and the performance of actual hardware is rather silly if I may say. Reminds me a lot the early times of computing ;)
Actually, both your suggested solutions will hopefully land in upcoming version.
(1) I just recently finaly figured up a way how to effectively debug on Android devices and am already on track of the problems in Android. I finally got it to react on interaction, but it's still awful slow. I'm working on this one right now and willl keep you updated in this thread.
(2) As part of another issue #20, Reel will also be "reelable" without any dragging as it will allow to turn left/right by clicking left/right half of the image.

Good news everyone! During the weekend I've put my hands on friend's shiny new Android 2.3.3 smartphone and tested the new development version with a little modification and... It worked! :)
A little background: In order to not overwhelm the browser by all drag events the browser may be sending (essentially every mousemove or dragmove), Reel syncs this with the timer, so that just one "pan" event per one tick is processed. To do that Reel maintains a private Boolean variable slidable, which basicaly turns false on every processed pan/slide and is turned back to true by every tick. And this is what's not happening under Android. Some googling revealed, setTimeout is being blocked when dragging and that means no slidable reset by the timer, hence no more than just one pan/slide.
Removal of the slidable part of "pan" handler's wrapping condition allows for working interaction as one would expect from a "touchy" device :) There's still some glitch in attachments of touch events, but once I know now that the dragging works and having the better/recommended way of events attachment already explored in one experimental branch, I think I can safely conclude, that Android support is a done deal for 1.2.

Finally, the glory will return to Android...  :D
Thanks for the super good news pisi !

And I've just verified with my shiny new Android 2.3.5, that it indeed works. From 2.3 up. Yay! :)

Current development branch contains support for touch interaction on Android OS available on 2.3+ (verified on 2.3.5 smartphone). @Martooo, can you please verify with your device? Thanks!

Thanks for the update. I currently do not have an Android 2.3 but I will upgrade or I will simply test it on my friend's phone and report back.

I was originally aiming for 2.2, but as it turned out there was a bug causing the setTimeout to not be fired while the view was dragged, so I got as far to be able to spin the Reel by touch, but be able to see just the change once the finger was released. No further. Bummer... Luckily, Google got this fixed in 2.3.

Calling for testers! Somebody with Android 2.3+ device, please, head on to the test sampler page of the current development branch, verify the interaction with the image is possible and drop me a note here. Thanks!

Yeah, tested and confirmed on Sumsung I9100 Galaxy. Work like charm!
I believe it is the time to release this version so that all others can see this exciting change.
May the force be with you!

Perfect! Thanks @Martooo
The time sure is now, but there is still one unresolved feature (#36) missing to be finalized. Then we're good to go.


Turkish translations and some edits

Getting started lessons full translate.

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree


Revert "Use better done file locations for some AuTests"

Reverts apache/trafficserver#8274

I got bit by this:
https://github.com/apache/trafficserver/issues/6909
My latest force push on #8274 passed locally and in the PR run (because it ran against the wrong commit) but seems to fail in CI after its merged in. I'll circle back on this PR, but in the meantime I'll unblock PRs by reverting this commit.

The problem turned out easy enough to fix so I created a PR for it:
https://github.com/apache/trafficserver/pull/8278


As a developer, I'd like to programmatically initiate household account renaming through apex

If using a custom naming format, such as "{{!FirstName} {!Middle_Initial__c} {!LastName}", the standard trigger on the Contact object will not flag a change to the Middle_Initial__c as a reason to re-evaluate the household naming. A custom trigger, however could detect that change and make a call to the NPSP class to handle the naming updates.

We should kick off renaming when the Household Naming Order field is changed.

**lurch: add

Tracking W-008268

We should also make an invocable method so people can kick it off from process builder, as well.


Listview - Resizable column widths

Category
[x] Enhancement
[ ] Bug
[x] Question
Currently we can only define minWidth/max-width. Im not sure if i just do not know how to properly set this, or if it is not available yet.

Thanks @haifoo for reporting this. I have just implemented the isResizable property on the IViewField interface. This allows you to specify for each field which can or should not be resizable.
const viewFields: IViewField[] = [
  {
    name: 'ListItemAllFields.Id',
    displayName: 'ID',
    maxWidth: 40,
    sorting: true,
    isResizable: true
  },
  {
    name: 'ListItemAllFields.Underscore_Field',
    displayName: "Underscore_Field",
    sorting: true,
    isResizable: true
  },
  {
    name: 'Name',
    linkPropertyName: 'ServerRelativeUrl',
    sorting: true,
    isResizable: true
  },
  {
    name: 'ServerRelativeUrl',
    displayName: 'Path',
    render: (item: any) => {
      return Link;
    },
    isResizable: true
  },
  {
    name: 'Title',
    isResizable: true
  }
];

This change will be part of the version 1.8.0 release. You can already test out the beta. More information about beta testing: https://sharepoint.github.io/sp-dev-fx-controls-react/beta/

Closing the issue as version 1.8.0 has just been released.

thanks for this!


compose: Show wildcard mentions according to policy.

Users are permitted to see  wildcard mentions for all/everyone during message composition in large streams according to restrict wildcard mentions policy.
Fixes #25429
Screenshots and screen captures:

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks for working on this, @lapaz17. Make sure you run the linter and commit & push the result.

LGTM

@amanagr it was used previously, just only within its module; it was exported only for testing I think.
Merged, thanks @lapaz17!