Add Resource Management automation for issue and PR Triage

Fixes #169
This is the initial onboard into the new fabricbot service. It adds basic triage for issues and PRs. Here are the automations that are added with this:

Add untriaged label to new issues
Add Codeflow links to new PRs
Add needs-author-action to PRs when a review is submitted and changes are requested
Monitors for issues and PRs with needs-author-action label that haven't been updated in 2 weeks, and will add no-recent-activity label as well as add a comment mentioning that it will be closed in one more week of inactivity.
Monitors for issues and PRs that have no-recent-activity label and haven't had activity for an additional week, and closes them.

cc: @jeffhandley @wtgodbe @mkArtakMSFT
FYI: @DamianEdwards @davidfowl @danmoseley

Merging so that we can start testing if automation can be enabled. Will address any feedback in a follow-up PR.


freebsd: add 11.2R

upstream URL is http://mfsbsd.vx.sk/files/images/11/mfsbsd-11.2-RELEASE-amd64.img so format hasn't needed to change.

Awesome, thanks!


Missing license

What type of license would suit this kind of project? I don't know much about them, and other decomp projects over at pret don't have licenses


Can't execute k3d widget notebook

Testing microsoft/vscode#177719

Open https://github.com/microsoft/vscode-jupyter/blob/main/src/test/datascience/widgets/notebooks/k3d_widgets.ipynb
Install k3d
Run first cell
:bug: Cell never completes, no errors in output or dev tools

Works for me, please do share the verbose logs

So I went and split out all the lines in that notebook, and it looks like this succeeds:
https://insiders.vscode.dev/github.com/microsoft/vscode-jupyter/blob/adfc442a2178eff34d9cdc5aa049acecfbc9e8a9/src/test/datascience/widgets/notebooks/k3d_widgets.ipynb#C4:L1-L9
But executing this line never returns
https://insiders.vscode.dev/github.com/microsoft/vscode-jupyter/blob/adfc442a2178eff34d9cdc5aa049acecfbc9e8a9/src/test/datascience/widgets/notebooks/k3d_widgets.ipynb#C4:L11

Thanks, but that doesn't help.
Please could you try https://github.com/microsoft/vscode-jupyter/issues/13131#issuecomment-1480149478

Tried with Python 3.10 from the Windows store, also didn't work.
No errors in dev tools or Jupyter console.
This notebook runs in the selected Python environment as confirmed with import sys; print(sys.executable)

Weirdly, if I run python3 -m pip install jupyter notebook from the CLI, I get a message saying that all packages are already installed, and if I rerun the k3d notebook, the cell still fails to complete. But if I run import sys; !{sys.executable} -m pip install jupyter notebook in a notebook cell, then the cell completes. So I think the k3d library and/or the ipywidgets library have not declared a dependency on the notebook submodule that is really required to execute that library. I don't think this is a Jupyter extension problem at all, probably something that needs fixing in k3d and/or ipywidgets.

robably something that needs fixing in k3d and/or ipywidgets.

Great, thanks for digging into this, will close this issue.
FYI: Instead of using python3 -m pip install, you need to activate the python environment you are working on and then run

python -m pip install or
Use this for virtual environments, conda envs and the like
<fully qualified path to Python> -m pip install, use the latter when dealing with globally installed python environments like Python in windows store or the like.


Add SQL Server Developer Edition

Tool information

Tool name: SQL Server Developer Edition
Tool license: License is provided free by Microsoft for development purposes
Add or update? Add
Desired version: 2016/2017/2019
Approximate size:
If this is an add request:

Brief description of tool: SQL Server database system
URL for tool's homepage:
Provide a basic test case to validate the tool's functionality: PowerShell modules or applications that want to test against an Edition that is likely to be in production environments. SQL Server Developer Edition has the same functionality as SQL Server Enterprise Edition.

Area for Triage:

Question, Bug, or Feature?:

Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?
I believe the easiest thing would be to offer images from Azure that include SQL Server Developer already installed, and then apply your build steps to that image. The code itself though for installing SQL Server can be done via PowerShell using dbatools module: Install-DbaInstance includes the code for this process if you want to strip it out from there.
Tool installation time in runtime
The average time depends on the server size, but based on the one I saw noted I would estimate around 10 minutes.
Are you willing to submit a PR?
Yes

Hello @wsmelton , this issue looks like duplicate of https://github.com/actions/virtual-environments/issues/1282, correct me if I am wrong

To some degree, yes. That is around using SQL Server Express Edition and/or SQL localdb. The Developer Edition offers more functionality that developers will want available to test against.
SQL Server Express won't be usable for my needs and purposes.

Could we consider adding SQL Server Developer edition and in tandem a newer LocalDB? Wouldn't that cover most needs? Is there any inherit benefit of using Express instead of Developer?

The footprint between Express and Developer for the Database Engine may be a little different, but not extreme. Patching is done the same, and install time is about the same as well. I've installed and configured SQL Server in about 12 minutes using PowerShell DSC. If you are using dbatools it is a bit longer, but less setup and code required.
Express Edition if folks want to include other components of SQL Server to test with (e.g. SSISDB) then Developer has to exists. They can simply add in their workflow to add the feature or component (the installer is cached in Windows for SQL Server).

Express/localDB does not support JDBC connection, Developer does

You can find the steps for installing SQL Server Express on Ubuntu here. It is used for Digital Ocean VM but can be used with other providers also.

@ghulamostafa not useful; this request is about "developer" on "windows", so a different product as well as a different OS.
for ubuntu it's really easy to just use the docker image provided by microsoft

Hello everyone, we had some internal discussion about it and decided to not pre-install SQL Server into the images due to maintenance concerns, tool size and tool popularity.

That's disappointing because it is the one blocker for us to move off Appveyor for our project.

has there been any positive news regarding this. I do not need SQL Server though I would have loved to have the IntegrationServices DLLs that come with it in the GAC... Reported at #8174 .


Fixes the content overflow in the web interface in chrome.

Fixes #609,
After the css changes:
Firefox

Chrome

The difference according to MDN may be because of

overflow-wrap
break-word
The same as the anywhere value, with normally unbreakable words allowed to be broken at arbitrary points if there are no otherwise acceptable break points in the line ... .

That is a great PR, I don't even have to make a review, I merge!
Thank you :)


Merge feature/update-nested-repository into master.

This change is

Coverage increased (+0.7%) to 92.83% when pulling 40cd1dd70ad6c6bfa2c022e90ccacb62716893be on przywartya:feature/update-nested-repository into f9645f41e11427fbc2c3caac205db261617a7aac on socialwifi:master.


Version 5.2.0 is missing on Ubuntu

I've installed Franz via apt-get and the latest available version on Ubuntu is 5.1.0-1337
Is there any chance to get 5.2 in those repos as well?

That's fine, I've left Franz anyways. Very interesting to see that the stale bot also closes this immediately. All without any reaction for 365 days!
You know what? I'd call this funny but it is so frustrating. I've been paying for the service, wait - there wasn't a service. I had other issues too and never received any feedback for anything.
Good luck with this, I can't recommend Franz for anything any longer.


!IDF_TARGET_ESP32 fails to exclude information in ESP-IDF docs for ESP32 (IDFGH-11355)

In the ESP-IDF docs, the parameter ESP_TIMER_INTERRUPT_LEVEL description has the following code:
        range 1 3 if IDF_TARGET_ESP32
        range 1 1 if !IDF_TARGET_ESP32

However, the resulting document built for the ESP32 target contains both the included and excluded lines.
ESP-IDF docs built locally exhibit the same issue.
It would be great if you could fix it.

Hi, thanks for reporting the issue, we will take a look and fix it ASAP.

The issue is gone now. Thanks a lot!


Status sampling + remove update!

This PR adds a status sampling config which allows to modify the br_status and gen_status fields per sample.
It also removes the update! methods since they are incompatible with status updates, and the performance gain isn't worth the maintenance burden since (especially with a sysimage) build time << solve time.

Closing to merge together with #118 since the old formulations error when not all components are active.


most channels not working...

most channels not working...
can you add a search button on android tv app

Fixed with https://github.com/kananinirav/Indian-IPTV-App/releases/tag/v2.1.0


Build XCTest extensions on linux

Nowadays, swift-corelibs-xctest exists, so it should be possible to use Spectre from XCTest on Linux. This change removes the #if guards around XCTest extensions, with the exception of some Xcode 12+ APIs that have not yet made it upstream.
Testing done
Builds and tests on a linux machine using the official Swift docker images:
$ docker pull swift
$ docker run -itv $PWD:/code swift swift test --package-path /code

Thanks @elliottwilliams that's awesome!

I've released this in Spectre 0.9.2

Sweet! Thanks for the quick release turnaround 🙂


Incorrect comment Task 303

Line 17 of task 303 reads:
    leds = 0xFF;   //Binary<PHONE_NUMBER>111111

should be:
    leds = 0xFF;   //Binary 11111111

should be
leds = 0xFFFF;      //Binary<PHONE_NUMBER>111111
I think it needs the 3rd F at least

Now corrected


From /kitti/oxts/gps/fix to /tf

Does anybody know where the world reference system of /tf is centered?
I am trying to convert /kitti/oxts/gps/fix (NavSatFix - lat, long, alt) in the /tf world (x, y, z) reference system.
Thanks.

The world frame origin is the location of the first GNSS fix position. For details see https://github.com/utiasSTARS/pykitti/blob/master/pykitti/utils.py#L133
https://github.com/tomas789/kitti2bag/blob/master/kitti2bag/kitti2bag.py#L58


Possible fix for hung debuggers

[Enter feedback here]
Tried debugging an app running on Ubuntu. The debugger was showing that it is attached, but nothing worked. Deleting ~/.vs-debugger folder on the target fixed the issue.

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

