Feature request: Support URI Templates

My APIs return hypermedia enabled responses. Specifically, they conform to the Hypertext Application Language (HAL) specification. The JSON version of the response contains a _links property where each property of _links represents a set of (possibly templated) href. These links guide the client on what may be done with the response.
As an example, imagine that we are working with a role (in the RBAC sense) and a role contains a set of members. To remove a member from a role, the response may include { _links: { "remove": { "templated": true, "href": "http://localhost/roles/{roleId}/members/{memberId}" }}}. I need a way to reify the template into its final form before Karate makes the call.
Karate currently supports path and query parameters; it does not support fragments, matrix parameters (i.e., Path-Style Parameter Expansion) or embedded path expansions. It also makes the assumption that all path parameters will be appended to the original URL rather than replacing certain path segments.
One solution is to introduce two new functions to com.intuit.karate.Actions: uritemplate and uriparam.  This would ensure we are not making any breaking changes to the currently installed code-base. An attempt to use both url and uritemplate in the same scenario would result in a test failure. Likewise, attempts to use uriparam and either param or path in the same scenario would result in a test failure.
Another solution would attempt to infer if the url is templated. If it is then all calls to param can be assumed to apply to any requested expansion in the template. Any call to path would result in a test failure since this call only provides a value when the template needs a key and value.
One concern I have with the adoption of this request is that the HAL specification includes the notion of curies. I don't have a concrete way of incorporating this concept into Karate at this time. I don't use curies and I'm hoping a simple implementation of URI Templates will suffice.
References:
RFC 6750

@fdutton haven't gone through the RFC in detail. but first a question.
assume you wrote a re-usable function responseToUrl(). this would use response._links and do whatever processing is needed.
then in your tests you simply do:
* url responseToUrl()`

wouldn't that work ?

and one more hint. since karate JS is now ES6 and supports variable interpolation within backticks, this would work. so maybe it is as simple as doing karate.eval() after pre-pending some $ symbols at the right places
* def roleId = 'foo'
* def memberId = 'bar'
* def after = `http://localhost/roles/{$roleId}/members/{$memberId}`

The first example would actually have to be somethings like
* url responseToUrl({ key1: val1, key2: val2 ...})
At this point I would be asking myself why I'm using Karate since I can do this Java.
In regards to the second example, one does not have the string literal http://localhost/roles/${roleId}/members/${memberId}. What you actually have is response._links.remove.href

@fdutton sure, by all means please just use java ;)
okay but seriously - how about you start out with a Java utility. once you see that working, you may find it more than sufficient for your use case. if you figure out the code and share it, we can then consider if it needs to be part of core and whether it is generic enough etc.
from my side - I am against adding syntax bloat to karate unless there is a very good case to be made, so far I'm not convinced
and by the way, this works:
* def first = 'foo'
* def second = 'bar'
* def template = '${first}/${second}'
* def result = karate.eval('`' + template + '`')
* match result == 'foo/bar'

Is it possible for Java utility classes to have access to what is currently defined in the Karate script? Otherwise, it would be nearly impossible to support a URI Template like http://localhost/roles{?like,size,page,sort} without passing all of the variables to the utility function. e.g.,
* def URITemplate = Java.type('my.URITemplate')
* def endpoint = URITemplate.resolve(response._links.search.href, { like: 'test', size: 20 })
* match endpoint == 'http://localhost/roles?like=test&size=20

@fdutton personally I think it is ok to pass all the required variables as a JSON the way you showed in your previous example. in JS you can do karate.get('varname') any time, but to keep things simple, please assume that you can't do such things in java. personally I would just write a JS function, put it in karate-config.js so that it is global - and call it a day. if you insist on using java, you could use a JS wrapper function that extracts all the variables you need so that your use in a feature is a one-liner. there are many other possibilities - one being that you can call ScenarioEngine.get() which is a thread-local - but I discourage that as undocumented / internal API
closing as won't fix, discussion can continue if needed

The reason for using Java is that URITemplate implementations already exist. I have one I wrote in TypeScript; I'll see if I can convert that one to JavaScript.
Thanks for the recommendations. If you change your mind about supporting this at the language level, let me know and I'll submit a pull-request.

sure. I hope you got the point about using JS to "massage" karate params into what for e.g. the URITemplate java code you have expects. here is an example: https://twitter.com/getkarate/status/1359369240227106816


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

Yup


Don't load profiler collector in not dev environment

Fixes #910

Hi @anyt
Yes, you are right.
The following snippet resolved my memory leak problem. By default profile is enabled in prod, but disabled in test  :confused:
# config/config_prod.yml
 ...
framework:
    profiler:
        enabled: false


Fix to prevent start listening on error pages

@amit-mb We already have a test case. So we don't need to add new one instead we might need to update one. Once you fix the other comment. lets discuss.


Use Pin Name instead of Pad number

It would be great to add the possibility of using Pin Name instead of Pad number (in some cases they do not match and the correct one is the Pin Name)
It might also need some kind of filtering as Pin Name could be "GPIO26" and we only want "26".
It could be implemented as a checkbox next to the dropdown menu.
I'm not quite sure how to do it, but I'd love to help.
You can contact me on Discord at PumaFPV#7895

Thank you for this issue!
By Pin Name, you mean the pin name in the schematic? Assuming my understanding is correct, I completely agree with the necessity. A quick research revealed that pad.GetPinFunction() does exactly what we need for this.
The big question, however, is the UI. I agree with adding a checkbox "Use pin name instead of number". But the filtering part is a bit more annoying to implement in a way that keeps the UX simple. I will implement the first part ASAP, and will think about the rest later.
Note: don't worry about contributing to the code base yet, it is currently in active development. Opening an issue to request features is already a big contribution!

Yup, the one that is inside of the part schematic. Nice! I dont know where to find the availabe functions...
Maybe you could make a dropdown menu appear when the checkbox is checked, which would allow you to select what kind of filtering you want (none, only keep numbers, only keed what's in front of the "/", what's behind...)
Attached is a picture of the component I'm thinking about

Btw it could also be great to be able to not write the power/unused pins instead of commenting them. (Maybe new issue?)

I finally got to implementing this feature as of https://github.com/CGrassin/kicad-pinout-generator/commit/851d7c9376290438e7e7dc71694ae605f44f4056. It was a bit tricky to make it not too confusing for the user, and there might be some rework in the future to make it more understandable...

There are now two additional controls in the interface:

A checkbox, only active when C enum, C define or python format is selected to chose whether to use the pin name, or the pin number.
A filter text box, only active when the checkbox is checked, that acts a filter. In your case, you would enter "GPIO". If left empty, the full pin name is used.


Wrong page indexing for an incomplete dataset

For certain incomplete datasets (https://github.com/mehta-lab/recOrder/issues/320), the current iohub (or waveorder.io) would gather the wrong page number (the first page has page number of 511):
0    /hpc/projects/comp_micro/rawdata/falcon/zebraf...
1                                                  511
2                                                  162
Name: (0, 0, 0, 0), dtype: object

The byte offset is also wrong because MicromanagerOmeTiffReader uses hard-coded magic numbers to determine page offsets:
https://github.com/czbiohub/iohub/blob/0fdca8bad97c8a4c6a1b69e54923c576a7e6b44c/iohub/multipagetiff.py#L122-L142
This results in reading the wrong image for the first frame:
>>> reader.get_image(0,0,0,0).mean()
30643.720915794373

And TiffFile('/path/to/first/image/').asarray()[0].mean() would return average intensity of 194, which is consistent with the MM GUI readout.

This results in reading the wrong image for the first frame

This raises a deeper concern about the OME-TIFF reader: the custom byte offset and memory mapping scheme can read any arbitrary bytes and will not give any notice even if the region on the disk does not contain valid array data.

Since the dataset did not finish acquiring, the micromanager index map may be longer than the actual tiff page list. For example in file Pos4_39.ome.tif', the channel index map (used by iohub to determine page count) is:
array([4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 5, 5, 5, 5,
       5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5,
       5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5,
       5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5,
       5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5,
       5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
       0, 0, 0, 0, 0, 0], dtype=uint32)

This array has a shape of (512,).
However there are only 130 pages in the file. Note that is that MM actually preallocates space for the (planned) complete acquisition, so that this incomplete file has the same 4 GB size as if it was a full TIFF file with ~500 pages.


all items selected when i add options via jquery on IE

Hi , i have an issue i think it's a bug , so i fill  a select with jquery and it work but when i fill on select with the data-live-search="true" attribute it will not work on IE , it will select all items by default and it will be impossible to select an item ,
here is a demo of the issue ,
so if you select region2 it will be impossible to select a city on IE
http://jsfiddle.net/nr54vx7b/17/
Thank you .

I have the same problem with IE 10 and 11.

Same. This is preventing us from upgrading from 1.5.4, which we'd like to do to get the fix for empty lists throwing a JS error.

Same for me. Problem occured by adding new Options to an empty sleect, or by adding Options after calling the .refresh() function in Javascript.
Workaround:
1.) Add your Options
2.) .selectpicker('refresh');
3.) .selectpicker('deselectAll');
4.) .selectpicker('refresh');
Updated "rizerzero"'s fiddle:
http://jsfiddle.net/nr54vx7b/63/

I have also same issue, data-live-search="true" attribute not working in IE, When click on select field, browser getting hang. If we remove data-live-search="true" attribute, Select drop down working fine.

What exactly is the issue here? I know there are issues with liveSearch when using liveSearchPlaceholder in IE, but that doesn't appear to be the issue here. Using the example in the original message, things appear to be functioning properly.

When rebinding the list of options and calling refresh, in IE, the data-original-index attributes on the  elements were all being set to 0 instead of properly incrementing, resulting in every item in the list being marked selected and active once refresh was called. You can see a video of this behavior here: http://screencast.com/t/xh2BwBFo6sWN
This behavior was introduced in 1.6.3, which was the current version at the time of the original posting, and appears to have been fixed in 1.6.4.

@steinhoff1989 Thanks, man! It worked for me.


arXiv meta data retrieval

