Should twrConsoleCanvas.twrConDrawSeq use noBlock?

The noBlock  twrLibrary option causes a the "RPC" message (sent from the Worker proxy to the JS Main Thread) to not wait for a return.  It can be used on functions that return a void.  For many functions this works fine and increases performance (the C code does not block, and two cores are potentially used -- the Worker and the Main thread).  For twrConDrawSeq, i think this would work, with the exception that certain calls (like measure text) need to fill in the results before the C code unblocks.  Think about if there is a way to solve for this.  For example, we could add a "WaitForDrawSeqToComplete" type call that is optional.  Or We could move functions like "measure text" out of twrConDrawSeq.  You might have other ideas.

I'll think about it, but from the options you suggested I think something like WaitForDrawSeqToRecomplete would be the easiest for the user. It ensures that twrConDrawSeq is called for all of the currently queued commands so that calls like measuretext have up to date info without having to flush everything. In addition, if flushing (because it uses twrConDrawSeq) is non-blocking, it could cause issues with already external functions (like load image) that might require flush() to be fully called before running.

WaitForDrawSeqToComplete seems like a simple solution.  Another, more backwards compatible options would be to add twrConDrawSeqNoWait.  (or twrConDrawSeqAsync at the risk of overusing the term Async)

That sounds good to me. Though, does backwards compatibility matter in this case? Shouldn't the user be using d2d_start_draw_sequence, d2d_end_draw_sequence, and d2d_flush rather than directly calling twrConDrawSeq directly? In that case, either works and all that would need changing is deciding which internal API is called depending on the function.
I did think of two things though:

What happens if two non-blocking calls or a non-blocking and blocking call are called back to back before the first one is actually processed. Does javascript ensure that the events are parsed in the proper order without anything being dropped?
What happens if non-blocking calls are made faster than typescript can parse them? I'm not sure if this will really be an issue with the canvas, but I'm wondering how much trouble code in C is processing and queuing functions faster than typescript can handle them could cause.

good point.   But the same issue exists with the C APIs (how to get the benefits of noBlock without breaking existing apps).
Regarding your 2nd questions -- here is some text from my latest doc update:

Note that the messages sent from the proxy thread to the JS main thread (for function execution) will cause execution of function calls to serialize, and so if a function that blocks (waits for results from JS main thread) is called after a call with noBlock, everything should work as expected.
Do not use noBlock if:

the function returns a value
the C code should not continue executing until the function completes execution.
if the following scenario could arise:

funcA (with noBlock) called
funcB called and returns a value or otherwise depends on funcA completing execution,  and funcA uses async keyword.

Use noBlock carefully.

