is it possible to replace the optional chain in hmr/hotModuleReplacement.ts with a writeup that is compatible with lower versions?

System Info
/
Details
https://github.com/web-infra-dev/rspack/blob/main/packages/rspack/src/builtin-plugin/css-extract/hmr/hotModuleReplacement.ts#L59
In hotModuleReplacement, this will cause an error to be reported if you are developing with a browser that does not support the Optional Chaining syntax.
Because some of the container versions for hybrid development are still relatively low
The current solution is to configure include in souce: [/[/\]node_modules[/\]@rspack/core[/\]/]
Reproduce link
No response
Reproduce Steps
/

Yes, I can do it when I got time, and It would be very great if you are willing to contribute

Yes, I can do it when I got time, and It would be very great if you are willing to contribute

Then I'll do it.


Delphi 7 compatibility

Please add
{$IF CompilerVersion <= 15} //Delphi 7
type
  TPointerList = array of Pointer;
  TListSortCompareFunc = function (Item1, Item2: Pointer): Integer;
{$IFEND}

to unit Clipper.Core for D7 compatibility. THX!
P.S.
The declaration of TPointerList in Delphi 2007 looks like this:
TPointerList = array[0..MaxListSize - 1] of Pointer;
while the one in Delphi 10.2 is this:
  TPointerList = array of Pointer;

Sorry, we have to dig deeper... That must be right above the class.
{$ifdef FPC}
  TPointerList = array of Pointer;
  TListSortCompareFunc = function (Item1, Item2: Pointer): Integer;
{$else}
{$if CompilerVersion < 23} //since XE2
  TPointerList = array of Pointer;
  TListSortCompareFunc = function (Item1, Item2: Pointer): Integer;
{$ifend}
{$endif}

  TListEx = class
  private

Hopefully finally fixed now 🤞🙏.

Everything runs like clockwork again. Thanks!


🐞 [Bug Report] - Hard coded project name in automation

Describe the bug
The service account for provisioning automation is hard coded here:
https://github.com/GoogleCloudPlatform/avocano/blob/main/provisioning/automation/cloudbuild.yaml#L59
AFAIU, it should instead be
serviceAccount<EMAIL_ADDRESS>
To Reproduce
Follow the https://github.com/GoogleCloudPlatform/avocano/blob/main/provisioning/automation/README.md
gcloud builds submit --config provisioning/automation/cloudbuild.yaml will fail with:
ERROR: (gcloud.builds.submit) INVALID_ARGUMENT: could not generate access token of 'build.service_account': generic::permission_denied: IAM permission denied for service account<EMAIL_ADDRESS>
ci-serviceaccount is created in admin-project-setup.sh, the email is created by GCP and contains PROJECT_ID

Hi @chgans
If memory serves, when this functionality was first added to Cloud Build, you couldn't use dynamic values in that part of the serviceAccount value.
I'd have to confirm this would work before applying the provided patch
Also of note: this automation creates new projects; which is only possible if you have permission to do so, and you are in an organisation. This functionality (probably, last I checked) won't work in an "individual" GCP account. This is part of our testing suite where we create new projects, deploy the app, run tests, then destroy the project, to test all the scripts involved in a "clean room" environment.
Docs and references
https://github.com/GoogleCloudPlatform/avocano/blob/main/provisioning/automation/README.md
https://cloud.google.com/build/docs/securing-builds/configure-user-specified-service-accounts
https://cloud.google.com/build/docs/configuring-builds/substitute-variable-values
https://cloud.google.com/resource-manager/docs/creating-managing-projects#creating_a_project_using_a_service_account

Hi,
I do have an organisation account, with project limit lifted.
I don't understand why admin-project-setup.sh creates ci-serviceaccount in a project but  the automation code requires another ci-serviceaccount.
There is no reference to<EMAIL_ADDRESS>in the docs.
This is confusing.
Do i need to create a dedicated 'avocano-admin' project and create a service account from there? Is this a "global" service account?
occurences of ci-serviceaccount: https://github.com/search?q=repo%3AGoogleCloudPlatform%2Favocano ci-serviceaccount&type=code
When i apply the proposed fix, then project-setup.sh fails on
SA_EMAIL=$(gcloud iam service-accounts list --project ${PARENT_PROJECT} --filter $SA_NAME --format 'value(email)')

Due to lack of permission. As i couldn't figure out how to fix this, i've hard coded this
<EMAIL_ADDRESS>
And now i get more permission issues:
ERROR: (gcloud.builds.submit) INVALID_ARGUMENT: invalid bucket "myproject-1-buildlogs"; builder service account does not have access to the bucket

So looks like i'm going nowhere. Maybe something is missing in the documentation.

There's two issues here, that I've got solutions for in #372
The service account email section of the serviceAccount value has changed since I last visited this code, and now it appears you are not blocked in using a substitution variable in this part of the variable.
The issue you experienced around the bucket would require running the admin-project-setup.sh script. If you configure a user provided service account, you need to provide a separate logging bucket, or disable logging to Cloud Storage and have logs only going to Cloud Logging. https://cloud.google.com/build/docs/securing-builds/configure-user-specified-service-accounts#setting_up_build_logs  You can set logging: CLOUD_LOGGING_ONLY to avoid this.


docs v0.6.0 denote required fields

closes #1809
closes #1931

Coverage increased (+0.06%) to 80.297% when pulling 8abc44a5def6cbbfb5362e989c9e69b1734eca0b on mparke:denote-required-fields into 024c9ddf081fb14cfb88f57f19e445935565201e on linode:master.

Nice.

Coverage remained the same at 80.246% when pulling 5df50aa4aae7e66aa560893c160fd35de421bf4a on mparke:denote-required-fields into 7143b761aa64099d738ad9248fa14e5864c6a8ab on linode:master.

Coverage remained the same at 80.246% when pulling 32f009f9084356c3899488a8af6a7b0b5c7b3391 on mparke:denote-required-fields into 7143b761aa64099d738ad9248fa14e5864c6a8ab on linode:master.

Coverage remained the same at 80.246% when pulling c4763a277e815f661bf23149b447ce087ef78e39 on mparke:denote-required-fields into 7143b761aa64099d738ad9248fa14e5864c6a8ab on linode:master.

Coverage remained the same at 80.246% when pulling 4104f2bf7b50bbeda81022e6e3b1a348fae40317 on mparke:denote-required-fields into 7143b761aa64099d738ad9248fa14e5864c6a8ab on linode:master.

requires https://github.com/linode/manager/pull/2023

Coverage decreased (-0.7%) to 79.525% when pulling d1099a74b7b7fea9a11e917ac761d66cf5cb091e on mparke:denote-required-fields into 7143b761aa64099d738ad9248fa14e5864c6a8ab on linode:master.

Coverage decreased (-0.7%) to 79.525% when pulling d1099a74b7b7fea9a11e917ac761d66cf5cb091e on mparke:denote-required-fields into 7143b761aa64099d738ad9248fa14e5864c6a8ab on linode:master.

Aside from the Example show/hide not being super clear lgtm.

Coverage decreased (-0.06%) to 79.525% when pulling b84aa41a70f2eddaae813b7f0b42f85c2d34e972 on mparke:denote-required-fields into 8c6c6c6a78d332c9fbb9afabb1ac672c5a4315c5 on linode:master.

Coverage decreased (-0.06%) to 79.525% when pulling 3f08161575bea9be6d932d7616006ca195d9cba3 on mparke:denote-required-fields into 8c6c6c6a78d332c9fbb9afabb1ac672c5a4315c5 on linode:master.

show/hide text added, right aligned:

Sub headings unbolded but increased in size, margins increased slightly:

Coverage decreased (-0.06%) to 79.525% when pulling e8bdf1a72e44981c0a42d22640900679cae6b503 on mparke:denote-required-fields into 8c6c6c6a78d332c9fbb9afabb1ac672c5a4315c5 on linode:master.

Coverage remained the same at 79.525% when pulling 4872d4383b99466bb55f5dd933ac2296cce61fea on mparke:denote-required-fields into 088dc8083ccd7d95fe86c01a40f87ceddbe87b13 on linode:master.

Coverage increased (+0.05%) to 79.649% when pulling 13dfa20f3340d4713c746805a85d68e4825ad4ac on mparke:denote-required-fields into 50c995c3f4c745d24a442b1992d7f17f94df2907 on linode:master.

Coverage remained the same at 79.084% when pulling b299db82f1f9d71788ee4be5762396000db525e7 on mparke:denote-required-fields into 768725a15f5d40b386fc22b13b343063e32d8897 on linode:master.


[sqllab] Help sqllab forget query history

In the sqllab asynchronous polling logic, it looks through the query history and continues to poll if there is a query in the history that is not successful.
This PR makes it look only at the last 6 hours of queries when checking for their states. The rationale is that a query that was started 6 hours ago is probably dead or problematic and should be rerun.
@graceguo-supercat @mistercrunch

I've been meaning to do something like this. I wonder if the state should be altered though, like setting the state to failed or timeout. Also build is failing...

I agree with @mistercrunch it would be great to change the state in the queries table for clarity and auditing purposes. Note this would actually be a backend change as opposed to a frontend change.

Okay, I will change the logic to set all pengind or running queries more than 6 hours old to failed.

Codecov Report

Merging #4833 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4833   +/-   ##
=======================================
  Coverage   76.91%   76.91%
=======================================
  Files          44       44
  Lines        8522     8522
=======================================
  Hits         6555     6555
  Misses       1967     1967

Continue to review full report at Codecov.

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

@timifasubaa I wonder if timedout is more descriptive that failed since the later can mean multiple things.

There is a lot of repeated logic with stop_query for the new timedout query logic.

I changed the logic to update the db's sate to timeout before it responds to polls.

