To field only accepts plain email

Issue Summary
When using '@sendgrid/mail' to send any email, the 'to' field only accepts plain email addresses, not name/email format (example: {name:'John Doe'<EMAIL_ADDRESS>).
Steps to Reproduce
const sgMail = require('@sendgrid/mail');
sgMail.setApiKey(process.env.SENDGRID_API_KEY);
const msg = {
  to: {<EMAIL_ADDRESS>name: 'John Doe' },
  from<EMAIL_ADDRESS>  subject: 'Sending with SendGrid is Fun',
  text: 'and easy to do anywhere, even with Node.js',
  html: 'and easy to do anywhere, even with Node.js',
};
sgMail.send(msg);

Technical details:

sendgrid-nodejs Version: master (latest commit: [commit number])
Node.js Version: 4.1.2

Hello @agostonbonomi,
Here are the supported methods of formatting email.
Please let me know if that helps.
Thanks!
With Best Regards,
Elmer

hey @thinkingserious , thanks for this. i missed this in the docs, works fine :)

Thank for the follow up @agostonbonomi!


Add Anytype icon

Added icon for Anytype. Icon looks a bit larger than some of the other icons, could be modified if needed.

Thought the new commits wouldn't apply to this PR, but there are minor changes to SVG markup here that might be helpful.


e2e test failing with missing dart file

> gulp test.e2e
[16:54:52] Using gulpfile ~/source/github/js_facade_gen/gulpfile.js
[16:54:52] Starting 'compile'...
[16:54:53] Finished 'compile' after 1.62 s
[16:54:53] Starting 'test.compile'...
[16:54:54] Finished 'test.compile' after 795 ms
[16:54:54] Starting 'test.e2e'...
Transpiling [] to .
Resolving dependencies...
Package test has no versions that match 0.12.13 derived from:
- e2e depends on version 0.12.13
Unhandled exception:
Could not load "file:///Users/kevmoo/source/github/js_facade_gen/build/e2e/helloworld.dart": FileSystemException: Cannot open file, path = '/Users/kevmoo/source/github/js_facade_gen/build/e2e/helloworld.dart' (OS Error: No such file or directory, errno = 2)
null
[16:54:56] 'test.e2e' errored after 1.88 s
[16:54:56] Error: 255
    at formatError (/Users/kevmoo/homebrew/lib/node_modules/gulp/bin/gulp.js:169:10)
    at Gulp.<anonymous> (/Users/kevmoo/homebrew/lib/node_modules/gulp/bin/gulp.js:195:15)
    at emitOne (events.js:96:13)
    at Gulp.emit (events.js:188:7)
    at Gulp.Orchestrator._emitTaskDone (/Users/kevmoo/source/github/js_facade_gen/node_modules/orchestrator/index.js:264:8)
    at /Users/kevmoo/source/github/js_facade_gen/node_modules/orchestrator/index.js:275:23
    at finish (/Users/kevmoo/source/github/js_facade_gen/node_modules/orchestrator/lib/runTask.js:21:8)
    at ChildProcess.cb (/Users/kevmoo/source/github/js_facade_gen/node_modules/orchestrator/lib/runTask.js:29:3)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)

The e2e test wasn't adding value. Removed.


Update home.elm

I found the existing tagline childish. Pete Hunt borrowed[1] a phrase when he talked about ReactJS which was "fall into the pit of success" which I think Elm helps users do quite well. Capitalization and periods are also good, we should use them.
[1] Originally coined here http://blog.codinghorror.com/falling-into-the-pit-of-success ?

Looks like the original source may be http://blogs.msdn.com/b/brada/archive/2003/10/02/50420.aspx.

Context:

Wouldn't it be nice to use a language designed to keep you from falling into the pit of despair? But avoiding horrific, trainwreck failure modes isn't a particularly laudable goal. Wouldn't it be even better if you used a language that let you effortlessly fall into The Pit of Success?

The Pit of Success: in stark contrast to a summit, a peak, or a journey across a desert to find victory through many trials and surprises, we want our customers to simply fall into winning practices by using our platform and frameworks. To the extent that we make it easy to get into trouble we fail.

Rico Mariani coined this term when talking about language design. You may give up some performance when you choose to code in C#, Python, or Ruby instead of C++. But what you get in return is a much higher likelihood of avoiding the miserable Pit of Despair-- and the opportunity to fall into the far more desirable Pit of Success instead.

I think the tagline could be better, but I disagree that this is the better one. Folks have done some brainstorming on elm-discuss, but I don't want to get too into this right now. I take design and branding passes over the website periodically, so I'll revisit this on the next pass.


Sample synthesis

This will be really cool.
Idea sketches:

Make a higher-ordered closure that takes in a &[u8] sample and returns a generator closure
Where to get samples?
Need to watch out for phase #4 (not sure if it's a big deal)
Pitch modulation?

Only a simple sample synthesis is done, it doesn't sound all that good right now because it's based off one single sample. Still, it's good enough!


⚠ Jenkins Plugins Artifactory has degraded performance

In 901e52d, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1249 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 0c141df after 25 minutes.


PDKClient.sharedInstance().getAuthenticatedUserBoards return value in Swift

The Objective-C function in the Pinterest API is defined as:

(void)getAuthenticatedUserBoardsWithFields: (NSSet *)fields
success:(PDKClientSuccess)successBlock
andFailure:(PDKClientFailure)failureBlock;

In Swift, I invoke the function as shown below.  Note how I am defining the variable "fields", which is to be populated by the function as the return value.   I believe I am defining this variable incorrectly to match (NSSet *)fields, which is defined in the Objective-C function.
I am getting a "Request failed: bad request (400)" response.
This following code is what I am using in Swift:
var fields = Set<AnyHashable>()

PDKClient.sharedInstance().getAuthenticatedUserBoards(withFields: fields, success: { (_ pdkResponseObject: PDKResponseObject?) in
  if let pdkResponseObject = pdkResponseObject
  {
    print(fields)
  }
}, andFailure: { (_ error: Error?) in
  if let error = error
  {
    print(error.localizedDescription)
  }
})

So, how do I represent (NSSet *)fields when invoking the function in Swift?
Thanks

