vcpkg bde dependencies (and why not ntf-core)

Is your feature request related to a problem? Please describe.
I'm reviewing this library to incorporate into my system and had a few initial questions.

Given that this library has a dependency on bde, would it be possible to get bde updated in vcpkg's ports? I'm not sure which version of bde is being used internally at Bloomberg for this project.
Why was boost-asio chosen as the socket library vs your internal ntf-core library?

Describe the solution you'd like
Updates to bde vcpkg port and just initial queries about the library
Describe alternatives you've considered
N/A
Additional context
N/A

Hi, the port for rmqcpp at https://github.com/microsoft/vcpkg/blob/master/ports/rmqcpp/vcpkg.json#L9 mentions the bde version requirements - it would use the latest external version on building (and has to be ><IP_ADDRESS>).
For the second question - boost-asio did the job well and the rest would be above my pay grade :)


test WSDL-generated SEI

As discussed in #85, here's an implementation of a test using a SEI generated from a WSDL. Some notes:

The WSDL used was taken from an official SoapUI tutorial so I just assumed it was both sufficiently representative and not encumbered by any confidentiality issues. In any case, no calls are being made to the original service, either to get the WSDL or to invoke the service.
This changeset includes a workaround for #85 (using a bindings file) so it will pass the tests for now. This bindings files should eventually be removed as a way to test that any fix for the issue is working correctly. Please note that this will rename the class Calculator back to CalculatorSoap so all references will have to be updated.
I don't really like the idea of testing both the server side and the client side at the same time from within Quarkus. I'm concerned that at some point one error in the extension implementation might cancel out when happening on both sides. So, I added a CXF "mini-server" that is started/stopped around the relevant tests and pretends that it is an external service. This "mini-server" does not use Quarkus or this extension. This part of the PR is not essential for the main functionality so you can remove it if you want.
Currently, none of the wrappers generated uses JAXBElement so that part is not being tested yet. I'm currently investigating how to get that part tested and may have something soon.

Thanks so much. I have added support for WebServiceClient annotation. So you can remove workaround

Thanks so much. I have added support for WebServiceClient annotation. So you can remove workaround


Facebook login broken with some profile pics

Describe the bug
Using TypeORM and MySQL, Facebook login fails for some users, as the url to their profile pics is longer than 255 characters. DB save operation fails.
Steps to reproduce
Try to save something longer than 255 characters in User.image
Expected behavior
No crashing due to any profile pic.
Additional context
I fixed it by increasing maximum allowed length of image string in UserSchema
image: {
      type: 'varchar',
      nullable: true,
      length: 1000
    },

Feedback
Documentation refers to searching through online documentation, code comments and issue history. The example project refers to next-auth-example.

[X] Found the documentation helpful
[ ] Found documentation but was incomplete
[ ] Could not find relevant documentation
[ ] Found the example project helpful
[ ] Did not find the example project helpful

Yeah I believe we increased the allowed length of that field a while back. I will close this issue for now. @WonderfulWorlds if you are still experiencing it, feel free to reopen.


Help needed for gettting started.

