Updated service tags note

Updated note regarding service tags. Previous warning that they are a preview feature was out of date (see https://azure.microsoft.com/en-us/updates/service-tags-nsgs-ga/)

@jtuliani : Thanks for your contribution! The author, @markgalioto, has been notified to review your proposed change.

#sign-off

@jtuliani , thank you for the update.


Review usage of (-webkit-)locale

My understanding is that webkit automatically maps lang and xml:lang attributes to the -webkit-locale CSS property → https://bugs.webkit.org/show_bug.cgi?id=67586">bug
It seems it is not directly related to hyphens but the font fallback and the text-transform property.
It could still be useful for spans of text in another language, especially if no xml:lang is provided in the xhtml but it should, in any case, be removed from hyphens and put into overrides (like -webkit-text-fill-color).

Fixed in v. 1.0.1 (moved to overrides)


Run all commands through bash

An alternative to #301
Changes all execs to go through bash --noprofile --norc -c .

#590 pretty much replaced this


Apply low-hanging fruit fixes

This is a meta issue that tracks issues and PRs from repositories that were merged into this one that I likely plan to fix (at least initially). They include

bats-core/bats-file#39
bats-core/bats-file#37
bats-core/bats-file#35
bats-core/bats-file#32
bats-core/bats-assert#31
bats-core/bats-assert#15
bats-core/bats-assert#10
bats-core/bats-assert#9

Here are the corresponding ones for this repository

[ ] #2

For each one, I'll submit a PR with a patch, (with a/, b/ paths modified to match the respective repository) to the original repository as well

Submitted PR for bats-core/bats-assert#31


Pagination

@rarcega  i have used the following script to fetch first 20 records.
instagram-scraper officialslystallone --media_metadata -t none -m 20
will it be possible to fetch records from 21?
is there a way to fetch using pagination.
Thanks

This information would be relatedly helpful for my needs as well. I am interested in grabbing metadata over a specific time-span. I've only recently started babbling in Python (due in part to this tool) and am at a loss for customizing this. And assistance would be greatly appreciated.


Add optional post_init/2 callback to erleans_provider

To resolve #21

Actually it is currently required, the test ets_provider uses it too:
post_init(_ProviderName, _Args) ->
    ets:new(?TAB, [public, named_table, set, {keypos, 1}]).

And it is run in erleans_app:
post_init_providers() ->
    Providers = erleans_config:get(providers, []),
    lists:foreach(fun({ProviderName, Config}) ->
                      Module = proplists:get_value(module, Config),
                      Module:post_init(ProviderName, Config)
                  end, Providers).

I agree it should be optional though, so the post_init_providers function should check if it exists for the provider. It is still unrelated to pooling but if it were optional I guess I'd move the ets table creation to init instead of post_init.
Also wonder if there is a better name than post_init for it.

Oh I was under the impression that post_init is only called when {pool, _} is returned, I will update that!

i don't find the name particulary bad, it was quite straight forward to understand why it exists. delayed_init might be an alternative?


[FLOC-4363] Increase other acceptance test timeouts

Now that we restart the nodes every time we toggle running the container agent or not, our tests take a lot longer to run.
Thus, we should increase all acceptance test timeouts rather than just the ones that GCE was slow on.

LGTM. Please merge when the acceptance tests pass.

All tests but the rackspace ones have passed. I'd rather get this in and re-run the tests on master since master has the other fix as well.


feat: add SAML SSO documentation

Ticket: ME-14
Changelog: None
Signed-off-by: Fabio Tranchitella<EMAIL_ADDRESS>

I'd hope to have an example on how to do this.

I'd hope to have an example on how to do this. There seems to be an outline of an example with Google, but getting the Idp with an example content is missing.

You can use https://samltest.id to test the integration.


When upgrading k3s or rke2 imported clusters, k8s version is not marked 'experimental'

Setup

Rancher version: 2.6-head Commit ID 39cecf0
Browser type & version: Chrome Version 96.0.4664.110 (Official Build) (x86_64)

Describe the bug
In the drop down menus for upgrading imported k3s or rke2 clusters, 1.22 is not marked as (Experimental) - as it is in the UI for provisioning clusters.
To Reproduce

Import a k3s and an rke2 cluster
From the cluster management page, select the 3 dots to the right of each cluster
In the k8s version drop-down for the cluster, scroll to v1.22

Result
1.22 Is not labeled 'experimental' parenthetically
Expected Result
v1.22 is experimental, and should be labeled as such on all upgrade paths
Screenshots

Correct presentation of v1.22

Additional context

Detailed Description

Context

PASS Verified in 2.6-head Commit ID 39cecf0

Upgrade an imported rke2 cluster from 1.21.4+rke2r3 to v1.22.4+rek2r2 and observe that 1.22 is tagged (experimental)
Upgrade a k3s cluster from v1.18.16+k3s1 to v1.22.4+rek2r2 and observe that 1.22 is tagged (experimental)


Support 'no' prefix for negatable flags (e.g. --verbose or --noverbose)

This is a feature request for the "no" prefix modifier where you can prepend 'no' to a flag, and it inverts its value.
For example, imagine a --verbose flag that enables debugging chatter. A user can explicitly use --noverbose to do the opposite action that disables the debugging chatter.
I've used this feature in a some other argument parsing libraries for other languages, so I know the feature has been in the wild. (I'm currently trying to port a Perl utility that uses the standard GetOpt::Long which has the feature.)
I've tried emulating this with 2 flags in a mutex, but it is clumsy. The first problem is you end up with two separate flags with separate variables that still don't know that they need to be inversions of one another. So they can easily be out of sync. In the base case, where neither switch is set, then both are nil. If I set defaults to each, then when the user sets the non-default way, then both flags result in being the same value (since they don't have a direct relationship). In either case, this requires me to write extra setup code to check all the permutations and explicitly set both counterpart values in case I'm inconsistent about which variable I use to compare logic later in my app.
The second problem is you need to write two separate description entries, which can be redundant and clutter things. It seems like there should be a way to combine the entries in some ways. (The switch itself can be abbreviated as --[no]verbose.)
Thank you

Something like this should work currently:
parser:flag "--verbose"
parser:flag "--noverbose"
   :target "verbose"
   :action "store_false"

It's a little, uh, verbose this way though, and there are still two descriptions, so it would be nice to have an easy built-in solution anyway.


How to specify resources in various containers

What did you do?
Similar to https://github.com/coreos/kube-prometheus/issues/372 I am in need of specifying resource requests/limits for all components. So far I failed at finding this info in the documentation and I got lost in the source code.
I need to be able to specify resources for the following containers (preferably via some jsonnet mixin):
 ➭ kubectl ops missing-resources
[...]
monitoring/alertmanager-main/alertmanager
monitoring/alertmanager-main/config-reloader
monitoring/prometheus-k8s/prometheus-config-reloader
monitoring/prometheus-k8s/rules-configmap-reloader
monitoring/prometheus-k8s/thanos-sidecar
[...]

(the above output is from a custom kubectl plugins which lists containers without requests).
Can anyone provide some guidance and/or reference to docs?
Did you expect to see some different?
Environment

Prometheus Operator version:
quay.io/coreos/prometheus-operator:v0.34.0

Kubernetes version information:
v1.15.7

Kubernetes cluster kind:
kops

All of these are coming from the Prometheus Operator.
In this case, you actually can't do anything with jsonnet, but need to adjust the Custom Resource for Prometheus and Alertmanager.

@metalmatze can you point me to some example or at least the fields you're referring to?

Searching for Resources in here https://github.com/coreos/prometheus-operator/blob/master/pkg/apis/monitoring/v1/types.go might help.
Let me know how far that gets you. I am not totally sure about the resource for the sidecars.
It might be worth checking what we do in OpenShift: https://github.com/openshift/cluster-monitoring-operator/tree/master/assets

Thank you @metalmatze this has been very useful, I managed to create a libsonnet file that I can import to setup the resources for most containers. I'm gonna paste it here in case someone in the future will find it useful:
{
  prometheus+:: {
    prometheus+: {
      spec+: {
        resources+: {
          limits+: {
            cpu: '2000m',
            memory: '8Gi'
          },
          requests+: {
            cpu: '200m',
            memory: '2Gi'
          },
        },
        thanos+: { // resources for the sidecar
          resources+: {
            limits+: {
              cpu: '4m',
              memory: '800Mi'
            },
            requests+: {
              cpu: '1m',
              memory: '20Mi'
            },
          }
        }
      }
    }
  },
  alertmanager+:: {
    alertmanager+: {
      spec+: {
        resources+: {
          limits:{
            cpu: '6m',
            memory: '200Mi'
          },
          requests:{
            cpu: '3m',
            memory: '20Mi'
          },
        }
      }
    }
  },
  // see https://github.com/coreos/kube-prometheus/blob/ce7c5fa3b45e5a613bcc4d99d3c6ba5ad38cef81/jsonnet/kube-prometheus/kube-prometheus-strip-limits.libsonnet#L20
  prometheusOperator+:: {
    deployment+: {
      spec+: {
        template+: {
          spec+: {
            local addArgs(c) =
              if c.name == 'prometheus-operator'
              then c + { args+: [ '--config-reloader-cpu=1m', '--config-reloader-memory=25Mi' ] }
              else c,
            containers: std.map(addArgs, super.containers),
          }
        }
      }
    }
  }
}

That said there is one issue that I'm not sure I can solve on my own: the cpu/memory values set for the config-reloader are only for the limits. I couldn't find a way to specify the requests and looking at the  code it would seem like only limits are set.
Should the requests be set at the same values as the limits or should there be other flags for the requests? (or is this left out intentionally for reasons that I couldn't figure out?)

Closing this as it was addressed in #2958


[Suggestion/Enhancement] Number of account per location

Option when putting a pin to decide how many accounts should be allocated to it. This way, if someone is using my accounts, he will be able to not throttle my current scanning.

See the FAQ


Book - LMS - go back link

@xinyil What if this is navigated to directly? Where would the back button go in that scenario?

Can we make this a breadcrumb rather than a back link?  I don't think the functionality should vary based on how someone got there.

The ideal behavior is that when you go one level back in the breadcrumb, you won't lose active search and filters if you have any.
One live example:
https://www.shopspring.com/lifestyle/home/home_decor/decorative_pillows?priceRangeMax=50&refineFlags=on_sale

I think this works properly but I have one comment -- when you push back, it takes you to the beginning of the list of results, instead of the "page" of results from which you clicked on the link to a book. I.e. Levina's Difficile liberté is on p. 7 of the initial results and if you click on "back" it brings you back to page 1, instead of the page from which you were on when you clicked. I'm not sure if this can be improved.

the go back link is not clickable now.


⚠ Bofrost.be has degraded performance

In 4439e2a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1114 ms

Resolved: Bofrost.be performance has improved in 5363ea6.


Update to 1.15.2 and add myself as maintainer

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@davehunt this seems now ready to be merged. 👍

@davehunt this will basically publish the package. It is the correct thing to do because all I did was update the package version. 👍

This will release a new package and add @nicoddemus as a maintainer. Does that make sense @davehunt?

@jakirkham yep, sounds good!

@davehunt if you are OK with it could you please merge the PR then? 😁

Thanks @davehunt!


chore(main): release 0.3.0

:robot: I have created a release beep boop
0.3.0 (2024-04-07)
Features

make colours responsive to background, replace sizes with vars (#36) (e945614)

Bug Fixes

clean up SVG roundel (e3968b0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/commonhorizoncommun/design-system/releases/tag/v0.3.0 :sunflower:


Unclosed resource links

There are many places where Scanner is used without being closed, creating resource leaks

Most of these are scanner-related. Luckily, v2.0 should quickly weed these out.


Reasonable Sync semantics for platforms without InvalidateEntry (OSXFUSE 2.x)

OSXFUSE 2.x does not support invalidation.
Thus, on OS X, incoming sync deleting files cannot invalidate kernel dentry cache, thus cat foo keeps working even after foo has been removed.
Maybe add a secondary mode where if !Protocol().HasInvalidate(), postpone syncs if node is in dir.active? Move reconciliation from Release to Forget?
Currently, the server will log FUSE invalidate error with EINVAL.
Perhaps a miracle will happen and OSXFUSE 3.x will see a stable release before this becomes relevant.

Unfortunately there's no viable open source FUSE for macOS anymore. Closing this, there's not much I can do. For more information, see https://github.com/bazil/fuse/issues/224


[QUESTION] Support for Llama on Azure AI Studio

Question
Is there any possibility to support llama from azure ai studio? Since llama from Ollama and Llamacpp is working

What is it the problem? You should enable azopenai profile and update azopenai.llm_model setting

So what should be the mode? azopenai?
Because when using azopenai i got this

The config is something like this

@jaluma your input will be kindly appreciated. Also i wonder why closing the question so soon?

Nevermind i got it to work with vllms instead of azopenai alone


Move what-is-ic.md to index.md

This moves the contents of the 'What is IC' page to the introduction page.
Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

Hadn't we discussed keeping the what-is-ic page name? Is there a specific reason for removing it and renaming it index.md?
We had agreed that removing the current introduction page index.md was a good revision, but couldn't we just change the 'Build' tab link to go to what-is-ic instead of index.md? Removing what-is-ic is going to potentially break a lot of embedded links throughout the docs.


MiniQtHelper: Add getter and setter for CollapseGroupBox;

A CollapseGroupBox is basically a checkable widget, but there is not support to get the collapsed state for this type of widget.
Here adds the support to get or set a CollapseGroupBox collapsed status in MiniQtHelper, for better extension writting experience.

This looks good, but can you please tidy up the commit history? As in the CONTRIBUTING documentation you should not have merge commits in your history and every commit should compile and comply with the formatting rules.

Sure, the commits have been merged into one commit: https://github.com/baldurk/renderdoc/commit/ac357ea5bc3a8ed750d0e67eb45fc51c46f0bbf0


[Bug] Registration not working

Describe the bug
When trying to register for the first time, it does not work correctly. It just sits on the register page and does not process.
Expected behavior
To register the first admin account.
Desktop (please complete the following information):

Browser: tried Chrome, Opera, Firefox and IE

Are you getting any errors in your browser console? Or in the main console?

So interestingly.. I am running this in an AWS EC2 Instance, and when I tried to run npm run build in the client folder, it came up with this:
ubuntu@cad:~/snaily-cadv3/client$ npm run build

<EMAIL_ADDRESS>build /home/ubuntu/snaily-cadv3/client
react-scripts build

Creating an optimized production build...
The build failed because the process exited too early. This probably means the system ran out of memory or someone called kill -9 on the process.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: react-scripts build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2020-11-03T16_26_40_274Z-debug.log

Might have to upgrade my VPS.. lol - but it was coming up with an error about localhost:3000 not matching the IP or domain name.

Yea, you have to change the localhost in client/src/config.ts to the IP or domain name of your instance. And make sure it matches the one in server/config.ts, make sure to run npm run build in both folders after updating it

so for me it would be: my example IP is c
My server is running on port: 3030
Client is running on port 80
Server config:
const config = {
  port: 3030,
  clientUrl: "http://<IP_ADDRESS>",
  host: "localhost",
  user: "root",
  password: "admin",
  databaseName: "snaily-cad",
  jwtSecret: "bongo super cat",
  env: "production", /* Do NOT change this unless you know what you are doing! */
};

export default config;

Client config:
const SERVER_URL = "http://<IP_ADDRESS>:3030";

export default SERVER_URL;

Hope you understand.


How to link libtorch.dylib to the program during compilation

I want to share my program with my friend, but I can't ask my friend to install libtorch,it's too hard for him.
I would like to know if there is any way to share libtorch.dylib along with others without having to configure it manually. Out of the box and ready to use.
dyld[10602]: Library not loaded: @rpath/libtorch_cpu.dylib
  Referenced from: <D066DB37-3B1F-35A4-B900-B33B6F9E09A3> /Users/ls/Downloads/tch-m1/target/release/tch-m1
  Reason: tried<EMAIL_ADDRESS>(no such file), '/usr/local/lib/libtorch_cpu.dylib' (no such file), '/usr/lib/libtorch_cpu.dylib' (no such file, not in dyld cache)
[1]    10602 abort      /Users/ls/Downloads/tch-m1/target/release/tch-m1

Saving session...completed.

We are on the same platform


VSCode has incorrect schema URL in config

Summary
When using the vsode plugin it has a typo in the schema URL causing it to fail to fetch the schema
schema URL seems that it should be:
https://raw.githubusercontent.com/ansible/ansible-lint/main/src/ansiblelint/schemas/tasks.json
However, in the plugin it refs:
https://raw.githubusercontent.com/ansible/ansiblelint/main/src/ansiblelint/schemas/tasks.json
Issue Type

Are you sure you tried the pre-release version? I am not seeing the bug inside the code which makes be think that maybe you are using the outdated last release.

I just ran into what looks to me like the same issue. It got resolved by updating the outdated VSCode extension.


Incompatible with React bootstrap 3

When we launch a React web app with react bootstrap, it doesn't works because react-bootstrap can't get the react version.
Console return this error "TypeError: Cannot read property 'split' of undefined" and debug tool show error in utils.js:50 of uncontrallable dependency of react-bootstrap.
Thanks

Please install latest version and retry.
I'm looking forward to hearing from you about the compatibility of Nerv and react-bootstrap.


CIV-1368 and CIV-135 for DJ Damages and OCMC RPA changes

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-1368
https://tools.hmcts.net/jira/browse/CIV-135
Change description
RPA feed for DJ Damages and OCMC
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X ] No

Its already merged with master


[Documentation] I think it would be useful to provide docs for artifacts caching

Hello,
When building Clojure apps a lot of artifacts get downloaded.
These can be cached between re-runs for people where this makes sense.
This should improve image build speed - esp in CI servers.
I think more people would use this feature if they new they could and knew where the artifacts are stored in the image so they can mount volumes there.
WDYT?
Eugen

Sure, we could use some more docs around this. However, there's nothing particularly special about Docker here. The local maven repo is in $HOME/.m2/repository like anywhere else.
And otherwise most of this involves adding in your project.clj, deps.edn, or boot.clj file first, installing deps, and then adding in the rest of your code / JAR / whatever so that you only break the deps cache when the file that defines what they are changes. That's pretty common across Docker usage w/ lang-specific dependency managers.
But I've been meaning to re-organize the docs anyway, so I can add a section that spells this all out in there when I do.

https://practical.li/blog/posts/build-and-run-clojure-with-multistage-dockerfile/ article covers using the Offical Clojure Docker image for building Clojure and leveraging the docker cache
Feel free to link or use the content (creative commons attribution license)
Thank you.


Announce renamed EVP_PKEY_cmp() and EVP_PKEY_cmp_parameters() in CHANGES

This is a follow-up of PR #12013 as requested by @t8m: https://github.com/openssl/openssl/pull/12013#issuecomment-638021740

This pull request is ready to merge

Merged.
BTW, ghmerge --nobuild 12026 levitte @t8m beldmit failed at first
apparently because my branch siemens:announce_EVP_PKEY_cmp_rename had become somewhat inconsistent with the master, such that the interactive rebase showed two spurious commits:
pick 890abe372c Announce renamed EVP_PKEY_cmp() and EVP_PKEY_cmp_parameters() in CHANGES.md
pick 5bab5e92b3 Prepare for release of 3.0 alpha 3
pick c83f6ab34d Prepare for 3.0 alpha 4

I solved this by rebasing my branch manually, force-pushing it, and then re-doing ghmerge.


Python3's distinguishing between str and byte-like objects types.

When I tried to run python tools/confugure.py with the required options. An exception is thrown that says
TypeError: cannot use a string pattern on a bytes-like object

All exceptions are thrown where a regular expression (regex) compiles or searches for something.
It seems that the strings are needed to be encoded accordingly. And that works for some scenarios. But there are so many cases where string needs to be encoded before they are used. I keep fixing one but another one appears. Please update your tools to work with modern python versions.

All the Python tooling is now Python2 only, and it will be converted to JS (Node.js) rather than Python3.


point_triangulator problem

Hello,
Thank you for your great work.
When I run pipeline_Aachen ,  use the Aachen dataset.  I face a problem,do you know how to deal it?
_==============================================================================
Triangulating image #1275 (4323)
=> Image sees 3134 / 3560 points
=> Triangulated 241 points
==============================================================================
Triangulating image #855 (4324)
=> Image sees 2652 / 2757 points
=> Triangulated 68 points
==============================================================================
Triangulating image #3514 (4325)
=> Image sees 2534 / 2967 points
=> Triangulated 222 points
==============================================================================
Triangulating image #3771 (4326)
=> Image sees 3402 / 3634 points
=> Triangulated 193 points
==============================================================================
Triangulating image #1486 (4327)
=> Image sees 1421 / 1804 points
=> Triangulated 62 points
==============================================================================
Retriangulation
=> Completed observations: 9730
=> Merged observations: 905932
==============================================================================
Bundle adjustment
[01/31/2021 18:34:39 WARNING] Problem with point_triangulator, exiting.
_

Hi,
Since there are no explicit error messages from COLMAP, I suspect that the point triangulator was simply killed by the OS, for example after running out of memory.
Additionally: please use code fences to report output logs here.
Thanks.

thanks for your reply.How much memory is needed to run this code.

I really depends on the data and on the hardware. For this specific dataset, I don't know. You can check it on your own machine.

thanks, I will try


Share-Screen error (SDK 3.1.0)

when I share-screen it can only share for 15 sec (every time) and it turn to grey screen. How can I solve this problem or it's a SDK bugs? @plutoless @PRWrl @guohai PLS help

What's your platform? Can you share related logs for that?

Sure! I am on iOS platform It doesn't show any error except it can only share-screen for 15 sec (every time) and it turn to grey screen

I think it a SDK <IP_ADDRESS> bugs😢😭

What's your os version and device？ @Apiphoom

@CavanSu iPad pro 11 inch / iPad Gen 7 / iPad Pro 10.5 inch / iPad Gen 5 all of it is iOS 13.5

@CavanSu the last SDK (3.1.0) also have this problem too (I try already try it in iOS 13.6 / 13.5 / 14)

@Apiphoom crash reason is 50 limit? we already know a problem that if the microphone button is turned off before the screen recording, and then start recording, turn on the microphone button during the recording process, the memory will increase above 50M and be killed by the system on ios 13. this problem is cause by apple, you can try to comment out all our code.

@CavanSu My Application didn’t crash but when I use SDK 3.1.0 or <IP_ADDRESS>
I can only share my screen for 15 sec and audience view will change to gray
screen but audience can hear audio but can’t receive any video (It turn to gray screen) This bugs not happens in SDK <IP_ADDRESS>

I try to reproduce, but I didn't reproduce this bug. can you upload broadcaster and audience rtc sdk log?  if you can record a video to tell me how to repruduce this bug, I should be able to fix this problem faster @Apiphoom

@CavanSu Ok I will send video as soon as possible


[DOCS] added some arvr docs

that was some well documented stuff :) easy job :)

Great work, thanks!


dotnet new mvc --auth Individual CreateIdentitySchema class breaks Sqlite migrations

Not sure if this issue might belong in https://github.com/aspnet/Mvc or elsewhere.
Looks like the generated CreateIdentitySchema class needs to be updated to match the current models used for identity so as to not create a migration requiring an AddForeignKey operation which isn't supported by Sqlite.
A simple way to repro the problem:
$ dotnet --version
2.0.0
$ dotnet new mvc --auth Individual
$ dotnet ef migrations add InitialMigration
$ dotnet ef database update

Produces the error:
Applying migration '20170815210528_InitialMigration'.
System.NotSupportedException: SQLite does not support this migration operation ('AddForeignKeyOperation'). For more information, see http://go.microsoft.com/fwlink/?LinkId=723262.
   at Microsoft.EntityFrameworkCore.Migrations.SqliteMigrationsSqlGenerator.Generate(AddForeignKeyOperation operation, IModel model, MigrationCommandListBuilder builder)
   at Microsoft.EntityFrameworkCore.Migrations.MigrationsSqlGenerator.<>c.<.cctor>b__63_1(MigrationsSqlGenerator g, MigrationOperation o, IModel m, MigrationCommandListBuilder b)
   at Microsoft.EntityFrameworkCore.Migrations.MigrationsSqlGenerator.Generate(MigrationOperation operation, IModel model, MigrationCommandListBuilder builder)
   at Microsoft.EntityFrameworkCore.Migrations.MigrationsSqlGenerator.Generate(IReadOnlyList`1 operations, IModel model)
   at Microsoft.EntityFrameworkCore.Migrations.SqliteMigrationsSqlGenerator.Generate(IReadOnlyList`1 operations, IModel model)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.Migrator.GenerateUpSql(Migration migration)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.Migrator.<>c__DisplayClass13_2.<GetMigrationCommandLists>b__2()
   at Microsoft.EntityFrameworkCore.Migrations.Internal.Migrator.Migrate(String targetMigration)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.UpdateDatabase(String targetMigration, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.UpdateDatabase.<>c__DisplayClass0_1.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)
SQLite does not support this migration operation ('AddForeignKeyOperation'). For more information, see http://go.microsoft.com/fwlink/?LinkId=723262.

The dotnet db database update will fail because an AddForeignKey call is required to get the schema up to date and the default Sqlite provider doesn't support that operation.
A workaround is to create the migration, then copy the contents of the newly created *.Designer.cs file to the *CreateIdentitySchema.Designer.cs file, and manually resolve the differences in the new db migration *.cs file with the old *CreateIdentitySchemaDesigner.cs, effectively folding the migration up into the initial schema creation. Then delete the db file so a new one is created.

Closing this. I've created an issue here which seems more appropriate: https://github.com/dotnet/templating/issues/1211


Sending Error

Hi,
I'm having an issue sending ada. I have no pending transactions in the transaction list.
This is what the interface looks like after adding an address and amount.

And here is the console error that I am getting in when this happens:

Also a note about this wallet, the amount in the wallet is almost exclusively rewards from staking. I have created another wallet and transferred all the ada from the old -> new.

Here's the UTXO message:

It is not possible to move rewards in case you have no UTXOs in that wallet as you can not cover fees in that case.
This error message you saw is misleading and has been fixed in this PR: https://github.com/input-output-hk/daedalus/pull/2450
It has already been released on Daedalus Flight 4.0.2-FC3 and we will soon release it on Daedalus Mainnet too.

Awesome! Thank you.


Consider changing autocomplete to any of the keyword words?

Hi :)

