scorecard seems to miss some status checks

Looking at the results of the SAST check I noticed that sometimes scorecard says that some status checks (namely lgtm-com) weren't run even though they were. GitHub API described at https://docs.github.com/en/rest/reference/checks#get-a-check-suite has no problem finding them:
https://github.com/libbpf/libbpf/pull/353
$ curl --silent  -H "Accept: application/vnd.github.v3+json"   https://api.github.com/repos/libbpf/libbpf/commits/aafe84ae9b8e7b837c6469e48e85f7c3769c79b6/check-suites | jq '.check_suites[] | .app.slug, .latest_check_runs_count'
"lgtm-com"
1
"github-actions"
13
"github-actions"
13
"github-actions"
13

https://github.com/mesonbuild/meson/pull/9576
$ curl --silent  -H "Accept: application/vnd.github.v3+json"   https://api.github.com/repos/mesonbuild/meson/commits/53120d13e56f8536dfb093a35fb90559c171295a/check-suites | jq '.check_suites[] | .app.slug, .latest_check_runs_count'
"lgtm-com"
2
"github-actions"
1
"github-actions"
3
"github-actions"
1
"github-actions"
1
"github-actions"
2
"github-actions"
1
"github-actions"
3
"github-actions"
12
"github-actions"
4
"azure-pipelines"
7
"codecov"
2

I'm not sure how exactly ListCheckRunsForRef is implemented but it seems it's different from what the "check-suites" returns.

It's kind of related to https://github.com/ossf/scorecard/issues/1322

It's using https://github.com/google/go-github/blob/master/github/checks.go#L250
They also expose check suites API, see https://github.com/google/go-github/blob/master/github/checks.go#L274
if you give it a try, let us know.
cc @azeemsgoogle

Apparently https://github.com/google/go-github/blob/master/github/checks.go#L250 is where lgtm-com gets lost because with check-runs instead of check-suites I can no longer see it anywhere:
curl --silent  -H "Accept: application/vnd.github.v3+json"   https://api.github.com/repos/mesonbuild/meson/commits/53120d13e56f8536dfb093a35fb90559c171295a/check-runs |
grep -i lgtm

so you're saying this is a GitHub issue maybe? Shall we file an issue on GitHub repo?

