Remove remaining uses of hashValues in the framework

See https://github.com/dart-lang/sdk/issues/25217

This is the engine->framework roll where hashValues was deprecated:
https://github.com/flutter/flutter/pull/105038


PLAT-141610: Fixed MediaPlayer layout issues.

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
[x] Documentation was verified or is not changed
[x] UI test was passed or is not needed
[x] Screenshot test was verified or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
There are various layout issues:
You can't see the progress value unless you hover over it on skins: Cobalt, Copper, Titanium.
Button size are quite small on skins Carbon, Cobalt, Copper, Electro, Titanium. Since there is no Ui guide,
it would be good to look the same as Gallium.
Buttons are too close to one another and they overlap with the progress bar on skins: Electro and Titanium.
Shuffle button active is too close to the color of the symbol on skins: Cobalt and Copper(only on night mode)
The play button overlaps with the progress bar bubble on Carbon skin.
Resolution
Added the @agate-mediaslider-fill-bg-color variable in color-base and fixed it for  Carbon, Cobalt, Copper, Titanium.
Changed the inherit values from @agate-mediacontrols... with some existing variables.
Additional Considerations
Links
PLAT-141610
Comments
Enact-DCO-1.0-Signed-off-by: Alexandru Morariu<EMAIL_ADDRESS>

I do not know the reason, but when the screenshot test is executed, the skin is not applied on MediaPlayer.
Could you please make a CLM issue so we can solve this problem later? We can do it later depending on the schedule, but it would be nice to have a ticket so we don't forget it.


README Formatting

  Please Check The Preview

General formatting / spacing

Added quicklinks

If you have any questions.

In case you do not agree with some of the changes made in this PR,
please first let me know about them, before disregarding it entirely so
we can possibly find a middle ground and have it not go to waste.

If you want to know more about Markdown check out:

@danwillm Could you please review this PR~

Sorry, have been away for a while due to examinations otherwise would have merged this in.
We're currently working on a refactor to the codebase to include a lot more features so this might be outdated soon if we merge it now. Do you mind if we leave it open for another week or so until we've got the refactor done, then we can talk more about what might be useful to add to the readme? Thanks for this though, it looks good :)

Well you could also merge it and in a week I could adjust it?
Sounds like only the content, such as features, will be modified?

??


Wrong printf format specifier width when building windows x64

libuv\src\win\fs-event.c(408): warning C4477: '_snwprintf' : format string '%.*s' requires an argument of type 'int', but variadic argument 2 has type 'std::size_t'
file_info->FileNameLength / sizeof(WCHAR) is 64bit, but printf expects int precision
i.e. add extra cast
file_info->FileNameLength / (DWORD)sizeof(WCHAR)

Dont you want to write the patch?

Sorry, where I my manners :) I'll submit pull request later today.

@knopp friendly ping.

I thought I had forgotten something.
https://github.com/libuv/libuv/pull/963


feat: add ilab model upload command

ilab model upload allows for uploading safetensor and GGUF models created by InstructLab users to
supported endpoints
Hugging Face is the only supported endpoint in this commit, but provisions have been added to eventually support OCI and S3 endpoints as well
Issue resolved by this Pull Request:
Resolves #2642
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[x] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[x] Unit tests have been added, if necessary.
[ ] Functional tests have been added, if necessary.
[ ] E2E Workflow tests have been added, if necessary.

Test Repos I am using:

https://huggingface.co/nathan-weinberg/test-upload-gguf/tree/main
https://huggingface.co/nathan-weinberg/test-upload-safetensors

@mbestavros would like you to take a look as well please!

@courtneypacheco I see to be getting a couple new mypy errors now - any idea if this might be related to the version change?
  Error: tests/test_lab.py:128: error: Argument 3 to "Command" has incompatible type "tuple[str, str]"; expected "bool"  [arg-type]
  Error: tests/test_lab.py:129: error: Argument 4 to "Command" has incompatible type "tuple[str, str]"; expected "bool"  [arg-type]

@cdoern not sure how I can correct for this? The command requires 3 args from a user and I don't want to add defaults that don't make sense

My largest comment is on what should live in src/instructlab/cli and what should live in src/instructlab/model Overall looks great. A few other comments I think should be addressed as well.
Additionally, are we going to be e2e testing this? Unsure If I feel comfortable merging this without it being tested. I know this will be difficult, so I would be fine scooping some work for e2e testing as long as it is concrete.

Just because something isn't "E2E tested" doesn't mean it's not tested - I've introduced a large number of unit tests here - we can look into more testing at a later date, but I won't be scoping any more for this PR.
I've tested it manually, feel free to do the same.


Reduced size of integration tests image

Motivation
The chown & chmod operations added in #9627 are causing docker to actually duplicate all the files in the overlay filesystem, increasing the image size by ~2GB.
After this change the test image size is ~2.5 GB.

Great catch @merlimat . I didn't notice this since I have been running with -Ddockerfile.build.squash=true which will pass --squash to the docker image build.


Entities with bounding spheres do not make use of EntityView.defaultOffset3D.

For example, load the CZML demo in Sandcastle, and try double-clicking some objects to track them with the camera.  The initial view when tracking a new object should have the horizon or the limb of the Earth visible in the default view.  The Molniya in particular should not be shown floating in space with no trace of the Earth anywhere.  It would be great if a unit test would check for Earth visibility in default views of things.
Git bisect shows this broke in 6a507f2e5.  Things work fine in the commit before, ea27cb7c0.

This is by design and is a stop gap until EntityView is rewritten or deprecated for a new class that better handles bounding spheres and tracking them.  It's going to have to happen soon because of the vector data on terrain changes.
I took a quick look and we can make it respect defaultOffset3D, but it should be the same default that the Camera uses for adjustBoundingSphereOffset, in Camera.js line 2347.  We can just update defaultOffset3D to match those values for now, ultimately they will probably be unified.  A slightly down looking view oriented north is an optimal default for most of our users.
This change may be wrapped up in larger changes for 1.10, but if not I'll look into opening a PR for just this at some point.

A reasonable workaround was merged into 1.10.  Closing.


Correcting spec for Next Gen MCAS risk levels

Who is this PR for?
Patch to the Next Gen MCAS risk level spec.
What problem does this PR fix?
Additional test was bad, didn't properly create assessments before testing the student risk level calculation.
What does this PR do?
Fixes my mistake. 😧

@jhilde New PR template! :D


Update YoutubeSearch to perform a single search instead of two

Currently, YoutubeSearch performs a snippet search, then gets more data by performing a contentDetails search. It seems like it should be possible to combine the two by using _ytService.Search.List("snippet, contentDetails") to get all the information in a single shot.

False alarm. service.Search.List does not return ContentDetails at all.


Unable to unmute audio player after page refresh

I am having a weird issue not being able to unmute an audio player if the page has been refreshed after muting the audio. The only way to get the audio unmuted is by increasing the volume, which is impossible in an audio player as the volume bar is not visible.

Yes, you're right, it should work.

Closed with https://github.com/2fdevs/videogular/commit/a8ca29909ec11bc6e121f4ab79e10894293dafd4

@cburger  how did you solve this issue??


[Question] why Swarrot\Broker\Message::$id binds on delivery_tag ?

I can't use application's message identifier like this :
$publisher->publish(new Message("body", [], 34));
$message = $consumer->get();
$message->getId(); // 34

The Message identifier return the delivery_tag.
I can make a PR but I don't understand this behavior, so I'm asking first.

The message id is precisely meant to represent the id in the queuing system (used to ack or nack the message). AMQP does not let you control that id.
If you want to have an application identifier, store in the message body or in the message properties.

The message have a property section :

properties are :

And message id is defined :

When I send a message trough Rabbit, the message identifier is well handled by PhpAmqpLib but it's not used by Swarrot (delivery_tag is used instead)

I agree that I can handle it in body or headers, I'm just trying to figure out what's the better way to do it.


API call for making a new offer

We add a new API call to allow building BOLT 12 offers! We need this for our integration tests over at lndk.

Reminder: I need to allow the caller to set a custom amount_msats value.

Merging this since my next PR depends on this.


Issue #469 Take ClassLoader into account when using default URI

Updated change with a test

Looks okay to me... needs rebasing now, but once that is done I'll be happy to merge.


Home assistant add devices..

I see vera and harmony can detect all the devices on those platforms.. is this feature supported for home assitant? Would be nice to not have to manually add 30 switches..
Cheers!

Will look into this

So, I will add the get for all "states" i.e. devices and have helpers for either, light (has dim) or Switch which is on and off.

That would be awesome. I'm currently controlling a couple of Home Assistant scenes with API calls but if ha-bridge could 'see' everything natively, that's a killer feature. Can't wait to test that out.

Check this script out for HASS import.
https://community.home-assistant.io/t/why-cant-emulated-hue-work-with-onhub-or-google-home/5794/40?
It would be great to be able to pull everything in at once and then enable / disable / edit the parts you want to use.

Hi GrubbyDungles,
I am very new to the programing/automation world. Now I have managed the set up the HA-bridge but have difficulty adding devices to it (from Home Assistant). I saw the script you paste but have no idea how to execute it. Would you please help me out here. Really appreciate!!

@Yungling Sorry - I just linked to that script because I thought it was relevant. If you want to use it the best thing to do would be to follow the link and ask the guy who wrote it.
Otherwise, there is a general guide with examples for home assistant here.
https://community.home-assistant.io/t/general-ha-bridge-guide-hue-bridge-for-alexa-or-google-home/6758
Once you add a few lights in you will get the gist, if you have any problems post in that thread and someone will help.

You can check out the beta version posted in #344 for HomeAssistant connection

