CORS Allow-Credentials and Expose-Headers support

I plan to create a PR to add support for Access-Control-Allow-Credentials and Access-Control-Expose-Headers in the CORS configuration block.
Right now you have to add swagger aws integration configuration for options requests to facilitate this. Having these setting in the CORS configuration block will make CORS configuration easier and especially less error prone .
We are using (the same) custom headers for our endpoints so moving this configuration from swagger into the CORS configuration block would improve our configuration workflow.
Adding support for these headers would complete CORS header support according to https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers

It appears to me that "Access-Control-Expose-Headers" is not meant to be set on preflight responses, but only on "actual" responses. SAM can't do anything with the application's responses; it's up to the application to set those. I think that adding it to a preflight response carries no meaning.

Hi @chrisoverzero , the Access-Control-Expose-Headers header is required to be set on the Options preflight call as well (at least in current Chrome version).

I can't seem to recreate that. I've created an API that responds to preflight requests without "Access-Control-Expose-Headers" and to non-preflight requests with "Access-Control-Expose-Headers". In Chrome 67.0.3396.79, I'm running this:
fetch(
  'http://localhost:5000/cors/exposed',
  {
    method: 'POST',
    headers: {
      'Content-Type': 'application/json',
      'Accept': 'application/json'
    },
    body: JSON.stringify({})
  }).then(r => console.log(r.headers.get('something-custom')))

…and I am able to read the value of the "Something-Custom" header.
Additionally, the spec calls out "Access-Control-Expose-Headers" as a header that appears in responses to non-preflight requests.

Access-Control-Allow-Credentials is valid for the responses to both preflight and actual CORS requests.

exactly and it should be part of preflight requests as well so CORS option in SAM should support this.

What should SAM do if a template sets Access-Control-Allow-Credentials and sets Access-Control-Allow-Origin to "'*'"?

Hi Chris,
IMO SAM can leave it as it is and pass it on to API gateway and browser will give error related to this as part of client request.
This is the same behaviour as API gateway.

Merged into develop. Thank you for this contribution!

Released!


Does not work with Sketch 53

I simply cannot export. Tried on several files and Sketch Measure freezes immediately on export. I love this plugin so please fix as soon as you can.

Confirming Sketch Measure doesn't work in Sketch 53.1. It locks-up during the export (when the progress bar appears) for me in macOS 10.14.3.

Confirmed

Confirmed. Hope there is a fix soon.

https://github.com/utom/sketch-measure/tree/abf69ce2ea96136cf725ddb238d4dbd0ba1612b1 Download this version if you have urgency.

I reverted back to Sketch 52.
Looks like the tool gets stuck when attempting to access external symbols & libraries.

That link is for v 2.7.5
I am using 2.7.7
Both have the same failure here. So I cannot fathom why yours works. Do you have symbols or nested symbols in your file? Or libraries?

@cpshanley @AndrewHaglund @luisalves73 @siddharthksuri Please give this a try: #536

#536 (version 2.7.9) worked an absolute treat. Brilliant stuff, and well done. One of my 'must-have' plugins, so I'm delighted.
Cheers


sign images published to ghcr

Describe the solution you'd like
[A clear and concise description of what you want to happen.]
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Eraser version:
Kubernetes version: (use kubectl version):

/hold


Model cleanup

As the Module and the CompositeModule classes are not abstract anymore since #76, i removed all their subclasses that did not provide any additional attributes (i.e. where the above mentioned non-abstract classes can be used now).
I also added missing @Table annoations to these classes.
I could successfully run the adapted webapp-archetype and start the application afterwards!
Please review and merge. Thx


Fix nil pointer when fetching snowmachineconfig that does not exist

Issue #, if available:
GA AppSec review finding: Potential Memory Violation / NULL Pointer Dereference when specifying a machineRef that does not exist in cluster spec.
Description of changes:
Add validation to check all the machine refs point to objs that exist. For controller we requeue and wait till the snow machine config object exists in the cluster.
Testing (if applicable):
Unit tests
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


feat: add total_samples as a field to logs being emitted

Today we emit a number of important values in the logs, but we do not explicitly announce how many samples
we are planning on iterating through. This commit emits total_samples as a field in both the initial log
as well as the subsequent logs during training. This way, this value can be accessed consistently by
any program interested in displaying information about its progress for convenience.
Fixes #325
Signed-off-by: Oleg S<EMAIL_ADDRESS>

I approved then put the hold- quick q, is len(dataloader.dataset) a cached length value or is it computed each time? we could cache it before we start the loop if it's not cached already.

@JamesKunstle What is your concern with the calculation of len(dataloader.dataset) if it's not cached? In Python, the __len__ implementation should be $O(n)$. If this isn't the case then it would be a bug that needs to be fixed elsewhere.

@RobotSail that tends to be the case for inbuilt stuff but this would be a length calculation for our custom Dataset objects. I confirmed that it's O(1) as best as I can tell so this should be good


OpenTofu infracost support

Trying to use the infracost extension with an opentofu workspace results in the following output:
***************************************
Command Priority 200
Running BASH
Script Information:
terraform show -json terraformLibrary.tfPlan > plan.json
infracost breakdown --path plan.json

***************************************
Begin Execution:
/home/cnb/.terraform-spring-boot/executor/85e6e129-2141-4e7d-b62d-14eb71af4b8d/9f326e6d-023e-461c-99f9-f9ee29c82e5b/userScript.sh: line 1: terraform: command not found
Error: Could not detect path type for plan.json

Use the --path flag to specify the path to one of the following:
 - Terraform plan JSON file
 - Terraform/Terragrunt directory
 - Terraform plan file
 - Terraform state JSON file

Update: A new version of Infracost is available: v0.9.11 ��� v0.10.37
  $ curl -fsSL https://raw.githubusercontent.com/infracost/infracost/master/scripts/install.sh | sh
Script Exit Code ==>1
End Execution...

Changing the plan to read as follows:
flow:
- type: "terraformPlan"
  step: 100
  commands:
    - runtime: "GROOVY"
      priority: 100
      after: true
      script: |
        import Infracost

        String credentials = "version: \"0.1\"\n" +
                "api_key: $INFRACOST_KEY \n" +
                "pricing_api_endpoint: https://pricing.api.infracost.io"

        new Infracost().loadTool(
           "$workingDirectory",
           "$bashToolsDirectory",
           "0.10.37",
           credentials)
        "Infracost Download Completed..."
    - runtime: "BASH"
      priority: 200
      after: true
      script: |
        tofu show -json terraformLibrary.tfPlan > plan.json
        infracost breakdown --path plan.json

Still does not yield working results (tofu: command not found) the output was:
***************************************
Command Priority 200
Running BASH
Script Information:
tofu show -json terraformLibrary.tfPlan > plan.json
infracost breakdown --path plan.json

***************************************
Begin Execution:
/home/cnb/.terraform-spring-boot/executor/85e6e129-2141-4e7d-b62d-14eb71af4b8d/9f326e6d-023e-461c-99f9-f9ee29c82e5b/userScript.sh: line 1: tofu: command not found
Error: No valid Terraform files found at the given path, try a different directory

Try setting --path to a Terraform plan JSON file. See https://infracost.io/troubleshoot for how to generate this.

Update: A new version of Infracost is available: v0.10.12 ��� v0.10.37
  $ curl -fsSL https://raw.githubusercontent.com/infracost/infracost/master/scripts/install.sh | sh
Script Exit Code ==>1
End Execution...

Any help would be greatly appreciated.

Hello @syserr0r , I did a small fix and created a new beta, you can test using the executor version  2.21.4-beta.1  and that one should fix your issue

Thank-you for the quick fix, yes this is working well for us now 👍


Use first non-empty row with IsFirstRowAsColumnNames (add option ?)

Sometimes I have to parse Excel files that contain an array, but the user added an empty row before the header row, which breaks everything.
It would be nice if ExcelDataReader could just ignore the empty rows when IsFirstRowAsColumnNames is set to true and use the first non-empty row.
As it would change the behavior of the current release, I guess an option could be added.

Is it closed because fixed/implemented, or too old ? :)

Is seems a "custom" implementation you would have in "reader" classes branching from dev build.


When trying to run export.sh I get no such file or directory: $HOME/esp/esp_idf/tools/idf_tools.py (IDFGH-2699)

Environment

IDF version: v4.1-dev-2263-g605da33c3
Operating System:     macOS
Using an IDE?: No

Problem Description
When running export.sh I get an error saying
Adding ESP-IDF tools to PATH...
idf_export_main:33: no such file or directory: /Users/victorgarcia/esp/esp_idf/tools/idf_tools.py

Before this error I get an error saying that IDF_PATH is not set although I'm using the path that is described in the getting started guide.

Hi @vmgarcia, thanks for reporting this. Couple of questions: if you run /Users/victorgarcia/esp/esp_idf/tools/idf_tools.py export from the shell, does it report the same error? Which shell are you using (bash, zsh)? What does which python report in your shell?

hi @igrr thanks for the quick response!
When I run /Users/victorgarcia/esp/esp_idf/tools/idf_tools.py export I get a similar error
zsh: no such file or directory: /Users/victorgarcia/esp/esp_idf/tools/idf_tools.py
which python returns /usr/bin/python

Okay, what about if you run python /Users/victorgarcia/esp/esp_idf/tools/idf_tools.py export?

