Sort not working properly after 4-5 pages.

Describe the bug
Sort Order is not working as expected. after some pages its starts back from starting.
To Reproduce
Steps to reproduce the behavior:
curl --location 'http://localhost:7700/indexes/packagesPackage/search' \ --header 'Content-Type: application/json' \ --data '{"q": "api","sort": ["vendor:asc"], "filter": "(  marking = '\''symfony_ok'\'' )", "limit":128, "offset":158 }'
I am using above curl to get the data.
its working fine till 125 item. but for 126, 127,128 item its reset vendor to "A" char
Check the below response:
apiResult.json
Expected behavior
it should be ASC sort
Screenshots

Meilisearch version:
latest version

Here's it rendered on the web page;

Which comes from

Which populates the data from meilisearch, as described above.
So sorting isn't working as expected.

Hey @kailashkds, this  kind of issue can come from multiple things:
Your ranking-rules order
By default, Meilisearch uses multiple rules to order the results it returns, and the sort rule is not the first one.
This means if you didn't update the order of the ranking rule, Meilisearch would put more weight on the documents it deems are the most relevant ones and only use the sort rules as a tiebreaker.
If in your use-case it makes sense to sort everything by the vendor and only then order the results by the query you should put the sort ranking rule first in the list.
See the ranking-rules documentation.

[!WARNING]
If you do that and never have a tie with your vendor (all vendors are different), Then meilisearch will end up not taking your query into account at all.

Mixing numbers and string
If the field you're sorting on contains a mix of numbers and string, the number is going to appear first, which means 1 < "0".
The search cutoff
Since most clients prefer having a quick and mostly relevant answer than a slow answer, meilisearch has a notion of « cutoff ».
That means after some time, meilisearch will stop ranking documents and just return the documents in the state they were ranked at this point.
This behavior can produce this kind of issue, especially when going far away from the pages.
The default value for the cutoff is 1.5s (or 1500ms), so it doesn't seem to be your case here except if you updated the value yourself.
Here's the documentation of the search cutoff.
Investigating this kind of issue yourself
And finally, to investigate this kind of issue you may be interested in retrieving the ranking score details.
With this parameter in your search, the engine will return precisely why each result has this position, so you should be able to see if it's because of the sort, the cutoff, or something else.

can the ranking rules be configured on an individual search, rather than the index settings?
I want to ignore the sort when searching (but keep it for filtering without a "search", meaning a full text search), or rather, I want to use the normal relevancy ranking when a user is simply searching, but if they've requested a sort, to use that.
I'm  not sure how to do that since the two conditions are different at search time, not an index setting.
The use case might be something like "give me all the Symfony bundles related to APIs, sorted by the most recently changed".  Although we were using the vendor example because it was easy to see the problem.  But now I'm not sure of the solution, I don't want to give up relevancy if that's all that's being searched.

Hey,

can the ranking rules be configured on an individual search, rather than the index settings?

No, we can't currently.
But if I understand correctly your use case, making the sort the top-ranking rule should work because if you don't specify any sort rule at search time, then it has no impact on the ranking.

The use case might be something like "give me all the Symfony bundles related to APIs, sorted by the most recently changed". Although we were using the vendor example because it was easy to see the problem. But now I'm not sure of the solution, I don't want to give up relevancy if that's all that's being searched.

And a tip about this kind of use-case: You may want to bucket your values to give meilisearch the opportunity of sorting each bucket.
For example, I guess your documents have a updated_at field or something that contains a timestamp.
When writing this timestamp to Meilisearch you may want to round it to the current day, weeks or months maybe depending on your use-case.

I've asked the datatables.net author if it's possible to simply disable the sorting options when a full-text search filter is active.
Alternatively, I could load the dataset into a temporary table and apply sorting rules from there.  It's not my favorite solution.
There are variations of this theme, but any chance do you think ranking-rules order will ever be configurable at runtime?

There are variations of this theme, but any chance do you think ranking-rules order will ever be configurable at runtime?

From what I know, it's technically possible, but not prioritized currently.
Can you update this discussion: https://github.com/orgs/meilisearch/discussions/593
And drop a message explaining your use case, which helps us prioritize feature requests please :pray:


Use BEM for all custom CSS

I think CSS is probably more different between personal developers than any other language; so we could do with some stricter guidelines on custom styling. Especially when it comes to the maintainability of the CSS going forwards.
Any rules are better than no rules to some extent, but I'd suggest we use BEM:

It's got clearly defined rules
All rules are on classes so specificity stays under control
It plays great with SCSS
Developers could write classes to perform actions without needing to look them up in the CSS.

e.g. We currently have this.

      Channel info

          <input type="checkbox" checked>

 About this channel

About this channel to be here

          <input type="checkbox" checked>

 Pinned items

Pinned items content to be here

          <input type="checkbox" checked>

 Members

Members content to be here

