Back button causes field to duplicate

First, let me say, I really appreciate this handy widget and thank you for the work.
I have ran into several small issues while using bootstrap-select, and have solved them in some way or another, so this post is more of a reflection than an issue report:

When pressing back to go back to a page that has a bootstrap-select field, bootstrap-select gets initialized again causing duplicated fields.

My solution:
I added some js code to check whether there is any 'bootstrap-select' class directly following another 'bootstrap-select' class, and if so, remove the duplicated field.
$('.bootstrap-select').each(function(bs) {
if ($(this).next().hasClass('bootstrap-select')) {
$(this).next().remove();
}
})

I used bootstrap-select primarily on a country and city field for my project. The city field is a template that gets re-rendered when the country value changes. It so happens that when the city field reloads, 'bootstrap-select' defaults to hiding itself (i.e. it gets reloaded as well, but it is not displayed)

My solution:
After testing 'bootstrap-select' event hooks, adding the following in the template seems to do the trick:
$('.selectpicker').selectpicker('refresh');

And, finally, as mentioned by others already, 'bootstrap-select' does not seem to be compatible with Bootstrap 4 at the moment. I've tried the patch mentioned by another user, but I couldn't get it to work.

Thanks for the feedback.


Add degree type content for degrees headers

Context
e need to enable candidates without a degree to apply for TDA courses. The existing intro content on the degree list page was originally written for postgraduate courses. See live site screenshot.
Changes proposed in this pull request
TDA feature flag off

TDA feature flag on

Guidance to review

When teacher degree apprenticeship feature flag is active, content wise all good?
When teacher degree apprenticeship feature flag is NOT active, content wise all good?

 TDA Your details: Update degree list page intro content


Set responder state from parent

If I have a custom class built on top of SUITextField:
struct OTextField: View {
    @Binding var text: String
    // ...

    var body: some View {
        SUITextField(text: $text)
        // ... bunch of modifiers
    }
}

I'd like to be able to set the responder modifier on the parent class:
    var body: some View {
            OTextField(text: $text)
                .responder($isFocused, equals: true)
    }

But it's not possible because responder is only a modifier for the SUITextField class. But looking at the implementation, I don't see any reason why it would not be possible to call it on a simple View?
You could simply add an extension:
extension View {

    func responder<Value>(_ binding: ResponderState<Value>.Binding, equals value: Value) -> some View where Value: Hashable {
        _ = binding.storage.values.insert(value)
        return self
            .environment(\.responderValue, value)
            .environment(\.responderStorage, binding.storage)
    }

}

I tried adding it to a local version and it seems to work well. Let me know if you want me to submit a PR.

This is interesting, I haven't thought this initially. I'll check your PR


Callable Return Type  for methods

Does this plugin has any kind of support for the generic model substitute such as Callable, ResponseEntity, and others. Springfox plugin provides java based configuration option .genericModelSubstitutes(Callable.class), but here I am looking for some configuration options for pom.xml.

Maybe https://github.com/kongchen/swagger-maven-plugin/issues/579#issuecomment-446534548 could help?


discuss with alex '*' doesn't seem to be a valid engine version

Issue created from code comment with imdone.io

TODO @joao: discuss with alex '*' doesn't seem to be a valid engine version id:71
src/vs/platform/extensionManagement/node/extensionGalleryService.ts:631
@imdone - Efficiently manage your project's technical debt. imdone.io

Issue closed by removing a comment.

TODO @joao: discuss with alex '*' doesn't seem to be a valid engine version id:71 gh:78
src/vs/platform/extensionManagement/node/extensionGalleryService.ts:631
@imdone - Efficiently manage your project's technical debt. imdone.io


health: Configure sysctl when IPv6 is disabled

When IPv6 is disabled, ensure that the health endpoint's device is
configured to disable IPv6 so that it doesn't emit any IPv6 autoconf
frames or similar.

This change is

test-me-please

Coverage decreased (-0.009%) to 44.788% when pulling c69bbb95ca93c28372612accdbf126b0e024b317 on joestringer:submit/disable-ipv6-in-health-ep into d0f2435b4c423dbff571ad8bcf206c177a18eef9 on cilium:master.


FullCalendar 5.4.0

Do you mind updating this plugin for FullCalendar 5.4.0 please?

Haha.  In 25 years doing open source this is the first time I have seen an author troll his own project!
A more appropriate answer would have been "I currently do not have time to dedicate to this project, but if someone provides a PR I will gladly review and merge it".   But I see you chose a different route...


Error: ID token verification failed:

Exception raised while login Inrupt Pod Spaces with this tool.
Reproduce:

➜  workspace npx @inrupt/generate-oidc-token

? What Solid Identity Provider do you want to log in to? https://broker.pod.inrupt.com
? Has your app been pre-registered by the administrator of the Pod server you are signing in to? No.
? What is the name of the application you are registering? fghj
? @inrupt/generate-oidc-token will start a local web server, to which the Solid Identity Provider can redirect the user back after
 they log in. On what port do you want to run this server? 3001

Log

