webkitgtk: 2.46.2 → 2.46.3

https://github.com/WebKit/WebKit/compare/webkitgtk-2.46.2...webkitgtk-2.46.3 https://webkitgtk.org/security/WSA-2024-0006.html
https://webkitgtk.org/2024/10/30/webkitgtk2.46.3-released.html
CVE-2024-44244 CVE-2024-44296
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build epiphany


[WIP] Provide a new id generation and serialization mechanism

Use three long to represent the trace segment id, and global trace id.
This is the official comments:

Generate a new id, combined by three long numbers.

The first one represents application instance id. (most likely just an integer value, would be helpful in protobuf)
The second one represents thread id. (most likely just an integer value, would be helpful in protobuf)
The third one also has two parts

a timestamp, measured in milliseconds
a seq, in current thread, between 0(included) and 9999(included)

Background story
In the prev edition, traceSegmentId or globalTrace is combined by these parts:

2 bytes, type:  T/S (1 char)
26 bytes, timestamp, e.g.<PHONE_NUMBER>046, (13 chars)
20 bytes, UUID hashcode, e.g.<PHONE_NUMBER>, (10 chars)
2 - 10 bytes, ** thread id**, most likely between 1 and 99999, (1 - 5 chars)
2 - 22 bytes, ** thread seq**,  between -2147483648 and<PHONE_NUMBER>. (1 - 11 chars)

So the ids used 52 - 80 bytes for each one.
cc @hanahmily

cc @liuchang0403

New ID since 3.2

Application instance id, Allocated by collector, @pengys5 . Most likely a short or integer is enough. Let's do the math. First we assume that the backend allocates new id with no reuse mechanism. Even we have 1,000 applications, each one of them has (avg) 50 instances, there cost 50, 000 application ids in every reboot. Reboot the cluster in every two days, (either excellent DevOps team or disaster product), the max application instance id is 18,250,000 ( Far from MAX_INT, 2,147,483,647 ).
Thread id. Allocated by JVM, in theory, it could be a very big int, but actually, if you application is high performance, most likely you use a thread pool, instead of creating thread everyone. So in this case, the id will not be too large. most likely between 1 and 99999, ( A short only )
timestamp + [0-9999] seq, definitely a long.

So in protobuf, the id is 1 short/int + 1 short + 1 long, which is 24 bytes. Much less than before.
And in cross-process propagation, mostly HTTP head, we use BASE64 for each part of id.

'#' + 4 chars for short
'$' + 8 chars for int
12 chars for long

So the id in BASE64 used 26 or 28 chars.

cc @ascrutae

Time may not be strictly monotonic, e.g. due to adjustments to correct clock drift. It is necessary to  save the latest generation time to prevent the clock from returning to the past is

ok i will deal it

@hanahmily Thank you for your suggestion. We will add time-shift-back recognition mechanism, if we found the time-shift-back happened, we do a greater range time shift, in order to make sure the repeated timestamp is harmless.
e.g. the Ops shifts back time to 5 seconds ago, the agent recognize it immediately( right? ), the agent will set the timestamp to 10 years ago, clearly no harm.
With the time going, 5 seconds passed, the agent recovery to use the real timestamp.
Does this seem good enough for you?
cc @pengys5 @ascrutae

Coverage increased (+0.1%) to 48.786% when pulling a31ee9cf14eeb95ad4171394638e7680c027e26e on feature/new-id into b5de29eb7299157e79302bd8aae45fadcb8b6fe7 on master.

Coverage increased (+0.2%) to 48.816% when pulling e88f98537c04616364816fea909d79e9792afef1 on feature/new-id into b5de29eb7299157e79302bd8aae45fadcb8b6fe7 on master.


ENH: Improve Arrays

[X] Tests added: Please use assert_type() to assert the type of any return value

Not ready for review.  Splitting #383 into dtypes, arrays and scalars.

@bashtage checking on status of this

Want to get the interval on in first.  Been busy of late but all will be revived.

I was hoping I can simply rebase, fix the CI, and compare it to the documentation&implementation to get it closer to being merged (naive me). I got the first two done but there is much more work left (the pandas implementation changed quite a bit). I will probably not continue this work as I don't use the Arrays directly and many of them are still marked as experimental.

Pretty stale.  I'll see if it can be revived.

Closing it so that it is a tiny bit easier to focus on current PRs. Feel free to re-open.


Manipulating stickers

Hi could anyone give me some help regarding to stickers.
I have a logo that can be aligned to the right or left corner with a select input. But when a try to 'remove' the sticker with clearRect the area becomes gray and I don't know how to make it transparent.
I have this code inside onDraw event:
ctx.drawImage(
  document.querySelector(`.${sticker.class}`),
  left * this.quality,
  top * this.quality,
  sticker.width * this.quality,
  sticker.height * this.quality
 );

Thank you

Hi, I hadn't thought about that, but it can be done like this (with<EMAIL_ADDRESS>https://codepen.io/zhanziyang/pen/oGVOZp?editors=1010
Note that croppa._draw() is a method to force canvas to draw the image again. But it's only available in v1.0.0+. If you are using version below v1, you can add a ref="croppa" on croppa tag and then call this.$refs.croppa._draw().
The reason we call this method is that sticker is drawn every time the image is drawn (or updated), so we need to force the image to be drawn again after anything about sticker changes.

That is exactly what I needed. Maybe you can add how to toggle stickers as example somewhere.
Thank you!

halo,请问你有没有想过这种截图功能可以像微信一样更换头像的时候图片位于下方，在他上方有一个半透明的框，这样就可以让用户第一时间知道这个是可以移动并且可以截图的，这个用在普遍的头像功能上面很普遍

halo,请问你有没有想过这种截图功能可以像微信一样更换头像的时候图片位于下方，在他上方有一个半透明的框，这样就可以让用户第一时间知道这个是可以移动并且可以截图的，这个用在普遍的头像功能上面很普遍

谢谢，这种功能的裁剪组件已经有很多了，vue-croppa只是希望提供另一种选择，不想重复造轮子。你可以搜一下，已经有很多你描述的这种开源组件了。
This conversation has nothing to do with the issue. Maybe we can open another one if you need to know more. Thanks.


How to add coordinates to new dimensions while using xarray.apply_ufunc?

Hi,
Please consider the following code:
import xarray as xr
class A:
   def __init__(self, x):
      self.x = x

da = xr.DataArray([A(5), A(9)], coords = [('a', ['a1', 'a2'])])
da2 = xr.apply_ufunc(lambda y: np.atleast_1d(y)[0].x, da, output_core_dims=[('b')], vectorize=True)

Output:
In [53]: da
Out[53]:
<xarray.DataArray (a: 2)>
array([<__main__.A instance at 0x7f0dfa812a28>,
       <__main__.A instance at 0x7f0dfa41fb00>], dtype=object)
Coordinates:
  * a        (a) |S2 'a1' 'a2'

In [57]: da2
Out[57]:
<xarray.DataArray (a: 2, b: 4)>
array([[5, 5, 5, 5],
       [9, 9, 9, 9]])
Coordinates:
  * a        (a) |S2 'a1' 'a2'
Dimensions without coordinates: b

Then if we do:
da2['b'] = ['b1', 'b2', 'b3', 'b4']

Then:
da2
Out[55]:
<xarray.DataArray (a: 2, b: 4)>
array([[5, 5, 5, 5],
       [9, 9, 9, 9]])
Coordinates:
  * a        (a) |S2 'a1' 'a2'
  * b        (b) |S2 'b1' 'b2' 'b3' 'b4'

How to get the final solution with just apply_ufunc?

As a further clarification I think the above problem can be thought of comprising of two steps:
(1) applying a function to each element of the foo's data which adds on a dimension to foo's data.
(2) name the new dimension e with coordinates e1, e2, e3.

How to get the final solution with just apply_ufunc?

You can't. If you want to insert new coordinates, you should just use xarray's normal API for that (either assignment or .assign_coords()).
At one in the development of apply_ufunc() I think I had a feature like this, but it didn't do anything beyond what you can do by assigning  the coordinate yourself, and apply_ufunc already has a lot of options.

Ohk, your code your rules!
Thanks for the response :)


Datatable plugin conflict

There is a conflict between Datatable Js plugin and this one. When i use $(something).datetimepicker(), dataTable doesn't work anymore, search function break down.
I spend times to find a solution (which was pretty easy), so here it is 👍
Just add this single line before using datetimepicker
$.fn.datetimepicker.noConflict();
Hope it was usefull

Just add this single line before using datetimepicker
$.fn.datetimepicker.noConflict();


Try to use the win.setShape(rects)

Transparency on Windows has a problem of not receiving the events if it's blurred in the foreground. On Linux we have to do polling. Both could be theoretically better if they would use setShape. We already store rectangles for both since we are tracking the webamp windows. Left clicky could then be applied on both Linux and Windows.
Unfortunately, setShape is currently experimental and doesn't work on the mac.

This API was not what I was hoping it would be. There is some tearing when we call it often with dragging. Furthermore, it's not any smoother than polling and I can't reproduce the issue on the Windows anymore. I've pushed this to a separate branch anyway in the case it would be worth it to revisit this in the future: https://github.com/durasj/webamp-desktop/tree/transparency-set-shape


rewrite relative imports

https://github.com/dagush/fMRI_Observables/blob/371b44c782295a476188c4e320041e3251379df6/fMRI_Obs/FC.py#L11
You can write this as
from . import BOLDFilters

Sorry about this, I forgot to answer. Yes, this is more or less what we ended doing... ;-)
Thanks!

You are welcome :)


n00b question regarding installing

I have Linux n00b question.
How do i install this rep on ubuntu?
I have installed nodejs and avahi, but i dont understand how i should get the rep in place and run npm update without getting a lot of errors...
thanks

Got it working. Was using wrong nodejs version


[Update Snap] XRPL

Checklist
All items in the list below needs to be satisfied.

[x] Is the summary of the change documented in this ticket?
[x] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[ ] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

Updating svg logo.

This updates the logo of the Snap to match the screenshots in the registry

There where some small code changes as well, here is the diff shared by the builder: https://github.com/Peersyst/xrpl-snap/pull/108/files


Planning for LMS support?

Is there a plan to add support for the LMS algorithm at some point in the near future now that it's available on the ACVP demo server?

Hello!
I have LMS support near complete; I am trying to verify all is working as expected using an implementation currently but if I am unable to I will have support committed to the libacvp_1_TBD_throttle branch the first half of next week.
Thanks,
Andrew

Hi Andrew,
Thanks for the update, and the hard work to get this functionality included.