@john-bodley @mistercrunch @graceguo-supercat ready for review

PING
Quick update on the state of the logic.
What is really happening is that the frontend ceases to poll for queries older than 6 hours old.
This is because on polls, it checks for all queries by that user whose state has changed after the last updated time. Meaning if a query is on pending for a very long time (> 6hours) and another query that's running at the same time changes state. We will no longer poll and also no longer receive any more updates.
Is this backend code worth having? Especially since it may trick people into believing we actually timeout the queries when the query will most likely not be updated to timeout.

current auto query refresh component work like this:

in client-side, from all queries in local storage, it find out

status in 'running', 'started', 'pending', 'fetching', and
started in < 6 hours

if has any queries, polling with latest updated timestamp

if there is no such queries, stop polling.

in server-side, for every poll request, we get all queries

by user_id, and
changed after updated timestamp

if some query went wrong, and query status kept as running but never got to updated after 6 hours, when client-side polling with a newer timestamp (< 6 hours), this query will not have a chance to be set as timeout state. But it won't cause client-side keep polling because now client-side added timeout threshold.

this PR LGTM. but will not merge until @mistercrunch feel comfortable. Thank you!

PING @mistercrunch @john-bodley


Using git to generate an automatic version number

###_MASTER TEMPLATE: Use for bugs, tasks, and future requests. Also, label accordingly, using the labels on the right.
###_Issue Type - is it a bug, a task, or a future request?:
Future
###_Severity (1-4; 1 - not severe, 4 - very severe):
1
###_Priority (1-4; 1 - lowest priority, 4 - highest priority):
1
###_Synopsis - describe bug/task/future request, in more detail:
When people view their existing instance, they may not know if they are working with the version of the code they are expecting.  It appears that developers have used git to generate automated version numbers.  There are several sites that show how they have done this, here is one:
https://cd34.com/blog/programming/using-git-to-generate-an-automatic-version-number/
###_Expected Behavior - what the behavior is supposed to be, for bug/task/FR:
###_Actual Behavior - what the behavior is instead, for bugs only:
###_Steps to Repro the Issue, for bugs only:
###_Other Notes, for bug/task/future requests:

CLOSING THIS ISSUE SINCE PRIOR QUARTERS WORK AT DATE POINT OF 2019_05_23


State not highlighted on the map when site is opened initially or after auto refresh

Describe the bug
By default state with recent activity is shown but the same is not highlighted on India Map
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.covid19india.org/' or wait for auto refresh

See error

Expected behavior
State should be highlighted
screenshot

If you consider this issue as valid, please assign it to me.

Didn't try to reproduce the error. But assigning to you

happening due to mouse-leave event, not a valid bug hence closing.


text-youtube is updated to new youtube red

missed text-youtube, adding it

release notessss

@emilybrick addedddddd


Still rewrites authorized_keys when builtin SSH server is enabled

Hi,
When I enable the builtin SSH server of Gogs it still writes to my ~/.ssh/authorized_keys file. Perhaps it is a good idea to disable writing authorized_keys.
Also when disabling the START_SSH_SERVER option it should rewrite the authorized keys in that case.
Kind regards,
Rik

It is now fixed in develop branch, please help test.

Close as fixed.


docs: fix feature snippet

Fixes #29533

Internal previews

📄 File
🔗 Preview link

aspnetcore/fundamentals/use-http-context.md
Use HttpContext in ASP.NET Core

@timdeschryver  Thanks for the contribution. Be sure to check out your name as a contributor about the 3rd of next month on the What's new page in the Community contributors section at the bottom.
Awesome work again.


sys/cpu: add Int8 matrix multiplication instructions CPU feature flag for ARM64

References:

https://github.com/torvalds/linux/blob/5bbd9b249880dba032bffa002dd9cd12cd5af09c/arch/arm64/include/uapi/asm/hwcap.h#L75C9-L75C31
https://developer.arm.com/documentation/ddi0601/2024-03/AArch64-Registers/ID-AA64ISAR1-EL1--AArch64-Instruction-Set-Attribute-Register-1

Pull request created against https://go.googlesource.com/sys instead


Expose diagnostics rolemap in Solver

As discussed in #30, I'm opening this PR as a draft so we can discuss the details and figure out the best solution. My first impression was that it would be simple to expose such a feature but it turns out quite a lot of changes are required.
Here I tried to preserve the module types definided in S, to avoid breaking user code that depends on those and therefore added everything I needed to the functor output signature directly. I'm happy to change that if you prefer.
The main problem I have here is that it seems that I need to expose quite a lot of internals to actually be able to get the RoleMap.t. I went for the quick and dirty solution here, just to see if I could interpret the result in opam-monorepo so I'm definitely willing to change all this as long as I can still properly interpret the rolemap in the end. Problem is, a lot of types and modules from Zeroinstall_solver functors are involved and trying to redefine the right subset here seems like a bad idea and something quite painful to achieve.
The current state of this PR allows me to interpret the rejections but the impl and role types, which I would also need are still abstract. Given how they are defined in Model, I don't think we should expose them. Exposing functions to convert them to OpamPackage.t or any relevant type in the opam library seems like the way to go but I'm not quite sure how to best do this. Do you have any suggestion on that front?
Please let me know if I missed something obvious that could simlify all this. Maybe the right solution here would to convert the 0install rolemap into something specific to opam-0install. There's quite a lot of indirection in both opam-0install and 0install and it's been quite the challenge to follow where to find the type and modules signatures. Converting to an opam-0install specific data structure could probably help solving this.

This PR looks reasonable to me. I'm not sure what you want to do with the results though - could you give an example? It might be easier to put the logic directly into opam-0install, if it's generally useful.

The first one is very specific to opam-monorepo and I can implement it on top of the current state of this PR: In opam-monorepo we use a custom Opam_0install.S.Context that, among other things, allows us to reject implementations that don't build with dune. Whenever the solver fails because some of the dependencies can't be satisified because they don't build with dune, we can detect it and suggest solutions to the user.
The other use cases are not as well defined but in general it revolves around improving the error messages when the solver can't find a solution. We might be able to upstream some of this into opam-0install but having the ability to experiment in opam-monorepo would be useful, both to allow improvements specific to opam-monorepo and to be able to iterate quickly so we can eventually upstream the good stuff back here!

I just added two functions to the Model functor, following the version function style to convert from opam-0install model to opam values.
I need to give this a quick spin on the opam-monorepo side but I think it puts this PR in a good spot. If that works out, I'll undraft!

Turns out it wasn't enough and I needed to re-expose them in the Solver functor output.
I think this has everything I need at least for a first iteration.
In the long run it might be a better solution to expose an opam specific data structure tailored specifically for opam-0install, referencing directly the opam types rather than using the possibly over generic 0install types.
One thing we could do here is mark those newly exposed modules and functions as experimental, hiding them from the generated doc until we have a better idea what the ideal API should be.
Another solution is that we maintain this as part of a fork that we'd vendor in opam-monorepo but that requires a bit more maintenance to keep it up to date with upstream opam-0install so if those changes seem reasonable to you, I'd prefer merging it here even if we have to break it later.
Let me know what you think!

I wonder if we could port this change to opam-0install-cudf and use it directly in opam to remove the dependency to dose3 entirely


[FEATURE] Loss as metric

🚀 Feature
Add a separate metric that calculates the loss only.
Motivation
Generate can take a long time, but is needed for current metrics.
Having a quick option with loss can be useful.
Issue is that validation insights cannot be generated in that case though.

Or maybe the best workaround is reducing the max inference length heavily? wdyt @pascal-pfeiffer

I would be fine with adding it as a metric. Or perplexity


Use Kover for Coverage report instead of JaCoCo

close #93
Changes

Use Kover for Coverage report instead of JaCoCo

Codecov Report

Merging #94 (125f5fd) into main (191de9b) will increase coverage by 4.51%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main      #94      +/-   ##
============================================
+ Coverage     27.70%   32.22%   +4.51%
============================================
  Files            47       48       +1
  Lines           628      602      -26
  Branches         87      117      +30
============================================
+ Hits            174      194      +20
+ Misses          453      400      -53
- Partials          1        8       +7

Impacted Files
Coverage Δ

...tfandkusu/template/viewmodel/MutableLiveDataExt.kt
25.00% <0.00%> (-75.00%)
:arrow_down:

...tfandkusu/template/usecase/home/HomeLoadUseCase.kt
33.33% <0.00%> (-66.67%)
:arrow_down:

...u/template/data/repository/GithubRepoRepository.kt
50.00% <0.00%> (-50.00%)
:arrow_down:

...com/tfandkusu/template/viewmodel/home/HomeEvent.kt
57.14% <0.00%> (-42.86%)
:arrow_down:

...emplate/viewmodel/error/ApiErrorViewModelHelper.kt
85.71% <0.00%> (-14.29%)
:arrow_down:

...in/java/com/tfandkusu/template/model/GithubRepo.kt
87.50% <0.00%> (-12.50%)
:arrow_down:

...dkusu/template/viewmodel/home/HomeViewModelImpl.kt
86.20% <0.00%> (-10.23%)
:arrow_down:

...emplate/api/response/GithubRepoListResponseItem.kt
60.00% <0.00%> (-10.00%)
:arrow_down:

...kusu/template/data/remote/GithubRemoteDataStore.kt
91.66% <0.00%> (-8.34%)
:arrow_down:

...ava/com/tfandkusu/template/view/main/AppContent.kt
0.00% <0.00%> (ø)

... and 18 more

Continue to review full report at Codecov.

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

koverReport  task includes testDebug and testRelease
It needs run unit tests for only debug build.


Can't show team shots

Hi @tylergaw, thanks for updating it to v2. I am wondering if you could help me set it up for team shots?

