Issue to load vit_h_14 model with pretrained weights (DEFAULT or IMAGENET1K_SWAG_E2E_V1)

🐛 Describe the bug
I faced the below issue after training vit_h_14 model with pretrained weights. If I do not load pretrained weights, everything is fine.
how to reproduce this bug
import torchvision
model = torchvision.models.get_model('vit_h_14', weights='DEFAULT')
#or
#model = torchvision.models.get_model('vit_h_14', weights='IMAGENET1K_SWAG_E2E_V1')
Traceback (most recent call last):
  File "train.py", line 545, in <module>
    main(args)
  File "train.py", line 225, in main
    model = torchvision.models.get_model(args.model, weights=args.weights)
  File "/usr/local/lib/python3.7/dist-packages/torchvision/models/_api.py", line 225, in get_model
    return fn(**config)
  File "/usr/local/lib/python3.7/dist-packages/torchvision/models/vision_transformer.py", line 764, in vit_h_14
    **kwargs,
  File "/usr/local/lib/python3.7/dist-packages/torchvision/models/vision_transformer.py", line 335, in _vision_transformer
    model.load_state_dict(weights.get_state_dict(progress=progress))
  File "/usr/local/lib/python3.7/dist-packages/torchvision/models/_api.py", line 66, in get_state_dict
    return load_state_dict_from_url(self.url, progress=progress)
  File "/usr/local/lib/python3.7/dist-packages/torch/hub.py", line 731, in load_state_dict_from_url
    return torch.load(cached_file, map_location=map_location)
  File "/usr/local/lib/python3.7/dist-packages/torch/serialization.py", line 726, in load
    with _open_zipfile_reader(opened_file) as opened_zipfile:
  File "/usr/local/lib/python3.7/dist-packages/torch/serialization.py", line 262, in __init__
    super(_open_zipfile_reader, self).__init__(torch._C.PyTorchFileReader(name_or_buffer))
RuntimeError: PytorchStreamReader failed reading zip archive: failed finding central directory
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 33723) of binary: /usr/bin/python3
Traceback (most recent call last):
  File "/usr/local/bin/torchrun", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.7/dist-packages/torch/distributed/elastic/multiprocessing/errors/__init__.py", line 345, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/torch/distributed/run.py", line 761, in main
    run(args)
  File "/usr/local/lib/python3.7/dist-packages/torch/distributed/run.py", line 755, in run
    )(*cmd_args)
  File "/usr/local/lib/python3.7/dist-packages/torch/distributed/launcher/api.py", line 132, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/usr/local/lib/python3.7/dist-packages/torch/distributed/launcher/api.py", line 248, in launch_agent
    failures=result.failures,
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
train.py FAILED
------------------------------------------------------------