Hello!
Sorry for the wait. LMS library support has been committed to the libacvp_1_TBD_throttle branch. (This will become the libacvp 2.0 release in next few months).
The registration is conditionally compiled by defining ACVPAPP_LMS_SUPPORT. the app_lms.c file contains a comment briefly describing the outputs for each test case.
Unit tests will come soon when I get them completed.
I will close this for now but please do not hesitate to reopen if you hit any issues or have any questions!
Andrew


(NOT TO BE MERGED) Introduce cancel request for client side.

PLEASE DO NOT MERGE!
This should give a first idea, how "cancel request" could be introduced into the "core" (here client side). This "idea" would make it possible, to move parts of "cancel on deregistration" into the core (if that is wanted).
Signed-off-by: Achim Kraus<EMAIL_ADDRESS>

closed with #100


Links broken on partial builds

When partially building documentation, links are only generated to the files that were edited. I'm guessing Sphinx uses clever shortcuts to reduce processing, which leads to the doctrees being read but not analysed, which hurts us. So either we:

figure out a way to stop Sphinx from taking the shortcuts, which could be annoying for huge builds and overall an unnecessary hurdle
cache source transformations as well

Yeah caching seems to be the better option. doctree-read is emitted for changed docs only. While we maybe could use doctree-resolved for both parsing and backrefs, I think the user's convenience of shorter build times with less code analysis trumps our convenience for not implementing caching.

What's more, autodoc-process-docstring isn't naturally emitted unless the object is in a document which changed. So maybe we should change our type hint finding approach to actually importing the module ourselves. That way we could also get type hints more cleverly, like through strings that resolve to names in the module.

AND that way we can get type hints for intersphinx modules as well, if they are installed. Yeah this is definitely the way to go.


createCellRendererFromComponent: importing a module containing a component with a templateURL crashes

I cannot guarantee that this is an AgGrid issue as the error messages don't provide proper stack traces, but based on my tests it surely looks like.
The newest iteration of Angular2 cell renderers requires to import modules such as CommonModule via createCellRendererFromComponent (as Angular2 no longer does so implicitly). My CellRenderer uses several custom directives from a module.
cellRenderer: this.agComponentFactory.createCellRendererFromComponent(AngularCellRenderer, this._viewContainerRef, [], [SharedModule])

If any component declared in SharedModule uses a templateURL instead of an inline template I receive the following error:
Error: Uncaught (in promise): Error: No ResourceLoader implementation has been provided. Can't read the url "../../views/activitybar.html"

with a stack trace that only include zone calls.
I have no way to test SharedModule by itself without AgGrid, but no other component/module shows that behavior (and without including the module my CellRenderer throws errors as some directive annotations are missing) so I assume something really is missing here.

Do you have a plunker you can share? Otherwise I'll take a look and see if I can reproduce it (and then fix!)
thanks

You guys seriously need a plunkr to start with :p
Anyway, I managed to reproduce it in a new plunkr.
http://plnkr.co/edit/O4cgOxirsrRU7ZGVyIDl?p=preview
This link will crash with
nhandled Promise rejection: No ResourceLoader implementation has been provided. Can't read the url "app/somedirective.template.html" ; Zone: angular ; Task: Promise.then ; Value: Error: No ResourceLoader implementation has been provided. Can't read the url "app/somedirective.template.html"(…) Error: No ResourceLoader implementation has been provided. Can't read the url "app/somedirective.template.html"

Going into app/app.component.ts and uncommenting the template/commenting the templateUrl fixes it.

Still happening with<EMAIL_ADDRESS>but it works fine in a plunker now. I'll see if I can find another cause for the issue :/

The plunker request was really just because I'm...lazy ;-)
I took a look at this and unfortunately the fix won't be trivial for us - it down to us angular creates resource loaders and resolves paths internally - as we're creating components dynamically angular is having a hard time working out where the file(s) references are, so defaults to a "no resource loader".
We will pick this up again, but given other work items I'm afraid this will probably have to be a limitation in the short term

Hi @stephanjust
Please take a look at the new 7.x beta thats available - this supports using templateURLs in both AOT and JIT mode: https://www.ag-grid.com/angular-2-beta-release/

I really should remember to check my open issues from time to time. Our project currently bundles all html files by auto inlining them, so this issue was indirectly solved by itself. Nevertheless this beta seems to be huge - not having AoT was a real bummer and ag-grid was in fact the only thing stopping us  (and it's also great in case we decide that bundling all the HTML is too much bloat and a few requests more won't hurt). Thanks for your work.


Owls 95639 - Automated domain upgrade using the conversion webhook and manual upgrade documentation

Documentation for the automated upgrade using the conversion webhook and manual upgrade using the domain upgrade tool.

Updated existing installation, auxiliary images and troubleshooting documentation.
Added new pages for the conversion webhook and domain resource upgrade.

I did what I could with the doc edits, there were many. However, there is a section missing, please see my review comments. Also, @robertpatrick had content-related comments about the Auxiliary Images document, in OWLS-95907: Operator User Guide Feedback and Updates for 4.0. Perhaps, they will be addressed with you in his review.

[ANIL] I have fixed the TOC, the section was removed as per earlier PR review comment but TOC update was missed. The edits look good to me. I'm not familiar with content-related comments about the Auxiliary Images document in OWLS-95907 and I'm not sure if those needs to be addressed in this PR or can be done in separate PR. @rjeberhard , @robertpatrick ?


Build failure: procmail

Steps To Reproduce
Steps to reproduce the behavior:

build procmail on macOS

Build log
$ nix log /nix/store/r4gpdy9wipms02bk0y2swlmgybjjy8zh-procmail-3.24.drv
warning: The interpretation of store paths arguments ending in `.drv` recently changed.   There's something fishy going on here.
You have a look and see if you detect anything uncanny:
*******************************************************
clang -O _autotst.c -o _autotst -lm
_autotst.c:1:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
main()
^
int
1 error generated.
::::
cc -O _autotst.c -o _autotst -lm
_autotst.c:1:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
main()
^
int
1 error generated.
::::
gcc -O _autotst.c -o _autotst -lm
./initmake: line 159: gcc: command not found
::::
*******************************************************
I suggest you take a look at the definition of CFLAGS* and CC
in the Makefile before you try make again.
make: *** [Makefile:129: init] Error 1

Additional context
I am guessing, based on the last-successful / first-failure info on Hydra (https://hydra.nixos.org/build/247686004), that this is probably fallout from the llvm 16 bump.
Notify maintainers
@gebner @reckenrode
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

Add a :+1: reaction to issues you find important.

Fixed by #286469


Add heuristics defender agent

Add Heuristics agent which does whack-a-mole
Make agents a package
Add base class for MalSimulatorAgents
Test scenario with heuristics agent

This was implemented in the tyr-monitor instead.


Upgrade tarpaulin to 0.22.0+

This seems to reduce the coverage by 15% for some reason. Maybe good reason. Let's see.

Replaced with grcov in #1888


docs: When using small screens "View HTML" brakes the layout

It brakes with my 1440x900 resolution (chrome, macos), on the topics searchBox and pagination
It seems that long HTML lines is the main issue

Good catch!


Add support for react 0.14-rc1 :)

Right now is forcing me to use 0.13

https://github.com/byronmwong/react-flexbox-layout/pull/8


Soft deprecates SendgridAdapter and adds SendGridAdapter

This PR aims to fix #141
This PR includes:

SendgridAdapter soft deprecation
Adds SendGridAdapter and SendGridAdapterTest

Ps: This is my first Elixir PR feel free to tell me what's wrong 🙂

@jbernardo95 Congrats on your first Elixir PR! I'm a bit busy this week, but I will take a look at this next week if I can. If I don't leave a comment feel free to ping me because sometimes these slip my mind! Thanks!

@paulcsmith No problem, I will ping you here.

@paulcsmith Ping

@jbernardo95 Thanks for the ping! After thinking about this a bit I think there may be a way to handle this that makes it really clear to people what they need to do. I think we can basically rename SendgridAdapter to SendGridAdapter and then create another module called SendgridAdapter that implements the deliver function of SendGridAdapter, but instead of sending the email it raises an error letting the user know they need to rename the adapter. Something like this
defmodule SendgridAdapter do
  @moduledoc false

  def deliver(email, config) do
    raise """
    Bamboo.SendgridAdapter has been renamed to SendGridAdapter (note the capital "G")

    Please use Bamboo.SendGridAdapter (with a capital "G") in your config. This was changed to
    correctly match how SendGrid is spelled.
    """
  end
end

I don't think this needs to be tested since it is so simple and will be removed once Bamboo hits 1.0
Does that make sense?

@paulcsmith Completely agree 👍
I renamed the old sendgrid_adapter.ex to send_grid_adapter.ex including module name and where it is used.
And added the error raiding to the old SendgridAdapter.deliver/2.

This might be a problem on Mac OS X? Its filesystem is case-aware but not case sensitive. The generated Beam files both are called the "same" thing in that scenario. If I run the tests more than once in a row I get an error:
22:15:44.150 [error] beam/beam_load.c(1278): Error loading module 'Elixir.Bamboo.SendgridAdapter':
  module name in object code is Elixir.Bamboo.SendGridAdapter

@geofflane Both my fails, now fixed.
@paulcsmith @geofflane Any idea how to fix the Error loading module 'Elixir.Bamboo.SendgridAdapter' error ? Isn't this a beam loading issue ?

@jbernardo95 I think it's because the code now has Bamboo.SendgridAdapter and Bamboo.SendGridAdapter modules that have the same names and only vary by case. I think this might be an unsupported scenario?

@geofflane Indeed, but this sounds like a basic thing, shouldn't the build system support this ?
@paulcsmith Since this a temporary thing, should we worry about this ?

It sounds like the load error is a blocker for merging this.
I think the best solution is to remove SendgridAdapter completely and I'll wait to merge this until Bamboo 1.0.
Thoughts?

I wouldn't be opposed to merging it earlier and documenting it as a breaking change in the Changelog. I think most of us expect some breaking changes pre-1.0. Especially one that's just a single character change, that's pretty easy to manage.

I removed the old SendgridAdapter module, and the error is gone.
@paulcsmith I am with @geofflane, I think it is a pretty simple change, I see no problem seeing it merged on a pre-1.0 release. Possibly 0.7 ?
What do you think ?
(I will update the changelog according to your answer)

I think it is an easy to manage change, but it's not an easily discoverable change. Meaning that if I run mix deps.update bamboo and use Sendgrid, all of a sudden Sendgrid totally breaks. This would be super strange to me. I might expect an error, but not that the module no longer exists. I don't mind adding breaking changes like this if they are easily discoverable or are much better, but since this is a small change and one that people might get with a simple mix deps.update I'd rather not break things.
If we wait for 1.0 then people will need to explicitly update their mix file to ~> 1.0. Since it's a big bump I don't feel as bad changing the behavior since you have to know about 1.0 and change it yourself. Does that make sense?

