Ensuring symmetric in Vxx, this is needed because numerial error - [merged]

In GitLab by @cmastalli on Apr 2, 2019, 00:09
Merges topic/vxx-symmetry-bug -> devel
Inspeting the memory warm-starting issue (see https://gepgitlab.laas.fr/memmo/memory-to-crocoddyl/issues/1) I realized that the reason the backward pass was failing. The main reason was having non-invertible $\mathbf{Q_{uu}}$ matrix. And this was produced because numerical errors were producing non-symmetric $ \mathbf{V_{xx}}$.
So, I applied a simple rule of thumb to ensure symmetric in this PR. With this changes, we're able to solve the warm-starting problem (with long double support duration) in 6 iterations. This is the log file for that problem:
iter 	 cost 	      stop 	    grad 	  xreg 	      ureg 	 step 	 feas
   0  3.06504e+01  2.70110e-02  3.53678e+01  1.00000e-09  1.00000e-09   0.5000     0
   1  2.83929e+01  2.12409e-01  1.47910e+01  1.00000e-09  1.00000e-09   0.5000     0
   2  2.79667e+01  9.69123e-02  5.96420e+00  1.00000e-09  1.00000e-09   1.0000     1
   3  2.74287e+01  1.00807e-01  1.14229e+00  1.00000e-09  1.00000e-09   1.0000     1
   4  2.73820e+01  1.64318e-02  9.27254e-02  1.00000e-09  1.00000e-09   1.0000     1
   5  2.73818e+01  1.58312e-06  2.29567e-04  1.00000e-09  1.00000e-09   1.0000     1
   6  2.73818e+01  2.94175e-08  1.02976e-05  1.00000e-09  1.00000e-09   1.0000     1

In GitLab by @cmastalli on Apr 2, 2019, 00:18
changed the description

In GitLab by @nmansard on Apr 2, 2019, 09:40
Thanks for this nice bug fix.
For the track report: we have K=L'\L\Qux, and Vxx=Qxx - Qux'K. So the right solution is to store TMP=L\Qux and to make Vxx=Lxx-TMP'TMP.
I don't see an easy solution to implement that with numpy, since we have the chol solver that does not allow it. Let's keep it in mind for the c++ version.
@proyan could you double check that Vxx is symmetric by construction in your boxddp, and possibly propagate the solution of @cmastalli there?

In GitLab by @nmansard on Apr 2, 2019, 09:40
mentioned in commit ce6f21119b44720e3345c0dd389cb74fda58a30f

In GitLab by @nmansard on Apr 2, 2019, 09:40
merged

In GitLab by @proyan on Apr 2, 2019, 09:48
Vxx is not re-implemented in boxddp. Only the computation of the gains is changed. So any change in SolverDDP would be inherited by boxddp.
I'll rebase on the devel and see if there are any conflicts. Otherwise, everything should be fine.

In GitLab by @cmastalli on Apr 2, 2019, 09:51
I created an issue regarding https://gepgitlab.laas.fr/loco-3d/crocoddyl/merge_requests/153#note_4221. This issue belowes to a new milestone, called https://gepgitlab.laas.fr/loco-3d/crocoddyl/milestones/11.


Revert "python37Packages.aiorpcx: 0.17.0 -> 0.18.3"

This reverts commit 299996bc38da10452680ce1d1e18ab5db07387b7.
Motivation for this change
electrum no longer builds as it has not yet released an update that supports version 0.18, and is the only package that depends on aiorpcx.
Things done

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

[ ] NixOS
[ ] macOS
[x] 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)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

We could consider backporting https://github.com/spesmilo/electrum/commit/41f160dd74dc60434acac882393374cb15a381f0 until electrum gets a new release.


d3-simple-slider not working on IE 11

Hi there,
I am using d3-simple-slider in one of my Vue.js project. It works perfectly fine on Chrome and Firefox but giving following error on IE 11:

Thanks!

Thanks for reporting this. I should get a chance to reproduce this next week.
From the screenshot it looks like a syntax error so possibly I'm using something which is unsupported by IE11 and is not being transpiled/poly-filled correctly.

Sorry for the duplicate. More info here: https://github.com/johnwalley/d3-simple-slider/issues/43


Simplified toolbar option

Instead of mandating the full toolbar option be created like so:
var simplemde = new SimpleMDE({
    toolbar: [{
            name: "bold",
            action: toggleBold,
            className: "fa fa-bold",
            title: "Bold (Ctrl+B)",
        },
        "|", // Separator
        ...
    ],
});

Allow a simplified version that uses just the names available in the chosen order:
var simplemde = new SimpleMDE({
    toolbar: ["bold", "italic", "|", "code", "quote"],
});

Then use the default actions, class names, titles, etc. for the chosen icons in the chosen order.

:+1:

Completed with ff48a87.


Fix various errors regarding the RBACProvider flow

This PR corrects a few issues found during testing of the RBAC execution flow.
closes #36

execution with rbac based on the available examples and the docker setup in the project.
 java -jar target/kafka-topology-builder-jar-with-dependencies.jar  --brokers localhost:9092 --clientConfig example/topology-builder-rbac.properties --topology example/descriptor-with-rbac.yaml
log4j:WARN No appenders could be found for logger (org.apache.kafka.clients.admin.AdminClientConfig).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
List of Topics:
_confluent-license
team.source.foo.bar.avro
_confluent-metadata-auth
confluent-audit-log-events
team.source.foo.foo
_confluent-metrics
team.source.bar.bar.avro
__confluent.support.metrics
List of ACLs:
[WARN ] 2020-06-28 09:22:02.787 [main] RBACProvider - Not implemented yet!
[WARN ] 2020-06-28 09:22:02.787 [main] RBACProvider - Not implemented yet!
Kafka Topology updated


feat: export OHOS_NDK_HOME to env by default

This can help us optimize the user experience, ohrs and some community adaptations use OHOS_NDK_HOME by default.

Is there a reason ohrs and other apps use OHOS_NDK_HOME instead of the variables hvigor expects or sets?

In fact, there is not. When I was doing related adaptation work in the early days, I did not find any documents related to the environment variables output by hvigor (and I don’t seem to see any now), so I referred to some community naming schemes of Android and used them. OHOS_NDK_HOME is used as an environment variable.

Should OHOS_NDK_HOME point to the parent directory of native?

Rebased and merged in #12

Should OHOS_NDK_HOME point to the parent directory of native?

I think you are right and how about the following variable definition?
OHOS_HOME:
the root directory of the sdk, the tree should be like:
./
├── ets
├── js
├── native
├── previewer
└── toolchains

OHOS_NDK_HOME:
the sub-folder of OHOS_HOME named native.
./
├── NOTICE.txt
├── build
├── build-tools
├── compatible_config.json
├── llvm
├── nativeapi_syscap_config.json
├── ndk_system_capability.json
├── oh-uni-package.json
└── sysroot

If you agree, I will gradually replace it with the above definition in the future. @jschwe

For the long-term I would suggest migrating to OHOS_SDK_NATIVE instead of OHOS_NDK_HOME, since that environment variable will be set by hvigor and is hence somewhat official. (I don't remember if it was merged in time for hvigor 5, or if it will need to wait for hvigor 5.1).
For the parent directory of native it seems there is no official variable name yet. Perhaps we should keep in mind that when developing for OpenHarmony (not HarmonyOS NEXT), DevEco Studio / hvigor require the parent directory to be the API level.
This may be useful for users that wish to compile the same application multiple times for different minimum API levels, just by switching the --product flag of hvigor.
./${OHOS_BASE_SDK_DIR}
|── 11
|── 12
  ├── ets
  ├── js
  ├── native
  ├── previewer
  └── toolchains

OK, i got it.
OHOS_BASE_SDK_DIR: the root directory for SDK and the sub-folders should be api-version and so on.
OHOS_SDK_NATIVE: the currently using native folder with api-version.
there is another question that for newer DevEco Studio version it will provide more complex struct.
./${OHOS_BASE_SDK_DIR}
└── default
    ├── hms
    │   ├── ets
    │   ├── native
    │   ├── previewer
    │   └── toolchains
    ├── openharmony
    │   ├── ets
    │   ├── js
    │   ├── native
    │   ├── previewer
    │   └── toolchains
    └── sdk-pkg.json
└── HarmonyNext-Beta.2
    ├── hms
    │   ├── ets
    │   ├── native
    │   ├── previewer
    │   └── toolchains
    ├── openharmony
    │   ├── ets
    │   ├── js
    │   ├── native
    │   ├── previewer
    │   └── toolchains
    └── sdk-pkg.json

So i think we need to define a variable for the parent folder with native, it can help us to use toolchains simpler.

there is another question that for newer DevEco Studio version it will provide more complex struct.

Actually it is different depending on if you use DevEco Studio to compile for OpenHarmony or for HarmonyOS NEXT.
When targeting HarmonyOS NEXT, then the base dir is called DEVECO_SDK_HOME and has the children default etc, like you wrote.
When targeting OpenHarmony, e.g. for the dayu-200 rk board, then hvigor expects the directory structure I wrote in my comment above and OHOS_BASE_SDK_DIR must be set.
This means we have two different environment variable names (OHOS_BASE_SDK_DIR and DEVECO_SDK_HOME), which hvigor reads depending on if the user is targeting OpenHarmony or HarmonyOS.


:seedling: scdiff: Add workflow to run scdiff against PRs on demand

What kind of change does this PR introduce?
scdiff workflow

[X] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
Running scdiff is manual
What is the new behavior (if this is a feature change)?**
Commenting /scdiff generate <checks to run> will start a workflow run which evaluates the changes introduced in the PR against the checks specified.
A link to the workflow run will be added as a comment, and the run will fail if any differences are detected in the output.
You can see an example of this in action in my fork: https://github.com/spencerschrock/scorecard/pull/185#issuecomment-1791256485.
The PR simulates a scoring change that we want to evaluate with scdiff. In this case it's a typo in the Maintained check which flips one of the scoring decisions.
diff --git a/checks/evaluation/maintained.go b/checks/evaluation/maintained.go
index a4fd2bfc..1799daf5 100644
--- a/checks/evaluation/maintained.go
+++ b/checks/evaluation/maintained.go
@@ -36,7 +36,7 @@ func Maintained(name string, dl checker.DetailLogger, r *checker.MaintainedData)
                return checker.CreateRuntimeErrorResult(name, e)
        }