Hi,
I was able to find the example on the web:
PDKClient.sharedInstance().getAuthenticatedUserBoardsWithFields(NSSet(array: ["id","name"]) as Set, success: { (responseObject: PDKResponseObject!) -> Void in
for object in responseObject.boards(){
let board = boardsModel()
board.boardName = object.name
board.boardId = (object.identifier)!!
self.boards.append(board)
}
So, the first parameter, (NSSet *)fields, in Objective-c,  is defined as the following in Swift:
NSSet(array: ["id","name"]) as Set
I don't understand that and will need to research it more.
But, it seems to work.


npm find 4 high vulnerabilities when building rpm package

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.0-beta.63
Electron version
v17.1.2
Operating system
Ubuntu 21.10 x64
Last known working Electron Forge version
No response
Expected behavior
Building of a  working rpm package
Actual behavior
npm audit report
minimatch  <3.0.2
Severity: high
Regular Expression Denial of Service in minimatch - https://github.com/advisories/GHSA-hxm2-r34f-qmc5
No fix available
node_modules/glob-stream/node_modules/minimatch
glob  3.0.0 - 5.0.14
Depends on vulnerable versions of minimatch
node_modules/glob-stream/node_modules/glob
glob-stream  0.2.0 - 5.2.0
Depends on vulnerable versions of glob
Depends on vulnerable versions of minimatch
node_modules/glob-stream
rpmbuild  *
Depends on vulnerable versions of glob-stream
node_modules/rpmbuild
4 high severity vulnerabilities
Some issues need review, and may require choosing
a different dependency.
Steps to reproduce
init app with electron-forge, install rpmbuilds and run npm audit
Additional information
No response

rpmbuild is not a dependency of Electron Forge, so this is not an issue with Electron Forge itself.

it made the same thing after i installed rpmbuild externaly

npm audit sends me this message

@malept hello pls help me

Unfortunately, due to the limited amount of time of the maintainers and the limited amount of information present in this issue, we cannot help debug your app at this time. Instead, you should consider using one of the community forums, including the official Electron Discord server, as there are more people there available to answer questions.

nvm by removing rpm build option it diseapeared, thanks for your support


Release v0.8.0

[x] Merge dev into master
[x] Add version tag
[x] Create a draft release on github
[x] Notify @d4rken when tag+draft release is created, to build a signed apk
[x] Up the version in the dev branch

Ah... i messed up the merging @_@

What happened?

Rebase merge made master deviate a lot from dev, says there is a ~20 commit difference now

No worries.
Check here for rebase vs merge.
Merging from dev into master is preferably done through a merge commit, not a rebase. Reserve rebase to update your own feature branches.
To fix this there are two options:

Rewrite history by force pushing master and overwriting the changes. This is frowned upon in public repositories because it screws (conflicts galore) over anyone that has checked out the master branch already.
Revert the commits and retry. This seems messy, but just costs us a bit spam in the log, and allows everyone who already cloned this repo to just pull the changes without conflict.

You already began the later option which is fine, though I think you also rebased them instead of merging them into master. Check the github button before pressing it 😉.
I'm now just merging the dev branch into the master again, I'll tag the final commit, build the apk and everything is in order.
To continue on dev with the next update I'll then pull the changes to master back into dev to get them to the same state.

👍

Ah, my bad 😢  thanks!
Given this is a fairly sizeable update any chance we might get a shoutout on the sub again for some installs? 🙋♂


Add Plausible snippet

@aivuk let me know if there's a better place for this

@aivuk let me know if there's a better place for this

It is the right place, but I moved to the 'else' condition because we don't want plausible loaded when hugo is serving the pages locally


Not Verified Pill Not getting update as credential gets re-verified from Credential Preview Screen

Identified after an installed LCW with verified credentials was offline for 30+ days and not able to connect to the Internet. Given the duration, LCW marks the credentials as not verified as expected--the verification status cache has expired and it can't connect to the Internet.
Connecting the phone to the Internet and previewing the credential triggers a new verification attempt. The main "Credential Verified" status changes but the pill above the credential name does not get updated at the same time. Selecting Credential Verified and going back does cause a status update to refresh the pill.

Related to issue https://github.com/digitalcredentials/learner-credential-wallet/issues/314

@dmitrizagidulin says this issue is out of date and should be closed if/until it comes up again


Support For TS Plugin in Figma Dev Mode's Codegen Plugins

Is your feature request related to a problem? Please describe.
There are some users who would like to use the plugin in the Codegen section of Figma's Dev-mode, but this is currently not possible as you can't get custom plugins in VSCode yet
Describe the solution you'd like
Currently blocked but may also need more information on this.
Additional context

proposing to archive as this is blocked by Figma


Footer: ticket link now linking to ticket area on start page

intends to resolve #701 .

Current coverage is 42.52% (diff: 100%)

Merging #735 into development will not change coverage

@@           development       #735   diff @@
=============================================
  Files                6          6
  Lines              689        689
  Methods             71         71
  Messages             0          0
  Branches           113        113
=============================================
  Hits               293        293
  Misses             396        396
  Partials             0          0

Powered by Codecov. Last update c8cc532...015f516

@shubham-padia just give the ticket button the id "ticket-button", that will do it. extra anchor tag not needed

@championswimmer extra anchor tag is needed to enable the smooth scrolling on the main page

@championswimmer And the inline-css in the anchor tag compensates for the header navbar, so that ticket button can be viewd below the navbar when the user clicks on it.


Delete users that haven't logged in for too long

Fixes #410.
Description

The previous user cleaner logged in to the MaMpf IMAP server and looked for hashes in mails it had sent before to users to make sure they didn't bounce. For bounces, the respective users were deleted immediately.
This behavior was deactivated as "unwanted" in #599. It's too rigorous. And maybe a mail server happens to be down that evening and whoop, that user is deleted. We don't want that.
Since #553, we track the last sign in date of users. This puts us in a new position: we can now use the last sign in date to delete users that haven't signed in for more than 6 months. We send deletion warning mails 40days, 14days, 7days and 2days before the actual deletion happens. The deletion of the user itself is irreversible.
For the complete flow, please see https://github.com/MaMpf-HD/mampf/issues/410#issuecomment-2136875776 or rather the docstrings for the UserCleaner class. Please complain if they don't explain some concepts that are actually implemented!

Changes

Implement the flow in the UserCleaner class (with unit tests).
Previous code in the UserCleaner is entirely deleted.
Schema changes: Remove old ghost_hash for users. Add new deletion_date field for users.
Don't miss out on the new initializer config/initializers/after_sign_in.rb

No mail bounce detection anymore
What about mail bounce detection? Sadly enough, detecting mail bounces is a non-trivial problem. Parsing the body of mails is not robust enough as texts like "Mail could not be sent" are not standardized. There exists a good strategy called Variable envelope return path (VERP) which sounds very promising. However, we might ask ourselves if it's worth the effort to implement this in light of the benefits we get.
Yes, a mail server of a user might be down for some time. But how likely is it that a user hasn't logged in to MaMpf for at least 6 months, then their mail server is down when we send the warning deletion mail 40 days, 14 days, 7 days AND 2 days before actual deletion? Considering how unlikely this event is, I think we can invest our time better instead of implementing a bounce detection algorithm that has to log in to our own IMAP server to retrieve all emails. It's also a lot less pain to maintain the UserCleaner without such a brittle component.
TODO

[x] Check what happens to comments that a generic user posted. We should have a "Deleted User" anonymous user that is assigned to those comments (or at least have that written in the comment title) -> comments are currently deleted, I will tackle this in a separate PR
[x] Check what happens to media that a generic user posted in a seminar talk. We probably want to completely delete it. -> while the user is deleted, the seminar still exists (with an empty user) and the media is preserved. We may want to change this  behavior in a separate PR.
[x] Add maximum limit of deletions per run.

Reviewers
The best place to start the review is probably the specs file for the UserCleaner.

[!CAUTION]
The code presented here might have very destructive consequences if something goes wrong. In the worst case scenario, it will delete active users that we didn't intend to delete. Unit tests should cover these cases but please be extremely vigilant with this PR and take extra steps and time to make sure the unit tests cover all edge cases and actually reflect semantically-wise what we want to happen.

[!TIP]
If tests don't work locally, you might have to run the migrations in the test docker container first.
That is: cd ./docker/test, docker compose up -d, docker compose exec mampf bash, rake db:migrate.

[!TIP]
In the Files changed tab on GitHub, you can mark files as done to get a better overview of what you've already reviewed. Every single line of code here should be looked through.

Before merge

[ ] Plan when this code will come to the main branch, so that everyone (including sysadmins) are on standby in case something goes wrong.
[ ] Check that our "import backups" strategy still works flawlessly and can be fully executed within 1 hour.
[ ] Set MAX_DELETIONS_PER_RUN env variable in production

Unit tests related to this PR all run through as of commit 18bd4f4f2. The whole suite of unit tests just fails due to #650 (watchlist tests).

@fosterfarrell9 Do we want to disable the UserCleaner for our staging environments? Otherwise, me might lose users over there as they haven't logged in since a while...

@fosterfarrell9 Do we want to disable the UserCleaner for our staging environments? Otherwise, me might lose users over there as they haven't logged in since a while...

I guess that would make sense.


Was missing the woff2 file from font-awesome.

Fix #575

@jakubgarfield Can you checkout if this fixes it?

It should 👍


Add EnhancedLogFactory

add Supplier<Object> log methods
add tracing to CloseSafeProducer

If you agree with the approach, I'll change all logging.

On hold - see https://github.com/spring-projects/spring-framework/issues/22726


BottomNavigationBar has no option to remove tooltip or change the tooltip from its icon label [Web] [Desktop]

BottomNavigationBar has no API for enabling/disabling its tooltips.  Widgets that implement and use tooltips must also include an API that can be used to set the tooltip string to a value that differs from its default tooltip string(s) and to completely disable its tooltips. BottomNavigationBar now always shows its icon label as a tooltip on Web and Desktop, which seldom adds any value to the often already shown icon label.
Then newer BottomNavigationBar that uses BottomNavigationBarItem that deprecates title in favor of label is currently not very desktop/web friendly when it comes to providing control over its tooltips. It misbehaves to the extent that it is an issue.

There is no way to turn of the tooltips!
There is no way to provide a tooltip label that differs from the icon label!

Q: What value does the additional tooltip label that is identical to the already shown icon label provide?

If there was a way to via the API e.g. as a new property in BottomNavigationBarItem to provide a separate tooltipLabel that could provide some additional info in addition to the already visible string in the icon label, then its tooltips might be made more useful.
For BottomNavigationBarItem it could e.g. be added so that it uses label for tooltips by default and a separate tooltipLabel value if one is provided. There still needs to be a way to turn of the tooltips as you might not want to have tooltips at all on desktop and web when using the widget.

Many web and desktop may want to implement a global user controllable feature that enables and disables tooltips in the app entirely. Tooltips are nice for beginners when you are not familiar with the app, but when you know the app they tend to get annoying and get in the way of using the app. Providing users a feature where you can turn off tooltips is a nice way to increase the UX for more experienced users of your app.
Currently many widget in Flutter that uses tooltips make it impossible (or at least more difficult than it needs to be) to implement this feature as they have no way of turning off tooltips at all. The BottomNavigationBar is one critical widget that fails in this respect.

This example with the bottom navigation bar is based on the API docs sample:
https://api.flutter.dev/flutter/material/BottomNavigationBar-class.html
Where the deprecated (in beta channel and higher) title was replaced with label. On web and desktop platforms this results in a bottom navigation bar where there always is a rather useless extra tooltip that is identical to the label, with no apparent way of turning it off in the API or changing the tooltip label to something else. The only way to fix this now seems to be to fork both BottomNavigationBarItem and BottomNavigationBar and make custom versions of them, which from a maintenance point of view is less than ideal.
Example BottomNavigationBar code snippet:
https://gist.github.com/rydmike/e97d29789886b9b406618d851d765fff
Can be tried in DartPad here:
https://dartpad.dev/e97d29789886b9b406618d851d765fff

Related issues
The BottomNavigationBarItem case is just one example case, there might be more Widgets with similar tooltip issues on web and desktop. One found related sister topic is the similar issue with PopupMenuButton https://github.com/flutter/flutter/issues/60418.
Background
This issue was previously included as the 2nd part of the new feature proposal "Global enable/disable of tooltips in Flutter apps and provide API control over widget tooltips" https://github.com/flutter/flutter/issues/66993. I removed this issue part from that proposal and added this specific topic as a more actionable issue instead, just mentioning it and also its related sister issue https://github.com/flutter/flutter/issues/60418 as case examples where the lack of proper tooltip APIs are an issue.
The proposal https://github.com/flutter/flutter/issues/66993 itself is a bit more broad and includes a suggestion of providing an built in app level toggle to enable and disable tooltips.

Flutter doctor
flutter doctor -v
[√] Flutter (Channel beta, 1.24.0-10.2.pre, on Microsoft Windows [Version 10.0.18363.1139], locale en-US)
    • Flutter version 1.24.0-10.2.pre at C:\Users\mryds\fvm\versions\beta
    • Framework revision 022b333a08 (4 days ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at C:\Users\mryds\AppData\Local\Android\sdk
    • Platform android-30, build-tools 29.0.1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

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

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] IntelliJ IDEA Community Edition (version 2020.2)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2018.3.1
    • Flutter plugin version 51.0.3
    • Dart plugin version 202.8070

[√] VS Code (version 1.51.1)
    • VS Code at C:\Users\mryds\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.16.0

[√] Connected device (4 available)
    • Nexus 7 (mobile) • 0a99f5e8   • android-arm    • Android 6.0.1 (API 23)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 86.0.4240.198
    • Edge (web)       • edge       • web-javascript • Microsoft Edge 87.0.664.41

• No issues found!

@mariamhas FYI

Reproducible on master as well.
flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.344, on Mac OS X 10.15.7 19H2
    darwin-x64, locale en)
    • Flutter version 1.24.0-8.0.pre.344 at
      /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision de56c6ad72 (16 hours ago), 2020-11-23 05:28:06 +0900
    • Engine revision 23a8e027db
    • Dart version 2.12.0 (build 2.12.0-62.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 51.0.3
    • Dart plugin version 202.8070

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (4 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11
      (API 30) (emulator)
    • macOS (desktop)             • macos         • darwin-x64     • Mac OS X
      10.15.7 19H2 darwin-x64
    • Web Server (web)            • web-server    • web-javascript • Flutter
      Tools
    • Chrome (web)                • chrome        • web-javascript • Google
      Chrome 86.0.4240.198

• No issues found!

Possible Solution :
Adding a showTooltip bool property in BottonNavBar
Adding a tootTip string property in BottomNavBarItem which will be shown if showTooltip is true.
@pedromassangocode I want to work on this one.

Thanks @pedromassangocode it is indeed a correct observation that it is now also reproducible on master.
Also since with the current stable version, the usage of title is also preferred over label, the issue now also applies to stable.
On stable the issue is perhaps not as obvious, as long as we can only build for Android and iOS on stable, and since a long press is required to show a tooltip on a device, the tooltips do not present themselves as easily and are seldom in the way or repetitive, even if the the icon label and tooltip are identical.
An easy way to disable tooltips on devices could still have it usage on devices too, especially since iOS does not have the concept of tooltips at all, thus if you want to use a Material component on iOS that has tooltips, then being able to hide/disable the platform foreign tooltips elements might also be useful there.
That being said, not being able to disable tooltips is more of an issue on Web and Desktop platforms. Currently the only way to hide the tooltips on ´BottomNavigationBaris to use it with theBotomNavigationBarItemdeprecatedtitle` property that takes a Widget based label and not a String label, and that has no default tooltip, unless you build one into the the used Widget yourself.

Thanks @yash1200 for offering to pick up the ball on this. Your suggestion is exactly what I would like to see on this particular case and Widget as well. Plus with backwards compatible default behavior, it would just be additive and not break anything.
Then maybe one day when all Widgets that have similar String based built in tooltips and implement a showTooltip flag, that flag could be connected to an optional MaterialApp level flag for showing hiding tooltips, that sets the current app wide default for all Widgets that have built in tooltips. See concept idea and suggestion here: https://github.com/flutter/flutter/issues/66993.
Until then you can of course easily implement such a feature yourself, as long as the individual Widgets you use support enabling/disabling the tooltips. The proposal https://github.com/flutter/flutter/issues/66993 is more about supporting such a feature out of the box and making coding with Flutter even more delightful than it already is.

An interesting observation about the BottomNavigationBar is also the fact that it currently shows tooltips that are identical to already visible icon labels and that this does not adhere to the Material guidelines concerning tooltip usage:

here: https://material.io/components/tooltips#placement

Same feature request.

Hi, we want to decrease infos ( warnings) in VSCode but we don't want to show tooltip.Therefore, as I understand, you suggest that usage of title but dart give warning because of the title usage :( , How can we decrease infos :(

Hi, we want to decrease infos ( warnings) in VSCode but we don't want to show tooltip.Therefore, as I understand, you suggest that usage of title but dart give warning because of the title usage :( , How can we decrease infos :(

Use label with tooltip = '' (empty string), this will work.

Use label with tooltip = '' (empty string), this will work.

@utkudenis
Use label with tooltip = '' (empty string), this will work.
bu it shows empty tooltip on long touch

@utkudenis
Use label with tooltip = '' (empty string), this will work.
bu it shows empty tooltip on long touch

@utkudenis Share your flutter doctor as well the code.

@utkudenis Share your flutter doctor as well the code.


NETOBSERV-1312: remove GC from ring buff mapper not needed

Description
doing GC for ring buff was adding GC cpu overhead while its not really needed, this was very obvious when use sampling: 1 and run with load ./hey-ho.sh -r 5 -z 10m -p -n 3 -q 800 -d 10
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.

[ ] Will this change affect NetObserv / Network Observability operator? If not, you can ignore the rest of this checklist.
[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.

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

[ ] 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):

[X] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] 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).

/ok-to-test

/ok-to-test

Will post Perf test results here once they are available :+1:

I have run a node-density-heavy test as detailed in our [QE test case] using this new eBPF image
Overall we did see performance improvements versus what is currently in our downstream bundles, see here for details
The full data sheet from the run can be found here

I'm fine to take this PR and I'll close my other one #184
They both show equivalent gains in CPU while this one keeping used memory lower.
/lgtm

/approve

/label qe-approved


Fix visibility of variables defined inside blocks and loops in debugger view

Purpose

$title

Fixes #32477
Approach
Samples
Remarks
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing this because the fix has been sent to 2.0.0-beta3-dev branch.


Handle escaped quotes within quoted terms

Fixes issue #1
Escaped quotes within a quoted term no longer cause a parse error. The resulting term's value contains the inner quotes unescaped. Promoted the preexisting pending unit test, with some tweaks.

Will this get merged at some point?


HHH-15221 fix a logging error in JdbcValuesCacheHit.java

https://hibernate.atlassian.net/browse/HHH-15221

Hi @NathanQingyangXu , yes please create a Jira. Thanks

Hi @NathanQingyangXu , yes please create a Jira. Thanks

done. Could we assume the issue was not flushed out by testing due to the missing coverage?

thanks @NathanQingyangXu ! merged


On the issues of Pixel2Mesh dependencies

Hi everybody,
I would like to ask you about dependencies of Pixel2Mesh so that I can apply the code to be used in python 3.6.9 since the example in github is for python 2.7 though.
Furthermore, my system also use tensorflow 1.14, and tflearn 0.3.2 as shown in the attached image.
When I tried to run Pixel2Mesh example in demo.py, I still got error "No module named layers"  at api.py in p2m directory.
How can I deal with this kind of problem?
Hope there will be solution for this problem

Hello all,
facing same issue :( , May I know any solution for this problem
@walsvid, did u get any solution
Many thanks in advance

Just use "from .layers import *" (add a dot) and also in layers.py do from .inits import *
But after that i am getting this error:
Traceback (most recent call last):
File "demo.py", line 73, in
pkl = pickle.load(open('Data/ellipsoid/info_ellipsoid.dat', 'rb'))
UnicodeDecodeError: 'ascii' codec can't decode byte 0xbe in position 3: ordinal not in range(128)

@WisarutBholsithi your issue is because they import without specifying the python module. Adding the p2m repo to the python path fixes all such issues.
For the unicode error can be an incompatibility with pickle between python 2.7 and 3.x I fixed using the following line
pkl = pickle.load(open('Data/ellipsoid/info_ellipsoid.dat', 'rb'), encoding='latin1')


Expose binary recording data to "progress subscribers"

My objective to broadcast binary audio data over a websocket as it's being recorded.
It seems the data sent to "progress subscribers" contains only currentTime. Would it be possible to send include binary audio data, similar to the WebAudio API's ScriptProcessorNode.onaudioprocess callback?
Maybe I'm going about this in the wrong way? Any tips or feedback would be greatly appreciated!

For now I'm working to port cordova-plugin-audioinput to React Native.
If I ever wrap my head around the native code I might try to send a pull request. No promises!

If you're working on the native side than it might be a better idea to do the streaming logic from the native thread. You would only inform the JS thread about streaming progress.
Exposing a new method to AudioRecorderManager or adding a property to startRecording would be the most straight forward way to do that on Android.
Cheers

I also want to get byte stream in JS thread despite the non-negligible performance cost. I believe that every functionality should be supported purely with JS because this is what RN eventually heads for and audio processing is pretty doable in mobile device now (even in JS thread). Also, I believe audio byte stream is worth supported in that it became a basic component for many apps. As of 2016, lots of speech related APIs now got publicly accessible like Google Speech API and IBM Watson Speech recognitions, which supports real-time streaming. As @RakanNimer mentioned, it is better if we can selectively enable this feature as the developer needs.
By the way, the progress update is not currently available for Android mainly because MediaRecorder class, which react-native-audio relies on, does not support progress update.
To make it realized, for Android implementation, we need to use AudioRecord class instead of MediaRecorder class, which makes inconsistency with the existing code. It would not be easy to support this feature without changing the core of this library. How about iOS side, any idea? (I can only read for RN and Android codes.)

And, also, RN native codes may not support ByteArray delivered from native source code. How would it be delivered from native code to JS code. Perhaps, we need to encode it to Base64?

If there's interest in this, happy to see a PR. Closing for now!

Is there any way i can get the recording sound in chunk to broadcast it to other clients via socket. If it is not supported can you help me how to convert the .aac file to a blob or arraybuffer?

@rohitgoyal basically you can't with any library. We are having the same problem. The issue is that there is no library that supports streaming chunks of data to the JS site. And there is no library to decode the aac file, as JS in react native doesn't have the audio context (unlike node and modern browsers) that all libraries depend on. We need to decode to the wave signal, soaybe you will have more luck finding an easier solution, but audio is really crippled at the moment in Reacht native. We are. Currently looking into porting the Cordova plugin to react native, but we are no experts on this either. If you come across any other solution, let us know.

@MrLoh According to you what is the best way currently to record a sound and broadcast to other users using socket.io on nodejs server?
I am using react-native-sound to play the sound.

@rohitgoyal As long as your en/de-coding is not audio specific and doesn't need an audio context, you should be fine, I guess. You will still have to write and read files all the time though and not sure how chunking the files would work well. It's sad that react native's js engine is so lacking.
Someone really need to port the cordova plugin though to do the relevant stuff on the native side as the js engine in react native is quite crippled when it comes to audio. And the current solution of reading and writing files is terribly unefficiant.

@MrLoh It is not audio specific. I used react-native-fs to encode the .aac to base64 string.
However there is not support in react-native-fs to decode that back to .aac.
Do you know of any way to play sound using base64 or a way to decode base64 back to .aac in react native?

@rohitgoyal no, sorry, I haven't worked with base64 in RN before.

@rohitgoyal I think base64 is just plan string with certain encoding rule, you can refer the resources on SO, I guess.
For your information, in my app's implementation, we worked on native code to stream the binary data to JS code using WritableArray. One issue here is that in Android implementation, the transferring through base64 encoding/decoding seems to be faster than the provided WritabelArray.

@ohtangza
This seems weird. You tried to write the array in chunks as the recording was done?
I have a base64 for a .wav file which while testing on some online base64 decoder gives me back the .wav file. But I am not able to get the correct .wav file from the base64 string on my react-native app.
I am using react-native-fs encode/decode for this. I think the decode functionality is not build for sound file in case of this library.
Which library are you using to decode/encode the base64 string?

@ohtangza Could you share your code?

@rohitgoyal @MrLoh You can refer the code here. We plan to make a pull request with this, but I am not sure if the author would accept it. In our application, streaming audio to JS code is pretty critical so we implemented it. Currently, this work is being done by https://github.com/ghsdh3409 who is my co-worker. Let us know if you have any idea or feedback.

@ohtangza Which one is the code? Are you referring to react-native-audio-toolkit?

@rohitgoyal Oh, sorry. I added the link above. It's still work-in-progress, but the binary streaming function should work.

@ohtangza hello, how about your branch  (https://github.com/hayanmind/react-native-audio/tree/feature/streaming-recording), when would it be merged?

any updates on it?

Is there a progress with live audio streaming on React Native? I found a react-native-audio-record library which returns base64-encoded sound online, but cannot make it work on iOS.

Hi, same problem as @teslavitas  has.


Mancala is a two-player strategy board game played with stones (pieces) and rows of pits (holes). It's also known as Bantumi. You can find detailed rules anywhere (for example: http://www.oware.org/mancala.asp) as it's a well-known game.
How do you play your game?

A / D - moves hand for pit selection left and right

K - select a pit to sow

L - start a new game

This is a version for 2 players

Select a pit to sow

Seeds are sown in an anticlockwise direction

If last seed ends in your store, play again

If last seed ends on your side in an empty pit, collect this and opposite pit (if there's any stone)

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[x] The image is in the /games/img directory.
[x] The name of the image matches the name of your file.

Thanks for your PR!

Hey, great job with this game! I'm merging this, and you should see it in the Sprig gallery starting tomorrow.

Hey, thank you so much! I've already made a game few days before and I've got my Sprig console on the way.


⚠ Bofrost.be has degraded performance

In c171708, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2135 ms

Resolved: Bofrost.be performance has improved in ce694c1 after 6 minutes.


Wrong history when using hot reload

Issue summary
Each time when hot reload, update the page that have the terminal new history is created.
Expected behavior
When a terminal is destroyed and created again, it should create the exact same terminal with the same history.
Steps to reproduce

Clone repo https://github.com/jcubic/jquery.terminal-docs
run in terminal cd docs && npm install && npm start
When browser open up type few commands
Modify something in the source code
Press up arrow
No history

Browser and OS
N/A
Additional notes
The terminal_id needs to be updated when a terminal is destroyed.

The problem is with Cycle::length, it returned array length and not how many items were in the array, removing the item is using delete data[index]. So after each hot reload, the terminal_id is the next value.
Changing the behavior of Cycle breaks half of unit tests.

This is only required by hot reload, so the fix was to add id option that replace old terminal with the same data in Cycle data structure.
With a single terminal on the page, you can use:
$('...').terminal(commands, {
   id: 0
});


[Mobile][iOS] The keyboard not showing up after the image is chosen from the device

The main problem is as stated in the title.
My issue is about [Mobile], and it's iOS specifically
I have tried running example directory successfully before creating an issue here.
And the issue reproduced in both the example from the package and our environment.
For further demonstration, here's a video recording of this case:
https://user-images.githubusercontent.com/13610283/129296896-0e270d11-5c39-43e5-b979-9343ee0a2446.MP4
As you can see, after the image is chosen from the device, the keyboard not showing up anymore no matter how I tap on the input area.
I have tried to use code like this to try to raise the keyboard up but none of the requestFocus method works...

      _quillController.addListener(() {

        if (MediaQuery.of(context).viewInsets.bottom == 0) {
         // method one:
          _editorFocusNode.requestFocus();
         // method two: and this two method I used them separately
         // but the keyboard just fail to show up
          FocusScope.of(context).requestFocus(_editorFocusNode);
        }

      });

Hello, we can't seem to run into this issue on our end.


RfTools Liquid Monitor doesn't work

Hi, I'm playing FTB Beyond 1.2.1 and I'm trying to automate the lava input in the deep resonance tank, but the liquid monitor of rftools doesn't work, it doesn't emit any redstone signal.

Note that the redstone signal comes out at the back of the liquid monitor (the side with the red arrow). Are you checking from there?

Omg, the only side I didn't check, I feel so noob lol.
Thank's buddy and keep up the good work

np


Dependency updates:

crossbeam-epoch 0.3.0 -> 0.4.0
libc

We're deliberately holding back crossbeam-deque to maintain compatibility with older rustc.  They only check the most recent stable rustc in their development, but they gave us extra support in https://github.com/rayon-rs/rayon/pull/480#issuecomment-364661178.  We probably should add a comment about this though, as you're not the first to try updating it -- like #535.
I see no reason to update the libc dependency either.  0.2.16 is just the version we chose when that dependency was added, but this acts like a minimum.  Cargo will automatically choose the latest compatible version possible, in this case anything from 0.2.16 up to (but not including) 0.3.0.

I pushed commit 0c8be8d2b82f5540465c7d9ff25f057743a88c22 to explain the older crossbeam-deque.


Bug: MapView opens up too far zoomed out

When I finished adding location tags to all my (relevant) notes, the MapView started opening like this (too far zoomed out)
https://user-images.githubusercontent.com/1289165/124168164-0eeffd00-daa5-11eb-8bfb-e57e100b39a8.mov

I saw this happening earlier today too! I'm working on making the plugin
load flow much more robust and that should take care of that too.
On Thu, Jul 1, 2021, 20:47 Emile van Krieken @.***>
wrote:

When I finished adding location tags to all my (relevant) notes, the
MapView started opening like this (too far zoomed out)
https://user-images.githubusercontent.com/1289165/124168164-0eeffd00-daa5-11eb-8bfb-e57e100b39a8.mov
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/esm7/obsidian-map-view/issues/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRZ2YAFMTT5J6ZJ3QJNQJTTVSS2VANCNFSM47VFYWJQ
.

I found this happened because I had one note with an incorrect longitude (2000+), which means it stretches out all the way. Oops! Maybe to prevent this, it's possible to sanitize inputs by computing the coordinates modulo the range.

Great, thanks!
On Thu, Jul 1, 2021, 21:09 Emile van Krieken @.***>
wrote:

I found this happened because I had one note with an incorrect longitude
(2000+), which means it stretches out all the way. Oops! Maybe to prevent
this, it's possible to sanitize inputs by computing the coordinates modulo
the range.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/esm7/obsidian-map-view/issues/5#issuecomment-872449114,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRZ2YFGOOJQQLY2SZ4YJV3TVSVNFANCNFSM47VFYWJQ
.

Fixed.


chore(master): release 11.0.0

:robot: I have created a release beep boop
11.0.0 (2024-02-01)
⚠ BREAKING CHANGES

Major rewrite, switching to ECMAScript instead of CJS.
Move InfluxDB settings to their own section in config file

Features

Add InfluxDB as destination for Windows service status monitoring (d2e4d25)
Add monitoring of Windows services (da6dfcc), closes #637
Add monitoring of Windows services (76247a1), closes #637
build: Create better API docs (9bad21a), closes #644
influxdb: Add Butler version tag to uptime data sent to InfluxDb (e60528a), closes #919
influxdb: Store failed reload info in InfluxDB (b45dedd), closes #866
log at startup current API rate limit (2fd7055)
log at startup which config file is used (2fd7055)
Log warnings when API rate limits exceeded (c15c45d)
Move InfluxDB settings to their own section in config file (d2e4d25), closes #670
mqtt: Add support for Azure Event Grid as MQTT broker (3794da3)
New command line option for setting API rate limit (2b08c5e), closes #659
Only get script log once from Sense server (f193538), closes #741
reload-alerts: Make app owner info available in reload failed alerts (1912d3e), closes #728
Store reload task success info in InfluxBD (e642ad4), closes #870
telemetry: Change to using PostHog for telemetry collection (4da339a), closes #744
Verify structure of config file on Butler startup (69e35ff), closes #675

Bug Fixes

Add missing fields to template config file (157f715)
Add verbose logging showing which NR accounts are used (907b9ae)
alerts: Empty alert recipient lists no longer cause errors (34421ce), closes #640
Allow empty New Relic settings in config file's uptime section (1de18e7), closes #562
Allow uptime reporting to New Relic without custom http headers (f455a38), closes #587
Automatic dependencies scanning (1d8201d)
Better log messages when rate limiting for reload notifications passes (0ee04fb), closes #571
Correctly report Docker status (72e1087), closes #939
deps: update dependency @fastify/swagger to v8.9.0 (b0f5440)
deps: update dependency yargs to v17.7.2 (79dd198)
Disable SMTP mail appender in sample config (4f63cb3)
disk-util: All subdirs under approved dirs are also approved (0ecc402), closes #831
docker: Docker image build fixed (d230e14)
docker: Fix broken Docker image (85f016e), closes #764
Enable Teams notif for Win service monitor now works (a0761cb), closes #758
Improve warning when custom property names in config file don't exist in Sense (a53cbe8), closes #586
Improve Winsvc checking efficiency (e9a1ff5)
Make path resoultion for QIX schema files more robust (568aa2e)
More robust generation of anonymous Butler instance id (5203d54), closes #863
mqtt: Better error handling when establishing MQTT connection (6299a2c)
mqtt: Better logging and check for cert existence (59dc4fa)
mqtt: Don't show MQTT startup info when MQTT is disabled (7f48bd3), closes #913
mqtt: More reboust startup code for MQTT & Win svc monitoring (577dba3), closes #897
new-relic: Send task failed event to New Relic (154a921), closes #750
Only initiate InfluxDB connection if it's actually enabled in config file (ba35605), closes #678
Only send to New Relic if event/log is enabled AND custom property name specified. (9fc81d0), closes #596
Only set up REST server if it's actually enabled in the config file (7746cee), closes #679
smtp: Add  verbose log of email recipients (8d4ad93)
Typos in Slack/Teams notifrication log messages (e28b715), closes #794
Verify that all required config file entries exist (38c42b0), closes #874
webhook: Deal with empty webhook list wo errors (3f42d02), closes #944
win-svc-monitor: Fix typo in log output from config file sanity check. (37d02b3), closes #708
winservice: Better handling of services that don't exist (56a18de)
winsvc: Bug fixes and better logging for win service monitoring (dc76b63)
winsvc: Don't send Win svc alerts when Butler starts (d716cfe), closes #896
winsvc: Make Windows service status checks quicker (001f99f), closes #912
winsvc: Optimize windows services checks (47e08ad)
winsvc: Win service monitoring no longer rely on New Relic (e47124c), closes #967

Code Refactoring

Major rewrite, switching to ECMAScript instead of CJS. (25f0ab2), closes #942

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ptarmiganlabs/butler/releases/tag/v11.0.0 :sunflower:


GetRewardEst iterates all over the locked tokens

GetRewardsEst(which is called in RewardsEst grpc query) calls FilteredLocksDistributionEst, which iterates all over the locked tokens(https://github.com/osmosis-labs/osmosis/blob/main/x/incentives/keeper/pot.go#L164). This causes issues on performance side, as it need to recalculate the reward every time when the query happens.
The suggested solution is limit the lockup period to the epoch, and make the lockup module updates the total amount when there is lock related event happens. The incentive module will read the cached state when queried.
ref @Thunnini

GetRewardsEst is for a single address, so we should be able to determine the rewards for that address without iterating over all locks. We just need to get the locks associated with that address, and the total amount of shares for each pot that the lock contributes to. Then we can calculate the reward going to that address

Adding a lock reference for the owner address and adding total share for a pot(which is updated by lock hook) would work.


Add discussion of OpenAI Democratic Inputs to AI grants to Deliberation and Voting chapters

https://openai.com/blog/democratic-inputs-to-ai-grant-program-update

I have added this discussion for the Deliberation part https://github.com/pluralitybook/plurality/pull/303

Glen could you please keep this open for a few hours as Vanessa is adding the Voting chapter?

The issue doesn't need to be open for her to add; I'll give her credit for the PR regardless.

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/

From: jonas-kgomo @.>
Sent: Friday, March 1, 2024 1:25 PM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Add discussion of OpenAI Democratic Inputs to AI grants to Deliberation and Voting chapters (Issue #291)
Glen could you please keep this open for a few hours as Vanessa is adding the Voting chapter?
—
Reply to this email directly, view it on GitHubhttps://github.com/pluralitybook/plurality/issues/291#issuecomment-1973712909, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSHW2KQXPUJ4ZD2XNXDYWDB3JAVCNFSM6AAAAABCXMEHZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZTG4YTEOJQHE.
You are receiving this because you modified the open/close state.Message ID: @.***>


chore: add types to i18next translations

Closes #171

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


core : less codenoise in ConceptProcessing

These comments are already gone. (and all code that was only there for debug purposes was removed)


Add method to describe required Options for a dataset

Currently, when someone is using a dataset for the first time, they might not know all the Options required in their config in order to call that dataset.
In order to help users, we should add a describe method where the user can see what options are required.
@dataset
def add(x: int = Option('X'), y: int = Option('Y'):
    return x + y

add.describe() = {'X', 'Y'}

The difficulty is that required options can change depending on other values in the options dictionary due to overloads. We would need to think through how to deal with overloads in this scenario.

This will be .explain on rewrite/monadic


puppeteerPoolOptions.launchPuppeteerOptions don't work in PuppeteerCrawler

It does unfortunately clash here
https://github.com/apify/apify-js/blob/master/src/crawlers/puppeteer_crawler.js#L297

I guess we can close this, the link points to stealth code that is no longer part of crawlee.


Change the name of the debug adapter.

I love the UI thank you for the hard work, and I'm not sure if this is the place to put this issue. If it isn't, I apologize. I looked all over the place and couldn't find an answer. when I debug my discord programs in the debug window I see this
[debug-adapter stdout] DBCFG INIT:  mongodb
Type 'dlv help' for list of commands.

[debug-adapter stdout] Settings updated

[debug-adapter stdout] Developer Mode:  true

Is there a way to change the name or disable the messages for debug-adapter ?

I'm afraid nvim-dap-ui and nvim-dap have no control of the text that is sent by the adapter. You could open an issue on the adapter's repo to see if they could allow changing it


Remove LatestLedger key from meta table

What
Fixes #233

Removes the LatestLedger key from meta table.
Use GetLedgerRange function for getting the latest ledger seq.

Why
We can use the GetLedgerRange instead of keep the key in the table
Known limitations
NA

@2opremio I will include caching the ledger range in another PR since I am testing things with it and this PR can be merged without it.

Caching the latestledger is pretty fundamental for performance, so I would rather wait to have caching in main.

Caching the latest ledger is pretty fundamental for performance, so I would rather wait to have caching in main.

@2opremio The latestLedger is still being cached - https://github.com/stellar/soroban-rpc/pull/256/files#diff-2b645f26f8eb502d1988698186482a5910b37509e606f968a3a60c927c397cfeL150. I was referencing the oldestLedger and the close times that comprise the range.

Ah ok, then the PR description and title are outdated. Can you update them?

Or do you mean that everything is cached and working except to the oldest ledger and the oldest ledger close time?

OK, I've taken a look at the code

@2opremio The latestLedger is still being cached - https://github.com/stellar/soroban-rpc/pull/256/files#diff-2b645f26f8eb502d1988698186482a5910b37509e606f968a3a60c927c397cfeL150. I was referencing the oldestLedger and the close times that comprise the range.

As I recall, this places it into the downstream cache, but doesn't really use the cache (as you can see in the code the code doesn't check the cache for the value it just writes to it). We need to be careful about using GetLedgerRange() as opposed to GetLatestLedger() here because it's more expensive.
One option is to actually read from the cache there but I think there were some subtleties preventing from doing so. Can you take a look?

Benchmarking GetLedgerRange after the new caching changes:
goos: darwin
goarch: arm64
pkg: github.com/stellar/soroban-rpc/cmd/soroban-rpc/internal/db
BenchmarkGetLedgerRange
BenchmarkGetLedgerRange-12    	   81654	     14526 ns/op

This is a reduction of 40% from the previous benchmarking time - https://github.com/stellar/soroban-rpc/pull/217#issuecomment-2197552507

I am missing a migration to remove the entry from the meta table.


Fix eslint issue when running unit tests

When running unit tests, eslint declares an error because of the missing empty line after import
ERROR in ./test/unit/index.js

  ✘  https://google.com/#q=import%2Fnewline-after-import  Expected empty line after import statement not followed by another import
  /home/toilal/idea-projects/vue-test-upstream/test/unit/index.js:1:1
  import Vue from 'vue';
   ^

✘ 1 problem (1 error, 0 warnings)

Thanks!


chore: add Mirror to official link

chore: add Mirror to official link

add Mirror to official link

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes :memo:

Any notes/thoughts that the reviewers should know prior to reviewing the code?

Mirror is deprecated.
Also please do all of the small changes in a single PR otherwise we'll have to reject it.


Profile manifest for com.twocanoes.xcreds-shares

Is there one available?
I can't modify the example mobileconfig in Profile Editor/ProfileCreator

It does!


flutter build APK failed

when running build APK it fails even it generate apk successfully local
attached log file for task
log.docx

Hi, can you please print the log here in a comment instead of attaching files. You do not need to print the whole log, just print the failing part of it.

@AhmedKhattab95 if your build is working now, please close this issue. Otherwise please print the error part of the log here.


how to cite the package and work

is there an arxiv number or publication one can use to cite this software ?

Hi Jean-Roch,
it would be best to cite the conference proceedings from Connecting the Dots 2018, but they are not yet published. In the meantime I registered this repo with zenodo, that should work too: https://zenodo.org/record/2590226
Cheers!


Issue 1: Automate Content Deployment for New Blog Posts

Description:
Automate the deployment of newly created blog posts. When a post is pushed to the main branch, it should trigger an automated build and deployment to GitHub Pages.
Tasks:
•	Create a GitHub Actions workflow (gh-pages.yml) to trigger on push events.
•	Configure the workflow to deploy new posts from /content/blog/ directory automatically.
•	Implement error notifications for failed builds.
•	Set up automatic commits to the gh-pages branch for deployment to GitHub Pages.

Next Steps:
•	Monitor deployments for potential failures.
•	Implement improvements to speed up deployment times.

Description:
Automate the deployment of newly created blog posts. When a post is pushed to the main branch, it should trigger an automated build and deployment to GitHub Pages.
Tasks:
•	Create a GitHub Actions workflow (gh-pages.yml) to trigger on push events.
•	Configure the workflow to deploy new posts from /content/blog/ directory automatically.
•	Implement error notifications for failed builds.
•	Set up automatic commits to the gh-pages branch for deployment to GitHub Pages.

Next Steps:
•	Monitor deployments for potential failures.
•	Implement improvements to speed up deployment times.


Semi-circle Slider

Are there props to change the slider from being a whole circle, to a 3/4 or half circle?
You can see an examples here: https://roundsliderui.com/demos.html#various-circle-shapes
(A similar question was asked here: https://github.com/devstark-com/vue-circle-slider/issues/11, but closed without any comments)

@melissaep no, this feature is not implemented for now


How to run Weave Scope on Kubernetes (behind a corporate proxy?

This is a continuation of https://github.com/weaveworks/weave/issues/2804 as a new issue however.
I follow the instructions on https://www.weave.works/docs/scope/latest/installing/#k8s
and do:
kubectl apply -f 'https://cloud.weave.works/launch/k8s/weavescope.yaml?k8s-service-type=NodePort'
The daemonset and container are installed.
I open the webbrowser and navigate to the scope page; it shows the basic items bu no data.
When I look in the log of one of the agents I see:
2017-02-15T13:12:12.177498099Z time="2017-02-15T13:12:12Z" level=info msg="publishing to: <IP_ADDRESS>:80"
2017-02-15T13:12:12.177714107Z <probe> INFO: 2017/02/15 13:12:12.177674 command line args: --logtostderr=true --mode=probe --no-app=true --probe.docker=true --probe.docker.bridge=docker0 --probe.kubernetes=true <IP_ADDRESS>:80
2017-02-15T13:12:12.177803038Z <probe> INFO: 2017/02/15 13:12:12.177773 probe starting, version 1.2.0, ID 7c511ac3b53023ca
2017-02-15T13:12:12.199456237Z <probe> INFO: 2017/02/15 13:12:12.199353 kubernetes: targeting api server https://<IP_ADDRESS>:443
2017-02-15T13:12:12.209850912Z <probe> ERRO: 2017/02/15 13:12:12.209780 Error fetching app details: Get http://<IP_ADDRESS>:80/api: dial tcp <IP_ADDRESS>:80: getsockopt: connection refused
2017-02-15T13:12:12.247439185Z <probe> ERRO: 2017/02/15 13:12:12.247308 conntrack stderr:NOTICE: Netlink socket buffer size has been set to 425984 bytes.
2017-02-15T13:12:12.257853856Z <probe> ERRO: 2017/02/15 13:12:12.257743 conntrack stderr:NOTICE: Netlink socket buffer size has been set to 425984 bytes.
2017-02-15T13:12:12.355343405Z <probe> WARN: 2017/02/15 13:12:12.355267 Cannot resolve 'scope.weave.local.': dial tcp <IP_ADDRESS>:53: getsockopt: connection refused
2017-02-15T13:12:12.355510221Z <probe> INFO: 2017/02/15 13:12:12.355482 Success collecting weave plugin status
2017-02-15T13:12:12.355625095Z <probe> INFO: 2017/02/15 13:12:12.355590 Success collecting weave proxy status
2017-02-15T13:12:12.371019479Z <probe> INFO: 2017/02/15 13:12:12.370936 Success collecting weave status
2017-02-15T13:12:12.843703967Z <probe> INFO: 2017/02/15 13:12:12.843620 Success collecting weave ps
2017-02-15T13:12:13.212639686Z <probe> INFO: 2017/02/15 13:12:13.212323 Control connection to <IP_ADDRESS> starting
2017-02-15T13:12:15.399595651Z <probe> INFO: 2017/02/15 13:12:15.399478 Publish loop for <IP_ADDRESS> starting
2017-02-15T13:12:42.285432501Z <probe> ERRO: 2017/02/15 13:12:42.285229 Error checking version: Get https://checkpoint-api.weave.works/v1/check/scope-probe?arch=amd64&flag_kubernetes_enabled=true&os=linux&signature=FlfgmzYXPJv7Gu9f1zDk1HlfM5s4UoN6KkQblH1z0AQ%3D&version=1.2.0: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused
2017-02-15T13:13:12.386132103Z <probe> ERRO: 2017/02/15 13:13:12.362332 Error checking version: Get https://checkpoint-api.weave.works/v1/check/scope-probe?arch=amd64&flag_kubernetes_enabled=true&os=linux&signature=FlfgmzYXPJv7Gu9f1zDk1HlfM5s4UoN6KkQblH1z0AQ%3D&version=1.2.0: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused

In this environment the hosts are running behind corporate proxy. Do I need to specify HTTP_PROXY / HTTPS_PROXY per container in the scope yaml file ?

Please re-open your issue at https://github.com/weaveworks/scope .
weaveworks/weave is for Weave Net.

The script has $(WEAVE_SCOPE_APP_SERVICE_HOST):$(WEAVE_SCOPE_APP_SERVICE_PORT) in it.
Do these need to be set, and to what?


Issue installing client - npm fixup

I am trying to install this and am receiving the following issue at Step 12.
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db
I've tried doing as it suggests, and it explains that there are issues with dependencies. I have tried keeping the legacy dependencies and updating them and however the error is still thrown when I try a redeployment.
Any ideas?

@trevorh13 please ensure you have the following prerequisite installed
Azure CLI readme about Azure CLI
We currently recommend using Microsoft Azure CLI version 2.27.0
To download the MSI installer for specific version, change the version segment in URL https://azcliprod.blob.core.windows.net/msi/version.msi and download it. Available versions can be found at Azure CLI release notes.
DotNet Core SDK .NET Core 3.1
Node.js
Powershell
Git
An Azure subscription
Please ensure you run the https://github.com/microsoft/Learn-LTI/blob/main/deployment/cleanup.bat to remove any failed deployments before running the setup again.


added support for autohide of NavigationTabBar at bottom of coordinator Layout

added support for autohide of NavigationTabBar at bottom of coordinator Layout
See new demo: Horizontal(Coordinator Layout) NavigationTabBar. The code was adapted from

Note that I did not change the version from 1.1.8. Please change it


[safesnap] pass tuple as a param for contract interactions

Summary
To create transactions with functions having a param of type Tuple, the array needs to be parsed before creating the transaction.

Looks great, thanks! :-)


Before request a new feature

[X] I have searched the issues of this repository and believe that this is not a duplicate.

Describe the feature you'd like supported
目前 是否有单独的接口或者说明 直接使用xquic传输rtmp 包，不使用http 3 等相关接口和验证功能，或者可选在编译的时候去掉多余的spyd部分。
Describe alternatives you've considered
No response
Additional context
No response

Is this a question and not a feature request?
If this is a feature request, please describe your requirements in more detail, otherwise please ask a question in the discussion forum.

可以试一下我的quic-fiber接口试一下 目前传控开发完毕
仍在开发阶段 没有把所有代码上传


Canary deployments

Canary deployments & Rolling upgrades
Link to official documentation

Learn feature and usage
Add this feature to nomad-job in the terraform-nomad module ( probably need to build another example in the module(s) )
Test (could be tricky)

simulate failures (using docker in vagrant hashistack, like just stop container. or use other tools like chaos-monkey)
test that blue/green deployments works
simulate update of service (optional)
test that the new version was deployed

Documentation

related https://github.com/fredrikhgrelland/terraform-nomad-minio/issues/64
related https://github.com/fredrikhgrelland/terraform-nomad-postgres/issues/44


Unable to deploy any edge function

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
No edge function can be deployed.
To Reproduce

Create a new function “test"

root> supabase functions new test

Deploy the function

root> supabase functions deploy test --debug --no-verify-jwt

The command fails with the following error
Bundling test
Error: Error bundling function: signal: abort trap
file:///Users/jdoe/Projects/sample/supabase/functions/import_map.json
file:///Users/jdoe/Projects/sample/supabase/functions/test/index.ts
assertion failed [block != nullptr]: BasicBlock requested for unrecognized address
(BuilderBase.h:550 block_for_offset)

Expected behavior
The function should be deployed.
System information

OS: macOS, Ventura 13.3.1 (a)
Chip: Apple M1 Max
Version of supabase CLI: 1.62.3
Version of Node.js: v16.20.0

Additional context
If you rerun the deploy command over and over again, eventually it might work.

This is very similar to another issue where x86 deno binary is used on M1 macOS. Could you try the fix here https://github.com/supabase/cli/issues/673#issuecomment-1341879953

Thanks for the hint. Indeed, my Deno binary was x86. I followed the steps in #673 and reinstalled everything. Unfortunately, it didn't help.
root$ brew config
HOMEBREW_VERSION: 4.0.19
ORIGIN: https://github.com/Homebrew/brew
HEAD: 4446fe603cc19316036fd07e5df1124c4d15ead2
Last commit: 2 days ago
Core tap JSON: 24 May 08:44 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 14.0.3 build 1403
Git: 2.39.2 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.88.1 => /usr/bin/curl
macOS: 13.4-arm64
CLT: <IP_ADDRESS>.1.1679647830
Xcode: 14.3
Rosetta 2: false
root$ deno --version
deno 1.33.4 (release, aarch64-apple-darwin)
v8 <IP_ADDRESS>
typescript 5.0.4
root$ supabase --version
1.62.5
root$ supabase functions deploy create-checkout-session
Version 1.30.3 is already installed
Bundling create-checkout-session
Error: Error bundling function: signal: abort trap
file:///Users/jdoe/Projects/sample/supabase/functions/import_map.json
file:///Users/jdoe/Projects/sample/supabase/functions/create-checkout-session/index.ts
assertion failed [block != nullptr]: BasicBlock requested for unrecognized address
(BuilderBase.h:550 block_for_offset)

Try rerunning the command with --debug to troubleshoot the error.
root$ arch
arm64
root$ supabase functions deploy create-checkout-session
Version 1.30.3 is already installed
Bundling create-checkout-session
Error: Error bundling function: signal: abort trap
file:///Users/jdoe/Projects/sample/supabase/functions/import_map.json
file:///Users/jdoe/Projects/sample/supabase/functions/create-checkout-session/index.ts
assertion failed [block != nullptr]: BasicBlock requested for unrecognized address
(BuilderBase.h:550 block_for_offset)

Try rerunning the command with --debug to troubleshoot the error.
root$ supabase functions deploy create-checkout-session --debug
Supabase CLI 1.62.5
Version 1.30.3 is already installed
Bundling create-checkout-session
Error: Error bundling function: signal: abort trap
file:///Users/jdoe/Projects/sample/supabase/functions/import_map.json
file:///Users/jdoe/Projects/sample/supabase/functions/create-checkout-session/index.ts
assertion failed [block != nullptr]: BasicBlock requested for unrecognized address
(BuilderBase.h:550 block_for_offset)

I take it back. I retried the scenario https://github.com/supabase/cli/issues/673#issuecomment-1341879953 again and it worked. Thanks for help!
In my case, it seems that the root of the issue is vscode terminal. I installed brew and all binaries (supabase, deno, etc.) though this terminal but later found that it's not arm64. Probably, it was the reason why the binaries were x86.


YJAF access for ttipler

Use this issue template if you do not feel comfortable raising a PR with the required changes to files in the terraform/ directory.
Please supply all the requested information, so that your request can be processed as quickly as possible.
Which MoJ repository/s are you requesting access to (e.g. money-to-prisoners)?
yjaf-assets
yjaf-auth
yjaf-bands
yjaf-bed-unlock
yjaf-case
yjaf-cmm
yjaf-conversions
yjaf-dal
yjaf-datateam
yjaf-documents
yjaf-gateway
yjaf-gateway-proxy
yjaf-mule-hub-connector
yjaf-mule-mis-processor
yjaf-placements
yjaf-refdata
yjaf-returns
yjaf-sentences
yjaf-serious-incidents
yjaf-transfers
yjaf-transitions
yjaf-ui
yjaf-views
yjaf-workflow
yjaf-yp
yjsm-hub
yjsm-hubadmin
yjsm-hub-svc
yjsm-ui
yjaf-utilities
What is the GitHub username of the person who needs access?
ttipler
What is their full name?
Thomas Tipler
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "Department for Education")?
NEC(YJAF)
What level of access should they have?

[ ] Read
[ ] Triage
[ ] Write
[ ] Maintain
[x] Admin

Why is this access being requested?
Existing devops - access expired
Which MoJ team/person is responsible for this request? This must be a ".justice.gov.uk" or ".digital.justice.gov.uk" email address.
This should be an email address<EMAIL_ADDRESS>Team email addresses are preferred, rather than individual email addresses, to make it easier to follow-up at a later date.
<EMAIL_ADDRESS>Review date
Date after which this collaborator's access should be reviewed. This should be a date in the format: 2021-01-15, no more than one year into the future from today.
2025-02-21

I think this might be an issue with my lap top and not the repo access

I just checked and its all repos think it might be my ssh keys
ssh: Could not resolve hostname github.com: Temporary failure in name resolution
fatal: Could not read from remote repository.


Spot price for USD->USD not found

Trying to last second get my taxes in, and found this project - super excited about it.
However, when I tried to process my Pionex data, it seems to fall apart on this line
"Date","Received Quantity","Received Currency","Sent Quantity","Sent Currency","Fee Amount","Fee Currency","Tag"
"01/01/2023 00:36:22","4.00000000","DOGE","0.27800000","USD","0.00400000","DOGE",""

With the following output
╰─ dali_us -s -o output -p try_ config.ini
INFO: Country: us
INFO: Initialized input plugin 'dali.plugin.input.csv.pionex'
INFO: No pair converter plugins found in configuration file: using default pair converters.
INFO: Reading crypto data using plugin 'dali.plugin.input.csv.pionex'
INFO: Resolving transactions
ERROR: Fatal exception occurred:
Traceback (most recent call last):
  File "/Users/ctucker/.pyenv/versions/3.10.13/lib/python3.10/site-packages/dali/dali_main.py", line 181, in _dali_main_internal
    resolved_transactions: List[AbstractTransaction] = resolve_transactions(transactions, dali_configuration, args.read_spot_price_from_web)
  File "/Users/ctucker/.pyenv/versions/3.10.13/lib/python3.10/site-packages/dali/transaction_resolver.py", line 246, in resolve_transactions
    transaction = _update_spot_price_from_web(transaction, global_configuration)
  File "/Users/ctucker/.pyenv/versions/3.10.13/lib/python3.10/site-packages/dali/transaction_resolver.py", line 144, in _update_spot_price_from_web
    raise RP2RuntimeError(
rp2.rp2_error.RP2RuntimeError: Spot price for __unknown:2023-01-01 01:21:10+00:00:USD->USD not found on any pair converter plugin
INFO: Log file: ./log/rp2_2024_04_09_22_01_55_583554.log
INFO: Generated output directory: output
INFO: Done

I'm using the for-cointracker.csv file directly as mentioned in the docs, but for some reason it looks like it's trying to find a spot price for USD -> USD?
Am I misunderstanding this? It seems that it shouldn't be trying to do this.
Please let me know if there's any other info I can provide to better debug this. I'm hoping maybe once I get my taxes done this week, maybe I'll have some time to contribute to this project.

The transaction it is choking on actually occurred at 2023-01-01 01:21:10+00:00.
The DOGE one that you submitted occurred at 01/01/2023 00:36:22
I just used the Pionex plugin to process my US taxes, but I never use fiat. I always trade and deal with USDT. So, there must be some issue with fiat processing.  Maybe it is a bank deposit?

My fault - I grabbed the wrong stack trace - I had removed the offending line to see if it was just that one, or USD in general...here's that trace:
╰─ dali_us -s -o output -p try_ config.ini
INFO: Country: us
INFO: Initialized input plugin 'dali.plugin.input.csv.pionex'
INFO: No pair converter plugins found in configuration file: using default pair converters.
INFO: Reading crypto data using plugin 'dali.plugin.input.csv.pionex'
INFO: Resolving transactions
ERROR: Fatal exception occurred:
Traceback (most recent call last):
  File "/Users/ctucker/.pyenv/versions/3.10.13/lib/python3.10/site-packages/dali/dali_main.py", line 181, in _dali_main_internal
    resolved_transactions: List[AbstractTransaction] = resolve_transactions(transactions, dali_configuration, args.read_spot_price_from_web)
  File "/Users/ctucker/.pyenv/versions/3.10.13/lib/python3.10/site-packages/dali/transaction_resolver.py", line 246, in resolve_transactions
    transaction = _update_spot_price_from_web(transaction, global_configuration)
  File "/Users/ctucker/.pyenv/versions/3.10.13/lib/python3.10/site-packages/dali/transaction_resolver.py", line 144, in _update_spot_price_from_web
    raise RP2RuntimeError(
rp2.rp2_error.RP2RuntimeError: Spot price for __unknown:2023-01-01 00:36:22+00:00:USD->USD not found on any pair converter plugin
INFO: Log file: ./log/rp2_2024_04_09_21_56_36_401006.log
INFO: Generated output directory: output
INFO: Done

And for completeness sake, here are the first 3 transactions:
"Date","Received Quantity","Received Currency","Sent Quantity","Sent Currency","Fee Amount","Fee Currency","Tag"
"01/01/2023 00:11:25","0.00337580","ETH","0.42960000","ATOM","0.00000338","ETH",""
"01/01/2023 00:36:22","4.00000000","DOGE","0.27800000","USD","0.00400000","DOGE",""
"01/01/2023 01:21:10","399629.00000000","SHIB","3.21701345","USD","399.62900000","SHIB",""

@tucamsam I wrote up a monkey patch that skips pricing of native fiat transactions that seems to resolve this.
Are you able to give it a try? Otherwise, we can try to get a release out with it, but it might take a while.

@macanudo527 - sure thing - I'd messed around with my own monkey patch, but it still ended up failing after running for a while

I'm closing this as #225 should have fixed it.


About evaluation method

Hi, Thanks for the great work. When I using the sample evaluation scripts, I got the following error. Could you please help me out?

Thank you for trying our tool and bringing the issue to our attention.
We found an error in the evaluation config, and we have now rectified it.
"e5-8b_flat" -> "flat_subset_499992"
The error you mentioned occurred because the specified retriever name in the evaluation config did not exist in the retriever config.
After the embedding process and construction of the Faiss index are complete, retrieval can be performed by correctly specifying the path to the built index in the retriever config (cf. indexing.md).
If you find indexing the corpus with the e5 model to be inconvenient, you can opt for BM25 indexing, which requires less time.
I hope this helps.

Thanks for response! It now works fine!


Minimum supported Rust version differs between Windows and other OSs

windows_registry.rs uses a number of features not supported on Rust 1.16.0 which appears to be the official minimum version. These include:

Iterator::find_map (Rust 1.30.0)
str::trim_start_matches (Rust 1.30.0)
Option::filter (Rust 1.29.0)

This seems to be slipping through CI as the Minimum Rust Job target only runs on Linux but most of the contents of windows_registry.rs is only built for cfg(windows) targets.

Actually it looks like the Minimum Rust Job isn't working at all at the moment (see Query rust and cargo versions).


[KeyVault] Managment API Version 2016-10-01 removes "all" from valid list of permissions

In API version 2015-06-01, these each had "all" as a valid option:
https://github.com/Azure/azure-rest-api-specs/blob/master/arm-keyvault/2015-06-01/swagger/keyvault.json#L298
https://github.com/Azure/azure-rest-api-specs/blob/master/arm-keyvault/2015-06-01/swagger/keyvault.json#L310
https://github.com/Azure/azure-rest-api-specs/blob/master/arm-keyvault/2015-06-01/swagger/keyvault.json#L322
In the latest, these values have been removed:
https://github.com/Azure/azure-rest-api-specs/blob/master/arm-keyvault/2016-10-01/swagger/keyvault.json#L405
https://github.com/Azure/azure-rest-api-specs/blob/master/arm-keyvault/2016-10-01/swagger/keyvault.json#L417
https://github.com/Azure/azure-rest-api-specs/blob/master/arm-keyvault/2016-10-01/swagger/keyvault.json#L429
This has resulted in breaking changes in downstream tools.  See: https://github.com/Azure/azure-cli/issues/3970
Is this value no longer accepted by the server in the new API version or is this simply an oversight?

@tjprescott The "all" permission was intentionally removed in this API version when we added the purge permission.  This was to avoid users unintentionally assigning the purge permission to users, and force them to be explicit.

Sounds reasonable.


doc: mismatch of callback function definition. cb(err, res) vs cb(res, err)

cb – callback following the callback convention, i.e. with signature function(exception, {output, errlog}).
vs
Most asynchroneous operations in this library take a callback with the conventional node signature cb(err, res).
there might also be some confusion in the code.
cb(errlog, output);
but then the output contains the following if there is no warning found.
e.g. console.log(output) gives the following
{ output: null,
errlog: 'No warnings or errors were found.\n\n' }

I get the feeling that I haven't yet understood your concerns here. I'll try to explain what I meant to say, and hope you can point out where the current formulation fails to convey that meaning.
Most asynchroneous functions in the Node API take a callback as last argument, and that callback gets invoked with two arguments: an error object and the result of the operation. Usually only one of these is set, the other is null or undefined. Think fs.readFile as an example. So I'm doing the same, but for me the result in case of a successful operation is itself a combined object, containing properties like output and error log. That way I can communicate various aspects about the operation in a way that can be easily adapted by future extensions without breaking compatibility. So yes, I'm following the cb(err, res) scheme, but for me, res = {output, errlog} if it is set at all, so the signature of the callback function as a whole is indeed function(exception, {output, errlog}). The choice of exception over err indicates that it's only set in case of a serious error, not for every error fixed by libtidy.
The title mentions cb(res, err), but I don't see that anywhere in the sources of documentation. Not sure what thats about.
That snipped cb(errlog, output); you quote occurs here, in the htmltidy compatibility layer. So here I'm not following my own convention, but that of the htmltidy and htmltidy2 packages. A few lines above I did output = res.output, so while res is an object as you describe it, console.log(output) should not log an object but just the output text as a string. If you actually observed something else while using the API, please provide a bit more context.

That snipped cb(errlog, output); you quote occurs here, in the htmltidy compatibility layer. So here I'm not following my own convention, but that of the htmltidy and htmltidy2 packages
I guess that's what confused me. your convention and the convention used in htmltidy are opposites.

Add to that the fact that your callback function is "function(exception, {output, errlog})" and I was lost.
closing as I don't know how it could be made clearer without making it lengthy.


feat: introduce stable sorting for traits with prefixes

This PR ensures stable sorting of derive traits with respect to their prefixes.
Before this change, traits with equal names but different prefixes were not sorted. For example, [b::Foo, a::Foo, Foo] would remain in the same order, when the expected order is [Foo, a::Foo, b::Foo].
This PR also adds support for distinguishing between traits in the custom order list and those that are not. If a trait like Eq is included in the custom order list, all variations with different prefixes will be sorted. If the trait Foo is not in the custom order and the preserve option is enabled, the original order of traits will be kept.
// Eq is in custom order, Foo is not.

// Case #1: sort all.
// Input:
#[derive(b::Eq, a::Eq, Eq, b::Foo, a::Foo, Foo)]
// Output:
#[derive(Eq, a::Eq, b::Eq, Foo, a::Foo, b::Foo)]

// Case #2: sort custom, preserve the rest.
// Input:
#[derive(b::Eq, a::Eq, Eq, b::Foo, a::Foo, Foo)]
// Output:
#[derive(Eq, a::Eq, b::Eq, b::Foo, a::Foo, Foo)]

@lusingander please take a look.

Great, thank you!


[2.3 beta] Selectable + Orderable conflicts

This one is an oldie but it's still present in the current integration branch: on tables that are both selectable and orderable, clicking on a row triggers mousedown.orderable first and click.selectable second. As a result, that same row changes its background-colour twice because the two CSS states (ordering and selected) have different styles.
A quick fix is to add a new class moving that only exists while the row is being dragged, then removing it when it's dropped. Perhaps it's not the most elegant fix in the world, but it's simple and effective (and works in my local branch).
This way, every time you click on a row only the selected style is applied. Then, as soon as the cursor starts moving, the moving styles overwrite the selected ones and the conflict is solved.
Any better ideas?

If we changed Orderable to allow ordering of all selected items at once, there would be no need for different colours for selecting and ordering.


Shared changes from coreclr

@alexperovich,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


[Event Request]   OnBeforeSetKeyAndFilters in "Reservation Engine Mgt."

Scenario, full tracking with full warehousing (Directed Put-away) when I pick 2 lotnumbers on a Prod. Component Line and pick+backflush on partial of the first lotnumer (its put into the machine), the second time I post output it takes the second lotnumber for consumption whilethat one is still besides the machine and not in the machine. So it keeps swapping the lotnumbers, while it should first consume the first lotnumber and then the second lotnumber when there is not sufficient inventory of the first lotnumber.
Where:
In codeunit "Reservation Engine Mgt."
local procedure SetKeyAndFilters(var ReservEntry: Record "Reservation Entry")
var
IsHandled : boolean;
begin
<< Start
OnbeforeSetKeyAndFilters(ReservEntry, IsHandled);
   if IsHandled then
      exit;

<< End
    if ReservEntry.IsEmpty() then
        exit;

    ReservEntry.SetCurrentKey(
      "Source ID", "Source Ref. No.", "Source Type", "Source Subtype",
      "Source Batch Name", "Source Prod. Order Line", "Reservation Status",
      "Shipment Date", "Expected Receipt Date");

    if ReservEntry.FindFirst() then
        ReservEntry.SetPointerFilter;
end;

New Event:
[IntegrationEvent(false, false)]
local procedure OnbeforeSetKeyAndFilters(var ReservationEntry: Record "Reservation Entry", var IsHandled: boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 45720.


Test: Add a test suite for paginate.js file

What would you like to share?
Add a test suite for the src/utils/paginate.js file.
Steps:

Add a new file paginate.test.js inside /__tests__/unit-tests
Add test suite for all possible scenarios

@MuminAhmadKhan  Maybe this one's yours.
Additional information
No response

@MuminAhmadKhan Maybe this one's yours.

Let me know if you want it and I'll assign you sir :+1:

Please assign me sir @MizouziE

Hi there , Please assign to me

Hi there , Please assign to me

Hi @sanyamgandhak , this issue is currently assigned to @MuminAhmadKhan , so it can only be re-assigned if they offer to step down or fail to respond after a long time.
Assuming that they'd like to finish this, I'd invite you to take a look through our issues as there are a number of very similar issues at the moment that we'd be very grateful for your contribution with.
Thanks for taking an interest in the project!

@MuminAhmadKhan I'd say to keep the tests just to the paginate function. Test things like:

If called, is the output as expected.
If number of articles is less than limit then there should not be a next: link
The data displaying the article count and page count etc is giving accurate readings

Use the __mocks__ folder and create all the dummy response data you need to test out the edge cases.
Let me know if that doesn't answer your question.


artifactory_scoped_token attempts to set include_reference_token when it is unset after upgrade

Describe the bug
When updating the provider to version 8.2.1 from 7.2.0, the provider attempts to set include_reference_token = false (the default) on artifactory_scoped_token resources. This fails since the token itself cannot be updated, leading to a state mismatch.
Requirements for and issue

[x] A description of the bug
[x] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue). If this is not supplied, this issue will likely be closed without any effort expended.
[x] Your version of artifactory (you can curl it at $host/artifactory/api/system/version
[x] Your version of terraform
[x] Your version of terraform provider

Expected behavior
When no value is provided for include_reference_token, the apply succeeds. Optionally, the state is updated to reflect that the value of that attribute is false since it was not possible to create a token without that explicitly set to true or import a token.
Additional context
Artifactory version: 7.59.9
Terraform version: 1.2.7
Attribute was added in https://github.com/jfrog/terraform-provider-artifactory/pull/723
Error message:
When applying changes to
module.user.artifactory_scoped_token.token[0], provider
"provider[\"registry.terraform.io/jfrog/artifactory\"]" produced an
unexpected new value: .include_reference_token: was cty.False, but now null.

This is a bug in the provider, which should be reported in the provider's own
issue tracker.

Example terraform:
resource "artifactory_user" "test_account" {
  name              = "test user"
  email             =<EMAIL_ADDRESS>  password          = "password"
  groups            = []
  admin             = false
  disable_ui_access = true
  profile_updatable = false
}

resource "artifactory_scoped_token" "token" {
  description             = "Terraform managed token for test user"
  username                = artifactory_user.test_user.name
  expires_in              = 31536000
  scopes                  = ["applied-permissions/user"]
}

Hi @bmanuel, I see that include_reference_token always had a default value of false, in theory, you should always have that attribute set. I'll look into it.

Just run into the same problem. Is there any progress? Reference: #761

@slzmruepp, @bmanuel working on it now.

include_reference_token was introduced in v7.7.0 and has had a default value of false since then, in your configuration, you don't have it, obviously.
Since the token can't be updated, I'll have to remove the default value from the attribute. Will be fixed in v8.3.1
Tested updates 7.2.0 -> latest, 7.7.0 - latest, 8.2.3 -> latest, no state drift.


[212] Continuous Deployment for FormsFlowWeb

Minor changes to Dockerfile to use bcgov artifactory.  Can potentially upstream these if need be for other bcgov formsflowprojects.

Created a single consolidated docker compose for deployment of all components (web, bpm & request management api) & externalization of profile specific properties in .env


4.0 版本【企业微信】，会有一个 GPU process crash 的问题，求解！

4.0 版本【企业微信】，会有一个 GPU process crash 的问题，求解！
[0331/105822.570061:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105822.570139:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 1 time(s)
[0331/105825.702591:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105825.702811:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105825.711249:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105825.711314:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 2 time(s)
[0331/105828.833708:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105828.833934:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105828.841266:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105828.841341:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 3 time(s)
[0331/105831.956455:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105831.956690:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105831.965939:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105831.966047:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 4 time(s)
[0331/105835.088990:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105835.089490:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105835.097126:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105835.097192:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 5 time(s)
[0331/105838.233016:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105838.233268:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105838.243349:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105838.243452:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 6 time(s)
[0331/105841.377750:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105841.378319:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105841.388266:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105841.388373:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 7 time(s)
[0331/105844.536098:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105844.542283:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105844.554762:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105844.554877:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 8 time(s)
[0331/105847.686510:WARNING:resource_bundle.cc(447)] locale_file_path.empty() for locale zh_CN
[0331/105847.687022:ERROR:alloy_main_delegate.cc(536)] Could not load locale pak for zh_CN
[0331/105847.694701:ERROR:gpu_process_host.cc(978)] GPU process exited unexpectedly: exit_code=5
[0331/105847.694765:WARNING:gpu_process_host.cc(1283)] The GPU process has crashed 9 time(s)
[0331/105847.694790:FATAL:gpu_data_manager_impl_private.cc(448)] GPU process isn't usable. Goodbye.
(lldb)

通过研究，crash 似乎和注入无关，发现只需要去掉企业微信的 codesign，就会出现上面的 crash，求助 @Sunnyyoung

本项目不支持企业微信


Making the command go to a new line breaks the program

Please fix this so I can display Shrek in its fullest potential.

(haha wrong repo)


Simplest way to get full HTTP request/response corresponding to specific API call

I'm calling REST api using Refit (I love it!  👍 ).
Now, for auditing/security reasons I need to get complete HTTP request/response (or body at least) corresponding to my API call that I just made.
What would be simplest way and what would be correct way to do that? (I hope simple=correct).
I've seen this issue about logging (#258), not sure how to apply this to my problem.
Thank you

Do you just want to pint it out, or do you want the object?

I need a string/stream of HTTP request/response in order to log it to my system for auditing purposes.

Change the return type to HttpResponse and do the deserializing of the object yourself. This way you can get headers, body etc. as needed.

Thank you, @Cheesebaron, I did not know it is possible.
In this case I would just use HttpClient and that's it. No point using Refit this way.

@alexsorokoletov You can get all the benefits of Refit and still get access to the raw request/response objects using a DelegatingHandler. See #258 for an example.

Sorry. Just noticed you referenced that.
Once you've created a delegating handler, you wire it up using the first code block in this message: https://github.com/paulcbetts/refit/issues/258#issuecomment-243394076