[HUDI-812] Migrate hudi common tests to JUnit 5

Migrate the test cases in hudi-common to JUnit 5.
Follows #1589
Migration status (after merging)

Package
JUnit 5 lib
API migration
Restructure packages

hudi-cli
✅
✅
-

hudi-client
✅
✅
-

hudi-common
✅
✅
🚧

hudi-hadoop-mr
✅
✅
-

hudi-hive-sync
✅
✅
-

hudi-integ-test
✅
✅
N.A.

hudi-spark
✅
✅
-

hudi-timeline-service
✅
✅
-

hudi-utilities
✅
✅
-

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@yanghua This marks completion of API migration. Once #1589 merged, I'll rebase this on master and update checkstyle to ban junit 4 import.

Codecov Report

Merging #1590 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1590      +/-   ##
============================================
+ Coverage     71.82%   71.83%   +0.01%
  Complexity      294      294
============================================
  Files           385      385
  Lines         16549    16549
  Branches       1661     1661
============================================
+ Hits          11886    11888       +2
+ Misses         3931     3929       -2
  Partials        732      732

Impacted Files
Coverage Δ
Complexity Δ

...che/hudi/common/util/BufferedRandomAccessFile.java
54.38% <0.00%> (-0.88%)
0.00% <0.00%> (ø%)

...ache/hudi/common/fs/inline/InMemoryFileSystem.java
89.65% <0.00%> (+10.34%)
0.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


Preliminaries/Probability

Job d2l-zh/PR-565/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-565/

Job d2l-zh/PR-565/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-565/

Job d2l-zh/PR-565/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-565/

Job d2l-zh/PR-565/4 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-565/


bugfix : Fix a place where invalid input terminates the server

I think in async func, the whole body of the function should be wrapped in a try-catch block.

Can you attach the prompt.yaml, prompt.md and change.sh files? They may make conflicts, but no worry I will resolve it. (We have #9 to work on the issue)
When using ChatGPT Pro on the web, I also share a link to the conversation and paste it to the commit message.

It seems that the promisify call on make makeExecutableAsync causes trouble. It is already an async function, we just need to await it.

@DarshanDixit05 I have fixed the issues so now it works in its simplest form.
Thank you for your contribution and feel free to continue to working on improving this or on other issues!
Please use Junior for the work, do not edit the files directly if possible. Check my video in the README for seeing how I am using it. More and better videos are coming in the following days.
Thank you again for the contribution, I have learned a lot from your experience and now an issue is fixed!


[Feat]: I want to add my account

What feature?
I have an idea
Add screenshots
Add screenshots
Code of Conduct

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

Now you can add your profile on 🌟 DevDisplay
& be sure to star the github repo 💌 This is a mandatory step for all contributors. & It's is the first priority of contribution in DevDisplay 💌
Thanks 👍🏻


feature/373 sun 10

Pull Request
Description

filter sun elevation data if >10
taken a simple approach where we just look at the elevation at London at that time

Fixes #416
How Has This Been Tested?

usual unittests

print statements show lat and lons are now in the UK

[ ] No

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

sun data is calculate on 2019, but then

LGTM! Thanks!
Just to check: Does self.elevation.index span from 2020-01-01 to 2021-08-31?

Oh thats a very good point. Sun data is actually from 2019 but then assumed it correct for all years. So ill need to add something for that.
ref

Sun data is actually from 2019 but then assumed it correct for all years. So ill need to add something for that.

I've been wondering for a while if we should have a config option for "start_date" and "end_date" for the entire dataset.  e.g. we'd set start_date=2020-01-01 and end_date=2021-08-31.  That might speed up processing (e.g. tell PVDataSource to only load data from start_date to end_date).  And would enable SunDataSource to "copy" its one year of Sun data to fill start_date to end_date.  Does that sound like an OK solution?

Sun data is actually from 2019 but then assumed it correct for all years. So ill need to add something for that.

I've been wondering for a while if we should have a config option for "start_date" and "end_date" for the entire dataset. e.g. we'd set start_date=2020-01-01 and end_date=2021-08-31. That might speed up processing (e.g. tell PVDataSource to only load data from start_date to end_date). And would enable SunDataSource to "copy" its one year of Sun data to fill start_date to end_date. Does that sound like an OK solution?

I think that would be a good thing to do. I guess most of the time the zarr data will about match all the other data. But yea I think that isnt too much work and would be fairly easily to do?

Sounds good.  I've started a new issue about implementing start_date and end_date in the config yaml: #425


Added my profile

Fixes or introduces:
Proposed Changes
-Added the profile details.

@swatisingh45 - Can you resubmit without the corrections to other listings please? 🙌

Hey! yeah, sure. Shall I ?


[Leaderboards] Reward Distribution

Upon hitting 0 earlier this morning the leaderboards only dispersed cosmetic keys to online players. Players who were offline and had top 3 placement received no rewards even upon logging in. Secondly, I believe the rewards are intended to be more than just cosmetic keys however unsure if players are meant to receive all listed rewards or only a select few from those in /lbrewards

Get a top 3 placement on the /warp leaderboards
wait for time to expire
See rewards
Repeat step 1 & 2 but be offline during expiration of leaderboard
Log in
See lack of any rewards
See error

For offline players to receive the rewards while offline or upon next join.
Screenshots of the leaderboards that expired:

Deathnight7976
Orion
I believe it was intended to extend all of these leaderboards however these 3 expired before the extension took place. As you can see in my inventory is the rewards from the PlayerExp Leaderboard which i was online for. However I left prior to the CatchShinies Leaderboard and received no rewards for it.
Current status of the 3 boards:

You may notice an inflated number in the current catch shinies leaderboard but that will be a separate ticket.

Forwarded!

Closed, ticket is no longer in effect.


Catch iframe errors

Is there a way for Bugsnag to catch also errors coming from webpage in iframe?

Hey @LionelPaulus, errors from iframes don't bubble up to the parent window so this is something you would have to do manually right now, either by calling the parent window's onerror method or by passing error data via window.postMessage.
A word of warning – it's fraught with complexity and you're likely to run into CORS errors unless all JS and content is served from the same domain. The simplest and most foolproof solution is to simply load Bugsnag in every frame, but if you care about page weight that's obviously not ideal.
The combination of the complexity (high) and the demand (low) for this feature is why we haven't investigated building it yet, but we are tracking it on our internal roadmap and I have added your interest to that.
Cheers


added focus-lock to modals

Ticket Link #988
Description

Forced modals to not allow tabbing (or shift+tab) out of the modal.

Reminders:

[x] Be mindful of impacts of changing Major/Minor/Patch versions of each elixir app

[x] If updating patch version, are you sure there are no chart changes required to maintain functionality? If so, you should bump minor version instead.
[x] If updating Major or Minor versions , did you update the sauron chart configuration?

[x] If changing elixir code in an "app", did you update the relevant version
in mix.exs?
[x] If altering an API endpoint, was the relevant postman collection updated?

[x] If a new version of smart_city is being used (new fields on a struct), were the relevant postman collections updated?

I'm open to adding a javascript library to address this if needed. Seems like the bundle size is pretty small<EMAIL_ADDRESS>Were there no other html native ways of doing this? Using tabindex seems interesting but maybe it's not an option, and x-trap is a better way to go. https://www.w3docs.com/snippets/html/how-to-prevent-tab-indexing-on-specific-elements.html

tabindexing was a no-go, as I would have to go into each field in the background and make it -1 every time we open a modal, which I think will turn ugly real quick.
There are bootstrap ways to build a modal that I know of via html (well, still kinda using something besides html), but don't didn't want to bring in that whole thing just for all the stylings to go to wack.
I can give x-trap a shot, hold a minute.


Description
A classic snake game with python
Fixes #126
Type of change
Choosing one or more options from the following as per the nature of your Pull request.

NOTE: These boxes can be checked using [X]

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

[X] New feature (non-breaking change which adds functionality)

[ ] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[X] I have named my files and folder, according to this project's guidelines.
[X] My code follows the style guidelines of this project.
[X] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[X] I have commented on my code, particularly in hard-to-understand areas.
[X] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[x] I have included a requirements.txt file (if external libraries are required.)
[X] My changes do not produce any warnings.
[X] I have added a working sample/screenshot of the script.

@AdityaJ7  please review...

Add a requirements.txt file

Done

thank you @AdityaJ7 , I didn't noticed the file game and version,
if there are any other changes please let me know,
otherwise please merge this Pull Request.

@all-contributors please add @infern018 for code


Blurry Demo Reconstructions

Both the 2D and 3D Shepp Logan as well as microscopy demo reconstructions look blurry.  See the attached images.  Are the prior parameters set wrong?

Xiao, the microscopy recon is always going to be a little blurry due to the limited data.
However, I agree that, on average, the recons tend to be a bit over regularized.
This is closely related to the issue #239, which recommends reducing "regularization" for the proximal map mode.
I propose we try two changes in the prerelease branch that address both problems consistently.
In auto_sigma_x we can change the scaling factor to 0.3 so that:

return_ 0.3 * auto_sigma_prior(sino, delta_channel, sharpness)

And in In auto_sigma_p we can change the scaling factor to 2.0 so that:

return 2.0 * auto_sigma_prior(sino, delta_channel, sharpness)

There could be some backward compatibility issues, but I think it's better to pull the bandaid off now.
If this is agreeable to everyone, I can make the changes, and then you can try running the new code to see if you like it better.

Sounds good!

I had an extensive discussion with Jordan about this, and I'm talking to different people to formulate the best solution. This is a very small change, but it needs to be done very carefully.

OK, my solution to this problem was just to increase the value of sharpness in the demos.
In the future, we may want to increase the default sharpness values in the code.


Cluster Provisioning v2: Linode - Create RKE2 clusters

Somewhat related to what's needed in Dashboard: https://github.com/rancher/dashboard/issues/3232
This exists in Ember via RKE1 so this should be a port for RKE2. We'd replace the generic components with machine templates and the API. There's a folder in Vue we can use.

Would include node pools and the other components.

Linode now available

@vincent99 Is this already done? It appears that the option to create a Linode RKE2 cluster is in the UI already.

These issues were for custom components that talk to the Linode API and show you the appropriate options and such.  What you're seeing now is the generic support for any driver.

To Validate:
As a standard user

Create Linode cloud credentials
Create Linode RKE2 cluster (note RKE2 version). - 3 etcd, 2 cp and 3 worker nodes. Cluster should come up Active
Create Linode k3s cluster (note k3s version). - 3 etcd, 2 cp and 3 worker nodes. Cluster should come up Active

Setup For Validation
Rancher: v2-6-head(1d2b746)
Install: Helm
Local k8s: v1.21.10
Test Cases For Validation

Create Linode cloud credentials (I didn't have access to these had to borrow creds)
Create Linode RKE2 cluster (note RKE2 version). - 3 etcd, 2 cp and 3 worker nodes. Cluster should come up Active
Create Linode k3s cluster (note k3s version). - 3 etcd, 2 cp and 3 worker nodes. Cluster should come up Active

Results
Was able to successfully to accomplish the above test cases.


fix(lineScaleRandom) Correction lineScaleRandom

What: "LineScaleRandom" animation fixed.

Why: These corrections are requested at issue #441

How: I used the previous animation code and set the new animation. I made some adjustments and adapted the tests cases.
I had to run the eslint script.
Closes #441

@Larissagilliane Thank you for taking the time to work on this PR and fixing issue #441
You are awesome 🎉

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Query fails when schema contains types with more than "one word", capitalised as per AWS standard convention (e.g. DashboardItems)

with the following Amplify graphql schema:
type DashboardItem
  @model
  @key(
    name: "byName"
    fields: ["name", "id"]
    queryField: "dashboardItemsByName"
  ) {
  id: ID!
  name: String!
}

Even though the default query variable is cased correctly:
const defaultQuery = "listDashboardItems";
The following error appears in the console:
Could not find query listDashboarditems
I'm assuming the case is being modified somewhere in the dataproivider?

Nope - ignore me. My bad - my resource was incorrectly cased!
Problem solved. User Error :)


update validationResolver section in api page translated into jp

completed: https://github.com/react-hook-form/react-hook-form-website/commit/d730fe15ca405e73d23a4eb7a85c09d4199b7065

ty @kotarella1110  :)


