WiktionarySplitter Could not find or load main class com.hughes.android.dictionary.engine.Runner

When I start ./Wiktionary.sh after ./compile.sh, I get Error: Could not find or load main class com.hughes.android.dictionary.engine.Runner Caused by: java.lang.ClassNotFoundException: com.hughes.android.dictionary.engine.Runner
How can I solve that issue?
My operating system is OS X Catalina.

Are you sure there are no errors from compile.sh, that it didn't just fail to compile? E.g. due to an incompatible Java version?

It would be helpful to quote the error messages of the compile.sh step.
But I fixed some compile errors now so it probably is fixed.


Nav enhancement: compact header on scroll

Introduce the supporting CSS required to support the scrolled compact header seen here.
There will be a follow up story in PF-React to introduce full behavior. @mattnolting did an initial implementation.
Visuals

Closing since #833 merged


SDecodingError

Hello, I used the 'Polkadot< httprcclient >.create' call,   I encountered an error while using the SDK to get user information：
error description: error: decoding(error: ScaleCodec.SDecodingError.dataCorrupted(ScaleCodec.SDecodingError.Context(path: ["RuntimeVersionedMetadata"], debugDescription: "Unsupported metadata version 13")))

Yep. We will need to add metadata v13. But it will not help now, we are working on this library, it doesn't have all API types implemented yet. We will add everything in a couple of weeks.

Yep. We will need to add metadata v13. We are working on this library, it doesn't have all API types implemented yet. We will add everything in a couple of weeks.

ok, hope can use it soon 😁

Pushed new updated version. Should work now


Update .travis.yml

Uncommented after_success:  - codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0bbb345). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master      #2   +/-   ##
========================================
  Coverage          ?   94.8%
========================================
  Files             ?       2
  Lines             ?     250
  Branches          ?       0
========================================
  Hits              ?     237
  Misses            ?      13
  Partials          ?       0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0bbb345...5f631fb. Read the comment docs.


[Library Manager] Please add mimuz-ch55x library

Hello.
I created a library to addition simple MIDI comminucation to ch55xduino core.
Could you add this to a list of libraries?
url: https://github.com/mimuz/mimuz-ch55x
Thanks.

Hi @tadfmac. We have a new system for submitting libraries. The instructions are here:
https://github.com/arduino/library-registry#adding-a-library-to-library-manager
Please let me know if you have any questions.
Regards, Per

Thank's your support!


Add flag to strip test directory off of test defs

Users won't have to strip tests sub-directories from the output of this tool using an external language (I'm looking at you Groovy).  Python regular expressions are easier. :)

So gherkin-diff now takes a directory argument? Neat!

gherkin-diff.py --test-dir tests --start origin/master --end SOMESHA


Setup Yarn 3

Summary
Let's see how it goes
Test plan

@satya164 maybe we should commit the cache to the repo. Any experiences, pros/cons?

@thymikee cons: cloning the repo took significantly more time and made diffs harder to read when there were dep changes, pro: super quick to switch branches with different deps, simple CI setup since no caching-related config needed.

Build fails with example-native:build: Unknown Syntax Error: Unsupported option name ("--force"). Seems like our yarn install --force invocation.

@satya164 thanks. For now I think we'll keep the global cache behavior. when switching branches locally on a hot cache it's pretty fast already

Closing as stale.


Payload execution error

When the exe payload executed in shows no module named "scarpy"
Traceback (most recent call last):
File "jarbou3.py", line 22, in
ModuleNotFoundError: No module named '

fixed


[Bug]: The text shadow display on the TextEditor is not correct

Package Version
4.0.8
Flutter Version
3.22.2
Platforms
Android, iOS
How to reproduce?
Hi @hm21. I added a text shadow, but its display on the TextEditor is not correct. I think it must be display the same MainEditor.
https://github.com/hm21/pro_image_editor/assets/26729342/15b681e5-f283-49e6-81e7-fe46ef85408e
Logs (optional)
No response
Example code (optional)
IconButton(
          tooltip: 'Blue Shadow',
          color: Colors.amber,
          padding: const EdgeInsets.symmetric(horizontal: 8),
          icon: const Text(
            'Blue Shadow',
          ),
          onPressed: () {
            textEditor.setTextStyle(
              textEditor.selectedTextStyle.copyWith(
                shadows: <Shadow>[
                  const Shadow(
                    offset: Offset(1.0, 1.0),
                    blurRadius: 1,
                    color: Colors.blueAccent,
                  ),
                ],
              ),
            );
          },
        ),

Device Model (optional)
iPhone SE (iOS 17.0)

Thank you for reporting this issue. I just released version 4.0.10 which should resolve this issue. Please let me know if this issue persists by you after you updated.

Hi @hm21. Thank you. Its working.


Can't launch the game

Describe the bug
When I hit launch game, it says "Game is running (DO NOT CLOSE THE LAUNCHER), but after that, all messages disappear, I can hit the Launch Game button again, but no game opens.
System Information (please complete the following information):

macOS: Sonoma 14.6.1
Device: 24-inch, M1, 2021
CPU: Apple M1
RAM: 8GB

Hii I had the same issue i think it is with the wine patch they are using I used Crossover and its wine patch it worked and even the graphics are slightly better
CrossOver gives free access and is paid but you can always go to the sea and get it
System Information (please complete the following information):
macOS: Sequia 15.1 beta 3
Device: 13-inch, M3, 2024
CPU: Apple M3
RAM: 16GB


[EMB-108] Use encoded download URL for MFR URL

Purpose
Fix Quick Files share links
Summary of Changes

use encoded download URL for MFR URL
build fileURL from config.OSF.url and guid instead of window.location
build mfrUrl from config.OSF.renderUrl instead of hard-coded

Side Effects / Testing Notes
These links should now work in all environments.
Ticket
https://openscience.atlassian.net/browse/EMB-108
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Coverage decreased (-0.04%) to 4.673% when pulling 57325e7cc08d2161760e973a8a5705bbe97dd1c6 on jamescdavis:EMB-108_quick_Files_Share_links_on_file_detail_page_dont_work into ad87ebe9830dea48bff4ce386a6ea79093134087 on CenterForOpenScience:master.


Consider making this lib modular

It would be nice if one can do require('validator/isString') or require('validator/isUrl') instead of requiring the all library. Backend might not be interested by this feature, but for the frontend every bytes can count at the end.
An good example is lodash & underscore.string, once installed via npm. Those lib can be required as a all, or by method.

This is too much overhead for one person to manage so I'm not interested in going down this path (at this stage).
You're free to copy regular expressions or whole validator functions into your own codebase as long as you link back to the library and/or license.


Bug Report: additional role assignment needed on the plan sp

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform: 1.6.6
azure provider: 3.86
module: v0.1.8
Description
Describe the bug

When deploying an ARM Template (e.g. telemetry), as part of the ALZ deployment in Azure, the plan fails on re-deploy. This is because the plan principal does not have the correct permissions, namely, Microsoft.Resources/deployments/exportTemplate/action
Steps to Reproduce

Deploy accelerator with complete starter module ensuring the disable_telemetry variable for caf-enterprise-scale is set to false.
Run the delivery pipeline.
Re-run the delivery pipeline.

Screenshots

