Improve [ERROR MPL-0034] Specified location results in illegal placement. Cannot place macro outside of the core.

Description
untar  https://drive.google.com/file/d/11sRN_Br8SXQBMT1F8_yvJwwnZ8kGNpXM/view?usp=sharing
$ ./run-me-BoomTile-asap7-base.sh
OpenROAD v2.0-15583-g45dd89bd8
Features included (+) or not (-): +Charts +GPU +GUI +Python
This program is licensed under the BSD-3 license. See the LICENSE file for details.
Components of this program may be licensed under more restrictive licenses which must be honored.
core.FpPipeline.fregfile.regfile_ext
[ERROR MPL-0034] Specified location results in illegal placement. Cannot place macro outside of the core.
Error: macros.tcl, 2 MPL-0034
openroad>

Suggested Solution
add coordinates and sizes to error message so that it is clear what exactly is wrong.
Additional Context
No response

@maliberty @AcKoucher There's a bug somewhere reproducible with the above test-case, problems with units...
I tried adding a better error message.
[INFO ORD-0030] Using 16 thread(s).
core.FpPipeline.fregfile.regfile_ext
[ERROR MPL-0034] Cannot place core.FpPipeline.fregfile.regfile_ext at (<PHONE_NUMBER>440 ) ( 1186966 2140728 ), outside of the core ( 1026 1080 ) ( 2024190 2024190 ).
Error: macros.tcl, 2 MPL-0034

The rect is in DBU not microns.

See ClusteringEngine::init() for example:
  logger_->report(
      "Floorplan Outline: ({}, {}) ({}, {}),  Core Outline: ({}, {}) ({}, {})",
      block_->dbuToMicrons(die.xMin()),
      block_->dbuToMicrons(die.yMin()),
      block_->dbuToMicrons(die.xMax()),
      block_->dbuToMicrons(die.yMax()),
...

@maliberty :facepalm:
Much better...
[ERROR MPL-0034] Cannot place core.FpPipeline.fregfile.regfile_ext at (888.598, 1741.44) (1186.966, 2140.728), outside of the core (1.026, 1.08) (2024.19, 2024.19).

FYI you can use 'fixes #' in the commit message to close on merge, see https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue


What service to call for change vacuum mode

Hi, I'm trying to add custom menu for changing vaacuum mode (vacuum, vacuum and mop, mop) to custom vacuum card using this platform for intagrating my viomi.v8 vacuum.
I'm trying to do it via this kind of code but this seems to do nothing even though I don't get an error:
  - menu_id: vacuum_mode
    icon: mdi:robot-vacuum
    label: Vacuum
    tooltip: Vyberte režim vysávania
    tap_action:
      action: call-service
      service: vacuum.send_command
      service_data:
        entity_id: vacuum.viomi_v8_8233_robot_cleaner
        command: set_mode
        params:
          mode: 0
  - menu_id: vacuum_mode
    icon: mdi:robot-vacuum-variant
    label: Vacuum and Mop
    tooltip: Vyberte režim vysávania
    tap_action:
      action: call-service
      service: vacuum.send_command
      service_data:
        entity_id: vacuum.viomi_v8_8233_robot_cleaner
        command: set_mode
        params:
          mode: 1
  - menu_id: vacuum_mode
    icon: mdi:water
    label: Mop
    tooltip: Vyberte režim vysávania
    tap_action:
      action: call-service
      service: vacuum.send_command
      service_data:
        entity_id: vacuum.viomi_v8_8233_robot_cleaner
        command: set_mode
        params:
          mode: 2

Whole YAML is attached.

xiaomi-vacuum-map-card.txt

Different vacuum but it may help you.
`

menu_id: vacuum_mode
icon: mdi:weather-dust
label: Vacuum
conditions:

entity: select.xiaomix_robot_x10_cleaning_mode
value: sweeping
tooltip: Choose mode
tap_action:
action: call-service
service: select.select_option
service_data:
entity_id: select.xiaomix_robot_x10_cleaning_mode
option: sweeping

menu_id: vacuum_mode
icon: mdi:robot-vacuum-variant
label: Vacuum and Mop
conditions:

entity: select.xiaomix_robot_x10_cleaning_mode
value: sweeping_and_mopping
tooltip: Choose mode
tap_action:
action: call-service
service: select.select_option
service_data:
entity_id: select.xiaomix_robot_x10_cleaning_mode
option: sweeping_and_mopping

menu_id: vacuum_mode
icon: mdi:water
label: Mop Only
conditions:

entity: select.xiaomix_robot_x10_cleaning_mode
value: mopping
tooltip: Choose mode
tap_action:
action: call-service
service: select.select_option
service_data:
entity_id: select.xiaomix_robot_x10_cleaning_mode
option: mopping
`

  - menu_id: vacuum_mode
    icon: mdi:robot-vacuum
    label: Vacuum
    tooltip: Vyberte režim vysávania
    tap_action:
      action: call-service
      service: vacuum.send_command
      service_data:
        entity_id: vacuum.viomi_v8_8233_robot_cleaner
        command: set_mode
        params: [0]


Performance(diff): faster rename detection

Fixes iterative/dvc#10515
Notes and questions

I presume the reason for sorting added and deleted before was to optimize the runtime (more likely to get an early hit in the inner loop). I included this now maintain the former output. In the case of multiple files with the same hash, changing this may result in other pairs being detected as renames. However, if maintaining this is not a priority, I think we can remove it, making it simpler. It will still be deterministic.
I used asserts for verifying change.new and change.old, as this seemed to be the convention. LMK. if I should change for exceptions, or if we should just assume _diff to always supply valid changes.
I'm only just aware of the https://github.com/iterative/dvc-bench repo. Should we add a test case to it that would have uncovered iterative/dvc#10515?
It is not entirely clear to me why HashInfo has unsafe_hash=True, however, as far as I understant, it should not affect this implementation.

Benchmark
Consider a similar setup as the minimal reproduction in iterative/dvc#10515.
We make three versions of our dataset:

Tag1: mkdir data; for i in {1..10000}; do echo $i > data/file_$i; done
Tag2: mkdir data; for i in {1..10000}; do echo $i > data/file_$i.ext; done
Tag3: mkdir data; for i in {1..10000}; do echo $((i + 20000)) > data/file_$i.ext; done
From Tag1 -> Tag2, we rename the files, without changing the content. From Tag2-> Tag3, we change the content of the files.

Run dvc diff <from> <to>.

Change
Old
New

Tag1 -> Tag2
0.49 s
0.50 s

Tag2 -> Tag3
0.46 s
0.46 s

Tag1 -> Tag3
14.6 s
0.56 s

Ps. Results show timings from only one experiment.

@Northo, can you share benchmarks or profiling information? Also, can you please add a description?

@skshetry, sorry. Inteded to make a draft pull request only, at this time. Will supply more detailed inormation soon. For now, see iterative/dvc#10515.

@skshetry, one benchmark test failed with regression. I cannot see that it is related to these changes, and also see that it first failed and then suceeded on #552. Have I made a mistake, or is it just flaky?

@skshetry, one benchmark test failed with regression. I cannot see that it is related to these changes, and also see that it first failed and then suceeded on #552. Have I made a mistake, or is it just flaky?

yeah, it's flaky. Please ignore.

Ok, thanks! In that case, consider my PR open for review:)

I tried your script above (but created 100,000 files instead of 10,000). But I did not see any meaningful changes, and in fact it was slightly slower than before.
Since we are using an iterator, I used the following patch to isolate _detect_renames.
diff --git a/src/dvc_data/index/diff.py b/src/dvc_data/index/diff.py
index 40fd5eb..491dfcf 100644
--- a/src/dvc_data/index/diff.py
+++ b/src/dvc_data/index/diff.py
@@ -5,6 +5,7 @@ from typing import TYPE_CHECKING, Any, Callable, Optional, cast

 from attrs import define
 from fsspec.callbacks import DEFAULT_CALLBACK, Callback
+import funcy

 if TYPE_CHECKING:
     from dvc_data.hashfile.hash_info import HashInfo
@@ -328,6 +329,9 @@ def diff(  # noqa: PLR0913

     if with_renames and old is not None and new is not None:
         assert not meta_only
-        yield from _detect_renames(changes)
+        changes_list = list(changes)
+        with funcy.print_durations("Detecting renames"):
+            changes = list(_detect_renames(changes_list))
+        yield from changes
     else:
         yield from changes

EDIT: Sorry, I was not deleting the directories between each stages. I see the improvements now. :)

Happy to hear that! I was struggling trying to figure out why it could not be reproduced;)
Thanks for the well-formed reproduction script!

Codecov Report
Attention: Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.

Project coverage is 70.99%. Comparing base (081a1d8) to head (9e53ce5).
Report is 33 commits behind head on main.

Files
Patch %
Lines

src/dvc_data/index/diff.py
88.23%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #550      +/-   ##
==========================================
+ Coverage   62.98%   70.99%   +8.00%
==========================================
  Files          62       67       +5
  Lines        4342     4941     +599
  Branches      740      829      +89
==========================================
+ Hits         2735     3508     +773
+ Misses       1448     1223     -225
- Partials      159      210      +51

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

Thank you for contributing, and making an amazing improvement in performance to dvc. 🙂

Thanks for really swift follow-up on the PR @skshetry !


Finish writing stage damage details file

The starting point is AggregatedStageDamageElement.ExportDetails. I wrote several comments outlining how i think this should go. I have a file writing out. I show how to pull some of the desired info from the cache. I create a datatable and write some values out. It basically just needs the correct details added to the datatable. Talk to Cody if you need any help getting the data you need.

@Brennan1994 I don't see this work in the commit. We now have the capability to write the equivalent of the FDA_StructDetail.out file?


Flaky bckn_electrs

We recently enabled bckn_electrs to use the electrum backend, but it's causing flaky tests in hourly runs.
https://github.com/fedimint/fedimint/actions/runs/10538652163/job/29201381921#step:7:1708
https://github.com/fedimint/fedimint/actions/runs/10534704117/job/29192736976#step:7:4775
A quick glance seems like it's taking too long to complete
fedimint-test-all-wasm32-unknown-unknown-ci> 00:01:20      TIMEOUT [  60.059s] fedimint-wallet-tests::fedimint_wallet_tests on_chain_peg_in_and_peg_out_happy_case

https://github.com/fedimint/fedimint/actions/runs/10534704117/job/29192736976#step:7:4294

Haven't seen it in a while.


Notebook explaining how to determine ip type used by site, and how to…

