[enhancement] - Fetching text from mulitple top nodes/divs

The current flow of text extraction seems to assume , that there is a single div or parent div which contains the entire wanted text data .
And hence for those sites which this is the case(generally most sites ) the Top_node logic pretty good.
But this logic fails on articles where there is wanted text on multiple different divs  , maybe separated by a ad div or some image/graph div -- in  that case goose will choose the best of all those divs and extract the top node and its text , even if that text is from center or end of the article ,-- at which point the text doesn't make any sense without the context of the above texts which were excluded .
See for example - https://t.co/ch74W7Rjke
you see here that one paragraph is from the itemprop = articleBody   tag , and others are from figcaption tafg which is actually the part of the news,
now if i add figcaption tag in the known_article_content_tags it will work fine , but in that case it will only fetch one paragraph from the center of the page .
In this case what is needed is a way to add multiple top nodes considering the possible wanted texts - combine them then clean and display the text in them ,
or someway to skip the top node section all together and directly clean the text if its retrieved from known_article_ content_tags .

A quick pointer here, @taherkhalil:
If you want to go into more sophisticated extraction, have a look at the https://github.com/dragnet-org/dragnet project. They do ML-based identification of relevant nodes. This means you can add your own training material if you're not satisfied with the performance for a specific site.

Thanks @trifle, just another example: https://www.theatlantic.com/magazine/archive/1982/02/have-you-ever-tried-to-sell-a-diamond/304575/. Probably makes sense to add workaround for such cases? I mean to calculate what percentage of visible/readable text is collected by current approach and try to use another technique in case if [captured sentences] / [all sentences] ratio is less than 0.25, for example?


Change endpoint url issue#73

What this PR does / why we need it: Change endpoint URL
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #73
Special notes for your reviewer:
Release note:

Codecov Report

Merging #75 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   39.12%   39.16%   +0.04%
==========================================
  Files          13       13
  Lines         887      886       -1
  Branches       91       91
==========================================
  Hits          347      347
+ Misses        517      516       -1
  Partials       23       23

Impacted Files
Coverage Δ

orchestration/api/services.py
17.02% <ø> (ø)
:arrow_up:

orchestration/api/instances.py
22.06% <100%> (+0.1%)
:arrow_up:

@kumarashit , @joseph-v - please hold this merge, till we get confirmation on the solution trial by @wisererik

it's not a issue, will close it.

handles the issue with a proper handling in PR #117 (dashboard)


[SPARK-25945][SQL] Support locale while parsing date/timestamp from CSV/JSON

What changes were proposed in this pull request?
In the PR, I propose to add new option locale into CSVOptions/JSONOptions to make parsing date/timestamps in local languages possible. Currently the locale is hard coded to Locale.US.
How was this patch tested?
Added two tests for parsing a date from CSV/JSON - ноя 2018.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #98489 has started for PR 22951 at commit 41154bd.

I will update docs soon.

Test build #98489 has finished for PR 22951 at commit 41154bd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98489/
Test PASSed.

@HyukjinKwon @dongjoon-hyun Please, review the changes.

Test build #98507 has started for PR 22951 at commit 93da760.

Test build #98507 has finished for PR 22951 at commit 93da760.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98507/
Test PASSed.

Looks good. I or someone else should take a closer look before getting this in.

Could you rebase this once again, @MaxGekk ?

Test build #98541 has started for PR 22951 at commit 6ab8501.

Test build #98541 has finished for PR 22951 at commit 6ab8501.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98541/
Test FAILed.

Test build #98543 has started for PR 22951 at commit 6ab8501.

Test build #98543 has finished for PR 22951 at commit 6ab8501.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98543/
Test PASSed.

Test build #98567 has started for PR 22951 at commit 759bca6.

Test build #98567 has finished for PR 22951 at commit 759bca6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98567/
Test PASSed.

Could you take a look once more, @HyukjinKwon ?

OMG, what does ноя 2018 mean BTW? haha

It is 3 letters prefix of Ноябрь which is November in Russian.

Test build #98583 has started for PR 22951 at commit 8834b4b.

Test build #98583 has finished for PR 22951 at commit 8834b4b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98583/
Test FAILed.

jenkins, retest this, please

Test build #98587 has started for PR 22951 at commit 8834b4b.

Test build #98587 has finished for PR 22951 at commit 8834b4b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98587/
Test FAILed.

jenkins, retest this, please

Test build #98591 has started for PR 22951 at commit 8834b4b.

Test build #98591 has finished for PR 22951 at commit 8834b4b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98591/
Test FAILed.

retest this please

Test build #98598 has started for PR 22951 at commit 8834b4b.

Test build #98598 has finished for PR 22951 at commit 8834b4b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98598/
Test PASSed.

Merged to master.

Actually let me leave a cc for @srowen. I remember we talked about it before.


Code debug instructions

I came here from the LERF project. It is the awesome example of how to build a new method using Nerfstudio. However for me as a newbie it is unclear how to debug such a project (for example how to debug ns-train command).
Maybe you can share some instructions on how you debug nerfstudio project with VSCode breakpoints and stuff.

You should just be able to add breakpoints. Just make sure that nerfstudio is installed with pip install -e .

I encountered unexpected behavior when debugging LeRF in VSCode. I tried to dive inside get_outputs_for_camera_ray_bundle function and python interpriter returned such warnings
PYDEV DEBUGGER WARNING:
sys.settrace() should not be used when the debugger is being used.
This may cause the debugger to stop working correctly.
If this is needed, please check:
http://pydev.blogspot.com/2007/06/why-cant-pydev-debugger-work-with.html
to see how to restore the debug tracing back correctly.
Call Location:
  File "/home/user/nerfstudio/nerfstudio/viewer/server/viewer_utils.py", line 63, in __enter__
    sys.settrace(self.func)
PYDEV DEBUGGER WARNING:
sys.settrace() should not be used when the debugger is being used.
This may cause the debugger to stop working correctly.
If this is needed, please check:
http://pydev.blogspot.com/2007/06/why-cant-pydev-debugger-work-with.html
to see how to restore the debug tracing back correctly.
Call Location:
  File "/home/user/nerfstudio/nerfstudio/viewer/server/viewer_utils.py", line 67, in __exit__
    sys.settrace(None)

At the end, VSCode was unable to debug and ignored breakpoints. I tried to comment settracce inside SetTrace class. This hack allowed me to debug the function, but broke Nerfstudio GUI toolbars. I don't want to bother you with such an unurgent issue, but I would appreciate it if you could fix this problem.

Have you solved the problem? I met a similar problem when diving inside tyro.cli(Commands).main() in the process_data.py. VSCode was unable to debug and ignored breakpoints too.

Hi @anas-zafar
Try these:

Install nerfstudio with pip install -e . (to allow python to see your local changes in the nerfstudio lib code)
Comment 63 line and change 67 line to "pass" inside SetTrace class
Use VScode launch configuration mentioned above to debug code

Hope this will help, but do not forget - this was just my fast hack to overcome a particular warning that caused debugger to stop working correctly.

You should also add: "justMyCode": false, in order to include breakpoints.
Complete example for me looks like:
{
"version": "0.2.0",
"configurations": [
{
"name": "Python: Current File with Arguments",
"type": "python",
"request": "launch",
"program": "scripts\train.py",
"console": "integratedTerminal",
"args": ["nerfacto", "--data", "some_path"],
"justMyCode": false,
}
]
}


chore(main): release 0.6.2

:robot: I have created a release beep boop
0.6.2 (2023-11-25)
Bug Fixes

core takes care of correct unmanaged msg type (27e36d3)
move the managed flag inside DeviceData::Controllable (6bd7740)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/FruitieX/homectl-server/releases/tag/v0.6.2 :sunflower:


fix context bug #1802

@dwightguth please review.

one minor thing but feel free to merge once you fix it.


"Slots" should appear on the right-hand-side of a rule

where you will be able to place squares

Done!


Fix for low performance on Android (10s+) - majirosstefan

I added MLKit dependency, so QR detection is done immediately + it should support bar code detection.

If no qr code in image, IndexOutOfBoundsException. Need add exception or check isEmpty rawValues


may Blazor be blessed ,and JavaScript die!

welll , i can't be thankful enough for Blazor.

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

ID: 5bc28995-6784-a7d2-5c0b-9811a8d19c1a
Version Independent ID: 1668c8fb-024c-c86d-d4d5-e22e3355c04b
Content: Introduction to Blazor in ASP.NET Core
Content Source: aspnetcore/blazor/index.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @EKanadily! :rocket:


Class 'MongoDB\Driver\Manager' not found in homestead

I install laravel and mongodb by homestead. When I register account then error Class 'MongoDB\Driver\Manager' not found is displayed. Please help me to fix it. Thank you so much

http://laradock.io/ is a better solution for local stacks.
you may need to add the mongodb extension to your php and reload php.

Hello,
See this comment how to install mongodb with homestead
https://github.com/jenssegers/laravel-mongodb/issues/1163#issuecomment-391971560
Thanks.


Global tests

[ ] Grasshopper
[ ] Dynamo

@xRadne  @JRiad  So last case then we can release. Could you help me out with either of the global tests?


Bump Azure Storage version to avoid NSP warning

nsp complains about dependencies of the currently used version of azure-storage. Bumping to ^1.3.0 addresses this.
Even though this bumps through a major version range looking at the JSDoc for azure-storage suggests the API for the table service hasn’t changed.
cc @jpoon

Thanks @wrumsby.
Also took this opportunity to update the dev dependencies. Pushed a new npm version with the changes.


Android crashes on taking photo

Issue Description
java.lang.Boolean cannot be cast to com.facebook.react.brigde.ReadableNativeMap

Steps to Reproduce / Code Snippets / Screenshots
Use CameraKitCameraScreen and try to take a photo (just tab the button)
(Is also happening if using CameraKitCamera and try to use camera.capture())
On iOS everything works fine.

Environment