I'm not sure it's a GitHub issue given that they provide both check-runs and check-suites (which I think should be different from each other. Otherwise, they wouldn't have been split in the first place I think). The question is what exactly check-runs and check-suites are and when they should be used.

According to https://docs.github.com/en/rest/guides/getting-started-with-the-checks-api#about-check-suites

A check suite is a collection of the check runs created by a single GitHub App for a specific commit. Check suites summarize the status and conclusion of the check runs that a suite includes

A check run is an individual test that is part of a check suite. Each run includes a status and conclusion.

so as far as I can tell scorecard should go over check_suites and get every check suite's runs. This way in https://github.com/libbpf/libbpf/pull/353 LGTM would have been found via https://api.github.com/repos/libbpf/libbpf/commits/aafe84ae9b8e7b837c6469e48e85f7c3769c79b6/check-suites and then https://api.github.com/repos/libbpf/libbpf/check-suites/3433098465/check-runs and in https://github.com/mesonbuild/meson/pull/9576 via https://api.github.com/repos/mesonbuild/meson/commits/53120d13e56f8536dfb093a35fb90559c171295a/check-suites and https://api.github.com/repos/mesonbuild/meson/check-suites/4359629405/check-runs.
I'm not sure why those runs aren't included in https://api.github.com/repos/libbpf/libbpf/commits/aafe84ae9b8e7b837c6469e48e85f7c3769c79b6/check-runs and https://api.github.com/repos/mesonbuild/meson/commits/53120d13e56f8536dfb093a35fb90559c171295a/check-runs. That's probably a GitHub issue assuming check-runs is supposed to return all the check runs triggered by a commit.

so as far as I can tell scorecard should go over check_suites and get every check suite's runs

On second thought it doesn't seem to be necessary to get every check suite's runs since "check suites summarize the status and conclusion of the check runs that a suite includes"

I'm inclined to say that it's probably a GitHub bug and it should be probably fixed elsewhere. Closing.


apply_changes/1: convert nested schemaless validations

Fixes #4514

Sorry if I am misunderstanding. But was the intent of the original issue to be able to validate maps before persisting? If so will this not ignore the validation

Good point, we will need to carry those accordingly and also document the feature, since this is specific to apply changes, we don’t handle changesets in maps otherwise.

@azizk Do you want to take that on? If not we can probably merge this and I could tackle the other stuff.

Yeah, I suspected something else was missing to make this a complete feature. I think I'd rather hand it over to you. Too many functions to look at. :sweat_smile:
Thanks!

No worries. Thank you for this change. @josevalim are you ok to merge just this one for now?

Or maybe I am getting ahead of myself. I have doubts now which one of these 2 we are going with

we only ever make the change for apply_changes and document it
We also try to do it for persistence. I.e resolve the value or report an error during repo operations

the second point sounds like a good feature to me but I haven’t evaluated whether it’s possible

The best would be likely to support this everywhere, because this solves the problem of us supporting nested schemaless changesets, but I am unsure how practical it is, especially if it will require us special casing map types everywhere.

special casing map types everywhere.

Sounds like introducing a schema 😉
I haven't come across the need to do nested schemaless changesets. I would think if you reach for nested schemaless, you should probably define a schema because it's about to get complex.

One thing I can see is if you have one or a few functions that do some basic validations on maps that can be reused. Like a function that takes in a list of keys and checks if they are present. As opposed to creating separate schemas for every type of map you might want to do this on.

sorry @azizk. so to close this out would you be able to add into the docs for apply_changes/1 that we allow this kind of schemaless changes? and then we'll take it away to see if it makes sense to extend it further

@greg-rychlewski

One thing I can see is if you have one or a few functions that do some basic validations on maps that can be reused. Like a function that takes in a list of keys and checks if they are present. As opposed to creating separate schemas for every type of map you might want to do this on.

That's a good example. I'd like to mention another one that came to me yesterday. It's about saving storage space.
I have been working on a work project dealing with OpenRTB. I have an embedded schema for the Bid object which has a lot of fields. When I store a Creative schema I only want to store a limited number of fields from the Bid embedded schema. So I define field :bid, :map instead of embeds_one :bid, Bid, because otherwise Ecto stores all properties from Bid even if the values are nil. I still want to validate the bid map field in the Creative changeset function to be sure and to avoid future surprises.
What I do atm is to use validate_change/3 and add an error if the Bid changeset is invalid.
Initially we wanted to allow schemaless changesets, but maybe it's worth thinking about allowing any kind of changeset. Otherwise it becomes necessary – in this case at least – to convert a schema changeset to a schemaless changeset (I guess by doing Map.update!(changeset, :data, &Map.from_struct/1?).
PS: I didn't want to open a new issue, so commented here, if that's okay.

@azizk Jose had a good idea where custom types can be utilized. I'm not sure if you saw this before but you could basically make your own type and control these things through the cast callback. And you could even make the type parameterized to make it generalizable.
Let us know what you think:
https://hexdocs.pm/ecto/Ecto.Type.html
https://hexdocs.pm/ecto/Ecto.ParameterizedType.html

As one example, you could get very close to mimicking cast_embed by making a parameterized type like MyValidatedMap that takes a :with parameter. That parameter is a function that does all the changeset functions and then as an extra step at the end checks if the changeset is valid and then applies the changes or returns an error.


Victor/http/http request

Adding some progress for HTTP Request class

Note that I am adding a delegate constructor to support having body as optional arg for creating an http request. This delegate constructor is a nice C++ 11 feature

Adding gtest as testing framework. Adding it as gtest recommends by asking cmake to download the src code to build directory and build it for us so we just need to link against it.  This is an elegant way to consume a library!  we should consider it !

Adding some initial tests for http request

Victor/http/http request

Please update the PR title to be descriptive. It ends up becoming the commit message and makes it easier to understand what each change did when looking back at the git history.
From contributing guide:
https://github.com/Azure/azure-sdk-for-cpp/blob/master/CONTRIBUTING.md#pull-requests

DO give PRs short-but-descriptive names (e.g. "Improve code coverage for Azure.Core by 10%", not "Fix #1234")


#1283: Enhance the kamel log command to show more detail prior to i…

…nstall

Sets up a polling prior to the complete install of an integration
First report whether it finds the integration
Assume it finds an integration then determine its status
If it gets to Building Kit (which can take a while) then find the
associated IntegrationKit and report its status
Finally return to the scraping of the log once the integration is up
and running

Release Note
NONE

Thanks @phantomjinx, having more info really helps.
I've not tested it, but I see there's some code reporting the state of the integration kit. There is already some info about that, because Kubernetes events are also printed.
Have you checked this when installing the operator + running kamel with a admin user (an user that can manage Kubernetes events), to see if there's some duplication.

Hey @nicolaferraro
Been developering & running this through as cluster-admin. The events for the integration-kit are more detailed in that they include information of conditions and scaling up. The reason I include mention of the integration-kit in the kamel log is due to the Build Running phase of the integration taking the bulk of the time. Without mention of the integration-kit, the integration log just sits there for 2-3 minutes. Anyway, this is an example of kamel log sample:
⌁66% [: … camel-k] master(1) 130 ± kamel log sample
Integration 'sample' not yet available. Will keep checking ...
The building kit for integration 'sample' is at: Build Submitted
The building kit for integration 'sample' is at: Build Running
Integration 'sample' is at: Deploying ...
Integration 'sample' is now running. Showing log ...
[1] Monitoring pod sample-746854ccb4-5zgct
[1] exec java -cp ./resources:/etc/camel/conf:/etc/camel/resources:/etc/camel/sources/i-source-000:dependencies/camel-k-integration-1.3.0-SNAPSHOT-runner.jar:dependencies/io.quarkus.arc.arc-1.8.0.Final.jar:dependencies/io.quarkus.quarkus-arc-1.8.0.Final.jar:dependencies/io.quarkus.quarkus-bootstrap-runner-1.8.0.Final.jar:dependencies/io.quarkus.quarkus-core-1.8.0.Final.jar:dependencies/io.quarkus.quarkus-development-mode-spi-1.8.0.Final.jar:dependencies/io.quarkus.quarkus-ide-l
....

Let me know if you think it needs tuning further.


Rolling luck with modifiers isn't possible...?

Right-clicking onto the "Roll Luck" option doesn't bring up the penalty/bonus menu while the agent's handbook sometimes mentions them with luck rolls. An example would be the Spending Illicit Cash optional rule at page 91 where it states "Make an Accounting roll or a Luck roll, player’s choice, at a −20% penalty for an Extreme expense."

For whatever reason I was never able to bind the right-click on the header buttons.  Perhaps it is possible, let me know if you've seen it done in a different system and I'll look at their implementation.
But you can shift + click to get the modify roll dialog for the Roll Luck button.  So what you are after is currently doable,  just need to use the alternate way of triggering the roll dialog (note - shift + click works for everything else as well).


update ember-cli

Update ember-cli to 2.13.0

thanks! :clap:

@bgentry, could you deploy the latest changes please so we can use it ? 😅

released v0.1.0, hopefully that works for you. I made some more progress upgrading the Ember version but didn't get it all the way working (LTS 2.12 tests fail for some reason).

Thanks @bgentry, I did not realise I only upgraded till 2.13 .. 😅
I have finished the upgrade in this PR https://github.com/bgentry/ember-currency/pull/4 to the latest stable version.


Crashes On Win10 Instantly

My Setup

Operating System: Windows 10 Build 1909
App Version: Latest as of 11/23/2020
Installation type: MSI installer & windows unpacked

[ X] I have tested with the latest version
[ X] I can simulate the issue easily

Description

Follow guidelines set in readme.md:

git clone
yarn
yarn release

Current Behavior

Everything builds fine, no errors or issues (https://imgur.com/1ChH35Y), doing 'yarn start' runs the application fine as well (https://imgur.com/degxluQ).
The MSI installer reads setup failed (https://imgur.com/xzo0d5T), no error log produced or information regarding why. The windows unpacked exe doesn't load anything.
No logs I can find, nothing in windows event viewer.
Expected Behavior

App should load/run or produce output regarding why installation/loading failed.
Might be related to:

Checking for update
Update for version 3.1.0-snapshot is not available (latest version: 3.0.7, downgrade is disallowed).

Thank you for your time and help.

I should also note that both application types (the MSI and unpacked) were ran on a system which has never had any Rocket.Chat software installed on it previous. Same aborted setup issue.

@starhound look at this https://github.com/RocketChat/Rocket.Chat.Electron/issues/1804

Thank you, this is a duplicate issue. Closing now.


Fix grammatical error in community.md

Category

[x] Content fix
[ ] New article
Related issues: fixes #X, partially #Y, mentioned in #Z

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
(DELETE THIS PARAGRAPH AFTER READING)

Guidance

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch. Released documents are in live branch.
  This is the expected behavior of the build system.

OPS Build status updates of commit 59f137f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/community/community.md
:white_check_mark:Succeeded
View

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

thx @clearab, appreciated.


Version 297

Spelling in commit message "requied "

Spelling in commit message "requied "

Solved and also corrected in CHANGELOG


[fix][broker] Key-shared subscription must follow consumer redelivery as per shared sub semantic

Fixes https://github.com/apache/pulsar/issues/21656

Motivation
SHARED or key-SHARED subscription must dispatch redelivered messages in any scenario. every shared subscription should dispatch already delivered unack messages. You can follow strict ordering for new messages which broker is reading first time by advancing readPosition of the cursor but broker can dispatch already delivered unack messages when its required without restricting any scenario.
However, key-shared subscription is incorrectly handling redelivered messages by keep reading redelivered messages , discarding them and not dispatching any single messages to the consumer by incorrectly changing the semantics of consumer delivery ordering. broker doesn't dispatch redelivery message if that message id is smaller than consumer's assigned offset-message-id when it joined. broker assigns cursor's current read position as consumer's min-message-id offset to manage ordering but delivered messageId can be smaller than that position and redelivery should not be restricted by ordering as we already discussed semantics of shared subscription earlier. But as broker handles it incorrectly in key-shared because of that key-shared subscription topics which have connected consumers with positive permits are not able to receive any messages and dispatching is stuck also broker is keep performing same cold reads across those stuck topics and wasting storage and CPU resources by discarding read messages. which impacts application, broker and bookies and such buggy handling is semantically and practically invalid.
Right now, such multiple topics with key-shared subscription and redelivery messages can significantly impact broker and bookies by keep reading large number of messages without dispatching them and client application are not able to consume any messages which also impacts application significantly.
Modifications
Allow dispatching of redelivered messages, avoid reading and discarding duplicate messages, and fix broken stuck dispatcher on unack messages.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report

Merging #21657 (eb86950) into master (2bf1354) will increase coverage by 0.11%.
Report is 15 commits behind head on master.
The diff coverage is 75.00%.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #21657      +/-   ##
============================================
+ Coverage     73.24%   73.35%   +0.11%
+ Complexity    32752    32739      -13
============================================
  Files          1893     1893
  Lines        140730   140738       +8
  Branches      15500    15500
============================================
+ Hits         103071   103235     +164
+ Misses        29563    29378     -185
- Partials       8096     8125      +29

Flag
Coverage Δ

inttests
24.10% <0.00%> (?)

systests
24.72% <0.00%> (+0.05%)
:arrow_up:

unittests
72.64% <75.00%> (-0.04%)
:arrow_down:

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

Files
Coverage Δ

...a/org/apache/bookkeeper/mledger/ManagedCursor.java
42.85% <ø> (ø)

...che/bookkeeper/mledger/impl/ManagedCursorImpl.java
78.90% <ø> (-0.23%)
:arrow_down:

...ava/org/apache/pulsar/broker/service/Consumer.java
86.30% <100.00%> (+0.02%)
:arrow_up:

...ersistentStickyKeyDispatcherMultipleConsumers.java
81.73% <72.72%> (-2.93%)
:arrow_down:

... and 98 files with indirect coverage changes

@codelipenghui
simply adding removed consuner's unack messages into redelivery/replay list will not work because there are scenarios where key-shared dispatcher adds additionally filtered messages(due to max-message limit, or ordering) into replay list which are associated with already connected consumers and we have to differentiate those messages and actually unacked messages, and this PR addresses that differentiation.

can we remove the blocker if there is no concern about the PR and can we merge it as it's really needed for various production systems.

The problem that this PR is addressing will be covered in PIP-379: Key_Shared Draining Hashes for Improved Message Ordering.
To address this issue, I'll update PIP-379 so that the updated contract will cover how negative acknowledgements and explicit redeliveries such as redeliverUnacknowledgedMessages are handled in Key_Shared in the updated design.
Currently it's not properly documented what happens when an application uses those methods when a Key_Shared subscription is used.
If an application uses "nacks", it should either fail with an exception or there should be a clearly defined behavior. Since failing with an exception on the client side isn't a real option, the remaining option is to clearly define the behavior and make it consistent.

I'm closing this PR since the problem will be covered by PIP-379 and it's implementation. Please see the previous comment.


Required attribute without predicates doesn't error as expected when missing

I have a simple schema which includes one required attribute:
schema = Dry::Validation.Schema do
  required(:foo)
end

When I call it without a foo key I don't get any errors:
schema.call({}).errors   # => {}

I would've expected an "is missing" error in there.
If I extend the schema to add a predicate to the required(:foo) part and try again I get the expected error:
schema = Dry::Validation.Schema do
  required(:foo).filled
end

schema.call({}).errors   # => {:foo=>["is missing"]}

Confirmed in v0.10.7 FWIW.

Will be fixed in the next version:
schema = Dry::Schema.define do
  required(:foo)
end
2.6.1 :001 > schema.({})
 => #<Dry::Schema::Result{} errors={:foo=>["is missing"]}>


List all Debian package versions in Packages

Closes #27783
This PR lists all and not only the latest package versions in the Packages index.

MSSQL failure related.

The test runs too fast and the ORDER BY created produces unstable results then.


[BACKLOG-32449] Hadoop Cluster- Update Kerberos Security options for …

…Client/Repository
Changes to support setting authentication cookies prior to opening
the cluster dialog on the server.
https://jira.pentaho.com/browse/BACKLOG-32449

:white_check_mark: Build finished in 10m 17s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1126, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


About WsgiApp with CORS

Now, we use http decorator only support one method.
@http('GET', '/hello')
def xxxx(self, request):
    pass

Why I need multiple methods support？
Because of that I use put or post method in browser, It have a default method with 'options' been caused.
It's base on CORS.

it's not very well documented yet unfortunately, but did you know that the following is currently possible:
    @http('GET', '/multiple')
    @http('POST', '/multiple')
     def method(self, request):
         return 200, request.method

Thank you very much, It's my fault.
Handle the CORS on browser, It's not good idea.
So I want to write it on config _yaml file.

@lagelalegal have you looked at this extension? https://github.com/harel/nameko-cors as an example of cors support?

@kooba good, thank you very much.

wanna add it the the community-extensions list? (which is on master but not yet on "stable" which is default on readthedocs :(

I'm bit worried about adding anything without tests to that list 😄 ... I might open PR against nameko-cors with some tests first 😉

Do you want add nameko-cors into community-extensions? I think I must remind the nameko-cors author at first.

This issue was moved to https://discourse.nameko.io/t/about-wsgiapp-with-cors/311


Multimedia samples video decode fix

Description of changes
Add a patch to fix video_decode multimedia sample package
Kudos to OE4T for already having a fix for this :)
Fixes https://github.com/anduril/jetpack-nixos/issues/131
Testing
Tested working on Orin AGX devkit

video_decode output after applying this fix:
Creating decoder in blocking mode
Opening in BLOCKING MODE
NvMMLiteOpen : Block : BlockType = 261
NVMEDIA: Reading vendor.tegra.display-size : status: 6
NvMMLiteBlockCreate : Block : BlockType = 261
Setting frame input mode to 1
Starting decoder capture loop thread
Video Resolution: 1920x1080
Video SAR width: 1 SAR height: 1
Decoder colorspace ITU-R BT.601 with standard range luma (16-235)
Query and set capture successful
Input file read complete
Got EoS at output plane
Got EoS at capture plane
Exiting decoder capture loop thread
App run was successful


Update postgresql to 42.4.2

Updates org.postgresql:postgresql from 42.3.6 to 42.4.2.
Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (42.3.6).
You might want to review and update them manually.
CHANGELOG.md

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.postgresql", artifactId = "postgresql" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.postgresql", artifactId = "postgresql" }
}]

labels: test-library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #2579.


New Logo

New named and without named logos

add screenshot also

what do you mean by screenshot?

Like how the logo is looking on website

add some image

use first one and add it to favicon of all pages

Ok

is it done?


Add Enumerable#sum

It's only added in Ruby 2.4 though. But I think it might be helpful to some people

I don't agree with copying stuff over from the Ruby standard library - and if they are to be included I think they should mimic their exact behavior (float rounding compensation, arbitrary +-ables).
Idea: Have these kinds of core extensions available as opt-ins, like require 'gtk/core_ext/array'? I'll admit I have reimplemented a lot of the Ruby and Activesupport standard libraries for myself in my own code and this would indeed be legitimately helpful.

I don't agree with copying stuff over from the Ruby standard library

I can see that bloating the DragonRuby standard library with lots of additional features might not be the best thing.
In my personal opinion, being able to sum a list of integers with one method call is something useful enough to be considered.
And if compatibility with mruby is a concern then this should be no problem either, since it's just adding a new method (just like Amir added lots of convenience methods on Array like x y w h etc). As long as it's not removing or altering a mruby feature it should be fine. Compatibility to mruby is probably only a concern in the first place to ensure that existing mruby gems/code can be easily ported to DragonRuby - I don't think it's a value of DragonRuby in itself (but then again I'm not Amir & team so I don't speak with authority on design decisions like that.

they should mimic their exact behavior

I agree! I added the support for the initial value and blocks for that exact reason since it's part of Ruby's API (https://devdocs.io/ruby~2.6/enumerable#method-i-sum)

(float rounding compensation, arbitrary +-ables).

Since a + b is just a method call (a.+(b)). This is automatically fulfilled. The Float class' implementation of + should take cake of this. Nothing to that needs to be taken care of from the point of view of Enumerable#sum

(float rounding compensation, arbitrary +-ables).

I was going off of the implementation that was linked in the Discord earlier  and not off the actual documentation for Enumerable#sum. You are right in your implementation I think; I missed the part where the init value is 0 by default in the array code also, that just struck me as weird.
Though now I'm questioning why Array#sum even does floating point rounding error correction  (not at all taken care of by simply Float#+) without it being documented...

You are right in that there is precedence for monkey patching of GTK-related code into core classes, but I think there should be a higher bar for introducing methods that "I miss this method from the Ruby standard library" when there is a whole section on the differences between standard libraries in the DragonRuby FAQ.
Again, I do miss these kinds of helper methods (and frequently copy them into my own projects), but I would very much like to see them as require-able opt-ins rather than monkey patched by default.

Keep the conversation going. My two cents on this is pretend MRI doesn't exist, but we have all the lessons learned from the past 25 years.

Consider:
args.fn.sum

This might be a good way to say "hey, this isn't in core, so we added it under the args.fn "namespace"?

Consider:
args.fn.sum

This might be a good way to say "hey, this isn't in core, so we added it under the args.fn "namespace"?

Closed. No activity.


Include URLs on graph edges

Summary
Excellent find from @konstin. If we have a package that's included in two forks at the same version, but with different URLs, we need to avoid collapsing them in the lockfile.
Closes https://github.com/astral-sh/uv/issues/5294.

This failure is on main already, so gonna merge.

Wait, nevermind, there's a separate Windows failure.


Liat/mgmt 597 - get next step operation logic

In general it is OK, but it touches same places that I added for this PR https://github.com/filanov/bm-inventory/pull/32.  We will need to be in sync for that

Closing this PR - not relevant anymore


Dev -> prod

:tada: This PR is included in version 1.30.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.30.2-dev.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Problem with using mask to keep an object in image generation

Hello, I am using the following command to use this package to generate some product images. I use the following prompt to keep my image and put it in various locations. But as you see, there is another image under my main image. It is bigger and lower parts of this product is shown under my product. I appreciate any help @brycedrennan
prompt: imagine --init-image 3.jpg --mask-prompt "shoes" --mask-mode keep "Product photography, shoes product, lying on the water surface, with flowers, stone on sparkling water, natural lighting, nature photorealistic photography, dreamy tones, focus on product, center composition, high resolution"  -r 1 -w 500 -h 500 --upscale --steps 50 --prompt-strength 50 --model LI-1.0

here is main image:

I will appreciate any help Mr. @brycedrennan
Let me thank you for your kindness.

@keloniton I can see that you are using a custom model but I am curious if you tried using a lower prompt strength (5-10).

@ahmed-mohamed-sn I tested with strength of 6. No thing changed, I think the problem should be some thing different.

did you try using the mask modifiers to expand the mask area and/or strength? (e.g. mask prompt: shoes {+5}{*3})

@ahmed-mohamed-sn @brycedrennan
I used the mas modifier to decrease the mask strength and area, It did not work. I think the problem is that, Imaginary places an object (for example shoes in my example) for creating image at first, and then if we use --mask-mode keep (or replace), it will put our init_image on that (some how overlay our image on it). If the size of imaginary shoes are bigger than our shoes, some parts of it are shown. so using mask modifier will not change the result.

I also tried out painting with PNG (transparent image) and white background image. It does not help.
Transparent:

White background

I think there should be another way. Any idea?

@brycedrennan
Do you have any idea how to solve this problem. I will appreciate your help. Thanks.

No one answers here?!
@brycedrennan


Missing pre-training code for beit-3?

Dear authors, I'm trying to pre-train beit-3 based on my own dataset. I wonder where can I find the pre-training code for beit-3?

How do you solve it in the end ？


It is hard to know when a Toil run is hung due to misconfigured autoscaling

I recently tried to run a Toil run that contained non-preemptable jobs, on AWS, with autoscaling having been given only a preemptable instance type.
What I observed was the cluster wanting to scale down to size 0 (unless I set minNodes to 1). But in either case, Toil didn't complain; it would just sit there happily chattering about how many nodes it estimated were needed and how many jobs were queued, without noticing that no jobs were running and that no jobs were ever going to get run (because it didn't have any nodes available to start that could run the jobs it had). I had to turn up logging to debug to work out that the available nodes weren't going to be sufficient:
DEBUG:toil.batchSystems.mesos.batchSystem:Got offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 for a preemptable slave with 244961.00 MiB memory, 32.00 core(s) and 598361.00 MiB of disk.
2018-05-01 22:26:45,888 - toil.batchSystems.mesos.batchSystem - DEBUG - Got offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 for a preemptable slave with 244961.00 MiB memory, 32.00 core(s) and 598361.00 MiB of disk.
DEBUG:toil.batchSystems.mesos.batchSystem:Offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 not suitable to run the tasks with requirements {'cores': 1, 'preemptable': False, 'disk':<PHONE_NUMBER>, 'memory':<PHONE_NUMBER>}. Mesos offered 2.56860225536e+11 memory, 32.0 cores and 6.27426983936e+11 of disk on a preemptable slave.
2018-05-01 22:26:45,888 - toil.batchSystems.mesos.batchSystem - DEBUG - Offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 not suitable to run the tasks with requirements {'cores': 1, 'preemptable': False, 'disk':<PHONE_NUMBER>, 'memory':<PHONE_NUMBER>}. Mesos offered 2.56860225536e+11 memory, 32.0 cores and 6.27426983936e+11 of disk on a preemptable slave.
DEBUG:toil.batchSystems.mesos.batchSystem:Although there are queued jobs, none of them could be run with offer value: "783b6158-8f8f-40f3-a2d7-695df13f606d-O102"
 extended to the framework.

Toil ought to complain loudly (at WARN log level at least) if it has jobs that can't be run on any of the node types available to it. Even better if it terminates the run with an error instead of waiting around hoping that the situation somehow changes.

Terminating ASAP seems the most user-friendly (and budget friendly) approach.
Adam Novak<EMAIL_ADDRESS>writes:

I recently tried to run a Toil run that contained non-preemptable jobs, on AWS,
with autoscaling having been given only a preemptable instance type.
What I observed was the cluster wanting to scale down to size 0 (unless I set
minNodes to 1). But in either case, Toil didn't complain; it would just sit
there happily chattering about how many nodes it estimated were needed and how
many jobs were queued, without noticing that no jobs were running and that no
jobs were ever going to get run (because it didn't have any nodes available to
start that could run the jobs it had). I had to turn up logging to debug to
work out that the available nodes weren't going to be sufficient:
DEBUG:toil.batchSystems.mesos.batchSystem:Got offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 for a preemptable slave with 244961.00 MiB memory, 32.00 core(s) and 598361.00 MiB of disk.
2018-05-01 22:26:45,888 - toil.batchSystems.mesos.batchSystem - DEBUG - Got offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 for a preemptable slave with 244961.00 MiB memory, 32.00 core(s) and 598361.00 MiB of disk.
DEBUG:toil.batchSystems.mesos.batchSystem:Offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 not suitable to run the tasks with requirements {'cores': 1, 'preemptable': False, 'disk':<PHONE_NUMBER>, 'memory':<PHONE_NUMBER>}. Mesos offered 2.56860225536e+11 memory, 32.0 cores and 6.27426983936e+11 of disk on a preemptable slave.
2018-05-01 22:26:45,888 - toil.batchSystems.mesos.batchSystem - DEBUG - Offer 783b6158-8f8f-40f3-a2d7-695df13f606d-O102 not suitable to run the tasks with requirements {'cores': 1, 'preemptable': False, 'disk':<PHONE_NUMBER>, 'memory':<PHONE_NUMBER>}. Mesos offered 2.56860225536e+11 memory, 32.0 cores and 6.27426983936e+11 of disk on a preemptable slave.
DEBUG:toil.batchSystems.mesos.batchSystem:Although there are queued jobs, none of them could be run with offer value: "783b6158-8f8f-40f3-a2d7-695df13f606d-O102"
extended to the framework.
Toil ought to complain loudly (at WARN log level at least) if it has jobs that
can't be run on any of the node types available to it. Even better if it
terminates the run with an error instead of waiting around hoping that the
situation somehow changes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.*


LCFS - Users can download transactions data

Description:

Enable the ability to download transactions information from the Transactions table into a spreadsheet
Download as Excel button (see wireframe)
For both BCeID and IDIR users, although downloaded information will be slightly different. For BCeID users, only their transactions are downloaded whereas IDIR users see all transactions.
Download the following data from the transactions table: (in order from left to right): ID, Compliance period, Type, Compliance units from, Compliance units to, Number of units, Value per unit, Category, Status, Effective Date, Comments (external)
For IDIR users, when the filtering function is in use (see card #192), only the filtered transactions should be downloaded. This functionality currently exists in TFRS for downloading the transactions table data.
File type = .xls
Workbook name format: BC_LCFS_transactions_YYYY-MM-DD.xls, where YYYY-MM-DD is the day the file was downloaded
Worksheet name: Transactions
Spreadsheet columns and values should be formatted in such a way that there is adequate spacing/width for the columns, number formatting uses comma's (43,343), and left or centre indent (see screenshot below)
The button label switches to "Downloading..." after the user clicks on the button and the sheet is being downloaded (loading). The button then switches back to the regular label after it has downloaded. This functionality currently exists in TFRS and is useful to communicate to user that the system is working (while loading), especially for IDIR users where the downloaded data can take some time due to its larger size.

Note: This feature is near identical to the one that exists in TFRS
Wireframe:

UXPin: https://preview.uxpin.com/59145260c7d8cc48969ac79401cc03d84843c3d9#/pages/165270334/simulate/no-panels

Screenshot of example spreadsheet from TFRS:

"Downloading..." button example from TFRS: Private Zenhub Image

Purpose and benefit to user:
Users can easily and efficiently download a list of transactions.
Acceptance Criteria:

[ ] Given I am a BCeID user, when I download transactions, then I should receive an Excel file containing only my transactions.
[ ] Given I am an IDIR user, when I download transactions without applying filters, then I should receive an Excel file containing all transactions.
[ ] Given I am an IDIR user, when I download transactions with filters applied, then the Excel file should only include the filtered transactions.
[ ] Given I am a user, when I click the "Download as Excel" button, then the button should change to "Downloading..." until the download is complete.

Development Checklist:

[ ] Use SpreadsheetBuilder from backend/lcfs/utils/spreadsheet_builder.py for generating the Excel file.
[ ] Refer to the implementation in frontend/src/views/Organizations/Organizations.jsx. Specifically, examine DownloadButton and handleDownloadOrgs for guidance on integrating the download functionality.
[ ] Write front-end tests to validate the download button behavior, including state changes and file generation. Ensure tests cover both BCeID and IDIR user scenarios.
[ ] Create Pytest cases to test.
[ ] Develop Cypress e2e tests to simulate the complete download process for both user types.

Notes:

Hey team! Please add your planning poker estimate with Zenhub @AlexZorkin @hamed-valiollahi @kevin-hashimoto @prv-proton

Found the following issues:

The transactions are downloading on the BCeID side, but not on the IDIR side for all organizations (Only works if you filter by organization). We want to be able to download all transactions from all organizations.

Compliance period, category and effective date columns are not showing any information in the Excel sheet. We should also include an 'Approval date' column if possible.

Please see screenshot of Excel sheet:

@Grulin , Thanks, Lindsy, for testing and reporting the issues.

I fixed the routing issue on the IDIR side.
For the "Compliance period", I couldn't find any stored information in the database, so I added the column, but the values for the records will be empty. Please let me know if this information is already stored in the database.
Regarding "category," it already shows the transfer category, but there is no category stored for admin adjustment and initiative agreement in the database.
The issue with the effective date is now resolved. If there is a transaction effective date stored for a transaction in the database, it will be shown.
For the "Approval date," I couldn't find any information in the database. I added the column, but the values for the records will be empty. Could you please clarify what it means for different transactions (transfer, admin adjustment, and initiative agreement)?

Thank you

Hi Hamed,
Thank you for the notes. Your first 4 points make sense. I only just realized that compliance period likely has not bee established yet since the fuel supply reporting sections are not complete. I think this will be added later.
For the final point, I think we should drop the column "Approval Date" and relabel the "Effective date" column to read "Recorded/Approved Date" if possible?

Hi Lindsy,
Thank you for reviewing my comments and providing your feedback.

... we should drop the column "Approval Date" and relabel the "Effective Date" column to "Recorded/Approved Date" if possible?

I'm sorry, I'm not sure if I understand this part correctly. Let me share my understanding so you can correct me if I'm wrong.
We have three types of transactions: Transfer, Admin Adjustment, and Initiative Agreement. Each type has its own "Effective Date", which we store as a date and time.
Also, each transaction type has several statuses, and we create a history record whenever the status changes with a date/time.
Transfer:

Draft
Deleted
Sent
Submitted
Recommended
Recorded
Refused
Declined
Rescinded

Admin Adjustment:

Draft
Recommended
Approved
Deleted

Initiative Agreement:

Draft
Recommended
Approved
Deleted

So, should we drop the "Approval Date" and "Effective Date" columns, and add a "Recorded" column for transfers and an "Approved" column for admin adjustments and initiative agreements?
Thank you

Hi Hamed,
Sorry for the confusion. Yes, your proposal here makes sense. Please drop the "Approval Date" column, but please keep the "Effective Date" column for admin adjustments and initiative agreements. This date can differ from the approved date for the admin adjustments and initiative agreements. It comes from the grey box in the transaction, rather than the status change:

Please add a "Recorded" column for transfers and an "Approved" column for admin adjustments and initiative agreements. Ultimately this is the date we track in our market reports for transfers, so it makes sense to have these displayed in this Excel sheet.

Thank you for the clarification, Lindsy. I appreciate it. I'll apply the requested changes.

Hi Hamed. This download looks great. The columns are now correct, but I was hoping we could only show "Approved" and "Recorded" status transactions in the "Status" column. Sorry I didn't make that clear before. Thank you.


Port to Stable - teacher tool: Change block display

Port of #10202
@jwunderl I'll wait to check this in until after the offline app ships, since I didn't get it in for testing today and it does touch non-teacher-tool code.

you'll wanna rebase on https://github.com/microsoft/pxt/tree/stable11.1, that's the new stable branch


[Bug] - Can't login to Github and Google

Describe the bug
Using PWA or just the website, with cookies available, postwoman can't log me using Github connection method.
To Reproduce
Steps to reproduce the behavior:

Go to the key icon top right
Click on it to deploy connections methods
Click on Github
The popup opens and closes itself instantly, without get any connection

Expected behavior
Be connected using Google or Github
Desktop (please complete the following information):

OS: Windows 10
Browser Vivaldi (chromium based)
Version 2.11.1811.47

This is clearly an issue with Browser itself.


feat: improve readability and touchability

Summary of PR
Improves readability and touchability across the app.

Tests for unexpected behavior

ios and android

Time spent on PR
3 hours

Thanks for the PR! Which tickets does this PR resolve?

Thanks for the PR! Which tickets does this PR resolve? Edit: I see from the branch name it's #181, but can't tell if there are any others.

#183 (parent) of #181 and #182
I didn't know how to mention it without it accidentally closing the issues when merged

Are we moving away from custom fonts like OpenGurmukhAkhar and Noto? I might argue a case for OGA but curious generally.

Noto is the default on Android. What are your arguments for OGA? My only arguments for OGA would be if we cannot display all the subscript characters with the system font properly (need to test it). I'm not fussed about anything else.

Ah, don’t mind it closing the issues tbh, since we have them in the project board still and can reopen if they fail UAT.

On 12 Dec 2021, at 14:29, Bhajneet S.K. @.***> wrote:

Are we moving away from custom fonts like OpenGurmukhAkhar and Noto? I might argue a case for OGA but curious generally.
Noto is the default on Android. What are your arguments for OGA? My only arguments for OGA would be if we cannot display all the subscript characters with the system font properly (need to test it). I'm not fussed about anything else.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or unsubscribe.


Subresource scanning based on @ApiOperation

Per discussions on #925 with @fehguy , creating new issue.
Through a subresource, if a path is not given at root level (because it gets subresource'd from inheritance), the api will not show up in swagger. For example:
@Path("/api/v1")
@Api(tags = "V1 APIs - All", value = "/api/v1", description = "MDM Services")
@Produces(MediaType.APPLICATION_JSON)
@Consumes(MediaType.APPLICATION_JSON)
public class BaseResource {

@Path("/somePath")
    @ApiOperation(value = "Returns Some service", response = SomeResource.class)
    public SomeResource getSomeService() {
        return this.someService;
    }
}
@Api(tags = "V1 Tenant APIs", hidden = true, value = "/tenants", description = "MDM Base Service")
@Produces(MediaType.APPLICATION_JSON)
@Consumes(MediaType.APPLICATION_JSON)
public class SomeResource {
@GET
    @Path("")
    @CacheControl(maxAge = 3, maxAgeUnit = TimeUnit.SECONDS)
    @ApiOperation(value = "get tenants", notes = "Sort Order Values can be 'ASC' or 'DESC'", response = String.class)
    public Response getStuff(@ApiParam(value = DATASOURCE_HEADER, required = true)
{
}

had to provide @Path("") for the swagger scanner to pickup that api. Otherwise it did not recognize the inherited path

I feel that the first @ApiOperation :
@Path("/somePath")
@ApiOperation(value = "Returns Some service", response = SomeResource.class)
public SomeResource getSomeService() {
   return this.someService;
}

Maybe could be removed as you defined the @ApiOperation on the GET of the scanned @Path("/somePath") resource, which define:
@GET
@Path("")
@CacheControl(maxAge = 3, maxAgeUnit = TimeUnit.SECONDS)
@ApiOperation(value = "get tenants", notes = "Sort Order Values can be 'ASC' or 'DESC'", response = String.class)
public Response getStuff(@ApiParam(value = DATASOURCE_HEADER, required = true) { ... }

In this case, what documentation would shown? "Returns Some service", "get tenants" or both...
I think that its the same path with two documentation definitions

@jmarreroc Not sure I understand. The first @Path("/somePath") is basically returning a subresource while the second @Path("") is just for swagger to scan and pick up that specific subresource method (yes, with the same path). The difference is we don't provide an HTTP method when a subresource is returned. JAX-RS won't understand a @GET on a subresource (although I haven't tried it first hand but that's what i'd expect)
The "returns some service" api operation actually doesn't do anything from what I can see. It's just a way to let swagger know that this is a subresource and you need to go inside this subresource class to scan the remaining methods (and corresponding paths). So you will only see the documentation for "getTenants" (as expected)

Ok, so finally swagger shown only the documentation defined over the HTTP method. Thx!

That's what I found so far .. yes ! And no problem.

this is fixed in develop_2.0

@fehguy Thank you ! Which snapshot/release I can test this with ?

If you wait a few, there will be a formal release.  I'm verifying for 1.5.0-M2 right now.

sounds good .. let me know what release .. currently I am on 1.5.4-Snapshot

@shivangshah - it was indeed released under 1.5.0-M2.

@webron @fehguy I just tried with 1.5.0-M2. The bug is still there.

@shivangshah - this should be fixed in develop_2.0 now. Can you try?

@webron do you know which snapshot i can test it with instead of branch? (I might not be able to clone the branch and test :( )

@shivangshah I don't think we have a published snapshot for it yet, so you'd have to build it on your own.

@webron I see ok .. when I get a chance, I will build and try it out ..

Sorry for the late response @webron  @fehguy . I just got around testing this on 1.5.0 release and it still does not work .. :(

@shivangshah - I'm afraid that at this point you'd have to provide us with a sample project that mimics the problem for us to further investigate.

@webron Let me set one up and send you guys the link soon ..

Running into the same problem. Only the most trivial kind JAX-RS application seems to be supported.

Which version are you using?

We are using the swagger-maven-plugin version 3.1.3 that is using swagger version 1.5.4 to document our REST services. I hope it's not a bug of the maven plugin itself and should be addressed there.

Okay, after removing all the @Api and @Api(hidden=True) annotations it worked as intended. Looks like I was just misguided by google and old workarounds/examples still floating around. Sorry for the noise.
If someone finds this via google: Resource locator methods do not need any swagger-specific annotation, only @Path. Sub-Resources classes do not need any annotation at all (No empty @Path and no @Api and especially no @Api(hidden=true)). Swagger will pick them up when it finds a @Path annotated sub-resource locator method.
Here is a working example:
@Path("/")
public class RootResource {
	@GET
	@ApiOperation("Service info")
	public String info() {
		return "info";
	}

	@Path("/subresource")
	public SubResoure subResourceLocator() {
		return new SubResoure();
	}
}

public class SubResoure {
	@GET
	public String info() {
		return "sub";
	}
}


Filters are removing every item and crushes

Hi,
for first: great job, thanks!
But one small problem. I can't get filters to work.
I have filtering method like in demo:
filter: function() {             this.$refs.grid.filter('all');         },
Options like:
sortBy: 'by_nick',                 getSortData: {                     by_nick: function(user) {                         return user.nick.toLowerCase();                     },                     by_age: function(user) {                         return user.age;                     }                 },                 getFilterData: {                     all: function() {                         return true;                     }                 }
And html:
<isotope ref="grid" :list="users" :options='isotopeOptions()'>                 <div class="user" v-for="user in users">                     <user v-on:invite="eventInvite({{ Auth::user()->id }}, user.id)" :user="user" self="{{ Auth::user()->id }}"></user>                 </div>             </isotope>
Sorting is working, but filtering is removing all items, no matter what I will return in filter function.
I think thats why Im getting error:
Uncaught TypeError: Cannot read property 'vm' of undefined
Do you might help, whats going on?
BTW. while npm installation, gulp crushes on trying load node_modules/masonry, coz that lib is called node_modules/masonry-layout originally, i think :)

@adrian1207 , could you create a jsfiddle please? It is quite didficult to understand what is going on just with this code sample.

Ok, I found what was the problem.
Somehow, Isotope reqired in node before jQuery is different than the same require after jQuery. This is why module.exports from your plugin gave me an object with empty '__underlying_element'.
So this notatnition:
window._ = require('lodash');
window.vueIsotope = require('vueisotope');
window.$ = window.jQuery = require('jquery');
Vue.component('isotope', window.vueIsotope);

works perfectly, but this:
window._ = require('lodash');
window.$ = window.jQuery = require('jquery');
window.vueIsotope = require('vueisotope');
Vue.component('isotope', window.vueIsotope);

works, but while filtering gives error:
Uncaught TypeError: Cannot read property 'vm' of undefined
in 47 line of vue_isotope.js:
const res =  getItemVm(itemElement); return value.call(this, res.vm, res.index);
But anyway problem is solved. Thanks. :)

Thanks for reporting that you found a solution.
I am closing the issue.


Allow the optional creation of an IAM user (without secrets generated) that can be used to access the bucket

Allow the optional creation of an IAM user (without secrets generated) that can be used to access the bucket. The default should not be to do this. Make sure the permissions are setup to access the bucket using the grant on the IAM user. This should be a read-only user that can simply read files from the bucket.

This bas been completed and will be in the next release. There is a createReadOnlyUser option.


Add list loadbalancers function

Proposed Changes
Generated proto file from dpservice feature/grpc_list_lbs branch
Added function to request list of loadbalancers

Not needed after merger with dpservice https://github.com/ironcore-dev/dpservice/issues/445
Functionality merged directly in dpservice repo.


New python pipe interface needs locking mechanism

This needs to be investigated. One node multiple clients, maybe the ZMQ communication already handles it under the hood...


REST API doesn't work properly in master

It seems the porting work to Flask hasn't finished yet. I encountered one serious problem in submitting files, which was not expected in the master branch.
The code should be tested properly before being merged to master. https://github.com/cuckoobox/cuckoo/commit/65e4491a3c26434bec196a5eaecfccaadd93be71#diff-3d03530109e7e7620d069801fbdbedf5
Flask has a different interface in reading POST request. Here's a modified version that works for me. (I just need to submit file without any parameters. Thus I did not try parsing the parameters.)
@app.route("/tasks/create/file", methods=["POST",])
@app.route("/v1/tasks/create/file", methods=["POST",])
def tasks_create_file():
    data = request.files['file']
    package = request.form.get("package", "")
    timeout = request.form.get("timeout", "")
    priority = request.form.get("priority", 1)
    options = request.form.get("options", "")
    machine = request.form.get("machine", "")
    platform = request.form.get("platform", "")
    tags = request.form.get("tags", None)
    custom = request.form.get("custom", "")
    owner = request.form.get("owner", "")
    memory = request.form.get("memory", False)
    clock = request.form.get("clock", None)

    if memory:
        memory = True
    enforce_timeout = request.form.get("enforce_timeout", False)
    if enforce_timeout:
        enforce_timeout = True

    temp_file_path = store_temp_file(data.read(), data.filename)
    task_id = db.add_path(
        file_path=temp_file_path,
        package=package,
        timeout=timeout,
        priority=priority,
        options=options,
        machine=machine,
        platform=platform,
        tags=tags,
        custom=custom,
        owner=owner,
        memory=memory,
        enforce_timeout=enforce_timeout,
        clock=clock
    )

    return jsonify(task_id=task_id)

You're right, the new api.py is has yet to be finished. @botherder might pick up where you left off? Either that or put it in a new branch, as-is it's just completely broken.

I wonder if there's similar problem in process.py. I encountered the error message as below with the current version.
ERROR:sqlalchemy.pool.QueuePool:Exception during reset or similar
Traceback (most recent call last):
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/pool.py", line 631, in _finalize_fairy
    fairy._reset(pool)
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/pool.py", line 771, in _reset
    pool._dialect.do_rollback(self)
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/dialects/mysql/base.py", line 2519, in do_rollback
    dbapi_connection.rollback()
ProgrammingError: (2014, "Commands out of sync; you can't run this command now")
    util.raise_from_cause(err)
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/util/compat.py", line 199, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb)
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/orm/loading.py", line 51, in instances
    for query_entity in query._entities
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/orm/query.py", line 3311, in row_processor
    polymorphic_discriminator=self._polymorphic_discriminator
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/orm/loading.py", line 313, in _instance_processor
    getter = result._getter(col)
  File "/home/xuweilin/.local/lib/python2.7/site-packages/sqlalchemy/engine/result.py", line 495, in _getter
    return self._metadata._getter(key)
AttributeError: 'NoneType' object has no attribute '_getter'
Traceback (most recent call last):
  File "./utils/process.py", line 149, in <module>
    main()
  File "./utils/process.py", line 140, in main
    autoprocess(parallel=args.parallel)
  File "./utils/process.py", line 120, in autoprocess
    pool.join()
  File "/usr/lib/python2.7/multiprocessing/pool.py", line 460, in join
    assert self._state in (CLOSE, TERMINATE)
AssertionError

If I'm not mistaken I've fixed this particular error in the monitor branch along with an attempt to fix some other errors as well.

Yes, I'm still to finish the Flask port. Working on it.

This should be fixed now.

Thanks @jbremer  and @botherder .
@jbremer , when are you going to merge the monitor branch? If it will not be soon, I would prefer keeping the old stable version of master in my running instance.


document "replicate hardware list"

Creating a model requires choosing hardware. This little tip should make it easier for model authors to figure out what hardware they can use.

Nice. Thanks, @zeke!
Sometime soon, we can make this subcommand interactive, so you can pick from a list.


fix(next): fixed issue for multi pages next applications

Proposed changes
Fixed an issue that was clearing the morfeo.css file for multi-pages apllications in nextjs
Types of changes

[x] 🐛 Bugfix (non-breaking change which fixes an issue)

Checklist

[x] 👀 I have read the CONTRIBUTING doc
[x] ✅ Lint and unit tests pass locally with my changes
[x] 🧪 I have added tests that prove my fix is effective or that my feature works
[x] 📚 I have added the necessary documentation (if appropriate)
[x] 🔀 Any dependent changes have been merged and published in downstream modules

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (028623d) 100.00% compared to head (2493889) 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##              main      #702   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          100       100
  Lines          879       879
  Branches       158       158
=========================================
  Hits           879       879

Impacted Files
Coverage Δ

packages/compiler/src/utils.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


InputDecoration.errorMaxLines's documentation does not match behavior

Steps to reproduce

Use a TextField
Add errorText
Don't define errorMaxLines in TextField.decoration

Expected results
Not providing a value will wrap the error text by default.
Actual results
Error text is truncated.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(
    const MaterialApp(
      home: const HomePage(),
    ),
  );
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Padding(
          padding: const EdgeInsets.all(32.0),
          child: TextField(
            decoration: InputDecoration(
              errorText: 'Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua. At vero eos et accusam et justo duo dolores et ea rebum.',
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel master, 3.26.0-1.0.pre.170, on Void Linux 6.6.51_1, locale en_US.UTF-8)
    • Flutter version 3.26.0-1.0.pre.170 on channel master at $HOME/.local/lib/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 38105edec4 (4 days ago), 2024-09-20 13:04:24 -0400
    • Engine revision 76b7abb5c8
    • Dart version 3.6.0 (build 3.6.0-269.0.dev)
    • DevTools version 2.40.0-dev.1

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at $HOME/.local/lib/android-sdk
    • Platform android-35, build-tools 35.0.0
    • Java binary at: $HOME/.local/share/JetBrains/Toolbox/apps/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/chromium

[✓] Linux toolchain - develop for Linux desktop
    • clang version 17.0.6
    • cmake version 3.30.1
    • ninja version 1.11.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2024.1)
    • Android Studio at $HOME/.local/share/JetBrains/Toolbox/apps/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

[✓] IntelliJ IDEA Community Edition (version 2024.2)
    • IntelliJ at $HOME/.local/share/JetBrains/Toolbox/apps/intellij-idea-community-edition
    • Flutter plugin version 81.1.3
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Void Linux 6.6.51_1
    • Chrome (web)    • chrome • web-javascript • Chromium 129.0.6668.58

[✓] Network resources
    • All expected network resources are available.

• No issues found!

I see the errorText is TextOverflow.ellipsis by default:
https://github.com/flutter/flutter/blob/e0ca1946601dd7c0aafe3301f6d06ff03c860669/packages/flutter/lib/src/material/input_decorator.dart#L447
On the linked PR https://github.com/flutter/flutter/pull/147409, I see the errorMaxLines docs is changed:
From:

/// Defaults to null, which means that the [errorText] will be limited
/// to a single line with [TextOverflow.ellipsis].

To:

/// Defaults to null, which means that the [errorText] is not limited.

Flutter versions:
Channel stable, 3.24.3
Channel master, 3.26.0-1.0.pre.200

/cc @bleroux for thoughts

My attempt:
Defaults to null, which means that soft line breaks in [errorText] are truncated with an ellipse while hard line breaks are respected. For example, an [errorText] that overflows the width of the field will be truncated with an ellipse. However, an [errorText] with explicit linebreak characters (\n) will display on multiple lines.

How would someone in this instance make the errorText wrap always? (aside from explicitly setting maxLines)

Good question, the answer is that there is no other solution than setting maxLines because errorText overflow behavior is hardcoded to TextOverflow.ellipsis:
https://github.com/flutter/flutter/blob/e0ca1946601dd7c0aafe3301f6d06ff03c860669/packages/flutter/lib/src/material/input_decorator.dart#L447
The hardcoded overflow is there since... almost forever :smile: : see https://github.com/flutter/flutter/pull/13734
https://github.com/flutter/flutter/pull/147409 was only a documentation change.
I don't think we should introduce an errorTextOverflow property, because:

there are already so many properties (and we will also have to introduce helperTextOverflow). And then it will open the door to errorTextAlign or maybe other properties.
InputDecorator.errorText is just a handy alternative to InputDecorator.error which offers more flexibility, for instance it can be set to a Text widget with a specific overflow value.

If we agree to not add such a parameter, what do you think of mentioning that 'always wrap' can be achieved by setting errorMaxLines? (problem is that there is no exact value to advice for, it should just be higher enough, and 'higher enough' depends on the particular errorText and the minimum width of a particular text field).

Based on our discussions, I filed https://github.com/flutter/flutter/pull/156468


quiche: implement SslConnectionInfo for QUIC

Signed-off-by: Dan Zhang<EMAIL_ADDRESS>Commit Message: pass boringSSL SSL object from QUICHE to Network::Connection::ssl() interface.
Risk Level: low
Testing: added unit tests and integration test

/retest


⚠ Redacted has degraded performance

In 08aec74, Redacted ($RED) experienced degraded performance:

HTTP code: 200
Response time: 3939 ms

Resolved: Redacted performance has improved in 9bdbe26.


When trying to paste the code: Can be used only on Instagram routes

When I try to paste the code, this message appears: Can be used only on Instagram routes.
Please note that I'm logged in on Instagram.
Thanks

When I try to paste the code, this message appears: Can be used only on Instagram routes.
Please note that I'm logged in on Instagram.
Thanks

Its strange, can you provide some photo? Its working for me at least..


Incorrect typings of Snackbar positions

Im trying to change position but im not able to.
Error message:
Type 'string' is not assignable to type 'Readonly<{ TOP_LEFT: string; TOP_MIDDLE: string; TOP_RIGHT: string; MIDDLE_RIGHT: string; BOTTOM_RIGHT: string; BOTTOM_MIDDLE: string; BOTTOM_LEFT: string; MIDDLE_LEFT: string; }>'.
Please move to this to discussion if this invalid issue.

I'm unable to diagnose the problem from what you've sent. Could you please share the rest of the code (ideally, as code-formatted text and removing as much code as possible such that the issue still persists)?

// example.svelte
<script lang="ts">
  import { SnackbarContainer, Button } from "attractions"
  import { SnackbarPositions } from "attractions/snackbar"
  let snackbar
  let position = SnackbarPositions.BOTTOM_LEFT
  const handleClick = () => {
    snackbar.showSnackbar({ props: { text: "Hello!" } })
  }
</script>

<Button on:click={handleClick} />
<SnackbarContainer {position} bind:this={snackbar} />

Problem:
Type 'string' is not assignable to type 'Readonly<{ TOP_LEFT: string; TOP_MIDDLE: string; TOP_RIGHT: string; MIDDLE_RIGHT: string; BOTTOM_RIGHT: string; BOTTOM_MIDDLE: string; BOTTOM_LEFT: string; MIDDLE_LEFT: string; }>'.
Probably typescript support? It actually works without lang="ts" on the script tag.

@paradis-A Yes, it was indeed a TypeScript problem from our side. Your code is totally correct, but we had the wrong typings. It should be fixed now with the v3.2.1 update.


Change test FTP server address

Changes ftp server used by the ftp staging test from ftp://ftp.cs.brown.edu which it turned out to be not too reliable to ftp://ftp.iana.org/.

👏  Thanks @lukaszlacinski


tasks don't get reconnected to when using a remote

This still seems present for me on web:
https://user-images.githubusercontent.com/2230985/186737984-ea6be9ba-127d-495e-a64c-364074ee8732.mp4
(disregard the long reload time--running the CLI in verbose debug mode 😛 )
Originally posted by @connor4312 in https://github.com/microsoft/vscode/issues/156284#issuecomment-1227602658

Yep, confirmed this happens with devcontainers too (and works on desktop)

probable cause - one of these is false
https://github.com/microsoft/vscode/blob/c416626b45b78bfa256b635999cdc70d94b8adf4/src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts#L2262-L2264

maybe have to do something like what was done for auto-run tasks in the web
https://github.com/microsoft/vscode/commit/55e23b504b4fd869c27a1f181644415d0a4b0ea0

when using code-server the logs show this, so they're not getting restored correctly because there aren't reconnectionProperties
[2022-08-25 12:53:17.306] [window] [trace] terminalInstance#ctor (instanceId: 1) {"attachPersistentProcess":{"id":3,"title":"Core - Build","titleSource":0,"pid":20272,"workspaceId":"-5fcd6d38","workspaceName":"vscode [meg]","cwd":"C:\\Users\\merogge\\repos\\vscode","isOrphan":true,"environmentVariableCollections":[["vscode.git",[["GIT_ASKPASS",{"value":"c:\\Users\\merogge\\.vscode-cli\\server-insiders\\bin\\9529e11f6481ae53bba821b05e34549491b9415e\\extensions\\git\\dist\\askpass.sh","type":1}],["VSCODE_GIT_ASKPASS_NODE",{"value":"C:\\Users\\merogge\\.vscode-cli\\server-insiders\\bin\\9529e11f6481ae53bba821b05e34549491b9415e\\node.exe","type":1}],["VSCODE_GIT_ASKPASS_EXTRA_ARGS",{"value":"","type":1}],["VSCODE_GIT_ASKPASS_MAIN",{"value":"c:\\Users\\merogge\\.vscode-cli\\server-insiders\\bin\\9529e11f6481ae53bba821b05e34549491b9415e\\extensions\\git\\dist\\askpass-main.js","type":1}],["VSCODE_GIT_IPC_HANDLE",{"value":"\\\\.\\pipe\\vscode-git-b2074c1322-sock","type":1}]]]]}}

the window logs aren't available when using the code-server, saying:

but in the remote pty host, I see this message 🤔
[warning] Persistent process "1": Process had no disconnect runners but was an orphan
[2022-08-26 06:31:37.365] [ptyhost] [info] Persistent process reconnection "1"
[2022-08-26 06:31:37.365] [ptyhost] [warning] Persistent process "2": Process had no disconnect runners but was an orphan
[2022-08-26 06:31:37.365] [ptyhost] [info] Persistent process reconnection "2"
[2022-08-26 06:31:37.451] [ptyhost] [info] Persistent process "1": Replaying 8729 chars and 1 size events
[2022-08-26 06:31:37.477] [ptyhost] [info] Persistent process "2": Replaying 9415 chars and 1 size events

the call to setLayoutInfo doesn't contain the info needed for task reconnection
[ptyhost] [trace] ptyService#setLayoutInfo [{"isActive":true,"activePersistentProcessId":5,"terminals":[{"relativeSize":0.5004508566275925,"terminal":4},{"relativeSize":0.4995491433724076,"terminal":5}]}]

they're not provided here
https://github.com/microsoft/vscode/blob/6ec6d9d94dc37e138009d59a33a57d13d142b853/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts#L1315-L1316
or here
https://github.com/microsoft/vscode/blob/6ec6d9d94dc37e138009d59a33a57d13d142b853/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts#L1325-L1326

@connor4312 today in insider's, when I run the code server task, tasks reconnect correctly there for me. do they for you?

/closedWith 212af8702db15a621195906375ab699044cedf4c

Tasks still aren't reconnecting for me when I use vscode-remote-try-node (Node.js devcontainer) and set up typescript along with a watch task there.
tasks.json
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "label": "watch",
            "type": "shell",
            "command": "npx tsc -w",
            "problemMatcher": {
                "base": "$tsc"
            }
        }
    ]
}

@rzhao271 do normal terminals reconnect? There's some nuance around when different servers get taken down and want to make sure that's not the reason.

Normal terminals don't reconnect for me on remote-containers.
I was able to verify the issue using the VS Code repository with code-cli and insiders.vscode.dev!


could you please Supply a full package

downloading offen crash,its fidgety.

What do you mean by "a full package". There are links to download to necessary files in README.

This issue was closed because it has been inactive.


NEXT-14809 Gallery image aspect ratio zoom config (tall images fix) and magnifier over gallery config

1. Why is this change necessary?
On tall images it's impossible to zoom the bottom edge of the image (this happens also on very long images). Also when you change layout of the product the magnifier over gallery is forced due to hardcoded value in the Shopware Core Code.

2. What does this change do, exactly?
Adds two options to the Gallery config modal in the admin that allows you to change the default Shopware behavior:

Keep aspect ratio on zoom (default checked to keep default shopware functionality)
Magnifier over gallery (default unchecked to keep default shopware functionality)

This PR keeps default shopware functionality

When Keep aspect ratio on zoom is disabled then the zoom overlay becomes square and user is able to reach the bottom edge of the image

In addition user can decide what to do with the magnifier over gallery option. Currenty in Shopware there is a bug that forces magnifier over gallery option as soon as user changes default product layout to the custom one. This PR adds checkbox to control that option.

3. Describe each step to reproduce the issue or behavior.
Case 1

Create a product and add very tall image
Go to the product page
Put mouse over gallery image and check zoom on the right

Current behavior: It's impossible to zoom the bottom edge
Expected behavior: User should be able to zoom even the bottom part. Since the right area is rather square the zoom overlay should be also square. The behavior should be configured in the admin.
Case 2

Create a new product layout and assign it to the product
Go to the product page
Put mouse over gallery and check what happens

Current behavior: Zoom is shown over the gallery but in the default shopware product template it was over buy area (right side)
Expected behavior: User should be able to change that behavior in the admin - currently it's hardcoded in the shopware code
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-14809
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Thanks for your Contribution! 💙


Add data caching

Implement data caching for logged in user. All information should be cached. On loading data first cached information is shown, then updated data from the network.
For storing data locally use realm

Because of the purpose of the app is getting actual build data there is no any sense to cache them.
User should always be able to download and check the latest state of the builds/jobs.


Update Subscriber source's configurations to remove ambiguity.

Update Subscriber source's configurations to remove ambiguity. Otherwise, a user may think that "subscription does not exist" means that the "subscription field is empty/unspecified".

Squashed commits and merging.


Unsupported video codec

Hello,
Demo code is not working and it gives "Error Domain=YTVimeoExtractorErrorDomain Code=2 "Unsupported video codec" " error. Can you please look into this?
Thanks.

Hi @Nisha-SpaceoTech,
If your are using the master branch or the latest release then it's related to issues #24 #25.
Regards

yes got it.
Thanks


Links to https://dropbox.com/sh/example/thing in stream descriptions fail to save

The new call to is_static_or_current_realm_url assumes that self.md.zulip_realm is defined:
https://github.com/zulip/zulip/blob/ec99c1b58dc319fa3eaeaa883f6e18ed1add425c/zerver/lib/markdown/__init__.py#L584
This is not set for markdown descriptions of streams.  As such, putting a URL of https://www.dropbox.com/sh/something/here into a stream description causes the description to fail to save, as rendering the markdown throws a 'NoneType' object has no attribute 'host' error:
https://github.com/zulip/zulip/blob/ec99c1b58dc319fa3eaeaa883f6e18ed1add425c/zerver/lib/subdomains.py#L73
We should:

Error-proof is_static_or_current_realm_url by fixing the type of realm to be Optional[Realm] and add realm is not None and  before the check on line 73
Fix do_change_stream_description to pass down the realm to the markdown renderer, so that we do get a realm in that spot, and can make use of it.

Do we need to do 1 if we ensure that all callers pass a realm?  I'm not sure that we actually have a use case for markdown rendering outside the context of a realm, except perhaps in some sloppy tests.
(We should make sure we check realm_description as well for the same bug)

1 isn't necessary if we actually cover all callsites, no.
I'm a little surprised that mypy didn't catch the Optional violation.  If we remove the Optionality of realms of the markdown renderer, that might catch other callsites.

Mypy didn’t catch this because the types-markdown stubs just say md: Any.

Ah, OK.  And, in fact, it'd probably be an error to reference md.zulip_realm because there's no way to know that's a zerver.lib.markdown.Markdown not a markdown.Markdown.

@PIG208 would you be up for looking into this one?

OK, looking into this.

Note that part of this (typing and None-checking) has been fixed in #23159. #23386 is mainly about fixing do_change_stream_description


find out and implement how to get cards (JSON against humanity) in the backend

Task List:

[ ] Read JSON File
[ ] create instances of cards white and black

Reading JSON file is still problematic. I tried to use relative path, so that it works on every device, regardless where the repo is. However, the system cannot find the path. I will work on absolute path to make some progress and then I can come back to path issue.


Add shield to tab row when elevated

Summary of the Pull Request
Adds a visible indicator that a Terminal window is elevated. This icon can be disabled with "showAdminShield" false in the global settings.
References

spec'd in #8455
Also in https://github.com/microsoft/terminal/projects/5
big picture: #5000

PR Checklist

[x] Closes #1939
[x] I work here
[n/a] Tests added/passed
[ ] Requires documentation to be updated - yea probably

Validation Steps Performed

Don't forget to update the docs/schema!

Don't forget to update the docs/schema!

frick I did, just didn't push lol

I think this breaks max and min width in showTabsInTitlebar=false mode.

I think this breaks max and min width in showTabsInTitlebar=false mode.

What the devil, this is broken even for unelevated, tabsInTitlebar windows:

WEIRD WEIRD WEIRD
If you do it with a StackPanel, the StackPanel will overflow without a care in the world.
If you do it with a Grid though, then it will obey the MaxWidth. Weird.

Came up with a better solution:

Totally forgot about TabView.TabStripHeader, which is exactly what it sounds like, and does exactly what we want.

qq, should we make it less "bright"? right now it's the primary text color which is very bright on dark displays; we could tone it down a bit to maybe the SystemAltMediumColor?

qq, should we make it less "bright"? right now it's the primary text color which is very bright on dark displays; we could tone it down a bit to maybe the SystemAltMediumColor?

ez

WEIRD WEIRD WEIRD
If you do it with a StackPanel, the StackPanel will overflow without a care in the world.
If you do it with a Grid though, then it will obey the MaxWidth. Weird.

#justxamlthings

In this one, the shield looks a few pixels too high:

In this one, the shield looks a few pixels too low:

I feel like it should be centered on the same vertical center line as the content of the tab...

Do tend to agree about the spacing! fix it in post?


How do I train SSD-MobileNet-v2 with my custom data?

Sorry for asking here, but I really can't find anything useful about training SSD-MobileNet-v2 to detect my own stuffs. Can anyone please provide something about it? Great thanks!

Hi Leo, for TensorFlow see here - https://forums.developer.nvidia.com/t/digits-or-somthing-else/110790/7?u=dusty_nv
For PyTorch, see github.com/dusty-nv/pytorch-ssd

From: 66Leo66<EMAIL_ADDRESS>Sent: Saturday, May 2, 2020 9:50:20 AM
To: dusty-nv/jetson-inference<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [dusty-nv/jetson-inference] How do I train SSD-MobileNet-v2 with my custom data? (#563)
Sorry for asking here, but I really can't find anything useful about training SSD-MobileNet-v2 to detect my own stuffs. Can anyone please provide something about it? Great thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-inference/issues/563, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVEGK53TYJMCURAY6HYGALRPQQJZANCNFSM4MXWC4CQ.

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.

Will SSD-Mibilenet-v2 work with Nvidia Digits?


Mood board generator

Name : Vyom Chaturvedi
Participant Role: GSSOC
<EMAIL_ADDRESS>

@vyom10445 mention the issue no

@vyom10445 mention the issue no

hey can you elaborate. i am new :)

got it but next time raise the issue in issue tab if issue is assign to you then only make a pr request


[LOC] "アプリ & 機能" needs to be replaced with "アプリと機能"

https://docs.microsoft.com/ja-jp/windows/application-management/add-apps-and-features

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: ecb7910b-900c-49cb-46ae-0c1de7cf3df6
Version Independent ID: 5f7b8dba-d1df-95b0-8e54-32a729a5cf27
Content: Windows 10 - アプリ & 機能からアプリを追加する方法
Content Source: windows/application-management/add-apps-and-features.md
Product: w10
Technology: windows
GitHub Login: @lizap
Microsoft Alias: elizapo

@officedocsbot assign @mypil
@officedocsbot assign @dongmicro
@officedocsbot add label localization

@dongmicro - Can you please assist with this Translation (Localization) issue?

The replacement could be achieved easily by reusing the と character from the page title.
Then again, I don't know if this needs to be worked on in some kind of automated translation process.

See: #3159

@officedocsbot unassign @mypil

@officedocsbot assign @e0i

@hyoshioka0128
It has been confirmed via localization team that this specific issue has been fixed. Thanks for your feedback.

@officedocsbot close


Add FrontendHostPort method to DevServer

What was changed
Added a new FrontendHostPort method to testsuite.DevServer that returns the host:port to the dev server, similar to Temporalite (here).
Why?
This would be useful when the client.Client can't be shared with the unit being tested, e.g. a Command struct that only accepts os.Args, environment strings or other form of configuration.

@cretz @Quinn-With-Two-Ns: apologize for the late changes, but I've made some additional updates after discovering a couple of things:

The tests in testsuite/devserver_test.go were not running in CI. I've fixed this in 9f63d87 - I moved the file to /test, the root of the integration tests and confirmed that they run now when using make integration-test-zero-cache and make integration-test-normal-cache. Previously, the test was not included in make unit-test because INTEG_TEST_ROOT matched both test/ and testsuite/.
The tests in testsuite/devserver_test.go were failing due to the way defer was being used to stop the server and how Go evaluates the statement. Fixed in 2eeb22b.

The tests in testsuite/devserver_test.go were not running in CI

I don't think we should move them, we should just do what is necessary to make them work with make unit-test

The tests in testsuite/devserver_test.go were failing due to the way defer was being used to stop the server and how Go evaluates the statement

Can you clarify this a bit? We want defer to work and we want these tests to show a normal way of starting and defer-stopping the test server instead of using Cleanup. Do we need to add exit code error checking to Stop()?

I don't think we should move them, we should just do what is necessary to make them work with make unit-test

Cool. I can do that, I thought it was integration after all because it's downloading the temporal-cli binary and running it - but maybe that can be considered an unit test.

Can you clarify this a bit? We want defer to work and we want these tests to show a normal way of starting and defer-stopping the test server instead of using Cleanup. Do we need to add exit code error checking to Stop()?

defer server.Stop() works, but defer require.NoError(t, server.Stop()) doesn't because the function parameters are evaluated right away (https://go.dev/play/p/Gf0A8g7FCwU). We don't have to use t.Cleanup - it's a habit of mine when testing, but we'd need to combine defer with a function literal, e.g.: https://go.dev/play/p/ytn5y_6ygPA.

Do we need to add exit code error checking to Stop()?

Not really, maybe that's actually a form of leaking that should be avoided. But I thought it'd be nice to look up the actual error since that's what we shoudl see when using cmd.Wait().

Thanks, @cretz! I'll get to it as soon as possible.

Opened https://github.com/temporalio/cli/issues/175 for issue where exit code is not always 0 here

Done.

Awesome, thanks for the contribution.


[Help Wanted] Custom mapping doesnt work for collection types?

Hello, Im having an issue where custom mapping doesnt seem to work for collection types. I have the following class in java and have added UUID to the custom map with value "uuid".
public class A(){
   UUID id;
   Set<UUID> manyIds;
}

The output is
"id" :{
   "type" : "string",
   "format" : "uuid"
 },
 "manyIds" {
    "type" : "array",
    "items" : {
        "type" : "String"
     }
  }

The problem is that I expect "format" : "uuid" to be a part of "items" for "manyIds". How do I get "format" : "uuid" associated with all uuid's in collection types? Any help would be appreciated, thanks!

Just wanted to follow up on this and say I figured it out. Before making the first post, I did try using injection and providers but neither would work for "manyIds". The problem was that the setter/getter function names for manyIds did not follow java convention and was annotated with @JsonProperty to get the desired field name. Simply fixing the function names enabled injections and providers to work as expected.


wine-{unstable,staging}: 4.9 -> 4.12.1

Motivation for this change
simple update
Things done
steam runs, successfully tested some games e.g.: Borderlands

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I'm going to try building the closure, and notify maintainers
CC @avnik @7c6f434c  @bendlas


In organ.c, the variable "wavelength" should called "period" or "timePeriod"

In the file organ.c in Chapter05_Serial-IO, there is a variable called wavelength that is the reciprocal of frequency. However, in every day usage, "wavelength" refers to the the distance between successive crests of a wave (wavelength = v/f, where v is the velocity and f is the frequency), and therefore wavelength is not equal to the reciprocal of frequency. On the other hand, the "period" of a wave is equal to the reciprocal of the frequency: T = 1/f.
Therefore, to increase code readability, in organ.c, the variable "wavelength" should called "period" or "timePeriod".

As a sound engineer by trade, I agree that 'wavelength' was a bit confusing at first.  At the same time, however,  a term like 'period' might seem discouraging to the music production kids.
How about "frequency"?  Its a comfortable word,  and I believe correct by definition?
/Etienne

Hi rithma, I appreciate your input on this manner. My issue with using "frequency" is that this variable we are talking about is not frequency, it is the inverse of frequency 1/f. That is why I suggested using "period", since the period T = 1/f in engineering.

I see.   Since the c function determines the duration of each cycle, 'period' would be a good name.
I need to stop thinking in Layman's terms, now that I'm learning code....

aararonsky, you're totally right.  (And since I wrote the code a few years ago, I have no idea why I chose wavelength over period -- it's not like I don't know the difference...)
If it's confusing for someone, it's worth changing.  I'll do that right away.

... and done!  Thanks again.


Partially completed query state not captured by SDK

Over here we don't have the query state QUERY_STATE_COMPLETED_PARTIAL.
May be related to #118 (allow partial flag).

@bh2smith is this on the roadmap? I ran into some issues with this missing state

I am sure it will be done at some point, although I don't have capacity for this at the moment. Maybe @waddahAldrobi?
We also welcome community contributions!

Hey guys, I faced the QUERY_STATE_COMPLETED_PARTIAL issue as well. I was wondering if that's an issue or if the query is supposed to keep going if there was a state for it
ValueError: 'QUERY_STATE_COMPLETED_PARTIAL' is not a valid ExecutionState

So then partial would also constitute a terminal state. I can add that to the PR logic.

Oh... so your "run_query" is failing because its a long running query and we are unable to parse the type so it fails! If this is a major blocker we can include the new query execution state without actually making use of it (so the program doesn't crash because of a parse error). I opened a PR #125.
If this is blocking your complex query from running (i.e. crashing the program), let me know and I can get a hot-fix out ASAP.

Yes, that's exactly right together with @RichardKeo 's comment. I am getting some transaction hashes for a bit of a long period using the API. I was thinking I would get partial results similar to using the UI. It's right now a blocker. Well can be considered a warning as well indicating I need to tighten my limits.

So then partial would also constitute a terminal state. I can add that to the PR logic.

I feel like it should be similar to the UI. Let's get the output out and show a warning this is a partial result. Is that what you mean as well?

hey folks any update on this issue? this is a major blocker for us

Sorry about the wait on this.
I have revived and dusted-off the proposed PR #125, merged and released under v1.7.5. Please let me know if you encounter any issues.
Note that this is an open source repo and could have taken this over at any point.


[ISSUE #7662] fix: updateAccessConfig when groupPerms is empty

Which Issue(s) This PR Fixes
Fixes #7662
Brief Description
org.apache.rocketmq.common.UtilAll#join return "" when list is empty,
When I updateAccessConfig,  the method UtilAll.split(requestHeader.getGroupPerms(), ",")
Returns a collection of empty strings.
The method  org.apache.rocketmq.acl.common.Permission#checkResourcePerms will not running well.
How Did You Test This Change?
yes, I updateAccessConfig successfully.

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

Comparison is base (6fce427) 43.21% compared to head (9c2e9f8) 43.18%.
Report is 9 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7664      +/-   ##
=============================================
- Coverage      43.21%   43.18%   -0.04%
- Complexity      9815     9818       +3
=============================================
  Files           1165     1165
  Lines          84548    84548
  Branches       10980    10980
=============================================
- Hits           36534    36508      -26
- Misses         43472    43501      +29
+ Partials        4542     4539       -3

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


Fee-market improvement

Issue
For now design, we reuse the relayer maker fee in order.
So if relayer assign and lock their collateral. They can not change their Maker Fee at any time.
Improvement
Store the relayer's maker fee in order.
And relayer could change their Maker Fee at any time.
Disadvantages
More storage will increase more gas usage.

Yes, that's an issue, relayer's make fee should be recorded in order after matched/assigned.


read new tag in following steps

is there a way to read the new tag when a changelog+bump are created in a following step?
I tried, but the repo is always the origina, and I don't see the modification.
Thanks a lot
Angelo

Could you expand? What do you mean by the repo is always the original? Inside the pipeline or you don't see your tag reflected in the repository? Do you get any error?
In the https://github.com/commitizen-tools/commitizen-action#sample-workflow there's an example on how to print the version in a following step

ok, you solved me with the output. but of course, if you read the last commit, it is not the bump commit, but the previous.
Thanks a lot


Merge fix-mac-tests into main: Correct tests to work with mac newlines

Fix direct use of newlines in tests

coverage: 95.936%. remained the same
when pulling 6efe8ed45dd387cc36a5901280242de0f7d0db34 on fix-mac-tests
into 241117c5a2c73d30baea789a7cfa4231139a1019 on main.