size of next and previous icons look a bit small

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal

Motivation and context
usability

@soniakaukonen to confirm if the icon is correct in Modus Styleguide, Modus Bootstrap and Modus React Bootstrap

I checked the behavior of Pagination and Breadcrumbs along with the right/left chevron @msankaran0712
Pagination:

With 24 px icon, the actual symbol is only 610 px meaning fairly small with 97 px padding
Breadcrumbs:
When 16 px icon is used, the actual symbol is anymore 4.57,5 px (padding 64 px)

This explains the poor visibility.
We have a fat chevron for in-field purposes but this is overwhelming the above mentioned components. The stroke is not as important as using most of the canvas.
Solution could be an additional chevron using more of the canvas within the 2424 and 1616 px.

Maha to confirm if there is any extra padding that can be removed

will wait for @soniakaukonen to update the figma


Can not hover an element that is outside viewport

@dotneet   I think this may be a somewhat challenging issue.
I have a document with a height of 1000px and a viewport (window) with a height of only 800px, if I attempt to hover an item which starts at 900px (using mouseMoved()) the hover fails.   This makes sense since in normal chrome it is impossible for a user to move the cursor beyond the window bounds.   However, it causes confusion when taking screenshots in the document (whole page) context.   This is especially true when using backstop because this was not a limitation of phantom.
I am wondering, would it be possible to maintain the user's intended cursor position (probably by simply aggregating arguments to all mouseMoved() calls) prior to switching to document context.  If so, would it also be possible to set the user's intended cursor position prior to taking screenshots?
This is a pretty big issue for backstop usecases.  What do you think?

@garris
Always enabling viewport emulation that is used in .screenshotDocument() will possibly be solution.
I'm not sure if it has possibility to cause another problem.
At least result of both .screenshot() and .scrollTo() will be changed. (this is allowable in backstopjs?)
Anyway I'll confirm whether this idea is possible and or find out another approach.

Thanks for the reply,  just to be clear -- the issue is happening with the  screenshotMultipleSelectors() usecase.
I like your first idea.  Perhaps adding a method to force the necessary emulation (before making any mouseMoved() calls) would ensure there would be no regressions for established chromy cases?


AnimeGo 1.3

[x] Fix https://github.com/HenryQuan/AnimeGo-Re/issues/22
[x] Seasonal anime page bug fix
[x] Upgrade to Flutter 2.2
[x] Fix potential UI issues
[x] Migrate to null safety
[x] Firebase integration
[x] #28
[x] Add license and ~privacy policy~ in Settings (privacy policy will be added later)
[x] ~Add age restriction~ I will add this later as it is not urgent
[x] Migrate to flutter_webview
[x] Reset project (a fresh flutter 2.0 to prevent potential issues)

I will try using the new version for a few days to make sure it is working properly.

Everything are fine so far. Next week, I will release a new build.

All good so far. A new build will be updated.


Battery UI fixes

Description

Disable minimum time at charge state for Standalone Battery (unclear why)
Generic Battery disabling for Automated dispatch custom weather file (similar to Standalone battery)
Generic Battery msgbox for PVSmoothing

Fixes #798
Type of change
Please delete options that are not relevant.

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

Checklist:
If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.

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

Test results were the same outside of ETES (is there variation in the optimizer?). Merging this PR.


  A few recent issues with swig/m2crypto have brought this up.  The M2crypto package is mostly (only?) used by the iam ssl watcher/auditor.
This M2crypto package should be ripped out and replaced with the Cryptography package.
The swig requirement should also go away.
@kevgliss will likely take the lead on this.

Completed in #188


Balanced Substrings

💥 Proposal
To identify the unique balanced substrings that can be formed using the given input string and print the balanced substrings as output

Balanced substrings are those in which count of occurrences of each character within the substring , are equal
Ex: aabb, abbcca, etc

Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Compile error with Gradle 4.0

While switching over to Android Studio 3.0, which uses Gradle 4.0 compiling fails with the following setup.

Android application depends on Java-library module
Android application has "apply plugin: 'com.archinamon.aspectj'"

As far as I understand Gradle 4.0 does not build .jar files anymore to speed up builds. Instead it works on .class files of dependencies directly.
The com.archinamon.aspectj plugin depends on the jar files, which makes the build (actually already the configuration phase) fail.
Example project: https://github.com/HaukeRa/Archinamon-GradleAspectJ-Android-FailureTestProject

Thanks for reporting! I'll take a look as soon as I'll have some time.

Hello again!
Please, try out my adapted code for gradle plugin 3.0.0-beta7:
classpath 'com.archinamon:android-gradle-aspectj:3.1.1-gradleApiBeta'
Sources available in separate branch (temporary): https://github.com/Archinamon/GradleAspectJ-Android/tree/feature/gradleApiBeta_3.0.0-beta7


Incorrect package number

after today's update (probably) neofetch is showing one less package then it should.
http://hastebin.com/dimesujoyu.coffeescript

I can reproduce this.

I've reverted the commit that caused this. I don't know if this affects all package managers so I might have to release another version of Neofetch.

For some reason it only affects system with pacman, I can reproduce this on my laptop, but not on my desktop (emerge).


Quartermaster

This is a new shaman borrowing heavily from the onboarder shaman in this repository. It's intention is to give the captain (address holding the captain hat) the ability to bring new members to a DAO (adding them to the crew). Each crew member receives the minimum shares. In addition there is an onboarding delay so that if members notice the captain is making poor decisions in bringing on new members they can either ragequit or mutiny. The delay is twice the voting period to allow for this action.
I'm happy to work with the team to figure out how best to approach this. If there is another approach to introducing this module please let me know.

Cool, I can keep it there. I borrowed heavily from the onboarder, so thought it would be good to at least offer it up, but because it is pretty specific to this use-case it makes sense to not include it in this repo. Thanks for checking it out though. I'll be sure to add it to the wiki when it's ready.

Cool, I can keep it there. I borrowed heavily from the onboarder, so thought it would be good to at least offer it up, but because it is pretty specific to this use-case it makes sense to not include it in this repo. Thanks for checking it out though. I'll be sure to add it to the wiki when it's ready.

@plor To be clear, the suggestion is less about restricting this contract from this repo, and more about wanting to highlight the work of ecosystem developers. Developers controlling their own repos is more in line with a robust ecosystem of modular hats-powered tools.
I should also have mentioned that you should definitely submit this as a contribution to the hats protoDAO.

Thanks, will do!


Update sandbox

Support solidus 3.2+

Think we'll have to update the orb version in circle config so that 3.2 specs run?


iOs issue: biometricKeyExists always false

On an iOs real device the result of biometricKeyExists() is always false, nevertheless  I generated a key pair and got the public key.

@chamodanethra I proposed the fix in PR.
Thank you.


[fprop] Add new method for tensor state updates

@cryptoannavan could you give an update on the PR to answer questions from @nottombrown? Otherwise, we will most likely close this PR in a few days

Due to inactivity, I'm closing this PR for now. Feel free to reopen once comments by @nottombrown where addressed and the CLA signed. Thanks!


[T4A7][T15-A2]Yoshiaki Nishimura

Ready for Review

I think you misunderstand the concept of TDD.
TDD asks you write a small test case first (so where is your test case?), then watch it fail and then write the implementation to make it pass. This process can repeat many times until you get desired result.

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


Ubuntu Repository Instructions - SSL Error

The following instructions for installing on Ubuntu fails due to a SSL error:
https://github.com/regolith-linux/regolith-desktop-website/blob/ad83e6b0892c8bc248a26622c5c234f328fb2441/static/install-release-ubuntu-22.04-amd64.txt#L2
Changing the https://regolith-desktop.io to https://regolith-desktop.org fixes the issue.

Hey,
its the certificate which expired a month ago. I guess the maintainer is on vacation?
Cheers

All cert issues should be resolved now.  All package URLs should be regolith-desktop.org.  Kindly re-open if any problems persist.


updated bug: on refresh, the app is not fetching the current logged i…

What does this PR do?
A solution to issue no.56
When I refresh the todos page I redirect me to the login page.
Test Plan
a little change in the API file. tested locally : create user -> refresh -> redirect to todo, and logout->login->refresh-> redirect to todo
Related PRs and Issues
https://github.com/appwrite/demo-todo-with-react/issues/56
Have you read the Contributing Guidelines on issues?
yes

thanks @talmosko !


Images UI doesn't load images correctly when clicking Search

This is an issue that remains from https://github.com/G-Research/fasttrackml/issues/1238. The images are currently loading correctly when refreshing or accessing the URL directly, however they don't seem to load correctly whenever there is an update (such as with the Search button).
There are two separate problems:

