Inconsistent contacts information between mjx and mujoco when the geoms are a capsule and a box.

Hi, I've found an inconsistency with the data.contact list between mujoco and mjx. I'm wondering if I've stumbled upon a bug or I've did something wrong.
Here's a minimal example that reproduces the problem:
import mujoco
from mujoco import mjx

XML=r"""
<mujoco>
<option>
  <flag gravity="disable"/>
</option>

<worldbody>
  <body pos="0 0 0">
    <freejoint/>
    <geom type="box" size=".1 .1 .1" name="box" rgba="1 1 1 0.25"/>
    <site pos="0 0.02500001 0.85" size="0.025" rgba="1 0 0 1"/>
    <site pos="0 0.02500001 1.15" size="0.025" rgba="1 0 0 1"/>
  </body>

  <body pos="0 0 1">
    <freejoint/>
    <geom type="capsule" size=".15 .15" name="capsule" rgba="1 1 1 0.25"/>
  </body>
</worldbody>

<contact>
  <pair geom1="box" geom2="capsule" margin="100"/>
</contact>
</mujoco>
"""

mj_model = mujoco.MjModel.from_xml_string(XML)
model = mjx.device_put(mj_model)
data = mjx.make_data(model)
data = mjx.forward(model, data)

print("mjx:")
print(f"contact.dist: {data.contact.dist}")
print(f"contact.pos: {data.contact.pos}")

mj_data = mujoco.MjData(mj_model)
mujoco.mj_forward(mj_model, mj_data)

print("mujoco:")
print(f"contact.dist: {mj_data.contact.dist}")
print(f"contact.pos: {mj_data.contact.pos}")

And here's the output of the above code:
mjx:
contact.dist: [1. 1.]
contact.pos: [[0.         0.02500001 0.85      ]
 [0.         0.02500001 1.15      ]]
mujoco:
contact.dist: [0.6 0.9]
contact.pos: [[0.   0.   0.4 ]
 [0.   0.   0.55]]

The outputs I get show different contact distances and positions between mjx and mujoco. Here are some screenshots showing the contact positions in red dots, left is mjx and right is mujoco:

Also, why does it return two contacts with different positions? It seems to me that the actual midpoint is [0, 0, 0.4], why does it return another midpoint with a different position?
Context:

Operating system: Ubuntu 22.04
Mujoco version: mujoco-mjx-3.1.2

I can answer the second question. Your geoms are so large with the margin, that the box-capsule collider is colliding the capsule as if it is two spheres (imagine a capsule embedded in a box, these collisions are what you want), therefore the two points returned by MuJoCo are the collision points for the two spheres. @btaba to comment about MJX collisions.

Thanks @yuvaltassa! I've just stumbled upon https://github.com/google-deepmind/mujoco/issues/1325 which I think answers my first question.
To follow up on my second question, why is it that if I only move the capsule along the x or y-axis, it'll return only one contact position as opposed to two?

If I assume that it'll return two different contacts most of the time, and I want to know the contact distance between those two geoms during simulation, would it be right for me to just take the contact with the smaller distance?

The images above are for MJX or MuJoCo? I'm not sure I know what you're asking without being able to reproduce.

If I assume that it'll return two different contacts most of the time, and I want to know the contact distance between those two geoms during simulation, would it be right for me to just take the contact with the smaller distance?

Yes

Hi @Manchewable , thanks for the bug report. It turns out margin wasn't really implemented for capsule-box, and so we just pushed out a change to raise an error if margin is present for those collisions in MJX.


Test-NetConnection cmdlet is missing from PowerShell Core 6.2.3 release

Get-Command Test-NetConnection
Get-Command : The term 'Test-NetConnection' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

Get-Command Test-NetConnection

CategoryInfo          : ObjectNotFound: (Test-NetConnection:String) [Get-Command], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException,Microsoft.PowerShell.Commands.GetCommandCommand

@sidsolutions-net the Test-NetConnection command is only available from the Windows PowerShell NetTCPIP module. You'll need to have the Windows PowerShell module loaded to be able to use the command. You may need to make sure your Windows PowerShell module locations are present in $env:PSModulePath to be able to load the module by name, otherwise you can always load it directly by the full path.

In latest builds you can load Core incompatible modules in background Windows PowerShell session (#10973):
Import-Module -UseWindowsPowerShell NetTCPIP

I believe NetTCPIP has now been ported as well, but you'll need a recent Windows edition:

Yes, I see
    CompatiblePSEditions = @('Desktop', 'Core')
``
in its psd1 file.

Hi,
Can you recheck again for 7.1.1? I intend to use ps core for linux but there's no entire NetTCPIP module at all
Name                           Value
----                           -----
PSVersion                      7.1.1
PSEdition                      Core
GitCommitId                    7.1.1
OS                             Linux 5.4.0-1031-azure #32-Ubuntu SMP Tue Oct 6 09:47:33 UTC 2020
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Hi,
Can you recheck again for 7.1.1? I intend to use ps core for linux but there's no entire NetTCPIP module at all
Name                           Value
----                           -----
PSVersion                      7.1.1
PSEdition                      Core
GitCommitId                    7.1.1
OS                             Linux 5.4.0-1031-azure #32-Ubuntu SMP Tue Oct 6 09:47:33 UTC 2020
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The NetTCPIP is a Windows module, it doesn't exist for Linux, and likely will not be able to be ported as it uses Windows-OS-specific APIs for its functionality.
You can use the pwsh-native Test-Connection cmdlet on any OS pwsh is available on. However, I will note that .NET Core / .NET 5 support for parts of the functionality it implements is rather lacking in places, so depending on your use case it may be more desirable to use a linux-native ping application.

The NetTCPIP is a Windows module, it doesn't exist for Linux, and likely will not be able to be ported as it uses Windows-OS-specific APIs for its functionality.
You can use the pwsh-native Test-Connection cmdlet on any OS pwsh is available on. However, I will note that .NET Core / .NET 5 support for parts of the functionality it implements is rather lacking in places, so depending on your use case it may be more desirable to use a linux-native ping application.


Setting default FFT window to Hanning

Issue #291 Have set the default FFT window to Hanning for calculation of all summary and spectral indices.
However left the Test methods using HAMMING window so do not have to make changes to the output.
Title of PR
What is the purpose of this PR?
Changes
List of changes added as part of this PR. This only needs to include the big changes which reviewers should carefully check.
Issues
Any issues caused by the PR that will need to be addressed in another PR.
These issues may be caused by another branch which will change how this branch functions.
Visual Changes
If the PR has any visual changes to the website, post pictures of the new pages and how they look.
Label any images if more than one is given.
Final Checklist

[ ] Assign reviewers if you have permission
[ ] Assign labels if you have permission
[ ] Link issues related to PR
[ ] Link any PRs or issues blocking this PR from being merged
[ ] Remove/Reduce warnings from edited files
[ ] Unit tests have been added for changes
[ ] Ensure CI build is passing

@towsey this introduced multiple test failures. See https://dev.azure.com/QutEcoacoustics/audio-analysis/_build/results?buildId=2193&view=ms.vss-test-web.build-test-results-tab
While you're fIxing those tests, maybe consider how we could make them less fragile?

Hi Anthony,
Clicking the link you provided does not take me to the failed test info.
Suggestion?
Dr. Michael Towsey
ORCID iD http://orcid.org/0000-0002-8246-7151
QUT Ecoacoustics Research Group  |  www.Ecosounds.orghttp://www.ecosounds.org/
Science and Engineering Faculty
S block, Level 10, Room 1002, Garden Point Campus,
Queensland University of Technology,
Brisbane, Queensland 4001
Australia
Phone +61 7 3138 9381
Fax     +61 7 3138 9390

From: Anthony Truskinger<EMAIL_ADDRESS>Sent: 30 July 2020 5:37 PM
To: QutEcoacoustics/audio-analysis<EMAIL_ADDRESS>Cc: Michael Towsey<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [QutEcoacoustics/audio-analysis] Setting default FFT window to Hanning (#349)
@towseyhttps://urldefense.com/v3/__https://github.com/towsey__;!!NVzLfOphnbDXSw!Uu97CcNOgzKo7R-SHlqfIH18ypLJ9g3C2XsJmRrmS8Y9i05F5_yUFbcXlfQCNhlqvjA$ this introduced multiple test failures. See https://dev.azure.com/QutEcoacoustics/audio-analysis/_build/results?buildId=2193&view=ms.vss-test-web.build-test-results-tabhttps://urldefense.com/v3/__https://dev.azure.com/QutEcoacoustics/audio-analysis/_build/results?buildId=2193&view=ms.vss-test-web.build-test-results-tab__;!!NVzLfOphnbDXSw!Uu97CcNOgzKo7R-SHlqfIH18ypLJ9g3C2XsJmRrmS8Y9i05F5_yUFbcXlfQC9xvnj6k$
While you're fIxing those tests, maybe consider how we could make them less fragile?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/QutEcoacoustics/audio-analysis/pull/349*issuecomment-666192993__;Iw!!NVzLfOphnbDXSw!Uu97CcNOgzKo7R-SHlqfIH18ypLJ9g3C2XsJmRrmS8Y9i05F5_yUFbcXlfQCMtaX-yc$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABXG7AYMSVINFVJXIUJS2QTR6EPMPANCNFSM4PMQIDLA__;!!NVzLfOphnbDXSw!Uu97CcNOgzKo7R-SHlqfIH18ypLJ9g3C2XsJmRrmS8Y9i05F5_yUFbcXlfQCSerk70o$.

https://dev.azure.com/QutEcoacoustics/audio-analysis/_build/results?buildId=2192&view=ms.vss-test-web.build-test-results-tab
It looks like the build number changed 🤷♂. The link above works


[FEATURE] "Now" Page standard

Now page
Most websites have a link that says “about”. It goes to a page that tells you something about the background of this person or business. For short, people just call it an “about page”.
Most websites have a link that says “contact”. It goes to a page that tells you how to contact this person or business. For short, people just call it a “contact page”.
So a website with a link that says “now” goes to a page that tells you what this person is focused on at this point in their life. For short, we call it a “now page”.
It was coined by Derek Sivers here

Examples of Now pages:
https://8-p.info/now/
https://sive.rs/now
https://marcjenkins.co.uk/now/
https://www.notkindacool.com/now/
https://garrickvanburen.com/now/

There's an aggregator website for now pages (showing widespread adoption).

if it's necessary, you can create a page for Now Page in content/now/index.md. Then add the link on the menu by using this frontmatter
  [[menu.main]]
    identifier = "Now"
    name = "Now"
    url = "/now/"
    weight = 3 #depends on your usages

Closed because now-page only contains heading and sentences, no need to create a different style. And using the _default/single.html is enough to display the now-page.

you can create a page for Now Page in content/now/index.md.


Simplify how the client is imported

Instead of var SpanContext = require('jaeger-client/dist/src/span_context.js').default; there should be a simpler way, e.g. require('jaeger-client').SpanContext

fixed by https://github.com/uber/jaeger-client-node/pull/37

Please add example to README


Pyrealsense2   How to Adjust temporal filter parameter

| Camera Model                       |  D400 } |
| Firmware Version                   |<IP_ADDRESS> |
| Operating System & Version |   Linux (Ubuntu 16/17) / MacOS  |
| Kernel Version (Linux Only)    |   4.15                               |
| Platform                                 | PC |
| SDK Version                            |  2.28.0                          |
| Language                            | python                       |
| Segment			|  {others }                          |

When tried to estimate the real XYZ coordinates of of corner points of a detected object ( using SIFT algorithm for feature identification) and  plotted XYZ data  of corner points , It is not forming rectangle. It continuously oscillating.  Is it due to unstable depth frame. I applied temporal filter again not improving. If the problem is with temporal filter how to change l filter parameters like alpha , Delta  in python

same way as in C++. You can either pass these parameters into the constructor for the temporal filter, or you can use the set_option API

Thank you for the answer .  I will try


Windows issue: Images in pop-up windows have the wrong dimensions

Describe the bug
Images in pop-up windows in AfterScan (custom template, crop frame) are displayed with the wrong aspect ratio. In Ubuntu the aspect ratio is the correct one.
To Reproduce
Steps to reproduce the behavior:

Launch AfterScan
Select a source folder
Click on 'Custom hole template' or 'Define crop area'
Image has the wrong aspect ratio:

Expected behavior
Image in pop-up window should have the correct aspect ratio:

Desktop (please complete the following information):

OS: Windows
Version: 11

Fixed in Afterscan 1.8.12.OpenCV handle differently windows dimensions and the image within. Therefore, in order to fix this one, the extra width added to the window displaying the custom hole template (to make the window more visible) has to be removed.


Open link in new tab

Right click to open a link in a new tab used to put that tab at the top of the list. Now it puts it at the very bottom. Not sure if this is a bug or a feature, but I prefer the previous behavior.

Right clicking on a link opens it in a foreground tab, i.e. focuses the newly opened tab, right?
If that's the case, the mentioned behavior is unexpected, since by design only background tabs are put at the end of the list, because you haven't visited them yet.


Question about ASP Resolution

Hi there, I am curious about the highest achievable vertical resolution with Stereo Pipeline. Say I have two images with 1 meter/pixel resolution in lat lon, does that mean that the height resolution is also 1 meter? Or something less?
Thanks!

The vertical resolution is very dependent on the convergence angle. As a rule of thumb, if you have a convergence angle of say 30-60 degrees, a vertical resolution of 2-3 meters looks achievable, given an input horizontal resolution of 1 m. With ASP it is suggested, for best results, to mapproject at 1 m/pixel in a local projection then use the asp_mgm algorithm.

@oleg-alexandrov Do know of a paper/reference that discusses the mathematics of this? I am interested in providing more detailed metadata for ASP generated DTMs and since one knows all of the view geometry, it should be possible to estimate the vertical resolution (I think?).
Thanks!

The earliest reference I can find that talks about expected vertical precision is Cook et al. (1996, section 3.2), but I feel like this is probably discussed before 1996, but maybe not?  Randy also uses this formulation in Kirk et al. (2003, para 38).  And purveyors of terrain models have used it since.
However, this is really a predictive measure: what is expected?  Frequently, the actual data will result in vertical precisions that can vary from a value computed this way.  The recent Bland et al. (2021) talks about applying this and how expected vertical precision is kidn of only one relevant metric.

@rbeyer Thanks for the Kirk reference and I had totally forgotten about Mike's paper. I'll be re-reading... :) Much appreciated