Regarding your 2nd concern:  The C call causes a message to be sent, so that will allocate memory and queue up messages to the JavaScript main thread.  If too many are made before they can process, i imagine eventually there would be a memory issue. It could also cause a lag in the results.  If that were to happen, the api users would need to start waiting (which wouldn't really solve their problem, but would probably make more clear what is happening)..


subSchemas pass validation when an empty array is provided as a value

I have a schema where there is a key with a reference to a subSchema.
// sub schema
const orders = new SimpleSchema({ category: String });

// main schema
const schema = new SimpleSchema({ name: String, orders: orders })

If I pass an empty array as the value for the orders key, I will pass the validation with no data type errors.
// insert call without validation errors
myCollection.insert({ name: "foo", orders: [] })

After looking the code of the typeValidator function in package/lib/validation/typeValidator/index.js (line 27), I noticed that there is a check
(keyValue === Object(keyValue) && !(keyValue instanceof Date))

which is always true when keyValue is an empty array.
Probably not a big issue and highly rare to happen by accident, but maybe a stricter type check for object can take place:
// something like this...
if (keyValue === Object(keyValue) && !Array.isArray(keyValue) && !(keyValue instanceof Date)) return;

Note: The same behaviour also occurs with schemas that defined a key of type object, but not specified any nested keys.
const schema = new SimpleSchema({ name: String, orders: Object });

I'm actually having same problem. I'm using SimplSchema for like 2y now, it got me confused how i didn't check this earlier.

Thanks @vtsio, a fix like you suggest and possibly another check to fix Object sounds good. If someone wants to submit a pull request, I can merge it. Please add test cases for these scenarios, too, to prove that it's fixed.

Since the above pull request was merged I can now close this issue


WTF ???
And some buttons like options won't work
Command Prompt/Terminal logs (if existing)
?

No

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.


serial print of integers

The serial print only works on text, and as such on variables.
Blockly.Blocks['serial_print'] = {
 ...
        .setCheck(Blockly.Types.TEXT.checkList);

Because of this, you cannot just print out what is read on an analog input. Or a number. You need to assign these to a variable, then print the variable.
As you can print integers, why not extend what can be printed to avoid this workaround?

This is a very good point, thanks @bmcage I'll add it in the next update.

Fixed in  6edff27 by removing the block input type, so any block can be attached to serial.print().


space utilities missing responsive variant by default?

v1.3.1
Can't use md:space-x-2.
<EMAIL_ADDRESS>I assume tailwind.config.js is supposed to have:
  variants: {
    space: ['responsive'],
  },

Perhaps same thing for divide?

Have you tried
 variants: {     stack: ['responsive'],   },
Adam used this in the video.

It was renamed away from stack.
Plugin even looks for variants('space'):
https://github.com/tailwindcss/tailwindcss/blob/5b3f8b5461a4e2a0a7eb6169985d37581f609ad5/src/plugins/space.js#L35
Just looks a mistake.
Release notes say:

All of these utilities include responsive variants by default, and their values and variants can be customized using the space, divideWidth, and divideColor configuration keys respectively.

But they're clearly not in the CDN stylesheet.

🤦 Just fixed in 1.3.2, thanks for catching.


import WebGLTile as TileLayer in front page example

👋 @tschaub
(I think it has a sign-off included, but it's a first.)

Hi, @arneke!  Thanks for this fix (quite an oversight to have the top example broken).
We’re still trying to decide how painful the sign-off requirement will be.  Thanks for conforming.
Hope you’re well.

I'm good, hope you are too. Got an Ext 3.4 / OL 2.12 application to rewrite 😅
I edited (my fork) directly in Github and committed with just "import WebGLTile as TileLayer in front page example". I didn't write the sign-off myself, discovered it was already present when I went back to amend.


[Feature Request]: Write rspack config in typescript

What problem does this feature solve?
Webpack config can be written in a ts file (webpack.config.ts). It would be very helpful to be able to do this for rspack as well, since we get type and ESM syntax support, and it makes it much easier to migrate existing configs to rspack. (existing webpack config may also have dependencies on other TS files, so it's not good to have to convert those to JS as well.)
What does the proposed API of configuration look like?
rspack.config.ts - follow webpack.

you can use ts-node/register to handle ts config support right now, we will support ts config out of box in the future

rspack.config.js

require('ts-node/register');
your config here

Hi rspack team, I found webpack uses rechoir and interpret to load JS variants. Is it the desirable way to do it? Or do you prefer using minimal dependencies?

we prefer minimal

What is the performance of using .js vs .ts rspack configuration files?

Also might be  bit early, but any idea how the NodeJS experimental Typescript Support will change the current approach?


CreateAndInject failed - Beta 2.7.5558.0

Moved from CodePlex
Originally submitted by etommVizubi
The same code that was working fine with the previous release, is crashing miserably with the last beta with the following error when starting with CreateAndInject
STATUS_INTERNAL_ERROR: Unknown error in injected assembler code.
I tryed both the source and the binary version.

See Issue #9 for the solution

I can confirm the problem seems solved

RhCreateAndInject for Overwatch.exe and after call it returns the error code:Unknown error in injected assembler code,but RhCreateAndInject for Wow64.exe is ok


state-management: Add libraries to create-react-webpack-project

Chat with @ulises-jeremias and/or @rpmolina to coordinate the efforts to add the examples to the react-extensions repo. After that they will be accessible from https://www.npmjs.com/package/create-react-webpack-project

@ulises-jeremias I would like to also work on this

Sure! I'll create specific tickets for this with .ore information today! @Mogakamo

Thank you @ulises-jeremias


missing the 'pw' tag, which is required for Arrow

Dear,
I get hifi reads using ccs by add --hifi-kinetics parameter. But I get error with missing the 'pw' tag when pbmm2 align and input into gcpp.  After reading documents of ccs software, I find the pw tag is change to fp and rp for both strands.
So, how to use the hifi reads to polished with gcpp or arrow?
Thanks.

You can't polish using gcpp. HiFi reads are already of sufficient accuracy.


Crashes on import() dynamic import

When trying to load the not-a-log module with a dynamic import() expression...
await import("not-a-log").then(mod => mod.default);

...the Proxy attempt to wrap around the 'then' prop and that throws an error:
$ node index.js
file:///Users/josh/repos/repros/node_modules/not-a-log/not-a-log.js:9
    return new Proxy(logger[prop], handler)
           ^

TypeError: Cannot create proxy with a non-object as target or handler
    at Object.get (file:///Users/josh/repos/repros/node_modules/not-a-log/not-a-log.js:9:12)
    at async file:///Users/josh/repos/repros/index.js:1:1

Node.js v22.4.1

I first experienced this trying to run code that imports not-a-log with Vitest: https://github.com/vitest-dev/vitest/issues/6115. @hi-ogawa noted this simpler reproduction in https://github.com/vitest-dev/vitest/issues/6115#issuecomment-2244128356 (thanks again!).

Just to clarify, there is definitely a bug in Vitest and ideally Vitest should support not-a-log as is.
What I meant is that not-a-log can be potentially made more robust around then (or any other unexpected property access) since that can happen with simple await expression (thought it's understandably odd use usage).


Use rename when replacing PascalCase literals

If a literal is already in used and the liter suggests to replace it with
a PascalCase variant use the document rename action instead of simple
text replacement. Because otherwise the code is broken after applying
the linters suggestion.

Thanks a lot!


Tidy up the code

The current implementation is a mess. I guess that's how it goes when you try to code late at night.
Messy implementations result in bugs and are harder to maintain. We need to re-structure, and get something better separated.
I think that the server code (what with callMethod()) gives us some hints on what we might need to do (that's not to say it gets off the hook, though).

The transport code probably shouldn't be in charge of providing the actual API. Also, using .bind() like we are currently is probably pretty bad.

You know what - screw it. I'm just going to rewrite it, and release a 0.2.0.

I gave in and organized things into classes. See branch 1.0-rewrite for progress.

Caller, the abstract class that provides the client API (and does most of the actual client-side JSON-RPC logic), is implemented.
Runner, the abstract class that provides the server API (and does most of the actual server-side JSON-RPC logic), is implemented.
LocalCaller, the Caller implementation that directly calls methods registered to a Runner, is implemented.

In this system, transports subclass Caller and Runner (separately). A Caller subclass only needs to provide the logic to connect to a server, send requests, and notify the super-class of responses. A Runner subclass only needs to provide the logic to listen for requests and send responses.
Note that, currently, the Caller subclass has to relate responses to requests itself, but I've just realized that it doesn't have to. That'll be the next thing I do. Speaking of things I have left to do and order of doing them:

[x] Implement Caller and Runner
[x] Implement LocalCaller
[ ] Link responses to requests from within Caller
[ ] Implement TCPCaller and TCPRunner
[ ] Implement USockCaller and USockRunner
[ ] Create API wrappers so I could have it like I did before (index.js and export.js).

The API might get a bit of a redesign.

We're ready to go! :tada:

Resolved in #6.


[bug]: SidebarMenuButton Tooltip hidden

Describe the bug
The hidden logic in the TooltipContent has a little bug.
I believe this is how it is supposed to be.
return (
  <Tooltip>
    <TooltipTrigger asChild>{button}</TooltipTrigger>
      <TooltipContent
	  side='right'
	  align='center'
	  hidden={state === 'collapsed' || isMobile}
	  {...tooltip}
    />
</Tooltip>
);

Affected component/components
Sidebar
How to reproduce

Add a tooltip for a SidebarMenuButton
Try to hover and trigger the tooltip

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
Mac 15.0.1

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

hmm state === 'collapsed' will hide the tooltip when collapsed. we want it to show on collapsed. wdyt?

You are right. I haven't really thought this scenario through it seems, but now we don't get to show a tooltip in the expanded state, which was what i originally wanted to do.
Though that may not be implemented frequently. So feel free to close the issue. 🙏


test(extension): coverage for new stake pool sorting

Checklist

[ ] JIRA - <link>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Smoke tests are failing due to unstable env


GUI: when coins are soon to be available through timelocked path, link to "recycle coins"

At the moment the user would have to figure out they need to create a spend transaction for those and do this manually.

And doing it manually is far from easy..


Changing vel_set_l/r from short to long

You need to update the version history at the top of each modified file so that the changes are documented. Please provide a new pull request.


Ar language is not supported

Steps to reproduce
well the app is very much good, all the web novels needed are found the issue is that the principles of the arabic language are not settled the text is not in order
Expected behavior
That's what it should be:

Actual behavior
That's what I see:

LNReader version
1.1.18
Android version
android 11
Device
galaxy f13
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.18.
[X] I will fill out all of the requested information in this form.

You can keep this issue open.

You could add this in Custom CSS for fixing the alignment.
body {
  direction:rtl;
}


Improved logging

No need for info logging of successful headers and data. Unnecessary fills up the log output.
Keeping it in track with the request logging in case of a correct response (<300). In case of status >=300, logging the headers and data.
(Personally, i wouldn't log headers and data in warning cases as well, typically, if someone wants to figure out what is happening on a failed call, debug output should be enabled).

Thanks


Tibber reports wrong values

Home assistant v 0.66.1
Tibber plugin
Shows incorrecr max and min price last 2 days from midnight(maybe all the time, but was awake at theese occasions). Max price looks like its the same as current price at 00:00. I convert max and min price to a percentvalue. 0=min and 100=max. All above 60% etx triggers powersaving. Very bad when i got wrong values.. Last night corrwct price updatet aprox 2am.

Could you please try to upgrade to 0.67.1?
Or this fix https://github.com/home-assistant/home-assistant/pull/13865

Jeg går egentlig bare ut ifra at kommunikasjonen kan foregå på norsk her jeg…
Skal oppdatere HA ila kvelden, senest i helga.
Jeg skal ta en titt på githublinken din, men er ennå ikke veldig bevandret i githubverdenen, så må sette meg litt inn i det ser jeg.
Med vennlig hilsen
Tore Austad
Elektriker
Nyvold Installasjon AS
-et selskap i Vintervollgruppen
Inge Krokanns vei 14 , 7340 Oppdal
Telefon: 72400600 Mobil: 92491059
E-post:  Tore.Austad<EMAIL_ADDRESS>www.nyinst.nohttp://www.nyinst.no
<EMAIL_ADDRESS>Fra: Daniel Høyer Iversen<EMAIL_ADDRESS>Sendt: fredag 20. april 2018 06.28
Til: home-assistant/home-assistant<EMAIL_ADDRESS>Kopi: Tore Austad<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [home-assistant/home-assistant] Tibber reports wrong values (#14009)
Could you please try to upgrade to 0.67.1?
Or this fix #13865https://github.com/home-assistant/home-assistant/pull/13865
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/home-assistant/home-assistant/issues/14009#issuecomment-382971812, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AkvZODn1ItRdRT4DpCxgdWRjsvoC0Jyqks5tqWPkgaJpZM4TcmQB.

You can try to upgrade to 0.67.1 first

This looks fine.
HA shows the current and the historic price (not the future values)
The max/min values should be max/min over the current day

I dont belive this is correct.
Its flawless from 2-4am. Now the sensor is reporting min_price of 0.318 and max_price 0.456. This is correct as of the screenshot from tibber-abb, but the values shown from screenshot 00.12 is not correct. Its not critical, i can just set my automations to not run 00.00-02.00, but the best would be correct update at midnight
I scale this price to a 0-100 value, where min_price is 0, and max_price is 100. I have a RGB strip that shows this value, 0% is green 100% bright and 100 is red 100%, 50% is 50/50 red green etc. 20.04.20018 between 00.00 and 0200 it was 100% red, and of course tibber sensor reporting max_price same as current(even tho the max price reported directly in tibber app was about 20 øre more or so. also weird/wrong min_price. At 02.00 the rgbstrip turned perfectly green and tibber sensor reported correct values…
Med vennlig hilsen
Tore Austad
Elektriker
Nyvold Installasjon AS
-et selskap i Vintervollgruppen
Inge Krokanns vei 14 , 7340 Oppdal
Telefon: 72400600 Mobil: 92491059
E-post:  Tore.Austad<EMAIL_ADDRESS>www.nyinst.nohttp://www.nyinst.no
<EMAIL_ADDRESS>Fra: Daniel Høyer Iversen<EMAIL_ADDRESS>Sendt: 21. april 2018 08:36
Til: home-assistant/home-assistant<EMAIL_ADDRESS>Kopi: Tore Austad<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [home-assistant/home-assistant] Tibber reports wrong values (#14009)
This looks fine.
HA shows the current and the historic price (not the future values)
The max/min values should be max/min over the current day
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/home-assistant/home-assistant/issues/14009#issuecomment-383272026, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AkvZOO7hbtnwcna5MUY0Ikty_0vgAH_Aks5tqtNRgaJpZM4TcmQB.

Maybe there is a timezone issue, so it would be great if you could try the latest version.

I been looking at this now, and realise i have no clue on how to actually perform this update. I will look further into it, but if youve had the time to put me in the right direction id guess that save me a lot of time…
Med vennlig hilsen
Tore Austad
Elektriker
Nyvold Installasjon AS
-et selskap i Vintervollgruppen
Inge Krokanns vei 14 , 7340 Oppdal
Telefon: 72400600 Mobil: 92491059
E-post:  Tore.Austad<EMAIL_ADDRESS>www.nyinst.nohttp://www.nyinst.no
<EMAIL_ADDRESS>Fra: Daniel Høyer Iversen<EMAIL_ADDRESS>Sendt: 21. april 2018 14:11
Til: home-assistant/home-assistant<EMAIL_ADDRESS>Kopi: Tore Austad<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [home-assistant/home-assistant] Tibber reports wrong values (#14009)
Maybe there is a timezone issue, so it would be great if you could try the latest version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/home-assistant/home-assistant/issues/14009#issuecomment-383290246, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AkvZOITqPv5CntSuimTIXBWiw1qsXek9ks5tqyG-gaJpZM4TcmQB.

Hello @austadt. Sorry to interfere in this thread, but I would really like to see your HA config regarding showing % of min/max and showing this on a RGB strip. Seems like something I could find neat :)

An update: It seems to be a timezone issue when calculating the min/max values. I hope to make a fix tonight.

How will this update be available for use? Is there som sort of automatic sensor code update or do i manually need to do somethi g?
Much appreciated  the work your doing! :)
I will share my code when i have my laptop around. Tonight, or tomorrow morning @elbogi

The last Tibber PR (#13865) is already merged and in the scheduled 0.68 release for 27.04. This fix (timezone issues) will most likely be in the next one, coming in approx. 14 days, so unless you want to run RC (release candidates) of HA, you should probaly wait :)


[Bug] Access Denied on download attempt (Mac)

What did I do
Hi, I tried to download the mqttx cli for Mac. I clicked on the download link or tried to run the shell script on https://mqttx.app/cli in the "Mac" tab.
What happened
The following error occured:
<Error>
  AccessDenied
  <Message>Access Denied</Message>
  <RequestId>5VMPB8C4DX8E74AR</RequestId>
  <HostId>N1n/rY7NY0lqt0YnrthfGgH5BM6W7uaEH7hmX//9+YaLXyHzKx6oDBGUd+ikW22Eba4QMtae4Y4=</HostId>
</Error>

Expected
A valid download for the mqttx cli.
Environment

OS: OS version, e.g. macOS Ventura

@kpko I'm sorry for the inconvenience. Indeed, there was an issue with the file we uploaded previously. We have since corrected the problem. Now, you should be able to download the mqttx cli without issues.
We highly recommend you use Homebrew to install mqttx cli. Run the following command in your terminal:
brew install emqx/mqttx/mqttx-cli

Thanks for your feedback and use!

If this issue does not occur again, I will close it. Thanks for your feedback.


Add ability to configure host node volume size.

By default the host volume size is 20gb which can be a bit lacking especially with larger machines.
I can make a pr if you'd like

@ewhal thanks!
Are you talking about https://www.terraform.io/docs/providers/aws/r/launch_template.html#block_device_mappings?
If yes, then it's supported:
https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/blob/master/main.tf#L17
https://github.com/cloudposse/terraform-aws-eks-workers/blob/master/main.tf#L201
If it's something else, PRs are very welcome.
Thanks

Configuring block device mappings doesn't seem to be working for me.
This configuration
block_device_mappings                  = [{
    virtual_name = "ephemeral0"
    device_name = "/dev/xvda"
    no_device = false
    ebs = {
      delete_on_termination = true
      volume_size = 50
      volume_type = "gp2"
      encrypted = false
    }
  }]

Gives me the following error:
attribute "ebs": attributes "iops", "kms_key_id", and "snapshot_id" are required.

If I give those attributes empty values and set the iops to 100, then my autoscaling groups don't work. They fail to scale new instances with the following error:
The parameter iops is not supported for gp2 volumes. Launching EC2 instance failed.

@aknysh any idea? Love this module so I'd love to get this working.

@CodechCFA
Take a look https://www.terraform.io/docs/configuration/expressions.html
null: a value that represents absence or omission. If you set an argument of a resource or module to null, Terraform behaves as though you had completely omitted it — it will use the argument's default value if it has one, or raise an error if the argument is mandatory

You can set all those attributes that are not used in your particular setup to null and terraform will behave as if they were not specified at all.

Ah that does work thanks @aknysh . Everything I saw in the terraform docs with block device mappings just omitted the fields that weren't needed so figured this would have worked the same way.


Changed reference to old m-lab-tools repo to point to simply m-lab.

The m-lab-tools Github account is no longer used, and everything in it was moved to the m-lab account.  This PR just updates the README.md file to reference the correct account.

LGTM.


Edit Event

create Edit Event views, refer to Create Event (Admin View): https://www.figma.com/file/h85P5QFeMPJlRy9piBQQxH/Wireframes?node-id=483%3A34165. For this task, reuse the create event components but populate the fields with existing event data.

/cib edit-event

Branch 67-edit-event created!


Update for zig 0.13

This fixes the build.zig script for zig 0.13 (tested with 0.13.0-dev.351+64ef45eb0 to be precise) and updates integration instructions in the Readme to use a build.zig.zon file.
zig-cache was also changed to .zig-cache, so I updated .gitignore accordingly.

Thank you!


DataTableColumnHeader sort arrow not linked to sort function

The DataTableColumnHeader component allows providing an onClick property, probably to allow for calling the sort function.  However this is not linked to the sort arrow direction which is only updated when you click on the inner button element.

Will this be fixed with the current DataTable PR @joshblack ?

Yupp! Should be fixed in #562


Color picker not available on mobile browser

Hi,
I was testing the widgets_overview_app.py example on different platforms and saw that the color picker tool is not visible to Safari on iPhone. It works perfectly on Chrome and MS Edge, although with different looks. Is this a known thing? The color picker is the center element for my project (I am building a kinetic art project where I will be changing the color of an led strip from web interface using a cell phone) so does anyone know how to solve this?
Thanks & Best Regards,

hello @bashkan it should be simple to implement a custom widget, with 3 Slider or SpinBox for red geen blue values to setup a color value, maybe you can add also a simple rectangle that changes the background color with the choosen rgb value. do you know how to do this?

eventually you can change the page view as desktop. is it possible on iPhone? Android has this option

Hello @dddomodossola ,
Your idea with 3 sliders is great! I will definitely work on that. But I don't know how to do the rectangle that changes background color. Can you please tell how to do that? Or at least how I should research it?
Regarding viewing the page as desktop, I don't think that is possible on iPhone.
Thanks for your reply and great idea :)

you can use the standard Widget and size it as you want. the backgroud can be set as
mywidget.style ['background-color']= 'rgb(%s, %s, %s)'%(sliderRed.get_value(), sliderGreen.get_value(), sliderBlue.get_value())

@dddomodossola, you rock bro :)
I will make the custom widget right now.
Thanks again...

@bashkan you are welcome ;-)

@dddomodossola,
I am sorry to bother you with this subject again. But I couldn't make it work no matter what I did. I have created a vBox with some buttons and sliders to change the color of another vBox which is on it. However to color of the vBox is grey and does not change when I move the sliders. Do I need to create onChange events for the sliders?
Here is the code for the vBox that I want to change the color: (I didn't include the whole code so it is easier to see it)
color_box = gui.VBox(style={'width':'300px', 'height':'100px', 'margin':'0px'})  color_box.style['background-color'] = 'rgb(%s, %s, %s)'%(self.red_slider.get_value(), self.green_slider.get_value(), self.blue_slider.get_value())
And here is the screenshot:

hello @bashkan , yes you should execute that code when onchange event happens on sliders

Worked exactly as you said :)
Many thanks...

wonderful :D have a good code development


Add validation to phone number fields

See #87
^.{0,20}$
Edit: This just checks for any character between 0-20 times, which feels not too necessary, but I guess we can implement it anyway.

Looking into this more: the regex is only for ECF phone numbers. Maybe we should use Docassemble's phone number validation function instead, since the only place we handle it is with attorneys and service contacts.


[sc-59294] Creating the httpscaledobject file for the Keda project

https://app.shortcut.com/findhotel/story/59294/research-and-poc-a-way-to-sleep-and-wake-pods-for-review-envs

httpscaledobject.yml file created;
Ref.: https://github.com/kedacore/http-add-on/blob/main/docs/ref/v0.2.0/http_scaled_object.md
kustomization.yaml file updated by adding a new patch setting and the httpscaledobject file;
KUSTOMIZE_VERSION updated to v5.0.1;
KUBECTL_VERSION updated to v1.24.11;

By using the Keda project to handle the HTTP request in our Review Environments, we don't need to take care of an ingress setting per each review environment because it should be moved to the keda namespace where we will have a wildcard covering all review environments then per each environment we just need to have an HTTPScaledObject where keda needs to collect metrics to scaling up/down the environment checking the HTTP requests.

I left some comments. Besides those, I missing update the docs about adding instruction on how to install keda and how to use or no kead.

Sure, I will update our docs.


Update adjusted server timestamp client.py

API error code -1021 server timestamp ahead 1000 ms
line 155
lot of users and myself enjoying this wrapper are running into the same errors. this should fix the inevitable time stamp error that occurs with frequent continual usage.

Hi @kyungtaekster sorry the issue with server time is on the client end. You need to sync time.
Also your patch effectively adds another call server to every signed request which is very inefficient.


SSL configuration

Add letsencrypt SSL certificate and automate certificate renew using Docker-compose + Nginx.

@g4s8/z I'm not managing this repo, remove the webhook or contact me in Slack, as explained in §11; I'm not managing artipie/central GitHub repository, you have to contact me in Slack first, see §11 /cc @yegor256/z

I'm not managing artipie/central GitHub repository, you have to contact me in Slack first, see §11

I caught it

I'm not managing artipie/central GitHub repository, you have to contact me in Slack first, see §11


nextra docs

Moving graphql-debugger.com to a site built with nextra and overhauling content on the way.
https://preview.graphql-debugger.com

https://github.com/rocket-connect/graphql-debugger/pull/212


Feature/polylane

This adds new lane classes with arbitrary shapes defined by polylines.
I admit that they are not pretty, but we did verify that the IDM model is still able do drive smoothly.
Use case:
We want to simulate with highway-env on custom real-world maps, i.e. we have to convert arbitrary maps to road graph format. It's much easier and flexible when we can just sample the lane boundaries and generate the "polylane".
I hope that this also enables others to write map converters -- let's see.

Codecov Report

Merging #267 (c7f99b5) into master (d75a1f3) will decrease coverage by 0.00%.
The diff coverage is 86.25%.

@@            Coverage Diff             @@
##           master     #267      +/-   ##
==========================================
- Coverage   77.51%   77.50%   -0.01%
==========================================
  Files          43       44       +1
  Lines        3949     4108     +159
==========================================
+ Hits         3061     3184     +123
- Misses        888      924      +36

Flag
Coverage Δ

unittests
77.50% <86.25%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

highway_env/road/lane.py
83.17% <75.86%> (-2.72%)
:arrow_down:

highway_env/road/spline.py
90.36% <90.36%> (ø)

tests/road/test_road.py
100.00% <100.00%> (ø)

highway_env/road/regulation.py
67.30% <0.00%> (-25.00%)
:arrow_down:

highway_env/envs/summon_env.py
94.91% <0.00%> (-1.70%)
:arrow_down:

Continue to review full report at Codecov.

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

Hi @m-stoll
Thank you very much for this PR, I think this is an incredibly useful feature!
The need for supporting arbitrary geometries has come up several times, including recently see e.g. https://github.com/eleurent/highway-env/issues/240

very nice implementation with great coding style and unit tests :D

Do you also have an environment that makes use of these PolyLanes? It would also be a great addition to the repository, I think. Please feel free to submit it in the future if that is the case.

Your kind reply is very much appreciated!
Well, our environments don't have more complex topologies compared to what is possible with highway-env natively, it's just that they match a custom map (as in the issue you linked). So at the moment, I don't think there's much benefit from that.
I'll let you know when there's more to share ;-)


Update 'install from conda-forge' workflow

The current version of the 'install from conda-forge' workflow includes pinned versions of anaconda, miniconda, and jpype, and these can/should possibly be updated. The conda versions are from last year, while jpype has version 1.2.1, and we now have compatibility with jpype 1.5.0 for python 3.11 and later.

Postponing due to lack of time/urgency.


Updating a PR should not throw an exception

process.slurp throws an exception in some cases (see stacktrace below), this shouldn't be the case
[2020-04-02T13:42:29.260Z] 2020-04-02 13:42:29,083 INFO  Process update org.scala-sbt:sbt : 1.3.8 -> 1.3.9
[2020-04-02T13:42:29.644Z] 2020-04-02 13:42:29,444 INFO  Found PR https://github.com/AutoScout24/XXX/pull/123
[2020-04-02T13:42:29.644Z] 2020-04-02 13:42:29,529 INFO  PR update strategy is set to always
[2020-04-02T13:42:29.644Z] 2020-04-02 13:42:29,529 INFO  Merge branch 'master' into update/sbt-1.3.9 and apply again
[2020-04-02T13:42:29.971Z] java.io.IOException: 'git checkout master' exited with code 1
[2020-04-02T13:42:29.971Z] error: you need to resolve your current index first
[2020-04-02T13:42:29.971Z] csv-to-json/project/build.properties: needs merge
[2020-04-02T13:42:29.971Z]
[2020-04-02T13:42:29.971Z]     at org.scalasteward.core.io.process$.$anonfun$slurp$9(process.scala:48)
[2020-04-02T13:42:29.971Z]     at org.scalasteward.core.io.process$.$anonfun$slurp$9$adapted(process.scala:44)

I'm looking into this

I see in the logs
CONFLICT (modify/delete): some-file deleted in master and modified in HEAD. Version HEAD of some-file left in tree.

It looks like this happens when a modified file is deleted on master

I think i was able to reproduce the issue in tests on https://github.com/kiranbayram/scala-steward/tree/issue-1393 branch

I unfortunately couldn't find a git command which would successfully do the merge in this case :(


Did you support mypy?

As title.

Doesn't look like I have a config for that, I can take a look at it later today and add it if I can.
Else others are free to make a PR on it if interested!

Added in eed632a
Try it out and let me know if there is anything missing.

I'll keep this open for a couple days, if there are no issues then I will close it.

Could this be added to supported-linters-and-formatters.md?

Done


Errornous files make idris2-lsp output compiler errors to stdout

When using idris2-lsp together with kak-lsp, it seemed the connection would randomly break every once in a while, and I had to restart it. After some investigation, it seems if the file being edited contains a compilation error, the error is outputted to stdout and thus sent to the LSP client. I'm guessing the clients that are used to test idris2-lsp will silently ignore this invalid data and thus continue working, but kak-lsp does not, triggering an error (ERRO Failed to parse header, module: kak_lsp::language_server_transport:74).
Here's a small example triggering the error: the following file, named X.idr, in the project /tmp/test-project:
main : IO ()
main = do
    putStrLn "Hello"
    return 5 -- Error!

Looking at the exact output of the lsp client (teeing the stdout of the server), gives the following: (full file attached below)
Content-Length: 1187

{"jsonrpc":"2.0","id":0,"result":{"capabilities"[...]1/1: Building X (/private/tmp/test-project/X.idr)
\x1b[38;5;9;1mError:\x1b[0m \x1b[1mWhile processing right hand side of \x1b[38;5;5;1mmain\x1b[0;1m\x1b[0m. \x1b[1mUndefined name \x1b[38;5;5;1mreturn\x1b[0;1m.\x1b[0m

\x1b[38;5;12mX:4:5--4:11\x1b[0m
 \x1b[38;5;12m1 |\x1b[0m main : IO ()
 \x1b[38;5;12m2 |\x1b[0m main = do
 \x1b[38;5;12m3 |\x1b[0m     putStrLn "Hello"
 \x1b[38;5;12m4 |\x1b[0m     return 5 -- Error!
         \x1b[38;5;9;1m^^^^^^\x1b[0m

Content-Length: 331

{"jsonrpc":"2.0","method":"textDocument/publishDiagnostics","params":{"uri":"file:///private/tmp/test-project/X.idr","version":1,"diagnostics":[{"range":{"start":{"line":3,"character":4},"end":{"line":3,"character":10}},"severity":1,"source":"idris2","message":"While processing right hand side of main. Undefined name return."}]}}

The \x1bs are the literal escape character, used by the Idris2 compiler to output colors, here's what the file looks like when cated.

Conceptually, the solution should be quite simple: disable the compiler output when compiling (or redirect it to stderr).
idr.log

Thanks for the issue. As mentioned in other issue and in other discussions, we are aware of the issue and we will work on a patch for the compiler.


import locale 20170321

Coverage increased (+0.01%) to 1.267% when pulling 3b5e9a557c8e4822d13c9c82ce669f33bb757697 on u9520107:import-locale-20170321 into 8cdb8e590aaa30c8c9684cdfa940a6ec62327246 on ringcentral:master.


Is this library still actively maintained?

If it's not, I would love to take it over.

It's so sad, that this powerful library is not up-to-date with the current SvelteKit Version, what can we do about that?

It's so sad, that this powerful library is not up-to-date with the current SvelteKit Version, what can we do about that?

I was hoping to have the current owner migrate to me otherwise I'll have to make another separate repo. I wanted to submit PRs but evidently the owner doesn't watch this repo and so I doubt there would be any merging even if I create a good PR with updates...

Yeah I think he doesn't look over this repo anymore.. unfortunately. Would you do another repo? And is it fine with the rights to do that? I would be a big fan of this library to be still maintained. You have my support! :)

Maybe it would be possible to send him a email or something.. i want this library so bad.

Yeah I think he doesn't look over this repo anymore.. unfortunately. Would you do another repo? And is it fine with the rights to do that? I would be a big fan of this library to be still maintained. You have my support! :)

That was one of the main reasons I reached out, but not super sure what to do with no response. Now most can be achieved with transitions and view transitions API now but still I used this framer motion library and like the declarative nature. And would like to make components that wrap with popmotion and svelte transitions and view transitions API

Maybe it would be possible to send him a email or something.. i want this library so bad.

That's a good idea, I'll try to email later and see if I get a response 👍

Yes, I really fell in love with the declarative way of animations. I think svelte-motion is the only one who achieved it so powerful!
Would really appreciate it, if u can keep me updated.. you have my support by the way. I would be down to support you with my contributions.

I'd like to follow updates here as well! :)

So far I've reached out by email and have not gotten a response back yet.

It's so sad, that this powerful library is not up-to-date with the current SvelteKit Version, what can we do about that?

I was hoping to have the current owner migrate to me otherwise I'll have to make another separate repo. I wanted to submit PRs but evidently the owner doesn't watch this repo and so I doubt there would be any merging even if I create a good PR with updates...

I think he does reply on PRs, at least he replied to mine updating all the dependencies. So, I would suggest that if you have some good updates, then just try and drop a PR.

It's so sad, that this powerful library is not up-to-date with the current SvelteKit Version, what can we do about that?

I was hoping to have the current owner migrate to me otherwise I'll have to make another separate repo. I wanted to submit PRs but evidently the owner doesn't watch this repo and so I doubt there would be any merging even if I create a good PR with updates...

I think he does reply on PRs, at least he replied to mine updating all the dependencies. So, I would suggest that if you have some good updates, then just try and drop a PR.

awesome, thank you! I might, the main idea I had was to update to support svelte 5 which the component type needs to change to support svelte 4 and 5. also there is a bug with AnimatePresence with it's type declaration.

So, my PR is merged. I guess simply try to open one yourself :)

Update: I will be opening PR soon with some of my ideas hopefully it gets merged 👍

Update: sent PR with the minor changes I mentioned (component type and AnimatePresence bug fix)

The library is great, but I feel like the updates from the original author are quite slow. I got it, life happens - but @JonathonRP seems to be keen to keep it alive?
@JonathonRP - would you consider re-publishing it under another name? rp-motion? :)

I might, one day but I don't want to take it from the current author. He hasn't experienced interest in letting me fork and take ownership so I'm just making contributions and have a port to typescript PR and plans to bring it more inline with modern features and implementation of framer motion

Whatever happens I'd love to be able to use this in my projects again. @micha-lmxt Please consider allowing @JonathonRP to fork or transfer to them if you are no longer interested in maintaining - no hard feelings :)

Anyone interested in porting this project over to svelte5/ kit2?

Anyone interested in porting this project over to svelte5/ kit2?

I'm doing that, and while porting to typescript btw

@JonathonRP anywhere we could track progress and/or assist in any way?

@JonathonRP anywhere we could track progress and/or assist in any way?

https://github.com/micha-lmxt/svelte-motion/pull/47 is the pull request, been doing my best with commit messages to track progress. I have some changes not completely tested on branch right now but I'll be checking them in soon. Could use help testing, last test the functionality and animations were not working but I think I finally figured out why, but unable to test with my project that I've been testing with. That's actually the reason for the slowness in progress. I have aspersions to update and add features that are missing in this lib from original as original is still maintained, and also looking at using svelte built in transitions/ animations down the road.

@JonathonRP anywhere we could track progress and/or assist in any way?

#47 is the pull request, been doing my best with commit messages to track progress. I have some changes not completely tested on branch right now but I'll be checking them in soon. Could use help testing, last test the functionality and animations were not working but I think I finally figured out why, but unable to test with my project that I've been testing with. That's actually the reason for the slowness in progress.
Note: I have aspersions to update and add features that are missing in this lib from original as original is still maintained, and also looking at using svelte built in transitions/ animations down the road. This typescript port begins taking steps in that direction.

changes are pushed, the main thing that could help speed process is having others test that functionality didn't change

How can I  help. Also will this pr be merged? @JonathonRP

How can I help. Also will this pr be merged? @JonathonRP

@cliffordkleinsr, if you are able to test functionality and contribute to PR changes that would be great help. the current PR is just to port to typescript and not change or break any current functionality.
my current understanding is it will get merged by current maintainer, I asked a question in issues here if he would accept a typescript port PR, he said yes. The PR is currently draft because during my testing functionality was broken but I think I fixed it, just need the latest changes tested to make sure that functionality is truly working and unchanged.

I honestly feel like it does not make much sense to wait on current maintainer - he's been busy/quiet for past couple of months (I also see that he became inactive on twitter).
It would totally make sense to fork this into a new library. I feel like svelte-motion is a bit confusing name to begin with (I know it comes from Framer motion), as svelte itself has a svelte/motion module.

I may fork if he doesn't merge but I wanted to have a resource for learning and self taught to learn about animation which is made easier by using typescript.
What name do you think would be good for a fork?

I don't know, some examples

rp-motion (gotta give yourself some credit)
declarative-motion (still motion, but not to be confused with svelte/motion)
motion-kit or motionkit (available on npm)
svelte-moves - a bit funky :)

@ermincelikovic
I was thinking

remotion
svelte/animation or svelte/animations
motion-start

I don't know, some examples

rp-motion (gotta give yourself some credit)
declarative-motion (still motion, but not to be confused with svelte/motion)
motion-kit or motionkit (available on npm)
svelte-moves - a bit funky :)

@ermincelikovic I was thinking

remotion
svelte/animation or svelte/animations
motion-start

motion-start is great imho!
remotion is a product (google it) - would be easily mistaken
svelte/animation could be mistaken for svelte/animate

Yeah I was actually back and forth between remotion and re-motion -little play on my name Reese but there's also something really nice about motion-start

svelte-framer
svelte-framerMotion
svelterMotion
@ermincelikovic @JonathonRP

svelte-framer
svelte-framerMotion
svelterMotion
@ermincelikovic @JonathonRP

I like svelteMotion.
I thought of some others too,

svelte/motion-start
svelte/framer-motion

+1 for svelte/framer-motion
Honestly  have been itching to port the library into svelte5

Check on npm packaging looks like I can't have svelte/framer-motion, because npm only allows @org/package name so it would have to be @svelte/framer-motion which I don't think svelte maintainers would appreciate... So it would have to be motion-start (framer-motion is the package) and (svelte-framer-motion seems a little too verbose) I could be swayed to svelte-motion-start

Welp, I guess that'll have to do. Can we register it? @JonathonRP

Welp, I guess that'll have to do. Can we register it? @JonathonRP

I will try to figureout how

Will also check tomorrow . Out of the office IRL

Will also check tomorrow . Out of the office IRL

Which name are you registering tomorrow,
svelte-motion-start or motion-start

I vote for motion-start

This

@cliffordkleinsr, @ermincelikovic btw the typescript port is basically ready aside from testing that no functionality was added or lost.

will try and do some tests tomorrow. Which org should we register for the package?

will try and do some tests tomorrow. Which org should we register for the package? Should also have a discord or something.

I wasn't going to us an org for the package...
if you want to setup discord server that is fine.
Also we should use changesets with this new package - this should have necessary information to add changesets to the code base and typescript port code base.

Will do then. Let's start with the repo forking into the new  motion-start lib.

Please fork @JonathonRP . Let's get this show on the road

@cliffordkleinsr, I already have a fork that was used to create PRs. https://github.com/JonathonRP/svelte-motion. should I change repo name and package.json name to motion-start or make it a template repo and make another with mentioned name?

@cliffordkleinsr, @ermincelikovic  please find new repo - https://github.com/JonathonRP/motion-start

Good start @JonathonRP . Will register the npm package when I get to the office.

Hey @JonathonRP Some good news I've created a bunch of UI tests you can run using the motion-start local repo here. make sure to do an npm link on the motion lib so you can test UI errors locally.
Some bad news is however that all the UI tests apart from the Animatepresence stack have failed. There might be some type errors with the Motion and MotionSSR components. There are also a lot of unused exports on the type definitions for these components

Good start @JonathonRP. Looks like you already registered the package.

I thought I already registered the name motion-start with npm by publishing the package to npm

Hey @JonathonRP Some good news I've created a bunch of UI tests you can run using the motion-start local repo here. You can do an npm link on the motion start  forked repo so you can test UI errors locally on the motion-test repository that I have shared
Some bad news is however that all the UI tests apart from the Animatepresence stack have failed. There might be some type errors with the Motion and MotionSSR components. There are also a lot of unused exports on the type definitions for these components

Interesting looking at this you're using svelte-motion and not motion-start

I tested with both. It's only worked for animatedpresence with the motion-start  library

Here is the discord btw no roles yet: https://discord.gg/CV48hNR7

Good news!
Framer Motion is now independent, introducing Motion.

For a long time, users have asked to use Framer Motion outside of React. Either with vanilla JS, or other frameworks like Vue.
Today, Motion becomes more than a React library, with an amazing set of vanilla APIs that bring Motion's hybrid engine and extensive feature set to everyone.


Fix ComplexLayoutApp not refreshed by state changes

Description
This PR fixes a bug that the controls in ComplexLayoutApp's drawer menu fail to update when tapped, because they're not hooked to their parent's properties.
Related Issues

N/A

Tests
I added the following tests:
Replace this with a list of the tests that you added as part of this PR. A change in behaviour with no test covering it
will likely get reverted accidentally sooner or later. PRs must include tests for all changed/updated/fixed behaviors. See Test Coverage.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

Close since we dont want to disturb the benchmark unless necessary


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1267 ms

Resolved: Bofrost.de performance has improved in 67b67b7.


Query entries in preview with token within sprig request

Question
We have a Sprig component which loads a part of the entry dynamically. However, when this entry is not enabled, a query like craft.entries().id(_productId).one() won´t return the entry since its not enabled yet. (_productId is provided by the sprig component) This happens for instance in craft´s preview. I can query the entry when using status(null) but a more elegant way is to use the preview token which is available through the query params. Do you know how I can use the preview token to query an entry which is not enabled yet?
Update: It also looks like I can´t use craft.request.isLivePreview() to determine if I should use status(null) or not. On the initial page load it is true, when dynamically loading this via sprig, it returns false in preview

This is a tricky one but let me look into it and I'll see if Sprig can do some magic with that token.

I've made it so that Sprig components now pass the token parameter along so that they work seamlessly in a live preview scenario. You can test this manually by copying the addition in this commit: https://github.com/putyourlightson/craft-sprig-core/commit/cc7b597897245b9f0d68ca2c4ebdeb0aef37c760

Released in version 1.9.0.

Hi Ben - Thank you for putting this into a new version!


Move limits from export section over to review set

In AeD, there is a distinction between "add to review set" vs final export. The "Add to review set" job lines up with the export limits in Core eD and they need to be added to the Review Set limits.
The export limits should only include limits related to exporting or downloading from Review set. Please make distinction clear between download and export.

@StefanieBier I revised your suggested changes. Can you please take a quick look and make sure they look ok. Thanks, markjjo.


Suppress exceptions by malformed LOG, such as:

Fatal Exception: java.util.MissingFormatArgumentException
Format specifier: 20V
Signed-off-by: Francisco Franco<EMAIL_ADDRESS>

Merged from my phone, but the Travis.ci log says there's a missing import 😉


Openssl git master build failure cross building Debian Stretch x86_64 to Debian Stretch arm64

I have an action that was compiling, successfully, openssl on multiple Debian based platforms.
All the configuration and settings are viewable in the modular actions steps.
https://github.com/userdocs/openssl-crossbuild/runs/5345431576?check_suite_focus=true

Note about this build
I use action built cmake + zlib-ng 1.2.11 (compatibility mode) but it was working fine previously and it's only recently the build started failing on this OS + arch
It's failing on arm64 + Stretch and this is where it fails
aarch64-linux-gnu-gcc  -Icrypto -I. -Iinclude -Iproviders/common/include -Iproviders/implementations/include  -DECP_NISTZ256_ASM -DKECCAK1600_ASM -DOPENSSL_BN_ASM_MONT -DOPENSSL_CPUID_OBJ -DOPENSSL_SM3_ASM -DPOLY1305_ASM -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DSM4_ASM -DVPAES_ASM -fPIC -pthread -Wa,--noexecstack -Wall -O3 -DOPENSSL_USE_NODELETE -DOPENSSL_PIC -DOPENSSLDIR="\"/etc/ssl\"" -DENGINESDIR="\"/root/build/opt/local/lib/engines-3\"" -DMODULESDIR="\"/root/build/opt/local/lib/ossl-modules\"" -DOPENSSL_BUILDING_OPENSSL -DZLIB -DNDEBUG -I/opt/local/include -c -o crypto/modes/libcrypto-shlib-aes-gcm-armv8-unroll8_64.o crypto/modes/aes-gcm-armv8-unroll8_64.S
crypto/modes/aes-gcm-armv8-unroll8_64.S: Assembler messages:
crypto/modes/aes-gcm-armv8-unroll8_64.S:5: Error: unknown architectural extension `sha3'
Makefile:9942: recipe for target 'crypto/modes/libcrypto-shlib-aes-gcm-armv8-unroll8_64.o' failed
make[1]: *** [crypto/modes/libcrypto-shlib-aes-gcm-armv8-unroll8_64.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory '/root/openssl'
make: *** [build_sw] Error 2
Makefile:3239: recipe for target 'build_sw' failed
Error: Process completed with exit code 2.

I have tried to google the error code but it does not help me.
Can anyone help me understand what might be going wrong?

This is regression on older assemblers from #15916
@xkqian Could you please look?

Sure， seems it's caused by the compiler versions. I will triage it.


Av sync rtcp

Implemented a simple version of rtcp a/v sync for the recorder.
I'm new to C, so I need someone to help me find bugs. this implementation seems working, but since chrome will choose random start rtp timestamp, I don't know how it reacts to overflow or negative rtp.
In 2 days testing found no problems.
Tested with pcmu / opus

No probl,
I made a better implementation in this PR https://github.com/ging/licode/pull/549
Unfortunately, as I wrote in the PR, I'm absolutely new to C and it's my first piece of code in this language. I really don't know where to start to make some unit tests, and currently my job schedule doesn't allow me to take some time to implement.
I'm glad to share my code and ideas, if anyone could help me with tests I'll be very glad.
Otherwise, I'll leave the PR opened, and hope in the future to be able to do these tests.
I'll close this one since it's quite obsolete


README Dev Docs Link is broken

Further Reading
PayID Developer Documentation   <<--- https://docs.payid.org/docs/payid-overview

Should be -->>  https://docs.payid.org/payid-overview
Link is broken

Should be fixed by https://github.com/payid-org/payid/pull/537. Thanks for logging the bug!


[* Currency] Added support for compound Currency entities in Portuguese and Spanish (#2479)

Partial fix to issue #2479.
Test cases added to PT and ES CurrencyModels.

@LionbridgeCSII as this partially fixes the referenced issue. Can you look into fixing the reported German case too?

@LionbridgeCSII as this partially fixes the referenced issue. Can you look into fixing the reported German case too?

ah, I missed the German case, yes I will address it too.

If you can propagate the fix to French, Dutch, Italian, and Swedish that would be great too! The bug was open as "*". ☺


Add failing test for sign in with multiple attrs

This is a failing test case to reproduce the error referenced in #54. We should close that issue in favor of this one.
Problem
The recommended implementation for sign in by email or username throws an error when looking up the User record.
Stack trace:
  1) User signs in with username or password
     Failure/Error: params.to_h

     TypeError:
       wrong element type String at 0 (expected array)
     # ./lib/monban/param_transformer.rb:25:in `to_h'
     # ./lib/monban/param_transformer.rb:25:in `sanitized_params'
     # ./lib/monban/param_transformer.rb:15:in `to_h'
     # ./lib/monban.rb:66:in `transform_params'
     # ./lib/monban/configuration.rb:57:in `block in default_find_method'
     # ./lib/monban.rb:90:in `lookup'
     # ./lib/monban/controller_helpers.rb:105:in `authenticate_session'
     # ./spec/rails_app/app/controllers/multiple_attributes_sessions_controller.rb:6:in `create'
     # ./lib/monban/back_door.rb:30:in `call'
     # ./spec/features/user/user_signs_in_spec.rb:25:in `block (2 levels) in <top (required)>'

@halogenandtoast The test passes on my local machine. Not sure what's going on - any ideas?

@graysonwright can you provide version numbers for your local setup:
Ruby/Rails/OS and anything else that might be relevant.

@halogenandtoast I think it's good to go.

@graysonwright can you just do a small write up about your solution. I assume the problem is that the params were being transformed twice. Was that the case?

@halogenandtoast As far as I can remember, yes - the params were being transformed twice.
It's been a while since I submitted this PR, and I've forgotten the exact details. But I remember that when the params were passed to the original default_find_method, they had already been transformed and were ready to submit directly to Model.find_by. Transforming them again broke the find_by call.


Add some more tests

This adds tests for more aspects of the hash, mainly testing the happy path for various inputs, but also a few other things. It also refactors the existing tests a bit, to make things easier.
Most of the new hashes were produced using the ed2k_hash tool (which produces blue), so they should be correct. The exception is the hashes for an empty input, for which ed2k_hash errors out; I don't have any outside verification that my assumptions for them are correct, as they were created manually using MD4 in the Go playground.
The last commit adds two tests that currently fail; both of them are for inputs that are smaller than one chunk (but not empty), so I expect that fixing one will also fix the other. The equivalent tests for inputs that are between one and two chunks, or above two, already pass. This PR does not attempt to fix this problem, it only exposes it.

Oops, found a bug in the failing tests - they weren't actually using the hashes they should. I've updated the PR to fix that. However, those tests still fail, even with the correct hashes.

So I was looking at this the other day and some stuff didn't add up for me.  Can you detail how you found the specific hashes to check against?

The one-chunk and two-chunk hashes are from your original code (so, from the AniDB page). For most of the rest, I created zero-filled files using truncate -s <size> <filename> and then ran ed2k_hash on the resulting files.
I got that ed2k_hash program from the ed2k-hash package in Debian, version 0.4.0+ds-2, which says it's from http://ed2k-tools.sourceforge.net/ed2k_hash.shtml -- this link is also on the AniDB page, which also mentions that 0.4.0 of ed2k_hash produces the blue hash.
Since the blue and red hashes should only be different on multiples of the chunk size, and most of the tests I added are not at such multiples, I used those hashes for testing both blue and red.
This leaves only the hash for the empty input, for which the ed2k_hash tool gave an error instead of a hash; so at the time, I used the Go playground to create the hashes for myself. I don't have the original code anymore, but to recreate it, it went something like this:
blue := md4.New().Sum(nil)
fmt.Printf("blue: %x\n", blue)
h := md4.New()
h.Write(blue)
fmt.Printf("red: %x\n", h.Sum(nil))

In other words, blue is the MD4 of an empty input, and then for red, it basically appends the MD4 of nothing to an empty hashlist, and hashes that. In other words, it does what the red part of the AniDB pseudocode says, which I think I did because I assumed that 0 counts as a multiple of the chunk size (which it technically is), and that's what made sense to me at the time. Apparently I was a bit confused, since the pseudocode doesn't really allow this.

Also, I have now grabbed two other ed2k hashing tools that were mentioned on the AniDB page (ed2k by Stephane D'Alu, and jacksum), both of which it says uses the red hashes, and ran them on the same files. They both give the same checksums, which match the known red sums, and match the blue ones (from ed2k_hash) for all the non-multiple-of-chunksize files.
They also don't error out on the empty file, and for that they show the MD4-of-nothing hash (what the playground code calls "blue"), which confirms my suspicion that my test for empty input is incorrect with regards to the red hash. So I should fix that test.
By the way, while this PR doesn't try to fix the bugs, I do have code that does (though I now need to fix it for empty inputs, since I wrote it for the different red hash). I didn't include it here both because it wasn't written yet, and because the fixes wasn't the main idea in that code - it involves a significant refactor that I'm not sure you'll want. :P Thus I figured it was better left to a separate PR. Also, since making it, I realized I hadn't asked if this project is one you actually want code contributions for... so... yeah. Sorry if it isn't.

OK, I've now fixed the tests, and rebased to keep the history cleaner - I moved the empty-input test to the last commit, which now holds the three failing tests for small inputs. (I pushed the fixes as a separate commit first, in case you want to see what actually changed in the code.)

That works for me, thanks!
I'll probably see if i can fix the bad hashes on stream tomorrow, but writing tests is a PITA, so you're more than welcome to fix those, LOL.


API Dash rpm doesn't loads on Fedora 40 . Closes before it even starts

Describe the bug/problem
When i click on apidash icon ,it's loads up the screen but closes immediately in 2 seconds .
Steps to Reproduce the bug/problem

Download it on Fedora 40 ( x86 rpm package)
Install it from software app on fedora
click run

Expected behavior
Should display a working app
Device Info (The device where you encountered this issue):

OS: Fedora
Version: 40

@shivam-jainn Can you confirm if you are running it on 32 bit or 64 bit CPU?

64 bit

@shivam-jainn  Thanks for the reply, will look into it.

is it weird that this works now? but didn
On Mon, 9 Sept 2024 at 00:26, Ashita Prasad @.***>
wrote:

@shivam-jainn https://github.com/shivam-jainn Thanks for the reply,
will look into it.
—
Reply to this email directly, view it on GitHub
https://github.com/foss42/apidash/issues/463#issuecomment-2336788381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVSETACVT3STX6ERW3AGPLTZVSMW3AVCNFSM6AAAAABN3CMHTGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZWG44DQMZYGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

This is a bit weird, but thanks for confirming. Since it cannot be reproduced, for now I am closing this issue.


Endings displayed on top of several staves

It is not uncommon to have endings brackets displayed on top of several staves. A default behavior for renderers could be to display them on top of each staffGrp. However, this might not always be desirable and I don't think there is currently a way to specify explicitly on top of what staves ending brackets are expected to be displayed.

The @ending.rend attribute, available on <scoreDef>, is intended to record where the ending brackets appear.  Possible values are: top (Ending rendered only above top staff), barred (Ending rendered above staves that have bar lines drawn across them), and grouped (Endings rendered above staff groups).

Great! I think you just wrote a paragraph I can add to the guidelines. Or did I overlook it?
One question: could you clarify the meaning of barred? I am not sure I can visualise that one.

barred was added in anticipation of the case where bar lines extend across staves or staff groups that aren't part of a larger, containing staff group.  You're safe in ignoring it for now as MEI doesn't yet have a way to indicate the extent of bar lines outside the context of staff groups.  We could even decide to remove this value at some point.


mkcloud, qa_crowbarsetup: Add support for SLE12 SP1 for Cloud 6

This is based on a new want_sles12sp1 option, that (hopefully) will be
dropped once the port to SP1 is fully completed and we won't use SLE12
GA.

Why hacloud == 2 ?

usually that was intended as a boolean. this is really misused, especially if it is not documented anywhere. if i would misuse it, i would misuse it correctly like hacloud == disable_ha_testrepo

usually that was intended as a boolean. this is really misused, especially if it is not documented anywhere.

I'm not disagreeing, but this is unrelated to this PR, really :-) Feel free to submit a PR to fix this (or create a card in the backlog).

+1

+1


chore(transactional-adapter-mongodb): fix mongodb peer dependency ver…

…sion
Fixes:
npm error code ERESOLVE
npm error ERESOLVE unable to resolve dependency tree
npm error
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/mongodb
npm error<EMAIL_ADDRESS>from the root project
npm error
npm error Could not resolve dependency:
npm error peer mongodb@"> 6" from<EMAIL_ADDRESS>npm error node_modules/@nestjs-cls/transactional-adapter-mongodb
npm error   @nestjs-cls/transactional-adapter-mongodb@"*" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error

You'll need to commit the update lockfile, too, though

Ups sorry did a quick fix through github. will do it properly in a few minutes.


[WIP] Manifest download improvements

add ability to suppress data files
add ability to suppress metadata files
parallelize download

@hannes-ucsc can we talk about this?

Won't be merged - experiment.
@jessebrennan use as inspiration!


LIVE-67-68-69: Create dashboard UI

From AC in Jira:
On the “All events” page, there are two categories of events that are displayed:

My events - events that have been created by the user
All events - all events that have been created by users are displayed chronologically (latest →  oldest)
We have "Upcoming events" and "Past events" which is the same as in design but I don't know which version was finally chosen

There is also sentence in Jira that the search bar is disabled - it works now, which is not a problem of course but again, I'm not sure what idea stands behind disabling it.
Active icons on sidebar should have color #BFCCF8 but now there are white.
Active accent should be wider (4px not 2px) because now it looks like some error with render, not as active accent.
HiText should be 500 weight (it's 600).
Font-family in the majority of components is system-ui not Roc Grotesk / Roboto.
Search input and Create event button should have the same height (56px) now one is 50px and the second 48px.
What about images? Maybe we should leave some mock images and in future just replace it with real photos?
Event presenters should have font-size 18px (line-height 28px) and font-weight 500 not 400.

I am not able to check the dashboard any more because the sandbox page is dead...

TODO:

Take a look at the error mentioned by @maciejewskakatarzyna
Fix problems with font family

I think that there should be margin after the last event on the list, because now it is hidden after shadow (screen).


How to save login credentials for auto-login?

Each time I open the application, I have to manually sign in again. Is there a way to automate this process?

hey @thekenu,
There is no way to automate it (at least I don't know about such way) but the app should remember login credentials. I already forgot my password to Todoist since I'm always logged in.
Could you please give me a bit more info about your operating system and about the way you installed the app.

Hey @KryDos thanks for the quick response.
I am on Ubuntu 20.04 and I used the pre-built 1.23.0 deb package from https://github.com/KryDos/todoist-linux/releases. Would you recommend that I build from source or anything like that?

@thekenu I don't think building it from source will help in any way unfortunately.
As far as I know Electron (the base for this wrapper) store cookies and and cache data in ~/.config/Todoist folder. Could you please take a look if that folder exists and if Cookies file is there?

Yeah ~/.config/Todist/Cookies and seemed to be populated be sane data. I actually just restarted my computer (It had been left on for days) and the auto-login seems to be working fine now! Closing the ticket ...


Fix null textures

There are lots of items that  show up as null in the website, most noticeably glass panes.
I'm not really sure how to fix this

It's possible that some packs may be involved with this thanks to them changing how glass panes work in menus.
Can you please share the profiles this occurs on?

Also your selected pack would help

This occurs on profile Mango and uses pack FurfSky Reborn

@CoderBoyo can you give us a link or ign of the player with the Mango profile where this is happening?

https://sky.shiiyu.moe/stats/Islappedmykey/Mango#Inventory Look in enderchest

How was that glass obtained?

I just normally crafted it as you would craft glass panes

I just normally crafted it as you would craft glass panes

Ok i'm able to reproduce, I'll try to find a fix later :)

For people who will look at this in the future:
Item: Glass Pane (vanilla, no color)
Css class: icon-102_0 (missing styles)
Missing in items.webp

related to #564

@metalcupcake5


Zookeeper: test with latest release r3.5.2-alpha

http://zookeeper.apache.org/doc/r3.5.2-alpha/releasenotes.html

Thanks for creating this, especially since I see that you encountered reliability issue with zookeeper (3.4.9) during the benchmarks.


Add support for reporting HTTP errors

When HTTP servers fail to fulfill a request, they will often return an HTML page with the error information to the client. In situations like these, curl will simply send the content of the error page to stdout and exit with a zero exit code. While this may be fine at the command line, it can be quite problematic in scripts, since it may lead to false positives. 😱
Fortunately, we can tell curl to consider HTTP errors (4xx and 5xx) for what they are and exit with a non-zero status code by passing it the -f --fail option.
Cake.Curl should support that in the CurlSettings class.

Hello, can I work on this as my first contribution?

@deqenq Sure! Let me know if you have any questions. 🙂

PR sent

@deqenq Merged in 89e2235. Thank you for your contribution. 🎉


Error while running amaon_comments_scraper.py

@naman-gupta99 thanks for reporting going to see that now

Amazon has changed their HTML so I'm going to update it.

update! Should be good
https://github.com/philipperemy/amazon-reviews-scraper/commit/f243bfcbc92e1767348924f98daf785c1409ab33


Improve Dev WF by providing shared test infrastructure

Summary
The lack of test execution and result context combined with the inconsistency of usage across the product teams creates inefficiencies and makes it more difficult - if not impossible at times - to implement needed features as well as manage tests over time.
Some of these now long requested features include:
Automatic handling of certain scenarios.  For example, some possibilities include:

Explore local testing

Ability to skip/drop known issues so they don’t fail the PR

Auto "bucket" based on dumps
Better retries
Custom downloaders (like runtime uses).  Zero byte files etc…..as they’re looking for specific files and they’re not found
Diagnosing timeouts
Caching of test payload and results
Agentless helix job waiting.
Much improved dump management
Bubble-up test failure messages in Helix work items to AzDO
Automatic detection of changes causing test failures

In addition, there is value brought to the ecosystem overall.

Consistency across teams which reduces the duplication of tooling, improves development consistency between repos, and allows for common infra improvements across a wider swath of the ecosystem more quickly and efficiently.  (think Arcade)
Decouples Helix from the test work loads (to a degree) which allows us to manage the service better over time  (e.g. we're scared to touch it today, and this will only get worse)
Provides better insights into the test work loads - allowing for specific behavior as needed by each product team.  For example, we had to mostly hack a backchannel data stream in for DevWF.
Provides a path for consistent test execution across repos and teams.  (similar value prop as with Arcade in this regard)
Removing the current "hacks" that are in place which try and derive state.  (e.g. the back channel test result downloader)

The way that tests are being managed and executed is becoming increasingly more fractured, making it more challenging to manage the quality of our builds.  This layer on top of Helix which is knowledgeable of the work loads is sorely needed.
Primary Business Drivers

Overlap and opportunity due to VMR scenario testing requirements
Decreased testing cost over time  (execution efficiencies, reduction of dev overhead, improved automation)
Increased ability to "reason" about tests such that more targeted execution is possible.

Delivering Succes

Long asked for, value add features delivered
Cost to run the tests go down (we get even more efficient)
We're able to push updates/bug fixes to our test infra with confidence
Product team overhead cost goes down  (consistency across repos, value "for free", shared infra)

Cost
6x Dev-Months of effort.   (this is a very rough estimate, and we'll only really know once we've determined scope)
Note
We should investigate cloud test again to confirm (or not) that continuing to invest in Helix is the right things to do.
Agentless Task Tasks

Task
Cost (Weeks)
Expected Completion
Completion

Create Framework for Agentless Task execution
3

Create Agentless Task that executes helix jobs
2

Enable smart job retry for the agentless task
2

We provide shared test infrastructure. Feels like the remaining items may fit better under Helix Tech Debt.


Handle DV that is in progress

What this PR does / why we need it:
When the "not finished DataVolume" is stored in backup it should restore correctly and CDI shoud retry the DV action to finish it.
DataVolume is stored as is, without applying "PrePopulated" annotation. The PVC for this DV is stored and marked InProgress, then it is not restored (skipped).
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Currently it is not possible to skip a PVC during backup.
Release note:

Handle DV that is in progress

/lgtm
/approve


metanorma site generate shows success despite PDF generation crashing

As title. See https://github.com/metanorma/mn-samples-iso/issues/72#issuecomment-818539081 for detailed output.

@CAMOBAP can you please help fix this? Thanks

Done


[FEATURE] Improve or add new spinner/loader on loading components (not main website loader) on [branch: dev]

Detailed description
Examples:

like above, where ever spinners are bieng used, create new or add new,
do not change anything in that spinner.

Context
.
Possible implementation
No response
Additional information
No response
Code of Conduct

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

Hi @kabir0x23, I want to work on this issue.

@byte-king
this issue has been resolved, can you work on other issues?

Sure.


Need a way to get paragraph outline level

Hi! First, thank you for your great library!
Some documents miss heading styles but still keep the sectining. Moreover, styles can be named in arbitrary manner, so there can be no chance to get document outline for some particular document.
Can we please provide a way to get outline elevel of a paragraph? It looks like Word stores it using <w:outlineLvl w:val="...">.
It is mentioned once in the code, but I see no highlevel way to get it out.

@dluciv  hi ,do you have way to get this  now ?   w:outlineLvl

Haven't actually checked after the March. Will check in few days, thank you!

I switch to docx4j .  If you have any good news, pls update in there to help others . Thank you

+1 to this issue. I'd like to be able to generate structured documents with content assigned to appropriate outline levels and it doesn't appear that I can do that currently with python-docx outside of just using the header levels.
If I want to add a bulletted list and set its paragraph as outline level 3, though, it seems I can't do that currently.


Pod install issue

flutter_libphonenumber: ^2.2.1
[!] CocoaPods could not find compatible versions for pod "flutter_libphonenumber_ios":
In Podfile:
flutter_libphonenumber_ios (from .symlinks/plugins/flutter_libphonenumber_ios/ios)
Specs satisfying the flutter_libphonenumber_ios (from .symlinks/plugins/flutter_libphonenumber_ios/ios) dependency were found, but they required a higher minimum deployment target.

same issue
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.16.5, on macOS 14.2.1 23C71 darwin-arm64, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[!] Android Studio (version unknown)
    ✗ Unable to determine Android Studio version.
[!] Android Studio (version unknown)
    ✗ Unable to determine Android Studio version.
[✓] VS Code (version 1.85.1)
[✓] Connected device (4 available)
[✓] Network resources

! Doctor found issues in 2 categories.

Any update?

Upgrade platform :ios to 12.0

@nhatvu95, what do you mean by "Upgrade platform :ios to 12.0" ?

Upgrade platform :ios to 12.0

@nhatvu95, what do you mean by that ?


Tap release not working correctly

Although new releases are being tagged and PHARs published it doesn't look like the Tap release is updating this repository.
Could this be due to the published artefact being uploaded after the tag?
Any idea @pierredup?

Looking at the code for tap-release, it tries to get the artefact immediately when a tag is created. So uploading the artefact afterwards indeed won't trigger the update.

is there anyway to fix that? I doubt I can change the workflow on Box side, maybe there is a hook we could trigger for tap-release

Opened https://github.com/toolmantim/tap-release/issues/67

Sorry, I didn't notice this issue and did a PR #3 to update the version to the latest...
Nothing better than automated. Please drop the PR #3 if any plan to update after fixing this issue.

The homebrew tap should now be updated automatically


Allow to specify env files location as absolute path

Allow to specify an absolute path for the configuration files for different environments. If envFile is not found try to load it as absolute path.

What is the status on this PR? Is there something that prevents it from being merged?
@baspellis Will this allow us to put the .env-file in a custom location, and not only in project root?

@nonameolsson yes, that is exactly the reason for this pull request

@baspellis Great! Will it automatically find the file, or is it required to specify the path?

With this PR you can specify the path a an absolute path (eg "/home/user/.env.staging")

@baspellis Sounds good. But is it possible to specify a path relative to the project root?

@nonameolsson For a path relative to the project root you don't need this PR

Hi @baspellis, could we get this merged in? It would be a great addition!

@Naoto-Ida I don't have write access. @pedro is your man.

Oops, sorry for dropping the ball folks! Merged manually to fix a conflict, released in 0.8.1


Skip remaining tests when a Given or When fails

I love the semantics this library adds to my tests.
In particular, I love how this allows me to write "sequential steps", which are clearly dependent on each other.  For example:
Given("I have an apple", ...);
When("I eat the apple", ...);
Then("I feel full", ...);
And("I feel healthy", ...);

This gives me very clear "arrange", "act", and "assert" sections (Given, When, Then).
Now imagine that one of my "arrange" or "act" steps fails -- eg. I fail to "eat the apple".
Unfortunately, this results in 3 failed tests.
However, that's not quite what I expect.  When an "arrange" or "act" fails, I would expect the "assertions" to be skipped.  In other words, if a Given or When fails, all Thens should be skipped.  And / But would probably be best categorized based on what preceded them.
Does this make sense?  I half-expected this to be the existing behavior, but a quick test reveals that if any Given/When fails, the rest of the assertions are still executed (which often gives confusing results).

Hi!
Thank you for your issue. I agree that it would be a nice to have feature. However, I believe this might be a bit tricky to achieve. I will look into it when I get some time over. If you have a clear idea of how to implement it feel free to create a PR :)
Running mocha with the --bail flag will stop the test suite as soon as an assertion fails. Not really what you are after perhaps, but anyway. :)

I've made this small plug-in to address this issue: https://gist.github.com/Lautenschlager-id/b2852091e72f4ade94c6212aa510f7b9
Sadly it's a bit too high level to be implemented in the source of mocha-cakes-2 I'm afraid.


Update setupProxy.js

The destructuring import causes an error. Removing the curly brackets fixes it.
createProxyMiddleware is not a function
[0] npm ERR! code 1
[0] npm ERR! path /Users/bleb/DEV/play/TWITTER/real-time-tweet-streamer
[0] npm ERR! command failed

solution found here : https://www.reddit.com/r/reactjs/comments/jzoo3y/createproxymiddleware_is_not_a_function_how_can_i/

 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.

@brandonleboeuf Thank you for submitting this pull request. This destructurring error occurs if you are using http-proxy-middlware v0.x. What version are you using?
The official create react app documentation from Facebook destructures the import
https://create-react-app.dev/docs/proxying-api-requests-in-development/
The http proxy middleware readme also destructures the import
https://github.com/chimurai/http-proxy-middleware

This project is using "http-proxy-middleware": "^0.19.1"

I'm using "http-proxy-middleware": "^0.19.1"

@brandonleboeuf Thanks for the quick reply. You will need to use http-proxy-middleware v1.x.x as defined in the package.json of this repo in order to avoid the destructure error. However, you are welcome to use an earlier version if you prefer and change the import as you have done in your own forked repo and/or local environment


Native cache

Provide optional caching of both mock and chroot
When the cache property is set to 'true' the action may cache both the mock container image and the initialized chroot environment. The intent of the caching implementation is to be convenient by being opinionated on which of these gets cached and when.
This commit also weaves in custom image support. When caching and using a custom image, the image will only be cached if mock was installed in the custom image. If mock is preloaded, the image will not be cached, which will result in a slightly faster execution.
An improved version of #6.  More significant time savings in this one.
Closes #2

Caching is integrated, after the first mock init you will see time savings until the spec is changed
      - uses: jw3/mock-srpm@cache2
        with:
          chroot: ...
          spec: ...
          cache: true

If you want to use a prebuilt image it exists
      - uses: jw3/mock-srpm@cache2
        with:
          chroot: ...
          spec: ...
          cache: true
          image: fedora:37


Fix padding calculation

Suppose header_align = 4, header_size = 8, and elem_align = 8.  The current code will align to 4 bytes (header_align) and add 4 bytes of padding after the header (elem_align - header_align).
But the allocater may return a buffer that happens to be 8-byte aligned, in which case the end of the header is also 8-byte aligned, and the correct padding is 0 bytes, not 4.
This patch fixes this case by aligning to max(header_align, elem_align) and adding padding based on header_size mod alignment.

Merged into #2 because it would conflict otherwise.


Woo! Excellent!

Donno how to chat with you, Parth. I just wanted to say that your improvements are fantastic. The tracks in my personal collection are named properly now, by the DJ. Woo!
-Gene

Hehe! 😄 I'm glad you liked the improvements. Even I found them to be a significant upgrade. Frankly, I'm surprised that ytmdl never conducted tests to measure their accuracy.
Now that the improvements are done, let's discuss the next release. I want to improve the existing state of Phoenix 10.1 before we jump into adding new features. For example, now that we have good logs, unit tests, and decent code quality, it would be nice to concentrate on the "news" and "on this day" actions. If you have any other suggestions, I would love to hear them as well!
I've enabled a Github discussion for this repo, so it would be great to chat there. Others can chime in too. And please feel free to email me anytime you want a private conversation. You can find my email on my website: https://pncnmnp.github.io/.

Discussion moved to https://github.com/pncnmnp/phoenix10.1/discussions/22


Drop MXNet 1.5 support

Description
To facilitate code cleanup based on https://github.com/apache/incubator-mxnet/pull/16280 in light of general refactoring effort.
Following MXNet 1.6 release, CI Pipeline for "stable" needs to be re-enabled.
References

#967

We also need to update the installation instructions on the website https://github.com/dmlc/gluon-nlp/issues/984


add invoke support for any and all for regular, blocking and reply promises

add invoke support for any and all for regular, blocking and reply promises
        reactor
                .all(Duration.ofSeconds(30),
                        saveToTodoTablePromise, saveToLookupTable)
                .catchError(returnPromise::reject)
                .then( v-> returnPromise.resolve(true))
                .invoke();

3.0.8 or later.


Add FizzBuzz to Interview_questions

I want to add FizzBuzz.py under Interview_questions.
Can you please add me as the Contributor to this issue.

Ok sure

can i add??


localize css to speed up the loading of google fonts

What's the purpose of this PR
localize css to speed up the loading of google fonts

Codecov Report

Merging #3660 (360d948) into master (e1a6cf9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3660   +/-   ##
=========================================
  Coverage     50.98%   50.98%
  Complexity     2329     2329
=========================================
  Files           449      449
  Lines         14026    14026
  Branches       1426     1426
=========================================
  Hits           7151     7151
  Misses         6388     6388
  Partials        487      487

Continue to review full report at Codecov.

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


converting API name to k8s requirements

Fixes #903
Signed-off-by: jasmingacic<EMAIL_ADDRESS>

Seems like something that should be abstracted 🤔


Computed fields do not trigger digest cycle

Newly computed values are not reflected in the scope when they are updated. Try modifying the inputs on the Plunkr. The computed "fullName" is always a change behind. (I'm on Chrome 41 OSX 10.10)
http://plnkr.co/edit/SHJA4RPdn03mRbvGFwcy?p=preview
I expected this to just work, but I guess the behavior makes sense since it's hard for js-data to figure out which scopes depend on the computed value to trigger $digest cycles for. Is there a better way to do this? I was hoping I could replace some angular filters.
Btw, loving this library a lot.

Hmm... maybe js-data-angular can call rootScope.$applyAsync() after updates to computed fields?

Except $applyAsync wasn't added to Angular until 1.3.x.
The fields are re-computed in response to a $digest cycle being called on the $rootScope, but apparently the changes aren't being batched within those $digest cycles.
It seems like it could cause an infinite loop of I change it so $digest triggers computed property recomputation which triggers $digest cycles which triggers computed property recomputation which triggers etc. etc. etc.

I've been meaning to have Resources emit a "change" event when the observer code detects changes in an object.  If the $digest then causes a change in the computed property's result again, then I'd say that's an application bug, though js-data would definitely need to do something to break the loop and error. You could probably do so by checking whether after calling scope.$digest(), the dependencies of the computed values changed and keep a counter on how many times that happens? Does sound ugly though.
The change event does sound useful in general. For my use cases, I think I'd only care if computed fields changed. Will there be some sort of granularity to filter the results by? And, will it be possible to subscribe to the change event on the Resource level and as well on the resource instance level?
Also, it'd be nice if there was some config option that could enable calling $applyAsync by default on computed field changes, for Angular 1.3+ code.

Here's another problem, all the code for handling computed properties is in js-data, which is framework-agnostic and has no knowledge of Angular or $digest stuff.
This might help you:
In your plunker you have:
.controller('FooController', function ($scope, DS) {
    $scope.user = DS.inject('user', {
      id: 1,
      firstName: 'Billy',
      lastName: 'Joe'
    });
    DS.bindOne('user', 1, $scope, 'user');
  });

Try changing it to:
.controller('FooController', function ($scope, DS) {
    $scope.user = DS.inject('user', {
      id: 1,
      firstName: 'Billy',
      lastName: 'Joe'
    });
    DS.bindOne('user', 1, $scope, 'user', function () {
      $scope.user.DSCompute();
    });
  });

It's not the most obvious thing, but it works. In fact, I could probably have js-data-angular do this for you...that would solve this problem. :)

Awesome, that should work. Thanks!
I think it'd be nice to also have a way to proxy bindOne like:
.controller('FooController', function ($scope, User) {
  User.find(1)
    .then(function (user) {
      myCtrl.complexModel.setUser(user);
      user.DSBind($scope);
    });
});

The example is kind of contrived yes. I'm just trying to show that a lot of the time the user won't be bound directly on $scope, but instead as a property of a much more complex working model. I think what I'm getting at sounds like making the expr optional.
Not an immediate need for it. I can ping back when it comes up. Still in the early stages of integrating js-data into our codebase.

If I understand your complex model question correctly, then this should already work for you:
// let's say the controllerAs is "FooCtrl"
.controller('FooController', function ($scope, User) {
  User.find(1);

  // "expr" can contain "." operators
  User.bindOne(1, $scope, 'FooCtrl.complexModel.user');
});

Hmm... Yes that works. I think I mostly find it strange that User is putting a value on the $scope for me, instead of me putting it on the my model and then telling the $scope to bind it.

Not a big deal. I'm happy with closing this out since the main issue has been resolved.


cargo build -j  does not start compiling

Problem
.
Steps

go into cargo repo
cargo build -j 100000
observe as cargo neither consumes any CPU nor starts actually compiling :thinking:

Possible Solution(s)
No response
Notes
No response
Version
cargo 1.69.0-nightly (39c13e67a 2023-02-12)
release: 1.69.0-nightly
commit-hash: 39c13e67a5962466cc7253d41bc1099bbcb224c3
commit-date: 2023-02-12
host: x86_64-unknown-linux-gnu
libgit2: 1.5.0 (sys:0.16.0 vendored)
libcurl: 7.86.0-DEV (sys:0.4.59+curl-7.86.0 vendored ssl:OpenSSL/1.1.1q)
os: Manjaro 22.0.3 (Sikaris) [64-bit]

with CARGO_LOG=debug cargo
        ): {
            "clone-impls",
            "default",
            "derive",
            "full",
            "parsing",
            "printing",
            "proc-macro",
            "quote",
        },
        (
            PackageId {
                name: "sha2",
                version: "0.10.6",
                source: "registry `crates-io`",
            },
            NormalOrDev,
        ): {
            "default",
            "std",
        },
    }
[2023-02-15T21:43:20Z DEBUG cargo::util::rustc] adding rustup info to rustc fingerprint
[2023-02-15T21:43:20Z DEBUG cargo::util::rustc] reusing existing rustc info cache
[2023-02-15T21:43:20Z DEBUG cargo::util::rustc] rustc info cache hit

This is likely the jobserver filling the pipe beyond pipe-max-size (which I think is 64k on most linux systems?).


EU4ToVic2: Update port blacklist

"Hi, I just used this converter for the first time, and Vic2 was crashing while converting history until I added the following to port_blacklist.txt: 2357
Seems to work flawlessly after that. Thanks to everyone who worked on this converter!"
-https://forum.paradoxplaza.com/forum/index.php?threads/the-eu4-to-v2-converter-project.707362/page-152#post-19930314

Fixed blacklist/whitelist test mode, ran tests.  Port blacklist has been updated.


Re-try using HTMLMinifier instead of the current python script

Would be better to only use mature node libraries. IIRC HTMLMinifier did not  play well with KaTeX so that would be something to  test again and see if there is an option to  fix it.

What would be a minimal example of something with KaTeX that would need to minimized? Maybe I can play around and see if I can get a pure Node solution.

if you just do a newsite() and try to minify that, it will contain some KaTeX :) after that, assuming it worked fine, we can do a test on my site which has quite a bit more maths 😆

Awesome. Sounds like a plan :)

this seems to work fairly well, I'll test a bit more and replace the current option if tests are satisfactory.