│ Error: retrieving Template Content for Subscription Template Deployment "xxx": resources.DeploymentsClient#ExportTemplateAtSubscriptionScope: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="AuthorizationFailed" Message="The client 'xxx' with object id 'xxx' does not have authorization to perform action 'Microsoft.Resources/deployments/exportTemplate/action' over scope '/subscriptions/xxx' or the scope is invalid. If access was recently granted, please refresh your credentials."
│
│   with module.enterprise_scale[0].azurerm_subscription_template_deployment.telemetry_management[0],
│   on .terraform/modules/enterprise_scale/resources.telemetry.tf line 19, in resource "azurerm_subscription_template_deployment" "telemetry_management":
│   19: resource "azurerm_subscription_template_deployment" "telemetry_management" {
Additional context

This was resolved in the latest major release where we added Bicep support. We now have custom role definitions.


msbuild task does not specify any outputs

The README states:

Prior to execution, this task will parse the provided project file and gather all its inputs (which are added to the task inputs):

included files (Compile, EmbeddedResource, None, Content)
ProjectReference (recursively gathers its inputs) // TODO: should use outputs instead ?
References with a HintPath

But when run, the task is found to be not up-to-date because "Task has not declared any outputs despite executing actions".
Is this expected behaviour? Is there an extra step required to register outputs? And if so, will this mean that with inputs/outputs registered, Gradle will do an up-to-date check without having to call msbuild directly?

Hi
To allow up-to-date check we should use the incremental build from msbuild (gradle does not understand msbuild's artifact)


Time zones read from Windows Registry lack historic DST rules

Windows has time zone information available in the registry. We reading that data, but only the currently active rules and extrapolate them to historic and future dates. This is error prone because DST rules tend to change over time.
Historical data is available in the Dynamic DST key under a time zone entry.
https://learn.microsoft.com/en-us/windows/win32/api/timezoneapi/ns-timezoneapi-dynamic_time_zone_information

Duplicate of #13518?


Have Martin group look over the updated sparse and dense MTs

I no longer think we need to have the Martin group look over the MTs. I did give Mary the sample annotation HT and asked her if she can take a look at that and make sure numbers are correct and outliers are correct.
I also got a script from Lindo on how to combine sparse MTs so I could test out that our sparse MT will actually combine with another subset. I used a subset of 15 gnomAD samples to test with and then used the following script to test that they can combine:
import hail as hl
import hail.experimental.vcf_combiner.vcf_combiner as vc
from hail import MatrixTable
from hail.ir import MatrixKeyRowsBy

tmp_path = 'gs://gnomad-tmp/test_sparse_mt_combine_tmp'

# USE CORRECT MT FOR mt_n
mt_n = hl.read_matrix_table('gs://gnomad/release/3.1.2/mt/gnomad.genomes.v3.1.2.hgdp_1kg_subset_sparse.mt')
mt_n = mt_n._filter_partitions(range(2))
mt_n = MatrixTable(MatrixKeyRowsBy(mt_n._mir, ['locus'], is_sorted=True))
parts, ty = vc.calculate_new_intervals(mt_n, 30, tmp_path)
#parts, ty = vc.calculate_new_intervals(mt_n, 120_000, tmp_path)

mt_n = hl.read_matrix_table('gs://gnomad/release/3.1.2/mt/gnomad.genomes.v3.1.2.hgdp_1kg_subset_sparse.mt', _intervals=parts)
mt_n = mt_n._filter_partitions(range(2))
mt_n = MatrixTable(MatrixKeyRowsBy(mt_n._mir, ['locus'], is_sorted=True))
mt_n.describe()

mt_k = hl.read_matrix_table('gs://gnomad-julia/gnomad_v3_1/small_sparse_mt_for_test.mt', _intervals=parts)
mt_k = mt_k._filter_partitions(range(2))
mt_k = MatrixTable(MatrixKeyRowsBy(mt_k._mir, ['locus'], is_sorted=True))
mt_k = mt_k.select_entries(
    'DP', 'END', 'GQ', 'LA', 'LAD', 'LGT', 'LPGT', 'LPL', 'MIN_DP', 'PID', 'RGQ', 'SB',
    gvcf_info=mt_k.gvcf_info.select(
        "ClippingRankSum",
        "BaseQRankSum",
        "MQ",
        "MQRankSum",
        "MQ_DP",
        "QUALapprox",
        "RAW_MQ",
        "ReadPosRankSum",
        "VarDP"
    )
)
mt_k.describe()

mt = vc.combine_gvcfs([mt_n, mt_k])

mt.write('gs://gnomad-tmp/gnomad.genomes.v3.1.2.hgdp_1kg_subset_test_sparse_join.mt')

Cluster:
hailctl dataproc start jg2 --master-machine-type n1-highmem-8 --worker-machine-type n1-standard-8  --properties=spark:spark.speculation=true --max-idle=540m --requester-pays-allow-all --autoscaling-policy=mw_gnomad_100

Job link: https://console.cloud.google.com/dataproc/jobs/419f2262fe3b42629a5ff71292ee6160/monitoring?region=us-central1&project=broad-mpg-gnomad
This worked, so I think we are good to go on the sparse MT, and are just waiting on Mary to have a look over the sample annotations.
Just a note that for combining the MTs, the rows need to all be present and in the same order, which is why I added the select. For the gnomAD MT (including on the raw MT) we have minimal entry annotations, so in order to join the other MT does need to be subset to only the entries present in the gnomAD sparse MT.


Adds Faculty page (WIP)

Have 1 bug IDK how to sort.  Tried setting css property, but still couldn't get it to work. Otherwise, all the elements are functional.
Deployed @ iiittd.surge.sh. Still haven't resolved the problem in surge.. will do it asap..so for now, page has to be accessed from the navbar only

i am seeing it.
Always refer to the issue you are solving in PR.
example Issue @ https://github.com/helloworld-iiitt/iiitt/issues/7
Github will show on the issue automatically that you refernced.

AFAICS

The image size is constrained by the container sizes we've kept.
Either we can fix height or the width.
So that's just happening bcoz of different image sizes(resolutions)
Leave it as it is,will look for images of uniform sizes.
Good work!! Finish the Staff Page as well(Quite Identical I Guess)

Added Staff and Faculty pages (#7). (WIP) Footer is buggy (doesn't stick to bottom of page)

@dinskid  push your changes to iiittd.surge.sh as that's where I can preview!!!
Always do that

Sorry about it.. I thought I did it, but looks like I didn't. Just a warning, that page looks ugly if I use just the current components. I didn't know how to design it just having 4 elements.
As always, it is in iiittd.surge.sh

@dinskid  Try to pull the latest changes from master branch.I've fixed the footer.
I am sure it would resolve the issue.

Also added the home icon against issue #9


SQL MI disk IOPS

IOPS count and bytes are listed under "Resource usage stats for managed instance". Are these representing the IOPS against the user database disks, i.e. the limits that are listed in this link?
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-managed-instance-resource-limits#file-io-characteristics-in-general-purpose-tier
Or are these metrics showing the IOPS against the system database disk? If so what limits is it subject to?

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

ID: 57944e9d-0efa-3503-9fe8-46c91119ecc9
Version Independent ID: 32caa5c0-aac4-75f8-42f8-b483d55a8392
Content: Metrics and diagnostics logging - Azure SQL Database
Content Source: articles/sql-database/sql-database-metrics-diag-logging.md
Service: sql-database
Sub-service: monitor
GitHub Login: @danimir
Microsoft Alias: danil

Thank you for reaching out.
We are actively investigating and will get back to you soon.

This is being assigned to the content author to clarify and update the documentation as appropriate.

These IOPS relate to cumulative IOPS for all databases running on the instance.

Thanks @danimir! Please consider documenting that either on this page or in the metric definitions.


Allows the method to be chosen in OidClient

The method must be one of the methods provided by the endpoint metadata

PR: https://github.com/pac4j/pac4j/pull/496

Additionnal PR (1.8.x): https://github.com/pac4j/pac4j/pull/507

PR for master: https://github.com/pac4j/pac4j/pull/508


Something unknown happened.

Hello,
I can’t connect, I get a “Something unknown happened” error.
What should I do?

Ultimately I'm not sure how Culligan devices interact. I don't know if they are regional or not. I added a region selector because the underlying Ayla API has different endpoints for certain EU devices.
You can try:

Setting region to Elsewhere
Enable debug logging and uploading here

Currently, the sign in code doesn't pass the EU bool to Ayla.

It doesn't change anything by activating EU or other.
And how to activate the journal since I can't activate it?

Enregistreur: custom_components.culligan
Source: custom_components/culligan/config_flow.py:94
intégration: Culligan (documentation, problèmes)
S'est produit pour la première fois: 12:10:26 (2 occurrences)
Dernier enregistrement: 12:10:39
Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/culligan/config_flow.py", line 94, in validate_input
await culligan.async_sign_in()
File "/usr/local/lib/python3.12/site-packages/culligan/uniapi_culliganiot.py", line 158, in async_sign_in
self._set_credentials(resp.status, await resp.json())
File "/usr/local/lib/python3.12/site-packages/culligan/uniapi_culliganiot.py", line 120, in _set_credentials
self._ayla_access_token     = login_result["data"]["linkedAccounts"]["ayla"]["access_token"]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^
KeyError: 'ayla'

You can try  v1.3.5-beta1 and see what debug/error messages are different.

There is a regex that is supposed to capture most valid email addresses. You can test to see if this regex is not validating your email and let me know what is missing.

sorry I didn't understand what to do (I'm French)

I recreated an account with a Gmail email address (before it was Hotmail), and I managed to connect.
Now the integration finds nothing.
Enregistreur: homeassistant.config_entries
Source: config_entries.py:635
S'est produit pour la première fois: 23:05:40 (1 occurrences)
Dernier enregistrement: 23:05:40
Error setting up entry Culligan - for culligan
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 635, in __async_setup_with_context
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/culligan/init.py", line 111, in async_setup_entry
if len(culliganiot_devices) > 0 and len(culligan_devices) > 0:
^^^^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'culligan_devices' where it is not associated with a value

So what I just saw is weird.
My new one already doesn't work anymore and the old one works. In fact, as soon as I add my softener to the app, I can't connect to the integration anymore.

You can try beta2. I tried to add additional logic to the device array selection code

Beta2
Enregistreur: homeassistant.config_entries
Source: config_entries.py:635
S'est produit pour la première fois: 15:35:33 (1 occurrences)
Dernier enregistrement: 15:35:33
Error setting up entry Culligan - for culligan
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 635, in __async_setup_with_context
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/culligan/init.py", line 111, in async_setup_entry
if culliganiot_devices and culligan_devices:
^^^^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'culligan_devices' where it is not associated with a value

Gotcha, so something is happening when the integration queries for devices ... and the checks aren't working as expected.
If you have debug logging enabled, there should be a few lines of logs regarding: x num of devices, etc.. Do you have any of those debug logs?

Where can I find these newspapers?

A great point. If you have VS Code plugin or another way to edit code, you can modify this integration's supported classes and include the class from the library. It doesn't include the generic CulliganIoTDevice, which it the device would possible be cast to.
If you can confirm that Smart Modernity is your device["name"], I'll get those changes added and we can test them.

I released beta3, which bumps the version for the Culligan library which now includes Smart Modernity. Fingers crossed for some progress :)

And thank you for the support in developing a fix!
1.3.5 released


RFC proposal around building and delivering different features

What does this PR do?
It proposes an RFC
Motivation
Making an evolution to the build system
Additional Notes
N.A.
How to test the change?
N.A.
For Reviewers

[ ] If this PR touches code that signs or publishes builds or packages, or handles credentials of any kind, I've requested a review from @DataDog/security-design-and-guidance.
[x] This PR doesn't touch any of that.

Thanks @r1viollet - looks perfect from my perspective (and to my existing knowledge)

I would recommend a mix of solution 1 and 2. For the languages that require shared libraries, we can use the feature solution (solution 2).
We can deliver a single static library (with all features) for languages that can link against a static library.

LGTM ! - worth double checking with @Anilm3 to ensure it aligns with ASM experience

Thanks @r1viollet - looks perfect from my perspective (and to my existing knowledge)

I would recommend a mix of solution 1 and 2. For the languages that require shared libraries, we can use the feature solution (solution 2).
We can deliver a single static library (with all features) for languages that can link against a static library.

LGTM ! - worth double checking with @Anilm3 to ensure it aligns with ASM experience

We don't have the issue of multiple distinct feature-sets within the single native library so perhaps my feedback isn't particularly helpful.
In terms of the options suggested:

Seems perfectly reasonable, I think the static library size is not particularly important as long as it's within reason. The only concern is the size of the final deployed binary and primarily only a concern for serverless, although it can still be an annoyance in other environments due to the need to deploy a binary per supported architecture and libc.
I'm not sure if this will scale well as you add more distinct features into libdatadog, but it's a start and if the goal is for everyone to use a single implementation I suppose languages will start to converge towards the complete libdatadog binary.

On the other hand, this is an age-old problem which is traditionally solved by having multiple interdependent dynamic libraries. The most simple scenario is the one in which you have a core dynamic library and then multiple others dependent on it. No doubt there's still some redundancy and this typically doesn't involve statically linked dynamic libraries only depending on each other and libc so I'm not sure if it's feasible.

@Anilm3 thank you for the feedback. To sum up, you are OK, though feel like solution 2 could get out of hand.
To answer this, I think only Ruby will continue linking dynamically (and we are talking about removing this requirement)
I can list the solution around having many different dynamic libraries as an alternative.
Though I am not in favour of this. This will increase the likelihood of duplication and adds complexity. The aim will be to integrate modules together (example: symbolization can be done within the same step as pprof aggregation), and reduce the need for FFIs. Happy to reconsider if someone can make a strong case for this.

@Anilm3 thank you for the feedback. To sum up, you are OK, though feel like solution 2 could get out of hand. To answer this, I think only Ruby will continue linking dynamically (and we are talking about removing this requirement)

That's interesting as with libddwaf, libraries have generally headed in the other direction (using the dynamic library more than the static one).

I can list the solution around having many different dynamic libraries as an alternative.
Though I am not in favour of this. This will increase the likelihood of duplication and adds complexity. The aim will be to integrate modules together (example: symbolization can be done within the same step as pprof aggregation), and reduce the need for FFIs. Happy to reconsider if someone can make a strong case for this.

I have no strong opinion about including this as an alternative option and I completely agree with the complexity argument. In most cases the approach I mentioned is typically used for general purpose libraries, while what we aim to do here is a little bit different.

@Anilm3 Thanks, I added the extra option. The fact it does not apply here does not mean we should not document it as a possibility

I agree that we need to be mindful to avoid a features explosion, but option 1 and 2 does make the most sense.
Yes, that is a risk. If we turn around and we have 10 different CI pipelines, we will know that we failed somewhere along the way.

Thanks for the feedback!


Debugging 422 errors when generating preview

I notice that the preview API callback handler has this:
    } catch (error) {
      return response.status(422).end()
    }

