⚠ Fetchers has degraded performance

In 2c7a80e, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 304 ms

Resolved: Fetchers performance has improved in 90b8097.


Fix build of unprivileged rawhide test container

useradd is no longer pre-installed, request it.
Pull Request Checklist

[x] implement the feature

Thanks, I was worried the fix could be affected by https://fedoraproject.org/wiki/Changes/DNFConditionalFilelists but it isn't: "Note that after this change, users can still use DNF without filelists metadata when querying file provides located in /usr/bin, /usr/sbin or /etc directories. "

I was so happy when I wasn't aware of this :) Glad it worked locally, I hope it will pass all the tests and we can move on to something less dynamic.

/packit test

/packit build


Provide nullability information for data models used in the API

For statically typed languages which differentiate between null and non-null types such as Kotlin, it gets difficult to handle null values in fields where we previously expected a non-null value.
For example, the Launch data for Starlink 5 right now looks like this:
{
    "flight_number": 91,
    "mission_name": "Starlink 5",
    "mission_id": [],
    "launch_year": "2020",
    "launch_date_unix":<PHONE_NUMBER>,
    "launch_date_utc": "2020-02-14T00:00:00.000Z",
    "launch_date_local": "2020-02-13T18:00:00-06:00",
    "is_tentative": true,
    "tentative_max_precision": "day",
    "tbd": false,
    "launch_window": null,
    "launch_site": {
      "site_id": null,
      "site_name": null,
      "site_name_long": null
    },
    ...
  },

All the fields in "launch_site" are null, which started causing a crash in my app. I would really love to have a schema of the data models returned by this API so that I don't have to fix null fields previously marked as non-null.
The schema should be adhered-to by the API, and the client apps, guaranteeing that the data shall be sent in that particular format only. The schema should include nullability information too.
Having a solid schema will also encourage people to write wrapper libraries for this API in different languages, and make life easier for developers of existing apps.

Currently, it's possible for any field to be nullable, depending on the information available.
If it's an array, the default state is an empty array [], otherwise null.
Wish I had a hard schema to give you, but each launch tends to have a different set of null possibilities.

As far as the future goes, I'm working on a new relational data model for the project, and exploring a rewrite of the api in a statically typed language.
Going to create a pinned issue for that over the weekend, with a potential rough draft of the schema to iterate on.
I'll put all the other details in the issue, but this should solve most of the existing shortcomings the api currently has 👍

Are you looking at Typescript, or something completely different than the JS ecosystem?

Go is the most likely candidate, but definitely moving away from Mongo as the db.

Will this new version be v4? Are there any more features planned for this release? Is there a ETA?

see #296


enhanceMTMV support session variable

MTMV support set session variable for task by set property starts with "session."


Support redux-thunk by accepting functions as actions

We use redux thunk and are therefore sending functions as actions sometimes. (https://github.com/gaearon/redux-thunk). We use your mock (thanks!) but would like it to also accept functions apart from object with type attribute. Would you like such a pull request?

Never mind. Saw the better solution (had old version)


Removed Jeweler and created modern .gemspec.

The cocoon.gemspec generate by Jeweler was including the rubysl gems. This
causes cocoon to fail to install on MRI right now (see eg this stackoverflow
post: http://stackoverflow.com/questions/20390594/rails-gem-install-error-coccon)
Jeweler was also emitting redundant dependencies for rspec and emitting a
dependency for actionpack despite that the dependency on rails would pull
in actionpack. Also, Jeweler was emitting version specs like ">= 4.0.0", which
is dangerous as there is no upper bound on the spec.
To replace the Jeweler-generated cocoon.gemspec, I added one created by
Bundler and migrated the settings over. It is a simple matter to keep this
gemspec current by hand.
Note that where Jeweler had emitted version specs like this [">= 0"], I did not
add a version spec in the new gemspec. This should probably be addressed.

Cleaning up (closing) old PR's. Thank you very much for your effort!


Remove requestOptions2GotOptions compatibility layer; run [*****]

Refs #4655
This PR is a bit of a beast but I've broken it down into smaller chunks as far as possible to make it easier to follow. Unfortunately 49476e30 is still really big but it was unavoidable.

Warnings

:warning:

:books: Remember to ensure any changes to config.private in services/nexus/nexus.service.js are reflected in the server secrets documentation

:warning:

:books: Remember to ensure any changes to config.private in services/sonar/sonar-base.js are reflected in the server secrets documentation

:warning:

:books: Remember to ensure any changes to config.private in core/base-service/base.js are reflected in the server secrets documentation

:warning:

:books: Remember to ensure any changes to config.private in services/jenkins/jenkins-base.js are reflected in the server secrets documentation

:warning:

:books: Remember to ensure any changes to config.private in core/base-service/auth-helper.spec.js are reflected in the server secrets documentation

:warning:

:books: Remember to ensure any changes to config.private in services/teamcity/teamcity-base.js are reflected in the server secrets documentation

:warning:
This PR modified service code for wordpress but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for wikiapiary but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for wercker but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for w3c but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for ubuntu but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for twitter but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for twitch but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for treeware but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for travis but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for swagger but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for stackexchange but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for sourceforge but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for snyk but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for security-headers but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for requires but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for readthedocs but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for opm but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for nodeping but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for nexus but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for mozilla-observatory but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for matrix but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for localizely but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for keybase but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for jira but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for jenkins but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for hsts but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for gitlab but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for freecodecamp but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for drone but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for dependabot but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for debian but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for coveralls but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for codeship but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for codecov but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for codacy but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for cirrus but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for circleci but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for bundlephobia but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for buildkite but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for bstats but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for bitrise but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for bitbucket but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for azure-devops but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for aur but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 2dbb0be964ec234a4556ffc04fe1c0d0dee6d6a2

Service test failures covered by known issues

I think most of our authenticated requests get skipped in CI since we don't typically have the relevant auth. I don't think this would be necessary based on the code diff, but do you think there's any authenticated-style tests we should run locally or in staging?

There's probably 3 categories of services that use auth:

Services that require auth to work at all
Services that allow low volume anonymous usage but need a key to get a higher rate limit
Services that allow access to public instances/resources with no auth but require auth to access private instances/resources

We've got credentials set for

GitHub
Libraries.io
OBS
SL Insight
Twitch
Wheelmap
Youtube

in CI. I think that actually covers all the services that require auth so we shouldn't be skipping any services in CI as far as I'm aware :crossed_fingers:  but if there are lets identify them and set a key in CI. There's another case where we only test against public resource but in principle you can also set credentials to access a private resource which isn't tested.


Add @ImpSy as reviewer

Purpose of this PR

Add @ImpSy as Spark operator reviewer
Contributed PRs: https://github.com/kubeflow/spark-operator/pulls?q=is%3Apr+author%3AImpSy

/hold for @ImpSy to be a member of Kubeflow org.

https://github.com/kubeflow/internal-acls/pull/715
Thank you for letting me be part of this amazing project 🙏

/verify-owners

@ImpSy Please accept the Kubeflow GitHub org invitation, you should get it to your email.

Done ✅
Sorry for the delay

/verify-owners

/unhold


Fix Transformer (for when registered before entity managed)

The transformer can be added before the entity is managed (e.g. if
specified in the entity spec, as is often done via yaml). In that case,
the entity’s management context is still a NonDeploymentManagementContext.
The subscriptions are stored in a queue. When the entity is managed,
the subscriptions are replayed.
However, the queued subscriptions did not store the entity context.
Therefore when the transformer received its events, it didn’t have
an entity context so could not determine which entity it was talking
about.
This caused brooklyn-library’s EnrichersSlightlySimplerYamlTest.testWithAppEnricher
to fail.

Makes sense, merging.


Switch to CircleCI 2.0

CircleCI is deprecating v1.0 next week so it'd be good to switch over.

Thanks 😄


[charts/redis-ha] merge nameOverride into fullnameOverride

What this PR does / why we need it:
Eliminate nameOverride setting and migrate its functionality to fullnameOverride.
The difference between nameOverride and fullnameOverride is not semantically clear, with the former adjusting the app labels while the latter adjusts the resource naming. In practice, this means that if you want to change either, you probably want to change both.
Which issue this PR fixes

fixes #167

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@DandyDeveloper can you please review?

I'm hesitant with this change as the impact is pretty big to people using it. Also, there is justifiable reasons for both.
This sounds more like I should investigate why there is no clear syntactical difference between them.

I dont see why the change of labels is required. This will break existing installations as the labelSelector of a Deployment/Statefulset is immutable.
Can you elaborate which change is really required?

@mkilchhofer for a detailed description of the problem being solved, see the issue it closes: https://github.com/DandyDeveloper/charts/issues/167

The chart works fine without this change as long as it's the only installation of the chart in the namespace. If you ever install a second redis in the same namespace, the two clusters will interfere with each other.
Probably most users only ever install a single redis cluster, which is why more people haven't hit this problem.

Ok. But it can't happen due to the selector. All selectors contains the .Release.Name.
So when you install the chart twice like this

helm install redis1 dandydev/redis-ha
helm install redis2 dandydev/redis-ha

Sectors should be fine?

There's a separate PR that covers the "hijacking" issue a little bit better. I'm closing this in favor of that.


Update unity-development-overview.md

HT loc

Docs Build status updates of commit 7be4768:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/unity-development-overview.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.


New package: CalibrationTests v0.1.0

Registering package: CalibrationTests
Repository: https://github.com/devmotion/CalibrationTests.jl
Created by: @devmotion
Version: v0.1.0
Commit: f9e48ff4158db8f11cbb58b8b38c3081d00b8054
Reviewed by: @devmotion
Reference: https://github.com/devmotion/CalibrationTests.jl/commit/f9e48ff4158db8f11cbb58b8b38c3081d00b8054#commitcomment-35559659

Your new package pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request after the mandatory waiting period has elapsed.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]


libs::glibc: use GNU_MIRROR

Good catch. Thanks.

ftp.gnu.org was down this morning...

Holy cow! :astonished:


Add CIFuzz integration

This adds CI integration with CIFuzz. CIFuzz will run the h3 OSS-Fuzz fuzzer for a short period of time for each PR. The benefit is that regressions and bugs has a chance of being captured early in the development process.
Signed-off-by: David Korczynski<EMAIL_ADDRESS>

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

Coverage remained the same at 98.2% when pulling d933e38b2f5c8704c6e447c01961144a410f5318 on DavidKorczynski:cifuzz-integration into 6d28d61254030b646713f8b23f2f68a93356762d on uber:master.

Thanks for making this more integrated!

No problem! Thanks for the quick review!


:seedling: Refactor the naming of transport configuration

Summary
Related issue(s)
Fixes # https://github.com/stolostron/multicluster-global-hub/pull/1116
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/lgtm


Have to authorise use of various command line programs on the first launch of the programme

This could just be documented for the time being in the wiki on how to install.

This is a duplicate of #21
I will add an entry to the wiki pointing to this issue,


Could you please share the models weight mentioned in the paper?

Hi, I can share the model weights with you. But they are gotten from another training pipeline. You can not directly load the weights in the existing repository. You may need to change the name of some parameters. BTW, I haven't added the other model scripts in the repository. Please tell me which models you are interested in.  I can send both the network scripts and model weights to you. I may need more time to complete the repository. Sorry for that.

Hi, I can share the raw result of SimTrack-MAE (https://drive.google.com/file/d/1TPir2JSrWuV7ZEA92oV1vvjjXn0KCRrx/view?usp=sharing) in Table.5 (Succ-70.7, NormPre-80.0, Pre-75.6 on LaSOT dataset), which is based on a ViT-B/16 model and has similar results with Sim-L/14. It may take me some time to find the results of Sim-L/14. I will update them later. Hope the SimTrack-MAE result is useful for you!


Missing requirements

Just a heads up after running local in conda from environment.yaml, I'm getting missing modules/packages.  I started to pip install them. Here the order I encountered them:
pyyaml
twisted
grpcio
hupper
wsgicors
protobuf
torch
service-identity
tqdm
diffusers
torchvision
I got to transformers and stopped, may be more.  Will pick up later if I have time.  Not entirely sure it will work because I don't know what versions of each it may require.  Let me know if I'm missing something here that will make this easier.

Hi. The dependencies are all in pyproject.toml. I deliberately don't duplicate them in environment.yaml (it's why the install instructions all have flit as part of the process). It's a bit awkward, but this the more "python standards compliant" way of doing it.


Using relative paths for files inside the collection's folder

looks great :)

