(R4R) Fix infinite looping

@pcstout, @afiedler - FYI

@afiedler, @pcstout - R4R... let me know if you want to hangout & discuss this.  I can provide a high-level overview or we can go into the nitty-gritty details.  I believe this change will prevent the unresponsive dyno issue.  There are other issues with this gem, though, and we should discuss them.

@marlonnelson Reviewed.

@pcstout, @afiedler - spec failures... I didn't notice them, thanks to Circle retrying a build that I cancelled.  looking into it now.

@pcstout, @afiedler - that was an easy-peasy fix - R4R
BTW, I ran a loadtest on staging, flooding it with 3000 requests to copy a minitask (Chalk Talk).  Previously, I could reliably generate the stuck dyno with 1000 requests.  With this fix, the dynos did not get stuck in the infinite loop.  :relieved:

:+1:


Fix build for non unix platforms

I am not sure this is the most idiomatic way to deal with this error, since I have never used error_chain before. In case there is a better way please tell me so I can update my code.
By the way, I think this project needs an AppVeyor build to make sure the build on Windows doesn't accidentally break.

bors r+

Not sure if windows support matters much, when it simply doesn’t work there anyway.

I didn't know that... I stumbled upon a blog post by @Manishearth and it looked dead simple so I decided to give it a try. I guess we should at least mention in the readme that only Unix platforms are supported.

We do.


admiraldev does not export negate_vars

I'm trying to explore a potential problem with the dev version of stringr, but I can't get your package to build because you have
#' @export
admiraldev::negate_vars