[ ] Images don't load correctly unless refreshed
[ ] Selected sequence names are not fetching correctly (it seems all are being fetched at once)

Moving this one back to the board as we're almost done with it!


Documentation request: How indicate end of querying?

Disclaimer: Might have overlooked some existing docu.
The general template to use TauProlog seems to be:
// Consult
session.consult(program, {
    success: function() {
        // Query
        session.query(goal, {
            success: function(goal) {
                // Answers
                session.answer({
                    success: function(answer) { /* Answer */ },
                    error:   function(err) { /* Uncaught error */ },
                    fail:    function() { /* Fail */ },
                    limit:   function() { /* Limit exceeded */ }
                })
            },
            error: function(err) { /* Error parsing goal */ }
        });
    },
    error: function(err) { /* Error parsing program */ }
});

Whats the recommended way to indicate that no more answers are requested?
Does the answer hook need to throw an exception or return false?

The answer method just looks for the next answer. If you need more than one answer, you must call answer again in the callback of the first answer. So if you don't need any more answers, you don't have to do anything.


Make BarChart inherit from Axes instead of using Line

Enhancement proposal

The BarChart right now uses Line to draw the axes which doesn't have much flexibility. But Axes can be used for this as it can be easier and more flexible for the users to use this class and also get more features.
https://github.com/ManimCommunity/manim/blob/375d9bb0b8713f4d804361767f9d426278e81bee/manim/mobject/probability.py#L278-L280
Additional comments

Sounds like a good idea

I think the same as you. Barchart should inherit from Axes. I'll try to fix this and make a PR

Done via #2387


Updated graph api permissions

Updated AD app graph permissions to remove User ReadWrite and replace with Read only.
Also removed permissions for the Azure AD graph and changed these for the MS Graph equivalent (due to Azure AD graph deprecation).

@abbodh as discussed


Geojson handling?

Can this handle geojson marker dataset, and present a field in the sidebar (i.e,, address, phone, email)?

this plugin does not care about the contents at all, you can add whatever you want as content including geojson data.


Too many metrics generated when using Path Variables

Using requests with PathVariables in reactive feign client
(ex.:     @GetMapping("/{cropId}/summary"))
leads to generating metrics for each request separately
(ex: /2339501/summary.snapshot.50thPercentile, /2339502/summary.snapshot.50thPercentile, etc.)
Then it finishes with this Warning:
WARN org.springframework.boot.actuate.autoconfigure.metrics.OnlyOnceLoggingDenyMeterFilter - Reached the maximum number of URI tags for 'http.client.requests'. Are you using 'uriVariables'?
Same requests didn't generate separate metrics in synchronous feign client.

Reactive Feign client has it's own Micrometer metrics. set reactive.feign.metrics.enabled to true to enable default Micrometer logger. Also make sure that you added actual Micrometer implementation module into your project dependency

The problem is not in that there are no metrics. The problem is that there are too many metrics: Reactive feign client generates separate metrics for same requests, for each PathVariable. In that case, if request contains UserId in path variables, Reactive feign will generate thousands of same metrics.
seting reactive.feign.metrics.enabled to true/false doesn't change situation

To many metrics generates Spring WebClient as reactor feign doesn't use it in form of template with substitutions. You need to turn off WebClient metrics and turn on reactive feign metrics.
Please check MetricsTest.

Thanks, it helped:
management.metrics.enable.http.client.requests: false
reactive.feign.metrics.enabled: true
Please, add this to documentation.


Capture cluster uuid

Cluster uuid is not exported by default.  It would be helpful to link data from other sources:
GET _cluster/stats?output_uuid=true

Could you be more specific about the other sources and how you are going to do this?

It's in the 2.0.2 release.


Trying to playlist causes mpd connection to go insane

INFO     2021-12-17 19:58:57,189 [188570:MainThread] mopidy.commands
  Starting GLib mainloop