… pull docker image from ipv4 registry vs ipv6 registry.

Handled by ./fabric_examples/fablib_api/docker_containers/docker_containers.ipynb


Update getting-system-information.md

Added "const qualifiers" in 3 places.
"string" is also called as "character constants", "string constant" and "anonymous string" in C++. This is because a "string"  in the C++ standard has the "const qualifier".
But the situation in C is slightly different. Since the "const keyword" did not exist in C until the "ANSI C 1989" standard,  "string literals" dont have a "const qualifier" for such historical reasons. Hence "non-const string" in C is evaluated as  "undefined behavior".
"string literals", i.e. the elements of the "envVarStrings" array are located in memory in the ".rodata section" of the "data segment", but the "pointers" of this array even though are in the ".rwdata section", they "point" to the "first  character addresses", i.e. "base addresses" of "strings" located in the "read-only section" and store these addresses. That is why must be added the "const character" indicating that they are "pointing" to "const qualifier".  Although the "printError" function has a "non-const" parameter - void printError(TCHAR* msg); - , the program string attempts to pass a "const" argument to this function. - "printError( TEXT("GetComputerName") );" To eliminate the problem the "printError" function parameter must have the "const qualifier".

This was difficult to do. With all the merge conflicts, it was essentially a case of studying piecemeal changes to the code spread across 5 PRs (with 5 incrementally changed versions of the code) and trying to merge them all down into one version that I hope captured your intent. And I had to do some merges twice, for some reason (UI said "out of date"). So yeah, this was tricky, and I can't be certain I've not made errors with all of the merges.
It'd be great, for any future changes, to do them in 1 PR, or else wait until a PR has completed, and the updates have gone into the repo, before doing another PR on the same section of code.
Thanks again!
-Steve

Yes, You are right. It will be done. I already gathered them together for some PRs in the future. I have looked at official page where is full program codes loacted and noticed that there are some shortcomings in the code sections again. Maybe it is related with merge conflicts as you mentioned. Since several repetitions of same lines present in the program code, that is why I fixed all without any shortages together. Please notify me if you can change them yourself or I can make new PR. I note all of them here too.
These lines should be removed:

"TCHAR infoBuf[INFO_BUFFER_SIZE];" line in main function
2nd "TCHAR sysMsg[MAX_PATH] = {'\0'};" line in printError function

These lines should be changed:

parameter 256 to MAX_PATH in FormatMessage function

These lines should be added:

TCHAR* p = sysMsg; line after "TCHAR sysMsg[MAX_PATH] = {'\0'};" in printError function
"p = nullptr;" to the last line of printError function

Thanks in advance)


[WIP] Virtual Kubelet Node Controller 2/6

Description
This PR is the second related to the NodeController refactoring.
What has been done:

move and NodeFromProvider functions from /cmd to /pkg/...

What has to be done yet:

move NodeErrorHandler from cmd to the NodeController.

address #474

/ok-to-test

/rebase

/approve

/merge


simple js file

Issue
Hello. Not an issue..
What are the chances of making a simple .js file available?  I've tried building this myself and have had many various issues as I don't know what I'm doing. I understand there are .js files via the CDN links in the readme file. I'm hoping for non-minimized, non-obfuscated versions so I can make changes.
thanks

No longer needed for my specific use-case, but still nice to have (and for next time). I'm also using version 1.3.3 and I'm stuck there.

No longer needed for my specific use-case, but still nice to have (and for next time). I'm also using version 1.3.3 and I'm stuck there.

Why don't you use the latest version?
Anyway, im rebuilded the code without minification https://gist.github.com/crashmax-dev/0e686a9f790bd59384272421d9d5c64f

Thank you!  I couldn't find the latest version non-minified and I've made changes to the version I have that I'll want to make to the new version also.  So I had the old version because modifying a minified version is a pain.
I guess you'll ask me what changes I've made. So here they are.

pass in the number of fireworks that will go off. eg fireworks.start(1) sets off just 1 and stops. fireworks.start(10) sets off 10 and then stops.
use my own function to play audio so I can adjust the volume and follow whether it's playing or not.
wait to load the audioContext until initTrace() so there's no warning about missing user interaction in Firefox.

pass in the number of fireworks that will go off. eg fireworks.start(1) sets off just 1 and stops. fireworks.start(10) sets off 10 and then stops.

https://github.com/crashmax-dev/fireworks-js#launchcount
You don't want to create a pr merge?


Fixed plutus-core-spec some typos

I think we can safely merge this.  I've got another PR on the same document in progress, so I might fix the "behavior/behaviour" thing there.


Very simple proof-of-concept packaging of binary

Codecov Report

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

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   38.46%
=======================================
  Files           ?        3
  Lines           ?       26
  Branches        ?       18
=======================================
  Hits            ?       10
  Misses          ?        6
  Partials        ?       10

Flag
Coverage Δ

Linux
38.46% <0.00%> (?)

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

Continue to review full report at Codecov.

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


Source seems to have a problem but image is showing

Challenge Make Images Mobile Responsive has an issue.
User Agent is: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:47.0) Gecko/20100101 Firefox/47.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

  <form action="/submit-cat-photo">
    <input type="radio" name="indoor-outdoor"> Indoor
    <input type="radio" name="indoor-outdoor"> Outdoor
    <input type="checkbox" name="personality"> Loving
    <input type="checkbox" name="personality"> Lazy
    <input type="checkbox" name="personality"> Crazy
    <input type="text" placeholder="cat photo URL" required>
    <button type="submit">Submit</button>
  </form>

@knoahg thanks for the issue. This test will be merged with other tests and updated when the next site update happens. For the time-being, it will work if you remove the space at the end of your URL. Consider asking for help in the Help Chat room for FreeCodeCamp.

looks like url https://bit.ly/fcc-running-cat is broken.
i go to url and get this message
"The link you requested has been identified by bitly as being potentially problematic. This could be because a bitly user has reported a problem, a black-list service reported a problem, because the link has been shortened more than once, or because we have detected potentially malicious content. This may be a problem because:"
thx

the valid url is https://bit.ly/fcc-running-cats


typescript: Update all non-major dependencies

This PR contains the following updates:

Package
Type
Update
Change

@types/node (source)
devDependencies
patch
20.14.12 -> 20.14.13

@types/node (source)
dependencies
patch
20.14.12 -> 20.14.13

GDataCyberDefense.Vaas
nuget
minor
7.2.0 -> 7.3.0

chai (source)
devDependencies
minor
4.4.1 -> 4.5.0

gdata-vaas
dependencies
patch
7.1.8 -> 7.1.9

de.gdata:vaas
dependencies
minor
8-future -> 8.2.5

:warning: Warning
Some dependencies could not be looked up. Check the warning logs for more information.

Release Notes

chaijs/chai (chai)
v4.5.0
Compare Source