Reading the multiple characteristic with the same UUID from the same service

Hi,
I am not sure if this is a bug issue or something I am doing wrong.
I have a BLE device with WIFI module(WINC3400).
I am able to perform a WIFI scan by writing to the corresponding Characteristics, but the results are read from particular Characteristics. It seems that each result is saved under the same Characteristics UUID.
When i perform a read on that Characteristics, it simply only outputting the first result only, which is the first Characteristics, but not the remaining ones.
So how can I read multiple Characteristics, which share the same UUID?

A characteristic can only have one value at a given time. But it can change its value over time.
I don't know your device, but it might be that it emits the results on one characteristic as notification one after the other. If that's the case, you should us startNotifications (and stopNotifications) instead of read.

Hi, Yes make more sense as my device is scanning its finding new values.
When every I do a startNotifications, my APP just crashes and closes. This is on an Andriod.  Is this a known issue?

Should the characteristic I am reading from the need to support Notifications?
This the error we are getting when setup the notification:
`E/Capacitor: Serious error executing plugin
java.lang.reflect.InvocationTargetException
at java.lang.reflect.Method.invoke(Native Method)
at com.getcapacitor.PluginHandle.invoke(PluginHandle.java:99)
at com.getcapacitor.Bridge$1.run(Bridge.java:542)
at android.os.Handler.handleCallback(Handler.java:914)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:224)
at android.os.HandlerThread.run(HandlerThread.java:67)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'byte[] android.bluetooth.BluetoothGattDescriptor.getValue()' on a null object reference
at android.bluetooth.BluetoothGatt.writeDescriptor(BluetoothGatt.java:1255)
at com.capacitorjs.community.plugins.bluetoothle.Device.setNotifications(Device.kt:241)
at com.capacitorjs.community.plugins.bluetoothle.BluetoothLe.startNotifications(BluetoothLe.kt:317)
at java.lang.reflect.Method.invoke(Native Method)
at com.getcapacitor.PluginHandle.invoke(PluginHandle.java:99)
at com.getcapacitor.Bridge$1.run(Bridge.java:542)
at android.os.Handler.handleCallback(Handler.java:914)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:224)
at android.os.HandlerThread.run(HandlerThread.java:67)
E/AndroidRuntime: FATAL EXCEPTION: CapacitorPlugins
Process: com.aardra.cloud, PID: 31543
java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
at com.getcapacitor.Bridge$1.run(Bridge.java:551)
at android.os.Handler.handleCallback(Handler.java:914)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:224)
at android.os.HandlerThread.run(HandlerThread.java:67)
Caused by: java.lang.reflect.InvocationTargetException
at java.lang.reflect.Method.invoke(Native Method)
at com.getcapacitor.PluginHandle.invoke(PluginHandle.java:99)
at com.getcapacitor.Bridge$1.run(Bridge.java:542)
at android.os.Handler.handleCallback(Handler.java:914)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:224)
at android.os.HandlerThread.run(HandlerThread.java:67)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'byte[] android.bluetooth.BluetoothGattDescriptor.getValue()' on a null object reference
at android.bluetooth.BluetoothGatt.writeDescriptor(BluetoothGatt.java:1255)
at com.capacitorjs.community.plugins.bluetoothle.Device.setNotifications(Device.kt:241)
at com.capacitorjs.community.plugins.bluetoothle.BluetoothLe.startNotifications(BluetoothLe.kt:317)
at java.lang.reflect.Method.invoke(Native Method)
at com.getcapacitor.PluginHandle.invoke(PluginHandle.java:99)
at com.getcapacitor.Bridge$1.run(Bridge.java:542)
at android.os.Handler.handleCallback(Handler.java:914)
at android.os.Handler.dispatchMessage(Handler.java:100)
at android.os.Looper.loop(Looper.java:224)
at android.os.HandlerThread.run(HandlerThread.java:67)
I/Process: Sending signal. PID: 31543 SIG: 9
I have attached the document of the BLE device for your reference:
Atmel-42683-ATWINC3400-BLE-WiFi-Scan-and-Connect-Services-Guide_UserGuide.pdf
`
The service is fb8c0001-d224-11e4-85a1-0002a5d5c51b’ and characteristic is "AP Details"(fb8c0100-d224-11e4-85a1-0002a5d5c51b)
Thanks

When every I do a startNotifications, my APP just crashes and closes. This is on an Andriod. Is this a known issue?

This was a bug in the plugin and has been fixed in the latest release. Thanks for reporting this.

Should the characteristic I am reading from the need to support Notifications?

Yes, the characteristic has to support notifications, otherwise you cannot use startNotifications.

The service is fb8c0001-d224-11e4-85a1-0002a5d5c51b’ and characteristic is "AP Details"(fb8c0100-d224-11e4-85a1-0002a5d5c51b)

Is it also possible to do a READ but select a particular instance of characteristics to read? Based on what I have read it is possible to a service could have multiple characteristics with the same UUID. https://developer.android.com/reference/android/bluetooth/BluetoothGattCharacteristic#getInstanceId()

Each characteristic needs to have it's own UUID. There cannot be several characteristics behind the same UUID. (But several characteristics can be in the same service.) Where did you find this UUID fb8c0100-d224-11e4-85a1-0002a5d5c51b? I didn't find it in the document you linked. Maybe the second WiFi is just under fb8c0101-d224-11e4-85a1-0002a5d5c51b and so on? You can use the nrf connect app to inspect your device and find all characteristics.

That UUID is what is used by the WINC3400 module. It seems that their library assigns the same UUID for multiple characteristics. I have now managed to modify it so that each characteristic now has its own UUID.
Therefore we can close this case.


Redesign edit profile

closes #673
Please let me know if you want me to apply CSS differently, cause I added import to new CSS file.
PS. I see 1 SCSS file, can we use SCSS or we're all working on CSS only?