this one is face_with_open_eyes_and_hand_over_mouth
or in your data file:
{ keywords: ["face with hand over mouth", "whoops"] }
i couldnt find a way to search for it:

eventually by looking at the code, i realised it can be searched by a spaces:

but tbh, its not really obvious :)
especially when other pickers dont use spaces. here's slack for example:

it can also search for any of the words:

which is easier to guess alot of the time, than typing "face with"
so the autocomplete is:
emoji.keywords.some((word: string) => word.startsWith(text))

would you consider changing to word.includes(text) ?

yours also does not have any way to see the keywords, so you can't go through the grid and learn. consider adding a tooltip to display the keywords?
even something "stupid and simple" like the html title attribute:

unless you wanna make a fancy one
thanks :)

Hi kg,
Thanks for the feedback. I've made a MR with your suggested changes here . Could you check and see if it's missing something you mentioned?

Hi kg,
Thanks for the feedback. I've made a MR with your suggested changes here . Could you check and see if it's missing something you mentioned?

That looks awesome! 😁

Hi kg,
Thanks for the feedback. I've made a MR with your suggested changes here . Could you check and see if it's missing something you mentioned?

That looks awesome! 😁

It's merged!

Hi kg,
Thanks for the feedback. I've made a MR with your suggested changes here . Could you check and see if it's missing something you mentioned?