Combined with a lot of styling based on element names, or nested styles. I'll point out some of the potential problem areas.
(I should point out, this is actually a nice pure-CSS solution for the info panel written by @nabiltntn, it's just the naming conventions and scalability that may need addressing).
.channel-info .title {} // .title could be used elsewhere, so this may not do what is expected
.channel-info .channel-accordion ul li {} // I actually tried to add another ul here for other reasons, heavily affected by this CSS

/* we're restricted to using a p element here */
.channel-info .channel-accordion {
 ul li input[type=checkbox]:checked ~ p {
       margin-top: 0;
       max-height: 0;
       opacity: 0;
      transform: translate(0, 50%);
     }
}

In BEM, we'd have something more like this:

      Channel info

          <input type="checkbox" checked>

 About this channel

About this channel to be here

and the classes used here make the CSS become markup agnostic.
// Blocks
.channel-info {}
.channel-accordion {}

// elements
.channel-info__title {}
.channel-info__section {}

.channel-accordion__list-item {}
.channel-accordion__list-item-content {}

// modifiers (or 'states')
.channel-info--out {}
.channel-footer--full {}
.channel-content--full {}

Lastly, using a pre-defined method would give us a great reference point to pass on to people trying to skin their own version; add new features; submit changes, etc.

What would the above CSS look like in SCSS with BEM?

Something like this:
 /* was .channel-info .title {}, reduced risk of unwanted overrides of .title */
 .channel-info__title {}

 /* was .channel-info .channel-accordion ul li, reduced specificity, markup dependance */
.channel-accordion__list-item {}

/* May be acceptable to still target input[type=checkbox] here rather than a class,
  specificity lowered, removed dependence on the ul>li structure and the paragraph element
*/
.channel-accordion__checkbox:checked ~ .channel-accordion__content {

}

I personally really don't like BEM.  I prefer the simple - identifiers and is[ClassName]

I like smaccs more when weer talking about structuring the CSS I even started off implementing aspects of smaccs in spacetalk as you can see

SMACSS is fine, very similar conceptually. Just needs something.
El El mié, 27 may 2015 a las 20:03, Dennis Stolmeijer <
<EMAIL_ADDRESS>escribió:

I like smaccs more when weer talking about structuring the CSS I even
started off implementing aspects of smaccs in spacetalk as you can see
—
Reply to this email directly or view it on GitHub
https://github.com/SpaceTalk/SpaceTalk/issues/82#issuecomment-106037363.

@rgoomar - I don't think anybody really likes the -- and __, but the - is just too ambiguous; and personal styles are too varied on a project with this many collaborators.
For example, if you have a class .channel-info. In BEM, I know that's a block in it's own right. Without that, I don't know if it's a block, or if it's an info element within a channel block; so my only way of knowing is to check the CSS. What most people do instead is just start overriding previous styles...
is[ClassName] is ok, but clashes with JavaScript where is[State] is usually a question (returns a boolean).
.channel-footer.isFull doesn't feel as natural to me when describing styles as .channel-footer-full, or, with BEM for maximum clarity: .channel-footer--full.
BEM is the only one of those three where you can look at the class name alone and know exactly which block it's operating on, and that it's a state of that block, without looking anywhere else. That's what the -- and __ gets you.

@adamdawkins That's fair and understandable. I would prefer SMACSS.

@rgoomar I once went a bit insane on a huge CSS project with SMACSS, I saw m-'s in my sleep. But I don't mind, that works for me.
I'll change the issue title to reflect that we need a CSS methodology (rather than just BEM)

What you guys think about Trello approach? I'm not doing a lot of front-end stuff but I think it solves a lot of tough problems with css.

@mpowaga I like trello's approach but idk to formalize this.. would be harder for new people to get in


[11.x] Add "Server has gone away" to DetectsLostConnection

Closes #51229

Probably this should target 10.x, if we want to treat this as a bug.

@GrahamCampbell you are right, I will resend on 10.x


Flag columns - inconsistent new column names

Need consistent naming schema. Do we want to add WQX when we rely on the WQX validation table? Alternatively, would users prefer all include TADA at the beginning?
Columns added:
BelowWQXLowerThreshold
AboveWQXUpperThreshold
WQX.SampleFractionValidity
WQX.ResultUnitValidity
WQX.MethodSpeciationValidity
WQX.AnalyticalMethodValidity
TADA.PotentialDupRowID
TADA.InvalidCoordinates
TADA.AggregatedContinuousData

add TADA. for all for consistency; also, no reordering

Suggestions to discuss with team:
TADA.ResultValue.BelowWQXLowerThreshold
TADA.ResultValue.AboveWQXUpperThreshold
TADA.SampleFraction.WQXValidation
TADA.ResultUnit.WQXValidation
TADA.MethodSpeciation.WQXValidation
TADA.AnalyticalMethod.WQXValidation
TADA.PotentialDupRowIDs
TADA.InvalidCoordinates
TADA.AggregatedContinuousData

@mthawley @ehinman @katiehealy What do you think of these new names (comment above) & format for all columns added by TADA functions?

I like them!

@cristinamullin Those look great to me, with one addition:
TADA.CensoredData.Flag
Does that look okay? Or suggest something different? This column lets user know if the limit and detcond agree/disagree.

Looks great!


overall styling improvements to site

In this pull-req I've made the following progress:

Closes #34
Progress on #29 and #36
Refactored SCSS a bit

This was merged as part of #46. Great work @bnvk


Feature/keyvault/7.5

Added CertificateProperties.getX509ThumbprintHexString() and updated service version to 7.5.
Will add test recordings soon.

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-security-keyvault-administration


Revert "1.0.0.RC1"

Reverts spring-cloud/spring-cloud-gcp#795

@meltsufin What's the reason for reverting?

@ChengyuanZhao The release has been performed. Reverting back to snapshots.


Fix "Controlling stylesheet precedence" example

Closes https://github.com/facebook/react/issues/31621

Apologies didn't see this, already merged https://github.com/reactjs/react.dev/pull/7235. Hope it's correct.
Please don't hesitate to merge this time, we really shouldn't have completely wrong docs for features stay up for months


Change error responses for Places API

Fixes issue #4   Change error responses for Places API

@acbuitrago Thanks for the contribution :+1:
I have added your commit and made some minor changes on the top of it. See this commit
Closing the PR!


Add spotless and editor-config

Add spotless and editor-config
Add spotless and editor-config

Current dependencies on/for this PR:

main

PR #46   👈

This stack of pull requests is managed by Graphite.


fix(mission_planner): revert "refactor(mission_planner): use combineLaneletsShape in lanele2_extension (#5535)"

revert (#5535)"
This reverts commit c4ca645cbbd0a7b919ba1c2cdf78742bb81b87b5.because of  degradation
only combine_lanelets can combine neighbor lanes
Description

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

sorry and thanks for the quick reply :bow:


Refactor: consolidate 'compareArrays' implementations

The consolidated implementation is below for reference.  It's believed to be semantically equivalent and backwards compatible.
/**
 * Compare two arrays.  Returns true if their elements are equivalent and in the same order.
 *
 * @param left - The first array to compare
 * @param right - The second array to compare
 * @param comparator - The function used to check if two `T`s are equivalent.
 * Defaults to `Object.is()` equality (a shallow compare where NaN = NaN and -0 ≠ 0)
 */
export const compareArrays = <T>(
	left: readonly T[],
	right: readonly T[],
	comparator: (leftItem: T, rightItem: T, index: number) => boolean = (
		leftItem: T,
		rightItem: T,
	) => Object.is(leftItem, rightItem),
): boolean => {
	// PERF: 'for-loop' and 'Array.every()' tied.
	//       '===' and 'Object.is()' tied.
	//       Trivial acceptance adds no measurable overhead.
	//       30% penalty vs. baseline for exported function [node 14 x64].
	return (
		left === right || // Trivial acceptance: 'left' and 'right' are the same instance
		(left.length === right.length && // Trivial rejection: 'left' and 'right' are different lengths
			left.every((leftItem, index) => comparator(leftItem, right[index], index)))
	);
};

sounds like package.json thing is on Tyler's radar and for now we should stick with spaces


nixos/stage-1: fix antiquotation

Motivation for this change
There is a glaring antiquotation error in the builder for extra-utils.
Things done

[ ] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Err. Any idea how did it even evaluate before?

...oh, I see https://github.com/NixOS/nix/issues/1017 now.


[Feature Request]: add news functionalily

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
the current new functionality is not working,

i will make new section with backend support
the news will automatically update

Use Case
their is a news section that meant to be used,
Benefits
No response
Add ScreenShots

Its loading for more than 10 mins.
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@manikumarreddyu assign me this issue under hacktoberfest and gssoc-ext, and i expect a level3 for this.

hi @AE-Hertz it is working fine..in local..there is no issues in code..it is deployment issue..

hi @AE-Hertz it is working fine..in local..there is no issues in code..it is deployment issue..

Ok then, is their anything else in which I can contribute? Then pls let me know

come with some new ideas. @AE-Hertz


occurrences details in search results

Our requirement is find all occurrences by giving  recurring id(appointment id) i.e expand view of search results
#1) To implement this I tried to use ExchangeService.FindAppointments , One way to find recurring series including occurance
https://msdn.microsoft.com/en-us/library/office/dn727655(v=exchg.150).aspx#sectionSection2
CalendarView  calView = new CalendarView(currentappt.getRecurrence().getStartDate(),
appt.getRecurrence().getEndDate());
calView.setPropertySet(recurringMasterProperties);
FindItemsResults<Appointment> recurringMasterResults = exService.findAppointments(	WellKnownFolderName.Calendar,calView);

Issues here is it includes other appointments also in result set as I am sending given appointment start date and end to Calenderview
In some cases results not showing all occurrence(it is not paging issue)
#2) I tried other workaround to achieve our requirement of showing occurrences of appointment in search results
Every appointment (single , reoccurrence ) will have unique conversation id , i.e All occurrences of given recurrence will have same conversation id
https://msdn.microsoft.com/en-us/library/microsoft.exchange.webservices.data.exchangeservice.getconversationitems(v=exchg.80).aspx
but It seems EWS JAVA API does not have API (Overloaded  api like EWS Microsoft ) to get Conversations based on ID
ConversationIndexedItemView conView = new ConversationIndexedItemView(10);
exService.findConversation(conView,folderID)
Example : Get items in a single conversation by using the conversation identifier
Is there any way to achieve to show all  occurrences details in search results .
Please let us know the correct approach with some sample code.
Thanks,
Showkath.

Hi,
The code which  i tried was working , did not test properly in the past  because  "In some cases results not showing all occurrence(it is not paging issue) , I could have  tested more calendar events
If any one facing similar issue , you can try below approach which works
1 ) "To implement this I tried to use ExchangeService.FindAppointments , One way to find recurring series including occurance
https://msdn.microsoft.com/en-us/library/office/dn727655(v=exchg.150).aspx#sectionSection2"


Return Location Header

As a User
I need  404 status code returned
So that I know that the record is not in the DB
Assumptions

You have read the professor's feedback on Sprint 1

Acceptance Criteria
Given all endpoints
When fetching a record that doesn't exist
Then it should return status code 404, not 500.

You need to check if you got something back before calling the serialize() method on a recommendation.


test(request): increasing test coverage of request module

Adding more tests for request.

Coverage increased (+0.14%) to 88.21% when pulling 2ce7ebc888bae97bb509c84069a312b50cbe39cf on winding-lines:result-cov into 221f2c9b3c232f40a8a1575f2f952530897e1840 on hyperium:master.

Coverage increased (+0.96%) to 89.02% when pulling 06c594063059689c5990e30c1311152701f8cb37 on winding-lines:result-cov into 221f2c9b3c232f40a8a1575f2f952530897e1840 on hyperium:master.

Coverage increased (+1.11%) to 89.18% when pulling 43e274cfce9442191e6cc04e569c785b2aeccbf9 on winding-lines:result-cov into 221f2c9b3c232f40a8a1575f2f952530897e1840 on hyperium:master.

Great, thanks!


update wp-hooks dependency and adjust new json format

Fixes https://github.com/humanmade/psalm-plugin-wordpress/issues/25

Thanks!


Update rust toolchain version for Kani 0.20

Biweekly update for the rust toolchain version. For Kani 0.20, we will target the new year version: nightly-2023-01-16.

#2045 updated the toolchain to nightly-2022-12-11.


Error message updated in case of directory create failure when parent directory doesn't exists or having only read-only permission.

Signed-off-by: Ashwin<EMAIL_ADDRESS>
Description
Hi..
With all my understanding I checked all these scenarios where I was able to reproduce the issue.
Here are the following scenarios. (Screenshots attached below.)
Case 1:

When the parent directory exists.

If the parent directory exists and is writable as well then we won't face any issue. Everything will look good. Please refer the
Screenshot below.

Screenshot:

Case 2:

When the parent directory exists but having on "read only" permission.

In this case we will be facing insufficient permission issue which is correct.

Screenshot:

Case 3:

When the parent directory doesn't exists.

In this case also we are seeing the insufficient permission issue but I guess we should see the different error message like - "Parent Directory does not exists"
I have made these changes accordingly. Please refer the Screenshot

Screenshots:

PS: Earlier in case if the directory was does not exists, then also we were seeing the insufficient permissions error. But I have added one extra condition to show the parent directory does not exists error.
Related Issue
#11641
Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

chefstyle errors need fixing

@ashwin-msys can you look at your unit test failures here?

@johnmccrae note to self - this might not rebase, in which case, close it. If it does rebase, see if it can be merged

@johnmccrae - ping?


Update Python.gitignore with .DSStore for macOS

Reasons for making this change:
Ignore .DS_Store. This file will exist in every folder if working on a Mac. Let's put it in the python.gitignore file so that we can use this one .gitignore file on a Mac.
Links to documentation supporting these rule changes:
https://en.wikipedia.org/wiki/.DS_Store

@gatto thanks for the contribution, but we have an existing Global/macOS.gitignore template that contains this and many other macOS-specific rules.
We recommend incorporating that into your projects.

Thanks @shiftkey, perhaps I tried to solve my issue in a way that wasn't appropriate. I created an issue https://github.com/desktop/desktop/issues/8284#issue-494649823 that is hopefully in a better spot to solve the problem I see.


[SEO][Rollout Monikered Canonical]Add URL moniker

Enable build config to generate canonical URL with moniker

Docs Build status updates of commit a660c74:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mipsdk-dotnet/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


changelog: 0.15.0 doesn't communicate the changes for the user

Description
The changelog for 0.15.0 doesn't sufficiently tell users of this repo what the actual changes from there PoV were. Especially for the newly introduced light-client it's just lists PRs:
https://github.com/informalsystems/tendermint-rs/blob/3a14d8b4bb806e9b04e22a11cdf7c2f9e1e8b10f/CHANGES.md#0150-2020-07-17
Solution
We've decided to rework the Changelog for 0.15.0 by 1) substantially shortening it and 2) to be more aligned with the contribution guidelines: https://github.com/informalsystems/tendermint-rs/blob/master/CONTRIBUTING.md#changelog

Unassigned my self as I'll be offline for a few days. Can take this when I'm back if it wasn't closed until then.


spyder not working

when i use pyqt5==5.9.2
then spyder works but when it upgrade to 5.10.1 it dont

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
What steps reproduce the problem?

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version:
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

then spyder works but when it upgrade to 5.10.1 it dont

And that's why the latest PyQt5 version we support is 5.9, so please keep using it.

but if one have to use  pqt5 of higher version for other purpose then what one must do?

Please read this to understand how to work with multiple envs:
https://github.com/spyder-ide/spyder/wiki/Working-with-packages-and-environments-in-Spyder

whats the reason its not working on 5.10?


Bluesky => AP: post with link got converted to reply

This is a weird one, I still don't understand what happened. @mackuba replied (now deleted) to this Bluesky post with a link embed to this Mastodon post. When we converted that reply to AS2, we for some reason set inReplyTo to the linked Mastodon post, which is clearly wrong. I haven't been able to reproduce that bug locally yet.
Bluesky post record:
{
  "$type": "app.bsky.feed.post",
  "createdAt": "2024-08-05T21:38:18.901Z",
  "facets": [
    {
      "$type": "app.bsky.richtext.facet",
      "features": [{
        "$type": "app.bsky.richtext.facet#mention",
        "did": "did:plc:6x3sa4qcuhtc4yky2f6eu2ki"
      }],
      "index": {
        "byteEnd": 18,
        "byteStart": 0
      }
    },
    {
      "$type": "app.bsky.richtext.facet",
      "features": [{
        "$type": "app.bsky.richtext.facet#mention",
        "did": "did:plc:esmiuxk53vmsllayghrq676w"
      }],
      "index": {
        "byteEnd": 40,
        "byteStart": 19
      }
    },
    {
      "features": [{
        "$type": "app.bsky.richtext.facet#link",
        "uri": "https://mkultra.monster/@cmdr_nova/112909755730245873"
      }],
      "index": {
        "byteEnd": 105,
        "byteStart": 74
      }
    }
  ],
  "langs": ["en"],
  "reply": {
    "parent": {
      "cid": "bafyreibbho5r2wj7thdv3insxt234taukpeszhbqd3kukxlpzbonz35mhy",
      "uri": "at://did:plc:6x3sa4qcuhtc4yky2f6eu2ki/app.bsky.feed.post/3kyt3df7obk27"
    },
    "root": {
      "cid": "bafyreibssi2nwd4mwfw5youwqmjtpmayfnpviz6enb7lbhnvuh2qbauekm",
      "uri": "at://did:plc:oio4hkxaop4ao4wz2pp3f4cr/app.bsky.feed.post/3kyme6oeauk2j"
    }
  },
  "text"<EMAIL_ADDRESS>@o.simardcasanova.net check out the last sentence here<EMAIL_ADDRESS>\ud83e\udee0"
}

AP inbox delivery log with full AS2 activity:
Task<PHONE_NUMBER>5723039811
Params: [('obj', 'ahBicmlkZ3ktZmVkZXJhdGVkcmQLEgZPYmplY3QiWGF0Oi8vZGlkOnBsYzpvaW80aGt4YW9wNGFvNHd6MnBwM2Y0Y3IvYXBwLmJza3kuZmVlZC5wb3N0LzNreXl0emhpazYyMjQjYnJpZGd5LWZlZC1jcmVhdGUM'), ('orig_obj', 'ahBicmlkZ3ktZmVkZXJhdGVkckELEgZPYmplY3QiNWh0dHBzOi8vbWt1bHRyYS5tb25zdGVyL0BjbWRyX25vdmEvMTEyOTA5NzU1NzMwMjQ1ODczDA'), ('protocol', 'activitypub'), ('url', 'https://mkultra.monster/users/cmdr_nova/inbox'), ('user', 'ahBicmlkZ3ktZmVkZXJhdGVkci0LEgdBVFByb3RvIiBkaWQ6cGxjOm9pbzRoa3hhb3A0YW80d3oycHAzZjRjcgw')]
Sending activitypub post at://did:plc:oio4hkxaop4ao4wz2pp3f4cr/app.bsky.feed.post/3kyytzhik6224#bridgy-fed-create to https://mkultra.monster/users/cmdr_nova/inbox
Signing with did:plc:oio4hkxaop4ao4wz2pp3f4cr 's key
requests.post https://mkultra.monster/users/cmdr_nova/inbox {
  "id": "https://bsky.brid.gy/convert/ap/at://did:plc:oio4hkxaop4ao4wz2pp3f4cr/app.bsky.feed.post/3kyytzhik6224#bridgy-fed-create",
  "actor": "https://bsky.brid.gy/ap/did:plc:oio4hkxaop4ao4wz2pp3f4cr",
  "published": "2024-08-05T21:38:19.556403+00:00",
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Create",
  "object": {
    "id": "https://bsky.brid.gy/convert/ap/at://did:plc:oio4hkxaop4ao4wz2pp3f4cr/app.bsky.feed.post/3kyytzhik6224",
    "url": "https://fed.brid.gy/r/https://bsky.app/profile/did:plc:oio4hkxaop4ao4wz2pp3f4cr/post/3kyytzhik6224",
    "content": "
<a<EMAIL_ADDRESS><a<EMAIL_ADDRESS>check out the last sentence here <a href=\"https://mkultra.monster/<EMAIL_ADDRESS>\\ud83e\\udee0
",
    "published": "2024-08-05T21:38:18.901Z",
    "contentMap": {
      "en"<EMAIL_ADDRESS>@o.simardcasanova.net check out the last sentence here<EMAIL_ADDRESS>\\ud83e\\udee0"
    },
    "attributedTo": "https://bsky.brid.gy/ap/did:plc:oio4hkxaop4ao4wz2pp3f4cr",
    "type": "Note",
    "inReplyTo": "https://mkultra.monster/users/cmdr_nova/statuses/112909755730245873",
    "tag": [
      {
        "type": "Mention",
        "name"<EMAIL_ADDRESS>        "href": "https://bsky.brid.gy/ap/did:plc:6x3sa4qcuhtc4yky2f6eu2ki"
      },
      {
        "type": "Mention",
        "name"<EMAIL_ADDRESS>        "href": "https://bsky.brid.gy/ap/did:plc:esmiuxk53vmsllayghrq676w"
      },
      {
        "url": "https://mkultra.monster/@cmdr_nova/112909755730245873",
        "type": "Article",
        "name"<EMAIL_ADDRESS>      },
      {
        "type": "Mention",
        "href": "https://mkultra.monster/users/cmdr_nova"
      }
    ],
    "cc": [
      "https://mkultra.monster/users/cmdr_nova",
      "https://www.w3.org/ns/activitystreams#Public",
      "https://mkultra.monster/users/cmdr_nova/followers"
    ],
    "content_is_html": true,
    "to": [
      "https://www.w3.org/ns/activitystreams#Public"
    ]
  },
  "to": [
    "https://www.w3.org/ns/activitystreams#Public"
  ]
}
Received 202