As for sass, it is enabled but only for global stylesheets (doesn't work with css modules). And since we've tried to minimize globals outside bootstrap customizations, we haven't used sass much. But it would make writing the media queries easier. I have not used sass much personally, my only concern is conflicting global styles.


Specify target architecture for PTX compilation, based on GPU capability

Without this, NVRTC compiler falls back to default 'compute_52' architecture and, in case of GPUs with lower capabilities (<=5.0), it results in a 'CUDA_ERROR_INVALID_PTX' error when loading module.
(while it may also underutilize GPUs with greater capabilities...).

Thanks but there is a problem in your patch: after your change,  you must also use the CUDA GPU compute capability version as key of the cache otherwise you may use the wrong (cached) PTX on GPU with different compute capability (in a PC with multiple GPUs).
Just add something like "cap-v." to the string used as cache file name here: https://github.com/LuxCoreRender/LuxCore/blob/7c257db81c9be6f990df6db3807d372f9b8881b7/src/luxrays/utils/cuda.cpp#L137
And it should work fine in any case.

Thanks but there is a problem in your patch: after your change,  you must also use the CUDA GPU compute capability version as key of the cache otherwise you may use the wrong (cached) PTX on GPU with different compute capability (in a PC with multiple GPUs).
Just add something like "cap-v." to the string used as cache file name here: https://github.com/LuxCoreRender/LuxCore/blob/7c257db81c9be6f990df6db3807d372f9b8881b7/src/luxrays/utils/cuda.cpp#L137
And it should work fine in any case.

Thank you for your feedback! I'll propose you a fix for that asap...

Thank you for your feedback! I'll propose you a fix for that asap...

Hello,
Here is some additional code to follow up on your remark...

Hello,
Here is some additional code to follow up on your remark...

Merged, thanks.

Merged, thanks.

Thanks to you!
On Tue, Jan 19, 2021 at 4:07 PM David Bucciarelli<EMAIL_ADDRESS>wrote:

Merged, thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LuxCoreRender/LuxCore/pull/492#issuecomment-762902233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIEPP4GVLXI5ZNEEX6HHW2LS2WN4RANCNFSM4V4TS2TQ
.

Thanks to you!
On Tue, Jan 19, 2021 at 4:07 PM David Bucciarelli<EMAIL_ADDRESS>wrote:

Merged, thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LuxCoreRender/LuxCore/pull/492#issuecomment-762902233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIEPP4GVLXI5ZNEEX6HHW2LS2WN4RANCNFSM4V4TS2TQ
.

Quick question, does this means that with higher capability GPU's this will make CUDA get more performance in those GPU's? (just curiosity)

Quick question, does this means that with higher capability GPU's this will make CUDA get more performance in those GPU's? (just curiosity)

Yes... at least according to Nvidia! They state (https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#ptx-compatibility):
"Note that a binary compiled from an earlier PTX version may not make use of some hardware features. For example, a binary targeting devices of compute capability 7.0 (Volta) compiled from PTX generated for compute capability 6.0 (Pascal) will not make use of Tensor Core instructions, since these were not available on Pascal. As a result, the final binary may perform worse than would be possible if the binary were generated using the latest version of PTX."
You may find some more detailed information about GPU features vs. compute-capabilities here: https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#compute-capabilities

Yes... at least according to Nvidia! They state (https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#ptx-compatibility):
"Note that a binary compiled from an earlier PTX version may not make use of some hardware features. For example, a binary targeting devices of compute capability 7.0 (Volta) compiled from PTX generated for compute capability 6.0 (Pascal) will not make use of Tensor Core instructions, since these were not available on Pascal. As a result, the final binary may perform worse than would be possible if the binary were generated using the latest version of PTX."
You may find some more detailed information about GPU features vs. compute-capabilities here: https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#compute-capabilities


Build on Windows - incorrectly saved .ppm format

Hi!
First of all thank you for great tutorials on 3D graphics.
There is an issue when building project on Windows with GCC. In initial commit in file tinyraycaster.cpp in function drop_ppm_image you have:
std::ofstream ofs(filename);
ofs << "P6\n" << w << " " << h << "\n255\n";

On Windows GCC replaces \n in output file with platform specific code 0x0D0A, what is incorrect (it should be 0x0A in the file). This result in incorrect *.ppm image representation.
This happens because of opening file in "text" mode. To fix this, file should be opened in "binary" mode, for example like this:
std::ofstream ofs;
ofs.open(filename, std::ofstream::out | std::ofstream::binary);
ofs << "P6\n" << w << " " << h << "\n255\n";

Then everything works correctly. I think this patch would not affect other platforms.

Yes, this issue had already been adressed by this commit.

I love your writeups and walkthroughs! Thanks for putting them together. However, a big obstacle for beginners is that the first code snippets (in both tinyraycaster and tinyraytracer repositories) do not produce matching output when copied/pasted and compiled on Windows machines. I understand that you and many other enthusiasts develop on Linux, but there is nowhere (that I could find) where the reader is warned that the code must be modified in order to follow along on Windows. A bold note at the beginning or quick reminder under the code snippet would be much appreciated. Thanks again for all your hard work.

From my experience, bold reminders are ignored by most readers. That is why I keep this current issue open even if in later commits the bug was fixed.

It's not only about ignoring reminders. It's about someone, who is trying to start from the very beginning step by step, as you recommend in your tutorials, and taking the link to initial commit to try it himself. Then he realizes that output is different comparing to what it should be and it's unclear why, and it takes some time to find where the problem hides. Also for the beginner it's not trivial to find the problem himself.
It's not criticism, just some thoughts. Also I don't know if there is a really good way to fix this in initial commit.

I do agree with you, this is problematic, but I do not see a satisfactory solution to the problem. I do create bugs (a lot). This repository is a real history of one of my saturdays, not something I prepared for months before. While modifying the very first commit is possible, I do not master git sufficiently to feel a firm ground under my feet with this kind of solution.

Just notice: I think, this is possible to edit it directly on github just as text file, but I can't guarantee consequences.
Summing up, I think that small reminder in README file about building on different platforms (including this issue) would be helpful anyway for those people, who would find it. :)
Thank you again for you work and sharing knowledge.

Windows with GCC.

You mean the gcc toolchain or the gcc or g++.
gcc is another rabbit hole where many people get lost and just never try developing using it. I can't blame them.
Is this tested with MSVC? I figure most people would use that on Windows rather than gcc.
thanks for this project

I think I would love to ask you a few questions about this project of yours - https://github.com/ssloy/triador
thanks ret394

well... go and ask there?

Um, you do know that you can rewrite git history, right? You can edit the early commits, and force push the repo to update it. I find that doing this via command line is no fun, and github doesn't provide any tools for that either - the power tool you want is smartgit - it's free for use in open source development. When you open this repo using smartgit, you can go to the log and either move the commit that fixes the issue back in the history, or right click on the first commits and edit them, rewriting the history. It's fairly unproblematic.

It's fairly unproblematic.

It's not, it will break the repo on future git pulls for everyone who cloned it pre-rewrite, and it'll also change all commit hashes anyway (so at least direct links to commits won't be fixed with this).
completely unrelated minor side note: maybe this repo would be a good match to be tagged 2-5d? https://github.com/topics/2-5d that tag could be great to unite pseudo-3d things, raycasters and others, but it's mostly unused right now

Wow!Very thank you，i have been in this problems for more than 2H!!!


fix doc

fix doc typo for IBM JVM support

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #1684 opened

Start Time: 2021-03-04T16:17:14.252+0000

Duration: 3 min 0 sec

Commit: d3d8e726f8f37b8fc514940a832542d2908edf68

Trends :test_tube:

:grey_exclamation: Flaky test report
No test was executed to be analysed.


Demo and explain the Clustering by feature algorithm

This is the algorithm from #127
The demo should include:

Running the notebook
Explaining the steps in the notebook
Explaining what insights are we gaining from the clustering

Initial Time Estimates:
Setup - Updating scripts for improved emotion code and re-running notebook:  1 hour
Research/Planning - examining updated sagemaker notebook output, analysis writeup: 2 hours
Deployment - Presenting to Floop: .25 hour
TOTAL ESTIMATE: 3.25 hours

Activities Documentation

Date
Activity
Time Spent

3/19/22
Setup
1 hour

Research
1 hour

3/20/22
Planning
1.5 hours

Research
.5 hour

3/21/22
Task Management
.25 hour

3/24/22
Deployment
.25 hour

Time Totals

Activity
Time Spent

Setup
1 hour

Research
1.5 hours

Planning
1.5 hours

Task Management
.25 hour

Deployment
TBA hour

TOTAL TIME:
TBA HOURS


Correction for raw affiliation IFB - South Green Bioinformatics Platform, Bioversity, CIRAD, INRAE, IRD – Montpellier, France; PHIM Plant Health Institute, University of Montpellier, CIRAD, INRAE, Institut Agro, IRD – Montpellier, France

Correction needed for raw affiliation IFB - South Green Bioinformatics Platform, Bioversity, CIRAD, INRAE, IRD – Montpellier, France; PHIM Plant Health Institute, University of Montpellier, CIRAD, INRAE, Institut Agro, IRD – Montpellier, France
raw_affiliation_name: IFB - South Green Bioinformatics Platform, Bioversity, CIRAD, INRAE, IRD – Montpellier, France; PHIM Plant Health Institute, University of Montpellier, CIRAD, INRAE, Institut Agro, IRD – Montpellier, France
new_rors: 05kpkpg04;051escj72;03fkrcm40;03rnk6m14;003vg9w96;05q3vnk25;05y5s3h28
previous_rors: 05kpkpg04;051escj72;03fkrcm40;03rnk6m14;003vg9w96;05q3vnk25
works_examples: W3185853800
contact: 9dd8bfb606b74aac30f25a5a975219dc:98867b71dba38ef29eca0a26043007b6b4a1 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


/settings

Let users change some settings related to their account:

pings on #potatoboard
autoreactions on/off
#suggestions emojis
pings when you level up

any other ideas?

pings on xp

/addon compact mode

/addon compact mode

Hm not sure, this is more about not flooding the chat

require the = on modmail, or ask on each

found a problem with this
already on the beta, i made the success emoji autoremove after a few sec because people didn't like it staying there
but if we make the = optional ppl wont know what was sent and what wasn't


x/tour: Local version is different from the production version

What version of Go are you using (go version)?
go version go1.15.6 darwin/amd64

$ go version

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
darwin, amd64
go env Output
$ go env

What did you do?
Run go run golang.org/x/tour to run the tour locally.

What did you expect to see?
The same version as it exists in production locally.
What did you see instead?
A different version with different pages.
Local

Production

This is done on purpose. The part of the content prefixed with #appengine: are only displayed when the tour is deployed, and they're rendered in the local version.

Thanks for the clarification! Is there a development workflow outlined anywhere? I was hoping to contribute to the content/tour!

There's https://github.com/golang/tour, and AFAIK that's it. Keep in mind that only a few people in the Go team @ Google can deploy the tour. You can still send changes as described in the README of course.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 306b676:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-04-13-06-38-48 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-04-13-06-38-48

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[core]remove_orphan_files support dry run and optimize output

Purpose

remove_orphan_files  support dry run，similar to the iceberg

The current remove_orphan_files (flink/spark) only outputs the number of deleted files after execution. I think it should output the orphan file path to be more transparent to users. as follows

remove_orphan_files procedures output in spark/flink procedures

remove_orphan_files  action output in flink action jar

Tests

API and Format

Documentation

Tests

API and Format

Documentation

please help review this pr,tks @JingsongLi @yuzelin


[SCH-932] Fix Tooltip on disabled buttons; tooltip doesn't close

Bug re-created: http://localhost:6006/?selectedKind=tooltip&selectedStory=Tooltip
Solution WIP

Maybe Tooltip should have never render when disabled is true?


FIX variable naming change in HTTPMessage.swift

var "l" can have ambiguous meaning.
while this function(func byAppending) scope is not broad now, it may be okay.
However, changing this ambiguous variable name to more obvious meaning such as "var lines" would be better in the long term purpose.

Closing pull request


progress number with decimal

Hi, i have a question, i seek in the issues and not find something similar.
my question is: is possible to add a decimal in the progress number when it is progressing.
Thanks a lot.
Greets from Colombia

try this - var startColor = '#f66733';
var endColor = '#f66733';
$('.progress').each(function(i) {
var circle = new ProgressBar.Circle(this, {
color: startColor,
easing: 'easeInOut',
strokeWidth: 12,
trailWidth: 15,
trailColor: '#685C53',
duration: 3000,
text: {
value: '0'
}
});
var value = ($(this).attr('value') / 100);
circle.text.style.fontSize = '1.35em';

circle.animate(value, {
    from: {
        color: startColor
    },
    to: {
        color: endColor
    },
    step: function(state, circle, bar) {
        circle.path.setAttribute('stroke', state.color);
        console.log(circle);
        circle.setText((circle.value() * 100).toFixed(1)+'%'); //here you can specify decimal places, for example 1 in this case.

    }
});

});

Yes you can override what the text shows by implementing a custom step function as @dhruv-jain demonstrated.


react native platform

Adds an initial implementation of the react native platform adapter. Major components of platform are present and it has been tried out with the cpu backend in an example react native application (not included in this repo). Not finished, but feels like a decent chunk of code for a first PR.
Included:

Platform implementation (fetch)
Two IOHandlers
Unit test setup with mocks for react-native platform stuff. (see design doc for more details on the approach to using unit tests with mocks vs e2e testing in the host platform).

Changes to core include:

Exporting types related to the tf.io namespace
The code used to test if we are running in a browser.
A responseType header set to arrayBuffer is added to requests for weights.

Things this does not include

Full unit tests, more will be added if we are generally happy with the mocking approach
The integration/e2e tests (in form of a react native app). Outlined in the design doc, not implemented yet. (Though as mentioned above I have tested this in a sample react native app I have outside this repo).
ModelManager registration or registration of save/load handers. Currently save and load handlers are created directly via a function. ModelManager registration can be added later as per the design doc.
Tests are not added to CI build yet as they need a new release of tfjs-core.

This change is

I also moved this to the top level as per in person discussion of repo organization.


Get-AzureADMSConditionalAccessPolicy only return 5 Conditionnal Access Policy

Hi.
I'm very happy about the new Get-AzureADMSConditionalAccessPolicy policy but I have an issue with where it only return 5 policies in the PowerShell. We are 3 in our company who made the test and we always have a maximum of 5 policies return by the Command. Why? Is this a limitation of the graph API?

Hi @SimonT-STHS
Thank you for your issue submission. Unfortunately, this appears to be a problem with the cmdlt and a support issue. Issues in this repo are for documentation issues. You can try one of the following links for additional help.
Microsoft Support: https://support.microsoft.com/
Azure AD forums: https://social.technet.microsoft.com/Forums/en-US/home?forum=windowsazureaditpro
Stackoverflow: https://stackoverflow.com/


Remove all occurrences of qWait in tests/

Fixes #2236.

So far I've taken the pragmatic approach of "only fix the tests that actually exhibit issues in practice". There are probably many more places that could fail under the right conditions. That said, I'll change those 4 methods as well, since that's pretty simple.
Maybe in a future refactor those methods could be unified into one, since they all do effectively the same.


@yield and @section is not working

Laravel Version: #.#.#
8.1
PHP Version: #.#.#
7.4.12
Database Driver & Version:

Description:

Home

Dashboars

Post

    @yield('content')

view/show.blade.php

@extends('welcome')

@section('content')

        Content

@endsection

### Steps To Reproduce:

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


[shapex] Handle case where shape.shape_of feeds into from_extent_tensor

Pre-fetching for upstream change where shape.shape_of can produce a
ranked tensor of index, rather than just a shape and post
canonicalization one has a shape.shape_of feeding into a
shapex.from_extent_tensor directly. The lowering of shape_of ti
get_ranked_shape then leads to an invalid ranked_tensor as input to
from_extent_tensor, e.g.,
%63 = "shapex.get_ranked_shape"(%55) : (tensor<?x10xf32>) -> !shapex.ranked_shape<[?,10]>
%64 = "shapex.from_extent_tensor"(%63) : (!shapex.ranked_shape<[?,10]>) -> !shapex.ranked_shape<[?,?]>

Avoid this by removing the from_extent_tensor in these cases and
propagating the value from get_ranked_shape directly.

Note: I have submitted this on the internal side to unblock the integrate there. We will hold this one open until successful.


how to set tensor-parallel-size and pipeline-parallel-size in llama structure?

I want to deploy hf 70b model on 4 * A6000.   Which kind of hyper parameters is better  tp=2,pp=2 or tp=1,pp=4 or tp=4, pp=1?

Hi @CaoYiwei, vLLM currently does not support pipeline parallelism. However, you can use tensor parallelism very easily. Please check out our docs.


Added distribution_matching_indices function

The utils.distribution_matching_indices function allows you to resample an input distribution (with replacement) so that it matches a desired output distribution. This is useful, for example, when comparing the properties of two galaxy samples with matching stellar mass functions. Visual demonstration shown in the figure below, which is also included in the docs.
@eteq - do you know whether this function already appears in Astropy?

@aphearin - definitely does not.  I think it definitely could make sense in astropy.stats, though, so I'd encourage trying there.
You might get some pushback along the lines of "maybe this should go in scipy.stats".  I have a suspicion it wouldn't make it in there (not clear that it's really what scipy.stats is about), but worth thinking a bit about. To me this is a fairly astro-common application that I don't see often elsewhere, so I think it's fair to include... But others might disagree.

Ok, sounds good. This should be easy enough to port over to astropy.stats since the module is standalone. Thanks for pointing me to the right place.


Bug 1877234: Drop recovery apiserver

The recovery apiserver is obsolete since we have cert auto-recovery.
For accessing API in situations when certs are expired KAS has localhost-recovery endpoint with long lived certs for localhost and local kubeconfigs on the master.
Requires:

[x] https://github.com/openshift/cluster-kube-apiserver-operator/pull/858

/hold
/cc @sttts @deads2k

/hold cancel
https://github.com/openshift/cluster-kube-apiserver-operator/pull/858 merged

/retest

/lgtm

/bugzilla refresh


Post release automated changes for mobilenetwork releases

Post release automated changes for azure-arm-mobilenetwork

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


Create a post-download survey and set up automated emails

Please label your issue with a label to specify its priority: breaking, high, medium, or low.
Context
Replace SUS with questions which will give us info we want to measure
Problem or idea

Figure out what we want to measure
Design a survey for that
Setup hubspot so it'll email people after they download data (limit to once)

Solution or next step
I will write up a list of things we are interested in.

Did this. New survey is in place


linger.ms sends one message per interval, not batch

Description
Not 100% sure whether this is a Confluent.Kafka issue or a librdkafka issue.
I'm using Confluent.Kafka 1.4.3 (librdkafka 1.4.2) in an Azure Function App and DotNet Core 2.1.
When I set the LingerMs setting to a high value, say 5000, the client sends one message every 5 seconds. Not a MessageSet containing all messages I've tried to send during those 5 seconds, just the one message. E.g. if I'm trying to send 200k messages this is going to take over 11 days!
As I understand it, the client should be batching up messages I'm trying to send within each 5 second interval, and then sending them as one MessageSet to the broker (Confluent Cloud in this case).
I'm viewing the messages coming in in the Confluent Cloud UI, and I see the successive messages on each partition having 1 higher offset than the previous one, so it's not that the UI is displaying one message per MessageSet or anything like that.
My client config:
var pConfig = new ProducerConfig
{
    BootstrapServers = _config["KafkaBootstrapServers"],
    SaslMechanism = SaslMechanism.Plain,
    SecurityProtocol = SecurityProtocol.SaslSsl,
    SaslUsername = _config["KafkaUsername"],
    SaslPassword = _config["KafkaPassword"],
    MessageSendMaxRetries = 10,
    MessageTimeoutMs = 30 * 1000,
    RetryBackoffMs = 250,
    Acks = Acks.Leader,
    LingerMs = 5000,
    CompressionType = CompressionType.Gzip,
    CompressionLevel = 12,
    SslCaLocation = "D:\\home\\site\\wwwroot\\cacert.pem"
};

Sample code which tries sending as quickly as possible:
public async Task sendToKafkaTopic<T>(string topic, List<T> reportDtos)
{
    var kafkaMessages = new List<Message<string, string>>();
    foreach(var reportDto in reportDtos)
    {
        string msgBody = JsonConvert.SerializeObject(reportDto);
        var message = new Message<string, string> { Key = Guid.NewGuid().ToString(), Value = msgBody };

        kafkaMessages.Add(message);
    }

    foreach (var kafkaMessage in kafkaMessages)
    {
        try
        {
            var deliveryResult = await _producer.ProduceAsync(topic, kafkaMessage);
            _log.LogInformation($"DEBUG: Delivered the following to {deliveryResult.TopicPartitionOffset}:\n\n{deliveryResult.Value}");
        }
        catch (ProduceException<string, string> e)
        {
            _log.LogInformation($"DEBUG: Kafka exception! (Error: {e.Error}. Message: {e.Message})");
            throw new Exception($"Kafka exception! --> Error: {e.Error}. Message: {e.Message}", e);
        }
    }
}

I've tried sending messages with and without keys - no difference.
How to reproduce

Create a function app in Azure.
Configure client with a high value in the LingerMs setting.
Create a loop that tries to send, say, 100.000 messages to a topic in Confluent Cloud.

Checklist
Please provide the following information:

[x] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[x] Confluent.Kafka nuget version.
[x] Apache Kafka version.
[x] Client configuration.
[x] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[x] Critical issue.

Sorry about the code formatting there, can't seem to get GitHub to parse that correctly...

Sorry about the code formatting there, can't seem to get GitHub to parse that correctly...

just put it in closing and starting tags:
public async Task sendToKafkaTopic(string topic, List reportDtos)
{
var kafkaMessages = new List<Message<string, string>>();
foreach(var reportDto in reportDtos)
{
string msgBody = JsonConvert.SerializeObject(reportDto);
var message = new Message<string, string> { Key = Guid.NewGuid().ToString(), Value = msgBody };

   kafkaMessages.Add(message);
}

foreach (var kafkaMessage in kafkaMessages)
{
   try
   {
       var deliveryResult = await _producer.ProduceAsync(topic, kafkaMessage);
       _log.LogInformation($"DEBUG: Delivered the following to {deliveryResult.TopicPartitionOffset}:\n\n{deliveryResult.Value}");
   }
   catch (ProduceException<string, string> e)
   {
       _log.LogInformation($"DEBUG: Kafka exception! (Error: {e.Error}. Message: {e.Message})");
       throw new Exception($"Kafka exception! --> Error: {e.Error}. Message: {e.Message}", e);
   }
}
}

because you're producing synchronously - you produce a message, librdkafka waits 5s for any additional messages to include in the batch (there won't be because you're blocking), then sends off the produce request to the broker. the response comes back, the produce request unblocks, repeat.

Aha. Thanks. For some reason I had just assumed librdkafka had decoupled this from the thread I was sending from - but yeah, that'd probably not be a good idea...

Aha. Thanks. For some reason I had just assumed librdkafka had decoupled this from the thread I was sending from - but yeah, that doesn't really make much sense. Sorry. I also now see that the docs specifically mention this. Double bad on my part :P

What corrections did you implement to make it work asynchronously?

Aha. Thanks. For some reason I had just assumed librdkafka had decoupled this from the thread I was sending from - but yeah, that doesn't really make much sense. Sorry. I also now see that the docs specifically mention this. Double bad on my part :P

What corrections did you implement to make it work asynchronously?

That was over two years ago. The latest version that we used was as follows:
public async Task sendToKafkaTopic<T>(string topic, List<T> reportDtos)
{
    List<Task<DeliveryResult<string,string>>> sendTasks = new List<Task<DeliveryResult<string,string>>>();

    _log.LogInformation("Starting sending to Kafka...");
    foreach (var reportDto in reportDtos)
    {
        string msgBody = JsonConvert.SerializeObject(reportDto);
        var message = new Message<string, string> { Key = Guid.NewGuid().ToString(), Value = msgBody };

        sendTasks.Add(_producer.ProduceAsync(topic, message));
    }

    await Task.WhenAll(sendTasks).ContinueWith((result) => {
        if(result.Exception != null)
        {
            throw new Exception($"Exception was thrown during sending to Kafka! Message: {result.Exception.Message}", result.Exception);
        }
    });

    _log.LogInformation($"Successfully sent {reportDtos.Count} messages to Kafka!");
}

Aha. Thanks. For some reason I had just assumed librdkafka had decoupled this from the thread I was sending from - but yeah, that doesn't really make much sense. Sorry. I also now see that the docs specifically mention this. Double bad on my part :P

What corrections did you implement to make it work asynchronously?

That was over two years ago. The latest version that we used was as follows:
public async Task sendToKafkaTopic<T>(string topic, List<T> reportDtos)
{
    List<Task<DeliveryResult<string,string>>> sendTasks = new List<Task<DeliveryResult<string,string>>>();

    _log.LogInformation("Starting sending to Kafka...");
    foreach (var reportDto in reportDtos)
    {
        string msgBody = JsonConvert.SerializeObject(reportDto);
        var message = new Message<string, string> { Key = Guid.NewGuid().ToString(), Value = msgBody };

        sendTasks.Add(_producer.ProduceAsync(topic, message));
    }

    await Task.WhenAll(sendTasks).ContinueWith((result) => {
        if(result.Exception != null)
        {
            throw new Exception($"Exception was thrown during sending to Kafka! Message: {result.Exception.Message}", result.Exception);
        }
    });

    _log.LogInformation($"Successfully sent {reportDtos.Count} messages to Kafka!");
}

Thank you for your response.


SIG soft delete CLI fix

Resource Provider

Azure Compute
Description of Feature or Work Requested

Currently when user creates a new gallery without properties, the softDeletePolicy is null, however, if user does a 'az sig update' call on this gallery, the input softDeletePolicy is not null, it is the default value, this needs to be fixed. When user does update, the input SoftDeletePolicy needs to be null too.
Minimum API Version Required

2021-03-01
Swagger Link

Target Date

Next sprint release

Adding @zhoxing-ms to help with with issue.

Related PR #19569


better pip cache

  - name: Setup Python
    uses: actions/setup-python@v3
    with:
      python-version: '3.8'

  - name: Upgrade pip
    run: |
      # install pip=>20.1 to use "pip cache dir"
      python3 -m pip install --upgrade pip

  - name: Get pip cache dir
    id: pip-cache
    run: echo "dir=$(pip cache dir)" >> $GITHUB_OUTPUT

  - name: Cache dependencies
    uses: actions/cache@v3
    with:
      path: ${{ steps.pip-cache.outputs.dir }}
      key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements.txt') }}
      restore-keys: |
        ${{ runner.os }}-pip-

  - name: Install dependencies
    run: python3 -m pip install -r ./requirements.txt

see https://github.com/marketplace/actions/github-pages-action#-static-site-generators-with-python

#7


Remove pegasyseng docker tag and push config

Fixes https://github.com/ConsenSys/web3signer/issues/427

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Simon Dudley seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix(update): remove no longer needed migrations thingy

https://discord.com/channels/1040316820650991766/1154514012143362151/1262541906026430544
I TOLD YOU ID DO IT

Thank you for fixing this!

Thankyou for fixing this


Execution Summary Failing

I'm getting errors on most runs lately after shutting down the keep-alive thread. Example trace:
 File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/hadoop/c2/dw/src/main/python/houzz/log/log_jobs.py", line 88, in <module>
    luigi.run()
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/interface.py", line 340, in run
    return interface.run(tasks, worker_scheduler_factory)
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/interface.py", line 195, in run
    logger.info(execution_summary.summary(w))
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/execution_summary.py", line 381, in summary
    return _summary_wrap(_summary_format(_summary_dict(worker), worker))
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/execution_summary.py", line 330, in _summary_format
    str_output += '{0}\n'.format(_get_str(group_tasks[status], status in _PENDING_SUB_STATUSES))
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/execution_summary.py", line 125, in _get_str
    if _ranging_attributes(attributes, unique_param) and len(attributes) > 2:
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/execution_summary.py", line 176, in _ranging_attributes
    if unique_param[1].next_in_enumeration(attributes[0]) is None:
  File "/var/pyenv/luigi_pipeline/local/lib/python2.7/site-packages/luigi-1.3.0-py2.7.egg/luigi/parameter.py", line 363, in next_in_enumeration
    return value + cls._timedelta
TypeError: unsupported operand type(s) for +: 'datetime.datetime' and 'abstractproperty'

Do you know what Parameter type it is? It seems like _timedelta isn't implemented for that DateParameter.
But I suppose it should be timedelta (public) as it's meant to be override by luigi users.

Are DateParameter and DateHourParameter abstract now? Neither one seems to have _timedelta implemented and I don't use any other subclasses of DateParameterBase.

Whoups it's not defined for DateHourParameter! Sorry about that. I'll fix it right away.

I think that DateParameter needs it too.

Not sure what you mean. We already have test cases for DateParameter and they pass.

Fixed in #1196.
By the way @daveFNbuck, when the Execution Summary have been "working". Have you found the summary useful?

So far I've only seen it work for trivial results. I'll let you know once I've seen it a few more times.

I just ran a successful job with it for the first time and noticed a bug where the execution summary shows jobs as having been run by another worker, but that other worker is the one that is showing the summary. This happens if you have multiple workers and one asks for a job while the others are running things.

but that other worker is the one that is showing the summary

o_O,
That's bad. I guess I need to test this with --worker 2. I wouldn't think luigid would reply that "you can't do any more work since you're already doing the work". But I guess it makes sense.  I'll fix. Thanks for reporting this @daveFNbuck!

Have you had time to try the new changes out @daveFNbuck? If you don't see any more issues. Let's close this ticket. :)

Yeah, it's fixed for me now.
On Tue, Sep 8, 2015 at 1:36 PM, Arash Rouhani<EMAIL_ADDRESS>wrote:

Have you had time to try the new changes out @daveFNbuck
https://github.com/daveFNbuck? If you don't see any more issues. Let's
close this ticket. :)
—
Reply to this email directly or view it on GitHub
https://github.com/spotify/luigi/issues/1192#issuecomment-138691906.

