[6.0] Fix an infinite recursion in NameMatcher if a position to resolve is in the leading whitespace before a token

Explanation: If the name matcher was requested at a location inside whitespace in front of a token, it would try to parse the source file from there in order to match eg. a compound decl name inside the comment. However, if the location was inside leading whitespace, it would just find the token itself again and infinitely recurse.
Limit the re-parsing to only the trivia text, which fixes the infinite recursion
Scope: Name matcher
Risk: Low
Testing: Added test case
Issue: rdar://129922267
Reviewer:   @bnbarham on https://github.com/swiftlang/swift-syntax/pull/2708

@swift-ci Please test


Increase timeout to support ap-southeast

The client is timing out for request to ap-southeast. This increases the timeout to support those regions

I think we should promote the parameter itself no?
I'm not sure I follow, are you suggesting we allow configuring the http client timeout? I'm not sure we should make this configureable, it'll be confusing to have two timeouts (one timeout for waiting for the db to be ready, another timeout for http requests to Timescale).

Sorry I have misunderstood the discussion. would be cool to bump dependencies at the same time though!

ah yeah Ill have to look at the dependencies if they're breaking versions, lets bump in a separate PR


feat: setup GitHub workflows

Description

Setup GitHub workflows

Related Issues
None
Checklist

[x] Title starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] Updated/added relevant documentation and examples (doc comments with ///)
[x] Included unit tests for all new/updated/fixed behaviors.
[x] All existing and new tests are passing (melos run test).
[x] The analyzer does not report any problems (melos run analyze).
[x] Code is correctly formatted (melos run format).

Breaking Change

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Codecov Report
Base: 39.13% // Head: 39.13% // No change to project coverage :thumbsup:

Coverage data is based on head (44f0af7) compared to base (1b43a9e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   39.13%   39.13%
=======================================
  Files          10       10
  Lines          92       92
=======================================
  Hits           36       36
  Misses         56       56

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


issue #3621 (gksu has no install candidate when installing with xfce)

Please paste the output of the following command here: sudo edit-chroot -
Can't get kali-rolling to start even, nothing to start, chroot doesn't finish set up.

(the original issue #3621 )
when trying to install kali-rolling, it gets to installing xfce and manages to choke this out...
Building dependency tree
Reading state information... Done
The following packages were automatically installed and are no longer required:
  libasound2-dev:i386 libc6-dev:i386 libdrm-dev libpthread-stubs0-dev
  libspeex-dev:i386 libspeex1:i386 libspeexdsp-dev:i386 libx11-dev libxau-dev
  libxcb1-dev libxdmcp-dev libxext-dev libxfixes-dev libxi-dev
  linux-libc-dev:i386 x11proto-core-dev x11proto-fixes-dev x11proto-input-dev
  x11proto-kb-dev x11proto-xext-dev xorg-sgml-doctools xtrans-dev
Use 'apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package gksu is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'gksu' has no installation candidate
Failed to complete chroot setup.
Unmounting /media/removable/crouton/chroots/kali-rolling...

Exact same thing mentioned on stack exchange 3 days ago (as of Friday, January 26). I have a feeling this is more of a bug with how crouton sets up kali than a bug with kali itself. Does anyone know where the setup script is in the chroot? I can't find it.

Any updates on the "Package 'gksu' has no installation candidate
Failed to complete chroot setup" issue? I encountered the issue today trying to install rolling-kali.

I'm new to using crouton.
[start irrelevant details]--->
Made one, liked it--it worked! NEAT!, but prefer Debian (never really used Ubuntu--but wanted to see if touch support worked nicely with gnome).
<----[end irrelevant details]
Just tried:
sudo sh ~/Downloads/crouton -r buster -t gnome-desktop,chrome,core,extension,keyboard -n debian
got error:

E: Package 'gksu' has no installation candidate

more before:

Processing triggers for man-db (<IP_ADDRESS>-4) ...
Setting up guile-2.0-libs:amd64 (2.0.13+1-4+b1) ...
Setting up xbindkeys (1.8.6-1+b1) ...
Processing triggers for libc-bin (2.26-4) ...
Installing target x11...
Installing target gtk-extra...
ar: creating /tmp/crouton.NEmuEi/crouton-network-manager.deb
Selecting previously unselected package crouton-network-manager.
(Reading database ... 21041 files and directories currently installed.)
Preparing to unpack .../crouton-network-manager.deb ...
Unpacking crouton-network-manager (0) ...
Setting up crouton-network-manager (0) ...
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages were automatically installed and are no longer required:
libasound2-dev:i386 libc6-dev:i386 libdrm-dev libpthread-stubs0-dev libspeex-dev:i386 libspeex1:i386 libspeexdsp-dev:i386 libx11-dev libxau-dev libxcb1-dev libxdmcp-dev libxext-dev libxfixes-dev libxi-dev
linux-libc-dev:i386 x11proto-core-dev x11proto-fixes-dev x11proto-input-dev x11proto-kb-dev x11proto-xext-dev xorg-sgml-doctools xtrans-dev
Use 'apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package gksu is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Package 'gksu' has no installation candidate
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/debian...

@brannon,
A fix for this issue has been proprosed by @rickybrent, you can wait until his pull request is reviewed and merged or you can look at https://github.com/dnschneid/crouton/issues/3623#issuecomment-360983767 for a workaround.
Hope this helps,
-DennisLfromGA

I had the same problem with Debian Buster because of gtk-extra target. The patch/fork from @rickybrent seems to fix the issue.

@DennisLfromGA -- did you mean to @ me? ..(Don't see a brannon in this thread).
With issue being still open, I'm guessing it wasn't merged into release?
Also, since I'm a newb, it looks like crouton that I downloaded doesn't ever need updating, because it is a script that downloads latest--is this correct?

@brizzbane,

did you mean to @ me? ..(Don't see a brannon in this thread).

Yup, sorry, my response was directed to you.
When I typed @br I hit the tab key it grabbed the wrong dude, not my fault. ;-)

With issue being still open, I'm guessing it wasn't merged into release?

Correct, it's not been merged just yet, it's still being reviewed.

Also, since I'm a newb, it looks like crouton that I downloaded doesn't ever need updating, because it is a script that downloads latest--is this correct?

Correct again, the devs implemented an 'indirect installer' a while back, here are some comments from the script:
# This is an indirect script. The actual installer will be downloaded, cached,
# and run with any parameters provided. Cached installer will be retained for
# 30 minutes unless this downloader is newer than the timestamp of the cache.
# Set CROUTON_BRANCH to change the branch downloaded.

Hope this helps,
-DennisLfromGA

Yea im getting this error too, hope there will be a fix soon.

@Fataly-Insane,
For a temporary workaround see: https://github.com/dnschneid/crouton/issues/3623#issuecomment-360983767
Hope this helps,
-DennisLfromGA

@rickybrent 's fork worked for me (installing the Debian version w/gnome ..the command I listed above).
Also was happy to see that touch support worked in gnome!  (Without adding to command line...said it wasn't supported when I tried to do that way).

Merged @rickybrent's fix.


Enhancement: add policy to allow IAM user to read SSM /system_user/#user#/ path

Have a question? Please checkout our Slack Community or visit our Slack Archive.

Describe the Feature
AWS IAM user created when user_enabled is set to true should have access to the AWS SSM path '/system_user/$USER/*
Expected Behavior
One should be able to retrieve any parameters using the following command:
VALUE=$(aws ssm get-parameters --names  /system_user/$USER/$PARAMETER --with-decryption --query Parameters[0].Value | sed -e 's/^"//' -e 's/"$//' )

Or read credentials using the following commands:
AK=$(aws ssm get-parameters --names  /system_user/$USER/access_key_id --with-decryption --query Parameters[0].Value | sed -e 's/^"//' -e 's/"$//' )
SK=$(aws ssm get-parameters --names  /system_user/$USER/secret_access_key --with-decryption --query Parameters[0].Value | sed -e 's/^"//' -e 's/"$//' )

Use Case
Useful for example to implement credential rotation or read any other parameters stored under the path, such as environment variable definitions.
Describe Ideal Solution
Add an additional policy to the user created, allowing it to read the SSM path under which the credentials are created.
Alternatives Considered
Creating a specific SSM policy and attach it to the user, but it's overall automation gets more complex.
Additional Context
None

Is this already completed?
https://github.com/cloudposse/terraform-aws-s3-bucket/blob/424de84a0102fd792b03ba1383ee1eb452722edf/main.tf#L347
https://github.com/cloudposse/terraform-aws-s3-bucket/blob/424de84a0102fd792b03ba1383ee1eb452722edf/main.tf#L356-L357


getConstructor may return null

https://www.php.net/manual/en/reflectionclass.getconstructor.php
Discovered by @phpstan

Thanks!

You're welcome.

We'll move the project to https://github.com/amphp/injector, if you have any feature requests, please open an issue there. We'll add a couple of features that are currently unavailable.


hint for if blue tile doesn't let you spill soup on the painting

p: Come on this has so much blue I'm feeling seasick.
t: Ahoy me matey! Welcome to the treacherous waters of the Blue Exhibit, or are you a mere landlubber laid out to dry?
p: Why are pretending to be a pirate: this is various shades of blue.
t: Just wanted to polish up my Blackbeard impression, didn't know you were easily frustrated.
p: Well, I didn't know either until you were cracking quips every time I pause.
t: Just wanting to make sure you stay fit for whatever may come your way!
t: No fun having an unprepared rival-
p: Hey cool it with the remarks!
t: so I may give words of encouragement.
p: God I wish Lapis Lalooney can take a break long enough for me to stain this painting, he goes on and on about how he doesn't like the paintings because they are blue and instead because of their Jenny Sequoia-
t: I think you meant je ne sais quoi.
p: whatever, he is in such denial.
t: That is the tragedy of passions: they can take over you if you ain't careful.
t: Who knows, maybe he would, say, get easily offended if his precious blues got slandered.

Toasty is not inside the blue exhibit

I like the pirate angle if we want to do something with that

Toasty is not inside the blue exhibit

She is Posty's mindddd

p concerned "Come on this has so much blue I'm feeling seasick."
t enthused "Ahoy me matey! Welcome to the treacherous waters of the Blue Exhibit: can you thrive amidst the high ocean or will you be a mere landlubber treading water?"
p annoyed "Why are pretending to be a pirate: this is various shades of blue."
t smug "Who sayeth aye was pretending? I didn't know ye were of short temper."
p "Well, I didn't know either until you were cracking quips every time I pause."
t smug2 "It's no fun having to fight a weakling who doesn't know the scent of danger from booty!"
t "No fun having an unprepared rival-"
p angry "Hey cool it with the remarks!"
t crossedarms "so I may champion you with fighting words."
p "God I wish Lapis Lalooney can take a break long enough for me to stain this painting, he goes on and on about how he doesn't like the paintings because they are blue and instead because of their Jenny Sequoia-"
t smug2 "I think ye meant je ne sais quoi."
p annoyed "whatever, he is in such denial."
t "That is the tragedy of passions: they can taketh over you if ye ain't careful."
t "Who knoweth, maybe he would, say, become easily offended if his precious blues hath been slandered..."
t "A dead man tells no tales..."
p suspicious "Note to self: never talk about being seasick in the Blue Exhibit."
p "Toasty will bust out her terrible pirate accent."

assigned to AnArtichoke

https://docs.google.com/document/d/1WhRUQwlUWJ4uZUPKA1kMKDqoVJVSBf-UWJIuxX7-DIQ/edit


Broken Link for PSO

The link for PSO is broken in the ReadMe.md

PR for the Fix: https://github.com/txt/mase/pull/22


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 197 ms

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


https://marketplace.visualstudio.com rejects HTTP HEAD requests

Describe the bug
Dead link checkers and other tools may use efficient HTTP HEAD requests instead of GET requests.  This results in unnecessary load on https://marketplace.visualstudio.com, downstream effects are a UX and carbon emission impact.
To Reproduce
Steps to reproduce the behavior:

Use curl -head --verbose --fail https://marketplace.visualstudio.com
Response code is 404.

Expected behavior
Return HTTP status response code 200 for valid URLs.

Thank you for contacting Visual Studio Marketplace Team. We will get back to you with a response as quickly as possible.
-Regards,
VS Marketplace Team


Update HelloWorld.kt

removed comma from Hello, World!

Hello, I can merge but it would be considered as spam.


Add API versioning and refactor routes

The API should have a versioning number and drop the /api prefix. For example: http://api.teamboard.org/v2/.
The routes should follow the guidelines posted here in order to prevent too deep nesting and allow us to separate the massive routes/board.js file into multiple files.
For example:
routes/board.js
http://api.teamboard.org/v2/boards/
http://api.teamboard.org/v2/boards/:id
http://api.teamboard.org/v2/boards/:id/tickets
http://api.teamboard.org/v2/boards/:id/access
http://api.teamboard.org/v2/boards/:id/access/:code

Note that the access/:code route contains multiple parameters, but it is not an actual nested resource in the same way that tickets are.
routes/tickets.js
http://api.teamboard.org/v2/tickets/:id
http://api.teamboard.org/v2/tickets/:id/comments

:ok_hand: :eyes: :heavy_check_mark:

:love_letter:


WEBGL_draw_buffers extension missing in externs/browser/webgl.js

WEBGL_draw_buffers extension is ratified by Khronos, but missing in externs/browser/webgl.js.

Care to send a PR?

@Dominator008 See PR #1406.

Sorry didn't notice that PR was sent already.

Closed by #1406.


Create 11ty-theme-by-design-2-seo.json

Thank you!

Note that author needs to be a Twitter username (unfortunately!)


Detect Orientation from Peer Recording/Video Stream

RecordRTC is successfully saving the video streams as webm/video files. However, portrait videos from WebRTC Peers are rotated sideways. Some are rotated 90 degrees clockwise and others are rotated 90 degrees counter clockwise, based on the device used. My iPhone 6 Plus is always rotated 90 degrees counter clockwise, other devices are rotated 90 degrees clockwise, etc.
So a portrait video stream that is 640x480 when displayed ends up being recorded sideways at a 480x640 video.
Its a simple enough procedure to rotate the completed video the correct direction using FFMpeg, but the recorded file does not have the "rotate" tag in the metadata. So I am unable to determine which direction I should rotate my video.
I have been unable to detect the stream orientation from the stream, the video element or the completed recording.
Thank you!

I'm having the same issue. Is there any reliable way to detect the orientation from the peer? Perhaps the peer can broadcast over the data channel it's current orientation when recording starts? Using the Beta from ondeviceorientation events, if the Beta is high, the device is portrait, otherwise it is landscape.
I'm not sure if there is any other way in the tagging. Obviously the browser knows the rotation needed to show the remote stream correctly and is handling it internally...

i am having same trouble, any idea for this ?

I posted that particular stack overflow question. I never got an answer from the community. What I ended up doing was asking the peer what its camera orientation is and saving it along with the stream chunk. I was never pleased with this answer, but it was all I was ever able to come up with.

@daryl3 can I ask what you did to configure the size and orientation for the RecordRTC config? I can certainly use the RTC data channel to pass this along before starting to record, but don't know where to configure this for the recording.

@yogaboy I used the data channel to ask the remote connection what its device dimensions were. Then I created a javascript array of the information about each remote connection. Every 5 seconds, I dump a blob of the information into a database and store the information I have about the connection. Once the recording is done, I take all the blobs and re-assemble them into a single .webm file. Which I then convert to the format I want using FFMPeg. I use the information I stored about the connection in the database to re-orient video using FFMpeg.

Thanks @daryl3. Unfortunately, my process is a little different, and the source webm is already messed up if it's portrait mode on the device. RecordRTC takes the captures with portrait sizing, but only uses the landscape height for the frame, tiling the top of the video in the remaining vertical space.
See below:

We experienced this same issue with the video stream being all bad. It only happened on iOS video feeds. Android video feeds were working just fine. We tracked it down to being a problem with Google's WebRTC builds, that we were using in iOS. We build Google's WebRTC Manually. We found that version 68 was working correctly.  But versions 69-71 all had the problem your describing. I don't know about the Coco Pods build versions. We reverted to build 68 and plan to look into more of why it is happening later and submit it to Google.

@muaz-khan Any thoughts on how we can configure recording to force the correct orientation and aspect ratio? While we can get information from the RTC peer, I don't know where I would alter the config to ensure this corrected information is applied for the recording capture.

Thanks @daryl3. Unfortunately, my process is a little different, and the source webm is already messed up if it's portrait mode on the device. RecordRTC takes the captures with portrait sizing, but only uses the landscape height for the frame, tiling the top of the video in the remaining vertical space.
See below:

I got the same issue on Mac. But I found it is OK on Windows. It seems to be caused by a bug in the MacBook Chrome.

Thanks @daryl3. Unfortunately, my process is a little different, and the source webm is already messed up if it's portrait mode on the device. RecordRTC takes the captures with portrait sizing, but only uses the landscape height for the frame, tiling the top of the video in the remaining vertical space.
See below:

I got the same issue on Mac. But I found it OK on Windows. It seems to be caused by a bug in the MacBook Chrome.

FYI for anyone that stumbles upon this... it's to do with the SDP offer that gets sent to participants. Found an old fix on another site that seems to fix this (with a slight modification)
Part of the SDP offer is something like:
a=extmap:4 urn:3gpp:video-orientation
Conveniently, if you remove this line from the SDP offer in transition, it solves the overlapping video above as well as another issue I was having, that the recorded video was rotated despite neither occupant rotating their device.
Note that the extmap number (4 above) can change, so the fix for me was to basically loop through the numbers 1->20 and remove that line if it was there (could probably do this with regex instead).
Hope this helps someone in the future!

This is fantastic news. We are using Twilio's SDK so I will inform them of this and see if they can apply it to their framework.
Thank you so much!

I posted that particular stack overflow question


NETOBSERV-1517: remove IPFIX agent mode

Description

Remove controller code for OVS configmap / CNO configuration
Remove related tests (and move console plugin related tests to its dedicated file)
Remove FLP ingester reconciler (was only used with ipfix)
Note that the IPFIX api is not removed to not introduce breaking changes in existing API, however it is now documented that using IPFIX would have no effect

Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[x] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[x] Does this PR require a product release notes entry?

[x] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[x] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

@jpinsonneau

Since the https://github.com/netobserv/network-observability-operator/pull/577 can't we get rid of the old IPFIX section + type field in this version ?

That would be a breaking change which is forbidden in a version that has already been released. We need to create a new API version in order to remove fields (we should have done that before release v1beta2 the first time).

@OlivierCazade I'm actually removing the monolith reconciler in the other PR with unified agent+FLP (not pushed yet). So yes it's going to be even more simplified but via that other PR :)

/ok-to-test

/label qe-approved

thanks @Amoghrd !
/approve


Replace %orig for returns with the actual id

tateu suggested that I should replace my %orig with just the returns but I might replace some with the return value as it makes it easier to control when the function is called.

Closed in: https://github.com/wizages/CleanSheets/commit/9395e7c22964cb19323176747fa8587bd10b6379


Add missing indexes

Description
Add the missing indexes on all the tables
Related Issue

Fixes #1046
Motivation and Context

This should speed up the DB queries
How Has This Been Tested?

Yes
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@nateberkopec @schneems PR is ready.


Errors when trying to use grpc reflection and also graphql SDL source

Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[x] 1. The issue provides a reproduction available on
Github,

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
I am encountering errors when using the latest alpha version of the grpc handler.

When I set the endpoint to our real gRPC server (written in .NET), I get this error:

Failed to generate the schema TypeError: Invalid value used as weak map key
    at WeakMap.set (<anonymous>)
    at GrpcHandler.getServiceClient (/Users/mbrowne/_temp/_tmp/graphql-mesh-bug-repro/node_modules/@graphql-mesh/grpc/cjs/index.js:209:53)

(above error corresponds to this line: https://github.com/Urigo/graphql-mesh/blob/8faf4e4733d67ef6eac8140e9ec66529f0251499/packages/handlers/grpc/src/index.ts#L309)
Unfortunately it's difficult to create a reproduction for this error because it doesn't happen when using a JS-based gRPC server. I can try to upload a simplified repro later using golang (if not .NET).

When copying part of our protobuf over and referring to the protobuf source directly, I was able to successfully run yarn mesh:start. But when I tried to point to the generated graphql file as the source, I got the following error:

Failed to generate the schema Error: Object at path artnet.coredata.playground.Playground is not a Service constructor
    at GrpcHandler.getServiceClient (/Users/mbrowne/_temp/_tmp/graphql-mesh-bug-repro/node_modules/@graphql-mesh/grpc/cjs/index.js:215:23)
    at GrpcHandler.processDirectives (/Users/mbrowne/_temp/_tmp/graphql-mesh-bug-repro/node_modules/@graphql-mesh/grpc/cjs/index.js:264:53)
    at GrpcHandler.getMeshSource (/Users/mbrowne/_temp/_tmp/graphql-mesh-bug-repro/node_modules/@graphql-mesh/grpc/cjs/index.js:550:14)
    at async /Users/mbrowne/_temp/_tmp/graphql-mesh-bug-repro/node_modules/@graphql-mesh/runtime/cjs/get-mesh.js:133:28

To Reproduce Steps to reproduce the behavior:
git clone git@github.com:mbrowne/graphql-mesh-bug-repro.git --single-branch --branch grpc-source-issue
yarn

To see the "Object at path artnet.coredata.playground.Playground is not a Service constructor" error, simply run yarn mesh:start.
End-to-end reproduction:

Change the source in .meshrc.yaml to this:

source: proto/playground.proto

Run yarn mesh:start

Copy mesh/sources/gRPC Example/schema.graphql to graphql-schema/schema.graphql.

Edit .meshrc.yaml again:

source: graphql-schema/schema.graphql

Run yarn mesh:start

Expected behavior
Mesh server should start without errors
Environment:

OS: Mac OS 12.5
@graphql-mesh/grpc: 0.93.3-alpha-20230516104729-a4602ed66
NodeJS: 18

Additional context
See previous discussion at https://github.com/Urigo/graphql-mesh/pull/5420

I was able to reproduce the WeakMap error using a JS gRPC server. It seems that it happens when there's more than one package in the protobuf. I pushed the change to my branch (https://github.com/mbrowne/graphql-mesh-bug-repro/tree/grpc-source-issue).
Steps to reproduce:
yarn grpc:start
yarn mesh:start

FYI @ardatan

I added a failing test and was able to fix the WeakMap issue when generating a schema with multiple packages via reflection. I'm still having some issues with using the generated schema so my PR isn't quite ready yet, but this is what I have so far...
https://github.com/Urigo/graphql-mesh/compare/master...mbrowne:graphql-mesh:grpc-sdl-fixes?expand=1

I found that the issue with the generated schema was caused by what seems to be a bug in graphql-tools and its handling of repeated directives (it's including only one directive in the schema it returns instead of multiple), specifically the @graphql-tools/stitch package. Now I just need to find the time to implement a proper fix and raise a PR.

Please share a clear and minimal reproduction on CodeSandbox.

@ardatan it's possible that @Francismb is experiencing a slightly different issue, but just a reminder that there's already a reproduction here:
https://github.com/mbrowne/graphql-mesh-bug-repro/tree/grpc-source-issue
See above for steps to reproduce.
Also, I already debugged the issue and it's related to this method call:
https://github.com/ardatan/graphql-tools/blob/7ad15a083d0ca2e5a514791a07c1744ed1c7bd9a/packages/stitch/src/mergeCandidates.ts#L121-L122
mergeType expects Record<string, DirectiveDefinitionNode> but that's not what mergeObjectTypeCandidates is passing to it.

I believe our issue may be the same @mbrowne. I have created an example repo here @ardatan
https://github.com/Francismb/graphql-mesh-issue
Reproduction steps:

Clone repo
Run yarn
Run yarn build
Run Yarn start

You should see the following error:
~/Projects/graphql-mesh-issue (main*) » yarn start                                                                                francismalloch-boe@Franciss-MBP
yarn run v1.22.19
$ mesh start
💡 🕸  Mesh - Server Starting GraphQL Mesh...
💥 🕸  Mesh - api Failed to generate the schema Error: Object at path api.EventsService is not a Service constructor
    at GrpcHandler.getServiceClient (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/grpc/cjs/index.js:213:23)
    at GrpcHandler.processDirectives (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/grpc/cjs/index.js:278:53)
    at GrpcHandler.getMeshSource (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/grpc/cjs/index.js:546:14)
    at async /Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/runtime/cjs/get-mesh.js:127:28
    at async Promise.allSettled (index 0)
    at async getMesh (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/runtime/cjs/get-mesh.js:122:5)
    at async Object.handler (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/cli/cjs/index.js:206:13)
💥 🕸  Mesh - Server Error: Schemas couldn't be generated successfully. Check for the logs by running Mesh with DEBUG=1 environmental variable to get more verbose output.
    at getMesh (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/runtime/cjs/get-mesh.js:161:15)
    at async Object.handler (/Users/francismalloch-boe/Projects/graphql-mesh-issue/node_modules/@graphql-mesh/cli/cjs/index.js:206:13)
error Command failed with exit code 1.

Also facing this issue due to multiple packages in our proto definitions

Closing this issue for now because it seems different issues are mixed up here :) Please create a new issue with a minimal reproduction if you still have any.

I believe this is related: https://github.com/ardatan/graphql-mesh/issues/6424


Images get cut off

I am struggling to get this to work responsively on different device sizes!
I simply cannot get this to work on different device sizes without getting my images cut off. All I want it to do is to display the image in 100% width, and calculate the height automatically.
Edit: I finally found a solution using [backgroundSize]="'contain'", but it would be great if this could be added to the readme page. Especially the fact that is has to be 'contain', not just contain, i.e. with the extra '

Some info: because you are using Angular bindings, and the backgroundSize property is a string, you must use a string.
When using the brackets ("[]") you are telling Angular that this property is essentially a javascript variable. If you were to take off the brackets, you would not need the quotation marks, because now Angular will assume this property value is a string.


COMPOSE-158 1.5.0-beta03 merge. Fix Clickable.kt

Apply changes from jb-main between the base commit 6022301db806601f282c53b8cbb5a981923a1589 and the pre-merge commit 4ac3bda449d4606e5190208d074654ba041d2537:

https://github.com/JetBrains/compose-multiplatform-core/pull/257

Fix OnClick.skiko.kt

Move large code to a ClickableExt to avoid merge conflicts

Have to use composed for creating FocusRequester (as for MutableInteractionSource). Asked for solution here: https://kotlinlang.slack.com/archives/G010KHY484C/p1689181951198299

https://github.com/JetBrains/compose-multiplatform-core/pull/660 another approach


fix to/from-array builders

ping @Adirelle

Pong! I'll give it a try.

Almost. See https://github.com/prolic/fpp/commit/4427acd66799922ea3071465c3131cf521bacd4b.
\$this->$argumentName can be null so you cannot use it directly in foreach. I lazily fixed it using an (array) typecast but a null check could also work, though the indentation has to be changed.

Fixed it. Thank you.

thanks @Adirelle


SSH paths - is not a directory

With version 0.1.1 on darwin I see an error-message when trying to backup per SSH like bellow:
/usr/local/bin/ssbak save project@domain.tld:/home/project/public_html/0project/current ./test-01.ssbak
Error: project@domain.tld:/home/project/public_html/0project/current is not a directory

Same command with sspak "works" but hit size limitation. Above SSH paths contain symlinks but it behaves the the same with real paths. Is there something obvious I’m missing?

It currently does not support SSH storage, see the very last line in the README under "Limitations".

Ohh sorry - I've overlooked that. Thank you for this new implementation of an ol workinghorse! The idea of --ignore-resampled is something I had in mind for much to long time now. Inspired form the superb bigfork/silverstripe-recipe I started to use ssbak in tandem with php-deployer - deployer does the ssh part now. I can provide a snipped later on if anybody is interested, but testing with xyGB of assets takes time.

We've switched to using ssbak at work, in development, staging and production environments, and it's been working great for the last year or so (when we need it). The size limitation of sspak was a real issue/bug that caught us out a couple of times so it just made sense for me to rebuild it in something far more efficient than PHP.  I haven't had the need for SSH, so I haven't built it in, but it is something I potentially plan to add at some stage.
I'm glad you are able to use it though!


Dynamic adding a plugin

At first I'm aware about Package and plugin support. I'm wondering if I can use dart-pad as a live preview of my plugin I'm planning to publish soon.
It seems that this is not possible out of the box, but is there a trick I could use? (By side to put all my classes into the sample code)

DartPad doesn't support arbitrary packages right now, you can learn more about how package support works on this page: https://github.com/dart-lang/dart-pad/wiki/Package-and-plugin-support

I think they were asking about hosting a DartPad box on their own site, pre-loaded with their own package. So not adding to the official DartPad site or for all DartPad users, but it could be useful on sites like https://riverpod.dev/ to be able to add any, eg, Riverpod packages to the DartPads on their site for readers to play around with, without petitioning for official approval.

Right now, DartPad compiles user code on the server, which uses a Dart/Flutter project with a pubspec.yaml file with all of the supported packages listed in the dependencies section. So this issue sounds to me like it's asking for any package on pub.dev to be supported, but I might be missing something here.
There are already packages that have forked this repo and hosted their own version (For example, https://docs.hivedb.dev) , but we don't maintain instructions since it's a somewhat involved process.

Again, could be wrong, but I believe they mean by providing the source of the code itself, not just "any package on Pub". It can probably be implemented as "include this package's Dart files to be compiled, but hide it from the UI's main.dart and allow the user's code to import it". Importing this package can then be done using a pubspec dependency_override path dependency. There's the problem of what packages can that package import... probably keep it to any package already allowed on DartPad today.
If that's a security concern that makes sense, but in principle it's just like the user entering all that code by hand, or all the code being at the bottom of main.dart.

My intent was just to let the developer play with my plugin with setting up a new project locally. DartPad looks like perfect for that use case. On the other hand as I wrote initially in my feature request I am aware for the normal process and I don't think that it is necessary to become one of that exclusive packages.
Also the point from Levi was already in my mind I could put everything in my main.dart, but that feels like a very dirty hack which I would like to avoid. Right now I'm thinking that I compile simply my example app for web and embedded it via <iframe>.


[bug] can't ChatGPTCompleteCode for range, but can ChatGPTRun complete_code

I'm getting the same behavior. Is there a specific syntax that's supposed to be used for :ChatGPTCompleteCode command. I might be executing it improperly


Use less CPU time if idle

RoboViz is using 10% of my CPU when it is idle (not connected to a server). I wonder why. I would expect that it is trying to connect once a second or so, but does not do other things in the meantime.
Ubuntu 12.04

Sorry, duplicate of issue #17

You can close issues you created yourself (next to the comment button), even if you don't have push access.


Options for border color

Thank you for the great software.
If the border color is one color, it may be difficult to see depending on the window color.
I had set this.
yabairc
#!/usr/bin/env sh

~/dotfiles/script/yabai/border_color &

# etc ...

border_color
#!/bin/bash

a=("0xFFFFFF00" "0xFFFF00FF" "0xFF00FFFF" "0x00FFFFFF" "0xF0F0F0FF" "0xF0F0FFF0")

while true
do
  sleep 0.1
  yabai -m config active_window_border_color $(shuf -n1 -e "${a[@]}")
done

Can you add a feature?
yabai -m config active_window_border_color multiple
yabai -m config active_window_border_redisplay 0.1

how do I convert a Hex color to those numbers(hexdecimal?)?
Forx example bd93f9 = 0x........


TS-4821: Make hwloc a required dependency

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/727/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/623/ for details.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/730/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/626/ for details.

@PSUdaemon don't forget to update the docs when you land this ... there are sections that talk about dependencies and building from source.

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/882/ for details.

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/777/ for details.

@jpeach we should close this, right? You found a case where we can't expect hwloc?

@jpeach can you elaborate on which platform it was that didn't have hwloc? If not, we are going to try to get this in still.

I'll try to rebase this soon.

The problem I saw was with hwloc crashing on startup. This was on a Chinese VM hosting service and was caused by hwloc doing a divide by zero. It was unclear what version of hwloc the bug was fixed in. Needless to say this was a bit of a showstopper for running Traffic Server.

On Dec 16, 2016, at 9:38 AM, Phil Sorber<EMAIL_ADDRESS>wrote:
@jpeach can you elaborate on which platform it was that didn't have hwloc? If not, we are going to try to get this in still.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

What was the underlying OS?

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1271/ for details.

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/1166/ for details.

Ubuntu 12 IIRC. But it is a problem with the virtualized hardware not the OS itself.

On Dec 16, 2016, at 10:07 AM, Phil Sorber<EMAIL_ADDRESS>wrote:
What was the underlying OS?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@PSUdaemon What do you want to do here ? Merge it ?

@zwoop /shrug. I think I need help getting the autoconf stuff to work. It was very circular and hurt my brain.

[approve ci]

@PSUdaemon  Please fix CI issues with this PR.

[approve ci]

[approve ci]

Lets make this happen this week.

@PSUdaemon can we meet today at finish this?

@PSUdaemon
Do you still want to land this?  If so can you please get it ready to merge.

what is the point of making it required?
in case of virtual machines running on modern hardware where you don't have multiple sockets at all since you get many cores with a single socket it just introduces runtime overhead and additional points of possible bugs

This PR hasn't been updated in more then 2 months and I am closing out PRs that haven't had any updates.  Please feel free to reopen if want to update it.


postgres : only one schema in search_path is allowed

#2636

Codecov Report

Merging #2663 (ae99695) into main (dff9653) will increase coverage by 0.08%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##             main    #2663      +/-   ##
==========================================
+ Coverage   59.67%   59.75%   +0.08%
==========================================
  Files         138      138
  Lines       23482    23500      +18
==========================================
+ Hits        14012    14043      +31
+ Misses       7646     7635      -11
+ Partials     1824     1822       -2

Impacted Files
Coverage Δ

pkg/meta/sql.go
64.69% <77.77%> (+0.05%)
:arrow_up:

pkg/object/sql.go
55.04% <77.77%> (+4.04%)
:arrow_up:

pkg/vfs/writer.go
76.89% <0.00%> (+0.39%)
:arrow_up:

pkg/meta/tkv.go
75.35% <0.00%> (+0.45%)
:arrow_up:

pkg/fs/fs.go
76.57% <0.00%> (+0.84%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Connection enters .runningQuery state and never exits it.

Have adopted the library to do direct SQL queries. At first I used a single shared connection. It was fast, but after about 30 minutes of production running, I found it would enter a .runningQuery state and never exit, jamming the whole app.
Passing a timeout option did not help, though I suspect the timeout is for the PostgreSQL DB itself, not the library code.
I have since moved to creating a new connection on every request. This is much more costly, but it at least does work. However, I do see gradual memory rise over a day (from about 12MB to 48MB). It is usable, but not nice. I suspect this is the same problem: connections entering the .runningQuery state, and never exiting the state, meaning all the associated state is kept around.
I know I could use a pool to make things faster, but the bigger concern is just this jammed state for the connection. Has anyone else seen this? Unfortunately, it takes a while to show up, so it is rare-ish. Makes testing more difficult. Perhaps the connection itself should have some sort of timeout. It would at least error and then continue on properly.

Thanks for reporting. Do you have a recreate we could use?

No familiar with the term, I'm afraid. What is a "recreate"?

Sorry. A testcase or some example code that triggers the problem, so we can try and recreate it ourselves.

No, I'm afraid not. That is part of the problem. It only appears after quite a time running in production. Otherwise I could probably be more specific about what is going on.
I guess it would be quite difficult to investigate without a test. Perhaps it would show up if you connection was put under duress for a while.

@drewmccormack Can you give some details on the kind of database operations you are using and the table structures they are operating on?
What version of SwiftKuery and SwiftKueryPostgreSQL are you using?
In the interim I'll have a comb through the code and see if I can see anything that looks amiss.

From eyeballing the code I think we may be missing self.setState(.idle) at line 428 of PostgreSQLConnection.swift.

I took a look. The code I am using, release 2.1.1, looks quite different at that point in the code to what is in master. So I guess it could not be directly the line you mentioned, since I can't even find that catch, but it could be a problem in the code that has been updated already in master.

I'm afraid I am completely confused. If I look in my code (version 2.1.1), the func you reference at line 428 looks completely different to what is in the master. And yet, using blame, there don't seem to be any changes made to that code since January (version 2.1.1 was released in April).
I downloaded the zipped code directly from GitHub to confirm that my copy of 2.1.1 is the same as the tagged version. It is.
What am I missing.

Pretty sure 2.1.1 was tagged on a branch created from an earlier release as we didn't want to publish all the commits at that time. Some commits would have been cherry picked from master onto the branch before it was tagged.
When we make the next release the commit history will all line up again.
I'll have another look at the 2.1.1 version.

So looking at the 2.1.1 version I don't see anything obvious.....
Can you let me know the sort of operations you are running against the database so I can try to recreate the problem please?

There is really nothing in the SQL that would explain it, I don't think. It is very straightforward stuff. And the server will run for many thousands of requests before a problem arises. If a particular query were the problem, I would expect it to fail as soon as that query was run the first time, but I don't think that is the nature of the issue.

I will piece something together and put some load through it to see if I can get the problem to occur.

May be we should request @drewmccormack to have the problem reproduced with a branch that his increased logging / debug prints? 🤔


peer to peer review

Jansue

there needs to be a live feed of the whole project for viewers to get a complete experience of your project.

Trevor

your pull request has to  be more descriptive to show what you did exactly in your code .

Thank you. I will work on that


pysal version

Hi,
Thanks for sharing this great work. I have been trying to reproduce your code and it seems that the pysal version that you were using is outdated. The first notebook that it crashes on is on the notebook 02-Scoring Data Download and Prep.ipynb with the following error:
AttributeError: module 'pysal' has no attribute 'queen_from_shapefile'

By googling a bit I have found that this method is available on the version 1.11 of pysal. After downgrading it, it seems that there are other methods that were not available back then and it crashes.
Could you please specify the pysal version that you were using for this?
Thanks in advance,
Feliu

I've updated the repository to include an environment.yml file which is a full 'dump' of my Anaconda repository, including all versioning information: in this case I was using PySAL 1.14 and PySAL 2.x represents a major change that I've not had time to address. The README has been updated accordingly so hopefully this is enough to get you up and running.

Hi,
Thanks for your quick answer. I have tried to create a new environment with the new environment.yml that you uploaded but then the new environment is failing to be resolved. In concrete, the error I get is:
Solving environment: failed

ResolvePackageNotFound:
  - libgdal==2.2.4=4
  - libxml2==2.9.8=0
  - libiconv==1.15=0
  - libtiff==4.0.9=0

I wanted to try removing these and install them with pip install after removing them from the .yml, but then after I remove them and try to run conda-env create -f environment.yml, I get this error:
Solving environment: failed

UnsatisfiableError: The following specifications were found to be in conflict:
  - gdal==2.2.4=py36_0
  - jpeg==9b=2
  - json-c==0.12.1=0

I am running this on a MacOS Mojave version 10.14, with the python version that you specified. I don't know what else could I try out, perhaps I should create a VM and try this .yml on ubuntu? I don't know. Thanks again for your time and patience.
Best,
F.

Hmmm, I'm still running High Sierra at the moment (since I know that major upgrades always break something and don't have the time to debug right now) and I wonder if there aren't (yet) conda packages for Mojave and something is broken therein? That said, I've checked all of the packages that you listed and they all seem to have been installed from conda-forge --- can you try something like I'm doing this quickly from memory, so I may have the syntax wrong):
conda install -c conda-forge gdal=2.2.4 jpeg=9b json=0.12.1

That's just forcing conda to look in the conda-forge channel instead of the 'default' one... though the setup.yml and environment.yml files should have forced that I'd have thought.
Are there any other errors in the history for your installation?
jon

Hi,
I still cannot get this done. I don't know if you updated the requirements with this command but could you please freeze the environment that you are using into a file with the following command:
conda list --explicit
There doesn't seem to be any other errors that I can provide for further details. Could you try to create a new fresh environment setting PYTHONUSERSITE=1. (see this link https://github.com/conda/conda/issues/6018 for further explanations).
Thanks in advance,

I've attached the dump from conda list in the hopes that that will help. It's hard to tell from a quick scan of the explicit output whether I am getting pollution from other environments of the mlgent environment. Just to check my reading of the other link, is it the case that I need to add:
export PYTHONNOUSERSITE=True

to my .bashrc file, or is this something that goes into a YAML script (the syntax looks bash/environment-like)?
jon
conda_list_--explicit.txt

Hi Jon,
Thanks for the quick answer. You just need to execute the command before you activate the conda environment, I don't think you need to add it to the .bashrc, although you can. It'll be set to show only the ones in the environment you are using. This is to avoid package leakeages from your conda base environment.
For now I am just going package by package, looking at your explicit list trying to set manually the packages that you specified.
Did you do the PYTHONNOUSERSITE before the creating explicit? You can check it by running:
import sys

print(sys.path)

and check whether there is some package leakage.
I don't know what else could I try, I am stuck, if you have any other suggestion or idea I'd appreciate it.
Thank you very much

Hi,
I am still stuck in this.  The "solving environment step" from conda keeps failing. I have tried several times to setup a new environment.yml and try to solve the dependencies manually but new errors keep showing up and I can't manage to solve it (plus it's never a good idea to do this). I've also tried to use pipenv -c . but it also doesn't work. Anything else I could try? Thanks

Hi -- I've not had time to dig into this but am trying to do so now by setting up a clean environment and restarting my own install from scratch to see what's what.
I've also now had a chance to re-run the environment dump. Output of sys.path is:
['', '/anaconda3/envs/mlgent/lib/python36.zip', '/anaconda3/envs/mlgent/lib/python3.6', '/anaconda3/envs/mlgent/lib/python3.6/lib-dynload', '/anaconda3/envs/mlgent/lib/python3.6/site-packages', '/anaconda3/envs/mlgent/lib/python3.6/site-packages/cycler-0.10.0-py3.6.egg']

Output of explicit is attached.
Jon
conda_list_--explicit.txt

To set up the clean env I'm going to actually create completely new v-env with the user site set to true and fixing some of the key libs to the versions that I need (I'll try with as few as possible to begin with).

I've made some pretty substantial changes to setup.yml and have updated to Python 3.7. On my Mac this seems to enable me to install a complete environment from scratch without errors (except a note about clobbering a jupyter lib). I need to clone and re-run the entire thing from scratch to be sure this works though. I can, however, at least run the first two notebooks with no errors and no warnings.

I've now tested this on a brand new environment and had no issues. Can you please checkout either the HEAD or tag v1.1 and let me know if it works for you? Thanks! I've provisionally closed it but happy to reopen if you're still having issues.

Hi, sorry for the late reply, that is so great. Thank you so much for this! Everything works smoothly, just had to follow the instructions from the new setup.yml.
Cheers


Remove xformers as dependency

Feature request  / 功能建议
Having xformers as an integral part of the model infrastructure is effectively locking CogVLM to CUDA as xformers only supports CUDA.  How hard is it to remove xformers as a dependency?
Motivation / 动机
I would like to run CogVLM inference on other hardware platforms like Intel GPUs
In today's environment, it's important to have a choice of hardware and not be locked to a single vendor
Your contribution / 您的贡献
a

It's... a bit hard to do that, because different attention implementations may have slight deviation error. We have already tried to replace the attention with normal torch implementation, but the output of model will have error around 1e-1.


AreaDetector device is not robust to invalid setting (typo)

RE(mv(cam_dev.cam.acquire_time, 0)) produces an error state in the Prosilica ioc because 0 is invalid for the exposure time.  We were unable to change the exposure time through bsui.
Another set() call is still in progress
We changed the value in CSS and the ioc removed the alarm state from the PV.  HOwever, bsui (expectedly) still had a set() in progress for the PV.  Restarting bsui corrected the situation, but should this be more robustly handled, aside from a timeout?

Yes, but this is an ophyd bug.


Fixed path and also added debugging and error reporting.

WakaTime CLI errors in Unity was failing silently. I added error logging.

Glad to help.
Did my pull request merge? Seems it's only "approved"


ADCP on CTD frame

Add an option to lower an ADCP on the CTD frame?

Proposal to make a new instrument that can do this


npm-build-docs

[x] fix lint errors
[x] detect current version number
[x] do not document WebRTCMedia class methods as bare functions

This is very out of date


Integrate Content Safety API

Is your feature request related to a problem? Please describe.
Govern the user prompts prior to sending requests to Azure AI Search and Azure OpenAI.
Describe the solution you'd like
Add a switch to the deployment to enable Content Safety, which would deploy the service and integrate it into the front end so that it analyzes user prompts prior to sending the request to Azure AI Search and Azure OpenAI.
Describe alternatives you've considered
Content filter is helpful but not as detailed of a solution as Content Safety. Content Safety reduces the need for OpenAI to govern its response and potentially reduces overall cost as user submissions that violate Content Safety policy will never be processed by Azure AI Search or Azure OpenAI.
Additional context
None.

hi paullizer
AOAI content filtering is applicable to both prompt and completions

you can even add addiitonal content filtering on both prompt and completions

That is content filtering with Block list. Which is processed by Azure OpenAI.
Content Safety is a separate service with its own API call with more comprehensive governance. This call would occur before generating the ada embedding, the request to Azure AI Search, and Azure OpenAI.
https://learn.microsoft.com/en-us/azure/ai-services/content-safety/overview
Respectfully,
Paul Lizer
Principal Technical Specialist
571-508-6416
Book time with Paul @.***?anonymous&ep=pcard>

From: ArpitaisAn0maly @.>
Sent: Wednesday, February 21, 2024 6:06:13 PM
To: microsoft/PubSec-Info-Assistant @.>
Cc: Author @.>; Subscribed @.>
Subject: Re: [microsoft/PubSec-Info-Assistant] Integrate Content Safety API (Issue #506)
hi paullizerhttps://github.com/paullizer
AOAI content filtering is applicable to both prompt and completions
image.png (view on web)https://github.com/microsoft/PubSec-Info-Assistant/assets/55892521/aa1d44b9-7176-4fa4-98f7-a69b2cce85bb
you can even add addiitonal content filtering on both prompt and completions
image.png (view on web)https://github.com/microsoft/PubSec-Info-Assistant/assets/55892521/43e78106-1acd-4dae-b967-d269ad25a4d9
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PubSec-Info-Assistant/issues/506#issuecomment-1958262184 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIJTSVYBRXHC5UD3PAZGF4TYUZ4WLBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVE3DAMJWGUZDGNRWQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGE2DONJVHAZTOMVHORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you authored the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Fix testcontainer wait strategy

Current wait strategy is...
WaitingFor:   wait.ForListeningPort("3306/tcp")

This may not work well on some cases, especially with Github CI. Needs fix.

Below looks promising, as it is clealy stated on the official documentation as well.
WaitingFor:   wait.ForHealthCheck().WithPollInterval(time.Second),

Will have merge PR for this.. since it will also resolves initial CI issue.


[proto] Fixed RandAug and all AA consistency tests

In a follow-up PR, we can port consistency tests into Philip's framework

Still, do you think it would be easier to revert some of the changes in the prototype AA transforms to have the same random parameter generation?

yes, maybe. On the other hand I would not say that prototype logic is badly coded, I like how it was done.

I guess it is fine for now given that these are only consistency checks and we are going to remove them when we roll-out the API.

I agree with that, so we can keep those weird tests as we'll remove them later.


Refactor tests to reduce redundancy

There are tons of codes redundancy in filesystems and containers tests. We need to refactor the tests to reduce the redundancy. Since the aim of ChainerIO is to unify the APIs and behaviors for different filesystems and containers, the test should also be unified.

Closing as the goal of this task isn't clear and I think it's hard to define cleanly.


Fix agent import error

Description

self.agent_instance = getattr(self.module_agent, self.module_agent.__name)(args.config) throws exception for not able to find module, if the agent is HumanAgent, RosAgent, DummyAgent, or NpcAgent. Because the filename and class name does not match.
Generate class name from __name__
Potential problem that this requires the Agent and file name matches in the following way:
HumanAgent vs. human_agent.py
NpcAgent vs. npc_agent.py

CapitalizedAgent vs. capitalized_agent.py

Fixes #
Where has this been tested?

Platform(s):  Ubuntu 18.04.3 LTS
Python version(s): Python 2.7 and 3.5
Unreal Engine version(s): 4.22.3
CARLA version: 0.9.6

Possible Drawbacks

This change is

It's failing the pylint check with error "Redefining built-in 'name' (redefined-builtin)".  Any suggestions?

Hi,
thank you for this PR. You can simply add a single line before the throw occurs with
agent_class_name = self.module_agent.__name__.title().replace('_', '')
This will convert for example human_agent to HumanAgent.

Hi,
thank you for this PR. You can simply add a single line before the throw occurs with
agent_class_name = self.module_agent.__name__.title().replace('_', '')
This will convert for example human_agent to HumanAgent.

That'll work for now. I updated. But it requires the class name matches the file name in certain way.

Hi,
thank you for this PR. You can simply add a single line before the throw occurs with
agent_class_name = self.module_agent.__name__.title().replace('_', '')
This will convert for example human_agent to HumanAgent.

That'll work for now. I updated. But it requires the class name matches the file name in certain way.

The old challenge_evaluator has a replacement already (can be used from the scenario_runner directly) and for the new challenge we are about to create a new setup. Therefore, I think this workaround is fine.


XN-1493 temp change warp to released version

References

XN-1493

Summary
temporarily change to a released version of warp and therefore disable server side client authentication.

LGTM👍
maybe we should hide the docs of the tls_client_auth setting

good point, done.

LGTM👍
maybe we should hide the docs of the tls_client_auth setting

good point, done.


SQL Editor Issue

Even ever i press multiple enter, and then do backspace their are scroll bar coming on editor screen, its irritating after few occurrence.

Please check workaround in #3916


Updated the app to fix the issue with alternate urls

Updated the app to fix the issue with alternate URLS not being handled correctly. Updated messages to include new notice about maintenance period.
Type of change
issue with alternate URLS not being handled correctly

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

[x] Local Pre-commit Checks
[x] CI/CD Build

Checklist

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

@downiec Is this ready to be tested on esgf-dev1 ?


If you modified fields in a maintenance window, it should prompt you if you want to leave the page

For example, if I start capturing a new item, and I refresh or open another page inside the app, I may end up losing what I wrote.
Either the prompt could work,  or it could maintain the data in-memory.

I don't know how to add tags, or if I can do it.

Thanks for reporting it in. Unfortunately, this is not something we'll add support for in the near future. Will make a note of it!


Wrong log and data timestamp showing for the wrong timezone

$ sw_vers

ProductName:	Mac OS X
ProductVersion:	10.13.3
BuildVersion:	17D47

$ uname -srm

Darwin 17.4.0 x86_64

$ ./prometheus --version

prometheus, version 2.1.0 (branch: HEAD, revision: 85f23d82a045d103ea7f3c89a91fba4a93e6367a)
build user:       root@6e784304d3ff
build date:       20180119-12:07:34
go version:       go1.9.2

When the prometheus was started, it showed wrong log timestamp.

When i opened the prometheus graph webpage, it showed wrong time also.

I noticed the following words in the prometheus FAQ webpage.

To avoid any kind of timezone confusion, especially when the so-called daylight saving time is involved, we decided to exclusively use Unix time internally and UTC for display purposes in all components of Prometheus. A carefully done timezone selection could be introduced into the UI. Contributions are welcome. See issue #500 for the current state of this effort.
from https://prometheus.io/docs/introduction/faq/#can-i-change-the-timezone?-why-is-everything-in-utc?

No matter how can i change the timezone on my laptop, the log and the data timestamp is wrong still, is there something else setting about timezone for prometheus ?

As those docs say, Prometheus uses UTC and has no notion of timezones.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


how to disable ipv6

hello
how to disable ipv6 please i don't even have a ipv6 still postal is trying to send emails from ipv6
[worker.2:42127] [2018-01-18T11:56:56.869] INFO -- : [D9OY3K9N] Cannot connect to 2607:f8b0:400d:c01::1b:25 (gmail-smtp-in.l.google.com) (Errno::ENETUNREACH: Network is unreachable - connect(2) for "2607:f8b0:400d:c01::1b" port 25)

is not all about i have or not.
Emails are going as spam when i configure ipv6 :confused:


pin pydantic to v1.10.10 everywhere

Recent version of pydantic takes BaseSettings to a different package:
https://docs.pydantic.dev/latest/concepts/pydantic_settings/
This was an issue for me when I installed pydantic recently for the watcher cli. It will probably cause issues with other sections of the code, so I pinned it everywhere after searching the repo for pydantic[dotenv]. It was already pinned in the microservices base image.

This is because of release of pydantic 2.0, we should migrate.


Parameterize service URL in feed actions

Currently, the feed actions have a hardcoded URL for the feed service provider. This should be parameterized so that this URL can be set during deployment.

It looks like the deployment scripts that @jasonpet is working on will set a package_endpoint parameter that will be bound to the correct value when the feed actions are installed. From our perspective, we can count on this parameter being set to <host>:<port> for the deployed feed service.


Fixed faster-simple-line-detector

IT WORKSSS!! @ShaharMS!! It compiles correctly!!

Why is CI failing like that?

Btw, the whole thing getUnsafePixel does is being a bit faster by not doing null/bound checking. OutsideBounds error usually means a getUnsafePixel call gone wrong.

Oh, thanks

Also, why not just test yourself if you're getting the error? Testing on interp is the fastest thing to do for problems like this one

Also, why not just test yourself if you're getting the error? Testing on interp is the fastest thing to do for problems like this one

Because I’m at school :'(, and I’m using the tablet

Oh wow, nice :D

Sorry for the notifications, if you don’t want notifications from this pull request, click here:

Nah it's fine, I think it's better seeing the progress. Feel free to ask me anything:)

IT WORKS!


rename makeViews(inView view: UIView) to makeViews(in view: UIView)

[Current coverage][cc-pull] is 92.95%

Merging [#28][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #28   diff @@
==========================================
  Files            42         42
  Lines          2114       2114
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1965       1965
  Misses          149        149
  Partials          0          0

Powered by Codecov. Last updated by [10c6aa0...3ccb344][cc-compare]
[cc-base-branch]: https://codecov.io/gh/linkedin/LayoutKit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/linkedin/LayoutKit/compare/10c6aa05046079b7ac6e6db2f91ecdaf68d6e80f...3ccb34449aa81a2cb54d5be015e00176c6b84a7f?src=pr
[cc-pull]: https://codecov.io/gh/linkedin/LayoutKit/pull/28?src=pr

Looks good to me.

Looks good to me too, thanks.


:smiling_face_with_three_hearts: emoji is missing

https://emojipedia.org/smiling-face-with-hearts/

@vaibhavsagar where to find the code where we have to make changes and add emoji?

Perhaps in zulip/tools/setup/emoji?

ok
Please assign it to me. I want to work on it

You're welcome to work on it, but I don't think I have the necessary privileges to assign it to you.

ok Please assign it to me. I want to work on it

You can claim this by typing @zulipbot claim .

This is effectively a duplicate of #19371.  I hope we'll be able to address that soon.


Unusually high memory usage when remapping jar in multiprojects

I have a multiprojected forge mod, it has 6 subprojects, when I run gradlew build or use idea to build, it causes abnormal high memory usage, up to 5953M.
The more subprojects to remap, the more usage of memory, otherwise, if limit the memory by -Xmx3G, it will freeze on the remapping of the 4th project(SinoFeast), and then java shows a oom message(not a exception). Although I tried to limit max gradle workers, it helps nothing.
Is it a known issue? Or what should I do to solve this problem?
My Project https://github.com/SinoCraftProject/SinoSeries/tree/dev/update-1.19.4

I believe it's an upstream issue with Loom and Tiny Remapper, see

FabricMC/fabric-loom#772
FabricMC/fabric-loom#793
FabricMC/fabric-loom#796

Is there any solution now?


change required maven version

Do we need a pinnng to 3.2.1 in https://github.com/rhuss/docker-maven-plugin/blob/master/pom.xml
Or is it okay to go with 3.2.*?

that just means 3.2.1 is the minimum version of maven required for the plugin. anything newer then that is fine to use.

@computerlyrik does this answer your question ?
In fact I'm not really sure what 'advanced' Maven features we are using, have to check and document this, too.

Tnx. Works with 3.2.5 for me.
Perhaps s/o should file a feature request to name the maven tag more explicitly (minVersion or sth)


tag and make a release

Could you tag the repo and make a github release, so that we can load the package_manager rules remotely ?
Right now your python image for example has a build that assumes local package_manager rules.
as much as I understand bazel :(

You can just load external rules via a git commit hash instead of a tag.

yes I know and that's what I did, but I think there should be a tagged release.

Do you need anything in the release other than a git tag?
You might have been bitten by this too, but the auto-generated github release tarballs aren't generally safe to use via http_archive WORKSPACE rules, since the tar SHAs can change.

ok, the full story is that I was building a container and I wanted to try the new way to load debs.
I realized the rules where in the package_manager in this repo and not in the rules_docker repo. So I had to add:
git_repository(
    name = "distroless_rules",
    remote = "https://github.com/googlecloudplatform/distroless.git",
    commit = "4a20649aac319b2ee2f9c058c6a100784904ce7d"
)

Which made me look up the latest commit since there was no tag available.
I guess I don't need a release but only a tag.


Importing ORTModelForFeatureExtraction fails with ImportError: libcuda.so.1 on CPU

System Info
- Latest optimum for onnxruntime
- Everything else is a default on Google Colab

Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)

Google Colab
CPU runtime
This code:

!pip install --upgrade-strategy eager install optimum[onnxruntime]
from optimum.onnxruntime import ORTModelForFeatureExtraction

Will result in the following error:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/cupy/__init__.py](https://ukwaaqc7ito-496ff2e9c6d22116-0-colab.googleusercontent.com/outputframe.html?vrz=colab_20231219-060136_RC00_592175853#) in <module>
     16 try:
---> 17     from cupy import _core  # NOQA
     18 except ImportError as exc:

16 frames
ImportError: libcuda.so.1: cannot open shared object file: No such file or directory

The above exception was the direct cause of the following exception:

ImportError                               Traceback (most recent call last)
ImportError:
================================================================
Failed to import CuPy.

If you installed CuPy via wheels (cupy-cudaXXX or cupy-rocm-X-X), make sure that the package matches with the version of CUDA or ROCm installed.

On Linux, you may need to set LD_LIBRARY_PATH environment variable depending on how you installed CUDA/ROCm.
On Windows, try setting CUDA_PATH environment variable.

Check the Installation Guide for details:
  https://docs.cupy.dev/en/latest/install.html

Original error:
  ImportError: libcuda.so.1: cannot open shared object file: No such file or directory
================================================================

The above exception was the direct cause of the following exception:

RuntimeError                              Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/transformers/utils/import_utils.py](https://ukwaaqc7ito-496ff2e9c6d22116-0-colab.googleusercontent.com/outputframe.html?vrz=colab_20231219-060136_RC00_592175853#) in _get_module(self, module_name)
   1353             return importlib.import_module("." + module_name, self.__name__)
   1354         except Exception as e:
-> 1355             raise RuntimeError(
   1356                 f"Failed to import {self.__name__}.{module_name} because of the following error (look up to see its"
   1357                 f" traceback):\n{e}"

RuntimeError: Failed to import optimum.onnxruntime.modeling_ort because of the following error (look up to see its traceback):

================================================================
Failed to import CuPy.

If you installed CuPy via wheels (cupy-cudaXXX or cupy-rocm-X-X), make sure that the package matches with the version of CUDA or ROCm installed.

On Linux, you may need to set LD_LIBRARY_PATH environment variable depending on how you installed CUDA/ROCm.
On Windows, try setting CUDA_PATH environment variable.

Check the Installation Guide for details:
  https://docs.cupy.dev/en/latest/install.html

Original error:
  ImportError: libcuda.so.1: cannot open shared object file: No such file or directory
================================================================

If you choose GPU runtime, running the same exact 2 lines (install and import) works perfectly! It's like something inside optimum.onnxruntime requires CUDA even though the onnxruntime is for cpu and not onnxruntime-gpu.
PS: Link to a reproducible Colab: https://colab.research.google.com/drive/11jRoskoIf1g8AP6DEtzG1bcQqA7pjV-d?usp=sharing
Expected behavior
I cannot import ORTModelForFeatureExtraction from optimum.onnxruntime in Google Colab while I am on CPU Runtime. As soon as I switch to GPU runtime, it works!
You should be able to import ORTModelForFeatureExtraction from optimum.onnxruntime regardless of whether you are on CPU or GPU.

Thank you for the report @maziyarpanahi, this should be fixed on main. Currently in colab:
!rm -rf optimum
!git clone https://github.com/huggingface/optimum.git
!cd optimum && pip uninstall -y optimum && pip install .[onnxruntime]

&
from optimum.onnxruntime import ORTModelForFeatureExtraction

works for me.

Thanks @fxmarty for your PR, I can confirm installing from the main branch fixes the issue:

Many thanks again 👍🏼

No luck for me. :( Still getting this error. Copy pasted the commands as is.
ImportError                               Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/cupy/__init__.py](https://localhost:8080/#) in <module>
     16 try:
---> 17     from cupy import _core  # NOQA
     18 except ImportError as exc:

16 frames
ImportError: libcuda.so.1: cannot open shared object file: No such file or directory

The above exception was the direct cause of the following exception:

ImportError                               Traceback (most recent call last)
ImportError:
================================================================
Failed to import CuPy.

If you installed CuPy via wheels (cupy-cudaXXX or cupy-rocm-X-X), make sure that the package matches with the version of CUDA or ROCm installed.

On Linux, you may need to set LD_LIBRARY_PATH environment variable depending on how you installed CUDA/ROCm.
On Windows, try setting CUDA_PATH environment variable.

Check the Installation Guide for details:
  https://docs.cupy.dev/en/latest/install.html

Original error:
  ImportError: libcuda.so.1: cannot open shared object file: No such file or directory
================================================================

The above exception was the direct cause of the following exception:

RuntimeError                              Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/transformers/utils/import_utils.py](https://localhost:8080/#) in _get_module(self, module_name)
   1353             return importlib.import_module("." + module_name, self.__name__)
   1354         except Exception as e:
-> 1355             raise RuntimeError(
   1356                 f"Failed to import {self.__name__}.{module_name} because of the following error (look up to see its"
   1357                 f" traceback):\n{e}"

RuntimeError: Failed to import optimum.onnxruntime.modeling_ort because of the following error (look up to see its traceback):

================================================================
Failed to import CuPy.

If you installed CuPy via wheels (cupy-cudaXXX or cupy-rocm-X-X), make sure that the package matches with the version of CUDA or ROCm installed.

On Linux, you may need to set LD_LIBRARY_PATH environment variable depending on how you installed CUDA/ROCm.
On Windows, try setting CUDA_PATH environment variable.

Check the Installation Guide for details:
  https://docs.cupy.dev/en/latest/install.html

Original error:
  ImportError: libcuda.so.1: cannot open shared object file: No such file or directory

@regstuff Are you installing from source?

@fxmarty, this is what I did:
!rm -rf optimum
!git clone https://github.com/huggingface/optimum.git
!cd optimum && pip uninstall -y optimum && pip install .[onnxruntime]
from optimum.onnxruntime import ORTModelForFeatureExtraction

That's weird, I can not reproduce on a colab :/

If I run
!rm -rf optimum
!git clone https://github.com/huggingface/optimum.git
!cd optimum && pip uninstall -y optimum && pip install .[onnxruntime]
from optimum.onnxruntime import ORTModelForFeatureExtraction

In the top cell of any Colab it produces the error. Is there a proper way to resolve this so that I can use onnx on CPU with colab?

Same error here. I believe this issue should be re-opened.

@fxmarty This should be reopened - the error is still there - with the fresh runtime in Colab and installation from source - in CPU runtime of Colab the error is there


In destructuring Parenthesized Object|ArrayPattern LHS is illegal and shouldn't parse

see https://github.com/jshint/jshint/issues/2269 for full discussion

wow. This is an old issue! Not solved!?

@crazyhuggins we're always looking for more contributors to help solve these types of issues. It may seem crazy that an issue this old isn't solved but when you have an all volunteer project, maintained mostly by one person, issues effecting a smaller group of people tend to go unsolved while larger issues and new features tend to get the attention.
I'm sure you weren't volunteering to solve the issue, so I'm unsure what you were hoping to gain with your comment, but I hope I provided a bit of insight as to how an issue like this that doesn't effect very many people can go unsolved for so long.


docs: add signal proposal to reference and fix typo

Description

closes: #48
Tasks

[x] Add Reference docs structure
[x] Add first signal proposal
[x] Fix typo in /README.md

References

https://github.com/tendermint/farming/pull/148

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Appropriate labels applied
[ ] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

@kogisin I just copy docs from https://github.com/tendermint/farming/pull/148 and fix farming words to budget and remove signal words because there would be not only signal proposal but also upgrade proposals on the proposals/ directory.
I missed that when reviewing https://github.com/tendermint/farming/pull/148.


Animate.css integration

Going with the modular pattern of the animate-on-scroll practice, perhaps it would be a good idea to separate out the actual animation CSS, so it can be replaced with a dedicated animation library such as Animate.css.

I'm not sure if this is the way I'd like to evolve this library. Tough, possibilty of changing default animation stack with other popular animations library seems tempting. I'd be glad to see if it's something that other users will appreciate.

+1 for this feature.
Would be great to use animate.css as the animation library.

I am also for :+1:
After to see the perf :/

animate.css is already doing a great job. there is no need to reinvent the wheel. You should provide simple animations and also a simple way to use animate.css or other animate libraries as well.
Many sites have already animate.css loaded in their pages for some other use. This will make all the animations consistent within the page.

Yeah, I totally agree @ErtugKaya. I'm really considering this in future releases.
Right now I'm sort of rewriting AOS using ES6 and webpack and preparing version 2.0.0.
It is possible that in this major release there will be an integration with animate.css
Stay tuned :)

Not sure but is animate.css not quite large? Saw someone using wow.js + animate.css and using a single Animation from it. If you make it AOS work with animate.css please make it optional.
Or maybe I do not understand animate.css properly, any pointers? thanks!

@cthedot, please mind that you may remove unused styles from css automatically with purifycss, uncss or similar if you are goint to host it.
Alternatively, you can customize the build.
Otherwise many users would already have it cached if you load it from the CDN it is hosted, so indeed it won't be downloaded at all.

@cthedot don't worry. If there will be animate.css integration it will for sure be optional :)

I'm using aos animation library, its working fine, but i have some issues, please clear me. i have disable animations on mobile devices using this code
AOS.init({ 	easing: 'ease-out',				 	duration: 1200	, 	disable: window.innerWidth < 769			 });
disabling working fine. but on browser resize (from small to large window), how can i enable animations again.

@Northout-Salman Enabling again requires initializing AOS again. It's not possible to dynamically enable/disable AOS on runtime.

Hey Michalsnik, Thanks for the above info.

+1

+1

I just had some time to work on this. I have the following prototype working.
Let me know what do you think about it:

    <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/animate.css/3.5.2/animate.min.css">

So it's essentialy an additional useClassNames setting, that on scroll sets passed to data-aos classes, instead of aos-animate. It allows to use any animation library without the need to use any AOS related styles. But it's also worth to mention that you should in this case also add extra CSS to hide those elements before target classes are set, so it's not blinking :)

With extra initClassName and animatedClassName even cleaner:

    <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/animate.css/3.5.2/animate.min.css">

initClassName - class added right after AOS initialization
animatedClassName - class added after scrolling to trigger point
useClassNames - sets classes from data-aos attribute after scrolling to trigger point

animatedClassName and classes set in data-aos are merged, so it's easier to set-up libraries like Animate.css without having to duplicate animated on every element.

Been playing around with this and the only issue is the abrupt exit. If you exit to re engage the animation (so scroll up) the element goes straight to hidden. I've tried to hack around and get it to be a smooth exit transition but haven't been successful so far.
Here's a quick demo of the issue: http://recordit.co/toGQRvActE (notice the form just goes straight to hidden)
It could be cool use case for an exit animation like data-aos-exit='fadeOut' which would add the class as it's exiting (and removing when re-entering)

Same thing is happening to me. To be able to set an exit animation would be helpful.

I released current implementation as 3.0.0-beta.1, you can install it by pointing to next tag:
yarn add aos@next
Feel free to fork AOS and play with the proposition @randallmlough. I'm happy to see PRs to the next branch :) I'm going to focus now a little bit more on documentation and new, appealing demo.

good job

I'm closing this issue for now. It's still on the next branch, but I made it the default one for the time being. I also added recepies section with informations how to integrate 3rd party animation libraries. Feel free to open dedicated issue with exit class proposition and we'll go from there :)


Usage in a namespaced application and running CREATE statements

Hi!

What's the best way to use this in a namespaced application, so that all my classes under namespace whatver have access to the connection QB?

What's the best way to use this for running table creation statements?

QB is meant for this, it's like global you'll have access to \QB anywhere in your app.

Table creation is not supported but feel free to use QB::query('CREATE TABLE....')

Ah, thanks. Early in the morning, and forgot the \


The REST API is not enabled for this Organization.

While creating new object with the specified steps in getting this error .
Sample:
var account = new Account() { Name = "New Account", Description = "New Account Description" };
var id = await client.CreateAsync("Account", account);
error : The REST API is not enabled for this Organization.
at Salesforce.Common.ServiceHttpClient.d__121.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at Salesforce.Force.ForceClient.d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at SalesForce.Controllers.HomeController.d__0.MoveNext() in c:\Users\bharat.bhatia\Documents\Visual Studio 2013\Projects\SalesForce\SalesForce\Controllers\HomeController.cs:line 41

There is nothing I can do to help you. You'll have to contact Salesforce and have them enable API access for your organization.


Documentation for Scroller extension

On this page
change:

Scroller is a virtual rendering plug-in for DataTables which allows large datasets to be drawn on screen every quickly.

to

Scroller is a virtual rendering plug-in for DataTables which allows large datasets to be drawn on screen very quickly.

Thanks for pointing that out - fixed now.


The new Developer for Spotify website

After ~1,5 half years using a real OpenAPI definition from Spotify, it was time for Spotify to completely rebuild the Developer for Spotify website (also see this blog post about some insights).
Unfortunately, this change also removed the underlying OpenAPI definition so I no longer have a source to apply my patches on 😟
First of all, I will continue to provide an OpenApi definition for Spotify. 🎉 I will start to reverse-engineer the website to extract the necessary information to build an OpenAPI definition. But please be patient as this will take some time 🙏

There is very good news because Spotify's OpenAPI definition at https://developer.spotify.com/_data/documentation/web-api/reference/open-api-schema.yml is reachable again 🎉 So I can continue this project like it is today 😃


Serialize chat template

Currently the .save() method does not work for ChatPromptTemplate. I am implementing it here. The goal is to save/load chat template messages. Had to add in the role to each message. Please let me know if there is a better way to store the role.
This is what a chatTemplate.save() looks like when saved to json after this change:
{
    "input_variables": [
        "text",
        "output_language",
        "input_language"
    ],
    "output_parser": null,
    "partial_variables": {},
    "messages": [
        {
            "prompt": {
                "input_variables": [
                    "input_language",
                    "output_language"
                ],
                "output_parser": null,
                "partial_variables": {},
                "template": "You are a helpful assistant that translates {input_language} to {output_language}.",
                "template_format": "f-string",
                "validate_template": true,
                "_type": "prompt",
                "role": "system"
            },
            "additional_kwargs": {}
        },
        {
            "prompt": {
                "input_variables": [
                    "text"
                ],
                "output_parser": null,
                "partial_variables": {},
                "template": "{text}",
                "template_format": "f-string",
                "validate_template": true,
                "_type": "prompt",
                "role": "human"
            },
            "additional_kwargs": {}
        }
    ],
    "_type": "chatPrompt"
}

Fixes # (issue)
Before submitting

Who can review?
Community members can review the PR once tests pass. Tag maintainers/contributors who might be interested:

@hwchase17 @dev2049 do you mind reviewing please?

stale, serialization has since been overhauled


Text search API

@notmasteryet As I understand it you'll be tackling text search soon. @hubgit and I have been talking about extracting text from PDFs, and it'd be nice if we have an API for that.
Does it make sense to build the search feature on top of a PDFDoc() API that extracts the text from each individual page?
@hubgit Feel free to chime in on your needs here.

I'm wondering how this will look from an infrastructure point of view. We can reuse the IR extracted by the PartialEvaluator and look for showSpacedText and showText commands or we add a new function PartialEvaluator.getText(), that only returns a stream of text as found on one page.
Calling such a PartialEvaluator.getText() should be more performant and as people asked frequently to extract only text content from a PDF, it might be useful to have such a function.

Calling such a PartialEvaluator.getText() should be more performant

That sounds good. It'd be nice if we could offer a more friendly wrapper in PDFDoc() or Page() so that consumers of the API don't have to deal with an object from the guts of the code though (PartialEvaluator).
How about page.getText(), since users already have to deal with the page object?


Can't use with Scala 2.12

I cant use it with Scala 2.12
I am getting   UNRESOLVED DEPENDENCIES  when import like this "ws.securesocial" % "securesocial_2.11" % "3.0-M3"
::::::::::::::::::::::::::::::::::::::::::::::
[warn]  ::          UNRESOLVED DEPENDENCIES         ::
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  :: com.typesafe.play#play_2.11;2.3.7: not found
[warn]  :: com.typesafe.play#play-cache_2.11;2.3.7: not found
[warn]  :: com.typesafe.play#play-ws_2.11;2.3.7: not found
[warn]  :: com.typesafe.play#filters-helpers_2.11;2.3.7: not found
[warn]  :: com.typesafe.play.plugins#play-plugins-util_2.11;2.3.0: not found
[warn]  :: com.typesafe.play.plugins#play-plugins-mailer_2.11;2.3.0: not found
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
and for "ws.securesocial" %% "securesocial" % "3.0-M3" there is missing folder on maven repository  for 2.12:
https://mvnrepository.com/artifact/ws.securesocial/securesocial_2.12/
Is there is any specific repository to access "ws.securesocial" %% "securesocial" % "master-SNAPSHOT" using 2.12 ?

@ser-shant currently master-SNAPSHOT should work with 2.12.  The artifacts are published here: https://oss.sonatype.org/content/repositories/snapshots/ws/securesocial/securesocial_2.12/master-SNAPSHOT/.
mvnrepository.com does not track snapshots as far as I understand.

@jaliss this one was very useful, thanks a lot. Probably we need to put that repo info somewhere, or I was just looking in the wrong place.
May be you could also explain where/how do we use that?
crossScalaVersions := Common.crossScalaVersions
It is declared in build.sbt but never used
Thanks

@ser-shant if you hadd resolvers += Resolver.sonatypeRepo("snapshots") to your sbt file it should find the snapshot versions.

@jaliss right, it works fine for me, thanks


RuntimeError: shape '[32, 1, 1000, 64]' is invalid for input of size 2273280

303                 # B x L x S -> B x K x C x S
304                 nc = seq_len // self.chunk_size
--> 305                 q = q.reshape(bsz, nc, self.chunk_size, self.zdim)
306
307             if ctx_len < self.chunk_size:
RuntimeError: shape '[32, 1, 1000, 64]' is invalid for input of size 2273280

This happens when I set chunk size to anything but -1.  I am trying to set the chunk size to some number between 2 and 10,000 because I am getting OOM errors, but this error happens every time.

Did find out the solution for this error? I had the same problem.


[createReactClass] remove createReactClass from SegmentedControlIOS.ios.js

Related to #21581 .
Removed createReactClass from the SegmentedControlIOS.ios.js
Test Plan

[x] npm run prettier
[x] npm run flow-check-ios
[x] npm run flow-check-android

Release Notes
[GENERAL] [ENHANCEMENT] [Libraries/Components/SegmentedControlIOS/SegmentedControlIOS.ios.js] - remove createReactClass dependency

@RSNara
I fixed by following your comments 🙇
But, I couldn't fix flow error below...  in RNTester/js/SegmentedControlIOSExample.js
/**
 * Copyright (c) Facebook, Inc. and its affiliates.
 *
 * This source code is licensed under the MIT license found in the
 * LICENSE file in the root directory of this source tree.
 *
 * @format
 * @flow
 */

'use strict';

var React = require('react');
var ReactNative = require('react-native');
var {SegmentedControlIOS, Text, View, StyleSheet} = ReactNative;

class BasicSegmentedControlExample extends React.Component<{}> {
  render() {
    return (
      <View>
        <View style={{marginBottom: 10}}>
          <SegmentedControlIOS values={['One', 'Two']} />
        </View>
        <View>
           // flow error!!
          <SegmentedControlIOS
            values={['One', 'Two', 'Three', 'Four', 'Five']}
          />
        </View>
      </View>
    );
  }
}

This is the error...
Cannot create `SegmentedControlIOS` element because property `enabled` is missing in props [1] but exists in object type [2]. (SegmentedControlIOSExample.js:22:12)

I think it may be  a flow bug....
If you come up with the solution, please tell me... 🙇

@empyrical
Your comments help me!! Thanks 😄

@empyrical @RSNara
Thank you for your patience 🙇

And thank you for writing these pull requests! 👍