Listening at: [http://localhost:3001].
Logging in to Solid Identity Provider  https://broker.pod.inrupt.com to get a refresh token.

Please visit https://broker.pod.inrupt.com/authorization?client_id=pwv1sAW3tHpf6UNxkGFzDutuL6fnmpYm&scope=openid%20offline_access&response_type=code&redirect_uri=http%3A%2F%2Flocalhost%3A3001%2F&code_challenge=N9PMPYNssgIhGAHTtrjnEkrSWvZKLK3G6oTfjXJPjdE&state=4UztNNMRkzyGs4MMAjc2JbWL3O6Lcqv7dQ5Gnikqx3s&code_challenge_method=S256 in a web browser.

Login into the Identity Provider successful, receiving request to redirect IRI [http://localhost:3001/?code=Ing1X8htU4vWrdnyyEducffTJbOFHGEF&state=4UztNNMRkzyGs4MMAjc2JbWL3O6Lcqv7dQ5Gnikqx3s].
(node:1822) DraftWarning: The DPoP APIs implements an IETF draft. Breaking draft implementations are included as minor versions of the openid-client library, therefore, the ~ semver operator should be used and close attention be payed to library changelog as well as the drafts themselves.
(node:1822) UnhandledPromiseRejectionWarning: Error: ID token verification failed: JWSSignatureVerificationFailed: signature verification failed
    at flattenedVerify (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/jose-legacy-modules/dist/index.js:2521:15)
    at processTicksAndRejections (internal/process/task_queues.js:85:5)
    at async compactVerify (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/jose-legacy-modules/dist/index.js:2554:22)
    at async Object.jwtVerify (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/jose-legacy-modules/dist/index.js:2978:22)
    at async Object.getWebidFromTokenPayload (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/solid-client-authn-core/dist/login/oidc/redirectHandler/IRedirectHandler.js:25:46)
    at async AuthCodeRedirectHandler.handle (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/solid-client-authn-node/dist/login/oidc/redirectHandler/AuthCodeRedirectHandler.js:73:23)
    at async ClientAuthentication.handleIncomingRedirect (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/solid-client-authn-node/dist/ClientAuthentication.js:58:34)
    at async Session.handleIncomingRedirect (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/solid-client-authn-node/dist/Session.js:45:35)
    at async /Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/generate-oidc-token/dist/index.js:85:9
    at Object.getWebidFromTokenPayload (/Users/petertc/.npm/_npx/7dfa8f34abd2be6f/node_modules/@inrupt/solid-client-authn-core/dist/login/oidc/redirectHandler/IRedirectHandler.js:32:15)
    at processTicksAndRejections (internal/process/task_queues.js:85:5)
(node:1822) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:1822) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hi @hrchu, thanks for reaching out. I could not reproduce the bug locally, could you try running npx @inrupt/generate-oidc-token@latest to ensure that we are running the same version ? I'll also ask the server team if some key rotation happened today which might have temporarily affected you.

@NSeydoux thanks for your information. I ran npx @inrupt/generate-oidc-token@latest and got the same error message. Is there anything I can do to help diagnose this issue??

I upgrade my node from v12 to v14 and the problem is solved. Thanks for your help!


Allow OmniAuthAuthenticityTokenProtection options to be configured

This will be useful for disabling csrf protection in test suites or configuring the csrf key when the next version of rack-protection is released.
ie:
OmniAuth.config.request_validation_phase = OmniAuth::AuthenticityTokenProtection.new(allow_if: ->(env) { true })

or
OmniAuth.config.request_validation_phase = OmniAuth::AuthenticityTokenProtection.new(key: :_csrf_token)

Thanks @jkowens, I had been turning it off in test suites by setting OmniAuth.config.request_validation_phase = nil or OmniAuth.config.request_validation_phase = proc {}, but this makes sense to have as well. Going to release it in a patch version release since it honestly should have been present in the 2.0.0 release if I hadn't overlooked it.

Thanks @jkowens, I had been turning it off in test suites by setting OmniAuth.config.request_validation_phase = nil or OmniAuth.config.request_validation_phase = proc {}, but this makes sense to have as well. Going to release it in a patch version release since it honestly should have been present in the 2.0.0 release if I hadn't overlooked it.

Cool, I didn't think of those options! Thanks for the quick release 💯

Cool, I didn't think of those options! Thanks for the quick release 💯


[#5] Correct application names for non-webkit browsers

Fixes #5
Only the webkit browser has an application name of "Playwright". Other browsers do not so they need to be mapped properly.

Believe currently blocked by https://github.com/guidepup/setup/issues/10

Suspect would benefit from https://github.com/guidepup/guidepup/pull/41 r.e. Chromium not being reliable with missing text.


More GUI stuff

Highlights:
Fixed incorrect Gui.onMousePressed(game, mouseY, mouseButton) -> Gui.onScaleChange(game, windowWidth, windowHeight)
Added mapping for WidgetButton.onPressed / onReleased
Mapped most of GuiBackupLevel / GuiBackupPrompt

Updated as per review


Non-nullable warning no longer present as generated code already makes Movie property nullable

This is in reference to the Fix the warning messages section using the Visual Studio Code instructions.
Nullable warnings are no longer encountered when following the tutorial because the generated RazorPagesMovieContext.cs file now makes the Movie property nullable.
Added by wadepickett:
The same update needs to occur for:  Part 4, add a model to an ASP.NET Core MVC app

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

ID: 6719f08e-3bd7-dc1a-71df-f2ef9fbca9d8
Version Independent ID: 7096fdb3-612e-9e00-bd0b-8ea4886a09ce
Content: Part 2, add a model
Content Source: aspnetcore/tutorials/razor-pages/model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

What version?  The scaffolded code generated more than one warning.

PS C:\> dotnet --version
6.0.300
PS C:\> dotnet tool list -g
Package Id                       Version      Commands
-------------------------------------------------------------------------
dotnet-aspnet-codegenerator      6.0.4        dotnet-aspnet-codegenerator
dotnet-ef                        6.0.5        dotnet-ef

@Rick-Anderson, The scaffolding has been updated just recently so there is no longer an issue that needs to be addressed. So this section is only needed for previous 6.0 versions.  I think we mention this in the MVC tutorial as well.  VS for Windows/VS Code and VS for Mac.  I just verified in the latest VS for Mac RC.
I'll update for both.

Hmm:  Just saw this new comment on that dotnet issue:
https://github.com/dotnet/Scaffolding/issues/1594#issuecomment-1122928785
"The error appears before the step Install-Package Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore which makes it disappear but the document text doesn't mention the error directly or that it will be resolved with the above step. Thanks!"


Flaky test_operator.test_correlation

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

FAIL: test_operator.test_correlation

----------------------------------------------------------------------

Traceback (most recent call last):

  File "/usr/local/lib/python3.5/dist-packages/nose/case.py", line 198, in runTest

    self.test(*self.arg)

  File "/work/mxnet/tests/python/unittest/common.py", line 157, in test_new

    orig_test(*args, **kwargs)

  File "/work/mxnet/tests/python/unittest/test_operator.py", line 2296, in test_correlation

    unittest_correlation((5,1,15,15), kernel_size = 1,max_displacement = 5,stride1 = 1,stride2 = 1,pad_size = 5,is_multiply = False, dtype = dtype)

  File "/work/mxnet/tests/python/unittest/test_operator.py", line 2266, in unittest_correlation

    assert_almost_equal(exe1.grad_dict['img1'].asnumpy(), grad1, rtol=1e-3, atol=1e-4)

  File "/work/mxnet/python/mxnet/test_utils.py", line 493, in assert_almost_equal

    raise AssertionError(msg)

AssertionError:

Items are not equal:

Error 24.067389 exceeds tolerance rtol=0.001000, atol=0.000100.  Location of maximum error:(4, 0, 7, 1), a=85.000000, b=83.000000

 a: array([[[[ 97.,  93.,  47., ...,  47., 117.,  59.],

         [ 47.,  43.,  89., ...,  77., 107., 117.],

         [ 45.,  47.,  99., ...,  31.,  31.,  41.],...

 b: array([[[[ 97.,  93.,  47., ...,  47., 117.,  59.],

         [ 47.,  43.,  89., ...,  77., 107., 117.],

         [ 45.,  47.,  99., ...,  31.,  31.,  41.],...

-------------------- >> begin captured logging << --------------------

common: INFO: Setting test np/mx/python random seeds, use MXNET_TEST_SEED=159153780 to reproduce.

--------------------- >> end captured logging << ---------------------

http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/master/550/pipeline/
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-10213/7/pipeline/
https://issues.apache.org/jira/browse/MXNET-239

Happened to me too
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-10628/3/pipeline/676

@haojin2 is looking into this

Seems already fixed in #10135

@szha this issue was created after the pr has been merged

@marcoabreu I think #10533 was fixing this flaky test

@marcoabreu my bad for including the wrong PR. Has there been more recent occurrences for this flaky test?

@marcoabreu This actually reminds me of the issue that the issue that sometimes CI is not using the latest master branch automatically, did you have a chance to take a look at that problem?

CI only does not use the latest master if you trigger a rerun in the web interface instead of making a new commit. otherwise, it should be working fine.

And I guess this issue could be closed now? I'm not seeing any new reports of this issue after the fix was merged.

@haojin2 could you report an issue on the CI problem you faced instead?
@marcoabreu is this issue still relevant? Do you have more errors on this test that we should look into?

@marcoabreu Well, then I guess we shall inform everyone with the right to re-run builds about this issue then?

Everybody should be aware. There are not many people who have that permission.
I will check for data tomorrow

@marcoabreu I just asked 2 commiters from my team and none of them were aware of this issue, so I guess there's still the necessity of informing the related ones about this?
And, please close the issue once you can confirm there's no more recent occurrences.

I think I explained it on dev@ a few weeks ago. But yeah, in general, we don't want anybody to use the retrigger-function in the web interface and consider it as a hidden undocumented feature. I'll send out an email when I got time
Closing since there have not been any recent failures.


feat(gatsby): gatsby reference store product pages

WHY are these changes introduced?
Addresses

ENG-4485

WHAT is this pull request doing?

Builds out Gatsby reference store product pages.

Gotchas

Due to gatsby-source-nacelle M1 mac issues, this does not utilize gatsby-source-filesystem.  In order to make it work you need to ensure that useGatsbyImage is false in gatsby-source-nacelle.  To do so you should comment out this line of code in<EMAIL_ADDRESS>
try {
    // require('gatsby-source-filesystem');
    // useGatsbyImage = true;
}

Since we are not currently using gatsby-image some images on the site may look misformatted.  This will be addressed in a follow-up PR once we have figured out the M1 mac issues.

Testing

git checkout ENG-4486/gatsby-reference-pdp
cd references-stores/gatsby && npm install
npm run develop
Visit /products/otto-shirt to ensure the PDP works as expected.

I did notice this issue on tablet size, but I assume that has to do with image formatting not being in place yet?


Move the regular expression, used with auto printing in the viewer, to web/ui_utils.js and also use it in the API unit-tests

Rather than having a copy of this regular expression in the test/unit/api_spec.js file, with a comment about keeping it up-to-date with the code in the viewer (note the incorrect file reference as well), we can just import it instead to simplify all of this.
This is something I noticed while looking at PR #11445, I'll rebase this once that one has landed.

/botio-linux preview

Looks good if the newly added unit test is also adjusted.

/botio unittest

Nice improvement!


Emoji vs Emoji_Presentation

AFAICT in determining whether a code-point should be wide or not the code (or at least the code that gens the tables) relies on checking whether it's emoji or emoji_presentation  (not 100% certain):
# https://www.unicode.org/reports/tr51/#def_basic_emoji_set
emoji = ((emoji_props['Emoji'] - emoji_props['Emoji_Component'])
         | emoji_props['Emoji_Presentation'])

But the current tr51 states:

The emoji code points are those with property values Emoji=Yes, Emoji_Component=No, and Emoji_Presentation=Yes.

It seems the | above is effectively doing an or, though I do not know python so I have no idea if the code is doing what I think it's doing.  However, I do see:
utf8::utf8_width(c('\u2139', '\u2728'))
## [1] 2 2

u2139 is not in the "Emoji Presentation" section of Emoji_data, but u2728 is.
On my system (mojave OS X terminal) this is what I see:

I don't pretend that my system is the end all be-all in terms of the correct display computation, but it appears to behave as per tr51.
There is additional ambiguity with some emoji with text presentation that actually have a wide-ish text presentation:

Though clearly the terminal treats it as 1-wide (FWIW, until recently the terminal also treated normal emojis as 1-wide...).

Agree that the text seems to suggest &. I experimented with this, noted that U+1F1E6 (regional indicator 'A') changes from width 2 to 1, which seems correct.

I just switched "&" to "|" in the emoji-width branch of utf8lite: https://github.com/patperry/utf8lite/tree/emoji-width . This fixes the cases @brodieG mentioned.
I haven't done a detailed before/after comparison, other than spot-checking a few other examples that changed.

I did not do a before / after, but I did compare the full set against R4.0.5 (also against R4.1, but that highlighted 4.1 has some regressions).  On the whole things seem reasonable.  Because R4.0.5 is on 12.1.0 there are differences, but they seem either related to the UCD change, or to different assumptions (e.g. unspecified code point widths).
@krlmlr presumably this will require updating the utf8lite sources embedded in utf8.

Thanks for checking this, @brodieG . The change is now in utf8lite master: https://github.com/patperry/utf8lite/commit/bb81e00a4f743711ff102435bee13f77f9710915

Done, merged.


Circular dep in Wasm import

Basically webpack displays this -
    WARNING in<EMAIL_ADDRESS>232:25-39
    Critical dependency: the request of a dependency is an expression

Due to schnorrkel_js.js importing schnorrkel_js_bg.js and that requiring schnorrkel_js.js

Additionally -
Uncaught RangeError: WebAssembly.Compile is disallowed on the main thread, if the buffer size is larger than 4KB. Use WebAssembly.compile, or compile on a worker thread.
    at Object.eval (schnorrkel_js_bg.js?9681:5)

Worked around the circular dep in https://github.com/polkadot-js/common/pull/306 and the made the WASM init async in https://github.com/polkadot-js/common/pull/307
(For now, basically some adjustments to the generated files which is being maintained in that repo until there are a general solutions available)

Last issue (that I don't have a work-around in for) is the require here -
module.exports.__wbg_require_86edd37cfda5f13d = function(arg0, arg1) {
    let varg0 = getStringFromWasm(arg0, arg1);
    return addHeapObject(require(varg0));
};

Cannot see why WASM would need to require. Anyway, for now will just have to deal with the warning in React apps and deal with the bug reports. (Leaning towards just removing/stubbing that and seeing if it still works)

I've seen the warning but never really paid much attention to it as

I personally prefer nodejs for testing.
I didn't see such an issue in the scope of this project at the beginning. Now with a version published in npmjs I actually do so we have a point here. I'll keep it open and try to find a better fix. It also looks like (as you pointed out) that wasm-pack will expand its publish methods so a plan B is to just wait if it doesn't crash anything.
The first circular import is strange and it is coming from code auto-generated from wasm-pack. Very strange.

I am no master in webpack and these new fancy import (back in my days there was just require and nothing more. no imports in the browser) but this like you mentioned:

Due to schnorrkel_js.js importing schnorrkel_js_bg.js and that requiring schnorrkel_js.js

Is actually what exists in the generated files originally, we are also writing it again as a part of our **fix import **.

Yeap, it is a wasm-pack generation issue, nothing that is introduced elsewhere.


I can add, but not delete, subprojects I don't own

I (stupidly) attempted to add a subproject before I had created it. It so happens that the project (apparently) already existed, and it added it despite me not owning it. When I try to delete the subproject, it says "Sorry, this page does not exist yet".
To confirm, I added another subproject (of a colleague of mine) called bprofile. Same deal. I can add, but not delete the subproject.
Details:
Site: readthedocs.org
Project: labscriptsuite
subprojects: labscript, bprofile
Based on what I've read, the system should not have allowed me to add a project I don't own.
Also, I would suggest allowing the ability to remove a subprojects from a project, even if you don't own the subproject, so that I can clean my project up (I really don't want these random projects to be subprojects of my project!)

Same issue,
Site: readthedocs.org
Project: pyfpdf
subprojects: wiki (that i wanna remove from subprojects)


openedge_function can only been triggered by local mqtt client

Currently according to user guide, MQTT client can access openedge_function via openedge_hub.
The data path is: MQTT client -> openedge_hub -> openedge_function -> openedge_hub->MQTT Client
if the MQTT client wants to access openedge_funcation via baidu_iothub, it will not work
The expected data path is: MQTT client -> iothub -> openedge_remote_mqtt -> openedge_hub->openedge_function.

@tongtongxiaopeng1979
More about remote module user-guide：https://openedge.tech/docs/tutorials/local/Message-synchronize-with-iothub-whith-remote-module
If you want to access openedge-function via baidu-iothub, the data path is : MQTT client(for baidu-iothub) -> baidu-iothub -> openedge-mqtt-remote -> openedge-hub -> openedge-function -> openedge-hub -> MQTT client(for openedge).

thanks for your reply .  there is one question
when you create a openedge_function module for the local hub, you have to fill in the "Privilege name". The priviledge name actually is the permission to local hub. so it means mqtt_remote not have permission to access openedge-function. is it correct?

@tongtongxiaopeng1979
Yeah, actually, all module messages need to go through the hub module, and messages processing depends on rule config.

@tongtongxiaopeng1979 先问个问题：如果消息（数据）已经上云了，为何不在云端接着处理消息？比如使用CFC。OpenEdge的使用场景是在边缘采集和处理消息，然后上云，而不是反过来。
Ask a question first: If the message (data) is already in the cloud, why not continue to process the message in the cloud? For example, use CFC. The use of OpenEdge is to collect and process messages at the edge and then sync to the cloud instead of the other way around.

@ludanfeng
You are right, it is typical use case for the data path you mentioned .  But there is a special use case that need to use FAAS in the openEdge to implement communication with other application or platform in the same host.
这个用例比较特殊，如果openEdge是架构在用户其他平台上面，用户可能会利用openedge FC功能和用户其他平台进行交互。该交互可以由远程一个client触发。

@tongtongxiaopeng1979 你看看下面几种方式哪个更适合你们：

[TODO]openedge的函数计算直接暴露grpc接口，提供给远程grpc client调用
[TODO]openedge支持远程mqtt server直接作为本地hub使用，即把mqtt remote当做本地hub
如同 @kuailehaibin 描述的方案将远程消息先使用mqtt client发送到openedge的本地hub，再通过路由规则触发函数计算

@ludanfeng 先按照mengtao的方法吧，我们公司不是做行业应用和业务的，只不过我手边有个项目有个特殊需求而已。
至于 1 和 2 做不做,还是你们和行业客户来决定吧。
好的，已经加入沟通交流微信群了。 这个问题我先关闭，谢谢你们指导  : )


feat(aws-lambda) add forward_request_uri_path_params config option

Summary
When defining a Kong Route with a regex with capturing groups such as:
/version/(?<version>\d+)/users/(?<user>\S+)
The matched params will be available inside the plugin on local router_matches = ngx.ctx.router_matches.
This adds the config option to forward those matched path params to the Lambda event body under request_uri_path_params.
Therefore, the above route regex will produce a Lambda event body of:
{
  request_uri_path_params = {
    user = "batman",
    ["1"] = "12",
    ["2"] = "batman",
    version = "12",
    ["0"] = "/versions/12/users/batman"
  },
}

Full changelog

Added forward_request_uri_path_params config option and passing params to upstream body
Add related tests.

 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.

Closing because of lack of activity. We would like to accept this contribution if anyone wishes to pick this back up.


[simdjson] Update to 1.0.2

Describe the pull request

What does your PR fix?
Bump version to 1.0.2.

Which triplets are supported/not supported? Have you updated the CI baseline?
All. Yes.

Does your PR follow the maintainer guide?
Yes.

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
<Yes / I am still working on this PR>

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Thanks for the update!


Padding issue of SiamFc

Hi! When evaluating the appearance models on SOT using SiamFC tracker, how does  padding operation, which breaks SiamFC's fullly-convolutional property,  affect the result?
Thanks in advance!

Hi @Morefre ,
Good question!
Considering that all the off-the-shelf SSL models we tested as appearance models are trained with zero padding, we did not evaluate how much it improves by removing zero paddings. But I guess it definitely will improve results in propagation-type tasks like SOT and VOS.


Fix rule

Tweak game rule to match real Texas hold 'em

Fix min raise amount from 5 to big blind amount
Fix order to ask players

before : from dealer button (button player == small blind player)
after : from small blind player (small blind player is next of button player)

Fix to ask action to big blind at least once in PREFLOP

Coverage decreased (-0.05%) to 97.687% when pulling 8ac6d75d2f5312f22ab68426b5f6ce2ee66831d6 on fix-rule into 51cd7b3ce108245e88c84c7973817e11189f323c on 0.1.x.

Coverage decreased (-0.05%) to 97.687% when pulling e739ce7742ad7cbae11aa9a9aa37deddc00998ff on fix-rule into 51cd7b3ce108245e88c84c7973817e11189f323c on 0.1.x.


Image color blend mode

Previously the color's blend mode was always BlendMode.srcIn. That remains the default.
Fixes https://github.com/flutter/flutter/issues/9219


BUG: Fixed broken dogbox trust-region radius update

I was solving some least-squares problems and bumped into quite a bug, which blocks dogbox solver progress.
Explanation: if the trial step was unsuccessful then trust-region radius is updated as Delta = 0.25 * step_h_norm, but in dogbox trust regions are rectangular, thus step_h_norm must be computed with infinity norm, otherwise Delta may not shrink and we get into infinite loop.
Please merge this fix asap.

How hard would it be to add a test for this?

Not very difficult, I can include one rather small problem. I guess you recommend me to do it)
btw: I hope to send a narrative documentation update for least squares in a few days.

Evgeni, a test is here.
I noticed one thing: when using finite diff. Jacobian estimation, we have to set max_nfev (if we want to) for method='lm' much higher than for other methods (as MINPACK counts function calls for Jacobian evaluation). It is not convenient. Maybe introduce automatic adjustment for that or it would be only worse?

The fix looks right, but the test doesn't seem to test it: if I cherry-pick https://github.com/nmayorov/scipy/commit/71d6c5bb909dc7a9016e2d6f4bd219944bbfc873 on top of master, optimize tests still pass.

@ev-br hmm, I deliberately tested for this before sending. Sorry, I will recheck.

Are you sure? When I remove ord=np.inf I immediately get:
FAIL: test_least_squares.TestDogbox.test_bvp
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/nmayorov/anaconda/lib/python3.4/site-packages/nose/case.py", line 198, in runTest
    self.test(*self.arg)
  File "/Users/nmayorov/PythonDev/scipy/scipy/optimize/_lsq/tests/test_least_squares.py", line 353, in test_bvp
    assert_(res.nfev < max_nfev)
  File "/Users/nmayorov/anaconda/lib/python3.4/site-packages/numpy/testing/utils.py", line 53, in assert_
    raise AssertionError(smsg)
AssertionError

Rrright.
OK, when I run the tests with the file name, it fails indeed.
Which prompts the question: Is it actually tested?
Let's see:

I checked out your branch, then $ python runtests.py -s optimize:  Ran 387 tests in 7.992s.
Now, $ rm scipy/optimize/_lsq/tests/* , and rerun tests: Ran 387 tests in 7.935s.

Can you please check this Nikolay?

There's no config.add_data_dir('tests') in _lsq/setup.py, so the test
files don't end up installed.

The numpy test runner also doesn't seem to run the tests on scipy.optimize.test() for some reason.

I always used nosetests .... I will modify setup.py as Pauli suggested.

@pv I'm struggling to get nose to test _lsq when running nosetests scipy.optimize. Can you help with that?
Maybe just put all tests in optimize/tests?

I don't understand distutils (or setuptools, whichever is this), but this change seems to do it:
$ git diff
diff --git a/scipy/optimize/_lsq/setup.py b/scipy/optimize/_lsq/setup.py
index b9222a0..49453ac 100644
--- a/scipy/optimize/_lsq/setup.py
+++ b/scipy/optimize/_lsq/setup.py
@@ -6,6 +6,7 @@ def configuration(parent_package='', top_path=None):
     config = Configuration('_lsq', parent_package, top_path)
     config.add_extension('givens_elimination',
                          sources=['givens_elimination.c'])
+    config.add_data_dir('tests')
     return config

With this, I get the number of tests run in scipy.optimize up to five hundred, the test time jumps threefold (from ~7 sec to ~20sec), and there's a failure on numpy 1.6.2:
======================================================================
ERROR: test_common.test_reflective_transformation
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/br/virtualenvs/scipy-dev/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/br/installs/scipy/build/testenv/lib/python2.7/site-packages/scipy/optimize/lsq/tests/test_common.py", line 230, in test_reflective_transformation
    x, g = reflective_transformation(y, lb, np.array([np.inf, np.inf]))
  File "/home/br/installs/scipy/build/testenv/lib/python2.7/site-packages/scipy/optimize/_lsq/common.py", line 526, in reflective_transformation
    x[mask] = np.minimum(y[mask], 2 * ub[mask] - y[mask])
TypeError: array cannot be safely cast to required type

----------------------------------------------------------------------
Ran 506 tests in 21.462s

FAILED (KNOWNFAIL=1, errors=1)

So basically you added the line config.add_data_dir('tests') and then python runtests.py -s optimize worked? Unfortunately in my setup it doesn't change anything. Not sure what I should do.

Nope, that was a stale build :-(.
What worked was: (i) add the line above, (ii) git mv scipy/optimize/_lsq scipy/optimize/lsq and rename the extension in setup.py of optimize and optimize/lsq. So it's a leading underscore which somehow throws it off.

Probably amount to a bug in numpy.testing, there might be some regexp discarding underscores.
Never looked into that code, though, so I don't know more.
I'd just move the tests to scipy/optimize/tests for now.

Thanks, Pauli. Evgeni are you OK with the decision to move tests?

Skipping _ prefix is a nose bug: https://github.com/nose-devs/nose/blob/master/nose/loader.py#L169

Evgeni are you OK with the decision to move tests?

Sounds like the best way out indeed.

W.r.t. the question of nfev, let's separate it: https://github.com/scipy/scipy/issues/5566

@ev-br
I modified tests of reflective transformation as not to crash on numpy 1.6.2, it doesn't affect real performance as optimization code works with float arrays (the issue was dtype mismatch which isn't handled by numpy 1.6.2)
So I think it's ready to be merged. The error on TravisCI is not related to the changes in this PR.

Indeed, CI errors seem unrelated and should be fixed by gh-5572.
But in Wine, I get two more test failures
======================================================================
FAIL: test_least_squares.TestDogbox.test_grad
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\nose-1.1.2-py2.7.egg\nose\case.py", line 197, in runTest
    self.test(*self.arg)
  File "C:\Python27\lib\site-packages\scipy\optimize\tests\test_least_squares.py", line 567, in test_grad
    assert_equal(res.grad, 2 * x)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 256, in assert_equal
    return assert_array_equal(actual, desired, err_msg, verbose)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 707, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 636, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 100.0%)
 x: array([ 4.])
 y: array(4.0)

======================================================================
FAIL: test_least_squares.TestTRF.test_grad
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\nose-1.1.2-py2.7.egg\nose\case.py", line 197, in runTest
    self.test(*self.arg)
  File "C:\Python27\lib\site-packages\scipy\optimize\tests\test_least_squares.py", line 567, in test_grad
    assert_equal(res.grad, 2 * x)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 256, in assert_equal
    return assert_array_equal(actual, desired, err_msg, verbose)
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 707, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "C:\Python27\lib\site-packages\numpy\testing\utils.py", line 636, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 100.0%)
 x: array([ 4.])
 y: array(4.0)

Looks like these would be easy to fix.

@ev-br I changed those tests.

Great, thanks Nikolay!


Fix spelling of "GitHub"

I used:
perl -i -pe 's/Github/GitHub/g' `git grep -l Github`

Thanks, @splattael. We have to ask you to sign the CLA before we can merge.

@beckettsean Sure, done.

Thanks! Merging.

Thank you :)


generators: Exercise cases helper methods

Separating out the changes to ExerciseCase from moving the test templates....
This PR goes before exercism/xruby#585
I also:

moved ExerciseCase under the Generator module (and gave thanks for sed)
moved the assert helper methods out into their own module, for ease of testing and because ExerciseCase was becoming bloated.
added test coverage for ExerciseCase

Travis is currently broken, tests pass when run locally.


k8s support in java-security

i will skip this PR and check PR #610 instead, it seems, that it provides the missing test case and the multiple xsuaa bindings are supported there...

Hi @liga-oz
could you please add K8SEnvironmentTest here?
Thanks!
Nena

test is available in #610


Add support for FP32/TF32 data formats in WH

add FP32/TF32 data formats for Tensors / CBs and basic metal test

a dependency to train / fine-tune a model

Current issues:

host side conversion bug from fp32 to uint32 and vice versa: fixed
matmul with small tensors (64,64,64) can pass, large tensor fail with bad pcc: investigating host side
llk third party changes causing pcc error: fixed by @rtawfik01
torch2tt_tensor, when store tensor in DRAM, the reader reads out wrong values, when in L1, the value is correct: debugging host side

update:
this function is causing issues on the device side, it does not have Float32 case before the change
inline __attribute__((always_inline)) constexpr static std::uint32_t MUL_WITH_TILE_SIZE(uint format, uint index) {     switch (format & 0x1F) {         case ((uint8_t)DataFormat::Float32): return (index << 12);         case ((uint8_t)DataFormat::Float16):         case ((uint8_t)DataFormat::Float16_b): return (index << 11);         case ((uint8_t)DataFormat::Bfp8_b):         // Keep default as Bfp8?         default: return ((index << 10) + (index << 6));     }; }

The fp32 (input, output, fp32_acc_en) works now for matmul, matmul_no_mcast, and matmul_1d.
I think the OPs needed to add fp32 support (at least to support for Falcon 40B) are follows:
matmul
nlp_create_qkv_heads
rotary_embedding
update_cache
unpad
transpose
group_attn_matmul
scale_mask_softmax_in_place
nlp_concat_heads
matmul_1d
all_gather

@tt-aho pls let me know the above are all the ops required in Falcon, and if there are any redundant ones.

There is also layernorm, interleaved_to_sharded and sharded_to_interleaved (which may be fused into one reshard op or optimized away), and potentially embeddings (currently on host but plan to move to device for inference at some point)
For F40B prefill I think ops are the same with exception of using fill_cache instead of update_cache

Met with @yugaoTT, the following are notes:

Every op has to be updated to have an extra fp32 dest accumulation parameter. It would be good to maintain a spreadsheet which ops support this and which not.
Fp32 as input format should already exist, although it can't be transferred from host to device. The fix should already be in place.
Fp32 as input format will get unpacked into tf32 by the unpacker - that's already in place.

milos

@ttmtrajkovic , can we stagger the merge such that the support for matmul for this feature is in main, and roll out every other ops incremetally as we go?

@jliangTT yes I can merge the matmul only. Then work on other ops.
For the other ops, do we want math_approx_mode, fp32_dest_acc_en, math_fidelity all exposed, or just fp32_dest_acc_en?

Every op has to be updated to have an extra fp32 dest accumulation parameter. It would be good to maintain a spreadsheet which ops support this and which not.

I like @ttmtrajkovic's idea here and we can reason through math_approx_mode, fp32_dest_acc_en, math_fidelity all exposed, vs. just fp32_dest_acc_en op by op.

@jliangTT,
That at topic is being discussed. I need some time to figure out how are ops defined and such and then prpose some updates to make these params common.
@yugaoTT, @jliangTT
The rollout can and should be incremental, but it would be great to see a spreadsheet of ops that support fp32 accumulation and the ones that dont.
Milos

here is the op list
https://tenstorrent.sharepoint.com/:x:/r/sites/Jasmina/_layouts/15/Doc.aspx?sourcedoc={08306401-7C55-4F8C-BD7A-8F2C4B8EB9C7}&file=Book.xlsx&action=default&mobileredirect=true

Update:
the list above is done, needs model side testing then merge.
Next step:
add fp32 to convs.

optimized_conv
groupnorm

Update: the list above is done, needs model side testing then merge. Next step: add fp32 to convs.

optimized_conv
groupnorm

@razorback3, fp32 support is added for matmul. Please check in main.

@yugaoTT,
Could you please update the ticket with the status of adding fp32 to the remaining ops? thanks.

the list above is done, but haven't merged yet. Since all the ops (convs, matmul, group_attn_matmul) that needs immediate fp32 support is merged, can we downgrade the urgency of this issue?

ok. i will lower it to P2 for now as this has meet the urgent need to add this to matmul.

Thanks, although I am not sure I fully understand. So all the ops from the list have fp32 added, to either a branch (unmerged) or in main (merged), is that the correct? If yes, please specify status of merged vs non-merged.
Other than the ops on the list, are there others that would need fp32 accumulation?
My suggestion would be to wrap up the task and enable this across the board since there will be users that for training need to propagate fp32 to pretty much all the ops, especially if they create more complex fused ops. All compute OPs that operate on DST, should have this parameter.
Milos

ok. i will lower it to P2 for now as this has meet the urgent need to add this to matmul.

@jliangTT, I'd prefer if this is P1 - feature needed as training will heavily rely fp32. in addition, this is still a feature that is needed, rather than nice to have.

it is in yugao/fp32_nlp_debug
merged:
matmul
matmul_1d
group_attn_matmul
optimized conv
unmerged
rotary_embedding
nlp_create_qkv_heads
nlp_concat_heads
update_cache
unpad
transpose
scale_mask_softmax_in_place
layernorm
all_gather
binary add
interleaved_to_sharded
sharded_to_interleaved
embeddings
fill_cache
It is not the full list but covers most ops, there are other ops in tt_eager/tt_dnn/op_library that is not in the list.
I can add them after developing GN and benchmarking DRAM. when will the model training start?

Thanks @yugaoTT.
The training is already in progress by Moreh and explicit request has been made for eltwise unary ops (to manipulate fp32 data in sfpu). I will work on adding that support.
Let's revisit status of this next week.

i am okay with keeping it as P1. From a tactical angle, Moreh is likely to integrate the top few ops already in main now, then more later.

thanks @ttmtrajkovic

@ttmtrajkovic  I can make a few PR to rollout the ops above batch by batch this week.


radicale: Fix runtime

Needed pkg_resources module, which apparently comes from setuptools
according to https://stackoverflow.com/a/10538412/6605742
Motivation for this change
Fixes radicale and therefore the NixOS test as well for #68361
Things done

[x] Ran the NixOS test successfully
[x] Run the radicale binary, which now doesn't throw a pkg_resourceS error anymore

@GrahamcOfBorg test radicale

Backported in 30eb7ba00b09beeaf9762ab5a4ea505d612cc1dc


feat: opt-in gh app integration

this PR adds opt-in GitHub App integrations to the server.
the main differences are:

integrate with an OAuth GitHub App by providing an app id and private key
use the GitHub App to generate the .netrc password for builds when the app is installed and it can access the repo
adds webhook handlers for GitHub App installation and installation_repositories events, and syncs existing repos when changes are made
~adds Checks functionality, pulled from the original hackathon efforts #1070~

pulled out this code, it will be a follow-up PR

New Flags

Key
Description
Default Value

VELA_SCM_APP_ID
set to the App ID for the GitHub App
N/A

VELA_SCM_APP_PRIVATE_KEY
the string value for the GitHub App private key generated through GitHub
N/A

Required GitHub App Configurations
Permissions
the GitHub App requires the following permissions at the very minimum:

contents:read
checks:write

builds would request write permissions through the git yaml block, see below.
Subscribed Events

Installation target

OAuth
the same configurations and oauth scopes should be assigned to the GitHub App, including:

oauth callback url set to /authenticate (like usual)
Webhook URL set to the base url (like usual)

New YAML block: git
integrating with a GitHub App allows the use of the git YAML block for customizing the permissions allocated to the netrc password embedded into Vela steps.
git:
  token:
    repositories:
      - foo/bar
      - helloworld
    permissions:
      contents: write
      checks: write

this lets users customize the list of repositories that the netrc password has access to, but that list is restricted to ONLY the repos that the GitHub App org installation has been given access to.
by default, the compiler will use the following configurations unless otherwise provided:
git:
  token:
    repositories:
      - VELA_BUILD_REPO
    permissions:
      contents: read
      checks: write

:warning: Considerations
this WILL impact builds, check out the following list of things to consider when migrating to GitHub App
Cloning Private Resources
due to the new restrictive policies set on the netrc token, Vela builds might lose the ability to read/write from certain private repos that the repo owner may have had access to.
GitHub Apps do not support providing access to repos that are outside the installation org. meaning, for a Vela build to access private repos, Go modules, etc, then the build author must provide override the clone step and use an alternative authentication method like a PAT

Can you expand on why the contents:write is required instead of contents:read?

Can you expand on why the contents:write is required instead of contents:read?

whoops thats a typo in the description. the code is accurate though, the minimum is contents:read 👍

but that list is restricted to ONLY the repos that the GitHub App org installation has been given access to.

I think I know the answer, but just to confirm: if I install the GitHub app on both the foo and bar organizations and run a build within a repo in the foo org, does that mean I won’t be able to clone a repo from the bar org, even if I specify bar/<repo> in the token section?

but that list is restricted to ONLY the repos that the GitHub App org installation has been given access to.

I think I know the answer, but just to confirm: if I install the GitHub app on both the foo and bar organizations and run a build within a repo in the foo org, does that mean I won’t be able to clone a repo from the bar org, even if I specify bar/<repo> in the token section?

that's correct, org installations are scoped to repos in that org, unfortunately. its extremely similar to fine-grained access tokens (the beta PATs)


modelPaths will not import a single file without glob 'magic'

Because of line
https://github.com/RobinBuschmann/sequelize-typescript/blob/master/lib/services/models.ts#L185
you cannot import a single model, because it appends /* to something like
modelsPath: [ __dirname + 'models/version.ts' ]

A quick workaround is to artifically insert glob syntax into the path (instead of version.ts use ?ersion.ts). This skips over the problematic if statement, although it obviously wont work if you have version.ts and Version.ts in the same directory.

Hi @Danlock, thanks for reporting. modelPaths wasn't intent to be used for targeting a single file. Use addModels of the Sequelize instance instead. Here you can pass the reference of the class:
import {Sequelize} from 'sequelize-typescript';

const sequelize = new Sequelize({...});

seqeuelize.addModels([Version]);

Probably it makes sense to allow passing model references in the options in the future as well 🤔

even if it isnt valid for a single file, this issue blocks users from passing in multiple single files as well, a common use case. I think you should at least document the glob workaround for users just passing in a few models files so they wont get confused like i did.


[DOC]  Document default size limit for rpc requests to view state

Is your feature request related to a problem? Please describe.
Developers are not aware that they cannot do a rpc call to view states of contracts that are very large.
Describe the solution you'd like
Document the 50kb default limit that we have on viewing state from rpc and also how to change it if you run your own nodes.

@bowenwang1996 @janewang Ideally, we need a doc about all the configuration (config.json) options, so we can reference to the specific tweaking option from the JSON RPC doc page.

Resolved in #811.
See it live: https://docs.near.org/docs/api/rpc/contracts#view-contract-state


⚠ Elibs has degraded performance

In 3c530b6, Elibs (https://elibs.unmer.ac.id) experienced degraded performance:

HTTP code: 200
Response time: 5351 ms

Resolved: Elibs performance has improved in 88d2781.


CCE 18 on Frontier & Benchmarking fixes among others

Attempting to upstream the bits from #544 that do not hinder performance.

Ugh, waiting on Phoenix GPU nodes. I will look more closely at that this afternoon and see what's up.


MAINT: Make copy of manifold_learner before using it, if passed to the constructor.

Currently no copy is made, and the passed learner is modified in-place.
https://github.com/zoj613/pyloras/blob/d39c5b583513df251059aee8b36f5022a7e82bab/pyloras/_loras.py#L129-L131
https://github.com/zoj613/pyloras/blob/d39c5b583513df251059aee8b36f5022a7e82bab/pyloras/_loras.py#L134-L139
A copy should be made using sklearn's clone function so as to preserve the passed object.

closed by #29


Upgrade awssdk to v2.21.5

Upgrade AWS SDK to latest version i.e. v2.21.5. The library seems to have gotten downgraded to 2.14.15 that was released 2020-09-10.

Thanks, a release will be coming soon, we're just completing some performance work


Laravel 5.0 ShouldQueue not found

"FatalErrorException in ChunkedReadJob.php line 10: Interface 'Illuminate\Contracts\Queue\ShouldQueue' not found"
get this error what am I missing, there is no such file in my laravel 5.0. Can you please guide me from where can I get this?

Resolved rename ShouldQueue to  ShouldBeQueued


RTSP|av_interleaved_write_frame(): Broken pipe

Which version are you using?
v1.5.0
Which operating system are you using?

[ x] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
I have a jellyfin server and I decided to create channels from my movie folders. I have setup 37 channels, 8 of them crash with
av_interleaved_write_frame(): Broken pipe
Error writing trailer of rtsp://<IP_ADDRESS>:8554/movies5: Broken pipe

Could be hours later or minutes later
Describe how to replicate the issue
The command I'm using is
ffmpeg -hide_banner -loglevel error -re -f concat -safe 0 -i "$playlist" -c copy -f rtsp -rtsp_transport tcp "rtsp://<IP_ADDRESS>:8554/$channel_id" >> /root/.ffmpeg_log.txt 2>&1 &

The playlist is a concat playlist containg avi, mkv and mp4 files all with h264/mpeg4 video and aac/ac3 audio
MediaMTX, FFMpeg and the script run as root
Did you attach the server logs?
no
Did you attach a network dump?
no

Hello, the information you posted don't provide any clue about where the error is, or whether there is any error in the server. Please reopen the issue and include a way to replicate the issue and full server logs as explicitly indicated in the issue instructions.


Kernel#p when given no arguments triggers a Rust unsafe precondition violation

Fuzzer breakage
The latest run of the fuzz targets have found another trophy:

https://github.com/artichoke/artichoke/actions/runs/7854703302/job/21435704557 – triggers on this fuzz target:[3.3.0] > [112, 46, 215, 215, 97, 101].map(&:chr).join
=> "p.\xD7\xD7ae"

which when invoked, gives an error that indicates that p is being called with no arguments:$ ruby -e 'puts [112, 46, 215, 215, 97, 101].map(&:chr).join' | ./target/debug/artichoke
Traceback (most recent call last):
(eval):1: undefined method 'ae' (NoMethodError)

https://github.com/artichoke/artichoke/actions/runs/7854703302/job/21435704471 – triggers on artichoke_backend::extn::core::kernel::tests::functional test:
https://github.com/artichoke/artichoke/blob/287b1fb132d1cf3bfb4800ac05c7bac6a2a71fd7/artichoke-backend/src/extn/core/kernel/kernel_test.rb#L611-L615

Reproducer
As of the following nightly, the simplest reproducer is to invoke p with no args:

 nightly version info
$ RUST_BACKTRACE=1 RUSTFLAGS="-Z sanitizer=address" rustc +nightly -Vv
rustc 1.78.0-nightly (6cc484351 2024-02-10)
binary: rustc
commit-hash: 6cc4843512d613f51ec81aba689180c31b0b28b6
commit-date: 2024-02-10
host: x86_64-apple-darwin
release: 1.78.0-nightly
LLVM version: 17.0.6

$ RUST_BACKTRACE=1 RUSTFLAGS="-Z sanitizer=address" cargo +nightly run --bin artichoke --target x86_64-apple-darwin -- -e 'p'
    Finished dev [unoptimized + debuginfo] target(s) in 0.09s
     Running `target/x86_64-apple-darwin/debug/artichoke -e p`
artichoke(83982,0x7ff851972b80) malloc: nano zone abandoned due to inability to reserve vm space.
thread 'main' panicked at library/core/src/panicking.rs:155:5:
unsafe precondition(s) violated: slice::from_raw_parts requires the pointer to be aligned and non-null, and the total size of the slice not to exceed `isize::MAX`
stack backtrace:
   0: rust_begin_unwind
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/std/src/panicking.rs:647:5
   1: core::panicking::panic_nounwind_fmt::runtime
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/core/src/panicking.rs:110:18
   2: core::panicking::panic_nounwind_fmt
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/core/src/panicking.rs:122:9
   3: core::panicking::panic_nounwind
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/core/src/panicking.rs:155:5
   4: core::slice::raw::from_raw_parts::precondition_check
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/core/src/intrinsics.rs:2634:21
   5: core::slice::raw::from_raw_parts
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/core/src/slice/raw.rs:98:9
   6: artichoke_backend::extn::core::kernel::mruby::kernel_p
             at ./artichoke-backend/src/extn/core/kernel/mruby.rs:69:16
   7: mrb_vm_exec
             at ./artichoke-backend/vendor/mruby/src/vm.c:1792:18
   8: mrb_vm_run
             at ./artichoke-backend/vendor/mruby/src/vm.c:1282:12
   9: mrb_top_run
             at ./artichoke-backend/vendor/mruby/src/vm.c:3112:12
  10: mrb_load_exec
             at ./artichoke-backend/mrbgems/mruby-compiler/core/parse.y:6918:7
  11: mrb_load_nstring_cxt
             at ./artichoke-backend/mrbgems/mruby-compiler/core/parse.y:6990:10
  12: <artichoke_backend::sys::protect::Eval as artichoke_backend::sys::protect::Protect>::run
             at ./artichoke-backend/src/sys/protect.rs:115:9
  13: protect_body
             at ./artichoke-backend/vendor/mruby/mrbgems/mruby-error/src/exception.c:13:10
  14: mrb_protect_error
             at ./artichoke-backend/vendor/mruby/src/vm.c:431:14
  15: mrb_protect
             at ./artichoke-backend/vendor/mruby/mrbgems/mruby-error/src/exception.c:20:10
  16: artichoke_backend::sys::protect::protect
             at ./artichoke-backend/src/sys/protect.rs:46:17
  17: artichoke_backend::sys::protect::eval
             at ./artichoke-backend/src/sys/protect.rs:25:5
  18: artichoke_backend::eval::<impl artichoke_core::eval::Eval for artichoke_backend::artichoke::Artichoke>::eval::{{closure}}
             at ./artichoke-backend/src/eval.rs:27:42
  19: artichoke_backend::artichoke::Artichoke::with_ffi_boundary
             at ./artichoke-backend/src/artichoke.rs:98:26
  20: artichoke_backend::eval::<impl artichoke_core::eval::Eval for artichoke_backend::artichoke::Artichoke>::eval
             at ./artichoke-backend/src/eval.rs:27:13
  21: artichoke_backend::eval::<impl artichoke_core::eval::Eval for artichoke_backend::artichoke::Artichoke>::eval_os_str
             at ./artichoke-backend/src/eval.rs:52:9
  22: artichoke::ruby::execute_inline_eval
             at ./src/ruby.rs:192:23
  23: artichoke::ruby::entrypoint
             at ./src/ruby.rs:150:9
  24: artichoke::ruby::run
             at ./src/ruby.rs:117:18
  25: artichoke::main
             at ./src/bin/artichoke.rs:34:11
  26: core::ops::function::FnOnce::call_once
             at /rustc/6cc4843512d613f51ec81aba689180c31b0b28b6/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
thread caused non-unwinding panic. aborting.
zsh: abort      RUST_BACKTRACE=1 RUSTFLAGS="-Z sanitizer=address" cargo +nightly run --bin

With the following diff, count is clearly correctly getting set to 0. I believe the issue is that mruby is not making any guarantees about the args pointer value if there are indeed no args pushed onto the VM's stack.
diff --git i/artichoke-backend/src/macros.rs w/artichoke-backend/src/macros.rs
index d888165fae..7b6bc4a78a 100644
--- i/artichoke-backend/src/macros.rs
+++ w/artichoke-backend/src/macros.rs
@@ -422,6 +422,8 @@ macro_rules! mrb_get_args {
             args.as_mut_ptr(),
             count.as_mut_ptr(),
         );
+        dbg!(count.assume_init());
+        dbg!(_argc);
         std::slice::from_raw_parts(args.assume_init(), count.assume_init())
     }};
 }

Fixing this should be as simple as returning an empty slice if count == 0.


Add customClaims to methods

add customClaims to attemp and generateTokenById

Please read the contributing guidelines before opening a PR.


Add a docs.oasislabs

we should run cargo docs across the full project and provide a documentation reference of current state of rust docs over the whole project.
Likely lives as a component in our deployment script and is hosted somewhere in AWS

Superseded by #2460 which will include docs.rs automatically.


Kruskal's algorithm

Kruskal's minimum spanning algorithm.
Kruskal's minimum spanning algorithm implemented in Python
Programming language

[ ] C
[ ] C++
[ ] Java
[x] Python

I can do this in C++. Please assign me this issue

I have already solved this issue using python

I'm participantnof LGM-SOC21. I can solve it through java . Please assign me this issue


Scale type is always fit center

The library is good, but image's scale type is always fit center even if set it to center crop, anyone who faced same issue?

Same issues here. I notice that the effective scaleType is always fitCenter, even though you had specified other scaleType
<com.github.chrisbanes.photoview.PhotoView xmlns:android="http://schemas.android.com/apk/res/android"
    android:id="@+id/image_view"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:adjustViewBounds="false"
    android:scaleType="centerInside" />

Even you tried to do it explicitly in Java code doesn't help
this.imageView.setScaleType(ImageView.ScaleType.CENTER_INSIDE);

I'm using
implementation 'com.github.chrisbanes:PhotoView:2.1.3'


Fix case when typedef references struct that is not yet declared

Generating bindings for /usr/include/uv.h gives lots of errors and does not output Scala code:
Failed to get declaration for struct __fsid_t
Failed to get declaration for struct _IO_FILE
Failed to get declaration for struct __va_list_tag
...

The problem is that typedef for struct can be located above struct declaration:
typedef struct myStruct mystruct;

struct myStruct {
    int a;
};

So when TreeVisitor visits typedef it is unable to translate struct myStruct because type is not yet in TypeTranslator::aliasesMap

This is similar to #78 or at least it looks like both would require the same sort of lazy behavior.

Yes, it is similar


Error: something like exposing (..) which replaced where in 0.17

(Let me prefix this by saying I'm completely new to elm, just started reading today.)
Every time I save a file, vscode-elm injects a module definition at the top of my code:
example:
module Main (..) where

For some reason this module definition results in the following error:
-- SYNTAX PROBLEM ----------------------------------------------------- test.elm

I ran into something unexpected when parsing your code!

1| module Main (..) where
               ^
I am looking for one of the following things:

    something like `exposing (..)` which replaced `where` in 0.17
    whitespace

As I understand it, this is because I'm running a newer version of elm compared to the injected syntax. Is there a way to disable this?
Thank you in advance

I'm having the same problem. Whenever I declare a module I get this error.
Have to reboot editor for it to work!
- -- SYNTAX PROBLEM --------- elm-stuff/packages/elm-lang/core/3.0.0/src/Array.elm

I ran into something unexpected when parsing your code!

2|     ( Array
       ^
I am looking for one of the following things:

    something like `exposing (..)` which replaced `where` in 0.17
    whitespace

The fix is to do package install, otherwise it defaults to some old built in elm version
elm package install
Some new packages are needed. Here is the upgrade plan.

  Install:
    elm-lang/core 5.1.1
    elm-lang/html 2.0.0
    elm-lang/virtual-dom 2.0.4

Do you approve of this plan? [Y/n] Y
Starting downloads...

  ● elm-lang/virtual-dom 2.0.4
  ● elm-lang/html 2.0.0
  ● elm-lang/core 5.1.1

Thank you for the info. I'm a bit swamped at the moment to check it out, but as soon as I have an evening I will check it out!


Change cleanup logic, improve admin and observability

Currently, the server assumes its external /cleanup path will be called at intervals (currently 2 minutes).   My thought was to use scheduled functions for this but that was a just a cutesy holdover from my DO days ... the code will be more understandable and self-contained (and I will be saved from writing a whole separate serverless component) if this capability is done with a go thread.
At the same time, there are certain things I want to do administratively that I currently can't.

Have to be able to observe the server's complete internal state (what games exist and the current player list and game state for each, along with idle counts).
Want to be able to do a logical restart of the server without having to actually kill and restart the process.   This comes down to just the "nuclear" form of cleanup (ignoring idle counts and removing everything).

In the long run, we will want the fancy form of this

This was addressed in the commit linked above, but the results have not been tested yet.  Also the new /dump route will cause the server to dump state to the console (visible in the runtime log) but does not return it to the caller.  That should be improved.
Because I'm now using a go time.Ticker instead of an external scheduler, I can use second granularity and coordinate the constants a little more tightly.  Cleanup will run every 15 seconds.   Player timeout is now 45 seconds and game timeout is 300 seconds.   The tight player timeout will be fine as long as clients are polling, but, eventually we might need a longer value since a player can certainly consume 45 seconds of think time between moves.
Once we switch to gRPC and push notifications for game state changes, there is no need for polling but we could still have liveness exchanges.  Also, there may be a way to detect a broken channel in real time and so we won't need player timeouts at all.

Accumulated changes both under this issue and others have substantially addressed the issue.   Cleanup is now working (perhaps the constants that govern times-to-live need further tweaking).   Both the /dump and the /reset routes are working and reflect server state back to the client.   This has proven quite useful in fixing other problems.   If we need even better instrumentation (probably will) that can be addressed down the road.


BS4 error from samair proxy parser

  File "/usr/lib/python3.6/site-packages/http_request_randomizer/requests/proxy/requestProxy.py", line 55, in __init__     self.proxy_list += parsers[i].parse_proxyList()   File "/usr/lib/python3.6/site-packages/http_request_randomizer/requests/parsers/SamairProxyParser.py", line 42, in parse_proxyList     headings = [th.get_text() for th in table.find("tr").find_all("th")] AttributeError: 'NoneType' object has no attribute 'find'

Hello @Hessu1337
Samair proxy provider looks unreachable to me. For instance,  http://samair.ru/proxy/time-01.htm page returns a 404 error - that's why the specific proxy provider is skipped when I am listing all the available proxies. Are you sure the provider is reachable?
Cheers,
Panagiotis

Hi @pgaref ,
It's reachable for me, made a pull request with bs4 findAll fix and working like a charm here.

Hello @Hessu1337
I guess they introduced geo-restrictions. May I ask the location(country) of your parser?
Thanks for the PR, please check some inline comments in the PR.
Cheers,
Panagiotis

Of course, the ip location is based in France.


Jdempotent-spring-boot-redis-starter not invoked (local debug mode)

Hello,
I have done some changes to support the IdempotentKey but I am unable to enable/activate the Jdempotent-spring-boot-redis-starter library in a local way

com.trendyol
Jdempotent-spring-boot-redis-starter
1.0.8
system
****MYPATH***/.m2/repository/com/trendyol/Jdempotent-core/1.0.8/Jdempotent-core-1.0.8.jar

I ahve included the new Jdempotent-core with the IdempotentKey just like the changes that I have done here https://github.com/IOhacker/Jdempotent
https://github.com/IOhacker/Jdempotent/commit/64e44f496d3189afd65d2d7af8e7cc741496f12a
I will try to debug which is the issue for not enabling the redis...
Thank you

My bad, newbie error.


Replace kapt with ksp

Goal
Replace usage of kapt with ksp now that our Kotlin version has been updated.
Testing
Relied on existing test coverage.

Merge activity

Jul 17, 1:06 PM EDT: @bidetofevil merged this pull request with Graphite.


Model CI Release

🚢 🚢 🚢  🚀 🚀🚀 Add model CI extension to Nucleus python client 🚀 🚀🚀 🚢 🚢 🚢

Draft release notes:
Adds an extension to the Nucleus Client for newly introduced Model CI functionality.  With the Model CI, extension, users can use Nucleus slice’s to define test cases that describe critical edge case scenarios. A unit test can have one or more evaluation criteria, which define how to apply evaluation functions to a unit test.  The newly introduced classes are:

UnitTest
EvaluationFunction
UnitTest Evaluation
UnitTestMetric

Documentation can be found at: Nucleus API Reference
Happy Coding!


org-mode parser npm license to MIT or BSD?

Hello daitangio,
Firstly thank you for the superb library. I wanted to use for my own org-mode files within my company, but unfortunately the policy does not allow us to request software with GPL licenses from Node.
Would it be possible to switch the npm version to another acceptable license (e.g. FreeBSD, MIT) so that I am able to use at work? (We import packages directly from npm, so the license would need to be changed there if this is possible)
Many thanks!

Hi, Okey I will change the license for better interop. Give me few days to decide. Do you have suggestion between BSD and MIT?

No preference either is fine with me. Many thanks!
On 16 Nov 2017 21:52, "Giovanni Giorgi"<EMAIL_ADDRESS>wrote:

Hi, Okey I will change the license for better interop. Give me few days to
decide. Do you have suggestion between BSD and MIT?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/daitangio/org-mode-parser/issues/15#issuecomment-345074424,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARyOUx_FZjyBVWnYTq2ehMiABd9-1Coks5s3K6EgaJpZM4QQK1h
.

I decided to change to MIT license, the same of nodejs and things like jQuery.
Please retain copyright attribution.
Also can you give me the name of the project will use the org mode parser (to track it)?

@daitangio https://github.com/xeijin/propositum
Many thanks!

@daitangio Would you also mind doing a release so that the updated license gets pushed to npm? (still shows GPL 2 on the NPM site)


docs: Documentation Link Misdirection

Description
I observed in the documentation the link Intro to Open Source is misdirecting users to an unintended destination. Instead of leading to the intended document, the link directs users to the GitHub repository.
https://github.com/open-sauced/docs/assets/106407013/682e3755-cfda-46e7-a4e2-eaa2f172e731
Description
I noticed that in the documentation, there's an issue with a link that leads to the wrong destination. The link in question is supposed to point to a document, but it is currently leading to the GitHub repository instead. This is causing confusion for users who are trying to access the correct document.

Expected Behaviour:
Clicking on the link should take the user to the relevant document.
Actual Behavior:
Clicking on the link leads to the GitHub repository instead of the intended document.
Affected Link:
https://github.com/open-sauced/intro
Proper Link:
https://intro.opensauced.pizza/#/
Steps to reproduce

Visit the URL: "https://docs.opensauced.pizza/community/100-days-of-oss/"
Locate the section titled "Where to Start?"
Click on the hyperlink provided in the second bullet point

Browsers
Chrome
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

.take


Is my project possible with this program?

I am considering purchasing the book - but I am an absolute beginner and don't understand even the introduction, so I am going to need to read very carefully. However, before I start, I want to know whether my project is even possible. What I want to do is have a Web App that shows a gallery of magazines (that have been created as PDFs). Either access to the Web App is via password or - more challengingly - via the purchase of a subscription that currently is handled by Stripe. The PDFs would have to be read in a reader - and ideally have the possibility for the two page view as well as the one page. What I do not know is whether a Web App can be accessed by people who have iPads?
So the idea would be that people would click on a thumbnail and open up a PDF. Pretty basic, but I have seen no examples of whether this would work with this program.
Please could you let me know?
Many thanks
JM

Hi @hightrees , thanks for the question.
Our book does not cover the exact project you explain. While it seems possible, you would need to modify the builderbook web app, and doing that will depend on your level of experience. You would have to use React and JS to create a grid (gallery) with password access and password + subscription access.
You can always work with our final builderbook web app on GitHub and learn React, Material-UI and JS to modify it at your own pace. We provide instructions to run this app on the README.

Many thanks for your email. I will have a think and then see what I can do!
Best wishes
Jeremy
On 15 Oct 2022, at 17:10, Kelly Burke @.***> wrote:
Hi @hightrees https://github.com/hightrees , thanks for the question.
Our book does not cover the exact project you explain. While it seems possible, you would need to modify the builderbook web app, and doing that will depend on your level of experience. You would have to use React and JS to create a grid (gallery) with password access and password + subscription access.
You can always work with our final builderbook web app https://github.com/async-labs/builderbook/tree/master/builderbook on GitHub and learn React, Material-UI and JS to modify it at your own pace. We provide instructions to run this app on the README.
—
Reply to this email directly, view it on GitHub https://github.com/async-labs/builderbook/issues/505#issuecomment-1279763872, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACP57J7XNIHIZF6HCCR6XX3WDLCMZANCNFSM6AAAAAAREILMGI.
You are receiving this because you were mentioned.


Create flutter folder

I create the flutter folder and add into the folder two books by following the CONTRIBUTING file.

Hi @cisse410, theses books are very useful. A lot of people have reached out to me asking me to add this kind of books.
Thank you for your support on building a where-to-go library.
Could you please share it with your network so that they can add new books?

@mr-mib It gives me great pleasure to contribute to this wonderful initiative.
I wouldn't hesitate to add more books.

@mr-mib Some of these books are available in this article
https://tonux-dev.com/roadmap-pour-devenir-un-developpeur-flutter-performant/

Thanks
Merci


Access website on root path (/)

As a user I'd like to access a website by writing the swarm reference in the address bar, without specifying a specific file to access (for example /index.html)

[ ] #647

I would like to start a discussion on what functionality this feature should provide.
Possible solutions:
1. Define default path for just the root of the collection
Specify which file should be served under /. For example localhost:8080/bzz/swarm.eth specifying the value for default path /index.html would serve localhost:8080/bzz/swarm.eth/index.html.
2. Define "index" file name for all directories
This is similar like the "index" directive in nginx configuration. Default path would be the filename relative to the current directory in the collection. For example, specifying "index.html" as defaultpath in this case would serve directory roots in with this mapping:
localhost:8080/bzz/docs.swarm.eth: localhost:8080/bzz/docs.swarm.eth/index.html
localhost:8080/bzz/docs.swarm.eth/installation: localhost:8080/bzz/docs.swarm.eth/installation/index.html
localhost:8080/bzz/docs.swarm.eth/quick-start: localhost:8080/bzz/docs.swarm.eth/quick-start/index.html
3. Define default path for every directory
Allowing to set a specific file for each and every directory would be the most robust setting, which is also possible with nginx configuration. In this case, every directory would be able to specify its own "index" file within its path.
My opinions
While 3. gives the most functionality to the user, I would say that it is not so common and that 2. should be the most useful option. Feature 1. is just too restrictive as it does not provide functionality for static websites serving. I would go with feature 2.
This feature is just one of the features that are needed for website hosting. And I think that it should be designed alongside features such are:

error page: allows custom error page, but more important serving the default path for any path that is not found, to support single page javascript apps with routers
directory listing: how a user would configure which directory should have a list of its files, or the default path should be served

cc @zelig @Eknir @acud @zbiljic @significance and anyone else who would like to participate

Great write-up @janos, many thanks for this. I agree that 2 would be the best option and gives at least the minimum set of features we need to deliver here, but the thing which I am only afraid of in this context is that if this setting is written only at the root of the manifest, then we lose the ability to redirect to the error/index page once we do not traverse the manifest from the root.
Also, since this is a recursive data structure, it is not clear to me if implementing 1 or 3 would actually be any different.

For 2. the setting would have to be implemented in the root of the manifest, or in order not to traverse from the root, to have it under every branch, which brings the option 3. very doable (as you said), just to figure out what would be the ux to configure such options.

Can this issue be closed?

I would like to hear if @zbiljic would have to have something to add he does some analysis, as we recently discussed about possible improvements to the pr that got merged.


unable to install latest git v2.40.1 because it has a dependency on package_has_been_revoked

Below error is seen while installing git v2.40.1
Collecting package metadata (current_repodata.json): done
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: |
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.                                                                                failed

PackagesNotFoundError: The following packages are not available from current channels:

  - package_has_been_revoked

Current channels:

  - https://repo.anaconda.com/pkgs/main/linux-64
  - https://repo.anaconda.com/pkgs/main/noarch
  - https://repo.anaconda.com/pkgs/r/linux-64
  - https://repo.anaconda.com/pkgs/r/noarch

According to Anaconda, this issue should be fixed now.
The latest build has been uploaded, and you can now upgrade to fix the issue you were facing.


Add the Indoor Pot code to the refactored Held Object code

@Jolly-Alpaca  - Thanks a lot for your comments and calling out where I didn't interpret your code correctly!  I'm going to be traveling for the next week so won't be able to make the code changes.  If you can push to my branch, feel free to do so!
Also, I did some playtesting, but haven't done any real stress-testing of the near-full-chest scenarios yet.  I'm figuring to stick with the plan of just having the Junimos do nothing if the chests aren't full.  It's not the most user-friendly thing that could be done, but it's simple and safe.
I also hope you'll give some thought to an event to unlock dedicated garden-pot junimos.  I'm also looking for new-spouse-meets-the-junimos event ideas.  Might also be fun to have the Junimos interact with crawlers and toddlers sometimes...

The most I've done with spouses is Krobus and never have had kids in stardew, so no ideas there.
For the garden-pot junimo event, I'm thinking something involving Evelyn since she's the one who gives you the garden pot in vanilla.
One script idea:

The player receives a letter from Evelyn asking the player to visit. The event can be at either Evelyn's house or the community gardens since Evelyn's dialogues mention she is in charge of tending to those. The latter may be more thematically appropriate for a junimo that tends to crops, but may be awkward if the event is triggered in winter.
Upon triggering the event, a cutscene plays involving Evelyn showing the player a garden pot (or maybe carrying it) and something junimo related happening, causing the pot to fall and break.  (I don't have any stardew event experience. Sorry! 😅 ). Perhaps the junimo is napping inside the garden pot when Evelyn picks it up, is startled, and causes Evelyn to drop the pot. Most people can't see junimo, which would explain why Evelyn didn't see it in the pot, but a lore check would be necessary to confirm that. (Alternatively, she could be grabbing it from a shelf where she can't see inside it w/o removing it from the shelf and the junimo scampers off before she sees it.)
After the event, the player is approached by the junimo who feels bad for breaking the pot and would like the player's help in creating a new pot. Undecided if this would be a cutscene triggered when the player returns to the farm or if it would be an overnight event.
The quest would involve getting the materials needed to craft a garden pot (could just full on use the vanilla recipe) and giving them to the junimo.
Overnight, magic happens and the junimo creates the pot and secretly delivers it to Evelyn.
The following morning, the player receives another letter from Evelyn, asking the farmer to visit again.
During this event cutscene, she tells the player how surprised she was to have found a pot had mysteriously appearing on her doorsteps. She goes on to explain that she had originally been planning to gift the garden pot to the player before it broke. She then gifts the garden pot to the player (as opposed to how it is normally obtained through mail in vanilla).
Overnight, the night time junimo event triggers with the junimo thanking the player for their help and saying they'll help tend to your garden pots, unlocking garden pot junimo.

@Jolly-Alpaca  - I've been kicking the event around in my head while traveling (and, heh, dealing with the inevitable post-travel cold!)  One thing I wanted was some notion of the player and the Junimos working together to help make the world better.  Here's my script outline:
Cutscene in Forest: (Requires Lewis @ 6 hearts, have flower pot access)
Farmer bumps into Lewis on his way out
Lewis - "Ah, you caught me"
p ?
"Some people say they come to the deep woods to try and connect with the world or something."
"Me, well, I guess this is a place I go to kick myself for things I should have done and said but didn't..."
"Being a bachelor at my age isn't the only way things could have turned out for me..."
"I'm not unhappy, but sometimes...  well..."
p smile
"It was nice unloading a little bit; I hope you don't mind.  I should get back to town."

<junimo appears at left bleeps, hops, bleeps>
[ "I think the Junimo wants to help.  but it needs a plant pot and a portal."]
+ Quest to bring stuff to woods
Cutscene in Forest (#2):  Have seen previous cutscene and have a plant-pot and portal in the inventory
Player puts plant pot and portal down,
Junimo dorks with it, magic happens, it gets transformed into the plant
player walks over to it and picks it up
+ Quest to bring plant to lewis and optionally visit Evelyn's house
+ new item - the "Mightabeen Rose", described as "A plant with flowers with a magical aroma."
Cutscene in the Mullner's house (Requires Maghtabeen Rose in inventory)(optional)
Enter Evelyn's house,
player does question mark
holds plant above head
Evelyn asks player to bring it "over here where the light is better"
player places it Evelyn looks back and forth over it.
"I've only seen one of these once, long ago.  It was always a happy, beautiful plant that my Grandfather grew in his study.  Whenever he fell into a funk, he'd go tend to his plants and emerge feeling better.  When he passed, my mother tried to take care of it, but it just slowly withered away.  It missed him too I guess!"
"I researched it as a young woman, and it turns out these plants are rare, notoriously hard to cultivate and seem to be a little magical.  The smell of the flower helps people to remember that what might have been isn't necessarily better than the here and now."
"But honestly, I don't feel anything at all from smelling it!  But you say you got it as a gift?"
"For Lewis?"
"Oh..."
"You know Lewis was quite the cassanova in his youth.  He made time with all the girls."
"There was one girl that I think was special to him, but she...  Well she wasn't the sort who wanted to worry about what her man was up to when her back was turned...  At the time, he played like he didn't care.  He was too cool to admit that there was a girl he couldn't get - and kept chasing other girls like it didn't mean anything to him."
"I've often wondered if she was the reason he never married.  But just between you and me, I think it turned out for the best for both of them."
Giving the plant to Lewis causes the quest to complete and this dialog:
"For me?  How considerate!  I do love to garden!"
"Mmm...  The flower's scent it...  it... brings back old memories... and some fresh ones too..."
"Yes!  I do think I will enjoy this plant.  I sure hope I don't kill it!"
Cutscene #4 (after completing the quest)

"Thanks for delivering the plant!"
"I can help you with indoor plants"

When the quest is complete, the plant appears in Lewis' house.


Create redirect pages

We do not serve pages for the main menu items. We should redirect to the first page of each section at least.
Github pages doesn't provide any HTTP server configuration, so we can't configure 30x HTTP responses. The options are either to use HTML redirects or JS.

Is this still relevant?

It's solved, mostly. index.html redirects don't seem to work correctly, will work on that.


Add missing syntax highlight in README

This is very minor but I noticed it while reading through the project README.

Thanks


[aerostack2] Create a utilities package

Feature Description
Packages to be include:

[ ] as2_external_object_to_tf
[ ] geozones - #388

Implementation Considerations
No response

@miferco97 is geozones package to be included? I believe it's going to be integrated in knowledge graph package soon.

Yes it is going to be included, the KG implementation may use it not integrate it


⚠ ClamAV has degraded performance

In 6d6d0a1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 244 ms

Resolved: ClamAV performance has improved in a5a5b39 after 5 minutes.


Add median servings for the rest of the fruits

Hi there,
I added the servings for the fruits where it was missing. I used a food database you can find here: https://www.yazio.com/en/foods. It's pretty convinient. For some fruits I had to switch to the german database - there are more entries.
Hope you like it, let me know if you want me to change anything.

Sure @martincollignon - I was thinking exactly the same thing, but didn't know if you allowed different conversions. I'll fix it and also the typos.

You're welcome, your work is amazing!


Merge upstream from dilruacs repo

Seems to be working, but needs more testing and I have found some bugs which will try to fix later.
Just concept for now :D

There are a lot of cool changes in here. If you can get this into a working state, it might be worth the effort to get this integrated into HACS


Add 'TrustedWebActivityIntentBuilder.setShowTitle'

TrustedWebActivityIntentBuilder already allows toolbar color customisation via setToolbarColor.
Would it be possible to also configure if the title should be displayed in the toolbar?
https://developer.android.com/reference/androidx/browser/customtabs/CustomTabsIntent.Builder#setShowTitle(boolean)
Describe the solution you'd like
I would imagine simply adding
    @NonNull
    public TrustedWebActivityIntentBuilder setShowTitle(boolean showTitle) {
        mIntentBuilder.setShowTitle(showTitle);
        return this;
    }

into TrustedWebActivityIntentBuilder would allow it?

We can look into doing this, but why would we want it?
The title wouldn't be shown while on the TWA's domain (since the entire top bar is hidden then), it would only be applicable when the user navigates off origin.

Yes, it would be useful for the scenario when we do not setup Digital Asset Link or if we fallback to custom tab and we want to display the title in addition to url.
I guess it shouldn't be difficult to support given that color customisation via setToolbarColor is already exists for the same scenario?

If you're not going to set up Digital Asset Links, then why not just launch the website as a Custom Tab which already has setShowTitle?

It seemed easier to do it via "android-browser-helper" (as shown in twa-webview-fallback demo) so that we can re-use implemented logic (for service connection and fallbacks).
I looked at the customtabs demo and looks like it would require more work for a simple use case to display external webpage?

Custom Tabs should definitely be easier to use than Trusted Web Activities, although yeah, we've not put as much effort into making convenience methods/templates. The Custom Tabs demo is pretty complex (I think it covers each and every feature in Custom Tabs).
So you just want a simple app that starts up and launches a Custom Tab to a web page?

@andreban for thoughts on making demos/documentation for Custom Tabs easier.

So you just want a simple app that starts up and launches a Custom Tab to a web page?

Yes.
Is there a similar component like com.google.androidbrowserhelper.trusted.LauncherActivity to launch Custom Tab and with logic to have WebView fallback?
If we decide to use Trusted Web Activity that allows Digital Asset Link, it could still be nice to expose customisation methods to display toolbar correctly when we have to fallback to custom tab?

The main reason I ask is that something being launched as a TWA but having the Digital Asset Links fail to verify counts as a failure, and at some point Chrome may change its behaviour to be less forgiving when that happens. (It will always be fine for a TWA to navigate off the verified origin and show the CCT UI, but failing verification is a failure.)
Having a setShowTitle method on TrustedWebActivityIntentBuilder wouldn't be too difficult, but it would take at least two months to roll out through the AndroidX release process and I'd want evidence of a wider need for that method.
Creating a CustomTabsLauncherActivity in Android Browser Helper has a less high bar and I'd be happy to do that if I get some spare time (@andreban in case you have some). It looks relatively simple, doing something like:

TwaProviderPicker.Action action =
    TwaProviderPicker.pickProvider(context.getPackageManager);
if (action.launchMode == LaunchMode.BROWSER) {
  // WebView fallback
  // Assuming that `this` is an Activity.
  startActivity(WebViewFallbackActivity.createLaunchIntent(
    this, url, LauncherActivityMetadata.parse(this)));
} else {
  // Launch Custom Tab
  CustomTabsIntent.Builder builder = new CustomTabsIntent.Builder();
  CustomTabsIntent customTabsIntent = builder.build();
  customTabsIntent.launchUrl(this, Uri.parse(url));
}

You can extend the existing LauncherActivity and provide your own fallback strategy for the Trusted Web Activity:
public class CustomLauncherActivity extends LauncherActivity {
    @Override
    protected TwaLauncher.FallbackStrategy getFallbackStrategy() {
        return (context, twaBuilder, providerPackage, completionCallback) -> {
            CustomTabsIntent intent = new CustomTabsIntent.Builder()
                    .setShowTitle(false)
                    .build();

            if (providerPackage != null) {
                intent.intent.setPackage(providerPackage);
            }

            intent.launchUrl(context, twaBuilder.getUri());
            if (completionCallback != null) {
                completionCallback.run();
            }
        };
    }
}


Chrisda to Master

Docs Build status updates of commit fa34fa1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-ImapConnectivity.md
:white_check_mark:Succeeded
View (exchange-ps)

skype/skype-ps/skype/Get-CsTeamsNetworkRoamingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsNetworkRoamingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsNetworkRoamingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsNetworkRoamingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Numbers cannot be defined using unit and scale aliases

As a user
I want to define a Number using Unit and Scale string aliases instead of an Enums
So I can have flexibility when declaring calculations
Number(unit=Units.liter, scale=Scale.milli, quantity=2)

would become something like
Number('mL', quantity=2)

relates to #16

may consider using a secondary constructor


Update DepthFirstTraversal.java

Input validation has been added for both the dfs method to check if the vertex is within the valid range and the initEdges method to ensure that the number of vertices is greater than 0. If the input is invalid, it throws an IllegalArgumentException with an appropriate error message.

Fixes #{ISSUE_NUMBER}
By submitting this pull request I confirm I've read and complied with the below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[x] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.
[ ] This pull request shall only be reviewed and merged once the Travis build passes. No maintainer or supporter shall be obliged to review it before this condition is met.
[ ] I have mentioned the issue number correctly (with hyperlink) in this pull request description.

@sanvi-05  Please refrain from sending pull requests directly. You must first be assigned to a task, after which your pull request will be reviewed and, if approved, merged.


modify url parameters on POST request

I'm building an app that requires modifying email body. The url.cmml parameter represents the length of body string. If body is changed, url.cmml needs to be updated to new body length, otherwise gmail server will not send the email.
I saw the comments in api.tools.xhr_watcher():
// TODO: recreate the url if we want to support manipulating url args (is there a use case where this would be needed?)
However, url is passed in win.XMLHttpRequest.prototype.open(), while email body is passed in win.XMLHttpRequest.prototype.send(). If I modify the body, I only find out about body in send(), which happens after open().
Any ideas on how to handle this?

Seems this is both solved and old. Closing.