That makes it a bit hard to debug why we're getting back a 422 response. Are there tips and tricks for debugging this in a containerized production environment where it isn't easy to say "oh just add some debugging quick." All of our secrets match up. I just have no idea where it's getting an error.
It's not preview URL validation as that handles !response.ok so it must be in:
      const validationPayload = await result.json()

      response.setPreviewData({
        resourceVersion,
        plugin,
        ...validationPayload,
      })

But I have no idea why. Do I just copy/paste this code and temporarily fork the preview endpoint?

Try configuring on Drupal in settings.php reverse proxy, for ex.:
$settings['reverse_proxy'] = TRUE; $settings['reverse_proxy_addresses'] = ['<IP_ADDRESS>']; // Depending on network configuration $settings['reverse_proxy_trusted_headers'] = \Symfony\Component\HttpFoundation\Request::HEADER_X_FORWARDED_FOR | \Symfony\Component\HttpFoundation\Request::HEADER_X_FORWARDED_HOST | \Symfony\Component\HttpFoundation\Request::HEADER_X_FORWARDED_PORT | \Symfony\Component\HttpFoundation\Request::HEADER_X_FORWARDED_PROTO | \Symfony\Component\HttpFoundation\Request::HEADER_FORWARDED;

Yep. That's configured and not the issue. This issue is about finding ways to get better error messages returned inside of the client here.

Have this very same issue using OOTB NextJs for Drupal. I have no idea why I am getting a 422 or where to begin debugging this issue.

In Drupal, I get this log but don't know where it is originating:
Symfony\Component\HttpKernel\Exception\MethodNotAllowedHttpException: No route found for "GET https://drupalnextjs.ddev.site/next/preview-url": Method Not Allowed (Allow: POST) in Symfony\Component\HttpKernel\EventListener\RouterListener->onKernelRequest() (line 131 of /var/www/html/vendor/symfony/http-kernel/EventListener/RouterListener.php).

The route definition for this from the NextJS module defines it as POST only.


Fix: show Overlay permissions Dialog

Fix: show current dialog state on add callback to the DialogController

!merge


fix test bed download button

Time Estimate or Size
tiny
Problem
Somewhere along the line we broke the test-bed download button, it was losing the context and state was coming back undefined, arrow function in button onClick restores context

Could this also be fixed by putting this.downloadfile = this.downloadfile.bind(this) in the constructor?

Yes, in the constructor or right here in the prop.
I think I'll change it to the more explicit binding since that matches the rest of the component better! I'm just very used to working in functional components so jumped to this solution first.

seems fine, but why not echo the pre-existing pattern of ()=>{this.downloadFile();} ?

When I look at the constructor and the various onClick props I don't actually see a consistently applied pattern, we use a mix of arrow functions, binding in the constructor, and binding as methods are passed as props.
One of several places I think the test-bed could use some TLC.

I was looking up at the ~80 lines above the change and they all looked the same but I didn't look over the whole file


Table pagination is not working on the farm IPs

Description
A clear description of the bug is:
I added more than 25 IPs to that farm, but I couldn't list all of them using the rows per page, nor could I navigate through the pages, making the table only show you the first 10 IPs, and the rest aren't usable.
It also keeps collapsing the farm whenever I change the rows per page.
Must include any relevant identifiers, like:

Network: Testnet and devnet

Version: 2.3.0-rc3

Twin ID: 586

Farm ID: 365

Steps to reproduce

Navigate to the dashboard.
Connect your wallet.
If you don't have a farm, create one.
Add more than 20 IPs (range IP would help with that).
Try changing the rows per page to more or less IPs.
Try to navigate to the next page.

Logs/Alerts
Must include, if available:
Screenshots or screen records.
screen-capture (15).webm

Issue Update
Investigation and Solution:
Upon further investigation, it has been determined that the issue is not directly associated with the farm table. The farm table successfully lists the farms and dynamically updates the displayed farms based on the pagination selection. Therefore, the issue is unrelated to the functionality of the farm table.
However, a separate issue has been identified concerning the length of the IPs. Despite utilizing the IP range feature and adding 16 IPs, only 10 IPs are currently visible. This behavior is inconsistent with the expected functionality.
To address this issue, a thorough examination will be conducted to identify the root cause. Regular updates will be provided as the investigation progresses, ensuring that any findings and solutions are communicated promptly. Your patience is appreciated as we work to resolve this matter.

Issue Update
Investigation and Solution:
Upon closer examination, the issue was identified in the listing of IPs. While more than 10 IPs were retrieved from the chain, the v-data-table was only handling 10 items due to the pagination constraints of the IPs table. Additionally, it was observed that the pagination controls were intentionally hidden by overriding the table's bottom section through slot access.
To address these issues, the following steps will be taken:

The override of the table's bottom section, which hides the pagination controls, will be removed to restore their visibility.

The configuration of the v-data-table component will be revised and optimized to accommodate the complete list of IPs.

A listener will be introduced to monitor the page event, ensuring that the displayed items are dynamically updated as the user navigates through the paginated data.

Issue Update
Work done

Removed the override on the table's bottom section to restore visibility of pagination controls.
Revised and optimized the configuration of the v-data-table to handle the complete list of IPs.
Introduced a listener for the page event to ensure dynamic updates as users navigate through paginated data.

Screenshots/Video

Screencast from 03-07-2024 02:08:24 PM.webm

Devnet, 17fffef
The items per page are working correctly, but the pagination is still not working.
screen-capture (18).webm

Table sorting is not working.
screen-capture (19).webm

Verified, Devnet d965ce4.
The farm IPs now have a table that can list all the IPs, change items per page, and paginate through it.
TC2636 - Farm IPs table

I created a separate issue for the sorting.


How to initiate adVolume ?

Hi Guys,
in Videos-vast-vpaid plugin is there any possibility to initiate the adPlayerVolume and then to change it when the mouse is over/out?
I try it to do it with the videoJs player.muted(true) or player.volume(0) and it is quiet working , but it is allways initiated as "unmuted".
I found something like setAdvolume but I didnt yet get how to manipulate it .
Best
versions
Videojs-vast-vpaid v.1.0.0
Videojs 5.0.0 rc2
browsers
Chrome/48.0.2564.95
FF 44

Yeah, it is a bit unstable.
I tried this way
player.volume(0);
-- or
player.muted(trure);
player.play()

and it haven't worked.
Then I move player.play() up above volume/muted methods call and it started to work. Unfortunately today it doesn't work this way any more. Again, moving play() underneath makes it working.
All this with the plugin assigned to player. Didn't check without plugin.
However both methods should be considered working.

I also figured out that there is some changes with status of activation of flash on the Browser .
As i deactivated Flash it works like a charm but when i activate it doesn't.
Any explanationn ? May be the fact to switch between HTML5 and Flash ?
thanks


Update plan for the helm charts with respect to the app version changes

Appreciate you offering your time to enable this helm chart. Currently I see the app version is lacking behind. How are you planning on keeping up with the app changes here? Are you open for PR's or can you quickly update this to support the latest versions?

yea there's https://github.com/halkeye-helm-charts/vouch/pull/11 which just needs an chart.yaml change
I honestly havn't touched this chart in so long, I need to spend some time this weekend to give it a bunch of love and get it all updated. Vouch is one of those things that has just worked and I never really needed to put any thought into it.

I have recently updated the app version to the most recent version 0.36, it should track the reviews of this minor version. Please let me know if there is anything else we can help with.

@punkle you've done such nice work getting this all setup.  Thank you!

No probs.


Compare ark-circom wasm to snarkjs wasm

Benchmark the wasm generated by the rust in ark-circom as compared to the one generated by snarkjs, and see if there's significant improvements speed or memory wise in the former.

Is this issue still open to work?

where can i find that issue ?

https://github.com/zkemail/email-wallet/issues/75


No way to restore a deleted playlist?

A playlist was accidentally deleted ...upon checking the trash under site contents, I see that it does not retain this information or any file (not sure the playlist makes a file). What is the process to restore other than re-creating all settings in the playlist?

@kyleg11 -- That's correct in the version of the webpart that's publically available we delete the item not recycle it so there is no way to restore it.  In the future version it will put the playlist in the recycle bin, so you will be able to restore it.

Deleted playlists will now show up in your site's recycle bin


helm: Permit selection of datasources in UI

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

Example dashboards in helm chart now permit selection of datasources from the grafana UI.

/test


CI: Bump deadsnakes commit for free-threaded CI

Reference issue

What does this implement/fix?

This PR updates deadsnakes action commit to a newer version, this is done in order to test if some stalling that came from the action got fixed.
Additional information

Thanks @andfoy. This CI issue has resolved itself, and the bump didn't make a difference, so I'll close the PR.


Make upcoming-v4 the main branch.

[x] Rename upcoming-v4 to main.
[x] Make main the default git branch.
[x] Rename master to branch-3.x.

To update your local branches to the new names:
To rename upcoming-v4 to main:
git branch -m upcoming-v4 main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

To rename master to  3.x-branch:
git branch -m master 3.x-branch
git fetch origin
git branch -u origin/3.x-branch 3.x-branch
git remote set-head origin -a


local path repo verbose

The new local path code is working well, but it prints out a lot of lines like:
Archiving xxx from repo file://../../microservice/xxx
and
Repository from local path: file://../../microservice/xxx
This information is nice but not needed most of the time and sometimes obscures more important info. Can we hide them under the --debug flag?

Would that be pretty easy to do @qwangrepos ?

@technosophos I just checked the dependency commands and the debug flag is not currently implemented in the command structure. It is okay to modify the structure for "--debug"?


Ruff: add pyupgrade

Add support for https://pypi.org/project/pyupgrade/ in Ruff

This PR depends on https://github.com/DefectDojo/django-DefectDojo/pull/9823

This PR is ready to merge


[modify] gws memo massage export spec

Coverage decreased (-0.008%) to 79.085% when pulling 9818aa241c9833d2b497267df9323c41861d6823 on ShinjiTanimoto:export into 9d43f61d5651840f4b7c858787e4a5d09f810c8b on shirasagi:master.