Thanks to @GrubbyDungles  @bwssytems
However i still cannot manually add any devices from Home assistant to HA bridge (i managed to add Harmony though. Works great!)
I pasted my web request i used to turn the lights on by using IFTTT...Not sure if this helps....
https://XXXXXXX.duckdns.org/api/services/light/turn_on?api_password=XXXXXX

@Yungling https:// will be your problem, something to do with letsencrypt, java and the location of certificates. The problem was solved, check this link to the solution.
https://community.home-assistant.io/t/why-cant-emulated-hue-work-with-onhub-or-google-home/5794/35?

@GrubbyDungles i tried right away and it worked! Now i can turn on/off my light thru HA bridge, which is a lot faster vs. thru IFTTT. Thank you very much for helping!


Are there plans to develop other client SDKs, such as Flutter Dart?

Or what about Swift?

In the future we'll definitely want to support other client SDKs! Right now though we want to focus on react and make the core Instant experience great! We're happy to highlight community SDKs in the repo, feel free to give us a ping on discord and we'll take a look!


Qustion about robot.compute_spatial_twist_jacobian()

Hello,
When I  compute joint velocity from twist, the function robot.compute_spatial_twist_jacobian() in 0.8 version I want to use disappears now in 3.0v.
So is there similar method to get the jacobian?
Thank you a lot!!!

All functionalities related to kinematics and dynamics are now only supported through Pinocchio. You can call robot.get_pinocchio_model() to create a pinocchio model and you can refer to this file for usage.
https://github.com/haosulab/SAPIEN/blob/dev/python/py_package/wrapper/pinocchio_model.py

Thank you!
I have used robot.create_pinocchio_model() to compute the ee link's jacobian.
And maybe the first 3dims of the jaccobian is last 3 dims of qvel, I mean the of sort of  linear velocity and angular velocity is different?

I am not sure what you are trying to achieve here. The Jacobian simply maps joint velocity (in the order of qvel) to link twist (in the order of $v, \omega$). Twist is not linear and angular velocity unless the link's frame aligns with its center of mass.


failed to retrieve pipelines, probably due to kfp-api crash loop backoff

I installed Kubeflow on a bare metal Linux (Ubuntu 20.04.1 LTS) server following https://charmed-kubeflow.io/docs/quickstart
watch -c juju status --color displays errors for (after +30 minutes of waiting):
katib-controller/0*
kfp-api/0*
kfp-persistence/0*
kubeflow-profiles/0*
seldon-controller-manager/0*
In all cases, the reason is a "crash loop backoff".
On the main Kubeflow dashboard I receive a red-labeled message: "Unable to contact Profile Controller"
When I try to access pipeline I see "Error occured while trying to proxy to: <IP_ADDRESS>.nip.io/apis/v1beta1/pipelines?page_token=&page_size=10&sort_by=created_at desc&filter=".
I would appreciate Your help.

The reason was (probably) not enough RAM (16GB).

Adding extra RAM (up to 32GB) didn't solve the problem.
There is a crash loop backoff ERROR for:
kfp-api/0*
kfp-persistence/0*
kubeflow-profiles/0*
seldon-controller-manager/0*


feat: publish the new openapi docs on /api/v2 and move apiary docs to /api/v2-old

I'm no Redocusaurus expert - are the fragments in the URLs the same as in the current docs?
e.g. https://docs.apify.com/api/v2#/introduction/rate-limiting/per-resource-rate-limit targets the page about the rate limiting?

Ah, cool, it seems to work. lgtm then 👍🏽


Add usage output to commands without arguments

Description:
On running commands like nerdctl cp without arguments only standard cobra.ExactArgs() error gets printed. IsExactArgs() function in pkg/errutil/errors_check.go is solving this issue.
Issue:
#1585
Test:

I have build the repo after the changes this PR consists of.
Used the binary and try running with less or no arguments to check the error output.

Testing example:
$ ./nerdctl pull
FATA[0000] "nerdctl pull" requires exactly 1 argument.
See 'nerdctl pull --help'.

Usage:  nerdctl pull [flags] NAME[:TAG]

Pull an image from a registry. Optionally specify "ipfs://" or "ipns://" scheme to pull image from IPFS.

Signed-off-by: Aman Sharma<EMAIL_ADDRESS>

Please squash commits

@djdongjin can we merge this?
Please let me know if there are any other changes.
Thanks!


Adds vscode settings to gitignore

Codecov Report

Merging #981 (2cd46d2) into master (278df51) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #981   +/-   ##
=======================================
  Coverage   81.22%   81.22%
=======================================
  Files          57       57
  Lines        4533     4533
=======================================
  Hits         3682     3682
  Misses        851      851

Continue to review full report at Codecov.

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


Add "fs/promises"

fs/promises is not recognized as part of fs. This fixes it.
Context: https://github.com/mozilla/node-fx-runner/pull/139/files/c92c14ec8b2897b52a8c9610d3c1fbaf52d980c7#r1713052619

I noticed process is missing too, should it be added?


Tooltips for Icons

Hey components team,
I'd love it if we could get tooltips for icons in V3. Once the app dev group gets the css for them, maybe we can implement that design for tooltips?
Courtney Fleeger needs to get it together and get that done for this to happen. Here's the drafted file that is a work in progress with some prototypes: https://www.figma.com/design/wB94K8rE0YJiMa02D6U9ng/WIP-%2B-Team-Troubleshooting?node-id=4480-981&t=9AmH9S6ey96WbLTn-1
Here's the NN group's example of how to use tooltips, which includes using them for icons: https://www.nngroup.com/articles/tooltip-guidelines/
Thanks!
Courtney Fleeger

If we create a tooltip component, I'm sure it could be incorporated into the icon component as well.
My only real concern is accessibility. Here's an implementation that's been discussed in W3C that seems to meet requirements:
https://zoebijl.github.io/apg-tooltip/
Important points for tooltips are:

Tooltip needs to make it clear which element it belongs to.
Opens with hover and focus, and stays open as long as focus remains or the mouse is on the triggering element or the tooltip itself.
Closes with escape without moving the focus or mouse.
Must not include interactive content in the tooltip content.

There is an alternative pattern called toggletip that allows for interactive content, but then the actual element that opens it must not itself be an action beyond opening the toggletip.

The tooltip file is updated: https://www.figma.com/design/wB94K8rE0YJiMa02D6U9ng/WIP-%2B-Team-Troubleshooting?node-id=4480-981&node-type=canvas&t=IuDskvFkZO5rH9LB-0
I also have some sorta confusing Figma css in a file for you if you wish to use that.

Great, I'll bring it up with the component developer team.
I see there's also a page for popovers in Figma, that would likely be part of the same component, since much of the code would be the same.
As far as Figma's CSS goes, that tends to hinder more than help. Figma doesn't take into account how that CSS fits into the rest of the page, and it has no concept of responsive design or accessibility. It's also often just plain wrong. I do sometimes use it for very specific things, like copying a gradient, but I tend to look at the "Properties" view in Figma rather than CSS.


Terraform provisioner for custom FE config file

As a developer I want to include the FE customisation config file as an external resource because this action will make readable and efficient the process of deployment.
Background
FE provides a specific customisation of the webapp configuration file. (Eg. background colour and components).
Currently terraform hosts the configuration file in the main file as a list of attributes/values.
Tasks

[x] FE: #1838
[x] BE/FE: to decide the location where to store the file
[x] BE: to change terraform to host the new info (provisioner)

Acceptance tests
When Terraform script is deployed the webapp must work with or without the customisation info.

Implemented as javascript configuration profiles under '/profiles' in the web application public folder.
The terraform provisioner has already been extended for supporting this mechanism in Open Targets Platform and Partner Preview deployments.


ffmpeg configuration

ffmpeg isn't working right. Here is my nging.config file. How to properly install ffmpeg for nginx-rtmp module.
With normal apt-get installation there was a codec issue with libfdk_aac and then i rebuild it from source.

I recommend you testing first separately ffmpeg directly from terminal.
and test only nginx rtmp alone.
sometimes is about permissions of nginx to execute ffmpeg, but first needs to make shure both awork correctly intependtly

@Somosmultimedia  So much thanks for reply. Actually i have tested that command in terminal and ffmpeg successfully encoded and saved livestream into file. Streaming to application 'live' broadcast RTMP and streaming to application 'hls' broadcast single bitrate m3u8 file but there is no multibitrate m3u8. I think nginx is not able to execute ffmpeg. Please let me know can i set nginx permission to execute ffmpeg binary.
Thanks again

in the nginx.conf uncomment and set
#user  nobody;
to:
user root;

then restart nginx

The problem o found is ffmpeg was exiting continuously and find this soluition.
hls_continuous on;
Thanks everybody.

path shouldn't be /usr/local/bin/ffmpeg ?


docs: Adds recipe for reusing containers

This commit adds a cookbook recipe showing how to use containers from one pipeline in another

I discussed this in detail with @helderco . My understanding from our call is that this code snippet only changes the visualization of the pipelines, not the actual execution ie. Test and Build pipelines will appear as siblings rather than as parent-child in TUI/Cloud output. Therefore, this is a visualization-only workaround without any actual impact on execution (in fact, the additional calls to id() make execution less efficient than simply passing the object instance as @helderco mentioned.
We also discussed that adding this workaround to the cookbook may increase end-user confusion as users may think this is a best practice whereas it's more of a workaround applicable for a narrow use case.
For the above reasons, we agreed to close this PR without merging it. I'll do that next.
FYI @marcosnils


[$500]  Mention - First letter of username becomes lowercase when entering full name after @ mention

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.10-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to any chat
Type @
Enter the display name of a user in which the first letter is in capital letter

Expected Result:
There will be no change in the username in the suggestion list when the display name is fully entered
Actual Result:
When the display name is fully entered, the first letter of the username in the suggestion list becomes lowercase
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011f33f898020e3b5a
Upwork Job ID:<PHONE_NUMBER>608507392
Last Price Increase: 2023-12-08

I am sorry,  adding the correct video
https://github.com/Expensify/App/assets/78819774/850dfdba-730d-40e1-aa93-23b4e66678cf

@lanitochka17 I still don't think that new video shows the issue correctly. I recorded my own video showing the bug and updated the description with my new video.

@situchan what do you think of @c3024's proposal?

reviewing proposal

@situchan were you able to review?

Sorry forgot to post approval stamp

@c3024's proposal looks good to me.
🎀 👀 🎀 C+ reviewed

I completed payment for @c3024.
@situchan could you please complete the BZ checklist so I can complete your payment as well? Thanks!

[x] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/14686
[x] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/14686/files#r1445045341
[x] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] Determine if we should create a regression test for this bug. N/A as caught by QA team
[x] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Thanks! Completed payment 👍


