The collections management of arangodb's web admin system does not support paginating

My Environment

ArangoDB Version:        3.7.6
Deployment Mode:       Single Server

My Arangodb database has over 1000 collections.The number of collections will be even larger in the future.The collections management of arangodb's web admin system does not support paginating.

yes, thats a valid request, however, you're not the first to whish this, hence closing as duplicate of https://github.com/arangodb/arangodb/issues/9264


Removing project

Hey @neutraltone
just removing an project which doesn't exist anymore.
Cheers!

No, it still exist. Now I can access.

No, it still exist. Now I can access.

Does look like this too me:

And then:

https://github.com/neutraltone/awesome-stock-resources/pull/214#issuecomment-697245679
wait for my pic

sorry @DashChord, I've let things slip a bit here! I'm tidying up now and will be refining things over the coming weeks.


Update how_to_order.html.erb

Context
Changes proposed in this pull request
Guidance to review

Updated guidance to reflect new policy and processes


description meta thrown off by  tag and <

https://a11y-dialog.netlify.app/further-reading/dialog-element/ starts with

The native HTML <dialog> element is notoriously inconsistent, so much so that it has long been discouraged to use it. Among other things, here are some of the issues with the HTML <dialog> element:

The description is
<meta data-rh="true" name="description" content="The native HTML ` element is notoriously inconsistent, so much so that it has long been discouraged to use it. Among other things, here are some of the issues with the HTML ` element:">

Looks like the < and > were rendered as < and > and sanitized away, and (some?)  became backticks.
Noticed because it can render funny. For example in Slack that backtick gets inline code styling:

You’re right! I wonder if there is an open issue on Docusaurus about this. Let’s see.

I think it should now be fixed, via https://github.com/KittyGiraudel/a11y-dialog/commit/401daae875a3959899c708a7c01c1db3465f229e. Can you confirm @olets?

Confirmed in the page source 👍 Still off in Slack for me, but I've seen that effect before and it isn't on you — I think their "unfurl" content is stronly cached.


dont assign font-sizes to headings

and change font-size classes to match margin/padding classes:
.font-size-small
.font-size-basic
.font-size-mid
.font-size-large

something like this

font-sizing of heading elements hardly ever corresponds to hierarchy. we should not be confusing semantics with style

yeah i like this idea

im already doing it on lanza and ketvoid and it works better


[SPARK-7886][SQL] Add built-in expressions to FunctionRegistry.

ExpressionBuilders is provided with helpers to create a function builder
for each Expression.
Built-in functions removed from SqlParser when possible. Added to
FunctionRegistry.

TO DO:

Decide between the reflection and macro implementations of the
expression builder helpers.
Fix Substring (whose constructor is not well suited for the helper).
Apply changes to Hive.

This is still a work in progress.
@rxin I would like some feedback about how to continue with the implementation. In ExpressionBuilders I added some helper methods that can convert a Expression into a function name and a function builder (i.e. (Seq[Expression]) => Expression). I added two alternative implementations, one with macros and one with reflection. How does it look?
Any other feedback will be really appreciated. Thanks.

Can one of the admins verify this patch?

I'm working on a simpler solution without macros or reflection involved.

Oh yes definitely prefer the no-macro way, since it is doable without macros. Thanks!

Ping me when you think this is ready for review. Thanks!

@smola I needed this functionality so I pulled yours in, fixed some small things, and pushed a new version here: https://github.com/apache/spark/pull/6710

@rxin Thank you! I'm closing this PR.


Nolinearsolve returns error  for documentation example in Julia 1.10,1

Kindly see the error in the julia REPL shown below.
             _
   _       _ _(_)_     |  Documentation: https://docs.julialang.org
  (_)     | (_) (_)    |
   _ _   _| |_  __ _   |  Type "?" for help, "]?" for Pkg help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 1.10.1 (2024-02-13)
 _/ |\__'_|_|_|\__'_|  |  Official https://julialang.org/ release
|__/                   |

julia> using NonlinearSolve

julia>

julia> f(u, p) = u .* u .- p
f (generic function with 1 method)

julia> u0 = [1.0, 1.0]
2-element Vector{Float64}:
 1.0
 1.0

julia> p = 2.0
2.0

julia> prob = NonlinearProblem(f, u0, p)
NonlinearProblem with uType Vector{Float64}. In-place: false
u0: 2-element Vector{Float64}:
 1.0
 1.0

julia> sol = solve(prob)
ERROR: MethodError: no method matching init(::NonlinearProblem{…})

Closest candidates are:
  init(::OptimizationProblem, ::Any, ::Any...; kwargs...)
   @ SciMLBase C:\Users\adeto\.julia\packages\SciMLBase\QqtZA\src\solve.jl:144

Stacktrace:
 [1] solve(args::NonlinearProblem{…}; kwargs::@Kwargs{})
   @ CommonSolve C:\Users\adeto\.julia\packages\CommonSolve\JfpfI\src\CommonSolve.jl:23
 [2] top-level scope
   @ REPL[6]:1
Some type information was truncated. Use `show(err)` to see complete types.

what version of NonlinearSolve are you using?

The version is 3.22. I update Julia every day but it refused to update it
to 3.73.
Regards.
On Tue, 27 Feb 2024, 12:21 am Avik Pal, @.***> wrote:

what version of NonlinearSolve are you using?
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/386#issuecomment-1965408889,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4NBY5QCYWE47VYQPAW5OTYVUDHFAVCNFSM6AAAAABD27B5FCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRVGQYDQOBYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Share ]st

This is the screenshot of the status.
On Tue, 27 Feb 2024, 8:59 am Christopher Rackauckas, <
@.***> wrote:

Share ]st
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/386#issuecomment-1965903190,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4NBY76H3SYADYDEFOHV2LYVV76JAVCNFSM6AAAAABD27B5FCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRVHEYDGMJZGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

The screenshot didn't get added

See the attached screenshot.
On Tue, 27 Feb 2024, 3:07 pm Avik Pal, @.***> wrote:

The screenshot didn't get added
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/386#issuecomment-1966508277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4NBY5CFKZMAX4KTDYGOGTYVXK7LAVCNFSM6AAAAABD27B5FCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRWGUYDQMRXG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

github doesn't forward attachments of email.

Kindly send your email so that I can send it to you directly.
On Wed, Feb 28, 2024 at 4:37 PM Avik Pal @.***> wrote:

github doesn't forward attachments of email.
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/386#issuecomment-1969125046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4NBY6VN34DWFFUH5UF55DYV46L7AVCNFSM6AAAAABD27B5FCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRZGEZDKMBUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I had this error report from someone else as well, some packages are not updated and that pulls NonlinearSolve to [8913a72c] NonlinearSolve v0.3.22, note the "v0.". The current version is v3, so remove any package that is holding NonlinearSolve back and it should be fine.

I have removed the differentialequations, mittagleffler and some Packages
whose dependencies are preventing the update. Linear solve.jl is now
updated to the latest version.
I think Julia should allow multiple versions of packages to co-exist so
that one can call the latest version.
Thanks.
On Wed, 6 Mar 2024, 6:42 pm Avik Pal, @.***> wrote:

I had this error report from someone else as well, some packages are not
updated and that pulls NonlinearSolve to [8913a72c] NonlinearSolve v0.3.22,
note the "v0.". The current version is v3, so remove any package that
is holding NonlinearSolve back and it should be fine.
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/386#issuecomment-1981307882,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4NBY2OA3BC7YQEASOFFX3YW5BOBAVCNFSM6AAAAABD27B5FCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBRGMYDOOBYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

DifferentialEquations.jl is fine with the latest NonlinearSolve. mittagleffler looks fine as well.


Adds upcloud to participants

Description
Adds UpCloud as a participant. Renders as this:

Issues Resolved
#161
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Note: This includes an SVG version of the UpCloud logo as the one included on #161 was too low resolution to use.

On review this needs a little tweak:
''Formed in 2012, UpCloud is a leading European cloud service provider with a global cloud platform spanning four continents. Offering a reliable, product portfolio available from 13 different data centres.
UpCloud's 100% SLA ensures uninterrupted services, empowering businesses worldwide to thrive."

@UP-Will Updated!


How does customize.lua work? Random questions

Disclaimer: new to lua (and programming):
Came across your video on Neovim and was wondering how does customize.lua work to quickly disable/enable plugins for testing. I know in your utilities.lua you have e.g. return require('traap.core.customize').harpoon for all the relevant plugins and that customize.lua contains a list of your plugins, but not sure how the return require(...).plugin works, e.g. whether it has to do with lua, Neovim, or both.
Also, do you see any reason to use after/ directories (e.g. ~/.config/nvim/after/plugin/telescope.lua) in runtimepath? I believe those are loaded after plugins are loaded so they should ensure nothing gets overwritten although I know you explicitly defined the order yourself. I briefly used Neovim about 2 years ago and it was popular for people to dump all their files in ~/.config/nvim/lua/personal_namespace but now it seems after/ directories are more common.
Lastly curious if utilizing lua to make code in config prettier has the potential to affect performance, e.g. in your options.lua config, you have a table of options and then at the end iterate through each to set them. Or e.g. splitting what would have been a large init.lua file into many configs, like one for each plugin which is popular.
Thanks and looking forward to more videos.

Came across your video on Neovim ...

Hi rieje - First!  Thank you for watching my video and taking time to ask me a question.

and was wondering how does customize.lua work to quickly disable/enable plugins for testing. I know in your utilities.lua you have e.g. return require('traap.core.customize').harpoon for all the relevant plugins

enabled = [ true | false ] is runtime equivalent  the function that does 'return require('traap.core.customize').harpoon.  LuaJIT (just in time compiler) does optimize.
I may update the code to something like:
local customize = require('traap.core.customzie')
which becomes an "alias" so require statement only appears once in the code.  LuaJIT still has to do work and can optimize away.   Then all enabled = function() could change to enabled = customize.harpoon, as an example.

and that customize.lua contains a list of your plugins, but not sure how the return require(...).plugin works, e.g. whether it has to do with lua, Neovim, or both.

This is lua syntax.  It is essentially returning true or false.

Also, do you see any reason to use after/ directories (e.g. ~/.config/nvim/after/plugin/telescope.lua) in runtimepath?

The Lazy package manager  loads all plugins define in the specification found in traap.core.plugins.  I tell Lazy to load files found at traap.plugins.  Lazy process these files alphabetically.    Neovim will load after/plugin/... files alphabetically  too which is okay.  The caveat to remember referencing any plugin before it is loaded will cause a failure.
I just posted a pull-request https://github.com/Traap/init.lua to ThePrimeagen to resolve the caveat I just mentioned.  If you want to use after/plugin/... just remember to use the pcall(requrie, 'harpoon') syntax.

