ExpressionChangedAfterItHasBeenCheckedError in [clrDgLoading]

Describe the bug
When using the clrDgLoading on the datagrid it gives the error:
ERROR Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'loading: undefined'. Current value: 'loading: true'.

How to reproduce
Implement the example from the Datagrid -> Server Driven example as stated in the docs.
https://stackblitz.com/edit/clarity-light-theme-v013-kdp43p?file=src%2Fapp%2Fapp.component.html
Versions
App

Angular: 6
Clarity: 0.13.2

Hi @farlock85
The initial refresh event is emitted in datagrid's AfterViewInit. So if you set your loading variable there (when you first set it to true before loading data) you will get this error as you are still in the same angular cycle.
You have several options:

Set the loading variable to true immediately when you declare it. As you use the refresh event for initially loading your data it won't be wrong to start in loading state. There is no "not-loading" state that should be visualized before the 1st loading spinner.
If you are also setting other variables which have bindings you will be getting more ECAIHBC errors. In this case it may be better to explicitly trigger a change detection by calling ChangeDetectorRef#detectChanges() in your component's ngAfterViewInit.


Does hlint still require happy to build?

I don't see the happy requirement mentioned in the modern hlint documents. Did hlint refactor out that piece at some point? Which versions of hlint require happy in order to build?

It still depends on haskell-src-exts, and haskell-src-exts still requires happy, so yes, it does.


Selected : true using ajax

I feel that I'm going round in circles here so would appreciate some assistance.
Firstly I initialise jsTree without any data:
$('.accessTree').jstree({         "core" : {              "themes" : {                   "variant" : "large"              },              'multiple': false,         },         "checkbox" : {              "three_state": false,              "whole_node": false,         },         "plugins" : [ "wholerow", "checkbox" ]    });
Then when a modal is opened the data is inserted into the tree via an AJAX call that as made. Below is the code that is added into the 'success' function of the AJAX
var folders = []                       for(x in returnedInfo.folders){                            if(returnedInfo.folders[x].parent == null){                                 var parent = '#';                            }else{                                 var parent = returnedInfo.folders[x].parent;                            }                            if(returnedInfo.folders[x].client_upload == 1){                                 var folder = {                                      id          : returnedInfo.folders[x].file_id,                                      parent      : parent,                                      text        : returnedInfo.folders[x].file_name,                                      state       : {                                           selected  : true                                      },                                 }                            }else{                                 var folder = {                                      id          : returnedInfo.folders[x].file_id,                                      parent      : parent,                                      text        : returnedInfo.folders[x].file_name,                                 }                            }                            folders.push(folder)                       }                       $('.accessTree').jstree(true).settings.core.data = folders;                       $('.accessTree').jstree(true).refresh();
The tree is built correctly. Although, it doesn't seem to set the checkbox to ticked.
I'm using the latest version of jsTree (3.3.5) and working in Chrome.

refresh will normally reload all data and reapply the last state, meaning that any state you have set using the data will not be applied. You can try setting the second parameter to true:
https://www.jstree.com/api/#/?q=refresh&f=refresh()


typo in reportDeprecated message

Environment data

Language Server version: 2024.3.2 (pyright version 1.1.351, commit 87171fbf)
OS and version: n/a
Python version (& distribution if applicable, e.g. Anaconda): n/a

Code Snippet
import typing_extensions

@typing_extensions.deprecated("This function is depracted")
def foo(x: int) -> str:
    return chr(x + 0x20)

foo(3)  # The function "foo" is deprecated   This function is depracted

The second line says "depracted" instead of "deprecated"

@typing_extensions.deprecated("This function is depracted")

That's the string that you provided to the decorator.

@typing_extensions.deprecated("This function is depracted")

That's the string that you provided to the decorator.

Welp I should sleep more apologies


Update sftp-write.adoc

typo: Crete -> Create

Thank you for reporting this. We already updated via https://github.com/mulesoft/docs-connectors/pull/1462, therefore, closing this PR.


rake commands fail if using rails 5

$ rake spec
rake aborted!
LoadError: cannot load such file -- rails/test_unit/sub_test_task
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/gems/draper-2.1.0/lib/draper/tasks/test.rake:7:in `require'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/gems/draper-2.1.0/lib/draper/tasks/test.rake:7:in `<top (required)>'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/gems/draper-2.1.0/lib/draper/railtie.rb:67:in `load'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/gems/draper-2.1.0/lib/draper/railtie.rb:67:in `block (2 levels) in <class:Railtie>'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/gems/draper-2.1.0/lib/draper/railtie.rb:67:in `each'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/gems/draper-2.1.0/lib/draper/railtie.rb:67:in `block in <class:Railtie>'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/railtie.rb:237:in `instance_exec'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/railtie.rb:237:in `block in run_tasks_blocks'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/railtie.rb:245:in `each'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/railtie.rb:245:in `each_registered_block'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/railtie.rb:237:in `run_tasks_blocks'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/application.rb:442:in `block in run_tasks_blocks'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/engine/railties.rb:13:in `each'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/engine/railties.rb:13:in `each'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/application.rb:442:in `run_tasks_blocks'
/home/vagrant/.rvm/gems/ruby-2.2.2@rails_5.0/bundler/gems/rails-1022796ac075/railties/lib/rails/engine.rb:460:in `load_tasks'
/home/vagrant/test_app/rakefile:7:in `<top (required)>'
(See full trace by running task with --trace)

example Gemfile
source 'https://rubygems.org'
gem 'rails', github: 'rails/rails'
gem 'sprockets-rails', github: 'rails/sprockets-rails'
gem 'arel', github: 'rails/arel'
gem 'sqlite3'
gem 'draper'

In test.rake Draper requires rails/test_unit/sub_test_task, but Rails 5 does not have that file in railties/lib/rails/test_unit anymore.
It was removed by 3297909 (see also: 8017e6a).

@audionerd do you have any plans regarding send pull request of your 'rails5' branch to the master branch of this repo?

@asiniy No, sorry, I haven't had time to dig further into Rails 5 support for Draper. My current branch was just a quick hack that "worked for me", but I didn't do any further testing.

can any one tell me how to resolve this error.

@mayank-istar - Try "gem 'draper', github: 'audionerd/draper', branch: 'rails5'"

@audionerd are you planning on opening a pull request?

@audionerd would be great to see it get merged.

I can't pull draper from github because I'm using it in an engine as a dependency. Are there any plans to add official Rails 5 support?

@ivanjolic95 : Did you try gem 'draper', github: 'audionerd/draper', branch: 'rails5'

I can't really use any fork because I cannot pull gems from github in engine's .gemspec file. I've tried to put it in the gemfile and it doesn't work.

Hi, folks!
I'm currently collecting all Rails 5 related fixes in single branch.
But we need an active maintainer to merge it and release new draper version.

Instead of using forked version of draper, you can do monkey patch like this:
# lib/rails/test_unit/sub_test_task.rb
class Rails::SubTestTask < Rake::TestTask
end

How come this is tagged as Closed and Merged when it very clearly isn't?
I've just created a new Rails 5 project and get this exact error.

My team is also receiving this error on various projects when attempting to move to Rails 5. Any progress on getting this fixed (without monkey patching)?

@JohnSmall @kyledecot - same.  Brand new Rails 5 project and just gem 'draper' in Gemfile.  Using the fork seemed to work though.

Guys, feel free to switch to the gem 'drape'. It's a draper designed for rails 5.

@asiniy do you mean https://github.com/MrEmelianenko/drape? It's not developed for the last 5 months, how can anyone be sure it will be maintained in the future?

omg, that's lame.

Still no solution for this? drape hasn't been updated in 8 months...

Checkout https://github.com/drapergem/draper/issues/772

Could someone please open a PR?

So, the issue is 1,5 years old and there is still no fix for that? Except the monkey patch and forked repos, of course...

Use:
gem 'draper', "3.0.0.pre1"
Not a fork and not a monkey patch. Seems to be the "best" solution right now.

It works with rails 5, thank you @MatthiasRMS

Hi, same issue with gem 'rails', '~> 5.0.1'
And gem 'draper' is not working for me, I try master, rails5 branch, 3.0.0.pre1

Is this still an issue in master?

I believe this is no longer an issue. Please correct me if I'm wrong and I'll take a look into it.

@codebycliff Is a new version going to be released on rubygems?

@kyledecot Yes, soon.. I'm hoping to have a release in the next couple weeks.


Replace changelog-linker with github_changelog_generator

https://github.com/symplify/changelog-linker is no longer maintained. I'd rather not use the suggested replacement ergebnis/github-changelog, because according to the README it will not work with squash merges, which I regularly used when a PR has more than 1 commit. The ruby gem https://github.com/github-changelog-generator/github-changelog-generator is one of the proposed alternatives.
I propose switching to github_changelog_generator and regenerate the changelog with it.

I use github_changelog_generator in most of my projects. It works great when your project fits within the allowed API limits for authenticated calls, but it gets super buggy otherwise. It has caused me much heartache over at CodeIgniter4 so I recommend running it a few times (with and without token) to make sure you aren't close to the size where it will become unwieldy.


Preserve the size of "Open Database Object" window

Hi.
It would be really convenient if the developers would implement functionality for preserving the size, especially the width, of the "Open Database Objects..." window (top menu "Navigate > Open Database Objects..."). When I open it, it is always has the same size even if I previously changed it (made wider):

I was expecting that if I change the size / width of this window, next time when I open it - it will be of the same width. Most of the time search results do not fully fit the window width (because tables are shown with the related fully qualified names / schemas), that is why I almost always have to increase the width of this window to have an aesthetically pleasant view of the results.
So it would be great to have the above described feature in future versions of DBeaver.
Thank you.

This feature request was initially created as part of DBeaver "Discussion":
https://github.com/orgs/dbeaver/discussions/23213#discussion-6410755

Thanks for the idea @informatik01

UPDATE
Just noticed that the requested functionality perfectly works in cases when you try to open a database object AND no connection is selected: in such cases a pop-up window appears notifying a user that no connection is selected ("No active datasource") and this window nicely preserves the last selected size (both width and height) and even position on a screen.

So in other words, the requested functionality is already present (indirectly), but ONLY for the notification window.
FYI.


maven.twttr.com is obligatory resolver as of 2.4.0

libraryDependencies += "com.twitter" %% "finatra-http" % "2.4.0" fails to build because stats-util is not on Maven Central.
Expected behavior
That, when I follow the documentation's instructions at http://twitter.github.io/finatra/user-guide/getting-started/#dependencies, there are no other required steps, at least, none that are particularly out of the ordinary. (An extra resolver is not too much to ask, but it should be asked.)
Actual behavior
finatra-http depends on finatra-jackson, which depends on finatra-utils, which depends on inject-utils, which depends on finatra-mux, which depends on stats-util, which isn't in Maven Central.
Steps to reproduce the behavior