"Also please check if you have esp-idf or esp_idf in the path (I see both in your last post)."
Nice callout, sorry I just copy pasted your command but when I ran export.sh using the previous command you told me to run I got a valid output
/Users/victorgarcia/esp/esp_idf/tools/idf_tools.py export returned
export IDF_PYTHON_ENV_PATH="/Users/victorgarcia/.espressif/python_env/idf4.1_py2.7_env";export OPENOCD_SCRIPTS="/Users/victorgarcia/.espressif/tools/openocd-esp32/v0.10.0-esp32-20191114/openocd-esp32/share/openocd/scripts";export PATH="/Users/victorgarcia/.espressif/tools/xtensa-esp32-elf/esp-2019r2-8.2.0/xtensa-esp32-elf/bin:/Users/victorgarcia/.espressif/tools/xtensa-esp32s2-elf/esp-2019r2-8.2.0/xtensa-esp32s2-elf/bin:/Users/victorgarcia/.espressif/tools/esp32ulp-elf/2.28.51-esp-20191205/esp32ulp-elf-binutils/bin:/Users/victorgarcia/.espressif/tools/esp32s2ulp-elf/2.28.51-esp-20191205/esp32s2ulp-elf-binutils/bin:/Users/victorgarcia/.espressif/tools/openocd-esp32/v0.10.0-esp32-20191114/openocd-esp32/bin:/Users/victorgarcia/.espressif/python_env/idf4.1_py2.7_env/bin:/Users/victorgarcia/esp/esp-idf/tools:$PATH"```

Nice callout, sorry I just copy pasted your command

Well, I just copy-pasted the path from your original post :)

no such file or directory: /Users/victorgarcia/esp/esp_idf/tools/idf_tools.py

So is IDF_PATH variable set to esp_idf or esp-idf?.. (check by echo $IDF_PATH)

So is IDF_PATH variable set to esp_idf or esp-idf?.. (check by echo $IDF_PATH)

I had set it to with an underscore....
When I do correct it I get
Adding ESP-IDF tools to PATH...
Checking if Python packages are up to date...
Traceback (most recent call last):
  File "/Users/victorgarcia/esp/esp-idf/tools/check_python_dependencies.py", line 42, in <module>
    default_requirements_path = os.path.join(idf_path, 'requirements.txt')
  File "/Users/victorgarcia/.espressif/python_env/idf4.1_py2.7_env/lib/python2.7/posixpath.py", line 70, in join
    elif path == '' or path.endswith('/'):
AttributeError: 'NoneType' object has no attribute 'endswith'```

Just to check, are you exporting IDF_PATH variable, or just setting it? I.e. export IDF_PATH=$HOME/esp/esp-idf versus IDF_PATH=$HOME/esp/esp-idf. It seems that check_python_dependencies.py can return this error if somehow IDF_PATH is not found in its environment.

@vmgarcia does the issue still exist?


Address TODOs related to calico's providerConfig in provider-local Shoot manifests

How to categorize this PR?

/kind cleanup
What this PR does / why we need it:
Address TODOs related to calico's providerConfig in provider-local Shoot manifests: TODOs introduced with https://github.com/gardener/gardener/pull/7684.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

NONE

/retest

/lgtm

/test pull-gardener-e2e-kind-operator

/test pull-gardener-e2e-kind-operator


Are there any further development plans for the framework beyond developing language model-based responses for emergency situations and accepting human natural language commands?

this comes from a reviewer in IVS2024.

the framework will add more random events and make them available through config files.
the framework will introduce more baselines, like optimization-based and meta-heuristic-based.
SUMO is considered to be too heavy for the environment and will not be implemented unless the community commit.


fix: magic-link URL protocol for localhost

Currently the protocol that reaches the email on localhost seems to be https instead of http.

This shouldn't be happening, what did I do wrong?
https://github.com/dev-xo/remix-auth-otp/blob/fff3628fa78b428e73bf29c57a85e08dea2c3be6/src/utils.ts#L60-L64

Maybe it would be safer to set some BASE_URL env and pass it as variable?

Fixed @leimantas!
Merging right now! 🎉


proxy_cache_path not honoring max_size or inactive parameters

Environment
Include the result of the following commands:

nginx -V

nginx -V nginx version: openresty/<IP_ADDRESS> built by gcc 11.2.0 (GCC) built with OpenSSL 3.0.3 3 May 2022 TLS SNI support enabled configure arguments: --prefix=/usr/local/openresty/nginx --with-cc-opt=-O2 --add-module=../ngx_devel_kit-0.3.0 --add-module=../echo-nginx-module-0.61 --add-module=../xss-nginx-module-0.06 --add-module=../ngx_coolkit-0.2rc3 --add-module=../set-misc-nginx-module-0.32 --add-module=../form-input-nginx-module-0.12 --add-module=../encrypted-session-nginx-module-0.08 --add-module=../srcache-nginx-module-0.31 --add-module=../ngx_lua-0.10.13 --add-module=../ngx_lua_upstream-0.07 --add-module=../headers-more-nginx-module-0.33 --add-module=../array-var-nginx-module-0.05 --add-module=../memc-nginx-module-0.19 --add-module=../redis2-nginx-module-0.15 --add-module=../redis-nginx-module-0.3.7 --add-module=../rds-json-nginx-module-0.15 --add-module=../rds-csv-nginx-module-0.09 --add-module=../ngx_stream_lua-0.0.5 --with-ld-opt=-Wl,-rpath,/usr/local/openresty/luajit/lib --with-openssl=/tmp/openssl-3.0.3 --with-pcre=/tmp/pcre-8.42 --with-file-aio --with-http_addition_module --with-http_auth_request_module --with-http_dav_module --with-http_flv_module --with-http_gunzip_module --with-http_gzip_static_module --with-http_image_filter_module=dynamic --with-http_mp4_module --with-http_random_index_module --with-http_realip_module --with-http_secure_link_module --with-http_slice_module --with-http_ssl_module --with-http_stub_status_module --with-http_sub_module --with-http_v2_module --with-http_xslt_module=dynamic --with-ipv6 --with-mail --with-mail_ssl_module --with-md5-asm --with-pcre-jit --with-sha1-asm --with-stream --with-stream_ssl_module --with-threads --with-stream --with-stream_ssl_module

uname -a