I believe those are loaded after plugins are loaded so they should ensure nothing gets overwritten although I know you explicitly defined the order yourself.

You are correct.

I briefly used Neovim about 2 years ago ...

Welcome back.

and it was popular for people to dump all their files in ~/.config/nvim/lua/personal_namespace but now it seems after/ directories are more common.

I see both techniques done.  I see rational for using either method or both.

Lastly curious if utilizing lua to make code in config prettier has the potential to affect performance, e.g. in your options.lua config, you have a table of options and then at the end iterate through each to set them. Or e.g. splitting what would have been a large init.lua file into many configs, like one for each plugin which is popular.

I merged most plugins into traap.plugins.utilities for two reasons.

I was implementing the customize flag for each plugin.
To "show off" the vim folding sytle I use.

Thanks and looking forward to more videos.

I am working content for my next Video now.  I  will demonstrate "neovim starter kit swapping."  That is the primary reason I wanted made a pull-request for ThePrimeagen.  Prime uses packer and lsp-zero, which is cool.  I want to show between my configuration and others without "fear of destroying" your Neovim environment.


Enable OSX

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #16.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Fixes #16

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@agcopenhaver, @conda-forge/r, this is ready.

Ah, I should have tried that before. Thanks!


Show possible mistakes when graphs don't end on an output node.

When hacking on the new runner, I added a done event that gives a bit of data about the last nodes that left some of their downstream nodes with missing inputs, and my cli tool outputs that if the last node wasn't an output node. This helped me pin down bugs a few times already. Example output:
◇  done ──────────────────────────────────────────────────────╮
│                                                             │
│  Last invoked node runJavascript-2 not an output.           │
│  These last invoked nodes led to incomplete opportunities:  │
│                                                             │
│  runJavascript-2:                                           │
│   receiving nodes:                                          │
│   incomplete nodes: output-1 (text)                         │
│   unused keys: text                                         │
│                                                             │
│                                                             │
├─────────────────────────────────────────────────────────────╯

(the bug here was that the wire was result->text instead of text->text)
I'll add unused wires as well.
But mostly, this would be great to have as well in breadboard-web. Happy to walk anyone through the new bits.

Just revisiting this and wondering if the EndResult should carry some of this information? Wdyt, @dglazkov?

Yes! I'll try to get wrap up the new runner code this or early next week and have it feed the new harness API.

@seefeldb sgtm. Do you want me to sketch out the EndResult protocol based on https://github.com/breadboard-ai/breadboard/issues/154#issue-2038456518 or just wait for you to switch over?


Add ARIA attributes

@thejameskyle Re: #50, this PR adds the role=tooltip attribute to the tooltip content element, and the aria-describedby attribute to the tooltip target.
Note that these changes require Drop v1.4.2 in order to work, due to the issue described at HubSpot/drop#134.
It would be nice to have aria-hidden as well, but that's really a change that should be made to Drop, since it's not Tooltip-specific. I'll take a crack at that later.

Awesome, thank you @TrevorBurnham

Could we bump to 1.2.0 since these changes are adding more than they are fixing? #nitpick

Done and done.

:tada:  :100:

It would be nice to have aria-hidden as well

On second glance, it looks like this would be redundant with the display: none style that tooltips have while hidden, at least (according to this source). Does that make sense to you, @thejameskyle?

They sound like they know what they are talking about far more than I do so I would listen to them.


KaTeX Plugin System

Discussed in #762
I did not add anything to contribution guidelines as this is still in beta but I think it is ready to be merged.
I will also add a plugin template to the project when the API is finalized.
Please check out the PluginManager class as it contains the main logic. Then you can check out a KaTeX plugin in action: persian-katex-plugin.
Here are some things that I was not sure about or I needed your guidance to implement:

[ ] KaTeX uses closures to store its state (functions, symbols, etc). This makes the plugins changes  permanent (you cannot reset the library to default). A good approach that I can think of is to use classes to keep these data structures but this will break the main KaTeX API as users have to instantiate the KaTeX class in order to use it. So I did not touch the old system.
[ ] I could not find a clean way to expose the KaTeX flow types to plugins. Do you have any suggestions?
[ ] For plugins to dynamically inject their new css classes I could think of 2 solutions:

Migrate KaTeX styles to a css in js solution like aphrodite
Ask user to add plugins css in addition to KaTeX.css.
I used this approach as I wasn't sure if it is OK to change the old behaviour.

[ ] I think migrating KaTeX development server and github.io website to a React App like the one used in persian-katex-plugin opens up whole new possibilities e.g. we can have buttons to apply different plugins to the library and check out their effect. I'm ready to implement that if you are OK.
[ ] persian-katex-plugin adds new symbols, fonts and fontMetrics so it only uses defineSymbol and addFontMetrics. I'm not sure if we need to expose anything more than defineFunction to other plugins as I haven't used it yet.
[ ] I also added yarn.lock as there is no harm for it to be there plus it provides faster and safer installation for people who use yarn and most known libraries usually have it in addition to package-lock.json.

All the current tests are passing.
I will also add more tests for the plugin system (and squash the commits) when we finalize the API.

Thank you for the pull request! Could you separate code style fixes to another PR?

@ylemkimon I dropped the cleanup commit

@kevinbarabash

I reverted all remaining cleanups
Added defineSymbol to main katex object only in this PR
removed the yarn.lock from the PR
created a new PR for changes related to addFontMetric: katex #1269
rebased and squashed all the commits

I'm not sure if this is the PR for it, but it occurs to me that exposing defineMacro (for global macros, as an alternative to passing them in) could also be handy.

Exposing defineMacro as __defineMacro could be a separate PR.

I could not find a clean way to expose the KaTeX flow types to plugins. Do you have any suggestions?

I need to read up on this more, but it sounds like you can define an index.js.flow file.

For plugins to dynamically inject their new css classes I could think of 2 solutions:

Migrate KaTeX styles to a css in js solution like aphrodite
Ask user to add plugins css in addition to KaTeX.css.  I used this approach as I wasn't sure if it is OK to change the old behaviour.

My hope is that we can reduce the need for CSS in general in the future.  It limits our ability to support other rendering backends in the future like canvas or SVG.  Obviously with the case of adding support for additional fonts some CSS has to be defined somewhere in order to define new font faces.  I think what you did is fine.

I think migrating KaTeX development server and github.io website to a React App like the one used in persian-katex-plugin opens up whole new possibilities e.g. we can have buttons to apply different plugins to the library and check out their effect. I'm ready to implement that if you are OK.

Being able to try out different plugins would be awesome.  I'd try making that work first in the dev server environment.  One thing I'd like to investigate before introducing React for the test app is look at restructuring this repo to be a mono-repo.  I'm going to open new issue that describes the benefits of such an approach.


Staging fails when pushed from Windows

Deployment type

[ ] Cloud Foundry Application (cf push)

Expected behaviour

git clone stratos-ui repo locally on Windows 10
cd statros-ui
cf push
App starts and can be visited at https://console.bosh-lite.com

Actual behaviour
cf push fails on Windows 10 machine.
cf push succeeds if executed from within bosh-lite Vagrant box (Linux env.)

To avoid this issue, set core.autocrlf=input or false when cloning.

Thanks Irfan, that would be perfect.

Closing this issue, we've added a .gitattributes file which overrides local configuration and makes sure the checkout uses LF for all files (except binaries).


wininet/winhttp

After OS reinstall now those two libraries are not needed and are making installer freeze.
I do not get it...

Uh. Weird. Sorry for the late reply, I will look into it :) Been super busy, so might take a while...

Uh. Weird. Sorry for the late reply, I will look into it :) Been super busy, so might take a while...

No problem at all ... just wanted to let you know.
Btw. wine 4.12 breaks Fusion. Today got the update and the application behaves like you would run it on DX11. You have panels and stuff but modeling area s frozen.
Downgraded to 4.11 and everything is fine.

Good to know, thanks! Unfortunately had to temporarily switch my machine to windows so won't be able to look into this for a couple of weeks :/

Oh I am sorry for your loss :(
That's ok. I just wanted to pass the info.

Yeah, it all works flawlessly for me after I remove wininet/winhttp ¯\(ツ)/¯
Weirdly it works for me without issues with 4.12.... oh well x)


tsec is not version locked

Every yarn could potentially install a different version of tsec. This is contrary to all other dependencies, where we use versions locked in yarn.lock. In the past, when we consumed a npm module from github, we included the git sha in the dependency.

@engelsdamien would it be possible for you to publish tsec to npm? The current form causes this issue and also some headaches when not explicitly using yarn upgrade tsec --latest


error messages empty

Lots of error messages are empty.
http://i.imgur.com/EP0Wk5H.png
or when someone gives a command while not being in the voicechannel:
http://i.imgur.com/itwoDS1.png

My guess is it is related to the API version changes that Discord is making. Think the discord.py lib used API v3 and soon the only allowed version is v6.

Well, I'm wondering if thats the case because only the messages where you use markdown dont work.
The reasons can be found in the console though:
http://i.imgur.com/Nf0jCwu.png - http://i.imgur.com/nJ7E31I.png
while messages without the markdown:
http://i.imgur.com/KfpwujU.png

Seems it's easily fixed:
http://i.imgur.com/2Ze2D4f.png
I'll open a pull request.

Thanks for the PR.

@TBK0000 No problem! Thanks for accepting :)


Data Engine: Add a progress bar for blob downloading

This PR adds a progress bar for downloading blob fields in a query result, similar to the one already used for files downloading.
The number in the progress bar corresponds to the total number of non-None blob fields, so it might be bigger or smaller than number of datapoints.
LMK if you want to change it and show something else instead (n_fields * n_datapoints, or just n_datapoints).

 Add a progress bar for blob downloading


Plugin causing memory leak

http://jsfiddle.net/alchen/rjqt2x91/12/
It seems there's a leak somewhere. Toggling the list between empty and some content will cause the memory usage to grow indefinitely, at least as far as I can tell.

Came here to report a similar issue but it seems to be way more noticeable in IE than Chrome for my case. Here is as simple of a demo as I can make it.
http://jsbin.com/poseli/2/edit?js,output

I confirm, IOS webkit crashes after a few clicks more or less randomly. From the logs it seems to be touch even related. Once I removed vue-touch it's fine. I am going to investigate that.

Closing this for inactivity. If the problem persists, please open a new issue.


Add image in Readme file

Checklists
General Checks