Here's how I tested the conversion locally:
o = Object.get_by_id('at://did:plc:oio4hkxaop4ao4wz2pp3f4cr/app.bsky.feed.post/3kyytzhik6224')
p(ActivityPub.convert(o))

{
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Note",
  "id": "https://bsky.brid.gy/convert/ap/at://did:plc:oio4hkxaop4ao4wz2pp3f4cr/app.bsky.feed.post/3kyytzhik6224",
  "url": "https://fed.brid.gy/r/https://bsky.app/profile/did:plc:oio4hkxaop4ao4wz2pp3f4cr/post/3kyytzhik6224",
  "inReplyTo": {
    "id": "https://bsky.brid.gy/convert/ap/at://did:plc:6x3sa4qcuhtc4yky2f6eu2ki/app.bsky.feed.post/3kyt3df7obk27",
    "url": "https://bsky.app/profile/did:plc:6x3sa4qcuhtc4yky2f6eu2ki/post/3kyt3df7obk27"
  },
  "content": "
<a<EMAIL_ADDRESS><a<EMAIL_ADDRESS>check out the last sentence here <a href=\"https://mkultra.monster/<EMAIL_ADDRESS>\ud83e\udee0
",
  "published": "2024-08-05T21:38:18.901Z",
  "attributedTo": "https://bsky.brid.gy/ap/did:plc:oio4hkxaop4ao4wz2pp3f4cr",
  "tag": [{
      "type": "Mention",
      "name"<EMAIL_ADDRESS>      "href": "https://bsky.brid.gy/ap/did:plc:6x3sa4qcuhtc4yky2f6eu2ki"
    },{
      "type": "Mention",
      "name"<EMAIL_ADDRESS>      "href": "https://bsky.brid.gy/ap/did:plc:esmiuxk53vmsllayghrq676w"
    },{
      "url": "https://mkultra.monster/@cmdr_nova/112909755730245873",
      "type": "Article",
      "name"<EMAIL_ADDRESS>    }
  ],
  "to": ["https://www.w3.org/ns/activitystreams#Public"],
  "content_is_html": true
}

Reproduced!

https://hachyderm.io/@shreyan/112573306835667555
https://mastodon.social/@psionides/112916384545607094

In the second case I even got a notification on that Mastodon account 💀

wowww. what the hell is even going on here 😠

:p

Uh… so… this happened again (nothing sensitive, can stay there), but this time also the whole subthread from Bluesky of people replying to my comment with the link got all sent there to that thread too! :O https://mastodon.social/@fediversecounter/112920443483887367

Uh… so… this happened again (nothing sensitive, can stay there), but this time also the whole subthread from Bluesky of people replying to my comment with the link got all sent there to that thread too! :O https://mastodon.social/@fediversecounter/112920443483887367

The latter part is expected, as ActivityPub replies only encode the direct parent.
(More specifically, Mastodon does keep track of threads in its database, but it resolves them on receipt of a reply by walking (fetching) backwards until it finds a post it already has. If that fails then the reply becomes permanently detached in some ways, iirc.)

Found the problem here, the culprit is orig_obj passed through send tasks to ActivityPub.postprocess_as2.  Trying to incrementally clean it up a bit now.
https://github.com/snarfed/bridgy-fed/blob/39aa16be6258d80ac619ae7c176f76e506a1f3d2/activitypub.py#L696-L700

OK, fix here should be deployed in ~5m. Thanks and props again to @mackuba for catching and investigating this!
Also, re the root cause here, orig_obj is cursed. Gotta get rid of it. Not sure how, since most fediverse software still doesn't understand multiply-valued inReplyTo or object (eg https://funfedi.dev/support_tables/generated/in_reply_to/)...but still, orig_obj is cursed. Gotta get rid of it.

👻


[feature request]improve discardcloudservice filter in yurthub component

What would you like to be added:
Because clients on edges nodes can not connect cloud applications through pod ip, so LoadBalancer service for cloud application is discarded in yurthub component in order to make kube-proxy unaware of LoadBalancer service on edge nodes. and only LoadBalancer service with annotation openyurt.io/skip-discard=true will be skipped.
the following are detail codes:
https://github.com/openyurtio/openyurt/blob/81d21b7f438be4fb912ebe9748f0c9a4a9296f5f/pkg/yurthub/filter/discardcloudservice/filter.go#L81-L86
but in the proposal: https://github.com/openyurtio/openyurt/blob/master/docs/proposals/20231121-high-availability-of-edge-services.md, LoadBalancer service across NodePools will be supported, this means that LoadBalancer service should be aware of on edge nodes.
so we will improve discardcloudservice filter in yurthub component, and only discard LoadBalancer service which has annotation svc.openyurt.io/discard=true, this means that LoadBalancer service will not be discarded by default.
others
/kind feature

@gnunu @JameKeal @YTGhost Do you have any comments?

It seems we first need to add a new annotation svc.openyurt.io/discard=true after line 48 in the following code
https://github.com/openyurtio/openyurt/blob/81d21b7f438be4fb912ebe9748f0c9a4a9296f5f/pkg/yurthub/filter/constant.go#L48
and then we need to modify the following code, discard LoadBalancer service which has svc.openyurt.io/discard=true, instead of which doesn't have openyurt.io/skip-discard=true.
https://github.com/openyurtio/openyurt/blob/81d21b7f438be4fb912ebe9748f0c9a4a9296f5f/pkg/yurthub/filter/discardcloudservice/filter.go#L81-L86
I wonder if I'm on the right track. I would like to try this issue.

