Allow selecting multiple files in the editor translation/remap dialogs

This makes it faster to add several translations.
The undo/redo messages were also tweaked to give better context.
I tested this locally with translations, remaps and POT files, it works fine (including undo/redo) :slightly_smiling_face:
This pull request can be remade for the 3.2 branch once it's merged in master.
This closes https://github.com/godotengine/godot-proposals/issues/2039.

Will need a rebase to fix CI.

Thanks!


Add Fine Grained Saliency algorithm

Since now there is a saliency module, I would like to add an implementation of fine grained saliency, which fits nicely in the static section of the module.
Fine grained saliency is based on algorithms described in:
Montabone, Sebastian and Soto, Alvaro. "Human Detection Using a Mobile Platform and Novel Features Derived from a Visual Saliency Mechanism." Image and Vision Computing, 2010.
This pull request implements the staticSaliencyFineGrained class, so that a new saliency algorithm FINE_GRAINED is available. The code was originally released at http://www.samontab.com/web/saliency
You can see an example of it here:

@vpisarev, @bhack
Hi guys. Do I need to do anything else with this?, or how to proceed from here?

@lenlen @bhack @vpisarev @fpuja
Just updated the source files to match the recent changes to the module structure.
Let me know if there is anything else that needs to be done to merge the code.

@vpisarev any reason why we are not including stuff like this? It passes all builds and has been standing around for ages :)

@samontab that's the status of PR? Could you rebase the latest master into your branch?

Replaced by #821


MWPW-139825 Workflow to clear Akamai cache

Add a GitHub Actions Workflow to clear Akamai cache. It will be used in conjunction with the deployment workflow. Currently the deployment of an EdgeWorker is manual. This workflow can be triggered manually after a deployment.

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

Comparison is base (648b2a4) 99.02% compared to head (fcab501) 99.05%.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #520      +/-   ##
==========================================
+ Coverage   99.02%   99.05%   +0.03%
==========================================
  Files          26       26
  Lines        2961     2961
==========================================
+ Hits         2932     2933       +1
+ Misses         29       28       -1

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


Service Request by [editing of demographics] on behalf of the Columbia team

[ ] Qualtrics Service Request Form was received from User
[ ] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Qualtrics Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

From: David Lounsbury<EMAIL_ADDRESS>Sent: Monday, March 14, 2022 1:52 PM
To: David Lounsbury<EMAIL_ADDRESS>Subject: IPA Tracker Support Request Form Submitted
IPA SERVICE REQUEST SUMMARY

CONTACT INFORMATION:

Jessica  Neufeld
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>

COUNTY

All counties

WHICH FEATURE(S):

MONITORING: Creating, updated or deleting information about an implemented SMART Strategy, REPORTING: Downloading data to create a table, figure or other summary report

BRIEF PROBLEM DESCRIPTION:

An edit needs to be made to the demographics section. "F)Pacific Islander or Native Amer " should be changed to "Native Hawaiian or Other Pacific Islander"

URGENCY:

50

ANYTHING ELSE TO SHARE:

Hi Donrie:
Thanks for setting up this issue card.
This request is easy, I think. Can you make this edit in LIST OPTIONS for Jessica?!
Best, D

Hi profe
I am working on this as we speak,
I got some error codes that I took screen shots of.
However, before I send that stuff to Jeff I want to be sure it is properly vetted. I am updating my computer because I noticed some funny stuff today to make sure that’s not part of the problem.
I’m upstairs, call me if need be.
Sent from my iPhone
On Mar 14, 2022, at 4:14 PM, David W Lounsbury @.***> wrote:

CAUTION: This email comes from an external source; the attachments and/or links may compromise our secure environment. Do not open or click on suspicious emails. Please click on the “Phish Alert” button on the top right of the Outlook dashboard to report any suspicious emails.
Hi Donrie:
Thanks for setting up this issue card.
This request is easy, I think. Can you make this edit in LIST OPTIONS for Jessica?!
Best, D
—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdlounsbu%2FNYHEALing-IPAtracker-support%2Fissues%2F88%23issuecomment-1067243717&data=04|01|donrie.purcell%40einsteinmed.edu|67135ef2eaa84895596308da05f74ca1|9c01f0fd65e040c089a82dfd51e62025|0|0|637828856981677242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Uob93B8aN4OayJXfMsAEmSoT%2BsfCvVllTrwb%2BwdJwxg%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATJKBBMTFEXLUCUZXVVMCW3U76M3ZANCNFSM5QWSQWWQ&data=04|01|donrie.purcell%40einsteinmed.edu|67135ef2eaa84895596308da05f74ca1|9c01f0fd65e040c089a82dfd51e62025|0|0|637828856981677242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=BIB5W6SCM%2FCpuGhjnBuLcTdatyQ%2FZolfAOb3FgIL8DA%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|donrie.purcell%40einsteinmed.edu|67135ef2eaa84895596308da05f74ca1|9c01f0fd65e040c089a82dfd51e62025|0|0|637828856981677242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=0M8VQW%2BcPRVAZJMuupw11xgFSuG9Q8mGtEaV4Il3RbA%3D&reserved=0 or Androidhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|donrie.purcell%40einsteinmed.edu|67135ef2eaa84895596308da05f74ca1|9c01f0fd65e040c089a82dfd51e62025|0|0|637828856981677242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=%2B02Ot3LCuMN%2FeltpuVwp4Mwr5BV1qBVCh5wsmmrMlrc%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>

Hi Donrie:
Ok. Let me know if you need me to work on this for JN.
DL

Hi Profe
The demographic change was completed and you were cc'd on the email.


insights from snmp integration

trap handler needs full path (/usr/local/bin/alerta-snmptrap)
snmptrapd needs to be loaded with "-m ALL", and MIB package needs to be installed, even for example to be parsed correctly

Thanks. I'll update the README.

Wow. I wish I'd updated the README earlier. See #358 for whole sorry story.


Delete a Node from the end in a Doubly Linked Lists

Is your feature request related to a problem? Please describe.
The problem is to delete a node from the end in a Doubly Linked List.
Describe the solution you'd like
I would like to provide a C++ coded solution to this problem

@smv1999 Please assign it to me.

Sure @kiruba-r11 , you can go ahead !


Block when  Call client inference in multiprocessing.Process

I tried to integrate mii into tritonserver, but encountered some problems
Below is part of my code
class TritonPythonModel:
    def initialize(self, args):
        import mii
        from transformers import AutoTokenizer
        tensor_parallel_size = 1
        cuda_env = env_manager.cuda_visible_devices
        if cuda_env is None:
            from torch.cuda import device_count
            tensor_parallel_size = device_count()
        else:
            tensor_parallel_size = len(cuda_env.split(",")) if cuda_env else 1
        self._model = mii.serve(self.base_model_path, deployment_name="atom", tensor_parallel=tensor_parallel_size)
        self._tokenizer = AutoTokenizer.from_pretrained(self.base_model_path, trust_remote_code=True)

    def execute(self, requests):
        responses = []
        for request in requests:
            self.process_request(request)
        return None

    def process_request(self, request):
            # self.create_task(self.mii_response_thread(request.get_response_sender(), request))
            thread = Process(target=self.mii_response_thread,
                                    args=(request.get_response_sender(), request))
            # thread.daemon = True
            thread.start()

    def mii_response_thread(self, response_sender, request):
        try:
            import mii
            event_loop = asyncio.new_event_loop()
            asyncio.set_event_loop(event_loop)
            req_desc = pb_utils.get_input_tensor_by_name(request, "JSON")
            req_json = json.loads(req_desc.as_numpy()[0])
            stop = req_json.get('stop', False)
            query, prompt, history, stream, gen_config, response_config, tools, mode  = self.process_input_params(request, req_json)
            client = mii.client('atom')
            output_tokens = []
            def callback(response):
                logger.debug(f"Received: {response[0].generated_text}")
                self.send(response_sender, response[0].generated_text)
                # print(f"Received: {response[0].generated_text} time_last_token={time_last_token}")
                output_tokens.append(response[0].generated_text)
            logger.debug("call mii generate")
            client.generate(prompt, max_new_tokens=4096, streaming_fn=callback)
            logger.info(f"output text is:{''.join(output_tokens)}")
        except Exception as e:
            logger.exception(f"Capture error:{e}")
            self.send_error(response_sender, f"Error occur:{e}")
        finally:
            self.send_final(response_sender)
            # self.handler.ongoing_request_count -= 1

the error is:
when i use
 thread = Process(target=self.mii_response_thread,
                                    args=(request.get_response_sender(), request))

mii block at
async for response in getattr(self.stub,
                                      task_methods.method_stream_out)(proto_request):
            yield task_methods.unpack_response_from_proto(response)

when i use
 thread = Thread(target=self.mii_response_thread,
                                    args=(request.get_response_sender(), request))

Able to infer normally, but grpc keeps reporting errors(Does not affect inference but the service is not stable)
https://github.com/grpc/grpc/issues/25364

I meet the similar case. Here is my code:
def worker(rank, this_model):
    try:
        if this_model is None:
            client = mii.client('qwen')
        else:
            client = this_model
        response = client.generate(["xxx"], max_new_tokens=1024, stop="<|im_end|>", do_sample=False, return_full_text=True)
        print("in worker rank:", rank, " response:", response)
    except Exception as e:
        print(f"Capture error:{e}")
    finally:
        print("final")

model = mii.serve(model_dir, deployment_name="qwen", tensor_parallel=xx, replica_num=replica_num)

job_process = []
for rank in range(0, replica_num):
    if rank == 0:
        job_process.append(threading.Thread(target=worker,args=(rank,model,)))
    else:
        job_process.append(threading.Thread(target=worker,args=(rank,None,)))
for process in job_process:
    process.start()
for process in job_process:
    process.join()

When using threading.Thread, it works well. However, it will be blocked in client.generate if using multiprocessing.Process.

I meet the similar case. Here is my code:
def worker(rank, this_model):
    try:
        if this_model is None:
            client = mii.client('qwen')
        else:
            client = this_model
        response = client.generate(["xxx"], max_new_tokens=1024, stop="<|im_end|>", do_sample=False, return_full_text=True)
        print("in worker rank:", rank, " response:", response)
    except Exception as e:
        print(f"Capture error:{e}")
    finally:
        print("final")

model = mii.serve(model_dir, deployment_name="qwen", tensor_parallel=xx, replica_num=replica_num)

job_process = []
for rank in range(0, replica_num):
    if rank == 0:
        job_process.append(threading.Thread(target=worker,args=(rank,model,)))
    else:
        job_process.append(threading.Thread(target=worker,args=(rank,None,)))