Hey @srizon Unfortunately that's no longer possible with Dribbble API v2. They no longer allow getting any team information. The full list of what they support is in their docs https://developer.dribbble.com/v2/

That's so sad. I know you had some discussions recently with dribbble api team. Do you know if they have plans to support it near future or removed permanently?
Recently we've created a team website and showing team portfolio from dribbble through jribbble. It was working just fine until api v2 broke the page :(

Yeah, it's a bummer. It doesn't sound like they plan to support it ever again. But, I'd suggest letting them know at https://twitter.com/dribbbleapi. I've tried contacting them via email/contact form and not got any response. They responded to tweets thought. They also had this survey https://dribbble-feedback.typeform.com/to/hliwT7 asking question about api usage. Might be worth it to do that too.

Thanks, man! dribbbleapi just replied to my tweet and they are, it's possible. I'll give it a try with team auth and let you know if that works. https://twitter.com/MamunSrizon/status/985897224457486337

Oh excellent! OK, yeah if it works out, I'll get that documented in the readme here.

Cool! Thank you again. I'll give it a try this weekend and will get back to you if I need any help :) Cheers!

Hi @tylergaw, it's me again. Would you mind checking this and let me know what am I doing wrong or any steps that I might have missed? http://creativealiens.com/demo/jribbblev3/
Thanks!

Hey @srizon yeah, I see it. In the console there's an error:
Uncaught TypeError: Cannot set property 'innerHTML' of null
    at (index):16
    at XMLHttpRequest.<anonymous> (jribbble.min.js:2)

That's saying that the script is trying to set the innerHTML of an element, but that element exist.
In the JS snippet:
document.querySelector(".dribbble-shots-list").innerHTML

querySelector is looking for an element with a class of "dribbble-shots-list". Since it doesn't find one, it returns null.
If you add an element to the body with that class like

, it should start working.
Give that a try.

Thanks, @tylergaw! it worked!

Excellent!


error: Expected ], got as at https://deno.land/std@0.150.0/flags/mod.ts:72:20

Trying to install fresh for the first time. I'm following the docs which ask me to run deno run -A -r https://fresh.deno.dev my-project. I get an error. Here is the output of the command:
➜  myfolder deno run -A -r https://fresh.deno.dev my-project
Download https://fresh.deno.dev/
Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>error: Expected ], got as at https://deno.land/std@0.150.0/flags/mod.ts:72:20

Some relevant info:
➜  myfolder deno --version
deno 1.4.2
v8 8.7.75
typescript 4.0.3
➜  myfolder uname -a
Linux MSI 4.19.104-microsoft-standard #1 SMP Wed Feb 19 06:37:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Running on ubuntu wsl2
Thanks!

I think it's your Deno version issue, you should have Deno 1.25.0 or later installed.

Deno 1.4.2 was released in september of 2020 (it's been two years). I think updating it would fix this issue.

This issue was solved i guess so it can be closed😄


Buttons

Buttons are hitting the API soon, and will offer new methods of interacting with messages.
We will need to support the initial components; ActionRow and Button (Potentially the dropdown menu as well depending on how quickly I get this done 😅 ).
Each will require a data class for receiving interactions, and a builder for sending them with messages.
Ideally we will also have some form of resuable "component collection", but this is still up in the air and an initial implementation can be shipped without it.

I see this has been moved to Done, could you link to the relevant documentation please?

https://drb.shardlab.dev/main/Discordrb/EventContainer.html#button-instance_method
https://drb.shardlab.dev/main/Discordrb/Events/ButtonEvent.html
And message sending methods have access to a yielded View, documentation for which can be found here https://drb.shardlab.dev/main/Discordrb/Webhooks/View.html
and examples here https://github.com/shardlab/discordrb/blob/main/examples/slash_commands.rb#L72-L86
https://github.com/shardlab/discordrb/blob/main/examples/slash_commands.rb#L97-L106

it looks like this code isn't in the gem on rubygems, what's the best way to install the gem with this functionality? Should I install directly from the github main branch or is that not considered stable?

If you want to run the latest code, we recommend using bundler with the following line in the Gemfile:
gem 'discordrb', github: 'shardlab/discordrb', branch: 'main'
Since the gem is so outdated, we recommend this until we have a release ready to go. There are bugs you can encounter with the latest gem since there have been updates on discords side but have been fixed on main.
Also be sure to check out the discord for the most up to date information and discussions


IT Support Contacts Report Fails to Load

Describe the bug
When user selects "Contact Name" for Report_rptContactsITSupport when launching via Form_fsubNavigationSubReports, only the names of the customer accounts loads with no other information.
To Reproduce
Steps to reproduce the behavior:

Log in as a valid user
Navigate to the "Reports" tab
Select the "IT Support Contacts by Customer" report
Select the "Contact Name" sort option and run the report
See error

Expected behavior
All columns present in the returned query should be visible.

as of v1.3.5


Secure routable controller methods

Description
Fix for https://github.com/codeigniter4/CodeIgniter4/issues/1849 - changes system/Controller.php methods to protected and adds a 404 route for initController method
Checklist:

[X] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[X] Conforms to style guide

Could you check out the breaking tests? I think this is the first place I saw those erroring out.

Error 1 of 3 is caused by CLI's routes attempting to parse callbacks as a string - see https://github.com/codeigniter4/CodeIgniter4/pull/1847#issuecomment-475621698.
This will be fixed by https://github.com/codeigniter4/CodeIgniter4/pull/1859

Errors 2 & 3 are both from system tests trying to call the now-protected methods directly. I believe this can be resolved by rewriting the test to use getPrivateMethodInvoker. The earliest I could look at doing this would be late next week, but I think it is a relatively easy change.
Relevant Travis CI logs:
2) CodeIgniter\ControllerTest::testCachePage
Error: Call to protected method CodeIgniter\Controller::cachePage() from context 'CodeIgniter\ControllerTest'
/home/travis/build/codeigniter4/CodeIgniter4/tests/system/ControllerTest.php:90
3) CodeIgniter\ControllerTest::testValidate
Error: Call to protected method CodeIgniter\Controller::validate() from context 'CodeIgniter\ControllerTest'
/home/travis/build/codeigniter4/CodeIgniter4/tests/system/ControllerTest.php:100

I believe this is ready now - test failures were resolved by using getPrivateMethodInvoker() as expected.

Looks good. Thanks!


NaN errors for the Bayesian evidence with dynesty

Hello.
When trying to find the best-fitting model for radial velocity data I have issues with the error of the Bayesian evidence for some models in wide volume priors. If I fit a linear trend or a linear trend + 1 planet to the data, everything works correctly. But, when I add an additional planet the Bayesian evidence is calculated but the error is NaN. If I considerably shrink the volume prior, the problem dissapears.
I do not know if anyone has found the same problem and has any idea on the solution.

Hi @jordibp99,
Thanks for bringing this up! Could you share a minimum working examples for this problem so I can take a look at it? Doesn't have to be your data, it can be a mock dataset.
Thanks in advance!
Néstor

My code is almost identical to the juliet tutorials:
`directory="/home/jordi/Documents/github/"
priors_linear_2p = {}
Name of the parameters to be fit:
params_linear_2p = ['P_p1','t0_p1','P_p2','t0_p2','mu_HARPS','mu_HPF',
'K_p1', 'ecc_p1', 'omega_p1','K_p2', 'ecc_p2', 'omega_p2','sigma_w_HARPS',
'sigma_w_HPF','rv_intercept',   'rv_slope']
Distributions:
dists_linear_2p = ['uniform','uniform','uniform',
'uniform','uniform','uniform','uniform','uniform','uniform','uniform','uniform',
'uniform', 'uniform','uniform','fixed','uniform']
Hyperparameters
hyperps_linear_2p =  [[0,1000],[2458473.75213848,2459025.63776411],[0,1000],[2458473.75213848,2459025.63776411],
[-10,10], [-10,10],[0,10], [0,1], [0,360],
[0,10], [0,1], [0,360], [0, 10], [0, 10],0,[0, 0.01]]
Populate the priors dictionary:
for param, dist, hyperp in zip(params_linear_2p, dists_linear_2p, hyperps_linear_2p):
priors_linear_2p[param] = {}
priors_linear_2p[param]['distribution'], priors_linear_2p[param]['hyperparameters'] = dist, hyperp
dataset_linear_2p = juliet.load(priors = priors_linear_2p,  rvfilename=directory+'data.txt', out_folder = directory+'rvs_lineartrend_2planet')
results_linear_2p = dataset_linear_2p.fit(n_live_points = 300)`
data.txt
Thanks for the quick answer!

I checked, and it seems this is a nested sampling issue, not a juliet one. Closing.


fix/check grRules for Oxphos reactions in iSM996

@CarlMalina has detected inconsistencies in some grRules related to the oxidative phosphorylation and electron transport chain pathways in the iSM996 model for K. marxianus.  After manual curation and a detailed revision of the genes involved in this pathway for Kmarx in KEGG and Uniprot, these are his findings:

rxnID
old grRule
new grRule

r_0175
( KLMA_20798 or KLMA_90002 or KLMA_90011 or KLMA_80159 or KLMA_80322 or   KLMA_40468 or KLMA_40467 or KLMA_40511 or KLMA_60198 or KLMA_90009 or   KLMA_70307 )
KLMA_90004 and KLMA_90002 and KLMA_90011 and KLMA_80159 and KLMA_40253   and KLMA_20491 and KLMA_20299 and KLMA_40066 and KLMA_40467 and KLMA_80322   and KLMA_40468

r_0176
( KLMA_90003 or KLMA_30267 )
KLMA_10723 and KLMA_90003 and KLMA_30267 and KLMA_70095 and KLMA_40479   and KLMA_50475 and KLMA_10034 and KLMA_10390 and KLMA_30208