Linux ... <IP_ADDRESS>-1.cm2 #1 SMP Sun Aug 18 19:16:21 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Description
The http file cache directory /var/blobs/cache is almost 14G in size, and all data is expired (assuming the default inactive of 10 minutes and given that there is no traffic to the server anymore.)
Yet, the cache data is not cleared. I've tried restarting nginx and the VM it's running on as well. Pasted the debug logs I collected in the gist below after restarting nginx.
nginx configuration
http
...
  proxy_cache_path /var/blobs/cache keys_zone=blobcache:30m max_size=10g;`
  ...
  server {
    ...
    localtion ... {
      ...
      proxy_cache blobcache;
      ...
    }
  }
...

nginx debug log
https://gist.github.com/avtakkar/b38a504df8204d7a5c86ccbe3cb202fd

@avtakkar NGINX performs lazy removal of cache entries on disk. From the docs:

The special “cache manager” process monitors the maximum cache size set by the max_size parameter, and the minimum amount of free space set by the min_free (1.19.1) parameter on the file system with cache. When the size is exceeded or there is not enough free space, it removes the least recently used data. The data is removed in iterations configured by manager_files, manager_threshold, and manager_sleep parameters (1.11.5).

If you set min_free to the same value as max_size, the cache entries should be deleted from disk as soon as expired.

It looks like this is not vanilla nginx, but instead an OpenResty version that is based on nginx v1.13.6 (this version of nginx was released on Oct 10th 2017). The current release version of nginx sits at 1.27.1. The functionality of the nginx proxy module lies within src/http/modules/ngx_http_proxy_module.c. This file has seen significant changes from versions 1.13.6 to 1.27.1 as they are almost exactly eight years apart.
OpenResty is very good at closely following the nginx's release versions, I would take advantage of this and update the server from OpenResty's <IP_ADDRESS> to <IP_ADDRESS>. If the issue persists, the opensource community will have a much easier time identifying the source of the bug.

Please note that 10 minutes is just the minimum limit of inactivity, the cache manager will only delete when the allocated space is exceeded or the storage space is insufficient. This deletion is not greedy, it is deleted on demand, so you can't expect it to help you clear all files that have not been used for more than 10 minutes.
Here is the documentation for the cache manager
https://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_cache_path

The special “cache manager” process monitors the maximum cache size set by the max_size parameter, and the minimum amount of free space set by the min_free (1.19.1) parameter on the file system with cache. When the size is exceeded or there is not enough free space, it removes the least recently used data. The data is removed in iterations configured by manager_files, manager_threshold, and manager_sleep parameters (1.11.5). During one iteration no more than manager_files items are deleted (by default, 100). The duration of one iteration is limited by the manager_threshold parameter (by default, 200 milliseconds). Between iterations, a pause configured by the manager_sleep parameter (by default, 50 milliseconds) is made.

This file has seen significant changes from versions 1.13.6 to 1.27.1 as they are almost exactly eight years apart.

Yes, I'll try to update on our end, and see if the behaviour reproduces. I was wondering if there are any known issues that could be causing this.

This deletion is not greedy, it is deleted on demand, so you can't expect it to help you clear all files that have not been used for more than 10 minutes.

Interesting. On most other machines, I have seen it clear the files very shortly after the configured thresholds are exceeded. Good to know, thanks.

I'm seeing this issue in two of my servers running nginx/1.21.6. The cache is configured to 30g, and I had to manually delete the cache data to free up space. One server had 46GB of cache files and the other one, 53GB.


feat: support private types

Adds a --private option which can be used to generate types for private
resources and modules (types that are activated in the current AWS account and region).
Resolves #40

Copying @hoegertn


[BUG] Error building Blazor WASM project with SkiaSharp.Views.Blazor in Docker Container

Description
When trying to build a Blazor WASM Aspnet Hosted Project in a Docker Container I get an build error.
Code
Repository with solution to reproduce the issue
Expected Behavior
To build correctly without an error like on Windows Desktop.
Actual Behavior
Builds with the following error:
C:\Program Files\dotnet\packs\Microsoft.NET.Runtime.WebAssembly.Sdk\7.0.2\Sdk\WasmApp.Native.targets(285,5): error MSB3073: The command ""C:\Program Files\dotnet\packs\Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.browser-wasm\7.0.2\Sdk\..\tools\mono-aot-cross.exe" --print-icall-table > "C:\src\BlazorSkiaSharp\Client\obj\Debug\net7.0\wasm\for-build\runtime-icall-table.h"" exited with code -1073741511. [C:\src\BlazorSkiaSharp\Client\BlazorSkiaSharp.Client.csproj]
Basic Information

Version with issue:  SkiaSharp.Views.Blazor version 2.88.3
Last known good version:  unknown
IDE:  Visual Studio 2022 Version 17.4.4
Platform Target Frameworks:

WASM

Target Devices:

Browser

Reproduction Link
Repository with solution to reproduce the issue

I was able to work around this issue using a Docker Container with Windows Server Core instead of Windows Nano Server. I'm not sure what the root cause of the issue was.


Missing support for locale en_GB

Locale en_GB is not supported.
Spelling differences that I have noticed are:

Color -> Colour
Canceled -> Cancelled
Canceling -> Cancelling
Gray -> Grey

I modified the existing "cups_en.po" as "cups_en_GB.po".
cups_en_GB.po.zip


Profile wizard does not work properly

Profiles are being sent multiple times with only one game

Fixed in #40


Single mode for particular dropdown

Discussed in https://github.com/xrutayisire/react-js-cron/discussions/34

Originally posted by abhijeet3112 September  3, 2022
Is there any functionality we have that supports setting the single dropdown selection for minutes dropdown while keeping other dropdowns with multi select option?

hi @abhijeet3112
Check my fork: https://github.com/mpycio/react-js-cron.git
I've extended this awesome component by extra prop:
componentProps={{       minute: {         unitFilter: ({ value, label }) => parseInt(value) % 5 == 0,         mode: "single",         allowClear: false,       },       hour: {         unitFilter: ({ value, label }) => !["22", "23"].includes(value),         mode: "single",         allowClear: false,       }, }}

it allows you to filter options for individual fields (I needed to limit minutes to allow selecting only 5/10/15,
allows you to configure clear button on each component
allows you to set a single/multiple mode on each component

@xrutayisire
If you accept PRs and this this is useful extension I can clean that code a bit and create a PR, we could discuss possibility of merging that if you want. There's already 55 forks no need for another.

Hello,
Definitely ok for PRs!
Open source projects are here for that! More than happy to discuss your work :)

Hello @mpycio @xrutayisire, is this PR merged? Is there anyway I can use mode on individual time units (hours, minutes etc)? Kindly help.

Hey,
I don't think I received a PR for that at the end. Maybe @mpycio you could open one with your work?

Hey @mpycio, could you kindly open a PR for the same? If they are busy or currently unable to raise a PR for whatsoever reason, I can create a PR for the same giving them the due credits. Let me know if that works @xrutayisire?

Sure no problem for me 🙂

Hi,
I think I stopped when I encountered some ant version conflicts and did not have time to resolve them. I've now rebased my fork from the @xrutayisire master and created PR. There's some cleanup to be done and I haven't tested it yet after rebase.

@prajwal-kulkarni please take a look to see if it would be good for you.
If @prajwal-kulkarni you can finish the PR it would be perfect, and I would approve it ASAP.
If you want me to finish it, I can find some time to do it, but it would be next week 😕

Thank you so much @mpycio !

Hey @xrutayisire
Sure, I can co-author @mpycio's PR to make the necessary changes(if any), code cleanup and dev-test it. I will start working  on it today, and expecting it complete by this weekend.

Hello @xrutayisire
I've created a new PR #58 which is worked on top of @mpycio's PR with some refactoring, code cleanup, and added some new test cases.
Since I couldn't directly add commits to their PR, I had to create a new PR, however, both of our commits are present in the PR. I've dev-tested the changes and also all the formatting/linting and test cases are passing. I kindly request you to review the PR.

Hey,
Thank you very much both of you, I'm not home at the moment. I will try to get that done this week-end 🙂
If everything looks good or close enough, I will merge, do some changes if necessary and release a new version directly. 🚀

Sure thank you, it'd be very helpful if you can release the new version at the earliest, as we're currently facing a blocker and need to progress ahead. Looking forward to the new release.

Sure, I will do my best.
I need to rework a bit the PR to make it really extendable and generic. It's not that much work, but I think I could be finished by the end of Monday or Tuesday.
Stay tuned 🚀

Hey, I worked tonight and at the end it was necessary for me to create something from scratch.
The goal is to have some generic and not specific for one use case.
In my commit, I create a dropdownsConfig prop for the Cron component. With that, you can override every property that currently exist that apply to a specific dropdown.
I then added allowClear as a global prop and so as a dropdownsConfig prop too.
I finally added filterOption as a dropdownsConfig prop only since i'ts specific to each dropdown.
I inspired myself from antd for naming and tried to be more generic compare to what you did.
I'm thankful for the two PRs you created, but it was not going in the direction I think this library should go to answer future needs. Anyway, at the end the result is here, and you will be able to configure each dropdown as you want 🙂
Please tell me @mpycio and @prajwalkulkarni if the commit I did answer your needs?
You can test it here with storybook, there is a new Story for the dropdownsConfig prop.
If yes, I will create a new minor release.
Here is what the new prop looks like:
  /**
   * Define a configuration that is used for each dropdown specifically.
   * Configuring a dropdown will override any global configuration for the same property.
   *
   * Configurations available:
   *
   * // See global configuration
   * // For 'months' and 'week-days'
   * humanizeLabels?: boolean
   *
   * // See global configuration
   * // For 'months' and 'week-days'
   * humanizeValue?: boolean
   *
   * // See global configuration
   * // For 'month-days', 'hours' and 'minutes'
   * leadingZero?: boolean
   *
   * // See global configuration
   * For 'period', 'months', 'month-days', 'week-days', 'hours' and 'minutes'
   * disabled?: boolean
   *
   * // See global configuration
   * For 'period', 'months', 'month-days', 'week-days', 'hours' and 'minutes'
   * readOnly?: boolean
   *
   * // See global configuration
   * // For 'period', 'months', 'month-days', 'week-days', 'hours' and 'minutes'
   * allowClear?: boolean
   *
   * // See global configuration
   * // For 'months', 'month-days', 'week-days', 'hours' and 'minutes'
   * periodicityOnDoubleClick?: boolean
   *
   * // See global configuration
   * // For 'months', 'month-days', 'week-days', 'hours' and 'minutes'
   * mode?: Mode
   *
   * // The function will receive one argument, an object with value and label.
   * // If the function returns true, the option will be included in the filtered set.
   * // Otherwise, it will be excluded.
   * // For 'months', 'month-days', 'week-days', 'hours' and 'minutes'
   * filterOption?: FilterOption
   *
   * Default: undefined
   */
  dropdownsConfig?: {
    'period'?: {
      disabled?: boolean
      readOnly?: boolean
      allowClear?: boolean
    }
    'months'?: {
      humanizeLabels?: boolean
      humanizeValue?: boolean
      disabled?: boolean
      readOnly?: boolean
      allowClear?: boolean
      periodicityOnDoubleClick?: boolean
      mode?: 'multiple' | 'single'
      filterOption?: ({
          value,
          label,
        }: {
          value: string
          label: string
        }) => boolean
    }
    'month-days'?: {
      leadingZero?: boolean
      disabled?: boolean
      readOnly?: boolean
      allowClear?: boolean
      periodicityOnDoubleClick?: boolean
      mode?: 'multiple' | 'single'
      filterOption?: ({
          value,
          label,
        }: {
          value: string
          label: string
        }) => boolean
    }
    'week-days'?: {
      humanizeLabels?: boolean
      humanizeValue?: boolean
      disabled?: boolean
      readOnly?: boolean
      allowClear?: boolean
      periodicityOnDoubleClick?: boolean
      mode?: 'multiple' | 'single'
      filterOption?: ({
          value,
          label,
        }: {
          value: string
          label: string
        }) => boolean
    }
    'hours'?: {
      leadingZero?: boolean
      disabled?: boolean
      readOnly?: boolean
      allowClear?: boolean
      periodicityOnDoubleClick?: boolean
      mode?: 'multiple' | 'single'
      filterOption?: ({
          value,
          label,
        }: {
          value: string
          label: string
        }) => boolean
    }
    'minutes'?: {
      leadingZero?: boolean
      disabled?: boolean
      readOnly?: boolean
      allowClear?: boolean
      periodicityOnDoubleClick?: boolean
      mode?: 'multiple' | 'single'
      filterOption?: ({
          value,
          label,
        }: {
          value: string
          label: string
        }) => boolean
    }
  }

Hey @xrutayisire , yes this looks good and fulfils my requirement, could you kindly release a minor version for the same?

Great 👍 I will do that now 🚀

Release v4.1.0 done 🚀
Thank you two for this discussion!
Don't hesitate to comment here if there is a problem with the release 👍


Update change_form.html

Refactor: Replace double quotes with single quotes in admin forms stylesheet link
This commit updates the stylesheet link in the admin forms block from double quotes to single quotes for consistency with the existing codebase. The change enhances code readability and aligns with the project's coding conventions.
Changed:
{% block extrastyle %}{{ block.super }}<link rel="stylesheet" href="{% static "admin/css/forms.css" %}">{% endblock %}
To:
{% block extrastyle %}{{ block.super }}{% endblock %}

Hi @RamosRRamos thanks for the PR.
Generally django avoids small fornatting changes like this as it makes looking at the history more complicated.
I'll therefore close this PR.
If you're looking to make your first contribution to django  I'd recommend looking at trac for a ticket in a component that your particularly interested in. Good luck!
https://code.djangoproject.com/

Sure, thank you Smith!


Creates a chemical health change component for medicines and poisons

This is a slight change to my previous pr, making it more useful in general. The reason I didn't just reopen my old pr is because I can't, since there isn't the normal reopen button next to comment

No progress in a long time. Closing.


Webpacker.dev_server.running? will always return false on precompile_assets.rb

Given that rspec will change RAILS_ENV to "test", and that Webpacker has no "dev_server" configuration at config/webpacker.yml for the test environment by default - which is required to run the socket connection check of Webpacker.dev_server.running?...

...the precompile_assets.rb script won't detect Webpacker dev server running, even if it's actually running.

Crap! RTFM
# Do not precompile if webpack-dev-server is running (NOTE: MUST be launched
# with RAILS_ENV=test)

I'm closing this one now.


4.7.0 is not updating apk on device

Hi,
I am using version 4.7.0 and cannot update any app on any phone.
Starting with java -jar raccoon-4.7.0.jar. There are no exceptions thrown in stout.
Phone details:
[ro.product.model]: [SM-G900F]
[ro.product.name]: [lineage_klte]
[ro.build.version.release]: [9]
[ro.build.version.sdk]: [28]
I also tried my tablet with a different Android version (8).
The problem appeared out of a sudden and already with version 4.6.0. There were no mayor upgrades on the phone.
adb logcat shows no exceptions or errors while trying to update using raccoon.
Any ideas?
Best regards!

Am 20.05.2019 20:28, schrieb ErikOlufson:

Hi,
I am using version 4.7.0 and cannot update any app on any phone.
Starting with java -jar raccoon-4.7.0.jar. There are no exceptions
thrown in stout.
Phone details:
[ro.product.model]: [SM-G900F]
[ro.product.name]: [lineage_klte]
[ro.build.version.release]: [9]
[ro.build.version.sdk]: [28]
I also tried my tablet with a different Android version (8).
The problem appeared out of a sudden and already with version 4.6.0.
There were no mayor upgrades on the phone.
adb logcat shows no exceptions or errors while trying to update using
raccoon.
Any ideas?

This is tech support. You'll have to file a ticket under Help|Support.


Which version of the MPI standard is made available?

Maybe the information is buried somewhere, but I couldn't find it... Anyways, there should be a small notice on the main page, which says what version of the MPI standard can be used with this implementation!

The README file says which version of MPI is supported by MSMPI. It is 2.2 with some features from 3.1.


Support Chinese

The programme is great,
but using the output only outputs English CSV....
Can you add Chinese google map output, thanks.

Use chinese language, and change download setting and disable english conversion

Yes, I have noticed this feature, but the vast majority is still in English, with only a very small part in Chinese.

We give you direct results we get from gmaps, if there are not in chineses, then results will the same.

We give you direct results we get from gmaps, if there are not in chineses, then results will the same.

"I was originally using hl=en, but if I change it to zh-TW, I believe I can successfully obtain the correct Chinese version.

https://www.google.com/maps/place/%E9%8A%93%E5%8F%8B%E9%9B%BB%E6%B1%A0-%E6%B9%AF%E6%B7%BA%E9%9B%BB%E6%B1%A0%E5%B0%88%E8%B3%A3%E5%BA%97/data=!4m7!3m6!1s0x34681f2166960915:0xc64090935296478a!8m2!3d24.9782994!4d121.3036157!16s%2Fg%2F11bwn4dz45!19sChIJFQmWZiEfaDQRikeWUpOQQMY?authuser=0&hl=en&rclk=1

https://www.google.com/maps/place/%E9%8A%93%E5%8F%8B%E9%9B%BB%E6%B1%A0-%E6%B9%AF%E6%B7%BA%E9%9B%BB%E6%B1%A0%E5%B0%88%E8%B3%A3%E5%BA%97/data=!4m7!3m6!1s0x34681f2166960915:0xc64090935296478a!8m2!3d24.9782994!4d121.3036157!16s%2Fg%2F11bwn4dz45!19sChIJFQmWZiEfaDQRikeWUpOQQMY?authuser=0&hl=zh-TW&rclk=1

Open backend/scrapers.py and add following line to use zh-TW.

You are my god, the test was successful, thank you so much.


Lookup for generators in the current location

Type of issue: Feature request
I have two problems with yeoman that I think should be discussed and resolved.
The first is that you have to install yeoman globally to have it work. And I just can't ask my developers to install something globally on their computer just to work on a project. A project should be self-contained and usable with nothing else that itself and its dependencies.
The second is that I absolutely don't want to publish my generators on NPM. They are copyright protected by my company, and each one are finely crafted specifically for each project.
Currently, yeoman's approach is based on NPM-published generators and a globally installed CLI tool. Or locally installed generators registered globally with npm link.
To cope with my strong principles (the main being: a good project must be self-contained), I had to find a way to use yeoman locally and have it lookup for generators in the local folder.
It actually was quite easy to do. I wrote a yo.js script that can be exceuted by nodejs and...voila! This script is located at the root of each project, yeoman is added as a dev dependency of the project and the generators are maintained locally inside the project.
 Take a look at this:
let path = require('path');
let resolver = require('yeoman-environment/lib/resolver');

resolver.getNpmPaths = function() {
  return [path.join(__dirname, 'lib')];
};

require('yo/lib/cli');

As you can see, the script consist in registering the current path (well, lib subpath in this case) to the yeoman resolver and calling the cli. But that doesn't feel right to me: replacing a function inside a dependency and calling a cli with a require is not what I consider a good practice.
I would rather write something like this:
let yo = require('yo');

yo.registerLocalGenerators(path.join(__dirname, 'lib'));
yo.run();

Basically the same thing but safe and clean.
Before trying to add something like this to Yeoman, I'd like to know what the community thinks about it. And ensure if it is not already on the roadmap.

mhmm. Not sure if I get your points.

You can of course install yo locally just like every other node cli

You could even use it without installing at all via npx

You can use local generators via yo ./path/to/some/generator

or use npm to install your generator from a private Git endpoint instead of the public registry.

Cheers, Michael

Hmmm, maybe something is wring with my setup, because after having installed it via npm install yo --save-dev I get yo: command not found when I execute the command yo.

And I didn't know you could execute a generator by its path. That's perfect!

Guess that’s something we should document more prominent. Going to open a separate issue for that.

Hmmm, maybe something is wrong with my setup, because after having installed it via npm install yo --save-dev I get yo: command not found when I execute the command yo.

Yup. There are three possibilities to use a locally installed CLI in your project:

use npx
explicitly call the locally installed binary via node_modules/.bin/yo
setup a npm script by adding the following to the package.json of your project"scripts": {
    "yo": "yo --version",
}

Calling npm run yo executes the defined command using the locally installed version.

This is awesome!
I've been using npm for years and I didn't know you could execute locally installed CLI with it. Thanks a lot for your help, it works perfectly. I can know happily delete my script and update my NPM scripts accordingly.
Many thanks.

My pleasure 😘

Also, with yarn, yarn xxx will automatically run any local binaries. No need for a specific script in package.json.

Oh. Nice, didn’t knew this.


Player.getLevel() not work?

function modTick() {
if(Player.getLevel() >= 40){
Entity.addEffect(getPlayerEnt(), MobEffect.digSpeed, 5*20, 1, false, true);
say("You have Reached Tier 1 Lvl");
}}
Or maybe im doing the code wrong?

Player.getLevel seems to work - are you sure your player has enough exp to get to level 40?

Oh, experience conquest yes?


Use allocator in yp string free

relies on: https://github.com/ruby/yarp/pull/1375
Moves frees from yp_string_free to the allocator. Review only the last commit for changes outside of creating the allocator.

I am going to restart this one tracking a malloc source instead of a free. There are too many sources for this alloc


Time type alias defined in different places

There is one definition of
type alias Time = Float

in Time.elm and an identical one in Task.elm.
This seems conceptually wrong, and potentially hurtful for later developments. So I think it would be better to have that definition only in one place, and import it everywhere else it is needed.
What would be options for this?

One natural place to put the single definition of Time would be Time.elm, but that is problematic, since Time.elm imports Signal.elm, which imports Task.elm, so having Task.elm import Time.elm would lead to a cycle.
The definition could be put into Task.elm and Time.elm could import it from there. It seems a bit odd, though, to have the primary definition of Time (and the place where it is documented) be in Task.elm.
The definition could be put into Basics.elm and be imported into Time.elm and Task.elm from there.

Is anything wrong with the third option? It would essentially give Time the same status as Int, Float, String as pre-defined Elm types. It seems that is an okay thing to happen.

Task does not expose the Time alias, and the compiler can figure out what type it wants, so I don't see much of a downside to the current arrangement. I suppose I could get behind placing it in Basics, if that wouldn't make Task and Time much more complicated, but again I don't see the need to.

It's not a big issue, of course. More a matter of principle and style and looking into possible futures. Consider:

At some point it is decided that Time should not be Float, but maybe Int or even some completely other type. Then there will be two places where this has to be changed. No big problem, I'll grant, because at least these two places are both in the same package core, so the compiler would likely catch this before release. (Actually, it would only catch it if there is some code in core in which Task.Time and Time.Time are unified. It might be the case that there is no such code. Then, core would compile just fine and could be released, but clients couldn't use the modules as intended, because they wouldn't be able to pass Times created with functions from the Time module to functions from the Task module that expect a Time.)
Or (and more likely, since it has always been said that this is coming), at some point something like Haskell's newtype feature is implemented via union types with exactly one constructor in Elm. Then certainly we will want to see type alias Time = Time Float (we should want this already now, for the same type safety reasons that usually motivate newtype in Haskell libraries). And then your argument vanishes. We will then have to put the Time definition in exactly one place. Which, at that time in the future, will force refactoring of more code than it means today.

So I stand with: This should be done. Better now than later.
Relatedly, this was done.

I think your point number 2 is wrong. I want to be able to say (4 * hour) so wrapping things up with type Time = Time Float seems bad given how certain things work now. It'd at least be a major undertaking to get this working nicely again.
I think point 1 is pretty weak. I think we can remember this kind of thing. If we don't remember it, we can fix it really easily.

Now that's a valid point, against type Time = Time Float. I had forgotten that the stuff like hour in Time are just constants, not functions like hours : number -> Time. So yes, if we want to write stuff like (4 * hour), and don't have type classes, wrapping Time as a newtype style thing is not an option.
And yes, as I already granted, the other point about potentially conflicting changes is not a major thing. It's at least likely it won't hurt much.
Anyway, I still think it would make sense to put Time into Basics. Isn't Time meant to be the "standard" type for this in Elm? We don't have to import module String to get Elm's String type into scope (same for lists). So why do we have to import module Time to get Elm's Time type into scope? Ultimately, the current setup tempts people to just use Float instead of Time in their own type signatures even where Time would be appropriate and even if they eventually want to pass those values on to something like Task.sleep. (They are tempted to do that because thus they can save the import Time exposing (Time) line.) Having Time defined in Basics and thus always in scope, would remove that temptation, which would result in better user code.
The fact that we wouldn't have two definitions of Time in core would be a nice side effect.
But feel free to close this issue if you really prefer the current setup.

I think having two is dumb as well, so to get back on track: is the proposal to put them in Basics now? The other options are filtered out?

Yes, I think so.

Let's either open an issue that describes the refined idea clearly or do a PR.

I've prepared such a PR: https://github.com/elm-lang/core/pull/415. (I've checked it locally with an example program that uses both Time.second and Task.sleep as well as an explicit type signature including Time. All I needed to do to make the program work with the revised core as per that pull request was to remove Time from the list in my import Time exposing (Time, ...) line. If I hadn't been using Time.second in that program, I could now have removed the whole import Time ... line.)


774 hs_labels

This new app allows user to assign labels (favorite, mine, label, folder etc) to resources. It is needed for some functionality of the new my resources page that @mauriel133 is working on for AGU release.

Can one of the admins verify this patch?

Will be fixing the failing tests soon.

See https://docs.google.com/document/d/1RHiwpJ1IoyAZzmC6yHykg9vE618M67la5ZZfyKaiDRU/edit?usp=sharing for detailed theory of operation.

@ironfroggy I have addressed most of your concerns now. See if it's ready for +1.

@pkdash since @ironfroggy   handed the +1 on this to you, do you +1 this?  @alvacouch  said time is of the essence. @xuezhaokun

@rayi113 I don't think I should be doing +1.

@ironfroggy can you +1?

+1

@ironfroggy thanks for the expedited +1 on this.


Pagination of UnorderedSet GasLimitExceeded error

Pagination of UnorderedSet implemented for SputnikDao factory as .skip().take() of iter() https://github.com/near-daos/sputnik-dao-contract/commit/f5d4e611b4e08e3df21ad3b32ff048143cd04c02#diff-3dd4449e06b690d9a171047ec7a41be850335de5eaa66aa192a71ac7880c1e24R62 throwing the GasLimitExceeded error
Steps to reproduce:

currently deployed on sputnikv2.testnet

run from index 500, with any limit >0 :

near view sputnikv2.testnet get_daos '{"from_index": 500, "limit": 1}'

required 495+ records in the set to throw an error.

SDK docs are not suggesting any best practice way to paginate  UnorderedSet as per https://github.com/near/sdk-docs/blob/93e2fa29f3f38fc3870d404555cf843b765ac34a/docs/contract-structure/collections.md#pagination-with-persistent-collections

https://github.com/near/near-sdk-rs/blob/master/HELP.md#pagination-with-persistent-collections

Many thanks, this approach is working.

huh, I guess the nth iterator implementation for the old Vector isn't optimized. Looking at the code it seems that it would be and that nth would skip the previous. I might dig into this to see where it loses the skip functionality, maybe because of the map on the iterator.
Note, if anyone in the future is reading this, this only affects v1 of Unordered collections, not near_sdk::store ones

huh, I guess the nth iterator implementation for the old Vector isn't optimized. Looking at the code it seems that it would be and that nth would skip the previous. I might dig into this to see where it loses the skip functionality, maybe because of the map on the iterator.
Note, if anyone in the future is reading this, this only affects v1 of Unordered collections, not near_sdk::store ones

Thanks for your reply and explanation, yes that would be useful for others to know, as I also noticed this implementation elsewhere (NFT I think).
When you say its only affects v1, did you mean that sdk v4+ will not have that issue? Thanks.

Yeah these are just pseudo versions, I just mean all new iterations of the collections are put under near_sdk::store and were introduced in 4.0 with unstable flag. It depends which data structure they use wrt this, but I will try to fix it for the old vector if possible when I have time


Information Technology Director should not be in the calc

https://niteo.co/salary-calculator

Why?

We said we don't put our salaries in the calc, and that's now only for us. It's also not defined in the Handbook.


update go.mod

Pull Request Test Coverage Report for Build 182

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

Totals

Change from base Build 180:
0.0%

Covered Lines:
593

Relevant Lines:
1025

💛  - Coveralls


CFP - update og_image

re: https://github.com/ecommerceberlin/ecommerceberlin.com/issues/206

resolved via 3e5ce87754ed7aa6e774afdba286fc0e171c1469


feat: AWS version

We can extract the plans documented in aws-pipeline for a further discussion, but the rest can go.


[Bug] Couldn't render a slider

Version
5.2.2
Link to Minimal Reproduction
No response
Steps to Reproduce
option = {
        dataset: [
            {
                source: klinesData
            },
            {
                source: ratiosData
            },
            {
                source: closeData
            }
        ],
        title: {
            text: chartTitle,
        },
        tooltip: {
            trigger: "axis",
            axisPointer: {
                type: "cross",
            },
        },
        legend: {},
        xAxis: {
            name: "时间",
            scale: true,
            type: "category"
        },
        yAxis: [
            {
                scale: true,
                name: "多空比",
                position: "left",
                type: "value",
            },
            {
                scale: true,
                name: "K线",
                position: "right",
                type: "value",
            }
        ],
        // 图表放大缩小
        dataZoom: [
            {
                type: "inside",
                start: 94,
                end: 100
            },
            // {
            //     type: "slider",
            //     show: true,
            //     start: 10,
            //     end: 100
            // },
        ],
        series: [
            {
                name: "K线",
                type: "candlestick",
                yAxisIndex: 1,
                datasetIndex: 0,
                itemStyle: {
                    color0: "#ec0000",
                    borderColor0: "#8A0000",
                    color: "#00da3c",
                    borderColor: "#008F28"
                },
                encode: {
                    x: 0,
                    y: [1,4,3,2]
                }
            },
            {
                name: "多空比",
                type: "line",
                yAxisIndex: 0,
                datasetIndex: 1,
                encode: {
                    x: 0,
                    y: 1
                }
            },
            {
                name: "收盘价",
                type: "line",
                yAxisIndex: 1,
                datasetIndex: 2,
                encode: {
                    x: 0,
                    y: 1
                }
            },
        ],
    };

Current Behavior
When I add a dataZoom.slider, it report ”Uncaught TypeError: Cannot read properties of undefined (reading '0')“ at DataStore.js:995.
And if I remove the slider, the chart will show up perfectly.
Expected Behavior
Render a slider.
Environment
- OS: Windows 10 20H2
- Browser: Chrome 96.0.4664.45

Any additional comments?
No response

Well, I swap the position of "K线" series and "多空比" series, and the problem solved.
Still don't understand, but it works now.


Deduplicate retry tests a bit

These 4 are doing almost exactly the same thing, might as well make it more clear.

Pull Request Test Coverage Report for Build 01816e1c-b45e-4e22-b544-398886ae8216

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

Totals

Change from base Build 01816e10-2da6-4d3e-9403-bf3c409f413c:
0.0%

Covered Lines:
12391

Relevant Lines:
19432

💛  - Coveralls


editor.selectionHighlight causing confusing highlighting noise when searching

Issue Type: Bug
I'm using the search panel to search for text across an entire workspace. I have case sensitivty and whole word matching enabled. When I move selection through the results in the panel it opens the file containing the result for previewing. Because this selects the text of the current match, editor.selectionHighlight kicks in and highlights all the occurrances of the same text in the file. This doesn't respect the case sensitivity and whole word settings and ends up adding a ton of visual noise within the newly opened editor.
I'm reasonably certain this has not always happened and is a bug introduced recently. Also, this behavior does not match what happens when searching within a single editor, where spurious highlighting never occurs.
Ideally, selectionHighlight is disabled while the search panel is open. Or at the very least respects search settings so it isn't destroying the user's ability to see actual search results.
Expected (selectionHighlight disabled):

Actual (selectionHighlight enabled):

VS Code version: Code 1.28.2 (7f3ce96ff4729c91352ae6def877e59c561f4850, 2018-10-17T00:23:51.859Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770K CPU @ 3.50GHz (8 x 3492)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.94GB (9.34GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (15)

Extension
Author (truncated)
Version

vscode-npm-script
eg2
0.3.5

tortoise-svn
fan
0.1.1

vsc-space-block-jumper
jmf
1.2.2

Lua
key
0.0.9

theme-monokai-pro-vscode
mon
1.1.8

python
ms-
2018.9.1

cpptools
ms-
0.19.0

csharp
ms-
1.16.2

Go
ms-
0.6.93

PowerShell
ms-
1.9.0

code-settings-sync
Sha
3.2.0

shader
sle
1.1.4

Align
ste
0.2.0

unity-debug
Uni
2.6.6

vscode-todo-highlight
way
1.0.4

(1 theme extensions excluded)

related #8050

The reason this works as expected in the editor find widget is that the selection highlight respects the editor's "match case" setting, which comes from the find widget but also affects cmd+D and the selection highlight.
Maybe we can set that option to match the search view in the editor when opening a search result. But it also makes sense that these two states are separate.

From Rob's comments it seems like this is expected behavior and thus a feature request to make this change. Since it is an old feature request with less than the required number of upvotes, I am closing it. Please let me know if you feel this is not the right course of action. Thanks!

Somewhere along the lines this got fixed.


Support for nu 0.96.1

Hi,
This great plugin doesn't support latest nu 0.96.1.
Thanks in advance.

Thanks @oscarvarto for reporting this, fixed in 0.12.0 :smile:


Change policy to 1 reviewer's approval

It looks like reviewers and commiters are pretty busy and the mandatory 2 approvals rule is significantly slowing down development. Can we change the policy to only require 1 reviewer's approval?
@smoy

I am cool with dropping this from 2 to 1 mandatory reviewer for now. We can re-visit this once we get more community members involved.

I have applied the change: Branch protection rule settings saved.


pluggable approach for request transformation

There are scenarios where running custom transformations before making a http request, ideally test_script is a way to write those out.
Not all transformations can be written in test_script, there should be a way for dothttp to figure out plugin and load those plugin and apply those tranformations easily.
This way users can write their own custom plugins, install and use.

Http item can have a syntax like
@plugin("<pluginid>", ["list of arguements"])
GET "https://httpbin.org/get"

using pluginid, dothttp will load that plugin and apply that transformation.
For example:
GET "https://httpbin.org/get"

hawkauth plugin will apply headers.

For Plugin after researching here is the plan, i think would make sense

leverage similar concept of aws where in, each plugin is python zip (includes all dependencies packaged).
dothttp, can provide a built-in way to install or provide a new installer packages from remote.

installer will store package in centralized locations.
installer will update metadata in specific file '~/.dothttp/installed_packages'

dothttp or dotextensions will load all installed packages beforehand.
plugin is used for request, it will try to load and apply that request transformer.

aws concepts
AWS lambda uses packaged .zip files to load lamdba function defined.
aws sam uses package to package lamdba python function to ease deployment.
ref: https://github.com/aws/aws-sam-cli/blob/develop/samcli/commands/package/command.py

module built via pyinstaller (like dothttp) can load packages from by appending path to packages to sys.path. after that, module can be loaded via importlib.import_module('package').


Logout problem

hey, could You help me with logout option.
$client = $this->clientRegistry->getClient('keycloak');
  $logoutUrl = $client->getOAuth2Provider()->getLogoutUrl();
  $response = $client->getOAuth2Provider()->getRequest('GET',$logoutUrl );

when i click logoutUrl on browser i can logout - but from application not

We are also facing the same issue. The logout is not working as expected and even after calling logout, it is not logging out from Keycloak and thus posing a issue.


DEV-13641 adding force destroy to s3 buckets

Confirmed this worked on both some random buckets and inside of the module. Created the buckets, manually uploaded some files into both, and then ran destroy successfully:
➜  aws-emr-s3 git:(dev-13641) ✗ terraform apply

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_s3_bucket.emr_hbase_logs_s3_bucket will be created
  + resource "aws_s3_bucket" "emr_hbase_logs_s3_bucket" {
      + acceleration_status         = (known after apply)
      + acl                         = "private"
      + arn                         = (known after apply)
      + bucket                      = "tamr-emr-hbase-logs-clara"
      + bucket_domain_name          = (known after apply)
      + bucket_regional_domain_name = (known after apply)
      + force_destroy               = true
      + hosted_zone_id              = (known after apply)
      + id                          = (known after apply)
      + region                      = (known after apply)
      + request_payer               = (known after apply)
      + website_domain              = (known after apply)
      + website_endpoint            = (known after apply)

      + versioning {
          + enabled    = (known after apply)
          + mfa_delete = (known after apply)
        }
    }

  # aws_s3_bucket.emr_hbase_rootdir_s3_bucket will be created
  + resource "aws_s3_bucket" "emr_hbase_rootdir_s3_bucket" {
      + acceleration_status         = (known after apply)
      + acl                         = "private"
      + arn                         = (known after apply)
      + bucket                      = "tamr-emr-hbase-root-dir-clara"
      + bucket_domain_name          = (known after apply)
      + bucket_regional_domain_name = (known after apply)
      + force_destroy               = true
      + hosted_zone_id              = (known after apply)
      + id                          = (known after apply)
      + region                      = (known after apply)
      + request_payer               = (known after apply)
      + website_domain              = (known after apply)
      + website_endpoint            = (known after apply)

      + versioning {
          + enabled    = (known after apply)
          + mfa_delete = (known after apply)
        }
    }

Plan: 2 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

aws_s3_bucket.emr_hbase_logs_s3_bucket: Creating...
aws_s3_bucket.emr_hbase_rootdir_s3_bucket: Creating...
aws_s3_bucket.emr_hbase_logs_s3_bucket: Creation complete after 7s [id=tamr-emr-hbase-logs-clara]
aws_s3_bucket.emr_hbase_rootdir_s3_bucket: Creation complete after 8s [id=tamr-emr-hbase-root-dir-clara]

Apply complete! Resources: 2 added, 0 changed, 0 destroyed.

Outputs:

s3_bucket_name_for_hbase_rootdir = tamr-emr-hbase-root-dir-clara
s3_bucket_name_for_logs = tamr-emr-hbase-logs-clara

➜  aws-emr-s3 git:(dev-13641) ✗ terraform apply
aws_s3_bucket.emr_hbase_logs_s3_bucket: Refreshing state... [id=tamr-emr-hbase-logs-clara]
aws_s3_bucket.emr_hbase_rootdir_s3_bucket: Refreshing state... [id=tamr-emr-hbase-root-dir-clara]

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  - destroy

Terraform will perform the following actions:

  # aws_s3_bucket.emr_hbase_logs_s3_bucket will be destroyed
  - resource "aws_s3_bucket" "emr_hbase_logs_s3_bucket" {
      - acl                         = "private" -> null
      - arn                         = "arn:aws:s3:::tamr-emr-hbase-logs-clara" -> null
      - bucket                      = "tamr-emr-hbase-logs-clara" -> null
      - bucket_domain_name          = "tamr-emr-hbase-logs-clara.s3.amazonaws.com" -> null
      - bucket_regional_domain_name = "tamr-emr-hbase-logs-clara.s3.amazonaws.com" -> null
      - force_destroy               = true -> null
      - hosted_zone_id              = "Z3AQBSTGFYJSTF" -> null
      - id                          = "tamr-emr-hbase-logs-clara" -> null
      - region                      = "us-east-1" -> null
      - request_payer               = "BucketOwner" -> null
      - tags                        = {} -> null

      - versioning {
          - enabled    = false -> null
          - mfa_delete = false -> null
        }
    }

  # aws_s3_bucket.emr_hbase_rootdir_s3_bucket will be destroyed
  - resource "aws_s3_bucket" "emr_hbase_rootdir_s3_bucket" {
      - acl                         = "private" -> null
      - arn                         = "arn:aws:s3:::tamr-emr-hbase-root-dir-clara" -> null
      - bucket                      = "tamr-emr-hbase-root-dir-clara" -> null
      - bucket_domain_name          = "tamr-emr-hbase-root-dir-clara.s3.amazonaws.com" -> null
      - bucket_regional_domain_name = "tamr-emr-hbase-root-dir-clara.s3.amazonaws.com" -> null
      - force_destroy               = true -> null
      - hosted_zone_id              = "Z3AQBSTGFYJSTF" -> null
      - id                          = "tamr-emr-hbase-root-dir-clara" -> null
      - region                      = "us-east-1" -> null
      - request_payer               = "BucketOwner" -> null
      - tags                        = {} -> null

      - versioning {
          - enabled    = false -> null
          - mfa_delete = false -> null
        }
    }

Plan: 0 to add, 0 to change, 2 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

aws_s3_bucket.emr_hbase_rootdir_s3_bucket: Destroying... [id=tamr-emr-hbase-root-dir-clara]
aws_s3_bucket.emr_hbase_logs_s3_bucket: Destroying... [id=tamr-emr-hbase-logs-clara]
aws_s3_bucket.emr_hbase_logs_s3_bucket: Destruction complete after 3s
aws_s3_bucket.emr_hbase_rootdir_s3_bucket: Destruction complete after 3s

Apply complete! Resources: 0 added, 0 changed, 2 destroyed.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Question] Relay token as client certificate

Solution environment is using oauth2 jwt tokens, but the resource services expects client certificate authorization.
That means on the fly JWT needs to be transformed to Client certificate where CN is based on the oid token prop.
That means for every request coming, gateway needs a fresh client certificate.
It's quite easy for singleton scope with HttpClientCustomizer but how to provide a certificate for the http client on each request.

Solution:
Overwrite NettyRoutingFilter.getHttpClient()
method is called each request and has access to ServerWebExchange.
inside we call super that gives us HttpClient instance, then we go for client.secure(sslContextSpec) where we pass SSL setup.
in return we get a new ssl aware HttpClient instance.


Custom resource id field

@Houdini One of my colleagues reached out to you recently with a security issue that was reported on our platform. Mainly - the same remember 2fa cookie value is used throughout multiple sign ins. So here's a quick attempt to resolve the issue when needed by using the custom method to set value of the cookie.
I also noticed that the exact cookie value is not being tested anywhere in any way, and I'd like very much to change that, but am not sure how exactly tackle this. Could you give me some pointers perhaps?
Or if you are already working on a patch like you mentioned in an email for my colleague, when you expect this to be merged?

Hi,
this commit doesn't help to protect from replay attack, it's just the way to refresh cookie each login. All these days I thought about how to do it inside gem, but you move it out of gem's scope. Nice.


Descriptive error messages on bad pipeline formation

If a stage is not fully connected, it returns more descriptive messages, indicating which stage is wrongly connected. E.g.:
pipeline stage "ingest1": pipeline stage from type "ingest" should send data to at least another stage
pipeline stage "decode1": pipeline stage from type "decode" should receive data from at least another stage
pipeline stage "decode1": pipeline stage from type "decode" should send data to at least another stage
pipeline stage "write1": pipeline stage from type "write" should receive data from at least another stage

Codecov Report

Merging #134 (92bd015) into main (4db06d3) will decrease coverage by 0.15%.
The diff coverage is 69.87%.

@@            Coverage Diff             @@
##             main     #134      +/-   ##
==========================================
- Coverage   57.14%   56.98%   -0.16%
==========================================
  Files          49       49
  Lines        2842     2878      +36
==========================================
+ Hits         1624     1640      +16
- Misses       1103     1123      +20
  Partials      115      115

Flag
Coverage Δ

unittests
56.98% <69.87%> (-0.16%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/pipeline/pipeline_builder.go
59.05% <69.87%> (+4.77%)
:arrow_up:

pkg/pipeline/utils/exit.go
52.17% <0.00%> (-47.83%)
:arrow_down:

pkg/pipeline/write/write_loki.go
67.04% <0.00%> (-0.19%)
:arrow_down:

pkg/pipeline/write/write.go
100.00% <0.00%> (ø)

pkg/pipeline/write/write_stdout.go
100.00% <0.00%> (ø)

pkg/confgen/flowlogs2metrics_config.go
0.00% <0.00%> (ø)

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 4db06d3...92bd015. Read the comment docs.


Investigate bad isReplaying value on direct query when workflow is not in cache

Expected Behavior
isReplaying()  is true when replaying and false when not replaying
Actual Behavior
isReplaying()  is false when the workflow is replaying for a query
Steps to Reproduce the Problem
1.Start a workflow that immediately waits for a signal
2.Restart the worker running the above workflow
3.Send a query to the workflow
Specifications

Version: v1.23.0
Platform: mac

Auditing the replay code
    if (replaying
        && !hasNextEvent
        && (event.getEventType() == EventType.EVENT_TYPE_WORKFLOW_TASK_STARTED
            || WorkflowExecutionUtils.isWorkflowTaskClosedEvent(event))) {
      replaying = false;
    }

https://github.com/temporalio/sdk-java/blob/bc726c9ad4646fc5d14e91bcea11190ff7537ca0/temporal-sdk/src/main/java/io/temporal/internal/statemachines/WorkflowStateMachines.java#L427
This check is flawed since if we receive a query after a workflow task that didn't generate a command will cause the Java SDK to incorrectly assume the workflow is no longer replaying.


Light refactor to move things into KubernetesComputeCluster

Changes proposed in this PR

These state atoms are compute-cluster specific. Move them within the compute cluster.

Why are we making these changes?

These state atoms are compute-cluster specific, so can't be global.

build
pr template
internal green

Need to rename the branch (because reasons), so need to respin this push request as #1171


Fix 'odo version' integration test.

/kind bug
What does does this PR do / why we need it:
Correct the regex to match all possible kinds of Server URL
Which issue(s) this PR fixes:
Fixes #3325
How to test changes / Special notes to the reviewer:
make test-generic

/ok-to-test

/assign @amitkrout

/retest

prow failure
/test v4.5-integration-e2e

/lgtm

/approve

/assign adisky
/assign mohammedzee1000

/refresh

/retest

/test all

/test all

@cblecker You just applied test all label, right ? I am asking because if i see same occurrence again, i will do the same without bothering you folks.

/test all

@cblecker You just applied test all label, right ? I am asking because if i see same occurrence again, i will do the same without bothering you folks.

Copy pasting from slack conversation
/retest only retests failed contexts (it got stuck, but prow didn’t see it as failing). Attempted a /test all.
Thanks @cblecker

/test v4.5-integration-e2e


Travis Build Speedup with build_ext

Attempt at speeding up Travis build by explicitly calling python setup.py build_ext to make use of threads as per @cekees suggestion.

Going to close this PR since no speedup was observed in build process. See #1039 for more in-depth discussion if this ever comes up again.


new-log-viewer: Replace existing codebase.

References

Fixes #4 , #8 , #75 .
new-log-viewer series: #45 #46 #48 #51 #52 #53 #54 #55 #56 #59 #60 #61 #62 #63 #66 #67 #68 #69 #70 #71 #72 #73 #74 #76 #77 #78 #79 #80 #81 #82 #83 #84 #89 #91 #93 #94 #96 #98 #100
Description

Remove existing codebase.
Move new-log-viewer code into project root.
Update paths in linting workflow.
Update docs.

Validation performed

Observed linting checks passed.

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Enhanced documentation in the README for improved clarity on functionality and usage.
Updated scripts for linting and analysis capabilities in package.json.
Introduced a comprehensive guide for building the project in docs/dev-guide/building.md.

Chores

Deleted unnecessary files and configurations to streamline the project structure.
Removed obsolete configuration files that could cause conflicts.

Documentation

Expanded README and component documentation to provide better guidance for users.
Added a "Validation" section outlining current manual testing procedures in docs/dev-guide/validation.md.

Final changes. What do you think about moving building.md back into the main README.md. I feel like this is what most people are looking for, so we should keep it out of dev docs?

Discussed offline - we added a paragraph in the project description to show how one can start a local server.


UI fixes in the navbar.

Description
When I hover on the navbar links , the background color makes the text on the navbar links unreadable and also ,it  does not matches with the theme of the website , like on other links it is a yellow background color with white color .
I want to fix this and , also other small navbar issues like the alignment of the logo image . I assure that I would be doing this in 6 hrs after assignment .
Steps to Reproduce

Expected Behavior
I feel that the background color must be light , corresponding to the theme of the site, which would also ,make the text readable .
Actual Behavior
No response
Screenshots
No response
Additional Information
No response
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@pbln sure go ahead

Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr like:
Closes #IssueNumber

even with live server , page looks like this ,

kindly help :(

please help me , so that I can start working on the issue ..


Removing unnecessary import

Few Clean ups

@sahil some clean ups

:+1: nice - no NetworkCallback in BranchRemoteInterface!


any trained result?

Hi, would you mind to provide any trained results?
Or how exactly train the model? I tried "run_experiment.sh", but it seems that the command in it should also incorporate with the path of the input images.
Thanks!

OhOh, I found the default settings for the input image directory.
Again, would you mind to provide some trained results? Thanks!

Were you able to train it?


Update google-auth-library-credentials to 1.10.0

Updates com.google.auth:google-auth-library-credentials from 1.8.1 to 1.10.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.auth", artifactId = "google-auth-library-credentials" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.google.auth", artifactId = "google-auth-library-credentials" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #197.


Add validation for root level items vs tumor level

a root level item (under the NaaccrData tag) should have the same value within the file. When it's not the case, a validation error/warning should be added.

Done and unit tested.


What does PDFDocument's get_dest method do?

Describe the bug
Traceback (most recent call last):
File "D:/Python/untitled1/mainTwo.py", line 11, in
document.get_dest("pm")
File "C:\Users\Lisys\AppData\Local\Programs\Python\Python35\lib\site-packages\pdfminer\pdfdocument.py", line 752, in get_dest
raise PDFDestinationNotFound(name)
pdfminer.pdfdocument.PDFDestinationNotFound: pm
To Reproduce
parse = PDFParser(open("./data/data.pdf", 'rb'))
document = PDFDocument(parse)
document.get_dest("pm")
Expected behavior
I don't understand this method.What can it do?

Solve： https://docs.adobe.com/content/help/en/experience-platform/rtcdp/destinations/destinations-overview.html


Allowed skip struct field in ScanStruct by tag -

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

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

Project coverage is 40.70%. Comparing base (5e60f7c) to head (000e031).

:exclamation:  There is a different number of reports uploaded between BASE (5e60f7c) and HEAD (000e031). Click for more details.
HEAD has 526 uploads less than BASE

Flag
BASE (5e60f7c)
HEAD (000e031)

macOS
27
0

unit
75
1

go-1.21.x
49
0

go-1.22.x
48
1

go-1.23.x
54
0

ubuntu
100
1

windows
24
0

integration
69
0

ydb-23.3
23
0

ydb-24.1
23
0

ydb-24.2
23
0

ydb-nightly
7
0

experiment
7
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1487       +/-   ##
===========================================
- Coverage   66.69%   40.70%   -25.99%
===========================================
  Files         353      350        -3
  Lines       36843    30777     -6066
===========================================
- Hits        24572    12529    -12043
- Misses      11194    17742     +6548
+ Partials     1077      506      -571

Flag
Coverage Δ

experiment
?

go-1.21.x
?

go-1.22.x
40.70% <100.00%> (-23.93%)
:arrow_down:

go-1.23.x
?

integration
?

macOS
?

ubuntu
40.70% <100.00%> (-30.55%)
:arrow_down:

unit
40.70% <100.00%> (-0.37%)
:arrow_down:

windows
?

ydb-23.3
?

ydb-24.1
?

ydb-24.2
?

ydb-nightly
?

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

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


A way (button or similar resource) to enable cursor to move from one monitor to another without blocking their movement.

📝 A button to enable/disable mouse movement block when it moves from one monitor to another, on multimonitor setups, and another button to enable cursor mouse to jump up or down when it's on the top or bottom (or left/right when one monitor is "placed" above or bellow other instead of left or right) of one monitor, and it moves from one monitor to another and the resolution is different than one's first.
The expected behavior of this button is to enable or disable features described above
---Attached to this issue comes a link to a movie I made that explains the problem (since the size exceeds the maximum file size of 10Mb:
https://youtu.be/LC-3-we3-Rw
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@clebermag
add your comments to https://github.com/microsoft/PowerToys/issues/1226


Simplify the register form

Description
The main idea is removed some inputs and make the register form most easy for the users;
Inputs

First name
Last name

Use a simple input like Full name. The backend can keep with the separate columns, but in the register process can parse it.

Username

This input can be removed in the UI, for the Backend can use the email as username, with this idea you can keep the old users.

Need modify the profile form

#48 Completed the profile user


Use app-relative pointer coords when previewing edges

The pointer coordinates used to draw the preview of an edge
(when the edge is being "pulled out" of a node) should be
relative to the DOM element containing the graph editor, rather
than relative to the whole view.
Fixes #206

NB apologies about the extra line ending changes. I can remove these if necessary.

Could you make it use the offsetLeft and offsetTop attributes passed into the-graph-editor element, as discussed here: https://github.com/the-grid/the-graph/pull/190 ... that way apps can also account for scrolling page (or some parent) if needed.

So do I need to modify TheGraph.getOffset() to account for the position of the element relative to the left and top of the page? I can't see this method being called anywhere, though. Or do I need to call that in renderPreviewEdge()?
renderPreviewEdge() in the-graph-graph.js already incorporates the offset passed in via props (this.props.app.state.offsetX/offsetY), but this doesn't include information about the element's position on the page, only the scroll position.

OK, I missed that this issue should be covered by #190. offsetX/offsetY is the responsibility of the encompassing app to change as scroll and element's offset change.

So that means there's no way to automatically adjust drawing edges based on the position of the enclosing <the-graph> element, other than manually tracking that element's position?
It seems a bit unwieldy to rely on the developer setting offsetX/offsetY properties on the element, rather than figuring it out automatically at the point when rendering happens. (The graph's DOM node is accessible to the rendering methods anyway.)

The concern is that a potentially expensive calculation will be happening on every mouse event, slowing the preview edge render.
But it has been a while since I have been working on this.

OK, I think I'll close this and try to find a solution by manually setting the offsets on the-graph. Thanks for the review.


Backend signature verification

Hello,
We're currently trying to implement an authentication workflow using the user's wallet signature.
We've managed to do it in the frontend side, but we're now struggling to implement its verification on the backend side.
Verifying the signature isn't a problem on the frontend since we do have the account instance so we can use account.verifyMessage.
How to do it on the backend side though ? Since we do not have the account instance.
We've thought about instanciating a new signer corresponding to the account on the backend and verifying the signature using this one, but we can't instanciate this signer without the account's public key, that we do not manage to get in the frontend, the account's signer's pubkey being null.
What are we doing wrong ? How can we fix this ?
Thanks!

Hey! Because of account abstraction, signature validation needs to be done onchain!
You can call https://github.com/argentlabs/argent-contracts-starknet/blob/develop/contracts/ArgentAccount.cairo#L486 with the signature from anywhere

@wdev11 I'm closing this, feel free to reopen if you have any more questions following @janek26's comment

Hey! I'm jumping here because I'm stuck implementing the off-chain verification.
Is it still not possible?
FYI This link https://github.com/argentlabs/argent-contracts-starknet/blob/develop/contracts/ArgentAccount.cairo#L486 returns 404.
Thanks!

Hey there!
I'm also looking for a solution to this problem: do you please have any news about it? 🙏🏻
@gianlukk994 I guess the correct page that @janek26 was linking is this.

@Agostin  @gianlukk994  I honestly don't know, but I didn't hear anything change about this in the meantime... so I guess this is how it works (on-chain)

@ivpavici Many thanks for your help here and on Discord!
@Agostin, thanks for the link!


Update results with LLaMa 3 models

Thanks for putting this project together.
Would love to see an update (or regular updates 👀) as models and providers evolve! Would be very useful for comparing vendors.
LLaMa (8B, 70B, and soon 405B) seem like suitable candidates as they're widely supported (eg. Together).
Please let me know how to help (eg. PRs or sponsorship).

Thanks for your interest @tedspare!
There's currently not enough resource from the original maintainers for the project. If you would love to make contribution to keep this up-to-date, it will be great!
If not, I would recommend using https://artificialanalysis.ai/ for a more up-to-date comparisons.


chore: Patch version into binary

Description
Changes proposed in this pull request (what was done and why):

set version in telemetry manager binary

Changes refer to particular issues, PRs or documents:

Traceability

[ ] The PR is linked to a GitHub issue.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] If the change is user-facing, the documentation has been adjusted.
[ ] If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
[ ] The feature is unit-tested.
[ ] The feature is e2e-tested.

depends on https://github.com/kyma-project/test-infra/issues/12224


Profiling keys must be static

We must not create new ones (like is done for example the MeshRenderer) but instead use one static per renderer/profiling code path. Internally in Stride they get added to a static dictionary and therefor cause a memory leak.
Stumbled upon this while memory profiling https://discourse.vvvv.org/t/working-memory-rises-and-rises-memory-leak/19609/7

yea, good point, they are static per name, so that would need a Pooled resource with the name as Key.


Change vars into Arrays

https://github.com/Vel-San/KF-WeaponStatConfig/blob/f144347f12fa44b73c221976a460fdaeb80ebd51/KFWeaponStatConfig/Classes/KFWeaponStatConfig.uc#L10
Reason
We don't need a single int var for each option. Array of ints with fixed size should do the trick and make it way less confusing or cluttered.

Won't Do.


Reward card summary visual refresh

The Chore
Need to rework these with Joelle's designs
Reward cards

[x] All reward cards should show the summary type data with teh button for teh detail ... even if there is only 1 reward behind it (so we have consistent experience)
[x] The button should count how many rewards are behind it and include in the text as shown
[x] The lock icon removed since this is now in the reward detail
[x] Instead the dispatch strategy is now in the top right and has icons for pro rata vs rank

Reward details

[x] Create new page with the sketch shown
[x] The table shows the actual detail from the transfers API and shows

[x] The settlement asset
[x] The market name (looked up from the code)
[x] The start and end epoch
[x] The "Rewards paid"

[x] This is taken from transferInterval
[x] If = 1 then display "Daily"
[x] If > 1 then display "X days"

[x] The lock period
[x] The reward amount
[x] The reward card (from capRewardFeeMultiple)

Additional details / background info
Sketch
https://www.figma.com/design/ywcCZvXdvG28bg0AmCY9mF/Console-v3?t=bOVxi9w1DHzKnGOE-0

Summary cards
We might have to change this count where it says "view rewards on 3 markets" ... because when you click through its actually 2 rewards across 10 total markets.

Maybe just "View reward details"?


Upload shapefile feature

This Pull request includes the Upload Shapefile button within campaign creation. The shapefile must be zipped with the .prj and .shx files

Works really well.  I tried it with a WGS84 zip and a UTM 13N projected shapefile. I'd like to clean up the unused python code that was used for shapefile upload and make sure it doesnt have any side effects found it here:
https://github.com/hotosm/MapCampaigner/blob/59e8c00a1b90f1373e87d62fa9f5d50bcc721ee0/campaign_manager/selected_functions/upload_shapefile.py
https://github.com/hotosm/MapCampaigner/blob/e9f0f977138bc0f526172a6ea1aa771b9106552d/flask_project/campaign_manager/insights_functions/upload_coverage.py
https://github.com/hotosm/MapCampaigner/blob/e9f0f977138bc0f526172a6ea1aa771b9106552d/flask_project/campaign_manager/data_providers/shapefile_provider.py
https://github.com/hotosm/MapCampaigner/blob/6bccc606b75edce3fe10235d057a81606acfd920/flask_project/campaign_manager/views.py#L189


wiki-js: 2.5.285 -> 2.5.286

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for wiki-js is: "A modern and powerful wiki app built on Node.js"
meta.homepage for wiki-js is: "https://js.wiki/"
meta.changelog for wiki-js is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.5.286 with grep in /nix/store/6bjlixq37pfiqcf8h9z7548zq4aiw89r-wiki-js-2.5.286
found 2.5.286 in filename of file in /nix/store/6bjlixq37pfiqcf8h9z7548zq4aiw89r-wiki-js-2.5.286

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wiki-js

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6bjlixq37pfiqcf8h9z7548zq4aiw89r-wiki-js-2.5.286 \
  --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 wiki-js https://github.com/r-ryantm/nixpkgs/archive/8f0df21eb35b2e8ec41bc82556b0da2fdc45fddf.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/6bjlixq37pfiqcf8h9z7548zq4aiw89r-wiki-js-2.5.286
ls -la /nix/store/6bjlixq37pfiqcf8h9z7548zq4aiw89r-wiki-js-2.5.286/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:

wiki-js

Maintainer pings
cc @Ma27 for testing.

I got error on the newest version: error: Package subpath './public/extractFiles' is not defined by "exports" in /nix/store/6bjlixq37pfiqcf8h9z7548zq4aiw89r-wiki-js-2.5.286/node_modules/extract-files/package.json


German Translation in New Teams Client is not correct

Free at "time" is translate with Kostenlos von "time". Kostenlos means for free. It should be "Frei" or "Verfügbar".
Please correct that asap

We are mainly responsible for Microsoft Teams app development related issues.
For Microsoft Teams product issues/failures your tenant Admin can reach out to Microsoft 365 Product Support.
For general questions about Microsoft Teams please post your question on Microsoft Teams Community.

Unfortunately, this is not a Tenant setting and cannot be rectified by Teams Support. This is an incorrect translation within the Teams app and leads to misunderstandings for many companies, as some already think that they will have to pay for teams and calls in the future. It would be good if the translation in Teams, which is used by several million people, was correct.
I have already received requests from several companies that users are unsure and that there will be an increase in inquiries. There seems to be a lack of understanding of the impact this has on the German-speaking market.  So please fix this incorrect translation as soon as possible.

We have raised this question on Microsoft QnA and added required tags. https://learn.microsoft.com/en-us/answers/questions/1419799/german-translation-in-new-teams-client-is-not-corr

Please track the issue in Microsoft QnA forum.
We are closing this issue for now, please feel free to reopen it, if required.


Add EVP demo for X25519 key exchange

This offers both a known answer test with fixed keys and also demonstrates a more realistic usage with random keys.
Fixes #14118.

Good work!
I'd suggest refactoring it slightly though so you do not do everything twice. Just split key generation (or key load) and the key exchange into separate functions and call these from main() for peer1 and peer2. This will make the demo even more real-world.

@t8m How's this?

Style fixes pushed.

More style fixes.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

Merged to both branches.  Thanks.


The default state of the form is incorrect

Form
@Form
@JsonClass(generateAdapter = true)
data class User(
    @Json(name = "userId")
    @MinLength(1)
    val userName: String = "",
    @Json(name = "password")
    @MinLength(1)
    val password: String = ""
)

Compose
Column {
        form(User::class) {
            field(User::userName) {
                TextField(
                    value = state.value?.value.orEmpty(),
                    onValueChange = this::setField
                )
            }
            field(User::password) {
                TextField(
                    value = state.value?.value.orEmpty(),
                    onValueChange = this::setField
                )
            }
            val formResult = this.formState.value
            Button(
                enabled = formResult is FormResult.Success,
                onClick = {

                }
            ) {
                Text(text = "Login")
            }
        }
    }

When I type nothing, the default button state is available

PS: @Preview Not available

I am using 0.3.1,because your project did not build successfully on the jitpack

@bwmgd could you please use the following artifacts? They're on mavenCentral now
val formConductorVersion = "0.4.0"
implementation("me.naingaungluu.formconductor:core:$formConductorVersion")
implementation("me.naingaungluu.formconductor:compose-ui:$formConductorVersion")

On the same note, applying @Preview on form composable is currently problematic since kotlin-reflection isn't available in compose preview's runtime. However, we can achieve this by state hoisting, a technique suggested by google to separate composables from logic.

Thank you very much. Everything is fine now


[Docs] Auto detect instruction isn't given for Minikube platform

Current State:
Docs mentioned that Meshery requires Docker, WSL2 and K3d cluster to start and run the app.
Desired State:
The instructions need to be refined a bit for new contributors and users who use Windows as their primary environment. (WSL2 can be run only on certain PCs and not all)

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Hey @alphaX86 I'm answering this is parts :

Do correct me if I'm wrong here but the K3d cluster is listed as an example (to be used for this particular guide only). However, we can definitely improve the language to visibly mark it as an example and not a requirement.

WSL2 can be run only on certain PCs and not all

To operate correctly on Windows, Kubernetes has to run inside a Linux virtual machine. We went with WSL because it is one of the easier VM's to install and run on Windows and as far as I know (I could be wrong here, please do correct me if I am), WSL can be installed on any Windows system (having windows 7 and above) whereas WSL2 requires windows 10 (version 1903 or above). We have included instructions for both.

Having a docker enabled system would be the ideal way to go as it would make the process easier on Windows but it is not in fact a requirement for running Meshery. Having either Docker or Kubernetes installed would do the job. We recommend installing Docker Desktop for Windows because it sets you up to get started with everything in a single click. I'm not sure we should remove the Docker Desktop installation step here but we could add an alternative step for users who only wish to install Kubernetes in WSL.

Hey @alphaX86 I'm answering this is parts :

Do correct me if I'm wrong here but the K3d cluster is listed as an example (to be used for this particular guide only). However, we can definitely improve the language to visibly mark it as an example and not a requirement.

WSL2 can be run only on certain PCs and not all

To operate correctly on Windows, Kubernetes has to run inside a Linux virtual machine. We went with WSL because it is one of the easier VM's to install and run on Windows and as far as I know (I could be wrong here, please do correct me if I am), WSL can be installed on any Windows system (having windows 7 and above) whereas WSL2 requires windows 10 (version 1903 or above). We have included instructions for both.

Having a docker enabled system would be the ideal way to go as it would make the process easier on Windows but it is not in fact a requirement for running Meshery. Having either Docker or Kubernetes installed would do the job. We recommend installing Docker Desktop for Windows because it sets you up to get started with everything in a single click. I'm not sure we should remove the Docker Desktop installation step here but we could add an alternative step for users who only wish to install Kubernetes in WSL.

@chandrashritii I agree with 2 and 3. But the first point is only the doubt I have. Anyways thanks for clarify. But after reading various topics, I spotted a doc error at minikube section. I'll share the screenshot at the moment....

@chandrashritii I agree with 2 and 3. But the first point is only the doubt I have. Anyways thanks for clarify. But after reading various topics, I spotted a doc error at minikube section. I'll share the screenshot at the moment....

At last part, there is no command for auto detect

At last part, there is no command for auto detect

Missing tag @alphaX86 :) Thank you for pointing it out.

Missing tag @alphaX86 :) Thank you for pointing it out.

Sure!

Sure!

Can we connect this issue with VS Code Atom GNU Emacs for a development environment maybe a "Milestone" is fine

Can we connect this issue with VS Code Atom GNU Emacs for a development environment maybe a "Milestone" is fine

Can we connect this issue with VS Code Atom GNU Emacs for a development environment maybe a "Milestone" is fine

Could you elaborate it @hwpplayer1 ? I'd like to know it

Can we connect this issue with VS Code Atom GNU Emacs for a development environment maybe a "Milestone" is fine

Could you elaborate it @hwpplayer1 ? I'd like to know it

Can we connect this issue with VS Code Atom GNU Emacs for a development environment maybe a "Milestone" is fine

Could you elaborate it @hwpplayer1 ? I'd like to know it

Like extensions both local testing, autocomplete, syntax check, all Editor features

Can we connect this issue with VS Code Atom GNU Emacs for a development environment maybe a "Milestone" is fine

Could you elaborate it @hwpplayer1 ? I'd like to know it

Like extensions both local testing, autocomplete, syntax check, all Editor features