floggy esm and jest fail

Description
floggy recently updated to be an esm. I use konn inside a nextjs jest seup. it throwsb:
    Jest encountered an unexpected token

    Jest failed to parse a file. This happens e.g. when your code or its dependencies use non-standard JavaScript syntax, or when Jest is not configured to support such syntax.

    Out of the box Jest supports Babel, which will be used to transform your files into valid JS based on your Babel configuration.

    By default "node_modules" folder is ignored by transformers.

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/ecmascript-modules for how to enable it.
     • If you are trying to use TypeScript, see https://jestjs.io/docs/getting-started#using-typescript
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/configuration
    For information about custom transformations, see:
    https://jestjs.io/docs/code-transformation

    Details:

    /home/jonass/Documents/lala/node_modules/floggy/src/chalk.ts:7
    import chalkPackageSingleton from 'chalk'
    ^^^^^^

    SyntaxError: Cannot use import statement outside a module

I tried workarounds using transformIgnorePatterns, but that didn't get me too far:
TypeError: Cannot read properties of undefined (reading 'Instance')

      at Object.Instance (node_modules/floggy/src/chalk.ts:9:48)
      at Object.<anonymous> (node_modules/floggy/src/chalk.cts:7:1)
      at Object.<anonymous> (node_modules/floggy/src/filter.cts:1:1)
      at Object.<anonymous> (node_modules/floggy/src/logger.cts:2:1)
      at Object.<anonymous> (node_modules/floggy/src/root-logger.cts:1:1)
      at Object.<anonymous> (node_modules/floggy/src/index.cts:1:1)
      at Object.<anonymous> (node_modules/konn/src/log.ts:1:1)
      at Object.<anonymous> (node_modules/konn/src/runners.ts:4:1)
      at Object.<anonymous> (node_modules/konn/src/provider.ts:3:1)
      at Object.<anonymous> (node_modules/konn/src/Providers/ChildProcess/ChildProcess.ts:3:1)
      at Object.<anonymous> (node_modules/konn/src/indexProviders.ts:2:1)
      at Object.<anonymous> (node_modules/konn/src/index.ts:1:1)

Using a resolutions field I pinned the version down to 0.3.1 successfully, but perhaps it would be nicer if konn could pin its dependencies.

Hm, floggy is a hybrid package. Thanks  for raising this!


SCB-929 Concurrent error in update resource APIs

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run go build go test go fmt go vet to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #446 into master will increase coverage by 0.01%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##           master     #446      +/-   ##
==========================================
+ Coverage   59.74%   59.76%   +0.01%
==========================================
  Files         149      149
  Lines       13167    13176       +9
==========================================
+ Hits         7867     7875       +8
- Misses       4729     4730       +1
  Partials      571      571

Impacted Files
Coverage Δ

server/service/microservice.go
71.4% <100%> (-0.06%)
:arrow_down:

server/service/instance.go
68.33% <66.66%> (+0.12%)
:arrow_up:

server/service/rule.go
70.28% <84.61%> (+0.11%)
:arrow_up:

server/service/tag.go
69.04% <87.5%> (+0.72%)
:arrow_up:

server/service/notification/websocket.go
83.87% <0%> (-0.65%)
:arrow_down:

server/broker/util.go
53.47% <0%> (+0.26%)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage increased (+0.03%) to 62.289% when pulling b1cd3f60f65334d72b5c973fa3306081476e15fa on little-cui:master into 3404f76e89342c8291d4e18eea63a183e0afe567 on apache:master.


How to pull object from Array knowing one field of it

"thanks": [
        {
          "postId": "eLodot7Q55RkLRxfb",
          "createdAt": "2015-11-09T07:17:16.474Z"
        },
        {
          "postId": "eQ5ot7kLQ5RLod55Rxfb",
          "createdAt": "2015-11-09T07:17:33.789Z"
        }
      ]

I have the postId
  user.pull('profile.extra.thanks', {
          postId: "eLodot7Q55RkLRxfb"
  });

I want to pull the whole object from the array and the code up is not right.
Thanks in advance :)

You can do it in the following way:
var thank = _.find(user.profile.extra.thanks, function(thank) {
  return thank.postId = 'eLodot7Q55RkLRxfb';
});
user.pull('profile.extra.thanks', thank);

Thank you so much and I really appreciate your prompt response :)

No problem :)


Table data rendering issue

React-table version : 6.5.3
I am trying to render table using sample data below
{     "tableData": [{         "a.status": "closed"     }, {         "a.status": "opened"     }, {         "a.status": "reopened"     }],     "columns": [{         "Header": "Activity Status",         "accessor": "a.status"     }] }
columns are rendering perfectly but data is not rendering. After long struggle when I change my data like this :
{     "tableData": [{         "a_status": "closed"     }, {         "a_status": "opened"     }, {         "a_status": "reopened"     }],     "columns": [{         "Header": "Activity Status",         "accessor": "a_status"     }] }
What is the solution, if I want to render with first set of data.

I think you will find that . is interpreted to mean object member access.
So a.status requires a data structure like
  {
    a: {
      status: 'closed'
    }
  }

@baerrach thanks for your reply...
{ "tableData": [{ "a":{"status": "closed" }}, { "a":{"status": "opened" } }, { "a":{"status": "reopened"} }], "columns": [{ "Header": "Activity Status", "accessor": "a.status" }] }
SO would work ?

Did you try it?

You have not shown us how your columns are configured. react-table supports dot syntax in the accessor (such as accessor: 'a.status') or you can use a function-based accessor (such as accessor: (row)=>row.a.status. Examples of these are all in the docs.

This is a question of implementation. We invite you to join the slack channel and ask for help there or on stack overflow. Thanks!


Fix rendering of special characters in station listing table

Closes #81.
The station listing notebook reads the CSV file using encoding latin1.  Not sure why that's the case.  file reports UTF-8:
$ file --mime-encoding solarstations.csv
solarstations.csv: utf-8

Removing the encoding argument (and thus using the default of utf-8) fixed the rendering of those characters in a local build.

The RTD build looks good: https://solarstations--84.org.readthedocs.build/en/84/station_listing.html

Wonderful. Thanks @kandersolar


[Model refactor] Remove Visualization::Member usages from json/visualizations_controller

Remove the remaining 6 Member usages from Api::Json::VisualizationsController. One of them, set_next_id, can be removed, because we currently don't support Odyssey. The rest of them (and the constants) can be changed to Carto::Visualization.
There are tests for like methods. That can be a nice first step. You should do tests for the other methods.
If any of the changes become unexpectedly difficult, split this issue in many.

Addendum: It would be nice to move this to Carto::VisualizationsController. We usually match Carto:: controllers with Carto:: models, primarily as convention, so we know what models should be used in each controller.


American English or British English?

In the README (and maybe in the codebase), there are occurrences of "optimisation" (e.g. in diagram and text) and "optimization" (in text). Also, "visualisation".
We should pick one for the whole repo and stick to it. I usually use American English in papers (optimization, visualization, generalization) so I would vote for American English. What do you prefer?

I vote for American English (also because I'd be lazy to change some functions as optimize_hyperparams in AgentStats.

ok


V-HTML

Plese use v-html for view content description or title  it's people needed it;
<3<3

At the moment I am using slot to change this, but if you add a change of this in the options to step I will be grateful to you)

:tada: This issue has been resolved in version 2.7.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release


implement logging

This patch addresses issue https://github.com/behdad/fonttools/issues/284
Basically, it replaces all the relevant uses of the print or warnings.warn functions with equivalent calls to the Python's built-in logging package.
For more info on Python's logging, you can read the official documentation here:
https://docs.python.org/3.5/library/logging.html
https://docs.python.org/3.5/library/logging.config.html
https://docs.python.org/3.5/howto/logging-cookbook.html
I think I'll wait a bit before merging this, as it affects almost all the modules in the library and would be nice to hear what others think.
The Python logging system can seem quite complex at the beginning (at least it did for me). I've tried to keep this simple and the most conservative possible.
The biggest change from the point of view of clients of the library is that one has to opt-in, rather than opt-out.
That is, no messages, not even of severity WARNING or ERROR, will be printed to stdout or stderr unless the user configures the logging system: which means, to the very least, adding a handler like StreamHandler to the "root" logger (ie. the parent of all loggers), which is what logging.basicConfig() does.
To make things easier I also added a configLogger function -- accessible directly from fontTools import configLogger -- which scripts can call to set up the top-level library logger with predefined options (they can be overridden via keyword args).
The individual commits contain more info on each change.
Please let me know what you think.
Thanks.

So, I would like to merge this if no one has objections or comments...
We can always fix things up later. Can I proceed?

I'm sorry for the insistence, but if this huge patch doesn't get merged soon-ish, it risks to generate conflicts with changes elsewhere.. That means I'll need to fix and rebase it forever -- or even worse I might tinker again with it :-P
So, did anybody have a look by any chance?
I'm open to discuss changes or provide more details on the implementation.
ping @behdad @brawer @moyogo

LGTM

I changed my mind...
I re-read the logging documentation once again in search for guidance as to the best practices when implementing logging for libraries such as fontTools.
Initially I had settled for the old pre-Python 3.2 behavior. That is, when a logging event occurred and no handlers were found, then either the events would be silently dropped (if logging.raiseExceptions is
set to False), or it would throw a single "No handlers could be found" warning
(this was the default). The pre-3.2 docs recommend in this case to attach a NullHandler to the top-level library logger, if the library developers don't want the above warning message to be presented to the user who doesn't configure logging.
After some thought, I think now it's better to follow the current Python 3 behavior for when a client of the library doesn't provide any explicit logging configuration.
The new default behavior in this circumstance is that events are output using a "handler of last resort". The latter works like logging.StreamHandler which writes messages without formatting to the current value of sys.stderr (thus respecting any redirections which may be in effect).
I believe this is better than forcing everyone to update their own scripts and deliberately configure logging.
I applied the change in the last two commits. Now everyone should be happy and I can finally merge.

I've got to say, the logging infrastructure blows my mind away whenever I try to understand it...  Thanks for taking care of this.  Will eventually study and learn.
In the mean time, let's see what @garretrieger has to say about the changes to subset module, as he uses a custom logger with the subsetter.

Yeah, I know what you mean.
Anyway, I'm happy to assist him or anyone who might have troubles with the logging configuration.
The new system actually enables greater flexibility on the user's side than it was allowed before. You won't need any custom logger anymore, you just need to understand how logging is organised in the library you are working with, and then configure it using all the facilities that the logging and logging.config modules provide.
You can then make it do anything you like.


Getting Error: No available Cask for lastpass-universal

Hey there,
I was able to successfully install your dotfiles on one macbook pro but not my new one. Here's the error I got:
 ⇒ brew cask install lastpass-universal ...
Error: No available Cask for lastpass-universal
Error: nothing to install
[error] failed to install lastpass-universal! aborting...

I'm on a brand new mid 2012 Macbook Pro

It may be that that cask doesn't exist anymore. I'll take a look.
I've been debating whether it's worth pulling all of the brew, brew cask and npm global installs into a yaml config file for the repo--but I haven't gotten around to it.
In the meantime, you can just comment out or remove https://github.com/atomantic/dotfiles/blob/master/osx.sh#L161 in a branch.

Fixed in latest commit: https://github.com/atomantic/dotfiles/commit/8f16ac8523048a9db6dedbb9d155c6fa1c178738
lastpass changed their cask from lastpass-universal to just lastpass :)