Less prominent upgrade message

When an upgrade is needed, the user sees the following message:

Now that we're not auto-updating the client anymore, this message is really disturbing to some users, as we're constantly publishing patch updates.
Let's make it a one-liner instead:

UPDATE AVAILABLE Run npm install -g now@canary to install Now CLI 15.9.0-canary.21

It would also be great to document how to avoid this check completely. It can be really annoying if something like a proxy doesn't play nicely with the network request. The now CLI simply hangs in this case.


ginac: 1.8.3 -> 1.8.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ginac/versions.
meta.description for ginac is: "GiNaC is Not a CAS"
meta.homepage for ginac is: "https://www.ginac.de/"
meta.changelog for ginac is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8.4 with grep in /nix/store/k4ll8gbhjgy4441gnkz9kdxik7m1022r-ginac-1.8.4
found 1.8.4 in filename of file in /nix/store/k4ll8gbhjgy4441gnkz9kdxik7m1022r-ginac-1.8.4

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ginac

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k4ll8gbhjgy4441gnkz9kdxik7m1022r-ginac-1.8.4 \
  --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 ginac https://github.com/r-ryantm/nixpkgs/archive/fcfdc8b58e4f3ede82b63721ceec4b8d3d7ca853.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k4ll8gbhjgy4441gnkz9kdxik7m1022r-ginac-1.8.4
ls -la /nix/store/k4ll8gbhjgy4441gnkz9kdxik7m1022r-ginac-1.8.4/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

1 package built:

ginac

Maintainer pings
cc @lovek323 for testing.

Result of nixpkgs-review pr 192042 run on x86_64-linux 1

1 package built:

ginac


Add createClient

kind of does https://github.com/PrismarineJS/bedrock-protocol/issues/35 (still need createServer)

Can you add it to docs ?

API.md


spring.cloud.vault.kv.profiles does not override the spring active profiles for use in the key-value backend

Describe the bug
We need to disable spring profiles and override it by spring.cloud.vault.kv.profiles. because it is causing 403 as vault policy does not allow access to spring profiles.
Current spring profiles: api-docs, dev.
Vault policy:
path "secret_kv/data/applicationA/sandbox" {
capabilities = ["list", "read"]
}
Spring yml file:
spring:
config:
import:
- vault://
cloud:
vault:
ssl:
trust-store: ***************
trust-store-type: PEM
host: *************
authentication: APPROLE
port: 8443
application-name: applicationA
kv:
backend: secret_kv
default-context: applicationA
profiles: sandbox
app-role:
role-id: *************
secret-id: ***********
app-role-path: **********
fail-fast: true
Spring log:
15:42:28.767 [restartedMain] ERROR org.springframework.boot.SpringApplication - Application run failed
org.springframework.vault.VaultException: Status 403 Forbidden [secret_kv/data/applicationA/api-docs]: 1 error occurred:
* permission denied
; nested exception is org.springframework.web.client.HttpClientErrorException$Forbidden: 403 Forbidden: [{"errors":["1 error occurred:\n\t* permission denied\n\n"]}
]
It is trying to connect to api-docs which is spring profile and fails due to policy on vault side.
Moreover, seems like the spring.cloud.vault.kv.profiles = sandbox is completely ignored. When I fix the permission to:
path "secret_kv/data/applicationA/sandbox" {
capabilities = ["list", "read"]
}
path "secret_kv/data/applicationA/*" {
capabilities = ["list", "read"]
}
path "secret_kv/data/applicationA" {
capabilities = ["list", "read"]
}
Spring log :
Requesting secrets from Vault at secret_kv/applicationA/api-docs using ROTATE
Vault location [secret_kv/applicationA/api-docs] not resolvable: Not found
Requesting secrets from Vault at secret_kv/applicationA/dev using ROTATE
Requesting secrets from Vault at secret_kv/applicationA using ROTATE
Vault location [secret_kv/applicationA] not resolvable: Not found
It retrieved passwords from secret_kv/applicationA/dev and ignored sandbox.
.
Not sure if this fix works
https://github.com/spring-cloud/spring-cloud-vault/commit/9bec609b8d7e6a3e86a777ff54adeb74de9d8e25
" profiles overrides the active profiles for use in the key-value backend"
Our version:
Manifest-Version: 1.0
Implementation-Title: Spring Cloud Vault Configuration Integration
Implementation-Version: 3.0.2
Build-Jdk-Spec: 1.8
Created-By: Maven Jar Plugin 3.2.0
Implementation-Vendor: Pivotal Software, Inc.

VaultConfigDataLocationResolver seems overrides the profiles with spring active profiles
private static VaultKeyValueBackendProperties getKeyValueProperties(ConfigDataLocationResolverContext context, Profiles profiles) {
    VaultKeyValueBackendProperties kvProperties = (VaultKeyValueBackendProperties)context.getBinder().bindOrCreate("spring.cloud.vault.kv", VaultKeyValueBackendProperties.class);
    Binder binder = context.getBinder();
    kvProperties.setApplicationName((String)binder.bind("spring.cloud.vault.kv.application-name", String.class).orElseGet(() -> {
        return (String)binder.bind("spring.cloud.vault.application-name", String.class).orElseGet(() -> {
            return (String)binder.bind("spring.application.name", String.class).orElse("");
        });
    }));
    kvProperties.setProfiles(profiles.getActive());  <<< Here
    return kvProperties;
}


tmkms v0.7.0 hang

no error, just hang in tmkms v0.7.0
After tmkms stops first, I get the following error from gaiad:
E[2020-01-01|06:07:02.792] SignerClient::SignVote                       module=privval err="read tcp <IP_ADDRESS>:26666-><IP_ADDRESS>:47828: i/o timeout: endpoint read timed out"
E[2020-01-01|06:07:02.792] Error signing vote                           module=consensus height=248863 round=0 vote="Vote{48:68F5BBEACEF1 248863/00/1(Prevote) DA64AD3E2B74<PHONE_NUMBER>00 @ 2019-12-31T21:06:59.79003623Z}" err="read tcp <IP_ADDRESS>:26666-><IP_ADDRESS>:47828: i/o timeout: endpoint read timed out"
E[2020-01-01|06:07:05.792] SignerListener: Ping timeout                 module=privval
E[2020-01-01|06:07:08.793] SignerClient::GetPubKey                      module=privval err="endpoint connection timed out"
E[2020-01-01|06:07:08.797] CONSENSUS FAILURE!!!                         module=consensus err="runtime error: invalid memory address or nil pointer dereference" stack="goroutine 883 [running]:\nruntime/debug.Stack(0xc000c8f358, 0x1047dc0, 0x1f50a00)\n\t/usr/local/go/src/runtime/debug/stack.go:24 +0x9d\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).receiveRoutine.func2(0xc002aa6700, 0x13aa1e0)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:602 +0x57\npanic(0x1047dc0, 0x1f50a00)\n\t/usr/local/go/src/runtime/panic.go:679 +0x1b2\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).signAddVote(0xc002aa6700, 0x3cc02, 0xc00f754b20, 0x20, 0x20, 0x1, 0xc00f754b80, 0x20, 0x20, 0xb)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:1745 +0x585\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).enterPrecommit(0xc002aa6700, 0x3cc1f, 0x0)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:1155 +0x12ec\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).addVote(0xc002aa6700, 0xc004c2adc0, 0xc0020a20c0, 0x28, 0xc000c8faa8, 0x4466f7, 0xc002aa6798)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:1663 +0xdfa\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).tryAddVote(0xc002aa6700, 0xc004c2adc0, 0xc0020a20c0, 0x28, 0xcc2f5edad59daf76, 0x109, 0x101)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:1527 +0x59\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).handleMsg(0xc002aa6700, 0x15612c0, 0xc00fda7e48, 0xc0020a20c0, 0x28)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:691 +0x252\ngithub.com/tendermint/tendermint/consensus.(*ConsensusState).receiveRoutine(0xc002aa6700, 0x0)\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:633 +0x6eb\ncreated by github.com/tendermint/tendermint/consensus.(*ConsensusState).OnStart\n\t/home/jun/go/pkg/mod/github.com/tendermint/tendermint@v0.32.7/consensus/state.go:334 +0x13a\n"

This looks potentially related to #385
Can you post the KMS-side logs, preferably with debug logging enabled?
Which signing backend are you using?

This looks potentially related to #385
Can you post the KMS-side logs, preferably with debug logging enabled?
nothing logs in kms.
I'll post if there's a problem after debug logging enabled.

Which signing backend are you using?
yubihsm

nothing logs in kms.

Are you running it with tmkms start -v to enable verbose logging?
What do you see immediately before it stops?
Can you perhaps try running it under strace? What OS are you using?

nothing logs in kms.

Are you running it with tmkms start -v to enable verbose logging?

tmkms did not start with -v when an error occurred.
now restarted with -v.
next time I'm having trouble with tmkms, I'll post  log.

What do you see immediately before it stops?

Before stopping, there is right sign log. (signed PreVote & PreCommit)

Can you perhaps try running it under strace? What OS are you using?

OS : CentOS7

OS : CentOS7

For what it's worth, we also use CentOS 7 and don't experience this issue, so it doesn't seem to be specific to that.

OS : CentOS7

For what it's worth, we also use CentOS 7 and don't experience this issue, so it doesn't seem to be specific to that.

more info..
i using 3 chain per 1 hsm. (cosmos, terra, kava)

Since the v0.7.1 update, it doesn't seem to be a problem anymore.
Even if  prevote occurs continuously, it works wall by reconnecting.
thank you.


scribus scripts cannot find Tkinter

Issue description
The python scripts included with scribus (at least the calendar wizard) fail with "cannot find Tkinter".
Steps to reproduce
install scribus. run it and select Scripts > Calendar Wizard.
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...) 16.09pre85931.125ffff
Nix version: (run nix-env --version) nix-env (Nix) 1.11.2
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
"16.09pre85931.125ffff"

Is this another python vs. pythonFull issue?

just adding python.modules.tkinter should fix this.


[FIX] TooManyRecords exception is defined in NestedAttributes not MasAssignmentSecurity