ID: 2ecc1202-a940-caa0-968f-9abfe2679d8b
Version Independent ID: 20ed4d8d-35ff-f1fe-74e4-ce2c27bb72c8
Content: Debug .NET Core on Linux - Visual Studio (Windows)
Content Source: docs/debugger/remote-debugging-dotnet-core-linux-with-ssh.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@TecsiAron Thanks for the suggestion! @gregg-miskelly Gregg, is this something I should add to the doc?

@Mikejo5000 We released a new version of the Linux debugger on Friday, and it looks like this issue is alerting us to a bug in the upgrade scenario. So this is an excellent work around right this second, but I am not sure it should be in the docs.

Thanks, Gregg. Closing this since there is no doc update planned.

FYI I new install script is now deployed which will correctly uninstall the old version of upgrade. So hopefully this will never be an issue again.


In cross-modal-search, 5 results are output no matter how many top_k is set.

How can i resolve it??

Hi @audrb96 , I will look into the issue, thanks.

Hey @audrb96 , sorry about the incovenience. The cross-modal-search example is currently being reworked to fix this issue.


nightly test error: profiling_camelyon_pipeline.ipynb

Describe the bug
Running ./pathology/tumor_detection/ignite/profiling_camelyon_pipeline.ipynb
Checking PEP8 compliance...
Running notebook...

Executing:   0%|          | 0/18 [00:00<?, ?cell/s]
Executing:   6%|▌         | 1/18 [00:00<00:16,  1.04cell/s]
Executing:  33%|███▎      | 6/18 [00:07<00:12,  1.06s/cell]
Executing:  39%|███▉      | 7/18 [00:08<00:10,  1.04cell/s]
Executing:  50%|█████     | 9/18 [00:18<00:19,  2.17s/cell]
Executing:  61%|██████    | 11/18 [01:28<01:24, 12.10s/cell]
Executing:  67%|██████▋   | 12/18 [01:30<00:53,  9.00s/cell]
Executing:  89%|████████▉ | 16/18 [01:30<00:12,  6.31s/cell]
Executing:  89%|████████▉ | 16/18 [01:30<00:11,  5.69s/cell]
Traceback (most recent call last):
  File "/opt/conda/bin/papermill", line 8, in <module>
    sys.exit(papermill())
  File "/opt/conda/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/papermill/cli.py", line 250, in papermill
    execute_notebook(
  File "/opt/conda/lib/python3.8/site-packages/papermill/execute.py", line 122, in execute_notebook
    raise_for_execution_errors(nb, output_path)
  File "/opt/conda/lib/python3.8/site-packages/papermill/execute.py", line 234, in raise_for_execution_errors
    raise error
papermill.exceptions.PapermillExecutionError:
---------------------------------------------------------------------------
Exception encountered at "In [7]":
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-7-e9fcc49d73ee> in <module>
     10
     11 # Normalize each transform range with total average time (percentage of transfom time)
---> 12 summary = summary[["Average", "Minimum", "Maximum", "StdDev"]] / summary["Average"].sum() * 100
     13 summary

/opt/conda/lib/python3.8/site-packages/pandas/core/frame.py in __getitem__(self, key)
   2910             if is_iterator(key):
   2911                 key = list(key)
-> 2912             indexer = self.loc._get_listlike_indexer(key, axis=1, raise_missing=True)[1]
   2913
   2914         # take() does not accept boolean indexers

/opt/conda/lib/python3.8/site-packages/pandas/core/indexing.py in _get_listlike_indexer(self, key, axis, raise_missing)
   1252             keyarr, indexer, new_indexer = ax._reindex_non_unique(keyarr)
   1253
-> 1254         self._validate_read_indexer(keyarr, indexer, axis, raise_missing=raise_missing)
   1255         return keyarr, indexer
   1256

/opt/conda/lib/python3.8/site-packages/pandas/core/indexing.py in _validate_read_indexer(self, key, indexer, axis, raise_missing)
   1302             if raise_missing:
   1303                 not_found = list(set(key) - set(ax))
-> 1304                 raise KeyError(f"{not_found} not in index")
   1305
   1306             # we skip the warning on Categorical

KeyError: "['StdDev'] not in index"

Additional context
Add any other context about the problem here.

could you please help here @drbeh, thank in advance!


refactor(client): delay the "enabled_in" to "v1.9.0" for "delete_data"

Pull Request Test Coverage Report for Build 989011061

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.58%

Totals

Change from base Build 985784494:
0.0%

Covered Lines:
4973

Relevant Lines:
5950

💛  - Coveralls


feat(app,components): add deck map SlotLabels component

Overview
adds a SlotLabels component that shows the Flex slot labels, to be used as a RobotWorkSpace child component. Uses LocationIcon as the label. according to @mmencarelli as explained in a loom video , styling is different in desktop and ODD versions of location icon, so location icon has responsive styling. x and y axis slot labels are svg flex foreign objects with hardcoded positioning relying on the standard Flex deck viewbox.
this component is in the @opentrons/components workspace for use in other workspaces like protocol-designer. to do this, LocationIcon had to be moved to components as well. as per conversation with @b-cooper , components is where other design system components now in the app directory will eventually live.
closes RAUT-511
follow-on work: add slot labels most places where Flex deck map is rendered: protocol-designer, LPC, intervention modals.
Test Plan

manually verified slot label styling and positioning

Changelog

Adds deck map SlotLabels component
Uses Flex slot labels for desktop protocol run setup and ODD deck thumbnails

Review requests

confirm the slots are showing up with responsive styling in the correct position relative to the deck map

Risk assessment
low

woohoo! it looks great and works as expected! looks like the storybook for this (RobotWorkSpace.stories) is broken though, would be great to be able to see it there too.

agreed, looks like a flex deck slot issue with the story in edge. i'll see if i can address


Unity 2019.4.40f1  Android  ： SQLite-net 1.0.0-preview10 — DllNotFoundException

Windows 11 run example success.
but package android  error:
2024/05/16 10:46:12.598 30752 30977 Debug Unity Unable to load library '/data/app/~~k3qegS3QM5QMY4EGtYaDYA==/A00.MFB.DYPY-wWHCn8tqlYuv5KY9ODpvrA==/lib/arm64/libgilzoide-sqlite-net.so', native render plugin support disabled: java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "acos" referenced by "/data/app/~~k3qegS3QM5QMY4EGtYaDYA==/A00.MFB.DYPY-wWHCn8tqlYuv5KY9ODpvrA==/lib/arm64/libgilzoide-sqlite-net.so"...

Hey @RichardsATcn, thanks for the report!
Hmm, I guess we the native build is missing the link of the math library. Should be easy to fix, I'll look into it possibly this weekend.

This is the same issue as #11.

Version 1.0.0-preview11 should work on Android now, please let me know if it works for you.
Cheers \o/


Improved documentation for the latest features

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 12 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 10 [r1] (raw file):

Use [vaadinelement] for top-level elements, that is, those having the vaadin- prefix.
If there's multiple locales, they are probably different, so rather just "multiple locales".

OK, it has support, but does the developer need to set the locale explicitly or does the element detect and use browser's locale preference automatically? If it doesn't, it would be good to describe in this section how it can be done.

docs/vaadin-date-picker-features.adoc, line 11 [r1] (raw file):
Rather use active voice here:
You can adjust the element to a specific locale with the [propertyname]#locale# property.

docs/vaadin-date-picker-features.adoc, line 12 [r1] (raw file):

If talking about a class, use [classname]#String#, but here it's perhaps better to talk about strings as concept, so just have it lower-case.
It's a bit technical and vague to talk about a locale format accepted by some library. I'd separate this into two things:
"The value should be a locale identifier given in the BCP 47 format, such as `en-gb`, `fi`, or `es`."
"The date picker supports all the locales supported by the link:http://momentjs.com/[moment.js] library. Please refer to its website for the complete list of supported locales."

docs/vaadin-date-picker-features.adoc, line 13 [r1] (raw file):

Comma after introductory clause: "Once set,"
No need to say "given"
Use [vaadinelement] here (and in other similar occurrences).

docs/vaadin-date-picker-features.adoc, line 22 [r1] (raw file):

Section names should be capitalized: Initial Position
It's a zero-level section name (=), but there may only be one such in an AsciiDoc file, so this would not even compile (without errors). If you want to have the section as a same level as the Localization section, you need to move it to a separate file.

docs/vaadin-date-picker-features.adoc, line 24 [r1] (raw file):

[vaadinelement]
Comma missing after introductory clause: "...dropdown, it..."
"selected" is rather strange here, as no one has selected anything yet. Rather talk about "initial date".

docs/vaadin-date-picker-features.adoc, line 30 [r1] (raw file):
As an API this looks a bit odd. It's not just any position, but a date, so why not talk about dates?
Also having an "initial-date" attribute looks a bit odd. I don't know what design idea is behind this or what is the convention with Polymer elements, but at least in basic HTML inputs, initial values are given with a "value" attribute, which then actually changes when user selects something different.

docs/vaadin-date-picker-overview.adoc, line 10 [r1] (raw file):

[vaadinelement]

docs/vaadin-date-picker-overview.adoc, line 11 [r1] (raw file):

I so associate "UI" as the entire UI, although in general it's true that also smaller elements of an UI have a UI. Nevertheless, I'd rather talk about "the picker" or something.
Maybe it's better to split this into two sentences. I'd also rephrase a bit: ".../tapped. It has two display modes, full-screen or desktop, which is determined according to the size of the screen."

docs/vaadin-date-picker-overview.adoc, line 12 [r1] (raw file):

I'd add: "...and then select a date."

docs/vaadin-date-picker-overview.adoc, line 14 [r1] (raw file):

Do not use contractions -> rather "The picker has a"
Does the "today" button just jump to the current month, but does not select the current day?
You could separate the cancel button to a separate sentence: ". A cancel button allows closing..."
I don't know what the buttons looks like, but maybe it should be a "today button" and "close button", no need to use quotes for those. "Today" sort of begs them more, while a "close button" is a more common concept.
Now it reads as if the "dropdown closes on the dropdown footer" – how does it "close on" there? I assume the cancel button is in the dropdown footer, so maybe rather: "A close button in the footer allows closing the dropdown."

docs/vaadin-date-picker-overview.adoc, line 21 [r1] (raw file):
Use [vaadinelement] here as well. I hope it works OK within a title

Comments from the review on Reviewable.io

Review status: 0 of 3 files reviewed at latest revision, 12 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 10 [r1] (raw file):
Done. The following sentences state that the user adjusts the locale.
Automatic locale detection has been discussed but isn't currently included and will not be included into any specific Vaadin element but possibly a separate utility i18n package.

docs/vaadin-date-picker-features.adoc, line 11 [r1] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 12 [r1] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 13 [r1] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 22 [r1] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 24 [r1] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 30 [r1] (raw file):
initialValue would indeed sound like some pre-selected value which isn't the case here. I hope the revised documentation better describes what the property is meant for.

docs/vaadin-date-picker-overview.adoc, line 10 [r1] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 11 [r1] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 12 [r1] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 14 [r1] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 21 [r1] (raw file):
Done.

Comments from the review on Reviewable.io

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 9 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 12 [r2] (raw file):

Names don't get articles, and don't want to use the lower-case name to start the sentence, hence  The [vaadinelement]#vaadin-date-picker# element

docs/vaadin-date-picker-features.adoc, line 28 [r2] (raw file):
Maybe rather: "When the [vaadinelement]#vaadin-date-picker# has no selected value and the dropdown is opened, it automatically scrolls to the current month."

docs/vaadin-date-picker-features.adoc, line 29 [r2] (raw file):

Article missing ("the ... property") and rephrase: "_You can make it scroll to another month by setting the [propertyname]#initialPosition# property to a date in that month."

docs/vaadin-date-picker-features.adoc, line 31 [r2] (raw file):
You can leave out "Note" – there's two cases and this sentence describes the other one.
What confuses me here is the ambiguity that "selected" can mean both "selected by the user" and "selected programmatically". Also, "if" is often confused with "when", in which case this expression could be interpreted: "When the user selects a value, the picker always scrolls..."
Maybe the following would make the distinction better: "When [vaadinelement]#vaadin-date-picker# already has a value selected, opening the dropdown always scrolls..."

docs/vaadin-date-picker-overview.adoc, line 21 [r1] (raw file):
There's #s missing: [vaadinelement]#vaadin-date-picker#

docs/vaadin-date-picker-overview.adoc, line 11 [r2] (raw file):
Hmm, rather use more formal "clicked or tapped".

docs/vaadin-date-picker-overview.adoc, line 12 [r2] (raw file):
Hmm, "which is determined" now refers to the two display modes.
Rather: "...desktop. The display mode to use is determined..."

docs/vaadin-date-picker-overview.adoc, line 17 [r2] (raw file):

Don't use contractions -> "It does not..."

docs/vaadin-date-picker-value.adoc, line 22 [r2] (raw file):

"this value" -> "the value"

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 9 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 12 [r2] (raw file):
Correctomundo

docs/vaadin-date-picker-features.adoc, line 28 [r2] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 29 [r2] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 31 [r2] (raw file):
Ok removed the Note

docs/vaadin-date-picker-overview.adoc, line 21 [r1] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 11 [r2] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 12 [r2] (raw file):
Done.

docs/vaadin-date-picker-overview.adoc, line 17 [r2] (raw file):
Done.

docs/vaadin-date-picker-value.adoc, line 22 [r2] (raw file):
Done.

Comments from the review on Reviewable.io

Reviewed 3 of 3 files at r3.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 12 [r2] (raw file):
I still can't see the "element" word there.

docs/vaadin-date-picker-features.adoc, line 28 [r2] (raw file):
Oh, now I did the same mistake – article for a name... Remove the "the" article from the front of the element name; you can use just the name here, as it's not in the beginning of the sentence.

docs/vaadin-date-picker-features.adoc, line 31 [r2] (raw file):
No I meant that just remove the word "Note". Sorry, a bit unclear. ;-)