React Native Navigation version: –
React Native version: 0.58.6
Platform(s) (iOS, Android, or both?): Android
Device info (Simulator/Device? OS version? Debug/Release?): both
React Native Camera Kit version: 7.3.6 (With the newer version I'm running into #258)

I am getting same error.
Did you find any solution for this?

me also has the same any fix?

After removing the react-native-camera package, it works

thanks @aniketcgi

@aniketcgi i dont have react-native-camera package but still i am getting the error.

@shukla12manish I was doing this:
const image = await cameraRef.current.capture({
  saveToCameraRoll: false,
});

Instead of this:
const image = await cameraRef.current.capture(false);

https://github.com/teslamotors/react-native-camera-kit/issues/259#issuecomment-507536910
How can we use both lib together?

@ShokiiCookiie Can you help me with this? Any workaround? I have tried renaming the files which are the same. but couldn't able to fix it.


Align outbound content-type to the payload type

If the outbound payload type is different from the inbound such then
the outbound content-type header should be aligned with the new payload type.
Discrepancy between the outbound payload type and the content-type metadata
may cause selecting incorrect message converter and therefore fail to encode
the outbound message into an appropriate wire format.
Due to https://jira.spring.io/browse/INT-4397 the @Transfomer annotated processors
blindly copy their input message headers into the output headers with no
regards of the outbound payload type.
The implementation of the @ServiceActivator doesn't suffer from this problem and can
be used as drop-in workaround.
Resolves #3

Looks like related to this one: https://jira.spring.io/browse/INT-4397

@artembilan, yes i believe it is caused by INT-4397

Cool, @tzolov !
One remark though: when we use annotations in the commit message (symbol @), we should wrap that into the code (backticks) otherwise we may call a user here on GitHub, when such a commit message becomes a PR description :smile:.


Change Github workflow according to Postman change

What this PR does:
Add an environment variable in Postman cli command.
Which issue(s) this PR fixes:
Fixes https://datastax.jira.com/browse/C2-2310
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

@Hazel-Datastax can we merge this?


integers-coordinates-of-the-vertices

Hello Anass,
I figured out that in https://www.cityjson.org/specs/1.1.3/#coordinates-of-the-vertices the vertices should be integers and not float.
I'd like to discuss and eventually submit a MR to change somewhere about here https://github.com/Yarroudh/ZRect3D/blob/main/zrect.py#L125  the floats into integers.
Maybe we should think update updating the transformation matrix if we do not want to round values to integers?
Not having integers is generally just fine (ninja etc are okay) but it could be annoying and throwing some errors when using some cjio functionalities (like vertices cleaning, removing duplicates, etc.)
Julien

Hello @collaer,
I think you are right. I did not pay attention to this in CityJSON specs. May be rounding the values of the coordinates can fix that.


how to save using PHP ??

hi,
i am trying to use this but i dont know how to save crop image and save in PHP after getting points and zoom score,,,

You should check out #62 and #79 - both of those have php examples.


Feature List

Login Authentication Possibilities
I should be able to configure if the following authentication possibilities are allowed or not.
Should we add to each method how long it should be valid? To get rid of the loginlifetimes?

[ ] Username (enabled & lifetime?)

[ ] suffix?

[ ] Password (enabled & lifetime?)

[ ] password complexity
[ ] password expiry
[ ] password reset allowed?
[ ] password reset code

[ ] Passkey

[ ] Code Generator

[ ] TOTP
[ ] SMS OTP

[ ] OTP Code Generator

[ ] EMail OTP

[ ] OTP Code Generator

[ ] External IDP

[ ] Mapping?
[ ] auto creation / linking/ update
[ ] manual creation / linking

(Remove from Login Policy

Username and Password allowed
Email login
Phone Login

Login

[ ] Trigger organization including settings and branding

OIDC
SAML
Selfservice Possibilities

[ ] Register Local User

[ ] Link to external form?

[ ] Register Authentication (add mfa/force mfa)
[ ] Register External User
[ ] Register Org
[ ] Terms of Service (accept, or show)
[ ] Privacy Policy (accept or show)
[ ] Help Link
[ ] Support Email
[ ] User Profile Page

Configurations

[ ] Minimal Authentication Level Step up authentication (replace force mfa).
[ ] Domain Discovery (Org Domains)
[ ] Ignore unknown usernames? (Better name?)
[ ] Default Redirect URL
[ ] Add more security prompt (mfa init check)
[ ] Re-authentication
[ ] lockout

To Remove

[ ] SMTP Sender Address matches Instance domain

Questions:

[ ] Is is possible to somehow inject CSS? e.g get rid of the password reset link?
[ ] Can I do so per organization?

General Flows:


chore: update package description

Since SvelteKit is not mentioned in the package name, description, or tags, the kit package cannot be searched in npmjs.com with the SvelteKit keyword.

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Edits

[x] Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.

Other than the --open flag (which can be reverted if wished - maintainer editing is enabled)
- npm run dev
+ npm run dev -- --open

This PR is basically a one word change in the description. Could this be reviewed and merged?
- The fastest way to build Svelte apps
+ SvelteKit - the fastest way to build Svelte apps

Thanks for the reminder and for pointing out this issue! I was just playing around with the search on npm and actually don't know if we should change the description either. I think keywords would be a better solution. If that alone isn't enough then maybe we can update the description. I sent https://github.com/sveltejs/kit/pull/12330 as an attempt to fix this. I'll leave this open while we evaluate whether it's enough

@sveltejs/kit is now searchable using the SvelteKit keyword.

great! too bad it's only the fourth result. I'll merge this to see if it helps boost it up the rankings


Need to track the progress based on particular taskId

I got the taskId but how to track the progress or result for that particular taskId

@Dheena32 did you find any solution ?

No, i didnt got any solution

hello @Dheena32.
the steps that i take to do this is:
1 - Create a class to hold the data of the files that i will upload like [progress] and [status]. (let's call it [UploadItem]).
2 - Create a stream subscription that will listen for the progress. (there is a property [FlutterUploader().progress])
3 - when i upload a file, i create and hold the state of a new [UploadItem]
4 - the progress stream has a [UploadTaskProgress] value that has the [taskId], [progress], [tag] etc. so every time the stream receives a progress i check in my state for the upload item that match the [taskId] or the [tag], then i update that value with the new one.
some points:
1 - I use flutter_bloc with an UploadBloc that is in charge of all this job
2 - I use flutter_uploader 1.2 ( not the pre release version)
for the [status] i do the same, create a stream with [FlutterUploader().result]
i hope this works for you, sorry if i misunderstood your problem or this does't work for you. by the way, i saw this approach in the example of the flutter_uploader 1.2

I think you have found a solution?


Problem streaming bigger torrent files

I have problem streaming some big torrent files - over 2GB. This is the problem:
The pre-buffer is downloading with good speed, i am connecting to good amount of seeders. When the stream starts i see the video cache is full, but then it starts emptying gradually until it reach zero and the stream freezes. If i hit pause i see that the download speed is good, so i don't know why the video cache is not filling.
Elementum is set to download the torrent to ram memory, and my upload and download speed is set to unlimited. I am using Kodi 17 on Windows 7. Elementum version is 0.1.37.

No log = no help.

Closing without getting response.


Spanish translation of createButton()

Addresses #1216
Changes:
Translate Description, Returns and Parameters text in createButton reference.

Thank you @Elliot-Hernandez for working on this. I am inviting the Spanish Translation stewards @Guirdo, @Andreu-G, @dnd-alv, @holomorfo to review this pull request.

Thank you @Elliot-Hernandez  and @holomorfo
Just added @Elliot-Hernandez to the p5.js contributors list 🎉


failure to create new tab elements when comma ends custom list

when there is a comma at the end of a custom team list in options and is left empty after, a new tab is not populated with elements
when looking at error for extension, the error has to do with the trying to load JSON. gives invalid JSON error (e.g. [341,]) is invalid

Rather than repeatedly making a random choice like in your PR, it would be best to just strip the commas.


Add a simple validation to the service name to prevent setup issues

Including dots in your service name is going to cause issues with traefik and initial Kamal setup.
This PR adds a simple regex to validate that the service name is alphanumeric and can include hyphens /^[a-z0-9-]+$/.
Feels a bit like overkill but just trying to ease everyone's first impression of Kamal to make it easier to get up and running.
Closes #671.

@nickhammond - I'm seeing this warning now, I guess we want to reverse the order of the - and _?
.../kamal/lib/kamal/configuration.rb:291: warning: character class has '-' without escape: /^[a-z0-9-_]+$/

@djmb Fixed in #708, sorry about that.


🔧 Update template

This pull request merges the current state of the template used here into this project so that it is up to date.

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

v1.1.1
GitHub release
Docker container (sebbo2002/fitness-first-ical-proxy:1.1.1)
v1.1.1

Your semantic-release bot :package::rocket:


margin: auto w/ flex elements

In the following example, I want to have 1,2 aligned to the left and 4 aligned to the right:

The only way I know to achieve this is by having a child 3 which has flex grow = 1 and set it invisible:

But I'd like to avoid having to add a child just for that.
In CSS flexbox, this can be easily achieved by setting margin-left: auto on 4, see the MDN documentation and example here it describes it very well.
There was an issue open 7 years ago, in 2016 but closed by OP because in the specific case of his app design, he realized he doesn't need it:  https://github.com/facebook/yoga/issues/198. However, the issue is quite important and I'm surprised nobody has raised the need for the auto margins behavior, what am I missing?

Oh I realized auto margin works now.. 😁


Fix the bug that removeExtremeOutliers not called.

What type of PR is this?
bugfix
What this PR does / why we need it:
Fix the bug that removeExtremeOutliers not called.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

changed


Work with TA or fix: Two commits for paper 1 and paper 2

This pull request can not be approved by my as it includes 135 file canges across different papers and a project. There is no separation of pull request so it is not possible to do a reasonable review of this pull request.
You will have to split this pull request up in several pull requests.
Naturally none of your stuff is reviewed by the TAs as they can not see what you contributed.
None of us have the ability to approve such a request due to the danger it poses on potentially othe projects.

please remove this pull requests and create multiple. For each paper you need 1 and for the project you also need one.

I AM CLOSING THIS PULL REQUEST NOW AS WE CAN NOT ACCEPT.
WE LIKE YOU TO MAKE MULTIPLE PULL REQUESTS. IMPORTANT IS THAT YOU DO NOT MIX COMMITS BETWEEN PAPER1, PAPER2, and PROJECTS if you check in more than 10 files
OBVIOUSLY WE WILL NOT SEE THE PAPERS IF YOU OD NOT DO THIS AND IT WILL EFFECT YOUR GRADE.

closed due to 5 day inactivity and explicit explanation on what to do here and on piazza
in case of questions talk to TA


Vulkan: TAA smears Skyboxes

Godot version
v4.0.beta.custom_build [ca25c6e0a]
System information
Arch Linux, KDE, AMD RX6800XT, Mesa, Vulkan, Clustered+
Issue description
Temporal Anti-Aliasing seems to recieve no motion vectors for skybox textures, resulting in motion smearing when rotating the camera.
This is especially aparrent with textures detailed enough so that TAA's blending threshold doesn't kick in:
Here's an example with PhysicalSky using a spreckly FastNoiseLite texture in the "Night Sky" slot to imitate stars. ooking around causes massive smearing. Note the difference between the textured ground and the sky - both were moving the same way on-screen.

Steps to reproduce
Use a very grainy, 4x2K resolution texture and put it either into PhysicalSky's Night Sky rtexture slot or into PanoramicSky.
Minimal reproduction project
Here's a simple project that demonstrates this with a rotating camera:
TAA vs Skybox.zip
I've also added an alpha-blended sprite to demonstrate how transparency overwrites motion vectors for anything that's behind it regardless of alpha value.

You mention motion vectors a few times in this Issue, are you seeing motion vectors when you turn the motion vector debug mode on? The skybox should not generate any motion vectors at all, so if it is, that could point us in the direction of the bug.

@clayjohn I tried that but I saw only pure blackness.

You mention motion vectors a few times in this Issue, are you seeing motion vectors when you turn the motion vector debug mode on? The skybox should not generate any motion vectors at all, so if it is, that could point us in the direction of the bug.

The sky shader doesn't appear to do any sort of TAA handling and isn't even being fed the previous frame's data yet.

Motion vectors are not calculated when the camera is rotated, which is why the sky is blurred when rotated. At the same time, for FSR2 the sky is calculated correctly.


forward verbosity to cargo setup

@bors r+

:pushpin: Commit 3378d9dd7a9c03e729c0e5b600ef627f873a4b31 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 3378d9dd7a9c03e729c0e5b600ef627f873a4b31 with merge b46a95f1c9ca0ff7ca9c42cfb3fbdf58b539a266...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing b46a95f1c9ca0ff7ca9c42cfb3fbdf58b539a266 to master...


Cleanup legacy Terraform modules

Issue summary

Once https://github.com/ChainSafe/forest-iac/issues/373 is completed and all is working as expected, the terraform directory should be purged and live forever in our Git history.
No workflow nor documentation should depend on them, e.g., https://github.com/ChainSafe/forest-iac/blob/main/composite-action/terraform/action.yml
Other information and links

Done.


Repo sync for protected CLA branch

The pull request is created from master637356381161825583 to master to fix git push error for protected CLA branch

@cmcclister : Thanks for your contribution! The author(s) have been notified to review your proposed change.


is saving as bfres possible?

is it possible? i see a option to save the bfres when right clicked on the bfres file in the right panel.
but whenever i clicked the save button it asks for location and saves a exact copy of the original bfres.
am i doing something wrong? or if it's impossible to save as bfres can you add that feature? i really need it for 3d world modding and reply soon cause i'm really stuck here unable to export a bfres model

i tried wexos toolbox but what i always get is this:

or this:

please help!

also the materials aren't loading either


scripting-addition must be updated for macOS Sonoma 14.2

title.

I am now using macOS 14.2 (23C64). I have updated the scripting addition and the sudoers.d/yabai file. The command yabai -m query --windows is functioning properly, but I am unable to switch spaces. Nothing happens. I have not encountered any error messages, including in the yabai_username.err.log file.
Should I temporarily revert to the default macOS method of switching spaces with animation, or is there something I am overlooking?

I am now using macOS 14.2 (23C64). I have updated the scripting addition and the sudoers.d/yabai file. The command yabai -m query --windows is functioning properly, but I am unable to switch spaces. Nothing happens. I have not encountered any error messages, including in the yabai_username.err.log file.
Should I temporarily revert to the default macOS method of switching spaces with animation, or is there something I am overlooking?

how did you update the scripting addition? I'm on MacOS 14.1.2 (23B92) and I'm getting

when I run
yabai -m signal --add event=dock_did_restart action="sudo yabai --load-sa"
sudo yabai --load-sa

and I switch spaces with yabai -m space --focus next

Same issue here. Things like toggling float on windows work, moving windows to spaces work, but switching to spaces (yabai -m space --focus next etc.) don't work for me.

same here

same here but the issue seems similar to #1880

@koekeishiya when you say "Scripting addition must be updated"  I am assuming it is a todo for you and not for the users?

@koekeishiya when you say "Scripting addition must be updated" I am assuming it is a todo for you and not for the users?

Yes, he mentioned somewhere that he will do that once it out of beta, I believe it's in RC(release candidate).

I encountered the same issue and have a workaround using MacOS's built-in shortcut for switching spaces/desktops.

Navigate to System Preferences > Keyboard > Keyboard Shortcuts.
Select Mission Control from the left sidebar.
Look for the option Switch to Desktop ... under the Mission Control dropdown.

I was concerned there would be a delay when switching spaces, which is why I used yabai, but that isn't the case. I should note that you will need to delete/comment out the yabai -m space --focus lines in your skhd config if that shortcut is the same as the one in the settings.

yeah, that's the workaround I'm using but the workspace switches are slow, there is a significant delay compared to yabai workspace focus change. I'm personally using sketchybar on top, and that gets redrawn every time I do a switch.

Same here, it is out of beta now, and after update, can not focus spaces, neither by click the sketchybar

@kalebris That could be because you have a shortcut set for moving left/right a space instead of switching to a specific desktop. There is a delay for me when moving left/right but not when switching to a specific desktop. If that doesn't work, it could be something with sketchybar.

macOS 14.2 23C64 is public now, no longer beta. Hope the fix is incoming soon 🤞🏼

I encountered the same issue and have a workaround using MacOS's built-in shortcut for switching spaces/desktops.

Navigate to System Preferences > Keyboard > Keyboard Shortcuts.
Select Mission Control from the left sidebar.
Look for the option Switch to Desktop ... under the Mission Control dropdown.

I was concerned there would be a delay when switching spaces, which is why I used yabai, but that isn't the case. I should note that you will need to delete/comment out the yabai -m space --focus lines in your skhd config if that shortcut is the same as the one in the settings.

After I upgraded to macOS 14.2 (23C64), the yabai -m space --focus related commands could not be used normally. Even if I uninstalled yabai and reinstalled scripting-addition, it still did not work. After reading this method of reply by @cronJohn, I temporarily used this alternative method to achieve a similar effect.
First configure the Mission Control shortcut keys:

Comment out the related shortcut key configuration for switching desktops in skhdrc:

Then I also found that moving the application window to the specified desktop and then switching to the corresponding desktop would also cause problems. I also made corresponding adjustments to the configuration in skhdrc:

Oh nooo, I updated blindly, not realizing that this is a very new update and that yabai doesn't have the updated scripting addition yet. Thanks in advance for updating the scripting addition @koekeishiya, really appreciate your work! I can't work without yabai anymore xD

It doesn't work for me as well.
Script addition doesn't work.
I got so used to yabai, that without it I can't work.
Yabai is the reason why I am not complaining about macOS.

It doesn't work for me as well. Script addition doesn't work.
I got so used to yabai, that without it I can't work. Yabai is the reason why I am not complaining about macOS.

What is so important about Sonoma that you cannot wait a few days?
I'm still on Ventura, and will stay there until GPG Mail is fixed. Then I'll update, and be assured that Yabai is tested on Sonoma.

What is so important about Sonoma that you cannot wait a few days? I'm still on Ventura, and will stay there until GPG Mail is fixed. Then I'll update, and be assured that Yabai is tested on Sonoma.

Sonoma is an exciting macOS update. Yabai worked well with the initial release and also with 14.1. I agree that, as Yabai "power users"", we should consider waiting for a week or two before upgrading to a point release. This will give the developer time to test and ensure Yabai works properly. It's always a cat and mouse game, unfortunately...

The fork by @FelixKratz has been updated to 14.2 (https://github.com/FelixKratz/yabai/commit/df5b037108c4a70dc5e854bb60ccbff9701da4f5)

The fork by @FelixKratz has been updated to 14.2 (FelixKratz@df5b037)

Yeah but I still get the same error while trying to focus a space using his fork:
cannot focus space due to an error with the scripting-addition.

The fork by @FelixKratz has been updated to 14.2 (FelixKratz@df5b037)

Thanks for this. I confirmed to successfully used his fork to correctly run yabai --load-sa on my 14.1.2 2022 M2 MBA
The steps working for me:

sudo yabai --uninstall-sa && brew uninstall koekeishiya/formulae/yabai uninstall the brew-version of yabai first
git clone https://github.com/FelixKratz/yabai cloned his fork
cd yabai && make all manually build the new/patched-yabai's binary
cd bin && sudo mv yabai /usr/local/bin/yabai moved this to $PATH (I think this is optional because somehow I need to manually specify the absolute path of the new yabai binary in my .skhdrc; up next)
nvim ~/.skhdrc replace the "yabai" executable with absolute paths /usr/local/bin/yabai
Restart macOS
sudo yabai --load-sa && yabai run the new yabai. Maybe also try brew services restart skhd

Finally able to enjoy instantaneous space-switching again 🤭 hope this helps anyone. Hopefully later I able to help merge this diff to yabai's main branch.

The fork by @FelixKratz has been updated to 14.2 (FelixKratz@df5b037)

Thanks for this. I confirmed to successfully used his fork to correctly run yabai --load-sa on my ~14.1.2~ macOS Sonoma 14.2 2022 M2 MBA. *edit: had just updated to 14.2 and still works
The steps working for me:

sudo yabai --uninstall-sa && brew uninstall koekeishiya/formulae/yabai uninstall the brew-version of yabai first
git clone https://github.com/FelixKratz/yabai cloned his fork
cd yabai && make all manually build the new/patched-yabai's binary
cd bin && sudo mv yabai /usr/local/bin/yabai moved this to $PATH (I think this is optional because somehow I need to manually specify the absolute path of the new yabai binary in my .skhdrc; up next)
nvim ~/.skhdrc replace the "yabai" executable with absolute paths /usr/local/bin/yabai
Restart macOS
sudo yabai --load-sa && yabai run the new yabai. Maybe also try brew services restart skhd

Finally able to enjoy instantaneous space-switching again 🤭 hope this helps anyone. Hopefully later I able to help merge this diff to yabai's main branch. *edit: I meant only the scripting-addition fix, not Felix's personal patches of course.

This works for me. Previously I tried it but I failed. Now I successfully make it work on my M1 Pro Macbook with Sonoma 14.2.
Below are some notes that may help others:

You can just use the commit https://github.com/FelixKratz/yabai/commit/df5b037108c4a70dc5e854bb60ccbff9701da4f5 as a patch on yabai's latest commit (https://github.com/koekeishiya/yabai/commit/9fffe016ea91d221cb60891f1bb9b39fae738808).
You should restart your computer. (I think this is the reason I failed before)
Remember to configure the scripting addition as described here.

The fork by @FelixKratz has been updated to 14.2 (FelixKratz@df5b037)

Thanks for this. I confirmed to successfully used his fork to correctly run yabai --load-sa on my ~14.1.2~ macOS Sonoma 14.2 2022 M2 MBA. *edit: had just updated to 14.2 and still works
The steps working for me:

sudo yabai --uninstall-sa && brew uninstall koekeishiya/formulae/yabai uninstall the brew-version of yabai first
git clone https://github.com/FelixKratz/yabai cloned his fork
cd yabai && make all manually build the new/patched-yabai's binary
cd bin && sudo mv yabai /usr/local/bin/yabai moved this to $PATH (I think this is optional because somehow I need to manually specify the absolute path of the new yabai binary in my .skhdrc; up next)
nvim ~/.skhdrc replace the "yabai" executable with absolute paths /usr/local/bin/yabai
Restart macOS
sudo yabai --load-sa && yabai run the new yabai. Maybe also try brew services restart skhd

Finally able to enjoy instantaneous space-switching again 🤭 hope this helps anyone. Hopefully later I able to help merge this diff to yabai's main branch. *edit: I meant only the scripting-addition fix, not Felix's personal patches of course.

how do you detach it so that it can run in the background?
/usr/local/bin/yabai --start-service does nothing

@kareemlukitomo thanks, it works. But for some reason I can see a strange overlap on the second screen for 1 and 2 windows, and it overlap a sketchybar if there is only 1 window on a screen:

and join the @SzymonMielecki question - how to make this yabai version work as a service ?

@SzymonMielecki @alpiua this is good enough for me (not exactly running as proper service though, since I don't have a good concept of how background service works on macOS)
https://chat.openai.com/share/348b1fd3-08c8-477a-8a3a-bdac028e6ec6
TL;DR
#!/bin/bash

while true; do
  yabai -c ~/.yabairc
  # Sleep for a bit to avoid rapid restarts in case of immediate failure
  sleep 1
done

Then running this into a detached screen/tmux/zellij session. (Gotta re-run on restart). Feel free to point out/give suggestions on how to improve this guys 👍

For now, I'll just use Felix's personal brew formula ✌ thank you for this @FelixKratz

yabai --uninstall-sa && sudo rm -rf /usr/local/bin/yabai cleanup first
brew tap FelixKratz/homebrew-formulae load this formula
brew install --HEAD felixkratz/formulae/fyabai install it
Configure SA and run yabai as usual
nvim ~/.skhdrc revert the previous changes that I suggested above :b

Is there any benefit in cherry-picking https://github.com/FelixKratz/yabai/commit/df5b037108c4a70dc5e854bb60ccbff9701da4f5 and creating a PR for the master branch in this original repo? Happy to help with this or anything else needed. Cheers!

This seems to fit here, but let me know if I should start a new thread instead:
I'm new to mac, and it seems these OS updates are a lot more common than I expected. Is there a way to tell if an OS update will break yabai, before updating? My company aggressively pushes OS updates, and while I can postpone them for a bit, it's cumbersome.

Was able to cherry-pick the commit from Felix's fork (https://github.com/PrayagS/yabai/tree/patched-for-14.2) and build the updated binary.
Tried placing the binary at yabai's original location (/opt/homebrew/bin/yabai) hoping that the background service commands would work but they didn't 🥲.
Used this old hack that I used to have when skhd didn't have a background service,
# Start skhd/yabai
if pgrep -x "yabai" >/dev/null
then
else
   yabai
   echo "yabai started"
fi

Placed this in my .zshenv and yabai is running as long as there's a terminal window open.

For now, I'll just use Felix's personal brew formula ✌ thank you for this @FelixKratz

yabai --uninstall-sa && sudo rm -rf /usr/local/bin/yabai cleanup first
brew tap FelixKratz/homebrew-formulae load this formula
brew install --HEAD felixkratz/formulae/fyabai install it
Configure SA and run yabai as usual
nvim ~/.skhdrc revert the previous changes that I suggested above :b

Is there any benefit in cherry-picking FelixKratz@df5b037 and creating a PR for the master branch in this original repo? Happy to help with this or anything else needed. Cheers!

It worked for me, thank you so much!

I can confirm v6.0.2 works!
Just by eliminating the workspace switching animation, yabai is saving an hour of my life every year, to say nothing of the emotional damage that vanilla macOS window management causes. Yabai is easily providing $100+ of value per year.
Åsmund's work finally inspired me to set up a Patreon account.
Thank you Åsmund! 🙏 🙏 🙏
-----> https://www.patreon.com/aasvi

Merry Christmas!! However I updated yabai to v6.0.2, but it seems still not working:
~ ❯ yabai -m space --focus 2
cannot focus space due to an error with the scripting-addition.
~ ❯ yabai --version
yabai-v6.0.2

anything I missed here? I tried to run sudo yabai --load-sa manually, there is no error, just not woking as expected

@marsjane did you edit your sudoers file after upgrading? In particular, the sha256 must be updated for the new binary
https://github.com/koekeishiya/yabai/wiki/Installing-yabai-(latest-release)#configure-scripting-addition

Oh yes it works! Sorry for missing this, this is the first time I update yabai ;) Many thanks !! @boronine

It works!
Thank you!

Hey,
With Sonoma 14.2.1, it didn't work. I spent 1 hour to try to make it working again but without any success. The patch and the fork of @FelixKratz worked! Thank you a lot!

Was able to cherry-pick the commit from Felix's fork (https://github.com/PrayagS/yabai/tree/patched-for-14.2) and build the updated binary.
Tried placing the binary at yabai's original location (/opt/homebrew/bin/yabai) hoping that the background service commands would work but they didn't 🥲.
Used this old hack that I used to have when skhd didn't have a background service,
# Start skhd/yabai
if pgrep -x "yabai" >/dev/null
then
else
   yabai
   echo "yabai started"
fi

Placed this in my .zshenv and yabai is running as long as there's a terminal window open.

The disown command will let you detach the yabai process from the controlling TTY allowing you to close the terminal window that you launched yabai from.  You can run something like:
yabai -V >> /tmp/yabai-HEAD.log 2>&1 & to launch the daemon and then disown will detach the process from your terminal and allow you to close it.  Hope this helps :)

I'm trying to get the scripting addition to work with Yabai on macos 14.2.1 with no luck. I setup everything and updated the sudoers file, but it gives this error when i try to run yabai:
yabai: scripting-addition failed to inject payload into Dock.app!

For now, I'll just use Felix's personal brew formula ✌ thank you for this @FelixKratz

yabai --uninstall-sa && sudo rm -rf /usr/local/bin/yabai cleanup first
brew tap FelixKratz/homebrew-formulae load this formula
brew install --HEAD felixkratz/formulae/fyabai install it
Configure SA and run yabai as usual
nvim ~/.skhdrc revert the previous changes that I suggested above :b

Is there any benefit in cherry-picking FelixKratz@df5b037 and creating a PR for the master branch in this original repo? Happy to help with this or anything else needed. Cheers!

This didnt work for me since it just installed different version of yabai and didnt link it to system. So I added brew uninstall yabai and brew link fyabai.

I would like to confirm that scripting addition on 6.0.6 works.

I would like to confirm that scripting addition on 6.0.6 works.
no

@koekeishiya is yabai managed by homebrew services or plist for now?


[BUG] rancher/machine change breaks long working functionality with OpenNebula

Rancher Server Setup

Rancher version: 2.8.3
Installation option (Docker install/Helm Chart): docker

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.2.7 1.2.8
Cluster Type (Local/Downstream): Local

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Opennebula

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom): Admin

If custom, define the set of permissions:

Describe the bug
We provision our cluster with the help of OpenNebula. Rancher integrates with OpenNebula with a NodeDriver based on docker machine (https://docs.opennebula.io/6.8/integration_and_development/automation_tools_integration/docker_machine.html). Since release 2.8.3 provision of hosts fails with "cloud-init: command not found". OpenNebula nodes do not offer cloud-init and cloud-init support is limited. Instead they use their own package (https://github.com/OpenNebula/one-apps). This worked nicely for years. Our nodes are ubuntu 22.04. Reason for the issue is propably this commit: https://github.com/rancher/machine/commit/1fc0db0b903517dd871886141a0878883d641b46.
I know OpenNebula is not among the officialyl supported architectures, but it used to work just fine. Could you please modify the hard dependency on cloud-init to a dependency only if cloud-init exists.
Instead of
p.SSHCommand("sudo cloud-init status --wait")
you could use
p.SSHCommand("which cloud-init && sudo cloud-init status --wait")
To Reproduce
Use Opennebula to deploy your cluster.
Result
Nodes can not be installed because of error: "cloud-init: command not found"
Expected Result
Docker installation should just continue as it did in the past.

Just a little addition. I could work around this issue by placing a dummy cloud-init script in the image which does nothing.

Same problem here, fixed downgrading to version 107....


Linear time sorting

Adds voracious_radix_sort to dependencies
Replaces (id, processing_time) pairs with a struct Task<T>
implementing Eq and Ord
Uses RadixSort in LPT and BnB

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is n/a.

Impacted Files
Coverage Δ

src/alg/bnb.rs
77.77% <ø> (ø)

src/alg/core.rs
60.00% <ø> (ø)

src/lib.rs
47.61% <ø> (ø)


support for touch events on mobile

I've made some changes to support touch events on mobile.
If you like the changes I'd be happy to see them merged.
This would close #14

Hey, thank you for the PR! This looks good from the first glance. I'll try it out today.
I'm curious though can you elaborate on what's the thinking behind the condition parameter here?
Thanks!

Hey, I just tried the your touch supported version in the example project and I'm getting some weird behavior.
Maybe I'm doing something wrong? If so, can you provide a CodeSandbox or similar example?
Here's the video of me trying to use the splitter on my phone

Thanks for trying out. I should have tested the example project with my changes.
The issue was the default scroll behaviour of the touchmove event. This is now handled with preventDefault in case of a touch event
Now I've tested the example project with firefox and chrome on mobile and on desktop.

Just tested it and all looks good. Thank you for the PR! I'll make a new release now.


[BUG] Swordsmachine light knockback on thrown sword parry

Description

Swordsmachine gets knocked down when thrown sword is parried
Exceptions caused by missing reference to animator

Reproduction
Describe steps to reproduce the issue. If the way of reproducing the issue is unknown, you can include as much information as possible on when it happens or how often it happens

Parry the thrown sword
Swordsmachine gets knocked down
Errors occur

Console logs
[Error  : Unity Log] NullReferenceException
Stack trace:
Ultrapain.Patches.SwordsMachineFlag.Update () (at <d5500e16cc764fd5817bd942ad7ba1db>:0)

Extras

Fixed on b93fd5e


requestDevice not working without dialogbox (showdialog = false)

In version 2.2.0 or earlier, I would like to use requestDevice without showing the dialog box (and de facto selecting the first device that matches my criteria).
So I change showDialog = false on this line: https://github.com/capacitor-community/bluetooth-le/blob/main/android/src/main/java/com/capacitorjs/community/plugins/bluetoothle/BluetoothLe.kt#L293
But then, it's not working. I have to patch the following:
---<EMAIL_ADDRESS>2023-04-07 17:21:14.954956599 -0700
+++<EMAIL_ADDRESS>2023-04-07 17:24:55.749245428 -0700
@@ -69,7 +69,8 @@
                 }
             } else {
                 if (allowDuplicates || isNew) {
-                    scanResultCallback?.invoke(result)
+                  savedCallback?.invoke(ScanResponse(true, result.device.address, result.device))
+                  savedCallback = null
                 }
             }
         }
