Clear cells and cellMap when items changes

I ran into an issue with ember-collection where indexes are not correctly set when the items array  changes to a new array which is a subset of the previous array. The use case was providing a filterable list of items to ember collection.
It seems the item indexes are cached on the cell and never updated. I've added calls in updateItems to clear _cells and _cellMap whenever items changes. I think this makes sense but I might be missing some broader performance considerations.

Sorry for not replying sooner, been busy with work and prepping to emberconf. I'm hoping to hack on ember-collection a bit this week and can dig into this more than, but II think you can just set the cells index here to itemIndex.
https://github.com/emberjs/ember-collection/blob/master/addon/components/ember-collection.js#L154-L169

Yep looks like that fixes the problem too. Thanks for looking at this. I'm at emberconf this week I'd love to chat to you about how exactly why this works if you have some spare time. I find that updateCells method quite complex and pretty hard to grok, I'd love to understand it a bit better and help decompose it.

@paddyobrien sure thing... I'll do my best to explain things, it's actually not tooo bad, just needs a bit of commenting.
Find me at emberconf, I'll be giving my talk end of day 1, so I might be less distracted day 2.

Any thoughts on merging this? I'm 90% sure the test failure is a false negative but I don't have permissions to kick it off again.

not ok 183 PhantomJS 1.9 - Browser "phantomjs /home/ubuntu/ember-collection/node_modules/ember-cli/node_modules/testem/assets/phantom.js http://localhost:7357/22/tests/index.html?hidepassed" exited unexpectedly with exit code null - I'll try to dig into this failure this week.

This problem still exists?

@raytiley I re-poked this to build again and it looks like it's passing now. I suspect the failure was a result of something being broken in the beta channel.

Merge that pr please, it works pretty good

+1

@raytiley is there anything else that needs to be done here?

+1

Any reason why this has not merged yet ?

Hey Everyone... Sorry for not really having a hand on the wheel here. I'll try to poke some people and see what the long term fate of this project is. Honestly I think that most of the early contributors don't have any itches to scratch with ember-collection at the moment, hence the radio silence from us.


updating governance

to talk about Package succession under special circumstances

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Policies/Governance.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore: bump versions

We should have dependabot support which would automatically create new PRs with update pkg deps

@vinayakkulkarni Agreed. We really need merge access on these PRs though 😅

@bencodezen Sorry for the delay, but I guess you bro have been the owner role to manage this group.
I've been through the most busy time of my life recently, but believe me, I'll be back with more free time.


[tool_crash] FileSystemException: Cannot create link, OS Error:Invalid function., errno = 1

Command
flutter --no-color pub get

Steps to Reproduce

added  'localstorage: ^4.0.0+1' to pubspec.yaml
executed 'pub get' from AndroidStudio

Logs
FileSystemException: Cannot create link, OS Error: Invalid function.
, errno = 1
#0      _Link.throwIfError (dart:io/link.dart:263:7)
#1      _Link.createSync (dart:io/link.dart:184:5)
#2      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#3      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#4      _createPlatformPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:1061:12)
#5      createPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:996:5)
#6      refreshPluginsList (package:flutter_tools/src/flutter_plugins.dart:1087:5)
<asynchronous suspension>
#7      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:359:5)
<asynchronous suspension>
#8      PackagesGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:177:5)
<asynchronous suspension>
#9      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#11     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#15     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)

[✓] Flutter (Channel stable, 3.0.0, on Microsoft Windows [Version 10.0.19044.1706], locale de-DE)
    • Flutter version 3.0.0 at D:\flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision ee4e09cce0 (8 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at D:\android\sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_SDK_ROOT = D:\android\sdk
    • Java binary at: D:\more-tools\AndroidStudio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Build Tools 2019 16.11.14)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
    • Visual Studio Build Tools 2019 version 16.11.32428.217
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2021.2)
    • Android Studio at D:\more-tools\AndroidStudio
    • 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
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.1)
    • IntelliJ at C:\tools\IntelliJ\IntelliJ IDEA Community Edition 2021.2
    • 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

[✓] Connected device (4 available)
    • NOTE 20 (mobile)  • TE626XA1M0C171564 • android-arm64  • Android 10 (API 29)
    • Windows (desktop) • windows           • windows-x64    • Microsoft Windows [Version 10.0.19044.1706]
    • Chrome (web)      • chrome            • web-javascript • Google Chrome 101.0.4951.54
    • Edge (web)        • edge              • web-javascript • Microsoft Edge 101.0.1210.47

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: ee4e09cce01d6f2d7f4baebd247fde02e5008851
Plugins
path_provider-2.0.10
path_provider_android-2.0.14
path_provider_ios-2.0.9
path_provider_linux-2.1.6
path_provider_macos-2.0.6
path_provider_windows-2.0.6

Closing as duplicate of https://github.com/flutter/flutter/issues/66224.


minor updates

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/advanced-hololens-emulator-and-mixed-reality-simulator-input.md
:white_check_mark:Succeeded
View

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


visualising inout dependencies

I feel a bit guilty that #164 was initially about visualizing inout variables, but that has not been properly addressed in the following discussion or PRs. Furthermore, the checking algorithm works, but may be a bit aggressive. The user still does not really have a good way to see all relationships, such as seeming cycles as @jvail initially intendet. Therefore, the following proposal for two different options in the visualisation:

the initial proposal by @jvail. Add dotted arrows from the last inout process to all first in processes for all variables. as an option in visualise model.visualize(cycle_arrows=True) e.g.:

in->inout->inout->in
 ^            /
  \- dotted -/

add color enhancement for a single variable.

This would make it clear for the user if processes are updated or not before they are used.
Both require a similar algorithm to work, but 2 is quite more complex to implement.

#184 will not address 2. I think that it will help users in debugging custom dependencies however, what are your thoughts on that?

added an example notebook for why 2. is useful. For example when a user does not understand the error messages, they can still visualize the graph with the different stages of that variable.


Inconsistent category folder handling

Some of my nodepacks have helper directories that aren't to be used as category directories. At some places I can load them, but eventually Nodezator stops me with this error.

Mr. Vári,
There's a chance I'm mistaken, but taking into consideration what you described, I suspect you might actually be confused about how Nodezator loads node packs.
Each node pack directory must contain only folders representing categories and each category folder must only contain folders representing node scripts. The prompt shown in the image indicates that Nodezator is assuming the templates folder is a category folder because it is inside the lazycanvas node pack folder.
In other words, only node script folders are allowed to have content other than the script defining the node.
This is intentional design with the purpose of making the node scripts as modular and free of side-effects as possible, in alignment with the many paradigms that Nodezator take inspiration from, like functional, dataflow, flow-based and component-based programming. Since you are new to Nodezator and may not be familiar with these paradigms or just used to how other node editors deal with this, it might be hard at first to see the advantages of doing things this way, and I'd not blame you for that if it was the case. However, I argue that node-based programming is much more effective when adopt such principles of avoiding side-effects and keeping the node scripts as modular as possible.
This doesn't hinder/harm nor makes any kind of workflow impossible, because there are other ways to share resources between your node scripts without having to include them as folders in your node pack folder or the categories folder within it. I'll present the ones that come to mind.
If you have code that you want to share between different node scripts, then you must put such code in a library and make that library available somewhere your Python interpreter can see, so that such code can be imported from your node scripts. For instance, you can put your module/package in your site-packages folder, that you can find with:
from site import getsitepackages
getsitepackages()

If the resource you want to share is not code (files, images, databases, etc.), you can also use the solution presented earlier, that is, put them inside the module/packages to be imported. There are 02 more alternatives to this though.
First, you could put them in a known central place in your system and reference them in your node script folders by their location. For instance, if you have an image you can put it on the pictures folder in your home folder and then any node script that may want to use that image can reference it like this, for instance:
from pathlib import Path
path_to_image = Path.home() / 'pictures' / 'my_image.png'

Second, you can put that resource in one of your node scripts and use it to access that resource instead. For instance, if you have a database that you want to access, you can use a node to do so, like in this pseudocode:
from pathlib import Path
from dbprocessing import get_db_obj

path_to_db = Path(__file__).parent / 'my_db.db' # this is inside this node script
MY_DB = get_db_obj(path_to_db)

def get_reference_to_db():
    return MY_DB

main_callable = get_reference_to_db

Then you can instantiate this node in your graphs and grab the database object from its output to send to other nodes.
There may be other Python-related ways to share resources like the ones I described, but they are the ones that came to mind for now. Again, not allowing nodes to share resources directly is an intentional design decision that is adopted in many programming paradigms and results in systems that are more easy to maintain, operate and extend.
Nonetheless, all feedback and even constructive criticism is always welcome, so feel free to share your mind on this.
Also, please, regardless of your opinion on this, let me know whether I was able to explain the cause of your problem accurately or you still think the problem may actually be something else altogether.

I totally see the approach here: and this is actually more sustainable in the long-run.
My first idea was to allow my libraries to function as node-packs themselves, but I might seperate some of them with an "_ndz" suffix. (so "xyz" as a library and "xyz_ndx" as a node-pack that depends on said library.)


serialize BigInts as numbers for Mongoose 7.1 support, add env D=1  support for tests

What this PR does:

Adds a new serializeCommand() function that converts BigInts to numbers for JSON. Otherwise, JSON.stringify() throws the following error: error: Error: Command "insertOne" failed with the following errors: [{"message":"Do not know how to serialize a BigInt"}]
For convenience, env D=1 npm test enables debug logging, including logging the request body. Very helpful trick for debugging, Mongoose has used this pattern for a decade now.

Which issue(s) this PR fixes:
Fixes #
Checklist

[ ] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CLA Signed: DataStax CLA

@vkarpov15

BigInt support is really a great addition, thank you for adding the serializeCommand function handling that.
env D=1 npm test works only when we don't run the tests in client.test.ts. I fixed the test cases in that in this commit a3daa74 to get the log level from logger.ts.
The issue seems be to be when the log level is changed by the tests in client.test.ts, the effect env D=1 flag is ignored on the subsequent tests.

So, even if we change the log level within a client, it reflects across the clients. Please see if this commit a3daa74 is good before I can merge this.


App crashes when trying to start scanning

I try to start scanning and then the app crashes immediately