[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[x] All CI checks pass
[x] all conflicts are resolved (if any)
[x] PR has a descriptive title
[x] PR has appropriate labels and milestones for easy identification
[x] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[x] It is in the appropriate column in the project board (if necessary)
[x] has short and clear description
[x] is linked to an issue (if it is related)
[x] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[x] the markdown source is formatted
[x] spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[x] all links and images work

Hi Neha, your img couldn't be open, you've made a mistake by writing file path


feat: add mean_success_log_probs_final to conversion logic

only merge after https://github.com/p-doom/gc-minimo/pull/44

Not needed any more due to change of logic in https://github.com/p-doom/gc-minimo/pull/44


new development commands: run-first, run-id

Hi at collectiveidea,
I'd like to propose some new commands for the delayed_job script to help in debugging:

run-first
run-last
run-id id

RAILS_ENV=development script/delayed_job run-first
This should pick the first job from the queue, work on it and then quit.
RAILS_ENV=development script/delayed_job run-first
This should pick the latest job enqueued from the queue, work on it and then quit.
RAILS_ENV=development script/delayed_job run-id id
This should pick the job with id from the queue, work on it and then quit.

Closing stale


Bad gap-filling on SWIS rooftop

lots of data is missing and we appear to be gapfilling.

i think we shouldn't gapfill any solar data across 30 minute boundaries on the FE.

@chienleng I think the FE should should none here when it's empty
I just re-ran the old APVI endpoint and it has those values again.

https://twitter.com/OpenNem/status/1592334328780161024

I'll leave this open even with that patch above. The data seems to shift from underneath us
APVI returns no interval timestamp for periods that are assumed to be 0 - the SQL out is casting those to 0 now

I'm going to open another issue with APVI data issues - I've got a monitoring script now and some sanity-checks

Quite a major one from 1 Dec 4 PM through to 3 Dec 4 AM. That's some pretty bold gap-filling, I too would prefer seeing zeroes in such cases. Could do the linear gap fill only if values are on the same day for solar, and otherwise zero.

That's some pretty bold gap-filling,

I believe that is less gap-filling and more graphing the line down to the next data point

I too would prefer seeing zeroes in such cases

100% agree. I have a changeset for this weeks release for this case

https://opennem.org.au/energy/wem/?range=7d&interval=30m
We eventually got complete data from APVI but this is more about how we handle cases where it fails

@nc9 what did you actually do to fix this? the underlying cause, if i recall correctly was twofold:

FE interpolates when it shouldn't. did we fix this?
data doesn't differentiate between 0 MW and missing data.


Odometry error is huge while rotate in place

Hi, I am trying to use Create2 as a move base to implement navigation.
while using AMCL to localize thre robot, it will fit the map when moving straightly. but when rotating, AMCL will not adjust at all.  As a result, if rotating angle is larger, say 30 degree, only rotation no translation, AMCL will never find the localization. It may be caused by inaccurate rotating angle computed by odometry. when I turn 360 degree. in rviz, it's about 420 degree.  I wonder whether I can increase the axle length value to relieve the huge error?  And I cannot find how to do it.

In my experience, the odometry from the Create platforms are not that great, especially when it comes to measuring rotation. So, I don't think what you are experiencing is unusual.
Currently, the axle length is hard-coded to a value of here to a value of 0.235m. You could try changing the value and then build libcreate and create_robot from source.
If you would like a programmatic way of changing the value, I'd be happy to review a couple pull requests. First, I would suggest adding setter methods here, and then expose a ROS param in the ROS driver here.
I'm going to close ticket in the mean time since I don't think this is a bug in this driver. Please correct me if you think I'm wrong.


Add EventGridTrigger samples for batch of events

Fixes https://github.com/Azure/azure-sdk-for-net/issues/30989

API change check
API changes are not detected in this pull request.


NTR: septin collar of invasive hyphae

A septin ~ring~ collar  in pathogenic fungi involved in  the constriction of hyphae at  the  plant plasmodesma enabling  penetrate of an adjacent cell.
PMID: 29567712
NTR: septic ring of invasive hyphae
Note that the definition of the parent term is currently too specific
https://www.ebi.ac.uk/QuickGO/term/GO:0005940
A tight ring-shaped structure that forms in the division plane at the site of cytokinesis
because these septin rings are not cytokinesis associated structures.
The parentage looks OK because it doesn't  reference cell division plane.
Note that there is a term
GO:0032161   cleavage apparatus septin structure
which represents the cleavage septins, so it seems that when this term was added the relationships were not fully resolved (AFAIK all cleavage septin structures are septin rings, but I  could be wrong?)

So is the action here to add a new term and to tweak the definition of an existing term? If so, what is the definition and placement of the new term and how should the definition be tweaked? Is there a reference for the definition tweak?

The new term definition is above:
A septic ring in pathogenic fungi involved in the constriction of hyphae at the plant plasmodesma enabling penetrate of an adjacent cell.

The new term is_a septin ring

The exisitng definition need to have the refernece to "cytokinesis" removed. I don't think this requires a definition. I think it is a mistake since we have the term
Any of a series of structures composed of septins and septin-associated proteins localized to the cleavage plane which are involved in cytokinesis. PMID:12101122 PMID:15774761 PMID:16009555

to represent the grouping term for the  septin structures that are involved in cytokinesis

Thanks. Will do this now.

It's a bit confusing because the budding yeast collar and ring are separately defined. Both are involved in cytokinesis.  The collar forms from a ring. It really seems to be a "different shaped ring", it's really just semantics. I don't think that we really need separate grouping terms for collars and rings, but separate grouping terms might be required for cytokinetic vs non-cytokinetic structures.
In fungal pathogens the septin structure may or may not be involved in cytokinsis.
Sone 'rings' are involved in narrowing hyphae. The authors call them "collars"....

see what I mena...
the collar is a 'ring'

and the ring is a ring ;)

So now I'm confused again. Is what you want exactly what you specified above?

I don't know. All I know is that the new ring/collar isn't a cleavage apparatus.
I now see that this term
https://www.ebi.ac.uk/QuickGO/term/GO:0032173
is not under cleavage apparatus, so it can go under here without causing a true path violation

So I will make the term a collar and put it under the existing one.

Yep sorry. Originally I thought collar and ring were used interchangably.
The rings will only be cytokinesis, because they only exist after the collar splits.
(In pombe  people only call the structure a septin ring - pre and post split, I have never seen the use of the collar- hence my confusion)


Textbox for sending messages loses focus as soon as cursor mouses out

The text box now loses focus and cannot be typed into anymore as soon as the user mouses out of the textbox area. I believe this is probably related to the code pushed to address #2524

This was caused by me adding support for keyboard scrolling. I've fixed this issue in this commit https://github.com/BlueBubblesApp/bluebubbles-app/commit/148e81dfa69753cde71e25c712144e2bff19887a and it will be in the next update (1.12.5)


Move assets, add to config and create svg helper (needs testing)

Haven't tested any of this stuff properly, but there's a few bits that you might like.

Closed in favour of #4.

:+1:


PointerUpEvent can contain non-zero pressure

Description
Pointer-up events are defined as the event when a pointer stops contacting the device, which intuitively should have zero pressure. However, it's possible for a ui.PointerData instance with PointerChange.up to contain non-zero pressure.
It can be reproduced in devicelab embedded_android_views_integration_test. This test simulates this real recording onto an Android device, and among the events received by PointerEventConverter such counter-intuitive instances can be found.
Related Issues

https://github.com/flutter/flutter/pull/30874 tries to remove pressure customization from pointers, but did not end up removing it from PointerUpEvent because of this.

cc @Hixie

I wonder if what's going on is that on Android, you get a single pointer up, and we decompose it into a move and an up, and we're putting the pressure into both.


'\n' is implementation dependent, ODBC driver may not resolve.

[Enter feedback here]
When installed on Linux and using pyodbc to perform BULK INSERT, "\n" is expected as "0x0A" and a truncation error occurs if the column is the same size as the capacity of  a Varchar(n) column.
UN/RSD|UNKNOWN ROADSIDE MAIL DELIVERY|UNKNOWN ROADSIDE MAIL DELIVERY
inserted into
create table XYZ.dbo.GNAF_ADDRESS_TYPE_AUT (
code varchar(8) not null,
name varchar(50) not null,
description varchar(30)
);
results in
'[42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Bulk load data conversion error (truncation) for row 36, column 3 (description). (4863) (SQLExecDirectW)'
so the value of '\n' is local to the OS. It may not be correct to assume CRLF.
(Can use dos2unix to convert text files - just an issue with the '\n' assumption).

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

ID: f5a5757e-e1f9-5064-8e10-5dbab37d2055
Version Independent ID: 9ff5c29d-79ec-1ee1-1c9d-bbcb1bfe5ef9
Content: Specify Field and Row Terminators (SQL Server) - SQL Server
Content Source: docs/relational-databases/import-export/specify-field-and-row-terminators-sql-server.md
Service: sql
Sub-service: data-movement
GitHub Login: @rwestMSFT
Microsoft Alias: randolphwest

@swellhunter -- Matthew, thank you for your thorough feedback.
@rwestMSFT -- Randolph, please look into this issue.

@swellhunter This article hasn't been updated for a while, so I'll refresh it and look into clarifying the information around \n. Thanks for your feedback.

