Load some winapi functions at runtime

Certain functions in winapi are only supported on newer versions of windows. If we want to run on older versions, we have to load these functions at runtime. If loading fails, that means that the function is not available. In that case, we have to fall back to doing something else at runtime.
I am running windows 8.1, so I am unable to run the program without these changes. This also means that somebody who runs windows 10 should to test this, to confirm that it still works as expected.
The interesting part of this PR is in src/util.rs. Here fn load_optional_functions tries to load functions using kernel32::LoadLibraryA and kernel32::GetProcAddress. The functions are stored in the OptionalFunctions struct, which is passed to wherever these functions are needed. If it is desirable, we might be able to store these functions statically somewhere, so we don't have to pass the struct around (I think the gl crate does this).
The functions which are currently loaded are:

GetDpiForSystem (https://msdn.microsoft.com/en-us/library/windows/desktop/mt748623(v=vs.85).aspx), windows 10 or later
GetDpiForMonitor (https://msdn.microsoft.com/en-us/library/windows/desktop/dn280510(v=vs.85).aspx), windows 8.1 or later
SetProcessDpiAwareness (https://msdn.microsoft.com/en-us/library/windows/desktop/dn302122(v=vs.85).aspx), windows 10 or later

Unfortunately, this is not working properly on Windows 10. It builds and runs correctly, but then it prints:
Could not load `GetDpiForSystem`. Windows 10 or later is needed
Could not load `SetProcessDpiAwareness`. Windows 10 or later is needed

And then it doesn't display in hi-dpi at all.
I could try to debug this, but maybe that's enough of a clue to figure out what's going wrong?
Thanks for the effort, I appreciate it.

Ack, accidentally merged, sorry about that.

Ok, I think the issue is just that I made a silly mistake. GetDpiForSystem comes from user32, while SetProcessDpiAwareness comes from shcore, but I had it the other way around in code. Also, SetProcessDpiAwareness is available on windows 8.1, so I can test that.
I'll have a commit ready in a few minutes, just need to test stuff.
Should I open another PR since this one is closed? Not quite sure how to do this without messing up git :/

Yes, this works fine and I'm happy to merge it. There's probably some fancy way to update a merged PR, but it's probably just easiest to open a new one. Thanks for seeing this through.


Don't check Twitter URLs

Twitter completely locked down and requires
a login to read tweets. (Temporarily) disable all
Twitter URLs to avoid false-positives.
For context:
https://github.com/zedeus/nitter/issues/919
https://news.ycombinator.com/item?id=36540957
https://techcrunch.com/2023/06/30/twitter-now-requires-an-account-to-view-tweets/
Fixes https://github.com/lycheeverse/lychee/issues/1108

What cookies should one add to headers = [] to get content from Twitter?

Surprising fact: twitter.com responds with 200 OK for a non-existent Tweet!

What cookies should one add to headers = [] to get content from Twitter?

I honestly never tried. I can see a few cookies in my Firefox storage tab inside the browser tools. Stuff like auth_multi and auth_token and twid. I would probably start with these. 🤷♀

Surprising fact: twitter.com responds with 200 OK for a non-existent Tweet!

Oh yeah, it's completely unpredictable what you might get at times. It leads to so many false-positives that it will be completely disabled by default in the next release.


Aliases unavailable in WHERE clauses.

Example: SELECT 1 AS foo WHERE foo = 1.

After discussions offline, let's not support this. MySQL does not allow it, and while SQLite supports some flavor of this (namely when the alias refers to a non-aggregate expression), it would take significant work to support it in SKDB with limited benefits.


Update Kotlin to 1.6.21, share versions within the project by using version catalogs

All the versions are stored in libs.versions.toml file. Even versions inside Kotlin modules are synced with that file because it gets placed to JAR resources. Versions are set manually only in README.md and examples.
Kotlin was updated to 1.6.21, and necessary changes are made (minimal support for context receivers, compiler API changes are handled).
Other dependencies were updated, too.
Gradle was updated to 7.4.2

@nbirillo approve the CI run, please


Remove sigar dependency, remove system MBeans

This is for issue#781.

Removed the dependency on Sigar.
Removed all management system classes and beans since they no longer provide any value now we have removed sigar. This means the System (MBean) node and child nodes is no longer visible in jconsole.
Cleaned up all compilation warnings in the management code.

LGTM

travis failure... I think it is unrelated to this change: https://travis-ci.org/kaazing/gateway/builds/176209485

Test failure was due to an error in the test (not due to this PR).


Update pydantic to v2 and update datamodel-code-generator to 0.25.6

I saw your PR https://github.com/koxudaxi/fastapi-code-generator/pull/379 and I needed this feature.
Also I needed pydantic v2 support.
I forked your branch, installed pydantic v2 and fixed the bugs.
@koxudaxi Please take a look and let me know this is fine or not.

Codecov Report
Attention: Patch coverage is 97.22222% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 92.26%. Comparing base (0c5db9a) to head (f11f922).
Report is 18 commits behind head on master.

Files
Patch %
Lines

fastapi_code_generator/__main__.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #408      +/-   ##
==========================================
+ Coverage   92.16%   92.26%   +0.10%
==========================================
  Files           6        5       -1
  Lines         383      401      +18
==========================================
+ Hits          353      370      +17
- Misses         30       31       +1

Flag
Coverage Δ

unittests
92.26% <97.22%> (+0.10%)
:arrow_up:

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

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

Thanks for the great PR.
To be honest, this project has been a lower priority than my other projects, so I have not been able to devote much resources to it.
I will review it as soon as possible.


fix: Allow setting images.prefix to empty string

Description

Setter for Images.prefix now allows to set to an empty string.
Added doc-string for the .prefix property.

Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[-] I have updated/added relevant examples in examples.

Breaking Change

[x] No, this is not a breaking change.

Related Issues

Changes LGTM, but the title of the PR is misleading, it was already possible to set an empty string

@erickzanardo There was a typo in this assert:
assert(prefix.isEmpty || value.endsWith('/'), _assertMessage);

(prefix instead of value), which prevented an empty value from being accepted.

Oh, got it, maybe we should just remove the Allow from the title then, because it was already allowed, but there was a bug on it.
Just want to make sure that we don't get confusions with the changelog

Ok, does this sound better now?


fix(pdk) kong.ctx.plugin.* must be stored in ngx.ctx

Summary
I'm working on a custom plugin that involves redis.incr and redis.get (almost same process than the rate-limiting plugin ).
I was surprised to see my redis key not follows my requests number (difference between 10% to 30%) when traffic increase.
This problem occurs when syslog or prometheus plugin are activated.
seems to be related to https://github.com/Kong/kong/issues/4379
Context
In my plugin I set log in access phase:
function plugin:access(plugin_conf)
  ...
  kong.log.inspect(kong.ctx)
  ...
end

Run apachebench ab -n 5 -c 1 ... :
everything ok, i see 5 times my plugin.
[kong] handler.lua:?:67 {
  keys = {
    plugin = {
      PRIORITY = 10,
      VERSION = "0.1",
      access = <function 1>,
      header_filter = <function 2>,
      log = <function 3>
    },
    <metatable> = {
      __mode = "v"
    }
  },
  <metatable> = {
    __index = <function 4>
  }
}

Run apachebench ab -n 80 -c 50 ... :
Like previous, i see many log with plugin PRIORITY = 10.
But now, i also have some plugin PRIORITY = 4 (which corresponds to syslog plugin)
[kong] handler.lua:?:67 {
  keys = {
    plugin = {
      PRIORITY = 4,
      VERSION = "2.0.0",
      log = <function 1>
    },
    <metatable> = {
      __mode = "v"
    }
  },
  <metatable> = {
    __index = <function 2>
  }
}

PDK problem research
kong.ctx.keys which stores plugin is a VM-local variable and will be shared across multiple requests being handled by different coroutines.
Proposal
Store plugin key in ngx.ctx because it isn't shared across requests.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@tdelaune, can you tell us what version of Kong you had this problem? I guess it is the latest 2.0.0 to 2.0.4?

The problem occurs on 2.0.0, I Haven't tried the previous releases.
When I found the ticket #5459, I migrated to 2.0.3, and it still present.
I resolved issue with a docker image build from my branch.

@tdelaune,
Can you show us your actual code where you use kong.ctx? Are you trying:
kong.ctx.blaa = "test" -- not supposed to work

OR
kong.ctx.plugin.blaa = "test" -- supposed to work (accessible only within the same plugin)

OR
kong.ctx.shared.blaa = "test" -- supposed to work

@tdelaune,
Can you show us your actual code where you use kong.ctx?

Nevermind, the case is as you explained. kong.ctx.keys may get overridden by other light-thread on same worker if your code yields (as it certainly will if you store anything in redis).

@bungle  exactly this is only related to kong.ctx.plugin key, if I use kong.ctx.shared it works.
kong.ctx.shared works because it's stored in ngx.ctx so only related to current request.

@thibaultcha thanks for your review !

Agree, prefix is a great value
I discover metamethod so it's relevant for me to replace .set by __newindex to be more clear with this metatable use. (first days on Lua for me :blush:)
Sorry about that, it should be fixed !

@thibaultcha, I add the tests case for this issue.
As location doesn't share their ngx.ctx, we are in same condition, pdk.ctx.plugin should not be shared across request.
I run this test on master and it fails, namespace isn't empty.

Something wrong with my changes ?

@tdelaune Nothing inherently wrong with it, except maybe for the test cases. I opened #5873 (of which I made you a co-author of the kong.ctx fix) with a more elegant fix (see details in the PR, we now avoid the kong.ctx.set which was the function implemented in this PR), and additional ones for kong.log as well.

Closing in favor of #5873.


Saved data in array

Hi
I just want to save simple JSON on parse object of type array:
var myVal = {"projet":myProjet,"token":token};
myVal = JSON.stringify(myVal);

I see on parse dashboard my saved data:
 [" {\"projet\":\"zt5iqPU8Ex\",\"token\":\"xxxx-11432797634-114260xxx3-17726860081-917b228dc2\"}"]
Why do I have these " and these \ added on my field ?

because you stringified your val, so it escapes the content of your JSON object

Yes but if I don't do that and send myVal directly on my cloud function it does not work
Thank you anyway
Parse.Cloud.define("slack_token", function(request, response) {

Parse.Cloud.useMasterKey();

var token = request.params.A;
var myId = request.params.B;

////
var query = new Parse.Query(Parse.User);

    query.get(myId, {
    success: function(user) {

    user.addUnique("slack_token",token);

    user.save(null, {
    success: function(user) {
        response.success(user);
    },
     error: function(user, error) {
        response.error(error.code);
    }
    });

    },
    error: function(object, error) {

    response.error(error);

    }
    });
////
//Fin
});

Parse.Cloud.useMasterKey is not available in parse-server;
you should pass { useMasterKey: true }

I did that change, thank you for that
Then I did not JSON.stringify myVal
myVal that is sended is then : [object Object]
and I get this answer parse_SDK.js:15 POST https://labcyboolo.herokuapp.com/parse/functions/slack_token 400 (Bad Request)

How do you call you cloud function?

Parse.Cloud.run('slack_token',{A:myVal,B:user.id},{
              		success: function(result){...

So in the cloud functions, the param A is [object Object] ? or is it what is stored into the database?

Yes it is my console.log for var myVal = {"projet":myProjet,"token":token}; if I don't JSON.stringify myVal
if i JSON.stringify(myVal) it is saved and I have my array on parse dashboard:
[" {"projet":"zt5iqPU8Ex","token":"xxxx-11432797634-114260xxx3-17726860081-917b228dc2"}"]
if i don't JSON.stringify, my params A is [object Object] and I have the bad request response

And this code was working well when I was on parse.com

So do JSON.parse in your cloud code function if you send it as JSON.stringify

I think it comes from addUnique which accepts only string attribute...

Maybe it comes from addUnique I am using in my cloud function?


Add Temporal.io Scaler

Proposal
Add a Temporal.io Scaler to Keda so that workers can scale when the number of Open Workflows increases.
Scaler Source
https://temporal.io/
Scaling Mechanics
Calling the Temporal SDK ListOpenWorkflows method and scaling based on the value
Authentication Source
mTLS
Anything else?
https://github.com/temporalio/temporal/issues/33

@tomkerkhove when you get time please review/ give feedback for this PR.
https://github.com/kedacore/keda/pull/4863/files
thanks

any movements on this issue?

PR is being revied and waiting for contributor to reply: https://github.com/kedacore/keda/pull/4863#pullrequestreview-1613227542


Add Ping API flow to hit the Shows endpoint

Both buttons can use the same output pane

Running the code in the


PHP 5.3

can we reduce the dependency for this repo to 5.3?
both lusitanian/oauth and oryzone/oauth-user-data requires 5.3
PHP 5.3 is used by 40.9% of all the websites.
http://w3techs.com/technologies/details/pl-php/5/all
I saw some short array syntax. Other than that is there any reason not to downgrade requirement to 5.3 ?

+1

+1

:-1:
w3techs is a terrible source of this type of information (how is 5.7 even on the chart?). This is a much better view, where in Nov of last year 5.3 was at 14.85%. I would be surprised if it passes 10% this year. Upgrade your app to 5.6 and don't drag the rest of the ecosystem down, especially since you have a perfectly valid alternative.


Display authors and their socials on kcp blog posts

Other CNCF projects (e.g. Kubernetes) usually list authors of their blog posts, and I'd like to add this information to our blog posts as well. This is part of my hopes to encourage people to write about their experience with kcp in the future maybe and get the attribution.
I've not attached author metadata to the 2022-era posts because I'm not sure their authors still want to be associated with kcp.
This is how it looks:

/cc @mjudeikis @sttts

/lgtm
/approve


feat(linter/tree-shaking): support ConditionalExpression

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2965  👈
#2964
main

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

Join @mysteryven and the rest of your teammates on  Graphite

Merge activity

Apr 14, 1:39 AM EDT: @Boshen started a stack merge that includes this pull request via Graphite.


Content about nodejs loader

@JakobJingleheimer could you review this content. Second part, I mean second post will be how to use nodejs-loaders .

@JakobJingleheimer  wanna be added as author ?

@JakobJingleheimer wanna be added as author ?

🥰 thank you, but you wrote 99.99% of this. I don't wanna steal your thunder.

Big thanks Jacob for your review

You're welcome!


Tensorflow 2 not supported by Ultra96v2

Hi, I tried the example in vitis ai tutorial repository for signal modulation detection. I was on keras on tensorflow 2. The dpu does not give any results on ultra96. It outputs all zero values. Do you know the reason?
thanks

Perhaps there is a large current consumption of FPGA.
https://github.com/Xilinx/Vitis_Embedded_Platform_Source/blob/2019.2/Xilinx_Official_Platforms/zcu104_dpu/petalinux/project-spec/meta-user/recipes-ai/dpuclk/files/dpu_clk
This script will reduce the frequency of the PL_CLK0 (100MHz) clock source that feeds the Clock Wizard that generates the multiple clock frequencies, available for Vitis.
See: https://www.hackster.io/AlbertaBeef/vitis-ai-1-1-flow-for-avnet-vitis-platforms-part-1-007b0e

Hi @hghari,
It is difficult to understand your situation without any sample of log or snip of code (your app code?).

The dpu does not give any results on ultra96

You meant you interact with DPU correctly, but output data is zero after you run DPU?
If so, could you put the code which run DPU and the code which get output from DPU?

Hi @AVprog  can the output being zero be related to clock rate?
Hi @hoadv-qh I am using the example in vitis tutorial. this link : https://github.com/Xilinx/Vitis-Tutorials/tree/master/Machine_Learning/Design_Tutorials/10-RF_modulation_recognition
I have trained it with both tensor 2 and tensor 1. Then quantized and compiled it successfully but when I use test_accuracy.py file it gives all zeros for the output. Thanks

Hi @hghari

I have trained it with both tensor 2 and tensor 1. Then quantized and compiled it successfully but when I use test_accuracy.py file it gives all zeros for the output. Thanks
You meant below, right?

1st: You created float model and trained, then got accuracy as your expectation
2nd: You quantized float model and got INT8 model. Then you evaluted INT8 model and also got accuracy as your expectation
3rd: You compiled INT8 model to Ultra96V2, and run on the board. BUT GOT ACCURACY 0%.

In this case, I prefer to debug on Ultra96V2 board with idea below.

Print out outputData[j] whether output tesors has "normal" data.
You can try to do post processing to get human-readable output by using same way as Float model or INT8 model
(as in vai_2018_RadioML_VAI_keras.ipynb)
I saw that post process of those models only uses numpy. So it is easy to run on Ultra96V2.


[Backport releases/v0.1] chore: align dependencies on FM fork of tonic_lnd

Description
Backport of #3235 to releases/v0.1.

Docs point to files on master that have been deleted

Can markdown do relative links?

Can markdown do relative links?

In our case there's no such thing, as how the markdown files get rendered and where shown (under what URL) is decoupled from the markdown files itself.
Because of this we use absolute URLs instead.
I think it's not the end of the world that links stopped working on a (soon to be) "old" release branch.
But generally I'd advise not linking to the code from docs and move code documentation into the code.


Link to paragraph
https://dbalague.pages.ewi.tudelft.nl/openlabook/Chapter5/DeterminantsGeometric.html#fig-detgeometric-paraped
What type of applet is this?
Interactive applet 3D
What state is the static image?
Yes, Dennis has created a static image
Static image

Requirements
Volume equals base area times height
What elements are interactive?
Must be able to rotate around the 3D figure.
Does this issue relate or depend on other issues?
No response

try to fill in the parallelopiped


Add support for specifying prefix when auto scanning for tags within directory

We should allow prefixes to be provided when scanning for tags within a directory.
For example, to scan single directory:
{
    "tags-dir": {
        "path": "./components",
        "prefix": "app-"
    }
}

To scan multiple directories:
{
    "tags-dir": [
        {
            "path": "./components",
            "prefix": "app-"
        },
        {
            "path": "./components/chat",
            "prefix": "chat-"
        }
    ]
}

I see how this could be helpful. Proposal looks good to me.


Lose information in the Name(AccountRef) field after deserialization

hello,
I want to call QBO API to get JournalEntry Entities, but I have a problem with the Name field, like next image, I'm losing information:

Before Deserialize response :

After Deserialize :

I'm losing this information !
I use Symfony 4.2 and "quickbooks/v3-php-sdk": "dev-master",
Thanks
Sidi

@sidikhalifa
This is a feature that will be fixed in the new Quickbooks PHP SDK. For current SDK, fixing the xml conversion issue(fix the library) will cause code break.


Completion not working

Description:
var x = function(int arg1, int arg2, string... a) returns int {
        return 12 + a<cursor>;
};

Invalid suggestions are provided at the cursor
Affected Versions:
v1.0.0 at least

Fixed in latest master, SLP8

Fixed in latest master, SLP8


Storage concurrently tests and bug fix

Storage concurrently tests and bug fix
pr https://github.com/prometheus/prometheus/pull/6770
issue #6714

A new fanout directory is created to add tests, because using teststorage.new (T) in the storage directory will cause the following error
github.com/prometheus/prometheus/storage
FAIL	github.com/prometheus/prometheus/storage [setup failed]
import cycle not allowed in test
package github.com/prometheus/prometheus/storage (test)
imports github.com/prometheus/prometheus/storage/tsdb
imports github.com/prometheus/prometheus/storage

pr #6770
Through the test case, it is found that in the last PR, one line of code is missing, resulting in the result being empty

tests code

tests PASS

Thanks!


how to disable the default extension (prosemirror-dropcursor) and import it manually?

I rewrite the dropcursor,  but do not know how to disable built-in plugins(prosemirror-dropcursor).

Or is there some way to rewrite the dropCursor style?
...
    // no way to change default cursor style (border-radius? :after? :before?)
    this.element.style.left = (rect.left - parentRect.left) + "px"
    this.element.style.top = (rect.top - parentRect.top) + "px"
    this.element.style.width = (rect.right - rect.left) + "px"
    this.element.style.height = (rect.bottom - rect.top) + "px"
...

You can pass options to the the dropCursor Plugin
new Editor({
  dropCursor: {
    color: '#f00',
    width: 5,
  },
});

A class on the dropcursor would be better but thats an issue of  prosemirror-dropcursor

@Chrissi2812 Thank you. I notice the options of dropCursor, but I want to disable it and use the modified version of dropCursor extension.

https://github.com/scrumpy/tiptap/pull/705

Merged. 👍  Will be part of the next version.


Widen swift-syntax dependency to 508.0.1..<510.0.0

Following discussion from #794, we can be a better citizen in the ecosystem and play nicely with those that have stricter dependencies.
I've made assertInlineSnapshot simply unavailable when another dependency locks to 508.0.1. I think there are too many significant changes in swift-syntax 509 to support the function in earlier versions.

This is a complete success, thank you very much! This solves #794 completely. :-)
I'll wait a little bit more for inline snapshot testing 😉


\angle symbol is not working

Describe the bug
I tried to display angle symbol but when I run app only red rectangle is displayed.
formulaControl.Formula = @"\oint_{S}\vec{E}d\vec{S}=\oint_{S}EdS*\angle (\vec{E},d\vec{S})=\oint_{S}EdScos, 0=\oint_{S}Eds=E\oint_{S}dS=ES";
To Reproduce

Steps to reproduce the behavior:

Call '...'
Set '....' to true
An InvalidOperationException is thrown with stack trace...

Expected behavior
A clear and concise description of what you expected to happen.
Environment (please complete the following information):

Platform: [e.g. CSharpMath.SkiaSharp, CSharpMath.Forms]
Package Version or Commit: [e.g. 0.0.0, a2315b4ac6d9cfdf8cf3dc2aaed37ee2f1dad07a]
Device: [e.g. iPhone 6, Pixel 2 XL, Xbox One X]

Additional context
Add any other context about the problem here.

Wrong repo?

I accidentally posted here instead on wpf-math


Create Disappearing Alert

Create a pop up, auto disappearing message for the user to let them know that the track they clicked to save was successfully saved

This issue was moved to kroehl/MusicDiscoveryApp#16


Support for gltf 2.0 texture compression

Support for gltf 2.0 texture compression
add
compressTexture.js
compressTextures.js
generateJimpImage.js
getTempDirectory.js
from 1.0 branch
update
/bin/gltf-pipeline.js
/lib/processGltf.js
/lib/writeResources.js
package.json

Hi @verybigzhouhai, thanks for the contribution but I don't think we'll end up merging this. Right now we're waiting for a proper glTF extension for compressed textures and don't want to maintain this custom extension. But you are welcome to keep using this code for your own needs.


add option to render summary or full content in RSS feed

Added option to generate full feed or only summaries.
RSS feed generation changed in 0.20. Some prefer new behavior, some prefer old (at least me and there is also #3341 I just noticed), thus an option. It is set to true by default to honor current (new) behavior.
I believe a custom template is too complex for this task, because this is probably the only thing I'd like to change and maintaining yet another template will require much more attention. And there is already rssLimit option, so why not? 😃

Either this or my pr would please me, although I think it's better for the client to decide to show a description or the full post, and to know if it's the full content. Maybe using the content namespace in this PR would be best?
<description>{{ .Summary | html }}</description>
{{ if ne .Site.RSSSummary }}<content:encoded>{{ .Content | html }}</content:encoded>{{ end }}

This way it's still an option for the site owner, and informs the client if it should display a "Read more" link.

although I think it's better for the client to decide to show a description or the full post

By this logic, wouldn't it make more sense to include .Description, the page variable> Summaries are typically auto-generated and confused with what is really the idea of a "lead" paragraph. This conversation has taken place before on a different PR/issue...

See this to prefer .Description over .Summary. I still it should be up to the client (and the publisher) to decide on full content vs a "description" not the CMS.


HTTP 3GP mobile fails to load

I am trying to get yt2009 web to work on my iphone 3g. It doesn't support modern codecs and file containers like mp4, so it requires 3GP. When you change the rstp flag to HTTP 3GP, the mobile video feed fails to load. (this also occurs on my PC.) It attempts to load till it times out. Nothing appeared in the output. This is normally under an apache proxy, but still occurs when connecting directly.
Here is my docker compose if it helps
services:
  yt2009:
    container_name: yt2009
    build: /opt/yt2009/repo/

    ports:
      - 3050:3050
    environment:
      - YT2009_ENV=prod
      - YT2009_IP=https://DOMAIN_HERE
      - YT2009_PORT=3050
      - YT2009_TOKENS=passphrase
      - YT2009_LOGGED_TOKENS=passphrase
      - YT2009_GDATA_AUTH=true
    volumes:
      - "/opt/yt2009/certs:/yt2009/certs"
      - "/yt2009/assets/site-assets"
      - "/opt/yt2009/assets:/yt2009/assets"
      - "/opt/yt2009/logs/access.log:/data/accessdata"
    restart: unless-stopped

i've identified the cause, it's because the ffmpeg build from alpine linux doesn't support amr-nb encoding
i'll try to work out a proper solution but in the meantime you can search and replace libopencore_amrnb with aac in back/yt2009mobile.js and videos should start working on your iphone


feat(stoneintg-507): use base image opm binary for fbc validation

After verifying that the base image is trusted (see STONEINTG-503) we need to use the opm binary from the base image to run fbc validation. Because Tekton tasks do not easily support running nested containers, this change extracts the base image and use the opm binary within it to validate the target image.

/retest

All my comments were marked as resolved, but I see no changed, have you forgotten to push cahnges?

All my comments were marked as resolved, but I see no changed, have you forgotten to push cahnges?

Apparently I had Friday brain last week.  Yes, changes were made and never pushed.  They were, however, verified in quickcluster.

/retest


TypeError: parse_instruction() takes 3 positional arguments but 4 were given

While generating the dataset I noticed that the function def parse_instruction(ins, symbol_map, string_map) in dataflow_gen.py, when called by the random_walk function “s.append(parse_instruction(g.node[n]['text'], symbol_map , string_map, str_list))”. There are four parameters and an error is reported. Is this parameter redundant, or is there an error in my calling method?

It's my fault. That parameter is useless in this function and you can delete it. It should be a historical issue. Sorry about that.


Does not pluralize table names

For example, my table is named "Summaries" and the model is named Summary. It is attempting to insert into Summary which does not exist.

Fixed in the latest version available at https://www.nuget.org/packages/EFBulkInsert/0.3.0


Add support for Wifi

Hello,
first thanks for your work providing this API.
I would like to add support for wifi clients listing so that we can use it for presence detection in HA.
I started working on it before reading that you advice on creating an issue before coding.
Specifically, current code is looking for "namespace" in response but the method wlan.getClientList does not have "wlan" namespace in the response.
Do you have any preference on how I should handle this ?
Thanks

I'm sorry it seems that I didn't see this message.
I don't have any preference right now, but feel free to create a PR and we can review that.


[BUG] Type-checker excepts ill-typed specs

Description
I was trying to write a default initialisation operator. I have many variables, say 10. I want to write tests, and each test touches 1 or 2 from 10. Initializing all 10 variables in every test is not a viable solution; so I want to factor out the defaults into one operator, and in each test to set only the values for the variables I am interested in. The solution I came with is probably not a type-correct one, but it passes the type-checking phase, and then triggers an unhandled exception:
PASS #11: OptimizationPass                                        I@11:54:29.855
 > Applying optimizations:                                        I@11:54:29.859
  > ConstSimplifier                                               I@11:54:29.859
  > ExprOptimizer                                                 I@11:54:29.862
(Please report an issue at: [https://github.com/informalsystems/apalache/issues],java.lang.IllegalArgumentException: Access to non-existent record field b)
Unhandled exception                                               E@11:54:29.865
java.lang.IllegalArgumentException: Access to non-existent record field b
	at at.forsyte.apalache.tla.pp.ExprOptimizer$$anonfun$transformFuns$1.applyOrElse(ExprOptimizer.scala:46)
	at at.forsyte.apalache.tla.pp.ExprOptimizer$$anonfun$transformFuns$1.applyOrElse(ExprOptimizer.scala:38)
	at scala.PartialFunction$OrElse.apply(PartialFunction.scala:172)
	at at.forsyte.apalache.tla.lir.transformations.impl.TrackerWithListeners.$anonfun$trackEx$1(TrackerWithListeners.scala:16)
	at at.forsyte.apalache.tla.pp.AbstractTransformer.$anonfun$transform$1(AbstractTransformer.scala:36)
	at at.forsyte.apalache.tla.lir.transformations.impl.TrackerWithListeners.$anonfun$trackEx$1(TrackerWithListeners.scala:16)
	at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:286)
	at scala.collection.mutable.ResizableArray.foreach(ResizableArray.scala:62)
	at scala.collection.mutable.ResizableArray.foreach$(ResizableArray.scala:55)
	at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:49)
	at scala.collection.TraversableLike.map(TraversableLike.scala:286)
	at scala.collection.TraversableLike.map$(TraversableLike.scala:279)
	at scala.collection.AbstractTraversable.map(Traversable.scala:108)
...

Input specification
Here is the MWE:
------------------------------------ MODULE expr_bug -----------------------------

EXTENDS Integers

VARIABLE
\* @type: Int;
a
,
\* @type: Int;
b

\* @type: (Str -> [a: Int, b: Int]) => Bool;
InitDefaultsExcept(vars) ==
  /\ a = IF "a" \in DOMAIN vars THEN vars.a ELSE 0
  /\ b = IF "b" \in DOMAIN vars THEN vars.b ELSE 0

Init ==
  InitDefaultsExcept([
    a |-> 1
  ])

Next ==
  UNCHANGED <<a,b>>

===============================================================================

The command line parameters used to run the tool
apalache check expr_bug.tla

System information

Apalache version : 0.15.9-SNAPSHOT build v0.15.8-10-g73fd704
OS: Ubuntu 20.04.2 LTS
JDK version: openjdk version "15.0.2" 2021-01-19

More problem context
Just for the context: while the example above is very simplistic, the problem I am trying to solve is a real one, with default initialisation for unneeded variables. Here is a larger example:
\* @type: ([ vats: VAT_TABLE, await: MESSAGE, runQueue: Seq(MESSAGE), koNextId: Int, kpNextId: Int,
\*   kernelToVat: KERNEL_TO_VAT, vatToKernel: VAT_TO_KERNEL,
\*   kernelObjects: OBJECT_TABLE, kernelPromises: PROMISE_TABLE]) => Bool;
InitDefaultsExcept(vars) ==
  /\ vats = IF "vats" \in DOMAIN vars THEN vars.vats ELSE EMPTY_VAT_TABLE
  /\ await = IF "await" \in DOMAIN vars THEN vars.await ELSE NoMessage
  /\ runQueue = IF "runQueue" \in DOMAIN vars THEN vars.runQueue ELSE <<>>
  /\ koNextId = IF "koNextId" \in DOMAIN vars THEN vars.koNextId ELSE 1
  /\ kpNextId = IF "kpNextId" \in DOMAIN vars THEN vars.kpNextId ELSE 1
  /\ kernelToVat = IF "kernelToVat" \in DOMAIN vars THEN vars.kernelToVat ELSE EMPTY_KERNEL_TO_VAT
  /\ vatToKernel = IF "vatToKernel" \in DOMAIN vars THEN vars.vatToKernel ELSE EMPTY_VAT_TO_KERNEL
  /\ kernelObjects = IF "kernelObjects" \in DOMAIN vars THEN vars.kernelObjects ELSE EMPTY_OBJECT_TABLE
  /\ kernelPromises = IF "kernelPromises" \in DOMAIN vars THEN vars.kernelPromises ELSE EMPTY_PROMISE_TABLE

TestCreateUnknownVatBefore ==
  InitDefaultsExcept([
    vats |-> [ id \in {"a", "b"} |-> NEW_VAT(FALSE)]
  ])

Right, that is the effect of the non-standard record unification rule, which was designed to accommodate unions in Paxos. We should implement #789 and this issue will go away.

FYI, I’ve managed to trick Apalache by introducing a local definition; this
------------------------------------ MODULE expr_bug -----------------------------

EXTENDS Integers

VARIABLE
\* @type: Int;
a
,
\* @type: Int;
b

\* @type: ([a: Int, b: Int]) => Bool;
InitDefaultsExcept(update) ==
  LET vars == update IN
  /\ a = IF "a" \in DOMAIN vars THEN vars.a ELSE 0
  /\ b = IF "b" \in DOMAIN vars THEN vars.b ELSE 0

Init ==
  InitDefaultsExcept([
    a |-> 1
  ])

Next ==
  UNCHANGED <<a,b>>

Inv == a = 1

===============================================================================

bypasses the type-checker completely, sets the initial state as I wanted, and satisfies Inv.

It's not really a bug, as it works according to the docs. This is a mismatch between user expectations and how the type checker is working. I agree that we should change this behavior as soon as possible. Most likely, in July.

As I've suggested elsewhere, it seems to me that an uncaught exception is a bug per se. A fix for the current behavior, would be a proper user error, reporting on the problem and how to fix it. All the better if the type checking makes this kinds of error impossible at run time, but presenting stack traces to users is broken behavior, imo.

Ah, but the bug part here does not fit the title of the issue. That's the bit that pertains to user expectations colliding with our unsound typing on records :)


Feat: adds health check endpoint

Before submitting

[x] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

What does this PR do?
Fixes #181 .

Adds health check endpoint /health to get the status of workers

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@aniketmaurya @bhimrazy did this ever get exposed in an endpoint?

hi @craigpfeifer, yes - you can access the health endpoint at path /health. If you are on localhost then it should be http://localhost:8000/health.

@aniketmaurya thank you I'd be happy to add this to the docs

@craigpfeifer It's already here in the docs.


renamed "property" to "item" in "where" method and removed spaces when it is a label

should there be a xor_where method?
also, there are no tests for and_where and or_where methods
is it possible to build a query using WHERE (n)-[:KNOWS]-({name: 'Timothy'})?

I could address some of these questions, but need confirmation (clarification?) whether to go in that direction.

Just a quick comment, before I check what you did here. All PRs should merge with develop branch, that is, you have to rebase this PR. In that way you'll be up to date with all the new changes for the new release.

It is not possible to build query is it possible to build a query using WHERE (n)-[:KNOWS]-({name: 'Timothy'}), since patterns in expressions are not supported in our implementation of Cypher.

should there be a xor_where method?
also, there are no tests for and_where and or_where methods
These two questions are too familiar for me, whether I asked them already, or implemented them, I'll get back to you once I check the develop branch or one of the current PRs.

xor_where, and tests are already in develop branch, I didn't see it as I was based in main.
So, all my questions are answered


Add support for new languages: French

(originally created by @KSLHacks)
Add support to enable French speaking users to utilize this bot service.
See Issue #1  for guidance.

Big thanks to @pada57 for closing this issue!


Test Fix: Handling case sensitive ls output

Minor Test bug Fix: Updating regex for handling case sensitive nature of ls output.

how did you see that ?

In my local setup, the user name and group is uppercased and when running test cases, this particular test errored out. Tracking back on that issue, updating regex fixed the issue.


build<EMAIL_ADDRESS>

Describe pull-request
This PR upgrades our core package to use the latest Stencil version: 4.6.0.
NOTE
This PR is based off of https://github.com/scania-digital-design-system/tegel/pull/374, this is due to jest and puppeteer being removed in that PR. This is a prerequisite for us to move to 4.6.0.
This PR should NOT be merged before  https://github.com/scania-digital-design-system/tegel/pull/374
Solving issue
Fixes: CDEP-2908
How to test

Check out branch
Run npm install in packages/core
Run npm run build

Needs rebase!

I don't think this should be rebased now. Like specified in the description, this PR isn't really relevant until the playwright PR has been merged. So I think we can leave it until then.

Needs rebase!

I don't think this should be rebased now. Like specified in the description, this PR isn't really relevant until the playwright PR has been merged. So I think we can leave it until then.

Alright! FYI, tried it out locally and compiles.

Needs rebase!

I don't think this should be rebased now. Like specified in the description, this PR isn't really relevant until the playwright PR has been merged. So I think we can leave it until then.

Alright! FYI, tried it out locally and compiles.

Steak sauce!

You will get some conflicts due to the "hotfix" release branch we merged in develop in the Tegel react wrappers release.  Ping if you need some help there, but in a nutshell @scania/tegel-react should have 1.1.0 as version too.


Is there a working react example somewhere?

Is there, by chance, a codepen or something somewhere that shows how to set this up on react, like your other components which are in storybook?
I keep getting this error:
TypeError: null is not an object (evaluating 'valuesArea.querySelector("[data-id=\"" + name + "\"] input").value = obj[name]')
(anonymous function)
src/react-scenejs-timeline/Timeline.tsx:407
  404 |     this.values = obj;
  405 |     const valuesArea = this.scrollArea.valuesArea.getElement();
  406 |     for (const name in obj) {
> 407 |         valuesArea.querySelector<HTMLInputElement>(`[data-id="${name}"] input`)!.value = obj[name];
      | ^  408 |     }
  409 | }
  410 |

I'm sure I'm doing something wrong, but I can't figure out what.
Edit: Looking a bit further, I can see Value.tsx is creating an input without the required attribute in Timeline.tsx. Again, I'm sure I'm doing something wrong.

@jessejamesrich
Temporarily fixed bugs and released. try updating
v1 version of react-scenejs-timeline is being prepared.
When released, I think we will be able to prepare a proper demo.
https://github.com/daybrush/scenejs-timeline/tree/v1/packages/react-scenejs-timeline

Oh yeah. That worked. Thank you! Your work is amazing!


Add support for returning the current field value.

@flixx

@flixx

@flixx

Hello @jackton1 !
First of all, thanks for the contribution!
Could you write a few sentences here on what the purpose of that change is? Why would you want to return the current value? I guess because you still want to use the deprecated field normally at runtime but just want to have a deprecation warning and an indication in the code that it is not meant to be used anymore? Is that correct?
Best,
Felix

Hello @jackton1 !
First of all, thanks for the contribution!
Could you write a few sentences here on what the purpose of that change is? Why would you want to return the current value? I guess because you still want to use the deprecated field normally at runtime but just want to have a deprecation warning and an indication in the code that it is not meant to be used anymore? Is that correct?
Best,
Felix


Define and add unit test for regularization tuning

In GitLab by @nmansard on Nov 28, 2018, 09:43

In GitLab by @nmansard on Nov 28, 2018, 09:47
Follows #18

In GitLab by @nmansard on Jan 21, 2019, 15:28
This is still valid for refact: the unittest on automatic tunic are not there yet. I keep it open for the refact branch, but this is not a top priority task yet.

In GitLab by @nmansard on Jan 30, 2019, 17:51
closed


perf(front): #664 worktree vs clone

When running current directory projects, use
a worktree instead of a git clone. Creating
a worktree is just amazingly fast and also
has access to the git history

it really depends on disk IOPS, but on my machine before this change:
$ time m .

real	0m4.475s
user	0m3.466s
sys	0m0.930s

$ time MAKES_GIT_DEPTH=0 m .
real	0m17.675s
user	0m20.986s
sys	0m3.749s

Now:
$ time m .
real	0m3.140s
user	0m1.133s
sys	0m0.607s

$ time MAKES_GIT_DEPTH=0 m .
real	0m3.143s
user	0m1.070s
sys	0m0.648s


style(all): add spaces to imports

Description:
add spaces to imports

@kwonoj halp? seems like there's a dangerjs issue. // @benlesh

@ladyleet Guess so, I'll try to peek asap.

Coverage remained the same at 97.402% when pulling 84f689047e5929a73015e262cc9ba06fa7b2fd00 on ladyleet:spaces-to-imports into 7bb8280181bf1d5b0a2d3338ede8bea5e30f64c7 on ReactiveX:master.

@ladyleet I got you and it's now good.

Thx @kwonoj! And @benlesh

I've reconciled the merge conflicts and merged manually. Thanks, @ladyleet!


[Function] Phase 2 of Prompt Library BE

[ ] Bookmarking functionality using additional prompt/user relation table
[ ] Add "most popular" tracker on prompts

Waiting for PR for phase 1 to be merge and FE component to finish

Make sure to add version ids and versioning for prompts

https://www.figma.com/design/90i9MKLBGrW9imf0wLWvZ0/Agent-Builder?node-id=856-72479&t=8ShlrHOPJtkuKdyd-1
Figma link


chore: make tests great again

Port tests from blinklabs-io/kupogo for datums and metadata and run an http server during tests to mock responses for live tests. Also, update the license headers to remove the really long lines.

Hey, thanks for this! I may spend some time to refactor out a small utility for defining a mock http server to reduce some of the repetition, but this looks great!

I am working on adding Script to complete the functionality. Can you hold off on the refactor until then? That'd be super.

The Script functionality is in #9 and is modeled after the http code in this PR


Problem with String comparison in VB.NET

Using VB.NET if i make a query that checks equality on a string field like this :
    Using st = store.QuerySession
        Dim rs = st.Query(Of User).Where(Function(a) a.Name = "Kevin")
        For Each itm In rs
            Console.WriteLine(itm.Id.ToString)
        Next
    End Using

i get :
System.ArgumentNullException: 'Value cannot be null.'
i must substitute '=' with a.Name.Equals() to get it done
but in this case, i can't use NOT to negate it
or i can substitute '=' with 'Is' or 'IsNot'  but this is not properly the same, i think...

@compucen you can use Dim rs = st.Query(Of User).Where(Function(a) Not a.Name = "Kevin"), this works!


Update Readme.md

Add Project Details
How to install
contributors
video
badges

I guess we've completed the required part.
Closing this issue


Overlapping content in the facility list

[This is a draft issue that will be updated with more details on how to reproduce the problem in development]

I've been looking into this issue a bunch this morning. This happens because we are setting a height: 55px on the divs within the table cells. We define this height to force alignment across each cell in a given row.
When the string bleeds beyond this 55px height we see the issue above.
If we remove the 55px height, then the cells no longer line up and it becomes very difficult to know which actions belong to which row. I have a potential solution in the form of truncating/tooltips. I'll be pushing a PR soon.

This bug is visible on the Puma T1 list dated 23rd June 2020:


the code are right with the direction, but don't get the result

What are wrong with my code?

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

jQuery Playground

Screenshot

@voeung, what challenge name?

@voeung thanks for the issue but if you get stuck or have questions with a challenge in the future, please first ask questions to the Help Room. Happy coding!


F-Secure.WithSecureElements version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

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

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree [company="{private}"]

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree [company="{}"]

Close with reason: Replaced by PR 82820 to avoid pull request being stale;


Resize agencies logos

Fixes https://github.com/OA-PASS/pass-ember/issues/206

resize logos

lookin' good!


Support "in progress" courses

[ ] add another tab for in-progress courses
[ ] add option to trigger degree status compute API with in-progress flag
[ ] support in-progress flag in compute_degree_status function

It would be great if the "in progress" courses could extend to future semesters as well, so you could use this tool to program the rest of your title.
Lets say I'm going to start semester 5, so I'd like to know that my future plans make sense.

@idanfi  that's the plan. :slightly_smiling_face:
btw, you can already simulate this behavior (although not as comfortably as it would be in a fully implemented feature) right now! create some semesters, add courses to them, and click "סגור את התואר".

I think the courses must have passing grade in order to be part of the calculation, which could be a bit annoying, but that's better than nothing :)

closed by #125


[pigeon] make toMap generated methods public

Use case
Consider the following pigeon file
class Some {
  int a;
  int b;
}

class SomeList {
  List<Some> list;
}

@HostApi()
abstract class ExampleHostAPI {
  void doSome(SomeList req);
}

Pigeon cannot serialize classes outside of primitives in Lists, so the way around that is to serialize the object to map<String, ..>.
Kotlin example:
var list = SomeList();
list.list = List(myList.size) { myList[it].toMap() };
...

Now a list of custom objects can be transported to flutter and vice versa. The issue is that the toMap functions are private as declared in both ios and android.
Map<String, Object> toMap() {
  Map<String, Object> toMapResult = new HashMap<>();

@interface SomeList ()
+(SomeList*)fromMap:(NSDictionary*)dict;
-(NSDictionary*)toMap;
@end

The method in java is not marked as public and for objective-c the fromMap and toMap are hiding in the .m file meaning they're private.
Proposal
There's no harm in the toMap/fromMap being public except that they pollute/may collide with existing functions. I'd propose they are made public or an option provided for them to be.

Cc @gaaclarke

Thanks for the suggestion, we have considered it.  There is actually harm in exposing those methods and they are left private on purpose.  They hide the implementation details about how messages are transferred which gives us the ability to change it completely without breaking people's code.  If we made them public we would have to maintain them even if we weren't using them for transmission anymore.

Thanks @gaaclarke , haven't considered that.
Can you suggest how to send List/Map types containing non-primitives?

Can you suggest how to send List/Map types containing non-primitives?

It never occurred to me that it wouldn't work.  You may be right, I don't think I ever tried it.  Can you please file an issue for that?  We'll have to switch the codec we use for channels probably to recursively encode correctly...  Thanks!

Pull out https://github.com/flutter/flutter/issues/80538


feat: remove sign in requirement for plugin

#289
#288  👈
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


Investigate auth service providers

Instead of implementing our own custom auth support, we should look into different options on the market already to get us faster to MVP. Some options:

https://auth0.com/
https://workos.com/
https://www.keycloak.org/
https://goauthentik.io/
https://ory.sh/
https://www.cloud-iam.com/

Things to take into consideration:

Possibility to use our own branding
Clear and simple migration path
Standard compliance (related to point above). Ideally, we want to use something that uses a well-defined standard protocol.
Available tooling for integration
Ability and ease of self-hosting the service or spinning up a custom instance

This may affect #7

Adding https://www.cloud-iam.com/ (hosted Keycloak)

So it seems that we want something that supports OIDC and propbably is on this list https://openid.net/developers/certified/

We went with auth0, which is by far the simplest and quickest to start with, feature-complete, and it also ticks all the boxes, since we can retrieve data about all our users, including provider-specific tokens via their API. There's functionality to trivially create a new tenant, we can customize a lot (but not everything).
Eventually we want to switch to https://www.ory.sh/, which we should be able to host ourselves completely and with some luck, run it in a keep one day. At the time of writing, their cloud offering is missing OIDC (Ory Hydra) functionality and we do not have the resources available to host it ourselves. The functionality should be available by Q4 this year, maybe earlier https://github.com/ory/cloud/discussions/50#discussioncomment-2848518. Let's revisit this then.


feat: add new group for dart

This PR
Following our community meeting here is the new group to prepare contributors for dart sdk server.

I am not sure why the CI is failing with the error
Configuration failed: failed to configure open-feature teams: failed to update sdk-dart-approvers members: failed to list sdk-dart-approvers(sdk-dart-approvers) members: return code not 2XX: 404 Not Found

@aepfli any idea on what can go wrong?


An error : <Signals.SIGSEGV: 11>

Prerequisite

[X] I have searched the existing and past issues but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version.

🐞 Describe the bug
apper an error when running with DDP  suddenly.
In this environment, running other python files are all ok, this error appear at specific epoch specific step. It very confused me.
Environment
fatal: unsafe repository ('/work/mmdetection' is owned by someone else)
To add an exception for this directory, call:
git config --global --add safe.directory /work/mmdetection
sys.platform: linux
Python: 3.8.5 (default, Sep  4 2020, 07:30:14) [GCC 7.3.0]
CUDA available: True
GPU 0,1,2,3,4,5,6,7: Tesla V100-SXM2-32GB
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.0, V11.0.221
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.7.1
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) Math Kernel Library Version 2020.0.2 Product Build 20200624 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v1.6.0 (Git Hash 5ef631a030a6f73131c77892041042805a06064f)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 11.0
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;
arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;ar
ch=compute_37,code=compute_37
CuDNN 8.0.5
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_VULKAN_WRAPPER -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.8.2
OpenCV: 4.6.0
MMCV: 1.6.1
MMCV Compiler: GCC 7.5
MMCV CUDA Compiler: 11.0
MMDetection: 2.25.0+
Additional information
No response

This is training logs, before it's all ok. It appear at specific epoch specific step.
2022-09-30 08:48:44,528 - mmdet - INFO - Epoch [13][250/7330]   lr: 1.000e-04, eta: 1 day, 1:49:05, time: 0.511, data_time: 0.026, memory: 12984,
loss_rpn_cls: 0.0325, loss_rpn_bbox: 0.0390, loss_cls: 0.1931, acc: 93.1592, loss_bbox: 0.2324, loss_mask: 0.2375, loss: 0.7345
2022-09-30 08:49:10,145 - mmdet - INFO - Epoch [13][300/7330]   lr: 1.000e-04, eta: 1 day, 1:40:39, time: 0.513, data_time: 0.026, memory: 12984,
loss_rpn_cls: 0.0326, loss_rpn_bbox: 0.0400, loss_cls: 0.1931, acc: 93.1221, loss_bbox: 0.2344, loss_mask: 0.2319, loss: 0.7320
2022-09-30 08:49:35,657 - mmdet - INFO - Epoch [13][350/7330]   lr: 1.000e-04, eta: 1 day, 1:33:17, time: 0.510, data_time: 0.026, memory: 12984,
loss_rpn_cls: 0.0283, loss_rpn_bbox: 0.0389, loss_cls: 0.1920, acc: 93.3005, loss_bbox: 0.2284, loss_mask: 0.2306, loss: 0.7183
2022-09-30 08:50:00,766 - mmdet - INFO - Epoch [13][400/7330]   lr: 1.000e-04, eta: 1 day, 1:24:57, time: 0.502, data_time: 0.024, memory: 12984,
loss_rpn_cls: 0.0294, loss_rpn_bbox: 0.0378, loss_cls: 0.1908, acc: 93.2974, loss_bbox: 0.2214, loss_mask: 0.2239, loss: 0.7033
2022-09-30 08:50:26,165 - mmdet - INFO - Epoch [13][450/7330]   lr: 1.000e-04, eta: 1 day, 1:20:09, time: 0.508, data_time: 0.025, memory: 13458,
loss_rpn_cls: 0.0303, loss_rpn_bbox: 0.0395, loss_cls: 0.1963, acc: 93.0486, loss_bbox: 0.2309, loss_mask: 0.2276, loss: 0.7245
Traceback (most recent call last):                                                                                File "/opt/conda/lib/python3.8/runpy.py", line 194, in _run_module_as_main                                                                          return _run_code(code, main_globals, None,
File "/opt/conda/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/opt/conda/lib/python3.8/site-packages/torch/distributed/launch.py", line 260, in                                                     main()                                                                                File "/opt/conda/lib/python3.8/site-packages/torch/distributed/launch.py", line 255, in main                                                        raise subprocess.CalledProcessError(returncode=process.returncode,                                                                            subprocess.CalledProcessError: Command '['/opt/conda/bin/python', '-u', 'tools/train.py', '--local_rank=7', 'configs/swin/mask_rcnn_swin-b-p4-w7_fpn_fp16_ms-crop-3x_coco.py', '--seed', '0', '--launcher', 'pytorch', '--auto-resume']' died with <Signals.SIGSEGV: 11>.

Maybe you can try to upgrade your torch version to 1.8.1 or higher.

Yeah, when I update higher torch version, this problem solved.


Doc: Add Fleet submariner plugin doc

What type of PR is this?

/kind documentation
What this PR does / why we need it:
Add documentation for submariner plugin implemented in #660 (UNDONE)
Which issue(s) this PR fixes:
#657
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/cc @hzxuzhonghu


task: add support for buyer option

Description: Adds support for the buyer option when used in React Native. In the native code, instead of getting each property of the object and pass it to the structs/classes, we pass the whole ReadableMap directly which is decoded as the desired target Gr4vyBuyer. We use JSONSerialization in Swift and jackson (added previously) for Java.
We could potentially refactor other objects (like statementDescriptor) to use the same in the future to clean up the code. Or decide to go for json strings like we did for connectionOptions (which will require changes in the other sdks first).
Tested manually with both simulators and for different scenarios (no buyer, no `billingDetails, etc)
Ticket: https://gr4vy.atlassian.net/browse/TA-8177

:rocket: PR was released in v1.6.0 :rocket:


[Feature Request] Desktop notifications

I not even know if it's possible, but if it is:
Will be pretty useful to get notified in the desktop or at least inside the VSCode when you get a new message, new match, etc.

Or at the very least notify with an email notification.


Adding ghostHunter search

Since Tapir/Tapirius seems to have permanently broken, I'm submitting a pull request to have ghostHunter be included as the search engine of Vapor. I run it on my own blog classicyuppie.com with success and the key benefit above other search capabilities is that it offers real-time search as you type, culling the results to a list of posts/pages that actually useful.

I've updated my fork to include the modifications to index.js in the theme's assets/js directory so the code exists for real-time search culling.


Update toc.yml

Changed Tutorials list order - swapped 3 and 4 into correct sequence. Configure networking comes before access a private cloud - without the network access, you cannot access the private cloud.

@rvandenbedem - can you resubmit this (and all future PRs) via the private repos? thanks

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.


Throwing an instance of 'librealsense::linux_backend_exception'

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 22.04

Kernel Version (Linux Only)
5.19.0-50-generic

Platform
PC

SDK Version
2.54.1

Language
C++

Segment
Robot

Issue Description
I use D415 to acquire an image once every 2 seconds and process the image in my program.
When running that program, the program terminates with the following error about 1 time in 1000 times.
terminate called after throwing an instance of 'librealsense::linux_backend_exception'
  what():  lockf(...) failed Last Error: Bad file descriptor
Segmentation fault (core dumped)

I dumped the core file and analyzed it using gdb and the result is as follows.
Core was generated by `MY_PROGRAM_NAME'.
Program terminated with signal SIGABRT, Aborted.
#0  __pthread_kill_implementation (no_tid=0, signo=6, threadid=140378762098240) at ./nptl/pthread_kill.c:44
44	./nptl/pthread_kill.c: no such file or directory.

The result of back-trace with gdb is as follows.
(gdb) bt
#0  __pthread_kill_implementation (no_tid=0, signo=6, threadid=140378762098240)
    at ./nptl/pthread_kill.c:44
#1  __pthread_kill_internal (signo=6, threadid=140378762098240)
    at ./nptl/pthread_kill.c:78
#2  __GI___pthread_kill (threadid=140378762098240, signo=signo@entry=6)
    at ./nptl/pthread_kill.c:89
#3  0x00007fac82842476 in __GI_raise (sig=sig@entry=6)
    at ../sysdeps/posix/raise.c:26
#4  0x00007fac828287f3 in __GI_abort () at ./stdlib/abort.c:79
#5  0x00007fac82ca2b9e in  () at /lib/x86_64-linux-gnu/libstdc++.so.6
#6  0x00007fac82cae20c in  () at /lib/x86_64-linux-gnu/libstdc++.so.6
#7  0x00007fac82cad1e9 in  () at /lib/x86_64-linux-gnu/libstdc++.so.6
#8  0x00007fac82cad959 in __gxx_personality_v0 ()
    at /lib/x86_64-linux-gnu/libstdc++.so.6
#9  0x00007fac8de1a884 in  () at /lib/x86_64-linux-gnu/libgcc_s.so.1
#10 0x00007fac8de1b2dd in _Unwind_Resume ()
    at /lib/x86_64-linux-gnu/libgcc_s.so.1
#11 0x00007fac8d7c1db2 in __gthread_mutex_unlock (__mutex=<optimized out>)
    at /usr/include/x86_64-linux-gnu/c++/11/bits/gthr-default.h:779
#12 std::mutex::unlock() (this=<optimized out>, this=<optimized out>)
    at /usr/include/c++/11/bits/std_mutex.h:118
#13 std::lock_guard<std::mutex>::~lock_guard()
    (this=<optimized out>, this=<optimized out>)
--Type <RET> for more, q to quit, c to continue without paging--
    at /usr/include/c++/11/bits/std_mutex.h:235
#14 librealsense::platform::named_mutex::unlock() (this=0x556268a34d30)
    at ./src/linux/backend-v4l2.cpp:191
#15 0x00007fac8d8c09c0 in librealsense::platform::multi_pins_uvc_device::unlock() const (this=<optimized out>) at ./src/backend.h:777
#16 0x00007fac8dc49e22 in std::lock_guard<librealsense::platform::uvc_device>::~lock_guard() (this=<synthetic pointer>, this=<optimized out>)
    at /usr/include/c++/11/bits/std_mutex.h:235
#17 librealsense::locked_transfer::send_receive(std::vector<unsigned char, std::allocator<unsigned char> > const&, int, bool)::{lambda(librealsense::platform::uvc_device&)#2}::operator()(librealsense::platform::uvc_device&) const
    (dev=..., __closure=<synthetic pointer>) at ./src/hw-monitor.h:219
#18 librealsense::uvc_sensor::invoke_powered<librealsense::locked_transfer::send_receive(std::vector<unsigned char, std::allocator<unsigned char> > const&, int, bool)::{lambda(librealsense::platform::uvc_device&)#2}>(librealsense::locked_transfer::send_receive(std::vector<unsigned char, std::allocator<unsigned char> > const&, int, bool)::{lambda(librealsense::platform::uvc_device&)#2})
    (action=..., this=0x556268b0a890) at ./src/sensor.h:361
#19 librealsense::locked_transfer::send_receive(std::vector<unsigned char, std::allocator<unsigned char> > const&, int, bool) [clone .constprop.0]
    (this=0x556268a34570, data=std::vector of length 24, capacity 24 = {...}, require_response=require_response@entry=true, timeout_ms=5000)
    at ./src/hw-monitor.h:219
--Type <RET> for more, q to quit, c to continue without paging--
#20 0x00007fac8db6763a in librealsense::hw_monitor::execute_usb_command(unsigned char*, unsigned long, unsigned int&, unsigned char*, unsigned long&, bool) const
    (this=0x556268a683a0, out=<optimized out>, outSize=<optimized out>, op=@0x7fac7a3f8de4: 0, in=0x7fac7a3f8df0 "\001", inSize=@0x7fac7a3f8de8: 1024, require_response=true) at /usr/include/c++/11/bits/shared_ptr_base.h:1295
#21 0x00007fac8db67840 in librealsense::hw_monitor::send_hw_monitor_command(librealsense::hw_monitor::hwmon_cmd_details&) const
    (this=this@entry=0x556268a683a0, details=...)
    at /usr/include/c++/11/array:257
#22 0x00007fac8db67c04 in librealsense::hw_monitor::send(librealsense::command, librealsense::hwmon_response*, bool) const
    (this=0x556268a683a0, cmd=..., p_response=0x0, locked_transfer=<optimized out>) at ./src/hw-monitor.cpp:177
#23 0x00007fac8d8e9a20 in librealsense::d400_device::get_device_time_ms()
    (this=<optimized out>) at ./src/ds/d400/d400-device.cpp:1179
#24 0x00007fac8db60c69 in librealsense::time_diff_keeper::update_diff_time()
    (this=0x556268a46ed0) at ./src/global_timestamp_reader.cpp:209
#25 0x00007fac8db60dda in librealsense::time_diff_keeper::polling(dispatcher::cancellable_timer) (this=0x556268a46ed0, cancellable_timer=...)
    at ./src/global_timestamp_reader.cpp:250
#26 0x00007fac8d9734a3 in std::function<void (dispatcher::cancellable_timer)>::operator()(dispatcher::cancellable_timer) const
--Type <RET> for more, q to quit, c to continue without paging--
    (__args#0=..., this=<optimized out>)
    at /usr/include/c++/11/bits/std_function.h:590
#27 active_object<std::function<void (dispatcher::cancellable_timer)> >::do_loop()::{lambda(dispatcher::cancellable_timer)#1}::operator()(dispatcher::cancellable_timer) const (ct=..., __closure=0x7fac7a3fa770)
    at ./third-party/rsutils/include/rsutils/concurrency/concurrency.h:444
#28 std::__invoke_impl<void, active_object<std::function<void (dispatcher::cancellable_timer)> >::do_loop()::{lambda(dispatcher::cancellable_timer)#1}&, dispatcher::cancellable_timer>(std::__invoke_other, active_object<std::function<void (dispatcher::cancellable_timer)> >::do_loop()::{lambda(dispatcher::cancellable_timer)#1}&, dispatcher::cancellable_timer&&) (__f=...)
    at /usr/include/c++/11/bits/invoke.h:61
#29 std::__invoke_r<void, active_object<std::function<void (dispatcher::cancellable_timer)> >::do_loop()::{lambda(dispatcher::cancellable_timer)#1}&, dispatcher::cancellable_timer>(active_object<std::function<void (dispatcher::cancellable_timer)> >::do_loop()::{lambda(dispatcher::cancellable_timer)#1}&, dispatcher::cancellable_timer&&) (__fn=...) at /usr/include/c++/11/bits/invoke.h:111
#30 std::_Function_handler<void (dispatcher::cancellable_timer), active_object<std::function<void (dispatcher::cancellable_timer)> >::do_loop()::{lambda(dispatcher::cancellable_timer)#1}>::_M_invoke(std::_Any_data const&, dispatcher::cancellable_timer&&) (__functor=..., __args#0=<optimized out>)
    at /usr/include/c++/11/bits/std_function.h:290
#31 0x00007fac8dbf16a5 in dispatcher::dispatcher(unsigned int, std::function<voi--Type <RET> for more, q to quit, c to continue without paging--
d (std::function<void (dispatcher::cancellable_timer const&)>)>)::{lambda()#1}::operator()() const () at /usr/include/c++/11/bits/std_function.h:590
#32 0x00007fac82cdc253 in  () at /lib/x86_64-linux-gnu/libstdc++.so.6
#33 0x00007fac82894b43 in start_thread (arg=<optimized out>)
    at ./nptl/pthread_create.c:442
#34 0x00007fac82926a00 in clone3 ()
    at ../sysdeps/unix/sysv/linux/x86_64/clone3.S:81

The program that causes this phenomenon is too complex and large to post here.
Does anyone have any idea what the cause might be?

Hi.
Can you share kernel log when that event happens?
dmesg > kernel.log
Do you disconnect camera in idle periods to conserve power?

Uploads the kernel.log when events occur.
kernel.log
Camera is not disconnected even when idle.
Additional info.
I am using two D415s, I have two processes up and running that take an image every 2 seconds and process the image.

Hi @shintaro-matsui  I would also add that kernel 5.19 is not yet supported by librealsense at the time of writing this.  5.15 is the current most recently supported kernel.  The SDK can be used with non-supported kernels but there may be unpredictable consequences in regards to stability.
If librealsense is built from source code with CMake then including the flag -DFORCE_RSUSB_BACKEND=TRUE in the CMake build instruction will cause the SDK to bypass the kernel and so avoid that instability, as an RSUSB build is not dependent on Linux versions or kernel versions and does not need to have a kernel patch script applied.
If you are using C++ then a script at https://github.com/IntelRealSense/librealsense/issues/2219#issuecomment-412899468 for capturing an image simultaneously from all attached D415 cameras may be a helpful reference if you have not seen it already.
@dmipx  I believe that your PR for 5.19 support at https://github.com/IntelRealSense/librealsense/pull/11837 was merged into the in-progress development version of librealsense on May 25 2023 but may have just missed the release of librealsense 2.54.1 and so will be incorporated into the next SDK release after 2.54.1?

Thanks for all the info.
I use librealsense installed with apt install librealsense2-dev currently.
I will try to install from source code in the development branch, and also try the option -DFORCE_RSUSB_BACKEND=true.
If that does not work, I will change to a machine with a different kernel and try again.
It will take some time to report the results.

It's no problem at all.  Please do update here when you have news to report.  Good luck!

First I removed the existing librealsense2.54.
sudo apt remove --purge librealsense2*
Next, I installed the development branch from the source code.
-DFORCE_RSUSB_BACKEND=TRUE flag was not used.
Then I deleted the build directory of my program and tried to build again, but I get the following error.
/usr/bin/ld: warning: librealsense2.so.2.54, needed by /usr/local/lib/libpcl_io.so, not found (try using -rpath or -rpath-link)

Is this a bug on the librealsense2.55?
Or did I not delete my librealsense2.54 enough?

The instruction below should ensure that all RealSense-related packages are removed.
dpkg -l | grep "realsense" | cut -d " " -f 3 | xargs sudo dpkg --purge

Hi.
While trying different versions of librealsense, for some reason CUDA stopped functioning properly and my PC stopped working after CUDA failed to reinstall.
I decided to reinstall Ubuntu, but I heard that the kernel version was important, so I decided to install Ubuntu 20.04.
My current environment is as follows.

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 20.04.06 LTS

Kernel Version (Linux Only)
5.15.0-82-generic

Platform
PC

SDK Version
2.54.1

Language
C++

Segment
Robot

However, I still get the same error.
Errors occur with the same frequency, about once every 1000 times.
terminate called after throwing an instance of 'librealsense::linux_backend_exception'
  what():  lockf(...) failed Last Error: Bad file descriptor
Segmentation fault (core dumped)

librealsense2 was installed by sudo apt install librealsense2-dev librealsense2-utils librealsense2-dbg.
Any advice?

Your apt install instruction does not include the most important package librealsense2-dkms which is the core of librealsense.  librealsense2-utils is the second most important as it installs the SDK's tools and examples.
librealsense2-dev and librealsense2-dbg are optional developer and debug packages that do not need to be installed in order for librealsense to work.
Please repeat the package uninstall instruction and then try this installation instruction:
sudo apt-get install librealsense2-dkms librealsense2-utils

CUDA requires an Nvidia graphics GPU and the librealsense SDK's support of it is intended for use with Nvidia Jetson single-board computers rather than a PC.  For this reason, the packages on the distribution_linux.md installation instruction page do not contain CUDA support.  Jetson boards have their own separate packages that contain CUDA support.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_jetson.md

I understand that librealsense2-dkms is important.
I will report back when there is more progress.

Hi. @dmipx, @MartyG-RealSense.
This error no longer occurs.
First, I installed librealsense from the source code with the v2.54.1 branch and tried -DFORCE_RSUSB_BACKEND=true in the following environment that I reported previously, but it did not solve the problem.
https://github.com/IntelRealSense/librealsense/issues/12140#issuecomment-1702116555
So I reinstalled with -DFORCE_RSUSB_BACKEND=false.
Next, I changed the implementation of the TCP/IP communication, and inexplicably this solved the problem.
In my program, the process that uses RealSense is the client and communicates with other processes.
Previously, an instance of client communication was created each time.
Here is a simple code to explain.
while(){

    // Generate client instance
    // Clinet connects to server

    // Fetch the realsense frame
    // Point cloud processing

    // Communicate to server, send and receive

    // Close the client
}

I changed it to keep an instance of the client communication and it no longer occurs.
// Generate client instance
// Clinet connects to server
while(){

    // Fetch the realsense frame
    // Point cloud processing

    // Communicate to server, send and receive

}
// Close the client

I report it, I don't know if this will help others.
Thank you for your help in discussing this.

You are very welcome.  It's great to hear that you achieved a solution.  Thanks very much for the update and for sharing the details of your solution!
As you have solved your issue, I will close this case.  Thanks again!


Doc update for indexed dynamic size parameters

add example and description to documentation about matching indexed dynamic size parameters

@marekkirejczyk please tell me if this is enough documentation like this

@marekkirejczyk please tell me if this is enough documentation like this

thanks @marekkirejczyk  ! looking forward to the next version (3.2.2 ?)

thanks @marekkirejczyk  ! looking forward to the next version (3.2.2 ?)


Added typescript definition file

Began using this in my typescript project, so decided to help out and write up an index.d.ts. I also added a "types" field to package.json so that compilers know where to look.

Awesome, thanks so much! Sorry that I only got around to see this right now.


nz-anchor invalid selector '?'

Reproduction link
https://stackblitz.com/edit/angular-sqgiqz?file=src%2Fapp%2Fapp.component.ts
Steps to reproduce
Use nz-anchor component and add '?' in nzHref
What is expected?
Scroll to nzHref even if url contains '?'
What is actually happening?
ERROR
Error: Failed to execute 'querySelector' on 'Document': '#components-anchor-demo-static?' is not a valid selector.

Issue helper is bugged for me


TinaCMS is not tree-shakable

Description
TinaCMS is bundled as CommonJS bundle only, meaning Webpack and Rollup can't statically analyze them for tree shaking, as tree shaking is only supported for ES Modules
Steps to reproduce

Install tinacms
Import tinacms
See that all dependant code of tinacms is included in the built app no matter what concrete implementation is imported
Try webpack-bundle-analyzer

Expected result
TinaCMS is deployed with both a commonJS module assigned to main in package.json, and an ES module assigned to module in package.json.
Also, the tinacms and other packages specify sideEffects in package.json for webpack tree shaking support .
FYI: this logic existed in the studio prototype.
Actual result
No tree shaking.
Environment
Tinacms Package Versions
Only<EMAIL_ADDRESS>
Tools

Tool
Version

browser
Chrome

os
Windows 10

node
14.16.2

npm
6.x.x

Hey Chris 😄
Are you saying, it's time to say hello to ESM modules?

@DirtyF hey-o!
This is currently a pretty big deal, as importing anything from tinacms (with or without the react UI stuff) adds 100kb gzipped (~300kb unminified) to your application JS.

@chrisdmacrae Did you try loading TinaCMS dynamically or through a custom route to minimize the impact on production?

@DirtyF webpack and rollup rely on the ES Module syntax for tree shaking:
// index.js
export * from './example';

// example.js
export const Example = () =>
I am an example

export const OtherExample = () => <1>I am another example

It actually parses the modules as text files and finds the export and import statements, and builds a map of what the code actually depends on.
So if you had code like:
import { Example } from 'example-package';

It's able to go "Oh, you specifically only imported Example from that library -- I'll only take its code along with me.
In fact, if you do glob imports like: import * as Thing from 'example-package';, you break tree shaking.
CommonJS creates modules that look like:
const exports = module.exports;

exports.Example = () =>
I am an example

exports.OtherExample = () =>
I am another example

This means that from a static analysis perspective, exports is the only real export from this module -- all of Example and OtherExample are properties of that main export. So it can't tree shake them, because it would actually break the code.
Since TinaCMS is built as a series of commonJS bundles, none of them can be tree shaken. The tinacms package imports a handful of things from @tinacms/forms, @tinacms/react-sidebar, etc. Mainly for types and such.
But because it does this, the entire package gets bundled when using anything from the tinacms bundle, even if you didn't import any of the TinaCMS react UI.

Closing now that we have TinaEditProvider to dynamically import Tina


Switch to Laravel HTTP client for compatibility with Pulse, Telescope, Sentry etc

The package currently uses Guzzle directly, which means requests won't appear in core tools like Pulse and Telescope. It would be great if Laravel's HTTP client could be used instead to ensure compatibility with the wider Laravel ecosystem.
It also doesn't work with things like Sentry's Laravel HTTP client integration: https://github.com/getsentry/sentry-laravel/pull/797
As a side note, it would also provide a solution to this issue since Laravel's HTTP client has built in retry functionality.

Hi @binaryfire
I am not sure if this i an easy change because of the underlying client being http client agnostic. Is the Laravel HTTP Client PSR-17 compatible?
If so, could you make a PR?

Hi @gehrisandro. Unfortunately no, I don't think Laravel's HTTP client is PSR-17 compatible. It's a bespoke wrapper around Guzzle.

You can probably utilize the Guzzle Watcher, a Telescope plugin, to view Guzzle request and response logs in the Telescope UI.
Laravel's "HTTP Client" is more of a PendingRequest builder rather than a client, and it is not designed to be PSR-17 compatible.

@andrzejkupczyk Thanks for the suggestion but it affects more than just Telescope. Several other core and third party packages integrate with the HTTP client eg. Pulse, Sentry etc.

Hi @binaryfire
If tinkered around a bit and came up with a solution, creating a PSR-18 compatible wrapper around the HTTP facade.
At least for now, the wrapper is in a separate project: https://github.com/gehrisandro/laravel-http-psr18
When using the package, you have to register the OpenAI client in your ServiceProvider:
$this->app->extend(ClientContract::class, static function (): Client {
    return OpenAI::factory()
        ->withApiKey(config('openai.api_key'))
        ->withOrganization(config('openai.organization'))
        ->withHttpClient($httpClient = HttpPsr18::make(Http::withHeader('OpenAI-Beta', 'assistants=v1')))
        ->withStreamHandler(fn(RequestInterface $request) => $httpClient->sendRequest($request))
        ->make();
});

Hope this helps and would like to hear your feedback.

Thanks @gehrisandro! I'll let you know how things go as soon as I have some time to test.

Thanks for the solution. Unfortunately for me, this method works only for the first request. When attempting to rerun a request, it will merge the headers and result in duplicate headers values, see example below :
{
	"content-length": "259",
	"user-agent": "GuzzleHttp/7",
	"openai-beta": "assistants=v1, assistants=v1, assistants=v1",
	"accept": "application/json",
	"content-type": "application/json, application/json, application/json",
	"host": "api.openai.com, api.openai.com",
	"authorization": "********",
	"openai-organization": "*********"
}

This will result in an 400 Bad request from Open AI API.
To fix that, it's better to use a new instance of Http client for each request. Here is a solution that works well. In addition, it doesn't need you to install the HttpPsr18 wrapper.
use Illuminate\Support\Facades\Http;
use OpenAI;
use OpenAI\Client;
use OpenAI\Contracts\ClientContract;
use Psr\Http\Client\ClientInterface;
use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;

// AppServiceProvider::register
$this->app->extend(ClientContract::class, static function (): Client {
            $client = new class implements ClientInterface {
                public function sendRequest(RequestInterface $request): ResponseInterface
                {
                    return Http::withHeaders($request->getHeaders())
                        ->asJson()
                        ->acceptJson()
                        ->send($request->getMethod(), (string)$request->getUri(), [
                            'body' => $request->getBody()->getContents(),
                        ])
                        ->toPsrResponse();
                }
            };

            return OpenAI::factory()
                ->withApiKey(config('openai.api_key'))
                ->withOrganization(config('openai.organization'))
                ->withHttpHeader('OpenAI-Beta', 'assistants=v1')
                ->withHttpClient($client)
                ->make();
        });


Using ngModel in MDTextfield

Is there a way to use "ngModel"? I tried it, but it doesn't work.
<MDTextField [(ngModel)]='model.mytext'></MDTextField>

It was working before. Will bring that back.

Actually can you try importing MaterialTextfieldModule from nativescript-material-textfield/angular/material-textfield.module? This should work.
Be careful we will change the path in the future releases

I can't import MaterialTextfieldModule. I will use textChange event for the moment.

@williamreetz that's the issue. need to make a new release

Hi there,
I'm using v2.2.17 and can not get the MDTextfield to work with [(ngModel)]. Furthermore I would love to use it with reactive forms.
It's reproducable with the default HelloWorld template. I have imported the
import { MaterialTextfieldModule } from 'nativescript-material-textfield/angular'; at the HomeModule and defined the following imports:
imports: [
NativeScriptCommonModule,
NativeScriptFormsModule,
MaterialTextfieldModule,
HomeRoutingModule
],
In the home.component.html I have defined:
<MDTextField
[(ngModel)]="title"
hint="i am an hint"
helper="help me!"
returnKeyType="next">
As soon as the app starts the following error is thrown:
ERROR Error: No value accessor for form control with unspecified name attribute
Is there any known issue with this or what can I do to make it working again?

@t0ms3n not sure about 2.2.17. We are at 2.3.16(which uses {N} ). Can you try?

In 2.5.4 the error persist


Fix options table links

We fixed issues with ID's not being unique whiched changed how the first example's ID is generated.
Now it contains the component name instead of a generic 'default' id.
This means when linking between components the macro options behaviour is currently broken.
You can see this by going to the textarea component macro options and press 'errorMessage', it will load the page but not show the macro options for the Error Message component.
Fixes https://github.com/alphagov/govuk-design-system/issues/1008
I'll need to update all the component READMEs in GOV.UK Frontend also.

You can preview this change here:
Built with commit ffd7babf74d05f068c927b77f86fa7e2d1c0208f
https://deploy-preview-1057--govuk-design-system-preview.netlify.com

@36degrees I've added some more details, does it make sense?


get_images bug (and possibly unrelated segfault)

Moving an email discussion to github. Original report from @sbillinge:

We were running a Count() subscribing to LiveImages and it crashed with a segmentation fault.  It looked like the crash was caused by Qwidget
QWidget::repaint:Recursive repaint detected) and now get_noImages of bluesky is not working.

the get_images error is
TypeError: slice indices must be integers or None or have an index method

I think the issue with get_images is a known problem: the "squashed TIFF" style of files exposed a simple bug in our handler function (the function which the databroker uses to read TIFFs). Therefore, with @timoHMM, we wrote a revised version of this handler and stashed it in /home/xf28id1/.ipython/profile_collection/startup/91-handlers.py. That file also contains code that de-registers the broken handler and registers our new one. Therefore, the fix should be invisible to the user -- it should "just work" on startup.
In the near future, that should be fixed upstream in filestore.
Of course, the file 91-handlers.py only affects the collection environment on the xf28id1-ws2 workstation. If you want to use that code in a notebook on jupyterhub, you'll have to copy/paste the contents of the file.
I am not sure about the segfault. Let us know if it is repeatable.

@timoHMM Is this resolved, then? If so, would you compare the code in 91-handlers.py to the implementation in filestore/handlers.py and make a pull request? Let me know if you need more information.

@danielballan  Handler should be the problem and I should have awared of it earlier. Local image retrival at xpd is fine.
The sefault probably came from LiveImage subscription; it happened when we do a scan with massive number of frames and small exposure time. (it happened in the middle of a scan with a few hundreds frames, 0.2 sec exposure time to each frame)

@chiahaoliu Great, thanks. For the record, we were moving quickly and not taking as much time as we'd like to explain, so I completely understand why would have missed the finer points. Do you have time to make a PR this week? If not I will take care of it.

@danielballan I can make a PR for sure. Sorry for late reply, just recovered from a crazy beamtime.

@chiahaoliu spoken like a true experimentalist!!!!!
On Thu, Nov 12, 2015 at 1:39 PM, Timothy Liu<EMAIL_ADDRESS>wrote:

@danielballan https://github.com/danielballan I can make a PR for sure.
Sorry for late reply, just recovered from a crazy beamtime.
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/Bug-Reports/issues/54#issuecomment-156196443.

--
Prof. Simon Billinge
Applied Physics & Applied Mathematics
Columbia University
500 West 120th Street
Room 200 Mudd, MC 4701
New York, NY 10027
Tel:<PHONE_NUMBER> (o) 851-7428 (lab)
Condensed Matter Physics and Materials Science Dept.
Brookhaven National Laboratory
P.O. Box 5000
Upton, NY 11973-5000
(631)-344-5661
email: sb2896 at columbia dot edu
home: http:// http://nirt.pa.msu.edu/bgsite.apam.columbia.edu/


python: get exe/args from shebang

Ref: https://github.com/neomake/neomake/issues/449#issuecomment-414305622

What's the status of this PR?

@euclio
I do not really like it to be used by default I guess (because it adds some overhead).
But we could have this as a helper and document how to configure it to be used.

@euclio
Made some improvements.
Would be good if you could try it and provide feedback.
It can be used via call neomake#config#set('b:python.InitForJob', function('neomake#utils#set_argv_from_shebang')) to enable it for the current buffer's python maker.

Thanks for this work, but I'm now using InitForJob to use pipenv for this functionality.


Dcwither/upgrade react 16

Coverage increased (+1.4%) to 96.296% when pulling a75ba3f1ec3af2028fd70228ae8f5560d7563ed7 on dcwither/upgrade-react-16 into 6bec810295e58b473977f103453bd29c1f246704 on master.


Profiling hide original service definition

Once decorated, the following plugin no longer have setAuthorization method available in the App\Http\Plugin\AuthorizationPlugin service because of being decorated with ProfilePlugin. To get access to the original plugin, you have to reference it using the App\Http\Plugin\AuthorizationPlugin.debug.inner name.
namespace App\Http\Plugin;

use Http\Client\Common\Plugin;
use Psr\Http\Message\RequestInterface;

class AuthorizationPlugin implements Plugin
{
    /**
     * @var string
     */
    private $authorization;

    /**
     * @param string $authorization
     */
    public function setAuthorization($authorization)
    {
        $this->authorization = $authorization;
    }

    /**
     * {@inheritdoc}
     */
    public function handleRequest(RequestInterface $request, callable $next, callable $first)
    {
        if (!$this->authorization) {
            return $next($request);
        }

        return $next($request->withHeader('Authorization', $this->authorization));
    }
}

App\Http\Plugin\AuthorizationPlugin:
    class: App\Http\Plugin\AuthorizationPlugin

The ongoing work with PluginClientFactory is supposed to fix this. The only thing to consider is about possible BC break.

This is a BC break indeed. Could it be solved by magic functions?

I think it should be solved by not using dependency injection decoration.


[vtk-m] Add new port

Addresses #9213

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: acdemiralp sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Port seems to only support x64. No ifdefs to differentiate between different architectures in Math.h

/azp run

@ras0219-msft This version of vtk-m can not be used with ParaView/VTK and does not install the configs correctly. The config is called VTKmConfig.cmake but gets installed into share/vtk-m so cmake cannot find it if used (also some paths must be fixed in the config due to vcpkg_fixup_cmake_targets).
I tried using in my ParaView PR #9202 but it seems like the API is changed between even minor versions. So ParaView will fix the version of VTK and VTK will fix the version of vtk-m except I let ParaView use an internal build of VTK/vtk-m within ParaView (which would probably make the ParaView PR a lot easier but felt against the spirit of vcpkg)

/azp run

/azp run

@acdemiralp
Could you please resolve the conflicts?

Hi @acdemiralp, thanks for this PR!
But vtk-m has been added in #11148.

All clear. Closing.


Remove interface test

This interface is named differently on some OSes. For example is lo0 on mac OS.

@joestein I don't see the value in a test binding to the loopback here. You would never want to do that when deploying the scheduler b/c the slave wouldn't be able to connect to it.

if:eth0 or if:eth1 is what the deploy would be when wanting to bind to nic. My point was preserve functionality to bind that way not necessarily to loopback.


hotfix for multiple initialization error

I reviewed the PR (#66) which introduced this line, and must have missed it.
FYI @efischer19

Sorry about that @pwnage101 - thanks for catching


lambda example for compression

We haven't tested the new compression extension on Lambda yet. A user reported that there were some issues with getting the package-lock.json file to be correct for the target operating system, and then when they deployed to lambda they got this error:
{
    "timestamp": "2024-04-24T15:25:04.685Z",
    "level": "ERROR",
    "message": {
        "errorType": "ImportModuleError",
        "errorMessage": "Error: Cannot find module '@mongodb-js/zstd'\nRequire stack:\n- /var/task/asset.js\n- /var/runtime/index.mjs",
        "stackTrace": [
            "Runtime.ImportModuleError: Error: Cannot find module '@mongodb-js/zstd'",
            "Require stack:",
            "- /var/task/asset.js",
            "- /var/runtime/index.mjs",
            "    at _loadUserApp (file:///var/runtime/index.mjs:1087:17)",
            "    at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)",
            "    at async start (file:///var/runtime/index.mjs:1282:23)",
            "    at async file:///var/runtime/index.mjs:1288:1"
        ]
    }
}

They said that they had to add some excludes in order to get it to deploy:
- '@mongodb-js/zstd'
- '@mongodb-js/zstd-darwin-arm64'
- '@mongodb-js/zstd-linux-x64-gnu'
- '@mongodb-js/zstd-linux-x64-musl'

But I suspect that excluding the main zstd package would cause the error above.
We need to either:

come up with some automated solution during install that will sort out the correct values for a given architecture, or
add some information to the docs/examples showing how to configure it successfully for a target architecture

If we find that the packaging for this particular zstd library is problematic, we can change to a different one; it should be possible to do that without breaking any APIs.

@bruuuuuuuce let's keep this open until we have augmented Nate's docs to include a link to it or whatever?


Infer: ensure that passing PKs are in headers

Input data should be checked to have corresponding to primary_key value column:
def infer(headers, values, row_limit=None, explicit=False, primary_key=None):

MERGED
#105


Feat: Improve the GitHub guide

The GitHub guide seems to lack a lot of information. Can you please add more guide on that and I always like to help but I don't know how to setup the project so if you please add a contribution guide.

Hey thanks, yeah I'll add the contirbution guide asap. And you can help with github guide if you have the time.