Operator GRU does not accept input buffer at index 2

Here is the error I got when do input buffer binding for operator GRU:
D3D12 ERROR: : the dispatchable object expects nothing to be bound at index 2, but a binding of type DML_BINDING_TYPE_BUFFER was provided. Use binding type NONE to bind nothing to this slot. [ UNKNOWN ERROR #1: STRING_FROM_APPLICATION]
According to GRU description below, it need at least three inputs: data input, weight and recurrent. The BindInputs function seems accept tensor at index 0 (data input) and index 1 (weight), but not index 2 (recurrent). Are there any misunderstanding? Are there any sample code for GRU usage?
https://docs.microsoft.com/en-us/windows/win32/api/directml/ns-directml-dml_gru_operator_desc

Hi, the documentation is correct here. The first three inputs of DML_OPERATOR_GRU are always required. We're not aware of any issues with this - could you provide some more details about how you're calling the API? What are the exact contents of the DML_GRU_OPERATOR_DESC struct that you're sending down to IDMLDevice::CreateOperator? And are you sure your binding table is referencing the correct compiled operator?

Belows are partial code in my project. Are there any code sample for GRU? If exists, I can also study it.
Here is code for create GRU operator by using DirectMLX API dml::GRU
	`{
		Dimensions modelInputSizes = { 1, 1, 1, GRU_IN };
		auto modelInput = dml::InputTensor(graph, 0, dml::TensorDesc(dataType, modelInputSizes, policy));

		auto gruWeight = dml::InputTensor(graph, 1, dml::TensorDesc(dataType, DML_TENSOR_FLAG_NONE, { 1, 1, 3 * GRU_OUT, GRU_IN }, policy));
		auto gruRecurrence = dml::InputTensor(graph, 2, dml::TensorDesc(dataType, DML_TENSOR_FLAG_NONE, { 1, 1, 3 * GRU_OUT, GRU_OUT }, policy));
		auto gru1 = dml::GRU(
			modelInput,
			gruWeight,
			gruRecurrence,
			std::nullopt,
			std::nullopt,
			std::nullopt,
			{ dml::FusedActivation::Sigmoid(), dml::FusedActivation::Tanh() },
			DML_RECURRENT_NETWORK_DIRECTION_FORWARD,
			false,
			dml::GRUOutputOptions::Both);

		DML_EXECUTION_FLAGS executionFlags = DML_EXECUTION_FLAG_ALLOW_HALF_PRECISION_COMPUTATION;
		m_dmlGraph = graph.Compile(executionFlags, std::array<dml::Expression, 2>{ gru1.sequence, gru1.single });
	}

`
Here is the code for create the binding table
`DML_BINDING_PROPERTIES executeBindingProps = m_dmlGraph->GetBindingProperties();
tableDesc.Dispatchable = m_dmlGraph.Get();
tableDesc.SizeInDescriptors = executeBindingProps.RequiredDescriptorCount;
DX::ThrowIfFailed(m_dmlDevice->CreateBindingTable(&tableDesc, IID_PPV_ARGS(&m_dmlBindingTable)));

`
Here is the code call BindInputs. Error occurs if I change inputBindings[2] to non-empty binding.
`DML_BUFFER_BINDING bufferBindings[] =
{
	{}, // model input
	{ m_modelGruWeight.Get(), 0, m_modelGruWeight->GetDesc().Width },
	{ m_modelGruRecurrence.Get(), 0, m_modelGruRecurrence->GetDesc().Width },
};
bufferBindings[0] = DML_BUFFER_BINDING{ m_modelInput.Get() };
DML_BINDING_DESC inputBindings[] =
{
	{ DML_BINDING_TYPE_BUFFER, &bufferBindings[0] }, // model input
	{ DML_BINDING_TYPE_BUFFER, &bufferBindings[1] },
	{ DML_BINDING_TYPE_NONE, nullptr },
};
m_dmlBindingTable->BindInputs(ARRAYSIZE(inputBindings), inputBindings);

`

B.T.W, for current implementation of GRU expression in DirectMLX.h, if the 'hiddenInit' parameter is std::nullopt, 'hiddenInitTensor.sizes[3]' will be not access-able in the function. Is it a bug?
inline GRUOutputs GRU(         Expression input,         Expression weight,         Expression recurrence,         Optional<Expression> bias,         Optional<Expression> hiddenInit,         Optional<Expression> sequenceLengths,         Span<const FusedActivation> activationDescs,         DML_RECURRENT_NETWORK_DIRECTION direction,         bool linearBeforeReset,         GRUOutputOptions outputOptions)

Hi, we've looked into this a bit deeper and you're right - this appears to be a bug. Specifically, DML may incorrectly reject valid bindings in some cases when the RNN, LSTM, or GRU operators are used inside a DirectML graph (which is what DirectMLX is based on). We'll try to fix this for the next release of DirectML. In the meantime, one possible way to work around this issue is to call the GRU operator directly, i.e. as a standalone operator using DML_GRU_OPERATOR_DESC without going through DMLX.
And the uninitialized access of hiddenInit in DirectMLX.h does appear to be wrong - we'll need to fix that. Thanks for the bug report!


Add My Home Assistant link to HACS

And removed the note about not being in HACS by default (but the link doesn't care anyway, works both with default and custom).

@tjorim Thank you! Will merge


Alvin's edits for previously completed pages.

Docs Build status updates of commit 7e57f9b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-troubleshoot-makes-command.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/introduction.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md

Line 28: [Warning] Invalid file link: '../media/create-azure-storage-account.png'.

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

Docs Build status updates of commit ece9bf7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/how-to-troubleshoot-makes-command.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/introduction.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md

Line 28: [Warning] Invalid file link: '../media/create-azure-storage-account.png'.

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

Docs Build status updates of commit 1d24c31:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/TOC.yml
:warning:Warning

Details

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/introduction.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/resources-troubleshoot-guide.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/TOC.yml

Line 51: [Warning] The toc head name is missing

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md

Line 28: [Warning] Invalid file link: '../media/create-azure-storage-account.png'.

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

Docs Build status updates of commit bc48404:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/TOC.yml
:warning:Warning

Details

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/introduction.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/resources-troubleshoot-guide.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/TOC.yml

Line 51: [Warning] The toc head name is missing

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md

Line 28: [Warning] Invalid file link: '../media/create-azure-storage-account.png'.

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

Docs Build status updates of commit 5677953:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md
:warning:Warning

Details

academic-services/knowledge-exploration-service/TOC.yml
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/introduction.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/resources-troubleshoot-guide.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md

Line 28: [Warning] Invalid file link: '../media/create-azure-storage-account.png'.

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

Docs Build status updates of commit de2f67a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/TOC.yml
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-create-api-instances.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/get-started-setup-provisioning.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/introduction.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/media/api-release-location.png
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:white_check_mark:Succeeded

academic-services/knowledge-exploration-service/resources-troubleshoot-guide.md
:white_check_mark:Succeeded

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


Change the user cursor to pointer when user hovers on the wave button

On the talent and mentor pages, when we want to wave at someone, hovering over the wave icon should change the cursor to a pointer.
Steps to accomplish the task:

If you haven't done already, create a new branch that you'll make your changes on.
Navigate to the file, src/modules/main/ProfileCard.vue
Find the div that has the class "profile-card__wave".
Add "cursor__pointer" to the class.
Follow the steps to submit your PR.

I'm working on this, thanks


remove qtauto image from available images / reformat README

With b2qt being conditional in meta-pelux, we only show this image as
suggested image when we are building with b2qt support. This requires a
change in the manifest to copy that file.
Signed-off-by: Tobias Olausson<EMAIL_ADDRESS>

@GordanM separated the commits now.


Remove sring

Superseded by go/lib/ringbuf.

This change is

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


tests: Enable kernel.device test for qemu_x86_64

Reworked platform_whitelist to enable the "test kernel.device" for
qemu_x86_64
kernel.device.pm is not yet enabled for qemu_x86_64 because
there is a link error
undefined symbol `_DEVICE_STRUCT_SIZEOF'
referenced in expression
Signed-off-by: Cinly Ooi<EMAIL_ADDRESS>

Codecov Report

Merging #13812 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13812      +/-   ##
==========================================
+ Coverage   52.26%   52.26%   +<.01%
==========================================
  Files         307      307
  Lines       45410    45410
  Branches    10504    10504
==========================================
+ Hits        23733    23734       +1
  Misses      16887    16887
+ Partials     4790     4789       -1

Impacted Files
Coverage Δ

arch/posix/core/posix_core.c
91.91% <0%> (+1.01%)
:arrow_up:

Continue to review full report at Codecov.

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


Ignoring some files, untracking some files, tracking version

version.h shouldn't be tracked since it's autogenerated by CMake


Error with Unreal 5.1

Here's the full error:
See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.ArgumentException: String was not recognized as a valid Boolean.Couldn't store <fals> in celEnabledByDefault Column.  Expected type is Boolean. ---> System.FormatException: String was not recognized as a valid Boolean.
   at System.Boolean.Parse(String value)
   at System.String.System.IConvertible.ToBoolean(IFormatProvider provider)
   at System.Data.Common.BooleanStorage.Set(Int32 record, Object value)
   at System.Data.DataColumn.set_Item(Int32 record, Object value)
   --- End of inner exception stack trace ---
   at System.Data.DataColumn.set_Item(Int32 record, Object value)
   at System.Data.DataRow.set_Item(DataColumn column, Object value)
   at ReadUPlugin(String FileUPlugin, DataRow* mDataRow)
   at AddUPlugin(String FileUPlugin)
   at FindUPlugin(String Path)
   at Searching(String Path)
   at UEPluginsDisableDefault.AppForm.cmbUEFolder_SelectedIndexChanged(Object sender, EventArgs e)
   at System.Windows.Forms.ComboBox.OnSelectedIndexChanged(EventArgs e)
   at System.Windows.Forms.ComboBox.WmReflectCommand(Message& m)
   at System.Windows.Forms.ComboBox.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

************** Loaded Assemblies **************
mscorlib
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9105.0 built by: NET481REL1LAST_C
    CodeBase: file:///C:/Windows/Microsoft.NET/Framework64/v4.0.30319/mscorlib.dll
----------------------------------------
UEPlugins_DisableDefault
    Assembly Version: <IP_ADDRESS>
    Win32 Version:
    CodeBase: file:///D:/Downloads/UEPlugins_DisableDefault/UEPlugins_DisableDefault.exe
----------------------------------------
System.Windows.Forms
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9075.0 built by: NET481REL1LAST_C
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
----------------------------------------
System
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9065.0 built by: NET481REL1LAST_C
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
----------------------------------------
System.Drawing
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9032.0 built by: NET481REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
----------------------------------------
System.Configuration
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9032.0 built by: NET481REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
----------------------------------------
System.Core
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9105.0 built by: NET481REL1LAST_C
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
----------------------------------------
System.Xml
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9032.0 built by: NET481REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
----------------------------------------
System.Data
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9110.0 built by: NET481REL1LAST_B
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_64/System.Data/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Data.dll
----------------------------------------
System.Numerics
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9032.0 built by: NET481REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Numerics/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Numerics.dll
----------------------------------------
Accessibility
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.9032.0 built by: NET481REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/Accessibility/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Accessibility.dll
----------------------------------------

************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.

For example:

<configuration>
    <system.windows.forms jitDebugging="true" />
</configuration>

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

I tried to look for a missing "e" at the end of every "false" in the plugins folder, but I wasn't able to find anything.
I think your tool is really useful, thank you!
If you need any info, you just need to ask!

Thanks for your feedback and detailed log.
I tried here and got the same error, it was cause by Mutable.plugin* that have "EnableByDefault: false" as last line and my code expected to always have a comma after the value. I'm uploading the fixed code and executable.
(*: Unreal\UE_5.1\Engine\Plugins\Experimental\Mutable\Mutable.plugin)


Switch SideEffects to have any input be apart of their function

As an alternative to #3, we could perhaps rewrite side effects to use a form of fn as_listener(register: SideEffectRegistrar, _: ()) {}.
Then, you would do register((as_listener, ())), or register((effects::state, 1234), (effect2, ())).
Not sure which approach I like better. Current approach requires more code indentation (returning a closure), but this one is forced to one side effect argument at all times, and requires wrapping everything in tuples.

Too unergonomic for actual use. Easier to write side effects, but writing side effects is on the rarer side, especially for the Rust implementation.

I suppose we could also try to support an API like:
let ((state, set_state), rebuilder)= registrar
    .register(effects::state, 1234)
    .register(effects::rebuilder, ())
    .complete()

And then for one side effect we can add a convenience registrar.single(state, 1234).

We could also ship a proc macro for side effects that allows us to do:
let ((state, set_state), _) = register.state(1234).as_listener().complete();

To me, this is the best of all worlds. Won't require better-api feature gate, method chaining is declarative and readable, and GATs should no longer be an issue. Might take a crack at implementing this.

Changed my mind; this is too unergonomic and is hard to implement. It is still easier to just write out register.register((effect1(), effect2(1234))) on stable rust than this alternative is, and that API makes the impl far far easier than this one would be.


Dev mode run not working in Kubernetes deployment

Running project in dev mode doesn't work in Kubernetes deployment. If I click Run in developer mode button the project starts to run, but no integrations get found (visible in the Routes startup log entry) and the process hangs for tens of seconds. After some time it retries, but the results is the same.
Logs after running in dev mode:

If I click Reload button while the project is running the needed files (application.properties and all other integration related files) get created and the project starts running successfully. This is the only way to get the project to run in dev mode.
Logs after clicking reload ("Creating file" entries appear after clicking Reload):

@mgubaidullin to me it looks like that for Kubernetes deployment (in some cases) project files are not copied to the project pod when running in dev mode. They are copied successfully after a user clicks Reload button. Any ideas why this could be happening?
P.S.
In local docker depyloment dev mode works as expected, which points o the issue only somewhere being in the KuberentesService.

Any error messages in karavan-web pod?

Nothing in the logs, no. Sorry, forgot to mention that.

@mgubaidullin one more thing: I just connected to the project pod after I ran the dev mode and there was not even one file from the project in /karavan/code folder. This goes in line with my assumption that project file copying doesn't work as expected in some cases for Kubernetes deployment.

Closing this, since the issue was with the implementation of the Kubernetes deployment


feat: paginate subscribe fromId

What is the feature you would like to implement?
I think it might be valuable to have a pagination option when subscribing fromId
Why is this feature important?
I'm not sure if this is the best way to get the behavior I'm looking for, so here's an example:
Let's say I want a rolling list of all events on a hub. I also want to be able to scroll and see all the events that have ever happened on the hub. Currently the way to do this would be to subscribe and set the fromId to 0. However, this would result in a massive response. It might make sense to instead paginate the response that comes from loading all event.
Will the protocol spec need to be updated??
No
Additional context
Once again, this could be the wrong way to think about this. Feel free to offer another suggestion!

Hubs right now only have ~ 3 days of event data, after which they get pruned.  We could still add a pagination if this three day response is too large to handle, but is that what you are looking for?

@varunsrin Yeah, I think it would still be worthwhile to paginate that response

This is now available in the HTTP API


this application doesn't run

there is no executable code in this program. I can't make it do anything. :)

Please check the branch called ps-intro-to-node and look at the two new files as of commit c61154f. There is a servertimout.js file and a server.js file that you can run with node. I'm currently developing using node v5.8.0.


GraphQL requests from next-drupal do not work after GraphQL module update to 4.6.0

Package containing the bug
next-drupal (NPM package)
Describe the bug
After updating the GraphQL module to 4.6.0, next-drupal GraphQL requests no longer work.
The GraphQL 4.6.0 update included this set of changes which checks headers' Content-Type on requests.
GraphQL requests in next-drupal use "application/vnd.api+json" for the Content-Type which does not match the checks in the module.
This results in an error when trying to navigate to any NextJS page that uses drupal.fetch() from next-drupal to fetch data from Drupal's GraphQL:
⨯ lib/drupal.ts (37:10) @ query
 ⨯ Error: Bad Request
    at query (webpack-internal:///./lib/drupal.ts:28:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async getStaticProps (webpack-internal:///./pages/index.tsx:39:18) {
  page: '/'
}
  35 |
  36 |   if (!response?.ok) {
> 37 |     throw new Error(response.statusText)
     |          ^
  38 |   }
  39 |
  40 |   const { data, errors }: QueryJsonResponse<DataType> = await response.json()
 ✓ Compiled /_error in 203ms (497 modules)

The server-side (Drupal) error is:
Symfony\Component\HttpKernel\Exception\BadRequestHttpException: The content type 'application/vnd.api+json' is not supported. in Drupal\graphql\Routing\QueryRouteEnhancer->assertValidPostRequestHeaders() (line 104 of /app/drupal/web/modules/contrib/graphql/src/Routing/QueryRouteEnhancer.php).

Expected behavior
Requests succeed by passing the expected Content-Type for the GraphQL module.
Steps to reproduce:

Make successful GraphQL requests from a next-drupal site to a Drupal site running GraphQL module version 4.5
Update GraphQL to 4.6
Clear Drupal cache
Re-run the next-drupal server and make the same request
You will see the errors detailed above

Additional context
Drupal slack thread: https://drupal.slack.com/archives/C01E36BMU72/p1699467051864729

Fixed in #598


not referencing question variable

Issue #, if available:
question variable was not referenced. Thus, when looping through the questions list, it outputted the same answer as the question was a static string.
Description of changes:
Switch out the static string to referencing the question variable
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM @NateMoneyLion


Deprecate and remove sentry nodes

After a long discussion, we're going to deprecate and remove sentry nodes as the complexity they introduce for Polkadot is way too big at the moment.

Closing in favor of https://github.com/paritytech/substrate/issues/6845.


Refactor login page to use Polymer

hey mina, the login page is not working properly if viewed from cellphone, please fix the styles thanks

@habibmn09


Full-text search for docs

It would be great to have all the docs content indexed, being able to search full text, for example using https://www.algolia.com/

Agree.

Working on this.

@mhausenblas  - Done.


Add more customization options

Add ability to change icons
Allow changing colors of elements

This can be changed with css. If any other customization is wanted, feel free to create an issue.


putField definition

Every if statement in this function is inlined at compile time. The methods is effectively branchless.

I will add the transform for this soon. For now we can merge this :)


MoM tutorial into spanish

Translation for Valkyrie MoM tutorial into Spanish
Hi @scrubbless
I've translated your MoM tutorial into spanish, for the user's maybe you want to share it here.
https://docs.google.com/document/d/1yogKLaSEFFZvnJ0VdVDMKzohv2k0XPsHlVxW30qha8g/edit?usp=sharing
Also one extra question : Do you give me permission to disseminate the translated file? Obviously without any lucrative intention, just so that more people can do scenarios.
Thank you!
¿How do you see it @eorahil ?

I absolutely give you permission, didn't write it for credit. Just as long as it's free to all share away.

Thanks for your kinds comments at the end @EtiCarasoft :) I had to Google translate it ;)

I added it to the wiki https://github.com/NPBruce/valkyrie/wiki/ExternalSites


Same Node Retry Policy added

What this PR does:

Added a retry policy that retries on the same node in all scenarios. This is useful when we have a proxy like to cqlrouter to communicate with the backend.
The SameNodeRetryPolicy inherits DefaultRetryPolicy provided by the driver and overrides few error scenarios to retry on the same node where the DefaultRetryPolicy retries on the next available node.
This can be enabled by setting -Ddatastax-java-driver.advanced.retry-policy.class=io.stargate.sgv2.jsonapi.service.cqldriver.SameNodeRetryPolicy

Which issue(s) this PR fixes:
Fixes #648
Checklist

[X] Changes manually tested
[X] Automated Tests added/updated
[X] Documentation added/updated
[X] CLA Signed: DataStax CLA

The implementation of these methods are marked as deprecated on the RetryPolicy.java
https://github.com/apache/cassandra-java-driver/blob/8d5849cb38995b312f29314d18256c0c3e94cf07/core/src/main/java/com/datastax/oss/driver/api/core/retry/RetryPolicy.java#L184
But, the alternate method provided still uses the deprecated method internally
https://github.com/apache/cassandra-java-driver/blob/8d5849cb38995b312f29314d18256c0c3e94cf07/core/src/main/java/com/datastax/oss/driver/api/core/retry/RetryPolicy.java#L209
So, following the implementation in the DefaultRetryPolicy marking the overridden methods also as deprecated for now.

The implementation of these methods are marked as deprecated on the RetryPolicy.java https://github.com/apache/cassandra-java-driver/blob/8d5849cb38995b312f29314d18256c0c3e94cf07/core/src/main/java/com/datastax/oss/driver/api/core/retry/RetryPolicy.java#L184
But, the alternate method provided still uses the deprecated method internally https://github.com/apache/cassandra-java-driver/blob/8d5849cb38995b312f29314d18256c0c3e94cf07/core/src/main/java/com/datastax/oss/driver/api/core/retry/RetryPolicy.java#L209
So, following the implementation in the DefaultRetryPolicy marking the overridden methods also as deprecated for now.

That works, but is there a reason for not just overriding new, un-deprecated methods?
Presumably the idea of deprecation, new methods, is to let new code use new methods, without breaking backwards compatibility of old methods?

Closing this PR since this is already taken care in #750


Add tests for mixed_precision + variables

Changes:

Added a new test file, dtype_policy_test.py.
Enhanced variables_test.py with more tests and added a TODO list for untested functions.

Codecov Report
Patch coverage has no change and project coverage change: -8.36% :warning:

Comparison is base (29a954a) 77.40% compared to head (c837a76) 69.05%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #18483      +/-   ##
==========================================
- Coverage   77.40%   69.05%   -8.36%
==========================================
  Files         331      331
  Lines       31972    31972
  Branches     6241     6241
==========================================
- Hits        24749    22077    -2672
- Misses       5646     8416    +2770
+ Partials     1577     1479      -98

Flag
Coverage Δ

keras
69.01% <ø> (-8.31%)
:arrow_down:

keras-jax
?

keras-numpy
56.21% <ø> (+0.04%)
:arrow_up:

keras-tensorflow
?

keras-torch
64.01% <ø> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 65 files with indirect coverage changes

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

Hi @fchollet,
I believe now is the perfect time to gather your feedback on how you'd like me to organize my contributions.
I'm committed to increasing the test coverage to 100% and am really enjoying it.
To be honest, only two days ago, I learned how to work on multiple PRs without any conflicts between them.
But I've always wondered: Do you prefer short PRs or long ones? I'm new to GitHub, but I believe each PR should focus on one specific topic, correct?
I suggest that, moving forward, I begin with testing fundamental functions, like those in the backend and utility folders, ensuring they are 100% covered, one PR for one test file a time . After that, I'll move on to higher-level functions. If I discover a bug, I'll attempt to fix it if it's straightforward. If it's more complex, would you like me to open an issue in such cases?
Please feel free to be very tough with your code feedback; I'll learn more this way. I'm here to learn so I can teach my son in three years :)
Thank you.

@fchollet
Please see the failed tests and give me your feedback how to fix them.
Thank you.

Thanks for the update!

def test__pos__(self):

I think we can replace the implementation of def __pos__ in keras/backend/common/variables.py:Variable with just return self.value. Is there any context in which this would be incorrect?

def test__div__(self):

If this only occurs for the numpy backend, we can override def __div__ in keras/backend/numpy/core.py:Variable to use an implementation relying on np.divide.

round

If it's just TF, we can likewise override __round__ on the TF Variable class and use tf.math.round.

def test__lshift__(self):

Let's just outright remove all *shift methods on keras/backend/common/variables.py:Variable and remove the tests. They're not very useful.

@fchollet
I have removed all *shift methods and tests.
I have kept the others to see the test results.
Please check the failed test summary for each backend. We got different results. I need your feedback.
Thank you.

Thanks for the update!

def test__pos__(self):

I think we can replace the implementation of def __pos__ in keras/backend/common/variables.py:Variable with just return self.value. Is there any context in which this would be incorrect?

def test__div__(self):

If this only occurs for the numpy backend, we can override def __div__ in keras/backend/numpy/core.py:Variable to use an implementation relying on np.divide.

round

If it's just TF, we can likewise override __round__ on the TF Variable class and use tf.math.round.

def test__lshift__(self):

Let's just outright remove all *shift methods on keras/backend/common/variables.py:Variable and remove the tests. They're not very useful.

@fchollet
I have removed all *shift methods and tests.
I've retained the others to see the test results.
Please check the failed test summary for each backend. We got different results, and I need your feedback.
Thank you.
*Sorry, I closed it by mistake; I have reopened it again.

Thanks for the update! You can fix AttributeError: 'ResourceVariable' object has no attribute '__pos__' by returning self.value in __pos__ in the TF backend.
You can fix the others by outright removing the divmod, rdivmod, and round` method. They just aren't very useful (hence why the array class of most backends don't implement them).

Removed div and rdiv methods.
These methods were related to Python 2 division operations and are not supported in Python 3.

Done @fchollet
Thank you for your feedback.


Absolute paths are broken

Suppose I had a server running on port 50 serving the file index.html:
Broken link
In redbird I setup
var proxy = require('redbird')({port: 80, xfwd: false}); proxy.register("example.com/test", "<IP_ADDRESS>:50");
When I access example.com/test, it redirects correctly to index.html.
But when you click on the link it directs you to example.com/something.html, not example.com/test/something.html

There's no issue here. Your link's href starts with a slash, so it's relative to the root of the domain.

It is a mixed problem of redbird and the browser.
Redbird should be able to set up a trailing slash, otherwise the browser interprets it as a file, not a folder.
Thus, this also makes loading additional resources impossible. E.g.: example.com/test should redirect to example.com/test/ so path resolving is done correctly.


[Notice] Nav2 Migrated to TwistStamped to replace Twist for cmd_vel Topics

Hi,
As a key community project using Nav2, I wanted to drop you a note that I am in the process of changing the default cmd_vel topic type from Twist to TwistStamped in efforts of enabling a broader set of applications and safety constraints. This has been an on-going discussion over the last 18 months and now has the technology stack in GZ, ROS Control, and Nav2 to enable it.
To migrate, your robot and simulation should soon accept a TwistStamped message rather than a Twist message. We have a couple of provided utilities that could make this easier for you, by switching to use a Twist Publisher and Twist Subscriber [1-2]. This is what is now used in all Nav2 nodes to either publish or subscribe to a Twist or TwistStamped message.
If you would like to continue to use a Twist, you can set enable_stamped_cmd_vel to false in all nodes in your configuration that use Twist Publisher/Subscriber objects, but this will be shortly switched to being default true.
[1] https://github.com/ros-navigation/navigation2/blob/main/nav2_util/include/nav2_util/twist_publisher.hpp
[2] https://github.com/ros-navigation/navigation2/blob/main/nav2_util/include/nav2_util/twist_subscriber.hpp
Nav2 migration:

https://github.com/ros-navigation/navigation2/pull/4779
https://github.com/ros-navigation/docs.nav2.org/pull/614
https://github.com/ros-navigation/nav2_minimal_turtlebot_simulation/pull/16

I'm not 100% sure if any updates to RQT are needed. Can a maintainer on this metaproject give me some guidance?
I only see rqt_robot_steering, but only in ROS 1 (but there is a branch for stamped https://github.com/ros-visualization/rqt_robot_steering/commit/ad65f939abd75404dc55d92ae26c8cc516d9f05a)

This issue has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/notice-nav2-migrated-to-twiststamped-to-replace-twist-for-cmd-vel-topics/40944/1

As far as I can tell, there are no users of Twist in rqt other than rviz (which is the ROS 1 version), and rqt_robot_steering.  So I'm going to close this one out, and suggest that you open an individual issue on rqt_robot_steering, which does indeed have a ROS 2 port.


Unpublish achutkiran / elm-material-color /

Hi @evancz can you unpublish this package (achutkiran/elm-material-color/) as original developer updated package to elm 0.19. This forked package just creates confusion

Add line to https://github.com/elm/package.elm-lang.org/blob/master/migration/Migrate.hs#L85.


Device appears offline on the devtools

My Genymotion VM is correctly connected to adb and runs my app, but the connection with the chrome dev tools is switching from authenticated to offline every few seconds.
Note that if I click on inspect the "session" will be opened and it works properly.
Configuration:

Genymotion 2.5
Stetho 1.1.1
Chrome 44.0.2403.157 (64-bit)
Fedora 20

This is a bug in Chrome itself and I believe has been fixed upstream, though the fix is not yet present in stable.  It's a race condition and it's unclear how it can be mitigated or worked around though some folks have proposed solutions in #213.


Add hamaluik/timecop

ℹ Repository information
The repository has:

[x] At least three issues with the good first issue label.
[x] At least 10 contributors.
[x] Detailed setup instructions for the project.
[x] CONTRIBUTING.md
[x] Actively maintained.

Just wondering: Why is permission to act on my behalf for Vercel required in order to get my pull request approved? I'm quite hesitant about giving this kind of broad permission just to have a project listed here.

Hi @jai-deepsource, sorry to intrude here. But can you help me with https://github.com/deepsourcelabs/good-first-issue/pull/470?


Arp-spoof device-to-device traffic

Write the starter/untested code for @kumarde and he'll take over.

device-to-device arp spoofing can be implemented around here:
https://github.com/noise-lab/iot-inspector-client/blob/master/v2-src/arp_spoof.py#L77

Duplicate of #53

Checked device-to-device arp spoofing and it appeared to work.
Setup:

Macbook Air and Chromecast both running and both under inspection
I tried to access the Chromecast's port 8008 from the web browser on the Macbook

I checked the mongodb and found the corresponding traffic.
On Chromecast (s2609885b11), I found this flow:
> db.flows.find({device_id: 's2609885b11', 'device_port': 8008}).limit(1).pretty()
{
        "_id" : ObjectId("5e9231068b51a7745506271c"),
        "remote_tracker" : "",
        "protocol" : "tcp",
        "user_key" : "wme5jCMPBLMsXhK2",
        "syn_originator" : "remote",
        "remote_hostname" : "(Local Network)",
        "ts_mod600" :<PHONE_NUMBER>,
        "remote_port" : 49229,
        "ts" :<PHONE_NUMBER>,
        "ts_mod60" :<PHONE_NUMBER>,
        "remote_ip" : "<IP_ADDRESS>",
        "ts_min" :<PHONE_NUMBER>.7350612,
        "remote_ip_country" : "",
        "is_inspected" : true,
        "remote_web_xray" : "",
        "device_id" : "s2609885b11",
        "client_ts" :<PHONE_NUMBER>,
        "total_byte_count" : 1718,
        "remote_hostname_info_source" : "dns",
        "remote_reg_domain" : "(Local Network)",
        "device_port" : 8008,
        "in_byte_count" : 526,
        "ts_mod10" :<PHONE_NUMBER>,
        "ts_mod3600" :<PHONE_NUMBER>,
        "out_byte_count" : 1192
}

On Macbook Air, I found this flow:
> db.flows.find({device_id: 'sc0c9f6f8ac', 'remote_port': 8008}).limit(1).pretty()
{
        "_id" : ObjectId("5e9230e08b51a7598104755e"),
        "remote_tracker" : "",
        "protocol" : "tcp",
        "user_key" : "wme5jCMPBLMsXhK2",
        "syn_originator" : null,
        "remote_hostname" : "(Local Network)",
        "ts_mod600" :<PHONE_NUMBER>,
        "remote_port" : 8008,
        "ts" :<PHONE_NUMBER>,
        "ts_mod60" :<PHONE_NUMBER>,
        "remote_ip" : "<IP_ADDRESS>",
        "ts_min" :<PHONE_NUMBER>.7053795,
        "remote_ip_country" : "",
        "is_inspected" : true,
        "remote_web_xray" : "",
        "device_id" : "sc0c9f6f8ac",
        "client_ts" :<PHONE_NUMBER>,
        "total_byte_count" : 66,
        "remote_hostname_info_source" : "dns",
        "remote_reg_domain" : "(Local Network)",
        "device_port" : 49211,
        "in_byte_count" : 66,
        "ts_mod10" :<PHONE_NUMBER>,
        "ts_mod3600" :<PHONE_NUMBER>,
        "out_byte_count" : 0
}

/cc: @kumarde Let me know if this data is good enough for your purpose?

Yeah, this looks great!
On Sun, Apr 12, 2020, 3:51 PM Danny Y. Huang<EMAIL_ADDRESS>wrote:

Checked device-to-device arp spoofing and it appeared to work.
Setup:

Macbook Air and Chromecast both running and both under inspection
I tried to access the Chromecast's port 8008 from the web browser on
the Macbook

I checked the mongodb and found the corresponding traffic.
On Chromecast (s2609885b11), I found this flow:

db.flows.find({device_id: 's2609885b11', 'device_port': 8008}).limit(1).pretty()
{
"_id" : ObjectId("5e9231068b51a7745506271c"),
"remote_tracker" : "",
"protocol" : "tcp",
"user_key" : "wme5jCMPBLMsXhK2",
"syn_originator" : "remote",
"remote_hostname" : "(Local Network)",
"ts_mod600" :<PHONE_NUMBER>,
"remote_port" : 49229,
"ts" :<PHONE_NUMBER>,
"ts_mod60" :<PHONE_NUMBER>,
"remote_ip" : "<IP_ADDRESS>",
"ts_min" :<PHONE_NUMBER>.7350612,
"remote_ip_country" : "",
"is_inspected" : true,
"remote_web_xray" : "",
"device_id" : "s2609885b11",
"client_ts" :<PHONE_NUMBER>,
"total_byte_count" : 1718,
"remote_hostname_info_source" : "dns",
"remote_reg_domain" : "(Local Network)",
"device_port" : 8008,
"in_byte_count" : 526,
"ts_mod10" :<PHONE_NUMBER>,
"ts_mod3600" :<PHONE_NUMBER>,
"out_byte_count" : 1192
}

On Macbook Air, I found this flow:

db.flows.find({device_id: 'sc0c9f6f8ac', 'remote_port': 8008}).limit(1).pretty()
{
"_id" : ObjectId("5e9230e08b51a7598104755e"),
"remote_tracker" : "",
"protocol" : "tcp",
"user_key" : "wme5jCMPBLMsXhK2",
"syn_originator" : null,
"remote_hostname" : "(Local Network)",
"ts_mod600" :<PHONE_NUMBER>,
"remote_port" : 8008,
"ts" :<PHONE_NUMBER>,
"ts_mod60" :<PHONE_NUMBER>,
"remote_ip" : "<IP_ADDRESS>",
"ts_min" :<PHONE_NUMBER>.7053795,
"remote_ip_country" : "",
"is_inspected" : true,
"remote_web_xray" : "",
"device_id" : "sc0c9f6f8ac",
"client_ts" :<PHONE_NUMBER>,
"total_byte_count" : 66,
"remote_hostname_info_source" : "dns",
"remote_reg_domain" : "(Local Network)",
"device_port" : 49211,
"in_byte_count" : 66,
"ts_mod10" :<PHONE_NUMBER>,
"ts_mod3600" :<PHONE_NUMBER>,
"out_byte_count" : 0
}

/cc: @kumarde https://github.com/kumarde Let me know if this data is
good enough for your purpose?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/noise-lab/iot-inspector-client/issues/43#issuecomment-612667244,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXBQLK2XYL4CDESJ4NSAUDRMILSXANCNFSM4J4A5XAQ
.


Repaint/Redraw issue

I've been using dvtm + abduco for a couple weeks now and I love it, all except for one issue I can't seem to figure out.
When using dvtm + abduco, there seems to be a redrawing issue when using ncurses/TUI's. For instance, if I type a few commands, open up vim, then close it, the cursor moves to the top of the screen, but the whole page isn't repainted/redrew.
$xerxes> dvtm -v
dvtm-v0.15-37-gb45828d-dirty © 2007-2016 Marc André Tanner

$xerxes> abduco -v
abduco-0.6 © 2013-2016 Marc André Tanner

$xerxes> st -v
st 0.7 (c) 2010-2016 st engineers

I do know C, so I can help, but I've never worked on a code base for something like this so I'd be a little lost at first. More than happy to help though!
Example:
BEFORE
[1][2][3][4][5][ ][                                                                                ]
──[#1]──────────────────────────────────────────────────────────────────────────────────────────────
$xerxes> cd ~/Desktop/
$xerxes> touch file1
$xerxes> echo "Hello!"
Hello!
$xerxes> vim

AFTER
[1][2][3][4][5][ ][                                                                                ]
──[#1]──────────────────────────────────────────────────────────────────────────────────────────────
$xerxes> oh godesktop/
bash: oh: command not found
$xerxes> problem!llo!"
Hello!
$xerxes> vim

EDIT
Here's some more info, random stuff in case you need it.
$xerxes> uname -a
Linux xerxes 4.11.7-1-ARCH #1 SMP PREEMPT Sat Jun 24 09:07:09 CEST 2017 x86_64 GNU/Linux
$xerxes> screenfetch
                   -`
                  .o+`                 zane@xerxes
                 `ooo/                 OS: Arch Linux
                `+oooo:                Kernel: x86_64 Linux 4.11.7-1-ARCH
               `+oooooo:               Uptime: 36m
               -+oooooo+:              Packages: 928
             `/:-:++oooo+:             Shell: bash 4.4.12
            `/++++/+++++++:            Resolution: 1792x1008
           `/++++++++++++++:           WM: dwm
          `/+++ooooooooooooo/`         GTK Theme:  NumixSolarizedDark [GTK3]
         ./ooosssso++osssssso+`        CPU: Intel Core i7-4850HQ @ 8x 800MHz [45.0°C]
        .oossssso-````/ossssss+`       GPU: nouveaufb
       -osssssso.      :ssssssso.      RAM: 536MiB / 15960MiB
      :osssssss/        osssso+++.
     /ossssssss/        +ssssooo/-
   `/ossssso+/:-        -:/+osssso+-
  `+sso+:-`                 `.-/+oso:
 `++:.                           `-/+/
 .`                                 `/

Also, just to note, a CTRL+l (ell) does successfully repaint the screen to what it should look like.

Maybe it's a false alarm? In my ~/.bashrc file I had an export TERM=xterm-256color. I removed this, and dvtm defaulted to dvtm-256color, and now everything seems to be working as it should. If this is the cause, maybe it should be put in the readme?

After a couple days, it seems that setting the TERM environment variable to dvtm-256color fixed my issues. Hopefully it helps someone else out there.
Feel free to close this issue.

Actually, seeing a new issue with that TERM variable now. When I ssh into a machine that doesn't have dvtm installed and/or dvtm isn't running, most TUIs/ncurses UI's fail because they don't know what dvtm is. So I have to set the TERM variable to something like xterm-256color, which makes the TUIs/ncurses UIs work, but then causes the same issue as before with dvtm not painting/drawing correctly.
[root] super-secret-hostname:~ # less .bashrc
WARNING: terminal is not fully functional
.bashrc  (press RETURN)

For the record, I fixed the above issue by setting a bash alias that sets the TERM variable before running SSH. Problem solved. (In case anyone else needs this)


Added new jvm.memory.non_heap.* and jvm.fd.* JVM metrics

Marks the metrics added via elastic/apm-agent-java#3147 as internal.

@elastic/apm-server I'd need some one from you to merge. I don't have the permissions to merge PRs on this Repo.


Ensure xctest plugin behave properly on non macOS system

Context
See https://github.com/gradle/gradle-native/issues/128.
Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[x] Verify documentation
[x] Recognize contributor in release notes

@lacasseio I'll merge this Thursday


Example button for demo

To give demos it would be nice to have the ability to load an example from one of the scenario pages. Refine it in the workflow builder, show the resulting caprieval result, browse the module output and possibly re-run a module like rescore.
To prepare for the demo you would have the example already completed before you give the demo.
Describe alternatives you've considered
Manually doing tutorial like https://www.bonvinlab.org/education/HADDOCK3/HADDOCK3-protein-glycan/ , but reconstructing the workflow in the builder from scratch would take too long and high chance of mistakes.

In #155 working on demo where you need to copy files and residues list from a Markdown file.


'ports check' supports for multiply IPs

Hi,
First of all thank you for the nice and really fast tool!
I am trying to check ports using goss v0.0.16 and have a problems with multiply IPs identification:
me@host:~# cat goss.json
{
    "port": {
        "tcp:1443": {
            "listening": true,
            "ip": "<IP_ADDRESS>"
        }
    }
}
me@host:~# netstat -anp | grep 1443
tcp        0      0 <IP_ADDRESS>:1443       <IP_ADDRESS>:*               LISTEN      23894/nginx
tcp        0      0 <IP_ADDRESS>:1443       <IP_ADDRESS>:*               LISTEN      23894/nginx
tcp        0      0 <IP_ADDRESS>:1443        <IP_ADDRESS>:*               LISTEN      23894/nginx
tcp        0      0 <IP_ADDRESS>:1443        <IP_ADDRESS>:*               LISTEN      23894/nginx
tcp6       0      0 ::1:1443              :::*                    LISTEN      23894/nginx
me@host:~# goss -g ./goss.json v
.F

Failures:
Port: tcp:1443: ip: doesn't match, expect: [<IP_ADDRESS>] found: [<IP_ADDRESS>]

Total Duration: 0.009s
Count: 2, Failed: 1
me@host:~#

In the same time the following config works well:
me@host:~# cat goss.json
{
    "port": {
        "tcp:1443": {
            "listening": true,
            "ip": "<IP_ADDRESS>"
        }
    }
}

I believe goss should check all available IPs and doesn't fail on the first failed match. Isn't it?
BTW, it would be nice to define multiply IPs like:
{
    "port": {
        "tcp:1443": {
            "listening": true,
            "ip": [ "<IP_ADDRESS>" , "<IP_ADDRESS>" ]
        }
    }
}

Thank you!

I've attached a compiled copy from that pull request. I don't have a machine with multiple IPs configured and haven't had the time to create a test for this in docker.
Let me know if the attached version works properly for you and I'll cut a release with this feature.
goss-linux-amd64.zip
This new version is not backwards compatible with old goss.json files, run the following on your existing files to migrate them:
sed -ri 's/("ip": )(".*")/\1[\2]/' goss.json

Tested, briefly looks OK. The initial problem 'ip: doesn't match' has gone.
In same same time the new functionality "ip": [ "<IP_ADDRESS>" , "<IP_ADDRESS>" ] doesn't really work well.
The current logic is: SUCCESS if the first IP is OK,  FAIL if the first IP is NOT OK.
While expected logic IMHO: SUCCESS if ALL IPs are OK,  FAIL if some IP is NOT OK.
Maybe we need to have ability to specify match criteria ALL or ANY:
{
    "port": {
        "tcp:1443": {
            "listening": true,
            "ip": [ "<IP_ADDRESS>" , "<IP_ADDRESS>" ],
            "criteria": "ALL"
        }
    }
}

In the same time it is a way for complication the logic, so maybe you just have to dismiss that new feature request. K.I.S.S.

I've broken my post down into two sections, please let me know if I'm understanding you correctly for both parts.
My understanding of the IP issue
Here's the behavior I'm getting in my contrived test:
{
    "port": {
        "tcp:22": {
            "listening": true,
            "ip": [
                "<IP_ADDRESS>",
                "<IP_ADDRESS>"
            ]
        }
    }
}

That gives me count: 2 failed: 0
Changing one of the <IP_ADDRESS> to anything else causes a failure regardless of which one.
That said, if there were 10 IP's listening on that port, and I only list 2 in my goss.json, only those 2 would be tested and I would get success.
This is the way all the other goss tests work, user resource defining groups is "user is a member of X, Y groups" will succeed even if the user is in X, Y AND Z groups.
I'll cut a release later tonight, thank you for your feedback and testing.
My understanding of new feature
The criteria feature is something that I want to implement at some point, but it would have to affect all resources/attributes. The hard part for me is figuring out a good syntax that's easy to read but allows for the flexibility. Also deciding what to support.
Features, that might be useful:

ALL/ANY for array attributes
eq, gt, lt, ge, le for numeric. Ex: user.uid ge 1000 (ensuring user doesn't have an id lower than 1000)
Negation. ex: file.filetype != "file"

I'm guessing ALL/ANY is the most important one for now. I'll create a ticket to brainstorm this further if this is what you meant.

This is fixed in v0.0.17

See #31 for enhanced matcher logic.

As of v0.1.0 you can now do:
{
    "port": {
        "tcp:1443": {
            "listening": true,
            "ip": { "consists-of": [ "<IP_ADDRESS>" , "<IP_ADDRESS>" ] },
            "criteria": "ALL"
        }
    }
}

Nice tnx! I will try it.
Is there any place for goss release-notes/changelog I can read?
As I can see you have released 0.0.22 (and even 0.1.0), while I am still using 0.0.16 features list only.


Adds a better version of the unattended OTP reader for iOS and macOS for session extension

improved upon #149

On second thought, automating otp would re-question the legality of the repo and bring more unwanted attention. I would suggest to keep this in your own fork. One of the reasons why PR 149 wasn't merged.  We are trying as hard as possible to not play in gray areas.

On second thought, automating otp would re-question the legality of the repo and bring more unwanted attention. I would suggest to keep this in your own fork. One of the reasons why PR 149 wasn't merged. We are trying as hard as possible to not play in gray areas.

Ok sure got it. I thought it had some hardcoded logic which you needed to be fixed. But I agree with view posted here.


Replace aws-sdk in favor of aws-sdk-s3

There is no need to use all of these dependencies, since we already have this modular S3 gem

@schneems any interest in merge this? I can rebase if you want to

Apparently I can’t re-open a PR if the branch is deleted. Can you in-delete it? It’s a good change.

@schneems done

@schneems rebased with current master

Sorry for the delay. Thanks for the patch!


Memory leak when displaying and removing multiple image widgets sequentially

Versions

Python: 2.7.13
OS: Raspbian 9 (stretch)
Kivy: v1.10.1.dev0, git-8921b6a, 20180425
Kivy installation method: manual

Description
I was trying to make a slide show on my Raspberry Pi 3 with official touchscreen. I noticed that as I got further along in the slide show, images would stop appearing, displaying only black boxes. It seemed to be related to the total size of the images displayed. So I made the test below, with stops displaying jpg files after about image number 7. Using larger image files results in images not appearing sooner. I think the images should be released when the widgets are cleared, but they don't appear to be doing that. Memory usage seems to be increasing continually even though widgets are cleared, if the testing below is correct.
Code and Logs
import kivy
kivy.require('1.0.6')
from kivy.clock import Clock
import urllib
from kivy.app import App
from kivy.uix.floatlayout import FloatLayout
from kivy.uix.image import Image
from kivy.uix.label import Label
from functools import partial
from kivy.core.window import Window
import shutil
import time
import psutil
import os

def memcheck():
    # return the memory usage in MB
    process = psutil.Process() #os.getpid()
    mem = process.memory_info().rss / float(2 ** 20)
    mem = str(mem) + "MB"
    return mem

def displayphoto(i, root, *largs):
    name=str(i)+".jpg"
    root.clear_widgets()
    print("cleared widgets, memory at " + memcheck())
    print("photo " + name + ", adding widget")
    img = Image(source=name, nocache=False)
    root.add_widget(img)
    label = Label(text=name)
    root.add_widget(label)
    print("added widget, memory at " + memcheck())

class NoscatterApp(App):
    def build(self):
        root = FloatLayout()
        #os.remove("*.jpg")
        print("downloading photo...")
        urllib.urlretrieve("https://upload.wikimedia.org/wikipedia/commons/thumb/6/68/Shed_with_green_roof_at_Lyngen_fjord%2C_2012_June.jpg/1200px-Shed_with_green_roof_at_Lyngen_fjord%2C_2012_June.jpg", "main.jpg")
        print('done!')
        for i in range(0, 10):
            name=str(i)+".jpg"
            shutil.copy("main.jpg", name)
            Clock.schedule_once(partial(displayphoto, i, root), i*2)
        return root

    def on_pause(self):
        return True

if __name__ == '__main__':
    NoscatterApp().run()

Did you try manual garbage collection? As in import gc and after clear_widgets do gc.collect()? It doesn't seem like it should be needed, but still worth double-checking. If it's related to gl, maybe the script linked by tito here can be helpful in tracking it down further

Ok, that helps! It still doesn't display images after 7 just adding garbage collection, but when I also change nocache to True in the Image creation, it does work perfectly!

seems the cache is doing its job, you might want to experiment with the limit parameter of the Image cache, as by default, it doesn't try to check what fits in memory or not. I tried to do a size limit feature quite some time ago, but it turns out estimating the size of python object is not easy, so users had to provide the function to do that themselves, which didn't make it easy to use, anyway, that's not currently a supported feature.


Update QUICHE from 3373df94b to 6f0b1d204

Commit Message: Update QUICHE from 3373df94b to 6f0b1d204
Additional Description:
https://github.com/google/quiche/compare/3373df94b..6f0b1d204
$ git log 3373df94b..6f0b1d204 --date=short --no-merges --format="%ad %al %s"

2024-03-05 birenroy Uses absl::StrSplit in DecomposeRepresentations rather than a hand-rolled while loop.
2024-03-05 birenroy Refactoring to simplify future changes.
2024-03-05 vasilvv Introduce MoQT demo ingestion server that writes incoming streams into a directory.
2024-03-04 vasilvv Switch moqt_messages.h types to use std::string instead of absl::string_view
2024-03-04 awillia Replace deprecated version of absl::HexStringToBytes
2024-03-04 vasilvv Actually set MoqtSession::control_stream_ on the server side.
2024-03-04 vasilvv Redo the way callbacks are integrated into MoQT server.
2024-03-04 wangsteve Move quiche_feature_flags to actual file in Piper.

Risk Level: Low
Testing: Existing tests
Docs Changes: None
Release Notes: NA
Platform Specific Features: None

/restest

/retest

/wait

/retest


Flickering when scrolling down

Hey there,
I have a tree view displaysing a couple of hundred items.
When scrolling down to a certain position it begins to flicker like crazy.
Going up and down from empty space to some elements above.
When I manipulate the rowsInBlock or blockInCluster to where no scrolling would be needed, it just works fine. But that is not what I would like to do to fix it.
Can you help me out?
Thanks in advance!

Hello,
Any news regarding this issue? Have you managed to find a solution?
Thanks!

Hello,
Any news regarding this issue? Have you managed to find a solution?
Thanks!

Nothing unfortunately,

That's too bad...
Have you explored other react tree libraries by any chance?

Hi @nevermind10844
You can try to create an example on CodeSandbox that will always reproduce the flickering issue. This can help accerlerate the issue resolution time.

v1.17.0 might help if you can find a good option to control the number of rows and blocks to be rendered
See PR https://github.com/cheton/infinite-tree/pull/54


Add smoke test to ensure java -version can be run on JRE

Currently, we have smoke test for java -version check for JDK. I think we should add a smoke test to ensure java -version can be run on JRE.

@OscarQQ could you help with this issue? Thanks

PR for this issue: https://github.com/adoptium/temurin-build/pull/2675


update patch file

This pr introduces a fix for when RHTAP builds the rekor-server image, basically when RHTAP would try to build this Dockerfile, the pipeline was trying to pull an image called "deploy"  this fix changes the deploy image to use the ubi9 one used in the rest of the dockerfile, It also copy's everything from each stage so nothing is lost and the image builds in the same way it did before.

/cherrypick midstream-v1.2.2


Support for schemas with OpenAPI 3 validators

The new Validator associated type and FleeceValidator superclass constraint on the Fleece class allow extensions to validation beyond lifting Haskell functions.
I added a class called OpenApi3Validator to the json-fleece-openapi3 package that captures most of the OpenAPI 3 validations. Fleece instances can implement the OpenApi3Validator class on their associated Validator type to gain access to static information associated with the validations.
The intent is that downstream users will leverage the additional information to generate richer OpenAPI schemas based on their Fleece schemas.
There is a new constraint called FleeceOpenApi3 that includes the OpenApi3Validator constraint on the Fleece instance's Validator. It is exported by the json-fleece-openapi3 package, along with associated schemas that use the constraint.
The schemas serve as a drop-in replacement for those in Fleece.Core.Schemas. They use the methods on OpenApi3Validator to implement their validations.
I didn't add support for the pattern validation because I don't have good sense of what regex library to use.

I'm going to close this and make a new PR with just the changes to json-fleece-core for now.


Wallet label name should be populated with Organization name

[x] Wallet label name should be populated automatically in Text box from Organization name
[x] Agent running Status should appear after the Wallet Creation. " Will check the changes require and Implement"

Tested on DEV environment and changes look fine

Wallet label name populated automatically in Text box from Organization name

2.Agent running Status should appear after the Wallet Creation. " Will check the changes require and Implement"
https://github.com/credebl/studio/assets/133863994/8c16363d-8ff0-439d-813f-a5221e6c0fe1


why use stan instead of pymc3?

Hi folks! Great lib btw.
Is there any specific point for using stan? I think that pymc3 can perform the same calculations.
And already have some implementations for Bayesian tests. (https://docs.pymc.io/notebooks/BEST.html)
Thanks!

hi @marcosmarxm, thanks for reaching out!
No, there is no specific point of using stan. :)
We started with it three years ago. Back then our developer was most familiar with stan. And then, pymc3 and edward become more popular in the Bayesian community. We are not using the Bayesian functionality internally at the moment, so we haven't put into agenda to improve the implementation.


C++ program to find the number of bits to be flipped to convert one number to another

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
C++ program to find the number of bits to be flipped to convert one number to another
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #3724

@all-contributors bot please add @Harikrishnan6336 for code


Rounded corners lost when invalid feedback on input group

This is visible in the documentation too. Here is a sample snippet

      Username

          @

        <input type="text" class="form-control is-invalid" id="validationServerUsername" placeholder="Username" aria-describedby="inputGroupPrepend3" required>

          Please choose a username.

You will notice that the rounded corners of the input are lost on the right side. Due to the input not being the last-child

Here is a live demo: https://jsfiddle.net/g6wn5nrL/

https://github.com/twbs/bootstrap/issues/25110

Duplicate of #25110.


[QnA Maker] Update API parameters

Modifying the Parameter to be use absolute path instead of relative, this helps with the current Try-It bug in the UX.

You don't have permission to trigger SDK Automation.
Please add yourself to Azure group from opensource portal if you are MSFT employee,
or please ask reviewer to add comment *** /openapibot sdkautomation ***.
Please ask<EMAIL_ADDRESS>(or NullMDR in github) for additional help.

@yangyuan - Could  you please help review this PR? I'm not able to figure the Prettier validation bug, do you have any suggestion for that? Thanks!

Can one of the admins verify this patch?

@sarangan12 / @yangyuan  - could you please look into this. I'm not able to figure the Prettier validation bug, do you have any suggestion for that? Thanks!

"Endpoint" itself should contain "https://"

This is recommended by the Docs team ( @jdanyaw  ) , as the current UI has a bug with the TryIt feature in it.

These changes are not required anymore.


Confirm the Gisborne tiles we are to predict on

Task
With a trained model it is now time to predict buildings. But for which tiles?
The task is to ensure we understand which Gisborne tiles did not have buildings extracted from. This is as some cloud affected areas were delivered after buildings were extracted from the bulk of the Gisborne imagery.
Definition of Done

Once it is clear which tiles require buildings to be extracted from. This includes a list of tiles against the this issue.
Also ideally @jducnuigeen has reviewed the steps taken to arrive at the list of tiles that require building detection in Gisborne.

Out of Scope
BOP tiles. There may be some BOP tiles that require building extracting. For now we will focus on Gisborne as the model was trained with an emphasis on Gis.
It would be very insightful to predict on bop by

using the Gis model to understand how transferable learning is
Use the Gis model as a base and train on some Bop data to tune it to Bop.

Discussion Required

Things are not always clear with imagery. So here is my working step by step.

Above: Gisborne region. Green = Gisborne 0.3m Rural Aerial Photos Index Tiles (2017-2019). Purple = gisborne-03m-rural-aerial-photos-index-tiles-2017-2018.
Simple eh, but lets look at neighbouring imagery. Here is Bay of Plenty 0.3m Rural Aerial Photos Index Tiles (2016-2017) added.

And while we are at it - Bay of Plenty 0.3m Rural Aerial Photos Index Tiles (2019) (Crimson)

so... Assuming all other data sets but  Gisborne 0.3m Rural Aerial Photos Index Tiles (2017-2019) (green) have had buildings extracted looking at the above image we want to extract the remaining green tiles.
but... Those green Gisborne tiles covered by the crimson in image. xxx are from 2019 while the crimson BOP images are 2016-2017. Therefore do we extract for Gisborne were it overlaps with the older BOP images also to update BOP?

If I understand you correctly, yes run it on the green tiles, and then on the red. But due to the name change in the Gisborne dataset, I'm not sure how this affects the existing building outlines already captured. Have to discuss with Daniel.

An example of a Gsborne tile (BD44_5000_0602) that over laps with BOP and we are not predicting on.
This has a new building constructed since BOP building extraction.

Closing - Conversations with @jducnuigeen and @dwsilk confirm we will prioritise just the "gap"


Checked constructors for CsVec and CsMat

This is a first step in disallowing invalid structures from being created, to uphold the guarantees of sortedness of the structures.
This limits the surface where non-sortedness could appear by calling new_trusted. We could add debug assertations to new_trusted if we are paranoid.
Methods to test if invariants are upheld has been moved to the utils module. They could be useful for library users, and could be made public.

This should not be a breaking change.

Perfect then.


Add settings id

So other LSP Handler Plugins settings can be added to the same sublime menu.

Hahahahahahahah PR number #666

Coverage remained the same at 37.732% when pulling da8c95648a34e55e64dc3ede1c1f01f33dd3a831 on predragnikolic:add_settings_id into 84ef6894e6abcae2e1c753b1525fbb324cf0748c on tomv564:master.


fix: add missing moment dependency

Issue: I noticed that<EMAIL_ADDRESS>depended on react-datetime which assumes a moment peer dependency, but did not provide it. This caused an error when starting Storybook.
What I did
Add the missing dependency
How to test
Try starting a Storybook with the knobs addon installed before/after this change.
Is this testable with Jest or Chromatic screenshots? No
Does this need a new example in the kitchen sink apps? No
Does this need an update to the documentation? No
If your answer is yes to any of these, please make sure to include it in your PR.
For maintainers only: Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

I agree that it's a peerDependency, but it's a peer dependency of react-datetime, not addon-knbos. It doesn't make sense that a user of addon-knobs would care at all about moment or the version installed.

Here's the problem:
— We depend on A@2 which has a peer dependency on B@3
— We add a direct dependency on B@3 to satisfy that
— User app has a direct dependency on B@2 and doesn't have any dependency on A
— npm hoists A@2 to root node_modules, but leaves B@3 under @storybook/*/node_modules to avoid clashing with B@2
— inside A@2, import 'B' resolves to B@2 instead of the expected B@3, which leads to an error
In this case, A is react-datetime, and B is moment. If user has some incompatible version of moment, the addon will break


Autocomplete: Improve menu behavior

Description

Follow up on #8758.
New API, Refactor, Prevent race conditions, Fix menu flashing.
Several tests were updated to handle the new code.
Improvements to API & Behaviour

Replaces ChangeMenuAsync with clearly defined OpenMenuAsync and CloseMenuAsync and makes them public
Adds classes to input adornment icon and clear button
Enables holding arrow keys to navigate between menu items (KeyDown instead of KeyUp)
Renames SelectOnClick to SelectOnActivation for clarity
Modifies KeyUp/KeyDown handlers to have protected access instead of internal
Closing menu now cancels search
Escape key will blur so you can interact with the rest of the page (Double escape to close dialog)

Resolved Issues

Prevented multiple menu openings on repeated clicks during search
Eliminated additional menu open after progress indicators if user clicks off
Fixed menu flashing issue in Firefox caused by onclick and onfocus conflict
Ensured menu closes after selecting an item

How Has This Been Tested?

unit
Type of Changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[X] I've added relevant tests.

Please set a breakpoint and find out what the exception and the stacktrace are and post it here.

@henon

''<' is an invalid start of a value. Path: $ | LineNumber: 0 | BytePositionInLine: 0.'
'   at System.Text.Json.ThrowHelper.ReThrowWithPath(ReadStack& state, JsonReaderException ex)\n   at System.Text.Json.Serialization.JsonConverter`1[[System.Collections.Generic.List`1[[MudBlazor.Examples.Data.Models.Element, MudBlazor.Examples.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]], System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)\n   at System.Text.Jso…xample.Search(String value, CancellationToken token) in C:\Users\danie\source\repos\MudBlazor\src\MudBlazor.Docs\Pages\Components\Autocomplete\Examples\AutocompleteStrictFalseExample.razor:line 25\n   at MudBlazor.MudAutocomplete`1.d__188[[MudBlazor.Examples.Data.Models.Element, MudBlazor.Examples.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].MoveNext() in C:\Users\danie\source\repos\MudBlazor\src\MudBlazor\Components\Autocomplete\MudAutocomplete.razor.cs:line 649'

@henon @ScarletKuro Why does the SearchFunc throw an exception here

Sorry, but I really don't know. I have no codebase knowledge on how this component functions at slightest. @henon @mckaragoz has more experience with this component.
It's also hard for me to adequately review this, you changed many places and this component code resembles delicate paper, susceptible to crumbling with the slightest disturbance :flushed: Because it has many race conditions and my async void -> Task change is a proof of this. I appropriate your effort on trying to fix it tho, but almost feels like it would be better to make a parallel component called "MudAutocompleteX" and do minimum at least for the mudblazor docs search, battle test it there and then extend the functionality.

@ScarletKuro I think I've definitely hardened the code and made it less brittle and susceptible to race conditions. It does need some heavy testing though. I would hate to discard these changes from the BCL because it has a lot of improvements for all users.

The exception means that the json coming from the web api is invalid. For whatever reason. I can't see why your changes would cause that

Maybe you can find out what the api call returns that can't be parsed?

Maybe you can find out what the api call returns that can't be parsed?

Probably the result is returned as plain text (server error) while json is expected.

Does it even work without your changes?

Does it even work without your changes?

@henon Works on https://dev.mudblazor.com/components/autocomplete#strict-mode

@henon

''<' is an invalid start of a value. Path: $ | LineNumber: 0 | BytePositionInLine: 0.'
'   at System.Text.Json.ThrowHelper.ReThrowWithPath(ReadStack& state, JsonReaderException ex)\n   at System.Text.Json.Serialization.JsonConverter1[[System.Collections.Generic.List1[[MudBlazor.Examples.Data.Models.Element, MudBlazor.Examples.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]], System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)\n   at System.Text.Jso…xample.Search(String value, CancellationToken token) in C:\Users\danie\source\repos\MudBlazor\src\MudBlazor.Docs\Pages\Components\Autocomplete\Examples\AutocompleteStrictFalseExample.razor:line 25\n   at MudBlazor.MudAutocomplete`1.d__188[[MudBlazor.Examples.Data.Models.Element, MudBlazor.Examples.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].MoveNext() in C:\Users\danie\source\repos\MudBlazor\src\MudBlazor\Components\Autocomplete\MudAutocomplete.razor.cs:line 649'

@danielchalmers I've seen this error before when I expect JSON but get HTML, such as when I forget to authenticate an HttpClient request.   If we can get the raw response from the HttpClient, then we can look at the actual response content and dig deeper.  Here's some temporary code which should do the trick:
  private async Task<IEnumerable<Element>> Search(string value, CancellationToken token)
    {
        // return await httpClient.GetFromJsonAsync<List<Element>>($"webapi/periodictable/{value}", token);
        var response = await httpClient.GetAsync($"webapi/periodictable/{value}", token);
        var content = await response.Content.ReadAsStringAsync();
        Debugger.Break(); // Examine the content variable here for issues, maybe an auth error?
        return JsonSerializer.Deserialize<List<Element>>(content);
    }

@jperson2000 Thanks for the great tip, it made me discover that the webapi returns 404, so I guess it's not available while debugging WASM and was never a regression @henon
The rest was just some weird logic that should be fixed and ready to review now.

Does it even work without your changes?

@henon Works on https://dev.mudblazor.com/components/autocomplete#strict-mode

Yes, but try to execute it locally instead from dev.mudblazor.ocm. It may well be the cause of the error.

Thanks for the great tip, it made me discover that the webapi returns 404

Are you starting docs as wasm standalone? There is WasmHost then the API should be available.

The Strict parameter and example troubles me. I can't think of a use-case where you want the Strict="true" behavior because it will show an empty dropdown when clicking the autocomplete which is just nonsense. I remember that this behavior was the default in the beginning. We added the parameter to avoid a breaking change because users wanted the Strict="false" behavior instead.
Unless somebody can think of a valid use-case for Strict="true" I would now consider taking the opportunity and making the breaking change of just removing this parameter.

Thanks for the great tip, it made me discover that the webapi returns 404

Are you starting docs as wasm standalone? There is WasmHost then the API should be available.

Yes this.  wasm standalone doesnt have any mvc controllers to provide the endpoints for the examples data.

@danielchalmers We still want to retain Strict==false, we just want to remove the undesired Strict==true behavior that results in empty lists.

@Mr-Technician @ScarletKuro Looks good?


Bug-fix in Filter and Limit statistics

Which issue does this PR close?

Related to #8078.
Rationale for this change

After enum Precision is introduced in DF, some bugs are discovered. This PR resolves 3 bugs:

singleton check for ColumnStatistics,
limit in datasource with num rows statistics,
min - max exactness of columns with constant value.

What changes are included in this PR?

1st fix: A column is labeled as singleton only if its min and max values are exact and equal.
2nd fix: To stop processing, only exact count of rows is regarded. Otherwise, we should continue to process until range estimation of precision implemented.
3rd fix: During the analysis in filter statistics, if a column is filtered with a constant value (e.g. c=1), we set its min and max values as exact.
Are these changes tested?

Are there any user-facing changes?

Given the subtlety that may be involved here I think we should have test for these changes. Given that no existing test breaks, that suggests to me that it isn't sufficiently covered 🤔

I have added tests covering the 1st and 3rd cases, but can't find a test suite for the 2nd case that I can demonstrate the need for the fix. Do you have any advice for me to do that easily?


Automatized tests

Hello @gustavooquinteiro! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file setup.py:

Line 12:10: E211 whitespace before '('
Line 20:1: W293 blank line contains whitespace
Line 35:17: E231 missing whitespace after ':'
Line 38:9: E131 continuation line unaligned for hanging indent
Line 40:1: W293 blank line contains whitespace
Line 40:1: W391 blank line at end of file

In the file tests/test-generator.py:

Line 4:1: E302 expected 2 blank lines, found 0
Line 5:46: E231 missing whitespace after ','
Line 10:1: W293 blank line contains whitespace
Line 12:36: W291 trailing whitespace
Line 13:45: W291 trailing whitespace
Line 20:50: W291 trailing whitespace
Line 23:57: W291 trailing whitespace
Line 24:33: E128 continuation line under-indented for visual indent
Line 28:33: E127 continuation line over-indented for visual indent
Line 30:71: W291 trailing whitespace
Line 32:1: W293 blank line contains whitespace
Line 35:1: E303 too many blank lines (3)
Line 39:80: E501 line too long (167 > 79 characters)
Line 40:18: E713 test for membership should be 'not in'
Line 41:10: W291 trailing whitespace
Line 48:1: W293 blank line contains whitespace
Line 52:1: W293 blank line contains whitespace


Component Error

Hello, when i use webrtc on change “My Device” on my application , have this problem “Component Error”. This problem wos testing on other browsers. Help help please :slight_smile:
Streamlit, version 1.10.0
Streamlit-webrtc  0.42.0

`import streamlit as st
from streamlit_webrtc import webrtc_streamer
st.title("My first Streamlit app")
st.write("Hello, world")
webrtc_streamer(key="example")`

when i use webrtc on change “My Device” on my application
What does it mean...?

And the error logs on the console may also help.


Application of EPro-PnP？

I wonder if EPro-PnP can only be applied to one-stage networks, and if it can be applied to some two-stage pose estimation networks, such as PVNet.

Applying EPro-PnP on two-stage networks could be tricky, unless you could make it end-to-end differentiable (e.g., differentiable pixel voting in PVNet).


python311Packages.xdg: fetch from PyPI instead of GitHub

the github source uses xdg_base_dirs, but pypi's sdist maintains xdg for compatibility. there are actually breaking changes in xdg_base_dirs, and libraries that want to support python 3.10 and below need to use xdg.
for example, https://github.com/Textualize/frogmouth/pull/59#issuecomment-1585504373
Because pythonImportsCheck was not set, hydra cannot detect a break in xdg itself, but it causes downstream packages to break. e.g. #267385
ZHF: https://github.com/NixOS/nixpkgs/issues/265948

python311Packages.rmcl: https://hydra.nixos.org/build/238937793
python311Pacakges.rmrl: https://hydra.nixos.org/build/238900130
rmfuse: https://hydra.nixos.org/build/238986343

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 267436 at fa18ab54 run on aarch64-darwin 1

3 packages marked as broken and skipped:

bitwarden-menu
python310Packages.rmcl
python311Packages.rmcl

4 packages built successfully:

python310Packages.rmrl
python310Packages.xdg
python311Packages.rmrl
python311Packages.xdg

What we want here is xdg, not xdg-base-dirs

What we want here is xdg, not xdg-base-dirs

seems reasonable. xdg-base-dirs should be a new package then.


[WIP] add coverage test in pfnci

add coverage test in pfnci

@Hakuyume we cannot calculate correct coverage because pfnci do CPU and GPU test separetely.

move to #881


add dependency for arch-based systems

for manjaro (based on arch linux), I was getting this error after building & install atom: /usr/local/share/atom/atom: error while loading shared libraries: libgconf-2.so.4: cannot open shared object file: No such file or directory
after running sudo pacman -S gconf, i could open atom. hope this is of use to others.

@kevinsawicki got this on Ubuntu 14.04:
~ » atom                                                           eos@eos-gaz9
/opt/atom/atom: error while loading shared libraries: libgconf-2.so.4: cannot open shared object file: No such file or directory


Continuous Video

Hi,
Thank you very much for you code, its working great! Do you know if its possible to record continuous video using the code, similar to a real camera? I know that you can alter the duration of the recording but if there is a way to stop/start recording during a stream or onto an SD card that would be great.
Thank you!

Update the sample programme:
https://github.com/limengdu/SeeedStudio-XIAO-ESP32S3-Sense-camera/tree/main/record_video_decorder/ESP32-CAM_MJPEG2SD


Determine webpack version and possibly upgrade

In our discussion the question came up which version of webpack we are using. For example, in PP we had various issues moving from v3 to v4. So, which version of webpack are we using? If we are using v3, we should move to v4. If we are on v4, it is probably the best to first establish the testing framework and then tackle the build process (and change the test framework again, if necessary).

Our current webpack version: 5.53.0
Most current webpack version: 5.75.0

OK, so we are on version 5. As far as I am concerned, this can stay as is.


Vd89:- Added the document

added the initial
added the few changes in the readme file

Thanks for the approval


Changed count query for PG stats

This should be considered a fix. I've put a count(*) that is a very expensive. With this PR it's changed to use the PG statistics.
This PR fixes issue #4478

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Please @juanignaciosl CR this PR (its related to other you reviewed). Thank you.

Frontend tests were OK :+1: (details)

All changes done. Thank you @juanignaciosl


Issue with HSA_XNACK

Occasionally, HSA_XNACK forces some page faults, and memory may not initialize as expected in specific cases.
This issue is under investigation and will be fixed in a future release.

Hi @Rmalavally, does this issue still exist in latest ROCm 6.0.2?  Thanks.

Closing the issue as no response from @Rmalavally.  Please re-open if this issue still exists with latest ROCm 6.0.2.  Thanks.


add some kind of MIDI feedback protection for S-1

I noticed with the Roland S-1 synth that if you send it a MIDI noteon, it sends one right back. I'm not sure if this is widespread but the MIDI sender could have some sort of switchable feedback protection to stop this happening (maybe)

Fixed in 18eb5fa
The feedback protection works by checking for each MIDI input if:

it's less than 20ms since the last midi output
AND it's the same note as the last note_on from that channel.

It's configurable in the MIDI block of configs with two keys:
feedback_protection = true
and
feedback_threshold = 0.02
By default it is not enabled.


Make scripts into extensions

This pull request turns the scripts like Semantic-MathML.js into actual extensions, so that they can be loaded by MathJax using its synchronization process for extensions.  This should resolve the remaining timing issue from #58, without requiring any chances to SRE.  The Semantic-MathML extension now forces the loading of SRE, and the Semantic-Collapse forces loading of Semantic-MathML (so that users don't have to know the internal dependancies).  I've also added a custom path ([RespEq]) so that the extensions can be loaded via [RespEq]/Semantic-Math, and so on.  Similarly, I added [SRE] for where the SRE is loaded by Semantic-MathML.  If it is not specified by the page, then the current progressiveaccess.com address is used, but this allows it to be loaded locally or from the third-party repository if we want.
I've also reorganized the structure of the Semantic-Lab directory to separate out the extensions from the examples from the labs themselves, and renamed the labs to make their names less redundant.  I fixed up the *-linebreak labs to work with the current versions of the extensions (though I'm not sure why they seem to be deleted and re-added below, as I renamed them just like I did the others).
The functionality of the Semantic-Action-* scripts have previously been moved to the SRE, so I've removed those files.
Let me know if you have any problems with the new organization.  Otherwise, I'll merge the changes in a day or two.

There is now a problem with the walker demo that it does not switch of collapse.

OK, give it a try now.  It should be disabled now.

LGTM.

I hit the wrong button!


Rename TargetPlatform.ios to ios_arm64

Pre-factoring for the addition of ios_armv7.

@cbracken can this land?

Dropping this for now as I expect I can avoid it.


fix: avoid division by zero when calculating statistics

This commit fixes an edge case where an empty directory is provided. In that case, dirEntries and totalBytes will be zero.
How to reproduce:
$ mkdir emptydir
$ find-duplicates --statistics ./emptydir
{}
json: unsupported value: NaN

Perfect, thank you very much!


What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #2128
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/approve
/retest-required

/lgtm

/retest

/retest

/cherry-pick release-4.12


feat: improve help text for all commands

:gift: Standardize all help text to be filtered through a prefix/name handling function which can deal with kn vs kn func vs kn-func. Work in progress.
Improve this PR to add

[ ] code documentation
[ ] standard text width for all command help text (I prefer 70)
[ ] move additional functions in cmd/root.go to cmd/help-helper.go or similar
[ ] implications with lack of fully formed formatting capabilities (e.g.) for features in cobra we don't use
[ ] address kn-func as a plugin
[ ] address kn-func as a standalone binary

/kind enhancement
Fixes: https://github.com/knative-sandbox/kn-plugin-func/issues/216

The command help text preprocessing ('func' vs 'kn func' template) aspect of this was handled separately, and once the PR is accepted, I can merge it (and the other nontrivial updates that have happened to the cmd package recently) in, leaving this PR to provide the updates to the help text itself.

I think this needs to be redone from the start, or perhaps isn't even necessary any longer. Closing


Moved existing unit tests to standard folder structure

Moved them in line with the other DSC modules.
VSCode also appears to have tidied up all the random whitespace which is screwing up the diff.

This change is

Labeling this PR as abandoned since it has gone 14 days or more since the last update. An abandoned PR can be continued by another contributor. The abandoned label will be removed if work on the PR is taken up again.

Hey @ChrisLGardner any chance you could fix that one? It could do with a rebase as well :)

Codecov Report

Merging #15 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #15   +/-   ##
=======================================
  Coverage   69.23%   69.23%
=======================================
  Files           2        2
  Lines          65       65
=======================================
  Hits           45       45
  Misses         20       20

Continue to review full report at Codecov.

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

Rebase done. I'll find some time to fix the way it's importing the module etc to use the standard format and make use of the standard tests etc.


Use WriteableBitmap to improve performance of WPF rendering

Added use of a WriteableBitmap to avoid unneeded allocations and rebinding the image element source when the backend bitmap is only updated (not changed)
Purpose:
Improves performance of WPF rendering and fixes https://github.com/ScottPlot/ScottPlot/issues/1387.

OK I cleaned it up - feel free to squash the commits before merge or let me know if you want me to.

We don't normally squash PRs into one commit, so I think that's ok.

Also, it's worth noting that this enhancement could likely be made to WinForms and Avalonia. Non-Windows platforms don't have System.Windows.Media.Imaging.WritableBitmap, but Avalonia seems to offer an alternative.

Non-Windows platforms don't have System.Windows.Media.Imaging.WritableBitmap, but Avalonia seems to offer an alternative.

I don't want to reinvent the wheel here... but is this something we could implement ourselves? The BMP file format isn't that bad, and an obligate writer could be more straightforward to implement than a reader that would have to support multiple/nuanced file structures and header types.

If you're interested that could become part of this or a later PR, or we can open an issue for someone to do that later.

I'm definitely interested in exploring this idea for the other two controls! A separate PR/issue sounds like a good idea because I see the warning sighs of a rabbit hole 😉

I'm going to test this now and hopefully merge it in tonight... Thanks @jbuckmccready for this PR, and @bclehmann for your input! 🚀

@swharden: but is this something we could implement ourselves?

@jbuckmccready: I didn't want to affect the backend API

I should have clarified https://github.com/ScottPlot/ScottPlot/pull/1388#issuecomment-948105381 was directed toward @bclehmann, and implementing an in-memory bitmap writer could be a goal for another issue/PR. This PR is looking great, and it certainly makes sense to use System.Windows.Media.Imaging.WritableBitmap for the WPF control 👍

utilizing the GPU to rasterize directly to a frame

I'm inclined away from this because I want to minimize the dependencies (software and hardware) this project takes on. If we went into GPU land there would be some interesting mathematical operations we could optimize too. I don't intend to go there any time soon (beyond Skia/OpenGL support for hardware-accelerated 2D drawing, #1036).

It should be possible to reuse all the bitmap buffers involved until a resize event occurs and the bitmap buffer must be increased or decreased

ScottPlot actually does this under the hood (re-using a System.Drawing.Bitmap and drawing on it, creating a new one only when resized).

double swap type of architecture so the consumer is never blocked

Double-buffering is an excellent concept to consider! I added it to the triage list #1028 to revisit in the future. I did some early experiments with this long ago, but I think it's definitely worth reconsidering in the future.
Note that the primary System.Drawing.Bitmap that is re-used by ScottPlot is displayed as an Image in a PictureBox in the Windows Forms control. The PictureBox Image is double-buffered out of the box, allowing the WinForms control to update that image in some situations without blocking the UI thread.


feat(views) configure handlebars in separte file

feat(views) configure handlebars in separte file

Moved configuration of handlebars to separate file:
Added helper for parsing JSON within templates

Should help to keep config/lib/express.js leaner if you want to add quite a few helpers.

The error that I was getting where:
user = ;

Doesn't seem to be present when testing on a MeanJS empty project. I'm just pushing incase it's useful to anyone.

Coverage remained the same at 72.998% when pulling ddd3d8631d3d2b6837814d3a2c1c823c98a73ca6 on Wuntenn:hbs-helper into dd809510ccf76bbc94f7516575b584d4961e102f on meanjs:master.

Could you explain how to replicate that user = ; problem?

+1 for moving handlebars to somewhere where it's easier to require to e.g. email-sending-worker-script. We did the same already earlier with Swig: https://github.com/Trustroots/trustroots/blob/master/config/lib/render.js

Sorry @simison for taking a while to get back I keep thinking about this one and getting sidetracked - not ignoring you. The code on master works fine when I'm logged in and populates the window.user object with the data from the safeUserObject but for some reason it renders user = ; when logged out.
I tried the code that you told me above: var user = {{#if user}}{{{ user }}}{{else}}null{{/if}}; (looks like list :D:D) somehow it didn't work and gave the same error.
I'd not setup the views before and was curious how to get the helpers into meanjs. When I figured how, and saw how people go about using the for parsing JSON moving them to their own config seemed the way to go. They're a lil hefty!
You're right though, it is most likely overkill but I like that it acts a simple stub to for future helpers and that it's re-usable so may in future cut down the odd one-liners used to stringify stuff.
Did you just say "sunlight" or the "T"-word!? :D Thanks for the test example. I need to write a ton of tests.


ACK runtime upgrade v0.16.2 => v0.16.3

Issue #, if available: https://github.com/aws-controllers-k8s/community/issues/1133
Description of changes:

Upgrade to ACK runtime v0.16.3
This release fixes the bug where ACK.ResourceSynced condition set by ReadOne/Update hooks was getting overwritten

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Percent-encoding (and case) normalizer, or "percent encoder"

Charactres to percent-encode depends on a spec, and the encoder is already implemented internally.
This crate should export it to prevent users from manually implementing it.

Characters that should be encoded depends on context, so the context should be some parameter (enum or type tag).
How it should be specified?

Dedicated string types for components (see #22) would be useful, but I'd like to think carefully what types would be useful and what would be not.

For now I'll implement encoder not depends on specific string types (i.e. an encoder operates on plain string types).

Released at v0.5.5.
See percent_encoding::PercentEncoded.