@swellhunter We are tracking this item internally (AB#70883), so this issue is being closed. Thanks again for your feedback. When we have an update to share, we will let you know here. In the meanwhile, please feel free to leave additional feedback.


[bug] Stream::exists

Jphp: 0.7
Java: 1.7
Environment: Concurrent

Description: Index 2 not exists
Problem areas: https://github.com/jphp-compiler/jphp/blob/master/jphp-runtime/src/php/runtime/ext/core/classes/stream/Stream.java#L140

Fixed in 0.7.2.


Allow to delete current avatar of organizations

An image

Same thing has been done for user settings, if someone want to take this, claim and go ahead.

This is claimed to be implemented by merging https://github.com/gogits/gogs/pull/2766

Close as implemented.


fixed issue related to checking offset on target rather than only window...

I was having a hard time figuring out why scrollfix was not working in my app, and noticed that the offset is always pulled from $window.
This is insufficient as it will not properly work when you have an absolutely positioned container for the main viewport of an app.
I have fixed this by having the offset from the $target since it will always default to $window when a parent ui-scrollfix-target is not found.
I also have the corresponding passing tests...

UI.Utils modules was split in individuals repositories. If still valid, please consider re-submitting the issue on its dedicated repository: https://github.com/angular-ui/ui-scrollpoint
See the README for details.


Updated the instance type for the text2text FM

Description

Motivation and Context

How Has This Been Tested?

[ ] I have tested and validated these changes using one or more of the provided examples/* projects
[ ] I have executed pre-commit run -a on my pull request

@joozero Can you review this PR?


Metrics query optimizations

This PR adds unittests to assert on the number of ORM queries made in the metrics endpoints, and updates to decrease the count of queries. This builds on #3438. Since several of the changes in #3438 were addressed in #3549, this separates out the remaining changes into a new clean PR.

Thanks! I did leave some comments as there are many unrelated (cosmetic) changes that there are some conflicts with other open PRs. I don't think we have any way of verifying this is all correct without doing lots of manual tests or checking the test cases in detail. An approach we sometimes follow for refactoring is that we first create a PR with the tests and then the refactoring PR after that to prove that it didn't break anything.
Is there any possibility the testscase can be submitted in their own PR first?

Thanks! I did leave some comments as there are many unrelated (cosmetic) changes that there are some conflicts with other open PRs. I don't think we have any way of verifying this is all correct without doing lots of manual tests or checking the test cases in detail. An approach we sometimes follow for refactoring is that we first create a PR with the tests and then the refactoring PR after that to prove that it didn't break anything.
Is there any possibility the testscase can be submitted in their own PR first?

thanks, but most of that test seems to be a big piece of expected html that was actually changed by the PR? Sorry but there's too much going on in the PR. Might be that most of it is linting but it I wouldn't know until going over each and every change. Can you move the tests that you created into the same folder as the existing unit tests so we can start by looking at what tests are added?

How would you like the new tests organized? I saw the unittests directory doesn't have UI-level tests, so there wasn't
The two new tests are simply confirming that this PR doesn't change behavior. The more important one is test_queries - https://github.com/DefectDojo/django-DefectDojo/pull/3730/files#diff-81d6a68c14530c1f33a14502d72099894bf5f813509ef610344809d389919769 We can just removed the HTML assertions. It was useful during development to make sure I didn't inadvertently changed the HTML but I don't see value in it now.

thanks, but most of that test seems to be a big piece of expected html that was actually changed by the PR? Sorry but there's too much going on in the PR. Might be that most of it is linting but it I wouldn't know until going over each and every change. Can you move the tests that you created into the same folder as the existing unit tests so we can start by looking at what tests are added?

How would you like the new tests organized? I saw the unittests directory doesn't have UI-level tests, so there wasn't
The two new tests are simply confirming that this PR doesn't change behavior. The more important one is test_queries - https://github.com/DefectDojo/django-DefectDojo/pull/3730/files#diff-81d6a68c14530c1f33a14502d72099894bf5f813509ef610344809d389919769 We can just removed the HTML assertions. It was useful during development to make sure I didn't inadvertently changed the HTML but I don't see value in it now.

@valentijnscholten see #3743 for tests isolated in separate PR

@valentijnscholten see #3743 for tests isolated in separate PR

Thanks, much easier to review now.


Add ZShape object and material for optimized rendering of digits

This is PR contains changed needed for visualization of digits.
The ROOT projects has this changes as well in its builtin sources
https://github.com/root-project/root/pull/14912
All of the changes have been squashed into one commit.

@ss4774,@CirilBohak, or @osschar please check.


What do the 2 new buttons do?

I cant seem to figure out the difference between the two buttons added to the control panel. I am talking about the one with x in, and one with just a line inside. I figured out that the X button closes the tab, but what does the other do? What is the difference?
On the same note, it would be nice to have the option to close a media source from the media center menu, without closing the tab entirely. On chrome's media center, you could do that.
Here is a mockup of what i mean:

What is the difference?

The one with x in it is for closing the tab, whereas the one with a line in it "discards" the tab, which means to stop the tab from running in background.

it would be nice to have the option to close a media source from the media center menu, without closing the tab entirely

Sounds like the discard button already does the same. Is there any difference between these functionalities?

Here is a mockup of what i mean

Well, I think currently though the discard and close buttons can be re-positioned into the top right corner like what you've shown in the mockup for a more intuitive interface.

one with a line in it "discards" the tab, which means to stop the tab from running in background.

It actually discards and refreshes the tab, which is not really what i had in mind. The feature i was talking about merely removes the media source from the list, without touching the tab at all. So, if you want it back in the media list, you can refresh the tab yourself.

I see. This sounds like a nice feature to have. Though it's a bit frustrating to accommodate so many buttons inside a small popup. I will definitely figure out a better UI design.

Yeah, that does sound nice. One small problem i have with the current "discard tab" button is, lets say i browse tumblr or reddit. Often, the media centre lists reddit as a media source because i played a video, but i scrolled past this. This remains in the media centre list until i delete the tab. However, if i were to click the discard tab button, it refreshes my reddit tab, and therefore i end up losing all my progress.
Does that make sense? I think just simply removing reddit from the media centre without touching the tab would be a little bit better.

You might consider putting some options behind a (...) button or right click to reduce the number of buttons.

I might suggest to add a bit of spacing between the "Remove from List" and "Close Tab" buttons, to prevent accidentally clicking on the former instead of the latter.
Rationale: accidentally clicking on "Remove from List" may be frustrating, as in order to get back the relevant tab in the extension popup, the user has to find back the browser tab and "reactivate" it by some means (e.g. reload page, pause-play having sound enabled…)
Similarly, I suppose that accidentally clicking on the latter instead of the former may be frustrating too.


Initial Xtensa support

I'm opening this up early, to get initial feedback, but also to ensure that this PR and subsequent PRs are reviewable.
With much help from @bugadani, this PR implements the following:

Support for communicating with the Xtensa Debug Module (Xdm)
Support for powering up the xdm, and reading back the status
Support for entering and exiting debug mode
Support for halt & resume
Support for random access memory reads, write is not yet implemented
Adds try_get_xtensa_interface & has_xtensa_interface to probes, and implements it for espusbjtag
Finally, an example which can be ran on the esp32s3 to read back a timestamp register. If you have an esp32s3 board, you can run it with RUST_LOG="info" cargo run --example xtensa

Example is working as expected for me!
 INFO  tracing::span > list_cmsisdap_devices;
 INFO  tracing::span > list_stlink_devices;
 INFO  tracing::span > list_jlink_devices;
 INFO  tracing::span > list_espjtag_devices;
 INFO  tracing::span > list_wlink_devices;
 INFO  probe_rs::probe::espusbjtag::protocol > Found ESP USB JTAG adapter, base speed is 40000kHz. Available dividers: (1..255)
 INFO  probe_rs::probe::espusbjtag           > Setting scan chain to [ScanChainElement { name: Some("main"), ir_len: Some(5) }, ScanChainElement { name: Some("second"), ir_len: Some(5) }]
 INFO  probe_rs::probe::espusbjtag           > JTAG dr scan complete, found 2 TAPs. [302003429, 302003429]
 INFO  probe_rs::probe::espusbjtag           > Found 2 TAPs on reset scan
 WARN  probe_rs::probe::espusbjtag           > More than one TAP detected, defaulting to tap0
 INFO  probe_rs::probe::espusbjtag           > JtagChainItem { idcode: 302003429, irlen: 5 }
 INFO  probe_rs::probe::espusbjtag           > JtagChainItem { idcode: 302003429, irlen: 5 }
 INFO  probe_rs::probe::espusbjtag           > Setting chain params: ChainParams { irpre: 0, irpost: 5, drpre: 0, drpost: 1, irlen: 5 }
 INFO  probe_rs::architecture::xtensa::xdm   > DebugStatus { .0:<PHONE_NUMBER>, exec_done: false, exec_exception: false, exec_busy: false, exec_overrun: false, stopped: false, core_wrote_ddr: false, core_read_ddr: false, host_wrote_ddr: false, host_read_ddr: false, debug_pend_break: false, debug_pend_host: false, debug_pend_trax: false, debug_int_break: false, debug_int_host: false, debug_int_trax: false, run_stall_toggle: false, run_stall_sample: false, break_out_ack_iti: false, break_in_iti: false, dbgmod_power_on: true }
 INFO  probe_rs::architecture::xtensa::xdm   > Found Xtensa device with OCDID: 0x0B339FD2
 INFO  probe_rs::architecture::xtensa::communication_interface > Entered OCD mode
 INFO  probe_rs::architecture::xtensa::communication_interface > Left OCD mode
SYSTEM peripheral date: 02101220

Thanks for the feedback! As this is a hard work in progress, I chose not to follow some of your advice (yet). The corrections are made in https://github.com/MabezDev/probe-rs/pull/132 because I don't have write access to Mabez's fork, merging that PR will hopefully include the changes here as well.

No worries :) I just added some random thoughts while I went through it :)
I am very aware a lot of it will possibly change :)


Add Code Coverage

Would be nice to generate a codecoverage number like libraries such as https://github.com/alteryx/featuretools

Should to add to both readme and github actions that run on every PR

take

awesome! is this possible without creating an account somewhere that's not github?

I think if you want to use Codecov, you'll have to setup the repo and put the upload token into the GitHub Actions secrets.

https://docs.codecov.com/docs/quick-start

Alternatively, I can try to setup coveragepy
The benefit of Codecov is that it will host the reports, where as with coveragepy the coverage reports (HTMLs) would have to be hosted somewhere.


Added consume cpu function to Resource Consumer

Part of #11570

PTAL

LGTM

You need to add bitbucket.org/bertimus9/systemstat to the godeps

PTAL


Submit is not working in my code

When I click submit in my code it doesn't work. Do nothing. I'm just pasting the example provided in
https://github.com/gerhat/material-ui-formik-components/blob/master/example/src/SimpleFormExample.jsx

It was a problem with Yup validation. Close it.


Add ability to execute SQL via update

We have a need to execute some specific SQL statements during our internal provisioning process to help configure new RDS databases in our environment. As a result, I felt being able to run these statements via an update parameter as the database's master user would provide a solution, so this is what I came up with. Please let me know your thoughts.

Would you be able to talk a bit more about what you need done during provisioning? SQL examples would be very helpful

Of course! To clarify, it's less about what needs to happen when provisioning the RDS database, and more about provisioning our apps that use them. That is to say, we would like the broker to be able to handle the updates we need to do to the database before we can bind a new app to it, particularly when these changes aren't necessarily in the scope of the app.
In our environment many of we utilize logical replication slots for streaming changes in data to other apps. When setting up this replication, we create a separate schema, privileges, a table, and a function that generates a "heartbeat" to ensure that data is flowing. None of these statements are inherently complex, but we want them to be done by a superuser rather than the user allocated for the app that uses this database operationally (the app doesn't care about its data being replicated, nor should it even need to know).

CREATE TABLE IF NOT EXISTS replication.heartbeat (id serial primary key, pulse timestamp);
INSERT INTO replication.heartbeat (pulse) VALUES (now());
...
SELECT * FROM PG_CREATE_LOGICAL_REPLICATION_SLOT('my_app','wal2json');

I am open to suggestions, alternatives, changes here; let me know!

We are not particularly comfortable with giving users superuser access to the database, as this makes it difficult to reason about the state of the database (which we need to do for the broker to work). For instance we do not allow more than one database within an RDS instance because it complicates the bind/unbind process.
Allowing a user to run arbitrary SQL would mean that a user could get their database into a very unknown state and potentially cause dataloss.
My understanding of Postgres replication past 9.6 is pretty shaky, but based on the example you gave me I would summarise your need as:

As a user binding an app to a Postgres database
I should be able to configure a logical replication slot for an encoding
so that I can use the logical replication features of Postgres

eg
---
applications:
  - name: my-logically-replicating-app

    services:
      - name: my-pg-db
        parameters:
          create_logical_replication_slot:
            encoding: wal2json

I understand the discomfort; I'm generally with you. The thought here for me was that the developer that provisions a database owns their database, and it's ultimately their responsibility to properly wield this foot-gun.
I think the disconnect here with your summary is about separation of concerns for us. Meaning,

As a Postgres database provisioner
I should be able to configure a logical replication slot irrespective of an application using it
So that I can use the replication features of Postgres without having to make applications aware of any downstream contexts

E.g.
---
applications:
    - name: my-app-that-doesnt-know-or-care-if-its-data-is-replicated

       services:
       - name:  my-pg-db

If we're drawing a hard line here, that's fine. I can take this back to my colleagues if that's the case.

In case it wasn't clear the code snippet above is Cloud Foundry manifest syntax, where the parameters configures the service binding (cf bind-service -c '{...}')
eg
---
applications:
    - name: my-app

       services:
       - name:  my-pg-db
         parameters:
           read_only: true // this binding can only perform reads

Although now I think about it, using a service binding for this wouldn't be ideal because it would create a replication slot for each application, not each application instance. 🤔
Another person from the GOV.UK PaaS team should have a look at this issue. My gut feeling is that this isn't something we would support (running arbitrary SQL as a superuser) as it is a very sharp knife.

I think logical replication is probably (I've never worked with it) a great tool for the times you need it, but not one that many people on the platform need. (n.b. user research might show I'm wrong here).
Until we can see a more wide-spread user need for something like this, I would definitely err on the side of not introducing complexity in and around the RDS broker. Databases are a core feature of the platform which almost(?) every single tenant makes use of, and allowing tenants to get their database in to a state where they, or we, can break it doesn't seem like a very PaaS-y thing to do.

I understand the discomfort; I'm generally with you. The thought here for me was that the developer that provisions a database owns their database, and it's ultimately their responsibility to properly wield this foot-gun.

The thing is that in the case of PaaS, the developer is NOT the one who provisions the database, developer asks the Platform (GOV.UK PaaS) to provision the database for them and the platform does it in a very opinionated way.
If your needs wrt the database management deviate greatly from the database that is provisioned by PaaS, you probably should look at managing your own database (where the above statement becomes true - i.e. the developer provisions a db) and you can bind this self-managed db via User-Provided-Service.
I'm happy to jump on call to discuss further if needed. You can find me in UK Government Digital slack channel.

Thanks all, for the input. As mentioned before, I'm fine with this stance.
Would we be more open to an update parameter that creates a user with REPLICATION privileges? Or perhaps a bind parameter? This would allow apps to have the access needed to manage replication slots and put the responsibility in the developer's hands. I can get a PR up for discussion if that would help.


Native support for remote-execution / caching

I'm opening this issue as a discussion to get opinion on whether there is interest in natively supporting remote-execution / remote-caching (through an open source protocol like https://github.com/bazelbuild/remote-apis).
With Ninja, the way I've seen most projects use remote-execution is by adding a wrapper to the command line that hijacks the actual action execution and then do remote-execution. While this model works, it has been cumbersome with some problems:

A binary has to wrap all action command lines (e.g., ./<wrapper-binary> <wrapper-binary-args> -- clang++ .... This makes locally reproducing an action failure more complicated (you have to remove this wrapper binary and its arguments to run the action locally). It can also be confusing at times to see the wrapper binary in the command line, especially for new users.

Skipping intermediate action results of the graph execution becomes very tricky. E.g., when executing a graph that contains a set of compile actions that result in a link action (all of which can be run and cached on a remote compiler farm), some users (or CI systems) would want to skip the intermediate object files since they aren't read locally and would only want to download the output of the link action. This becomes complicated to do in a remote-execution system implemented with wrappers since the individual wrappers don't have sufficient knowledge of the overall buildgraph to effectively skip downloading the output of some actions.

So the question is - is there interest in natively supporting such the remote-apis protocol in N2?

Sounds cool!  I don't have a use for it myself but I'd be happy to review or advise on any appropriate changes if they aren't too invasive.
One thing to be aware of is that n2 currently wants to read the mtimes of intermediate files to determine whether they're out of date.  This might be more complex if you intend to leave the intermediate objects cached remotely.  It's all just code of course, and fixable.

We plan to implement this in the android fork of n2 as well.
We'd like to implement action sandboxing (ideally using nsjail) before remote execution, as that will allow us to easily find all dependency issues that would be a blocker for RE. And if an action worked in a sandbox, there would be no extra work for it to also work in RE.
We'd also like to try switching to file-hash-based manifests, initially just to cut down on unnecessary rebuilds locally, but eventually also to integrate with a hash-based remote fileystem. (ABFS)
Note that skipping intermediate actions may not be trivial, I don't think the bazel remote execution APIs are set up well for that, hence why bazel doesn't support it either. Though we are interested in that as well.

Thanks - its good to know that there's interest!

One thing to be aware of is that n2 currently wants to read the mtimes of intermediate files to determine whether they're out of date. This might be more complex if you intend to leave the intermediate objects cached remotely. It's all just code of course, and fixable.

This is a challenge yep. I've seen some attempts at this do a fake file on disk to satisfy the mtime check, while others maintain this information with a custom log file.

Note that skipping intermediate actions may not be trivial, I don't think the bazel remote execution APIs are set up well for that, hence why bazel doesn't support it either. Though we are interested in that as well.

I think Bazel does support Build without the bytes? https://blog.bazel.build/2023/10/06/bwob-in-bazel-7.html

In that case, you'd probably just want to add this functionality to the android fork of n2, as the multithreaded parsing change has stirred up the internal datastructures a bit.

Ah interesting.. I will checkout Android's N2 fork!


bug: aws-s3-encryption-customer-key error shows when using Cloudfront OAI

Describe the bug
When using CloudFront OAI and using s3 bucket for storing the logs, currently AWS doesn't support encryption using SSE-KMS and it only supports encryption using SSE-S3. If we use SSE-KMS key the OAI is not working as expected
To Reproduce
Steps to reproduce the behaviour:

Create a CloudFront OAI resource
Add encryption using S3 key

Expected behaviour
The error is not expected to be shown as it not supported by AWS
System Info

tfsec version: v1.21.0
terraform version: v1.2.1
OS: Linux and Mac

it seems that aws now supports encryption with AWS KMS keys: https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingKMSEncryption.html


Formatting changes in project-structure.md doc

Title
Formatting changes in project-structure.md doc
Summary
Fixing some bold text issues and a URI in project-structure.md to point to a relative in the doc repo.
Suggested Reviewers
@mairaw, @cartermp

Hi @conniey, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Thanks @conniey  This looks great. :shipit:
Merging now.


bootstrap/runtime: remove support for v2 bootstrap runtime field.

This is the first PR in a series to remove v2 API support from Envoy.
The process is as follows:

Identify uses of a hidden_envoy_deprecated_* field in source/.
Remove implementation support for occurences identified in (1).
Remove any tests in test/ covering (1). It is necessary to validate
manually that other tests exist for replacement functionality; in
some cases the deprecated test might be providing functional coverage
not applied to the deprecated feature's replacement.
Wash, rinse and repeat.

Risk level: Low
Testing: Added a unit test to cover otherwise missing behaviors in
runtime.
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>

Thanks a lot for @htuch .. let me check this

would like to work on this with you...  :) . please suggest how to proceed ?

@ankatare follow the set of steps 1-4 described in the commit message above.

/retest

@htuch Just thinking if we have any  milestone set for this activity ?

@ankatare I'd hope we could have all v2 dependencies gone by next release.

@htuch Ok. Gt it. Thanks

@htuch @tyxia just thinking if below code also need to be removed ?
https://github.com/envoyproxy/envoy/blob/main/source/common/protobuf/utility.cc#L372-L396
please suggest.

I think so, but we might want to wait until generated_api_shadow/ is gone to be sure; I'll be doing the PR for this in the next week.

@htuch @tyxia just thinking if below code also need to be removed ?
https://github.com/envoyproxy/envoy/blob/main/source/common/protobuf/utility.cc#L372-L396
please suggest.
@ankatare

Yea, this is being worked on in PR #17924, but as htuch suggested we will wait for the generated_api_shadow/ removal first.

@tyxia @htuch cool. Thanks for Confirmation 👍

Hi @tyxia cc @htuch i Could see some pending work in my local main. Please let me know if you are working on those OR i can take anyone if not.
/envoy$ grep -rnw ./source/ -e 'hidden_envoy_deprecated_.*'
./source/extensions/tracers/zipkin/span_buffer.cc:53:        "hidden_envoy_deprecated_HTTP_JSON_V1 has been deprecated. Please use a non-default "
./source/common/protobuf/utility.cc:372:      if (absl::StartsWith(field.name(), "hidden_envoy_deprecated_")) {
./source/common/protobuf/utility.cc:380:              "Illegal use of hidden_envoy_deprecated_ V2 field '", field.full_name(),

also under test/ Directory as below
envoy$ grep -rnw ./test/ -e 'hidden_envoy_deprecated_.*'
./test/extensions/tracers/zipkin/span_buffer_test.cc:469:      "hidden_envoy_deprecated_HTTP_JSON_V1 has been deprecated. Please use a non-default "
./test/proto/deprecated.proto:41:  string hidden_envoy_deprecated_is_deprecated = 2 [deprecated = true];
./test/proto/deprecated.proto:42:  string hidden_envoy_deprecated_is_deprecated_fatal = 3 [deprecated = true];
./test/proto/deprecated.proto:45:    string hidden_envoy_deprecated_inner_deprecated = 2 [deprecated = true];
./test/proto/deprecated.proto:46:    string hidden_envoy_deprecated_inner_deprecated_fatal = 3 [deprecated = true];
./test/proto/deprecated.proto:48:  InnerMessage hidden_envoy_deprecated_deprecated_message = 4 [deprecated = true];
./test/proto/deprecated.proto:51:  repeated InnerMessage hidden_envoy_deprecated_deprecated_repeated_message = 7 [deprecated = true];
./test/proto/deprecated.proto:56:    hidden_envoy_deprecated_DEPRECATED_DEFAULT = 0 [deprecated = true];
./test/proto/deprecated.proto:58:    hidden_envoy_deprecated_DEPRECATED_NOT_DEFAULT = 2 [deprecated = true];
./test/server/server_corpus/clusterfuzz-testcase-minimized-config_fuzz_test-5702999713513472:196:  hidden_envoy_deprecated_config {
./test/server/server_corpus/crash-ac725507195d840cdb90bed3079b877e6e9419e3:6:hidden_envoy_deprecated_runtime {
./test/server/server_corpus/clusterfuzz-testcase-minimized-config_fuzz_test-5186283155750912:40:        hidden_envoy_deprecated_disable_overprovisioning: true
./test/server/server_corpus/clusterfuzz-testcase-minimized-config_fuzz_test-5186283155750912:51:  hidden_envoy_deprecated_config {
./test/common/protobuf/utility_test.cc:1889:      hidden_envoy_deprecated_is_deprecated: hidden_field1
./test/common/protobuf/utility_test.cc:1900:      "Illegal use of hidden_envoy_deprecated_ V2 field "
./test/common/protobuf/utility_test.cc:1901:      "'envoy.test.deprecation_test.UpgradedBase.hidden_envoy_deprecated_is_deprecated'");

@tyxia would you please look on my above comments.

i am seeing below WI is pending.. please suggest if i can take this to mark "remove v2 support "as finished
envoy$ grep -rnw ./source/ -e 'hidden_envoy_deprecated_.*'
./source/extensions/tracers/zipkin/span_buffer.cc:53:        "hidden_envoy_deprecated_HTTP_JSON_V1 has been deprecated. Please use a non-default "

@htuch Need your inputs on this remaining WI.

@htuch @tyxia Sorry, It's My Bad :( .
i can see, now there is no pending WI for V2 support removal. Above cases are just exceptions which is required to handle the cases.

@htuch @tyxia Sorry, It's My Bad :( . i can see, now there is no pending WI for V2 support removal. Above cases are just exceptions which is required to handle the cases.

@ankatare Yea, the remaining hidden_envoy_deprecated*s are either used for exceptions to catch invalid configs or used to skip unused cases . So, in short I think there is no pending work for removing hidden_envoy_deprecated*. There are a few cleanup works though and I have a draft CL.

@tyxia Great, Thanks for update.


Strange joint limit behavior

Hello! I think I've isolated a bug in mjx. I think when using joint constraints it affects other joints in strange ways.
Here are two mujoco setups

import jax
import mujoco
from mujoco import mjx

XML = r"""
    <mujoco>
        <worldbody>
            <body name="wheel" pos="0.5 0 0.75">
                <joint name="wheel_axis" axis="0 0 1" />
                <geom type="capsule" size="0.2 0.1" />
                <inertial pos="0 0 0" mass="0.5" diaginertia="0.01 0.01 0.02" />
            </body>
            <body name="block" pos="-0.5 0 0.75">
                <joint name="other_axis" axis="0 0 1" range="-0.5 0.5" />
                <geom type="capsule" size="0.2 0.1" />
                <inertial pos="0 0 0" mass="0.5" diaginertia="0.01 0.01 0.02" />
            </body>
        </worldbody>
    </mujoco>
    """

model = mujoco.MjModel.from_xml_string(XML)
mjx_model = mjx.device_put(model)

def scanf(carry, _):
    qpos, qvel = carry

    mjx_data = mjx.make_data(mjx_model)
    mjx_data = mjx_data.replace(qvel=qvel, qpos=qpos)
    next_data = mjx.step(mjx_model, mjx_data)

    return (next_data.qpos, next_data.qvel), next_data.qpos

(final_pos, final_vel), poss = jax.lax.scan(
    scanf,
    (jax.numpy.array([0.0, 0.0]), jax.numpy.array([100.0, 0.0])),
    None,
    1024,
)

print(poss)

import jax
import mujoco
from mujoco import mjx

XML = r"""
    <mujoco>
        <worldbody>
            <body name="wheel" pos="0.5 0 0.75">
                <joint name="wheel_axis" axis="0 0 1" />
                <geom type="capsule" size="0.2 0.1" />
                <inertial pos="0 0 0" mass="0.5" diaginertia="0.01 0.01 0.02" />
            </body>
            <body name="block" pos="-0.5 0 0.75">
                <joint name="other_axis" axis="0 0 1" />
                <geom type="capsule" size="0.2 0.1" />
                <inertial pos="0 0 0" mass="0.5" diaginertia="0.01 0.01 0.02" />
            </body>
        </worldbody>
    </mujoco>
    """

model = mujoco.MjModel.from_xml_string(XML)
mjx_model = mjx.device_put(model)

def scanf(carry, _):
    qpos, qvel = carry

    mjx_data = mjx.make_data(mjx_model)
    mjx_data = mjx_data.replace(qvel=qvel, qpos=qpos)
    next_data = mjx.step(mjx_model, mjx_data)

    return (next_data.qpos, next_data.qvel), next_data.qpos

(final_pos, final_vel), poss = jax.lax.scan(
    scanf,
    (jax.numpy.array([0.0, 0.0]), jax.numpy.array([100.0, 0.0])),
    None,
    1024,
)

print(poss)

These are exactly the same, the only difference is that in the first one the second joint is limited.
When I run these though, the first joint should just spin like crazy unresisted forever in both cases right?
Instead in 1 we get:
[[ 2.0000000e-01  0.0000000e+00]
[ 3.5800007e-01  0.0000000e+00]
[ 4.8082006e-01  0.0000000e+00]
...
[-3.0173851e-17  0.0000000e+00]
[ 3.6536127e-15  0.0000000e+00]
[ 2.9679528e-17  0.0000000e+00]]
and in 2 we get:
[[2.0000000e-01 0.0000000e+00]
[4.0000001e-01 0.0000000e+00]
[6.0000002e-01 0.0000000e+00]
...
[2.0439803e+02 0.0000000e+00]
[2.0459802e+02 0.0000000e+00]
[2.0479802e+02 0.0000000e+00]]
Sorry if this is not a bug, or if this is already known!

Hi @mishmish66 , we pushed out a couple fixes relating to this bug. 843360b and dd841c9
Can you see if the state at HEAD works for you?

Hi @mishmish66 , we pushed out a couple of fixes relating to this bug. 843360b and dd841c9
Can you see if the state at HEAD works for you?

Hey! I got totally swamped with schoolwork, but now that the semester is over I can confirm that with the current version 3.1.1 on pypy this is fixed. Thank you!


Gem reports error Version 22 not supported use Version 23

@dburkes, I've been using the databasedotcom gem (1.2.3) with the databasedotcom-api-demo app and have run into an error when running the app. Since I'm logged into Salesforce and have specified my client key and secret for API access when running the app it errors out rendering the user_home view. The error is:
Version 22 was a Tech Preview and is no longer supported. Please use Version 23 instead.
file: client.rb location: ensure_expected_response line: 376

I'm not sure why this is occurring since the latest version of the databasedotcom gem uses Version 23 as a default. I didn't see anywhere in the documentation where I can change the version. Can I specify Version 23? Does this error have to do with my login perhaps?
On a side note I'll be contributing some changes to the databasedotcom-api-demo as some gems have been updated and cause the demo to break and some a camelize method doesn't exist in the project (this is a Rails method).

I've resolved my issue. Sorry about that. Shortly, I will submit a minor update where self.version is defaulted to "22.0" in authorize method rather than the latest which should be "23.0".

I've resolved my issue. Sorry about that. Shortly, I will submit a minor update where self.version is defaulted to "22.0" in client.rb rather than the latest "23.0" although not sure if that is on purpose.


Implement skipFiles

I have checked that this feature is not already implemented

[X] This feature does not exist

Use case
The debugger often goes into ruby internals and gems. But we often just want to debug our own code. skipFiles is the default vscode way of handling thins
Description
skipFiles specified as an array inside of the launch.json configurations
{
      "type": "ruby_lsp",
      "request": "attach",
      "name": "Attach to existing server",
      "skipFiles": ["/Users/me/.rbenv/**/*.rb",],
    }

