Controlling DOM elements instead of / along with Canvas bodies

I couldn't find a way to control a DOM element instead of generated bodies on canvas. For example: in the beach balls sample, I'd like to have 4 divs instead of 4 canvas circle bodies, but I'd like to keep the same physics behavior - dragging, collision (with "walls" and with each other), perhaps random moving instead of gravity. I'd also like these divs to have a certain onClick behavior, that should not interrupt the dragging. So is it possible to control the DOM elements in such way and how? Perhaps bind a div to a certain body and update its position whenever the body's position changes on drag / gravity change?

I think you'd need to make your own HTML rendering engine.

You'll need to create a custom renderer, see the wiki page for more info.
You may want to use something like three.js since it has a built in DOM rendering mode.

Someone made an example for this recently, check it out:
https://jsfiddle.net/mikatalk/suj0ymwh/

I couldn't find a way to control a DOM element instead of generated bodies on canvas. For example: in the beach balls sample, I'd like to have 4 divs instead of 4 canvas circle bodies, but I'd like to keep the same physics behavior - dragging, collision (with "walls" and with each other), perhaps random moving instead of gravity. I'd also like these divs to have a certain onClick behavior, that should not interrupt the dragging. So is it possible to control the DOM elements in such way and how? Perhaps bind a div to a certain body and update its position whenever the body's position changes on drag / gravity change?

Did you find a solution to this?


chore(main): release 3.1.0

:robot: I have created a release beep boop
3.1.0 (2024-07-24)
Features

add optional search and replace integration (grug-far.nvim) (#250) (b512d38)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v3.1.0
:sunflower:


Feature request: oblique volume slice

It maybe nice to have a similar feature like obliqueslice in MATLAB. I now have an ugly but working script:
using LinearAlgebra
using LazyGrids
using CoordinateTransformations, Rotations, StaticArrays, Interpolations

meshgrid(y,x) = (ndgrid(x,y)[[2,1]]...,)

"""
   obliqueslice(v, point, normal; method=Linear(), fillvalue::Real=0)

Extract an oblique slice from a 3-D volume `v`, by using the `point` on the output slice and
`normal` to the output slice.

# Arguments
- `point::Vector`: vector containing X, Y, Z coordinates of the point in which output slice
passes through.
- `normal:Vector`: a normal vector to the output slice.
- `method`: `Constant()` for nearest neighbor interpolation, `Linear()` for linear
interpolation.
- `fillvalue::Real=0`: Numeric scalar value used to fill pixels in the output slice that are
outside the limits of the volume.

# Output Arguments
- `slice`: interpolated values on the oblique plane.
- `xdata, ydata, zdata`: the X, Y and Z coordinates of the extracted slice in volume `v`.
"""
function obliqueslice(v, point, normal; method=Linear(), fillvalue::Real=0)

   point = Float64.(point)
   sz = size(v)
   xhalf, yhalf, zhalf = ntuple(i -> round(sz[i] / 2 + 1e-12), Val(3))

   # Initial normal vector along z axis
   ẑ = [0., 0., 1.]

   # Unit normal vector
   n̂ = normalize(normal)
   n̂ = normalize(normal)

   # Compute axis of rotation
   if ẑ == n̂
      W = n̂
   else
      W = ẑ × n̂
      normalize!(W)
   end

   # Compute angle of rotation in radians
   angle = acos(ẑ ⋅ n̂)

   # Quaternion rotation matrix
   t_quat = quat_matrix(W, -angle)

   # X, Y and Z coordinates of a plane with origin as the center
   (xp, yp), zp = let
      planeSize = 3*maximum(sz)
      xrange = round(-planeSize/2):round(planeSize/2)
      yrange = round(-planeSize/2):round(planeSize/2)
      meshgrid(yrange, xrange), zeros(length(yrange), length(xrange))
   end

   # Rotate coordinates of a plane
   xr, yr, zr = similar(xp), similar(yp), similar(zp)
   for i in eachindex(xp)
      coords = SVector(xp[i], yp[i], 0.0)
      xr[i], yr[i], zr[i] = t_quat * coords
   end

   # Shift user input point, relative to input volume having origin as center
   point[1] -= yhalf
   point[2] -= xhalf
   point[3] -= zhalf

   # Find the shortest distance between the plane that passes through input point and origin
   D = n̂[1]*point[1] + n̂[2]*point[2] + n̂[3]*point[3]

   # Translate a plane that passes from origin to input point
   xq = @. xr + D*n̂[1] + yhalf
   yq = @. yr + D*n̂[2] + xhalf
   zq = @. zr + D*n̂[3] + zhalf

   # Obtain slice at desired coordinates using interpolation
   nodes = (axes(v,2), axes(v,1), axes(v,3))
   vpermute = permutedims(v, (2,1,3))
   itp = interpolate(nodes, vpermute, Gridded(method))
   etp = extrapolate(itp, fillvalue)

   # Make bounding box around the data in oblique slice
   slicelimit = @. (xq>=1) & (xq<=sz[2]) & (yq>=1) & (yq<=sz[1]) & (zq>=1) & (zq<=sz[3])

   slicerange = find_bounding_box(slicelimit)
   XData = xq[slicerange[1], slicerange[2]]
   YData = yq[slicerange[1], slicerange[2]]
   ZData = zq[slicerange[1], slicerange[2]]

   croppedslice = etp.(XData, YData, ZData)

   croppedslice, XData, YData, ZData
end

"Return Quaternion matrix for a rotation by a given angle `θ` about a fixed axis `w`."
function quat_matrix(w, θ)
   a_x, a_y, a_z = w

   c = cos(θ)
   s = sin(θ)

   t1 = c + a_x^2*(1-c)
   t2 = a_x*a_y*(1-c) - a_z*s
   t3 = a_x*a_z*(1-c) + a_y*s
   t4 = a_y*a_x*(1-c) + a_z*s
   t5 = c + a_y^2*(1-c)
   t6 = a_y*a_z*(1-c)-a_x*s
   t7 = a_z*a_x*(1-c)-a_y*s
   t8 = a_z*a_y*(1-c)+a_x*s
   t9 = c+a_z^2*(1-c)

   r = RotMatrix{3}(t1, t2, t3, t4, t5, t6, t7, t8, t9)
   q = QuatRotation(r)
end

"Find the position of the smallest box containing the 2D `region`."
function find_bounding_box(region)
   edges = Vector{CartesianIndex}(undef, 4)

   for j in axes(region,2), i in axes(region,1)
      if region[i,j]
         edges[1] = CartesianIndex(i, j)
         break
      end
   end

   for i in axes(region,1), j in axes(region,2)
      if region[i,j]
         edges[2] = CartesianIndex(i, j)
         break
      end
   end

   for j in reverse(axes(region,2)), i in reverse(axes(region,1))
      if region[i,j]
         edges[3] = CartesianIndex(i, j)
         break
      end
   end

   for i in reverse(axes(region,1)), j in reverse(axes(region,2))
      if region[i,j]
         edges[4] = CartesianIndex(i, j)
         break
      end
   end

   imin, imax = extrema(x -> x[1], edges)
   jmin, jmax = extrema(x -> x[2], edges)

   imin:imax, jmin:jmax
end

A simple comparison shows identical results with MATLAB defaults:
using PyPlot
point = [4,4,6]
normal = [1,1,1]
v = [i + (j-1)*10 + (k-1)*20  for i in 1:10, j in 1:20, k in 1:20]
slice, XData, YData, ZData = obliqueslice(v, point, normal)

imshow(slice, cmap="gray")

Julia:

MATLAB:

Is this already available somewhere in Makie? If not, please consider adding a similar feature like this! The sample code above is not so elegant, but simply for demonstration purpose.

It is a great work, I try it works.
Can we create an oblique slice from any kind of functions in 3 dimension, e.g. f(x,y) = x^2 + y^2 ?

Can we create an oblique slice from any kind of functions in 3 dimension, e.g. f(x,y) = x^2 + y^2 ?

If you mean f(x,y,z) = x^2 + y^2 + z^2, with this kind of approach we first need to generate a discrete 3d array from the analytical formula. Something like
point = [4.0,4.0,6.0]
normal = [1,1,1]
v = [x^2 + y^2 + z^2  for x in 0:1.0:10, y in 0:1.0:10, z in 0:1.0:10]
slice, XData, YData, ZData = obliqueslice(v, point, normal)

should work? The tricky part may be how to handle the size of oblique slices, and what values to set for outliers.

It works, but I still need to learn oblique slice more.


Fixes to ClosureLifetimeFixup's copy elimination for noncopyable values

On finding unrecognized user, reset initialization of the stack so that we
don't inadvertently eliminate copy
For a mark_dependence user of the stack, look through the mark_dependence chain
to match the partial_apply

@swift-ci smoke test

Isn’t the latter resolved by: https://github.com/swiftlang/swift/pull/74593?

@Azoy https://github.com/swiftlang/swift/pull/74593 did not handle cases like testMultiCapture where markDepChain was not initialized because the capture was a copyable type.

@swift-ci smoke test

@swift-ci test windows platform

@swift-ci smoke test Linux platform


Feature/heat story/story cards

Added story cards instead of just the accordion. Clicking a card moves to the map and shows the story.
Story markers added to the map, although it cannot find the png resource for the marker
Reset button added to the map when selecting a story
Legend created on layer add / remove using hooks for leaflet

Thanks @tzinckgraf! nb @etorem - merged into feature-heat-story


Cannot compare tz-naive and tz-aware timestamps on concat

What happened:
When concatenating two dask dataframes with indices dype=datetime64[ns, UTC], I get a TypeError: Cannot compare tz-naive and tz-aware timestamps. One of the the dask dataframe was created with dd.from_pandas and the other with dd.read_parquet
What you expected to happen:
An happy concatenation ;-)
Minimal Complete Verifiable Example:
import dask.dataframe as dd
import pandas

filename = "outfile.parq"

# create a DF with tz=UTC and write to parquet file
df = pandas.DataFrame(
    columns=["A"], index=pandas.date_range("2018", "2019", freq="MS", tz="UTC"), data=1.0
)
ddf = dd.from_pandas(df, npartitions=1)
ddf.to_parquet(filename)