@paulcsmith Fair enough ! We should wait 🙂

Just a heads up. Been busy with some other projects, but I haven't forgotten about this. I think we're getting fairly close to 1.0. Once it does we'll get this in!

@paulcsmith Sure, no problem 😉

I rebased this and merged it here: https://github.com/thoughtbot/bamboo/commit/6b582f80781f0072bd4051084a3286991bfde2d0
Thanks @jbernardo95!

👍


fix AnimatePresence children get stuck when state changes quickly

this is a fix to the below issue by simply updating framer-motion to the latest version
https://github.com/user-attachments/assets/5c2ab8b7-f20c-4341-bec5-7cca848e8a67

Nice catch, thank you!


Adding access flags to JavaType.Class

This PR adds "kind", "access flags" and an owner class (to support the generation of inner classes) to JavaType.Class. While doing this work, I refactored JavaType.Method and JavaType.Variable to store their access flags using the same bitmap used in the Java Language Spec rather than a Set<Flag>.  This change should make Rewrite more memory efficient and reduce the size of serialized types.
These changes are a precursor for using ASM to generate bytecode from the meta-data in Rewrite's JavaTypes.
These changes introduce two breaking changes:

The JavaClass.Class.build() methods have method signature changes to add kind, flags, and owner. It is possible to add more overloaded methods to make this a non-breaking change but it felt wrong to have so many overloads.
The serialization of ASTs (and their types) will now serialize the long value rather than a Set<Flag>. Again, we could probably add a custom serializer to fix this as well, but I wanted to get input on these changes first.

This looks sensible to me 👍


perf improvements: generate_iso_surface_vertices and generate_sparse_density_map

Hello! So as apart of investigation into potentially improving perf, I've collected some stats, and I've identified two target areas that appear to occupy largest portion of meshing budget:
Reconstructed 11790 vertices (indices=64464) from 1000 particlces in 43.492334ms and pushed in 43.679657ms
reconstruct_surface: 100.00%, 43.49ms/call @ 22.99Hz
  compute minimum enclosing aabb: 0.01%, 0.01ms/call @ 22.99Hz
  neighborhood_search: 11.67%, 5.07ms/call @ 22.99Hz
    parallel_generate_cell_to_particle_map: 26.25%, 1.33ms/call @ 22.99Hz
    get_cell_neighborhoods_par: 5.06%, 0.26ms/call @ 22.99Hz
    calculate_particle_neighbors_par: 64.24%, 3.26ms/call @ 22.99Hz
  parallel_compute_particle_densities: 0.47%, 0.21ms/call @ 22.99Hz
  parallel_generate_sparse_density_map: 41.18%, 17.91ms/call @ 22.99Hz
  triangulate_density_map: 46.62%, 20.28ms/call @ 22.99Hz
    interpolate_points_to_cell_data: 91.94%, 18.64ms/call @ 22.99Hz
      generate_iso_surface_vertices: 84.61%, 15.77ms/call @ 22.99Hz
      relative_to_threshold_postprocessing: 15.36%, 2.86ms/call @ 22.99Hz
    triangulate: 8.04%, 1.63ms/call @ 22.99Hz

So for meshing every frame the 1k particles, it takes from 30-50ms; Ideally we can get this down somewhere close to 16ms, so that we could have a one-frame latency delay on generating the meshes for a realtime sim in 60fps.
As such, it looks like generate_iso_surface_vertices (15.7ms) and parallel_generate_sparse_density_map (17.9ms) are good candidates.
I don't know much about fluid simulations, so I'll defer to you on matters here, but I have done a lot of work in perf and optimization; do you think there's any place to attack here, and if so, mind giving me a pointer so I could start/take a look? :)
I'm also wondering perhaps is there any data structures we don't have to compute every frame? Perhaps the density map? Or similar to #4 we could perhaps reuse container structures to reduce allocation strain?
Thanks, and looking forward to your insights here :)

I'll come back to this in the next days, I have to get back into the code.
For parallel_generate_sparse_density_map, I think the main problem is the insertion into the hash map. I currently use DashMap for this, not sure if there is an obvious alternative. The author of DashMap is working on a lockfree rewrite that might improve performance but this does not seem to be ready soon.
Even an alternative with less parallel overhead might not be enough, due to the cash unfriendliness of this approach. If we cannot find a more or less drop-in replacement for the hash map with satisfying performance, I think the only true alternative is a completely different approach of implementing marching cubes that doesn't use a map at all.

Re density map, interesting!
I will look into seeing if reusing the hashmap can help as well (similar to vertices, indices), though its a bit more complicated since the DensityMap is an enum

I made some improvements to the current reconstruction approach that should increase performance with a lot of threads.

So I noticed the inline(never) annotation everywhere, as mentioned in the inout branch. Is the rationale there for the profiling infra to have more accurate reporting? If yes, perhaps we should cfg the inline annotations along with the profile feature?


Allow grafana to use prometheus_ip even when external_url is provided

In some network configurations it's advantageous to have grafana communicate directly with prometheus even when prometheus has been configured with web.external_url.
This PR adds a flag to allow this which defaults to preserving the existing behaivor of using the external url when one is provided.
cc: @jmcarp

Thanks @cnelson!


feat: support for enclave migration in React Native

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces support for enclave wallet migration in React Native, updates dependencies, and improves error handling. It also refines the ecosystem handling across various wallet functionalities.
Detailed summary

Added support for enclave wallet migration in React Native.
Updated dependencies, including thirdweb and @thirdweb-dev/react-native-adapter.
Changed ecosystemId to ecosystem in multiple files for consistency.
Improved error handling in getCognitoRecoveryPasswordV2.
Enhanced ConnectedButton and ConnectedModal components to accept client prop.
Added new methods for wallet migration and encryption in migration.ts.

The following files were skipped due to too many changes: pnpm-lock.yaml

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4992  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joaquim-verges and the rest of your teammates on  Graphite

Merge activity

Oct 13, 4:18 PM EDT: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..


New helpers to manipulate a Google OAuth2 token

Coverage decreased (-2.2%) to 90.381% when pulling 58b6a0ab77f8cf49614a3d1f8185ed379b28e94d on feat/google-helpers into c1003f903de7848241551631478a12938e0cec1e on master.

Coverage increased (+0.4%) to 93.037% when pulling ec179ed34dde89387fcf5b5579245430c20abb37 on feat/google-helpers into c1003f903de7848241551631478a12938e0cec1e on master.

Coverage increased (+0.4%) to 93.037% when pulling 5c7f87185d94beba20e98aa019ae5ebc83cc8121 on feat/google-helpers into c1003f903de7848241551631478a12938e0cec1e on master.

Coverage increased (+0.2%) to 92.829% when pulling 8ded92627de83b24ccca9491a88123bf215d524b on feat/google-helpers into c1003f903de7848241551631478a12938e0cec1e on master.

Coverage increased (+0.2%) to 92.829% when pulling 63be6aa5f6603343997e413bb6db8bc173f8a8cc on feat/google-helpers into c1003f903de7848241551631478a12938e0cec1e on master.

Changes Unknown when pulling add89afd83f032c1b7f451d5321a792bc35d1719 on feat/google-helpers into ** on master**.


Teams not being able to render an OAuth card

Version
4.13.0
Describe the bug
Both Web Chat and Bot Framework Emulator are able to render an OAuth card generated by the SDK, while Teams is not.
This is not limited to Java as I tested the C# SDK with the same result.
To Reproduce

Use the Teams Auth (#46) sample
Configure the Bot on Azure with an OAuth connection according to this document
Follow the instructions in the sample README file
Run the sample locally and configure the ngrok tunnel
Upload the App on Teams (Either Web Client or the Native App)
Verify how the bot succeeds on sending the welcome message but fails to render the OAuth card

Expected behavior
Teams is expected to render an OAuth card enabling the user to authenticate with the OAuth provider
Screenshots
N/A
Additional context
I put a breakpoint and captured the Login URL that is intended to be displayed on the button of the card. Opening this URL in the browser I could continue the authentication flow normally even though the card is not displayed

I also tried to create the card manually by copying this snippet so it's rendered after the welcome message without any interaction from the user, followed by a statement, and verified that its rendered on Web Chat and not on Teams

@ldardick, unfortunately, I am unable to repro the issue. Testing the sample using the 4.13.0 version of the SDK renders the OAuth prompt every time for me in Teams.
That being said, could you verify you are using an updated copy of the sample? A bug was identified in the sample just prior to the release of the SDK that was resulting in the card not displaying.

@stevkan I can confirm I'm using the latest version from master (Using the SDK 4.13.0 in Java and 4.14.1 in C#).
I tested this once again with the same result (Working in Web Chat but not in Teams).
Please let me know if you want me to provide any additional information that can be use to reproduce the issue or even if you prefer to jump into a quick call so we can review this together.

To clarify... The latest Java version is 4.14.0. What is in master is actually 4.15.0-SNAPSHOT.  Though it should be very close to what is in the 4.14 branch.
Does this happen using the 4.14.0 packages from Maven?

@tracyboehrer I tested this with the latest version from BotBuilder-Samples, which is using 4.13.0. However, I'll try once again using 4.14.0 to verify this.

My bad, I thought that BotBuilder-Samples was using 4.13.0 when it's actually using 4.14.0.. Nevertheless, I just checked that it also happens modifying the sample to use 4.13.0.

I'm taking off the customer reported tag, this was reported during internal testing by the Southworks team and doesn't need to be tracked as a customer reported issue.

@LeeParrishMSFT, could you please assign an owner for this? I'm not sure who that should be...if that is someone from Southworks or another. Thanks!

@stevkan I'll assign it to myself since I'd like to run some tests and see what I can find since you didn't see this in 4.13 and yet ldardick said they did. I remember verifying the Teams Auth sample in 4.13 and don't remember seeing any problems, so it has me curious.

@LeeParrishMSFT, if I can be of help, let me know!

I just verified that it works with 4.14 and 4.13 as expected within Teams, the Auth card is shown as expected. My thought is that the Readme.md file is not explicit enough about requiring the step to add authentication to the bot here: Add authentication to a bot and as such if this step isn't completed the sample doesn't work. This step is just show in the Further Reading section and not called out that it must be performed. I'll make a modification to the Readme.md file in the Samples repo to make this more explicit. Tracking this through this issue in the samples repo: Java Sample 46.Teams-Auth documentation missing required step


Issue #186: Do not accept locale environment variables passed over SSH

Not sure where you'd prefer having this?

Looks good to me!


Is it the intended behaviour for ErrorCode to be set to null when WithMessage is used?

No that shouldn't happen. Looking at the history, there was a pull request that made a change to the PropertyValidator.cs file, that always sets ErrorCode to the value of errorSource.ResourceName (which can be null if not using a resource). This is wrong, and I missed that when I reviewed the pull request - I'll change it back


Missing comma is readme example

Sorry for the silly issue, but I thought it would be best to let you know:
use({
  'nvim-neotest/neotest',
  requires = {
    ...,
    'olimorris/neotest-rspec',
  } -- <== missing a comma here :)
  config = function()
    require('neotest').setup({
      ...,
      adapters = {
        require('neotest-rspec'),
      }
    })
  end
})

