ARView+SelfSceneRecordable Crash

When I set the minimum deployments 'iOS 10.0' and run the example ,
_cancelable = scene.subscribe(         to: SceneEvents.Update.self       ) { [weak sceneRecorder] (_) in         sceneRecorder?.render()       }
SceneEvents.Update will crash, can you help to me?

RealityKit is iOS 13+, there are known issues with lower version usage, maybe https://stackoverflow.com/questions/71000365/realitykit-app-and-lower-ios-deployment-target will help you.

iOS 16 is available already I will be dropping iOS 12 support anytime soon.

RealityKit is iOS 13+, there are known issues with lower version usage, maybe https://stackoverflow.com/questions/71000365/realitykit-app-and-lower-ios-deployment-target will help you.

thank you

RealityKit is iOS 13+, there are known issues with lower version usage, maybe https://stackoverflow.com/questions/71000365/realitykit-app-and-lower-ios-deployment-target will help you.

I have participated in the discussion below this link. Now that xcode14 has been fixed, can the code here be used without combine?

I have participated in the discussion below this link. Now that xcode14 has been fixed, can the code here be used without combine?

SceneRecorder needs to be notified when a new frame rendered.
If you can find another approach to get such information form RealityKit you are welcome to contribute :)
you can trigger sceneRecorder?.render() manually, for example by timer or CADisplayLink, but it will be inaccurate.

I have participated in the discussion below this link. Now that xcode14 has been fixed, can the code here be used without combine?

SceneRecorder needs to be notified when a new frame rendered. If you can find another approach to get such information form RealityKit you are welcome to contribute :)
you can trigger sceneRecorder?.render() manually, for example by timer or CADisplayLink, but it will be inaccurate.

Thanks for your answer, I will try it

I have participated in the discussion below this link. Now that xcode14 has been fixed, can the code here be used without combine?

SceneRecorder needs to be notified when a new frame rendered. If you can find another approach to get such information form RealityKit you are welcome to contribute :)
you can trigger sceneRecorder?.render() manually, for example by timer or CADisplayLink, but it will be inaccurate.

I have finished it.  Can you help to check it, is it ok?
@available(iOS 13.0, *) extension SceneRecorder: ARSessionDelegate {   public func session(_ session: ARSession, didUpdate frame: ARFrame) {     render()   } }

`import Foundation
import RealityKit
import SCNRecorder
import Combine
import ARKit
private var cancellableKey: UInt8 = 0
@available(iOS 13.0, *)
extension ARView: SelfSceneRecordable {
var _cancelable: Cancellable? {
get {
objc_getAssociatedObject(
self,
&cancellableKey
) as? Cancellable
}
set {
objc_setAssociatedObject(
self,
&cancellableKey,
newValue,
.OBJC_ASSOCIATION_RETAIN
)
}
}
public func injectRecorder() {
do {
sceneRecorder = try SceneRecorder(self)
#if !targetEnvironment(simulator)
session.delegate = sceneRecorder
#endif
}
catch { assertionFailure("(error)") }
}
}
@available(iOS 13.0, *)
extension SceneRecorder: ARSessionDelegate {
public func session(_ session: ARSession, didUpdate frame: ARFrame) {
render()
}
}
`
I have tested it

hm, interesting, it should be a bit inaccurate but fine.

inaccurate

emmm,   but my project must be set to iOS10.0(minimum deployments)

please don't confirm third party classes to third party protocols. it might lead to unexpected results.
I don't remember if I used session delegate somewhere inside the library, if not it should be fine.
It might be a bit inaccurate because the session(_ session: ARSession, didUpdate frame: ARFrame) function is called a moment before the content is rendered. But it might be ok.

Another thing here is that the content might be changed from somewhere else, without changing ARFrame, so this callback won't fire. but It should still work for reality kit.

public func renderer(_ renderer: SCNSceneRenderer, updateAtTime time: TimeInterval) ok thanks for your suggestion, do you think this protocol will be better?

public func renderer(_ renderer: SCNSceneRenderer, updateAtTime time: TimeInterval) ok thanks for your suggestion, do you think this protocol will be better?

it will, but RealityKit doesn't use SceneKit, so it just won't work.
You can just create you own delegate conforming to ARSessionDelegate and call render from there.


feat(account): update phone number

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


Failing test: requestToken__ensure_the_message_published_does_not_have_a_clientId

Run locally via make test_macOS:
✗ requestToken__ensure_the_message_published_does_not_have_a_clientId, expected to have Array<ARTMessage> with count 1, got 0
✗ requestToken__ensure_the_message_published_does_not_have_a_clientId, unexpected exception raised: *** -[__NSArray0 objectAtIndex:]: index 0 beyond bounds for empty NSArray

Also failing in Travis.
┆Issue is synchronized with this Jira Bug by Unito

Closing outdated issue. The codebase changed drastically since then and all skipped tests were unskipped, see PRs for https://github.com/ably/ably-cocoa/issues/1406 and https://github.com/ably/ably-cocoa/issues/1848
If you encounter frequently failing test again, please create a new issue with a best guess why it might be happening.


Errors_branch

Updated some of the frontend and cleaned up some code.
Added README as well.
See commits for more details.

Changes look good, might need to repush my userDashboard.html, routes and posts to ensure that they are still working.


[How to?] Embeddings for each .wav file in dataset folder

Hi! I am using a custom data which is represented as folder of .wav type files. I am curios how to create x-vector for each of the file in dataset. I tried using 'extract.py' and pytorch_run.sh only with changing according dirnames and others.
But for now, I understood there must be some feature extraction and splitting processes first. So there is a help request of how to solve the problem above using your pretrained models (I use xvec_preTrained/checkpoint_step309.tar from readme link).

If you need to only extract x-vectors, check out egs/sv_voices.sh.
The first part of the script performs x-vector extraction (before scoring, line 62) - you might need to add an exit statement to stop the script at that point. You will need to supply these variables: kaldiDir, wavDir, modelDir and transformDir

If you need to only extract x-vectors, check out egs/sv_voices.sh.
The first part of the script performs x-vector extraction (before scoring, line 62) - you might need to add an exit statement to stop the script at that point. You will need to supply these variables: kaldiDir, wavDir, modelDir and transformDir

Thank for your point! That helped a lot!

Thank for your point! That helped a lot!


Refactor regex validation checks to a dedicated method

The code has been updated to move regex validation checks to a Utility method, improving reusability. Multiple classes including IsDomain, IsFloat, IsGeohash, IsInt, IsUsaMarketName, RegEx, CsvValidator, all now utilize this utility method. This enhancement provides a centralized place for handling regex validations ensuring uniform error handling across the application. Minor modifications were also made to related comments and tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

15 of 18 (83.33%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 96.185%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Utils.php
4
7
57.14%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
958

Relevant Lines:
996

💛  - Coveralls


Upgrade sample to Expo 49 and fix warnings

This package now produces warnings about Stylesheet.compose when running in a project with Expo 49. These warnings should be addressed, and dependencies should be updated.

I'm new to using this repo but looks like it's this line that causes it https://github.com/khevamann/rn-responsive-styles/blob/c13e27e188226527053f19bcbae7216f62567a4e/src/hooks/useResponsiveStyle.ts#L15


[Azure Communication Services][CallAutomation]Create Customcontext class

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.
#customers' feedback:

SIP UUI format: The format used by ACS (userToUser) is not aligned to  RFC format.  Developers being more familiar with the latter, I recommend we align to RFC format (User-to-User) in SDK as well as EG payload, just we like we do for custom headers.

In line with No.1, it might be a better experience to manage the strict header key format at SDK layer, so that every developer doesnt have to remember to set the correct format in the request.
eg. Instead of callInvite.SipHeaders.Add("userToUser", "616d617a6f6e5f6368696;encoding=hex");
we do callInvite.SipHeaders.AddUserToUser("616d617a6f6e5f6368696;encoding=hex");

eg. Instead of callInvite.SipHeaders.Add("X-MS-Custom-", "434erdr454tf"),
we do callInvite.SipHeaders.AddCustomHeader("", "434erdr454tf");
#Design:
1.1.1	PSTN target
CallInvite callInvite = new CallInvite(new PhoneNumberIdentifier(targetParticipant), callerIdNumber);
callInvite.CustomContext.Add(new SIPUUIHeader(“jason”));
callInvite.CustomContext.Add(new SIPCustomHeader("Frank", "Rizzo")); // SDK will add prefix "X-MS-Custom-"

Then SDK will send this payload to PMA:
customContext: {
        "sipHeaders":
        {
                "User-to-User": "jason",
                "X-MS-Custom-Frank": "Rizzo"
        }
}

1.1.2	VoiP Target
CallInvite callInvite = new CallInvite(new CommunicationUserIdentifier(targetParticipant))
callInvite.CustomContext.Add(new VoipHeader("Foo-1", "Bar")); //no prefix

Then SDK will send below payload to PMA:
customContext: {
        "voipHeaders":
        {
                "Foo-1": "Bar"
        }
}

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.CallAutomation


#1466 Display_test UI enhancement and features Added

Hi Rahul ,
I have contributed in this project , in a program Display_test ..
A very good program , but there is some issues with functioning and Features , so I updated the features , enhance the UI and made it responsive and complete this program , I wish you can merge my pull request .. I want this to merge form GSSOC contribution .
Issue No. - 1466
The features I added in it -

Expanded Color Palette: Add more colors to the selection.
Current Color Display: Show the currently selected color's hex code.
Reset Button: Allow users to reset the background to the default color.
Smooth Transitions: Add smooth background color transitions.
Responsive Design: Ensure the website looks good on all devices.
Custom Color Picker: Let users add their own colors.
Copy to Clipboard: Enable users to copy the hex code easily.

the changes are -

hey I have mentioned the issue number , my issue number is - 1466 , Please merge it .. I hope you will and merge it in refernce to GSSOC level 2 or 3 according to your analysis of my work

fix #1466


Dockerfile doesn't work

$ docker build -t cloudmapper .

$ docker run -ti --rm cloudmapper bash
root@7f0ad00a93e6:/opt/cloudmapper# python cloudmapper.py prepare --config config.json.demo --account demo
Building data for account demo<PHONE_NUMBER>12)
Traceback (most recent call last):
  File "cloudmapper.py", line 72, in <module>
    main()
  File "cloudmapper.py", line 66, in main
    commands[command].run(arguments)
  File "/opt/cloudmapper/commands/prepare.py", line 831, in run
    prepare(account, config, outputfilter)
  File "/opt/cloudmapper/commands/prepare.py", line 660, in prepare
    cytoscape_json = build_data_structure(account, config, outputfilter)
  File "/opt/cloudmapper/commands/prepare.py", line 419, in build_data_structure
    for region_json in get_regions(account, outputfilter):
  File "/opt/cloudmapper/shared/common.py", line 161, in get_regions
    regions = pyjq.all(".Regions[]{}".format(region_filter), region_data)
  File "/usr/local/lib/python3.7/site-packages/pyjq.py", line 49, in all
    return compile(script, vars, library_paths).all(_get_value(value, url, opener))
  File "_pyjq.pyx", line 209, in _pyjq.Script.all (_pyjq.c:2561)