Implementation
No response

Thank you for the feature suggestion! And thank you for the precise response, @marknuzz.

skipFiles is the default vscode way of handling thins

Do you have a reference for this in the docs? I can't seem to find anything in the VS Code API documentation saying that this is the recommended way of handling it.
Extension that provide debug adapters are the ones that contribute those configurations to the launch tasks, so maybe skipFiles is just a configuration that is provided by some other debug adapter?
If we can confirm that this is the recommendation by VS Code in some piece of documentation, than I agree we should add it. But if this is just a configuration field contributed by debug adapters, then we should match what the debug gem expects and just document what @marknuzz replied somewhere.


wrong last commit link in <shields.io>

please check..

Hi! Sorry, I don't understand. Could you elaborate?

last commit badge in <shield.io>, link on the back is 	https://img.shields.io/github/commits/google/skia/last.svg
But it’s should be https://img.shields.io/github/last-commit/google/skia.svg

Thanks for the report. This was fixed recently in #1297, which is pending deploy.


Incredibly low fps when looking at waterwheels

Describe the Bug
When looking at 144 waterwheels that are all connected with encased chain drives, my frames per second get reduced by about 60%
Here's a video demonstrating that
https://redstoner.me/downloads/videos/low create fps.mp4
Here's the same build in a superflat world