@@ -98,7 +99,7 @@
             if (showDialog) {
                 dialogHandler = Handler(Looper.getMainLooper())
                 showDeviceList()
-            } else {
+            } else if (scanResultCallback != null) {
                 savedCallback?.invoke(
                     ScanResponse(
                         true, "Started scanning.", null

Clearly the first hunk was totally wrong before my patch. The second hunk leverages the fact that scanResultCallback is null for requestDevice.
You probably didn't test this scenario as showDialog is not exposed. Could you fix the same problem on iOS too? Would you be OK to expose showDialog on the web interface of requestDevice?

If you don't want to show a dialog, you should use requestLEScan. For your use case, you can just use the first scan result you get from requestLEScan and then call stopLEScan.
The two methods requestDevice and requestLEScan are modeled after the Web Bluetooth API, where requestDevice shows a dialog for the user to pick a device and requestLEScan returns the scan results in a callback function.

Would you be OK to expose showDialog on the web interface of requestDevice?

This is not possible, as there is no showDialog API in requestDevice of the Web Bluetooth API. The dialog is always shown. The showDialog flag in the Android code was never meant to be exposed. It's only there to be able to reuse code internally in the plugin.
Your patch actually breaks the requestLEScan method, so you might want to revert it to use requestLEScan.


Title is now set to class in citrix

Bug or Regression?
Regression
System Info and Version
Since verison 41 my citrix session now has the title
Window 602fb9a46640 -> wfica:
	class: Wfica
	title: wfica
	initialClass: Wfica
	initialTitle: wfica

Description
Before it had an appropriate title to the window it was, something changed.
How to reproduce
launch citrix workspace on the new version.
Crash reports, logs, images, videos
No response

Are you sure it was not a program update itself?
does downgrading hyprland fix it?

Yup, downgrading changes it back to it's normal title
Window 564b3a931560 -> Office 365 Desktop (SSL/TLS Secured, 256 bit):
mapped: 1
hidden: 0
at: 3840,0
size: 1920,1060
workspace: 3 (3: )
floating: 0
monitor: 1
class: Wfica
title: Office 365 Desktop (SSL/TLS Secured, 256 bit)
initialClass: Wfica
initialTitle: Office 365 Desktop (SSL/TLS Secured, 256 bit)
pid: 286861
xwayland: 1
pinned: 0
fullscreen: 0
fullscreenmode: 0
fakefullscreen: 0
grouped: 0
swallowing: 0
focusHistoryID: 1

can you bisect?
(it's prolly the wayland rewrite)

Is there a guide you recommend for this? I have no idea what I'm doing.

Is there a guide you recommend for this? I have no idea what I'm doing.

https://wiki.hyprland.org/Crashes-and-Bugs/#bisecting-an-issue

After doing some test builds of Hyprland, and starting Citrix Workspace, I confirmed that this was fixed by a95df6b57e68785c13fd2d7f93d9eba28b9406ed.  The title and initialTitle now display correctly.
This issue can be closed.

Yup, it's fixed.


[Feature Request] Auto-Link Table of Contents to Referred Pages in PDF

I have a pdf with this (just an example):

I am considering a tool that would auto-link the page numbers to the referred pages in the document. So, when the user clicks the number, it should go to the referred page.

Due to high variability of PDFs and complexity of detecting this, we don't plan to support this, Sorry!


perf(tracing): do not create spans from timers

Summary
Before this change timers would generate spans, which means DB and DNS spans in recurring timers would be continuously generated and garbage-collected.
This commit checks the exact ngx phase and runs it against a whitelist to ensure timer phase does not generate spans.
Checklist

[ ] The Pull Request has tests
[x] (no) A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[x] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference
KAG-3169

I'm open to suggestions in case there is a faster way to detect that we are in a timer context


StreamLabs Desktop uses undocumented NEWS NotificationSubtype

StreamLabs sometimes returns a Notification with a subtype of NEWS, despite this not being mentioned in the documentation (at time of writing).
Handle the new value of the subtype.

Fixed in version 1.2.5


22.18.6 Flashing Powerflow Animation

This morning I noticed my Powerflow animation is just flashing on the dashboard. I checked my firmware version, and it looks like I got updated to 22.18.6. Is anyone else noticing this issue?
Navigating to port 8675 shows the same behavior. The page just keeps reloading making the animation "flicker"

Ok, the fix is easy!  The issue is that pypowerwall proxy caches the responses from the Powerwall, including the setup JS.  You just need to restart pypowerwall and refresh your browser:
docker restart pypowerwall

Please let me know if this fixes the issue for you too @cwagz.
I'll have to look to see if there is an easy way to have the proxy detect version upgrades and flush the cache.

@jasonacox - I restarted pypowerwall and it is no longer flashing. Is that what you were expecting or did Tesla change something back to the way it was?

Awesome!  Thanks for opening the issue and confirming the fix.
Tesla didn't change anything back.  Restarting the pypowerwall proxy clears the local cache of Javascript that contained the old firmware version of the animation.  Once cleared, pypowerwall rebuilds the cache with the new firmware version.
I'm looking to see if we can detect and auto-clear the pypowerwall proxy cache anytime the firmware upgrades the animation.


Add "correlation_id" to the logging context

Adding another field to the logging context. So if a task is broken down into multiple steps we can later bind them together. An example is a task that is sent to SQS as multiple messages where each message then triggers an AWS Lambda function. Each function call gives us a new "aws request id" but we also want to be able to look them up together as a set of functions.
Example use in AWS CloudWatch Logs Insights:
filter correlation_id = "03df790a-2e9c-46c4-8911-d12b90f04bd7"
| fields @timestamp, aws_request_id, message
| sort @timestamp desc
| limit 20

The correlation_id can be used, say, for tracking all logic that processes a message:
def handle_event(event, context):
    json_logging.update_from_lambda_context(context)
    with json_logging.log_stack_trace(logger):
        for record in event["Records"]:
            json_logging.update_context(correlation_id=record["messageId"])

So definitely on the user of the library to figure out an appropriate value.


Error on iOS emulator (12.0) with SWIFT_VERSION mismatch ...

When I am running in debug mode on an IOS emulator -- it won't build because of the following error..
Not sure how to fix :(
=== BUILD TARGET get_version OF PROJECT Pods WITH CONFIGURATION Debug ===
       Check dependencies
       The “Swift Language Version” (SWIFT_VERSION) build setting must be set to a supported value for targets which use Swift. Supported values are: 3.0, 4.0, 4.2. This setting can be set inthe build settings editor.

[   +6 ms] Could not build the application for the simulator.
[   +2 ms] Error launching application on iPhone X.
[   +3 ms] "flutter run" took 6,431ms.

OK -- Self-solved.  Looks like a config must be done in Flutter with the new XCode etc. I followed the instructions here
Swift Plugin Fix


Transform normal to cam space

Hi, below shows two example from small_city/street/train/small_city_road_down: 0930 and 1228. Different columns are rgb, normal in cam space; mask of pixels that norm of normal is not equal to 1.

As can be seen in the image, the normal of ground is significantly different but from the rgb image, the two cameras seems to have a similar angle witg repsect to the ground. Is there anything wrong with the camera rotation?
Why there exists so many normal that norm is not equal to 1?

Thank you,
Yanrui

Our normals are in the world coordinate. We provided script to read normal in load_data.py, where we normalize the normal. An explanation is provided Depth and Normal Map Note.
For your case, these two cameras have a similar angle with respect to the ground, which means these two cameras have same pitch. Maybe you can check if the cameras of these two images have different yaw, which will also affect normal in camera space.


Output more PR metadata.

Is your feature request related to a problem? Please describe.
In our workflow, we continue to do additional automation on it, such as set correct milestones or add it to a project board. Usually these Github APIs require either the pull request number (like 215) or id (like PR_kwDOAOaWjc5mP_GU), which is annoying to get from just the URL returned by this action.
Describe the solution you'd like
Output additional data from the created PR, such as number (which it looks like the action already uses) and ID. These should continue to default to being empty, like the URL, if the PR is not created.
Describe alternatives you've considered
We currently work around this by using an additional step to run a bash script with gh pr view to get the ID and number, but it would be much easier if the action just output this natively.

@joshhunt already available in the latest Action version! v1.19.0 (v1 as well)


Update ui-personalization-windows-client.md

Resolve #142

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

File
Status
Preview URL
Details

dynamics-nav-app/ui-personalization-windows-client.md
:bulb:Suggestion

Details

dynamics-nav-app/ui-personalization-windows-client.md

Line 3, Column 14: [Suggestion-duplicate-descriptions] Attribute 'description' with value 'Learn how to customize the user interface to suit your way of working.' is duplicated in 'ui-personalization-manage.md(3,14)', 'ui-personalization-overview.md(3,14)', 'ui-personalization-user.md(3,14)', 'ui-personalization-windows-client.md(3,14)'.
Line 89, Column 25: [Suggestion-disallowed-html-tag] HTML tag 'Page' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.
Line 303, Column 257: [Suggestion-disallowed-html-tag] HTML tag 'UI' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.

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


Fails to build for wasm

Due to the quickcheck dependency tree having a dependency on an old version of memchr, this crate won't compile for the wasm32-unknown-unknown target.

It appears that this repo is unmaintained as the owner hasn't been seen in >2 years.
For posterity:

Use #25 to solve the quickcheck dependency for wasm.
If you encounter errors involving rustc-serialize (deprecated), fork and upgrade num to at least 0.2.0.

I've done both of these here. No maintenance guarantees. For your convenience:

git submodule add https://github.com/Sarvadi/glm-rs path/to/submodule
add glm = { path = "path/to/submodule" } to your Cargo.toml

Or fork it and do as you wish.


Simulator hangs or exits abruptly (again)

I've tracked several issues related to this problem is the past, and many attempts to fix it, but it seems it's not working for everybody (including me)
There was a critical change in commit 0ef671ef003c227c74fcfa0d308fe189ca4d8d9f to fix this issue, but it seems it worked for someones and broke it for others.
So, before the modification I found these related issues: #1922, #1971, #1976, #1977
After the modification I found these issues: #1983, #2487, #2315, #2412
I will tag some people involved in this issue: @jorisw @dlasher @Wheaties466 @tenaciousd93
It would be great to find a solution that works for everybody
Right now, I'm using node version 10.23, mongo version 3.2.11 and testing latest version of Zenbot unstable branch.
The command I'm using to test is:
node zenbot.js sim binance.BTC-USDT --strategy=noop --period=1m --days=7
For me, rolling back sim.js to version previous to the mentioned commit fixes the problem.
How can we procedd?

Try to use node 8 or 14, in my case, node 10 always makes the process hang.

I've tried with different versions and they behave differently, but with none of them I got a consistent behaviour. So, I think that's not a fix for this problem. My idea is to find the way to fix this issue and make it work for any version, as that is what stated as requirements for Zenbot.

I welcome anyone to try Zenbot paper or live mode, first against Binance, then against GDAX, and debug the code that was altered in the mentioned commit. The change was necessary for me to keep Zenbot running against Binance, unfortunately it seems to have broken continuous operation against GDAX. I for one unfortunately don't have time to debug is, but I am confident that the code can be made to work for all exchanges.

But the problem I'm reporting has nothing to do with paper or live trading. It's in simulation and the modification was inside sim.js, especifically in getNext() function.
Also, I'm using Binance too. I don't know if it's related to the exchange.
I've been debugging it and I've seen the same behavoir described in #2487 .

I think this is harder than expected. I've been tracking other recent related changes and found these other issues:
#2412 : simulation not working
#2425 : fixed something related to lolex and mongodb that fixed simulation
#2600 : reverted back last change because sims results were incorrect
I think this is highly related to the simulator problem. And tracking back when this lolex thing was added (#1456), I found it was to fix simulation results against live trading.
I agree the problem is being caused by lolex and the recent incompatibility with newer versions on mongodb, but it seems the fix proposed in #2425 wasn't fully correct or something else is missing to be done.

I think the "recent versions of mongo" issue should be addressed pinning the mongo version to a fixed one that its well known to work. This should be done in the docker-compose files. For example:
mongodb:
    image: mongo:latest

to:
mongodb:
    image: mongo:4.4.2

I'm also (still) getting this issue. It's definitely something to do with lolex <-> Mongo, but I'm having trouble narrowing it down. I can only get it to appear when I'm doing a lot of sims at once (eg. using the genetic backtester) which makes it difficult to get a debugger involved.
So far I've managed to narrow down exactly which settings are required for lolex to keep sims consistent:
// engine.js withOnPeriod function
    if (!clock && so.mode !== 'live' && so.mode !== 'paper') {
      clock = lolex.install({
        shouldAdvanceTime: false,
        now: trade.time,
        toFake: ['setTimeout', 'Date']
      });
    }

The lack of faking 'date' was what broke sims in the previous PR. (#2425) Unfortunately, the above doesn't actually fix the issue.
Switching to a lower mongo version and changing Node versions didn't appear to help me, neither did dropping the mongodb library version down to 3.6.1.  I even tried 3.5.0, but that slowed down sims to the point of being unuseable.
I've currently got a proof of concept replacing Mongo with Redis, so I'll see if this issue crops up there too.

After all night of messing with it, I'm no closer to fixing it, but I did implement a workaround to kill sims that seem to be stuck:
diff --git a/lib/backtester.js b/lib/backtester.js
index f225d30f..d1860983 100644
--- a/lib/backtester.js
+++ b/lib/backtester.js
@@ -228,6 +228,9 @@ let ensureDirectoryExistence = function (filePath) {
   fs.mkdirSync(dirname)
 }

+let simPercent = {};
+let simProcs = {};
+
 let monitor = {
   periodDurations: [],
   phenotypes: [],
@@ -366,6 +369,13 @@ let monitor = {
           slowestP = p
           slowestEta = eta
         }
+        if (!simPercent[c.iteration]) simPercent[c.iteration] = [];
+        simPercent[c.iteration].push(percentage)
+        let lastPercents = simPercent[c.iteration].slice(Math.max(simPercent[c.iteration].length - 30, 0));
+        if (lastPercents.length === 30 && new Set(lastPercents).size == 1 && simProcs[c.iteration]) {
+          console.log(`${c.iteration} is stuck at ${lastPercents[0]}, killing it off!`);
+          simProcs[c.iteration].kill('SIGKILL')
+        }

         if (homeStretchMode)
           inProgressStr.push(`${(c.iteration + ':').gray} ${(percentage * 100).toFixed(1)}% ETA: ${monitor.distanceOfTimeInWords(eta, now)}`)
@@ -543,6 +553,7 @@ module.exports = {
     var cmdArgs = command.commandString.split(' ')
     var cmdName = cmdArgs.shift()
     const proc = spawn(cmdName, cmdArgs)
+    simProcs[command.iteration] = proc;
     var endData = ''

     proc.on('exit', () => {

Absolutely not ideal, but it appears to only happen to about ~2% of my sims, so I'll take it for now...


Giver Staff Admin role permission to disable 2fa on user accounts

Description
As a staff admin, I need the capacity to disable 2fa on user accounts to better manage users' access to the platform. The staff admin role overseas user accounts and troubleshoot technical glitches via the WagTail admin in Hypha.
Solution
Staff admin role has permission to disable 2fa for all user accounts except for Admin

From research into 2FA management processes, these are the 2 workflows I've seen:
1. Disable 2FA for user affected

Possible design approach documented in Miro board.
2. Generate TOTP code for user affected

Possible design approach documented in Miro board.


opencv3: added CUDA 8.0 specific patches

Motivation for this change
Need this in order to add CUDA support for graphics cards beyond CC 3.5. This new version of OpenCV is compiled with a newer version of OpenCV 3, on order to allow this compilation to occur. This will bring us up to date with the repository around August. The CC support will now be up to 6.0.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

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

opencv3: reverted defaults

cc @viric

cc @flosse
Build is failing only because it doesn't have enough memory on the machine. Just tested the sandbox'ed build on my machine

Also cc @FRidh (think you reviewed or participated in my last OpenCV 3 PR)

We generally do not package random git checkouts (you'll find tons of packages that do, but that's no reason to add more). If you need this, I think it's better to prod upstream to make a release or use a local package override.

@joachifm, with the addition of CUDA 8.0 to nixpkgs, the OpenCV 3.1.0 build cannot detect the appropriate PTX code to generate for graphics cards with a Compute Capability past 3.5. With this specific commit, the system can detect the appropriate version to compile with. Should we really force people to pull from upstream specifically to gain this capability?
I'm just worried about people downloading CUDA 8.0 with a Pascal card, downloading the current OpenCV 3 derivation, overriding the package to enable CUDA, and then complaining that the algorithms/GPU bindings don't work.

I'm just worried about people downloading CUDA 8.0 with a Pascal card, downloading the current OpenCV 3 derivation, overriding the package to enable CUDA, and then complaining that the algorithms/GPU bindings don't work.

I see that as an argument for upstream (OpenCV) to make a release, not for nixpkgs to package an unreleased opencv version.

@bjornfor agreed. Can I make a custom .diff/.patch file to upgrade the 3.1.0 form, constructed from part of the code within master?
Or should I cherry-pick a specific commit to integrate that functionality in?

@mdaiter: I don't know the details of that particular feature to have anything to say about it.
But my general attitude is that upstream decides when features are ready for downstream consumption by making a release.
Have you asked upstream to make a release?

@bjornfor I have not. I will do that.

@bjornfor for now, I'm going to apply a specific commit patch file. Is that alright? It solves the problem reasonably (allows building up to 6.0, not 6.1, but as long as the major number is the same, we should be fine for building for Pascal and Maxwell architecture)

@bjornfor does this work better? The first patch is specifically to allow CUDA 8.0 to compile with OpenCV 3.1.0; the second, to allow for the detection of different (later-gen) graphics cards

@mdaiter: It still goes against my principles. Although I don't feel strongly enough about it to object a merge. It's just that this is a local fix (to nixpkgs) and I think it'd be better for everyone if upstream made a release. Because the latter would benefit all opencv users, not just the nixpkgs+opencv ones.
If someone wants to merge, that's fine.

@bjornfor understood. This is just the officially recommended fix for CUDA 8 compatibility from the OpenCV community, as stated here: https://github.com/opencv/opencv/issues/6677 . I concur that there really SHOULD be an update to the main branch (e.g. 3.1.1, etc.); however, this seems unlikely due to the release cycle OpenCV has historically displayed.
I can try to open an issue/request for an update to the main branch, but I can't guarantee anything. This goes against my principles as well, but there isn't really anything we can do about this ASAP. I'd rather have this stable fix in the repo so nixpkg users can use the CUDA bindings in OpenCV 3.1.0, regardless of platform (the Nix way!)

@bjornfor, I just asked on that OpenCV 3 issue whether a new branch could be created containing the first patch or a cherry-pick of that specific commit onto 3.1.0 could be performed.
As to the second patch within this build, I can open a request on OpenCV's page.

@mdaiter: Great!
BTW, do you know about overrideDerivation (https://nixos.org/nixpkgs/manual/#sec-pkg-overrideDerivation)? It seems like a good solution to use until this is solved upstream (either opencv or nixpkgs). You add it to ~/.nixpkgs/config.nix (or /etc/nixos/configuration.nix) and it applies to any nixpkgs version you use. IOW, you don't have to build from your nixpkgs clone to have the fix on your systems.

@bjornfor I'll probably do that! Thanks!

@bjornfor @mdaiter I think overrideAttrs is the preferred function now. It overrides the call to mkDerivation instead of the call to derivation if I understood correctly.

@FRidh: Ah, good! I remember seeing the PR but then I forgot about it. Good stuff. We just need to document it.

@bjornfor @joachifm @FRidh it appears as though OpenCV 3 uses tags instead of branches to manage their versions; therefore, it'd be difficult for them to append two new commits onto their 3.1.0 tag as rewriting history would cause massive rebasing. Therefore, @bjornfor 's solution should work for now.
If we won't merge this entire commit, can we at least merge the upgrade from GCC 4.9 to GCC 5? It does fix an error with compiling OpenEXR bindings.

I think this looks good now.

@FRidh I'm a bit confused: do you mean the commit looks good, or @bjornfor 's solution looks good?

The current changes.
It's unfortunate that upstream doesn't make a release. The changes in this PR don't seem to be breaking and as long as it indeed doesn't break opencv it should be, in my opinion, okay to merge.

@FRidh thanks so much! I don't have write access, so if you'd like to do the honour of merging, feel free :)


fix error in language codes of Telugu and Tamil in README

swapped language codes of Telugu and Tamil in README

later noticed that the codes for hindi, kannada, konkani and gujarati are also probably wrong

Thanks a lot for pointing out!
@JosephGeoBenjamin
It seems to be correct in the PyPI README:
https://pypi.org/project/ai4bharat-transliteration/
Did you recently change it here? If yes, please fix :)
Thanks,
Gokul NC

The Language codes Issue has been fixed. So closing the PR.


Navigation in activity log: "older" is falsely enabled

Hi there,
I stumbled upon a smaller bug in the activity log navigation. The buttons "newer" and "older" usually work correctly, but if you have exactly 10 (or a multiple of 10) entries in the log, the "older" button on the last page suddenly is enabled and clickable, resulting in the same 10 entries being loaded again.
I can reproduce this in your open demo, but a backport of the fix for this in the v1.10 branch would help us a lot. Thanks in advance,
best regards,
@cawolf

Hi @cawolf
Thank you for reporting the issue!
We greatly appreciate all contributions to Oro projects.


AppVeyor build fails

https://ci.appveyor.com/project/bacongobbler/draft/builds/30972088
build github.com/Azure/draft/cmd/draft: cannot load github.com/Azure/azure-storage-blob-go/2016-05-31/azblob: open C:\go\src\github.com\Azure\azure-storage-blob-go\2016-05-31\azblob: The system cannot find the path specified.
Command exited with code 1

Thanks for the report. I will be looking into moving CI over to Github Actions. We've had more success with that CI platform with other projects.


Support TCP_CORK, TCP_THIN_LINEAR_TIMEOUTS and TCP_QUICKACK

Hallo,
for my project I need to use the socket options TCP_CORK, TCP_THIN_LINEAR_TIMEOUTS and TCP_QUICKACK and I'd like to implement them for socket2. As far as I know these options are Linux only.
Disclaimer: I am quite new to Rust and this is my first PR for a Rust project.
I am not sure where to implement this stuff in https://github.com/rust-lang/socket2/blob/master/src/sys/unix.rs and https://github.com/rust-lang/socket2/blob/master/src/socket.rs. I found a unix only section but I don't know if these options are available for other Unix-like systems as well.
My first though is to put the option flags into https://github.com/rust-lang/socket2/blob/master/src/sys/unix.rs#L1004 and mark them with #[cfg(target_os = "linux")] and put the implementation into https://github.com/rust-lang/socket2/blob/master/src/socket.rs#L1445 and mark them also with the linux target.
The implementation itself would be more or less a copy/paste of nodelay/set_nodelay.
What are your thoughts about it?
Thanks

Hallo,
for my project I need to use the socket options TCP_CORK, TCP_THIN_LINEAR_TIMEOUTS and TCP_QUICKACK and I'd like to implement them for socket2. As far as I know these options are Linux only.
Disclaimer: I am quite new to Rust and this is my first PR for a Rust project.
I am not sure where to implement this stuff in https://github.com/rust-lang/socket2/blob/master/src/sys/unix.rs and https://github.com/rust-lang/socket2/blob/master/src/socket.rs. I found a unix only section but I don't know if these options are available for other Unix-like systems as well.

They should indeed for into the Unix only block here: https://github.com/rust-lang/socket2/blob/271fd2e878be19aab9bbd416ea18ed54ac31af61/src/sys/unix.rs#L1003-L1004 Please keep them in alphabetical order.

My first though is to put the option flags into https://github.com/rust-lang/socket2/blob/master/src/sys/unix.rs#L1004 and mark them with #[cfg(target_os = "linux")] and put the implementation into https://github.com/rust-lang/socket2/blob/master/src/socket.rs#L1445 and mark them also with the linux target.

That the right approach, also see set_mark: https://github.com/rust-lang/socket2/blob/271fd2e878be19aab9bbd416ea18ed54ac31af61/src/sys/unix.rs#L1264-L1284 Both Android and Fuchsia are Linux based/Linux compatible so try adding should to the target_os as well. If they don't support them that no problem either. You can use make check_all_targets for this: https://github.com/rust-lang/socket2/blob/271fd2e878be19aab9bbd416ea18ed54ac31af61/Makefile#L14-L20

The implementation itself would be more or less a copy/paste of nodelay/set_nodelay.
What are your thoughts about it?

Sounds good!

Thanks


Add installation to CMake

I see the installation part is still commented out:
https://github.com/osamu620/OpenHTJ2K/blob/main/CMakeLists.txt#L169
The README also so far mentions just the build process and no install. So I assume this is intentionally not ready yet :)
I just wanted to open this issue to keep track of this that I can get notified when it's ready.
I started to create a test package for openSUSE in my home repo. Just to experiment with it and later once everything is ready push it to the main distribution.

Hi @jubalh , thanks for raising this issue.
Because I'm unfamiliar with packaging stuff with CMake, yes, it has been intentionally commented out.
Your suggestion and/or knowledge are more than welcome :)

I will set put it on my TODO to check this :) Don't know quite yet when I will find the time.

FYI - I have filled the installation part (fe70b1b1879e59eac499ca323e07bd732245b65e).

Great! Thanks!
I updated the package and it builds and install fine.
I noticed that the imgcmp binary isn't installed. Is this intentional?

@jubalh
Yes, imgcmp is just a tool for conformance testing. I think tools like compare in ImageMagick are much more universal. Make sense?

Perfect :)


Support API namespaces in local discovery doc generation

Current coverage is 79.99% (diff: 84.61%)

Merging #35 into master will increase coverage by 0.07%

@@             master        #35   diff @@
==========================================
  Files            36         36
  Lines          5005       5083    +78
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           4000       4066    +66
- Misses         1005       1017    +12
  Partials          0          0

Powered by Codecov. Last update 36533c9...9829426


Update for maven plugin

Thanks for spotting the issue with ResourceTextFileReader. Your PR prevents the NPE that is happening when invoking the CLI from the maven plugin, however the resources such as the templates are still not correctly loaded according to my tests. The only solution I found so far that works both in the CLI and in the Maven plugin is to ask the WebFxCLI class to load the resources and always pass relative paths to it (absolute paths don't seem to work in the context of the Maven plugin for any reason).
Regarding the other changes (replacing inferred types with explicit types), they don't seem necessary to make the Maven plugin work according to my tests.
So I pushed the solution I mentioned above to the main branch (at this time of writing I think I should have created another branch... anyway it's too late).
Could you please check out this new version and confirm that the Maven plugin works with it?
Then we will focus on the maven plugin itself.
Thank you.

Hi Bruno, I’ve re-sync’d and built using the mvn command line for the webfx-cli project, all looks okay,I’ve updated the README.md on the webfx-maven-plugin project to add info on how to build and run on the command line from within a maven project. To test the template resource reading... 1. I created a ‘dummy’ project using command line:     mkdir webfx-example    cd webfx-example    webfx init org.example:webfx-example:1.0.0-SNAPSHOT 2. Inside the project, I tested template resource reading fixes by re-generating the same hello world project but via the mvn command (the command line below required a pre-existing pom.xml to work):mvn -Dargs=”init,org.example:webFX-example:1.0.0-SNAPSHOT” dev.webfx:webfx-maven-plugin:cli 3. I also tested the create application using mvn command (no spaces between commas in args) :mvn -Dargs="create,application,--prefix,webfx-example,org.example.webfxexample.WebFxExampleApplication,--helloWorld" dev.webfx:webfx-maven-plugin:cli        and that ran ok. The other webfx-cli code fixes ...I found that the code without my changes built on the command line, however when importing into Eclipse IDE, without my fixes I got a set of errors – maybe an Eclipse bug / config / source version issue. I’ve attached a screenshot below just for info (I may try another IDE at some point but will live with my patches locally for now)   Alex  Sent from Mail for Windows From: Bruno SalmonSent: 18 March 2023 21:36To: webfx-project/webfx-cliCc: Alex; AuthorSubject: Re: [webfx-project/webfx-cli] Update for maven plugin (PR #6) Thanks for spotting the issue with ResourceTextFileReader. Your PR prevents the NPE that is happening when invoking the CLI from the maven plugin, however the resources such as the templates are still not correctly loaded according to my tests. The only solution I found so far that works both in the CLI and in the Maven plugin is to ask the WebFxCLI class to load the resources and always pass relative paths to it (absolute paths don't seem to work in the context of the Maven plugin for any reason).Regarding the other changes (replacing inferred types with explicit types), they don't seem necessary to make the Maven plugin work according to my tests.So I pushed the solution I mentioned above to the main branch (at this time of writing I think I should have created another branch... anyway it's too late).Could you please check out this new version and confirm that the Maven plugin works with it?Then we will focus on the maven plugin itself.Thank you.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
The issue raised by Alex with ResourceTextFileReader when called from Maven is already fixed in the main branch. So I'm closing this PR.


Update Node version in README

As per https://github.com/gilbitron/Raneto/commit/b15568971f030632478e73a38724f8e3051a3012#commitcomment-14078134

I'm confused about this.
Is Node v4.x a requirement to function moving forward?
Or is this just a direction decision to not support anything below v4?

@ryanlelek Cannot answer the question, but Node 0.x is 'alpha/beta' anyway (according to semver), so not supporting anything below v4 is acceptable in my opinion. (NB: as you probably already know, it's technically "v4", but it's really just "v1", so everything below v4 is 0.x)

Ok, >= v4.x seems fine.
Worst-case scenario is a "WARN" in NPM for those legacy users


Make msgpack support optional

This can wait until after the release.
This makes YAML and msgpack support each options that can be selected independently (either, both, or neither).

I'm testing this now. How do I select neither?


Add check to avoid index out of range panics

@chipshort yes it actually happened in some tests in tgrade. I will add some unit test for it 👍

By double checking I'm not sure anymore if it is supposed to happen or not.  Let's keep it open for now

Closing as not relevant


Test migration client for symmetric key encryption

Description:
When migrating from an old version to the latest version, if the old version is using asymmetric encryption to encrypt internal data, we have provided a migration tool to convert those data to symmetric encryption.
The migrating data is identified in two flows.

Admin flows

Event publisher configuration passwords
Tenant keystore passwords
Policy subscriber passwords
BPS profile password
Kerberos security policy password
Secondary userstore password

User flows

TOTP secret key
OAuth tokens, authorization codes, client secrets
Workflow request credential

Completed testing migration of encrypted data to symmetric key encryption for both RSA/ECB/OAEPwithSHA1andMGF1Padding and RSA algorithms.
Focused scenarios for this migration got success.

Admin flows

[x] Event publisher configuration passwords
[x] Tenant keystore passwords
[x] Policy subscriber passwords
[x] Secondary userstore password
[ ] BPS profile password - A fix pending (https://github.com/wso2/carbon-kernel/pull/2758 and https://github.com/wso2/carbon-crypto-service/pull/21)
[ ] Kerberos security policy password - unable to test as STS is depricated

User flows

[x] TOTP secret key
[x] OAuth tokens, authorization codes, client secrets
[x] Workflow request credential

Following PRs contains fixes for some issues in the migration client.

https://github.com/wso2-extensions/identity-migration-resources/pull/133
https://github.com/wso2-extensions/identity-migration-resources/pull/134
https://github.com/wso2-extensions/identity-migration-resources/pull/132

Identified few issues and reported as following,

https://github.com/wso2/product-is/issues/9297
https://github.com/wso2/product-is/issues/9307
https://github.com/wso2/product-is/issues/9308
https://github.com/wso2/product-is/issues/9335
https://github.com/wso2/product-is/issues/9337
https://github.com/wso2/product-is/issues/9339


fipid in ecreq

Discussed with Vamsi and was okay in theory

@vaibhav-cams Marking this change to be released as v1.2.0.


Using FederatedSignIn in with aws-amplify-react-native

Hello. I couldn't find any examples with Federated Identities sign in using only the aws-amplify-react-native. Would it be possible to use FederatedSignIn of aws-amplify-react in a react-native mobile app? If yes, what would be the best way of doing this? If not what's the alternative? Thanks

@marioskonstantinou Federated auth is currently in our backlog and should be available within the next couple releases, if not next release.
Alternatively, the cognito js sdk provides a federated example (federated identities go on the "Logins" parameter): https://github.com/aws/amazon-cognito-identity-js, see use case #4, and docs here: http://docs.aws.amazon.com/cognito/latest/developerguide/cognito-user-pools-identity-federation.html.
Currently federated identities is in this sprint, so we are hoping to get this out within the next couple weeks!

Any updates on this functionality?

when is federated identity likely to be introduced for aws-amplify-react-native?

Seems to be released, but I am experiencing token expiration issues with FB logins. The fb token has a long expire time, but for some reason, getting "Access Denied" on my API requests after a little while. Almost seems like the aws token is failing to refresh my fb access token.


Is project still alive?

Hi,
While using node-disk-info, I stumbled upon a bug with volumes containing a space in Darwin/Linux. So I created an issue, and since it was quite an easy fix, made a PR... Before noticing there was already a PR with the very same fix that has been ready to be merged since more than a year.
So I wonder: is the project still alive?
In the meantime I created a fork and integrated the fix in it, but I'd like to avoid publishing another package on npm just to have this fix applied (npm is full of such packages).
If you have no time or other interests (maintaining software takes a lot of time) would you consider handing over this work to someone with more time?

I Will try to contact him tonight on another platform to see if he is ok to add another maintainer to the project.

@cristiammercado can you please do something for this repo ?
Thanks.


org.parceler.ParcelerRuntimeException: Unable to create ParcelableFactory

I don't know why happens this problem.
Can you review my code, I really think this is correct. wtf...
https://github.com/ppamorim/Amazing-MVP/blob/master/amazingMVP/src/main/java/com/amazing_mvp/domain/model/Genre.java

Hmm, not sure.  Maybe proguard is removing the generated classes?

proguard is disabled.

Looking at your Genre class, I don't think you need the BEAN configuration as you have also configured the fields to be parcel properties.

WIth or without BEAN. It still not working.

@johncarl81 I've added a test class and guess it! Still giving error.

@johncarl81 I've found the problem.
Please remove the "provided" from readme and add "apt". "provided" never works with me.

Relevant discussion on provided vs apt: #87.  Provided should work.


Create TempleOSRSPlugin

Does the same thing the CML plugin used to but sends it to the new TempleOSRS site

Superseded by https://github.com/runelite/runelite/commit/553e56862927f8591fd37ad3e86425e640b23eaf

closing since RL master has this now


Font-Face doesn't appear to work on Windows. (Works on Mac)

Our site uses @font-face fonts. When using html2canvas everything works great on the Mac, but on windows all text comes out in a default serif font. Here is a fiddle that demonstrates this (http://jsfiddle.net/omne8o2L/1/). On the Mac both lines will be "Raleway Bold" (A San-serif font), on windows the top line (Real text) will be formatted correctly, but not the image.
Here are screenshots of the results of the Fiddle.
Mac (Correct)

Windows (Incorrect)

I've tested this in Firefox and Chrome on Windows and get the same result. Any ideas?

Upon further investigation and another JsFiddle (http://jsfiddle.net/omne8o2L/3/) turns out the bug we're experiencing is due to the letterRendering option being turned on.

Is this still an issue with v1.0.0? If so, could you please share an example on jsfiddle.

I'm having the same issue.