but negate_vars is not exported by admiraldev (https://github.com/pharmaverse/admiraldev/blob/main/NAMESPACE).

I'm still seeing the problem with the main branch of admiral.
Anyway, perhaps you can diagnose if dev stringr is causing problems with admiral, since I can't seem to load your package.

Sorry - our updates are in the devel and pre-release branches.  Going out on Monday to CRAN and once accepted to CRAN we will merge into main.
We can test out the changes for dev-stringr and let you know of anything untoward.

@hadley  I don't see a dev branch on stringr github page?

The development version of stringr is in the main branch (this is standard practice).


Found that the example's element mkdirp is deprecated

File: get-started/part2.md

Thanks for sharing the issue. The getting started guide has been significantly updated and it appears that the issue no longer exists. If the issue still does exist, please let us know.


Ability to exclude

I'm generating a sitemap without specifying a lastMod intentionally since that property is optional, but the library is adding it regardless. Can we add an option to not include  <lastmod> unless explicitly set?

Fixed in 3.0.0.
If you're not specifying a url.lastMod option, then when sitemap.xml is produced, we no longer automatically include today's date.
So this:

https://bitmidi.com/3
2020-11-22

Becomes this:

https://bitmidi.com/3

If you want the old behavior, then use a url object like this:
{
url: '/3',
lastMod: true
}


Uncaught ReferenceError: window is not defined

when I set worker-loader in creat-react-app , I received this error waring
the codes are all the same with the official sample;
"worker-loader": "^2.0.0"
"webpack": "4.19.1",

@snoopy1412
add  globalObject : 'this'
You can go look   https://github.com/webpack/webpack/issues/6642

As this is not the default option in webpack, it would be nice to see this in the documentation.
I case anyone is wondering, globalObject option goes in the output options in webpack config.
For example:
output: {        path: path.join(__dirname, 'dist'),        filename: 'bundle.js'        publicPath: 'http://localhost:3000',        globalObject: 'this' },

Came up against this when trying to implement the final comment on this issue with CRA:   https://github.com/facebook/create-react-app/issues/1574
For anyone that happens to find this that is trying to use EpubJS in create react app and can't get the service worker imported properly, the ONLY thing that worked for me was to eject the app from CRA and add globalObject: 'this' to the webpack config myself.
If you're reading this out of desperation, I feel your pain... god speed.

so in case anyone bumps into something similar, our worker broke because a file that it depended on was importing something that in turn imported CSS, and the style-loader uses the window object.
very silly regression. but hey. it happened.

This problem seems to be the same one as #142.

Close in favor #174


Create a "concept note" feature that goes before the funds

Applicants can then be invited to do a proposal in any of the existing funds
This would be used for OTFs "one button" workflow.

This doesn't need to be an issue - I think it's better to make this into a discussion as it is just a discussion point at the moment.
I'm going to close this and move it to discussions.


Port NSString fix from overlay

Cherry-picks https://github.com/apple/swift/commit/7322b63c4495e3b32f8197fd26307f948f7b3148#diff-550fd197960077d36f395b46e5a9f445 from the overlay.

@swift-ci please test


reporting the wrong resolution

Description
neofetch is reporting the wrong resolution

[ x] Does this issue still occur in the master branch? (Required if issue)
Yes, copy pasted it to my own system

Neofetch version
7.1.0
Screenshot

Config file
None
Verbose log
neofetchlog.txt

neofetch doesn't support connecting wayland display server. You may try fastfetch instead.


Reduce pull quote padding by 30% on top and bottom

Top + bottom margin - pull quote: reduce by 30%
On desktop - I'm having trouble giving clear numbers because of the coding, but let's do 65px total on top and bottom (so adjust the top padding down to 41px; plus the paragraph padding 24px).  For the bottom margin, aim for 65px total on

On ipad and mobile, the top margin should increase to match the bottom margin.

@pfulton - giving this a nudge instead of adding a new issue. Let me know if you have any questions. Lots of math in this one. 🤦🏻♀

Just adjusted this via our conversation in Slack! Will be closed via #222


Decorator lines for modified/added lines without SCM

I am not using SCM, but I would like to see decorators in the gutter for modified/added/removed lines for any locally opened file from disk. I have used (and still using) some text editors that allow similar thing without any Source control or Timeline or anything fancy like that - just open any file, start editing it, and these editors will show which lines are modified/added, like this:

or this:

Related to #80870
@bpasero does the new Local History feature open up any new opportunities here?

Just to explain. With the editors on these screenshots you just open any file from disk, even for the first time without any previous saving, and right when you start modifying it you get decorators for modified/added lines.
Besides, I cannot get Timeline and Local History to show me these decorators no matter what I tried. The animated screenshot for Local History is not very informative and maybe I am doing something wrong.

Thought there was a duplicate, please try to find it.

I tried to find it before reporting this issue, but didn't find anything similar. I am sorry if I missed it.
Please note that my suggestion is completely unrelated to SCM and Timeline/Local History, and I would remove that timeline-local-history tag. I don't want to create a backup of an existing file using Local History, just to be able to see decorators when I start modifying it.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Add 'controller' to README

merged in https://github.com/thoughtbot/ember-cli-rails/commit/abd8901b302b56bf14ed4621fd124111512fb0f9.
Thanks!


How to fill labels dynamically according to raw api data in chartistjs in vanillajs?

Hey there,
I'm pretty sure someone might have asked this (if so please direct me there).
I've looked over the documentation of chartistjs, but did not find any satisfactory answer.
So I have created a line chart (which works beautifully). However, the documentation has hardcoded the values for labels field followed by series like so:
var chart = new Chartist.Line('.ct-chart', {
labels: ["1", "2", "3", "4", "5"],
series: [{
name: 'series-1',
data: [0, 2, 4, 8, 10, 8, 4, 2, 0, -2, -4, -4, 0, 4, 8, 4, 2, -5, -4, 0]
}, {
name: 'series-2',
data: [-2.5, 0, 2, -2, 0, 2, 4, 8, 4, 2, 4, 8, 10, 8, 2, 0, -2, 4, 8, 10]
}, {
name: 'series-3',
data: [-5, -5, -5, -5, -4, -2, 0, 2, 4, 8, 10, 10, 7, 8, 2, 0, 0, 0, 0, -3]
}]
}
However, since I'm getting raw data from an API, I'd like to set the labels field according to the data taken from API, which means any number of labels can differ.
So I have this method which filters my data:
var getLbl = function(callBack) {
let labels = []
DashBoard.CachedAPI.GDP(
2010, 2015, i => {
i.filter(j => j.country.value === "Arab World")
.forEach(p => callBack(p.date))
}
)
};
Let's assume the response for the method returns: 2010, 2011, 2012, 2013, 2014, 2015
How can I take each value above and place it in the labels field?

Hello @bitabs 👋
Chartist has recently been updated to version 1.0 with some new exciting changes like ESM and TypeScript support. The migration guide can be found right here.
Please let me know if the question is still relevant to you so we can further look into it!


fix: merge generation of input-data for issue_closures and list_repos

Description

[x] fix #64
[x] combine the Producer functionality for issue_closures and list_repos into one producer

[x] fix associated consumers

[x] add some clarifying comments & doc-comments throughout

r? @rylev

Changes since last review:

[x] removed IssueClosure struct
[x] fixed comment nits
[x] attempted parallelization of the various count queries... but hoping to have some more experienced eyes on that :smile:

--replay-graphql bug

I found an issue with the --replay-graphql... not sure if this is a bug I introduced here or if it existed.

still need to dig deeper and figure out why this is happening

the changes in this branch are working great without the --replay-graphql option; but as soon we run it WITH that flag, the GQL query is loading response data from the same file for both metrics::util::count_pull_requests and metrics::util::count_issues

running with RUST_LOG=debug demonstrates this issue well

expand to see logging output
❯ RUST_LOG=debug cargo run -- report data/optopodi/2021-07-31
   Compiling optopodi v0.1.0 (/Users/chazadmin/code/opensource/github-metrics)
    Finished dev [unoptimized + debuginfo] target(s) in 7.84s
     Running `target/debug/optopodi report data/optopodi/2021-07-31`
[2021-07-31T18:13:32Z DEBUG reqwest::connect] starting new connection: https://api.github.com/
[2021-07-31T18:13:32Z DEBUG reqwest::async_impl::client] response '200 OK' for https://api.github.com/graphql
[2021-07-31T18:13:33Z DEBUG reqwest::async_impl::client] response '200 OK' for https://api.github.com/graphql
[2021-07-31T18:13:33Z DEBUG reqwest::async_impl::client] response '200 OK' for https://api.github.com/graphql
[2021-07-31T18:13:33Z DEBUG optopodi::metrics::util] Fetching issue closure info for optopodi/optopodi
[2021-07-31T18:13:33Z DEBUG reqwest::async_impl::client] response '200 OK' for https://api.github.com/graphql
[2021-07-31T18:13:33Z DEBUG optopodi::metrics::util] Fetching issue closure info for optopodi/optopodi
[2021-07-31T18:13:33Z DEBUG reqwest::async_impl::client] response '200 OK' for https://api.github.com/graphql
[2021-07-31T18:13:33Z DEBUG optopodi::metrics::list_repos] Retried issue closure info for optopodi/optopodi: IssueClosuresCount { opened: 2, closed: 5 }

❯ RUST_LOG=debug cargo run -- --replay-graphql report data/optopodi/2021-07-31
    Finished dev [unoptimized + debuginfo] target(s) in 0.15s
     Running `target/debug/optopodi --replay-graphql report data/optopodi/2021-07-31`
[2021-07-31T18:13:35Z INFO  optopodi::metrics::gql] loading response data from `data/optopodi/2021-07-31/graphql/all-repos/0.json` rather than github
[2021-07-31T18:13:35Z INFO  optopodi::metrics::gql] loading response data from `data/optopodi/2021-07-31/graphql/repo-participants/0.json` rather than github
[2021-07-31T18:13:35Z INFO  optopodi::metrics::gql] loading response data from `data/optopodi/2021-07-31/graphql/repo-infos/0.json` rather than github
[2021-07-31T18:13:35Z DEBUG optopodi::metrics::util] Fetching issue closure info for optopodi/optopodi
[2021-07-31T18:13:35Z INFO  optopodi::metrics::gql] loading response data from `data/optopodi/2021-07-31/graphql/repo-infos/0.json` rather than github
[2021-07-31T18:13:35Z DEBUG optopodi::metrics::util] Fetching issue closure info for optopodi/optopodi
[2021-07-31T18:13:35Z INFO  optopodi::metrics::gql] loading response data from `data/optopodi/2021-07-31/graphql/repo-infos/0.json` rather than github
[2021-07-31T18:13:35Z DEBUG optopodi::metrics::list_repos] Retried issue closure info for optopodi/optopodi: IssueClosuresCount { opened: 5, closed: 5 }

is it possible that would happen due to the multi-threaded context?

@bors r=rylev

@bors r+


Add Cordon and Drain Related Admin Endpoints for New Geneva Actions

This PR creates admin endpoints that can be used to cordon, uncordon, and drain ARO cluster nodes. New endpoints will be targeted by new Geneva Actions (Phase 3)
Which issue this PR addresses:
USER STORY 13912018 - Create Cordon or Uncordon Node Geneva Action
USER STORY 13912064 - Create Drain Node Geneva Action
What this PR does / why we need it:
This PR adds admin endpoints that can be used to cordon, uncordon, and drain ARO cluster nodes. These endpoints will eventually be targeted by new Geneva Actions that have been requested by ARO support (Phase 3 Geneva Actions).
Test plan for issue:
Unit tests have been added for each new endpoint. I also ran the RP locally, created a cluster, and manually...

curled the new "cordonnode" endpoint to confirm the schedulability of cluster nodes updated as expected
curled the new "drainnode" endpoint to confirm cluster nodes drained appropriately

Note that E2E tests should be added for each endpoint as these are potentially destructive actions, however I personally have no way of testing E2E locally at the moment. CloudFit/MS/Red Hat are still working through simply secure issues affecting the ability to run E2E tests locally. No ETA on a resolution, but didn't want that to prevent getting more eyes on this PR.
Once E2E tests can be developed and tested locally in simply secure environments,  either this PR will be updated or a separate PR will be created specifically for those tests - whichever Red Hat prefers.
Is there any documentation that needs to be updated for this PR?
No - once the Geneva Action frontends have been created, that documentation will be updated to include the functionality of these new endpoints.

Happier with this version - ready for review

want to point out that I updated the timeout for the drainer for the drain node endpoint - during this latest round of testing I would hit the 60 second timeout consistently with an "empty" cluster

/azp run ci

/azp run e2e

Demo of cordon and drain admin api endpoints uploaded to: https://microsoft.sharepoint.com/:v:/t/AzureRedHatOpenShift/Ee8_hG5ODjRDqH8660XS8aQBVAIirHaORR7NmCpeKM_hPg?e=lleJhZ


Any drag and drop / sortable implementation possible?

Hi, I'm wondering if someone has ever implement a masonry drag and drop / sortable feature, or has some informations. Thank you in advance.

Badly in need of this feature, has anyone has more info on this.

Open to taking a PR on this, I just haven't had time to work on this project as I'm not using react-native often these days!


ciscoise_internal_user import results in Error

Prerequisites

[ yes ] Have you tested the operation in the API directly?
[ yes ] Do you have the latest Terraform provider version?
[ yes ] Review the compatibility matrix before opening an issue.

Describe the bug
ciscoise_internal_user import unconditionally results in Error: Failure when setting GetInternalUserByName response
parameters.0.custom_attributes: '' expected type 'string', got unconvertible type
'isegosdk.ResponseInternalUserGetInternalUserByNameInternalUserCustomAttributes', value: 'map[]'

Tested with different ISE deployments, different users, import by name or id. The same error is shown when enable_auto_import is true.
Expected behavior
Expected successful import of internal user
Screenshots

Environment (please complete the following information):

ISE version and patch: 2.7 p6, 3.1 p1, 3.2 p1
Terraform version: v1.3.9
ISE provider version: v0.6.18-beta
OS version: Windows 10 Pro 22H2

Additional context
currently no known workaround exists

Hi, @criedl01 please try it again with the new provider version and let me know how does it go.

closed due inactivity


Problem downloading configuration file

When I select to download "Universidad Rey Juan Carlos" configuration file, appear an error reading this file. ¿Any suggestion?

For which operating system was that?

Sorry,
Android 8.0.0 at Huawei P10

When the error appears, you can go to the web and download and install the file correctly using the app.

Ok, we've had more than one report about this. I believe the developer is already aware.

Hello,
I believe i have resolved this issue with the latest build. But it is not yet available to users. I am hoping to publish it next week, I will update this issue when i do for you to test.
Gareth

Hello,
The latest release is available to users now, so i believe this issue is resolved, I will close it now until i hear differently.

we need to open again this issue.
When I select to download "Universidad Rey Juan Carlos" configuration file, appear an error reading this file. ¿Any suggestion?

thanks in advance

Data point: the user wrote in another issue that he sees the issue on Huawei devices with Android 7 only.
With an Android 7 on a Samsung device, I see no issues with the configuration profile of Rey Juan Carlos.

Addendum: the exact devices were:
Huawei Media Pad M3
Huawei P10


Fix buffer overlow in case method docstring

Prior to this commit the add_docstring function had a
buffer overlow issue that was causing memory to be
clobbered.
This commit changes the add_docstring function to
use a printbuf to accumulate the string ensuring
no more buffer overflow issues.
Fixes #1609

@jemc Can you please test/confirm that this PR resolves #1609 for you?
In my testing, this resolves the error I get the same as the other two changes I mentioned in the ticket.

@dipinhora Yep, this solves the issue - sorry to assume that your hashmap implementation had to be the source of the problems - I guess the memory issue here was spilling over into the memory of the hashmap and causing the assert failures there.
Thanks so much for figuring it out!

@dipinhora mea culpa; I wrote that funcion.  Can you point out the buffer overflow to me for my re-education, please?

@jemc No problem. It took me many hours split across multiple days until I finally got lucky and figured out the real issue (which I only discovered because I switched to using malloc/free in pool.c along with using valgrind to try and ensure I wasn't doing something incorrect within the resize function of hash.c).

@kulibali No problem. It happens.
The following is what I understand to be occurring in the old code:

print_params is always passed a buffer of 128 bytes with no checking to see if the concatenated string has overflowed the 128 bytes or not as more and more bytes are copied in resulting in overflow. This is because the last argument to strncat is supposed to be the # of bytes to copy and not the total size of the buffer. The code would have been correct if it was using strlcat instead of strncat. See: https://linux.die.net/man/3/strncat and http://stackoverflow.com/questions/6903997/how-can-i-use-strncat-without-buffer-overflow-concerns
add_docstring calculated len using:const size_t len = id_len + 2 + m_ds_len + p_ds_len + c_ds_len + 2;

Adding up to 8 extra bytes total (4 from the line quoted and another 2 as part of m_ds_len and the last 2 as part of p_ds_len, but all the string literals used potentially add up to 10 extra bytes (2 for "\n\n", 1 for "`", 1 for "(", 1 for ")", 2 for ": ", and 3 for "`: "). There's also a possibility where the 2 extra bytes from p_ds_len are not added to len if p_ds_len = 0 but the string literals those 2 extra bytes represent are added regardless of the value of p_ds_len.
This means the buffer could be undersized in some circumstances. Combine this with the previous note about strncat and the buffer can overflow.

In the process of rewriting the logic for print_params to automagically resize the memory allocated, if needed, I noticed that printbuf is used in other passes and so decided to use that instead. Switching both functions to use printbuf simplifies the implementation and also resolves buffer overflow and potential string truncation on concatenation concerns.

Thanks!  I should have noticed the use of printbuf elsewhere.

Thanks again!


DBServiceActor is failing with Timeout Exception

In the DBServiceActor.scala,  we pre-start it by reading the DBService method to fetch all active power plants. But this message is never received in the receive block of this actor as there is no match case to pattern match this message type.
This message needs to be processed in the receive block and the actor has to be context becomed so that successive messages are handled.
Here is the exception from the logs:
[INFO] [08/11/2017 13:57:59.390] [application-akka.actor.default-dispatcher-4] [akka://application/user/$a/$a] Pre-start DBServiceActor
[INFO] [08/11/2017 13:57:59.402] [application-akka.actor.default-dispatcher-4] [akka://application/user/$a/$a] done with pre-start DBServiceActor *****
akka.pattern.AskTimeoutException: Ask timed out on [Actor[akka://application/user/$a/$a#-1046088460]] after [5000 ms]. Sender[Actor[akka://application/user/$a#-293595750]] sent message of type "com.inland24.plantsim.services.database.DBServiceActor$PowerPlantEvents$".
	at akka.pattern.PromiseActorRef$$anonfun$1.apply$mcV$sp(AskSupport.scala:604)
	at akka.actor.Scheduler$$anon$4.run(Scheduler.scala:126)
	at scala.concurrent.Future$InternalCallbackExecutor$.unbatchedExecute(Future.scala:601)
	at scala.concurrent.BatchingExecutor$class.execute(BatchingExecutor.scala:109)
	at scala.concurrent.Future$InternalCallbackExecutor$.execute(Future.scala:599)
	at akka.actor.LightArrayRevolverScheduler$TaskHolder.executeTask(LightArrayRevolverScheduler.scala:331)
	at akka.actor.LightArrayRevolverScheduler$$anon$4.executeBucket$1(LightArrayRevolverScheduler.scala:282)
	at akka.actor.LightArrayRevolverScheduler$$anon$4.nextTick(LightArrayRevolverScheduler.scala:286)
	at akka.actor.LightArrayRevolverScheduler$$anon$4.run(LightArrayRevolverScheduler.scala:238)
	at java.lang.Thread.run(Thread.java:745)

Kind of fixed it. So I'm now passing in the SupervisorActor reference to the DBServiceActor so that the DBServiceActor can push messages into the SupervisorActor instead of the SupervisorActor asking for events from the DBServiceActor. So this way, I can avoid any nasty timeouts!


Forking for a different kind of project

hey Vinduv, I'm trying to make a custom switch controller using an arduino (by controller I don't mean like a bot, I mean the actual controller you hold in your hand to input buttons and controls into the switch).
Were you able to get your arduino bot working with the switch? If you could that means I could use this method to make a simple custom arduino-switch controller right??

Hi! Thanks for your interest in this project.
To answer your question, I have used of my project on a real Switch and it actually works, without issues. (Though I haven't used it for quite a while now so I hope it has not been broken by a Switch software update...)
If you want to use my project, you can write an "automation" code that will poll the state of the buttons/axis and send it to the USB interface handler, in a loop. Note however that this will introduce a latency. Not a problem for automation, but a bit more for a real controller...
If you can use another microcontroller, I'd suggest the Teensy 2.0++ (or any microcontroller supported by LUFA). Its avantage over the Arduino Uno is that the same microcontroller can communicate over USB and read/write the I/O pins (on the Arduino is is the task of two different microcontrollers). You can use this code, modifying it to get the state of real buttons/axis in refresh_and_send_controller_data instead of using a static report_data table.
Don't hesitate to ask if you have more questions.

Hey Vinduv,
Thanks for getting back to me. That's super cool how you got the bot
working!
I'll definitely look into getting the Teensy controller instead of the
arduino uno for it's io pins. I think though first I need to understand how
to program using C xP. (So far I only have experience using the Arduino
IDE.)
I plan on starting this project maybe a month from now, so I'll hit you up
if/when I run into any problems. Thanks again for your help!
Arthur
On Wed, Dec 2, 2020, 5:34 AM VinDuv<EMAIL_ADDRESS>wrote:

Hi! Thanks for your interest in this project.
To answer your question, I have used of my project on a real Switch and it
actually works, without issues. (Though I haven't used it for quite a while
now so I hope it has not been broken by a Switch software update...)
If you want to use my project, you can write an "automation" code that
will poll the state of the buttons/axis and send it to the USB interface
handler, in a loop. Note however that this will introduce a latency. Not a
problem for automation, but a bit more for a real controller...
If you can use another microcontroller, I'd suggest the Teensy 2.0++ (or
any microcontroller supported by LUFA). Its avantage over the Arduino Uno
is that the same microcontroller can communicate over USB and read/write
the I/O pins (on the Arduino is is the task of two different
microcontrollers). You can use this code
https://github.com/VinDuv/switch-arduino-controller/blob/master/src/usb-iface/standalone-usb-iface.c,
modifying it to get the state of real buttons/axis in
refresh_and_send_controller_data instead of using a static report_data
table.
Don't hesitate to ask if you have more questions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/VinDuv/switch-arduino-controller/issues/1#issuecomment-737233087,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALVCNAFXI6QUPJFLMYS5DITSSY66VANCNFSM4UHGV3EQ
.


protocol with init definition can not be mocked

The following is generated without the initializer function and therefor does not conform to the protocol.
protocol SomeProtocol {
init(value: Int)
}
I think it would be okay to just generate the init function for a protocol mock and have it discard the arguments and/or call the no arguments init.
Please share your thoughts, I'll create the code and pull request for it.

@TadeasKriz fixed


fix empty slice declaration via literal

I want the empty slice construction here since there is a change a query might return an empty result. The construction helps to document that.

@ardan-bkennedy ok, got it. Thx for your training!


[#1] Fix commits were duplicated on update

https://github.com/notwaldorf/tiny-terminal-care/issues/1
Due to the mechanism which was used to update the codes
(appending data to the existing and then calculating the length of the result)
the commits were duplicated on update, as on update we received the same commits
as we had before it and thus the length of the commits array became
l = l_prev + l_prev + l_new

where

l is the total number of commits displayed,
l_prev is the number of commits before update
l_next is the number of new commits

Cleaning the content of the commits boxes right before updating it
resolves the issue.
Close #1

Thanks so much!


Version bump to 2.1.0

Auto-generated by fastlane 🤖
Changes since release '2.0.5':

Update README.md (#7385)
Fix not showing the lane list and selection when not passing any arguments (#7549)
Add "--fail-on-errors" support to Danger action (#7487)
Updated docs description for pilot/testflight action (#7550)
fix platform (#7360)
[gym] Remove promotion for use_legacy_build_api (#7379)
user_error if plist error (#7547)
more tests for aliases (#7546)

Since my Ruby is broken, can someone else take over that release? 👍


Question regarding BucketIterator

hi, I'm apporaching sentiment analysis with torchtext and I've recently been studying the concept of Iterator. From what I understand it is used to automatically convert strings in vectors, batching them (that is, getting the set of vectors that shall be used for training) and then move them to the computing device.
I saw that BucketIterator tries to get a batch in which all the sentences have similar length, to reduce the amount of padding. My question is: if a sentence is shorter than the fixed length it is padded, but what if a sentence is longer? Is it truncated? If yes, how exactly?
Thanks in advance.

An Iterator will sample a batch of sentences, each will (usually) have different lengths, say: 32, 25, 10 tokens in each sentence. It will then look at the longest sentence within the batch, here 32 tokens, and then all sentences shorter than that will be padded to that length, 32 tokens. This will be done by appending a pad_token, which is <pad> by default, to the end of the sentences.
The problem with the standard  Iterator is that it creates batches by sampling sentences from the dataset randomly. This means we'll probably have batches of sentences with wildly different lengths and thus some sentences will have a lot of padding, e.g. in the above example the sentence with 10 tokens will have 22 padding tokens appended to the end of it. If we're using an RNN/LSTM type model, this means the last 22 time-steps on that sentence will just be over padding tokens - which is wasted computation.
The BucketIterator helps with this problem by trying to give you batches where each sentence is roughly the same length, i.e. a batch of lengths 32, 30, 28 is more likely than a batch of 32, 16, 10. This cuts down on the amount of padding within the batch, which means less computation over padding tokens and leads to less computation time overall.
The exact way the BucketIterator works is given in the code here. It buckets your dataset into buckets of size 100 * batch size, sorts each bucket by their length (using the sort_key), then cuts each bucket into 100 batches of batch_size each, then yields a random batch from a random bucket. These random batches have been sorted by their length, so usually have similar sentence lengths and thus less padding.
I'm not sure what you mean by "fixed length" in your question above. There is no set fixed length to pad to, padding is always done so all sentences within the batch are equal to the longest sentence.

Ok now I understand this. Than you for the time spent explaining it!

Is there a way for having the same feature on a regular DataLoader ?


Define predicate in Proof Request that checks expiration date and returns true/false

I have a credential in my wallet that contains property "licenseCategoryB_DateOfExpiry" and it has value for example:
{         "name": "licenseCategoryB_DateOfExpiry",         "value": "2028-02-25"     }
Once I send proof request that requests licenseCategoryB_DateOfExpiry I would like to just return true/false depending on the current date, not whole value (2028-02-25). Is there a way to do that with Predicates?
I saw couple of examples online with age but in that case there is only single integer and <,>,>=... Is there something like that for date type?
Thanks! :)

The "easy" way to do it today in anoncreds is to make your date data a number (not a string), so that in your case, the value would be the integer 20280225.  If you do that, and in the proof request, the verifer asks that licenseCategoryB_DateOfExpiry > 20220128, the predicate will return true in zero knowledge.  This convention is described here: https://github.com/hyperledger/aries-rfcs/tree/main/concepts/0441-present-proof-best-practices#dates-and-predicates  The nice thing about this approach is that at least the integer looks to a person like a date  -- much better, for example, than a Unix epoch representation.
Why this way?  In AnonCreds, the signed data is an integer encoded based on the "raw" data being asserted. Thus, there must be agreement between the issuer and verifier about how the "data to integer" encoding is done for signing by the issuer. Currently, that encoding algorithm is very simple -- integers are unchanged, everything else is a string, which is hashed to an integer.  Thus, the only data that can be used as a predicate is an integer. The convention above is helpful given those limitations.
We are starting on the path of standardizing AnonCreds and one possibility in the future is adding in more rules in to the encoding, so that a date string might be encoded as a sortable integer, and that a presentation predicate proof would handle the expression in the same way.  But that's not there yet.

The "easy" way to do it today in anoncreds is to make your date data a number (not a string), so that in your case, the value would be the integer 20280225. If you do that, and in the proof request, the verifer asks that licenseCategoryB_DateOfExpiry > 20220128, the predicate will return true in zero knowledge. This convention is described here: https://github.com/hyperledger/aries-rfcs/tree/main/concepts/0441-present-proof-best-practices#dates-and-predicates The nice thing about this approach is that at least the integer looks to a person like a date -- much better, for example, than a Unix epoch representation.

Note the attribute name SHOULD have the suffix _dateint
The alice/faber demo includes this:  https://github.com/hyperledger/aries-cloudagent-python/blob/main/demo/runners/faber.py#L109


Don’t leave temporary files around, d’oh!

Related: #22

Oof, I can't believe I missed that.

That makes us two!

I'm guessing you had that there as a debugging feature. Do you think we should make that conditional rather than removing it entirely?

Nah, to benefit from it in a production environment, we’d have to have a way to expose it, too (in the CLI and in a Koji or mock plugin). And for working on the sources, it’s easy enough to add it back. I’d rather not leave something in which is able to fill /tmp that easily (just deleted ~12,000 files on my laptop, just from a couple of runs of the test suite).


Custom remote devise authentication

I am using Rails 4.2 & Ruby 2.2 app with devise 3.5.2. We don't have any database(external through APIs). Currenty we are using custom devise Remote Authentication.
In devise.rb:
    config.warden do |manager|
       manager.strategies.add(:remote, Devise::Strategies::RemoteAuthenticatable)
       manager.default_strategies(:scope => :user).unshift :remote
    end
    Devise.add_module :remote_authenticatable, :controller => :sessions, :route => { :session => :routes }

But now due to some requirements, we have to use custom session controller & a custom method to sign up & directly login.
How to call the remote authentication from this custom method?
In remote_authentication.rb:
    module Devise
        module Strategies
          class RemoteAuthenticatable < Authenticatable
            def authenticate!
              auth_params = authentication
              if validate(resource){ resource.remote_authentication(auth_params) }
                success!(resource)
              end
            end
          end
        end
    end

Thanks.

Please use the mailing list or StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.

My suggestion would be to move the shared logic to a common place you can invoke both from the controller and the strategy. Better than coupling them.


Bump govuk_content_models

For https://github.com/alphagov/govuk_content_models/pull/348

:ship:

It would be better if the commit said what the intent was, rather than just that you bumped the gem.

Amended commit.


test: images and sheets

The imageStore needs to be in the workbook, not in the worksheet.
The goal is coming into view.
To be continued tomorrow.


Rename steps

Change syntax around transitions and focus steps to simplify markdown source and JS code. Examples of new syntax:
{change-to=“sticky1” transition=“fade-in”} (this could be a default value)
{change-to=“sticky1” transition=“fade-left”}
{focus-on=“sticky1” highlight-spans=“myspan”}
{focus-on=“sticky1” highlight-lines=“1-3”}
{focus-on=“sticky1” zoom-to=“100, 120”  zoom-level=“2”}

Have just tacked a pair of commits on here (2d3a104 and 176dcb4) to:

make the scroll progress OJS variable consistently numeric (starts at 0, although I think there's an argument for null too)
break the scroll direction out into its own OJS variable (starts at null)

@andrewpbray I like the switch to using IDs instead of indices generally! I wasn't sure if this is a draft PR, since it seems like the rescaling functionality is turned off at the moment, with the window resize handler disabled and updateStickies removed from the scroll handlers.

What is a draft PR? I think the answer to that question, though, is yes =).  That's to say, I wanted to try the indexless approach while keeping the other code around for reference. I also wasn't quite sure what to do with the rescaling. That was packaged up with the highlighting, but think it'll work as a separate function?

It also occurred to me that on window resize, the only function we'll need to call is the rescaling stuff, right? (not the transitions/highlights, etc)

I think so, if highlighting doesn't include zooming/translating to focus on a span!

Probably... I think so!

I definitely let scope-creep take over this PR. It was meant to be just swapping out the step names (eliminating the from) and grew to include both removing the indices from the sticky activating system and refactoring the highlighting code. I'd like to tie this guy off before he gets more convoluted.
Right now, there's no zooming functionality on this branch, so merging it will be a regression in the functionality of demo-block-types. What do you think the best way to restore that functionality?

Mmm, sorry, I jumped in on this one a bit too! I think once we're happy with #28 and #36, that might make things easier to align on and merge in.

@jimjam-slam I'd like to put a bow on this so that we can move forward with the simpler step syntax of just using focus-on. The rub is that I foolishly started a quite different project in here: refactoring recalculateActiveSteps to split out zooming, highlighting, and swapping .cr-active in anticipation of expanding each of them for new block types. The main commit is https://github.com/qmd-lab/closeread/pull/33/commits/3c42e63f784002dbc9b8106f1666dab34f8915a3 and it succeeds in breaking the poem.
I'm happy to snip out that commit and reimplement elsewhere but I believe you may be using this version in the feature-ojs branch. What do say: snip out the commit or merge and fix with another PR?

I'm happy for you to keep going on this branch if you want! I can monitor and merge extra commits back over to feature-ojs if needed 😊

@jimjam-slam A few of the threads that we've been working on - grid layouts, text zooming, and highlighting - are mixed together across a few branches. In an effort to isolate them, I propose:

Merging thing branch into main knowing it'll break poem zooming.
Create a zoom-to branch, where we work on restoring that functionality, either through the original transform approach or the adaptive font size approach (which I think is on the branch with the grid layouts?)
Create a highlight branch where we can extend text highlighting to include code highlighting.

What do you think?

This sounds good to me! I should be on a bit later in the day!


Missing the copyright file

This project is useful for me to study the goose protocol.
But I cannot find the copyright file.
Is it released under the BSD license as Zeek?

Yes, our code is released under BSD-3. We have uploaded the LICENSE accordingly


[MINOR][DOCS] Remove extraneous whitespace from SQL tuning page

What changes were proposed in this pull request?

Remove extraneous whitespace.
Change Liquid formatting block into "normal" Markdown code block.

Why are the changes needed?
Minor documentation cleanup.
Does this PR introduce any user-facing change?
No. Here is the rendered output, which is unchanged.

How was this patch tested?
I built and reviewed the docs locally.
Was this patch authored or co-authored using generative AI tooling?
No.

I usually do (e.g. see #44955), but in this case the rendered output is unchanged. Nonetheless, I've added a screenshot.

Merged to master.


Running VELOCIraptor on large RAMSES cosmo hydro snapshots

Compilation ok (but might need to adjust a few options like longint for all the particle types - both stars and DM particles)
Configfile ok (but might need to adjust for a few parameters. Need to adjust properly code units, especially because of RAMSES peculiar  units)
Start running ok
Segmentation fault after Read header - Try several configurations (with up to 660 nodes, different nb of tasks per node and different nb of threads, try also different -s and -Z options in the command line )

Hi @sorcej so I think it is likely the read header isn't reading the header correctly. I will say that I have tried to make it work but I've encountered several different ramses formats with different structure in the header. Since the binary is not self-describing I did not know how to make it work. Could you provide a description of the binary data?
Also, are you trying the development branch? It actually is better than main (I need to change this to be the default).

Thanks! Ok so I switch to the development branch. I have the same issue.
I do not have a reader in C++ unfortunately but common reader in Python or Fortran typically work on the simulation (For instance, https://github.com/florentrenaud/rdramses/blob/master/rdramses.f90 works). So there should not be anything specific but for the fact that I had to use longint for both DM and star particles

Here might be some of the problems in the reader:
// Total number of Stars over all processors
Framses.read((char*)&dummy, sizeof(dummy));
Framses.read((char*)&ramses_header_info.nstarTotal, sizeof(int));
Framses.read((char*)&dummy, sizeof(dummy));
-> since I am using longint for nstarTotal (because more star than 2^31). However it is probably not the only one... not sure whether it is easy to have an option for longint for both nstartot and nparttot.

Ok so I fixed and adapted a few things, now I can read the header. At the end of reading it, I have the proper number of particles. Now it crashes again. Could you please help me with figuring out how many nodes/cpus/tasks I need to use?
The sentence is
There are<PHONE_NUMBER> particles in total that require 768GB of memory, each particle requiring 96 bytes at the minimum
So this is the proper number of DM particles.
I am not sure about the numbers that should follow the -s and -Z options for instance.
Thanks

Hi @sorcej , apologies I'll be a bit slow in replying for the next two days as I have to finishing marking assignments for a high performance computing course. Do you mind creating a draft PR with your proposed changes so I can have a look?

Thanks @pelahi and sorry for the delay. I cannot do any PR from the supercomputer unfortunately. Anyway, I tried to understand where it was further crashing and I finally managed to pinpoint. It is in: MPIInitialDomainDecompositionWithMesh(opt). Not sure if it is because I am using too many or not enough cores perhaps?

After several days, I decided to simply remove the option opt.impiusemesh to try to go further. I will try later to re-add it...

@pelahi ok so now I am again stuck a bit further. I again had to bypass Ok , dmp_mass = 1.0 / (opt.Neffopt.Neffopt.Neff) * (OmegaM - OmegaB) / OmegaM; as it gives something negative so I had to use Particle IDs instead in mpiramsescio.cxx this time.
It starts running and counting properly particles (note that I did not understand why we had to do it twice as it was already done but ok). Now I have a
malloc(): corrupted top size
[mpiexec@i06r03c04s12] control_cb (../../../../../src/pm/i_hydra/mpiexec/mpiexec.c:1324): assert (!closed) failed
[mpiexec@i06r03c04s12] HYDI_dmx_poll_wait_for_event (../../../../../src/pm/i_hydra/libhydra/demux/hydra_demux_poll.c:80): callback returned error status
[mpiexec@i06r03c04s12] main (../../../../../src/pm/i_hydra/mpiexec/mpiexec.c:2045): error waiting for event
...
I am trying to fix that one too.

Hi @sorcej , so the dmp_mass calculation was based on reading some ramses data where it was it was easier to quickly calculate the mass for dark matter particles using the matter density - the baryon density and the effective resolution of the simulation. Regarding your error, it could be something to do with ints being used and where values would exceed 2e9 and then give an unsigned number. Could this be the case?

@pelahi ok thanks, now I get the dmp_mass. I actually had to change manually Neff though. I will try to add it as an option (unless there is an option that I missed). Regarding the malloc. I am not sure yet but I think something went wrong when reading the particle positions thus they cannot be properly assigned to the different tasks. I am trying to fix it.

@pelahi Ok I have fixed the particle positions. I still need to understand something with the IDs but it looks better now. I think I have now yet another problem to solve when writing: [mpiexec@i05r01c01s04] HYD_sock_write (../../../../../src/pm/i_hydra/libhydra/sock/hydra_sock_intel.c:360): write error (Bad file descriptor)
Any idea where precisely there could be something wrong here? I am trying to pinpoint it but it is not obvious.
I still of course have load balancing issues but for now I leave it as is.
I might also have to fix some units but I will see later.
Thanks again :)

Hi @sorcej , not certain about the write error. When does this happen? Could you provide the associated velociraptor output?

@pelahi The code writes the files .configuration, .siminfo and .units.

@pelahi ok finally managed to narrow down where it crashes. It is when trying to build a new tree:
tree = new KDTree(Part.data(),nbodies,opt.Bsize,tree->TPHYS,tree->KEPAN,1000,0,0,0,period);
I have yet to find out why it crashes for some of the tasks. Sometimes it gives me a free() invalid pointer and sometimes a double free or corruption (out) but I do not get why and why only for some of the tasks

Hi @sorcej , would you have a small ramses input example I could try? It would help me debug the issue.

Hi @pelahi, sorry for not getting back earlier. I was attending conferences (actually I will be leaving again on Sunday). Anyway, it works with a small example. That said I managed to get outputs when dealing only with DM particles even with the large simulations now.  I need though to check these outputs and whether they make sense. Then I will try to deal also with the stars, etc. Thanks

Hi @pelahi , any chance that you still have the config file used for that paper https://arxiv.org/pdf/1806.11417 ? I am interested in finding galaxies and their properties. I tried using and adapting the config file sample_galaxycatalog_run.cfg in the example folder but with no success yet. Thanks

Hi @sorcej , apologies for the late reply so the best config is https://github.com/pelahi/VELOCIraptor-STF/blob/development/examples/sample_galaxycatalog_run.cfg as you noted but you might want to try keeping the 3DFOF envelop if looking at replicating Rodrigo's paper. Now can you provide some information as to what issues you are encountering? Is it a processing issue? or is it that you are not getting the intracluster light you were expecting?

Thanks a lot @pelahi for your answer and sorry for the delay in getting back to you. I had to put aside this analysis for a while. By keeping the 3DFOF envelop, you mean Keep_FOF=1 correct? About the issues, I had several. At first, it was not even running but it ended up being apparently a problem with the machine... Now it seems to be fixed. Currently, I am a bit perplex regarding the catalogs I get. If only star particles are used, I do not really understand how to read the output file "properties" that seems to have information similar to what I will get with dark matter particles. But perhaps I did not activate the proper output? Thanks for your help.

I @sorcej , so the properties autocalculated were very focused on typical halo + galaxy properties people often require. If you have an idea of the properties you'd like to calculate I could quickly make a branch that might calculate the output you need. Otherwise, you can you the python tools to load the properties of the particles files in in python (it's a simple hdf5 output) and then read the input data and get the particles per group and calculate the desired properties. There's an example of this in the tools directory.

@pelahi , thanks a lot for your answer. In the outputs though I am missing the galaxy properties you mentioned. I must have done something wrong there. I only have typical halo properties but on star particles... This is why I find this a bit obscure to me. I will have a look in the meantime to the tools you mentioned. Thanks

Hi @pelahi, sorry I am still super confused:
velociraptor_00218.catalog_particles.unbound.0
Total_num_of_particles_in_all_groups: 239447203
velociraptor_00218.catalog_particles.0
Total_num_of_particles_in_all_groups:<PHONE_NUMBER>
Why is the sum of the two not equal to the total number of the DM particles in the simulation?
Another question why is Sum(Group_Size) not equal to the Total_num_of_particles_in_all_groups (i.e.<PHONE_NUMBER>)?
Thanks for your highlights. I am clearly struggling to understand the outputs.

Hi @sorcej  so the *.catalog_particles *.catalog_particles.unbound arose from the format of listing particles used by subfind which listed "bound" and "unbound" particles in separate files. Now this distinction really only makes physical sense when extracting data from the entire simulation (all particle types). In your case with just stars, I would just combine the lists and not treat the distinction as anything real.

Hi @pelahi, sorry, I was unclear. I was first looking at the results using dark matter particles trying to understand the outputs. I understand your point using star particles though. Could you please point me to some explanations on how to understand these outputs so that I can read the particles (either DM or stars) belonging to a given object (either halo or galaxy) to derive properties that are not in the velociraptor_00218.properties.0 files? Thank you so much

Sure @sorcej , it's here https://velociraptor-stf.readthedocs.io/en/latest/output.html. Note to have extra star properties read and then used, you will likely need to adjust the ramses io to ensure that these are stored and also you need to compile the code with -DVR_USE_HYDRO=ON which will then have particles store extra specific set of properties, like metallicity or age and calculate properties based on this.
You can event store extra custom properties with specific names (see for example the config https://github.com/pelahi/VELOCIraptor-STF/blob/development/examples/sample_swifthydro_3dfof_subhalo_extra_properties.cfg) specifically entries like
Star_internal_property_names=BirthDensity,InitialMass,
However, both will likely need an update to the ramses interface and substructure_properties files to calculate what you desire.
Can you be precise on exactly what you want to calculate and then we can see if updates are actually needed?

Thanks @pelahi , weirdly enough I think what I need would be with -DVR_USE_STAR=ON but when I compile with it, STARON is still not defined... a minima what I would need is not only to have the IDs of the particles that belong to the galaxies but also their pos, vel, mass, age and metallicity so that I can derive galaxy properties.


Discrepancies between using new Timber\Post(); Timber::query_post(); and Timber::get_post(); using LearnDash Plugin

Very new to WP here, so please ask if I need to explain more or provide extra information
Expected behavior
After installing Learndash and activating the Timber Starter Theme we created a Course along with a related Lesson and a Quiz. We expected to view the Course along with it's Lesson and Quiz on the Course page. This works in default 2022 WP theme.
Actual behavior
We only see the Course and it's fields, but we don't see the Lesson or Quiz. In single.php we can use
$context         = Timber::context();
$timber_post     = Timber::query_post();
$context['post'] = $timber_post;

and we can see the Course, it's Lesson, and it's Quiz. If we tweak single.php, we realized that if we do $timber_post = new Timber\Post(); or $timber_post     = Timber::get_post(); we can no longer see those items.
Also, when we dump the output of $timber_post in those situations Timber::query_post(); is similar to new Timber\Post(); while Timber::get_post(); gives much more information. You can view the output differences here
I would expect that the output for these methods is the same. If not, could you explain the difference?
Also, I've read that in the future Timber::get_post(); and Timber::query_post(); are going to be deprecated and we should no longer use this. Is there another way to handle this so that the related items are shown?
Thank you in advance!
What version of WordPress, PHP and Timber are you using?

LearnDash 4.0.2
WordPress 5.9.0
PHP 7.4
Timber 1.19.1
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
Installed Timber via WP.org

Sorry, closed by accident!

Hi @MarcAtNuRelm
Without having a copy of learndash on hand, it's a bit hard to debug this exact issue, but from my experience we have seen similar inconsistencies in the past - I believe a lot of this was cleaned up, but it's possible some still exist - I believe this is the reason behind some of the upcoming changes in 2.0, and why the interface / entry points to Posts are shrinking to just Timber::get_post() or Timber::get_posts().
To clarify a couple things.
Previous communication around Timber::get_post() disappearing in v 2.0 appears to have been walked back, and this is now the only way to get Post and child classes, via it's Factory and use of the class map. Use of direct instantiation such as $post = new Post();  will straight up not work in v2.0!
Finally, I appreciate that the dump output is different, but if you try to access these properties, they should still resolve via the magic getters / setters, is this not the case?
If you have something like {% for step in post.ld_course_steps %} in your twig template and ld_course_steps isn't populated on the Postat that point in time, then it would call the magic  method__get()eventually resolving to a call toPost->meta('ld_course_steps');which would return data via WordPress'get_post_meta()` core function.
If you'd like to future-proof your code for version 2.0 and beyond then calls to meta in your template should be done using the meta function, so the above example would become {% for step in post.meta('ld_course_steps') %}.
My personal preference / favourite, is to heavily utilise the class maps and extend Timber\Post for each CPT on my site, and to create methods to access the properties I need, allowing for extra processing etc - so I'd have something like....
<?php

class Course extends Timber\Post {

    public function steps(){

        $steps = $this->meta('ld_course_steps');

        //maybe do some additional processing, extra data etc

        return $steps;
    }

}

And then in my template I can just go {{ for step in post.steps }}.
Hop this helps, sorry I don't have a copy of LearnDash to debug, my guess is that it's something to do with the global Query and one of these being seen as "the one" and getting meta front-loaded (by timber, or LD, not sure).

Hi @MarcAtNuRelm I hope the excellent exploitation of @aj-adl cleared things up. I am going to close this as stale for now.


Create a List of Contributors

Create a list of Contributors for this project.
The list should be displayed in a tabular format.
Info to be included - Name ,  their Github profile link.

Can I work on this issue??

Yup, you can work on this.

@atharva0300 We can close this issue now I believe

Should I work on this issue using json file and fetching the data ??

@james3gh
Sure, you can.


client: optionally wait for lnd to be unlocked

This PR adds an option to wait for lnd's wallet to be unlocked before lndclient returns, similar to what we have done waiting for the chain to sync.
Pull Request Checklist

[x] PR is opened against correct version branch.
[x] Version compatibility matrix in the README and minimal required version
in lnd_services.go are updated.

Updated this to use the wait for ready grpc call option to deal with the case where lnd is unlocking. Behavior is now as follows:

lnd down: will wait for waitInterval then fail
lnd locked: will query lnd periodically, backing off waitInterval between queries, until lnd is unlocked or ctx cancelled
lnd unlocked, race: will wait waitInterval for the main server to come up after unlock

Updated this to use the wait for ready grpc call option to deal with the case where lnd is unlocking. Behavior is now as follows:

lnd down: will wait for waitInterval then fail
lnd locked: will query lnd periodically, backing off waitInterval between queries, until lnd is unlocked or ctx cancelled
lnd unlocked, race: will wait waitInterval for the main server to come up after unlock


CI: Use ephemeral self-hosted runners (again)

Return to using ephemeral self-hosted runners rather than leaving two runners on 24/7.
As these runners are only on while a build is running, we can make them larger. My account currently has a quota for CPU cores so I've had to allocate 4 cores rather than the desired 8, but I'm hoping to get this resolved.
The x86 VM is an Intel Sapphire Rapids, and the AArch64 VM is a Google Axion. Both have 4 cores and 32 GiB memory.

When this is merged, I'll have to cancel the build this starts, terminate the temporary CI instances, then restart the cancelled build (otherwise we hit quota limits)

Something's going wrong with GitHub Actions, the builds are running on all four VM's but only one of them is posting its status back to GitHub

I'll retry the build when https://github.com/lilyinstarlight/nixos-cosmic/actions/runs/12191241539 completes (3/4 of those jobs are accidentally running on ephemeral instances)


Switch to using govuk_admin_template and Bootstrap 3

Review using ?w=1 as many changes involved adding or removing HTML wrappers and altering indentation.
This is a large but also minimal pull request. I have aimed to keep it as small as possible – so that the majority looks and works exactly as it did before. The commits are predominantly class changes and the large bulk of it is:

switching from Bootstrap 2 to Bootstrap 3 grids
switching to Bootstrap 3 button classes
updating modals to new syntax
updating icon classes to use new glyphicons
switching to Bootstrap 3 form classes, styles are no longer applied automatically (mostly form-control and form-group)
updating alert- classes, mostly alert-error to alert-danger
minor typography improvements along the way
adding list-unstyled to remove Bootstrap 3’s default list styles where they aren't wanted

Some parts did require more attention and it's worth pointing those out:

The force publish modal on a link needed a shim to work with Bootstrap 3
The admin gem was updated so the Whitehall header could be used without significant changes
The words to avoid highlighter picked up some styles from the admin gem because of a class clash
Bootstrap 3 is more flexible in its use of horizontal form fields, but they now depend on grid classes, making a generic helper more complex. Horizontal form fields have been removed.
Bootstrap 3 doesn't support IE6, IE6 specific styles have been removed (4 sessions  since January used IE6)
Use a Chosen Bootstrap 3 theme

All changes should only affect the admin parts of Whitehall. It's probably worth leaving this on Preview for a short while once we're happy with it, to try and spot anything that's been missed.
https://www.pivotaltracker.com/story/show/96072212
Before and after screenshots
New document
Before

After

Document list
Before

After

Dashboard
Before

After

Featured documents
Before

After

Collections
Before

After

Edit document
Before

After

Document summary
Before

After

All the Ruby/Railsish stuff looks good to me :+1:

On the new document page, the govspeak helper box code examples no longer wrap. This page has a bunch of changes that I think are fine, this is the only one that we should probably fix as the helper is now quite difficult to use:
(before)

(after)

@alicebartlett I put back the missing pre styles (white-space: pre-wrap) so that lines would wrap correctly and be readable. This has introduced a new problem, where wrapping happens at a point that can cause confusion. In the example below it suggests that a line break is needed in link markup (I've seen this sort of thing cause problems in Whitehall training).
What are your thoughts?
Before

After

@fofr I would defer to what the whitehall behaviour was before. In this case, the behaviour was to have the wrap, so the markdown looks like:
[a url]
(https://your-link-here.com)
or maybe even
[a url]
(https://your-link-
here.com)
which is what you have.

@alicebartlett Preview looks like:

@fofr (depending on your viewport width:

OK, I'm done reviewing now. Happy for this to be merged.

Thanks @alicebartlett! :smile:
We're going to deploy this branch to preview for a few days to see if anything else surfaces.

(p.s) Well done for getting this done, it was epic!

:heart_eyes_cat:


How to convert the weights to detectron2 format for mask-rcnn pretrained on mmdetection?

There is a demo that can convert d2 faster rcnn to mmdet format.
https://github.com/BIGWangYuDong/mmdetection/blob/d2_tools/tools/misc/faster_rcnn_d2_to_mmdet.py

Also, we will support training and testing detectron2 models in MMDetection soon. You can refer to this PR https://github.com/open-mmlab/mmdetection/pull/8672

Thanks a lot for all this.

We have supported training Detectron2 models in dev-3.x branch. Welcome to try and feel free to open an issue if you met problems.


Gas Estimation Not Implemented in Contract Call Builder

Please implement the estimation function in the contract call builder.
Can't use this object in a relayed transaction v2 builder because it tries to build the Transaction object and can't handle gas limit being 0...

The matter should now be resolved in the new transaction factory for relayed transactions:

https://github.com/multiversx/mx-sdk-py/blob/main/multiversx_sdk/core/transactions_factories/relayed_transactions_factory.py
https://docs.multiversx.com/sdk-and-tools/sdk-py/sdk-py-cookbook/


Where should the .vsz file be placed?

This documentation always feels like it isn't written from the perspective of a developer actually trying to do things. So hard to find the answers to the basic questions.

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

ID: 16d15a44-760b-bed4-5dc3-415f183c19e8
Version Independent ID: 1e257d1d-026d-e6d9-5500-5a49dffb3e7c
Content: Wizards - Visual Studio
Content Source: docs/extensibility/internals/wizards.md
Product: visual-studio-dev15
GitHub Login: @gregvanl
Microsoft Alias: gregvanl

Thanks for submitting your feedback and we apologize for the delayed response.  Due to current resourcing and substantial extensibility changes we are currently working on that will result in significant, structural doc changes, we are closing this lower priority issue.  We will return to this issue on a later date and address it based on it's relevance after those extensive extensibility updates are made. Feel free to comment here as necessary.


Issue with installing

Describe the bug
When installing upchatpy using a requirement file, the installation fails due and thrwos an error.
To Reproduce
Steps to reproduce the behavior:

Add upchatpy to requirement.
Install requirement file
Try to use library

Expected behavior
No error
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
ubuntu 20.04.6 lts
using python 3.11

@Vondyy have you tried running pip install aiohttp before trying to install the library? It is a requirement in the setup.py file but seems like its not being installed properly. Based on the screenshots I can only make assumptions that you're trying to install the library outside of a virtual environment or that the aiohttp install is broken somehow.

So, I use aiohttp on my own for my personal project so I do already have it previously installed before trying to install this. I use pterodactyl(isolated Docker containers) to host my project and I use requirement files as you can't directly use the console for discord.

Try adding a line in your dockerfile or whatever entry point you use to manually install aiohttp, I don't use pterodactyl and thus can't directly support it so you may be on your own a bit figuring that one out. Sorry I cant help more than that.

@Vondyy any luck?

I tried your solution but it didn't work. I ended up using your repo and copying the things I needed and it works fine now. I'm just unable to install it directly.

interesting, its always something weird with Pterodactyl, glad you got it working using the code directly though! Guess we can close this now, best of luck!


Updates repos.md with new descriptions and missing submodules

https://github.com/eliotsykes/real-world-rails/pull/7916
Fixes https://github.com/eliotsykes/real-world-rails/issues/7924
Fixes https://github.com/eliotsykes/real-world-rails/issues/7926

repos.md generation is moving to a github action.


OAuth parameters lost after login using OAuth Support

Hi.
My name is Hugo, and I am trying to configure OAuth support on a custom CAS Server.
However, I'm having issues with some information saved on http session being lost after login.
I managed to enable the OAuth endpoints /oauth2.0/*, to register the OAuthCallbackAuthorizeService and a simple OAuthRegisteredService for testing.
I was able to access:

https://my-host.com/cas/oauth2.0/authorize?client_id=my-client-id&redirect_uri=https://my-host.com/app/test

It redirects to CAS Login Form with the service parameter pointing to the Callback Service.
However, after login in, the session is renewed (session fixation protection), and when the callback controller tries to restore the redirect_uri information from session, it's lost.
I tested with a fresh CAS server installation (versions 4.0.3 and latest 4.1.1), and happens the same.
Is there any other configuration that I am missing?
Thanks in advance.

I think I figure out the problem.
I was connecting to CAS Server through HTTPS (8443) but the Callback Service was registered on HTTP (8080), using ${server.name} which defaults to cas.properties; server.name=http://localhost:8080.
I further tested accessing only with HTTP, and worked fine.


Using canonicalized path to launch cmd from hook results in incorrect working directory

On Windows, if a hook's command is cmd it results in the error
'\\?\C:\Users\me\Documents\my_app'
CMD.EXE was started with the above path as the current directory.
UNC paths are not supported. Defaulting to Windows directory.

This is being caused by hooks setting the current directory which are canonicalized.

Thanks for reporting this. Could you come up with a PR for this?

Closed by  #891


Stack programs by name

Enhancement on PR #90
Present behaviour:

new program parameter set adds a new program to the program list.

Stacking behaviour:

new program parameter set can update and stack parameters for a program by name.

This is useful for building programs from other programs or subtly modifying programs in A/B testing, etc.
Next steps:

decide if this is actually useful and people would make use of this functionality.

I think I understand this and support this.  Lets discuss in the next meeting

Not sure I understand fully. Let's discuss. @KeeganShaw-GIS  can you please add to next week's agenda?


Profiler does not record network requests even if told to (by code from Detox.framework)

Description
The recent pull request to Detox ("Record network and events in Detox Instruments artifacts")
should have enabled recording network requests in the integration of Detox and Detox Instruments
(introduced originally in wix/Detox#1165).  However, it does not seem to work as intended.
This demo project demonstrates the bug.
Environment

MacOS Mojave (10.14.4)
Xcode 10.2.1 (10E1001)
Detox Instruments 1.10 (10233)
Node.js 10.15.0 (installed via nvm)
Detox 12.6.3
iPhone X (in Simulator)

Steps to reproduce

Clone git clone git@github.com:noomorph/bug-reports.git
Checkout to branch detox/no-network-recording.
Run npm install.
Run npm start.
In another terminal tab, run npm test.
Wait till the test completes.
Open with Detox Instruments artifacts/ios.<YYYY-MM-DD HH-mm-ssZ>/✓ Example should have welcome screen/test.dtxrec.

Expected result
Network Activity timeline should have at least 1 request:
GET 200 http://worldclockapi.com/api/json/utc/now

Actual result
Network Activity timeline is empty.

Moved the issue to Detox, because this is where it will be fixed.

I found the issue. It wasn't related to late framework load. It's related to swizzle order between Profiler and Earl Grey. Investigating.

Fixed


setValue without triggering change events

We only want to listen to user-triggered events. It looks like _resetValue would allow for this (I get it, it's supposed to be private). However, we can't seem to use it because the necessary function toRawText defined on TextModel seems to be unaccessible.
How can we change the editor text value without triggering a change event?

I'm not sure I understand the intent here. Any changes done to the model will result in change events being emitted (e.g. to update the view or the language services that provide rich language features). What would be the use case for changing a model and inhibiting events?

The distinction lies in the event source. Are events triggered by the user or code? In my case I want to listen to user triggered events, but also set the editor content and cursor position using code. The current solution is to unbind and rebind listeners each time.

This is a common problem I also hit often. Usually I use a code pattern like the following:
var ignoreEventBecauseIAmTriggeringIt = false;

emitter.onBla(function(e) {
  if (ignoreEventBecauseIAmTriggeringIt) {
    return;
  }
  console.log('Now I should react to ', e, ' because someone else caused it');
});

...

// Do something that makes `onBla` fire:
try {
  ignoreEventBecauseIAmTriggeringIt = true;
  emitter.changeBla(..);
} finally {
  ignoreEventBecauseIAmTriggeringIt = false;
}

Thats essentially what I'm doing @alexandrudima. To avoid race conditions I think it would be great to have a way to know what caused the event, e.g. the user or the code. This would also be valuable for widgets and other feedback.

+1. For eg. In Codemirror for every change you also get it's origin - if it was a user input or setValue call.

agreed it would be useful to have this distinction - would make it play with angular's ControlValueAccessor API much better: https://github.com/angular/angular/issues/20230#issuecomment-361799387

It would be great if the 'source' parameter of the 'executeEdits' function could be obtained in the 'onDidChangeModelContent' function.

after toggling to a different file , im unable to get previous changes using ctrl + z for previous file..
any help would be appreciable

This is still an issue and a fairly important one. Any update on this?

This has cropped up for me as well. Trying to build an instance so an instance of the editor and model can be edited from a web socket event, but I don't want it to trigger the event

I'll just add here that it's fairly standard practice to be have this functionality.... Ace editor, Code mirror, even back as far as Jquery widgets have this.

I had the same problem and found a solution/workaround : isFlush property from onDidChangeModelContent event. Try this :
editor.onDidChangeModelContent(e => {
  if (e.isFlush) {
    // true if setValue call
  } else {
    // false if user input
  }
});

Any update on this issue?

Hi, I have same problem.
I want to use websocket to sync every code change, but is will trigger a dead loop

I'm not sure I understand the intent here. Any changes done to the model will result in change events being emitted (e.g. to update the view or the language services that provide rich language features). What would be the use case for changing a model and inhibiting events?

Hi, I have same problem.
I want to use websocket to sync every code change, but is will trigger a dead loop.
Can I get which source did it come from like onDidChangeCursorSelection?(ICursorSelectionChangedEvent)

I need this to prevent event if the change is made by formatter.
e.isFlush seems not the solution to me. :(

@DanielJefferson1990
if (ev.reason === CursorChangeReason.ContentFlush) {
    // true if setValue call
  } else {
    // false if user input
  }


Replace CUtility::toString with C++11's std::to_string

This pull request is now managed with CodeReviewHub. Refresh.

:+1:

:+1:

@dawagner please rebase

:+1:


helm history grpc trying to send message larger than max

$ helm version
Client: &version.Version{SemVer:"v2.14.2", GitCommit:"a8b13cc5ab6a7dbef0a58f5061bcc7c0c61598e7", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.2", GitCommit:"a8b13cc5ab6a7dbef0a58f5061bcc7c0c61598e7", GitTreeState:"clean"}

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.7-eks-c57ff8", GitCommit:"c57ff8e35590932c652433fab07988da79265d5b", GitTreeState:"clean", BuildDate:"2019-06-07T20:43:03Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

I need to roll back to a previous deployment, but can no longer find what a working deployment was from the history list due to the grpc error.
$ helm history prometheus-operator --max=10
Error: grpc: trying to send message larger than max (22327930 vs. 20971520)

$ helm history prometheus-operator --max=9
REVISION	UPDATED                 	STATUS  	CHART                     	DESCRIPTION
74      	Fri Jul 19 15:35:45 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: no PodSecurityPolic...
75      	Fri Jul 19 15:36:28 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: no PodSecurityPolic...
76      	Fri Jul 19 15:36:50 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: no ConfigMap with t...
77      	Fri Jul 19 15:39:14 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: no ConfigMap with t...
78      	Fri Jul 19 15:39:41 2019	DEPLOYED	prometheus-operator-5.16.0	Upgrade complete
79      	Fri Jul 19 15:39:59 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: no PodSecurityPolic...
80      	Fri Jul 19 15:48:35 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: kind PodSecurityPol...
81      	Fri Jul 19 15:49:49 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: kind ConfigMap with...
82      	Fri Jul 19 15:50:01 2019	FAILED  	prometheus-operator-5.16.0	Upgrade "prometheus-operator" failed: kind PodSecurityPol...

https://github.com/helm/helm/issues/4865

Ah, thanks!


Switch from GH doc issue feedback to new feedback control

Per e-mail communication, switching to the new feedback control for Docs.

@JonDouglas Hey Jon, could we get this PR merged?

Thanks, Jon!


[Issue]: Ubuntu 24.04.1 LTS incompatibility with rocm 6.2

Problem Description
rocminfo show
ROCk module is NOT loaded, possibly no GPU devices
I previously used rocm 6.1 on Ubuntu 24.04.4 (previous version but the number is somehow larger .4 vs .1). There was no problem, but I format my PC. Then, I install 24.04.1 and rocm 6.2, and rocminfo cannot load the GPU device.
I am not sure if this is ROCM's fault or Ubuntu's fault.
Operating System
Ubuntu 24.04.1 LTS (Noble Numbat)
CPU
13th Gen Intel(R) Core(TM) i5-13600K
GPU
AMD Radeon RX 7900 XTX
ROCm Version
ROCm 6.2.0
ROCm Component
rocminfo
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Hi @Wintoplay, I wasn't able to reproduce the ROCk module is NOT loaded, possibly no GPU devices error when installing ROCm 6.2 on Ubuntu 24.04.1 with the quick install instructions. Could you please specify the steps you followed to install ROCm 6.2 and also give the quick install instructions a try?
Currently, only ROCm 6.2 is supported on Ubuntu 24.04 while ROCm 6.1 is supported on Ubuntu 22.04 and other OSes as specified in the compatibility matrix.

I previously used rocm 6.1 on Ubuntu 24.04.4 (previous version but the number is somehow larger .4 vs .1). There was no problem, but I format my PC. Then, I install 24.04.1 and rocm 6.2, and rocminfo cannot load the GPU device.

At time of writing, there's no Ubuntu 24.04.4 - you sure you didn't mean 22.04.4?

Closing out this issue. @Wintoplay, if you are still experiencing the ROCk module is NOT loaded, possibly no GPU devices with ROCm 6.2 on Ubuntu 24.04, please provide your installation steps. Thanks!


Reset input field after user send a message

Return an additional ChatInput() in send_message to make sure the user input field is reset after the Send button is clicked.

Added show:bottom to hx_swap to enable scrolling of assistant messages.

The show:bottom makes it impossible to scroll up and read the start of a long message while it's still generating, since it keeps jumping to the bottom of the message. So let's remove that for now and then I'll merge :)

I see. Thanks @johnowhitaker I removed the show:bottom commit, please check if it is ok now :)

@inweb3 thank you :)


Merge pull request #1 from madcowswe/master

Update 17/08/2020

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


Travis CI should be used to run tests

We can copy the config from fsharp/FsAutoComplete

I see that we have a config already, so I've just turned on the Travis support.


Find a way to create several pipelines from shader files

Currently there is no nice way in Vulkano to create a pipeline from a shader file. That's why I am currently creating only one "master" shader which is used by every material. However, I want this to be changeable.

This might be changed to static shader types per lighting model and blending types like

pbr opaque
pbr masked
pbr blend
phong opaque
...

To implement this I need to move the engine modules back to a shader agnostic storing system.
The uniform manager should not store an pbr_fragment::ty::data struct but an custom data which can be transferred to an pbr_fragment::ty::data when creating the descriptor sets.
When this is done:
Don't reference the pipeline by name, but by Arc<pipeline> in the materials.
When creating the descriptor set, read the needed inputs from an custum PipelineInputs struct and create the descriptor set based on it.
also: Don't read the the descriptor sets one by one in the renderer, but at once from the material and return an tubel filled with the right amount. (I need to test how to do this though).

[ ] Move to independent data sets
[ ] Store pipeline reference
[ ] Create descriptor sets based on PipelineInputs
[ ] Create high level get_descriptorsets() function

I just decided that the UniformBufferPools should also be moved to the pipeline because they are pipeline dependend.

currently adding new shader files to define some struct globally (to be copied by the shader writer).
In the future I might write a small string analyser who searches for a key word in a shader string and replaces it with a module. But I have to find out how to do this at compile time. Ref: https://github.com/SiebenCorgie/jakar-engine/issues/15

I decided that it is best to leave the descriptor set generation to the materials.
However, it would still be nice to decide the amount of generated descriptor sets based on the currently used pipeline inputs.
I'll add a small check function to each of the getter function to check if we really need the descriptor set. If not, they won't do anything and just return a ().
The get_descriptor_sets() function then can return some kind of tubel storing a descriptor set or a ().

The get descriptorsets function won't make it because I cant find a way to return variable tubel length without some Option nightmares.


Update compose to v1.6.8

This PR contains the following updates:

Package
Type
Update
Change

androidx.compose.ui:ui-tooling-preview (source)
dependencies
patch
1.6.7 -> 1.6.8

androidx.compose.ui:ui-tooling (source)
dependencies
patch
1.6.7 -> 1.6.8

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Superseded by #112.


Supports resource attribute for MetaData

I have to make this patch for configuring USB permission: https://developer.android.com/develop/connectivity/usb/host.
PS: I don't know how to request for permission at runtime without having Java and Gradle: https://github.com/rust-mobile/cargo-apk/issues/19, because UsbManager.requestPermission() is not a blocking function.

Currently this crate requires the patch: https://crates.io/crates/android-usbser.


refactor(main-doc): improve bff docs

Summary

remove the concepts of function mode and framework mode.
use api/lambda/xxx instead api/xxx in BFF docs.
remove api/*.tsx docs, add api/lambda/*.tsx docs
improve sdk.mdx and function.mdx

Related Links

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

LGTM!


Option to delete models/loras/embeds etc from the UI

Feature Idea
I have a lot of models that i either downloaded to try and they sucked or ones that downloaded to the wrong folder or with the wrong name.
Being able to easily delete those from the UI would be very convenient instead of having to do so through file explorer.
Just a button in the model options menu would be enough.
Other
No response

duplicate of https://github.com/mcmonkeyprojects/SwarmUI/issues/48


Fix bug where long data pill labels would break over multiple lines when in a group

Fixes https://github.com/chec/dashboard/issues/722

Do you want them to break or not break? The PR makes it break anywhere, even in the middle of words.

Yeah I did this wrong. I think we need inline-block on the pills when in the group, will update.


[Minor] Fix dockerized GPU builds in dev_menu

Description
see tittle, nvidia docker is needed to build (genop)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@mxnet-label-bot add [Build]

@mxnet-label-bot add [pr-awaiting-review]


Add sysfsutils to dep packages

sysfsutils is used for THP (transparent huge pages) settings.

/lgtm.


provided distributed team friendly language

I added a ton of new copy, suggesting tools such as video conferencing and chat channels to create this document.
This relates to issue #44
cc/ @deathbearbrown

can you add that personas are usually based on data, not hopes? I think a lot of people when we did the workshops thought of personas in the marketting way?  Like Target saying something stupid like "our client is well to do and likes the finer things in life"

Also do you think it would be good to have like a "refine and focus" part where you pick what personas you want to work with?

Good ideas @deathbearbrown - I'll incorporate now.

I'm going to merge this because I get confused with lots of changes.


Add Auto-layout functionality

Node layout helpers that come in two flavors:

Placement aids, e.g, snap to grid functionality, place between pins, etc

The behavior class determines which placement aid is used.

An autoformatter, e.g, StraightenConnections, AlignNodes.

In the editor, these should be implemented the same way as normal, using UI_COMMAND
For runtime, everything will likely need to be rebuilt. Write these as a command function, taking in any graph/group of nodes, and dynamically re-layout'ing it/them.

In addition to straighten connection horizontal and vertical, make a radial layout for perk trees.

This system is now in place in a solid enough state for now


JWT Scopes

Hi,
Why is it that fastendpoints only works with claims and permissions on a token?
It doesn't seem to work with scopes. Any idea how I can it working with scopes?
PermissionsAll(SCOPE_NAME);

Create a policy.

you can change the claim type by setting this property via startup config: https://github.com/FastEndpoints/FastEndpoints/blob/78ba7dd3685ce904dba0113d6a912eab2f7037b1/Src/Library/Config/SecurityOptions.cs#L23

can you show how to set this option?

@ahmadlashgari
app.UseFastEndpoints(c => c.Security.PermissionsClaimType = "scopes")


Dbz 5504 remove invalid datatypes from db2 connector doc

DBZ-5504
Replaces #3795, because the original branch had grown too stale.
Removes invalid MONEY, SMALLMONEY, DATETIME2, and SMALLDATETIME entries from the table in Data type mappings section, including the following  DATETIME2 entries:

|DATETIME2(0), DATETIME2(1), DATETIME2(2), DATETIME2(3)
|INT64
|io.debezium.time.Timestamp +
+
Represents the number of milliseconds since the epoch, and does not include timezone information.
|DATETIME2(4), DATETIME2(5), DATETIME2(6)
|INT64
|io.debezium.time.MicroTimestamp +
+
Represents the number of microseconds since the epoch, and does not include timezone information.
|DATETIME2(7)
|INT64
|io.debezium.time.NanoTimestamp +
In the paragraph in the Timestamp types section, I removed references to the invalid SMALLDATETIMEandDATETIME2 types so that the sentence now references only the DATETIME type and the semantic type io.debezium.time.Timestamp.  Per @Naros' comment in the original PR, the value<PHONE_NUMBER>945104 that was used with the io.debezium.time.MicroTimestamp semantic type in the original version probably does not make sense with the io.debezium.time.Timestamp semantic type. Need to verify this and get input on a valid replacement value.

Hi @roldanbob this is marked "waits for another PR", is there a specific PR this is waiting on other than the reference to the question from the other PR you had?
In terms of the value, I believe it should read<PHONE_NUMBER>000 since it's in milliseconds rather than micro or nano -seconds.

this is marked "waits for another PR", is there a specific PR this is waiting on

Hey, @Naros. Actually, when I closed the other PR because that branch had grown too stale to salvage, I just copied that from the original PR to preserve the metadata. I think Jiri had originally applied that label? Anyway, I don't have anything that I was waiting on. I know that in #3795 you had mentioned that you'd hit a snag in the Db2 testing.

@Naros @roldanbob Yes, that's true. I added the label as that was the way how I interpreted the last @Naros comment on the obsoleted PR.

Hi @Naros

In terms of the value, I believe it should read<PHONE_NUMBER>000 since it's in milliseconds rather than micro or nano -seconds.

I updated this to revise the value in the Timestamps example per your comment. Is there anything further needed here?


Clear storage on disconnect

well, I connected Nami, I want to disconnect and connect etrnl, revoked the site from nami, but I cant get etrnl to connect, it always asks to connect nami

Worked into #100


Add fetching of username

Pull Request Test Coverage Report for Build 241

2 of 6 (33.33%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 31.562%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sagas/backend.ts
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/sagas/backend.ts
1
38.1%

Totals

Change from base Build 234:
0.03%

Covered Lines:
1237

Relevant Lines:
3443

💛  - Coveralls


mesh: moved defined number of segment retransmissions to syscfg

Current SEG_RETRANSMIT_ATTEMPTS value is defined as 4. And hence, while sending segmented messages, each segment is being re-transmitted 4 times. When it comes, to the timing I observed that a message with three segments takes at least 8-15 seconds between successive re-transmissions. For reliability, if the whole message is being transmitted multiple times, I believe segment re-transmissions becomes redundant (please correct me if I am wrong) and thus SEG_RETRANSMIT_ATTEMPTS can be set to lower value to reduce the time between successive re-transmissions.

@hardik-proxy also please add new MYNEWT_VAL to
porting/examples/linux_blemesh/include/syscfg/syscfg.h
and possible to others ports (check Travis for this)


⚠ Bofrost.de has degraded performance

In 4563595, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 5174 ms

Resolved: Bofrost.de performance has improved in 21d3ccf.


Firebase Auth with Serverpod serverResponse.userInfo! Null check operator used on a null value

I am Doing All Necessary Step From this documentation (https://docs.serverpod.dev/concepts/authentication#setting-up-sign-in-with-firebase)
All Checks

Firebase connected Successfully
Serverpod Server Running
Database connected
Necessary Packages and CLI tool installed
both firebase_service_account_key.json and google-services.json added thier right place

but at the end authentication successfully Done its throw error Null check operator used on a null value at  this line=>   serverResponse.userInfo!,
// Store the user info in the session manager.
var sessionManager = await SessionManager.instance;
await sessionManager.registerSignedInUser(
serverResponse.userInfo!,
serverResponse.keyId!,
serverResponse.key!,
);
And also User is successfully Created on firebase i can seen from firebase console

Trying to Create Account

@tusharbhambere How are you signing in as the user? and how are you collecting serverResponse ? Could you please share the code block for it?

for connecting firebase i added serverpod_auth_firebase_flutter which provider me this SignInWithFirebaseButton and in caller i simply pass caller
SignInWithFirebaseButton(
caller: ref.watch(sessionManegerProvider).caller,
onFailure: () {
log('fail');
},
onSignedIn: () {
log('success');
},
providerConfigs: const [
EmailProviderConfiguration(),
// PhoneProviderConfiguration()
// GoogleProviderConfiguration(
//     clientId:
//         '')
]),

Have you gone through session manager setup? Just asking...

Not Setup above email and google part
just follow Firebase part (https://docs.serverpod.dev/concepts/authentication#setting-up-sign-in-with-firebase)
Yes read many time but for this part
Server-side, you need to create a set of authentication keys in the Firebase console, download the JSON file, rename it to firebase_service_account_key.json and place it in the config folder.
i was go to firebase console>project> service account > generate new key and simply change that file name and paste in server>config folder as firebase_service_account_key.json

Okie, I'm trying to reproduce it.

I could reproduce it on v0.9.19

I will go through the complete authentication process before the 1.0 release to ensure everything is working!

Hey all,
anyone already have an idea what could caused this issue?
For me I am able to login with Firebase and get FirebaseUser but the serverResponse.success is false and userInfo is always null.
AuthStateChangeAction<SignedIn>((context, state) async {
...

  final user = state.user!;

  final idToken = await user.getIdToken();

  final serverResponse = await caller.firebase.authenticate(idToken);

})

@avatarnguyen Have you added the database tables for the auth module?

@vlidholt I think I did. but not sure whether I miss something. here is all the table I have:

Adding the real message from firebase admin sdk in the server error response would be the best option imho. I had the same problem and only found out that somehow my firebase service account json was corrupt by manually modifying the serverpod code to print the error from firebase admin sdk.

I have taken a look at this, so I think most likely the issue is that the json key is corrupt, I would try to create a new one and delete the old as the first step.
Second, we had an issue where we would only allow providers that provided an email with the user, I.E. phone authentication didn't work. I fixed this in the PR above.
I have also added better logging so in case the json key is the issue you will now get an error logged on the server.


[Feature][Seatunnel-translation] Support Spark3.3.x

Search before asking

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

Description
Support spark3.3.x
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

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

Code of Conduct

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

Hello, is this going on? I am interested in it. @CalvinKirs

we can change the project structure in seatunnel-translation-spark, because there are some common class using by different spark version. So, firstly, I want to add a seatunnel-translation-spark/seatunnel-translation-spark-common module. what do you think？ @CalvinKirs

I will raise a umbrella issue about support spark-3.x.

#3984


Popups may appear with huge offset when right-clicking unfocused editor

Godot version:

011d201
OS/device including version:

Windows 10
Issue description:

This happens when right-clicking while editor is unfocused. Right-clicks on focused editor makes the popup appear at cursor correctly.

I have found the same issue, to add to that the popup may appear partially off screen.

Was fixed (likely) along with #45740


Documentation of extension not handled out of the box

Fusebox 4@next has been working great for me.  Added a file @ src/assets/shaders/msdf.pixi.vert and getting default/src/assets/shaders/msdf.pixi.vert has not been processed by any plugins  error.  The documentation for the equivalent of file-loader from webpack doesn't seem to exist.  Any guidance here?

Hi, it's just called differently :) v4 takes a lot of extensions by default, I assume this is what you are looking for
https://github.com/fuse-box/fuse-box/blob/master/docs/resource_links.md#via-plugin

Sorry I'm probably missing something.  I wanted to load the contents of the file as a string/buffer instead of the url of the file.

then pluginRaw is the one you need.
https://github.com/fuse-box/fuse-box/blob/master/src/plugins/core/plugin_raw.ts#L17
pluginRaw("*.ver")


JDK 11 support for the product is

Currently, the Identity Server will not start or run on java 11.

This has all last fixes(kernel update) needed to run the product is in jdk 11 https://github.com/wso2/product-is/pull/5955