Update type detect (#1631)  1a36d35

What's Changed

Update type detect by @koddsson in https://github.com/chaijs/chai/pull/1631

Full Changelog: https://github.com/chaijs/chai/compare/v4.4.1...v4.5.0

GDATASoftwareAG/vaas (gdata-vaas)
v7.1.9
Compare Source

Configuration
:date: Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
:ghost: Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
:warning: Warning: custom changes will be lost.

Autoclosing Skipped
This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.


feat(gql-tag-operations-preset): Allow useFragment to take arrays and nullable values as arguments

Description
Related #7808, #7809

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Further comments
I considered adding configuration values such as fragmentMasking.allowList and fragmentMasking.allowNullable. However, I decided to accept list and nullable by default instead of adding the settings for the following reasons:

Being able to mask a list is always a necessary feature
There is no disadvantage in accepting nullable if the argument is non-nullable and the return type is also non-nullable.

Thank you for your code review 😻
I have updated this PR ✅

1a6920f Update examples
6cc9b84 Add testcase


Rest -> Example 1 not working from data flows

Unable to set pagination variables in QueryParameters in data flows as described in Example 1: Variables in QueryParameters. Documentation provides current example:

which i adopted to my needs like this:

This works correctly with Copy data activity, but unfortunately does not work for me in data flows. When fetching Data preview i get the following error:

Am i doing something wrong here?

Dokumentinformasjon
⚠ Ikke rediger denne delen. Den kreves for koblingen docs.microsoft.com ➟ GitHub-problem.

ID: c9da1816-3f64-bd45-443f-5bdb48bacc24
Version Independent ID: e649f787-62da-fc97-2e9a-ba89765beed4
Content: Copy and transform data from and to a REST endpoint - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/connector-rest.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @jianleishen
Microsoft Alias: makromer

Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

Hello @axcap,
You seem to be using the parameters incorrectly in the data flow, causing this error. Please look into this video below and document and see if it helps.
https://www.youtube.com/watch?v=vpuuQcFojt8&ab_channel=AzureDataFactory
https://docs.microsoft.com/en-us/azure/data-factory/parameters-data-flow
If you have any further questions, please let us know.

Hello @BhargavaGunnam-MSFT and thank you for your reply.
I believe there was a misunderstanding, i'm not using data flow parameters. The QueryParameters i'm refering to are for PaginationRules in Rest connector. I managed to get it working in Copy activity, but not inside my data flow using the same values for Rest connector. That's why i was wondering if the code for Rest connector differs from activity vs data flow even though the share the same documentation.

Hello @axcap,
Thank you for more details.
Example 1: Variables in QueryParameters.
If you are following the example mentioned here, this is for a copy activity in the pipeline. Also in the document, it is mentioned that -The pagination rules in mapping data flows are different from it in copy activity:
So you can't use the same syntax mentioned in the document for the mapping dataflow(this is only applicable for copy activity in the pipeline)

I see there are two sections in the mapping dataflow

query parameters
Pagination rules.

I believe the way you are passing the values here is having issues.
It would be easy to troubleshoot in a screen-sharing session. For deeper investigation and immediate assistance, If you have a support plan, you may file a support ticket; else, please let me know so that we can enable one-time-free support for you to work closely on this matter.

@axcap  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@BhargavaGunnam-MSFT
Looks like i got it to work, but i believe there is a bug.
Given following API with continuationToken query parameter i have to add an extra letter (underscore in this example) in data flow definition as show on next image. Can you confirm?

Hi @axcap,
Thank you for letting us know. I will check and get back to you.

@BhargavaGunnam-MSFT any updates on this?

Hi @axcap,
Sorry, I am still checking with my internal team on this. I will get back to you at the earliest.

Hi @axcap .  So if I understand correctly, the issue now is you need to add a single character, ANY character to the beginning of "continuationToken" where "continuationToken" is a key in your query parameters?
Have you tried testing this same call using anything other than Data Factory?  There is the possibility there is a bug in the server, rather than the Data Factory.  I notice from your screenshot it appears you are developing some service of your own.
To determine whether the fault is in the continuationTokenAPI you are building, or in Data Factory, it is necessary to try with a different tool.
  With your current screwdriver no matter how much you turn, the screw hasn't come out.  The problem could be the hole's threads are stripped, or the problem could be the screwdriver's dull point not catching the screwhead.  You can't exactly swap out the screw, since it is stuck in the hole, but you can certainly try with a different screwdriver.  If a new sharper screwdriver cannot extract the screw, then we know the problem is in the hole.
Another option would be trying the same screwdriver on a different screw & hole combination.
So I'd like you to try with Postman or curl, or any number of other tools.

Hi @MartinJaffer-MSFT, i have tried with Thunder Client and it works just as expected.
Here is my demo setup:

A simple minimal API in .Net:

A source configured as follow:

And here is the Data preview:

As you can see, it eats up the first letter in the query parameter.
Here is the same API called from Thunder Client:

Alright.   You have convinced me @axcap .  Since it doesn't happen when I do it, but you clearly can reliably reproduce the bug, I'd like to give you a support ticket so our engineers can dig into the backend and find out what's causing this.
If you do not have a support plan, email us at<EMAIL_ADDRESS>.
For subject: attn MartinJaffer-MSFT
In body please include your subscription id, and this thread url.

Hello @axcap ,
Please let us know if you need any help with the support request.

@BhargavaGunnam-MSFT, @MartinJaffer-MSFT here is the support ticket id:<PHONE_NUMBER>000304.

Hi @axcap,
Thanks for raising the support ticket. We will track this issue via the support request that you have submitted. Hence we are closing this issue from my end.


vcloud-login returns extra carriage return character

vcloud-login 1.1.0 returns an additional carriage return before the export command, this causes issues when scripted.
cannot find command /rexport

Thanks @jambo2kuk. We've had other reports of this and indeed I've seen it myself. The workaround for now is to avoid the eval, e.g.:
FOG_CREDENTIAL=xyz vcloud-login

You should take care however if copying and pasting the resulting export line to avoid it being saved in your shell history - in Bash you can insert a space before the line to prevent it from being saved to your Bash history, for example.
We'll look into this as soon as we can.

Thank you for your interest in this project but we now consider it
both feature complete for our own usage and in maintenance mode.
While we will not be investing any more development time in this
project we may still include pull requests.
Hopefully the work around shown by @mattbostock is sufficient for your usage.


Hpatches

Hello，I am learning your code.  I didn't find your code about table 2(Comparison to the state of the art on HPatches). I hope to get your code and data set about this part.   thank you very much!!!

Hi,
The code for HPatches evaluation is indeed not online for now. Before releasing it, I would need to clean it and to re-run some methods and this will take some time. Unfortunately, I don't think I will have time to do it before the CVPR deadline in a month...
But otherwise you can find the HPatches dataset here: https://github.com/hpatches/hpatches-dataset at least. I'll try to push the dataloader for this dataset after the CVPR deadline.

OK , Thank you very much！！！

Hi,
I finally found some time to add the processing and evaluation of the HPatches dataset. You can find the data loader in lisrd/datasets/hpatches.py and the evaluation notebook in notebooks/hpatches_evaluation.ipynb. Note that the numbers in this notebook do not correspond exactly to the ones in the original papers, as they were obtained with SuperPoint keypoints and a different training.
Let me know if you encounter issue with it!

Hello, When I am running your evaluation notebook in notebooks/hpatches_evaluation.ipynb, I get the error: No such file or directory: '/home/victor/LISRD/Documents/datasets/Hpatches_sequences/i_nijmegen/1.ppm.kornia_sift'. Where can I get these pictures?

Hello, When I am running your evaluation notebook in notebooks/hpatches_evaluation.ipynb, I get the error: No such file or directory: '/home/victor/LISRD/Documents/datasets/Hpatches_sequences/i_nijmegen/1.ppm.kornia_sift'. Where can I get these pictures?

Hi,
Please find the answer in your other issue https://github.com/rpautrat/LISRD/issues/7#issuecomment-757790550.

Hi,
Please find the answer in your other issue https://github.com/rpautrat/LISRD/issues/7#issuecomment-757790550.


Add table with currently available NN models in the demo script

This PR introduces the following table

Also, a mention about deeplabv3p_person network has been added

Thanks, both!


Batch download/upload support

Hello,
Since we have batch delete supported (delete_blobs on ContainerClient), have you considered adding batch download/upload support to the library?
If yes, when might we expect it?

Thank you for the feedback @Laleee . Ill route this to the proper folks on the storage sdk.

Hi @Laleee Lazar, I have spoken with the team, and it sounds like this is very low on the priority list (and even possibly likely to never be implemented in the near future). With that being said, I have noted that there is at least interest in the feature, and if this feature becomes something that more and more users would like, it may move up in the priority list.
With that being said, I will close out the issue.
Thanks!


[$injector:unpr] Unknown provider: $uibModalInstanceProvider <- $uibModalInstance when injecting $uibModalInstance

Bug description:
Getting below error, when trying to inject $uibModalInstance in the controller of an angular Component to close/dismiss the modal window. I am using uibModal as a ui router state using onEnter hook of router.

Tried calling $close/$dismiss methods on $scope as described here, but these methods aren't available on scope.
Something mentioned here (http://angular-ui.github.io/bootstrap/#/modal) in javascript source which states:

// Please note that $uibModalInstance represents a modal window (instance) dependency.
// It is not the same as the $uibModal service used above.

Note: If I am using template with controller (i.e not using component), I can inject $uibModalInstance into controller without error.
Is there any way to close/dismiss modal window without having modalInstance we have as a result of calling $modal.open method when using component?
Link to minimally-working plunker that reproduces the issue:
Version of Angular, UIBS, and Bootstrap
Angular:
"angular": "1.5.8",
UIBS:
"angular-ui-bootstrap": "2.2.0",
Bootstrap:
"bootstrap": "3.3.7",

$uibModalInstance is a local, not a normal injectable.
Please reproduce this in Plunker - this is highly likely a user error.

Sorry wasn't able to provide example as I was still figuring out the meaning of:

$uibModalInstance is a local, not a normal injectable.

Also, when I modiefied the modal example from ui website > only modal dialog
I found this hidden Gem:

// Please note that the close and dismiss bindings are from $uibModalInstance.

And that works for me as my whole application is tree of components and using the same in modal makes sense.
Thanks @wesleycho

$uibModalInstance is a local, not a normal injectable.
What does it mean, @wesleycho and what should we do in that case ?


Update the c8y mapper to declare child capabilities

In order to support configuration management for child devices but any other child device operation, the Cumulocity mapper must be extended to notify Cumulocity of any change of the supported operation list for a child device.
As a first step, one assumes that:

a Cumulocity supported operation for a child device is declared on the thin-edge device by

an empty file named after the Cumulocity name for the operation (say c8y_DownloadConfigFile or c8y_UploadConfigFile)
stored in a directory named $TEDGE_CONFIG_DIR/operations/c8y/$CHILD_DEVICE_ID/ where $CHILD_DEVICE_ID is the name for the child device.

The capability list for a child device $CHILD_DEVICE_ID  is the set of names of all these files under `$TEDGE_CONFIG_DIR/operations/c8y/$CHILD_DEVICE_ID/.
a more flexible mechanism to declare a supported operation for the child device from the network is under specification.

The expected behavior for the tedge_mapper c8y is to:

notify Cumulocity of the supported operations not only for the main device but also for all the child-devices

A 114 smartrest  message has to be sent to c8y/s/us with the comma separated list of the main device supported operations
A 114 smartrest  message has to be sent to c8y/s/us/$CHILD_DEVICE_ID with the comma separated list of the $CHILD_DEVICE_ID supported operations

send all these messages on start
send a message for the updated list on each change (when a capability is added or removed for a device)

I would really like to avoid here the term "capabilities", but call it instead "supported operations". As I can see that's also what C8Y doc names it: https://cumulocity.com/guides/reference/smartrest-two/#114
Also existing thin-edge doc does use the term "supported operation": https://github.com/thin-edge/thin-edge.io/blob/main/docs/src/tutorials/supported_operations.md

I would really like to avoid here the term "capabilities", but call it instead "supported operations".

I updated the description accordingly.

Note:

The child device directory and operation files can be created dynamically, so in the runtime of mapper. Need to support such usecase.

Resolved with #1474

Steps to test:
Install thin-edge from https://github.com/thin-edge/thin-edge.io/actions/runs/3243471923
Test 1:
Create a child config directory in /etc/tedge/operations/c8y/child_optest
sudo tedge connect c8y
Check if the child_optest is created on the c8y web UI.
Test 2:
sudo touch /etc/tedge/operations/c8y/child_optest/c8y_DownloadConfigFile
Now check in the c8y cloud under the child device child_optest configuration is visible
Test 3:
sudo rm /etc/tedge/operations/c8y/child_optest/c8y_DownloadConfigFile
Now check in the c8y cloud under the child device child_optest configuration  should not be visible

QA check passed, therefore close this issue.


function_exists requires list_functions permission

The current implementation of function_exists requires the calling account to have list_functions permissions. It also iterates through the functions. A better implementation would be to use the get_function function.

@bonghodrum


Use Isolates (compute()) for JSON De-serialization

The library currently is causing jank due to deserialization on the main thread. This can be pushed to an isolate without changing the library's apis as mentioned here - https://flutter.dev/docs/cookbook/networking/background-parsing

@awanishraj
Please have  a try https://github.com/flutterchina/dio_flutter_transformer

@trevorwang but library generates code, which runs in main thread!
As example
@override
  sites(vanId, start, length) async {
    ArgumentError.checkNotNull(start, 'start');
    ArgumentError.checkNotNull(length, 'length');
    const _extra = <String, dynamic>{};
    final queryParameters = <String, dynamic>{
      'start': start,
      'length': length
    };
    final _data = <String, dynamic>{};
    final Response<List<dynamic>> _result = await _dio.request('/Site',
        queryParameters: queryParameters,
        options: RequestOptions(
            method: 'GET',
            headers: <String, dynamic>{},
            extra: _extra,
            baseUrl: baseUrl),
        data: _data);
    var value = _result.data
        .map((dynamic i) => SiteModel.fromJson(i as Map<String, dynamic>))
        .toList();
    return Future.value(value);

Okay, we see that request could be in separate process, but other code, runs in main thread and cause UI freezes

This is critical and leads to serious ui jank as soon as the returned json gets too big.
The solution is pretty simple:

Wrap everything related to converting the response to the result type in a static method on the generated class with the following signature:

static <ResultType> _ResultType_fromResponse(Map<String, dynamic> json) {
// Do whatever is necessary to convert the response
return ResultType.fromJson(json);
}

After awaiting the response, instead of directly converting the response to the target type, simply call the static method from above wrapped in compute:

final Response<Map<String, dynamic>> _result = await _dio.request(
    '/apicall',
    queryParameters: queryParameters,
    options: RequestOptions(
        method: 'GET',
        headers: <String, dynamic>{'authorization': authorization},
        extra: _extra,
        baseUrl: baseUrl),
    data: _data);
final value = await compute(_ResultType_fromResponse, _result.data);
return Future.value(value);

The only problem with this is that compute is part of the flutter sdk and not part of dart itself. To combat this, we can write our own compute method that simply uses Isolate.spawn under the hood.
I tested this by manually modifying the generated code, but unfortunately I don't know how to modify the generator to emit the additional static methods.
Launching a new isolate takes ~25ms on a Pixel 2 in Profile Mode. This delay can be completely removed by instantiating the isolate on the first request and caching it afterwards. However, users of the generated class would then have to dispose() the class to clean up the cached isolate. For now, I guess the 25ms hit is acceptable, since a network request usually takes longer anyways, but we might want to keep this in mind as a future optimization.

I created a pull request with a first draft for this in #94, any feedback would be great.

Are there any plans to address or reopen this issue?

I guess not. So Retrofit should be avoided with Flutter, because it's useless here.

This issue is the only thing that's stopping me from using retrofit. Even if there's no solution currently, it would be nice to have a workaround to bypass this issue temporarily.

Yes, my case is the same. Dev should reconsider to add support for it. Otherwise, make it not compatible with Flutter, as it useless with Flutter. Note that 90% of Dart development is Flutter, so that's not wise to drop Flutter support.

@giaur500 It really is not as big of a deal as you are making it out to be for most developers. It's a nice to have, but not the make or break. I tried with isolates myself and found that it makes all api calls much slower. So, I would suggest you do some experimentation by yourself. The lib might not suit you, but it's definitely useful for majority of flutter developers. No need to give the dev a hard time on this.

@giaur500  I agree with @awanishraj.    Currently, we don't have a nice workaround for this.

@giaur500 I really agree with @awanishraj .
Currently, we don't have a nice workaround for this issue.  And I would suggest that have a try in another way to check whether it really caused a performance issue.
I was considering a big refactor for this library, but I don't have much time these days.  Trying to cover this issue in the next big release.

Do you mean compute/isolates is does not make any noticeable difference? That's recommended way to json deserialize in Flutter. Of course, I can also don't care about that and deserialize json in main thread, but that's not recommended, as on slow device it may cause lags.

The early commits of this project https://github.com/frencojobs/scrapbook can be seen to be using retrofit. But since the API I was consuming was too large and not paginated, the UI lag/junk gets super annoying on my low-end device. If I am not mistaken, the junk lasts maybe up to 10s, which is a lot. So, I rewrote all the API codes as plain functions and made it utilize the compute function, and the results are far superior. Isolates may be making all the API calls a bit slower, but there won't be at least an annoying UI junk.
But I can't rush an opensource project which I am using without paying incentive. Everyone's having a hard time currently. I could understand that. I'm just putting two cents in for a project I really love.

Use isolates/compute please, they really do help according to my experience
Btw, the generated code is not formatted according to pedantic, so it's causing lots of warnings

Looking forward to the big release : )

any new with this issue ?

Can you see any new comments from devs? If not, no news. No need to ask.

@giaur500  A new question for the issue prevents the bot from closing it and gives the issue importance to the author. Is it necessary to be rude? If yes then you can ignore the question. If no, you are welcome to answer

Ok fine. But I think author knows about importance, but still it has low priority. Simply, don't use Retrofit, as it's useless with Flutter.

Would the dio flutter transformer work with retrofit to deserialize in an isolate?
https://github.com/flutterchina/dio_flutter_transformer

No. The library still generates code to serialize the response, which runs in the main thread. Something like this,
final value = ResponseModel<OtpModel, Null>.fromJson(_result.data);
// ☝ this still runs in the main thread
return value;

Yes, exactly.   Currently, I haven't found a way to use one method to handle the type converting due to generic types.
Any ideas or PRs are appreciated

I think retrofit is doing smth weird - just got the same problem with the long API response - huge JSON array, 4500 items, 20 to 50 fields each.
final MyResponseObject res = await myRetrofit.fetchData() takes:

5 secs for the data transmission - UI is ok
15 secs for smth (thought it was json parsing but read next) - UI is completely frozen

now, without going to isolates, just made the same call using the standard http.Client() to fetch the json as plain string

5 secs for the data transmission - UI is ok
157 ms for MyResponseObject.fromJson(...) !!!  Note, I'm calling the same fromJson factory retrofit does.

So, despite the JSON payload is huge, json is being parsed and deserialized very fast. I might even skip doing this in isolate (as this is a web app and no active UI interactions like scrolling, etc are made during this call)
Even if I go back to retrofit and adjust method to return plain string - from Future fetchData() to Future fetchData() - I see the same 15 secs freeze.
@trevorwang any idea what is causing this?

@livotov do you have a minimal self-contained example we can use to identify the root cause?

@NicolaVerbeeck I think I can create one - this endpoint is public and contains no sensitive data.

Sorry, but I think you should create separate issue related to your problem. What retrofit is doing - you can look into auto geneted code and you will know what is it.
Regarding to isolates support, is there any new idea?

@giaur500 isolates support is in development for 2.0 as far as I can see.

Good idea for a separate ticket btw. @NicolaVerbeeck I'll create one and attach there a minimal sample app.

@NicolaVerbeeck this appears to be a DIO issue - https://github.com/flutterchina/dio/issues/961
so, not sure if we need a separate issue, however, as this makes retrofit unusable for requests, producing medium to large responses, this probably worth mentioning in the docs here.

have any idea for this problem? my app is so slow when calling multiple APIs.


Delay When Click on Tabs Change (in swap fast and without delay)

Hello
When i Click on tabs, Changing tabs has a delay (may for render content) but in swap change tabs it works correct and without delay
if it's because render, there is any way to change tab then render content ?
my code :
const renderTabBar = props => (
    <TabBar
      {...props}
      indicatorStyle={{ backgroundColor: '#639' , height: '100%' , borderRadius: 20 }}
      style={{ backgroundColor: '#000000' , height: 30 , marginHorizontal: 10 }}
      labelStyle={{fontFamily: 'Vazir'  , height: 30 , marginTop: -10}}
      tabStyle={{height: 30}}
      pressColor='#00000000'
    />
  );

const renderScene = SceneMap({
    day: Day,
    week: Week,
    month: Month,
  });

<TabView
navigationState={{ index, routes }}
renderScene={renderScene}
onIndexChange={setIndex}
initialLayout={initialLayout}
renderTabBar={renderTabBar}
swipeEnabled={false}
removeClippedSubviews={true} //in IOS Must be False (don't use in IOS)
lazy={true}
renderLazyPlaceholder={() => <View style={{flex: 1}}></View>}
/>

"react-native": "0.63.2",
"react-native-reanimated": "^1.13.0",
"react-native-gesture-handler": "^1.7.0",
"react-native-tab-view": "^2.15.2",

same issue

same issue here.

same issue here.

In my case the delay initially does not seem so long (although I notice a difference compared to the manual slide that is instantaneous) but I noticed that if you keep clicking for a while several times on the different buttons on the TabBar the delay will accumulate causing the application several seconds to complete all index exchange requests and their animations.

same issue

same issue

Same, any news? Do you guys need more info?

Smooth manual slide

Delay on click custom tabbar Item. Note that after clicking several times and moving the cursor off the screen, the transitions still continue for a few seconds until all queued index exchange requests are finished. (Wait for the GIF to be fully loaded to see it better)

OBS: Top tab bar with icons (on example project) don't have this delay issue...

Well, I recommend using react-native-viewpager, that project is updated more frequently and the index change is practically instantaneous with much less visual bugs than react-native-tab-view.

Same issue here the problem appear when i used
navigationState={this.state}
and
onIndexChange={index => this.setState({index})}
instead of
  const [index, setIndex] = React.useState(0);
  const [routes] = React.useState([
    { key: 'first', title: 'First' },
    { key: 'second', title: 'Second' },
  ]);

cause my components need to use the state.
I tried to do some pureComponents but it's the same..

+1 There seems to be a delay in changing tabs due to setState.

I'm facing this same issue and even after I follow all steps recommended in 'Optimization tips' guide it's super laggy. I read that onTabPress set a new index state and swipe movement does not because it uses jumTo method, so I tried to:
const renderTabBar = (props: any): ReactNode => (
    <TabBar
      {...props}
      onTabPress={({ route, preventDefault }) => {
        preventDefault();
        props.jumpTo(route.key);
      }}/>

...And nothing changed. I have a custom renderScene map and I'm trying to implement jumpTo on my routes. Something like:
const renderScene = ({ route, jumpTo }) => {
    switch (route.key) {
      case 'GENERAL':
        return (
          <ExtractPointsList
            content={pointsTransactions.pointTransactions}
            emptyMessage={emptyMessages.get('GENERAL')}
            onRequestTransactions={onRequestTransactions}
            jumpTo={route.key}
         />

Swipe movement is working nice with my big list but onPress is suffering. :(

has anyone tried to use a class component instead of a functional component and see if this issue still persists?

Any update?

same issue

issue when enable debug, if disable debug
no have issue

any update?

any updates ??

Hello,
Apologies, something urgent came up. I can no longer attend this interview.
Regards,
Don Christian C. Abao
On Tue, May 3, 2022, 5:41 PM Mohamed-Raid @.***> wrote:

any updates ??
—
Reply to this email directly, view it on GitHub
https://github.com/satya164/react-native-tab-view/issues/1091#issuecomment-1115911360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4AYUOCXD5IUJMLFW47BG3VIDYETANCNFSM4TF3HUHQ
.
You are receiving this because you commented.Message ID:
@.***>

I think the problem arises from mounting and unmounting so many views all at once. My use-case was a bottom tab navigation without swiping, so it was pretty easy to build the navigation myself. In the main top content, I control tab visibility via opacity, not display, and definitely not conditional rendering. This solution may cause additional strain on the device (and more backend calls) since all tabs are always rendered and mounted. But it does accomplish the goal of absolutely instantaneous transition when clicking on tabs. There is no delay.

Facing same issue, (swipe faster but delay and lag on tab press). I am using material top tabbar


Consider disabling Snyk

Context
The Snyk workflow fails on most PRs

Problem
If you click in to see details about the failure, it says

Example: https://app.snyk.io/org/celestia/pr-checks/e13ca18a-fe15-4a57-8751-8df403d7b327
Proposal
I don't think the Snyk workflow has significantly improved this codebase. So far it seems like an unhelpful red ❌ on PRs.

cc: @celestiaorg/devops

Closed by @MSevey


Create fuzzing tests, off-by-one error tests, and edge case tests for the Transactions module to achieve 100% code coverage.

I'm checking it

on it


Add widget that displays widgets

This PR adds 3 files. Start by taking a look at the one currently named reference/03.00-widget-list.ipynb. I strongly recommend that you do that by running the notebook rather than just looking at the code.
The notebook presents a widget which displays all of the widgets in ipywidgets using, of course, some widgets. In that fancy widget, clicking on the name of a widget (e.g. FloatRangeSlider) takes you to a fuller description of the widget.
Potentially useful for giving folks a very quick intro to the types of widget available.

Thanks, since these reside in a separate reference/ directory, I am merging them in for now.  I'll get the files integrated with the tutorial this weekend.


[Bug] java.lang.NoClassDefFoundError: org/apache/hadoop/hive/common/ValidWriteIdList in HiveDelegationTokenProvider#initialize

Why are the changes needed?
Fix #2879
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Thanks, merging to master

cc @zwangsheng as you are the original reporter.


Release Checklist 1.28.1

Issues in Release

[x] #3752 make iquest more restrictive so as not to pick up temporary zips @hyi
[ ] #3754 Fix SOLR harvesting crashes due to incomplete and/or corrupt resources @alvacouch

Beta Deployment

[x] Diff RC to master to identify and manually make changes in the following files:

hsctl
config/hydroshare-config.yaml
nginx/config-files/hydroshare-ssl-nginx.conf.template
scripts/templates/docker-compose.template

[x] Changes in hydroshare/local_settings.py need to be coordinated for manual update
[x] Deployed to Beta
[x] check_resource beta results match current www results
[x] Review the search and discovery pages
[x] Create a new user and update profile
[x] Create iROD account, test connection and delete iROD account
[x] Create a new resource, check sharing/permission settings, delete new resource
[x] Developers test around issues
[x] QA testing around issues
[x] Stakeholders approval

Production Deployment

[ ] Deployed to Production

Make manual changes to files identified in Beta Deployment

[ ] Maps API key is correct and maps are displaying correctly
[ ] check_resource www results match pre-deployment www results
[ ] Review the search and discovery pages
[ ] Create a new user and update profile
[ ] Create iROD account, test connection and delete iROD account
[ ] Create a new resource, check sharing/permission settings, delete new resource
[ ] Developers test around issue

Notes relevant to deployment

I checked out the SOLR encoding with the separate sanity/debugging command solr_prepare and it works fine. I ran the solr_update and it works fine:
SOLR count = 4287
Django count = 4287
Django contains 4237 discoverable resources and 50 replaced resources
0 resources in Django refreshed in SOLR
SOLR contains 4237 discoverable resources and 50 replaced resources
0 resources not in Django removed from SOLR

It checks out.


🐞 Fix compatibility issue with Internet Explorer

Supercedes #91.
Fixes the same issue, but without double negation and lint violation.

Travis CI errantly linked a build from a different branch to this PR. :grimacing:


Pre-generated json peaks array not loading

Hi,
I'm trying out your library to replace my own homegrown player based on audiowaveform. Upon upload, I'm generating json via audiowaveform so that I can pre-load the peaks in your player.  Unfortunately, wavesurfer doesn't seem to like the json array range generated by audiowaveform (or more likely, I'm doing something boneheaded).
The generated json looks like this (snip):
[0,0,-12,7,-12,8,-14,15,-16,10,-11,13,-15,11, ... ]
Here's a test page with the array loaded at runtime
http://www.internetdj.com/test.html
Any tips? Many thanks in advanced.
Michael

Huh, I was using the MediaElement backend in this case.  I tried the default WebAudio and what do you know? It worked perfectly.

The problem was with the negative values generated by audiowaveform.  I simply stripped them out of my peaks array and now we're in business.

Great that you figured it out. I actually thought negative peaks were not a problem. I'll mark this as a bug to add a Math.round to the drawing code.

Thanks for the confirmation. Love this library!

Having a unit test for this would be nice!

@mbordash  how are you making to draw waveform with json data can you share some code ?

Perhaps something like audiowaveform -i some-file.mp3 --pixels-per-second 10 -b 8 -o some-file.json.

@katspaugh  Yeah i find it and i got file like this http://collabshot.com/show/0T3kzy but when I am loading it with wavesurfer I think it creates incorrectly   see : http://collabshot.com/show/CovV4L
wavesurfer.load('fifties_rock.mp3', result);
I tried also :
wavesurfer.load('fifties_rock.mp3', result.data);

@btomashvili I guess you also need to filter out negative values like the OP did.
result.data.filter(function (val) { return val >= 0; })

thanks ;)

Closing: inactive


Add link to helpful issue

[Enter feedback here]

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

ID: bcfc7f38-7c0f-cc95-92c4-e99e0ad1812e
Version Independent ID: 9a1dccda-d7bf-1eb5-e5e3-8aba4aeb444f
Content: Troubleshoot ASP.NET Core Localization
Content Source: aspnetcore/fundamentals/troubleshoot-aspnet-core-localization.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @hishamco
Microsoft Alias: riande

Associated WorkItem - 72556

Fixed in #28732

@Rick-Anderson I'm not sure why we need this while we already already described everything in the troubleshooting section!!

It's to get folks looking a closed issues before asking the same questions.


Fixing bug nectarcam pulse shape

This is a branch Raul opened over a year ago and it is not clear why it is called nectarcam pulse shape.
I however think that the small additions can be merged into the master branch.

Codecov Report

Merging #280 (336d3d9) into master (ed18de6) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
+ Coverage   85.39%   85.40%   +0.01%
==========================================
  Files          77       77
  Lines        6306     6311       +5
==========================================
+ Hits         5385     5390       +5
  Misses        921      921

Impacted Files
Coverage Δ

simtools/db_handler.py
72.59% <100.00%> (+0.08%)
:arrow_up:

simtools/model/telescope_model.py
81.22% <100.00%> (+0.33%)
:arrow_up:

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 ed18de6...336d3d9. Read the comment docs.


Changes to support PHP 8 + remove support for PHP 7.2 or higher

Agreed, but we should also update the readme.md and the composer.json. And then tag it as 3.0 (afaik).

I made some adjustments in this PR:

Now PHP 8.1 is the minimum required version
A ChangeLog was added; might be useful to those who use the package and want to update
ReadMe was updated
Commits were rebased
Laminas dependencies minimum version has been increased

As for the tag, should I tag the commit or will this be done by a maintainer?
I also cannot contact the owner of this repository unfortunately. @rbroen are you a maintainer by any chance?

Sry for the delay - will merge and add a new version tag.


[occm] Internal Service might be exposed when LB is shared

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
When you create a Service with service.beta.kubernetes.io/openstack-internal-load-balancer: "true" annotation and then create another regular non-internal Service that's supposed to share the LB with the previous one, CPO will allow that and will attach the FIP to the load balancer. This will expose the both Services outside, including the one that supposed to only be internal. While a bit obscure, it's a potential security risk that a confused user might create without knowing.
What you expected to happen:
Either sharing internal LB with an external Service is blocked or there's a way to only expose one listener from a share LB. I don't think the latter is possible.
How to reproduce it:
Explained above.
Anything else we need to know?:
Environment:

openstack-cloud-controller-manager(or other related binary) version:  master
OpenStack version: Doesn't matter
Others:

Trying to wrap my head round this. Do you have example yaml for both services?

@mdbooth the bug relates to a shared loadbalancer: https://github.com/kubernetes/cloud-provider-openstack/blob/master/docs/openstack-cloud-controller-manager/expose-applications-using-loadbalancer-type-service.md#sharing-load-balancer-with-multiple-services
Indeed we should not mix up the private and public loadbalancers as a shared entity, since the FIP is assigned to a loadbalancer, not to a listener. This also should be mentioned in docs.

good catch, I Think Either sharing internal LB with an external Service is blocked  is the best option
@dulek guess you will submit a PR ,correct? we can update Doc to indicate this restriction as well if needed


python3Packages.jaxlibWithCuda: init at 0.1.75

Motivation for this change
In the spirit of https://github.com/NixOS/nixpkgs/pull/156808, we'd like end-users to be able to select which "edition" of jaxlib they would like to use. This is akin to the tensorflow vs tensorflowWithCuda precedent. By adding jaxlibWithCuda to python packages, cache.nixos.org should be able to provide builds for users, freeing them from having to run a very time consuming and expensive build for themselves locally.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Ah, good point... Added jaxlibWithoutCuda in https://github.com/NixOS/nixpkgs/pull/157055/commits/d045385f09ab6c4b4748d26fffc9c30b2302f44e.
I'll merge tomorrow unless anyone objects.

By adding jaxlibWithCuda to python packages, cache.nixos.org should be able to provide builds for users, freeing them from having to run a very time consuming and expensive build for themselves locally.

Hydra won't build it because of the unfree cudatoolkit.

Damn... what do the tensorflow folks do then?

Maybe they use the -bin versions of the packages that use pre-built wheels from PyPI.

Also, is it really such a big deal to build it once every what, couple of weeks?

Compiling from source is pretty nasty with these ones. iirc building tensorflow from source is over 24 cpu-hours. Last time I had to debug that derivation I had to create an ec2 instance with as many cores as I could find.

I went ahead and merged since the changes are relevant with or without hydra, but I would like to keep talking about that side of things to figure out how we can get builds cached somewhere.


Decide process for documenting milestones for SHR w/ Trello/github sync up

Overall project (high level) is being tracked on Trello. Our subtask is being tracked here.


Solving Power Cell bug

Closes #647

I think the fix is actually to remove the collision component from here completely. The game should still load fine since it should load the collision component from the prototype. The same thing needs to be done with pretty much all the entities in the map.

Also, the issue #647 isn't just about the power cell. It also affects all the other entities by overwriting their collision boxes from what the prototype says. This makes it harder to fix collision bugs because changing the prototype doesn't change the collision box of entities spawned when the map loads.


Warning: Ambiguous class resolution, "Google\Cloud\PubSub\V1\PublishRequest"

Hello,
I have this error  Warning: Ambiguous class resolution, "Google\Cloud\PubSub\V1\PublishResponse" was found in both "/api/vendor/google/cloud-pubsub/src/V1/PublishResponse.php" and "/api/vendor/google/proto-client/src/Google/Cloud/PubSub/V1/PublishResponse.php", the first will be used.
Warning: Ambiguous class resolution, "Google\Cloud\PubSub\V1\ListTopicsRequest" was found in both "/api/vendor/google/cloud-pubsub/src/V1/ListTopicsRequest.php" and "api/vendor/google/proto-client/src/Google/Cloud/PubSub/V1/ListTopicsRequest.php", the first will be used.
when installing this three depends.
"google/proto-client": "^0.31",
        "google/protobuf": "^3.6",
        "google/cloud-pubsub": "^1.7",

Thank you for you help

Hi @mayro,
You no longer need to include google/proto-client. The classes found within that library have been moved into google/cloud (and the components such as google/cloud-pubsub). google/protobuf is also required by existing dependencies, so you don't need to require that in your project either. I suggest you also upgrade to google/cloud-pubsub v1.11.0. There should not be any breaking changes.

Hello @jdpedrie
Thank you for your reactivity. Exactly i moved the proto-client and it works like a charm !!
I will migrate to pubsub  v1.11.0 too :)

@jdpedrie Sadly I have some error withe the latest version of pubsub :/ some conflict with proto extension     - google/gax 1.0.0 conflicts with ext-protobuf[<IP_ADDRESS>].
I think i will keep the old pubsub error, because there is a issue already  on gax bib.
Anyway Thank you for you help ;)


F1 score of 0 for character level sequence labelling task

Training data looks like:

and training results look like:

Also worth adding that whenever I add a new period at the end of each character sequence, it never begins training and stops at Epoch 1/15 with no progress bar. I'll be happy to share training data with you if that will help reproduce the issue.

I think you cannot just use any tag label schema and expect evaluation function to work correctly, without editing anago.metrics.get_entities - it expects BILO.

Thanks @tc64
@riteshpanjwani one workaround  that made it work for me wass to add B- to the beginning of every label.

Not use BILO. Use IOB2.
Because anago ver0.0.5 assumes that training data is labeled by IOB2.
I will fix the problem in ver 1.0.0.


index out of bounds panic in array_walker

Given this json response from http://api.worldbank.org/v2/country/br?format=json
[
    {
      "page": 1,
      "pages": 1,
      "per_page": "50",
      "total": 1
      },
      [
        {
          "id": "BRA",
          "iso2Code": "BR",
          "name": "Brazil",
          "region": {
            "id": "LCN",
            "iso2code": "ZJ",
            "value": "Latin America & Caribbean (all income levels)"
          },
          "adminregion": {
            "id": "LAC",
            "iso2code": "XJ",
            "value": "Latin America & Caribbean (developing only)"
          },
          "incomeLevel": {
            "id": "UMC",
            "iso2code": "XT",
            "value": "Upper middle income"
          },
          "lendingType": {
            "id": "IBD",
            "iso2code": "XF",
            "value": "IBRD"
          },
          "capitalCity": "Brasilia",
          "longitude": "-47.9292",
          "latitude": "-15.7801"
        }
      ]
    ]

I can get incomeLevel by:
$ jql '[1].[0]."""incomeLevel"""."""value"""' br.json

However, when I try to fetch an out-of-bounds value, instead of failing gracefully, jql panics:
$ jql '[2].[0]."""incomeLevel"""."""value"""' br.json

thread 'main' panicked at 'index out of bounds: the len is 4 but the index is<PHONE_NUMBER>3709551615', C:\Users\jsmith\.cargo\registry\src\github.com-1ecc6299db9ec823\jql-3.1.1\src\array_walker.rs:37:34
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

@jqnatividad good catch!
I've created a fix and will make a new release when the CI passes.


Help:Adding Multiple Students to the Couse Api not accepting the body.

Hello,
I tried to hit this api
POST /api/students/<course_id>: Add multiple students to course
The body which I am trying to send is :
{"students":
[
{
"username": "liustudent01",
"first_name": "liustudent",
"last_name": "lastname",
"email"<EMAIL_ADDRESS>}
]
}
Even though I am passing students key to the body.
This give me response:
{
"success": false,
"message": "Please supply students"
}
Can anyone please help me with this.
Thank you in advance !

The POST body should be an HTTP form where the key is student and value is a JSON array. You are sending a JSON dict where key is student and value is a JSON array.
If you use vngshare (see docs), here is the CURL command you should send:
curl 'http://<IP_ADDRESS>:12121/api/students/course2' -X POST --data-raw 'students=%5B%7B%22username%22%3A+%22kevin%22%2C+%22first_name%22%3A+%22kevin_first_name%22%2C+%22last_name%22%3A+%22kevin_last_name%22%2C+%22email%22%3A+%22kevin_email%22%7D%5D&user=abigail'

Please ignore the user=abigail at the end in actual ngshare. This is how vngshare authenticates.
This is what is sent when viewed by Wireshark:

I see. The documentation (docs/api/course.rst) is misleading as it says:
POST /api/students/<course_id>
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
*Add or update students. (instructors only)*

If the request syntax is correct, will return 200 and report whether each student is added correctly.

Request (HTTP POST data)
""""""""""""""""""""""""
.. code:: javascript

    {
        "students":
        [
            {
                "username": /* student ID */,
                "first_name": /* student first name */,
                "last_name": /* student last name */,
                "email": /* student email */
            },
            ...
        ]
    }

Thank you for your response.


Wrong dependencies in version 6.0.216

In version 6.0.216 you are referencing old version of Abstractions (6.0.190)

I just checked the dependencies of the ADS Packages. I don't see any wrong dependency there. Which package shows the problem?

Eventually you have a problem with your local nuget package cache. You could try to clean it with nuget CLI commands.

it's not visible on the package, it is a runtime exception

Sure, but I expect that within your project a wrong version of the dll is located on the disk, because it is not properly updated by nuget, or because your application build is not renewing the dll from the correct package properly (these are common problems in VisualStudio that bite from time to time).
The runtime simply loads the Assembly it finds in its search path because different Nuget package versions have all the same AssemblyVersion.
You could check the location where the wrong Dll is loaded from in the Vs-Debugger (Modules View) and check that version.
So please ensure:

remove all TwinCAT.Ads*.dll files from bin and obj folders (or delete these folders) within your VS-Solution
If that doesn't help delete Nuget package caches and install the Nuget Dependencies again.

Because the package dependencies seem to be correct, it must be an issue in updating/copying the package/dlls on your system. Or picking the wrong dll from an unexpected location during runtime.

the problem was with an old version of one of our own nuget packages having a dependency on 6.0.190
thanks for pointing me to the solution


Handle different serial devices

Actually we hardcode /dev/ttyUSB0. This should be passed as a param instead

Fixed in 8f8367b1d006d86752e2f2ced7acb6f176417e53


Recreation/Stress buster and motivation corner

Is your feature request related to a problem? Please describe.
The general issue when it comes to current generation is unavoidable stress and high self esteem.Everyone wants to continuously achieve and fail to accept small setbacks and get extremely worked up.
Describe the solution you'd like
"Laughter is the best medicine",says a famous quote upon asking the user his preferences initially when he launches the app,we can provide memes and funny videos personalised to his taste and with the chatbot data can be tracked and mood analysed. We can also provide with healing and self-worth uplifting quotations which provides a sense of belief to the person.
Kindly assign me with this task

@chinmaiyee assigned!

Thank You! Will work on it!


chore(docker): init

next step would be helm chart or kustomize, probably.
sad that image size couldn't be reduced
question: interested in auto ci build&push on docker hub or any other registries?

I tried once more building on musl, i passed multiple compilation errors (eg: openssl), now i struggle with rust-rocksdb, that needs rocksdblib-sys, using bindgen in non-static mode: which forbidden the import of libclang dynamically
55.93 error: failed to run custom build command for `librocksdb-sys v0.11.0+8.1.1`
55.93
55.93 Caused by:
55.93   process didn't exit successfully: `/kaspa/target/release/build/librocksdb-sys-cd495b1d7dbbf10b/build-script-build` (exit status: 101)
55.93   --- stderr
55.93   thread 'main' panicked at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/bindgen-0.65.1/lib.rs:603:31:
55.93   Unable to find libclang: "the `libclang` shared library at /usr/lib/llvm15/lib/libclang.so.15.0.7 could not be opened: Dynamic loading not supported"
55.93   note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
55.93 warning: build failed, waiting for other jobs to finish...

here are issues i found useful

closing it in favor of #511


Vulnerability Roundup #9 Mass Rebuild Rollup (WIP)

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/20462

jasper: 1.900.21 -> 1.900.28 https://lwn.net/Vulnerabilities/705673/, https://lwn.net/Vulnerabilities/705824/

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Merged with
[master 1eb545d] jasper: 1.900.21 -> 1.900.28
 Date: Wed Nov 16 08:36:51 2016 -0500
 1 file changed, 5 insertions(+), 2 deletions(-)
[master 7ed55dc] xinetd: patch for CVE-2013-4342
 Author: Graham Christensen<EMAIL_ADDRESS> Date: Wed Nov 16 13:31:20 2016 -0500
 1 file changed, 9 insertions(+), 1 deletion(-)
[master d0d3330] shutter: add patch for CVE-2015-0854 with remote code
 Author: Graham Christensen<EMAIL_ADDRESS> Date: Wed Nov 16 13:57:45 2016 -0500
 1 file changed, 9 insertions(+), 1 deletion(-)

and backported with
[master 1eb545d] jasper: 1.900.21 -> 1.900.28
 Date: Wed Nov 16 08:36:51 2016 -0500
 1 file changed, 5 insertions(+), 2 deletions(-)
[master 7ed55dc] xinetd: patch for CVE-2013-4342
 Author: Graham Christensen<EMAIL_ADDRESS> Date: Wed Nov 16 13:31:20 2016 -0500
 1 file changed, 9 insertions(+), 1 deletion(-)
[master d0d3330] shutter: add patch for CVE-2015-0854 with remote code
 Author: Graham Christensen<EMAIL_ADDRESS> Date: Wed Nov 16 13:57:45 2016 -0500
 1 file changed, 9 insertions(+), 1 deletion(-)


[FEAT] Default tiff_folder_key parameter for ImportTiffPipeline

Usually we treat the input-data folder as the default when running some pipelines, where this is the location of input data to be used in the specific pipeline.
Usually it was not the case that one had to provide specific folder keys for input-data, so perhaps I would suggest to use the input-data location as a default in cases where the tiff_folder_key parameter is not provided when running the  ImportTiffPipeline. This would make the tiff_folder_key parameter optional.
IMHO since the input-data is always there and is just one, there is no need to explicitly provide it as a folder key.

Agreed. Prepare a MR or remind me if i forget :)


Tab Icon Jumps (flicker) when pressing on tabs in footer

I don't know if someone already posted a bug here or not. Opening a simple tab-bar example form Ionic on iOS9 brings a bug when pressing tabs in footer. They move when an icon transition is made.
There's already a forum post on official Ionic forums here: http://forum.ionicframework.com/t/tab-icon-jumps/34316
Person buunguyen also recorded the issue which can be seen here: http://jmp.sh/0A5ql5d
There are currently two options to fix the issue:

Either edit the ionic.bundle.js or
add new .config to your app which will replace the part of the code where the transition happens.

Person buunguyen also posted a code in ES6 format, here's the same in ES5:
angular
.module('ionic')
.config(['$provide', function ($provide) {
  $provide.decorator('ionTabNavDirective', ['$delegate', function ($delegate) {
    var directive = $delegate[0]
    directive.template = ' {{badge}}  '
    return $delegate
  }])
}])

We have also tried to get the latest #master but that didn't solve anything.
:+1:  to developers of Ionic :)

Greetings @lordgreg!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Duplicate of https://github.com/driftyco/ionic/issues/4395
TLDR: Apply the patch from http://blog.ionic.io/ios-9-potential-breaking-change/

Thank you.

Actually no @simplesmiler.
We have implemented ionic Patch and the tab jumping is still there.

There was a duplicate bug #4006, where I commented about a CSS workaround I used:
/* For a moment there's two -elements inside tab-item, don't allow both to be shown. */
.tabs .tab-item i:nth-of-type(2) {
  display: none;
}

This doesn't fix the underlyng problem, but prevents the icon flickering.

+1 @markahon, I also applied the patch and had to disable the second icon. Merci

Thanks @markahon, this css workaround is gold.

@markahon Thanks for the fix. It works well on browser. I need to use
display: none !important;

to make it work on iOS 9 as well.

Hello! Im glad you all found a workaround! Thanks @markahon ! Are you all still experiencing this issue?

Seems to me that at least on ionic version 1.2.4 the flicker is gone when I remove the css workaround.

Awesome! Im glad to hear that! Ill be closing this issue for now as it looks like it was resolved in 1.2.4. Thanks again!

This is still an issue in the latest nightly (v1.2.4-nightly-2826) on iOS 9.3. Applying the patch above fixes it.

I can confirm that I have this issue as well on 1.2.4.

Yup, me too


Conferences need to be markable as "meetup"

One responding action would be "don't tweet CFPs for meetups"

Not sure that I'm currently feeling the need for thisl. Symposium isn't for meetups.


Transaction File Sorting

API and UI need for sorting transaction files from bank.

[x] sort single transaction file using description map serialized to yaml file
[ ] auto-generate description map based on pre-existing account entry files. (optional)
[ ] add a menu for handling optional duplicate entries.

As of 345ab0724d6ea6eec73a2d2192e44a83df899008 the sorting kinda works. However, for a list of key phrases each phrase must match entirely to count towards categorizing the entry. Currently this isn't the case.


Coverage

[ ] Fix coverage running.
[x] Increase test coverage.
[ ] Add solidity coverage reporting to CI pipeline.

The pipeline integration might need to wait until after release or a subscription service for private repos.  Something like coveralls might work.

Closing as a dupe of #460

Closing as a dupe of #460


Change some labels to be more consistent with the device

01x02 vs RingMod
MD vs MW
VL vs Vel
AF vs AT

Closed via 49fb06182f82c7fd0e213ab7c0bf23a950b266ae


Webpack and RSPack plugins fail when there are files with other extensions

When there are other file types/loaders used, an attempt is made to process them as valid input for stylex leading to a compilation failure
example if the index.html file has a different extension for template compilation.
ERROR in   Error: Child compilation failed:
  Module parse failed: Unexpected token (1:0)
  File was processed with these loaders:
   * ../../node_modules/.pnpm/html-webpack-plugin<EMAIL_ADDRESS>   *<EMAIL_ADDRESS>  You may need an additional loader to handle the result of these loaders.
  > <!DOCTYPE html>
  | <html lang="en">
  |

https://github.com/eryue0220/unplugin-stylex/releases/tag/v0.3.2

I've been runnning into this with more and more extensions since bundlers deal with fonts, images, svg, xml, json and numerous other extension. Could we flip to a model that only validExts are processed by this plugin.
Maybe something like
'''
[ ".js", ".cjs", ".mjs", ".ts", ".jsx", ".tsx"]
'''
By default.


Vim9script - Bug in the autoload mechanism?

Steps to reproduce
(I originally posted this question on vi.stackexchange, and I'm posting it again here upon D. Ben Knoble's suggestion.)
Put this content in ~/.vim/plugin/messingaround.vim
vim9script noclear

import autoload 'messingaround.vim' as xyz

if exists("g:loaded_messingaround")
  finish
endif
g:loaded_messingaround = 1
var save_cpo = &cpo
set cpo&vim

if !hasmapto('<Plug>MessingAround;')
  map <unique> <Leader>a  <Plug>MessingAround;
endif
noremap <unique> <script> <Plug>MessingAround; :echo <ScriptCmd>xyz.Say(expand("<cword>"))<CR>

&cpo = save_cpo

and this content in ~/.vim/autoload/messingaround.vim
vim9script noclear

export def Say(str: string): string
  return str
enddef

Temporarly empty your ~/.vimrc (I couldn't reproduce the problem with -u NONE, but I haven't looked much into it) and open a new Vim session, type a word into it, and hit Leadera.
You'll see the command line being populated with :echo  (trailing space included), but no trace of Say being run.
Expected behaviour
The word under the cursor should be echoed.
Notice that if I don't use the autoload mechanism, i.e. if I

define Say in the ~/.vim/plugin/messingaround.vim file (and delete the file in autoload),
remove the keyword export
change <ScriptCmd>xyz. to <SID>

the function behaves as I expect.
Not sure if in the original case I expect the wrong thing, though.
Version of Vim
8.2.4651
Environment
Operating system: Archlinux (up-to-date)
Terminal: URxvt
Value of $TERM: rxvt-unicode-256color
Shell: GNU bash 5.1.16(1)
Logs and stack traces
No response

You'll see the command line being populated with :echo (trailing space included), but no trace of Say being run.

:echo expects an expression as argument, but <ScriptCmd>...<CR> is not an expression.  <ScriptCmd>...<CR> just asks Vim to execute an arbitrary Ex command regardless of the current mode.
In contrast, <SID>Say(...) is an expression, so it works.

I suspect that you want to refer to an autoload function in a mapping using the script.func syntax, but you can't.  There are still indeed a few contexts where it's not possible:

:help i_CTRL-R_=
:help c_CTRL-R_=
:help c_CTRL-\_e
:help stl-%{
:help 'statusline' and :help 'tabline' (%!Func())
:help input() (3rd {completion} argument)

Not sure if we need yet another pseudo-key, some other mechanism, or just more documentation on this pitfall.
In any case, if you need to refer to an autoload function in a context where script.func is not valid, you can still use the legacy name script#func.

Unrelated, but you don't need to save and restore 'cpoptions'; it's done automatically:

A side effect of :vim9script is that the 'cpoptions' option is set to the
Vim default value, like with: >
:set cpo&vim
One of the effects is that |line-continuation| is always enabled.
The original value of 'cpoptions' is restored at the end of the script, while
flags added or removed in the script are also added to or removed from the
original value to get the same effect.  The order of flags may change.
In the |vimrc| file sourced on startup this does not happen.

See :help vim9-namespace.

In any case, if you need to refer to an autoload function in a context where script.func is not valid, you can still use the legacy name script#func.

Now that I think about it, that will be an issue when we start to refactor import autoload into import lazy.  Then, there will no longer be any legacy name to fall back on.  So, maybe we do need some new syntax/mechanism; I don't know.

In any case, if you need to refer to an autoload function in a context where script.func is not valid, you can still use the legacy name script#func.

Now that I think about it, that will be an issue when we start to refactor import autoload into import lazy. Then, there will no longer be any legacy name to fall back on. So, maybe we do need some new syntax/mechanism; I don't know.

Do you mean I shouldn't follow your original suggestion if I want my code to be compatible with wherever Vim9 will get to?

Do you mean I shouldn't follow your original suggestion if I want my code to be compatible with wherever Vim9 will get to?

import lazy will not deprecate import autoload, so if you keep using the latter, then you will be able to use the legacy name script#func anywhere an autoload function name is accepted.
The issue will arise if you later decide to refactor import autoload into import lazy.

Regarding

:echo expects an expression as argument, but <ScriptCmd>...<CR> is not an expression. <ScriptCmd>...<CR> just asks Vim to execute an arbitrary Ex command regardless of the current mode.
In contrast, <SID>Say(...) is an expression, so it works.

and

I suspect that you want to refer to an autoload function in a mapping using the script.func syntax, but you can't. There are still indeed a few contexts where it's not possible:

:help i_CTRL-R_=

would you mind quoting the bits of the documentation from which I'd have deduced the 3 phrases I've styled in bold above?

... is not an expression

From :help <ScriptCmd>:
<ScriptCmd> is like <Cmd>

Then from :help <Cmd>:
The special text <Cmd> begins a "command mapping", it executes the command
directly without changing modes.

This states that a command is executed.  Not that an expression is evaluated.  At no point in the rest of this help tag is the word "expression" ever used.

<SID>Say(...) is an expression

<SID> is irrelevant here.  What matters is Say() which is a function call.  And a function call is an expression.
From :help expression-syntax or :help expr10:
function(expr1, ...)	function call

but you can't

To be clear: you can use script.func if and only if you're in a Vim9 context.  The issue is that a mapping is run in the global context where only the legacy syntax is allowed.  That's why <ScriptCmd> was introduced: to let you use the Vim9 syntax in a mapping.  In practice, this covers most cases, but not all.  For example, in your original mapping:
noremap <unique> <script> <Plug>MessingAround; :echo <ScriptCmd>xyz.Say(expand("<cword>"))<CR>
                                                     ^---------^

I guess you originally wanted to write something like this:
noremap <unique> <script> <Plug>MessingAround; :echo <C-R>=xyz.Say(expand("<cword>"))<CR>
                                                     ^----^

Then found out that it didn't work.  I don't think this limitation is currently documented, which is why I wrote:

Not sure if we need yet another pseudo-key, some other mechanism, or just more documentation on this pitfall.

It was mentioned before that a limitation of a mapping is that it just replaces typeahead by other text.
No script context can be used when the mapping expands.  Thus the relation with the script must be included when
the mapping is defined.  For that  works fine.
Since you cannot use  for a function like importName.Func() you need to split it up:
def LocalFunc(): string
  return xyz.Say(expand("<cword>"))
enddef
noremap <unique> <script> <Plug>MessingAround; :echo <SID>LocalFunc()<CR>

Disclaimer: I haven't tried it!
Perhaps it's possible to add something like  for calling an imported function, but is it worth it?
Moving as much as possible into a compiled function is more useful.
Although, if the only thing that LocalFunc() would do is call the imported function, it does add some overhead.

Hmm, perhaps we can make this work:
:echo <SID>xyz.Say(expand("<cword>"))<CR>

Need to recognize "{name}{dot}", replace with "{scriptnr}_" and when called load that script if needed, before calling the function.
Seems a bit like magic, but I think we have all the information needed.

Perhaps it's possible to add something like for calling an imported function, but is it worth it?

I don't know, but at least we might want to improve the help with regards to how a user is expected to refer to a function in a mapping.  In particular:

mappings are run in the global context where the legacy syntax is used
an exception is made for commands between <ScriptCmd> and <CR>, and for <expr> mappings, which are run in the script context; this lets you use the Vim9 syntax if your mapping is installed in a Vim9 script
if you need to refer to a function in a mapping using a Vim9 syntax, but the latter is not allowed, then use a wrapper function; for example:

vim9script
import 'script.vim'
def Wrapper(): string
  return script.func()
enddef
ioremap <key> <C-R>=<SID>Wrapper()<CR>

All of this might already be documented here and there, but maybe we should have 1 help tag somewhere at :help vim9 which centralizes all of this information.

Hmm, perhaps we can make this work:

Yes, I think this would fix most, if not all the remaining cases.

Seems a bit like magic, but I think we have all the information needed.

It's not really magic.  From a mapping, we can refer to a script-local function via <SID>.  Why couldn't we do the same for a variable?  I already mentioned this pitfall here.

That's yet another pitfall which has always bothered me. We can access a script-local function from the global context, provided we know its ID (via the pseudo-key ), but we can't do the same for a script-local variable.

It's not really magic. From a mapping, we can refer to a script-local function via . Why couldn't we do the same for a variable?

Ah, I guess script is not technically a variable.  Still, it looks like one.

With patch  8.2.4747  this should work.

There actually is another solution, which was mentioned before and is used in a test: Use a funcref variable:
import './XsomeExport.vim' as some
var Funcy = some.Funcx
nnoremap <F3> :call <sid>Funcy(42)<cr>

But the straigtforward way of using <SID>some.Funcx() is much nicer.

With patch 8.2.4747 this should work.

It works for a function defined under import/:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
&runtimepath = dir
dir ..= '/import'
dir->mkdir('p')
var lines =<< trim END
    vim9script
    export def Func()
        echomsg 'from Func()'
    enddef
END
lines->writefile(dir .. '/script.vim')
import 'script.vim'
nnoremap <F3> <Cmd>call <SID>script.Func()<CR>
feedkeys("\<F3>")

from Func()

But not for a function defined under autoload/:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
&runtimepath = dir
dir ..= '/autoload'
dir->mkdir('p')
var lines =<< trim END
    vim9script
    export def Func()
        echomsg 'from Func()'
    enddef
END
lines->writefile(dir .. '/script.vim')
import autoload 'script.vim'
nnoremap <F3> <Cmd>call <SID>script.Func()<CR>
feedkeys("\<F3>")

E117: Unknown function: <SNR>2_Func

Could it work?

Oh yes, a function from an actual autoload script isn't defined with  but with # in the name.

To be clear: you can use script.func if and only if you're in a Vim9 context. The issue is that a mapping is run in the global context where only the legacy syntax is allowed. That's why <ScriptCmd> was introduced: to let you use the Vim9 syntax in a mapping. In practice, this covers most cases, but not all. For example, in your original mapping:

I think I have a deep misunderstanding here. Does the above mean that using :*map commands means using non-Vim9 syntax?

Does the above mean that using :*map commands means using non-Vim9 syntax?

In a mapping, the Vim9 syntax is allowed if and only if you're in a Vim9 script, and the code is:

between <ScriptCmd> and the next <CR>
on the rhs of an <expr> mapping

Here are 2 examples:
vim9script
nnoremap <F3> <ScriptCmd>echo (() => 'this is a Vim9 lambda')()<CR>
feedkeys("\<F3>")

this is a Vim9 lambda

vim9script
nnoremap <expr> <F3> (() => ':echomsg "this is a Vim9 lambda"<CR>')()
feedkeys("\<F3>")

this is a Vim9 lambda

But still, the above looks a bit like the mappings will always be legacy.

I think you expect a mapping to let you use the Vim9 syntax just because it was installed from a Vim9 script.  That would be possible if mappings were running in the context of the script where they were installed (just like user-defined Ex commands and autocmds).  But unfortunately, they do not.
Mappings run in the global context.  This means that when the mapping is actually processed, Vim has no clue about its script (let alone whether it's a Vim9 script or a legacy script).  That's why we need this weird <SID> pseudo-key.  This tells Vim that when it installs a mapping, it should remember the script ID, so that later, when the mapping is finally used, Vim knows in which script it can find the function to call.
It was annoying/confusing to have to use the legacy syntax in a mapping installed in a Vim9 script, which is why I opened the issue https://github.com/vim/vim/issues/9499
In response, Vim improved the mappings in 2 ways:

the patch 8.2.4059 lets Vim remember the script context in the rhs of an <expr> mapping: this allows you to use the Vim9 syntax in the rhs of an <expr> mapping
the patch 8.2.4099 introduces the new pseudo-key <ScriptCmd> which again makes Vim remember the script context: this allows you to use the Vim9 syntax between <ScriptCmd> and the next <CR>

Besides, thanks to your report, Vim has improved in 2 other ways:

the patch 8.2.4748 lets you refer to an imported function via <SID>script.Func()
the patch 8.2.4751 lets you do the same for an autoload function

Maybe I just don't understand how legacy and vim9 script are meant to overlap,

Ideally, there would never be the need to use the legacy syntax in a Vim9 script.  But there are still a few cases where it's required.  It all boils down to this issue: where you are when you write your code, and where Vim is when it runs this code, are not necessarily the same places.  If Vim is in the global context when it runs your code, then the Vim9 syntax will not work.
In my experience, those cases are rare though.

for how long, and when (if) the latter will be a standalone language that needn't be combined with legacy to allow everything legacy allowed.

Vim9 will be released in a month or 2.  What you can see right now will not be much different from the release.
That doesn't mean that the syntax can not be improved further.  It can.  I have a few ideas, but not enough time.

To be clear: you can use script.func if and only if you're in a
Vim9 context. The issue is that a mapping is run in the global
context where only the legacy syntax is allowed. That's why
<ScriptCmd> was introduced: to let you use the Vim9 syntax in a
mapping. In practice, this covers most cases, but not all. For
example, in your original mapping:

I think I have a deep misunderstanding here. Does the above mean that
using :*map commands means using non-Vim9 syntax?

map commands themselves have no syntax at all.  Only replacement of "<>"
items and escaping rules apply.  Mappings specify to change typed
characters into other characters, which will then be handled in whatever
context that happens.
So basically, if I type "this" then do like I have typed "that".
Expression mappings are different though, as lacygoill explained.
--
For society, it's probably a good thing that engineers value function over
appearance.  For example, you wouldn't want engineers to build nuclear power
plants that only look like they would keep all the radiation inside.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Sorry for posting in a Closed issue, but I couldn't find the answer in the :h pages.
I read many interesting workaround (including a wrapper, a funcref,  re-defining your function so that you just <ScriptCmd>:call xyz.Say(...), and Bram also proposed the actually  nicer <SID>xyz.Say()).
My question is: as it is today (April 2023), which is the suggested method?
Does  <SID>xyz.Myfunc() work in Vim9? If so, is there any plans to update the docs?


Allow parens for optional keyword value calls in Style/MethodCallWithArgsParentheses

Yet-another-edge-case for the omit_parentheses style.

Before:
def foo(bar: baz(42))
                ^^^^ Omit parentheses for method calls with arguments.
end

After:
def foo(bar: baz(42))
end

This looks good 🌟 Would you add the CHANGELOG entry?

@koic, thanks for the reminder. Added the changelog entry, the change should be good now.

Thank you @gsamokovarov!


Add info method to the new console to fix #5 issue

Simple fix for #5 issue, where newConsole.info function was called from game.js but failed due to it being not defined.

Thx for the fix


Hello Ask a question

Hello, I want to handle item to the top out into to the left out, excuse me, can you help me?

I also want to know how to use RapidFloatingActionButton to expand my CardList from top to down & centered instead of bottom to top & left.
Is there some way to change expand direction?


feat(data_classes): Cognito custom auth triggers

Issue #, if available:
Description of changes:
Add support for "Custom Authentication Challenge Lambda Triggers":

Add response helpers
Implement "Define auth challenge", "Create auth challenge" and "Verify
auth challenge response"
Checklist

[X] Meet tenets criteria
[X] Update tests
[X] Update docs
[X] PR title follows conventional commit semantics

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #178 into develop will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           develop      #178    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           52        52
  Lines         2154      2275   +121
  Branches        97        99     +2
==========================================
+ Hits          2154      2275   +121

Impacted Files
Coverage Δ

...bda_powertools/utilities/data_classes/ses_event.py
100.00% <ø> (ø)

.../utilities/data_classes/cognito_user_pool_event.py
100.00% <100.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 738e261...5d0932e. Read the comment docs.

dev builds fixed - please rebase and apologies for the inconvenience.
https://github.com/awslabs/aws-lambda-powertools-python/runs/1165140290?check_suite_focus=true