That looks awesome! 😁

Changes are in version 2.2.27


identify3

identify3 is mainly a shim wrapper around identify3; it runs the existing identify2 engine with an identify3.UIAdpater

this adapter speaks the old IdentifyUI to the identify2 engine, and the new keybase1.Identify3UiInterface to electron

most of the code is in a new directory call identify3/
but we plumb the underlying state through to all G, and that code is in libkb/identify3

this state needs to persist as long as the window is open, so maybe for a while. though we do send a timeout message up to electron after 24 hours

the code tested so far is only for the flow in which electron starts with identify3. Still to come is showing an ID3 popup card as a result of a CLI activity

@patrickxb this is a pretty low-risk change, since it's not being called from anywhere yet.

cc @chrisnojima  this'll go in once green


Extend ModifyManifestTextProcessor flexibility

WIP on extending the flexibility of ModifyManifestTextProcessor to make it easier to do more complicated manifest processing involving manifest entries called something other than "text" or "pred_text".
Need to update the existing unit tests and MLS config as they will not work with the new changes.
Signed-off-by: Elena Rastorgueva<EMAIL_ADDRESS>

Looks good, but should ideally add some more tests for new functionality (e.g., to use attributes other than "text" and "pred_text"). Can be done in later PRs if you plan them


python3: add "--enable-optimizations" to configure flags

Motivation for this change
Python added support profile guided optimizations(PGO)[1]
[1] https://bugs.python.org/issue26359
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

Duplicate of https://github.com/NixOS/nixpkgs/pull/43442.


Upgrade vue (and related dependencies) to 2.7.2

Attempts to resolve errors from #398

@trumanc I'm hoping it will be sufficient to land this PR and then let dependabot bump from 2.7.2 to 2.7.4.
I think going from 2.6.x to 2.7.x is the transition that dependabot isn't able to make automatically.
For the record, all I did here was flush the yarn.lock file and rebuild it so that transitive dependencies of the vue npm package would also be picked up.


Bug fixes

Fixed bugs in centos mirror for son-cli
Restricted "networkx" version to 1.11 (new version is incompatible) - fix issue #242

test this please


index: don't forget to set entry.meta

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.05 :warning:

Comparison is base (e190c55) 57.71% compared to head (ae218e8) 57.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
- Coverage   57.71%   57.66%   -0.05%
==========================================
  Files          51       51
  Lines        3417     3413       -4
  Branches      597      499      -98
==========================================
- Hits         1972     1968       -4
- Misses       1359     1364       +5
+ Partials       86       81       -5

Impacted Files
Coverage Δ

tests/index/test_diff.py
100.00% <ø> (ø)

src/dvc_data/index/index.py
61.42% <100.00%> (+0.09%)
:arrow_up:

... and 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Documentation: How to deploy Policies

Documentation Issue
Hello,
It is possible to include some example how to deploy policies?
Example:
https://github.com/Azure/data-management-zone/blob/main/infra/Policies/PolicyDefinitions/Synapse/params.policyDefinition.Append-Synapse-Purview.json
This policy needs the value for "[parameters('purviewResourceId')]"
If we want to deploy to Subscription we can use:
az deployment sub create -l northeurope -f deploy.policyDefinition.sub.json -p ./PolicyDefinitions/Synapse/params.policyDefinition.Append-Synapse-Purview.json
But I fail to understand two things:

How we can pass the value for the parameter purviewResourceId
how to "mass" deploy multiple policies using a single parameter file