Would you have more details available?
Which device and Android version are you using? Are you using fdroid or full variant of the application? Does this happen every time?
Also if you have any logs, that would be very useful

Would you have more details available?
Which device and Android version are you using? Are you using fdroid or full variant of the application? Does this happen every time?
Also if you have any logs, that would be very useful

Samsung J8 Android 10
Downloaded from F-Droid
Version 1.1.3
It happened every time I pressed "Start scanning"

I have experienced the same crashing behaviour with a Redmi Note 9 Pro on Android 11 and the F-Droid Version, the Full Version works fine.
Haven't had time to investigate more but will try to do so tomorrow or the day after tomorrow.

I encountered the same problem on Android 11, version F-Droid.
In the file xyz.malkki.neostumbler.location.PlatformLocationSource I replaced :
  locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, locationIntervalMillis, 0.0f, locationListener)
with
  locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, locationIntervalMillis, 0.0f, locationListener, Looper.getMainLooper())
(line 40)
and it works again.

I still experience crashes when trying to start a scan or accessing the map view with the F-Droid version. This doesn't happen with the Full version.
Device: Shift6mq, Android 10
NeoStumbler version: 1.2.3
Logs for both versions:
f-droid.log
full.log
In the F-Droid log there is a permission violation error regarding location and then the App crashes.
However location and phone permission are granted

@billie80 Do you have GPS turned on? From the logs it looks like what might be happening is that GPS is turned off and that causes the crash because the callback function is not implemented

Ah, sorry you are correct. GPS was deactivated. It works if it is active.
Maybe this should be checked to avoid the crash.

Ah, sorry you are correct. GPS was deactivated. It works if it is active. Maybe this should be checked to avoid the crash.

I've created a fix for this, #138


Closes #78 | Create dataset loader for COCO-35L

Closes #78
Please name your PR after the issue it closes. You can use the following line: "Closes #ISSUE-NUMBER" where you replace the ISSUE-NUMBER with the one corresponding to your dataset.
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Hi, I saw some confusing lines, also w/ several commented lines. Mind giving short comments on the scripts regarding the reasoning, clear commented lines (if they're not being used anymore), and reformat your dataloader py using makefile fomatter? Thx!
It'll give much better clarity on its codes and what you're trying to do with every lines, also ease the review process (and end-user understanding process of this dataloader).

Hi, I saw some confusing lines, also w/ several commented lines. Mind giving short comments on the scripts regarding the reasoning, clear commented lines (if they're not being used anymore), and reformat your dataloader py using makefile fomatter? Thx!

I think this is very strange as I did run the make check before submitting it here. It also happens with my other PR if I'm not mistaken.

It'll give much better clarity on its codes and what you're trying to do with every lines, also ease the review process (and end-user understanding process of this dataloader).

I'll add some description. Those comments are from my debugging, and I want to keep that just in case I need to make some modifications. Will be removed before merging. Just let me know.

Hi, I saw some confusing lines, also w/ several commented lines. Mind giving short comments on the scripts regarding the reasoning, clear commented lines (if they're not being used anymore), and reformat your dataloader py using makefile fomatter? Thx!

Already did the make check before PR. Strangely, I got similar request frequently. Am I doing it wrong?

It'll give much better clarity on its codes and what you're trying to do with every lines, also ease the review process (and end-user understanding process of this dataloader).

Will add brief explanations in the next commit. Those comments will be removed later. It is there just to keep track and make it easier for me to debug.

Let's wait for Holy's review on this ya, @IvanHalimP

152520 image ids are not found in the coco 2014 training caption. validation set is ok Using COCO 2014 train and validation set.

Could you please note this in the _DESCRIPTION?

Hi @IvanHalimP, have you got the time to check on the feedback based on @holylovenia's review?

Hi all, thank you for the comments.
I nee to take care of something else for the time being, so please kindly wait. ;)

@IvanHalimP No problem, please take your time! Happy holidays!! ❄


feat: add support for Swift macros

Add support for packages using Swift macros (fixes #700)
Update swift-composable-architecture to 1.4.0 in the TCA example

Also, I just want to confirm that TCA does use macros. If we did not add macro support, would the TCA upgrade fail?

Yes. @Reducer is a new macro in TCA 1.4.0 and App.swift in our example uses it.

I’m having trouble understanding the CI failures and could use some help. do_test in the TCA example directory passes locally for me.

Is it possible that ${{  secrets.BUILDBUDDY_API_KEY }} isn't resolving for this PR because it's coming from my fork?
https://github.com/cgrindel/rules_swift_package_manager/blob/6023ab6055d225c64cff50f73da87dd78d3491d3/.github/workflows/ci.yml#L139

Is it possible that ${{  secrets.BUILDBUDDY_API_KEY }} isn't resolving for this PR because it's coming from my fork?

Possible. However, I did not think that is how GH workflows worked. If you want to remove that from the equation, comment out the remote cache flags in the ci.bazelrc file.

Due to the dangers inherent to automatic processing of PRs, GitHub’s standard pull_request workflow trigger by default prevents write permissions and secrets access to the target repository. However, in some scenarios such access is needed to properly process the PR. To this end the pull_request_target workflow trigger was introduced.

https://securitylab.github.com/research/github-actions-preventing-pwn-requests/

I’ve only tested this with external dependencies.

Split off just the functionality change into its own PR: https://github.com/cgrindel/rules_swift_package_manager/pull/753
Once CI supports Xcode 15 we can update the TCA example.

#747 & #755 will complete this


Updated Ubuntu installation packages docs for 15.04

As @dec0dedab0de pointed in issue #3630 that the correct apt-get packages for Ubuntu 15.04 were not specified; added it into documentation. Turns out they're the same as < 12.04!

@Zen-CODE, as you seem to take care of majority of the documentation stuff here :)

Great stuff. Thanks for that :-)


Rename policy context file and use cx-policy prefix

Mostly cosmetic changes:

Rename the policy context from namespace to policy
Use cx-policy instead of cx as the Json-ld prefix

Linked Issue
Closes #6

@jjeroch Here is the PR for renaming the cx prefix to cx-policy. Can you have a look?


ccs broken after firefox 108.1

hey after ff 108.1 seemed to have broken the visuals https://i.imgur.com/K9tV7fI.png

hey after ff 108.1 seemed to have broken the visuals https://i.imgur.com/K9tV7fI.png

It's too late, but I want to wish you a Happy New Year. I fixed some things which I seen. So you can try reinstall and check, if I missed something, then write more specifically what I need to fix.

ill have to get used to the audio icon to be hidden until i hover over it now lol but it's fixed now thankyou so much and happy new year to you too :)

ill have to get used to the audio icon to be hidden until i hover over it now lol but it's fixed now thankyou so much and happy new year to you too :)

Yes, I restored the standard firefox behavior, new firefox version broke my custom styles for that, maybe I rewrite it later.

ohh okay i see well thankyou for fixing it

ohh okay i see well thankyou for fixing it

Ok, enjoy ;)


Add cached evaluators

This PR is built upon #103, #112, #113 and #110
This PR adds CachedEvalulator which uses caching techniques in computing impacts of fixes on module.
Caching is activated by default, to disable it, following flag must be passed to Annotator:
-dfc, --disable-fix-cache

It can also be controlled in JSON format for config:
{
    "CACHE_IMPACT_ACTIVATION" : true
}

Changes in this PR:

Adds Impact class which stores the set of triggered errors and set of triggered fixes from downstream for a fix.
Adds ImpactCache which is Interface for a caches saving impacts of fixes on modules.
Renames GlobalAnalyzer to DownstreamImpactCache for a better unified structure.
Renames flag -do, --disable-optimization to -dpp, --disable-parallel-processing as we have two separate optimizations.

@msridhar @lazaroclapp Thank you for the review, this is ready for another round. Thank you.

@msridhar @lazaroclapp This is ready for another review. Thank you very much.


code style: update lint.py

Changed lint.py to use fstrings consistently.
Checklist

[x] No CHANGELOG update needed

[x] No new tests needed

[x] No documentation update needed

please fix the black code style violations identified by CI, then we can merge
https://github.com/mandiant/capa/actions/runs/4365366205/jobs/7635007299

Added the correct formatting which was required


Suppressions