Great spot!!! Thank you. Guaranteed everyone who has downloaded the plugin got hit with that error 😄 .
Thanks for your comments as well. Feel free to open up any other issues to discuss them. I've haven't looked at the output from the tests yet so will try and address tomorrow.
Also...still think there are likely to be weird RSpec use cases so if any tests aren't running please let me know.


[Extensions] Introduce Identity Plugin to Core

Description
Takes over for https://github.com/opensearch-project/OpenSearch/pull/5925 since Peter is out.
Per the origing PR:
Adding a new module and plugin interface that provides identity and access control inside of OpenSearch. This is the founding building block, see more high level thoughts here on our recent blog.
The new extension point, IdentityPlugin, is added with IdentityService handling that plugin interface. IdentitySevice authenticates users and enables access control systems. Adding HTTP basic authentication in the RestController the default NoopIdentityPlugin changes no behavior.
When the identity feature flag is enabled the identity-shiro module is loaded into the IdentityService. This implementations includes an admin user backed through the Apache Shiro system. By partitioning these features through a plugin interface it isolate the dependencies if we ever switch out the underlying library.
Project tracking addition features and functionality: https://github.com/orgs/opensearch-project/projects/89?pane=info
Issues Resolved:
#5880
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

Codecov Report

Merging #7246 (af4f452) into main (9f6c067) will decrease coverage by 0.56%.
The diff coverage is 51.41%.

:exclamation: Current head af4f452 differs from pull request most recent head 9a0ad21. Consider uploading reports for the commit 9a0ad21 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #7246      +/-   ##
============================================
- Coverage     71.18%   70.62%   -0.56%
+ Complexity    59887    59533     -354
============================================
  Files          4859     4874      +15
  Lines        285505   285678     +173
  Branches      41143    41160      +17
============================================
- Hits         203223   201764    -1459
- Misses        65924    67291    +1367
- Partials      16358    16623     +265

Impacted Files
Coverage Δ

...opensearch/identity/shiro/ShiroIdentityPlugin.java
0.00% <0.00%> (ø)

...pensearch/identity/shiro/ShiroSecurityManager.java
0.00% <0.00%> (ø)

...identity/shiro/UnsupportedAuthenticationToken.java
0.00% <0.00%> (ø)

...pensearch/common/settings/FeatureFlagSettings.java
50.00% <ø> (ø)

...opensearch/identity/tokens/RestTokenExtractor.java
0.00% <0.00%> (ø)

.../opensearch/test/OpenSearchSingleNodeTestCase.java
90.84% <0.00%> (ø)

.../main/java/org/opensearch/rest/RestController.java
76.72% <5.26%> (-5.91%)
:arrow_down:

...java/org/opensearch/identity/noop/NoopSubject.java
33.33% <33.33%> (ø)

...va/org/opensearch/identity/shiro/ShiroSubject.java
44.44% <44.44%> (ø)

...n/java/org/opensearch/identity/NamedPrincipal.java
50.00% <50.00%> (ø)

... and 12 more

... and 486 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Hi @dblock, this should be ready for a further review. I am hoping to get this merged as soon as possible to unblock further work in core. Let me know what else you would like done. I addressed the comments left on the previous PR other than the addition of a @NotNull annotation to the interface. Adding that annotation was causing the connection to hang and failing the BasicAuth tests.

Hi @dblock, @saratvemulapalli, @reta, and @andrross, I know you are all busy but if any of you had the availability today to provide some reviews so we could move forward with some of the Security support for Extensions that would be greatly appreciated. If you don't have time today, I totally understand but figured I would tag you as it looked like you had all interacted with Peter's PR previously. Thank you!

@scrawfor99 I like it, a few minor comments but LGTM otherwise! Apologies for a delay in review

@scrawfor99 I like it, a few minor comments but LGTM otherwise! Apologies for a delay in review

Hi @reta, don't apologize! I appreciate all your comments and will go ahead and address them now. Apologies about pinging you only for the tests to now fail on a text change but such is life... I will address your comments and try to correct the test issue now. Thank you again :)

@scrawfor99 What do you think about creating a PR that only introduces the next extension point IdentityPlugin with a companion PR that shows how it will be implemented in the Security plugin for extensions use-cases? I put a PR together against my own fork with only IdentityPlugin related changes - no identity-shiro module: https://github.com/cwperks/OpenSearch/pull/56

Hi @cwperks, this PR is picking up where Peter's PR left off a couple weeks ago. This likely would have already been merged had Peter been the one driving this instead of myself. I am doing my best to take over where he left off however and would prefer to not scrap it for an alternative unless requested by the repository maintainers. That being said, I am happy to review your demo and perhaps you would be interested in opening your own PR against this branch or against core as a follow-up.

Thanks for jumping on this @scrawfor99 I'll take a look!

Hi @peternied,
Thank you for offering to take a look.
Here are the differences from your original PR so you can be aware:

Updated to match core main.
Added noop IdentityPlugin
Removed the idea of a noop principal and just made this a static case of the NamedPrincipal
Added documentation saying that getPrincipal should never be null
Fixed old tests that had broken from changes
Removed the IdentitySmokeTest class and related tests. They were causing the cluster to hang on the basic auth tests. I took them out and added the configuration setup into the regular SmokeTest location under /http/ and that works now.
Renamed a couple things per other's requests and added a couple error messages.

@scrawfor99 By introducing a native module and enforcing that only 1 IdentityPlugin can be installed at any time, how does a user install a custom plugin? With this PR does it default to using the new identity-shiro module being introduced or does it pick up the NoopIdentityPlugin?

Hi @reta, @dblock, and @saratvemulapalli. I am tagging you because we are trying to move this PR forward but wanted your input. As mentioned by @cwperks, in this comment, the way that this PR is configured will need to be modified so that the IdentityPlugin being used is configurable. Currently, we cannot install any other IdentityPlugins since Shiro will always be installed by virtue of being a module.
Before I change this, I wanted to get the opinion of some core maintainers about whether we could move forward with this PR afterwards. An alternative approach to getting our changes into Core would be to start with a barebones Identity Interface and then merging smaller changes. This may be easier to review, but will not show the same functional demonstration that the Shiro implementation @peternied setup would.
Which direction do you prefer I take?

As mentioned by @cwperks, in this comment, the way that this PR is configured will need to be modified so that the IdentityPlugin being used is configurable.

I think this is a good point but do we want it to be configurable? I mean if in 99.9% use cases the default is enough, the other 0.1% of the users could always uninstall it and replace with the custom plugin. Also, do we always want to bundle it? May be we could just move it to plugins folder to become opt-in plugin?

Hi @reta, I think moving it from modules to a bundled plugin would be an appropriate solution. The reason we are planning on bundling it would be so that users could test the functionality of the Identity Interface and potentially create their own implementations. Basically, we are going a similar route as the SDK's HelloWorld extension where we doubt anyone will choose to use Shiro but this lets us demonstrate how the interface works.
Moving the code from a module to a plugin should allow us to ignore the installation issue and we could look into further configuration changes as needed.


fix: Views always long-resume due to envKey race

MuxStats needs the envKey in CustomerPlayerData when created. If you wait to supply it by setting CustomerData after creating MuxStats, the first event will be consumed before the envKey is available, which would cause it to be silently discarded. If you start playback soon enough, the dispatcher can pick up the envKey from subsequent events but this is timing-dependent
This causes all views generated by the core sdk to be marked Long-Resume. Core Java sequence-numbers events starting at 1 for historical reasons, and the first event the collector receives won't be 0 or 1
This change also explicitly makes it an error to pass a null envKey into postWithCompletion, which should make this an easier case to debug in the future

What do you think about adding a unit test for this fix as well, to catch any future regressions?

@andrewjl-mux Unfortunately I don't know if I really can test this situation. It's the result of a bunch of logic in the java core lib, much of which can't be injected from the outside.
Unless you had a test in mind, I mean.

@andrewjl-mux Unfortunately I don't know if I really can test this situation. It's the result of a bunch of logic in the java core lib, much of which can't be injected from the outside.
Unless you had a test in mind, I mean.

Yeah that's a good point, testing the underlying core lib logic shouldn't be done here.
An easy test I can think of is a check to make sure that this propagates through if no envKey is set
coroutineScope.launch(Dispatchers.Main) {
    completion?.onComplete(false)
}

Wdyt?


feature(Dogepedia): Resources

/dogepedia#resources resources.

Will aim to complete this by 23rd December


Clean up the names of types and constructor functions

For example, we can currently create a Range by:

boost::hana::make_range(...)
boost::hana::make<boost::hana::Range>(...)
boost::hana::range(...)
boost::hana::range_c<...>

Furthermore, the type of these objects is some boost::hana::_range<...> type, which is implementation-defined. This could be cleaned up to follow a better convention.
Also, the type of _tuple<...> is not implementation-defined, but it has a leading underscore. That is ugly.
Ideas:

All the implementation-defined types go into the detail namespace, so there's no mistake with that.
Instead of using the word unspecified-type, we should use implementation-defined. Everybody understand that.

Thoughts:
tuple<> // proper documentation
pair<> // proper documentation

set<> // implementation defined type
map<> // implementation defined type
range<> // implementation defined type

Hi, IIUC, when the standard says implementation defined, the intent is to left the implementation some latitude to implement it as they want, however we expect that an implementation will be define some the expected behavior where.
So, implementation defined has not its place in the documentation of a library the user can use.
You can say that the type name returned by a function is not in the public interface, but the function must define what the user can do with its result.