Thanks!
Code of Conduct

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

@marvinbuss


[bug]: Type error found in command.tsx at CommandDialogProps

Describe the bug
The error suggests
An interface declaring no members is equivalent to its supertype.eslint@typescript-eslint/no-empty-object-type

at interface CommandDialogProps extends DialogProps {} in new-york style
Affected component/command.tsx
How to reproduce
It has came after installing the command from cli
Codesandbox/StackBlitz link
No response
Logs
No response
System Info
HP windows

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I don't think this is bug, Eslint tells you that you are creating an interface CommandDialogProps derived from another interface DialogProps and since you are not adding additional members to this CommandDialogProps is same as it's super type DialogProps.

Ohh i see. Thank for letting me know that. Had I new insite today. Though it was a bug


Is there support for 256 colors on the integrated terminal for Windows?

I have configured ZSH as integrated terminal on my VS Code instance on Windows. Like this:
"terminal.integrated.shell.windows": "C:\\Cygwin\\bin\\zsh.exe",
"terminal.integrated.shellArgs.windows": [
    "-lic",
    "cd $OLDPWD; exec zsh"
]

This is working, but colors are not that great. It seems that only 8 are supported.
Here's ZSH running on Mintty (it also comes from Cygwin, no extra setup):

And here's VS Code integrated terminal configured as mentioned above:

I have run set TERM=xterm on VS Conde before running the spectrum_ls command (which prints all the colors). I've also tried set TERM=xterm-256colors but it didn't work either.
Am I missing something?

Hmm, works fine on Linux:

Can you try running this so I know what is being run?
for fgbg in 38 48 ; do #Foreground/Background
  for color in {0..256} ; do #Colors
    #Display the color
    echo -en "\e[${fgbg};5;${color}m ${color}\t\e[0m"
    #Display 10 colors per lines
    if [ $((($color + 1) % 10)) == 0 ] ; then
      echo #New line
    fi
  done
  echo #New line
done

Here you go:

Thanks I can repro, it's likely an upstream issue on Windows.

Meaning a fix will probably take a while?

Created an upstream issue, we'll see what the response is https://github.com/rprichard/winpty/issues/108

@Tyriar I'm not sure if this issue is related but... I've updated my integrated terminal to this configuration (slight change):
"terminal.integrated.shell.windows": "C:\\Cygwin64\\bin\\bash.exe",
"terminal.integrated.shellArgs.windows": [
    "/bin/xhere",
    "/bin/zsh"
]

And I'm using the Oh My Zsh agnoster theme. Thing is, the integrated terminal doesn't even render a black background properly:

The first prompt segment (the one my username@host) should have a black background as you can see from the next segment transition (the > symbol). For some reason, in that first prompt segment, the black background is actual "transparent".
Is this related or a different issue? Or not at all a VS code issue?

@rfgamaral this is an upstream problem, we just render what's given back from winpty https://github.com/rprichard/winpty/issues/108

Based on the discussion in rprichard/winpty#108, does it make sense for VSCode to use wslbridge directly, or is it better to work on implementing support in winpty for "Cygwin/MSYS2/WSL" as mentioned in https://github.com/rprichard/winpty/issues/108#issuecomment-296307763?

Closing in favor of the upstream issue, we don't plan on investing in this area.


[UPLOAD-942] clinicians without a clinic

to handle UPLOAD-942 to guide clinicians without a clinic yet to go join or make one on Tidepool Web

@clintonium-119 could I get a re-review (just did some design feedback tweaks)?


Implement "safe" summary mode for DDSs

If DDS factory registration is removed from data store, then DDS will fail to summarize, causing "safe" summary at data store level.
We can do better and push safe summary down to DDS level in such cases, improving perf. Without that, failure at that level will result in data store to always using safe summaries, and very quickly - very slow perf.

Created a branch with the core work done: dds-failure-summary as a baseline in the future when we want to implement this.
It takes care of the following:

Enable failure summaries in SummarizerNodes at DDS level
Call loadBaseSummary when loading base snapshots at DDS level
Change DDS op handling from invalidate to recordChange to track ops

Holding off for now, since the work to consolidate op storage is not done and is nontrivial.

perf: move op storage to leaf SummarizerNodes or to root node with readdressing to prevent duplicating op tracking

I believe this is a punt. Mostly because we learned the hard way that going too far with this approach makes things worse - system limps along, but perf starts going down quickly, and complexity is too high, making more room for errors.
@arinwt - please reactivate if you disagree.


question about dataset for only train background without foreground？

I want to train a scenario where there is no foreground. But I found that when loading the dataset, frames without objects will be filtered out, and simply retaining all images will result in corresponding other errors. How should I modify this situation?

Hi! Thanks for your interest! Could you please provide more information about the issue you are encountering?

Thank you very much for your prompt response！
I found that the division of training and testing sets is based on ”visible_objects“。 If there are no ”visible_objects“ in my scene, some errors will occur due to "indices" being zero or None。
For example：
”obj_info = torch.cat([obj_nodes_tensor[i : i + 1] for i in indices], dim=0)“

Hi! If you only need the background node, I would suggest using the original Nerfacto model from nerfstudio. Because the main goal of our repository is to compose nodes together rather than obtaining a single node.


Update pollution API implementation

@xaque208 i think the Main.Aqi can be an int based on this:

Thanks for the contribution!


Error compiling npm distribution with txt type bundle module

When compiling an npm dsitrubution and it has a module that has a text bundle together with another code bundle, the following error is displayed

Hi @FranciscoSanvicente ,
. To assist you better, we'll need more information and specific details about the problem. Could you please provide the following:
Code Snippets:
Share relevant parts of the code, especially the module that contains the text bundle and any other code bundles associated with it. This will help us analyze the structure and identify potential issues.
Reproduction Steps:
Please provide step-by-step instructions on how to reproduce the error. Include any specific commands, configurations, or dependencies required for the compilation process. This will allow us to recreate the issue locally and pinpoint the problem.
Once we have this information, we'll be better equipped to understand and address the issue you're facing. Feel free to reach out if you have any questions or need further clarification.
Thank you for your cooperation.
Best regards,

Hola
Para reproducirlo seguir los siguientes pasos:
Crear un proyecto con beyond con
npx @beyond-js/create-package@latest

Crear modulo con bundle de tipo code y txt:
/module.json
{
  "name": "header",
  "platforms": [
    "web",
    "ios",
    "android"
  ],
  "txt": {
    "path": "/txt",
    "multilanguage": true,
    "files": "*"
  },
  "code": {
    "ts": {
      "path": "/code/ts",
      "files": [
        "*"
      ]
    }
  }
}

/code/ts
import * as React from 'react';
import { useTexts } from '@beyond-js/react-18-widgets/hooks';
import { module } from "beyond_context";

export /*bundle*/ function Header(): JSX.Element {
    const [ready, texts]: [boolean, any] = useTexts(module.specifier);

    if (!ready) return null;
    return (
        {texts.title}
    )
};

/txt/texts.json
{
    "es": {
        "title": "Esto es un titulo de un bundle de texto en un modulo que tiene un bundle de tipo code"
    },
    "en": {
        "title": "This is a title of a text bundle in a module that has a code bundle."
    }
}

En el home, importar y usar el componente creado de la siguiente forma:
import * as React from 'react';
import { Header } from 'test-npm/header.code'
export /*bundle*/
	function View(): JSX.Element {
	return (

	);
}

Luego de eso corremos beyond, compilamos seleccionando la distribucion npm dando el error


Invalid completion suggestions given

Expected behavior
Autocompletion only suggests commands from the completion list.
Actual behavior
Commands not present in completion list are suggested.
Steps to reproduce
See: https://jsfiddle.net/a8po3ran/1/
Type base and press TAB. It will be completed to base64code which is not in the list.
Browser and OS
Ubuntu 16.04, both Firefox and Chrome.

Btw this was working fine in 1.5.2. Not sure though at what version exactly it was broken.

the afected version was 1.8.0 that fixed one issue with mixed case and case insensitive completion.
The fix will be in 1.11.1


fix: stringify unknown errors better

PR-Codex overview
This PR focuses on enhancing error handling in the codebase by introducing a new prettifyError function and updating existing error formatting to use this function. It also adds a prettifyTransactionError function for transaction-specific errors.
Detailed summary

Added prettifyError function in src/utils/error.ts for improved error message formatting.
Removed the old formatError function in src/server/middleware/error.ts.
Updated error handling in src/utils/transaction/queueTransaction.ts to use prettifyError.
Updated error handling in src/server/routes/contract/write/write.ts to use prettifyError.
Updated error message formatting in src/worker/tasks/sendTransactionWorker.ts to use prettifyError.
Introduced prettifyTransactionError function in src/utils/error.ts for transaction error handling.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Merge activity

Oct 8, 6:16 PM EDT: @arcoraven started a stack merge that includes this pull request via Graphite.


Add prometheus endpoint for key ages

The app is going to be pretty ephemeral by nature so probably not wise to have a prom endpoint


CI publish release job not including ARM64 wheels for Python

It looks like the latest release of the python module (0.26.0) does not contain packages for ARM64. This can also be seen in the download files list on pypi: https://pypi.org/project/oso/0.26.0/#files.
There were files for prior releases, such as 0.25.1: https://pypi.org/project/oso/0.25.1/
Unfortunately in comparing the tags between 0.25.1 and 0.26.0 it is not clear to me why this happened, so unable to propose a PR.

Hey @lambertjosh ! Thanks for raising this, looking into it now.

You're absolutely right. The ARM wheels were built as part of the CI create release job (https://github.com/osohq/oso/actions/runs/1746903004) but for some reason not uploaded to pypi. I downloaded these and uploaded them manually, so now the ARM builds for 0.26 should be there.
I'll update this issue to reflect the underlying issue so we can track the problem here. Thank you again for letting us know there was a problem.

I'm stumped. Leaving what I found.
If I download the wheel.zip artifact from the build, it has the ARM wheels. But the release job omitted it when zipping them and attaching to the tag