Reproduction Steps
Here's a schematic of the superflat build
https://redstoner.me/downloads/schematics/waterwheels2.nbt
Expected Result
I expected maybe a bit lower fps but this is insane. About one third the fps is more than I could tolerate
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Windows 10
Mod Version
0.4.0d
Minecraft Version
1.18.1
Other Mods
No response
Additional Context
This is on build 413. I tried build 445 and it was the same. That build had other issues though which I will be posting after this

flywheel is not enabled, this is likely identical on forge. use /flywheel backend instancing to enable.

fixed through discord by enabling instancing.


Fix primitive extensions on choice types

Resolves #3018

coverage: 93.859% (+0.001%) from 93.858% when pulling 9b2bb819ccdedfa0db05eacb5113a755d8b96ccd on choice-type-primitive-ext into a0258f35728e54a3a00b4d5738b867a4076b6862 on main.


not available on CS2

CS2 console log:

exec slam
[InputService] exec: couldn't exec '{*}cfg/slam.cfg', unable to read file

I forked the repository and added Counter-Strike 2 as a game.
SLAM recognized CS2 correctly then and I could "exec slam", but Source 2 doesn't have the voice_inputfromfile command which is necessary for SLAM to work.

I forked the repository and added Counter-Strike 2 as a game.
SLAM recognized CS2 correctly then and I could "exec slam", but Source 2 doesn't have the voice_inputfromfile command which is necessary for SLAM to work.

so it cant be used in cs2 right now
right?

@nikobear1 Yes, that's right. SLAM depends on that feature in the source 1 engine to work. So using SLAM in CS2 is not possible.