_pyjq.ScriptRuntimeError: Cannot iterate over null (null)
root@7f0ad00a93e6:/opt/cloudmapper#

This issue does not appear to be related to Docker.

@0xdabbad00
It does.
Use the same image I build with exist Dockerfile, but I mount the repo to container (-v $(pwd):/opt/cloudmapper ), it works.
Could you try it on your computer? this problem is easily duplicated
docker build -t cloudmapper .

# doesn't work, that's the issue I reported here.
$ docker run -ti --rm cloudmapper bash
root@3095bc277b04:/opt/cloudmapper# python cloudmapper.py prepare --config config.json.demo --account demo

# does work
$ docker run -ti --rm -v $(pwd):/opt/cloudmapper cloudmapper bash
root@fab09cf1fd9b:/opt/cloudmapper# python cloudmapper.py prepare --config config.json.demo --account demo
Building data for account demo<PHONE_NUMBER>12)
- 16 nodes built in region us-east-1
- 3 external CIDRs built
- 35 connections built
root@fab09cf1fd9b:/opt/cloudmapper#

This issue does not appear to be related to Docker.

@0xdabbad00 I'm seeing exactly this same problem.
What do you think is the problem?

This issue does not appear to be related to Docker.

@0xdabbad00 I'm seeing exactly this same problem.
What do you think is the problem?

I figured out the problem; I was missing the account-data/demo because .dockerignore prevented the account-data from getting added to the docker image(a good thing).
To run the demo using docker, you must bind mount the account-data directory, e.g.
docker run --rm -it -p 8000:8000 -v `pwd`/account-data:/opt/cloudmapper/account-data  cloudmapper /bin/bash


Call narrow:scan from vim-mode-plus search

First, let me say thank you so much for all the work you've been putting into the narrow package. It feels like Christmas almost every time I open the editor lately.
I especially like narrow:scan as a replacement of narrow:lines, and it would be even better if it could be called from the vim-mode-plus search-input like narrow:lines can. I'm sure this will be added eventually, but since I didn't see an issue for it I thought I would add one.
Actually, I got it working on my system by adding the line:
'vim-mode-plus-user:narrow:scan': => @narrow('scan', uiInput: confirmSearch())

near the end of the main.coffee file.
If you'd like I can make a pull request so you don't have to worry about it.

Agree will introduce in next version. No need to PR. Thanks for your feedback.

Sounds great, thanks.


BUG: Long values displayed incorrectly in json response

hi, i am using following dependencies
<dependency>
		    <groupId>io.springfox</groupId>
		    <artifactId>springfox-swagger2</artifactId>
		    <version>2.7.0</version>
		</dependency>

		<dependency>
		    <groupId>io.springfox</groupId>
		    <artifactId>springfox-swagger-ui</artifactId>
		    <version>2.7.0</version>
		</dependency>

		<dependency>
		    <groupId>io.swagger</groupId>
		    <artifactId>swagger-jersey2-jaxrs</artifactId>
		    <version>1.5.17</version>
		</dependency>

when i am making a get request with postman as follows:
http://localhost:8081/hai/api/v1/findDonatorOldProjects/100000000000000614
i get the response correctly as follows:
{
    "errorCode": "SUCCESS_CODE",
    "errorCodeDesc": "تمت العملية نجاح",
    "errorCodeValue": 0,
    "response": {
        "data": [
            {
                "id":<PHONE_NUMBER>00001478,
                "name": "الرغيف الخير - الأردن"
            }
        ]
    }
}

but when i make the request with swagger ui, i get the following response
{
  "errorCode": "SUCCESS_CODE",
  "errorCodeDesc": "تمت العملية نجاح",
  "errorCodeValue": 0,
  "response": {
    "data": [
      {
        "id":<PHONE_NUMBER>00001470,
        "name": "الرغيف الخير - الأردن"
      }
    ]
  }
}

the long value<PHONE_NUMBER>00001478 is displayed incorrectly as<PHONE_NUMBER>00001470

Same problem here!
I use springdoc-openapi library (uses swagger-core internally) and faced with this issue. But when I started investigate it the discussion with the maintainer brings me to this issue.
See: https://github.com/springdoc/springdoc-openapi/issues/2174

This is because js has right bound: 2^(53)-1 and java: 2^(63)-1. Seems nothing that we can really do here.


Exempt qemu from tmpfs sandbox

Summary: Tmpfs does not work with qemu because we can't use tmpfs to host a plan9 volume right now.
Relevant Issues: #709
Type of change: /kind test-infra
Test Plan: This works without changing sandbox tmpfs setting:
bazel test -c opt --config=qemu-bpf  --cache_test_results=no --remote_download_outputs=all //src/stirling/source_connectors/socket_tracer:redis_trace_bpf_test --test_output=streamed --//bazel/test_runners/qemu_with_kernel:kernel_version=latest
The rest of the test should pass in Jenkins. #ci:bpf-build.

@pixie-io-buildbot test this please.


Custom themes

custom themes can be defined using this functionality, integrates with the bat.exe installed on the system if present.

Very cool, that you implemented this functionality on your own. But that doesn't go in the direction that I would like it to go.
I'm only using bat for the built-in themes. So I don't want to use anymore of it. The themes are a concept from syntect as far as I understood the docs and the loading of them should happen in this way. If the cache is even useful is another thing, you probably load one or two custom themes and that is probably speedy enough. If not, I would go in the direction that plugins are cached and use a file with msgpack. Also I don't see the usefulness in passing the directories where to load and cache themes as flags, they should be config entries only. Also, we can totally avoid using env variables by using the config which I would prefer.
I will definitely give implementing custom themes a go, but I would also like to use features that I implemented into nushell myself recently, so they will be only available in the next nu version.
I suggest you use your implementation for now, I will tag you in a PR with my impl and then in the next nu release, we should have this in the release of this plugin. Does that sound good for you?

Here's the kicker, I tried to have a look so I don't need to build the cache like that, however Highlightingassets doesn't provide a method to extend the themes from created highlighting assets.
This would result in the following issue: "if a custom theme path is set and used, the default themes become unavailable". For this reason a cache needs to be built if you want to keep the old themes available.
Also, I don't see what you mean with defining the paths as flags, they are defined in the $env.config.plugins.highlight and then with their respective name, just like the default theme you can set for example. The only flag I added is the --build-cache flag, because actually building the cache takes a couple seconds, which is unecessary and annoying delay. Unless I'm missing something of course.
I understand calling bat multiple times through the codebase is annoying for you if you don't want to use bat.exe. But it solves errors with mismatching versions of cache bat version definition and bat.exe version.
If you can find a way without invoking bat under the hood to extend the themeset, that would be better for sure, but I haven't found a way. I'll keep using my implementation for now, but I'll keep an eye on your attempt.


Load a single column into multiple target columns

Fixed with https://github.com/marcin-krystianc/JollyJack/pull/40


No snapshots, breaks version workflow

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

Project coverage is 100.00%. Comparing base (115fe79) to head (c0ad859).

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          265       265
=========================================
  Hits           265       265

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


Tariffs: add time-dependent charges (peak and off-peak)

Refs https://github.com/evcc-io/evcc/discussions/12331#discussioncomment-8508798, https://github.com/evcc-io/evcc/discussions/12690

Mit: 0.124.9+1710209577
Fehlermeldung,:
`[ main  ] ERROR 2024/03/12 05:42:10 failed configuring grid tariff:
cannot create tariff type 'energinet': 1 error(s) decoding:

'' has invalid keys: zones`