Files uploaded to release
Run zip --junk-paths -r oso-python.zip wheel/
  adding: oso-0.26.0-pp37-pypy37_pp73-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp310-cp310-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp39-cp39-win_amd64.whl (deflated 1%)
  adding: oso-0.26.0-cp38-cp38-musllinux_1_1_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp39-cp39-musllinux_1_1_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp39-cp39-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp310-cp310-macosx_10_9_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp37-cp37m-win_amd64.whl (deflated 1%)
  adding: oso-0.26.0-cp39-cp39-macosx_10_9_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp38-cp38-win_amd64.whl (deflated 1%)
  adding: oso-0.26.0-cp36-cp36m-musllinux_1_1_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-pp37-pypy37_pp73-macosx_10_9_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp37-cp37m-macosx_10_9_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp37-cp37m-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-pp37-pypy37_pp73-win_amd64.whl (deflated 1%)
  adding: oso-0.26.0-cp36-cp36m-win_amd64.whl (deflated 1%)
  adding: oso-0.26.0-cp310-cp310-musllinux_1_1_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp36-cp36m-macosx_10_9_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp310-cp310-win_amd64.whl (deflated 1%)
  adding: oso-0.26.0-cp38-cp38-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp37-cp37m-musllinux_1_1_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp36-cp36m-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (deflated 1%)
  adding: oso-0.26.0-cp38-cp38-macosx_10_9_x86_64.whl (deflated 1%)

No idea how the wheels are disappearing between the steps.

Thanks for taking such a quick look @samscott89 and resolving the immediate issue with availability on PyPi.


Add tmux support.

I followed along this post and it seems that modern versions of tmux have a special pass-through sequence that lets you get to the outer terminal. Chunking doesn't appear to be required.
Lightly tested, but seems to work.

sorry, but this repo is no longer maintained.  please send a CL via gerrit to the new home:
https://chromium.googlesource.com/apps/libapps/
documentation for submitting can be found here:
https://chromium.googlesource.com/apps/libapps/+/HEAD/HACK.md


Deprecate Multiple Videos

Deprecate ability to play multiple videos one-after-another in the same view, due to issues with how Apple handles each video. This feature will be removed in release 3.0.0.

[ ] Add deprecation message for next patch release
[ ] Remove feature in release 3.0.0
[ ] Add alternative solution using FFMPEG to concatenate video assets

For reference this is a rough cut of the code that worked for me for merging vids (sound removed):
func merge(arrayVideos: [AVAsset], completion: @escaping (AVAssetExportSession) -> ()) -> Void {
        let mainComposition = AVMutableComposition()
        let compositionVideoTrack = mainComposition.addMutableTrack(withMediaType: .video, preferredTrackID: kCMPersistentTrackID_Invalid)

        var insertTime = kCMTimeZero

        for videoAsset in arrayVideos {
            try! compositionVideoTrack?.insertTimeRange(CMTimeRangeMake(kCMTimeZero, videoAsset.duration), of: videoAsset.tracks(withMediaType: .video)[0], at: insertTime)
            insertTime = CMTimeAdd(insertTime, videoAsset.duration)
        }

        let outputFileURL = URL(fileURLWithPath: NSTemporaryDirectory() + "merge.mp4")

        let fileManager = FileManager()
        try? fileManager.removeItem(at: outputFileURL)

        let exporter = AVAssetExportSession(asset: mainComposition, presetName: AVAssetExportPresetHighestQuality)

        exporter?.outputURL = outputFileURL
        exporter?.outputFileType = AVFileType.mp4
        exporter?.shouldOptimizeForNetworkUse = true

        exporter?.exportAsynchronously {
            DispatchQueue.main.async {
                completion(exporter!)
            }
        }
    }

Maybe we can keep it in...there are uses for it, like when you only know what videos you want to play at runtime.
That said it should really only be promoted if we:

make a vid merging lib
pull in that lib as a dependency and merge vids, then play

But yes we should def encourage using something like FFMPEG to pre-merge your vid wherever possible (such as a static app welcome screen vid, which is prob mostly what this lib is used for)


Random background pattern 🐚

...just like the seashell ejomi in the title is random

Moving away from pattern backgrounds.


Serverless stops working after adding middleware (only in production)

serverless.yml has this configuration:
    - serverless-middleware # Must stay before serverless-offline and serverless-webpack
    - serverless-webpack
    - serverless-offline
    - serverless-prune-plugin

custom:
    middleware:
        pre:
            - src/common/request_middleware/index.beforeRequest
        pos:
            - then: src/common/request_middleware/index.afterRequest
            - catch: src/common/request_middleware/index.errorHandler

It works well when running locally, but when running in production, all api requests return a status code 500, Internal Server Error. What might be the cause?

I have absolutely no idea with the given info 😅
What does the error say?
What do the logs say?

Sorry, on cloudwatch I get this for all functions:
ERROR	Uncaught Exception
{
    "errorType": "Runtime.ImportModuleError",
    "errorMessage": "Error: Cannot find module 'flomatika-api-production-settings'\nRequire stack:\n- /var/runtime/UserFunction.js\n- /var/runtime/index.js",
    "stack": [
        "Runtime.ImportModuleError: Error: Cannot find module 'flomatika-api-production-settings'",
        "Require stack:",
        "- /var/runtime/UserFunction.js",
        "- /var/runtime/index.js",
        "    at _loadUserApp (/var/runtime/UserFunction.js:100:13)",
        "    at Object.module.exports.load (/var/runtime/UserFunction.js:140:17)",
        "    at Object.<anonymous> (/var/runtime/index.js:43:30)",
        "    at Module._compile (internal/modules/cjs/loader.js:999:30)",
        "    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)",
        "    at Module.load (internal/modules/cjs/loader.js:863:32)",
        "    at Function.Module._load (internal/modules/cjs/loader.js:702:16)",
        "    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)",
        "    at internal/main/run_main_module.js:17:47"
    ]
}

The deployment logs seem all right:
Serverless: Middleware: setting 4 middlewares for function flomatika-api-production-settings
Entrypoint .middleware/flomatika-api-production-settings = .middleware/flomatika-api-production-settings.js .middleware/flomatika-api-production-settings.js.map

I created a test endpoint and applied middleware only to it. The rest didn't break, which means the plugin is not breaking everything, but the test endpoint had the same error:

    test-middleware:
        handler:
            - src/common/request_middleware/index.beforeRequest
            - src/organization-settings/middleware_test.getEverything
            - then: src/common/request_middleware/index.afterRequest
            - catch: src/common/request_middleware/index.errorHandler
        memorySize: 1792
        events:
            - httpApi:
                  path: /middleware-test
                  method: GET
                  authorizer: ${file(./serverless/authorizer.js)}````

2021-11-10T13:28:14.880Z	undefined	ERROR	Uncaught Exception
{
"errorType": "Runtime.ImportModuleError",
"errorMessage": "Error: Cannot find module 'flomatika-api-production-test-middleware'\nRequire stack:\n- /var/runtime/UserFunction.js\n- /var/runtime/index.js",
"stack": [
"Runtime.ImportModuleError: Error: Cannot find module 'flomatika-api-production-test-middleware'",
"Require stack:",
"- /var/runtime/UserFunction.js",
"- /var/runtime/index.js",
"    at _loadUserApp (/var/runtime/UserFunction.js:100:13)",
"    at Object.module.exports.load (/var/runtime/UserFunction.js:140:17)",
"    at Object. (/var/runtime/index.js:43:30)",
"    at Module._compile (internal/modules/cjs/loader.js:999:30)",
"    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)",
"    at Module.load (internal/modules/cjs/loader.js:863:32)",
"    at Function.Module._load (internal/modules/cjs/loader.js:702:16)",
"    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)",
"    at internal/main/run_main_module.js:17:47"
]
}

The error seems clear: the package flomatika-api-production-settings can't be found.
This most likely has nothing to do with the plugin and you are just missing it in the package.json file.

The error seems clear: the package flomatika-api-production-settings can't be found. This most likely has nothing to do with the plugin and you are just missing it in the package.json file.

Hi, @juanjoDiaz, thanks for trying to help.
So, the package flomatika-api-production-settings is just my lambda... It exists and works when I remove the middleware configuration, that seems to happen to all 118 lambdas on the api in prod. When I set up only one endpoint to have middleware, then only that endpoint gets this error, and only in prod.
I would guess it is related to the fact that with the plugin, we get a path .middleware/something. Seems to me that when the path for the function wrapped with middleware is set up, then the path to the actual function (which is used within the wrapped function) becomes unrecognized.

https://github.com/juanjoDiaz/serverless-middleware/issues/33
This issue has a similar error, in his case it was caused by having the plugins in the wrong order.

I'm having a similar issue with this
using sls offline start As suggested in this answer solved -parcially- the issue, but when deploying to production, same error occurs again ServerlessError: Configuration error at 'functions['get-samples-sample'].handler': should be string

Which version of serverless are you using?
That last issue seems to point towards a validation issue by the serverless framework.
It expects the property handler to be a function. However, this plugin overrides it to be also array.
I've just released v0.0.15 which changes the plugin so all middlewares must be declared in the custom.middleware section.
Can you please test?

Will do, I'll come back later

Closing since there was no response.
Feel free to reopen if you feel that it's needed.

Hi, @juanjoDiaz I have added serverless middleware to lambda, my serverless middleware throws an error but lambda runs perfectly fine. Although if middleware breaks it should throw an error and should not let lambda run, am i right? Please reply it's urgent. I'll share my cloudwatch logs as if required.


Call ptr->reset() before CVD_FREE

Releated:

#64
#97

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

Project coverage is 90.35%. Comparing base (c3ab8f6) to head (0dd6267).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage   90.35%   90.35%
=======================================
  Files          35       35
  Lines        5327     5327
=======================================
  Hits         4813     4813
  Misses        514      514

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


added a flush method to IPC writers

Which issue does this PR close?

Part of #6099.
Rationale for this change

While the writers expose get_ref and get_mut to access the underlying io::Write writer, there is an internal layer of a BufWriter that is not accessible. Because of that, there is no way to ensure that all messages written thus far to the StreamWriter or FileWriter have actually been passed to the underlying writer.
What changes are included in this PR?

Here we expose a flush method that flushes the internal buffer and the underlying writer. While the issue was specifically about StreamWriter, I don't see a reason for which this wouldn't be applicable to FileWriter and it keeps the API consistent between the two, so it is added there as well.
Are there any user-facing changes?
Two new public methods:

arrow::ipc::writer::StreamWriter::flush
arrow::ipc::writer::FileWriter::flush

No breaking changes.

Thanks again @V0ldek
FYI this (just) missed the 52.2.0 release candidate so it will likely be released as part of 53.0.0 in about a month


ui: Align adapter cards and make it vertically even