When this cdoe path is hit, I am getting:
uninitialized constant ActiveRecord::MassAssignmentSecurity::NestedAttributes::TooManyRecords
This is because TooManyRecords is actually defined in
ActiveRecord::MassAssignmentSecurity::TooManyRecords
Now, I'm quite there is a more clever way to get to it on the class hierarchy, without spelling out the absolute hierarchy, but I haven't found it in a quick search. :(

Hi @AvnerCohen, the fix for that issue has already been made and merged here

How weird is it to have the other PR taken, nearly a year after mine this one was sent and was totally ignored for over a year..
/cc @rafaelfranca
As I would like to understand for the future how to better contribute, would love to understand what was the reason for that.

Sorry about that. The only explanation I have is that I didn't saw your PR (it is true) and there is nothing you did wrongly. It was my fault, really sorry.


add TopbarWallet

Removed unused data from useNavbar - that caused problems with old components so cleaned non existing data occourences - these components will get removed anyway


MM-25832 Prevent permalink view from loading posts multiple times

Summary
The Permalink view was using getDerivedStateFromProps lifecycle which wasn't needed as well as having two other issues:

Channel name was not present at times as the title was not being update with prop changes
Posts were being loaded multiple times (~4-5 times) when the view loaded as well as when retrying.

On this PR we removed the unnecessary lifecycle as well as we load post once as needed.
Test 1: Open a permalink from the same team.
Test 2: Open a permalink from a different team that you have loaded
Test 3: Open a permalink from a different team that you have not loaded
Test 4: Reset cache -> go to the channel with a permalink -> Disconnect from internet -> Open permalink -> Connect to internet -> Tap Retry
Ticket Link
Found while trying to reproduce https://mattermost.atlassian.net/browse/MM-25832

@mattermod Build Apps for PR is getting stuck

@metanerd @jaydeland @khos2ow any ideas ?

@josephbaylon I guess in the meantime you could trigger the build manually

@josephbaylon I'll  take a look.. just out of curiosity, what are you seeing a blank screen while it loads ?

@enahum I'm attaching screenshots for PR and old versions after I deleted cache and tapped on the same permalink

PR
old

@enahum The ones above is for iOS. For android, tapping on any permalink causes a crash (see below)

@enahum  There was an issue with the latest mattermod release and in the meantime it's already been rolled back. You can give it another try now.

Building app in separate branch.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9557
https://circleci.com/gh/mattermost/mattermost-mobile/9556

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/9557
https://circleci.com/gh/mattermost/mattermost-mobile/9556

Artifact links:
https://9557-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://9556-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

Artifact links:
https://9557-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://9556-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

Failed to delete ref. @mattermost/core-build-engineers have been notified. Error:
GET https://api.github.com/repos/mattermost/mattermost-mobile/commits/refs/heads/build-pr-4696/status: 404 Ref not found []


Picovoice Console Issue:

How I can get data like name, email and phone number from a intent?

@Christianhg10 this is a duplicate of #305. In the future please do not open duplicate issues.


Date Input

Summary of the changes
Date Input build.
Includes:

Unit, E2E, Visual Regression and a11y automated
Web component and react stories
Updates to github action

Related issue
#334
Checklist

[x] I have added relevant unit and visual regression tests.
[x] I have manually accessibility tested any changes, if relevant.
[x] I have ensured any changes match the Figma component library.

I found a bit of a weird bug where the date input focuses if you click anywhere alongside it. Other than that, it looks really good 😊
https://github.com/mi6/ic-ui-kit/assets/111882034/da7a8338-d03a-4959-8d15-e313bd4fddc6

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jd239:x: github-actions[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

Components should now follow the Stencil style guide

Components should now follow the Stencil style guide

Done :)

I found a bit of a weird bug where the date input focuses if you click anywhere alongside it. Other than that, it looks really good blush
Screen.Recording.2023-06-19.at.13.38.18.mov

Sorry if I've missed discussion of this issue, but I've noticed this on other components in our design system. One that caught me out was the ic-switch component which I accidentally triggered doing the same thing in the video.

In the story 'with value, after changing the value and logging, the mm/yy values change but not the dd value, possibly just a SB story nuance?

It was a problem with the component not update the relevant states.

Was initially going to set the :host to inline-flex to fix this but then thought this will need to be added to all form controls. The issue with that is, switching from block to inline could break a lot applications which do not have the input wrapped in a blocked div so left it out.
Will create a ticket to review for the next major version.

I only seem to be able to paste a date into the input when I right click and select paste, it doesn't do anything when I use a keyboard shortcut. Is this meant to be the case or should I be able to use both methods?

Enhancement to paste functionality:

Before it was possible to paste more than two characters into the day input. This has now been restricted to only two.
It is now possible to paste individual 'numeric' values into the inputs. For example, it is now possible to copy and paste '12' into the month input.
Validation has been set to prevent invalid values into relevant inputs for example, it is not possible to paste '13' into the month input. A validation message below will appear which says, 'Please paste a valid day/month'. This does not affect year as any number can be placed in there
Moving to the next or previous input while the paste validation has appeared will remove the message.

Couple of small potential improvements:

The error message when attempting to paste an invalid value into a single filed is great. For example, pasting '13' into a month field. However, if I try to paste '123' in the field, then no message is displayed and the paste fails. It would be great to provide the same validation message and treat this as an invalid input in the same way.
When pasting a full date string '25/12/2017' we can use '/' and '-' as the separator keys. It would be great to also include the '.' symbol as a separtor allowing dates formatted '25.12.2017' to be pasted in the same way.
It would be great to allow the selection of the whole date when using the 'cmd+a' key press, and then allow the deletion of the whole date with one press of the delete key.
On the month element, should we allow a user to enter a letter corresponding to a month? For example, entering 'S' would result in an '09' entered into the month element. Entering 'J' would put the first month beginning with J - January - 01. Entering 'JU' would put the first month beginning with JU - June - 06. Entering JUL would then allow July - 07.
The allowable character entries for the above month text behaviour would be as follows:
-- JA
--F
---MAR
--AP
--MAY
--JUN
--JUL
--AU
--S
--O
--N
--D

Couple of small potential improvements:

The error message when attempting to paste an invalid value into a single filed is great. For example, pasting '13' into a month field. However, if I try to paste '123' in the field, then no message is displayed and the paste fails. It would be great to provide the same validation message and treat this as an invalid input in the same way.
When pasting a full date string '25/12/2017' we can use '/' and '-' as the separator keys. It would be great to also include the '.' symbol as a separtor allowing dates formatted '25.12.2017' to be pasted in the same way.

These have been added.

It would be great to allow the selection of the whole date when using the 'cmd+a' key press, and then allow the deletion of the whole date with one press of the delete key.

Based on the three inputs which we use to generate the day/month/year, this does not seem possible as all three inputs would need to be focus and selected at the same time.

On the month element, should we allow a user to enter a letter corresponding to a month? For example, entering 'S' would result in an '09' entered into the month element. Entering 'J' would put the first month beginning with J - January - 01. Entering 'JU' would put the first month beginning with JU - June - 06. Entering JUL would then allow July - 07.
The allowable character entries for the above month text behaviour would be as follows:
-- JA
--F
---MAR
--AP
--MAY
--JUN
--JUL
--AU
--S
--O
--N
--D

This will be added to a new ticket rather than added to this one. The date input specifies 2 characters (which would be assumed as numbers instead of text) for the MM. Having the ability to type text in seems like a nice to have rather than a must have.


Topic 2

Good work but, you must not include .classpath, eclipse configuration files or version control files.
Please fix and I will review.


[BUG] can't write BigTiff file using ImageOutput

Describe the bug
Hi ,
I tried to write a bigtiff file containing one floating channel (more than 4Go) using openimage 2.3.0.
I both tried using tiling and scanlines writing, but It doesn't work (stop at 3,99Go).
However, when I try to directly write the tiff using LibTiff and opening Tiff file using :
tif = TIFFOpen("file.tif", "w8");
It does work (Tiff header has magical BigTiff value 43)
The issue may come from the fact that openimage tiffoutput.cpp doesn't  call TIFFOpen with option "w8" enabled : seems missing the "8" digit which enables BigTiff option.
To Reproduce
Doesn't work
unsigned int sizeX = 40000;
unsigned int sizeY = 40000;
std::vector<float> pixels;
pixels.resize(sizeX);
const QString fileName = QString("temp/resScanline.tif");
std::unique_ptr<ImageOutput> out = ImageOutput::create(fileName.toStdString());

// Use Create mode for the first level.
ImageOutput::OpenMode appendmode = ImageOutput::Create;
ImageSpec spec(sizeX, sizeY, 1, TypeDesc::FLOAT);

spec.attribute("compression", "none");

if(!out->open(fileName.toStdString(), spec, appendmode)) {

	exit(0);
}
for(size_t y = 0; y < sizeY; ++y) {
	for(size_t offset = 0; offset < sizeX; ++offset) {
		pixels[offset] = 0;
	}

	if(!out->write_scanline(y, 0, TypeDesc::TypeFloat, pixels.data())) {

		exit(0);
	}
}
out->close();

Works
const size_t width = 40000;
const size_t height = 40000;
const double pixels_per_cm = 1;

float* data = new float[width * sizeof(float)];

int x, y;

for(x = 0; x < width; x++) {
	data[x] = 0;
}

TIFF* tif= TIFFOpen("temp/scanline.tif", "w8");
std::cout << "write";
TIFFSetField(tif, TIFFTAG_IMAGEWIDTH, width);
TIFFSetField(tif, TIFFTAG_IMAGELENGTH, height);
TIFFSetField(tif, TIFFTAG_BITSPERSAMPLE, 32);
TIFFSetField(tif, TIFFTAG_COMPRESSION, COMPRESSION_LZW);
TIFFSetField(tif, TIFFTAG_PHOTOMETRIC, PHOTOMETRIC_MINISBLACK);
// TIFFSetField(tif, TIFFTAG_FILLORDER, FILLORDER_MSB2LSB);
TIFFSetField(tif, TIFFTAG_SAMPLEFORMAT, SAMPLEFORMAT_IEEEFP);
TIFFSetField(tif, TIFFTAG_ORIENTATION, ORIENTATION_TOPLEFT);
TIFFSetField(tif, TIFFTAG_SAMPLESPERPIXEL, 1);
TIFFSetField(tif, TIFFTAG_ROWSPERSTRIP, 8);
TIFFSetField(tif, TIFFTAG_RESOLUTIONUNIT, RESUNIT_CENTIMETER);
TIFFSetField(tif, TIFFTAG_XRESOLUTION, pixels_per_cm);
TIFFSetField(tif, TIFFTAG_YRESOLUTION, pixels_per_cm);
TIFFSetField(tif, TIFFTAG_PLANARCONFIG, PLANARCONFIG_CONTIG);

tdata_t l_buffer;
unsigned long l_buffer_size = width * sizeof(float); // LINE buffer for float

l_buffer = _TIFFmalloc(l_buffer_size);

for(int row = 0; row < height; row++) {
	// std::cout << row << height;
	memcpy(l_buffer, data, l_buffer_size);

	int ret = TIFFWriteScanline(tif, l_buffer, row, 0);
	if(ret == -1) {
		TIFFClose(tif);
		return -1;
	}
}

_TIFFfree(l_buffer);

TIFFClose(tif);
delete[] data;
return 0;
}