rm -r ~/.ivy2/cache, because if it was working for you before, you've probably got it cached locally (sbt clean has no effect on the system cache)
Remove the resolvers += ... line from your build.sbt, which is not required for packages published to Maven Central.
sbt build

@chbrown Are you referring to "com.twitter" %% "util-stats"? If so, that is in Maven Central here.

@taylorleese I found the naming super confusing at first too! But no... follow the finatra-http chain up all the way and it fails wanting "com.twitter.common" % "stats-util" % "0.0.58" -- see http://maven.twttr.com/com/twitter/common/stats-util/0.0.58/

@chbrown Ah, yeah that is from https://github.com/twitter/commons/blob/2097ff289d8e04040a6b004d2c6ba64a061661f0/src/java/com/twitter/common/stats/BUILD#L130. We have an internal ticket to start publishing commons to maven central, but we haven't started on it yet.

@taylorleese Yeah it works just fine with resolvers += "Twitter Maven" at "https://maven.twttr.com" in my build.sbt. It'd be nice if commons was was on Maven Central but mostly my issue was with the lack of documentation in the Getting Started guide, because it's kind of hard to debug Maven dependency resolution errors.

@chbrown noted about the documentation. But to be honest this need for the extra resolver has existed since Finatra's inception and is encoded in every example and in the Lightbend Activator templates. We're sorry you're hitting it and will feature it more prominently in the documentation as required until such time as the dependencies can be published to maven central. Thanks!

@cacoco cool, thanks. Sorry, I see now that my "as of 2.4.0" could be a little misleading; I merely meant "at least as of" — 2.4.0 is the first Finatra version I've ever touched.
And yeah, looking at an example project's build.sbt was how I figured out what the problem really was, and how to solve it.

@chbrown documentation has been updated in the getting started section: http://twitter.github.io/finatra/user-guide/getting-started/#dependencies. Thanks!


Steps point to an old unexisting environment

Describe the bug
Steps point to an old unexisting environment when said environment was deleted, but at the GUI level the environment is set to an existing one.
To Reproduce

have a project with at least one environment
have a pipeline with a step pointing to said environment
delete the environment
create a new environment (or, alternatively, start with 2 environments in your project)
in the pipeline editor the step is reported to have switched to an environment that exists (i.e. it's not using  the old, deleted environment)
run the step
error given by the fact that the step definition is actually pointing to the old environment

Expected behaviour
What the GUI reports and the step definition should be in sync, one way or another, i.e. either the step having no set environment or the definition being updated with the new environment.

I think there is actually a small nuance here:

Have a project with one environment
Have a pipeline step point to that environment
Delete the environment
Create a new environment
Run the step (without looking at the step properties in the UI first)

Now the step will try to execute using the old environment (because the old environment is still in the pipeline definition):

Instead if you were to replace step 5. with:
5. Open the step properties in the pipeline editor and then run the step
Then the pipeline definition is updated to use the new environment. Probably because of the drop-down selector behavior.
Potential fix
When deleting the environment we should probably delete all usage of the environment in the pipeline definitions of the project. The pipeline editor can then handle the fact that no environment is defined.
Alternatively, the pipeline editor should not cause automatic switching to newly created environments BUT still switch automatically to newly build environments, i.e. building an existing environment should cause the resulting environment to be used.


Screen flickers to black if you loop a video

I have a loopable video that I am looping by doing this in the render loop (Android):
Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);
 if (videoPlayer != null) {
            if (!videoPlayer.render()) {
                try {
                    videoPlayer.play(movieFile);
                } catch (FileNotFoundException e) {
                    videoPlayer = null;
                    Gdx.app.log("EXCEPTION", "Could not loop video", e);
                }
            }
        }

Whenever the video is done playing, it will flicker once to black before it resumes playing. Currently I remove the glClear to make it less visible, but this is not really an approach I'd like to keep in the future.
Anyone have any suggestions or is this a limitation in the android mediaplayer?

I can't remember having this issue back when I tested a looping video on android. It is probably something device specific. It means that probably the completion callback on the android mediaplayer is arriving too late. I will see if I can reproduce it.

For your information, the pad in question is the Xiaomi Mi Pad 2

Can also reproduce the flicker on NVidia Shield

Here's a workaround: If you disable screen clearing the issue is still there in practice, but not visible anymore. The issue with this is that the moment you'd like to do something like fade the screen in / out and you do it when the video loops, you will still have a flicker.

Could you try if the issue persists with the newest snapshot?
I've added a setLooping method to let the own media player of android loop the video. So the issue shouldn't persist anymore


fix: prevent record/pick from introducing non-existing properties

With exactOptionalPropertyTypes being set, { foo: undefined } is no longer valid for type {foo?: string}. Although it is not part of the strict set, I wish we could respect the difference.
The current implementation of record/pick may introduce non-existing properties when such optional key(s) are picked:
pick("foo")({})
// { foo: undefined }

Proposed implementation is slightly modified from the code I committed to fp-ts-std.
I changed the signature to comply with Ramda (and also fp-ts-std). A single array of keys is more composable generally IMHO.
Please let me know if this makes sense. Thank you.

Would you also mind running deno fmt to clear that check-format flag?

Please let me know if there’s something I left.


[Feature Request] Composer Support

Hello and thanks for this beautiful repo:)
Do you have plans to add support for Composer? It would be great to use this with Drupal, Symfony or Laravel.
Best,
Orkut

Not on our roadmap, but feel free to contribute!
Here are the instructions: https://github.com/e2b-dev/fragments?tab=readme-ov-file#adding-custom-personas

Thanks for your kind answer:)


Correct missing import and update SDXL example

Also now added to diffusers here: https://github.com/huggingface/diffusers/pull/4439

thanks @patrickvonplaten !


Fix sliver offset calculation again

Sorry that I misunstertood it again... Now this should fix https://github.com/flutter/flutter/issues/18731 and I've added unit test for it.

CC @ ebeem : thanks again for the reporting!


Enable preventing engine build at runtime

Adds a new API argument allow_build_at_runtime which allows users to prevent building TRT engines at runtime if desired. The existing behavior can be achieved by setting this argument to True which is also the default value.
This argument is useful for users of the build() method that try to build all the engines offline before doing any inference, and then want to avoid any optimization during inference, keeping low latency. In such cases, if there is no engine found in the cache, then native TF is used instead.

I just fixed the api compatibility test by adding the new argument to the golden api.

Hi @pooyadavoodi, could you help to resolved the conflicts? Thanks.

This PR has to wait until https://github.com/tensorflow/tensorflow/pull/35198 gets merged.

@sanjoy @aaroey
When I run any c++ test that executes native TF function instead of TRT, then I get this error: Executor failed to create kernel. Not found: No registered '_Arg' OpKernel for 'GPU' devices compatible with node.
I don't think it's related to _Arg because I replaced it with other nodes and still fails.
It also seems unrelated to this PR because I can repro the error without the PR if I change the test to force running native TF functions. For example, you can substitute ops::Add by ops::Less in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/compiler/tf2tensorrt/kernels/trt_engine_op_test.cc#L60 to repro the issue. We currently don't have any tests that execute native TF functions and that's why we never caught this.
This is how I run the test: bazel test -c opt --config=cuda --cxxopt=-D_GLIBCXX_USE_CXX11_ABI=0 --noincompatible_do_not_split_linking_cmdline //tensorflow/compiler/tf2tensorrt:trt_engine_op_test
Let me know if the problem is how I run the test.
I am kind of stuck in root causing this.

@pooyadavoodi It could be that you're not linking in the correct kernel registrations.  For instance, the GPU implementation of the _Arg op is defined in function_ops.cc and you need to link that in to be able to execute _Arg ops.
Can you try linking in //tensorflow/core/kernels:function_ops and //tensorflow/core/kernels:array to the test binary and see if that fixes or at least changes the failure?

@pooyadavoodi It could be that you're not linking in the correct kernel registrations. For instance, the GPU implementation of the _Arg op is defined in function_ops.cc and you need to link that in to be able to execute _Arg ops.
Can you try linking in //tensorflow/core/kernels:function_ops and //tensorflow/core/kernels:array to the test binary and see if that fixes or at least changes the failure?

Thanks @sanjoy. Linking changed the error. Now I am looking at another problem.

Thanks @sanjoy. Linking changed the error. Now I am looking at another problem.

Looks like this isn't ready for review so removing the tag.  @pooyadavoodi please LMK if this is indeed ready for review.

I triaged the problem down to OpsTestBase.  With this change patched in I now get a different failure from the test.

@nluehr @cliffwoolley

@sanjoy I can see the following error in the log you linked:
2020-02-01 06:22:10.737479: E tensorflow/compiler/tf2tensorrt/utils/trt_logger.cc:42] DefaultLogger ../rtSafe/safeContext.cpp (105) - Cudnn Error in initializeCommonContext: 4 (Could not initialize cudnn, please check cudnn installation.)

This usually happens when TRT can't run at all. The reason could vary from bad GPU or OOM.
It could be also because of having multiple GPUs (due to a known issue in TF-TRT). Could you rerun the test with 1 gpu?

This usually happens when TRT can't run at all. The reason could vary from bad GPU or OOM.
It could be also because of having multiple GPUs (due to a known issue in TF-TRT). Could you rerun the test with 1 gpu?

My machine does have two GPUs so that's probably it, but I haven't checked.

My machine does have two GPUs so that's probably it, but I haven't checked.

Just to clarify, the next step here is: someone ( @DEKHTIARJonathan ?) needs to commandeer this PR and apply the patch from the previous comment.  Then the tests should start passing and we can continue reviewing the PR.

I tried on single GPU, the convert step works very well but calibrate step costs all of the GPU memory (6GB) and shows the following error:
E tensorflow/compiler/tf2tensorrt/utils/trt_logger.cc:41] DefaultLogger ../rtSafe/safeContext.cpp (105) - Cudnn Error in initializeCommonContext: 4 (Could not initialize cudnn, please check cudnn installation.)
I think it is because tftrt use out of memory. So, is there some method to limit the memory cost during calibrate step.

I have applied the patch from @sanjoy and rebased to latest master. The error in TRTEngineOpTestBase.AllowBuildAtRuntime was actially an error in defining the test: I have corrected the check for empty engine creation.  I have not encountered any errors during calibration.
To fix CLA issues (original PR author has left the company), I have pushed the corrected branch to my repository, and opened a new PR #36852. This PR can be closed.

Thanks @tfeher for the confirmation.
Closing this PR, as its change has been moved to #36852