# read back this dask df into rddf
rddf = dd.read_parquet(filename)

# print indices
print(ddf.index)
print(rddf.index)

# attempt concatenation of dask df from parquet and new dask df
nddf = dd.concat([rddf, ddf])
# this raises the following traceback
# Traceback (most recent call last):
#   File ".../tst_parquet_tz.py", line 22, in <module>
#     nddf = dd.concat([rddf, ddf])
#   File "...\site-packages\dask\dataframe\multi.py", line 1108, in concat
#     if all(
#   File "...\site-packages\dask\dataframe\multi.py", line 1109, in <genexpr>
#     dfs[i].divisions[-1] < dfs[i + 1].divisions[0]
#   File "pandas\_libs\tslibs\timestamps.pyx", line 281, in pandas._libs.tslibs.timestamps._Timestamp.__richcmp__
#   File "pandas\_libs\tslibs\timestamps.pyx", line 295, in pandas._libs.tslibs.timestamps._Timestamp._assert_tzawareness_compat
# TypeError: Cannot compare tz-naive and tz-aware timestamps

Anything else we need to know?:
When printing both indices, we see that the dtype of both indices are datetime64[ns, UTC] yet the representation of the index coming from read-parquet shows tz-naive dates.
Dask Index Structure:
npartitions=1
2018-01-01 00:00:00+00:00    datetime64[ns, UTC]
2019-01-01 00:00:00+00:00                    ...
dtype: datetime64[ns, UTC]
Dask Name: from_pandas, 2 tasks
Dask Index Structure:
npartitions=1
2018-01-01    datetime64[ns, UTC]
2019-01-01                    ...
dtype: datetime64[ns, UTC]
Dask Name: read-parquet, 2 tasks

Environment:
Dask version: 2.30.0
Python version: 3.8
Operating System: win10
Install method (conda, pip, source): pip

Thanks for raising this. I was able to reproduce using fastparquet, but found that it works as expected when using pyarrow (engine="pyarrow").
It seems fastparquet isn't storing the timezone information properly in the metadata file, but it isn't clear to me whether or not this is a bug in the dask code base or if it lives in fastparquet. Either way there is an issue with divisions (which tries to know the edges of the partitions). As a workaround you can do rddf = rddf.clear_divisions() or you can use pyarrow.
Ping @martindurant for fastparquet expertise.

Will look...
I'll mention that parquet does not store time zones, this information only goes into the pandas metadata (i.e., only for data that came from pandas originally), so it is quite likely that this hasn't been implemented in fastparquet at all.

More about fastparquet and timezones in https://github.com/dask/fastparquet/issues/532

If I can get some guidance on this topic, I can try a PR. I have no view on what is happening before/after neither on the divisions/partitions... So maybe a bit though for me.

Actually, it may be worth trying again versus fastparquet master, because previously failure to set the timezone was being ignored, and that particular failure should no longer happen. However, it might need more work to apply the same thing to the min/max values.

I have checked with master for dask and fastparquet and the issue is still there.


fix: don't assume e has a message property

this also make sure we print the stack traces of the failing test too

It looks like CI is failing because we need to use bash instead of sh to install the js-sdk now

Fixed the CI here. Will go ahead and merge


[Broker] Fix possible data race in getFirstAvailableConsumerPermits

Motivation

there's a chance for a data race so that the consumer's available permits goes
below zero after it has been checked in isConsumerAvailable

Modifications

check that availablePermits > 0 before returning

@lhotari - in order to call get on the atomic wrapper just once, what if we added a new method that returns an Optional<Integer> that is a combination of the isConsumerAvailable and the getAvailablePermits methods? It would introduce additional overhead for the new objects, so perhaps it's an unnecessary change. Do you have a preference?

In this part of code I believe that there's a goal of zero allocations since this is part of a hot code path. Therefore, omitting the usage of Optional is also desired.

This PR fixes the issues introduce by #10417, so don't need to cherry-pick to branch-2.7


ZoomToShowLayer sometimes not openPopup()

When the marker is overseas map.
I can not understand what is the reason.
ZoomToShowLayer work fine console.log show this, but popup dont open sometimes

Problem in 1.0.0-beta.2.0 when click Gunsan link.
Marker 'Gunsan' dont show or popup not open, I think this is due to  cluster animate (zoom level = 4).
https://jsfiddle.net/3Lf6kqmu/13/

Hi,
Panning before calling zoomToShowLayer() initiates a "movestart" event whereas MCG did not attach the event listener yet. Therefore, it will think it did not move, and call the callback right away, whereas the parent cluster is still on map. Hence it thinks it has to spiderfy it => popup opens on spiderfied marker. But when the zoomToBounds in zoomToShowLayer completes, the cluster is split, unspiderfies, which closes back your popup and shows the marker on its correct position.
Now it looks like you still show some bug in your exact configuration, even when the panning before zooming is removed. Not sure exactly what could be the reason...
https://jsfiddle.net/3Lf6kqmu/14/

@ghybs Thanks!


Koalas very slow compared to pandas

Hello,
we want to use koalas. We did some tests and compared it to pandas. However, koalas was in all cases significantly slower. We tried koalas in local[32]-Mode (but the results are similar in our distributed spark cluster):
Environment:
Koalas 1.0.1
PySpark 2.4.5 (similar results with PySpark 3.0.0)
Following Code:
os.environ["PYSPARK_PYTHON"] = "/opt/conda/envs/MY_ENV/bin/python"
spark = SparkSession \
        .builder \
        .master("local[32]") \
        .config("spark.executor.memory", "20g") \
        .config("spark.driver.memory", "50g") \
        .config("spark.sql.execution.arrow.enabled", "true") \
        .config("spark.sql.execution.arrow.fallback.enabled", "true") \
        .config("spark.cores.max", "32") \
        .appName("MY_APP") \
        .getOrCreate()
ks.set_option('compute.default_index_type', 'distributed')

We made these performance comparision with koalas and pandas:
NROWS =<PHONE_NUMBER>

From now on we executed the code with
import pandas as ks
#and
import databrick.koalas as ks

Performance:
t = time()
data = ks.DataFrame(columns=['col'], data=np.random.randn(NROWS))
print(time()-t)

Pandas: 3,9 s
Koalas: 12,19 s
t = time()
print(data['col'].sum())
print((time()-t))

Pandas: 0,87 s
Koalas: 3,7 s
t = time()
print(data.describe())
print((time()-t))

Pandas: 8,13 s
Koalas: 8,1 s
t = time()
x = len(data.loc[data['col'] == 1])
print(x)
print((time()-t))

Pandas: 0,43 s
Koalas: 2,3 s
Can you give us some hints, why koalas is so slow in our test with 100 Million Random Numbers?

pandas can run much faster if the data fits within a single node, whereas Spark has an overhead of the distribution.
Could you try with larger dataset which doesn't fit within a single node? Koalas will still run with it.

We reconstructed the code above using plain Spark 3. There was no significant difference between koalas and Spark regarding the time. However spark / koalas needed much much much more CPU resources than plain pandas (x10 - 20 in my feeling). But since this is no koalas issue, this issue is closed.
By adding more columns (strings and numbers) to the 100-million-row-dataframe and applying complexer transformations (string operations, group bys,...) koalas could outperform pandas.

Thanks for your input, @Lukas012.


Check for mesh element type with spring-based method

Closes #124.

Added d22e09a15c7bb4cdb4279e222958d93c0cd1fd98 to get test coverage back up to 100%.


iPhone restore lost

After upgrade from iPhone 6S to iPhone XR, I lost all my accounts. I tried a full backup restore from iTunes and iCloud without success.
When opening the application, it lost all my accounts and I have no backup. I was expecting this to be backup. Please help!!!!!!