but bible bot is rockin it unhindered on day one :(

but bible bot is rockin it unhindered on day one :(

maybehe is useing soundpad lol,but it needs 5 $

@nikobear1 Yes, that's right. SLAM depends on that feature in the source 1 engine to work. So using SLAM in CS2 is not possible.

Has anybody been able to update SLAM yet? Any news?

The command is still not available in CS2 and I don't think Valve plans to implement it.

The command is still not available in CS2 and I don't think Valve plans to implement it.

Thats a real bummer. Is there anything that can be done? Could a program coded from scratch theoretically work like SLAM, or is it all in that one command?


Can not upgrade from ubuntu 18.04 to 20.04

Windows Build Number
windows 10 Enterprise version: Dev OS build: 21382.1
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
4.19.128
Distro Version
Ubuntu 18.04
Other Software
No response
Repro Steps
sudo apt update
sudo apt upgrade
sudo apt --purge autoremove
sudo apt install update-manager-core
sudo do-release-upgrade -d
Expected Behavior
system upgrade completed with success
Actual Behavior

Diagnostic Logs
cat /var/log/dist-upgrade/main.log
2021-05-17 11:34:03,250 INFO release-upgrader version '20.04.32' started
2021-05-17 11:34:03,253 INFO locale: 'en_US' 'UTF-8'
2021-05-17 11:34:03,302 WARNING estimate_kernel_initrd_size_in_boot() returned '0' for kernel?
2021-05-17 11:34:03,302 WARNING estimate_kernel_initrd_size_in_boot() returned '0' for initrd?
2021-05-17 11:34:03,321 DEBUG Using 'DistUpgradeViewText' view
2021-05-17 11:34:03,352 DEBUG enable dpkg --force-overwrite
2021-05-17 11:34:03,409 DEBUG creating statefile: '/var/log/dist-upgrade/apt-clone_system_state.tar.gz'
2021-05-17 11:34:04,679 DEBUG lsb-release: 'bionic'
2021-05-17 11:34:04,680 DEBUG _pythonSymlinkCheck run
2021-05-17 11:34:04,681 DEBUG openCache()
2021-05-17 11:34:04,681 DEBUG quirks: running PreCacheOpen
2021-05-17 11:34:04,681 DEBUG running Quirks.PreCacheOpen
2021-05-17 11:34:05,400 DEBUG /openCache(), new cache size 71589
2021-05-17 11:34:05,401 DEBUG need_server_mode(): can not find a desktop meta package or key deps, running in server mode
2021-05-17 11:34:05,401 DEBUG checkViewDepends()
2021-05-17 11:34:05,404 DEBUG running doUpdate() (showErrors=False)
2021-05-17 11:34:06,856 DEBUG openCache()
2021-05-17 11:34:07,626 DEBUG /openCache(), new cache size 71589
2021-05-17 11:34:07,626 DEBUG doPostInitialUpdate
2021-05-17 11:34:07,626 DEBUG quirks: running focalPostInitialUpdate
2021-05-17 11:34:07,626 DEBUG running Quirks.focalPostInitialUpdate
2021-05-17 11:36:08,256 DEBUG abort called
2021-05-17 11:36:08,259 DEBUG openCache()
2021-05-17 11:36:09,057 DEBUG /openCache(), new cache size 71589

I had similar issue, took me some time to debug it, but here is what I found out and how I resolved it in the end:
Upgrade tool uses some snap package system which is currently not supported in WSL. I disabled its usage by commenting one line in upgrade script and then it went fine. I upgraded from 20.04 --> 20.10, but I presume it would be some for any other upgrades.
===============================
Run sudo do-release-upgrade
After it fails with abort - go to /tmp, there should be some subfolder something like /tmp/ubuntu-release-upgrader-49jp2ru6/ for example - where instance of upgrade is copied. Inside you will found out file called DistUpgradeQuirks.py.
Fine lines 128, 129 and comment them:
#        if cache['snapd'].is_installed:
#            self._checkStoreConnectivity()

Then inside that folder run this command to prompt upgrade again from this edited instance of upgrade tool:
sudo ./dist-upgrade.py --frontend=DistUpgradeViewText

It should run fine now.
A bit complicated, but once the upgrades are done, there are done.

Landing zone do-dist-upgrade /dupe #6942

FWIW, this applies equally to WSL 1.


feat: added text overrides on CartItemDetail

Part of: #409
Impact: minor
Type: feature
Component
CartItemDetail now has overrides when i18next will be implemented (see #409)
Breaking changes
No breaking chances, all added fields are optional
Testing

Add a new prop quantityText to <CartItemDetail />
Add the value "test" to quantityText
Label should display "test" when test is given as a value to the prop. When the prop quantityText is not added, it should default to the values given in default props

🚀 Preview deployed
Built with commit 02017c567cdcf90d23eab9dfd864ae1ba6ab4f3f
https://deploy-preview-450--stoic-hodgkin-c0179e.netlify.com

done

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Calling wren method handle from inside a bound foreign method body

Let say that I have these two wren classes:
foreign class Item {
    construct new() { }
}

class Builder {
    foreign static create(type)
}

Somewhere in the main module I'm calling Builder.create(Item).
And let say that Builder.create is bound to this method:
static void Create(WrenVM* vm)
{
    wrenEnsureSlots(vm, 2); // There should be 2 slots (0 for caller and 1 for the argument)
    WrenHandle* classHandle = wrenGetSlotHandle(vm, 1); // Get the class type from arg 1 (Item)
    WrenHandle* constructorHandle = wrenMakeCallHandle(vm, "new()"); // Make constructor method handle

    wrenSetSlotHandle(vm, 0, classHandle); // Set class handle into slot 0 to call constructor on it
    WrenInterpretResult res = wrenCall(vm, constructorHandle); // Assume checking result
    void* data = wrenGetSlotForeign(vm, 0); // Get return in slot 0, proceed with casting/use

    // Clean up ...
}

The idea behind this is that the Builder.create function will construct the class that is passed in it's argument with a new() call, and return the new variable with some processing on the host side.
The problem is that even tho I've verified the class handle to be of type 'Item metaclass', when calling wrenCall the following line of code vm->apiStack = NULL; will remove the handle from the stack. Since wrenSetSlotHandle internally calls vm->apiStack[slot] = value; And will cause the app to crash as it's trying to call a function from a different class, in this case, the class that originally called Builder.create which happens to be called 'Game'.
Is this intended behaviour? And if so, then how can I achieve the functionality I need?
Note that if I change the create function to simple do this on wren side it works as expected (but without the host processing):
class Builder {
    static create(type) {
        return type.new()
    }
}

So it should in theory be possible to do this on the host side.

The VM is not currently reentrant, meaning a wrenCall can't happen inside a foreign binding. If you run a debug build you'll likely get an assertion about that? (always test in debug a bit for the assertions to be present atm)

@ruby0x1 well that's rather disappointing, I was hoping there was a way to do what I'm trying.
And yes of course, I'm running and testing on a debug build. But there are no asserts happening until it tries to run a method on the wrongly associated class (Game).
The weird thing is that the Game handle that it tries to run it from is grabbed from the stackTop, and I inspected it's values a bit and it turns out the Item class value is somewhere in that stack, just not where I expect it to be. Whether or not there's a reason behind it I thought it was interesting to note.

You can also split the static in 2 parts. A wren one that handle part of
the logic and one ore more private static foreign that handles the part
that needs to be done in C land.

Depending on your needs it's easy enough to fire up TWO entirely different Wren VMs... and one can call into the other - since it's not reentrancy at that point.  I've used this strategy before.


Two controllers of same type not recognised

Issue
I have two Xbox controllers both connected via an Xbox Wireless Adapter.
Only one at a time is working.
It seems to be because of this Distinct region in the GameControllerProcess.
For my case at least the ProductID is the same for controllers of the same type.

Workaround
Disable the distinct region in an IF region

I assume that Distinct region is there to handle some particular case.
Could it be optional? Or another way to handle the issue that region is meant to solve?

I merged your commit, thanks for helping out!
The distinct region may be a relict from an earlier state, but not really sure.


OIC client and coap: Make contexts re-entrant to prevent double free

Fixes gh-2054
Signed-off-by: Gabriel Schulhof<EMAIL_ADDRESS>Changes: Addressed review comments
Replaces #2080

just minor nitpicks, the patch looks good overall :-)

@barbieri Thanks for reviewing!


syntax error, unexpected '|', expecting ';' or '{'

Hi @PhiloNL,
thank you for this package,
I am facing this error when I am trying to upload to the live server on forge.

I am using Laravel 8, PHP 8.0 and spotlight 0.1.8
I have tried everything but none worked

The issue is fixed.


Problem with compiling tool from source

When run
 go get -u github.com/heroku/force
get the following exceptions

go: missing Mercurial command. See https://golang.org/s/gogetcmd
package bitbucket.org/pkg/inflect: exec: "hg": executable file not found in %PATH%

I had missing package for Mercurial. Can be found here - https://www.mercurial-scm.org/downloads
It would be nice to have some info in documentation about this pre-requisite.

That is a golang thing not specific to the CLI
On Mon, May 1, 2017 at 11:23 PM, Yury Bondarau<EMAIL_ADDRESS>wrote:

I had missing package for Mercurial. Can be found here -
https://www.mercurial-scm.org/downloads
It would be nice to have some info in documentation about this
pre-requisite.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/heroku/force/issues/408#issuecomment-298507145, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHGHmgwtavvfKOmWkOiHJ2ly23TBEhBks5r1svLgaJpZM4NNHa5
.


feat(mutator): add the optional chaining mutator

Add the "OptionalChaining" mutator:

foo?.bar -> foo.bar
foo?.[1] -> foo[1]
foo?.() -> foo()

If you don't want to use this mutator, you can exclude it with:
{
  "mutator": {
    "excludedMutations": ["OptionalChaining"]
  }
}

We have a small challenge with placing these mutants. Take this example from utils:
- findUnserializables(child)?.map((description) => {
-   description.path.unshift(index.toString());
-   return description;
-   })
- )
- .filter(notEmpty);
+const things = Object.entries(thing).flatMap(stryMutAct_9fa48("128") ? () => undefined : (stryCov_9fa48("128"), ([key, val]) => stryMutAct_9fa48("129") ? findUnserializables(val).map(description => {
+  if (stryMutAct_9fa48("130")) {
+    {}
+  } else {
+    stryCov_9fa48("130");
+    description.path.unshift(key);
+    return description;
+  }
+}) : (stryCov_9fa48("129"), findUnserializables(val)?.map(description => {
+  if (stryMutAct_9fa48("130")) {
+    {}
+  } else {
+    stryCov_9fa48("130");
+    description.path.unshift(key);
+    return description;
+  }
+})))).filter(notEmpty);

As you can see, in this case, we'll have to duplicate the call expression ?.map(), because it is part of an optional chain. Not only that, but we're also duplicating the entire mutated AST. This is a side effect that results from the somewhat naive implementation we currently have. It is not needed, since only 1 mutant can be active at any one time.
This would blow up a more serious code block. Just think about what would happen if we mutate something like this:
foo?.find(bar => {
  console.log('lots');
  console.log('of');
  console.log('code');
  return bar?.find(baz => {
    console.log('lots');
    console.log('of');
    console.log('code');
    return baz?.find(qux => {
      console.log('lots');
      console.log('of');
      console.log('code');
      return qux?.find(quux => {
         console.log('lots');
         console.log('of');
         console.log('code');
         // ...
         return;
      });
    });
  });
});

It results in 386 lines of javascript with only 20 unique mutants in it.
All this being said, I think we should move this feature to v5.1 for now.

I think that in reality, you won't often have big blocks of code inside of an anonymous function that is called using optional chaining so I don't think the increase in code will be all that important.

Yeah that's a valid point. However, this is a more fundamental oversight of the current mutant placement strategy. For example, we also want to add the method expression mutator which would run into the same problem.

Mutation placing was fixed in #2964 , so I'm proceeding with this


Security: Memory Leak in the function TEE_PopulateTransientObject

Affected components：
affected source code file: /tee/lib/libutee/tee_api_objects.c, affected functions: TEE_PopulateTransientObject and __utee_from_attr
Attack vector(s)
To exploit the vulnerability, invoke the function TEE_PopulateTransientObject and pass a large number of the parameter "attrCount"
Suggested description of the vulnerability for use in the CVE
Memory leak in TEE_PopulateTransientObject and __utee_from_attr functions in Samsung Electronics mTower v0.3.0(and earlier) allows a trusted application to trigger denial of service and information disclosure via invoking the function TEE_PopulateTransientObject with a large number of the parameter "attrCount".
Reference(s)
https://github.com/Samsung/mTower
https://github.com/Samsung/mTower/blob/18f4b592a8a973ce5972f4e2658ea0f6e3686284/tee/lib/libutee/tee_api_objects.c#L283
Additional information
The TEE_PopulateTransientObject  function takes a number "attrCount" and create an array  "ua". This value is passed by TA, and TEE_PopulateTransientObject does not check its size. Then it is passed to __utee_from_attr. The __utee_from_attr function tries to copy data from "attrs" to "ua". The problem appears in the assignments in the for loop. If the attr_count is too large, "ua" will overlap the memory region of other TAs' (tampering data such as global variables, or causing TEE crash and triggers denial of service because of illegal address dereference).
Contact
<EMAIL_ADDRESS>

The issue will be reviewed and fixed as soon as possible.


Security vulnerabilities detected in azure-spring-data-cosmos 5.5.0 dependencies

The following security vulnerabilities are reported when using azure-spring-data-cosmos:jar:5.5.0 or azure-cosmos:jar:4.49.0:
https://www.mend.io/vulnerability-database/CVE-2023-44487
https://www.mend.io/vulnerability-database/CVE-2023-4586
They are caused by transitive dependencies to io.netty:netty-codec-http:jar:4.1.97.Final and netty-handler-4.1.97.Final.jar. See dependency tree below:
[INFO] |  \- com.azure:azure-spring-data-cosmos:jar:5.5.0:compile
[INFO] |     +- org.springframework:spring-tx:jar:6.0.11:compile
[INFO] |     +- org.springframework.data:spring-data-commons:jar:3.1.3:compile
[INFO] |     +- com.azure:azure-cosmos:jar:4.49.0:compile
[INFO] |     |  +- com.azure:azure-core:jar:1.42.0:compile
[INFO] |     |  |  \- com.azure:azure-json:jar:1.1.0:compile
[INFO] |     |  +- com.azure:azure-core-http-netty:jar:1.13.6:compile
[INFO] |     |  |  +- io.netty:netty-handler:jar:4.1.97.Final:compile
[INFO] |     |  |  |  +- io.netty:netty-resolver:jar:4.1.97.Final:compile
[INFO] |     |  |  |  \- io.netty:netty-transport:jar:4.1.97.Final:compile
[INFO] |     |  |  +- io.netty:netty-handler-proxy:jar:4.1.97.Final:compile
[INFO] |     |  |  |  \- io.netty:netty-codec-socks:jar:4.1.97.Final:compile
[INFO] |     |  |  +- io.netty:netty-buffer:jar:4.1.97.Final:compile
[INFO] |     |  |  +- io.netty:netty-codec:jar:4.1.97.Final:compile
[INFO] |     |  |  +- io.netty:netty-codec-http:jar:4.1.97.Final:compile

@kushagraThapar please take a look, shall we need to add netty;4.1.100.Final dependency to fix here?

@rafaljaw - this will be fixed in this month's spring-data-cosmos SDK release. We are currently working on shipping hotfixes for this CVE in our azure core SDKs which will then transitively solve this issue for spring-data-cosmos SDK.
@Netyyyy  - We don't need to add netty dependency to spring-data-cosmos directly, as we rely on azure-core SDKs for all netty and reactor versions to avoid dependency conflicts.
cc - @alzimmermsft / @srnagar

@rafaljaw @Netyyyy, to add to what @kushagraThapar stated, please see this comment: https://github.com/Azure/azure-sdk-for-java/issues/37152#issuecomment-1759921522
Azure-core SDKs do not use Netty directly. It is utilized through Reactor Netty which has hostname verification enabled by default, therefore our SDKs are not vulnerable to the CVE against netty-handler-4.1.97.Final.jar.


[New game]: Add a ball and Paddle game

🎮 Game Request
Game logic and basic description
Add a ball and paddle game
Control a paddle to keep a ball in play and break bricks.
Point down the features
Game points

Control the paddle to get score using arrow keys
Once you lose the score resets

Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @debangi29 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4657

Hey @debangi29 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @debangi29 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hello @kunjgit
I have completed the previous issue;
Can I now start working on it?


Update Google Maps Dependency to 7.0.0-beta in Carthage

Hey there,
I just wanted to know when the lib is ready to support Google Maps 7.0.0-beta and why it only supports Google Maps 6.0.1-Beta right now?
We established a fork which links to Google Maps 7.0.0-beta and it seems to work fine.

This is blocked by https://github.com/Carthage/Carthage/issues/3289.  We have submitted a PR with a fix but it has yet to be accepted by the Carthage project. Please consider engaging on the issue or https://github.com/Carthage/Carthage/pull/3290 to encourage the maintainers to prioritize.

Yes we submitted the PR but it's up to Carthage maintainers to approve and merge it. They have not merged it or many other PRs.

With no response from the Carthage maintainers in over 2 years and the consolidation of iOS dependencies to use Swift Package Manager, closing this issue as not planned.


🐛 Bug Report: k8s pugin: Cannot read properties of null (reading 'scrollWidth')

📜 Description
When clicking on the Kubernetes tab and clicking on the detail view for a pod, the screen errors out with the above error.
👍 Expected behavior
Should show details of the pod.
👎 Actual Behavior with Screenshots
Failed with the error. Cannot read properties of null (reading 'scrollWidth')

👟 Reproduction steps

Go to the Kubernetes tab.
In the list of clusters, expand one of the clusters having pods.
In the list of pods, expand the pod details.

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

One of our components has the same issue, but others work, so it's not a global bug.

As far as my confirmation goes, over time it went away, probably due to some upgrade. I cannot replicate it now.

@jenil-harness is this still an issue for you? :pray:

Not happening for me as well. Closing the issue.


OfpActionHeader has invalid size

About
According to OpenFlow 1.3 specification, ActionHeader will be 8 byte structure.
But now implementation, Size method of ActionHeader return 4.
Current Implementation
Current Implementation of ActionHeader#Size is following bellow.
As you see now, this method return 4.
func (h OfpActionHeader) Size() int {
	return 4
}

https://github.com/Kmotiko/gofc/blob/master/ofprotocol/ofp13/ofp13_parser.go#L3777
Expected
ActionHeader must be 64-bit aligned, so ActionHeader#Size is expected to return 8.

Temporary solution
I'll fix ActionHeader#Size to following code.
func (h *OfpActionHeader) Size() int {
    return 8
}

And more, change code where using OfpActionHeader#Size method , mostly index calculation in Parse/Serialize method, to hard-coded number.
This is interim solution.
// example
h_packet := a.ActionHeader.Serialize()
copy(packet[index:], h_packet)
index += 4 // forward 4 element as size of ActionHeader, not 8.

Future work
Current solution which describe above is not so good.
In future work, I should review it once again.
May be that modification impact to interface definition of OfpMessage.
I deal it as another issue.


fix rwlock requires non const

this fixes:
/Users/aep/.cache/zig/p/122086b7931284e391223ce165a9bee94b624a7fb34531ecc95cafb22f049887d175/src/database.zig:433:18: error: expected type '*Thread.RwLock', found '*const Thread.RwLock'
        self.lock.lockShared();
        ~~~~~~~~~^~~~~~~~~~~
/Users/aep/.cache/zig/p/122086b7931284e391223ce165a9bee94b624a7fb34531ecc95cafb22f049887d175/src/database.zig:433:18: note: cast discards const qualifier
/Users/aep/zig/0.13.0/files/lib/std/Thread/RwLock.zig:47:24: note: parameter type declared here
pub fn lockShared(rwl: *RwLock) void {
                       ^~~~~~~
referenced by:
    columnFamily: /Users/aep/.cache/zig/p/122086b7931284e391223ce165a9bee94b624a7fb34531ecc95cafb22f049887d175/src/database.zig:124:38
    main: src/main.zig:30:26
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

thanks for the contribution! if you can sign your commit, well merge it


Update to work with Clearwater again

Juju provide the IP instead of name in unit-get private-adress
Enable SRV and NAPR methods for bind
Re enable pointing to cloud dns as forwarder if bind is used.

Overall this merge looks great, there is however some missing tests for validating the forwarding bits.
DOnt worry about the drone failing check above, I have some tests that need refactored as well, but before I accept this patch to the DNS Forwarding bits, i'd like to see some tests validating it works, and move that bit of logic into its own method being called from the install method.
Great start @thomnico, and thanks for the contribution! :+1:


fix(button): menuToggle

Short description of what this resolves:
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x
Fixes: #

@brandyscarney e2e test is missing though

@manucorporat Thanks, I'll add one when I review this. 👍

@manucorporat Can you test this on Windows Edge? We ran into an issue with menu toggle on edge where it was making all attributes lower case and therefore the styles weren't working, so I'm not sure if this will work on there.

An alternative to this is using menuToggle as an Input:
 /**
  * @private
  */
@Input()
set menuToggle(val: any) {
  this._attr('_mt', 'menutoggle', true);
}

And then changing _mt to a  string. I also submitted a PR to move the logic out of button and into menuToggle. This could be tweaked further, but I think we should keep the styling as a class otherwise it is too specific of a selector. See: https://github.com/driftyco/ionic/pull/9314