Versions
Collecting environment information...
PyTorch version: 1.13.0.dev20220810+cu113
Is debug build: False
CUDA used to build PyTorch: 11.3
ROCM used to build PyTorch: N/A
OS: Ubuntu 18.04.5 LTS (x86_64)
GCC version: (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.26
Python version: 3.7.5 (default, Dec  9 2021, 17:04:37)  [GCC 8.4.0] (64-bit runtime)
Python platform: Linux-5.4.0-122-generic-x86_64-with-Ubuntu-18.04-bionic
Is CUDA available: True
CUDA runtime version: 11.2.152
GPU models and configuration:
GPU 0: NVIDIA GeForce RTX 3090
GPU 1: NVIDIA GeForce RTX 3090
GPU 2: NVIDIA GeForce RTX 3090
GPU 3: NVIDIA GeForce RTX 3090
Nvidia driver version: 470.141.03
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.1.0
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
Versions of relevant libraries:
[pip3] numpy==1.21.6
[pip3] pytorch-lightning==1.7.1
[pip3] pytorch-lightning-bolts==0.3.2.post1
[pip3] torch==1.13.0.dev20220810+cu113
[pip3] torchaudio==0.13.0.dev20220810+cu113
[pip3] torchmetrics==0.9.3
[pip3] torchvision==0.14.0.dev20220810+cu113
[conda] Could not collect

@giangdip2410 I can't reproduce the problem. The following works fine with me:
import torchvision
torchvision.models.get_model('vit_h_14', weights='DEFAULT')

Judging from your error message:
RuntimeError: PytorchStreamReader failed reading zip archive: failed finding central directory

It seems that the local weights were not downloaded properly or in the right location locally. They typically get stored at your ~/.cache/torch/hub/checkpoints. Try deleting the existing vit_h_14_* from there to force their redownloading and ensure that the path is accessible via your script when you run the analysis.

I also had the same problem when using pre-trained weights to train the ssd model,the command I used is 'torchrun --nproc_per_node=8 train.py
--dataset coco --model ssd300_vgg16 --epochs 120
--lr-steps 80 110 --aspect-ratio-group-factor 3 --lr 0.002 --batch-size 4
--weight-decay 0.0005 --data-augmentation ssd --weights-backbone VGG16_Weights.IMAGENET1K_FEATURES'.
and i have already checked the weights in ~/.cache/torch/hub/checkpoints , can you give me some advice?


Kchakrabarty/jsonvalidation

Patch Description
Added code to check for validity of json in jsonl files in the index building function. Also added two new tests to make sure that empty jsonl files as well as corrupt jsonl files throw up the right errors.
Testing steps
ran the new tests to ensure that it works as expected

Great tests!


79 bug query view next page last page option available when less results than page size

:tada: This PR is included in version 0.14.1 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


fix: remove unsupported .doc file ingest

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

Remove legacy .doc support from RAG ingest. The docx2txt loader does not support the legacy format, and using unstructured's loader requires that we install LibreOffice to handle the conversion.

Can't this happen that a .docx is actually called .doc? I'm thinking the file extension is not super reliable in general to begin with so might as well try just in case no?

Thanks!


Add git-annex and S3 repository types

closes #400

@apdavison thanks for taking this over. looks good!


Always auto save settings

Settings should always be auto saved
I ran into the issue too many times that my settings is dirty and that some ui can not write to it resulting in a bad user experience

Example, autosave off, make some changes to your settings.json
Try to change theme

Issue 1 is very frustrating to solve...

Given the issues like #32146, auto saving settings will not be implemented in near future. Hence closing this.


No disk space causes etcd to stall

I ran into an issue when the leader of the cluster ran out of disk space. It made the entire cluster unreachable for etcd and it seems that it couldn't re-elect a different leader.
I was able to fix the issue by simply removing the offending (leader) instance, but this is not an ideal situation.
Here's the last journal message I had on the no disk space instance (note: that peer relates to a follower): warning: heartbeat time out peer="e6ee689cb13e28ec6219cb0d131f979d
On the follower instance the log shows as follows (Jun 26 22:33:25 is when the leader ran out of space):
[etcd] Jun 26 22:33:24.764 INFO      | e6ee689cb13e28ec6219cb0d131f979d: term #20 started.
[etcd] Jun 27 09:24:52.703 INFO      | e6ee689cb13e28ec6219cb0d131f979d: term #21 started.
[etcd] Jun 27 09:24:52.703 INFO      | e6ee689cb13e28ec6219cb0d131f979d: leader changed from 'b1105d6b63f75e122ab33eaa3779a776' to ''.

At Jun 27 09:24:52 I removed the leader from the cluster and it seems to have triggered a new leadership election - everything started working again.

Please monitor your disk space. etcd 0.4 will not handle this situation. etcd2 will error on this situation and exit.


stylelint 12.x as peer dependency

Hi,
stylelint 12.0.0 was released but do you have a plan to update the peer dependency?
https://github.com/kristerkari/stylelint-high-performance-animation/blob/8a1227dc0f61e35506f55f04181f60dfd6a1b355/package.json#L47
I get the following error when installing stylelint@12 and stylelint-high-performance-animation. 😢

npm ERR! peer dep missing<EMAIL_ADDRESS>|| ^8.0.0 || ^9.0.0 || ^10.0.0 || ^11.0.0, required by<EMAIL_ADDRESS>
It would be great if you could consider it.
Thanks.

Yeah, I'll need to update all my plugins once again :) For now you can just ignore the warning.

OK, thanks 👍

Since there was a PR for it, I went and released a new version with stylelint 12 support already: https://github.com/kristerkari/stylelint-high-performance-animation/releases/tag/1.4.0

Thank you so much! 😊


feat: adds JSON get and write

Manual Tests Run.
Currently not validating JSON, use with caution.

This is super unpolished, and I did this as a mini project. The functionality is untested, and the UI is not consistent with the rest of your project, but it works. I meant to make this PR to my fork (I'm relatively new to Github). Love this project!


Signal handling overhaul

This is an implementation of the macroless from #15, described in this comment.

Problem - GDK's FromEvent is private, which means I can't have param_event and have to resort to the uglier API. This was not a problem for the BuilderSignal derive macro, because it could just use downast without having to specify a trait bound, but now that we use generics it becomes a problem.
I opened a ticket - https://github.com/gtk-rs/gtk-rs/issues/392. But I may have to accept this ergonomic hit for now, and add param_event in a future release...

I must admit that I'm a bit overwhelmed by the complexity of the current code – or maybe it's just the fact that I lost track of all the approaches and design iterations we already had. There are also quite a few use cases to cover …

Yes, this is a big change. But so far 62% of it is line deletions, so I consider it a simplification of what we previously had.

I think the examples are what you should be looking at - after all, they are the thing that represents how the new API looks like.

I'd really like to try it out right now, but I fear that I'm blocked on the resolution of #21

#21 should not block you - you can still use woab::route_signal. Take a look at tests/connect_nonbuilder_signals.rs.


Concat not supported along batch axis

Hi, i'm trying to perform a concatenation of 4 layers as follows:
basically input tensor is padded and cropped to 4 identical sized tensors stored at ''res'' eventually to be concatenated.
i get this error:
  File "/anaconda3/envs/onnx/lib/python2.7/site-packages/onnx_coreml/converter.py", line 468, in convert
    _convert_node(builder, node, graph, err)
  File "/anaconda3/envs/onnx/lib/python2.7/site-packages/onnx_coreml/_operators.py", line 1908, in _convert_node
    return converter_fn(builder, node, graph, err)
  File "/anaconda3/envs/onnx/lib/python2.7/site-packages/onnx_coreml/_operators.py", line 818, in _convert_concat
    return err.unsupported_op_configuration(builder, node, graph, "Concat not supported along batch axis")
  File "/anaconda3/envs/onnx/lib/python2.7/site-packages/onnx_coreml/_error_utils.py", line 56, in unsupported_op_configuration
    "Error while converting op of type: {}. Error message: {}\n".format(node.op_type, err_message, )
TypeError: Error while converting op of type: Concat. Error message: Concat not supported along batch axis

a python snippet to reproduce the error is attached below, thank you!
class multiMaxPooling(nn.Module):
    def __init__(self, kW, kH, dW, dH):
        super(multiMaxPooling, self).__init__()
        layers = []
        self.padd = []
        for i in range(0, dH):
            for j in range(0, dW):
                self.padd.append((-j, -i))
                layers.append(nn.MaxPool2d(kernel_size=(kW, kH), stride=(dW, dH)))
        self.max_layers = nn.ModuleList(layers)
        self.s = dH

    def forward(self, x):

        hh = []
        ww = []
        res = []

        for i in range(0, len(self.max_layers)):
            pad_left, pad_top = self.padd[i]
            pad_left, pad_top = abs(pad_left), abs(pad_top)
            h, w = x.size()[2], x.size()[3]
            _x = x[...,pad_top:h-pad_top,pad_left:w-pad_left]
            _x = self.max_layers[i](_x)
            h, w = _x.size()[2], _x.size()[3]
            hh.append(h)
            ww.append(w)
            res.append(_x)
        max_h, max_w = np.max(hh), np.max(ww)
        for i in range(0, len(self.max_layers)):
            _x = res[i]
            h, w = _x.size()[2], _x.size()[3]

            pad_top = np.floor((max_h - h.item()) / 2).astype(int)
            pad_bottom = np.ceil((max_h - h.item()) / 2).astype(int)
            pad_left = np.floor((max_w - w.item()) / 2).astype(int)
            pad_right = np.ceil((max_w - w.item()) / 2).astype(int)
            _x = F.pad(_x, [pad_left, pad_right, pad_top, pad_bottom], value=0)

            res[i] = _x
        return torch.cat(res, 0)

what is the shape of x that you used in the function call forward(self, x)?

input shape to forward(self, x) is [1,8,972,1292].
the expected output would be [4,8,486,646]. this is accomplished by splitting the x last two dimensions into 4 at the size of 972/2=486 and 1292/2=646 stored as 4 different batches.

CoreML tensors are rank 5 (internally) : [Seq, Batch, C, H, W]. Concat operation is not supported along the Batch axis. In fact its only supported along C axis, the converter internally transposes the tensors to support all other axis except batch. (transpose layer does not move the batch dimension which is sort of fixed)
so maybe you can try reshaping the tensor to rank 5 before concat to give (4,1,8,486,646).
Are you trying to do a max pooling operation with padding? The CoreML pool layer should be able to do that directly. See https://apple.github.io/coremltools/coremlspecification/sections/NeuralNetwork.html#poolinglayerparams, https://github.com/apple/coremltools/blob/master/docs/APIExamples.md#building-an-mlmodel-from-scratch-using-neural-network-builder
what is F in F.pad in the code snippet?

@erezposner support for concat across any axis is added with latest beta https://github.com/onnx/onnx-coreml#new-beta-onnx-coreml-converter-with-core-ml-3
Feel free to re-open if you still face any issue for concat conversion


Spelling mistake in ManipulationHandler.md

Describe the Issue
Spelling mistake in README_ManipulationHandler.md in the "Events" topic: Manupulation
Existing Doc Link
Unity/Documentation/README_ManipulationHandler.md

Thanks for filing this! I don't know who you are, but I'm really happy that you actually took the time to file this issue - I feel like most people would have been like "huh" and then moved on with their lives.
tl;dr issue reporter is the the hero we need.


fix: split lubuild and luis publish

Description
Cause: do luis build and luis publish even not luis bot in azurePublish plugin.
Solution: split lubuild and luis publish. Publishing luis when lu file not empty.
Task Item
close #3969
Screenshots
EchoBot can deploy without authoringKey.

Coverage remained the same at 56.747% when pulling bbc8a163d665c0345586aedecb6136a7f565ab5d on wenyluo/fix into 04d459e98c68d3be25d294d2309c295b7734901a on main.

I've tested this and found

it is working when no content in lu files and no luis.authoringKey configured.
there is an error when there is content in lu files and luis.authortingKey and endpointKey configured.

I will work with Wenyi tomorrow to figure this out.

There is a new issue that, then both luis and qna contents exist in the bot, sometimes the qna can be successfully published but sometime not,  we have investigated and found the root cause is:
It is not recommended to use async and await in the forEach loop.
crossTrainResult.forEach(async (value, key) => {
const fileName = path.basename(key);
const newFileId = path.join(this.interruptionFolderPath, fileName);
await fs.writeFile(newFileId, value.Content);
});
we will update on both localPublish and azurePublish side on this.


MINOR:  Add validation to the delete_idempotent_data method in GrpcPlacementService and fill in the test case

What type of enhancement is this?
Refactor
What does the enhancement do?
Add validation to the delete_idempotent_data method in GrpcPlacementService and fill in the test case.
Tips:
You can implement the ValidateExt trait on DeleteIdempotentDataRequest, then invoke validate_ext function to achieve the validation functionality.

you can locate the ValidateExt trait in the file: src\placement-center\src\server\grpc\validate.rs
you need to add validation to the delete_idempotent_data  method in this file:  src\placement-center\src\server\grpc\service_placement.rs
you need to fill in the test case in this file: src\grpc-clients\tests\placement_test.rs

references:

#526
#509

Implementation challenges
No response

@soumyasen1809 Yes, you can definitely work on this issue. Looking forward for your PR. Happy coding!

Hi @Pikady I am new to Rust and would like to take a shot at this item. Is it possible to work on this item?

Welcome @soumyasen1809

@socutes @Pikady
please find the PR https://github.com/robustmq/robustmq/pull/561 related to this


Install binary subcommand

@jsancio updated

LGTM @tamarrow !! :confetti_ball:

@tamarrow I just realized that there aren't any tests for this PR. We should at least have tests for a working "executable" subcommand, a working "zip" subcommand and a invalid "zip" subcommand.


Error: Building package gdal:x64-windows failed with: BUILD_FAILED

The GDAL BUID fails since it was upgraded the LIBMYSQL to a version that does not have the 'my_global.h' file.

Thanks for reporting! I've pushed a fix to master.


Update RAID disk regex

Add "md.+" so that raid devices of the form "md123" will match the regex.

Description
This fixes an issue I have where my raid device is named "md127", which does not match the unmodified regexes. This results in a case where the monitors will not detect if my raid array is degraded or not.
Type of change
What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[ ] FEATURE (non-breaking change which adds functionality)
[X] BUGFIX (non-breaking change which fixes an issue)
[ ] ENHANCEMENT (non-breaking change which improves existing functionality)
[ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry
Please put a one-line changelog entry below. Later this will be copied to the changelog file.

Modify mdadm regex to include arrays with the naming scheme "mdXXX".

@PhilipGough what are the next steps to getting this reviewed?

@AshtonKem your going to need to update the jsonnet and generate the yaml from those changes to pass CI.

Ah, I see. I went about this backwards. I'll create a new PR.


inconsistent names

when asking for us timezones valid today I get
TimeZoneNames.TZNames.GetTimeZonesForCountry("us","en",Date.UtcNow)
I get two mountain time zones
America/Phoenix Mountain Time (Phoenix)
America/Denver Mountain Time (Denver)
however when asking for the name of
TimeZoneNames.TZNames.GetNamesForTimeZone("America/Denver","en")
TimeZoneNames.TZNames.GetNamesForTimeZone("America/Phoenix","en")

I get for both a name of "Mountain Time"
is this by design or a bug?

Sorry for the delayed response.
It's by design, in that GetTimeZoneForCountry appends the city name when necessary to disambiguate results.  GetNamesForTimeZone simply returns the raw result.


should guild developer to use restangular with observable better!

a snippet of code
public page$ = new Subject();
public param1$ = new Subject();
constructor(
    private rest: Restangular
  ) {
  }
  ngOnInit() {
    this.stream$ = Observable.combineLatest(this.page$, this.param1$).throttleTime(200)
      .mergeMap(([page, param1], i) => {
          return this.rest
            .one('contract/contracts')
            .get({page, param1});
        }
      );

    this.stream$.subscribe(_ => {
      // this.users = _.data.data;
      this.data = _.data;
    });

    this.page$.next(1);
    this.param1$.next(5);
  }

  onDgPageChange(event) {
    this.page$.next(event);
  }

wish to have a better readme to guild developer to write observable with restangular

Hello @LinBoLen, restagular returns just observable. The usage of restangular supposes good knowledge of rxjs, but we'll try to refactor documentation in terms of more detailed guiding developer to use Observables.


Address docker hub rate limit for image mfenwick100/sonobuoy-cyclonus

Hi team,
Thanks for the cool project, just wonder do we have an alternative to download the image mfenwick100/sonobuoy-cyclonus
It's easy to hit rate limit issue in an enterprise network.
Thanks

Hi @ydp thanks for taking a look!  If there are a lot of people using this, perhaps we could get these images into a public gcr repo under the kubernetes project (somebody would need to work with the kube team to do that, I don't have the ability to do that by myself!)
As a workaround, you could set up a local image registry and mirror the images there, so they won't have to be pulled from dockerhub.


Normal Textures and MipMapping

Remove general flaws with Texture class and Mipmapping
The Texture Constructor's arguments default to

bool generateMipMaps = true and
TextureFilterMode filterMode = TextureFilterMode.LINEAR

which doesn't make sense because if you want to have mipmaps generated, you also probably want to have them used in interpolation, so filterMode should default to TextureFilterMode.NEAREST_MIPMAP_LINEAR.
Standard OpenGL allows to specify different values for the magnification and the minification filter. Not sure, if the possibility to specify only one single filter for both works well in all possible cases.
The TextureFilterMode enum values should be standard C# camel case (NearestMipmapNearest).
It should be possible to change the TextureFilterMode Property after construction.
Make Mipmapping work with Normal maps
When setting filterMode to TextureFilterMode.NEAREST_MIPMAP_LINEAR the result is a surface rendered in black (possibly with some highlights), even if the normal map intensity is set to 0.0f like so:
bumpeffect.SetEffectParam("NormalMapIntensity", 0f);

I changed the filter mode in the default constructor to TextureFilterMode.NEAREST_MIPMAP_LINEAR and moved the gl.GenerateMipmap calls in RenderContextImp under the gl.TexImage calls (I only got black textures before I did the second part).
To me it seems like the mipmaps are working as intended now.
As for min and mag filter:  Is it possible that we want so have different filter methods when magnifying and minifying? In this case I think we should divide the filter method in the constructor into min and mag. At the moment the singe filter parameter is resolved into min and mag in RenderContextImp (desktop version).

Derived ToDos:

[x] Set default filter mode to TextureFilterMode.NEAREST_MIPMAP_LINEAR
[x] Move the gl.GenerateMipmap calls in RenderContextImp under the gl.TexImage calls
[x] Allow different filter methods when magnifying and minifying? No...
[ ] Add Methods to Texture, RC and RCI that allows us to change texture paramers (filter mode, possibly wrap mode) at runtime.


Files not being excluded via Atom, but command line flake8 does

This is the setup.cfg file. I know that the linter reads the file because ignore is honored so there's something going on with the exclude. The first screenshot shows the linter raising an error in apollo/config/settings/local.py while the second screenshot clearly shows that this file was ignored. Let me know what additional info is needed.
Using Mac OS, virtualenvwrapper
[flake8]
ignore = D100, D210, D400, D413
exclude =
    .git,
    __pycache__,
    __init__.py,
    */migrations/*,
    apollo/config/settings/*.py,
    apollo/manage.py
max-line-length = 99
max-complexity = 10

Location of flake8: ~/Developer/.virtualenvs/apollo/bin/flake8
Project structure
.
├── apollo
│   ├── config
│   │   ├── __init__.py
│   │   ├── settings
│   │   │   ├── __init__.py
│   │   │   ├── base.py
│   │   │   ├── local.py
│   │   │   ├── production.py
│   │   │   ├── settings.py
│   │   │   ├── staging.py
│   │   │   └── test.py
│   │   ├── urls.py
│   │   └── wsgi.py
│   └── manage.py
└── setup.cfg

Screenshots

I was going to comment that flake8 has no method of specifying the file name for stdin content, but apparently it does as of v3.0 so this might be possible to fix now.

Ping on this?

v2.3.0 should fix this, let us know if it doesn't 😉.

@Arcanemagus I'm on 2.3.0 and still seeing this issue.

I am taking a look at this.

Currently, the linter executes Flake8 inside the project's root directory and it expects the configuration file to be located there. By project's root, I mean the topmost directory of the project according to Atom.
In the example provided by the OP, the files are correctly ignored. However, in any other situation where the configuration file is not in the project's root, flake8 does not recognize the file.
@Arcanemagus  I am not sure what the best approach would be. How the other linters solve this kind of issue? Should we try to locate the nearest configuration file and execute Flake8 inside that directory?

@lucasdf I can reproduce this with my config in the root of my project. I use a setup.cfg file as most Python tooling supports reading it.

However, in any other situation where the configuration file is not in the project's root, Flake8 does not recognize the file.

@lucasdf Ideally these providers should be "the same as running the tool in the CLI", unfortunately what that actually means can get really tricky. I generally follow the rules of:

"if you are in the directory of the file and run the CLI tool, the provider should match that"
"if you are in the project root (as seen by Atom) and run the CLI tool, the provider should match that"

Which one of those a provider falls under is just the start of the trickiness 😉.
I think this provider should follow the second one, unless this issue is describing a situation where that doesn't match what running flake8 in the CLI does, in which case we may need to change it?


Set displayname when getComposeComponent

Coverage increased (+0.5%) to 94.545% when pulling ee7b02c83138b0810be1cf2c2e5cb3c0d078c900 on fix-displayname into f5e77314a6f3d733126a23c767c2da6ec6f6080b on master.


Support RNPM, CocoaPods, react-native > 0.25

Supercedes #66, #74

Add support for CocoaPods (Closes #49)
Add support for rnpm (Closes #59)
Update demo to support react-native >=0.25 (Closes #69, Closes #71)

I've done some pretty exhaustive testing of the different installation options, but I would love some additional feedback.
@JAStanton I pulled in your changes to the podspec (other than the name), and your commits to remove AppDelegate from the library files.  Hope that's cool.
@nevir Could you comment on this implementation and whether or not it could cause any problems like those mentioned in #66?

Yeah, looks good from my side!  Just a couple smallish comments

@nevir Thanks for the input!  I was following the original implementation from Orientation.m, which was the cause of the confusion.  The following example works fine, and I've updated the README and the demo to match.  Thanks, again!
#import "Orientation.h" // <--- import

@implementation AppDelegate

  // ...

  - (UIInterfaceOrientationMask)application:(UIApplication *)application supportedInterfaceOrientationsForWindow:(UIWindow *)window {
    return [Orientation getOrientation];
  }

@end

This is super helpful thanks! 👍

@yamill Glad I could help!
PS, if you're looking for additional maintainers for this project, I'd be happy to jump in.

@mcrumm awesome! Thanks for doing this, and totally cool you pulled in my changes!

Just fyi, this change broke my project's setup (due to the moving files into the ios/ folder) so I had to limit the version range to patch upgrades until I can sit down and fix it. I'm not sure of npm's definition of "minor change", perhaps a change that doesn't affect JS code qualifies. If not then maybe this should have been a major version bump?


docs block usage outside of description field in YAML fails to render ('doc' is undefined)

Describe the bug
If I try to reference a docs block outside of a description field, in this case, a parameter needed for dbt-external-tables, LOCATION, I get this error.
Failed to render models/source/src_schema.yml from project mart_ml: Compilation Error
  Could not render {{ doc("munge_headcount") }}: 'doc' is undefined

Steps To Reproduce
models/source/blob_paths.md
{% docs munge_headcount %}
N'/azureml/f4a1a8bd-f0c3-424d-a2ed-d1273ab6bdcb/munged_hc/HeadcountMunged.tsv'
{% enddocs %}

models/source/src_schema.yml
version: 2
sources:
  - name: ret_mng
    schema: munged
    description: blahblah
    tables:
      - name: headcount
        description: |
          the output file from `munge_headcount.py`
          blob path: {{ doc("munge_headcount") }}  # <- this works
        external:
          data_source: AttritionPipeline
          FILE_FORMAT: TabDelimited
          LOCATION: '{{ doc("munge_headcount") }}' # <- this does not
        columns: ...

Expected behavior
I expect it to render and for the doc function to be found?
Screenshots and log output
stacktrace
 Failed to render models/source/src_schema.yml from project mart_ml: Compilation Error
    Could not render {{ doc("munge_headcount") }}: 'doc' is undefined
2021-04-22 18:22:16.416451 (MainThread): Traceback (most recent call last):
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/dbt/clients/jinja.py", line 501, in catch_jinja
    yield
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/dbt/clients/jinja.py", line 533, in render_template
    return template.render(ctx)
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/dbt/clients/jinja.py", line 187, in render
    return self.environment.handle_exception()
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/jinja2/environment.py", line 832, in handle_exception
    reraise(*rewrite_traceback_stack(source=source))
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/jinja2/_compat.py", line 28, in reraise
    raise value.with_traceback(tb)
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/dbt/clients/jinja.py", line 138, in quoted_native_concat
    head = list(islice(nodes, 2))
  File "<template>", line 1, in top-level template code
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/jinja2/sandbox.py", line 460, in call
    if not __self.is_safe_callable(__obj):
  File "/Users/anders.swanson/opt/miniconda3/envs/dbtenv/lib/python3.6/site-packages/jinja2/sandbox.py", line 360, in is_safe_callable
    getattr(obj, "unsafe_callable", False) or getattr(obj, "alters_data", False)
jinja2.exceptions.UndefinedError: 'doc' is undefined

System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[x] other (specify: Azure Synapse)

The output of dbt --version:
installed version: 0.19.1
   latest version: 0.19.1

Up to date!

Plugins:
  - sqlserver: <IP_ADDRESS>
  - synapse: <IP_ADDRESS>

The operating system you're using: macOS
The output of python --version: Python 3.6.12 :: Anaconda, Inc.
Additional context
Perhaps I'm misappropriating the intended purpose of doc() and that this is really a better use case for var()?
Update
If I had a variable to dbt_project.yml
vars:
  headcount_path: '{{ doc(munge_headcount) }}'

then this will render properly...
LOCATION: '{{ var("headcount_path") }}'

@swanderz Appreciate your thought process here! I agree with your assessment at the end:

Perhaps I'm misappropriating the intended purpose of doc() and that this is really a better use case for var()?

The intended purpose behind docs blocks is that dbt'ers can write Markdown code, in Markdown files, and pass it into any description fields they want. They're not intended a way to share information across different types of properties.
By contrast, vars are usable almost anywhere. As soon as you want to share a value beyond description fields, vars are a reasonable way to do it. So I think this will actually work, no docs blocks needed:
vars:
  headcount_path: "N'/azureml/f4a1a8bd-f0c3-424d-a2ed-d1273ab6bdcb/munged_hc/HeadcountMunged.tsv'"

version: 2
sources:
  - name: ret_mng
    schema: munged
    description: blahblah
    tables:
      - name: headcount
        description: |
          the output file from `munge_headcount.py`
          blob path: {{ var("munge_headcount") }}  # <- this works
        external:
          data_source: AttritionPipeline
          FILE_FORMAT: TabDelimited
          LOCATION: '{{ var("munge_headcount") }}' # <- this does not
        columns: ...

Just be advised that vars must take literal values—string literals, numeric literals, or dict/array literals. They are not Jinja rendered. So if you set a var to:
vars:
  headcount_path: '{{ doc(munge_headcount) }}'

The value of the headcount_path var will be the string literal {{ doc(munge_headcount) }}. Those aren't Jinja curlies, they're just the characters {{ and }}.

ok cool. I figured as much. Perhaps this change would prevent another newb like me from misconstruing doc()? We never say that doc() is only intended for the description key.
https://github.com/fishtown-analytics/docs.getdbt.com/pull/647


about the architecture of network

why did you use the bilinearupsanpling2D you wrote instead of the upsample2D layers from master keras?

Because the Upsampling2D layer uses tf.image.resize_nearest_neighbor(X, new_shape) to upsample the feature map, but we need bilinear upsampling.


mcount_return_trampoline: Don't disable interrupts

This fixes #8 for me.

Huh yeah, seems I missed that one. Now wondering how it ever worked?
The latest commit was done a few months after my thesis was complete and I was cleaning up my working directory, might have not tested it fully?
If i recall correctly, the reason for the changes in the latest commit were some changes in libhermit which lead to frequent crashes (tracing interrupt-based filesystem writes?). Something with interrupt at inopportune moments, trashing the stack/flags register.

Ah, I see. That makes sense. I have only tested with the rust example, and that one always crashes for me on the latest commit without this fix, as explained in #8.

Ah, nevermind, I did not test this change correctly. I'll rework this.

Ah, sorry. I must have been not very concentrated. Anyway, I found the issue now. The stack has been handled correctly.
The problem is calling cli in user space, which is not possible. When you tested this in libhermit you might have tested kernel space only. Removing cli makes the example work for me. Why would we need to disable interrupts there in the first place? Any interrupt handler would make sure not to clobber any registers, wouldn't it?

I think one issue I tried to fix was that an interrupt was arriving while the kernel was in mcount(). The interrupt in turn called mcount(), which then overwrote the return pointer of the first mcount() with the trampoline, which broke things.

Ah, so then the problem is the access to RETSTACK? I was trying to reproduce the issue to be sure, but I can't get RustyHermit to build with rftrace.


dropping store, external labels are not unique

Thanos, Prometheus and Golang version used
Thanos, version 0.1.0 (branch: master, revision: 0f15be67b6a51b4b86c25d12271c90986684b2cf)
Prometheus: 2.5
Go: 1.10

What happened
i got an error like "dropping store, external labels are not unique"
What you expected to happen
I used the k8s file prometheus.yaml in kube
seems that this env is not right
       replica: '$(HOSTNAME)'
How to reproduce it (as minimally and precisely as possible):
None
Full logs to relevant components

Logs

level=debug ts=2018-12-18T08:28:58.079368885Z caller=stdlib.go:89 component=cluster caller="peers2018/12/18 08:28" msg=":58 [DEBUG] memberlist: Initiating push/pull sync with: <IP_ADDRESS>:10900"
level=warn ts=2018-12-18T08:29:02.243258145Z caller=storeset.go:222 component=storeset msg="dropping store, external labels are not unique" address=<IP_ADDRESS>:10901
level=warn ts=2018-12-18T08:29:02.243310781Z caller=storeset.go:222 component=storeset msg="dropping store, external labels are not unique" address=<IP_ADDRESS>:10901
level=warn ts=2018-12-18T08:29:07.243061016Z caller=storeset.go:222 component=storeset msg="dropping store, external labels are not unique" address=<IP_ADDRESS>:10901
level=warn ts=2018-12-18T08:29:07.243116069Z caller=storeset.go:222 component=storeset msg="dropping store, external labels are not unique" address=<IP_ADDRESS>:10901
level=warn ts=2018-12-18T08:29:12.243430343Z caller=storeset.go:222 component=storeset msg="dropping store, external labels are not unique" address=<IP_ADDRESS>:10901
level=warn ts=2018-12-18T08:29:12.243482038Z caller=storeset.go:222 component=storeset msg="dropping store, external labels are not unique" address=<IP_ADDRESS>:10901

**Anything else we need to know**
None
<!--
Uncomment and fill if you use not casual environment or if it might be relevant.
<p>Environment:

OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Others:

-->

it just warning level

And that is correct behaviour right?
You don't want to ship this data to Thanos as this will cause you overlaps.

data is right,after i read the doc twice i figured it out


A lot of "(not set)" errors

Hello everyone, lately I have been reciving a lot of errors from google analitycs about "(not set)" pages. I am using @nuxtjs/google-gtag 1.0.4 and this is my config:
modules: [
    [
      '@nuxtjs/google-gtag',
      {
        id: process.env.GOOGLE_ANALYTICS_ID,
        config: {
          send_page_view: false // if true, double hit is sent
        }
      }
    ]
  ]

My events are simple as well
this.$gtag('event', 'play', {
    event_category: 'welcome_video',
    event_label: '[play icon]'
})

Is it a problem with nuxtjs/google-gtag library or am I wrong?

Recently I found the new repository for the library https://github.com/nuxt-community/gtm-module. Consider adding the link in the description anyway.

@MackYoel is the gtm-module the replacement for google-gtag? Are you using library instead? I noticed in the gtm-module you it says to put id: GTM-XXXXXXX, what is this id? Is it the adwords ID AW-XXXXXXXXX?


[Request] Ability to customize top level metadata

What would you like to be added:
Users are looking to create SBOMs that meet the NTIA Minimum Fields requirements. Users typically do this through a two step process.

Generate an SBOM with syft
Augment it with additional top level data

Who the top level component supplier is
What license
etc ...

This second step often requires a lot of custom scripting with jq, python or using a tool like sbomasm.
This feature request is to include a method to specify user defined top level metadata as options when calling syft or through a configuration file.
The files proposed are ...

Document

author
supplier
lifecycle

Top Level Component

author
supplier
name
version
description
copyright
repository
cpe
purl
hash
license

Why is this needed:
Most of the fields required for NTIA Minimum Field compliance need to be defined by the creator of the software product and there is no easy way for syft to discover their values. Allowing users to specify these field in a syft call reduces the steps required to create a "complete" SBOM.
Additional context:
This is a recommendation that originates from the CISA SBOM Generation Reference Implementation Tiger Team. Today https://github.com/interlynk-io/sbomasm is being used in the reference implementations.
CC/ @joshbressers

This is related to: #3098

I spoke with @wagoodman about this and we think a reasonable path forward is the following:

Add a pkg.Package to the source.Description, which is used to fill in any additional top-level package data when output in other formats; there will be overlap with the source-name and source-version, which were effectively setting these values on the package (I recommend deprecating the existing source.name/version, and setting both with --source-name/version flags)
To enable #3098, specifically, add a new pkg.Supplier struct with direct reference on the pkg.Package to it
We would support setting any top-level package elements here, not anything that ends up in the metadata sections

We believe that the values provided to configure this package have a moderate likelihood to be dynamic, such as the --source-version and being configurable from the CLI is important rather than only via configuration file / environment variables.

Some other suggestions for the CLI
Add many unique flags:
```
syft --source-name asdf --source-supplier-type org --source-supplier-name blah --source-supplier-url https://... myimage:latest
```
Use JSON:
syft --source-name asdf --source-package '{"supplier":"{"type":"org","name":"blah"}}' https://... myimage:latest

Multiple args:
syft --source-license id=MIT --source-supplier org=name --source-supplier url=https://...

... my preferred idea is to consolidate based on an object structure to a format like <field>(.<field>)*=<value>(,<field>(.<field>)*=<value>)*. For example:
syft --source-package name=asdf,supplier.type=org,supplier.name=blah,supplier.url=https://... myimage:latest

There will likely be some overlap with the existing name/version options e.g. these may result in predominantly the same thing:
syft --source-name asdf --source-package name=asdf


Module not found: Error: Can't resolve './check'

ERROR in ./packages/BaseInput/index.tsx
07:04:03 PM | Module not found: Error: Can't resolve './check' in '/zeit/53f9802/dashboard-ui-kit/packages/BaseInput'

ERROR in ./packages/BaseInput/index.tsx
07:04:03 PM | Module not found: Error: Can't resolve './check' in '/zeit/53f9802/dashboard-ui-kit/packages/BaseInput'

Thanks for the report. Can you try upgrading your package? It was a casing bug.. :)

problem solved


Support cp something ~/.bin

~/repos/shell$ cp target/release/shell.exe ~/bin
cp: could not copy target/release/shell.exe to ~/bin: The system cannot find the path specified. (os error 3)

The ~/bin does not work.

Same with ls and also launching programs using ~/bin/shell.exe

I think this relates to the idea of pre-processing the ~ string and expanding it to $HOME.

Note that there is no $HOME on Windows, but shell already knows how to get it via deno's dirs::home_dir(), which is a package.

This now works, it was fixed in https://github.com/prefix-dev/shell/pull/85.


proxy support

A production server I use has outgoing connections firewalled, and we have to use a proxy.  This plugin needs to be able to connect via the proxy for geocoding requests, similar to https://rails.lighthouseapp.com/projects/8994/tickets/2133-activeresource-http-proxy-support

#126 has code attached


FastReport Community Report Designer 2024.1.0 cannot Run

Describe the bug
The Designer can open an existing Report but when try to run the report the following error appears.
Error CS1705: Assembly FastReport, Version=2<IP_ADDRESS>, Culture=nutral, PublicToken=xxxx users FastReport,Compat, Version2<IP_ADDRESS> xxxx....
To Reproduce

Steps to reproduce the behavior:
Download the FastReport Designer
Open an existing Report
Click Run

Expected behavior
Runs report without error.
Screenshots
If applicable, add screenshots to help explain your problem.

Device (please complete the following information):

OS:  Windows 10
Browser  chrome
Version latest

Additional context
Add any other context about the problem here.

Good morning,
I have the same problem reported when opening the Issue, but in my case, it is occurring when a Script is created in the Code tab.
Thanks

For me it was working until I updated FastReport source code and recompiled, my guess is that after recompiling it updated some of the assemblies causing the issue.

This is happening to me too.  Is there any resolution?

Update:  This seems to happen only when I add the 1st Calculated Column.  Can anyone help?

Hello!
Thank you for reporting this problem! We will try to fix it as soon as possible.

@Yamasiash, if this is closed does that mean that the issue is fixed? If so, will there be a new download? Thanks!
(sorry for the duplicate, forgot the @)


Add Lightning Button/Toggle

Motivation:
Make it easier to find lightning enabled tools/services
Ambition:
You go to bitcoin-only, click on a page (i.e. wallets) and see everything exactly like you do now except there is a toggle switch in the corner which has two options (on and off) and a lightning logo next to it. By default this is set to off (lightning logo is grey)
You toggle the switch to on, the lightning logo is now yellow and all wallets on the page support lightning (because setting the toggle to on hides all wallets that don't support lightning)
Implementation:
I believe this can be done using an id=lightning HTML tag & some JavaScript but will need to have a closer look.

Nice implementation:
https://dev.to/ananyaneogi/create-a-dark-light-mode-switch-with-css-variables-34l8


Added restart ssh session directions to setup steps

People reading through these steps that are unfamiliar with bash/SSH will not understand that ansible will not be available until the session is restarted.
I don't know if the language I've added is the best, but thought I'd open a PR rather than an issue to make discussion easier.

Thanks! This should help a lot of people; I added a follow-up commit to move the text into parens on the same line.


How bind left alt + h j k l to arrows in fedora gnome

Help please

[ids]
*
[main]
[alt]
h = left
j = down
k = up
l = right

In the future, please mention what you've tried and where the manual was unclear :)


sorbet: Use Tapioca more since srb rbi is in maintenance

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

I was reading about modern Sorbet and found that srb rbi is in maintenance mode. The Sorbet project recommend using Tapioca. They even made a migration guide. It wasn't that hard to follow given we already use Tapioca in some places.
According to the migration guide, hidden-definitions isn't necessary any more. There's also another gem called Spoom for bumping typed: sigils that's recommended over srb --suggest-typed. For consistency, I've kept the brew typecheck flags the same for now, but it's a bit of a misnomer since Spoom actually does make the changes that we'll have to commit and push.
After all these changes, brew typecheck comes back with no new or existing errors, and we're further along with modern Sorbet tooling adoption.

Review period will end on 2022-10-12 at 00:11:15 UTC.

According to the migration guide, hidden-definitions isn't necessary any more.

I'm tad sceptical given we've not gone fully typed yet. You haven't actually removed the file so try do that and see if it's still no errors.

Review period ended.


feat: add secret manager version rotation support

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: C8mKfk066PnlSkAOei2n


feat(generator): implement Hann window function

See #37

It's quite fun to play around with this
test.zip


Apply static logger

This applies static logger.

Make all class var and method as static.
Remove logger param being passed around and import
Remove deprecated methods
Classes now have logging tag

While doing the above, one method appendLine was added into Logger
because ToolRunner needs both append and appendLine.
For #662
ONE-vscode-DCO-1.0-Signed-off-by: Hyun Sik Yoon<EMAIL_ADDRESS>


[Feature Request]:

Describe the feature / enhancement and how it would improve things
@bogdanfinn  hello sir would you compile this and release a windows executable file .exe for windows 10/11
YOu release a DLL file for windows , i tried rename the .dll to .exe it doesn't work
Describe how your proposal will work, with code and/or pseudo-code
@bogdanfinn  hello sir would you compile this and release a windows executable file .exe for windows 10/11
YOu release a DLL file for windows , i tried rename the .dll to .exe it doesn't work

This is a library, to use it as a standalone app, use the tls-client-api, which has .exe build available. It is available here; tls-client-api

@LyonSquad please stop spamming the issues with your personal requests. this is not what they are meant to be.


Don't gensym return_type

Just a suggestion, and a rather inconsequential one at that---but there doesn't appear to be any need for a gensymmed name for the return type variable. Using a plain symbol here slightly improves the readability of the outputs of @macroexpand, @code_warntype, et cetera.

Oh I see why it's needed now, to avoid conflicts with the actual function body. A bit too quick on the trigger there, sorry.


Admission Controllers?

It seems that PodSecurityPolicy isn't enabled by default.  I believe this means anyone can use things like hostNetwork, hostPath, privileged, without any restrictions?
https://kubernetes.io/docs/admin/admission-controllers/#podsecuritypolicy
It would be useful to be able to tweak admission controllers or at least enable that one by default?

One of the benefits of using kubernetes-incubator/bootkube is that we get those changes when they're added upstream. Track the work being done on this in https://github.com/kubernetes-incubator/bootkube/issues/782


Add simulation time to FrameData for ASESimulation

This PR adds the time elapsed during the simulation to the FrameData delivered to the user during an ASE simulation using the ASESimulation class. To do so requires a conversion from the internal units of ASE to femtoseconds.

We should probably add this to OMM too if it's not too difficult to include it in this request?

We should probably add this to OMM too if it's not too difficult to include it in this request?

I believe OMM already delivers this information, this is to bring ASE in line with OMM


java.time.Instant support

When using the Spring JDBC implementation, if using the CRUD interface, Instant is handled properly:
jdbcTemplateCrud =
        JdbcTemplateMapperFactory.newInstance()
            .crud(modelClass, idClass)
            .lazilyTo(jdbcTemplate.getJdbcOperations(), tableName);

jdbcTemplateCrud.create(entity, entity::setId);

But, when using a standard Spring JDBC Template query and the provided sql parameter source:
sqlParameterSourceFactory =
        JdbcTemplateMapperFactory.newInstance().newSqlParameterSourceFactory(modelClass);

KeyHolder keyHolder = new GeneratedKeyHolder();

jdbcTemplate.update(
      insert(dataColumnNames),
      sqlParameterSourceFactory.newSqlParameterSource(entity),
      keyHolder);

I get the following exception from MySQL:
com.mysql.cj.jdbc.exceptions.MysqlDataTruncation: Data truncation: Incorrect datetime value: '\xAC\xED\x00\x05sr\x00\x0Djava.time.Ser\x95]\x84\xBA\x1B"H\xB2\x0C\x00\x00xpw\x0D\x02\x00\x00\x00\x00Zs\xABC+\xDEx\x00x' for column 'last_modified_date' at row 1

Do I need to do anything special to map Instant type parameters when not using the CRUD?
As I understand it, the included SqlParameterSourceImpl is more performant than Spring JDBC's BeanPropertySqlParameterSource as noted http://simpleflatmapper.org/0104-getting-started-springjdbc.html. I originally had an extended version of BeanPropertySqlParameterSource that handled Instant and Enum, will I need to do the same for SqlParameterSourceImpl or can I leverage the handling that the CRUD interface already includes?

I tried setting manually specifying the SqlType of my Instant columns as follows:
JdbcTemplateMapperFactory jdbcTemplateMapperFactory = JdbcTemplateMapperFactory.newInstance();
      jdbcTemplateMapperFactory.addColumnProperty(
          "created_date", SqlTypeColumnProperty.of(Types.TIMESTAMP));
      jdbcTemplateMapperFactory.addColumnProperty(
          "last_modified_date", SqlTypeColumnProperty.of(Types.TIMESTAMP));

SqlParameterSource sqlParameterSource = jdbcTemplateMapperFactory
      .newSqlParameterSourceFactory(modelClass)
      .newSqlParameterSource(entity);

System.out.println(sqlParameterSource.getSqlType("created_date"));

It doesn't seem to propagate through to the resultant SqlParameterSource. The SqlType on the last line is marked as UNKNOWN.

Thanks for the big report will check asap

so I'm using spring javaTypeToSqlParameterType to get the sqlType from the java class and it does not recognise Instant.
also it does not get into account SqlTypeColumnProperty which it should will fix all that.
should be able to release that soon

got a fix for instant also added conversion, as I had assumed that spring would deal with it, but does not really. will be able to release it tonight GMT

Thanks @arnaudroger for looking into it. I'll give it a shot after it's released.
FYI, Spring might not handle Enums properly either? I haven't tried this, but from my experience with plain Spring JDBC, I had to add special handling for both Instant and Enum types.

Just put 3.15.5 to maven central, should be available in the next 30mn.
The converter should take care of the Enum, you might need to specify the sqltype though, so it can choose the ordinal or string version.


Uploader styling improvements + proper upload queue

Codecov Report
Attention: Patch coverage is 0% with 45 lines in your changes are missing coverage. Please review.

Project coverage is 3.01%. Comparing base (258ddb0) to head (41a5bea).

Files
Patch %
Lines

server/components/Uploader.tsx
0.00%
45 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #268       +/-   ##
==========================================
- Coverage   22.55%   3.01%   -19.55%
==========================================
  Files          95      88        -7
  Lines        3595    2520     -1075
  Branches      565     523       -42
==========================================
- Hits          811      76      -735
+ Misses       2784    2444      -340

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


Move config file text to end of help output (#207).

Sorry, had to revert this since unittests stopped working:
https://travis-ci.org/github/bw2/ConfigArgParse/builds/759318560
If you'd like to resubmit, please makes  sure you can successfully run
python -m unittest tests.test_configargparse
python3 -m unittest tests.test_configargparse


Test against Python 3.10

Tests are passing in 3.10 now: https://github.com/simonw/datasette-graphql/actions/runs/1472851804


Explicitly tell the frontend to enable default cross-module-optimization

So far, the swift-frontend decided by itself if CMO can be enabled. This caused problems when used with an old driver, which doesn't consider CMO.
Now, the driver decides when to use default CMO.
Depends on the swift changes https://github.com/apple/swift/pull/58529

@swift-ci test

@swift-ci test


Allow rule suppression

PSSA2 should allow the suppression of rules:

Current suppression attributes should be supported
Comment-based suppression should also be supported and work based on token reading
Ideally suppression should occur before rules create or emit a record to minimise overhead. This is most like easiest by making diagnostic emission use a method that can filter out diagnostics
In particular, it may be useful to pre-process input and find excluded extents rather than each rule performing the same suppression discovery over and over

Suppression should be possible directly in a script and at current-level only. What if you only want to ignore the usage of e.g. Write-Host called at the start or end of the script, but not inside the functions in the script?
Currently requires placing the code inside a unnecessary function just to suppress that one instance. Or you could add an unnecessary param() block at the beginning of the script for the suppression-attribute to work on at "root"-level, but this also enables the suppression for all functions in the script as well AFAIK. Possible to improve that behavior? Or even better, provide line-based suppression (related #849)?

@fflaten what you're talking about is what I refer to as comment-based suppression above.
Something like:
#|pssa:disable:PS/AvoidUsingWriteHost
Write-Host "Hello"
#|pssa:enable:PS/AvoidUsingWriteHost


don't reinitialize the interpreter when it is shutting down

Related to #3386.
If auto-intialize is enabled in an extension-module, we can hit a nasty C crash on interpreter shutdown where auto-initialize attempts to re-initialize it. I added an internal mark_python_initialized_from_import to disable auto-initalize when a #[pymodule] is imported (as the interpreter must be initialized at that point).
Second I add an assertion that the interpreter is initialized before attempting to acquire the GIL, to avoid odd crashes after the interpreter has been shut down (and just panic).

I added an internal mark_python_initialized_from_import to disable auto-initalize when a #[pymodule] is imported (as the interpreter must be initialized at that point).

Since our own START can only protect us from multiple threads calling prepare_freethreaded_python, I thought it would be best effort in any case? Meaning that shouldn't prepare_freethreaded_python itself somehow be able to detect if we in shutdown on its own? Put differently, I would not want us to rely on the state of START beyond its original intention of curbing multiple threads racing initialization.

Agreed, I will scrap this. START will be consumed anyway if with_gil is called at any other point during program execution. The example from the discussion was an edge case that we don't need to cater for.


[TS] bleed property on elements

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
TS implementation of #2109

Implemented with #2300


cannot narrow empty

[@jvasileff] This function (which is suspect, but valid) doesn't compile:
Iterable<Element, Absent> emptyify<Element, Absent>
        (Iterable<Element, Absent> stream)
        given Absent satisfies Null {
    if (stream.empty) {
        assert (is Iterable<Element, Absent> empty); // error
        return empty;
    }
    else {
        return stream;
    }
}

$ ceylon compile
source/simple/run.ceylon:5: error: cannot find symbol
        assert (is Iterable<Element, Absent> empty);
                ^
  symbol:   variable empty
  location: class emptyify_
ceylon compile: There was 1 error

[Migrated from ceylon/ceylon-compiler#2417]

I've taken a quick look at this one and I think it shouldn't be too hard to fix.
I think the StatementTransformer.IsCond class should make sure that a temporary variable gets created, but the heuristics for doing that seem to fail in this particular case,

Especially there's this code snippet:
        boolean useTempVar = !var.isErasedToObjectOptimization() && !var.isNothingOptimization();

that get evaluated to false because var.isErasedToObjectOptimization() returns true, which I don't think it should, but I have no (logical / proper) idea on how to fix it.
Perhaps you have an idea @tombentley ? If not I'll move it to 1.3

Workaround is:
assert (is Iterable<Element, Absent> e = empty);

so this surely must be very trivial to fix.
Why was this assigned to 1.4??


scripts/upgrade-zulip-from-git: Don't cache the repo URL, only the contents

We document the deployment.git_repo_url setting in /etc/zulip/zulip.conf
to control where this script fetches from, and don't say that it's
only read on the first such upgrade and cached thereafter.  The documented
behavior seems like the right behavior.  So use the currently configured
URL every time, by writing it anew into the config of our cache repo.

@timabbott, probably best for you to review when you're back. (There's no rush to land this.)
I haven't yet tested this change at all. Do we have any tools or infrastructure for a "dev prod" environment in which to test this kind of change? If not, making a crude one (and over time a less-crude one) probably goes on my agenda.

Automated message from Dropbox CLA bot
@gnprice, it looks like you've already signed the Dropbox CLA. Thanks!

Codecov Report

Merging #5510 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5510      +/-   ##
==========================================
+ Coverage   86.65%   86.69%   +0.03%
==========================================
  Files         432      432
  Lines       53568    53568
  Branches     2090     2090
==========================================
+ Hits        46418    46439      +21
+ Misses       7150     7129      -21

Impacted Files
Coverage Δ

zerver/tornado/ioloop_logging.py
93.33% <0%> (+46.66%)
:arrow_up:

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 df83cee...78b4c70. Read the comment docs.

Hi @gnprice! I'm not aware of a specific prod-testing environment, I've been using a separate VM for it. I'd be interested to hear ideas for creating one.
I tried your change (from your branch, install a tarball then upgrade) and I got an error. There was a long pause at npm install before the exception. I restarted the upgrade and got the same result. I hope this information is useful.
2017-06-25 15:41:10,325 upgrade-zulip-stage-2: Building static assets...
Cached version not found! Installing node modules.
+ rm -rf /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
+ mkdir -p /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
+ cp package.json /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
+ npm install --production --prefix /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c

Error running a subcommand of ./tools/update-prod-static: npm install --production --prefix /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
Actual error output for the subcommand is just above this.

Traceback (most recent call last):
  File "./tools/update-prod-static", line 38, in <module>
    setup_node_modules(npm_args=['--production'], stdout=fp, stderr=fp)
  File "./tools/../scripts/lib/node_cache.py", line 43, in setup_node_modules
    copy_modules=copy_modules)
  File "./tools/../scripts/lib/node_cache.py", line 71, in do_npm_install
    run(cmd, stdout=stdout, stderr=stderr)
  File "./tools/../scripts/lib/zulip_tools.py", line 98, in run
    subprocess.check_call(args, **kwargs)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['npm', 'install', '--production', '--prefix', '/srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c']' returned non-zero exit status 1
Traceback (most recent call last):
  File "/home/zulip/deployments/2017-06-25-15-40-31/scripts/lib/upgrade-zulip-stage-2", line 82, in <module>
    preexec_fn=su_to_zulip)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['./tools/update-prod-static', '--authors-not-required', '--prev-deploy', '/home/zulip/deployments/current']' returned non-zero exit status 1
Traceback (most recent call last):
  File "/home/zulip/deployments/current/scripts/upgrade-zulip-from-git", line 81, in <module>
    deploy_path, "--from-git"] + deploy_options)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '/home/zulip/deployments/2017-06-25-15-40-31/scripts/lib/upgrade-zulip-stage-2', '/home/zulip/deployments/2017-06-25-15-40-31', '--from-git']' returned non-zero exit status 1

This implementation looks correct.  I think for something like this, I would just test it on chat.zulip.org, but I think it would be worth having a more "test only" deployment, e.g. test.zulip.org, that isn't really used and can be used for testing more disruptive upgrades.
Want to create such a thing @gnprice?

Thanks @feorlen for testing!
Can you try upgrade-zulip-from-git on master and confirm that that works, and that when you switch to this PR's version you still reproduce the issue you saw? That is a puzzling issue -- it's hard for me to see why this change would cause it. So I'd like to see if it's a pre-existing issue, in which case I'll merge this and then we can debug that separately.
I just tried this version of the script on chat.zulip.org, and it worked as expected. It got to use a cache for the node modules, though, so it didn't run through the codepath that @feorlen ran into. I'll retry now forcing it to miss that cache.

OK, I tried it with the npm install actually running (by removing the empty file /srv/zulip-npm-cache/657d2dbff31ec62024ba61f1b687a74c3e1ee7ff/node_modules/.success-stamp first), and it still worked. The npm install took about a minute, but succeeded.
So I'm going to optimistically merge this, but I'm still very interested in debugging and fixing the issue @feorlen hit (and if it does turn out to be related to this change, then oops.)
@feorlen: Would you take a look at /home/zulip/deployments/*/var/log/update-prod-static.log? Specifically the var/log/update-prod-static.log that's inside the deployments/* directory created by one of your attempted upgrades that hit that bug. The stdout and stderr of the npm install should have gone there, so it probably has some clues. If you'd just post that whole file, say as a gist at https://gist.github.com/ , that'd be very helpful.

@timabbott And I've just filed #5611 to make a test instance as you suggested.

@gnprice I'll try my test again and see what happens

I tried the same test, using master, and it failed.
2017-07-01 02:48:14,882 upgrade-zulip-stage-2: Building static assets...
Cached version not found! Installing node modules.
+ rm -rf /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
+ mkdir -p /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
+ cp package.json /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
+ npm install --production --prefix /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c

Error running a subcommand of ./tools/update-prod-static: npm install --production --prefix /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c
Actual error output for the subcommand is just above this.

Traceback (most recent call last):
  File "./tools/update-prod-static", line 38, in <module>
    setup_node_modules(npm_args=['--production'], stdout=fp, stderr=fp)
  File "./tools/../scripts/lib/node_cache.py", line 43, in setup_node_modules
    copy_modules=copy_modules)
  File "./tools/../scripts/lib/node_cache.py", line 71, in do_npm_install
    run(cmd, stdout=stdout, stderr=stderr)
  File "./tools/../scripts/lib/zulip_tools.py", line 118, in run
    subprocess.check_call(args, **kwargs)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['npm', 'install', '--production', '--prefix', '/srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c']' returned non-zero exit status 1
Traceback (most recent call last):
  File "/home/zulip/deployments/2017-07-01-02-47-43/scripts/lib/upgrade-zulip-stage-2", line 82, in <module>
    preexec_fn=su_to_zulip)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['./tools/update-prod-static', '--authors-not-required', '--prev-deploy', '/home/zulip/deployments/current']' returned non-zero exit status 1
Traceback (most recent call last):
  File "/home/zulip/deployments/current/scripts/upgrade-zulip-from-git", line 81, in <module>
    deploy_path, "--from-git"] + deploy_options)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '/home/zulip/deployments/2017-07-01-02-47-43/scripts/lib/upgrade-zulip-stage-2', '/home/zulip/deployments/2017-07-01-02-47-43', '--from-git']' returned non-zero exit status 1