for process in job_process:
    process.start()
for process in job_process:
    process.join()

When using threading.Thread, it works well. However, it will be blocked in client.generate if using multiprocessing.Process.

Since the threading.Thread is fake in python due to GIL, this code can not make full use of concurrency. It means that I still need multiprocessing.Process to start a new client. However, it does not work well mentioned above.

I meet the similar case. Here is my code:
def worker(rank, this_model):
    try:
        if this_model is None:
            client = mii.client('qwen')
        else:
            client = this_model
        response = client.generate(["xxx"], max_new_tokens=1024, stop="<|im_end|>", do_sample=False, return_full_text=True)
        print("in worker rank:", rank, " response:", response)
    except Exception as e:
        print(f"Capture error:{e}")
    finally:
        print("final")

model = mii.serve(model_dir, deployment_name="qwen", tensor_parallel=xx, replica_num=replica_num)

job_process = []
for rank in range(0, replica_num):
    if rank == 0:
        job_process.append(threading.Thread(target=worker,args=(rank,model,)))
    else:
        job_process.append(threading.Thread(target=worker,args=(rank,None,)))
for process in job_process:
    process.start()
for process in job_process:
    process.join()

When using threading.Thread, it works well. However, it will be blocked in client.generate if using multiprocessing.Process.

Since the threading.Thread is fake in python due to GIL, this code can not make full use of concurrency. It means that I still need multiprocessing.Process to start a new client. However, it does not work well mentioned above.

I find the official example. Maybe we should start the server and clients like these ways.


Rangeview augmentation

Hi there,
Nice work and thanks for sharing the code!
I wonder if there's any code for the rangeview augmentation mentioned in the paper? Thanks!
Best,
YS

data_augment.txt
This is our internal aug code, but may be not completely compatible with current code. FYI.


New NPM Module -- File Not Found 404

Not able to import new custom module

This repository is now deprecated. The Angular Quickstart project was a nice starting point for creating Angular applications. Now we recommend using the Angular CLI to create new Angular projects.
Starting from 1 November 2017, all the Angular documentation, at angular.io, is based on the Angular CLI.
Let's get started


Cacti 1.x special map settings UI doesn't work at all

Clicking on the Settings link for a map brings up the right page. Adding a new setting results in a blank page saying "there are no settings for this map", and no settings added.
Same for group-level settings and global ones.

Adding entries manually to database, they even show up in the wrong places. Maybe a PDO/WeathermapManager issue?


[Security] "Require Unlock" isn't working when opening app

Steps to reproduce
Enable "Require Unlock" setting on Settings -> Security.
Lock when idle = Always
Close the app and open it again.
Expected behavior
TachiyomiSY should display the "Unlock Tachiyomi" screen (Android's "insert password/unlock device" screen) when opening the app.
Actual behavior
After upgrading to Preview r479, TachiyomiSY isn't displaying the "Unlock Tachiyomi" screen, and Tachi opens without requiring to insert the password/unlock the device, even with the aforementioned settings.
Crash logs
No response
Tachiyomi version
Preview r479
Android version
7.1.1
Device
Samsung Galaxy J2 Pro (SM-J250M)
Other details
The "Unlock Tachiyomi" screen appears normally when minimizing and then returning to the app. (same behavior as before)
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

This is a main tachi bug, you should open this issue there if one isnt already open

No problem, thanks for the info.


Improvements on requirements discory for host

It would be interesting to add a way to recover the modules loaded after the execution of the setup.py. So grayskull will be able to analyse all the modules loaded to install the package

Recipes to test with:

pytest
ciso
gsw


Tag the source

It would be very helpful if you could tag releases as well. This would enable distributions who want to run your tests to fetch the package from GitHub instead of PyPI if they want.
Thanks

Hello,
I've taken note of your comment and will create releases for new versions from now on.


Rutorrent webui timing out. Same issues as #355

So I ran the script and everything went well then i rebooted but when accessing rutorrent by webui it's giving me connection timed out error. So I reinstalled rtinst script but still facing same issue.  Using (Ubuntu18.04 lts)
Summary of Installation (Important Information, please read
SSH Configured
SSH port set to<>
root login directly from SSH disabled
login with lelouch and switch to root using: sudo su
FTP Server
vsftpd 3.0.3-9build1 installed
ftp port set to<>
ftp client should be set to explicit ftp over tls using port <>
rtorrent torrent client
rtorrent 0.9.8 installed
crontab entries made. rtorrent and irssi will start on boot for lelouch
RuTorrent Web GUI
RuTorrent 3.9 installed
rutorrent can be accessed at https://<>/rutorrent
rutorrent password as set by user
to change rutorrent password enter: rtpass
If enabled, access https downloads at https://<>/download/lelouch
IMPORTANT: SSH Port set to <>
IMPORTANT: SSH Port set to <>
IMPORTANT: SSH Port set to <>
Please ensure you can login BEFORE closing this session
The above information is stored in rtinst.info in your home directory.
To see contents enter: cat /home/lelouch/rtinst.info
To install webmin enter: sudo rtwebmin
SCROLL UP IF NEEDED TO READ ALL THE SUMMARY INFO
PLEASE REBOOT YOUR SYSTEM ONCE YOU HAVE NOTED THE ABOVE INFORMATION
Thank You for choosing rtinst

Can you run sudo rutupgrade to upgrade to latest ruTorrent 3.10 and see if that solves the issue? This issue likely not related to rtinst itself, but rather ruTorrent. So, ask for input over at https://github.com/Novik/ruTorrent/issues if upgrading to the latest ruTorrent doesn't solve the issue.

So I upgraded it to 3.10 but what's interesting is if run cmd rt it shows rtorrent is running but still facing the same issue . Also facing same issue as connection timed out when accessing webmin.
Also do you have discord or something for contact might help . Thanks

Nope, don't have discord. So, rTorrent is working fine through terminal and you can add/remove torrents and so on? If you get the same connection timed out with both ruTorrent and Webmin, the issue might be related to NGINX. Which version of NGINX are you running? Is it nginx-full 1.14.0-0ubuntu1.7? Can you post the whole error message you get? In addition, take a look at NGINX log (https://docs.nginx.com/nginx/admin-guide/monitoring/logging/) and post them here.
You can try to install a newer version of NGINX as your version is out-dated. Do the following (https://nginx.org/en/linux_packages.html):
echo "deb http://nginx.org/packages/ubuntu `lsb_release -cs` nginx" | sudo tee /etc/apt/sources.list.d/nginx.list
curl -fsSL https://nginx.org/keys/nginx_signing.key | sudo apt-key add -
sudo apt update
sudo apt install nginx-full

solved the issue. The issue was it did not accept connections for ip through the port had to open it. Also after i reinstalled os and script is working great now :) Thanks

Glad to hear!


Filemask "*" overriding specified filetypes

Currently in master:easygui/boxes/fileboxsetup.py, line 67, allFileTypeObject is being inserted into position 0 of filetypeObjects. This causes the filemask "*" to be the first option and the specified filetype as option, which can be activated with a dropdown. I think it should be inserted at position -1.
Example:
yml_file = fileopenbox(title="Select YML file with log2settings flags", filetypes=["*.yml"],multiple=False)

results in

Good suggestion. Thanks also for the clear screenshot and code snippet.


Allow custom shapes

This addresses an issue concerning the upstream dependency of euro-calliope on the output of this workflow. Namely, the introduction of custom shapes. I've gone for quite a simple approach:

You can define a new value that isn't related to nuts, gadm, or lau in the layer config (e.g. my-custom-shapes)
You can define a location for the shapefile for that new value in data-sources by using the same string name (my-custom-shapes: path_to_shapefile.geojson)
The rule will then build administrative_borders.gpkg based on all data sources requested from config. This allows it to add in new layers for any custom shapes and to avoid building a layer entirely if not requested by a layer in config. For example, one could remove the need to consider lau units by not having the municipal layer in their config.

This has led to injecting a python script into the snakemake rules, which has then led me to restructure the renewablepotentialslib package to handle that in a more lightweight way.
Two potentially brittle parts:

it expects any trailing digits on the data source description to be indicative of the layer in the data source (e.g. nuts2 -> nuts, but also it will think of my-custom-shapes999 as referring to data source my-custom-shapes). I think this is fine, as the process will fail if it removes trailing digits and then can't find a corresponding data source.
it expects the user-input shapefile to correspond to the layer schema (below) + a level column, referring to either the data source name (e.g. my-custom-shapes) or level number (e.g. 2 for my-custom-shapes2). The level part could be optional, to default to the source name if not in the geodataframe.

{
    "properties": {
        "country_code": "str", # id of the country to which the unit belongs
        "id": "str", # a unique id of this unit
        "name": "str", # the name of the unit, not necessarily unique
        "type": "str", # the type of the unit
        "proper": "bool" # flag indicating proper administrative unit (not the case for water bodies e.g.)
    },
    "geometry": "MultiPolygon"
}

I have now moved this function out of the lib and it sits instead in with the snakemake rules in a specific util file. I have updated the testing mechanism to be able to cover this, at least within the snakemake test rule. This repo could probably restructure according to https://github.com/calliope-project/euro-calliope/pull/120 in the longer run.


Fix: Remove directly import of font-family in root

This adjustment prevents discrepancies when you need to change the font.
Before, if you changed only the $fontFamily variable, it was not considered, as it was imported directly into :root.

This has been rolled back as it breaks all styles in the core build. We can think of a different solution.


Added missed windows libs

looks good to me, thanks for contributing recently, I really appreciate the effort :)
Also is it possible if you can add your own credits to the credits section of the README? If you wanna add it then you can append it with your user on it, but it's your choice. I'll merge it but I kinda wanna kill 2 birds with one stone with a single PR so yeah, lmk

thanks for kind suggestion, appended
and lets leave the birds alive :)

thanks for kind suggestion, appended and lets leave the birds alive :)

haha, sounds good :smile:


[cfg] Add back in bool return value, returning a value to it this time.

EOM

@swift-ci smoke test and merge


Add G4S account for cap data warehouse

G4S want to start with cap_dw system so adding in their account.

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform


Does not work anymore

The latest package version does not work anymore (Firefox 84)
Apparently <a> button (if created) is not clicked automatically anymore because nothing happens after a selecting the start number and pressing "ok"

Hey, @drzraf thanks for using WaGE. Unfortunately, I'm not able to spare my time to work on this anymore :(

Hey, @drzraf thanks for using WaGE. Unfortunately, I'm not able to spare my time to work on this anymore :(


