[Angular - NgRx - SCSS] Create repository card

Create repository card
Background
This will be a reusable component that displays details about a repository. On the home page and profile page, a user will see a list of their repositories. This component is intended to be the building block for that - a single card for a repository, showing pertinent data about that repo.
Acceptance

[ ] Card view for use on repository lists
[ ] should always have:

[ ] repo name
[ ] visibility tag (public / private)
[ ] description (if available)
[ ] main language with color dot
[ ] star icon with number of stars on repo
[ ] last updated text
[ ] optional - star button on right to add a star to the repo (does not need to function yet)

Notes

It appears that the profile page has its own implementation of the card that fits most of these criteria. Adapting this component to be re-usable would satisfy this requirement probably.

CC: @dustinsgoodman

CC: @dustinsgoodman

Let's keep as a "make reusable" ticket


SN74AHCT125 replacement?

Looks like SN74AHCT125 isn't available anymore. Can you suggest any replacements?

In stock at digikey and mouser. digikey says it has over 8000 available to ship immediately.

Alternatively, a BSS138-based logic level converter (like this one: SparkFun Logic Level Converter - Bi-Directional) should work too


wwhrd misidentifies ISC license in go-spew as 0BSD

Observed in https://github.com/docker/compose-on-kubernetes, for some reason the vendored LICENSE from go-spew 1.1.1:

Permission to use, copy, modify, and/or distribute this software for any
purpose with or without fee is hereby granted, provided that the above
copyright notice and this permission notice appear in all copies.

is being recognised as 0BSD:

Permission to use, copy, modify, and/or distribute this software for any purpose with or without fee is hereby granted.

instead of ISC:

Permission to use, copy, modify, and/or distribute this software for any purpose with or without fee is hereby granted, provided that the above copyright notice and this permission notice appear in all copies.

Unlike #18 it's definitely finding a license, but reporting the wrong one.
pkg.go.dev detects it correctly as ISC.
wwhrd Version:
version dev
commit 1961213
date 1961-02-13T20:06:35Z

I don't know where version numbering comes from, but the checkout in my go/src/github.com/frapposelli/wwhrd directory is 749cb0f413602c9a4992cbeeb3d125ac325d9a30

Hi @TBBle I've looked at the problem and can definitely reproduce.
The issue at hand is that the license-detector library gives the go-spew license a 93% probability of being 0BSD and 84% of being ISC.
wwhrd takes the best result and uses it. Not sure why it's being misidentified but I've had similar troubles in the past with this package in particular, I'll dig deeper and file an issue on go-spew if needed.

The only other thing I noticed is that the Go source files are still missing the "/or" for which #18 was filed. But I am assuming the library doesn't look at them, as that would equally mismatch 0BSD and ISC.

I went spelunking in my history, looks like the previous license detector library was content with the /or fix which was added here davecgh/go-spew@e250ec7f597aa5cd2c9f38b9cbf3dbc81cc1e3bb
The new library is using a different, more holistic approach, so I wouldn't be surprised if it evaluates other files and not just LICENSE. I'll try a fix in the coming days and send out a PR to davecgh/go-spew if it solves the problem 😄

Hi @TBBle, I've just cut version v0.3.1 but your issue hasn't been fixed by the library update.
Looks like the library has changed hands now, I've opened an issue in the new repo (go-enry/go-license-detector#4). I'll keep this one open until we get to the bottom of it.


The value for one of the HTTP headers is not in the correct format

Hi,
I am using the cpp lite version of Azure Storage Client Library.
I am using v0.3 version.
I am connecting to a Dell Azure Stack Hub of version
IdentitySystem     : AzureAD
OemVersion         : 2.1.1908.3
StampVersion       : 1.2002.19.73
Can the v0.3 version connect to the above Azure Stack Hub versions? Am i using the right version of the client library?
std::shared_ptr<storage_credential> credential = std::make_shared<shared_key_credential>(account_name, account_key);
std::shared_ptr<storage_account> account = std::make_shared<storage_account>(account_name, credential, /* use_https */ true, blob_endpoint);
auto bC = std::make_shared<blob_client>(account, 16);
std::string containerName = "fs-magicians-container1";
std::string blobName = "nbu-nagini-unmanaged-disk-v2.ee86dfba-2320-451f-8efd-af7ff9b341c4.screenshot.bmp";
bool exists = true;
blob_client_wrapper bc(bC);
exists = bc.container_exists(containerName);
if (exists)
{
	std::cout << "Success: The container exists";
}
else
{
	std::cout << "Failed to find container" << errno << std::endl;
}

@majumd Hi, we haven't officially supported Azure Storage Hub yet, but cpplite sdk should be able to work with it.
Can you provide some information so that we can further investigate?

What does the endpoint look like for Azure Stack Hub?
What kind of authentication are you using?
Which HTTP header isn't in the correct format? What is it supposed to be?

In addition, please avoid using that blob_client_wrapper and always stick to blob_client like in this sample.

Hi,
I have modified code as per sample given by you to use blob_client but still it gives error.
"The value of one of the HTTP headers is not in correct format"