Expected behavior
Should work as when using directly libtiff
Evidence
Do you have error messages? (please quote exactly) Screenshots? Example
command lines or scripts that reliably reproduce the error? If it only
happens with certain image files, can you attach the smallest image you
can make that reproduces the problem?
Platform information:

OIIO branch/version: 2.3.0
OS: Windows 10
C++ compiler: MSVC
Any non-default build flags when you build OIIO:

IF YOU ALREADY HAVE A CODE FIX: There is no need to file a separate issue,
please just go straight to making a pull request.

You're quite right. I've proposed a fix in #3158. It's not perfect, there are some trade-offs, as discussed in the PR. But I think it's the right approach.

Hi Larry , thank you for this answer. i'll test your fix.


Microsoft.Compute/virtualMachines/runCommand/action permission missing

Hello,
Permission Microsoft.Compute/virtualMachines/runCommand/action, which is mentioned in the article, under the Limiting access to Run Command part, is no longer available.
There are only available permissions read, write, and delete.
Thanks,

@wladinho31 - Thank you for your question.
Can you please share with us the URL of the document that you are having issues with?
Please note that we scope issues on this repo to feedback related to the docs. If you have a product question, please post it on Microsoft QnA or Stack Overflow. Thank you for your understanding.

Hello,
This is a link for official docs. https://docs.microsoft.com/en-us/azure/virtual-machines/linux/run-command#limiting-access-to-run-command
A mentioned permission Microsoft.Compute/virtualMachines/runCommand/action is not visible in portal.
If I try to create a custom role, as explained on this page, but there are only read, write, and delete permissions.

After troubleshooting this issue, I found that permission is visible in the portal as Other: Run Command on Virtual Machine.

It would be nice to clarify that inconsistency because there are customers that prefer to use Azure Portal instead of ARM templates for many management tasks.
Regards,

@wladinho31 - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Refactor Gradle integration tests

This PR serves two purposes:

Move all Gradle integration tests to use the Gradle testkit API. There is no more unit test depending on Gradle internal API itself.

One benefit is less maintenance over the future Gradle upgrade
Another benefit is that after #3356, IntelliJ is not really working with the new intTest source set and we cannot launch those tests in IDE. IDEA-218815 is an issue that's applicable to a lot of people in the recent versions. So I am planning to remove the intTest and merge it back to test.

Set up testing infrastructure for #3327 and I did verify that the RFC in #3334 is actually not working :)

Note
While this mostly maintains the test coverage, I did reduce the following test cases:

Test for com.android.test is removed. Since there are many other plugin's like com.android.dynamic-feature, com.android.lint and com.android.reporting, it is not necessary to test all different android plugins. As long as the base plugin com.android.build.gradle.api.AndroidBasePlugin is applied, we do not need to worry too much about other subplugins.
Removed one flavor dimension sally from name. This does not provide too much value in my opinion while contributes significantly the lines of code in tests.

LGTM.
Just one note, should we merge #3453 before this?

Good call. Now we have one remaining non-testkit gradle unit test


Add new command to change configuration environment

Context
Implement a VSCode command to change configuration

For reviewer: I'd appreciate feedback specifically on the TS logic since I am relatively new to TS.

amazing!

I think my wish of having a status bar you can select is pretty straight forward to do given you have everything you need here:
https://github.com/microsoft/vscode-extension-samples/tree/main/statusbar-sample
and here is a short ChatGPT example:
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
    // Create a status bar item
    const statusBarItem = vscode.window.createStatusBarItem(vscode.StatusBarAlignment.Right, 100);
    statusBarItem.text = 'Select Option';
    statusBarItem.command = 'extension.showComboBox';
    statusBarItem.show();
    context.subscriptions.push(statusBarItem);

    // Register the command
    const disposable = vscode.commands.registerCommand('extension.showComboBox', async () => {
        const options = ['Option 1', 'Option 2', 'Option 3'];
        const selectedOption = await vscode.window.showQuickPick(options, {
            placeHolder: 'Select an option',
        });
        if (selectedOption) {
            vscode.window.showInformationMessage(`You selected: ${selectedOption}`);
            statusBarItem.text = selectedOption;
        }
    });
    context.subscriptions.push(disposable);
}

export function deactivate() {}

@noklam In terms of code, all good, but I tested it locally with debugger and on selecting command I am getting two options base and local, by selecting local I am getting error Error: Pending response rejected since connection got disposed and server stoped. same with base.

@jitu5 I see, I test it in this way:
1. make build
2. Install the .vsix (right click on it in VSCode)
3. Open a spaceflights project to test

Debug mode doesn't work because every time a configuration change, the server is restarted (maybe there is a lighter way to do this?). You will experience the same if you do kedro: restart server. For now I reuse the TS function startServer. Maybe there is a way to register command on the server side, which may be better.

@jitu5 Have you tried to use the build? does it work?

@jitu5 Have you tried to use the build? does it work?

Not yet I will try this today with below steps.

make build
Install the .vsix (right click on it in VSCode)
Open a spaceflights project to test

@jitu5 Have you tried to use the build? does it work?

@noklam I followed your steps and now its working without debugger also select environment from command. 👍🏼

Thank you, I merge this now!


Removing a redundant background-color declaration.

@mdo :) CI build is failing on a property-order issue. The out-of-order prop looks redundant.

Doh! Thanks :).

np!


PDF: Error: Property ID "foreword" (found on "fo:block") previously used

Source issue: https://github.com/metanorma/metanorma/issues/357#issuecomment-1986258804
From .pdf.err:
Property ID "foreword" (found on "fo:block") previously used; ID values must be unique within a document! (See position 65:41)
org.apache.fop.fo.ValidationException: Property ID "foreword" (found on "fo:block") previously used; ID values must be unique within a document! (See position 65:41)
javax.xml.transform.TransformerException: org.apache.fop.fo.ValidationException: Property ID "foreword" (found on "fo:block") previously used; ID values must be unique within a document! (See position 65:41)

<foreword id="foreword" obligation="informative" displayorder="2">
	<title>Foreword</title>
	...
	<review id="_e761ef6b-8203-d5a1-e38a-57a1cb66e1d1" reviewer="ISO" date="2017-01-01T00:00:00Z" type="todo" from="foreword" to="foreword">

Note: foreword id="foreword", but ISO XSLT means that foreword in from="foreword" to="foreword" is the element name (<foreword).

Fixed.


Opening up structs/traits/fns to make imdl useable as a library

Hey Casey!
I'm writing something torrent-related, and it'd be great to use intermodal in tests to verify my implementation. However, currently everything is pub(crate), which makes shelling out to imdl the only option. Would you be open to making things just pub?
I understand that downstream code directly depending on innards of imdl would invite compatibility complaints. However, maybe making it clear in the docs that you will break compatibility in point releases and using imdl as a library is not something you're going to support might be enough to minimise the extra support burden?

i would also like to do the same actually ^^ +1 👍

Same here. I would like to use it to create temporary torrents for E2E testing for this project.


Improved tpcdsenv.sh to work non-interactive

Allow user to import bin/tpcdsspark.sh and use functions manually. Example:
source bin/tpcdsspark.sh
set_env
download_and_build
cleanup

LGTM


viz: folium_mapping.ipynb AttributeError: 'Map' object has no attribute '_build_map'

Not sure if this broke with the 2-3 port or ?
Cell [11] is the culprit

huh, yes I see. I think this is actually a change in Folium since when I built the tool.
I was just getting the raw HTML/JS and then displaying it in an IPython iframe. Now, I guess they've removed _build_map() from Folium objects, meaning you can't embed them the way I was doing.
Will look into fixes.

As noted in the pull, I've submitted a fix to @pdoongarwal that, if merged, will update #759 with the correct folium code.

Hmm... now I can't figure out why my branch is incompatible. Should I submit both of our commits separately?

@ljwolf Yes, that seems the only way.

closed by #764


When I udpate my npm package,turbo still uses the old node_modules/.cache

What version of Turborepo are you using?
1.3.1
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Mac
Describe the Bug
When I udpate my npm package,turbo still uses the old node_modules/.cache.
I want to debug/update a npm package,but everytime I update package, I find I need to delete the node_modules/.cache first.
Expected Behavior
I hope when I update my npm package,turbo can update itself.
To Reproduce

use turbo to build
update a npm package version
you find it will still use the old version util you delete the node_modules/.cache

My tubro.json
{
  "pipeline": {
    "build": {
      "dependsOn": [
        "^build"
      ],
      "outputs": [
        "dist/**",
        ".next/**"
      ]
    },
    "build:dev": {
      "dependsOn": []
    },
    "lint": {
      "outputs": []
    },
    "dev": {
      "cache": false
    }
  }
}

I also want to know, why I run turbo run dev --parallel --no-cache, it can't help before I delete  node_moduels/.cache

What you're reporting is likely by design; we do not include any information about the contents of node_modules inside of our cache calculation. If you run a build after mutating a package inside of node_modules Turborepo will not know that the cache needs to be purged. Further, if you install a package without updating package.json that will also not trigger invalidation. However, if you do update package.json it will do what you want:
~/repos/turbo-vercel-repro (main *)$ git status
On branch main
Your branch is up to date with 'origin/main'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   apps/api/package.json
	modified:   yarn.lock

no changes added to commit (use "git add" and/or "git commit -a")
~/repos/turbo-vercel-repro (main *)$ yarn run build
yarn run v1.22.19
$ turbo run build
• Packages in scope: @acme/api, @acme/utils
• Running build in 2 packages
@acme/utils:build: cache hit, replaying output 0a51811ae984c6fb
@acme/utils:build: warning package.json: No license field
@acme/utils:build: $ tsup src/index.ts --format esm,cjs --dts
@acme/utils:build: CLI Building entry: src/index.ts
@acme/utils:build: CLI Using tsconfig: tsconfig.json
@acme/utils:build: CLI tsup v6.1.2
@acme/utils:build: CLI Target: node14
@acme/utils:build: ESM Build start
@acme/utils:build: CJS Build start
@acme/utils:build: ESM ⚡ Build success in 248ms
@acme/utils:build: CJS ⚡ Build success in 23ms
@acme/utils:build: ESM dist/index.mjs 242.00 B
@acme/utils:build: CJS dist/index.js 1.18 KB
@acme/utils:build: DTS Build start
@acme/utils:build: DTS ⚡ Build success in 383ms
@acme/utils:build: DTS dist/index.d.ts 67.00 B
@acme/api:build: cache miss, executing e4a56c6b12ae0aba
@acme/api:build: $ rimraf build/ && tsc -b

 Tasks:    2 successful, 2 total
Cached:    1 cached, 2 total
  Time:    1.005s

✨  Done in 1.18s.

You can see that adding a new dependency to the api workspace successfully triggered a cache invalidation of the api workspace.