r_0177
( KLMA_90010 or KLMA_50597 or KLMA_90009 or KLMA_90012 or KLMA_10024 or   KLMA_60192 or KLMA_70216 or KLMA_20085 or KLMA_10717 or KLMA_40417 )
(KLMA_70140 and KLMA_60435 and KLMA_50237 and KLMA_30187 and KLMA_80022   and KLMA_30431 and KLMA_90010 and KLMA_10765 and KLMA_90012 and KLMA_60151   and KLMA_50351 and KLMA_30306 and KLMA_40511 and KLMA_90009 and KLMA_10227   and KLMA_60137) or (KLMA_70140 and KLMA_60435 and KLMA_50237 and KLMA_30187   and KLMA_80022 and KLMA_30431 and KLMA_90010 and KLMA_10765 and KLMA_90012   and KLMA_60151 and KLMA_50351 and KLMA_30306 and KLMA_40511 and KLMA_90009   and KLMA_10227 and KLMA_80086) or (KLMA_70140 and KLMA_60435 and KLMA_50237   and KLMA_30187 and KLMA_80022 and KLMA_30431 and KLMA_90010 and KLMA_10765   and KLMA_90012 and KLMA_60151 and KLMA_50351 and KLMA_30306 and KLMA_40511   and KLMA_90009 and KLMA_50135 and KLMA_60137) or (KLMA_70140 and KLMA_60435   and KLMA_50237 and KLMA_30187 and KLMA_80022 and KLMA_30431 and KLMA_90010   and KLMA_10765 and KLMA_90012 and KLMA_60151 and KLMA_50351 and KLMA_30306   and KLMA_40511 and KLMA_90009 and KLMA_50135 and KLMA_80086)

r_0175 corresponds to complex IV; r_0176 to complex III; r_0177 to ATP synthase.

As it is shown in the table above, the previous grRules assume that these reactions are catalyzed by isoenzymes with a single subunit; in contrast, the suggested new grRules indicate that these reactions are catalyzed by isoenzymes, each of them consisting on enzyme complexes with several subunits.
This issue might not represent a major problem for phenotype predictions with the iSM996-GEM, however this is a major problem for its enzyme-constrained version (available here), as the enzyme requirements for these three reactions are currently under-representing the enzyme demands of the full respiratory complexes.
A repercussion of this is observed in the fact that the sigma factor its fitted to a value of 0.3 by the GECKO pipeline for eciSM996 model (evidence of this can be found in the log of the automated ecModels pipeline for this model, available here). This represents the average saturation factor for all enzymes for K. marxianus growing on conditions of unlimited glucose, which takes a value around 0.5 for all of the other organisms that have been tested with this pipeline.

@simas232 This is something to think about, regarding the representation of OxPhos in iSM996.

@simas232 This is something to think about, regarding the representation of OxPhos in iSM996.

It sounds to me like this problem would be better solved upstream. ecModels could then, at least for a while, use a specific branch of the source repository with a direct link to the .xml.

It sounds to me like this problem would be better solved upstream. ecModels could then, at least for a while, use a specific branch of the source repository with a direct link to the .xml.

I agree @mihai-sysbio, sounds like the solution should be in the original repo. To accelerate the process I can replicate this issue in the original GEM repo and create a new PR with my suggested solution (already implemented locally).


clean imported entities and update oeo-import-edits

Description of the issue

[x]  Changes in imported entities should be done in oeo-import-edits, but are done anywhere, #1776
[ ]  Some annotations are imported twice, e.g. from BFO and IAO. This has to be fixed
[x] In the hotfix 2.0.1 we changed unit and prefix in uo-extracted: #1742

duplicated SubClassOf
changed definition (unit)
This should be made a permanent change in oeo-import-edits / in the import script in the long run

[x] Clean RO annotations: move OEO-made annotations on RO relations into oeo-import-edits (many of them are stored in oeo-shared)

Ideas of solution
Tidy!
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

[ ]  Clean RO annotations: move OEO-made annotations on RO relations into oeo-import-edits (many of them are stored in oeo-shared)

Taking care of this now.

Seems to be solved.


After upgrade to sys-apps/openerc-0.13.11 softlevel in boot PXE not run

With openrc-0.12.4 softlevel in boot from PXE run fine, but after upgrade run softlevel to none.
See the bug in gentoo:
https://bugs.gentoo.org/show_bug.cgi?id=541176

There is no new information here, so this is being worked on the Gentoo bug.


Docker Jenkins

How to link Jenkins and Maven Containers in Docker ?
command that is working

Link is for services. I dont recommend jenkins & maven to be linked. Maven container with jenkins agent would be my slave

So can we write Dockerfile with maven installation steps in docker image of jenkins


Text field not catching values change

Hi,
I have textfield, when i try to enter values through code there, in observe next i can see only old value.
For example, i have entered "test", then i did :
 self.srchTxtf.text = ""
In observe next, value will be :  "test"

Unfortunately Bond does not support sending an event when you programatically change the value - only when user types something in.


Improve documentation

Let's use this issue to collect all the places in which the documentation should be improved. If you think there's something missing from the list, please comment and we'll update the list.
As we work through this list, we'll update the description with links to PRs addressing each of the items.

[ ] nn.Collection
[ ] Module.apply
[ ] Module.call
[ ] Module should get a docstring
[ ] nn.stateful
[ ] nn.stochastic
[ ] Module.get_param (explain when it should be used)

Some of these documentation tasks may be best done as part some broader rethinking of core APIs, such as in #208

(And assessing how simple it is to write docstrings for new APIs will help make sure the APIs have a clear mental model)

https://github.com/google/flax/blob/master/flax/training/common_utils.py could benefit from docstrings, especially since the methods in it are used in many flax examples. Thanks!

I'll close this issue because it's related to the old flax.nn API and we've now switched to Linen.
@lisa-1010 I filed your comment about common_utils in a separate issue: #729


fix(icons): use same icons from the canvas in Catalog

fixes https://github.com/KaotoIO/kaoto-next/issues/307

Thanks @tplevko, your PR in combination with the new icons make the project look fancier :smile:


LICENSE

Yes, MIT sounds fine, even though it could be public domain as well.
I was unsure about the compability of the MIT license, as the levenshtein function is from php.js but, php.js is also MIT.

Thanks :)


dbt adapters improvement

Describe the feature
Improve "building an adapter" documentation.
The current available docs (https://docs.getdbt.com/docs/contributing/building-a-new-adapter) does not cover some of the the common patterns I found in existing adapters (catalog.py, relation.py). Some of the macro mechanisms are not obvious (what are some parameters passed to macros ? e.g. : what is information_schema, what are macros/method signatures).
Following additions would be nice

the method signatures in the Editing the adapter implementation  section
explaining the relation between macros and SQL logic  (before digging in the dispatch mecanisms)
documenting base macros  signatures in the required macros section (what is information_schema_name, why does it take database, where does it come from)

As there are so many contributed adapters out there, I feel i'm certainly missing something :)
Describe alternatives you've considered
Looking at the code in repositories for spark, presto adapters, as well as code from core adapters
Additional context
No
Who will this benefit?
Any adapter contributor for databases not covered and that could be a dood fit for dbt (e.g. : firefolt, trino )
Are you interested in contributing this feature?
Sure

@moshir I really appreciate (and agree with) this issue! I'm invested in making our resources for adapter authors + maintainers as good as they can possibly be.
One of the challenges here is making sure that any public scripts or documentation stay current with the latest version of dbt. I've got another issue open (https://github.com/fishtown-analytics/dbt/issues/3002) to revisit create_adapter_plugins.py and ensure it stays up to date with new releases. Given that the "Building a New Adapter" docs live in a separate repo, we'll need to be especially good about coordinating and calling out updates, as we would for any other versions-specific feature/changes. We are stabilizing the interfaces here, so I do believe that (e.g.) adapter macro signatures are stable enough to be worth documenting, rather than simply linking to an associated line of code in the dbt repo.
In any case, I'm going to transfer this issue to the docs.getdbt.com repository, since that's where these documentation improvements should happen. I'm supportive of the three areas for improvement you've identified, and I'd be more than happy to review a PR that adds to these sections :)


Sort remote participants according to their priority

Fix #142
Fix #143

Thanks! It feels a bit jumpy, so perhaps we might think about including a timer or some other strategy to delay the update. But that could be a separate ticket :)


Add the ability to change the headers from outside

Given the fact we stream content, some tools like NGinx requires cancellation of buffering for example. Giving more elasticity here.

Note that I did not bump the version but I could if you want me to.

Bumping :)

This is done right?

I did not bump in the end, apparently, would you want me to ?

not necessary I'd say!

just let me know if it's done and I merge


How to reproduce the WER score of Wav2vec-U

❓ Questions and Help
What is your question?
I have run the pipeline of wav2vec-U and trained a GAN on Librispeech 960h. When using Viterbi decoding, I can get the PER of about 20%. But when I use kaldi decoder, I got WER as 40%. Will you release the inference script with an LM so that we can reproduce the  WER score reported in the paper?
Code

The arguments I used for decoding are as follows:
w2l_decoder: KALDI
post_process: silence
blank_weight: 0
sil_is_blank: true
blank_mode: add
unsupervised_tuning: 0
targets: wrd
lexicon: /path/to/text/lexicon_filtered.lst
kaldi_decoder_config:
acoustic_scale: 5
hlg_graph_path: /path/to/text/fst/phn_to_words_sil/HLG.phn.kenlm.wrd.o40003.fst
output_dict: /path/to/text/fst/phn_to_words_sil/kaldi_dict.kenlm.wrd.o40003.txt
What have you tried?
What's your environment?