Unfortunately Google Authenticator was made to not have backups for security purposes. :(

There are no account backups in any of the apps by design.

If you lost access to your Google Account, you can try to reset your codes by going through account recovery. For other websites where you were using Authenticator codes for 2FA, you'll have to go through the respective websites' recovery options.

I have been adding the use of 2 factor authentication bit by bit over the years.  Always careful to keep backups of my iPhone both local on itunes and on icloud.
I had a hardware failure that required me to replace the phone this week.
Doing a restore from icloud resulted in ALL of my accounts being deleted from Google Authenticator.
I did a second restore from itunes and this time 3 out of 7 or so accounts were restored.  Thank you for that.  But the others are just gone.
Unfortunately the ones that exist still are the only ones that I have printed rescue codes for.
Looking into this, I now see comments like it is not designed to be backed up and restored.  But if that is true.  This is a very dangerous tool to be using.  You really need to be showing such warnings and advice for its use.
I will now start the process of trying to get into these various accounts again.  But it seems I have shot myself in the foot badly by trying to be safe.
Sigh.

Agreed, there should be a clear warning in the application. End user expects that a full backup being a backup. It's actually not by design, so the end user should receive a warning when creating a new account.


enable progress bar for FreeBSD

As FreeBSD uses a unsigned long for the IOCTL syscall this code would
previously fail to compile. Adding a call to into() fixes this problem.
This code should still work on other platfroms as into() is able to
'convert' an u32 into a u32.
Also change the cfg attributes so that the working code is used.
This may also work on other not yet supported platforms, but it was not
tested.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nrc (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+

:pushpin: Commit c5c7227e42d545f08a35d73696e7b5a08a163a75 has been approved by alexcrichton

:hourglass: Testing commit c5c7227e42d545f08a35d73696e7b5a08a163a75 with merge 275de19ab1b9d9ace2589b3e3f088ad5d6fd5bd6...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 275de19ab1b9d9ace2589b3e3f088ad5d6fd5bd6 to master...


GAS - When searching game, removes the "The"

Removing the The on search and then it doesn't find the game,
Exmple: The Bridge

Fixed


Union of two arrays

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Sort both arrays then iterators for arr1, arr2 then go along both arrays if an element is lower than another, increment iterator in that array, else part if we find equal elements add element to the result and increment iterators of both arrays
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
This issue related to #4960

@all-contributors please add @soumik2012 for his code


Use OpenSea as an alternative ERC721 information source

Some ERC721 tokens don't comply to the standard, yet OpenSea has information for them. We agreed on just using blockchain public information to avoid depending on a centralized service.
Should we add OpenSea as a new source for the collectibles in the backend (at least until they limit or close the API)?

Just for the record:
Not yet complete, but here's the preliminary result https://docs.google.com/spreadsheets/d/15Ul00Vd-wvr0qLwBXE3hdYraI8a0MqOG9136MpsDMck/edit?ts=5f9ae5eb#gid=0
From the top types of NFTs currently held in Safes, there’s only 2 were OpenSea has more info. And they are not standard.
Based on this, there’s imo no need to integrate OpenSea.
There is a Slack conversation about this.


Having documentation preview in PR would be really handy for reviewing

As mentioned in a https://github.com/probabl-ai/skore/issues/733, I think it would be really worth having a preview for the documentation in PR to be able to do a proper review and see if there is any glitches in the rendering.
Circle CI + a GitHub Actions to redirect is something that we use in scikit-learn and skrub.
I did a couple of documentation PRs and it is something that I missed.

https://github.com/probabl-ai/skore/pull/1027#issuecomment-2563892478


Missing $HOME/.config/polybar/system

Hey, I might be overlooking something but it seems like the .config/polybar/system directory is missing in the dotfiles.

Heyy, could you please give me an screenshot (which part of polybar isn't working) or give a brief about the issue so i can update the dotfiles as per it.

Hello, thanks for bringing it up, i have updated the dotfiles.


Performance in HoloLens2

Hi, @SitronX ,
Sorry for post here, because I cannot find discussion session.
I have reduced my segment according to your video step-by-step https://www.youtube.com/watch?v=affo3I7i2-8 (lower to 500KB) and still got bad performance in my HoloLens2 app natively, so do you have any other suggestions?
I see both the FnO-Model-visualisation and FnO-Volume-visualisation video of HoloLens, it seems like your project have nice FPS, can I know the value of FPS you run and how you record it? Because when I record using HoloLens device portal, the performance will always be bad.
Thank you for your work! It really helps me a lot!

@shiauyang  Hello.
How many vertices does your model have? That is the most important factor, not space on disc.
Here are my performance results i did back then (when actively developing).
For volumetric version:

And here for model version:

The fps on both versions was measured by my script, so both versions had same conditions and i was not relying on 3rd party software (i think the FPS script can still be found somewhere in the scene, but i am not sure). But it was also quite problematic due to internal vsync On Hololens 2 and Quest Air link (that is why some values from table are capped).


Add sha256 hash calculation to CHANGELOG

Changes:

Adds the sha256 hash for each binary in the CHANGELOG.md

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[X] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[X] No

In case you checked yes, did you write tests?

[ ] Yes
[X] No

Comments about testing , should you have some (optional)
Further comments (optional)
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report
Base: 57.39% // Head: 57.39% // No change to project coverage :thumbsup:

Coverage data is based on head (ea1214b) compared to base (fc4e55f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #214   +/-   ##
========================================
  Coverage    57.39%   57.39%
========================================
  Files           59       59
  Lines         3915     3915
========================================
  Hits          2247     2247
  Misses        1576     1576
  Partials        92       92

Flag
Coverage Δ

unittests
57.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


added contributing guidelines to the project

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
If there are any UI change, please include the screenshots also.
Fixes #(issue)
Type of change
Just put an x in the [] which are valid.

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.

[ ] ./gradlew assembleDebug assembleRelease
[ ] ./gradlew checkstyle

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Project sounds very interesting and I'm looking forward to contribute more, here is my first contribution on contributor guidelines. please review my pull request. Thank you.

@santoshhiremani Welcome to project travel mate! and congrats on your first commit!
You can join slack also here for better communication!


fix(v2): center align content if no sidebar exists

Motivation
Fixes #4512
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
See To Reproduce section in the issue above.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks, that's an improvement compared to the current state so I'm going to merge
Still feels it's a bit weird to have the text not totally centered. We can improve this later.


Add horizontal segment for boxplot whiskers and option to control its size

Result should look like combination of boxplot and errorbar:

PR: https://github.com/JetBrains/lets-plot/pull/578


ICredentialProvider - Internal Server Error

Bot Info

SDK Platform: .NET
SDK Version: 3.11.0.
Deployment Environment: Localhost with Bot Emulator

Issue Description
I am trying to run the code sample shown in the following comment but keep running into ‘[500] Internal Server Error’ with “System.Net.Http.HttpRequestException: 'Response status code does not indicate success: 401 (Unauthorized).'” thrown.
https://github.com/Microsoft/BotBuilder/issues/2258#issuecomment-280506334
Is there an update to the above code sample? Basically what we are trying to achieve is avoid storing the AppID and password in the web.config file.
Code Example
https://github.com/Microsoft/BotBuilder/issues/2258#issuecomment-280506334
Reproduction Steps

Create new Bot Application project in Visual Studio 2017
Replace code in MessageController with the code from the GitHub comment above.
Replace AppID and Password in the code.
Build and run the project.
Launch Bot-Framework-Emulator, fill in the AppID and Password and connect to the Bot on localhost

Expected Behavior

Able to communicate with bot with no error thrown.

Actual Results
Bot emulator shows
[11:11:40] -> POST 202 [conversationUpdate]
[11:11:41] -> POST Request to 'http://localhost:3979/api/messages' failed: [500] Internal Server Error
Notes
Hard coding the AppID and Password in the web.config works as excepted without any errors thrown.

@Manriat could you please explain your use case for us so we can better assist you?

So this is a legitimate bug we have a fix merged already and will go out in the next release.  For now you can add the following class to your project:
    public static class Class1
        {
        public const string AppPasswordClaim = "appPassword";

        public static string GetAppIdFromClaims(this ClaimsIdentity identity)
        {
            if (identity == null)
                return null;

            Claim botClaim = identity.Claims.FirstOrDefault(c => c.Type == "appid" || c.Type == "azp");
            if (botClaim != null)
                return botClaim.Value;

            botClaim = identity.Claims.FirstOrDefault(c => c.Issuer == "https://api.botframework.com" && c.Type == "aud");
            if (botClaim != null)
                return botClaim.Value;

            return null;
        }

        public static string GetAppPasswordFromClaims(this ClaimsIdentity identity)
        {
            return identity?.Claims.FirstOrDefault(c => c.Type == AppPasswordClaim)?.Value;
        }

        public static MicrosoftAppCredentials GetCredentialsFromClaims(this ClaimsIdentity claimsIdentity)
        {
            var appId = claimsIdentity.GetAppIdFromClaims();
            var password = claimsIdentity.GetAppPasswordFromClaims();
            return new MicrosoftAppCredentials(appId, password);
        }
    }

And update your MessagesController.cs constructor to this:
        static MessagesController()
        {
            // Update the container to use the right MicorosftAppCredentials based on
            // Identity set by BotAuthentication
            var builder = new ContainerBuilder();

            builder.Register(c => Class1.GetCredentialsFromClaims(((ClaimsIdentity)HttpContext.Current.User.Identity)))
                .AsSelf()
                .InstancePerLifetimeScope();
            builder.Update(Conversation.Container);
        }

Please let me know if this works for you. Good luck!

Thank you for the quick response @JasonSowers. I will test it out and let you know if I run into any issues.

@JasonSowers Just tested your fix and it works! Thanks again!

@JasonSowers
I am trying to solve exactly same problem i.e. Passing AppId,Secret manually and not from web config.
I followed steps mentioned this post and #2258 (comment)
However, I get following exception. (Exception details from AppInsights)
POST to the bot's endpoint failed with HTTP status 500.
Exception Details :
{"method":"Microsoft.Bot.ChannelConnector.BotAPI+d__29.MoveNext","level":0,"line":312,"assembly":"Microsoft.Bot.ChannelConnector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","fileName":"d:\a\1\s\ChannelSDK\Microsoft.Bot.ChannelConnector\API\BotAPI.cs"}
Any idea what am i doing wrong?
Thank you for your help!

@rohantare hard to say without seeing your code, can you put it in a repo?

Hey Jason,
Thank you for your message. Here is the link for my MultiBotPOC
https://github.com/rohantare/MultiBotPOC/
Issue 01:
What I think causing issue is following code in POST method of messages controller :
ConnectorClient client = new ConnectorClient(new Uri(activity.ServiceUrl));
var reply = activity.CreateReply( "TEST");
await client.Conversations.ReplyToActivityAsync(reply);
If I remove this piece of code, multiple bot authentication works fine.
Issue 02:
I also pass bot name to Messages controller's POST method like following (Not in my checked-in code)
public async Task Post(string botName, [FromBody]Activity activity)
{
if (activity.Type == ActivityTypes.Message)
{
ConnectorClient client = new ConnectorClient(new Uri(activity.ServiceUrl));
var reply = activity.CreateReply("Bot Name is:  "+botName);
await client.Conversations.ReplyToActivityAsync(reply);
await Conversation.SendAsync(activity, () => new Dialogs.RootDialog());
}
}
This also does not work.
Exception details from AppInsights are :
POST to the bot's endpoint failed with HTTP status 500.
Exception Details :
{"method":"Microsoft.Bot.ChannelConnector.BotAPI+d__29.MoveNext","level":0,"line":312,"assembly":"Microsoft.Bot.ChannelConnector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","fileName":"d:\a\1\s\ChannelSDK\Microsoft.Bot.ChannelConnector\API\BotAPI.cs"}
Can you tell me what is causing above two issues?
Thanks a ton in advance!
Regards,
Rohan

@JasonSowers
I think it was my bad, I wasn't resolving the connector properly. It is working for me.
Thanks!

@rohantare Can you please share the fix?

Fuck. I hate this issues tickets. A lot of searching and a lot of closed tickets without any shared help.


default to site.name for page titles

When I first set up a project to use DOCter, I was a little puzzled and annoyed that the page title was still DOCter after I updated my config file. This defaults the page titles to the site.name variable for faster setup.
Additions

add check for page.title in default layout
use site.name as title if there is no page title defined
also use site.name when page.title is defined like so "Page Title - SiteName"

Removals

remove title from index page front matter so it defaults to the site name from site config

Testing

check out locally
run jekyll serve --baseurl ''
observe page titles on index page and the example page to see the change
edit site.name to see the index page title change dynamically (I think you have to restart the serve task though)

Review

@ascott1

Screenshots
index page title

example page title with site.name in it

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

:+1:


Compatibility with terraform 0.10.0

At least one change in terraform 0.10.0 has broken terragrunt...
Specifically this change:

The terraform init subcommand no longer takes a SOURCE argument to copy to the current directory. The behavior has been changed to match that of plan and apply, so that a configuration can be provided as an argument on the commandline while initializing the current directory. If a module needs to be copied into the current directory before initialization, it will have to be done manually.

Terragrunt uses the old two argument form of init, and terraform complains:
$ terragrunt plan
[terragrunt] [/foobar] 2017/07/05 16:55:38 Running command: terraform --version
[terragrunt] 2017/07/05 16:55:39 Reading Terragrunt config file at /foobar/terraform.tfvars
[terragrunt] 2017/07/05 16:55:39 Downloading Terraform configurations from git::ssh://git@githost/org/proj.git?ref=master into /tmp/terragrunt/Q9fDMTFutsTifIUWoQCicP-_WVY/b8EcM0nU_fk63G8D7qkIzC15V0Y
[terragrunt] 2017/07/05 16:55:39 Running command: terraform init -backend=false -get=false git::ssh://git@githost/org/proj.git?ref=master /tmp/terragrunt/Q9fDMTFutsTifIUWoQCicP-_WVY/b8EcM0nU_fk63G8D7qkIzC15V0Y
The init command expects at most one argument.

Usage: terraform init [options] [DIR]

There might be more incompatibilities, but this is the first one that I noticed.  Would be good to do some investigation into other incompatibilities.

 Thanks for pointing this out. I just left a comment asking about workarounds and alternatives for this completely backwards incompatible change: https://github.com/hashicorp/terraform/pull/15032#issuecomment-313277025.

Since the comment doesn't seem to be attracting any attention from the devs, perhaps it would be worthwhile to open it as a new issue terraform's issue tracker?

@jre21 A bug on Terraform's issue tracker sounds worthwhile!

https://github.com/hashicorp/terraform/issues/15655

Thanks!

It looks like terraform 0.10.0 was released this morning with the init -from-module syntax included.

Guess who spent yesterday working out an AWS multi-env, multi-az terraform workflow using terragrunt and found it was broken this morning after brew update? :)
I'm reverting to 0.9.11 but would be good to see a release soon.
As a side note, would be good to see a Homebrew formula for terragrunt. I'll raise a separate issue for that.
R.

Any update on whether or not this is going to happen soon? Currently blocked because I am missing a new built-in with the VPN connections. aws_vpn_gateway_route_propagation resource is only available in 0.10.0. :(

We're discussing the best way to handle this in #261. There are some non-trivial things to decide. Feel free to chime in!

Yeah I saw that. I'm looking at the PR now. :D

Fixed by #261.


[SPARK-6856] [R] Make RDD information more useful in SparkR

Can one of the admins verify this patch?

Jenkins, ok to test

@His-name-is-Joof Could you also add a unit test for this ? Something similar to https://github.com/apache/spark/blob/master/R/pkg/inst/tests/test_sparkSQL.R#L637

Test build #30857 has started for   PR 5667 at commit 123be65.

Test build #30869 has started for   PR 5667 at commit c8c0b80.

How's that? Very new to contributing to large projects in general, so criticism welcome. Excellent bugtracker and starter bugs!

Test build #30857 has finished for   PR 5667 at commit 123be65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Test build #30869 has finished for   PR 5667 at commit c8c0b80.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Thanks - this is looking good. I had minor comment inline. Also could you bring your PR up to date with the latest master branch ?

Test build #30903 has started for   PR 5667 at commit 9d2295e.

Test build #30903 has finished for   PR 5667 at commit 9d2295e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

LGTM. Thanks @His-name-is-Joof . Merging this

Actually I just tried this out locally and have a minor comment: Could you add a new line character to the end of the toString ? Right now on the shell the output I get looks like
> a <- parallelize(sc, 1:20, 2L)
> a
ParallelCollectionRDD[0] at parallelize at RRDD.scala:450>

Thanks for your patience shivaram.

Jenkins, retest this please

Thanks for the update. I tried it out and change looks good. Waiting for Jenkins now

@shaneknapp Jenkins doesn't seem to picking up the re-test request. Is this related to the outages we had yesterday ?

I'm out of town for the weekend. I can check Sunday night.
On Apr 25, 2015 12:17 PM, "Shivaram Venkataraman"<EMAIL_ADDRESS>wrote:

@shaneknapp https://github.com/shaneknapp Jenkins doesn't seem to
picking up the re-test request. Is this related to the outages we had
yesterday ?
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/5667#issuecomment-96264560.

jenkins test this please

jenkins test this please

ok, fixed
On Sat, Apr 25, 2015 at 1:36 PM, shane knapp ☠<EMAIL_ADDRESS>wrote:

I'm out of town for the weekend. I can check Sunday night.
On Apr 25, 2015 12:17 PM, "Shivaram Venkataraman" <
<EMAIL_ADDRESS>wrote:

@shaneknapp https://github.com/shaneknapp Jenkins doesn't seem to
picking up the re-test request. Is this related to the outages we had
yesterday ?
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/5667#issuecomment-96264560.

Can one of the admins verify this patch?

Jenkins, ok to test

Test build #31060 has started for   PR 5667 at commit f8814a6.

Test build #31060 has finished for   PR 5667 at commit f8814a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

@His-name-is-Joof -- Could you let me know what your JIRA username is ? I would like to assign this issue to your

Joof
On Apr 27, 2015 2:41 PM, "Shivaram Venkataraman"<EMAIL_ADDRESS>wrote:

@His-name-is-Joof https://github.com/His-name-is-Joof -- Could you let
me know what your JIRA username is ? I would like to assign this issue to
your
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/5667#issuecomment-96813694.

@srowen Could you add Joof as a contributor on our JIRA ? The assignee auto-complete doesn't seem to pick this up right now


install -V does not show the filename

$ touch file
# With GNU:

$ install -t sub4/c -Dv file
install: creating directory 'sub4'
install: creating directory 'sub4/c'
'file' -> 'sub4/c/file'

# With Rust
$ ./target/debug/coreutils install -t sub4/c -Dv file
install: creating directory 'sub4'
'file' -> 'sub4/c'

The GNU has file but not with Rust
Tested here:
https://github.com/coreutils/coreutils/blob/master/tests/install/basic-1.sh

Will need:
https://github.com/uutils/coreutils/pull/3651


Open internal pages link on the new tab

Link to the internal pages has a mention type in notion API:
{
            "type": "mention",
            "mention": {
                "type": "page",
                "page": {
                    "id": "2468a3-44a5-73dd-458c7-7863adc09"
                }
            },
            "plain_text": "link test",
            "href": "https://www.notion.so/2468a344a573dd458c77863adc09"
}

And there is no option to make it open on the new tab- this would be useful to have such behaviour by default, or on-demand.

Hi! @ixzzd Version 3.11.3 is out, im really sorry for the delay 😢
Thanks for your PR again :)
Release: https://github.com/9gustin/react-notion-render/releases/tag/v3.11.3

I can confirm that the issue is resolved, now the nested links opens on the new tab as expected.
Thank you @9gustin for the great package :)


updating k8s monitoring charts

Pull request checklist
Use the following template to make sure your PR fits the GoQuorum documentation standard.
Before creating the PR
Make sure that:

[x] You've read the contribution guidelines.
[x] You've previewed your changes locally.

Describe the change
updating k8s monitoring charts
Impacted parts

For content changes:

[x] Documentation content
[ ] Documentation page organization

For tool changes:

[ ] Build and QA tools (e.g., lint)
[ ] MkDocs templates
[ ] MkDocs configuration
[ ] Python dependencies
[ ] Node dependencies and JavaScript
[ ] GitHub integration

After creating your PR and tests have finished
Make sure that:

[ ] You've fixed any issues raised by the tests.
[ ] You've previewed your changes on GitHub Pages

One suggestion, then LGTM

:+1: thankyou @alexandratran !


Add support for bynmonthday and bynweekday, refs #104

The changes in nlp.js are required to make the sample code in #104 work.

Thanks @clochix. I think this doesn't address the root cause of #104, but I'm merging it for now as a workaround. (See also https://github.com/jkbrzt/rrule#instance-properties.)

Thanks a lot @jkbrzt, this will allow us fix cozy/cozy-ical#39 ☺

:+1:


When using a nexus3 repository, the pipeline will pull old artifacts if there are more than 50 results

The API used for finding the latests artifacts in the Nexus3 repository uses pagination for it's results. The pipeline is unaware of this, and so if you have more than a page's worth of artifacts in your search results, you won't necessarily be getting the newest artifacts when you do the deploy step.
Specifically in the 'downloadAppBinaryFromNexus3' function the curl to ${nexusApiSearchUrl} will only return that first page of results.

Hi! It's been some time, sorry about that. Can you point to the exact place you're referring to?


Update update-management.md

Per our specific limits docs, the concurrent jobs limit is 200, not 2000. Please double-check with an SME!
https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/azure-subscription-service-limits#automation-limits

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

@dcbrown16 - It appears an extra zero was accidentally added and it wasn't intentional.  It should have been caught but wasn't, so thanks for bringing this to my attention. I approve this PR for merge.


OCPBUGS-31459: Fix downstreaming sync PRs

In OCPBUGS-31459 we identified that ART PRs get overridden by our downstreaming PR because ART PR and downstreaming PR get created periodically get the same base commit.
We made some changes to repo CI to make sure that we merge PRs with rebase merge strategy (except to downstreaming PRs) to make sync easier.
This PR contains two commits:

Revert of 1000454d77913bac70d6a65aa81a3c9a951e8f2d which we will drop on the next sync
Cherry pick of 1000454d77913bac70d6a65aa81a3c9a951e8f2d which is basically a revert of the revert above

This makes the diff of this PR empty, but this should fix the order of commits for further downstream syncs.

/approve
/lgtm

/approve

/approve


Refactor Websocket server parser to support space char in values

Description

Ref: https://github.com/DiceDB/dice/issues/1029#issuecomment-2435065742
For string value with a space (e.g., "name":"New York"), the command parsing and argument splitting in the command parser for websocket server, does not handle this scenario correctly.
Refactor parser to handle such scenarios
We can refer to httpServer parser for references.

Tasks

Refactor websocket server command parser to support space char as key values.
Add tests for the given scenario.

I can take this up @lucifercr07

Hi @lucifercr07, I am new to the community and would love to take up this issue. Can you assign it to me?

@helios2003 let me know incase any details are required for recreating this issue.

I have a small question here: Should the key and value both be able to handle in-between spaces or is it only for the value?


Mantine v6 support?

First off, thanks for this project. I know it was just released, but I wanted to ask if you've tested this with Mantine v6? I just gave the add-on a shot and I'm not sure if the issues I'm seeing are related to my theme or because I'm using v6.

I've just release v1.3.0 to NPM which add's support for V6 :)


Changing close button

Is there a way to change the close button image to either a simple X character or to update the path to a different close button?

You can set another image with closeSrc: 'path/to/your/image.png',

That attribute is not working for me:
closeSrc: 'https://static.wixstatic.com/media/34b396_c3723d3c68e941229e546a46a69311eb~mv2.png/v1/fit/w_750,h_750/34b396_c3723d3c68e941229e546a46a69311eb~mv2.png'


Fixes #1564. Remove latlon basic constructor

Description

Per #1564, @bena-nasa said:

@tclune @weiyuan-jiang This interface is not being used as far as I'm aware. I didn't even know it existed. I've only used the from parameters one and if I've never used it nobody else has. If we were, we would have hit it.
Just delete this interface.

So, we do just that.
Related Issue

Closes #1564
Motivation and Context

Gets rid of a buggy interface to generate LatLon grid.
How Has This Been Tested?

Well...if the CI builds and runs, I guess it's okay?
Still, I'll try and do a build of GEOSgcm to test.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[x] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

I ran a GEOSgcm run with this update and it didn't crash. So, I guess it's good!


Create cluster fails while trying to verify candidate authority certificate "kube-ca"

Attempt to create cluster fails:
[controlPlane] Failed to bring up Control Plane: Failed to verify healthcheck: Service [kube-apiserver] is not healthy on host [<IP_ADDRESS>]. Response code: [401], response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Unauthorized","reason":"Unauthorized","code":401} , log: E0819 18:51:55.129928 1 authentication.go:62] Unable to authenticate the request due to an error: [x509: certificate signed by unknown authority, x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kube-ca")]; exit status 1, error: You must be logged in to the server (the server has asked for the client to provide credentials)