Adam,
Thank you so much for the quick response and fix. I love the automation and have been pushing your dot files to everyone I know.
Keep up the good work.
Kind regards,
Wes

On Feb 19, 2015, at 9:39 AM, Adam Eivy<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Fixed in latest commit: 8f16ac8 https://github.com/atomantic/dotfiles/commit/8f16ac8523048a9db6dedbb9d155c6fa1c178738
lastpass changed their cask from lastpass-universal to just lastpass :)
—
Reply to this email directly or view it on GitHub https://github.com/atomantic/dotfiles/issues/1#issuecomment-75098510.


patch pyjwt 2.7, 2.8 and 2.9 python support

See https://github.com/conda-forge/pyjwt-feedstock/issues/33
Checklist

[x] Used a static YAML file for the patch if possible (instructions).
[x] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[x] Ran pre-commit run -a and ensured all files pass the linting checks.
[x] Ran python show_diff.py and posted the output as part of the PR.
[x] Modifications won't affect packages built in the future.

$ python show_diff.py --subdirs noarch
================================================================================
================================================================================
noarch
noarch::pyjwt-2.8.0-pyhd8ed1ab_1.conda
-    "python >=3.7"
+    "python >=3.7",
+    "typing_extensions"
noarch::pyjwt-2.9.0-pyhd8ed1ab_0.conda
-    "cryptography >=3.3.1"
+    "cryptography >=3.4.0"
-    "python >=3.7"
+    "python >=3.8"
noarch::pyjwt-2.8.0-pyhd8ed1ab_0.conda
noarch::pyjwt-2.7.0-pyhd8ed1ab_0.conda
-    "python >=3.6"
+    "python >=3.6",
+    "typing_extensions"
noarch::apache-airflow-providers-celery-3.8.1-pyhd8ed1ab_0.conda
-    "python =3.8,<4.dev0"
+    "python ==3.8,<4.dev0.*"

@ocefpaf do you mind reviewing?


Generic Tuple Extensions

@DougGregor your opinion is welcome on this.  To me, this (sadly) seems clearly out of scope for Swift 3.

Sadly, yes, this is out of scope for Swift 3. The proposal looks roughly reasonably, but implementing it is a fairly significant undertaking because the assumption that only nominal types can conform to protocols is pervasive. So even if this proposal were perfect, it couldn't happen in Swift 3. Please bring this up again once we start scoping the next major Swift release.


[Theming] Hard-code label line-height for focus state compatibility

Undoes the labelStyle portion of the changes in #704 for better compat.

Merging in to resolve Asana ticket.


Enable discussions on github for sharing of beets config files, etc

I'd like a way to share and talk about different configurations for beets with this plugin. I've made tweaks to path templates and filenaming in my set up that others may find interesting.

I've just enabled it, see https://github.com/Neurrone/beets-audible/discussions

Thanks


Delete copy ctor and copy-assignment for log::Batch

🦟 Bug fix
Summary
Fixes rule-of-five violation and reduces flakiness of playback tests.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

do you mind to sign the commit ?

Looks like the error is on the previous commit https://github.com/gazebosim/gz-transport/pull/501/commits/f043ddb910105fcc661950fec5a5afebe85a1225 already on main?


feat: Add dice notation to drop Lowest or Highest rolls

Drop the highest roll by suffixing -H, example: [2d20-H] - useful to roll with disadvantage.
Drop the lowest roll by suffixing -L, example: [2d20-L] or [4d6-L] - useful to roll with advantage, or when rolling character stats.

Ah, good point.  I'd taken the notation from https://en.wikipedia.org/wiki/Dice_notation#Selective_results where the -H / -L options are described, but completely understand wanting to keep similarity with the closer competition.  There's probably no harm in implementing both options, so long as they're both well documented.
I'd stray away from using colour or greying-out the dice rolls for accessibility reasons.  I'd considered implementing a strikethrough for the dropped dice, but that was a bit more complex than the basic implementation so far, so wanted to be sure there was appetite for implementing this kind of feature before doing that.
Happy to go ahead and update the syntax, and look at also clearly showing the dropped dice if that sounds like a good approach.

I've seen other apps use ^ and v to mean take the highest and lowest respectively.
For example:
2d20^1 for DnD advantage.
As a more general example:
8d4v4 to take the 4 lowest rolls.

It should be pretty straightforward to implement the ^ / v syntax as well - would be good to provide several familiar options, to support as many folk transitioning over as possible.

Avrae (DnDBeyond Discord bot) uses kh/kl for keep highest/lowest. It seems like everybody does it differently =/

The representation here is actually quite nice I think: https://wiki.roll20.net/Dice_Reference#Drop.2FKeep (greyish color)

The problem I see with that is that the colour contrast is too low for people with visual impairments.  The equivalent opaque foreground colours for the dropped rolls in that example are #999999 (for non-crits), #C79B9B (for crit-fail) and #A7C79B, all of which fall well below WCAG 2.1 AA colour contrast ratio of 4.5:1 for normal text (text below 16pt), with contrasts of 2.84:1, 2.44:1 and 1.86:1 respectively, when used on a white background.
I think it must be possible to find a better approach that is more inclusive - D&D after all is played by a very wide range of people, including those with various disabilities, so tools like this should surely be trying to support as wide a range of those as possible? (Full disclosure, I'm one of those sight impaired players!)

@PurpleGranite I totally agree with you on that! If you have any recommendations feel free to share/implement it.

I made progress implementing the different roll notations last night - I'll get that out of the way first and posted for review, then move on to looking at the display options.  I'm thinking something like - 17 (d20) + 10 (d20) (dropped: 5 (d20), 4 (d20)) = 27 (for [4d20k2]) - I just need to work out how to get the dropped rolls over to the chat system for display.

That's the rolling rules discussed above now in place - quite a challenge because of limitations in the upstream library, but all should be working now.

Hit me up on discord or here if you have any relay/graphql related questions!

Hey, @PurpleGranite Thanks for starting this! I picked your code and refactored it a bit in https://github.com/dungeon-revealer/dungeon-revealer/pull/1226

Thanks @n1ru4l - apologies for effectively dropping it, work got pretty hectic - glad it turned out to be useful as a starting point :)


Add dynatrace monitoring module

I added same metrics as in other monitoring modules. Example app for this: https://github.com/commercetools/commercetools-sdk-java-v2/pull/623

We decided not to add dynatrace module for now.
Reason: it would add too much overhead as it uses HTTP to create new metrics. This would significantly slow down every request to commercetools. The other way for now for dynatrace is to use opentelemetry.


Intellisense and symbol search stops working in large Javascript project

From @markus-brt on September 27, 2017 8:47

VSCode Version: 1.16.1
OS Version: Win10

We have a large JS project with approx. 8000 files. Since one of the latest releases the Intellisense (for all kinds of symbols, also DOM) stops working and also Symbol search (Ctrl-Shift-O). It will work for a few minutes, then it will only show "Loading..." or the loading animation of the search bar.
I noticed that the vscode process will allocate memory up to 1,2 GB and then go down to 500 MB again, (see the screenshot) and consumes 25-30% CPU. This continues for 1/2 hour. When "finished" the features works again, but only a few times until it starts over with "Loading..." and again the same CPU and memory usage.
It is also strange that my colleagues who use the same project are not affected by this issue.