Cool. Thanks! Let me know when you have feedback for somewhat less trivial runs. :)


How can we show good encapsulation and OOP practice?

In Pull request #438 (Gaussian elimination in Java) the discussion came up, how we can show good practice in terms of OOP and encapsulation, and which are the appropriate algorithms to use those techniques.

The goal of the AAA is not to show off good practice in certain languages. The main goal is to explain algorithms with text and code examples. If an algorithm calls for object oriented programming, then that will reflect in the code examples. Take a look at Stable Marriage, for example. The Java implementation utilizes a Person class with properly encapsulated data and so do all language implementations that support OOP.
The suggestion made in #438 is not a good example of encapsulation and good OOP practice and I know that the majority of the contributors of this project would agree with me. It's a hack around having to pass arguments to a method that only works because the example code's main method resides in the same static class as the method in question.

I already stated this in the PR #438, but I'll repeat it here just for the sake of discussion:
"it doesn't matter a lot having it as a variable or as a member of the class in terms of readability, i just want to have code in this project to be of the maximum quality possible (in my opinion at least, seeing that in this case some of you disagree with me) but if you guys think that it is fine as it is, so be it :)"

In my opinion it actually does matter. A class member is there to describe a concrete instance of a class. As an example a circle can have multiple radii and thus be strictly different things. An algorithm on the other hand always (or mostly always depending on how much randomness you introduce) behave the same no matter the parameters. So they are indeed NOT different instances of the same class but the same instance just with different input.
The only time I would put an algorithm into a class is in a task queue. But then it is a task and not an algorithm i want to abstract of.
Another reason I would disagree with you is that it leads to bad and/or to verbose code.
Take this as an example.
int x = Math.abs(num);