Here is the end of update-prod-static.log
entire file at https://gist.github.com/feorlen/44dad4ac7ad7b45ef019398b8bb471cf
npm http 304 https://registry.npmjs.org/vector-tile
npm WARN deprecated<EMAIL_ADDRESS>This module is now under the @mapbox namespace: install @mapbox/whoots-js instead
npm WARN deprecated<EMAIL_ADDRESS>This module has moved: please install @mapbox/vector-tile instead
npm http 200 https://registry.npmjs.org/earcut
npm http GET https://registry.npmjs.org/amdefine
npm http 404 https://registry.npmjs.org/mapbox-gl-shaders
npm http 304 https://registry.npmjs.org/amdefine
npm ERR! TypeError: Object.keys called on non-object
npm ERR!     at Function.keys (native)
npm ERR!     at installTargetsError (/usr/share/npm/lib/cache.js:708:24)
npm ERR!     at /usr/share/npm/lib/cache.js:638:10
npm ERR!     at saved (/usr/share/npm/node_modules/npm-registry-client/lib/get.js:142:7)
npm ERR!     at /usr/lib/nodejs/graceful-fs/polyfills.js:133:7
npm ERR!     at Object.oncomplete (fs.js:107:15)
npm ERR! If you need help, you may report this log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>
npm ERR!  Additional logging details can be found in:
npm ERR!     /home/zulip/deployments/2017-07-01-02-47-43/npm-debug.log
npm ERR! not ok code 0