Steps to Reproduce: ?
Reproduces without extensions: Yes
Copied from original issue: Microsoft/vscode#35212

From @markus-brt on September 29, 2017 10:44
This is the Typescript output/logs:
[Info  - 10:34:31] Using tsserver from: c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib [Info  - 10:34:31] Forking TSServer PATH:... [Error  - 10:44:39] TSServer exited with code: 3 [Info  - 10:44:39] Using tsserver from: c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib [Info  - 10:44:39] Forking TSServer PATH:... [Error  - 10:44:39] 'getApplicableRefactors' request failed with error. Error: Service died. 	at TypeScriptServiceClient.serviceExited (c:\Program Files\Microsoft VS Code\resources\app\extensions\typescript\out\typescriptServiceClient.js:405:32) 	at ChildProcess.childProcess.on (c:\Program Files\Microsoft VS Code\resources\app\extensions\typescript\out\typescriptServiceClient.js:316:30) 	at emitTwo (events.js:111:20) 	at ChildProcess.emit (events.js:194:7) 	at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12) [Error  - 11:38:25] TSServer exited with code: 3 [Info  - 11:38:25] Using tsserver from: c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib [Info  - 11:38:25] Forking TSServer PATH:... [Error  - 11:38:25] 'getApplicableRefactors' request failed with error. Error: Service died. 	at TypeScriptServiceClient.serviceExited (c:\Program Files\Microsoft VS Code\resources\app\extensions\typescript\out\typescriptServiceClient.js:405:32) 	at ChildProcess.childProcess.on (c:\Program Files\Microsoft VS Code\resources\app\extensions\typescript\out\typescriptServiceClient.js:316:30) 	at emitTwo (events.js:111:20) 	at ChildProcess.emit (events.js:194:7) 	at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)

@markus-brt Thank you for the logs. Does your project have a jsconfig.json?
Can you also try looking at the TypeScript server logs to see why it is crashing:

In VSCode, set "typescript.tsserver.log": "verbose"

Restart vscode and reproduce the issue

Open the VS Code output panel and go to the TypeScript section.

At the top of this output, there should be a message that looks like:
[Info  - 10:54:24] Using tsserver from: /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib
[Info  - 10:54:24] TSServer log file: /var/folders/_0/611900jn0yb43f1rc47_2wmh0000gp/T/vscode-tsserver-log-N4iPEB/tsserver.log

Open the TSServer log file at that path

Look for any errors or stack traces in this log file. The file also should have lists of all files that are being included in the project. Glance through these lists and see if any unexpected files are being included

Yes, we have jsconfig.json in two folders which exclude 3 sub-folders, which seems to work.
The log file contains all included files, but I did not find any traces of any errors (not even the one that is shown in the output (TypeScriptServiceClient)

@markus-brt  can you share more information? can you share your project? we would be happy to sign any required NDAs. Do you see the same results on a clean copy of your project (i.e. no tmp files, extra npm packages)? can you give it a try on typescript@next ? (VSCode ships with an older version of TypeScript. Please see Using Newer TypeScript Versions documentation for more details on updating your VSCode to use a different version of TypeScript.)

Solved - kind of:
The cause of the problem was the jsconfig.json files in two of the subfolders. Which is also the reason why colleages with the same project were not affected. Since I deleted them everything works as before.
I have no idea how these two files were created, including an "exclude" entry in one of them. I did not do this manually... are there any circumstances in which vscode will create these files?

Face the same problem.
I only have on jsconfig.json file in top-level.
[Error  - 22:42:58] 'getApplicableRefactors' request failed with error.
Error processing request. Cannot read property 'getMissingFilePaths' of undefined
TypeError: Cannot read property 'getMissingFilePaths' of undefined
    at ConfiguredProject.Project.updateGraphWorker (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79310:56)
    at ConfiguredProject.Project.updateGraph (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79254:39)
    at ConfiguredProject.Project.getLanguageService (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:78994:26)
    at IOSession.Session.getApplicableRefactors (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82616:32)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:81586:61)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82799:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82790:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82799:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82819:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:83945:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)
[Error  - 22:42:58] 'occurrences' request failed with error.
Error processing request. Cannot read property 'getMissingFilePaths' of undefined
TypeError: Cannot read property 'getMissingFilePaths' of undefined
    at ConfiguredProject.Project.updateGraphWorker (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79310:56)
    at ConfiguredProject.Project.updateGraph (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79254:39)
    at ConfiguredProject.Project.getLanguageService (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:78994:26)
    at IOSession.Session.getOccurrences (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:81937:43)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:81557:61)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82799:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82790:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82799:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82819:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:83945:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)
[Error  - 22:42:59] 'getApplicableRefactors' request failed with error.
Error processing request. Cannot read property 'getMissingFilePaths' of undefined
TypeError: Cannot read property 'getMissingFilePaths' of undefined
    at ConfiguredProject.Project.updateGraphWorker (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79310:56)
    at ConfiguredProject.Project.updateGraph (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79254:39)
    at ConfiguredProject.Project.getLanguageService (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:78994:26)
    at IOSession.Session.getApplicableRefactors (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82616:32)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:81586:61)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82799:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82790:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82799:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:82819:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:83945:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

@fenglu09  please file a new issue, and give us some context to be able to reproduce the issue. for instance, the version of the compiler you are using, the source project that triggered the error.. etc..


How To Make A Prediction

I can create model successfully according to sample code. Now I should make a prediction with my unit data. How can I do this?

Ok, I found it. model.Predict(object o ) is used for this

You can also do model.PredictValue(object o) if you just want the label prediction (and not have the system fill in the value). Sometimes the type cannot be written to so this call makes more sense.


[WIP]Add Karma and webpack debugging

If you don't have gulp-cli you have to run
npm install -g gulp-cli

gulp 4
Reworked all gulp tasks
Added build:watch to quickly rebuild
Separated to tasks folder to reduce gulpfile clutter
Fixed a Karma issue.
Added debug argument

Let's expose this as a command line argument? --debug ?

Can use yargs.argv['debug']

so --karma-debug and --webpack-debug?

Works for me if you need that granularity, otherwise we could just lump them under a single flag.

@bluepnume Ready for an initial review.  I need you to check that the files are building properly, specifically the values in the DefinePlugin.

Gonna close this for now as many of these changes were merged. If you wanna add a separate PR for gulp 4 that's cool.


compliance-tool: Fix compliance tool imports and unitests

Currently there are quite some wrong imports in the compliance-tool directory. Even when those are fixed many Unitests still fail.
Here these imports are fixed.

The reason why the python 3.8 compliance-tool-test does not work seems to be because it seems to be generating older aas examples with spelling mistakes. The 3.10 and 3.12 Versions generate the correct example data. I am just not sure why this is the case.

I just remembered that we do not support Python 3.8 anymore so that test not working does not matter. I will adapt the ci accordingly.


Track Metric doesn't accept properties

I think we should be able to add meta data to the metrics we track similar to the C# library.

Thanks, @kthayer424. I think this was a point-in-time back-end limitation. We'll try to get this into the next release.

:+1:

TrackMetric takes properties now - https://github.com/Microsoft/ApplicationInsights-node.js/blob/master/Library/Client.ts#L111


Not able to run main_task.py on SemEval2010 Task 8 dataset

Hi,
I was trying to fine tune the model on SemEval2010 Task 8 dataset on google colab. I tried the following code
!rm -rf ./data
!mkdir ./data
!unzip <path1>/SemEval2010_task8_all_data.zip -d ./data/

!python <path2>/BERT-Relation-Extraction/main_task.py  \
--train_data ./data/SemEval2010_task8_all_data/SemEval2010_task8_training/TRAIN_FILE.TXT \
--test_data ./data/SemEval2010_task8_all_data/SemEval2010_task8_testing/TEST_FILE.txt \
--num_classes 9  \
--batch_size 100 \
--num_epochs 3 \
--lr 0.0001 \
--model_no 0 \
--model_size bert-base-uncased \
--train 1

But this throws an error, saying:
Traceback (most recent call last):
  File "<whatever>/BERT-Relation-Extraction/main_task.py", line 50, in <module>
    net = train_and_fit(args)
  File "<whatever>/BERT-Relation-Extraction/src/tasks/trainer.py", line 33, in train_and_fit
    train_loader, test_loader, train_len, test_len = load_dataloaders(args)
  File "<whatever>/BERT-Relation-Extraction/src/tasks/preprocessing_funcs.py", line 336, in load_dataloaders
    df_train, df_test, rm = preprocess_semeval2010_8(args)
  File "<whatever>/BERT-Relation-Extraction/src/tasks/preprocessing_funcs.py", line 67, in preprocess_semeval2010_8
    sents, relations, comments, blanks = process_text(text, 'test')
  File "<whatever>/BERT-Relation-Extraction/src/tasks/preprocessing_funcs.py", line 39, in process_text
    assert re.match("^Comment", comment)
AssertionError

What am I missing here?

I presume you have changed something in the semeval dataset? The assertion error will be raised if the dataset format doesnt exactly match the one given in the original link

Actually I have not changed anything. It's the vanilla dataset downloaded from here: https://github.com/sahitya0000/Relation-Classification/blob/master/corpus/SemEval2010_task8_all_data.zip
Are the arguments correct that I'm providing for the train and test data?

I see the error now. You are using the wrong test data folder. Please see the default arguments provided in main_task.py.

Thank you, that resolved it. Sorry, I should've seen that.


Fix creat recognize request of call automation media