https://sendgrid.com/docs/API_Reference/Web_API_v3/Advanced_Suppression_Manager/suppressions.html
The data payload is not showing up, but I see it in the source: http://www.screencast.com/t/sAwLaPeMe
The data payload should look like:
{"recipient_emails":["test1<EMAIL_ADDRESS>

Hello @thinkingserious,Thanks for your suggestion or question! In an attempt to clear out our backlog,                                              we are closing out issues over 90 days old.                                             If this is still an issue, please feel free to reopen it.                                             Team SendGrid DX-Docs


Write text on Image

I want to write some text on image
I want to save image after modification like measurement,highlight ,probe
Multiple highlight on image

will you please help me out
thanks

Look at the cornerstoneTools.drawTextBox API. It is not a movable tool, but you can use it to draw text on an image if you need to. It's also easy to just place HTML elements on top of the image, if static text is what you're after

You can save an image using the cornerstoneTools.saveAs function. See https://rawgit.com/chafey/cornerstoneTools/master/examples/saveAs/index.html

If you mean save and restore state, see the Application State example:
https://rawgit.com/chafey/cornerstoneTools/master/examples/toolState/index.html

Multiple copies of the highlight tool are not yet supported, but we would welcome a PR with this feature. You can also check the rectangleRoi tool to see if that suits your use case.

Please direct general questions like this to the google group (https://groups.google.com/forum/#!forum/cornerstone-platform) so we can keep Github for defects/feature requests.
Hope that helps!


Fix sizing of bottom row of member logos

Top row correctly contextualized size to each other, bottom row varying sizes:

there is no size to fix, there is a hint that the logo should be in 250x250px.

I think the correct way to do this would be to account for it in CSS... trying to wrangle every different member to properly submit PNGs is tough.

Not really. I know it will work with css, but I think we should prefer the right size, because it will make the file size for downloading/displaying much smaller! But i will have a look at this later on.

Do you currently allow SVG? That would solve a lot of problems.

SVG would be best.

SVG would be best.

Of course! Let's please standardize if possible. Would make our images look much better across a range of devices, as well as load faster. Even opens up the possibility of SVG animations, if we want to get fancy 😄

Yes SVG is nice, but SVG need to implement in a different way to normal image files.
We can do it with the switch in the mebres.yml like:
- organization: ito
  country: Germany
  logo: images/members/ito-logo.svg
  svg: true
  url: https://www.ito-app.org/

ore
- organization: ito
  country: Germany
  logo:
     png: images/members/ito-logo.png
     svg: images/members/ito-logo.svg
  url: https://www.ito-app.org/

ore
- organization: ito
  country: Germany
  logo: images/members/ito-logo.png
  svg-logo: image/members/ito-logo.svg
  url: https://www.ito-app.org/

i don't know wich is the best....

I found another solution...
 {{ path.Ext "a/b/c/news.html" }} → ".html"

i check the path extension for svg so i can switch between object and img tag.
https://github.com/TCNCoalition/website/commit/ce426ae7dd7ed566e018d76a60646a019a010c07
The only reason why you want a png/jpg is for fallback if the browser dosn't support the SVG.

I think you can just put svg in an IMG tag

Yeah lol ^
On Mon, Apr 13, 2020, 7:10 PM Harper Reed<EMAIL_ADDRESS>wrote:

I think you can just put svg in an IMG tag
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TCNCoalition/website/issues/26#issuecomment-613140548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGN4ARGX6DA4JIXLRUFE7UDRMOLUXANCNFSM4ME6LKFA
.

Off course @harperreed but you got some a few limitations while doing so!
https://vecta.io/blog/best-way-to-embed-svg

yea. let's just use the img tag.
The linked article is very good. Thanks for linking it. Their recommendation despite limitations is to also use the img tag.

I think we can close this issue because the image tag is aelrady there and can be used for svg. You can see a test with the ito svg logo on my commit 1bc85a7 above.

wfm. closing


Change vote_value to use market prices. Median prices are only closer…

… estimates of payout for votes sold on post older than 5.25 days (7 day minus half median time)

Median prices are only closer estimates of payout for votes that will pay out in less than half the 3.5 day median, 1.75 days (5.25 days old posts)


Fix python dialog selectors in Python smoke test

Contents from both pages of the Python Dialog are on the DOM, the hidden one has the 'dialogModal-hidden' class.
I changed the selectors to correctly select items that are not hidden.
Note: :not(.dialogModal-hidden) must be placed next to the correct element selector. Ex. previously I placed it in the nextButton selector but that didn't work since the nextButton element doesn't actually have the .dialogModal-hidden class, it's parent div (.footer-button) is what has the hidden class.

Pull Request Test Coverage Report for Build 961800668

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

Totals

Change from base Build 961457326:
0.0%

Covered Lines:
26330

Relevant Lines:
55638

💛  - Coveralls


Update guide-install.md

Hello,
I took the opportunity a a fresh abapGit install to update installation procedure. I only use offline import so I may have misunderstood some part of the online install.
BR,
Alexandre

Hi Alexandre,
we appreciate you suggestions and I agree, there's some clarification necessary. For the installation, it's not really about "online" vs "offline".
There are two versions of abapGit:

A developer version
A consumer version (I guess, I just invented this name, so not sure if others agree calling it this way)

The consumer version contains the same features as the developer version but all compiled into a single - very large - ABAP program. This consumer version is sometimes referred to as "merged version" or "full version" of abapGit. This makes the initial installation very easy (cut & paste or upload).
Either version can be installed whether your SAP system is connected to the Internet or not. And either version will be able to work with online or offline repositories.
The recommended way is to install and use the consumer version. If you want to contribute to the abapGit project, then we recommend to you use the consumer version in order to install the developer version (by adding it as an online repository).
Hope that helps,
Marc
PS: In case your system is not connected to the Internet, you can install the developer version also by adding it as an offline repository. But this is rather an exceptional case which obviously won't be able to interact with the Github repository.

Hi Marc,
Interesting. As a matter of fact, when using consumer version I got at some point an erreur message asking me to install abapgit
That why I deduced that ZABAPGIT_FULL was an installer and needed the full project. SAPlink worked that way after all.
I may have misunderstood the error message.
If the project agree on a common naming for developper/consumer version, please let met know and I'll rephrase the doc page.
Alexandre

That why I deduced that ZABAPGIT_FULL was an installer and needed the full project

Hmm, maybe it would be clearer to rename it to ZABAPGIT_MERGED (or something) ? Indeed, full is misleading...
Just a side note.

thanks, merging
welcome to abapGit as a contributor 🎉

Thanks!
I got time to rethink about what I clumsily called online and offline install. What would you think of calling it standalone and developper (as suggested by @mbtools) version?
If so, it might be interesting to rename report ZABAPGIT_FULL to ZABAPGIT_STANDALONE.
What do you think?
Alexandre

yea, feel free to open more pull requests with additional changes


Add step extension definition for set-body (KaotoIO/kaoto-ui#1080)

sorry the URL has to be changed to upstream


CB-12843 Make sure we have PKs on every CB table, test PR, should fail

See detailed description in the commit message.

LGTM! Closing this as verified.


Better Text To Speech

With modern machine learning, there are better ways of TTS.

https://github.com/CorentinJ/Real-Time-Voice-Cloning
https://github.com/andabi/deep-voice-conversion
https://github.com/r9y9/wavenet_vocoder

It would be very helpful cause right now, we hit the rate limiting on ttsmp3.com really often!

Exactly, and with these repos you can create custom voices.

Also please take a look at

https://github.com/search?o=desc&q=speech+synthesis&s=stars&type=Repositories
https://github.com/search?o=desc&q="text+to+speech"&s=stars&type=Repositories

Some of the repos are really good.


Improve 2D velocity estimation to be less fragile to poor data timing

This is the underlying issue for https://github.com/googlecartographer/cartographer/issues/233 and for the tuning problems in https://github.com/googlecartographer/cartographer_ros/issues/317.
Right now, if two scans are very close in time, velocity estimation will become very big and mess up position quite profoundly.

Offensive code is here: https://github.com/googlecartographer/cartographer/blob/b4b83405ce4009ea0c1ac22c7ab9edeeb9d48a42/cartographer/mapping_2d/local_trajectory_builder.cc#L204

This is probably also the cause of the issues @ojura had in #193 .
This made me take a second look at the lms1xx driver (both ojura and I are using lms1xx devices at the moment) - it marks the timestamp as being when the message was processed by the driver - https://github.com/clearpathrobotics/LMS1xx/blob/master/src/LMS1xx_node.cpp#L170 - in this case, unfortunately, it ignores the timestamp provided by the scanner with each scan.
For LMS1xx users, the solution appears to be a bit of driver rework so it uses the laser's timestamps as a time reference.  For other people - I'm not sure if the best solution is an intermediary filter node (which massages timestamps to roughly track the average scan time - possibly explicitly telling it the expected scan rate?) or to implement some sort of timestamp filtering within Cartographer?
@ojura - for your existing bags - maybe you can do a timing analysis of the inter-scan timestamp deltas, if there's significant jitter you might be able to implement something that filters the timestamps to keep them close to the expected 50 Hz?

I think it is wrong to add a guesstimate inside Cartographer. It's not Cartographer's to second guess the data - it will do the best possible job it can, but garbage in - garbage out. Also, any filtering would probably hurt users that have really good timing.
I think the filtering node is the way to go. And timing of the scans should be simple too: Lasers are pretty precise in their timing - if they promise a laser shot every 10ms, this will be accurate. It is usually pretty okay to just take a starting time stamp and trust the laser that its timing stays solid. This will be way better than the system time of processing the packets.
While we talk about data representation: another problem is ROS'es data format, most notably sensor_msgs/LaserScan which assumes that the laser rotates evenly - which is basically never true, and certainly not if your spin direction is not aligned with gravity. The correct data format is a tuple of (azimut, time, distance) + intensity or whatever data the sensor additionally provides.

Hi everyone! I think deltas are okay in my bag, they are ranging from 18-22 milliseconds (should be 20, i.e. 50 Hz).
@Entropy512 now that you mention it, I tackled this exact issue a few months ago when updating the ROS driver of a similar scanner, Sick S300. I wrote a timesync class which filters the difference between the system clock and the sensor clock, thereby eliminating system clock jitter -- it trusts the sensor clock locally, and globally, it gently pulls the timestamp to the system clock, so there's no drift. If you want to play with it, the class is available as a ROS package in this repo. For documentation, check the headers, and here is a commit in which the class is used in a driver node for the S300 laser scanner -- it's not much work. The main thing is the call to the sync() method (line 159), to which you pass the system clock and the sensor clock (which is, in that example, calculated by multiplying the sensor sample rate with the message count).
The filter works by first filtering the system-sensor timestamp difference with a median filter (some day I am planning to replace this step with a convex hull filter like here to eliminate the delay caused by the median filter). Then the resulting signal is again filtered with a simple second-order exponential filter (Holt-Winters) which estimates the slope of the difference, i.e. the clock drift. To obtain the best results, you have to tune the filter parameters; there's also a Matlab version in the util folder which plots the results. A cool feature is that the sensor and the system clock are published to a debug topic, so if you bag these it's possible to recalculate the stamps in a bag with different filtering parameters (although I haven't written a script which will do that...yet). If you want to play with this, be my guest :-)

It sounds like something like ticsync/triggersync might make sense here TriggerSync pdf, which adds some integration on top of TicSync.
ROS code:
https://github.com/englishar/trigger_sync

Good to know! That's the same approach I took, just done properly. :-) Although the problem isn't only in stamp synchronization - scan matcher noise can also break 2D velocity estimation.

It appears that we recently added a time filter window of 1ms for IMU. I'm not sure that is enough smoothing, but it might do something. I think that parameter should get exposed for configuration.


Workflow Link in Side Nav does not work in Automate

Describe the bug
When Workflow is turned on and the link appears in the side nav, clicking on the link takes the user to the Dashboard.
In order to use the link, a user has to Shift+Click to open the window.
To Reproduce
Steps to reproduce the behavior:
Acceptance

goto https://a2-workflow-local-inplace-upgrade-acceptance.cd.chef.co/infrastructure/client-runs
Click on 'Workflow' in the side nav
System goes to Dashboard

Local Dev

in the studio you can use start_workflow_service

if you've already started all services, you can chef-automate config patch with this file:
[deployment.v1.svc]
products = ["automate", "workflow"]

Expected behavior
When clicking on 'Workflow' it should open a new tab with the workflow app.
More Context
I did notice that if I hold down Shift+Click on Workflow it does open up the workflow page, but in a new window.
Versions (please complete the following information):

OS: Mac OSX
Browser Chrome/Firefox
Automate Build Number<PHONE_NUMBER>5207

That could be the problem.

this work has been deprioritized, so closing.


Updates vendored grpc to version 1.18.0

@StevenYCChou it looks like your previous PR omitted some subpackages and some directories didn't get locked to the same version as the rest (v1.18.0). FYI, we re-imported the full tree and fixed everything to the same version.


Tags, vref, and license fix for BrickMod

#8886 missed a few things.

There's a version file in the wrong folder in the ZIP, but Netkan can still read it.
Tags.

License?  Didn't it take it from Spacedock?

Was the first time I used Spacedock to make the netkan, not really happy, will go back to my local automated process

It's just a parts mod, if you can fix the tags, I'll appreciate it

Thank you

FYI, the remote version file isn't online. Not sure if it should be yet.

The remote version file is not online yet, working on it
It's in process now

Was the first time I used Spacedock to make the netkan, not really happy, will go back to my local automated process

What didn't you like about it? The problems here weren't the fault of that system.

Probably because I'm used to my system, which does the tags, license, etc.

The license was included in the original automatic PR, it's just that you entered it wrong on SpaceDock.

ok.  Anyway, thanks.  Remote file is up there, and when I do the next release, the version file will be in the correct place


Assigning role to aria labels

Assigning the role "region" to aria labels so that it creates a web spot

Is there a specific place that assigning the "region" role would help? As I understand it the order of preference is Semantic Tags > Semantic Landmark Roles (banner, main, contentinfo, complementary, or navigation) > "region" landmark role, ie if semantic tags are available they can be used instead of assigning a role and if that's not possible, semantic landmark roles should be preferred over "region" landmark role.


Segoe MDL2 License

I have searched for information about licensing of the Segoe MDL2 font, but I was unable to find any information about that. Is there a possibilty to add this information to this page? 😅

I'm not sure exactly what you would like to do with Segoe MDL2, but perhaps this page would give you the information you need.
https://docs.microsoft.com/en-us/typography/fonts/font-faq

I am thinking about applications that use the icons as resources to use e.g. as an icon creator or an app to combinate and overly multiple Segoe MDL2 Assets icons (as outlined here). Is it allowed to export those, and if so in which format?

For your reference, the following is from the Segoe EULA, included with the ZIP file on this page  (go to  "How do I get this font?" > "On a Mac", or go straight to the download (aka.ms/SegoeFonts)):
You may use the Segoe MDL2 Assets and Segoe UI fonts or glyphs included in this file (“Software”) solely to design, develop and test your programs that run on a Microsoft Platform, a Microsoft Platform includes but is not limited to any hardware or software product or service branded by trademark, trade dress, copyright or some other recognized means, as a product or service of Microsoft. This license does not grant you the right to distribute or sublicense all or part of the Software to any third party.  By using the Software you agree to these terms. If you do not agree to these terms, do not use the Software.

Thank you so much @llamax1 for sharing those information with me! I'm also a bit surprised as to why they provided a download for Mac if it's prohibited via the EUAL. I can only speculate here but maybe Office 365 or Sharepoint count as Microsoft Platform here so you would be allowed to use them in there? But that's just a speculation so.

I'm going to leave the page as-is. As @llamax1 points out, the EULA is included with the font. Legal documents like licensing agreements require specialized localization, so I'm inclined to stick with the existing setup.

Alright, thanks for the update @hickeys !


Faster fix for dangling tasks upon supervisor termination

Fixes issues with delayed supervisor termination during certain transient states

Description
Tasks can be created during supervisor termination and left behind since the cleanup may not consider these newly added tasks.
https://github.com/apache/druid/pull/12178 adds a lock for the entire process of taskCreation to prevent this.

The above lock in runInternal() leads to a deadlock when a supervisor is stopping and submits a graceful shutdown notice while a run notice is running simultaneously. The run notice cannot be completed as it is waiting on the lock, and the graceful shutdown notice is queued after the run notice. This always leads to a forced termination of the supervisor after the supervisor's graceful shutdown period.
This PR adds an additional check before getting the lock to see if the supervisor is STOPPING to prevent unnecessary lock contention.

maybeSetState is called by the RunNotice and may set the state to something other than STOPPING when called again after the supervisor has begun stopping. This would prevent the changes made above from taking effect after subsequent runs.
A supervisor once in STOPPING state must remain in it. Any operation to update or remove it first removes and then creates a new supervisor, so there is no need for further state transition handling in this case.
This also needs to be synchronized to prevent races from multiple threads leading to an incorrect supervisor state, particularly when one of them tries to set it to STOPPING while the others attempt to set it to a different state.

Key changed/added classes in this PR

SeekableStreamSupervisor -> additional check for stopping state before processing task creation block
SupervisorStateManager -> constrain state setting when in STOPPING state

This PR has:

[x] been self-reviewed.

[x] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[x] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

@gianm @kfaraz thank you for the review!

Merged as the failures were only for coverage of a single line, which is being tested in other tests.


Amplify not honoring Incremental Static Regeneration due to sqs.fifo deduplication

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

JavaScript Framework
Next.js
Amplify APIs
DataStore
Amplify Categories
storage, hosting
Environment information

# Put output below this line
  System:
    OS: Windows 10 10.0.22000
    CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
    Memory: 8.38 GB / 15.93 GB
  Binaries:
    Node: 14.15.1 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.8 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22000.120.0), Chromium (97.0.1072.55)
    Internet Explorer: 11.0.22000.120
  npmPackages:
    @ampproject/toolbox-optimizer:  undefined ()
    @aws-amplify/datastore: ^3.7.3 => 3.7.3
    @babel/core:  undefined ()
    @peculiar/webcrypto:  undefined ()
    @tailwindcss/typography: ^0.5.0 => 0.5.0
    @vercel/nft:  undefined ()
    amphtml-validator:  undefined ()
    arg:  undefined ()
    async-retry:  undefined ()
    async-sema:  undefined ()
    autoprefixer: ^10.4.1 => 10.4.1
    babel-packages:  undefined ()
    ci-info:  undefined ()
    cli-select:  undefined ()
    comment-json:  undefined ()
    compression:  undefined ()
    conf:  undefined ()
    content-disposition:  undefined ()
    content-type:  undefined ()
    cookie:  undefined ()
    cross-spawn:  undefined ()
    date-fns: ^2.28.0 => 2.28.0
    debug:  undefined ()
    devalue:  undefined ()
    escape-string-regexp:  undefined ()
    find-cache-dir:  undefined ()
    find-up:  undefined ()
    formdata-node:  undefined ()
    fresh:  undefined ()
    glob:  undefined ()
    gzip-size:  undefined ()
    http-proxy:  undefined ()
    icss-utils:  undefined ()
    ignore-loader:  undefined ()
    is-animated:  undefined ()
    is-docker:  undefined ()
    is-wsl:  undefined ()
    json5:  undefined ()
    jsonwebtoken:  undefined ()
    loader-utils:  undefined ()
    lodash.curry:  undefined ()
    lru-cache:  undefined ()
    micromatch:  undefined ()
    mini-css-extract-plugin:  undefined ()
    nanoid:  undefined ()
    native-url:  undefined ()
    neo-async:  undefined ()
    next: latest => 12.0.7
    node-libs-browser:  undefined ()
    npm-check-updates: ^12.0.5 => 12.1.0
    ora:  undefined ()
    postcss: ^8.4.5 => 8.4.5 (8.2.15)
    postcss-flexbugs-fixes:  undefined ()
    postcss-modules-extract-imports:  undefined ()
    postcss-modules-local-by-default:  undefined ()
    postcss-modules-scope:  undefined ()
    postcss-modules-values:  undefined ()
    postcss-preset-env:  undefined ()
    postcss-safe-parser:  undefined ()
    postcss-scss:  undefined ()
    postcss-value-parser:  undefined ()
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2
    react-markdown: ^7.1.1 => 7.1.2
    react-server-dom-webpack:  undefined ()
    rehype-highlight: ^5.0.2 => 5.0.2
    rehype-raw: ^6.1.1 => 6.1.1
    remark: ^14.0.2 => 14.0.2
    remark-gfm: ^3.0.1 => 3.0.1
    remark-html: ^15.0.1 => 15.0.1
    remark-images: ^3.1.0 => 3.1.0
    resolve-url-loader:  undefined ()
    sass-loader:  undefined ()
    schema-utils:  undefined ()
    semver:  undefined ()
    send:  undefined ()
    source-map:  undefined ()
    string-hash:  undefined ()
    strip-ansi:  undefined ()
    tailwindcss: ^3.0.8 => 3.0.9
    terser:  undefined ()
    text-table:  undefined ()
    to-vfile: ^7.2.2 => 7.2.3
    ua-parser-js:  undefined ()
    unistore:  undefined ()
    uuid:  undefined ()
    web-streams-polyfill:  undefined ()
    web-vitals:  undefined ()
    webpack:  undefined ()
    webpack-sources:  undefined ()
    ws:  undefined ()
  npmGlobalPackages:
    @aws-amplify/cli: 7.6.5
    cross-env: 7.0.3
    jest: 26.6.3
    nodemon: 2.0.7
    serverless-export-env: 1.4.4
    serverless: 2.48.1

Describe the bug
Issue
Next.js Incremental Static Regeneration (ISR) will periodically request a new version of a page on a website. Behind the scenes on Amplify this appears to create a message in an sqs.fifo queue.
This queue has a deduplication setting on message group. I'm not sure how this group ID is generated, but it appears to be the same across browsers. Interestingly the message deduplication id is different across browsers.
Because of this, after I update data in amplify studio, a new visitor may or may not generate a new page. It depends on if they visited within this 5 minute deduplication window. If you had a spelling error you wanted to fix quickly, you're going to have to wait at least 5 minutes from the last time your page was generated.
This 5 minutes is from the SQS 5-minute deduplication interval. (https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/using-messagededuplicationid-property.html)
Expected behavior
ISR should generate a new web page
Reproduction steps

Launch a next.js website using ISR on amplify using amplify datastore and amplify studio
Make a new content entry
Refresh your website twice to see that new content entry (once to kick off isr, another to view the new static page)
Make an update to that same content in amplify studio
Refresh your page again, and notice that your content will not update for at least 5 minutes.

Code Snippet
// Put your code below this line.

Log output

// Put your logs below this line

aws-exports.js
No response
Manual configuration
No response
Additional configuration
No response
Mobile Device
No response
Mobile Operating System
No response
Mobile Browser
No response
Mobile Browser Version
No response
Additional information and screenshots
No response

Hi @jayfry1077 👋🏽 thank you for raising this issue and providing reproduction steps. I will try to reproduce this behavior and update once I have more information.

Hi,
Any news on this, perhaps a work around while waiting some fixes ?

Hello, any news on this please?


Exception on signing PUT requests using AwsRequestSigningApacheV5Interceptor

I am trying to sign (AWS Sig v4) opensearch requests using this library.
The client library I'm using is opensearch java client and in order to use http5, I'm using ApacheHttpClient5Transport class.
This is how I create a bean of AwsRequestSigningApacheV5Interceptor
    @Bean
    @Scope(ConfigurableBeanFactory.SCOPE_PROTOTYPE)
    AwsRequestSigningApacheV5Interceptor awsRequestSigningApacheV5Interceptor(@Value("${AWS_REGION}") String awsRegion) {
        return new AwsRequestSigningApacheV5Interceptor("es", Aws4Signer.create(), DefaultCredentialsProvider.create(), Region.of(awsRegion));
    }

Here is how I use the interceptor:
    @Bean
    public ApacheHttpClient5Transport apacheHttpClient5Transport(Configuration cfg, AwsRequestSigningApacheV5Interceptor requestInterceptor) throws URISyntaxException {
        return ApacheHttpClient5TransportBuilder.builder(org.apache.hc.core5.http.HttpHost.create(new URI(cfg.serverUri)))
                .setHttpClientConfigCallback(clientBuilder -> clientBuilder
                        .addRequestInterceptorLast(requestInterceptor)
                        .setConnectionManager(PoolingAsyncClientConnectionManagerBuilder.create()
                                .setDefaultTlsConfig(TlsConfig.custom().setVersionPolicy(FORCE_HTTP_1).build())
                                .build())
                )
                .build();
    }

When my applications start, GET requests works as expected, but I see the following error on PUT request:
org.opensearch.client.transport.httpclient5.ResponseException: method [PUT], host [*****], URI [*****], status line [HTTP/1.1 403 Forbidden]
{"message":"The request signature we calculated does not match the signature you provided.
Check your AWS Secret Access Key and signing method. Consult the service documentation for details.

The Canonical String for this request should have been
'PUT
*****
accept:application/json; charset=UTF-8
content-length:47
content-type:application/json; charset=UTF-8
host:*****
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
x-amz-date:20240312T030211Z
x-amz-security-token:IQoJb3JpZ2luX2VjEDMaDmFwLXNvdXRoZWFzdC0yIkcwRQIgGzc/Is/bMwbyz1qGxwgHINfduFEuDhsNSwbweO+9gd4CIQD6MjY/9FbaUEySm4MEZgqgOnUWvielLFRfJPGytopEtSrKBQg8EAAaDDkzNjY2NDQ2Mzg1NiIMNuVyrsSnflj7w9TyKqcFJwp4G4w7mETv0KS6zjtJWOwXexZaK2Ho9XbuJzSUioO4yYrQKEzAIKaDJlrbDihjBbFgp//oq7/L2EkgvGrOXoKcSvl5xCTERnyo2urX/ImyzGC9RmvzbYt4rQaKzQRssZGP5TwGs1XPDpIcu1iVnUsfu0xNZG2YhtOuLardM7yQcn5Jqt8FXOR4NZZlzjBsa/FfcpymwdbdS0TrKPpTG24s8H84HyfoSMYlPk1PpgMt1ZVJWiKRsKWquesLcpGCiF6utK++L8cimFAsEf4LxEZ4ST/lrdq8unupDN+Edm862yiC96u1FB/GdASwWHaKt3285Vau3Ce/ipjFzUEpXcs3Rd6itcS3OOIU5IrQiqgXYxtBLLbFliztDxQ+CeLvAstOB7oh2i8tyi+5dDH2ZRX5JeR0NitNJQF3plpe5ktuXjtVqZn1Wwpk4UmykJPuVfUK+CKCIebMScma0JOa589JHB/ngWbkHVZCHzIEuDrwkOK2mA7mbCXEjChg8rV244CoWg6XwEGk/+8KowL8L2CL66HJx0CLkLZ6opwmnrUlmwlsSPrbbPqU95zVQ4aOKSLjt6w7/kQTUq0TKTh0fWoPuhuU8axHfUv3JU4NUNzvyocJW16F6iajrZ+mcne5a4/0Y3MNAE4Ch4dDrKGkonWqClOeXPuttvKS0JlTI3qRbyerTMYweUPUFKmmrA6IKlog5a9l4U7xUu84l1Vt0redHPyNyImSiyWT9f73PaYEDYyGCBRFFUe9MVWgE5jSxEJ3M6Q4CvhKBfX0mwQBHXiVBBcCObCkRNAWbGORNr1++RjA5hoDEEuyEw0napxgIDawHSW5tp3rJzo5TMFzz/T1/5hVXsQKQS8uEBYk5jzBgdJqK6My4WLVOjq8L8hOjocw6rz8izCGib+vBjqxAah0pFU06yYGC7BLy56IgKp9J3I5cw6+mRgLBSPFZiCZfS3XPHL7XT2FGwkEK6gbDDg8qqT/tl2ZfGD6N853GJ97MQD2Gpm8S0vzeR3/odCftDvXXNZJxf5tMXtnXtGNcOVBr6rs4KSSGEOkVKeAk8xA62hoz2DDDzActYxXbB6myfa8+q3dXQN0XhgYuw27onJJ5faPSkG4FHiJHuXdUSQKz5OUOsw4AFzeQDYPhQsNyh==

accept;content-length;content-type;host;x-amz-content-sha256;x-amz-date;x-amz-security-token
4db4276a52606b18093df202d794d9cebaae2e1785ef411ca549520d7a2be21a'

The String-to-Sign should have been
'AWS4-HMAC-SHA256
20240312T030211Z
20240312/ap-southeast-2/es/aws4_request
2a099805b1c4f747bdc82cc40cde2826d8e703817ea6a280eeba2c5d7907cf60'
"}
	... 30 common frames omitted

Is this a bug in the interceptor class?

What was the actual value of the Authorization header sent? Double check it ended up having ap-southeast-2/es/aws4_request and that only the signature was different.
Stepping back, since the GET request is working, and PUT is not, we can safely assume that the interceptor code is not signing the same body as the one being sent to the server, or appending a header after the signature is calculated. I'd look for what is actually being signed.
Build a sample with your code off https://github.com/dblock/opensearch-java-client-demo/tree/opensearch-2.x  (feel free to PR into that project), or make a new project, and we can try to help debug.

Thanks for the help @dblock ,
I tried to intercept headers for a Get and Put request before and after signing in AwsRequestSigningApacheV5Interceptor.
As the next step, I am going to build  sample code with the repo you mentioned above.
Headers before signing for a PUT request:
Accept:application/json; charset=UTF-8,
Connection:keep-alive,
Content-Length:47,
Content-Type:application/json; charset=UTF-8,
User-Agent:opensearch-java/2.8.1 (Java/17.0.7),
x-amz-content-sha256:required

Headers after signing for a PUT request:
Accept:application/json; charset=UTF-8,
Authorization:AWS4-HMAC-SHA256 Credential=ASIA5UFMNPHYII5VG2IN/20240313/ap-southeast-2/es/aws4_request, SignedHeaders=accept;content-length;content-type;host;x-amz-content-sha256;x-amz-date;x-amz-security-token, Signature=309bb0178cd4b780f8bfd2d43a92987dfd0880714f2fc50a8930ca6aa328b68e,
Connection:keep-alive,
Content-Length:47,
Content-Type:application/json; charset=UTF-8,
Host:abc.ap-southeast-2.es.amazonaws.com,
User-Agent:opensearch-java/2.8.1 (Java/17.0.7),
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855,
X-Amz-Date:20240313T080742Z,
X-Amz-Security-Token:IQoJb3JpZ2luX2...==

Headers before signing for a GET request:
Accept:application/json; charset=UTF-8,
Connection:keep-alive,
User-Agent:opensearch-java/2.8.1 (Java/17.0.7),
x-amz-content-sha256:required

Headers after signing for a GET request:
Accept:application/json; charset=UTF-8,
Authorization:AWS4-HMAC-SHA256 Credential=ASIA5UFMNPHYII5VG2IN/20240313/ap-southeast-2/es/aws4_request, SignedHeaders=accept;host;x-amz-content-sha256;x-amz-date;x-amz-security-token, Signature=585c68ab6d0404437c542e0b96deb0b7628b79d737e6529fecd46de486d2b54f,
Connection:keep-alive,
Host:abc.ap-southeast-2.es.amazonaws.com,
User-Agent:opensearch-java/2.8.1 (Java/17.0.7),
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855,
X-Amz-Date:20240313T080741Z,
X-Amz-Security-Token:IQoJb3JpZ2lu...==

I forked the branch and created a sample application (created a PR to the branch you mentioned) accessible here: https://github.com/vahidiazar/opensearch-java-client-demo/tree/opensearch-2.x-ApacheHttpClient5Transport
I could replicate the problem,
The app has 2 sections:

check cluster health which uses GET request and works fine,
Creating an index request which PUT request fails due to signature issue.

Could you please have a look when you free?

The interceptor is seeing a BasicHttpRequest but only knows how to sign a ClassicHttpRequest which has a body (entity). I think this is why we can't implement support for ApacheHttpClient5TransportBuilder in the interceptor, but I could be wrong (and @acm19 or @reta can correct me)?
The ApacheHttpClient5TransportBuilder class is from opensearch-java. There's another dedicated transport option there called AwsSdk2Transport which uses whatever AWS SDK uses and works out of the box. Is thee a reason why you can't use that? example on main in my demo.

As @dblock, what you're experiencing is this issue: https://github.com/acm19/aws-request-signing-apache-interceptor/issues/101#issuecomment-1532174922. I'll re-open it btw, as we should either try to solve it or document that it's a limitation. Back when I checked there didn't seem to be a solution without some hacks.
In your case you're using ApacheHttpClient5TransportBuilder, which is using an async client under the hood that has this problem. If you use a sync client you won't experience the issue. And also there's @dblock recommendation about the out of the box transport, you could give that a try as well and let us know if it helped.

@vahidiazar help us with contributing to the README?

@dblock,
Thanks for the response,
Initially We were using an old ES client (Jest) which internally using ApacheHttpClient 4.
Inorder to hand 429 and internal errors, we were using HttpClientBuilder setRetryHandler and setServiceUnavailableRetryStrategy.
After migrating to OpenSearch client, We started to use RestClient first which current release uses ApacheHttpAsyncClient 4 which does not support setRetryHandler and setServiceUnavailableRetryStrategy. That's why I decided to try ApacheHttpClient5Transport.
Regarding your question about why I'm not using AwsSdk2Transport, I have the same problem regarding retrying with it, also not able to disable https on local integration tests.
Any suggestion you have for handling retries?
Happy to help on README, what you expect me to add there?

@vahidiazar It's the first time you mention retries - does the first PUT request actually work, and it's the retry that fails with ApacheHttpClient5Transport? If so that is a different bug. Can you try to put a repro together maybe by modifying my simple demo.

@dblock , Sorry if I confused you.
I just tried to clarify why I'm trying to use ApacheHttpClient5Transport.
I created a demo here which demonstrates the bug (mentioned above) and it is not related to retry (there is not retry): https://github.com/vahidiazar/opensearch-java-client-demo/tree/opensearch-2.x-ApacheHttpClient5Transport

Thanks @vahidiazar. I'm going to close this issue as a dup of https://github.com/acm19/aws-request-signing-apache-interceptor/issues/101. Thank you for providing an example. If you want to turn it into a failing unit/integration test and PR that, I think we could get one step closer to an implementation, but I am not sure how that could work given the limitations of what the client exposes.


fixes #4 : Feature one

Added feature one with free vector and demo text

resolve conflicts, please.

It seems everything is fine but again branch conflict.

This no longer shows conflicts, I'm going to pull it in. It does not align with the pdf style wise though and will need to be adjusted in another issue. I have added the assets for the projects section to the images folder now.


Error while running TileCacheLayer in arcgis-runtime-samples-qt

While running TileCacheLayer on Qt, I got the blow error-
C:\Users\HP\Downloads\arcgis-runtime-samples-qt-main\arcgis-runtime-samples-qt-main\ArcGISRuntimeSDKQt_CppSamples\Layers\build-TileCacheLayer-Desktop_Qt_5_12_12_MSVC2015_64bit-Release\release\TileCacheLayer.exe crashed.
The project was built successfully but there is a problem while running.
Below are the Qt details:
Compiler: Qt_5_12_12_MSVC2015
Based on: Qt 5.15.2
Qt Creator 5.0.2

@kunalkhosla93 what version of the Runtime are you using? With our previous few releases, they all require Qt 5.15.2 minimum and MSVC 2019 on Windows. It could be a Qt and/or compiler toolchain mismatch with our minimum requirements.

Hi, i was unable to check as i am away from my device. But is it somehow possible to view offline tiled maps (downloaded in a directory) using one of the projects here. If so, kindly help me with the correct project name. Meanwhile ill check the requirements.
Thank you once again:)


⚠ ClamAV has degraded performance

In f2b547a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 140 ms

Resolved: ClamAV performance has improved in dea2dc9 after 5 minutes.


Running Time-LLM on GPT2

Hello, would it be possible to share the scripts (e.g. for TimeLLM.py, modeling_utils.py, etc.) adapted for GPT2 please? I do not have enough compute power to train on LLaMA in a feasible amount of time. I have attempted to adapt it on my own but am running into a lot of errors. Thank you very much!

Comment out the llama and replace it with the following code, which is useful for me and hopefully helpful for you.
from transformers import GPT2Tokenizer, GPT2Model, GPT2Config

self.gpt2_config = GPT2Config.from_pretrained(
    gpt2_root,
    output_attentions=True,
    output_hidden_states=True
)
self.gpt2 = GPT2Model.from_pretrained(gpt2_root, config=self.gpt2_config)
self.tokenizer = GPT2Tokenizer.from_pretrained(gpt2_root)

And make corresponding modifications to the following places.

Thank you, @lizhuoq 😄. I hope this solved your quesiton, @NimeeshaChan. Please re-open this issue if there are other questions.


Improve touch menu closing

open in chrome iOS: http://docs.meteor.com/api/accounts.html#Meteor-userId
open menu
try to close menu by swiping from the right side of the screen
instead, content scrolls horizontally

On some pages you can swipe the menu closed, but the header doesn't go back into place:

The scrolling issue is now fixed. I'll use #6 to record the swipe to open/close enhancement. Thanks for the screenshots, much appreciated! Closing this now.

@dominicnguyen welcome, nice redesign :)
I'm still able to horizontally scroll on api/accounts.html (1st shot above)


Fix get_point_cloud() due to API change

Did you run all the samples to see that they are okay?

Yes, that is how I was figuring out if each of them needs to be modified.


readme: Update Flathub badge

See https://fosstodon.org/@bragefuglseth/112753049810610017. This primarily serves as a way to let me sleep at night again.

LGTM, thanks :3
I'll bump it on the websites + tuba too later!

Great!


Add a flag to support showing serial output prior to ssh startup

So this flag (let's call it -ssh-early-console) would show the serial startup until we get a successful ssh connection, then stop that output. Not 100% sure this is possible, but it would be really nice to have.

Thinking about it, I think the solution here is to spin off a thread that just proxies the piped output from the qemu console to stdout, until a flag is set to indicate it should stop.


HLS support

Hi there,
It is not really an issue, but I could find another way to talk with you.
Does videojs-wavesurfer works with HLS live Stream?
Sorry to attack you like that, here where I work we have couple HLS videos on a monitoring page that I need to display its audio without unmute the videos.
thanks in advance,
Anderson Santos

hi @anderdev, have you tried https://github.com/videojs/videojs-contrib-hls in combination with this plugin?

Not yet, but will right now....thanks.

Hi @thijstriemstra,
Today together of a bunch of other things I had a chance to try it as per your suggestion.
Unfortunately, I might be doing something wrong...I create my videos thru an iteration....working very well.
with:
$.ajax({
type: 'GET',
url: '/spi/live-events/monitor',
dataType:'json',
success: function (data) {
$("#body").html(template(data));
for (i = 0; i < data.devices.length; ++i) {
devices.push(data.devices[i].id);
var player = videojs("video_"+data.devices[i].id,
{
plugins: {
wavesurfer: {
src: "https://tvnzioslive01-i.akamaihd.net/hls/live/245926/tvnzhlsingest/Duke/playlist.m3u8",
msDisplayMax: 10,
waveColor: "grey",
progressColor: "black",
cursorColor: "black",
hideScrollbar: false
}
}
});
// error handling
player.on('error', function(error) {
console.warn(error);
});
getNagiosInfo(data.devices[i].id);
}
}
});
my problem here is that on console I get this:
videojs.wavesurfer.min.js:4 Uncaught TypeError: Object prototype may only be an Object or null: undefined
I dont know what am I doing wrong.
Looks like that my src is not been accepted, is that the correct way to use wavesurfer?
thanks very much for you help.

@anderdev could you try this with a development version (e.g. non-minified) of videojs.wavesurfer so we can see the exact stacktrace in the console?

Hey @thijstriemstra , thanks very much for been helping me.
I just change to use the dev version and console still showing same error.

note that I changed src to my HLS stream.

@anderdev it looks like it cannot load WaveSurfer, are you sure it's loaded? Have you tried with the latest wavesurfer.js? Also make sure wavesurfer is loaded before this plugin.

So, do we have an order here?
I will try:

wavesurfer
2.videoJS
3.handlebars

let you know the results.

@anderdev yea, see https://github.com/collab-project/videojs-wavesurfer#using-the-plugin

other way around...hehehe videojs, wavesurfer and handlebars.
thanks a lot @thijstriemstra .

and probably the HLS plugin before or after videojs.wavesurfer..

@anderdev any news?

Hi buddy, not really...I have changed imports order and haven't had much success.
In the end, I get very busy with my Scrum Master role that I have put it aside for now...I might be able to be back on this during next sprint, let you know,.
thanks for asking.

I have been trying to do this without any luck. Tried combining the two plugins but wavesurfer throws DOMException: Unable to decode audio data when you the source to an HSL format video even with the contrib-hsl plugin installed. I was wondering if it would be possible to do a live streaming like the microphone but with hsl metadata instead? I really don't know ¯_(ツ)_/¯

With videojs-wavesurfer v2.0.0 I'm seeing the same error using:
var player = videojs('myAudio', {
    controls: true,
    autoplay: true,
    fluid: false,
    width: 600,
    height: 300,
    plugins: {
        wavesurfer: {
            src: 'https://d2zihajmogu5jn.cloudfront.net/bipbop-advanced/bipbop_16x9_variant.m3u8',
            msDisplayMax: 10,
            debug: true,
            waveColor: 'grey',
            progressColor: 'black',
            cursorColor: 'black',
            hideScrollbar: true
        }
    }
});

VIDEOJS: ERROR: Error decoding audiobuffer
The buffer passed to decodeAudioData contains an unknown content type.

Eventually got it to load using:
var player = videojs('myAudio', {
    controls: true,
    autoplay: true,
    fluid: false,
    loop: false,
    width: 600,
    height: 300,
    plugins: {
        /*hls: {
            src: 'https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8',
            type: 'application/x-mpegURL',
            withCredentials: true
        },*/
        /*
        wavesurfer: {
            msDisplayMax: 10,
            debug: true,
            waveColor: 'grey',
            progressColor: 'black',
            cursorColor: 'black',
            hideScrollbar: true
        }*/
    }
});

player.on('ready', function(error) {
    var url = 'https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8';
    console.log('Loading', url);
    player.src({
        src: url,
        type: 'application/x-mpegURL',
        withCredentials: true
    });
});

But then died with CORS error:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at ‘https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8’.
(Reason: Credential is not supported if the CORS header ‘Access-Control-Allow-Origin’ is ‘*’).

I set up a jsfiddle to demonstrate the issue.  I used unminified versions for easier debugging, and confirmed that swapping the order of this plugin and videojs-contrib-hls still produces these errors:

VIDEOJS: ERROR: Error decoding audiobuffer
Uncaught (in promise) DOMException: Unable to decode audio data

Maybe I'm off base, but it looks like videojs-wavesurfer calls WaveSurfer.load(url) directly.
The load method expects an audio URL, not a playlist, so maybe there's a way to grab the data coming from videojs-contrib-hls and use loadBlob instead.  If not, this might be a problem for wavesurfer.js to solve.

@jessefalzone thanks for the feedback but you're not actually using their video.js plugin in that example (the plugins section of the config only lists wavesurfer but not hls).

@thijstriemstra you're welcome.
I believe it is working, since videojs-contrib-hls registers itself automatically.  You can test this by commenting out the wavesurfer options
var player = videojs('my-video', {
	controls: true,
	plugins: {
		/*wavesurfer: {
			src: src,
			normalize: true,
			debug: true,
			barWidth: 3,
		},*/
	},
});

and the m3u8 video will play correctly (and then it will break if you remove the videojs-contrib-hls script).  The hls plugin option appears to be optional.

That's weird, how would I specify any initialization options? video.js provides an API that allows me to detect if other plugins are in use and videojs-contrib-hls is currently not able to be detected using that API, making it hard to interact and work with that plugin.

I've changed your earlier example to the following (notice hls is nested in html5, not plugins):
var player = videojs('myAudio', {
    controls: true,
    autoplay: true,
    fluid: false,
    loop: false,
    width: 600,
    height: 300,
    html5: {
        hls: {
            src: 'https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8',
            type: 'application/x-mpegURL',
            withCredentials: true
        },
    },
    plugins: {
        wavesurfer: {
            msDisplayMax: 10,
            debug: true,
            waveColor: 'grey',
            progressColor: 'black',
            cursorColor: 'black',
            hideScrollbar: true
        }
    }
});

CORS headers will need to be correct though, but I'm not sure it's necessary to specify these options since it's able to use the source in the <source> tags.
At any rate, if you can't detect contrib-hls then I'm stuck :-) I'm not too familiar with the internals of videojs.

What about this API? https://github.com/video-dev/hls.js/blob/master/doc/API.md

See https://github.com/katspaugh/wavesurfer.js/issues/1078 which basically has to be fixed before we can fix this ticket.

Figured out how to create a local playlist.m3u8 for local streaming.
Grab latest ffmpeg build from https://www.johnvansickle.com/ffmpeg/ and unpack it somewhere.
Create a test dir:
mkdir ffmpeg-hls
cd ffmpeg-fls

Create a convert.sh script containing:
export app=./ffmpeg-git-20171018-64bit-static/ffmpeg

# convert the input file to TS segments using the libx264 and aac encoder
# and create an M3U8 live playlist to use for live HLS source
${app} -i $1 -map 0 \
-c:v libx264 -c:a aac \
-f ssegment -segment_list playlist.m3u8 \
-segment_list_flags +live -segment_time 10 \
out%03d.ts

cat playlist.m3u8

Make it executable:
chmod +x convert.sh

And run the script on some mp4 file, e.g. source.mp4:
./convert.sh source.mp4

This will create a playlist.m3u8 file and related media files.
Now enable a local Apache server to serve the file. Add new mimetypes to /etc/apache2/mods-available/mime.conf:
AddType application/x-mpegURL .m3u8
AddType video/MP2T .ts

And add this line to your apache vhost:
Header set Access-Control-Allow-Origin "*"

Restart the Apache server.
Finally, make the file accessible for Apache:
mv ffmpeg-hls /var/www/html/

The stream is now available on http://localhost/ffmpeg-hls/playlist.m3u8

HLS is now supported in video.js (since v7.0):

Play HLS, DASH, and future HTTP streaming protocols with video.js, even where they're not natively supported. Included in video.js 7 by default! See the video.js 7 blog post

More at https://github.com/videojs/http-streaming
So this should already be possible.. testing..

I tested the multi example with this:
const audioPlayerOptions = {
    controls: true,
    autoplay: false,
    loop: false,
    muted: false,
    fluid: false,
    width: 600,
    height: 300,
    bigPlayButton: true,
    plugins: {
        wavesurfer: {
            backend: 'MediaElement',
            displayMilliseconds: false,
            debug: true,
            waveColor: '#3e5226',
            progressColor: 'black',
            cursorColor: 'black',
            interact: true,
            hideScrollbar: true
        }
    }
};

// create audio player
let audioPlayer = videojs('myAudio', audioPlayerOptions, function() {
    // load m3u8 file
    audioPlayer.src({
      src: 'https://d2zihajmogu5jn.cloudfront.net/bipbop-advanced/bipbop_16x9_variant.m3u8',
      type: 'application/x-mpegURL'
    });
});

And see this error:
 Security Error: Content at http://localhost:8080/examples/multi.html may not load data from blob:http://localhost:8080/81fa0ed5-0627-43e0-9122-8c0baa5044a5.

It seems video.js vhs implementation creates a blob url here:
this.mediaSourceUrl_ = window.URL.createObjectURL(this.masterPlaylistController_.mediaSource);

this.tech_.src(this.mediaSourceUrl_);

And passes it to the videojs-wavesurfer middleware. Unfortunately there is some Security Error when trying to load this blob:http://localhost:8080/81fa0ed5-0627-43e0-9122-8c0baa5044a5 Blob.

Sorry to bother, any news about this problem? I just bumped with this problem, wavesurfer.js cannot get the file while using m3u8 to load audio. The problem is similar to "Unfortunately there is some Security Error when trying to load this blob:http://localhost:8080/81fa0ed5-0627-43e0-9122-8c0baa5044a5 Blob.".


When attempting to return an embedding for a string containing the character ë I get the error "Your request contained invalid JSON: 'utf-8' codec can't decode byte 0xeb in position 443". When I remove the character it works. I am retrieving the embedding from the _predict endpoint:
POST http://localhost:9200/_plugins/_ml/models/{model_id/_predict
{
    "parameters": {
        "input": ["This is a string containing Moët Hennessy"]
    }
}

Removing the special character and replacing it with e works. If I request the embedding directly from open AI (with special character) it works fine.
POST https://api.openai.com/v1/embeddings

{
    "input": "This is a string containing Moët Hennessy",
    "model": "text-embedding-ada-002",
    "encoding_format": "float"
}

How can one reproduce the bug?
Steps to reproduce the behavior:

Setup Open AI connector described here:
Retrieve embedding for string described above.

What is the expected behavior?
Embedding should be returned for strings containing special characters.
What is your host/environment?

Ubunutu
Opensearch 2.11 running in a local docker setup.

Do you have any screenshots?
N/A
Do you have any additional context?
N/A

Thanks for reporting this issue. Will fix this in PR https://github.com/opensearch-project/ml-commons/pull/1691


Dependencies between resources

It is often necessary to declare dependencies between resources in a composition.
Some default crossplane cloud providers have dependencies between specific resources. The crossplane docs contain an example of a MySQL managed resource that depends on a resource group. The resource group reference fields are implemented in the manifest of the MySQLServer managed resource.
However for the kubernetes provider we would need a more arbitrary resource dependency declaration on the Object, because we don't know what the resource actually is. Maybe a dependsOn array of objects with resource references defined by group, version, kind, name and namespace.
The dependsOn property should be a sibling to the manifest property.
Only once status.condition named Ready is true for all referenced resource, the resource is actually templated and posted to the server.
We could reference resources from the composition or resources that aren't even part of the composition. For referenced resources within the composition we could simplify the reference and don't need to put all defining properties like group, version, kind, name and namespace, but rather only the name of the resource in the composition. All the rest can be deferred from the manifest of the referred resource in the composition.

@jonnylangefeld I just created https://github.com/crossplane-contrib/provider-kubernetes/issues/13 for a design proposal to enhance provider kubernetes with a few new features and it looks the one that you are looking for is exactly what I'm proposing.

I created a reference implementation at: https://github.com/morningspace/provider-kubernetes/tree/capabilities-shim.
A demo project to demonstrate the idea at: https://github.com/morningspace/capabilities-shim.
All design details on the features that I'm proposing are documented at: https://github.com/morningspace/capabilities-shim/blob/main/docs/enhanced-provider-k8s.md.

Feel free to make comment or suggestion.
cc: @turkenh @negz

Maybe a dependsOn array of objects with resource references defined by group, version, kind, name and namespace.

This has been addressed since https://github.com/crossplane-contrib/provider-kubernetes/pull/21 is merged. See examples at: https://github.com/crossplane-contrib/provider-kubernetes/tree/main/examples/object/references
@jonnylangefeld

I lost alot of time thinking dependsOn would prevent the resource from being composed only to find out crossplane was not waiting. misleading. thanks for this issue.


Switch modes issue

There are many points, so the app provides you a point selection according to points’ numbers. From here you can switch editor modes, when you go to an “entrances/houses” one and switch back it is impossible to move a map anymore. However this state can be reset by tapping a “+” button

https://t.me/everydoor_ru/2942 video is here

Done in 0.7.


Could not determine java version from '9.0.1'.

When i try to build it on Windows machine, i got java error
FAILURE: Build failed with an exception.

* What went wrong:
Could not determine java version from '9.0.1'.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Sorry this project is unmaintained. I dont assist with build issues, but the code is all there. You can see how it works.


Wiki nodejs版本需要更新

需要从目前的v10+变成v12+
$ yarn install
yarn install v1.22.10
[1/4] Resolving packages...
[2/4] Fetching packages...
warning<EMAIL_ADDRESS>Invalid bin field for "url-loader".
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=12". Got "10.19.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

升级成v12/14可以正常运行

改成 13+ 好了，毕竟Action一直都是在node 13下，而我本地开发一直四 node 15


multi-tissue functionality

This implementation modifies the concatenate function and allows to work with multiple tissues in the same anndata object.

Connected to theislab/scanpy#1073 and https://github.com/giovp/spatial-scripts/blob/master/multiple_slices_functionality.ipynb

Do you want to update requirements.txt already for latest anndata in the PR from scanpy?

Sorry @giovp, but we need to make this generic or add something in scanpy. And I think making it generic will benefit us more. Please let’s continue discussion in #295


Add method to add to a MemSource after creation.

This PR adds a method that allows a MemSource to be extended dynamically after creation. It is useful when implementing an interface that assumes this is possible (for me, the motivating interface is in https://github.com/linebender/piet).

I guess there might be an issue around what dynamic functionality you would be willing to support. If this library computes information about fonts eagerly and non-incrementally, then adding a new font could be a very expensive operation, requiring a rebuild of a lot of things. I still think its worth having the methods in that case, with a caveat in the docs that they should be avoided if possible.

I've also added https://github.com/derekdreery/font-kit/tree/multi_downcast to get at a MemSource dynamically for my needs. I don't want to open loads of PRs at the same time, but would invite comments on it.

@bors-servo r+
Looks good!

:pushpin: Commit 258a4da has been approved by jdm

:hourglass: Testing commit 258a4dadadac2005c46da8b882337440bbdafbd6 with merge 4b1ede011ff407e947c965942662dc0a177d9bd0...

:sunny: Test successful - checks-travis
Approved by: jdm
Pushing 4b1ede011ff407e947c965942662dc0a177d9bd0 to master...


SAMPLR PR. DO NOT MERGE

THIS PR IS CREATED SO THAT THE DOCUMENTATION PREVIEW LINKS CAN BE SHARED WITH THE SALES TEAM. ##DO NOT MERGE THIS PR##

@cbms03 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement
@microsoft-github-policy-service agree [company="Microsoft"]


do not limit the length of the input

Please do not limit the length of the input, as I will use AI to analyze the code, thanks

This is a limitation of Bing. Even if the character limit of textarea is modified, the API will report an error when it exceeds 2000 characters.

You can split the content you want to send and send Bing a command to receive your content in multiple times.

Got it, thanks for the reply.


storage-attached hook fails

http://paste.ubuntu.com/24518169/

is this on localhost or public cloud?

AWS

Where is your charm located so I can reproduce? Also is this for the irc shout server? If so, you should check out lounge which is a fork of that (https://thelounge.github.io/)

@battlemidget thanks, I'll check it out.
This is the charm: cs:~marcoceppi/xenial/shout-irc-0


Add CLI on agent

Can't be test because mock schedule won’t work.

@theotchlx  I thougth i merged all the necessary but it disapeared. Let me check that. I m confused

LGTM. I tested your CLI and it works :) :+1:


Import component into .mdx file?

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
Import a component form the app/components directory into an .mdx file.
Expected Behavior
Import and render a component.
Actual Behavior
It renders a text string, not an imported component.

@freekrai this is really nice template, thank you. Kind of standardises a number of approaches, love the option between remote versus local. I am genuinely confused about the component import though.

This uses Stripe’s Markdoc  for rendering so it’s a tad different.
You’d have to add components via https://github.com/freekrai/remix-docs/blob/main/app/components/Markdown.tsx and https://github.com/freekrai/remix-docs/blob/main/app/utils/markdoc.server.ts

I'm working on some docs around this stack which will include a section on markdoc and adding more components.

Ah yes, my bad. Thanks 🙏🏻

MarkDoc docs are solid, might only need a quick overview of it.

Yeah markDoc is nice


WRQ-31515: Fixed Scroller, Slider, VirtualList and VirtualGridList to set prop value to default when undefined is passed for the prop value

Checklist

[x] I have read and understand the contribution guide
[x] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
In https://github.com/enactjs/sandstone/pull/1613, we removed defaultProps and implemeted this feature manually.
After this implementation, if undefined is passed for a prop value, the prop value remains undefined.
However, defaultProps(https://legacy.reactjs.org/docs/typechecking-with-proptypes.html#default-prop-values) sets prop value to default value when undefined value is passed for the prop.
Resolution
Fix Scroller, Slider, VirtualList and VirtualGridList to set prop value to default value when undefined is passed for the prop value
Additional Considerations

I checked React's defaultProps for falsy values(null, undefined, false, NaN, 0, -0, "") and found that it sets the prop value to its default value only when undefined is passed.
https://react.dev/reference/react/Component#static-defaultprops

I found that the default value of wheelInterval in controls of Slider sampler was not loading properly. So I fixed it in this PR.
https://github.com/enactjs/sandstone/pull/1658/files#diff-4f80023ae6bb7b165147824d15263e1a011ab278fdf4a5f6ddbea07fecca1a31R471

Links
WRQ-31515
Comments

@juwonjeong, Please resolve the merge conflict.


rename package

The new v4 API is now completely changed. Since it's all about writing models now, the package's name should be changed.
Proposal:

RawModel

So, the name has been changed from contextable to rawmodel? That threw me for a loop for a second there. LOL!
Scott


Explorer not updating

Issue Type
Bug
Description
Related to #44691
The explorer doesn't seem to update when changing git branches.
This can be fixed by closing vs code and reopening.
VS Code Info
VS Code version: Code 1.20.1 (f88bbf9137d24d36d968ea6b2911786bfe103002, 2018-02-13T15:23:28.607Z)
OS version: Darwin x64 17.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6267U CPU @ 2.90GHz (4 x 2900)

Load (avg)
3, 3, 3

Memory (System)
16.00GB (0.41GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron .

Screen Reader
no

VM
31%

Extensions (16)

Extension
Author (truncated)
Version

EditorConfig
Edi
0.12.1

vscode-docker
Pet
0.0.25

sort-lines
Tyr
1.4.1

vscode-wakatime
Wak
1.1.22

html-css-class-completion
Zig
1.17.1

vscode-eslint
dba
1.4.7

docthis
joe
0.6.0

VS-code-vagrantfile
mar
0.0.7

atom-keybindings
ms-
3.0.4

debugger-for-chrome
msj
4.1.0

color-highlight
nau
2.3.0

marko
pca
0.2.2

subtle-brackets
raf
2.1.0

sass-indented
rob
1.4.8

preview-vscode
sea
1.4.2

stylelint
shi
0.33.0

(1 theme extensions excluded)

Reproduces without extensions

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/


Better codecov conf

wait all 3 build before reporting
color range between 70 and 100
round down with 2 precision
treshold of 0.1% for success in ci
require coverange change for commenting

@Meakk : this does not seems to work as expected, investigation needed.


Cache error fallback (bis)

Hello 👋 !
This subject has already been talked about in this issue :

If the call to the cache store fails. Should it fall back to the wrapped function?

We are facing the same issue/requirement using the cache.wrap(...) function : if a call to a multi-cache store fails (in-memory + redis), let's say because Redis is unreachable, then we would expect the cache-manager to call the wrapped function instead of raising an error.
Shouldn't the cache act as a performance optional path to get data, but not interfere to get this data if it is facing errors ?
There used to be a ignoreCacheErrors flag to activate this behavior, but it has been removed in the 5.0.0 version.
Is it possible to reintroduce it ?
Thanks 🙌

Hi @MathieuGuillet - we are working on this issue with Keyv version 5 as it should not cause a failure and will use the same code to fix this module. This might take about 60 days to resolve. Just FYI

Hello @jaredwray do you have an issue to follow to know when it is good?


[Flink AI Flow][AIFlowClient]

Modify AIFlowClient to affiliate notification service's high availability modifications.

code merged


Dramatically reduce encoding times for some equality formulas

Mitigates (but does not entirely fix) #108.

This is definitely an improvement: examples/select-flatmap.ds no longer hangs doing formula encoding. However, now it hangs waiting for Z3. We may need a different approach.


fix db transaction and ModelFactory Create

@mlantz should be ok now?

I strongly disagree with having the transactions removed from the try catch. The transaction and commit only occurs within the try, so with the catch doing a rollback what is it rolling back? As I see it there has not been a db action to rollback. I'd be ok with the content of the try being wrapped inside a DB::transaction(function(){ //content in here }); but thats about as far as I would go. These are 'templates' for an app structure so I would encourage people to mod them as they need within their own apps. Please remove those and put in a new PR for the transactions if you want. Thanks.

@mlantz I found these problem when i invite a person and email not configured correctly,Actually i think transaction should start before user created,code like this:
public function invite($info)
{
    $password = substr(md5(rand(1111, 9999)), 0, 10);

    DB::beginTransaction();
    try {
          $user = $this->userRepo->create([
              'email' => $info['email'],
              'name' => $info['name'],
              'password' => bcrypt($password)
          ]);
          $this->create($user, $password, $info['roles'], true);
            DB::commit();
        } catch (Exception $e) {
            DB::rollBack();
            throw new Exception("We were unable to generate your profile, please try again later.", 1);
      }
      return $user;
   }

if not doing this,when email send fail the new user already created in db.
and can you tell me why DB::rollBack() can't write in catch block? in my opinion, when start a db transaction, database will lock related tables , after excute DB::commit() or DB::rollBack() release this lock, and it doesn't matter how php code write.Any Exceptions occur in try block  we rollback what db have changed seems won't have a problem.

I'd rather just have them wrapped in the DB::transaction closure. Its much easier to read that way and achieves the same thing. I didnt really think it through on first review, but you're right the rollback would just open up the lock. In the end I dont like how it reads. I mostly dont like the idea of the DB::beginTransaction being outside of the try catch. I'd much rather have it all bundled together with the closure. This should also be a separate PR since its addressing another issue.

@mlantz , in this place if we want use DB:: transaction, must change some logic, because except databases operations ,there have a email send function.when email can not send,should rollback too or give different notice message.so it's better let yourself modify this part.

Hey @Ray-Cheng I just went ahead and changed it. but I'd like to get your other fix in. I appreciate how much energy you've put into our chats. Can you drop your transactions, so we can merge in the rest? I haven't done a new release yet but I will be soon.

@mlantz ,rolled back commits related db transaction


feat: adapt configurable max_headers

This PR adapts configurable max_headers by hyperium/hyper#3523.

Thank you!