I hope you agree that that is the easiest and most understandable way of getting the absolute value of num.
According to your argument the code should look like following
absClass abs = Math.abs(num);
int x = abs.invoke()

and that is very unnecessary in my opinion.
In general I agree with @Butt4cak3 that this is bad OOP style and just a hack around having to pass arguments, which isn't helping at all because you have to pass the arguments somewhere anyway. And if I have the choice between passing it to a constructor or to an algorithm directly my choice will be the algorithm in 99% of the cases.

Yeah i absolutely agree with you @Liikt, the code is better and more OOP without creating instances of a class that doesn't make sense to create, and yes, probably in a real application this code would be just a part of a function inside a class that does other things, absolutely, i was not referring to that in my review at all. the class methods are static, so you can access them without creating an object, and the methods can access the array when it is a member the same way as if it were an argument.
Now the thing that i think you guys are trying to convey is that the code is not usable or extensible if we put the array as a member of the class because if we want to change the array from other class we have to create an object or calling a "set" method or something similar, and that's absolutely correct, if we want code here to be copy&paste-friendly then we should do it like you guys say, my opinion only works for a proof of concept code that is by its own (which i thought we were going to put in the archive, but it seems not) so sorry for all the useless discussion, really

Since the discussion hasn't officially resolved yet, can we get an official statement, or should I just close this issue?