Contributing to the Azure SDK
The createRecognizeRequests needs to map the cognitionconfiguration field called InitialSilenceTimeout to InitialSilenceTimeoutInSeconds. Currently the recognizeRequestInternal for recognize choice miss this field. This fix adds the population of the field from cognitionconfiguration

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

Could you please provide a description of the issue, what is the solution, and also test cases that cover it?

Yes, basically it adds the missing  InitialSilenceTimeoutInSeconds field in  the RecongnizeRequest. The error is found at E2E validation, and should be working at the sample E2E application.

Could you please provide a description of the issue, what is the solution, and also test cases that cover it?

@cochi2 , The issue is that InitialSilenceTimeoutInSeconds is not passed to PMA from SDK, this fix should resolve it.
@wangrui-msft Just want to confirm did we test this ?

Basically it is consistent with dtmf tone setup. As dtmf serialize works, the choice recognize also work as expected as they inherit the same class.


Exhibition: create separate display field for subtitle

Type

✨ Feature

Problem to be solved / User Story
Some Exhibitions require long title to describe the exhibition.
e.g. Ayurvedic Man: Encounters with medicine man
Long exhibition title doesn't scale well for tags, exhibition hero info box and promo.
UI Requirements
Allow the exhibition team to add optional subtitle on Prismic and support following display logic.

Tags: display title and suppress subtitle (e.g. Part of: Ayurvedic Man)
Exhibition hero info: apply smaller font size for subtitle to provide appropriate visual hierarchy and fit in the exhibition info box
Promo: display title and suppress subtitle

We are going to close this for now and follow up with the content team to provide guidelines for exhibitions and events titles.

Reopening this due to several issues with
Smoke and Mirrors: The psychology of magic
Jo Spence and Oreet Ashery: Misbehaving Bodies

What we want to do is have the notion of a Full Title and Short Title.
Short title should appear wherever we have 'Part of', so as to avoid this:

Short title should appear wherever we have 'Part of', so as to avoid this

What would we expect on a card?

Where it's a 'part of', it's the first part of the title: Part of Ayurvedic Man
The full title will need to be used on the exhibition page itself, and would appear as the full title on the card when it's an Exhibition card - e.g. the card would say Ayurvedic Man: Encounters with Indian Medicine.

Thanks! Just wondering what we call it to make it clear to editors that that's what it'll be used for.
Short title kind of makes sense, but feels a little clunky.

Let’s ask Danny & co. tomorrow to see how they’d think of it.
From: James Gorrie<EMAIL_ADDRESS>Sent: 20 February 2019 17:45
To: wellcometrust/wellcomecollection.org<EMAIL_ADDRESS>Cc: Jennifer Phillips-Bacher<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [wellcometrust/wellcomecollection.org] Exhibition: create separate display field for subtitle (#1716)
Thanks! Just wondering what we call it to make it clear to editors that that's what it'll be used for.
Short title kind of makes sense, but feels a little clunky.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fwellcometrust%2Fwellcomecollection.org%2Fissues%2F1716%23issuecomment-465682809&data=02|01|j.phillips-bacher%40wellcome.ac.uk|a2b30e92b7164d82953408d6975b20a9|3b7a675a1fc84983a100cc52b7647737|0|0|636862814976701859&sdata=L08PMBvPb8UmvMq6Jj9Kc%2FynQm8w3HSiX7Pla2UF3nc%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABYQ-4SgmO8r0b-wFYoaFsqgkRVTGOwRks5vPYmRgaJpZM4P9ck_&data=02|01|j.phillips-bacher%40wellcome.ac.uk|a2b30e92b7164d82953408d6975b20a9|3b7a675a1fc84983a100cc52b7647737|0|0|636862814976701859&sdata=mED6WM64RM31EivbSh%2FWv9mScUctwvvjOGaewzqwTH8%3D&reserved=0.

I wonder if this is something we should apply to the common content model e.g.

Title
Promo text
Promo image
Contributors (still needs putting on page)

Seems to make sense that it would be used on anything that is displayed on "Part of"?
Easy enough to do on exhibitions first though.

Or if the model is broken up into title and subtitle?
Then we just set the model to "{The title: the subtitle}" which follows more standard naming of fields?


[CVE Fix] Update Protobuf Version for CVE-2024-7254

The current protobuf version has a CVE: https://avd.aquasec.com/nvd/2024/cve-2024-7254/
Updating to 4.28.2 fixes it.
I have verified from https://mvnrepository.com/artifact/com.google.protobuf/protobuf-java/4.28.2 that this does not have any compile dependencies.

Fixes #XXXX.

Description

Fixed the bug ...
Renamed the class ...
Added a forbidden-apis entry ...

Release note

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[ ] been self-reviewed.

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

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] 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
[ ] 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.
[ ] been tested in a test Druid cluster.

Closing this PR as the version has been updated by https://github.com/apache/druid/pull/17249


[5.5][AST] Return null type in AbstractClosureExpr::getResultType if getType is null

Cherry-pick of https://github.com/apple/swift/pull/37341 into release/5.5

Explanation: If getType returns a null type, also return a null type in AbstractClosureExpr::getResultType. Previously, we were crashing.
Scope: Invalid code where the type checker fails to set a type on closure expressions
Risk: Very low
Testing: None, the fix was created based on a crash trace without a reproducer
Issue: rdar://77565983
Reviewer: Pavel Yaskevich (@xedin)

@swift-ci Please test macOS

@swift-ci Please test Linux

@swift-ci Please nominate


Qualys and Qualys_Vuln sections

Hi,
I was trying to make VulnWhisperer work only with the qualys_vuln section, and for that reason, I only had set the values concerning that segment. When running the software, I noticed that the values at that section are useless, only the "enabled" flag is being taken under consideration because the qualyslib used for establishing the connection to Qualys is looking at the [qualys] section of the config file. So even though this section contains the "enabled" flag false, the values must be set in order to use the [qualys_vuln] section.
Ex:
[qualys]
enabled=false
hostname = qualysapi.qualys.eu
username = XXXXXXXXX
password = XXXXXXXXX
write_path=/opt/VulnWhisperer/data/qualys/
db_path=/opt/VulnWhisperer/data/database
verbose=true
max_retries = 10
template_id = 126024
[qualys_vuln]
enabled=true
hostname = testurl
username = testuser
password = testpw
write_path=/opt/VulnWhisperer/data/qualys/
db_path=/opt/VulnWhisperer/data/database
verbose=true
Cheers.

Hi @psantos1,
This is something that has already been fixed in #107, but is still in the testing branch as I was waiting for having the qualysapi library updated on pip and be able to get rid completely of the qualysapi fork that we are currently maintaining in the repository.
If you want to have the patched version for that feature and help out testing, you can use the testing branch code :)
Cheers!


Extend spec to give any app icons in usr/share/icons preference

Draft for approaching #19 (as well as #16):

mention icon files below usr/share/icons/hicolor to be taken into account preferred if present
use $APPICON as variable name for the application icon identifier, to remove the otherwise unneeded expectation for being the same string as used for the basename of the desktop file name (the Icon= would be unneeded otherwise)

Sounds reasonable to me. What do others say?

Thanks @kossebau. Please open tickets in any project you may encounter that does not implement this yet, linking to https://github.com/AppImage/AppImageSpec/blob/master/draft.md#the-filesystem-image.


chore: (GRO-381) remove sign in with apple feature flag

https://artsyproduct.atlassian.net/browse/GRO-381
Primary change
This removes the ENV var and supplementary logic (including tests) that put sign in with apple behind a feature flag. Here is the original PR and the specific commit.
Secondary change
When committing my changes I experienced an error during the pre-commit hook that executes yarn lint --fix:
error  Anchor used as a button. Anchors are primarily expected to navigate. Use the button element instead. Learn more: https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/anchor-is-valid.md  jsx-a11y/anchor-is-valid

I reached out to #dev-help and the recommendation was to replace the <Link> component with <Clickable>.  The scope of this change affects only src/v2/Components/Authentication/Footer.tsx elements. This change also affected a number of tests that were updated in this PR. I tried to keep the behavior and look exactly how it was before.
Demonstration of changes

staging login
local login

staging signup
local signup

Lastly there was an extra space after the word Facebook which was removed in both Login and Signup auth footer.

Link to a review app

(@ovasdi - if this is ready to go be sure to assign it to someone)


Failed to import k2 and icefall.         Notice that they are necessary for         hlg_onebest/hlg_rescore decoding and LF-MMI training

I keep getting an error about not being able to import k2 and icefall when running s0 in aishell dataset, but I already have k2 installed. please answer.

Just ignore it if you do not use hlg_onebest/hlg_rescore decoding ro LF-MMI training


Pin dependencies

Depencies should be pinned for stable release versions.

Done: https://github.com/luisgc93/stock_reminder_bot/commit/6a796eb9f17fbd4015108d5658d03e7713e3f073


Clone constraint on recursive()

Hi there!
I'm playing around and trying to put together a simple proof-of-concept parser for the Clarity smart contract language (S-expression based, "mostly") using Logos + Chumsky, but have run into an issue I haven't been able to figure out. After trying many different approaches and more than likely combining one too many things from one too many examples, I figured I'd ask here to see if I can't find some "clarity" :)
Here's the source for a very simple contract:
;; Multiplayer Counter contract
(define-map counters principal uint)

(define-read-only (get-count (who principal))
	(default-to u0 (map-get? counters who))
)

(define-public (count-up)
	(ok (map-set counters tx-sender (+ (get-count tx-sender) u1)))
)

define-map, define-read-only, default-to, map-get?, define-public, ok, map-set, + are built-in functions, and the rest is identifiers, types and/or literals.

