[FEATURE]Add dedup PPL Command

Is your feature request related to a problem?
Add dedup command for PPL spark based execution driver
PPL Dedup
Do you have any additional context?

https://github.com/opensearch-project/opensearch-spark/issues/408
https://github.com/opensearch-project/sql/blob/d3ecb2d75712db2bcbd2945e62728c6599580657/docs/user/ppl/cmd/dedup.rst

I'm starting to contribute it.

Design Doc
Syntax
dedup [int] <field-list> keepempty=<bool>] [consecutive=<bool>]

int also known as allowedDuplication, the number of duplicate events must be greater than 0.
We can separate this feature to 2 parts:

allowedDuplication = 1
allowedDuplication > 1

Part 1
For allowedDuplication = 1,  we translate following three PPL dedup commands to different Spark Logical Plans. (assuming the <field-list> is a, b)

keepempty=false: | dedup [1] a, b [keepempty=false]

Deduplicate ['a, 'b]
+- Filter (isnotnull('a) AND isnotnull('b))
     +- Project
          +- UnresolvedRelation

keepempty=true : | dedup [1] a, b keepempty=true

Union
:- Deduplicate ['a, 'b]
:   +- Filter (isnotnull('a) AND isnotnull('b))
:        +- Project
:             +- UnresolvedRelation
+- Filter (isnull('a) OR isnull('b))
     +- Project
          +- UnresolvedRelation

consecutive=true: | dedup [1] a, b [keepempty=true] consecutive=true

UnsupportedOperationException("Consecutive deduplication is not supported")

Question
Deduplicate or DeduplicateWithinWatermark?
https://issues.apache.org/jira/browse/SPARK-42931 introduced a new LogicalPlan DeduplicateWithinWatermark to deduplicate events within the watermark and was available in 3.5.0+
I am not clear the architecture of Flint, but seems it performs on Spark Streaming. Deduplication on streaming dataset has different semantic with batch dataset:

We document the behavior clearly that the event time column should be a part of the subset columns for deduplication to clean up the state, but it cannot be applied to the customers as timestamps are not exactly the same for duplicated events in their use cases.

Need more guidance on the purpose of this repository and how it should be used in production or in future. For current time-being, I choose Deduplicate in this PR (DeduplicateWithinWatermark required 3.5.0+).
Part 2
To translate dedup command with allowedDuplication > 1, such as | dedup 2 a,b to Spark plan, the solution is translating to a plan with Window function (e.g row_number) and a new column row_number_col as Filter.

For | dedup 2 a, b keepempty=false

Project ['a, 'b] // ensure the row_number_col is removed. not a and b only, should contain all fields except row_number_col
+- Filter ('row_number_col <= 2) // allowed duplication = 2
   +- Window [row_number() windowspecdefinition('a, 'b, 'order_key ASC NULLS FIRST, specifiedwindowframe(RowFrame, unboundedpreceding$(), currentrow$())) AS 'row_number_col], ['a, 'b], ['order_key ASC NULLS FIRST]
       +- Filter (isnotnull('a) AND isnotnull('b)) // keepempty=false
          +- Project
             +- UnresolvedRelation

For | dedup 2 a, b keepempty=true

Union
:- Project ['a, 'b]
:  +- Filter ('row_number_col <= 2)
:     +- Window [row_number() windowspecdefinition('a, 'b, 'order_key ASC NULLS FIRST, specifiedwindowframe(RowFrame, unboundedpreceding$(), currentrow$())) AS 'row_number_col], ['a, 'b], ['order_key ASC NULLS FIRST]
:        +- Filter (isnotnull('a) AND isnotnull('b))
:           +- Project
:              +- UnresolvedRelation
+- Filter (isnull('a) OR isnull('b))
   +- Project
      +- UnresolvedRelation

Question
What is the 'order_key in Window node?
AFAIK, there always should be a time series field existing for log analysis, such as _time or @timestamp. How to get it in AST?
If we cannot point out an existing time series field as order_key, the syntax of dedup command might change to
| dedup 2 a, b order by c
| dedup 2 a, b orderkey=c

FYI: In Spark, window function requires window to be ordered. Query fails if order by clause is not specified. For example, SELECT <wf>(expr) OVER ([PARTITION BY window_partition] ORDER BY window_ordering) from table
But in MySQL or opensearch-sql, the order by clause can be omitted.


Update msbts-hostinstance-install-method.md

Update according to KB5029626.

#sign-off

@ecfan, can you help merge this? Thanks.


Add cache to avoid loading committee from KVStore for every block

          This is a hot path that reads data from the committee KVStore for each block - we should add a cache at some level. Maybe we should have a committee cache in SeatManager and clear it whenever SybilProtection.Reset() is called? Not necessarily as part of this PR, but then we should create an issue for that.

Originally posted by @piotrm50 in https://github.com/iotaledger/iota-core/pull/673#discussion_r1461859333
Might also be an issue in other places

The extend of this can be easily seen when running a test with a big committee: almost 2/3 of the entire runtime is spent loading the committee.

Steps to reproduce:
go test -run=TestBigCommittee -tags rocksdb,dockertests -v -failfast=true -count=1 -cpuprofile big-committee.profile.out
go tool pprof -http=:8181 big-committee.profile.out

file: big_commitee_test.go
package tests

import (
	"fmt"
	"testing"

	"github.com/samber/lo"

	"github.com/iotaledger/iota-core/pkg/testsuite"
	iotago "github.com/iotaledger/iota.go/v4"
)

func TestBigCommittee(t *testing.T) {
	var (
		genesisSlot       iotago.SlotIndex = 0
		minCommittableAge iotago.SlotIndex = 2
		maxCommittableAge iotago.SlotIndex = 4
	)

	ts := testsuite.NewTestSuite(t,
		testsuite.WithProtocolParametersOptions(
			iotago.WithTimeProviderOptions(
				genesisSlot,
				testsuite.GenesisTimeWithOffsetBySlots(1000, testsuite.DefaultSlotDurationInSeconds),
				testsuite.DefaultSlotDurationInSeconds,
				3,
			),
			iotago.WithLivenessOptions(
				10,
				10,
				minCommittableAge,
				maxCommittableAge,
				5,
			),
		),
	)
	// defer ts.Shutdown()

	for i := 0; i < 32; i++ {
		nodeName := fmt.Sprintf("node%d", i)
		ts.AddValidatorNode(nodeName)
	}

	for i := 32; i < 50; i++ {
		nodeName := fmt.Sprintf("node%d", i)
		ts.AddNode(nodeName)
	}

	ts.Run(true)
	fmt.Println("TestBigCommittee")

	ts.IssueBlocksAtSlots("", lo.RangeFrom(iotago.SlotIndex(1), 5), 4, "Genesis", ts.Nodes(), true, false)

	ts.AssertNodeState(ts.Nodes(),
		testsuite.WithLatestCommitmentSlotIndex(3),
		testsuite.WithEqualStoredCommitmentAtIndex(3),
	)
}


Database notifications component does not listen to DatabaseNotificationsSent event

Package
filament/filament
Package Version
v3.0.0-beta2
Laravel Version
v.9.52.10
Livewire Version
v3.0.0-beta.1
PHP Version
PHP 8.2.7
Problem description
After I installed the broadcasting functionality (via Pusher) for Filament correctly, the database notifications component does not listen to the broadcast event that a database notification is sent, which should refresh the component. The broadcast functionality was setup properly as a broadcast notification is received. In addition, the database-notifications.sent event is received in Pusher.
Expected behavior
The expectation is that the database notifications component is refreshed after the event dispatches.
Steps to reproduce
For the reproduction repository:

Setup the repository as with the regular Filament Demo
Add working Pusher keys to the .env file
Go to the dashboard
Push the button 'Send notification'
You can see that nothing happens. If you refresh the page, you'll see that the notification was sent
When you manually call window.dispatchEvent(new CustomEvent('EchoLoaded')) and you push the button again, the component will refresh properly

Reproduction repository
https://github.com/damiandejong/filament-echo
Relevant log output
No response

See #7263 for a possible fix.

Fixed by #7263


Pre-polulate steps view in execution reporting

Is your feature request related to a problem? Please describe.
Currently, the playbook executions steps get added to the execution information only when triggered. Hence, a retrieved execution report only shows the steps that are currently in execution, or have been executed, thus reporting a partial overview over the whole execution status.
Describe the solution you'd like
We should add a "not yet executed" status value for the step execution report properties. With this, we should populate all steps of a playbook being executed with related status. When updating a step, we can check on that status to determine whether the status precondition for the given update operation is correct.

@MaartendeKruijf  @lucamrgs  as i have been working on the SOARCA-GUI, providing the playbook name, and action name fields would be nice and greatly reduce additional queries to the backend.

@lucamrgs  @MaartendeKruijf  What are your thoughts?


chef_gem's compile_time property must only be set if it is available

Instead of just setting the property, it should be checked whether it is available on the resource for backwards compatibility. Compare Chef 12.1.0 chef_gem resource warnings, section "Community Cookbooks".
recipes/default.rb L23  and test/fixtures/cookbooks/test/recipes/chef_vault_secret.rb L33 should read
  compile_time true if respond_to?(:compile_time)

Sorry about that. Fixed in the latest patch release

Things happen. Thanks for the quick fix.

This is still broken for me, On chef 11.16.4
  ================================================================================
  Recipe Compile Error in /var/chef/cache/cookbooks/chef-vault/recipes/default.rb
  ================================================================================

  ArgumentError
  -------------
  wrong number of arguments (1 for 0)

  Cookbook Trace:
  ---------------
    /var/chef/cache/cookbooks/chef-vault/recipes/default.rb:23:in `block in from_file'
    /var/chef/cache/cookbooks/chef-vault/recipes/default.rb:21:in `from_file'

  Relevant File Content:
  ----------------------
  /var/chef/cache/cookbooks/chef-vault/recipes/default.rb:

   16:  # distributed under the License is distributed on an "AS IS" BASIS,
   17:  # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   18:  # See the License for the specific language governing permissions and
   19:  # limitations under the License.
   20:
   21:  chef_gem 'chef-vault' do
   22:    version node['chef-vault']['version']
   23>>   compile_time true if respond_to?(:compile_time)
   24:  end
   25:
   26:  require 'chef-vault'
   27:

  Running handlers:
  Running handlers complete
  Chef Client failed. 0 resources updated in 20.247337875 seconds

@martinisoft Could you provide a Stacktrace leading to the error?
You should see something like FATAL: Stacktrace dumped to ... in the chef output, telling you where to find it.

@sunruh Seems this was rolled back already and I don't have the log handy anymore. Apologies for that.


Performance Improvement ItemTag

After migrating to Zabbix 5.4 we see a huge performance impact on Grafana.
With Zabbix 5.4 (where applications are replaced by item tags) our Grafana dashboards are loading much slower (some of them even time out).
Before Zabbix 5.4 you filtered on the application id 'appids' if an application was filled in.
But since Zabbix 5.4 applications no longer exist, so the value will always be 'nil'.
Looking into the code in pkg/zabbix/methods.go I see this:
var allItems []*Item
if len(appids) > 0 {
  allItems, err = ds.GetAllItems(ctx, nil, appids, itemType)
} else if len(hostids) > 0 {
  allItems, err = ds.GetAllItems(ctx, hostids, nil, itemType)
}

if isZabbix54orHigher && itemTagFilter != "" {
  allItems, err = filterItemsByTag(allItems, itemTagFilter)
  if err != nil {
    return nil, err
  }
}

If I understand it well, you are querying all items without an itemTag filter and then you do the filtering on the Grafana side (func filterItemsByTag).
Before 5.4 the application filter was taking into account in this api call.
So now when we query on a lot of hosts and specify a specific item tag, it will still query all items on all of these hosts regardless of the item tag.
It is only afterwards that in the plugin itself you will filter on the item tag.
For us this makes a huge difference since sometimes we have 200 hosts with 5000 items each. That's a lot of items and a lot of data to query.
I think this is not really scalable as it returns too much items from Zabbix.
Better would be to include the ItemTag filter directly in the Zabbix-api call instead of doing the filtering on the Grafana side after having returned a bunch of items.
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

8.1.5
5.4.4
4.2.3

Yes, you're right. Unfortunately, Zabbix does not provide a separate API method for pulling list of tags, so I have to get it from items. But I definitely can improve performance by querying only items with specific tags.

Hello,
Do you have an idea when this could be implemented?
For us grafana is unusable for the moment, because we have many items on 1 host and the performance is really killing us due to the tags and how it's implemented.
Thanks!
Erik

Fixed in c1b0d963cd7375b69e050b91f3989be87cecc16b


AdHoc created needs partially inherit need details

requires #2520
When creating an AdHoc need, that need copies details from the original need. This is currently an issue for locations, as only the location coordinates, but not the human readable name are added to the AdHoc need. This causes an error during location parsing.
To reproduce:

Create a need with a location
With another account, create an AdHoc connection to that need, either via share link or via hint from WhatsX
Check the RDF representation of the AdHoc need to see incomplete location information (i.e. the coordinated from the first need, but not the place name)

there is no need in duplicating the data within the response needs at all, this does not serve any purpose other than clutter, currently we only add certain details anyway (only title, description, tags, location) but instead of figuring out a way to add all the content of the remoteNeed into the seeks of the direct Response need, and adding "Direct Response to...." or "Re: ...." we should simply create the need with an added flag and add the needUri as a responseToNeed detail as well


Exception: Class could not be loaded

I've watched the video on Laracasts, tried the code listed in README for this extension, and played around with various different incantations of including the extensions but nothing works. No matter what I do it always comes up with the below error when I run behat:
[Behat\Testwork\ServiceContainer\Exception\ExtensionInitializationException]
`Laracasts\Behat\ServiceContainer\BehatExtension` extension file or class could not be located.

First attempt from my behat.yml based on the video:
  extensions:
    Laracasts\Behat\ServiceContainer\LaracastsExtension: ~
    Behat\MinkExtension\ServiceContainer\MinkExtension:
        default_session: laravel
        laravel: ~

Second attempt, based on discovering the class name had changed and the comments:
  extensions:
    Laracasts\Behat\ServiceContainer\BehatExtension: ~
    Behat\MinkExtension\ServiceContainer\MinkExtension:
        default_session: laravel
        laravel: ~

Third attempt based on what the repository says:
default:
    extensions:
        Laracasts\Behat:
            # env_path: .env.behat
        Behat\MinkExtension:
            default_session: laravel
            laravel: ~

Am I missing something here? Fatal errors every time.
(Oh the stinging irony in wanting to test but not being able to get the test runner to run. Ha!)

The solution to this for some ungodly reason is not to use a global installation of Behat but, instead, to specifically call /vendor/bin/behat in your project root. Seems to be a new issue with the 3.x branch. If anyone actually knows why that is, feel free to post additional info. Hopefully this helps someone else and spares them 2 hours. :+1:


Add context manager for adding DLL directories to the search path

The list of DLL directories is read from the specified environment variable and added
to the Windows search path for DLLs. This only applies to Python 3.8+.
Any values in the environment variable that are not directories are ignored.
I've updated https://github.com/ros2/rclpy/pull/558 to demonstrate how this is used. After a review, I'll update other Python packages that need a similar patch.

Linux
Linux-aarch64
macOS
Windows


v2.3 Install after Uninstalling and removing config from previous install

Completely removed all configuration for previous versions.. whether the Scenes from the Sonrig are still somewhere in there ... no showing in the scenes menu though
Install via HACS 2.3
No Devices attached, start config message in UI

Logs from HASS
2024-10-20 02:30:39.787 WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to import_module with args ('custom_components.dirigera_platform.config_flow',) inside the event loop by integration 'config' at homeassistant/components/config/config_entries.py, line 179: return await super()._post_impl(request, data) (offender: /usr/src/homeassistant/homeassistant/loader.py, line 1296: return importlib.import_module(f"{self.pkg_path}.{platform_name}")), please create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+config%22
For developers, please see https://developers.home-assistant.io/docs/asyncio_blocking_operations/#import_module
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module>
    sys.exit(main())
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main
    exit_code = runner.run(runtime_conf)
  File "/usr/src/homeassistant/homeassistant/runner.py", line 189, in run
    return loop.run_until_complete(setup_and_run_hass(runtime_config))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once
    handle._run()
  File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/local/lib/python3.12/site-packages/aiohttp/web_protocol.py", line 477, in _handle_request
    resp = await request_handler(request)
  File "/usr/local/lib/python3.12/site-packages/aiohttp/web_app.py", line 559, in _handle
    return await handler(request)
  File "/usr/local/lib/python3.12/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 92, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 210, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 26, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 85, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 242, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/headers.py", line 32, in headers_middleware
    response = await handler(request)
  File "/usr/src/homeassistant/homeassistant/helpers/http.py", line 73, in handle
    result = await handler(request, **request.match_info)
  File "/usr/src/homeassistant/homeassistant/components/http/decorators.py", line 81, in with_admin
    return await func(self, request, *args, **kwargs)
  File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 74, in wrapper
    return await method(view, request, data, *args, **kwargs)
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 172, in post
    return await self._post_impl(request, data)
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 179, in _post_impl
    return await super()._post_impl(request, data)

2024-10-20 02:30:39.788 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading flow for integration dirigera_platform: cannot import name 'EndTriggerEvent' from 'dirigera.devices.scene' (/config/deps/lib/python3.12/site-packages/dirigera/devices/scene.py)

Tried the install 1.8.9 and then upgrade to 2.3 and get the following..

2024-10-20 23:24:33.154 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration 'dirigera_platform': Unable to import component: cannot import name 'EndTriggerEvent' from 'dirigera.devices.scene' (/config/deps/lib/python3.12/site-packages/dirigera/devices/scene.py)
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1007, in async_get_component
    comp = await self.hass.async_add_import_executor_job(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1067, in _get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/util/loop.py", line 200, in protected_loop_func
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/config/custom_components/dirigera_platform/__init__.py", line 10, in <module>
    from .ikea_gateway import ikea_gateway
  File "/config/custom_components/dirigera_platform/ikea_gateway.py", line 18, in <module>
    from dirigera.devices.scene import Trigger, TriggerDetails, EndTriggerEvent
ImportError: cannot import name 'EndTriggerEvent' from 'dirigera.devices.scene' (/config/deps/lib/python3.12/site-packages/dirigera/devices/scene.py)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 334, in _async_setup_component
    component = await integration.async_get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1027, in async_get_component
    self._component_future.result()
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1019, in async_get_component
    comp = self._get_component()
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1067, in _get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/util/loop.py", line 200, in protected_loop_func
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/config/custom_components/dirigera_platform/__init__.py", line 10, in <module>
    from .ikea_gateway import ikea_gateway
  File "/config/custom_components/dirigera_platform/ikea_gateway.py", line 18, in <module>
    from dirigera.devices.scene import Trigger, TriggerDetails, EndTriggerEvent
ImportError: cannot import name 'EndTriggerEvent' from 'dirigera.devices.scene' (/config/deps/lib/python3.12/site-packages/dirigera/devices/scene.py)

Okay.. the dirigera dependency didn't update... in deps/lib/python3.12/site-packages
@sanjoyg I'm not sure on the deployment flow of HACS in this case..
My resolution was to remove the old package from the above folder
Run
pip install dirigera and ensure it was in the same location..

I'm running Home Assistant via TrueNAS app store. Dirigera integration wouldn't work since 2.0 update. Your solution helped me fix it.
Thanks @s3than

Closing this as it's a install issue.. :shrug:


Is there a way to only allow select at lowest level?

We are using v1.17, and we only want to select at the deepest level of the tree as per this photoshopped mock-up.  Is this possible in 1.17?  If not is it available in V2.5.0?

The way it is now:

Yes, set the allowBatchSelection param to false.

Thanks for the quick response. Is this adjustable to perhaps the bottom 2 levels?

No, it's an on or off option.


tfsec ignores

what

tfsec ignores

why

references

@andycognitoiq looks like this was opened by mistake, can you close it?

@andycognitoiq friendly ping on this PR -- Can you please share if this was a mistake to open upstream or your reasoning behind this PR?

closing


fix middleware getting duplicated at api level

Codecov Report
Patch coverage: 88.67% and project coverage change: -0.06 :warning:

Comparison is base (13a77b3) 85.81% compared to head (4839561) 85.75%.

:exclamation: Current head 4839561 differs from pull request most recent head 7a34a50. Consider uploading reports for the commit 7a34a50 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #168      +/-   ##
===========================================
- Coverage    85.81%   85.75%   -0.06%
===========================================
  Files           82       82
  Lines         1445     1453       +8
  Branches       143      157      +14
===========================================
+ Hits          1240     1246       +6
- Misses         203      204       +1
- Partials         2        3       +1

Impacted Files
Coverage Δ

src/api/queues/v0/queues.ts
96.15% <ø> (ø)

src/resources/bucket.ts
73.68% <0.00%> (ø)

src/resources/topic.ts
86.66% <0.00%> (ø)

src/faas/v0/context.ts
76.22% <20.00%> (ø)

src/api/errors/deadline-exceeded.ts
25.00% <25.00%> (ø)

src/faas/v0/start.ts
32.50% <50.00%> (ø)

src/resources/secret.ts
77.77% <50.00%> (ø)

src/resources/common.ts
96.22% <66.66%> (-0.07%)
:arrow_down:

src/resources/queue.ts
78.37% <75.00%> (-0.57%)
:arrow_down:

src/resources/collection.ts
77.27% <80.00%> (+0.52%)
:arrow_up:

... and 32 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Allow the WSFed Resolve to only release credential attributes

Presently, the resolver releases both the credential (ADFS) and the principal attributes. Ideally, the external authN provider should be trusted with all attributes it's releasing since it's the authoritative subject on which attributes, how and when should be released. CAS should just act as proxy that it is, moving attributes back and forth from the IDP to the SP.
Custom use cases that require principal attributes, well, have to be handled in a custom manner.

This should be merged back into 4.2 and 4.1


Document code blocks can't be highlight selected

Document code blocks can't be highlight selected.
Need to be able to select code lines to copy and paste.
Tested on ubuntu 21.04 chrome

this is skateui docs issue.
closing


Support VILA with lmdeploy

Hi, thanks for your great work! This is a feature request, it will be great if VILA can be supported by the lmdeploy framework.

Thanks for the suggestion. So far we do not have bandwitdth to support vila in other frameworks. You are welcome to submit PR if you have any good trials.


Html5History and query parameter

Hi,
if i'm on this url: /test?a=1 and i use setToken to move to /test?b=2 i get the url rewritten like this:
/test?b=2?a=1
how can i solve this?
Here is the code if you want to take a look:
https://github.com/open-company/open-company-web/blob/finances-component/src/open_company_web/router.cljs#L27

Questions should be asked on the public group closure-library-discuss.

For anyone looking for a solution, see here: https://gist.github.com/andrejewski/3887f205fd834eea1b506a908db76e38


CQ: Aggregation grouping by tagging - Not working

Hello,
Just upgraded to the last master verison #fb83158 to test drive the new SE: tsm1, and I have just noticed the following issue:
I can no longer create a CQ using the grouping by tagging 'server' feature:
Does no longer works:
CREATE CONTINUOUS QUERY cpu_coreUtilization_average_percent_1h ON ESXi BEGIN
SELECT MAX(value)/100 as MAX, MIN(value)/100 as MIN INTO cpu_coreUtilization_average_percent_1h FROM "raw"."cpu_coreUtilization_average-percent" WHERE instance = 'avg' GROUP BY time(1h), server
END

ERR: error parsing query: found SERVER, expected identifier, string, number, bool at line 1, char 265

This works:
CREATE CONTINUOUS QUERY cpu_coreUtilization_average_percent_1h ON ESXi BEGIN
SELECT MAX(value)/100 as MAX, MIN(value)/100 as MIN INTO cpu_coreUtilization_average_percent_1h FROM "raw"."cpu_coreUtilization_average-percent" WHERE instance = 'avg' GROUP BY time(1h), *
END

You have to quote server, i.e. "server". It appears to be a reserved word now.

Yes, that did the trick.

Thanks, @KiNgMaR! I added SERVER to the reserved keywords with https://github.com/influxdb/influxdb.com/commit/ba1c2453b3e2d3d8a464fe8bb3fd752a2055a433


CerebNet submodules API's

I reviewed and tested these changes. They are ready to be merged as far as I can see.


[management] Consolidate network map firewall rules into summaries

Describe your changes
Optimizes the network map firewall routing rules by consolidating individual peer rules into summarized rule sets.
Issue ticket number and link
Checklist

[ ] Is it a bug fix
[ ] Is a typo/documentation fix
[ ] Is a feature enhancement
[x] It is a refactor
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Looks fine to me other than failed tests

Looks fine to me other than failed tests

The failed tests are due to the missing client implementation after updating the proto definitions.


Unable to find a specification for YelpAPI when I try to install the pod

Is the pod deprecated?

@Hsanka6 : Not deprecated --- I have a hunch this might be fallout from our yelp-ios 2.0.0 release.
What's the full error message text?  Are you specifying a version number when installing YelpAPI?

the full message is: " Unable to find a specification for YelpAPI " and which one should I specify it to?

@d9chen , @SSheldon : Thoughts?

@Hsanka6, have you run pod repo update before trying to install? Could you paste the results of pod env here for us to help debug?

Yes I have and I have still got errors and I will next week when I'm done with my finals

Sounds good @Hsanka6, I'll take a look once you have time to provide that information. Good luck on your finals 😊

I think I had an issue with my other project because I just tried using a different branch and adding the pod and it worked

Is there any examples of how to use the api in swift btw?

We're working on adding more Swift examples, for now the closest thing is one of the tests is written in Swift.
Glad everything's working now!


enable autowiring for every discoverable service

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets
N/A

Documentation
TODO

License
MIT

This continues the work done in #142. By declaring an alias for every service.
This also remove the DiscoveryPass in favor of plain xml configuration as I think the provided configuration is equivalent to the compiler pass but easier to maintain.

Before merging, I will map FQCN aliases to main_alias when relevant. This will avoid to break #142 changes.


pkg/scheduled_test: include changelog.md

So folks can see what's changed between versions

This issue has been moved to dart-lang/scheduled_test#11.


Make Cython a dist dependency

Make setup.py directly use cython(ize) instead of generated C source and header files.

Originally reported by: Yung-Yu Chen (BitBucket: yungyuc, GitHub: yungyuc)

Bitbucket: https://bitbucket.org/solvcon/solvcon/issue/123

Done by using Cython.Distutils


Enhance the experience when using diamond

This is a follow up to #263
This adds the functions done in the compile-cli to a post-compile script for diamond to use while compiling.
diamond compile -o output.css input.scss would automatically run all of the required PostCSS stuff.
I can change filenames or do anything if needed. (Note: PostCSS deps can be moved to devDependencies)

Looks good to me, I'll release a new minor version soon with those changes. I was trying to do something similar before but I never did. Thanks!

😃

Since you are so interested, feel free to follow the project at https://diamond.js.org/ and suggest changes/new features.

Thanks! Although this is useful right now, I have plans to turn this project into a preprocessor instead of a Sass/PostCSS framework, and that would include a new package manager. So I don’t see myself involved in the project too much, but I will keep an eye on it!

diamond was originally designed to work with multiple preprocessors as well as CSS, unfortunately all but Sass had to be dropped due to only Sass having custom importers. If you design your preprocessor with this in mind, diamond could easily be adapted to work with it if you wish. This would allow, like when other preprocessors were supported, for certain Sass packages to be imported by your preprocessor, and vice versa.

That sounds interesting, I'll certainly keep it in mind once I get there. The idea of importing packages from other preprocessors is very appealing to me.

I forgot to include something in my last comment. Importing packages from other preprocessors would only be supported if your preprocessor is backwards compatible with CSS like most others, or if I hand write a script to handle it.

Yeah, I'll try to make it as compatible as possible. I plan to use PostCSS to get something working as fast as possible, then I'll see where it takes me. The idea is still blurry, I have to think about a lot of things yet, so we will see what happens once I get to that phase.

While we are on the subject of this, the diamond team (2 people is still a team :P ) was considering writing a new preprocessor, and one of the biggest things I wanted was built in dependency management. I think you should consider this when writing your preprocessor.

Interesting, like when you download a package, also download the "includes" of that package?

That (which I should get to writing for diamond 😬 ) and importing of packages and their dependencies built in, without a custom importer.

What do you mean with "without a custom importer"?

To import diamond packages in sass, you either have to use the diamond compile command (it just adds a importer under the hood, and some other stuff) or specify the importer when compiling with node-sass like so node-sass --importer diamond where diamond is the folder generated on install. It would be sweet if this was supported by the compiler itself. I also am available for chat on Discord or Slack, chatting through github feels clunky.

Oh I see. I will have to think more about this thing, I have been focused thinking on the language itself but not too much on the package stuff. You can invite me to Slack with this address<EMAIL_ADDRESS>if you want.

I have invited you to my slack, had to clear away all the old testing stuff :P

Cool, I accepted the invitation, but I'll be online tomorrow, right now is a little late 😅


Add USB GPU support

Machines with DisplayLink adapters would fail to get GPU information using the resources API.
This fixes the source of the issue (nested devices) and attempts to fill in a USBAddress field too.

Testsuite passed


add complete proxy options to request config

I added proxy options to some strange cases when the complete proxy configuration is set on runtime time or simply is not configured on environment variables and the url is dynamic too.

This PR has been closed as "abandoned" as a part of an automated cleanup. If this is a mistake and you are still interested in merging this PR, please do the following:

[ ] Rebase your changes onto the master branch (even if there are no merge conflicts, we want to make sure your change is under the most recent tests).
[ ] Push your updated branch to Github to update this PR.
[ ] Make sure this PR's Travis tests are still passing. (Run npm test locally if tests are still failing).
[ ] Comment here to let us know that you've updated the PR and are still interested in getting it merged. Somebody from the project will respond and work with you to review.

Thank you for your understanding. If you have any questions, please check out https://github.com/request/request/issues/2556 and feel free to comment / ask anything there.


waffle examples failures with public mandala testnet

currently all these examples pass with latest local mandala acala/mandala-node:943f80a5, but has some failures with the public mandala testenet wss://node-6870830370282213376.rz.onfinality.io/ws?apikey=0f273197-e4d5-45e2-b23e-03b015cb7000

dex
10 passing (26s)
  5 failing

1) Dex
       getSwapTargetAmount works:
     Error: -32603: execution revert: Other("Dex get_swap_target_amount failed"): 0x000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002a4f746865722822446578206765745f737761705f7461726765745f616d6f756e74206661696c65642229

 2) Dex
       getSwapSupplyAmount works:
     Error: -32603: execution revert: Other("Dex get_swap_supply_amount failed"): 0x000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002a4f746865722822446578206765745f737761705f737570706c795f616d6f756e74206661696c65642229

  3) Dex
       swapWithExactSupply works:
     Error: -32603: execution revert: Other("InsufficientLiquidity"): 0x000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001e4f746865722822496e73756666696369656e744c69717569646974792229

  4) Dex
       swapWithExactTarget works:
     Error: -32603: execution revert: Other("InsufficientLiquidity"): 0x000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001e4f746865722822496e73756666696369656e744c69717569646974792229

  5) Dex
       addLiquidity and removeLiquidity works:
     Error: -32603: execution revert: Other("InsufficientLiquidity"): 0x000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001e4f746865722822496e73756666696369656e744c69717569646974792229

oracle
  1 passing (59s)
  1 failing

  1) Prices
       getPrice works:
     AssertionError: Expected "0" to be equal<PHONE_NUMBER>0000000000000

erc20
  9 passing (3m)
  3 failing

  1) BasicToken
       Transfer emits event:
     Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/shunjizhan/Acala/bodhi.js/examples/waffle/erc20/test/BasicToken.test.ts)

  2) BasicToken
       Can not transfer from empty account:
     AssertionError: Expected transaction to be reverted, but other exception was thrown: Error: -32603: execution fatal: Module { index: 180, error: 2, message: None }

  3) LP ACA-AUSD Token
       "before all" hook for "get token name":
     Error: -32603: execution revert: Other("BalanceTooLow"): 0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000164f74686572282242616c616e6365546f6f4c6f772229

state-rent
  0 passing (2s)
  1 failing

  1) StateRent
       "before all" hook for "stateRent works":
     Error: 1014: Priority is too low:<PHONE_NUMBER>8264 vs<PHONE_NUMBER>6309): The transaction has too low priority to replace another transaction already in the pool.

e2e
  1) e2e test
       evm block number:
     Error: 1014: Priority is too low:<PHONE_NUMBER>8264 vs<PHONE_NUMBER>9848): The transaction has too low priority to replace another transaction already in the pool.

scheduler
  0 passing (54s)
  5 failing

  1) Schedule
       ScheduleCall works:
     Error: Timeout of 50000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/shunjizhan/Acala/bodhi.js/examples/waffle/scheduler/test/Scheduler.test.ts)

  2) Schedule
       CancelCall works:
     Error: 1014: Priority is too low:<PHONE_NUMBER>49248 vs<PHONE_NUMBER>1479): The transaction has too low priority to replace another transaction already in the pool.

  3) Schedule
       RescheduleCall works:
     Error: 1014: Priority is too low:<PHONE_NUMBER>49248 vs<PHONE_NUMBER>1479): The transaction has too low priority to replace another transaction already in the pool.

  4) Schedule
       works with RecurringPayment:
     Error: 1014: Priority is too low:<PHONE_NUMBER>49248 vs<PHONE_NUMBER>120): The transaction has too low priority to replace another transaction already in the pool.

  5) Schedule
       works with Subscription:
     Error: -32603: execution error: OutOfFund: 0x

Priority is too low error is because previous transaction have not confirmed yet and it is sending transaction with same nonce.

that makes sense, is there any trick to run tests to avoid such situation? One way I can think of is to manually change test cases to wait for one block before sending each next tx, but this doesn't sound to be the most elegant way

Other than wait for tx to be confirmed in test, the test could also add a simple nonce tracker to bump nonce for each transactions

closes this since it's very outdated


Add support for Fastify-sensible errors

The error handler of the plugin uses a property on the error object to decide if it should redact the message. I think it's nice to check for the same property and add support for the plugin errors:
https://github.com/fastify/fastify-sensible/blob/48ef8a00ef7b9396a538711b58e7d09b410e1025/index.js#L47

:tada: This issue has been resolved in version 3.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature Request: Fully support PopoverController API options for ion-select

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[x] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Currently, PopoverController API options cannot be applied to ion-select. AlertController and ActionSheetController are supported by passing selectOptions, but options like mode are not supported when the interface is set to "popover".
Expected behavior:
Expect to be able to set API options like mode for PopoverController
Steps to reproduce:
Add ion-select component and set interface="popover" and define selectOptions for the component as:
{
    mode:'ios'
  }

The component's functionality breaks with this defined.
Related code:
https://stackblitz.com/edit/ionic-pvhxw7

Hello! Thank you for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#332


feat: getEnvironment()

Enhancement

Simplify CDK environment pull from config with:

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

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: e22debe2cbcff80246f2f48134b7a5b2d01516aa
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: 18fbdacae9cadc6189ae5fba349220673a9575a8
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


just show tags, don't stay active

Not permit select tags when property "readOnly" is true

@Gustavoaz the new version 0.4.2 has changed structurally so I cannot accept your pull request. You will still find the function you requested with the "pressEnabled" attribute


Introducing new logger, logging debug to file and info to stderr

I see you changed from rlog.
I opened PR that added "global flag" and it is nice in the way of adding debug so we probably want to merge those changes somehow (I think you solution is better - just need to understand how will it work with my changes (that are in progres now).
Also mizu config file will be names .mizu and I see you added .mizu folder.. it might cause conflict in some way so lets check and see if the config file should move or the log should move

one thing that I want to think about from now (even if we are not implement it) is log rotation..
I don't want to have increasing file with no limit (although it should not increase too much)
maybe it can be easy to get this rotation (max time to save logs / max size for the log).


Fix python

Removes the hard coded python versions entirely!

@Alessandro-Barbieri this should remove the Python dependencies. Does this look acceptable to you?

Why not move all in the eclass?


Show basic application integration testing (#2896)

Objective
Show an example of how the integration testing of a bevy application can be structured, for #2896.
I went for basic window interaction and user input for the examples, but there are certainly other areas that would be good to demonstrate, I just haven't worked with the engine enough to point them out. Suggestions are welcome :)

Great to see this; this is something I've been asked about a lot.

Apologies for replacing the commit twice. Turns out if you do the cargo run -p ci on nightly, it will come up with more lints than when it is run on stable, so on my pc it always failed instead of showing what the ci on github would say about the code. Which made it a bit difficult to pre-empt the remarks of the ci 😄 .

Seems a shame to lose this: adopted in #16489. Thanks @Wcubed!


Fix broken tests for Laravel 9

This PR fixes an issue with the dump handler not registering correctly on Laravel 9+.
I'm not sure if this is the best solution, but it is a solution.  Suggestions for improvements/alternatives are welcome.

@freekmurze Request changes complete :+1:

Thanks!


Update WLS patch for k8s to have correct patches

Removed patch p27117282_122130_Generic.zip which is not required
Added patch p28298734_122130_Generic.zip which is required (it is a prereq for p28076014_122130_Generic.zip and it supersedes p27117282_122130_Generic.zip)

Mark can you edit the README as well?  I extend from the developer image because it is a smaller image.


Travis CI tests failing

Not the end of the world, but I wanted to note that it seems like most PRs after f830975aa5e32498f202c92c6c4ac77404bf28c5 have been failing in Travis. I'm not sure how to resolve except to delete the lines from .travis.yml. :shrug: There's probably a better way.

Looks like a tape-run issue


monitor crawls

The June 15 2014 mobile crawl failed because the schema for "requestsmobile" table was not updated. The result was no requests were saved. In this specific situation there were 5000 pages created, but it had "0" for all the expected columns (reqTotal, etc.).
We should have general checks that monitor the progress of the crawl and make sure it's working.

Added bulktest/batch_monitor.php


Add 314 DO nodes from Hypriot

As we're always using Docker on a Raspberry Pi it's easy to contribute the right number of nodes. 😏  🐳
Signed-off-by: Dieter Reuter<EMAIL_ADDRESS>

LGTM
Thank you @DieterReuter !!


Ability to pass event handlers as part of the Flow constructor

Currently, a full initialization consist of the constructor, followed by event handler attachment.
I don't quite see how constructor (options, ...) would affect the behavior of fire.
But on the other hand, I a real advantage into being able to pass events handlers straight from the constructor: It allows to do all the configuration with plain a object in one step before passing it to the actual instantiation of the Flow object.
That means instantiation can more easily be delegated to a wrapper (Angular, VueJs, or a similar intermediary layer)
Should I provide a PR supporting this?

Totally agree, you should be able to bind events on the initial step.

Totally agree, you should be able to bind events on the initial step.


vitest fails if coverage is configured but disabled

Describe the bug
If coverage is configured but disabled vitest fails and produces an error.
RUN  v2.0.5 /home/runner/work/pnpm_test/pnpm_test
[14](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:15)
⎯⎯⎯⎯⎯⎯ Unhandled Errors ⎯⎯⎯⎯⎯⎯
[15](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:16)

[16](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:17)
Vitest caught 1 unhandled error during the test run.
[17](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:18)
This might cause false positive tests. Resolve unhandled errors to make sure your tests are not affected.
[18](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:19)
⎯⎯⎯⎯⎯⎯ Unhandled Error ⎯⎯⎯⎯⎯⎯⎯
[19](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:20)
TypeError: coverage.reporter.find is not a function
[20](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:21)
 ❯ node_modules/.pnpm/vitest@2.0.5/node_modules/vitest/dist/chunks/cli-api.OKfd3qJ0.js:13975:46
[21](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:22)
 ❯ serializeConfig node_modules/.pnpm/vitest@2.0.5/node_modules/vitest/dist/chunks/cli-api.OKfd3qJ0.js:13984:7
[22](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:23)
 ❯ WorkspaceProject.getSerializableConfig node_modules/.pnpm/vitest@2.0.5/node_modules/vitest/dist/chunks/cli-api.OKfd3qJ0.js:14323:20
[23](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:24)
 ❯ getConfig node_modules/.pnpm/vitest@2.0.5/node_modules/vitest/dist/chunks/cli-api.OKfd3qJ0.js:8425:33
[24](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:25)
 ❯ node_modules/.pnpm/vitest@2.0.5/node_modules/vitest/dist/chunks/cli-api.OKfd3qJ0.js:8452:19
[25](https://github.com/nekrich/pnpm_test/actions/runs/10679047402/job/29597406906#step:6:26)
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

Reproduction

Create a new project.
Add vitest with configured coverage (checked with istanbul and v8).
Disable coverage.

Sample project: https://github.com/nekrich/pnpm_test .
As you can see from commits:

When coverage is enabled vitest succeeded. Workflow run.
Disabling action with this commit leads to failure. Failed Workflow run.

System Info
System:
    OS: macOS 15.0
    CPU: (12) arm64 Apple M2 Max
    Memory: 109.88 MB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 22.7.0 - ~/.local/share/mise/installs/node/22.7.0/bin/node
    Yarn: 1.22.19 - ~/.local/share/mise/shims/yarn
    npm: 10.8.2 - ~/.local/share/mise/installs/node/22.7.0/bin/npm
    pnpm: 9.9.0 - ~/.local/share/mise/installs/pnpm/latest/bin/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 128.0.6613.120
    Safari: 18.0
  npmPackages:
    @vitest/coverage-istanbul: ^2.0.5 => 2.0.5
    vitest: ^2.0.5 => 2.0.5

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of https://github.com/vitest-dev/vitest/issues/6260.
Use array in reporter until fix is released.


Tool keeps crashing (Process NAVSTREETS™ Street Data)

I try using the tool "Process NAVSTREETS™ Street Data" for ArcGIS Desktop 10.8 or later. The tool keeps crashing.
Do you have any idea how to make it work?
I use ArcMap 10.8.2

Do you have the 64-bit Background Geoprocessing extension?  The tool will not work with that extension, so you have to turn off background geoprocessing.  https://desktop.arcgis.com/en/arcmap/latest/analyze/executing-tools/foreground-and-background-processing.htm

Did you ever resolve this problem?  Crashes in ArcMap weren't unusual, since it was a 32-bit application that consequently had memory limitations.  Often running out of memory would lead to crashes.
A new version of the tool for processing HERE data is now available for ArcGIS Pro.  If you're still processing HERE data, please try this new tool version and let us know if you still experience hangs and crashes.  Because ArcGIS Pro is a 64-bit application and because the new tool was fully rewritten to be less resource intensive, we don't expect the same types of limitations that the old ArcMap version had.

It's been three months, and there has been no response on this issue, so I'm closing it.  If you're still having problems, feel free to re-open.


LP7 - lab 1 ex 3 - unable to create incident rule

Contact Details
No response
What happened?
LP7
Lab1
Ex 3
Task: 00
Step: 31
Description of issue:
Unable to create Incident rule
Create new automation rule - Analytic rule name "unknown analytic rule. a different analytic rule must be selected
Repro steps:
1.
1.
1.
Lab
Other
Relevant screenshots
Create new automation rule - Analytic rule name "unknown analytic rule. a different analytic rule must be selected

Do you want to help us? 👏

[ ] Create a Pull Requestt by following this project's Contribution Guide

I have seen that happen before (recently) in a hosted environment. I had to back out and restart the rule process. Not a lab bug.


Dynamic DNS on the status overview page shows "Config error" under Network.

LEDE Reboot 17.01.1 r3316-7eb58cf109 / LuCI lede-17.01 branch (git-17.100.70571-29fabe2)
After ADDING a ddns service in LuCI the ddns updating works just fine, but shows "Config error" under Network column on status overview.  This is because these options are missing from the newly added service's config:
    option interface
    option ip_source
    option ip_network

Once those options are manually added to the new ddns service's configuration and ddns is restarted, the "config error" goes away.  Please see the attached document.
config ddns error.docx

Same problem, but seems it's already fixed. Good job!

This issue still exists with LEDE 17.01.2


Added Health Endpoint

Currently only sends a default json and a 200 if spot is running. In future PR we could implement a check that beam is actually reachable.

Lets merge this after #19 I think the merge conflicts here will be easier to resolve then the other way around

We usually still merge into main and develop is really outdated can you rebase our changes to main and change the pr destination


possible memory leak with keepalive

If the http agent has keepAlive enabled, and the request is for an ip address or unix domain socket, the lookup even will not fire, so the socket's lookup listeners will grow with every request.

Good point!


helm 3 alpha 1: helm chart fails badly when chart path is invalid

Output of helm version:
version.BuildInfo{Version:"v3.0.0-alpha.1", GitCommit:"b9a54967f838723fe241172a6b94d18caf8bcdca", GitTreeState:"clean"}
Attempting to push a chart with a badly formed ref fails with a stack trace. Ideally this error would be a bit more friendly and/or could be prevented at helm chart save.
helm chart save mychart mychart

Using default tag: latest
32e5dd9: Saving meta (146 B)
5282241: Saving content (3.1 KiB)
Name: mychart
Version: 1.0.8
Meta: sha256:32e5dd94fcba61d0bc36664fc0f9e908e39552507f7694c21f7a34f19eb82572
Content: sha256:5282241bf2a576f84ab38c681cf325ad6f03d35100f76bfb31be119d86f9d675
latest: saved

helm chart push mychart
Using default tag: latest
The push refers to repository [mychart]
Name: mychart
Version: 1.0.8
Meta: sha256:32e5dd94fcba61d0bc36664fc0f9e908e39552507f7694c21f7a34f19eb82572
Content: sha256:5282241bf2a576f84ab38c681cf325ad6f03d35100f76bfb31be119d86f9d675
panic: runtime error: slice bounds out of range

goroutine 1 [running]:
helm.sh/helm/vendor/github.com/containerd/containerd/reference.Spec.Hostname(0xc0006e1490, 0x6, 0x0, 0x0, 0xc0006e1490, 0x7)
	/go/src/helm.sh/helm/vendor/github.com/containerd/containerd/reference/reference.go:129 +0x87
helm.sh/helm/vendor/github.com/containerd/containerd/remotes/docker.(*dockerResolver).base(0xc000629900, 0xc0006e1490, 0x6, 0x0, 0x0, 0x0, 0x0, 0x0)
	/go/src/helm.sh/helm/vendor/github.com/containerd/containerd/remotes/docker/resolver.go:307 +0x88
helm.sh/helm/vendor/github.com/containerd/containerd/remotes/docker.(*dockerResolver).Pusher(0xc000629900, 0x270f6e0, 0xc0006e7b30, 0xc0006e1329, 0x6, 0x0, 0x0, 0xc0006bc810, 0x0)
	/go/src/helm.sh/helm/vendor/github.com/containerd/containerd/remotes/docker/resolver.go:281 +0x158
helm.sh/helm/vendor/github.com/deislabs/oras/pkg/oras.Push(0x270f6e0, 0xc0006e7b30, 0x270b1e0, 0xc0006e2ae0, 0xc0006e1329, 0x6, 0x26d9760, 0xc0006e40c0, 0xc0006f55e0, 0x2, ...)
	/go/src/helm.sh/helm/vendor/github.com/deislabs/oras/pkg/oras/push.go:36 +0x246
helm.sh/helm/pkg/registry.(*Client).PushChart(0xc000629940, 0xc0006e7aa0, 0xc0006e7aa0, 0x0)
	/go/src/helm.sh/helm/pkg/registry/client.go:100 +0x2fd
helm.sh/helm/pkg/action.(*ChartPush).Run(0xc000761d70, 0x26dbee0, 0xc0000ce000, 0x7ffeefbfe7d9, 0x6, 0xc000761d30, 0x0)
	/go/src/helm.sh/helm/pkg/action/chart_push.go:43 +0x7b
main.newChartPushCmd.func1(0xc000434c80, 0xc0006e2a70, 0x1, 0x1, 0x0, 0x0)
	/go/src/helm.sh/helm/cmd/helm/chart_push.go:44 +0x74
helm.sh/helm/vendor/github.com/spf13/cobra.(*Command).execute(0xc000434c80, 0xc0006e2a30, 0x1, 0x1, 0xc000434c80, 0xc0006e2a30)
	/go/src/helm.sh/helm/vendor/github.com/spf13/cobra/command.go:762 +0x465
helm.sh/helm/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0xc00057c280, 0x26dbee0, 0xc0000ce000, 0xc0000d2010)
	/go/src/helm.sh/helm/vendor/github.com/spf13/cobra/command.go:852 +0x2ec
helm.sh/helm/vendor/github.com/spf13/cobra.(*Command).Execute(...)
	/go/src/helm.sh/helm/vendor/github.com/spf13/cobra/command.go:800
main.main()
	/go/src/helm.sh/helm/cmd/helm/helm.go:55 +0xa6

@jdolitsky yes, that sounds right. Looking briefly at the OCI spec, both helm chart save and helm chart push should validate that <ref> is a valid url, which may optionally include a digest or tag.

@willholley I'm not sure if I agree this should be required for helm chart save. I see this as sort of the equivalent of docker tag. There are several issues surrounding this UX we need to address, please join us on the dev calls Thursday 9:30am PST

@jdolitsky do you think this use case has been fixed, or is this still an issue in core?

@bacongobbler just tested it out, this issue is still present on push

This issue appears to be fixed in Helm 3.0.0+:
$ helm chart push mychart
Error: Chart not found: mychart


Use React.useId if available

Intended to get a conversation started. Needs a lot more testing and backward compatibility analysis.
This uses React 18's useId and useSyncExternalStore to replace our useSSRSafeId and useIsSSR hooks, with fallbacks for older reacts to our existing implementation. SSRProvider becomes a noop and apps can remove it if they are on React 18. This should improve support for streaming server rendering, as well as performance by removing the useLayoutEffect in the SSRProvider at the root of the app.
Two points in terms of backward compatibility:

This will change the format of everyone's ids, which will break snapshot tests. I don't think this can be avoided.
Ids may start clashing if there are multiple versions of React on the page at once. For client only apps, we currently generate a random prefix to avoid this. With React.useId, this can be achieved using the identifierPrefix option of React.createRoot, but that's not something React Aria can control. So apps that upgrade which have multiple copies of React (e.g. micro frontends) may see ids clashing. This is something we need to verify.

To do

[ ] Update docs

For point number 2, we could probably still prepend our own random prefix to the start of the id returned by React.useId. We would need to do this only when client rendering to avoid hydration mismatches, the same way we do today.

@devongovett Thanks for this. How can I track/know when this change will be released? Need it because of next13 /app dir

It'll be in our next stable release in the next few weeks. In the meantime, you can try out the nightly version if you like.

@devongovett added PR that addresses this problem https://github.com/adobe/react-spectrum/pull/4738


NuGetSupport: Replace license URLs with SWH content hashes

Many NuGet packages only declare license URLs which, in contrast to
license names, cannot reliably be mapped to SPDX license expressions as
the content the URL refers to might change over time.
To support a reliable mapping if only a license URL is present, do not
use the URL itself as a declared license, but replace it with an
ORT-specific SPDX LicenseRef that includes the Software Heritage
hash over the content of the file the license URL refers to.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

Alternative solution could be: Extend declared license mapping to allow specifying a content hash.

Alternative solution could be: Extend declared license mapping to allow specifying a content hash.

Isn't that exactly what's being done with e.g. https://github.com/oss-review-toolkit/ort/pull/5259/files#diff-4e93601aa35e18daab883e47b59f0e54cc0614a6b0d76e3f60acb78f8965f57aR321? Or do you mean a simpler syntax that does not incorporate the content hash into a LicenseRef, but just lists the content hash?

Isn't that exactly what's being done with e.g. https://github.com/oss-review-toolkit/ort/pull/5259/files#diff-4e93601aa35e18daab883e47b59f0e54cc0614a6b0d76e3f60acb78f8965f57aR321? Or do you mean a simpler syntax that does not incorporate the content hash into a LicenseRef, but just lists the content hash?

I meant a solution which works without creating new generated license identifiers per distinct (url, hash).

I meant a solution which works without creating new generated license identifiers per distinct (url, hash).

IMO that's the case already for the current proposal. New entries are only generated for each new (filename, hash) pair, but not for each (url, hash) pair. And I'd also be fine with dropping the filename from the LicenseRef.

IMO that's the case already for the current proposal. New entries are only generated for each new (filename, hash) pair, but not for each (url, hash) pair. And I'd also be fine with dropping the filename from the LicenseRef.

I believe using the a content hash from time = now() is wrong anyway, because the content at the point in time the artifact has been created is relevant but not the one from now().  Either the content from back then is unkown or it can be found manually, e.g. using the wayback machine. When doing so, the process is manual anyway, the result could just be encoded using a declared license mapping.  I simply to not understand how this proposal can help solve the problem in a better way.

I believe using the content hash from time = now() is wrong anyway, because the content at the point in time the artifact has been created is relevant but not the one from now(). Either the content from back then is unkown or it can be found manually, e.g. using the wayback machine. When doing so, the process is manual anyway, the result could just be encoded using a declared license mapping. I simply to not understand how this proposal can help solve the problem in a better way.

The original goal of this PR was not to implement a way to have declared license mappings for the contents of license from back when the package was published, but as a first step to simply have something like "permalinks" to the current content of the license. Finding out the right permalink for a previous version of the contents could then be done as a second step.

The original goal of this PR was not to implement a way to have declared license mappings for the contents of license from back when the package was published,

From the PR desccription, I got that this is beeing proposed in order to support a reliable mapping. However, the (current) content as I pointet out is not relevant to the mapping.

To support a reliable mapping if only a license URL is present, do not
use the URL itself as a declared license, but replace it with an
ORT-specific SPDX LicenseRef that includes the Software Heritage
hash over the content of the file the license URL refers to.

From the PR desccription (following excerpt), I got that this is beeing proposed in order to support a reliable mapping.

I see how this can be misleading based on individual expectations, but "reliable" was more meant like "deterministic" regarding the content a URL points to.


adds round support and images sized for round

Signed-off-by: Dan Cunningham<EMAIL_ADDRESS>

No other text position changes were needed?


bump to telegraf@4

telegraf@4 requires too many native node modules and globals that are not available in the cfworker environment.
I can't make the compiled product work in cfworker.

Hi!
Just managed to get telegraf@4 working with Webpack 5 (using native Cloudflare Worker events instead of @cfworker/web because I want to use Toucan :P can be easily migrated though).
Unfortunately, due to Gitpod's outage, my efforts are gone 😢 But it shows that it is actually possible to achieve this (with some dirty hack in webpack.config.js).
I'm also trying building telegraf with rollup, as it's almost 10s faster than Webpack 5 with esbuild as a minifier. Who don't like speedy builds?
However, XHR was introduced in rollup's node core polyfills, which Cloudflare Workers lacks support of as it's a pretty old API. What's worse, the mixing of CommonJS and ES Modules in the codebase is not friendly with that and prevents CommonJS statements being transpiled. No luck so far, I guess I would just stick with Webpack at this point.

Example webpack.config.js:
const path = require('path')
const webpack = require('webpack')
const { ESBuildMinifyPlugin } = require('esbuild-loader')

const mode = process.env.ENVIRONMENT.toLowerCase() || process.env.NODE_ENV.toLowerCase() || 'production'

module.exports = {
  entry: './src/index.ts',
  target: 'webworker',
  output: {
    filename: 'worker.js',
    path: path.join(__dirname, 'dist')
  },
  mode,
  devtool: 'source-map',
  module: {
    rules: [
      {
        test: /\.tsx?$/,
        loader: 'esbuild-loader',
        options: {
          loader: 'ts',
          target: 'es2019'
        }
      }
    ]
  },
  resolve: {
    fallback: {
      stream: require.resolve('stream-browserify'),
      http: require.resolve('stream-http'),
      https: require.resolve('https-browserify'),
      url: require.resolve('url/'),
      path: require.resolve('path-browserify'),
      util: require.resolve('util/'),
      crypto: require.resolve('crypto-browserify'),
      buffer: require.resolve('buffer/'),
      fs: false
    }
  },
  plugins: [
    new webpack.ProvidePlugin({
      Buffer: [require.resolve('buffer/'), 'Buffer'],
      process: require.resolve('process/browser')
    })
  ],
  optimization: {
    minimizer: [
      mode === 'production' ? new ESBuildMinifyPlugin({
        target: 'es2019'
      }) : null
    ]
  }
}

esbuild is optional.

And add a polyfill in entrypoint:
import 'abortcontroller-polyfill/dist/polyfill-patch-fetch'

Confirmed working in:
"dependencies": {
  "abortcontroller-polyfill": "^1.7.3",
  "buffer": "^6.0.3",
  "crypto-browserify": "^3.12.0",
  "https-browserify": "^1.0.0",
  "path-browserify": "^1.0.1",
  "process": "^0.11.10",
  "stream-browserify": "^3.0.0",
  "stream-http": "^3.2.0",
  "telegraf": "^4.4.1",
  "tiny-request-router": "^1.2.2",
  "url": "^0.11.0",
  "util": "^0.12.4"
},
"devDependencies": {
  "@cloudflare/workers-types": "^2.2.2",
  "@types/node": "^16.7.1",
  "esbuild": "^0.12.21",
  "esbuild-loader": "^2.15.1",
  "ts-standard": "^10.0.0",
  "typescript": "^4.3.5",
  "webpack": "^5.51.1",
  "webpack-cli": "^4.8.0"
}

ebpack 5 (using native Cloudflare Worker events instead of @cfworker/web because I want to use Toucan :P can be easily migrated though).

Anyway to get it working with webpack 4 though? Since wragnler uses webpack 4 by default and I want to use wrangler dev for local testing

ebpack 5 (using native Cloudflare Worker events instead of @cfworker/web because I want to use Toucan :P can be easily migrated though).

Anyway to get it working with webpack 4 though? Since wragnler uses webpack 4 by default and I want to use wrangler dev for local testing

Bringing your own bundler is actually the preferred way in official docs for now, give it a try!

I finally make it works! I'm sorry I'm too lazy
I will release version 2.0 as soon as possible.

Thank you for your efforts!


Unhelpful failure reporting

In http://www.w3c-test.org/html/browsers/the-window-object/window-properties.html the first failure I get with Firefox, for test "EventTarget method: addEventListener", is:
assert_equals: expected function "function addEventListener() { [native code] }" but got function "function addEventListener() { [native code] }"

I don't see how we can do better than the current; that test should probably define an error message.

This issue has been moved to https://github.com/w3c/web-platform-tests/issues/7154; please continue all discussion there.


lxd-generate: Create filtered statements from pre-existing "objects" statement.

As per our discussions on irc, I've updated lxd-generate's statement generation for all statements of the type objects-by-FIELD to first search the package for a variable declaration of the form
var <entity>Objects = RegisterStmt(`SQL String`)

And then extract the SQL String from the syntax tree, and append a WHERE clause with the given filters, and create a new variable declaration of the form
var <entity>ObjectsByField = RegisterStmt(`SQL String with WHERE clause`)

This will allow us to specify our own <entity>Objects for more complex queries, and still be able to use lxd-generate to create filterable statements.
Additionally, I went ahead and created a lxd-generate.md under the docs directory with an overview of all the functionality of lxd-generate including how to formulate structs and how lxd-generate will behave based on that, as well as all of the types of generation and associated tags that lxd-generate can use.
I'll send up another PR soon that will add an sql=<table>.<column> tag to each field that will let us bypass the generator's automatic generation of table names and column names.

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.10870/

If we would prefer the lxd-generate doc to be under lxd/db/generate as a README, I'll move it over so it's not included among other more usage-related docs. For now I've put it under internals.

@ru-fu @stgraber are you happy to have lxd-generate in the "internals" section of the docs, or in the docs at all (and perhaps a separate docs directory in the lxd-generate directory)? As I understand we don't really want to encourage external usage of this tool.

@ru-fu @stgraber are you happy to have lxd-generate in the "internals" section of the docs, or in the docs at all (and perhaps a separate docs directory in the lxd-generate directory)? As I understand we don't really want to encourage external usage of this tool.

That really depends on the intention - if we don't want users to run this tool and the internal workings aren't relevant for anybody who doesn't use it, it should rather be a README in the directory that contains the tool.
If it is good for users to know that this tool exists, or to have an idea of how it work, we should have it in the docs, and the Internals section is probably a good place for now.

@ru-fu Thanks, that makes me think a README is best, as this will only be relevant to LXD development, and entirely irrelevant for users.

@ru-fu Thanks, that makes me think a README is best, as this will only be relevant to LXD development, and entirely irrelevant for users.

Sounds good to me.

jenkins: test this please

jenkins: test this please


Build libcoro to web assembly with the emscripten SDK

Be able to build libcoro with the emscripten SDK so it can run in web assembly.

#221
#201


Fixed Godot 4+ .gitignore file

A follow-up to these that were left open since LAST YEAR:
https://github.com/github/gitignore/pull/4414 & https://github.com/github/gitignore/pull/4385
Reasons for making this change:

As a game dev on Godot 4+, I have to constantly include these small changes in the .gitignore file.
To prevent errors while working with a team: such as class_name conflict when renamed, because local cache does not understand the changes made.
Links to documentation supporting these rule changes:
All I can say, is that files such as "icon.svg.import" or "gam3E1D.tmp" are now ignored from git commits.
It would greatly help the community as well.

You didn't include a link to documentation supporting the idea that the .import files are useless, after doing some digging there are documentation pages about the .import files: https://docs.godotengine.org/en/4.0/tutorials/assets_pipeline/import_process.html#files-generated

Importing will add an extra .import file next to the source file, containing the import configuration.
Make sure to commit these files to your version control system, as these files contain important metadata.

Only for custom assets. Normal assets that are unmodified will most likely work normally.


Test versions 3.5 and 3.6 with travis

The original goal of this PR was to test versions 3.5 and 3.6.
However it turned out that the tests fail entirely because the latest version of tornado (a dependency) no longer supports Python 2.6 and 3.3 due to security vulnerabilities. I think this is a strong motivation for dropping support in this library too. For reference, the last Python 2.6 release was 2013-10-29 and the last 3.2 release was 2014-10-12.
Thoughts @callum-oakley?

I'm going to go ahead and merge this.

The tests now seem to be failing, and I'm not sure why. I'm going to revert this branch on master and figure out what the problem is when I have more time.

Ahh sorry @WillSewell, didn't see this the other day :grimacing: In response to the original question dorpping support sounds like a good idea :+1:
Happy to take a look at the test failures with you at some point if that's still an issue.


⚠ ClamAV has degraded performance

In 749ed75, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 604 ms

Resolved: ClamAV performance has improved in d3c6284.


Wrong folder name in readme.md

Merging even if the openshift regression is not green. Going to work on this.
Anyway, things are not related.


Pre-upload hook not properly awaiting termination of hook action before upload is started

I am trying to manipulate a json file in a pre-upload hook and then sync the file as an additional file. However, the upload process starts before the hook action is terminated and the original json is therefore uploaded, instead of the manipulated json. I am using a powershell script to manipulate the file, use the pre-upload hook in the settings.yml file and utilize a remote repo. This can be replicated by creating a script that simply waits 5-10 seconds (Start-Sleep -Seconds 5), where it can be observed that the success message "Your settings have been uploaded (user -> repository)" appears before the script terminates in the terminal.

Ya, I haven't thought of that...


[bug] Any imports on a clean project fail

Describe the bug
Exactly as the title says, I create a new project, go into the main.js file and switch out
const { invoke } = window.__TAURI__.tauri;
for
import { invoke } from "@tauri-apps/api/tauri";
This same behavior occurs with any other attempts at using import as well.
I have tried providing a relative path, to be met an error like this:
Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "text/html". Strict MIME type checking is enforced for module scripts per HTML spec.
Reproduction
Create a new vanilla, npm, javascript frontend project
Switch const { invoke } = window.__TAURI__.tauri; for import { invoke } from "@tauri-apps/api/tauri"; in main,js
Expected behavior
Imports @tauri-apps/api/tauri without issue
Platform and versions
[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 115.0.1901.203
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.71.1 (eb26296b5 2023-08-03)
    ✔ Cargo: 1.71.1 (7f1d04c00 2023-07-29)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 16.14.0
    - npm: 8.3.1

[-] Packages
    - tauri [RUST]: 1.4.1
    - tauri-build [RUST]: 1.4.0
    - wry [RUST]: 0.24.3
    - tao [RUST]: 0.16.2
    - @tauri-apps/api [NPM]: 1.4.0
    - @tauri-apps/cli [NPM]: 1.4.0

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../src
    - devPath: ../src

Stack trace
Uncaught TypeError: Failed to resolve module specifier "@tauri-apps/api/tauri". Relative references must start with either "/", "./", or "../".

In the inspector/DevTools console

Additional context
No response

Thanks for reaching out :)
The issue is that if you want to use npm packages you realistically won't get around frontend bundlers like vite or webpack. The vanilla javascript is the only template that doesn't include it (mainly for devs that don't want anything nodejs in their app). You could however choose the vanilla typescript template instead, which comes with vite (you can still use javascript in that project).


Use Dispatcher interface in Server

This PR makes Server use the Dispatcher interface, rather than using OscDispatcher directly.
This change allows users of this library to switch out OscDispatcher for other implementations as/where needed (perhaps for one which trades-off full OSC compliance for extra performance, e.g. "exact" lookups on message addresses rather than pattern lookups).

@AlCutter I'd also like to use the Dispatcher interface for a separate Dispatcher and I think it would be better to pull out the current OscDispatcher from the Server so that it should be configured first, rather than whilst part of the Server.
I won't be needing the AddMsgHandler method on the Dispatcher that I'm using and it's lead me to believe that it's not the responsibility of the Dispatcher interface to have one.
I'll post a PR tonight hopefully and would love any feedback you have on it.

@AlCutter https://github.com/hypebeast/go-osc/pull/33

Hey @AlCutter. I just merged the PR from @glynternet. Please, have a look and let me know what you think. I think it matches your requirements quite good.

Closing this PR since the requested change is already part of it.


Include other-code source-ref in some compiler error messages

Implements https://github.com/SystemRDL/systemrdl-compiler/issues/231

coverage: 94.653% (-0.2%) from 94.826%
when pulling 817f5ff4b6e7e616d130370a45905f89c357564f on nachumg:main
into 649519b67a5a6e0eac6a1e54b491abb05132fda6 on SystemRDL:main.


Fixes issue #73. Multiple classes in schema where last class doesnt match schema name.

If you have multiple classes it will check the class with OMI_BaseResource matches the schema name.
Fixes: #73

This change is

Codecov Report

Merging #75 into dev will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             dev      #75      +/-   ##
=========================================
+ Coverage   60.1%   60.22%   +0.12%
=========================================
  Files          1        1
  Lines        990      993       +3
  Branches       3        3
=========================================
+ Hits         595      598       +3
  Misses       392      392
  Partials       3        3

Impacted Files
Coverage Δ

xDSCResourceDesigner.psm1
60.22% <100%> (+0.12%)
: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 a357bee...749867d. Read the comment docs.

Thanks for fixing this issue! Just a minor comment.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed, 1 unresolved discussion (waiting on @mcollera)

a discussion (no related file):
Could you please add an descriptive entry, for each change/issue, to the Unreleased section of the change log in the file README.md? If an entry resolves an issue please reference the issue where applicable. You may suffix (it's optional) each entry with your name. Example of the format for the entry.
* Descriptive entry ([issue #123](https://github.com/PowerShell/xPSDesiredStateConfiguration/issues/123). [Name/Alias (@github_account)](https://github.com/github_account)

Comments from Reviewable

Review status: all files reviewed, 2 unresolved discussions (waiting on @mcollera)

a discussion (no related file):
If we add a sub class to this helper function will that regression test this change?
https://github.com/PowerShell/xDSCResourceDesigner/blob/a357bee377c863a0878927c85a1079ef0be3cf8b/Tests/xDscResourceDesigner.Tests.ps1#L262

Comments from Reviewable

Review status: 1 of 3 files reviewed, 2 unresolved discussions (waiting on @johlju)

a discussion (no related file):
Previously, johlju (Johan Ljunggren) wrote…
Could you please add an descriptive entry, for each change/issue, to the Unreleased section of the change log in the file README.md? If an entry resolves an issue please reference the issue where applicable. You may suffix (it's optional) each entry with your name. Example of the format for the entry.
* Descriptive entry ([issue #123](https://github.com/PowerShell/xPSDesiredStateConfiguration/issues/123). [Name/Alias (@github_account)](https://github.com/github_account)

Done.

a discussion (no related file):
Previously, johlju (Johan Ljunggren) wrote…
If we add a sub class to this helper function will that regression test this change?
https://github.com/PowerShell/xDSCResourceDesigner/blob/a357bee377c863a0878927c85a1079ef0be3cf8b/Tests/xDscResourceDesigner.Tests.ps1#L262

Done.

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable

@mcollera Thank you for this one! 👍 😃


Proable bug in neo430_bswap

The current implementation of neo430_bswap seems to be buggy. In my case, the compiler selected a totally unrelated register to call swpb on (not the one containing a), which obviously caused the rest of the program to fail.
I'm also confused why swpb is called with two operands, since the instruction seems to have only one...
This code seems to work:
uint16_t neo430_bswap(uint16_t a) {

  register uint16_t r = a;
  asm ("swpb %0" : "+r" (r));
  return r;
}

Note, that I also dropped the volatile qualifier because the ASM code is "pure" (i.e. the outputs depend only on inputs). I would also recommend to move the function to the header and mark it as inline.

Sorry for the late answer.

The current implementation of neo430_bswap seems to be buggy. In my case, the compiler selected a totally unrelated register to call swpb on (not the one containing a), which obviously caused the rest of the program to fail.

Lately, I have been playing a lot with inline assembly / intrinsics and maybe asm ("some instruction with some registers"): seems to be bad practice in some cases as it leaves too much freedom to the compiler to "optimize" in the wrong direction ("constant propagation" seems to be a certain issue). I am experimenting with a more sophisticated approach (-> example). I think this might be a cleaner (still hacky, somehow) and clearer way - or maybe not. I am still trying to figure it out.

I'm also confused why swpb is called with two operands, since the instruction seems to have only one...

Good point. I cannot remember why I have used two operands...

This code seems to work: ...
Note, that I also dropped the volatile qualifier because the ASM code is "pure" (i.e. the outputs depend only on inputs). I would also recommend to move the function to the header and mark it as inline.

Thanks for the modification. I will check it out.


Couldn't open PCM: Device or resource busy.

I'm trying to record audio from my cell phone on my computer.  For the debian sid version of "blualsa" 3.x did not want to connect at all.
dpkg -l | grep blu
ii  bluetooth                        5.66-1                         all          Bluetooth support (metapackage)
ii  bluez                            5.66-1                         i386         Bluetooth tools and daemons
**ii  bluez-alsa-utils                 4.0.0-1                        i386         Bluetooth Audio ALSA Backend (utils)
ii  libasound2-plugin-bluez:i386     4.0.0-1                        i386         Bluetooth Audio ALSA Backend (plugins)**
ii  libbluetooth-dev:i386            5.66-1                         i386         Development files for using the BlueZ Linux Bluetooth library
ii  libbluetooth3:i386               5.66-1                         i386         Library to use the BlueZ Linux Bluetooth stack

OK, I'll check that - maybe something was broken in recent commits...

bluealsa: D: ba-transport.c:1494: Created new IO thread [ba-a2dp-sbc]: A2DP Sink (SBC)

According to quick check, you've got code from around bluez-alsa v4.0.0 :
git clone https://github.com/arkq/bluez-alsa.git && cd bluez-alsa
while [ $(git grep 'Created new IO thread' |awk -F: '{print $2}') != 1494 ]; do git checkout HEAD^; done

I've just checked that version and I was able to record audio using the same command as you: arecord -D .... With bluez-alsa (from this repo) there is one constrain - you can open the PCM device only once. Trying to open it more than once will result in "Device or resource busy" error. However, it seems that it's not the case here - there is only one org.bluealsa.PCM1.Open() in bluealsa logs.
Anyway, please try the most recent version of bluez-alsa. Maybe you've hit some non-common bug which potentially might be fixed already. If master branch will not work for you as well, we will have to do some "printf" debugging :)

I notice that the Debian bluez-alsa-utils package includes bluealsa-aplay with a systemd service file. Just to be absolutely sure that this is not being started silently in the background, what is the output of
systemctl status bluealsa-aplay.service

Everything is already working. I can't repeat this error.
Thank you all for your help.


 If I run boot2docker on the same Dockerfile, it tells me how stupid I was.
eg. Dockerfile with one incorrect line
from tianon:apache2

running boot2docker from a terminal tells me
Sending build context to Docker daemon 69.63 kB
Sending build context to Docker daemon
Step 0 : from tianon:apache2
Pulling repository tianon
2014/09/12 23:14:09 Error: image tianon not found

the same Dockerfile processed by Kitematic only shows me
Step 0 : from tianon:apache2

For the testers out there, the corrected line should be
from tianon/apache2

Thanks for the input!
Fixed in 0.5 with building being a future feature.


Bug: Pasting any X_{Y}.Z becomes X_{Y.Z}

Pasting something like

gives

for anything in the form of X_{Y}.Z
When I disable this extension, the bug no longer appears.

Thanks for sending this in.
As a temporary workaround, you can disable the Wolfram To Desmos plugin:

Fixed by a71241ce778d31cc170ccfae4c99957358f26e7c


ABCNews Add hls support

ExampleURL:http://abcnews.go.com/ThisWeek/video/week-exclusive-irans-foreign-minister-zarif-20411932
data_feed_url json in the page has two mp4 files and one hls list

Though slightly different, is already requested in #3992.

ABC news will be supported in the next version.


"use-after-free" occurs inside mrb_unpack_backtrace()

For easy reproduction, conf.enable_debug and MRB_GC_STRESS are required after applying the patch #6159, but can be checked with bin/mruby -e 'eval "0/0"'.
% rake -m MRUBY_CONFIG=ci/gcc-clang clean gensym $(pwd)/build/full-debug/bin/mruby

% lldb17 -- build/full-debug/bin/mruby -e 'eval "0/0"'
(lldb) target create "build/full-debug/bin/mruby"
Current executable set to '/var/tmp/mruby/@WIP/build/full-debug/bin/mruby' (x86_64).
(lldb) settings set -- target.run-args  "-e" "eval \"0/0\""
(lldb) r
Process 16851 launched: '/var/tmp/mruby/@WIP/build/full-debug/bin/mruby' (x86_64)
Process 16851 stopped
* thread #1, name = 'mruby', stop reason = signal SIGSEGV: address not mapped to object (fault address: 0xffffffffffffffff)
    frame #0: 0x000000000047e776 mruby`get_file(info=0xffffffffffffffff, pc=7) at debug.c:12:10
   9      mrb_irep_debug_info_file **ret;
   10     int32_t count;
   11
-> 12     if (pc >= info->pc_count) { return NULL; }
   13     /* get upper bound */
   14     ret = info->files;
   15     count =  info->flen;
(lldb) bt
* thread #1, name = 'mruby', stop reason = signal SIGSEGV: address not mapped to object (fault address: 0xffffffffffffffff)
  * frame #0: 0x000000000047e776 mruby`get_file(info=0xffffffffffffffff, pc=7) at debug.c:12:10
    frame #1: 0x000000000047eb15 mruby`mrb_debug_get_position(mrb=0x00000504b4a09000, irep=0x00000504b4a6a0f0, pc=7, lp=0x00000008212edc0c, fp=0x00000008212edc00) at debug.c:138:35
    frame #2: 0x0000000000475052 mruby`mrb_unpack_backtrace(mrb=0x00000504b4a09000, backtrace=0x00000504b4a2b910) at backtrace.c:170:10
    frame #3: 0x0000000000475276 mruby`mrb_print_backtrace(mrb=0x00000504b4a09000) at backtrace.c:265:63
    frame #4: 0x00000000004169c0 mruby`mrb_print_error(mrb=0x00000504b4a09000) at error.c:633:7
    frame #5: 0x0000000000403e6f mruby`main(argc=3, argv=0x00000008212edee8) at mruby.c:367:9
    frame #6: 0x0000000823318afa libc.so.7`__libc_start1 + 298
    frame #7: 0x0000000000403820 mruby`_start at crt1_s.S:83

I will describe the amendments that came to mind. Here are two that seem realistic.

(1) Use packed_backtrace() to convert to line numbers and file names (mrb_sym). Do not use struct backtrace_location::irep.
=> (disadvantage) Increased slowdown on exceptions. Exceptions can be ignored if we are aware that they are really "exceptions".

(2) Add MRB_TT_BACKTRACE to refer to RProc instead of struct backtrace_location::irep.

The second proposal may be better. I haven't done the actual work yet, so please comment if someone else will work on a fix that includes a different method.
Here is the proposed modification that is problematic.

(3) Add the reference count of irep after each_backtrace(mrb, ciidx, pack_backtrace_i, &ptr) in packed_backtrace(),.
=> (fatal flaw) It is expected to fall into a loop of exception generation due to the occurrence of reference count overflow.

(4) Change struct backtrace_location::irep to a reference to RProc.
=> (fatal flaw) There is no way to mark it in GC.

(5) Use MRB_TT_ARRAY instead of MRB_TT_DATA.
=> (big disadvantage) it increases memory usage.


Page Size not working dynamicilly?

I am using jsPDF for multiple page pdf using add HTML feature. I have divided my page in the array of elements. Calculating each div height size using js, I set each pdf page size.
Here is the sample:
var pdf = new jsPDF('p', 'in', set_height, 11]);  // set_height is HTML Div elements height calculated using offsetHeight or using window.computedStyle. By end, I am getting same result. When I zoom my browser, the content in pdf not fixed and large blank space found from bottom.

Right now we dont have a dynamical generation of pdfs.

Right now we don't have a dynamical generation of pdf?
I don't understand what you said. I am not saying the dynamic generation of PDF. I am saying how can we fixed the height of PDF for each page if I have multi-page. I resolved the multi-pages issue and the only thing I need is to fix the height.
See for example:
var pdf = new jsPDF('p', 'in', set_height, 11]); // set_height is HTML Div elements height calculated.
The set_height is not working the desired behavior. Even it's not set my pdf height perfectly. You can think the size differs by 200px which is not acceptable. I am using this JSPDF for the actual live site. Please help me here and do not close this ticket until you ensure there is no solution so I can move to another plugin.

I write like this initially.
new jsPDF('p', 'in', [set_height, 13]);
After that, I put this syntax to add the new page.
pdf.addPage('p', 'in', [set_height, 13]);
window.getComputedStyle(totalPage[i].height);   // Total page is the offset height for each container in pixel.
 (TotalHeight * 0.0104166667).toFixed(); // Calculate in Inch.

The height of pdf is not the exact height calculated in inch.

Hmm
Actually the parameters for addPage are height and width and not orientation and size.
And this cant be right: window.getComputedStyle(totalPage[i].height);
It has to be window.getComputedStyle(totalPage[i]).height;
Why not directly totalPage[i].offsetHeight ?
And pixels to inch depends on your system i think. PC mostly uses 1/96 like you did and Mac uses 1/72. But you say, that your document is taller than expected. So probably TotalHeight is not correct?

Actually the parameters for addPage are height and width and not orientation and size.
I only show you an example. I know how to calculate height. It's a typo.
totalPage[i].offsetHeight  // Tried also not an issue
PC mostly uses 1/96 like you did and Mac uses 1/72????? // please explain it.
Also, It's not an issue based on mac and PC.  Why will height be different if I am setting the height?
Please suggest your solution if I need to set height according to the browsers Div container height.

let totalPage = this.singlePage.nativeElement.querySelectorAll('.description');
let set_height = [];
totalPage.forEach((description, index) => {
console.log(window.getComputedStyle(totalPage[index]).height);
set_height.push((Number(window.getComputedStyle(totalPage[index]).height.toString().substring(0, window.getComputedStyle(totalPage[index]).height.toString().length)) * 0.0104166667).toFixed());
});
var pdf;
console.log(set_height);
var pdf = new jsPDF('p', 'in', [set_height[0], 11]);
for (let i = 0; i < totalPage.length; i++) {
  pdf.addHTML(totalPage[i], () => {
    pdf.setPage(i + 1);
    if (i < totalPage.length - 1)
      pdf.addPage(set_height[i+1], 11);
    if (i == totalPage.length - 1) {
            let data =pdf.output('dataurl');
            data = this.b64toBlob(data.substring(28, data.length),'application/pdf',512);
            var url = URL.createObjectURL(data);
            var html = '' +
             '' +
             '' +
             '' +
             '';
             window.document.write(html);
             window.document.close();
         //   window.location.href = blobUrl;
         var a = document.createElement("a");
         document.body.appendChild(a);
         a.style.display = "none";
         var blob = new Blob([data], {type: "octet/stream"});
         var url = window.URL.createObjectURL(blob);
         a.href = url;
         a.download = 'Enhance Therapy 3 Per Page exercises.pdf';
         a.click();
         window.URL.revokeObjectURL(url);
         this.completePdf = true;
    }
  });
}

I recommend to read the book "Clean Code" from Uncle Bob.
This part is totally fucked up:
totalPage.forEach((description, index) => {
console.log(window.getComputedStyle(totalPage[index]).height);
set_height.push((Number(window.getComputedStyle(totalPage[index]).height.toString().substring(0, window.getComputedStyle(totalPage[index]).height.toString().length)) * 0.0104166667).toFixed());
});

Should be more like
totalPage.forEach((description, index) => {
console.log(window.getComputedStyle(totalPage[index]).height);
let heightOfElement = parseInt(window.getComputedStyle(totalPage[index]).height);
let heightOfElementInInch = heightOfElement / 96; // conversion is muliplication with 1/96
set_height.push(heightOfElementInInch.toFixed(3));
/* If we dont supply a parameter to toFixed it will be rounded to integer, resulting in a too short or too long page */
});

Sorry, but actually I am sick to fix the height. I tried all hell of the solution, reading each and every thread of issues in this. I did this code very long ago but now my site is live so I am fixing the issues
This part is totally fucked up:  :) every  developer story

Did it work?