Add support for sending multiple embeds in one message

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming features.

Feature Request
Add support for sending multiple embeds in one message.
Also add methods addEmbed, setEmbeds, etc to MessageBuilder and MessageAction.
Example Use-Case
Discord has a feature that lets you send multiple pictures (up to 4) in one embed (visually). It needs that all embeds with pictures have equal title URL.
Example webhook JSON payload doing such thing (with random images from google):
{
  "content": "Test",
  "embeds": [
    {
      "title": "Test",
      "url": "https://example.org/",
      "color": 1723644,
      "author": {
        "name": "Test"
      },
      "image": {
        "url": "https://lizasenglish.ru/wp-content/uploads/2016/09/Tests.png"
      }
    },
    {
      "url": "https://example.org/",
      "image": {
        "url": "https://cryptodiffer.com/src/images/post/testtesttest.png"
      }
    },
    {
      "url": "https://example.org/",
      "image": {
        "url": "https://cryptodiffer.com/src/images/post/testtesttest.png"
      }
    },
    {
      "url": "https://example.org/",
      "image": {
        "url": "https://cryptodiffer.com/src/images/post/testtesttest.png"
      }
    }
  ]
}

This feature can also be used by bots.

https://ci.dv8tion.net/job/JDA/javadoc/net/dv8tion/jda/api/MessageBuilder.html#setEmbeds(java.util.Collection)
https://ci.dv8tion.net/job/JDA/javadoc/net/dv8tion/jda/api/requests/restaction/MessageAction.html#setEmbeds(net.dv8tion.jda.api.entities.MessageEmbed...)

I see these methods in development branch (my bad, searched PRs, issues and projects, but not repository), but in 4.3.0 there are no setEmbeds methods neither in MessageAction nor MessageBuilder, according to Ctrl+F in browser and IDE:
https://github.com/DV8FromTheWorld/JDA/blob/v4.3.0/src/main/java/net/dv8tion/jda/api/MessageBuilder.java
https://github.com/DV8FromTheWorld/JDA/blob/v4.3.0/src/main/java/net/dv8tion/jda/api/requests/restaction/MessageAction.java
And for example, there's no "use setEmbeds instead" in code below:
https://github.com/DV8FromTheWorld/JDA/blob/24aed4862649b9591532e3f14a26d2963a04e1dc/src/main/java/net/dv8tion/jda/api/MessageBuilder.java#L129-L143
View tag v4.3.0

You shouldn't be using a tag that was created months ago if you want up-to-date features, the latest version is 4.3.0_339 and that is what you see on development.

Oh, I understood, I had only looked latest release date on github (several times a month, including several moments before writing issue) and subscribed to discord channel waiting updates, but got no news and was thinking I use latest version (277 if you interested). Even documentation wasn't able to enlighten me: I was reading javadoc.
Thank you so much)

Both the readme and release have a badge showing the latest release on the dev branch.
Alternatively can you execute !latest in the #spam channel on the JDA Discord to get latest version, just fyi.


Forward Port issue266  from 4.0 to  CE/EE

LGTM, waiting for feedback from Chris.

LGTM other then comment

@mjolie, it looks like you removed the tests, not moved them.  I think we will still need them

@mjolie was not finished with my commits,   but back to the logging issue, it can not be a class in gateway.server , need to avoid circular dependency

The build problems have nothing to do with my changes

Looks good, one minor nit about the dependency management

Latest build passed: https://s3.amazonaws.com/archive.travis-ci.org/jobs/105762465/log.txt.  Not sure why didn't update.


No Bluetooth connection for setting up via app

Hi all,
Two of my three miners are syncing now, so right now I have no problems with them. But the third one causes problems.
The green led won't appear. I plugged power and waited a few minutes. Nothing happened. I pressed the button for more than 15 seconds but nothing happened.
What can I do to start the process of initial setup?
KR Drsokrates

Please contact<EMAIL_ADDRESS>


feat: Number of wins and Loses on Mobile

Prerequisites

[X] I have searched for existing issues that already include this feature request, without success.

Describe the Feature Request
On mobile it doesnt say W.1 or W.2 or L.1 and L.2 so you dont know how many items you have won or lost. I find this really usesful on PC as it allows me to judge my soft bans but when im out and use my mobile I dont have this feature. Please could you add this to the mobile aswell. Sorry if this is posted in the wrong thing
Describe the Use Case
To be able to judge my softbans
Describe Preferred Solution
No response
Describe Alternatives
No response
Additional Information
No response

Added in new release


Connection not establiched

Description
Can't connect HUE Apps or HASS with the emulated Bridge. This project is now dead?
Expected behavior
Nothing happened
Reproduction steps
Reinstall
Relevant log output
-

Operating system
HassOS
Install method
HassOS Addon
Version
0.3.0
Dev Version
No response
Any other information that may be helpful
No response

Yes this project is dead.
I have also tried fixing it. I get far enough to connect via the app, but then the app requests a config change via an API endpoint and then it never finishes that request.
Too bad.


There had been some work on the electric and the guy had not correctly enable the circuit to which the airco's are connected. :-(

Never mind.... sorry.....
There had been some work on the electric and the guy had not correctly enable the circuit to which the airco's are connected. :-(