I've gotten as far as getting to the (default-to u0 (map-get? counters who)) statement, which is where nested expressions start needing to be accounted for, and this is where I'm running into the issue with the Cloneconstraint. Here's the minimal function where recursive is yelling at me (I'll keep it to this function only for now for brevity, but can paste more later if it's not obvious from this what I'm doing wrong):
fn expr_parser<'a, I>() ->
impl Parser<'a, I, SExpr, extra::Err<Rich<'a, Token>>>
where
    I: ValueInput<'a, Token = Token, Span = SimpleSpan>
{
    recursive(|expr| {
        let default_to = just(Token::OpDefaultTo)
            .ignore_then(literal_parser())
            .then(expr_parser())
            .delimited_by(just(Token::ParenOpen), just(Token::ParenClose))
            .map(|(default, tail)| {
                eprintln!("default: {default:?}, tail: {tail:?}");
                SExpr::Op(Op::DefaultTo(DefaultToDef {
                    default, tail: Box::new(tail)
                }))
            });

        default_to
    })
}

The cryptic error message I'm getting is:
error[E0277]: the trait bound `impl chumsky::Parser<'_, _, SExpr, chumsky::extra::Full<chumsky::error::Rich<'_, lexer::Token>, (), ()>>: Clone` is not satisfied
   --> src/parser.rs:188:5
    |
188 |     recursive(|expr| {
    |     ^^^^^^^^^ the trait `Clone` is not implemented for `impl chumsky::Parser<'_, _, SExpr, chumsky::extra::Full<chumsky::error::Rich<'_, lexer::Token>, (), ()>>`
    |
    = note: required for `Then<IgnoreThen<Just<Token, _, Full<Rich<'_, Token, SimpleSpan, &str>, (), ()>>, ..., ..., ...>, ..., ..., ..., ...>` to implement `Clone`
    = note: the full type name has been written to '/home/cylwit/Code/github.com/cylewitruk/clarity-chumsky/target/debug/deps/clarity_chumsky-6b6a893297ce6796.long-type-9289247432681493460.txt'
    = note: 2 redundant requirements hidden
    = note: required for `Map<DelimitedBy<Then<IgnoreThen<Just<Token, _, ...>, ..., ..., ...>, ..., ..., ..., ...>, ..., ..., ..., ...>, ..., ...>` to implement `Clone`
    = note: the full type name has been written to '/home/cylwit/Code/github.com/cylewitruk/clarity-chumsky/target/debug/deps/clarity_chumsky-6b6a893297ce6796.long-type-5704332481353913678.txt'
note: required by a bound in `chumsky::recursive::recursive`
   --> /home/cylwit/.cargo/registry/src/index.crates.io-6f17d22bba15001f/chumsky-1.0.0-alpha.6/src/recursive.rs:271:30
    |
267 | pub fn recursive<'a, 'b, I, O, E, A, F>(f: F) -> Recursive<Direct<'a, 'b, I, O, E>>
    |        --------- required by a bound in this function
...
271 |     A: Parser<'a, I, O, E> + Clone + MaybeSync + 'b,
    |                              ^^^^^ required by this bound in `recursive`

I've checked and double-checked that all of my types derive/impl Clone, so my guess would be that the issue doesn't really have anything to do with Clone itself and rather that I've combined something in the wrong way that's confusing the compiler...
Does any of the above give anyone a clue on what I might have messed up here? :)
If it helps, I pushed my broken code here. Just beware of the dragons, I've been messing around a lot in there ;)

This issue originates from a misuse of recursive:
fn expr_parser<'a, I>() ->
impl Parser<'a, I, SExpr, extra::Err<Rich<'a, Token>>> /* 2 */
where
    I: ValueInput<'a, Token = Token, Span = SimpleSpan>
{
    recursive(|expr| {
        let default_to = just(Token::OpDefaultTo)
            .ignore_then(literal_parser())
            .then(expr_parser()) /* 1 */
            .delimited_by(just(Token::ParenOpen), just(Token::ParenClose))
            .map(|(default, tail)| {
                eprintln!("default: {default:?}, tail: {tail:?}");
                SExpr::Op(Op::DefaultTo(DefaultToDef {
                    default, tail: Box::new(tail)
                }))
            });

        default_to
    })
}

The unsatisfied Clone bound originates from your recursive call to expr_parser (1). Because the function returns an impl Parser<...> without a + Clone bound (2), it fails to satisfy recursive's Clone requirement.
Additionally, if the code did compile, this would result in a stack overflow due to infinite recursion at runtime as the parser would recurse trying to construct itself.
To use recursive correctly, you must use the argument you are given in the closure, expr (this is the parser itself). Therefore, the solution should be to replace expr_parser() with the argument expr.

Thanks @wackbyte for the quick and thorough response! That solved my issue, I learned something, and it now it produces the error I expected :)
I'll close this issue, thanks again!


Make SDK / Objects compliant to document and cleanup of code

Refer to: https://wiki.lfedge.org/display/EVE/Objects+and+methods+for+SDK

Closing for now


feat: add docker configuration for the simulator

Description:
This PR simplifies the simulator development process by introducing Docker containerization and improves documentation clarity.

Add Docker setup for simulator components
Add new Gradle task startSimulatorDockerContainers
Reorganize configuration documentation with consistent table format
Simplify metrics viewing process in quickstart guide

The changes streamline the development process by replacing manual configuration steps with automated Docker container setup, making it easier for users to run and monitor the simulator in both Publisher and Consumer modes.
Related issue(s):
Fixes #412
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Quick note. Consensus teams were looking to use the simulator for their own testing, and found the readme and quickstart woefully outdated.
Would you mind creating an issue to update those documents after this PR merges to reflect all of the recent changes?

Quick note. Consensus teams were looking to use the simulator for their own testing, and found the readme and quickstart woefully outdated. Would you mind creating an issue to update those documents after this PR merges to reflect all of the recent changes?

Yes, I've noticed that too, mostly about configuration and opened a issue couple of days ago. https://github.com/hashgraph/hedera-block-node/issues/407


Enhancement/681 documentation nesting

Looks like we can't auto merge. @sonejah21 can you take a look?

@sonejah21 ?


SDL.h: No such file or directory (Win7, no pkg-config installed)

