use pairs on std::map<string, int> but get the wrong key

I want to use the follow code to traverse the std::map, print the key and value of the std::map.
I expect it to print:
  key_1 10
  key_2 20

but actually it print:
 1 10
 2 20

the key is not I expect.
Is the implement of sol2::pairs on std::map wrong ?
Any way, How can I  traverse std::map and print the key and vlaue correctly?
I use Lua5.3.4 and Visual Studio 2015 on window 7.
#include <map>
#include <string>
int main()
{
	std::map<std::string, int> try_map = {
		{ "key_1", 10 },
		{ "key_2", 20 },
	};

	sol::state l;
	l.open_libraries(sol::lib::base);
	l.set("try_map", try_map);
	l.script(R"(
		for key, val in pairs(try_map) do
			print(key, val)
		end
	)");
}

I actually cannot reproduce this bug. It works fine for me:

Make sure you update sol2 to the latest version.


Testing Systems? (this really should be a Discussions thread)

(This really should be a Discussions thread, especially as we don't have a specific mailing list for non-issue discussions)
So, y'all know that I've been testing PRs on the following systems:

Windows 10: Via PyCharm, via PSCore, in Cygwin
Ubuntu 18.04: on WSL 1.0, on VirtualBox
Ubuntu 20.04: on VirtualBox
FreeBSD 12.2: on VirtualBox

I'm considering dropping the Ubuntu-on-VirtualBox test systems because they practically duplicate the GitHub Actions (GA) test systems anyways. In their place, I'll implement the following:

OpenSUSE Leap 15.2: on VirtualBox
AlmaLinux (when it comes out): on VirtualBox

Note: I'll keep the "Ubuntu 18.04 on WSL" test system because tests have conclusively proven that WSL have some "gotchas" that need to be taken into account (e.g., this test completely fails on WSL).
Do you think I need to implement additional testing systems? Feel free to suggest additional test systems, on one condition: The suggested systems must NOT be a rolling distro, and preferably it has some kind of "LTS" edition.
(OpenSUSE Leap does not seem to have an "LTS" edition, but I'm still including it because a survey by CloudLinux seem to indicate that quite a sizable number of respondents are thinking of switching to OpenSUSE.)

If you noticed, in 7fce786 I dropped mention of AlmaLinux from README.
Reason being that for unknown reasons it runs tests really slowly, far beyond what's acceptable. Also, tests seem extremely flakey.
I've asked a question on Reddit (currently AlmaLinux's community forum as said by AlmaLinux.org themselves), and until I get an actionable (and effective) answer, I'll leave out AlmaLinux from the list.

I have half the idea of referring "Windows 10" as "WinX" 😜
As in, yanno... "WinX Club"? 😁


[AIRFLOW-300] Add Google Pubsub hook and operator

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

AIRFLOW-300

Only publishing and topic creation are included. Topic consumption
was explicitly not included in this feature request.
Testing Done:
Unit tests are included and pass locally.
$ nosetests tests.contrib.operators.pubsub_operator
Ran 3 tests in 0.005s
OK
$ nosetests tests.contrib.hooks.gcp_pubsub_hook
Ran 4 tests in 0.007s
OK

Nice! I think this is something for @alexvanboxel or @criccomini to review.

I'm sorry, can you please elaborate on what's incorrect about my commit message? I tried very hard to abide by the rules laid out in

@wwlian just remove all the info about the test's run in the commit message. This is information that you can supply with the PR message. The commit message is set in stone, with the PR message is for the reviewers.
I'll do a passover, but it's for 1.8.1 anyway so better to approve it after the release.

Gotcha, thanks for the clarification. Done :)

@alexvanboxel  A friendly bump.

Sorry for the delay. Let me give it a shot.

@wwlian just saw some activity on the mailing list github log. If you rebase, I can do a review and merge this for you.

Sorry for the long delay. PTAL

Codecov Report

Merging #2036 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2036      +/-   ##
==========================================
+ Coverage   69.27%   69.28%   +<.01%
==========================================
  Files         146      146
  Lines       11230    11230
==========================================
+ Hits         7780     7781       +1
+ Misses       3450     3449       -1

Impacted Files
Coverage Δ

airflow/models.py
87.31% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