-       if r.ArchivedStatus.Status {
+       if !r.ArchivedStatus.Status {
                return checker.CreateMinScoreResult(name, "repo is marked as archived")
        }

My message: /scdiff generate Maintained triggers a comparison of the scoring changes from this change. And links to the workflow run here. We can see the golden-test job failed, and prints the diff.
Here is some of the output below
  pkg.ScorecardResult{
  	Repo:      {Name: "github.com/ossf/scorecard"},
  	Date:      s"0001-01-01 00:00:00 +0000 UTC",
  	Scorecard: {},
  	Checks: []checker.CheckResult{
  		{
  			Name:    "Maintained",
  			Version: 0,
  			Error:   nil,
- 			Score:   10,
+ 			Score:   0,
- 			Reason:  "30 commit(s) out of 30 and 0 issue activity out of 30 found in the last 90 days -- score normalized to 10",
+ 			Reason:  "repo is marked as archived",
  			Details: {},
  			Rules:   nil,
  		},
  	},

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Fixes #2462

Special notes for your reviewer

The repos use about 2200 api quota for a full run of all checks, which is about half of the hourly quota for the PAT which this uses. This number is dramatically lower if individual checks are specified (checks like Contributors are expensive). For this reason, I limit who can trigger the tool to project members.

the 15 repos used were picked based on various criteria to try and get a wide selection of repos:

Repo
Reason

https://github.com/airbnb/lottie-web
JS library

https://github.com/apache/tomcat
Java, Large project, middle score

https://github.com/Azure/azure-functions-dotnet-worker
C#

https://github.com/cncf/xds
Starlark, non GitHub workflow repo

https://github.com/google/go-cmp
Go, High Scoring, Small

https://github.com/google/highwayhash
C++, non github review system, no github cicd

https://github.com/googleapis/google-api-php-client
PHP

https://github.com/jacoco/jacoco
java library, github releases, non github CI

https://github.com/ossf/scorecard
Go, Uses GitHub Releases, our repo

https://github.com/pallets/jinja
Python, has a .github repo in org

https://github.com/polymer/polymer
JS, Low Scoring, Maintenance mode

https://github.com/rust-random/getrandom
RUST, small

https://github.com/yaml/libyaml
C

https://gitlab.com/baserow/baserow
GitLab, Python

https://gitlab.com/cryptsetup/cryptsetup
GitLab, C

Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

Temporarily closing this while I work on the author association filter, so I trigger the CI/CD here.

@raghavkaul it wont let me re-request since your first one was while it was a draft. But this is ready for another look


Fix bug in step! that prefents stepping past t_end

Purpose
This PR fixes step! so that it can be called arbitrarily many times.
Closes #73
Content
This adds a check for !isempty(tstops) to DiffEqBase.step! for our DistributedODEIntegrator, allowing the integrator to step past the final tstop.

bors r+


Train3

This PR refactors the code to use the new interface and is tested for text and N2V using SG and CBOW. I have not yet finished polishing the code. I was not able to use the hetgraph_tf version. Instead, I modified the hetgraph to return a raggedtensor. We will need to figure out if we are happy with this interface and theoretically we can continue to make things more efficient. It is looking though that on the GPU server, training is now relatively quick.

lots of tests not passing, looks like mainly the graph csf_graph and hetnode2vec code. Might be worth holding off on fixing these until we feel really good about the remainder of the tf changes. Is that your thinking on this too @pnrobinson ?

hmm
I was not able to get csf_graph_tf to work but perhaps we do not need this?
I think Vida fixed tests in hetnode2vec -- possibly in another branch.
Let's touch bases early this week!

hmm
I was not able to get csf_graph_tf to work but perhaps we do not need this?
I think Vida fixed tests in hetnode2vec -- possibly in another branch.
Let's touch bases early this week!

Sounds good to me. I will have some time mid-week that I can take a deep dive into this too.

hmm
I was not able to get csf_graph_tf to work but perhaps we do not need this?
I think Vida fixed tests in hetnode2vec -- possibly in another branch.
Let's touch bases early this week!

I fixed hetnode2vec and its corresponding tests in a separate branch which was merged to develop. I have some coursera homework assignments tonight, but I will work on these issues tomorrow.

I am going to merge this and then work on the tests in the keras branch.


updated blender to 2.77

Thank you for the contribution. It was merged directly as 815cd1bc4f18edcf7b81a513a99998e93c40530d to keep commit history cleaner. Your contribution is still credited to you.


Create an explicit type for Contexts

Sometimes you want to interact with a React Context in an abstract way, such as an implementation for a higher order component that spreads a context consumer into a component.  The way that createContext is currently typed makes this a little more difficult than it probably needs to be, so I created a type called React$Context<T> and exported the type from the react package as well which seems to be the standard approach for types like this.

This seems reasonable. I'll get it merged...


Plex Dashboard Shows Episode Playing Twice

FFMPEG is set to transcode with H264_nvenc and up until tonight it hasn't been working. I didn't make any changes at all, just started DTV and it worked, but when I checked to see if it was Transcoding in plex I found this. Is this supposed to happen?

Nevermind, I get it.


Connector plugins are not working with Tableau Server

I am following the same steps as given in the document to achieve connector in tableau server, but i'm getting below error. Please suggest
C:\Users\Administrator>tsm configuration set -k native_api.connect_plugins_path -v C:/tableau_connectors
Logging in as user 'Workstation\Administrator' since no --username flag was given.
Password:
Configuration error: At least one configuration value you specified does not match a known configuration key. This applies to the following keys: '[native_api.connect_plugins_path]'
Use this parameter to override unknown key error: --force-keys

Are you using 2019.1 Beta 2?

Yes, Tableau Server 2019.1 Beat 2.

Any update for me on the above issue?

Thank you for your patience. Right now, native_api.connect_plugins_path is not listed as a "known configuration", but it does work. Adding --force-keys to the end of the command will bypass the check and add the configuration key as intended:
tsm configuration set -k native_api.connect_plugins_path -v C:/tableau_connectors --force-keys
Once the changes are applied and the server is restarted, any valid plugins in the folder specified should appear in the connectors list.
Let me know if the plugins still do not appear after using --force-keys.

Thanks for your help & It worked after using the --force-keys. Now i'm able to publish the connector workbooks.
But I can't able to see my connector on Tableau server list of connector & also i can't able to see some of my peers, Can you please help me how can i see my connector on Tableau Server connectors list.


CHEBI:140503 undefined mixture with formula

CHEBI:140503 defines kaoline as mixture of the mineral kaolinite and containing varying amounts of other minerals such as muscovite, quartz, feldspar, and anatase. However, the entry has a structure which should be probably associated with one of the ingredients, not the mixture. Do you agree?

This is a bit tricky. Although in nature, kaolin is a mixture containing various other minerals, the main component is kaolinite, and quite a few journal articles show the same structure depicted in CHEBI:140503. We have only displayed the structure for visualization purposes. if you want i can remove it so its no longer visible.

No need for change if your policy regarding mixtures allows a structure for the main component. I'll just ignore such cases then.

Usually we don't show structures for mixtures but sometimes we do make exceptions.


cleanup: fixes cleanup images script.

Description

Remove update-transaction deletion as this should be part of clean-devices.
Fixes delete directory by removing the url path separator from the beginning.
Set only an amount of 10 images to handle for testing purpose, if everything OK revert back.
Set the scheduler to one hour from now for testing.

FIXES: https://issues.redhat.com/browse/THEEDGE-3449
Type of change
What is it?

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[X] Tests update
[ ] Refactor

Checklist:

[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I run make pre-commit to check fmt/vet/lint/test-no-fdo

Codecov Report
Patch coverage: 36.36% and project coverage change: +0.01 :tada:

Comparison is base (8f4186e) 60.47% compared to head (78a9da9) 60.49%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2123      +/-   ##
==========================================
+ Coverage   60.47%   60.49%   +0.01%
==========================================
  Files          75       75
  Lines       11323    11324       +1
==========================================
+ Hits         6848     6850       +2
+ Misses       3880     3879       -1
  Partials      595      595

Impacted Files
Coverage Δ

cmd/cleanup/cleanupimages/cleanupimages.go
53.71% <36.36%> (+0.60%)
:arrow_up:

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


Federate JSON config parameters have mixed style

Looking at ....examples/example_files/example_value_fed.json (among many other places), the JSON config. file has mixed camelCase and snake_case and shouldn't:
"only_update_on_change":false,
"only_transmit_on_change":false,
"source_only":false,
...
"timeDelta":0.0,
"outputDelay":0,
"inputDelay":0,
...

Thanks for pointing this out @trevorhardy . @phlptp this is coding style issue rather than anything else. I personally don't like mixed variable nameing styles especially on class members. So far the only culprit I can find is the CoreFederateInfo class. Fixing this would be a breaking change but since we've have incompatibility between 2.0 and 1.x anyways I'd say this is worth fixing. Opinions?

Strike that. I don't see the mixed case in 2.0 branch for CoreFederateInfo so that's good. It appears that we are supposed to be using CamelCase for everything however I do see only_update_on_change and only_transmit_on_change in the example_message_fed.json configuration file but I can't find which src file has those properties.

The general formatting rule is that variables and member variables are camelCase.  snake_case is used for constants and flags to distinguish them.  Classes are CamelCase, with one exception those are that typedefs or wrappers for fundamental types  (void *),  int32_t, enumerations.    I am not entirely sure this is perfectly followed everything in the code base but when I see something not matching I usually fix it if possible.    That being said there are probably some inconsistencies that didn't get fixed in 2.0 yet for the configuration files.   I will take a look at that at some point soon.


Bootstrap dropwdown issue

Hello there,
I'm using Bootstrap dropdown menu like this:

			        <button class="btn btn-default dropdown-toggle dropdown-toggle-off" type="button" id="dpMenu1" data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
						{{ currentProject }}

				</button>

						{{p.name}}

But when I'm including ng2-datetime-picker to a input field my bootstrap dropdown doesn't work anymore: When I click the dropdown button nothing happens. But ng2-datetime-picker is working fine.
Any help for this case? Thanks in advance!

datetime-picker is also an overlay(works like dropdown). Two different directive compete each other? I guess it should not be used together?
However, it's just a guess. Without a working example to reproduce the issue, it's hard to see the issue.


Subscribing to ChannelCreated/Removed in a queued listener leads to "Target [Laravel\Reverb\Protocols\Pusher\Contracts\ChannelConnectionManager] is not instantiable"

Reverb Version
1.4.0
Laravel Version
11.26.0
PHP Version
8.3.0
Description
Hi
I am trying to listen to the ChannelCreated and removed events.
I am subscribing to them in my event listener
public function subscribe(Dispatcher $events): array
    {
        return [
            ChannelCreated::class => 'onWebSocketChannelCreated',
            ChannelRemoved::class => 'onWebSocketChannelRemoved',
        ];
    }

The event listener is queable
class UserEventSubscriber implements ShouldQueue
{

In the functions themselves I do nothing at the moment, only debugging.
If I run it in sync, no issues, all work fine (locally).
But as soon as I deploy it (in this case a Forge server with Queue and everything running) it throws the following error
[2024-10-17 08:40:32] development.ERROR: Target [Laravel\Reverb\Protocols\Pusher\Contracts\ChannelConnectionManager] is not instantiable while building [Illuminate\Queue\Console\WorkCommand]. {"exception":"[object] (Illuminate\\Contracts\\Container\\BindingResolutionException(code: 0): Target [Laravel\\Reverb\\Protocols\\Pusher\\Contracts\\ChannelConnectionManager] is not instantiable while building [Illuminate\\Queue\\Console\\WorkCommand]. at /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Container.php:1195)
[stacktrace]
#0 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Container.php(947): Illuminate\\Container\\Container->notInstantiable()
#1 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Container.php(815): Illuminate\\Container\\Container->build()
#2 /home/forge/.../vendor/laravel/framework/src/Illuminate/Foundation/Application.php(1046): Illuminate\\Container\\Container->resolve()
#3 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Container.php(751): Illuminate\\Foundation\\Application->resolve()
#4 /home/forge/.../vendor/laravel/framework/src/Illuminate/Foundation/Application.php(1028): Illuminate\\Container\\Container->make()
#5 /home/forge/.../vendor/laravel/framework/src/Illuminate/Foundation/helpers.php(124): Illuminate\\Foundation\\Application->make()
#6 /home/forge/.../vendor/laravel/reverb/src/Protocols/Pusher/Concerns/SerializesChannels.php(27): app()
#7 [internal function]: Laravel\\Reverb\\Protocols\\Pusher\\Channels\\Channel->__unserialize()
#8 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/CallQueuedHandler.php(98): unserialize()
#9 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/CallQueuedHandler.php(249): Illuminate\\Queue\\CallQueuedHandler->getCommand()
#10 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Jobs/Job.php(234): Illuminate\\Queue\\CallQueuedHandler->failed()
#11 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Jobs/Job.php(213): Illuminate\\Queue\\Jobs\\Job->failed()
#12 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Worker.php(603): Illuminate\\Queue\\Jobs\\Job->fail()
#13 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Worker.php(549): Illuminate\\Queue\\Worker->failJob()
#14 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Worker.php(472): Illuminate\\Queue\\Worker->markJobAsFailedIfWillExceedMaxAttempts()
#15 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Worker.php(446): Illuminate\\Queue\\Worker->handleJobException()
#16 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Worker.php(390): Illuminate\\Queue\\Worker->process()
#17 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Worker.php(177): Illuminate\\Queue\\Worker->runJob()
#18 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Console/WorkCommand.php(139): Illuminate\\Queue\\Worker->daemon()
#19 /home/forge/.../vendor/laravel/framework/src/Illuminate/Queue/Console/WorkCommand.php(122): Illuminate\\Queue\\Console\\WorkCommand->runWorker()
#20 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(36): Illuminate\\Queue\\Console\\WorkCommand->handle()
#21 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Util.php(43): Illuminate\\Container\\BoundMethod::Illuminate\\Container\\{closure}()
#22 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(95): Illuminate\\Container\\Util::unwrapIfClosure()
#23 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(35): Illuminate\\Container\\BoundMethod::callBoundMethod()
#24 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Container.php(690): Illuminate\\Container\\BoundMethod::call()
#25 /home/forge/.../vendor/laravel/framework/src/Illuminate/Console/Command.php(213): Illuminate\\Container\\Container->call()
#26 /home/forge/.../vendor/symfony/console/Command/Command.php(279): Illuminate\\Console\\Command->execute()
#27 /home/forge/.../vendor/laravel/framework/src/Illuminate/Console/Command.php(182): Symfony\\Component\\Console\\Command\\Command->run()
#28 /home/forge/.../vendor/symfony/console/Application.php(1047): Illuminate\\Console\\Command->run()
#29 /home/forge/.../vendor/symfony/console/Application.php(316): Symfony\\Component\\Console\\Application->doRunCommand()
#30 /home/forge/.../vendor/symfony/console/Application.php(167): Symfony\\Component\\Console\\Application->doRun()
#31 /home/forge/.../vendor/laravel/framework/src/Illuminate/Foundation/Console/Kernel.php(197): Symfony\\Component\\Console\\Application->run()
#32 /home/forge/.../artisan(35): Illuminate\\Foundation\\Console\\Kernel->handle()
#33 {main}
"}

It seems this is an issue of ordering, as the event receives the Channel, which in its constructor tries to get the ChannelManager from app() leading to this issue.
Is there any workaround or fix in order potentially to get this to work?
Thanks a lot in advance
Steps To Reproduce

Create a queueable event listener
Subscribe to ChannelCreated
Handle queue via queue:work

Hey there,
While this may be a legitimate issue, can you first try posting your problem or question on one of the support channels below? If this issue can be definitively identified as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


Tasks and tool folder caching

Describe your feature request here
Hello,
We are currently running the Azure DevOps runners in our own infrastructure using Kubernetes.
Our runners are ephemeral, therefore, every time a job completes, the runner is killed and a new one starts.
To optimize the time it takes to run 1 job, we decided to implement some caching logic for the _tasks and _tool folders in the _work directory.
Our current strategy makes use of rsync for Linux and Robocopy for Windows in order to populate the 2 folders before starting the agent and at the end of the agent process to retrofit any modifications to our persistent storage.
In the ADO runners documentation, there is a section suggesting to cache the tools but nothing about the tasks even if it seems to be working based on our testing.
However, even if we can choose a shared folder for the tools, it seems that tasks such as UseDotNet aren’t handling concurrent write (currently it uses the cp command from ShellJS). Same issue happens with the download of the tasks when the job is in the init phase.
Would it be possible to add concurrent write support for both use cases to allow us to simply mount our caching volume rather than doing the rsync / robocopy at the beginning and at the end of the job?

Hi @taktakpeops! Thank you for your question.
We are currently working on higher-priority issues, but we will get back to this one as soon as we can.

Hello @aleksandrlevochkin - any update regarding this feature request?


get favicon from frontend-assets

Add a favicon, taken from frontend-assets.
This is related to PR#3 on frontend-assets.
This addresses task #8070.

Homerun.


Remove resource limitation for the operator pod

Which problem is this PR solving?

#2220

Short description of the changes

Remove the resource limitation in the operator

From the OpenShift guidelines:

Thx @iblancasa, can you update your branch? I can not, seems you restrict this :)


[v2] New Behavior

@jagi
I have created a new behavior based on your timestamp: https://github.com/settlin/meteor-astronomy-meta-behavior. It is more like a v2 of timestamp, but I created a new repo instead of a PR because of the unsuitability of the repo name.
If you agree, I would like to transfer the ownership to you and you can publish it on atmosphere.

No it's ok that you're the owner this repository. You can publish it under your name and if you want you can just add be as contributor. I will create new sections in Astronomy readme: community packages where I will list all packages worth using and also the Made with Astronomy section where I will show what apps where made using it.

@jagi added you as admin.
https://atmospherejs.com/settlin/astronomy-meta-behavior


Failed to pull image from quay.io

Running on a raspberrypi single master node cluster
Failed to pull image "quay.io/cilium/operator-generic:v1.11.3@sha256:5b81db7a32cb7e2d00bb3cf332277ec2b3be239d9e94a8d979915f4e6648c787": rpc error: code = NotFound desc = failed to pull and unpack image "quay.io/cilium/operator-generic@sha256:5b81db7a32cb7e2d00bb3cf332277ec2b3be239d9e94a8d979915f4e6648c787": no match for platform in manifest: not found
Environmental Info:
K3s Version:

k3s version v1.22.7+k3s1
Node(s) CPU architecture, OS, and Version:

NAME          STATUS   ROLES                  AGE   VERSION        INTERNAL-IP      EXTERNAL-IP   OS-IMAGE                           KERNEL-VERSION   CONTAINER-RUNTIME
raspberrypi   Ready    control-plane,master   94m   v1.22.7+k3s1              Raspbian GNU/Linux 11 (bullseye)   5.15.32-v7l+     containerd://1.5.9-k3s1
Cluster Configuration:

Describe the bug:

Steps To Reproduce:

Installed K3s:

Expected behavior:

Actual behavior:

Additional context / logs:

Backporting

[ ] Needs backporting to older releases

It appears that Cilium doesn't have images for arm/arm64. Is there something that needs to be fixed on the K3s side?

ah, yes you are right. I had doubts about whether its an issues with pulling images from the quay.


Single line TextInput onSubmitEditing doesn't work

Environment

react-native -v: 0.62.13
npm ls react-native-macos: 0.61.0-0
node -v: 12.18.0
npm -v: 6.14.4
yarn --version: 1.22.4
xcodebuild -version: 11E608c

Issue
TextInput in single-line mode doesn't trigger onSubmitEditing event when I press enter.
Steps to Reproduce

Hookup onSubmitEnter to a callback.
console.log a debug line in callback.
Type something, press enter

Expected Behavior
The callback should be called, producing console output.
Actual Behavior
The callback is not called - no message printed to console.
It does work, however, in multiline when I hit Cmd-Enter.

Thanks.   I think this was resolved in a PR for another issue. @tido64 : can you confirm?

Only onSubmitEditing in multi-line TextInput was fixed (#442). I can take a look at this one next.


fix(redis.js): support async redis calls context propagation

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Timezone issue

Hi, when someone sends a message to the bot, and the time [15:02] is Not correct to the original time. Like, it says [15:02] and it is 16:00.
The message looks like this: [15:50] Username: Message here

This is with threadTimestamps set to true, right? Those timestamps are always in UTC (i.e. +00:00), not localized to any specific region. There's currently no option to change this.


Implement signature verification (not rule-based validity of codes yet)

https://www.cc-d.bit.admin.ch/trust/v1/keys/updates\?certFormat\=ANDROID seems to be the dev environment URL to obtain the keys.
The prod version gives a 403 Forbidden, not sure what sort of authentication is required.
https://github.com/admin-ch/CovidCertificate-SDK-Android/commit/414887526a78969d1860e6d863b221cd4cc94f06 might be a good pointer for how to do verification.

Status update: cryptographic verification is now implemented, but providing the keys is up to the user.
Maybe we can also provide a helper to use the common key sources, though it’s not quite clear yet to me what those sources are.

The coronadecode tool now displays details about the signature and they are available programmatically. Calling this done :)


javadocs

@h1alexbel thank you for reporting this. I'll assign someone to take care of it soon.

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-06-04T09:52:27.895919.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@h1alexbel thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Api token page UX issues

Hello :)
This pr address UX issues on the Token API page discussed here:  https://issues.redhat.com/browse/AAH-826

The left nav and page title  changes to API token management.
Padding is added above Button Primary, Load Token (both standalone and insights mode).
In standalone mode, a white card background is added to the body content.

Before:

After:

There's one more API Token in tests..
test/cypress/integration/view-only.js
25:        'Collections > API Token',

Apart from that (and the potentially missing load-token from insights), LGTM :+1: :)


Retaliation in groups request

Would it be possible to assign a "group number" variable to a sentinel? So that if the particular sentinel was attacked, all the sentinels belonging to the same group number would attack the attacker.
Like an extension of the retaliation feature except instead of just the single NPC retaliating, the whole group(NPCs with the same group number) targets the entity that attacked one of them.
Please let me know if this is already possible in some other way.
Thank you for your amazing plugin.

So... basically, NPC ZombiePigmen! I can add that.

Added to dev build 63 :)


[dataio] Handle unlocalized camera and Maya keyframes with the Alembic animated camera

Since the animated camera has been introduced in OpenMVG for the localization part, some enhancements need to be done to improve its usage :

Change the keyframe system to an usable keyframe for Maya. Actually the animated camera has a Alembic node connected to the time, and it streams values to the camera according to the current scene time.

Handle situations when the camera is not localized. Actually if there is no localization success for an image, the localizer register the new camera position at origin (R = identity and t = [ 0 0 0 ]). A more convenience way is to not register new position (i.e new keyframe) and to let the camera at its previous position (i.e the last position that localizer found)

[ ] keyframe system

[x] unlocalized camera

From what I saw on internet,it is not possible to have keyframes from an Alembic importation. The only way we (maybe) have is a Maya script.

Yes, I also think that it is not possible...
Yes, we can do the maya script because in all case, we have to setup the image plane, etc.

OK.
Btw, where will I push maya scripts for image plane and so ?

finished?

Pull request waiting for merge : https://github.com/poparteu/openMVG/pull/90

keyframe system is implemented? can I close this then?

Yep


fix: Set scaleType programmatically for floating action buttons

Fixes #675
Screenshots for the change:

@liveHarshit Let the scaleType remain in XML so that when it is fixed. it is easy to revert

Also, create an issue to revert when it is fixed

Updated


feat(ws): add namespace dropdown to UI

Please note that, to unblock others, #167 needs to be merged before this PR.


Fix broken link to inexistant netlify github repo

Improve documentation
Link
Add a link to the page which needs improvement (if relevant)
Describe the problem
The mass rename renamed "gotrue" to "auth", including for the original netlify repo. netlify/auth doesn't exist
Describe the improvement
Point to netlify/gotrue
Additional context
Fixed by https://github.com/supabase/gotrue/pull/1491

Closing  as it seems fixed but let us know if there are any remaining concern


Additional Markup tags

First of all excellent work.
I was wondering if there is a way to modify or add more tags that support features such as:
-Clock
-Scrolling text
- Live RSS feeds
-Twitter, facebook. Instagram and youtube fragment for live updates
-Countdown Timer
-Weather Forecast for the day
-Calendar(I've seen this issue started somewhere)
If there is a way i can go about adding any of the above to slides, please let me know.

Anyone around?

Sure there's a way to do that. It'll require some programming though.
The markup tags are defined in src/node_modules/ls-markup/eval_defs.js. It's quite simple to implement new tags just by replicating the existing ones in that file. You'll have to write all the code that makes the markup tags work yourself though and that might take some time.
A calendar markup tag is indeed something I need to look into but I haven't had the time yet.

Thanks for the update. I really need those specific tags I've outlined. If you have time try and implement some. In the meantime let me see if i can sort one or two tags.

I will eventually work on them but it's unfortunately not my top priority right now. I have another big change I'm working on and I'm also rather busy right now.

This is of particular interest to me as well.  I'm looking at implementing this software in our small school, and the weather functionality is something that has been requested.  I'll take a look at the source, but I'm not sure there is too much I can do. NodeJS/NPM isn't my strong suit, and I don't have the time to dedicate to picking up another language right now!

@Qualitymix I'll get around to implementing it at some point but cleanly integrating something like this into LibreSignage is not really a trivial thing to do. I've also been really busy with other things recently so I haven't had much time to work on LibreSignage.

Taking a look at how the tags work, they are just implementing HTML code, which is awesome.  Really, if there could be a way to build a custom URL out of options passed through a tag, we could get a [weather] tag implemented very easily.  Using a site like https://weatherwidget.io/ would work great!

So I can compile a hardcoded version of what I'm looking for, however I run into an issue with what seems like the javascript not liking the

@Qualitymix  That's odd. What changes did you make to the markup transpiler code?

All I did was add a tag in the eval_defs.js file inside ls-markup.
weather: {    has: {},    handle: () => {       return [          '<div....  and so on.  This part is what was returned from using the willyweather.com widget creator' </div>,
``
];
}
},`
This code works with the willweather widgets.  Anythingn with the

That's odd. I need to look into this myself sometime.

How about [html] and [script] tags?
Then the user can add whatever they like.
I tried a workaround for Fontawesome:
[p][font f="Font Awesome 5 Brands"][/font][/p]
but it sanitizes the ampersand, so the hexcode will be shown

I think instead of implementing HTML and script tags it'd make more sense to implement a way to create slides entirely using HTML, CSS and JS scripts. Mixing HTML and scripts with the LibreSignage markup would probably introduce all kinds of hard to debug problems with slide layouts. I've thought of implementing HTML & CSS slide editing before and it's a feature that will probably be implemented in the future.

I have added a raw HTML tag that works okay.

@Qualitymix Yeah, it's certainly possible to add a HTML tag to the markup transpiler, but it's really a matter of whether it's actually a good idea to do that. Like I commented before, I'm not too keen on implementing a HTML tag in the official LibreSignage builds, but if it works for you then that's great. I need to look into implementing bare HTML & CSS slide editing in the future though.

I have added a raw HTML tag that works okay.

Good evening, could you share some hint on how you do that?


feat: Add Button [Part 1]

Description: adds "Create Button" button on signed-in page
Test Plan: open feat-add-button-1.paybutton.io and check if "Add Button" button is displayed

Deployment is not working again, investigating if it's a supertokens thing or netlify.

Fixed, should be working properly now.


Russian Localization

Pls add it. Thx for the mod

Thanks for the contribution! There's one small issue, though, the loca file is still a xml. You need to convert the xml file to a loca file using the LSLib
After that's fixed, would you mind if I mention you on the Readme page and on the Nexus Mods' page? If that's ok, what name would you want displayed, and do you have any page (GitHub profile, Nexus Mods profile or other) you want to be linked with your name?

Hi, you can refer to me as Aks_kun. Here is my profile on Nexus Mods: https://next.nexusmods.com/profile/Aks2aidos.


Any type of Rope Constraint no longer spawns.

You name it rope, elastic, slider, and any related constraint cannot even be spawned now because Adv Dupe 2 is having errors, here is a list of examples
Trying to spawn Rope:
[Advanced Duplicator 2] lua/includes/modules/constraint.lua:506: attempt to index local 'rope' (a nil value)

oldFunc - lua/includes/modules/constraint.lua:506
Rope - lua/advdupe2/sv_misc.lua:85
3. LeftClick - gamemodes/sandbox/entities/weapons/gmod_tool/stools/rope.lua:60
4. unknown - gamemodes/sandbox/entities/weapons/gmod_tool/shared.lua:214

Trying to spawn Slider:
[Advanced Duplicator 2] lua/includes/modules/constraint.lua:735: attempt to index local 'rope' (a nil value)

oldFunc - lua/includes/modules/constraint.lua:735
Slider - lua/advdupe2/sv_misc.lua:85
3. RightClick - gamemodes/sandbox/entities/weapons/gmod_tool/stools/slider.lua:138
4. unknown - gamemodes/sandbox/entities/weapons/gmod_tool/shared.lua:239

Trying to spawn Elastic:
[Advanced Duplicator 2] lua/includes/modules/constraint.lua:604: attempt to index local 'rope' (a nil value)

oldFunc - lua/includes/modules/constraint.lua:604
Elastic - lua/advdupe2/sv_misc.lua:85
3. LeftClick - gamemodes/sandbox/entities/weapons/gmod_tool/stools/elastic.lua:57
4. unknown - gamemodes/sandbox/entities/weapons/gmod_tool/shared.lua:214

Might be the new color update that was added to ropes.

Precision Alignment Ropes (or related) constraints still work, but it's just the base game ones that are having an issue, so probably is. :)

Why's your stacktrace contain 'oldFunc' at the SetColor positions? Seems strange.

I don't know why all I know is this started happening after the June updates to Gmod.

Works fine on my server. I think you have an addon that broke

Well thanks, sorry for the inconvenience, I'll have to figure it out.

You can use PrintTable(debug.getinfo(FindMetaTable("Entity").SetColor)) to figure out which addon is overriding the function (hence oldFunc)

Well the problem there is 'nil rope' so something is causing the keyframe_rope to not spawn as well.

https://github.com/Facepunch/garrysmod-issues/issues/4992


skinsrestorer integration doesn't work properly.

skinsrestorer integration doesn't work properly.

Can you be more specific? what about it isnt working properly? Does is error when loading? Does it show the wrong skin? What version of skinrestorer are you using (full version number)?

my server runs on "Folia version git-Folia-24 (MC: 1.20.4)" I am using the latest version of skinsrestorer(v15.0.7) with a database and Bluemap is also up to date. when someone joins the world skin doesn't show up on the map without any errors or something else but it says "[BlueMapSkinIntegration] Checking SkinsRestorer"

The latest SkinsRestorer is 15.0.15 ( https://www.spigotmc.org/resources/skinsrestorer.2124/ ), The latest BlueMap is 5.2 ( https://github.com/BlueMap-Minecraft/BlueMap/releases ). I have never tried Folia, but since it adds multithreading officially it might cause some strange race condition.
I tested today with the above mentioned versions using my current Paper force PURPUR which worked as expected. My current Mojang skin https://namemc.com/profile/Cerothen.1 was overridden by my skinsrestorer skin, as outlined in the check order for my config file.

If your version is 15.0.7 I would consider updating to the actual latest version and try again.

i am using skinsrestorer on the proxy mode which can be the cause of the problem.


Stuck in Connect() method

In 1.20.3 and 1.20.4, the Connect() method sometimes never resolves.
I think this is because of the PhysicsPlugin, which calls await WorldPlugin.WaitForChunks(), which never resolves because the client does not received any chunks.
My guess is that the bot messes up the ChunkBatchReceivedPacket packet with a ChunksPerTick of 0, so the server never sends any chunks again.
Edit: The bug also occurs in 1.20.1. The PlayerPlugin never finishes initialization.

ChatPlugin, WindowPlugin and PlayerPlugin had calls to Bot.Client.WaitForPacket<>() or Bot.Client.On<>() in the Init() method of the plugin. If the server sent these packets before Init() was called, they were probably lost, and the plugins could never initialize.
I moved these calls into the constructor, and haven't gotten the bug since, but it appeared quite irregularly anyways...

PlayerPlugin sometimes threw an exception in Init() due to a race condition.
The race condition is removed (a61736e), and other exceptions are now catched and logged.


Message payload description sounds like it constricts payload type

Describe the bug
We discussed this in the following meeting: https://www.youtube.com/watch?v=8Yjles2pEHo
There was a bit of confusion about the part of the description that said:

.... It must match the schema format, including encoding type - e.g Avro should be inlined as either a YAML or JSON object NOT a string to be parsed as YAML or JSON.

Some of the discussion was around does that mean that you are not allowed to inline JSON string values as payload definition if using a custom or non-JSON one.
As far as the conclusion of the discussion, this section was meant to be a guidance to explicitly state that Avro for example should be formatted as a regular JSON object and not as a string value. But should not be seen as overwriting the definition before hand which state it can be any type:

It can be of any type but defaults to Schema object.

still relevant


Kousa: block merge to staging if CI tests don't pass

code in staging might not pass tests; proposal is to use at least a minimal github actions that will run unit tests on the elixir code and prompt the IC to fix them before merging.

seems like this has been addressed


[FEATURE Clarify README's regarding frontend or backend instructions

Is your feature request related to a problem? Please describe.
Currently, there are two README's, one focused on the backend, and the other focused on the frontend
Describe the solution you'd like
Mention at the top of the initial README that first time users encounter, where to go if they would like to run the frontend or backend.
Additional context
This was based on just a cursory glance on the starting out instructions, assuming that first time users will be unfamiliar with the project structure and to anticipate any confusion/troubleshooting that may arise

I'm going to take a look at this tonight.

After chatting with Erika and Darrell, I made some very small changes to the root README, and the frontend README.  Made sure the two sets of instructions link to each other to help newcomers find their way through the setup more easily.  I'll make a PR for the small changes now.


Node pool names can be defined by user and don't get appended a random suffix

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Currently the additional node pools get assigned a random MD5 hash suffix. This behaviour isn't document in irrupting to the user in my opinion.
See https://github.com/Azure/terraform-azurerm-aks/blob/main/main.tf#L581
I'd prefer the names are check for uniqueness via TF and in case of conflicts get rejected.
New or Affected Resource(s)/Data Source(s)
variable "node_pools"
Potential Terraform Configuration
// Validation inside "node_pools" variable definition
validation {
  condition     = length(values(var.node_pools)[*].name) == length(distinct(values(var.node_pools)[*].name))
  error_message = "Err: each of the node pool names must be unique."
}

References
No response

@oteichmann the random suffix is needed during nodepool upgrades, to have a "create before destroy" lifecycle of the nodepools.
If I understand correctly your assumption was that the random suffix was needed to avoid multiple nodepools with the same name. As explained above this is not the goal, so the validation you propose does not meet the goals of the module.

@oteichmann thanks for opening this issue. As @zioproto explained, if we don't add this suffix, when users want to recreate the pool, they would have to delete the existing pool first, then create a new one since the pool's name could be the same, and that could lead to downtime, so we have no choice but adding this random suffix, so we can create a new pool before we destroy the old one (since now they have different names), and this procedure gives the cluster's operator a chance to evict all workloads from the old pool to the new one to avoid downtime.
Do you have any further questions or suggestions?

Thanks for the explanation @zioproto and @lonegunmanb . No further questions. This vital, operational arguments do of course outweigh my striving for order and beauty ;)
I might come back on this once the "azurerm_kubernetes_cluster_node_pool" resource does also support some rotation logic as it is already the case for the default pool.


Update default repo names for registration

Migrate databases with default values
https://bugzilla.redhat.com/show_bug.cgi?id=1138853

@gmcculloug @Fryguy Please review

@miq-bot add_label bug

@brandondunne To remove the warning there, just wrap the assignment in parens, or don't do the assignment.  I prefer the latter, but it's your choice.

@Fryguy updated

:+1: will merge when green

@brandondunne Had a few nitpicks above.  :tomato:

@kbrock @jrafanie Updated based on your comments

I love these database migration specs.  It's a little tricky to get started on them but they are so easy to verify your migration does the right thing :+1:


[HOLD for payment 2023-10-10] [HOLD for payment 2023-10-09] Room - App crashes when typing room name during room creation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch New Expensify app
Go to + > Send message > Room
Start typing room name

Expected Result:
App does not crash
Actual Result:
App crashes
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.75-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/a9f470ed-0a3f-4118-b670-905d08117f84
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Unable to repro this on DEV

@mountiny Issue reproduced  v1.3.75-3
https://github.com/Expensify/App/assets/93399543/1abf64da-003d-4c0e-99d3-a8fe5ea355bc

CPing the linked PR from Situ

fixed

@flaviadefaria I think only pending thing would be oto pay $250 to @situchan for their fix of the blocker

@situchan are you still paid in UW or do you request payment directly in ND?

UW please

Switching this to daily so that I can issue payment this week.

[x] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/27846
[x] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/27846#discussion_r1338536705
[x] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] Determine if we should create a regression test for this bug.
[x] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

This was caught by QA team so no need regression test

@flaviadefaria I think only pending thing would be oto pay $250 to @situchan for their fix of the blocker

Payment Summary:
@situchan = $250
Offer sent!

Paid!


Mobile-friendly-rendering

Hey Sean, I added a media query similar to the one you had added in the About.module.css. I was able to test this on my web browser using the Chrome DevTools simulator of various mobile devices. As an extra - I added in some minor tweaks to the tables - for better readability. I noticed in many cases when there's missing values it is squished and could be hard to tell which column is which.

Also made a slight change to the CSS for the ScanQR view, which you mentioned was a tad wider than the other tabs.

Thank you - sounds great! I'll push it out now. I'll ping the rest of the team to see if they can test on their devices as well.


Refactor SignatureModel.parse_values_from_connection_kwargs

This is a draft attempt to start refactoring process (#203, #191).
After reading through the code base, I decided to start with something small—function signature.
I have couple of questions about continuing the work:

If there's no unit tests, should I add them? Also, there's a lot of—what can be considered—integration tests. If I add an isolated test that duplicates what already has been tested in an integration test, should I remove previous one?
Is SignatureModel a public API?
Do you see the benifit of proposed changes?

I won't review this now, but -

Yes, add full unit tests to ensure code is not broken

Don't remove any existing test unless there is  compelling reason to do so.

SignatureModel is completely internal. As a rule if thumb, anything not in the docs is internal.

Was this method in the sonar report?

@peterschutt Just checked it up: no.

| @peterschutt Just checked it up: no.
All good, I went looking to see what the complexity rating was and couldn't see it.
Don't worry about breaking off one of the bigger changes if you are worried about it. The ones in the report are the low hanging fruit, so best value for our time!

Thanks @vrslev!


After saving in vscode with this extension enabled, recompile in IEx session does not work

After saving in vscode with this extension enabled, recompile in IEx session does not work, and the new code will not be loaded.  This forces you to quit IEx completely and start a new session.

Hi! Could you maybe disclose the exact reproduction steps, and what exactly happened, and what you would have expected to happen?

Sorry, I can't reproduce this anymore.  Either it's fixed, or there's some more nuance to when it happens.  If I learn anything more I'll come back and add more details; for now I'll close.


Add initial orchestrators idea

Im not sure if the general ideia fits in ophyd scope, but this is an initial attempt.
The idea is to have abstractions to perform tasks that are possibly frequent to the plugins and drivers, but not basic enough to be put in the initial plugin/driver abstractions. For example, a common method to configure a grid of peaks in SimDetector or to one to define how to average images in processPlugin.
I wanted to set a default behavior in the method but in a way that parameter default values could be overriden by an user who knows what they are doing.
Unfortunately, the only way i could come up to do this was by manually coding the properties as possible keys to kwargs (eg. line 22 in ProcessPluginOrchestrator). I tried dynamically checking if any argument in kwargs was equal to one of the component_names but for that i needed to use something like getattr(self.cam, component_name).put() and for some reason that didnt work.

Could these be better handled as plans?

@tacaswell yes, now that you mention i think its a purpose more suited for a bluesky plan. Probably the class should be created there, also?
Waiting for an answer about the NotImplementedError before i close the PR.

I don't think you need to create any classes.
for k, v in kwargs.items():
    yield from bps.set(getattr(obj), k), v, group='my_group')
yield from bps.wait(group='my_group')

(or yield from bps.mv(obj1, v1, obj2, v2, ...) but I can't write the flattening on-liner off the top of my head)


docs: update end-of-life messages

This change updates the messaging about AngularJS end-of-life in several places:

Docs homepage
Docs banner
Security and support pages
GitHub README

cc @aaronfrost
@gkalpak am I missing anything here? I also want to update the changelog, but assume that happens automatically as part of the release process?

@Splaktar I've changed the top banner message to be inside the same container as the version picker to avoid the overlapping and updated the various top styles to adjust, seems okay now on various viewport sizes.  I also made the container opaque because I wouldn't ignore the transparency 😑
I've also changed that "CDN" text to "Release archive".


Different windows for editor and results

Is your feature request related to a problem? Please describe.
I would like to have the editor and the results displayed in different windows. This would make it much easier to work with large queries and many result columns on multiple monitors.
Describe the solution you'd like
There should be a new menu entry near „Detach tab“ to move the results to a new window. Further executions of queries in the editor should re-use the new window to display results.
Describe alternatives you've considered
The window could be stretched across multiple monitors, but this is cumbersome (especially when working at multiple locations with different monitor setups) and it’s almost impossible to utilize the full area of multiple screens with different resolutions and/or alignments (vertical/horizontal).

Please take a look at this feature. It sounds like what you want https://github.com/dbeaver/dbeaver/wiki/SQL-Editor#multiple-results-in-one-tab

However, currently it's not possible to detach this view.

My desired screen layout looks like this:

Just a single result list, but on a different screen than the editor.

Got it. Thank you :)

We wonder how many users seek this feature.
So, for someone who is interested, please vote for the issue with  👍  under the issue description to promote it to a higher priority.


changed beam decoder stopping condition

stopping condition of the beam decoder now works as follows:
-- case return only the first beam (stop_early=True): it will stop when the first beam is 'unbeatable' by the other alive beams. (basically the previous behavior)
-- case return all beams (stop_early=False): it will stop when all the first N beams are 'unbeatable' by the other alive beams.
Note that checking that the last beam (in the top N) is unbeatable implies that all the top N beams are unbeatable as well.
Note w.r.t. the test changes: I duplicated testNotGreedyBeamTwo so that there is now a version with stop_early=True and one with stop_early=False. The first checks only the first beam, the latter will check all the beams.
Other two tests changed as well: they were checking the beam output length (checking it was equal to INPUT_LENGTH + decode_length) - disabled these checks given that this is not true anymore.

Speeds things up considerably for our team. Great job!
Can this be merged soon so that we can continue using the master branch version?

happy to help @KasparPeterson
again, it is my first MR, not sure how to proceed to get it accepted.
@rsepassi , any procedure that should be followed?

Taking a look, sorry didn't spot this sooner - @mirkobronzi @KasparPeterson

Many thanks @mirkobronzi -- this is awesome!

great! thanks @afrozenator :)

Thanks for taking care of this issue! Is this currently working? I install from the codes and tried the decoding with returning multiple beams, but it still takes long time. Is there anything I missed? Do I need to set any parameters? Thanks!

@shakier : no, it should now use the new code when you ask for multiple beams in the results.


HSEARCH-3048 Search 6 groundwork - Add the missing common field type options compared to Search 5

https://hibernate.atlassian.net/browse/HSEARCH-3048

norms
searchable
doc_values ( test only, it seems it had already been implemented well )
termVector
precision_step not exists anymore (from ES 5+)

options added => preview. I think that all missing attributes have been handled now

Pull Request Test Coverage Report for Build 2

227 of 235 (96.6%)  changed or added relevant lines in 82 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 89.073%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/types/dsl/impl/AbstractElasticsearchStandardIndexFieldTypeContext.java
3
4
75.0%

backend/lucene/src/main/java/org/hibernate/search/backend/lucene/types/dsl/impl/AbstractLuceneStandardIndexFieldTypeContext.java
7
8
87.5%

backend/lucene/src/main/java/org/hibernate/search/backend/lucene/types/codec/impl/LuceneStringFieldCodec.java
6
8
75.0%

backend/lucene/src/main/java/org/hibernate/search/backend/lucene/types/dsl/impl/LuceneStringIndexFieldTypeContext.java
9
13
69.23%

Totals

Change from base Build 262:
0.05%

Covered Lines:
18243

Relevant Lines:
20481

💛  - Coveralls

@yrodiere thanks for the review. I applied the changes here.
I'm going to add further options  in the subsequent commits.

@yrodiere thanks for the review, anything else to improve here?

Merged, thanks

thank you


Making a request to the blockcypher client

Upon executing the blockchain example code with both a free and paid plan from blockypher and a token these following error messages pop up.

The first issue is familiar. This is not an issue with the code but you ran into the rate limiting of blockcypher.  Check your blockcypher dashboard, there you'll see how many requests you have done.
For the second screenshot I really need to know what code was executed. Can you provide that?

according to blockcypher i only made a tiny amount of requests

also for second screenshot is just this

both issues came from this piece


ERROR: Could not find a version that satisfies the requirement XbitOps (from versions: none)

I met this problem when  I ran this code below. How to solve it?
pip install git+https://github.com/wejoncy/QLLM.git

Hi, Thanks for your reporting.
Could you work around it by pip install git+https://github.com/wejoncy/QLLM.git@custom ?
Will fix it in few days.


python3Packages.lark: 1.0.0 -> 1.1.2

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I left it as draft b/c I found some downstream build errors, and was trying to fix those. Haven't run a full nix-review.

for the kaldi packages, we should really upsteam some options to not follow their VCS-based dependency management.

for the kaldi packages, we should really upsteam some options to not follow their VCS-based dependency management.

I don't have the time or interest to do that currently. Pinging maintainer @ckiee

No changes, just squashed commits.

I have been summoned! FYI there's already someone who was working on fixing the build but didn't open a PR. (I'd ping them but I don't know their GH handle)

for the kaldi packages, we should really upstream some options to not follow their VCS-based dependency management.

I suffered so much while hacking that mess (sorry daanzu!) into working order. I do not want to touch their mess of a build system ever again.
..but it'd be nice.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 162429 run on x86_64-linux 1

37 packages built:

ansible (python39Packages.ansible-core)
ansible_2_11
charliecloud
home-assistant-component-tests.homekit_controller
pdm
python310Packages.aiohomekit
python310Packages.ansible-core
python310Packages.cassandra-driver
python310Packages.cirq
python310Packages.cirq-rigetti
python310Packages.commentjson
python310Packages.dragonfly
python310Packages.flake8-bugbear
python310Packages.gdtoolkit
python310Packages.gremlinpython
python310Packages.hypothesmith
python310Packages.kaldi-active-grammar
python310Packages.lark
python310Packages.pyquil
python310Packages.radish-bdd
python310Packages.resolvelib
python39Packages.aiohomekit
python39Packages.cassandra-driver
python39Packages.cirq
python39Packages.cirq-rigetti
python39Packages.commentjson
python39Packages.dragonfly
python39Packages.flake8-bugbear
python39Packages.gdtoolkit
python39Packages.gremlinpython
python39Packages.hypothesmith
python39Packages.kaldi-active-grammar
python39Packages.lark
python39Packages.pyquil
python39Packages.radish-bdd
python39Packages.resolvelib
terraform-compliance


Avy size

Bigger avy size, at least 250x250 x 1000kb, this has probably been suggested before but I couldnt find the issue.

Good enough ^.^


$email is not defined in script

$email is not defined in script

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

ID: 8984951b-7ece-4964-ed6e-5266174d2ffb
Version Independent ID: 163b743d-fd7f-9c50-f01c-79a4521c5236
Content: Tutorial for bulk inviting B2B collaboration users - Azure Active Directory
Content Source: articles/active-directory/b2b/bulk-invite-powershell.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@Stuzest Thank you for the feedback. We are actively investigating and will get back to you soon.

@msmimart Could this e checked and rectified ?

I was able to fix this by changing InvitedUserEmailAddress on the template.

Change InvitedUserEmailAddress to Email on the template.

Thanks for bringing this up: "$email" is the iterator variable and it gets defined automatically by Powershell when the foreach loop runs. I just tried the sample and it ran successfully--for now I'll close this issue, but please comment if it's not working.
#please-close


Aggregating Data in Query Objects

Min
The Min method retrieves the lowest value of fields in the column within a group. To set up a Min method on the Quantity column of the sample query, set the Method property to Min. The name of the Quantity column automatically changes to Min_Quantity and the query is automatically grouped by the No. and Name column
How do i reference an aggregate column in a query. I want to use it as the condition in my join, however it uses the value of the original column instead of the aggregated column.
E.g  column(Amount;Amount)
{
Method = Min;
}
         dataitem(SalesLineArchive; "Sales Line Archive")
         {

DataItemLink = "Amount"=SalesLineArchive.Amount;
}}
uses the value of all the amount in the column instead of just the maximum value. In the article it says the name of the column changes to Min_Quantity but this is not the case.
[Enter feedback here]

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

ID: b2cd8989-54cc-e96c-c12b-10d5ce791a3e
Version Independent ID: 54120d28-3288-13bd-bfca-01f3b3021f83
Content: Query Totals and Grouping - Business Central
Content Source: dev-itpro/developer/devenv-query-totals-grouping.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: solsen

uses the value of all the amount in the column instead of just the maximum value. In the article it says the name of the column changes to Min_Quantity but this is not the case.

It means that the name of the query field changes to that, but data item links/references always use the original field name from the table itself; they refer to the 'real' field, not any version of it declared by the query.


Upgrade to base-4.11

base-4.10 has imported a lot of typeclasses into the standard library (Semigroup, MonadIO, etc) which are very useful. We should upgrade to it.

Whats your opinion on fixing this and #657 together?

This upgrade would imply that the default prelude would be the one included with 4.10 (with foldable-traversable and other changes), no?
If so, it would be somewhat related with https://github.com/typelead/eta-proposals/pull/4

Yep, definitely. :smile:

@rahulmutt what do you think what would be the best approach? i tried to get base-4.10 and build with etlas/eta and i has a lot of features of ghc > 8 (type in type, type parameters, ..). Maybe we should do a list of desired changes and try to backport them to the actual version, but it seems it will suppose a lot of work...

I think instead, we should be discussing which changes we will not be able to port and what we should do about it. In most cases we'll probably just have to avoid porting specific changes but in some cases I think they are reasonable alternatives we can provide.

I'm trying to adapt base-4.11.0 to eta in this repo: https://github.com/jneira/packages-base.

thank to the amazing work of @Jyothsnasrinivas i think that one is fulfilled

Indeed it is! Closing.


Test: Circumvented a pip-check-reqs issue by excluding its version 2.5.0

No review needed.

coverage: 25.0%. remained the same when pulling 8fd4ead179dab17935678fe9972d7721af92ae13 on andy/fix-pip-check-reqs-issue into 8aab12001dc183c5190b069ab589c11ea9d9c933 on master.


Advanced search

The formatting of the advanced search seems broke.

Thanks. I don't even know how to get here, where is this?

here it is: https://twitter.com/search-advanced (i'll see if i can fix it when i fork @sdras!)

Oh wow, yeah, that's really jacked haha. I've never seen that page before. I would appreciate it @rachelnicole! But no pressure either :)


[Feature] Pushble and Pullable Checkbox

Hi.
Can you add a checkbox to define is the token is pullable? Tks

Fixed as of release 1.1.1


[Bug] Changed name of the file to quicksort-custom.json5 and now it just crashes

As the tittle suggests, i tried following the instructions in order to add modded chests from reinforced chests. it crashed, so I un-did the changes but left it named as the custom one, and it still crashes
https://mclo.gs/iyG50RC

Now it just kinda crashes altogether, claiming the config file couldn't be loaded even though it's just a copy paste of the one from the wiki.
https://mclo.gs/gQXFSAY

https://mclo.gs/Yt0Mwy9
worked before, now even by itself it doesn't work. All i had done was follow the instruction so i could add compat with another mod. then it would neither work with the edits nor without. then it deleted it's own config, so i added it back manually- it's named right, right file type (afaik) and filled with the config from the example.

Maybe it's an issue with the file type it's saving it as on my pc? As I technically saved it as a .txt so I'll try changing the file type and see if that works


New Version  56.1 available for go-e charger

New Version  056.1 (beta) available for go-e charger

Firmware 56.1 is released

0.17.4 released


Fixes cython compilation failures in Python 3.11.

What do these changes do?
Note that CI is not updated.
Related issue number
Fixes #251

I have attched links in comments for reference.

Thanks for your fix, I'll add an action workflow for Python 3.11 later in this PR.

It looks like there are other compilation issues, I'm trying to address them.

Cannot reproduce the build failure on CI with conda's Python 3.11 (conda create -n xorbits python=3.11).


Preparer handling dates

At some point, a date in ISO 8601 will need to be accepted for a field for the user module. This may be useful for other modules, so we probably want to handle this in the preparer.

How about this?
type Blah struct {
	Timeout    time.Duration `hcl:"timeout"`
	ExpiryDate time.Time     `hcl:"time"`
}

if the parser hits a time.Duration it will parse it with time.ParseDuration and expose the error (if any) to the user.
If the parser hits a time.Time, it will parse it as "YYYY-MM-DDThh:mm:ssz". If that fails, it will try parsing as "YYYY-MM-DD" and set the time of day to zero and the timezone to the system timezone.

Yes, that seems to make sense.

Closing this issue for the implementation of time.Duration. Opening issue #490 to handle adding time.Time.


OVA file overwrite should not happen when a conversion is attempted with same image file name.

Test Case:
Identify a separate file system or dir for saving all converted OVA images.
Convert a qcow2 to OVA for image1
When another conversion is attempted for image2 and same image name is used , then the older OVA image file is getting rewritten/overwritten. It is a loss of converted OVA image.
Command used:
[root@rhel83-arc-pvsadm image]# ./pvsadm-linux-ppc64le image qcow2ova --image-name arc-centos-82 --image-dist centos --image-url https://cloud.centos.org/centos/8/ppc64le/images/CentOS-8-GenericCloud-8.2.2004-20200611.2.ppc64le.qcow2
Expected results:
The script should check for the presence of a same file name before saving it for the current conversion.
If an OVA with same file/image name is already available in the same folder, then the script should append some number or timestamp sbefore saving it as an OVA file to the filename so that the older OVA file doesnt get overwritten.

/bug
/assign

/kind bug


Span isn't closed if JDBC implementation errors on the getUpdateCount method

Describe the bug
The JDBC plugin throws an error and doesn't deactivate and end the span if the JDBC implementation errors on the getUpdateCount method.
                span.getContext()
                    .getDb()
                    // getUpdateCount javadoc indicates that this method should be called only once
                    // however in practice adding this extra call seem to not have noticeable side effects
                    .withAffectedRowsCount(statement.getUpdateCount());

                span.captureException(t)
                    .deactivate()
                    .end();

This admittedly is an edge case where my company's application uses a partial implementation of the JDBC API in RMI connections between services.  It doesn't include the getUpdateCount method currently.
Steps to reproduce

Find a JDBC driver that doesn't implement getUpdateCount
Then call any method that queries the driver
See WARN error in logs: 2020-01-23 19:52:36.660 [thread] WARN co.elastic.apm.agent.jdbc.helper.JdbcHelperImpl - Ignored exception
All further spans will then fall under the JDBC span that doesn't get closed, causing the transaction to hit the max spans limit

Expected behavior
An error should be logged and the span should be closed.
Debug logs

Click to expand
Time | Class | Log
Jan   23, 2020 @ 13:52:36.660 | co.elastic.apm.agent.jdbc.helper.JdbcHelperImpl | 2020-01-23 19:52:36.660   [http-apr-8080-exec-9] WARN co.elastic.apm.agent.jdbc.helper.JdbcHelperImpl -   Ignored exception
Jan   23, 2020 @ 13:52:36.660 | co.elastic.apm.agent.impl.transaction.AbstractSpan | 2020-01-23 19:52:36.660   [http-apr-8080-exec-9] DEBUG   co.elastic.apm.agent.impl.transaction.AbstractSpan - decrement references to   'Cpaas#query' 00-e111478a4e929d0ab905aca8d4e812ba-b38e98da3afb26b9-01   (618fb395) (4)
Jan   23, 2020 @ 13:52:36.660 | co.elastic.apm.agent.impl.ElasticApmTracer | 2020-01-23 19:52:36.660   [http-apr-8080-exec-9] DEBUG co.elastic.apm.agent.impl.ElasticApmTracer - }   endTransaction 'Cpaas#query'   00-e111478a4e929d0ab905aca8d4e812ba-b38e98da3afb26b9-01 (618fb395)
Jan   23, 2020 @ 13:52:36.660 | co.elastic.apm.agent.impl.ElasticApmTracer | 2020-01-23 19:52:36.660   [http-apr-8080-exec-9] WARN co.elastic.apm.agent.impl.ElasticApmTracer -   Deactivating a span ('Cpaas#query'   00-e111478a4e929d0ab905aca8d4e812ba-b38e98da3afb26b9-01 (618fb395)) which is   not the currently active span ('/*messages_withstyle*/
Jan   23, 2020 @ 13:52:36.660 | co.elastic.apm.agent.impl.ElasticApmTracer | 2020-01-23 19:52:36.660   [http-apr-8080-exec-9] DEBUG co.elastic.apm.agent.impl.ElasticApmTracer -   Deactivating 'Cpaas#query'   00-e111478a4e929d0ab905aca8d4e812ba-b38e98da3afb26b9-01 (618fb395) on thread   728
Jan   23, 2020 @ 13:52:36.644 | co.elastic.apm.agent.impl.ElasticApmTracer | 2020-01-23 19:52:36.644   [http-apr-8080-exec-9] DEBUG co.elastic.apm.agent.impl.ElasticApmTracer -   Activating '' 00-e111478a4e929d0ab905aca8d4e812ba-f333510d189a9399-01   (1d4a9d76) on thread 728
Jan   23, 2020 @ 13:52:36.644 | co.elastic.apm.agent.impl.transaction.AbstractSpan | 2020-01-23 19:52:36.644   [http-apr-8080-exec-9] DEBUG   co.elastic.apm.agent.impl.transaction.AbstractSpan - increment references to   '' 00-e111478a4e929d0ab905aca8d4e812ba-f333510d189a9399-01 (1d4a9d76) (2)
Jan   23, 2020 @ 13:52:36.644 | co.elastic.apm.agent.impl.transaction.AbstractSpan | 2020-01-23 19:52:36.644   [http-apr-8080-exec-9] DEBUG   co.elastic.apm.agent.impl.transaction.AbstractSpan - increment references to   'Cpaas#query' 00-e111478a4e929d0ab905aca8d4e812ba-b38e98da3afb26b9-01   (618fb395) (4)
Jan   23, 2020 @ 13:52:36.644 | co.elastic.apm.agent.impl.transaction.Span | 2020-01-23 19:52:36.644   [http-apr-8080-exec-9] DEBUG co.elastic.apm.agent.impl.transaction.Span -   startSpan '' 00-e111478a4e929d0ab905aca8d4e812ba-f333510d189a9399-01   (1d4a9d76) {
Jan   23, 2020 @ 13:52:36.644 | co.elastic.apm.agent.bci.ElasticApmAgent | 2020-01-23 19:52:36.644   [http-apr-8080-exec-9] DEBUG co.elastic.apm.agent.bci.ElasticApmAgent -   Method match for instrumentation ExecuteUpdateWithQueryInstrumentation:   (((name(equals(executeUpdate)) or name(equals(executeLargeUpdate))) and   hasParameter(hasTypes(with(0 matches erasure(is(class java.lang.String))))))   and isPublic()) matches public int   com.corp.app.AppStatement.executeUpdate(java.lang.String,java.lang.String[])   throws java.sql.SQLException

@HoveringHalibut Did you get the chance to verify the fix?


EnhancementShowPartitionsStmt support forward to master

Proposed changes
Issue Number: close #16347
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.41 seconds
load time: 518 seconds
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230202103753_clickbench_pr_89440.html


[Docker Desktop Extension] Add a feedback modal in docker extension

Current Behavior

There is no feedback from user might want to reach with there queries.
Desired Behavior

Use the existing sistent feedback component and use in meshery docker desktop extension
Implementation

Acceptance Tests

Mockups

Contributor Guides and Handbook

📚 Meshery Build and Release Strategy
🎨 UI Design For Meshery Extensions Figma in Figma (open invite)
📚 Meshery Extensibility Providers
⌨ Meshery API Docs
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

// @humblenginr


[Windows 10] With current master, error while executing "ci/check_bats.bash -f .precondition"

Describe the bug

ci/check_bats.bash -f .precondition

The above attempt for execution does not find the required script file in current master branch.
To Reproduce
Steps to reproduce the behavior:

Checkout master branch
Follow the Readme.md file for setup details
Scroll down to "Getting Started"
Execute "ci/check_bats.bash -f .precondition"
It will stop & throw error :

bash: ci/check_bats.bash: No such file or directory

Expected behavior
May be once the code is in master this error should disappear, but steps to avoid such error in Readme.md will be helpful
Screenshots
Not supplied
Desktop (please complete the following information):

OS: Windows 10

Smartphone (please complete the following information):

N/A

Additional context

We executed : git submodule update --init --recursive --remote
and then started with "ci/check_bats.bash -f .precondition", it worked fine

Thanks for raising the issue with details.
ci is a git submodule to this mono repository. The sequence in the ReadMe.md has to be changed.
$ git submodule update --init --recursive --remote
and then followed by
$ ci/check_bats.bash precondition
Will update in develop branch and later merge it to master

Fixed and Available in Master


cannot  set HttpOnly attribute to false for cookie

Hi I am setting a cookie for tracking purposes in response object like this :
res.cookie('tracking_cookie', 'cookievalue',{ httpOnly:  false});
We have some client side script that want to read the cookies but it still set HttpOnly flag.
It seems that it something overrides the statement .
The reason I say that is when set the cookie with set or append like this:
res.append('Set-Cookie', 'tracking_cookie'=cookie_name; Path=/; HttpOnly=false');
I see this on the client side response header
Set-Cookie:tracking_cookie=cookievalue; Path=/; HttpOnly=false; Secure;  HttpOnly
this means something set the HttpOnly  attribute after the statement.

Hi @raugustin-amplify I wasn't able to reproduce your issue with the latest Node.js on the latest Node.js 6.x. Here is what I tried:
$ npm i express
npm notice created a lockfile as package-lock.json. You should commit this file.
+<EMAIL_ADDRESS>added 42 packages in 2.58s
$ node -v
v6.11.0
$ cat app.js
const express = require('express')
const app = express()

app.get('/', (req, res) => {
  res.cookie('tracking_cookie', 'cookievalue', { httpOnly: false })
  res.send('hello!')
})

app.listen(3000)
$ node app &
[1] 45570
$ curl -I http://<IP_ADDRESS>:3000/
HTTP/1.1 200 OK
X-Powered-By: Express
Set-Cookie: tracking_cookie=cookievalue; Path=/
Content-Type: text/html; charset=utf-8
Content-Length: 6
ETag: W/"6-j32I6QGlrToF2MwN6TMT/XYCj4w"
Date: Fri, 16 Jun 2017 19:45:40 GMT
Connection: keep-alive

You can see that the Set-Cookie header is just tracking_cookie=cookievalue; Path=/ so certainly nothing in the simple app there is forcing the HttpOnly. That means that whatever is causing your issue is contained within the difference between my app above and your app.
Can you please provide all the following?

Your Node.js version
Your Express.js version
Full, complete code I can run that reproduces the issue
Complete instructions for how to run your code and what call to make to see the issue

Thanks!

I am running node v8.0.0 and express 5.0.0
I get a similar result when I curl on my dev server....regarding the Set-Cookie header.
it dosen't set the http flag on my local server.
We run the host on a test environment with https connection:
here is a snippet of the relevant server code I extracted :
`
let express = require('express');
let http = require('http');
let _ = require('underscore');
let url = require('url');
let util = require('util');
let os = require('os');
let NodeCache = require('node-cache');
let nodeCache = new NodeCache({ stdTTL: 10000, checkperiod: 120 });
let cookieParser = require('cookie-parser');
let app = express();
let router = express.Router()
app.use(cookieParser());
let setCookie = function(req, res, next){
res.cookie('traking_cookie', 'cookieValue ,{ httpOnly: false});
next();
};
router.get('*', isAuthorized, setCookie, express.static('www/'));
app.set('views', __dirname + '/views');
app.set('view engine', 'ejs');
app.use('/ckla', router);
let server = http.createServer(app);
server.listen(9443);
`

Hm, I'm still not able to reproduce with that code. What is the call you are making against that server and seeing the issue with? Without being able to reproduce, I'm not sure what I can debug against.

hi @raugustin-amplify   what you are trying to do with
router.get('*', isAuthorized, setCookie, express.static('www/'));
router.get ('*', [middleware1, middleware2] , (req,res)={
 // do your stuff here api
res.cookie('name', 'express').send('cookie set');   //cookie is set with key value
  });

//"* " ==> means that you are redirecting all you get api's to this router through router level middle ware

Closing since I couldn't reproduce, so don't know what the resolution would be.


bug Use cached visible version when Fe replay

Proposed changes
Issue Number: close #xxx
Use cached visible version which will be persisted in log so that we can avoid to send rpc in fe replay.

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

run buildall

run buildall

run buildall


Handling broken connections?

As far as I can see connection errors are not handled and thrown to the global exception handler, which terminates the process. But in many cases it might be nice to support reconnecting.
If using redis as a single key/value cache... it would be preferred to reconnect and retry get/sets instead of crashing the entire process.
I think it's possible to handle errors by setting up a zone and handling the error, but:

This seems like an unfortunate way to this,
All pending commands are going to hang because the futures aren't resolved.

I think we could solve (2) by resolving all _unreplied commands in:
https://github.com/jcmellado/dartis/blob/00a30ebb97806e7f70ebe046ddb734dc17045f19/lib/src/client/client.dart#L282-L285
Otherwise, we essentially have futures that will never be resolved. We can just complete all the futures with some generic error, but without an error downstreamer are going to hang..
Am I missing something? I'm not sure exactly how to expose the error from (1), but if the commands return with a specific error when the connection is broken then it becomes a lot easier to wrap with retry logic.

As far as I can see connection errors are not handled and thrown to the global exception handler, which terminates the process. But in many cases it might be nice to support reconnecting.

Yes, but it seemed a bit hard to implement correctly to me, so I left out it from the very first version of the library. The main purpose of the library is implement the Redis protocol, of course.

If using redis as a single key/value cache... it would be preferred to reconnect and retry get/sets instead of crashing the entire process.

Indeed.
But note that the reconnection could not satisfy the client expectation when using the "pipelining" feature. And the reconnection should be disabled when using the "fire and forget" feature.

I think it's possible to handle errors by setting up a zone and handling the error, but:
This seems like an unfortunate way to this,
All pending commands are going to hang because the futures aren't resolved.
...

Well, the onError handler should be provided and a custom exception could be thrown. I saw that you already did that in your PR. I'll take a deeper look to it after.

I merged https://github.com/jcmellado/dartis/pull/12, but I can not get it working on my local machine.

TimeoutException after 0:00:30.000000: Test timed out after 30 seconds.

I got this eror on the two new tests added to client_test and connection_test.
Travis is working fine.

Hmm, I tried it on a cloud instance and I also get some timeouts...
I'm not exactly sure if there is a better way to do this... Maybe we should be piping manually and then break the pipe, it could be that calling destroy while something is being piped is undefined behavior.
Maybe sockets just behave differently on windows, I'm rather unfamiliar with Windows.
But in general, I find that there aren't many good ways to simulate flaky connections.

I can't get this working. Any idea?


options for sorting genomes in pancore

possible strategies for ranking the genomes

absolute ranking

minimal area in between core and pan curves (greedy -- slow)
minimal area in between core and pan curves (optimal -- very slow)

relative ranking (first genome remains fixed and serves as a reference)

distance order (very fast)
maximal core (fast): can be done left to right or right to left (the latter first needs to compute the right pan/core)
minimal pan (fast): can be done left to right or right to left (the latter first needs to compute the right pan/core)

possible strategies for selecting the first genome if relative ranking is chosen

fixed (keep manual selection - direct)
maximal number of peptides (very fast)
minimal number of peptides (very fast)

Original issue by @pdawyndt on Wed Apr 10 2013 at 20:21.
Closed by @bmesuere on Tue Jul 30 2013 at 14:08.

implemented:

sort by name
sort by size

Original comment by @bmesuere on Wed Jul 17 2013 at 17:23.

added:

maximize core
minimize pan
optimal (minimal distance between pan and core)
These methods all start from the current first node and calculate the optimal next one.

Original comment by @bmesuere on Tue Jul 30 2013 at 14:08.


Add simple WHERE time > absolute value test

Unit test only, this test case did not exist already.


[BSO] =cl corp missing divine sigil.

🐛 Bug Report
=cl corp missing divine sigil.

Fixed in next update.


Mask and DatePicker issue with Floating Label Plugin

Reproduction
https://formkit.link/8e1dd443d03b687d09f9b62642318d2d
Describe the bug
When I pre-filled value via ref on Mask or DatePicker Input, it's not showing the value. But when I focus on the inputs it shows the value.

Environment
• OS: [MacOS]
• Browser [Arc, Chrome]
• Version [0.17.x]

This is a issue with the CSS and not having target to look at on the input if there is a value in FormKit input. Right now the color is being controlled by:
[data-floating-label="true"][data-type="mask"]:not(:focus-within) input,
[data-floating-label="true"][data-type="datepicker"] input {
  transition-duration: 0.2s;
  transition-delay: 0.1s;
  color: transparent; /* causing issue above */
}

Possible simple addition to formkit to fix above issue:
if used the same data-has-value="true" found on the floating label element and adding it to the sibling input (with same functionally as the label) you could then target the input to set the color in CSS to color: inherit;

This has been fixed in the latest release via the new data-empty prop available on core and pro component outer wrappers. You may need to clear cache: https://formkit.link/cb9c1b3d233e2c36b34061f1c441f334


'lein midje' works, but 'C-c ,' doesn't

I hope somebody can explain this behavior--I've spent a lot of hours trying to understand what's wrong. Here's my sole test:
(against-background

  [(before :contents
           (doall
            (db/SYSsetup-InfWb "brain" "test")
            (db/SYSload "four-notecards")))

   (around :checks   ;was :facts
           (let [IGNORE       (db/SYSclear-all) ; new test; clear local data
                 test-icard   (nth (db/get-all-icards) 0)
                 test-sldata  (db/new-sldata test-icard)
                 test-slip    (nth (db/get-all-slips) 0)

                 frame1    (PFrame.)
                 canvas1   (.getCanvas frame1)
                 layer1    (.getLayer canvas1)
                 dragger   (PDragEventHandler.)
                 IGNORE    (.setMoveToFrontOnPress dragger true)
                 IGNORE    (.setPanEventHandler canvas1 nil)
                 IGNORE    (.addInputEventListener canvas1 dragger)
                 IGNORE    (println
                            "Check the topmost window; it should show"
                            "one slip")
                 ]
             ?form ))

   (after :contents
          (doall
           (Thread/sleep 8000)
           (db/SYSdrop "four-notecards")))
          ]

(fact
    ; FCNS TESTED: show
    (db/show test-slip 50 150 layer1) => nil)

  )   ; close "against-background"

What's odd is this: running 'lein midje works':
22:23 (work2) infwb $lein midje infwb.test.system-basics
Check the topmost window; it should show one slip
All claimed facts (1) have been confirmed.
22:38 (work2) infwb $

...but in Emacs, executing C-c , with the cursor inside the fact returns the following error in a new frame:
Unable to resolve symbol: test-slip in this context
  [Thrown class java.lang.Exception]

The value of test-slip must be visible to the fact for lein midje to pass it, so why doesn't the midje-mode command succeed?
Many thanks for your ideas and help!

This is a midje-mode question--I posted here by mistake. Sorry.