Comments from the review on Reviewable.io

Review status: 2 of 3 files reviewed at latest revision, 3 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 12 [r2] (raw file):
Done.

docs/vaadin-date-picker-features.adoc, line 31 [r2] (raw file):
Done.

Comments from the review on Reviewable.io

Review status: 2 of 3 files reviewed at latest revision, 3 unresolved discussions.

docs/vaadin-date-picker-features.adoc, line 28 [r2] (raw file):
Done.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from the review on Reviewable.io


fix: BUG UI Enhancement of Image Gallery

Title and Issue number

Title :BUG UI Enhancement of Image Gallery
Issue No. :#761
Close #638

Description

I have enhanced the Image Gallery , and now it does not waste any space and the alignment looks user friendly.
Video/Screenshots (mandatory)

https://github.com/apu52/Travel_Website/assets/102676949/1abdf592-4d9c-4ee3-82ab-abc2c534b31b
Made for both light/dark mode:

Made it responsive as well :

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

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

Additional context:

##Are you contributing under any Open-source programme?

GSSoC'24

@apu52 Changes done


Connect to Kitware dockerhub organization

Create dockerhub repos for the worker and the web containers.
Add the webhooks to autobuild those containers on PR merges to main.

Alternative may be to use the Github packages and keep the docker containers there instead.


Move/migrate/close old pull requests on https://github.com/Kodeworks/liquidator-backend/pulls

Since everything is in one repository now, the pending pull requests in the liquidator-backend repository should be migrated if they are still relevant, or simply be closed.

Done.


Server Side Rendering Problem

This problem happens when using this package with webpack.
Trying to fix this problem.

Seems like fixed, but still there is a problem that widget is rendered twice.


Error warning at startup

error warning at startup. Nothing happens but the game loads slightly slower than normal totsugeki.
Does this issue happen when you don't use Totsugeki? no

Totsugeki Version and logs

Version:  1.1.1
Steps to Reproduce

start up totsugeki
its the first time it happens.

#32 suggests this is probably fixed by using a more up to date version of the program.

Worth noting that this same error occurs in version 1.2.2

I got the same error, but only the first time I ran it. It seems once I ran totsugeki after opening GGST (as the comment above suggested), it ran fine every time after that (even when running totsugeki before opening GGST).

Happened on the 3rd time I ran it. No arguments uses. I to have color and models mods installed. But those only affect in game assets.

Error
Totsugeki has encountered a fatal error.
Please report this to https://github.com/optix2000/totsugeki/issues
===================
Error: couldn't find base module for GGST-Win64-Shipping.exe
goroutine 7 [running]:
runtime/debug.Stack(0x0, 0x0, 0x0)
runtime/debug/stack.go:24 +0xa5
main.panicBox(0x123fb00, 0xc00021c070)
github.com/optix2000/totsugeki/main.go:60 +0x2d
main.main.func3.1()
github.com/optix2000/totsugeki/main.go:236 +0x5c
panic(0x123fb00, 0xc00021c070)
runtime/panic.go:965 +0x1c7
main.watchGGST(0x0, 0x1320038, 0xc000034d00)
github.com/optix2000/totsugeki/main.go:130 +0x869
main.main.func3(0xc00000dff0, 0xc00000d0bb, 0x1320038, 0xc000034d00)
github.com/optix2000/totsugeki/main.go:240 +0x8a
created by main.main
github.com/optix2000/totsugeki/main.go:231 +0x994

OK

I added a fix in 71074eefcead9bd3806ce445d98cebe0a7fc5195 to hopefully fix this once and for all.
Check out the new release v1.4.0 once it comes out later today.


Support for more sensor values

Hello,
first of all many thanks for putting together this awesome HomeAssistant integration!
I can confirm that the integration can be installed via HACS and connects correctly to the HAN port of my SmartMeterGateway.
Would it be possible to expose the rest of the OBIs values as entities in HomeAssistant?
I have attached a manual (in German) that documents the full list of OBIs values on page 27.
Handbuch-fuer-Verbraucher.pdf
I had a brief look at the source code and would be happy to help with this.

Thank you for the suggestion! Adding the values would be possible and should be doable relatively easy.
I'm currently short on time and am working on some refactorings of the component so adding more OBIS codes will probably take some time

Thanks for the quick reply!
I'd be happy to try my hand at PR, though if you're planning to refactor the code we might end up with conflicts.
Let me know what you think.

Brauche Hilfe,
wie bekomme ich das  über HACS installiert ?

I just added further documentation regarding the installation in the Readme.
Please create a dedicated issue in case you need further help.

danke fürs readme
bekomme leider eine fehlermeldung

Please add this repo as custom repository in HACS.
I'll add this to the Readme as well.

hat geklappt mit der installation .
warte auf mein password von westnetz

I have now received my Han login from the provider after 3 months. Login via the web interface works. I have entered all the data into the integration, but I am not getting any data from the counter. Is there another trick?

@dl6dbh please do not use this issue for support requests. That's not how GitHub works. Create your own issue, and maybe someone will help you there.

Hi @jacksgt,
it took way too long to refactor the initial version for better maintainability.
I just pushed #16 which should simplify the handling of additional values.
Could you send me a list of values that you'd like to have supported? I'll add them to the exposed sensors then :)

@jannickfahlbusch Thanks for your work! I put together a quick prototype at https://github.com/jannickfahlbusch/ha-ppc-smgw/pull/20 , though unfortunately I cannot test it currently because my HAN port credentials don't work anymore / changed.
I'll let you know once I get the opportunity to test it locally :-)

mit der Version 0.0.8 wird "import total" angezeigt."export Total" ist allerdingst unbekannt.

vielleicht hat westnetz 2.8.0 ( einspeisung) nicht freigeben ?

Gruß und schöne feiertage
Meinolf

ich denke westnetz hat 2.8.0 nicht ihm zähler freigeschaltet .
oder gibts da noch einen einstellungstrick ?

vg
Meinolf