fairseq Version (e.g., 1.0 or main):
PyTorch Version (e.g., 1.0)
OS (e.g., Linux):
How you installed fairseq (pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

我的uer一直是九十多，降不下来


Increment version for core releases

Increment package version after release of azure-core

/adp run python - core - ci

/azp run python - core - ci

Succeeded in https://dev.azure.com/azure-sdk/public/_build/results?buildId=978107&view=results.

/check-enforcer override


Call to method markAsRead() on an unknown class App\Models\collection.

Larastan Version: 1.0
--level used:
Pull request with failing test:

Description
I'm using  Notifiable for notification that has the markAsRead method but larastan returns the following error
Laravel code where the issue was found
  16     Call to method markAsRead() on an unknown class App\Models\collection.

Solved by Adding above the class name
 * @property-read \Illuminate\Notifications\DatabaseNotificationCollection|\Illuminate\Notifications\DatabaseNotification[] $unreadNotifications

Nice!


[Documentation] Add Table to Display Repository Statistics in README.md

Issue Description:
The issue involves adding a well structured table to display repository statistics, such as the number of stars, forks, issues, and pull requests, directly in the README.md file. This will help contributors and users track the activity and contributions at a glance.
Steps to Implement:
Create a table in the README.md file displaying the following repository statistics:
Stars
Forks
Open Issues
Open Pull Requests
Closed Pull Requests
Use markdown syntax to include badges or icons for each statistic.
Add dynamic links to the corresponding sections of the repository for easy access (e.g., link to the "Issues" or "Pull Requests" tabs).
Expected Behavior:
The README.md file will contain a neatly formatted table that displays real-time repository statistics, making the repository's activity more visible and engaging for contributors and users.
@jindalpriyanshu101 kindly assign me this issue with the labels of gssoc extended,hacktoberfest and level

Sounds good, you can work on it. @KGupta2601

Marked as duplicate, as the issue is already being handled in #125


Update documentation.md for DevCommunity

Description

The changes made in this pull request should be briefly described.
There were minor issues in the documentation that didn't enable the methods of the DevCommunity to be seen as part of a code. This PR will fix the issue.

Resolves: [Issue Number]
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

Screenshots

Uploading a screenshot illustrating the approach you developed for validation is required.
I have attached the before and after pictures of the update.

Additional Notes/Comments
This is an update to the documentation for the DevCommunity module.

Any additional remarks or suggestions concerning this pull request are welcome.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Thank you very much @nikhil25803 .


support ccp4 convention in add_rfree function

It seems useful to make rs.utils.add_rfree function also support ccp4 convention rfree flags.
Can add a boolean argument named ccp4_convention default False. When set to True, change the column name to FreeR_flag and convert value 0 to be test set.

Is there any specific issue should be considered with this new tiny extension? If not, I am happy to make this happen and create a PR.

I think that this seems reasonable, but I will need to look into it a bit to see if there are any gotchas here. I think your suggestion all sounds worthwhile though to ensure we support both CCP4/Phenix formats.

Related PR has been opened, once reviewed and merged, this issue can be closed.


Changes to the configuration

Changed the configuration of the Avatax templates from SyliusAdminBundle:Crud to "@SyliusAdmin\Crud"

Thanks @Nana-asiamah98


remove free plan injection on experiment marquee

disabling free-plan-tag for experiment page by removing the injection code in the block
Resolves: MWPW-NUMBER
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/?experiment=ccx0167%2Fcontrol
After: https://marquee-exp-fix--express--adobecom.hlx.page/express/?experiment=ccx0167%2Fchallenger-1

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

Comparison is base (531ed30) 33.86% compared to head (c3ce3ff) 33.83%.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #654      +/-   ##
==========================================
- Coverage   33.86%   33.83%   -0.04%
==========================================
  Files         129      129
  Lines       24893    24887       -6
==========================================
- Hits         8431     8421      -10
- Misses      16462    16466       +4

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


chore Porting to Clang-16

Proposed changes
Issue Number: close #xxx
Problem summary
This PR ports the codebase to Clang-16.
Upgrade some third-party libraries:

Apache BRPC: 1.2.0 -> 1.4.0  (Some bugs are fixed and all patches for 1.2.0 can be removed.)
Boost: 1.73.0 -> 1.81.0  (Porting to Clang-16)
libclucene: 2.4.6 -> 2.4.8  (Porting to Clang-16)

Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[x] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run p0


[BUG] make tre-destroy fails as can't delete diagnostics firewall and route table

Describe the bug
After deploying the TRE, using make TRE-destroy fails to destroy the diagnostics firewall and route table resources
│ Error: deleting Monitor Diagnostics Setting "diagnostics-firewall-jjgriff93" for Resource "subscriptions/b9ea69a5-4c3f-49e1-9fea-30145415316f/resourceGroups/rg-jjgriff93/providers/Microsoft.Network/azureFirewalls/fw-jjgriff93": insights.DiagnosticSettingsClient#Delete: Failure responding to request: StatusCode=409 -- Original Error: autorest/azure: Service returned an error. Status=409 Code="ScopeLocked" Message="The scope '/subscriptions/b9ea69a5-4c3f-49e1-9fea-30145415316f/resourceGroups/rg-jjgriff93/providers/Microsoft.Network/azureFirewalls/fw-jjgriff93/providers/Microsoft.Insights/diagnosticSettings/diagnostics-firewall-jjgriff93' cannot perform delete operation because following scope(s) are locked: '/subscriptions/b9ea69a5-4c3f-49e1-9fea-30145415316f/resourceGroups/rg-jjgriff93/providers/Microsoft.Network/azureFirewalls/fw-jjgriff93'. Please remove the lock and try again."
│
│
╵
╷
│ Error: deleting Route Table "rt-jjgriff93" (Resource Group "rg-jjgriff93"): network.RouteTablesClient#Delete: Failure sending request: StatusCode=400 -- Original Error: Code="InUseRouteTableCannotBeDeleted" Message="Route table rt-jjgriff93 is in use and cannot be deleted." Details=[]

Steps to reproduce

Deploy AzureTRE by following instructions
make tre-destroy

Should be covered as part of #510.


Feature Request: Multiple Windows For Desktop Applications.

This would be absolutely huge. I desperately want to use Kivy to build a number of applications but this hold back is the one and only reason I can't do it. There would be no reason to use another GUI framework.

This has come up before - I believe it's likely to be non-trivial, as Kivy's internals were written to not worry about multiple window states, I think as a deliberate choice when it emerged that full multi-window support (with multiple opengl contexts etc) would be hard. A PR to add this feature or work towards it would probably be welcome, but for these reasons I don't think any core dev is likely to look at it soon.

I'd like to add that you can do pseudo-multiwindow with kivy in three ways(app = file with <main class>().run()):

Single app, one space - widgets as forms("windows")

Single app divided into admin & client parts launched multiple times where admin would probably handle communication

One+ app (launching forms from running app):

Running another interpreter instance via subprocess (python another.py or similar)
Running an app via multiprocessing (if crashes/freezes, probably all will go down)

However it's not that hard to have multiple windows with kivy as it is with those steps, rather the communication is (or at least for me) kind of problematic and non-elegant (read ugly).
Also, I don't want it to sound rude, but what is that special snowflake use case that needs multiple windows?

You're right, opening a second window isn't terribly difficult. You can do it in about 12 lines of code with subprocess. You can even handle killing it with subprocess. The issue is passing data back and forth between windows.
The scenario is multiple displays. I need to display a window on screen two that takes in data from the client portal on screen one. Then, the data needs to be refreshed as the client sees fit. It's not massively complicated but passing data back and forth is, you're right, ugly.

That's why I mentioned that bad bad communication :yum: What you may try is using a local server between opened windows. I've had this idea in mind since I've seen (now it's called) remi, which is nicer than using e.g. files or accessing the app with some other ugly way. This way you may write app that autoconnects & checks something on_start()
Or... look at this Q&A that is about multiprocessing and sharing stuff (object there), but the deeper you go, the worse it'll be. Also, the more users will use those client-like apps, the better seems to me to use a local server and with your main app only manipulate data in that. If I remember correctly, Kivy supports Twisted, but it may be an overkill too, so you can look at remi's server.py

@KeyWeeUsr I don't think it's particularly unreasonable to need multiple windows. It's easy to come across places where you want one app across e.g. two monitors, in which case one opengl context isn't a great fit. You can always hack something together, but from a user perspective it would certainly be much easier if kivy did the abstraction internally.

I'm not saying it's unreasonable in general, I'm trying to say that there is thin line between use cases of multiple forms(one app handles all forms) and using two separate apps(windows) with even server to manage the communication. I'm mentioning it because it'd probably do more harm than provide benefits and it'd probably be great time-consumer with rebuilding kivy. However, I don't say I'm right about the cost of this feature and maybe it's easier than I expect it to be now.

Thanks for the suggestions and the help. I ultimately did get this up and running. There are still some issues to deploying a work around like this but alls well that ends well. Here's a write up I did on the methods used.

I'm glad you found a way with Kivy. This could be a very good contribution to the wiki if you have some working example. :yum:

That'd be great. The project is coming right along. As soon as I have it ready for release I'm all for it.

I think this is never going to happen in the future in Kivy.
This is very difficult to achieve, by the fact we are in OpenGL and there is only one global context that we need to switch when you want to draw on one windows or another.
Unless there is a considerable effort from the community or a core dev, best is to open multiple windows as separate process, and use IPC to communicate.

An alternative for passing data between windows is to use Tendo's Singleton pattern. I haven't used it for multiple windows, but it should work pretty much the same. In my case, I use a single implementation of the Singleton pattern, with an added callback parameter to __init__ which is executed it in place of the logging.error("Another instance is already running, quitting.") instances.  This allows subsequent calls from the command line to forward arguments to the initial instance of the app.
Here's a simplified version of my implementation.
If you need more flexibility from the passed arguments, you can implement argparse or maybe send JSON strings as arguments.