and npm-debug.log
172729 info installOne<EMAIL_ADDRESS>172730 info installOne<EMAIL_ADDRESS>172731 info installOne<EMAIL_ADDRESS>172732 info /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c/node_modules/plotly.js/node_modules/gl-plot3d/node_modules/glslify/node_modules/static-module/node_modules/static-eval/node_modules/escodegen/node_modules/source-map unbuild
172733 info /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c/node_modules/plotly.js/node_modules/gl-plot3d/node_modules/glslify/node_modules/static-module/node_modules/static-eval/node_modules/escodegen/node_modules/esprima unbuild
172734 info /srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c/node_modules/plotly.js/node_modules/gl-plot3d/node_modules/glslify/node_modules/static-module/node_modules/static-eval/node_modules/escodegen/node_modules/estraverse unbuild
172735 silly lockFile ca1239b9-er2-node-modules-readable-stream tar:///srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c/node_modules/plotly.js/node_modules/gl-plot3d/node_modules/glslify/node_modules/static-module/node_modules/duplexer2/node_modules/readable-stream
172736 silly lockFile ca1239b9-er2-node-modules-readable-stream tar:///srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c/node_modules/plotly.js/node_modules/gl-plot3d/node_modules/glslify/node_modules/static-module/node_modules/duplexer2/node_modules/readable-stream
172737 error TypeError: Object.keys called on non-object
172737 error     at Function.keys (native)
172737 error     at installTargetsError (/usr/share/npm/lib/cache.js:708:24)
172737 error     at /usr/share/npm/lib/cache.js:638:10
172737 error     at saved (/usr/share/npm/node_modules/npm-registry-client/lib/get.js:142:7)
172737 error     at /usr/lib/nodejs/graceful-fs/polyfills.js:133:7
172737 error     at Object.oncomplete (fs.js:107:15)
172738 error If you need help, you may report this log at:
172738 error     <http://github.com/isaacs/npm/issues>
172738 error or email it to:
172738 error<EMAIL_ADDRESS>172739 error System Linux 4.4.0-78-generic
172740 error command "/usr/bin/nodejs" "/usr/bin/npm" "install" "--production" "--prefix" "/srv/zulip-npm-cache/716364d3d1b55eb050ddfd8e801b29891151945c"
172741 error cwd /home/zulip/deployments/2017-07-01-02-47-43
172742 error node -v v0.10.25
172743 error npm -v 1.3.10
172744 error type called_on_non_object
172745 verbose exit [ 1, true ]