I think it's ok to close this issue. It ultimately amounts to better reviewing of OOP languages


Moving virtual ball on OLED and LEDs

I've made new mode mode_roll testing the concept of moving a virtual ball in a circle with the accelerometer and having LEDs (that are in a ring that I imagine surrounds the circle) light up as it passes. The mode appears as ROLL in the menu. Set the intensity of the LEDs to low (as in dance mode), a ball on the OLED rolls as you tilt the swadge.
It's a mash up of your demo and dance mode. It is not very efficient and duplicates much of the code from demo and dance so could share the routines.
https://youtu.be/N0L6-arMcYQ
https://github.com/bbkiwi/Swadge-Devkit-Fw/tree/masterB
This branch incorporates the PR that fixes use of button 0, however it seems I have to hold button 0 down a bit longer to get into menu mode. Not sure why!

Slight downside with button 0, if down hold down firmly when menubar gets to end sometimes jumps to first entry ie demo mode.
In this branch also tweaked dance mode, and roll mode and colorchord mode to use NUM_LIN_LEDS so I can drive bigger rings than size 6.

I wonder if it's detecting a single press after reboot if the button is held? Should be easy to detect & fix, maybe by initializing button values to GPIO values rather than 0.
The other modes you tweaked NUM_LIN_LEDS for will likely be removed, but good that they've been fixed for now.

The downside disappears if button 0 is for menu only.
Using NUM_LIN_LEDS was just an exercise, but I think it is a good idea to such defined parameters to make the code more flexible.
mode_roll is just a concept test at this point to develop tools to use the accelerometer to move things. I’ve variations that can roll a virtual ball on the screen, so a game of solving a maze coukd be implemented.

This is a more general comment related to having more common code. I wanted to reuse stuff from dance and demo, but found myself having to rename stuff (probably my lack of experience with c).
Also here are comments from my friend Richard (who has not yet received a slack invite). Can he join the meeting this Thursday?
Hi Bill,
I had a few thoughts on the reflector game code that you showed me.
The game code that we looked through merged the game with the communications protocol.
This should be unpicked so that the game and the protocol are in separate files.
Then multiple games could use the same well tested protocol, and game simulations and game testing need not run on the hardware.
Maybe this has been done already?
Also the protocol needs to be adapted for large numbers of cooperating swadges.

Rolling a ball through a maze, just like this:

Don't worry about cross-mode code reuse for now. Dance and Demo will be deleted eventually.
To Richard, I know the communication protocol has to be extricated from the previous game code. It's on my own to-do list, but I'll make an official ticket for it. Adapting it for multiple simultaneous connections is a step after that, and trickier.

I've just added an ODE solver to this branch (that anyone can use - found it on the web and required very little tweaking) and used it to simulate a circular pendulum. I will now model a rolling ball and use it to implement the maze!

The ball is moving around the maze quite well, so I'll call this done.


"Synchronously" spelling correction

Summary
Corrected spelling.

and Thank you! @gewarren for considering this PR.


Not working with prettier-plugin-organize-imports

It appears that either prettier-plugin-tailwindcss or prettier-plugin-organize-imports affect the file, based on which one appears last in the config plugin list, but not both.
Prettier Version: 2.5.1
prettier-plugin-tailwindcss version: 0.1.4
prettier-plugin-organize-imports version: 2.3.4
I am currently able to somewhat merge the two plugins manually by using the preprocess function from prettier-plugin-organize-imports, but it would be nice if the two got along without the patch:
// merged-prettier-plugin.js

const tailwind = require('prettier-plugin-tailwindcss')
const organizeImports = require('prettier-plugin-organize-imports')

const combinedFormatter = {
  ...tailwind,
  parsers: {
    ...tailwind.parsers,
    ...Object.keys(organizeImports.parsers).reduce((acc, key) => {
      acc[key] = {
        ...tailwind.parsers[key],
        preprocess(code, options) {
          return organizeImports.parsers[key].preprocess(code, options)
        },
      }
      return acc
    }, {}),
  },
}

module.exports = combinedFormatter

// .prettierrc.js

module.exports = {
  plugins: [require('./merged-prettier-plugin.js')],
  ...
}

I have the same issue. I'm going to wait until this has been resolved to use the tailwind plugin.

Hey! Thanks for reporting this. I've created a new issue (#31) dedicated to keeping track of Prettier plugins that are not compatible with this plugin. Closing this issue in favor of that 👍
Please feel free to post this incompatibility with prettier-plugin-organize-imports  in that issue, it would be great to have it there for quick reference.

Update on this one — we've added support for this plugin in the latest (beta) version of our Prettier plugin. You can read more about that here: https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/31#issuecomment-1312202554


Added my profile

Fixes #(issue)
#29

Everything seems alright. About to merge.
Make sure to drop a ⭐ Star to the repository @kom-senapati


Move jenkinsfile.run_parallel_tests() in try block

jenkinsfile.run_parallel_tests()
 needs to be moved in try block. Otherwise, the error is misleading.
https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1066/console
ERROR: Cannot delete workspace :Remote call on test-aws-rhel8-x64-1 failed
[Pipeline] }
[Pipeline] // timeout
[Pipeline] echo
Exception: hudson.AbortException: Cannot delete workspace: Remote call on test-aws-rhel8-x64-1 failed
[Pipeline] }
[Pipeline] // node
[Pipeline] }
[Pipeline] // stage
[Pipeline] End of Pipeline
hudson.remoting.ProxyException: groovy.lang.MissingPropertyException: No such property: jenkinsfile for class: WorkflowScript
	at org.codehaus.groovy.runtime.ScriptBytecodeAdapter.unwrap(ScriptBytecodeAdapter.java:53)
	at org.codehaus.groovy.runtime.ScriptBytecodeAdapter.getProperty(ScriptBytecodeAdapter.java:458)
	at org.kohsuke.groovy.sandbox.impl.Checker$7.call(Checker.java:355)
	at org.kohsuke.groovy.sandbox.GroovyInterceptor.onGetProperty(GroovyInterceptor.java:68)
	at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.SandboxInterceptor.onGetProperty(SandboxInterceptor.java:354)

The real error is Cannot delete workspace :Remote call on test-aws-rhel8-x64-1 failed, not the MissingPropertyException.

This issue is fixed.


crash

when i run the FastValveMaterial.py it do nothing, when running it from cmd it says "[FVM] Program terminated with exit code 0"

did you specify arguments

could you explain it a bit more i never used python

It has nothing to do with python. The repository contains a file named "config.md" which you need to adjust based on your material setup.
Another possibility could be that you didn't install the dependencies (see README).

I configured it,the dependencies should be installed using "python -m pip install --upgrade Pillow" And
"python -m pip install --upgrade numpy" ?

If you have pip installed, just use pip install

Yes, that's how it's supposed to be installed. Would you mind sharing your image file names and config.md file?

https://prnt.sc/227sj3h
`# Input format ("png", "tga")
tga
Input naming scheme (The endings of the image names in order: color map, ao map, normal map, gloss/rough map, metal map - If one parameter isn't set to a valid value, the corresponding method won't be called)
_col
_o
_n
_g
_cmask
Input path
images/
Map used to determine material name
_n
Output path (Can also be multiple subfolders, e.g. folder1/folder2/output/ - This path will be referenced in the VMT file!)
folder1/
Gamma adjustment (0-255) - Only change this from 215 if you know what you're doing, other values might break the PBR effect!
215
Export converted images as tga as well (False/True)
False
Material setup ("gloss", "rough")
rough
Print debug messages (False/True)
True`

when i run the FastValveMaterial.py it do nothing, when running it from cmd it says "[FVM] Program terminated with exit code 0"

Did you even specify arguments?
Also, paste the image instead of placing it on a third party website.

i dont really know what specify arguments mean,I typed "python FastValveMaterial.py" in the cmd

I think I see the issue. You can't have the images in subfolders for the input (Although you can put images from different materials in there) In this case, your images aren't in 'images/' like you specified, but in 'images/body'.
Either move the images to 'images/' or change the config to the new path.

now its show this in the console

and then close
when running it in cmd it says
[FVM] Loading:
Material:       body
Color:          images//body_col.tga
Occlusion:      images//body_o.tga
Metalness:      images//body_cmask.tga
Normal:         images//body_n.tga
Glossiness:     images//body_g.tga
Traceback (most recent call last):
File "C:\Users\Lenovo\Downloads\FastValveMaterial_1207\FastValveMaterial.py", line 289, in
glossImage = convert_roughness_to_gloss(glossImage)
File "C:\Users\Lenovo\Downloads\FastValveMaterial_1207\FastValveMaterial.py", line 118, in convert_roughness_to_gloss
return ImageOps.invert(mIm)
File "C:\Users\Lenovo\AppData\Local\Programs\Python\Python310\lib\site-packages\PIL\ImageOps.py", line 526, in invert
return _lut(image, lut)
File "C:\Users\Lenovo\AppData\Local\Programs\Python\Python310\lib\site-packages\PIL\ImageOps.py", line 58, in _lut
raise OSError("not supported for this image mode")
OSError: not supported for this image mode

That's an error on my side, I'll fix it and make a new release.
Thanks for the report.

ty, should i close the the issue ?

I'll close it when I'm able to confirm that it's working.
I'm assuming this is happening because your image doesn't contain data in all RGBA channels.

Alright, this should be fixed now.


[WIP] Automatically create git tag for new versions

Addresses #541.
@JonasVautherin should we also do the merge with master automatically?

So why on Earth did travis to decide to run this PR?

Let's see if this works in a dry-run.

Unfortunately this didn't trigger:
https://travis-ci.org/Dronecode/DronecodeSDK/jobs/436987724#L9751


haskellPackages.mkDerivation: passthru attributes for executables, testExecutables and benchExecutables

Hey :wave:
I've recently migrated a project from haskell.nix to nixpkgs/cabal2nix.
One thing I found quite useful in haskell.nix was the ability to directly reference executables, test executables and benchmark executables, e.g.:
Cabal:
executable my-package-foo
  main-is: MyPackageFoo.hs

executable my-package-bar
  main-is: MyPackageBar.hs

my-package-foo = my-package.exes.my-package-foo
my-package-bar = my-package.exes.my-package-bar