Implement arXiv as an information provider (see #254).
arXiv will be a verified provider, so all records retrieved from arXiv will be verified (see #252).
Need to study the arXiv API:
https://info.arxiv.org/help/api/index.html

The arXiv service should also normalize the URL. arXiv URLs can point to PDF (https://arxiv.org/pdf/2408.01417) or abstract (https://arxiv.org/abs/2408.01417). We should process PDF URLs to point to the abstract page

It important o take into account that arXiv also adds versioning information to the URL, when the a specific version is linked to

This feature has already been released to production. Let me know if there are any bugs.


Client must print understandable errors

[aptomi] aptomictl policy delete -u Sam -f examples/03-twitter-analytics/policy/                                         10:42:01  ☁  master ☂ ⚡ ✭
FATA[0000] Error while deleting policy: received object kind error doesn't match expected policy-update-result

Same thing happens when you trying to do operation that requires user passed but it isn't


Transcripts / Captions Per Video - en

Seeing this message on the Video Engagement tab.  Unfortunately, clicking on the message doesn't provide any additional data.

In the three-dot button, you can refresh the data. After you click that, is it working?
Can you provide logs for Superset and Clickhouse?

I'm seeing this on the beta site, so I don't have access to the logs.  I'll try refreshing and see what happens.

Marking this one as stale, we haven't seen it since the Superset 3.0 upgrade. @e0d ping the ticket if you happen to see it again and we'll keep digging.

Still haven't seen this recently, so I'm going to close it.


Unable to build for Xcode

The installation instructions tell me to use the included project file if I'm using Xcode, but it doesn't say which project file, so I'm assuming I should clone the project, build and use the Nifty.framework.
So I tried that, but I get the following errors:
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_fft.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_cumsum.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_sort.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/find.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/interp1.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_poly.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_cumprod.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_repmat.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/TimeFrame.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_linspace.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_shuffle.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_ifft.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/TimeSeries.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/DataFrame.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/DataSeries.swift'
<unknown>:0: error: no such file or directory: '/Users/rodrigoruiz/Downloads/Nifty/Sources/_meshgrid.swift'
Command /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swiftc failed with exit code 1

Sorry! You caught me in the middle of cleaning up some old files. I usually just build on Linux so I forgot to update the Xcode project to not try to build old files. It's been fixed so if you try again it should work. I do apologize!


Error throw when accessing queryAssignedNodes getter (No version >2.4.0)

Description

The issue was fixed and merge to master but no new tag/release
was pushed.
I don't want to use master branch installation.

What is your protocol for these fixes?

Closing as this fix has been released.


add logging example page

Adds an example page for the log filter, which allows you to log any data on the page to the console.

closing as its a long way from master at this point


we very very need support for 32 bit executables!

Windows build number:  10.0.17035.1000

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

we very very need support for 32 bit executables!
I really wonder when the great WSL can make it 'cause we can't WAIT for it any more!

Many distribution now does not or will not release 32 bit os. Why did you want 32 bit support?

Thanks for the feedback. This is covered by #711.
You can also vote here on this feature: https://wpdev.uservoice.com/forums/266908-command-prompt-console-bash-on-ubuntu-on-windo/suggestions/13377507-please-add-32-bit-elf-support-to-the-kernel.

sorry but i'm still a student. WSL is really convenient and i really like it. Howevet
r, when we learn the course, ICS, we need to run the 32 bit executables to learn what happened to the computer.


ED25519 key fingerprint is SHA256:DrkULQhbEmZoSWyHJDXncdZxbGS6UfdXX9nKAvyZJSo.
This key is not known by any other names
Are you sure you want to continue connecting (yes/no/[fingerprint])? yes
Warning: Permanently added '<IP_ADDRESS>' (ED25519) to the list of known hosts.
Welcome to Ubuntu 18.04.6 LTS (GNU/Linux 5.4.0-1065-aws x86_64)

 * Documentation:  https://help.ubuntu.com
 * Management:     https://landscape.canonical.com
 * Support:        https://ubuntu.com/advantage

  System information as of Thu Mar 17 10:32:15 UTC 2022

  System load:  0.6                Processes:           199
  Usage of /:   2.0% of 116.27GB   Users logged in:     0
  Memory usage: 4%                 IP address for ens5: <IP_ADDRESS>
  Swap usage:   0%

 * Ubuntu Pro delivers the most comprehensive open source security and
   compliance features.

   https://ubuntu.com/aws/pro

6 updates can be applied immediately.
To see these additional updates run: apt list --upgradable

The programs included with the Ubuntu system are free software;
the exact distribution terms for each program are described in the
individual files in /usr/share/doc/*/copyright.

Ubuntu comes with ABSOLUTELY NO WARRANTY, to the extent permitted by
applicable law.

$


coverage v3 fixaloo

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

Comparison is base (59b4098) 82.68% compared to head (45a45dc) 71.94%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master      #58       +/-   ##
===========================================
- Coverage   82.68%   71.94%   -10.75%
===========================================
  Files           5        9        +4
  Lines         179      474      +295
  Branches       22       74       +52
===========================================
+ Hits          148      341      +193
- Misses         11       90       +79
- Partials       20       43       +23

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


Boolean attributes as strings

What's the best way to handle boolean attributes that are passed around as strings? For example, I have a component that needs to pass an attribute down to a child component. Something like this:
define(class extends Component {
  static is = 'parent'

  static props = {
    foo: prop.boolean({ attribute: true })
  }

  renderCallback() {
    const { foo } = props(this);

    return [
      <child foo={foo}></child>
    ]
  }
});

define(class extends Component {
  static is = 'child'

  static props = {
    foo: prop.boolean({ attribute: true })
  }

  renderCallback() {
    ....
  }
});

what's the best way to handle passing the attribute around? I want to be able to pass the attribute like I am in that renderCallback
<child foo={foo}></child>

but this ends up passing it as a string, which is then always true. Is there some way to do this, but pass it as a boolean instead?

It seems like you can set the true state correctly...
<child foo={foo ? 'foo' ? ...}></child>

but how do you model the false state? Should the ... above be undefined

Never mind. Using undefined for false seems to work. The syntax isn't ideal but it does work.

What's not ideal about the syntax?

I'd love to be able to just pass it a Boolean directly:
<child foo={foo}></child>

instead of having to pass it like this:
<child foo={foo ? 'foo' : undefined}></child>

but that's not a big deal.

@alexlafroscia Skate follows standard HTML behaviour for string and boolean attributes. For example, true boolean attributes work just fine: http://www.webpackbin.com/4JDv3FdtM. If you need to represent a boolean as a string, you'll need to write a custom prop for that that uses coerce / serialize / deserialize to check specific values you're wanting to represent falsy and truthy.

Yeah, totally makes sense 😄  I was confused because it works differently than React, but once I read up more on how real HTML attributes work, I was able to answer my own question. Thanks!

Hmm

Thanks for that example by the way, I didn't realize that you can just pass the property directly like that. When I was encountering the issue with them coming through as strings, I was using .setAttribute from outside the component to set the values, and I think that that was turning it into a string as the follow was set in the DOM. It's great to know that within the components, you can just pass the values directly, like your example shows.

Hi
I am having some problems passing boolean values when i create the custom elements.
I have a ColumnLayout component from Material UI.
ColumnLayout has container and item as boolean values.
I created my custom element and specified my props for container and item as skateProps.bool
When i try to use it in an example, lets say
<column-layout container spacing=24>
    <column-layout item xs=12>

some value

    </column-layout>
    <column-layout item xs=12>

some value

    </column-layout>
</column-layout>

If i try to pass the container and item values like that, they are always undefined.
If change the skateProps type to string and pass the values as
<column-layout container=true spacing=24>
    <column-layout item=true xs=12>

some value

    </column-layout>
    <column-layout item=true xs=12>

some value

    </column-layout>
</column-layout>

It works like that but React gives warning because it expects a bool type but it receives as string.
How can pass the bool value?
Thank you.


Cannot call this API with an app-only principal.

Cannot call this API with an app-only principal.
What kind of permissions are needed for these commands Set-PnPSiteVersionPolicy -EnableAutoExpirationVersionTrim $false -MajorVersions 100  -ExpireVersionsAfterDays 0  -MajorWithMinorVersions 0

Hello @harshdamaniahd
I tried with App only approach and got the same error.
Required permissions for this command is

Type
API/ Permission Name
Admin consent required

Delegated
AllSites.FullControl
yes

With below approach using parameters ClientId, DeviceLogin, LaunchBrowser
Connect-PnPOnline -ClientId xxxxxxxx-0000-4xxxx-xxxx-xxxxxxxxxxx -Url "https://xxxx.sharepoint.com/sites/xxxx" -DeviceLogin -LaunchBrowser -Tenant xxxxx

I was able to execute this succesfully
PS C:\Research\PnPCommunity\powershell> Set-PnPSiteVersionPolicy -EnableAutoExpirationVersionTrim $false -MajorVersions 100 -MajorWithMinorVersions 10 -ExpireVersionsAfterDays 200
WARNING: The setting for new libraries takes effect immediately. Please run Get-PnPSiteVersionPolicy to display the newly set values.
WARNING: The setting for existing libraries takes at least 24 hours to take effect. Please run Get-PnPSiteVersionPolicyStatus to check the status.
WARNING: The setting for existing libraries does not trim existing versions.
PS C:\Research\PnPCommunity\powershell>

@gautamdsheth I will add required permissions for these two commandlets in the PR #4488
Thanks,
Nish

Thanks I should try this


Improve error message when config db is not initialized

An error message should be improved when config db is not initialized, it is still suggesting to use vmdkops_admin.py even though we have esxcli integration now.
After esxcli integration with our vDVS driver(admin CLI) following message is not appropriate.
[root@promc-2n-dhcp105-97:~] esxcli storage guestvol vmgroup access ls --name=_DEFAULT
Configuration not found. Please init configuration with 'vmdkops_admin.py config init'.

user will be confused easily by Please init configuration with 'vmdkops_admin.py config init even though it is possible to initialize config through esxcli.
We should provide a generic message or a message like Configuration not found. Please init configuration with 'vmdkops_admin.py config init' or 'esxcli storage guestvol config init' (if using esxcli) or just "esxcli" command.
cosmetic fix required at https://github.com/vmware/docker-volume-vsphere/blob/8de97721d2f1ff1e62d0f918c502450782134072/esx_service/utils/error_code.py#L134
 ErrorCode.INIT_NEEDED: "Configuration not found. Please init configuration with 'vmdkops_admin.py config init'.",

/CC @pdhamdhere @tusharnt

Closed through #1836


Support querying all messages in private conversations without specifying friends' ID

Closed because I just found that turms-service has implemented the feature...


Build Fail (V2.7.0)

bootstrap-slider.js is not in repo, and not installed by npm
Running npm install then grunt I get:
Running "requirejs:builder" (requirejs) task
>> Error: ENOENT: no such file or directory, open
>> '/users/me/storymap-shortlist/src/lib-app//bootstrap-slider/js/bootstrap-slider.js'
>> In module tree:
>>     storymaps/tpl/BuildConfigBuilder
>>       storymaps/tpl/core/MainView
>>         storymaps/common/ui/autoplay/Autoplay

Thanks for reporting @regan-sarwas, I just pushed up the appropriate files

That did it.  Thanks.


Dependency convergence

Fixes #2534
Closes #2238

@natros could you have a look before I merge this? (As original author of these changes)

@natros could you have a look before I merge this? (As original author of these changes)

Hi,
It has been a long time (3 yrs) since I made that PR. I haven't been using maven for a while, but I'm still using querydsl every day.
I think you need to update junit to 4.13.1 for security reasons.
You have been doing a good work keeping this project alive.
LGTM
Thank you!

Hi,
It has been a long time (3 yrs) since I made that PR. I haven't been using maven for a while, but I'm still using querydsl every day.
I think you need to update junit to 4.13.1 for security reasons.
You have been doing a good work keeping this project alive.
LGTM
Thank you!

Thanks!

Thanks!


Question: Docs suggest old jquery version 1.11.1?

The docs suggest using jquery from https://code.jquery.com/jquery-latest.js, but this url points to jquery version 1.11.1 (from 2014). Is jquery terminal developed against jquery version 1.11.1? If not, perhaps the docs should be updated? For instance, https://cdn.jsdelivr.net/npm/jquery or https://code.jquery.com/jquery-3.5.1.min.js both point to up-to-date minified copies of jquery.

Thanks for the report will update the docs, I didn't use that url for a long time. I usually use jsdelivr because it's fast and it also allow to use files from GitHub.

The link was updated. Thanks.

https://blog.jquery.com/2014/07/03/dont-use-jquery-latest-js/

Thanks for your quick response!

Yes, I was not using that because it was outdated, but I've forgot to update the docs.


Add feature: View Transition API

This feature makes it possible to create real, native "page transitions". From this article:

The View Transition API makes it easy to change the DOM in a single step, while creating an animated transition between the two states. [...] View Transitions give you an easier way, by allowing you to make your DOM change without any overlap between states, but create a transition animation between the states using snapshotted views. [...] Additionally, although the current implementation targets single page apps (SPAs), this feature will be expanded to allow for transitions between full page loads, which is currently impossible.

See draft spec: https://drafts.csswg.org/css-view-transitions-1/

Ahh, sorry! I did search but I did not find that issue. Closing this. Thanks.


Fix HasModifiedFiles to check layer by layer

(almost) Complete rewrite of the check
Ignore any path that starts with "etc/" or "var/"
Ignore any RPM file that has config, doc, or config(missingok) flags
Check each layer against the last, rather than all the flattened layers at once

Fixes #846

Coverage: 81.966% (-2.2%) from 84.186% when pulling 4ed1645a6feb878e0e3d4fdb6cd3aa192054dc3e on bcrochet:hmf-to-main into 6726317abf54188d020cb563f89be36d5710c607 on redhat-openshift-ecosystem:main.


If spans come in with existing Sample Rate > 1, we should annotate them

Is your feature request related to a problem? Please describe.
A customer had traces that were matching a rule, but had different sample rates. We eventually figured out that it was due to upstream telemetry setting sample rate (head sampling) but it took a while to get there, because the field "meta.refinery.original_sample_rate" was blank.
This was because we only populate that field if we're going to change the upstream value -- but here Refinery's desired rate was 1, so we weren't changing it.
We should either:

Populate that field anytime it's not 0
Populate some other field that indicates that it was set upstream

Describe the solution you'd like
I suggest the first one.
Describe alternatives you've considered
Not being confused.
Additional context

I agree, let's populate it any time it is not 0.


Sync Seasocks back to upstream.

Just syncing back to the latest Seasocks now that https://github.com/mattgodbolt/seasocks/pull/129 is merged. I want to be on upstream as much as possible such that I don't wonder why I'm on a fork somewhere in the future.
Fyi @jasonimercer, @efernandez, we'll sync internal only when there's other changes we want.

:+1:


Null safety version

possible?

Thanks for reminding!!!
It's now support null-safety..
Closing this now feel free to raise any issues if you face


Have the bot automatically increase the delay

If ArchiveBot hits too many 5xx or 0 NetworkTimedOut errors in a row (or hits them too frequently), it should automatically increase the delay by 50-200. This would require less user intervention, but may cause problems if a site being archived links to a dead site or a site that blocks ArchiveBot (like vine.co) and should probably be togglable through a command (e.g. !autodelay d0jkf5u0j2d62iqzq2tskn552 off).

Possible duplicate of #83? Maybe this issue should be about new !autodelay.

Yeah, duplicates #83.


Symbol literals with interpolation are not allowed in hash patterns

Found this while importing your tests into YARP. On this line:
https://github.com/mbj/unparser/blob/8260882beae39cd3d88a29625d07dd31bbc66ece/test/corpus/literal/pattern.rb#L20
It's actually a syntax error.
case foo
in {"#{"a"}": 1}
end

Running ruby -c gives:
test.rb: test.rb:2: symbol literal with interpolation is not allowed (SyntaxError)
in {"#{"a"}": 1}
    ^~~~~~~~~

Interesting, whitequark/parser seems to accept it right now, producing this AST, its accepted in 2.7, 3.0, 3.1 and 3.2 here.
(case-match
  (send nil :foo)
  (in-pattern
    (hash-pattern
      (pair
        (dsym
          (begin
            (str "a")))
        (int 1))) nil nil) nil)

On MRI: Ruby 2.7 accepts it. But 3.0, 3.1 and 3.2 not.
Unparsers goal was to be able to unparser whitequark/parser so it does the job "right". This pattern tests are extracted from whitequark parser corpus. So you should have found it also via your test extraction.
I think we have to file this issue in whitequark/parser. Once they remove support unparser will also.

For full visibility, I've reported this in: https://github.com/whitequark/parser/issues/919. I do not want to filter it out unparser side for the moment. I'll do so with a fixing parser release.

@kddnewton if you are importing unparsers tests, you can just ignore this pattern, or would it be better for you I pushed a removal?

That's okay! I'm just ignored that one line for now. I appreciate the explanation!

Well I'd be happy to use YARP in the future as source AST for unparser. So thanks to you.
I hope you made dstrings less ambigious in your native AST, I'll check it out soon. Dynamic strings are the only construct that unparser fails on in edge cases right now.

@kddnewton yes helps me already, the story goes a bit deeper. I'll reply here later explaining it.


adds a codecov file.

Adding a codecov file to adjust slightly what "good" and "bad" coverage is for the repo.

Codecov Report

Merging #50 (bcb5f61) into develop (ae291c5) will decrease coverage by 0.42%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #50      +/-   ##
===========================================
- Coverage    67.89%   67.46%   -0.43%
===========================================
  Files            9        9
  Lines         3124     3083      -41
===========================================
- Hits          2121     2080      -41
  Misses        1003     1003

Impacted Files
Coverage Δ

idelib/transforms.py
68.87% <0.00%> (-1.16%)
:arrow_down:

idelib/unit_conversion.py
74.23% <0.00%> (-0.77%)
:arrow_down:

idelib/matfile.py
20.69% <0.00%> (-0.43%)
:arrow_down:

idelib/dataset.py
78.99% <0.00%> (-0.26%)
:arrow_down:

idelib/parsers.py
81.78% <0.00%> (-0.15%)
:arrow_down:

Continue to review full report at Codecov.

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

why did this get closed on approval?

That's how github responds when you (or at least, I), use the gitflow plugin.


8.3 Pascal Triangle- Type 2 ques (Optimal Approach)

My code is clear and well understanding
Please assign it to me

Need to add a description and your approach. Thanks, Closing as of now.


Feat fetch pokemon from service

Known issues:

pictures mostly don't work, this is due to the links returned by the listservice
pokemon viewer currently can't display all pokemon, this is due to the viewer using the fallback list to retrieve information instead of its own fetch call - to fix this, we need to either update the fallback list (necessary anyway) or refactor pokemon viewing to also use the listservice fixed
pokemon viewer uses placeholder icons until fixed pokemon picture links are available

Added Do Not Merge until new pokemon pictures have been tested and added


Choppy/lagging airhorns

So I just added this bot to my discord server and every time we use one of the airhorn commands we can hear it, but very laggy/choppy. My server region is Central Europe so I don't see a problem there.

Nevermind it's smooth now.


test/test_unit.py: replace hardcoded "python3" executable name

do not use python3, rather use sys.executable to use the same python version this test is running with

Should I rebase, or will this interfere with the CI?

Rebace works fine with GHA, in fact is always a good idea to rebase.

Hooray, everything green! Can this be merged?

Can someone merge this? Before I have to rebase again :-)


[enhancement] strengthen the IPC module

After some discussions last week with mlcache users, it appears that several improvements could be made to the IPC module in order to make it more robust for intensive use-cases:

[x] First of all, document the timeout argument in the cache:update() method.
[ ] Initialize self.idx not in new(), but at the time of the first call to poll(). The reason behind this is that users creating their cache (and IPC module) in init_by_lua will not benefit from the fix implemented in #88. When a worker crashes and the master process spawns a new worker, the init_by_lua context is not called, and the IPC new() method is not called again. This causes respawned workers to replay all events (events which have probably been evicted from the shm since then).
[x] Add a guard in case self.idx >= shm_idx
[ ] Investigate a "max timeouts threshold" values: when poll() timed out N times in a row, maybe we should bump self.idx all the way to shm_idx (thus assuming that all events in between self.idx and shm_idx have been discarded). This allows resuming normal polling operations, but means that some events may be left unprocessed.

About the last item: an alternative, to not drop events, would be to run a binary search between the two ids

@juancampa If you have a minute to spare, let me know what you think of #97 to address 2. here.
Not sure if we should try to do something about 4. anymore...


TR3 - Item Randomizer

[x] Type Rando
[x] OneLimit
[ ] Positions
[x] Key Items
[ ] Ammo allocation based on enemy difficulty for unarmed levels

Some thoughts on auto-generated item locations.

Have a defined list of banned rooms per level - i.e. can't be reached without glitches
Ignore any flipped rooms
If randomizing secrets, ignore reward (final) room
Ignore rooms that have no collisional portals (like 15 in Crash Site)
For all others, scan every sector and:

Ignore if shared with any other entity (other than ones we're relocating)
Ignore if shared with a collidable static mesh (are there any that span multiple sectors?)
Ignore if inside a wall (sector.Floor == -127 && sector.Ceiling == -127)
Ignore if RoomBelow != 255
Ignore if slidey slope ((BoxIndex & 0x7FF0) >> 4 == 2047)
Scan FD:

Ignore if underwater current trigger (maybe lower strengths permitted)
Ignore if pickup trigger, maybe? Would avoid items ending up with secrets at least.
Ignore if FDKillLaraEntry

This should give us all tiles Lara can stand on (or swim to) I think. For those with slant or triangulation FD, we'd need to work out the suitable pickup point, but for flat sectors it could just default to centre-tile (although it could even be randomly placed within flat tiles).
The manual part would be going through each level and listing banned rooms. That's 2727 across the game. More preferable maybe than tens of thousands of individual locations to plot/validate. But of course it relies on the above rules being completely accurate. We could either have a utility to generate the JSON and load it in during randomization, or if the algorithm is reasonably fast, just eliminate the large JSON file. Perhaps there should be an exclusions file for anything that can't be covered with rules easily.
This is a list of non-flipped rooms for all levels.
nonflippedrooms.json.txt

Progress for listing banned rooms and locations (for auto-generated locations):

[x] Jungle
[x] Ruins
[x] Ganges
[x] Kaliya
[x] Coastal
[x] Crash
[x] Madubu
[x] Puna
[x] Thames
[x] Aldwych
[x] Lud's
[x] City
[x] Nevada
[x] HSC
[x] Area51
[x] Antarctica
[ ] RXTech
[ ] Tinnos
[ ] Chamber
[ ] Hallows

Total valid locations so far: 18,720


RFC: Add a history button to the dashboard kind of like Cacti

This pull request should supercede #123.
@spellik: Please do look over this commit. I manually squashed your 3 commits in the previous pull request and removed all of the bonzai bits while keeping your name as the author so you get credit.
I then fixed a bug in that commit where graphs without title names would show up as undefined 1 hour, undefined 1 day, etc using a fancy javascript ternary operator. Since zoom support was removed, it didn't make much sense to keep the "Cacti" button, so I removed that as well and put the History button under "Graph Operations".
@nleskiw, @mleinart, and/or @obfuscurity please do give this a shot and merge if you like it. It certainly works well in master.

@SEJeff Are you still interested in seeing this go in?

Changed applied to master in #2100.


Is there a roadmap for TFRT to replace the current runtime of TensorFlow?

Hi all, I've seen the post which claims that TFRT will replace the TensorFlow existing runtime. So is there a roadmap or any progress for the replacement process?

Hi,
Thank you for checking in. Please see this thread
https://groups.google.com/a/tensorflow.org/g/tfrt/c/yrunkxBy_as/m/qUWOIoInDAAJ
for more context.
On Wed, Jan 13, 2021 at 8:30 PM Niu Chong<EMAIL_ADDRESS>wrote:

Hi all, I've seen the post
https://blog.tensorflow.org/2020/04/tfrt-new-tensorflow-runtime.html
which claims that TFRT will replace the TensorFlow existing runtime. So is
there a roadmap or any progress for the replacement process?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/runtime/issues/34, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHMYR3CI75IDDYVFCL4H2TSZZXORANCNFSM4WB22WDQ
.

Hi,
Thank you for checking in. Please see this thread
https://groups.google.com/a/tensorflow.org/g/tfrt/c/yrunkxBy_as/m/qUWOIoInDAAJ
for more context.
On Wed, Jan 13, 2021 at 8:30 PM Niu Chong<EMAIL_ADDRESS>wrote:

Hi all, I've seen the post
https://blog.tensorflow.org/2020/04/tfrt-new-tensorflow-runtime.html
which claims that TFRT will replace the TensorFlow existing runtime. So is
there a roadmap or any progress for the replacement process?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/runtime/issues/34, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHMYR3CI75IDDYVFCL4H2TSZZXORANCNFSM4WB22WDQ
.

Thanks, I will follow the thread!

Hi, Thank you for checking in. Please see this thread https://groups.google.com/a/tensorflow.org/g/tfrt/c/yrunkxBy_as/m/qUWOIoInDAAJ for more context.
…
On Wed, Jan 13, 2021 at 8:30 PM Niu Chong @.***> wrote: Hi all, I've seen the post https://blog.tensorflow.org/2020/04/tfrt-new-tensorflow-runtime.html which claims that TFRT will replace the TensorFlow existing runtime. So is there a roadmap or any progress for the replacement process? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#34>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHMYR3CI75IDDYVFCL4H2TSZZXORANCNFSM4WB22WDQ .

Thanks, I will follow the thread!

Hi, Thank you for checking in. Please see this thread https://groups.google.com/a/tensorflow.org/g/tfrt/c/yrunkxBy_as/m/qUWOIoInDAAJ for more context.
…
On Wed, Jan 13, 2021 at 8:30 PM Niu Chong @.***> wrote: Hi all, I've seen the post https://blog.tensorflow.org/2020/04/tfrt-new-tensorflow-runtime.html which claims that TFRT will replace the TensorFlow existing runtime. So is there a roadmap or any progress for the replacement process? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#34>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHMYR3CI75IDDYVFCL4H2TSZZXORANCNFSM4WB22WDQ .


Improve ColumnTypeDatabaseTypeName and ColumnTypeScanType

Codecov Report
Patch coverage: 100.00% and project coverage change: +1.50 :tada:

Comparison is base (aa28174) 80.12% compared to head (952540d) 81.62%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   80.12%   81.62%   +1.50%
==========================================
  Files           8        8
  Lines        1112     1105       -7
==========================================
+ Hits          891      902      +11
+ Misses        184      166      -18
  Partials       37       37

Flag
Coverage Δ

local
81.62% <100.00%> (+1.50%)
:arrow_up:

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

Impacted Files
Coverage Δ

godynamo.go
58.66% <100.00%> (+12.51%)
:arrow_up:

stmt.go
91.90% <100.00%> (+0.07%)
:arrow_up:

stmt_index.go
79.56% <100.00%> (+0.30%)
:arrow_up:

stmt_table.go
84.38% <100.00%> (+0.15%)
:arrow_up:

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


Remove Node 8 from workflows

We have some responses/code examples that use Node v8 - that version is deprecated, we should remove it! If we still want to show multiple versions, I'd recommend 12, 14, thought 10, 12 is also okay!

Solved.  We fixed this for the Satellite workshops, we now use 10,12,14 for this course.


Add simple Ruby server example

fix apache/arrow#40479

Thanks @kou!
I tested this with all the client examples. The only problem I found was with the C GLib client:
When I run the C GLib client with this this Ruby server, the server prints this message:
::1 - - [12/Mar/2024:09:14:43 -0400] "GET / HTTP/1.1" 200 - 0.8649
[2024-03-12 09:14:44] ERROR Errno::ECONNRESET: Connection reset by peer @ io_fillbuf - fd:18
	/opt/homebrew/lib/ruby/gems/3.3.0/gems/webrick-1.8.1/lib/webrick/httpserver.rb:82:in `eof?'
	/opt/homebrew/lib/ruby/gems/3.3.0/gems/webrick-1.8.1/lib/webrick/httpserver.rb:82:in `run'
	/opt/homebrew/lib/ruby/gems/3.3.0/gems/webrick-1.8.1/lib/webrick/server.rb:310:in `block in start_thread'

Ah, it's happen because GLib client uses keep-alive and Ruby's HTTP server is sensitive for disconnection check of keep-alive connection.
I'll disable keep-alive of GLib client.

I've add missing .gitignore. Good catch!

GLib client: https://github.com/apache/arrow-experiments/pull/20


error message concerning un-updated descriptor refers to "draw" upon call to vkCmdDispatch

I call vkCmdDispatch after creating, updating, and binding some descriptor sets.  There's something wrong with the way I've configured the descriptor sets, so I should and do get an error message:
"DS 0x5b encountered the following validation error at draw time: Descriptor in binding #3 at global descriptor index 3 is being used in draw but has not been updated."
But the vkCmdDispatch call that triggers the message is not a "draw" call; it's just running some calculations in a compute shader.  I do have some drawing calls in other places in my code (and they are working fine, and producing no error messages).  Thus this error message tells me that I should look in the wrong place when trying to fix the error, wasting my time.
Maybe it should use the word "dispatch" rather than "draw".  Naming the exact function that triggered the message ("vkCmdDraw" or "vkCmdDrawIndexed" or "vkCmdDispatch" or whatever else) might be even better, unless it's infeasible or inappropriate for some reason.

Hi @mjwach
Thanks for noting this.
We've already updated this error message in the master branch and the fix will be included in the next SDK. You can see the updated code in core_validation.cpp.


Open in terminal uses root drive path instead of folder path

Explain the bug
The open in terminal option seems to open it in C: rather than the current working directory.
To reproduce

Go to any folder except the root drive (C: in my case)
Right click and hit open in terminal
See the error

Expected behavior
Write a clear description of what you expected.
It should open in the current working directory instead of the root drive location.
System Information

OS Version: Windows 10 Pro 20H2 build 19042.964
App version: <IP_ADDRESS>

Additional comment
Add any other comment about the problem here. Does this problem happen again after restarting the app?
This problem exist in all there terminal app options - CMD, PowerShell, Windows terminal

It works fine for me in the new windows terminal. You will properly have to upload your logs.

This issue appears to be resolved in main.


posting with image bug?

Failed to publish: Cannot read properties of undefined (reading 'split')
      await threadsAPI.publish({
        text: text,
        image: 'scr.png',
      });

Finding similar error on lots of functions

Similar to #111 devs are aware of the bug


Update Gradle Wrapper from 6.8.3 to 7.0

Update Gradle Wrapper from 6.8.3 to 7.0.
Read the release notes: https://docs.gradle.org/7.0/release-notes.html

The checksums of the Wrapper JAR and the distribution binary have been successfully verified.

Gradle release: 7.0
Distribution (-all) zip checksum: 81003f83b0056d20eedf48cddd4f52a9813163d4ba185bcf8abd34b8eeea4cbd
Wrapper JAR Checksum: e996d452d2645e70c01c11143ca2d3742734a28da2bf61f25c82bdc288c9e637

You can find the reference checksum values at https://gradle.org/release-checksums/

🤖 This PR has been created by the Update Gradle Wrapper action.

Need help? 🤔

If something doesn't look right with this PR please file an issue here.

Unable to set all the PR reviewers, check the following usernames are correct:

@bigmichi1

Please refer to the documentation for the reviewers and team-reviewers input parameters.

🤖 This is an automatic comment by the Update Gradle Wrapper action.


报的什么错

librealsense
2.55.1 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
RealSense error calling rs2_create_context_ex(api_version:25501, json_settings:{):
    failed to convert special folder: errno=42

Hi @admin123yan  My Intel RealSense colleagues provided advice about the errno=42 problem.
In SDK 2.55.1 there was a pipeline context crash problem due to an uninitialized json variable.  The issue was fixed on the development branch of the SDK and also included in the 2.55 release.  But sometimes 2.55 still has a problem due to mixed libraries.
Clean the folders and make sure that you have the correct version of the SDK libraries.  Builds made from the development branch didn’t have the problem after the fix.
https://github.com/IntelRealSense/librealsense/tree/development
Another possible cause of the error is that the SDK uses a json file to store settings.  Sometimes, due to an app not cleanly closing down, those json files are empty and the SDK will crash on the next time that it is launched.  The json files can be found at the following location on a Windows computer:
C:\Users\[user]\AppData\Roaming
The folder contains files like 'realsense-config.json'.  The workaround is to delete the empty json files at this location and then relaunch.

嗨@admin123yan 我的 Intel RealSense 同事提供了有关 errno=42 问题的建议。
在 SDK 2.55.1 中，由于未初始化的 json 变量，存在管道上下文崩溃问题。该问题已在 SDK 的开发分支上修复，也包含在 2.55 版本中。但有时 2.55 仍然会因为混合库而出现问题。
清理文件夹并确保您拥有正确版本的 SDK 库。修复后，从开发分支构建的版本没有出现问题。
https://github.com/IntelRealSense/librealsense/tree/development
错误的另一个可能原因是 SDK 使用 json 文件来存储设置。有时，由于应用程序未干净关闭，这些 json 文件为空，SDK 将在下次启动时崩溃。 json 文件可以在 Windows 计算机上的以下位置找到：
C:\Users\[user]\AppData\Roaming
该文件夹包含“realsense-config.json”等文件。解决方法是删除此位置的空 json 文件，然后重新启动。

Do you require further assistance with this case, please?  Thanks!
您需要进一步协助处理此案吗？谢谢！

Case closed due to no further comments received.
由于未收到进一步评论，案件结案。

My Intel RealSense colleagues have provided some further advice about the errno=42 error.
Given that the problem seems to be related to certain charcters being used in a Windows user account name, I asked whether creating a secondary Windows user account with a name that has no special characters might help.  This was their response.
You can try to run it as Administrator instead of switching the account.  On Windows 11, it also has a feature to run the app as a different user.  That option can be enabled with the instructions at the link below:
https://winaero.com/how-to-run-apps-as-different-user-in-windows-11/

我的英特尔 RealSense 同事提供了一些有关 errno=42 错误的进一步建议。
鉴于该问题似乎与 Windows 用户帐户名称中使用的某些字符有关，我询问创建一个名称中没有特殊字符的辅助 Windows 用户帐户是否有帮助。这是他们的回复。
您可以尝试以管理员身份运行它，而不是切换帐户。在 Windows 11 上，它还具有以其他用户身份运行应用程序的功能。可以按照以下链接中的说明启用该选项：
https://winaero.com/how-to-run-apps-as-different-user-in-windows-11/


Co op to windows dedicated conversion successfull for the host, but regular joining player has to create a new character

Hi, i got the save game from my friend and followed the steps. Before the conversion (Step 4), i can join the Server and everything is fine for me. The conversion seems successful (no Errors and i get the have fun message, also my friend can join the new  dedicated server with his old co op host charcter). But now i have to create a new character.
I can see how the game overrides my old guid.sav once i create a new character (the file gets smaller to 2 KB). ( my guid.sav was also in the tempoary "players" folder if thats important).
I can override my character with an Backup, but if i do so, im no longer in out the Guild (i was the leader before the conversiona nd my co op host fried left the guild) and we dont have access to our base because we are not members of the guild any more.

I transferred the whole save folder as well and had the same problem.
I told my friend to create his new character and the new GUID was different.
I'm going to try to repeat the process for my friend's save file now.

Ok so I just repeated the process only with my friends old and new GUID, transferred the save and it works.
We lost all Pals that were working at the base at the moment though so maybe recall them and put them in the Palbox first?
Hopefully it works for you as well, good luck.

Ok so I just repeated the process only with my friends old and new GUID, transferred the save and it works. We lost all Pals that were working at the base at the moment though so maybe recall them and put them in the Palbox first? Hopefully it works for you as well, good luck.

So after using the python script on your old GUID and new GUID, you then repeated the process with the rest of the players in your co-op world, correct? Did you have to rename the SAV files of their old GUIDs to<PHONE_NUMBER>0000000000000000000001 when using the script again for it to work? And one more question, in your dedicated server's Players folders, do you have the SAV files of your friends from your co-op world in there as well or did you replace them with the new GUID SAV files?

No renaming of files, I wouldn't try that altogether.
I used the script to fix the host's save file according to the instructions. The host had his old character now but the coop player had to create a new character which created a different, new .sav file on the server.
I then copied the whole save file from the server again and used the script with the old and the new GUID of the coop player's character. Neither GUID was<PHONE_NUMBER>0000000000000000000001.
We are playing in a duo so I had no possibility to test it with multiple coop players.
I hope this is more understandable now.

It looks like this is a duplicate of https://github.com/xNul/palworld-host-save-fix/issues/26 so I'll close this issue. Please pay attention to that issue for any updates about the bug.


Tabnine switching between Pro and Starter edition

please complete the following information:

OS version: [Win 11]
Editor: [vscode]
Editor version:1.79.2
Programming language: [ts]
TabNine extension version: 3.6.60

Issue Details:
I am also experiencing the same problem that someone else logged. My Tabnine starts in Pro but if I alt-tab or just leave the editor for a minute or 2 the it switches to Starter and then I have to restart the editor to get it back to Pro. This is very counterproductive.
gz#24468(related to Zendesk ticket #24468)

@NumicDev
Are you behind Proxy/VPN/Firewall/Antivirus? Please follow the steps here.
Also, please send us the log files. See here how to do that.

@TzufTabnine
No I am not. I already sent the Log files to TabNine


[FEA] Add Spark 3.5.2 snapshot support

Is your feature request related to a problem? Please describe.
Once Spark 3.5.1 is released add the Spark 3.5.2 snapshot build and shim.
Note: Need to enable both Scala 2.12 and 2.13

Note 3.5.2 is up for vote in Apache

fyi - vote passed in Apache Spark community so should be available soon if not already

I will setup nightly tests after https://github.com/NVIDIA/spark-rapids/pull/11334

@pxLi can you confirm if the nightly has been enabled for 3.5.2? #11334 has been merged.

@pxLi can you confirm if the nightly has been enabled for 3.5.2? #11334 has been merged.

yes, it is. We have the nightly CI running for days


Remove VMware Workstation Player and Workstation Pro from winget repository

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?

Note: <path> is the name of the directory containing the manifest you're submitting.

Unfortunately Broadcom has ceased VMware Download servers.
All downloads go trough Broadcom's servers.
Workstation Pro is now free for personal use, but the dowload is locked behind a Broadcom account and extensive data sharing.
The download link is https://downloads2.broadcom.com/?file=VMware-workstation-full-17.5.2-23775571.exe but it doesn't work without your personal authentication headers.
Workstation Player has been discontinued after Workstation Pro has been made free for personal use. Still, VMware download server has been discontinued.
These packages are in a broken state right now.
There's no solution to this sad change, hence removing these packages.
Source: I am a Broadcom (ex VMware) employee.

There's no validation to be done.

URL: https://download3.vmware.com/software/WKST-PLAYER-1750/VMware-player-full-17.5.0-22583795.exe
Response status code does not indicate success: 522 ().
(Automated message - build 881)

The pull request contains more than one manifest. Either singleton or multi file manifest under one version directory.

We have a limit of 1 manifest change (add/modify/remove) per PR. Can one of these 2 manifest removals be moved to another PR? Then both should pass this check.

I don't know how to move the manifest removal, hence I am closing this PR and I will create two new PRs.


nixos/kanata: fix enabling kanata using nixos-rebuild switch

Description of changes
When enabling services.kanata on a running system using nixos-rebuild switch (or nixos-rebuild test), the kanata service may fail to start if hardware.uinput is not enabled before.  Note that this issue never happens if you use nixos-rebuild boot to enable service.kanata.
kanata needs to use uinput, so the services.kanata module sets hardware.uinput.enable = true.  The config of hardware.uinput is set by systemd-modules-load.service and systemd-udevd.service.  These two systemd services are dependencies of sysinit.target.  According to man systemd.special, systemd automatically adds dependencies of the types Requires= and After= for this target unit to all services.  So the kanata service will run after these two services during boot.  However, nixos-rebuild switch do not know the dependency and order between the kanata service and these two services, so the kanata service may fail to start.
This patch makes sysinit-reactivation.target depend on these two services so that the dependency and order of the kanata service and these two services are preserved by nixos-rebuild switch.
Ideally, sysinit-reactivation.target should depend on all "sysinit units".  This is tracked in https://github.com/NixOS/nixpkgs/issues/334017.  But that is beyond the scope of this PR.  This PR can be seen as a partial fix for https://github.com/NixOS/nixpkgs/issues/334017.
Closes https://github.com/NixOS/nixpkgs/issues/317282

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

The issue this PR fixes may look like a specific one of services.kanata, but it is not.
I would say the real issue is that currently sysinit-reactivation.target ignores almost all order dependency of sysinit.target except systemd-tmpfiles-setup.service (or systemd-tmpfiles-resetup.service).
I would like to hear ideas of a more general fix instead of doing it case by case like this PR.  That being said, don't let perfect be the enemy of good.  I am going to merge this PR as is in 2 weeks if no reviews are made. If you want to review but do not have time recently, please let me know and I will wait for you.

I'd prefer this to be fixed in a more general case. It'll be very hard to find out where fixes like these are made all across nixpkgs, and it is arguably something that should work out of the box.
However, if it can't be fixed in a reasonable time, specific workarounds are fine IMHO, as long as they're clearly marked as such - which currently is not the case.
Can you start by opening an issue in nixpkgs, describing the general issue, linking to here (and maybe other places) as examples, and make sure we link to this issue in a comment close to where the workaround is added?

The general issue is that we cannot just restart sysinit.target because all other services have an automatic dependency on sysinit and thus everything will restart and the whole world explodes. That's why we need this synthetic restarting target.
The only general solution to this problem is probably systemd's soft reboot.

Can you start by opening an issue in nixpkgs, describing the general issue, linking to here (and maybe other places) as examples, and make sure we link to this issue in a comment close to where the workaround is added?

Done.  https://github.com/NixOS/nixpkgs/issues/334017


Type variable warning on static generic method

The following code produces a bogus warning that type parameters are not allowed on static members:
class C {
  static /*=T*/ f/*<T>*/() => null;
}

[warning] Static members cannot reference type parameters (/Users/leafp/tmp/scratch.dart, line 2, col 13)
dartanalyzer version 1.14.0-edge.daf5f6d258a99428cc4dc9ff17645a33e4e4bebd

ah, yup. That is likely happening from error verifier. We'll need to refine the check:
class C<E> {
  static /*=T*/ f/*<T>*/(E e) => null;
}

usage of "E" should be an error but not "T"

https://codereview.chromium.org/1508183002/


Raspberry Pi in read-only mode

Given that the average user will install Gymnasticon on a Raspberry Pi once, configure it, power it up and then leave it running, I'm wondering if it makes sense to place the SD-card into read-only mode.
Benefits would be:

The SD-card isn't worn out by writes
Reduce the risk of corrupted file systems during sudden power disconnects

Besides log files there isn't really anything written to the device that is of interest and even log files are only useful for troubleshooting, mostly during development.
Which mode would make the most sense?

Raspberry Pi image is by default enabled for read-only. Mode can be disabled by placing "no-readonly" file into boot partition of SD-card.
Raspberry Pi image is by default in read-write mode (like today). Read-only mode can be enabled by placing "readonly" file into boot partition of SD-card.
Raspberry Pi image is by default enabled for read-only. Mode cannot be disabled, by command-line shortcuts to remount filesystem into read-write exist.

I'm in favor of read-only and making the standard SD install method an "appliance" for durability reasons.

makes sense. Now that the Keiser bike is gonna be supported shortly, I expect a large# of people getting interested as the alternative is a bluetooth connector from Keiser which they sell for 130 and is notoriously unstable. Likely, a lot of them won't know much about raspberry pi's ;-)

In this case, auto-detection of the bike may be required though rather than using a config file?

I agree this would be a nice feature!
There is already an option in raspi-config to use OverlayFS. Turning it on/off requires a reboot but the average user hopefully would not have a need to do that.
@tompijls Good point about the config files. Maybe all we need to do is modify the startup files to cp the config files from /boot instead of mv.
We already don't have swap enabled so that's taken care of.
Anything else to consider?

@ptx2 Yes, using cp instead of move on the config file would be a good start. I'll also open another issue for extending the auto bike mode to the two new bike types. That should reduce the need for a config file quite a bit.
We might also want to disable the syslog service ( sudo systemctl disable syslog.service) as it could otherwise eat up all the RAM via the overlayfs. You can still use journalctl to look at logs during runtime.
Also with the overlayfs support built into raspi-config, you can't activate/deactivate it via CLI, which stands in the way of making this user friendly.
But the script that raspi-config uses under the hood can also be found here. That script (or an adaptation thereof) could be used here for gymnasticon. The only element that I see in the script which requires Internet access is the installation of initramfs-tools, which can be taken care of via pi-gen.
Depending of whether we want to make the use of overlayfs the default or not, this could either become part of the pi-gen process or be invoked as a first-boot service after checking for the readonly / no-readonly file in /boot.
Also this approach comes with the script overctl that can be used to enable / disable overlayfs afterwards.
Imho, we should make the read-only mode the default mode. That would mean incorporating overlayfs.sh into the pi-gen process.

Turns out that you can actually can call raspi-config functions via CLI. See here.
Looks like all that's needed is to save that bash script in /usr/local/sbin and then call it via overctl -r. It should be possible to do all that from the current pi-gen process.

Create pull request #58 for this.


Add unit testing workflow

This PR adds an automated unit testing workflow. This partially fulfills the requirements for #61.

r2g


first stab at adding functions to download the history

WIP
using https://github.com/OpenSourceFieldlinguistics/FieldDB/issues/1946

I messed up merging this pull requests so I just copy-pasted your code into d54e147.

@jrwdunham  lets handle this one in the scrum meeting. it might be a long meeting... :(


issue-381 : Remove additional setup script of kafka, zookeeper

Additional setup script used in kafka and zookeeper dockerfiles is removed and setup file contents are now included in start scripts.

Closing this PR and creating against issue-381 branch instead of development branch


Update ignore files.

Changes

Update ignore files to ensure build/ (static pattern library for Github pages) and npmauth (NPM credentials, yikes!) don't get pushed to NPM or Github.

For review: @DoSomething/front-end

good to me :+1:


pull policy is always but image has been referred to by ID

Hi Team,
I am facing below issue while running operator-test-playbook using below script.
https://raw.githubusercontent.com/operator-framework/operator-test-playbooks/master/upstream/test/test.sh
previously it was working fine.
TASK [validate_operator_bundle : Pull index image 'kind-registry:5000/test-operator/kogito-operator:v2.0.0-snapshot' (operator_sdk_none fix)] ******************************************************************************************
fatal: [localhost]: FAILED! => changed=true
  cmd: podman pull kind-registry:5000/test-operator/kogito-operator:v2.0.0-snapshot
  delta: '0:00:00.061042'
  end: '2021-05-11 12:24:14.337491'
  msg: non-zero return code
  rc: 125
  start: '2021-05-11 12:24:14.276449'
  stderr: 'Error: pull policy is always but image has been referred to by ID (kind-registry:5000/test-operator/kogito-operator:v2.0.0-snapshot)'
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>

My bad, issue occurs because of wrong parameter. Thanks for your promt response.


Add QEMU CI runner for the s390x architecture

A next step might be to include a builder to build binary packages as well (which would provide tarballs for download on releases, etc.); if you think that's worthwhile, I'm happy to review a followup PR.

Sure, that sounds good.  Is this just about adding to the matrix in the "build" workflow target, or are there other changes needed?   Not sure if I can actually test this ahead of time ...

I think, so, yeah, basically we just want to copy what's already there for aarch64.


Migrate Minicluster module to JUnit5

Part of issue #2441 This PR contains changes for the conversion from JUnit4 to JUnit5 for the accumulo-minicluster module.

LGTM. I have verified all tests pass.


W requests

@trp07
Here it is.

See except block. No longer raising MessageSendError exception because any 'raise' there interfered with API test run, wasn't able to do any assertions on raised errors.
So I print them and return exception that is passed to the tests where I can assert.
Added benefit of the twp print statements is that now all errors in the console and repl now also show descriptive twilio error response, not just "bad request".
The atch is that there needs to be a return statement for response and exceptions to be asserted in API tests. Tests look for the response and act on it.

Downside of the above is that now pytest run outputs the two printed error messages for each test.
For example when I run the third test
pytest -s tests/integration_tests/test_text_api.py -k 'test_03' --verbose
I get output
tests/integration_tests/test_text_api.py::test_03_send_from_unavailable_number 400 Client Error: BAD REQUEST for url: https://api.twilio.com/2010-04-01/Accounts/AC860b9f5f7da8af65e7c49295595b1006/Messages.json The From phone number +15005550000 is not a valid, SMS-capable inbound phone number or short code for your account. PASSED
And so for each test.

Place where users can pass their twilio profile for API tests in as parameter in pytest fixture (test_text_api.py file). We can talk about what is best place for this.

Tests still to add. And to figure out any other relevant tests:

Your account doesn't have the international permissions necessary to SMS this number +15005550003
This number is blacklisted for your account +15005550004
This number is incapable of receiving SMS messages +15005550009
This area code doesn't have any available phone numbers.
This area code has an available number. (no error)

Travis fails because we don't supply a valid twilio profile in API tests (currently we use mtwilio_profile which a test profile).
In test_api_tests.py:
@pytest.fixture()
def get_twilio(profile='mtwilio_test'):


Tests and cleanup for Pipeline

Use numpydoc standard for documentation
Add tests
Remove param dependency
Update notebooks for slight change in call signature

Ping @jbednar for review. Only spelling change here is df and glyph aren't keyword arguments. This is because they're required parameters, and so no default makes sense.

Got the go ahead to merge externally in conversation. Merging.


Cannot find module 'eslint-config-defaults/configurations/eslint'

Hi, Im trying to install ESLint for my React Native app by the tutorial here:
My packages.json:
{
	"name": "TodosList",
	"version": "0.0.1",
	"private": true,
	"scripts": {
		"start": "node node_modules/react-native/local-cli/cli.js start",
		"test": "jest"
	},
	"dependencies": {
		"react": "~15.4.0-rc.4",
		"react-native": "0.38.0"
	},
	"devDependencies": {
		"babel-jest": "17.0.2",
		"babel-preset-react-native": "1.9.0",
		"jest": "17.0.3",
		"jest-react-native": "17.1.0",
		"react-test-renderer": "~15.4.0-rc.4",
		"eslint": "^3.11.1",
		"eslint-plugin-react": "^6.7.1",
		"eslint-plugin-react-native": "^2.2.0"
	},
	"jest": {
		"preset": "react-native"
	}
}

and I keep getting this error:
Cannot find module 'eslint-config-defaults/configurations/eslint' Referenced from: c:\Users\lednh\.eslintrc
and I don't have eslint installed globally.

@lednhatkhanh: can you try running ESLint from a terminal opened on the VS Code workspace that shows the problem. Will that work ?

Sorry for late respond! I realized that there's a global eslintrc left and vscode tried to find the configs in that file, not in my local file. So I deleted that file and everything started working fine.

@lednhatkhanh good to hear that it is working now. Will close the issue.

The global .eslintrc file was hiding in C:\Users\User for me. Deleting that solved this issue.


Is there a way to update options dynamically?

I want to update the options of sortable element dynamically. I just don't understand how can I access the existing instance of sortable element. Let's say I created this new sortable instance. Then how can I access it and pass new options for "group > put"?
Sortable.create(requests, {       group: {         name: 'requests',         put: (['standBy', 'rooms']).concat(roomCards)       },       animation: 250     });

As an option try to use Object.assing in order to update object of options. It is working for me

Sortable options can be changed by calling the option method :
this.sortable.option('disabled', false)

Any solution for that please?

@asmaLouahdi The option method

Unfortunately .option('disabled', true) won't work dynamically

Can this code code work for you?
const dynamicOptions  = (disabled) => {
 return {
   group: 'group',
   disabled
  //...rest options
 }
}
Sortable.create(requests, dynamicOptions(true))
Sortable.create(requests2, dynamicOptions(false))

I had the same problem with an dynamic list.
Solved adding "filter: '.disabled'," to default options  and toggling the class at the handle.


When forwarding request "Host" header is not being propagated

I found this issue in latest docker image (3.10.8). "Host" header is not being propagated. It works correctly in previous docker image (3.10.7).

I can't re-produce this issue.  I am going to close this issue, however, if you can provide some more information (such as log files, code, etc) then I'm happy to re-open this issue.


Reproduction with node-postgres of "Cannot import named export from a typescript file in a workspace package if entry is using es modules and other package is not."

Acknowledgements

[X] I read the documentation and searched existing issues to avoid duplicates
[X] I am reporting a confirmed bug (not seek debugging help)
[X] I understand this is a collaborative project and relies on my contributions
[X] I read and understood the Contribution guide

Minimal reproduction URL
https://stackblitz.com/edit/stackblitz-starters-g6pt9g?file=index.mts
Problem & expected behavior (under 200 words)
Reproduced https://github.com/privatenumber/tsx/issues/38 with node-postgres
Bugs are expected to be fixed by those affected by it

[ ] I'm interested in working on this issue

Compensating engineering work will speed up resolution and support the project

[ ] I'm willing to offer $10 for financial support

Thanks for providing the repro—it helps a lot with understanding what you did.
I've never used node-postgres before so I assumed the package was node-postgres.
The pg package doesn't do named exports in a statically analyzable way, so it's hitting the same limitation in Node as with lodash (this isn't a tsx issue or limitation).
You can either file an issue with pg to make imports statically analyzable or with Node to improve their static analysis.


Update ryuk version to 0.7.0

Hi, have you seen this one? https://github.com/testcontainers/testcontainers-java/pull/8436
The pull request would enable Windows containers using the enhanced multi-arch image of ryuk.


[Bug] False throw of valid-title

Let's have following test file
import { TestDetails, test } from '@playwright/test'

test.describe('Suite', () => {
  async function makeTest (testTitle: string, testOptions: TestDetails, urlPath: string) {
    test(testTitle, testOptions, async ({ page }) => {
      await page.goto('www.mywebsite.com/' + urlPath, { waitUntil: 'load' })
    })
  }

  makeTest('Test1', { tag: ['@smoke'] }, 'url_path_1')
  makeTest('Test2', { tag: ['@smoke'] }, 'url_path_2')
  makeTest('Test3', { tag: ['@smoke'] }, 'url_path_3')
})

I get this error

which is wrong regarding docs https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/valid-title.md

package
version

eslint-plugin-playwright
1.6.2

@playwright/test
1.44.0

@MartinPilny This is not a bug. The Playwright ESLint plugin does not have type information from TypeScript, so it must rely solely on static analysis. Your code is a correct error according to the plugin and should be explicitly ignored with an ESLint disable comment.


Fixed newRecycler func for HostPath & NFS

@thockin only PV can do recycling, not regular volumes.  Both conditional statements will never be true for VolumeSource.   The NFS one, in particular, was a poor cut/paste oversight.

LGTM


fix fuzz target

I was trying to build the fuzz target of promql and noticed it doesn't compile anymore.
This change fixes it.
I looked briefly but couldn't figure out if ParseStmts is gone or is available under a different name, so for now I've deleted it.

@fabxc @msiebuhr @juliusv

Related to this, I recently ran into https://fuzzbuzz.io/ that will fuzz (some) OSS projects continuously.
Perhaps this is something Prometheus should sign up for?

 We fuzz OSS projects for free and we help
contribute the PR. Is there any reason you do not want to use us?
You can see all the OSS projects we fuzz on our site.
On Thu, Aug 8, 2019, 3:46 PM Morten Siebuhr<EMAIL_ADDRESS>wrote:

Related to this, I recently ran into https://fuzzbuzz.io/ that will fuzz
(some) OSS projects continuously.
Perhaps this is something Prometheus should sign up for?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/5835?email_source=notifications&email_token=AD52CDX6XW4SGPLVWADTB6DQDQIRVA5CNFSM4IJN6MPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD33PWGA#issuecomment-519502616,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD52CDSKALTXDVWYYRW773TQDQIRVANCNFSM4IJN6MPA
.

@msiebuhr
some differences between fuzzbuzz and fuzzit.dev

Our cli is open-source https://github.com/fuzzitdev/fuzzit which is think is a best practice
we provide regression fuzz testing during PRs that will help catch bugs early on in a PR
IMHO we have currently a bit of wider adoption is OSS porjects - you can see on our size.
In the end I guess you should try both and decide what best for the project.

Please let me know and I'll be happy to contribute a PR once the fuzz targets works.
Cheers,
Yevgeny (Founder @fuzzitdev)

Hi both, I haven't heard of either of those services, but fuzzbuzz looks interesting. @yevgenypats would the main benefit of putting this into a separate service rather than our normal CI with go-fuzz be that fuzzing may take some time, so it's generally good to separate it out? Plus, then you have a specialized UI around fuzz test results. I know little about fuzzing technology details, but I would be interested to see what an integration for Prometheus would look like.

@yevgenypats Oh sorry, misunderstood you and thought you were part of fuzzbuzz, not fuzzit, so just take my same comment for fuzzit :)

@juliusv lol the names are indeed confusing:) So first if you won't to deep dive a bit on how continuous fuzzing workflow looks like you can take a look at our blog https://fuzzit.dev/blog. but TLDR:

The fuzzing is indeed infinite process and this is not blokcing the CI. every push to master new fuzzers are built and updated to fuzzit so they can continue running and will alert via email/slack/other channel if new crashes are found.
Fuzzing generates thousands of useful test-cases as well as crashes. We make use of those test-cases and run them through the fuzzers on every PR. This usually takes seconds (minutes) and prevent from old crashes propagating again or new bugs being introduced. This saved our users literally hundreds of bugs from being introduced.

I hope it was a good TLDR but more info on our blog and feel free to ask me more questions. also you can join our slack:) https://slack.fuzzit.dev

@kjk Looks great besides the nit, thanks!
CI is still failing due to missing / wrong gofmt:
>> checking code style
gofmt checking failed!
diff -u ./promql/fuzz.go.orig ./promql/fuzz.go
--- ./promql/fuzz.go.orig	2019-08-09 17:24:58.072665453 +0000
+++ ./promql/fuzz.go	2019-08-09 17:24:58.072665453 +0000
@@ -38,7 +38,6 @@
 //
 // Repeat for FuzzParseOpenMetric, FuzzParseMetricSelector and FuzzParseExpr.

-
 // Tuning which value is returned from Fuzz*-functions has a strong influence
 // on how quick the fuzzer converges on "interesting" cases. At least try
 // switching between fuzzMeh (= included in corpus, but not a priority) and

Please ensure you are using go version go1.12.7 linux/amd64 for formatting code.
make: *** [Makefile.common:117: common-style] Error 1
Exited with code 2

Also, could you add a Signed-off-by line to make the DCO check pass (see https://probot.github.io/apps/dco/)?

@yevgenypats I don't know what the rest of the Prometheus team thinks, but that sounds good - would like to take you up on your offer to send a PR about that :)

@juliusv sounds good! once we get this passing the CI and merged we will create a PR for Fuzzit integration:)

@juliusv  I'll open another PR. I followed DCO instructions but git push --force-with-lease origin fix-fuzz was apparently not the right thing to do.
The concurrent changes on master made this a mess of rebasing.

Oh hum weird, yeah, something's off here now :)

Replaced by https://github.com/prometheus/prometheus/pull/5851


hubble: avoid rounding up power of 2 specified flows buffer capacity

The ring buffer has an actual capacity of 2^n-1. Thus, to avoid power of 2 specified capacity to be rounded up (eg 1024 -> 2047), subtract one to the provided capacity when the provided value is a power of 2 as this probably best reflects the actual user's intent.
Fixes: #11650

test-me-please

GKE flakes are unrelated to this change.

I discussed this PR offline with @gandro and as this change has a potential user impact, especially considering that Hubble is now enabled by default in 1.9, we decided to backport this fix to v1.9. I pushed an extra commit to the PR to document the behavior change and the potential impact in the upgrade doc.

Note: no need to re-run the CI, I just pushed a doc change.

I discussed this offline further with @gandro. There are implications regarding changing the default value for hubble.flowBufferSize to 4096, notably because the current Cap() implementation of the ring buffer is actually lying (it reports 4096 as the actual capacity when, in fact, only 4095 flows can be read from the ring buffer as 1 slot is reserved for writing and never readable). We thus decided NOT to backport this PR and try to find an actual solution. Marking the PR as draft for the time being.

Closing this PR in favor of #13894.


[No QA]Update readme to include link for updating encrypted secrets

@AndrewGable will you please review this? Lmk if there's any other info we want to add here
Details
This PR updates the workflow.md to include the help link for updating encrypted secrets used in the repo.
Tests
See the changes in https://github.com/Expensify/Expensify.cash/blob/joe-encrypted-secrets-readme/.github/workflows/README.md#secrets

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


[Tracker] Improvements and fixes to improve the demo

Higher priority:

[x] Make the character throw grenades in the character's look direction when not in aim mode
[ ] #14
[ ] Add a more fitting audio soundtrack
[x] Limit the rate of shooting (both grenade and rifle)
[x] Make shooting with the rifle continuous when holding the left mouse button down
[x] Bug: The jump pad does not always work

Lower priority:

[ ] The punch attack would benefit from having two animations
[ ] Add an aiming animation and make the character aim their arm where they're looking
[ ] Add a reticle when aiming with the gun
[ ] The coins should have an easing when moving to the character

Known Bugs
Beetle: Add a NavigationAgent3d Node to Beetle,
change Beetle code line 37 to set_target_position and line 39 to get_next_path_position.
DemoPage: tween.tween_property(demo_page_root, "modulate", Color.WHITE, 0.3) change white to BLACK or you can't see controls (or just change "blur" in tree color to 000000a9 or something closer to black).
Player: have to # out all "punch" code references because they don't work. # _character_skin.punch()
PlayerSkin: #func punch(): #animation_tree["parameters/PunchOneShot/active"] = true Shaders Error: even after reimport/tool: E 0:00:00:0527 SCREEN_TEXTURE has been removed in favor of using hint_screen_texture with a uniform. To continue with minimal code changes add 'uniform sampler2D SCREEN_TEXTURE : hint_screen_texture, filter_linear_mipmap;' near the top of your shader. <C++ Source> :7 ^has to do with the voronoi at top of shader on smoke_ball shader. It still works even with error though. These lines: uniform sampler2D voronoi_sampler; varying float voronoi; replace as error says, then varying float error occurs.

Thanks for taking the time to list these! In general, I think the project may need to be ported to Godot 4.2 or 4.3. Nothing should be commented out, just properly ported if anything stopped working with the latest Godot release.
We won't have time to do this ourselves soon, but contributions are welcome if anyone wants to tackle this.


Instrument Extension Mission as Array?

Another planetary question for everyone.
We have some mosaics of Europa (I can hopefully share a link sometime soon). The mosaics are made from data collected by Voyager I, Voyager II, and Galileo.
The instrument spec defines the mission as a string. Right now, I have a STAC file that defines the missions as: "Voyager I, Voyager II, Galileo". The instrument extension as an array of values. Does it make sense to also allow the mission to be an array of strings?
This is an issue for us planetary folks only when we are building mosaics (that we would like to share with STAC metadata) that use multiple instruments that flew on different missions. One other example would be HiRISE / MOLA DEMs.

For mosaics, you'd just not specify the missions as the data are basically newly computed values from different sources IMHO. I think the instruments extension is meant to list the metadata for the source. In STAC, you'd link to the source data, which then lists the missions for each of the sources. If you'd want to list all things that went into a mosaic, I guess you'd best define a mosaic extension, which defines those semantics (i.e. "the missions the data is composed from"). Mosaics came up quite often in the past, but no one took the effort yet to write an extension for it.

You could also use the themes extension.


Wrong cropper dimensions when cropAspect == sourceAspect [BUG]

When cropAspect == sourceAspect the cropper dimensions are smaller than they should be.
Plunker: https://embed.plnkr.co/GdlRu0q3EEWEnvnwRFom/
The following code is wrong:
(Extracted from imageCropper, lines 667 to 688)
if (cropAspect > sourceAspect) {
   var imageH = Math.min(w * sourceAspect, h);
    var cropW = imageH / cropAspect;
    tlPos = PointPool.instance.borrow(cX - cropW / 2, cY + imageH / 2);
    trPos = PointPool.instance.borrow(cX + cropW / 2, cY + imageH / 2);
    blPos = PointPool.instance.borrow(cX - cropW / 2, cY - imageH / 2);
    brPos = PointPool.instance.borrow(cX + cropW / 2, cY - imageH / 2);
} else if (cropAspect < sourceAspect) {
    var imageW = Math.min(h / sourceAspect, w);
    var cropH = imageW * cropAspect;
    tlPos = PointPool.instance.borrow(cX - imageW / 2, cY + cropH / 2);
    trPos = PointPool.instance.borrow(cX + imageW / 2, cY + cropH / 2);
    blPos = PointPool.instance.borrow(cX - imageW / 2, cY - cropH / 2);
    brPos = PointPool.instance.borrow(cX + imageW / 2, cY - cropH / 2);
} else {
    var imageW = Math.min(h, w);
    var cropH = imageW * cropAspect;
    tlPos = PointPool.instance.borrow(cX - imageW / 2, cY + cropH / 2);
    trPos = PointPool.instance.borrow(cX + imageW / 2, cY + cropH / 2);
    blPos = PointPool.instance.borrow(cX - imageW / 2, cY - cropH / 2);
    brPos = PointPool.instance.borrow(cX + imageW / 2, cY - cropH / 2);
}

I believe deleting the 'else' block and changing the 'else if' to just 'else' solves the problem. I'm going to submit a PR in a few hours.

Are you talking about the fact that the resulting output is not filling the cropped canvas?

Also, is this affecting your production code? Do you need it released on NPM asap?

Don't worry, it just affects the dev code.
I'm talking about the fact that the cropper selector (I don't know if that's the proper way to call it, I'm referring to the white rectangle) it's not the size of the picture. The picture size is 200x150, and I setted the corpper selector size to 200x150, so it should cover the whole picture.
If I didn't make myself clear, you should try changing (in the plunker) the cropper selector size to 201x150 and see the difference it makes.