I realize this is old and closed, and also that I'm only one person, but for what it's worth, I agree this would be a huge deal. I'm just getting started trying out Kivy, and I'm really liking it so far (it's so much more Pythonic and understandable than the Python bindings for Qt), but I'm not looking forward to figuring out the workarounds when it comes time to try and make multiple windows/processes communicate with each other.
If I get to the point of learning and understand Kivy's internals, I'll absolutely see what I can do about contributing toward this. For now, this is just a hopeful +1, and an expression of appreciation for all you've done so far.


Migration to Samvera: Challenges to Making the Move and What the Community Can Do To Welcome New Users

In this panel we will briefly discuss the current landscape of needs and reasons for users to consider moving from an established repository, and the challenges facing users of a variety of platforms, both cultural and technological.  We will also consider work currently underway such as "Bridge to Hyku", a grant-sponsored project empowering Content DM users to migrate, successes in DSpace-to-Samvera migration and what's on the horizon for BePress.  In discussing these challenges, we hope to present the Samvera Community with an opportunity to grow the portofilo of users and create the potential for standards and teams to assist those who wish to be a part of the Samvera Community
Ryan Steans<EMAIL_ADDRESS>Nabeela Jaffer (njaffer@umich.edu)/
Todd Crocken (tncrocke@central.uh.edu)/ Crystal Richardson (crystal@notch8.com)/ Jose Blanco<EMAIL_ADDRESS>

Speakers confirmed and recording OK - from Ryan 8/10

Todd registered 8/14 - system info

Crystal, Jose and Nabeela nudged 8/23

Nabeela registered 8/28 - system info

Jose registered 8/28 - system info


Hive connector can return maps with null keys

Null map keys are not supported in Presto and cannot be returned from the StatementResource.

This seems to be a regression in MapBlockReader: entries with null keys used to be discarded (this was lost in the memory representation change).


Binding instances aren't properly initialized after Fragment recreation

I'm using 2.3.0 but this issue was present in earlier versions as well.
If I have a binding in the androidSupportFragmentScope, and the Fragment used with that scope has a child fragment, then in that binding calling instance<Fragment>() returns the child Fragment, while calling (it as KodeinInjected).kodein().value.instance<Fragment>() correctly returns the parent Fragment.

The issue was that I was doing something after onCreate in my SupportFragmentInjector that I should have been doing before onCreate (see https://github.com/SalomonBrys/Kodein/pull/52).


increase buffer size

hi. my stack trace is not completed and it trimmed the end of symbol names after 63 chrachter. I think it is rooted here :
https://github.com/ivanarh/ndcrash/blob/8ab2a288a3c918cc1693b0749f8e9f9278e4b9ed/src/ndcrash_dump.c#L102
please consider increasing the buffer size to somthing like 100 or 128.

char proc_cmdline_content[64], proc_comm_content[32];
These 2 arrays are used to store process and thread names respectively.

A limit for function name is here:
https://github.com/ivanarh/ndcrash/blob/8ab2a288a3c918cc1693b0749f8e9f9278e4b9ed/src/ndcrash_private.h#L62

Would 96 bytes be sufficient for you? I don't want to increase this buffer significantly because this code may be run in a signal handler with a very limited stack size.

yeah that is enough, but 96 bytes or 128 byte are not that much to worry about.

BTW, thanks for this great tool. I was using Crashlytics but it does not work! I've spent 3 weeks to fix it but no result! The support guy said they have a limited stack size in their server which leads to symbol missing!!!

Done


Expose open_dir and read_dir from FATFS(or make MOS calls for this)

Currently, we have no possibility read directory from applications(for example, for making file managers, CP/M emulation layer or something like this).
I'd like get FatFS or MOS API for reading directory.

Closing this card as it is a duplicate of #52


Debug messages always on

At first i didn´t noticed because i wanted it and set
i2c.Debug = true
but now i want to deactivate it but the debugmessages are always on even if i set
i2c.Debug false
Can you please tell me if it is a bug or if i missing something.
Thank you

Was missing the actual check before each debug was printed. Added a pull request adding that bool check before every debug print that will fix your issue.
You can just add if v.Debug == true { } before every lg.Debugf()

Hi @rpm91! Sorry for late response. In fact field i2c.Debug has been abandoned and will be removed from code, because this variable have no effect to the current log output logic.
In order to manage verbosity output of d2r2/go-i2c package you should insert in your main() function call
	// Uncomment/comment next lines to suppress/increase verbosity of output
	logger.ChangePackageLogLevel("i2c", logger.InfoLevel)

, because in this case verbosity of package will be switched from logger.DebugLevel to logger.InfoLevel.


Semi colon ; in query

Problem
sqldump.etl.Query replacing ; with empty string while loading queries from file. (see line 60)
This will case the problem for some case for example below query
with temp as ( SELECT 'value1;value2;value3' text FROM  dual )
select regexp_substr(t.text, '[^;]+', 1, levels.column_value) as text from temp t,
	table(cast(multiset(select level from dual connect by level <= length (regexp_replace(t.text, '[^;]+')) + 1) as sys.OdciNumberList)) levels

Possible solution:

Should not replace the ; (ask in documentation/readme to not put semi-colon in file)
Remove only the last ; from query (in the end of file)

Hi there,
thanks for reaching out and many thanks for the solutions. I'd go for the second one since that poses fewer restrictions (and reading the readme requirements) in comparision to the first one.
If you want to submit a PR you are very welcome to do so. I'll update the code as soon as I have time otherwise.
Cheers

I have created pull request. Also closing the issue

cool, thanks-a-lot! merged.


Implement filter logic for facility- Emissions filter logic

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[x] Does this have a dev link been provided in the design?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a custom data download user I want the filter logic to function for facility so that I can avoid producing a query with no results
Acceptance Criteria
Given that a user has selected an emissions subtype
When the functionality is displayed
Then the user is required to apply filter criteria for time period first
AND the optional filters are greyed out until time period criteria is applied
Given that a user has selected an emissions subtype on the custom data download
When filter criteria is applied for time period
Then the filter logic is engaged
AND the facility filter is narrowed based on the selected time period and subsequent selections
Given that the filter logic is engaged
When the facility filter panel functionality is displayed
Then non applicable selections are removed from the combo box
Given that the filter logic is engaged
When filter criteria is applied for facility
Then the optional filters are narrowed based on the applied facility filter criteria
Given that the filter logic is engaged
When the facility filter tag is removed
Then the facility filter is narrowed based on applied criteria
Development Notes
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[x] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Testing
Test Case: TC457
Tester: @JanellC
508  Testing:
Tester: @aprematta
Definition of Done

[x] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[x] Has the code been unit tested and peer reviewed?
[x] Have the functional tests been executed?
[x] Have all defects been reviewed and dispositioned, resolved, or deferred?
[x] Is the user story ready to be deployed to the test/staging environments?

TC457 passed @tlageman-epa. This ticket is ready for 508 testing @aprematta


Support vendoring for Python

From @Kristinita on January 21, 2018 4:6
1. Request
It would be nice, if would be possible upgrade dependencies, not only version numbers in requirements file.
For example, it would be nice add an option, that bot will run command after each version changing in requirements file.
2. Steps to reproduce
I develop plugins for Sublime Text, written on Python. By technical reasons I need add dependencies to my repository.
In my repository for testing I add outdated version of PyPI gsearch module to requirements.txt file. I use command:
pip install -r requirements.txt -t . --upgrade

3. Justification
Automatically dependencies upgrading may save developers time.
I test Dependabot on my test repository. Dependabot only change dependency version in requirements.txt file, but not upgrade my gsearch dependency, that I include to my repository.
I need upgrade my dependency manually; after merging I need run command again:
pip install -r requirements.txt -t . --upgrade

It would be nice, if Dependabot will run this command automatically after each version change in requirements.txt file.
Thanks.
Copied from original issue: dependabot/feedback#76

Thanks for the feedback @Kristinita. This one will require quite a bit of work on our side, given the way we currently update Python files (it's using regexes, not shelling out to Python). I'd like to do it in future, though, so will keep it here as a reminder.

@greysteil is the generic problem here that there is a need for user specified post-run commands? Though I'm guessing there are some security issues with something like that

Not exactly - Dependabot doesn't clone your repo (for security reasons), so it can't currently run user specified post-run commands. They're probably more suited to a CI environment.

Not solved.
Thanks.


 Error: Could not resolve './_physx/engine.js' from src/Physx.js because the /src/_physx directory is not in the repo. Is there a way to get that code? Thanks.

I see that it is the same situation with HAVOK which seems to be a paid product now...

Hi thanks for your interest.
yes for the moment my work on physx and havok and smoke is private.
it is possible to use compile version anyway.
the next version will be more complete, but it's a long research work
I'm thinking of opening everything but i need more Sponsors

@lo-th Have you ever considered just releasing the private version and then starting a Udemy course on how you made all those demos to teach people to use your engine? I'd pay money for a course on how to use your engine so I could use Havok in Three.js.


Develop

Finished Chapters 1,2,3,4,5,6

@dalia-sharra  you sure pushed everything on the repo? There are no guard or http services here

@dalia-sharra you sure pushed everything on the repo? There are no guard or http services here

Yes, sorry I forgot to push all the commits to the branch. I did it now.


KAFKA-7666: KIP-391 Producing with Offsets for Cluster Replication

initial commit
Co-authored-by: Edoardo Comar<EMAIL_ADDRESS>Co-authored-by: Mickael Maison<EMAIL_ADDRESS>first implementation of https://cwiki.apache.org/confluence/display/KAFKA/KIP-391%3A+Allow+Producing+with+Offsets+for+Cluster+Replication
current limitation - a Producer can only be used to send Records with or without offsets, it cannot mix
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest this please

closing as KIP has been retired


not able to encode at 1 fps output rate

Hi,
I am using Openh264 to encode YUV data which is coming at 1 frame per second. I want to save in h264 file and transmit using RTSP with 1 FPS. But instead i am getting 25 fps with ffprobe and 30 fps with elecard stream analyser.
How can I get output at 1 FPS?
following is the default encoder params i set :
memset (&param, 0, sizeof (SEncParamBase));
param.iUsageType = EUsageType::CAMERA_VIDEO_REAL_TIME;
param.fMaxFrameRate = (float)1;
param.iPicWidth = 1280;
param.iPicHeight = 800;
param.iTargetBitrate = 512000;
param.iRCMode = RC_MODES::RC_OFF_MODE;
And the following are the Information logs :
4 - [OpenH264] this = 0x0C1E15A8, Info:CWelsH264SVCEncoder::SetOption():ENCODER_OPTION_TRACE_CALLBACK callback = 74941ED0.
4 - [OpenH264] this = 0x0C1E15A8, Info:CWelsH264SVCEncoder::InitEncoder(), openh264 codec version = 56c6fc9+M
4 - [OpenH264] this = 0x0C1E15A8, Info:iUsageType = 0,iPicWidth= 1280;iPicHeight= 800;iTargetBitrate= 512000;iMaxBitrate= 0;iRCMode= -1;iPaddingFlag= 0;iTemporalLayerNum= 1;iSpatialLayerNum= 1;fFrameRate= 1.000000f;uiIntraPeriod= 0;eSpsPpsIdStrategy = 1;bPrefixNalAddingCtrl = 0;bSimulcastAVC=0;bEnableDenoise= 0;bEnableBackgroundDetection= 1;bEnableSceneChangeDetect = 1;bEnableAdaptiveQuant= 1;bEnableFrameSkip= 1;bEnableLongTermReference= 0;iLtrMarkPeriod= 30;iComplexityMode = 1;iNumRefFrame = 1;iEntropyCodingModeFlag = 0;uiMaxNalSize = 0;iLTRRefNum = 0;iMultipleThreadIdc = 1;iLoopFilterDisableIdc = 0 (offset(alpha/beta): 0,0;iMaxQp = 51;iMinQp = 0)
4 - [OpenH264] this = 0x0C1E15A8, Info:sSpatialLayers[0]: .iVideoWidth= 1280; .iVideoHeight= 800; .fFrameRate= 1.000000f; .iSpatialBitrate= 512000; .iMaxSpatialBitrate= 0; .sSliceArgument.uiSliceMode= 0; .sSliceArgument.iSliceNum= 0; .sSliceArgument.uiSliceSizeConstraint= 1500;uiProfileIdc = 66;uiLevelIdc = 0
4 - [OpenH264] this = 0x0C1E15A8, Info:WELS CPU features/capacities (0x4000763f) detected: 	HTT:      Y, MMX:      Y, MMXEX:    Y, SSE:      Y, SSE2:     Y, SSE3:     Y, SSSE3:    Y, SSE4.1:   Y, SSE4.2:   Y, AVX:      N, FMA:      N, X87-FPU:  Y, 3DNOW:    N, 3DNOWEX:  N, ALTIVEC:  N, CMOV:     Y, MOVBE:    N, AES:      N, NUMBER OF LOGIC PROCESSORS ON CHIP: 8, CPU CACHE LINE SIZE (BYTES):        64
4 - [OpenH264] this = 0x0C1E15A8, Info:WelsInitEncoderExt() exit, overall memory usage: 13539910 bytes
4 - [OpenH264] this = 0x0C1E15A8, Info:CWelsH264SVCEncoder::SetOption():ENCODER_OPTION_DATAFORMAT, m_iCspInternal= 0x17
4 - [OpenH264] this = 0x0C1E15A8, Info:CWelsH264SVCEncoder::SetOption():ENCODER_OPTION_FRAME_RATE,m_pEncContext->pSvcParam->fMaxFrameRate= 1.000000
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 22.000000, fAverageFrameRate=0.000000, LastFrameRate=0.000000, LatestBitRate=0, LastFrameQP=26, uiInputFrameCount=1, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=85703 at Ts = 215493
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 15.000000, fAverageFrameRate=0.020809, LastFrameRate=0.000000, LatestBitRate=0, LastFrameQP=34, uiInputFrameCount=2, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=86013 at Ts = 311603
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 12.666667, fAverageFrameRate=0.017319, LastFrameRate=0.000000, LatestBitRate=1785, LastFrameQP=34, uiInputFrameCount=3, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=86773 at Ts = 388711
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 16.500000, fAverageFrameRate=0.014797, LastFrameRate=0.000000, LatestBitRate=1785, LastFrameQP=30, uiInputFrameCount=4, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=154275 at Ts = 485820
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 16.600000, fAverageFrameRate=0.013534, LastFrameRate=0.000000, LatestBitRate=1785, LastFrameQP=33, uiInputFrameCount=5, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=171529 at Ts = 584930
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 16.333334, fAverageFrameRate=0.012231, LastFrameRate=0.000000, LatestBitRate=2575, LastFrameQP=33, uiInputFrameCount=6, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=188925 at Ts = 706042
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 15.571429, fAverageFrameRate=0.011811, LastFrameRate=0.000000, LatestBitRate=2575, LastFrameQP=34, uiInputFrameCount=7, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=194727 at Ts = 808153
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 15.250001, fAverageFrameRate=0.011432, LastFrameRate=0.000000, LatestBitRate=2575, LastFrameQP=34, uiInputFrameCount=8, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=198339 at Ts = 915263
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 12.785715, fAverageFrameRate=0.019836, LastFrameRate=1.000000, LatestBitRate=49930, LastFrameQP=35, uiInputFrameCount=14, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=219790 at Ts = 921263
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 11.400000, fAverageFrameRate=0.028099, LastFrameRate=1.000000, LatestBitRate=5808, LastFrameQP=35, uiInputFrameCount=20, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=224236 at Ts = 927263
4 - [OpenH264] this = 0x0C1E15A8, Info:EncoderStatistics: SpatialId = 0,1280x800, SpeedInMs: 11.115384, fAverageFrameRate=0.036223, LastFrameRate=1.000000, LatestBitRate=60229, LastFrameQP=35, uiInputFrameCount=26, uiSkippedFrameCount=0, uiResolutionChangeTimes=0, uIDRReqNum=0, uIDRSentNum=1, uLTRSentNum=NA, iTotalEncodedBytes=255927 at Ts = 933263
Thanks in advance.
Regards,
Ashesh

VUI in h264 bitstream contains fps info. But openh264 doesn't write this info in encoder.
ffprobe and elecard stream eye should have a default FPS.so you got 25fps in ffprobe and 30fps in elecard tool.

Thank you for your response.
Is there any way/tweak in existing encoder params, so I can set FPS to 1 or 2?

#1: there is no such parameter in openh264
#2: no plan on this.
#3: maybe you can have a try. but I don't think it is a good solution.
you can wrap h264 into mp4/avi or other files.
hope it can help you. thanks!

unfortunately, mp4/avi will not be the option for me, as I also need to transmit every h264 frame through RTSP, and writing and reading in mp4/avi will make extra latency.
I implemented 3rd option of changing VUI information outside encoder, and its working. Just VLC is not able to play at lower FPS of 1.
Thank you.

It is fine that you found the solution for you.  thanks!


[Feature Request] can add an directive in the el-table, then the el-table's Max height is the free space, the scroll only in the el-table but not all window s

Existing Component
是
Component Name
el-table
Description
can add an directive in the el-table,then the el-table's max height is the free space, the scroll only in el-table but not all window 能不能在el-table上增加一个指令，根据剩余空间，自动设置最大高度，这样表格不会超出整个页面，滚动条只会出现在表格内部

Sorry, we have no plan to support this.


Add Tasks to failed test match.

Noted this while checking a failed build in #4941. cc @bdoyle0182 this may help you in the future.

Add a regex to catch failed Tasks as in
> citool monitor  723150406.2
> Task :tests:validateSwaggerJava FAILED

Done. Your build exited with 1.

Also add a --relax flag to match failed ansible tasks.
> citool monitor -r  723150406.2
> Task :tests:validateSwaggerJava FAILED
fatal: [ansible -> localhost]: FAILED! => {"changed": true, "cmd": "cp \"/home/travis/build/apache/openwhisk/ansible/../tests/performance/gatling_tests/build/gatling.log\" \"/home/travis/build/apache/openwhisk/ansible/../logs/perf-reports/\"", "delta": "0:00:00.010214", "end": "2020-09-01 18:21:47.890812", "msg": "non-zero return code", "rc": 1, "start": "2020-09-01 18:21:47.880598", "stderr": "cp: cannot stat '/home/travis/build/apache/openwhisk/ansible/../tests/performance/gatling_tests/build/gatling.log': No such file or directory", "stderr_lines": ["cp: cannot stat '/home/travis/build/apache/openwhisk/ansible/../tests/performance/gatling_tests/build/gatling.log': No such file or directory"], "stdout": "", "stdout_lines": []}

Done. Your build exited with 1.

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[x] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

LGTM


Add a SubmitResult type to hold a transaction submission result

This type is isomorphic with the Maybe it replaces, but the name
better describes its purpose and usage.

bors r+


Make exclude_unset work in pydantic models dependencies in form body

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import Depends, Form, FastAPI
from pydantic import BaseModel

app = FastAPI()

class Item(BaseModel):
    x: str = Form()

@app.get("/")
async def foo(item: Item = Depends()):
    print(item.dict(exclude_unset=True))

Description
Open the browser and call endpoint "/"
It prints {}
Wanted Solution
I would like it not to fill dependency model with nulls, but leave it unset, with a parameter in Depends for backward compatibilty
Wanted Code
from fastapi import Depends, Form, FastAPI
from pydantic import BaseModel

app = FastAPI()

class Item(BaseModel):
    x: str = Form()

@app.get("/")
async def foo(item: Item = Depends(leave_unset=True)):
    print(item.dict(exclude_unset=True))

Alternatives
Make Form() in fields in models work without Depends() in handler function. Right now it is considered as application/json
Operating System
Windows
Operating System Details
Windows 11
FastAPI Version
0.78.0
Python Version
3.10.6
Additional Context
No response

are you want to access http://localhost:8000/ with parameter unset?

are you want to access http://localhost:8000/ with parameter unset?

I want to access it with parameter unset and in this case the print be {} instead of {"x": None} as it is now

Open the browser and call endpoint "/"
It prints {}

And

I want to access it with parameter unset and in this case the print be {} instead of {"x": None} as it is now

Seem to be the opposite of each other..

Open the browser and call endpoint "/"
It prints {}

And

I want to access it with parameter unset and in this case the print be {} instead of {"x": None} as it is now

Seem to be the opposite of each other..

Really? May be I don't construct my sentences correctly. It seems perfectly same to me...
What I mean is: now it prints {"x": None}, but I want it to print {} (because exclude_unset is True and "x" wasn't passed in request)

I see. You could use exclude_none=True like so:
class Item(BaseModel):
    x: str | None = Form()

@app.get("/")
async def foo(item: Item = Depends()):
    print(item.dict(exclude_unset=True, exclude_none=True))

This would print {} as the default value is None. Not exactly the same you were trying to achieve but maybe might work in your specific use case? It would mean that you would not allow null values as a valid input in certain Form() fields.

I see. You could use exclude_none=True like so:
class Item(BaseModel):
    x: str | None = Form()

@app.get("/")
async def foo(item: Item = Depends()):
    print(item.dict(exclude_unset=True, exclude_none=True))

This would print {} as the default value is None. Not exactly the same you were trying to achieve but maybe might work in your specific use case? It would mean that you would not allow null values as a valid input in certain Form() fields.

I know about exclude_none, but that's not exactly what I wanted. That's different things

It looks like I have similar problem, so...
I have this schema:
class GetStatisticsQueries(BaseModel):
    match_id: int | None = None
    game_id: int | None = None
    statistics_type: StatisticsType | None = None

And I want to use it as query parameters schema this way:
@router.get(
    '',
    summary='Get statistics by the given fields',
    response_model=list[StatisticsSchema],
)
async def get_statistics(
        queries: GetStatisticsQueries = Depends(),
        ...
) -> list[dict]:
    queries_dict = queries.dict(exclude_unset=True)

The problem is that dict(exclude_unset=True) returns all values every time even if value was not set. I guess it happens because of Depends() call.


How to apply focal loss in Cascade RCNN?

Hi, due to the high imbalance of the class distribution of my own dataset, I want to use focal loss to solve it. I tried change all the 'loss_cls' in cascade rcnn to
loss_cls=dict(                 type='FocalLoss',                     use_sigmoid=True,                     gamma=2.0,                     alpha=0.25,                     loss_weight=0.005),
but the loss gets bigger and bigger. I wonder how to change the config file of cascade RCNN to use focal loss, or do you have any advice to help with the imbalance problem? Thanks!

It is not surprised. FocalLoss may not work well on two-stage detectors. For the imbalanced problem, it is beyond the scope of the codebase usage. My personal suggestion is that you may read some related papers, to name a few, OHEM, GHM, Libra R-CNN, etc.


merge from template-node-app

package.json

chore(package): add pacman/nginx.conf (pacman.content./etc/nginx/config.d/${name}.conf)

:tada: This PR is included in version 4.2.24 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v4.2.24

Your semantic-release bot :package::rocket:


Changes in 1.0.6 breaks babel-eslint

See this https://github.com/babel/acorn-to-esprima/compare/v1.0.5...v1.0.6
And this https://github.com/babel/babel-eslint/issues/207

TBH, this change breaks backward compatibility so it means someone used semver incorrectly.

I didn't know it would break at the time but yeah I can revert.

Reverted and made 1.0.7


Newsboat 2.20 requires space before semicolon in macros

It would be cool if I could via macro invoke custom bash script and pass url as an argument.
For instance, I've created a scripts which downloads a yt video and then translate English subtitles to language of choice and automatically play it out via mpv.
Simplified youtube-dl config:
-o ~/movies/%(id)s.%(ext)s
--write-sub
--exec 'trans_sub {} && mpv {}'

which trans_sub i my script translating and saving subtitles.
It work sreally well when I do:
youtube-dl http://youtubevideourl.com
But adding to newsboat config:
macro m set browser "youtube-dl %u"; open-in-browser ; set browser "elinks %u"

doesnt work at all.
Would be cool If I could do something like:
macro m execute "youtube-dl %u"

Hi! Sounds similar to #702. Does it work if you add a space before semicolons? Like this:
macro m set browser "youtube-dl %u" ; open-in-browser ; set browser "elinks %u"

#702 is related, but is not the root cause of this (and it actually asks to make the space optional). This bug is caused by https://github.com/newsboat/newsboat/commit/4b327fbff264c69bb78765304e16c2403c65b5e1, which made utils::tokenize_quoted drop the semicolon when tokenizing set browser "youtube-dl %u";. Reverting that commit fixes the issue, but I'm thinking of alternative solutions as well (because I don't want to re-introduce the second loop if one is sufficient).
Either way, I'll release Newsboat 2.20.1 in the middle of this week, which will contain a fix for this bug.
Sorry for the inconvenience!

Unfortunately adding a space doesn't fix this. It actually downloads video to the right place but won't even start to translate subtitles, therefore this config line is not even executed:
--exec 'trans_sub {} && mpv {}'
From youtube-dl man:
--exec CMD
              Execute a command on the file after downloading and post-processing, similar to find's -exec syntax.

Also in theory after downlaod completes it should executes this CMD but it only returns back to my rss feed with
-Browser returned error code 1

Okay, looks like I was a bit too hasty in attributing this to 2.20 breakage. A few additional questions then:

what Newsboat version are you running? You can find out with newsboat --version;
does --exec 'true && mpv {}' work?
does --exec 'mpv {}' work?
does exec 'true' work?

If none of that works, I'll need your trans_sub script, and a feed URL to test this on. Right now, I can't reproduce this because I can't re-create your configuration.
Just to make it clear: the proposed execute op would work exactly the same as set browser … ; open-in-browser dance, so I need to figure out why the existing approach doesn't work; just renaming it won't change anything.

I basically hijacked this issue, but please don't be deterred by that, @pbakiewicz: report the results of the tests I suggested above, and if it's a different bug, we can just make a new issue. Sorry about the confusion, but by this point it's easier to roll with it than to re-tag everything to point someplace else.


Microsoft 365 vs Office 365 references

Still several references to Office 365, looks like some have been changed to Microsoft 365 but not all.
[Enter feedback here]

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

ID: a6e20438-8387-bf21-297c-9feb5efe1f1c
Version Independent ID: ee6a90dd-4050-fa9f-70f4-795f8e7bc3ca
Content: Load non-Microsoft 365 data into a review set - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/load-non-Office-365-data-into-a-review-set.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@adamfowlerit Thanks for the heads up. Documentation will eventually make transition from O365 to M365 as the service transitions.


Improve dot vs colon method invocations

This implements #108, as well as a couple other fixes:

Adds a !DotMethod decorator, which can be used on classes, interfaces, and methods to indicate they should be called using dot syntax (e.g. instance.method() as opposed to instance:method())
Fixes static methods, which are now always called using dot syntax
Adds a test that ensures that the appropriate syntax is used with classes, interfaces, instance methods, static methods, and the decorator.

Closed in favor of #111


Update app-designer-overview.md

Updated description and deleted descriptions that are no longer true.

Learn Build status updates of commit d0baa85:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerapps-docs/maker/model-driven-apps/app-designer-overview.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit a83ae4e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerapps-docs/maker/model-driven-apps/app-designer-overview.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit ba9480b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerapps-docs/maker/model-driven-apps/app-designer-overview.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit fab5a77:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerapps-docs/maker/model-driven-apps/app-designer-overview.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

Since this needs rework (adding images) moving to the private repo. New pr is here: https://github.com/MicrosoftDocs/powerapps-docs-pr/pull/7453
Closing this PR without merging..


filters

Fixes
#265

Description
mentioned in README.
Type of change
Added new project.

[x] New project (non-breaking change which adds functionality)

How Has This Been Tested?
A video has been given for the same.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have included a README.md file
[x] My project follows this structure: Domain Folder/ Project Name Folder/ Project files and Readme.md files
[x] I have added demonstration in the form of GIF/video file
[x] I am a GSSoC'22 Participant

Your work and Video is good But please put images or gif for all the other filters too. And have you added Python/filters/jkr.png for experimental purpose?

Mam jkr.png was supposed to be removed , but I forgot to do so


Adaptive Quality break in Unity 5.5.2

It happens showing in the picture when framerate drop, I am using Unity 5.5.2 .
In Unity 5.5.1's log :
Fixed the usage of VRSettings.renderViewportScale in Camera's OnPreCull so that it was not a frame latent on all supporting SDKs. Fixed issues with Valve's The Lab Renderer adaptive quality feature.
Is it a correct result? How could I make it right.

My bad it should be Multi Pass to work


add a button to assign random width and height to nodes

Add a button in the demo, which upon clicking will assign some random width and height to the nodes of the graph.

Can we increase the variation (upper limit) a bit? Upper limit could be 2 times the current dimension.

I increased it. But then we had a problem with rank gap as well because we were using half of the max height to each side of a rank. Now the rank gap max height + constant (20).