2024-12-23 16:53:16.547 INFO (MainThread) [custom_components.ppc_smgw] Got cookie response, assuming we are logged in2024-12-23 16:53:16.547 INFO (MainThread) [custom_components.ppc_smgw] Requesting meter readings2024-12-23 16:53:17.494 INFO (MainThread) [custom_components.ppc_smgw] Got meter readings, parsing...2024-12-23 16:53:18.493 INFO (MainThread) [custom_components.ppc_smgw] Found 2 rows2024-12-23 16:53:18.494 DEBUG (MainThread) [custom_components.ppc_smgw] Parsing row: <td id="table_metervalue2024-12-23 16:53:18.495 DEBUG (MainThread) [custom_components.ppc_smgw] Found timestamp: 2024-12-23 16:45:012024-12-23 16:53:18.495 INFO (MainThread) [custom_components.ppc_smgw] trying to log out2024-12-23 16:53:19.267 DEBUG (MainThread) [custom_components.ppc_smgw] Got response: <Response [200 OK]>Content: b'\r\nSmart Meter Gateway Customer Interface\t\t        <meta http-equiv="Cache-control" content="no-cache2024-12-23 16:53:19.268 INFO (MainThread) [custom_components.ppc_smgw] Found 1 readings2024-12-23 16:53:19.268 DEBUG (MainThread) [custom_components.ppc_smgw] Readings:{'1-0:1.8.0': Reading(value='445.8801', unit='kWh', timestamp=datetime.datetime(2024, 12, 23, 16, 45, 1), isvalid='1', name='Elektro Wirkarbei2024-12-23 16:53:19.268 DEBUG (MainThread) [custom_components.ppc_smgw.coordinator] Finished fetching ppc_smgw data in 12.519 seconds (success2024-12-23 16:53:19.269 INFO (MainThread) [homeassistant.components.button] Setting up ppc_smgw.button2024-12-23 16:53:19.270 DEBUG (MainThread) [custom_components.ppc_smgw.entity] Initializing restart_gateway. EntryID: 01JFT085MY3FQK3WWPQSY5RN2024-12-23 16:53:19.270 DEBUG (MainThread) [custom_components.ppc_smgw.button] Adding button entities: 2024-12-23 16:53:19.275 INFO (MainThread) [homeassistant.components.sensor] Setting up ppc_smgw.sensor2024-12-23 16:53:19.275 DEBUG (MainThread) [custom_components.ppc_smgw.entity] Initializing 1-0:1.8.0. EntryID: 01JFT085MY3FQK3WWPQSY5RNHV2024-12-23 16:53:19.276 DEBUG (MainThread) [custom_components.ppc_smgw.entity] Initializing 1-0:2.8.0. EntryID: 01JFT085MY3FQK3WWPQSY5RNHV2024-12-23 16:53:19.278 DEBUG (MainThread) [custom_components.ppc_smgw.sensor] Data: Information(readings={'1-0:1.8.0': Reading(value='445.8802024-12-23 16:53:19.279 DEBUG (MainThread) [custom_components.ppc_smgw.sensor] Data: Information(readings={'1-0:1.8.0': Reading(value='445.8802024-12-23 16:53:19.279 DEBUG (MainThread) [custom_components.ppc_smgw.sensor] Found no value for 1-0:2.8.0


Expose destinationState with property

I suggest to expose also destinationState.
This would be useful if you want to do different things in _Exit() depending on your next state.
Example:
 private IEnumerator SOMETHING_Exit()
    {
        if (_fsm.DestinationState == States.IDLE)
        {
           Debug.Log("Wish you fun during idle");
        } else
       {
           Debug.Log("Come back again");
       }
   }

So, I would be happy if you could add these lines:
 public T DestinationState
        {
            get
            {
                if (destinationState == null) return default(T);

                return (T)destinationState.state;
            }
        }

Thank you

This is now exposed as a NextState property


NIFI-9403: Deprecating FetchElasticsearch processors in favor of GetE…

…lasticsearch
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
One of the last steps in modernizing the Elasticsearch processors by preferring use of the ElasticsearchClientService is to deprecate the FetchElasticsearch processors now that GetElasticsearch is available.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks for the contribution @gresockj, will merge (hopefully)


Build/watch error undefined

Hey,
I would love to use your boilerplate for developing Figma plugins. However, I run into an undefined error during the build or watch scripts and can't get it to work. Omitting the last --yes parameter does not make a difference.
I hope you can help me out!
My environment:

Node: v14.17.4
NPM: 6.14.14
Yarn: 1.22.11

Terminal output:
$ npx --yes -- create-figma-plugin my-plugin
npx: installed 129 in 6.354s
info Scaffolding a new plugin...
? display name My Plugin
? template ui
info Copying "ui" template...
info Installing dependencies...
success Done
$ cd my-plugin
$ yarn run build
yarn run v1.22.11
warning package.json: No license field
$ build-figma-plugin --typecheck --minify
error undefined
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Unfortunately I can’t seem to replicate the issue you’re facing. This is my terminal output:
$ npx --yes -- create-figma-plugin my-plugin
info Scaffolding a new plugin...
? display name My Plugin
? template ui
info Copying "ui" template...
info Installing dependencies...
success Done
$ cd my-plugin
$ yarn run build
yarn run v1.22.10
warning package.json: No license field
$ build-figma-plugin --typecheck --minify
info Type checking...
success Type checked in 3.138s
info Building...
success Built in 1.100s
✨  Done in 5.05s.

I was afraid that was the case... Your boilerplate seemed so easy to use, so I was eager to test it out myself! I get the same error on both my iMac and Macbook Pro, unfortunately.
The error itself is not really helpful either, so I'm not sure where to look for the solution. Could it be some generic setup error that causes this, maybe?

I suspect it could be an issue in the build-figma-plugin CLI.

It’d help if you can try to identify where exactly the error is occurring. Potentially you could try to add console.log statements in the<EMAIL_ADDRESS>file to trace where the error is being thrown? It looks like the info Type checking... line is not being printed in your terminal output, which suggests that the error occurs before this specific line in the build-async.js file: https://github.com/yuanqing/create-figma-plugin/blob/78d944331293b0442bf87c41161ec026425e9b70/packages/build/src/build-async.ts#L18

Alternatively, you can try to build the entire project locally and symlink to the local copy of the project in your plugin, and then try to debug the source of the error from there. Instructions are in the README.

I’ll consider adding a --verbose flag to the CLI to help with debugging such issues in the future.

Ha! Interesting... Your first approach helped me resolve the issue.
I found that the buildCssModulesTypingsAsync function returned an error. This caused the build-async.js script to jump straight to the catch clause. Fortunately, the error message revealed helpful information for me to successfully fix the issue.
The error was caused by the name of the parent folder, which contained spaces ("Figma plugins/my-plugin"). Renaming that folder resolved the issue.
Thanks a lot for your time and suggestions!

PS: A --verbose would be a welcome addition ;)

Great! Glad to know it worked out!

The error was caused by the name of the parent folder, which contained spaces ("Figma plugins/my-plugin"). Renaming that folder resolved the issue.

Just cut a 1.2.4 release that fixes this specific issue.


Preserving inline tables

Hey @uiri, are there any plans to support preserving inline tables in the dumps function. If not would you be open to a PR implementing such functionality?
Right now, I have a project with users supplying TOML, often using the inline tables syntax:
value = {extra = ["values",]}
While loads handles this just fine, serializing back to TOML creates a subsection. This is creating maintainability problems, in some cases, since things are moving around in the TOML file unexpectedly. It would be nice if the data structure was written back out as it was ingested, if possible.
Thanks for your consideration!

Hi Nate,

Hey @uiri, are there any plans to support preserving inline tables in the dumps function. If not would you be open to a PR implementing such functionality?

I do not have any immediate plans to add this support. However, I am always open to pull requests.

While loads handles this just fine, serializing back to TOML creates a subsection. This is creating maintainability problems, in some cases, since things are moving around in the TOML file unexpectedly. It would be nice if the data structure was written back out as it was ingested, if possible.

I would recommend calling loads with the _dict parameter set to an OrderedDict. This (or another custom dictionary type) might help your issue. If you still find the need to make changes to the toml code, I would recommend adding a boolean flag to dumps. That way the current behaviour can stay as the default.

I would recommend calling loads with the _dict parameter set to an OrderedDict. This (or another custom dictionary type) might help your issue.

Unfortunately, we're already doing that to no avail.

I would recommend adding a boolean flag to dumps. That way the current behaviour can stay as the default.

Yep, I was thinking a preservation flag for opt in, so it won't affect current users. I'll take a look into setting that up, thanks!

Fixed at 87993ce (I believe)


Hebrew fails with error 500

It doesn't matter which text you enter, they all fail, for example:
http://localhost:8999/api?text=hello&from=en&to=he
{"statusCode":500,"error":"Internal Server Error","message":"Evaluation failed: TypeError: Cannot read properties of undefined (reading 'parentElement')\n    at isInRecentScope (pptr://__puppeteer_evaluation_script__:6:36)\n    at pptr://__puppeteer_evaluation_script__:22:13"}

All other languages seem to be working OK. It might be because Hebrew is RTL?

Nevermind, seems Google is now using iw code on site for hebrew. Website also maps he correctly.

It might be because Hebrew is RTL?

I'm not sure. I haven't considered RTL problem but it's fine with arabic language IIRC.

seems Google is now using iw code on site for hebrew

What is iw code? Could you give me some clues so I can fix this?

The language code for Hebrew is now "iw" on Google Translate, it used to be "he". If you pass in "iw" to Translateer it's working correctly.

Ah, I see. Thanks for the helpful information for anyone else who may come across this issue.


CGA-can't open assembly part

When I open a recent CGA job, I can click on the little flag for annotation and it will overwrite the page to show the results of the annotation job.  when I click on the little flag for assembly, it doesn't open.  I am trying to figure out what assembly program was selected when I chose Auto, and I can't if this won't open

To replicate:

Go to this job result view for a comprehensive genome analysis:
<EMAIL_ADDRESS>Workshop/Comprehensive Genome Analysis/Buchnera aphidicola strain Tuc7

Click on the "annotation" job result.

Note that the navigation works and everything looks fine.

Hit the back button, and then try clicking the "assembly" job result.

Notice that the url changes, but the view does not.  If you refresh the page now, you should get the correct view:

However, also note that the back button doesn't work / has no history.
So, it seems like maybe a routing/url-state issue... but pretty peculiar since the routing for annotation works fine.

WorkspaceBrowser.js is not setting a viewer for GenomeAssembly at line 1365.

ah, thanks.  This seems prone to bugs... and not ideal since it works when loading the view directly.   Do you want to fix?  Perhaps at least add a console.error if there's no mapping?

I am working on a fix and investigating related behavior.  For example, clicking on the parent folder from an annotation view does not go back to the Comprehensive Genome Analysis view.

From: nconrad<EMAIL_ADDRESS>Sent: Saturday, May 16, 2020 12:17 AM
To: PATRIC3/patric3_website<EMAIL_ADDRESS>Cc: Porter, Jacob S (jsp4cu<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [PATRIC3/patric3_website] CGA-can't open assembly part (#2354)
ah, thanks. This seems prone to bugs... and not ideal since it works when loading the view directly. Do you want to fix? Perhaps at least add a console.error if there's no mapping?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/2354#issuecomment-629585547, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AALWRUOOITMWOY6JO4DR3CDRRYHT3ANCNFSM4MQCSZRQ.


Better looking Readme

I can improve the look of readme and make it more atteactive

The readme file is already looking good and contains all the required information.

Is this Project participate in Hacktoberfest 2023??

This place is not meant for asking questions.
Yes, this project is participating in Hacktoberfest 2023.
If you want more information, please check the project's README
And if you want to contribute, refer to CONTRIBUTING.md.

thanks @sohansai


Add prime numbers example

This is a great little example of how to use flux::generator, thank you!


Api unable to get event name from jsdoc @event

I think it is a good solution  to force recognition of an event that is not a litteral but has the @event JSDoc.
in previous version the library was able to get event name from jsdoc so It could be a regression instead of and enahance?

related to https://github.com/vue-styleguidist/vue-styleguidist/issues/279


should consider https://github.com/eclipse/che/ for dev workflow?

this could fit our vision for the dev platform along with teracy-dev.

related: https://github.com/wernight/kubernetes-che/

not relevant anymore, close now.


Scaffolder: permissioning actions

Hey, I just made a Pull Request!
This PR is a follow up to https://github.com/backstage/backstage/pull/15798 where we introduce permissions for templates in the scaffolder.
In this PR, we introduce permissions for actions, which can be used as a security layer in case adopters have the need to restrict actions to certain users.
Example: the following permission policy will disallow the debug:log action.
class MyPermissionPolicy implements PermissionPolicy {
  async handle(
    request: PolicyQuery,
    user?: BackstageIdentityResponse,
  ): Promise<PolicyDecision> {
    if (isPermission(request.permission, actionExecutePermission)) {
      return createScaffolderActionConditionalDecision(request.permission, {
        not: scaffolderActionConditions.hasActionId({
            actionId: 'debug:log'
        })
      });
    }

    return {
      result: AuthorizeResult.ALLOW,
    };
  }
}

Example: the following permission policy will disallow passing message: Test as an input to the debug:log action. Any other input passed to debug:log will be allowed.
class MyPermissionPolicy implements PermissionPolicy {
  async handle(
    request: PolicyQuery,
    user?: BackstageIdentityResponse,
  ): Promise<PolicyDecision> {
    if (isPermission(request.permission, actionExecutePermission)) {
      return createScaffolderActionConditionalDecision(request.permission, {
        allOf: [
          scaffolderActionConditions.hasActionId({
            actionId: 'debug:log'
          }),
          scaffolderActionConditions.hasStringProperty({
            key: 'message',
            value: 'Test',
          })
        ],
      });
    }

    return {
      result: AuthorizeResult.ALLOW,
    };
  }
}

The actions in a template are authorized one by one in the workflow runner. The execution of the template ends once an action is not authorized and a proper error is displayed in the frontend:

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Nice @vinzscam! Nothing that stands out to me, is docs something that is shipped as part of this PR or later on?

Here is a draft about the docs https://github.com/backstage/backstage/pull/17379


WIP Nested angle bracket components error

Addresses #359. Note that this is not in the context of Ember but rather standalone Glimmer.
This works:
{{#my-component}}
  {{#my-other-component}}
    Hello!
  {{/my-other-component}}
{{/my-component}}

This results in an error:
<my-component>
  <my-other-component>
    Hello!
  </my-other-component>
</my-component>

TypeError: Cannot read property 'scan' of null
    at BlockScanner.addStatement (http://localhost:7357/amd/glimmer-runtime.amd.js:6937:54)
    at OpenElement.tagContents (http://localhost:7357/amd/glimmer-runtime.amd.js:8200:25)
    at OpenElement.scan (http://localhost:7357/amd/glimmer-runtime.amd.js:8145:22)
    at BlockScanner.addStatement (http://localhost:7357/amd/glimmer-runtime.amd.js:6937:55)
    at BlockScanner.scan (http://localhost:7357/amd/glimmer-runtime.amd.js:6904:22)
    at buildStatements (http://localhost:7357/amd/glimmer-runtime.amd.js:6885:71)
    at Scanner.scanEntryPoint (http://localhost:7357/amd/glimmer-runtime.amd.js:6850:27)
    at asEntryPoint (http://localhost:7357/amd/glimmer-runtime.amd.js:9104:51)
    at Object.render (http://localhost:7357/amd/glimmer-runtime.amd.js:9117:28)
    at render (http://localhost:7357/amd/glimmer-tests.amd.js:3991:27)

My understanding of the problem:

<my-component> kicks off a scan of its children in OpenElement#tagContents.
First child is <my-other-component>, which also kicks off a scan of its children.
The <my-other-component> scan completes with </my-other-component>.
At this point the <my-component> scan's nesting variable (in OpenElement#tagContents) is at a value of 2. It started at one and was incremented when it hit <my-other-component>, so 2.
We now hit </my-component>, but because nesting is at 2 and not 1 we cannot decrement to 0 and break the loop; thus we continue…
The BlockScanner instance is out of statements, so now a call to #next results in null.
scanner.addStatement(null)
💥

The once failing test now passes thanks to https://github.com/tildeio/glimmer/pull/376 🎉

@robbiepitts hm, I'm missing something here. If you're saying that nested angle bracket component don't work, shouldn't the build fail?

@locks Yes, when I originally opened the PR the test failed. But with #376 the test passes, so this PR just adds a passing test now. I updated the title of the PR and the commit message to reflect this.


Issue with OPT-ONNX Transformer Example

Hello,
I am having issues running the OPT transformer example in the ONNX quantization flow. All the steps for setup are completed without issue, and the quantized model is run on the CPU successfully; however, when running with AIE flag for the NPU the program crashes during the warm-up stage of run.py:

I am running on the Minis Forum U790 Pro Venus series provided by Hackster and AMD for the Pervasive AI Developer Contest. I have discussed the issue with other's in the Hackster discord and a few others seems to be running into the same issue. The results can be reproduced by following the transformer example. The OPT example using the PyTorch quantization flow seems to work, and I will try that for my custom model.
Thank you!

The OPT example using the PyTorch quantization flow seems to work, and I will try that for my custom model.
Do you run the OPT example using PyTorch as described and see NPU utilization for the PyTorch flow? For me it works but without NPU.

Hi @pyuvaraj37 ,
We have seen this issue recently. We need to do an update to fix this issue with OPT-ONNX.
You can run OPT PyTorch flow, which works correctly.

Thank you. Yes I was able to use the PyTorch flow with my own transformer as well.

Good to know, closing this issue


Enable R8 for all sample apps

This enables R8 (in full mode) for all sample apps.
The version of kotlinx.coroutines is bumped to 1.4.1 to fix some warnings about missing classes (https://github.com/Kotlin/kotlinx.coroutines/commit/d2ed1d809805d9eadd8e8379427971f0e14380ce).
The version of OkHttp is bumped to 4.9.0 in Jetcaster to allow repackaging into the top-level (https://github.com/square/okhttp/pull/6094/commits/e18aa1aadf0c4e4f39e9ce1d4fedd5b44dd66a11).

I changed branch to the lates dev branch (per the contribution guide) but looks like there are some conflicts.

Sorry @christofferqa, looks like this was auto-closed once we merged in parent branch. I can't re-open the PR for some reason, so manually re-opened in #284


ArcRotateCamera: Modified zoomOn to use same logic as FramingBehavior

A user on the forum found that the zoomOn function was found to be incorrectly determining radius for zooming on provided meshes.  This PR contains a change that takes some of the logic used for the ArcRotateCamera framing behavior and uses that to determine how far to zoom.
Forum Link: https://forum.babylonjs.com/t/arcrotate-camera-zoomon-not-working-as-expected/38130/1

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14387/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14387/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14387/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14387/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14387/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14387/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14387/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14387/merge#BCU1XR#0


image path is not resolve in DOM with vue template runtime compiler

Version
3.1.1
Reproduction link
Playground
Steps to reproduce
<template>
  <component :is="`tab`" />
</template>

<script>
import { defineAsyncComponent } from "vue";

export default {
  setup() {
    return {};
  },
  components: {
    tab: defineAsyncComponent(
      () =>
        new Promise((resolve, reject) => {
          resolve({
            template: `<img class="h-12"<EMAIL_ADDRESS>alt="" />`,
          });
        })
    ),
  },
};
</script>

//vite.config.js

What is expected?
the image path is resolved in DOM according to the alias @ and the public path
What is actually happening?
the image path is not resolved in DOM based on the alias @ and the public path

Such aliases are a build time feature. They can not work in the runtime compiler because the compiler doesnt know about them - that's the build tool's job (here:vite)

Hint: you can import the image in JS and then insert the path you get from that import to your runtime template.
I suggest you use chat.vuejs.org to ask for more help.

@LinusBorg Hence, its should be a feature request for vite js or vue 3 js?

I don't see a reason for a compiler feature here. This should be enough for most situations:
{
 template: `<img class="h-12" src="${img('./assets/logo.png')}" alt="" />`,
}

and define img() as a central helper function somewhere in your code.
If you want to continue arguing for such a feature, you should do so in our /rfc repository where feature requests can be discussed and then go through the RFC process.

@LinusBorg, Could you please clarify what can be in img() helper function? I couldn't see a way to resolve dynamic paths in either template or functions.

I don't see a reason for a compiler feature here. This should be enough for most situations:
{
 template: `<img class="h-12" src="${img('./assets/logo.png')}" alt="" />`,
}

and define img() as a central helper function somewhere in your code.
If you want to continue arguing for such a feature, you should do so in our /rfc repository where feature requests can be discussed and then go through the RFC process.


formats links and references in RFC style

See http://tools.ietf.org/html/rfc2223#section-8

going to close as the current handling of references and links seems to be RFC style


Very quick question

Hi,
So I have an elixir app, but for reasons beyond my control I've been asked if the app can be deployed in a linux distro without a shell (bash, sh etc).
Calling: ./burrito_out/my_app_linux directly.
Does Burrito assume a shell exists?
Thanks

The only things Burrito needs for linux are the following:

tempfs mounted at /tmp to extract its static musl runtime (only needed if you use our pre-compiled ERTS)
write access somewhere to deploy the app payload, you can override where this extraction takes place with an environment variable, which is computed based on the name of your app: https://github.com/burrito-elixir/burrito/blob/efd6a329f26e4039e7d46d00e571dc1df5b69262/src/wrapper.zig#L161

(In the case of the :example_cli_app app the env variable would be EXAMPLE_CLI_APP_INSTALL_DIR=...
Overall I think this is entirely doable, if you were to set up your linux system properly and have the burrito app be the first app that runs, everything should work as expected.

Many thanks


Add connectToNodeAsExtension in TransportService

Description
Companion PR in sdk repo: https://github.com/opensearch-project/opensearch-sdk-java/pull/619
Companion PR in security repo: https://github.com/opensearch-project/security/pull/2599
Adds new methods inside TransportService for extension to connect to opensearch as an extension. When performing the handshake for security, the extension needs to identify itself as an extension. This PR is modeled after cross cluster functionality where the node identifies which cluster it belongs to using the _opendistro_security_remotecn ThreadContext header, for this PR I chose to have the extension identify its request using the extension_unique_id thread context header which the security plugin needs to permit the handshake and other transport requests used in extension bootstrap process.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #6866 (0003dcc) into main (805e6d9) will increase coverage by 0.00%.
The diff coverage is 0.00%.

:exclamation: Current head 0003dcc differs from pull request most recent head 4d87934. Consider uploading reports for the commit 4d87934 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##               main    #6866   +/-   ##
=========================================
  Coverage     70.64%   70.64%
- Complexity    59160    59218   +58
=========================================
  Files          4812     4814    +2
  Lines        283750   283761   +11
  Branches      40915    40916    +1
=========================================
+ Hits         200452   200476   +24
+ Misses        66817    66805   -12
+ Partials      16481    16480    -1

Impacted Files
Coverage Δ

...ava/org/opensearch/transport/TransportService.java
80.18% <0.00%> (-2.34%)
:arrow_down:

... and 493 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Hi @saratvemulapalli, from what I can tell in the code. For cross-cluster use-cases, the security plugin enriches all outgoing transport requests with the cluster name:

https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/transport/SecurityInterceptor.java#L132

When receiving the request, the Security plugin first confirms if the request is coming from a trusted node by comparing the distinguished name from the node to the full list of node_dns:

https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/transport/DefaultInterClusterRequestEvaluator.java#L82-L102

If its coming from a trusted node, it then goes on to compare the cluster name that the node belongs to:

https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/transport/SecurityRequestHandler.java#L311-L327

With the method (connectToNodeAsExtension ) added here, I proposed adding a threadcontext header extension_unique_id on the handshake with the OpenSearch node.
The call to extensionTransportService.connectToNode(extensionsRunner.opensearchNode); in ExtensionsInitRequestHandler will not succeed with the security plugin installed because it initiates a transport handshake action which the security plugin forbids since it originates from outside the cluster.
The companion Security plugin PR checks for the extensions_unique_id threadcontext header and permits the request as coming from a trusted source even though it is not a node in the cluster.
Thank you for reviewing this! I wanted to get a PR out to show the problem with secure communication between OpenSearch nodes and an extension to get more eyes on the problem.

Hi @saratvemulapalli, from what I can tell in the code. For cross-cluster use-cases, the security plugin enriches all outgoing transport requests with the cluster name:

https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/transport/SecurityInterceptor.java#L132

When receiving the request, the Security plugin first confirms if the request is coming from a trusted node by comparing the distinguished name from the node to the full list of node_dns:

https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/transport/DefaultInterClusterRequestEvaluator.java#L82-L102

If its coming from a trusted node, it then goes on to compare the cluster name that the node belongs to:

https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/transport/SecurityRequestHandler.java#L311-L327

With the method (connectToNodeAsExtension ) added here, I proposed adding a threadcontext header extension_unique_id on the handshake with the OpenSearch node.
The call to extensionTransportService.connectToNode(extensionsRunner.opensearchNode); in ExtensionsInitRequestHandler will not succeed with the security plugin installed because it initiates a transport handshake action which the security plugin forbids since it originates from outside the cluster.
The companion Security plugin PR checks for the extensions_unique_id threadcontext header and permits the request as coming from a trusted source even though it is not a node in the cluster.
Thank you for reviewing this! I wanted to get a PR out to show the problem with secure communication between OpenSearch nodes and an extension to get more eyes on the problem.

Thanks @cwperks , this really helps.

The changes are good, could you take care of gradle check.

@saratvemulapalli The CI is passing now.


Incorrect paths in config

From @monolithed on October 19, 2016 17:53
Config
{
   seleniumLogs: '/Users/a.abashkin/workspace/mail.ru/e.mail.ru/tests/cache/logs'
}

Result
/Users/a.abashkin/workspace/mail.ru/e.mail.ru/Users/a.abashkin/workspace/mail.ru/e.mail.ru/tests/cache/logs

Expected
/Users/a.abashkin/workspace/mail.ru/e.mail.ru/tests/cache/logs

Copied from original issue: webdriverio/webdriverio#1656

Are you using the selenium-standalone service? If yes I think we should file the issue there.

From @monolithed on October 19, 2016 18:19
Yep, selenium-standalone

Should be as simple as changing:
let absolutePath = path.join(process.cwd(), filePath)

to
let absolutePath = path.resolve(filePath)

in getFilePath.js.


EVEREST-574 Upgrade Everest

CHANGE DESCRIPTION
Problem:
EVEREST-574
Short explanation of the problem.
Now the CLI can upgrade everest automatically.
CHECKLIST
Jira

[x] Is the Jira ticket created and referenced properly?

Tests

[ ] Is an Integration test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Codecov Report

Merging #210 (993293e) into main (aeec43a) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #210   +/-   ##
=====================================
  Coverage   5.38%   5.38%
=====================================
  Files         13      13
  Lines       2805    2805
=====================================
  Hits         151     151
  Misses      2627    2627
  Partials      27      27

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:20: cython_must_be_in_host: Cython should be in the host section
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:28: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:42: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:50: missing_description: The recipe is missing a description
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:50: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_1azmupxgv_/clone/recipe/meta.yaml:50: http_url: http://ctruong.perso.math.cnrs.fr/ruptures-docs/build/html/index.html is not https
Errors were found


Issue replicating "huge-apps" example on latest beta3

I have been trying to replicate the huge-apps example but it fails when require.ensure gets called with this error:
TypeError: require.ensure is not a function
I checked the code and "ensure" is undefined.
I have loaded the polyfills with:
<script src="https://cdn.polyfill.io/v1/polyfill.min.js"></script>

But not making any difference.
I moved from using browserify to webpack and having different issues:
Unexpected reserved word
You may need an appropriate loader to handle this file type.
| import React from 'react';
| import { history } from 'react-router/lib/BrowserHistory';
| import { Router } from 'react-router';
This is the code that fails using browserify:
module.exports = {
    path: 'my-path',

    getComponents (cb) {
        require.ensure([], (require) => {
            cb(null, require('./components/MyComponent'))
        })
    }
}

Would appreciate any tip to fix this problem.
Thanks

For anyone having the same issue, this was the solution using webpack on laravel:
var elixir = require('laravel-elixir');
require("laravel-elixir-webpack");

elixir(function(mix) {
    mix.webpack('main.js', {
        module: {
            loaders: [
                { test: /\.js$/, loader: "babel-loader" }, // use this loader
            ]
        },
        outputDir: "public/js",
        output: {
            filename: "main.js",
            publicPath: '/js/'        // Don't forget this path, this is the base path to load files from the public folder
        },
        resolve: {
            extensions: ["", ".js"] // use this extension, with any other you have
        }
    });
});

Docs in beta3 link is broken:
https://rackt.github.io/react-router/tags/v1.0.0-beta3.html#Advanced Usage
https://rackt.github.io/examples/huge-apps


Added deploy to S3 option+repo

Other files are formatting auto-changes; only real change here is to the pom.xml
Added the following S3 based maven repos for geowave:
	<distributionManagement>
		<repository>
			<id>geowave-maven-releases</id>
			<name>GeoWave AWS Release Repository</name>
			<url>s3://geowave-maven/release</url>
		</repository>
		<snapshotRepository>
			<id>geowave-maven-snapshots</id>
			<name>GeoWave AWS Snapshot Repository</name>
			<url>s3://geowave-release/snapshot</url>
		</snapshotRepository>
	</distributionManagement>

mvn deploy will now push to releases or snapshots (based on if ${project.version} contains "SNAPSHOT" in the string).
The repo is publicly viewable, but in order to publish you need the access keys - you can put them in your ~/.m2/settings.xml - here is the skeleton fragment:
  <servers>
    <server>
      <id>geowave-maven-releases</id>
      <username>ACCESS_KEY_ID</username>
      <password>SECRET_ACCESS_KEY</password>
    </server>
    <server>
      <id>geowave-maven-snapshots</id>
      <username>ACCESS_KEY_ID</username>
      <password>SECRET_ACCESS_KEY</password>
    </server>
  </servers>

Issue: #315

Coverage decreased (-0.01%) to 32.86% when pulling 34591dc96674b882c53bfb6b6c9f335a0aa1f49f on GEOWAVE-315 into<PHONE_NUMBER>ffc710216c49a00fed3d51e01399bc on master.

Removed the fragments from the repositories tag (no point here) - we should add the location to the docs though - it's
		<repository>
			<id>geowave-maven-releases</id>
			<name>GeoWave AWS Release Repository</name>
			<url>http://geowave-maven.s3-website-us-east-1.amazonaws.com/release</url>
			<releases>
				<enabled>true</enabled>
			</releases>
			<snapshots>
				<enabled>false</enabled>
			</snapshots>
		</repository>
		<repository>
			<id>geowave-maven-snapshot</id>
			<name>GeoWave AWS Snapshot Repository</name>
			<url>http://geowave-maven.s3-website-us-east-1.amazonaws.com/snapshot</url>
			<releases>
				<enabled>false</enabled>
			</releases>
			<snapshots>
				<enabled>true</enabled>
			</snapshots>
		</repository>

Coverage decreased (-0.01%) to 32.86% when pulling 2d9cca1b0fb619466b64c7561e1fc7724e1495b7 on GEOWAVE-315 into<PHONE_NUMBER>ffc710216c49a00fed3d51e01399bc on master.


workaround to build uber pacakge in build.cmd using shade-plugin

Without breaking debug mode in IntelliJ

LGTM


Update free-courses-en.md

Added a course on C programming.
What does this PR do?
Add resource(s)
For resources
Description
This is a beginner-friendly course that helps to cultivate a greater understanding of the topic.
Why is this valuable (or not)?
This course covers all the important concepts of C programming language and helps to grasp the fundamentals in an easier way.
How do we know it's really free?
This is a free course on YouTube.
For book lists, is it a book? For course lists, is it a course? etc.
This is a free course on Youtube.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please fix the lint errors. Click on details on the failed test to know the reason for failure and fix it. Read Contributing.md for more information


demo on the bids apps website

hey
quick question from a bids maintainers: would it make sense to create a mini app using snakebids and add it as an example / template on the bids app repo ?
we have one template already there but I am thinking that we could have a second one that has some better BIDS practices
do you think that snakebids is stable enough to be put out there?

That's a great idea! Yes we can put something together for the bids app repo, I am thinking one that just replicates what the existing bids app example does would be useful for learning.

That was exactly my thoughts too. Fantastic.

Took an initial stab at it here:
https://github.com/akhanf/example-snakebids
Let me know what you guys think, and PRs are welcome of course

I updated the example app again, @tkkuehn @kaitj @pvandyken please have a look when you can.
FYI I also made the simpler example (not bids app) workflow we talked about, that is here: https://github.com/akhanf/example-snakebids-simple

@akhanf thanks for all this.
Will try to review and give you suggestions. Apparently I am better at opening issues than reviewing PRs. 🙈

That would be super, thanks Remi! Actually the PRs on that repo are merged so you can still work to your strengths and add any issues there! 😉


Migration from CLS to CLS-hooked time outs requests

I don't know how to describe this issue and hope that you'll give me some hints of what is going on. Any, really any hints are highly appreciated as I can't debug the app to give me any useful info whatsoever. (It seems like a limit in my skills. :)
Background
Since we are using async/await, in our logging lib we have migrated from CLS to CLS-hooked. It fixed the context and we could reach any values needed again. However, when running tests, they started to time-out and the overall app started to react strangely.
All the migration meant only replacement of const continuationStorage = require('continuation-local-storage') // v3.2.1 for const continuationStorage = require('cls-hooked') // v4.2.2.
Node Versions tested
We have tested against these versions:

Node 10.14.2
Node 11.2.0

How do we use CLS?

This file takes care of context creation. An excerpt:

const continuationStorage = require('cls-hooked')

const createNamespace = continuationStorage.createNamespace
const getNamespace = continuationStorage.getNamespace

const NSP_REQUEST = 'logzio-node-debug-request'
const KEY_MDC = 'mdc'
const requestNamespace = createNamespace(NSP_REQUEST)

function createContext(next) {
  const mdc = getAll()
  const mdcCopy = Object.assign({}, mdc)
  requestNamespace.run(() => {
    requestNamespace.set(KEY_MDC, mdcCopy)
    next()
  })
}

We create a new context for every incoming request. We're using socket.io

// MDC i
const MDC = require('logzio-node-debug').MDC

socket.use((packet, next) => {
      MDC.createContext(() => {
        MDC.put('key', 'value')
        next()
      })
    })

We also put other values during code execution:

const MDC = require('logzio-node-debug').MDC
MDC.put('key2', 'value2')

Question
Is this a bug? Are we doing something wrong?

I managed to identify that the problems happen only with database tests. We are using mysqljs/mysql. Is it possible that it handles queueing improperly?


chore(STADTPULS-485): Deactivate signup

This PR removes the link to the signup page on the login page.

Cool! Thanks also for the TODO reminders. Can you also add this as a Jira ticket, just so it doesn't get lost?

There is one: https://citylab.atlassian.net/browse/STADTPULS-485

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


API key is not set

Hi Nikan,
Thanks for the api.
I'm getting "API key is not set" error while trying to use db.get_bots_lists(), even though I've set the API Key.
Full message:

101

API key is not set.

The code:
import debot
db = debot.DeBot('zUr0hNi8koEQ.................')
db.get_bots_list('2017-12-04')

Hi,
Our servers are down for maintenance. We will back to service in a couple of days. Thanks for your patience.


Having Image in Dark Theme while turning an Widget to Image

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Having a Widget in my project
Wrapping in around RepaintBoundary Class along with a key property
Calling the necessary function to turn it into image from widget
Then i am using social_share package to share it to different apps where i am seeing the UI issue

Expected results

Actual results

Code sample
import 'dart:io';
import 'dart:ui';
import 'package:flutter/material.dart';
import 'package:flutter/rendering.dart';
import 'package:font_awesome_flutter/font_awesome_flutter.dart';
import 'package:google_fonts/google_fonts.dart';
import 'package:path_provider/path_provider.dart';
import 'package:share_plus/share_plus.dart';
import 'package:social_share/social_share.dart';
class BadgeDetailsModal extends StatefulWidget {
final String badgeName;
final String imageUrl;
final String condition;
final DateTime achievedDate;
const BadgeDetailsModal({
Key? key,
required this.badgeName,
required this.imageUrl,
required this.condition,
required this.achievedDate,
}) : super(key: key);
static void show(
BuildContext , {
required String badgeName,
required String imageUrl,
required String condition,
required DateTime achievedDate,
}) {
FocusScope.of().unfocus();
showModalBottomSheet(
backgroundColor: Colors.transparent,
isScrollControlled: true,
context: _,
builder: (BuildContext _) => BadgeDetailsModal(
achievedDate: achievedDate,
badgeName: badgeName,
condition: condition,
imageUrl: imageUrl,
),
);
}
@override
State createState() => BadgeDetailsModalState();
}
class BadgeDetailsModalState extends State {
// DateTime selectedDate = DateTime.now();
// DateFormat dateFormat = DateFormat.yMMMMd('en_US');
// String dateString = 'Select Date';
final GlobalKey globalKey = GlobalKey();
Future<String?> captureWidget() async {
final boundary =
globalKey.currentContext?.findRenderObject() as RenderRepaintBoundary?;
final image = await boundary?.toImage();
final byteData = await image?.toByteData(
format: ImageByteFormat.png,
);
final imageBytes = byteData?.buffer.asUint8List();
if (imageBytes != null) {
  final directory = await getApplicationDocumentsDirectory();
  final imagePath =
      await File('${directory.path}/container_image.png').create();
  await imagePath.writeAsBytes(imageBytes);
  return imagePath.path;
}
return null;

}
@override
Widget build(BuildContext context) {
return Container(
constraints: BoxConstraints(
maxHeight: MediaQuery.of(context).size.height * 0.75,
),
padding: const EdgeInsets.all(21),
decoration: BoxDecoration(
color: Theme.of(context).cardColor,
borderRadius: BorderRadius.circular(10.0),
),
child: SingleChildScrollView(
child: RepaintBoundary(
key: globalKey,
child: Column(
children: [
Padding(
padding: const EdgeInsets.symmetric(vertical: 8.0),
child: Text(
widget.badgeName.toUpperCase(),
style: GoogleFonts.montserrat(
fontSize: 20,
color: Theme.of(context).primaryColor,
fontWeight: FontWeight.w700,
),
),
),
Row(
children: [
Flexible(flex: 1, child: Image.network(widget.imageUrl)),
SizedBox(width: 20),
Flexible(
flex: 2,
child: Column(
children: [
Text(
'Conditions to get the Badge:',
textAlign: TextAlign.center,
style: GoogleFonts.montserrat(
fontSize: 15,
fontWeight: FontWeight.w700,
),
),
SizedBox(height: 10),
Text(
widget.condition,
textAlign: TextAlign.center,
style: GoogleFonts.montserrat(
fontSize: 15,
),
),
],
),
)
],
),
SizedBox(height: 10),
Padding(
padding: const EdgeInsets.symmetric(
horizontal: 12.0,
vertical: 4.0,
),
child: Text(
'In sagittis, ante ut consequat iaculis, massa augue vehicula lorem, eu maximus sem mauris nec enim.',
textAlign: TextAlign.center,
style: GoogleFonts.montserrat(
fontSize: 15,
fontWeight: FontWeight.w700,
),
),
),
SizedBox(height: 20),
Padding(
padding: const EdgeInsets.symmetric(
horizontal: 12.0,
vertical: 4.0,
),
child: Text(
'CONGRATULATIONS! YOU’VE EARNED THIS BADGE ON 03.1 5.2023!',
textAlign: TextAlign.center,
style: GoogleFonts.montserrat(
fontSize: 15,
color: Theme.of(context).primaryColor,
fontWeight: FontWeight.w700,
),
),
),
SizedBox(height: 5),
Row(
mainAxisAlignment: MainAxisAlignment.center,
children: [
IconButton(
onPressed: () {},
icon: FaIcon(FontAwesomeIcons.twitter,
color: Theme.of(context).primaryColor),
),
IconButton(
onPressed: () {
print('Working');
SocialShare.shareTwitter(
"This is Social Share twitter example",
hashtags: ["hello", "world", "foo", "bar"],
url: "https://your-url-here/",
);
},
icon: FaIcon(FontAwesomeIcons.facebook,
color: Theme.of(context).primaryColor),
),
IconButton(
onPressed: () async {
final path = await captureWidget();
if (path != null) {
// ignore: deprecated_member_use
await Share.shareFiles([path],
text:
'check out my website, Flutter is a open-source Google UI Framework https://flutter.dev/',
subject: 'Look what I made!');
}
},
icon: FaIcon(
FontAwesomeIcons.share,
color: Theme.of(context).primaryColor,
),
),
],
),
],
),
),
),
);
}
}
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
➜  empwr-app git:(profile) ✗ flutter doctor -v
[!] Flutter (Channel stable, 3.7.9, on Ubuntu 22.04.2 LTS 5.19.0-40-generic, locale en_US.UTF-8)
• Flutter version 3.7.9 on channel stable at /home/debajyoti/development/flutter
! Warning: dart on your path resolves to /home/linuxbrew/.linuxbrew/Cellar/dart/2.19.6/libexec/bin/dart, which is not inside your current
Flutter SDK checkout at /home/debajyoti/development/flutter. Consider adding /home/debajyoti/development/flutter/bin to the front of your path.
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 62bd79521d (3 weeks ago), 2023-03-30 10:59:36 -0700
• Engine revision ec975089ac
• Dart version 2.19.6
• DevTools version 2.20.1
• If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and
upgrades.
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at /home/debajyoti/Android/Sdk
• Platform android-33-ext5, build-tools 33.0.2
• Java binary at: /usr/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.18+10-post-Ubuntu-0ubuntu122.04)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at google-chrome
[✓] Linux toolchain - develop for Linux desktop
• Ubuntu clang version 14.0.0-1ubuntu1
• cmake version 3.22.1
• ninja version 1.10.1
• pkg-config version 0.29.2
[!] Android Studio
• Android Studio at /usr/local/android-studio
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
✗ android-studio-dir = /usr/local/android-studio
✗ Android Studio not found at /usr/local/android-studio
• Try updating or re-installing Android Studio.
• Consider removing your android-studio-dir setting by running:
flutter config --android-studio-dir=
[✓] VS Code
• VS Code at /snap/code/current
• Flutter extension version 3.62.0
[✓] Connected device (3 available)
• M2101K6P (mobile) • 8a7477f3 • android-arm64  • Android 12 (API 31)
• Linux (desktop)   • linux    • linux-x64      • Ubuntu 22.04.2 LTS 5.19.0-40-generic
• Chrome (web)      • chrome   • web-javascript • Google Chrome 112.0.5615.121
[✓] HTTP Host Availability
• All required HTTP hosts are available
! Doctor found issues in 2 categories.
➜  empwr-app git:(profile) ✗ C
➜  empwr-app git:(profile) ✗

Hi @Debajyoti14
For the issue to be workable, it needs to be reproducible with a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code. Please provide such a sample so that we may verify the issue properly.
Also, in order to narrow down the issue (to determine whether it is a Flutter framework bug or a 3rd package bug), I would suggest you check the output image after executing await boundary?.toImage() to see if it has dark color or not.


Huge memory consumsation of isWithinTokenLimit

I am experiencing 200MB increase after implementing gpt-tokenizer the only function that I am using from this library is isWithinTokenLimit. Here is an image of my memory consumtion before and after deployment.
Here is how I am using is
function getRequestTokenCount(req: ChatCompletionRequestMessage[]) {
  const extraTokensDueToPromptForEachMessage = 7
  return req.reduce((acc, curr) => {
    const tokensInText = isWithinTokenLimit(curr.content, Infinity) || 99999
    return acc + tokensInText + extraTokensDueToPromptForEachMessage
  }, 0)
}

same here

hi @aminsol and @olboghgc, can you provide a little more context about the platform?
is this under node, bun, the browser?
are you using a bundler of any sort?
which GPT encoding are you trying use?

@niieani  Node 18/typescript/babel 7.0.0 in a firebase cloud function, with gpt 3.5 and 4o(but that shouldn't really matter since a simple import already cause a spike in memory usage)

btw I was able to +/- ignore this issue by doing a conditional require so I don't always waste memory when I don't need to call it

New version should have more than 4x smaller memory footprint (data for GPT-3.5-turbo tokenizer):

Package
Version
Load time (ms)
Memory (Heap) consumption (MB)
Total Memory (RSS) (MB)
Encode Avg (ms)
Decode Avg (ms)

gpt-tokenizer
2.2.3
253.04
45.83
150.42
0.0208
0.0033

gpt-tokenizer
2.3.0
44.58
9.65
35.76
0.0102
0.0025


Workload Identity Support for AzureComponentFactory

Resolves #37943

Allow AzureCredentialFactory to create instances of WorkloadIdentityCredential based on the new "workloadidentity" credential value

Workload Identity credentials must have a Tenant Id, Client Id, and a Token File Path specified
Add new tokenFilePath field to the configuration expected by ClientFactory.CreateCredential

Update Azure.Identity version to 1.9.0 across the repository to support WorkloadIdentityCredential

API change check
API changes are not detected in this pull request.

Hi @wsugarman.   Thanks for your patience.  I was able to find the right folks for perspective, and we're in agreement that this serves a core scenario and is an enhancement that we'd like to include.  If you're willing, would you please add an entry in the changelog that describes the new value?   Beyond that, it looks like we're good to merge.

@jsquire Awesome! I've added to the changelog (as well as added a comment in the code). Please let me know if you'd like to see any other changes.

Thanks, @wsugarman.   Looks good.  I'll get this merged in and we'll look to release the new version as part of our August cycle next week.


mantain object keys

Hi,
why when I add a fuzy filter, my object keys are reset?
Look the example:
The original object is as follows:
var items = {
    10: {...},
    20: {...},
    ...
};

when I try to filter the object so:

 {{key}}

the result I expect is

10

20

...

0

1

...

with the library, the keys are reset as an array ( with indexes ), without the filters of the lib, the keys return to the original.
sorry for my bad english :)

@satore11 you should use toArray of you care about the key.
see: jsbin

It works.
@a8m thank you verry much

Cheers :beers:

In my use case, if I use toArray: true, groupBy: 'bid' stop working. If I take groupBy: 'bid' off, toArray: true works. Here is a screen shot with out groupBy: 'bid'...

Here is a screen shot with groupBy: 'bid'...

Here is my code...

{{item.bid}}

    {{item | json}}

{{item.name}}

{{key}}

        {{key | json}}

{{item.name}}

How can I make both work together?


Add support for ID3 tags in mp4

We use mux.js to parse ID3 tags in ts streams.
Mp4 also supports embedding ID3.
We should see if we can use mux.js to parse those or implement the parsing using our mp4 parser.

Please consider not using mux.js, because this adds an external dependency that might not be desired.

@joeyparrish already done with ID3 parsing in the Shaka.

Fixed in PR #4409.


Change default storage key base to Pouch.

The time is nigh!

Can do pending #2605

This is done now in ToT, see
https://github.com/PolymerLabs/arcs/blob/master/shells/configuration/constants.js#L10


Avoid loss for too short input to be infinity

fix #5614

Jenkins CI test (for commit 339360945b3689c60c6632c30b4f0d45131d281b, target branch master) succeeded!

ping @unnonouno

Thank you for reviewing. I heard that cuDNN's CTC also supports this case but its behavior is not defined in the document. I want to make behavior of my fix same as cuDNN's one. I'll check it after #5650 is merged.

Jenkins CI test (for commit 339360945b3689c60c6632c30b4f0d45131d281b, target branch master) failed with status FAILURE.


Support automatic real nesting of comments without placeholders, if supported by the language.

E.g. for haskell, scheme, scala, lisp, ocaml, sml. Fixes issue #131
Implemented by adding 'nested'/'nestedAlt' keys to the language delimiter defintions.

Hey @klaasb and thanks for taking the time to contribute. I've recently been added as a maintainer on this project and am working my way through the backlog of issues and contributions. I'm having some trouble reviewing this one and was wondering if I could get some help. For one thing with all the other merges I've done there are now some merge conflicts in this branch. Can you rebase it to take care of those?
Also I'm having trouble duplicating the issue that this would solve (#131). The exact procedure explained there behaves correctly for me (by changing the nested comment to an alternate syntax) so I was wondering if there is even still a need for this. If it's a better fix that whatever has already been done I'd love to consider it, but can you at least review it and comment on whether and why it is still needed?

Fixed the merge commits, rebased against master and added nesting attributes for Racket, as I saw that scrolling past. However, more languages with nesting might have been added since I wrote this PR.

Thanks a bunch @klaasb. This seems to work in my testing so I've gone ahead and merged it. I guess now we wait for people who actually use nested languages to tell us if we broke something. Thanks for the help.