launching a Vue project using webview (C++)

What OS are you using (uname -a, or Windows version)?
windows 10
What programming language are you using (C/C++/Go/Rust)?
C++ with VS2019
What did you expect to see and what you saw instead?
I'm trying to launch a vue project.
The option using a single-page is easy to start, but the project assembled using the vui-cli is not loaded via index.html .
Is it even possible to run using webview, or is it necessary to run a third-party server, and connect to webview via localhost (I tried this option, it turns out).
my c++ code:
#include "webview.h"
#include <iostream>
#include <filesystem>

int main()
{

	webview::webview w(true, nullptr);
	w.set_title("Window");
	w.set_size(980, 650, WEBVIEW_HINT_NONE);
	w.set_size(980, 650, WEBVIEW_HINT_MIN);

	wchar_t rawPathName[MAX_PATH] = { 0 };
	GetModuleFileNameW(NULL, rawPathName, MAX_PATH);
	std::string executablePath = std::filesystem::path(rawPathName).parent_path().string();

        std::cout << executablePath << std::endl;

        //w.navigate("file://" + executablePath + "/simpleVue.html");
	w.navigate("file://" + executablePath + "/srv/index.html");

	w.run();
	return 0;
}

simpleVue.html - works

{{ message }}

simple vue example from tutorial - not works
index.html with js & css directories
Thanks!

Vue can't link js and css files. Solved the problem by editing index.html . It is necessary to delete the first "/" in the paths and linking to the relative address worked successfully for me.

Based on the above comment, I assume this issue can safely be closed.


shorten the name created for the container instance