Useful
Info

Versions
Rancher v2.0.7 UI: v2.0.67

Access
local admin

Route
authenticated.cluster.nodes.index

Please provide steps to reproduce, see https://rancher.com/docs/rancher/v2.x/en/contributing/#bugs-issues-or-questions
What kind of cluster, what kind of nodes? Are nodes re-used? If so, are the locations mentioned in https://rancher.com/docs/rancher/v2.x/en/faq/cleaning-cluster-nodes/ cleaned before re-using?

Let us know if we can help investigate.


HSC VO TAP demo from MAST newsletter

Hubble Source Catalog demo using VO Table Access Protocol MAST service and astroquery client.
Originally published in MAST newsletter, updated to match style guide.

Current build is failing on importing scipy. Scipy is only used for the last image generation, not the bulk of the TAP example processing; it could arguably be cut, though it really is the payoff visualization of the demonstration. How should we proceed?

/ cc @eteq

Scipy is a reasonable dependency! Just an oversight by me to not include it.  I'll just push up a commit here adding it and we'll see what happens when I've done that.

The new failure now looks to be real.  @theresadower, you may find it useful to follow the instructions here (which I just added...) showing how to execute the notebook locally.


Small edit to the limits of a global reader

[Enter feedback here]
Following statement should be added under Global Reader and Users with this role cannot do the following:
Cannot view Defender XDR Reports for other users

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Microsoft Entra built-in roles - Microsoft Entra ID
Content Source: docs/identity/role-based-access-control/permissions-reference.md
Service: entra-id
Sub-service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@NaserQujaq
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

