Loading into memory and processing

I have a use case, where I need to process the entire video. The transformation for each of the frames is different - adding an artifact to the original frame which is pre-computed. I want to read the entire video into a numpy array (these videos are short - less than 30 seconds) and I have 32GB memory host.
In fact, I have this accomplished, by writing the images to a directory and operating on the individual frames and write back. But this is time consuming - takes about a minute.
Here is the sample code:
    ffmpeg.input(inp_video).output('dir/%03d.bmp').run()

    for f in os.listdir(file_dir):
      if "bmp" in f:
           # Add a red block
	   image = cv2.imread(file_dir+'/'+f)
	   image[1:100,1:100,:] = [255,0,0]
	   cv2.imwrite(file_dir+'/'+f, image)

   ffmpeg.input('dir/%03d.bmp').output(out).overwrite_output().run()

Are there ways to fasten this process by loading the entire video into memory and operating?

@lravindr Have you found some faster ways to deal with this case ?

Unfortunately, I didn't find a way to fasten this process.

@lravindr
How about grabbing frames with cv2 instead of writing temporary images ?
Here's sample code. In my environment, I got about 3x faster improvement.
# ffmpeg-python issue 401 solution
# https://github.com/kkroening/ffmpeg-python/issues/401

import os
import sys
from functools import wraps
import logging
import time

import ffmpeg
import cv2

# =============================================================================
# Performance logger
# =============================================================================
time_logger = logging.getLogger("perf_measure")
time_logger.setLevel(level=logging.DEBUG)
_handler = logging.FileHandler(filename="perf_measurement.log")
time_logger.addHandler(_handler)

def time_it(func):
    """Time func"""

    @wraps(func)
    def wrapper(*args, **kargs):
        _start = time.perf_counter()
        _result = func(*args, **kargs)
        _elapsed_time = time.perf_counter() - _start
        _elapsed_time = round(_elapsed_time, 3)
        time_logger.info(
            f"Time It: {func.__name__} - processed {_elapsed_time} [s]")
        return _result

    return wrapper

# =============================================================================
# Main function
# =============================================================================

@time_it
def sample_func(temp_dir: str, input_path: str, output_path: str) -> None:
    """sample func from Iravindr
        https://github.com/kkroening/ffmpeg-python/issues/401#issue-665948504
    """
    print("export bmp")
    ffmpeg.input(input_path).output(f'{temp_dir}/%03d.bmp').run()

    print("write images")
    for f in os.listdir(temp_dir):
        if "bmp" in f:
            # Add a red block
            image = cv2.imread(temp_dir + '/' + f)
            image[1:100, 1:100, :] = [255, 0, 0]
            cv2.imwrite(temp_dir + '/' + f, image)

    print("output")
    ffmpeg.input(f'{temp_dir}/%03d.bmp').output(
        output_path).overwrite_output().run()

@time_it
def proposal_func(temp_dir: str, input_path: str, output_path: str) -> None:
    """proposal func with cv2 grabbing"""
    print("start processing")
    cap = cv2.VideoCapture(input_path)
    if not cap.isOpened():
        raise ValueError("Could not open video file")

    frame_width = int(cap.get(3))
    frame_height = int(cap.get(4))
    out = cv2.VideoWriter(output_path,
                          cv2.VideoWriter_fourcc('M', 'P', '4', 'V'), 30,
                          (frame_width, frame_height))

    while cap.isOpened():
        ret, frame = cap.read()
        if ret == True:
            # Add a red block
            frame[1:100, 1:100, :] = [255, 0, 0]
            out.write(frame)
        else:
            print("end of video")
            break

    cap.release()
    out.release()
    cv2.destroyAllWindows()

if __name__ == "__main__":
    temp_dir, input_path, output_path = sys.argv[1:]

    sample_func(temp_dir, input_path, output_path)
    output_path_proposal = output_path.replace(".mp4", "_2.mp4")
    proposal_func(temp_dir, input_path, output_path_proposal)

executing above code, then I got this log file.
# perf_measurement.log
Time It: sample_func - processed 8.013 [s]
Time It: proposal_func - processed 3.096 [s]

I used sample video whose duration is 30secs / fps is 30 / resolusion is 1280x720


az restore-disks to secondary region

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az backup restore restore-disks
Errors:
The command failed with an unexpected error. Here is the traceback:
'NoneType' object has no attribute 'properties'
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 657, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 712, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/backup/_exception_handler.py", line 11, in backup_exception_handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 691, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/backup/custom_base.py", line 416, in restore_disks
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/backup/custom.py", line 965, in restore_disks
AttributeError: 'NoneType' object has no attribute 'properties'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az backup restore restore-disks --container-name {} --item-name {} --rp-name {} --storage-account {} -v {} --use-secondary-region --target-resource-group {} -g {}

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.30.0

Extensions:
application-insights 0.1.14
automation 0.1.1
datafactory 0.5.0
log-analytics-solution 0.1.1

Additional Context

route to service team