Use .editorconfig with prettier

PR Checklist

[x] Addresses an existing open issue: fixes #51
[x] That issue was marked as accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
Replace .prettierrc with .editorconfig for consistent styling across editors.

👋 just checking in @estevezluis, is this still something you're interested in?

👋 just checking in @estevezluis, is this still something you're interested in?

Ah I thought I completed the task 😅
Is there a feature I should I add to this PR?

Ah sorry you're right, you were waiting on me! I'm surprised Prettier is working with just an .editorconfig and no explicit indication to use it. Cool!
Anyway, thanks for this 😄 looks great!


Add checker

The checker should export a collection of functions which can be used to check the code for various errors or warnings.
These types of checks should include but are not limited to

Making sure stop is only used inside a loop body.
Detecting when a function has divergent return paths (this should probably done by building a control flow graph using basic blocks, which would also allow for rudimentary infinite loop detection).
Checking that function calls contain the right number of arguments.
Checking that no const variables are being reassigned.
Warning about expression statements that have no side effects (e.g. can be removed with no impact on the result of the program).

As more checker functions are thought of, they will be added to this issue.

Closing. Some of the points here have already been addressed elsewhere. Now that the project has been restructured, it doesn't make sense to have a single "checker" file that exports absolutely everything related to semantic checks. Individual issues will be opened for new semantic passes.


oversized undo/redo icons in live edit take half the screen

Running microweber 1.3.4
When editing the pages in "live edit" mode, the undo/redo svg icons are sized incorrectly, taking up nearly half of the screen, making it difficult to see the advanced menu and the upper portion of the page.
What also seems odd is that on mouse over the whole row gets highlighted on top for the undo button and below it for the redo button, but I suspect that the undo/redo buttons would be next to each other on the same row

Hello, thank you for the report.
In which browser do you see this issue?

@raikov
Using FireFox 96.0.3 (64 bit) on Windows 8.1pro.
The template I'm using in MW is "New World" but I don't think that would matter.
In FF "inspector", if I edit the SVGs from viewbox="0 0 24 24" to width="24" height="24" viewbox="0 0 24 24" then the icons appear where they should with a normal size.
I also found the same issue when trying to switch the header logo to "Text Logo", in the edit window all the toolbar icons are oversized.
Here's a screenshot showing the issue on both the top undo/redo and logo editor window:

Thank you. Yes it works fine after I upgraded to the latest release of FF (v113.0.1)
v96 is only 1 year old so I wasn't expecting CSS compatibility issues.

Closing issue as it is now solved.


SharpDX Samples - Why is there a performance problem when using WinForms and worse in WPF?

Hello,
Currently I am using a WriteableBitmap to output images from a camera and I need at least 60FPS or more for it. So I wanted to use pure hardware rendering for this task because the WriteableBitmap doesn't perform that well when maximizing the window.
I used the Direct2D example and modified the Render methode to iterate over 5 Full-HD pictures.
When using the Direct2D1DemoApp as Base I am able to achieve 200-250 FPS.
But when I change the Base class to Direct2D1WinFormDemoApp the FPS are stuck at 60FPS.
I also created an example with WPF and used it as an ImageSource but the performance is even worse. A small window is able to achieve 60FPS aswell but when I maximze the window the FPS drops to 30FPS.
Can someone help me why this is happening?
Is WinForms locked to 60FPS or is it bound to the refresh rate of my monitor or something?
Why is WPF performing so bad? Should I use a WinForms Host inside WPF to achieve at least 60FPS?
Thank you in advance.

Sounds like it's running at vsync rate. I don't know the particulars of that demo code, but you may be able to unlock it. Though I don't see a reason to even worry about it. You aren't typically rendering video at >60fps. I would just forget it and get on with the job.
As for the 30 fps, that is also suspiciously like it's missing it's 60fps frame target and dropping down to the next available vsync rate - 30fps. So yeah, your hunch is probably right. I'm not sure why it wouldn't be meeting it's 60fps target though.

@Gavin-Williams Thank you for your feedback.
The reason I need more then 60 FPS is that it's an acoustic camera and its sending high speed images.
Video images and acoustic images and I want to display them with the FPS the camera is sending.

Ok. The following was the problem with the WinForms.
HwndRenderTargetProperties properties = new HwndRenderTargetProperties();
            properties.Hwnd = DisplayHandle;
            properties.PixelSize = new SharpDX.Size2(demoConfiguration.Width, demoConfiguration.Height);
            properties.PresentOptions = PresentOptions.Immediately;

You have to send the PresentOptions to Immediately to break out of the 60FPS cap.
The problem now is how can I do it with WPF...

The problem now is how can I do it with WPF...

Don't expect anything good FPS wise when using WPF. Hosting HWND into a WPF partially solve this, but WPF is intrinsically not designed to handle this correctly.

@xoofx I found an example that us a HWND inside WPF. Maybe I am lucky with that implementation.
Currently I am looking into the RenderLoop how I can create my own inside the WPF control.

@datoml,  Can you share your WPF code?

@Mitra-M I used this control to render D3DImages inside WPF. Hope it helps you a bit :).

@datoml , Thanks you.
I'll try that.

@Mitra-M If you have some further question I'll try to help as good as I can :).

@datoml , I've tried it now, But has Airespace problem.
I prefer to use HelixToolkit.
Thanks anyway.


Architecture Contract

Need to establish an AC for EDS. This should cover both information sites (e.g. Storefront / Storybook) and technical solutions including the React Components.
Ref. https://github.com/equinor/architecturecontract

https://github.com/equinor/architecturecontract/pull/414
AC PR is up. Next are meetings for review.

AC PR is merged in. Closing this issue.


Last message in LogSearchShipper.log isn't shipped

It looks like the last message in the LogSearchShipper.log isn't shipped until a new line is written to the logfile (which itself isn't shipped until the next line is written etc)
This is most likely because the json log line is written without a newline character

Closing as won't fix


Why QuerySelectField does not work in custom edit view?

I have the following problem: on stack Flask, Sqlalchemy, Flask-Admin created the following models:
class Store(db.Model):
id = db.Column(db.Integer, primary_key=True)
address = db.Column(db.String(200))
users = db.relationship('User', backref='store', lazy='dynamic')

class User(UserMixin, db.Model):
id = db.Column(db.Integer, primary_key=True)
store_id = db.Column(db.Integer, db.ForeignKey('store.id'))
name = db.Column(db.String(128))
login = db.Column(db.String(20))
password = db.Column(db.String(20))
I use the following forms and views:
class UserForm(FlaskForm):
store_id = QuerySelectField('Склад', query_factory=lambda: Store.query)
name = StringField('Name')
login = StringField('Login')
password = StringField('Password')
class AdminSet(AdminModelView):
def edit_form(self, obj=None):
form = UserForm(obj=obj)
return form
The view works without problems, but when I try to save the changes, the following error appears:
Failed to update record. (psycopg2.ProgrammingError) can't adapt type 'Store' [SQL: UPDATE "user" SET    store_id=%(store_id)s WHERE "user".id = %(user_id)s] [parameters: {'store_id': <Store 2>, 'user_id': 2}] (Background on this error at: http://sqlalche.me/e/f405)
Why is this happening and what am I doing wrong?

QuerySelectField returns the actual object which you are then assigning to the id column where it expects an integer. Add a store = db.relationship(Store) to the model then change store_id = ... to store = ... in the form and I think it will work.


Zabbix_utils does not use REQUESTS_CA_BUNDLE env var

During testing we noticed that the Zabbix side of the sync script no longer works with our internal certificates.
This seems to have been introduced when we switched from pyzabbix to zabbix_utils.
Zabbix_utils doesn't seem to respect the REQUESTS_CA_BUNDLE environment variable by default.

Patch looks good and works well in our env. ✅ Thanks @wouterdebruijn

Merged and closed, thanks @wouterdebruijn


Why not read gzip and snappy parquet

It is common for production pipelines to dump intermediate data. The problem occurs when this data is in parquet format. Wouldn't it be nice if tv could read parquet.

I am worried about this being feature creep. I am not sure if I want this in tv or if it should be a separate cli.

maybe a better solution would be to have a cli that converts parquet to csv output


Throw exception if casting from double to datetime

For Spark 3.3.0+ there is a check added to throw an exception if casting from double to datetime in ANSI mode.
fixes #4342
Signed-off-by: Raza Jafri<EMAIL_ADDRESS>

build


Can't start container following README

I am following the README, am I missing something?
$ docker pull activestate/dockron
Using default tag: latest
latest: Pulling from activestate/dockron

Digest: sha256:b382ca64ac55a196d625469bbb3d66a25a67cf5827c50016ea5777a2259c604c
Status: Image is up to date for activestate/dockron:latest
$ alias dockron="docker run --rm \
       -v $(which docker):/usr/bin/docker:ro \
       -v /var/run/docker.sock:/var/run/docker.sock \
       activestate/dockron"
$ alias
...
alias dockron='docker run --rm        -v /usr/local/bin/docker:/usr/bin/docker:ro        -v /var/run/docker.sock:/var/run/docker.sock        activestate/dockron'
...
$ dockron "0 * * * *" docker run ubuntu /bin/bash -c "echo Hello world"
exec: "/go/src/app/app": stat /go/src/app/app: no such file or directory
Error response from daemon: Cannot start container 4dd6c3ab2beed26296549eb19a00f731a3a55aa977e78d1a0ff4119802b5b740: [8] System error: exec: "/go/src/app/app": stat /go/src/app/app: no such file or directory

Looking at the Dockerfile and Makefile I'm not seeing how dockron.go is ever executed/run? Does golang:1.3.1-onbuild do something special?

I'm not sure why ENTRYPOINT ["/go/src/app/app"] is in the Dockerfile?

@michaellopez The golang:1.3.1-onbuild image has some ONBUILD triggers which copy the content of this project folder into the container /go/src/app folder, follow by a call to go install -v. The app binary is then installed in $GOPATH/bin. (You can read more about this on the Golang official Docker Hub repo.)
Meanwhile, you can get this to work by replacing ENTRYPOINT ["/go/src/app/app"] with ENTRYPOINT ["/go/bin/app"]. I will open a PR to fix this.