It seems we first need to add a new annotation svc.openyurt.io/discard=true after line 48 in the following code
https://github.com/openyurtio/openyurt/blob/81d21b7f438be4fb912ebe9748f0c9a4a9296f5f/pkg/yurthub/filter/constant.go#L48
and then we need to modify the following code, discard LoadBalancer service which has svc.openyurt.io/discard=true, instead of which doesn't have openyurt.io/skip-discard=true.
https://github.com/openyurtio/openyurt/blob/81d21b7f438be4fb912ebe9748f0c9a4a9296f5f/pkg/yurthub/filter/discardcloudservice/filter.go#L81-L86
I have raised a pr to implement the logic described above to fix this issue.

@huangchenzhao yes, you're right. by the way, unit tests also should be tuned.


.sh support

I believe .sh for bash scripting would be a great addition for support to this plugin.

Duplicate of #451.


bug(overlay): overlays are sometimes blurred in Chrome

I think you should change the text to "Use your eye glasses to see tooltips"

PS: My chrome zoom is set to 100%

+1
This is because the transforms on the parent .md-overlay are not integers, any way those can be rounded to the nearest pixel?

Indeed, it would be nice to have them rounded.

@zigzag95 Are there any specific steps or browser requirements to reproduce this?

Actually, I we were able to recreate it only when using a Macbook Pro with a non-retina display on Chrome.  Let me know if this happens under different circumstances as well.