turbo run dev --parallel --no-cache tells turbo not to write a cache, it doesn't tell turbo to run every task node. You're looking for --force.
https://turborepo.org/docs/core-concepts/caching#force-overwrite-cache

As a workaround for your debugging workflow, I suspect that you can remove node_modules from your .gitignore which will cause turbo to consider node_modules as part of its cache invalidation.
I'm going to close this as the behavior you're reporting is as expected, and not a bug.


Error not able to visualize results

Hi,
I am getting the following error when I try to run the demo and visualize:
Instructions for updating:
Use tf.cast instead.
train_dir experiments\All\dropout_0.5\epochs_1\lr_0.001\residual\depth_2\linear_size1024\batch_size_64\no_procrustes\maxnorm\batch_normalization\use_stacked_hourglass\predict_17
Could not find checkpoint. Aborting.
Traceback (most recent call last):
File "src/predict_3dpose.py", line 537, in
tf.app.run()
File "C:\Users\k1749021\AppData\Local\Continuum\anaconda3\envs\Lift_Deep1\lib\site-packages\tensorflow\python\platform\app.py", line 125, in run
_sys.exit(main(argv))
File "src/predict_3dpose.py", line 532, in main
sample()
File "src/predict_3dpose.py", line 424, in sample
model = create_model(sess, actions, batch_size)
File "src/predict_3dpose.py", line 137, in create_model
raise( ValueError, "Checkpoint {0} does not seem to exist".format( ckpt.model_checkpoint_path ) )
AttributeError: 'NoneType' object has no attribute 'model_checkpoint_path'
The details requried are as below

Your operating system-Windows 10
Your tensorflow version-1,0
Your python version-3.7
The stack trace of the error that you see

Did you find the solution?
I am facing this too
Please guide me
Thank you in advance!


Automated cherry pick of #9785: guest_deploy_task: log other deploy details with i18n

Cherry pick of #9785 on release/3.5.
#9785: guest_deploy_task: log other deploy details with i18n

Codecov Report

Merging #9788 (7bf0734) into release/3.5 (6a15e02) will increase coverage by 11.92%.
The diff coverage is 1.85%.

@@               Coverage Diff                @@
##           release/3.5    #9788       +/-   ##
================================================
+ Coverage         5.98%   17.90%   +11.92%
================================================
  Files              796       85      -711
  Lines           125693     8923   -116770
================================================
- Hits              7522     1598     -5924
+ Misses          117504     7153   -110351
+ Partials           667      172      -495

Flag
Coverage Δ

aFlag
17.90% <1.85%> (+11.92%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

.../monitor/alerting/conditions/suggestrulereducer.go
0.00% <0.00%> (ø)

pkg/monitor/alerting/conditions/suggestquery.go
2.00% <2.00%> (ø)

pkg/util/seclib2/seclib.go
80.39% <0.00%> (-5.66%)
:arrow_down:

pkg/util/ansiblev2/session.go
0.00% <0.00%> (ø)

pkg/hostman/hostinfo/hostpinger.go

pkg/compute/models/snapshotpolicydisks.go

pkg/compute/models/natgateways.go

pkg/multicloud/aws/zone.go

pkg/multicloud/esxi/vdisk.go

... and 590 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3df9bb5...7bf0734. Read the comment docs.

Codecov Report

Merging #9788 (7bf0734) into release/3.5 (6a15e02) will increase coverage by 11.92%.
The diff coverage is 1.85%.

@@               Coverage Diff                @@
##           release/3.5    #9788       +/-   ##
================================================
+ Coverage         5.98%   17.90%   +11.92%
================================================
  Files              796       85      -711
  Lines           125693     8923   -116770
================================================
- Hits              7522     1598     -5924
+ Misses          117504     7153   -110351
+ Partials           667      172      -495

Flag
Coverage Δ

aFlag
17.90% <1.85%> (+11.92%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

.../monitor/alerting/conditions/suggestrulereducer.go
0.00% <0.00%> (ø)

pkg/monitor/alerting/conditions/suggestquery.go
2.00% <2.00%> (ø)

pkg/util/seclib2/seclib.go
80.39% <0.00%> (-5.66%)
:arrow_down:

pkg/util/ansiblev2/session.go
0.00% <0.00%> (ø)

pkg/hostman/hostinfo/hostpinger.go

pkg/compute/models/snapshotpolicydisks.go

pkg/compute/models/natgateways.go

pkg/multicloud/aws/zone.go

pkg/multicloud/esxi/vdisk.go

... and 590 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3df9bb5...7bf0734. Read the comment docs.


fix key error xgbregressor

Overview
Closes #3665
Description of the changes proposed in this pull request:

Fix error in deserialization of xgboost regression model

Checklist

[x] All pre-commit checks pass.
[x] Unit tests added (if fixing a bug or adding a new feature)

@frizfealer please have a look here.
I believe we could look at the loaded dictionary with the model parameters and figure out if it is a dart model and then only implement the custom loading process (basically the lines I added in this PR) for this model.


Refactor batch data access into own class or function

At the moment, we have mixed up Data fetching with rendering the data. Therefore, we can't reuse the data fetching logic and we duplicate this logic.
In addition to that, we are not able to test this functionality in the future.
So far, we have the same logic in application form and batch details.
The code to access the Firestore database should be refactored in a separate class/function and this new function should be used instead.

If no one is working on this at the moment, then I'd like to take over this :)

There is already something done here. :)
https://github.com/DigitalProductschool/website/tree/refactorBatchDetails

@danielbedo Can we merge the changes in staging at least?

I think this was not merged :) As i am making some changes in the batch details i will merge it now :)


Resolved: GO Portal is back up in 406ab09.


black formatting stop working

Environment data

VS Code version: 1.51.0 (user setup)
Extension version (available under the Extensions sidebar): v2020.11.311084263
OS and version: Windows_NT x64 10.0.19042
Python version (& distribution if applicable, e.g. Anaconda): 3.8, conda
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions:

black: black20.8b1

Value of the python.languageServer setting: pylance v2020.11.0

Expected behaviour
I have a file which contains line too long errors, black should format the file when I issue the command 'Format Document' from command panel.
Black works fine yesterday, I don't know why it stop working today. And I have applied an insider update of vscode this morning, don't know if it's related.
Actual behaviour
black is called but nothing changes.
problems still exists.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]
Logs
this is from Python output:
**_> ~/miniconda3/envs/omega/bin/python ~/.vscode-server/extensions/ms-python.python-2020.11.311084263-dev/pythonFiles/pyvsc-run-isolated.py black --diff --quiet /mnt/c/workspace/omega/omega/fetcher/abstract_quotes_fetcher.py_**
cwd: /mnt/c/workspace/omega
> ~/miniconda3/envs/omega/bin/python ~/.vscode-server/extensions/ms-python.python-2020.11.311084263-dev/pythonFiles/pyvsc-run-isolated.py flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s /mnt/c/workspace/omega/omega/fetcher/abstract_quotes_fetcher.py
cwd: /mnt/c/workspace/omega
> ~/miniconda3/envs/omega/bin/python ~/.vscode-server/extensions/ms-python.python-2020.11.311084263-dev/pythonFiles/pyvsc-run-isolated.py flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s /mnt/c/workspace/omega/omega/fetcher/abstract_quotes_fetcher.py
cwd: /mnt/c/workspace/omega

##########Linting Output - flake8##########
159,87,E,E225:missing whitespace around operator
159,89,E,E501:line too long (99 > 88 characters)
159,89,E,E222:multiple spaces after operator

Don't understand why flake8 --format is here since I've specified black as the formatter

close this issue. It's caused by black itself.
Black failed to format doc string which contains \t characters (exclude the indent characters)
it's curious why black report errors. the document says no matter with/without --quiet, any errors will be reported.


How do you package an .exe file with electron

Hello I am trying to call an .exe file like such var exePath = 'file://' + __dirname + '/run.exe';
This points at the build folder which clears every time you run the applicaiton with npm test. My question is can you  package and call a .exe file in your distribution if so how?

The community is a more suitable place for this kind of questions.


RPC not working: What I am doing wrong? Is there an example of its implementation?

Greetings,
Will you help me?
I apologize to bother but I am having troubles implementing RPC correctly.
Perhaps someone can point out what I am doing wrong in the code since I couldn't find in the documentation a full example on how to implemented.
// Publish RPC from this client to server
var extension = {
 id: UUID.random(),
}

var body = {...};

rabbit.publish('rpc.extension.request', {
    routingKey: 'extend.rpc',
    correlationId: extension.id,
    replyTo: "rpc.extension.response",
    contentType: "application/json",
    body: body,
    expiresAfter: 60000 // TTL in ms, in this example 60 seconds - can be removed after consumer working, just here to prevent junk accumulating on servers.
  }).then(
    () => { // a list of the messages of that succeeded
      log.info(`Success publishing extension for rpc ${extension.id}`);
    },
    (failed) => { // a list of failed messages and the errors `{ err, message }`
      log.info(`Error publishing extension for rpc ${extension.id} ->  ${failed.err}: ${failed.message}`);
    });

-----------------------
// Consuming RPC result sent by server to this client.

module.exports.setup = function () {
	...

	const handleRPC = rabbit.handle({queue: 'rpc.extension.response'}, handleExtendRPCResponse);
	handleRPC.catch(onHandleError);
};

function handleExtendRPCResponse(msg) {
	logger.info(`RPC Received message : ${msg}`)

	try {
		messageHandler.processMessage(msg, function (err) {
			if (err) {
				logger.error(`ERROR in RPC Process message: "${err.error}"`);
				msg.reject();
			} else {
				logger.info("Process RPC message passed");
				msg.ack();
			}
		})
	} catch(e){
		logger.error(`ERROR while Processing RPC message: "${e.message}"`);
		msg.reject();
	}
}

function onHandleError (err, msg) {
    // not able to hit this with a test...not sure how to.
    logger.error('Error:', JSON.stringify(err), 'Message:', JSON.stringify(msg));
    // Temporary, but if we can't handle it we don't want it requeued.
    msg.reject();
}

Thanks for your time and the creation of this package.

Maybe I got your question wrong, but if you trying to send back a result from the rpc call, you could use rabbit.request instead of rabbit.publish.
We do it like this:
rabbit
  .request('exchange', {
    type: 'req',
    body: task,
    routingKey
  })
  /*.progress(function(reply) {
    // if multiple replies are provided, all but the last will be sent via the progress callback
  })*/

  .then((final) => {
    // the last message in a series OR the only reply will be sent to this callback

    final.ack();
    if (
      Array.isArray(task.params) &&
      typeof task.params[task.cbPos] === 'function'
    ) {
      task.params[task.cbPos].apply(this, final.body.args);
    } else {
      //...
    }
  });