Implementation of defense with some dat type classes

Implemetation of the defeneses. The tests couldn't be carried out due to the lack of implementation of other classes, for example: Enemy

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9c71afa). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##             master       #4   +/-   ##
=========================================
  Coverage          ?   12.19%
  Complexity        ?        5
=========================================
  Files             ?       16
  Lines             ?       82
  Branches          ?        3
=========================================
  Hits              ?       10
  Misses            ?       72
  Partials          ?        0

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


Wasm builds fail

Thanks for making this template, but when I try to follow the steps to build for web on a fresh clone of this repo, it fails with the following output:

Finished dev [optimized + debuginfo] target(s) in 2m 01s
Running wasm-server-runner target/wasm32-unknown-unknown/debug/bevy_game.wasm
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', /home/pixelartist/.cargo/registry/src/github.com-1ecc6299db9ec823/wasm-bindgen-cli-support-0.2.79/src/descriptor.rs:208:15

I get the same error when I try to use Github Actions to build this template from tags. It seems like an issue with the wasm-bindgen step?
However, I am able to build the standard Bevy examples to web just fine.
Is anyone else running into this issue? I'm running the latest version of Pop OS (Linux) and am able to build the template for Linux just fine, just not for web.

This seems to be an issue with the latest version of wasm-bindgen-cli.
See rustwasm/wasm-bindgen#2776