Fix NPE in execute API (#16722)

Description
Parts in multipart requests can appear in any order. Earlier we assumed that the parts should maintain a strict order and that assumption leads us to this infrequent NPE.

Solution
Parts in multipart requests can appear in any order. In order to avoid NPE, the original name of the parameters along with the client-side data type are set here as it's guaranteed at this point that the part having the parameterMap is already collected.

Fixes #16722
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually through Appsmith App and the Postman client

Checklist:

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

/ok-to-test sha=bae082c

Could you please add a unit test for this so we remember this possibility in case of any refactors?

I am working on it. I am focussing to write two unit tests

With the usual/expected order of the parts
Changing the order of the parts (very likely that parameterMap will be the last part in this case)

A small update: There's no successful test case written on the /execute API so I am trying to achieve that as well.
@nidhi-nair

/ok-to-test sha=90f27ab


Introduced getpass since 1.8.1

Hi All,
It seems you introduced getpass module since 1.8.1, but getpass's license is MPL 2.0 but not MIT, any limitation to our usage?

I've just changed the license on getpass to MIT as well, to make this simpler. There is no problem generally with combining MIT and MPL-2 works (MPL-2 is purely file-based), but the getpass code is so small, honestly, that it seems easier to just make it MIT.
Hope that helps!


Updated broken links.

Updated broken links for 18 apprenticeships.

✅ By submitting this PR, I have verified the following

[✅] Checked to see if a similar PR has already been opened 🤔
[✅] Reviewed the contributing guidelines 🔍
[✅] Added my name to the bottom of the list under the Credits section in the README.md with a link to my website or GitHub profile 👥

Also had a question. I saw a couple that were closed. Would it all operate fine if I deleted the markdown file that had the information regarding it?

Also had a question. I saw a couple that were closed. Would it all operate fine if I deleted the markdown file that had the information regarding it?

Sorry for the delay here but yep, it would be okay to remove them in a separate PR if you confirmed the apprenticeships have permanently ended.


New Crowdin updates

:tada: This PR is included in version 0.30.0 :tada:
The release is available on:

v0.30.0
GitHub release

Your semantic-release bot :package::rocket:


fix for resource files not listed in resource map document

This also fixes the duplicate creation date for the Aggregation element.

+1. This can be merged in.


Browser tab freezes when trying to snap it back with other tabs already zoned

This is a continuation of this issue #3536
Microsoft PowerToys version
0.47.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
This reproduction method works on 100% of tries for me (I can make a video if this helps).

Install Google Chrome
Make sure FancyZones is on
Open up two tabs in the same window
Zone this window using FancyZones
Take one tab out, and zone the newly created window in the same zone as the old one (on top of it).
Take the tab and try to snap it into the window under it
It should instantly become white and unresponsive

Another way of reproducing the issue in case the one before didn't worked for you:

Open up two tabs in separate windows.
Make sure both tabs are zoned in different zones
Drag one tab into the other
Draw one tab into the other to merge them into one window.

It seems to make it easier to reproduce when the target window have many tabs in it.
✔ Expected Behavior
Tab should snap in without crashing
❌ Actual Behavior
Tab is crashing and becomes unresponsive when you snap it in.
It can happen both when you take the tab out of the zone or when actually snapping it into the other window.
Sometimes the tab simply disappear when taking it and can only be closed through the task manager.
This doesn't happen when FancyZones is off.

Other Software
Google Chrome 94.0.4606.81 (Official build) (64 bits)

@ion-elgreco

Does this happen with Edge as well?

Just tested and yes, but instead of becoming white, it freezes the content
On this image, none of the buttons of the front tab works event the minimize, maximize and close

I can't reproduce this on Win 10. Let me try Win 11, it might be something Win 11 specific

I can't reproduce this on Windows 11.

Here is my Edge version infos:
Microsoft Edge | 94.0.992.50 (Official build) (64-bit)
Revision | 80a49a71b4e2d00a1499ea10b608fe26bb6f41e7
Operating system | Windows 11 Version 21H2 (Build 22000.258)

I am also able to reproduce this with 100% consistency on v0.23.2 which I was on before and, after updating to v0.51.1, still almost 100%
As far as minimum repreducible steps:

Open a tab in any chromium browser (i've tested chrome + edge successfully, IE did not have the same issue)
Assign that tab's window to a fancy zone
Open a 2nd window (or have it open from the start). It's position does not matter at all, zone or not
Clicking and dragging on the original tab header to move the tab itself (not the window), drag it on top of the tabs of the 2nd window. You don't even have to let go of the tab, just drag it to the spot that you would drop it on.
The dragged tab will freeze, sometimes it will completely crash and disappear instead.

The tab that you drag has to be the only tab in the window, dragging a tab from a window with multiple tabs will not cause it to crash. Dragging all of the tabs from a window however (by selecting multiple with shift+click) will cause the crash. As long as you are dragging the entire window (via tabs) and not creating a new window by dragging the tab, it works.
When the window freezes (and doesn't just crash) it is completely unresponsive in every except that it can be moved. Attempting to interact with it does nothing. Trying to close the window via the X button, or via the taskbar does nothing. I have to resort to either the chrome or windows task manager to close the tab.
I'd be happy to provide any other info / screenshots/ videos or even a video call to show it off. I highly value fancy zones and this bug is very annoying.

I went ahead and made a video showing the issue, hopefully a bit more clearly than the other video provided. I made sure to enable the PowerToys mouse utility for clarity. In the video are tabs both disappearing and freezing, not being able to close the frozen tab except with task manager, and showing that it still crashes when using multiple tabs.
The video is here: https://imgur.com/a/54kWTmH

@harrison-munitz thank you very much for the clarification. I was able to reproduce the problem using the steps you described, but it happened just once for me. Can you please attach a bug report to help us investigate it?

/bugreport

I took this log immediately after booting up my computer and recreating the bug once. Hopefully it should be as simple as possible to see when the issue occured: PowerToysReport_2021-12-16-12-41-02.zip
Also, I went and tested a few more configurations of settings to see if the bug still happened. Disabling PowerToys and using Windows default edge/corner snapping does not trigger the bug, but changing my settings for how FancyZones was activated did have an effect! I now believe it has something to do with the overlay that appears when you move the zones around:

I usually use the "hold shift to activate zones" option, where the overlay won't activate until I'm holding shift while dragging a window. This causes the crash 100% of the time.
If I disable that option, the overlay will show up every time when I drag a window. This never crashes while merging tabs.
If I enable the "Use a non-primary mouse button to toggle" option, then I can at will enable and disable the overlay. 100% consistently, merging tabs with the overlay on screen does not crash, but if I disable the overlay while I'm dragging the tab, but before it collides with the other window, it will crash.

I'll try to take another video to better demonstrate what I'm saying. But basically, if the overlay isn't on screen when merging the tab, it will do the crash.
I reckon this is why you were having so much trouble recreating it.

Here, with and without the overlay: first with, and then twice with me turning it off. Although admittedly, it was happening much less consistently this way. Enabling the "Hold shift" option, and nothing else, like in the first video I shared, is the most consistent way I've found to recreate it.
https://imgur.com/a/t3t4vG9

@harrison-munitz Thank you for helping me to replicate the problem!
With the "hold shift to activate zones" option, I can replicate it. Not 100% of the time on my machine, but it can be used for testing now.

@harrison-munitz Thank you for giving a better reproduction method, I was able to reproduce it nearly every single time using this new method.

I consistently encounter this issue. I'll have many instances of Edge open each snapped to a fancy zone. Multiple tabs open in each instance. When I drag any tab to a different instance, dragging hangs up for about 10 seconds with the desktop in the fancy zone drag state (everything greyed a bit). Otherwise really loving and needing the fancy zones feature.

New PowerToys release v0.56. will be out soon. Could you check if the issue is still there when you update to v0.56 and if yes, post the new bug report here?
Thanks in advance!

Do you have any idea when that update will be live? Or is there a beta branch that I could download now?

@harrison-munitz 0.56 is out now! Head over to https://aka.ms/installpowertoys to try it out! /needinfo

The problem seems to be happening exactly as before. I've attached a bug report below.
Am I supposed to do anything with my settings to fix it? I saw some changes to popup windows in the patch notes but I haven't enabled the setting.
PowerToysReport_2022-02-28-19-24-56.zip

I can confirm the issue still persist on 0.56.1

I can still reproduce it 100% of the time using @harrison-munitz's method, but now it always makes the tab completely crash and disappear, no more frozen tab. I can do another video recording of it if this can help you recreate it.

does this issue still happen with v0.73.0? /needinfo

does this issue still happen with v0.73.0? /needinfo

No for me not anymore.


Accessibility on-screen keyboard

Issue Type: Feature Request
With all other programs, the entering of text will show an enabled on-screen keyboard (enabled via system accessibility preferences). With VS Code, when entering text in any file, the on-screen keyboard does not appear? Would be good to have an option to enable this, similar to screen readers.
VS Code version: Code - Insiders 1.48.0-insider (20cba51fb305868238eddc69758d259c29b3230d, 2020-08-07T05:39:52.666Z)
OS version: Linux x64 5.4.0-42-generic snap

@sevencookies Can you explain a bit more how does the on-screen keyboard help? I am just curious.
I believe this would need to be supported by electron.

Hello,
The on-screen keyboard can help people who have difficulty using a regular keyboard and find it easier to use a mouse, rollerball, joystick, headpointer or some other pointing device. I have trouble using some keys on my keyboard so a virtual accessibility keyboard is useful for me.
I was thinking options to extend the accessibility features of VS Code could be considered in addition to some other accessibility features already implemented (e.g. screen reader).
I found with other programs (with Ububtu) the on-screen keyboard appears automatically (e.g. terminal) whenever text can be entered but it doesn't appear in VS Code which maybe a local issue for me - I am not sure.

@sevencookies thanks for claryfing, this makes sense.
Yes, we could add either add a setting, or a global command which would show the keyboard or hide it.
However for this we would need some electron support.
I see that in the past the keyboard was being shown by accident, captured here https://github.com/microsoft/vscode/issues/76560
I am not sure if the electron currently has API to control this, more details here https://github.com/electron/electron/issues/6430
Also assigning this to @deepak1556 so he can provide more details.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Make richie a distributable / packaged Django application

Purpose
At the time of writing, Richie is a Django project embedding FUN's opinionated vision of what such portal should be. Let's make Richie a distributable Django app that can be customized and integrated in an existing Django (CMS) project.
Proposal
We need to make the project tree evolve to a more standard distributed app tree. I think the Django-Oscar project is a good example of what our repository should look like:

[ ] make richie a meta application that bundles sub-apps (core, courses, persons, search, etc.)
[ ] move the application to a src/ directory
[ ] move tests in a test/ directory
[ ] add a sandbox/ directory (the minimal Django project that will be used for development/tests)
[ ] package the application (setup.{py,cfg})*
[ ] automate packaging in the CI*

* Those two last points the latest point can be handled in a separated PR.
Edit: we need to package the app to make the whole refactoring working

Yup, we can close this.


Update 03-build-graphql-servers-with-prisma-e001.mdx

Fixes #3132, remove  data in createDraft

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


Added mfem to gcc8 and clang10 dockers

Summary

This PR is for adding the mfem feature to the docker containers.
It does the following (modify list as needed):

Adds mfem to the docker containers

Were you doing just +mfem in this PR or were you trying to update gfortran and CMake here as well? I am okay with leaving CMake at the current version because we test against newer versions on LC.  It may not be worth updating it here...

Were you doing just +mfem in this PR or were you trying to update gfortran and CMake here as well? I am okay with leaving CMake at the current version because we test against newer versions on LC. It may not be worth updating it here...

gfortran and cmake are in other branches and aren't working. I am keeping them separate. Is there any dependency on mfem for gfortran or cmake?

You need to update the azure-pipelines.yml to the new docker image names with the time date stamp in them.  It should be in the build logs I think.


[Navigation Menu]feat: 1/3

Todos

[ ] Make modals inside viewport
[ ] Make arrow center relative to button
[ ] Make viewport center relative to list container
[ ] Make viewport dynamic width/height

Related
#14

@zernonia do you think the navigation menu viewport has any similarity with scroll area's viewport?
https://www.radix-ui.com/docs/primitives/components/navigation-menu

@zernonia do you think the navigation menu viewport has any similarity with scroll area's viewport? https://www.radix-ui.com/docs/primitives/components/navigation-menu

yeah there's some similarity there

The keyboard nav should be similar RadixUI now. I've check against the list above, and seems like work fine! 😁
@khairulhaaziq


FR: Hide the trail with frontmatter field

I personally would love to be able to hide the trail for specific notes.
My personal suggestion on what to call the variable to hide/show trail is hide trail; Usually, you'll want to show the trail, so it makes more sense to me to explicitly mention when to hide it.

We just implemented this together, so this can be closed 🥳


Failed to get average tp from indicator.

2016-07-21 11:58:09 (INFO):     Starting to watch the market: BTC-e /
2016-07-24 11:55:36 (ERROR):    Failed to get average tp from indicator.
Waiting 3 days for an error using CCI

Another time :
2016-07-25 21:07:50 (INFO):     Starting to watch the market: BTC-e /
2016-07-27 04:08:45 (ERROR):    Failed to get average tp from indicator.
Just modified history size in both:
// CCI Settings
config.CCI = {
constant: 0.015, // constant multiplier. 0.015 gets to around 70% fit
history: 45, // history size, make same or smaller than history
thresholds: {
up: 100, // fixed values for overbuy upward trajectory
down: -100, // fixed value for downward trajectory
persistence: 1 // filter spikes by adding extra filters candles
}
};
and :
config.tradingAdvisor = {
enabled: true,
method: 'CCI', //'MACD',StochRSI
candleSize: 60,
historySize: 50,
adapter: 'sqlite',
talib: {
enabled: false,
version: '1.0.2'
}
}

I'll check it out!

It work with the default values.
btw i donot have good working values foreach

Same issue here... with latest version of gekko. After one day of gathering stats i received "Failed to get average tp from indicator." error.
Any idea how to get rid of it?
Setup for CCI.
config.tradingAdvisor = {
enabled: true,
method: 'CCI',
candleSize: 17,
historySize: 10,
adapter: 'sqlite',
// CCI Settings
config.CCI = {
constant: 0.015,
history: 90,
thresholds: {
up: 160,
down: -10,
persistence: 0
}
Exchange -> BTC-E

Hello everyone,
Getting same issue after i update to the latest vesrsion.

Getting it too on CCI

I'm also seeing this with CCI.

me too

and me

anyone solved this?

Hi, I'm having this issue. Not sure if it's blocking. Trying to setup using CCI.

Same problem with CCI on binance @askmike how to fix it?

This has been fixed since #2162.


Same typ for JWT and SD-JWT?

Should we consider to use the same typ header value for both variants, JWT and SD-JWT? I came across this when I was thinking about the OID4VC credential format profile for JWT-based VCs. A single typ would also result in a single format identifier in the presentation request, which might everyone's live easier. The only reason to distinguish would be that a dedicated format identifier for sd-jwt would allow the processor to directly decide whether selective disclosure is allowed in the presentation definition.
Originally posted by @tlodderstedt in https://github.com/vcstuff/draft-terbu-sd-jwt-vc/pull/120#discussion_r1229300126

Would it make sense to have the spec be only about SD-JWT VCs and treat the plain JWT VC as just a special case of an SD-JWT with no selectively disclosable claims? Then there wouldn't be variants and a single typ header value falls out naturally.

I guess after we merged PR #131 it seems we have decided to go with a single value for JWT VCs and SD-JWT VCs. Can we close this issue @tlodderstedt ?

As per editor's call from June 23rd, closing this since it was addressed by #131.


fix missing 'Bearer' in noaa tokenrefresher and add noaa reconnect wh…

…en streaming

LGTM

LGTM


[FancyZones] Messenger desktop window does not resize to selected zone

ℹ Computer information

PowerToys version: v0.29.3
PowerToy Utility: FancyZones
Running PowerToys as Admin: yes
Windows build number: 19042.685

📝 Provide detailed reproduction steps (if any)

Open Facebook Messenger desktop application (Windows Store version)
Try to move window to defined zone

✔ Expected result
Window gets correct position and size as defined by zone
❌ Actual result
Window gets correct position, but stays the same size. It does not matter if the window is larger or smaller than the defined zone. It does not get resized.
Window can be manually resized to fit in zone, so it is a resizable window.
📷 Screenshots

Duplicate of https://github.com/microsoft/PowerToys/issues/7425

Duplicate of https://github.com/microsoft/PowerToys/issues/7425

Disagree, #7425 was closed and redirected to #8383. Which is not the same problem. In my case, the zones show up and the window snaps correctly, it just does not resize. In #8383 the zones do not show at all and the window can not be snapped.

Disagree, #7425 was closed and redirected to #8383. Which is not the same problem. In my case, the zones show up and the window snaps correctly, it just does not resize. In #8383 the zones do not show at all and the window can not be snapped.

Still facing this issue after the update yesterday

/dup https://github.com/microsoft/PowerToys/issues/30148


Fix possible NoSuchBeanDefinitionException in Spring Monitor

Workaround waiting for the release of this OTel fix.

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-starter-monitor


Incorrect text in log message

lib/awscloudformation.litcoffee:270
grunt.log.ok "Stack<EMAIL_ADDRESS>does not exist."
should be
grunt.log.ok "Stack<EMAIL_ADDRESS>exists."

@ToneD, Hey man.  Just saw this, did you mean to add this to the upstream repo?

@mitchelldavis i was going to include it with my next pull request, but if it's the type of thing that annoys you then it's just a quick update :)

No, that works.  I was just curious.  Thanks for everything @ToneD


Running tests alongside actual webpack build screws up output.

I am using Gulp with webpack-stream, and running the actual build of my app at the same time as my tests.
Doing so caused my bundled output file main.js to be named main without the .js extension.
I got around this by nulling the output property in my config for the tests.
var webpackConfig = Object.assign({}, require('./build/webpack-config'), {
  stats: false,
  output: null // Fixes a Webpack issue where running the tests alongside the actual build would screw up the output.
});

Not sure how this happened. If you need any other info from me let me know.

This has been annoying me for a while as well - you end up with two stats listings for the webpack build, and the test one quickly scrolls past the main app output
Setting stats: false isn't working for me, once I have some time I'll see if I can track it down further

@jeffijoe @MattKunze - I'd need to see full configs as well as your gulp task to give you any useful information.
That said, if I had to guess I'd say you are watching from two different places || triggering from two different places || not waiting for the rebuild task to return before test execution if it's in a single gulp task.


[FEATURE]: I want to run Colossal AI in H800 with torch 2.2, Cuda 12.0

Describe the feature
I want to run Colossal AI in H800 with torch 2.2, Cuda 12.0. But existing example codes is built atop torch 1.12 and too old. I can not run it in torch2.2. How can I solve the compatibility problem

@gothicx

@ryanrussell @jeffra


Fix various style and logic issues.

Test URLs:

https://bug-styles--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/
https://bug-styles--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/fragments/404
https://bug-styles--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/author/john-pullo
https://bug-styles--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/posts/2023/12/top-10-best-new-cigars-of-2023

may be we can re-visit the pages once the article-teaser changes are merged to this branch.


Enable experimental for gst-plugins-good

This enables v4l2sink, a way of piping input into v4l2loopback. This is
useful for using a phone camera as a webcam, for example.

Another cc fail in travis:
checking for XShmAttach in -lXext... creating build/lib.linux-x86_64-2.7
cc -pthread -shared -lgcc_s build/temp.linux-x86_64-2.7/swig/pycdio_wrap.o -L/nix/store/j3sq8p7bp62w4sqqf3wrlfsj771p0qrx-libcdio-0.93/lib -L/nix/store/l8rhnzq9yxqhv3il4kmzqhkx8q06jzgv-python-2.7.9/lib -R/nix/store/j3sq8p7bp62w4sqqf3wrlfsj771p0qrx-libcdio-0.93/lib -lcdio -lm -lpython2.7 -o build/lib.linux-x86_64-2.7/_pycdio.so
gcc: error: unrecognized command line option '-R'
error: command 'cc' failed with exit status 1
builder for ‘/nix/store/l4aw0r9j98ipg9y8jjdiiw90lx90rxc6-python2.7-pycdio-0.20.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/vrah0awqw68fqxp08vqbbq5pbrmpzh98-morituri-0.2.3.drv’: 1 dependencies couldn't be built

Who is responsible for the gcc->cc change?

@lethalman: this CC=cc problem is addressed by 7dc57a389e3 (github seems to claim it's contained in our master, but my git only claims it's contained in staging).

Look good to me.
Just a little curious, why not use gstreamer-1.4.x?
IIUC, The bug label have nothing to do with this PR,
so I merge it.
Thanks!


Relocate the Address on the ActorSystem class

I'll merge this. the address is not a critical path anyway.
Many thanks @Damian-P !


xcb: Fix typos

Hmmm, I thought I fixed these in #357

Woups sorry didn't see that. And I didn't update in between.


[SWT-NNNN] Return the thrown error from #expect(throws:) and #require(throws:).

This PR changes the signatures of the various throws: overloads of #expect and #require so that on success they return the error that was thrown rather than Void. This then allows more ergonomic inspection of the error's properties:
let error = try #require(throws: MyError.self) {
  try f()
}
#expect(error.hasWidget)
#expect(error.userName == "John Smith")

For more information, see the proposal document.

Further PR Details
It is not possible to overload a macro or function solely by return type without the compiler reporting `Ambiguous use of 'f()'`, so we are not able to stage this change in using `@_spi(Experimental)` without breaking test code that already imports our SPI.
This change is potentially source-breaking for tests that inadvertently forward the result of these macro invocations to an enclosing scope. For example, the compiler will start emitting a warning here:
func bar(_ pfoo: UnsafePointer<Foo>) throws { ... }

withUnsafePointer(to: foo) { pfoo in // ⚠ Result of call to 'withUnsafePointer(to:_:)' is unused
  #expect(throws: BadFooError.self) {
    try bar(pfoo)
  }
}

This warning can be suppressed by assigning the result of #expect (or of withUnsafePointer(to:_:)) to _:
func bar(_ pfoo: UnsafePointer<Foo>) throws { ... }

withUnsafePointer(to: foo) { pfoo in
  _ = #expect(throws: BadFooError.self) {
    try bar(pfoo)
  }
}

Because #expect and #require are macros, they cannot be referenced by name like functions, so you cannot assign them to variables (and then run into trouble with the types of those variables.)
Finally, this change deprecates the variants of #expect and #require that take two closures:
#expect { // ⚠ 'expect(_:sourceLocation:performing:throws:)' is deprecated: Examine the result of '#expect(throws:)' instead.
  ...
} throws: { error in
  guard let error = error as? FoodTruckError else {
    return false
  }
  return error.napkinCount == 0
}