Signed-off-by: Abhishek-kumar09<EMAIL_ADDRESS>Description
Adapter cards was mis aligned. The PR fixes alignment issues with the cards.

Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #3695 (a7c8730) into master (ed2326b) will decrease coverage by 7.80%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3695      +/-   ##
==========================================
- Coverage   20.59%   12.79%   -7.81%
==========================================
  Files          45       45
  Lines        3807     3807
==========================================
- Hits          784      487     -297
- Misses       2817     3205     +388
+ Partials      206      115      -91

Flag
Coverage Δ

gointegrationtests
?

unittests
12.79% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.11% <0.00%> (-53.54%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.90% <0.00%> (-39.10%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.42% <0.00%> (-21.71%)
:arrow_down:

Continue to review full report at Codecov.

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

Nice! And they remain responsive?

Yes

Wow, this looks great! @Abhishek-kumar09


Update docs for ResourceQuota and QoS

Changes
This commit updates compute resources documentation to reflect how Tekton
currently works with ResourceQuotas and how to achieve different levels of
quality of service for Tekton pods.
Closes #4533.
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
n/a Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
NONE

/retest

/lgtm

/test pull-pipeline-kind-k8s-v1-21-e2e

/retest

/test pull-pipeline-kind-k8s-v1-21-e2e

/test all

/test all

@lbernick fyi, /test all won't stop kicking off the pull-tekton-pipeline-kind-* jobs for another ~40 minutes, when the cronjob updating the Prow config runs.

fyi, /test all won't stop kicking off the pull-tekton-pipeline-kind-* jobs for another ~40 minutes, when the cronjob updating the Prow config runs.

ah got it thanks Andrew!

/test all

/lgtm


Does not save watermark in FILE_STORAGE_BASE_DIR

Hi.
I noticed that imagor does not save the source file used for the watermark.
If you make 3 requests:
/filters:watermark(test/watermark.png,repeat,repeat,0,100,100)/test/1.jpg
/filters:watermark(test/watermark.png,repeat,repeat,1,100,100)/test/1.jpg
/filters:watermark(test/watermark.png,repeat,repeat,2,100,100)/test/1.jpg
access.log s3 will show:
GET /test/1.jpg
GET /test/watermark.png
GET /test/watermark.png
GET /test/watermark.png

Have to pay more for the s3 service due to unnecessary requests =(

To understand more on the context, could you please share the environment variables being configured for imagor? You shall mask any sensitive info such as AWS_SECRET_ACCESS_KEY

# IMAGOR_SECRET is a secret key that will help you subscribe to the hashtag links to maps
IMAGOR_SECRET="imagor-secret"
# IMAGOR_SIGNER_TYPE - hashing method and signature uri
IMAGOR_SIGNER_TYPE="sha256"
# IMAGOR_SIGNER_TRUNCATE - the length of the hash in the uri
IMAGOR_SIGNER_TRUNCATE=40
# IMAGOR_STORAGE_PATH_STYLE - the path to the image and its replacement with e6/86/1a810ff186b4f747ef85f7c53946f0e6d8cb
IMAGOR_STORAGE_PATH_STYLE="digest"
# IMAGOR_RESULT_STORAGE_PATH_STYLE - path to files and their replacement with statically deleted files e6/86/1a810ff186b4f747ef85f7c53946f0e6d8cb
IMAGOR_RESULT_STORAGE_PATH_STYLE="digest"
# IMAGOR_REQUEST_TIMEOUT - Maximum time to execute an imagor request
IMAGOR_REQUEST_TIMEOUT=60s
# IMAGOR_PROCESS_CONCURRENCY - A few pros can be recycled daily
IMAGOR_PROCESS_CONCURRENCY=4
# IMAGOR_PROCESS_QUEUE_SIZE - Some information that can be expected today (429 times in a row)
IMAGOR_PROCESS_QUEUE_SIZE=64
# HTTP_LOADER_DEFAULT_SCHEME - as I understand it, this is just an e-book reader application (if there is no HTTP_LOADER_BASE_URL)
HTTP_LOADER_DEFAULT_SCHEME="https"
# Modified sources and base URL
#HTTP_LOADER_ALLOWED_SOURCES=*.example.com
# HTTP_LOADER_BASE_URL - link to the site for any changes
HTTP_LOADER_BASE_URL="https://source.example.com/"
# IMAGOR_UNSAFE is only 1, then you don't need to use a hash to send requests
IMAGOR_UNSAFE=0
# IMAGOR_AUTO_WEBP - if 1, then it will now transmit webp in jpeg format if it does not support webp
IMAGOR_AUTO_WEBP=1
FILE_STORAGE_MKDIR_PERMISSION=0755
FILE_STORE_WRITE_PERMISSION=0666
FILE_STORE_EXPRESSION=24h # not working
FILE_RESULT_RESULT_MKDIR_RESULT=0755
FILE_RESULT_RESULT_RELEASE=0666
EXPIRATION_RESULT_RESULT_FILE=168h
SERVER_ACCESS_LOG=0
VIPS_CONCURRENCY=1

# Maximum allowed image resolution and dimensions
VIPS_MAX_RESOLUTION=225000000
VIPS_max_WIDTH=15000
VIPS_MAX_HEIGHT=15000

If you wish activate file storage, FILE_STORAGE_BASE_DIR needs to be present:
https://github.com/cshum/imagor?tab=readme-ov-file#file-system
FILE_STORAGE_BASE_DIR=/mnt/data # enable file storage by specifying base dir
FILE_STORAGE_MKDIR_PERMISSION=0755 # optional
FILE_STORAGE_WRITE_PERMISSION=0666 # optional

If you wish activate file storage, FILE_STORAGE_BASE_DIR needs to be present:
https://github.com/cshum/imagor?tab=readme-ov-file#file-system
FILE_STORAGE_BASE_DIR=/mnt/data # enable file storage by specifying base dir
FILE_STORAGE_MKDIR_PERMISSION=0755 # optional
FILE_STORAGE_WRITE_PERMISSION=0666 # optional

Sorry. I have not shown all the settings, this part is written in another file. I completely forgot about him.
However, as I wrote at the beginning, all files except the watermark are saved normally.
        environment:
            PORT: 8000
            IMAGOR_UNSAFE: ${IMAGOR_UNSAFE} # unsafe URL for testing
            IMAGOR_AUTO_WEBP: ${IMAGOR_AUTO_WEBP}
            IMAGOR_SECRET: ${IMAGOR_SECRET}
            IMAGOR_SIGNER_TYPE: ${IMAGOR_SIGNER_TYPE}
            IMAGOR_SIGNER_TRUNCATE: ${IMAGOR_SIGNER_TRUNCATE}
            IMAGOR_STORAGE_PATH_STYLE: ${IMAGOR_STORAGE_PATH_STYLE}
            IMAGOR_RESULT_STORAGE_PATH_STYLE: ${IMAGOR_RESULT_STORAGE_PATH_STYLE}
            IMAGOR_PROCESS_CONCURRENCY: ${IMAGOR_PROCESS_CONCURRENCY} # Maximum number of image process to be executed simultaneously. Requests that exceed this limit are put in the queue. Set -1 for no limit (default -1)
            HTTP_LOADER_DEFAULT_SCHEME: ${HTTP_LOADER_DEFAULT_SCHEME}
            HTTP_LOADER_BASE_URL: ${HTTP_LOADER_BASE_URL}
            FILE_LOADER_BASE_DIR: /mnt/data/source # enable file loader by specifying base dir
            FILE_STORAGE_BASE_DIR: /mnt/data/source # enable file storage by specifying base dir
            FILE_STORAGE_MKDIR_PERMISSION: ${FILE_STORAGE_MKDIR_PERMISSION} # optional
            FILE_STORAGE_WRITE_PERMISSION: ${FILE_STORAGE_WRITE_PERMISSION} # optional
            FILE_RESULT_STORAGE_BASE_DIR: /mnt/data/result # enable file result storage by specifying base dir
            FILE_RESULT_STORAGE_MKDIR_PERMISSION: ${FILE_RESULT_STORAGE_MKDIR_PERMISSION} # optional
            FILE_RESULT_STORAGE_WRITE_PERMISSION: ${FILE_RESULT_STORAGE_WRITE_PERMISSION} # optional
            VIPS_CONCURRENCY: ${VIPS_CONCURRENCY} # VIPS concurrency. Set -1 to be the number of CPU cores (default 1)

I see. Yes currently the watermark image gets loaded and does not get saved.
But if it is the same watermark for every images, you may be able to resolve your problem by requesting the watermark image alone
/test/watermark.png

then the subsequence watermark image would be loaded from storage instead.

I see. Yes currently the watermark image gets loaded and does not get saved.
But if it is the same watermark for every images, you may be able to resolve your problem by requesting the watermark image alone
/test/watermark.png

then the subsequence watermark image would be loaded from storage instead.

thanks. It really works.
I repeat the request periodically in case the source is cleared:
1 * * * * curl -I https://img-example.com/hash/test/watermark.png --header 'user-agent:imagor-crontab' > /tmp/crontab.imagor.watermark.fix.txt


plugins/formatter-nvim: init

Add support for formatter-nvim
TODO:

Look for missing formatter packages (help apreciated)
Allow user to choose custom formatter packages
allow disabling auto install of formatters

The only thing still missing are some packages. I couldn't find packages for all supported formatters, in part because some seem to be old and unmaintained


Mouse Inputs Passthrough from egui to bevy_panorbit_camera

When not directly scrolling / dragging / clicking egui ui elements, all inputs get passed through to bevy_panorbit_camera when using the following code to add ui systems:
add_systems(Update, (draw_title_bar, draw_file_tree).chain())

As demonstrated here:
https://github.com/Plonq/bevy_panorbit_camera/assets/1371787/cf471495-b9d0-4da2-9d53-e7a148f7520e
This appears to be a scheduling issue, and doing the following somewhat resolves the issue and blocks scroll events, but not mouse drag events:
add_systems(PreUpdate, (draw_title_bar, draw_file_tree).chain().after(EguiSet::BeginFrame))

As demonstrated here:
https://github.com/Plonq/bevy_panorbit_camera/assets/1371787/ddb11fee-1224-4b8d-9061-3ac5916e4fc8
Using the solution mentioned in #39 yields no further improvements.
Here is the code used for both of the above:
repro.zip

Hey, thanks for the repro. This appears to be a quirk of egui, haven't found a solution yet.
So, bevy_panorbit_camera tracks a boolean value in EguiWantsFocus reflecting the values of Context::wants_pointer_input and Context::wants_keyboard_input. If either of those return true in the current and previous frames, then panorbit ignores inputs.
The problem is that these values are false when they should seemingly be true. I tested your repro and logged to console whenever EguiWantsFocus changed. The results are surprising.
First I tested without your half-workaround, i.e. using add_systems(Update, (draw_title_bar, draw_file_tree).chain()). Most of the time, this resulted in neither dragging nor scrolling working as expected. Sometimes it resulted in scroll being blocked correctly. If I use the half-workaround, it consistently blocks scroll. There's obviously some sort of scheduling issue, but it looks like it's within egui somehow.
This first video shows what happens when both scrolling and dragging 'pass through' the egui side panel. You can see that EguiWantsFocus never changes, which means obviously panorbit assumes egui doesn't need input and thus uses the input. I added an egui Window as well, and that works as expected, which tells me egui treats anything that isn't a Window differently when it comes to input.
https://github.com/Plonq/bevy_panorbit_camera/assets/7709415/f4b52e9c-a1c5-4c7f-b848-80f17a4c606f
The second video shows the half-workaround. You can see the difference is that on hover EguiWantsFocus correctly changes to true, blocking scrolling. But as soon as you click (on mouse down), it changes back to false, thus panorbit uses the drag/rotate input.
https://github.com/Plonq/bevy_panorbit_camera/assets/7709415/bbeb5600-ecdf-41f5-8929-7cc5720c2a1d
My conclusion so far is that there's nothing this plugin can do - it's a matter of wants_pointer_input and/or wants_keyboard_input acting differently for side panels vs windows. I'm not really familiar with egui (haven't actually used it to build UI or anything), so maybe you have some ideas?
P.S. I tried moving all panorbit's systems into PostUpdate and it made no difference.

@Plonq after some investigation, I've found one solution that you can implement for now, the following code:
let new_wants_focus = windows.iter().any(|window| {
    let ctx = contexts.ctx_for_window_mut(window);
    ctx.wants_pointer_input() || ctx.wants_keyboard_input()
});

Should also check if the cursor has entered an egui area:
let new_wants_focus = windows.iter().any(|window| {
    let ctx = contexts.ctx_for_window_mut(window);
    ctx.wants_pointer_input() || ctx.wants_keyboard_input() || ctx.is_pointer_over_area()
});

Which then results in the following behavior, which I would say is most wanted/expected:
https://github.com/Plonq/bevy_panorbit_camera/assets/1371787/0960ae6b-24ed-4ab6-85e0-4e1447be6517
Do note however, as mentioned here, this should occur in the frame as late as possible. So perhaps we should combine it with PostUpdate as well, as to not breaking it for other plugins.

That is a decent temporary workaround, but it's not ideal. If you start dragging outside egui, and drag over egui, it should continue that drag operation (e.g. rotating the camera). This is a very common UX pattern. For example if you click and drag a scrollbar you can move the mouse anywhere - it doesn't just stop working if you move your mouse outside the scrollbar track.
For this reason, I have implemented it as an optional configuration, disabled by default. See https://github.com/Plonq/bevy_panorbit_camera/pull/76, which has been released in v0.18.1

Two possible "solutions" to avoid the camera handling stopping to work if the cursor quit the Bevy viewport:

Grab the cursor so it does not move while handling the camera
Warp the cursor around the viewport (Blender style). If the cursor quit the viewport by going to the left, put it on the right of the viewport. This does not completely because of toolbars or windows possibly overlapping the viewport. Also if the cursor quit the application window entirely, there is no known way (to me at least) to get the cursor position.

Unfortunately this is platform specific and even depends on the windowing system (X11 or Wayland) on Linux.

I don't particularly like the sound of either of those solutions.

Grabbing the cursor would feel odd and unintuitive
The egui panels display on top of the viewport, so this would need to be egui-specific

The best idea I have is to save some sort of state that regarding the current 'gesture', e.g. 'panning', and then if currently panning, ignore the 'egui wants focus' check. When the user lets the mouse go, the 'panning' gesture stops and thus the 'egui wants focus' check is once again in effect.
The only thing is, this adds a bunch of complexity for something for a very specific use case that shouldn't be a problem in the first place, which is why I'm hesitant.

One solution I found for bevy_blendy_cameras (not committed yet) is pretty much the following:

Include the check to see if pointer is over an area in the check_egui_wants_focus system
Run the pan_orbit_camera system even if egui want the focus but only if ActiveCameraData is "set"
In the active_viewport_data system: Do not set the ActiveCameraData if the drag starts when egui wants the focus. In those case, set ActiveCameraData to default()

This fixes this issue and another related (invert case):

When the drag starts on an egui area overlapping the viewport, the camera orbiting happens when the cursor goes out of the egui Area.
This one can be tested by setting the widow to immovable (movable(false)) in the egui example.

Thanks for commenting with this idea! I will try it out. Or if you're willing I'd be glad to accept a PR.
bevy_blendy_cameras sounds great. I'm guessing you skipped the smoothing (like blender)? That was the one thing that made adding auto-depth ultimately impractical to implement in this plugin. (At least with the way I implemented smoothing.) So it's cool that there's another plugin that implements it.

Edit: Unrelated to this issue but I thought you might want to know. While developing/using bevy_blendy_cameras (I have an example and multiple apps using bevy_blendy_cameras), I had a lot of issues related to systems ordering. Some example app where functioning fine, other not at all saying egui wanted the focus all the time. This behavior was changing at each launch of the app/example even without recompilation or with recompilation of small code change unrelated to the camera. It took me a lot of time experimenting with systems orders and debugging and I am still not sure it is correct, but the only way to get consistent behavior was to put the check_egui_wants_focus system in PreUpdate after EguiSet::BeginFrame.

Interesting. I'm guessing you used a lot of the same code? I wonder whether it's a shared problem, or something unique about your plugin. I'd expect to have bug reports about it if people were experiencing that with bevy_panorbit_camera. Have you replicated with this plugin?

Yes, I removed the smoothing. I also added a Fly camera controller, the possibility to switch between those, the possibility to set the viewpoint (top, bottom, left, right, front and read) and the possibility to frame the camera view around entities.
For the PR, I do not have time available this week, but I will try this week-end.
For the egui focus issue: Yes I used a lot of the same code. I did not replicated the issue with bevy_panorbit_camera but even with bevy_blendy_cameras it worked fine a lot of the time. It is only when I had multiple applications using it (some with the use of egui in the PostUpdate stage because of bevy-inspector-egui) that the issue began to appear. And even than, it was very sporadically. It was a real pain to find something that works consistently in all the apps and I am still not sure it is the correct solution.

I implemented my "solution" in PR #81
I tried to reproduce the other issue in bevy_panorbit_camera but did not managed to do it yet.

Trying out this plugin now and hitting this. I tried the bevy_egui feature as well. I'm not sure I understand this thread fully yet, but I'm curious if I'm doing anything wrong or if it's still broken.

We will need more informations:

You definitely need the bevy_egui feature enabled to prevent the camera from moving when interacting with the GUI. But without this feature enabled, this issue should not be present at all.
Are the camera controls working when starting the click in the viewport but stopping when the mouse cursor goes on top of an egui Area? Or are the camera controls simply, sometimes, not working at all even when the mouse cursor is still not on top of anything egui related? If this is the second case, this is a separate issue (the second issue I am talking about) but occasional, very rare, and difficult to replicate in "simple" games/application but can happen in much bigger/ CPU intensive code base). If this is the first case follow to the following question.
Have you updated bevy_panorbit_camera to version 0.19.2? If the Cargo.toml file does not contain the last pat (.2) than you need to make sure the Cargo.lock is using this last version (possibly with cargo update with some options if you only want to update this plugin and not all the dependencies). I think, I am new to Rust so do not quote/thrust me on this.

@nixpulvis Can you clarify which issue you are having? A video would be helpful.

I tried with both the bevy_egui feature enable and without it. Both experience the same behavior.
recording.webm

The video is working for me. This looks like the OP issue (which I mistakenly called "the other issue"). Where the check to see if egui wants the focus returns false even when it should be true.
Note that this behavior is normal and expected without the use of the bevy_egui feature.
With the feature enabled, it is quite rare and difficult to replicate. And the only times I replicated it, it was not consistent sometimes being present or not varying between launch of the executable (without code change/recompilation). Also it was with another camera controller crate (but with identical check/code to see if egui wants the focus).
Can you try using the code from this git repository (and not the crates.io version if it is the case) and modify the scheduling of the check_egui_wants_focus system to be in PreUpdate after EguiSet::BeginFrame to see if this fix/workaround works for you too? The changes are realy minimals. It would be possible to create a separate branch with those changes for you to check them but you would still need to use a git version of this code and not the one from crates.io.
diff --git a/src/lib.rs b/src/lib.rs
index eba6521..a23c82f 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -66,9 +66,9 @@ impl Plugin for PanOrbitCameraPlugin {
             app.init_resource::<EguiWantsFocus>()
                 .init_resource::<EguiFocusIncludesHover>()
                 .add_systems(
-                    PostUpdate,
+                    PreUpdate,
                     egui::check_egui_wants_focus
-                        .after(EguiSet::InitContexts)
+                        .after(EguiSet::BeginFrame)
                         .before(PanOrbitCameraSystemSet),
                 );
         }

As @Plonq  said, there is a big chance that this issue is not coming from to this plugin but either from egui itself or from bevy_egui. And there is also a bigger chance that my "fix" is not the correct way to fix it but just a lucky workaround that worked for my cases. (I never saw this bug since this change, but it was quite rare for me even before)

@thmxv unfortunately that patch on top of bdf1ccb doesn't change anything for me.
You mentioned not being able to reproduce. Perhaps looking at my code could shed some light? I'm still learning both Bevy and egui, so I'm not sure I'm doing everything correctly either, but still I think this should work. There's not much special going on here from my end. Anyway, here's where I'm using it in case it helps: https://github.com/nixpulvis/galos/tree/master/starmap
I'm also happy to try some more debugging for you.

This call to try_ctx_for_window_mut is always returning None for me. Seems wrong.
https://github.com/Plonq/bevy_panorbit_camera/blob/bdf1ccbe8c479313839fa316d412e580a9ff3489/src/egui.rs#L42

@nixpulvis, thanks, I will try your code to replicate the issue. On a quick glance I did not spotted anything that could cause it.
The call to try_ctx_for_window_mut always returning None is definitely not normal and most probably the cause of the issue but not similar to what I experienced the few times I (inconsistently) reproduced the same issue.

Oh wow, good call. I should have checked that. Let me resolve the issue and report back.

Boom, that was it! Thanks for catching that. It looks like I was running bevy_egui = v0.29.0 and this crate only supports up to v0.28.0. No big deal for me, I'll grab the update when it's available.

Glad a solution was found, and thanks @thmxv for helping

Turns out try_ctx_for_window_mut was removed in bevy_egui 0.29. I've just released v0.19.3 of this crate which updates bevy_egui.

This is a long issue and at least one issue has been solved, and the way this plugin handles egui right now feels pretty solid to me. Thus, I am closing this issue. Open a new one if you think there's still room for improvement here.


Repair JOIN Function SQL Statement Splicing Error

$db->table('test as t')->join('foo as f', 't.id', 'f.t_id')->where('t.status', 1)->getAll();
# Should Output: "SELECT * FROM test as t JOIN foo as f ON t.id=f.t_id WHERE t.status='1'"
# Actually Output: "SELECT * FROM test as t JOIN foo as f ON test as t.t.id=foo as f.f.t_id WHERE t.status='1'"

@imsole , You're right. It seems like a bug. I'll merge this PR but, need to examine in detail.
Thanks.


Upgrade css selector for gutter

Fixes #1

I want to apologize, it was not up to you to do this, I did not see that you were part of the linter team

No no, it's fine. Perfectly fine :smiley:

:smile:


DOCS-11107: fix "on error continue" section text

it is referencing "on error propagate", but it should be "on error continue"

thx!


Source code for http://socketo.me/demo?

Not an issue...
Is the source code running in http://socketo.me/demo available for people to look and learn?

Hey guys. Where is the js&css files?


Fix custom levels colormap dialog

This PR fixes 2 issues on the ColormapDialog which was forbid the user to custom the levels of the colormap when the image was often refreshed.
It was still possible to custom the values during the time of a single frame.
This PR:

Make sure the result of the histogram range edition is used (it was not the case)
Make sure the handle moved bu the user are to reset during the interaction

Closes #3148

If i could vote, I would be happy to backport this issue in a bug fix release.

I was thinking so too.
Maybe we wait a little bit to see if more bugs like this one are showing up.

I think we can wait until mid/end of august without problem.

This fixes the issue while dragging the markers, but no when trying to set the min or max in the LineEdit.
The latter remains impossible.
I don't get why the displayed min/max are updated when changing the dataset while the colormap is not in autoscale.

You are right. I did not try. I will take a look.
The widget have to know the available theoretical values, in order to abort wrong values for example. But it should just be filter on it's own.

Here is a fix.
The first commit is only a clean up. There was a lot of dead code.
The second filter the setValue update to do it only if it is needed.

Now it works fine when setting the bounds from the LineEdit.
It is still a bit weird when changing the bounds with the markers on the histogram since the values in the LineEdit are reset to their previous value at each update of the image.
CI seems to be unhappy.

Ill take a look. If there is too much things to change i don't think it is a good idea for a big fix release. In this case it is better to stay with a small glitch.

The tests was not use the API the right way.
And there was a filter inhibiting the change of the colormap when dragging. Now it's much better. The image color is updated during the interaction, which was not the case before.

While i was there i have removed an unused function.
It's fine to be reviewed.

Which is btw the smartest if i never saw.

LGTM, works smoothly, thanks!


Allow to change FROM clause using QueryBuilder

Description
As a user, I would like to have the ability to change/add FROM clause in which the entity's table select/update/delete will be executed.
Motivation
For example, implementing the seek pagination with the ability to navigate to the previous page, you have to reverse the ordering of the result set (ASC vs DESC) using the same predicate so that the results you want are to the right of the cursor:
SELECT *
FROM (
  SELECT *
  FROM users
  WHERE createdAt < ?
  ORDER BY createdAt DESC
  LIMIT 10
) AS u
ORDER BY createdAt ASC;

Currently, this is possible to do only using knex.fromRaw(), exposing the implementation details of QueryBuilder. What could be a issue from design perspective. As soon as the implementation of either QueryBuilder or knex is changed (e.g. mikro-orm starts using a different query builder from knex), the user will have to rework a significant part of his code.
Just by declaring a from() method, we are able to give the user a powerful mechanism to handle this without necessity to rely on knex:
const qb = orm.em.qb(User, 'user')
  .where({ createdAt: { $lt: new Date() } })
  .orderBy({ createdAt: 'DESC' })
  .limit(10);

const users1 = qb.from(qb.getKnexQuery())
  .orderBy({ createdAt: 'ASC' });

const users2 = qb.from(qb.clone())
  .orderBy({ createdAt: 'ASC' });

Possible solution
The new method(s) can be introduced to be used to change/add FROM clause, removing all previously set from-s:
export declare class QueryBuilder<T extends AnyEntity<T> = AnyEntity> {
    from<TFrom extends AnyEntity<TFrom> = T>(entityTarget: TFrom | Knex.QueryBuilder | QueryBuilder<TFrom>): QueryBuilder<TFrom>;
    addFrom<TFrom extends AnyEntity<TFrom> = T>(entityTarget: TFrom | Knex.QueryBuilder | QueryBuilder<TFrom>): QueryBuilder<TFrom>;
}

I hope you also plan to send some PRs, and not just a huge round of issues :]