Everything works fine for me on 0.2.78
I pinned the crate to this version in the makefile for now. Let me know if that resolves the issue for you, too.

Hey, great template, thanks very much for your hard work!
Getting the same issue as descirbed above, building on OSX 12.1:
Finished dev [optimized + debuginfo] target(s) in 2m 24s
     Running `wasm-server-runner target/wasm32-unknown-unknown/debug/bevy_game.wasm`
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', /Users/freddywordingham/.cargo/registry/src/github.com-1ecc6299db9ec823/wasm-bindgen-cli-support-0.2.79/src/descriptor.rs:208:15
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Building and running natively works no problem

I also tested it on 0.2.79 and it works fine there, too.

Do you still get this issue using the new web setup with trunk?

With the new setup the web workflow should have improved a lot.
I'll close this for now. If you still run into issues with the new setup, feel free to open a new issue :slightly_smiling_face:


Hover Tips sometimes get stuck

Sometimes the hover tips get frozen and won't go away until restart Atom.  See screenshot example where I've already toggled out of git blame and closed all files.

thanks for the report. this is a dupe of #170. I haven't had time to properly debug this quite yet


Rebuild rust lib

I'm interested in trying out a few changes in rust backend and repackage it into automerge-swift. It appears running cargo build --release generates a single universal static lib. How would I go about generating libautomerge.a for simulator and catalyst? Thank you!