Of course, we always define what the user can do with the result. For example, the [documentation](file:///Users/ldionne/code/hana/build/doc/html/structboost_1_1hana_1_1map.html#a0f32260f859b2510dd16235b7bc011ac) for the keys function says "Returns a Sequence of the keys of the map, in unspecified order.". Since we know the return type is a Sequence, what the user can do with it is properly specified. That is always the case.


calendarView.toggleViewWithDate has no effect if shouldAutoSelectDayOnMonthChange implements with return false

if my calendarView delegate implements method with
func shouldAutoSelectDayOnMonthChange() -> Bool {
return false
}
when we call calendarView.toggleViewWithDate(date: ),if presentMonthView has no selected day,it do nothing

If i return false for ShouldAutoSelectDayOnMonthChange() method , i won't allow to change view mode . if i currently in month view try to change to WeekView won't happen

CalendarView doest not retain its selected date on Mode change, if shouldAutoSelectDayOnMonthChange() && shouldAutoSelectDayOnWeekChange() are returning false. This should not be the case.
This methods seem to be a flag for automatically selecting a date on next/previous month/week, while I am having issue with retaining their already selected date on Mode change (like MonthView to WeekView and vice versa).


Change triton version to be used for the model navigator

I saw that this model_navigator uses triton version 21.05
If I want to use the version of 21.02 , how can I config it as I have errors on different version of tensorrt and i think is because different triton version?

Have you checked container_version in YAML or --container-version CLI option? You can provide the desired version using that argument. Conversions and evaluation would goes on the same version of containers.
If you provide different version of container than default you would also need to use triton_launch_mode=docker or --triton-launch-mode docker.

@TANKIANAUN did that help solve the problem?


Add the ability to configure the Title attribute

The title attribute should be set for SEO purposes

The title attribute should be set for SEO purposes

you can do that trick
instead of using class or id you can add manually in laravel-share-fa4.php
than use the attr :class or :id as a title.
for example
<a title=":id" href=":url" target="_blank" class="yourclass" id="your id">
in view
page(null, ['id' => 'your title'])->facebook()
Hope this help you xD

PR #50 merged to add support for this


Modify user password through the API

Discussed in https://github.com/nautobot/nautobot/discussions/3436

Originally posted by rithikyennam March 16, 2023
Is there a way to create a superuser using Nautobot API
I see there is no is_superuser flag in the API for creating users.
Is there any other way of creating nautobot superuser using a python script?
If  python script is not possible can you suggest a way to create it using ansible module?
We are using ansible module for deployment, using ansible it is allowing to create a super user but we cannot set a password for it
Unable to set the is_superuser field through the API nor set the password on an existing user. Should either/both of these be supported?

is_superuser is already being tracked in #2208

Implemented for 2.0 by #3722.


Heart-trail-animation

Hey I made Heart-trail-animation Project Which Is Very Good!
This Project is very Helpfull,
I Make This Project With the help of HTML, CSS & JavaScript,
The CSS & JavaScript Code are Available in very well mannered! & Easy to understand!

@kalashvasaniya Add ss or video!

@kalashvasaniya ?


Honor 429s & RetryAfter when interacting with a Rest based Source

Description of the new feature/enhancement
With the introduction of client->Rest based sources, the client may encounter error scenarios different from what it handles when sources are purely local. For example, a rest source may have a maximum throughput limit and deem it necessary to return 429 Too Many Requests.  Ideally, the client would understand how to do backed-off retry for a small window of time before failing and informing the user that the source is currently busy.
Proposed technical implementation details
Honor
HTTP/1.1 429 Too Many Requests
Content-Type: text/html
Retry-After: 2
For example.

After reinstalling windows i tried to install all my packages with winget, but get 429s from github as many of my apps are hosted there. It would be nice if winget can go to sleep for some time after 429 and retry later.

This was implemented with some of the CDN related improvements made.


shake --debug=log doesn't put in any newlines

Self explanatory.

Self explanatory, unless you can't remember what -d does :).
Thanks for the spot, now fixed.


Strings contain trailing zeroes

Hi!
Those strings coming from unused bytes of fixed size arrays are converted to UTF-8 along with trailing zeros. I think this zeros can be trimmed safely.
let caps = device_info.query_caps()?;
println!("caps = {:#?}", caps);

caps = Capabilities {
    driver: "uvcvideo\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}",
    card: "A4 TECH USB2.0 PC Camera E: A4 \u{0}",
    bus: "usb-0000:00:14.0-3.2\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}\u{0}",
    version: (
        5,
        3,
        18,
    ),
    capabilities: VIDEO_CAPTURE | EXT_PIX_FORMAT | STREAMING,
}

Fixed in https://github.com/raymanfx/libv4l-rs/commit/82cce00844c187479bde3888620b105614731131.


Edit domainnames

I re-installed a machine and now the hostname is not anymore identical to the domainname.
The first domainname for hostname production-example was production-example.netbird.cloud. After reinstalling this is now production-example-1.netbird.cloud.
I would like to:

Change this.
Clean up the list of domainnames.

Using the cloud version.

Hello @twl9, this happened because the original production-example peer was in the system, and on domain name collision, we add a -N where N is incremental.
You can update the domain name in the Dashboard by deleting or renaming the original node and then renaming the new peer.
The name edit is available in the individual peer view:

@mlsmaycon Clear! This is then a bug, and not a feature request.
Why? The peer was already deleted (before the new peer was added). And the domainname was still not equal to the hostname.
I solved it by changing the hostname fully, and then later back to the orginal name. Its not yet resolving, but I assume a matter of a few minuted.


Make DIRT support Weak perspective Camera?

Hi~ @pmh47 is there any plan to make DIRT support a weak-perspective camera model?
Thanks!
Best,
Frank

No, but it is easy to do yourself. Two possible ways:

define a suitable projection matrix -- rotation (for orthographic projection part) then scale. Ensure that the clip-space w coordinate of the result is one, so OpenGL's perspective division does not do anything. If you have more than one object, and they have different depths, then use one scaling matrix per object.
cheat: scale your object along the camera-z-axis by a factor slightly larger than zero, large enough that the z-order is still correct, but small enough that the perspective between the near and far parts of the object is (almost) zero. Then use the usual perspective projection matrix.

OK! I will make it.
On 04/26/2020 20:04, Paul Henderson wrote:
No, but it is easy to do yourself. Two possible ways:
define a suitable projection matrix -- rotation (for orthographic projection part) then scale. Ensure that the clip-space w coordinate of the result is one, so OpenGL's perspective division does not do anything. If you have more than one object, and they have different depths, then use one scaling matrix per object.
cheat: scale your object along the camera-z-axis by a factor slightly larger than zero, large enough that the z-order is still correct, but small enough that the perspective between the near and far parts of the object is (almost) zero. Then use the usual perspective projection matrix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I also have a question about how to render multiple objects in DIRT. At issue #65.
Can you give me some advice?
Thanks!
On 04/26/2020 23:30, Zhiyang-Dou wrote:
OK! I will make it.
On 04/26/2020 20:04, Paul Henderson wrote:
No, but it is easy to do yourself. Two possible ways:
define a suitable projection matrix -- rotation (for orthographic projection part) then scale. Ensure that the clip-space w coordinate of the result is one, so OpenGL's perspective division does not do anything. If you have more than one object, and they have different depths, then use one scaling matrix per object.
cheat: scale your object along the camera-z-axis by a factor slightly larger than zero, large enough that the z-order is still correct, but small enough that the perspective between the near and far parts of the object is (almost) zero. Then use the usual perspective projection matrix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

No, but it is easy to do yourself. Two possible ways:

define a suitable projection matrix -- rotation (for orthographic projection part) then scale. Ensure that the clip-space w coordinate of the result is one, so OpenGL's perspective division does not do anything. If you have more than one object, and they have different depths, then use one scaling matrix per object.
cheat: scale your object along the camera-z-axis by a factor slightly larger than zero, large enough that the z-order is still correct, but small enough that the perspective between the near and far parts of the object is (almost) zero. Then use the usual perspective projection matrix.

Hi~ Dear Paul, I am still a little confused about setting the matrix. For a weak-perspective camera, the projection from one Point in 3D [X,Y] to 2D [x,y] should be defined as:

I do not fully understand the pipeline you provided. Am I missing something?
Thanks!

Vertex locations passed to DIRT's rasterise operations are 4D vectors (x_c, y_c, z_c, w_c) in OpenGL's clip space. To project them onto screen coordinates (x_s, y_s) in range [-1, 1], the rasteriser does (x_s, y_s) = (x_c / w_c, y_c / w_c). (x_c, y_c, z_c, w_c) are calculated in the user's tensorflow code by the view-space coordinates (x_v, y_v, z_v, 1) multiplied by the projection matrix (which typically sets w_c = -z_v, as the camera by convention looks along negative z axis).
To make a weak perspective camera, you need to set up the projection matrix such that the overall effect of the above pipeline is to reproduce the equation you quoted. If you have just one object, and so all vertices share the same value of Z_ave, then the projection matrix is
[
    [f, 0, 0, 0],
    [0, f, 0, 0],
    [0, 0, -(far + near) / (far - near), -2. * far * near / (far - near)],
    [0, 0, 0, -Z_ave]
].T

Note that again I used the standard OpenGL convention that the camera looks along the negative z-axis, meaning the view-space z_v and Z_ave are negative for the object to be visible!
The transpose accounts for the fact that DIRT uses a vector-times-matrix multiplication convention (not matrix-times-vector as is common in OpenGL).
The first, second and fourth rows do what I described. The third ensures that the depth range is set so the depth-ordering of faces within the object is correct: far is a large distance (e.g. twice as far as your scene's farthest vertex distance); near is a small distance (e.g. 0.1 * your scene's nearest vertex distance).
Note the units of f may need adjusting, depending what convention you use elsewhere.
The important difference to the full perspective projection matrix is that in the bottom row, the fixed depth Z_ave is used, instead of the view-space z coordinate z_v.
If you have multiple objects, they typically have different values of Z_ave, so use different projection matrices of this type.
Actually this is a little different to the two methods I said in the other post (and hopefully more intuitive), but they should all give the same results.

Hi~ @pmh47 Thank you for your generous help!

@Frank-Dz
Would you like to share the code for the weak perspective projection？I would appreciate it.


[BUG] get_block_traces (and _sync) can't handle block with traces without 'function_invocation'

What happened
from starknet_py.net.gateway_client import GatewayClient

mainnet_client = GatewayClient("mainnet")
call_result_get_block_traces = mainnet_client.get_block_traces_sync(block_number=21844)

The above example will raise an exception because some of the traces in this block number have no function_invocation in the trace object.
I'm using starknet-py version 0.14.0a0 on ubuntu 22.04 with python 3.8.16
Stack trace
    call_result_get_block_traces = mainnet_client.get_block_traces_sync(block_number=21844)
  File "/home/itamar/my_project/.venv/lib/python3.8/site-packages/starknet_py/utils/sync/sync.py", line 15, in impl
    return sync_fun(*args, **kwargs)
  File "/home/itamar/my_project/.venv/lib/python3.8/site-packages/asgiref/sync.py", line 240, in __call__
    return call_result.result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 437, in result
    return self.__get_result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 389, in __get_result
    raise self._exception
  File "/home/itamar/my_project/.venv/lib/python3.8/site-packages/asgiref/sync.py", line 306, in main_wrap
    result = await self.awaitable(*args, **kwargs)
  File "/home/itamar/my_project/.venv/lib/python3.8/site-packages/starknet_py/net/gateway_client.py", line 113, in get_block_traces
    return BlockTransactionTracesSchema().load(
  File "/home/itamar/my_project/.venv/lib/python3.8/site-packages/marshmallow/schema.py", line 722, in load
    return self._do_load(
  File "/home/itamar/my_project/.venv/lib/python3.8/site-packages/marshmallow/schema.py", line 909, in _do_load
    raise exc
marshmallow.exceptions.ValidationError: {'traces': {35: {'function_invocation': ['Missing data for required field.']}, 130: {'function_invocation': ['Missing data for required field.']}}}```

### Steps to reproduce

1. run the above script

### SDK Version

0.14.0a0

### Python version

3.8.16

### What operating system are you using?

Linux

### Is there an existing issue for this?

- [X] I have searched the existing issues and verified no issue exits for this problem.

Thanks for the bug report. Seems like a relatively simple issue with the schema, so the fix shouldn't be complicated.

Thanks for the quick response. The fix is working great


Is the ica.fit function compatible with GPU?

Describe the new feature or enhancement
Hi currently the ica.fit function takes 15min to run for a 4GB raw data type. May I know if it is run-able on a GPU?
Describe your proposed implementation
Describe how you think the feature or improvement should be implemented (e.g.,
as a new method on an existing class? as new capability added to an existing
method?) If you're not sure, please delete this section and the next section.
Describe possible alternatives
If you've suggested an implementation above, list here any alternative
implementations you can think of, and brief comments explaining why the chosen
implementation is better.
Additional comments
Add any other context or screenshots about the feature request here.

No, currently it's CPU-only. My only thought is that you could consider using decim to speed things up -- in principle you shouldn't need to fit all 4GB worth of samples to get a good ICA decomposition. For further optimization questions, feel free to post to https://mne.discourse.group/


Web3-cardGame using react and solidity(blockchain)

@Kritika30032002  This is a web3 card-game that utilizes the functionality of react, blockchain, smart contracts to make the game more fair and interesting.

program name?

@Kritika30032002 hii, please assign me this under codepeak program my project name is web3-cardgame-avaxgods.

hey @Kritika30032002 can u assign me i could work on this and also learn about web3

@Bayyana-kiran r u working on this?

sorry for the delay and kindly check the PR @Kritika30032002

update readme and show video

Kindly check the updated readme file?


Javascript/JSX single-quote/apostrophe breaks syntax highlighting

Basically, this is the issue https://i.gyazo.com/45d9921e631ea8274d97c4af9c062628.png

I know it says to report issues to specific language repo, but the JSX repo seems to be version-locked/deprecated: https://github.com/github-linguist/language-babel

Yeah, this is a bit of a known issue and stems from changes made to the upstream grammar that aren't compatible with the regex engine (Ruby's PCRE compiled without the PCRE_UCP flag - see https://github.com/github/linguist/issues/3291#issuecomment-255750842 why) forced us to lock to a particular version. More details in https://github.com/github/linguist/issues/3044
I've done a quick crude test and maybe switching to https://github.com/babel/babel-sublime may be the best solution going forward. It's not as actively developed as https://github.com/gandm/language-babel but it is recommended for Sublime by Facebook here and doesn't experience the problem...

with the apostrophe - Lightshow example
or the simple import lines - Lightshow example
or the larger example from https://github.com/github/linguist/issues/3044 (though this does show correctly now because of the locking) - Lightshow example

Thoughts @Alhadis, @larsbrinkhoff & @pchaigno

I've done a quick crude test and maybe switching to https://github.com/babel/babel-sublime may be the best solution going forward.

:+1:

Also an open PR with an attempt to fix #3677.
Would be so rad to see a fix for this, would be more than happy to help if someone can point me in the right direction of the fix you'd like to see.

@lewisblackwood Yup. Your fix is also feeling the apostrophe pain.  I'll spin up a PR to switch the source for the grammar.

@lildude I saw that your PR was closed on this. Is there anything we can do here to get this fixed? Still a big frustration for us!

Is there anything we can do here to get this fixed?

Beg the original upstream maintainer to relax his ways would be ✨  but sadly that's not likely to happen. 😢
The only alternative is to either update and keep maintaining our own fork of https://github.com/gandm/language-babel (not ideal) or find a better, yet still maintained, source for the grammar as https://github.com/babel/babel-sublime is lacking.
@Alhadis, I'm not an expert with writing/maintaining grammars, but how hard would it be to update https://github.com/github-linguist/language-babel with the upstream changes and tweak things to play nice with Linguist?  At the moment, that's looking like the quickest solution to this long running issue.  I'm not asking for any obligation for you to do it, just your expert opinion.

I don't imagine it'd be terribly difficult. Theoretically, it should be straightforward to hack together a script which converts Oniguruma-only grammars to PCRE-friendly ones. Both engines have different syntax for what're ultimately the same features.
This would be my preferred approach, because the actual grammar itself is long and beastly. Updating that by hand would be absurd. 😀

I have found one case to break.
const a = ["I'm", "foo"];

switch (true) {
  case /I'm foo/.test(a.join(' ')):
    console.log(true);
    break;
}

any update here?

any update here?

Nope. This is still a problem with the grammar and my attempt to switch it out wasn't successful - see https://github.com/github/linguist/issues/3044#issuecomment-459420383

Actually, looks like the upstream Atom grammar will get updated to reflect this. See https://github.com/github/linguist/issues/3044#issuecomment-461474332 onwards with a reference to https://github.com/atom/language-javascript/issues/641 in the upstream grammar. So as it stands, we're waiting on the upstream grammar.


Credo formatter 02

@ConnorRigby, tackled these this time.

lib/farmbot/farm_event/manager.ex
lib/farmbot/system/gpio/gpio.ex
lib/farmbot/repo/worker.ex
lib/farmbot/farmware/* -> in bulk.

Let me know if too large and I can split it up.

This looks good to me, but could you do one thing for me? i want to keep track of what files have and have not been formatted. Can you make a file in the root of the repository called formatted_files or something along those lines, and add each individual file that has been formatted? i want to add mix format --check-formatted in CI to make sure they stay formatted.

there is this too https://github.com/jasongoodwin/elixir-mix-format-pre-commit-hook/blob/master/pre-commit


Advanced Obstacle Avoidance

As a Driver I want the car to be partially autonomous, so that I have the option to not always control it manually
Acceptance criteria:

The car shall be able to continue its journey around the environment even if it finds an obstacle.
The car should choose the path with the most space to it’s left or right if it encounters an obstacle

Issue resolved in Advanced Obstacle Avoidance branch. May not need to be merged.


.yml file. "dubbo.registries.xx.address"  don't work.

I want to configure multiple registry centers
Referring to the "Auto-Configure" document, I found this configuration:
dubbo.registries.registry1.address = zookeeper://<IP_ADDRESS>:32770
So in my project:
dubbo:
registries:
xxxx:
protocol: zookeeper
address: @dubbo.xxxx.address@
client: zkclient
group: xxxx
But it doesn't work.
See DubboRegistryFactory.getRegistries(); size 0.
Finally, I configure it like this:
dubbo:
config:
multiple: true
registry:
xxxx:
protocol: zookeeper
address: @dubbo.xxxx.address@
client: zkclient
group: xxxx
Succeed。
See DubboRegistryFactory.getRegistries(); size 1.
Version: dubbo 2.6.0 springboot 1.5.8
I have wasted a long time.
I want to know why, how to use the ‘registries’ in document.
Which document I should look at on this question.

You can see https://github.com/alibaba/dubbo/pull/1401
dubbo 2.6.1 has fixed this bug

OK,Thank you!


change reverse grid text

if you set reverse grid to "1", that doesn't reverse anything (because it only reverses the first car with itself), so in championship events the text needs changing

could also add it to the explanation text in custom race creation


Update translation-builder-localize-label.md

Fixed a typo in world -> word.

@maggiesMSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Good catch, @QueryCoordinator!
#sign-off


Download files without compressing them

*fix issue #28 : hp.json can be downloaded uncompressed with this commit

I tried
BioDownloaderBuilder builder = BioDownloader.builder(destination);
builder.hpoJson()
.hpDiseaseAnnotations();
BioDownloader downloader = builder.build();
List files = downloader.download();
but got again a binary file for hp.json that was only 1.9 Mb
1.9M Mar  4 06:37 hp.json

Another tries gave this download
!DOCTYPE html>
<html lang="en" data-color-mode="auto" data-light-theme="light" data-dark-theme="dark" >
  <head>
    <meta charset="utf-8">
  <link rel="dns-prefetch" href="https://github.githubassets.com">
  <link rel="dns-prefetch" href="https://avatars.githubusercontent.com">
  <link rel="dns-prefetch" href="https://github-cloud.s3.amazonaws.com">
  <link rel="dns-prefetch" href="https://user-images.githubusercontent.com/">
  <link rel="preconnect" href="https://github.githubassets.com" crossorigin>
  <link rel="preconnect" href="https://avatars.githubusercontent.com">
(...)

I then tried to comment out the new line
/ //conn.setRequestProperty("Accept-Encoding", "");
and set the URLs to
hp.json=https://github.com/obophenotype/human-phenotype-ontology/blob/master/hp.json
#http://purl.obolibrary.org/obo/hp.json
#hp.obo=http://purl.obolibrary.org/obo/hp.obo
hp.obo=https://github.com/obophenotype/human-phenotype-ontology/blob/master/hp.obo
but I still got
!DOCTYPE html>
<html lang="en" data-color-mode="auto" data-light-theme="light" data-dark-theme="dark" >
  <head>
    <meta charset="utf-8">
  <link rel="dns-prefetch" href="https://github.githubassets.com">
  <link rel="dns-prefetch" href="https://avatars.githubusercontent.com">
  <link rel="dns-prefetch" href="https://github-cloud.s3.amazonaws.com">
  <link rel="dns-prefetch" href="https://user-images.githubusercontent.com/">
  <link rel="preconnect" href="https://github.githubassets.com" crossorigin>
  <link rel="preconnect" href="https://avatars.githubusercontent.com">
(...)

strange. Should we try some pair./triple programming? It is not obvious to me what the problem is

I do not see any of the above mentioned issues. When adding the following test to ``, the code works fine, including downloading uncompressed hp.json.
@Test
@Disabled("Requires internet access to run")
public void testHpoFileDownload() throws Exception {
    BioDownloader.builder(Path.of("target"))
            .hpoJson()
            .hpDiseaseAnnotations()
            .gencodeGrch38v39Basic()
            .build()
            .download();
}

The way I tested this was with the following code:
    @Test     public void testHpJsonFileDownload() throws Exception {         BioDownloader bioDownloader = BioDownloaderBuilder.builder(resourcePath).proSite().build();         List<File> downloadedFile = bioDownloader.download();    }
And then checking the folder where the file had been created and opening it showed the readable json as opposed to the previously downloading a zipped/binary file.
Let me know if that works for you Peter, so I can then merge to the dev branch.

@belkassaby , an issue with using resourcePath in the test above is that there already is a mock hp.json (four JSON lines) at src/test/resources/hp.json. So it sounds suspicious if the above works for you. It should not, as the download should be skipped.

My bad, I also added the .overwrite(true) so it would overwrite the existing file.


Fix issue #1561

Description
Fix issue #1561
Fixes https://github.com/RedHatInsights/insights-results-aggregator/issues/1561
Type of change

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

Testing steps
To be checked on CI soon.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #163 (6030be9) into master (c92b49f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #163   +/-   ##
=======================================
  Coverage   28.91%   28.91%
=======================================
  Files           4        4
  Lines         166      166
=======================================
  Hits           48       48
  Misses        112      112
  Partials        6        6

Continue to review full report at Codecov.

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


Fix: Small change to make it work for Vite

There is an issue where it will not import the components for my vue 3 setup using Vite with a standard config file:
 > node_modules/vue3-tabs-component/src/index.js:1:16: error: Could not resolve "./components/Tab"
    1 │ import Tab from './components/Tab';
      ╵                 ~~~~~~~~~~~~~~~~~~

 > node_modules/vue3-tabs-component/src/index.js:2:17: error: Could not resolve "./components/Tabs"
    2 │ import Tabs from './components/Tabs';

I had to add .vue extensions to the components in the index.js file in order to fix it:
src/index.js:
import Tab from './components/Tab.vue';
import Tabs from './components/Tabs.vue';

Hello,
I've tested it and can confirm that was the issue.
Version 1.0.1 has been released which fixed this issue.
Thank you for your contribution.


feat: provide options to transform module path

before generating import statements

What:

Why:
To resolve the problem described in #24

How:
Provide an option in babel-plugin-macros.config.js to transform module path before generating import expressions.

Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged

[ ] Added myself to contributors table

@all-contributors please add @chengjianhua for code, tests, and docs

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New Sample : Functional Programming architecture using KΛTEGORY

Since 7 months ago, we have been working on a functional programming library called KΛTEGORY. It brings functional data types and abstractions over Kotlin, and it's open to integrations with different platforms and libraries. We are moving very fast towards a first stable release, and we are actually very close to it. Already completing last steps on documentation and so on.
We have been talking about it on events like Mobilization Poland or Kotlinconf.
KΛTEGORY is a paradigm enabling library, so it could be a framework by itself. You can write any platform related apps or systems with it, as long as Kotlin is supported on the platform.
We would love to provide an architecture sample using it, the very first sample showcasing a different paradigm, similar to what you can already find on this sample repo.
Our idea over this is to provide a base implementation and iterate over it, so we would like to end up working on integrations with some "community wide accepted" Google libs like Architecture Components and Room. This could lead to very interesting collaborations! :)
You can have a look to the integration we have for RxJava2 (please look at kategory-effects-rx2 module) as an example.
Thanks in advance!
The KΛTEGORY team.

Feel free to create the sample based on one of the branches and let us know.

Will do. Thanks!

One question. Should we fork and open a PR to you guys with the sample or do you want us to provide it as a completely separate repo to just link to it under the external samples? @JoseAlcerreca

Separate repo. We only add external samples if they're about an relevant pattern or they use a library commonly used by the community.
I suggest you use blueprints to show how to migrate from todo-mvp to your lib. Like https://android.jlelse.eu/moving-from-dagger-to-koin-simplify-your-android-development-e8c61d80cddb

Ok, thanks.


Add nearby players collision check when trying to place blocks

Description
Added a function in world/mod.rs that checks nearby players by a given point and a radius.
Using this function I switched the function that gets the nearby players to the new one and chose a radius of 20.
So every time a player tries to place a block it searches up players in the radius of 20 blocks and checks the collision with them.
If no collision was found it places the block. With that change I also fixed the bug that players could place blocks at their head height and start to swim.

Testing
Tried to place blocks beside or in myself and my 2. account. Both tries worked as expected. Feels like original vanilla minecraft.
I also tried to change the game modes and place from different angles.

Checklist

(Not needed) More testing with more than 2 players. [ ]

[x] Code is well-formatted and adheres to project style guidelines: cargo fmt

[x] Code does not produce any clippy warnings cargo clippy

Overall looks good.
Thanks @Tiiita


Doesn't support Laravel 11

Feature description
Hi, the package seems need an update
./composer.json has been updated
Running composer update stephenjude/filament-blog
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.
Problem 1
- filament/tables v3.2.39 requires filament/support v3.2.39 -> satisfiable by filament/support[v3.2.39].
- filament/tables v3.2.40 requires filament/support v3.2.40 -> satisfiable by filament/support[v3.2.40].
- filament/tables v3.2.41 requires filament/support v3.2.41 -> satisfiable by filament/support[v3.2.41].
- filament/tables v3.2.42 requires filament/support v3.2.42 -> satisfiable by filament/support[v3.2.42].
- filament/tables v3.2.43 requires filament/support v3.2.43 -> satisfiable by filament/support[v3.2.43].
- filament/tables v3.2.44 requires filament/support v3.2.44 -> satisfiable by filament/support[v3.2.44].
- filament/tables v3.2.45 requires filament/support v3.2.45 -> satisfiable by filament/support[v3.2.45].
- filament/tables v3.2.46 requires filament/support v3.2.46 -> satisfiable by filament/support[v3.2.46].
- filament/tables v3.2.47 requires filament/support v3.2.47 -> satisfiable by filament/support[v3.2.47].
- filament/tables v3.2.48 requires filament/support v3.2.48 -> satisfiable by filament/support[v3.2.48].
- filament/tables v3.2.49 requires filament/support v3.2.49 -> satisfiable by filament/support[v3.2.49].
- filament/tables v3.2.50 requires filament/support v3.2.50 -> satisfiable by filament/support[v3.2.50].
- filament/tables v3.2.51 requires filament/support v3.2.51 -> satisfiable by filament/support[v3.2.51].
- filament/tables v3.2.52 requires filament/support v3.2.52 -> satisfiable by filament/support[v3.2.52].
- filament/tables v3.2.53 requires filament/support v3.2.53 -> satisfiable by filament/support[v3.2.53].
- filament/tables v3.2.54 requires filament/support v3.2.54 -> satisfiable by filament/support[v3.2.54].
- filament/tables v3.2.55 requires filament/support v3.2.55 -> satisfiable by filament/support[v3.2.55].
- filament/tables v3.2.56 requires filament/support v3.2.56 -> satisfiable by filament/support[v3.2.56].
- filament/tables v3.2.57 requires filament/support v3.2.57 -> satisfiable by filament/support[v3.2.57].
- filament/tables v3.2.58 requires filament/support v3.2.58 -> satisfiable by filament/support[v3.2.58].
- filament/tables v3.2.59 requires filament/support v3.2.59 -> satisfiable by filament/support[v3.2.59].
- filament/tables v3.2.60 requires filament/support v3.2.60 -> satisfiable by filament/support[v3.2.60].
- filament/tables v3.2.61 requires filament/support v3.2.61 -> satisfiable by filament/support[v3.2.61].
- filament/tables v3.2.62 requires filament/support v3.2.62 -> satisfiable by filament/support[v3.2.62].
- filament/tables v3.2.63 requires filament/support v3.2.63 -> satisfiable by filament/support[v3.2.63].
- filament/tables v3.2.64 requires filament/support v3.2.64 -> satisfiable by filament/support[v3.2.64].
- filament/tables v3.2.65 requires filament/support v3.2.65 -> satisfiable by filament/support[v3.2.65].
- filament/tables v3.2.66 requires filament/support v3.2.66 -> satisfiable by filament/support[v3.2.66].
- filament/tables v3.2.67 requires filament/support v3.2.67 -> satisfiable by filament/support[v3.2.67].
- filament/tables v3.2.68 requires filament/support v3.2.68 -> satisfiable by filament/support[v3.2.68].
- filament/tables v3.2.69 requires filament/support v3.2.69 -> satisfiable by filament/support[v3.2.69].
- filament/tables v3.2.70 requires filament/support v3.2.70 -> satisfiable by filament/support[v3.2.70].
- filament/tables v3.2.71 requires filament/support v3.2.71 -> satisfiable by filament/support[v3.2.71].
- filament/tables v3.2.72 requires filament/support v3.2.72 -> satisfiable by filament/support[v3.2.72].
- filament/tables v3.2.73 requires filament/support v3.2.73 -> satisfiable by filament/support[v3.2.73].
- filament/tables v3.2.74 requires filament/support v3.2.74 -> satisfiable by filament/support[v3.2.74].
- filament/tables v3.2.75 requires filament/support v3.2.75 -> satisfiable by filament/support[v3.2.75].
- filament/tables v3.2.76 requires filament/support v3.2.76 -> satisfiable by filament/support[v3.2.76].
- filament/tables v3.2.77 requires filament/support v3.2.77 -> satisfiable by filament/support[v3.2.77].
- filament/tables v3.2.78 requires filament/support v3.2.78 -> satisfiable by filament/support[v3.2.78].
- filament/tables v3.2.79 requires filament/support v3.2.79 -> satisfiable by filament/support[v3.2.79].
- filament/tables v3.2.80 requires filament/support v3.2.80 -> satisfiable by filament/support[v3.2.80].
- filament/tables v3.2.81 requires filament/support v3.2.81 -> satisfiable by filament/support[v3.2.81].
- filament/tables v3.2.82 requires filament/support v3.2.82 -> satisfiable by filament/support[v3.2.82].
- filament/tables v3.2.83 requires filament/support v3.2.83 -> satisfiable by filament/support[v3.2.83].
- filament/tables v3.2.84 requires filament/support v3.2.84 -> satisfiable by filament/support[v3.2.84].
- filament/tables v3.2.85 requires filament/support v3.2.85 -> satisfiable by filament/support[v3.2.85].
- filament/tables v3.2.86 requires filament/support v3.2.86 -> satisfiable by filament/support[v3.2.86].
- filament/tables v3.2.87 requires filament/support v3.2.87 -> satisfiable by filament/support[v3.2.87].
- filament/tables v3.2.88 requires filament/support v3.2.88 -> satisfiable by filament/support[v3.2.88].
- filament/tables v3.2.89 requires filament/support v3.2.89 -> satisfiable by filament/support[v3.2.89].
- filament/tables v3.2.90 requires filament/support v3.2.90 -> satisfiable by filament/support[v3.2.90].
- filament/tables v3.2.91 requires filament/support v3.2.91 -> satisfiable by filament/support[v3.2.91].
- filament/tables v3.2.92 requires filament/support v3.2.92 -> satisfiable by filament/support[v3.2.92].
- filament/tables v3.2.93 requires filament/support v3.2.93 -> satisfiable by filament/support[v3.2.93].
- filament/tables v3.2.94 requires filament/support v3.2.94 -> satisfiable by filament/support[v3.2.94].
- filament/tables v3.2.95 requires filament/support v3.2.95 -> satisfiable by filament/support[v3.2.95].
- stephenjude/filament-blog[0.1.0, ..., 0.1.4] require illuminate/contracts ^8.73|^9.1.0 -> found illuminate/contracts[v8.73.0, ..., v8.83.27, v9.1.0, ..., v9.52.16] but these were not loaded, likely because it conflicts with another require.
- stephenjude/filament-blog[0.1.5, ..., 0.1.6] require illuminate/contracts ^8.0|^9.0 -> found illuminate/contracts[v8.0.0, ..., v8.83.27, v9.0.0, ..., v9.52.16] but these were not loaded, likely because it conflicts with another require.
- stephenjude/filament-blog[1.0.0, ..., 1.1.2] require illuminate/contracts ^8.0|^9.0||^10.0 -> found illuminate/contracts[v8.0.0, ..., v8.83.27, v9.0.0, ..., v9.52.16, v10.0.0, ..., v10.48.16] but these were not loaded, likely because it conflicts with another require.
- stephenjude/filament-blog[2.0.0, ..., 2.0.3] require illuminate/contracts ^10.0 -> found illuminate/contracts[v10.0.0, ..., v10.48.16] but these were not loaded, likely because it conflicts with another require.
- filament/support[v3.2.39, ..., v3.2.95] require ext-intl * -> it is missing from your system. Install or enable PHP's intl extension.
- filament/tables[v3.0.49, ..., v3.2.35] require illuminate/console ^10.0 -> found illuminate/console[v10.0.0, ..., v10.48.16] but these were not loaded, likely because it conflicts with another require.
- filament/tables[v3.2.36, ..., v3.2.38] require illuminate/console ^10.45 -> found illuminate/console[v10.45.0, ..., v10.48.16] but these were not loaded, likely because it conflicts with another require.
- stephenjude/filament-blog 3.0.0 requires filament/tables ^3.0.49 -> satisfiable by filament/tables[v3.0.49, ..., v3.2.95].
- Root composer.json requires stephenjude/filament-blog * -> satisfiable by stephenjude/filament-blog[0.1.0, ..., 0.1.6, 1.0.0, 1.1.0, 1.1.1, 1.1.2, 2.0.0, 2.0.1, 2.0.2, 2.0.3, 3.0.0].
To enable extensions, verify that they are enabled in your .ini files:
- C:\xampp\php\php.ini
You can also run php --ini in a terminal to see which files are used by PHP in CLI mode.
Alternatively, you can run Composer with --ignore-platform-req=ext-intl to temporarily ignore these required extensions.
You can also try re-running composer require with an explicit version constraint, e.g. "composer require stephenjude/filament-blog:*" to figure out if any version is installable, or "composer require stephenjude/filament-blog:^2.1" if you know which you need.

@badrshs can you send a PR?


Fix typo in UG

github markdown doesnt detect extra spacings as html collapses spaces so    has to be added to differentiate John Doe and John     Doe
also fixed a small numbering error on the top of the UG

Codecov Report

Merging #270 (2ec35fe) into master (c06192e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #270   +/-   ##
=========================================
  Coverage     67.92%   67.92%
  Complexity      829      829
=========================================
  Files           121      121
  Lines          3037     3037
  Branches        441      441
=========================================
  Hits           2063     2063
  Misses          788      788
  Partials        186      186

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Debug screenshot found in the nuget version of the library

I was having a hard time figuring this one out. It looks like you've published a different version to nuget than the one that is being listed here on github.
In BrowserClient.SendRequestAsync(string url) you have an await page.ScreenshotAsync("D:/err.png"). Could you please remove this one and  double check for other specialties, that are not being pushed to github? Thanks!

Ah looks like you've removed it on 0.1.9.

Hi, this debug screenshot was included by mistake when package was compiled into new NuGet package and I didn't notice. Thanks for bringing that up. This code should be removed from 0.1.9.
I checked compiled package and version 0.1.9 doesn't contain any extra code. It matches master branch. In the future I would like to create CI/CD pipeline to release packages automatically without me needing to compile it myself.


Returning None for Task.due for tasks with only date set (not date and time)

Bug description
I have a task with only a date, when I try pulling it with the library using gettasks() it is returning none for the due attribute. It only appears for returning Task.due for tasks with date and time set.
Expected behaviour
Task is pulled in, task.due is not None. Based on the API guide, I would expect task.due.date to have a value but task.due.datetime to be empty.
Is reproducible
Yes
To reproduce

Create task with date only
Try pulling it in
Check due for null

Steps taken to try to reproduce
Attempted to create multiple tasks with date only but they never pulled in the values.
Screenshots
N/A
Version information:

Package version: 2.1.7
Python version: 3.12

Additional information
N/A

I can't reproduce based on the steps above.
Example task:

Getting it:
>>> from todoist_api_python.api import TodoistAPI
>>> api = TodoistAPI(TOKEN)
>>> api.get_task(task_id="8291811638").due
Due(date='2024-10-02',
    is_recurring=False,
    string='2 Oct',
    datetime=None,
    timezone=None)
>>>

Can you share an example?

Apologies, it appears that this was an issue on my end in converting the string back to a datetime object. The simpler example helped clear it up, thanks. Closing this now.


purge-cluster.yml does not support osd_scenario=lvm

I have a ceph cluster deployed with osd_scenario=lvm
I tried running purge-cluster and no go... it needs 'devices' defined.
fatal: [gprfs041.sbu.lab.eng.bos.redhat.com]: FAILED! => {"failed": true, "msg": "The conditional
check 'devices|length == 0' failed. The error was: error while evaluating conditional (devices|length
== 0): 'devices' is undefined\n
Since I am running with osd_scenario=lvm I have not defined that var.
FYI, the vars in 'group_vars/osds' look like this
osd_scenario: lvm
lvm_volumes:
cached-lv0: /dev/nvme0n1p5
cached-lv1: /dev/nvme0n1p6
cached-lv2: /dev/nvme0n1p7
noncached-lv0: /dev/nvme0n1p8
noncached-lv1: /dev/nvme0n1p9
noncached-lv2: /dev/nvme0n1p10

Thanks for reporting this. We currently have not build support into purge-cluster.yml for the lvm osd scenario. @alfredodeza and I were just discussing a ceph-volume lvm zap command to assist with this process, but we're not sure when it'll land in ceph. In the meantime I could try to add some tasks to purge-cluster.yml to handle this outside of ceph-volume.

I am trying to do some performance testing with lvm osd scenario which will require tearing down and rebuilding the cluster with various lvm configurations.
I could really use a  purge ability. Thanks.


docs: update cheat sheet link

Commit 91bdfb4 (Update broken GitHub cheat sheet link, 2016-05-27) updated the link on https://git-scm.com/doc, but not the identical one on /docs, with an s. The former is the main documentation landing page (which mentions the manpages along with the book, videos, etc), and the latter is just the manpages.

Thanks for fixing the issue.  I'm not familiar with Ruby, and I forgot that there might be another page with the same link.
-Dave


Add support for BPF_LD_MAP_FD pseudo instruction

BPF_LD_MAP_FD pseudo instruction is a LD IMM64 with register SRC == BPF_PSEUDO_MAP_FD(1)
Behavior is to resolve the map file descriptor and replace it with the address of the map.
Add new function ubpf_register_map_resolver to allow caller to provide callback to resolve map FD to address.
Resolves #45

Coverage decreased (-1.4%) to 94.645% when pulling 337c18566ca1b667fed6751dcf925399119d4fcf on Alan-Jowett:lddw_maps into 089f6279752adfb01386600d119913403ed326ee on iovisor:master.

This should have an interpreter implementation and some tests, including a test with one program using multiple maps. I also think that the callback should accept the ubpf_vm as first argument, so that in the future we can attach map information directly to the vm structure.

This should have an interpreter implementation and some tests, including a test with one program using multiple maps. I also think that the callback should accept the ubpf_vm as first argument, so that in the future we can attach map information directly to the vm structure.

Agreed about making the map resolver jitter aware. Modified both the register and the callback to accept void * context so that consumers have the choice of how to implement maps.

Agreed about making the map resolver jitter aware. Modified both the register and the callback to accept void * context so that consumers have the choice of how to implement maps.

Added tests for BPF_LD_MAP_FD pseudo instruction for both positive and negative (invalid FD) cases.

Added tests for BPF_LD_MAP_FD pseudo instruction for both positive and negative (invalid FD) cases.

Note: The goal is to allow consumer of ubpf to add support for maps by implementing the BPF syscall. This change allows the consumer to implement fd -> address mapping.

Note: The goal is to allow consumer of ubpf to add support for maps by implementing the BPF syscall. This change allows the consumer to implement fd -> address mapping.

Any reason why this was closed?

Short version
This overlaps with functionality in another library we are using and is redundant.
Background
My team is building a runtime + execution context (environment) that permits eBPF programs to be hosted in either user mode, kernel mode or an enclave. The runtime invokes the prevail eBPF verifier to parse and verify the eBPF program. As part of verification, callbacks are made to create maps. Net result is that ubpf is agnostic of maps in this model.
Post verification, the runtime then does:

Scan the eBPF byte code to extract list of helper functions and map FDs.
Resolve the helper function and map addresses (via call to execution context).
Replace LD_MAP_FD pseudo instructions with LDDW with the real map address.
Provide helper function addresses to jitter.
Loads the code into the execution environment.

Note:
Once I have it ready, there will be a smaller patch that allows makes ubpf more flexible about the environment it runs in. Likewise once we exit POC phase for the project we will be pushing it up to public github and will provide a link if you want.


URL Preview Widget reloads every time the listview is updated

Can't replicate, fine on my end

I think this was happening due to an incremental sync bug that I introduced in a commit. I don't think it's an issue anymore

No longer an issue