@ihcsim Thanks for the info. I did a fork of this repo (https://github.com/weahead/dockron) where I updated the image and changed the Dockerfile to not use onbuild given the recommendations in golangs docker hub repo
But I'll see if your PR fixes my issue then I'll be glad to drop my fork as it would no longer be needed.

@ihcsim When do you think this PR will get merged?

@michaellopez The PR is merged. I have also updated the Dockerfile to use golang-1.5.2-onbuild as its base image, since 1.3 has been removed.
Let me know if this works.

@ihcsim Works perfect now. Dropped my fork, not needed now. Thanks for your effort!


Strip packaged binaries to reduce buildpack size

Stripping the packaged binaries will reduce the buildpack size, which will help with builder image pull times, and also local development/integration test runtimes (the copy of local buildpacks into the ephemeral builder image when using --buildpack is surprisingly slow).
Rust 1.58 stabilised the -C strip option:
https://doc.rust-lang.org/stable/rustc/codegen-options/index.html#strip
However it's not until Rust 1.59 (currently in beta, due for release 2022-02-24) that corresponding support will be added to Cargo:
https://doc.rust-lang.org/beta/cargo/reference/profiles.html#strip
The feature can be enabled by adding this to Cargo.toml for the crate in question:
[profile.release]
strip = true

However this would mean every user of libcnb would need to remember to do this, so I think it would make more sense if we configure it automatically in libcnb-cargo.
I think we may want to enable it for non-release libcnb-cargo builds too, since it will speed up development use-cases as mentioned above. (And debug symbols don't exactly help much when the buildpack is run inside the container as part of the lifecycle, since no easy way to run via a debugger anyway.)
Profile options like this can't be passed via cargo build arguments:
https://doc.rust-lang.org/beta/cargo/commands/cargo-build.html
...but we should be able to enable it via env var:
https://doc.rust-lang.org/beta/cargo/reference/config.html#environment-variables
For example:
CARGO_PROFILE_RELEASE_STRIP=true
CARGO_PROFILE_DEV_STRIP=true

Experimenting with this using Rust 1.59 beta, by running CARGO_PROFILE_RELEASE_STRIP=true cargo libcnb package --release, I get the following size reductions:

example-01-basics: 12.0 MB -> 4.1 MB
example-02-ruby-sample: 20.5 MB -> 8.5 MB

And for non-release builds (using CARGO_PROFILE_DEV_STRIP=true cargo libcnb package), I get:

example-01-basics: 50.4 MB -> 10.4 MB
example-02-ruby: 97.6 MB -> 20.7 MB

During the build, this warning appears:
   Compiling libcnb-proc-macros v0.1.1 (/Users/emorley/src/libcnb.rs/libcnb-proc-macros)
warning: stripping debug info with `strip` failed: exit status: 1
  |
  = note: /Library/Developer/CommandLineTools/usr/bin/strip: error: symbols referenced by indirect symbol table entries that can't be stripped in: /Users/emorley/src/libcnb.rs/target/release/deps/liblibcnb_proc_macros-f34f391f2c4d9c8e.dylib
          __NSGetArgc
          __NSGetArgv
          __NSGetEnviron
          __NSGetExecutablePath
          __Unwind_Backtrace
          __Unwind_DeleteException
          __Unwind_GetCFA
          __Unwind_GetDataRelBase
          __Unwind_GetIP
          __Unwind_GetIPInfo
          __Unwind_GetLanguageSpecificData
          __Unwind_GetRegionStart
          __Unwind_GetTextRelBase
          __Unwind_RaiseException
          __Unwind_Resume
          __Unwind_SetGR
          __Unwind_SetIP
...

The build completes successfully however.
What's interesting about this, is that the path referenced in the error message is under target/release/ not target/x86_64-unknown-linux-musl/release/. This plus the fact it's for the libcnb-proc-macros crate, makes me think it's the stripping of the compile-time regex validation that's failing (which is compiled for the host), rather than the cross-compiled crate itself.
Trying again using a strip mode of debuginfo rather than true (true being equivalent to symbols, the maximum level), the warning goes away, and I get the following (slightly larger) output sizes:

example-01-basics: 5.4 MB (release), 14.5 MB (dev)
example-02-ruby: 11.3 MB (release), 28.9 MB (dev)

The Cargo docs mention a build-override setting:
https://doc.rust-lang.org/beta/cargo/reference/config.html#profilenamebuild-override

The build-override table overrides settings for build scripts, proc macros, and their dependencies. It has the same keys as a normal profile.

We can take advantage of that by using:
CARGO_PROFILE_DEV_STRIP=true CARGO_PROFILE_DEV_BUILD_OVERRIDE=debuginfo cargo libcnb package
...which prevents the warning, whilst still giving us the full output size reductions.
It seems like perhaps this might be a Rust bug where it's not fully taking into account the strip options required for the host vs the target, but either way, there is an easy workaround for now :-)

It seems like perhaps this might be a Rust bug

Filed:
https://github.com/rust-lang/rust/issues/93988

Now available in stable:
https://blog.rust-lang.org/2022/02/24/Rust-1.59.0.html#creating-stripped-binaries

Filed:
rust-lang/rust#93988

The warnings seen when stripping using local path dependencies (ie when hacking on libcnb in this repo) will now be resolved in Rust 1.63, thanks to:
https://github.com/rust-lang/rust/pull/97991
That needn't block the work on enabling stripping in libcnb, since (a) they are only warnings, (b) they aren't shown when people use libcnb from crates.io (only local path based builds).


Add inverter models with current limiting

The existing inverter models do not include current limiting. It would be good to include some models from the literature that have current limiting.  @ciaranrob has started an implementation.
One key question is how to make the addition of current limiting modular and extensible? Does it make sense to define a new model (e.g. CurrentLimitedVoltageModeControl)  which is essentially equivalent to VoltageModeControl with the addition of an Imax parameter or is there a better way? @jd-lara @rodrigomha do you have thoughts on this?

That proposed option is okay for now, but it is not reasonable to do that more than once, so if we want to try other limiters we need a long-term solution.
I think the proper solution requires some major changes.
An intermediate solution would be similar of how the outer control work: create a struct for InnerControl that contains the VoltageControl, CurrentLimiter and CurrentControl.
Then, you can probably dispatch the combinations that you want in a clever way (using _mdl_ode_functions to dispatch internally any combination you want) or repeat code.
This would require to do a bunch of things:

[ ] Define a NoCurrentLimiter struct
[ ] Define a NoVoltageControl struct
[ ] Split VoltageModeControl into something like VoltageControlVImpedance, NoCurrentLimiter and StandardCurrentControl
[ ] Split CurrentModeControl into something like NoVoltageControl, NoCurrentLimiter and StandardCurrentControl

What do you think @jd-lara ?

looking into this issue it seems we need to modify the structures of the inverters to incorporate current limiter DynamicComponents. We need to PoC this for PSY 3.0.

This closed in #383


"From" field in emails are not rendering correctly

Issue Summary
The 'From' field of sent emails are not rendering correctly. Relevant
To Reproduce

Set desired email name in production.config.json
Send an email by publishing a test post that sends out to newsletter subscribers.
The 'From' field takes the blog title as the sender name.

Technical details:

Ghost Version: 4.12.1
Node Version: 14.17.2
Browser/OS: Firefox/Ubuntu 20.04
Database: MySQL

Related forum posts
I've previously posted about this but hasn't seen a good fix.

A thread I read
A thread with a proposed fix, but it is hardcoded
A thread I started

Steps taken to attempt a fix

Playing with the config.production.json file: remove quotes from name, tried escaped double quotes, swapped single and double quotes.
Looking at the getFromAddress function in GhostMailer.js, validator.isEmail (though not the @tryghost one), and nodemailer, but the logic looks right

Really have no idea where the issue is. Perhaps there is something wrong with the message.from object but I can't tell.

In server services/mega/mega.js
In getfromadress:
const siteTitle = settingsCache.get('title') ? settingsCache.get('title').replace(/"/g, '\\"') : '';
return siteTitle ? '"${siteTitle}"<${fromAddress}>' : fromAddress;
So it's always the siteTitle as name from (if siteTitle is set).
In my opinion we would need a new setting (members_from_name) for that
I can work on it, if needed.

That's great news!
2022年4月28日(木) 19:43 Hannah Wolfe @.***>:

Closed #13280 https://github.com/TryGhost/Ghost/issues/13280.
—
Reply to this email directly, view it on GitHub
https://github.com/TryGhost/Ghost/issues/13280#event-6514391049, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFPNRM6AW2JVSAH66G7FT3DVHJ2UPANCNFSM5DNFR3SA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Gain and gamma controls before applying Display/View

It is great to be able to adjust Gamma and Gain controls before applying Display/View to the image.
If that would invalidate the cache, maybe making it optional?
Very useful for tech-checking shots for example.
Thank you

It is great to be able to adjust Gamma and Gain controls before applying Display/View to the image. If that would invalidate the cache, maybe making it optional? Very useful for tech-checking shots for example.

AFAIK, gamma and gain are adjusted before OCIO.  Do you see it happening otherwise?  That might be a bug.
Currently the order of shaders is hard-coded in tlRender code.  If you need flexibility from switching from sometimes applying them before and other times applying them after, you should add a feature request with Darby Johnston at:
https://github.com/darbyjohnston/tlRender/issues
If it is a feature you need right now you may want to look into OpenRV which has a completely flexible shader pipeline.


Fix/fence yaml merge

Copy of https://github.com/uc-cdis/gen3-helm/pull/151/files
Changes added on to PR 151:
Adjusting the fence public config so it pulls from FENCE_CONFIG_PUBLIC in the values.yaml.
Changing the manifest-fence configmap to "fence-config-public.yaml" so it is properly mounted in the container.
Editing presigned URL to save the fence public config to a file.
New Features
Users will now have the ability to keep sensitive Fence configurations separate from Fence public configurations.

Do we need to update base branch as well? Might be some conflicts after that 😬

Thank you! Yes, just updated base branch and it looks like there are no conflicts :)
Merging!


Upgrade schema different from default

Discovered while testing deployment upgrades, see comment in #1344 - the schema that results from running migrations on our old (v3.4.10) production databases is different from the schema that we have checked into our repo / results from running the migrations on a clean database. This is probably due to some ill-advised migration tweaking that occurred before my time, not sure if this is critical but wanted to have a place to discuss it.

Here's the git diff:
diff --git a/db/schema.rb b/db/schema.rb
index 727249e..7535577 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -17,8 +17,8 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.text     "message"
     t.date     "starts_at"
     t.date     "ends_at"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
   end

   create_table "app_configs", force: true do |t|
@@ -64,8 +64,8 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.text     "notice"
     t.integer  "created_by"
     t.text     "blackout_type"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",    null: false
+    t.datetime "updated_at",    null: false
     t.integer  "set_id"
   end

@@ -73,8 +73,8 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.string   "name"
     t.integer  "max_per_user"
     t.integer  "max_checkout_length"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",                              null: false
+    t.datetime "updated_at",                              null: false
     t.integer  "sort_order"
     t.datetime "deleted_at"
     t.integer  "max_renewal_times"
@@ -86,16 +86,16 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
   create_table "checkin_procedures", force: true do |t|
     t.integer  "equipment_model_id"
     t.string   "step"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",         null: false
+    t.datetime "updated_at",         null: false
     t.datetime "deleted_at"
   end

   create_table "checkout_procedures", force: true do |t|
     t.integer  "equipment_model_id"
     t.string   "step"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",         null: false
+    t.datetime "updated_at",         null: false
     t.datetime "deleted_at"
   end

@@ -104,11 +104,11 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.string   "serial"
     t.boolean  "active",                               default: true
     t.integer  "equipment_model_id"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",                                           null: false
+    t.datetime "updated_at",                                           null: false
     t.datetime "deleted_at"
-    t.boolean  "csv_import",                           default: false, null: false
     t.string   "deactivation_reason"
+    t.boolean  "csv_import",                           default: false, null: false
     t.text     "notes",               limit: 16777215,                 null: false
   end

@@ -120,8 +120,8 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.integer  "max_per_user"
     t.boolean  "active",                                              default: true
     t.integer  "category_id"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",                                                          null: false
+    t.datetime "updated_at",                                                          null: false
     t.datetime "deleted_at"
     t.string   "photo_file_name"
     t.string   "photo_content_type"
@@ -150,14 +150,29 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.integer "equipment_model_id", null: false
   end

+  create_table "equipment_models_reservations", force: true do |t|
+    t.integer  "equipment_model_id"
+    t.integer  "reservation_id"
+    t.integer  "quantity"
+    t.datetime "created_at",         null: false
+    t.datetime "updated_at",         null: false
+  end
+
+  create_table "equipment_objects_reservations", force: true do |t|
+    t.integer  "equipment_object_id"
+    t.integer  "reservation_id"
+    t.datetime "created_at",          null: false
+    t.datetime "updated_at",          null: false
+  end
+
   create_table "requirements", force: true do |t|
     t.integer  "equipment_model_id"
     t.string   "contact_name"
     t.string   "contact_info"
     t.datetime "deleted_at"
     t.text     "notes"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",         null: false
+    t.datetime "updated_at",         null: false
     t.string   "description"
   end

@@ -169,8 +184,8 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.date     "due_date"
     t.datetime "checked_out"
     t.datetime "checked_in"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",                          null: false
+    t.datetime "updated_at",                          null: false
     t.integer  "equipment_model_id"
     t.integer  "equipment_item_id"
     t.text     "notes"
@@ -181,11 +196,15 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.integer  "flags",               default: 1
   end

+  add_index "reservations", ["due_date"], name: "due_date", using: :btree
+  add_index "reservations", ["equipment_model_id"], name: "equipment_model_id", using: :btree
+  add_index "reservations", ["start_date"], name: "start_date", using: :btree
+
   create_table "sessions", force: true do |t|
     t.string   "session_id", null: false
     t.text     "data"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
   end

   add_index "sessions", ["session_id"], name: "index_sessions_on_session_id", using: :btree
@@ -199,8 +218,8 @@ ActiveRecord::Schema.define(version:<PHONE_NUMBER>2612) do
     t.string   "phone"
     t.string   "email"
     t.string   "affiliation"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+    t.datetime "created_at",                                               null: false
+    t.datetime "updated_at",                                               null: false
     t.boolean  "terms_of_service_accepted"
     t.string   "view_mode",                             default: "normal"
     t.string   "role",                                  default: "normal"

To follow up - a lot of it is just whether or not we call null: false on our timestamps (we apparently do in our legacy databases but don't in our schema). Otherwise there are a few extra indices that we don't appear to have any more and some weird relation tables that are also no longer used.

Ok, well it looks like @adambray added these to the schema in 24b541cf but didn't add the relevant migrations... that's also the commit where they started tweaking the migration timestamps (I think, didn't dig too deeply into it). @dgoerger, any thoughts on whether or not this is something we need to address? Is there any way to see if those database items (primarily the indices and extra relation tables) are ever being called / if we can remove them somehow?

The created_at and updated_at fields are never updated directly, right? I thought that was built-in Rails stuff. Assuming this is the case, they should/can never be null as a practical matter. If we ever call these fields we'll want them to not be null, obviously, but how and whether you want to enforce that is up to you. Maybe it's residual from the Rails 2 iteration? The BMEC db is fairly old.
In determining whether or not a table is used... is it not sufficient to grep through the code base to see if we're calling it? If not, I guess we import the db locally, run a migration to remove the tables, and clicky-click through the app to see if anything breaks....

Re: created_at and updated_at - yes, they are automatically updated by ActiveRecord, and I'm guessing the null: false is a holdover from Rails 2. Tbh, this was the least concerning of the differences, and in my mind won't impact anything moving forward.
The equipment_models_reservations and equipment_objects_reservations tables are join tables used for has_and_belongs_to_many associations, I believe. Since those associations no longer exist (I think we now use scopes for those), I guess the tables should just have been removed whenever the associations were taken out, but weren't (or alternatively someone was testing out different associations on a production database).
I'll try to mess with this locally and drop those tables to make sure nothing in our test suite breaks. We won't want to write migrations for this / commit them in since this is specific to our database but what we'll probably want to do is migrate PROD over to DEV, drop it manually in DEV, and then repeat in PROD. Thanks for the feedback!

:+1: agreed on the proposed table drop procedure. Let us know!

We should look into this sooner rather than later.


Bad results

Without composition, just looking at the output of places 512 model, it looks so horrible, even the inpainted area doesn't seem up-to-par with the paper. Look at this example
Here's the original image

Here's the mask

Here's the direct output (without composition)

Looks so horrible, how did you report such a low FID score?

Hello. This method is not intended to be used without composition. If you take a look into the training strategy for this method, you'll see that the generated result is being blended with the original image before being passed as an input to the discriminator. So learning to copy the known region from the input to the output is not included in the objective of MI-GAN.
In summary, blending with the original image is a necessary part for MI-GAN (at least with the training strategy described in the paper and implemented in this codebase), and all reported metrics were computed with blending, for all methods in comparison.


bug: WalletConnect switchChain method uses the wrong block explorer URL

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
2.0.0-rc.0
Current Behavior
While testing the WalletConnect connector, I found an issue where if you need to switch chain it fails with this error

Uncaught (in promise) SwitchChainError: An error occurred when attempting to switch chain.
Details: {"code":-32602,"message":"Expected null or array with at least one valid string HTTPS URL 'blockExplorerUrl'. Received: [object Object]"}

Expected Behavior
The switch chain works as expected.
Steps To Reproduce

Connect wallet via WalletConnect
Select from and to chain and token (should be different from connected chain in the wallet)
Start swapping/bridging
Error will be thrown to the console and connected wallet with WalletConnect do nothing

Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
https://playground.li.fi/
Anything else?
I found that here https://github.com/wevm/wagmi/blob/rc/packages/connectors/src/walletConnect.ts#L266 we are trying to provide default block explorer URL, but actually providing object {name: 'BlockExplorer', url: 'Some URL'}

I've provided a PR for this https://github.com/wevm/wagmi/pull/3341


PoS NetworkManager Review

#1119  👈
feature/pos-networking-and-syncing

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AeonSw4n and the rest of your teammates on  Graphite


explorer no longer updating operations/transactions on account or asset view.

When looking into my account using stellar.expert, the last shown transaction seems to be from 2021-11-23 09:12:34 UTC and it looks like there was nothing more happening since then.
The same is true if i look into an asset, like XLM for example. There to, the last shown transaction claims to be from 2021-11-23 09:13:51 UTC..
Anything up on your database maybe? Live transaction log is still updating .. but not reflecting in the detail pages..

same on my end, can't retrieve live info

Made the same comment. Annoying when doing complicated stuff with labs.

Hello,
One of our Stellar Core nodes fell out of consensus, which led to the disruption of our historical data ingestion process. We addressed the root problem, and the service has been restored today around 2AM UTC.
Sorry for the inconvenience.


Improve code_block tab/indentation

Reported on Meta.SE

The Ctrl+K shortcut is what I always used to indent/unindent code blocks but...

it only worked for one-level indentation,
not many users knew about it, and...
currently, pressing Ctrl+K does nothing while in a code block in the Rich Text mode anyway.

Enabling Tab and Shift+Tab in code blocks will be a great addition.

@yellis I believe so, I'd need to take another look at it to confirm.


Cache for Build server strategy

Q
A

Issue Type
Feature Request

Deployer Version
N/A

Local Machine OS
N/A

Remote Machine OS
N/A

Description
If using the "Single server" strategy, update_code task uses option git_cache. If using the recommended "Build server" strategy, no cache is available by default. It is easy enough to create task, but maybe this could be integrated into core somehow.
task('deploy:update_code', function () {
  if (has('previous_release')) {
    run('rsync -a {{previous_release}}/ {{release_path}}/');
  }

  upload('.', '{{release_path}}', ['options' => [
    '--checksum',
    '--delete',
  ]]);
});

I am cleaning up on issues. Fill free to reopen if needed.
Also, we have a new section available: discussions.


syscall: win.GetMessage syscall.Syscall6 crash

golang version 1.15
Windows server 2012
Exception 0xc0000005 0x0 0x1749111c 0x77e47594
PC=0x77e47594
signal arrived during external code execution
syscall.Syscall6(0x7598a5b0, 0x4, 0x27066f8, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
C:/Go/src/runtime/syscall_windows.go:201 +0xbb
GoProxy/vendor/github.com/lxn/win.GetMessage(0x27066f8, 0x0, 0x0, 0x0, 0x1)
D:/svn_code/src/GoProxy/vendor/github.com/lxn/win/user32.go:2564 +0x8a
GoProxy/vendor/github.com/lxn/walk.(*FormBase).mainLoop(0x2eb9a000, 0x0)
D:/svn_code/src/GoProxy/vendor/github.com/lxn/walk/mainloop_default.go:20 +0x95
GoProxy/vendor/github.com/lxn/walk.(*FormBase).Run(0x2eb9a000, 0x0)
D:/svn_code/src/GoProxy/vendor/github.com/lxn/walk/form.go:373 +0xa3
main.main()
No problem has been found in win7 environment for the time being. This problem has been found in 2012 server. It's not a startup error, it's other groutines that call programs with a desktop.

goroutine 54 [syscall, locked to thread]:
syscall.Syscall(0x77689050, 0x2, 0x92c, 0xffffffff, 0x0, 0x0, 0x0, 0x0)
C:/go-build/Go/src/runtime/syscall_windows.go:172 +0xcf
syscall.WaitForSingleObject(0x92c, 0xffffffff, 0x0, 0x0, 0x0)
C:/go-build/Go/src/syscall/zsyscall_windows.go:737 +0x53
os.(*Process).wait(0x2e936930, 0x0, 0x0, 0x0)
C:/go-build/Go/src/os/exec_windows.go:18 +0x5a
os.(*Process).Wait(0x2e936930, 0x2e820600, 0x0, 0x0)
C:/go-build/Go/src/os/exec.go:125 +0x21
os/exec.(*Cmd).Wait(0x2ebc8210, 0xb6bea8, 0x0)
C:/go-build/Go/src/os/exec/exec.go:465 +0x43
GoProxy/utils/shell.RunCmdWithTimeout.func5(0x2ebc8210, 0x2e92605c, 0x2eb0e1c0, 0x2eba3d00, 0x2eb8a000, 0x24)
D:/gseeproxy/src/GoProxy/utils/shell/shell.go:204 +0x43
created by GoProxy/utils/shell.RunCmdWithTimeout
D:/gseeproxy/src/GoProxy/utils/shell/shell.go:198 +0x7f4
eax     0xffffffff
ebx     0x0
ecx     0x30
edx     0x0
edi     0x24
esi     0x1
ebp     0xffffffff
esp     0x0
eip     0x77e47594
eflags  0x0
cs      0x0
fs      0x14f63580
gs      0xbfd70d2f

This kind of crash will happen if you pass bad arguments, such as invalid pointers, to the Windows system call.  There's no particular reason to think that it is a problem with Go.  Different versions of Windows act differently.
If you are convinced that this is a bug with Go, we'll need a way to reproduce the problem ourselves.

@ianlancetaylor  MSG is new i don't where  invalid pointers?The problem has been hotly discussed. But it seems that the real reason has not been found, and I have this problem again.
https://github.com/lxn/walk/pull/493#issue-274104188
https://github.com/lxn/walk/issues/483
type MSG struct { 	HWnd    HWND 	Message uint32 	WParam  uintptr 	LParam  uintptr 	Time    uint32 	Pt      POINT }
`	msg := (*win.MSG)(unsafe.Pointer(win.GlobalAlloc(0, unsafe.Sizeof(win.MSG{}))))
defer win.GlobalFree(win.HGLOBAL(unsafe.Pointer(msg)))
for fb.hWnd != 0 {
	switch win.GetMessage(msg, 0, 0, 0) {
	case 0:
		return int(msg.WParam)

	case -1:
		return -1
	}`

The code you show isn't valid Go, but, more importantly, the MSG structure doesn't have the lPrivate field.  Perhaps the structure is too small.

@ianlancetaylor  I tried to add lprivate. The problem is still that this is not the cause.
winuser.h
`/*

Message structure
*/
typedef struct tagMSG {
HWND        hwnd;
UINT        message;
WPARAM      wParam;
LPARAM      lParam;
DWORD       time;
POINT       pt;
#ifdef _MAC
DWORD       lPrivate;
#endif
} MSG, *PMSG, NEAR *NPMSG, FAR *LPMSG;`

In order to help you, we'll need a way to reproduce this problem ourselves.

No reproducer or news for almost a year, I think we can close here.


Develop the Trending section on the Event page

https://www.figma.com/design/w1Hoj9ESFonR8tDl5WLedU/Event-Management?node-id=0-1&node-type=canvas&t=LaYd43Mtw6Tn6FYp-0

@ariyoaresa I can't find a trending section on the event page. Kindly review the figma design and you can also send a screenshot so i can ascertain which area you're working on.

This is the second section of the event page, just before the upcoming events

@ariyoaresa You can proceed. I'm sorry about the delay i just needed to confirm where you wanted to work on.

Merged #17


OOM error

Hi,
Thanks for sharing this awesome repo. I'm having memory problem with running run.sh with 1 H100 gpu. I tried reducing batch sizes to 1, but still I can not fine-tune a 7B LLM. Any idea how to work this around?

Hey did you ever get around this? How much memory was required? I will drop from fp32 to bf16 and reduce the batch size, but wanted to see if I could get any insight. Thanks for the help!

I didn't unfortunately. It seems like you gotta try multi-gpu training for this one

Ahh no worries, I appreciate the response!


changing link of cloud native page

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

@mvelosop if I merge this as-is can you take care of removing the other occurrences of that same link on this topic afterward?

Sure!

Done right here @mairaw.


Open special workspace with gesture

Description
I would love to be able too swipe up to open my special workspace.
It would just seem right with the current swipe left/right features for regular workspaces.
example :
swipe up with three fingers : special workspace opened
swipe down with three fingers: special workspace closed
The conf would not need to be very complicated:
gestures {
    	special_workspace_swipe = true
        special_workspace_name = scratchpad

 }

Tell me if that sounds like a terrible idea but I would use

it's not a bad idea, but most likely will come as a part of a more-or-less rewrite of how gestures are handled in general, akin to the deco positioner migration.
Would want to add more gestures in general, and make them configurable, eg:
gesture = 3, down, workspace, special:amogus


fix: rewrite misleading/unclear error messages

Validate
In an attempt to make the GitHub error reporting as clear and concise as
possible, this commit touches upon many of the tool's error outputs.

Don't print commit message rule violations multiple times
More clearly mark PR title errors
No longer refer to "commit messages" in the rule texts, as they're used
for PR titles as well

Bump
This also adds specialized handling in the bump action for the specific
case where tag creation fails due to an existing tag (often somewhere
on a test/dev branch).

Fix-up squashed.


Postgres 16 client support with wolfi-base

Which image/versions are related to this issue/feature request?
The latest postgres image (ish)
Issue/Feature description
Hi,
I used the postgres image as inspiration to create a really simple Postgres 15 client image like this
FROM cgr.dev/chainguard/wolfi-base

RUN apk update && apk add --no-cache --update-cache glibc-locale-en busybox su-exec bash postgresql-15-client libpq-15

ENV DURATION=30m
CMD sleep ${DURATION}

It's just used as a way for devs to spin up a container that can be used to connect to a Postgres server, and has been working fine.
Then Postgres 16 was released, and I thought I would try to upgrade the client image to 16 as well
RUN apk update && apk add --no-cache --update-cache glibc-locale-en busybox su-exec bash postgresql-16-client libpq-16

This works, but when connecting to the server I get
~ $ psql -U <user>
psql: error: connection to server at "<server>" (<ip>), port 5432 failed: FATAL:  no PostgreSQL user name specified in startup packet
connection to server at "<server>" (<ip>), port 5432 failed: FATAL:  no PostgreSQL user name specified in startup packet
double free or corruption (out)
Aborted (core dumped)

I've not been able to understand the cause of this. Do you have any input on what could be missing to make it work with Postgres 16?
If this issue doesn't belong here, feel free to close it.
Thanks in advance.

@jasaltvik I'm experiencing a similar issue, but I'm trying to create a minimal postgres image to run as a server to reproduce without all of our extra "fluff" I'm struggling to get it to throw this error for some reason (e.g. basic postgres image with just a single username/password set up). Are you able to share how you are running your postgres server? Is it just a simple docker setup or something similar?

Digging into this we found an issue with the way provider_priority was being handled (it gets dropped when generating the index) which was leading to an older version of libcrypt1 being installed by the solver (r7 vs. r5 for the rest of the stuff from glibc).
We're fixing that regardless, but if you have an easy repro, I am curious whether installing libcrypt1=2.38-r7 explicitly resolves this for you.
ref: https://github.com/chainguard-dev/melange/pull/878

@mattmoor I was able to find out that it has to do with TLS/SSL being enabled on the server. I've been going back and forth in a ticket (but did eventually end up in this issue after hours of trying to figure out specific server config was causing the issue lol) and did find a way to reproduce it. Installing that package specifically doesn't seem to help. I'll post the steps here as well for anyone that needs it.
To start, you have to generate a certificate and key and then set the permissions for the postgres user in the container.
openssl genrsa -out server.key 2048
openssl rsa -in server.key -out server.key
openssl req -sha256 -new -key server.key -out server.csr -subj '/CN=localhost'
openssl x509 -req -sha256 -days 365 -in server.csr -signkey server.key -out server.crt

sudo chown 70:70 server.crt server.key
sudo chmod 600 server.crt server.key

Then you can use this compose file to launch postgres using that cert/key and with TLS enabled. The chainguard postgres image can work as the server here, but it has some issues after the first login (likely due to the same issue, it will go into recovery mode due to some tls issue), so just use the docker hub one.
services:
  postgres:
    container_name: postgres
    image: postgres:15.5-alpine
    command: -c ssl=on -c ssl_cert_file=/cert.pem -c ssl_key_file=/key.pem
    ports:
      - "5432:5432"
    volumes:
      - ./server.crt:/cert.pem
      - ./server.key:/key.pem
    environment:
      - POSTGRES_USER=sensify
      - POSTGRES_PASSWORD=password

Even when installing that package you mentioned explicitly it seems to still be an issue
❯ docker run --rm -it cgr.dev/chainguard/wolfi-base:latest
705e3607b3c8:/# apk add --no-cache --quiet libcrypt1=2.38-r7 postgresql-15-client libpq-15

705e3607b3c8:/# apk list --installed | grep -E 'postgres|pq|crypt'
WARNING: opening from cache https://packages.wolfi.dev/os: No such file or directory
libcrypt1-2.38-r7 x86_64 {glibc} (GPL-3.0-or-later) [installed]
libcrypto3-3.2.0-r0 x86_64 {openssl} (Apache-2.0) [installed]
libpq-15-15.5-r3 x86_64 {postgresql-15} (BSD) [installed]
postgresql-15-client-15.5-r3 x86_64 {postgresql-15} (BSD) [installed]

705e3607b3c8:/# psql --host <IP_ADDRESS> --port 5432 -U sensify
psql: error: connection to server at "<IP_ADDRESS>", port 5432 failed: FATAL:  no PostgreSQL user name specified in startup packet
connection to server at "<IP_ADDRESS>", port 5432 failed: FATAL:  no PostgreSQL user name specified in startup packet
Segmentation fault (core dumped)

@camden-xage One of the other things I noticed diffing the good/bad digests you shared is that the one meaningful dependency change (other than epoch bumps) was openssl went from 3.1 to 3.2, so I am curious (given that this relates to crypto) whether you've tried pinning / downgrading to openssl 3.1?

This seems very similar to the issue here (h/t Natacha for the pointer): https://github.com/Homebrew/homebrew-core/issues/155651

Awesome.  Seems like we have our smoking gun.  Will sync up with our folks tomorrow to get the postgres patch in place, and the above turned into a test case 🤞

@camden-xage / @mattmoor Thanks guys! Nice catch with the openssl upgrade.
@camden-xage I guess this is not very relevant anymore, but the client was used to connect to a Azure Database for PostgreSQL flexible server.

hey folks, I believe the pieces are in place now to close this. since opening, we have:

pulled in openssl3.2 patches for postgresql-{12,13,14,15,16}

16: https://github.com/wolfi-dev/os/commit/d3d725648df8ac378bd55f38b7f2b20b40ab8ad1
15: https://github.com/wolfi-dev/os/commit/c2c94758f0b6ba343a909e0734765a68ff586e65
...

added some basic tests to the postgres image to ensure we do not regress on tls again

as the ticket is written (1) should be sufficient. @jasaltvik can you verify and close if so? if not please lmk what else we need 🙏

@joshrwolf Sorry for the late reply. Just verified that the client works now, thanks!


Replace internal proprietary API "Base64Util" class with "Apache Commons Codec 1.10 API"

Hello Eric!
Is possibel to replace internal proprietary API "Base64Util" class with "Apache Commons Codec API"?
I think you like "Apache API's" (well done!) and "Apache Commons Codec API" seems a good alternative to remove compillation warnings like:
SmartPutty\src\Dao\Base64Util.java:3: warning: BASE64Decoder is internal proprietary API and may be removed in a future release
import sun.misc.BASE64Decoder;
With this our code will be more stronger.
(Feel free to close when done)

Done,  removed Base64Util.java
replaced with commons-codec-1.9.jar


Writing

# [ALLAN'S TEC 105 PROJECT 2.docx](https://github.com/Owink/Beholder-/files/8179448/ALLAN.S.TEC.105.PROJECT.2.docx)

Is it great,,though I have begun


Automatic syncing?

After syncing local dir to drive, isn't there any way that if any changes are detected in local directory it automatically syncs the content to drive?

@amitsharma1729 Did you get the rightsuitable solution?


is it possible to visit the URL provided from a variable?

hi there !
is it possible to visit the URL provided from a variable?
example:
read //a[contains(text(),'Some text')]/@href to url
visit `url`

Hello @marcelocecin
Currently, visit is not part of the TagUI syntax so you cannot use it directly.
Here's a quick workaround you can look into, which is to pass the href link_text variable to a DOM step through a dom_json variable, and use window.location to go to the URL.
Here's an example of how it works:
https://github.com/kelaberetiv/TagUI/issues/936
read //a[text()="Security"]/@href to link_text
dom_json = {url: link_text}

dom begin
window.location= dom_json.url;
dom finish

perfect !
thanks @ruthtxh !

Adding on, below are 2 more examples how you can do it in 1 line.
TagUI detects whether a step is going to a webpage by checking if the step starts with https:// or http://
Below works by using the https:// keyword and then truncating away the https:// prefix in url variable.
url = 'https://google.com'
https://`url.split('//')[1]`

url = 'https://yahoo.com'
https://`url.replace('https://','').replace('http://','')

Adding on a video link with below demo - https://www.youtube.com/watch?v=qbXFouBC3Cg
url = 'https://google.com'
https://`url.split('//')[1]`
snap page

perfect !
thanks @kensoh !

hello @kensoh
is it possible to use this solution using a variable declared within a csv database?
thanks

Hi Marcelo! Possible, you can specify the datatable using tagui flow.tag datatable.csv.
Above error and file naming is probably you are referring to the old documentation.
For current doc - https://tagui.readthedocs.io/en/latest/advanced.html#datatables

hello Ken, I opened issue #953


[WIP] Initial support for class-values differentiation

Tagging @rxwei @BradLarson @dan-zheng
Currently there are some known issues mostly around derivative thunks. Tests were not updated as well.


[ja] Merge main into branch dev-ja

Describe your changes
Merge main into branch dev-ja to sync-up (This pr is like #2799)
Related issue number or link (ex: resolves #issue-number)
no issue related in this PR
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Hi @seokho-son, @jihoon-seo :)
PTAL and merge this PR!!


Post-processing of extracted text > structure lists

Hey team! Please add your planning poker estimate with ZenHub @eurolife @kdutia @opyate


Bump marathon to 1.6 dev

High-level description
Latest build of Marathon 1.6 Dev
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

MARATHON-7916 Release Latest Marathon to DCOS 1.11

Latest features

Simplify LeaderElection
Change Int.MaxValue to 8 for maximum concurrency
Introduction of Workload Bursting
Allow underscore for network names
Improved Reliability for Persistence Migration

Checklist for all PRs

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): https://github.com/mesosphere/marathon/compare/7501d6f...66a63ef
[x] Test Results: https://jenkins.mesosphere.com/service/jenkins/view/Marathon/job/marathon-pipelines/job/master/493/console
[x] Code Coverage (if available): None

@mesosphere-mergebot label Ready for review

@mesosphere-mergebot bump-ee

@orsenthil these failures do not seem related to the marathon bump
any help here?

@mesosphere-mergebot label merge-it

@mesosphere-mergebot merge-it


Checkin for the test harness interfaces

Interfaces created for the config reading and config setup creation steps
2. The operator interface for each task
3. The Configreader will read the config create a properties file and then ConfigStepsGenerator will generate individual steps based on the config.
4. Step is made of 1 or more setup operators.
5. Operator is individual task
6. ValidationRule will take Input and output and verify if the rule is passed or failed
7. TestHarnessExecutor launched individual test job based on the config

We have to add the license header to every new file created. For example, the build.gradle file has the proper license header. Can you copy that header to every other new file.

Can you also add a bit more detail into this Pull Request, about what each interface is responsible for, and how the interfaces are documented. It would be best if this is all documented.

I had a few thoughts about the interface layout. None of this is necessary, they are just some ideas I had:
A TestHarnessLauncher interface, it simply has a launchJob(Properties props) method, which takes in a Java Properties object. The TestHarnessLauncher is responsible for launching all the integration tests. The given props object contains global configuration properties. The props object also points to a directory containing the individual test files. The launchJob method takes all these test files, loads them into memory and creates a IntegrationTest for each one. Each IntegrationTest can composed of a series of SetupOperators, a single JobLauncher, and a series of ValidationOperators.

1: Is Validator still necessary now that ValidationRule is present? Same with Operator and SetupOperator
2: Can we remove the unnecessary dependencies from the build.gradle
3: Which interface is the Driver? What class gets invoked first? Is it ConfigReader?

@swami-veeramani if there are no other changes I can go ahead and merge the code.

Merged


RadioButton not working inside Toolbar

https://codesandbox.io/s/8768xzjp2
the problem can also been seen in the storybook:
http://react.carbondesignsystem.com/?selectedKind=Toolbar&selectedStory=Default&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel
The onChange is not fired when selecting the radio button in v6.82.0 onwards.
Was working correctly in v6.81.0

I think this may be related to #1805 as well


update rancher helm version to v2.10.0

Version: rancher/master
Need to bump up the helm version to v2.10.0, requires crd-hook for rancher catalog apps e.g, CRD operators,  and the latest version will address some common bugs of helm
install, https://github.com/helm/helm/releases/tag/v2.10.0.

using the latest version of helm v2.10 will only be required to remove -probe of the tiller, as it is removed on the tiller v2.10 version.

verified fixed and close it, master 09/07.


feat(#9): test renamed

closes #9

PR-Codex overview
This PR removes a dummy test from test_objects.py.
Detailed summary

Removed a dummy test from test_objects.py that was not needed.
Improved code quality by eliminating unnecessary test code.

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

@rultor merge

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 12min)


Multi-arg eachindex results in error

On both 0.5 and nightly:
x = rand(3)
y = OffsetArray(x, -length(x))
eachindex(x, y)

result on nightly:
ERROR: size not supported for arrays with indices (-2:0,); see http://docs.julialang.org/en/latest/devdocs/offset-arrays/
Stacktrace:
 [1] errmsg(::OffsetArrays.OffsetArray{Float64,1,Array{Float64,1}}) at /Users/twan/code/julia/MixedIntegerExperiments/v0.6/OffsetArrays/src/OffsetArrays.jl:48
 [2] eachindex(::Array{Float64,1}, ::OffsetArrays.OffsetArray{Float64,1,Array{Float64,1}}) at ./abstractarray.jl:798

Come to think of it, there isn't really a 'correct' output for this function, is there? Should it just throw an error?

This should definitely error. The only question is what the error message should be.

Thanks for the report. Quite agree, it should be an error.

This is fixed on current versions of Julia.


Organiser's checklist (logistics)

From ODD guide:

[x] A lot of spaces to plug in electronic devices or the ability to place more power strips out --> we have power strips
[x] A projector and screen
[ ] Dongles or adapters, because different devices need different plugs --> or just share from Naomi's macbook
[ ] Name badges or identifiers for people to know their abilities --> sticky labels? We have coloured sharpies
[ ] Snacks, water, etc --> see Catering #5
[ ] USB Drives --> bring a couple?
[x] Directional signage. Bathrooms, workshop rooms, etc --> not needed, small venue
[ ] Wi-Fi and the password for everyone --> details to be shared during intro
[x] Registration for your event --> share Eventbrite when ready
[x] Open Data Day logo

For venue:

[ ] print sign-up list and check in/out
[x] ensure all know where fire exits are and what to do in the event of a fire --> in intro talk
[x] make sure they know where the toilets are --> intro talk
[ ] No smoking allowed on site --> add to intro talk
[ ] request additional cleaning

I’ll revisit this tomorrow with venue specific things

Event is over! :)


Replace HTML representation of the NWB file with Neurosift

This PR uses an iframe to point the Neurosift application to the locally hosted stub file being previewed to provide a comprehensive look at the contents of the file.

Looks great! Slight delay while it launched but then was A-OK
One question maybe; is it possible for neurosift, or the way you've integrated it, to fail?
And if so, is it possible to catch that failure and instead fallback to the basic HTML display in that event (basically treating this like icing on the cake, but always offering at some some minimal representation of the output)

Actually storybook confirms what happens in event of failure I suppose: https://www.chromatic.com/test?appId=640a4d93e34604f275368983&id=64e38b652c12a1ee283afdde
but IDK if that's because the input to that page is not suitable for the story or what-not

As far as I can think, there is only one case where the Neurosift iframe will fail and we can still get the HTML representation: when there is a bug introduced on the Neurosift site itself that is undesirable.
All other cases (e.g. server down, file not created, etc.) will result in us being unable to get any representation of the file.

As mentioned in the last meeting, the clearest case where this could be useful is when internet is down—so we can't actually gain access to the Neurosift URL.


Adding unit tests for the C++ code and the official Lua plugins

[ ] for C++ code
[ ] for Lua plugins provided by MCHawk's core

As a C++ unit test framework, I suggest Catch2: https://github.com/catchorg/Catch2


Yarn pnp nodeLinker does not work

I am using pnp linker as follow:

Node Version: v20.10.0

# .yarnrc.yml
compressionLevel: mixed
defaultSemverRangePrefix: ''
enableGlobalCache: false
nodeLinker: pnp
yarnPath: .yarn/releases/yarn-4.0.2.cjs

I am using import-from-esm at https://github.com/straw-hat-team/nodejs-monorepo/tree/master/packages/%40straw-hat/openapi-web-sdk-generator
Here is the config:
# .openapi-web-sdk-generatorrc.yaml
generators:
  - path<EMAIL_ADDRESS>    config:
      outputDir: 'src/lib/services/flashbot-api/operations'
  - path<EMAIL_ADDRESS>    config:
      outputDir: 'src/lib/services/flashbot-api/react-query'
      packageName: '../operations'

And the package.json
{
  "packageManager"<EMAIL_ADDRESS>  "engines": {
    "node": "20.x"
  },
  "type": "module",
  "scripts": {
    "codegen:sdk": "sht-openapi-web-sdk-generator local --config='./openapi.json'"
  }
}

When I do DEBUG=* yarn codegen:sdk, I get the following outcome:
  @straw-hat/openapi-web-sdk-generator:helpers Configuration file /Users/ubi/Developer/myapp/.openapi-web-sdk-generatorrc.yaml loaded +0ms
  @straw-hat/openapi-web-sdk-generator:open-api-web-sdk-generator Loading Generator: @straw-hat/openapi-web-sdk-generator/dist/generators/fetcher/index.js +0ms
  import-from-esm Executing importFrom('/Users/ubi/Developer/myapp'<EMAIL_ADDRESS>+0ms
  import-from-esm<EMAIL_ADDRESS>is not a file module +0ms
  import-from-esm Trying to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js' +0ms
  import-from-esm Failed to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js': Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@straw-hat/openapi-web-sdk-generator' imported from /Users/ubi/Developer/myapp/noop.js +1ms
  import-from-esm Trying to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js' +0ms
  import-from-esm Failed to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js': Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@straw-hat/openapi-web-sdk-generator' imported from /Users/ubi/Developer/myapp/noop.js +1ms
  import-from-esm Trying to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js' +0ms
  import-from-esm Failed to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js': Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@straw-hat/openapi-web-sdk-generator' imported from /Users/ubi/Developer/myapp/noop.js +0ms
  import-from-esm Trying to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js' +0ms
  import-from-esm Failed to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js': Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@straw-hat/openapi-web-sdk-generator' imported from /Users/ubi/Developer/myapp/noop.js +0ms
  import-from-esm Trying to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js' +0ms
  import-from-esm Failed to resolve<EMAIL_ADDRESS>from 'file:///Users/ubi/Developer/myapp/noop.js': Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@straw-hat/openapi-web-sdk-generator' imported from /Users/ubi/Developer/myapp/noop.js +1ms
  import-from-esm Cannot find module<EMAIL_ADDRESS>+0ms
    Error: Failed to load configuration file @straw-hat/openapi-web-sdk-generator/dist/generators/fetcher/index.js.
    Cannot find module<EMAIL_ADDRESS>
If I change the linker to node-modules then it works just fine, so something is broken using pnp linker and this package.
Any thoughts?!

From https://yarnpkg.com/advanced/pnpapi:
Finally, it can be noted that findPnpApi isn't actually needed in most cases and we can do the same with just createRequire thanks to its resolve function:
const {createRequire} = require(`module`);

// We'll be able to inspect the dependencies of the module passed as first argument
const targetModule = process.argv[2];

const targetRequire = createRequire(targetModule);

const resolved = targetRequire.resolve(`eslint`);
const instance = targetRequire(resolved); // <-- still important

I found the issue,
function tryResolve(moduleId, baseURL) {
  debug(`Trying to resolve '${moduleId}' from '${baseURL.href}'`);
  try {
    // THIS WORKS
    return import.meta.resolve(moduleId, baseURL.href);
    // THE PROBLEM
    // return moduleResolve(moduleId, baseURL, new Set(['node', 'import']));
  } catch (error) {
    debug(`Failed to resolve '${moduleId}' from '${baseURL.href}': ${String(error)}`);
  }
}

I created https://github.com/sheerlox/import-from-esm/pull/81 to open the conversation, that solves the issue but I am not sure why it wasn't done that way to begin with, so I am gonna way to find more context about it

Related to https://github.com/wooorm/import-meta-resolve/issues/10
I am a bit confused, but the fact that native import.meta.resolve does work but the package does it, I feel the package is at fault here.
https://yarnpkg.com/getting-started/qa#does-yarn-support-esm

Resolved by #81.
The release is available on:

npm package (@alpha dist-tag)
GitHub release


⚠ Samsung has degraded performance

In b54ee89, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2728 ms

Resolved: Samsung performance has improved in 63ff42a.


[$250] Avatar, name and timestamp are greyed out in the system message for an IOU

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

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
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
Expensify/Expensify Issue URL:
Issue reported by: @rafecolton
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1716587435905939
Action Performed:

Create an IOU from user A to user B
As a user B pay the IOU with Pay with expensify
As a user A observe the IOU system message for "paid"

Expected Result:
Only message should be greyed out and not Avatar, name and timestamp
Actual Result:
Avatar, name and timestamp are greyed out in the system message for an IOU
Workaround:
unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @Pujan92
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d53d770ef7ce4cd3
Upwork Job ID:<PHONE_NUMBER>379024550
Last Price Increase: 2024-06-10

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@deetergp I believe you said you wanted to take this one? It looks like the message was pending, and the bug was having a pending message somehow even though the network was connected and the payment went through

This needs reproduction

This needs reproduction

Not able to reproduce

Thanks @kavimuru! Going to close this one.


Make OAuth2 open so other modules can subclass it

I'm making a custom OpenID Connect auth endpoint that's not unlike Google's authorization, but with custom login URLs (for Computer Science House).
I need to be able to subclass OAuth2 outside the TurnstileWeb module.

The Facebook test case is failing because there's no client ID set in the environment variable. Not sure how you want to proceed, given that. 🙃

Conversation on Slack:
Edward Jiang (Stormpath) [7:09 PM]
hey @harlan!

[7:09]
just saw your PR

Harlan Haskins [7:09 PM]
:smile:

Edward Jiang (Stormpath) [7:09 PM]
I haven’t thought too much about subclassing just yet & the scenarios where things could break if I made all of the public classes open instead

Harlan Haskins [7:10 PM]
I can definitely understand that

Edward Jiang (Stormpath) [7:10 PM]
haha Swift 3 defaults

[7:10]
make it more difficult for people consuming APIs

[7:10]
but I’m curious what you’re trying to do? from what I read you were trying to make an OpenID connect server

Harlan Haskins [7:10 PM]
Yep, but I think it promotes thoughtful discussion

Edward Jiang (Stormpath) [7:10 PM]
and the OAuth class is more for being an OAuth client (edited)

Harlan Haskins [7:10 PM]
I’m trying to interface with a Keycloak server that we just deployed

[7:10]
From a Vapor web service

Edward Jiang (Stormpath) [7:10 PM]
ah kk

[7:11]
so you’re trying to be an OpenID Connect client

Harlan Haskins [7:11 PM]
Yep!

Edward Jiang (Stormpath) [7:11 PM]
this is functionality that I’d like to see in Turnstile in the near future

Harlan Haskins [7:11 PM]
Me too :sweat_smile:

Edward Jiang (Stormpath) [7:11 PM]
so — instead of subclassing (for now, let me think about it a bit overnight)

[7:11]
would love to invite you to contribute and perhaps help write that functionality

Harlan Haskins [7:12 PM]
I don’t know anything about OpenID Connect beyond what I’ve just implemented locally, unfortunately. But I could always learn!

Edward Jiang (Stormpath) [7:12 PM]
=] that’s the spirit!

[7:13]
but the other way

[7:13]
you could do things w/o subclassing

[7:13]
is just embedding the OAuth class inside another class

Harlan Haskins [7:13 PM]
Yep, that would work fine

Edward Jiang (Stormpath) [7:14 PM]
ok

[7:14]
otherwise, if we make it open

[7:14]
I think we should do a general find -> replace of all `public` with `open`

Harlan Haskins [7:15 PM]
Well, for classes

Edward Jiang (Stormpath) [7:15 PM]
yeah

Harlan Haskins [7:16 PM]
Yep, so if we choose to allow subclassing, I’ll expand my PR

Edward Jiang (Stormpath) [7:19 PM]
sounds great! either way, would love to see OpenID connect, so if you end up building it in your fork, I’d love to pull it back upstream

Harlan Haskins [7:20 PM]
Awesome! I just saw how well it’s integrated into Vapor and I couldn’t pass it up!

I think I'm going to make the Facebook test disable itself if the environment variables aren't set


[Authentication] [Authorization] [Enterprise server] Update/improve diagrams

The diagrams in the Authentication/Authorization 2.0 and the Enterprise server section of the documentation need a review/upgrade.
Enterprise server: Add  a diagram explaining the flow/sequence of events when logging in to a cluster via the Enterprise server

Closing - This section has been updated multiple times to feature helm. Part of those updates has included customer eng feedback.


Build issue in 0.2.0

In 0.2.0, the file keyring_linux.go was renamed to keyring_unix.go to enable dbus on more platforms.
Unfortunately, the dbus implementation in godbus/dbus requires cgo for both dragonflybsd and freebsd. This means that if you are building with cgo disabled to build static binaries or for easier cross-compilation, upgrading to v0.2.0 results in build failures, with no way to disable this functionality.
One option would be to put a build tag constraint on cgo in keyring_unix.go to not compile in the code if cgo is disabled. That would probably require a copy of the file to be available for non-dragonflybsd/freebsd platforms that still compiles if cgo is disabled.

@jefferai What are the flags used for compiling? I'm trying to replicate the issue, but fail with this simple example:
package main

import (
    "log"

    "github.com/zalando/go-keyring"
)

func main() {
    service := "my-app"
    user := "anon"
    password := "secret"

    // set password
    err := keyring.Set(service, user, password)
    if err != nil {
        log.Fatal(err)
    }

    // get password
    secret, err := keyring.Get(service, user)
    if err != nil {
        log.Fatal(err)
    }

    log.Println(secret)
}

CGO_ENABLED=0 GOOS=linux go build
file go-keyring
go-keyring: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, Go BuildID=3YHhCPmk4UlD9_lXoVn1/N8KAugbp9D6MWwgxw4DH/ztCw71yTYMPVgjk8hI8h/yBdWR56JPIAmab6xbFtI, not stripped

@jefferai New version v0.2.1 addresses the issue.

Thank you!