Great for bringing this up.
At the moment I working on a build script which install all required components and builds all required architectures. But I will not be able to finish it until next year.
In the meantime you could just use the resulting libautomerge.a and copy it into AutomergeRSBackend.xcframework/macos-x86_64. This would replace the current binary with a newer one. By doing so you can try out the changes in the rust backend. But be aware that this workaround only works on macOS x86_64.

👍 thanks!

@zeeyang I pushed an initial repo to build & distribute  automerge-rs as xcframework. Have look here


How to disable ?

I want to disable rating on one of my page

The disable feature has not been implemented in react-animated-rating v1.0.6. Will add it to the next release.

When new release will come ?

@codingwithashu will try to make the next release by this Friday.

Great thanks

@codingwithashu added the disable functionality. Please check the latest release v1.1.0. You can refer to the demo for reference on how to use the disabled prop.


Remove ColumnDefExt trait in data::Schema,

ast::ColumnDef is in the same crate so it is not necessary to expand using extra trait. Replace ColumnDefExt to use adding direct impl to ast::ColumnDef.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 98.423%

Files with Coverage Reduction
New Missed Lines
%

core/src/executor/aggregate/mod.rs
2
85.56%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
39269

Relevant Lines:
39898

💛  - Coveralls


file.d: fix unsafe use of .ptr

endless cases of these.
Blocks https://github.com/dlang/dmd/pull/5860

core.exception.RangeError@std\file.d(370): Range violation

@9il fixed

Auto-merge toggled on


ArgoCDAddon: Admin password name uses default profile only

Describe the bug
I passed cdk deploy --profile someprofile, and everything worked, except ArgoCDAddon tried to use the default profile to get the secret manager key instead of profile.
Expected Behavior
--profile something
gets the secret using the credentials from the profile account
Current Behavior
used the credentials from the default account, instead of the one I specified, wasting a bunch of time trying to figure out if it was a permission, or format issue with the name.
Reproduction Steps
const prodBootstrapArgo = new ArgoCDAddOn({
adminPasswordSecretName: 'argocd-admin-eks-password',
bootstrapRepo: {
...bootstrapRepo,
path: 'envs/prod',
}
});
Put the secret in a different profile then default, and run cdk deploy with the different profile
Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
2.85.0 (build 4e0d726)
EKS Blueprints Version
1.9.1
Node.js Version
v18.16.1
Environment details (OS name and version, etc.)
Mac M1 Max, Ventura 13.4 (22F66)
Other information
I have multiple profiles

@wrexbe We fixed it. Please see if this works so we can close.


'vapoursynth.VideoFrame' object is not subscriptable

Error on frame 15 request:
'vapoursynth.VideoFrame' object is not subscriptable
py3.6.4
vs.core.version:
VapourSynth Video Processing Library\nCopyright (c) 2012-2018 Fredrik Mellbin\nCore R44\nAPI R3.5\nOptions: -\n
torch.version:
1.10.0+cu111
vpy:
import vapoursynth as vs
import sys
sys.path.append("C:\C\Transcoding\VapourSynth\core64\plugins\Scripts")
import mvsfunc as mvf
sys.path.append(r"C:\Users\liujing\AppData\Local\Programs\Python\Python36\Lib\site-packages\vsrealesrgan")
from vsrealesrgan import RealESRGAN
core = vs.get_core(accept_lowercase=True)
source = core.ffms2.Source(sourcename)
source = mvf.ToRGB(source,depth=32)
source = RealESRGAN(source)
source= mvf.ToYUV(source,depth=16)
source.set_output()

Your VapourSynth version is too outdated. Try updating first.

Your VapourSynth version is too outdated. Try updating first.

I update py version to py3.9.5 now.
@HolyWu When I update the version of vs to R54, I meet the same question when processing "source = RealESRGAN(source)"