I had a look at fromPackageDescription, as well as the Pretty instance of Derivation,
and I think it wouldn't be too hard to generate something like this:
mkDerivation {
  # ...
  passthru = {
    executables = {
      my-package-foo = haskell.lib.compose.setBuildTarget "my-package-foo" my-package;
      my-package-bar = haskell.lib.compose.setBuildTarget "my-package-foo" my-package;
    };
    testExecutables = {
      my-package-foo-spec = haskell.lib.compose.setBuildTarget "my-package-foo-spec" my-package;
      my-package-bar-integration = haskell.lib.compose.setBuildTarget "my-package-bar-intergration" my-package;
    };
    benchExecutables = # ...
  };
}

What do you think?
Am I missing something? Would you consider this a useful feature?
An obvious downside would be the need for my-package to be passed in via callPackage.
This could be mitigated by providing executable names instead of derivations.

The implementation could likely mostly live in nixpkgs with something equivalent as https://github.com/NixOS/cabal2nix/pull/617 being implemented for cabal2nix to list out executable targets. As you can see from the discussion on that PR, there are some reservations about generating this additional metadata, though, mainly code size.
I don't think having individual derivations for the components as you propose would be a good idea, since nixpkgs doesn't have component-level granular derivations, but has one big one for everything. Making tons of variants of the same derivation would cause a lot of needless rebuilds of the same stuff potentially since we can't share the work between derivations. I don't think the idea you propose is as useful if you want to use .exes.foo and .exes.bar which both depend on the same internal lib, but that library has to be built twice.
(Maybe not the 100% most accurate thing to transfer the issue since it does need some groundwork in cabal2nix, but clicked too fast…)

That's a fair point about individual derivations for components.
I suppose the only option to address code size concerns for this feature would be to put it behind a CLI flag.


Release V7.0.7

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.31%. Comparing base (0de9728) to head (3b1ec21).

Additional details and impacted files
@@           Coverage Diff           @@
##           stable     #284   +/-   ##
=======================================
  Coverage   89.31%   89.31%
=======================================
  Files          70       70
  Lines        2434     2434
=======================================
  Hits         2174     2174
  Misses        260      260

Flag
Coverage Δ

unittests
89.31% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Polishing :: input form contrast improvement

grey background is now removed
its color is darker
the input label is darker as well
add the same style for the placeholder when the multiselect input is clicked
a bit of CSS cleaning, it's quite messy for the form style => we will need a refactoring soon.

 augmenter contraste des gris dans le VQB


Bug

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.

It's changing my account want let me sign in

We have the same issue


Broken preview on website

Many preview images (such as Consolas, FiraFlott, Hack and more) on the website are broken. Hope they will get fixed.

I think I found the issue. Is Consolas working for you now?

Yes I believe so, especially after I cleared browser cache.

I have one question: why won't you directly use woff2 to display the actual font on the webpage, or at least provide that option?

BTW, when I said it's working earlier, I meant on my mobile device. It's still not working on my desktop as for now.

Should be all set. Thanks for pointing it out.  No real dependency on a certain technology. But an even bigger reason is being able to include paid fonts without maintaining a license.

As of now, for example Hasklig is still not working. The following address
https://res.cloudinary.com/css-tricks/image/fetch/w_1200,q_auto,f_auto/https://coding-fonts.css-tricks.com/screenshots/anonymous-pro/html-dark.png

returns 404 with the error header:
x-cld-error: Resource not found - Error in loading https://coding-fonts.css-tricks.com/screenshots/hasklig/html-dark.png - HTML response

Full headers as follows:
accept-ranges: bytes
access-control-allow-origin: *
access-control-expose-headers: Content-Length,Content-Disposition,Content-Range,Etag,Server-Timing,Vary,X-Cld-Error
cache-control: private, no-transform, max-age=0, no-cache
content-length: 0
content-type: image/gif
date: Wed, 26 Jan 2022 01:21:08 GMT
etag: "d41d8cd98f00b204e9800998ecf8427e"
expires: Wed, 26 Jan 2022 09:06:36 GMT
last-modified: Tue, 25 Jan 2022 09:06:37 GMT
server: Cloudinary
server-timing: akam;dur=4;start=2022-01-26T01:21:08.356Z;desc=hit,rtt;dur=38
strict-transport-security: max-age=604800
timing-allow-origin: *
vary: Accept,User-Agent,Save-Data
x-cld-error: Resource not found - Error in loading https://coding-fonts.css-tricks.com/screenshots/hasklig/html-dark.png - HTML response

Now accessing https://coding-fonts.css-tricks.com/screenshots/hasklig/css-dark.png directly only results in HTML content of the website (not by http redirection, but by directly return the HTML) even with referer header.


add hate module

add hate module
Short description of what this resolves:
Changes proposed in this pull request:
-add hate module

Fixes: #

Thank you for taking the time to contribute! This is sincerely appreciated. 😇
(Powered by GitMate.io)

Leave a comment


Custom theming

This is amazing helpful - your repo saved me ages of work. Thank you!!
I was wondering if you know an easy way to install a custom theme like this one with this setup? https://github.com/apogiatzis/ctfd-neon-theme

Thanks! The easiest way to do this will be to create a custom docker image that includes the theme. You can then override the docker image for this helm chart in the values file.

Thanks, I actually ended up doing exactly that like 1h ago. Just cloned a theme in the first stage of the ctfd Dockerfile.

Nice!


Cache and circular deps

Fixes #35, #33

Hey @michalkvasnicak , I guess you already accidentally published to npm. However, since 1.2.3 I'm now getting:
TypeError: /path/to/my/project/src/shared/components/Header/index.jsx: Cannot read property 'extractCss' of undefined
    at PluginPass.post (/path/to/my/project/node_modules/babel-plugin-css-modules-transform/build/index.js:158:34)
    at File.call (/path/to/my/project/node_modules/babel-core/lib/transformation/file/index.js:628:18)
    at File.transform (/path/to/my/project/node_modules/babel-core/lib/transformation/file/index.js:551:12)
    at /path/to/my/project/node_modules/babel-core/lib/transformation/pipeline.js:50:19
    at File.wrap (/path/to/my/project/node_modules/babel-core/lib/transformation/file/index.js:564:16)
    at Pipeline.transform (/path/to/my/project/node_modules/babel-core/lib/transformation/pipeline.js:47:17)
    at Object.transformFileSync (/path/to/my/project/node_modules/babel-core/lib/api/node.js:152:10)
    at compile (/path/to/my/project/node_modules/babel-register/lib/node.js:118:20)
    at loader (/path/to/my/project/node_modules/babel-register/lib/node.js:144:14)
    at Object.require.extensions.(anonymous function) [as .jsx] (/path/to/my/project/node_modules/babel-register/lib/node.js:154:7)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/path/to/my/project/src/shared/App.jsx:3:1)

@hardchor Yes, I am sorry, I misunderstood how release channels work in npm. I published 1.2.5. Could you try to update if it fixes your problem?


FAIL TO SAVE player's DATA (Can't access registry)

When some users disconnect from server, the following error is printed on console. The server is a JAVA/Bedrock mixed server through Geyser but I have tested both clients with my user and I think this only happens when the player has certain objects in their inventory.
MCServer: 1.21
Fabric Loader: 0.15.11
[23:24:01] [Thread-221/INFO]: saving player's data...
[23:24:01] [localSession-7-1/INFO]: player ha sido desconectado del servidor Java debido a Cliente de Bedrock desconectado
[23:24:01] [Thread-221/ERROR]: Can't access registry ResourceKey[minecraft:root / minecraft:enchantment]
[23:24:01] [Thread-221/ERROR]: FAIL TO SAVE player's DATA:
[23:24:01] [Thread-221/INFO]: [STDERR]: java.util.NoSuchElementException: No value present
[23:24:01] [Thread-221/INFO]: [STDERR]:         at java.base/java.util.Optional.orElseThrow(Optional.java:377)
[23:24:01] [Thread-221/INFO]: [STDERR]:         at mpds.mpds.MPDS.lambda$ondisconnect$11(MPDS.java:378)
[23:24:01] [Thread-221/INFO]: [STDERR]:         at java.base/java.lang.Thread.run(Thread.java:1583)
[23:24:01] [Server thread/INFO]: player lost connection: Disconnected
[23:24:01] [Server thread/INFO]: player left the game

I'm so sorry for being late. I was so busy this summer because I'm high school entrance examination student.
I'm trying to find out what's causing it, but haven't figured it out yet. I think it takes a long time...sorry:(

hello, thank you for waiting.
can you tell what is the item, "certain objects"?

Heeeello Nekozuki, no, not at all as you see the error references just minecraft:root / minecraft:enchantments.
With an empty character this did not happen. I guess that it could be related to the fact that the user built something that kept in his inventory but I am not sure.

Are you using some mods that adding something new item?


Support GPU compute that doesn't require closed-source graphics drivers

From the README, you are actually giving a rationale for GPU compute generally when explaining the choice of CUDA:

Note that we are primarily targeting CUDA - the performance, cost, and power usage of cheap GPUs like the Tesla P4 and GTX 1060 is too good to ignore. We'll make our best effort to support CPU wherever possible for current and future functionality but our emphasis is on performant latency-sensitive tasks even with low-end GPUs like the GTX 1060/Tesla P4 (as of this writing roughly $100 USD on the used market - and plenty of stock!).

I understand that nVidia dominates the GPU market, and that CUDA is ubiquitous, but using CUDA puts your project in the ironic position of effectively requiring closed-source software on users' machines for usable performance, which would seem to be antithetical to Willow's stated ethos (and additionally asks users to give money to a company that's notoriously hostile to OSS).
I'm not really hugely familiar with the GPU compute landscape, but maybe a first step could be for the project to use HIPIFY to (partially?) automate the generation of HIP code, which would at least provide a performant alternative without requiring closed-source drivers and on hardware from a far more open-source friendly company - and AIUI HIP is supported on the open-source amdgpu driver.
I don't know what the picture is like for Intel GPU hardware, but finding a solution there would be great too, since their integrated graphics are ubiquitous and would presumably still be better than CPU-only?
There's also OpenCL of course but I don't know what the features/porting situation is like there.

Thanks for the quick response.

AMD is at least a decade behind

Without CUDA there is simply no way (again today, in the real world)

and again, AMD/Intel/etc GPU is a non-starter currently

Is there specific functionality that's missing from AMD's stack/HIP that precludes its use for the inference server, or would it be that it's potentially less performant/less pleasant to use but still could be sufficient on both counts (which is not to suggest that it wouldn't potentially be a significant amount of work to add support for other stacks)?

with the tiny exception... of a binary driver used by millions of people

We differ in opinion on how tiny an exception a very large closed-source kernel module is ;) Also Alexa is also used by millions of people, so... 😅