Jobs run using an automatically created Container Instance.  Those have a limit of 63-char names.
The name is a combo of   <resource-prefix>-<user-prefix>-<uuid>
resource-prefix :  set by the Logic app when then container instance is created
user-prefix  : entered by the user, and put through slufigy to match requirements of Container Instance
uuid :  set by app, 8 chars to ensure job is unique
the "resource-prefix" too long at 'geneplexus-container-'     It is set in  /azure/aci_logicapp_template.json https://github.com/krishnanlab/geneplexus_app/blob/main/azure/aci_logicapp_template.json
as follows where the JOB name is a parameter to the logic app.   Rebuilding this at of 12/17 is failing so can't update immediately
"Create_or_update_a_container_group": {
    "runAfter": {},
    "type": "ApiConnection",
    "inputs": {
        "body": {
            "properties": {
                "containers": [
                    {
                        "name": "@{concat('geneplexus-container-',triggerBody()?['envvars']?['JOBNAME'])}",
                        "properties": {
                            // etc
                        }
                    }
                ]
            }
        }
    }

back-end architecture changes were deferred to after publication submission per Arjun/Chris.

this is fixed in the latest ARM template for the LogicApp in the /azure folder, but the currently published app/system in the resource group krishnanlab-geneplexus-dev still uses the previous LogicApp which has this limitation.  However this issue is made obsolete by using azure functions in #218, so closing.


Update Doc Links

Fixes #404 #311
What changes to the docs does this PR provide?

Fix Broken Links
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Correctly picked the right branch to base the change off (master for new features, live for typos/improvements)
[ ] For new pages, used the provided template
[ ] For new features, added an entry in the Table of Contents
[ ] Ran against a spell and grammar checker
[x] Contains NO breaking changes

Other information

Docs Build status updates of commit 795077f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Maintainer issue: yarn shipit not working

Bug
Not sure what is going on - may be related to https://github.com/sindresorhus/np/issues/460
The yarn shipit command to make a new release is not working for me at least. Running the underlying commands directly via yarn (not using np as the front-end) fails with similar error messages
When I attempt to run an npm publish that works, so I think the fix is to switch shipit back to np --no-yarn but I haven't tested it yet.
This is mostly a heads-up in case someone else attempts to do a release - the shipit command might fail and you'll need to do an npm publish manually, commit the version bump change, make a tag, git push, git push --tags, and draft the release etc all manually
@Johan-dutoit

trying np --no-yarn did not work for me, I'm publishing manually for now :-(
Command failed with exit code 1 (EPERM): yarn publish --tag next

yarn shipit is working for me now, even with 2FA on, but I have to do an npm login pretty recently before I attempt it or I get an EPERM. So that's my workflow now.


training hangs with lightning ddp and cloud dir?

🐛 Bug
Hi, we are using lightning with litdata on our local machine and aws s3 system. However, training would hang randomly during the very first iterations with ddp and remote cloud directory.
I tried several different configurations, but I'm not sure what I should check next.
GPU / Strategy / FileOn / results
1 / No DDP/ local ssd / OK
1 / No DDP/ remote(s3) / OK
8 / DDP/ local ssd / OK
8 / DDP/ remote(s3) / Stuck.
To Reproduce
I'm following the exact steps on the imagenet demo. And I write a trainer myself here.
Just run python train.py with different CUDA_VISIBLE_DEVICES is enough

Code sample
# train.py
import numpy as np
import lightning as L
import torch, torch.nn as nn, torch.utils.data as data, torchvision as tv, torch.nn.functional as F

class LitAutoEncoder(L.LightningModule):
    def __init__(self):
        super().__init__()
        self.decoder = nn.Sequential(nn.Linear(32, 128))

    def training_step(self, batch, batch_idx):
        loss = self.decoder(batch).mean()
        print(self.trainer.local_rank, loss)
        self.log("train_loss", loss)
        return loss

    def configure_optimizers(self):
        optimizer = torch.optim.Adam(self.parameters(), lr=1e-3)
        return optimizer

from lightning.data import StreamingDataset, StreamingDataLoader

class ImageNetStreaming(StreamingDataset):
    def __init__(self, ):
        if 1:
            input_dir = "s3:// xxxxx"
            cache_dir = None
        else:
            input_dir = "val"
            cache_dir = None

        max_cache_size = "200GB"
        super().__init__(
            input_dir = input_dir,
            max_cache_size = max_cache_size,
            shuffle = True,
        )

    def __getitem__(self, idx):
        data = super().__getitem__(idx)
        return np.float32(123.)

dataset = ImageNetStreaming()
dataloader = StreamingDataLoader(
    dataset,
    batch_size = 32,
    num_workers = 2,
    pin_memory = True,
    shuffle = True,
    drop_last = True
)

autoencoder = LitAutoEncoder()
trainer = L.Trainer()
trainer.fit(autoencoder, dataloader)

Expected behavior
Training should finish
Additional context
Due to some regulations here we can not put we data or training scirpts on lightning-studio. I'm not sure if something's wrong with our s3 bucket or our our network configuration.
One thing I notice is that even if the training stucks at some iterations(<50), we can still observe large network throughputs on our machine (around 100mb/s), but the local chunk directory( ~/.lightning/chunks) stops growing.

 A similar issue  is already open for Sagemaker.
Does it hang after 1st epoch?

@deependujha, Many thanks. BTW, the above code sometimes gives the FileNotFoundError (and the training loop continues for several iterations and hangs), and sometimes it just hangs. Not sure if it will help or not, but i'm still pasting it here.
Epoch 0:   0%|                                              | 3/10008 [00:04<4:05:42,  0.68it/s, v_num=3]1 tensor(-45.3273, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-45.3273, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                              | 4/10008 [00:05<3:41:55,  0.75it/s, v_num=3]1 tensor(-61.0723, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-61.0723, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                              | 5/10008 [00:05<2:57:38,  0.94it/s, v_num=3]Exception in thread Thread-3:
Traceback (most recent call last):
  File "/data/miniconda3/envs/pl/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/data/miniconda3/envs/pl/lib/python3.10/site-packages/litdata/streaming/reader.py", line 153, in run
    self._maybe_delete_chunks()
  File "/data/miniconda3/envs/pl/lib/python3.10/site-packages/litdata/streaming/reader.py", line 117, in _maybe_delete_chunks
    self._apply_delete(self._chunks_index_to_be_deleted.pop(0))
  File "/data/miniconda3/envs/pl/lib/python3.10/site-packages/litdata/streaming/reader.py", line 91, in _apply_delete
    os.remove(locak_chunk_path)
FileNotFoundError: [Errno 2] No such file or directory: '/data/.lightning/chunks/b515aeecb3a09f152677fce166405b10/1730182031.4955728/chunk-4-1309.bin.lock'
0 tensor(-76.8173, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                              | 6/10008 [00:05<2:45:20,  1.01it/s, v_num=3]1 tensor(-76.8173, device='cuda:1', grad_fn=<MeanBackward0>)
1 tensor(-92.5623, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-92.5623, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                              | 7/10008 [00:08<3:18:18,  0.84it/s, v_num=3]1 tensor(-108.3073, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-108.3073, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                              | 8/10008 [00:08<2:53:34,  0.96it/s, v_num=3]1 tensor(-124.0523, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-124.0523, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                              | 9/10008 [00:08<2:34:20,  1.08it/s, v_num=3]1 tensor(-139.7973, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-139.7973, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                             | 10/10008 [00:09<2:38:37,  1.05it/s, v_num=3]1 tensor(-155.5423, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-155.5423, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                             | 11/10008 [00:09<2:30:48,  1.10it/s, v_num=3]0 tensor(-171.2873, device='cuda:0', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                             | 12/10008 [00:09<2:18:17,  1.20it/s, v_num=3]1 tensor(-171.2873, device='cuda:1', grad_fn=<MeanBackward0>)
0 tensor(-187.0323, device='cuda:0', grad_fn=<MeanBackward0>)
1 tensor(-187.0323, device='cuda:1', grad_fn=<MeanBackward0>)
Epoch 0:   0%|                                             | 13/10008 [00:10<2:13:49,  1.24it/s, v_num=3]1 tensor(-202.7773, device='cuda:1', grad_fn=<MeanBackward0>)

Hey @rxqy. Could you try to add a try / except around it ?

Hi @tchaton. I think this should be on the lightning side?
Probablilty related: https://github.com/Lightning-AI/litdata/issues/411 . I'm observing a quite similar behavior, e.g., the exact same dataloader length with different num of gpus.
From the issue above, we should init our streaming dataloader after ddp init, but with lightning's trainer, we are initializing dataloader before ddp init, and the dataloader gives the correct length.
I wrote a pytorch ddp demo. With the exact same dataloader, we can finish training quite smoothly.
import os
import torch
import torch.distributed as dist
import torch.nn as nn
import torch.optim as optim
from torch.nn.parallel import DistributedDataParallel as DDP

import numpy as np
import lightning
from lightning.data import StreamingDataset, StreamingDataLoader
from train import ImageNetStreaming
from tqdm import tqdm

def demo_basic():
    torch.cuda.set_device(int(os.environ["LOCAL_RANK"]))
    dist.init_process_group("nccl")
    rank = dist.get_rank()
    print(f"Start running basic DDP example on rank {rank}.")
    # create model and move it to GPU with id rank
    device_id = rank % torch.cuda.device_count()
    model = ToyModel().to(device_id)
    ddp_model = DDP(model, device_ids=[device_id])
    optimizer = optim.SGD(ddp_model.parameters(), lr=0.001)

    dataset = ImageNetStreaming()
    dataloader = StreamingDataLoader(
        dataset,
        batch_size = 32,
        num_workers = 2,
        pin_memory = True,
        shuffle = True,
        drop_last = True
    )
    print(len(dataset), len(dataloader))

    for _, data in enumerate(tqdm(dataloader, disable=(rank!=0))):
        optimizer.zero_grad()
        outputs = ddp_model(data)
        loss = outputs.mean()
        loss.backward()
        optimizer.step()
        print(rank, loss)

    dist.destroy_process_group()
    print(f"Finished running basic DDP example on rank {rank}.")

if __name__ == "__main__":
    demo_basic()

Just to clarify, I made no code change to my litdata or lightning package.  And we are not using fabric in our trainer. Not sure what's the correct way to init streaming dataloader with lightning trainer.

You should instantiate the dataset in the setuo hook of the datamodule or directly within the dataloader hook

Hi @tchaton , what's the dataloader hook you are mentioning here?
I'm looking at the datamodule's doc here, it seems that only the setup method (for spliting train/test datasets) is called on every device, but not the train_dataloader method.

@tchaton  We're running into an identical issue. We also are getting:
GPU / Strategy / FileOn / results
1 / No DDP/ local ssd / OK
1 / No DDP/ remote(s3) / OK
8 / DDP/ local ssd / OK
8 / DDP/ remote(s3) / Stuck.

When using DDP with remote data, we get 1 iteration/second in terms of speed. After the 1st epoch, 15-16 steps run forward at 1 iteration/second and then training stalls for 3-5 minutes (no GPU utilization). Any ideas what the underlying issue could be?


Data links dead

It looks like the links on the data page, and the ref to the paper are dead https://github.com/anhaidgroup/deepmatcher/blob/master/Datasets.md

I just tried it now, it seems to be working - could you retry and let me
know? If there is an error message, please paste it here.
On Fri, Jan 10, 2020 at 9:30 AM Matt Judell<EMAIL_ADDRESS>wrote:

It looks like the links on the data page, and the ref to the paper are
dead https://github.com/anhaidgroup/deepmatcher/blob/master/Datasets.md
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/anhaidgroup/deepmatcher/issues/47?email_source=notifications&email_token=ABWE5ZPUH5LFV7N5QZ4NUNLQ5CWFBA5CNFSM4KFK746KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IFM5HMA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWE5ZNZTPNK4CINYXXZEJTQ5CWFBANCNFSM4KFK746A
.

If you are using a DNS inside wisc.edu, you may be getting a local mirror?
https://pages.cs.wisc.edu/~anhai/data1/deepmatcher_data/Structured.zip
Returns:
Sorry! The URL you requested was not found on our server.
Is it Sunday between 4 and 8 PM (CST)?
If so, the server may be undergoing regularly scheduled downtime.
Otherwise, please contact the maintainer of the referring page and ask them to fix their link.
Thanks!

For some reason you seem to have got an https link, try this:
http://pages.cs.wisc.edu/~anhai/data1/deepmatcher_data/Structured.zip
On Fri, Jan 10, 2020 at 9:41 AM Matt Judell<EMAIL_ADDRESS>wrote:

If you are using a DNS inside wisc.edu, you may be getting a local mirror?
https://pages.cs.wisc.edu/~anhai/data1/deepmatcher_data/Structured.zip
Returns:
Sorry! The URL you requested was not found on our server.
Is it Sunday between 4 and 8 PM (CST)?
If so, the server may be undergoing regularly scheduled downtime.
Otherwise, please contact the maintainer of the referring page and ask
them to fix their link.
Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/anhaidgroup/deepmatcher/issues/47?email_source=notifications&email_token=ABWE5ZIKU665FEPIDWRWI3DQ5CXK5A5CNFSM4KFK746KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIUVLHQ#issuecomment-573134238,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWE5ZNYLHACDYDF2NBIALLQ5CXK5ANCNFSM4KFK746A
.

Worked like a charm.  Many thanks!


Move custom resources to Dataset Environment extension

Feature or Bugfix

Refactoring

Detail

moved dataset related custom resources to Dataset module
created EnvironmentExtension for the custom resources
fix one small bug in creation of datasets - the KMS alias name defaulted to sse-s3 for created datasets, while it should be the bucketName

Relates
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I have only one question. The rest changes look good to me!


HEAD installs broken.

Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?

brew install pond --HEAD -v
...
==> go build -tags nogui -o /usr/local/Cellar/pond/HEAD-72b0708_1/bin/pond ./client
Error: Empty installation
/usr/local/Library/Homebrew/formula_installer.rb:592:in `build'
/usr/local/Library/Homebrew/formula_installer.rb:237:in `install'
/usr/local/Library/Homebrew/cmd/install.rb:269:in `install_formula'
/usr/local/Library/Homebrew/cmd/install.rb:149:in `block in install'
/usr/local/Library/Homebrew/cmd/install.rb:149:in `each'
/usr/local/Library/Homebrew/cmd/install.rb:149:in `install'
/usr/local/Library/Homebrew/brew.rb:87:in `<main>'

ls -1 /usr/local/Cellar/pond/HEAD-72b0708_1
INSTALL_RECEIPT.json
LICENSE
README.md
bin

/usr/local/Cellar/pond/HEAD-72b0708_1/bin/pond
>>> Pond...
>>> Pond keeps private keys, messages etc on disk for a limited amount of time and that information can be encrypted with a passphrase.

@vladshablinsky

😥I'm on it.

The problem is in update_commit. When we install pond we go to stage_deps which triggers resource downloaders fetches and on each of these fetches we rewrite last_commit of the formula with the commit of resource.


Fix confusing CTC documentation

From the docs it seems that CTC decoder have problems and it can't handle labelings like A B B, but if you will look at the code, it is not the case, everything is OK, just typo in the docs.

Can one of the admins verify this patch?

I signed it!

Hi, @Jihadik.
If you try this piece of code
import numpy as np
import tensorflow as tf

#A B B * B * B, where * means blank
abb_b_b = np.array([[1, 0, 0], [0, 1, 0], [0, 1, 0], [0, 0, 1], [0, 1, 0], [0, 0, 1], [0, 1, 0]]) # One hot encoding
abb_b_b = abb_b_b[None, :, :].transpose((1,0,2)) # Added batch axis and transpose (it's time major notation)

sess = tf.InteractiveSession()
merge = tf.sparse_tensor_to_dense(tf.nn.ctc_greedy_decoder(abb_b_b, seq_len)[0][0]).eval()
non_merge = tf.sparse_tensor_to_dense(tf.nn.ctc_greedy_decoder(abb_b_b, seq_len, merge_repeated=False)[0][0]).eval()

print(merge)
print(non_merge)

you'll get this output
[[0 1 1 1]] # Which means ABBB
[[0 1 1 1 1]] # Which means ABBBB. There is a extra 1 in the docs!

So, the docs seems to have a problem, but not the problem that you've point to. If you look at this line of code, you can see that if merge_repeated = False, then the greedy decoder will append to output only the maximum target if it isn't blank. In case of merge_repeated=True, the maximum target will only be appended to output if it isn't the blank label and if it isn't equal the previous label.
Look at the comment here. As you can see, there is a typo. Instead of * A B B B B B if merge_repeated=False., it should be * A B B B B if merge_repeated=False.
So, I recommend you to change your PR to fix this typo! :)

@tensorflow-jenkins test this please.

@tensorflow-jenkins test this please.

You're right, thanks!

@tensorflow-jenkins test this please.

Are we merging it, @ebrevdo?

Let's pause on it.  I need to find out how many people the true fix would break.

Ping for @ebrevdo -- let me know approximately when you think you'll have found your answer?

Btw, what is the problem with merging comments? :)

As I thought :), thanks!

@vrv, I fixed it!

(Thanks, now will wait for @ebrevdo to review)

@ebrevdo, ping!

It's not the documentation that's wrong - it's the code.  Would you like to fix the code instead?

No problems, I will look at the code a little bit later.

Can confirm that I ran into this the other day. I was doing some training with labels that had repeated symbols, but it would output the warning "No Valid Path Found", and would return a cost of infinity.

Ling, I'm not sure that is related.
On Nov 20, 2016 5:53 AM, "Ling Zhang"<EMAIL_ADDRESS>wrote:

Can confirm that I ran into this the other day. I was doing some training
with labels that had repeated symbols, but it would output the warning "No
Valid Path Found", and would return a cost of infinity.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/5194#issuecomment-261779617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim_DFjlGmmPt89m0z6vbeibCvFcmaks5rAFDGgaJpZM4KgErU
.

Actually yeah, I investigated further and it was just that I did not make the seq_len long enough to allow for the blanks to be between the labels, so thus "No Valid Path Found"
Though I'm confused about your earlier comment @ebrevdo where you said it's not the documentation but the code that is wrong. The unit test linked and @igormq seems to show merge_repeated demonstrating the correct behavior?

The unit test is either wrong, or doesn't actually test repeats correctly.
On Nov 20, 2016 10:10 AM, "Ling Zhang"<EMAIL_ADDRESS>wrote:

Actually yeah, I investigated further and it was just that I did not make
the seq_len long enough to allow for the blanks to be between the labels,
so thus "No Valid Path Found"
Though I'm confused about your earlier comment @ebrevdo
https://github.com/ebrevdo where you said it's not the documentation
but the code that is wrong. The unit test linked and @igormq
https://github.com/igormq seems to show merge_repeated demonstrating
the correct behavior?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/5194#issuecomment-261794277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtimz6ECS67TyVxKr4Xvcxboa0bJzknks5rAI0RgaJpZM4KgErU
.

Ping! What's the next step for this?

Soon I will try to fix it. I was really busy last few weeks.

I have run into several issues in the CTC decoding.

The greedy decoder seems to work right but has a problem with the documentation (still). Documentation online still shows
A B B * B * B (where '*' is the blank label) becomes
A B if merge_repeated=True.
A B B B B B if merge_repeated=False.

Both results are wrong.
This should read:
A B B B if merge_repeated=True.
A B B B B  if merge_repeated=False.

I had a problem with the beamsearch decoder. Running the code of igormq above, but switching in the beamsearch decoder in place of the greedy decoder via:
`merge = tf.sparse_tensor_to_dense(tf.nn.ctc_beam_search_decoder(abb_b_b, seq_len, merge_repeated=False)[0][0]).eval()

non_merge = tf.sparse_tensor_to_dense(tf.nn.ctc_beam_search_decoder(abb_b_b, seq_len, merge_repeated=False)[0][0]).eval()
print(merge)
print(non_merge)
`
the results for both are:
[[0 1 0 1]] or A B A B
which makes no sense.
Additionally, I have also had the problem where the beamsearch decoder did not properly deal with repeated characters and had to switch merge_repeated to False.
In summary, documentation problems in greedy decoder,  code problems in beamsearch decoder.

You have merge_repeated=false in both cases for your beam search code. For
greedy decoder, the documentation isn't wrong, the code is. I'll look into
the beam search decoder in early January.
On Dec 13, 2016 3:22 AM, "saseptim"<EMAIL_ADDRESS>wrote:

I have run into several issues in the CTC decoding.

The greedy decoder seems to work right but has a problem with the
documentation (still). Documentation online still shows
A B B * B * B (where '*' is the blank label) becomes
A B if merge_repeated=True.
A B B B B B if merge_repeated=False.

Both results are wrong.
This should read:
A B B B if merge_repeated=True.
A B B B B if merge_repeated=False.

I had a problem with the beamsearch decoder. Running the code of
igormq above, but switching in the beamsearch decoder in place of
the greedy decoder via:
`merge = tf.sparse_tensor_to_dense(tf.nn.ctc_beam_search_decoder(abb_b_b,
seq_len, merge_repeated=False)[0][0]).eval()

non_merge = tf.sparse_tensor_to_dense(tf.nn.ctc_beam_search_decoder(abb_b_b,
seq_len, merge_repeated=False)[0][0]).eval()
print(merge)
print(non_merge)
`
the results for both are:
[[0 1 0 1]] or A B A B
which makes no sense.
Additionally, I have also had the problem where the beamsearch decoder did
not properly deal with repeated characters and had to switch merge_repeated
to False.
In summary, documentation problems in greedy decoder, code problems in
beamsearch decoder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/5194#issuecomment-266736700,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtimxvHCRCkf9gd3YMhAeF2t6nmSgdJks5rHpv8gaJpZM4KgErU
.

Regarding the beamsearch, you are right, I copied the code wrong (now it is fixed) but the results are the same - [[0 1 0 1]] - still incorrect.
As for the greedy decoder, I do not understand why you think the code is wrong. The code acts for me as expected. The comments, on the other hand, are certainly wrong, just the fact that the result has 5 B's and the input has 4.

For greedy beam search, the code should act like the documentation... Not
the other way around. It's a bug.
On Dec 13, 2016 11:12 PM, "saseptim"<EMAIL_ADDRESS>wrote:
Regarding the beamsearch, you are right, I copied the code wrong (now it is
fixed) but the results are the same - [[0 1 0 1]] - still incorrect.
As for the greedy decoder, I do not understand why you think the code is
wrong. The code acts for me as expected. The comments, on the other hand,
are certainly wrong, just the fact that the result has 5 B's and the input
has 4.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/5194#issuecomment-266981939,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim0_STlBnqmWH-4NVgaQ4b7x_TCRtks5rH7LjgaJpZM4KgErU
.

@ebrevdo, I really disagree with you. If merge_repeated=True then it is exactly Graves' CTC and in case A B B * B * B firstly it merges repeated labels and make A B * B * B then it removes all blanks and our sequence become A B B B, not A B as in docs now. If it firstly remove all blanks and the merge labels then we will never have two consecutive labels in a row which is not good and was not inteded by Graves.
If I'm wrong, please prove it.

I agree, there is a problem with the documentation of the greedy decoder and the code of the beamsearch decoder.

@Jihadik I really appreciate your willingness to contribute. Basically, the way it works is that google staff is maintaining the code and needs to make the final decision on changes we're comfortable with. We can't really merge @ebrevdo has a different opinion.

I'll revisit tomorrow.
On Jan 11, 2017 3:50 PM, "drpngx"<EMAIL_ADDRESS>wrote:

@Jihadik https://github.com/Jihadik I really appreciate your
willingness to contribute. Basically, the way it works is that google staff
is maintaining the code and needs to make the final decision on changes
we're comfortable with. We can't really merge @ebrevdo
https://github.com/ebrevdo has a different opinion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/5194#issuecomment-272032990,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim3hTJN830TME1FmFxFfujCZTINu7ks5rRWrAgaJpZM4KgErU
.

@Jihadik, you are right.  lgtm.
On Jan 11, 2017 4:00 PM, "Eugene Brevdo"<EMAIL_ADDRESS>wrote:

I'll revisit tomorrow.
On Jan 11, 2017 3:50 PM, "drpngx"<EMAIL_ADDRESS>wrote:

@Jihadik https://github.com/Jihadik I really appreciate your
willingness to contribute. Basically, the way it works is that google staff
is maintaining the code and needs to make the final decision on changes
we're comfortable with. We can't really merge @ebrevdo
https://github.com/ebrevdo has a different opinion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/5194#issuecomment-272032990,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtim3hTJN830TME1FmFxFfujCZTINu7ks5rRWrAgaJpZM4KgErU
.

Jenkins, test this please.


account subcommand: show mint/freeze authority for mint account

Presently a mint only shows
+--------------+---------------------------------------------+
| Mint Account | METADDFL6wWMWEoKTFJwcThTbUmtarRJZjRpzUvkxhr |
+==============+=============================================+
|   Decimals   | 9                                           |
+--------------+---------------------------------------------+
|    Supply    | 20,885.999862108                            |
+--------------+---------------------------------------------+

but should probably also show mint authority and freeze authority.

closed by #9 and follow up commit


How to get transactions by accountBlock?

By your example.
slcTx := txKV.Value
var rawTx tlb.Transaction
if err = tlb.LoadFromCell(&rawTx, slcTx.MustLoadRef()); err != nil {
return fmt.Errorf("faled to parse transaction: %w", err)
}
But as ton's doc said, transaction's type is as below:
acc_trans#5 account_addr:bits256
transactions:(HashmapAug 64 ^Transaction CurrencyCollection)
state_update:^(HASH_UPDATE Account)
= AccountBlock;
slcTx's type isn't CurrencyCollection??
why CurrencyCollection has a ref?
why the code as below not work?
err := tlb.LoadFromCell(&txb, txKV.Key.MustLoadRef())
Do you have any doc that explain how to load cell?

HashmapAug is compatible with Hashmap for parsing, augmented element concatenated with value, so first you should parse CurrencyCollection from slice and then load tlb.Transaction, then should work

got it!


"Visual Studio 2017 Tools for Azure Functions" under "Next steps" links to non existent page (404)

The passage links to https://devblogs.microsoft.com/aspnet/azure-function-tools-for-visual-studio-2017/ which results in a 404.

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

ID: 369aefa6-74da-3001-f835-15abdfffd5d6
Version Independent ID: 2cc2a2db-d059-3252-7a1a-4261b7d3c37c
Content: Develop Azure Functions using Visual Studio
Content Source: articles/azure-functions/functions-develop-vs.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@dominikmontada thanks for letting us know about this bad link.

@mike-urnun-msft thanks, but we've already addressed this issue 😁


Sigma mints are shown as pending even when confirmed if migrating from old wallet

Actual behaviour
Sigma mints never show as confirmed and are forever pending even after 6 confirmations.
They however can be spent if using coin control.
Confirmed to be purely a GUI issue but troubleshooting as to why it only happens on migrated old wallets.
Desired behaviour
Sigma mints should be shown as available after 6 confirmations.

Solved by #566


Credit Card Fraud Detection

Credit Card Fraud Detection using machine learning.

GSSOC Participant .
I would like to work on this issue.

Hey, @aka9ksha  can you try to work on a little bit different issue, as it is very common.

Alright! :)


fix: typos

Fix incorrect fields in registry/paraswap/calldata-AugustusSwapper.json
Fix path typo in registry/paraswap/calldata-AugustusSwapper.json
Fix path typo in registry/uniswap/calldata-UniswapV3Router02.json

Validation will pass when https://github.com/LedgerHQ/python-erc7730/pull/87 is merged


Add clear-pr-merge-commit-message feature

Closes https://github.com/sindresorhus/refined-github/issues/2564

Test
Find a PR:

you can merge
has some Co-Authors (or doesn't)

How about we present a list of commit titles underneath or something? So I can still scan it, but it's not included by default.

I am nervous that not having the commits at all will cause some backlash

https://github.com/sindresorhus/refined-github/issues/2579 would help


Removed spaces when sending parameters in header for some implementations

I've noticed that some oauth-server implementations choke on the empty space after a key so that the rest of the parameter-keys in the header don't get evaluated and the server will throw a 400 Bad Request.
Since this is probably a wrong implementation on the server-side, it isn't that big of a problem in this lib. But it can't be wrong to make the client more flexible to work with these situations without breaking things; this commit fixes the problems I was facing. Existing test modifications consist of making them work without the spaces after a comma, which they do now.
I am just going ahead and send in this pull request; decide for yourself if it is going to be pulled in.
My project def. needs this patch; caused me headaches and several hours of digging and poking around in Wireshark ;)

Hi,
I have recently come onboard to help maintain oauth-xx.
Thank you so much for your contribution, really, big or small, help is always welcome.
But this is a very old pull request and therefore I have to assume this problem has probably already been solved by somebody else and this PR was forgotten in time.
It would be very good if you could rebase or re-patch this so we can all merge this.
If you do believe there is any chance this issue has not been fixed, please click on re-open and I will come help you with it.
Best,
James Pinto.


[revert] [FLINK-3944] [tableAPI] Reverts "Add rewrite rules to reorder Cartesian products and joins."

This reverts commit 85793a25a78ba5be96fabc2a26569318c6b53853.
Added rewrite rules blow up search space which cannot be effectively pruned without cardinality estimates.

merging


ie 11 not working in asp.net webform

G6 Version:3.5.10

Platform: Asp.net webform

Mini Showcase(like screenshots):

CodePen Link:

I use G6 in my asp.net webform application to show the bpm process!
it works well in Chrome, but not working in IE.
I tried to download the source code and rebuild the code and copy the dist code to my application, it's not working too,  then i see the "Object.assign" is still in g6.min.js.
I see the official solution to compatible with IE is for vue,react,umi... but not for asp.net webforms.
I appreciate your help. thanks.

G6 can not support IE @Tony518

es6 is not working on IE，use polyfill please @Tony518

Thanks richard,
Do you mean I download the source code of G6, and add polyfill then rebuild?

@Tony518 You can import babel-polyfill before import G6


Faulty RPC get_events parameter

Describe the bug (observed vs expected behavior)
The RPC get_events method declaration doesn't conform to the specification, a single filter parameter should be used (link).
For the payload:
{
    "jsonrpc": "2.0",
    "method": "starknet_getEvents",
    "params": {
        "filter": {
            "from_block": "latest",
            "to_block": "latest",
            "chunk_size": 1
        }
    },
    "id": 0
}

testnet replies with a successful response
devnet replies with the error code 22 and message "get_events() got an unexpected keyword argument 'filter'"

Not reproducible on alpha-goerli

[x] This issue is only present on Devnet and cannot be reproduced on alpha-goerli (check the box if true).

Devnet version

I am using Devnet version: 0.4.4
[x] This happens with a dockerized Devnet (check the box if true)

Hi, @penovicp I'll take a look at it today. I remember that there were 2 versions and we implemented a newer one.
Related issue: https://github.com/Shard-Labs/starknet-devnet/issues/231

Any updates here?

I'm on it, I have unfinished code on my local.


[quickstarts] (Visual Studio) Add note about console for VS

Fixes #1676.
This gives quick-start users information about finding the console.log output the quick start add-ins generate.

Docs Build status updates of commit c277b2c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/includes/console-tool-note.md
:white_check_mark:Succeeded

docs/quickstarts/excel-quickstart-jquery.md
:white_check_mark:Succeeded

docs/quickstarts/powerpoint-quickstart.md
:white_check_mark:Succeeded

docs/quickstarts/word-quickstart.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Move lua binding wrappers into separate module

Increasing separation of pure haskell code, wrappers, and
raw C API functions.

@sproctor feedback welcome

Coverage remained the same at 69.418% when pulling 3a1b1aa9b0a27298f3dad786b8c5368063ea8de2 on tarleb:bindings-module into d7f2bce61808c9738dbf42aab24f95894f7c8c40 on osa1:master.

I don't love this. It seems almost arbitrary which file a function definition ends up in. I could get behind this if there was a higher level API and a lower level API. newcfunction and freecfunction are firmly in the latter, IMO. What would this higher level API look like? Stack manipulation functions and convenient wrappers to execute Lua code? I'll do some thinking and try to come up with a proposal in the next few days.

You have a point.  My idea was to put all functions that are directly related to c_lua_* functions into a separate module, but I guess you are right in that this detail is irrelevant to users.
The reason putting those functions into a separate module is that we can then have one module whose interface can be clearly defined, no matter which lua version was used. I'll see if I can think up a better way to achieve this.

I pushed some changes that make this PR obsolete. The new module structure isn't optimal yet. Currently there is Foreign.Lua.Functions for (more or less) plain monadic wrappers of the C API functions, and Foreign.Lua.Interop for all things enabling data transfer between haskell and lua. I'm not very fond of the name of the latter. That module should be restructured; most of it would better fit into a  submodule of Types.


support for openlp 3.0

new version with support of openlp 3.0, version can be selected in config, feedbacks structure updated

I think you might need an upgrade script for this so that the self.config.version property is not undefined for older instances/users of this module. If I recall, it is not enough to set a default value for the property in the config fields, because that won't actually get set until the user goes to edit the module config and clicks update.
Also I noticed that you throw a warning for an action not supported in version 2 - but since you know the version, what about just not even providing that action as a choosable option if that's the version they selected?

upgrade script added, v3 only function hidden when running v2


create sonic builder and server [development_sonic]

[x] create builder for sonic
[x] create jumpscale server for starting and configuring sonic server

verified by

test sonic in wikis and make sure it works successful by follow steps here
https://github.com/threefoldtech/digitalmeX/tree/development/docs/publishing_platform
test that sonic builder works by j.builders.appa.sonic.build/install()


fix: padding at input that has no label

resolve #327

I tried this and it fixes the case when the label is empty. But there is a similar problem on the bottom when the helper text is empty, there some unused space on the bottom.


[BUG] Explicit Optimizer naming in logs

DeepSpeed log should explicitly state the optimizer details including

Provider: Torch vs DeepSpeed vs Apex
Variant: AdamW vs Adam
Fused vs NonFused

Example: https://github.com/huggingface/transformers/blob/main/src/transformers/training_args.py#L122-L131

@stas00, FYI

@xiexbing, DS currently prints out optimizer name here. We need to update printed name to be more informative.

the function is added and this PR is merged to the master


Escape . in the tooltips of the one-click review radio buttons

The regular expression /.$/ did not escape the .. Observe:

Two of these do not end with a . What /.$/ did was transform them all to reduce one character at the end, leading to this possibility:

Simply escaping the . will resolve this issue:
https://github.com/refined-github/refined-github/assets/33201955/81cd80b2-8315-4689-bdc3-e5a8a3ee19ca

I'm stupid 🥲 😂
Thank you for the fix, I forgot to escape it


IMN-252 - Risk Analysis Validation

Closes IMN-252
💡 looks scarily big, but most of the diff are JSON templates
Scala code is in interop-commons/risk-analysis
Tests
Automated tests based on the ones in the Scala codebase pass ✅

@galales addresses / replied to all comments - ready for review again :)


exc_bad_access when setting your own factory on SafariURLHandler

Description:
When i set my own custom factory closure i get an exc_bad_access crash. This does not crash on version 1.2.1.
let urlHandler = SafariURLHandler(viewController: self, oauthSwift: oAuth, modalPresentationStyle: .overFullScreen)
urlHandler.factory = { url in
    Theme.setButtonTintColorForSavariWebView()
    let safariViewController = ChangedColorSafariViewController(url: url)
    safariViewController.preferredBarTintColor = .primaryColor
    safariViewController.preferredControlTintColor = .white
    return safariViewController
}

OAuth Provider? (Twitter, Github, ..):
OAuth Version:

[ ] Version 1
[x] Version 2

OS (Please fill the version) :

[x] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[x] CocoaPods
[ ] Manually

Library version:

[ ] head
[x] v1.2.2
[ ] v1.2.1
[ ] v1.2 (Swift 4.0)
[ ] v1.0.0
[ ] v0.6
[ ] other: (Please fill in the version you are using.)

Xcode version:

[x] 10.1 (Swift 4.2)

[ ] 9.3 (Swift 4.1)

[ ] 9.0 (Swift 4.0)

[ ] 9.0 (Swift 3.2)

[ ] 8.x (Swift 3.x)

[ ] 8.0 (Swift 2.3)

[ ] 7.3.1

[ ] other: (Please fill in the version you are using.)

[ ] objective c

Scratch all this, it was an annoying xcode compiling issue. I had to do a deep clean on the project cmd + shift + option + k. After that it worked...


Remove git dependencies

Currently the following dependencies are sourced from git forks made by ben:

[x] #223
[x] redis https://github.com/shotover/shotover-proxy/pull/128

We should upstream or remove reliance on the changes in these repos.

Closing because we have #224 tracking our last git dep


Adding announcement CBL-Mariner - removing CBL-D

Replacing CBL-D update announcement with CBL-Mariner update.

@theJasonHelmick, dont merge this.  We will post this as a blog instead and link to the release which has the included tools listed.

Will not merge 😉
From: Danny Maertens @.>
Date: Wednesday, July 27, 2022 at 2:08 PM
To: Azure/CloudShell @.>
Cc: Jason Helmick @.>, Mention @.>
Subject: Re: [Azure/CloudShell] Adding announcement CBL-Mariner - removing CBL-D (PR #224)
@theJasonHelmickhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FtheJasonHelmick&data=05|01|jason.helmick%40microsoft.com|f147bcc7b6a840ca33c408da701428ba|72f988bf86f141af91ab2d7cd011db47|1|0|637945529143814406|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=vN0O99zUt7v2jYMPBtQqM8E7SpVp3cY4CDtrSOHYxsI%3D&reserved=0, dont merge this. We will post this as a blog instead and link to the release which has the included tools listed.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FCloudShell%2Fpull%2F224%23issuecomment-1197369962&data=05|01|jason.helmick%40microsoft.com|f147bcc7b6a840ca33c408da701428ba|72f988bf86f141af91ab2d7cd011db47|1|0|637945529143814406|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oxziCyYEqb5HWZNpmyh5ERjqYP41qkhfaOAqs46bulE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADATMBVUMJXTLYXQDIGR7O3VWGQM5ANCNFSM542UYDIA&data=05|01|jason.helmick%40microsoft.com|f147bcc7b6a840ca33c408da701428ba|72f988bf86f141af91ab2d7cd011db47|1|0|637945529143814406|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=P2JCs8EbGMei48sA3pc%2B0%2BQ9qEpZVWS9dDDa5n7HgoI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Closing as content has been released as a blog and release notes


Review and improve asynchronous support

In relation with: https://apacheignite-net.readme.io/docs/asynchronous-support

Consider removing WithCancellation extension
Review multi-threaded use of IgniteClient

Linked to #18 and partially fixed in d70ae7297c1f94f66dbcd12c00bd22604094663e

Obsolete by #39


createQuickPick API: missing intellisense for property values

Issue Type: Bug
Test #52820
Unsure if this is related, but when I attempted to use ${state.name} on a property there was no intellisense to help guide me towards the right value (after selecting state nothing was shown to proceed).
Expected

Result

VS Code version: Code - Insiders 1.25.0-insider (923e9ccb3fc44bfaaf4b4049015c84e42bc40f95, 2018-06-26T13:00:22.932Z)
OS version: Darwin x64 17.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

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

Load (avg)
3, 2, 2

Memory (System)
16.00GB (0.08GB free)

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

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

svgo
100
0.8.0

vscode-svgo
ajh
0.1.0

gitlens
eam
8.4.1

python
ms-
2018.6.0

(4 theme extensions excluded)

This works for me:

Does it work after reloading the window?

@chrmarti this comes back after reloading the window. Is this a bug worth looking into or should we close it?

Does any intellisense work in that case?

@mjbvz it does work after reloading the window. In my original bug, I had the window open for an extended period of time (a few hours) and am wondering if something timed out.

OK. Let me know if you hit this again


Implement new Transform API

Issue #101

Don't merge this. Broken for Library Projects. Investigating...

@JakeWharton could you have a look at this fantastic contribution and possibly merge it?

FWIW I've packaged this branch up and it's available for public use at https://jitpack.io/#Turbo87/hugo/1.2.2
I've also just tested this on a mixed Java and Kotlin Android app and it seems to work quite well. Good job @williamwebb!

@Turbo87 How exactly do I include it in my build.gradle?
buildscript {
    repositories {
        maven { url 'https://jitpack.io' }
    }
    dependencies {
//        classpath 'com.jakewharton.hugo:hugo-plugin:1.2.2-SNAPSHOT'
          classpath 'com.github.Turbo87.hugo:hugo-annotations:1.2.2'
    }
}
apply plugin: 'hugo'

It doesn't seem to recognise the plugin id's : com.jakewharton.hugo, hugo or com.github.Turbo87.hugo.

I couldn't get it to work. Please provide specific instructions.

@Turbo87 would be great if you could provide us with more information on how to use this. Removed apply plugin ... and replaced it with your jitpack implementation, but gradle fails.

@guidedways
I could get it to work with
buildscript {
    repositories {
	...
	maven { url 'https://jitpack.io' }
   }
    classpath `com.github.Turbo87.hugo:hugo-plugin:1.2.2`
}

and
apply plugin com.jakewharton.hugo


DataContainer error at training stage

When I train singlesStageDetector like SSD, dataloader with pipline DefaultFormatBundle.
dataloader output list of DataContainer object., then these DC object go to function forward_train()
in SingleStageDetector.
forward_train(img) img should be torch.Tensor type but got DC type
TypeError: conv2d(): argument 'input' (position 1) must be Tensor, not DataContainer
any solutions?

It seems that your model is not encapsulated by either DistributedDataParallel or DataParallel.

I use DataParallel in pytorch, when I change to mmcv.parallel.MMDataParallel back to norm~
Thanks.

Thank you!


Create gm.txt

https://phongdl.uit.edu.vn/gmail-cho-sv

@vh1509 I am sorry, the domain uit.edu.vn is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


Remove jQuery requirement?

Any chance you'd consider a pure JS version of this?

At the moment I don't see myself rewriting Paroller.js to plain JS.
Since this is an open-source project I would like to say that  anyone is welcome to fork repository and merge their solutions/upgrades to this plugin 😉


Add support for inline  blocks

In Markdown, enclosing text in a single backtick would cause the text to HTML to get rendered as <code>...text...</code>

Feature implemented bf1615cbc3cb73b1bfcc12328654383adeb27a3d


Adding an example showing an explicit MHT application using MFA components

This PR adds an example of a Multi-Hypothesis tracking using the Multi-frame assignment components.
This example was supposed to rely more on Stone Soup components (e.g., GaussianMixtureInitiator and MultiTargetTracker), however the current implementation of such components is not aligned with the MFA components making complex to deal with it (e.g. #896).
This example shows how to set up this type of simulations in a more general way than the example present in the documentation.

Hi @orosoman-dstl and @sdhiscocks : I have applied the suggested changes to the text and references.
Now all tracks follows the ground-truths better and there is no more track drifting.
Regarding @sdhiscocks question about the number of tracks: I tried using the GaussianMixtureInitiator with unknown number of tracks (e.g. using the MultiTargetGroundTruthSimulator) but I was not able to make it work properly with the tracking loop needed for the MFA data associator. Hence my approach of fixing the number of targets and start the tracks in this way.
Happy to discuss a more general solution (if possible with the current setup).

Regarding @sdhiscocks question about the number of tracks: I tried using the GaussianMixtureInitiator with unknown number of tracks (e.g. using the MultiTargetGroundTruthSimulator) but I was not able to make it work properly with the tracking loop needed for the MFA data associator. Hence my approach of fixing the number of targets and start the tracks in this way.

To be honest, I wasn't sure if there was a limitation in the actual algorithm itself that means it assumes fixed number of targets, or limitation of the implementation. I've found when trying to use with varying targets, you can get an RuntimeError due to "Infeasible primal problem", which I suspected could be related to varying the number of tracks.


Unable to configure firewalld

I'm unable to configure firewalld on a CentOS instance on a VC1S server. Every time I try to change the configuration, I get an ERROR: INVALID_ZONE message.
This is what I'm seeing in /var/log/firewalld.
2016-05-26 18:50:42 ERROR: ebtables not usable, disabling ethernet bridge firewall.
2016-05-26 18:50:43 ERROR: INVALID_ZONE
2016-05-26 18:58:15 ERROR: ebtables not usable, disabling ethernet bridge firewall.
2016-05-26 18:58:18 ERROR: INVALID_ZONE
2016-05-26 18:58:18 ERROR: INVALID_ZONE
2016-05-26 19:00:48 ERROR: ebtables not usable, disabling ethernet bridge firewall.

I already deleted one instance and recreated it because I thought I just buggered up something in the initial setup, but since this seems to be quite persistent, I'm guessing the particular instance wasn't the only thing that was wrong.
Trying to run ebtables --list also only logs errors.
[hamuko@hamuko ~]$ sudo ebtables --list
modprobe: FATAL: Module ebtables not found.
The kernel doesn't support the ebtables 'filter' table.

The following packages are installed on the instance:

ebtables-2.0.10-13.el7.x86_64
firewalld-0.3.9-14.el7.noarch
iptables-1.4.21-16.el7.x86_64

Hi @Hamuko, thank you for reporting,
I think we just need to add the ebtables kernel module on the kernel used by Centos, can you do:

from your current server, give me the result of the command uname -a
stop your server
from the console, change your bootscript to the one with Docker in the name (this alternate bootscript does have ebtables, I just expect it will allow you to boot with Centos)

As soon as we have this information, we can add the ebtables modules in your current bootscript

Linux hamuko.moe 4.4.6-std-2 #1 SMP Mon Mar 21 13:41:00 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
firewalld seems to configure fine now if you run yum install firewalld as root. None of the previous errors pop up and I was able to add exceptions for ssh, http and https.  Not really sure if this is a huge problem but I figured I should mention it just in case.
2016-05-27 07:22:46 ERROR: Exception DBusException: org.freedesktop.DBus.Error.AccessDenied: Connection ":1.4" is not allowed to own the service "org.fedoraproject.FirewallD1" due to security policies in the configuration file
2016-05-27 07:22:51 ERROR: Exception DBusException: org.freedesktop.DBus.Error.AccessDenied: Connection ":1.5" is not allowed to own the service "org.fedoraproject.FirewallD1" due to security policies in the configuration file
2016-05-27 07:23:32 ERROR: Exception DBusException: org.freedesktop.DBus.Error.AccessDenied: Connection ":1.9" is not allowed to own the service "org.fedoraproject.FirewallD1" due to security policies in the configuration file

Hi,
I've managed to lock myself outside when I've tried to configure firewalld on a centos image (same kernel as above) using ansible. The worst part was I didn't set up a root password, so I wasn't able to connect over console :) (fist time using scaleway) ... but fixed in the end using rescue.
I've got the following errors when ansible tried to setup firewalld rules:

TASK [firewalld : install firewalld] *******************************************
changed: [web01]
TASK [firewalld : ensure firewalld is running] *********************************
changed: [web01]
TASK [firewalld : add firewalld rules for services from vars] ******************
failed: [web01] (item={u'service': u'ssh'}) => {"failed": true, "item": {"service": "ssh"}, "module_stderr": "", "module_stdout": "/etc/profile.d/lang.sh: line 19: warning: setlocale: LC_CTYPE: cannot change locale (UTF-8): No such file or directory\r\nTraceback (most recent call last):\r\n  File \"/tmp/ansible_lqZz5B/ansible_module_firewalld.py\", line 605, in \r\n    main()\r\n  File \"/tmp/ansible_lqZz5B/ansible_module_firewalld.py\", line 395, in main\r\n    is_enabled = get_service_enabled_permanent(zone, service)\r\n  File \"/tmp/ansible_lqZz5B/ansible_module_firewalld.py\", line 251, in get_service_enabled_permanent\r\n    fw_zone = fw.config().getZoneByName(zone)\r\n  File \"\", line 2, in getZoneByName\r\n  File \"/usr/lib/python2.7/site-packages/slip/dbus/polkit.py\", line 103, in _enable_proxy\r\n    return func(*p, **k)\r\n  File \"\", line 2, in getZoneByName\r\n  File \"/usr/lib/python2.7/site-packages/firewall/client.py\", line 52, in handle_exceptions\r\n    return func(*args, **kwargs)\r\n  File \"/usr/lib/python2.7/site-packages/firewall/client.py\", line 1505, in getZoneByName\r\n    path = dbus_to_python(self.fw_config.getZoneByName(name))\r\n  File \"/usr/lib64/python2.7/site-packages/dbus/proxies.py\", line 70, in __call__\r\n    return self._proxy_method(*args, **keywords)\r\n  File \"/usr/lib/python2.7/site-packages/slip/dbus/proxies.py\", line 50, in __call__\r\n    return dbus.proxies._ProxyMethod.__call__(self, *args, **kwargs)\r\n  File \"/usr/lib64/python2.7/site-packages/dbus/proxies.py\", line 145, in __call__\r\n    **keywords)\r\n  File \"/usr/lib64/python2.7/site-packages/dbus/connection.py\", line 651, in call_blocking\r\n    message, timeout)\r\ndbus.exceptions.DBusException: org.freedesktop.DBus.Python.dbus.exceptions.DBusException: INVALID_ZONE: public\r\n", "msg": "MODULE FAILURE", "parsed": false}

PS: ON/OFF this kind of VPS is awful.

Is there a fix for this.  I tried to switch to the Docker bootscript and nada.  Tried Hamuko's "fix" but that doesn't work either.

Not sure it's related, but when running firewalld --debug=10 I get the following error in /var/log/firewalld:
2016-06-24 09:27:24 DEBUG2: firewall.core.ipXtables.ip6tables: /sbin/ip6tables -w2 -t filter -I INPUT 1 -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT
2016-06-24 09:27:24 Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/firewall/server/decorators.py", line 40, in handle_exceptions
    return func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/firewall/server/firewalld.py", line 78, in start
    return self.fw.start()
  File "/usr/lib/python2.7/site-packages/firewall/core/fw.py", line 242, in start
    self._start()
  File "/usr/lib/python2.7/site-packages/firewall/core/fw.py", line 158, in _start
    self._apply_default_rules()
  File "/usr/lib/python2.7/site-packages/firewall/core/fw.py", line 516, in _apply_default_rules
    self.__apply_default_rules(ipv)
  File "/usr/lib/python2.7/site-packages/firewall/core/fw.py", line 505, in __apply_default_rules
    self.rule(ipv, _rule)
  File "/usr/lib/python2.7/site-packages/firewall/core/fw.py", line 575, in rule
    return self._ip6tables.set_rule(rule)
  File "/usr/lib/python2.7/site-packages/firewall/core/ipXtables.py", line 160, in set_rule
    return self.__run(rule)
  File "/usr/lib/python2.7/site-packages/firewall/core/ipXtables.py", line 156, in __run
    " ".join(_args), ret))
ValueError: '/sbin/ip6tables -w2 -t filter -I INPUT 1 -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT' failed: ip6tables: Protocol wrong type for socket.

And indeed:
[root@test ~]# /sbin/ip6tables -w2 -t filter -I INPUT 1 -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT
ip6tables: Protocol wrong type for socket

Having the same issue here. FirewallD is complaining about the lack of ebtables despite it being installed with yum already.

Hi @xcellardoor, can you use the latest docker bootscript and retry ?

Already have I'm afraid, it didn't work for me.
If there's anything else you'd like me to try go ahead. I've swapped back to using Debian with UFW as a firewalled VPS solution for the time being.
I've read online that some components of FirewallD don't play well with OpenVZ? Is that the VM solution scaleway utilise?

When you switch over to the Docker bootscript you need to do a Hard Reboot from the console.  Then I had to:
sudo yum reinstall firewalld
to get mine working.  After a reload of firewalld or maybe a reboot (I cant remember)  firewalld should be using the tables.
They didn't provide much help but I hope this does.

Oddly had to remove and then reinstall FirewallD twice... but it seems to be working now!

Cheers glad it worked!  You would think that since this is a core package for CentOS 7  they would have a guide at least (a fixed default bootscript would be nice too). But I will stop moaning  ;)

@biomap thanks man, you helped a lot!
scaleway's attitude towards centos is a bit weird))

I tried reinstalling but this didn't work..
I had to reboot the host and it started working.

Unfortunately, it seems the issue with this new version firewalld-<IP_ADDRESS>-14.el7.noarch.rpm.
Anyway, rebooting the system fixes this issue. 🎉
Also make sure to stop iptables:  systemctl status iptables must return Unit iptables.service could not be found.

@abdennour thanks very much
reboot worked for me

sudo yum reinstall firewalld
sudo systemctl restart dbus
sudo systemctl restart firewalld
systemctl status -l firewalld

Active: active (running)

@zishang520 Not sure if that will leave it running after boot.  You might also need to:
systemctl enable firewalld
:smile:

@justinclift thx


this.container undefined

Where do you instantiate this.container?

In Scene.js where I am creating a ref to the div.
<div
  ref={(container) => {this.container = container}}
  style={{width: width, height: height, position: 'absolute', overflow: 'hidden'}}
>
</div>

Could you please share some repro steps.


Initial readme

Signed-off-by: Radu Andries<EMAIL_ADDRESS>
What type of PR is this?
/kind documentation
What this PR does / why we need it:
This imports the README.md from the PR in evolution.

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign admiral0
You can assign the PR to them by writing /assign @admiral0 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 00f12d604cf6ce68e3a7250fd5b1092cdcb3080b

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


The scanner view does not show up on ios and android

Describe the bug
The following code does not show up on ios and android devices.
<body>
  <!-- ... -->
  <div style="position: absolute; left: 0px; top: -2px; height: 1px; overflow: hidden; visibility: hidden; width: 1px;">
    <span
      style="position: absolute; font-size: 300px; width: auto; height: auto; margin: 0px; padding: 0px; font-family: Roboto, Arial, sans-serif;"
      >BESbswy</span
    >
  </div>
  <!-- ... -->
</body>

When I run the project with "ng serve" and open it on the browser, it is working correctly and I am able to scan the code.
But when I run the project on android or ios devices and inspect it, that scanner code does not show up in HTML.
Screenshots

Version
"@ionic/angular": "^6.1.9",
"@capacitor/android": "^4.7.3",
"@capacitor/app": "4.1.1",
"@capacitor/core": "4.7.0",
"@capacitor-community/barcode-scanner": "^3.0.3",

Ionic will behave differently on different devices and "display modes" (dark or light mode).
There are classes and variables added to different DOM elements, you need to verify that your complete tree is transparent, only looking at the body will not be enough in this case.

Ionic will behave differently on different devices and "display modes" (dark or light mode).
There are classes and variables added to different DOM elements, you need to verify that your complete tree is transparent, only looking at the body will not be enough in this case.

Thank you for your reply, @thegnuu .
But it's not related to transparent style. I can't find HTML code that this plugin adds to DOM at all.

This is expected as well. Only the web implementation will add HTML video elements :)
On iOS/Android the camera view is rendered behind the webview and will not be visible in the DOM.

Oh, I got it. How can I verify that it's working fine but just a transparent issue?

If you are not getting any errors in the console you should be fine.
The easiest way is to scan a code and check the logs if you receive something.
If this works, which it usually does if we receive an issue like this, you can check the readme and some other issues regarding the transparency "problems", there are a few different solutions for different setups. If it still not works it is most likely a permission issue.

Oh, right. I just tried to make every elements of DOM to be transparent and I can see the scanner now.
Sorry about the confusing and thank you for your help!

No worries, you're welcome :)

For Ionic this is what I did:
body.scanner-active #root {
opacity: 0;
}
Making background: transparent is not enough.


It seem's JSPatch can only support DISPATCH_QUEUE_SERIAL dispatch_queue

Scene I
I tried to create a DISPATCH_QUEUE_CONCURRENT dispatch_queue, multi tasks in the dispatch_queue share one JSContext, then a got a exception with message "TypeError: undefined is not an object (evaluating 'self.__c')"
Scene II
I tried to create a DISPATCH_QUEUE_SERIAL dispatch_queue. when the background tasks were running, I click a button to call another patched method. The exception above occurred, too.
The test code is in my fork branch
https://github.com/Robert-Qiu/JSPatch/tree/multitherad_test

Fixed, thank you!


I can also attempt to embed other whitespace characters into the name.  Here are two tabs:
$ go run cmd/turso/main.go auth api-tokens create "               "
Error: failed to create token: parse "/v1/auth/api-tokens/\t\t": net/url: invalid control character in URL
exit status 1

It would be reasonable to place the same restrictions on token names as the names of databases.

Another observation: we don't seem to be doing URL string escaping at all, which could be prone to bugs:
$ go run cmd/turso/main.go auth api-tokens create "%2b%2b"

$ go run cmd/turso/main.go auth api-tokens list
NAME
++


Process test migration from v7 legacy runtime to code generation (Issue #1131)

Migrated some BPMN processes which are being compiled using legacy v7 to code generation approach.
The tests can be identified by referring to Activitytest.java:
https://github.com/apache/incubator-kie-kogito-runtimes/blob/main/jbpm/jbpm-tests/src/test/java/org/jbpm/bpmn2/ActivityTest.java
Issue: https://github.com/apache/incubator-kie-issues/issues/1131

hi @Abhitocode few notes in here: your branch should be called as the issue in this case
"incubator-kie-issues-1131"
the subjects needs to change to
[incubator-kie-issues-1131] Process test migration from v7 legacy runtime to code generation
the rest of the comments inline


Don't show run/stop button when there is no active manifest

Not sure if we want the same with keyboard shortcuts

Not sure if we want the same with keyboard shortcuts

Not sure what you mean?

Not sure if we want the same with keyboard shortcuts

Not sure what you mean?

If we want to disable keyboard shortcuts when there are no manifest found

Not sure if we want the same with keyboard shortcuts

Not sure what you mean?

If we want to disable keyboard shortcuts when there are no manifest found

Ah yes I think we should indeed.


Distributed Training Examples/Benchmarks

Currently, FluxMPI has only 1 example. It would be good to showcase training of more image models -- ViT (https://github.com/FluxML/Metalhead.jl/pull/105), ResNets, etc. from Metalhead and also benchmark their scaling across multiple GPUs.

I am not sure if it's appropriate to raise the concern here and it is vaguely related to this issue but for benchmarks, Can I suggest something of the sort like mlpack benchmarks. I really like how they are using valgrind for memory benchmarks and profiling, sqlite to store results, etc. The comparison amongst other ML libraries provides a better depiction of Flux and why to use Flux over other libraries.

I think that might be more relevant for FluxBench. I mainly want to test the scalability across GPUs something like https://horovod.readthedocs.io/en/stable/benchmarks.html

Can I ask for a minimal example without FastAI.jl? e.g. I'd like to see how this script should be changed for distributed training:
https://github.com/FluxML/model-zoo/blob/master/vision/vgg_cifar10/vgg_cifar10.jl


Update build

Update build to the latest Prometheus build style.

Use the Prometheus Makefile.common.
Update to go modules.
Update all vendoring.
Use new promhttp package.
Update promu config for Go modules.
Convert to CircleCI

Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

Marking WIP pending https://github.com/prometheus-community/stackdriver_exporter/pull/73

LGTM though I don't see the job result from Circle CI. Did you enable this repo in Circle CI?

It's enabled, but it's not picking this up. It should trigger as soon as it hits master.

Oh, I figured it out, "build fork pull request" wasn't enabled in CircleCI.


Issue computed fields based on other computed fields stop working some cases

Odoo v11
I have sale_margin and sale_management modules so
I  inherit sale_order on new model sale_option
Created new views for this new model
If I try to get computed margin on new model sale_option it doesnt work.
¿Something about memory maybe?
This sale_option module works on a linux server without docker copier versions
If we try on copier docker version (devel or prod ambients) I have bad results on computed fields or on_change methods they simply stop working on some cases.
Case Samples:
Inherith product_template and add some on_change method depending on other fields
If you open product_template normal form (Maybe Sales->Master Data -> Product) it works, but if you try to create a product from a Many2One selector and open the form from this point, this on_change methods stop working.

Sorry, but this has nothing to do with Doodba itself. You should look for other things.

Do you think  maybe memory cached values are affected in some way by this configuration?
Its has not happened to some one else?

I see other issues like this on proffessional_templates module where calculates num2words on multiple anidated functions that doesnt work inside copier instance but it works outside dockerized ambient.
On this kind of operations failed:
words=num2words(self.amount_total,lang=self.partner_id.lang)
currency=self.currency_id.currency_name or ''
amount_words=words + ' ' + currency
self.amount_words = amount_words.upper()
I have to change this algorithm to get it work so, maybe something on this doodba image need to be tunned ¿what you think?

You where right from mi original Issue it was a problem with methods that access to different objects.
I will check inside other issues like this before to see problems on doodba thank you Manuel.


Update nf-winuser-getwindowthreadprocessid.md

Mention return value in case of error and GetLastError

@justanotheranonymoususer : Thanks for your contribution! The author(s) have been notified to review your proposed change.


event.end is null when drap fullDay event drop to agenda.

Hi all pro,
When I drag a created allDay event drop into agendaWeek / agendaDay, I notice there is no event.end date assign to it. If this is not a bug, how to get the time frame is assign to current event?
Thank you!

This feels like it would be more to do with the underlying behaviour of the fullcalendar lib, rather than this repo's plugin.

ok. noted.

I've done trick on this. Hope it help others.
event.end will add 2 hours into it when it drag from all-day to timeline.
eventDrop(event){
  if (event.allDay==false && !event.end){
    event.end = moment(event.start).add(2,'h');
  }
}


Dynamic response values for Stubs deployed on Nomad

Hi team,
We have our WireMock Stubs deployed on Nomad architecture - CI/CD pipeline. Field values in Stubs are populated. We have validations on response such as some field should have the same values as Request (e.g : message id) . Does WireMock supports the functionality of building Dynamic response  based on requests ?

Please post questions to the mailing list as per the README


Adding docs and missing @BetaApi annotations [gax-httpjson2 branch]

Codecov Report

Merging #391 into gax-httpjson2 will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             gax-httpjson2     #391   +/-   ##
================================================
  Coverage            68.93%   68.93%
  Complexity             596      596
================================================
  Files                  136      136
  Lines                 2868     2868
  Branches               215      215
================================================
  Hits                  1977     1977
  Misses                 808      808
  Partials                83       83

Impacted Files
Coverage Δ
Complexity Δ

...le/api/gax/grpc/GrpcUnaryRequestParamCallable.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...va/com/google/api/gax/rpc/FixedHeaderProvider.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...com/google/api/gax/grpc/GrpcLongRunningClient.java
95.45% <ø> (ø)
4 <0> (ø)
:arrow_down:

...com/google/api/gax/grpc/GrpcOperationSnapshot.java
100% <ø> (ø)
7 <0> (ø)
:arrow_down:

.../java/com/google/api/gax/rpc/NoHeaderProvider.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...m/google/api/gax/rpc/TranslatingUnaryCallable.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...google/api/gax/grpc/GrpcOperationTransformers.java
90.9% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../grpc/GrpcServerStreamingRequestParamCallable.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...a/com/google/api/gax/grpc/GrpcExtraHeaderData.java
50% <ø> (ø)
1 <0> (ø)
:arrow_down:

...a/com/google/api/gax/grpc/GrpcCallableFactory.java
73.17% <ø> (ø)
7 <0> (ø)
:arrow_down:

... and 1 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 7549932...a46ef2b. Read the comment docs.

Do you know what could be causing the java 7 buffer overflow error in travis? I just reran the failing build with the same result, so it doesn't seem to be an intermittent failure. The code diff itself looks fine.

No idea... it's been failing in master too.


Add VIAMaskParser for parsing polygons as mask annotations.

In order to do this, I had to remove VIABBoxParser's ability to parse bounding boxes from both rect and polygon shape attributes and assume that rect annotations are for bounding boxes and polygon annotations for masks. If not, then the polygon annotations were added as an additional bbox and this didn't allow it to be parsed due to length mismatch betwen number of boxes and masks.
I've left some of the older code commented out rather than delete it, will be happy to clean that up after some feedback.

Qualitative output check:

An option that we have is to keep the bbox parser able to handle both "polygon" and "rect" and overwrite the function bboxes to only parse rect on the mask parser. (I'm just throwing this out there, I'm not sure this would be the best strategy)

I think it would be good to have that option. We can have the following scenarios:

(As you described) Using only VIABBoxParser -- parse from either rect or polygon (not both..?)
Using VIAMaskParser with only polygon annotations -- derive bboxes and masks both from the same polygon annotation (could save a user a lot of time when annotating)
Using VIAMaskParser with both rect and polygon annotations -- this is what's been implemented in this PR as of now

@jerbly I'm unable to load the test via_bbox.json file. I'm using via-2.0.10. On trying to load the project, I get a Cannot import project from a corrupt file at the bottom of the browser.

An option that we have is to keep the bbox parser able to handle both "polygon" and "rect" and overwrite the function bboxes to only parse rect on the mask parser. (I'm just throwing this out there, I'm not sure this would be the best strategy)

As a user you're already selecting mask=True so why not just push the shape parsing down to the specific class and continue to support polys converted to bboxes. This would also allow you to convert rects to masks!

@jerbly I'm unable to load the test via_bbox.json file. I'm using via-2.0.10. On trying to load the project, I get a Cannot import project from a corrupt file at the bottom of the browser.

via_bbox.json is an annotation export file, not a project file. Try importing the annotation file.

Made some heavy-ish changes based on the feedback. The parser is now quite flexible, bbox and mask annotations can be derived from both rect and/or polygon annotations. The choice is left entirely to the user and defaults have been set to maintain previous behavior.
I've also separated the logic to create BBox or MaskArray from the annotations into a separate function so we can tweak that more easily in the future if needed. Masks from rect annotations are yet to be implemented.
Curious to hear what you guys think :)

I'm considering adding keypoint parsing for VIA too so this gets pretty messy. There comes a point where the factory function is more cumbersome than just instantiating the concrete class yourself. So I think either remove the factory function or do something like this...
Instead of all the boolean flags in the factory function we should use two enums. One for ParserType which IMO should be moved up into the parser.py for all parsers. This can be used to fix the factory function coco too. (Which hasn't been extended to include keypoints @lgvaz ). A second enum VIAShape specifically describes the shapes. The user can then provide a set of shapes to exclude from the input.
Something like this:
from enum import Enum
VIAShape = Enum('VIAShape', [('POLYGON', 'polygon'), ('RECT', 'rect'), ('POINT', 'point'),
                             ('POLYLINE', 'polyline'), ('ELLIPSE', 'ellipse'), ('CIRCLE', 'circle')])
ParserType = Enum('ParserType', 'BBOX MASK KEYPOINT')

def via(
    annotations_file: Union[str, Path],
    img_dir: Union[str, Path],
    class_map: ClassMap,
    label_field: str = "label",
    parser_type: ParserType = ParserType.BBOX,
    exclude_shapes: set = None
) -> Parser:
    if parser_type is ParserType.BBOX:
        parser_cls = VIABBoxParser
    elif parser_type is ParserType.MASK:
        parser_cls = VIAMaskParser
    else:
        raise NotImplementedError()
    return parser_cls(annotations_file, img_dir, class_map, label_field, exclude_shapes)

As a user, if I wanted a mask parser that ignored rect shape attributes in the via file I would call this:
p = via('file', 'dir', class_map, parser_type = ParserType.MASK, exclude_shapes = {VIAShape.RECT})

Then in the parser classes you can get the union set of not-supported-shapes and excluded-shapes to determine which shapes to parse. Oh, and by using the string for the enum value in VIAShape you can do things like this:
shape = VIAShape(shape_attr["name"])
if shape not in exclude_shapes:
...

I'm up for removing the factory function and just instantiating the class, let's go with that?

I'm up for removing the factory function and just instantiating the class, let's go with that?

Yes. I think that's much cleaner. Also it means we don't need BBoxFrom and MaskFrom since the contract is now just in the concrete class. I think it means a lot of code can be deleted! (MY favourite type of coding).

a lot of code can be deleted! (MY favourite type of coding).

Specially when it's untested code, then I just see the coverage percentages going up, really nice! hahaaah
Let's do that then. @rsomani95 what do you think?

Concur, I think the Enum approach + instantiating from class is much cleaner

Hi hi, I hope you both had a great new year!
I'm returning back to this, @rsomani95 should I go and make the changes or you would like to do it?

I think the Enum approach + instantiating from class is much cleaner

Just to be clear, I think if we're instantiating directly from the class we don't need Enums anymore

Hi hi, I hope you both had a great new year!
I'm returning back to this, @rsomani95 should I go and make the changes or you would like to do it?

I think the Enum approach + instantiating from class is much cleaner

Just to be clear, I think if we're instantiating directly from the class we don't need Enums anymore

Hi @lgvaz, thanks. hope you had a good one too!
I’m a bit caught up with some tasks at work so feel free to make the changes if you’d like :)

Hi @lgvaz, thanks. hope you had a good one too!
I’m a bit caught up with some tasks at work so feel free to make the changes if you’d like :)

@lgvaz - I think there's potentially still value in the VIAShape enum to tidy up the parsing and to allow the user to optionally specify shapes to exclude. But, perhaps that's an extension from initial mask support for another PR some day?

@lgvaz - I think there's potentially still value in the VIAShape enum to tidy up the parsing and to allow the user to optionally specify shapes to exclude. But, perhaps that's an extension from initial mask support for another PR some day?

@jerbly I'm a bit confused about exclude_shapes, in the example you provided we have:
...parser_type = ParserType.MASK, exclude_shapes = {VIAShape.RECT})

If I understood correctly this would not parse bboxes, but to train the segmentation models we need them. Maybe what you have in mind is that if rect gets remove we calculate the bboxes from the masks?

@jerbly I'm a bit confused about exclude_shapes, in the example you provided we have:
...parser_type = ParserType.MASK, exclude_shapes = {VIAShape.RECT})

If I understood correctly this would not parse bboxes, but to train the segmentation models we need them. Maybe what you have in mind is that if rect gets remove we calculate the bboxes from the masks?

@lgvaz - no, that's not what I intended. The VIA spec allows you to mark up multiple shapes like rectangles, circles, ellipses etc. The exclude_shapes set was intended to allow you to take a dataset but only parse a subset of the shapes. This is independent of the parsing type be that bboxes, mask or keypoints. Think of shapes as the input and bbox/mask/keypoint as the output.
I can have a go at a PR for this if that makes more sense? I'm just a little short on time this week.

@lgvaz - no, that's not what I intended. The VIA spec allows you to mark up multiple shapes like rectangles, circles, ellipses etc. The exclude_shapes set was intended to allow you to take a dataset but only parse a subset of the shapes. This is independent of the parsing type be that bboxes, mask or keypoints. Think of shapes as the input and bbox/mask/keypoint as the output.
I can have a go at a PR for this if that makes more sense? I'm just a little short on time this week.

@jerbly - Aaah okay, gotcha!!
Let's do that as a subsequent PR as you suggested, I'll work and merge this one this week and then you can open a new PR next week, what do you think?

@jerbly - Aaah okay, gotcha!!
Let's do that as a subsequent PR as you suggested, I'll work and merge this one this week and then you can open a new PR next week, what do you think?

@lgvaz - sure, I can if you think there's value in it?

@lgvaz - sure, I can if you think there's value in it?

@jerbly - I don't have a definitive answer right now, let me explore VIA a little bit more this weekend while I work on this PR, then I can provide you with a better answer without BSing :sweat_smile:

@jerbly - I don't have a definitive answer right now, let me explore VIA a little bit more this weekend while I work on this PR, then I can provide you with a better answer without BSing :sweat_smile:

I'm falling behind on this, but just want to let you know that I didn't forget, I'll get back to this after mmdetection support has been tackled

I'm falling behind on this, but just want to let you know that I didn't forget, I'll get back to this after mmdetection support has been tackled

@lgvaz @jerbly closing this as it's quite outdated with the new record API :)


Reasons doesn't work in entry select field "Add new entry" modals

I've got an entry type with a lightbox to toggle between to fields - a Matrix field and a Super Table field. The toggle applies to the entire field and works perfectly in the normal "Add new entry" from the "All Entries" dashboard view.
What doesn't work is the toggle when you opt to add a new entry from the "Select an Entry" entry field. Please see diagram below for explanation

.

Thanks for reporting that! Just tagged Reasons 2.3.0 which should resolve this issue.


There is no information in the documentation about some widgets.

In the example on the path fluent_ui/example/lib/screens/inputs.dart on the 110th line there is a mica widget, but there is no information about it in your documentation.

A lot of widgets aren't documentated or a poorly documentated :/
Duplicate of https://github.com/bdlukaa/fluent_ui/issues/40


--locked does not always report an error even when the lockfile needs to be regenerated

environment

uv: tested on 0.5.2 and 0.5.4
python tested on 3.11, 3.12, 3.13
windows

to reproduce
sorry this isn't a minimal repro, i can't get it to happen reliably so this is the best i can do.
note that the project uses pyprojectx so it installs uv automatically

git clone https://github.com/detachhead/basedpyright (commit 6d94c2aca2683aaa1f36e5d1687c6240e239b315)
cd basedpyright
run ./pw uv lock --locked (no error)
delete uv.lock
run ./pw uv lock
run git diff uv.lock

diff --git a/uv.lock b/uv.lock
index aca6ec6ee..f165035b0 100644
--- a/uv.lock
+++ b/uv.lock
@@ -1,8 +1,7 @@
 version = 1
 requires-python = ">=3.8"
 resolution-markers = [
-    "python_full_version < '3.9'",
-    "python_full_version >= '3.9' and python_full_version < '3.11'",
+    "python_full_version < '3.11'",
     "python_full_version == '3.11.*'",
     "python_full_version >= '3.12'",
 ]
@@ -250,8 +249,7 @@ name = "docify"
 version = "1.0.0"
 source = { registry = "https://pypi.org/simple" }
 dependencies = [
-    { name = "libcst", version = "1.1.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.9'" },
-    { name = "libcst", version = "1.5.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
+    { name = "libcst" },
     { name = "tqdm" },
 ]
 sdist = { url = "https://files.pythonhosted.org/packages/b2/cc/d403e8dbe89826340a18909e0aa26ed917981d41579ba91145e2e1b954c8/docify-1.0.0.tar.gz", hash = "sha256:8d779f8348dd8d46796709c0db7d710cbc6354935d90a727c9ff3ab77b4cd3bf", size = 7186 }
@@ -356,13 +354,10 @@ wheels = [
 name = "libcst"
 version = "1.1.0"
 source = { registry = "https://pypi.org/simple" }
-resolution-markers = [
-    "python_full_version < '3.9'",
-]
 dependencies = [
-    { name = "pyyaml", marker = "python_full_version < '3.9'" },
-    { name = "typing-extensions", marker = "python_full_version < '3.9'" },
-    { name = "typing-inspect", marker = "python_full_version < '3.9'" },
+    { name = "pyyaml" },
+    { name = "typing-extensions" },
+    { name = "typing-inspect" },
 ]

(the diff is larger than this but i only included the first few changes)

@DetachHead Actually, uv sync will automatically check if the lock file is up-to-date, unless you've used the --frozen flag.
So theoretically, as long as you've used uv sync before committing, your lock file should be up-to-date.
uv sync
--frozen
Sync without updating the uv.lock file.

Instead of checking if the lockfile is up-to-date, uses the versions in the lockfile as the source of truth. If the lockfile is missing, uv will exit with an error. If the pyproject.toml includes changes to dependencies that have not been included in the lockfile yet, they will not be present in the environment.

May also be set with the UV_FROZEN environment variable.

i think it only happens when deleting the lockfile before running uv sync or uv lock again. if the lockfile isn't deleted first, it doesn't seem to regenerate it which would explain why --locked doesn't think there are any changes

This is generally intentional... --locked confirms that the uv.lock is up-to-date with the given requirements, not that a fresh uv lock would produce the same lockfile. These are subtly different. For example, we might change how we represent some metadata in uv.lock, or we might change prioritization within the resolver. Both of those changes could be backwards-compatible, so existing uv.lock files work fine and still represent a valid resolution for the given inputs. But if you remove the uv.lock, we might then make different choices in resolving.
If you actually want to enforce this guarantee, I'd probably do it manually by moving mv uv.lock before.lock then uv lock and diffing the two files.


MWPW-160929: Match desktop CTA shape on mobile

Resolves: MWPW-160929
Test URL:

https://main--dc--adobecom.hlx.page/acrobat/online/test/sign-pdf?martech=off
https://mwpw-160929--dc--adobecom.hlx.page/acrobat/online/test/sign-pdf?martech=off

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

Project coverage is 84.75%. Comparing base (e4e440a) to head (3a39ae3).

Additional details and impacted files
@@           Coverage Diff           @@
##            stage     #870   +/-   ##
=======================================
  Coverage   84.75%   84.75%
=======================================
  Files          48       48
  Lines        6114     6114
=======================================
  Hits         5182     5182
  Misses        932      932

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

@joaquinrivero why?

why?


Changed optical image size from 25MB to 50MB

@intsco can be merged to the next release branch(that is , master)


runtime: GDB tests failing on linux-mips64-rtrk since CL 380755

--- FAIL: TestGdbBacktrace (1.30s)
    runtime-gdb_test.go:76: gdb version 8.1
    runtime-gdb_test.go:428: gdb output:
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbBacktrace2309338065/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbBacktrace2309338065/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        "/tmp/gobuilder-mips64/tmp/TestGdbBacktrace2309338065/001/a.exe": not in executable format: File format not recognized
        No symbol table is loaded.  Use the "file" command.
        Make breakpoint pending on future shared library load? (y or [n]) [answered N; input not from terminal]
        No executable file specified.
        Use the "file" or "exec-file" command.
        No stack.
        The program is not being run.
    runtime-gdb_test.go:454: could not find '#0.*main\.eee' in backtrace
--- FAIL: TestGdbPanic (2.48s)
    runtime-gdb_test.go:76: gdb version 8.1
    runtime-gdb_test.go:636: gdb output:
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbPanic797438989/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbPanic797438989/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        "/tmp/gobuilder-mips64/tmp/TestGdbPanic797438989/001/a.exe": not in executable format: File format not recognized
        No executable file specified.
        Use the "file" or "exec-file" command.
        No stack.
    runtime-gdb_test.go:650: could not find '(#.* .* in )?main\.crash' in backtrace
--- FAIL: TestGdbConst (20.83s)
    runtime-gdb_test.go:76: gdb version 8.1
    runtime-gdb_test.go:578: gdb output:
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbConst1900233325/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbConst1900233325/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        "/tmp/gobuilder-mips64/tmp/TestGdbConst1900233325/001/a.exe": not in executable format: File format not recognized
        No symbol table is loaded.  Use the "file" command.
        Make breakpoint pending on future shared library load? (y or [n]) [answered N; input not from terminal]
        No executable file specified.
        Use the "file" or "exec-file" command.
        No symbol table is loaded.  Use the "file" command.
        No symbol table is loaded.  Use the "file" command.
        No symbol table is loaded.  Use the "file" command.
        No symbol table is loaded.  Use the "file" command.
        No symbol table is loaded.  Use the "file" command.
    runtime-gdb_test.go:586: output mismatch
--- FAIL: TestGdbAutotmpTypes (18.97s)
    runtime-gdb_test.go:76: gdb version 8.1
    runtime-gdb_test.go:510: gdb output:
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbAutotmpTypes830246952/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        BFD: /tmp/gobuilder-mips64/tmp/TestGdbAutotmpTypes830246952/001/a.exe: unable to initialize decompress status for section .debug_abbrev
        "/tmp/gobuilder-mips64/tmp/TestGdbAutotmpTypes830246952/001/a.exe": not in executable format: File format not recognized
        No symbol table is loaded.  Use the "file" command.
        Make breakpoint pending on future shared library load? (y or [n]) [answered N; input not from terminal]
        No executable file specified.
        Use the "file" or "exec-file" command.
        The program is not being run.
        All types matching regular expression "astruct":
    runtime-gdb_test.go:527: could not find []main.astruct; in 'info typrs astruct' output
FAIL
FAIL	runtime	104.355s

greplogs --dashboard -md -l -e 'unable to initialize decompress status for section' --since=2022-03-01
2022-03-17T08:43:51-c379c3d/linux-mips64-rtrk
2022-03-17T08:43:15-599d539/linux-mips64-rtrk
2022-03-17T03:41:50-9956a54/linux-mips64-rtrk
2022-03-17T03:04:30-8427429/linux-mips64-rtrk
2022-03-17T02:43:05-f839aaa/linux-mips64-rtrk
2022-03-17T01:29:17-6686536/linux-mips64-rtrk
2022-03-17T00:12:40-1cfe100/linux-mips64-rtrk
2022-03-16T23:03:54-e5e638e/linux-mips64-rtrk
2022-03-16T20:39:14-ed4db86/linux-mips64-rtrk
2022-03-16T20:10:07-91631bc/linux-mips64-rtrk
2022-03-16T18:34:31-81d3c25/linux-mips64-rtrk
2022-03-16T07:03:20-5fd0ed7/linux-mips64-rtrk
2022-03-16T05:32:52-d34287a/linux-mips64-rtrk

These failures started as of CL 380755 (attn @mengzhuo, @cherrymui, @thanm; CC @golang/runtime).
The tests are only failing on the one builder (attn @bogojevic, @milanknezevic), it is not a first-class port, and the failure may be due to a bug or deficiency in the installed gdb — so it seems to me that it would be fine to resolve this issue by skipping the affected tests on that builder. (As noted in https://github.com/golang/go/issues/39204#issuecomment-634175471, we already skip those tests on a shockingly large fraction of builders.)

I'll take a look.


[BUG] Not working

After patching everything up and replace things in the directory.
I found that the game UI said I am "Old Host", and when I entered game, I have to create a new character.

You find anything out? The script patched all my files, i replaced files, but none of us has Characters, only Map.

Patched my coop host save file, everyone else can still load in to the Dedicated server fine but I end up at an infinite load screen myself.

Are you sure you are letting the script run to completion?

Are you sure you are letting the script run to completion?

Yes

You find anything out? The script patched all my files, i replaced files, but none of us has Characters, only Map.

Map is only associated with "LocalSave.sav" I guess, which is something the script didn't change.

OK so after reading the README carefully, I found that this only works when you try to move the sav to the  dedicated server.
But what I was trying to do is to copy the sav from my co-op host to my local computer.
I found I can only play the sav using the character of co-op host. And I want to make my character the "00001.sav", and play it.
Simply rename the sav file just doesn't work.

NOTE / Reply to the last issue, others: This WILL NOT work on a steamcmd dedicated server. You can only transfer the save file successfully from a co op server to the dedicated server installed via the steamUI. For some reason, the dedicated server installed via steamcmd is slightly different.
HOWEVER - you can run this dedicated server offline without steam open. So I installed steam on my server, then installed the dedicated server, then closed steam and ran the server directly.

NOTE / Reply to the last issue, others: This WILL NOT work on a steamcmd dedicated server. You can only transfer the save file successfully from a co op server to the dedicated server installed via the steamUI. For some reason, the dedicated server installed via steamcmd is slightly different.
HOWEVER - you can run this dedicated server offline without steam open. So I installed steam on my server, then installed the dedicated server, then closed steam and ran the server directly.

Can you clarify this? I thought the way to run the server was to install the server tool via regular steam, then do something with appupdate in steamcmd, then launch the server via the steamui

Can you clarify your install process please?

I have successfully move the file from co-op host to dedicated server using https://github.com/thijsvanloef/palworld-server-docker