Migration support for elasticsearch

provide compatible migrate command
helper methods for simple schema changes
allow convenient re-indexing for more complex changes http://www.elasticsearch.org/blog/changing-mapping-with-zero-downtime/

:+1: for this one

Migrated to https://github.com/yiisoft/yii2-elasticsearch/issues/38


Show message length limit indicator in the "edit message" UI

We should show a message length limit indicator in the "edit message" UI, just like we do in the compose box.
PR to continue from: #19535
CZO thread

@alya , i would like to know if this issue is still open.

Yes, though of course the design should match the current design for the counter:

We can place it next to the "cancel" button, I think.

@zulipbot claim

@opmkumar assigning to you, but please remove the assignment if you are not currently available to pick this up.

@opmkumar assigning to you, but please remove the assignment if you are not currently available to pick this up.

Yup, I am currently working on this issue.


#554

Related Issue
[#554]
Description
[File move to respective directory and change path]
Type of PR

[x] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

There have some conflict issue. check it once


Revert "chore(deps): bump langchain from 0.1.4 to 0.1.11"

Reverts yihong0618/xiaogpt#455

之后我手动改吧


Incorrect documentation in RtlStringCchVPrintfExA and RtlStringCchVPrintfExW

RtlStringCchVPrintfExA and RtlStringCchVPrintfExW doesn't have pszSrc parameter, but both documents has following descriptions.

If set, either pszDest or pszSrc, or both, can be NULL. NULL pszSrc pointers are treated like empty strings (TEXT("")), which can be copied. NULL pszDest pointers cannot receive nonempty strings.

Neither pszSrc nor pszDest can be NULL unless the STRSAFE_IGNORE_NULLS flag is set, in which case either or both can be NULL. If pszDest is NULL, pszSrc must either be NULL or point to an empty string.

These descriptions may be copied from RtlStringCchCopyExA which has both pszDest and pszSrc parameters.

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

ID: 0f25ff3a-8591-3d14-4588-8c393903c416
Version Independent ID: 14e68879-487c-8744-5b8a-291785fd1c70
Content: RtlStringCchVPrintfExW function (ntstrsafe.h) - Windows drivers
Content Source: wdk-ddi-src/content/ntstrsafe/nf-ntstrsafe-rtlstringcchvprintfexw.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks!!!


Sundry

Note that sequenceArrList, while unsafe, doesn't need the ArrowChoice instance, which is an advantage over sequenceArrListZip. Maybe we can keep both, and make sequenceArrList internal (not public)? It can be safely accessed over the Vec interface.

Note that sequenceArrList, while unsafe, doesn't need the ArrowChoice instance, which is an advantage over sequenceArrListZip. Maybe we can keep both, and make sequenceArrList internal (not public)? It can be safely accessed over the Vec interface.

That's true, good point. I agree lets keep sequenceArrList for internal use, which makes sequenceArrVec not need ArrowChoice. I think you just need to add sequenceArrListZip to the export list, right ?

I'm ready to merge, if you think all is okay.

I've added a few things ;)


Error parsing

I have the following code, and \begin{figure} is highlighted as unexpected token.
% Command for centering image, keeping aspect ratio
\newcommand*{\includeimage}[2][]{%
	\centering{
		\begin{figure}
			\includegraphics[width=\textwidth, height=0.8\paperheight, keepaspectratio]{#2}
		\end{figure}
	}
}

@olegs Thank you for reporting. Issue has been fixed and will be available with the new v0.3 release.

@hsz Thanks, looking forward for the new releases!


Can't map over an array in Dialog

Trying to map over an array to produce dom elements inside a dialog.

[ x] This is a v1.x issue (v0.x is no longer maintained).
[ x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

I expect element to be repeated as usual.
Current Behavior

When I map over an array none of the dom elements show.
Steps to Reproduce (for bugs)

You can see code here https://codesandbox.io/s/5z22jl7354?module=%2Fdemo.js
I simply took the text field dialog example and added an array to map over. I would expect to see multiple text fields but I don't. Same result with divs and such in my personal project.
Context

Your Environment

Tech
Version

Material-UI
v1.x

React
16.3.2

browser
chrome

-{arg.map(a => {
+{arg.map(a => (
  <TextField
    autoFocus
    margin="dense"
    id="name"
    label="Email Address"
    type="email"
    fullWidth
  />
-})}
+))}

@Nick-t-go You need to return something. It's a JavaScript issue, not a Material-UI issue :).


Selex for reference points should be weighted averages of F

Selectivity used to compute reference points should be weighted by the terminal year fleet-specific Fs (lines 281 - 283 in PR) (i.e., (sel_hal * hal_f + sel_trwl * trwl_f) / max((sel_hal * hal_f + sel_trwl * trwl_f))).

Fixed with 9ea8953


Adding Blazor

Any plans to add another fronted type like https://github.com/bradwellsb/blazor-contacts blazor server?

Yes. Adding Blazor is on the way.


Fix clang-tidy errors in RegexAST.hpp.

Add GSL for gsl::at and gsl::owner
Use [maybe unused]
Use std::array in place of c-array
Follow rule of 5
Did not move RegexAST.tpp into RegexAST.hpp yet to reduce the size of the PR. In following PR will fix RegexAST.tpp, then another PR to combine them.

For the PR title, how about:
Fix clang-tidy errors in RegexAST.hpp; Add Microsoft GSL as a dependency.


Support all operators

https://www.nushell.sh/book/operators.html
Some of the operators on the official documentation are not currently supported by the parser. Some examples include floor division (//), mod,  bit operators (like bit-or).

To add on (so that everything is consolidated in one place), flags for keywords like while, for etc are not supported yet.


Allow bicep to transpile its own .bicep files, to bring parameters/variables to the top of the file

As Bicep works as a transpiler of the .bicep file into an ARM template, and it does not have any positional requirements for parameters & variables, it would be nice if I could write .bicep files keeping the variables with the relevant resources but then also have a command (eg. bicep recompile .\main.bicep or bicep transpile .\main.bicep) which reorders the content of the file to have all parameters at the top, then the variables, then targetScope (if it exists), then the resources (in the order they were already in)

This will be covered by the Bicep linter, which will include canonical formatting rules and associated code fixes. Once the code fix capability exists, we are also planning some sort of "fix all" command in the CLI (and VS code) that would take care of it. An alternative approach would be to allow customization of the auto formatter.
Either way, the behavior should be configurable so users who want it can opt-in.

Linking to #428 for context

Duplicate of #6602

@anthony-c-martin I've jut had a read of #6602, and while it does look relevant I dont think this is duplicate of that item (unless i'm miss-reading it?).
Having said that i'm not too tied to the need for a transpile option any more, so happy for this to stay closed if it's no longer a valid use-case or priority.

@NWiddup, apologies - I missed that this was specifically about adding a CLI command. The linked issue proposes adding a linter, which would have a code fix to automatically fix the issue. This would provide the functionality you're looking for in the editor (VSCode), but not via a CLI command.
Something we could look to introduce in the future is a way of running linter fixes through the CLI, which would address both needs.

Ah cool - understand. Happy for this to stay closed then & I'll follow the other one


The idea of this pull is to make decoding capability as a plugin.

The decoding capability doesn't necessarily be for barcode. Perhaps face recognition, fingerprint scanning and 3rd party secure code.
Changes made

Removed ZXing from Camera.MAUI and moved it into a new project as plugin.
Added Google MLKit as a new project as plugin.

This would be amazing. Would love to see a Tesseract plugin for OCR on image data.


[BUG] oc-mirror authentication error intermittently with "unauthorized: Access to the requested resource is not authorized"

Version
$ oc-mirror version
Client Version: version.Info{Major:"", Minor:"", GitVersion:"4.11.0-202205101549.p0.gfa91e43.assembly.stream-fa91e43", GitCommit:"fa91e43d39da0630296fe74cfe756fc0ed7d6756", GitTreeState:"clean", BuildDate:"2022-05-10T18:07:11Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}

What happened?
This error is received intermittently.
error: unable to push registry.redhat.io/advanced-cluster-security/rhacs-roxctl-rhel8: failed to retrieve blob sha256:e3bdc6c23d58940ba064805bbbc9b5842b2cd11bba834312f00245c1935e5826: unauthorized: Access to the requested resource is not authorized
error: unable to push registry.redhat.io/advanced-cluster-security/rhacs-roxctl-rhel8: failed to retrieve blob sha256:4f4fb700ef54461cfa02571ae0db9a0dc1e0cdb5577484a6d75e68dc38e8acc1: unauthorized: Access to the requested resource is not authorized

What did you expect to happen?
No error
How to reproduce it (as minimally and precisely as possible)?

apiVersion: mirror.openshift.io/v1alpha2
kind: ImageSetConfiguration
mirror:
  operators:
    - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.9
      packages:
        - name: elasticsearch-operator
          minVersion: '5.4.1-24'
          channels:
            - name: stable
        - name: rhacs-operator
          channels:
            - name: latest
              minVersion: '3.69.0'

$ oc-mirror --config imageset.yaml file://

Anything else we need to know?
Originally reported as BZ, duplicating here for project planning
References

https://bugzilla.redhat.com/show_bug.cgi?id=2063154
/kind bug

I've tried to reproduce this several times and have not been able to.

@afflom This appears to be an issue with oc and quay integration. So far this issue has only been reported by users that are pushing to Quay or mirror-registry. It would be best to start testing there.


Exist is not working on element

Currently working on the automation of report download from website. After passing parameters when click on the Run report, it opens the new tab and generate the PDF embed report. During the report generation process trying to find out element //html/body/embed element exits or not so that I can do the downloading process. But it is not showing it element is exists even its present.  if I open the developer tool and find the element using XPath , it is showing but not in tagui.
Can you please help me ? I already raised another issue related to this, to download embed PDF file.
Thanks is advance.

Sorry I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
I've replied to your other issue https://github.com/kelaberetiv/TagUI/issues/819
From the description, it sounds like you are trying to interact with a popup tab. In that case, the step popup is required - https://tagui.readthedocs.io/en/latest/reference.html#popup
Otherwise TagUI will not know which tab you are referring to when providing the identifier. 2 different tabs can have the same identifier and thus it will not be unique. By using popup step, you can set which tab to automate.
Alternatively, using visual automation and keyboard step would be easier to do these download and save PDF workflow.
Closing issue for now, but please lemme know if you have any questions.


Suggestion: Create list of ao3 resources to add

Tags and user profiles now supported; it would be nice to have a list to reference of what's missing for people to chip away at, and by "people" I mean "me because my brain refuses to produce a list of things I might want from ao3 in a vacuum".

Working on this now: see #9 , #10


TJSheets export shifted to invalid cells

The repro steps given were:

Open existing QBJ file in MODAQ
Export the reloaded game to TJ Sheets

This is what the export looked like:

I couldn't reproduce this.


.debug() not working in custom configurations

Describe the bug
Not sure if this an issue of CA or something I might have missed but I am facing the following:
I have created a new project, renamed debug scheme to dev and copied it to create another one called staging.
I have also edited each scheme to use the correct configuration.
I did not edit/delete/create any new Custom Flags / Active compilation conditions.
When I call .debug() in one of my reducers it does not print anything to the console.
I have also tried to use #if DEBUG directly on my project, which surprisingly, works.
To Reproduce
The behaviour I am seeing can be reproduced here:
CADebugBug.zip
Expected behavior
.debug() should print on the console.
Environment

Xcode [Version 11.6 (11E708)]
Swift [5.2.2]
OS: [iOS 13.6]
CA version: 0.7.0

I've been experiencing this too.

I have noticed this too in a similar setup.

IIRC SPM doesn't support custom configurations for now (only debug and release) and Xcode tries to map your configuration name to either debug or release. If you rename your configuration to DevDebug Xcode might build your SPM dependencies with the debug configuration. You might want to differentiate configurations from schemes and use only two configurations and a scheme per environment.

I solved by renaming the configuration to Debug for now. Thanks for the pointer @AlexisQapa!

This is a known issue being tracked by Apple

https://forums.swift.org/t/swift-package-manager-and-custom-build-configurations/29181/5
https://forums.swift.org/t/custom-build-configuration-names/29167/2

I'm going to convert this to a discussion in case folks would like to share their various workarounds.


imod.rasterio.write and non-equidistant grids don't play nice

In GitLab by @Huite on Jul 30, 2019, 22:52
imod.rasterio.write requires equidistant grids. Modflow supports non-equidistant grids, as do IDFs, so it's quite possible that DataArrays are not equidistant. Currently, this results in an error, when dx or dy is not a scalar value. It ought to provide a more informative error, maybe with a suggestion to regrid, or reindex.
Additionally, it'll even even fail if the grid is equidistant, but dx and dy are single valued arrays.

In GitLab by @Huite on Sep 2, 2019, 22:34
Fixed by ca9bf826faa4b7a67c95e6fbf99fe16c09b60b15

In GitLab by @Huite on Sep 2, 2019, 22:34
closed


Build

What is your question?
What Is The Command Line To Run The Build/Installed Version Of Pysch Engine Using Haxelib

You mean.. "Lime test windows"?

To build, use Lime build windows

you can also change "windows"`` to your os


Improvements for services-cache

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ConstraintError thrown on 'put'/'add' or 'bulkPut' operations - Nondescript error message.

Hi, first thanks for this great library. We're encountering a strange issue, on some clients but not others, when using bulkPut or trying to update or store any new data. But are having difficulty tracking down the source of the issue because of vague error messages and inconsistent reproducibility.
We're trying to refresh data stored in a Dexie IDB table (it's identical except one field has been modified), but are receiving errors on constraints for some reason. It seems once the database gets into this state, no more values can be added whatsoever, though reads do work.
Simply doing this, where records is an array of values that do not yet exist in the database (of the same format / integrity that has been stored up to the point where errors start occurring):
await this.database.cards.bulkPut(records)

Results in:
{\"_e\":{},\"name\":\"BulkError\",\"failures\":[{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{}],\"message\":\"cards.bulkPut(): 45 of 45 operations failed. Errors: ConstraintError: A mutation operation in a transaction failed because a constraint was not satisfied.\"}

However, this error message does not say which constraint was not satisfied - and that information would be very helpful! I suspect it's not actually a real constraint error, but rather something else happening. I could be mistaken however.

The same code works 90% of the time, but seems to get into this state after 10,000 or so records have been stored -- only sometimes. It's not consistent.
Would appreciate any thoughts or tips on this? Is there a way to find out which constraint was not met?
Database definition:
export class CardDatabase extends Dexie {

  static LATEST_VERSION = 1;

  readonly cards: Dexie.Table<Card & HasQueriedTimestamp, string>;

  constructor() {
    super('cardcache');
    try {
      // INFO: https://dexie.org/docs/Tutorial/Design
      // WARNING: DO NOT MODIFY THIS. CREATE A NEW VERSION WTIH UPGRADE BELOW
      this.version(CardDatabase.LATEST_VERSION).stores({
        cards: '&id,collector_number,name,oracle_id,set_code,[set_code+collector_number],queried'
      });

      this.cards = this.table('cards');

    } catch (error) {
      console.error('Failed to init Card Database', error);
      throw error;
    }
  }
}

Additional note: Deleting or renaming the IDB database instance and letting it be re-created by the code above will get things working again, with the same data being saved.

The error message when attempting to store objects one at a time:

Code above was latest NPM 3.0.3 version of Dexie, Same thing using the new Alpha.9 version:

Calling db.clear() also appears to resolve the issue. Though in doing so, it may have revealed what's going on...
It looks like the index type somehow changed to 'number' (incorrect), from 'unique string uuid' (correct)

Should be:

I'm not sure why this would happen though.

Interestingly enough. This database does not want to be deleted. All attempts to delete it silently fail. Even using straight JavaScript via developer tools: indexedDB.deleteDatabase('cardcache')

If it helps anyone, the web application was installed as am iOS PWA / Home-Screen bookmark on an iPad. So I had to delete the bookmark/installed app to remove the database. I could not clear the data via the normal Safari -> Advanced -> Website Data approach since the app did not show up in the list of sites to clear. After completely deleting the app, and re-"installing" the bookmark, the database was re-created with correct index keys. Not sure how this happened. Will keep an eye on it in the future.

ConstraintError in IndexedDB is a result of violating primary key uniqueness or a unique index. Put operations can never violate primary key uniqueness though. If you still get this in a put operation and the only unique index you have is the primary key, then there seems to have been some instable state of your database. Safari has had a hisitory of unstable IndexedDB implementations but much efforts seems to have been made on it to make it much more stable for every version from Safari 10 and forth. The latest instability I know of was in version 11.
How is your indexeddb schema defined? Are you using any addon? There is no native support for auto-generated UUIDs as primary keys unless you are using an addon.

Hey @dfahlander - Thanks for taking a look. Definitely at least partially familiar with Safari's .... "implementations" ... of IndexedDB. It has definitely gotten better as of late, thankfully. The days of database version changing spontaneously and constantly doubling in size seem to be over :p
Schema is defined as seen above:
this.version(CardDatabase.LATEST_VERSION).stores({
   cards: '&id,collector_number,name,oracle_id,set_code,[set_code+collector_number],queried'
});

I'm now actually thinking that it's possible the schema may have been incorrectly defined on these devices during some kind of prototyping build that somehow made it to production before the schema was finalized. Because all of my local testing shows consistent behavior on every device I try, and the only way to reproduce the issue, as far as I can tell, is to intentionally incorrectly define the initial schema, and not do a proper upgrade.
I believe there were some attempts at using both an auto-increment id and a provided UUID key, but we opted to go with just the UUID as there was concern over performance issues when not performing queries via the primary key.
If this theory is correct, the schema could have been defined using the incorrect auto-increment primary key, via the erroneous hot-fix build, and then when the correct build went out and in a sense, corrected the issue, the databases that were defined with the incorrect schema would appear to start spontaneously breaking once they received a the correct patch.
I'm willing to call this user error unless it comes up again.

If a previous version of your app did define a unique index else than the primary key, and had the same value of CardDatabase.LATEST_VERSION - then a bulkput could result in ConstraintError. In the log, it would also warn for not incrementing the version number when changing schema ( if using dexie 3.x ).

Ill take a look, but I think WARN logs are disabled in prod deployments, but I think that could be the issue.
Regarding the subject of this issue. Is there any way to resolve or report in which constraint failed when attempting to put/add data? I'm guessing IDB isn't that specific when rejecting entries (if it even does.) But. Never hurts to ask!

The browser's indexeddb implementation generates the error message. But there is only one constraint in indexeddb, and that is index uniqueness for indexes marked as unique. add operations can fill if providing a duplicate primary key.

index uniqueness for primary keys and indexes marked as unique

That's good to know, and couldn't you have more than one unique constraint? E.g. https://dexie.org/docs/API-Reference#quick-reference -> using the & modifier on multiple indices when defining table schema? Or am I misunderstanding?

Add-operations (but not put) can fail with ConstraintError if providing a duplicate primary key.

Yeah ok good, that's what I thought, and why I was super confused to see this issue with bulkPut.

I meant there is only one type of constraint. You can have many unique indexes.

@dfahlander AH. Okay haha. We're on the same page. Right. So then based on what you said about IDB reporting the error, I'm guessing there's no way to tell which one was violated during an add/put. Thanks! Much appreciated.
PS. Really interesting to see what you're doing with Dexie Cloud. Keeping my eye on it :)

There's no way to know which index that was violated but there's a new property failureByPos in BulkError that maps the errors to the position of the object given to bulkPut (in dexie@=>3.1.0-alpha.7). See BulkError.


Variants: Add to cart buttons are misaligned on the grid view of PLP

Environment Details
Spartacus: variants branch
Browser: Chrome
Device: mac
Happens on: http://dev-com-15.accdemo.b2c.ydev.hybris.com:4200/apparel-uk-spa/en/GBP
Steps to Reproduce

On chrome, go to a category listing page, for eg: http://dev-com-15.accdemo.b2c.ydev.hybris.com:4200/apparel-uk-spa/en/GBP/category/220000
Go to grid view

Actual Result
The alignment of the "Add to cart" button

is not consistent.

asking Anthony

after talking with jeanne and miguel,
step 1:

with pictures aligned to the top, align the Addtocart buttons to the bottom of the imaginary box
(if you need help from miguel we'll wait for next week)

step 2:

the layout gets quite airy (add to cart button seems connected to the image underneath for example); let's put a box or a line around the items in the grid. however Miguel will investigate the easiest/best/safest approach to this and then we'll share findings.

so for this ticket, let's do step 1.


Update the number of nuget packages downloads on the Platform website

The number of total downloads of packages on the About us page of the Platform website should be updated to 3.7M now:

the line that needs to be changed is in the following link:
https://github.com/bitfoundation/bitplatform/blob/27634e0ffb8ea149275016b9dca2533119c59a5d/src/Websites/Platform/src/Bit.Websites.Platform.Web/Pages/AboutUsPage.razor#L49

@msynk is it ok for me to take care of this issue?

@zahra-yousefi sure, go ahead 🚀


Add json file in derivatives/labels?

Do we need / want to add a json file for each label file?
Which could contain rater name and date for example?

Ah geeze I forgot about this :-(. Such a pain to create those. I’m tempted to say that GH does inform about this, but once the data is downloaded the info is not visible. So yes, maybe we should enforce that.

will be addressed in https://github.com/spine-generic/spine-generic/issues/143

Fixed in #143


Fix Helm Error 403 on chart repository

A user reported getting errors when trying to destroy an old qhub deployment using terraform destroy -auto-approve, more details on the issue on #218.
The probable cause is Helm (see epic #224) and a possible solution can be found at Stack Overflow.

This is resolved. Based on the comments in this PR, my guess is the core problem was the use of helm version 2.x. I was unable to reproduce the problem before merging the above PR.
The PR for the terraform modules was merged in any case to explicitly pull from the latest URL for helm charts.

This is resolved. Based on the comments in this PR, my guess is the core problem was the use of helm version 2.x. I was unable to reproduce the problem before merging the above PR.
The PR for the terraform modules was merged in any case to explicitly pull from the latest URL for helm charts.

This is resolved. Based on the comments in this PR, my guess is the core problem was the use of helm version 2.x. I was unable to reproduce the problem before merging the above PR.

By the looks of it, he just noticed there was no Helm installed on his dependencies. Check out #218 again.

This is resolved. Based on the comments in this PR, my guess is the core problem was the use of helm version 2.x. I was unable to reproduce the problem before merging the above PR.

By the looks of it, he just noticed there was no Helm installed on his dependencies. Check out #218 again.

@laisbsc and @tylerpotts , I upgraded helm in my qhub environment, but I'm still getting the error:
Error: failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden

when I try:
cd infrastructure
terraform destroy

Do I need to do terraform render again?

@laisbsc and @tylerpotts , I upgraded helm in my qhub environment, but I'm still getting the error:
Error: failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden

when I try:
cd infrastructure
terraform destroy

Do I need to do terraform render again?

Do I need to do terraform render again?

I reckon that could sort out the problem, @rsignell-usgs. Do you mind re-rendering and coming back to us?
Chat soon!

Do I need to do terraform render again?

I reckon that could sort out the problem, @rsignell-usgs. Do you mind re-rendering and coming back to us?
Chat soon!

@laisbsc , okay, here's what I've done so far:
After upgrading helm, I did:
  qhub render -c qhub-config.yaml -o test -f
  cp qhub-config.yaml test
  cd test
  cd infrastructure
  terraform destroy

This didn't work, told me I needed to run terraform init, so I did that and then ran terraform destroy again.
This ran for a awhile and then produced this error:
Error: Error deleting IAM Role (earthmap-dev-eks-node-group-role): DeleteConflict: Cannot delete entity, must detach all policies first.

    status code: 409, request id: 3ae09f82-6fc6-4d68-b850-fe31538cb013

@laisbsc , okay, here's what I've done so far:
After upgrading helm, I did:
  qhub render -c qhub-config.yaml -o test -f
  cp qhub-config.yaml test
  cd test
  cd infrastructure
  terraform destroy

This didn't work, told me I needed to run terraform init, so I did that and then ran terraform destroy again.
This ran for a awhile and then produced this error:
Error: Error deleting IAM Role (earthmap-dev-eks-node-group-role): DeleteConflict: Cannot delete entity, must detach all policies first.

    status code: 409, request id: 3ae09f82-6fc6-4d68-b850-fe31538cb013

@laisbsc or @tylerpotts , should I just go ahead and start deleting stuff (like this IAM role) using the  AWS console, or do I want to try to wait and fix via Terraform or AWS CLI?

@laisbsc or @tylerpotts , should I just go ahead and start deleting stuff (like this IAM role) using the  AWS console, or do I want to try to wait and fix via Terraform or AWS CLI?

@rsignell-usgs I chatted with @costrouc this morning, and he mentioned that he also ran into some problems going from helm 2.x to 3.x. His suggestion is to delete the .helm directory in your home directory and try again. I'd also suggest uninstalling helm 2.x off your computer if you no longer need it

@rsignell-usgs I chatted with @costrouc this morning, and he mentioned that he also ran into some problems going from helm 2.x to 3.x. His suggestion is to delete the .helm directory in your home directory and try again. I'd also suggest uninstalling helm 2.x off your computer if you no longer need it

@tylerpotts , I deleted the .helm directory and tried again, but still getting:
Error: Error deleting IAM Role (earthmap-dev-eks-node-group-role): DeleteConflict: Cannot delete entity, must detach all policies first.
        status code: 409, request id: d1374ebd-dbc5-48ec-be96-85474c5757e4

Should we keep trying or should I punt and use the console?

@tylerpotts , I deleted the .helm directory and tried again, but still getting:
Error: Error deleting IAM Role (earthmap-dev-eks-node-group-role): DeleteConflict: Cannot delete entity, must detach all policies first.
        status code: 409, request id: d1374ebd-dbc5-48ec-be96-85474c5757e4

Should we keep trying or should I punt and use the console?

Ah, I should have read your error more closely. This doesn't look like a helm error to me. I think this is AWS-ism that you will have to go into console to resolve manually.
In my experience, Terraform doesn't do well if the destroy is interrupted halfway and often requires manual deletion via console.

Ah, I should have read your error more closely. This doesn't look like a helm error to me. I think this is AWS-ism that you will have to go into console to resolve manually.
In my experience, Terraform doesn't do well if the destroy is interrupted halfway and often requires manual deletion via console.

I'm happy to report that:

I was able to destroy the rest of the infrastructure manually on our old qhub
I was able to simply retrigger the github action that failed and it looked like it pulled the updated qhub-terraform-modules and it succeeded.  So we are back in business over at https://jupyter.qhub.esipfed.org !   (and I added you guys as authorized users if you want to login and check it out)

I'm happy to report that:

I was able to destroy the rest of the infrastructure manually on our old qhub
I was able to simply retrigger the github action that failed and it looked like it pulled the updated qhub-terraform-modules and it succeeded.  So we are back in business over at https://jupyter.qhub.esipfed.org !   (and I added you guys as authorized users if you want to login and check it out)

I'm happy to report that

The happiness! <3 We're happy you got that sorted too. Wondering if this deserves a 'Troubleshoot' section on the docs.

So we are back in business over at https://jupyter.qhub.esipfed.org !   (and I added you guys as authorized users if you want to login and check it out)

I went ahead and checked the URL. And damn this is (happy) tear-inducing! The visualizations are so neat!
And I'm also delighted that we (as in Tyler) was able to help with the hiccup.
Take care and chat soon!

I'm happy to report that

The happiness! <3 We're happy you got that sorted too. Wondering if this deserves a 'Troubleshoot' section on the docs.

So we are back in business over at https://jupyter.qhub.esipfed.org !   (and I added you guys as authorized users if you want to login and check it out)

I went ahead and checked the URL. And damn this is (happy) tear-inducing! The visualizations are so neat!
And I'm also delighted that we (as in Tyler) was able to help with the hiccup.
Take care and chat soon!

@jkellndorfer and I are trying to delete our old qhub deployment, and using the above approach bombed out on the terraform destroy command with:

Error: Unsupported argument

  on kubernetes.tf line 2, in provider "kubernetes":
   2:   load_config_file       = false

what now, oh Qhub gurus?

@rsignell-usgs I'd suggest going into that kubernetes.tf file and deleting that load_config_file argument

@tylerpotts  you da man!  commented that out, ran terraform destroy again, got:
Destroy complete! Resources: 67 destroyed.

@rsignell-usgs Great news!
I think the problem is/was that the module for kubernetes has been updated and no longer supports that argument. This is part of the reason we're hoping to more tightly pin dependencies in this next release


Can't sort Templates or ISOs.

ISSUE TYPE

Bug Report

COMPONENT NAME
UI

CLOUDSTACK VERSION
4.16.0

CONFIGURATION
XCP-NG 8.2
Advanced Zone
OS / ENVIRONMENT
Oracle Linux 8.4 - Management server, MYSQL 8.0.26
XCP-NG 8.2 - hypervisor
SUMMARY
Can't sort Templates or ISOs.
In Templates section buttons are presented but when you try to click them nothing happends.
In ISOs section buttons are missed (were in old UI)
STEPS TO REPRODUCE
Try to sort templates.

EXPECTED RESULTS
Can place templates in any order

ACTUAL RESULTS
Can't place templates in any order. When you try to click sort buttons nothing happends. In ISOs section buttons are missed.

@utchoang can you check this please. thanks.

Through testing, I found the problem is that when getting listTemplates with showunique: true it sorts wrong. But when I remove this parameter, the returned data is sorted correctly, but the result is a sequence of duplicate samples. As far as I can see is it coming from the API?

With showunique: true

Without showunique: true

I think with API updateTemplate /updateIos to update sortKey need one more parameter to be able to update for duplicate templates/isos?

Through testing, I found the problem is that when getting listTemplates with showunique: true it sorts wrong. But when I remove this parameter, the returned data is sorted correctly, but the result is a sequence of duplicate samples. As far as I can see is it coming from the API?

With showunique: true

Without showunique: true

@utchoang With 'showunique: true', API returns unique templates irrespective of zones, and without 'showunique: true', all the templates in the zones are returned. So, if a template is registered for mutiple/all zones, it is present multiple times.
From UI, the templates are always listed with 'showunique: true'.

@utchoang Fixed the API response in the PR #5800.

Fixed with #5800. Closing the issue.


CASMINST-5403 use image with newer helm installed

Summary and Scope
Summarize what has changed. Explain why this PR is necessary. What is impacted? Is this a new feature, critical bug fix, etc?
Is this change backwards incompatible, backwards compatible, or a backwards compatible bugfix?
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves [issue id](issue link)
Change will also be needed in <insert branch name here>
Future work required by [issue id](issue link)
Documentation changes required in [issue id](issue link)
Merge with/before/after <insert PR URL here>

Testing
List the environments in which these changes were tested.
Tested on:

<development system>
Local development environment
Virtual Shasta

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?
Were continuous integration tests run? If not, why?
Was upgrade tested? If not, why?
Was downgrade tested? If not, why?
Were new tests (or test issues/Jiras) created for this change?

Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

Merge after https://github.com/Cray-HPE/hms-build-environment/pull/3 is merged and tagged.


Rundown - UI broken for large stacks and groups when user creates large number of stacks and groups

Adding screenshots and video clips

@GloandCo


Fix incorrect typing of the onOpen callback

Fix issues

[ ] #issuenumber

Description
Looking at the source code, the optional onOpen callback in the ColorPickerProps is called with a boolean which indicates whether the dialog is opened or closed.
The boolean was missing from the index.d.ts file
Check List

[x] respect code conventions and usages
[x] tests and 100% coverage
[x] well documented
[x] self review

Review targets:

nodejs / npm / yarn

@robin-xyzt-ai thanks !


Hide "latest entries" section header from safari's reader mode

The "latest entries" section is currently an <aside>, but perhaps it should be a <nav>?
Currently Safari's reader mode displays the <header> within that section, but not the rest of the contents. Making it a nav will probably hide it all.

Unfortunately making it a <nav> doesn't change anything.

I find it really interesting that the social-sharing aside's header (and its other contents) are completely hidden from reader mode, while the latest-entries aside's header is shown, but not the rest of the contents. Their basic html structure is the same, an aside with a header and then an unordered list... 🤷♀

Yeah... following above experiment ☝ I tried adding classname socia to the aside, and then it shows in reader mode. But when I add that last l completing the word social it is hidden...
Does anyone have a list of words in classnames that hides content from Safari Reader Mode, that'd be really useful? 😀

I've also tried the instapaper_hide classname (suggested by https://www.ctrl.blog/entry/browser-reading-mode-content.html) but that doesn't hide the element in our case 😞


Bug importing the library

I am trying to run sounderpy in colab, specifically the notebook "beginners_sounderpy_cookbook.ipynb". I run:
!pip install sounderpy
and when I run the second line: "import sounderpy as spy", I get the following error:

TypeError                                 Traceback (most recent call last)
 in <cell line: 1>()
----> 1 import sounderpy as spy
12 frames
/usr/lib/python3.10/dataclasses.py in _process_class(cls, init, repr, eq, order, unsafe_hash, frozen, match_args, kw_only, slots)
988         # Raise an exception if we're frozen, but none of our bases are.
989         if not any_frozen_base and frozen:
--> 990             raise TypeError('cannot inherit frozen dataclass from a '
991                             'non-frozen one')
992
TypeError: cannot inherit frozen dataclass from a non-frozen one
I've tried several things but I cannot find any solution. Has this happened to someone else? or would you have an idea of how to solve it? Thank you in advance.

Hmmmm, strange.
This is happening in Google Colabs? I haven't seen this error before. I am away from my computer today, so I wont be able to dig into this for a little bit.
If you are attempting to run the actual cookbook notebook, attempt to create a new blank notebook and try to install and import again. Let me know if you still get the error.

Okay, now it works with the new blank notebook. Thank you for your reply


Instances of inherited scenes are not updated properly after changing any property

Tested versions
Reproducible in: 4.3Beta1, 4.2, 4.1, 4.0
System information
Godot v4.3.beta1 - Linux Mint 21.3 (Virginia) - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Laptop GPU () - Intel(R) Core(TM) i5-10500H CPU @ 2.50GHz (12 Threads)
Issue description
This bug occurs when you change any property on a base scene that other scenes inherit from.
Expected behavior: If there are no overrides set on the instance itself then you should be able to -> update property on base scene  -> see updated property in inherited scene  -> see updated property in instances of inherited scene.
Current behavior: Update property in base scene -> see updated property in inherited scene -> see previous value (incorrect) on instance of inherited scene.
Workarounds-

Re-instantiate a scene.
Click the revert icon near property on instance.

Note: The small revert (or undo) icon appears on the instance of inherited scene even though no overrides were applied

Note: This bug only occurs with already instantiated inherited scenes. Newly instantiated inherited scenes will have the correct updated value.
Gif of bug in action -

Similar to:
https://github.com/godotengine/godot/issues/43032
https://github.com/godotengine/godot/issues/90418
https://github.com/godotengine/godot/issues/60846
Steps to reproduce
3 Scenes will be open in MRP. Main, Base, and Inherited.

Instantiate inherited scene as a child of Main.
Change any property on Base scene.
Check to see if property updated on Inherited Scene (not instance). It should show the correct property value.
Check to see if property updated on instance of Inherited scene (child of main).

Minimal reproduction project (MRP)
inheritedsceneminimalproject.zip

Demonstrated to me and confirmed as an issue that goes back to even Godot 3.6 . The expected behavior is that the instanced children's default values should reflect changes made to it's parent in the chain of inheritance. Currently, even if the values remain default, if the parent's values are changed they are not reflected by the children. This only applies to instanced children, not any of the inherited scenes directly.

Still present in 4.3 Stable


Update README.md

Easy run for linux users

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Please also read the contribution guide regarding DCO and semantic PR validations.

Would you like to close this pr and open another one for the contrib guide?

yes


Embedding layers with LSTM, GRU and Convo1D are added.

Fixes Issue🛠
Closes #109
Description👨💻
I added embedding layers with LSTM, GRU and Convo1D using Keras.
Type of change📄
I changed some part of code.

[] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
I tested my code by running locally on my computer.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I  am an Open Source Contributor

Screenshots/GIF📷

Pls add the plots and accuracies screenshots and also resolve the conflicts @Soumya1219

Pls run black main.py on your editor terminal to pass the Linting workflow and then commit the new indentation changes @Soumya1219

Pls resolve the conflicts @Soumya1219

Pls run black main.py on your editor terminal to pass the Linting workflow and then commit the new indentation changes @Soumya1219

You mean original main.py file?

Yes @Soumya1219

LGTM, Merged!! Congrats on getting your PR merged to this repo @Soumya1219

LGTM, Merged!! Congrats on getting your PR merged to this repo 🎉 @Soumya1219


Add demo.gif on README

This is much better.

Now use mp4 instead. https://github.com/OpenEmojiBattler/open-emoji-battler/pull/29

new shorter gif


Build Index page

As a feature requester, I want to be able to see a list of all tickets after I log in, so that I can quickly assess all the tickets before deciding to take further action.

As a feature requester, I want to be able to see a list of requested features after I log in, so that I can quickly assess all the open features before deciding to take further action.

[ ] Write a spec
[ ] Build a basic table UI
[ ] Build the data type
[ ] Make the query
[ ] Make some seed data


Support fetching witnesses via eth-tx-proof

Currently zero-bin allows fetching block trace data from a Erigon instance with the proper witness APIs as well as loading this trace data from a file and kicking off proofs.
To enable easier integration testing, as well as to give users a nice way to target chains which do not yet have the proper trace APIs, we would like to integrate the code from eth-tx-proof into zero-bin such that we can fetch information from an existing RPC node and kick off the proving process for a block or transaction.

Yeah if someone from your side needs it @Nashtare feel free to grab this.

I would be happy to pick this up if no one has already been assigned? In the future I would like to build a tracer integration for reth so I think this would be a good introductory issue to pick up. I would be doing this work on behalf of Fractal who intends to use the Type 1 EVM in the future.
Let me know what you think @Nashtare @pgebheim.

@frisitano Thank you for offering to tackle this, that'd be great! Note that eth-tx-proof is currently being refactored (including a lot of common logic with zero-bin) to support padding / withdrawals logic, see https://github.com/0xPolygonZero/eth-tx-proof/pull/15. Probably best to base off the work from this branch (should be merged soon).

@Nashtare Sure thing, I've made a start on this.

Ok, just wanting to let you know that the refactor PR mentioned about (#15) is merged back in.

Thanks for the heads up @BGluth.  I've got an initial implementation completed.  I need to refactor and complete some integration tests for more cases but simple transfers and empty blocks are proving fine.
Here's the PR I'm working on: https://github.com/0xPolygonZero/zero-bin/pull/51

I now have https://github.com/0xPolygonZero/zero-bin/pull/51 working on blocks produced by jerigon including smart contract interactions but for mainnet blocks I run into a KernelPanic error. I also experience similar KernelPanic errors when trying to prove mainnet blocks with eth-tx-proof. Do we have some specific test vectors / mainnet blocks that I can use to debug this issue?

@frisitano Which blocks are you trying to prove? The current zk_evm is Shanghai compatible, but does not support Cancun yet (it is implemented, and in process of debugging at the moment).

@Nashtare I'm proving block number 17735424 which is a Shanghai block. I'm using an alchemy rpc to source the witness data for both eth-tx-proof and zero-bin, both of which are resulting in kernel panic. Do you have any suggestions of how I can debug the panic?

@frisitano Just FYI, we're experiencing some bugs on eth-tx-proof side that @BGluth and I are investigating at the moment. zero-bin should be fine though, so the kernel panic may be coming from somewhere else. Would you mind sharing me the logs you obtained with it? (you may need to turn on RUST_LOG=evm_arithmetization=trace).

Which version of zero-bin are you using? Your prover is not processing any withdrawals (as in, it thinks there are none), while the block 17735424 has 16 withdrawals.
cf (cleaned up) end of your logs below, step at Cycle 387691 is checking if we loaded with prover inputs U256::MAX (sign of the end of the list), and we indeed did, which makes us jump to the final checks post execution.

Note that the error that makes the kernel panic is a gas used discrepancy. But given that you are missing withdrawals, which would have caused you to fail generating a proof anyway later on (state root mismatch with the block header), I'd first like to clarify the version used in case some recent fixes haven't been integrated.

I am confused, the transaction hash seems to be 0xc106edf40809d27dbaf88d7c305668f7fb2d03f236aeecf90481d328721b059c (this is the hash used during ECRECOVER signature verification, which passes successfully, but this doesn't appear to be in block 17735424 (or any block FWIW)... Are you targeting Ethereum mainnet?
This can be seen in this line (top of the stack is the txn hash)
Cycle 116868, ctx=0, pc=ecrecover, instruction=Dup(1), stack=[87308623151916798375216749502438337832795253495607303634714898271170639627676, 28,<PHONE_NUMBER>2052192061477373449743179050685448427087360131414109079395239272403,<PHONE_NUMBER>570036313773666492659193980943799342832065023943496369727278541227, 60042, 45179, 0, 128, 2, 1]

The mismatch is actually surprising. The memory cell that should contain the cumulative gas used post txn execution, i.e. typically the gas used by this txn as it is the first one, is instead set with the txn gas limit.
Would you happen to have your GenerationInputs corresponding to this txn? More specifically, the value of inputs.gas_used_after.

tx number before: 0
tx gas used after 365576

  2024-04-17T19:35:03.479761Z ERROR paladin::runtime: execution error: Fatal { err: KernelPanic in kernel at pc=panic, stack=[1, 2, 1], memory=[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 43, 112, 112, 18, 139, 144, 32, 231, 122, 76, 3, 111, 179, 111, 64, 224, 119, 249, 119, 106, 169, 109, 143, 94, 251, 152, 166, 125, 134, 54, 248, 24, 49, 229, 91, 22, 146, 213, 89, 153, 216, 199, 43, 213, 157, 248, 133, 179, 86, 123, 45, 119], strategy: Terminate }, routing_key: 5f23c8b763784f6aae9bceec611b604f
    at /Users/f/.cargo/registry/src/index.crates.io-6f17d22bba15001f/paladin-core-0.4.2/src/runtime/mod.rs:727

Error: Fatal operation error: KernelPanic in kernel at pc=panic, stack=[1, 2, 1], memory=[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 43, 112, 112, 18, 139, 144, 32, 231, 122, 76, 3, 111, 179, 111, 64, 224, 119, 249, 119, 106, 169, 109, 143, 94, 251, 152, 166, 125, 134, 54, 248, 24, 49, 229, 91, 22, 146, 213, 89, 153, 216, 199, 43, 213, 157, 248, 133, 179, 86, 123, 45, 119]

In regards to versions:
zero-bin - branched from commit hash 60f258d49106c2356536a90588820d9719cd24ea of develop
zk_evm - branched from HEAD of develop

Hmm interesting, tx gas used after should be 219,133 according to this. I'm curious about this parsing issue, eth-tx-proof gives me the valid value (0x357fd = 219133) and we never faced something like so on zero-bin side. 🤔

Good catch. I can confirm the transaction receipt object has the correct gas value that you quoted. I'll investigate and locate where this discrepancy arises and report back.

@Nashtare the erroneous value was associated with bad mapping on the adapter I built. I've fixed it and that is no longer an issue.
However, now I am experiencing the following issue which I believe is related to state root consistency checks at the end of the kernel.  See log extract below:
2024-04-18T14:42:52.693963Z TRACE evm_arithmetization::witness::transition: Cycle 546361, ctx=0, pc=49193, below mpt_hash, instruction=Push(3), stack=[0,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 32, 3724, 45259]
2024-04-18T14:42:52.693968Z TRACE evm_arithmetization::witness::transition: Cycle 546362, ctx=0, pc=49197, below mpt_hash, instruction=Jumpi, stack=[49203, 0,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 32, 3724, 45259]
2024-04-18T14:42:52.693974Z TRACE evm_arithmetization::witness::transition: Cycle 546363, ctx=0, pc=49198, below mpt_hash, instruction=Swap(1), stack=[6938084400579631288715715963269591884260219133622291964305563008489015499207, 32, 3724, 45259]
2024-04-18T14:42:52.693979Z TRACE evm_arithmetization::witness::transition: Cycle 546364, ctx=0, pc=49199, below mpt_hash, instruction=Swap(2), stack=[3724, 32,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 45259]
2024-04-18T14:42:52.693984Z TRACE evm_arithmetization::witness::transition: Cycle 546365, ctx=0, pc=49200, below mpt_hash, instruction=Swap(0), stack=[45259, 32,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 3724]
2024-04-18T14:42:52.693989Z TRACE evm_arithmetization::witness::transition: Cycle 546366, ctx=0, pc=49201, below mpt_hash, instruction=Pop, stack=[32, 45259,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 3724]
2024-04-18T14:42:52.693994Z TRACE evm_arithmetization::witness::transition: Cycle 546367, ctx=0, pc=49202, below mpt_hash, instruction=Jump, stack=[45259,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 3724]
2024-04-18T14:42:52.694000Z TRACE evm_arithmetization::witness::transition: Cycle 546368, ctx=0, pc=45259, below check_state_trie, instruction=Push(5), stack=[6938084400579631288715715963269591884260219133622291964305563008489015499207, 3724]
2024-04-18T14:42:52.694005Z TRACE evm_arithmetization::witness::transition: Cycle 546369, ctx=0, pc=45265, below check_state_trie, instruction=MloadGeneral, stack=[21474836490,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 3724]
2024-04-18T14:42:52.694011Z TRACE evm_arithmetization::witness::transition: Cycle 546370, ctx=0, pc=45266, below check_state_trie, instruction=BinaryArithmetic(Sub), stack=[61333081217480535940718494550699021384310283736156928076268435607788798060026,<PHONE_NUMBER>579631288715715963269591884260219133622291964305563008489015499207, 3724]
2024-04-18T14:42:52.694017Z TRACE evm_arithmetization::witness::transition: Cycle 546371, ctx=0, pc=45267, below check_state_trie, instruction=Push(3), stack=[54394996816900904652002778587429429500050064602534636111962872599299782560819, 3724]
2024-04-18T14:42:52.694022Z TRACE evm_arithmetization::witness::transition: Cycle 546372, ctx=0, pc=45271, below check_state_trie, instruction=Jumpi, stack=[60095,<PHONE_NUMBER>6900904652002778587429429500050064602534636111962872599299782560819, 3724]
2024-04-18T14:42:52.694033Z  WARN evm_arithmetization::witness::transition: Kernel panic at panic
Error: Fatal operation error: KernelPanic in kernel at pc=panic, stack=[3724], memory=[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 63, 201, 26, 58, 253, 112, 57, 92, 212, 150, 198, 71, 213, 166, 204, 157, 75, 43, 127, 173, 104, 210, 92, 16, 189, 45, 210, 5, 86, 52, 23, 176, 8, 86, 15, 35, 139, 122, 149, 68, 114, 55, 252, 239, 85, 231, 33, 100, 5, 63, 193, 3]

I've compared the transaction traces for this transaction with the state diff here and the data looks consistent so I'm not sure why it would yield a different state root.

I've attached the GenerationInputs for the transaction below:
inputs.json

I fixed issues related to gas and storage slot deletions. I'm working through the transactions iteratively and fixing bugs as I go. Will keep this issue updated with progress.

After some changes I have been able to successfully prove block 17735424. I'm currently using a branch from 'develop' for the prover but it looks like zero-bin currently uses v0.3.0. Are we happy to change zero-bin to use develop or  how would we like to proceed? The required changes to the prover can be seen here - https://github.com/0xPolygonZero/zk_evm/pull/157/files.

Awesome!
As for zero-bin deps, we only use released versions on develop (note that @BGluth will be yanking today trace_decoder v0.3.0 and publish the fix). We usually test things on zero-bin@testing which is regularly updated with latest zk_evm@develop branch.

Done with #81


Add build typescript GitHub action

Details are in PRE-RELEASE.md

This doesn't work for serverless functions - since github requires authorization, and npm install in serverless comes unauthorized


Running and browsing on cloud9

I added the running and browsing options for cloud9 users on Step3 and Step4. Please check it out and let me know if there is something wrong.

Looks good @broccolinisoup. These steps will work if the environment we choose in cloud9 is "Node.js" and not a blank one. So @lauragieco will have to mention in the talk that we need to choose Node.js dev environment in cloud9.
@lauragieco , please review


Helm version selection rules

Output of helm version:
Client: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.2", GitCommit:"cff46ab41ff0bb44d8584413b598ad8360ec1def", GitTreeState:"clean", BuildDate:"2019-01-10T23:35:51Z", GoVersion:"go1.11.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.6", GitCommit:"ab91afd7062d4240e95e51ac00a18bd58fddd365", GitTreeState:"clean", BuildDate:"2019-02-26T12:49:28Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):  AKS
We have a lot of charts deployed into dev helm repository using CI/CD gitlab pipeline.
The naming convention used for the develop env is X.Y.Z-SHORT_SHA_COMMIT .
When i try to install the chart using the command
helm install --devel --name chartname repo-develop/chartname

the version installed is, for example, 2.1.0-fa68129e that is not the latest ( date 2019-02-28T10:03:17.075Z )
The real latest chart is 2.1.0-c0821a70 ( date 2019-03-28T11:01:23.177Z) .
But the latest chart version is lexicographically lower than the previous one.
How i can solve this issue without specifiy the version in the install command?
Regards

You should probably change the versioning... as you say, commits are not lexicographicly sorted.
Two alternatives i can think of are the timestamp (choose your format), or gitlab's CI_PIPELINE_ID (which is more traceable).
Hope this helps!!

Closing as answered, but please feel free to re-open if @maorfr's answer did not answer your question. Thanks.


feat(stream): append-only TopN with ties

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
close https://github.com/risingwavelabs/risingwave/issues/5642
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

cc @BugenZhao PTAL


[* Unit] Units should not be lumped together into Volume unit and Weight unit

Describe the bug
The definition of a number of units are lumped together into a generic class like "Volume unit" or "Weight unit".  There is no reason to lump these units together.  I suspect it is because the definition of some of the units depend on country or context, but that is not a concern of the NLU layer.
To Reproduce
Steps to reproduce the behavior:

Add dimension entity to LUIS
Try an utterance like "2 quarts".

Actual: Units resolution is "Volume unit"
Expected: Units resolution is "Quart"
Expected behavior and proposal
Change https://github.com/Microsoft/Recognizers-Text/blob/master/Patterns/English/English-NumbersWithUnit.yaml#L3 as below.
Remove and change Volume unit.
    Volume unit: fluid dram|gill|quart|minim|cord|peck|bushel|hogshead|barrels|barrel|bbl

Dram: fluid dram
Gill: gill
Quart: quart
Minim: minim
Cord: cord
Peck: peck
Bushel: bushel
Hogshead: hogshead
Barrel: barrel, bbl, barrels

Remove and change Weight unit.
Weight unit: pennyweight\|grain\|british   long ton\|us short hundredweight\|stone\|dram

Pennyweight: pennyweight
British long ton: british long ton
US short ton: us short ton
US short hundredweight: us short hundredweight
Stone: stone
Dram: dram

It looks like this is an artifact of an old design decision and is no longer needed. We'll try to include the change in the next package release.

It looks like this is an artifact of an old design decision and is no longer needed. We'll try to include the change in the next package release.


Revert "Merge pull request #1733 from vrutkovs/3.9-disable-centos-repo"

This commit was intended to fix the gcp job, but had a side effect of breaking install job, so we would like to revert.
This reverts commit 7e1dfd65108dbbf034dc0fa2e26b2b101744733d, reversing
changes made to 46a12315e806d5954b738cb37504cf687aa9acb2.
/cc vrutkovs

The sideeffect which broke 3.9 install jobs was that a repo with openvswitch was not installed


Create text transformations

Implement a configuration or similar class to apply text transformations. For instance, to replace back and forward characters like á. Also, to remove trailing spaces.

The support for text tables will be handle as a new encoding for DataWriter and DataReaders. Furthermore, it will extend some API to apply the conversion to text.
For other text transformation like trailing spaces needs to happen at a higher level since it needs to apply to all the encodings. That should be handled by the Converter reading the external text file, since depending on the file it may be or not supported removing trailing spaces. We could group these methods and utilities into a class.

Implemented Replacer.
For general text transformation, I could implement a "Transformer" with several fields configured. We could a field into the TextReader, TextWriter, DataReader and DataWriter so they are done automatically.


Do not track timestamps with nanosecond precision

Trivial change to DB structure to remove ns from timestamp + delay data but requires checking that this does not cause conflicts with SystemTime::now() instances elsewhere in the codebase.

resolved


Can fix-nonhinting be moved into .glyphs Custom Parameters?

https://github.com/Lacquer-Font/Lacquer/pull/1/files#diff-2a16a4efc5dc106a1a3df65f51a852ceR9
Can that be avoided by placing 2 custom parameters to set the GASP and PREP table values within glyphsapp files?

sounds like that would be great!

It's a good idea, but these days we fix glyphs files in googlefonts/gf-glyphs-scripts.


package request/idea: gvisor

"Sandboxed container runtime"
https://github.com/google/gvisor
Bonus points for NixOS option for docker support :P. But just having it to poke at would be neat, i think!

I've looked quickly at this, and it seems like the main limitation in implementing this is https://github.com/bazelbuild/rules_go/issues/1376 . I think we either have to patch Bazel in NixOS or any Bazel package that compiles Go to fix this...


Allow PDF buttons to trigger foundry JS scripts

Update the field mapping object so that PDF buttons can be triggered and will call an appropriate function on the field object.

Q1 - how are PDF buttons supported by the pdfjs library?

It looks like setValue will always be called (even if the value hasn't changed, because pdf-pager doesn't know if the application value has changed).

I can't find a free PDF editor which will let me edit buttons (not checkable or radio buttons), so that I can set just the "alternativeText" which is required if there isn't an "A" or "AA" link associated with the button.

The simplest option at the moment is to create a checkbox and set the setValue function to trigger the corresponding Foundry JS code.

Version 0.49.1 has some basic support for clicking on a button.
I haven't been able to get hold of a form-fillable PDF which has buttons configured it which will get displayed by the pdfjs library (the button needs to have an A or AA link in it).

Version 1.5.1 added support for making arbitrary text snippets within the PDF clickable. That seems like the only reasonable implementation that can be achieved.


[check] Basic, non-CFG, after-return dead code detection

I had some code that was similar to:
import sys

def is_python3():
    if sys.version_info >= (3, 0):
        return True

    return False

(This matches the style required by RET505.)
If I then run:
% ruff check --select F401,UP036 foo.py --target-version py39  --unsafe-fixes --fix
Found 2 errors (2 fixed, 0 remaining).

I get an output of:

def is_python3():
    return True

    return False

It would both:

Be good if UP036's (admittedly unsafe) fix didn't leave obviously dead code behind,
Be good to have some rule that catches at least trivial dead code like this (which doesn't need any sort of CFG analysis to find).

Mostly filing this as a feature request for the latter, because I'm… in that state now.

Related to https://github.com/astral-sh/ruff/pull/10891


Fix vulnerability

*Issue reported in our internal security scan - please refer screenshot and recommendation

Description of changes:
upgrade to<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@AbhilashVijayakumar
The tests are failing. Can you please take a look?

@AbhilashVijayakumar The tests are failing. Can you please take a look?

Updated unit test @atshaw43 commit https://github.com/aws/aws-xray-sdk-go/pull/378/commits/7f55dcee1cca4204110b02e56969883749d7f43b

@atshaw43 all tests are passing in https://github.com/SPHTech/aws-xray-sdk-go is the github actions in this repo pulling the incorrect repo to test against can help to review

I pulled down the code locally and tested it and it ran fine. Can you kick off the tests again? I don't seem to have permission to.

You might also try pulling the changes from this fork into yours and re-testing.

I fixed the tests in a side pull request and merged them since I could not edit this one.
This PR can be closed.
https://github.com/aws/aws-xray-sdk-go/pull/390


New owners added as "UNKNOWN"

I thought we solved this issue before, but I looked and I cannot find a ticket for this particular issue (if there is one, we could close this one as duplicated)
Adding a new owner that is not in the address book will load as UNKNOWN

URL: https://safe-team.dev.gnosisdev.com/app/safes/0xB14A96CCaccF190fb0c4cC0f67a1Ef8269ca7De1/transactions
Steps:
Create a new account (in metamask for example)
Add it as an owner in a safe
After you finish the tx adding the owner you see that is called UNKNOWN instead of the name you set for him

@lukasschor reminder to add priority to this one


feature request: replace history with corrected command.

It would be a nice feature to correct the command and the history.
I would also like an option to not add {fuck,thefuck} to the history.

Cool, but maybe it is decided by the shell?

I use zsh, and It seems that the HISTIGNORE trick it's not working. (but It's not so important I've already set to not save duplicates).
I think thefuck could execute "history" command to update the history. I've not tested it but man says:
history change newValue ?event?
Replaces the value recorded for  an  event  with  newValue.   Event
specifies  the  event to replace, and defaults to the current event
(not event -1).  This command is intended for use in commands  that
implement new forms of history substitution and wish to replace the
current event (which invokes the  substitution)  with  the  command
created through substitution.
is it cross-shell compatible?   Ideally the original command would also be left intact (in case thefuck was wrong).
While fc can work differently in different shells, the example below works great in bash - removing the fc command from history while keeping the original and modified commands in the history. In zsh the example works the same, except the fc command remains in history.
$ ls -l /usr
$ fc -e- /usr=/etc

History in bash afterward:
ls -l /usr
ls -l /etc

Implemented for zsh and bash in 1.35+

As soon as I have some spare time I can dig this one and implement it for Fish.

The solution to this issue Fish-wise depends on fish-shell/fish-shell#825 available in the upcoming release, [Fish 2.2.0](/fish-shell/fish-shell/milestones/fish 2.2.0).

So now it works.


Safe extension should process the padding value of \bbox

Currently, \bbox[10px]{math} doesn't pass the 10px through any filter, but it should (especially if #1182 is implemented).

The issue1182 branch of my fork of MathJax adds support for this.

==> Merged

==> In testsuite
Configuration/Safe/issue1183.html


Alt+Cmd+T does not close other opened tabs on editor

VSCode Version: Code - Insiders 1.9.0-insider (9486bf472c5c558e4f1793e0e746da88d071b1cc, 2016-12-29T07:05:11.443Z)
OS Version: Darwin x64 14.5.0
Extensions:

Extension
Author
Version

Handlebars
andrejunges
0.2.0

vscode-eslint
dbaeumer
1.2.2

githistory
donjayamanne
0.1.4

LogFileHighlighter
emilast
1.0.1

vscode-firefox-debug
hbenl
0.7.7

gitblame
waderyan
1.3.0

Steps to Reproduce:

Open several files on a single editor
Press Alt+Cmd+T to close all but the one you're on

Nothing happens. All files remain open.
Thanks

Caused by https://github.com/Microsoft/vscode/commit/bc46e789ac57f327f409260fb8cc1f6e40e62a7b


Improvments to bat file

Mainly: Don't call out to PoerShell (expansive!) if Dart SDK is upto date.

LGTM


Improve performance of lit_utf8_string_calc_hash_last_bytes.

JerryScript-DCO-1.0-Signed-off-by: Evgeny Gavrin<EMAIL_ADDRESS>Native build on RaspPi2
                           Benchmark |         RSS
(+ is better) |        Perf
(+ is better) |
                           --------- |                          --- |                         ---- |
                          3d-cube.js |          160->   160 (0.000) |        5.984-> 5.726 (4.311) |
              access-binary-trees.js |           96->    96 (0.000) |        3.082-> 3.062 (0.649) |
                  access-fannkuch.js |           48->    48 (0.000) |       14.096->14.032 (0.454) |
                     access-nbody.js |           68->    68 (0.000) |        6.316-> 5.816 (7.916) |
         bitops-3bit-bits-in-byte.js |           36->    36 (0.000) |        4.742->  4.68 (1.308) |
              bitops-bits-in-byte.js |           36->    36 (0.000) |        6.608-> 6.608 (0.000) |
               bitops-bitwise-and.js |           28->    28 (0.000) |       4.056-> 4.076 (-0.493) |
            controlflow-recursive.js |          224->   224 (0.000) |        3.578-> 3.534 (1.230) |
                date-format-xparb.js |          108->   108 (0.000) |        3.762->  3.43 (8.825) |
                      math-cordic.js |           48->    48 (0.000) |        7.486->   7.3 (2.485) |
                math-partial-sums.js |           40->    40 (0.000) |        3.064->  2.83 (7.637) |
               math-spectral-norm.js |           56->    56 (0.000) |         4.19-> 3.862 (7.828) |
                     string-fasta.js |           52->    52 (0.000) |      35.024-> 28.67 (18.142) |
                     Geometric mean: |            RSS reduction: 0% |            Speed up: 4.7813% |

LGTM.

LGTM

@sand1k @zherczeg @ruben-ayrapetyan guys, I've updated PR, could please recheck it again.

still LGTM

@ruben-ayrapetyan except comments, are there any issues? May I push it after fixing?

@egavrin, after adding the comments, the patch would look good to me.


Cannot import a package from Tensorflow

Click to expand!
Issue Type
Build/Install
Source
source
Tensorflow Version
tf 2.0.0-rc2
Custom Code
Yes
OS Platform and Distribution
Windows 11
Mobile device
No response
Python version
3.7
Bazel version
No response
GCC/Compiler version
PyCharm version: 2019.3 EAP - Build #PY-193.3793.15
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Cannot import packages from Tensorflow in Pycharm. There is a confusion of whether this error had occured due to the version of IDE or the version of Tensorflow.

Standalone code to reproduce the issue
CODE:
from tensorflow.keras.models import Sequential
from tensorflow.keras.layers import Conv2D

Relevant log output
ERROR:
Cannot find reference 'keras' in '__init__.py'
Unresolved reference 'Conv2D'

@Tharunika-21 I was able to import the packages in latest TF version of TF, please find the gist here for reference.
Could you please try with the latest and let me know if it helps?
Thank you!

Yes. It worked  with the latest version. Thank you for your support!

@Tharunika-21 Thank you for the update!
Closing this issue as it is resolved in the latest version. Please feel free to reopen the issue if you still have a concern.
Thank you!