Another unlucky Windows user here.
Is there a way to solve this without using pkg-config? I'm still getting the "fatal error: SDL.h: No such file or directory  // #include <SDL.h>".
Do I have to put the SDL.h file in one of the go-sdl2 folders?
Do I need to put in on the path?
I've tried experimenting with putting SDL.h in various go-sdl2 folders and running make.bat, but no luck.
Using x86_64-w64-mingw32's GCC, Go 1.4.1, Win 7.
(I kind of don't even know why I bother... everything just works on Linux...)

SDL.h would have to been in your mingw's include directory, specifically (your include directory)\SDL2\SDL.h
Also, don't forget to put SDL's lib files in your lib folder.

I put the source files in the mingw include\SDL2\SDL.h and the .dll's in the lib folder. Still getting the same error.

The strange thing is that according to the GCC -v printout, it is looking in the include dirs and lib dirs that I have put the SDL2 files in:
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=C:/Program\ Files/mingw-w64/x86_64-4.9.2-posix-seh-rt_v3-rev
1/mingw64/bin/../libexec/gcc/x86_64-w64-mingw32/4.9.2/lto-wrapper.exe
Target: x86_64-w64-mingw32
Configured with: ../../../src/gcc-4.9.2/configure --host=x86_64-w64-mingw32 --bu
ild=x86_64-w64-mingw32 --target=x86_64-w64-mingw32 --prefix=/mingw64 --with-sysr
oot=/c/mingw492/x86_64-492-posix-seh-rt_v3-rev1/mingw64 --with-gxx-include-dir=/
mingw64/x86_64-w64-mingw32/include/c++ --enable-shared --enable-static --disable
-multilib --enable-languages=ada,c,c++,fortran,objc,obj-c++,lto --enable-libstdc
xx-time=yes --enable-threads=posix --enable-libgomp --enable-libatomic --enable-
lto --enable-graphite --enable-checking=release --enable-fully-dynamic-string --
enable-version-specific-runtime-libs --disable-isl-version-check --disable-cloog
-version-check --disable-libstdcxx-pch --disable-libstdcxx-debug --enable-bootst
rap --disable-rpath --disable-win32-registry --disable-nls --disable-werror --di
sable-symvers --with-gnu-as --with-gnu-ld --with-arch=nocona --with-tune=core2 -
-with-libiconv --with-system-zlib --with-gmp=/c/mingw492/prerequisites/x86_64-w6
4-mingw32-static --with-mpfr=/c/mingw492/prerequisites/x86_64-w64-mingw32-static
--with-mpc=/c/mingw492/prerequisites/x86_64-w64-mingw32-static --with-isl=/c/mi
ngw492/prerequisites/x86_64-w64-mingw32-static --with-cloog=/c/mingw492/prerequi
sites/x86_64-w64-mingw32-static --enable-cloog-backend=isl --with-pkgversion='x8
6_64-posix-seh-rev1, Built by MinGW-W64 project' --with-bugurl=http://sourceforg
e.net/projects/mingw-w64 CFLAGS='-O2 -pipe -I/c/mingw492/x86_64-492-posix-seh-rt
v3-rev1/mingw64/opt/include -I/c/mingw492/prerequisites/x86_64-zlib-static/incl
ude -I/c/mingw492/prerequisites/x86_64-w64-mingw32-static/include' CXXFLAGS='-O2
-pipe -I/c/mingw492/x86_64-492-posix-seh-rt_v3-rev1/mingw64/opt/include -I/c/mi
ngw492/prerequisites/x86_64-zlib-static/include -I/c/mingw492/prerequisites/x86
64-w64-mingw32-static/include' CPPFLAGS= LDFLAGS='-pipe -L/c/mingw492/x86_64-492
-posix-seh-rt_v3-rev1/mingw64/opt/lib -L/c/mingw492/prerequisites/x86_64-zlib-st
atic/lib -L/c/mingw492/prerequisites/x86_64-w64-mingw32-static/lib '
Thread model: posix
gcc version 4.9.2 (x86_64-posix-seh-rev1, Built by MinGW-W64 project)
Is the make.bat in go-sdl2 defaulting to some other settings?

Okay, seems this is not a problem with go-sdl2 but rather with my SDL2 installation. I can't get hello-world with SDL in C/C++ to compile, so I'm closing this and going down that troubleshooting route.

In case anyone finds this in the future, I put together a tested and working step-by-step guide here: https://gist.github.com/Decker108/7df1726192433a3bdb32


bmj.com

http://www.bmj.com/search/advanced/text_abstract_title%3Aautism%2Bautistic%2B text_abstract_title_flags%3Amatch-any limit_from%3A2017-12-01 limit_to%3A2018-12-31 numresults%3A100 sort%3Apublication-date direction%3Adescending format_result%3Astandard

https://github.com/ceberous/AutismMastadonBot/commit/fc7fb294507e0cfebf43ff42461907e19a5dedcf

https://github.com/ceberous/AutismMastadonBot/commit/fc7fb294507e0cfebf43ff42461907e19a5dedcf


DOCS-124 - Menus for Cloud SIEM and Cloud SOAR

Purpose of this pull request
This pull request adds the following sections:

https://help.sumologic.com//docs/cse/introduction-to-cloud-siem/#cloud-siem-menus
https://help.sumologic.com//docs/cloud-soar/overview/#cloud-soar-menus

Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
DOCS-124

Thank you, @JV0812!


[WIP] Fix versioning-related Utapi metrics [SC3-368]

To-do:

objectPut, objectCopy, completeMpu: track correct old content length when overwriting null version that is not the latest version;
deleteObject: when creating a delete marker when versioning is suspended:

decrement storage utilized by the null version, if any.

number of objects should remain unchanged (since we both delete a version, and put a delete marker).

end to end tried here: http://ci.ironmann.io/gh/scality/Integration/17865

PR has been updated. Reviewers, please be cautious.

@ironman-machine
SCALITY_INTEGRATION_BRANCH=fix/utapi-versioning-metrics
try

Hello @electrachong
"SCALITY_INTEGRATION_BRANCH=fix/utapi-versioning-metrics": Success
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/17910' with the following env. args:
{
    "DEFAULT_BRANCH": "rel/7.0",
    "REPO_NAME": "S3",
    "SCALITY_INTEGRATION_BRANCH": "fix/utapi-versioning-metrics",
    "SCALITY_S3_BRANCH": "FIX/utapi-versioning-metrics"
}

:sunny: :+1: circleCI test succeeded!

@ironman-machine try

Hello @electrachong
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/17920' with the following env. args:
{
    "DEFAULT_BRANCH": "rel/7.0",
    "REPO_NAME": "S3",
    "SCALITY_INTEGRATION_BRANCH": "fix/utapi-versioning-metrics",
    "SCALITY_S3_BRANCH": "FIX/utapi-versioning-metrics"
}

:broken_heart: :umbrella: circleCI test failed.

@ironman-machine try

Hello @electrachong
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/17933' with the following env. args:
{
    "DEFAULT_BRANCH": "rel/7.0",
    "REPO_NAME": "S3",
    "SCALITY_INTEGRATION_BRANCH": "fix/utapi-versioning-metrics",
    "SCALITY_S3_BRANCH": "FIX/utapi-versioning-metrics"
}


Can't use ZuulException in the filters

You cant use ZuulException  in the filter's run method because run not throws ZuulException  and ZuulException  is not RuntimeException

+1 on that issue. We can workaround that but with unnecessary logic of putting the exception in Request Context. Also the try - catch block in ZuulServlet serves no purpose if exceptions are happening in the filters and the error filters cannot be executed at all.
Can the run method in IZuulFilter interface throw a ZuulException. In that case  any other Exceptions in the filters can be wrapped in the ZuulException.
If the above approach is fine, I can submit a pull request

+1.  I'm no groovy expert, but appears that groovy doesn't have the same checks and allows the exception to be thrown.  See e.g. https://github.com/Netflix/zuul/blob/1.x/zuul-netflix-webapp/src/main/groovy/filters/pre/Routing.groovy

Any update on this?? I'm also having the same issue

Why pull request was closed? :( I also need this. Currently throwing zuul exception is a headache in java code

+1
There is a workaround (https://jmnarloch.wordpress.com/2015/09/16/spring-cloud-zuul-error-handling/) but that feels quite dirty and is creating more overhead than needed. It would be nice to throw a ZuulException with statuscode + errormessage from a filter.

Why pull request was closed? :( I also need this. Currently throwing zuul exception is a headache in java code

I was doing some cleanup on my fork and deleted the branch before it was merged, which auto closed the pull request. I can see about getting a new PR out, though keep in mind the PR was out there for 5+ months and not merged.


Hugging Face Hub intergration

It's really great to see so much work going into open-source HTR!  It could be cool to have closer integration with the Hugging Face Hub. I know you are already sharing models there, but having a direct method for downloading and pushing models to the Hub would be super nice and could help increase the usage of open models.
Happy to offer some guidance on the best approach for doing this if it is of interest 🤗

Hey @davanstrien
Thanks for reaching out to us. We are definitely interested in this feature and have been trying to find some time to do the integration already. While this won't be in the next release, this is on our roadmap for the future ones.
As the pinned issue says, development is no longer done here but on GitLab instead, as pointed on Hugging Face and in the documentation. So please prefer using the issue tracker over there.
I'm closing this as a dupe of https://gitlab.teklia.com/atr/pylaia/-/issues/29.


Mapping: wrong behaviour for collections

Describe the bug
When mapping dtos to models the behaviour for collections is to create a new collection instance instead of reusing an existing collection.
This leads to problems with ViewModels that are listening to collections changes but do bot get notified of a new collection.
Expected behavior
If a target collection exists, it should not be overwritten, but updated to match the items of the source collection. Existing items with the same id should also be reused if possible.
It might be useful to make Collections readonly to prevent thi from happening in the future.

This will be fixed for release v0.7.4


PowerToys Run ran into an issue

Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.18363.0
IntPtr Length: 8
x64: True
Date: 08/01/2021 10:22:13
Exception:
System.ArgumentException: Destination array was not long enough. Check the destination index, length, and the array's lower bounds. (Parameter 'destinationArray')
at System.Array.Copy(Array sourceArray, Int32 sourceIndex, Array destinationArray, Int32 destinationIndex, Int32 length, Boolean reliable)
at System.Collections.Generic.List`1.System.Collections.ICollection.CopyTo(Array array, Int32 arrayIndex)
at System.Collections.ArrayList.InsertRange(Int32 index, ICollection c)
at System.Windows.Data.ListCollectionView.b__1_0()
at MS.Internal.Data.SynchronizationInfo.AccessCollection(IEnumerable collection, Action accessMethod, Boolean writeAccess)
at System.Windows.Data.BindingOperations.AccessCollection(IEnumerable collection, Action accessMethod, Boolean writeAccess)
at System.Windows.Data.ListCollectionView.RefreshOverride()
at System.Windows.Data.CollectionView.RefreshInternal()
at System.Windows.Data.CollectionView.PostChange(NotifyCollectionChangedEventArgs args)
at PowerLauncher.ViewModel.MainViewModel.<>c__DisplayClass141_0.b__0()
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
2021-01-08.txt

@jgeernaert
please update .NET Core https://github.com/microsoft/PowerToys/issues/8921#issuecomment-755178682
It might not solve the issue but it allow to know that is not a problem with the runtime.
Thanks.
Closing as duplicate of https://github.com/microsoft/PowerToys/issues/8921

@jgeernaert
please update .NET Core https://github.com/microsoft/PowerToys/issues/8921#issuecomment-755178682
It might not solve the issue but it allow to know that is not a problem with the runtime.
Thanks.
Closing as duplicate of https://github.com/microsoft/PowerToys/issues/8921


[Llama] Support batched prefill

This PR supports the Llama modeling with batched prefill, which can bring higher throughput for the overall prefill process in serving.
Besides, the PR splits the attention function used in batching settings into two separate ones, so that we do not dispatch to the prefill/decode attention functions at runtime.

@MasterJH5574 @sunggg
In the line of 1206 and 1027 , extern function is not define，I don't know whether I have a problem with the way I use it or have forgotten the definition. I hope you can help me answer it when you are free


have any requirements for the browser version？

I have deploy the project 1.0.4.
But when I use chrome version 83.0.4103.116 and firefox  68.12 ，I can not open the web，(The page displays the circle statu)
when I use chrome v90 or upper, It can work.
so need to make any configuration，when use the lower version browser？
ths！

Thank you for your feedback.

You can see the browser versions we support so far from /ui/README.md.

In the future, we will add automatic prompts for unsupported browsers and inform users to upgrade their browsers to the corresponding version.


[stable/kong] Added clusterIP as a possible configuration value to service-kong-proxy.yaml

What this PR does / why we need it:
Adding the possibility to set clusterIP to none, makes it possible to use kong with cert-manager and it's hostport feature
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @hbagdi

/ok-to-test
This will be merged in after #19678 so please rebase on master once it is merged in and resolve all the conflicts. Thank you!


add double tap to zoom functionality

adds gesture detection for single and double taps, adds two new xml variables for enabling double tap zoom and setting the scale for it
also adds adjustments to constructors, removes chaining

How to enable double tap to zoom, im unable to do that

I'll be pushing this functionality to a snapshot today, I will let you know when it's done @blackhawk389

@jsibbold done?