@maxdestors You can go ahead and merge the PRs here if you think they're good. I don't have permission to merge


Double-check how JavaType.Flag is a Set of same type of J.Modifier.Type (List) in J.java

Feel free to close, probably need to rephrase this issue body. I'm just dumping this out of my head as I can since I'm seeing it and don't want to forget:
The enum of Flag appears the same as J.Modifier.Type (?)
https://github.com/openrewrite/rewrite/blob/ceee8084a8295cf97399673e4294015c2317368e/rewrite-java/src/main/java/org/openrewrite/java/tree/Flag.java#L20-L29
https://github.com/openrewrite/rewrite/blob/ceee8084a8295cf97399673e4294015c2317368e/rewrite-java/src/main/java/org/openrewrite/java/tree/J.java#L3058-L3091
and in JavaType, Flags (which appear same as J.Modifier.Type?) are of a Set, Whereas in J, J.Modifier.Type is often contained within a List. https://github.com/openrewrite/rewrite/blob/ceee8084a8295cf97399673e4294015c2317368e/rewrite-java/src/main/java/org/openrewrite/java/tree/JavaType.java#L393
Wanted to point this out as something to double-check. Should Modifier Types be referenced as a Set more often rather than a List (considering you can't have multiple of the same modifier)? Should "flag" and "J.modifier.type" be collapsed into one? etc.

@aegershman They are a list in the AST because the preservation of order matters. In the type attribution, it doesn't matter.


Merge background step into each scenario

This is developed and scheduled for release in 0.3.0
https://github.com/trivago/cluecumber-report-plugin/tree/0.3.0


Cherry-pick to branch-2.0 for 2.0.1 release

Cherry-pick following commits

https://github.com/delta-io/delta/commit/ef49ae21cc6130a2b2e4d9e7b10409d43e3a540d
https://github.com/delta-io/delta/commit/ed9ff6e1eced91bc54b55d059805b203273838d9
https://github.com/delta-io/delta/commit/7e876792efdd92a85aa3f7b81d81f34c8b276d7b
https://github.com/delta-io/delta/commit/9017ac0d811c0a42ba8ac45720bddf06c8f17e63
https://github.com/delta-io/delta/commit/6dbc55db53332c985e5bc8470df6c95106afac25
https://github.com/delta-io/delta/commit/23751f8eead9d73f743f4d2325657331997eb684

Could you add https://github.com/delta-io/delta/commit/23751f8eead9d73f743f4d2325657331997eb684 that just got merged to master?
In addition, let's remove

https://github.com/delta-io/delta/pull/1448/commits/14565db90d525bd9776fa7db18e6895cd10eb1bb
https://github.com/delta-io/delta/pull/1448/commits/df80d16fa9b9215f688b5089b66565420b75f86a

from the list since we don't backport other streaming changes.

Could you add 23751f8 that just got merged to master?
In addition, let's remove

14565db
df80d16

from the list since we don't backport other streaming changes.

Updated the PR with above suggested changes.


How can I do tests in my controller?

I'm trying to do some tests in my controller with a authenticated user.
Trying to use this config:
setup %{conn: conn} do
  user = %User{email<EMAIL_ADDRESS>  conn = Pow.Plug.assign_current_user(conn, user, otp_app: :my_app)

  {:ok, conn: conn}
end

What are you testing?

@danschultzer simple actions, like create and fetch.
I’m just receiving unauthenticated error

Are you testing OAuth 2 provider endpoints or api routes that are using e.g. ExOauth2Provider.Plug.VerifyHeader?

@danschultzer api protected routes, not OAuth ones

With an OAuth 2.0 API you have a resource owner, a resource owner can also be a client rather than a user. See the docs here for how to fetch the current resource owner: https://github.com/danschultzer/ex_oauth2_provider#current-resource-owner-and-access-token
You could generate an access token with preloaded resource owner and then use the plug to set it, like this:
setup %{conn: conn} do
  resource_owner =
    %User{}
    |> Ecto.Changeset.change(email<EMAIL_ADDRESS>    |> MyApp.Repo.insert!()

  {:ok, access_token} = ExOauth2Provider.AccessTokens.create_token(resource_owner, %{scopes: "read write"}, otp_app: :my_app)
  access_token = MyApp.Repo.preload(access_token, [:resource_owner])

  conn = ExOauth2Provider.Plug.set_current_access_token(conn, access_token)

  {:ok, conn: conn}
end

You may also want to create an application and add it to the create_token params.
You can fetch the current resource owner by calling ExOauth2Provider.Plug.current_resource_owner(conn)

Still having Unauthenticated error,  but now I have the auth object inside my conn
%Plug.Conn{
  adapter: {Plug.Adapters.Test.Conn, :...},
  assigns: %{},
  before_send: [],
  body_params: %Plug.Conn.Unfetched{aspect: :body_params},
  cookies: %Plug.Conn.Unfetched{aspect: :cookies},
  halted: false,
  host: "www.example.com",
  method: "GET",
  owner: #PID<0.556.0>,
  params: %Plug.Conn.Unfetched{aspect: :params},
  path_info: [],
  path_params: %{},
  port: 80,
  private: %{
    ex_oauth2_provider_default_access_token: %MyApp.OauthAccessTokens.OauthAccessToken{
      __meta__: #Ecto.Schema.Metadata<:loaded, "oauth_access_tokens">,
      application: #Ecto.Association.NotLoaded<association :application is not loaded>,
      application_id: nil,
      expires_in: 7200,
      id: 3,
      inserted_at: ~N[2019-11-14 13:20:18],
      previous_refresh_token: "",
      refresh_token: nil,
      resource_owner: %MyApp.Users.User{
        __meta__: #Ecto.Schema.Metadata<:loaded, "users">,
        confirm_password: nil,
        current_password: nil,
        email<EMAIL_ADDRESS>        id: 4,
        inserted_at: ~N[2019-11-14 13:20:18],
        password: nil,
        password_hash: nil,
        updated_at: ~N[2019-11-14 13:20:18]
      },
      resource_owner_id: 4,
      revoked_at: nil,
      scopes: "",
      token: "de25ba731b787c27325924a4adfdfc528e748017aba0372e943e0e3185894296",
      updated_at: ~N[2019-11-14 13:20:18]
    },
    phoenix_recycled: true,
    plug_skip_csrf_protection: true
  },
  query_params: %Plug.Conn.Unfetched{aspect: :query_params},
  query_string: "",
  remote_ip: {127, 0, 0, 1},
  req_cookies: %Plug.Conn.Unfetched{aspect: :cookies},
  req_headers: [{"content-type", "application/json"}],
  request_path: "/",
  resp_body: nil,
  resp_cookies: %{},
  resp_headers: [{"cache-control", "max-age=0, private, must-revalidate"}],
  scheme: :http,
  script_name: [],
  secret_key_base: nil,
  state: :unset,
  status: nil
}

How does the API pipeline in your routes look?
There's also a previous issue for this: https://github.com/danschultzer/phoenix_oauth2_provider/issues/25

This issue helped me, thanks!


add Dub

https://dub.sh/

Thanks for the addition :)

awesome! thanks, @vihar cc @steven-tey for the record


Add auto enhance

Add the opts04 argument to use auto enhance and fix dark faces
tested locally with http://localhost:8080/pushd/00000000-0000-0000-0000-000000000000/rotate_0__width_1200__height_1600_fit__dither_fs:opts04:sp__759ABF50-E64F-4A02-BCB9-C96D7BCC8C68.png .Face is much brighter than before

Looks good


Quantum volume does not accept an arbitrary list of qubits (new device)

Informations

Qiskit Experiments version: No idea
Python version: Python 3.9.7
Operating system:  Operating System: Manjaro Linux. Kernel: Linux 5.15.48-1-MANJARO. Architecture: x86-64

What is the current behavior?
On running the quantum volume functions I obtain the error
LayoutError: 'Integer list length (4) must equal number of qubits in circuit (7): [3, 4, 5, 6].'
in the step
 qiskit.compiler.transpile(qv_circs[i], backend=backend, initial_layout=qubit_lists[0], optimization_level=3)
Steps to reproduce the problem
qubit_lists = [[3,4,5,6]] # !!!!!
ntrials = 100
shots = 10000
qv_circs, qv_circs_nomeas = qv.qv_circuits(qubit_lists, ntrials)
a = qv_circs_nomeas.copy()
a_meas = qv_circs.copy()
backend = provider.get_backend('ibm_oslo')
for i in range(ntrials):
a[i] = qiskit.compiler.transpile(qv_circs_nomeas[i], basis_gates=['u1','u2','u3','cx'])
a_meas[i] = qiskit.compiler.transpile(qv_circs[i], backend=backend, initial_layout=qubit_lists[0], optimization_level=3)
b_meas = []
for trial in range(ntrials):
b_meas.append(a_meas[trial][0])
What is the expected behavior?
There shouldn't be a problem in choosing any combination of qubits in the variable qubit_lists. This error does not appear if one selects instead this variable as qubit_lists = [[0,1,2,3]].
Suggested solutions
This error does not appear in other old devices. For instance in ibmq_manila I can set qubit_lists = [[0,1,3,4]] and there is no problem whatsoever. Thus I think it's something going on with the new device.

it looks like you are using Ignis code rather than qiskit_experiments. Ignis is a deprecated repository, and it might have bugs with newer features.
with qiskit_experiments the creation of the Quantum Volume circuits is much simpler and there is no need to create manually the circuits with and without measurements (running the circuits and analyzing the results is simple too!)
in order to create the circuits (and setting all the options) using qiskit_experiments you can write:
from qiskit_experiments.library import QuantumVolume

qubit_list = [3,4,5,6]
ntrials = 100
shots = 10000
qv_exp = QuantumVolume(qubit_list)
qv_exp.set_transpile_options(optimization_level=3)
qv_exp.set_experiment_options(trials=ntrials)
qv_exp.set_run_options(shots=shots)
qv_circs = qv_exp.circuits()

if you want to run and get the analysis of the results you can do (using the backend of your choice):
exp_data = qv_exp.run(backend).block_for_results()
result_data = exp_data.analysis_results(0)

you can see more examples and documentation here:
https://qiskit.org/documentation/experiments/tutorials/quantum_volume.html

Ok thanks, I'll give it a try. I really wanted to avoid the new module because there wasn't documentation on it (the link you posted is not referenced in the qiskit textbook and there wan't anything helpful here in the docs).

Closing this now. If you have issues with running QV in Qiskit Experiments, feel free to open another issue.


[android] Push test binaries w/o modifying executable name.

At least test_rth.swift is checking the name of the executable
during the test, so renaming every executable to __executable in
Android will never work.
Also, during the rth tool execution, all the results from before
and after are pushed for every test. Since Android copies the
passed files without relative paths, the library files will
overwrite each other, making the test fail.
Depends on #19949 (more or less)
@modocache: I suppose that you might not remember why you renamed every binary to __executable, but I think that since the arguments end up in the text files, the length of the binary name should not be that important anymore. If you think it would be a problem, I might include an special case for the binaries pushed by the resilience tests (before_before, before_after, after_before, after_after).
@jrose-apple: I had to limit the glob from only one of the directories (before or after). I don’t think this would break the test, or invalidate the testing results. When testing before, the after directory would not be there, so trying to link against it would not find the library, failing the test. If you think this is not correct, I can try to modify #19949 to use relative paths, but it might get hairy.

@drodriguez Just an FYI, JordanR is on vacation.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - ab69e7d4682b7fd904e1b64a3860ec3f181fa7ec

Build failed
Swift Test OS X Platform
Git Sha - ab69e7d4682b7fd904e1b64a3860ec3f181fa7ec


Formatting the Plain Text View

Hi,
I really often use the plain text view. But I don't like its formatting:
birthday  |total_count|
----------|-----------|
1985-10-25|         16|
1985-10-26|         16|

Is there an option to enable a line for the left table side? Having the frame line only on the right side makes it look broken in my personal opinion.
And secondly I would like to have one space between the data and the column delimiters:
|   birthday | total_count |
|------------|-------------|
| 1985-10-25 |          16 |
| 1985-10-26 |          16 |

Maybe it could be a formatting option if it does not exist yet.

I've tried but the extra space has been added only at the end of the column not in both beginning and end:
|id     |released   |
|-------|-----------|
|     1 |2001-11-04 |
|     2 |2002-11-03 |
|     3 |2004-05-23 |

instead of
| id     | released   |
|-------|------------|
|     1 | 2001-11-04 |
|     2 | 2002-11-03 |
|     3 | 2004-05-23 |


Support semantic analyzing for native constructs

from 0.88 onwards native constructs are also treated in the same way as constructs written in ballerina. hence closing this issue.


OpenAI Gym Latest Version Incompatible with Example

The latest OpenAI gym doesn't come with BreakoutDeterministic-v3.
They mention it here: https://github.com/openai/gym#whats-new

2017-05-13: BACKWARDS INCOMPATIBILITY: The Atari environments are now at v4. To keep using the old v3 environments, keep gym <= 0.8.2 and atari-py <= 0.0.21. Note that the v4 environments will not give identical results to existing v3 results, although differences are minor.

To downgrade you can run:
pip install gym[atari]==0.8.2

This gets the Breakout_A3C preset to work. I haven't tried replacing the preset with BreakoutDeterministic-v4 yet.
Looks like Breakout wouldn't work without installing gym[atari] vs just gym anyway:
https://github.com/NervanaSystems/coach/blob/6009b73eb60f6a79daa2a0028f8feafda3635f7d/install.sh#L171
And it's for this preset:
https://github.com/NervanaSystems/coach/blob/e813eaf304d728a7655b5343ca1eb7cd2ba32e74/presets.py#L1123

Hi Zac, thanks for bringing this up!
We are working with the latest gym version so v4 should be used.
We pushed a fix for this issue in the following commit:
https://github.com/NervanaSystems/coach/commit/43bc35916621589bbac101eee11c5640a521067d


Pull out all SQL schema changes since v0.268.3 into separate migrations

0.268.3 has been deployed, and since that version stores state in the database, we can't recreate the database from scratch anymore.

Fixed in #1975


nonterminal, handled tasks no longer prevent graph from finishing

solves https://rackhd.atlassian.net/browse/RAC-1938
changed the mongo store checkGraphSucceeded method to succeed on graphs where the only task dependencies we find with waterline.taskdependencies.find are non-terminal.
@rolandpoulter @pscharla

👍


Canon MP540 - Double Sided Printing Not Available

I have the above printer working great with Cups however i cannot seem to access the option for double sided printing when setting the defaults (or via the print dialog (typically windows))
The option is listed in the overall printer info: Defaults:	job-sheets=none, none media=iso_a4_210x297mm sides=one-sided
Hopefully this is a user error?
Thank you

This might be a limitation in the driver you are using - if it doesn’t report duplex support, then CUPS won’t offer it... Given the age of the printer, I’m guessing you are using the Gutenprint driver which does not support duplex printing, unfortunately...


Update EIP Guide with new Mediators

Check whether EIP Guide need to be updated with newly introduced mediators.

EIP guide is updated with 4.3.0 release, hence closing this issue.


suppress compiler warnings generated by clang

The commits suppresses compiler warnings due to implicit conversions between integer types.
note: for some parts of 9a0a24f it might be better to return some kind of size error rather than just converting size_t to uint32_t

@kazuho
I can confirm the first patch to fix clang warnings, but my clang 3.4-1ubuntu3
does not complain about the second patch.  What version of clang did you use?

Adding -Wconversion enables these warnings, though the flag warns about tons of other codes...

I am using Xcode 6.1.1 (so the clang version is presumably Apple LLVM version 6.0 (clang-600.0.56) (based on LLVM 3.5svn)).

Picked e718871feab0f0162ced30f2a9702830aae5a844.
Replaced 9a0a24f033b1085025f10be3eb913c4f3c4d66e1 with 2e15825ddd43e9c44dfc1f15c678106097e6915b.
Thank you!


🐛 Bug Report: Error: Missing required config value at 'app.https.certificate.cert' in 'app-config.yaml'

📜 Description
I have a proxy (load balancer) sitting in front of the app component which does TLS termination.
so app.baseUrl should start with https:// despite it doing http server. I thought that if app.https.certificate is not set, then it automatically assumes its http. However instead its just throwing the fact that its missing this cert.
👍 Expected behavior
if app.https.certificate is not set, don't expect it, just because baseUrl says its https.
👎 Actual Behavior with Screenshots
baseUrl dictates whether app.https.certificate should be set or not.
👟 Reproduction steps
set up demo but with app.baseUrl as a https link.
📃 Provide the context for the Bug.
No response
🖥 Your Environment
OS:   Linux 6.12.1 - linux/x64
node: v20.16.0
yarn: 1.22.22
cli:  0.29.3-next.1 (installed)
backstage:  1.34.0-next.1
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Hi @adminy 👋🏻 , are you using yarn start in production to run the application? This configuration is only required for local development, which is why we are asking...


No compatibility with react 0.45.1

This version requires react 16.0.0-alpha.12

Can anyone elaborate on this?  We upgraded to 0.45.1 from 0.42 and I'm not seeing the 'notificationOpened' handler called.  Was just investigating this issue and found your bug about no compatibility with 0.45.1.  Is this what you mean?  Is there any workaround or do we need to go to 0.46+?


Debugging Event

Debugging !
Prakruti Bhatt
Period 2 Mortenson
March 4th

[X] Start backend using Debugging
The picture below showcases the backend, enabled by debugging

[X] Set break point at the beginning of endpoint code
The picture below indicates the beginning of the endpoint code and the cherry red dot on line 15 is representative of the added breakpoint

[X] Start in frontend with split screen loading source for an API fetch using GET.
I have started the frontend and then split-screened to see the API fetch to the port 8923

[X] Set break point on fetch, inside .then, inside .fetch
In the picture below, I have set breakpoints on fetch, the body, .then, and .catch

[x] Run frontend, screen capture break at fetch while examining Body
Here I inserted inputs into the frontend to run it, then captured the break at fetch. This has paused at the fetch breakpoint (highlighted in yellow)

[x] Press play on frontend, observe stop inside of backend
When I press play on the frontend, I get a message from the localhost saying that the quote failed to submit (there is an error). When I view the backend, we can see that there is a stop on line 15.

[X] Press step over on backend until you have obtained data from database, screen capture Python Object
Step over in backend

[X] Press play button to end backend debugging session.

Ended backend debugging session

[X] Return to frontend debug session
when pressed play on frontend the quote is now submitted succesfully

[X] Step in until you see data, screen capture capturing break point and Data.**
quote stored now on backend-- data

0.86: shows good understanding of how to use the breakpoints to fix the errors, but could show a bit more on the process of how it works

0.86; shows good understanding of how to use the breakpoints to fix the errors, but could show a bit more on the Python object in the backend instead of the schema table.

The run at the beginning is loading wrong file.  Not sure data screen are show key data from database read.


Dev

upgrade Telegraf to 1.21.3 and fixed the Grafana-package (seemed that I used the armv6-package instead of armv7, but it was working without problems).

Reverted Grafana back to 8.3.4. New image for 8.4.0beta will follow.


feat(material-date-fns-adapter): add support for date-fns 3

Feature Description
date-fns 3 is now available (https://blog.date-fns.org/v3-is-out/). It would be great Angular Material supports it in material-date-fns-adapter package.
Use Case
No response

Same request here👍
I have also date-fns v.3 in project and need to implement adapter.

Same here :)

No, we don't backport features.


Configure CosmosDBOutput behavior to handle more load

I'm using the CosmosDBOutput with .NET Isolated model to write multiple documents to CosmosDB for each function invocation.
The function is basically a Webhook receiving a Json array of items, parsing it and outputting the items through the CosmosDBOutput.
It is hosted in a p0v3 Windows app service plan.
When a heavy load is applied to the function, we're starting to face errors such as
One or more errors occurred. (Only one usage of each socket address (protocol/network address/port) is normally permitted. (alphacosmos.documents.azure.com:443)
or
Unexpected end of request content
Is there anything I can do to improve the behavior of the CosmosDBOutput to handle more load without issue?
Is there a way for the function instances to share the CosmosClient object?
Is batching used when returning an array of items? Is there a way to configure the batch size?
Is there any retry policy at the output level?
Is there a way to define a lighter consistency level to improve throughput?
Thanks a lot

Some code issues were fixed and that should close the issues we were facing (especially an old manual cosmosDB client not shared between executions).
However, I'm still interested to know if the CosmosDBOutput can be fine-tuned to handle more load (consistency level, batching, retry policy, ...)
Thanks a lot.

@ealsur - do you have any guidance for Cosmos optimization?

CosmosDBOutput just calls UpsertItem on the SDK, there is no complexity or further optimization.
If you want to look at Cosmos DB performance, you need to look at the overall picture:

Which are the machine resources? Limit of connections, CPU, network bandwidth, accelerated network enabled.
Are you using Direct mode or Gateway mode? Gateway mode is the default. Direct mode is the performance mode, but it requires more resources. Reference: https://learn.microsoft.com/en-us/azure/cosmos-db/nosql/how-to-configure-cosmos-db-trigger#changing-the-connection-mode-and-protocol
If you want to increase the concurrency, just use IAsyncCollector and execute the operations in parallel.

For more Cosmos DB related performance pointers, see: https://learn.microsoft.com/en-us/azure/cosmos-db/nosql/performance-tips-dotnet-sdk-v3

Thanks for your insights @ealsur
Actually, most of the tips you're referring to can't be applied to the CosmosDBOutput binding, right?
Does that mean that the advice is to use the SDK directly instead of the output binding if optimisation is needed?

Actually, most of the tips you're referring to can't be applied to the CosmosDBOutput binding, right?

Can you clarify why not? Is it because you are running in non-.NET Functions? If that is the case, then I guess the answer is partially.
Regardless of the language, you can increase the machine resources if that is the bottleneck and change to Direct mode for better network latency. The only thing you cannot do in non-.NET is increase the concurrency of the requests, which you can in .NET.


[Diagnostics] Improve argument labeling diagnostics

Extend new labeling diagnostics (via fixes) to support
member references and subscripts.

@swift-ci please test


egui_glow - GL error

Describe the bug
Following the recent 0.22 update, my application which uses custom painter callbacks started to continuously log this message:
2023-05-24T21:47:36.285502988+01:00 ERROR egui_glow - GL error, at /home/anghosh/.cargo/registry/src/index.crates.io-6f17d22bba15001f/egui_glow-0.22.0/src/painter.rs:406 (callback): GL_INVALID_ENUM (0x500). Please file a bug at https://github.com/emilk/egui/issues

To Reproduce
Steps to reproduce the behavior:

Grab this branch of my project
Run the app with the following environment variable: ROM_PATH=path/to/SMW.smc
From the menu bar select "Prototypes > Level editor" or "Prototypes > Sprite map editor"
Observe the console output being flooded with errors.

Expected behavior
No errors should be logged out.
Screenshots

Desktop (please complete the following information):

OS: Arch Linux

Additional context
I could not reproduce this issue by running the custom_3d_glow example. In case there's anything wrong on my end, here are all the files in my project that use CallbackFn:

smwe-widgets/src/vram_view.rs
smw-editor/src/ui/editor_prototypes/level_editor/mod.rs
smw-editor/src/ui/editor_prototypes/sprite_map_editor/layout.rs

This turned out to be a problem on my side. I bound two buffers as ARRAY_BUFFER where it should have been UNIFORM_BUFFER.


Difference in measurements between map-labels and tool

Using the mapapps-sketching-enhanced bundles (v. 3.4.4) I get some differences in the measurement details. Those differ between the map-labels and the information provided in the tool-window.
As far as I know the problem does not occur in SRS 3857, but at least in my tested SRS systems 25832, 25833 and 4326.

Fixed by PR #186


No longer manually starting / ending run loop for Stripe request.

Using Ember.run.begin() before making the Stripe request prevents the UI to update with any pending changes until the Ember.run.end() is called.  This is problematic in case you want to show a spinner while the Stripe request is happening.
Here's a jsbin that demonstrates this:
http://jsbin.com/qacava/1/edit?html,js,output

clicking the Without Run Loop button will display proper messages while running a long task
clicking the With Run Loop button will suppress message changes

What was the reason for manually starting the Ember loop?
It seems to work fine (both resolve and reject) without the manual Ember loop.

hey @johnnyoshika, thanks for this. The reason to use the Ember loop manual was because in slow connections the promises were resolving before the requests were happening, this was also several Ember versions ago so this may not be an issue any more. I'll run some tests to make sure things are good in our implementations and if all looks good I'll merge.
Thanks again,

Thanks @buritica! I look forward to your finding.  If you merge this into source, I'll delete my fork of your repository and use yours directly again.
Cheers!

hey @johnnyoshika unfortunately just by switching out your branch and linking with npm breaks 4 of my acceptance tests. I'm going to have to dig deeper into this, but unfortunately I can't merge this yet. I'm going to try to replicate those scenarios in tests here and see what happens.
I'll keep the PR open until I have some solid findings, thanks again.

Hey @johnnyoshika, after much research here are my conclusions:

it is the recommended approach to use Ember.run to wrap any asynchronous code.
as much as I wanted the higher-level Ember.run() to function, I believe the way Stripe deals with it's ajax requests (they use a private iframe) prevents Ember from detecting what is going on in the background. This means that I must use the lower-level manual .begin and .end to abide by the run loop.
even though your jsbin shows the UI is not updated, I think that is the way it is supposed to work for long tasks. The run loop is designed to ensure "any deferred actions including bindings and views updates are flushed at the end."
I understand your use case, so I'm open to a counter proposal, but I believe removing the run loop is not the way to proceed. I could consider emitting custom events to inform about the life-cycle of the request? Haven't done this before, but I think it may be a more elegant solution than getting rid of the async flow control which could have unintended consequences.

Please let me know your thoughts, for now I'm closing this PR.

@buritica, that's very interesting. Thank you for the research.  I must admit that I'm not all that familiar with the run loop, but your logic makes sense.  To get around my problem, what do you think about wrapping my code that updates the UI in a run loop so that the UI updates before your stripe service is called?  I modified the jsfiddle to demonstrate:
I added this:
      Ember.run.begin();
      this.modelFor(this.routeName).set('message', 'start');
      Ember.run.end();
      // now call taskWithRunLoop

http://jsbin.com/sitegiwoxi/2/edit

I think rather than you calling the runloop manually, you can schedule the Stripe.createToken (or any task with a run loop) to happen afterRender. Have a look at the third task I added to your JSbin http://jsbin.com/nofoviwiho/1/edit

Excellent!  Thank you for your help.  That's what I'll do.

Interesting.  I just upgrade to your latest release (v2.0) and without making any code changes, the UI change is no longer being postponed.  This worked even without implementing your afterRender technique.  I see that you're still using Ember.run.begin() and Ember.run.end(). Did anything change?

Interesting. Nothing really changed, we are no longer injecting ourselves and rather using new 1.10 service injection.
We also removed ember-data deps. Thats it.
--
Juan Pablo Buritica
VP of Engineering - Ride
t: @buritica
On Mon, Mar 23, 2015 at 10:32 AM, johnnyoshika<EMAIL_ADDRESS>wrote:

Interesting.  I just upgrade to your latest release (v2.0) and without making any code changes, the UI change is no longer being postponed.  This worked even without implementing your afterRender technique.  I see that you're still using Ember.run.begin() and Ember.run.end(). Did anything change?
Reply to this email directly or view it on GitHub:
https://github.com/ride/ember-stripe-service/pull/10#issuecomment-85026399

I stand to be corrected.  I don't know why it was working as I previously described.  Maybe some old caching?  Anyways, I do indeed need the afterRender technique after all.


[TablePagination] Fix responsive display issue

[x] I have followed (at least) the PR section of the contributing guide.

Fixes #18561

Thank you


class BatchNorm2D in fanet.py

I am confusing about it! You defined batchnorm, but in the forward function, you only use activation, can you explain this?
`
class BatchNorm2d(nn.BatchNorm2d):
#(conv => BN => ReLU) * 2
def init(self, num_features, activation='none'):
super(BatchNorm2d, self).init(num_features=num_features)
if activation == 'leaky_relu':
self.activation = nn.LeakyReLU()
elif activation == 'none':
self.activation = lambda x:x
else:
raise Exception("Accepted activation: ['leaky_relu']")
def forward(self, x):
    return self.activation(x)

`

I am confused as well. I don't understand how a non-linearity could be called as a BN layer. In the actual ResNet model I remember using a normal nn.BatchNorm2D layer without any overrides

I am not very sure, but on analyzing the model with Netron, I can see BN layers. Probably overriding of forward function just appends the activation(LeakyReLU) after the BN.

@Eashwar93 If the last part of what you said were true, it would be a major Pytorch bug. Printing out the named modules shows that there is obviously no nn.BatchNorm present.

@feinanshan This bug raises suspicions about the correctness of the rest of the implementation too vs. the model actually used to generate the results for the paper. It would be very helpful if you could release a pre-trained model.

Hi,  this code is just for speed evaluation, so the BN layers are removed as they can be merged into the Conv layers before them.

There is a different bug here then. The bias terms in the convs being fused with BN should be true.

Yes, it should be. Yet that doesn't affect the efficiency.

@Eashwar93 If the last part of what you said were true, it would be a major Pytorch bug. The derived module carries the parameters of batch norm, but the forward call is overridden.

@Eashwar93 If the last part of what you said were true, it would be a major Pytorch bug. The derived module carries the parameters of batch norm, but the forward call is overridden.

@mehtadushy I am pretty sure you are right. Visualizing the BN layers in Netron confused me. Thanks for the clarification.

Hi, this code is just for speed evaluation, so the BN layers are removed as they can be merged into the Conv layers before them.

@pinghu6  It would be nice if you could share a trainable model if you can't provide us with a trained model. If that is not possible either could you please clarify if there are any other aspects that I need to take care while I can try developing a trainable model based on your test model.
Thanks once again!!

@Eashwar93  BiseNet also has the same BN override

@mehtadushy Sorry I was being careless and I didn't see the Class definition. Actually the class definition is slightly different.
In BiseNet
class BatchNorm2d(nn.BatchNorm2d):
    '''(conv => BN => ReLU) * 2'''
    def __init__(self, num_features, activation='none'):
        super(BatchNorm2d, self).__init__(num_features=num_features)

    def forward(self, x):
        return x

In FANet

class BatchNorm2d(nn.BatchNorm2d):
    '''(conv => BN => ReLU) * 2'''
    def __init__(self, num_features, activation='none'):
        super(BatchNorm2d, self).__init__(num_features=num_features)
        if activation == 'leaky_relu':
            self.activation = nn.LeakyReLU()
        elif activation == 'none':
            self.activation = lambda x:x
        else:
            raise Exception("Accepted activation: ['leaky_relu']")

    def forward(self, x):
        return self.activation(x)

No ReLU activations as well in case of BiseNet I suppose


🐛 Bug Report: Sent Button invisible.

📜 Description
Not able to see send button in Android neither in normal mode nor in desktop mode.
👟 Reproduction steps

Goto https://www.hacksquad.dev/
Go to my squad,
Here you will see send button is not visible neither in normal mode nor in desktop mode.

👍 Expected behavior
Should responsive and visible to user.
👎 Actual Behavior with Screenshots

Desktop mode

💻 Operating system
Something else
🤖 Node Version
16
📃 Provide any additional context for the Bug.
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

Are you willing to submit PR?
Yes I am willing to submit a PR!

can i work on this

Hi @Saifullah00 , could you please assign me this.
Thank you

@ashikjhonson sorry,
I'm not the member 😄, I too want to work on this issue, but not get assigned.

@ashikjhonson sorry, I'm not the member 😄, I too want to work on this issue, but not get assigned.

XD, my bad. Good luck

Assign me!!


new(app_actions)!: adjust base_syscalls option, add base_syscalls.repair

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine
/area tests

/area proposals

/area CI

What this PR does / why we need it:
Finalize https://github.com/falcosecurity/falco/issues/2433.
Fixes missing PPM_SC_SCHED_PROCESS_EXIT safety enforcement when user_positive_sc_set non empty.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

new(app_actions)!: adjust base_syscalls option, add base_syscalls.repair

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: incertum
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Seeing that there are issues with the test cases, because of extending them a bit. Let's investigate, locally they passed, but I posted in another comment that when I tried to add init_module into the mock up rules it was troublesome. Happy to revert test extensions a bit and we can address them in a separate PR after @jasondellaluce refactors final aspects of ppm sc API.

@incertum I had a list of suggestions to propose with where impossible-ish to report through the GH UI, so I proceeded by pushing a commit on your branch with my review suggestion. Please drop it if you feel like it's wrong.
The topic is to fix a couple of logic/preference flaws, and strengthen the test cases (also re-including your interim test enhancements). Hopefully this will fix the CI on GH as well, since it works for me locally too.

Thanks a lot for fixing up the tests :pray: will address the other changes shortly when I have some time.

@jasondellaluce I think we still have a some problems with the tests. They can interfere with each other and then some of them can fail. It appears to be a known issue, was reading up on https://chromium.googlesource.com/external/github.com/google/googletest/+/refs/tags/release-1.8.0/googletest/docs/FAQ.md -> we may need better SetUp and TearDown mechanisms for these ConfigureInterestingSets tests?
How to replicate issues: This is the interesting part, first all worked again locally, changing order of tests made it then fail for 2 of them. And then without changing anything after rebasing some tests failed, but for example if you comment all other tests out the one that failed will surely pass. Hope one of these conditions helps you to simulate these type of apparently a bit random interferences.

we may need better SetUp and TearDown mechanisms for these ConfigureInterestingSets tests?

I think all the tests should be "stateless", and they just access the static globals for reading. It can be that the static global variables are not initialized in the proper execution order maybe, in which case SetUp and TearDown would definitely help. Since they are passing so far, I would leve this test flakyness improvements for another PR, WDYT?

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: incertum, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 70810f8df1c10e271f13dd53c90294a2aef24110

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: incertum, jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Spatial types / locs / 2d?

I'm trying to figure out how to define a field as a location / point / set of points. Is this possible with SimpleSchema?

+1

@jackjwilliams I haven't tried but please post here for others if you figured it out. You can always use blackbox: true.


Mixed up shared_credential_files with shared_config_files

Problem
Reference:

https://registry.terraform.io/providers/hashicorp/aws/latest/docs#shared-configuration-and-credentials-files

Right now we have this: https://github.com/MaterializeInc/terraform-aws-kafka-privatelink/blob/main/providers.tf#L12:
provider "aws" {
  region                   = var.aws_region
  shared_credentials_files = var.aws_config_file
  profile                  = var.aws_profile
}

But var.aws_config_file defaults to ["~/.aws/config"], which is a config file, not a credential file.
Suggestion
The AWS provider already has sensible defaults for environment variables and AWS config / credential locations. I suggest we remove those from variables.tf entirely and keep only the region. If users want further customization, they can use environment variables.
If we do want to keep what's in variables.tf, let's change aws_config_file to aws_credentials_file and set the default to ["~/.aws/credentials"]. Or I suppose it might be easier to change shared_credentials_files to shared_config_files in providers.tf.

This fixed the issue for me


Using this Planner for other RTS?

Hi! First of all, thank you for the amazing job!
I was curious if it is possible to reuse this project for other RTS games?
To be more precise, how much work is required to:

Fork and remake the planner for another RTS, let's say Warcraft 3, without adding any new mechanics (i.e., the initial build might be incomplete).
Refactor existing codebase to be more like a library/engine with an SC2 app on top of it, so other games can easily reuse the codebase?

Hey,
that is quite a good question to consider for other RTS games.
In https://github.com/BurnySc2/sc2-planner/blob/d4b88c289d95081b5115a69090224e725a60cdff/src/game_logic/income.ts I tried to emulate the income, which works because normally bases will be built always in the same distance to minerals and vespene, and bases usually have 2 vespene geysirs.
Furthermore the construction of units, structures and upgrades is constant except for when chrono boost is used.
To name a few RTS and why you will run into issues with my approach:

Age of Empires 4 allows construction of buildings with multiple villagers, as well as different races have different gather rate (e.g. HRE gathering faster if near a prelate).
Supreme Commander (and Beyond All Reason) has a flowing economy where you have a steady income and are constantly spending your resources at a different rate, as well as multi construction vehicles may be used to build units and structures
Warcraft 3 lumber income depends on how far away the lumber is (similar to AOE4 lumber)

I have yet to see how the income works in Stormgate, Immortal: Gates of Pyre and Zerospace.
Additionally you will have to keep the project up to date with the game version (patches), so you need to find a way to automate the process of updating unit cost, build time and tech-tree requirements.
For SC2 I found now roughly 3 ways: Using the Bot API, exporting and parsing XML with the map editor, or extracting the XML files directly.
Other than that, the project might be reuseable. However I am more of a backend programmer (so this progamming style might not be ideal in this project) and will prefer to not use any JS frameworks (angular, react, vue or svelte) in the future, but rather server side rendering with HTML + htmx.


Background Video too large

Background image is very large (42mb) it affects the performance and not that good looking in small device
We can replace it with animated background made using js library like particle.js or other backgrounds made with vanilla js and css
which will be much lighter and responsive.

Hey, I'm gssoc'21 participant, I would like to give my hand in this project. Please assign this issue to me.

can you implement and send ss here? @Deepak-dex


Use Tempfile for generation

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Run '...'

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Ubuntu]
Source [e.g. snap, npm]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

When using TempDir the workspace is lost when e.g. scripts are canceled


WFCORE-6428 Error occurred in starting fork when running with jacoco profile

https://issues.redhat.com/browse/WFCORE-6428

Thanks @chengfang


change the order of controls inside a container

I already asked this question on stackoverflow link.
How can I change order of controls inside a container without rebuilding entire layout with new order? since that controls are saved inside a seq, I think we should be able to manipulate the order of them.
and two more question,
is there Grid layout in NiGui?
has NiGui written in pure Nim? and if not, why not?
Thank you

Reordering of widgets is not yet supported other than rebuilding the list.
There is no grid layout. You can use multiple horizontal and vertical containers.
NiGui is written in Nim only, but it needs to call into WinAPI/Gtk libraries to make windows/widgets.

Thank you for your response. I tried to create a grid layout containing textboxes in two similar ways. The problem is it is unexpectedly so slow. I would appreciate if you could tell me how to make it run faster. Here is the code (I prefer using newTreeContainer2, since this way we can access all_widgets and all_hlayouts and change their properties)
import nigui
app.init()
type
    Tree = ref object of LayoutContainer
        all_widgets*: seq[seq[TextBox]]
        all_hlayouts*: seq[LayoutContainer]
proc newTreeContainer2(num_rows,num_cols: int): Tree =
    var layout = newLayoutContainer(Layout_Vertical)
    result = new Tree
    result.init()
    result.add( layout )
    for i in 0..num_rows-1:
        result.all_hlayouts.add( newLayoutContainer(Layout_Horizontal) )
        layout.add(result.all_hlayouts[i])
        result.all_widgets.add(@[])
        for j in 0..num_cols-1:
            result.all_widgets[i].add( newTextBox(  ) )
            result.all_hlayouts[i].add(result.all_widgets[i][j])
proc newTreeContainer(num_rows,num_cols: int): LayoutContainer =
    var
        layout = newLayoutContainer(Layout_Vertical)
        all_hlayouts: seq[LayoutContainer]
        all_widgets: seq[seq[TextBox]]
    for i in 0..num_rows-1:
        all_hlayouts.add( newLayoutContainer(Layout_Horizontal) )
        layout.add(all_hlayouts[i])
        all_widgets.add(@[])
        for j in 0..num_cols-1:
            all_widgets[i].add( newTextBox(  ) )
            all_hlayouts[i].add(all_widgets[i][j])
    return layout
var
    window = newWindow("Tree Example")
    container = newLayoutContainer(Layout_Vertical)
    tree = newTreeContainer(5,8)
container.add(tree)
window.add(container)
#echo tree.all_widgets.len
window.show()
app.run()


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Addressed the issue wherein the usernames not starting with alphabets were getting validated.


Terrain Mesh Stamp

[x] Test Mesh Stamp to level out terrain beyond boundary: https://forum.unity.com/threads/leveling-terrain-below-gameobjects.764567/

Brush size is huge unusable


user: fixed #542, masterkey being written to pcapng multiple times.

@chenzhiguo14-qq  try this PR


useSelector API reference is self-contradictory

Do you want to request a feature or report a bug?
bug
The useSelector API reference states two things that contradict each other:

When an action is dispatched, useSelector() will do a shallow comparison of the previous selector result value and the current result value. If they are different, the component will be forced to re-render. If they are the same, the component will not re-render.
useSelector() uses strict === reference equality checks by default, not shallow equality (see the following section for more details).

What is the current behavior?
useSelector() uses strict === reference equality checks by default, not a shallow comparison.
What is the expected behavior?
The docs should probably be amended to say something like this instead:

When an action is dispatched, useSelector() will do a shallow comparison of compare the previous selector result value and the current result value. If they are deemed different, the component will be forced to re-render. If they are the same Otherwise, the component will not re-render.

This was fixed for the primary version in #1326. It looks like, since we now have a 7.2 release, they haven't been updated since that version was generated. I'll get that cut now.


Combine addons

Is it possible to use multiple addons for one story, and if so, how?

Could we have more info on this?

Lets say I wanna use the the info and the knob addon:
storiesOf('Button')
  .addWithInfo(
    'simple usage',
    `
      This is the basic usage with the button with providing a label to show the text.
    `,
    () => (

        <Button label="The Button" onClick={action('onClick')}/>

          Click the "?" mark at top-right to view the info.

    ),
  );

stories.add('with a button', () => (
  <button
    disabled={boolean('Disabled', false)}
  >
    {text('Label', 'Hello Button')}
  </button>
))

is it possible to combine this two?

Yes sure. You could use both.

Thanks


Angular 1.3, Bootstrap Tabs (or 'display: none') and resizing

1.2.26: http://jsfiddle.net/eosrz86r/
1.3.0: http://jsfiddle.net/82d9ejtL/
1.2.26 seems to be fine from my initial interpretation. If you switch tabs, the grid doesn't resize at all.  Not resizing when hidden is actually a good thing, as it doesn't trigger redraws to the content of each item (in my case it can be expensive).  In 1.3.0, if you switch tabs, the grid sizes all of it's content down to 0 when hidden.  But once you switch back, it doesn't resize at all leaving a blank page.
I would expect no sizing should happen when display: none or offsetWidth: 0
This can also be seen with a simple ng-show and $interval, but it has a slightly different outcome.  In Angular 1.3, you'll see a genie-flicker effect because the display: none caused a recalculation of sizes to be 0.  When it's displayed again, it auto-resizes from 0 to the width of it's container.  This is slightly different from the tabs because the tab does't trigger a resize on it's content.
1.2.26: http://jsfiddle.net/52wj5t0h/
1.3.0: http://jsfiddle.net/qw7pc0cc/
Besides this, I didn't notice any other breaking changes with the upgrade.  This seems to be my only hold up for migrating to 1.3 though, which is better than I expected :)

I have been discussing with @mtraynham this problem because it is happening to me using the latest versions of all the dependencies. When you put a gridster element inside a bootstrap-ui tab, the grid does not show up unless you manually resize the browser. It seems like the width and height of the grid its calculated when the tab still have the CSS rule display:none so the effective size is always zero. When bootstrap-ui activates the tab (a few milliseconds later), it changes the CSS rule to display:block but gridster is never notified about this.
The documentation states that we should use the jquery version of the module javascript-detect-element-resize but when you use a generator like angular-gulp who handles those dependencies for you, it loads the javascript version of the module, not the jquery one because the bower.json manifest of the module javascript-detect-element-resize points to the file detect-element-resize.js and not the recommended file jquery.resize.js. If I manually change the reference everything works just fine.

@barretodavid were you able to replicate this in a fiddle?

By the way, I tried out your recommendation of using the jquery.resize.js instead and it works appropriately.  I wonder if the other implementation just doesn't catch as many resize cases...

@mtraynham there must be something about how jsfiddle works because I wasn't able to replicate the problem there. The problem shows when working locally on my computer.

@mtraynham my guess is that is a race condition, and because jsfiddle has to deal with more things than when working locally, the grid loading happens after the tab content is changed to display:block so it calculates its size correctly.

@barretodavid I'm still unable to recreate it locally... Even created a test project for it.
https://github.com/mtraynham/angular-gridster-151

@mtraynham both of your examples works fine on my computer, the only difference is that I can see a glitch when using the jquery plugin. But I had the same problem as you describe, I still haven't manage to implement a proper solution on my main project. I still think is a race condition, maybe the jquery plugin just slows things down a bit and thus gives enough time to the tab plugin to change the display rule.

@barretodavid Found the problem.  You can't have both the detect-element-resize.js and jquery.js on the page.  I pushed a new html page to that repo that shows the issue.
https://github.com/mtraynham/angular-gridster-151/blob/master/test-detect-element-resize-with-jquery.html
This is sort of unfortunate, as the detect-element-resize.js is probably more performant and doesn't rely on jquery at all.  Nor does it have that flicker effect.  That and I was using it else-where :).

@barretodavid So a little bit more digging.  It works on 0.11.7, but not 0.12.0+.  https://github.com/ManifestWebDesign/angular-gridster/commit/e650ad2e146f98aaf682aa79af761b7acb50d44a is the commit that breaks it.

My thinking is it might be good to remove this if clause:
https://github.com/ManifestWebDesign/angular-gridster/blob/master/src/angular-gridster.js#L747
Because both detect-element-resize.js and jquery.resize.js append window.addResizeListener to the scope, there's no need for this if clause.
There is still the random flicker of sizing here...  which may be a separate issue.

@mtraynham let me see if I understand correctly. The purpose of that if clause was to know if the jquery.resize.js plugin was present but it is being triggered always, even when detect-element-resize.js was being used?

Correct.  jQuery has it's own resize function (as noted above).  The jquery.resize.js script overrides that function.  So, if you have either jQuery.js or jquery.resize.js on the page, it falls into that if block.
As noted, the standard jQuery resize handling is not what we want.
This check is pointless though, as both scripts from javascript-detect-element-resize append the window.addResizeListener, so it could just always use that instead.

In regards to the flicker, I would assume that an initial rendering of a gridster-item should have opacity: 0, then fade in to opacity: 1.  Although, I believe this may be counter-acted by the display: none to display: visible switch.

@barretodavid The flicker can be fixed based on the solution obtained from here.  Although, I don't know of the performance ramifications involved in this change... I imagine display: none is better for a large amount of tabs.
In your own CSS:
.tab-content > .tab-pane {
    display: block;
    height: 0;
    overflow-y: hidden;
}

.tab-content > .active {
    height: auto;
}

@mtraynham great, thank you!

I'm going to leave this open still... I'd much rather see a fix for handling of display: none parents that offers better performance.  But hopefully that should hold you off for the time being.

Please try the latest version and let me know if it clears up some of these issues.

@danomatic Thanks for accepting that PR.  The jquery+javascript-detect-element issue does look resolved.
I still see the flicker with jQuery.  My guess might be that the parent going from display: none to visible, triggers the resize of the gridster element before the tab allocates a proper width (height should be gererated by the gridster element.  So the gridster will initially render in mobile mode.
Then, the tab will calculate it's proper width and the gridster will readjust accordingly, causing the flicker.

The latest branch is attempting to avoid triggering is mobile or any other size changes if the gridster element is display none.  However if as parent element is display none, it doesn't handle that.  I'm not seeing a flicker when using ng show and hide directly on the element.

Yeah, display: none doesn't really propagate to the child nodes in any form.  jQuery has the $(element).is(':visible') feature, but this walks the node tree looking for a parent with display: none... I wouldn't say that is particularly performant.  Tabs do have the active bindable property though, and I've used that in the past as a flag when to render things.  So maybe that is not a terrible alternative, binding to a property that disables/enables resize updates?  Just throwing it out there.

We haven't worked with angular-gridster for a while, but restarted it now.
I the mean while we migrated to angular 1.3
And now my dashboard with ui-bootstrap tabs and a dashboard in each tab is not working properly.
From the discussion above I can not clearly extract a work around or solution.
Is there a way to get this working again ?

I might have a solution for your problem ReinierGielen.
My problem was, i had a navigation with Buttons, which i can drag and resize. After i switched the tabs from Bootstrap, the Buttons didn't had their inline CSS anymore. Only if i resized or double clicked somewhere, the inline CSS got reinserted.
So i look what the issue was, and i think that angular gridster has a problem to update the "isMobile" variable and the watch method, according to the tab.
in your Gridster Options set "mobileModeEnabled" to false.
When you switch tabs, the size changes, so you get in the mobile mode, even if the "isMobile" Options is set to false.
I hope that solve your problem.

Thanks for the advice, but it does not seem to work.
However I have found a work around.
I placed the dashboards outside the tabs.
So basically the tab's are the tab buttons but there is nothing in the content area.
Below the tabs I have a ng-repeat that generates all my dashboards.
Each dashboard has a ng-show linked to the active tab.
Works like a charm.


--relative flag for making the embedded path relative

Currently if I use cargo edit-locally ../../some/path it will still hardcode the final absolute path into Cargo.toml.
For gecko, this means that I have to further edit things to make it relative again so that I can e.g. push to try or otherwise share the commit for others to try out.
With the option for a manifest-path other than . this is nontrivial, but would be good to have.

Oh, looks like you do this already, but via let path = path.strip_prefix(ws.root()).unwrap_or(&path);. strip_prefix can only strip the common prefix when the base dir is the parent of the other, not handle the general case of two directories.

https://stackoverflow.com/questions/39340924/given-two-absolute-paths-how-can-i-express-one-of-the-paths-relative-to-the-oth
probs should put this up on crates.io

Seems like reasonable behavior to me, would probably be fine to just turn it on by default as well


request can't seem to parse the body of https://www.yahoo.com/

Summary

Simplest Example to Reproduce
use request('https://www.yahoo.com/', callback);
and the response body doesn't seem to have normal text.  It just seems to have has garbled noise.
No options, just a callback.

Your Environment
windows, but i'm not sure it matters.

software
version

request
2.88

node
v10.16.0

npm
6.9.0

Operating System
windows 10

Something to do with the encoding, set the correct header.

Something to do with the encoding, set the correct header.

Why does this particular site require a particular header?  Do browsers detect the encoding and request does not?

Something to do with the encoding, set the correct header.

Why does this particular site require a particular header? Do browsers detect the encoding and request does not?

Exactly.
const request = require('request');
request({
    uri: 'https://www.yahoo.com/',
    headers: {
        'accept-language': 'es-ES,es;q=0.9,ru;q=0.8',
        'accept-encoding': 'br',
        'accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9'
    }
}, function(error, response, body) {
    console.log(body);
});```
Worked for me

Ok, fair enough.  It would be cool if there was an option for that.  Thanks!


Text Wrangler keeps using python 3 when I need python 2.x mac

Hi guys
so im using textwrangler and wanted to try python 3, my question is how do I revert back to python 2.x ?
im using a mac
any advice would be great
many thanks in advance

Since this clearly has nothing to do with Atom, I'm closing this issue.


pixel art homework

comfort: 4
Lots of google searches, but I think I understand most of what I read (and copied)
completeness: 5
Did not do bonus, as usual

:+1: excellent commit messages!


Add show more functionality to Statistics view

[x] Pagination

This functionality should be moved to #40.


add initial aspect ratio

@benyaminbeyzaie Thank you for your contribution and let me close this PR with migrating this PR to the current architecture at #166.


Specify screenshot versions

Where to find the issue
Screenshots
https://www.coronawarn.app/en/screenshots/
https://www.coronawarn.app/de/screenshots/
Describe the issue
The screenshots pages refer only to the "current release" without a specific version number.
Suggested change
Add the version number for Android and iOS to the text:
"Browse through the screenshots to get an overview of the features that Corona-Warn-App offers in the current release."

@MikeMcC399 see https://github.com/corona-warn-app/cwa-website/pull/1208

@dsarkar
Thank you for adding the version number! Since the issue has been addressed, I will now close.


PersonaBar Users Send Password Reset Link Error

Description of bug
PersonaBar Users Send Password Reset Link Error
Steps to reproduce
List the precise steps to reproduce the bug:

Create multiple portals
Set Host settings SMTP
Set different Portal Settings SMTP that is "suppose" to use administrator account to send
Log in with the super user account into a portal with a different administrator account
Send password reset link will result in error.

Current behavior
If logged in as another administrator with an email account different from the one setup for portal settings it will use your current users email address to send instead of PortalSettings.Email
Expected behavior
Use the Portal Administrator account to send the email so that the SMTP server uses that email address for authorization of the email to be sent.  Most SMTP servers must have a local user email account so if you send while using a different account you get this error.
Screenshots

If applicable, provide screenshots to help explain the bug.
Error information

AbsoluteURL:/API/PersonaBar/Users/SendPasswordResetLink
Message:User not local; please try a different path. The server response was: Sender address is not valid for your login. Check your email program settings.
StackTrace:
at System.Net.Mail.MailCommand.CheckResponse(SmtpStatusCode statusCode, String response)
at System.Net.Mail.MailCommand.Send(SmtpConnection conn, Byte[] command, MailAddress from, Boolean allowUnicode)
at System.Net.Mail.SmtpTransport.SendMail(MailAddress sender, MailAddressCollection recipients, String deliveryNotify, Boolean allowUnicode, SmtpFailedRecipientException& exception)
at System.Net.Mail.SmtpClient.Send(MailMessage message)
at DotNetNuke.Services.Mail.Mail.SendMailInternal(MailMessage mailMessage, String subject, String body, MailPriority priority, MailFormat bodyFormat, Encoding bodyEncoding, IEnumerable`1 attachments, String smtpServer, String smtpAuthentication, String smtpUsername, String smtpPassword, Boolean smtpEnableSSL)

Additional context
Documentation at least can be updated to show how to properly manage a site for sending out notificitions that you must be logged in using the administrator account with the assigned email address for SMTP settings to function properly.
or fix the SMTP settings to use only the HOST email settings or PORTAL email address assigned settings
I believe these notifications of different classes could also be something that is set in "Communications" tab of site settings to allow setting the different email addresses for different use cases such as notifications, accounts, billing, shopping and support.
I believe I will try to at least document this so that it is known to use the multi-portals with care in respect to sending out messages while logged in a host user instead of the assigned administrator account for SMTP to work if you have Email as Username.
A switch to ONLY use the portal SMTP account settings as email address as the ONLY from address unless overridden by modules would be nice as well so then you can set anything.
Thing work but only if you understand the issue at hand here.  I don't think it is working 100% as intended as it should be using the host administrator account for default account to send from instead of the user logged in sending it.   Which is why I believe this is a bug that can get ironed out.
Affected version

[ ] 10.0.0 alpha build
[ ]  9.5.1 alpha build
[ ]  9.5.0 latest supported release
[ ]  9.4.4

Affected browser

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer 11
[ ] Microsoft Edge (Classic)
[ ] Microsoft Edge Chromium

Found my issue I believe I had wrong administrator account assigned to the portal.  I am reviewing settings more.

on another note, Primary Administrator should be able to have any email account such as GMail but for mail servers like mine you need to authenticate prior to sending and you can only send using that account provided as sender.
Primary Administrator message should also include this is the email used for "Portal SMTP Settings" for sending emails from the system such as verification emails and notifications.
I believe there needs to be a second account created for each purpose again so Gmail administrators can get the emails sent by the system notifications.  They should both get emails from the "Send From Only" possibly setting being made so DNN uses only the Username provided or a certain email address as things can be configured both ways for different purposes or projects desired results.

I would not disagree to add a site email in addition to the primary administrator email. The admin email would receive most stuff and the site email can be a<EMAIL_ADDRESS>or some such. We could make the site email fallback to the primary admin if it is unset. This solution we can probably do inside platform, now nothing says all third party modules would implement it any soon... But yeah, I would not find that a bad idea.


fix(imports): prevent circular import

Summary

A circular dependency can cause issues, which is happening here because getServerState is importing from the index, but it's also exported from that same index.
Result
fixes #6292
also copied the code we have in InstantSearch.js to prevent circular dependency into react and vue instantsearch. Before the fix the build failed thanks to these onwarn changes.
This PR removes the circular dependency and also goes through all imports in React InstantSearch core and makes sure they're consistent (but the only circle was in getServerState).

Yes, the rollup configs are very similar, I think a common one could be handy


How does Two-Factor Auth play with OAuth 2.0?

Hello!
Currently looking into how I can integrate Two-Factor Authentication into my Ember App using Ember Simple Auth OAuth 2.0.
Briefly pouring over the RFC, I see it doesn't leave any room for the client to provide additional parameters outside of the required set of grant_type, username, password– Which you've followed to a T.
My current plan for the backend is to, when provided with the above required set for a 2FA'd user, respond back with an error (be it X-Some-OTP-Header: required, or in the error and error_description keys) informing the app that 2FA is required. A subsequent request is then sent with the OTP, provided by the user, in the header as X-Some-OTP-Header: 123123. But, that's just spitballing and would require ember-simple-auth to support custom headers.
How do you envision a world where 2FA and OAuth 2.0 coexist in Ember, using your library? Is it possible?

I don't know how 2FA would integrate with OAuth 2.0 but of course you can implement a custom authenticator in Ember Simple Auth that does what you describe above.

Seeing that GitHub and others use the X- header strategy as a workaround, would you be opposed to a PR that extends Ember Simple Auth OAuth 2.0 to support the passing/parsing of headers in the options hash?

Don't think it should be added to the library right away since the OAuth 2.0 authenticator can easily be extended. Would be interested in the solution you come up with in a custom authenticator though.

@marcoow I've created a working solution by amending the OAuth 2 authenticator's makeRequest to send a X-VendorName-OTP header if the authenticate options hash contains a OTP parameter.
In order to get access to the headers during an error/reject (as the backend would fail, indicating that a OTP is required only through headers) I had to change authenticate to reject with the raw xhr instead of the response text/json:
makeRequest: function(url, data, otp) {
  return Ember.$.ajax({
    url:         url,
    type:        'POST',
    data:        data,
    dataType:    'json',
    beforeSend:  function(xhr) {
      if (otp) {
        xhr.setRequestHeader('X-VendorName-OTP', otp);
      }
    },
    contentType: 'application/x-www-form-urlencoded'
  });
},

authenticate: function(options) {
  var _this = this;
  return new Ember.RSVP.Promise(function(resolve, reject) {
    // skipping edited code...
    _this.makeRequest(_this.serverTokenEndpoint, data, options['otp']).then(function(response) {
      // ...
    }, function(xhr, status, error) {
      Ember.run(function() {
        reject(xhr);
      });
    });
  });
},

All other OAuth 2 authenticator methods are untouched. All I needed was to deal with headers :smile:

Then simply, I can drive my Ember app as follows:
// app/routes/login.js
import Ember from 'ember';
import UnauthenticatedRouteMixin from 'simple-auth/mixins/unauthenticated-route-mixin';

export default Ember.Route.extend(
  UnauthenticatedRouteMixin,
{
  actions: {
    sessionAuthenticationFailed: function(xhr) {
      if (xhr.getResponseHeader('X-VendorName-OTP') === 'required') {
        // display that a two-factor verification code is required to
        // the user
        this.set('controller.hideUsernameAndPasswordFields', true);
        this.set('controller.displayOTPField', true);
      } else {
        // regular error handling here
      }
    }
  }
});

Looks good - you shouldn't use the sessionAuthenticationFailed though but handle the rejection of the promise returned from this.get('session').authenticate(…) as the sessionAuthenticationFailed is going to be removed in 1.0 anyway.


ring tactic

Implemented ring for addition multiplication and exponentiation only.
Fixed normNum and added support for exponentiation

Don't forget to include documentation and author attributions from the original file! Author name lists should always be monotonic.

Don't forget to include documentation and author attributions from the original file! Author name lists should always be monotonic.

Of course! Sorry I just added it

@AurelienSaue, Mario wrote "monotonic", not constant. You can still add your name.

@AurelienSaue, Mario wrote "monotonic", not constant. You can still add your name.

How about the year? Should it be updated?

I think the usual convention is to leave the earliest year. Probably not how it's supposed to be done but it does give a sense of the age of the file and leads to less churn than updating all the years all the time.


[Add] Add 7 MMOCR projects

Thanks for the contribution. We have added your mentioned projects manually to resolve the conflict


Services - Not able to get details of service secrets using acorn secret reveal

acorn version -v0.7.0-alpha1-39-g696e7f52+696e7f52
Steps to reproduce the problem:
Create acorn service with following Acornfile:
services: "secret-getter": {
  generated: job: "create-secret"
}

jobs: "create-secret": {
  image: "ubuntu"
  files: "/create.sh": """
  #!/bin/sh
  set -x
  cat > /run/secrets/output<<EOF
  services: "secret-getter": {
      secrets: ["output"]
   }
   secrets: "output": {
     type: "opaque"
     data:
        key: "value"
   }
  """
  entrypoint: ["/create.sh"]
}

secrets: output: {
  type: "generated"
  params: job: "create-secret"
}

Consume this service in another app:
containers: {
  nginx: {
    image: "nginx"
    env: {
     "SECRET"<EMAIL_ADDRESS>}
}
}
services: {
     mysec: image: "docker.io/sangeetha/secgetter:v0.0.1"
}

Try to get details of secrets in the consumed service using acorn secret reveal
 % acorn secrets
NAME                       TYPE      KEYS      CREATED
mytestnginx.mysec.output   opaque    [key]     4m44s ago

% acorn secrets reveal mytestnginx.mysec.output
  ✗  ERROR:  secrets.api.acorn.io "mytestnginx.mysec.output" not found

I added an EOF to the end of the shell script and this works now. My Acornfile looks like this:
services: "secret-getter": generated: job: "create-secret"

jobs: "create-secret": {
        image: "ubuntu"
        files: "/create.sh": """
                #!/bin/sh
                set -x
                cat > /run/secrets/output<<EOF
                services: "secret-getter": {
                    secrets: ["output"]
                }
                secrets: "output": {
                  type: "opaque"
                  data: key: "value"
                }
                EOF
                """
        entrypoint: ["/create.sh"]
}

secrets: output: {
        type: "generated"
        params: job: "create-secret"
}

Moving to QA.

Tested with acorn version v0.8.0-alpha7-98-gbb130866+bb130866
Created service acorn and consumed it in an app using the acorn files provided in the bug .
Able to get contents of the secrets using acorn secret reveal as expected.
acorn secret reveal mytestservicenew.mysec.output
NAME                            TYPE      KEY       VALUE
mytestservicenew.mysec.output   opaque    key       value


Doc review chaos 3443

Doc review for CHAOS 3443

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659e3a4c89a086ad54d9aafe--harness-developer.netlify.app


chore(submodule): not sync with .git/config when init project

Hey @hangleang, would you mind explaining what this PR does?

Hi @PaulRBerg, I did use your template for most of my works with solidity, really really thank for build this up.
by the way, the problem is when I publish the source code into git repository, those dependencies within the lib module is not sync with submodules, instead it show as a normal folder.
when I log the config of git repo by command git config --list, it's not show any submodule. I guess it's not sync with those submodules.

Below is the two repo init from this template and my fork PR:

this template: https://github.com/hangleang/foundry-template-test
this fork PR: https://github.com/hangleang/foundry-template-demo

Thank you for explaining.
Unfortunately, I cannot find any issue with the current format of the .gitmodules file. See my conversation with ChatGPT.
It looks like there is an issue with your particular machine and installation of git.

It looks like there is an issue with your particular machine and installation of git.

Ohh that might be the problem of git initialization with my machine, thank for your suggestion.
I'll close the issue for now.

@n00b21337 this template is no longer using submodules.
https://twitter.com/PaulRBerg/status/1736694426578137247


[redfish] Add Redfish authentication

Enable Basic authentication and token authentication as required for
Redfish v1.0
Enable Session Service:
POST: /SessionService/Sessions to login
GET: /SessionService/Sessions to query current sessions
DELETE: /SessionService/Sessions/:id to logout

@RackHD/corecommitters @uppalk1 @keedya @tannoa2

test this please

+1

:+1:

:+1: