API Testing does not run together with HTML Report

API Testing does not run together with HTML Report. I get error output in the console, as of illigal characters.
Sample output:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.2.2:test (default-test) on project API_QTAF:  [ERROR] [ERROR] Please refer to C:\Projekte\Qytera\API_QTAF\target\surefire-reports for the individual test results. [ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream. [ERROR] There was an error in the forked process [ERROR] Illegal char <>> at index 137: C:\Projekte\Qytera\API_QTAF\logs\2024-03-05\15-48-20-firefox-688f2f39-8d08-4f4f-8d8b-75957dce5395\scenario_Login-Test statusCodeIs(200) -> expect a logMessage that indicates PASSED.html [ERROR] org.apache.maven.surefire.booter.SurefireBooterForkException: There was an error in the forked process [ERROR] Illegal char <>> at index 137: C:\Projekte\Qytera\API_QTAF\logs\2024-03-05\15-48-20-firefox-688f2f39-8d08-4f4f-8d8b-75957dce5395\scenario_Login-Test statusCodeIs(200) -> expect a logMessage that indicates PASSED.html [ERROR]         at org.apache.maven.plugin.surefire.booterclient.ForkStarter.fork(ForkStarter.java:631) [ERROR]         at org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:285) [ERROR]         at org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:250) ...

The feature was still in the design phase. The approach followed there for API testing will not be pursued further. A different approach to implementing API tests using QTAF is being considered.
As this issue relates to the old approach, which is now no longer being pursued, the issue is closed.


Datagrid styles get clobberd when there is a Compact Form with a Compact Datagrid inside it

Broken header with checkbox in DataGrid when multi-item selection enabled
everything works but looks bad, see screenshot below
clarity 3.0.1
angular 9.0.4
<clr-datagrid class="datagrid-compact" [(clrDgSelected)]="selected">
<clr-dg-column>id</clr-dg-column>
<clr-dg-column>name</clr-dg-column>
<clr-dg-column>value</clr-dg-column>
<clr-dg-column>type</clr-dg-column>
<clr-dg-row *clrDgItems="let ch of characteristic" [clrDgItem]="ch">
<clr-dg-cell>{{ch.id}}</clr-dg-cell>
<clr-dg-cell>{{ch.name}}</clr-dg-cell>
<clr-dg-cell>{{ch.value}}</clr-dg-cell>
<clr-dg-cell>{{ch.type}}</clr-dg-cell>
</clr-dg-row>
</clr-datagrid>

Hi there,
I'm not able to see your issue. We ask that reported issues also submit a stackbltitz that isolates the component and demonstrates the issue.
When I take the code provided and try to replicate the issue I'm not seeing it. Is it possible that somehting in the app style is overriding the datagrid style?
I've gone ahead a forked one of our v3 starter apps in stackblitz. Please modify it so tit replicates your issue: https://stackblitz.com/edit/gh-issue-4397

Hi,
unfortunatelly I am unable to reproduce issue on stackblitz.
App does not contain any custom css styles and only clarity components + "angular-split": "3.0.3" component which seems to be not related to the issue. Problem visible in different browsers on different pc.

Perhaps cloning your application and stripping out anything unnecessary could be useful to isolate the problem. I would also try to disable angular-split just in case there is a conflict. Without a reproduction, we're really unsure how else to proceed.

ok, i found out how to reproduce, just place datagrid inside form tag:

Clarity

Home

id
name
value
type

{{ch}}
{{ch}}
{{ch}}
{{ch}}

		<clr-datagrid [(clrDgSelected)]="selected">
			<clr-dg-column>id</clr-dg-column>
			<clr-dg-column>name</clr-dg-column>
			<clr-dg-column>value</clr-dg-column>
			<clr-dg-column>type</clr-dg-column>
			<clr-dg-row *clrDgItems="let ch of [1,2,3,4,5]" [clrDgItem]="ch">
				<clr-dg-cell>{{ch}}</clr-dg-cell>
				<clr-dg-cell>{{ch}}</clr-dg-cell>
				<clr-dg-cell>{{ch}}</clr-dg-cell>
				<clr-dg-cell>{{ch}}</clr-dg-cell>
			</clr-dg-row>
		</clr-datagrid>
    </form>
	</div>
</div>

https://gh-issue-4397-48msfa.stackblitz.io/

Ths issue I see in the stackblitz that needs more investigation is as follows: the datagrid-compact styles on a compact datagrid inside of a compact form seem to be getting clobbered. e.g its not just the checkbox label, cell content in rows (but not headers)  is also misaligned and possibly other vusual defects.
Link to the stackblitz editor: https://stackblitz.com/edit/gh-issue-4397-48msfa
I marked it as a regression but I'm not sure how it previously behaved in older versions. It has been added to the backlog.

Its getting clobbered even without "clrLayout="compact"" on form and without class="datagrid-compact" on datagrid

I'm trying to keep this issue focused on compact form with comapct datagrid inside it.
The fix will likely apply to non compact forms and non compact datagrids as well.
Hard to say at this point but they are two distinct use cases.

This can be solved by using Clarity Core forms instead, as they are more isolated and won't absorb styles the same way as Clarity UI did. I'm going to mark this as resolved by using the Core version of the components. See https://clarity.design/core-components/form/


MediaTransportControls is overriding AudioMuteButton's accessible name (and others)

Describe the bug
We cannot set a custom accessbile name to AudioMuteButton. The value we set is overridden by the MediaTransportControls implementation.
This is also breaking the automatic keyboard accelerator integration in the button tooltip.
The same behavior applies to all the other MTC buttons. The MediaTransportControls implementation is forcing all the values to some default we may not want.
Steps to reproduce the bug
See the sample app for the full repro: VolumeAccelerator.zip
The most important steps are

Copy the default MediaTransportControls style within a project
Change the AudioMuteButton by adding a custom automation name and a keyboard shortcut

<AppBarButton
    x:Name="AudioMuteButton"
    Margin="12"
    HorizontalAlignment="Center"
    VerticalAlignment="Center"
    AutomationProperties.Name="My custom automation name"
    Style="{StaticResource AppBarButtonStyle}">
    <AppBarButton.Icon>
        <SymbolIcon x:Name="AudioMuteSymbol" Symbol="Volume" />
    </AppBarButton.Icon>
    <AppBarButton.KeyboardAccelerators>
        <KeyboardAccelerator Key="Number0" Modifiers="Control" />
    </AppBarButton.KeyboardAccelerators>
</AppBarButton>

Run the application

Expected behavior
The button uses My custom automation name as its accessible name and the tooltip displays the keyboard shortcut.
Actual behavior
The button uses Mute as its accessible name and the tooltip is not displaying the keyboard shortcut.
Here is what Accessibility Insights is seeing:

Looking at the code, in OnApplyTemplate we set the automation name unconditionally. This is definitely a bug but @vgromfeld you may be able to work around this by setting the automation name after the template is applied (like in the OnLoaded event).  This will require winui 3 before we can fix.
@Austin-Lamb is reach the MTC owner?

Closing stale external bug. If you are still seeing this issue, please open a new issue in the Feedback Hub so that it may gather related logs/diagnostics.


Docsy Link Fix (again)

I added this fix in my PR but anyways you raised it then merge it now.

Also, it has conflicts.


incrementing a register

Type of issue:  feature request

Impact:  API addition (no impact on existing code)

Development Phase:  proposal
Incrementing a register (a counter) is such a common operation. Maybe we should support an increment operator:
cntReg += 1.U

such an operator should rather be named :+= to highlight the actual hardware connection taking place behind the scene
it's fairly easy to provide a naive user-side implementation (scastie) with an implicit class but it raises some interesting discussion.

Discussion
Bindings (Reg, Wire, IO, etc.) are not a first class citizen in Chisel API, only a property of Data (not even reflective from a Chisel-user point of view).

val b = Reg(UInt()) and val b = Wire(UInt()) share the same type : instances of UInt and cannot be distinguished at compile time
it is hence not possible to provide a compile time implementation of such :+= operator only for a given Binding
as the scastie illustrates, when the operator is be implemented on  UInt, it can easily allow compilation and elaboration of incorrect circuits (only caught later down the flow, during FIRRTL CheckCombLoops transform)

If the binding was reflexive from chisel-user point of view, it would be at least possible to write a partial implementation, throwing error during chisel elaboration  if the underlying Data is not bound to Reg.
I am quite in favor to open the binding API as read-only, for example through DataMirror object, in order to provide more reflexivity to end-user and library-developers (without using hacks such as overloading chisel3 package or here using the result of Data.toString)


Create mechanism for supporting different TargetFrameworks for Track1 and Track2

For now (this might change) we need to have different requirements for the TargetFrameworks for Track1 and Track2.
The current list for Track1 is net452;netstandard1.4 and for Track1 is net461;netstandard2.0
The build system currently requires all the SDKs to support the same set of TargetFrameworks.

@AlexGhiondea
There is a subtle difference the way you have mentioned it above, and I would say the underlying issue here like below:
The current build system will block from "Publishing" nuget pacakages that are not in the supported list of TargetFramework. This we do to maintain consistency for all azure .net SDK packages and most importantly consistency with external dependencies like Newtonsoft and ADAL.
But the build system will build any target Framework and will allow you to run test against any TargetFx, as long as that targetFx is installed on the CI machine.
For e.g. all SDKs to maintain consistency are targeted net452, net461, netstandard1.4 and netstandard2.0
If you happen to have a requirement to build against net462, you can still do it, but the package cannot be published if it's targeting .NET 4.6.2

@shahabhijeet the build system that @weshaggard has put in place has a check that will ensure that you are targeting at least net452 and netstandard1.4.
If you need to build a library that targets netstandard2.0 because you need to use an API that is only present in that TargetFramework, you cannot build the netstandard1.4 target (since the API is not there). And removing the build for netstandard1.4 trips the check we have to ensure that we have a consistent build.
This came up when I tried to port the Azure.Core package to the central repo. Because that package is a Track 2 package and because the guidelines for Track 2 are that the packages target net461 and netstandard20 I wasn't able to build the project (without a workaround). This issue is filed to make sure we have a path forward without a workaround.

Supported added with https://github.com/Azure/azure-sdk-for-net/pull/5298


How to find the SteamID of a save from internet?

I have a save downloaded from internet and it doesn't contain infomation about its original steamID. Is there a way to find it (with Hex editor or something) so I can use this tool?

Hi, unfortunately I still haven't fixed a method responsible for bruteforcing SteamID.
You can try to put 411651526 or 0 as INPUT SteamID and See if any of them work. If not then I am afraid the only option is to locate the original file owner and ask him about its SteamID.


Configure vs ConfigureServices

I think some guidance, guidelines, or intentions on what belongs in ConfigureServices versus Configure could be useful.  The answer may be really simple, but I am struggling to see the right division myself at the moment.
For example with the MVC middleware, configuration is done in ConfigureServices and cannot be specified via a UseMvc overload.
services.Configure<MvcOptions>(
    options =>
        {
            //  ...
        } );

However, the Diagnostics middleware requires configuration happen in Configure:
app.UseErrorPage( new ErrorPageOptions { ShowCookies = true, ShowSourceCode = false } );

On the surface this seems inconsistent.  While I am sure consistency cannot be forced, an accepted convention would make it easier for users (developers) to know where or whether to do configuration when working with a new component.
I do think that overall the distinction between the two can be useful (current multi-tenancy issues aside), I just have not fully grokked what goes where.  I first mentioned this in aspnet/Docs#57, help in the docs,  been resolved and I did not find help in the docs, I am raising this here for viability.

The ConfigureServices method defines the services used by your application and the Configure method is used to define what middleware makes up your request pipeline.

@jakeymvc That seems logical.  In practice how does that apply to the distinction between MvcOptions and ErrorPageOptions?

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


GCP ProjectSink destinations are missing examples

File: themes/default/content/registry/packages/gcp/api-docs/logging/projectsink/_index.md
PubSub Example
destination: pubsub.googleapis.com/projects/$PROJECT_NAME/topics/$SUBSUB_TOPIC

Can confirm that there are examples on google_logging_project_sink | Resources | hashicorp/google | Terraform | Terraform Registry that are not present in the linked doc as we would hope.


Saasinterface

Fixes Issue
Closes #242
Changes proposed
it will save the time for beautiful product design research
Screenshots

Note to reviewers
Any kind of feedback will be appreciated

Content-wise, your PR looks great! :) Branch wise, don't push into the main as it makes it difficult the PR to be merged.

now review the pr

One more thing @Parma11111!:) Don't create PR direclty to base: main instead, select base:dev from the dropdown. Other than LGTM! 😀

From now I will push in base: dev


Add methods for making non-identity transform matrix

title, currently only an identity or zero matrix can be constructed

Implemented in 09e19d0cea0f89a3435e2fc7c9755466963211d6


Combine Option and Option<T&> impls

It's hard cuz C++ and templates like to remove references but should be possible..

sus_move_preserve_ref() and decltype(auto) will help by allowing moving from a field if it's a non-reference and copying the reference itself if it's a reference.
we had to figure this out for sus::Tuple since there's an unbounded combinatorial specialization problem there otherwise, and it's now working there.

Fixed by https://github.com/chromium/subspace/pull/136


fix: CVE-2022-32149

Will create a proper release for the mentioned fix.

The fix commits weren't properly semantic (fix: xxx). This means no release was prompted, because they were not detected as "user-facing" commits. The manual update of the manefest then really confused release please, because it couldn't find the reference 0.2.6 release.


How to train a transformer model with multi-source encoders ?

I wonder how to modify the configuration file to train a multi-source based transformer model with different attention types.

Hi, currently, the Transformer decoder only supports the multi-head scaled dot-product attention from the "Attention is All You Need" paper. If you provide multiple encoders, you can choose which attention combination strategy you want to use, one of serial, parallel, hierarchical, and flat.

I wonder how to specify the combination strategy for multiple encoders in the configuration file,  have any examples?

just specify the attention_combination_strategy parameter in the transformer decoder configuration. It can be one of serial, parallel, hierarchical, and flat.


F@H Control not working anymore after Update

Version of the custom_component

2.0.8
Configuration

websocket_api script: Error executing script. Unexpected error for call_service at pos 1: 'dict' object has no attribute 'config_entry'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 359, in _async_step
    await getattr(self, handler)()
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 559, in _async_call_service_step
    await service_task
  File "/usr/src/homeassistant/homeassistant/core.py", line 1480, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1515, in _execute_service
    await handler.job.target(service_call)
  File "/config/custom_components/foldingathomecontrol/services.py", line 46, in async_call_folding_at_home_control_service
    await async_pause_service(hass, service_data)
  File "/config/custom_components/foldingathomecontrol/services.py", line 98, in async_pause_service
    if hass.data[DOMAIN][config_entry].config_entry.data[CONF_ADDRESS] == address:
AttributeError: 'dict' object has no attribute 'config_entry'

[140369749619376] Error handling message: Unknown error
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 18, in _handle_async_response
    await func(hass, connection, msg)
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 439, in handle_execute_script
    await script_obj.async_run(context=context)
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1195, in async_run
    await asyncio.shield(run.async_run())
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 341, in async_run
    await self._async_step(log_exceptions=False)
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 359, in _async_step
    await getattr(self, handler)()
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 559, in _async_call_service_step
    await service_task
  File "/usr/src/homeassistant/homeassistant/core.py", line 1480, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1515, in _execute_service
    await handler.job.target(service_call)
  File "/config/custom_components/foldingathomecontrol/services.py", line 46, in async_call_folding_at_home_control_service
    await async_pause_service(hass, service_data)
  File "/config/custom_components/foldingathomecontrol/services.py", line 98, in async_pause_service
    if hass.data[DOMAIN][config_entry].config_entry.data[CONF_ADDRESS] == address:
AttributeError: 'dict' object has no attribute 'config_entry'

Describe the bug
After updating the Core the control isn't working and i become this error. I can not find the issue. I reinstalled F@H and F@H Control but not working.
My Automations:
  alias: F@H stoppen
  description: ''
  trigger:
  - platform: template
    value_template: '{{ states("sensor.total_consumption_live")|int >= -100 }}'
  condition: []
  action:
  - service: foldingathomecontrol.pause
    data:
      address: localhost
      slot: ''
  mode: single

Debug log

Add your logs here.

Thank you for reporting this issue. This is a leftover of the last Bugfix. I will fix this and release a new version in a few hours

Thank you for reporting this issue. This is a leftover of the last Bugfix. I will fix this and release a new version in a few hours


SilverStripe files in "Public" directory

Hey this module seems great. I recently wrote a bash deployment script which I've been using but yours is much more extensive with all the migration stuff. Really cool.
I'm keen to give it a go but am a bit put off by having to have the site in a public subfolder as this means I can't just drop it in to my existing way of doing things. Just wondering why it's set up like that? And how possible (or not) would it be to make the subfolder an optional/configurable option?

Hey Shea,
happy you like the module!
In order to get development on this module going, I started out going with my own practices, but I do realise that people have very different ways of doing things, and I realise that forcing SilverStripe files to have to live inside of the public directory, might be a deal breaker for many, so I'm all in for making this configurable. I think I even talked to @leomeloxp about it at StripeCon last year. My friend @zauberfisch is another full hearted hater of my practice of using the public directory.
A little reasoning why I'm doing like I'm doing
I see a git repository as a self documented entity, preferably containing as much as possible about a project, including documentation, server configuration, etc.
As terminal tools is checking the entire git repository out on the server, I don't want all of this to be accessible from the web root, which is why I'm having SilverStripe specific files inside of a sub directory, which I call public.
Below is an example of how the directory structure for a project could look like. There I'm also keeping uncompiled JavaScript and Scss files (resources) as well as node_modules out of the web root.

Making the web root directory configurable
It shouldn't be too hard making the root directory configurable. A quick search in affected modules shows this:

https://github.com/titledk/ttools-silverstripe/search?utf8=✓&q=public
https://github.com/titledk/ttools-sitesync-silverstripe/search?utf8=✓&q=public

As to how to go about the configuration I propose to add an optional Webroot configuration, which could be similar to how Sitesync is used here: https://github.com/titledk/ttools-silverstripe/blob/master/install/install-ttools.sh#L92
I'd say that by default it would be public/, (Webroot: public/) but could be anything else, also /.
I'm not too happy about using / (root) as that way the ttools directory would be installed inside of the web root, and the ttools/config.yml could be accessed from the web, but there are ways around it by just protecting that directory via .htaccess. (I need ttools to also be available on the server for for syncing purposes.)
vars.sh from core would just read that configuration out, and it should be accessible as a variable in the global scope (just like $Sitesync_FrameworkModule is here: https://github.com/titledk/ttools-sitesync-core/blob/495bac886c2e55baf2022877787bdd166891470d/lib/dump-current-site.sh#L67).
vars-for-env.sh contains some hints on how to deal with default values. I'd love a refactor/simplification of the configuration system, but we'll take that another day :)

@sheadawson I've just added the labels enhancement & help wanted. I think this would be a great feature, but it's just nothing I need in my day-to-day work.

Nice one @anselmdk and thanks for the explanation. If I find some time I'll definitely be keen to contribute


[DRAFT] Add new authentication event types to the proto

This draft models the ES authentication events in Santa's schema with expected enrichments.

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

This was merged to the new protos repo here: https://github.com/northpolesec/protos/pull/3


feat: scheduling bounded context

this PR is to address https://github.com/odpf/optimus/issues/583
associated proton pr : https://github.com/odpf/proton/pull/217
Note:
old implementations have been removed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1193 of 1278 (93.35%)  changed or added relevant lines in 26 files are covered.
92 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.9%) to 81.563%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/utils/map.go
8
9
88.89%

core/scheduler/service/executor_input_compiler.go
100
102
98.04%

core/scheduler/service/job_run_asset_compiler.go
68
70
97.14%

ext/notify/slack/slack.go
31
33
93.94%

ext/scheduler/airflow/dag/upstream.go
19
21
90.48%

service/job_run_service.go
0
3
0.0%

internal/errors/errors.go
0
7
0.0%

core/scheduler/service/notification_service.go
52
61
85.25%

ext/notify/pagerduty/pagerdutyservice.go
0
11
0.0%

ext/scheduler/airflow/dag/compiler.go
38
50
76.0%

Files with Coverage Reduction
New Missed Lines
%

ext/notify/pagerduty/pagerdutyservice.go
1
0%

ext/notify/slack/slack.go
1
82.05%

service/job_run_service.go
90
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.9%

Covered Lines:
9985

Relevant Lines:
12242

💛  - Coveralls


Fix issue 20064,20069 - format functions make their final width and digits grouping conform to POSIX definition

The format functions of FloatingPointType and IntegralType are adjusted and optimized to make their Final width and digits grouping conform to POSIX definition.

@thewilsonator, It's all green   ^_^


Iterate over templating

I was about to do some templating and gave it another iteration. Data structures moved into template/.
Added some functions I consider necessary. pairs being a way for ordered iteration that I think is understandable to everyone and simplifies our data structure and also the usage for simple ordered iteration (no use of index function and reference to parent scope).
I want a first round of feedback before continuing with templates themselves.
@brian-brazil @beorn7

:+1:


MIRI CI check fails intermittently with thread 'main' panicked at 'invalid time'

Describe the bug
Ever since we have enabled MIRI checks on CI in https://github.com/apache/arrow-rs/pull/323 (epic thanks to @roee88 ) we have started to see intermittent failures of the check
To Reproduce
Run CI checks on a PR
Here are some example failures
https://github.com/apache/arrow-rs/runs/2656075800
https://github.com/apache/arrow-rs/runs/2658362992
An excerpt from
https://github.com/alamb/arrow-rs/runs/2659329294?check_suite_focus=true
test array::array_primitive::tests::test_time32_millisecond_array_from_vec ... ok
test array::array_primitive::tests::test_time32second_fmt_debug ... ok
thread 'main' panicked at 'invalid time', /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/chrono-0.4.19/src/naive/time.rs:412:67
stack backtrace:
   0:          0xa9fa3ef - std::backtrace_rs::backtrace::miri::trace_unsynchronized::<&mut [closure@std::sys_common::backtrace::_print_fmt::{closure#1}]>
                               at /usr/share/rust/.rustup/toolchains/nightly-2021-03-24-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/../../backtrace/src/backtrace/miri.rs:65:18
   1:          0xaa266e9 - std::backtrace_rs::backtrace::miri::trace::<&mut [closure@std::sys_common::backtrace::_print_fmt::{closure#1}]>
                               at /usr/share/rust/.rustup/toolchains/nightly-2021-03-24-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/../../backtrace/src/backtrace/miri.rs:51:14
   2:          0xa9dc49b - std::backtrace_rs::backtrace::trace_unsynchronized::<[closure@std::sys_common::backtrace::_print_fmt::{closure#1}]>
                               at /usr/share/rust/.rustup/toolchains/nightly-2021-03-24-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   3:          0x298f21f - std::sys_common::backtrace::_print_fmt
                               at /usr/share/rust/.rustup/toolchains/nightly-2021-03-24-x86_64-unknown-linux-g

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Caused by:
  process didn't exit successfully: `/usr/share/rust/.rustup/toolchains/nightly-2021-03-24-x86_64-unknown-linux-gnu/bin/cargo-miri /home/runner/work/arrow-rs/arrow-rs/target/x86_64-unknown-linux-gnu/debug/deps/arrow-d73b9ed71d5fffd0 --skip csv --skip ipc --skip json` (exit code: 255)
test compute::kernels::length::tests::bit_length_test_string ...
Error: Process completed with exit code 255.

🤔

I see that it's (almost?) always in bit_length_test_string. Did you consider skipping this test? Should we open an issue in miri to help us debug this? It might reveal something important about this test.

I see that it's (almost?) always in bit_length_test_string. Did you consider skipping this test? Should we open an issue in miri to help us debug this? It might reveal something important about this test.

@roee88  I did not -- I tried to skip array::array_primitive::tests::test_time32second_invalid_neg in #346 but that did not seem to help. I don't have much time to devote to this issue at the moment (most of my arrow bandwidth is going towards getting the releases in shape) so any help here would be most appreciated

I think that the issue description here is wrong and this is expected for test_time32second_invalid_neg.
I did a quick memory usage check and identifier the following:

bit_length_test_string  consumes on its own ~8g accumulated to a peak of ~10g that are not released after the test is done
bit_length_test_large_string has similar memory consumption but the memory is freed after the test is done
match_single_group  consumes 2.7g accumulated to a peak of ~4.5G that are not released after the test is done

I tested ignoring bit_length_test_string  and github actions checks pass (5 re-runs). I think it might be worth to ignore all three though. Some questions:

Is it possible that there is a memory leak in bit_length_test_string and match_single_group?
Should we disable bit_length_test_large_string  just because it consumes a lot of RAM while running?
Should a support ticket be opened in Miri?

I will submit a PR based on the answers.

Thank you @roee88

Just to understand, the memory consumption is only when running the test via MIRI, right?

Is it possible that there is a memory leak in bit_length_test_string and match_single_group?

I don't know but it sounds like it is worth more investigation

Should we disable bit_length_test_large_string just because it consumes a lot of RAM while running?

I think we should disable the test under MIRI

Should a support ticket be opened in MIRI?

If there is some bug or improvement you can articulate, I ams ure the MIRI developers would appreciate a ticket


package.json: fix repo link

Noticed the link to the GH repo on npm was broken. This fixes it. Thanks!
Changes

pkg: fixed repo link and moved to shorthand notation

:metal:
Didn't know that was supported yet, but it is!  Hat tip to @iarna

\o/ Yeah, it got back ported early!


WFLY-4305: exit immediately on non-zero status

Windows Build 941 is now running using a merge of 2d467dd24085e9def93b5f10706ea3598fc220c3

Linux with security manager Build 817 is now running using a merge of 2d467dd24085e9def93b5f10706ea3598fc220c3

Linux Build 5832 is now running using a merge of 2d467dd24085e9def93b5f10706ea3598fc220c3

Linux with security manager Build 817 outcome was SUCCESS using a merge of 2d467dd24085e9def93b5f10706ea3598fc220c3
Summary: Tests passed: 839, ignored: 316 Build time: 0:06:27

Linux Build 5832 outcome was FAILURE using a merge of 2d467dd24085e9def93b5f10706ea3598fc220c3
Summary: Tests failed: 2 (1 new), passed: 2919, ignored: 312 Build time: 0:48:08
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.smoke.webservices.WSTestCase.testManagementDescriptionMetrics: java.lang.AssertionError: null
	at org.junit.Assert.fail(Assert.java:86)
	at org.junit.Assert.assertTrue(Assert.java:41)

org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

Windows Build 941 outcome was FAILURE using a merge of 2d467dd24085e9def93b5f10706ea3598fc220c3
Summary: Tests failed: 1, passed: 2920, ignored: 312 Build time: 0:48:46
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

please ping me when it's rebased so I can hit the green button.

Superceded by #7153 (the rebase.)


Add GraphCDN extension

Description
List your GraphCDN services and inspect their configuration
Type of change

New extension

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast

Not yet no, Just added the functions to do so if I (or someone else) want to add this functionality


Library not loaded: libavdevice.58.dylib

🐛 Describe the bug
I got an error while importing Streamer.
from torchaudio.prototype.io import Streamer
Torchaudio couldn't load ffmpeg library. But the files exists in<EMAIL_ADDRESS>Traceback (most recent call last):
  File "inference.py", line 5, in <module>
    from torchaudio.prototype.io import Streamer
  File "/***/emf_venv/lib/python3.8/site-packages/torchaudio/prototype/io/__init__.py", line 4, in <module>
    torchaudio._extension._load_lib("libtorchaudio_ffmpeg")
  File "/***/emf_venv/lib/python3.8/site-packages/torchaudio/_extension.py", line 51, in _load_lib
    torch.ops.load_library(path)
  File "/***/emf_venv/lib/python3.8/site-packages/torch/_ops.py", line 244, in load_library
    ctypes.CDLL(path)
  File "/Users/***/.pyenv/versions/3.8.13/lib/python3.8/ctypes/__init__.py", line 373, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: dlopen(/***/emf_venv/lib/python3.8/site-packages/torchaudio/lib/libtorchaudio_ffmpeg.so, 6): Library not loaded: @rpath/libavdevice.58.dylib
  Referenced from: /***/emf_venv/lib/python3.8/site-packages/torchaudio/lib/libtorchaudio_ffmpeg.so
  Reason: image not found

Versions
PyTorch version: 1.12.0.dev20220427
Is debug build: False
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A
OS: macOS 11.6.4 (x86_64)
GCC version: Could not collect
Clang version: 12.0.0 (clang-1<IP_ADDRESS>)
CMake version: version 3.19.3
Libc version: N/A
Python version: 3.8.13 (default, Apr 28 2022, 12:42:44)  [Clang 12.0.0 (clang-1<IP_ADDRESS>)] (64-bit runtime)
Python platform: macOS-11.6.4-x86_64-i386-64bit
Is CUDA available: False
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
Versions of relevant libraries:
[pip3] numpy==1.22.3
[pip3] torch==1.12.0.dev20220427
[pip3] torchaudio==0.12.0.dev20220427
[conda] Could not collect
ffmpeg version 4.4.2 (Also tried the latest version)

Hi @innarid, thanks for sharing the issue. It seems cmake didn't recognize the ffmpeg path when building torchaudio. Could you try to build torchaudio from source using:
<EMAIL_ADDRESS>BUILD_FFMPEG=1 python setup.py clean develop

@nateanl That's build time discovery. The issue is about runtime discovery.
@innarid Is<EMAIL_ADDRESS>set as library search path? Can you set LD_LIBRARY_PATH=/usr/local/Cellar/ffmpeg@4/4.4.2/lib/:${LD_LIBRARY_PATH} before starting Python, and see what happens?

Hi @innarid, thanks for sharing the issue. It seems cmake didn't recognize the ffmpeg path when building torchaudio. Could you try to build torchaudio from source using:
<EMAIL_ADDRESS>BUILD_FFMPEG=1 python setup.py clean develop

Thak you! It helped


chore(master): release 1.3.0

:robot: I have created a release beep boop
1.3.0 (2023-02-05)
Features

hello (0379cfb)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/igar1991/swarm-search/releases/tag/v1.3.0 :sunflower:


Installing the SQL Database Projects is giving error

Version: 1.47.0-insider (system setup)
Commit: f1467cdd72de9f22ef7e433df114523663b7dabe
Date: 2023-10-12T05:44:36.268Z
VS Code: 1.79.2
Electron: 22.3.25
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Steps to Reproduce:

Installing the 'SQL Database Project' extension on insider is giving below error.

2023-10-12 21:53:06.349 [error] Error: net::ERR_CONNECTION_REFUSED
at SimpleURLLoaderWrapper. (node:electron/js2c/browser_init:2:49386)
at SimpleURLLoaderWrapper.emit (node:events:513:28)
2023-10-12 21:53:06.352 [error] An unknown error occurred. Please consult the log for more details.
2023-10-12 21:53:06.352 [error] net::ERR_CONNECTION_REFUSED: Error: net::ERR_CONNECTION_REFUSED
at SimpleURLLoaderWrapper. (node:electron/js2c/browser_init:2:49386)
at SimpleURLLoaderWrapper.emit (node:events:513:28)

Does this issue occur when all extensions are disabled?: Yes/No

I wasn't able to repro this locally on the latest insiders. Can you try the installing the latest sql projects extension (version 1.4.0 released today) on insiders and see if it's still not working for you?

Even the latest insiders also giving me the same error.

Even the latest insiders and latest SQL project extension also giving me the same error.

Can you try disabling "Http: System Certificates" setting and restart ADS to try again?
Any other http proxy related changes in your env?
Is everyone else in your network facing the same error or it's specific to your system?

This seems to behave differently based on the network. From my home network, this is still a problem but when I try it from office network this works fine.

After discussing with Erin, I have tried again with latest update
Version: 1.47.0-insider (user setup)
Commit: 9c5bbf6251ba3f4b9d01ce7469eb96b905ec5d04
Date: 2023-10-25T05:34:15.846Z
Electron: 25.8.4
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
After disabling the "Http: System Certificates"

2023-10-25 20:53:16.152 [error] XHR failed: Error: XHR failed
at S.onerror [as __zone_symbol__ON_PROPERTYerror] (vscode-file://vscode-app/c:/Users/cpichuka/AppData/Local/Programs/Azure%20Data%20Studio%20-%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:151:1408) []
at XMLHttpRequest.wrapFn (C:\Users\cpichuka\AppData\Local\Programs\Azure Data Studio - Insiders\resources\app\node_modules.asar\zone.js\dist\zone.js:815:43) []
at ZoneTask.invokeTask [as invoke] (C:\Users\cpichuka\AppData\Local\Programs\Azure Data Studio - Insiders\resources\app\node_modules.asar\zone.js\dist\zone.js:525:38) []
at invokeTask (C:\Users\cpichuka\AppData\Local\Programs\Azure Data Studio - Insiders\resources\app\node_modules.asar\zone.js\dist\zone.js:1727:22) []
at globalCallback (C:\Users\cpichuka\AppData\Local\Programs\Azure Data Studio - Insiders\resources\app\node_modules.asar\zone.js\dist\zone.js:1758:31) []
at XMLHttpRequest.globalZoneAwareCallback (C:\Users\cpichuka\AppData\Local\Programs\Azure Data Studio - Insiders\resources\app\node_modules.asar\zone.js\dist\zone.js:1794:20) []

Do you have any value in http.proxy setting in ADS or HTTP_PROXY or HTTPS_PROXY environment variables?
Have you tried disabling http.proxyStrictSSL setting?


OTC framework generated code compilation fails due to an extra string "class " with a space

Description:

I created an OTCS script file with a copy-source command to copy from source to target of GregorianCalendar type.

Executed a test project to generate the object conversion source code.

A word "class " prefixed to the 2nd parameter when OTC framework's DateCoverterFacade is invoked in the generated code.

originDepRequest.setDate(DateConverterFacade.convert(date, class XMLGregorianCalendar.class));
4. Generated command from OTC editor as below:
copy: id: CP1 from: objectPath: request.flightRequest.flightRequestOriginDestinationsCriteria.originDestCriteria[].originDepCriteria.date to: objectPath: request.flightRequest.originDestRequest[].originDepRequest.date factoryClassName: CP1

This issue is fixed and is available in 2.7.0 release.


Support beautiful file/folder icon

What's the problem this feature will solve?
Use a open-source icon lib, such as vscode-icons, support file/folder icon.

I'm not really sure this would add much to the server, the directory listing is meant to be an aid to development. Would this add anything other than just improving the way it looks a little?


DO NOT MERGE: Explore faster GCC build times in Travis

GCC builds with joinwithbinaryexpressions turned on takes a very long time since the parallelism is set to 1. Explore if we can improve on this by moving up the gcc version that is more robust to template generation.

Thanks @hakanmemisoglu for PR#161. Closing this.


Revert "Bug/gravitation page"

Reverts Amit366/FunwithPhysics#82

@laisfrigerio ur code is not working.
After clicking on gravitational force it should appear  a page where there are topics related to gravitation(refer to the live website), then on clicking on the topic it should open the calculator. But in ur code there is no calculator and also no topics page.

@Amit366 I made a change into App.js. But this changed was ignored in merge, probably.
I'll correct here and make a new pull request.

@laisfrigerio join the discord server so that it would be easy to discuss.


wip:added design doc for monitoring

added design doc for monitoring

Please consolidate with Stevens documentation and make a pullrequest to feature/health-monitor


Connections to method window doesn't expand to fit the text

Godot version:
3.2
OS/device including version:
Win 10
Issue description:
New 3.2 Connections to method button is great addition to the engine and I enjoy it a lot but it has one small limitation that it doesn't dynamically scale to a text inside the window reducing it usability.
Steps to reproduce:
Connect new signal to script and click on Connections to method button observe window being to small
Minimal reproduction project:
Bug.zip

Here's a screenshot for reference:

Source and signal names are truncated, because the dialog is too small. Not sure if making it dynamically-sized is a simple task, but making it bigger could be enough too.


Usage from background worker

Would you say that this is recommended to use in background worker for email templates?
I see only ASP usages mentioned

Should be fine, it doesn't add any more dependencies than Razor already does. So as long as you're ok with using Razor (which depends on parts of ASP.NET Core) it shouldn't be an issue. I'll use this issue to track adding an example of this scenario to the sample app.


Does Youtube-dl support YouTube mixes?

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Sorry for a question.
So the question in the topic. I use discord bot and when I want to add a youtube mix (Eample: https://www.youtube.com/watch?v=IcrbM1l_BoI&list=RDQMVCAPd5cwBcg) it just download first video that's all. And i don't know is it my trouble or youtube-dl doesn't support yt mixes?

It does.
Carefully read new issue template and provide all requested information.


INFO - Server information updating too slowly

Describe the bug
Information from the server only updates every 5-10 minutes
Growatt Device Type

Device(s): All
Home Assistant information

Home Assistant version: N/A
Growatt Server API Integration version: N/A

Additional context

In a recent release of the Core integration the logic was changed to only poll the server every 5 minutes since that is how often the ShineLink system is configured to push data to the servers by default. However, it is possible to update this frequency and therefore some users wish to poll the system more often.
This issue has been fixed in release 1.0.1 of this plugin however this ticket exists purely to notify people.
There is an FAQ page here with links on how to configure your system to poll more frequently.
NOTE - You can still configure your Home Assistant Integration to poll more regularly even if you don't update the ShineLink configuration, it will cover the difference between when the data is pushed and when it is polled.

@koosoli - You raised this as an issue over in the Core Integration recently. I separated it out as a specific bug so that you could see that there is a fix for it.
Please could you let me know when you have seen this?

@stubersa - I saw you post over on the HomeAssistant Core Documentation repo about the update interval for the Growatt.
NOTE - This repo is a fork of the one I maintain in the Core release of Home Assistant as I got fed up with having to wait on their release cycle to fix critical issues.
I recommend reading the README page here for more info on that side of things.
As for updating more frequently, read the body of this ticket above and the FAQ page and it should help you out. NOTE - That functionality is not yet in the Home Assistant core release of the integration as I haven't ported all the changes upstream yet, so if you want that functionality, you'll have to switch to this custom component.

Thank you @muppet3000!
I have a Jinko Solar inverter with a ShineWIFI-X module. Your integration is working great!
As far as the update interval goes, I was able to follow the guide posted here
That allowed me to get the dongle into AP config more, all the way to the advanced setting, where I could see the interval set to 5 min, but I need a password to unlock the settings, and seems Jinko has a different password to growatt.
I was however able to implement the automation in HA to bring that update interval down, so now HA is basically as up to date as the app/site.
Thanks for all your work on this, and all your help.
Cheers


Object Store: Allow custom reqwest client

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

I am connecting to our Ceph cluster and getting intermittent connection closed errors from the server. I have tried both multipart and non-multipart uploads and have had the same issues.
Others have run into this issue with the S3 API with reqwest/hyper and have needed to change their client configuration to resolve the issue.
See:

https://github.com/hyperium/hyper/issues/2136
https://github.com/hyperium/hyper/issues/2136#issuecomment-861826148

Describe the solution you'd like

I would like to expose the reqwest client for custom configuration.
Describe alternatives you've considered

An alternative would be to expose individual configuration settings in the Reqwest client and continue to shield creation of the client from the end user. This seems like unnecessary complexity for the object store to expose.
Another alternative would be to make the reqwest client public so that it can be modified after the S3Client has been created. Exposing the client in this way is bad API design. Keeping the client private while allowing initial configuration seems to me to be the best route.
Additional context

label_issue.py automatically added labels {'object-store'} from #3252


Update plugin intellij

Code space plugin for intellij is outdated, do you have plans to maintain it?

Agreed. Gateway is currently completely useless for Github's Codespaces. I guess I'll stick with their web ui or switch to vs code.

I guess I'll stick with their web ui

Do you mean we can connect to GitHub Codespaces without this plugin? If yes, can you please show me how to do that?


Create define tags of all gods

I just change some tags for detail and make the user can know exactly what role of each god

Hi @trungnguyenM01,
Great work on this update! The images look fantastic, and I really appreciate the effort you put into this contribution. There is a deployment error, but I believe I can fix it on my end. I’ll go ahead and merge this into the main branch and address the issue.
Thank you again for your hard work—I look forward to seeing more of your contributions in the future!


Change accordion header colors depending on all answers complete

Currently, the accordion header color in the certification survey changes color from blue to orange when all answers are completed (independent of whether the answers are correct).
It would be helpful to the user to have the color change to green if all answers are correct.

This is a bit involved since the front-end JavaScript doesn't calculate or keep track of the correct answers.  The JavaScript code that changes the color is in the file WebContent/resources/certsurvey.js near the end of function updateSectionQuestionCounts(section).
There is enough information returned in the function getSurvey() to accomplish the calculations, but it needs to take into account several factors.  It would essentially duplicate code in the Java backend.
A different, perhaps better approach would be to replace the current JavaScript code which looks to see if all questions have been answered with an Ajax call to the backend to check on how many questions are answered and if all questions are correct.  The color could be updated in the response for the Ajax call.


Fix WebSocket import error for Python 3.12 compatibility

This pull request addresses an issue encountered specifically in Python 3.12 related to importing the WebSocket extension in Bottle applications. The error arises due to the absence of the bottle.ext.websocket
Additionally I added tox configurations for Python 3.11 and Python 3.12
This PR should also resolve the following issues:
#723

It verry good

@samuelhwilliams Can you please have a look at pulling this in for 3.12 support

Thanks for the nudge and sorry for the delay. This looks good - will merge.

Thanks Samuel!


[feature request] has ability to stop executor in CRI

Ⅰ. Issue Description
When we use SyncExec with timeout, we should stop and kill the executor in the container if timeout.
Ⅱ. Describe what happened
If SyncExec runs the never-return command with timeout setting, the command is still running even though we return timeout error
Ⅲ. Describe what you expected to happen
The command should be stopped/killed if timeout.
Ⅳ. How to reproduce it (as minimally and precisely as possible)

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

already fixes it.


Large files cause freeze when embedding

Describe the bug
Large files cause freeze when embedding
To Reproduce
Embed a file > 500 lines or so.
Expected behavior
It should work
Screenshots
na
Logging
API Provider
Ollama
Chat or Auto Complete?
Embedding
Model Name
All models
Desktop (please complete the following information):

OS: Ubuntu

Additional context
Nothing else

I was noticing this but I also am noticing that I can't seem to get Ollama to keep the embeddings model in memory.    4 mins into the embedding process the model disappears and the Twinny process hangs there.    I thought it was large files but I'm now thinking its the model being unloaded.   I have set the env variable to tell it to keep models in memory for 60m.

some additional debugging info in case it helps.
I ran ollama with a keep alive of 10h and debug turned on..       verified the model is in memory and ollama ps says its there for 10 hours.
I clicked on the "embed documents"..  I see the twinny dialog in vscode.. it shows progress and then stops at 8.20%.   the file its on is about 350 lines of Ruby code.
in the ollama debug I see this.
time=2024-12-02T22:32:56.359-05:00 level=DEBUG source=sched.go:575 msg="evaluating already loaded" model=/Users/gpinkham/.ollama/models/blobs/sha256-797b70c4edf85907fe0a49eb85811256f65fa0f7bf52166b147fd16be2be4662 time=2024-12-02T22:32:56.362-05:00 level=DEBUG source=runner.go:752 msg="embedding request" content="�\x02 REMOVED CONTENTS �\x01�\x01" time=2024-12-02T22:32:56.362-05:00 level=DEBUG source=cache.go:104 msg="loading cache slot" id=0 cache=71 prompt=8 used=0 remaining=8 [GIN] 2024/12/02 - 22:32:56 | 200 |   12.461333ms |       <IP_ADDRESS> | POST     "/api/embed" time=2024-12-02T22:32:56.371-05:00 level=DEBUG source=sched.go:407 msg="context for request finished" time=2024-12-02T22:32:56.371-05:00 level=DEBUG source=sched.go:339 msg="runner with non-zero duration has gone idle, adding timer" modelPath=/Users/gpinkham/.ollama/models/blobs/sha256-797b70c4edf85907fe0a49eb85811256f65fa0f7bf52166b147fd16be2be4662 duration=10h0m0s time=2024-12-02T22:32:56.371-05:00 level=DEBUG source=sched.go:357 msg="after processing request finished event" modelPath=/Users/gpinkham/.ollama/models/blobs/sha256-797b70c4edf85907fe0a49eb85811256f65fa0f7bf52166b147fd16be2be4662 refCount=0
10 minutes later the twinny dialog has not moved beyond that file and the 8.20%.    and the model is still in memory (per ollama ps) so its not ollama unloading the model causing the hang in twinny.


Added autodiscovery mechnism for getting vagrant's full command list

I'm using vagrant with openshift plugin, so I'm having reacher list of commands available with vagrant. So I though it might be good idea to create such auto-discovery within your autocompletion. So here it is :)

@soltysh Shouldn't etc/bash_completion.d/update be located somewhere like setup/update or something? Otherwise, this seems to make sense as long as the subcommand extraction is robust.

@dobbymoodge you're right, I've fixed it, by moving to suggested setup directory. I've also rebased the PR on top of recent master changes.

@kura any chance of a re-review with the recent changes?

:-1:
I'm not the repo owner but I do got some comments that I think should be addressed before merging.

I do not think it's a great idea to have the commands being set during install. Then I have to make install every time commands are added/removed.
Modifying source files with sed makes it hard to read/understand the source code.
The feature to discover subcommands/flags should be put in a separate pull request.
I might sound negative but I do like the intent just not the way it's implemented!

@Mattias- thx for you input :smile:. My initial goal was to do live reading of the subcommands, unfortunately vagrant response times were so long (and trust me that delay wasn't acceptable) I resigned of that idea at the time of writing it and created the autodiscovery mechanism during make install.
I'm not sure why I haven't used the list-commands, though. I'll have a look if it's usable. Also the caching idea makes sense, I'll have a look at it, but I can't promise anything in short term :blush:
As for the flags, well the project contains some of them, so I though of having them all, since the mechanics for that are in place, it's debatable, I can be convince to do it one way or the other.

@Mattias- +1 for --machine-readable. It would be nice if there were something equivalent for sub-subcommands (e.g. box list).
I don't think that caching is a good idea, because it's an unexpected side-effect, and potentially introduces a race condition1. I also think that rebuilding the subcommand list every time you hit "tab" is unacceptably slow.
I think the "make install" behavior is the least evil option here. It gets us up-to-date completion without sacrificing speed, and it ensures that any changes to the system made by this package are the result of user intent.

1: This could be addressed using mktemp I suppose, but then users with multiple shells open might see different completion options after installing a new vagrant plugin. Another option would be to use a lockfile. I'll leave it up to @soltysh to determine if the added complexities from these techniques are acceptable/sustainable.


[hi] Update data/i18n/hi/hi.toml to include missing strings

This is a Bug Report

Problem:
The current data/i18n/hi/hi.toml on branch dev-1.22-hi.1 is not up to date with the English version.
Proposed Solution:
Compare en.toml with hi.toml and add the missing strings (19 in number)
Page to Update:
https://github.com/kubernetes/website/blob/dev-1.22-hi.1/data/i18n/hi/hi.toml

Note:
The current working branch for hi localization work is dev-1.22-hi.1. Please make sure to open a PR against dev-1.22-hi.1.
For more info see Hindi localization progress tracking issue #29353
/language hi
/kind bug

/triage accepted

Hi I would like to work on this issue.

/assign

/close
resolved by #31072


#4 #5 fix warnings and zeroconf on docker

Hi,
First thanks a lot for this integration !
Here is a proposal to fix #4 and #5.
For #4, actually Mopidy publishes the "_mopidy-http._tcp.local." zeroconf so it's a good way to detect a Mopidy instance without trying to connect. I also removed the warnings in the logger which should not have any more  sense now.
For #5, the problem has been discussed a lot of times in Home Assistant integrations. Either the zeroconf service publishes an IP or a FQDN other than .local, in which case it can be used, either we just get the IP.  Even better, just after getting the IP, we reverse resolve it into the FQDN so the configuration remains stable even with an IP change of the device.
Thanks!

Hey,
Thanks for figuring this out and submitting this.
I wasn't aware of the '_mopidy-http._tcp.local.' zeroconf endpoint, but that would mean it needs to go into manifest.json to make it clean, and auto-available for HA.
One issue I can see with the reverse lookup is when you do not have a dedicated DNS server, and solely rely on zeroconf. Granted, if you have a docker instance you're probably skilled enough to have one. But I think there should be a fallback just in case, what do you think?

Hi!
Good catch, in fact I already had _mopidy-http._tcp.local. in manifest.json but I forgot to commit it, since I'm developing outside of the repository.
You're right too about the reverse lookup. My internet box auto provides a .home network and I didn't think about it. In a Docker scenario, we could try to reverse lookup and just use the IP if there is no resolution, which is not ideal but still better than nothing. In a non-Docker scenario, it would however be best to keep the .local one. However I can't see how to easily combine both scenari, except with a configuration which would kind of defeat the zeroconf stuff. What do yo think ?

In fact we could probably do something like this:

try to resolve the .local address (no Docker scenario)
if it fails, try to reverse resolve the IP to a FQDN (Docker scenario with DNS)
fallback on IP in last resort

I'm gonna try this!

In fact we could probably do something like this:
1. try to resolve the .local address (no Docker scenario)

2. if it fails, try to reverse resolve the IP to a FQDN (Docker scenario with DNS)

3. fallback on IP in last resort

This is the way! (pun intended)

Done, however I can't test the cases 1 and 3. In which case are you ?

Thank you for bearing with me! I really appreciate it.
Just noticed the displayname for devices are a bit awkward due to zeroconf.
Mine are named like this: figrindan_mopidy_http_tcp_local. See what I can find

You changed
self._name = discovery_info["properties"].get("name", self._host)
to
self._name = discovery_info[CONF_NAME]
Is there any particular reason for that?

Thank you for bearing with me! I really appreciate it.

No worries, the more we are to look at the code, the better it is, thank you!

Just noticed the displayname for devices are a bit awkward due to zeroconf.
Mine are named like this: figrindan_mopidy_http_tcp_local. See what I can find

For me too.

You changed
self._name = discovery_info["properties"].get("name", self._host)
to
self._name = discovery_info[CONF_NAME]
Is there any particular reason for that?

I thought there was nothing in discovery_info["properties"].get("name") and I wanted to take advantage of the Mopidy server name as defined in the zeroconf key in the http section of mopidy.conf. But it's appending the *_mopidy_http_tcp_local anyways, which is not great. Also, I'm not sure why the entity_id does not generate from self._uuid but from self._name, but it must be a HASS thing.

I'm considering changing it to
self._name = discovery_info["properties"].get("name", self._host) + "@" + str(self._port)
Would that break things on your end?

I think it's great :)

How about
self._name = discovery_info[CONF_NAME][:-22]
to strip off _mopidy_http_tcp_local ? That'smaybe even better!

Sure! Adding the port in the end was a good idea also, in case of someone has multiple Mopidy instances.

There, I've updated the code so the name is supposed to be the real zeroconf name with port number and I've cleaned up the code so it's conform with the HA style guide.
Merci pour votre contribution!

I've pulled the latest code and it's really great like this.
Avec plaisir !


fix: support parsing empty tile terrain values

Description

adjust the parsing of the Tile.terrain list to support empty terrain values
fix https://github.com/flame-engine/tiled.dart/issues/60

Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples.

Breaking Change

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Related Issues

https://github.com/flame-engine/tiled.dart/issues/60

LGTM, but I think title should be "fix!" since it's a breaking change

Ah, because of the type change from Tile List<int> terrain to List<int?> terrain? I suppose that would be - people using that API would see CI failures until they updated to handle nullable elements. I'll update the PR title and checkboxes. I'm not sure about the process wrt updating the changelog and pubspec versions.

I'm not sure about the process wrt updating the changelog and pubspec versions.

melos will do that automatically next time we cut a release based on the PR titles. We decided recently to try to get all the breaking changes in now before we get to v1.0, so it's good timing 👍


Demo site does not work

Hello ! Thank you for sharing your work. Please make a demo site, and there is a lack of documentation to deploy the project.

Hey, thanks for the issue. I tried visiting https://nuxt-headless.netlify.com/ and can see it working. Do you have any more information on the issue?
Deployment should be in the readme. I suggest netlify for static sites or now for node apps

@bovas85    I see Demo site is not working. Could you please look into this?
I am attaching the screenshot.

Thanks for signalling, I'll take a look now

That is fixed now, thanks

I am seeing a picture of Butterfly and when i go down nothing is visible apart from Email. Is this expected?

yes

@bovas85 are these above sites SSR sites or offline generated?  I also happen to see the performance here https://developers.google.com/speed/pagespeed/insights/ it's slightly on the lower side.
Anyway this is not a negative comment, just a observation.

Antilophia is SSR, while others are static.
Unfortunately due to the budget the performance wasn't taken too much into consideration when building them. In terms of client base the performance was deemed acceptable.
https://nunziellasalluce.com is the only one I spent some time on performance gains and should be higher in score I believe.
Feel free to make a PR on that repo if you have any more performance boosts, it'd be appreciated. It's https://github.com/bovas85/nunziellasalluce.com

Yes you are right https://nunziellasalluce.com has good performance boost.
I will see when my site goes on what score i will get :)

Indeed, please let me know if you need any help

Thank you very much for the support.
Have you hidden the actual wordpress content from viewing?
I went here https://api.nunziellasalluce.com/ thinking there will be wordpress pages/posts but none is visible. I see rest-api is accessible.

The admin panel is wp-admin but password protected of course.
REST API is at https://api.nunziellasalluce.com/wp-json/wp/v2/ example for pages : https://api.nunziellasalluce.com/wp-json/wp/v2/pages

Not Admin site. I ma talking about the Default homepage of your wordpress installation. my expectation was you will have posts or pages on that first page. Definitely it's not for clients.
I am not sure whether you are getting me or not.
Meanwhile I tried looking for you in discordapp, could not find you.

Wordpress doesn't serve the frontend at all. It's headless as the name implies. My name in Discord is Moustache Design, should be in the blog post too. However I'm on holiday currently so unlikely to read it

Enjoy your holiday :)
When you say headless are you saying do you do some kind of changes to the wordpress during the installation?  I was under impression it's a regular implementation of wordpress. just that it's front end is not exposed to clients rather nuxt front end is exposed. looks like I am wrong.

Ok looks like you activate kind of blank theme to give a feel of headlless may be

Headless cms means there's no Frontend..
You are looking too much into this.
API. Is not meant to have a Frontend. The website url is completely different


chore: create CODEOWNERS

@crwilcox requested teams for repo access across the board for SoDa teams. Adding a CODEOWNERS file with yoshi-php as default and with the SoDa team.

Yep fair John, closing and filing there


Hide Drift icon when footer is in sight

Feature
Why is this feature being added?
On Desktop, the Drift icon blocks links.
On mobile, the Drift icon will block links when the footer is fixed for mobile (lol). It's currently in the way depending on where you scroll in the footer still.
What should your feature do?
Fade out the drift icon when the <Footer> component is in sight, and fade it back in as the component "drifts" farther away with scrolling.

Trying to find this Drift icon in the src folder. I'm trying to find it along the lines of widgets but I can't seem to find it. Can you point me to where the src code for this Drift icon is located?

Correct me if I'm wrong, @hollomancer
I believe it's a plug-in imported in index.html within /public
https://github.com/OperationCode/operationcode_frontend/blob/master/public/index.html

Drift is some drop in code provided by Drift itself. It's located here: https://github.com/OperationCode/operationcode_frontend/blob/master/public/index.html#L35

Any ideas on implementation @rickr this actually seems tough.

I have a similar feature on my website for the scroll to top button. It doesn't show until the user scrolls down, so it's kind of the opposite. If I wasn't busy with the jobs portal I'd be of more help, but feel free to hack my css. :)

.btn {
	-webkit-transition: all 0.125s ease-in-out 0s;
	   -moz-transition: all 0.125s ease-in-out 0s;
		-ms-transition: all 0.125s ease-in-out 0s;
		 -o-transition: all 0.125s ease-in-out 0s;
			transition: all 0.125s ease-in-out 0s;
}

https://github.com/sethbergman/sethbergman-com/blob/master/assets/js/custom.js#L12

We talked about this on Slack a bit. seth agreed that the above comments won't solve the problem. I'm gonna flag with "Needs Engineer" cuz I haven't gotten to use that flag before, and all my friend's are stumped.

Although @lebilly if you've found a solution, please share - I'm dying to know.

Drift has an API that you can use to hide or show the icon:
https://help.drift.com/article/lVI0NBilKN-widget-api

Until we get the widget configured to hide / show the icon, and since I was already working on the topNav and footer, I've made some CSS tweaks that look to be a quick solution.
Footer:

Header:

Forgive me if I'm wrong, but from that screenshot it looks like the links are on a slightly higher line than the the email in the left section.
Also, I got info on fixing this issue from a React Slack team:
The widget has an ID of #drift-widget making it easy to select in CSS. Knowing that we can easily select it in CSS was half the battle. Now we need to decide how we can operate on that element from within index.css

So we'll probably have to polyfill so we can get to the DOM selectors. The way React works, it is not so friendly with the webcomponent spec, and we're currently using PostCSS for our encapsulated styles. This is a Webpack preprocessor that modifies each CSS selector inside a component to make it unique in order to avoid CSS "leakage". I know Polymer does a good job of getting to these selectors, but I haven't looked at the drift API yet. I think we can just use a webpack plugin to load Sentry, Raven and Drift at compile time. :crossed_fingers:

Here's the video:
https://vimeo.com/223805833#t=53s
I don't have access to login to the dashboard and create the custom controls.

We could just use our global stylesheet, I think.

#402 fixes #346

The global stylesheet won't work as the current snippet is injected outside of the root div where all the React app lives. We'd need to bring the snippet code into its own component in order for this to be feasible IMO.

Per convo in #oc-projects. Kill Drift.
Hide it... permanently.


Add Google Login Provider

As a new Site User, I would like to be able to log in utilizing Google.

#29


feat: disable flood publish

Motivation
Since we increase target peers to 100 we more missed block proposals due to a delay in gossipsub, see #6596
Description
As analysed by lighthouse, flood publishing contribute to this delay so we should experiment disabling it
part of #6596

:tada: This PR is included in v1.19.0 :tada:


Update favicon output

Currently Ghost outputs favicons in the document <head> like this
// If a publication icon is set
<link rel="shortcut icon" href="/favicon.png" type="image/png" />

// If a publication icon is NOT set
<link rel="shortcut icon" href="/favicon.ico" type="image/x-icon" />

This is a slightly dated implementation based on how browsers have evolved, and could do with a small update.
We'd like to update this syntax to work as follows:
// If a publication icon is set
<link rel="icon" href="/favicon.png" type="image/png" />

// If a publication icon is NOT set
// No output in <head>

[ ] rel shortened to just icon
[ ] no output in <head> if no publication icon is set

Hey, i would like to help with that issue.
I saw that it will need changes to the ghost_head.js and the related test file.
There are 2 .hbs files with the line (error.hbs, private.hbs)
<link rel="shortcut icon" href="{{asset "favicon.ico"}}">
shoud  these also be adapted?

I think the favicon references in those two files can be completely removed, yes :)

@JohnONolan I was thinking about this when looking at merging the PR, and realised there is a missing case for subdirectory sites.
For any site with a subdirectory this change means there's no longer a default Ghost favicon - and Ghost will only render the uploaded custom favicon if it is a png, not if it's an .ico.
I think the actual logic we want is to only output the favicon link tag in head if the favicon path is not exactly /favicon.ico, which is what the browser looks for by default.
I also got a bit confused trying to test this, as our icon uploader requires a file that's between 60px and 100px and square - which I think is something to do with, but our default favicon.ico is only 48x48px, which is also the biggest size I can get out of https://favicon.io/favicon-converter/ 😬

Ok after discussing this quickly, my logic is incorrect as well - we don't care about outputting a default favicon for subdirectory sites, it's fine and actually correct to fallback to using the favicon.ico from the root site.
But the logic should be don’t output anything if no custom pub icon is set, which is also not what is implemented on the PR.

Thank you @RenCloud ! :)


Are you planning to update webpack?

Are you planning to update webpack and maybe other components/libs?

updated to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>in<EMAIL_ADDRESS>I had already updated a bunch of the other components previously.
Am working on the rxjs@6 update since it is a bit more involved.


Out of scope fetching

How to reproduce
You need to specify more pages than there's available on the platform for specific role
py octoffers djinni fetch devops --pages 7 # out of 5

Imagine that there's only 5 pages available for devops role, after 5th page djinni will fetch the same general page which is usually do not include devops roles.

#12


Adding a gnark tutorial circuit

Adding a gnark circuit (based on gnark version v0.9.0) that inputs an ascii string and compresses via run-length encoding.
Reviewer Instructions:

[x] Submit ./gnark/compress.tar.gz to forge.sindri.app/ either via the frontend or API code.
[x] Request a proof with JSON input similar to `{"original": "aaabbcccccc"}".  You can alter the string and see how the public output from the proof-detail endpoint changes.
[x] Proofread gnark/compress/README.md for any glaring errors.  More information can be populated here after the API tutorial is written.

Appendix:
Example proof output from proof detail endpoint for the string "aaabbcccccc"
  "public": {
    "Original": [null ,null, null, null, ...],
    "Compressed": [97, 51, 98, 50, 99, 54, 0, 0, ...]
   }

@sangaline LGTM as of commit https://github.com/Sindri-Labs/forge-sample-data/pull/30/commits/2baa8052a48fac3680d9395248e0e8d2c8633fea
Running
npm install
node compile.mjs

gave expected output as well as
go build -o verify verify.go
./verify proof-detail-response.json

Rerunning the verifier after changing a character in the proof or an entry of the public variable resulted in failed verification...as expected.
Great additions!


is it possible to implement quantum natural gradient descent optimzier with torchquantum?

I need to implement QNGD optimizer with torch+torchquantum circuit for optimization purposes. Please let me know and help any source file exists in it.

Hello! I’m fairly certain we don’t currently have an implementation of QNGD yet but would be happy to help as I can if you run into any difficulties!


Radio group field is not responsive

Description
When using the radio group field button variant and you resize the sidebar the button does not resize with it.
Video: https://www.loom.com/share/1366454af483429d9596d27052c37b26
cc @spbyrne

The whole sidebar is not responsive, as it was just set loose of it's default fixed with, same happens with other components like buttons and such.

There are some low-level issues with the way this is handling refs. Tabling the animation challenges and keeping it simple will allow us to move forward quickly

As far as I can tell we are not using this UI anymore


Validation message for duplicate name on creating new client and project should be available at front end

Is your feature request related to a problem? Please describe.
In current implementation the error message is displayed from back end, when we click on create button. The enhancement is required for checking validation in the front end.
Describe the solution you'd like

Not a priority right now.


Integration with RAIL

Since the project code is stable now I'm starting to prepare a release and integrating it correctly with the RAIL ecosystem. There are a few items that are not fully clear to me:

Did I choose the correct path structure in src/? As what kind of algorithm should the code be classified and in which place exactly should the final YawSummarizer stage go?
Where do the examples and notebooks belong? Do they stay in this repository or should they move some of the top level RAIL repos?
It would be great if someone experienced with RAIL could have a look at the repository in general to ensure that the quality requirements are met.
Are there any other steps required other than releasing the package on PyPI and registering it as a RAIL algorithm?

Thanks for setting this up!

The actual informer/summarizer pair should be in a single module in src/rail/estimation/algos. To match style, it would be best for the one module per algorithm containing all informer/estimator/classifier/summarizer stages that use it to be called yaw.py (or yaw_cc.py, yaw_clust_z.py, etc.) without having 'summarizer' in the module name, as we hope to eventually support both summarization and estimation with more methods that can actually be applied in both ways, and users can see what stages are available in a module pretty easily using python's standard help functionality. However, I don't think we've actually achieved standardization on whether anything besides the __init__.py can go in src/rail/[algo-name] (seems like usually no, but there are examples that do have other modules there), and we're a bit all over the place on what other supporting modules can go in src/rail/estimation/algos (and similarly for creation/engines, creation/degraders, and evaluation/metrics). Input from others would definitely be helpful here! For now, I'd recommend at least prepending an underscore if you don't expect users to directly call anything in the supporting modules and ensuring that there aren't already modules with identical names in the RAILiverse (to make it easier for folks to debug something like an import error that could arise in any of the same-named modules that might be hinted at in error messages without showing the full path).
Demos should indeed go in the primary RAIL repo's examples/estimation directory, as that's what gets synced for rendering the notebooks on ReadTheDocs. However, while the package is in development, it's fine to leave them in a top-level `examples' directory.
This should probably be reviewed by more than one person and perhaps not yet, so I'll leave that for later -- definitely worth mentioning it in a tag-up when you're ready!
I think @hangqianjun and @ztq1996 can advise best (and maybe identify if there are any holes in the development principles doc to help address all these questions).

I think one item to add is to add the algorithm to the citations page on the RAIL docs.

The code does not fit into the currently existing summarizers (CatSummarizer, PZSummarizer, SZPZSummarizer) because, even if you wrap/group some of the stages, there are always different numbers or types of inputs as can be seen here:

So subclassing anything is not an option at the moment.
If I understand you correctly @aimalz, you are suggesting that I put all five stages into src/rail/estimation/algos/yaw.py and move the package with the remaining code to something like src/rail/yaw/?

I restructured the code accordingly on the associated branch. Feel free to check if that better matches the RAIL principles.

[x] 1. Code structure
[ ] 2. Collecting all examples and moving them to RAIL
[ ] 3. Code review
[ ] 4. Registering package in rail/rail_packages.yml
[ ] 5. Listing package in rail/docs/source/citing.rst
[ ] 6. Publishing v1.0

Gotcha, thanks for the diagram, that makes a lot of sense. (Also, I would love to make the figures for the v1 paper using whatever you used for this -- let's compare notes offline!)
Organizationally, I don't want to suggest significantly rearranging the code you've written when it's already grouped reasonably given the complexity of the algorithm. I can, however, think of a way to use import statements so users can find the YAW functionality in parallel places relative to other algos. My suggestion is to make two new modules:

src/rail/tools/yaw_handles.py would contain import statements for all the YAW-specific DataHandle subclasses.
src/rail/estimation/algos/yaw_xcorr.py (where "xcorr" could be replaced with "clust_z", "cc_z" or whatever shorthand folks prefer for the spatial correlation/clustering-based approach to non-photometric redshift estimation) would contain import statements for all the YawRailStage subclasses.

Relatedly, I would also propose the following changes to follow the current naming conventions:

YawCacheCreate --> YawCacheMaker (because Creator already means something unrelated in the RAILiverse)
YawCacheDrop --> YawCacheDropper
YawCrossCorrelate --> YawCrossCorrelator
YawAutoCorrelate --> YawAutoCorrelator
YawSummarize --> YawSummarizer

As for where to put the modules currently in src/rail/yaw_rail, I think it would be best to move them out of there because, while I don't know the details of how the namespace trick works, nearly all the other rail_* repositories have nothing but an __init__.py in their equivalent of src/rail/yaw directory (yes, another suggested renaming, sorry), so . (Exceptions include rail_sompz, rail_deepdisc, and rail_bpz, which each have a utils.py that should be renamed to disambiguate if not also moved elsewhere in the directory structure.) In any case, the closest analogue to YAW is rail_tpz, which pretty reasonably has directories under src/rail/estimation/algos, so I'd suggest following that model of moving src/rail/yaw_rail into src/rail/estimation/algos until we establish a standard (and this is something we should definitely add to the development guide).
How does that sound?


Aave V3: update/merge ext

Move Extended schema to the main aave v3 subgraphs. The compiled binaries are the same and the subgraphs should sync instantly. Deployments are set to true for all Aave V3 subgraphs.
Testing:

https://okgraph.xyz/?q=dmelotik%2Faave-v3-arbitrum > did not instantly sync
https://okgraph.xyz/?q=dmelotik%2Faave-v3-avalanche
https://okgraph.xyz/?q=dmelotik%2Faave-v3-fantom > did not instantly sync
https://okgraph.xyz/?q=dmelotik%2Faave-v3-harmony > did not instantly sync
https://okgraph.xyz/?q=dmelotik%2Faave-v3-polygon > did not instantly sync
https://okgraph.xyz/?q=dmelotik%2Faave-v3-optimism

Seems to be syncing quickly, lmk if you want me to graft the ones that did not sync instantly. Otherwise they will start at where my test deployments are synced to. Since those are exactly the same

@ishraq8 FYI we are going to migrate the aave v3 subgraphs from the extended version (i.e. upgrading the aave v3 subgraphs from 1.3.0 to 2.0.1). I can either wait till after you've reviewed 1.3.0 or do it now. Let me know which you prefer.

@this-username-is-taken 1.3.0 and 2.0.1 looks like they don't have errors across multiple chains. Go ahead and pull and i can do a final check after.

@this-username-is-taken should be good, arbitrum / harmony should be the only 2 that won't insta sync


What's the licensing for this?

This is some really cool stuff, would love to build on it a bit and try to introduce some optimizations for us poor plebs with lower-end hardware. But just how open source is this? Am I allowed to make some small edits and re-release it publicly? Will I get sued if I make my own "marble marcher" style game based on this code and sell it for money?

I've added an MIT license to the repository.


Fix resource update

Fix editing existing Resources and improve preview formatting.

There are likely better ways to do some of this. Feel free to edit or strike anything.


feat(Algebra/Homology/Embedding): the left homology of an extension of homological complexes

Given an embedding e : c.Embedding c', K : HomologicalComplex C c, and degrees such that e.f j = j', we construct a left homology data for K.extend e in degree j' from a left homology data for K in degree j.

[x] depends on: #18500

Thanks!
maintainer delegate


Save Icon - Toolbar

If there was the SAVE icon in toolbar it would be quicker to edit shortcuts !
Thanks

Hi. I am very sorry for the delay in offering a response, but I have had this project quite forgotten and abandoned.
Responding to your question, this would be the the prototype:

Additionally, a new option to let the user show or hide that toolbar.
All this it will be implemented in the next release.
This may take me some time (from few days, to weeks) as there are several other Issues in this project that I also need to offer solutions to. Please be patient.
Sincerely,
Elektro.

Perect, and thanks for all.

The fix has already been implemented in the new source code update. See change log.
Please wait for the new release to be published.
If you have more questions feel free to continue this debate. For now, I consider it completed.
And sorry again for the delay in my answers.


Zar import sortmem

This adds -sortmem flag to zar import.
It works as expected in the code, but so far, it only make sense to use the flag for sorting zeek log files of less than 100 MB. For larger files, setting $TMPDIR to spill to disk (Ramdisk or SDD) gives better performance. For zeek files of more than a few GB, the -sortmem option can cause the zar process to run out of memory.
There are a series of PR comments below that I am leaving in to capture the results of multiple tests of this feature.

It is also worth mentioning that the '-sortmem' approach may not be very resource efficient:
When I tried running on a lower memory instance (a c5d.large with 4GB RAM rather the the 8 GB Arm on the m5d.large) the process ran out of memory.
So, the -sortmem=2000000000
Which would use at least 2GB, actually chewed up enough RAM to exceed 4GB. The process completed on the 8GB instance, but I did not measure max process memory.

I will leave comments on measurements in this PR.
I checked to see it the "chuck size", that is the '-s' flag has an effect on IO wait. It does not appear to be a significant factor. Here are two run with chuck side of 25MB and 90 MB:
[ec2-user@ip-172-31-19-39 ~]$ time ~/go/bin/zar import -s 25MB -R s3://zqd-demo-1/mark/zeek-logs/dns5 s3://brim-sampledata/wrccdc/zeek-logs/dns.log.gz
Doing an external merge sort that spills to disk, MemMaxBytes=134217728
real	17m3.258s
user	2m11.579s
sys	0m17.645s
[ec2-user@ip-172-31-19-39 ~]$ time ~/go/bin/zar import -s 90MB -R s3://zqd-demo-1/mark/zeek-logs/dns5 s3://brim-sampledata/wrccdc/zeek-logs/dns.log.gz
Doing an external merge sort that spills to disk, MemMaxBytes=134217728
real	16m45.278s
user	2m10.630s
sys	0m15.954s
[ec2-user@ip-172-31-19-39 ~]$

To isolate the IO problem, I ran the same import in stages, using a local ramdisk. This eliminated all the IO wait, and the overall process ran as fast as could be expected given the CPU time required and the 10 Gigabit/sec transfer rate to S3.
This is strong evidence that the IO wait is coming from the S3 access in zar import.

Copy the 264 MB S3 object to a local ramdisk:
$ sudo mkdir /mnt/ramdisk
$ sudo mount -t tmpfs -o size=1024m tmpfs /mnt/ramdisk
$ time aws s3 cp s3://brim-sampledata/wrccdc/zeek-logs/dns.log.gz /mnt/ramdisk/dns.log.gz
download: s3://brim-sampledata/wrccdc/zeek-logs/dns.log.gz to ../../mnt/ramdisk/dns.log.gz

real	0m2.391s
user	0m1.772s
sys	0m1.126s
Note that the time to copy is just 2.4 seconds -- this is perfectly in line with the expected performance on this EC2 instance of a 10 Gigabit/s transfer rate from S3.

Run the zar import on dns.log.gz, sending the output back to the ramdisk. We use the -sortmem flag to make sure we do not spill to local disk.

$ time ~/go/bin/zar import -s 25MB -sortmem<PHONE_NUMBER> -R /mnt/ramdisk/dns-dir /mnt/ramdisk/dns.log.gz
Doing a single in-memory sort, MemMaxBytes=2000000000
real	1m5.028s
user	1m11.120s
sys	0m2.105s
The real time is slightly less than the CPU time, since some small portion of the user CPU time can run in parallel.

Copy the ZNG file back to S3, using the sync command. We omit most of the output here to get straight to the timing:

$ time aws s3 sync /mnt/ramdisk/dns-dir s3://zqd-demo-1/mark/zeek-logs/dns-dir
upload: ../../mnt/ramdisk/dns-dir/20170324/1490386867.002293.zng to s3://zqd-demo-1/mark/zeek-logs/dns-dir/20170324/1490386867.002293.zng
...
real	0m6.087s
user	0m3.221s
sys	0m1.767s
The sync takes longer than the download from s3, which is expected, since the ZNG files are about 50% larger and are divided into multiple directories.

Isolating this further, based on Al's suggestion, the following run reads from local ramdisk and writes to S3:
$ time ~/go/bin/zar import -s 25MB -sortmem<PHONE_NUMBER> -R s3://zqd-demo-1/mark/zeek-logs/dns-dir2 /mnt/ramdisk/dns.log.gz
Doing a single in-memory sort, MemMaxBytes=2000000000
real	1m11.241s
user	1m16.770s
sys	0m2.763s
It is faster than steps (2) and (3) of the previous run combined.
So the problem seems to be just with the way zar import reads from S3. The write pattern seems fine.
I I had to guess, I would say the our use of the "Range" parameter on the s3.GetObject is problematic.
https://github.com/brimsec/zq/blob/cae3cca70957b7d656bf6a28a1113cc1468af1d1/pkg/s3io/reader.go#L82
getObj := &s3.GetObjectInput{
	Bucket: aws.String(r.bucket),
	Key:    aws.String(r.key),
	Range:  aws.String(bytesRange(off, len(p))),
}

We should see how many separate calls we are doing for S3 with a Range parameter. I will investigate that now.

I added a counter and println to s3io.ReadAt.
Running zar on a small log (about 5 MB), we did nearly 1500 reads from S3:
$ time ~/go/bin/zar import -R s3://zqd-demo-1/mark/sample-http-zng s3://zq-sample-data/zng/http.zng.gz
...
1495 s3io.ReadAt s3.GetObjectInput: zng/http.zng.gz
Doing a single in-memory sort, MemMaxBytes=134217728
real	0m21.333s
user	0m1.655s
sys	0m0.263s
Next I'm going to try running a zq query on the same file to see how often it reads.

I won't use this branch, but this closed PR will be a record of the testing.


Remove version: Tableau.Reader version 24.2.931

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://downloads.tableau.com/tssoftware/TableauReader-64bit-2024-2-2.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 896)


[MessageBar] Add an optional toggle for the fade in animation.

Pull Request
📖 Description
Add a options to disable the animation in the MessageBar component.
👩💻 Reviewer Notes
📑 Test Plan
Existing message bar tests have been updated with the new animation CSS class. And a new test has been added.
✅ Checklist
General

[x] I have added tests for my changes.
[x] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[x] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[x] I have added Unit Tests for my new component
[x] I have modified an existing component
[x] I have validated the Unit Tests for an existing component

⏭ Next Steps

@yspanj please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

Great work.
Just a quick discussion on "how the browser can detect when to apply the animation?".
In general, I prefer to use an HTML element attribute like this. I found the code cleaner and easier for future updates.
What are your thoughts?

   ...

.fluent-messagebar[animation="fadein"] {
  animation: fadein 1.5s;
}

animation

Thanks!
I agree that the HTML element attribute looks cleaner. But I think that that would resolve in a bigger change is how the components is created.
Are you thinking about implementing something like an AnimationBuilder similar to the CssBuilder? Or am I missing something that's already in the library that can be used for this?

Are you thinking about implementing something like an AnimationBuilder similar to the CssBuilder? Or am I missing something that's already in the library that can be used for this?

No, nothing exists about that. And I don't think we need to use an AnimationBuilder: FluentUI Blazor uses the WebComponents which will do this part.
A MessageBar is a component to "Communicates important information about the state of the entire application". By default, it is not necessary to display an animated message like explained in the FluentUI React components. Probably a difference between the Toast.
But we already display the MessageBar using this animation :-(
Si I think the code should be clean (using a attribute) and keeping the existing behavior (FadeIn=true like you did).

@yspanj we are pulling this PR in now so it will be available in v4.10.1.
Please submit a follow-up PR to change the behavior to use an attribute so we can include that for a later version. Thanks!

I agree that the HTML element attribute looks cleaner. But I think that that would resolve in a bigger change is how the components is created.
Are you thinking about implementing something like an AnimationBuilder similar to the CssBuilder? Or am I missing something that's already in the library that can be used for this?

You can just set the attribute to the fadein value if the parameter is true and otherwise make it null. Empty attributes won't be rendered. Something like:

   ...

@yspanj we are pulling this PR in now so it will be available in v4.10.1.
Please submit a follow-up PR to change the behavior to use an attribute so we can include that for a later version. Thanks!

I agree that the HTML element attribute looks cleaner. But I think that that would resolve in a bigger change is how the components is created.
Are you thinking about implementing something like an AnimationBuilder similar to the CssBuilder? Or am I missing something that's already in the library that can be used for this?

You can just set the attribute to the fadein value if the parameter is true and otherwise make it null. Empty attributes won't be rendered. Something like:

   ...

Thanks.
Opened the pull request #2723 for this.


fix: Replace topEnv with  check of input string.

When I wrote the PR for {equation}, {align}, etc., I included a parameter named topEnv. I was trying to follow up on suggestions in issue #2042, but I did not have a firm grasp on what topEnv was supposed to accomplish. I ended up using topEnv to try to ensure that AMS environments were written at the top level.
I believe I botched that part. Currently master will return a ParseError for any AMS environment if the rendering options do not include either (1) strict = false, or (2) topEnv = true.
That's a weird condition to impose on users. So this PR removes topEnv and instead checks the input string to see if the environment was defined at the very start. It's a better check. I was even able to strengthen some of the unit test by changing strict from false to "warn".
There is another issue, #2381, which suggests allowing KaTeX input strings to include the $ delimiters that many editors use to delineate math. I think that is a pretty good idea, but it is a different thing and belongs in a different PR. Right now, topEnv is just an awkward, unnecessary check parameter and it should go. We can address #2381 later.

Codecov Report

Merging #2709 (29bd2ba) into master (5c44c47) will not change coverage.
The diff coverage is 75.00%.

@@           Coverage Diff           @@
##           master    #2709   +/-   ##
=======================================
  Coverage   93.93%   93.93%
=======================================
  Files          87       87
  Lines        6294     6294
  Branches     1293     1293
=======================================
  Hits         5912     5912
  Misses        352      352
  Partials       30       30

Impacted Files
Coverage Δ

src/Settings.js
81.63% <ø> (-0.37%)
:arrow_down:

src/environments/array.js
98.09% <75.00%> (+<0.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 5c44c47...29bd2ba. Read the comment docs.


Feature request: setting to auto-run showPretty command on file open

@haixuanTao the extension you linked to uses a webview for its preview feature, losing some features in the process.
I explicitly made this extension to use VSCode's built-in editors instead of webviews. This means that, for example, any extension that inspects an editor's text will continue to work with this preview - like a spell checker, for example. Especially since, when I began development, even basic things, like Ctrl+F used to only work in built-in editors.
Unfortunately, this prevents me from easily implementing the preview-by-default feature. Don't want to go deep into technical details here, but I've tried to come up with a solution, and found none.

Leaving this issue open until I either find a way to do it, or document this limitation in the README.


HTTP Server and Upstream Timeouts

Related to issue 263. Permitting the users to set varioues timeouts on the http.Server and upstream proxy

sorry for the delay @jangaraj ... can you review please?

LGTM 👍 . Note: default non zero values for server read/write timeouts can be breaking change, because default Golang zero values are used at the moment = no timeouts. Probably it'll be good to mention it in breaking changes section.
Thanks!

good point ... i'll put it into the changelog!


Certain Resolutions Cause tokio-runtime-worker Panic

Running the publish_hue.py example with WIDTH, HEIGHT = 2560, 1920 or WIDTH, HEIGHT = 5120, 960 results in participant joining the room but without publishing the hue video. The behavior is accompanied by the following output:
INFO:root:connected to room my-room
thread 'tokio-runtime-worker' panicked at livekit/src/room/options.rs:184:5:
internal error: entered unreachable code
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Larger resolutions, such as 4K UHD WIDTH, HEIGHT = 3840, 2160 work fine.
Reproduce by running the publish_hue example from inside the following Dockerfile:
FROM ubuntu:20.04
ENV DEBIAN_FRONTEND=noninteractive
RUN apt-get update && apt-get install -y curl python3.9-dev=3.9.5-3* python3-pip
RUN curl -sSL https://get.livekit.io | bash
RUN python3.9 -m pip install --upgrade pip
RUN python3.9 -m pip install livekit==0.6.0 livekit-api==0.3.0 numpy opencv-python-headless

This will be fixed on the next release

You can already try 0.7.0.dev1


Fix Language Server diagnostics timer publish requiring foreground thread

This was a basic oversight in the publishing of Razor diagnostics. Pretty much if there was work queued while publishing existing diagnostics then the re-queue of work would explode because we'd try to start the diagnostics publish timer on the background thread.
Added tests to validate that work re-queuing works.

aspnet/AspNetCore#14324

@dougbu or @wtgodbe would you mind merging this for me?