INFO     2021-12-17 19:58:57,780 [188570:MpdSession-14 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42160
INFO     2021-12-17 19:59:07,434 [188570:MpdSession-15 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42162
INFO     2021-12-17 19:59:13,441 [188570:MpdSession-16 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42164
INFO     2021-12-17 19:59:18,949 [188570:MpdSession-17 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42166
INFO     2021-12-17 19:59:24,456 [188570:MpdSession-18 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42168
INFO     2021-12-17 19:59:29,962 [188570:MpdSession-19 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42170
INFO     2021-12-17 19:59:35,468 [188570:MpdSession-20 (_actor_loop)] mopidy_mpd.session
  New MPD connection from [::ffff:<IP_ADDRESS>]:42172

Which eventually goes to
  Rejected connection from [::ffff:<IP_ADDRESS>]:42288
WARNING  2021-12-17 20:01:29,635 [188570:MainThread] mopidy_mpd.network
  Rejected connection from [::ffff:<IP_ADDRESS>]:42290
WARNING  2021-12-17 20:01:30,637 [188570:MainThread] mopidy_mpd.network
  Rejected connection from [::ffff:<IP_ADDRESS>]:42292

How odd.  Adding yt:playlist:PLey33PeMtXtf9wqZglQi7mMiX5OK1SI77 in musicbox_webclient works just fine for me.  As does adding https://www.youtube.com/playlist?list=PLey33PeMtXtf9wqZglQi7mMiX5OK1SI77 using the new mopidy-youtube webclient.
Are you able to provide any further details?

Ok, i went for family awhile, and running this in musicbox is fine, so is the mpc command. But ncmcpp just spazzes it out.

Ok, so i just realized, now apparently the search, doesn't just pull from wherever. Adding it in ncmpcpp, actaully works. Closed.


JRE timezone database not accessible compiling with Java 9 or above

package day30_datetime;
import java.time.LocalTime;
public class C2_LocalTime {
public static void main(String[] args) {
    LocalTime saat = LocalTime.now();
    System.out.println(saat);
}

}
i get this error messages
can't open C:\Users\Tec Pro\AppData\Local\Temp\eoi1CD3.tmp\plugins\org.eclipse.justj.openjdk.hotspot.jre.minimal.stripped.win32.x86_64_15.0.1.v20201027-0507\jre\lib\tzmappings.
Exception in thread "main" java.lang.Error: java.io.FileNotFoundException: C:\Users\Tec Pro\AppData\Local\Temp\eoi1CD3.tmp\plugins\org.eclipse.justj.openjdk.hotspot.jre.minimal.stripped.win32.x86_64_15.0.1.v20201027-0507\jre\lib\tzdb.dat (The system cannot find the file specified)
at java.base/sun.util.calendar.ZoneInfoFile$1.run(Unknown Source)
at java.base/sun.util.calendar.ZoneInfoFile$1.run(Unknown Source)
at java.base/java.security.AccessController.doPrivileged(Unknown Source)
at java.base/sun.util.calendar.ZoneInfoFile.(Unknown Source)
at java.base/sun.util.calendar.ZoneInfo.getTimeZone(Unknown Source)
at java.base/java.util.TimeZone.getTimeZone(Unknown Source)
at java.base/java.util.TimeZone.setDefaultZone(Unknown Source)
at java.base/java.util.TimeZone.getDefaultRef(Unknown Source)
at java.base/java.util.TimeZone.getDefault(Unknown Source)
at java.base/java.time.ZoneId.systemDefault(Unknown Source)
at java.base/java.time.Clock.systemDefaultZone(Unknown Source)
at java.base/java.time.LocalTime.now(Unknown Source)
at day30_datetime.C2_LocalTime.main(C2_LocalTime.java:6)
Caused by: java.io.FileNotFoundException: C:\Users\Tec Pro\AppData\Local\Temp\eoi1CD3.tmp\plugins\org.eclipse.justj.openjdk.hotspot.jre.minimal.stripped.win32.x86_64_15.0.1.v20201027-0507\jre\lib\tzdb.dat (The system cannot find the file specified)
at java.base/java.io.FileInputStream.open0(Native Method)
at java.base/java.io.FileInputStream.open(Unknown Source)
at java.base/java.io.FileInputStream.(Unknown Source)
... 13 more

The tzmappings resource files in the java.base module (Java 9 and above) is not accessible when executing the j2objc binary.
Workaround is to install Java 8 and use that to execute j2objc.

FYI, compiling j2objc-generated files requires Mac hardware, due to Apple's iOS SDK license. If you don't have a Mac, there's therefore no point in using j2objc.

Another workaround is to use j2objc-2.7 with either Java 1.8 or 11.

After building from the latest source, I was able to compile your example with Java 15 and Java 11 without issue. We're planning a 2.8 release in the next couple of weeks.


typo in readme?

Dataset with the training ready format + only the is_useful == rows is ...
You missed something after == sign? should it be "== true"?

Thanks! I've made the fix.


MVC4 throws exception when using autofac and MVC async controller actions

Seems to have a problem with autofac lifetime scopes.

Facing exceptions with Autofac and MVC4 in asynchronous controller actions can be challenging. Ensure that your dependencies are correctly registered and handled for asynchronous operations. Reviewing the specific exception details and your code structure might help funding status pinpoint the issue. Good luck in resolving this.

Awesome work thanks for sharing this Fix  info with us.

Check your Autofac configuration for async support. Ensure  you register async dependencies correctly. Also, verify if you're using the latest version of Autofac compatible with MVC4 and async actions.

MVC4 throws exceptions when using Autofac with MVC async controller actions due to improper configuration. Ensure that the async controllers and their dependencies are correctly registered in Problems Autofac. Updating to the latest Autofac and MVC packages might also resolve compatibility issues. Properly configuring dependency injection can prevent these exceptions.


Bump metrics git hash to incorporate default executor change

High Level Description

This change bumps the commit hash of the dcos-metrics module so that the STATSD_UDP_HOST and STATSD_UDP_PORT envvars are exposed to tasks running on the Mesos default executor (https://github.com/dcos/dcos-metrics/pull/106).

Related Issues

None

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: This merely changes a reference to a commit in another repo that has not seen any interface changes.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): commit
[x] Test Results: results
[x] Code Coverage (if available): coverage

Does this override https://github.com/dcos/dcos/pull/1613/files? (which is a sad example for how bad our integration work sometimes is. Sorry @philipnrmn).

@jgehrcke Yes, it bumps the ref of dcos-metrics to a later commit that includes all other changes that have been made in that repo since then. However, I'm waiting for https://github.com/dcos/dcos-metrics/pull/108 to land before I update this PR to actually reference a merged commit.

@mesosphere-mergebot bump-ee

If you want that latest commit to work @mrbrowning you'll need to change the ref_origin to your branch.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ready for review

@mesosphere-mergebot label ready for review

@mesosphere-mergebot bump-ee

Superceded by: https://github.com/dcos/dcos/pull/1715

Superceded by: #1715

Closing this PR as of this statement.


Switching themes between modus-vivendi and modus-operandi doesn't work.

On switch to dark theme to light theme and then back to dark it looses all the highlights.

It should be fixed by 16460aa92cdf729d1e94d9abe5f7c9a3de68a1af


Update security-release.md

Updated phrasing around working days in line with mattermost/mattermost-developer-documentation#442 but retained working days as opposed to business days for consistency across the doc. One minor typo fix.

@justinegeffen can you help resolve the conflict, then this should be good to merge?

My bad, didn't see that merge conflict! All fixed.


Add documentation about performance and optimizations

Note to myself: I will need to change the examples to use import.


Normalize forum name on creation

@gvilarino could you elaborate?

Forums should only accept URL-friendly names on creation, and our associated endpoints (i.e.: api/<forum-name>) should lower-case any input. The same should be done for anything in the URL path
So: hostname.org/SOMEFORUM/TOPIC/444FFF should be interpreted as hostname.org/someforum/topic/444fff

test

test

we saw this today
https://app.democracyos.org/generationavenir/topic/55f876492ad72f100099fefb works but https://app.democracyos.org/GENERATIONAVENIR/TOPIC/55F877312AD72F100099FF03 doesn't

The forum name was already beign normalized, but, nevertheless this issue is closed by #1085

This issue should only handle the redirection when the forum name is capitalized, not any part of the route. To avoid the uncapitalization of form params, and other unhandled errors.

anything in the valid path should be uncapitalized. Not id or some other form of data but yes for path segments (e.g.: forum, topic, comment, etc)


Implement support to the SESSION variables in libmodsecurity

Session is a collection that needs to be implemented.
More information about it can be found at ModSecurity reference manual: https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual

Support to SESSION was added at: c9131e8


Fix bugs in implementation of Advanced Blending Presentation State IOD

There is currently a check that prevents reference of images from more than one series in presentation state instances. However, in case of the DICOM Advanced Blending Representation State IOD, images from more than one series may be blended together (as long as the images are defined in the same frame of reference). This PR relaxes the check to allow the referencing images from multiple series.
The PR also fixes the inclusion of attributes of the Palette Color Lookup Table Macro in the Advanced Blending Sequence. The attributes should be added to an item of the Palette Color Lookup Table Sequence rather than the item of the Advanced Blending Sequence.

It doesn't exist yet :) I have been working with David on a correction proposal (CP) to include it into the standard. The changes in this PR are necessary to comply with the CP.


[bug] <rh-card>: incorrect spacing between header and paragraphs

Note: Will be true after #1380 merges

https://deploy-preview-1380--red-hat-design-system.netlify.app/elements/card/demo/variants/
Per the design spec we should only have 32px here. However we have 32px margin on #body + 16px margin on the <p>.

Related future state css: https://developer.apple.com/videos/play/wwdc2023/10121/?time=238

it's not clear to me that this can be done from the shadow root.

there's no :first-of-type-in-slot pseudo class,
we can't ::slotted(* + p:first-of-type) because complex selectors aren't allowed
we can't :first-child, because what if the heading is the first child?
we can't :first-of-type, because the p that comes after the body image is the first of type

we could add a special slot for the logo image, but i'm preeeeetty sure that's not a proper solution (too specific)
or we could say "this is a pattern-level issue" i.e. require light-dom css

it's not clear to me that this can be done from the shadow root.

This was my findings, unfortunately just no way of doing it given complex selectors not being allowed.


Build error on latest version when building with cmake -DWITH_HWDEC=0 -DCMAKE_BUILD_TYPE=RelWithDebInfo

I get some errors when building with make  -DWITH_HWDEC=0 -DCMAKE_BUILD_TYPE=RelWithDebInfo but not with cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo ..  it builds just fine.
I'm using Linux mint 17.2 and Firefox 40.0.3 and pepperflash <IP_ADDRESS>. I build from the latest source on here. I installed ffmpg from ppa:kirillshkrogalev/ffmpeg-next in case I needed it as a prerequisite for using the -DWITH_HWDEC=0 option,
[ 25%] Building C object src/CMakeFiles/freshwrapper-obj.dir/main_thread.c.o
[ 26%] Building C object src/CMakeFiles/freshwrapper-obj.dir/reverse_constant.c.o
[ 27%] Building C object src/CMakeFiles/freshwrapper-obj.dir/tables.c.o
[ 28%] Building C object src/CMakeFiles/freshwrapper-obj.dir/trace.c.o
/home/gaining/freshplayerplugin/src/trace.c: In function ‘trace_var_as_string’:
/home/gaining/freshplayerplugin/src/trace.c:100:44: error: expected ‘)’ before ‘PRId64’
res = g_strdup_printf("{ARRAY:%d:%"PRId64"}", ref_count, var.value.as_id);
^
/home/gaining/freshplayerplugin/src/trace.c:100:44: warning: format ‘%d’ expects a matching ‘int’ argument [-Wformat=]
/home/gaining/freshplayerplugin/src/trace.c:100:44: warning: spurious trailing ‘%’ in format [-Wformat=]
/home/gaining/freshplayerplugin/src/trace.c:105:49: error: expected ‘)’ before ‘PRId64’
res = g_strdup_printf("{DICTIONARY:%d:%"PRId64":%s}", ref_count, var.value.as_id, s_dict);
^
/home/gaining/freshplayerplugin/src/trace.c:105:49: warning: format ‘%d’ expects a matching ‘int’ argument [-Wformat=]
/home/gaining/freshplayerplugin/src/trace.c:105:49: warning: spurious trailing ‘%’ in format [-Wformat=]
make[2]: *** [src/CMakeFiles/freshwrapper-obj.dir/trace.c.o] Error 1
make[1]: *** [src/CMakeFiles/freshwrapper-obj.dir/all] Error 2
make: *** [all] Error 2

trace.c doesn't include inttypes.h (where PRId64 macro is defined) directly, but on my machine it's somehow included nonetheless.
Added explicit inclusion in 8cc1005bdb1651af8b105bd5630d539e35634b00, that should fix it.

Yep, it did. Thanks again. I really appreciate the work that you do here. :)


[wip] windows

These are the changes mentioned here, though again work in progress.  At this point, it builds but doesn't want to load the shared library.  I'll have to dig in a bit further..

It may be better to discuss this on the isl mailing list, see http://isl.gforge.inria.fr/

(Also, thanks for working on this!)


Feature/rocket booking

For this task, the following changes were introduced:

Followed the same logic as with the "Reserve rocket" but set the reserved key to false.
Dispatches this action upon clicking on the corresponding button.
Added conditional rendering of buttons

Hi @jfoyarzo ,
✔   Status: APPROVED 🏆 🎉 🎊
You have done a great job on this project, keep it up 👍 🥇. Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
congratulations!  🎉💯🌟
Top highlight 🎯

Excellent Implementation
The PR title and description are professional and descriptive ✅
Linters are working properly ✅


Declarations are unexpectedly used from node_modules in parent directory

TypeScript Version:  3.4.0-dev.20190227
Search Terms: nested node_modules module resolution
I have a project nested inside another project. Both package.jsons import the same library, but they are two different versions of the library.
Issue
In the files inside the inner project, I am able to import something that is only exported by the version of the library used by the outer project. When I open just the inner project in VSCode, I see two entries in the import intellisense for each export from the shared library, one from each version. It appears that Typescript is recursing up the directory tree, looking for node_modules and merging their declarations even through the library is already installed in the local node_modules.
Code
Here's the import that should fail. I should only be able to import from "fp-ts/lib/HKT" here:
https://github.com/leemhenson/fp-ts-oom-repro/blob/master/subproject/user.ts#L2
Here's the inner project's import of the library:
https://github.com/leemhenson/fp-ts-oom-repro/blob/master/subproject/package.json#L7
Here's the outer project's import of the library:
https://github.com/leemhenson/fp-ts-oom-repro/blob/master/package.json#L8
I've tried settings types: [] and typeRoots, exclude: ["../node_modules"] etc.
Related
https://github.com/Microsoft/TypeScript/issues/11994
https://github.com/Microsoft/TypeScript/issues/11363
https://github.com/Microsoft/TypeScript/issues/11257

Did you tried using Path mapping? Be aware that will not rewrite your imports.

    "paths": {
      "fp-ts/lib/*": ["./node_modules/fp-ts/lib/*"]
    },

and variants have no effect.

I have the same problem.  what I think is happening is that:
node_modules/typescript/lib/typesMap.json
has an entry for 'md5' (and other things) and it's using the parent mode_modules for resolving it.
I then get a ton of other associated errors due to module differences.
I tried changing typeRoots, exclude, and paths and none of them work.  The documentation is incorrect that parent directories will not be used.
The only work around I've found is to move the parent node_modules out of the way.
The root cause is that typescript tries to pull it's own things in which I didn't specify and there's no way to disable this or tell it to not load from ../node_modules

I also need to note that moving the node_modules temporarily out of the way is a really bad work around as it causes IntelliJ to go to 100% and needs to re-index all my modules for every build.

Also, an sub -project with only one .ts file (empty) still generates errors as typescript tries to load modules in its type map from the ../node_modules

I have exactly the same issue as @burtonator and @leemhenson. Tried every possible combination of compilerOptions.typeRoots and compilerOptions.types, but the TS compiler keeps including ../../node_modules/@types no matter what I do.

 "compilerOptions": {
       "types" : []
   }

Will do the job.

 "compilerOptions": {
       "types" : []
   }

Will do the job.

Will not do the job (fully). You should read documentation for typeRoots and types. It only works for global modules, not imported. For important, you have to use paths and baseUrl.
Either the docs are not correct, I did not understand them, or there is bug, because using it does not seem to restrict module resolution.
I have the following set in an inner project's tsconfig.json:
{
  "compilerOptions": {
    "strict": true,
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "lib": ["dom", "es2017"],
    "jsx": "react",
    "typeRoots": ["./node_modules/@types"],
    "baseUrl": ".",
    "paths": {
      "*": ["./node_modules/*"]
    }
  },
  "include": ["src"]
}

Packages from parent/node_modules are still resolved.
Used typescript 3.7.4.

This is still an issue as of 3.9.3.

Correction. In my case "types" : [] does solve the problem.
I originally was placing it outside of compilerOptions.
A few observations:

Without types:[] nodejs is trying to resolve a bunch of typings implicitly. In my case it all starts with:

======== Resolving type reference directive 'estree', containing file '/Users/sly/Work/foo/bar/frontend/__inferred type names__.ts', root directory '/Users/sly/Work/foo/node_modules/@types'. ========

I don't know what implies estree. It is a dependency of a sibling project.

"classic" module resolution is the default and stays the default for backward compatibility. Maybe TypeScript 4 would a great time to change that?

Even with moduleResolution: "node", half of my filesystem is walked for implied typing modules, but the compilation succeeds.

Setting typeRoots to /dev/null also solves the problem.

It seems the root issue is how the default typeRoots is calculated combined with "classic" being the default module resolution method.

While I can understand not wanting types from a parent node_modules folder to show up in a subproject when there are conflicts, our behavior is mimicking Node's resolution behavior:
Welcome to Node.js v14.7.0.
Type ".help" for more information.
> process.cwd()
'D:\\dev\\scratch\\fp-ts-oom-repro\\subproject'
> require.resolve("fp-ts/lib/Either")
'D:\\dev\\scratch\\fp-ts-oom-repro\\subproject\\node_modules\\fp-ts\\lib\\Either.js'
> require.resolve("fp-ts/lib/HKTv14_1")
'D:\\dev\\scratch\\fp-ts-oom-repro\\node_modules\\fp-ts\\lib\\HKTv14_1.js'
> require("fp-ts/lib/HKTv14_1")
{}
>

As far as I can tell, this behavior is the correct behavior and is not a bug.

@rbuckton If I understand @leemhenson correct, his issue is that the types package is found locally, but the types from parent node_modules is still used and merged. AFAIK node's behavior is to only go up, when package has not been found locally.

My comment was intended to serve as proof that node still navigates up when a part of a package sub-directory isn't found.

My comment was intended to serve as proof that node still navigates up when a part of a package sub-directory isn't found.

I don't know what Node is doing, but what TypeScript is doing doesn't seem right. In my case it's trying to parse type files and fails with errors like this:
ERROR in<EMAIL_ADDRESS>[tsl] ERROR in /packages/app-cli/node_modules/@types/jest/index.d.ts(462,62)
      TS1005: ']' expected.

ERROR in<EMAIL_ADDRESS>[tsl] ERROR in /packages/app-cli/node_modules/@types/jest/index.d.ts(462,65)
      TS1005: ';' expected.

I can't make any sense of these errors, but more importantly why should I care that TypeScript can't parse files that aren't even used by my project? If I move said project to a different place, it works just fine.
Assuming this is, anyway, the right way to load packages (also it feels very wrong), at least there should be a clear options to disable this behaviour.

These errors look to be coming from a third-party tool (tsl?), not TypeScript.


AnimationController.animateTo() shouldn't take a Curve

If you want to curve the animation, you should just plug a CurvedAnimation onto the controller.

@abarth looked at this and I think decided it was a WONTFIX.


Unexpected token < when use component

Hello. Sorry for creating issue about specific component, but this one is too good. I am talking about https://github.com/drewjbartlett/vue-flickity
I made it work with non-SSR Vue app, but got errors with Nuxt.
[vue-router] Failed to resolve async component default: SyntaxError: Unexpected token <
[vue-router] uncaught error during route navigation:
/home/***/Documents/WORK/EXAMPLES/nuxt-examples/node_modules/vue-flickity/flickity.vue:1
(function (exports, require, module, __filename, __dirname) { <template>
                                                              ^

SyntaxError: Unexpected token <

I never expect such errors before, so I ask your help.
By the way, error appears when I refresh the page.

Have you tried to create a custom plugin with ssr:false option?
module.exports = {
  plugins: [
    { src: '~/plugins/vue-flickity', ssr: false }
  ]
}

I can't pass option when I tried to create one, so no. I didn't tried. But the guy from discord channel says he tried and it didn't help

@NicoPennec as for no-ssr I tried EGOIST plugin. I think Nuxt using it too? It didn't help.

By the way, I can't make original library work with Nuxt too - https://flickity.metafizzy.co/

I ended up using this:
// plugins/vue-flickity.js

import Vue from 'vue';
import Flickity from 'vue-flickity';

Vue.component('flickity', Flickity);

// nuxt.config.js
plugins: [
    { src: '~/plugins/vue-flickity', ssr: false }
]

@homerjam then how do you pass flickityOptions? It gives error

Can you show the error?
Note you don't need the following in your component:
components: {
    Flickity
},

Otherwise just pass the options as seen in the example here https://github.com/drewjbartlett/vue-flickity

You guys are right. It was simple. My mistake was I used Vue.use(Flickity) not Vue.component(Flickity)
Also, I wrapped it in no-ssr to fix hydration.
Thanks all!

@iamdubx I am having the same issue, can you explain this?

Also, I wrapped it in no-ssr to fix hydration.

@danielpost there is now a no-ssr component built into nuxt. You can wrap the troublesome component with this to fix hydration (by preventing it from being rendered server side). Also not that in this case Vue.component() was required to prevent missing template error.

@homerjam Perfect, I missed that! Thank you!

@danielpost hi. personally I used this package https://github.com/egoist/vue-no-ssr
As I understand, now it's build-in Nuxt. Haven't tried it yet.
Can you please write is it working out of the box with new Nuxt?

@iamdubx Yeah, out of the box. I just wrapped the component in <no-ssr>component here</no-ssr>.

Hi guys!
I am having the same issue. My code is:
<no-ssr>
  <slick class="product-list" ref="slick" :options="slickOptions">
      ...
  </slick>
</no-ssr>

components: {
    Slick
  }

data () {
    return {
      slickOptions: {
        slidesToShow: 3,
        variableWidth: true,
        centerMode: true,
        arrows: true,
        infinite: false
      }
    }
  }

the error:
Unexpected token <
What i am going wrong?
vue-no-ssr is buil-in with nuxtjs? (v1.0.0-rc11)
Thank you

Does something like this work?
import Flickity from '~/node_modules/vue-flickity/src/flickity';

@InstanceOfMichael yes but you should still load this as a plugin with the SSR option as false. I beleive flickity expects window or dom and will break during SSR if you have that enabled.

I've tried all the recommeded solutions above but i keep getting the "Unexpected token <" error. Here is my file https://gist.github.com/jean-bertrand/65533804d52047e3f51688a6fe9992ba.
Do i have to create a separate component for flickity that I then import into the page?
Thanks!

Were you able to solve it? I am also stuck. I tried everything said above.


Compilation Failure

Version and Platform (required):

Binary Ninja Version: v4.0.491-stable#b7c8e9bf (API)
OS: Windows
OS Version: 21H2
CPU Architecture: x64

The problem
When attempting to compile a plugin (from Rust) with MSVC 14.38.33130, the following errors are thrown:
error LNK2019: unresolved external symbol BNFreeRelocationList referenced in function _ZN95_$LT$binaryninja..relocation..Relocation$u20$as$u20$binaryninja..rc..CoreOwnedArrayProvider$GT$4free17hf8de5a29268efdb7E
error LNK2019: unresolved external symbol BNGetAnalysisTypeContainer referenced in function _ZN61_$LT$binaryninja..types..Type$u20$as$u20$core..fmt..Debug$GT$3fmt17h638ee282ee98c8dfE
error LNK2019: unresolved external symbol BNFreeTypeContainer referenced in function _ZN61_$LT$binaryninja..types..Type$u20$as$u20$core..fmt..Debug$GT$3fmt17h638ee282ee98c8dfE
error LNK2019: unresolved external symbol BNFreeTypeAndNameList referenced in function _ZN100_$LT$binaryninja..types..QualifiedNameAndType$u20$as$u20$binaryninja..rc..CoreOwnedArrayProvider$GT$4free17h773c5ec21df86477E

Are you using the master branch of the API with your 4.0 stable build? If you are, please also try the most recent 4.0 stable build, which is 4.0.5336.
These are the two most likely culprits. If it's still broken, let me know and we can troubleshoot further.


Update to homekit==0.12.0

Description:
This bumps the underlying homekit_controller code to use homekit==0.12.0. It uses the new Controller API in that release. This starts to remove assumptions about IP based homekit accessories in anticipation of BLE support in a future release.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @Jc2k,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

For the future, please don't squash commits after review has started to make it easier for readers to track changes.


Adding ARM support for getcanary

As discussed in #54
Stack canary is stored at __stack_chk_guard in arm. A simple load should do the trick.
Also fixing #56 by adding libseccomp dependency in README

Thanks!


Only show-whitespace for leading or trailing whitespace

Related to #3375, it would be nice if show-whitespace could have an option (or split it into separate features, if that's how it works) to only make whitespace visible when it's at the start or end of the line, i.e. whitespace within a line is not made visible.

https://github.com/sindresorhus/refined-github/issues/3375 will be fixed and this change is not worth making it more complex and less performant. I suggest using a userscript or forking Refined GitHub (but only keep that feature)


Extract dynamic color adjusting codes to Color classes. #522

Ready for check again.

This doesn't work yet. This is how it looks like with a very light list color:

That should have been darkened.

@lemonboston dmfs/Bolts#17 has been merged

I've updated this pull request so that none of the color adjusting calculations are changed in effect, so this is only about extracting them to Color classes. Since we ran into issues, questions with the color encodings, I assumed it's better to address any actual change in separate ticket.
Squashed, rebased, ready for review.

@lemonboston please resolve conflicts

Rebased.


error in postprocessing when file extension missing

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS>. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
postprocessing doesn't work when specifying an output filename which does not include a file extension. It appears ffmpeg is unable to work out what output file type to use. Could yt-dlp be modified to pass the output file type to ffmpeg for the postprocessing stage?
Verbose log
[debug] Command-line config: ['-Uv', '--sponsorblock-remove', 'sponsor', '-f', 'worstaudio', '-o', 'blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2', 'https://www.youtube.com/watch?v=LP5z2Fkk95M']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2<IP_ADDRESS>
[debug] Lazy loading extractors is disabled
[debug] Python version 3.10.0 (CPython 64bit) - Linux-5.15.4-201.fc35.x86_64-x86_64-with-glibc2.34
[debug] exe versions: ffmpeg 4.4.1 (setts), ffprobe 4.4.1
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
yt-dlp is up to date (2<IP_ADDRESS>)
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=LP5z2Fkk95M
[youtube] LP5z2Fkk95M: Downloading webpage
[youtube] LP5z2Fkk95M: Downloading android player API JSON
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[SponsorBlock] Fetching SponsorBlock segments
[debug] SponsorBlock query: https://sponsor.ajay.app/api/skipSegments/9ff4?service=YouTube&categories=%5B%22sponsor%22%5D
[SponsorBlock] Found 2 segments in the SponsorBlock database
[info] LP5z2Fkk95M: Downloading 1 format(s): 139
[debug] Invoking downloader on "https://r1---sn-n02xgavv250-ntqe.googlevideo.com/videoplayback?expire=1638332767&ei=_6SmYfvvKrLB1AaQm6mACg&ip=2401%3Ad002%3A4b02%3Ae00%3A%3A36b&id=o-AK-qc9cp_sJ_q2xtyETEyKuyHzKz158AamWWcip7aAu_&itag=139&source=youtube&requiressl=yes&mh=AV&mm=31%2C29&mn=sn-n02xgavv250-ntqe%2Csn-hxa7zn7s&ms=au%2Crdu&mv=m&mvi=1&pcm2cms=yes&pl=40&initcwndbps=1613750&vprv=1&mime=audio%2Fmp4&gir=yes&clen=35119145&dur=5759.245&lmt=1637800674427396&mt=1638310874&fvip=1&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=6311224&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIgZX4Whrzjusx-uFWcH5jwjcNWZGZheA-8L1QQHnN2H5QCIQDoSa06Rz667vEQHB_mpr-6QwuZX2a2aDIIbaaDhNV19w%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgTiT81Q-PWwU5kllLlKXuXSQoleI4wTjOyiCaAQ9MI80CIAd5QLNn1Ls30ehyim-ebZzwVfwI8ztadRKs63eUPp5j"
[download] Destination: blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2
[download] 100% of 33.49MiB in 00:13
[FixupM4a] Correcting container of "blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2 -c copy -map 0 -dn -f mp4 file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp
[debug] ffprobe command line: ffprobe -hide_banner -show_format -show_streams -print_format json file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2
[ModifyChapters] Removing chapters from blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2
[debug] Writing concat spec to blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp.concat
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -hide_banner -nostdin -f concat -safe 0 -i file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp.concat -c copy file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp
[debug] Input #0, concat, from 'file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp.concat':
  Duration: N/A, start: 0.000000, bitrate: 48 kb/s
  Stream #0:0(eng): Audio: aac (HE-AAC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 48 kb/s
    Metadata:
      handler_name    : ISO Media file produced by Google Inc.
      vendor_id       : [0][0][0][0]
[NULL @ 0x5631f2e52080] Unable to find a suitable output format for 'file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp'
file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp: Invalid argument
ERROR: Postprocessing: file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp: Invalid argument
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 2919, in process_info
    info_dict = self.post_process(dl_filename, info_dict, files_to_move)
  File "/usr/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 3078, in post_process
    info = self.run_pp(pp, info)
  File "/usr/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 3021, in run_pp
    files_to_delete, infodict = pp.run(infodict)
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/common.py", line 22, in run
    ret = func(self, info, *args, **kwargs)
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/common.py", line 112, in wrapper
    return func(self, info)
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/modify_chapters.py", line 64, in run
    in_out_files = [remove_chapters(info['filepath'], False)]
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/modify_chapters.py", line 62, in remove_chapters
    return file, self.remove_chapters(file, cuts, concat_opts, self._force_keyframes and not is_sub)
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/modify_chapters.py", line 321, in remove_chapters
    self.concat_files([in_file] * len(concat_opts), out_file, concat_opts)
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 364, in concat_files
    self.real_run_ffmpeg(
  File "/usr/lib/python3.10/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 307, in real_run_ffmpeg
    raise FFmpegPostProcessorError(stderr.split('\n')[-1])
yt_dlp.postprocessor.ffmpeg.FFmpegPostProcessorError: file:blah/ytdl-21d3ebf6b5cdefa815234a825abf0cf2.temp: Invalid argument

A more sensible solution is imo to force the correct file extension irrespective of user-provided one. This is what is already done in Merger

I worked around my issue by adding .%(ext)s to the output filename. @pukkandan as you say it might be better to force correct file extension sent to ffmpeg.

I had this problem with the FFmpegEmbedSubtitle postprocessor.
Using the file extension to communicate the output container type sure is an interesting design choice. I fixed this by setting the container field in the format dict to the detected output ext (or container format). This is then used to set the ffmpeg -f  argument, so that the format does not depend on the output file name.
Here's a patch:
diff --git a/yt_dlp/YoutubeDL.py b/yt_dlp/YoutubeDL.py
index 7ba6441e1..2cd212e30 100644
--- a/yt_dlp/YoutubeDL.py
+++ b/yt_dlp/YoutubeDL.py
@@ -2550,6 +2550,8 @@ def is_wellformed(f):
                     format['format_id'] = '%s-%d' % (format_id, i)
                 if format.get('ext') is None:
                     format['ext'] = determine_ext(format['url']).lower()
+                if format.get('container') is None:
+                        format['container'] = format.get('ext')
                 # Ensure there is no conflict between id and ext in format selection
                 # See https://github.com/yt-dlp/yt-dlp/issues/1282
                 if format['format_id'] != format['ext'] and format['format_id'] in common_exts:
diff --git a/yt_dlp/postprocessor/ffmpeg.py b/yt_dlp/postprocessor/ffmpeg.py
index a726a962f..c8bd3fc2f 100644
--- a/yt_dlp/postprocessor/ffmpeg.py
+++ b/yt_dlp/postprocessor/ffmpeg.py
@@ -579,7 +579,9 @@ def __init__(self, downloader=None, already_have_subtitle=False):

     @PostProcessor._restrict_to(images=False)
     def run(self, info):
-        if info['ext'] not in self.SUPPORTED_EXTS:
+        # TODO: Any other container format string fixes necessary?
+        cont = info['container'].replace('_dash', '')
+        if cont not in self.SUPPORTED_EXTS:
             self.to_screen(f'Subtitles can only be embedded in {", ".join(self.SUPPORTED_EXTS)} files')
             return [], info
         subtitles = info.get('requested_subtitles')
@@ -599,7 +601,6 @@ def run(self, info):
             return [], info
         '''

-        ext = info['ext']
         sub_langs, sub_names, sub_filenames = [], [], []
         webm_vtt_warn = False
         mp4_ass_warn = False
@@ -611,15 +612,15 @@ def run(self, info):
             sub_ext = sub_info['ext']
             if sub_ext == 'json':
                 self.report_warning('JSON subtitles cannot be embedded')
-            elif ext != 'webm' or ext == 'webm' and sub_ext == 'vtt':
+            elif cont != 'webm' or (cont == 'webm' and sub_ext == 'vtt'):
                 sub_langs.append(lang)
                 sub_names.append(sub_info.get('name'))
                 sub_filenames.append(sub_info['filepath'])
             else:
-                if not webm_vtt_warn and ext == 'webm' and sub_ext != 'vtt':
+                if not webm_vtt_warn and cont == 'webm' and sub_ext != 'vtt':
                     webm_vtt_warn = True
                     self.report_warning('Only WebVTT subtitles can be embedded in webm files')
-            if not mp4_ass_warn and ext == 'mp4' and sub_ext == 'ass':
+            if not mp4_ass_warn and cont == 'mp4' and sub_ext == 'ass':
                 mp4_ass_warn = True
                 self.report_warning('ASS subtitles cannot be properly embedded in mp4 files; expect issues')

@@ -629,10 +630,11 @@ def run(self, info):
         input_files = [filename] + sub_filenames

         opts = [
-            *self.stream_copy_opts(ext=info['ext']),
+            *self.stream_copy_opts(ext=cont),
             # Don't copy the existing subtitles, we may be running the
             # postprocessor a second time
             '-map', '-0:s',
+            '-f', EXT_TO_OUT_FORMATS.get(cont, cont),
         ]
         for i, (lang, name) in enumerate(zip(sub_langs, sub_names)):
             opts.extend(['-map', '%d:0' % (i + 1)])

Something like this could probably be used to fix the other ffmpeg postprocessors as well. Should I make a PR, or are there any problems with this approach?

We want to avoid the use of -f and let ffmpeg determine it automatically where possible

In the case of user supplied file name it obviously does not work automatically. At least in the case where ext is not a known format, using -f would be useful as a fallback.

ext is not the user-provided file extension though. It is the extension as determined by the extractor. Not sure what you are trying to achieve by copying it into container

+            '-f', EXT_TO_OUT_FORMATS.get(info['ext'], info['ext']),

I got that. My point is that only this part your patch is needed to fix it. The rest is redundant.
But anyway, why are you hardcoding extension in outtmpl instead of just using %(ext)s?

Let's say that my output file name is hardcoded (or it would be a pain to change it). This line makes the ext non-sensical, if the file name in outtmpl  does not have a (sane) extension and the output file exists and overwrites is False. I happened to have this situation (the file is created before ytdl call to monitor it during download).
I'm thinking how (and if) this should be fixed, and I would appreciate your input.
This patch fixes FFmpegEmbedSubtitlePP if using -f is not recommended:
diff --git a/yt_dlp/postprocessor/ffmpeg.py b/yt_dlp/postprocessor/ffmpeg.py
index d490f13a4..335f1dc72 100644
--- a/yt_dlp/postprocessor/ffmpeg.py
+++ b/yt_dlp/postprocessor/ffmpeg.py
@@ -644,7 +644,7 @@ def run(self, info):
                 opts.extend(['-metadata:s:s:%d' % i, 'handler_name=%s' % name,
                              '-metadata:s:s:%d' % i, 'title=%s' % name])

-        temp_filename = prepend_extension(filename, 'temp')
+        temp_filename = prepend_extension(filename, 'temp') + '.' + ext
         self.to_screen('Embedding subtitles in "%s"' % filename)
         self.run_ffmpeg_multiple_files(input_files, temp_filename, opts)
         os.replace(temp_filename, filename)

With https://github.com/yt-dlp/yt-dlp/commit/7aaf4cd2a8fd8ecf2123b981782c3d12dce80d78, there is now a note in the docs about the filename specified by -o being possibly altered by post-processing. I still view it as surprising that -o output filename is maybe not the actual output filename and I am not aware of any other command line tool that behaves like this. However, with the added note to the docs, I think it's fair play. :+1:
For completeness, I want to include an idea for a possible future option, from #5471, that I don't see mentioned here in this issue thread, yet:

Option --force-filename which renames the file after post-processing to whatever -o said in the first place (because if the user wants to have an output filename without extension or a wrong extension, why not allow it?)

This could be implemented by a simple file rename/move from whatever the filepath currently is (after post-processing, what --print after_move:filepath would print) to whatever filepath has been originally specified by -o.


refactor: migrate legacy decorators to ES decorators

Adding support for ESM resulted in some issues with the legacy decorators in transpiled TS code. Unfortunately, field decorators can no longer be used for reflection purposes with ES decorators, so they had to be moved to class decorators.

@zwave-js-bot automerge


Update the spec based on recent HostEnsureCanCompileStrings changes

https://github.com/tc39/ecma262/pull/3222 - Updated the HostEnsureCanCompileStrings definition
Rendered version: https://specs.lukewarlow.dev/dynamic-code-brand-checks/

Related TT spec PR https://github.com/w3c/trusted-types/pull/445


command not found: bunx

Describe the bug
When install bun following the official installer curl -fsSL https://bun.sh/install | bash, bunx cli is also installed.
However when installing bun via mise, bunx is missing.
To Reproduce
$ mise use -g bun@latest
$ bunx
zsh: command not found: bunx

Expected behavior
bunx command executable.
mise doctor output
mise version:
  2024.2.16 macos-arm64 (2024-02-15)

build:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Thu, 15 Feb 2024 02:24:55 +0000
  Rust Version: rustc 1.75.0 (82e1608df 2023-12-21) (Homebrew)
  Profile: release

shell:
  /bin/zsh
  zsh 5.9 (x86_64-apple-darwin23.0)

mise dirs:
  data: /Users/user/.local/share/mise
  config: /Users/user/.config/mise
  cache: /Users/user/Library/Caches/mise
  state: /Users/user/.local/state/mise
  shims: /Users/user/.local/share/mise/shims

mise environment variables:
  MISE_SHELL=zsh

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cargo_binstall = true
  color = true
  disable_default_shorthands = false
  disable_tools = []
  experimental = false
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  node_compile = false
  not_found_auto_install = true
  paranoid = false
  plugin_autoupdate_last_check_duration = "7d"
  python_compile = false
  python_default_packages_file = "/Users/user/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  python_venv_auto_create = false
  raw = false
  trusted_config_paths = []
  quiet = false
  verbose = false
  yes = false
  ci = false
  debug = false
  trace = false
  log_level = "info"

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

activated: yes
shims_on_path: no
config files:
  ~/.tool-versions
  ~/.config/mise/config.toml

plugins:
  bun      (core)
  deno     (core)
  go       (core)
  java     (core)
  node     (core)
  python   (core)
  ruby     (core)

toolset:
<EMAIL_ADDRESS>bun@latest, <EMAIL_ADDRESS><EMAIL_ADDRESS>
No problems found

Additional context
Add any other context about the problem here.

same for me

I don't have much availability right now. Someone else should figure out what is going on here and recommend a fix.

Some context:

bunx is "an alias for bun x" but also a CLI https://bun.sh/docs/cli/bunx
bunx is not that new, looks like it was introduced at the end of 2022 https://twitter.com/jarredsumner/status/1606163655527059458
Here's where we install bun https://github.com/jdx/mise/blob/bf8ff551d066b6a2c976687f6862cb7e2d50f231/src/plugins/core/bun.rs#L76-L91

Will throw something out for this

Before my change lands, you can fix this early with:
ln -s "$(which bun)" "$(dirname "$(which bun)")/bunx"

(in a posix-y shell, with mise active and bun installed)

The fix will be available in the next release, so 2024.2.17 or later.


Feed images

During tracking #140 , it was raised that ddev.com 's feed images were not appearing in feedly (rss reader).

The top-level image also appears to be using a default image (same as "Matt Glaman" and "PHP" feeds for reference):

We should investigate if this is a feedly issue or an issue with our feed.

The feed is valid, but we probably need to make updates to appease Feedly specifically.
The site’s favicon does not exist at the root (ddev.com/favicon.ico), for example, and I’d bet that’s exactly and only what Feedly is looking for. Similarly with those images, I assume Feedly is looking for a summary image we don’t have in place.

It turns out Feedly does have some automatic and not-so-automatic things it looks for. Just merged a PR that should now include images and an icon.

Thank you @mattstein
Feed icon is now displaying

I'm not seeing images for feed items. Not sure if this is a caching issue, or if they will start appearing with new items.

I checked the feed on https://rss.app/ (online RSS reader) and images display there so the URL is right.

I wasn’t seeing the icon yet @tyler36, so I guess that’s progress! 🤞

There was a new item in DDEV feed but the image did not show up in Feedly.

The image source does not appear valid: https://ddev.comhttps://www.drupal.org/files/project-images/ama-live-updates.gif

I thought feedly might be doing something so I checked it in https://rss.app/ and the image appeared broken.

This walkthrough is based on the blog post Acquia Migrate: Accelerate — now open source! by Wim Leers.

Checking the raw feed: https://rss.app/feeds/GwRYLvc0Z8ezl2yl.xml

Thanks for catching that, @tyler36!
This is because the featureImage in this new post is using a hotlinked, absolute image and not a local one.
I’d recommend bringing the image into ddev.com for longevity, but I also fixed the feed generator so absolute feature image URLs won’t be a problem there. This should be reflected in the feed momentarily.

Images are appearing with the recently published content ...

Thank you.

We did it! Thanks again for pointing this out and checking up on it, @tyler36.


Microsoft.ML.OnnxRuntime.OnnxRuntimeException When changing Width and Height properties of a custom model.

Link to onnx model and test image: https://drive.google.com/file/d/12MVcF0Ir0sEnNjmKHKjDnPItvEw6l9mi/view?usp=sharing
Model works fine with detect.py, although the bounding box isn't as wide as it should be in the provided test.jpg (fixed that problem by training at a higher res, but haven't converted that .pt file to .onnx yet).
Unhandled exception. Microsoft.ML.OnnxRuntime.OnnxRuntimeException: [ErrorCode:InvalidArgument] Got invalid di
mensions for input: images for the following indices
 index: 2 Got: 320 Expected: 640
 index: 3 Got: 320 Expected: 640
 Please fix either the inputs or the model.
   at Microsoft.ML.OnnxRuntime.NativeApiStatus.VerifySuccess(IntPtr nativeStatus)
   at Microsoft.ML.OnnxRuntime.InferenceSession.RunImpl(RunOptions options, IntPtr[] inputNames, IntPtr[] inpu
tValues, IntPtr[] outputNames, DisposableList`1 cleanupList)
   at Microsoft.ML.OnnxRuntime.InferenceSession.Run(IReadOnlyCollection`1 inputs, IReadOnlyCollection`1 output
Names, RunOptions options)
   at Microsoft.ML.OnnxRuntime.InferenceSession.Run(IReadOnlyCollection`1 inputs, IReadOnlyCollection`1 output
Names)
   at Microsoft.ML.OnnxRuntime.InferenceSession.Run(IReadOnlyCollection`1 inputs)
   at Yolov5Net.Scorer.YoloScorer`1.Inference(Image image) in C:\Users\david\repos\yolov5-net\src\Yolov5Net.Sc
orer\YoloScorer.cs:line 131
   at Yolov5Net.Scorer.YoloScorer`1.Predict(Image image) in C:\Users\david\repos\yolov5-net\src\Yolov5Net.Scor
er\YoloScorer.cs:line 310
   at Yolov5Net.App.Program.Main(String[] args) in C:\Users\david\repos\yolov5-net\src\Yolov5Net.App\Program.c
s:line 17

Process finished with exit code -532,462,766.

And here is my class:
public class YoloCrosswalksModel : YoloModel
{
    public override int Width { get; set; } = 320;
    public override int Height { get; set; } = 320;
    public override int Depth { get; set; } = 3;

    public override int Dimensions { get; set; } = 85;

    public override float[] Strides { get; set; } = new float[] { 8, 16, 32 };

    public override float[][][] Anchors { get; set; } = new float[][][]
    {
        new float[][] { new float[] { 010, 13 }, new float[] { 016, 030 }, new float[] { 033, 023 } },
        new float[][] { new float[] { 030, 61 }, new float[] { 062, 045 }, new float[] { 059, 119 } },
        new float[][] { new float[] { 116, 90 }, new float[] { 156, 198 }, new float[] { 373, 326 } }
    };

    public override int[] Shapes { get; set; } = new int[] { 80, 40, 20 };

    public override float Confidence { get; set; } = 0.20f;
    public override float MulConfidence { get; set; } = 0.20f;
    public override float Overlap { get; set; } = 0.45f;

    public override string[] Outputs { get; set; } = new[] { "output" };

    public override List<YoloLabel> Labels { get; set; } = new List<YoloLabel>()
    {
        new YoloLabel { Id = 1, Name = "crosswalks" },
    };

    public override bool UseDetect { get; set; } = true;

    public YoloCrosswalksModel()
    {

    }
}

@dstampher
inference:
using var image = Image.FromFile("Assets/crosswalk.jpg");
using var scorer = new YoloScorer<CrosswalksModel>("Assets/Weights/crosswalks.onnx");
List<YoloPrediction> predictions = scorer.Predict(image);

model config:
public class CrosswalksModel : YoloModel
    {
        public override int Width { get; set; } = 640;
        public override int Height { get; set; } = 640;
        public override int Depth { get; set; } = 3;

        public override int Dimensions { get; set; } = 6;

        public override float[] Strides { get; set; } = new float[] { 8, 16, 32 };

        public override float[][][] Anchors { get; set; } = new float[][][]
        {
            new float[][] { new float[] { 010, 13 }, new float[] { 016, 030 }, new float[] { 033, 023 } },
            new float[][] { new float[] { 030, 61 }, new float[] { 062, 045 }, new float[] { 059, 119 } },
            new float[][] { new float[] { 116, 90 }, new float[] { 156, 198 }, new float[] { 373, 326 } }
        };

        public override int[] Shapes { get; set; } = new int[] { 80, 40, 20 };

        public override float Confidence { get; set; } = 0.20f;
        public override float MulConfidence { get; set; } = 0.25f;
        public override float Overlap { get; set; } = 0.45f;

        public override string[] Outputs { get; set; } = new[] { "output" };

        public override List<YoloLabel> Labels { get; set; } = new List<YoloLabel>()
        {
            new YoloLabel { Id = 0, Name = "crosswalk" }
        };

        public override bool UseDetect { get; set; } = true;

        public CrosswalksModel()
        {

        }
    }

result:

@dstampher please update NuGet package or clone latest version.

@kkarahainko Using newest version, it works for inference now, same exception happens when I change width/height to 320 however.  My dataset images were all 300x300 so I trained at 320x320.
Still works at 640x640 for inference, but this inability to change width/height might be a problem for other users of this repo.

@dstampher not sure what do you mean, you've trained your model with --img 640, inspect your onnx file with netron.app to be sure, you also may run Predict with any size of images. no need to change w.h to 320 In model config C# class.

@kkarahainko Ah yes, sorry about that.  My 320x320 training was a previous run.  Note to self:  don't touch width/height in my class that inherits from YoloModel.  Got it.

@dstampher no problem, you may touch it ) it depends on what size you training, for P6 models it may be 1024 for example etc, for P5 up to 640 the best option.


Upgrade jest

Upgrading Jest to support step debugging in Webstorm. As part of this work, the babel runtime polyfill for promises and esmodules is no longer used.

Can you update the title to follow the commit format please? https://github.com/onflow/fcl-js/blob/master/CONTRIBUTING.md

I'm getting this


Make the go cli optional by default

This disables by default the build of brooklyn-client. It can be reenabled by doing:
mvn clean install -Dclient

This is to fix the inconsistency pointed out in this email thread<EMAIL_ADDRESS>

LGTM


implement ON DUPLICATE KEY UPDATE for MySQL

I've chosen to have a similar syntax to .set for .on_duplicate_key_update to keep updating values consistent
there's a new Values wrapper to represent the VALUES term in the new clause

Coverage decreased (-0.1%) to 97.811% when pulling 2438b9bd163f6e6cad1de88fc547bffa6322d949 on inkmix:implement-on-duplicate-key-update into eeea26c0d4d2e1a162cd45ae9e2484d61cd9d585 on kayak:master.


searching on certain fields returns no matches

I have Chewy setup with very basic search index defined
class SearchIndex < Chewy::Index

define_type Post do
   field :title
   field :description
   field :rating
   field :published_date

end
Searching by rating I can get matches. Searching on title, even when using exact title I get no matches returned
(term: {title: 'yada..'}) nothing
(term: {id: '2'}) bam, match!
Search by the record's ID though and I get a match. What am I missing that is causing this issue?

Are you using filters for searching or queries? This is about ES knowledge, not chewy actually. Read how do ES works carefully


Weird Markdown preview

Hello,
After upgrading to latest version of glow.nvim (no idea from which version I jumped to latest), highlighting is weird red on my neovim, see attached screenshot.

Setup information:

Neovim v0.8.2
Ubuntu 22.02
Terminal emulator terminator, but same effect happens in gnome-terminal

Further notes:

Glow does work perfectly when invoked directly via terminal (1.4.1 (4863f57))
glow.nvim markdown preview panel is working, able to scroll, quit, etc.

My init.vim config can be found here
Any suggestions on this one, please?
Kind Regards!

Turns out it was due to my trailing whitespace highlighting:
highlight ExtraWhitespace ctermbg=red guibg=red
But this wasn't causing any issues in the previous versions.

The workaround for me was simply replacing the trailing whitespace method in my init.vim file:
https://github.com/attilarepka/neovim-config/commit/56023dc7775c15914766ce40795c202530d3771c
Maybe a hint for others, if they have similar issues.

cool, thanks for sharing


Update pod-lifecycle.md

Relaled to #24444
/language zh

Please do NOT update a page by tracking PRs made to the upstream English page. It will be an exhaustive work and difficult to automate. Instead, please use the following command to identify changes that need to be sync'ed:
./scripts/lsync.sh content/zh/docs/concepts/workloads/pods/pod-lifecycle.md

/hold


Code editor isn't rendering in the test.

Please describe the task that needs to be done
It appears that current code editor isn't really rendered. This might be actually an issue with Monaco Jest or missing some dependencies or mocks.
This is current render of SourceCodeEditor component in the test:
body>
  <div>
    <textarea />
  </div>
</body>

@lordrip is this still happening? How can I check?

Yes, it's still happening. This is because the monaco editor is a bit complex to run it through the JSDom environment.
We're mocking the whole component here
In order to fix this, the idea is to split the buttons from the code editor itself, so we can at least check the interaction between Kaoto and the code editor.


leaderbord

im getting this error when i add score on leaderboard
You must enter a score. UnityEngine.Debug:LogWarning (object) GameJolt.API.LogHelper:Warning (string,object[]) (at Assets/Plugins/GameJolt/Scripts/API/LogHelper.cs:32) GameJolt.API.Core.Response:.ctor (UnityEngine.Networking.UnityWebRequest,GameJolt.API.Core.ResponseFormat) (at Assets/Plugins/GameJolt/Scripts/API/Core/Response.cs:118) GameJolt.API.GameJoltAPI/<GetRequest>d__17:MoveNext () (at Assets/Plugins/GameJolt/Scripts/API/GameJoltAPI.cs:108) UnityEngine.GUIUtility:ProcessEvent (int,intptr,bool&)

well, as the error says: you must enter a score... 😉
GameJolt requires you to enter a numeric score value (used for actual sorting/ranking) and also a textual representation of that score, which is shown on GameJolt.
So I guess you just did something like: Scores.Add(value, ""), but the string must not be empty, so instead you should do:
Scores.Add(value, value.ToString())
or Scores.Add(value, $"{value} Points")

well, as the error says: you must enter a score... 😉 GameJolt requires you to enter a numeric score value (used for actual sorting/ranking) and also a textual representation of that score, which is shown on GameJolt. So I guess you just did something like: Scores.Add(value, ""), but the string must not be empty, so instead you should do: Scores.Add(value, value.ToString()) or Scores.Add(value, $"{value} Points")

i made this
Scores.Add(score, scoreText.text, 656734);

I guess scoreText.text is simply empty

i made this on Update
scoreText.text = "score: " + score;

put a debug output right before you call Scores.Add to see what's actually used:
Debug.Log(scoreText.text);
Scores.Add(Score, scoreText.text, 656734);

put a debug output right before you call Scores.Add to see what's actually used:
Debug.Log(scoreText.text);
Scores.Add(Score, scoreText.text, 656734);

don't appear anything on console

allright i figured out the error was on trigger

thanks for your help anyway

yeah no problem, you're welcome


Lost edits

Windows 10/IDE 2.4.2
IDE crapped out. When restarted, local edits lost.

Hey @jcpny1—thanks for reporting.
We'd need to know more what you mean by "local edits" in order to diagnose this further.
The IDE, generally, will not persist "local edits". It is designed to treat the remote server you are connected as the source of truth. So unless changes are saved (which happens on the server first) when you attempt to save a file, they will usually be lost.
The exception to this is that Atom itself does some work to recover buffers that have not been written to the local disk. I'm not sure of the details with that though, so I can't so for sure if/how/when it would catch something like this.
Also, if a local file that is managed by the IDE is edited from outside of the IDE, those edits will be overwritten when the IDE is launched. Again, the server is taken to be the source of truth.

By local edits, I meant local edits that I made using the IDE while on my computer. I suppose 'local' was unnecessary. I suspect the problem is unreliable server service as I've had quite a few problems in the past. Most recently, when I Ctrl-S save a file, a blue circle still appears on the file tab as if the file had not been saved. When I go to close the tab, it asks me if I want to save changes, even though the changes had already been saved (as noted when re-executing the file to see the expected new behavior). Other current problems are: expanding the width of the tree view causes the console to not be formatted properly, and when the IDE is first opened, the console width seems to be set at about 35 characters and cannot be changed without doing an IDE reconnect.

Also, the Help->Report Issue link takes you to the Atom github page, which would not be the correct place to report an issue.


Question about React Native license

This is a simple question, just looking for clarification.
I know about facebook/react#10191 - but comments are locked for non-collaborators on that issue. I heard the good news today about React being relicensed under MIT.
Does React Native fall under that umbrella? Or will React Native stay on the same BSD + Patent license? Not looking for a debate, just an answer. :)
Thanks and keep up the great work RN team!

I don't speak for Facebook so someone who works there should close this but one of the engineering directors clarified that for now RN will keep its current license: https://twitter.com/dmwlff/status/911348886882607104.

I hope RN could be relicensed.

Thanks @ide

RN is so great! I also hope RN could be relicensed.

Hope RN could be relicensed.

no zuo no die why you try

I'm waiting for the license to change...

@shengxinjing This is the act of the software license agreement, is the shame of things. I have abandoned react-native, too insecure！

Locking this since license discussions get too heated :)

React Native (including Metro, Fresco, and Yoga) has just been relicensed as MIT.
https://github.com/facebook/react-native/commit/26684cf3adf4094eb6c405d345a75bf8c7c0bf88


RaftRouter does not have to use tokio::sync::RwLock for routing_table and isolated_nodes

RaftRouter does not have to use tokio::sync::RwLock for routing_table and isolated_nodes.
std::sync::Mutex is good enough.
routing_table and isolated_nodes do not have to hold a lock across await.
We can just: acquire a lock(), clone the Raft or Arc< StoreWithDefensive> then release the lock.
https://github.com/datafuselabs/openraft/blob/4ce50a3674732e41feafb0115f37c38632c52ca3/openraft/tests/fixtures/mod.rs#L123-L129

/assignme


Setup a linter

Setup a linter with its configuration and make sure the linting happens automatically on each push

https://simpleisbetterthancomplex.com/packages/2016/08/05/flake8.html


Mbit_IR is undefined MakeCode IDE

Get error when using
Mbit_IR.onPressEvent(RemoteButton.Up, function () {
})
MakeCode says: Cannot read property "onPressEvent" of undefined
And
Mbit_IR.init(Pins.P0)
basic.forever(function () {
})
MakeCode says: Cannot read property "init" of undefined

same bug. Do you solve the problem?

Use the makecode version that is on Yahboom's website, attached link.

You can also use the makecode version that is on Yahboom's website, attached link.

If it doesn't work for you, check out Yahboom's FAQ, attached [link.]((http://www.yahboom.net/study/Building_bit) (You need to go down)