@manuel-reil, thanks for answering to my question.
Actually, I have a microservice that is trying to do an RPC request, via RabbitMQ, and then consume the result that comes back, via RabbitMQ.
Based on your feedback, I guess my RPC request was wrong because I was using publish() instead of request(). Am I correct?
Also, based on your feedback, I don't need to implement a  consumer since request() is going to wait for the return of the result of the RPC request and put it into the chain .then((final) -> ...)
Am I correct also here?

@acarlstein you are correct.  You want request() not publish() for rpc.  Under the covers during connection a unique rpc queue is automatically made for you to act as the listener queue to handle the return messages and abstracted to the promise level for ease of use. The rpc request uses the replyTo header that is added during the request() call in order to send it back the the right service that made the rpc request originally.

I figured out the issue. In the topology, when creating the exchange, I needed to add 'subscribe:true' property since the queue was already created but used as publish instead of request.
Thanks anyways for the feedback. I appreciate it.

Ah yes, the old subscribe:true I have been there myself

@zlintz,
I am having an issue with this as well. following your exact example above...
rabbit
  .request('myExchange', {
    type: 'req',
    body: task,
    correlationId: 'bat',
    routingKey: 'key'
  })
  .then((final) => {
    final.ack();
    log.info('received message');
  });

I am ale to see that my consumer (a java process ) is responding and that there is a message in the temporary queue, but 'received message' is never logged. When I turn onDEBUG=* I see messages like:
  rabbot.exchange debug +15s Publishing message ( type: 'req' topic: 'extend.fa', sequence: '0', correlation: '', replyTo: '{"type":"req","contentType":"application/json","
contentEncoding":"utf8","correlationId":"","replyTo":"xxx.response.queue","messageId":"087f6300-4751-11e9-98b1-f3c4b7b64947","timestamp":1552674897204,"a
ppId":"JSampson5530.npm.18772","headers":{},"mandatory":false,"persistent":true}' ) to direct exchange 'assignment.requests' on connection 'default'
  rabbot.acknack debug +365ms New pending tag 1 on queue xxx.response.queue - default
  rabbot.unhandled warn +1ms Message of xxx.response.queue on queue 'xxx.response.queue', connection 'default' was not processed by an
y registered handlers
  rabbot.acknack debug +3ms Marking tag 1 as nack'd on queue xxx.response.queue - default
  rabbot.acknack debug +83ms nack ALL (1) tags on xxx.response.queue up to 1 - default.
  rabbot.queue debug +1ms Nacking tag 1 on 'xxx.response.queue' - 'default'

The use of `subscribe:true' on the connection definition does not seem to have an effect.

@jtsampson can you provide a sample of the message coming back from the java consumer which is replying on the replyTo queue including all the headers. There are specific expectations that rabbot has around the RPC message coming back on the replyTo queue in order to connect the return message back the correct request() promise response. Most of these expectations are handled for you if you have  rabbot on both sides.

@zlintz ,
Thanks for getting back to me!
[aside] For sake of this discussion, and if it matters, the REQUESTER is a node app originally on Rabbit 1.1.0, but now upgraded to Rabbot 2.1.0 , the RESPONDER is a Java app using ampq-client-3.3.5 We have this working asynchronously and want to add a synchronous functionality using the same (or slightly modified) RESPONDER code.
I was pursuing this line of debugging earlier this week to try and determine exactly what the form of the message should be comming back from my RESPONDER but I hadn't gotten around to finishing it up.
Here is what I did so far.
Using the example code and topology in request.spec.js, and dupmping the response object from line 99 to the console I see the following
response: { fields:
   { consumerTag: 'MACHINE.NAME.PROCESS.NAME',
     deliveryTag: 2,
     redelivered: false,
     exchange: '',
     routingKey: 'MACHINE.NAME.PROCESS.NAME' },
  properties:
   { contentType: 'text/plain',
     contentEncoding: 'utf8',
     headers: { sequence_end: true },
     deliveryMode: undefined,
     priority: undefined,
     correlationId: '41dd6c20-4c22-11e9-b0db-c5719578f79c',
     replyTo: 'MACHINE.NAME.PROCESS.NAME',
     expiration: undefined,
     messageId: undefined,
     timestamp:<PHONE_NUMBER>918,
     type: 'polite.reply',
     userId: undefined,
     appId: undefined,
     clusterId: undefined },
  content: <Buffer 3a 44>,
  ack: { [Function: bound ack] [length]: 0, [name]: 'bound ack' },
  reject: { [Function: bound reject] [length]: 0, [name]: 'bound reject' },
  nack: { [Function: bound nack] [length]: 0, [name]: 'bound nack' },
  reply:
   { [Function]
     [length]: 2,
     [name]: '',
     [arguments]: null,
     [caller]: null,
     [prototype]: { [constructor]: [Circular] } },
  type: 'polite.reply',
  queue: 'MACHINE.NAME.PROCESS.NAME',
  body: ':D' }

I am guessing that my Java consumer has to respond with similar headers and properties but I am not exactly sure which ones...I tried to set up the minum number of these in my RESPONDER  code:
      channel = rabbitMqManager.createChannel();

      Map<String,Object> headers = new HashMap<>();
      headers.put("sequence_end", true); //  possibly expected by node.js rabbot library

      AMQP.BasicProperties props =
          new AMQP.BasicProperties.Builder()
                  .appId("CONSUME_APP") // this is the java consume app name not the rabbot publisher app name
                  .contentEncoding("UTF-8")
                  .contentType("application/json")
                  .correlationId(correlationId) // correlation id sent by publisher from rabbit properties
                  .type((String)properties.getType()) //  possibly expected by node.js rabbot library
                  .headers(headers).build();

      // reply to in this case is 'MACHINE.NAME.PROCESS.NAME'
      channel.basicPublish("", replyTo, props, (body.toJSONString().getBytes()));
      // thisgoes to the default exchange

Above I am  assumung (possibly wrongly) that I do not need to have the RESPONDER reply with any of the properties that are undefined from the dump and that the appId does not matter. Then my message will look something like this, I haven't trapped the actuall message yet, but am faily confident it looks like...
{
	"headers": {
		"sequence_end": true
	},
	"properties": {
		"contentEncoding": "UTF-8",
		"contentType": "application/json",
		"corelationId": "passed from PUBLISHER",
		"type": "type passed from PUBLISHER"
	},
	"body": "body"
}

Hopefully I am on the right track.
Do you know exactly which properties and headers I need?

@jtsampson
I did some testing and digging. In order for the response to make it back to the request() the consumer must take the properties.message_id from the message and then give that back as the properties.correlation_id on the return message on the reply to queue. The headers should also have "sequence_end": true
For example the message sent to be replied to on the queue...

And the response would be like...

I can explain more of the details if needed, let me know if this helps.

@zlintz,
Thanks for the run down, I will try this in my code, should take about a day or so to try out.. If that works I will definately be back for details, and I can certainly add a pull request for added documentation.

@jtsampson any update?
@acarlstein does this address your scenario as well?

@zlintz,
Sorry for the late reply.  This solves my issue perfectly and I PMed @acarlstein directly  and this was actually the solution to his problem as well.  You can close this issue.

Solution was also posted here: https://github.com/arobson/rabbot/issues/76

That makes sense why this problem seemed so familiar.  I helped @fifthfloorsoftware with this back then


New example showing subprojects

Please note that it has a bug that I am hoping to resolve quickly.
See issue #20

Thank you for opening this PR and raising the issue. This is still something I'd like to implement, hopefully for the next release.
#22 is open to track this feature, so I'm going to close this PR.

No problem at all. Thanks for the heads up.


Entities endpoint (Profile access) documentation update

Issue in ./help/profile/api/entities.md
Can we provide documentation on the guardrails associated with this API?

What is the expected latency?
How many calls can be made per second?
How many profiles can be requested in the multi-profile API lookup?

Tracking internally: https://jira.corp.adobe.com/browse/PLAT-110667


Added endpoints for seasonal forecast PDFs

Added endpoints:
seasonal_forecast_pdf (country, district) - returns URL for pdf
seasonal_forecast_list (country) - returns a list of districts with URL for PDF
This is currently only set up to work with Malawi using the data within https://www.metmalawi.gov.mw/docs/district_fcsts/
@chrismclarke Let me know if the link to the PDF is sufficient. Would you like me to do more than this?

Fixes #19

The 'webScrape' in app/core/webScrape/webscraper.py would be more Pythonic if it were changed to 'webscrape' or 'web_scrape'. I suggest using less nesting e.g. app/webscraper.py would probably suffice.
The logic in the web scraper can be simplified greatly by using CSS selectors:
links = [url + anchor["href"] for anchor in soup.select("tr:nth-child(n+5) td:nth-child(2) a")]

I think the endpoints could be reconsidered from a resource-oriented viewpoint. The resource is a seasonal forecast.
 Each forecast contains the following information:
{
  "country": "<country>",
  "district": "<district>",
  "formats": {
    "pdf": "<pdf-link>"
  }
}

This kind of design might be less susceptible to changes in the future.

@chrismclarke I have now changed this code to work from google buckets only. The plan will be that we populate the Google buckets from either a Google Drive or another endpoint that will allow users to upload files. This endpoint could be accessed from Climsoft.
This means that the bucket will be a flat structure with a strict file naming convention (holding the metadata we need)
The upload endpoint will use required parameters to create the file name
The drive will have code that will take whatever structure that suits the users (as long as it holds the data we need) and use that to populate the file name when it is transferred.
This will be the structure when requesting what forecasts needed
{
  "forecasts": {
    "mw": [
      {
        "id": "malawi_pdf_forecasts/-annual_forecast-en.pdf/1706176331764598",
        "date_modified": "2024-01-25T09:52:11.792000+00:00",
        "language": "en",
        "filename": "-annual_forecast-en.pdf",
        "district": "",
        "type": "annual_forecast"
      },
      {
        "id": "malawi_pdf_forecasts/district1-downscaled_forecast-en.pdf/1706176326207060",
        "date_modified": "2024-01-25T09:52:06.229000+00:00",
        "language": "en",
        "filename": "district1-downscaled_forecast-en.pdf",
        "district": "district1",
        "type": "downscaled_forecast"
      },

Note: I will not be using station_id as the forecasts are done per district. It could be possible to have more than one station in a district. Districts will correspond to the districts given in the station data
For getting the PDF just country and filename would be needed. As we are using a flat structure this will always be unique
I still need to sort out the current files in the buckets to make sure we have some good examples

Hey @ChrisMarsh82 - that looks incredibly useful, thanks for putting in the time to update.
Just to check, is there anything further to review in this PR at this point or would it be better to wait until changes are made?

@chrismclarke I think these changes can be merged now. Just need you to update your review