@.*** Qoujaq (ESTARTA @.***> for visibility
From: AjayBathini-MSFT @.>
Sent: Monday, May 13, 2024 7:08 AM
To: MicrosoftDocs/entra-docs @.>
Cc: Naser Qujaq @.>; Mention @.>
Subject: Re: [MicrosoftDocs/entra-docs] Small edit to the limits of a global reader (Issue #778)
@NaserQujaqhttps://github.com/NaserQujaq
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/entra-docs/issues/778#issuecomment-2106597417, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3WKBZFC5D6T6VQUSGPW5MLZCA4DNAVCNFSM6AAAAABHMZQ5K2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGU4TONBRG4.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Hi @NaserQujaq
Thanks for your feedback. We've sent your feedback to the team to review. We'll update the docs based on their review.
We are migrating to a new doc feedback tool, so we will close this GitHub issue. Let us know if you have additional feedback.
thanks

#please-close


Update to RequireJS 2.1.16

Update to RequireJS latest version:
http://requirejs.org/docs/release/2.1.16/minified/require.js

Could you make the changes, do some local sanity testing, and send a PR?

Thanks for the PR! Closing this issue.


add a helper for the v8::JSON object and its methods Parse & Stringify

As the V8 API for the JSON object has changed over the course of the versions of V8 used in node, it makes good sense to add a helper for it, especially now that the Stringify method has been made accessible in node 7.
v8::JSON has two methods, Parse and Stringify ...  For a link to the v8 docs used in node 7, see v8::JSON Class Reference
As for the changes in v8::JSON in the versions of node supported by NAN:
v8::JSON::Parse was first available in node version 0.12.x - it is not otherwise available in versions 0.8 or 0.10.x
v8::JSON::Stringify was first available in node version 7 - it is not otherwise available in earlier versions

I submitted PR #651 to add this functionality.
Anybody that is interested in using this functionality before it is reviewed / merged into NAN can use the native-json npm package.   Also maintained on github: node-native-json
I just think something like this makes more sense as a part of NAN rather than a separate package.  Comments are welcome.


Evaluate removing Spring Security

Currently using Spring Security doesn't seem to be providing great value. There is much boilerplate needed for making it work and removing it will probably make the code base smaller. Dynamic routes can be added after login depending on the user's role as an alternative security solution.
The current boilerplate could be moved to a separate addon. In that case it could make sense to use it still.
Relates to #736

Covered by https://github.com/vaadin/bakery-app-starter-flow-spring/commit/9a19c9a7f1cb3d2f1e139acd4e9f51342e893b3e.


⚠ Samsung has degraded performance

In 345bac2, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2997 ms

Resolved: Samsung performance has improved in 979c7e6.


spdlog leaks into librmm (?)

Not sure if this is intended --- I downloaded the librmm package and inspected the files locally, and this is what I saw:

There are headers and also a static library for spdlog, but should these be included in the librmm package? I thought spdlog is a standalone package?
https://github.com/conda-forge/librmm-feedstock/blob/32b6251bf7baaaa908701b3ff6ac54eba070461e/recipe/meta.yaml#L27
My guess is these are due to spdlog being fetched by cmake instead of conda.
cc: @kkraus14 @jakirkham @rlratzel

Yep, need it in the host section instead of just the run section

lib64 needs to be changed to lib

Yep, need it in the host section instead of just the run section

It has run_exports so I think we just need the host section
https://github.com/conda-forge/spdlog-feedstock/blob/c96430f1cb8e576908e2cd295e9bf73a4a245d57/recipe/meta.yaml#L15-L16

@kkraus14 What exactly does the build script do? Why isn't it generating a librmm.so shared library?

(I mean I know it's a header-only library, so an equivalent question is why do we need a cmake build script for rmm? We didn't need one for Thrust/CUB.)

I don't quite remember if the CMake install process generates the correct CMake pieces to allow someone to do a find_package(RMM)

OK. There are rmm-*.cmake included in the package, so I think this is why.

Think we need it in both host and run since it is also needed by libraries including RMM

Yes, @jakirkham, I just pushed #3.

Thanks Leo! 😄
Proposing we mark the old packages broken in PR ( https://github.com/conda-forge/admin-requests/pull/259 ). So that users don't get these by accident


Audio to Video

Q
A

Bug?
no

New Feature?
no

Version Used
Specific tag or commit sha

FFmpeg Version
FFmpeg or AVConv and version

OS
Your OS and version

Is it possible to create a Video from a MP3/Audio Source with a Image Preview with this Lib?
The Command for FFMPEG will be
ffmpeg -i $audiopath -f image2 -loop 1 -r 30 -i $imagepath \-shortest -c:a copy -c:v libx264 -crf 23 -preset veryfast $outputpath

Only custom filters. Added as a feature request for v1.x

@jens1o I am also trying to perform the same task. Is there any documentation for contributing and code design?

@jens1o Thanks for mentioning the method. I have done the work and have a passing build on my end.
You could see here:
https://github.com/teddywest32/PHP-FFMpeg/pull/1
On which branch shall i do the pull request on?


column not stacking and going to 100% on mobile

Title says it all really so i just provide my code for the component and a screen shot of what is happening. I have the below code snippet in the other mjml, mj-body, mj-container tags

<mj-section>
        <mj-column>
            <mj-image src="http://placehold.it/600x469/000/fff?width=600" href="http://www.syndicut.com" width="600" padding="0 10px 0 0"/>
            <mj-text>
                Header
                Fermentum est venenatis ex facilisis, quis bibendum lacus faucibus. Pellentesque vulputate auctor libero, ac maximus lacus mollis eget.
            </mj-text>
            <mj-button href="/" align="center">
                Button
            </mj-button>
        </mj-column>
        <mj-column>
            <mj-image src="http://placehold.it/600x469/000/fff?width=600" href="http://www.syndicut.com" width="600" padding="0 0 0 10px"/>
            <mj-text>
                Header
                Fermentum est venenatis ex facilisis, quis bibendum lacus faucibus. Pellentesque vulputate auctor libero, ac maximus lacus mollis eget.
            </mj-text>
            <mj-button href="/" align="center">
                Button
            </mj-button>
        </mj-column>
    </mj-section>

What i want them to do on mobile is stack and have the images 100%

Columns stack by default so this is what should happen as you can see here: https://mjml.io/try-it-live/B1FiUBVsl. Or did I misunderstand what you want to do?

Hi @ngarnier just after i posted on here i tried it in the "try it" and get the the result you have. do you know why i'm not seeing this in my build? i'm using the grunt task if that helps

Hi @adamjw3 can you share the HTML being rendered ? and how did you test it ?

Hi @iRyusa I'm just testing it in the browsers (chrome). I've attached the generated html.
component-content.zip

It works fine on my chrome, your screenshot doesn't look like to be under the mobile breakpoint (480px) can you confirm that ?

 Is one thing, when you do go below 480px my images don't span full width. i've attached a screenshot

This is a known issue, and atm we didn't find any fix on that. You can see more details here : #263

@iRyusa how come it works if i put it in the mjml editor https://mjml.io/try-it-live

The width of mj-image is calculated according to the desktop version. To illustrate roughly, the width of the image will be equal to:

its container width if there is one column
its container width / 2 if there is two equal columns
its container width / 3 if the image is contained in a 33% column
etc

We can't set the width of the image to 100% because of lack of support in Outlook.
The result of the image totally scaling up you're seeing in the try it live is actually a bug introduced in the beta version the try it live is running on, it's fixed in beta.8 (cf https://github.com/mjmlio/mjml/issues/578).
I'm closing as it's a known issue: https://github.com/mjmlio/mjml/issues/263

I found a guide on creating your own components. Is it possible to create a custom component to get round this issue?

hey @adamjw3, you could totally create your own image component and use a different HTML indeed! If you find a nice workaround, feel free to share, we might reintegrate it into MJML :)

Im having this problem with 2 mj-column inside a section. The surrounding table is not set to 100% width so the images do not scale up. Is there a fix for this yet?
<mj-section>
    <mj-column>
       <mj-image src="pic.jpg" />
    </mj-column>
    <mj-column>
       <mj-image src="pic.jpg" />
    </mj-column>
</mj-section>


Switch from Dependabot to Renovate

Apparently Renovate respects dependencies which Dependabot clearly doesn't.
https://github.com/apps/renovate
https://github.com/apps/renovate-approve

Sounds like maybe this might cause more problems than it solves.


Error building persistence bundle

It seems there is a depedency missing for the blueprint maven plugin:
javax/annotation/PostConstruct is missing, see below:
Steps taken

clone repo
cd Karaf-Tutorial/tasklist-blueprint-cdi
mvn clean install

BOOM
[INFO] model .............................................. SUCCESS [  2.154 s]
[INFO] persistence ........................................ FAILURE [  0.309 s]
[INFO] ui ................................................. SKIPPED
[INFO] features ........................................... SKIPPED
[INFO] service ............................................ SKIPPED
[INFO] angular-ui ......................................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 4.122 s
[INFO] Finished at: 2017-05-03T07:47:39+02:00
[INFO] Final Memory: 25M/85M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.aries.blueprint:blueprint-maven-plugin:1.4.0:blueprint-generate (default) on project persistence: Execution default of goal org.apache.aries.blueprint:blueprint-maven-plugin:1.4.0:blueprint-generate failed: A required class was missing while executing org.apache.aries.blueprint:blueprint-maven-plugin:1.4.0:blueprint-generate: javax/annotation/PostConstruct
[ERROR] -----------------------------------------------------
[ERROR] realm =    plugin>org.apache.aries.blueprint:blueprint-maven-plugin:1.4.0
[ERROR] strategy = org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy

I tried the same and the build worked fine. What OS, java version and maven version do you use?

I updated the example to the newest versions of the dependencies. Maybe that could also help.

sorry, my bad, I inadvertently had java 9 set as my current JVM, it doesnt work there. It works with java 8 though.


Disable external IPs for GCP

we can use the GCP NAT gateway to enable egress traffic, see PR for bosh-bootloader: https://github.com/cloudfoundry/bosh-bootloader/pull/586

Thanks! @jochenehret


Consider adding scala support

Gradle supports Scala already through the Scala plugin (http://www.gradle.org/docs/current/userguide/scala_plugin.html).
Vaadin supports Scala through Scaladin (https://vaadin.com/directory#addon/scaladin) which hopefully soon will get Vaadin 7 support as well.
The gradle vaadin plugin could easily generate scala Vaadin application templates with a dedicated createVaadinScalaProject task as well as automatically include Scaladin as a dependency if the scala gradle plugin is used in the project.

I have a multi-module project. I have to define this in top-level build.gradle
    sourceSets.main.scala.srcDir "src/main/java"
    sourceSets.main.java.srcDirs = []

to be able to build both Java and Scala in the same project when Java code depends on Scala.
(as suggested here https://discuss.gradle.org/t/how-to-compile-a-java-class-that-depends-on-a-scala-class-in-gradle/6747 )
now, if I run "gradle build", I get

What went wrong:
Execution failed for task ':webui:vaadinUpdateWidgetset'.

No source sets was found.

my env:
vaadinVersion = '7.7.10'  // set in top-level build.gradle
and this is in my /webui/gradle.build file:

buildscript {
repositories {
maven {
url "https://plugins.gradle.org/m2/"
}
}
dependencies {
classpath "com.devsoap.plugin:gradle-vaadin-plugin:1.2.0.beta1"
}
}
apply plugin: 'war'
apply plugin: "com.devsoap.plugin.vaadin"

note - I used another (older)  plugin at first, which worked fine with Java (only):

plugins {
id 'fi.jasoft.plugin.vaadin' version '1.1.11'
}

now I'm trying to build this app with both Java+Scala and that plugin started failing, so I tried this newer one - com.devsoap.plugin.vaadin . but same error...
so, how can I use this plugin in a project

which has BOTH Java and Scala
AND where Java code depends on Scala
?

solved this by adding this:
vaadin.mainSourceSet = sourceSets.main.allSource
vaadin {
    version = "$vaadinVersion"
}


Update the generate azure sdk bom script.

Description
Update the bomgenerator script to sync the version_client.txt. As of now there is a gap where if a release is done from a non-main branch, the version_client.txt is not updated to get the correct version (this is understandable, due to potential merge conflicts). Adding a script that will sync things from maven and get the correct information.
As of now the script does not add new SDKs to the list but updates the existing ones, which is likely where we would be releasing from the non-main branch.

/check-enforcer override


[Feature][style] Add spotless maven plugin for automatic style fix.

Purpose of the pull request

Add spotless maven plugin for automatic style fix.
This PR closes: #10963

Brief change log

Add spotless maven plugin in pom.xml.
Add eclipse style check for spotless plugin.
Spotless only checks and fixes formatting errors for difference with HEAD.

Verify this pull request

Verified by manual tests.

Actually we could keep both Spotless and Checkstyle as long as we make the requirements of those two compatible. We will use Spotless for fixing style errors automatically and use Checkstyle in CI for double-check.

Actually we could keep both Spotless and Checkstyle as long as we make the requirements of those two compatible. We will use Spotless for fixing style errors automatically and use Checkstyle in CI for double-check.

AFAIK. Apache ShardingSphere is using this method, FYI:
https://github.com/apache/shardingsphere/blob/19265081c966cb0a839267b060593b258b0883b4/pom.xml#L1001-L1045
https://github.com/apache/shardingsphere/tree/master/src/resources
Also here is a related instructions written in Chinese:
https://shardingsphere.apache.org/blog/cn/material/spotless/

Codecov Report

Merging #11272 (b79eeb2) into dev (78e9122) will decrease coverage by 0.00%.
The diff coverage is n/a.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Can we set the import order? And please change the related doc.

I've tested locally and it seems it does check the import order and fix it automatically, although I have figured it out how it does it. Anyway I will double check it. Thx

I will update the related docs in the following commits of this PR.

Can we set the import order? And please change the related doc.

I've tested locally and it seems it does check the import order and fix it automatically, although I haven't figured it out how it does it. Anyway I will double check it. Thx

I think spotless by default sorts the imports in alphabetical order unless we want to sort in another order we don't need extra configuration maybe

Can we set the import order? And please change the related doc.

I've tested locally and it seems it does check the import order and fix it automatically, although I haven't figured it out how it does it. Anyway I will double check it. Thx

I think spotless by default sorts the imports in alphabetical order unless we want to sort in another order we don't need extra configuration maybe

Ok, in fact, we have our own order of imports, but it's ok for me if we change to alphabetical order.

For developer's convenience, I suggest setting Spotless executions configuration as follows:
<executions>
  <execution>
      <goals>
          <goal>apply</goal>
      </goals>
      <phase>compile</phase>
  </execution>
</executions>

In this way, Spotless will automatically fix the style / formatting issues when compiling the project. However, we need to keep Checkstyle so that CI could make sure the style is correct in case developers do not compile and test the code before pushing it to remote repo.
Another way is to use git pre commit and remove Checkstyle. WDYT @kezhenxu94 @ruanwenjun

For developer's convenience, I suggest setting Spotless executions configuration as follows:

<executions>

  <execution>

      <goals>

          <goal>apply</goal>

      </goals>

      <phase>compile</phase>

  </execution>

</executions>

In this way, Spotless will automatically fix the style / formatting issues when compiling the project. However, we need to keep Checkstyle so that CI could make sure the style is correct in case developers do not compile and test the code before pushing it to remote repo.
Another way is to use git pre-commit hook and remove Checkstyle. WDYT @kezhenxu94 @ruanwenjun

I personally used pre-commit hook in my own project and I prefer to use this. Also even we have the hook we should still keep the style check in CI because that can be skipped in developers' local machine.

For developer's convenience, I suggest setting Spotless executions configuration as follows:

<executions>

  <execution>

      <goals>

          <goal>apply</goal>

      </goals>

      <phase>compile</phase>

  </execution>

</executions>

Also, developers might skip running maven compile if they develop in an IDE (mostly)

For developer's convenience, I suggest setting Spotless executions configuration as follows:
<executions>
  <execution>
      <goals>
          <goal>apply</goal>
      </goals>
      <phase>compile</phase>
  </execution>
</executions>

In this way, Spotless will automatically fix the style / formatting issues when compiling the project. However, we need to keep Checkstyle so that CI could make sure the style is correct in case developers do not compile and test the code before pushing it to remote repo.
Another way is to use git pre-commit hook and remove Checkstyle. WDYT @kezhenxu94 @ruanwenjun

I agree with removing Checkstyle in ci, if we keep the Checkstyle we may need to maintain the style is the same in Spotless and Checkstyle.

For developer's convenience, I suggest setting Spotless executions configuration as follows:
<executions>
  <execution>
      <goals>
          <goal>apply</goal>
      </goals>
      <phase>compile</phase>
  </execution>
</executions>

In this way, Spotless will automatically fix the style / formatting issues when compiling the project. However, we need to keep Checkstyle so that CI could make sure the style is correct in case developers do not compile and test the code before pushing it to remote repo.
Another way is to use git pre-commit hook and remove Checkstyle. WDYT @kezhenxu94 @ruanwenjun

I agree with removing Checkstyle in ci we need to use Spotless to check if the style is correct, if we keep the Checkstyle we may need to maintain the style is the same in Spotless and Checkstyle.

Also even we have the hook we should still keep the style check in CI because that can be skipped in developers' local machine.

I mean we should check the code style with spotless in ci

For developer's convenience, I suggest setting Spotless executions configuration as follows:
<executions>

  <execution>

      <goals>

          <goal>apply</goal>

      </goals>

      <phase>compile</phase>

  </execution>

</executions>

In this way, Spotless will automatically fix the style / formatting issues when compiling the project. However, we need to keep Checkstyle so that CI could make sure the style is correct in case developers do not compile and test the code before pushing it to remote repo.
Another way is to use git pre-commit hook and remove Checkstyle. WDYT @kezhenxu94 @ruanwenjun

I personally used pre-commit hook in my own project and I prefer to use this. Also even we have the hook we should still keep the style check in CI because that can be skipped in developers' local machine.

I think it is a good point that we should check the style in CI in case users skip it.
To recap:

Remove Checkstyle and only keep Spotless so that we will not need to maintain two style check files.
Change the execution goal of Spotless from apply to check so that CI will check the style with Spotless.
Encourage users to use git pre-commit hook.

May I ask there is any best practice for setting up git pre-commit hook? Should we provide users with a script to set it up or just give them a doc? WDYT @kezhenxu94 @ruanwenjun

We can add a pre-commit script and  add a doc to ask developers to copy it into .git/hooks folder

We can add a pre-commit script and add a doc to ask developers to copy it into .git/hooks folder

Sure, that sounds good to me. I will add it in the next commit.

Please take another look when available, thx~ : ) @kezhenxu94 @ruanwenjun

Can you please also update this ?
https://github.com/apache/dolphinscheduler/blob/8ded072f799376d9fadb669cb990edd2197d1062/.github/actions/sanity-check/action.yml#L34-L53

Sure, just for confirmation, I'm supposed to update this with Spotless related stuff instead of simply removing it, correct?

Can you please also update this ?

https://github.com/apache/dolphinscheduler/blob/8ded072f799376d9fadb669cb990edd2197d1062/.github/actions/sanity-check/action.yml#L34-L53

Sure, just for confirmation, I'm supposed to update this with Spotless related stuff instead of simply removing it, correct?

Yes right


can't contact ldap server on production server, works fine on Test server

Laravel Version: 8.26.1
Adldap2-Laravel Version: 10.3.1
PHP Version: 7.3.5
LDAP Type: OpenLDAP

I deploy my laravel code on 2 similar servers (red hat 8 with apache server). One is for tests the other one is for the production.
I have no trouble on the testing server, my laravel code with adldap2 works fine, but I have the error "Can't contact ldap server" on the production server:

local.DEBUG: AuthenticatesUsers::attemptLogin {"BindException":"[object] (Adldap\Auth\BindException(code: -1): Can't contact LDAP server at /data/Myproject/vendor/adldap2/adldap2/src/Auth/Guard.php:107)

On both servers I succeed to connect to my ldap through command line with this kind of command:

ldapsearch -x -H ldap://myldap.com -D "LDAP_USERNAME" -W -b "my LDAP_BASE_DN" cn

My .env file is the same on both servers. I'll try with SSL or TLS without success (the both protocols works fine on my testing server)

The ldap.conf file are the same on both servers.

I clear all artisan caches on both servers in order to be sure that last parameters from .env file are set:

php artisan config:cache
php artisan config:clear
php artisan cache:clear

I create a little php script to check ldap connection through php command and it's works (Successfully connected to LDAP !) on the both server too:

$ldapconn = ldap_connect("ldap://myldap.com")
or die("Impossible to connect to the LDAP server.");
$ldapbind = ldap_bind($ldapconn, 'LDAP_USERNAME', 'LDAP_PASSWORD');
// check binding
if ($ldapbind) {
echo "Successfully connected to LDAP !" . PHP_EOL;
$dn = 'LDAP_BASE_DN';
$sr = ldap_list($ldapconn, $dn, 'cn=*');
if (false === $sr) {
die('Impossible to use the dn: ' . $dn . PHP_EOL);
}
$info = ldap_get_entries($ldapconn, $sr);
if ($info['count'] === 0) die('No entries :(');
for ($i=0; $i < $info["count"]; $i++) {
echo $info[$i]["cn"][0] . PHP_EOL;
}
} else {
$var = '';
ldap_get_option($ldapconn, LDAP_OPT_ERROR_STRING, $var);
echo "Connection to LDAP failed..." . PHP_EOL . $var . PHP_EOL;
}

what other points between the 2 servers I could check to solve my problem?

switch to Adldap2 page: https://github.com/Adldap2/Adldap2/issues/790


Update Meta “tags”

Automatically generated by Netlify CMS

👷 Deploy Preview for jchendriks-s5 processing.
🔨 Explore the source changes: 2e029fdabc6d0e28fea5b16dcbe15defb47981c3
🔍 Inspect the deploy log: https://app.netlify.com/sites/jchendriks-s5/deploys/61d441cae810b10007095d39


FRQ # 1 Feedback

Haoxuan's FRQ's

A. 1 point for each FRQ. 1.1 or .9 basis for above or below the line per FRQ. Correct complete and running in Jupyter Notebook meets the basis.
Ran all code cells and verified outputs
Grading based on 0.9 basis
FRQ #1: 0.9/0.9
I saw all the code and it was running correctly, and it had all the correct outputs that needed to be there
FRQ #2: 0.9/0.9
I saw the code here and it works and everything was running correctly as I saw all 5 instances of test data
FRQ #3: 0.85/0.9
You had outputs for each one, but I think that part a was a little bit too modified, but it was nice that you were able to simplify it down completely
FRQ #4: 0.9/0.9
In this FRQ, for part a you initialized a numList which wasn't necessary, but according to CollegeBoard's rubric you would get full points. For parts b and c, I think you did pretty good and had the correct outputs.
B. Please assist with FRQ association issue. Read or review a couple of my comments, including Vishnu comments to qualify you to grade. 4 or 3.6 is basis.
3.6/3.6
This was quite the impressive connection to PBL. I thought the same thing after reading FRQ 2, I was like hey this is just like that popular game wordle. So it was interesting how you decided to base a project around it, I mean the code is already done, all you need are some changes on spring, I also think a good addition, something that Vishnu did is add the topics you struggled with and make it a priority in order to ensure you understand these topics before the AP Exam
Notable Mentions
I really liked seeing how Haoxuan tackled this problem because it seemed really cool, and it was really nice and easy, and much more simpler than my code. I also liked that he implemented char, which I did not but should have. I also liked his comment about how it was just Wordle but with OOP. It's good he converted the strings to chars, I think this was a unique way to approach it.

I also liked how he was able to simplify the constructors given by CollegeBoard in FRQ #3, I think that it was really cool and that he was still able to produce an output.


Support different casing

First and foremost, thanks a lot for your surely hard work on this project! Greatly appreciated.
For almost all databases the columns naming convention is snake_case, but with MapDataReader it won't map back to the entity if it's in C#'s naming convention - PascalCase.
What do you think about changing GetDeterministicHashCode behavior to remove underscores when looking for a match in addition to ignore casing differences?
https://github.com/jitbit/MapDataReader/blob/1948ca25dea3ad8adde6b7ac07e9e430eed94486/MapDataReader/MapperGenerator.cs#L116-L134
Maybe it can be a flag on GenerateDataReaderMapperAttribute?

@alex-jitbit
Update: Personally for me, I went a different route because how horrible snowflake's SDK for dotnet is.
I had to do many modifications that required a lot of manual casting and serialization.
This FR is not important for me anymore, but I believe for others it 100% would.
Thanks a lot of the library, looks awesome❣

Currently the workaround would be to either use DTO class or SQL aliases
SELECT my_colymn AS MyProperty
I'll consider make it case-insensitive

Actually, it is already case-insensitive, fixed in d59601e1763c43786346e5b38162b3ccc1717982
As for the "removing underscores" - I believe it would be an easter egg. Try using aliases in your query like described above

Unforturently I couldn't use a different DTO or aliases to my queries because I can hot-swap the database on request, so exactly the same SQL query should behave no matter the database.
Only for the snowflake provider I had to go down the datareader level.
And since the interface is simply List<T> GetData(string sql), I can't really use a different DTO either.
Like I updated earlier for me, it's not that important anymore as we took a different path, but I believe it's a very important feature to have, so I wouldn't close as completed but your call.
Cheers mate!


ngx-monaco-editor support for Angular 11

Hi. I would like to know if ngx-monaco-editor supports Angular 11? This version of Angular is not mentioned in the readme and it runs into errors when I install the npm package.
Angular version: 11.2.14
OS: Windows 10
Repro steps:

Install Angular version 11.2.14
Add your first angular project
npm install monaco-editor ngx-monaco-editor --save
Add { "glob": "**/*", "input": "node_modules/monaco-editor", "output": "assets/monaco-editor" } to assets array in angular.json file.
Add MonacoEditorModule.forRoot() to imports array in app.module.ts
Run ng s

The error:
Error: ../node_modules/ngx-monaco-editor/lib/base-editor.d.ts:16:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵFactoryDeclaration'.

16     static ɵfac: i0.ɵɵFactoryDeclaration<BaseEditor, never>;
                       ~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/base-editor.d.ts:17:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵComponentDeclaration'.

17     static ɵcmp: i0.ɵɵComponentDeclaration<BaseEditor, "ng-component", never, {}, { "onInit": "onInit"; }, never, never>;
                       ~~~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/diff-editor.component.d.ts:15:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵFactoryDeclaration'.

15     static ɵfac: i0.ɵɵFactoryDeclaration<DiffEditorComponent, never>;
                       ~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/diff-editor.component.d.ts:16:21 - error TS2694: 0mNamespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵComponentDeclaration'.

16     static ɵcmp: i0.ɵɵComponentDeclaration<DiffEditorComponent, "ngx-monaco-diff-editor", never, { "options": "options"; "originalModel": "originalModel"; "modifiedModel": "modifiedModel"; }, {}, never, never>;
                       ~~~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/editor.component.d.ts:21:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵFactoryDeclaration'.

21     static ɵfac: i0.ɵɵFactoryDeclaration<EditorComponent, never>;
                       ~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/editor.component.d.ts:22:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵComponentDeclaration'.

22     static ɵcmp: i0.ɵɵComponentDeclaration<EditorComponent, "ngx-monaco-editor", never, { "options": "options"; "model": "model"; }, {}, never, never>;
                       ~~~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/editor.module.d.ts:10:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵNgModuleDeclaration'.

10     static ɵmod: i0.ɵɵNgModuleDeclaration<MonacoEditorModule, [typeof i1.EditorComponent, typeof i2.DiffEditorComponent], [typeof i3.CommonModule], [typeof i1.EditorComponent, typeof i2.DiffEditorComponent]>;
                       ~~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/editor.module.d.ts:11:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵInjectorDeclaration'.

11     static ɵinj: i0.ɵɵInjectorDeclaration<MonacoEditorModule>;
                       ~~~~~~~~~~~~~~~~~~~~~

Error: ../node_modules/ngx-monaco-editor/lib/editor.module.d.ts:9:21 - error TS2694: Namespace '"C:/Users/KOVLTP000/Deepak/Angular Apps test/monaco/example-app/node_modules/@angular/core/core"' has no exported member 'ɵɵFactoryDeclaration'.

9     static ɵfac: i0.ɵɵFactoryDeclaration<MonacoEditorModule, never>;
                      ~~~~~~~~~~~~~~~~~~~~

Error: app/app.component.html:335:50 - error NG8002: Can't bind to 'ngModel' since it isn't a known property of 'ngx-monaco-editor'.
1. If 'ngx-monaco-editor' is an Angular component and it has 'ngModel' input, then verify that it is part of this module.
2. If 'ngx-monaco-editor' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.
3. To allow any property add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component.

335     <ngx-monaco-editor [options]="editorOptions" [(ngModel)]="code"></ngx-monaco-editor>
                                                     ~~~~~~~~~~~~~~~~~~

  app/app.component.ts:5:16
    5   templateUrl: './app.component.html',
                     ~~~~~~~~~~~~~~~~~~~~~~
    Error occurs in the template of component AppComponent.

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 87:10-28
"export 'ɵɵFactoryTarget' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 253:10-28
"export 'ɵɵFactoryTarget' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 406:10-28
"export 'ɵɵFactoryTarget' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 494:10-28
"export 'ɵɵFactoryTarget' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 111:0-27
"export 'ɵɵngDeclareClassMetadata' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 277:0-27
"export 'ɵɵngDeclareClassMetadata' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 427:0-27
"export 'ɵɵngDeclareClassMetadata' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 512:0-27
"export 'ɵɵngDeclareClassMetadata' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 79:18-39
"export 'ɵɵngDeclareFactory' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 243:23-44
"export 'ɵɵngDeclareFactory' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 398:27-48
"export 'ɵɵngDeclareFactory' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 488:26-47
"export 'ɵɵngDeclareFactory' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 505:26-48
"export 'ɵɵngDeclareInjector' (imported as 'i0') was not found in '@angular/core'

Error: ./node_modules/ngx-monaco-editor/fesm2015/ngx-monaco-editor.js 496:26-48
"export 'ɵɵngDeclareNgModule' (imported as 'i0') was not found in '@angular/core'

I have the same problem in Angular 11 :-(
Someone have an idea how to fix this problem?

same issue

Using<EMAIL_ADDRESS>worked for my Angular 11 Project.

@here I've a new fork with latest angular 14, ngZone upgrade and fixed production missing assets, have a look, works for my company now:
https://www.npmjs.com/package/ngx-monaco-editor-v2

I am also facing the same issue for angular 11, Let me know if anyone has solution for this.

face the same issue


Benchmarks - Keep BatchNorm as fp32 for pytorch cnn models cast to fp16

Description
The BatchNorm operator is not numerically stable in fp16.  PyTorch documentation recommends to keep the BN op in fp32 for fp16 AMP models.  Refer to https://pytorch.org/docs/stable/amp.html#ops-that-can-autocast-to-float32.  Preserving BN in fp32 for superbench more accurately reflects real workloads.

/azp run

/azp run


Fixed green reticle issue and corrected Target Assist

Fixed green reticle issue and corrected Target Assist IDX
Closes #392
Additions/modifications proposed in this pull request:

Fixed issues with sending Targets to the Client (fixing #392)
Set m_assist_target_idx server side since the client never sends us this information

Outstanding:

TODO: Outstanding issue with /assist because target is overwritten by client to old value

I meant to post this here...
Unfortunately @nemerle, our theory about unqueue_all did not resolve the /assist issue. It didn't have any impact.
In fact, unqueue_all only fires when switching target, so maybe we do want to reset m_target_idx and m_assist_target_idx there? I'll leave them for now. Unless you think it's better to comment them out.

Since this PR resolves the original targeting issue #411  I'm going to go ahead and merge this, and open a new issue for the /assist problem. Especially since without any sort of powers, using /assist is really low priority anyway.


 Command is like ./swarm -l "debug" manage -H <IP_ADDRESS>:2372 consul://<IP_ADDRESS>:8500/swarm. You should be able to see logs on your load command like
DEBU[7267] HTTP request received                         method=POST uri=/v1.21/images/load

As you may know it is not trivial to put anything before manage with Docker Machine and I'm too lazy to do that manually.
However, some more info:

error doesn't happen with 1 node (which is master by coincidence in Docker Machine-created Swarm)
I see nice long stacktrace in logs of master container (it is too long to place it here): http://pastebin.com/a7gQri6y

@nazar-pc I think your suspicion of memory allocation failure is right. The swarm load operation creates multiple writers and copies content from reader to writers. The master with both reader and a writer may run out of memory when the image size is close to node's memory. I can trigger cmalloc failure by controlling memory limit on master node. (Docker load is implemented differently from swarm.)
Also it's recommended not to put agent on swarm master node to avoid resource contention.

Thanks for confirmation.
I'd be happy to separate them, but it is not currently possible just that easily with Docker Machine (https://github.com/docker/machine/issues/2302) and this is just my laptop for experiments, not production setup.
In any case it would be nice if Swarm can detect this issue and handle it somehow without being crashed.

Related: https://github.com/docker/swarm/issues/1163
Maybe using the registry is a good workaround

Yes, according to description looks like this is a duplicate of mentioned issue.
I'm closing this, hopefully issue will be resolved soon.


Translate texts

Hi,
How can I change the language of the texts to pt-br, for example?

@lucas-marciano  You will have to add the translated strings the corresponding resource qualifier folders . values-pt-br in your case. I would be glad if you could translate the strings to your language of choice . I will merge it for the community's good

Tks man, i do it.

@lucas-marciano Did u do it ? Can i have look (example) that how did you use your own language! Thankx

@deshario I overwritten the library strings inside in my own string.xml file, see in the code what are the names.


Problem with error message translations if use valibot in form

Environment

Node Version:     v20.11.1
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      telemetry, ssr, app, spaLoadingTemplate, extends, css, modules, ui, i18n, devtools, devServer
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Version
v2.16.0
Reproduction
https://stackblitz.com/edit/nuxt-starter-cf4613?file=app.vue
Description
Hello! 👋  Thanks for package!
I very likes use Nuxt Ui / Pro in my projects !
I use valibot in forms and set globally translation. ( please see reproduction )
Expected behavior: after set globally translation language in valibot, issues message shows in set globally lang
Actual behavior: issues message shows in default language "en"
Under the hood we used ._parse method
#https://github.com/nuxt/ui/blob/main/src/runtime/components/forms/Form.vue
Please see string 263.
fabian-hiller says:

._parse is an internal API and should not be used directly. If you parse your schema with parse or safeParse, it should work as expected: https://valibot.dev/guides/parse-data/

For this reason we don't get messages with current translation.
Issue in valibot repo: https://github.com/fabian-hiller/valibot/issues/576
Please help with this problem, or if need i can try solve this problem and send new pull request!
Additional context
No response
Logs
No response

Tagging  @fabian-hiller for feedback

Short explanation: Valibot captures the configuration when calling parse and safeParse and then passes it to ._parse. Since ._parse is called directly, the global configuration is probably ignored. I am happy to help with any questions.

This should be fixed by https://github.com/nuxt/ui/pull/1848 and wrapping your schema in v.safeParser.


Speed up geometry validation

This partially addresses the slow validation here: https://github.com/gmggroup/omf/issues/43

Codecov Report

Merging #44 into dev will decrease coverage by 0.2%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev      #44      +/-   ##
==========================================
- Coverage   85.56%   85.35%   -0.21%
==========================================
  Files          10       10
  Lines         478      478
==========================================
- Hits          409      408       -1
- Misses         69       70       +1

Impacted Files
Coverage Δ

omf/surface.py
90.74% <100%> (ø)
:arrow_up:

omf/lineset.py
93.1% <100%> (ø)
:arrow_up:

omf/pointset.py
95.23% <100%> (ø)
:arrow_up:

omf/data.py
74.41% <0%> (-1.17%)
:arrow_down:

Continue to review full report at Codecov.

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


hsphfpd: 2020-10-25 -> 2020-11-27

Motivation for this change
Update.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 105807 run on x86_64-linux 1

1 package built:

hsphfpd