Konfiguration:
tariffs:
  currency: DKK
  grid:
    type: energinet
    region: dk1
    charges: 0.966
    zones:
      - days: Mo-So
        hours: 0-6
        price: 0.11012
      - days: Mo-So
        hours: 6-17
        price: 0.33035
      - days: Mo-So
        hours: 17-21
        price: 0.99105
      - days: Mo-So
        hours: 21-0
        price: 0.33035
    tax: 0.25
  feedin:
    type: energinet
    region: dk1
    charges: -0.01
    tax: 0.0```

Hi, Any news about this issue, would help. Since in Finland there can also be different prices for night and daytime tariffs. So that would be Entso-E + additional tariffs based on zones.


merge 2 or 2+ pages into 1 single page

Is there any demo how to merge 2 or 2+  pages into 1 single page like pyPDF2?
I have looked to the folder pyhanko-tests, however, due to I'm not familiar the low level details how the PDF was assmbled, the atempt to add a qr code imgage and a text with link was never success.
Would you please advise the equivalent of pyPDF2 as bellow for pyhanko?
    newPdf = PdfFileReader(buffer)
    filename = os.path.join(path, self.draft)
    existingPdf = PdfFileReader(open(filename, 'rb'))
    page = existingPdf.getPage(0)
    page.mergePage(newPdf.getPage(0))
    output = PdfFileWriter()
    for page in range(existingPdf.getNumPages()):
        output.addPage(existingPdf.getPage(page))
    output.write(buffer)

Closing this one as wontfix due to lack of activity. Feel free to reopen if there's new information. :)


[Backport 2.16] Fix undeclared dependencies on CTR_DRBG (and add test)

This is the 2.16 backport of #3378. Differences from the main PR:

first commit (test suites): the SSL test suite doesn't have the new stuff testing full handshakes, so nothing to change here
second commit (programs/fuzz): skipped entirely as there is no programs/fuzz here
third upstream commit (all.sh): apart form context conflicts, used config.pl instead of config.py which is not in 2.16.

I've amended the last commit in order to fix the issue found by Gilles.

The only failure in pr-head was an infrastructure issues, and pr-merge fully passed, so this is good enough CI for merging.


Add ipv4/enabled deviation

This pull adds a deviation for ipv4/enabled leaf. It is expected that ipv4 to be enabled by default for the devices with this deviation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1462

Relevant Lines:
2012

💛  - Coveralls

/gcbrun


added s3 permissions

I found out how to add extended permissions to Elastic Beanstalk, but I still think these permissions should be granted by default.
So you can merge @EliranMoveo


Multiple GiphyDialog listeners conflict each other

🐛 Bug Report
I have multiple screens in my app that use GiphyDialog.addListener which seems to be the only way to take action on the picked Giphy.
Some of those screens are in the same navigation stack, meaning I can be in a screen that has such listener, and navigate to another screen that also uses its own giphy listener => such navigation won't unmount the previous one obviously (and I don't want to; navigation.navigate vs navigation.replace)
PROBLEM is: if the new screen I am into triggers the listener, it will also trigger the underlying listener of the previous screen in the stack that is still listening!
Obviously not what I want, since they both are supposed to target a very different purpose and task.
To Reproduce
Clearly described above
Expected behavior
I would like a way to differentiate listeners; or another way to act upon the picked giphy from the screen itself
(Write what happened. Add screenshots, if applicable.)
Your Environment

Giphy React Native SDK version: 2.4.1
React Native version: 0.69.12
Platform(s) (iOS, Android, or both?): Both
Device info (Simulator/Device? OS version? Debug/Release?): All

Reproducible Demo

Hey @pierroo,
We've concluded that it would be most effective to put the control of event listener management in the hands of the SDK users.
Every application has unique requirements and navigation flows. By allowing you to manage event listeners directly, our SDK provides the flexibility to adapt to various use cases and navigation structures.
This way, you can subscribe and unsubscribe to events in a manner that aligns seamlessly with your application's lifecycle and user navigation patterns.
Managing an internal state listeners within the SDK can lead to unnecessary complexity and potential bugs.
In our example app, I used the following approach to "isolate" views:
  import { useIsFocused } from '@react-navigation/native';
  const addMediaRef = useLatest(addMedia);
  const isFocused = useIsFocused();
  useEffect(() => {
    if (isFocused) {
      const listener = GiphyDialog.addListener('onMediaSelect', (e) => {
        addMediaRef.current(e.media);
        GiphyDialog.hide();
      });
      return () => {
        console.log('Removing listener');
        listener.remove();
      };
    }
  }, [isFocused, addMediaRef]);

This approach ensures that your event listener is active only when the component is in focus.
Please let me know if that works for you.

Hi @ALexanderLonsky ,
Thank you so much for taking the time to answer.
This focusRef is finally exactly what we ended up doing, it works well enough for now.
Although another question related would be about the giphy configuration.
If I use this in a screen:
GiphyDialog.configure({
mediaTypeConfig: [GiphyContentType.Sticker],
})
and a different mediaTypeConfig in another, then the first (or last I'm not sure) will overwrite the other even if I come back later to said screen.
Ideally I would also want this configuration to be related to the screen it's called in only.

For your case, you can reconfigure it similarly:
just put the config inside the useEffect:
useEffect(() => {
    if (isFocused) {
      GiphyDialog.configure({
                mediaTypeConfig: [GiphyContentType.Sticker],
            });
      const listener = GiphyDialog.addListener('onMediaSelect', (e) => {
        addMediaRef.current(e.media);
        GiphyDialog.hide();
      });
      return () => {
        listener.remove();
      };
    }
  }, [isFocused]);

Then you'll have a specific config for a particular view.


Updated my name in CONTRIBUTORS.md file and added the fonts in the fonts.json file

Please review my changes.

thanks @AshKemp


Capitalize log in link on logged-out page

https://github.com/fetlife/fetlife-web/pull/13420#issuecomment-676447582

Of the 900 translations in this file there are quite a few that are lowercase. Without knowing the context they're used in I'd hesitate to update them all.

@seanlinsley oh, sorry. I meant capitalization for just that one link for log_in.

But you are right, maybe it's not that simple and someone speaking the language should check if it makes sense. We can revisit later on, as Rene suggested https://github.com/fetlife/fetlife-web/pull/13420#discussion_r472424266.
All good. Thanks for doing this!


python310Packages.androidtv: 0.0.69 -> 0.0.70

Diff: https://github.com/JeffLIrion/python-androidtv/compare/v0.0.69...v0.0.70
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 205556 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.androidtv
python310Packages.androidtv
python39Packages.androidtv


Trashcan icon and “close all results” is confusing

What do you think would be better here? "Clear results" instead of "close all results"?
I think the trash can is appropriate at the item-level and makes sense. There's a tooltip on hover for "Close result."


Allow IsarLinks to work directly with IDs instead of objects.

Is your feature request related to a problem? Please describe.
I am trying to parse a large list of linked objects. In order to set up links, I currently have to load the target object from Isar, and then put it into the link to establish the relationship. I would like to be able to just set the id instead of having to load the full object.
It is also a possibility that the target object has not been put into Isar yet. If isar were to validate the existence of the target object, it would be ideal if this validation could be deferred until all objects are supposed to be created (transaction complete?)
Describe the solution you'd like
ObjectBox allows you to set ToOne.targetId, which works well for this use-case.
Describe alternatives you've considered
I looked at IsarCollection.attach, but that looks like it's for internal use only. Maybe that could be exposed in the API in usable way?
Version

Platform: Android, Web
Flutter version: 3.0.5
Isar version: 3.0.0-dev.8

I like this idea. Isar already performs validation whether a linked object exists so it should be rather straightforward to implement. The only problem is finding a nice API that works with the existing one.


Restore coveralls integration

Coveralls/Coverage integration with Github build checks is gone for all PR builds.
No way to directly drill-down into PR build coverage from the PR.
Could you please look into this when you have a chance?

Looking into it.

Thx!

Pretty sure it's working again. I get coverage data on #221 for a recent rebase.


HDDS-10645. Support x-amz-metadata-directive in CopyObject

What changes were proposed in this pull request?
Need to support x-amz-metadata-directive: specifies whether the metadata is copied from the source object or replaced with metadata that's provided in the request. When copying an object, you can preserve all metadata (the default) or specify new metadata. If this header isn’t specified, COPY is the default behavior.
Valid Values: COPY | REPLACE
See: https://docs.aws.amazon.com/AmazonS3/latest/API/API_CopyObject.html#API_CopyObject_RequestSyntax
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10645
How was this patch tested?
Unit test and acceptance test.
Clean CI run: https://github.com/ivandika3/ozone/actions/runs/8589734492

@kerneltime @DaveTeng0 @SaketaChalamchala @vtutrinov Could you help take a look when you have time?

Thanks @ivandika3 for the patch, @tanvipenumudy for the review.

Thank you for the reviews @adoroszlai @tanvipenumudy


Running start.rb results in an error

Running start.rb after cloning the project results in a Ruby error.
OS X Yosemite 10.10.5
ruby 2.3.3p222 (2016-11-21 revision 56859) [universal.x86_64-darwin17]
Reagans-Mac-Pro:hello-treehouse reagan$ ruby start.rb
Starting...
start.rb:5:in `<main>`: Don't start! It will explode! (RuntimeError)

same here: running start.rb after cloning the project results in ruby error.
windows 8.1
ruby 2.4.2p198(2017-09-14 revision 59899)
$ ruby start.rb start.rb:5:in '<main>': Don't start! It will explode! (RuntimeError) Starting...


BUG: previous and next button is overlapping at cards

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
when images are scrolling the buttons are lie over the image
https://github.com/user-attachments/assets/57fcae86-b8e7-4176-96b2-eb8292546e11
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Cuda problem

Hi ,
Each time that I try to run diff dock I get the following message:

/.conda/envs/diffdock/lib/python3.8/site-packages/torch_geometric/typing.py:31: UserWarning: An issue occurred while importing 'torch-scatter'. Disabling its usage. Stacktrace: /.conda/envs/diffdock/lib/python3.8/site-packa
ges/torch_scatter/_scatter_cuda.so: undefined symbol: _ZN2at4_ops6narrow4callERKNS_6TensorElll
  warnings.warn(f"An issue occurred while importing 'torch-scatter'. "
/.conda/envs/diffdock/lib/python3.8/site-packages/torch_geometric/typing.py:42: UserWarning: An issue occurred while importing 'torch-sparse'. Disabling its usage. Stacktrace: /.conda/envs/diffdock/lib/python3.8/site-packag
es/torch_sparse/_diag_cuda.so: undefined symbol: _ZN3c106detail19maybe_wrap_dim_slowEllb
  warnings.warn(f"An issue occurred while importing 'torch-sparse'. "
Traceback (most recent call last):
  File "/.conda/envs/diffdock/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/.conda/envs/diffdock/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/DiffDock/DiffDock/inference.py", line 10, in <module>
    from torch_geometric.loader import DataLoader
  File "/.conda/envs/diffdock/lib/python3.8/site-packages/torch_geometric/__init__.py", line 1, in <module>
    import torch_geometric.utils
  File "/.conda/envs/diffdock/lib/python3.8/site-packages/torch_geometric/utils/__init__.py", line 8, in <module>
    from .dropout import dropout_adj, dropout_node, dropout_edge, dropout_path
  File "/.conda/envs/diffdock/lib/python3.8/site-packages/torch_geometric/utils/dropout.py", line 6, in <module>
    import torch_cluster  # noqa
  File ".conda/envs/diffdock/lib/python3.8/site-packages/torch_cluster/__init__.py", line 18, in <module>
    torch.ops.load_library(spec.origin)
  File ".conda/envs/diffdock/lib/python3.8/site-packages/torch/_ops.py", line 643, in load_library
    ctypes.CDLL(path)
  File "/.conda/envs/diffdock/lib/python3.8/ctypes/__init__.py", line 373, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: /.conda/envs/diffdock/lib/python3.8/site-packages/torch_cluster/_grid_cuda.so: undefined symbol: _ZN2at4_ops6narrow4callERKNS_6TensorElll

Would you any suggestion to solve this issue?
Thanks in advance

Hi @H-EKE
This seems to be an issue with torch_cluster. Unfortunately I also frequently have issue with torch_cluster when setting up new environments.
My suggestions are (in this order): create a new environment, install PyTorch being careful to select on the PyTorch website the command for the CUDA version that you have installed on the GPU, install PyTorch geometric with all its packages being careful that you are taking the one supporting the same CUDA version.
I hope this helps,
Gabriele

Dear @gcorso ,
Thanks for your fast reply!

see issue #113


(regression?) Program page "Add indicator" button: Low and medium roles should not see it

Issue
At the moment, logged in as a Low or Medium user, I can see the "Add indicator button at the top of the program page:

On Prod, I don't see it at all, and this is desired behavior.

PASSED. No longer seeing the "Add indicator" button on program pages when I log in as a Low or Medium user.


[codesandbox] bug report app fails to load

Describe the bug
Codesandbox isn't working with the library. It appears to be an issue with Codesandbox, not our bundle. I've opened codesandbox/codesandbox-client#4456. This issue is simply to track the progress there.
Related to #1037

removing from sprint until Codesandbox  issues a PR


Unable to use tfp.optimizer.lbfgs_minimize, error after 2 iterations

Hi I am unable to use tfp.optimizer.lbfgs_minimize, following the tutorial on https://www.tensorflow.org/probability/examples/Optimizers_in_TensorFlow_Probability.
Similar to https://github.com/tensorflow/probability/issues/398,
I am facing error "Inputs to operation Select of type Select must have the same size and shape.  Input 0: [110] != input 1: [1] [Op:Select]"
I noticed that when my cost function if of shape (1,) and my gradient of shape (x,1) it says x!=1
But what I don't understand is why does my cost function need to be of shape(x,1)? When I add a placeholder return which returns shape (x,1) for cost value as well, it runs.

@prathak
Please share a simple stand alone code along with tensor flow version for us to replicate the issue faced.


Add support for PEP 561, typed API

:wrench: Add support for PEP 561, typed API.
The whole internal API is type annotated, this marks it to add support for PEP 561, so that mypy can check users' (developers') code.
Related to #53

Codecov Report

Merging #58 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   99.96%   99.96%
=======================================
  Files         166      166
  Lines        2924     2924
=======================================
  Hits         2923     2923
  Misses          1        1

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 095da33...7b5a050. Read the comment docs.


Create a web page to style the text using  CSS and java script

Create a web page that is showing your name which should change the style such as :-
your name should become bold when you click on "bold" button and
your name should become italic when you click on "italic " button and
your name should be underlined when you click on "underline" button.

can you assign this task to me

can you pls assign me this issue?

please assign this task to me


Copy EUI64/Application key functionality doesn't work

https://github.com/chirpstack/chirpstack/assets/795984/efd3c0fa-16bb-464f-919a-749739290ba2
I'm using image: chirpstack/chirpstack:4.
Tested in the latest Chrome and Safari, MacOS Ventura 13.4

Strange, it works for me using the latest version (v4.4.1). Could you try to reproduce using the latest version?
chirpstack/chirpstack:4 means any version from v4.0.0 - v4.4.1. You can use :4.4.1 to explicitly set the version. Then also make sure you re-create the container.

How do you access Chirpstack? With HTTP or HTTPS?
I can reproduce your issue if I access my instance with HTTP but when I use HTTPS it works.
This is also what a quick web search for this issue shows, there are a lot of reports for various web apps...

Ah, then it might be a "security feature" of the browser. With localhost (which might be treated as an exception) I do not experience this issue over HTTP. But my test-server is using HTTPS and I also was not able to reproduce.
Actually, I believe getting an approximate location using the browser geolocation API is also blocked when using HTTP.
I think this issue then should be changed to:
Show error when access to clipboard API is not allowed due to restrictions.

Ya that's what I found, localhost is also treated as safe, some mention that there is some command line option for Chrome and other browser to deactivate that check and other sources talk about possible workarounds but also mention that those workarounds might not be forever...

I don't want to go into browser specifics, but I think showing a warning or error with at least a hit should be helpful. It looks like now for the user it looked like the copy to clipboard worked, which is not true.

Ya its ok, I just wanted to provide additional context...
Actually the state-of-the-art-way has this "limitation" and that's it.
A message would be fine so...


Cannot import Arn as Maven dependency

Bug Details
Impossible to import Arn as a Maven dependency
[WARNING] Failed to download maven-metadata-sonatype.xml [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download maven-metadata-efekosdev.xml [https://efekos.dev/maven/]
[WARNING] Failed to download arn-paper-0.4.pom [https://efekos.dev/maven/]
[WARNING] Failed to download arn-paper-0.4.pom [https://repo.papermc.io/repository/maven-public/]
[WARNING] Failed to download arn-paper-0.4.pom [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download arn-paper-0.4.pom [https://repo.maven.apache.org/maven2/]
[WARNING] The POM for dev.efekos.arn:arn-paper:jar:0.4 is missing, no dependency information available
[WARNING] Failed to download arn-paper-0.4.jar [https://efekos.dev/maven/]
[WARNING] Failed to download arn-paper-0.4.jar [https://repo.papermc.io/repository/maven-public/]
[WARNING] Failed to download arn-paper-0.4.jar [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download arn-paper-0.4.jar [https://repo.maven.apache.org/maven2/]
Cannot resolve dev.efekos.arn:arn-paper:0.4

To Reproduce the Bug:
Add Arn as a Maven dependency, then try to download it

I forgot to upload the new dependencies to the server. I'll upload it and close this issue in a few minutes.

It's still not working, but when editing my import to use version "0.4.0-SNAPSHOT" instead of "0.4", I'm getting this error :
[WARNING] Failed to download maven-metadata-efekosdev.xml [https://efekos.dev/maven/]
[WARNING] Failed to download maven-metadata-sonatype.xml [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download maven-metadata-sonatype.xml [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download maven-metadata-papermc-repo.xml [https://repo.papermc.io/repository/maven-public/]
[WARNING] Failed to download maven-metadata-local.xml [file://C:\EfekosDev\Maven/]
[WARNING] Failed to download maven-metadata-efekosdev.xml [https://efekos.dev/maven/]
[WARNING] Failed to download maven-metadata-papermc-repo.xml [https://repo.papermc.io/repository/maven-public/]
[WARNING] Failed to download maven-metadata-sonatype.xml [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download arn-root-0.4-SNAPSHOT.pom [https://efekos.dev/maven/]
[WARNING] Failed to download arn-root-0.4-SNAPSHOT.pom [https://repo.papermc.io/repository/maven-public/]
[WARNING] Failed to download arn-root-0.4-SNAPSHOT.pom [https://oss.sonatype.org/content/groups/public/]
[WARNING] Failed to download arn-root-0.4-SNAPSHOT.pom [file://C:\EfekosDev\Maven/]
[INFO] Validation error:

[ERROR] Failed to read artifact descriptor for dev.efekos.arn:arn-paper:jar:0.4.0-SNAPSHOT
[ERROR] [server] Maven transfer artifact problem: dev.efekos.arn:arn-root:pom:0.4-SNAPSHOT
Cannot resolve dev.efekos.arn:arn-root:0.4-SNAPSHOT

It's looking for arn-root, but only arn-common is on your server, maybe this is the reason?

arn-root isn't on my local repo either, it isn't even uploaded to anywhere when you deploy the project. Assuming that you used arn-paper or arn-spigot and didn't add arn-root manually. It shouldn't try to find arn-root.

arn-root isn't on my local repo either, it isn't even uploaded to anywhere when you deploy the project. Assuming that you used arn-paper or arn-spigot and didn't add arn-root manually. It shouldn't try to find arn-root.

True, I didn't add it myself in my pom file.
I thought you splitted the dependency, with the arn-root base and the arn-paper would implement it with a few differences from the spigot version
(Sorry for my poor english, it is not my native language)

I just found out why it works on my machine. I once deployed an old version of Arn where arn-root was also deployed, and my IDE saved it to .m2 folder in the user folder when installing it again. So my IDE thought arn-root was present and didn't give me any errors this whole time :/. I'll move classes of arn-common to arn-root and delete arn-common so arn-root is also deployed.

arn-root is now uploaded to the server and everything should be fine.


Adds name and getPtr to Meta

Gives way to name holders using registerName.
Adds name() function to MetaHolder which returns the registered name of the class.
Interface is a bit icky (have to call two functions), open to suggestions.

I can agree with registering class name (though you don't have to include <string.h> in Meta.h, remove it and I'll accept this part of PR)
But getPtr is pretty dangerous, because there are situations where you can create Member without passing member pointer to it. And why would you need to get raw pointer to member anyway? People rarely use them and it can make code ugly.
After all, there are Member::get/set functions after all which do all the checking :)
Can you show some cases where you need pointer to member from Member class? Maybe we can come up with some better functions for it.

you don't have to include in Meta.h,

Oops,  might be leftover from when I was returning a string instead of char*. I'll remove this.

why would you need to get raw pointer to member anyway?

My use case is for passing to python bindings.  It's convient to expose these ptrs so python can directly read/write to c++ objects.
Example:
py::class<MyClass> pyclass(meta::<MyClass>::getName(), py::init<>)

meta::doAllMembers([&pyclass] (const auto& m) { def_readwrite(m.getName(), m.getPtr())  });

Okay, I see. Well then, you'll need to add function named hasPtr() which will return hasMemberPtr. If it's false and you still call m.getPtr(), it needs to throw std::runtime_error, because there's no way for caller to check if he got correct pointer or not.
And then I'll accept all your changes :)
Btw, it would be great to have an example of defining py::class using MetaStuff lib which will use getter/setter functions to define python properties (instead of just using raw pointers to members)

Ok, I'll merge it and I'll add my proposed changes myself :)


Adding New Google Sign in Button

Feature description
The  google sign button is blurry and has a bad user experience. Changing it to new one with better pixel can have a good user experience.
Adding the screenshot of previous google sign in button. @maityamit

👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I agree to follow this project's Code of Conduct

added in Pr #67


Introduction missing for Protein Translation exercise

The Introduction for the  Protein Translation exercise is shown as the single word "Introduction" with no text below that.

Are you seeing that on the website? I just checked and I could see the full description, it should be the text in the README here - https://github.com/exercism/cpp/tree/master/exercises/protein-translation
There doesn't seem to be an issues here, but it might be an issue with the Website if you're still seeing it. In which case we'd need to open an issue in https://github.com/exercism/exercism

Yes it does appear to be just the website.


ros2 service replier

Implements ROS2 service

This resolves #51

ping @OpusK, hey can you take a look?

ping @OpusK, hey can you take a look?

Regarding this CI failure:
https://travis-ci.org/github/ROBOTIS-GIT/ros2arduino/builds/751887104#L405
In file included from /home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/cores/arduino/SafeRingBuffer.h:25:0,
                 from /home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/cores/arduino/Uart.h:23,
                 from /home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/variants/mkr1000/variant.h:164,
                 from /home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/cores/arduino/Arduino.h:48,
                 from sketch/subscriber.ino.cpp:1:
/home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/cores/arduino/sync.h:28:22: error: expected unqualified-id before 'for'
 #define synchronized for (__Guard __guard; __guard.enter(); )
                      ^
/home/travis/arduino_ide/libraries/ros2arduino/src/ros2/xrcedds/micro_xrce_dds/lib/include/uxr/client/core/session/session.h:97:10: note: in expansion of macro 'synchronized'
     bool synchronized;
          ^~~~~~~~~~~~
/home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/cores/arduino/sync.h:28:44: error: '__guard' does not name a type; did you mean '__Guard'?
 #define synchronized for (__Guard __guard; __guard.enter(); )
                                            ^
/home/travis/arduino_ide/libraries/ros2arduino/src/ros2/xrcedds/micro_xrce_dds/lib/include/uxr/client/core/session/session.h:97:10: note: in expansion of macro 'synchronized'
     bool synchronized;
          ^~~~~~~~~~~~
/home/travis/.arduino15/packages/arduino/hardware/samd/1.8.11/cores/arduino/sync.h:28:61: error: expected unqualified-id before ')' token
 #define synchronized for (__Guard __guard; __guard.enter(); )
                                                             ^
/home/travis/arduino_ide/libraries/ros2arduino/src/ros2/xrcedds/micro_xrce_dds/lib/include/uxr/client/core/session/session.h:97:10: note: in expansion of macro 'synchronized'
     bool synchronized;
          ^~~~~~~~~~~~
exit status 1

It has been reported here: https://github.com/arduino/ArduinoCore-samd/issues/596
I don't think it is related to the changes made in this pull request in any way. It's simply that the breakage was introduced in the 1.8.11 release of the Arduino SAMD Boards platform on 2020-12-24, which came after the last CI run, but before this PR was submitted. If you compile the ros2arduino code from the master branch with Arduino SAMD Boards 1.8.11 you'll get the same error.

@amfern Sorry for the late.
Unfortunately, I no longer have administrative rights to this repo...

Thanks for coming back, i learn alot when tinckering with this repo. And i already moved on to micro_ros_arduino


Make docs and RTD docs build

Fixes #458

Codecov Report

Merging #552 (c7c19c4) into main (0c67e2b) will increase coverage by 24.51%.
The diff coverage is n/a.

Additional details and impacted files

@ijpulidos @mikemhenry once again here we are getting workers crashing on tests unrelated to these changes. Any ideas? I will turn off parallelism in data CI, but still very strange.


update "gulp-webpack" dependency name

I could not install the dependencies of the project, because the "gulp-webpack" dependency name was changed to "webpack-stream" with version 2.0.0 ^.
I am using Node v4.1.1 and npm v2.14.4.
gulp-webpack changed for webpack-stream

The rename should not hinder you from installing the project.

To try "npm install" it appears that:
npm WARN deprecated<EMAIL_ADDRESS>Renamed to https://www.npmjs.com/package/webpack-stream

npm ERR! registry error parsing json
npm ERR! registry error parsing json
npm ERR! registry error parsing json

and kept repeating "npm ERR! registry error parsing json" for a good time. I changed the dependency name to the new version and it worked

Those two are unrelated.

Ok, I try reinstalling everything then. Thanks for listening.

You can try 'git co package.json' to get the version from git.

Hopefully that fixes the situation.
Happy Coding!


Use a configuration first method (modifiers and states) for Toolbar Layout

Relates directly back to #273

Closed because of the large updates being made to the toolbar layout


Upgrade react-scripts to 3.0.0 and enable absolute-import rule + lint fix

I can't configure baseUrl to "." because of react-scripts restrictions, so we can't have paths like "src/*"
React-scripts doesn't support paths so we can't add prefix before absolute imports :/
And finally tslint doesn't support grouped-imports with baseUrl enabled 👌
Still better than relative imports IMO
Edit : If we really want a prefix before all absolute imports, like src/, we could embed all files/directories actually in src in another subdirectory... 💩

I have this warnings :
./src/pages/Settings.tsx
  Line 112:  Emojis should be wrapped in <span>, have role="img", and have an accessible description with aria-label or aria-labelledby  jsx-a11y/accessible-emoji
  Line 113:  Emojis should be wrapped in <span>, have role="img", and have an accessible description with aria-label or aria-labelledby  jsx-a11y/accessible-emoji

@notFloran Fixed

Also fixed issue 20 https://github.com/TheGrowingPlant/gitlab-reviewer/issues/20 if gitLab variable is not initiated, early return in componenDidMount


Remove version from aws-sdk gem

Remove version from aws-sdk gem, the needed version is determined by the amazon provider.

see my PR here https://github.com/ManageIQ/manageiq-providers-amazon/pull/57
I think we should lock at "~>2"

@durandom I think we need to make it versionless here first (otherwise CI won't pass). So the version will be driven only by the amazon provider gem. Then we can remove it from here, once it's not needed.

TBH I'd say we should lock at '~> 2' here as well, because we are using it here.
Let's lock both at 2 and we are fine

@miq-bot assign blomquisg

LGTM go for it


Pattern: Content block - segmented Design QA testing

Additional Information

alpha Dev Issue link (#1020)
Design specs sketch
Functional specs Issue link (#1028)
Testing scope includes only Chrome browser

Tasks
Ensure the Content block - segmented pattern has been tested and reviewed by design + dev.
Acceptance criteria

[ ] Design review performed and Github issues created for all defects found
[ ] Defects are created and reviewed after fixes are made
[ ] Approval that the code delivered works as designed

@wonilsuhibm @RobertaJHahn this is ready to be reviewed
https://github.com/carbon-design-system/ibm-dotcom-library/pull/1437

All design defects for this round have been opened. Closing this issue.


[Bug]: Appsmith become unresponsive with unoptimised postgres queries

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When a data source gets really slow and lots of queries on Appsmith are timing out, the api gets unresponsive eventually while health checks and supervisor states are fine.
Unresponsive data sources should not turn Appsmith into unusable state.

Steps To Reproduce
In our case, we have postgres queries that are not optimized and consumes high cpu.
The cpu is at 100% then queries starts timing out. Keep trying for about 5 minutes and then all other apis gets broken.
Public Sample App
No response
Version
Self Hosted - CE v1.7.8

@Nikhil-Nandagopal just some observation that might help with this. The v1.9.25-26 still got this problem and we also noticed that upgrading to v1.9.44 made this problem worse.


Refactor SlowRenderingDetectorImpl so that it's thread-safe

Fixes #359
Removed the FrameMetricsAggregator, because it kept internal state that was not subject to any sort of synchronization whatsoever, and was accessed from multiple threads in an unsafe way. I refactored (simplified perhaps?) SlowRenderingDetectorImpl so that it uses FrameMetrics directly -- which, what's interesting, will allow us to collect per-activity metrics in the future, if we wish to do so.

Removed the FrameMetricsAggregator, because it kept internal state that was not subject to any sort of synchronization whatsoever, and was accessed from multiple threads in an unsafe way.

This is interesting because every usage of the frameMetrics instance within SlowRenderingDetectorImpl was definitely guarded by the lock, so I'm curious about where you observed unsafe accesses? Maybe you were thinking about the lack of coordination between the internal synchronization and our usage of it?

(but I would still like to better understand where the original thread safety concern came from).

FrameMetricsAggregator internally started a thread (a HandlerThread for running the frame metrics listener) that modified the SparseIntArray[] metrics field without any sort of synchronization. So, while we were synchronizing all the reads in our own, the writes happening in the core library were completely unguarded.


Table level timeout implementation

Introduced 3 levels of query timeout, where the first available one will be used:

Query-level: Passed under the query options with key 'timeoutMs'
Table-level: Configured under the QueryConfig inside TableConfig
Instance-level: Configured in the instance config, or default

The timeout will be picked up on the broker side, and passed to the server under the query options.
If the query already timed out before broker sending the request to the servers, it will early terminate with BROKER_TIMEOUT_ERROR.
If the query timed out on the server side, broker should already returned with the partial response.
This PR also supports refreshing routing properties on table config change automatically.
Because the table config refresh is implemented using the Helix USER_DEFINE_MSG, both controller and broker need to be deployed in order to support it.
No specific deployment sequence is required (warning will be logged in BrokerUserDefinedMessageHandlerFactory when only one component is deployed).

Codecov Report

Merging #5169 into master will decrease coverage by 0.17%.
The diff coverage is 77.77%.

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

@jackjlli @mcvsubbu Addressed all the comments. I'll use null to denote that the timeout is not configured everywhere for readability.


Added parents recursive endpoint

Added /r/parents/<INSCRIPTION_ID> and /r/parents/<INSCRIPTION_ID>/<PAGE> endpoints. This should take care of https://github.com/ordinals/ord/issues/3309.

@gmart7t2  would it be worth making this endpoint like your doing with the new children endpoint returning Vec<InscriptionRecursive> instead of Vec<Ids>? https://github.com/ordinals/ord/pull/3761

@raphjaph I've made the change you requested.


is it possible to define new customized filed in log format

my tomcat log format is
pattern="%a %{jira.request.id}r %{jira.request.username}r %t "%m %U%q %H" %s %b %D "%{Referer}i" "%{User-Agent}i" "%{jira.request.assession.id}r"
is it possible to show the  username  in the goaccess result ,  I could only ignore it by ^% so far,
But it's really necessary to get the statistics of username hints.

%e should display the authenticated user.

Closing this. Feel free to reopen it if needed.


[WebUI] Refactor speak button to its own Comopnent

displays different button images (including spinner) depending on
the TTS state

@antonkast-google Please review this PR. Thanks.


rfc: versioned kv

Signed-off-by: fredchenbj<EMAIL_ADDRESS>

@overvenus PTAL

Left a few questions, the overall LGTM, well done!

What does it mean to accept/merge this RFC? If we are to merge the RFC and implementation, would it be easy to remove the implementation in the future if it turns out to have too high a maintenance burden?
One issue I have with this design is that it is a very specific mechanism and does not scale well. If we want to support N data models and APIs, we have to add O(N) amount of code to TiKV and we introduce O(N^2) possible interactions between interfaces. I would much prefer a design that let us support any data model and API, e.g., by adding a new client.
I'm also a bit wary of adding such a big feature without analysing the benefits and costs more closely. Is this likely to bring value to existing users? Bring new users? How many? What potential future feature would this RFC make more difficult to implement? How much code is likely to change when implementing, etc.?

What does it mean to accept/merge this RFC? If we are to merge the RFC and implementation, would it be easy to remove the implementation in the future if it turns out to have too high a maintenance burden?
One issue I have with this design is that it is a very specific mechanism and does not scale well. If we want to support N data models and APIs, we have to add O(N) amount of code to TiKV and we introduce O(N^2) possible interactions between interfaces. I would much prefer a design that let us support any data model and API, e.g., by adding a new client.
I'm also a bit wary of adding such a big feature without analysing the benefits and costs more closely. Is this likely to bring value to existing users? Bring new users? How many? What potential future feature would this RFC make more difficult to implement? How much code is likely to change when implementing, etc.?

@sunxiaoguang PTAL. Could you explain a little about how VerKV would be a useful  application case for Zhihu?

What does it mean to accept/merge this RFC? If we are to merge the RFC and implementation, would it be easy to remove the implementation in the future if it turns out to have too high a maintenance burden?
One issue I have with this design is that it is a very specific mechanism and does not scale well. If we want to support N data models and APIs, we have to add O(N) amount of code to TiKV and we introduce O(N^2) possible interactions between interfaces. I would much prefer a design that let us support any data model and API, e.g., by adding a new client.
I'm also a bit wary of adding such a big feature without analysing the benefits and costs more closely. Is this likely to bring value to existing users? Bring new users? How many? What potential future feature would this RFC make more difficult to implement? How much code is likely to change when implementing, etc.?

@sunxiaoguang PTAL. Could you explain a little about how VerKV would be a useful application case for Zhihu?

I would say it could be think of as a brand new enhanced version of RawKV that fixes some legacy issues that couldn't be fixed easily. Since the abstraction can fully cover what RawKV does right now, with acceptable extra costs. We can probably  deprecate RawKV in the future and only support Versioned KV instead.

In our business situation, the RawKV had met some problems like increment backup, change data capture, ttl etc. Those problems couldn't be solved well without the meaning of timestamp or version. So we would like to use VerKV to replace RawKV in the future.  Now this rfc is not complete on some aspect for my limited capability, so everyone is welcome to give help or advices.

PTAL @overvenus

This RFC has been also a proposal for GSoC, and someone seems interested in this project. https://github.com/tikv/tikv/issues/6508

What does it mean to accept/merge this RFC? If we are to merge the RFC and implementation, would it be easy to remove the implementation in the future if it turns out to have too high a maintenance burden?
One issue I have with this design is that it is a very specific mechanism and does not scale well. If we want to support N data models and APIs, we have to add O(N) amount of code to TiKV and we introduce O(N^2) possible interactions between interfaces. I would much prefer a design that let us support any data model and API, e.g., by adding a new client.
I'm also a bit wary of adding such a big feature without analysing the benefits and costs more closely. Is this likely to bring value to existing users? Bring new users? How many? What potential future feature would this RFC make more difficult to implement? How much code is likely to change when implementing, etc.?

@sunxiaoguang PTAL. Could you explain a little about how VerKV would be a useful application case for Zhihu?

I would say it could be thought of as a brand new enhanced version of RawKV that fixes some legacy issues which couldn't be otherwise fixed easily. Since the abstraction can fully cover what RawKV does right now, with acceptable extra costs. We can probably deprecate RawKV in the future and only support Versioned KV instead.

@nrc PTAL again

LGTM


added reduce and eq instance

Sorry I didn't notice your PR in time and made some changes. Can you rebase this?
Also, I think reduce should be private and we could reduce the value in %. What do you think?

Sounds good, just made those changes. How is it now?

I have some quickcheck tests here locally using your Eq instance. The tests fail after you added the reduce calls to the instances, not exactly sure why. Anyways, let's remove the reduce calls from the instances and from (%) since that calculation is redundant. I think it's enough if we only reduce in Eq.

Oh, and can you squash the commits?

@Thimoteus I cherry-picked your stuff, so closing this. Thank you for working on this. :+1:


docs: add secrets management (vault & doppler)

Fixes https://github.com/daytonaio/enterprise-docs/issues/147

Looks good to me, content-wise 👍


added section about using istanbul on Windows

As promised in #64

Thank you.

sorry @gotwarlost I made a mistake in the example. Could you merge my update? #494


How to specify the cell to change the background color of the table header by using jexcel's setStyle function?

Hi,
On the website, there are examples about changing the background color of a specific cell like A1, B2 etc. but I couldn't find an info about changing the styling of the table headers.
On the table, which you can find an attached image of it, what I want to do is to change the header's (Product Origin, Description etc) colors. How can I specify them like we do for the cells of A1 or B2?

Hi,
On onload property of JExcel you can defined that with this code
var instance = el.jexcel;
var x = 0; // column number 0 = first (Index)
instance.thead.firstElementChild.children[x].style.backgroundColor = "red";

Hi,
Thank you for your reply!
I am using jquery and I am able to find the cell I want using this:
var cell = $('table > thead').children().find("[data-x=0]");
and I'd like to use the jexcel's setStyle function like this:
table.setStyle('A1', 'background-color' , 'red');  but when I pass the "cell" variable to the function, it doesn't work.
so how should I pass the cell to this function?

Hi,
the function setStyle of JExcel is only for cell of tbody (not header), first parameter is an string not an DOMElement (i.e A1, B12, ...).
If you use jquery, you can use code of the first answer or this jquery code :
var table = jexcel(DOMElement, {
    onload: function(el, instance) {
           var x = 0; // column number 0 = first (Index)
           instance.thead.firstElementChild.children[x].style.backgroundColor = "red";

           // Or in JQUERY
           x = 1 // column A
           $(instance.thead).find("tr").first().children().eq(x).css("background-color", "blue");
        },
};

Hi @Guillaume-Bo,
It solved my problem! Thank you so much!


[DOI-1692] Exclude abandoned cart time option when mercadoshops

MercadoShops creates abandoned carts every 3 hours, so users shouldn't use the 2 hours configuration because that will never return an event

👍🏼


gusb: 0.2.11 → 0.3.0

Motivation for this change
Size impact: 6% overall increase (115 MB → 122 MB due to 7 MB hwinfo). But we also split the outputs now, reducing the size of out to 51 MB.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: gusb, hwdata
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on aarch64-linux (full log)
Attempted: gusb, hwdata
Partial log (click to expand)
--- stdout ---
/gusb/context:
--- stderr ---
**
libgusb:ERROR:../gusb/gusb-self-test.c:97:gusb_context_func: assertion failed (array->len > 0): (0 > 0)
-------

Full log written to /build/libgusb-0.3.0/build/meson-logs/testlog.txt
builder for '/nix/store/693c227k4rc3pnc9b6wzq3296w1l4fk5-gusb-0.3.0.drv' failed with exit code 1
error: build of '/nix/store/693c227k4rc3pnc9b6wzq3296w1l4fk5-gusb-0.3.0.drv' failed

Failure on x86_64-linux (full log)
Attempted: gusb, hwdata
Partial log (click to expand)
--- stdout ---
/gusb/context:
--- stderr ---
**
libgusb:ERROR:../gusb/gusb-self-test.c:97:gusb_context_func: assertion failed (array->len > 0): (0 > 0)
-------

Full log written to /build/libgusb-0.3.0/build/meson-logs/testlog.txt
builder for '/nix/store/1iizbpll1dj3vlvfq0q94qqm5b44dvks-gusb-0.3.0.drv' failed with exit code 1
error: build of '/nix/store/1iizbpll1dj3vlvfq0q94qqm5b44dvks-gusb-0.3.0.drv' failed

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: gusb, hwdata
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: gusb, hwdata
Partial log (click to expand)
checking for references to /build in /nix/store/dvxrsnhv3zmsws0z3xlw9vjg1g5q9hg9-gusb-0.3.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/a6g3i94gkkwpc6ypmkjwgp5ah5hkhv57-gusb-0.3.0-dev
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/a6g3i94gkkwpc6ypmkjwgp5ah5hkhv57-gusb-0.3.0-dev/lib
patching script interpreter paths in /nix/store/a6g3i94gkkwpc6ypmkjwgp5ah5hkhv57-gusb-0.3.0-dev
checking for references to /build in /nix/store/a6g3i94gkkwpc6ypmkjwgp5ah5hkhv57-gusb-0.3.0-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/2pb5lfni59pm9x65kvpxdphznfbcrpxx-gusb-0.3.0-devdoc
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/2pb5lfni59pm9x65kvpxdphznfbcrpxx-gusb-0.3.0-devdoc
checking for references to /build in /nix/store/2pb5lfni59pm9x65kvpxdphznfbcrpxx-gusb-0.3.0-devdoc...

Success on x86_64-linux (full log)
Attempted: gusb, hwdata
Partial log (click to expand)
checking for references to /build in /nix/store/67sj8iyk3f8zcvrn24n5fib0cb8s8f4l-gusb-0.3.0...
shrinking RPATHs of ELF executables and libraries in /nix/store/b4k186x9sfc45fjzjk0wplza7s5qzln4-gusb-0.3.0-dev
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/b4k186x9sfc45fjzjk0wplza7s5qzln4-gusb-0.3.0-dev/lib
patching script interpreter paths in /nix/store/b4k186x9sfc45fjzjk0wplza7s5qzln4-gusb-0.3.0-dev
checking for references to /build in /nix/store/b4k186x9sfc45fjzjk0wplza7s5qzln4-gusb-0.3.0-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/k74z4s427vi7wkbxwxb997gbm3pmv9k8-gusb-0.3.0-devdoc
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/k74z4s427vi7wkbxwxb997gbm3pmv9k8-gusb-0.3.0-devdoc
checking for references to /build in /nix/store/k74z4s427vi7wkbxwxb997gbm3pmv9k8-gusb-0.3.0-devdoc...


Why did you deprecate the LoadBalancerService and use ShardingContainerPoolBalancer?

I understand the ShardingContainerPoolBalancer use 'horizontal' sharding to not collide with fellow loadbalancers, but according to the following code snippet, you seem to not fully use warm containers.
You just return an invoker that has some slots on it, not an invoker that have some warm containers which are specialized to a particular function on it.
Do you have any plan to change this to fully use warm containers?
/**
   * Scans through all invokers and searches for an invoker tries to get a free slot on an invoker. If no slot can be
   * obtained, randomly picks a healthy invoker.
   *
   * @param invokers a list of available invokers to search in, including their state
   * @param dispatched semaphores for each invoker to give the slots away from
   * @param index the index to start from (initially should be the "homeInvoker"
   * @param step stable identifier of the entity to be scheduled
   * @return an invoker to schedule to or None of no invoker is available
   */
  @tailrec
  def schedule(invokers: IndexedSeq[InvokerHealth],
               dispatched: IndexedSeq[ForcableSemaphore],
               index: Int,
               step: Int,
               stepsDone: Int = 0)(implicit logging: Logging): Option[InstanceId] = {
    val numInvokers = invokers.size

    if (numInvokers > 0) {
      val invoker = invokers(index)
      // If the current invoker is healthy and we can get a slot
      if (invoker.status == Healthy && dispatched(invoker.id.toInt).tryAcquire()) {
        Some(invoker.id)
      } else {
        // If we've gone through all invokers
        if (stepsDone == numInvokers + 1) {
          val healthyInvokers = invokers.filter(_.status == Healthy)
          if (healthyInvokers.nonEmpty) {
            // Choose a healthy invoker randomly
            val random = healthyInvokers(ThreadLocalRandom.current().nextInt(healthyInvokers.size)).id
            dispatched(random.toInt).forceAcquire()
            logging.warn(this, s"system is overloaded. Chose invoker${random.toInt} by random assignment.")
            Some(random)
          } else {
            None
          }
        } else {
          val newIndex = (index + step) % numInvokers
          schedule(invokers, dispatched, newIndex, step, stepsDone + 1)
        }
      }
    } else {
      None
    }
  }

@markusthoemmes

@rabbah If you want an Action getting assigned a potential stem cell container instead of a warm container, why bother load balancer? Just simply put a queue in between Controllers and Invokers, an Invoker pull Actions from queue when there are free slots on it, but I think in this case, there are a lot more 'cold start' than which has load balancer.

The loadbalancing algorithm was/is the same in both loadbalancers. The "old" one used asynchronous state-sharing while the new one uses sharding.
The "old" loadbalancer was deprecated and removed because under load the state-sharing was far too slow and caused loadbalancing issues. Algorithm wise, both are the same.
Towards the algorithm: It is an algorithm that gives you statistical probability on warm starts. Of course we want to "fully use warm containers", that requires extending the state in the controllers + removing all scheduling logic from the invokers. We're not there yet.

You might want to participate in #3621.

Closing because of inactivity.


Fix Occluders to properly share resources

In order to properly support the resource sharing paradigm, Occluders are split into Instances and Resources in the VisualServer. Instances are owned by a Scenario, and Resources are global. OccluderShape resources can now correctly be shared by multiple OccluderInstances.
Fixes #58003
Notes

I realised when we were discussing resources for another PR that I previously wasn't handling these correctly as they needed to be able to be able to handle being shared between more than one Occluder instance.
A fair amount of housework to get this working to split the local space data (resources) from the world space instance data, but the actual implementation etc is unchanged.
The resources are held as a member variable of VisualServerScene. This seemed like the most logical place to put them, as they are not held within a scenario, and other resources are held within VisualServerScene.

Thanks!


Zowe Training and Digital Badge - ZLC discussion - request to OMP

3rd party has been offering free Zowe education and badge -  work with the OMP has stalled and education is being withdrawn. The education is to the benefit of the community when many people are hungry for education. Request discussion to get facts on the request to OMP and status plus how we can help.

Maybe we could setup a call to chat on this? ( noticed you kept this very vague so perhaps you'd rather handle offline ). Also this sounds like a potential vendor product question which is outside the scope of the ZLC IIRC.

Sure - the call is the ZLC next week as I do think the zlc need to be
understand - you should already be aware of the situation - involves
Interskill Learning -
Bruce Armstrong
IBM System Z Offering Manager- zowe.org
4205 S MIAMI BLVD, DURHAM NC 27703-9141
Email<EMAIL_ADDRESS>Tel:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
From:   John Mertic<EMAIL_ADDRESS>To:     zowe/zlc<EMAIL_ADDRESS>Cc:     armstro<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Date:   10/02/2019 09:06 PM
Subject:        [EXTERNAL] Re: [zowe/zlc] Zowe Training and Digital Badge

ZLC discussion - request to OMP  (#149)

Maybe we could setup a call to chat on this? ( noticed you kept this very
vague so perhaps you'd rather handle offline ). Also this sounds like a
potential vendor product question which is outside the scope of the ZLC
IIRC.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm told issue resolved - thanks John - no need for ZLC discussion
https://www.youracclaim.com/org/ibm/badge/interskill-basics-for-zowe


GKE A100 slow inference

I'm trying to run in GKE on an A100 exllamav2, but I'm having trouble getting it to warm up. I'm currently using the old stream generator+speculative decoding with my own modifications and my own server on fastapi+uvicorn. I've done a few different warmup options:

I send a prompt of length 4096-256, ask to generate 256 tokens.
I send a prompt of length 1 token, ask to generate 4000 tokens.
I send a prompt of length 40 in increments of 1 to 4000: 1, 41, 81, etc, 4001.
The problem is that on GKE after this warmup, when I submit a prompt of length 1000, 128 tokens are generated 3-4 seconds for the first request and about 1.5 for all subsequent requests. After that I submit a prompt of length 2000 and the situation is the same - the first generation takes about 7 seconds, subsequent ones about 2. When I run the same thing via docker compose - everything works fine and there is no problem with the first requests being so slow after warmup.

Do you have any suggestions what this problem could be related to?

I'm sorry, I don't really know much about GKE. From what you describe I'd have to assume there's some kind of lazy initialization going on on Google's end?

Hi, I have found the reason for this behavior. The problem is in the embeddings layer, which is on the CPU and does not move to the gpu at startup. When I send some prompt, it warms up (probably puts from CPU to GPU) only those input_ids that are in the prompt, not the whole dictionary. So prompt "hello"*1000 does not warm up prompt "bye"*1000. For some reason, this problem exists only in GKE. Everything works fine with docker compose. Probably need to add something to move embeddings to GPU at startup.
I fixed the problem for myself with the help of:
def embeddings_warmup():
    prompt_length = 1000
    for j in range(model.config.vocab_size // prompt_length + 1):
        ids = [
            k
            for k in range(j * prompt_length, (j + 1) * prompt_length, 1)
            if k < model.config.vocab_size
        ]
        if ids:
            model.forward(torch.LongTensor(ids).view((1, -1)))

I use exllamav2 with my modifications, so I don't have the latest version of exllamav2. Maybe in the new version this has been fixed, however I have not checked.


Reasons for the poor effect of Speculative Sampling

I tested the Speculative Sampling method with llama2-7b and llama2-70b on the a800, but their boost effect was almost zero and negative in most cases.
llama2-7b   base  103.25 tokens/s
llama2-7b   Speculative Sampling  104.52 tokens/s
llama2-70b   base   14.55tokens/s
llama2-70b   Speculative Sampling  13.41 tokens/s

Can you print out and check if the aggregate_metrics['accept_counts'] makes sense? accept_counts means how many token prediction from the draft model has been accepted by the verifier model. If it's too low, you can't get too much performance boost from speculative sampling.


Add support for route based code splitting w/ lazy loading

Maybe using vaddin
https://github.com/vaadin/vaadin-router/
Maybe could help out with this?

Interesting link
https://twitter.com/brianleroux/status/1004456650198036480?s=11

I tried lit-redux-router and was able to successfully build a basic routing example. If you want, I'll submit it as a new repo and write a tutorial in the wiki.

Just added a lazy load route package to evergreen-web-components

PR for lazy loaded routes added to lit-redux-router
Suggest we incorporate lit-redux-router by default.


[Add] Provider example app

In this PR I added an example app to showcase Beamer in combination with Provider. I also updated the README file.
As explained in https://github.com/slovnicki/beamer/pull/243 I think it's better to separate the provider example app and the location builder example app.

Thanks @devj3ns!


[USERSNAP] Cannot vote no on https://gov.tools/connected/governance_actions/15f82a365bdee483a4b03873a40d3829cc88c048ff3703e11bd01dd9e035c916#0.  Console says missing id and the yoroi window just hangs.

Taken at: https://gov.tools/connected/governance_actions/15f82a365bdee483a4b03873a40d3829cc88c048ff3703e11bd01dd9e035c916#0
Fields:

Add a comment: Cannot vote no on https://gov.tools/connected/governance_actions/15f82a365bdee483a4b03873a40d3829cc88c048ff3703e11bd01dd9e035c916#0.  Console says missing id and the yoroi window just hangs.

Browser: Chrome 128 (Linux)
Screen size: 2560x1440
Open #130 in Usersnap
Powered by Usersnap.

Forwarded to Yoroi team

Duplicate issue https://github.com/IntersectMBO/govtool/issues/2069


feat: well-founded or transitive relations don't have cycles

Match https://github.com/leanprover-community/mathlib/pull/18512

<EMAIL_ADDRESS><EMAIL_ADDRESS>

bors r-
bors merge

bors merge


[Media] WebRTC, Fx 72 tasks

Trello
https://trello.com/c/ejryeYyB/73-webrtc-10-fx-72
Acceptance criteria

[ ] Stop supporting "rid=" in simulcast attributes once ESR doesn't serialize it anymore

[ ] Add rtpTimestamp support to RTP synchronization and contributing sources

#1 should be trivial, since it's just "The protocol used for simulcast changed some time ago and we no longer support the old format, so communication with old browsers may not work." Likely just a Firefox for developers mention. Add a little time to double-check that there aren't places this should be mentioned (there are some possibilities in the articles that dive into SDP a bit), and I guess about half a day.
#2 is the addition of one property to each of two or three interfaces. It's a simple property, and the whole thing should take no more than half a day.


tlmgr install doesn't work anymore (older TeX Live version)

Hey,
there is the same Problem  (#15) as last year :)
See:
tlmgr: Local TeX Live (2021) is older than remote repository (2022).
Cross release updates are only supported with
  update-tlmgr-latest(.sh/.exe) --update
See https://tug.org/texlive/upgrade.html for details.

Quick fix
Run for qmcgaw/latexdevcontainer:latest:
docker buildx build -t qmcgaw/latexdevcontainer --build-arg TEXLIVE_VERSION=2022 https://github.com/qdm12/latexdevcontainer.git

Run for qmcgaw/latexdevcontainer:latest-full:
docker buildx build -t qmcgaw/latexdevcontainer:latest-full --build-arg TEXLIVE_VERSION=2022 --build-arg SCHEME=scheme-full https://github.com/qdm12/latexdevcontainer.git

duplicate of #29 ^^


Update _index.md

Fixed typos

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
e65615530505416ba98003248f44c0d25a772390

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/63dfc24603c2c200083a89ef


"scipy>=1.5.4,<1.6",

can you update the setup file to support newer version of scipy and numpy? currently there is a problem to install it on py38 and only working with py37.
py38 is necessary run with apple silicone.

Hi I updated the requirement to just using the latest. You can also just specify the version you want when you install.

Hello, @gaotianyu1350
still when installing using pip (PyPi) the requirement for scipy>=1.5.4,<1.6 is there
Shon


find_pip_from_context failing on Windows platform

The recent changes to the rez pip module have broken pip installs on Windows,
https://github.com/nerdvegas/rez/blob/6db191f4aa01903e22955cf38e8342cb8697a499/src/rez/pip.py#L162
When doing a context.which call to find the python executable the major version token is being appended to 'python' which on Windows returns None.
This causes an issue with the subsequent subprocess call which attempts to find a pip install.

Hi @theceebee , thanks for the report. I'm the one who changed this part of the code. I'm being told by @instinct-vfx that the executable with a version in its name on Windows only exists in Python 3. :neutral_face: Someone (me or @instinct-vfx probably) will create a PR shortly to fix this.

Fantastic. I'll keep an eye out.

@theceebee If you want to test, you can test with #798 . Just checkout and install and see how it goes.


Push Official Container Image on Release

Could an official container image be pushed to a registry on release? We would like to use a nerdctl image for cicd but do not want to have to build and manage it ourselves.

Now available: ghcr.io/containerd/nerdctl
https://github.com/containerd/nerdctl/pkgs/container/nerdctl


onRender, afterRender not working as I thought it would...

Hi,
thanks so much for this code, I don't really need the UI of the cart itself, so I'm trying to use the custom events to update divs that I want to use to display specific cart data. For example, within the header I have a div where I want to display the cart's subtotal. This works great with 'afterAddToCart' and 'afterRemoveFromCart', but when the user moves onto another page, I would assume that the following code would repopulate that div for me
afterRender: function () {
//document.getElementById("cart").innerHTML = "$"+this.calculateSubtotal();
alert("onRender: " + this.products.length + " product(s), total of " + this.calculateSubtotal() + "!");
},
I have commented out the first line because I'm not getting any results, so I'm using an alert window to tell me what I have. No matter which function I use (onRender, afterRender, onShow, afterShow) the alert window always tells me the cart total is $0.00, then the cart loads on the page and displays the correct amount.
Am I doing something wrong?
http://physicalviral.com/_dev2/

That's actually a creative usage of the events!
You're seeing unexpected results, because the afterRender event is called after the initial cart is rendered. Directly after that the data is read and the cart products are rendered. I may want to consider putting some events around that as well, but given your usage I would agree that the afterRender method is executed prematurely and should be moved.
I should be able to make that change fairly quickly, but let me see if there will be any other issues from it first.

Thanks!
I should probably know how to pull the data from the cookie storage, but I can't code javascript very well; I can only modify it with my limited knowledge. I might suggest that you update the readme file if you do include these events, as well as including 'onRemoveFromCart', 'afterRemoveFromCart', and any others i missed (that aren't already listed in the readme).
Best Regards,
Alyssa Davis
<EMAIL_ADDRESS>/ http://Alyssa-Davis.com

On Nov 9, 2011, at 9:59 PM, Jeff Harrell wrote:

That's actually a creative usage of the events!
You're seeing unexpected results, because the afterRender event is called after the initial cart is rendered. Directly after that the data is read and the cart products are rendered. I may want to consider putting some events around that as well, but given your usage I would agree that the afterRender method is executed prematurely and should be moved.
I should be able to make that change fairly quickly, but let me see if there will be any other issues from it first.

Reply to this email directly or view it on GitHub:
https://github.com/jeffharrell/MiniCart/issues/26#issuecomment-2691861

Hi Alyssa,
Sorry for the delay in getting this updated. The latest code should now fire the afterRender event after everything has rendered. You'll also see that I documented the remove from cart methods. Oddly, they were actually there and I just hadn't put them in the documentation ;)
Thanks for catching both of those and let me know if it solves your issue.


When document path is mapped against another mounted folder, writing of process.json gets out unhandled

Writing the process.json should be handled in a try... catch

:tada: This issue has been resolved in version 2.0.0-dev.1 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.1-dev.1 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


initialization of empty structs

Copied on request from https://github.com/zig-lang/zig/issues/548#issuecomment-338086782.
Zig requires initialization, even for empty structs:
const foo = struct {};

pub fn main() -> %void {

  var f : foo;
}

results in :

O:\zig0.1.1>zig build-exe b.zig
O:\zig0.1.1\b.zig:11:3: error: variables must be initialized
  var f : foo;
  ^

While empty struct does not make much of sense, asking for its initialization is IMO error.

You have 2 options for initializing a struct, whether it has 0 or more fields:
Option 1, recommended, list all the fields in the initializer. With a struct with no fields, there are 0 fields in the initializer. This option is recommended because if you add a field later, you'll get a compiler error here for not initializing all fields.
var f = foo {};

Option 2, leave all fields undefined:
var f: foo = undefined;


Added async search plugin security info

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

@kolchfa-aws: Since this also applies to previous versions of OpenSearch, we can go ahead and merge this in.


Bun can't build

Maybe it's me who's doing something wrong but I can't use Bun build command with this package installed.
Running bun build src/index.ts --outdir build
Gives me the following error:
1 | import { debuglog } from "node:util";
             ^
error: No matching export in "node:util" for import "debuglog"
    at PATH_TO_PROJECT\node_modules\elysia-ip\dist\services\debug.js:1:10
error: script "build" exited with code 1

Also, not entirely sure if this is a Bun issue and not an issue of this project! Sorry if that's the case!

What is your Bun version?
On Thu, 27 Jun, 2024, 15:58 Torwent, @.***> wrote:

Also, not entirely sure if this is a Bun issue and not an issue of this
project! Sorry if that's the case!
—
Reply to this email directly, view it on GitHub
https://github.com/gaurishhs/elysia-ip/issues/29#issuecomment-2194334347,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASZRZBZDZROETAWS26RWPUTZJPSK7AVCNFSM6AAAAABJ7UA2LGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJUGMZTIMZUG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

version 1.1.13


Can I use muxy on Fargate containers

How can I use muxy on Fargate containers?
Do I have to setup a packet dealy server?

Sorry I missed this. I've never tried it on Fargate, but I imagine you could. As long as you can run a docker container with this you could start it.
One challenge for network shaping is that I think you'd need the --all linux capability, which I'm fairly confident would not be available on Fargate, so I think that wouldn't work.


Collection pipeline is passing

At current the test pipeline is failing on an error:
[2020-08-06T14:53:06.214Z] UnicodeDecodeError: 'ascii' codec can't decode byte 0xe3 in position 25: ordinal not in range(128)

[2020-08-06T14:53:08.113Z] Service 'ansible' failed to build: The command '/bin/sh -c pip install pytest==2.9.2 testinfra ansible && mkdir -p /conjurinc/' returned a non-zero code: 1

Jenkins link
AC:

[ ] The build is updated so that it will pass

Filed in the wrong repo, this is a duplicate of https://github.com/cyberark/ansible-conjur-host-identity/issues/39


ChaosOrca: Adds more metrics and Grafana graphs

Adds visualization using Grafana.

Screenshot

Original plan was to use the csv+d3js to make some nice looking graphs. But Grafana enables an easier way.

Now has first version of support for start/stop before/after each experiment.

Now includes TTorrent container, more experiment data and some other smaller things.

Merged, thanks for the contribution!


[USE] UISC = Consonant_Initial_Postfixed should be mapped to CONS_FINAL

The Indic_Syllabic_Category value Consonant_Initial_Postfixed is not mapped to any USE category. It should be mapped to CONS_FINAL, as in harfbuzz/harfbuzz#1048.

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

ID: 194a6d3c-4137-46e9-3a4b-44b990200986
Version Independent ID: a0c8e788-5228-aa28-670e-3ba1ac3faecd
Content: Creating and supporting OpenType fonts for the Universal Shaping Engine - Typography
Content Source: typographydocs/script-development/use.md
Product: typography
GitHub Login: @Anglass
Microsoft Alias: anglass

Current HarfBuzz actually maps to CONS_MED, thanks to
https://github.com/harfbuzz/harfbuzz/pull/1714


XFA - Save filled data in the pdf when downloading the file

when binding (after parsing) we get a map between some template nodes and some data nodes;
so set user data in input handlers in using data node uids in the annotation storage;
to save the form, just put the value we have in the storage in the correct data nodes, serialize the xml as a string and then write the string at the end of the pdf using src/core/writer.js;
fix few bugs around data bindings.

/botio test

/botio makeref


"Tasks: Configure Default Build Task" incorrectly adds "type: " to the tasks.json "label" property

VSCode Version:
OS Version:
Version: 1.45.1 (user setup)
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-14T08:27:35.169Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041

Steps to Reproduce:

Install the C/C++ extension (repros with other extensions that provide tasks may also be possible) -- I don't believe the bug is with our extension.
You will need to have a C/C++ compiler installed on your machine, such as cl.exe, gcc, or clang (let me know if you need help getting that, e.g. sudo apt install gcc).
Open a new folder.
Add a new test.cpp file to activate the C/C++ extension.
Use the Tasks: Configure Default Build Task command. This triggers code in our extension that generates a task with a label that doesn't have the "shell: " on it. See the screenshot:

Select a task.

Bug: Notice "shell: " appears in the UI quickpick, when it shouldn't and the resulting tasks.json has a label with "shell: " in it when it shouldn't (or our extension doesn't want it).

Does this issue occur when all extensions are disabled?: N/A -- repro is with the C/C++ extension.

Oops, it's showing the "source: " and not the "type: ". I still think showing "source: " in the "label" is undesirable, but I'm closing this for now, since I believe we can workaround the main issue by changing the "source" to "C/C++" and keeping the "type" as "shell".


Bug 1909992: Allow private bundle images within private indexes

In #1878, the secrets passed in spec.secrets field of a catalogsource
were attached to the catsrc's corresponding SA that was used by the registry
pod, thereby having access to the secrets. This allowed for pulling of
private index images. However, the job that unpacks the bundle images did
not have access to the secrets, and as a result private bundle image included
in the catsrc were not installable using the secrets.
This PR attaches the secrets from the catsrc to the job that unpacks the bundles,
thereby allowing the inclusion of private bundle images within index from private
indexes.

Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/retest

/retest

/retest
we really need to fix up these flakes to get some meaningful signals

/retest
we really need to fix up these flakes to get some meaningful signals

/lgtm

/lgtm

/hold
looks like the console tests are consistently failing.

/hold
looks like the console tests are consistently failing.

Looks like the PR to fix the test was merged: https://github.com/openshift/console/pull/7835
/hold cancel
/retest

Looks like the PR to fix the test was merged: https://github.com/openshift/console/pull/7835
/hold cancel
/retest

Would love to see this backported to 0.17.