Hi I am using Zest + Enzyme for testing my browser plugin.  My plugin consists of background.js, contentscript.js and dev  tools.
  chrome.extension.onConnect.addListener(function (incoming_port) {
    let port = incoming_port;
    if (port.name === '3pm-panel') {
      port.onMessage.addListener(function (message) {
        switch (message.type) {

          case 'reloadWithDebug':
            // This is called even for iframe is loaded
            chrome.tabs.query({
              active: true,
...

I am trying to test if the onConnect.addListener is called once,  reloadWithDebug is called once .  I am not sure how to proceed with the testing. Can you elaborate with the below example. Thank you in Advance for your help
describe('your function to test', () => {
  it('should have called a webextension API', () => {
    yourFunctionToTest();
    expect(chrome.tabs.update).toHaveBeenCalled();
  });

Does chrome.extension.onConnect work?  I'm not doing extension development anymore but looking at the docs I don't see the onConnect attribute https://developer.chrome.com/extensions/extension
I believe you're supposed to use runtime for this now.

@clarkbw  Thank you quick response. Actually I am using runtime now.  Still I am not sure How to test my function. Can you provide some details please?

I'd probably do something like this:
describe('your function to test', () => {
  it('should have called a webextension API', () => {
    expect(chrome.extension.onConnect.addListener).toHaveBeenCalled();
    expect(chrome.tabs. query).toHaveBeenCalledWith({ active: true });
  });

If you wanted to test the port.onMessage.addListener pieces I think you'd have to submit a PR that allows for that kind of testing as well.  Currently we don't return a port object you can work with.

Thank you @clarkbw . It makes sense now.
I just want to check if my understanding is right.
Suppose If I want to test
 port.onMessage.addListener(function (message) {
        switch (message.type) {
             case 'reloadWithDebug':
                                 case 'reloadWithDebug':
            // This is called even for iframe is loaded
            chrome.tabs.query({
              active: true,
              lastFocusedWindow: true
            }, function(tabs){
              let currentUrl = tabs[0].url;
                chrome.tabs.update({url:currentUrl});
               }
}

Is there a way to pass a message manually to the listener and check if chrome.tabs.update is called?


Docker Container 0.16.2 Delivery

The team will deliver the 0.16.2 Docker Container to cover the basic 1311 Functionality that was not captured in 0.16.1.
The  Docker Container 0.16.2 Delivery should also include the Test case Suite

@TessForGithub2  please advise the new ETA for the docker container.

ETA is Monday, 7/29. I expect it will contain all the CRUD for Fluids Report (basic sanity cases only; we will have to spend time reviewing edge cases & special cases) & <IP_ADDRESS> fixes.

Card updated to note that the team will NOT deliver the 0.16.2 container this week but  on Monday 7/29 will instead deliver the Docker Container 0.17.0  to cover the basic 1311 Functionality that was not captured in 0.16.1.  This container (0.17.0) will include the basic sanity test suite.
::NOTE:: A second container (0.17.1) will be delivered on 7/31 to include the full FsR test case Suite.

Update: Update will NOT be included in container 0.17.0. I will deliver this in 0.17.1

Container 0.17.0 has been delivered and is in use by SLB.


gtk3-macros doctests are compile_fail

See https://github.com/gtk-rs/gtk-rs/pull/466#discussion_r615895426
Is this intentional? @bilelmoussaoui @BrainBlasted ?

If the problem is that the file isn't available, then we should either give a path which exist or mark those examples as ignore instead. Because currently, it sends tells readers that the code is invalid.

Agreed, but maybe this can also be made to pass compilation at least (and have it no_run). I wanted to get this failure out of the way for now so someone who knows how this macro works in detail can solve it properly.

I think the problem is that the file specified doesn't exist, and that's check at compile time (otherwise, it would compile and the test would fail ;) ).

The struct in the example needs to implement ObjectImpl & WidgetImpl traits for it to compile

That too in addition to the missing .ui file, yes


provide raw data on zenodo as well

Is the Zenodo URL in the README (https://zenodo.org/record/477093) supposed to work? I am getting a page not found error when accessing it.

Thanks!
I corrected the url (https://doi.org/10.5281/zenodo.4770937).

Raw data from ECA&D which was used to create the dataset is now on zenodo as well: https://doi.org/10.5281/zenodo.4964287


The process to checkout button should be disabled if the product is out of stock.

Bagisto Version: Master
Steps To reproduce--
1. Go to the shop.
2. Add any item to the cart which is out of stock.
3. Click on the process to checkout button.
Expected result: The process to checkout button should be disabled if the product is out of stock.
Actual result: The process to checkout button is not disabled if the product is out of stock. Please find the screenshot Here

Additional info If allow backorders is enabled the checkout button should not be disabled

@jitendra-webkul
Now, this enhancement is done. Please find the screenshot Here.


[MINOR][UI] Simplify columnDefs in stagepage.js

What changes were proposed in this pull request?

Simplify columnDefs in stagepage.js
Why are the changes needed?

Reduce hardcode in stagepage.js and potential inconsistency for hidden/show in future changes.
Does this PR introduce any user-facing change?

no
How was this patch tested?

Locally verified.

cc @ulysses-you

I tested it locally with accumulator, looks fine to me

thanks @ulysses-you for the verification

cc @gengliangwang and @sarutak FYI

GA failure seems not to be relevant to this change.
Merging to master. Thank you @yaooqinn and all!


dev -> master

This change is

Reviewed 6 of 7 files at r2, 1 of 1 files at r3, 2 of 2 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


caesar-cipher

If we see this encryption technique in mathematical way then the formula to get encrypted letter will be:
c = (x + n) mod 26
where, c is place value of encrypted letter,
x is place value of actual letter,
n is the number that shows us how many positions of letters we have to replace.
On other hand, to decrypt each letter we’ll use the formula given below:
c = (x – n) mod 26

Can I rename the file to caesar-cipher.py and can you then merge it cause I'm new here and getting used to the GitHub interface! So can you then merge it?

you have renamed the file.
Now move that file to caesar-cipher directory locally.Then commit the changes & push.

There was some error while pushing up the file so I kind of had to upload it! Can't it be merged like this? Even my other pull request have been merged like that! Ok but still let me try

Please try to move your code to  caesar-cipher directory.If you face some issue.let me know.


Fix Bad Regex Causing Error

Fixes Cannot read property 'replace' of undefined error.
Regex is not working with app generated with latest version of vue-cli due to use of double quotes.
This fix address this issue

How can I use this change while this is not of the master branch ?

How can I use this change while this is not of the master branch ?

If you change the code in main.ts where it calls mount('#app') to use double quotes instead, it should work.

Sorry for the late response :bow:


Update to SUSE Hana - HA cluster creation : force primary node to be master before completing task set.

Problem
In SLES based HA - HSR ,  the SAPhanaSR script hook (from SUSE)  under some cases will make secondary node as primary during cluster creation.  This causes DBLoad playbook to fail as the task_3.3.1 (BOM Utility) populates the DB name as primary VM instead of Cluster name.
This is not being seen in RedHat based HSR clustering ( using PCS ).
Solution
The fix is to force HANA to move to master using the below Jinja2 shell sequence :
"crm resource move msl_SAPHana_{{ db_sid | upper }}_HDB{{ db_instance_number }} {{ primary_instance_name }} force"
right before removing the cluster from maintenance mode. The resource constrains + automatic_failover: false should prevent any automatic movement of HANA master role to secondary VM unless requested by cluster admin.
Alternate workaround ( not implemented at the moment ) is to add logic to taskset 3.3.1 ( BOM  utility) to populate SAP HA virtual name instead of HANA primary VM name in the DBload template.
Another workaround is to do a DB load before HSR creation , this could however increase cluster creation time as DB backup and restore will take long time. The effect of this approach is not known on Scale out setups ( NetApp or HA+HSR ) approach and needs to be investigated.
Tests
Previously, 2 in 5 HANA deployments will cause secondary node to become master during HSR clustering. Currently tested infra showed subsequent deployments forcing primary node to be actual HANA master. The behavior is more prevalent in VM SKU where HANA takes a long time to start post the Stop/Start  sub tasks.
Notes
Recommend another round of testing before merging into main.  Although this command should not cause any issues or incompatibilities with current working scenarios as there is no actual DB replication going on at that stage and role migration to node if applicable is instantaneous. If Primary node is Master, this command is ignored.
Its important that DB load or any data import activity is done post cluster creation. If HSR - HA is being done on HANA with already existing data, ensure that the database is already replicated to the second Hana before running HSR/HA setup.
RHEL will use "pcs" instead of "crm", currently attempting to replicate problem in RHEL environment.
This change is referencing the official SLES HA guide at link : "https://learn.microsoft.com/en-us/azure/sap/workloads/sap-hana-high-availability'
Update: This minor fix is currently being used in a customer deployment

Hi @shekharsorot , I think this is more of change in the order of execution. I will get back to you on this. Until then, can you put this PR in draft state please?
cc: @nnoaman , @KimForss , @mkdeegan

putting PR in draft.

removing PR , patch to be pushed with equivalent fix for RHEL HANA cluster added to SLES


Add [24,409.7469194] #personal GDJF3H…DJSGX5

Address: GDJF3HLA53SLZ47BV6M2RHO5EBWT44RAYDPAOI6ENIDI4F44LODJSGX5
Tags: personal
Title: 24,409.7469194
Requested by @maycondci

My funds wont Credit to my account, i try transfer form Stormgain to Binance, só not Credit, then binance return to my account in stormgain, and not Credit again, please can u help me

@Maycondci
Stellar expert is only a tool for exploring information on the Stellar blockchain. You will need to contact Stormgain and Binance support.
If you have specific transactions I might be able to help in some way.
The wallet you posted GDJF3HLA53SLZ47BV6M2RHO5EBWT44RAYDPAOI6ENIDI4F44LODJSGX5 is a bitgo wallet. You might talk to them if Stormgain works through them.
Looking Towards The Future With You,
Rob

Hello @Maycondci
Please feel free to reopen this issue with new/more information if needed.
Looking towards the future with you,
Rob


Give Alphabet a contains(...) method

As discussed in #6, it should be possible to check if a particular Letter is found in an Alphabet -- I was able to resolve this quite succinctly with a default method based on Alphabet.letters().

Safe to close?

No, I have a hold over from adding all the _ is Letter statements when I originally was making all letters satisfy Hashable and Equatable[Letter]. I need to revert these changes so this PR is only doing what it says it is doing.


Deprecated project?

I can see the last commit was in January 2020 and the related web https://aksctl.io is offline. Does this means this project has been abandoned/deprecated?

Hi emibcn,
The development of this project is inactive for a year now, but we are considering setting new milestones and bringing the Project back.
Let us know if you are expecting some specific features or improvement suggestions

Let us know if you are expecting some specific features or improvement suggestions

Not really (yet), just considering different options (plain az, Terraform, etc) before deploying AKS on our platform. We don't want to start using an already deprecated/abandoned tool.
@nanusefue FYI


ROI fields not propogating from cloud EEPROM?

I can't see why this would be the case from the code, but needs testing.

resolved in #480


how to process 194 landmarks (helen dataset) ?

I had trained DLIB's shape_predictor for 194 face landmarks using helen dataset. but, when I call the function of "get_face_chip_details" for face-recognition, DLIB throw an exception.
so, how to modify the DLIB's source code in order to adapting the 194 face landmarks ?
thanks.

Look at the code for that function.  It should be clear what to do.  But in any case, I'm not going to type the code out for you, and it's not likely anyone else will either.

but, how to fix the array variable of mean_face_shape_x and mean_face_shape_y ?


Feature request: change from [1, ..., 12] grid to number

With that change made I can use 0.5 etc for different grid sizes. Doing that means it should also not break anything.

@gerbenmeyer any chance of this in 2.1?

Sure, seems like a good idea, just thinking about whether there would be any downside to it.

So it's there, version 2.1.


Add additional game settings

As a user, I would like to have some control over how the game is set up
E.g. smaller deck, different point values, different bots, e.t.c.

Basic framework done by d4b67df239faaf643522

Rescoping to be adding additional settings to the existing games. (Probably means this issue will be long-living)

Rescoping to be adding additional settings to the existing games. (Probably means this issue will be long-living)


Doesn't Work - New Content is Corrupted

The examples work,but any content generated from the generator doesn't work.
It would appear this library is out of date. The references ./unishox are actually ./test-unishox
Generated code is using uint8_t whilst the examples are using byte

Hi, you are right I changed ./unishox to ./test-unishox and have not updated this repo. Are you still facing any issue? byte is the same as uint8_t as long as the typedef is present. I am sorry I have not been able to update this repo yet.


TorchScript: Unknown function 'torch.resize_'

Netron app and version: 6.6.8
OS and browser version: Ubuntu 18.04
torch version  1.12.1+cu102

It seems the torchscript model converted from pytorch model inserted with observers for quantization fails to open. It reports error with something like "unknown function: torch.item" or "unknown function: torch.resize_".
Steps to Generate target model:
import torch
from torchvision.models import resnet18
from torch.ao.quantization import get_default_qconfig
from torch.ao.quantization.quantize_fx import prepare_fx, convert_fx
model = resnet18(pretrained=True, num_classes=1000)
model.eval()
qconfig = get_default_qconfig('fbgemm')
qconfig_dict = {"": qconfig}
prepared_model = prepare_fx(model, qconfig_dict)
model_scripted = torch.jit.script(prepared_model)
torch.jit.save(model_scripted, 'r18_scripted.pt')

r18_scripted.pt.zip

Got another error on min/max attribute.

TorchScript support is experimental. The code tries to trace the file guessing input sizes (pytorch/pytorch#48525). A possible workaround is to save a trace instead of script. Supporting script blocks without tracing is tracked by #1061.


[Improvement]: Improve compiler diagnostics

Description
The current format of compiler error diagnostics in Ballerina could be enhanced to provide more helpful information to users. Currently, it outputs errors in the following format:
ERROR [filename:(startPosition, endPosition)] error message

This format lacks detailed information about the area of the error code, making it less user-friendly compared to other compilers like Rust. By enhancing the output to include the area of the error code, users will have a clearer understanding of where the error occurred, which can significantly improve the user experience.
Describe your problem(s)
No response
Describe your solution(s)

Modify the compiler error diagnostics format to include the area of the error code.

Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

21/02/2024 Update

Semantic diagnostics rendering works.
Syntax Diagnostics,

Used args that are passed into createDiagnostic method to get the missing token/keyword for BCE0001 to BCE0400. https://github.com/ballerina-platform/ballerina-lang/blob/838820f39c3b737af03a00d1819b1c807ed6c917/compiler/ballerina-parser/src/main/java/io/ballerina/compiler/internal/parser/SyntaxErrors.java#L96

Used jline for resizing diagnostic rendering based on terminal window size.

Tasks

[ ] Improve Diagnostic Rendering using Diagnostic API and Syntax API.
[ ] Add a bal explain <error_code> command to explain error codes.
[ ] Prepare Diagnostic structure guide


Update ClaimEventForm.tsx

updated UX writing

The frontend has been separated in https://github.com/Magickbase/ckb-testnet-faucet-frontend and the corresponding code is at https://github.com/Magickbase/ckb-testnet-faucet-frontend/blob/4db87c004e46f5b5cc1c006fc1ca8ed171910eb6/src/sections/Content/ClaimForm.tsx#L124


Feature Request : Add profile as an option in change AMT password job (RPC and RPS)

Describe the bug 🪲
Need an option in RPC for changing only password based on profile
Prerequisites

AMT needs to be activated in ACM using Open AMT Cloud Toolkit
Device needs to be on Static IP
Host IP address should be different from AMT's IP address. This can be done using tool like MeshCommander.
Intel Local Management Service should not be installed

To Reproduce 🪜
Steps to reproduce the behavior:

Create an ACM profile with Static IP and TLS
Add AMT device to Open AMT using the profile created during 1st step
To change password we have to run sudo ./rpc maintenance changepassword -static New_AMT_Password -u wss://[Dev-IP]/activate -n -password Old_AMT_Password

Expected behavior

Change the password in the above created ACM profile
Run something like sudo ./rpc maintenance changepassword -profile TestProfile -u wss://[Dev-IP]/activate -n -password Old_AMT_Password instead

AMT Device (please complete the following information): 🖥

OS: Centos 7.9
AMT Version: v11.8.65
AMT Configuration Mode: ACM
Network Configuration : Static

Service Deployment (please complete the following information): ⛈

Deployment Type: K8s
Component & Version: v2.7

Hi @KevinBK1998,
If I recall correctly, what was prohibiting this flow before was that Static IP was causing an issue where, during reconfiguration, the configured different AMT IP vs Host IP would be overwritten and synced.
In the v2.11.0 release of RPS, we have added a new flag called ipSyncEnabled (see closed feature PR here).  In your Profile, you can disable this feature to prevent the sync from happening.
See Network Configuration section of Profile:

Now, one solution is to rerun the activate command with an edited profile to reconfigure the AMT device.  In your existing AMT profile, you can update the AMT password.  Then, run:
sudo ./rpc activate -u wss://{RPS-SERVER}/activate -n -profile UpdatedProfile -password OldPassword

Let me know if this solution works for you and we will close the PR, thanks!

we are fine with rerunning activate command, but the ask was whether changepassword maintenance task can accept profile name also
I have also replied in https://github.com/open-amt-cloud-toolkit/rps/issues/890

We are still seeing that the ip address is getting lost on running rpc activate command with ACM Profile settings:  IP getting lost after ACM activation:

Hello,
Now that we have confirmed that the IpSyncEnabled flag feature (#890) is now working as intended, I will close this issue.
We do not have plans to add the ability to pass a profile into the changepassword maintenance command of RPC.  Functionality is provided through other means already.

If passing newpassword via CLI is okay, then users can use the current implementation of the changepassword cmd.
If wanting to hide newpassword from CLI, then users can rerun the activate cmd on an activated/configured device with an updated profile containing the new static AMT password. This will not alter or change other pieces of the existing configuration, just the device's AMT password.  The flow to update password is equivalent to the one used by the changepassword cmd.

Thank you.  Please reach back out if you have more questions about this request.


Parse date manually as DateTimeFormat is not cross browser consistent

Fixes: https://issues.redhat.com/browse/LOG-3583
DateTimeFormat behavior is not consistent across different browses, so this PR adds manual parsing on the problematic case that we need to create a time format that has 24 as hours.

/cherry-pick release-5.6

/cherry-pick release-5.5

/lgtm


Add local action

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

Project coverage is 100.00%. Comparing base (ab4147d) to head (fe35449).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #197   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          167       167
  Branches        23        23
=========================================
  Hits           167       167

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


Microsoft Office COM Interop on Mac

Hi All,
Is it possible to do COM Interop between a .NET Core app and Office on Mac?

COM is not supported with Office on Mac. You will want to use AppleScript or JavaScript APIs to do this. Thanks.


[Feature]: Support Calbacks in the UI for Execute JS Actions

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Currently, the JS Objects tied to widgets' events such as onClick, onFilesSelected, etc., do not display Callbacks.

At this jucture, the users need to handle this in the JS Object function itself or by clicking the JS icon to write custom JavaScript in the onClick event.
Why should this be worked on?
By supporting callbacks in the UI for JS object functions, users would be able to develop their apps faster, having an enhanced seamless experience.

@ame-appsmith For an API or Query, the success criteria is the call being successful and getting the expected response. What will be the success criteria for the success of a JS Action? Only once we have the success criteria, we would be able to support callbacks like onSuccess and onFailure to it.
OR
we might need to rename the callbacks to something generic so that .then and .catch support could be added to async JSObject function.


Unable to run the extension migration

I'm on Postgres13, getting this:
django.db.utils.OperationalError: could not open extension control file "/usr/local/share/postgresql/extension/vector.control": No such file or directory

Hi @peter-stuart-turner, that error means pgvector isn't installed on your database server.

Ah thanks very much, have sorted it now


Overflow in tf.raw_ops.SparseReduceSumSparse when there are too large values in input_shape.

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
tf 2.15
Custom code
No
OS platform and distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Overflow in tf.raw_ops.SparseReduceSumSparse when there are too large values in input_shape.  See the colab link below for details.
Standalone code to reproduce the issue
https://colab.research.google.com/drive/17mqn3t23p9sb4e3stOsPvZBTHENqJYku?usp=sharing

Relevant log output
Non-OK-status: InitDims(dim_sizes) status: INVALID_ARGUMENT: Encountered overflow when multiplying<PHONE_NUMBER>000 with 6000000, result: -8246744073709551616

@xl1043237213,
Thank you for reporting the issue. I was able to reproduce the issue on tensorflow v2.15, v2.16 and tf-nightly. Kindly find the gist of it here.

Hi @xl1043237213,
I was reproduced the issue or error with latest Tensorflow version (TF 2.18.0). Please refer to this gist file.
The issue occured because TensorFlow is attempting to calculate a very large product<PHONE_NUMBER>000 * 6000000) as part of the
sparse tensor operations, leading to integer overflow. The result exceeds the maximum value of a 64-bit integer, causing the calculation
to fail.
To resolve this error, could you please reduce the scale of the input_shape values if possible.
Thank you.


Confirm your address - do users read and choose correct address?

User research to check if this is not current address do users actually read the address that's returned from Verify? Do they have problem with entering a new address?

User research (concessionary travel r2) shows that most people notice this, but also that they object to re-keying address data.

@petegale do you think the latter part of this finding should be transformed into a new issue?

Yes

Evidence form latest round of user research on Concessionary travel:


Update ValueObject.cs

Add overloads for "==" and "!="

I have added overloads for the "==" and "!=" operators, enabling the following operations:
ValueObject1 == ValueObject2

ValueObject1 != ValueObject2


OOMKilled error in qualitycontrol-check-cmip6 with dodola validate-dataset

After #296 we're getting OOMKilled (exit code 137) from workflows/templates/qualitycontrol-check-cmip6.yaml.
Logs give
INFO:dodola.services:Starting validate dodola service
INFO:dodola.repository:Read gs://downscaled-288ec5ac/stage/ScenarioMIP/NOAA-GFDL/GFDL-ESM4/ssp370/r1i1p1f1/day/ta
smax/gr1/v20211103182935.zarr

Looks like it's still reading in the entire zarr store and bombing when this zarr store is bigger than available memory.
I'll revert #296 to get things working again and then deal with this bug upstream in dodola.


[v4.11.0] Could we have minisigs, please?

The minisigs have been incredibly helpful when packaging soupault for ArchLinux/AUR, is there a chance we could have them back? :pray:

Sorry, that was a mix-up on my side! They are in [https://github.com/PataphysicalSociety/soupault/releases/tag/4.11.0] and on the web server now.

Great, thanks! :+1:


Update PPP to fit page limit

Exclude certain portions to fit the page limit

Pull Request Test Coverage Report for Build 454

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.114%

Totals

Change from base Build 450:
0.0%

Covered Lines:
2748

Relevant Lines:
3267

💛  - Coveralls


403  error message propagation

Description
Changes proposed in this pull request:

propagate full response object so that error message could be propagated to nopermissions page

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

/test pre-busola-core


handle async and await tokens

https://github.com/jsdoc2md/jsdoc-to-markdown/wiki/How-to-document-ES2016-features


Does python-saml support enveloped signatures?

Since OneLogin only supports HTTP-POST binding at SAML Responses, the enveloped signatures is only supported for SAML Responses.
Logout Request and Logout Responses are supported only by the HTTP-Redirect binding so no enveloped signatures supported for them.


clean up outputs for validate command

Prints out each schema of the provider
Format health message nicer

Updated to pretty print the json


Remote branch show null when unspecified

When unspecified a branch name, the message show #null. Should we improve it?

In my opinion, I think we should omit #null when using the default branch. How do you think?

Yup, let's do that. 👍

There is too long with no update, so I closed it


Migration from .NET 3.5 to .NET Core 2.

This is a migration from .NET 3.5 to .NET Core 2.
The migration was performed due to a need to use TinCan.NET on a ASP.NET Core project which runs on Linux.
Creating this pull request as I rather not have to maintain a fork to support .NET Core. Please be explicit and reject this request in writing should you not want to support .NET Core.
Build confirmed to work on macOS, Linux and Windows.
All tests pass except one, the one failing fails because TinCan.NET does not support xapi 1.0.3 (see https://github.com/RusticiSoftware/TinCan.NET/issues/27)

@reedmclean It should be trivial to make the code compile both .NET 4.5 and .NET Core assemblies. We've done it in the past quite easily.

@JohnGalt1717, that's good to hear! Would we have to release two builds with different targets, or would one build work in both 4.5 and .NET Core applications?

@reedmclean All that you have to do is add multiple targets like this:
netstandard1.1;net452
Then Setup property groups to define compiler variables for anything that is unique between the two (don't think there is but...)

You can also put your nuget packages in each of those as package references so you get the right ones for the right assemblies.
Then when you build you'll end up with 2 directories in the bin folder and you can nuget publish and it will show as targeting both.


Use holiday names from code du travail

Dear reviewers,
We've got an important matter to discuss. France is watching us.
See discussion on data.gouv.fr. The question was: should we name bank holidays according to the code du travail or should we stick to names used by people?
Switching to names from code du travail
Pros:

we don't have to pick names
people have trouble to associate dates and names (When is Victoire des alliés again?)

Cons:

it's a breaking change
it's farther away from how people speak

To discuss:

[ ] Rename methods?
[ ] Document name <-> method mapping?

Like I said, I kinda like the "code du travail" names and it's an obvious way to justify our choices.

Very tough decision indeed...

I agree with choosing "code du travail" names.
If we want to be super precise, we could use two fields, one "nom_jour_ferie" related to "code du travail" and another one like "nom_usage_jour_ferie" related to "service-public" denomination.
But @AntoineAugusti I'm afraid to ask you that. I could take this on me after this is merge if you want.

I updated all the method names, documented the mapping, bumped the version and added a CHANGELOG 😄


Unable to find file LiipImagineBundle

Q
A

Bug report?
[ ]

Feature request?
[ ]

Support request?
[ ]

Hello,
I have a problem with the installation.
I followed this link to install.
Finally, when I run the Base Url I getting this error.
what should I do to fix this?
Thanks in Advance.

Hi,
I am also having the same issue as below when trying to install the app on Ubuntu server.
Unable to find file<EMAIL_ADDRESS>in @LiipImagineBundle/Resources/config/routing.xml (which is being imported from "/var/www/html/Symfony/src/Cocorico/CoreBundle/Resources/config/routing.yml"). Make sure the "LiipImagineBundle" bundle is correctly registered and loaded in the application kernel class. If the bundle is registered, make sure the bundle path<EMAIL_ADDRESS>is not empty.
I am trying to install Cocorico with the below server specs,
PHP 7.1(with required extensions and drivers)
MongoDb 4.2.0
MySQL 5.7
Apache 2
2 GB RAM(2GB Swap Memory)
Any idea to fix this error?

I googled a little and assummed that needed to install liip/imagine-bundle, this is what I got
php.exe composer.phar require liip/imagine-bundle
Using version ^2.1 for liip/imagine-bundle
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Removal request for liip/imagine-bundle == 2.9999999.9999999.9999999-dev
- Can only install one of: imagine/imagine[v0.7.1, 1.2.2].
- Can only install one of: imagine/imagine[v0.7.1, 1.2.2].
- Can only install one of: imagine/imagine[v0.7.1, 1.2.2].
- liip/imagine-bundle 2.1.0 requires imagine/imagine ^0.7.1,<0.8 -> satisfiable by imagine/imagine[v0.7.1].
- Installation request for liip/imagine-bundle ^2.1 -> satisfiable by liip/imagine-bundle[2.1.0, 2.x-dev].
- Installation request for imagine/imagine (locked at 1.2.2) -> satisfiable by imagine/imagine[1.2.2].
Installation failed, reverting ./composer.json to its original content.

Hi @gabilanbrc
liip/imagine-bundle will be installed on our initial composer command and I dont think we need to install it again.
After some research, I found that the installed Imagine bundle on my server have routing.yaml and not the routing.xml(https://prnt.sc/p11hgu). You can also see the details from the below links,
https://symfony.com/doc/1.0/bundles/LiipImagineBundle/installation.html
https://symfony.com/doc/2.0/bundles/LiipImagineBundle/installation.html
You can try replacing @LiipImagineBundle/Resources/config/routing.xml with @LiipImagineBundle/Resources/config/routing.yaml on the file Symfony/src/Cocorico/CoreBundle/Resources/config/routing.yml to fix this issue.

Thanks @MunirajM , that seemed to work!

Hi @gabilanbrc
liip/imagine-bundle will be installed on our initial composer command and I dont think we need to install it again.
After some research, I found that the installed Imagine bundle on my server have routing.yaml and not the routing.xml(https://prnt.sc/p11hgu). You can also see the details from the below links,
https://symfony.com/doc/1.0/bundles/LiipImagineBundle/installation.html
https://symfony.com/doc/2.0/bundles/LiipImagineBundle/installation.html
You can try replacing @LiipImagineBundle/Resources/config/routing.xml with @LiipImagineBundle/Resources/config/routing.yaml on the file Symfony/src/Cocorico/CoreBundle/Resources/config/routing.yml to fix this issue.

The issue seems coming from a composer update. This command should resolve the issue :
php composer.phar require liip/imagine-bundle:"^1.9"


An host entity is created for each kube-event-pod

Currently an entity is created when a kube-event is deployed.
This is not the expected behaviour, since the infra sidecar is used as a forwarder only.
We should investigate if the behaviour changed in the latests version or we always had this issue even setting NRIA_IS_FORWARD_ONLY=true and NRIA_IS_SECURE_FORWARD_ONLY=true

Context on the entity creation:

https://docs.google.com/document/d/1zuyMPlhLDG6aZXSP2PCQ8x5FMCJQrA0_Gf8S4Kb-Hvo/edit#

Entity creation should be disabled via connect_enabled: false see https://github.com/newrelic/infrastructure-agent/blob/30d0e84f3526ab0e1398d7908cc6e73193a21109/pkg/config/config.go#L885-L890

This is fixed. And the custom attributes issue that this change raised is fixed too.
Closing


Different results between Scispacy - Demo and my code for the same sentence

Hello everyone,
text =  "we do expect (06:27) submitted for Rayaldee in Europe to get those approvals and launch the product in Europe this year. "
Scispacy - Demo  works well. The NER model Scispacy - Demo produce desired outputs when I choose en_ner_bc5cdr_md model because Rayaldee is correctly classified as ENTITY  with label CHEMICAL.
Rayaldee CHEMICAL 8 9 35 43

However, when I replicate it locally, Rayaldee is classified as PROPN.  There is no ENTITY identified. Does anyone has idea of why this is the case? Thanks a lot!
import scispacy
import spacy
ner = spacy.load("en_ner_bc5cdr_md")

text  = "we do expect (06:27) submitted for Rayaldee in Europe to get those approvals and launch the product in Europe this year. "

doc = ner(text)

for token in doc:
    print(token.text, token.pos_, token.ent_type_)

for ent in doc.ents:
    print(ent.text, ent.start_char, ent.end_char, ent.label_)
we PRON
do AUX
expect VERB
( PUNCT
06:27 NUM
) PUNCT
submitted VERB
for ADP
Rayaldee PROPN
in ADP
Europe PROPN
to PART
get VERB
those DET
approvals NOUN
and CCONJ
launch VERB
the DET
product NOUN
in ADP
Europe PROPN
this DET
year NOUN
. PUNCT

Hi @JinDenguchicago the demo uses an older version of scispacy, and I don't have access to the demo page any more to update it to the latest scispacy version. This means that the results will be different between the demo page and the latest scispacy version. Please see https://github.com/allenai/scispacy/issues/342 for more discussion. I will also update the readme ot note this. As for why the newer version is "worse", I don't know the answer. It might just be randomness on this examples. On our eval sets the results were similar, but it is entirely possible that something changed for the worse between spacy 2.x and spacy 3.x training.

Hi @dakinggg, Thanks so much for your explanation! Do you know where I could get previous version link as discussion in https://github.com/allenai/scispacy/issues/342#issuecomment-804993320 did? I want to try pervious NER model en_ner_bc5cdr_md.

Second EDIT: (I just tried the pip install with specific version link and it works)


Token-based authentication for command line

Similar to how Jupyter does it; make sure other users can't take over a session on the same machine just by connecting to the port.
Follow-up to #1074

Note that we do have --token-path, but that's not very user-friendly for manually-started libertem-servers. It should be possible to automatically generate a token, and include it in the link that is displayed and in the link automatically opened in the browser.


"promise" being shown at useMutation return type and documentation

Hello everyone.
While creating my application, I found out that the MutationResultBase interface and the useMutation's documentation are mentioning a promise key in the MutationResultObject but promise isn't returned at useMutation.js.
Is it a bug in useMutation implementation or in the documentation?
I can open a PR for either, just want to know where the fix should be done. :)

I don't know much about the types per se, but mutate is asynchronous, so there are some generics and options in there for that promise. Feel free to open a PR with any other suggestions you may have. Thanks!


H1 title overlap with in-page navigation column

Original GitHub issue https://github.com/dotnet/docs/issues/7175 from @zhiliangxu
https://docs.microsoft.com/en-us/dotnet/api/system.func-17?view=netframework-4.7.2
Title is long: Func<T1,T2,T3,T4,T5,T6,T7,T8,T9,T10,T11,T12,T13,T14,T15,T16,TResult>
It overlaps with "in this article"

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

ID: d95cd525-746d-313d-b7f9-4d442d2319df
Version Independent ID: 930861a9-778b-7428-f582-58043958e890
Content: Func<T1,T2,T3,T4,T5,T6,T7,T8,T9,T10,T11,T12,T13,T14,T15,T16,TResult> Delegate (System)
Content Source: [xml/System/Func17.xml](https://github.com/dotnet/dotnet-api-docs/blob/master/xml/System/Func17.xml)
Product: dotnet-api
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

#log-suggestion

🚀 ATTENTION: Internal request logged.

@mairaw the bug should be fixed - I triggered a full content rebuild to validate that.

@dend it works fine on Chrome browser but doesn't work on Edge.

@Powerhelmsman the content is still being re-built on the live branch, so I can't tell yet whether the fix is propagated.

This issue has been fixed, and the header renders properly. Thank you all for the feedback!


big rewrite of tutorial, adding notebooks with and without answers. s…

…till need to address dask

For @fmaussion and @byersiiasa to take a look. Dask part still isn't there, I want to try to get some data from IIASA tomorrow.

Can we merge this or should I start to work on that branch too?

ok nevermind, working on this notebook now

hey @fmaussion, we're finishing up a few things in the dask section now. how doy ou want to proceed?

I have made cosmetic changes to the sections before dask. Let's merge this first, and see if git can sort this out for us? (I think it will)

ok, merged!


Still need to merge this branch?

@GrantPDonnelly, @johnridge , @gdhungana. I had misunderstood that this branch was merged to master.  I just merged @johnridge’s branch to this one, but rc1.0.5 appears still to be outside master.  Can you comment on the status of this overall branch?  I would rather merge once I know what the state is.

@rkehoe, rc1.0.5 was merged into master following @GrantPDonnelly's update to the ceph_tools documentation, however, master is now behind rc1.0.5 after my pull request that added integrated lccal/unconex. I have identified an issue with the latest update to lccal and I would recommend that we refrain from merging this branch into master until I have fixed it. The issue is that plots are still generated and the calibrated data is filtered with unconex even if their respective arguments are false. I have fixed the issue on my local repo and I should have a pull request up in a few minutes to merge my fixed fork.

@johnridge  and @GrantPDonnelly I have merged PR #11, so this PR should now be ready.  There are no conflicts, so I’ll merge in a minute.


SSL-Only Vhost

Is it possible to create an SSL only vhost so that nginx will only listen on 443.
There's ssl_only options for locations, but none for vhosts

This is covered in the README: https://github.com/jfryman/puppet-nginx#https-only-vhost


Update authConfig.js

The proposed changes improve the clarity of the configuration for developers using our documentation. This sample code is used in Tutorial: Sign in users and call the Microsoft Graph API from a JavaScript single-page app (SPA) using auth code flow

@derisen please review

#sign-off


KNE to use the arista-ceoslab-operator

These changes are to integrate the new cEOS-lab operator (shared privately with Marcus and Alex) with KNE. My assumption is that you guys would take it for a spin and if it looks alright we can proceed with open-sourcing and merging this PR. We have a small amount of red tape to cut through on our end.
This only works with a very recent version of cEOS because of a required fix to how read-only boot scripts are handled at startup. I'll be sharing this with you shortly and integrating this into the next drop.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

39 of 136 (28.68%)  changed or added relevant lines in 1 file are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-2.5%) to 53.481%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

topo/node/ceos/ceos.go
39
136
28.68%

Files with Coverage Reduction
New Missed Lines
%

topo/node/ceos/ceos.go
8
44.4%

Totals

Change from base Build<PHONE_NUMBER>:
-2.5%

Covered Lines:
1928

Relevant Lines:
3605

💛  - Coveralls

Okay things look good on my end, once the controller repo + controller container are opensourced and a mainline version of ceos that contains the necessary fixes for cert gen is released and ingested by us we can get this merged

Okay once that comment is addressed I will do my end to end test again (with the newly released ceos image) to make sure everything is working one last time and then we can move forward with finally getting this merged

okay doing a final check


Advanced administration features for rooms

When creating a room admins would be able to:

Set answerable questions for users who request to join
Limit how many messages users can send in a row
Limit how many stickers or media files users can send in a row or within defined period of time
Ban sending of certain media files or all the media files
Create customized right sets for people. Right sets could be saved in some kind of right set pack like sticker pack. There could be for example type 1 users: are able to do everything and have admin rights, type 2 users: can send everything but no admin rights, type 3 users: can only send text messages.

5 is already possible with how the permissions system works.

points 2, 3, 4 are not compatible with E2EE

I would say that there is plenty of demand for 2,3 and 4 to prevent spamming in public rooms. One of the problems of Telegram in bigger chat groups is sticker spam and there are no efficient tools to limit it.

2,3,4 would require changes to the s-s federation consensus protocol also so are major changes

Also if folder sharing is going to be implemented administrators would be able to set limit how much data people need to share to enter the room.

I feel like

how much data people need to share to enter the room.

is infeasible. What's to stop me running a client which claims I'm willing to share 20TB of data, but when someone asks for it I pretend to be unavailable?

Admins can check what people share by investigating the folder that they share. They can try to download some files from people and check if their size is the claimed size.

But there could be some kind of check that if client claims to share x amount of data, it needs to have folder actually shared.

Right but that check is technically impossible, I could share 20TB of utter crap, like Lorem Ipsum or RNG images or whatever

A lot of this is supported by other parts of the API, or by particular homeservers. For instance, if you're concerned about spam then your homeserver (and therefore your users) can be protected by an anti-spam solution - a function of synapse you can plug in yourself.
As for custom sticker packs for particular users, that'd be a role for the integrations manager. Although it isn't a first class citizen yet (#1286), it would be the thing that would manage that sort of stuff.
If this is asking for spam control at the federation level, then that's a whole other thing (which is technically possible, but enforcement would be difficult).


Support X509 certificates in netstandard

Fixes #807

Awesome, thanks a lot! :smile:
What's the timeframe for a new NuGet release?

I would expect next Wednesday 2016-08-17. But this isn't a hard guarantee ;)
@henkmollema Just out of curiosity, are you planning to use X509 certificates with service accounts in .NET Core?

@chrisdunelm yes, I've been using X509 certificates with service accounts in .NET Core for a while now. I call the Calendar API using a service account and the REST API. Somehow I couldn't get the JSON client secret working. Anyway, it would be nice to use the Google Auth API's again.

@henkmollema interesting, thanks. If you think the JSON client secret problem is due to the .NET client libraries, please file an issue.


Crash in Windows

Hi,
when compressing multiple streams in parallel the whole JVM crashes.
Dump File is attached.
hs_err_pid3436.txt

Same happen in single thread with another resource.

Hi,
thank you for this feedback.
May I ask you, how your code in multi-thread and single-thread scenario looks like?
Does it correlate to this known issue: https://github.com/MeteoGroup/jbrotli/issues/2 ?
I also would like to give some background info. I'm working on adopting new brotli v0.6.0 release, where the above mentioned crash is fixed.

@nitram509 any update on adopting 0.6.0?
I think you already updated to 0.5.2 and fix a memory leak but I think you did not release a version containing those. Would you like/appreciate some help?

Hi,
of course, I always appreciate help 👍
This new 0.6.0 release seems very promising, because C-API is meant to be fixed.
This leads me to skip 0.5.2 and directly jump towards the newest release.
As I already looked into, there are quite a few changes, which probably will also introduce breaking changes for jBrotli - which I consider OK.
It's "just" more work than an ordinary "increase version + recompile + publish" move ;-)
I would love to see pull requests (also partially ones) or snippets to move forward.
Regards
Martin

Thanks for your quick reply Martin!
I will have a look tomorrow. My plan is to try compile master locally and then open a branch for 0.6.0 release and start porting one step at a time.

Just awesome 👍
Feel free to do intermediate PRs or something alike.


Rodoviária de Lisboa .gtfs data issue

Descreva o erro
When trying to use the gtfs data from Rodoviária de Lisboa, it seems that there is an issue with the UTF-8 format, not allowing us to use properly the data.
Can you please have a look and try to correct the issue?
Área(s) afetada(s)
Quais as áreas afetadas pelo erro identificado:

[ ] Área 2 (Rodoviária de Lisboa)

Kind regards
Mamede Barreiros

Hello,
I'm an independent developer, not afiliated with Carris Metropolitana. However, I don't understand your report as currently data consumers are not using operator-specific GTFS files.
The current GTFS in usage is a combined GTFS for all areas of Carris Metropolitana, named "CarrisMetropolitana.zip". The per area GTFS files are mostly for backward compatibility with consumers that onboarded the GTFS file at an earlier stage. If you have prior files from Rodoviária de Lisboa from other sources or from prior to the migration to Carris Metropolitana, these are no longer supported.

Are you reporting issues in the main CarrisMetropolitana.zip GTFS file?
The GTFS file is updated up to several times per week. Are those issues happening in the latest committed version? Do you know the commit hash or the exact timestamp of the file you downloaded?
Can you say specifically what file, line and what character is causing you issues with parsing the file?
Do you have a log output you can provide from your data consumer software?

Hello @mamedebarreiros, the area-specific GTFS files may not have the latest network information. It is strongly recommended to use CarrisMetropolitana.zip for all analysis. Regardless, we have updated today with new files. If you'd like you can test again. Thank you!
Also, thank you @ricardojorgerm for the quick response.

Hello @joao-vasconcelos,
Many thanks for feedback and support. Appreciated from our side and my technical colleagues are already using properly the data available.
Also thanks to @ricardojorgerm for the feedback and support.
kind regards


style: use tuple in type hints

Similar to #153.

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

Project coverage is 100.00%. Comparing base (b1fe832) to head (5112735).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #154   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           14        14
  Lines          248       247    -1
  Branches        19        19
=========================================
- Hits           248       247    -1

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


python310Packages.gaphas: 3.11.3 -> 4.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.gaphas is: GTK+ based diagramming widget
meta.homepage for python310Packages.gaphas is: https://github.com/gaphor/gaphas
meta.changelog for python310Packages.gaphas is: https://github.com/gaphor/gaphas/releases/tag/4.0.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.0.0 with grep in /nix/store/93xarzhbmwshm0bkc6k31zf70q7ixzkp-python3.10-gaphas-4.0.0
found 4.0.0 in filename of file in /nix/store/93xarzhbmwshm0bkc6k31zf70q7ixzkp-python3.10-gaphas-4.0.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
gaphor
python310Packages.gaphas
python311Packages.gaphas

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/93xarzhbmwshm0bkc6k31zf70q7ixzkp-python3.10-gaphas-4.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.gaphas https://github.com/r-ryantm/nixpkgs/archive/268fbf1f093e53d79750484b74090533784a1e7a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/268fbf1f093e53d79750484b74090533784a1e7a#python310Packages.gaphas

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/93xarzhbmwshm0bkc6k31zf70q7ixzkp-python3.10-gaphas-4.0.0
ls -la /nix/store/93xarzhbmwshm0bkc6k31zf70q7ixzkp-python3.10-gaphas-4.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

gaphor
python310Packages.gaphas
python311Packages.gaphas

Maintainer pings
cc @WolfangAukang for testing.

4.0 - GTK 4 only

Superseded by #283526


[release-1.13] add OTel provider validation

This is an automated cherry-pick of #38470

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio_release-1.13
f8f5f974101d6ad5a7b2fc842232723aba437228
link
true
/test integ-pilot-multicluster_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio_release-1.13
f8f5f974101d6ad5a7b2fc842232723aba437228
link
true
/test integ-pilot-multicluster_istio_release-1.13

integ-security_istio_release-1.13
f8f5f974101d6ad5a7b2fc842232723aba437228
link
true
/test integ-security_istio_release-1.13

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Too heavy WebRepository::CollectAccountStatistic() function (5+ minutes execution) with 100% CPU load

Describe the bug
100% CPU utilization once a hour due to WebRepository::CollectAccountStatistic() processing which takes more than 5 minutes to complete.
To Reproduce
Steps to reproduce the behavior:

Run node
Enable statistics logging ( pocketcoin-cli logging stat)
Enable benchmarking logging ( pocketcoin-cli logging bench)
Wait several hours
Examine execution statistics (grep "Height-\|Web"  ~/.pocketcoin/debug.log | grep ProcessNextHeight:)

Example statistics:
...
2024-04-15T06:23:18Z     - WebPostProcessor::ProcessNextHeight: 80.92ms
2024-04-15T06:29:52Z     - WebPostProcessor::ProcessNextHeight: 363978.11ms
2024-04-15T06:29:52Z     - WebPostProcessor::ProcessNextHeight: 187.70ms
...
2024-04-15T07:21:09Z     - WebPostProcessor::ProcessNextHeight: 1.85ms
2024-04-15T07:28:51Z     - WebPostProcessor::ProcessNextHeight: 358967.93ms
2024-04-15T07:28:51Z     - WebPostProcessor::ProcessNextHeight: 339.76ms
...
2024-04-15T08:27:32Z     - WebPostProcessor::ProcessNextHeight: 172.40ms
2024-04-15T08:34:23Z     - WebPostProcessor::ProcessNextHeight: 361021.17ms
2024-04-15T08:34:23Z     - WebPostProcessor::ProcessNextHeight: 7.62ms
...

Desktop (please complete the following information):

OS: Linux
Version: 0.22.4

Thanks for the comment! I am aware of the problem and I am thinking about optimizing these functions - it is necessary to collect statistics in a cumulative way, instead of counting every time for everyone.

Sometimes it prevents node shutdown too...

Yes, the node cannot close until all active database connections are closed - this is necessary to preserve the integrity of the database.

scheduled

It's still not fixed!


[README] install cfssl with brew

use brew to install the client tools on mac, because on high sierra it gives some issues otherwise

Duplicate of #308


Java enums Introduction and Structure

Description 📜
Added documentation for Java enum Introduction and Structure.
Fixes #3722

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[x] Java_Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@garvitraj is there any issue with the PR that I need to resolve?

@AkMo3 there was another issue assigned on those content that's the reason I didn't add it under this. Rest for everything I have added the examples like the value method and all.

In the issue created named Java17.2 : Java enums class and inherited methods, I believe that constructors and methods should be covered there. Still if required I will add.

@techabhi08 Just give examples. No need to explain them as I suggested.

@AkMo3 I have closed this PR and opened a new PR. I was not able to make changes to it. Please review that one.


CMake CMP0064 policy warning

@ 5f8911cab86df546cc18bceccd817f44482b027f
CMake Warning (dev) at src/iamf_enc/dmpd/asc/3rdparty/decimate/CMakeLists.txt:13 (if):
Policy CMP0064 is not set: Support new TEST if() operator.  Run "cmake
--help-policy CMP0064" for policy details.  Use the cmake_policy command to
set the policy and suppress this warning.
TEST will be interpreted as an operator when the policy is set to NEW.
Since the policy is not set the OLD behavior will be used.
This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) at src/iamf_enc/dmpd/asc/3rdparty/stft/CMakeLists.txt:14 (if):
Policy CMP0064 is not set: Support new TEST if() operator.  Run "cmake
--help-policy CMP0064" for policy details.  Use the cmake_policy command to
set the policy and suppress this warning.
TEST will be interpreted as an operator when the policy is set to NEW.
Since the policy is not set the OLD behavior will be used.
This warning is for project developers.  Use -Wno-dev to suppress it.

This is still present @ f48a9398c41ee9ccfa0c5e5a1c36b0b2471c40f5.


My turnip prices spiked unable to compute

my turnip prices spiked and the graph isn’t able to predict. Morning was $198 afternoon was $348
Things to check

Have you forced a refresh in your browser (e.g. ctrl+F5 in Chrome/Firefox)?

yes/no

Have you made sure the first time buyer option is correct? If you're unsure, try it both ways

yes/no

Have you time-travelled this week? Travelling backwards resets the prices, and therefore you must not put prices in from both before & after the time-travel

yes/no

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Permalink to your prices
Please click the Copy Permalink button and paste it here
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Closing issue since the bug report template wasn't filled in.


Add support for adding an active class to the slider handle(s)

Touch/drag input doesn't work with the :active pseudo selector. This allows us to style the :active state directly with a className by manually managing this state

👍


flip_x should always be disabled when asking permissions

For readability ;)

That's really strange, I thought I handled all those cases.  Can you tell me how you reproduced this?  Did you use one of the built-in demos?  What browser / OS?

It only happens after the first time permissions are granted. e.g. open camera, grant permissions, accept photo, open camera, see flipped permissions dialog. I'm testing on chrome latest.

Darn, I am unable to reproduce it from here.  It never re-asks me for permissions after the first photo, and Chrome uses getUserMedia, not Flash.
I'll keep trying to figure it out tho.

Oh hmm. I'll keep testing across browsers. I'm using force_flash on Chrome, for now.

Ah, I see.  I don't think I've tested the combination of both force_flash and flip_horiz on Chrome.  I'll do that as soon as I can, and try to correct the bug.  Thanks!

Okay, let me know if this fixes it.
https://github.com/jhuckaby/webcamjs/commit/fa2514fa4b48aea707d5c6ffed0705795c7ae8d2

Joe

Do I need to call reset manually? I don't see it referenced anywhere else.

Ah, I was assuming you were testing on a page that called reset(), because I cannot figure out any other way that Flash could have showed you the permissions dialog again after the initial time.  This was a shot in the dark fix at best.
If you aren't calling reset, then the bug is elsewhere.

Hello huckaby,
This is third mail from me regrding the issue, I have
already sent lines of my code to work in IE and safari. The IE is not able
to load the flash content when loading the page. As far the demo you have
shown, please try webcam.on(error) or live, function triggered once the
user presses allow or deny in the flash dialogue box. I am not getting the
image captured in IE and safari. after webcam.on(live) function, In all the
other browsers, html5 getusermedia is supported. I have tried with firefox
17, which goes with flash fallback, and it worked fine with the image
capture. Please, let me know how can I rectify this error. I really
appreciate your good work.
On Fri, Apr 10, 2015 at 12:01 AM, Joseph Huckaby<EMAIL_ADDRESS>wrote:

Ah, I was assuming you were testing on a page that called reset(),
because I cannot figure out any other way that Flash could have showed you
the permissions dialog again after the initial time. This was a shot in the
dark fix at best.
If you aren't calling reset, then the bug is elsewhere.
—
Reply to this email directly or view it on GitHub
https://github.com/jhuckaby/webcamjs/issues/64#issuecomment-91320288.

@mani541 I think you have the wrong thread.

Aha! I found it! Flash ask permission again if the containing div is set to display: none. It automatically disables the webcam if the swf is hidden from view. Tricky!
Maybe there is some way to detect this on the flash side?

Webcam.set(
{
width: 320,
height: 240,
image_format: 'jpeg',
jpeg_quality: 90,
});
Webcam.attach( '#cam' );
Webcam.on( 'error', function(err)
{
error = true;
alert("We could not access the webcam. Please allow the Webcam Access to
Proceed");
Webcam.attach( '#cam' );
});
Webcam.on( 'live', function()
{
error = false;
$('#cam').hide();
captureimage();
});
I have just loaded the js file, and tried setting eith above code, but I
couldn't make this work in IE and safari. Do you have any suggestion to
make?
On Fri, Apr 10, 2015 at 12:19 AM, Josh Beckwith<EMAIL_ADDRESS>wrote:

@mani541 https://github.com/mani541 I think you have the wrong thread.
—
Reply to this email directly or view it on GitHub
https://github.com/jhuckaby/webcamjs/issues/64#issuecomment-91325377.

@mani541, if you file an issue here https://github.com/jhuckaby/webcamjs/issues it will be much easier to help you. Currently, you're hijacking a thread about something totally unrelated.

Thanks for your suggestion. I just overlooked a mail, and sent it to you.I
have no intention to hijack your thread.
On Fri, Apr 10, 2015 at 12:37 AM, Josh Beckwith<EMAIL_ADDRESS>wrote:

@mani541 https://github.com/mani541, if you file an issue here
https://github.com/jhuckaby/webcamjs/issues it will be much easier to
help you. Currently, you're hijacking a thread about something totally
unrelated.
—
Reply to this email directly or view it on GitHub
https://github.com/jhuckaby/webcamjs/issues/64#issuecomment-91330071.

Ah, good find @positlabs!
I have no idea how to detect that from the Flash side.  In JavaScript, I believe Webcam.container.offsetWidth will be 0 if any parent element is collapsed with display:none, but I don't know exactly when to act on this information.  Meaning, I have no idea when you restore the DIV, and Flash shows you the permissions dialog the second time -- there is no notification event.
Would it be possible to have you call Webcam.unflip() when you hide the containing DIV?  This will reset the styles and remove the CSS transform that mirrors it horizontally.

Yep, that does the trick. I don't think this is really worth addressing in webcamjs since it's such an edge case, so I'll close the ticket. Thanks!

Cool, thanks!


Add witness builder

This is the same witness builder PR I made to serialization-lib that never got merged

Merged since this was reviewed previously when submitted to CSL


Allow specifying the NPM version via a marker or environment variable.

This is an attempt to solve #22. A npm version can be set either via the NPM_VERSION_URL environment variable or by setting the .openshift/NPM_VERSION_URL marker.
Two things might be worth discussing:

I also allow setting the npm version via an environment variable. This is a direct result of #23, not sure whether you want to include this.
In theory there is no need to use a semver.io URL, npm install could resolve the version. So instead of setting an URL we could just allow setting the version and npm would handle something like npm install -g<EMAIL_ADDRESS>The reason I chose the URL approach anyway are:

a) It is consistent on how the node version gets resolved
b) It is more flexible, e.g. I could run my own service where I can configure the node and npm version to use for all my gears

Thanks, Philipp, and sorry for not being able to reply sooner. This definitely looks interesting and will hopefully pull it tomorrow. I'll also make sure to update the README.


Add HelloWorld.fai

Description

Added "Hello, World!" for FAI.
Types of changes

[ ] Bug fix or improving code quality
[x] New language
[ ] New feature

Checklist:

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.

Hey, @TheUnlocked! Would you be interested in adding your contribution to a repo that's maintained?


voter long options.

Currently I have added it as following -
usage: start-voter.sh[bat]
-h,-help                                Help
-o,-override host:port                Override vote
-s,-server host:port[,host:port...]   Server host:port

usage: start-voter.sh[bat]
-help                                Help
-override host:port                Override vote
-server host:port[,host:port...]   Server host:port


Remove unexpected build flag --reserve-size

When building using the shootout makefile in the container with:
cd /lucet
make bench
or outside with:
./devenv_run.sh make bench
an error occurs:
error: Found argument '--reserved-size' which wasn't expected, or isn't valid in this context
Did you mean --min-reserved-size?
This patch removes the unrecognized flag.

Looks like this has been patched. Will abandon.


autoConfig and debugging

syntax meets eslint-airbnb
added option to set autoConfig
added option to enable debug logging while in development

Thanks for the PR. I will test it soon and get back to you. One thing that i wish we can do here is removing this login part in dist file so there aren't any extra checks of debug in final bundle.

The problem with that is the npm package refers to the dist file so debugging would never be included. Overall I don't think you need to be using webpack and instead just regular a regular babel build to ./lib since most people using this plugin will be running their own webpack builds anyway like I am, you could still include a dist file for anyone including it directly by script tag and in there perhaps remove the option for debugging but an npm package without debugging is mostly useless to me.

For debugging of pixel, there is pixel chrome extension that shows all the event.

Yeah you can also pretty easily spot them in the Network tab just easier for me if I don't have to keep switching or use 3rd party extensions.
Completely up to you if you want to include it, I use react-ga which has similar logging and has been helpful, either way I published to a private npm package to use for now.

@zsajjad The ability to change autoConfig settings is pretty important. I recommend merging this.


Unable to turn off auto_constraint in virtual hierarchy

Please see the failing test: test_auto_const.py::test_disable_auto_constraint_virtual
Despite the constraint: {"constraint": "ConfigureCompiler", "auto_constraint": False, "propagate": True}
compiler is still generating auto-constraints for virtual hierarchy.

This PR is ready for review.


Horizontal axis numbers price history chart

These numbers need to go over the lines not under....
see screenshot

fixed - closing


State messages received from terminated vehicles

If a car leaves the traffic, that means it sent an empty join message. The designated state client for that car unsubscribes from its state topic, yet it still may receive state messages after the unsubscription. This results in log warnings, because we cannot find the car for which the state messages arrived.

Seems like the many warnings was because of the connection to the maqiatto.com broker.
Unrelated to the branches


Example project is not working on Raspbian Linux.

Raspberry Pi 4B is supposed to have Bluetooth LE. I tried to run the example project from bluetooth_low_enery/example directory. The example app running without any error. But the screen has no widgets.

This is caused by the PeripheralManager api is unimplemented on Linux, just comment the await peripheralManager.setUp() in onStartUp method should resolve this error.
void onStartUp() async {
  Logger.root.onRecord.listen(onLogRecord);
  WidgetsFlutterBinding.ensureInitialized();
  await centralManager.setUp();
  // await peripheralManager.setUp();
  runApp(const MyApp());
}


Use = to separate a named parameter from its default value

This fixes the health suggestions from pub.dartlang.org:

Thanks, @LinusU !


Fixed most common vulnerabilities

Fixes:

Disallowed players to use the toolgun for props, which are used for sitting.
Example: This prevents players from using precision to push through walls
Disallowed players to undo actions from Precision tool.
Info: This fixes a common way for minges to get into skybox and create lags for the server. Example (0:13)
Disallowed players to use physgun on props, which are used for sitting.

Thanks!


Running mutation tests against Ooze itself fails

After introducing https://github.com/gtramontina/ooze/pull/5, running mutation tests against Ooze itself (and I think it is safe to assume that other projects too) fails with the following (or similar) error:
https://github.com/gtramontina/ooze/actions/runs/4843531326/jobs/8631215763#step:4:26
Error: 	panic: failed type checking for file 'internal/cmdtestrunner/cmdtestrunner.go': internal/cmdtestrunner/cmdtestrunner.go:7:2: could not import github.com/gtramontina/ooze/internal/ooze (can't find import: "github.com/gtramontina/ooze/internal/ooze")

goroutine 6 [running]:
testing.tRunner.func1.2({0x6ef2c0, 0xc000202d20})
	/opt/hostedtoolcache/go/1.18.10/x64/src/testing/testing.go:1389 +0x[24](https://github.com/gtramontina/ooze/actions/runs/4843531326/jobs/8631215763#step:4:25)e
testing.tRunner.func1()
	/opt/hostedtoolcache/go/1.18.10/x64/src/testing/testing.go:1392 +0x39f
panic({0x6ef2c0, 0xc000202d20})
	/opt/hostedtoolcache/go/1.18.10/x64/src/runtime/panic.go:838 +0x207
github.com/gtramontina/ooze/internal/gosourcefile.(*GoSourceFile).Incubate(0xc000124bd0, {0x7ab3e0?, 0xc000010078})
	/home/runner/work/ooze/ooze/internal/gosourcefile/gosourcefile.go:46 +0x453
github.com/gtramontina/ooze/internal/ooze.(*Ooze).Release(0xc0001dfe50, {0x95f6e0, 0xe, 0xc0000b7e30?})
	/home/runner/work/ooze/ooze/internal/ooze/ooze.go:83 +0x2[25](https://github.com/gtramontina/ooze/actions/runs/4843531326/jobs/8631215763#step:4:26)
github.com/gtramontina/ooze.Release(0xc000097860, {0xc0000b7f[30](https://github.com/gtramontina/ooze/actions/runs/4843531326/jobs/8631215763#step:4:31), 0x6, 0x3a?})
	/home/runner/work/ooze/ooze/release.go:138 +0xa3a
github.com/gtramontina/ooze_test.TestMutation(0x0?)
	/home/runner/work/ooze/ooze/ooze_mutation_test.go:12 +0xeb
testing.tRunner(0xc000097860, 0x75a558)
	/opt/hostedtoolcache/go/1.18.10/x64/src/testing/testing.go:1439 +0x102
created by testing.(*T).Run
	/opt/hostedtoolcache/go/1.18.10/x64/src/testing/testing.go:1486 +0x[35](https://github.com/gtramontina/ooze/actions/runs/4843531326/jobs/8631215763#step:4:36)f

Interesting. I guess simply using importer.Default is not enough. golangci-lint seems to use a custom importer. Will have to dig deeper into this.


Next Gen Routing

[Short description explaining the high-level reason for the pull request]
Additions

Removals

Changes

Testing

Screenshots
Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows project standards (link if applicable)
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[ ] Windows
[ ] Linux
[ ] Browser

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

Since this is targeted to merge to a development branch and now master, I support merging this and working from the development branch to improve the implementation and integration.

Since this is targeted to merge to a development branch and now master, I support merging this and working from the development branch to improve the implementation and integration.

I have a few things going on this week. I plan to make a few updates to this next week, which I think would be best to have before merging to a development branch.

I'll push some of my changes to your fork for now


Compress animated webp

Currently compressing animated webp result in a still image instead of the animation.
Any way to supply an regexp to match animated webp files would be brilliant.

I've added animated: true to the animated branch. You can try to install with, as a temporary solution:
npm install -D -E astro-community/astro-compress#animated

I'll look into this further, as there are a few other problems with webp that do not surface on the regular.

had same issue with gif (& webp) images where the image becomes static after compression

also had same issue with gif (& webp) images where the image becomes static after compression

Try the animated branch: npm install -D -E astro-community/astro-compress#animated I still haven't updated astro-compress to detect automatically but it should work just fine.


Clean up UpdateEngine() and UpdateReplica() in the volume controller

The current logic updates the engine and replica immediate in the volume controller after we modified it. It may cause conflict later if the engine/replica controller response to the update and change the status, but the volume controller still need to update the engine and replica further down in the code path.
It caused the issue like engine/replica controllers constantly fetches the log and update the status, so the volume cannot continue with updating engine/replica to detached/attached.

Done.


Canvas tiles instead of img tiles

Is it possible to use canvas tiles instead of img tiles?
I'm having bad performance on mobile because I have to many markers.

@tombatossals

Mmm... there's a canvas layer in Leaflet, here documented:
http://leafletjs.com/reference.html#tilelayer-canvas
But this is a canvas where you are the responsible of drawing things, so you would need to manage your markers inside the canvas, I see it a little difficult.
Any hints?

This issue was moved to angular-ui/ui-leaflet#165

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


fix #14: reduce number of mandatory fields in SwitchDevice

This is a quite PR to resolve #14 as I was seeing exactly the same issue. I have removed all the Field from SwitchDevice except for name and description and migrated these down to the concrete classes. The Regex code would have been repeated across three of the concrete classes so I've added a class in the hierarchy called "RegexSwitchDevice" which contains this common code.
I've also done a couple of things which you may like me to back out, and I'd be happy to do so:

I've knocked the minimum python version up to 3.10 as this version introduces the kw_only keyword to the dataclass decorator and greatly cleans up dataclass hierarchies. No longer is it necessary to have all the mandatory fields strictly before the optional fields throughout the tree.

I've explicitly declared SwitchDevice and RegexSwitchDevice to be abstract classes by inheriting from abc.ABC - under the hood this is exactly the same as throwing a 'NotImplementedError' but it looked a little cleaner to my eye.

If you still have an appetite for it, I'd be happy to bash out a few of the other things on your todo list. I notice that you'd like to migrate away from print and instead use logging? That shouldn't take too long. I'd also be happy to take a look at fixing your SmartThing test.

Hey thanks for the interest. The changes look good and I approve of 1. and 2.
I'm not really using this repo much at the moment as I came to the conclusion that MAAS was too much overhead for my tiny cluster of 4 RASPIs - but I had fun making this and happy to collaborate on any further changes you are interested in.
You have PR CI permission now and I see that CI has failed. No time to look at this this evening but will take a look probably Sat morning.
Cheers,
giles.

Great stuff! We're making extensive use off MAAS and of course our PDUs aren't natively supported, so I'm happy to put in quite a bit of time to maaspower, it's been a real life saver.
CI is failing cause the docs arent building. I confess I didn't test that before I raised the PR. I'll push a fix ahead of Saturday so it will all be green by the time you get there.

Documentation is working locally for me now - the pipenv lock file was out of date. I've pushed up the change, so hopefully the build will now succeed.

Thanks for the contribution.

@rvodden I've just noticed that the CI failed on this. Mainly a python version thing.
I think we should drop support for 3,8 but need to look at why 3.12 is failing and also 3.12 will be the cause of lint failing

Ah. I'm a little busy tonight, but I can take a look tomorrow morning.

great, thanks!


Taskbar Autohide Quick Toggle

Description of the new feature / enhancement
A keyboard shortcut or icon placed in the tray that allows a toggling on and off of the taskbar autohide feature.
Scenario when this would be used?
Moving between laptop and tablet (touch) mode for hybrid devices such as the Surface.
Supporting information
No response

/dup #16641


chore: jwt: SAST (sonar) Fixes

JWTActiveScanRule > Removed unnecessary constructor.
JWTConfiguration > Properly create logger. Remove unnecessary brackets, and use isEmpty for Set content check.
JWTConstants > Create a constant for the component regex. Use diamond operator.
JWTExtension > Remove overridden method that simply uses a call to the super method anyway.
JWTFuzzPanelView > Use SwingConstants where applicable. Use lambdas
for anonymous inner classes. Remove unnecessary brackets.
JWTHolder > Don't use intermediate assignment when returning.
JWTI18n > Add private constructor to hide the implicit public one.
JWTMessageLocation > Remove duplicate condition.
JWTOptionsPanel > Use lambda for anonymous inner class.
JWTUIUtils > Add private constructor to hide the implicit public one.
JWTUtils > Add private constructor to hide the implicit public one.
SignatureAttack > Remove unnecessary throws declaration. Actually throw the constructed exception.

Note this seemed to build fine, but I don't have any kind of testing environment.
Signed-off-by: kingthorin<EMAIL_ADDRESS>

With SonarLint in Eclipse.

Addressed all those.

Thanks you @kingthorin

No problem, happy to help!


A11y button fix

Added type=button back and removed unnecessary tabindex=0
https://github.com/kenwheeler/slick/commit/98631b2ba5d01662921bb9ff5fbff6bdbc80c257#commitcomment-22595882

@scottaohara look good?


Upgrade Domain Controllers: update 2 URLs

Description:
As reported in issue ticket #3373 (Broken link), the KB link to "Disable the NoLMHash policy" in the Deprecated features section is broken. The target page has been removed from the Microsoft Support pages.
Thanks to Mike Lazarus (@GLComputing) for reporting the issue.
While browsing the page for whitespace issues, random other issues, and typos, I also found a broken link with the following text: For more information about why the Windows engineering team implemented these changes, see Minimizing restarts after automatic updating in Windows Update. - URL https://blogs.msdn.com/b/b8/archive/2011/11/14/minimizing-restarts-after-automatic-updating-in-windows-update.aspx
The page title text I found useful to place there instead: How to reduce your chances of being prompted to restart your computer.
URL: https://docs.microsoft.com/troubleshoot/windows-server/deployment/why-prompted-restart-computer#how-to-reduce-your-chances-of-being-prompted-to-restart-your-computer
Changes proposed:

replace the link and text "in KB article 946405." with the text "in the page How to prevent Windows from storing a LAN manager hash of your password in Active Directory and local SAM databases." and the URL https://docs.microsoft.com/troubleshoot/windows-server/windows-security/prevent-windows-store-lm-hash-password
replace the link and text "see Minimizing restarts after automatic updating in Windows Update." and the URL https://docs.microsoft.com/troubleshoot/windows-server/deployment/why-prompted-restart-computer#how-to-reduce-your-chances-of-being-prompted-to-restart-your-computer

Whitespace & Codestyle improvements:

added editorial blank line between the metadata and the page title
added missing MD indent marker compatibility spacing for the "Applies To:" line
added blank line between a H2 header and its section text (codestyle)
replaced period dot with a colon character at the end of 2 lines (preceding descriptors)
added missing dashes and spaces in a MarkDown table header (MD standard codestyle)
added missing NewLine at end-of-page/end-of-file (codestyle)

Ticket closure or reference:
Closes #3373

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

#sign-off

@illfated: I'm sorry - only the author of this article, @iainfoulds, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


[libzen,libmediainfo] update and no absolute paths

For #20469

Can you please merge to master?
Thanks.

Thanks for the updates!


[WIP] PSBT RPC calls

This compiles, but it doesn't work. Making this PR just some code can be shared.
Question on formatting: The format functionality in Intellij changed some whitespace here and there. Is there any configs I/we can tweak to make it work the same for everybody? I guess #172 is trying to tackle exactly that...

There's also some Strings that can be turned into more specific types here.


Feature request

Hi,
I need to generate RPSL output and I have made some small modification to the file output-csv.c to do just that. I have not made any modification in the rest of the source files needed to - for example - take an additional command line option.
Is it possible to have this feature integrated into the source? Perhaps better that I apply the patch after new releases.
br
/mm

Please just send us your change as a unified diff and we'll have a look at it. You can also do a pull request if that is easier for you.

Hi
I’m not skilled enough to add another output format the right way, so I just modified an already defined output format, csv, to generate RPSL
br
/mm

/*

Copyright (c) 2019 Claudio Jeker<EMAIL_ADDRESS>
Permission to use, copy, modify, and distribute this software for any
purpose with or without fee is hereby granted, provided that the above
copyright notice and this permission notice appear in all copies.

 IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*/

#include <stdlib.h>
#include "extern.h"
int
output_csv(FILE *out, struct vrp_tree *vrps, struct stats *st)
{
char		 buf[64];
struct vrp	*v;
    char            tbuf[26];
    struct tm       *tp;
    time_t          t;

    time(&t);
    setenv("TZ", "UTC", 1);
    tp = localtime(&t);
    strftime(tbuf, sizeof tbuf, "%FT%TZ", tp);

RB_FOREACH(v, vrp_tree, vrps) {
	ip_addr_print(&v->addr, v->afi, buf, sizeof(buf));
	if( v->afi == AFI_IPV4) {
	   if (fprintf(out, "route: %s\norigin: AS%u\ndescr: RPKI attestation\nmnt-by: NA\ncreated: %s\nlast-modified: %s\nsource: %s\n\n", buf, v->asid,tbuf,tbuf, v->tal) < 0)
		return -1;
	}
	   else
	   if (fprintf(out, "route6: %s\norigin: AS%u\ndescr: RPKI attestation\nmnt-by: NA\ncreated: %s\nlast-modified: %s\nsource: %s\n\n", buf, v->asid,tbuf,tbuf, v->tal) < 0)
		return -1;
}
return 0;

}

On 29 Dec 2020, at 17:44, Claudio Jeker<EMAIL_ADDRESS>wrote:
Please just send us your change as a unified diff and we'll have a look at it. You can also do a pull request if that is easier for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/rpki-client/rpki-client-portable/issues/29#issuecomment-752153377, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2U6DMO725MWBGZY2QBWX3SXIBQLANCNFSM4UR4DIFQ.

@job, what's your opinion about an RPSL output?
I think people should script something up using the JSON file instead of adding yet another output format.

@job, what's your opinion about an RPSL output?
I think people should script something up using the JSON file instead of adding yet another output format.

Dear @matsm
Thanks for reaching out! What exactly is the use case to generate RPSL output?
Keep in mind that modern IRRd (v4.1.0 and higher) have native RPKI integration, this means there no longer is a need to generate RPSL as the IRRd "generates" this itself.
Kind regards,
Job

Dear @matsm
Thanks for reaching out! What exactly is the use case to generate RPSL output?
Keep in mind that modern IRRd (v4.1.0 and higher) have native RPKI integration, this means there no longer is a need to generate RPSL as the IRRd "generates" this itself.
Kind regards,
Job

Hi

On 6 Jan 2021, at 12:54, Job Snijders<EMAIL_ADDRESS>wrote:
Dear @matsm https://github.com/matsm
Thanks for reaching out! What exactly is the use case to generate RPSL output?
Keep in mind that modern IRRd (v4.1.0 and higher) have native RPKI integration, this means there no longer is a need to generate RPSL as the IRRd "generates" this itself.

Ok, I will look into this.
br
/mm

Kind regards,
Job
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/rpki-client/rpki-client-portable/issues/29#issuecomment-755256982, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2U6DJ7CQINZAU6URZ3CSTSYRFNTANCNFSM4UR4DIFQ.

Hi

On 6 Jan 2021, at 12:54, Job Snijders<EMAIL_ADDRESS>wrote:
Dear @matsm https://github.com/matsm
Thanks for reaching out! What exactly is the use case to generate RPSL output?
Keep in mind that modern IRRd (v4.1.0 and higher) have native RPKI integration, this means there no longer is a need to generate RPSL as the IRRd "generates" this itself.

Ok, I will look into this.
br
/mm

Kind regards,
Job
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/rpki-client/rpki-client-portable/issues/29#issuecomment-755256982, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2U6DJ7CQINZAU6URZ3CSTSYRFNTANCNFSM4UR4DIFQ.

@matsm feel free to email me at<EMAIL_ADDRESS>if you want to discuss in more detail what your goals are and exchange suggestions on how to achieve those!

@matsm feel free to email me at<EMAIL_ADDRESS>if you want to discuss in more detail what your goals are and exchange suggestions on how to achieve those!


Add disableScrollHandling function

Fixes https://github.com/sveltejs/kit/issues/2794 by adding a new disableScrollHandling function. Calling it in onMount or inside an action will, if a navigation is currently happening, prevent Kit from applying its normal scroll handling. (Calling it when navigation is not currently happening will cause an error to be thrown.)
See https://github.com/sveltejs/kit/pull/2986#issuecomment-1002302203 for previous discussion.
I prefer the imperative disableScrollHandling to a declarative export const autoscroll = false for various reasons:

This should feel a bit awkward. Disabling default scroll behaviour will, in general, disorient your users. It should be a last resort. Adding an autoscroll page option feels like we're blessing that behaviour rather than simply tolerating it.
The implementation is more straightforward
If provides more flexibility — it's easier to use it inside an action (etc). Any page that could use autoscroll could also use disableScrollHandling, but the reverse isn't true

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

cc @bluwy since you've been looking a lot at scroll stuff lately

just one minor formatting comment. otherwise lgtm

Just wanna thank you so SO much for this PR! I haven't updated my Sveltekit projects since version 192 because of the navigation scroll issues. Much appreciated!