These variants are no longer needed because you can simply examine the result of the other variants:
let error = #expect(throws: FoodTruckError.self) {
  ...
}
#expect(error?.napkinCount == 0)

Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

Nice!

@swift-ci test

@swift-ci test Linux

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test


PlayStation 4 cd, will it be able to run it?

PlayStation 4 cd, will it be able to run it?

probably too early to know that, since it involves a bunch of things working first like encryption/decryption and sourcing keys and BDVD functionality if even planned


sticky property of table component is missing in ng-zorro

What problem does this feature solve?
ant-design(React) has add this property in version 4.6.0, but ng-zorro still not.
when will ng-zorro add this property?
What does the proposed API look like?
Set sticky header and scroll bar

any suggestion for this issue?

i would love this.


release(v1.6.2): Prepare the release

existing users of badger will be familiar with these keys.

Under the hood, I load Badger from the keys without a real mapping using a json marshal and it's the same keys than the Badger json configuration file. I will add a link to the Badger documentation too.
The configuration block is the generic approach to handle the custom configuration for each provider (Badger and Olric) and with that, we abstract the configuration structure. I know it forces the users to read the provider documentation but as it's a low level configuration and an edge-case I think they can make the effort to read it and understand what they configure.
And it doesn't generate conflicts with existing keys (e.g. the path key could conflict depending the provider).
Thanks for your review @mholt :)

Gotcha; yeah, sounds like a reasonable way to do it. Good work!


How can I call a native from another plugin? I'm getting "Script not found" error.

I need to call natives of your plugin from another plugin.
But I'm getting a runtime error: "Script not found".
Calling natives to other plugins works good, but with this one not.
I'm using sampgdk_FindNative(name) to find your natives, this works OK.
But using sampgdk_InvokeNativeArray(native, format, args) not executes it and prints "[Pawn.RakNet] BS_ReadValue: Script not found" on the console. This happens with all your natives, not only the mentioned.
I'm doing something wrong?

No, sampgdk does something wrong. It invokes a native function using faked amx which Pawn.RakNet don't know about.
Calling once AmxLoad explicitly from Pawn.RakNet with sampgdk_fakeamx_amx() as argument may help you.