a very surface-level understanding from the perspective of someone unfamiliar with the (at least) tens of millions of dollars they have invested in the open source GPU compute ecosystem

I'm not immersed in GPU compute, you're right, I've just seen the ample evidence over the years of nVidia's failures to play nice in with OSS, e.g. with driver development (reclocking), upstream APIs (EGLStreams vs. GBM). And I know that at least it's not just a layman's opinion that they're pretty awful, as Torvalds so (in)famously expressed.

Let me try to put it simply - if you were able to get a pytorch "hello world" equivalent to run on AMD or Intel GPU compute reliably across multiple distros, kernel versions, etc over a span of months without spending days/weeks on one-off hacks, mailing lists, issue trackers, patches, etc on a regular basis you will have already achieved something the best ML engineers in the world have so far been unable to. One of millions of examples. Just to get pytorch to do some basic math on a GPU from AMD or Intel... At this point AMD's approach to GPU compute has been so comically inept I'm convinced they've conceded their position in this space. Intel may be able to do better over time but already requiring an ancient (in ML terms) specific fork and patch rev of pytorch is yet again, ridiculous and a complete non-starter.
Now start looking at higher level frameworks we leverage like ctranslate2, auto-gptq, etc that have zero support (or even plans, to my knowledge) for these platforms. Repeat this across dozens of components.
I don't want to seem dismissive, argumentative, disrespectful, or even patronizing but Willow is in a somewhat-strange position of clashing worlds - the GPU compute space where these are all well known, blatantly obvious facts and a user base who's only understanding of these issues (in many cases) comes from driving a display on their desktop - like your comment regarding EGLStreams.

Thanks for the additional context. In case you know offhand of any good deep dives on the (reasons for the) current state of compute on AMD/Intel I'd be interested to know.

No worries!
I'd like to point out I'm aware of these issues (and more, ugh) because I follow the "alternative" GPU compute platforms closely. I long for the day they're reasonably competitive because I don't like monopolies and I don't like the driver situation with Nvidia either. I truly wish there were a remotely competitive alternative in the space but practically speaking we're likely years away from that (if ever, sadly).
I don't know of any resources specifically but I already touched on the fundamental reason. Someone at Nvidia made the really hard, difficult, and expensive decision 15 years ago to say "We're going all in on GPGPU compute, on all of our hardware. Whatever it takes. Period.". Absent that level of consistent commitment and effort from the likes of AMD and Intel they will never get to where they need to be. Unfortunately they have the additional slow-mover challenge of going up against an entire generation of GPGPU engineers, frameworks, etc that have built on CUDA. As the issues I noted and more demonstrate they have an extremely difficult challenge.

I long for the day they're reasonably competitive because I don't like monopolies and I don't like the driver situation with Nvidia either.

It won't be that long. Definitely not in the next year, but probably a lot closer to one year than 10.
The "AI" world in investing exploded from nothing to everything with the publicity of Chat GPT. Like you said, Nvidia is a million miles ahead, but the competitors who viewed the field as niche have suddenly seen their death flash before their eyes. The amount of money that is about to be spent playing catch up is going to make our heads spin. But they will catch up. They always do.
Doesn't mean anything for WIS in the near-term, but the profits are too juicy to be ignored anymore, so Intel, AMD, and probably Apple are going to either show up or wither on the vine. I'd bet Amazon starts making acquisitions as well. And you never know what Elon is going to do next.


In parallel create new testhost if datacollection is enabled.

@mayankbansal018,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Wrong name resolution in intrusive-rs

I'm getting a similar error on intrusive-collections.
Steps to reproduce:

git clone https://github.com/Amanieu/intrusive-rs.git
Open in vscode
Open src/linked_list.rs

It will show the following errors, but only if the linked_list.rs file is open:
{
	"resource": "/home/amanieu/code/intrusive/src/linked_list.rs",
	"owner": "_generated_diagnostic_collection_name_#0",
	"severity": 8,
	"message": "Missing structure fields:\n- link1\n- link2\n",
	"source": "rust-analyzer",
	"startLineNumber": 1366,
	"startColumn": 21,
	"endLineNumber": 1369,
	"endColumn": 10
}
{
	"resource": "/home/amanieu/code/intrusive/src/linked_list.rs",
	"owner": "_generated_diagnostic_collection_name_#0",
	"severity": 8,
	"message": "no such field",
	"source": "rust-analyzer",
	"startLineNumber": 1367,
	"startColumn": 13,
	"endLineNumber": 1367,
	"endColumn": 30
}

linked_list.rs
Originally posted by @Amanieu in https://github.com/rust-analyzer/rust-analyzer/issues/2799#issuecomment-573356222

My wild guess is the intrusive_adapter! expand incorrectly. I will take a look later.

The issue is funciton local item support. The Obj struct resolves to the one in the mod tests scope instead of the function local redefinition.

So this is different symptoms of #1559

I am glad to hear there is no bug in that complicated intrusive_adapter! macro-rules ! 😃

@kiljacken We now support local items, and I don't see any diagnostics on the current version of that file. I'm not sure what commit you were using though, so I can't reproduce this precisely.

@jonas-schievink Not "my" issue, just split out the comment from @Amanieu. I can try to replicate if he's not around though, although I probably won't have much better luck then you.

This issue no longer reproduces, you can close it.


how to restart mongo-connector in linux ?

i have installed mongo-connector with this command  pip install mongo-connector now i wan to restart mongo-connector please help me

As far as I know, you can just kill the process.

@vimuston is correct. Ctrl-C or kill -SIGINT:
$ mongo-connector <args>
Logging to mongo-connector.log.
^C
$ mongo-connector <args>
Logging to mongo-connector.log.


Update Onboard-Windows-10-multi-session-device.md

Updated to Azure Virtual desktop

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/5547

@denisebmsft Please merge this PR. Thanks.

@denisebmsft Gentle reminder, please merge this PR. Thanks


Hidden CI failures

Some PR CI failures are not treated as errors by AZP and GH. This might lead to merging broken PRs.
E.g. https://dev.azure.com/vcpkg/public/_build/results?buildId=105435&view=logs&j=7922e5c4-0103-5f8f-ad17-45ce9bb98e80&t=8ceef4ed-9be5-594f-e707-4e1dd58a3d21

x64-windows failed but green.
x64-osx failed and red - it also has a ##[error]REGRESSIONS: output line.

This might lead to merging broken PRs.

This might will lead to merging broken PRs.

This did lead to merging broken PRs already: #40004.

Thanks for the fix.
(I was wondering if I should or shouldn't ping you directly @BillyONeal , but I thought that quick escalation must work in the regular proces, in particular in case you are not available.)

@dg0yt:

So:

stronger wording. might is too weak to catch attention.
PING all the maintainers when there is already a big fire! (in hope one opens the associated e-mail and sees the issue and escalates it)

We need to be loud on all channels especially the next three weeks.
(I am also kind of wondering why I don't have an Github E-Mail titled "Hidden CI failures" from Saturday)

(I was wondering if I should or shouldn't ping you directly @BillyONeal , but I thought that quick escalation must work in the regular proces, in particular in case you are not available.)

If pinging me directly works and usual channels do not that's my and my team's problem, not yours :)


Question: swagger-routes-express and express-openapi-validator

Hi , first of all thanks for open sourcing this. This is exactly what I'm looking for. I was wondering why do you need both swagger-routes-express and express-openapi-vaidator. It seems like they do the same thing which automatically creates the routes using the the open api schema? Or am I missing something.
Thanks in advance!

Hi @hlim7,
swagger-routes-express builds routes automatically to the functions exported in src/api/controllers/index.ts using operationId parameter as a function name for each path in config/openapi.yml.
express-openapi-vaidator, validates all query parameters, request and response body based on the schema in config/openapi.yml.
If it doesn't match the schema, it will respond to a client's request with a default response, or with the one you specify in src/utils/server.ts:
// error customization, if request is invalid
server.use((err: any, req: express.Request, res: express.Response, next: express.NextFunction) => {
  res.status(err.status).json({
    error: {
      type: 'request_validation',
      message: err.message,
      errors: err.errors
    }
  })
})

Hi @losikov I believe express-openapi-validator also has the option to connect the routes automatically using  x-eov-operation-id and x-eov-operation-handler in the openapi.yaml so if you specify it like this
paths:
  /hello:
    get:
      description: Returns 'Hello <name>/stranger!!!' to the caller
      tags:
        - greeting
      operationId: hello
      x-eov-operation-id: hello
      x-eov-operation-handler: controller/user

Here's the doc ref from express-openapi-validator re: how to use operationHandlers https://www.npmjs.com/package/express-openapi-validator#example-express-api-server-with-operationhandlers

@hlim7, thanks for showing it! I will try it.


Resolving deltas: 100% (3947/3947), done.
go: go.mod file not found in current directory or any parent directory.
'go get' is no longer supported outside a module.
To build and install a command, use 'go install' with a version,
like 'go install example.com/cmd@latest'
For more information, see https://golang.org/doc/go-get-install-deprecation
or run 'go help get' or 'go help install'.
go: go.mod file not found in current directory or any parent directory.
'go get' is no longer supported outside a module.
To build and install a command, use 'go install' with a version,
like 'go install example.com/cmd@latest'
For more information, see https://golang.org/doc/go-get-install-deprecation
or run 'go help get' or 'go help install'.
go: go.mod file not found in current directory or any parent directory.
'go get' is no longer supported outside a module.
To build and install a command, use 'go install' with a version,
like 'go install example.com/cmd@latest'
For more information, see https://golang.org/doc/go-get-install-deprecation
or run 'go help get' or 'go help install'.
$ go install github.com/mattn/go-sqlite3@latest; go install golang.org/x/net/publicsuffix@latest; go install github.com/miekg/dns@latest
go: downloading github.com/mattn/go-sqlite3 v1.14.17
package github.com/mattn/go-sqlite3 is not a main package
go: downloading golang.org/x/net v0.15.0
package golang.org/x/net/publicsuffix is not a main package
go: downloading github.com/miekg/dns v1.1.56
go: downloading golang.org/x/sys v0.12.0
package github.com/miekg/dns is not a main package

go version go1.20.3 linux/amd64

I build it and its not working however if you want to try
make a new folder, cd to that folder then git clone  https://github.com/google/namebench.git src/github.com/google/namebench *without the export GOPATH=pwd then paste the go get cmds. if it throws $go: go.mod file not found in current directory or any parent director" : go mod init namebench then paste the go get commands again. if this error showed up: "$GOPATH/go.mod exists but should not" then do: export GOPATH=path to the directory and finally try the go get cmds again and it should be fine. and after its done building : ./namebench and it shows an url, paste it in your browser.