End of support Azure Database for MariaDB for creating over Azure Portal / CLI

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [X ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

In alignment with the Azure Database for MariaDB retirement announcement, we stopped support for creating MariaDB instances via the Azure portal or CLI as of March 19, 2024.
https://learn.microsoft.com/en-us/azure/mariadb/whats-happening-to-mariadb

Any log messages given by the failure

Deployment Error Details: InvalidElasticServerType: The provided server type value 'Azure Database for MariaDB - Single Server' is invalid.

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Having the same issue

Changing this line to 'true' resolved it for me

MariaDB service has been removed with Azure Database for MySQL flexible server.


Updates (::) documentation: Adds one more example.

This makes clear that to call the (::) as a function you need to wrap the :: in parenthesis by adding another example.
The example uses (::) as a function so it shows how it would be used with a pipe operator.
--This example doesn't use the pipe operator itself |> so it's much simpler than the previous one I submitted.

@jvoigtlaender Janis, this one is another documentation improvement. Please consider.

I'd rather not have this in the docs here. Beginners are likely to see 1 :: [2,3] pretty early on with Elm, whereas knowing that infix operators can be used as functions is kind of advanced. I suspect that adding this will be more confusing for beginners who do not need to know about this feature that is not directly related to cons.

There is one reason I disagree.
The pipe operator.
If you don't know you can do that you can't chain cons in a pipe operation.
It is rather advanced so maybe we can put it in another place?
Sent from my mobile.

On Apr 27, 2016, at 8:28 PM, Evan Czaplicki<EMAIL_ADDRESS>wrote:
I'd rather not have this in the docs here. Beginners are likely to see 1 :: [2,3] pretty early on with Elm, whereas knowing that infix operators can be used as functions is kind of advanced. I suspect that adding this will be more confusing for beginners who do not need to know about this feature that is not directly related to cons.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I mean, it should be in the docs somewhere that you can use infix operators as functions. Not here though.


Windows fatal exception code 0xc06d007e when running simple numpy code

To avoid replicating the issue I initially reported on numpy's GitHut, please see https://github.com/numpy/numpy/issues/25809
Because another person reported having the same issue with matplotlib, and was successful in running Spyder with version 5.4, I'm assuming it is an issue with the new Spyder 5.5.1.
Note: all the tests I described on the issue were done on dedicated environments - i.e. very uniquely it was due to a corrupt installation.

Hey @Rational-IM, thanks for reporting. This seems a duplicate of issue #21749, which will be fixed in our next version (5.5.2), to be released in a couple of weeks.
Could you test that starting Spyder from the command line solves the issue for you? That's the workaround we can offer you until 5.5.2 is released.

Sorry, I just saw that you mentioned that already in the Numpy issue you referenced above: https://github.com/numpy/numpy/issues/25809#issuecomment-1945201830
So, I'm going to close this issue as duplicate of #21749.


[✨] How to distinguish between link preload and page visit

Is your feature request related to a problem?
I'm trying to collect some analytics when a certain page is visited. I'm storing an event every time a routeLoader is executed, but when a Link to that page is hovered, the routeLoader is executed without the user actually visiting the page
Describe the solution you'd like
Be able to distinguish if the current request is a preload or an actual visit to the page, in the routeLoader
Describe alternatives you've considered
Calling a server function from a useVisibleTask$ but seems overkilling
Additional context
No response

assigned to @yousefmarey12 (can't do it directly with github for some reason atm) it might be best to send a prefetch header with prefetch calls to detect when requests are only prefetching

@juanpmarin Don't worry, I've been working on the issue day and night. Basically, I do not know if it's possible to make an accurate solution. If prefetching is true, The routeloader is executed the second the user hovers over the <Link>. I would assume you would like to consider the case as in what if the user hovers for 5 minutes on a link and then clicks on it. By then, the preload would have finished but the user at the end of the day, the user did visit the link.

We can try working with the onClick$ event handler so it will set the  x-preload header to be false if the request is still ongoing and the user clicked on the event. But what if the request finished and then the user clicked? I just don't want to give you wrong analytics. That's just my update.

Thanks @yousefmarey12 !
@juanpmarin we'll evaluate a proper solution for it after V2.
In the meantime, instead of using a visible task, what do you think about using a regular task that calls a server$ function instead? (in case you analytics logic is server side only)
Something like:
const sendAnalytics = server$(() => {
  console.log('sending');
});

export default component$(() => {

  useTask$(() => {
    sendAnalytics();
  });
  ...

That way no javascript will run on initial page load (unlike with visible task which "resumes" and executes its javascript on page load)

We moved this issue to qwik-evolution repo to create a RFC discussion for this.
Here is our Qwik RFC process thanks.


Added basic git rename-branch

@kevgo @charlierudolph @allewun
Still need to support undo correctly. Closes #474
Is there a way to generate the man pages? Or are they created manually?

Seriously cool feature, @ricmatsui! Branch names encounter a good amount of technical drift, and being able to rename them at any time will be super useful! Thanks for taking this on!

The man pages are created manually at this point. We were thinking about generating them somehow, but haven't gotten around to invest the time into finding tool support and setting all this up. Writing man pages by hand is interesting though, not something one does every day, and a great chance to see how they work.

Sounds good, thanks. I have addressed the comments and I think we can have it abort if the tracking branch is not in sync for now so no work is lost. In a later pull request we can add merging/rebasing automatically as well.
Do you think implementing undo by rename-branching again would work? One of the steps is a push, so it won't undo completely using the undo_steps_for.

I think making this undoable would be very usefule and consistent. And it could easily be made possible if we implement renaming branches following these steps:

rename the branch locally
remove existing tracking branch (if one exists)
add (the new) tracking branch
add the new non-feature branch name
remove the old non-feature branch name

Each of these commands has a simple and easy way of undoing it, which can de defined in undo_steps_for.
What do you think?

Yes, sorry I wasn't very clear. The issue I'm having is push has this undo step:
function undo_steps_for_push {
  echo "skip_current_branch_steps $UNDO_STEPS_FILE"
}

Which basically causes the destination branch to still exist locally and remotely after the undo if there is a tracking branch since when the rename occurs the destination branch is checked out and pushed. What do you think of having another reversible_push command or something similar that doesn't have this undo step?

Yes, I think we need to separate the push command into two: one called create_tracking_branch that creates a new remote branch (and is undone by deleting the remote branch), and one that simply pushes updates to an already existing tracking branch. Which of these command is used has to be determined based on whether the tracking branch already exists.
The fact that both commands are implemented differently in Git is another indicator that these really are two different commands.
I have created #519 to track and implement this separately from this change. I think we can implement the undo for git-rename-branch in a separate PR as well, to keep things simple and manageable here.

Okay, this looks good to go from my end once the remaining few comments are addressed and if you decide to implement the proposed change in behavior in a separate PR. Nice work!

:rocket:


neonvm: Use the right file extension for iso images

@sharnoff afaik this doesn't interfere with live migration. Once we attach a disk and mount on the VM, it'll be there over live migrations.

this doesn't interfere with live migration. Once we attach a disk and mount on the VM, it'll be there over live migrations.

@shayanh BTW, how does migration with disks actually work?
I know there is virsh migrate  --copy-storage-all, but I am not sure how it works in our case/on QEMU level. Does it have to run block jobs for every disk attached?
@cicdteam Do you know?

@Omrigan I don't know the details.

@cicdteam thanks. I guess those are deprecated, but can be replaced 1:1 with the migration capability, so no need to tinker with blockjobs.


add code threshold for highlighting

Starter Task 1

Add option to highlight plugin to stop running highlighting after certain code length threshold
Threshold currently set to 5

@lateefazeez Maybe just rebase this branch onto main again and then push it to remote? For some reason the E2E test suite doesn't to seem to have run completely.


Reduce block size for compute gradient in SVM

Description
Please include a summary of the change. For large or complex changes please include enough information to introduce your change and explain motivation for it.
Changes proposed in this pull request:

/intelci: run


Improve error message when feeding a non-y4m input

Currently rav1e gives this:
The application panicked (crashed).
  cannot decode the input: ParseError
in src/libcore/result.rs, line 1165
thread: main

With the merged PR above on the y4m repo new error messages will look like:
degausser on mac in ~/code/rav1e/target/debug
[deg4uss3r/rav1e.git:master]> ./rav1e video.mp4 --output test.ivf
Backtrace (most recent call first):
  File "rust:src/libcore/result.rs", line 983, in core::result::Result<T,E>::expect
  File "src/bin/rav1e.rs", line 349, in rav1e::run
  File "src/bin/rav1e.rs", line 264, in rav1e::main

The application panicked (crashed).
  cannot decode the input: ParseError(Error parsing y4m header)
in src/libcore/result.rs, line 1188
thread: main

Currently putting in a WIP PR that will update rav1e with the new y4m changes, but should not be merged until y4m is bumped with the new error changes.


Release 0.4.0

Requirement

[x] Update giganto-client version to 0.20.0
[x] Update review-protocol version to 0.6.0

Checklist

[x] Submit PR
[x] Tag annotated with version number
[x] Push tags

Since this issue has been resolved, I will now close it.


[Rendering]Facing issue when the adaptive card submit button click data is getting lost from the card

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/7455
Details from original post:
Target Platforms
Other
SDK Version
1.3
Application Name
Microsoft teams
Problem Description
i have created a adaptive card to fetch data from database it working fine, but the issue here is once we fill the data in adaptive input field and clicks submit the data is getting lost from card. how to maintain the adaptive card data through out the teams.
Screenshots
No response
Card JSON
{
    "type": "AdaptiveCard",
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.3",
    "body": [
        {
            "type": "TextBlock",
            "text": "code",
            "wrap": true,
            "id": "4"
        },
        {
            "type": "Input.Text",
            "placeholder": "Placeholder text",
            "id": "1"
        },
        {
            "type": "TextBlock",
            "wrap": true,
            "id": "2",
            "text": "location"
        },
        {
            "type": "Input.Text",
            "placeholder": "Placeholder text",
            "id": "3"
        },
        {
            "type": "ActionSet",
            "actions": [
                {
                    "type": "Action.Submit",
                    "title": "Action.Submit"
                }
            ]
        }
    ]
}

Sample Code Language
No response
Sample Code
No response---
Document Details
⚠ Do not edit this section.It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

@nameissainani - We are investigating your query. We will get back to you once we have any update.

Sure, thanks for letting me know.
On Tue, 7 Jun 2022 at 1:34 PM, Nivedipa-MSFT @.***>
wrote:

@nameissainani https://github.com/nameissainani - We are investigating
your query. We will get back to you once we have any update.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/msteams-docs/issues/6113#issuecomment-1148333844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN3LSEO6ITZVAIVFJ57XBDTVN37AHANCNFSM5YAUYRJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@nameissainani - This is the by design behaviour. After submit action the data will be lost. For workaround you have to saved Adaptive Card input values in submit activity.


HDDS-9524. Clean up wait for leader OM in integration tests

What changes were proposed in this pull request?
MiniOzoneHAClusterImpl:
getOMLeader() returns current OM leader.
getOMLeader(boolean) optionally waits until OM leader is elected
There are some tests that implement the same "wait for leader OM" logic using the non-waiting getOMLeader().
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestAddRemoveOzoneManager.java
209:    GenericTestUtils.waitFor(() -> cluster.getOMLeader() != null, 500, 30000);
437:    GenericTestUtils.waitFor(() -> cluster.getOMLeader() != null, 500, 30000);

hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerHA.java
466:        getCluster().getOMLeader() != null, 500, timeout);

hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerHASnapshot.java
126:    await(120_000, 100, () -> cluster.getOMLeader() != null);
253:    await(180_000, 100, () -> cluster.getOMLeader() != null);

Goals of this task:
rename getOMLeader(boolean) to waitForLeaderOM() (the method is only called with true)
make waitForLeaderOM() public
replace duplicated wait logic in tests with a call to waitForLeaderOM()
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9524
How was this patch tested?
Made changes and ran unit test.

Thanks @david1859168 for the patch, @aswinshakil for the review.


idea?

Great toold! I have no issue, just have an idea to upload custom email template. Would that be difficult to build?

That you just send or "Edit"?

Lets say I create an html page. It would be great to use my html template in PhishMailer.

Well, just create your own html page and send it via the tool ?


cargo-leptos: 0.2.1 -> 0.2.2

Description of changes
Changelog: https://github.com/leptos-rs/cargo-leptos/releases/tag/0.2.2
Closes #266789

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Breaking for me with this:

Something I'm missing?

Pretty sure it's got an old Cargo.lock, I've merged a PR to fix it, but I'm not quite sure what to do for Nix

Pretty sure it's got an old Cargo.lock, I've merged a PR to fix it, but I'm not quite sure what to do for Nix

Can the Nix package also not use the updated Cargo.lock that you merged?

There's no reason it can't but I think I'd either have to cut another release or see if Nix lets me edit a merged package. Which I must admit I don't know the Nix policy for
On Thu, Nov 16, 2023, at 8:22 PM, Saikat Das wrote:

Pretty sure it's got an old Cargo.lock, I've merged a PR to fix it, but I'm not quite sure what to do for Nix

Can the Nix package also not use the updated Cargo.lock that you merged?
—
Reply to this email directly, view it on GitHub https://github.com/NixOS/nixpkgs/pull/267198#issuecomment-1815726695, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVBTCJJYWNKYZFF2VRJWN3YE3Q6VAVCNFSM6AAAAAA7JAJWVOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJVG4ZDMNRZGU.
You are receiving this because your review was requested.Message ID: @.***>

There's no reason it can't but I think I'd either have to cut another release or see if Nix lets me edit a merged package. Which I must admit I don't know the Nix policy for
…
On Thu, Nov 16, 2023, at 8:22 PM, Saikat Das wrote: > Pretty sure it's got an old Cargo.lock, I've merged a PR to fix it, but I'm not quite sure what to do for Nix > Can the Nix package also not use the updated Cargo.lock that you merged? — Reply to this email directly, view it on GitHub <#267198 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVBTCJJYWNKYZFF2VRJWN3YE3Q6VAVCNFSM6AAAAAA7JAJWVOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJVG4ZDMNRZGU. You are receiving this because your review was requested.Message ID: @.***>

Sounds like a new release to me 🙈

The trouble is I'm not a huge fan of pushing a release for non Nix users with no substantial upgrades. But it might be the best option
On Thu, Nov 16, 2023, at 9:13 PM, Saikat Das wrote:

There's no reason it can't but I think I'd either have to cut another release or see if Nix lets me edit a merged package. Which I must admit I don't know the Nix policy for
… <#>
On Thu, Nov 16, 2023, at 8:22 PM, Saikat Das wrote: > Pretty sure it's got an old Cargo.lock, I've merged a PR to fix it, but I'm not quite sure what to do for Nix > Can the Nix package also not use the updated Cargo.lock that you merged? — Reply to this email directly, view it on GitHub <#267198 (comment) https://github.com/NixOS/nixpkgs/pull/267198#issuecomment-1815726695>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVBTCJJYWNKYZFF2VRJWN3YE3Q6VAVCNFSM6AAAAAA7JAJWVOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJVG4ZDMNRZGU. You are receiving this because your review was requested.Message ID: @.***>

Sounds like a new release to me 🙈
—
Reply to this email directly, view it on GitHub https://github.com/NixOS/nixpkgs/pull/267198#issuecomment-1815758978, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVBTCJR327BUFCXYLWQLELYE3W57AVCNFSM6AAAAAA7JAJWVOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJVG42TQOJXHA.
You are receiving this because your review was requested.Message ID: @.***>

I've made another PR that fixes the lockfile issue here #268036


Avoid using generic css variable names that conflict with popular framework

Describe the bug 📝
I can see that in this commit some css variables have been introduced.
The issue with the chosen name is that it conflicts with popular frameworks, like angular material: in fact when updating to openbim-components 1.5.0 we noticed changes in our colors, coming from the following definitions:
:root {
  --primary-color: #BCF124;
  --secondary-color: #6528D7;
  /* TODO: Probably this needs to be called background-color */
  --secondary-color-100: #000000bb;
  --secondary-color-120: #1a2128;
}

This definitions end up in the head section of the html page

and as a result start affecting the entire page.
I would suggest to find more specific, unique names, especially when creating variables in such a broad scope as :root
Reproduction ▶
No response
Steps to reproduce 🔢
No response
System Info 💻
System:
    OS: macOS 14.1.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 66.70 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.19.0 - /opt/homebrew/opt/node@18/bin/node
    npm: 10.2.3 - /opt/homebrew/opt/node@18/bin/npm
  Browsers:
    Chrome: 124.0.6367.201
    Safari: 17.1

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hey!
Since version 2.0 of the library, we have decided to get rid of everything about UI. Instead, we created a Web Components library to keep logic and user interface separated.
In the new UI library, all styles are encapsulated as they are using the Shadow DOM (that's how Web Components work). Also, global styles are scoped with --bim-ui_ so it shouldn't have any conflict with existing popular names.
Closing this issue!


Select range

I like the datepicker very much because it's easy to use and looks nice.
But I'm really missing a range functionality. Do you consider to integrate something like that?
<input id="start" data-toggle="datepicker"> <input id="end" data-toggle="datepicker">
Right now you can enter a date in the second one which is prior to first one, which of course shouldn't be the case if you want to enter a range.
Additionally highlighting the selected range in the second field would be useful.

If you only use two dates on one page its pretty easy to do.
$('#startDate').datepicker(); 	$('#endDate').datepicker({ 		filter: date => { return (new Date($('#startDate').val()) >= date) ?  false : true; }  	});

Or you could use a date-range picker made specifically to do this

If you only use two dates on one page its pretty easy to do.
$('#startDate').datepicker();
$('#endDate').datepicker({
	filter: date => { return (new Date($('#startDate').val()) >= date) ?  false : true; }
});

your method only suits click startDate first, what if i want click the endDate first sometimes, then the startDate can select a later date, how to solve this?  (╹▽╹)

If you only use two dates on one page its pretty easy to do.
$('#startDate').datepicker();
$('#endDate').datepicker({
	filter: date => { return (new Date($('#startDate').val()) >= date) ?  false : true; }
});

your method only suits click startDate first, what if i want click the endDate first sometimes, then the startDate can select a later date, how to solve this? (╹▽╹)

already figure it out.  not so familiar with the document (#^.^#)
$('#startDate').on('pick.datepicker', function (e) {         $('#endDate').datepicker('setStartDate', $('#startDate').datepicker('getDate'));     }); $('#endDate').on('pick.datepicker', function (e) {         $('#startDate').datepicker('setEndDate', $('#endDate').datepicker('getDate'));     });


Using php-SAML with a Java Spring-boot 1.3.5 Application

Hello All,
I have a peculiar situation. The Java Spring-boot version that we are using is an old one (v1.3.5). The JAVA SAML version we have seems to only work with later version.
Here is what we are trying to do:
We have got the php SAML working. What we would ideally like to do is make a call to the phpSAML from our Java Application using a web service and get the response back from the php SAML.
Can you folks let us know how to do that?
Thanks in Advance.
Regards,
Arun

Gonna be the PHP code and the Java code in the same machine? otherwise, you will need to secure somehow the traffic between the 2 machines...
I believe that at the Java App, when you need to SSO, you will redirect the user to the PHP SSO Login page, it will generate an AuthNRequest to the IdP, then receive a SAMLResponse, at the SP ACS endpoint the SAMLResponse will be
processed, if fails, an error should be shown, otherwise, you need to make the Java app aware that need to create a session for such user, one possible way is to enable at the Java server an endpoint protected by some OAuth/Oauth2, and the PHP app will be able to retrieve an access token and then, be able to communicate with the java Server in order to make it log the user... but if that is made in a back-channel,  how the session/cookie gonna be created in the user browser?


[MM-14238] Add make command to generate unsigned x86_64 build for VM/simulator

Summary
Add make command to generate unsigned x86_64 build for VM/simulator.  The resulting app (in zip file) is verified as working for Rainforest.  Generated and submitted zip file can be found here - Mattermost-unsigned-x86_64.app.zip.
This PR does not include yet the following:

code signing
auto-upload to S3

Will address separately if those are necessary to be incorporated in our build workflow.
Note that our existing Android release app is confirmed to be working on Rainforest's VM/emulator.
Ticket Link
Jira ticket: MM-14238
Device Information
This PR was tested on: [iOS simulator]

@enahum PR updated per comment.  Also, I added common functions such as start_packager and stop_packager.

I intentionally left them after seeing behaviour of not continuing operation. I suspect it's related to else condition to those targets, but I can take a look further on why.

@enahum Added those functions to start and stop targets.  I ran several run, start and build and seems fine. PTAL


Point MM to TeachOSM TM

I think its in here:
https://github.com/MissingMaps/partners/blob/master/app/assets/scripts/main.js
endpoint redirect everything from HOT to TeachOSM tasking manager
sub TeachOSM for HOT where it occurs

@DanielJDufour


0.4.1 point release

Bump version to 0.4.1. Before merging:

[ ] set back channels to pytorch from pytorch-test used for pre-release testing

cc #308 pytorch/vision#2011

replaced by #475


How to add to pool while it's running?

How can I add work to a pool while it's already running and make it return only that part of work after that part was finished? I need a pool that's is supposed to run multiple requests in a single pool, requests come from different functions and while pool is still running some old requests. The only requirement is to not to run more that a number of requests simultaneously.

You'd have to make sure the pool stays active and have the producer function deliver a new promise once it becomes available. While it's feasible using this package, you could also consider Bottleneck or one of Sindre's packages.


Ts deploy

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
367

Relevant Lines:
367

💛  - Coveralls


docs: fix and document use of keyboard focus classes on text fields

Description
Our Storybook was adding the is-keyboardFocused class on click, which isn't the intended design. Click focus and focused by use of a keyboard have different styles. This update stops the class from being applied on click, and documents how the class should be added by the implementation.
How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps

[ ] Clicking in a text field or text area does not add the keyboard focused class (blue focus indicator does not appear)
[ ] How implementations should apply the is-keyboardFocused class is documented correctly on both the text field and text area Docs

Regression testing
Validate:

The documentation pages for at least two other components are still loading, including:

[ ] The pages render correctly, are accessible, and are responsive.

If components have been modified, VRTs have been run on this branch:

[ ] VRTs have been run and looked at.
[ ] Any VRT changes have been accepted (by reviewer and/or PR author), or there are no changes.

Screenshots
New Docs story:

Design reference for styling differences between focus and focused by keyboard:

To-do list

[x] I have read the contribution guidelines.
[x] I have updated relevant storybook stories and templates.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[ ] ✨ This pull request is ready to merge. ✨

@jawinn Overall, the PR looks great!
Out of curiosity, why isn't there a keydown function in the template to showcase isKeyboardFocused indicator when tab is pressed. In the documentation you mentioned if the tab key is pressed this class should be added. When I press tab is-focused gets added.
If it's up to SWC to handle this kind of functionality then showing the Docs example and clearly stating what class has to be added would be enough.

@jawinn Overall, the PR looks great!
Out of curiosity, why isn't there a keydown function in the template to showcase isKeyboardFocused indicator when tab is pressed. In the documentation you mentioned if the tab key is pressed this class should be added. When I press tab is-focused gets added.
If it's up to SWC to handle this kind of functionality then showing the Docs example and clearly stating what class has to be added would be enough.

It's getting a little bit into implementation as it's a bit tricky to test for (keycode is not provided on the focus event), but I found what looks like an okay way to show this behavior. I've updated the PR and validation instructions.


Add Github Actions to create a new release automatically

Describe the Feature
When a PR is approved, generate a new release with release notes automatically.
Expected Behavior
Merged PRs generate new releases.
Use Case
When there is a contribution to code, after the PR approval a new release is published so users can updated it.
Describe Ideal Solution
GHA using templates readily available on the Internet.
Alternatives Considered
For now, manual publish releases.

Closed by #16


[ToggleButton] Add a color prop

Summary 💡
Add a color prop to ToggleButtonGroup to allow the active button to have an alternative color
Examples 🌈
One use-case: #23754

(Note that this example also uses a darker color than the default for the unselected text and icons.)
Edit: It seems there's an example in the spec:

https://material.io/design/interaction/states.html#activated ("Active state inheritance")
However this is in contrast to the Toggle Button spec: https://material.io/components/buttons#toggle-button
Motivation 🔦
The above example needed custom styling.

Hi guys, I'd like to address this issue... Will be making a PR in a few days.


updated readme

Added markdown documentation from POD to the readme so it is easy to reference.

Thank you.
I have taken over maintenance of Terminal::ANSIColor.
The README is now automatically generated from the pod.  Closing this PR.


Which version of watson-developer-cloud is safe to upgrade to, if using middleware version 1.8.2?

I recently upgraded my bot code to botkit-middleware-watson version 1.8.2 (as I was migrating out of Cloud Foundry authentication). Yesterday we received an authorization error from Watson indicating that the refresh token in use had expired:
Error: { .... removed attributes here... ,"errorCode":"BXNIM0408E","errorMessage":"Provided refresh token is expired"} {"cause":{"code":400},"isOperational":true,"code":400}
This suggests to me that the watson-developer-cloud sdk (used by the middleware) is not appropriately handling token refreshes. Am I right? If so, knowing that the version (of watson-developer-cloud) used by botkit-middleware-watson is 3.18.4, and assuming that later versions of the same module might correct the problem, which version should I upgrade to that is compatible with the middleware?
Or, I'll be glad to hear other advise too, of course.

I have no idea what the refresh token is.
There is no version of Watson middleware which supports Botkit 0.7 and node SDK 4 (new name of the package is ibm-watson).
If you want to keep using Botkit 0.7, it is up to you to update v1 of the middleware to support newer versions.

Thanks for replying!. The refresh token is supposed to be managed by the IBM sdk and is used for authorization (in a manner similar or analogous to OAuth2 tokens apparently). Apparently the IBM sdk is buggy as it respects to managing these tokens in earlier versions (as I'm only now finding this out, as it became apparent only after my recent adoption of IAM for authentication within Watson Assistant,  when I recently migrated the code out of Cloud Foundry authentication ), and they told me that later versions of the sdk (4 and 5) don't even use refresh tokens, so this should resolve the problem.
I understand (and agree with) your point about the responsibility for updating to newer versions of the Watson sdk. I guess I'm just wondering if you know a bit more about the relationship between the two (middleware vs watson sdk), that would hint at the difficulty level of this task.
After all, I'd have to somehow evaluate the difficulty of this task vs. that of upgrading to a current version of botkit (with all that that entails, of course).  As I'm writing this, I'm realizing that I should ask this specific question possibly in the other project too.


ItemsRepeater does not respect control alignment

ItemsRepeater does not currently respect the HorizontalAlignment/VerticalAlignment of child controls.
This seems to work on UWP so there's something up with our port.

This does actually seem to work correctly when the ItemsRepeater is not contained within a ScrollViewer which has horizontal scrolling enabled (as in ControlCatalog). I do however think it could handle this a bit better, need to double check with UWP and raise an issue there if necessary.

@grokys I recently stumbled upon ItemRepeater not honoring HorizontalAlignment. To my surprise, ListBox does o-O
<StackPanel
            Orientation="Vertical"
            VerticalAlignment="Center" HorizontalAlignment="Center"
            >
  <ItemsRepeater Items="{Binding RecentDirectories, ElementName=Root, Mode=OneWay}"
                 HorizontalAlignment="Center"
                 >
    <ItemsRepeater.ItemTemplate>
      <DataTemplate>
        <Button Content="{Binding Name, Mode=OneWay}"
                Command="{Binding OpenFolderCommand, ElementName=Root, Mode=OneTime}"
                CommandParameter="{Binding FullPath, Mode=OneWay}"
                CornerRadius="0"
                />
      </DataTemplate>
    </ItemsRepeater.ItemTemplate>
  </ItemsRepeater>

  <Button ... />
</StackPanel>

With ItemsRepeater (e.g. above XAML):

If I replace ItemsRepeater with ListBox in the above:

Avalonia 0.10.18 on Windows


Sharepoint migration tool vulnerable

The sharepoint migration tool can be installed without the agreement of the terms of service and privacy policy. If you agree on the terms the install button will be greyed out. Legal vulnerability

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

ID: 36d01d11-eea9-7f49-693d-25c3fecc6253
Version Independent ID: 6749a619-2b75-1561-81e9-b45888c58c4b
Content: How to use SharePoint Migration Tool - Migrate to Microsoft 365
Content Source: migration/how-to-use-the-sharepoint-migration-tool.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@my62   Thanks for your feedback. Can you please provide the steps you took to skip the terms of service and install regardless?  I just tested all the possible download links and the Install button stays successfully greyed out until I accept the terms of service.  Thanks!  joanne

@my62   Just following up -- can you provide the steps you took to be able to install SPMT without accepting the terms?

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

Sorry for the late reply,
It seems that it worked now.
I simply opened the below address for the problem:
https://spmt.sharepointonline.com/install/default.htm


Please add owned version, or remove lifetime from err

This code provides lifetime err but really should not
let v = lenient_semver::Version::parse(version.as_str()).context("wrong version format")?;
let version = semver::Version::from(v);

error[E0597]: `version` does not live long enough
  --> lib/src/ecosystems/lua/rockcpec.rs:41:48
   |
36 |         let version = match VERSION_REGEX.captures(input) {
   |             ------- binding `version` declared here
...
41 |         let v = lenient_semver::Version::parse(version.as_str()).context("wrong version format")?;
   |                                                ^^^^^^^---------
   |                                                |
   |                                                borrowed value does not live long enough
   |                                                argument requires that `version` is borrowed for `'static`
...
79 |     }
   |     - `version` dropped here while still borrowed

Regular semver::Version doesnt require static lifetimes
let version = semver::Version::parse(version.as_str()).context("wrong version format")?;

this works fine

Hi there

This code provides lifetime err but really should not

lenient_semver::Version borrows from the input, the pre-release and build identifier components are references with the same lifetime as the input.
If you want to have an 'static-ally lifetimed one, you can use disassociate_metadata.
Similarly, the error also borrows from the input and has an owned method to create an owned version of it, which no longer borrows from the input.

Regular semver::Version doesnt require static lifetimes

lenient_semver::Version is not (and was never meant to be) a drop-in replacement for semver::Version. If you wanna parse into semver::Version, but with the lenient rules from this parser, you can do so and just use semver::Version. This is actually the default behavior and ever since the 1.0 rewrite of semver, there is little need to use lenient_semver::Version, especially not if you don't want to have it borrow from the input.

this part from readme doesnt work

The doctest for it runs green, how does it not work? What sort of error are you getting?


Domain setting

It closes #308
It closes #340
Add UI to enter a domain manually.
Update all requests to add Host header if daemon behind proxy.

We should remove the info in the service detail page about running port as not relevant, and rather have a link to the baseURL now
cc/ @filopedraz

We should remove the info in the service detail page about running port as not relevant, and rather have a link to the baseURL now
cc/ @filopedraz

Just the line Running Port       8111?
Which your screenshot is not showing

Resolve conflicts


[Bug] Corrupt/illegal blocks on longest chain cause node halting

If we have a block in the longest chain that is invalid for the node's consensus (i.e. 2 coinbase transactions) -- it will cause the node to halt syncing and freeze.

Closing, as it's no longer applicable to testnet2.


I met an error

Hello there
I encountered an error
When I was training the second epochs
Graphics card GTX1050TI
RAM 16G

Hi this is due to the number of bounding boxes per image exceeded the 100 preset. Please set the --yolo_max_boxes flag to 200 and then try again

It worked, thank you!!


BigQuery: fixes python 2.7 system test error caused by PR 4354

Fixes error caused by #4354

Fingers crossed: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/4563


[Memo]  parentId  could be hidden from the user-facing API

This might require a bit of discussion since I might not fully understand the API.
createDirectory and rename have a parentId argument. This works fine, but it requires path parsing and a call to lookup the fileId on the user side. Would it make more sense to pass a path (instead of a name), parse the path, and lookup the file's parentId on the Rust side?

We assumed that you would have the file ids at your disposal during rendering your tree interface, since they are available on the entries. Are you performing renames in other situations where you don't have the file id readily available? It seems like the id-based approach is more general. It covers situations where you have a file id, such as during a drag and drop, while still allowing a lookup on the path in other cases. Are you hitting any cases where the id based approach saves effort, or are you always using paths?

We assumed that you would have the file ids at your disposal during rendering your tree interface, since they are available on the entries.

We don't do this now, but that makes a lot of sense. We use paths like ids now, but I'll try thinking about passing FileEntry objects around instead.

We've actually decided to go ahead and hide file ids from the public API. Buffer ids will continue to exist, but file operations will be mediated through paths.


Unsatisfiable requirements detected

Trying to use the package as outlined in README.md in julia 1.11 yields the following error:
    Updating git-repo `https://github.com/TensorBFS/TropicalTensors.jl.git`
   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package TropicalNumbers [b3a74e9c]:
 TropicalNumbers [b3a74e9c] log:
 ├─possible versions are: 0.1.0 - 0.6.2 or uninstalled
 ├─restricted to versions 0.3 - 0.4 by TropicalTensors [d36a0d72], leaving only versions: 0.3.0 - 0.4.1
 │ └─TropicalTensors [d36a0d72] log:
 │   ├─possible versions are: 0.1.0 or uninstalled
 │   └─TropicalTensors [d36a0d72] is fixed to version 0.1.0
 └─restricted by compatibility requirements with TropicalYao [8a7e9035] to versions: 0.1.0 - 0.2.3 — no versions left
   └─TropicalYao [8a7e9035] log:
     ├─possible versions are: 0.1.0 - 0.1.5 or uninstalled
     └─restricted to versions 0.1 by TropicalTensors [d36a0d72], leaving only versions: 0.1.0 - 0.1.5
       └─TropicalTensors [d36a0d72] log: see above

Details on julia:
julia> versioninfo()
Julia Version 1.11.2
Commit 5e9a32e7af2 (2024-12-01 20:02 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: 32 × 13th Gen Intel(R) Core(TM) i9-13980HX
  WORD_SIZE: 64
  LLVM: libLLVM-16.0.6 (ORCJIT, alderlake)
Threads: 1 default, 0 interactive, 1 GC (on 32 virtual cores)

I have tried installing local version of TropicalYao with bumped requirements on TropicalNumbers, but it results in another suite of errors, and in particular Unsatisfiable requirements detected for package CuYao, as I am in the end interested in using Tropical Tensors with GPU.
What is the recommended way of proceeding forward? Are there any plans to update this package?

Sorry, I forgot to archive this repo. The latest development is in: https://github.com/QuEraComputing/GenericTensorNetworks.jl


typo fix

Thanks! Merged in https://github.com/ruimarinho/docker-bitcoin-core/commit/bd57e3c47d5394f8bd8fc4bcc05b07eebd69632e.


feat: add option to toggle footer theme name display

This PR:

Introduced footerThemeName parameter in hugo.toml to control the display of Hugo and Hermit-V2 theme tribute in the footer.
Updated footer.html to conditionally display the tribute based on the footerThemeName parameter.

Related issue:

#28

Hi @v0y4g3r,
Thanks for the PR. Looks good, works well.
However, I would like to suggest a change. What you have done is set footerThemeName to false by default; it is preferable to default to true, but can be changed to false. Reason: Keeping same functionality as Original Theme.
If you want to work on this, you can, else I would have to make the change. Let me know your thoughts, or a suggestion

I changed the option to footerHideThemeName in b7def36 so that the footer will remain the same if param.footerHideThemeName is not set in hugo.toml.

Hi @v0y4g3r,
Thanks a lot for the revision.
It has been merged.


Fonts broken with rerun 0.17 on Windows

Describe the bug
Fonts are unreadable and mangled on Windows build. Look https://github.com/andresv/rerun-win-fonts
To Reproduce
Here is a minimal app that can be used to reproduce this issue: https://github.com/andresv/rerun-win-fonts
Screenshots

Desktop
Windows 10 Pro 19045.3930 VM running under Parallels 19.4.0, host is macOS Sonoma 14.5.
Also tried to build under github actions and it looked the same.
Rerun version
0.17

I sent exe to my friend who has win11 and it runs fine in there.

This looks like a graphics driver issue to me. Have you perhaps changed the default settings for them?

Tried with normal rerun and it is the same:

I haven't tweaked windows VM graphics settings.


DID resolver and debugger tool

Description of change
Links to any relevant issues
https://github.com/iotaledger/identity.rs/issues/326
Type of change
Add an x to the boxes that are relevant to your changes.

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

How the change has been tested
Describe the tests that you ran to verify your changes.
Make sure to provide instructions for the maintainer as well as any relevant configurations.
Change checklist
Add an x to the boxes that are relevant to your changes.

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

future development will take place in the https://github.com/iotaledger/explorer repo


Friendly Title for Docs

Adds a customization in the title of a technical documentation.
This helps to make a friendlier name.
The property is title which is inside metadata, if title does not exist the name property is used.

apiVersion: backstage.io/v1alpha1
kind: Component
metadata:
  name: documented-component
  title: Custom Title Here
  description: A Service with TechDocs documentation
  annotations:
    backstage.io/techdocs-ref: dir:.
spec:
  type: service
  lifecycle: experimental
  owner: user:guest

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

In general I like it, just want to leave here that mkdocs also provides a global title field which is used here and there in techdocs (I think in the title of a techdocs page). Not sure if we need two approaches. Maybe we just stop using the mkdocs field?


:arrow_up<EMAIL_ADDRESS>

Description of the Change
Updates apm to 2.0.1 to benefit from the changes mentioned in this PR: https://github.com/atom/apm/pull/803
Benefits

Shorter Atom installation paths on Windows
One step closer to having side-by-side installation channels on Windows https://github.com/atom/atom/issues/9247

Possible Drawbacks
None.
Verification Process

[ ] Run a full installer build on Windows to ensure that the installer can be built successfully and installs with much shorter paths
[ ] CI is green

Applicable Issues
#9247

Tell me what I’m supposed to do I don’t want to be around Liz but I want help I want to be a part of my daughters life
Julesvicious
On Aug 6, 2018, at 10:40 AM, David Wilson<EMAIL_ADDRESS>wrote:
Description of the Change
Updates apm to 2.0.1 to benefit from the changes mentioned in this PR: atom/apm#803
Benefits
Shorter Atom installation paths on Windows
One step closer to having side-by-side installation channels on Windows #9247
Possible Drawbacks
None.
Verification Process
Run a full installer build on Windows to ensure that the installer can be built successfully and installs with much shorter paths
CI is green
Applicable Issues
#9247
You can view, comment on, or merge this pull request online at:
https://github.com/atom/atom/pull/17789
Commit Summary
:arrow_up<EMAIL_ADDRESS>:arrow_up<EMAIL_ADDRESS>Re-enable Windows builds in the release build definition
File Changes
M apm/package-lock.json (10349)
M apm/package.json (2)
M script/package-lock.json (9869)
M script/package.json (2)
M script/vsts/release-branch-build.yml (4)
Patch Links:
https://github.com/atom/atom/pull/17789.patch
https://github.com/atom/atom/pull/17789.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


post_source_hooks not called (?)

Warning:  I will close the bug issue without the minimal init.vim and the reproduce ways.
Problems summary
I do not see post source hooks being called
Expected
I expect to see them called.
I do not understand "VimEnter" code in docs, and how to use it for my purposes, maybe I just misunderstand this?
Putting the line in my init.vim did not help, as the init was called when vim-sneak wasn't loaded (so I couldn't call, e.g. sneak#state())
Environment Information (Required!)

dein.vim version(SHA1): 5c8f935d062db0ebfa16983d1be9a9487526b875

OS: Linux

Vim/neovim version: 0.3.1

Provide a minimal .vimrc with less than 50 lines (Required!)
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/dein.vim

call dein#begin('~/.cache/dein')

call dein#add('~/.cache/dein')

function! s:add_hook()
  echom 'Add for ' . g:dein#plugin.name
endf

function! s:post_source_hook()
  echom 'Post source for ' . g:dein#plugin.name
endf

call dein#add('justinmk/vim-sneak',
      \ {'hook_add': function('s:add_hook'),
      \  'hook_post_source': function('s:post_source_hook')})

call dein#end()

The reproduce ways from Vim starting (Required!)

Start neovim.
Do not see "Post source" message.

Please see dein.vim documentation.
					*dein-options-hook_post_source*
hook_post_source (String) or (Function)
		It is executed after plugins are sourced.

		Note: In Vim initializing, you must call the
		"hook_post_source" hooks manually in |VimEnter| if needed.
>
	autocmd VimEnter * call dein#call_hook('post_source')
<

So, you must call dein#call_hook('post_source') manually.
...
call dein#end()
autocmd VimEnter * call dein#call_hook('post_source')

The below code is invalid.
call dein#add('~/.cache/dein')

Putting the line in my init.vim did not help, as the init was called when vim-sneak wasn't loaded (so I couldn't call, e.g. sneak#state())

I don't understand.  Why you don't upload the code?
I think you have misunderstood.

Because of:
call dein#add('~/.cache/dein')

function! s:post_source_hook()
  echom 'Post source for ' . g:dein#plugin.name
  " This lines throws exception:
  echom string(sneak#state())
endf

dein#end()

autocmd VimEnter * call dein#call_hook('post_source')

Did not work when I tried it. Now I have corrected it to call dein#add('~/.cache/dein/repos/github.com/Shougo/dein.vim'), it works.
I copied this from https://github.com/Shougo/dein.vim#unixlinux-or-mac-os-x does this need updating?

Thank you for your help!

I have tested your vimrc and it works.
I will fix the README.
QED.


Enable more win32 specs

Followup on #8670. This PR enables a lot of more specs for building on win32.
Often just a few specs needed to be disabled in order to make the rest of a spec file compile and pass. This affects mostly specs working with big numbers or yaml.
All specs pass on win32.
With this and #8668 we should get 6464 examples passing on win32, that's 70% of all standard lib specs.

Finished in 8.64 seconds
6466 examples, 0 failures, 0 errors, 57 pending

force pushes :(

Sorry for the force pushes, but the changes were really insubstantial and this makes it easier to review for everyone who didn't take a look at it before.

Just dropped a few notes in the conversations above.

rebase time!

Rebased and squashed.

Somehow win32 specs hang at some point. I've now added verbose option. Next step is to disable IO specs because that's where execution seems to stop.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 212 ms

Resolved: ClamAV performance has improved in 7b61d92.


Corp Vcr Cluster Helix Config

This is the config to create corp vcr cluster.

We do not need to add this config on open source code.


AERender Options When Using --file Argument

Hello and thanks in advance for any guidance you might be able to provide.
We're rendering by invoking the CLI on Windows and passing in the --file parameter to a job file. I've spent some time reading over the documentation and it's not clear to me how (or if) we can adjust some of the settings like enabling multi-frame rendering or controlling CPU/Memory limits when using the CLI.
Is it just something I'm overlooking in the documentation? If so, please feel free to tell me to RTFM (but a link to the portion of the manual that I've missed would be really appreciated). Thanks again!

Hey @antoinne85
The CLI documentation is a bit lacking in that regard. I would suggest taking a look at similar performance parameters for Worker. They will be identical for CLI: https://github.com/inlife/nexrender/tree/master/packages/nexrender-worker#information
Please do let me know if that helps!


Add labels for local statistics for screenreader users

Problem:
If any number of locations are added to monitor the local corona numbers, there is absolutely no way to know which number belongs to which location. The location names are not read out by screen readers. So blind people only get the numbers, "151" for example.
Solution:
Add the location name to the numbers. It could look like "Berlin - 7-day incidence of 151". That way, it is possible to monitor multiple locations in CWA even if you are blind.
If you need more information, I am happy to help to the best of my ability.

Internal Tracking ID: EXPOSUREAPP-10650

@WestphalDenn this issue is addressed by pr #4427

@WestphalDenn
Could you check this is fixed for you? I tried on CWA 2.21.1 using TalkBack on a Local 7-Day Incidence tile for Bavaria and it read me back the name "Bavaria".
If all is well, then you could close the issue.

Yes everything works well now. Thanks for asking.

@WestphalDenn Thanks for participating here and your feedback. Best wishes, DS

Corona-Warn-App Open Source Team


Support setting disk caching policy

Being able to configure disk caching would be especially useful when running KVM based testbeds / CI
Example:
disk {
  volume_id = "..."
  cache = "unsafe"
}

Thanks!

Is this a common setting enough to expose it in the schema?
Would the XSLT transform feature in master to set the value in your situation?

@dmacvicar  we tried with xslt and worked out. @FedericoCeratto if you want it on the plugin, i will let this issue open ( as good first issue), or if you are fine with xslt we can close this, as you wish :sunflower:

Yop  is good


wakatime: rename to wakatime-cli

Description of changes
Rename the package to correspond with the mainProgram and the repository name.
The Makefile BINARY_NAME name variable defaults as well to wakatime-cli and this is also the prefix for all the release archives.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

It doesn't work apparently:
https://hydra.nixos.org/build/262247964/nixlog/1/tail

Oh no!


Enter link editor in edit mode when already editing

Current behavior: while editing/inserting an object and click into a link to populate it, you then have to separately enter edit mode on that link
Expected behavior: Clearly you are already editing, so the link editor should be in edit mode already
EdgeDB 4.1+0c628ea (repl 4.0.2+500be79)
    type Organization {}

    type User {
        required link organization: Organization;
    }

Related papercut in this situation https://github.com/edgedb/edgedb-ui/issues/229


error when running the app

<EMAIL_ADDRESS>dev
next dev

Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/parser' is not defined by "exports" in /node_modules/next/node_modules/postcss/package.json

any fix ?

Adjust dependencies as following:
"dependencies": {
    "autoprefixer": "10.2.3",
    "next": "10.0.5",
    "next-seo": "4.18.0",
    "npm": "^6.14.11",
    "postcss": "8.2.4",
    "react": "17.0.1",
    "react-dom": "17.0.1",
    "tailwindcss": "2.0.2"
  }

And use this node version v16.20.2
Should be working now.


Specification in documentation

Hi,
Firstly thanks for this useful function.
Having a question concerning the usage of this function. Shouldn't we specify the Vault and CDN / CDN Endpoint on which it has to do the rotation ? Does the function detect it automatically ?
I didn't see any parameter in the function concerning those resources.

It is not necessary to specify the CDN name or Front Door name because it automatically recognizes the CDN Profile / Endpoint / Front Door that has been permitted access by RBAC.
However, only the CDN / Front Door that exists in the same subscription as the Function are included.

Thank you, that's all I wanted to know


Added hubot-web-hook service name as required by hubot-notifier fixes fabric8io/fabric8#4146

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/534/
Test PASSed.


Include Additional  Overall Position Details

Aside from the current overall position details (e.g. HF, Borrowing Power, etc.) add some additional details.
These details should live in a separate area from the main details, and probably be collapsed/hidden by default. Additional details to add include:

Current overall liquidation threshold
Current overall loan to value
Max loan to value
% borrowing power used

Thanks to rockrider45#7033 for the inspiration!

This is live now


Subscribe Button not working

I added API key and List Id of MailChimp and got this error on clicking on the button:
Error: GraphQL error: Cannot read property 'subscribe' of undefined

Hi @Michael-Reich ,
This has been fixed in the devel branch. Please see sample_settings.json file in devel. You can see the correct parameters for Mailchimp.

It looks like sample_settings.json in devel branch has been cleaned up. Please see this doc instead:
http://docs.vulcanjs.org/newsletter.html#Settings

Thank you will test it today!

Worked


Add copy to clipboard function for shadows

@joelcmk fyi I made a change to remove the jumpiness. Shown here: https://www.loom.com/share/2443386335c245febe4d9afbb8683d42
I move the padding to parent, instead of child.


The change spoken form of word scope to sub

Fixes #1918
Checklist

[-] I have added tests
[x] I have updated the docs and cheatsheet
[x] I have not broken the cheatsheet

benefit of reduced conflicts outweighs the costs of the less intuitive word

Agree.


SpotifyApi.authorizationCodeGrant: add parameter codeVerifier.

Expose codeVerifier when building AuthorizationCodeGrant from SpotifyApi.authorizationCodeGrant.
Issue: https://github.com/rinukkusu/spotify-dart/issues/201

@gmpassos please provide a small description (e.g. relation to an issue)

FYI: @rinukkusu

Please check the comment - looks good otherwise!

Resolved 👍.
After merging this branch, could you please publish it? I prefer to use the published package in our code.
Afterward, I will submit a PR that brings the code into compliance with lints: ^3.0.0.
Thanks in advance.
Best regards.


JUnit CVE-2020-15250 vulnerability

Hello team,
I appreciate the fantastic work done by you all in building this library. I have used this in my project and it works great!
However, there's been a vulnerability discovered during 2020 that pertains to JUnit4. Can we please update JUnit to the latest version (https://mvnrepository.com/artifact/junit/junit/4.13.2)?
Thanks in advance,
Cheers!

This is addressed in #81b4ba8. Thanks!


Addquotes: add option to ignore / force exact match

In the file defining the knowledges, add the possibility of writing keywords (in comments) in the same line as the definition of some knowledge kl to desactivate some features of the addquotes command, such as:

Keyword
Effect

kl-addquotes:ignore
do not propose to the user to add quotes around occurrences of kl

kl-addquotes:force-exact-match
do not propose to define synonyms of kl

kl-addquotes:only-case-insensitive
proposes to define kl' as a synonym of kl only when kl' is the same as kl, up to letter case

For every keyword kwd, add a keyword kwd-syn which has the effect of applying the effect of kwd to kl but also to all its synonyms. Add a keyword kwd-all to apply this effect to every knowledge of the file.

This is far from being a priority…

Might get useful, if you use single letter for definition, you get proposition of synonyms for every word containing this letter!


DEV:FR:Finish Fluids Report Converter

Chris provided the basis for the Fluids Report Converter in --
https://github.com/hashmapinc/WitsmlObjectsLibrary/blob/df-fluidReport/src/main/java/com/hashmapinc/tempus/WitsmlObjects/Util/FluidsReportConverter.java
He advises that if I follow what he started and finish this converter, we will have all of the mappings needed for Fluids Report. After that, we should be able to provide the basic CRUD.
There is no RTC testing for Fluids Report. Therefore, we just require the standard set of Soap UI CRUD tests.

Update: Committed df-fluidsreports (my branch based on Chris' branch df-fluidsreport) and then merged into master as 1.1.37.


Add docs for screenshare

https://docs.100ms.live/android/v2/features/Screen-Share

@lavi-moolchandani  PR to be raised against main branch of https://github.com/100mslive/100ms-docs/
.mdx file to be added at this path: https://github.com/100mslive/100ms-docs/tree/main/docs/react-native/v2/features


Add index yaml

Add index.yaml for helm chart releases action

Is the description in this chart accurate for this project?

Description is correct (for common and postgresql chart dependencies)
But indeed the part for this chart was missing.


[BUG] Stuck at Login

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

Describe the bug
Stuck forever on logging in or writing password.
Copy and paste your error
[LOGIN] Logging-in...
[LOGIN] Writing email...
[LOGIN] Writing password...
Screenshot of the bug

Value of dashboard variable
Uploaded to paste since it was too long, GitHub limited it.
https://paste.ee/p/fNSWG

This happened to me too. What I did was login into the stuck account on browser to clear the 2fa, and then let the bot run again

This happened to me too. What I did was login into the stuck account on browser to clear the 2fa, and then let the bot run again

Didnt do anything in my case, still stuck :(

Could you please run it with -v argument and screenshot what's happening

Closing, you're welcome to reopen


Add ParametrizedEvolution to OQC Lucy's supported operations

Description of changes:
Updated translation.py to append "ParametrizedEvolution" to the frozen set of supported operations if the device arn matches OQC Lucy.
Testing done:
Added unit test
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[x] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks good to me once the mocking is sorted out with the tests! Good luck 😅

The failing tests look unrelated to me - I'm assuming we will see these on all the OQC PRs, and we should deal with them as a separate PR


Fix missing package in github action

It seems like the ubuntu20.04 image don't have flex installed
which is needed. This is fixed in this commit.

The CentOS 7 test failed with the error:
configure.ac:36: error: possibly undefined macro: AM_PROG_LIBTOOL
It seems like this might be due to an outdated version of
GNU autotools. I think we can can comment out CentOS 7 for now
and fix that later if we need to test on CentOS 7.

we can use emqx-builder images
Search in emqx.git master branch

@zmstone Thanks for the comments. I will look at this again once the emqx-builder repo's github action has completed successfully so that the new docker images are published. The github action build of the raspian images seems to time out after 5h59m. I restarted the build one time already which made one of the raspbian image finish before the time out and now I have restarted the build again for the only raspbian image that is left.

maybe squash the commits to for a cleaner history

@zmstone

maybe squash the commits to for a cleaner history

Done! fe3ab5f
Thanks for the help. Now, I just need an approving review and then I can merge.


camera mjpeg verify_ssl False not being honored

Home Assistant release with the issue:

0.97.2
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Hassbian
Component/platform:

mjpeg/camera.py
Description of problem:
The verify_ssl False setting is not being honored.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
camera:
  - platform: mjpeg
    name: Kitchen
    mjpeg_url: https://weewx-pi.borgnet.us:8081/
    verify_ssl: False

camera 2:
  - platform: mjpeg
    name: Office
    mjpeg_url: https://office.borgnet.us:8081/
    verify_ssl: False

Traceback (if applicable):
requests.exceptions.SSLError: HTTPSConnectionPool(host='office.borgnet.us', port=8081): Max retries exceeded with url: / (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])")))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.7/site-packages/aiohttp/web_protocol.py", line 418, in start
    resp = await task
  File "/srv/homeassistant/lib/python3.7/site-packages/aiohttp/web_app.py", line 458, in _handle
    resp = await handler(request)
  File "/srv/homeassistant/lib/python3.7/site-packages/aiohttp/web_middlewares.py", line 119, in impl
    return await handler(request)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/http/real_ip.py", line 37, in real_ip_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/http/ban.py", line 69, in ban_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/http/auth.py", line 228, in auth_middleware
    return await handler(request)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/http/view.py", line 124, in handle
    result = await result
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/camera/__init__.py", line 493, in get
    return await self.handle(request, camera)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/camera/__init__.py", line 510, in handle
    image = await camera.async_camera_image()
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/mjpeg/camera.py", line 113, in async_camera_image
    image = await self.hass.async_add_job(self.camera_image)
  File "/usr/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/mjpeg/camera.py", line 145, in camera_image
    req = requests.get(self._mjpeg_url, stream=True, timeout=10)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/api.py", line 75, in get
    return request('get', url, params=params, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/srv/homeassistant/lib/python3.7/site-packages/requests/adapters.py", line 514, in send
    raise SSLError(e, request=request)

Additional information:

Hello ????

Ping.. this is still not working with the HA 0.99.3


⚠ Bofrost.fr has degraded performance

In 6f24772, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 909 ms

Resolved: Bofrost.fr performance has improved in 187b5cc after 9 minutes.


[Feature]: inconsistent vocab_sizes support for draft and target workers while using Speculative Decoding

🚀 The feature, motivation and pitch
Currently, vllm with Speculative Decoding requires that the draft model and target model have the same vocab size. However, the target model may have a larger parameter size and use more data to train, so the vocab size may also be slightly larger (If we consider using a 0.5B model as the draft model and a 72B model as the target model).  In spec_decode_worker.py this file, line 564: asserts all(vocab_sizes[0] == vocab_size for vocab_size in vocab_sizes) making the inference pipeline unable to run. If this line is removed, the program may encounter a reshape error. So is inconsistent vocab sizes implementation possible? Such as using padding for the vocab size for the draft model?
Alternatives
No response
Additional context
No response

Found a way to work around.

How did you work around this? Facing the same issue now with llama3 and tinyllama

How did you work around this? Facing the same issue now with llama3 and tinyllama

You can modify the model file (i.e., vllm/model_executor/models/your_tinyllama_model.py) to concat the embed_tokens.weight with padding tensor. Then change the vocab_size param to align with your llama3 model in your_tinyllama_path/config.py  to work around.

Found a way to work around.

Have you solved the problem of inconsistent vocab_size? If you solved it, can you show me how you did it?

I have a simple work around.
my case:

vllm version: 0.5.1
draft model: Qwen1.5-0.5B-Chat, with vocab size 151936
target model: Qwen1.5-14B-Chat, with vocab size 152064

method:

update the vocab_size config of draft model (Qwen1.5-0.5B-Chat/config.json) to 152064
modify the following two lines to pad vocab embedding tensor in vllm/model_executor/layers/vocab_parallel_embedding.py:

line 326
    # original
    assert loaded_weight.shape[output_dim] == self.org_vocab_size

    # modified
    assert loaded_weight.shape[output_dim] <= self.org_vocab_size   # from `==`  to  `<=`

line 329
  # original
  loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

  # modified
  if shard_size <= loaded_weight.shape[0]:
            loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

That's all.

I have a simple work around.我有一个简单的解决方法。
my case: 我的情况：

vllm version: 0.5.1 vllm版本：0.5.1
draft model: Qwen1.5-0.5B-Chat, with vocab size 151936草稿模型：Qwen1.5-0.5B-Chat，词汇大小151936
target model: Qwen1.5-14B-Chat, with vocab size 152064目标模型：Qwen1.5-14B-Chat，词汇量152064

method: 方法：

update the vocab_size config of draft model (Qwen1.5-0.5B-Chat/config.json) to 152064将草稿模型 ( Qwen1.5-0.5B-Chat/config.json ) 的 vocab_size 配置更新为 152064
modify the following two lines to pad vocab embedding tensor in vllm/model_executor/layers/vocab_parallel_embedding.py:修改以下两行以填充 vllm/model_executor/layers/vocab_parallel_embedding.py 中的词汇嵌入张量：

line 326 第326行
    # original
    assert loaded_weight.shape[output_dim] == self.org_vocab_size

    # modified
    assert loaded_weight.shape[output_dim] <= self.org_vocab_size   # from `==`  to  `<=`

line 329 329行
  # original
  loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

  # modified
  if shard_size <= loaded_weight.shape[0]:
            loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

That's all. 就这样。
I tried the aforementioned method using Qwen2-7B-Instruct  and Qwen2-0.5B-Instruct , and it works on a single GPU. However, on multiple GPUs, it shows the following error:RuntimeError: start (76032) + length (76032) exceeds dimension size (151936).

I have a simple work around.
my case:

vllm version: 0.5.1
draft model: Qwen1.5-0.5B-Chat, with vocab size 151936
target model: Qwen1.5-14B-Chat, with vocab size 152064

method:

update the vocab_size config of draft model (Qwen1.5-0.5B-Chat/config.json) to 152064
modify the following two lines to pad vocab embedding tensor in vllm/model_executor/layers/vocab_parallel_embedding.py:

line 326
    # original
    assert loaded_weight.shape[output_dim] == self.org_vocab_size

    # modified
    assert loaded_weight.shape[output_dim] <= self.org_vocab_size   # from `==`  to  `<=`

line 329
  # original
  loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

  # modified
  if shard_size <= loaded_weight.shape[0]:
            loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

That's all.

I tried the aforementioned method using Qwen2-7B-Instruct and Qwen2-0.5B-Instruct, and it works on a single GPU. However, on multiple GPUs, it shows the following error:

I have a simple work around.
my case:

vllm version: 0.5.1
draft model: Qwen1.5-0.5B-Chat, with vocab size 151936
target model: Qwen1.5-14B-Chat, with vocab size 152064

method:

update the vocab_size config of draft model (Qwen1.5-0.5B-Chat/config.json) to 152064
modify the following two lines to pad vocab embedding tensor in vllm/model_executor/layers/vocab_parallel_embedding.py:

line 326
    # original
    assert loaded_weight.shape[output_dim] == self.org_vocab_size

    # modified
    assert loaded_weight.shape[output_dim] <= self.org_vocab_size   # from `==`  to  `<=`

line 329
  # original
  loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

  # modified
  if shard_size <= loaded_weight.shape[0]:
            loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

That's all.

I tried the aforementioned method using Qwen2-7B-Instruct and Qwen2-0.5B-Instruct, and it works on a single GPU. However, on multiple GPUs, it shows the following error

Try modifying the line 400 of vllm/model_executor/models/qwen2.py from weight_loader(param, loaded_weight) to
              if name == "model.embed_tokens.weight":                     if loaded_weight.shape[0] == 151936:                         padding = torch.zeros((128, 1024), dtype=loaded_weight.dtype, device=loaded_weight.device)                         loaded_weight = torch.cat([loaded_weight, padding], dim=0)                     weight_loader(param, loaded_weight)                 else:                     weight_loader(param, loaded_weight).
You might have to do the padding on weight actually to pass tons of tensor_shape assertion checks.

I have a simple work around.
my case:

vllm version: 0.5.1
draft model: Qwen1.5-0.5B-Chat, with vocab size 151936
target model: Qwen1.5-14B-Chat, with vocab size 152064

method:

update the vocab_size config of draft model (Qwen1.5-0.5B-Chat/config.json) to 152064
modify the following two lines to pad vocab embedding tensor in vllm/model_executor/layers/vocab_parallel_embedding.py:

line 326
    # original
    assert loaded_weight.shape[output_dim] == self.org_vocab_size

    # modified
    assert loaded_weight.shape[output_dim] <= self.org_vocab_size   # from `==`  to  `<=`

line 329
  # original
  loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

  # modified
  if shard_size <= loaded_weight.shape[0]:
            loaded_weight = loaded_weight.narrow(output_dim, start_idx, shard_size)

That's all.

I tried the aforementioned method using Qwen2-7B-Instruct and Qwen2-0.5B-Instruct, and it works on a single GPU. However, on multiple GPUs, it shows the following error

Try modifying the line 400 of vllm/model_executor/models/qwen2.py from weight_loader(param, loaded_weight) to
                if name == "model.embed_tokens.weight":
                    if loaded_weight.shape[0] == 151936:
                        padding = torch.zeros((128, 1024), dtype=loaded_weight.dtype, device=loaded_weight.device)
                        loaded_weight = torch.cat([loaded_weight, padding], dim=0)
                    weight_loader(param, loaded_weight)
                else:
                    weight_loader(param, loaded_weight)

You might have to do the padding on weight actually to pass tons of tensor_shape assertion checks.

If I need to modify the vllm source code, is there any quick way? Currently I use pip install -e<EMAIL_ADDRESS>using python editable mode, but this is too time-consuming


Script migrations (e.g. .py) aren't loaded

Which version and edition of Flyway are you using?
10.15.2 Community
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
N/A
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-Line as packaged in flyway/flyway-docker
Which database are you using? (Type & version)
PostgreSQL
Which operating system are you using?
Ubuntu 20.04
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)

Mounted my directory with my migrations at /migrations in the Docker container
Ran flyway info migrate info

/migrations directory structure:
__init__.py
config.py
V1__initial_setup.py

V1__initial_setup.py imports config.py (as import .config).
What did you expect to see?
I expected to see my migration picked up and run. When debugging, I expected to see the ScriptMigrationResolver class' resolveMigrations method called, and I expected that the CompositeMigrationResolver class would register ScriptMigrationResolver as a migration resolver (possible at line 71, with the plugin register?).
What did you see instead?
I saw that my Python scripts were always filtered out as resources:
Current version of schema "flyway": null
Schema "flyway" is up to date. No migration necessary.
DEBUG: Memory usage: 57 of 504M
DEBUG: Scanning for filesystem resources at '/migrations'
DEBUG: Scanning for resources in path: /migrations (/migrations)
DEBUG: Scanning for resources in path: /migrations/__pycache__ (/migrations)
DEBUG: Found filesystem resource: /migrations/V1__initial_setup.py
DEBUG: Found filesystem resource: /migrations/__init__.py
DEBUG: Found filesystem resource: /migrations/__pycache__/__init__.cpython-311.pyc
DEBUG: Found filesystem resource: /migrations/config.py
DEBUG: Filtering out resource: /migrations/V1__initial_setup.py (filename: V1__initial_setup.py)
DEBUG: Filtering out resource: /migrations/__init__.py (filename: __init__.py)
DEBUG: Filtering out resource: /migrations/__pycache__/__init__.cpython-311.pyc (filename: __init__.cpython-311.pyc)
DEBUG: Filtering out resource: /migrations/config.py (filename: config.py)
DEBUG: DDL Transactions Supported: true
DEBUG: Schemas:
DEBUG: Default schema: flyway
DEBUG: Scanning for SQL callbacks ...
DEBUG: Filtering out resource: /migrations/V1__initial_setup.py (filename: V1__initial_setup.py)
DEBUG: Filtering out resource: /migrations/__init__.py (filename: __init__.py)
DEBUG: Filtering out resource: /migrations/__pycache__/__init__.cpython-311.pyc (filename: __init__.cpython-311.pyc)
DEBUG: Filtering out resource: /migrations/config.py (filename: config.py)
DEBUG: CherryPickConfigurationExtension not found
DEBUG: Filtering out resource: /migrations/V1__initial_setup.py (filename: V1__initial_setup.py)
DEBUG: Filtering out resource: /migrations/__init__.py (filename: __init__.py)
DEBUG: Filtering out resource: /migrations/__pycache__/__init__.cpython-311.pyc (filename: __init__.cpython-311.pyc)
DEBUG: Filtering out resource: /migrations/config.py (filename: config.py)
DEBUG: Filtering out resource: /migrations/V1__initial_setup.py (filename: V1__initial_setup.py)
DEBUG: Filtering out resource: /migrations/__init__.py (filename: __init__.py)
DEBUG: Filtering out resource: /migrations/__pycache__/__init__.cpython-311.pyc (filename: __init__.cpython-311.pyc)
DEBUG: Filtering out resource: /migrations/config.py (filename: config.py)
DEBUG: Filtering out resource: /migrations/V1__initial_setup.py (filename: V1__initial_setup.py)
DEBUG: Filtering out resource: /migrations/__init__.py (filename: __init__.py)
DEBUG: Filtering out resource: /migrations/__pycache__/__init__.cpython-311.pyc (filename: __init__.cpython-311.pyc)
DEBUG: Filtering out resource: /migrations/config.py (filename: config.py)

I also attempted to specify the ScriptMigrationResolver class as a resolver with the CLI argument -migrationResolvers=org.flywaydb.core.internal.resolver.script.ScriptMigrationResolver, but I got an error that it doesn't have a default constructor.

Can I get confirmation which docker image you are using?
Is it flyway/flyway or redgate/flyway?

Hi Barry, it's flyway/flyway

I think there has been a bit of miscommunication, both in our documentation and internally. Script Migrations are currently available in Redgate Community edition. As a result, if you were to use redgate/flyway at community level you would have access to this feature.
However, we will be putting this into the OSS edition in a future release.


Helm3 should not decode URLs

Related helm v2 issue - #6048
Output of helm version:
version.BuildInfo{Version:"v3.0+unreleased", GitCommit:"cc450563835088945aa79dbfa0bce0bc031fb390", GitTreeState:"clean", GoVersion:"go1.13.1"}

$ helm3 repo add blah http://localhost:8080/blah%2Fpath/blah
Error: looks like "http://localhost:8080/blah%2Fpath/blah" is not a valid chart repository or cannot be reached: Get http://localhost:8080/blah/path/blah/index.yaml: dial tcp [::1]:8080: connect: connection refused

Closing as https://github.com/helm/helm/pull/6602 is a port of #6060 and not a new issue.


Updated README

Added Fedora instructions
Updated OS requirements and message for Ubuntu 22.04

I verified the Fedora instructions on Fedora Server 40.


Allow to define different connection URIs for each simulated CS from the same template

Done in version 1.1.45


[Feature request] Don't show block title

If I embed "test#heading" it would be nice if there was an option to not show the heading in the embed, just the content after the heading. I've tried to use the 'after:' syntax but it doesn't seem to work for headings (or I'm misunderstanding it).
test.md
foobar

# Heading

this is a test

the following embed
path: [[test#Heading]]
heading: false

would show
this is a test

rather than
# Heading

this is a test

To use after, the body would be:
path: [[test#Heading]]
ranges: after "# Heading"

I just tested again, so it should work, but let me know if you have troubles.
In order to not add too many features, I'd prefer to keep it to this syntax for now, since it allows for more broad functionality.

Thanks.
I fiddled around for ages trying to get something like that to work, but somehow just completely misunderstood the documentation.


Increase the length limit of long notes

What feature do you want to get added on the base engine?
Sometimes there is just a part of your song with a very long note, which the limit of 3200 cannot cover.
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

ignore the previous version of this request lol

While you could go into the json to change it I understand that's difficult. I would like to see this added.

i think you can edit this in source.

Doubled the length once again, should be enough to fill 4 sections


CLI unchecked_clear

and optional unchecked_clear for "vacuum" & "snapshot"

LGTM

This only works with --vacuum or --snapshot right? Why?

Also, if we're going to keep it that way, we should warn/error if it's used without those options.

No, it also works separate

Oh, yeah I missed that.


feat(default-flatpaks): Integrate flatpaks into ISO once it's supported by isogenerator

Users would not need to wait for flatpaks to install once they boot into the system.
That installation step will be in the ISO installer screen instead.
Post-install systemd service would still remain, as it's still needed for applying updated changes from default-flatpaks recipe.
It's also needed to support image rebase installation method.
Related issue:
https://github.com/ublue-os/isogenerator/issues/9

This should be actually part of the ISO generation action if these Flatpaks are configured at ISO generation time and not in the image.

This should be actually part of the ISO generation action if these Flatpaks are configured at ISO generation time and not in the image.

You're right, I transfered this issue to this one:
https://github.com/blue-build/github-action/issues/20

Did you know that you can actually transfer issues, including the discussion, using a button down below? Anyways, nothing was lost here, good call on moving the issue!

I thought that only those with permissions could do so, like moderators or maintainers.
I learnt something new at least.

Oh yeah, this repo isn't moved yet so can't...


Fix vertical alignment of text between buttons

Fix vertical alignment in new ad preview page
Before:

After:

Make 'or' text translatable in submission form
Rename forum.css to custom.css, to make it more evident what file needs to be changed to adjust the site's stylesheet.

@moba in case you had looked at this PR before and was puzzled by the before/after screenshots, that's because I had inadvertently switched them! :sweat_smile:
I have now put them in the proper order, so it should make more sense now.


Added Doxygen For PKCS #11.

This commit syncs the Doxygen code that was used to generate the FreeRTOS.org PKCS #11 Doxygen.

Description

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

/bot run checks


update Julia version in README to match REQUIRE

Coverage remained the same at 0.0% when pulling faec755028c91f32c7e176cccee53d50e552dc93 on catawbasam:patch-1 into 318bf89bcec6a286666ccb6b4c602d6e7ba63308 on stevengj:master.

Coverage remained the same at 0.0% when pulling faec755028c91f32c7e176cccee53d50e552dc93 on catawbasam:patch-1 into 318bf89bcec6a286666ccb6b4c602d6e7ba63308 on stevengj:master.


Allow the use of the ssh agent for credentials

To improve security, I would rather not have my credentials known in any way to Terraform, and it would be great if this library allows the use of the SSH agent for credentials. According to https://github.com/appleboy/easyssh-proxy/blob/master/example/ssh/ssh.go#L15 this is already implemented in your proxy library, so this seems to be possible simply in the provider. For example, you could add a flag use_agent to explicitly enable this scenario and still give an error when a user unintentionally does not give either a password or private key.

Thanks for the suggestion, agree the default should be to use SSH agent to prevent the private key from making it into the state. I'll create a PR shortly

@FWest98 implemented in v0.3.0, if you can test that would be appreciated. Thanks!


"include FactoryGirl::Syntax::Methods" doesn't work in test_data block (Test::Unit)

#test_helper.rb
class Test::Unit::TestCase
  include FactoryGirl::Syntax::Methods
end

#some_controller_test.rb
require 'test_helper'

class SomeControllerTest < ActionController::TestCase
  test_data do
    create(:model)  #EXCEPTION
#=> NoMethodError: undefined method `create' for SomeControllerTest:Class
  end

  setup do
    create(:model) #WORKS
  end
end

So the short syntax methods don't work i test_data blocks for some reason, even if I include just before create.

History of lib/factory_girl/syntax/methods.rb shows that all methods was removed by commits c8c1a801ed22ae3695db7d2f29b2d8ff47971b9d and 95a4626daad3ec6e6833f48fb07ec9becaa13bd5. Maybe there is a new mechanism of dynamic loading of such methods, but it isn't described in the docs.

@artptr in those commits, we move to calling module_exec on FactoryGirl::Syntax::Methods to define them dynamically, so that shouldn't be the issue.
@BartlomiejSkwira does test_data yield an instance of Test::Unit::TestCase? If not, including in Test::Unit::TestCase won't work correctly.

Closed ude to inactivity; feel free to re-open if the issue still exists.


Add ability to send files with absolute filepath

Updated readme to include sendFile method

@wtfaremyinitials any timeline on when this will be merged?

any update?

It works, and has no conflicts. @wtfaremyinitials hit that green button 🙏

Bumping this since it's been about 6 years. @wtfaremyinitials , any chance you can merge this?


Clearing the input for a combobox of type "single" sets the value to an empty array

Combobox clearInput sets the value to an empty array when type is "single"

Using reactive forms - clear a selected value in the combobox (type="single") input using the x icon. The formcontrol value gets set to an empty array. For consistency I think it should be set to null, which is the value before any item is selected. The code  seems to have been originally intended to be fired for type "multi" only which would explain the inconsistency.

In combobox.component.ts
public clearSelected() {
		this.items = this.items.map(item => {
			if (!item.disabled) {
				item.selected = false;
			}
			return item;
		});
		this.view.items = this.items;
		this.updatePills();
		// clearSelected can only fire on type=multi
		// so we just emit getSelected() (just in case there's any disabled but selected items)
		const selected = this.view.getSelected();
		this.propagateChangeCallback(selected);
		this.selected.emit(selected as any);
	}

Bug

Combobox

Any update on this?


Refactor Toggler out of Tooltips

Issues
#99
The Tooltip and Popover patterns are almost identical. I've pulled out most of it into a Toggler pattern - Tooltip is now inheriting from Tooltip.
Impacted Areas
Tooltip and Popover patterns
Steps to Reproduce

Run npm install and the tests should still pass.
Link nightshade-core within nightshade and make sure the patterns still work.

Added click and hover tests! 🎉

@stewartyu I'm DTM upon fixing tooltip test

@stewartyu Feel free to merge this PR.

Thanks! Sorry, forgot to actually merge this.


more permissive angular version specifier

The documentation says it works on any Angular after 1.2.26. With this specific version modifier, it is causing conflicts with my other angular libraries.

Nevermind, i see this is already an open pull request.