@B4nan I hope so, but I can't do this without your help)


Support follow operation

Support the follow operation defined at https://www.w3.org/TR/activitypub/#follow-activity-outbox

Users can now follow FediUni users from other platforms such as Mastodon and Pixelfed. Following users on different platforms from FediUni seems inconsistent, following someone on Mastodon works while someone on Pixelfed fails.


Updated Bonfire and Exaltation

Bonfire visuals to replace benches and Exaltation error fix
New PR to include a bonfire placement fix

Just realized I didn't need to close and reopen PR 🤣


⚠ Bofrost.de has degraded performance

In 4712280, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1695 ms

Resolved: Bofrost.de performance has improved in 34d7756 after 20 minutes.


specifiy dependency on babel-cli in package.json

updateSchema.js appears to expect babel-node, but it is not specified in package.json. npm installing babel-cli makes this work for me.
>npm run update-schema

><EMAIL_ADDRESS>update-schema /home/brent/git/forks/relay-treasurehunt
> babel-node ./scripts/updateSchema.js

sh: 1: babel-node: not found

npm ERR! Linux 3.19.0-42-generic
npm ERR! argv "/home/brent/.nvm/versions/node/v4.1.1/bin/node" "/home/brent/.nvm/versions/node/v4.1.1/bin/npm" "run" "update-schema"
npm ERR! node v4.1.1
npm ERR! npm  v2.14.4
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>update-schema: `babel-node ./scripts/updateSchema.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>update-schema script 'babel-node ./scripts/updateSchema.js'.
npm ERR! This is most likely a problem with the relay-starter-kit package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     babel-node ./scripts/updateSchema.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls relay-starter-kit
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/brent/git/forks/relay-treasurehunt/npm-debug.log

Just noticed there are a couple other PRs open with the same fix. Will close this one.


Version update: Spring Boot to 2.1.4 and Spring Cloud Contract to 2.1.1

Thanks @jessbringlarsen Merged.


Leave space for trailing \0 after MAX_FILENAME

Detected by gcc 8.0.1 warnings
Signed-off-by: Bernhard Rosenkränzer<EMAIL_ADDRESS>

Ping?
Reviewed-by: Andreas Färber<EMAIL_ADDRESS>Also needed for openSUSE Tumbleweed with GCC 8.1.1.

I will take it to our git for the next drop. Cannot merge with the existing
release here.
Kosta
On Sun, Jun 10, 2018, 20:16 Andreas Färber<EMAIL_ADDRESS>wrote:

Ping?
Reviewed-by: Andreas Färber<EMAIL_ADDRESS>Also needed for openSUSE Tumbleweed with GCC 8.1.1.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MarvellEmbeddedProcessors/atf-marvell/pull/9#issuecomment-396065705,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIhiYj8dypTiBepJzsnZpPuEs0ewnwS9ks5t7VRRgaJpZM4TvVWx
.

added couple of similar fixes and pushed to internal ATF 1.5 branch for the next release. Will push it into ATF mainline once our SoC support sources are accepted there.

This fix is still missing in the new 18.06 branches - needed a slight rebase.


Passwordbox issue

Describe the bug
when any char is used as the passwordchar and it happens to your password contains this char, unless this char is in the first poistion of your password, or the control would not accept your input to this char.
To Reproduce

set passwordchar="*"
input "abc*"
you will find last char "*" can't be inputted
set passwordchar="_"
input "abc_"
you will find last char "_" can't be inputted

Expected behavior
any legal char should be accepted no matter which char passwordchar is.
Screenshots
No response
OS version
Win11
.NET version
.NET 8 and .NET 4.8
WPF-UI NuGet version
3.0.4
Additional context
NA

Don't know what you got, it's working for me.

 Don't know what you got, it's working for me.

I believe what they mean is to input at runtime, rather than assigning a value directly.

 Don't know what you got, it's working for me.

I believe what he mean is to input at runtime, rather than assigning a value directly.

I did that too and it worked just fine for me.

 Don't know what you got, it's working for me.

I believe what he mean is to input at runtime, rather than assigning a value directly.

I did that too and it worked just fine for me.

That's peculiar, because in my development environment, I can consistently reproduce this bug.

 Don't know what you got, it's working for me.

I believe what he mean is to input at runtime, rather than assigning a value directly.

I did that too and it worked just fine for me.

That's peculiar, because in my development environment, I can consistently reproduce this bug.

Can you take a video?

 Don't know what you got, it's working for me.

I believe what he mean is to input at runtime, rather than assigning a value directly.

I did that too and it worked just fine for me.

That's peculiar, because in my development environment, I can consistently reproduce this bug.

Can you take a video?

https://github.com/lepoco/wpfui/assets/69706040/dadd76e5-4d5f-4fd4-bc90-f993865777a7

 Don't know what you got, it's working for me.

I believe what he mean is to input at runtime, rather than assigning a value directly.

I did that too and it worked just fine for me.

That's peculiar, because in my development environment, I can consistently reproduce this bug.

Can you take a video?

2024-06-21.23-46-44.mp4

Seems working to me.

 Don't know what you got, it's working for me.

I believe what he mean is to input at runtime, rather than assigning a value directly.

I did that too and it worked just fine for me.

That's peculiar, because in my development environment, I can consistently reproduce this bug.

Can you take a video?

2024-06-21.23-46-44.mp4

Seems working to me.

That is indeed peculiar