Hi @jeremypope-valpak - thanks for reaching out. We are not able to reproduce the issue when we enter the correct parameter values. However, we were able to repro the error message when we provide an incorrect value for  rp-name parameter (eg. providing an rp name that doesn't exist).
Could you confirm that all the parameter values are correct? If possible, kindly share an example command.
In the future, we will make sure to add more descriptive error message for incorrect parameter scenarios

Hi,
I use “az backup recoverypoint list” with --use-secondary-region to list the available recovery points and then take one of the “name” values from the json output  – which is an 18 digit numeric value - as the –rp-name value.  Redacted command for the restore follows:
az backup restore restore-disks --container-name MYSERVER --item-name "VM;iaasvmcontainerv2;source-rg;MYSERVER " --rp-name  "8561670527120728668" -g source-recoveryvault -rg  -v source-recoveryvault  --use-secondary-region --target-resource-group target-rg  --storage-account crrstagingaccount
I can’t find a good definition in available documentation of what –rp-name should be, so I may have made the wrong choice.
I ran the same command using the id value from the JSON as –ids in the command and received a similar error:
option '--resource-group' will be ignored due to use of '--ids'.
option '--vault-name' will be ignored due to use of '--ids'.
option '--container-name' will be ignored due to use of '--ids'.
option '--item-name' will be ignored due to use of '--ids'.
Command group 'backup' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
The command failed with an unexpected error. Here is the traceback:
'NoneType' object has no attribute 'properties'
Traceback (most recent call last):
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 657, in execute
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 720, in _run_jobs_serially
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 712, in _run_job
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/backup/_exception_handler.py", line 11, in backup_exception_handler
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 691, in _run_job
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 328, in call
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/backup/custom_base.py", line 416, in restore_disks
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/backup/custom.py", line 965, in restore_disks
AttributeError: 'NoneType' object has no attribute 'properties'
Jeremy.
@.http://
Jeremy Pope | DevOps & Cloud Engineer
dd: tel<EMAIL_ADDRESS>tel:

t: 03450 682 572tel:03450 682572 valpak.co.ukhttp://www.valpak.co.uk
Unit 4, Stratford Business Park, Banbury Road, Stratford-upon-Avon CV37 7GW
@.https://twitter.com/valpak_ltd @. https://www.linkedin.com/company/valpak-limited @.*** https://www.facebook.com/ValpakLimitedUK/ @.*** https://www.youtube.com/channel/UCsEDdqEZVWZBJc2YziCMxSg
a Reconomy Group Company
@.***https://www.communigator.co.uk/login/Instances/valpaklz/Documents/valpakwhitepaper-ukplasticpackagingtax.pdf
Please consider the environment before printing
This email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you have received this email in error please notify the system manager. This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. If you are not the intended recipient, you are notified that disclosing, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited.
From: Aditya Balaji @.>
Sent: 30 November 2021 05:47
To: Azure/azure-cli @.>
Cc: Jeremy Pope @.>; Mention @.>
Subject: Re: [Azure/azure-cli] az restore-disks to secondary region (Issue #20523)
CAUTION: This email originated from outside of the organisation. Do not click links or open attachments unless you recognise the sender and are sure the content is safe.
Hi @jeremypope-valpakhttps://github.com/jeremypope-valpak - thanks for reaching out. We are not able to reproduce the issue when we enter the correct parameter values. However, we were able to repro the error message when we provide an incorrect value for rp-name parameter (eg. providing an rp name that doesn't exist).
Could you confirm that all the parameter values are correct? If possible, kindly share an example command.
In the future, we will make sure to add more descriptive error message for incorrect parameter scenarios
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-cli/issues/20523#issuecomment-982307427, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARILS5G5BKMIOA5GU7HSK53UORQNDANCNFSM5I3CTUMA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi @jeremypope-valpak. Unfortunately, we are still not able to reproduce the issue. Could you please provide us with the debug logs of both the command(az backup recoverypoint list and az backup restore restore-disks) so that we can investigate if it is a client-side issue or a service-side issue?
For getting debug logs you just need to run the commands with --debug flag. Sorry for the inconvenience. Please reach out in case of any clarification. Thanks.

Hi @jeremypope-valpak - just wanted to check back on the below, whether you were still facing this issue. If so, could you help share the debug logs for the same?

Hi, the issue was resolved by Microsoft -  turned out to be an issue with with the underlying service, rather than the AZ CLI.
Thank you,


Allow modification of Rides

Let the driver manually add and remove people from their car. That way  I can remove freshmen I do not like from my car.

:+1:

:+1:


fix: use more safe method to detect ci environments

NODE_ENV=CI is an unusual way to detect CI environments. This will not work on most environments.
I have chosen the same package as husky for compatibility reasons, although there are better maintained packages such as https://www.npmjs.com/package/@npmcli/ci-detect

Thanks for your contribution @ernscht! 🎉


Multiple issues: edit the model to add a diagram

@thomaskonrad reports:
The fact that I have to edit the model to add a diagram: That seems counter-intuitive to me. I'd put the diagrams into the edit view of the model, and only put metadata into the edit dialog.

@thomaskonrad : Moving this issue to repo owasp-threat-dragon-core because this is the repo that will have any code changes for this issue


Upgrade storybook to resolve Dependabot warning

https://storybook.js.org/docs/migration-guide
For now I just dismissed the warning.

Seems like it got resolved another way.


Scheduler perf

Coverage remained the same at 91.631% when pulling c30946c3498c90daa7b8cbb989939ecff0386281 on scheduler-perf into 92c186a3090b1e7de087d66e28ddd76d728e717e on master.


[TableHeader] Can't unselect all when enableSelectAll={true}

When enableSelectAll is enabled, the TableHeader checkbox can be used to enable all rows, but doesn't disable all when clicked.

Steps to reproduce:

Go to the docs site: Components / Table / Complex example
Turn on Multi-Selectable and Enable Select All
Click the header checkbox (all items selected)
Click it again. Nada.

FYI, I've been diving into area of code as a result of an unrelated issue. What I've discovered is, if I remove the following code inside of table-body.jsx > componentWillReceiveProps, this behavior is no longer broken.
if (this.props.allRowsSelected && !nextProps.allRowsSelected) {
  newState.selectedRows = this.state.selectedRows.length > 0
    ? [this.state.selectedRows[this.state.selectedRows.length - 1]]
    : [];
} else {
  newState.selectedRows = this._calculatePreselectedRows(nextProps);
}

New to MUI (and react) so I don't necessarily have the context required to know how to fix this, or any 2nd or 3rd order effects, quite yet.

Hit the same issue. Any plans on fixing it?

I have the same problem. I am unable to unselect all rows using main checkbox in header.

Sorry this never got fixed.
Table has already been migrated to (and of I'm not mistaken completely rewritten for) next and so should be resolved there, so I'm closing this for now.


Develop 2

I have changed my mind about how it would function over and over but this is the final outcome. Please let me know if the code can be more efficient. I feel I could make the eventListeners on the animal images more efficient by adding a universal function like createPet(); which would include the display rules but I can't seem to get it to work. Thanks!

This is very well thought out, I like that you used psudocode and planned out what you were going to build before you build it.
Your code looks clean and easy to understand, I've not run this code personally but it reads well. I can test it tomorrow over lunch but if it works feel free to merge it :)

You didn't need to do this today! But thank you very much. Appreciate it :-)


PKGBUILD file for ArchUserRepo submission

As someone suggested in this issue, I've added the PKGBUILD file and necessary for submitting to the Arch User Repository.
    Alternatively, if you will merge PR, users will be able to just makepkg -ci inside the cloned directory and build the package locally followed by a pacman -U installation, though, since cargo is a make dependency and they will have already cloned the repo, it is of questionable use as they could just take the cargo install route.
    Just wanted to check in with you regarding the project building and everything before submitting the file to AUR. As per check() doc, this function could be removed. Since this a release package, this function currently serves to locally check that you did not release a broken version. I would remove it as it decreases installation time with little to no dangers, let me know what you think.

Thanks, actually cargo is not a dependency. if it's not there, some functionality won't work like automatic install of cargo plugins.

ok, i see what i'm missing. cargo is used by the pkgbuild.

@CosminGGeorgescu sorry for late reply.
Many things in these files are hardcoded (hashes, version numbers) and would require manual work to setup.
won't it be better to setup some script to update a template and generate these?
this library uses cargo-make so it could be a task there (gen-pckbuild) that i can latter easily hook into my release flow

Many things in these files are hardcoded (hashes, version numbers) and would require manual work to setup.
won't it be better to setup some script to update a template and generate these?

That's how they should be. The maintainer (me) will take care of updating them every time you publish a new release. Just give me a few days to take care of an exam and I will finish everything.

@CosminGGeorgescu this is still a bit strange to me. I've got no experience with pkgbuild files, but seems like this could be a template and a script to automate the updating of the template (via cargo-make preferred).
relying on you to update on each new release (thanks) doesn't scale and best to have instructions (script) to do that. no?
also do you submit this anywhere?

also do you submit this anywhere?

https://aur.archlinux.org/packages/duckscript .
Here it is, sorry for the delay.
Now, if there will be enough people that find this useful and vote for it, it will reach the official Archlinux package database and people will be able to install it using pacman.

but seems like this could be a template and a script to automate the updating of the template

Yes, it is and it's called updpkgsums as the checksums are the only variables that should be updated automatically.

doesn't scale and best to have instructions (script) to do that. no?

I'm completely with you on this one, but it's just a case of idiomatic thing. Normally maintainers have a small script to update the pkgfiles and, as this is my first package maintain, I see no reason to unnecessarily add to your cargo-make something that would take me 3 mins tops by hand.

Think I'll close this PR in a few days as there is nothing more to do. If you've anything more to add, let me know. I'll send a new PR next week for the install. Hopefully, now more people will able to reach your project and if it'll be popular enough with Arch folk, it will reach the official package database. Thank you for everything !

the only thing 'im missing (we talked about it) is a script or something that creates the file with the checksums.
or at least, put some instructions at the top of the PKGBUILD file in a comment


upcoming: [M3-7608]: Placement Groups Landing page empty state

Description 📝
Empty state UI on the landing page for the upcoming Placement Groups feature.
Changes  🔄

Introduces a unit test for the landing page empty state.

Preview 📷

Light Mode
Dark Mode

Unit Test

How to test 🧪
Prerequisites

Follow these steps to test the changes in this PR before Alban's PG Landing Page PR is merged.

Checkout the Alban's draft PR
Pull the changes from this PR down.
Open the serverHandlers.ts file and change line no. 1875 as follows: return res(ctx.json(makeResourcePage(placementGroupFactory.buildList(0))));
Run the project and turn on the MSW.

If testing after Alban's PG Landing Page PR is merged, only follow steps 3 and 4, listed above.

Verification steps

Run the unit test yarn test path/to/cloud-manager/manager/packages/manager/src/features/PlacementGroups/PlacementGroupsLanding/PlacementGroupsLanding.test.tsx
Visually inspect the UI for any defects.

As an Author I have considered 🤔

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

This PR is dependent on this PR. In order to commit my changes in the current state, I had to bypass the husky pre-commit hook. Changes will need to be verified before merging this PR.


Square brackets in folder name do not work

Info

Operating System: Arch

Shell: zsh

Terminal: kitty

nvim --version:  v0.9.0

fzf --version:  0.27.2

[X] The issue is reproducible with mini.sh

Steps to reproduce

Create a folder named [something]
Create a file inside it with any name
Run :lua require('fzf-lua').files({ cwd = '/path/to/[something]' })
You won't see any files inside of the directory

Investigation
After a bit of debugging it seems like the issue is caused by this function: https://github.com/ibhagwan/fzf-lua/blob/main/lua/fzf-lua/core.lua#L572-L574
Running str_to_str("/path/to/[something]") outputs [[/path/to/[something\]]] which seems incorrect to me. In addition in looks like fd isn't expecting escaped square brackets because deleting the gsub entirely seems to solve this specific problem.

https://github.com/ibhagwan/fzf-lua/commit/e1046726cf0cedf15ae0ff40537a421b79968c3e
@WolfeCub - should work now

Oh, and ty @WolfeCub for the investigation, saved me a bit of time :smile:

Amazing! Thanks for the quick turnaround :)


error: The package product 'Introspect' requires minimum platform version 10.15 for the macOS platform, but this target supports 10.13 (in target 'ScalingHeaderScrollView' from project 'ScalingHeaderScrollView')

When select My Mac, and build failed
How to fix this?
error: The package product 'Introspect' requires minimum platform version 10.15 for the macOS platform, but this target supports 10.13 (in target 'ScalingHeaderScrollView' from project 'ScalingHeaderScrollView')

Hey @webserveis, sorry this package is not supposed to support macOs in the first place. All fixed now, thanks for bringing this to our attention. Have a great day!


Secure license usage and update renovate config

coverage: 93.75%. remained the same
when pulling 22ec15f96ca95041f130ae4a92110368496f047a on hazendaz:master
into d60cd5d6e422d3a548f2f93a918af809a3d5288c on mybatis:master.


🐛 BUG: Unbound workers don’t respect subrequest limit in --local mode

What version of Wrangler are you using?
2.0.24
What operating system are you using?
Linux
Describe the Bug
Miniflare recently updated to support new subrequest limits for Unbound workers (cloudflare/miniflare#274), but it requires the caller to pass usageModel. In --local mode, wrangler dev doesn’t support the new subrequest limits because it’s not passing the parameter.
I tested this locally, and passing unbound to the compiled Miniflare CLI results in the correct behaviour.
I’ll put together a PR to read this from wrangler.toml and pass it through.

Appreciate the work!! 🎉


Possibility to use with byt5-small ?

Hi, XL model way too large for me! I tried byt5-small but crash:
Traceback (most recent call last):
  File "/home/nuck/discore/src_core/lib/corelib.py", line 79, in invoke_safe
    func(*kargs, **kwargs)
  File "/home/nuck/discore/src_core/classes/printlib.py", line 193, in wrapper
    return func(*args, **kwargs)
  File "/home/nuck/discore/src_core/rendering/renderer.py", line 106, in emit
    cb(v, name)
  File "/home/nuck/discore/sessions/pinchsky-dance/script.py", line 101, in callback
    paella.txt2img(seed=v.seed, steps=v.steps, chg=clamp(v.chg, 0.1, 1), w=v.w, h=v.h, cfg=v.cfg, p=v.prompt, negprompt=v.negprompt, sampler=v.sampler)
  File "/home/nuck/discore/src_plugins/paella/PaellaPlugin.py", line 47, in txt2img
    ret = colab.txt2img(prompt, steps=steps, size=size, cfg=cfg, seed=seed)
  File "/home/nuck/discore/src_plugins/paella/colab.py", line 219, in txt2img
    sampled_tokens, intermediate = sample(model,
  File "/home/nuck/discore/src_plugins/paella/colab.py", line 66, in sample
    logits = model(sampled, t, **model_inputs, attn_weights=attn_weights)
  File "/home/nuck/discore/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/nuck/discore/plug-repos/paella/Paella/utils/modules.py", line 273, in forward
    c_embed = self.gen_c_embeddings(byt5, clip, clip_image)
  File "/home/nuck/discore/plug-repos/paella/Paella/utils/modules.py", line 224, in gen_c_embeddings
    seq = self.byt5_mapper(byt5)
  File "/home/nuck/discore/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/nuck/discore/venv/lib/python3.10/site-packages/torch/nn/modules/linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
RuntimeError: mat1 and mat2 shapes cannot be multiplied (1x1472 and 2560x1024)
mat1 and mat2 shapes cannot be multiplied (1x1472 and 2560x1024)

Unfortunately, Paella requires at least 30GB of memory since we trained on big conditoning models. We will change that in future models.

So not even on RTX 3090? That's a pretty huge restriction if I do say so

Definitely. Paella is not a finished research project and we are still working on improving on many things. One thing that you can do is just not load t5 at all and enable t5=False in the sampling script. Then you can also do del clip.visual and then it should work probably. Let me know if that works for you!

I tried but I think there are references to t5 stuff even outside the checks for t5, and there seem to be more complications in the Paella model with the byt5_mapper

I managed to use Paella on colab using ~10GB of VRAM by simply using the CLIP image model to create variations, and deleting the T5 model and the prior model (I could also delete the text clip encoder).
If you have a rtx 3090 I think you can try loading the byT5-XL model alone, save the embeddings, delete the model and load the rest (if it doesn't work, try loading the model in half precision).

I managed to use Paella on colab using ~10GB of VRAM by simply using the CLIP image model to create variations, and deleting the T5 model and the prior model (I could also delete the text clip encoder).
If you have a rtx 3090 I think you can try loading the byT5-XL model alone, save the embeddings, delete the model and load the rest (if it doesn't work, try loading the model in half precision).

Can you share your code for image variations? Thank you.


Added .editorconfig

Nicked from https://github.com/dotnet/corefx
An attempt at preventing this (indent_style = space, indent_size = 4) and this (trim_trailing_whitespace = true) in the future :wink:

What does this https://github.com/NancyFx/Nancy/pull/2035/files#diff-1e70daafb475c0ce3fef7d2728279182R11 do? Not add an extra blank line at the end of the file I hope?
Also, are all the Xml related settings compatible with the default indentation? Don't want it to affect all web/app.config etc files that we already have in our solution

What does this https://github.com/NancyFx/Nancy/pull/2035/files#diff-1e70daafb475c0ce3fef7d2728279182R11 do? Not add an extra blank line at the end of the file I hope?

Yes. That's exactly what it does. Some background:
http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline
http://stackoverflow.com/questions/5813311/no-newline-at-end-of-file
https://robots.thoughtbot.com/no-newline-at-end-of-file

Also, are all the Xml related settings compatible with the default indentation?

Probably not, but it's not as if this magically goes through all config files and changes them. This will make sure the indentation is changed correctly over time.

You and your "I'll make perfect sense` personality ;)


purefb_open_events_total metric is not found in pure-fb-om-exporter:0.9.0 version

purefb_open_events_total metric is not found in pure-fb-om-exporter:0.9.0 version
Is it replaced by purefb_hardware_health metric?

It has been replaced by the purefa_alerts_open metric in this specific exporter (FlashArray), and by purefb_alerts_open for FlashBlade.


内存泄漏

+(instancetype)reachabilityWithHostname:(NSString*)hostname
{
SCNetworkReachabilityRef ref = SCNetworkReachabilityCreateWithName(NULL, [hostname UTF8String]);
if (ref)
{
id reachability = [[self alloc] initWithReachabilityRef:ref];
    return reachability;
}

return nil;

}
Reachability/Reachability.m:98:36: Call to function 'SCNetworkReachabilityCreateWithName' returns a Core Foundation object with a +1 retain count
Reachability/Reachability.m:99:9: Assuming 'ref' is non-null
Reachability/Reachability.m:101:9: Object leaked: object allocated and stored into 'ref' is not referenced later in this execution path and has a retain count of +1

Duplicate of #114


Support for puppet using gcloud CLI

When installing the plugin on puppetserver, it fails on installing grpc dependencies on JRuby - here is the similar thread: https://github.com/grpc/grpc/issues/6705
As a workaround, you can use gcloud CLI to communicate with GKMS and encrypt/decrypt values with hiera-eyaml.
The requirement is to have gcloud CLI installed and perform authorization.

@craigwatson thanks for your input. I will try to hide this behind the option per your suggestions.


Adding bit coin rainbow chart

I want to add "bit coin rainbow chart"

I want to add "bit coin rainbow chart"

Oky do it


Can't in any way make a Release version .exe from my bot

How do you even do this?

For general C# help, it would be better suited for the C# Discord http://aka.ms/csharp-discord


Allow different resampling filters

Allow all possible resampling from Pillow for flexibility and/or user's preference.
Kept Image.LANCZOS as default.

Thansk @silverkorn for this PR.


Add Fool to Tarot in tables.py

I am also open to removing INVALID and making _missing_ return FOOL, or removing both INVALID and _missing_ and considering the enum complete, a la StatType
I'm not sure if maintaining Tarot.INVALID is necessary, or if that was just added this way to support the (then presumably placeholder) values of the Coffee Cup teams. I'm also not even sure it's fair to say that a team like e.g. The Data Witches have the Fool as a tarot card, or even whether any of this will be relevant ever again with the current Reader v Coin happenings.
All that said, -1 is not an invalid value for the tarot attribute, all breach teams have -1 as their tarot value and the Fool as their card.

This is technically a breaking change, will merge it with the other breaking changes in a sec


Update gcloud maven plugin version

Switch to java 8
Add HTTPS everywhere tag to web.xml
Switch to app.yaml

LGTM

LGTM


07 feature/refactor encryption decryption

Proposed changes
Refactored encryption and decryption methods into one class.
Added docstrings to better describe functions.
Updated tests
Types of changes
What types of changes does your code introduce to Bannin?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add support to authenticate to GKE clusters using authentication plug-ins or tokens.

Description
Add support for requesting Kubeconfig with and without and authentication provider when creating a GKE cluster.
Blast Radius
None since tests use a pinned version
Dependencies and documentation
Documentation

[x] I have updated user facing documentation.
[ ] I have updated all Runbooks affected by this change.
[ ]  I updated DEVELOPING.md with any dev tricks I used to work on this code efficiently.
[ ] My changes do not have any impact on documentation.

Monitoring

[ ] I have verified that any changes to alerts work as expected.
[ ] My changes affect monitoring rules and/or dashboards, and I made sure they didn't break.
[x] My changes do not have any impact on monitoring.

External dependencies

[ ] I have validated that dependencies impacted by this change work as expected.
[x] My changes do not impact external dependencies.

Rosie the Robot checklists

[ ] My changes affect one or more Rosie checklists.
[x] My changes do not have any impact on Rosie's checklists.

Testing

[x] I have validated that my changes work as expected.
[ ] My changes do not require testing.

Testing Strategy
Used test workflows
Security

[x] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

Related Issues or corrective actions
Deployment plan
N/A

@AidanWenzel I pushed a couple of changes to fix the build failures. Please take a look again


question about making datasets for semantic-nerfw

I recently find it's hard to get datasets properly for semantic-nerfw through reading https://github.com/ethanweber/sitcoms3D. I tried to make custom datasets by using ns-process-data, however, I find the generated image_4 and image_8 lost their original names(turned into frame**) resulting in error of ns-train(no such directory: image_4).
Also, I don't know how to generate semantic_4 and semantic_8.
Hope to see official documents about making semantic-nerfw datasets!

Hi @cezarbbb, you can look at my comment here in Issue #1275. We don't plan to prioritize semantic documentation/support at the moment, but we are very open to contributions! The idea would to modify the nerfstudio data parser to have semantic support (much like we do for the masks).
The semantic masks are produced with custom detectron2 code, but you could use any semantic segmentation model to create segmentation images.


Failed to load model with error: Unknown model file format version.

I'm submitting a…
Bug report (I searched for similar issues and did not find one)
Current behavior

I have created and trained a model using pytorch and then converted it to onnx. But while using it with WinML I am getting this error.
"Failed to load model with error: Unknown model file format version."
The error message is not much descriptive so I am not sure what is going wrong here.
I will attach the script that I used to create the mode.
Attachments.zip
Could you help me resolve this issue?
Expected behavior
The model should be loaded properly with out any error
Minimal reproduction of the problem with instructions
Use winMLRunner.exe to test the model, it basically fails to Load and Bind
Environment
Windows Build Number: 19041.876 (10.0.19041.0)
Visual Studio

[x] 2017 (version: )

Hi,
Please refer to the details here regarding opset support in Windows Machine Learning:
https://docs.microsoft.com/en-us/windows/ai/windows-ml/release-notes
Note that for build 19041 (Windows 10, version 2004)
Support for ONNX 1.4 and opset 9 (CPU and GPU)
I looked at your model and notice that your model is marked as requiring opset 10 and file format version 6.
19041 only has support for file format version 5 and opset 9.
The NuGet package currently has the support you need: https://www.nuget.org/packages/Microsoft.AI.MachineLearning/
Will this work?

Hi @smk2007  I am using this Windows SDK version(19041) and I meet the same question too.
I tried the "ONNX=1.10 and opset=9" combination, but it failed. Is that necessary to ensure both the version of "ONNX=1.4 and opset=9" at same time?
Sorry that I have less knowledge about this, what's the meaning of the "file format version" in your previous answer?
Could you give some suggestions to me?
thx


os-lifecycle-policy.md: Fixed typo

Also added a year annotation for the Fedora note. (In general, when speaking about "later this year" etc, it's always best to be explicit with which year you are talking about, since documentation otherwise grows old faster than necessary.)

@leecow


Autoupdating via Github Pages

Is your feature request related to a problem? Please describe.
I love your theme, but I wonder why there is no auto-updates using imports and Github Pages? Doing this saves the user lots of time and checking.
Describe the solution you'd like
Make it so the scss compiles under a file on the repo and use imports to automatically update your theme.

I remember there was a metro for discord entry in the theme repo plugin. I was surprised I haven't found one for fluent
.
Would love to see it as well! Anyway, amazing theme, love your work :)

This is implemented now: Download from https://github.com/TakosThings/Fluent-Discord/releases/latest
Note that Fluent-Discord-static.theme.css doesn't auto update.

Hell yeee lets gooooo 🎉🎉


Merge development branch into main

Summary of changes
missionPlan acted as the development branch for the navigation code and logic (despite the misleading branch name). Merging into main.
Checklist before Requesting Review

[x] Self-Review: I've reviewed my changes.
[x] Unit Tests: If applicable (and it almost always is for Flight Termination), I've added relevant and meaningful unit tests.
[x] Test Status: All unit tests are passing locally (run python -m unittest discover).

We do not need to include training datasets for the model.

Leaving these datasets in the repository until we move them to Google Drive.


Aurelia does not log an error if a processContent( ) functor throws an exception.

Fairly self-explanatory. I had an exception in my processContent functor, and Aurelia did not log anything to the browser console to indicate the call had failed.
This in turn seems to destroy the whole view loading process within Aurelia as nothing gets displayed.

@kevmeister68 We can add some logging around this scenario, no problem. But, shouldn't it still fail? What would be the alternative for the component if it's content process logic fails?

But, shouldn't it still fail?

I guess so - probably just debugging took too much time without any hints(errors) from browser console.


useBoundAction hook

New Features
What is the new or updated feature that you are suggesting?
It would be nice to incorporate the mapDispatchToProps functionality into a new hook/hooks
For example, current functionality:
import { useCallback } from 'react';
import { useDispatch } from 'react-redux';

const incrementActionCreator = ...;

const MyComponent = () => {
  const dispatch = useDispatch();
  const increment = useCallback(() => dispatch(incrementActionCreator()), []);
}

This could be condensed to:
import { useBoundAction } from 'react-redux';

const incrementActionCreator = ...;

const MyComponent = () => {
  const increment = useBoundAction(incrementActionCreator);
}

Why should this feature be included?
This feature is offered with the current connect HOC when you provide an object to the mapDispatchToProps argument.  Increasing parity between hooks and hocs increases the versatility of the library.
It also is convenient because it removes a lot of boilerplate making the code more terse and increasing readability.
What docs changes are needed to explain this?
A new documentation section describing the new hook and its usage.
Suggested implementation
Looking at useDispatch, it seems like a simple implementation could be done like this:
import { ReactReduxContext } from '../components/Context'
import { useDispatch as useDefaultDispatch, createDispatchHook } from './useDispatch'

/**
 * Hook factory, which creates bound actions to the dispatch bound by a given context.
 *
 * @param {Function} [actionCreator] Action creator to wrap with dispatch`<Provider>`.
 * @param {React.Context} [context=ReactReduxContext] Context passed to your `<Provider>`.
 * @returns {Function} A `useBoundAction` hook bound to the specified context.
 */
export function createBoundActionHook(actionCreator, context = ReactReduxContext) {
  const useDispatch =
    context === ReactReduxContext ? useDefaultDispatch : createDispatchHook(context)

  return function boundAction() {
    const dispatch = useDispatch()
    const action = actionCreator.call(this, arguments)
    return dispatch(action)
  }
}

/**
 * A hook to bind an action creator to the dispatch function.
 *
 * @returns {function} action bound to redux store's `dispatch` function
 *
 * @example
 *
 * import React, { useCallback } from 'react'
 * import { useBoundAction } from 'react-redux'
 *
 * const setCounterActionCreator = count => ({
 *   type: 'set-counter',
 *   payload: {
 *     count
 *   }
 * })
 *
 * export const CounterComponent = ({ value }) => {
 *   const setCounter = useBoundAction(setCounterActionCreator);
 *   return (
 *

 *       {value}
 *       <button onClick={() => setCounter(0)}>Reset counter</button>
 *

 *   )
 * }
 */
export const useBoundAction = /*#__PURE__*/ createBoundActionHook()

We've already decided against including this kind of hook before. Here's Dan's comment that sums it up pretty well: https://github.com/reduxjs/react-redux/issues/1252#issuecomment-488160930
If you want it, we have a recipe for an equivalent in the docs, but it is not recommended: https://react-redux.js.org/api/hooks#recipe-useactions

We had this as a useActions hook in our original alphas, but Dan strongly argued that we should drop the notion of "binding action creators" and not ship it.  After consideration, we agreed, and dropped it.
Not something we're planning to do at this point.
See https://blog.isquaredsoftware.com/2018/11/react-redux-history-implementation/#v7-1-hooks for recap and links.

Hmmmm.... hard to want to disagree with a genius here, especially with one so paramount in bringing forth such an awesome set of tools for working on the front end.  Unfortunately, it's not what I was hoping for.
I see a lot of the reasoning being done around useActions which accepted an object and returned an object and I can get on board with those decisions.  Is there any chance future consideration might be given to a singular version of the function (e.g. useAction)?  It might seem like semantics, but the change does reconcile several of the concerns mentioned by Dan Abramov in his advocacy for its removal.
Also, if Dan happens to read this, sorry for being contrarian and "overly fixat(ing) on these one-line "shorter" changes and miss(ing) the bigger picture."  However, I do appreciate the abstraction. :D
I understand creating and maintaining a package like this is not easy.  Thanks for your work on redux and in taking the time to review my feature request.

Yeah, really not something we're planning to offer officially, but there's nothing stopping you from writing and using your own (per the recipe in the docs).


[Task]: Fix flakiness of cypress/e2e/Regression/ServerSide/GenerateCRUD/MongoURI_Spec.ts

SubTasks
Test flakiness on this PR https://github.com/appsmithorg/appsmith/pull/33856 which has only shell script change

Hey team! Please add your planning poker estimate with Zenhub @AmanAgarwal041 @carinanfonseca @rishabhrathod01 @rohan-arthur @sneha122

I notice the usage of xPath in the test spec which could lead to flakiness. As it is uncertain, i will estimate this task as 2.


Suppress erlang:now/0 deprecation warnings

Since the topic is access and creation time comparison for purposes of cache eviction, it doesn't seem important whether or not those values are monotonically increasing, nor particularly accurate. I also noted that os:timestamp/0 is already used a couple places in the existing code. However, based on the manual, perhaps we should consistently change them all to erlang:timestamp/0?

OK, as long as you're sure....
Problem is erlang:timestamp/0 doesn't show up until Erlang 18, and we still support 17 for now.

@iilyak you are absolutely correct -- we don't need it at all. While you're here, do you have any opinion on whether it's safe to replace erlang:now with os:timestamp in those locations?

I don't think changing to os:timestamp is safe. We rely on uniqueness of an entry here https://github.com/apache/couchdb-ets-lru/pull/7/files#diff-ec11d0df06913b8bbe0b140772e6f7a4L173

I'm with @iilyak - atime is a primary key on one of the set tables, it has to be unique, so we'd want to change it not for the os:timestamp/0, but for a wrapper similar to couch_util's unique_monotonic_integer/0.
A wild idea to get a speed boost would be os:timestamp/0 with a small random number added to microsecs or even v2 uuid, but I'm not confident about this.

@jaydoane if you are eager to push us towards the future, we could raise the minimum version of Erlang to 19 and start using the new functions that guarantee monotonicity.
If you want to make this change (across the board) please start a thread on<EMAIL_ADDRESS>this needs at the very least lazy consensus.

@iilyak @eiri thanks for pointing out that atime has to be unique as an ets key, and thus why os:timestamp/0 would not be safe here. I'll look into doing this with a wrapper as Eric suggests.
I and other Erlang devs would presumably be happy to see OTP 18 as the minimum supported Erlang (@wohali you mentioned 19 in your last message, but I believe it's actually OTP 18 where the new time functionality was added). Is the main consideration for this which version gets shipped by major OS vendors? I think at this point just to run the test suite we need OTP 19 for Elixir tests, so perhaps that is a better target than 18?

@jaydoane @davisp or @chewbranca told me on IRC that 19 is the minimum for us to start seriously considering maps for real use in CouchDB. That's something I think everyone would like to use if possible.
Also, 99% of CouchDB users are installing from our binaries/Docker image, which have Erlang embedded in them. We build these now with Erlang 19.6.3.
So, in reality, just about all users of CouchDB-recent in the pipeline are using 19.

I tested the most recent change by temporarily adding the 'P' compiler option to rebar.config and inspecting the resulting ets_lru.P file, which showed for OTP 20.3:
unique_now() ->
    erlang:timestamp().

and for OTP 17.5:
unique_now() ->
    now().

Unfortunately, it appears that travis builds ets_lru using rebar3, (and also with numerous Erlang version no longer supported in couchdb).
Should we change .travis.yml to match couchdb/.travis.yml's otp_release list, and also to use rebar? Or maybe this is a sign that we really should just propose making OTP 19 the minimum required?

Travis now seems happy, and no more compile warnings. If I can get a +1, I'll squash into a single commit. (I'm also working on an email to dev proposing changing minimum supported version OTP to 19.)

@jaydoane But erlang:timestamp/0 marked as not a monotonically increasing time, erlang docs recommend to add erlang:unique_integer([monotonic]) into the mix to get determined order of events with time of event. http://erlang.org/doc/apps/erts/time_correction.html#how-to-work-with-the-new-api

Thanks @eiri. Obviously I should not try to read docs (or write code) with a head cold.
Apparently there's no simple drop in replacement for erlang:now/0 pre OTP 19 that will give the same {MegaSec, Sec, MicroSec} time shape and is also guaranteed unique, so AFAICT we would need to change the shape of atime everywhere. That would be a lot of work if we also had to maintain OTP 17 compatibility, but fortunately it seems like everyone is pretty on board with the idea of making OTP 19 the minimal version. I'm going to close this PR now, and open a new one using only the new extended time functionality once we no longer have to support pre OTP 19 versions.


League of Legends LightFX no longer work after new patch

A few days ago I was notified by the game launcher, that the developer (Riot Games) was moving some game files around. Since then, the LoL wrapper lighting has not worked. I've been using it for about a month and a half now with no problems, I've had it working in this current game patch, but after that notification, it hasn't worked once. Automatic patching in Aurora no longer seems to work. When I click "Patch League of Legends" nothing happens (this is also new since the launcher notification). I manually patched both releases in my game folder, but to no effect. I checked both directories manually and the LightFX.dll file is located in the "deploy" folder in both. Other than patching manually instead of automatically, I changed nothing else. Later I also tried creating a new LoL profile and kept it default and it didn't work either.
I have a Wooting Two keyboard.
Expected behavior:
When in-game in League of Legends, the LoL Aurora profile should change the keyboard lighting in the wrapper layer depending on the in-game HP.
Actual behavior:
The wrapper layer stays dark (off) as if I wasn't in-game or the game wasn't properly patched, etc. It doesn't turn on.
How often does this problem occur?
This problem occurs every time.
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Patching League of Legends (with LightFX enabled in the config)
Starting the game

What is the version of Aurora you are experiencing this issue with?
My version is... beta 0.6.4b-624-lkowurnn
This problem did not exist in this same version until recently.
Does the previous version of Aurora have this issue?
I don't know, I haven't used any other version.
Please include any logs, screenshots, or files that are related to this issue:
Logs:
2019-04-14 19.49.56.log
2019-04-11 19.55.18.log
Screenshot:
game directories for patching

That "league_client_sln" folder is the folder for the launcher, not the actual game. If you could go into a training match and find out where the actual game executable is that would be helpful. It used to be in "lol_game_client_sln" but it's not there in your screenshot.

The "league_client_sln" folder and the folders shown in the screenshot are what Aurora directed me to in the Overview section for League of Legends. I located the game executable in "C:\Riot Games\League of Legends\Game" through task manager and patched that folder. The lighting now works.
The new patched location looks like this.
As far as new game patches go, I guess we'll see. Maybe now that the LightFX is in this location, we won't have to patch the files after every game patch...?

That looks a lot easier for us actually. Not having to patch every single update is awesome. I'll change the button to reflect those changes when I can. Thanks for your help!

So, here's an update. I tested the previously mentioned method in practice tool and the lighting worked at the time. I went on with my day without playing. Soon after tho, the lighting stopped working. I attributed this to the new game patch, thought I just needed to re-patch the the game files like previously and it would be good. Turns out this is not the case. Patching the LightFX in this new location requires the game files to be re-patched every time the launcher is opened. When the launcher opens, the LightFX dll disappears.
Looks like I was too quick to jump to conclusions. With this new location, the lighting effects do work, which is an improvement over the old location, where there was no effect, but having to patch the game every time after opening it is not a great fix.

It's an improvement nonetheless. Try going into the properties of the dll after patching and set it to read only. That might prevent the launcher from deleting it.

I tried it. Checking the "Read-only" check mark in the properties did nothing, so I moved on to permission editing. After some trial and error it seems that the game launcher only cares about permissions from "Authenticated users". It needs some permissions like read and execute (not exactly sure which ones exactly, didn't bother) and it requires the delete permission (see image). If it doesn't have this permission, the launcher will attempt to repair the files after starting and fail. The game files will not be verified/game-ready if the game doesn't have the delete permission over the LightFX dll when starting the launcher and it's impossible to launch a game. Patching the game files and/or editing permissions to the dll still only works while the launcher is already opened and only until the next launcher restart. When starting the launcher, it will either delete the dll or try to repair the files and fail.

That's really shitty from Riot.... I'm not sure what we can do to get over this issue unfortunately.

I tested this again and the old fix i commited seems to work just fine.

I just merged this old fix and it seems to work correctly on my machine. Closing the issue. Reopen if needed


No module named request

$ python instagram.py Sami09.1 dictionary.txt -m 0
Traceback (most recent call last):
File "instagram.py", line 6, in
from lib.bruter import Bruter
File "C:\Users\lan\Instagram-master\lib\bruter.py", line 6, in
from .browser import Browser
File "C:\Users\lan\Instagram-master\lib\browser.py", line 6, in
from requests import Session
ImportError: No module named requests

run the following code: pip install bs4 future requests colorama


Using es-dev-server and express

Is there any way to pass a js file representing an express server to the es-dev-server process ?
Basically something like es-dev-server --app server.js would be nice because usually the server that treats some back requests is the same server that provides these front-end interfaces.
Thanks

In the docs there is a section on how to set up middlewares for the server, from there you can proxy to an express server or add your own middlewares.
The server is written in koa, which is a much better dev experience than express so I'd really recommend it. There are plugins however which let you adapt express middlewares for use with koa.


Bump up grid color steps

Right now we only have three color steps for coloring the grid. This makes areas with <5 but greater than 0 very light and now also makes zoomed out views one color:

Could be useful to add a couple steps to show more granularity in density.

Added one level in https://github.com/hotosm/oam-browser/pull/191


SPM-compatible project structure. Linux support. Better podfile. CocoaPods deploy action

Hello.
I did a couple of updates to the repo and fixes:

Updated project structure to be more SPM-like
Moved MG code to MGTools target and added it to SPM so it can be run built on Linux
Moved MG Tests to own target
Added tests to the Podfile
Added CocoaPods tests run phase to CI.
Added Linux tests to CI.
Fixed and added tests for: https://github.com/mkrd/Swift-BigInt/issues/76
Fixed == and != for negative integers and added tests for it.
Added deploy workflow to the CI for CocoaPods repo push.

@ypopovych Hey, thank you so much for your contribution! Looks good on first sight, I will review it in detail later. @twodayslate, I would also appreciate your review :)


[codespaces] bump oryx 20210308.3

Bumps oryx to latest vso-focal build.  Also changes how we symlink dotnet to dynamically accept the versions included with oryx.
Tested with vscode dev containers locally.
Have not validated that this oryx build fixes the python issues we are seeing.  I think the easiest way to validate that is to get this merged and use the generated dev image to validate - right @Chuxel ?

@joshspicer You can also just copy the .devcontainer folder into a repo, update devcontainer.json to point to "base.Dockerfile", and open it in a codespace.  It takes a long time because of how huge the image is, but it works. The devcontianer.json calls oryx build as postCreateCommand... you can also call it manually of course.

ah yes, running the post-create manually makes sense :)

@joshspicer LGTM! Should we merge it or do you want to do more checking?

I'm going to merge this in since it looks good and I need to push an update to fix a bug anyway.


chore: tidb test add conditional trigger

Add conditional trigger and skip report to perform a few days of grayscale testing.

/hold

/lgtm

/retitle chore(jobs/pingcap/tidb-test): add conditional triggers

/unhold


Add circle CI support [WIP]

This PR adds support for the CircleCI build server. Unfortunately, I cannot give CircleCI the permissions on the exercism/xcsharp repository, as I am not the owner. This PR should thus not be merged, which is why I marked it as WIP.
@kytrinyx, once you are back from your well-deserved break, would you mind granting the CircleCI repository access to  the xcsharp (xfsharp) repositories? You can find CircleCI here: https://circleci.com

I've added Circle CI to this repo.

Thanks @kytrinyx!


CVC5 works only with batch mode

This doesn't work:
$ boogie -proverOpt:SOLVER=cvc5 -timeLimit:10 Test/textbook/DutchFlag.bpl
<hangs>

This finishes in less than a second:
$ boogie -proverOpt:SOLVER=cvc5 -timeLimit:10 -proverOpt:BATCH_MODE=true Test/textbook/DutchFlag.bpl
Boogie program verifier finished with 1 verified, 0 errors

@atomb :
What does BATCH_MODE do for CVC5?
Is the problem only for this program or is BATCH_MODE required for any Boogie program to verify with CVC5?

Batch mode is the opposite of incremental mode. That is, each query is entirely self-contained with no (push) or (pop) commands, for use with provers that don't support incremental queries (though it's also entirely possible to use it with provers that do support incremental queries, too).
In principle, CVC5 should work with the incremental interface, but something is going wrong. It could have to do with the ping/pong stuff used to determine whether the solver is still alive.
I think it affects most programs, not just the one above.
And, just to be clear, I'm not filing this to put any more work on anyone else's plate. I plan to investigate it and fix it myself.


CHANGES.txt: Mention CVE-2022-27470

P.S.: Does anyone know whether the SDL-1.2 version is affected? (Not that anyone cares about it, but for completeness.)


Compile classpath issue with CONSUMER_FIRST sort order

Expected Behavior
When using sortArtifacts(ResolutionStrategy.SortOrder.CONSUMER_FIRST); current module's class output directory should come first in classpath before the dependencies.
Current Behavior
Dependencies are ordered as per the selected sort order but the class output directory of the module (which is the consumer) is not the first classpath entry.
As the module itself is a consume, we expect module class output directory come first before the dependencies while compiling java source.
Context
We have a special case where we enhance a class from a dependency by duplicating it in a module. So when incremental compilation is enabled, compilation sometime fails as compiler can't find enhanced version of the class because of the classpath ordering.
Steps to Reproduce
example.tar.gz
Here I am providing a test example.
$ ./gradlew classes
$ sed -i 's/j.smith/john.smith/g' src/main/java/com/example/Main.java
$ ./gradlew classes

As we changed one class, recompiling will fail because of incremental compilation and wrong compile classpath entry.
Your Environment
Gradle 6.2.2

Any updates on this? BTW, we can override the classpath and prepend module's classes directory to solve the issue but I think it should be the default behaviour (and it was in 5.x release).


Create ADR with proposal for Authorization for factorized fleet Manager

The ADR has been opened and it is now in review process


Brightness slider and change label dialog

I added a brightness slider and set label option to a submenu for each bulb. I had to change some project settings to get it to compile (it couldn't find the LIFX framework for some reason, and I don't have a Mac developer account). So the project file would need to be changed if you merge this.

Oh, wow. Sorry for the (massive) delay. I most have missed Github's email notifying me at the time. I was just browsing Github and noticed the pull request count was '1'.
I'll take a look at this tomorrow and merge it. Thanks!


Performance comparison

Hi there!
Im building a relative heavy app (TradeJS), but with minimal electron requirements (so in a way perfect for Electrino).
Only requirement I have is to open multiple windows.
Is it possible to make a performance comparison with some standars JS tests? Like kraken etc.
Maybe it's no use, cause there are already existing benchmarks between chrome and default OS JS engines, but still it would be wise to mention it I think, because performance is one of the downsides in Electron/Web-Wrappers :)
Thanks!!!!

Is there something that makes you unable to run these tests outside of the Electrino scope? I don't seem them being related. Just run kraken in your safari versus whatever v8 version that corresponds to electron.

Yes, the JavaScript engine in Safari is the same one as Electrino is using, so I would assume benchmark results are identical.
@DutchKevv If you have a list of Electron APIs your app needs, let me know! I'm planning to compile a suggested roadmap soon.


Fix #469, Adding coverage tests for cfe_es_apps.c

Describe the contribution
Fixes #469
Added tests to complete coverage on the following functions:

CFE_ES_RunAppTableScan
CFE_ES_ParseFileEntry
CFE_ES_LoadModule
CFE_ES_CleanUpApp
CFE_ES_CleanupObjectCallback

Testing performed
Manually ran unit tests for ES: $ make -C build/native/default_cpu1/es test
Verified coverage on files with $ gcov cfe_es_apps.c.gcno and $ gcov cfe_es_apps.c.gcno
Expected behavior changes
Increased coverage percentage to greater than 99% in cfe_es_apps.c.
System(s) tested on
Ubuntu 18.04 VM
Additional context
There is still 1 untested line in file.
Contributor Info - All information REQUIRED for consideration of pull request
Jose F. Martinez Pedraza / NASA GSFC

CCB:2021-07-21 APPROVED


Fix: README fixes

what

Fix link to example in README
Extend README module description to explain that this module is technically regional but only needs to be deployed once per account.

why

A link was broken in the README, some extra context on module usage was also required.

references

N/A

/test all

/test all


Breaks Ember 2.5.0

Ember 2.5.0 breaks completely (not even the command 'ember' is executable in the terminal) when ember-cli-rename is installed.

@Znegl I would recommend uninstalling this addon. Unfortunately I haven't been actively developing this project, there were some major issues that I was unable to get past to make this function safely and consistently. I do plan on picking work back up on it in the future, but it may not be for a while.

@trabus Understand this is a few years old, but I'd highly recommend you mark this as deprecated in some way to prevent developers from trying to use this in new projects.


Populating replies

Possible to populate replies? If so then how?

@0x2BabyBink you can change the data structure of the comment in your own implementation to store a replies array replies: [] in the json.
Since this extension only takes an id for the reference, you can always decide what data structure of the comment model you'd actually like to have.


Replace reflective backend with kotlinx-metadata-jvm

This backend is lighter and faster that kotlin-reflect, so it should reduce the footprint of this module.

I've made progress with this, but migrating from a KConstructor to a Constructor is tough.

Hello @oharaandrew314
Just wanted to ask if you still trying to solve it. We have some requirements when we are using GraalVM so kotlinx-metadata-jvm would be very beneficial for us.
Thx for your support

No, I'm not actively working on it anymore.  It's quite difficult to build the data class constructors without the assistance of kotlin-reflect.  For instances where I need to avoid kotlin-reflect, I use http4k-connect dynamodb, which can be configured to map the document with a generated JSON adapter (via kotlinx-serialization or kotshi), which is even faster than kotlinx-metadata-jvm anyway.  It's a pretty advanced maneuver that needs better documentation, but I've at least covered it in a video.
For what it's worth, both kotlin-reflect and kotlinx-metadata-jvm use reflection.  I was under the impression Graal could be configured to retain reflective capabilities with some configuration.


ARROW-3544: [Gandiva] [C++] Create function registry in multiple compilation units to reduce build times

Refactored function_registry into separate files. The function signatures are now split across
3 different files viz. function_registry.cc, function_registry_binaryfn.cc & function_registry_unaryfn.cc.
This approach reduces the build times on my setup with little refactoring.
I looked into tensorflow as well, as Wes had suggested, however tensorflow heavily utilizes tempalate based
approach, which is quite different from this & may require quite a bit of refactoring in gandiva.

looks like unary and binary is the factor we are using to distribute the functions..will that hold in the future when we add more functions? should we instead use type (time vs string vs arithmetic etc)..

Agreed, it will make more sense to use a semantic categorization (either parameter types or general operation category - such as math, calendar, etc.).

In general it seems there's a lot of shared code between FunctionRegistryBin and FunctionRegistryUnary can we do it with a single class and template?

Codecov Report

Merging #3051 into master will increase coverage by 1.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3051      +/-   ##
==========================================
+ Coverage   87.05%   88.17%   +1.12%
==========================================
  Files         492      445      -47
  Lines       69122    65425    -3697
==========================================
- Hits        60171    57686    -2485
+ Misses       8850     7739    -1111
+ Partials      101        0     -101

Impacted Files
Coverage Δ

cpp/src/gandiva/native_function.h
100% <ø> (ø)
:arrow_up:

cpp/src/gandiva/function_registry_string.cc
100% <100%> (ø)

cpp/src/gandiva/function_registry_datetime.h
100% <100%> (ø)

cpp/src/gandiva/function_registry_datetime.cc
100% <100%> (ø)

cpp/src/gandiva/function_registry_arithmetic.cc
100% <100%> (ø)

.../gandiva/function_registry_timestamp_arithmetic.cc
100% <100%> (ø)

cpp/src/gandiva/function_registry_hash.cc
100% <100%> (ø)

...c/gandiva/function_registry_timestamp_arithmetic.h
100% <100%> (ø)

cpp/src/gandiva/function_registry_arithmetic.h
100% <100%> (ø)

cpp/src/gandiva/function_registry.cc
100% <100%> (ø)
:arrow_up:

... and 81 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 1f099c7...83ab23b. Read the comment docs.


Comparing with jwt

Hi, great gem.
I'm have just one question. How this gem architecture compared to json web tokens? And one more, when exactly a request hit the database?

The security measures are listed here.
The expected token format is described here.
As far as I can tell, jwt doesn't offer any advantages over this gem's token format.

With jwt I can check validity of token just by decrypting it, without comparing with token in database. Does devise_token_auth comparing and hitting database when you checking tokens for validity?

I've made a conscious decision to compare the token with the database for each request.
The main reason is for access control. For example, if a user's account has been disabled since their last login, the only way to know would be to check against the database.

I understand, but it is heavy constraint.  For example, let's imagine chat application, when you send message you can checked access control for thread by passing user_id and check manually if this id present in thread permissions, that's way you only need read thread and insert message, but with this approach i need load user for every request. This can become issue with scaling as you have more users and requests.

There are other issues as well.

Tokens are effectively passwords. If an attacker has your access token, they can do anything with your account. For this reason, we store tokens using bcrypt. This is how passwords are normally hashed. Once the token is hashed using bcrypt, it cannot be recovered. So to check a token's validity, we perform the following actions:

search the DB for a user by their uid
if the user is found, hash the token from the request
compare that hash with the hash that's stored in the DB
if the hashes are equal, the token is valid.

With jwt, the encrypted token is usually included with each request (that's how it saves the trip to the DB). And it's encrypted using an algorithm that can be easily brute-forced (HMAC or something). Would you do this with your password?

If you're building a real-time app, you should be using a socket connection (like faye or something). This gem won't work with websockets anyway.

I would be open to de-coupling the validation system so that it can be easily overridden (using jwt or something else). Feel free to send me a PR.

Access Control is not matter of authentications, it's another issue.
https://github.com/Notsew/ruby_jwt support different algorithms as well as custom. So anyway if you want brute force SHA-512, good luck :)
If attacker stole token with devise_auth_token he can do anything too and encryption doesn't help.

Thanks for answers, I'm trying wrap my head around this issues to have best from both worlds.

There's a good discussion about this here if you're interested.

We encrypt token not a password, and only we can decrypt because we have secret key. So hashing token doesn't make sense. I think :)


Camera Preview is Out of Screen on IOS

Describe the bug
CameraPreview() is out of bond / Screen when application start for the very first time / coming from lock screen.
When user unlock application preview is out of screen again.
To Reproduce
Steps to reproduce the behavior:

Open Application very first time / Lock & Unlock screen when application is running.
go to any other view and coming back to home page is working fine.

Expected behavior
CameraPreview stay on its place do not change position on lock screen or First time application start.
Screenshots

Smartphone (please complete the following information):

Device: iPhone
OS: iOS 15.5

Additional context
Working fine on Android. This bug is happening on IOS only. When user Install application and open first time this is happening or user Unlock their phone this is happening. Coming back to home screen form other screen this is not happening.

Hi @Saqib92 I'm getting crazy with issue #221 , but it seems that your app is not affected by that. I have a couple of questions:

What version of Capacitor, Ionic and plugin are you using?
Can you reproduce the issue with the demo app?

Hi @Saqib92 I'm getting crazy with issue #221 , but it seems that your app is not affected by that. I have a couple of questions:

What version of Capacitor, Ionic and plugin are you using?
A: Capacitor 3.6.0, Ionic: 6.1.9, Plugin: 3.1.0
Can you reproduce the issue with the demo app?
A: Not checked with demo app.

But to overcome these issues i removed capacitor plugin and used Cordova camera preview. and everything is working fine now.

thank you @Saqib92 , I too tried with Cordova camera preview but unfortunately toBack option does not work when using that plugin with Capacitor and I need that option in my app.

Hi @Saqib92, sorry to hear you've got problems with the position of the camera. I've tried to reproduce this and I can't make it happen with Capacitor 3.2.5 / Ionic 5. I don't yet have an Ionic 6 app to test with to see if that's changed how the layout works, but will try to upgrade to Capacitor 3.6 soon and see what happens.

Thank you. If i get a chance i will create Ionic 6 and capacitor 3.6+ project for your help.

Hi @Saqib92, sorry to hear you've got problems with the position of the camera. I've tried to reproduce this and I can't make it happen with Capacitor 3.2.5 / Ionic 5. I don't yet have an Ionic 6 app to test with to see if that's changed how the layout works, but will try to upgrade to Capacitor 3.6 soon and see what happens.

Still facing same issue on Capacitor 4, ionic 6,
Reproducible  git repo:
capacitor-preview-camera-test

thank you @Saqib92 , I too tried with Cordova camera preview but unfortunately toBack option does not work when using that plugin with Capacitor and I need that option in my app.

did you found a solution for iOS for toBack?

@Saqib92 no, Cordova Camera Preview is incompatibile with Capacitor, because the framework handles the WebView in a different way than Cordova.

@sertal70 in my application cordova plugin is working fine. jsut toBack is not working in ios.

@Saqib92 you're right, with "is incompatible" I was implicitly referring to the toBack feature

@Saqib92 you're right, with "is incompatible" I was implicitly referring to the toBack feature

ohh okay. But is the any workaround for this??


Device data is not updated in a timely manner

The update time is sometimes half an hour, sometimes an hour ago without this problem

The latest version has been resolved


[gtest] Build error

Host Environment

Host: x64-linux (WSL Ubuntu 23.04 on Windows 11)

Compiler: GNU 12.2.0

vcpkg-tool version: 2023-02-16-12e657924d99511514c0287ca5ce46882d3657c7
vcpkg-scripts version: a7b6122f6 2023-02-24 (6 weeks ago)

Host is a WSL Ubuntu distro running on a Windows 11 Pro Insider Preview Build 25330

What I see is that pkg-config is running from a Strawberry Perl installation from Windows, which is not part of WSL nor Ubuntu. I tried installing pkg-config on Ubuntu but failed miserably.

The repo where the code is stored is in a Windows drive mounted on /mnt/d as is with the WSL default configuration.

To Reproduce
./vcpkg install
Failure logs
-- Using cached google-googletest-v1.13.0.tar.gz.
-- Cleaning sources at /home/eugeniomiro/vcpkg/buildtrees/gtest/src/v1.13.0-0674f4ed11.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/eugeniomiro/vcpkg/downloads/google-googletest-v1.13.0.tar.gz
-- Applying patch 001-fix-UWP-death-test.patch
-- Applying patch clang-tidy-no-lint.patch
-- Applying patch fix-main-lib-path.patch
-- Using source at /home/eugeniomiro/vcpkg/buildtrees/gtest/src/v1.13.0-0674f4ed11.clean
-- Configuring x64-linux
-- Building x64-linux-dbg
-- Building x64-linux-rel
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest_main.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-all.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-assertion-result.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-death-test.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-filepath.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-internal-inl.h
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-matchers.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-port.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-printers.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-test-part.cc
-- Installing: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/src/gtest-typed-test.cc
-- Fixing pkgconfig file: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/lib/pkgconfig/gmock.pc
-- Fixing pkgconfig file: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/lib/pkgconfig/gmock_main.pc
-- Fixing pkgconfig file: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/lib/pkgconfig/gtest.pc
-- Fixing pkgconfig file: /home/eugeniomiro/vcpkg/packages/gtest_x64-linux/lib/pkgconfig/gtest_main.pc
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:135 (message):
  /mnt/c/Strawberry/perl/bin/pkg-config --exists gmock failed with error
  code: No such file or directory

      ENV{PKG_CONFIG_PATH}: "/home/eugeniomiro/vcpkg/packages/gtest_x64-linux/lib/pkgconfig:/home/eugeniomiro/vcpkg/packages/gtest_x64-linux/share/pkgconfig:/mnt/d/repos/server/vcpkg_installed/x64-linux/lib/pkgconfig:/mnt/d/repos/server/vcpkg_installed/x64-linux/share/pkgconfig"
      output:
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:200 (z_vcpkg_fixup_pkgconfig_check_files)
  buildtrees/versioning_/versions/gtest/3f70f42192aacc5f8242c14d959d02a802f7559c/portfile.cmake:49 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:147 (include)

Additional context
vcpkg.json
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
  "name": "server",
  "version-string": "6.6.0",
  "builtin-baseline": "a7b6122f6b6504d16d96117336a0562693579933",
  "dependencies": [
    {
      "name": "boost-program-options",
      "version>=": "1.78.0"
    },
    {
      "name": "boost-filesystem",
      "version>=": "1.78.0"
    },
    {
      "name": "gtest",
      "version>=": "1.11.0"
    }
  ]
}

@eugeniomiro, Thanks for posting this issue, I use your vcpkg.json file install gtest successfully in linux machine:
vzhli17@vzhli17001:~/vcpkg$ ./vcpkg install
Detecting compiler hash for triplet x64-linux...
All requested packages are currently installed.
Restored 0 package(s) from /home/vzhli17/.cache/vcpkg/archives in 1.8 us. Use --debug to see more details.
Total install time: 15.3 us
The package boost is compatible with built-in CMake targets:

    find_package(Boost REQUIRED [COMPONENTS <libs>...])
    target_link_libraries(main PRIVATE Boost::boost Boost::<lib1> Boost::<lib2> ...)

The package gtest is compatible with built-in CMake targets:

    enable_testing()

    find_package(GTest CONFIG REQUIRED)
    target_link_libraries(main PRIVATE GTest::gtest GTest::gtest_main GTest::gmock GTest::gmock_main)

    add_test(AllTestsInMain main)

I found that your issue is very similar to the error of this issue, so I will try to fix this error locally

@eugeniomiro, sorry for reply you so late, could you please re-clone vcpkg with another disk and try again? Because of this error, I can't reproduce it locally, and I found an issue with the same error as yours, it is because of the path problem.

@JonLiu1993, I figured my actual problem. I saw in the log I started with that pkg-config was taken from
/mnt/c/Strawberry/perl/bin/pkg-config

which is not a Linux path but a Windows one, therefore the binary was not aware of WSL. Then I installed pkg-config using
sudo apt install pkg-config

in my WSL environment, logged out, logged back in and it worked.
Thank you very much for taking the time to help me.

How do these Windows paths get into the WSL vcpkg? It must be fixed if it is breaking builds.
The failing script does vcpkg_find_acquire_program(PKGCONFIG) which will pick pkg-config from the PATH or raise an error.
@eugeniomiro Did you install the Linux tools mentioned in https://github.com/microsoft/vcpkg/#installing-linux-developer-tools?

Hm, build-essential doesn't include pkg-config. Maybe it should add this package and other essentials (autoconf etc.).

How do these Windows paths get into the WSL vcpkg? It must be fixed if it is breaking builds.
The failing script does vcpkg_find_acquire_program(PKGCONFIG) which will pick pkg-config from the PATH or raise an error.
@eugeniomiro Did you install the Linux tools mentioned in https://github.com/microsoft/vcpkg/#installing-linux-developer-tools?

Hi @dg0yt. I installed everything as shown on the vcpkg getting started page in the Linux tab.

Regarding the build essentials tools, I was migrating a project which already had all those requirements, I only had to add zip by hand, which I did while I was migrating the CMake script.

This problem has been solved, I will close this issue, if you have other problems, I will reopen it.


"No repository linked" dialog is showing twice at startup.

When we build and run the Gitfx application for the first time, the dialog box is showing twice.

Thanks for reporting this. I will look into it.

The issue is fixed. TODO: Add tests in GitFxAppTest to test this scenario.


Spending limit - Creating a spending limit is trying to add the module again

Issue:
After the Spending limit module is enabled, any future spending limit creation should just create the "Allowance" for the owner to use tokens, but the tx is trying to enable the module again
Steps:
Create a new safe with a "2 owners" policy
Create a spending limit with ETH
Check that the owner is listed in the spending limit table
Delete that spending limit
Try to create another spending limit for any owner
Expected result:
The app should check whether the spending limit module is enabled

If yes, just the set allowance, no need to enable the module again.
if no, the module first needs to be enabled.

Current result:
The tx has the action to enable the module again
Check this safe, that has the tx in Queue where is trying to enable the module again, then check also the "Advanced" settings, where the module for spending limit can be seen already active:
https://pr2535--safereact.review.gnosisdev.com/rinkeby/app/#/safes/0x2733739C73DaFe449BBd4d962CbE270221A85944/transactions

I've checked and this issue is actually in prod v3.9.0 so is not a blocker for the release.


Data Audit log error - "RecordId" violates non-null constraint

Hi
Has anybody used DataAuditLog of serenity. I am facing issues with adding new record.
When I save new record it gives the following error.

Found the issue.
The error comes when the application is not able to generate Id automatically. So add Identity in the xxRow
[DisplayName("Status Id"), PrimaryKey,Identity]


Request to add sensor for Nederlandse vliegveldverwachtingen

Did you read the instructions?

[X] I have read the README, including the known-limitations section.

The request
Would be awesome if you could add the text in https://www.knmi.nl/nederland-nu/luchtvaart/vliegveldverwachtingen as a sensor. Even better if it's one sensor per airport code.
Additional information
No response

This component makes use of the weerlive API, https://weerlive.nl/delen.php and that doesn't support those


Certain games won't load

My psone 4.5 model only seems to load up toy story and very few games. I've installed the exploit onto those cheap fake memcards sold on eBay.
It loads up fine on unirom. But the resident evil trilogy refuses to work.
What am I doing wrong? The games are NTSC from vimmr lair and I burn on verbatim CD-R at the lowest speed possible

Help anyone please? :(

Hi there!  In my experience, there are some issues with burning games downloaded from the Internet, simply because they haven't been dumped and verified properly.  The best way to know if a backup disc is going to work properly is to rip a copy of the actual disc, using ImgBurn (because that program will create the proper CUE sheet and BIN file) and then burn that BIN/CUE pair to a CD-R at 4x (which is the "safe" burn speed).  CUE sheets downloaded from the Internet vary wildly and most of the time aren't constructed properly.  I did some testing with Breath of Fire IV some time ago and the CUE sheet created with ImgBurn from a real disc wasn't anywhere close to several CUE sheets I downloaded from the Internet.  Just because a site says that they have verified a BIN/CUE pair, doesn't mean that it's verified for burning.  It might work in an emulator just fine, but you need a BIN/CUE pair taken directly from the real disc in order to verify that the burn will run successfully on real hardware.
Hope that helps!
SephirothFan


[T5A3][T11-B3] Tan Hong Yu

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


Full screen markdown editor

Based on Boris compose PR,

[x] Support some wrapping tags
[x] Supports autocomplete of streams, people, emoji
[x] Support list tags

Screencast:

Autocomplete list:

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

Supports live preview now :)
@borisyankov Any chance you can merge your compose PR sooner?

@borisyankov Updated with test's!

Rebased and fix merge conflicts!

@borisyankov Updated, this will be safe to merge as it doesn't comes in b/w in any functioning of the app, and already it had one round of review!

@borisyankov Updated

We will work on finishing this, once we release publicly the iOS and Android (hopefully very soon).
I am also holding on working on cool new features till then.
My idea was to pop the editors with 'Toggle compose tools' command (in addition to the 'topic' editor)

@borisyankov Okay, sounds good 👍

Found some time in my busy schedule of Lab exams + end semester exams on my head, so updated this to go with the latest awesome update!
@borisyankov rebased this to master!

Updated

A lot of the code of the PR is now OK, but also there is a lot to comment on... which will drag on... and you know why... because you did combine several features in one PR.
There are three features here:

preview (great one)
edit tools (good one, needs some polish)
new screen (somewhat useful, maybe not that much anymore)

Focus on a PR that includes only the preview first. Then we will merge it quickly (after a review) and move on to the second one.

Ok, will split out the PR into parts!
Though I believe a full-screen text editor is required for ease of markdown and writing big messages!

Closing, to be posted in multiple PR's!


Remove Preference Dialog

The preference dialog has no use at all since it's not even working! 😞

Thus, the preference dialog should be removed.

The reason why I closed this is because of this: #34


Add parallel disk usage tool into the coder dev container

It's just as good as the Flashblade toolkit, less intensive, and without the awkward licensing.
https://github.com/KSXGitHub/parallel-disk-usage

On hold working through cache Rust crate index.

Putting a hold on this.


Publishing messages from the HTTP API

I've been using VerneMQ for a while, it's one of the best MQTT brokers I've used however the lack of publishing messages from the HTTP API is really frustrating.
I need to publish MQTT messages from my backend to the client after a specific job is done and this requires publishing messages from the HTTP API, Currently am using the open source version of emqx, it allows publishing messages from the HTTP API, We would love to have this feature in VerneMQ, since its fully open source, unlike emqx some features require the enterprise version.

@AmirOsman2020
I'm not against improving the existing HTTP layer, or even adding specific features like a publish option. Internally, this has low priority; but I'm happy to review PR's if anyone wants to have a go at the HTTP publish call.

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

@AmirOsman2020 for multi-protocol bridge you can take a look at Mainflux. It adds HTTP adapter on top of VerneMQ (or any other MQTT broker), as well as WebSocket, CoAP and other protocols. It also adds user management, grouping, authorization layer, storage layer, message transformation and many other features needed for an enterprise IoT system.

This is interesting, I will try it out, Thanks for the help.

If you can use Erlang，it's very easy ,
rpc:cast(MQTTNode, vmq_reg, subscribe, [true, SubscriberId, NewTopics]),
rpc:call(MQTTNode, vmq_reg, unsubscribe, [true, SubscriberId, NewTopics]),
rpc:call(MQTTNode, vmq_reg, publish, [true, vmq_reg_trie, VmqMsg]),

If you can use Erlang，it's very easy ,
use cowboy to create an http server , and connect to Vernemq node ,do everyting.
rpc:cast(MQTTNode, vmq_reg, subscribe, [true, SubscriberId, NewTopics]),
rpc:call(MQTTNode, vmq_reg, unsubscribe, [true, SubscriberId, NewTopics]),
rpc:call(MQTTNode, vmq_reg, publish, [true, vmq_reg_trie, VmqMsg]),

I didn't know this is possible, sounds great, can you provide a full details on how to achieve this?

@AmirOsman2020
https://github.com/boboxiaodd/web_agent


Fix mispelling in README

haha the rogue to rouge misspellings were bound to happen
can you consolidate any edits fixes into one PR? #44 #43


Default location of eclipseMirror no longer available

The default location for the the minimal setup of eclipse plug-ins required to build an eclipse p2 update site is defined by eclipseMirror, it's default setting: https://dl.bintray.com/simon-scholz/eclipse-apps/eclipse-p2-minimal.tar.gz, is no longer available since JFrog has sunset the Bintray.com service sine 2021-05-01.

Could this be fixed please? We are now unable to run a build on servers that are missing this dependency. Is there a workaround?

I just stumbled over the same issue. Any alternative locations from where to get this?

I have a local copy, and I can not remember where I got it from.
If you have such a local copy you could add it in the .gradle/bnd-platform folder or set the location via
eclipseHome = new File('

Hi Guys,
sorry for the very late replay.
I'll create a new one for you.
Basically the product was added to this repo by me https://github.com/stempler/bnd-platform/blob/master/minimal-product.product
Any recommendations for an easy to use artifact repository like bintray used to be?
Unfortunately bintray has been decommissioned and therefore it does not work any more.
Sorry for the inconvenience.

eclipse-p2-minimal.zip

Thanks @SimonScholz -- In the meantime I found the original version in my backups and was able to recover my builds. It's 4.8 MB small. Could this probably be hosted here in the repository, and someone update the URLs?
eclipse-p2-minimal.tar.gz (SHA d930164f51a9eadb0cdcdff35a3fa5761598520d)

The original artifact has been added to the repository and the respective URL is now the default. Available in version 2.0.0.

Reopening this since it turned out that using the original minimal Eclipse product provided by @SimonScholz seems to produce incomplete update sites in some cases (plugins are missing from the update site that were in the source plugins folder).
This seems to be resolved in newer Eclipse versions (tried with 2023-09).
I tried setting up a build for the minimal product with Maven Tycho, but the resulting product does not work for me (complains about missing services). See here.
So for now I changed the default eclipse artifact that is downloaded to a full Eclipse 2023-09 download, which is rather large.
Reproduction case:
platform {
  bundle 'com.vladsch.flexmark:flexmark-all:0.64.0' // com.vladsch.flexmark.util-misc and other are missing from the update site (though source bundles are added)
}


Displays PDF fields in template creation form

The PDF creation form is not working properlly as it shows PDF specific fields in every situations

It has been fixed with the webpack refact


Subgraphs.xyz Transaction count checker issue

There is a warning being thrown on the validation end saying the sum of transactions doesn't equal dailyTransactionCount.

I believe this is an error on the validation end as the transactions check out in the graphql playground: https://api.thegraph.com/subgraphs/name/messari/compound-ethereum/graphql?query=query+MyQuery+{
++usageMetricsDailySnapshot(id%3A+"18176")+{
++++dailyTransactionCount
++++dailyWithdrawCount
++++dailyRepayCount
++++dailyLiquidateCount
++++dailyDepositCount
++++dailyBorrowCount
++}
}

@dmelotik this should be resolved on dashboard v1.1.10 (verified against Compound and Solarbeam-Moonriver)


Rename to brainglobe-atlasapi

The names of tools are inconsistent, and bg isn't very informative. We agreed in the last dev meeting to move to brainglobe-xyz as a standard.

But also see https://github.com/brainglobe/BrainGlobe/issues/43

Given https://github.com/brainglobe/BrainGlobe/issues/43, it seems logical that this happens at the same time as the merge so that we don't end up publishing another package to PyPI only to deprecate it seconds later.
Closing this and moving the content to https://github.com/brainglobe/BrainGlobe/issues/43 as an additional task


Allow publishing accelerations in WholeBodyStateRosPublisher

The current implementation of the WholeBodyStateRosPublisher defines the acceleration vector as a vector of 0s.
I think it could be nice to overload the function, allowing the user to publish the acceleration information if desired.
@cmastalli If you agree, I will implement it

Yes, I do. It is a good idea!


Added ability to add labels to k8s pod

Describe changes
Added the ability to add labels to k8s pod
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to docs, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Probably wants a tiny docs update at least mentioning that people can do this? Probably in the orchestrator docs itself

@schustmi do we need a migration for this or something?


NilEncoding always dropKey when encoding an Encodable object with optional properties

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
I am using URLEncodedFormEncoder with the option NilEncoding.dropValue to encode an object that has optional string properties.
What did you expect to happen?
Assuming the following struct :
struct MyStruct: Encodable {
    let p1: String
    let p2: String?
}

If I encode myStruct I expect to get the following query p1=string&p2=.
do {
    let myStruct = MyStruct(p1: "string", p2: nil)
    let formEncoder = URLEncodedFormEncoder(nilEncoding: .dropValue)
    let query: String = try formEncoder.encode(myStruct)
    print(query) // prints "p1=string"
} catch {
    print(error)
}

What happened instead?
Got p1=string.
Alamofire Environment
Alamofire Version: 5.7.1
Dependency Manager: Cocoapods 1.12.1
Xcode Version: Xcode 14.3.1
Swift Version: 5.7
Platform(s) Running Alamofire: iOS
macOS Version Running Xcode: Ventura 13.3.1
Demo Project
Code above is sufficient to reproduce the issue.
Suggestion
Digging into Alamofire I think the issue might by in extension _URLEncodedFormEncoder.KeyedContainer: KeyedEncodingContainerProtocol {.
In fact when the encoder found a nil value it seems to prefer using the encodeIfPresent method instead of encodeNil.
Using this rather naive implementation I managed to get my Struct encoded properly :
func encodeIfPresent(_ value: String?, forKey key: Key) throws {
    if let value {
        try encode(value, forKey: key)
    } else {
        guard let nilValue = nilEncoding.encodeNil() else { return }
        try encode(nilValue, forKey: key)
    }
}

Confirmed, thanks for the report!
Oddly, this isn't the case for [String: String?], so I'll have to figure out why that is. I've replicated your solution but it would only fix it for String? values, and I'd rather not have to have an overload for every type.

I've fixed this in #3779, feel free to take a look and ensure it works for you before I merge.

This has been merged and will go out in the next release. Depending on whatever else I may want to go out, it may be as soon as tomorrow or this weekend.

Thank you for your quick reply! Awesome 🙏

Coming back here for an update, do you plan to release a version with the fix soon?

I was hoping to release it alongside WebSocketRequest but I'm still working on things there. I'll do a release.

I've released 5.8.1 with this fix but CocoaPods availability will be delayed pending a backend issue on their side. SPM / Carthage should see it now.


Create simple-service-broker-writing.md

At the Catalog SIG offsite, I brought up the topic that we need to have some way to make service broker authoring simple (whether part of the Catalog deliverables or separate) or that we may not get people to adopt the technology. There was some pushback that this was already handled and not a challenge and that I should put forth use cases that we can test the current plans against. This document is the response to that.

I understand that @pmorie is gonna do a follow up, so I'm LGTMing this as-is

LGTM

LGTM