Is there anyone who can help? it's been two years...

I know almost nothing about sampgdk and the c language, and I was wondering if anyone could fix the situation, and not just raknet, but other plugin libraries like samp-cef can't be called through samp-node. The underlying samp-node should be the sampgdk_InvokeNativeArray that uses sampgdk.
I am willing to pay a certain amount of money to the person who solves the problem, in fact I am sponsoring open.mp, although only £18 GBP at the moment.


cURL error 77

Hi,
Greetings of the day!!
I tried integrating google analytics to my website, after following the documentation of google analytics plugin, I got the following error:
cURL error 77: error setting certificate verify locations: CAfile: /etc/pki/tls/certs/ca-bundle.crt CApath: none (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)
My site dont have ssl certificate.
Somebody please help me resolve this.
Thanks in advance.

Sounds more like there's a problem with your server configuration such that cURL can't access a HTTPS URL, not that your site itself needs to support HTTPS. Try contacting your hosting provider.

Sounds more like there's a problem with your server configuration such that cURL can't access a HTTPS URL, not that your site itself needs to support HTTPS. Try contacting your hosting provider.

Thank you so much. I changed the php version to 7.2 and it worked.


Use hyphen as the separator in the openverse-catalog image

Description

All our services used images named openverse-*. The catalog was the exception because it used _ instead of -, which this PR rectifies.
This is only a change to the development environment, the image name was already openverse-catalog in GHCR.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (./ov just catalog/generate-docs for catalog
PRs) or the media properties generator (./ov just catalog/generate-docs media-props
for the catalog or ./ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

The catalog tests are passing just fine for me locally so I have no clue why the tests are failing in CI. @AetherUnbound @stacimc if you have a moment, could you help me with debugging this? Thanks!

Oh that's right, lemme look at the tests.

The index.json within the catalog artifact is:
{"schemaVersion":2,"manifests":[{"mediaType":"application/vnd.docker.distribution.manifest.v2+json","digest":"sha256:b7e073ea8840f0922b82ef1fc6dada298e5216e0500e50efd76f2fd6f06b3d7d","size":5985,"annotations":{"io.containerd.image.name":"docker.io/library/openverse-catalog:latest","org.opencontainers.image.created":"2024-06-19T23:39:52Z","org.opencontainers.image.ref.name":"latest"},"platform":{"architecture":"amd64","os":"linux"}}]}

So it looks like it's being saved as openverse-catalog correctly at least 🤔

I figured out what the issue was. Once we fixed the catalog image name to use hyphens, the test job started using the prod version of the image to run tests with. Earlier it would just not find it and build a dev version on demand.
I've updated the test job to build a dev image, identical to our process for the API. Thanks @sarayourfriend and @AetherUnbound for nudging me in the right direction!

I agree with @sarayourfriend's comment about the mixed usage being "ugly" and the name change causing a hassle for everyone using ov. Personally I value consistency over ugliness, and would justify the hassle worth it, at the moment, where we're ironing out the kinks with ov.
So I won't merge the PR till we can be sure this is actually worth merging.


Vulnerability roundup 92: mujs-1.0.6: 1 advisory [7.8]

search, files

[ ] CVE-2020-24343 CVSSv3=7.8 (nixos-20.03)

Scanned versions: nixos-20.03: 925ae0dee63.
Cc @pSub

See also: #90890

20.03 is frozen and EOL.
Closing issue.


Document tool calling capabilities

There's a handful of tool-calling tokens in tokenizer.json. Any chance this will be documented? What about at least few examples?

Here is function calling example with Coder v2.
https://github.com/foldl/chatllm.cpp/blob/master/docs/tool_calling.md


Republish organisation pages when information changes

This commit adds after_save and after_destroy callbacks to the following models that have their data included on an organisation homepage:

Corporate information pages
Features
Featured links
People
Roles
Role appointments
Social media links

This republishes the content item for the relevant organisation homepage(s) each time one of these models is changed, so that the content item stays up-to-date. This is because this data is stored in the details hash of the relevant organisation(s).
These can be removed once they’ve been fully migrated and are represented as links from the organisation content item.
Trello: https://trello.com/c/V4xp9H9C/51-republish-organisation-home-pages-every-time-a-relevant-field-changes

Got some failures in CI :-(


remove duplicate set_nb definition

This function was (identically) defined twice, causing a warning during precompilation.
(In general you might want to go through the code and delete anything that is no longer needed.)

CI failure is unrelated to this PR — it's due to the draft cholesky.jl file (which doesn't parse) sitting in the src directory.  In general, I would recommend working on features in a branch, and only merging when they are functional (and passing CI).

I fixed the problem from my computer.


add schedule endpoints

#192

Get Channel Stream Schedule
Update Channel Stream Schedule
Create Channel Stream Schedule Segment
Update Channel Stream Schedule Segment
Delete Channel Stream Schedule Segment
I only missed Get Channel iCalendar

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

49 of 49 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 92.934%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1644

Relevant Lines:
1769

💛  - Coveralls


Height not calculated correctly for tooltips with images on Safari

For tooltips containing images I'm finding that the image height does not seem to be included in the height calculation. I'm only observing this in Safari desktop browser - Chrome, Firefox, Brave etc all seem fine. If the tooltip only contains an image, the height is ~20px (basically the padding of the container). If the tooltip contains text and image, the height is only big enough to display the text. The tooltip is scrollable.
I've read some accounts that there may be issues with 'getBoundingClientRect' in Safari?

Hi, thanks for your message. Does it happen only the first time the image is loaded? If yes, please read the FAQ about thr issue of async content.

thanks for the quick reply .. it's not only the first time .. tooltip height is still not calculated correctly even if i open, close and then reopen the same tooltip
but thanks for the FAQ (sorry i missed that) .. adding explicit dimensions with css to the images seems to have solved my problem
it does seem like there is something specific with Safari going on however as it worked fine for me in other browsers without needing to add the CSS dimensions
anyhow thanks!

 Your Safari might have cache disabled.


Remove RingBufReader and RingBufWriter types

As discussed in #4, this removes RingBufReader and RingBufWriter.
The only drawback of this change is that you now have to use UFCS to disambiguate the method names in certain circumstances (specifically, if you have both Buf and MutBuf in scope and call advance or remaining on a RingBuf).
I don't see any way to avoid this, except giving Buf's and MutBuf's methods distinct names, but I also don't really think this is a big problem.

I've also fixed the the affected places in mio, but since the pull request needs to include the new bytes version number, it doesn't make sense to send the pull request until the new version of bytes is on crates.io.

Sorry for the delay. I wanted to get MIO reform done before tackling this and I guess I forgot about it :( I will take care of it now.

Woops, I just noticed in your comment that you said that you also updated mio (I already pushed the changes to mio myself :cry:). I guess I am tired!
Thanks for the PR though! Merged 6f00c6117b72189018866b18bc839600021a479e


Added config option for US version of MDY

Gives the option in mycroft.conf to add `"date_format": "MDY-us"
Makes it more grammatically correct for US english

This corresponds to this commit in skill-ovos-homescreen-lite

I wonder if MDY should have this behavior rather than another variant? Is there a locale that writes "September Wed 07, 2022" or is it a leftover from the previous default format (DMY) "Wed 07 September 2022"

This variant wont be recognized in other core places, its not officially supported in core and changing it in mycroft.conf may cause unexpected breakage when if comparisons start failing. Should this be default for MDY instead?

This variant wont be recognized in other core places, its not officially supported in core and changing it in mycroft.conf may cause unexpected breakage when if comparisons start failing. Should this be default for MDY instead?

I think this should be default MDY since I assume the US user base would be the largest using MDY format and this is the format used here. I agree adding variants will probably break other things that just do string comparisons


Unable to stop the timer => crash

This is starting to make me crazy during the last few days.
I am runing a timer and I decide to stop it, but the app wont let me and it does not respond. After a few seconds the app crashes. In some cases it does not even crash and I have to stop the timer manually in the web app and then the desktop app crash.
I have sent feedback directly from the app. I am not sure if you get logs this way.
Version 7.2.36 (stable)

which OS?

Ah, sorry. I use Windows 8.1.

This is starting to be really frustrating. It crashes multiple times per an hour. I am here in case that you will need some help.

looks like it's a known issue at the moment. the windows cannot render more than a couple hundred of time entries, before running out of window handles. we're working nonstop on replacing the windows app UI tech with WPF what will allow for more UI controls.
please use the web app until its fixed. we're super sorry.

Thanks, but where I can track this when this is closed?

please keep an eye on #1139


Change Similarity from enum to string

Currently exists as an enum with a single value.

Closing as #3863 is merged


Get Rid of Meme on Failed Login

I think it would be a good idea to remove the meme shown on the sidebar (popup) when a user fails to log in.
For me personally, being shown it once was enough to turn away from the software and seek other alternatives. Just doesn't feel like a well-thought-out or (possibly) secured platform when these images are shown to the user.
It's cool and all, but truly a useless feature that might conflict with users desire to utilize this software- or take it seriously.

This tool is not "serious" as in professional, it's not made for profit, it's free and if people don't take it seriously just because of that, well too bad for them 🤷♂


upgrade: swc_ecmascript, swc_common

Upgrades swc related crates to latest versions and unpins them.
Closes https://github.com/dprint/dprint-plugin-typescript/issues/78

Thanks!

@bartlomieju do you want a release?

@bartlomieju do you want a release?

@dsherret I don't have an immediate need, I can do that later next week : )


Add an atomic swap contract for cross-evm-erc20-chains sending of Tokens

🎉 Description

[ ] 🐛 This is a bug report.
[X] 📈 This is a feature request.

📝 Details
A number of projects are planning to build on their own evm compatible chain. I think we should add an exchange contract that allows secure transaction of tokens from one chain to the other.
This would work in 5 steps:

Alice (who has tokens on chain 1) picks a secret value and transmits the hash of it to Bob (who has tokens on chain 2)
Alice locks her tokens on chain 2, such that Bob can take out this tokens if he knows the secret
Bob sees the confirmation of that transaction, and then proceeds to lock his tokens on chain 2, such that Alice can take out these tokens by revealing the secret
Alice takes out the tokens from chain 2, revealing the secret (this should be included in an event log for easy access)
Bob sees the transaction on chain 2 and takes out his tokens from chain 1

Refund scenario:
If Bob never locks his tokens on chain 2, then Alice can, after a grace period, execute a refund to recover her tokens.
Scribble
PLEASE DO NOT USE THIS CODE as it's just a scribble and it needs A LOT of work.

contract ERC20 {
    function transferFrom(address, address, uint) public;
}

contract Atomic {
    // Mapping from receiving address to preimage to amount
    mapping (address => mapping(bytes32 => uint)) public contracts;

    // Mapping from sending address to preimage to time
    mapping (address => mapping(bytes32 => uint)) public refundTime;

    // Only one atomic swap per address at a time (prevents hash reuse)
    mapping (address => bool) public locked;

    event Execution(address indexed sender, address indexed receiver, bytes32 secret, bytes32 indexed hashed);

    uint public GRACE_PERIOD = 5 days;

    ERC20 public token;

    /**
     * Store a given `amount` of ERC20 tokens to be released by `receiver` upon
     * revealing a secret that hashes to `hashedSecret`.
     */
    function store(address receiver, bytes32 hashedSecret, uint amount) public {
        require(!locked[msg.sender]);
        contracts[receiver][hashedSecret] = amount;
        locked[msg.sender] = true;
        refundTime[msg.sender][hashedSecret] = now + GRACE_PERIOD;
        token.transferFrom(msg.sender, this, amount);
    }

    /**
     * Execute an atomic swap by releasing the preimage
     */
    function execute(address receiver, bytes32 secret, address sender) public {
        bytes32 hash = keccak256(secret);
        uint amount = contracts[receiver][hash];
        delete locked[sender];
        delete contracts[msg.sender][hash];
        delete refundTime[sender][hash];
        token.transferFrom(this, receiver, amount);
        Execution(sender, receiver, secret, hash);
    }

    /**
     * Fallback in case the counterparty never executed an equivalent transaction
     * in the other chain
     */
    function refund(address receiver, bytes32 hashedSecret, address sender) public {
        require(now > refundTime[sender][hashedSecret]);
        delete locked[sender];
        delete contracts[receiver][hashedSecret];
        delete refundTime[sender][hashedSecret];
        token.transferFrom(this, sender, contracts[receiver][hashedSecret]);
    }
}

Great idea.
Can you expand one the reason for the locked bit?

That’s just a quick fix for an attack that can happen because of the way I was coding it.
I can “burn” the tokens for someone if I send twice a request. There are far better ways to solve this.

@shrugs I also have a HTLC implementation
see: https://github.com/HarryR/shwop/blob/master/contracts/HTLC.sol
I think it would be a very good idea to have a standard ERC for hash-time-locks
Do you fancy working together on this together? There are a few other projects I've linked in a ticket: https://github.com/HarryR/shwop/issues/5
There are a few issues which need documenting and enforcing, such as unequal timeouts to provide one party enough time to withdraw before a refund.

We would certainly welcome an HTLC PR!

Indeed, a more generic hash-time-lock framework does seem more useful (and could benefit from our intended new escrows: #1099). Closing for now due to a perceived lack of interest.


Variable buffer size

Added "bufferSize" property.

+1

thanks @karlisl much appreciated ✌
if you have some time to add this to the swift 3 branch too, that'd be awesome

Oh... sorry...
May be this is dangerous.... 😰
For example,
when user paused by button, and then, may be video can restart to play by buffer. 😰
So, I think that the control by buffer should be done by user of this Library.
This commit should revert... 🙏  😢

@entotsu Player will restart video playback only when PlaybackState is "Playing". If video is manually paused by user, then it won't be restarted.
Anyway, I will make changes so that by default "variable buffer size" option is turned off.

oh! 😅 sorry...!
Thank you for your great PR! :relieved:


Potential Issue with vicuna models

System Info
Runtime environment:
Target: x86_64-unknown-linux-gnu
Cargo version: 1.69.0
Commit sha: b4aa87db58ea3d87ff820b6d7abeaa20a987df00
Docker label: N/A
nvidia-smi: N/A
Note that I'm using an AMD MI250 GCD so nvidia-smi and flash attention are not applicable. Each MI250 GCD has 64GB memory so it could host a 13B model without sharding. It means the model implementation that TGI used is CausalLM.
Information

[ ] Docker
[X] The CLI directly

Tasks

[x] An officially supported command
[X] My own modifications

Reproduction
Command (server):
CUDA_VISIBLE_DEVICES=0 WEIGHTS_CACHE_OVERRIDE=$HOME/models/vicuna-13b-v1.1 \
text-generation-launcher --model-id lmsys/vicuna-13b-delta-v1.1 --port 3000

Command (client):
from text_generation import Client

client = Client("http://<IP_ADDRESS>:3000")

print(client.generate("What's the penalty in California if I registered my vehicle late?",
         max_new_tokens=128, temperature=0.7, top_p=0.9).generated_text)

Expected behavior
The response I got is ��language. However, I tried the same prompt and configuration (temperature and top_p) at https://chat.lmsys.org/. The response looks quite reasonable:
In California, if you fail to register your vehicle within the required timeframe, you may be subject to a late registration penalty. The amount of the penalty will depend on the length of time your vehicle was unregistered. The penalty is typically assessed as a percentage of the annual registration fee, and can range from 10% to 50% of the fee, depending on how long the vehicle was unregistered.

In addition to the late registration penalty, you may also be subject to other fees and penalties, such as a delinquent registration fee and a possible suspension of your vehicle

The issue happens on both vicuna 7B and 13B models, so I suspect there might be some issues with the model implementation in TGI. I'll do some investigations, but would appreciate if anyone has any insights.
cc @OlivierDehaene

Update: I changed the server command to the following:
CUDA_VISIBLE_DEVICES=0 WEIGHTS_CACHE_OVERRIDE=$HOME/models/vicuna-13b-v1.1 \
text-generation-launcher --model-id $HOME/models/vicuna-13b-v1.1 --port 3000

This workarounds the issue. However, I then encountered the same error mentioned in #224. Again, I'm not able to enable flash attention in my environment so there's no way for me to workaround it.

Final update: I finally resolved this issue. This is because the base LLaMA model config specifies pad token ID to be -1, but the vicuna tokenizer uses pad token ID 0, so tokenizer.pad_token_id = self.model.config.pad_token_id results in OverflowError. I replaced the LLaMA config.json with the vicuna one to resolve this issue.


[TODO] send.js should create a constructor instead

Related to #182
Location: https://github.com/ansble/monument/blob/master/utils/send.js#L52
Description: think about making this a constructor that returns the modified response object instead of being added as it is          in the router.js file.

Thanks for opening the ticket!


Bump Dependencies #5020354458

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/5020354458

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Bug] [dolphinscheduler-server] master log  user task log info

Search before asking

[X] I had searched in the issues and found no similar issues.

[X] Yes I am willing to submit a PR!

Code of Conduct

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

relative issue: #7661


Improve Newsfeed UI

The UI on the newsfeed is basic and can be improved
I would like to Work upon developing and improving the UI of newsfeed to improve is UI/UX, make it more beautiful, appealing and precise.

Hi, could you make a design prototype for it.


Adding seeed target to the card list

Tested on:

[x] Edge 16
[x] IE
[x] Firefox
[x] Chrome Mac
[x] Chrome Windows

@Jaqster when you get a moment, waiting for your Approval.

Approved for Beta.  But not a huge fan of the image - maybe we can switch it out for something that looks a bit more polished for RC.

That's the image they supplied, let's give them our feedback and ask for another.

I don't think we can hold off posting the card any longer.  Let's publish, and aim for a nicer image next time.  Also, what's the logic behind the ordering of cards.    I kind of like the simplicity of in the order that they onboarded...

Here's a proposal for a process:

Three categories of editors: Non-beta (RC editor), Beta targets, app targets (eg: WW)
Ordered in the order of which they were onboarded, with non-beta categories first, beta targets next, and then finally app targets.

This means:

microbit, adafruit, minecraft, chibi
sparkfun, grove zero
WW

Merging for now, will discuss in person later.


Use VSCode product icons

I think the icons in the title bar don't really fit in with the default look of VSCode.

It would be better to use the native product icons, to achieve a more standard look:

If you use the debugger icons (e.g. debug-restart, debug-stop) you could even add some color. Also people could customize the look of the test explorer (or at least the title bar) with product icon packs.

@hbenl I can have a shot at this issue if you would wish to proceed with this.

There is a PR, also waiting for that :)


no element "glvideoflip" error for coral dev board

pose_camera.py seems to be failing with:
$> python3 pose_camera.py
Loading model:  models/posenet_mobilenet_v1_075_481_641_quant_decoder_edgetpu.tflite
Detected Edge TPU dev board.
Gstreamer pipeline:  v4l2src device=/dev/video0 ! video/x-raw,width=640,height=480,framerate=30/1 ! decodebin ! glupload ! glvideoflip video-direction=identity ! tee name=t
               t. ! queue max-size-buffers=1 leaky=downstream ! freezer name=freezer ! glsvgoverlaysink name=overlaysink
               t. ! queue max-size-buffers=1 leaky=downstream ! glfilterbin filter=glbox name=glbox ! video/x-raw,format=RGB,width=641,height=481 ! appsink name=appsink emit-signals=true max-buffers=1 drop=true

Traceback (most recent call last):
  File "pose_camera.py", line 163, in <module>
    main()
  File "pose_camera.py", line 159, in main
    run(run_inference, render_overlay)
  File "pose_camera.py", line 124, in run
    jpeg=args.jpeg
  File "<...>/project-posenet/gstreamer.py", line 368, in run_pipeline
    pipeline = GstPipeline(pipeline, inf_callback, render_callback, src_size)
  File "<...>/project-posenet/gstreamer.py", line 42, in __init__
    self.pipeline = Gst.parse_launch(pipeline)
GLib.Error: gst_parse_error: no element "glvideoflip" (1)

i have the same error, any solution?

Are you running Mendel Chef or Mendel Day ? Please be sure to reflash your board with the latest
Mendel release (Day) from here: https://coral.ai/docs/dev-board/get-started/
I just flashed a clean board and followed those instructions and the pose examples should work out of the box (be also sure to run install_requirements.sh after setting up the board and cloning this repo)

Feel free to reopen this issue if the issue persists.

Same issue, flashed the coral dev board last week with the latest image 5.0 Eagle (Jul 2020)
Isn't install_requirements.sh meant to deal with the dependencies? Definitely not working out of the box...

Same issue, flashed the coral dev board last week with the latest image 5.0 Eagle (Jul 2020)
Isn't install_requirements.sh meant to deal with the dependencies? Definitely not working out of the box...

hi,
i have the same error,
  File "/home/USER/Documents/project-posenet/gstreamer.py", line 42, in __init__
    self.pipeline = Gst.parse_launch(pipeline)
gi.repository.GLib.Error: gst_parse_error: no element "freezer" (1)

there is any solution ?


[Event Request] Database::"Warehouse Journal Line" - Function "Unit of Measure Code" -OnValidate - OnValidateUnitOfMeasureCodeOnBeforeValidateQuantity

We like to request the following Event:
Object: Database::"Warehouse Journal Line"
Function: "Unit of Measure Code" -OnValidate
Publisher Event: OnValidateUnitOfMeasureCodeOnBeforeValidateQuantity
Reason: To implement custom business logic
Where is the event placed:
trigger OnValidate()
var
IsHandled: Boolean;
begin
if not PhysInvtEntered then
TestField("Phys. Inventory", false);
if "Item No." <> '' then begin
TestField("Unit of Measure Code");
GetItemUnitOfMeasure;
"Qty. per Unit of Measure" := ItemUnitOfMeasure."Qty. per Unit of Measure";
CheckBin("Location Code", "From Bin Code", false);
CheckBin("Location Code", "To Bin Code", true);
end else
"Qty. per Unit of Measure" := 1;
IsHandled := false;
OnValidateUnitOfMeasureCodeOnBeforeValidateQuantity(Rec, IsHandled);
if not IsHandled then
Validate(Quantity);
end;
Event Code:
[IntegrationEvent(false, false)]
local procedure OnValidateUnitOfMeasureCodeOnBeforeValidateQuantity(var WarehouseJournalLine: Record "Warehouse Journal Line"; 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. We will update this issue with information about availability.


Mathematical Functions on Number Lists

Is your feature request related to a problem? Please describe.

There should be a list block that allows the user to carry out various mathematical functions on lists. For example, the block should add, subtract, multiply, divide the numbers present in the same positions and create a new list with the result.

Describe the solution you'd like

A list block that carries out mathematical functions for numbered lists.

Describe alternatives you've considered

Allowing number lists to be attached to the mathematical function block.

Additional context

This would help in making complex programs such as converting binary numbers to decimals.

After some consideration we decided against this. As lists and math operations should stay mostly based on the original Blockly setup.


no member named 'level'

Not sure if this is related to #163 or not.
At this step in the README.md
Build the Optech Taproot branch of bitcoind locally. See the build documentation (build-xxxx.md) in the Bitcoin Core repository docs directory for additional documentation on building bitcoind on Unix, macOS, or Windows.

Was following this https://github.com/bitcoin/bitcoin/blob/master/doc/build-osx.md got to the make
➜  bitcoin git:(a1d284e50) make
Making all in src
  CXX      wallet/libbitcoin_wallet_a-walletutil.o
wallet/walletutil.cpp:77:23: error: no member named 'level' in 'boost::filesystem::recursive_directory_iterator'
        } else if (it.level() == 0 && it->symlink_status().type() == fs::regular_file && IsBerkeleyBtree(it->path())) {
                   ~~ ^
1 error generated.
make[2]: *** [wallet/libbitcoin_wallet_a-walletutil.o] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all-recursive] Error 1

running into this issue in prep for the workshop.

this looks like a bug in MacOS building of bitcoin core.
try to build master of https://github.com/bitcoin/bitcoin and if you still get it please open an issue there :)

You’ll need to cherry pick two commits if you’re on macOS Catalina. See here:
https://gist.github.com/bitschmidty/882fd70beb3d6610d28cfe3fa57c7e0e#troubleshooting

Tried this, it didn't work for me.
The second commit is failing
➜  bitcoin git:(2c63baa6d) git cherry-pick 0bb33b5348dbddd65b88a7f00449c965562355d3 a64e97dd476bda7c7981979d045b0d06d6f7ce47

fatal: bad object a64e97dd476bda7c7981979d045b0d06d6f7ce47

make command still resulting in same error.

@miketwenty1 The commit is in the bitcoin/bitcoin repo so I think youll need to fetch that remote as well in order to apply that commit. Since you have the first one, it appears you just need to fetch to get the more recent commit.
https://github.com/bitcoin/bitcoin/commit/a64e97dd476bda7c7981979d045b0d06d6f7ce47


Uploader throws error Build has already finished, uploads rejected

Describe the bug
Codecov uploader throwed Error uploading to https://codecov.io: Error: There was an error fetching the storage URL during POST: 404 - Not Found - {'detail': ErrorDetail(string='Build has already finished, uploads rejected.', code='not_found')}
We have 2 independent tests run and reports are uploading independently. It's not strongly reproduced, but already happened in a few builds
<-- reports list  -->

codecov: OK
+ chmod +x codecov
+ ./codecov
[2022-04-07T23:09:49.279Z] ['info']
     _____          _
    / ____|        | |
   | |     ___   __| | ___  ___ _____   __
   | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
   | |___| (_) | (_| |  __/ (_| (_) \ V /
    \_____\___/ \__,_|\___|\___\___/ \_/

  Codecov report uploader 0.1.20
[2022-04-07T23:09:49.305Z] ['info'] => Project root located at: /root/code
[2022-04-07T23:09:49.309Z] ['info'] -> No token specified or token is empty
[2022-04-07T23:09:49.318Z] ['info'] Searching for coverage files...
[2022-04-07T23:09:50.862Z] ['info'] Warning: Some files located via search were excluded from upload.
[2022-04-07T23:09:50.862Z] ['info'] If Codecov did not locate your files, please review https://docs.codecov.com/docs/supported-report-formats
[2022-04-07T23:09:50.862Z] ['info'] => Found 7 possible coverage files:

<-- packages-->

[2022-04-07T23:09:50.912Z] ['info'] Detected CircleCI as the CI provider.
[2022-04-07T23:09:50.913Z] ['info'] Pinging Codecov: https://codecov.io/upload/v4?package=uploader-0.1.20&token=*******&branch=km-*****-notification&build=7*85&build_url=https%3A%2F%2Fcircleci.com%2Fgh%2F******%2F******-*****-****%2F79585&commit=9e51f5052***06dc5c5d4b&job=0&pr=&service=circleci&slug=******%2F******-***-**&name=&tag=&flags=&parent=
[2022-04-07T23:09:51.265Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io: Error: There was an error fetching the storage URL during POST: 404 - Not Found - {'detail': ErrorDetail(string='Build has already finished, uploads rejected.', code='not_found')}

another report, which runs in parallel was uploaded without errors.

@RingerJK This means that CircleCI's API told us that the build was completed. You can avoid this check by passing an upload token, this is only checked for tokenless uploads.
This message is from the Codecov server, not something that the uploader can control.


Add initial support for finalizeImage

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

Add option to provide a final script that modifies the chroot before creating an image
Change Log

Add option to provide a final script that modifies the chroot before creating an image

Does this affect the toolchain?
NO
Associated issues

#41439909

Test Methodology

Functional testing done: tested path with different number of scripts (no script, 1 script, multiple script paths), with and without args

We should have a section in the docs in formats/imageconfig.md for this.


implemented the settings page and made changes to some file

I have implemented the settings page and, i made the routing and implemented some of the Components in it, i also added Logout button to the Sidebar, since the Figma design have been updated and made changes to the FormInput in the Topbar.

Please let me know if you have any questions or concerns about these changes. I'm open to feedback and willing to make changes based on your input.

Awesome.  The speed is very good.
For codacy, propstyping on the concerned component will solve the issues. Form and SwitchForm.

Awesome job.
Many thanks Rasheed.
Keep it up !


refactor(mercury): optimize temp path generation

Depending on the platform, this pr reduces the time on the single function generate_temp_path by half (0.30% -> 0.15%) to 3/4 (~2% -> ~0.5%).

Preallocate enough space for PathBuf to prevent expensive reallocation which once happened every time.
Call _to_string() only once. I expected this optimization to be done by the compiler, but it appears not.
Replace to_string() by _to_string(). I don't like this hack very much, but it's 50% faster.

@el-ev  Amazing, 50% faster! I’m curious about how you tested it. I’d love to measure it on my own machine as well.

@el-ev Amazing, 50% faster! I’m curious about how you tested it. I’d love to measure it on my own machine as well.

I tested it in test_decode_large_file_stream with cargo flamegraph --unit-test mercury --root -r -- test_decode_large_file_stream. The actual performance may vary, but you can see the large realloc part eliminated.
cargo flamegraph

before
after

Maybe you are referring to the _to_string()? It is shown as such on my device.

In Release mode, is there still such a performance difference between _to_string() and to_string()?

In Release mode, is there still such a performance difference between _to_string() and to_string()?

cargo flamegraph --unit-test mercury --root -r -- test_decode_large_file_stream
                                             ^

Of course I won't do profiling under debug mode...

Amazing, but mass usage of _to_string() will become unbearable.

Although I don’t know why, it is indeed much faster. I tested it with the following code, and the difference was particularly noticeable during the first execution.
 to_string first time : 5.717µs
_to_string first time: 221ns
 to_string second time: 230ns
_to_string second time: 166ns
 to_string first time : 170ns
_to_string first time: 154ns
 to_string second time: 248ns
_to_string second time: 209ns
 to_string 1000 times: 187.449µs
_to_string 1000 times: 163.898µs

test code
    #[test]
    fn test_performance_of_to_string() {
        let test_cases = [
        "8ab686eafeb1f44702738c8b0f24f2567c36da6d",
        "9ab686eafeb1f44702738c8b0f24f2567c36da6d",
    ];

    for &hash_str in &test_cases {
        let sha1 = SHA1::from_str(hash_str).unwrap();
        time_it!(" to_string first time ", { sha1.to_string(); });
        time_it!("_to_string first time", { sha1._to_string(); });
        time_it!(" to_string second time", { sha1.to_string(); });
        time_it!("_to_string second time", { sha1._to_string(); });
    }

    time_it!(" to_string 1000 times",  {
        for i in 0..1000 {
            let sha_temp = SHA1::new(&format!("{}", i).bytes().collect());
            sha_temp.to_string();
        }
    });
    time_it!("_to_string 1000 times", {
        for i in 0..1000 {
            let sha_temp = SHA1::new(&format!("{}", i).bytes().collect());
            sha_temp._to_string();
        }
    });
    }


carthage: init at 0.34.0

Motivation for this change
Adding a package manager that is a popular tool when developing for Apple platforms.
Things done

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

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

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

This looks really fragile to maintain.

I based it on how Swift itself is built - https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/compilers/swift/default.nix.
However another approach would be to just allow Swift Package Manager to fetch all dependencies from git repositories as in original code before patching. This would simplify the derivation but we would lose caching that Nix provides in this case. I am not sure how big of a problem it is, maybe it makes more sense to keep the derivation simple.

Here's my shot at packaging carthage using code from https://euandreh.xyz/swift2nix.git:
https://github.com/bmwalters/nixpkgs/tree/carthage/pkgs/development/mobile/carthage/default.nix
It avoids hardcoding dependencies while still fetching them through nix. It has several unsolved problems before it's ready for nixpkgs though. For example it may be cleaner to rely on editing Package.swift to point to local checkouts of dependencies as you've done here and as is done in this other swift2nix: https://github.com/zachcoyle/swift2nix.


When will the simulation be added?

Hello, just want to ask when will the simulation be added?

Hello!
We currently have the model with ros2_control working, both in gazebo classic and gazebo harmonic, but we need to add the controllers for the movement of this one. I hope to be able to report in a few weeks that we have this up and running.
Regards.


Error while - Querying the index -Vector Search with Firestore

[REQUIRED] Step 2: Describe your configuration

Extension name: _<EMAIL_ADDRESS>Extension version: _ 0.0.3
Configuration values:

_ collection path: testembed
_ Default query limit(Optional): 1
_ Distance measure COSINE
_ Input field name: input
_ Output field name: embedding
_ Status field name: status
_ Cloud Functions location: us-central1

[REQUIRED] Step 3: Describe the problem
Receiving an Error while querying the index.
error: '{"error":{"message":"The function must be called while authenticated.","status":"UNAUTHENTICATED"}}'
executionId: x96sgykheo49
Steps to reproduce:
As per the instructions provided  in the How to use this extension section on the extension
Querying the index
*Important: Before you will be able to query the collection, firestore will have to build a vector index. This extension will trigger the basic index (no prefilters) upon installation or reconfiguration.
Once the index is created, you may query it either through a callable cloud function deployed by the extension:
gcloud functions --project examples-c0581 call ext-firestore-vector-search-queryCallable --data '{"data": {"query":"test"}}'
Or by adding a document to the collection _firestore-vector-search/index/queries with a query field and an (optional) limit field.
Expected result
Expected a query result for the searched query - "test"
Actual result
Received error using both the approaches
1)
gcloud functions --project examples-c0581 call ext-firestore-vector-search-queryCallable --data '{"data": {"query":"test"}}'
error: '{"error":{"message":"The function must be called while authenticated.","status":"UNAUTHENTICATED"}}'
executionId: x96sgykheo49
2) creating a query field under the collection _firestore-vector-search/index/queries
query "test"(string)
status(map)
textQuery(map)
createTime 11 May 2024 at 21:37:58 UTC-4 (timestamp)
startTime 11 May 2024 at 21:38:03 UTC-4 (timestamp)
state "ERROR" (string)
updateTime 11 May 2024 at 21:38:07 UTC-4

I had to manually create an index using the following command, and the approach with creating a query field under the collection _firestore-vector-search/index/queries started working.
gcloud alpha firestore indexes composite create --project=examples-c0581 --collection-group=testembed --query-scope=COLLECTION --field-config=vector-config='{"dimension":"768","flat": "{}"}',field-path=embedding
However the first approach with queryCallable still throws the error. "The function must be called while authenticated."

I found some explanation of the problem!
https://cloud.google.com/functions/docs/securing/managing-access-iam#allowing_unauthenticated_function_invocation

Hm so there seem to be two issues here, the index isn't being created initially, and then the docs are definitely missing the authentication instructions (the function does require some kind of auth)
Looking into this today

Would be it be helpful to have an optional parameter to disable the auth check by the way?

OK it's actually quite annoying. We need to authenticate through firebase and get an id token if we want to call the function through the CLI.
I should update the docs to reflect this, and provide a different snippet.

Were you using Vertex AI for embeddings by any chance? I think I've found the index problem

Does  Firestore's firestore.indexes.json support configuring vector indexes directly?


[DM-35881] Document user private groups and primary GIDs

Add some explicit documentation of user private groups, since that
wasn't fleshed out very well.  Document that they may not be in use
for OpenID Connect deployments.  Document the new primary GID field
and mention that it's required for the Notebook Aspect.  Discuss how
UID and GID are used when spawning lab pods.

On further consideration, I don't think this is a good use of your time to review. I've been merging changes to these tech notes without review and there will be a lot of them, so I'm going to go ahead and merge this. Sorry about the extra notification noise! (And of course if you do have feedback it's always welcome and I'm happy to go back and fix things.)


Move dtor information from ResourceAny into a Store

This commit moves the internals of the ResourceAny type related to destructors into the auxiliary data within a Store. This avoids the need to carry around pointers with ResourceAny and additionally makes it easier to work with.
As part of this commit this also updates the behavior of ResourceAny::try_from_resource to no longer need an InstancePre and type information. This was required originally to get ResourceAny::resource_drop working to drop the host resource. In retrospect I'm not sure if this was the best goal to achieve because Resource<T> already has no destructor support and one of the more common use cases for the conversion is to simply turn around and give it back to a component where a component already has enough destructor information.
In the end this should make both ResourceAny and Resource<T> pretty close to "just an index" which is easier to reason about when working with resources than carrying additional pointers.

cc @rvolosatovs as this is touching on the work of https://github.com/bytecodealliance/wasmtime/pull/7688

Good points! And sorry again for not realizing this is probably the best way to go sooner, I realize this is a bit of a runaround with the API :(


[1.10.0] Smithing Table Recipe Book

Closes #37

[x] Cycle armour over time
[x] Group by armour type
[x] Centered interface support
[x] Pinning support
[ ] Instant Craft
[x] Ghost recipes
[x] Ghost result
[x] Armour stand preview
[x] Remove unnecessary AW's
[ ] Fix clicking trim recipe twice crafts

Progress so far

Item transformations are completely functional

Technically instant craft can be implemented in this screen as well

I'll do some testing this weekend.
One thing I might change is the ghost recipe. From a maintainability standpoint it might be better to make BrewingGhostRecipe the general purpose ghost recipe including all the features needed. That's why I originally named it MyGhostRecipe and not just to do with Brewing.
IIRC BrewingGhostRecipe doesn't do anything potion specific. It just gives you more functions/features to use.

Yeah your right about ghost recipes
I just wanna completely rewrite the whole mod in kotlin every time I update it lol

I'd recommend merging the code from SmithingGhostRecipe to BrewingGhostRecipe and naming it back to MyGhostRecipe as hovering over the ghost recipe renders the tooltip even if the slot is filled. Causing two tooltips to be rendered. MyGhostRecipe has a rendering predicate for this purpose. This can be caused by clicking on a recipe you can't craft and then clicking on one you can.
I've noticed a few other bugs:
The ghost recipe doesn't disappear when clicked, it instead blocks items from being placed.
It also seems to randomly craft items when I spam between the options.

I'd recommend merging the code from SmithingGhostRecipe to BrewingGhostRecipe and naming it back to MyGhostRecipe as hovering over the ghost recipe renders the tooltip even if the slot is filled. Causing two tooltips to be rendered. MyGhostRecipe has a rendering predicate for this purpose. This can be caused by clicking on a recipe you can't craft and then clicking on one you can.

vanilla bug

I think for right now vanilla & the mod has bugs with ghosts and id like to fix them all but probably at a later date


Re-download docker-compose binary when compose_version is changed

To support this I think it will make sense to download docker-compose to a file that has the version number in it and symlink the current version. Then on run it can check for the presence of the specific version and download it if necessary, and then change the symlink as well.

@teohhanhui This was added in #24 and is now released in version 1.0.0. Hope it's helpful!


Add file_name to Audio and Video

A new field file_name was added to Audio and Video classes in Telegram Bot Api 5.0.

Added the field file_name to the classes Audio and Video, containing the name of the original file.

https://core.telegram.org/bots/api#audio
https://core.telegram.org/bots/api#video
We should add support for this new field.

Closed by #179


Plugin not showing

I was able to run the install without errors, but I don't see the plugin listed in data sources or plugins.

I also see this in the logs despite the fact that I've added it the list of unsigned plugins in grafana.ini
t=2022-01-26T14:33:16+0000 lvl=warn msg="Skipping loading plugin due to problem with signature" logger=plugin.loader pluginID=grafana-mqtt-datasource status=unsigned

Since this plugin is not signed, as the log entry suggests, you need to add an exception to the plugin security in your /etc/grafana/grafana.ini file.
Scroll down to the [plugins] section.  Uncomment and update the allow_loading_unsigned_plugins entry to
allow_loading_unsigned_plugins = grafana-mqtt-datasource


Don't use Red Hat Fedora as Favicon

Did we have one before? I don't think we should use the Fedora, as this is an upstream project website, not a Red Hat property (the Fedora is fine in the header and footer, as it's about the sponsoring relationship there).
@mdrillin @uidoyen @indraraj

Pending PR #580

No, we didn't have a favicon in the existing one. How about adding the debezium logo(one without text) as a favicon? @gunnarmorling

Yes, that'd be nice. I'm just not sure how recognizeable it'd be in the small size. Perhaps you can give it a try and share here how it looks?

FYI We used the same for the UI POC also

FYI We used the same for the UI POC also

Ah, sweet, that looks great! Let's do the same then here. Thanks!

Done.


Eloquent update() method not using mass-assignment rules

I realized that when I run my query with updateOrCreate() method I am getting mass-assignment error which is fine because the array I am trying to assign contains values which are not defined as fillable. ($fillable is an empty array)
However, if I use update() method it updates the row without any complaints, ignoring the fillable/hidden fields. I believe this is because update is not using the fill() method.
I think this is a bug because I can pass an array to the update() function from a form, for example updating user details and the user can put extra fields to it. This is exactly what the mass-assignment rules are but update() method does not obey the rules.
Tested on Laravel 5.2

@yurtesen update() method is fillable look here https://github.com/illuminate/database/blob/master/Eloquent/Model.php#L436

You are right, but I think I should have given more info. Because that is never called in my case (as far as I can tell).
I have 2 tables, 'users' and 'activation_tokens', each user can have 1 token. So User model has

    public function ActivationToken()
    {
        return $this->hasOne(ActivationToken::class);
    }

I am updating the token with the following line of code:
            $user->ActivationToken()->update(['token' => $this->getNewTokenString()]);
That line updates the token perfectly but the update fails with mass assignment exception if I use the following:
            $user->ActivationToken()->updateOrCreate(['token' => $this->getNewTokenString()]);
I tried to follow what is happening, first Eloquent\Relations\HasOneOrMany,php's update() function is called, this calls update in Eloquent\Builder.php from there it goes to Illuminate\Database\Query\Builder.php's update() method, from there to Connection.php and run the update query and does not do any filling whatsoever.... If I understood it correctly :) on the other hand updateOrCreate() tries to create an instance and fill() it.
Am I wrong? it does not call fillable in this situation?

Correct, it does not use fillable as you are not updating a 'model instance'. You are calling update on a query builder, which deals with records not models.

I don't quite get what you mean. Do you mean that even though I am working on a "model" User() and "model" ActivationToken(), I am updating a model instance when I call updateOrCreate()  and then the same thing is all of a sudden query builder when I call update()? What in my code makes you think that I am not updating a model instance? How come my object is all of a sudden a Model and at the same time a query builder object?
It is more like eloquent is making a mistake by not using fillable in update() method in HasOneOrMany,php relation. Shouldn't relationships on Models to begin with?

Could you tried this to see it work or not:
$user->ActivationToken->updateOrCreate(['token' => $this->getNewTokenString()]);

Well the line you put works the same way as the one with ActivationToken()
But if I use:
$user->ActivationToken->update(['token' => $this->getNewTokenString()]);
I get:
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'id' in 'where clause' (SQL: update activation_tokenssettoken= LONG_TOKEN_STRING whereid is null)
So the result is that there is some discrepancy on how these functions work?

That is update on a model and it looks like that model hasn't been told that there isn't an id primary key to use.

@lagbox  I may be mixing up things, you are probably right. But how eloquent is working is very inconsistent, and there is a bug. Maybe my report was not clear due to me mixing things up.
For example:
$user->ActivationToken->update(...) -> uses model//instance/fillablable
from Eloquent\Model but it DOES NOT work at all. (see my previous post above for error)
$user->ActivationToken->updateOrCreate(...) -> uses model/instance/fillable
from Database\Eloquent\Builder and it updates the token normally.
$user->ActivationToken()->update(...) -> uses query builder
from Relations\HasOneOrMany DOES NOT use fillable
$user->ActivationToken()->updateOrCreate(...) -> uses instance/fillable
from Relations\HasOneOrMany
If this is not a bug, can you just explain me the answers to these 2 questions below please?:

Why $user->ActivationToken->update(...) does not work at all?

If as you say, ActivationToken()->updateOrCreate(...) is using the query builder, can you tell:

Why is it using fillable? In your own explanation, the fact that it is using query builder should make it exempt from using fillable. No?

Thanks

@GrahamCampbell Can you please explain why you closed the issue without an explanation?

Because there is no issue here. This is the expected behavior as explained.


在mac的M2pro上运行chinese-alpaca-2-13b-16k-hf报错

提交前必须检查以下项目

[X] 请确保使用的是仓库最新代码（git pull），一些问题已被解决和修复。
[X] 我已阅读项目文档和FAQ章节并且已在Issue中对问题进行了搜索，没有找到相似问题和解决方案。
[X] 第三方插件问题：例如llama.cpp、LangChain、text-generation-webui等，同时建议到对应的项目中查找解决方案。

问题类型
模型量化和部署
基础模型
Chinese-Alpaca-2-16K (7B/13B)
操作系统
macOS
详细描述问题
convert模型以及量化模型的时候都正常通过了。但是在llama.cpp中执行./main的时候报了一下的错误。
建议你checkout之前的commit（比如2-3天前的），然后重新编译。

llama.cpp那边也有人碰到类似的问题，可能是新的commit里有bug。 建议你checkout之前的commit（比如2-3天前的），然后重新编译。

回退后已经解决。感谢


Add initial Talos II support to skiboot

This series adds intiial support for Talos II to skiboot.  It includes:

Descriptor copy / rename
LPC IPL status patches
MTD fixups

Thanks! Could you send these to the skiboot mailing list? Generally we do reviews there rather than github PRs.  For example, the last patch marks things as read only, but doesn't mention why you are doing that.
What is happening can reconstructed later, but the why can't, so it's best to put this in the commit message.

@mikey Uploaded, https://lists.ozlabs.org/pipermail/skiboot/2019-April/014094.html

Merged in master after ML review.


Is the device_trackers not work?

Im very love this folder structure, it's the elegance of breakthrough the planet!
I see the phone device_tracker.jonathans_iphone_x in /entities/persons/jon.yaml. but I dont see anything in /entities/device_trackers. Is it working correct?

The stricture of this config was not developed by me - its similar to the one from https://github.com/frenck/home-assistant-config
The iphone device tracker is added via the home assistant app automatically.

The stricture of this config was not developed by me - its similar to the one from https://github.com/frenck/home-assistant-config
The iphone device tracker is added via the home assistant app automatically.


Will there be a dependency update for version 3 of React Scripts?

I wonder if react-app-rewired will be updated to version 3 of React Scripts.
I see you're still using 2.

React Scripts version 4.0.0 is coming soon too.

I was using rewired with CRA version 3 for a while without issues. But CRA 4.0.0 came out a few hours ago and broke it.

ok....  i got some information when i upgrade react-srcript to v4.0 ,

so, i have to  downgrade  react-script 3.x

yes updating to 4 is also breaking my build

Yes, same error for me rewiring eslint in combination with react-scripts 4...

Same

Was only really using this to import stuff outside of src, have migrated to using yarn workspaces which is a much better fit for my use case. 👍🏼

Closing this --- should be compatible up to v5 now
https://github.com/timarney/react-app-rewired/pull/530#issuecomment-1001025219


On iOS when switching between apps only half the Flutter app is shown

Steps to Reproduce
Double tap the home button to switch between running apps on iOS. Only half of the flutter app is shown. See screenshot:

Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.4 16E195, locale en-US, channel master)
    • Flutter at /Users/vik/Projects/flutter
    • Framework revision 7397bb9071 (2 days ago), 2017-05-31 12:43:18 -0700
    • Engine revision 75c74dc463
    • Tools Dart version 1.24.0-dev.3.0

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.2)
    • Android SDK at /Users/vik/Library/Android/sdk
    • Platform android-25, build-tools 25.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_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.2, Build version 8E2002
    • ios-deploy 1.9.1
    • CocoaPods version 1.2.0

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Gradle version 3.2
    • Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2016.3.4)
    • Dart plugin version 163.13137
    • Flutter plugin version <IP_ADDRESS>

[✓] Connected devices
    • iPhone SE • B19052F4-91FF-48C7-85B4-26C1D2D1A148 • ios • iOS 10.3 (simulator)

Also interesting that there is a shadow below the bottom bar.  I suspect we're doing unnecessary work to draw that.

I may be responsible for that shadow :P

@cbracken is the right person to know.  Unclear if this is a regression.

Do you have more specific repro instructions? The height of the gap at the bottom looks suspiciously like the keyboard region height, but I'm unable to repro on my end either with keyboard open (I see the keyboard in the keyboard frame) or closed (behaves as expected).
Looks like you're using gBoard? I'll give it a go with gBoard as well and see if that changes things.

I haven't seen this lately, even with the exact same build. Before it was reproducible every single time. Will update this issue if I can find out what the trigger is.

Please do reopen the bug if you see this again. We can't repro.


add post conditional actions

Add additional options  to pipiline_stop action which should be run on build success/fail or always.
Added backward compatibility in case if non of new options (on_success, on_fail, always) are provided.
Signed-off-by: Andrii Holovchenko<EMAIL_ADDRESS>

agreed to address here: https://github.com/Mellanox/ci-demo/commit/9ad409b6ef64ee2cae051fce5d44c135408fdc0a


Use provisioning scripts in CSEScriptsPackage.

Since aks-engine has been marked as deprecated in docs: Update AKS-Engine project status to deprecated . We decide to copy Windows provisioning scripts from https://github.com/Azure/aks-engine/tree/master/staging/provisioning/windows  to AgentBaker and we will include these scripts in AKS CSE scripts package. cc @marosset  @jsturtevant
Will set ProvisioningScriptsPackageUrl to "" with new CSE pacakge.

@ShiqianTao Building AKS Windows VHD with WS2022 failed for that a new omsagent-win image was added in omsagent march 11 2022 release


[Debugger] For connectors, complex values should be renamed to a proper indicator.

more information here https://github.com/ballerinalang/composer/issues/2984

Can you please elaborate on this issue please. An example would be great.

In the given sample req and res are empty structs (earlier it was listed as complex_value).
We could include some information in req to ease debugging process.

@lafernando closing this since this is a exact duplicate of https://github.com/ballerinalang/ballerina/issues/4124