ChromeOS (I'm going to assume that will include Linux) also has the issue. Can test on Windows soon.

Ah, okay.  We tested on Linux and were not able to recreate it so far.  Haven't checked Windows yet.

@robertmesserle Although I've tested with the menu component, both Windows and Linux have this issue. I would assume the amount of blur also varies with ppi.

Yea, it should impact anything built on top of Overlay (ie. anything floating).
Some notes:

Rounding pixels values should fix some use-cases, but not all.
Percent-based translation will still cause blurring with no easy workarounds (at least not that we've found yet).

@robertmesserle Since currently transform is used only for overlay centering, maybe other centering technique should be used instead.
Getting rid of transform could also make animation more flexible

Summing up status on this so far: this is caused by subpixel rendering in some browsers. Solution is to always round the transform values for overlay positions.

We should try out this trick with calc:
http://stackoverflow.com/questions/39099656/blurry-text-when-using-translatex-translatey-in-percentage-on-chrome

I think it may work for even numbers of width and height only

@jelbourn Some warning about using calc there, IE/Edge can't animate properties with it. That won't be an issue though if the .md-overlay's transform isn't being animated anyway.

@crisbeto for connected position strategy, this should be as straightforward as calling Math.round on the translate values. For global position strategy, we may need to getBoundingClientRect after applying the percent translates and then reapply the rounded values (unless we can find a cleaner approach).


using onBefore/onStart for authentication problem

Hi,
The docs don't really cover this scenario and I'm having issues getting this to work in the latest ui-router version ( "angular-ui-router": "1.0.0-beta.3" ).
upon any route change, i need to make sure that a user is authenticated.
i don't want to reroute them to a 'login' state, I just want to make an API call which populates cookie, and then only after that call returns, Do I want the app to be loaded and routed to the intended state.
this current implementation does not wait for the doAuth function to finish it's callback, therefore I have components being loaded and getting login errors etc.
 $transitions.onBefore({ }, () => {
            const deferred = $q.defer();
            session.checkAuth().then(() => {
                deferred.resolve(true)
            }, () => {
                deferred.resolve(false)
            });
            return deferred.promise;
        }, {priority: 10});

I know this implementation sucks so far, but can someone provide a little bit of guidance ?
the repo at https://github.com/ui-router/sample-app-ng1-to-ng2 does not cover this type of scenario. it just redirects to a 'login' state.
Thanks

The docs for HookResult explain how promises work when returned from a hook:

Promise: the transition will wait for the promise to resolve or reject

If the promise is rejected (or resolves to false), the transition will be cancelled
If the promise resolves to a TargetState, the transition will be redirected
If the promise resolves to anything else, the transition will resume

In your case, your hook should check if the users is authenticated.  If they are not, set the cookie and let the transition continue.
Using standard promise behavior, call .catch() to handle the case where they are not authenticated.  If you reject the promise (or return the rejection from checkAuth() the transition would be cancelled.  Instead, handle the rejection and let the transition continue.
$transitions.onBefore({ }, () =>
  session.checkAuth().catch(() => cookieservice.setcookie()));

Hope this helps

HI! How would I do the same thing, but instead of setting a cookie when the checkAuth() request from my API fails, it would redirect the user to the login page without the transition hook successfully resolving?
This is the code I have:
app.js
angular.module('app')
  .run(function ($state, $transitions, AuthService) {
    $transitions.onBefore({ to: 'auth.**' }, function() {
      AuthService.isAuthenticated().then(function (isAuthenticated) {
        if (!isAuthenticated) {
          $state.go('login');
        }
      });
    });
  });

I'm using the $state service to redirect the user to the login page when the unauthenticated user tries to access an auth restricted view. But with this implementation, the transition onBefore() is already resolved, so the transition is succeeding before my checkAuth() method finishes. So it's still showing the view its going to for a (split) sec, before it transitions to the login view.
Here is the implementation of the auth service methods used in the code above:
auth.service.js
authService.isAuthenticated = function () {
  // Checks if there is an authenticated user in the app state.
  var authUser = AuthUserStore.get();
  if (authUser) {
    return Promise.resolve(true);
  }
  // Do check auth API request when there's no auth user in the app state.
  return authService.checkAuth()
    .then(function () {
        return !!AuthUserStore.get();
      });
};

authService.checkAuth = function () {
  return $http.get(API.URL + 'check-auth')
    .then(function (res) {
        // Store the user session data from the API.
        AuthUserStore.set(res.data);
        return res;
      });
};

@neilpette hi neilpette, i view your code (er i am a fresh man in ui-router), but as christopherthielen said, may be u should return the promise in transition hook
that means your code should be
angular.module('app')
  .run(function ($state, $transitions, AuthService) {
    $transitions.onBefore({ to: 'auth.**' }, function() {
      return AuthService.isAuthenticated().then(function (isAuthenticated) {
        if (!isAuthenticated) {
          $state.go('login');
        }
      });
    });
  });

as transition hook will wait for the promise to resolve or reject so it will wait for your authentication completed
but there is one thing confused me, that is what the difference between $state.go and return a TargetService  in transition hook ?


Run tests using testcontainers

On top of #260

Use testcontainers to run tests instead of docker-compose.
Use minio/minio for all but one S3 tests (minio doesn't support files with trailing slashes).
Use adobe/s3mock for all but two S3 tests (s3mock doesn't support advanced metadata).

Should tell scalac about targetting jdk 11 maybe, scalacOptions += "-target:11"

Should tell scalac about targetting jdk 11 maybe, scalacOptions += "-target:11"

val javaVersion: Int = {
var sysVersion = System.getProperty("java.version")
if (sysVersion.startsWith("1."))
  sysVersion = sysVersion.drop(2)

sysVersion.split('.').head.toInt

}
val javaOpts: Seq[String] = javaVersion match {
case x if x < 10  => Seq("-source", s"1.$x", "-target", s"1.$x")
case x if x >= 10 => Seq("-source", s"$x", "-target", s"$x")
}


My feature

The submitted GitHub PR diff shows changes made to multiple files. The changes will be listed below according to the file index:

MusicPlayerCore/index.ts: The code snippets that start with - represent old code that got removed, whereas those starting with + highlight new code additions. In this file, variables and function names have been changed, some formatting updates have been made (blank lines added), a comparison operation has been changed from assignment to equality (===), and a function name is altered in a non-standard form (PrevSong to Prev-Song).

VueMiniPlayerCore/index.ts: This file also experienced changes in variable and function names. It seems to violate the coding standard as well, where identifiers were changed from camelCase to a non-standard form.

VueMiniPlayerCore/index_2.ts: This is a new file added to the repository. However, as per the given diff, it does not conform to the coding standard for multiple reasons, such as naming conventions, inconsistent indentation, and potential variable shadowing (reuse of loop counter named p).

VueMiniPlayerCore/new-index.ts: This is also a new file added to the repository. It presents similar violations to the coding standard as observed in the index_2.ts file.

Do note that this assessment is based merely on the changes shown by the diff PR. A comprehensive check would involve going through the files in their entirety. However, from the given information, it is clear that the changes in the PR potentially cross some of the coding conventions set out in the Coding Standards Document.


ARROW-559: Add release verification script for Linux

Since we're accumulating a bunch of components, I started this script which we can refine to make verifying releases easier for others.
I bootstrapped some pieces off https://github.com/apache/parquet-cpp/blob/master/dev/release/verify-release-candidate, very helpful!
This script:

Checks GPG signature, checksums
Installs temporary Python install for the duration of these tests
Builds/install C++ and runs tests (with Python and Plasma)
Builds parquet-cpp against the Arrow RC
Python (with Parquet and Plasma extensions)
C GLib (requires Ruby in PATH and the gems indicated in README)
Integration tests
JavaScript (requires NodeJS >= 6.0.0)

There are some potentially snowflake-y aspects to my environment:

BOOST_ROOT is set to a Boost install location containing libraries built with -fPIC. I'm not sure what to do about this one. One maybe better option is to use system level boost and shared libraries
Maven 3.3.9 is in PATH
NodeJS 6.11.3 is in PATH

There are probably some other things that Linux users will run into as they run this script.
I had to compile GLib libraries in this since the ones at system level (Ubuntu 14.04) are too old.
cc @kou @xhochy

I'm going to change the Boost stuff to use shared libraries since using static libraries will cause problems for people using libraries from a Linux package manager

This looks nice, I'm working on Darwin support

OK, no problem


Install Angular Material 2

Angular Material 2 doesn't work  :(
https://github.com/tessa3/uxgraph/tree/secondary-toolbar


Logo Design Offer as Open Source Contribution

Hello Sir. I'm a UI/UX and Graphics Designer. I want provide a logo for you. Would you mind if I propose a new logo design for your project as my Open Source Contribution?
Thanks before.

No, certainly not. Actually, I am planning to do this.

Okay Sir, No problem.

Thank you!


what role of ticket in this dao cods ?

Hi
I have a question about your codes . I Could not understood ticket function , why you use ticket function ?

otherwise it is possible to vote as many as they want.
thank you !!

this might be a resource consumption issue. but it is nonsense to skip substrate's governance pallets and go further.


clay lava bucket return a iron bucket

buckets from https://www.curseforge.com/minecraft/mc-mods/ceramic-bucket
return iron bucket when used

They did not set up the bucket with proper empty bucket as containerItem, this is on ceramic-bucket side


Live Share not launching sign-in in browser

Issue Type: Bug
click join link
click "sign in"
choose microsoft or github
nothing happens
Extension version: 1.0.2302
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T09:03:20.462Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz (8 x 2794)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.95GB (21.42GB free)

Process Argv

Screen Reader
no

VM
0%

bump

seeing the same

+1

+1

Same here. I should add that it does prompt the browser login/auth once I press cancel...

Same here. I should add that it does prompt the browser login/auth once I press cancel...

Yep, same here. I dunno why we aren't getting answers from the devs. This is beyond a joke now.

Thanks for reporting this issue!
@Ravenwulf, @mrskme, @freben, @kelyndm: It sounds like you were encountering this problem when attempting to join a Live Share session from a browser. If this is the case we recently fixed the issue. Please let us know if you're still seeing the problem.
@barnabasmolnar, @ryz0o: were you attempting to join from a browser as well or from VS Code desktop? We're only aware of this particular issue for the Live Share browser experience so if you're encountering it on VS Code desktop we'll need to investigate further. It might be a separate sign in issue as the canceling behavior described above sounds different. If you're still able to repro on desktop can you provide a few more details as well as the version of VS Code you're using. thanks!

@jramsay I myself encountered it first through the VSCode software (and it is still doing it), and have been using the browser version because it's the only way I can use VSCode.
If you need more information you can view my issue post aswell.
https://github.com/MicrosoftDocs/live-share/issues/3643

@jramsay mine occurred in the browser as well same problem as what @ryz0o linked to. I am going to add an additional detail to that issue.

I am using the desktop app on MacOS, vs code version 1.46.0, live share extension version 1.0.2354, chrome is default browser.
When I click someone's link to a collaboration session, it takes me to chrome, which asks me if I want to open in vs code, and I confirm. Vs code opens up a new window, says it's loading folder tree, etc, then tells me i need to sign in

I click sign in, which asks me if it's github or microsoft, I choose github, and then it hangs, like so:

The app for some reason won't redirect me to the browser to handle the authentication process, so I'm stuck.
I found ways to get myself signed in apart from this process, once (only once) I was able to use the code method, but usually I just have to sign in to a new window without joining a liveshare session. BUT, once signed in properly, when I attempt to manually join a session by pasting the url, hitting enter causes the entire vs code window to reload. Reloading isn't a problem in itself, but apparently it causes everything about my signed in session to be lost, and I find myself right back at the screenshots above. I have never been able to bypass it.
The ONLY workaround is that I have to be the one hosting the liveshare session, and my coworker has to join. He is using the same vs code version, but liveshare v1.0.2302 and he can join my session fine.
I hope this description was helpful to a swift resolution. I would love to be able to use this feature properly again.

@tehpsalmist: thanks for the detailed description! I was able to repro this issue following your instructions. It looks like the fix for this issue was just made by VS Code:
https://github.com/microsoft/vscode/issues/100333
I checked VS Code (Stable) updates and they released 1.46.1. After installing the latest update I'm no longer hitting the issue. I'm going to resolve this item as fixed but please do let us know if it doesn't work for you after you upgrade. Appreciate your help investigating this!

Awesome! Thanks! I actually can't wait to try it out with a coworker tomorrow morning! :)


reset_all_databases should be called before the setup, not after

in:  https://github.com/guilleiguaran/fakeredis/blob/433a40f1a15f965690fcba16c2570c2b9680b1ac/lib/fakeredis/minitest.rb#L19
The setup may want to populate some initial data inside the database, it would be better to call super only after the reset_all_databases call

@Aethelflaed good point, do you want to send a PR fixing this?


Trouble getting the pdf to display

Objective

User selects a .pdf file from a list
'Preview File' modal loads with pdf

Using the library I was receiving a null context error due to the javascript running before the modal loads. (The Bootstrap devs don't want manipulation to happen from the controller, so there's no event catch for the modal being loaded or ready)
As a lame solution I made it so that the user can press a preview button on the modal. This calls a function injectPdf which adds the rest of the neccessary code to display the PDF. Now it's a simple implementation, but still not working.
Modal Controller
this.pdfUrl = "http://www.jeremyzerr.com/sites/default/files/Getting-Started-with-AngularJS.pdf";
injectPdf() {
 var html = '<ng-pdf canvasid="pdf-canvas" template-url="pdf.html" debug="true"></ng-pdf>';
 document.getElementById("elemId").innerHTML = html;
}

Modal HTML

  <button ng-click="modal.injectPdf()">Preview</button>

pdf.html
<canvas id="pdf-canvas"></canvas>
Html gets injected so no clue what is lacking here. Currently it doesn't display and I'm not getting any console errors. Thanks.

Hi @SensationSama, can you provide a full JSFiddle (ore something else) with your code?
Thanks!

@dennybiasiolli I'm not on the current version of angular but it's approximately like this.
I doubt my script imports are also working in the fiddle.

@SensationSama I doubt that your example is an angular app. Without a real example I can not tell you where the problem is. Sorry

I obviously can't port my entire codebase. If I fix the script references will that suffice, or does something else stand out to you as not an angular app

@SensationSama maybe you have to compile your injection? For the next time remember that deleted comments are sent via email anyway :).

@dennybiasiolli @simobasso I ended up using https://pdfobject.com/
Thanks anyway fellas


Roll engine e886d7b08295..5bbc90852fd2 (1 commits)

https://github.com/flutter/engine/compare/e886d7b08295...5bbc90852fd2
git log e886d7b082953589569169513b0b7d8d78975055..5bbc90852fd2ae3ece5334224b99411a262f1556 --no-merges --oneline
5bbc90852 Roll src/third_party/skia 9577f01ba081..cc82972af115 (1 commits) (flutter/engine#9249)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (bmparr@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6414398118166528

Commit queue failed; closing this roll.


Update schema and FD icon

@andosca Two questions:

Should the Schema be updated?
Should we update to the new FD logo in the FemDesign.Core/Resources/icon so that it is used in the NuGet packages?

Originally posted by @xRadne in https://github.com/strusoft/femdesign-api/issues/319#issuecomment-1114611044

the latest schema FD 21.00.002 strusoft.xsd doesn't seem to have any substantial conflict with FD 21.00.001 strusoft.xsd.
I have compare the two files and the only difference occurs in the following type. Nothing important I believe
  <xs:simpleType name="geometry_text1023"> <!-- From FD21.00.002, for text_type only! -->
    <xs:restriction base="xs:string">
      <xs:pattern value="[

 -�]{0,1023}" />
    </xs:restriction>
  </xs:simpleType>

@andosca @xRadne @johannariad
Which version do you prefer as icon for the NuGet package?
I am pointing towards the minimal version with no text.

Since the logo is displayed quite small, I prefer the minimal logo

I haven't seen that the icon was going to be so tiny 😄
Minimal is definitely the way to go!

The schema is not going to be update as .cs file is missing


TASK: Add CodeQL code vulnerability check

Task Description
Add a code vulnerability check to our PRs using [CodeQL](https://codeql.github.com/]. There's prior art in the heureka repo.
Sub-tasks

[ ] Add check that can be run locally
[x] Add a workflow so the check runs automatically for every PR

In the GitHub repository under the security settings, there's an option to enable a built-in CodeQL check. We recently activated it, and it now runs code checks on every PR as expected.
However, we haven't yet found a way to run these checks locally.

I think this is sufficient for now. We can start by running the checks exclusively on GitHub, and if a local solution becomes necessary later, we can explore it then.


Add links to headers with ids

Resolves #395
It is probably only necessary to target h2, h3, h4, but targeting everything isn't that much more code and future proofs the change.

Coverage increased (+0.3%) to 94.795% when pulling 71ba8e66b1941bb9df7cc2b25e2090ec4c80c7f4 on Gerrit0:header-links into 1c065596f5de0b29410f037b3617f8434c780b62 on dsherret:dev.

Works perfectly! Thanks so much @Gerrit0!


[BEAM-4176] enumerate primitive transforms in portable construction

Fix many Java portable VR tests by improving the check for primitive transforms, per Slack discussion with @lukecwik
Several unittests are still failing; I've marked them with @Ignore to flag for discussion here. I fixed several tests by making sure every transform has a spec with a URN set, but I'm not clear what URN to use in some of the remaining cases (and it's possible some that I've added are misleading, e.g. for side-inputs).
Here are the results of running the Java Flink portable VR tests:
./gradlew -p runners/flink/job-server validatesPortableRunner

Before:

After:

R: @angoenka
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

The 3 cases in ReferenceRunnerTest are failing, two are
No evaluator for PTransform "beam:transform:group_by_key:v1"

and one is
PCollectionNodes … were consumed but never produced

I opened up https://github.com/ryan-williams/beam/pull/1 against your branch with suggestions on how to fix up tests. Effectively I changed everything to use a primitive transform URN.
@tweise I'm not sure how you implemented native sources/sinks in your Flink integration but this change will mean that you will need to use a primitive urn like read/pardo without an environment as the way to signal to Flink that it should intercept this transform and replace it with its own implementation.

Ryan, as a follow-up to this change it would be good to make unknown transforms that aren't composites an error. This would be any transform that returns a different PCollection then one it consumed if the URN is not a primitive.

@lukecwik here is the example: https://github.com/lyft/beam/blob/release-2.8.0-lyft/sdks/python/custom-source-example.py
and the translator extension: https://github.com/lyft/beam/blob/release-2.8.0-lyft/runners/flink/src/main/java/org/apache/beam/runners/flink/LyftFlinkStreamingPortableTranslations.java#L52

Can you elaborate what the need for this change is? (It's not obvious why the use of custom URNs should be restricted.)

The shared libraries need to understand all the primitives given to them as they are required to satisfy the contracts that each primitive has. The code within QueryablePipeline/ExecutableStage/GreedyPipelineFuser/... make assumptions around what neighboring primitives exist and how they are compatible with each other.
The runner can choose to intercept the pipeline before it gives it to the shared libraries to do fusion/... and update the representation to be compatible with those libraries or those libraries need to expose hooks for runner known primitives to describe how they should treat these primitives (e.g. should they break fusion, or be fusable with the SDK transforms).

@lukecwik this is probably something we should take to the mailing list for broader visibility. Currently the Flink runner does the pipeline surgery before translation. With the proposed change the customization referenced above would need to spread over multiple components (fuser/prep and translator) while it's encapsulated in a single place now.
In the example the Flink sources really are primitives that should remain handled on the runner side as such (effectively like ParDo without inputs). Would it perhaps also make sense to assume that as a default when our common fusion code does not recognize a URN? If I was to introduce a custom URN that requires fusion break, then yes, it would be necessary to provide that information to the fuser. That seems cleaner than the runner temporarily switching URNs to appease the fuser.

Will bring to ML

Looking into some remaining failures in ReferenceRunnerTest:
pipelineExecution / GBK:
java.lang.NullPointerException: No evaluator for PTransform "beam:transform:group_by_key:v1"
	at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:864)
	at org.apache.beam.runners.direct.portable.TransformEvaluatorRegistry.forApplication(TransformEvaluatorRegistry.java:96)
	at org.apache.beam.runners.direct.portable.DirectTransformExecutor.run(DirectTransformExecutor.java:93)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)

testSDF:
java.lang.IllegalArgumentException: PCollectionNodes [PCollectionNode{id=ParDo(PairStringWithIndexToLength)/ParMultiDo(PairStringWithIndexToLength).output, PCollection=unique_name: "ParDo(PairStringWithIndexToLength)/ParMultiDo(PairStringWithIndexToLength).output"
coder_id: "KvCoder7"
is_bounded: UNBOUNDED
windowing_strategy_id: "WindowingStrategy(GlobalWindows)"
}] were consumed but never produced
	at org.apache.beam.repackaged.beam_runners_core_construction_java.com.google.common.base.Preconditions.checkArgument(Preconditions.java:383)
	at org.apache.beam.runners.core.construction.graph.QueryablePipeline.buildNetwork(QueryablePipeline.java:183)
	at org.apache.beam.runners.core.construction.graph.QueryablePipeline.<init>(QueryablePipeline.java:106)
	at org.apache.beam.runners.core.construction.graph.QueryablePipeline.forPrimitivesIn(QueryablePipeline.java:69)
	at org.apache.beam.runners.core.construction.graph.GreedyPipelineFuser.<init>(GreedyPipelineFuser.java:67)
	at org.apache.beam.runners.core.construction.graph.GreedyPipelineFuser.fuse(GreedyPipelineFuser.java:89)
	at org.apache.beam.runners.direct.portable.ReferenceRunner.executable(ReferenceRunner.java:128)
	at org.apache.beam.runners.direct.portable.ReferenceRunner.<init>(ReferenceRunner.java:102)
	at org.apache.beam.runners.direct.portable.ReferenceRunner.forInProcessPipeline(ReferenceRunner.java:114)
	at org.apache.beam.runners.direct.portable.ReferenceRunnerTest.testSDF(ReferenceRunnerTest.java:207)

Based upon community feedback, I opened up https://github.com/ryan-williams/beam/pull/2 against your branch. Please merge or advise on different course of action.

I just ran VR tests again on this PR and am seeing the original failure count; I suspect I made a mistake running the tests, but will update here as I learn more.

I didn't have docker daemon running :) a re-run hung in FlattenTest though, but I'm assuming that's not related to this change…

Seeing significant perf degradations running the Java PVR tests now, presumably due to unrelated changes that have gone in since I opened this PR.
I can't get through a run of all the tests without a hang; in some (all?) cases it seems that Docker is running out of space in its virtual filesystem.
Previously all 200+ cases ran in a bit over an hour, but now it grinds to a halt about 50 cases in (always on a different test case, n=3) in about an hour.

My working theory is:

something has caused us to use dramatically more of Docker's virtual space
PVR tests still pass or fail in isolation as one would expect (and presumably we're still down to 127 failures, as we were at the start of this PR)
however, once you've run a certain total number of tests, your local Docker space will be full and everything will fail from that point on

Debugging this doesn't have to hold up this PR, but I don't think there's a huge rush to get it in either, so I can spend some more time on it.

We stopped doing lazy cleanup of docker containers which might have a minor impact of the number of containers created and hence their memory footprint.
With 200+ test cases we are expecting atleast 200 containers anyways.
We are introducing process based sdkhanress which should remove creation of docker containers whole together.
For now please try cleaning the dockers containers and see if it helps.

ah, yea docker container ls -a seems to show old containers for every test-case, so that sounds like a good theory about the problem
is there something better to do than manually clean those a few times while PVR tests are running?
do you have a link handy to where we disabled the cleanup?

its in this commit https://github.com/apache/beam/commit/992ece0a4ad444dee9faea21e4729c495dea6b42
I don't think this will make a lot of difference though as we anyways create 200+ containers, 1 for each test atlease.
The best way to go about it is to use process based SDK Harness which is in development at the moment https://github.com/apache/beam/pull/6373

It seems like we should be able to reuse a single docker container image for all the test runs instead of having each test create one.

OK, I managed to get a run to complete:

Notes on getting it to run:

I increased my local Docker image size to 256GB (from 64GB)
I also ran docker container prune -f multiple times during the run
idk if either of the above are necessary or sufficient
docker system df is also useful to determine how full the image is.

Notes on the results:

10 more cases are passing since I started this PR

possibly due to fixes I added for unittests over the course of this PR
or other OOB changes

runtime is much worse! 2h30m. seems like nothing to be done there atm, and is related to an increased number of containers being created (thanks for explaining @angoenka)

Thanks Ryan.

Thanks Ryan!
Also the good part is that the test cases are not getting stuck.
Many a times the test use to get stuck instead of failing.

Yea, to be clear, the tests themselves (apparently) weren't getting stuck, but attempting to run them was, due to local Docker issues, which I agree is preferable 😆

For this category of changes, we should in the future run the post commit PVR tests in Jenkins before merging.

I think Thomas is alluding to https://issues.apache.org/jira/browse/BEAM-5441.

@ryan-williams how to run the PVR tests: Run Python Flink PortableValidatesRunner
see https://github.com/apache/beam/pull/6340


fe-assignment-2

screenshot:

Metric
Value

alert_status
:white_check_mark: OK

bugs
7

code_smells
0

reliability_rating
C (at least 1 Major Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


fix db bottlenecks

LoadSpec

SpikeSpec

StressSpec

Fix or find reason why it's not fixable

LoadSpec - Old vs New (json)

SpikeSpec - Old vs New (json)

StressSpec - Old vs New (json)

Little to no improvement, therefore the Slick Implementations seem to be limited by the Library/MySQL


SVG-related changes Firefox 69

See https://trello.com/c/g2gJFbtM/205-svg-related-changes-fx-69 for original trello card
Acceptance criteria

[ ] Implement isPointInFill() and isPointInStroke() methods of SVGGeometryElement (already documented, need examples adding and BCD updating)

Note: rel note added to https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/69#SVG, so it was there in time for the 69 release. Other work still needs doing.

I've added a PR to update the BCD data. Examples will follow.
Sebastian

Both isPointInFill() and isPointInStroke() now also have live samples.
Sebastian

@SebastianZ thanks Sebastian, you are super-awesome!


Daylight Timezone Support

Most countries in North America and Europe observe Daylight Savings Time, where clocks are set forward one hour for half of the year. Currently the website doesn't account for this when displaying times.
As an example, for timezones such as America/New_York, currently the time should be displayed in Eastern Daylight Time (EDT, or UTC-04:00), but it is still displayed in Eastern Standard Time (EST, or UTC-05:00)
Daylight Savings Time is complicated and varies in implementation between countries, so it'd probably be easiest to use something like moment.js and moment-timezone to take care of that stuff. If this isn't feasible or too time consuming, a simple manual toggle in the Settings menu would be enough. (e.g. checking a checkbox would add an hour to all times displayed)

moment.js is deprecated, so we are currently using Luxon for date calculation. This code is responsible for determining the server timezone, but it's not considering DST because it returns UTC offsets instead of actual timezones.

Oh wow, I didn't know Moment wasn't getting updated anymore. In that case I'll leave things to your better judgement.

Oh wow, I didn't know Moment wasn't getting updated anymore. In that case I'll leave things to your better judgement.

Fixed!


Refactor md class

The markdown classes should be refactored. Instead of returning requirements, backmatter and core markdown separately, this should be given as a single return value for core and backmatter and as a classmethod for the requirements.
The core and backmatter markdown should certainly be returned as a single type instance. The reason is that otherwise, additional coordination needs to be done regarding saving of ids or the page_path where the object is supposed to be stored.
For the requirements, the reason is that the requirements only depend on the class, not individual instances.

Closed in https://github.com/hhoeflin/mkreports/commit/fe1bda6fcf0933a8e54b578d11a70b3d01c246f7.
Requirements are still instance methods as otherwise the MdSeq does not work properly or other containers that need to pass along requirement information.


Error: Failed to launch the browser process!

Running 116117bot in docker with default params, it refuses starting.
user@host:~/repo/116117bot$ sudo docker run -it --name 116117bot -e URLS="https://123-iz.impfterminservice.de/impftermine/suche/XXXX-XXXX-XXXX/12345" 116117bot
><EMAIL_ADDRESS>start /app
> ts-node index.tsBooting 116117bot(node:20) UnhandledPromiseRejectionWarning: Error: Failed to launch the browser process!
Fontconfig warning: "/etc/fonts/fonts.conf", line 100: unknown element "blank"
[31:31:0227/173849.591166:ERROR:browser_main_loop.cc(1439)] Unable to open X display.
[0227/173849.599140:ERROR:nacl_helper_linux.cc(307)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

    at onClose (/app/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:193:20)
    at Interface.<anonymous> (/app/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:183:68)
    at Interface.emit (events.js:327:22)
    at Interface.EventEmitter.emit (domain.js:467:12)
    at Interface.close (readline.js:424:8)
    at Socket.onend (readline.js:202:10)
    at Socket.emit (events.js:327:22)
    at Socket.EventEmitter.emit (domain.js:467:12)
    at endReadableNT (internal/streams/readable.js:1327:12)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:20) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:20) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
^C

Default should be headless according to docs, correct?
When I set -e HEADLESS="false" the service seems to start (but runs into another issue with Puppeteer, I'm going to open a separate issue for this.

Yeah, the headless parameter was unnecessarily confusing. I changed it to be not headless by default, and having the HEADLESS env variable set to anything now makes it headless. This is the default in the Dockerfile as well, so the docker image should now be headless by default. The related issue in #9 has been fixed as well.
Kindly give it a try, thanks.

Thanks, works as expected now


Feat: Managed models

Initial implementation of Managed models. This defines the general semantics and contains an implementation for Snowflake, based on Dynamic Tables.
As a side effect, this PR also fixes issue 1049.
Tasks:

[x] Decide on name
[x] Pydantic v1 compatibility
[ ] Integration test for Snowflake

Follow-up PR's:

Support for other engines
Cleanup / compaction support, essentially the ability for something like sqlmesh clean to drop any objects that aren't referenced in a specific environment. The assumption up until now is that storage is cheap so keeping old snapshots around for a while isn't a problem. However, costs could add up quickly with managed models because users would be paying for models to be refreshed that aren't actually being referenced

not sure i like engine managed.
what about dynamic, or live, or auto

not sure i like engine managed.
what about dynamic, or live, or auto

Dynamic = Snowflake terminology
Live = Databricks terminology
I'm not against those names per se, but I did think we were trying to remain vendor neutral.
Auto = could work? "Automatic Models"?
The other way to do this could be to drop any pretense of a generic implementation have a Kind per Engine. That way, we can name them accordingly and also "strongly type" the physical_properties and make them part of the Kind instead of arbitrary key-value pairs, which would allow easier validation via pydantic.
For example:
MODEL (
   name foo,
   kind SNOWFLAKE_DYNAMIC_TABLE (
     warehouse "small",
     target_lag '20 minutes'
   ),
  ...

The downside is that engine-specific Kind's sort of defeat the purpose

maybe auto then

FYI in case it's somehow relevant to this PR: https://github.com/tobymao/sqlglot/commit/60fa5e3f8a6eab3abb12064366a6bde907d9e9de.

FYI in case it's somehow relevant to this PR: tobymao/sqlglot@60fa5e3.

Ahh, thanks, good to know that exists now.
I think it's more relevant in a future PR. Using it now would involve modifying a bunch of method signatures (for example, drop_table would need to be modified to take table properties and the current mechanism of specifying table properties elsewhere assumes key-value pairs instead of individual properties for anything that isnt "top level")


tar_unknown_generation_error

When running this command MIX_ENV=prod mix release --verbosity=verbose
I got this error:
===> Provider successfully run: resolve_release
===> Running provider overlay
===> Provider successfully run: overlay
===> Running provider release
===> Including Erts from /usr/local/Cellar/erlang/17.5/lib/erlang
===> release successfully created!
===> Provider successfully run: release
===> Running provider tar
{{case_clause,
     {'EXIT',
         {{badmatch,514},
          [{erl_tar,create_header,3,[{file,"erl_tar.erl"},{line,414}]},
           {erl_tar,add1,4,[{file,"erl_tar.erl"},{line,322}]},
           {erl_tar,foreach_while_ok,2,[{file,"erl_tar.erl"},{line,999}]},
           {systools_make,add_to_tar,3,
               [{file,"systools_make.erl"},{line,1878}]},
           {systools_make,add_appl,7,[{file,"systools_make.erl"},{line,1749}]},
           {systools_make,'-add_applications/5-fun-0-',6,
               [{file,"systools_make.erl"},{line,1568}]},
           {lists,foldl,3,[{file,"lists.erl"},{line,1261}]},
           {systools_make,add_applications,5,
               [{file,"systools_make.erl"},{line,1567}]}]}}},
 [{systools_make,'-add_applications/5-fun-0-',6,
      [{file,"systools_make.erl"},{line,1568}]},
  {lists,foldl,3,[{file,"lists.erl"},{line,1261}]},
  {systools_make,add_applications,5,[{file,"systools_make.erl"},{line,1567}]},
  {systools_make,mk_tar,6,[{file,"systools_make.erl"},{line,1561}]},
  {systools_make,mk_tar,5,[{file,"systools_make.erl"},{line,1537}]},
  {systools_make,make_tar,2,[{file,"systools_make.erl"},{line,335}]},
  {rlx_prv_archive,make_tar,3,[{file,"src/rlx_prv_archive.erl"},{line,82}]},
  {relx,run_provider,2,[{file,"src/relx.erl"},{line,286}]}]}
===> Provider (tar) failed with: {error,
                                         {rlx_prv_archive,
                                          {tar_unknown_generation_error,
                                           "polyglot_web","0.0.1"}}}
==> Failed to build release. Please fix any errors and try again.

I am using elixir 1.0.4 and exrm 0.18.1, and I ran the command inside a phoenix project which is under an umbrella project.

Strange, doing a clean checkout solves the issue

That's odd, if you see it again, let me know!


fix: no onSelect event when same item selected

@lwhiteley changes here are for this ticket https://github.com/iambumblehead/react-dropdown-now/issues/96

@lwhiteley the more I think about it... the current behaviour before the changes here seems correct. Maybe we should close the issue and update the test to verify that onSelect is called even when the value is the same.
What do you think?

Think the issue is that the events should not be triggered when there is no user interaction

@lwhiteley I had the same impression at first and I even added a test that rendered two dropdowns referencing the same value --changing one dropdown did not change the value or trigger events at the other dropdown. I could re-add that test.

@lwhiteley upon second thought, the two-dropdowns tests used a primitive value "one" and maybe using an object value instead triggers the onChange event.

I'll add the two-dropdowns test in a few hours

@lwhiteley please review

@lwhiteley I was able to reproduce the issue. Changing the value of the second dropdown triggered onChange at the first dropdown. To resolve it, the dropdown reads the event target to detect if the event originates at the dropdown.

There could issues if the dropdown later needed to support keyboard selection and input, but if you're OK with the current change I will be too.

do you use Matrix @lwhiteley? https://matrix.org/clients/

do you use Matrix @lwhiteley? https://matrix.org/clients/

never heard of it. what would be the purpose of it?

never heard of it. what would be the purpose of it?

actually seems i have used it without knowing. i used gitter.im before

@lwhiteley this page has a little blurb describing matrix https://tatsumoto-ren.github.io/blog/join-our-community.html

Matrix is great because it's completely libre and federated, and you don't need an email address, or a phone number to make an account. Just type your name and a password, and you're in! Before you join, install a Matrix client and register an account on any server that is not matrix.org.

I know about it because there are clients for ubuntu touch and gnome3, which I use on my phone and desktop. Also, I like messaging applications and have been building a service that follows the matrix spec in many parts. I would like to be your friend on Matrix or another messaging system.

can chat privately on https://gitter.im/
and discuss further

I signed in there with my github proifile and if I type "lwhi" or "whit" it doesn't auto-complete "lwhitely" there.


Suppress deprecation warning in controller specs

Such as the following:
get :show, params: { id: 1 }, session: { user_id: 1 }
process :update, method: :post, params: { id: 1 }
 (called from block (4 levels) in <top (required)> at
 /Users/tricknotes/src/github.com/24pullrequests/24pullrequests/spec/controllers/projects_controller_spec.rb:164)

👌


Hooks update

Update and consolidations of hook docs. We should tag this release.

common.md is 621 non-blank lines now. It will grow with a new big-populate hook, and with other enhancements accumulating in the issues.
Should we break some of the larger sections into their own files?
This should not stop this PR being merged though. I'm looking forward to proof-reading the results.

@eddyystop I was thinking that too. I think let's address chopping things up when we overhaul the docs. Likely coming up soon once auth and the generator are done.

We have to bring the built-in hooks page back.

@daffl all the hooks are in common.md now or did I miss some?


Agent应用对话时，一直报：Failed to allocate model service,No model service available!!

Search before asking

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

Operating system information
Windows
Python version information
3.10
DB-GPT version
latest release
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
CPU
Models information
LLM_MODEL=chatgpt_proxyllm
EMBEDDING_MODEL=text2vec
What happened

如图，在Agent对话中，一直报：Error:The answer is not output in the required format.
What you expected to happen
i don't know
How to reproduce
xx
Additional context
xx
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

could you show the dbgpt server error log?

could you show the dbgpt server error log?
日志如下：


Synchronized genlib2 with changes in the generated files.

This is a preliminary to #52, as the generated code and genlib2 have diverged a bit.
Also, added some documentation for the development dependencies (in particular, added the option to change the python interpreter, for platforms where python3 is preferred).

Coverage increased (+0.01%) to 72.819% when pulling c97fe671d16fcbaa85514bf47f85e73679c79ed2 on bezineb5:sync-genlib2 into ea8ef1f6e3b11e8476fb8ac81ae37fe6f749fc18 on gorgonia:master.

Coverage increased (+0.01%) to 72.819% when pulling c97fe671d16fcbaa85514bf47f85e73679c79ed2 on bezineb5:sync-genlib2 into ea8ef1f6e3b11e8476fb8ac81ae37fe6f749fc18 on gorgonia:master.


fix(discovery): add a test showing that Configs' custom unmarshalling/marshalling is broken.

This went under the radar because the utils are never called directly.
We usually marshall/unmarshal Configs as embeded in a struct using UnmarshalYAMLWithInlineConfigs/MarshalYAMLWithInlineConfigs which bypasses Configs' custom UnmarshalYAML/MarshalYAML

Came over this during https://github.com/prometheus/prometheus/pull/14987/files (need to marshall Configs precisely to build a hash)

The test panics https://github.com/prometheus/prometheus/actions/runs/11069911833/job/30758443827?pr=14993#step:6:221 as getConfigType isn't adapted for Configs

Since when is this broken?

The utils were added as part of https://github.com/prometheus/prometheus/commit/4e6a94a27d64f529932ef5f552d9d776d672ec22, but since they're not used in Prometheus, it's hard to say.


small typo in section 1, paragraph 2 ('property -> properly')

In the sentence 'The image for perplexity 100, with merged clusters, illustrates a pitfall: for the algorithm to operate property, the perplexity really should be smaller than the number of points', I believe the authors intended to use the word 'properly'.

Thanks for pointing this out. Fixed by 172f676. Changes will propagate to the live site in a bit.


Updated code snippets to v2.2.0

This is a prep branch for holding code that complies with v2.2.0. All standalone scripts should be updated with this PR, though a handful of "full framework" examples can be updated when v2.2.0 is properly released.

I wonder also, if it's worth, for consistency, discussing whether to go either have use statements or not?

For snippets that show up on NDP, we've not used use statements and instead preferred the full class name for clarity. This makes it easier to copy/paste the code. The user could always clean that up after-the-fact. It leads to uglier, but more quickly-usable code.
Historically there wasn't a way to easily show things like use or import (for other languages).  Now that we can specify use statements in NDP, we should start to look at using them and see if that impacts usage.


Wrong docker client version is discovered

What version of dep are you using (dep version)?
v0.3.1-151-g1d73a4a2
What dep command did you run?
dep ensure
What did you expect to see?
  name = "github.com/docker/docker"
  ...
  version = "v17.05.0-ce"

What did you see instead?
  name = "github.com/docker/docker"
  ...
  version = "v1.13.1"

Hey, I'm using github.com/fsouza/go-dockerclient/ so I have
github.com/docker/docker as the transitive dependency.
I suspect that dep takes v1.13.1 as latest version because of ce in the v17.05.0-ce version.
I tried to constrain the version or branch by adding
required = ["github.com/docker/docker"]
and
[[constraint]]
  branch = "master"
  name = "github.com/docker/docker"

But it gave me constraint errors during ensuring process.
I suspect it is because github.com/fsouza/go-dockerclient uses like 20 subpackages from docker, but I'm constraining the whole repo.
Do I need to write a constraint for each subpackage or how to overcome this issue?

@Trane9991 hi, since docker/docker is a transitive dependency, you have to use override. We have a FAQ entry for the same how-do-i-constrain-a-transitive-dependencys-version.
Also, I gave it a try to ensure it works.
Add
[[override]]
  name = "github.com/docker/docker"
  version = "v17.05.0-ce"

to your Gopkg.toml and run dep ensure. You would see v17.05.x added to Gopkg.lock.
branch = "master" works as well. Refers the latest commit revision.
Hope this helps 🙂

I love how Docker's silly versioning choices are haunting me across multiple repos. I had the same problem with the Docker Version Manager too! 🤦♀

I am inclined to consider this a bug in dep. Just because a value looks like semver doesn't mean that dep should assume it is, ignore that the version is in fact a tag on the repo, and then completely fail to solve.
Gopkg.toml
[[constraint]]
  name = "github.com/moby/moby"
  version = "v17.05.0-ce"

$ dep ensure -v
Root project is "foo"
 1 transitively valid internal packages
 1 external packages imported from 1 projects
(0)   ✓ select (root)
(1)	? attempt github.com/moby/moby with 1 pkgs; at least 1 versions to try
(1)	    try<EMAIL_ADDRESS>(2)	✗<EMAIL_ADDRESS>not allowed by constraint ^17.5.0-ce:
(2)	    ^17.5.0-ce from (root)
(1)	    try<EMAIL_ADDRESS>(2)	✗<EMAIL_ADDRESS>not allowed by constraint ^17.5.0-ce:
(2)	    ^17.5.0-ce from (root)
(1)	    try<EMAIL_ADDRESS>(2)	✗<EMAIL_ADDRESS>not allowed by constraint ^17.5.0-ce:
(2)	    ^17.5.0-ce from (root)
(1)	    try<EMAIL_ADDRESS>(2)	✗<EMAIL_ADDRESS>not allowed by constraint ^17.5.0-ce:
(2)	    ^17.5.0-ce from (root)
... TRUNCATED
(1)	  ← no more versions of github.com/moby/moby to try; begin backtrack
  ✗ solving failed

Currently, when we read in the manifest we check if the value is valid semver, and then convert it to use an implicit caret immediately. So by the time the solver is running, there's no way to know that we've munged the version and fallback to a tag.
@sdboyer I am going to try out what it would look like if when semver.NewConstraintIC is called the resulting Constraint remembered that it is using an implied operator (it didn't have = or ^ explicitly in the original string). Then the solver could try a value first as a constraint, then if that totally fails and the operator was implied, try it as a plain version before backtracking.

Nevermind! I was wrong about why it wasn't picking v17.05.0-ce. Seems like the real problem is this:
v17.05.0-ce: Could not introduce<EMAIL_ADDRESS>as its subpackage github.com/docker/docker does not contain usable Go code (*build.NoGoError).. (Package is required by (root).)

Sorry about the issue spam. I went off down the wrong track and nothing I said was right! 😊

go-dockerclient now supports dep.


Opni Chart does not deploy workloads

Summary:
I am attempting to install an Opni demo via the Rancher UI by following the instructions provided here: https://opni.io/installation/opni
It would appear when following these instructions with the values provided below, results in nothing more than CRDs being deployed. No workload of any type is deployed.
Values Provided:

Hostname: "opni.test.lan"
Service Type: "ClusterIP"
Auth Provider: "noauth"

Logs:
helm upgrade --install=true --namespace=opni --timeout=10m0s --values=/home/shell/helm/values-opni-crd-0.12.1.yaml --version=0.12.1 --wait=true opni-crd /home/shell/helm/opni-crd-0.12.1.tgz
checking 26 resources for changes
Looks like there are no changes for CustomResourceDefinition "alertingclusters.core.opni.io"
Looks like there are no changes for CustomResourceDefinition "bootstraptokens.core.opni.io"
Looks like there are no changes for CustomResourceDefinition "datapreppers.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "gateways.core.opni.io"
Patch CustomResourceDefinition "grafanadashboards.grafana.opni.io" in namespace
Patch CustomResourceDefinition "grafanadatasources.grafana.opni.io" in namespace
Patch CustomResourceDefinition "grafananotificationchannels.grafana.opni.io" in namespace
Patch CustomResourceDefinition "grafanas.grafana.opni.io" in namespace
Looks like there are no changes for CustomResourceDefinition "keyrings.core.opni.io"
Patch CustomResourceDefinition "logadapters.logging.opni.io" in namespace
Looks like there are no changes for CustomResourceDefinition "loggingclusterbindings.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "loggingclusters.core.opni.io"
Looks like there are no changes for CustomResourceDefinition "monitoringclusters.core.opni.io"
Looks like there are no changes for CustomResourceDefinition "multiclusterrolebindings.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "multiclusterusers.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "natsclusters.core.opni.io"
Patch CustomResourceDefinition "opensearchclusters.opensearch.opster.io" in namespace
Looks like there are no changes for CustomResourceDefinition "opensearchrepositories.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "opniclusters.ai.opni.io"
Looks like there are no changes for CustomResourceDefinition "opniopensearches.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "preprocessors.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "pretrainedmodels.ai.opni.io"
Looks like there are no changes for CustomResourceDefinition "recurringsnapshots.logging.opni.io"
Looks like there are no changes for CustomResourceDefinition "rolebindings.monitoring.opni.io"
Looks like there are no changes for CustomResourceDefinition "roles.monitoring.opni.io"
Looks like there are no changes for CustomResourceDefinition "snapshots.logging.opni.io"
beginning wait for 26 resources with timeout of 10m0s
Release "opni-crd" has been upgraded. Happy Helming!
2023-11-21T10:53:38.152239937-06:00 NAME: opni-crd
2023-11-21T10:53:38.152245721-06:00 LAST DEPLOYED: Tue Nov 21 16:53:35 2023
2023-11-21T10:53:38.152258457-06:00 NAMESPACE: opni
2023-11-21T10:53:38.152261634-06:00 STATUS: deployed
2023-11-21T10:53:38.152265151-06:00 REVISION: 2
2023-11-21T10:53:38.152267890-06:00 TEST SUITE: None

2023-11-21T10:53:38.155118785-06:00 ---------------------------------------------------------------------
SUCCESS: helm upgrade --install=true --namespace=opni --timeout=10m0s --values=/home/shell/helm/values-opni-crd-0.12.1.yaml --version=0.12.1 --wait=true opni-crd /home/shell/helm/opni-crd-0.12.1.tgz
2023-11-21T10:53:38.164128701-06:00 ---------------------------------------------------------------------
2023-11-21T10:53:38.164132789-06:00 helm upgrade --install=true --namespace=opni --timeout=10m0s --values=/home/shell/helm/values-opni-0.12.1.yaml --version=0.12.1 --wait=true /home/shell/helm/opni-0.12.1.tgz
Error: "helm upgrade" requires 2 arguments
2023-11-21T10:53:38.192701309-06:00
2023-11-21T10:53:38.192705592-06:00 Usage:  helm upgrade [RELEASE] [CHART] [flags]

You'll need to manually enter a release name for the chart; it's possible that our chart is missing some configuration that rancher needs in order to auto-generate a default release name.

Looks like you are correct, I appreciate the response.

@Daemonslayer2048 What would you think about repurposing this ticket to request that a default release name be provided. If you encountered this issue, I'm sure others will as well.

I think there are two options really:

Create a default release name
Update the docs

I can repurpose the ticket no problem, just thought I would toss out the 2nd option before doing so.

Either is valid with option one being the preference for most people probably.

We can reuse this issue to track the chart fix. :+1:


The "clientId" will be removed when the mobile client reconnect

The network is unstable when the client runned at mobile phone,  the mqtt client will be reconnect repeatedly, but the old connection will be colsed lead to the "ProtocolProcessor" removes the cliendId.
As a result, the new message cannot arrive to the new connection.

Could you specify better, because when the client disconnects or get
disconnected by the broker because ping req doesn't happen in certain time,
the broker drop client data if not specified with cleanSession=false.  What
do you intend that the message cannot arrive?
Please give more details when you open a a bug report.
Andrea
On Wed, Jul 22, 2015 at 7:04 AM, wangyonghe<EMAIL_ADDRESS>wrote:

The network is unstable when the client runned at mobile phone, the mqtt
client will be reconnect repeatedly, but the old connection will be colsed
lead to the "ProtocolProcessor" removes the cliendId.
As a result, the new message cannot arrive to the new connection.
—
Reply to this email directly or view it on GitHub
https://github.com/andsel/moquette/issues/71.

No news and no details, so I close the issue.

Seems related to #4


Add threads to run importers in parallel

Signed-off-by: Tushar Goel<EMAIL_ADDRESS>

With threads:
Importing data using vulnerabilities.importers.nginx.NginxImporter
Importing data using vulnerabilities.importers.gentoo.GentooImporter
Importing data using vulnerabilities.importers.istio.IstioImporter
Successfully imported data using vulnerabilities.importers.nginx.NginxImporter
Successfully imported data using vulnerabilities.importers.gentoo.GentooImporter
Successfully imported data using vulnerabilities.importers.istio.IstioImporter
--- 54.45800304412842 seconds ---

Without threads:
Importing data using vulnerabilities.importers.nginx.NginxImporter
Successfully imported data using vulnerabilities.importers.nginx.NginxImporter
Importing data using vulnerabilities.importers.gentoo.GentooImporter
Successfully imported data using vulnerabilities.importers.gentoo.GentooImporter
Importing data using vulnerabilities.importers.istio.IstioImporter
Successfully imported data using vulnerabilities.importers.istio.IstioImporter
--- 94.81378507614136 seconds ---

can you also try an alternative with a process pool?
One easier way may be to use https://docs.python.org/3/library/concurrent.futures.html as the ProcessPoolExecutor and ThreadPoolExecutor have the API and this abstract from the lower level processes and threads details


ci: use latest published version of turbo in CI instead of latest canary

The failing examples tests here are fixed in #5240


Support for changing audio output devices

Operating System

[X] Windows
[X] macOS
[X] Linux
[X] FreeBSD
[X] OpenBSD
[X] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
Add functions to detect audio output devices on the machine, and Context can be created with a specific device.
Why is this needed?
No response

This would be a task for github.com/ebitengine/oto


Upgrade to go 1.17 to fix ineffassign issues with new stdlib functions

time.Time.UnixMilli was added in go1.17

Looks like ineffassign has other hiccups too: for github.com/thediveo/netdb it reports missing identifiers that clearly exist.

I want to upgrade it!!!!! plz!!

@dechristopher @TheDiveO @jimschubert I believe this has been fixed and I don't see ineffassign issues on the repos you posted above. Feel free to reopen if that's not the case.

looks good, thank you very much!