Thanks @gfranco93


fix: fix mypy errors for publish

The publish process is held up by errors in the mypy step so I've fixed the errors to the best of my ability.

@relekang this should fix the issues with the publish process :D

Thanks 🙌


Add vector database initialization for both attacks and benign inputs

Just commenting here to confirm that this is on near-term roadmap. Its a required part of #10


Update contributing.md

Pull Request Requirements

[x] I have followed the directions and format under contributing.md and readme.md
[x] Arrange your username in alphabetical order based on the first letter, then the second letter, and so on. (For example, if your username is "JohnDoe", it should         be grouped under the "J" category and placed in alphabetical order based on the next letter "o" and so on.
[x] I added my github link
[x] I added a personal goal for this year

Congrats and Thank you!

hi! make sure to do it on both files, " I have followed the directions and format under contributing.md and readme.md"

Hi,
I’ve made the changes and updated the README as well. Please review and let me know if anything else is needed.
Thank you!

Hi,
I’ve made the changes and updated the README as well. Please review and let me know if anything else is needed.
Thank you!


Body tracking converted to screen coordinates

I'm sorry to bother you.
I use C# for program development.
I want to convert body tracking to screen coordinates.
How to draw a point on the screen after getting the position coordinates of the Head joint.
Could you please tell me, thank you very much.

Look at the body tracking viewer sample code here.

Could you please tell me, thank you very much.
I use C# for program development.
Are there any C# examples?

Just look into the API Reference:
TransformTo2D

Just look into the API Reference:
[TransformTo2D]

Thank you for your guidance.
I tested TransformTo2D to convert the screen's x and y coordinates, but the coordinate error is large.
Is there a C# program example for TransformTo2D?
Could you please tell me, thank you very much.


var.additional_security_group_ids not in use by the module

Just a heads up that var.additional_security_group_ids is not being used by the aws_network_interface resource.
A visible fix is to point security_groups   = [aws_security_group.main.id] to security_groups   = local.security_groups
Great work with the module!

A quick PR: https://github.com/RaJiska/terraform-aws-fck-nat/pull/28

Hey @danielmapar, I thought the same thing too and also raised a PR for this, but this is the intended behaviour - see the following comment https://github.com/RaJiska https://github.com/RaJiska/terraform-aws-fck-nat/pull/9#issuecomment-1962929091


Improve the maintainability of upgrading the distros/versions used in various CI/testing legs

This is not easy to do depending on which distro is being updated.  The worst is updating the distro leg used in PR validation (currently the lowest centos version).  This leg/artifacts are referenced across several places in the code base.   At a minimum using a shared variables template would help solve the majority of references. There are some outliers, for example the prep script references the default rid to retrieve the artifacts for.
Example illustrating the places that need updating - https://github.com/dotnet/installer/pull/19119

Part of this work should include updating the SDK and artifacts tarballs that are stored in blob storage so that they are version-less with respect to the distro. Today they have the full RID:

dotnet-sdk-9.0.100-preview.4.24222.1-centos.9-x64.tar.gz
dotnet-sdk-9.0.100-preview.4.24223.1-alpine.3.19-x64.tar.gz

Ideally we could even get rid of the distro:

dotnet-sdk-9.0.100-preview.4.24222.1-linux-x64.tar.gz
dotnet-sdk-9.0.100-preview.4.24222.1-linux-musl-x64.tar.gz

Or at the very least discard the distro version:

dotnet-sdk-9.0.100-preview.4.24222.1-centos-x64.tar.gz
dotnet-sdk-9.0.100-preview.4.24223.1-alpine-x64.tar.gz


Add a DataDog Connector for sending metrics via Rest API

Describe the problem you are trying to solve
Support for writing custom metrics to DataDog using REST API.
Describe the solution you'd like
Either a separate offramp or a batch operator to send data in batches using the REST offramp.

Important info here:

All requests to Datadog’s API must be authenticated. Requests that write data require reporting access and require an API key. Requests that read data require full access and also require an application key.

https://docs.datadoghq.com/api/latest/authentication/
A datadog connector needs to have an api key and an application key configured. As well as a datadog_host config.
We could pull this off by adding an auth config on the rest offramp/connector specialized for datadog.
The required format for metrics is:
{
  "series": [
    {
      "host": "test.example.com",
      "interval": 20,
      "metric": "system.load.1",
      "points": [
        [
<PHONE_NUMBER>,
          0.5
        ]
      ],
      "tags": [
        "environment:test"
      ],
      "type": "rate"
    }
  ]
}

So each event needs to be wrapped in a json object, which we could either support via the batch operator or a group by select statement.
But i think it is more convenient for users to have a ready-made datadog-metrics connector to just send stuff to.

we can avoid a datadoc conector by setting this in a script

i know, this would solve the immediate pain of not being able to send stuff to datadog. So, in this sense, it is already possible right now. But having users go through all those hoops to send stuff to datadog is really inconvenient.
For the sake of usability, although it would duplicate code we have around here and there, i would suggest to create a datadog connector, instead of providing it the ikea way (bits and pieces to put together on your own).

I would disagree, I think packaging this up with modular queries instead of extra connectors is in the long term a better usability. having hundreds of different connectors, is not as useful as having a few well polished  ones and pluggable pieces in between


Add Nong Song:

Song Name
Speed of Paul
Artist Name
KingSammelot
Source
Direct file download
Youtube Link
No response
Direct File Link
https://drive.google.com/uc?export=download&id=1U5li6uw4CvZFd_aymqDcPy_kZ8NdhZEc
Song ID
467339
Start Offset [ms]
No response
Extra
No response

reject
non-YouTube sources are not supported