When I update it to >=R54-test2-API4 (R54-test2-API4,R55,R56,R57), error happen when processing "source = mvf.ToRGB(source,depth=32)":
File "C:\Program Files\Python39\lib\site-packages\mvsfunc.py", line 446, in ToRGB
clip = core.fmtc.matrix(clip, mat=matrix, fulls=fulls, fulld=fulld, col_fam=vs.RGB)
File "src\cython\vapoursynth.pyx", line 2531, in vapoursynth.Function.call
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc0 in position 0: invalid start byte
(But "source = RealESRGAN(source)" can be processed.)
mvsfunc.py is from https://github.com/HomeOfVapourSynthEvolution/mvsfunc/blob/master/mvsfunc.py, the latest version.
What can I do to process the whole pipeline?

I update the version of fmtconv.dll and fix the problem QAQ.
So these pytorch scripts for vs only supports version > R54, am I right?

Yes, the version requirement is specified here.


Can not use -S to pickup version for scala-cli repl

In the document,

When you want to control the Scala version, you can control it from the command line using the --scala option (with -S and --scala-version aliases):

But that does not work, not composiable .
C:\Users\hepin>scala-cli -S 3.3.1 repl
[error]  repl is not a C:\Program Files\scala-cli-x86_64-pc-win32\scala-cli.exe sub-command and it is not a valid path to an input file or directory.
Try viewing the relevant help to see the list of available sub-commands and options.
  C:\Program Files\scala-cli-x86_64-pc-win32\scala-cli.exe --help

and
C:\Users\hepin>scala-cli --scala 3.3.1 repl
[error]  repl is not a C:\Program Files\scala-cli-x86_64-pc-win32\scala-cli.exe sub-command and it is not a valid path to an input file or directory.
Try viewing the relevant help to see the list of available sub-commands and options.
  C:\Program Files\scala-cli-x86_64-pc-win32\scala-cli.exe --help

The --scala option belongs to the sub-commmands it can be used with, like repl, compile, run, etc.
The correct notation is to put the flag after the sub-command, so:
scala-cl repl -S 3.3.1

or
scala-cli repl --scala 3.3.1

Yes, I think this issue can be closed, but yes, very easy to wrong use:)


Rendering wrong update message

This should be rendering the update message with -g instead:

Gonna throw this here since it's kinda related. yarn global upgrade now somehow doesn't update now. But yarn global add now does. Seems like yarn's fault, but maybe change the message until it's fixed?

@erikras The yarn update message was already changed in a newer version, the version in your screenshot 15.8.5, showed indeed an incorrect command

This probably only happens with nvm, for everyone else it will display the correct update message.


Fixes for tests using enable_user_defined_functions

Some quick fixes after 2.1 -> 3.0 merge around the 'enable_user_defined_functions' configuration.
Tests that use this config option are failing with < 2.2 cassandra versions since C* doesn't allow unknown configs in cassandra.yaml.  Added a configVersionRequirements map to CCMTestsSupport that contains a listing of configurations and their version requirements.  If a config is found that requires a certain version and the version requirement can't be met, it is not used.  Not sure if this is the best approach, but does the job.
Also updated FunctionExecutionExceptionTest to use enable_user_defined_functions.
There may be more issues with the merge.  I noticed that the 3.0 C* version builds didn't run well, but 2 of the 3 failed because of what appeared to be CI issues, so there might not actually be a problem there.

Added a little commit with a dummy fix for a test failing because the query trace wasn't available, I hope you don't mind.
I opened #573 for fixes that apply to both 2.1 and 3.0; let's keep this pull request for fixes applying only to 3.0.

:+1:

I think this is good to merge, but think it would be good to wait until #573 is on 3.0

Squashed and rebased with #573 on 3.0.


Enhancement: reserve additional signature types

Current EOSIO reserves 3 signature types: the R and K curves, and webauthn.
If any third party project decides to introduce a new type of a signature, they would have to pick a value for the leading byte.
It will be great if core eosio team assigns these numbers in advance, in order to avoid future confusions. At least for the most common types, like RSA and some additional ECC curves.

Rk


⚠ ClamAV has degraded performance

In 9841bae, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 210 ms

Resolved: ClamAV performance has improved in 683a2a2.


OCPBUGS-38709: Join default HTTPS port to Spec.Configuration.Proxy.HTTPSProxy if missing

What this PR does / why we need it:

If a port is not provided by the user, kubernetes-default-proxy pod will fail as its expecting a host:port address. This checkes if the port is missing and joins the default 443 port to the HTTPS proxy address when passing it to the container.

Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/jira refresh

/lgtm


fix(learn, i18n): crowdin issues

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the main branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Resolves a couple of issues with Crowdin parsing, based on flagged strings by our translators.
Signed-off-by: nhcarrigan<EMAIL_ADDRESS>

In an effort to get Crowdin updated, I am merging this without a second review.

Thank you for your contribution! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Editing region (polygon) points

We need a method to add and delete points from a polygon region. This can be holding down "shift" while clicking to add a new point and holding down "ctrl" or "alt" while clicking to delete. On a tablet it would have to involve a different editing mode, e.g., "region (edit)" which does not add new regions but adds and deletes existing points. Delete is disabled if there are <= 3 points remaining.

How are polygon points added or deleted in tablet mode for an existing polygon?


Port Identity UI changes to scaffolding

Tracking for issues that require ports to scaffolding for Identity UI fixes
https://github.com/aspnet/AspNetCore/pull/12109
https://github.com/aspnet/AspNetCore/pull/11768

https://github.com/aspnet/Scaffolding/pull/1033


Removed name restrictions on Main

Documentation says only the [Main] attribute is needed. The code requires the name SharpmakeMain. The error messages refer to the method as SharpmakeMain but also say it must be named Main.
The attribute alone should be enough to find the main method regardless of name.

Hi!
Very sorry for the delay, and thank you very much for the merge request :)
As part of a bigger feature (commit #689f39c17c1a36a944535dc94706ffb4c1bb1b1d), we had to lift that restriction too, and your changes were very helpful :)
I'm gonna close this pull request.
Thanks again!


Feature request: Stylistic changes to the app design

Is your feature request related to a problem? Please describe.
As for me, the Receive page in the app is redundant, as it's just an embellishment and serves no function, but only confuses users, as if it suggests that files can only be received on this page. Many app use the separation of Send and Receive, but there it is structurally justified, because only on the Receive tab can you receive files. In this app, you can receive files from anywhere in the app. And so, to transfer files, you need one extra click. I understand that the Send tab is also used as a starting page where there is a logo, but perhaps it's still worth putting the Receive tab first and renaming it to Send/Receive, or something more concise, and renaming the current Send tab, for example, to Info, so as not to change anything too constructively and so that the app on mobile is not so poor with only two tabs. Perhaps, style it slightly differently.
It's also desirable to move the file transfer history to the Send/Receive page for quick access to it, given that there is quite a lot of free space on the page.
As an example, Snapdrop can be mentioned, which visually mimics AirDrop or Send Anywhere, which also use a combined page for receiving and sending files.
Describe the solution you'd like

Describe alternatives you've considered
No response
Additional context
No response

Thanks for your feedback. Please add comments to #300 instead of adding a new issue.

Thanks for your feedback. Please add comments to #300 instead of adding a new issue.

Sorry, I didn't notice that there's already an appropriate issue.


Add WAM error code documentation for TPM issues

Can the following errors be added to
https://learn.microsoft.com/en-us/entra/msal/dotnet/advanced/exceptions/wam-errors
"WAM Error    Error Code:<PHONE_NUMBER>   Error Message: Keyset does not exist   Internal Error Code: 545133655",
WAM Error    Error Code:<PHONE_NUMBER>   Error Message: The device that is required by this cryptographic provider is not ready for use.   Internal Error Code: 545133655",

Added as part of #302. Need feedback from our WAM engineer to move forward.


[Snyk] Upgrade joda-time:joda-time from 2.9.2 to 2.12.7

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade joda-time:joda-time from 2.9.2 to 2.12.7.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 33 versions ahead of your current version.
The recommended version was released 2 months ago, on 2024-02-04.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

A

A


Introduce RedirectEnvRestorer and VarEnvRestorer traits

This allows us to abstract over other redirect/variable restorer implementations which will be significant when we introduce built in commands (e.g. running exec < foo may not want to actually restore redirects)
Marked all methods directly defined on RedirectRestorer and VarRestorer as deprecated in favor of their respective new traits
Also made any applicable futures generic over the respective trait instead of a concrete implementation

Codecov Report

Merging #1 into master will decrease coverage by 0.02%.
The diff coverage is 90.54%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   89.27%   89.24%   -0.03%
==========================================
  Files         130      130
  Lines        8623     8657      +34
==========================================
+ Hits         7698     7726      +28
- Misses        925      931       +6

Impacted Files
Coverage Δ

tests/redirect_or_var_assig.rs
98.79% <100%> (ø)
:arrow_up:

src/spawn/local_redirections.rs
93.65% <100%> (+0.79%)
:arrow_up:

src/env/reversible_redirect.rs
82.92% <100%> (+2.37%)
:arrow_up:

src/eval/redirect_or_cmd_word.rs
90.54% <100%> (ø)
:arrow_up:

src/eval/redirect_or_var_assig.rs
90.36% <100%> (ø)
:arrow_up:

tests/reversible_redirect.rs
100% <100%> (ø)
:arrow_up:

tests/redirect_or_cmd_word.rs
100% <100%> (ø)
:arrow_up:

tests/reversible_var.rs
92.3% <100%> (+0.64%)
:arrow_up:

src/spawn/simple.rs
83.04% <75%> (+0.09%)
:arrow_up:

src/env/reversible_var.rs
87.23% <76.92%> (-12.77%)
:arrow_down:

... and 2 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 d157c74...77b1d8d. Read the comment docs.


Add code for AppAuthentication library

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Creating a new PR. @shahabhijeet, can you please cancel/ close the previous one? Thanks!

@azuresdkci retest this please

@varunsh-msft at this point. This library cannot be merged in the repo.
Continue to ship from the private repo you have.


The SQL stuff is filthy, just filthy.

Optimizations:
Rather than iterating per ID actually generate a mogrified query for white/black card cardset record insertion.  Use RETURNING id to do this.
Add RETURNING id to the card set insert query and just get rid of all that horseshit MAX(id) stuff.
Just general cleanliness of the db.py file.  Old dirty file is old and dirty.

Much cleaner now.