TCP_NODELAY set
Connected to fsmagicstorageacc1.blob.dazs1.wwtrrratc.com (10.253.xx.xx) port 443 (#0)

HEAD /fs-magic-container1?restype=container HTTP/1.1
Host: fsmagicstorageacc1.blob.dazs1.wwtrrratc.com
Accept: /
x-ms-date: Fri, 19 Jun 2020 08:13:25 GMT
x-ms-version: 2018-11-09
Authorization: SharedKey fsmagicstorageacc1:xxxxxTt4=

Mark bundle as not supporting multiuse
< HTTP/1.1 400 The value for one of the HTTP headers is not in the correct format.
< Transfer-Encoding: chunked
< Server: Microsoft-HTTPAPI/2.0
< x-ms-request-id: e372d166-3981-0721-6837-d070dcd34264
< Date: Fri, 19 Jun 2020 08:28:28 GMT
<
Connection #0 to host fsmagicstorageacc1.blob.dazs1.wwtrrratc.com left intact

I have used the following
std::string account_name = "fsmagicstorageacc1";
std::string account_key = "xxxxQ==";    // Storage account key if using shared key auth
std::string blob_endpoint = "fsmagicstorageacc1.blob.dazs1.wwtrrratc.com/";
The names are correct names. I have given just for representation. The key used above is the user access key obtained by logging into portal.
Dashboard > Storage accounts >  - Access keys
I used key1 in the user access keys.
Use access keys to authenticate your applications when making requests to this Azure storage account. ...

@majumd can you also share the error response body?

This is the only HTTP header response which is being returned.
HTTP/1.1 400 The value for one of the HTTP headers is not in the correct format.
It would have been nice if it could return which header is not in correct format.

Hi @majumd, the request header looks completely normal and I believe it's going to work if running against Azure Storage server.
This C++ sdk is targeted at Azure Storage server. Any other kind of server including Azure Stack is supposed to expose exactly the same interface as Azure Storage if they want to be able to work with client sdk. So I think your request failed because Azure Stack server behaves differently.

Hi,
For the error specified by @majumd, I am getting the error response body as :
Code : InvalidHeaderValue
Message :
The value for one of the HTTP headers is not in the correct format.
RequestId:6abd80a3-3c81-0a33-6566-4c83d6bd9132
Time:2020-06-19T12:57:25.5124783Z
HeaderName : x-ms-version
HeaderValue : 2018-11-09
From the error body it seems that, storage service version "2018-11-09" is used while getting the container properties.
Does this version is supported by cpp lite version of Azure Storage client library(v0.3 version)?

Hi @snehalw777 , seems your Azure Stack doesn't support this API version. Can you edit here and try 2019-02-02, 2019-07-07 and 2019-10-10?

Thanks for the reference @JinmingHu-MSFT
It worked with older API version.

@snehalw777 Okay, I think you can use the old API version. But 2018-11-09 is an API version with breaking changes, which means that there might be one or more API in this sdk that cannot function properly if you just change the API version. So Please do use test cases to cover those API you're going to use to make sure everything works fine.

I am closing this issue because this lite version v0.3 started working using API version 2017-11-09 on an Azure Stack Hub setup of version 2002(latest version).


status not working

Im having this issue too any fixes?

Same

Hey guys,
i managed to fix it by changing the prefix to something else. I am not sure if its related to this issue or not but i got this error
    self.retry_count.pop(server["address"])
KeyError: '<your ip>'
I removed line
 self.retry_count.pop(server["address"])  and it worked for me

Given the deprecation of discord.py, I have no immediate plans to update this project. I will accept PRs if appropriate

This should be fixed in #74 / #76


_prepare_deepspeed(): TypeError: unsupported operand type(s) for *: 'NoneType' and 'int'

System Info
- `Accelerate` version: 0.18.0.dev0
- Platform: Linux-6.2.7-zen1-1-zen-x86_64-with-glibc2.37
- Python version: 3.10.10
- Numpy version: 1.23.5
- PyTorch version (GPU?): 2.1.0a0+gitbdaf402 (True)
- `Accelerate` default config:
        - compute_environment: LOCAL_MACHINE
        - distributed_type: DEEPSPEED
        - mixed_precision: fp16
        - use_cpu: False
        - num_processes: 1
        - machine_rank: 0
        - num_machines: 1
        - rdzv_backend: static
        - same_network: True
        - main_training_function: main
        - deepspeed_config: {'gradient_accumulation_steps': 6, 'gradient_clipping': 1.0, 'offload_optimizer_device': 'none', 'offload_param_device': 'none', 'zero3_init_flag': False, 'zero_stage': 2}
        - fsdp_config: {}
        - megatron_lm_config: {}
        - downcast_bf16: no
        - tpu_use_cluster: False
        - tpu_use_sudo: False
        - tpu_env: []
        - dynamo_config: {'dynamo_backend': 'NVFUSER'}

Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
steps

have deepspeed enabled. i don't think the specific deepspeed config will be especially relevant here.

use accelerate.prepare(*args) where a DataLoader is present in args. the DataLoader must be the only obj in args that has the attribute batch_size. the value of batch_size in the DataLoader must be None.

a wild None appeared
a value of None for batch_size appears to happen when batch_sampler is set for initializing the DataLoader. i believe this is intended behaviour.
in the relevant torch code from DataLoader's __init__ found in torch/utils/data/dataloader.py, we see:
if batch_sampler is not None:
    # auto_collation with custom batch_sampler
    if batch_size != 1 or shuffle or sampler is not None or drop_last:
        raise ValueError('batch_sampler option is mutually exclusive '
                         'with batch_size, shuffle, sampler, and '
                         'drop_last')
    batch_size = None
    drop_last = False

then below:
self.batch_size = batch_size
self.drop_last = drop_last
self.sampler = sampler
self.batch_sampler = batch_sampler
self.generator = generator

so when the DataLoader is passed to accelerate.prepare(), it has a batch_size attribute, but it's set to None.
a speedy deceleration
now, in _prepare_deepspeed() found in accelerate.py, we have:
is_dataloader_present = any(isinstance(obj, torch.utils.data.DataLoader) for obj in args)
if deepspeed_plugin.deepspeed_config["train_micro_batch_size_per_gpu"] == "auto" or is_dataloader_present:
    result = [
        self._prepare_one(obj, first_pass=True) if isinstance(obj, torch.utils.data.DataLoader) else obj
        for obj in args
    ]

    batch_sizes = [obj.batch_size for obj in args if hasattr(obj, "batch_size")]
    if self.split_batches:
        batch_sizes = [batch_size // self.num_processes for batch_size in batch_sizes]
    if len(batch_sizes) == 0:
        raise ValueError(
            "When using DeepSpeed `accelerate.prepare()` requires you to pass at least one of training or evaluation dataloaders "
            "or alternatively set an integer value in `train_micro_batch_size_per_gpu` in the deepspeed config file"
            "or assign integer value to `AcceleratorState().deepspeed_plugin.deepspeed_config['train_micro_batch_size_per_gpu']`."
        )

batch_size_per_device = min(batch_sizes) if deepspeed_plugin.is_train_batch_min else max(batch_sizes)

since we have a DataLoader, we enter the if statement. when batch_sizes is set using that list comprehension that uses hasattr(), with our setup, it should get set to [None].
interestingly, python let's you use min() and max() on a list with None iff None is the only element in that list. in that case, it returns None. otherwise, you get a similar error to this one with >.
consequently, batch_size_per_device gets set to None.
so when we reach
config_kwargs = {
    "train_micro_batch_size_per_gpu": batch_size_per_device,
    "train_batch_size": batch_size_per_device
    * deepspeed_plugin.deepspeed_config["gradient_accumulation_steps"]
    * self.num_processes,
    "gradient_clipping": 1.0,
    "zero_optimization.stage3_gather_16bit_weights_on_model_save": False,
}

we end up multiplying with a Nonetype when setting "train_batch_size", leading to the error in the title.
Expected behavior
`batch_size_per_device` in `_prepare_deepspeed` gets set properly based on the min or max of `batch_sizes`.

cc @pacman100

@j4ded, above PR will raise an error when any dataloader has None for the batch_size, so no more speedy decelration until the user fixes it as per the error suggestions.


docs: add getting started tutorial (using AKS Fleet Manager)

Description of your changes
This PR adds a getting started tutorial which uses Azure Fleet Manager.
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested
N/A
Special notes for your reviewer
N/A

Closing as non-upstream docs are not kept in the repo.


Run ErizoController via HTTPS for chrome update to version 36

As Chrome blocks non-SSL websocket connection over HTTPS
I changed some code  in erizoController
var https = require('https');
var fs = require("fs");
var options = {
key: fs.readFileSync('cert/key.pem').toString(),
cert: fs.readFileSync('cert/cert.pem').toString()
};
var server = https.createServer(options);
and set config in licode_configure
config.erizoController.ssl = true
however I get ice failed  later,
13 2014-08-01 21:40:30,088 - WARN: NiceConnection - Ice Failed 0x3c75ba0
https://groups.google.com/forum/#!topic/lynckia/S1odJXkRx1k
there is similar issue before, and it is not solved  ,  so it should not be closed.

@jcague Some tutorial for it?

I'd suggest to use NGINX to proxy WS connections via secure connections. This configuration is currently working in production environments.


add comments for fill_program_hash_trace function

Add comments to the function to make the code easier to understand for the reader

Thanks for the PR. :blush: I would argue that Sponge absorption is more than just application of the permutation to the current state; replacing part of the Sponge's state with the chunk of size rate is already part of absorption. That said, comments or better variable names can indeed contribute to readability here. :+1: How about:
fn fill_program_hash_trace(&mut self) {
    let padded_program = Self::hash_input_pad_program(&self.program);
    let mut sponge = Tip5::init(); // I think the names here make it clear that this is initialization
    for chunk_to_absorb in padded_program.chunks(Tip5::RATE) { // renaming from `chunk`
        sponge.state[..Tip5::RATE]
            .iter_mut()
            .zip_eq(chunk_to_absorb)
            .for_each(|(sponge_state_elem, &absorb_elem)| *sponge_state_elem = absorb_elem);
        let hash_trace = sponge.trace();

        self.increase_lookup_multiplicities(hash_trace);
        let trace_addendum = HashTable::trace_to_table_rows(hash_trace);
        self.program_hash_trace
            .append(Axis(0), trace_addendum.view())
            .expect("shapes must be identical");
    }
	// …
}

Yes, you are right. The for chunk... loop is also part of the sponge absorption process. Initially, I didn't have a suitable place to commit, so i just committed to the final line. Your decision to change the variable name is better than my initial approach and makes it easier for readers to understand😄. I agree with your changes and will close this pull request. I will follow your changes and then submit a new pull request.

I think keep program_sponge is better than using sponge because it makes it clear to the reader that this sponge process is specifically for program data.


fix(@angular/ssr): ensure compatibility for Http2ServerResponse type

Updated the Http2ServerResponse interface to eliminate dependency on generics, ensuring compatibility across multiple versions of @types/node.
Closes #28965

The changes were merged into the following branches: main, 19.0.x


Support visual studio graphics debugger in D3D12 backend.

This is just a nice-to-have suggestion.
Currently it seems d3d11 backend works fine in visual studio graphics debugger, while d3d12 backend will cause the visual studio graphics debugger stop working and sometimes crash (by infinite recursion?). It would be nice to get it also working.

If VS graphics debugger crash, it does not seem like a Diligent bug.

Maybe it's not a bug at all... i'm not certain. This web page says that IDXGraphicsAnalysis should be used when rendering to a texture, and indeed i am since i'm using MSAA, so it'll be nice anyway if Diligent Engine can gain support for IDXGraphicsAnalysis somehow...
I'm quite a beginner though so take my words with a grain of salt =)

If your goal is to capture, debug and analyze the frame it is much easier to use special tools such as renderdoc. Programmatically capturing the frame would be a big hassle.

Closing this issue. Feel free to ask questions on gitter/discord or open a new issue.


DCE prunes pinvoke methods and related types in some scenarios

For some reason DCE doesn't seem to be visiting pinvoke methods. As a result this means that custom pinvoke marshaler types get killed by the analyzer. I spent an hour or so trying to fix this and couldn't figure out how to fix it - the DCE code is nigh-incomprehensible, I can't figure out where and why it's pruning types. It seems like it skips methods and types in a dozen different places based on different criteria.

Can you provide test example?
DCE walks through all types/methods from starting point, skipping types/methods marked with Ignored/External meta attributes.
After this pass, it will says to JSIL skip translations of all types/methods that are not in list from previous step.
Central point here is DeadCodeInfoProvider.WalkMethod(...) andDeadCodeInfoProvider.AddMethod(...). First of it is responsible for collecting all references from method body IL stream, second - for adding all dependencies from method declaration (instance type, argument types, return type, generic arguments).

I tried using AddType to add the type referred to by the PInvoke method, but that didn't seem to affect whether it was walked or not. I tried various things and nothing worked, with the exception of adding a blanket whitelist entry (Namespace\.TypeName::.*).

I've already thought about implementing some log for DCE - in that case it will be easier to understand why some type/method was included (what other type reference it) or why it was excluded (really it should be only if method/type was marked as IsIgnored or IsExternal or IsStubOnly).
And DCE always ignores every method that is not reachable from Starting Point/white listed points.
Currently DCE could not look through custom method implementation in Proxies/FakeMethods - so probably you have such case. DCE is working only on origanal assembly and read  IsIgnored, IsExternal, IsStubOnly` properties.

Specifically the problem was with MarshalAs and custom marshalers:
https://msdn.microsoft.com/en-us/library/s2t3h416(v=vs.90).aspx
The attribute referred to a marshaler type, but no matter what I did I couldn't exempt that marshaler from DCE. Adding a whitelist regex would fix it but the regex made compilation like 50x slower.

Adding whitelist regexp should affect only on DCE analyze step. Is it do so big performance hit?
How long is your full translations takes?
On my project DCE analyze takes about 7 s seconds and saves me several minutes of compilation (I use lot's of regex white list records).
Definitely DCE has no ideas about walking to type that defined as sting on some attribute :), but we can think how we can taught it.
Can you try to mark custom marshal type with JSDeadCodeEleminationClassEntryPoint?

I believe I tried the entrypoint attribute and it didn't work.
It wasn't using the string attribute, it was using the typeref one - where the value is typeof(T)

The type:
https://github.com/flibitijibibo/SDL2-CS/blob/master/src/LPUtf8StrMarshaler.cs
The usage:
https://github.com/flibitijibibo/SDL2-CS/blob/master/src/SDL2.cs#L79
JSIL's PInvoke layer handles all this stuff currently, DCE just needs a way to interact with it. I spent a couple hours trying to fix it at the time and gave up.

Even in this case DCE cannot reach any member of marshaler type. I suppose that in this case it saves type, but ignores all its members.
On other side I'm even unsure if DCE goes through type references declared on attributes. It definitely should, but probably it was not implemented. In that case it will eliminate even type reference.

It would be fine if it were special-cased for the pinvoke attribute, as long as there were a way for JSILc to tell DCE to include the whole type or vice versa.
Also, DCE should never eliminate interface members as that breaks the interface. In this case the marshaler declares that it implements the interface, so pruning its methods is incorrect.

If type can be reached, there will be preserved all methods that implement interface members that could be reached (and everything that are reachable from that members).
If it not work - there are several options:

Interface or interface members are non-reachable.
Type reference is non-reachable (I suppose we haven't gone to type reference inside attribute argument).

Vaguely related: I'd be really pleased if there were some basic API-only tests for DCE that cover basic reachability/pruning scenarios without doing a full compile, like the APITests that exist already. That would make it easy to add tests covering issues and get an understanding of how DCE behaves.

Test coverage is still on radar.  :)

@kg, I'm thinking about this issue. It' looks for me, that there is no class that calls ICustomMarshaler  methods. It means that even if implementation will be preserved, all implementation methods will be stripped.
Really, now I can see only two options - mark ICustomMarshaler with JSDeadCodeEleminationClassEntryPoint to preserve all its members and MarshalAsAttribute with JSDeadCodeEleminationEntryPoint.
But here we have another problem - DCE currently does not check for attributes on proxies. I'll fix it - really it my next priority task.

That makes sense! Having a proxy to indicate that ICustomMarshaler shouldn't be stripped seems like the right approach. I'm not actually sure MarshalAsAttribute gets compiled into the IL in the first place (it seems to be in special PInvoke metadata in cecil's object model) - I just couldn't figure out any way to get DCE to whitelist it.
A simpler approach might be to just write some special case code to mark custom marshalers as fully used. I may take another swing at that now that you've added test coverage.

Added #767 to support it. Until I'll fix proxy processing - just add somewhere call of something like this to preserve ICustomMarshaler methods implementations:
    private static void PreserveMarshaler()
    {
        if (new object() == new object())
        {
            ICustomMarshaler m = null;
            m.CleanUpManagedData(null);
            m.CleanUpNativeData(new IntPtr());
            m.GetNativeDataSize();
            m.MarshalManagedToNative(null);
            m.MarshalNativeToManaged(new IntPtr());
        }
    }

Last thing that we need to resolve it - mark ICustomMarshaler with JSDeadCodeEleminationClassEntryPoint thriugh proxies. @kg, will you try and report if this issue is resolved?


Fix cargo build warning

Which issue does this PR close?

Closes #.
Rationale for this change

Fix the following warning when building the project.
warning: some crates are on edition 2021 which defaults to `resolver = "2"`, but virtual workspaces default to `resolver = "1"`
note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest
note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @alamb


Add CodeQL security scanning

Hi, I'm a PM on the GitHub security team. This repository is eligible to try the new GitHub Advanced Security code scanning beta.
Code scanning runs a static analysis tool called CodeQL which scans your code at build time to find any potential security issues. We've tuned the set of queries to be only the most severe, most precise issues. We'll show alerts in the security tab, and we'll show alerts for any net new vulnerabilities on pull requests as well. We've tried to make this super developer friendly, but we'd love your feedback as we work through the beta.
If you're interested in trying it out, you can merge this pull request to set up the Actions workflow. You can also get this set up yourself in any additional repositories in this organization by following these instructions

Coverage remained the same at 40.503% when pulling aae43c373ea771253144fb76b958a2c6ad7f49c1 on jhutchings1:codeql into 069fe49c923db0679b3e8b1cc661a911078a0a60 on microsoft:master.

@jhutchings1 thanks for the PR! Is this something that will be able to replace the LGTM analysis that we are currently doing?

@a-b-r-o-w-n Yes, you could move from LGTM to code scanning. I'll note that we use a more restrictive set of queries by default, but you can get the same ones LGTM uses by adding the security-and-quality pack into the configuration.

@jhutchings1 excellent. I'll go ahead and merge this and let the two co-exist for a while.


Mix Fixes before release

This PR included a few changes for release 26

Fix device name , after recent PR https://github.com/vultisig/vultisig-ios/pull/539 , all device name become Unknown
Fix Keygen for 2/3 and M/N device type
Open external block explorer for bitcoin as well

LGTM


V0.3 s3 bucket

PRs #1 and #2 merged together now that we have a v0.3 branch that is stable

Issue with running newzealidar in debug mode, works fine in run mode.
https://github.com/GeospatialResearch/Digital-Twins/issues/278

Closing for now, because it looks like we won't need AWS S3 access directly, instead it can be handled with file mounting


For debugging consider passing CUDA_LAUNCH_BLOCKING=1
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
Exception raised from c10_cuda_check_implementation at ../c10/cuda/CUDAException.cpp:43 (most recent call first):
frame #0: c10::Error::Error(c10::SourceLocation, std::string) + 0x96 (0x7fe2d5e9af86 in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libc10.so)
frame #1: c10::detail::torchCheckFail(char const*, char const*, unsigned int, std::string const&) + 0x64 (0x7fe2d5e49d10 in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libc10.so)
frame #2: c10::cuda::c10_cuda_check_implementation(int, char const*, char const*, int, bool) + 0x118 (0x7fe2d5f75f08 in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libc10_cuda.so)
frame #3: c10d::ProcessGroupNCCL::WorkNCCL::finishedGPUExecutionInternal() const + 0x56 (0x7fe287f683e6 in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libtorch_cuda.so)
frame #4: c10d::ProcessGroupNCCL::WorkNCCL::isCompleted() + 0xa0 (0x7fe287f6d600 in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libtorch_cuda.so)
frame #5: c10d::ProcessGroupNCCL::watchdogHandler() + 0x1da (0x7fe287f742ba in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libtorch_cuda.so)
frame #6: c10d::ProcessGroupNCCL::ncclCommWatchdog() + 0x10c (0x7fe287f766fc in /root/anaconda3/lib/python3.12/site-packages/torch/lib/libtorch_cuda.so)
frame #7:  + 0xdbbf4 (0x7fe2d56c7bf4 in /root/anaconda3/bin/../lib/libstdc++.so.6)
frame #8:  + 0x94ac3 (0x7fe2d6bd9ac3 in /usr/lib/x86_64-linux-gnu/libc.so.6)
frame #9:  + 0x126850 (0x7fe2d6c6b850 in /usr/lib/x86_64-linux-gnu/libc.so.6)
E0806 10:11:47.450000<PHONE_NUMBER>87872 torch/distributed/elastic/multiprocessing/api.py:833] failed (exitcode: -6) local_rank: 0 (pid: 100) of binary: /root/anaconda3/bin/python
Traceback (most recent call last):
File "/root/anaconda3/bin/torchrun", line 8, in
sys.exit(main())
^^^^^^
File "/root/anaconda3/lib/python3.12/site-packages/torch/distributed/elastic/multiprocessing/errors/init.py", line 348, in wrapper
return f(*args, **kwargs)
^^^^^^^^^^^^^^^^^^
File "/root/anaconda3/lib/python3.12/site-packages/torch/distributed/run.py", line 901, in main
run(args)
File "/root/anaconda3/lib/python3.12/site-packages/torch/distributed/run.py", line 892, in run
elastic_launch(
File "/root/anaconda3/lib/python3.12/site-packages/torch/distributed/launcher/api.py", line 133, in call
return launch_agent(self._config, self._entrypoint, list(args))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/anaconda3/lib/python3.12/site-packages/torch/distributed/launcher/api.py", line 264, in launch_agent
raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
tools/train.py FAILED
Failures:
<NO_OTHER_FAILURES>
Root Cause (first observed failure):
[0]:
time      : 2024-08-06_10:11:47
host      : Lab-PC
rank      : 0 (local_rank: 0)
exitcode  : -6 (pid: 100)
error_file: <N/A>
traceback : Signal 6 (SIGABRT) received by PID 100
Environment
ubuntu 22.04, GeForce RTX3090, Driver 550.67, cuda 12.4, python3.12.4, torch2.4.0, torchvision0.19.0
Execution Command
CUDA_VISIBLE_DEVICES=0 torchrun --master_port=9909 tools/train.py -c configs/rtdetrv2/rtdetrv2_r50vd_m_7x_coco.yml --seed=0
I try to use custom coco dataset for training, so I only modified the rtdetrv2_pytorch/configs/dataset/coco_detection.yml with the corresponding "num_classes" and "remap_mscoco_category: False". I'm not sure whether there's any other configs that need to modify which causes the above corruption. Would anyone give a hint about how the above happened and how to solve it, that would be very helpful, thanks!

num_classes should be actual no.of classes + 1 when you set remap_mscoco_category: False
That's why you are getting index error!


Upgrade project to Angular 7.1

Upgrade project dependencies to Angular 7.1
Should Close #494

Why is it taking so long to merge this one.. I hope it fixes the bootstrap issue..

Can we get this merged please?


SPA & Static

This module seems to be incompatible with static and SPA modes (because only server calls are rewritten). I would like to get a confirmation for this statement, then we could do something like:

Add disclaimer in README.md and during nuxt generate
Add disclaimer in modules that use this module (like axios module?)
Consider a way for this plugin to work for other modes (rewriting URL's during generating?)

I'd appreciate a PR for the documentation/README files of at least this module.

#10

I don't think there is a "suitable" way to make it working for SPA/generated mode.
URL rewrites are something different than using a proxy. Leaving people using that package in the belief that they'll achieve "the same" is wrong IMO.

Excuse my superficial server knowledge but can't a static server also forward a request to another server? I understand that a static build directory does not by itself have this capacity, but it would be awesome if npm start (which runs a local server for testing) could proxy requests to the API. This is what we're doing on our production server.


High CPU usage

I thought I saw an rclone update in my terminal, a day or so ago (Arch Linux package manager).  Since then my rclone backups to Dropbox (fine previously; negligible CPU use) now at times uses way more CPU resources --  up to 100% (all 8 cores, sustained), at times.  That is, in the first image below, CPU use will be 800%.
rclone v1.45
Arch Linux x86_64
rclone sync --ignore-size --tpslimit 10 --skip-links /mnt/Vancouver/ "dropbox:backups/vancouver" --filter-from /home/victoria/backups/rclone/rclone_filters-vancouver.txt
Edit: it was due to my filters file -- see my reply, below.

However, I also remembered recently adding a LOT of lines to my filters file: files with names ending in { .css | .DS_store | ...}; ~500 lines, approximately -- files that Dropbox excludes from backups.

Ah!  rclone keeps the filters as a linear list and will scan through them for each file which probably explains the CPU usage.  Things will run much quicker if you can reduce the number of filter lines eg using
*.{jpg,png,gif}

rather than having 3 separate lines
*.jpg
*.png
*.gif

It would be possible for rclone to be a bit cleverer about the filters, but I have to say this is the first report of it causing a problem.


Added SCHISM as a submodule

Added SCHISM as a submodule

are these changes implemented in #44 ?

I guess you can reject this And we'll work on #44

closing, working #44 and #32


Add support to use product types from oms package from product module

What is the motivation for adding/enhancing this feature?

What are the acceptance criteria?

[ ] Product should be typed across app

Can you complete this feature request by yourself?

[x] YES
[ ] NO

Additional information

Duplicate ticket: https://github.com/hotwax/preorder/issues/82
Hence closing


Refactor default command to accept raw text

Hi,
during my attempts to use this library, I found that it was missing simple view that allowed me to just reply to a message with raw text. Creating a template can sometimes be an issue, or an overkill, depending on the situation. Especially when new to Django which is my case.
I have tried to refactor the TemplateCommandView the best I could, with a new BaseCommandView which could be the base for other types of actions (creating records for instance), and another base SendMessageCommandView which is the base for anything to send a message, which TemplateCommandView now extends.
Keeping full backwards compatibility has been in my mind. Which I think I achieved, except for the import of TemplateCommandView from telegrambot.bot_views.generic.base which now is in message, but importing from telegrambot.bot_views.generic still works.
Let me know what you think.
(This is rebased on my branch tests which #23 is based on as well, so only this commit is new: 6d78ea1)

Coverage decreased (-0.6%) to 95.501% when pulling 6d78ea107012f044be1ee97af03ca019cc2da356 on FMCorz:send-message-command into ef6a5c2081a94aeb95735f54ecbcffbda79b3c9f on jlmadurga:master.

I'm trying to figure out why I've got all these failures for Python 3 now...

Coverage decreased (-0.5%) to 95.64% when pulling 637a68f77c915b0df8d2493b39fba8c54d560695 on FMCorz:send-message-command into 5141ed0bd0ffbdf98d09e5c8a24845d18c166cd8 on jlmadurga:master.

Coverage increased (+0.2%) to 96.408% when pulling 30d8c0962700d21f23899fb31e1937386f15df65 on FMCorz:send-message-command into 5141ed0bd0ffbdf98d09e5c8a24845d18c166cd8 on jlmadurga:master.


[IDEA] Add ability to share Juicebox contribution to Twitter

Problem to solve
Raised in:

https://discord.com/channels/775859454780244028/876252266594721873/929484817677053962
https://twitter.com/altryne/status/1479925866985631747

Juicebbox should prompt you to share immediately after contribution, with the link to the project and a hashtag of their choice TBH.

Proposal
TBA, open to proposals. Probably use https://developer.twitter.com/en/docs/twitter-for-websites/web-intents/overview
More details
Add any other notes or screenshots about the feature request here.

Decided against this one


Fix node.duplicate() crash when duplicating a node that cannot be instantiated.

closes #90991

what the hell?

did it fix it?

Your rebase went wrong and it added you as owner for other commits

Your rebase went wrong and it added you as owner for other commits

Yeah no, it was my own fork's master branch. I used git merge on it, and it resulted in that happening.
Now i used git reset --hard mainstream/master to properly sync my master branch so it wont happen again.

Thanks!


Extraction

Testing the tool 'pull-request' with Sasha's bug patches on the extraction branch.

Branch has bugs and won't auto-merge with master.


[FR]: NSSecureCoding support

Description
Hi guys, I notice that some chunks in the code in iOS (like FirebaseMessaging) is using NSCoding protocol
I know you are working on that incrementally, but I would like to know when can we have full support for NSSecureCoding in all products?
API Proposal
No response
Firebase Product(s)
Messaging

A git grep shows the following NSCoding occurrences in Messaging and Database:
 $ git grep NSCoding
FirebaseDatabase/Sources/Persistence/FPendingPut.h:@interface FPendingPut : NSObject <NSCoding>
FirebaseDatabase/Sources/Persistence/FPendingPut.h:@interface FPendingPutPriority : NSObject <NSCoding>
FirebaseDatabase/Sources/Persistence/FPendingPut.h:@interface FPendingUpdate : NSObject <NSCoding>
FirebaseMessaging/Sources/Token/FIRMessagingAPNSInfo.h:@interface FIRMessagingAPNSInfo : NSObject <NSCoding, NSCopying>
FirebaseMessaging/Sources/Token/FIRMessagingAPNSInfo.m:#pragma mark - NSCoding
FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.h:@interface FIRMessagingTokenInfo : NSObject <NSCoding>
FirebaseMessaging/Sources/Token/FIRMessagingTokenInfo.m:#pragma mark - NSCoding

Only updating Messaging for now. FPendingPut in RTDB does not have tests and may only be used in an outdated code path.


Error with emailed PDF

Good Morning,
While trying to make an invoice today -- every time I email -- the PDF is not generating correctly.
Please see attached example
Thanks!
invoice.pdf

@wseagar How do we solve this issue?

This is happening for me as well but instead I get a blank pdf.


document says that Observable has let operator as default but not in implementation

RxJS version:
v5.5, v6
Code to reproduce:
with v6.0.0-alpha.3:
require('rxjs/index').Observable.create().let(x => x);
require('rxjs/index').Observable.create().pipe(x => x);

in v5.5, replace 'rxjs/index' with 'rxjs/Observable'
Expected behavior:
same result
ref. doc/pipeable-operators.md

The let operator is now part of Observable as pipe and cannot be imported.

Actual behavior:
require('rxjs/index').Observable.create().let(x => x); // let is not a function
require('rxjs/index').Observable.create().pipe(x => x); // properly works

Which is correct, implementation or document?

did U sure create() is OK ?
THIS IS MY CODE
node v7
rxjs v6 aplha
const { Observable, Subject, ReplaySubject } = require('rxjs');
const log = x => console.log(x);

Observable.of(1)
    .let(x => x)
    .subscribe(log);

RxJS v6.0.0-alpha3 has a problem about main field of package.json.
This problem causes the constructor of Observable imported by require('rxjs') is patched all of operators, and this problem is fixed in #3316.
FYI: src/internal/Observable.ts does not have let operator, but has pipe operator.

This is a documentation issue... the current docs are for v5... in v6 let is just pipe with one argument.


Merge in latest changes to gmake-refactor branch

This pull request is to get the ball rolling with premake/premake-core#516.

Merges in latest changes from premake/premake-core
Fixes merge conflicts regarding premake/premake-core#628 and premake/premake-core#623
Fixes broken tests
Fixes a regression where custom build commands would not be executed if the file extension wasn't in the filesets table, and adds tests to make sure that this doesn't happen again

I've tested this branch with a personal project of mine, and from what I can tell, the exporter seems to be working fine.
The D language module is still broken in this branch. I'm not very familiar with it, so I don't feel too comfortable with fiddling around. I'll have a look, and if it's something that I can easily fix, I'll create a separate pull request for it.

awesome, thank you for your help...
would you mind it if I tried re-basing this branch to master, so it just gets your changes on top?


Updating DefaultTabController with length greater than 2 in a StatefulWidget, then tapping the last tab, results in "setState() called after dispose()" exception

Steps to reproduce

Run the included sample app in web or any platform.
Tap the + fab to create three tabs.
Tap the last tab, and exception will be thrown.

Expected results
Expected exception to not be thown.
Actual results
Exception was thrown. See the Logs for details.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MaterialApp(home: MyHomePage()));
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _add(int i) => setState(() => _counter += i);

  @override
  Widget build(BuildContext context) {
    final intList = [for (var i = 0; i < _counter; i++) i];
    return Scaffold(
      appBar: AppBar(title: Text('TabController test with $_counter tabs')),
      body: intList.isEmpty
          ? const Center(child: Text('Tap + 3 times to create 3 tabs, then tap the last tab.'))
          : DefaultTabController(
              length: intList.length,
              child: Column(
                children: [
                  TabBar(tabs: intList.map((i) => Tab(text: 'Tab $i')).toList()),
                  Expanded(
                    child: TabBarView(
                        children: intList.map((i) => Center(child: Text('View $i'))).toList()),
                  )
                ],
              ),
            ),
      floatingActionButton: Row(
        mainAxisSize: MainAxisSize.min,
        children: [
          FloatingActionButton(onPressed: () => _add(1), child: const Icon(Icons.add)),
          const SizedBox(width: 16),
          FloatingActionButton(onPressed: () => _add(-1), child: const Icon(Icons.remove)),
        ],
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
Launching lib/main.dart on Chrome in debug mode...
This app is linked to the debug service: ws://<IP_ADDRESS>:53608/FENKUkPN7nc=/ws
Debug service listening on ws://<IP_ADDRESS>:53608/FENKUkPN7nc=/ws
Connecting to VM Service at ws://<IP_ADDRESS>:53608/FENKUkPN7nc=/ws
══╡ EXCEPTION CAUGHT BY ANIMATION LIBRARY ╞═════════════════════════════════════════════════════════
The following assertion was thrown while notifying listeners for AnimationController:
setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
This error happens if you call setState() on a State object for a widget that no longer appears in
the widget tree (e.g., whose parent widget no longer includes the widget in its build). This error
can occur when code calls setState() from a timer or an animation callback.
The preferred solution is to cancel the timer or stop listening to the animation in the dispose()
callback. Another solution is to check the "mounted" property of this object before calling
setState() to ensure the object is still in the tree.
This error might indicate a memory leak if setState() is being called because another object is
retaining a reference to this State object after it has been removed from the tree. To avoid memory
leaks, consider breaking the reference to this object during dispose().

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 297:3       throw_
packages/flutter/src/widgets/framework.dart 1167:9                                <fn>
packages/flutter/src/widgets/framework.dart 1201:14                               setState
packages/flutter/src/widgets/transitions.dart 129:5                               [_handleChange]
packages/flutter/src/animation/listener_helpers.dart 161:11                       notifyListeners
packages/flutter/src/animation/animation_controller.dart 865:5                    [_tick]
packages/flutter/src/scheduler/ticker.dart 258:12                                 [_tick]
packages/flutter/src/scheduler/binding.dart 1386:7                                [_invokeFrameCallback]
packages/flutter/src/scheduler/binding.dart 1233:11                               <fn>
dart-sdk/lib/_internal/js_dev_runtime/private/linked_hash_map.dart 21:7           forEach
packages/flutter/src/scheduler/binding.dart 1231:16                               handleBeginFrame
packages/flutter/src/scheduler/binding.dart 1148:5                                [_handleBeginFrame]
lib/_engine/engine/platform_dispatcher.dart 1361:5                                invoke1
lib/_engine/engine/platform_dispatcher.dart 237:5                                 invokeOnBeginFrame
lib/_engine/engine/initialization.dart 176:36                                     <fn>
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 550:37  _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39  dcall

The AnimationController notifying listeners was:
  AnimationController#fca21(▶ 0.077)
════════════════════════════════════════════════════════════════════════════════════════════════════
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)
Another exception was thrown: setState() called after dispose(): _AnimatedState#e0e73(lifecycle state: defunct, not mounted)

Flutter Doctor output
Doctor output
√ ~/Development/flutter_app_tests 🌀 flutter doctor -v
[✓] Flutter (Channel stable, 3.19.6, on macOS 14.4.1 23E224 darwin-arm64, locale en-US)
    • Flutter version 3.19.6 on channel stable at /Users/ronbooth/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 54e66469a9 (2 weeks ago), 2024-04-17 13:08:03 -0700
    • Engine revision c4cd48e186
    • Dart version 3.3.4
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/ronbooth/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)

[✓] VS Code (version 1.88.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.88.0

[✓] Connected device (4 available)
    • Ron’s iPad Pro (mobile)             • 00008103-00084486018A001E            • ios            • iOS 17.4.1 21E236
    • iPad mini (6th generation) (mobile) • AEABF899-FFD4-41D8-9757-0320607C6F2B • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)                     • macos                                • darwin-arm64   • macOS 14.4.1 23E224
      darwin-arm64
    • Chrome (web)                        • chrome                               • web-javascript • Google Chrome
      124.0.6367.119

[✓] Network resources
    • All expected network resources are available.

• No issues found!
√ ~/Development/flutter_app_tests 🌀

Here's a link to the sample app for reproducing the bug in DartPad: DartPad example

Looks like this bug is a duplicate of https://github.com/flutter/flutter/issues/144087, which has been fixed, just hasn't made it to stable yet.


Modifying settings.json causes an error

Whenever I modify my settings.json file, I receive this error.
(json-to-ast) Cannot tokenize symbol </> at (/Users/svillars/Library/Application Support/Code/User/settings.json:1:1)
> 1 | // Place your settings in this file to overwrite the default settings
    | ^
  2 | {
  3 |     "vim.insertModeKeyBindings": [
  4 |         {

Disabling this extension makes it go away.
vscode Version 1.14.1 (1.14.1)
macOS 10.11.6 (15G1421)

Duplicate of #149
Comments are not part of JSON spec. The parser currently used is strict with that 😢


Fix CI Error for Python 3.x

I included this fix in #616, but I figured I'd break it out for a quick and easy fix.

Thanks, I've meant to fix that for a while, but kept putting it off :(

No worries.  I kept running into the error while working on #616 and got annoyed enough that I had to figure it out and fix it 😆


Correct typos

I'm guessing this is just a typo. Saw it in the email link and couldn't help but fix it lol.

thank you @niieani

thank you @niieani. @elitan is the "type" of the ticket used anywhere? I don't think so.

Sometimes it is but not for passwordlessEmail. This PR is OK. I'll merge.


[Feature request] option to expand link only post

Many RSS feeds, miss-use RSS and just post a link to the website page, could you add a tick box option to the feed options to follow the first found link and expand that instead? Also for the thumbnail if it isn't found.
Explosm.net and sciencedaily.com are examples of this.
Thanks! As a fellow developer I know how annoying it is, but it would seriously improve the app with minimal work ;).

Thanks! Sure, let me look into this as soon as time permits.

I second this, a mechanism like the one in Flym or in Feeder (F-Droid) would be very welcomed. Reading the article in the browser is the second best, but this disrupts a little the user experience. Thank you!


3.0 Code Updates

Migrated to SDK-style project
Upgraded to SpecFlow 3
Upgraded to MSTest.Framework and MSTest.Adapter
Added ability to attach files to MSTest test results context
Added ability to set the browser time zone for Microsoft Edge (Chromium) and Google Chrome
Added web driver logging support
Support Win App Driver, keeping track of the page/window/dialog history
Added TempWebElement
Added support for double-clicking, right-clicking, and mouse cursor hover
Added support for closing the browser/dialog
Added support for entering text
Added support for maximizing and switching windows, and focusing the window before clicking on elements
Added support for navigating to the previous page (going back) and refreshing the page
Added support for validating the page URL query string
Added support for setting element data using JavaScript
Added support for setting the browser page load strategy from the configuration
Added support for validating virtual properties
Added support for page sections and contexts
Added support for waiting until validation table
Improved support for elements that require custom logic to get and set values
Fixes #48
Fixes #66
Fixes #71
Fixes #118
Fixes #170
Fixes #171
Fixes #176
Fixes #183
Fixes #185
Fixes #186
Fixes #187
Fixes #192
Fixes #197

Fixes #48
Fixes #66
Fixes #71
Fixes #118
Fixes #170
Fixes #171
Fixes #176
Fixes #183
Fixes #185
Fixes #186
Fixes #187
Fixes #192
Fixes #197

Hi Rami,
This is a big contribution, thanks! Just as a heads up, the review on this is going to take a while. 247 files and 13 issues in a PR is a lot to mentally digest. I'll make comments as we go, so others can understand the conversation. I do think we should version this as 3.0 given the number of changes.
One larger theme that comes to mind is should we continue support of CodedUI. Based on this article: https://devblogs.microsoft.com/devops/changes-to-coded-ui-test-in-visual-studio-2019/ the framework seems to be deprecated. Thoughts?

It's true that Coded UI is being deprecated.
I'm thinking we can at least try to keep it around as long as it's not too hard to maintain.
I'm not expecting all features in SpecBind to be implemented in Coded UI.
For example, some features may only exist when using Selenium.

Thank you for the review.

* I'm still a bit concerned with the size of this PR overall, when I've added actions it makes it easy to check off all the boxes at the step, action, driver, what each platform implements and docs when you can see it. We need at a minimum to change the PR name to '3.0 Code Updates' and then document them all in the description alongside the issue.

I changed the name of the PR to "3.0 Code Updates" and will work on the documentation alongside additional tests.

* Since docs are now embedded in the code base we should have documentation updates

I updated the GitHub pages to use a customized version of the just-the-docs jekyll theme for easier navigation and will will work on adding more documentation.

* The whole application concept needs to be covered and we should think about wither it needs to be in a separate assembly. Long term I see this running on .NET core and some of the interop code makes that challenging.

.NET Core support seems like a feature outside the scope of this PR.
I'm not sure if the changes I've made are the only things that would prevent us from migrating to .NET Core.
Creating a separate assembly which adds support for WinAppDriver also seems like a feature outside the scope of this PR.
SpecBind.Selenium isn't currently designed to support initializing Selenium drivers from external assemblies.
I think this can be addressed in a subsequent PR.

* Please make sure that any new actions, classes etc. added have unit tests covering them as close to 100% as possible. With few maintainers, high test coverage is the only way we can refactor without the worry that we broke something.

I can work on improving code coverage by adding tests for the code that was added and changed, as well as adding documentation for the new features.
The coverage is 81.52% right now. It was previously 84.43%.

* I appreciate the switch to GH actions, I do want to make sure we aren't loosing any steps along the way. I haven't had time to do a full build comparison yet, but I'd like to.

Appveyor's environments don't have Visual Studio Enterprise installed, whereas GitHub does.
This allows us to use the official Coded UI dependencies on the build agent's machine instead of the unofficial Coded UI NuGet package.
The only other thing I noticed is Appveyor.yml sends a notification to gitter.im.
This isn't included in the GitHub workflow and I don't think it's a show stopper.
There hasn't been any activity in https://gitter.im/dpiessens/specbind since July of 2018.
I'm not sure if that space should eventually be moved to https://gitter.im/specbind/specbind, and/or GitHub Discussions enabled for this repo.

Thanks for the updates to date. To focus the discussion down to a few areas:

I'm not sure a discussion forum is needed at this point, so we can ignore that. I'm fine with the GH actions setup, and I will need to move package release processes over to complete that migration.

Thanks to the updates to themes and docs, given your keen eye to complete documentation in the past I figured that would be an important part of the new actions added.

In terms of code coverage, I'm less concerned about the percentage want to focus more what's covered.  The steps, actions and top level driver things are the ones I always tried to get as close to 100% as possible as it's small wire-up features there that are the easiest to break and having those tests give confidence when we need to refactor. The lower level interactions are more difficult and if you're confident they work then we should add an  [ExcludeFromCoverage] attribute explaining why. I realize I haven't expressed these goals in writing prior, so I'll be adding a contribution guide for this.

I agree that moving to .NET core is a separate PR. I don't want to add additional work to that goal, by taking dependencies on the registry and native methods that might make that difficult. If you're against splitting out the native method portions due to the work involved, then I want to label the windows interaction as a "beta" feature and subject to breakage as we move to .NET core.

Hi, what is left to do on this pull request? There are some features that were added that I would like to leverage.

I had some difficulty trying to isolate the code paths which require test coverage when developing locally to satisfy bullet point #3.
The feedback loop was taking a little too long.
Do you have any advice or recommendations on a better approach which can quickly identify the code paths that need test coverage?
Thank you.


unable to choose a snapshot in image list

I use ovh public cloud, and it is compatible with open stack.
Because windows images there do not support cloud init, I have created my vm where I have installed cloudbase init, then shut it down and taken a machine snapshot, and this snapshot appears in the "openstack image list".
However, image selection in jenkins shows only public images, does not show my snapshot at all, and it is not possible to select it nor to type it's id or name or to do whatever.

I have no experience with or access to the stack you mention. Ware you able to verify by some other kind of openstack client that the snapshots are listed? Like the console tools (nova/glance.? Can you compare the attributes with some of the working image like visibility / sharing (if applicable)?

not sure if sharing is supported by that api version, but visibility is
private.
As I said before, it is listed by "openstack list image", "openstack" is
a new style command line client replacing nova/glance/whatever, I see it
and was able to use it few times, and the jclouds plugin sees it.
However jclouds plugin seems to have other problems.

I have just noticed that even some public images are not displayed. like probably mostly/only those used for gpu instances, but I believe (from what I read in the code) that they should be displayed provided that they are not filtered based on any criteria. Not sure if that is a plugin bug, openstack4j bug, or what.

Okay, I know what's going on. The plugin probably does not actually apply the set region name, that is:
When creating an "Openstack" instance, the OSClient object is build, and the useRegion method is called on it, but from what I realized, this client is not used for anything, except that the access token is retrieved from the instance and stored, however the token does not contain any info about used regions.
Then, when the openstack plugin wants to use the client for anything, it retrieves it using a ClientProvider object containing the token, the new instance is always created with OSClientFactory.clientFromAccess, but useRegion is not called on it, and region is not really applied anywhere while doing actual api calls.

@webczat, spot on! Will be part of next release.


Configurable Path

Make paths configurable in all  NullAwayAutoFixer artifacts.

I would strongly prefer if there wasn't a global path, even as a default. Can we just make the checker fail with a configuration error if run with the autofix flag on and without an explicit value for -XepOpt:NullAway:AutoFixOutPutDirectory?
Also, reminder that another thing that needs to happen is that directory being write-only from the point of view of NullAway, and non-persistent from the point of view of the external toolchain, so the python scripts probably still need another directory to copy ongoing working data to (with the idea being that the AutoFixOutPutDirectory is cleared between compilations).

I would strongly prefer if there wasn't a global path, even as a default. Can we just make the checker fail with a configuration error if run with the autofix flag on and without an explicit value for -XepOpt:NullAway:AutoFixOutPutDirectory?
Also, reminder that another thing that needs to happen is that directory being write-only from the point of view of NullAway, and non-persistent from the point of view of the external toolchain, so the python scripts probably still need another directory to copy ongoing working data to (with the idea being that the AutoFixOutPutDirectory is cleared between compilations).

Ok sure will do that, I will apply these changes now. Thank you.

I would strongly prefer if there wasn't a global path, even as a default. Can we just make the checker fail with a configuration error if run with the autofix flag on and without an explicit value for -XepOpt:NullAway:AutoFixOutPutDirectory?
Also, reminder that another thing that needs to happen is that directory being write-only from the point of view of NullAway, and non-persistent from the point of view of the external toolchain, so the python scripts probably still need another directory to copy ongoing working data to (with the idea being that the AutoFixOutPutDirectory is cleared between compilations).

Hi @lazaroclapp, I have one question regarding configurable paths. Both script and NullAway should be informed about the output directory to interact with each other. In Gradle projects we can pass it via setting this flag -XepOpt:NullAway:AutoFixOutPutDirectory in tasks.withType(JavaCompile) function in build.gradle file and also should configure path in the python script. These two directories should be set in two places rather than only once place. Even if I find a way to automatically modify build.gradle files with python script, I am not sure how we can do it at Uber. Would you please tell me how we can address this issue or we can temporarily move forward with the current situation.


[BUG]: DeploymentQuotaExceeded - Creating the deployment ... would exceed the deployment location quota

Task name
AzureResourceManagerTemplateDeployment
Task version
3
Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[X] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
ubuntu 22.04
Task log
##[error]At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.
##[error]Details:
##[error]DeploymentQuotaExceeded: Creating the deployment ... would exceed the deployment location quota. The current scope allows deployments to be created in at most '10' locations. The following locations are used for deployments: 'WESTUS,NORTHEUROPE,EASTUS,JAPANEAST,WESTUS2,EASTUS2,CENTRALUS,WESTEUROPE,AUSTRALIAEAST,UKSOUTH'. Please create your deployment in other locations. Please see https://aka.ms/arm-deploy-resources for usage details.
##[error]Check out the troubleshooting guide to see if your issue is addressed: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/deploy/azure-resource-group-deployment?view=azure-devops#troubleshooting
##[error]Task failed while creating or updating the template deployment.

Relevant log output
Please see Task Log

Aditional info
I am trying to use West Us 3. The valuethat I am using is westus3. I have deployed arm directly using the cli there. Not sure why the task is breaking.

Why did this go stale?


Fixes a bug in Runs to return all runs

Fixes a bug in runs to return all runs instead of a single one, because
as it seems when we use sqlalchemy.func.count, we need to do a group_by.
Test plan:

see the tests
run sapp and goto runs page to see all runs there

Co-authored-by: Esieboma Jeremiah<EMAIL_ADDRESS>Signed-off-by: Abishek V Ashok<EMAIL_ADDRESS>

tests are failing to things unrelated in this PR. This PR fixes the runs tests :)


Does xlnet supports multiple GPU？

I wonder whether the xlnet code support  multiple GPUS？
Thanks in advance!

Yes. It's given in README.md.
Use CUDA_VISIBLE_DEVICES=0,1,2,3 for 4 multi-GPU finetuning.

@Sumegh-git what  about pretraining?

train_gpu is intended for multi-gpu pretraining.

Great job！