Update and maybe auto update ?

Hey,
I've tried updating the content by running every tasks, but it did not give me much: https://github.com/BuonOmo/ruby-changelog
Are you aware of a change that may make this codebase not working anymore?
Also, don't you think it would be a great idea to have some kind of monthly autoupdate of this site ? I could work on that bit

hi @BuonOmo, thank you for your comment and really sorry for not replying back then 🙈 I didn't spend much time on the project in `22..
Few things have changed since then. I updated README with current rake tasks:
rake fetch:cve
rake doc readme

The idea with auto-update is not so simple, because I really want to pick the most important changes by hand. When I was starting this project it was also meant to be an occasion for me to stay up to date with Ruby latest features 😅

Worth mentioning - since then I've also added a timeline page - https://rubychangelog.com/timeline

Got you, closing then thank you !


React Native

The same approach I suggested for react-intl https://github.com/yahoo/react-intl/issues/600

I'm curious what this would entail for rebass specifically. I'd imagine most of that logic could be contained within the Base component.

Can't be, because Base can not require / import both from React and React Native to be usable universally. Conditional require is no-go since dead code removal would not work in Webpack 2. I think we should have two different higher order components for that.

I'm curious what this would entail for rebass specifically.

span can not be rendered in React Native. We need Text. Otherwise almost everything should be the same, except platform specific stuff of course.

The world needs universal UI with the common dictionary. Your library is the unicorn because it's only presentional and very thoughtful.

Hi @jxnblk , I like your work and wanted to use something like rebass for native, so I built react native component library off a modified core of rebass. @steida, it definitely isn't striving toward universality, but if you're looking for a similar solution maybe check it out. :)
https://github.com/thomaswright/bayst

I'm hoping with the new setup in v1 that exporting a react native version using react-primitives is something I can get to in the near future, but for now I'm considering this library web-centric as I never do any work on native applications

Hi @jxnblk we're interested in having a universal UI library, if this is something that aligns with the goals of this project we could help out with the work required to take it in that direction

I am also interested in this !!!

I'm open to considering this. Still not sure if it's something that should be part of this repo, or a separate one, but would be open to a PR for discussion. I think the first thing I'd try is using styled-components/primitives and a mapping between a component's type and a relevant react-primitive

I found the following article relevant to this topic: https://medium.com/styled-components/announcing-primitives-support-for-truly-universal-component-systems-5772c7d14bc7. Should the issue be reopened to keep the discussion more visibly alive ?

Thanks, @jxnblk that's exactly what we were thinking and @rsaccon we were basing a lot of the ideas in that article.
We gave it a quick go and it still feels like styled-components/primitives are not ready, we had a few issues and found it a bit too limiting. This isn't an immediate need for us so we won't be exploring it further for a least a few months, but we'll come back to this once we start working on our react-native integration to see if things have changed.
Thanks

@airato maybe open sourcing zbase could fill in this gap...


Heartbeat management

Hi CanOpenTeam,
I'm trying to use heartbeat management to check the communication between two boards (one could work as the master). When slave stop to trasmit his hearbeat (due to hardware issue for example), master board, that is the consumer for the slave's hearbeat signal, publish an error message to the bus, but I don't see any callback that I can use to do something else on the application layer.
Can anyone help me to understand what is the better solution to manage the hearbeat?
Best regards,
Giulio Dalla Vecchia

I've found the solution on my own. It's just enough to set CO_HBconsumer_initCallbackTimeout callback.


Fix delete operation for OraLlamaVS

Description

Fix delete operation in OracleDB implementation for VectorStore.

Motivation

delete method should implement delete operation using document id, not node id.

class BasePydanticVectorStore(BaseComponent, ABC):
    @abstractmethod
    def delete(self, ref_doc_id: str, **delete_kwargs: Any) -> None:
        """
        Delete nodes using with ref_doc_id."""

Fixes # (issue)
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[x] Yes
[ ] No

Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

[x] I've tested this in my project
[ ] I added new unit tests to cover this change
[ ] I believe this change is already covered by existing unit tests

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Hmmm... might need to adjust that coverage threshold. I wish it just looked at the integration package instead of pulling in core as well

If we aren't already testing the delete method though in oracle db, let's add a test for it?

Let me add a test for this on Monday. I don't have this repo cloned, been adding changes through Github UI


Fixed TypeError when formData is given

PR Checklist:

[x] I have run npm test locally and all tests are passing.
[x] I have added/updated tests for any new behavior.

[ ] If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description
Fixed TypeError: The header content contains invalid characters when formData is given

I'm closing this because no action has been taken for almost a year


Test Requirements

The following is the proposed test environment for the Hive Metastore connector.

The following issue is to track the progress of implementation and review.

The first step is to build helm charts that deploys the two egeria platforms and adds the connector jar.

See #5 to get a hive metastore setup.
For Egeria one route we have that would probably work:

Use the egeria-base helm chart - this deploys kafka & a single egeria platform
Switch off the auto-configure & Initially use an external script to configure the two required servers
Implement a CI/CD process for this connector that also publishes a docker image (as we do for postgres)
Override the configuration of the helm chart to use this custom image

Subsequently the configuration of the servers could be done by either
a) Modifying the base chart to inject a custom configuration script
b) Moving to the operator, and providing server configuration documents
For CI/CD of this connector, we might be able to use Kind (see https://github.com/odpi/egeria-database-connectors/blob/main/.github/workflows/connector-test.yml) depending on the ram/storage requirements - in particular of the hive aspect.

I don't have a full enough understanding of where Egeria is now, how we got here or even where we are trying to go , to be able to address these comments as fully as I'd like, and it will be after the connector goes into production that I'lll be able to action this.
In essence this connector is a simple java jar application , so it's difficult to understand why there is an element of risk around building a basic CI pipeline. Have we considered using some of the industry standard tools such as some of the following
CI Tools
These tests will be extremely generic , and can be used by any OMRS connector , all that changes it the entity details on both ends of the connector / cohort. i.e What goes into the system connected to , and what gets returned from an Egeria query on the other side of the cohort.
@mandy-chessell @cmgrote  I was wondering if this could be an opportunity to centralize some of the testing needed by all OMRS connectors, I'm primarily thinking  of trying to reduce the maintenance concerns around some of our existing connectors.
Could we develop a CI pipeline that could accept any connector jar, and run a generic set of tests at the getEntity level.
It would be nice if the base Egeria image didn't contain kafka , if we split the kafka service out it will simplify the charts, especially if we can reuse kafka charts maintained elsewhere.
a chart for egeria platform where the JVM command line can support loader.path options
Everything else is can be configured via issuing REST commands in whatever binding we choose.

CI/CD
From that list, we used to use Jenkins in the early days - that has since changed a lot into Jenkins-X which is based on Tekton, and this does form part of the CI/CD pipeline native to Openshift. It's been used to build a connector for another client. Within the core Egeria project we also used Azure DevOps initially, but then moved to GitHub Actions, in part as it has the largest palette of actions, is native to our source code management system, and provides up to 20 7GB 'executors' without charge.
Dev/test workflows should be buildable with any of these tools, so I would suggest that we stick with GitHub actions unless we need capability not available. If we were to look elsewhere, Tekton is the most interesting as it would be good to also ingest metadata from it.
We have an Egeria CTS chart which @cmgrote put together. This already offers the ability to inject a connector jar in for testing. We don't yet have the execution of this automated, but I absolutely see that is relevant to support development of a repository connector. The CTS workbenches are written in custom java
For other testing, we have FVTs -- but these run in a generally simpler environment with a single platform, no kafka, and are scripted through groovy.
We have several aspects to all of these tests

Providing the environment for their execution (egeria platforms, kafka etc)
Providing the specific connector (this repos output jar)
A framework to execute tests

In the postgres repo I provided a simple pipeline which deployed a helm chart with egeria etc & got to the point where we execute tests. It used Kind (kubernetes in docker) -- but the tests themselves were not written.
So could we have connector testing in a ci/cd pipeline - absolutely

initially try with GitHub actions & native agent, plus CTS
consider github actions with an external agent, or remote to a k8s cluster if ram limits etc insufficient
then can look at better ways of managing the test driver -- there are tools for this.

Kafka
The base egeria image contains a kafka connector (including client jars). It does not contain a kafka server. That connector could be split out into a seperate repo, but we'd really need to have a 'good enough' default connector built in. perhaps the in-mem one.
However I think your point is about the kafka service. For the base/lab charts this is included as a Helm dependency - this means we only spec that dependency, and any overriding values. We do not maintain/include the chart itself. We are ALREADY reusing... Currently we use strimzi for those charts, and bitnami (used previously) for the cts/pts
In the early vdc charts we also had an option to just use an existing kafka deployment, such as IBM Event Streams, and testing was done for this. That's not in the current charts, but easy to add if required. Also the operator itself focusses on core egeria and it's config, not other components - and will use helm to aggregate other components needed for a deployment (demo, prod, test etc)
JVM options
These can already be set. We use the redhat UBI8 openjdk image which provides a set of configuration options, and other environment variables can be set in the PATH. This can be made better (ie to append/prepend rather than replace), but there is no limitation to pointing to a connector jar already
REST commands
This is already how the charts work - the base one, postgres, cts. They all configure via scripts. The operator will augment this and use configs directly.

Links:

Our old devops environment
Postgres kind test pipeline - source and action
CTS helm chart & automation issue
UBI image
Strimzi
VDC charts

Is it possible to remove the Spark Driver application and populate HMS via a script?

@mandy-chessell
The script would have to mimic how spark utilized that metastore, you'd have to basically build an emulator.
Building a spark sample that creates the necessary entities would match what the spark team are requesting.

Support for spark process information is not very interesting to Egeria ecosystem because of the OpenLineage support that captures the processing of spark jobs.  What I hoped we were getting was a catalog of Hive tables that are being created and accessed.
If the spark processor is needed then it suggests that a set of archetypal spark jobs that perform all of the processing patterns of interest are also needed to drive the right patterns of metadata into HMS.

what your getting isn't that much different,
you will be getting a catalog of the Hive tables that are being created and archived as part of higher level spark processing. service.


Feature/nanovdb

moved nanovdb -> nanovdb/nanovdb
removed Makefile, docker, ci, and .clang-format
Added several improvements to NanoVDB itself

@Idclip - can you confirm that this PR reflects some (not all) of the changes we discussed on the last TSC?

@kmuseth this looks good to me!

Closing this PR (since it removed the viewer) and create a new merge_candidate


Question - Error messages

This is a question about error messages that are sent to persistent notifications.
There are situations in that we expect an error to occur. Like turn off charging, when margin is not active.
On personal automations, is easy to mitigate, but with other integrations or blueprints, it's not.
Would be nice to have the option to disable these notifications.

I saw the that workaround. It would be nice to have the option to enable/disable persistent notifications (keep the log only)
But I understand that this is a minor case.
Thanks

The automation approach gives more flexibility if you only want to receive some notifications rather than simply turning them all off. And  each charger determines what is sent as an event. I use it to suppress SystemTime events and keep the rest for example.


cannot use a netstandard class library in an ASP.NET 4.6.2 project

i know we're not supposed to file bugs here, but none of the other repos seemed appropriate.
i'm getting the following error when running an ASP.NET 4.6.2 web application that includes a .netstandard1.5 class library:
Could not load file or assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.

I think the issue is that the reference assemblies aren't copied to the bin directory of the class library, and aren't copied to the bin directory of the ASP.NET project. What's worse is that there could be dozens of such missing reference assemblies, there's no error/warning during build that they're missing, and it's only at runtime do these errors become apparent. the behavior here is different from the non-portable class library case where assemblies referenced by the class library are transitively referenced by the project that imports the class library.
i should also note that this isn't specific to ASP.NET projects, other 4.6.2 project types (eg console) also suffer from this failure to correctly import assemblies referenced by netstandard class libraries.
Assembly Load Trace:
Assembly manager loaded from:  C:\Windows\Microsoft.NET\Framework64\v4.0.30319\clr.dll
Running under executable  C:\Program Files\IIS Express\iisexpress.exe
--- A detailed error log follows.

=== Pre-bind state information ===
LOG: DisplayName = System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
 (Fully-specified)
LOG: Appbase = file:///c:/WebApplication3/WebApplication3/
LOG: Initial PrivatePath = c:\WebApplication3\WebApplication3\bin
Calling assembly : ClassLibrary1, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null.
===
LOG: This bind starts in default load context.
LOG: Using application configuration file: c:\WebApplication3\WebApplication3\web.config
LOG: Using host configuration file: C:\Users\user\Documents\IISExpress\config\aspnet.config
LOG: Using machine configuration file from C:\Windows\Microsoft.NET\Framework64\v4.0.30319\config\machine.config.
LOG: Post-policy reference: System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
LOG: Attempting download of new URL file:///C:/Users/user/AppData/Local/Temp/Temporary ASP.NET Files/root/4dfb4bb4/2cddecba/System.Runtime.DLL.
LOG: Attempting download of new URL file:///C:/Users/user/AppData/Local/Temp/Temporary ASP.NET Files/root/4dfb4bb4/2cddecba/System.Runtime/System.Runtime.DLL.
LOG: Attempting download of new URL file:///c:/WebApplication3/WebApplication3/bin/System.Runtime.DLL.
WRN: Comparing the assembly name resulted in the mismatch: Build Number
ERR: Failed to complete setup of assembly (hr = 0x80131040). Probing terminated.

here's the repro (using VS2015.3):

create new project (WebApplication3)

4.6.2
ASP.NET empty web application
select 'create directory for solution'
set as startup project

right click WebApplication3

choose 'Add Global Application Class', Global, OK

add new 'C# Class Library (Portable)' project (ClassLibrary1)

choose .NET Framework 4.6 & ASP.NET Core 1.0 targets

get properties on ClassLibrary1

click 'Target .NET Platform Standard'
select .NETStandard1.5

add reference to ClassLibrary1 to WebApplication3
in Global.Application_Start, add line:

"new ClassLibrary1.Class1();"

f5

here's the project:
WebApplication3.zip

@nguerrera sorry to bother you Nick, is there any chance you can take a look at this one, too?

i should also add that there's no warning when building after importing a nuget package that depends on a version of the .netstandard higher than that which the classlibrary is targeting.
than that which the classlibrary is targeting.

moved to here: https://github.com/Microsoft/msbuild/issues/1463


Needs those issues to be solved:

[ ] golangci/golangci-lint/issues/2649

https://github.com/cailloumajor/opcua-proxy/pull/87 superseded this.


Misplaced configuration file on non-Windows systems

The configuration file gets saved as a file named config\forcecrawl.cfg on non-Windows systems. This is caused by this line which assumes the file separator is a backslash, which (to my knowledge) is a Windows only thing.
This can be fixed by replacing the line with either of these:
// This should still work on Windows systems, but might not on some other systems
public static File configFile = new File(FabricLoader.getInstance().getConfigDir() + "/forcecrawl.cfg");

// This ensures using the correct separator on every system
public static File configFile = new File(FabricLoader.getInstance().getConfigDir() + File.separator + "forcecrawl.cfg");

// This uses the fact FabricLoader#getConfigDir() returns a Path object
public static File configFile = FabricLoader.getInstance().getConfigDir().resolve("forcecrawl.cfg").toFile();

Attached is a screenshot of my .minecraft folder

As this does not break the loading/saving of the config, I would understand if it were a low priority issue.

This will be fixed in the next release.


Duplicate paragraph #12

Minor fixes (proposal)

This change is

Coverage remained the same at 54.731% when pulling 08ae0fef1236f3792d2aff30fcb2148961bd180f on kant:patch-2 into 1878cf7d92f378da99ae7ac9a76534d27bc67a20 on IBM:master.

@kant  good catch!
(I changed it first to dev, and then will go to master)


Render only what on the screen but not overall

When I download to pdf, it's able to get or render what shows on the screen but not all, and everything else after that is blank. Is there any solution to this?
html2pdf(document.getElementById('export-to-pdf'), {    filename: 'download.pdf',    margin: 5, })

Found the solution by using windowHeight.
html2pdf(document.getElementById('export-to-pdf'), {                     filename: 'download.pdf',                     margin: 5,                     html2canvas: {                         scale: 5,                         imageTimeout: 15000,                         logging: true,                         useCORS: true,                         allowTaint: true,                         letterRendering: true,                         height: window.outerHeight + remainingHeight,                         windowHeight: window.outerHeight + remainingHeight,                     },                     pageBreak: { mode: ['avoid-all', 'css'] },                 })


[Bug] App hides itself when minimizing and sometimes even closes

Describe the bug
When minimizing the app, it doesn't show up in the recent items.
I have set up DuckDuckGo to clear browsing data when I close the app. This bug makes it look like I closed the app, while I did not, so next time I open it, since I had never actually closed it, it will keep the browsing data from my last session.
Not always, but sometimes it also closes when I minimize, so I lose the opened tabs.
How to Reproduce
Here is a video. This video will be automatically deleted in 3 or so months, so if you need me to reupload it in the future, please let me know
https://streamable.com/lk5ptm
Expected behavior
The app should show up in the recent apps list
Environment
- DDG App Version: 5.207.2
- Device: Motorola Edge 40
- OS: Android 14

@bergamin Thanks for reporting this. It should be fixed. Can you please try again using the latest version?

Hello, @anikiki. It's working now! Thank you very much!


[banks-server, rpc] fix: Enable CPI tracking by default

Problem
Before this change, simulating a transaction from banks-client (with BanksClient::simulate_transaction) did not include inner instructions in TransactionSimulationDetails. That's because the CPI tracking feature was not enabled by default in RpcSimulateTransactionConfig.
Not enabling it by default makes it impossible to test "compression" (logging events through noop program) in Rust tests (with solana-program-test), because there is no way to enable that feature through solana-program-test or solana-banks-client.
Enabling it makes such test cases possible[0]. It makes it easy to simulate a transaction, then fetch and serialize an associated event logged through noop program in Rust tests, without having to do it in JS/TS or relying on external indexers.
[0] https://github.com/Lightprotocol/light-protocol/blob/9e8c13e26c1e019f92b32025c580809ea04fd375/test-utils/src/lib.rs#L212-L232
Summary of Changes

Always set enable_cpi_tracking to true in banks-server.
Set instruction_data in Default impl of RpcSimulateTransactionConfig to true.

@joncinque sorry for late reaction. Sounds great, I will make new PRs.


[GHSA-5vj8-3v2h-h38v] Remote Code Execution in next

Updates

Affected products
Description
Source code location

See also discussion at https://github.com/github/advisory-database/pull/179

Thank you for the report @medikoo!
In reviewing this advisory, we noticed that it had a link to this repo discussing a directory traversal vulnerability, which is in reference to CVE-2018-6184 and reflected in this other advisory. We've removed this link from GHSA-5vj8-3v2h-h38v, but in doing so we had trouble finding other public references to include for this particular vulnerability advisory. We weren't able to find details for this in the 5.1.0 release notes either.
Do you or the current maintainers (@timneutkens and @ijjk) know of any other public references for GHSA-5vj8-3v2h-h38v?


Better rationale of repo and diff with existing rlnjs

Currently https://github.com/Rate-Limiting-Nullifier/rlnjs has existed for a while, and we created a very similar js-rln repo.
It should be clear (a) why there's another repo and (b) what is different/similar
(I know ~roughly why in terms of Zerokit etc, but it should be clear for new people and also reference existing work).
cc @fryorcraken

Need to add README to specify that js-rln is a wrapper to zerojit's RLN feature (exported as WASM for the browser). that is opiniated to Waku.
In addition to allowing:

generation of credentials
seeded generation of credentials (ie, from a signature done using an Etheruem account)

It implements js-waku interfaces to enable:

attaching proof to ongoing waku messages
API to check proof of incoming waku messages


False Positive 0001:doctype/present about file with PHP processing instruction

Given the following well-formed and valid XHTML 5.2 file
<!DOCTYPE html>
<?php ?>

    <title>Milo Test</title>

Milo Test

When running milo review test.xhtml, it reports a false positive with the following output:
pi.xhtml:3:1: doctype tag must be present in the document (0001:doctype/present)

Thanks for reporting this one. I think is related with the fact that we're checking that the doctype is next to the html tag. Will cover it soon.


Reading Numbers

Hello,
I have a column with a formula to get numbers and i cannot retrieve those numbers:
package main

import (
	"fmt"

	"github.com/extrame/xls"
)

func main() {
	if xlFile, err := xls.Open("file.xls", "utf-8"); err == nil {
		if sheet := xlFile.GetSheet(0); sheet != nil {

			for i := 2; i <= (int(sheet.MaxRow)); i++ {
				row := sheet.Row(i)
				if row != nil {
					col := row.Col(3)
					fmt.Print("\n", col)
				}
			}
		}
	}

}

Everything else (text) i can retrieve

The column that i want to retrieve if has numbers i get  "General", if the column has formulas i get nothing in the screen.

This problem solved in my fork https://github.com/sergeilem/xls .


Fix libc.dylib import on El Capitan

El Capitan SIP protection causes the LoadLibrary call to fail unless an absolute path is supplied for libc.dylib.

Merged with some adjustments: cc80a806548e99702aff98529f80f44a8bbb5f74.  Thanks!


Automatic Price Conversion

Automatic Price Conversion #383
Also discussed in https://github.com/lunarphp/lunar/discussions/763
In addition to the price conversion, there were other requirements for implementing the feature.
Most important part was support for background jobs, both in core and admin hub.
The PR contains 3 main parts:

Background jobs support in lunar
Progress bar column for livewire tables
Automatic price conversion

Background jobs

Background jobs are implemented in lunar core based on laravel job batches.
I opted to use job batches (instead of single jobs) because of the features for execution flow of jobs and also support for grouping/naming them.
The job_batches table is a slightly extended version of the laravel's official table, but is compatible with it.
Also added a JobBatch model for working with job batches data in the admin hub.
A basic job batches listing screen added into the hub setting (below the activity logs)

Automatic price conversion

The Price and Currency observers are responsible for dispatching price conversion job batches, when:

a non-default currency exchange rate changes
default currency price of a purchasable changes

The jobs execute in background (as a job batch)
a new config file pricing.php has been added (main parts are fully configurable).

possible improvements
There is still some places which can be improved:
For example, some events could be triggered in certain points of conversion process.
Another improvement could be in product/variant edit page (price section), to inform staff about the last auto-update operation.
Other things that are still missing:

Docs
Tests

Thanks for this @armezit , certainly looks interesting. Having an initial play around I got this error:
Call to undefined method Illuminate\Database\Eloquent\Casts\Attribute::make()

Steps to reproduce:

Created a new currency
Updated the exchange rate (see row in job batches)
Go to the background jobs page.

Call to undefined method Illuminate\Database\Eloquent\Casts\Attribute::make()

@alecritson Fixed it now.
The make() method was introduced in laravel 9.2. Replaced it with older syntax to support laravel 8.x too.
Also added a unit test for JobBatch model.

Looks like there are still a couple of issues, possibly with L8
When updating prices on a product: https://flareapp.io/share/Vme4Z2z5#F98
When viewing the background jobs page: https://flareapp.io/share/B5ZwXrgm#F106

Yes I see. Seems with L8 we need some refactors.
I'll check the overall process on L8 and write more tests

Added more unit tests, which covers most of the code.
Also fixed reported issues (incompatibilities with Laravel 8).

We still need to review this properly, but priorities have prevented it. Due to this and file conflicts, I have converted to Draft for the moment.

We are looking to take a different approach for this, which allows dynamically converted prices. If we decide to incorporate this functionality, it will be in 1.x now, so I will close this PR.


Bind Port in server

can be use single port on the server machine instead of 2?

You'll need 1 port per session / path.


Encounster "RuntimeError: CUDA error: device-side assert triggered" issue to reproduce finetune of scripts/finetune_guanaco_7b.sh

Falied to finetune finetune_guanaco_7b:
 File "/home/ubuntu/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 465, in _prepare_decoder_atten                               tion_mask
    combined_attention_mask = _make_causal_mask(
  File "/home/ubuntu/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 49, in _make_causal_mask
    mask = torch.full((tgt_len, tgt_len), torch.tensor(torch.finfo(dtype).min, device=device), device=device)
RuntimeError: CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with `TORCH_USE_CUDA_DSA` to enable device-

Here is the screenshot:

My CDUA is CUDA Version: 11.8 and  I use the Nvidia A10 which is 24 GPU memory to finetune this.

FYI: was able to reproduce the training using Nvidia A10 at AWS a couple of days ago without any error. I was using AWS-supplied ubuntu 20.04 Pytorch 2.0.0 AMI image.

@JustinZou1  I was getting the same error with decapoda-research/llama-7b-hf but the error went away using huggyllama/llama-7b.

@ag1988 I tried huggyllama/llama-7b also,  it works.Thanks for you help.


"Add type annotations for all fixture references in this file" code action is shown in non-test file

Code Snippet
import pytest

@pytest.fixture
def first_entry():
    return "a"

def foo(first_entry):
    pass

Repro Steps

pip install pytest
Place the code above in a file that does not match pytest's file filter (ex. does not start with test_ or end with _test).
Click somewhere in the file.

Expected behavior
No pytest code actions.
Actual behavior
"Add type annotations for all fixture references in this file" code action is suggested.
"Add type annotation to first_entry" is suggested if you click on the first_entry parameter of foo
Rationale
Not a huge deal, but as a first time user of the pytest features (and pytest) I was confused about why Pylance was presenting these code actions yet none of the other new behaviors were lighting up. It was because my filename wasn't matching what pytest expected.
For consistency, I think PytestExtensionImpl.addCodeActions should be doing no work unless _isTestFile is true.

Any file with fixtures in it would benefit from the code actions too. I believe it's already checking that? Let me double check.

Yes, that's correct.

The inlay hints work in that file though, right? I'm wondering what other functionality didn't work in a fixture only file?

As far as I could tell nothing else worked because _isTestFile was returning false. My filename was "PytestExamples.py" and I didn't have a pytest config file telling it to search for test files in a non-default way.

Yeah it checks for

Test functions (have to match the function and file pattern)
Fixture functions (should work anywhere)

The latter was what I was wondering wasn't working.

I believe this is working as designed.


phantom 4DBTi data includes a bunch of hidden macOS files

After downloading all the example data while setting up a new system, I noticed that a __MACOSX folder ended up inside mne_data.  It contains a bunch of finder / spotlight indexing files for the phantom 4DBTi dataset. The __MACOSX folder is not listed in the manifest on the OSF repo, but is present in the zip file.  In examining the OSF repo, I also noticed .dropbox and .DS_Store files inside the MNE-phantom-4DBTi folder, as well as a zero-byte text file called Icon (I dunno if this last one is a mac-generated file or a legitimate part of the dataset?)

it's a mac thing. You can clean this up !

OK, I've requested access to MNE's OSF project. Once you grant that, I can fix.

Granted


Why rewrite in golang?

Hi,
May I know why rewrite it in golang? Is there any benefit? Or it just an experiment?

We believe there are advantages in using Go.


[BUG/ISSUE] text cut out welcome screen

I guess this got fixed by https://github.com/marticliment/WingetUI/pull/1043

Yes


Update TripleLift endpoint

Could we leave the supplier_id as "GET_FROM_TRIPLELIFT" in the open source version? As long as that wouldn't be propagated to any host company that's successfully running our adapter today. id 39 is specific to Magnite's integration.


Inknown exception ?

The problem
Deze fout is ontstaan door een aangepaste integratie.
Logger: custom_components.nest_protect
Source: custom_components/nest_protect/pynest/client.py:177
Integration: Nest Protect (documentation, issues)
First occurred: 05:31:41 (1 occurrences)
Last logged: 05:31:41
Unknown exception. Please create an issue on GitHub with your logfile. Updates paused for 5 minutes.
Traceback (most recent call last):
File "/config/custom_components/nest_protect/init.py", line 141, in _async_subscribe_for_data
await entry_data.client.authenticate(entry_data.client.auth.access_token)
File "/config/custom_components/nest_protect/pynest/client.py", line 177, in authenticate
nest_response = await response.json()
File "/usr/local/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 1103, in json
raise ContentTypeError(
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: ', url=URL('https://home.nest.com/session')
What version of this integration (ha-nest-protect) has the issue?
v0.3.6
What version of Home Assistant Core has the issue?
2022.5.0
Device / Model
Nest Protect (Battery)
Diagnostics information
No response
Home Assistant log
Logs
```
Copy/paste any log here, between the starting and ending backticks (`)
```

Additional information
No response

Thanks @stefanuytterhoeven! I will have a look if I can improve the logging to figure out the source of this issue. Nest returns something that we don't expect, but in this case the response is not correctly logged...

Fyi:
I have three battery powered nest protects

Hi, I'm on 0.3.7 with more logging and also get this error.
2022-05-20 01:27:24 ERROR (MainThread) [custom_components.nest_protect] Unknown pynest exception. Please create an issue on GitHub with your logfile. Updates paused for 1 minute. Traceback (most recent call last): File "/config/custom_components/nest_protect/pynest/client.py", line 259, in subscribe_for_data result = await response.json() File "/usr/local/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 1103, in json raise ContentTypeError( aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: text/plain', url=URL('https://czfe34-front01-iad01.transport.home.nest.com/v6/subscribe') During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/config/custom_components/nest_protect/__init__.py", line 144, in _async_subscribe_for_data result = await entry_data.client.subscribe_for_data( File "/config/custom_components/nest_protect/pynest/client.py", line 263, in subscribe_for_data raise PynestException( custom_components.nest_protect.pynest.exceptions.PynestException: 400 error while subscribing -

Good day @iMicknl
I have also started seeing the Unknown pynest exception error this morning
`
This error originated from a custom integration.
Logger: custom_components.nest_protect
Source: custom_components/nest_protect/pynest/client.py:263
Integration: Nest Protect (documentation, issues)
First occurred: 08:21:42 (1 occurrences)
Last logged: 08:21:42
Unknown pynest exception. Please create an issue on GitHub with your logfile. Updates paused for 1 minute.
Traceback (most recent call last):
File "/config/custom_components/nest_protect/pynest/client.py", line 259, in subscribe_for_data
result = await response.json()
File "/usr/local/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 1103, in json
raise ContentTypeError(
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: text/plain', url=URL('https://czfe74-front01-iad01.transport.home.nest.com/v6/subscribe')
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/config/custom_components/nest_protect/init.py", line 144, in _async_subscribe_for_data
result = await entry_data.client.subscribe_for_data(
File "/config/custom_components/nest_protect/pynest/client.py", line 263, in subscribe_for_data
raise PynestException(
custom_components.nest_protect.pynest.exceptions.PynestException: 200 error while subscribing -
`

@craibo which version do you run?


Nested function for VS (ability to group nodes)

Issue description (what happened, and what was expected):
I was thinking about this issue for long time. In general we can group VS nodes by creating new functions, but I think it's not always practical to create artificial functions that have no other purpose than to 'group nodes' for some other parent function (and bloat function list view with many helpers function this way).  This is why I would like to propose 'Nested Functions' for VS. What I'm proposing is ability to group nodes by creating nested functions. I will try to explain how I imagine this feature with gif:

I think this would allow us to have very readable VS scripts, and use them also in 'creating algorithm phase' where you could add couple empty blocks with nice names and implement them later when your 'high level logic' is on the place.

How can add more node if all others faded in black when edit inner function?
And there should be "Ungroup" also, regrouping is needed in some case.

@volzhs I have limited gif creations super powers :) By 'everything is faded' I meant that main function vs sheet is faded. It's an idea proposal, not exact way how I would like to see it when introduced :)
Also dont think ungroup/regroup is needed in first implementation, you can simply copy/cut nodes (it works already) to use them in different group or in parent function.

maybe, i'm still not sure how useful is this, give you can create as many
functions as you want and copy/paste the nodes there.
On Wed, Aug 31, 2016 at 2:50 PM, kubecz3k<EMAIL_ADDRESS>wrote:

@volzhs https://github.com/volzhs I have limited gif creations super
powers :) By 'everything is faded' I meant that main function vs sheet is
faded. It's an idea proposal, not exact way how I would like to see it when
introduced :)
Also dont think ungroup/regroup is needed in first implementation, you can
simply copy/cut nodes (it works already) to use them in different group or
in parent function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6353#issuecomment-243844915,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z21WYgr4rXyzugxw2OA6W2aZL9178ks5qlb71gaJpZM4Jx3np
.

@reduz so there is no performance hit for creating multiple functions v.s. explicitly placing the nodes in VS

With Godot seeing more use of node-based systems in general, wouldn't it be better to have generic node groups with custom inputs and outputs like in Blender?
If anyone is not sure what the usefulness would be, consider that...

Node Groups could be selected from a list and be reused anywhere in a tree or in multiple trees (so long as the node-tree type is the same)
It allows for clean organization (no having to have hundreds of nodes visible at once in a very complex situation)
It allows for mass-scale changes with just a few tweaks (what would be changed in a node group for shaders would change every shader using it for example)
It allows the user to break down a complex setup into a set of easy-to-use, intuitive controls (no having to hunt down the right values to changes, especially since input values can be shared with many nodes).

In general, it's become a rule of thumb that a good quality node editor has something to group or organize nodes with, otherwise it can become very hard to work with.

I understand you guys are used to node groups from other systems, but as
far a now I am unconviced about the need for them, so they won't be
implemented anytime soon.
You can create functions and sub-call nodes, which should clearly cover the
same use cases.
On Sep 2, 2016 18:44, "Ace-Dragon"<EMAIL_ADDRESS>wrote:

With Godot seeing more use of node-based systems in general, wouldn't it
be better to have generic node groups with custom inputs and outputs like
in Blender?
If anyone is not sure what the usefulness would be, consider that...

Node Groups could be selected from a list and be reused anywhere in
a tree or in multiple trees (so long as the node-tree type is the same)
It allows for clean organization (no having to have hundreds of
nodes visible at once in a very complex situation)
It allows for mass-scale changes with just a few tweaks (what would
be changed in a node group for shaders would change every shader using it
for example)
It allows the user to break down a complex setup into a set of
easy-to-use, intuitive controls (no having to hunt down the right values to
changes, especially since input values can be shared with many nodes).

In general, it's become a rule of thumb that a good quality node editor
has something to group or organize nodes with, otherwise it can become very
hard to work with.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6353#issuecomment-244494643,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2xCtskBiMuCYWreQ8qyOPSEpEEOSks5qmJjYgaJpZM4Jx3np
.

@reduz to move some nodes into a new function you have to copy the nodes, create a function, paste them in, create input and output ports, connect the wires... vs. one click solution proposed in this issue

That's another issue, how about one-click create function? :)
On Fri, Sep 2, 2016 at 7:22 PM, procedural<EMAIL_ADDRESS>wrote:

@reduz https://github.com/reduz to move some nodes into a new function
you have to copy the nodes, create a function, paste them in, create input
and output ports, connect the wires... vs. one click solution proposed in
this issue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6353#issuecomment-244500897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2zl-bD-Yg1bB8UQtiiOTV3hKUIb9ks5qmKGsgaJpZM4Jx3np
.

@reduz sounds good :)

GamesFromScratch was recording first VS introduction yesterday, and one of his first comments was that it's not yet possible to do what I requested in the first post :P It's in this video, basically exactly at 13 minute:
https://youtu.be/dSUMCUsOeyU?t=13m1s

I agree with reduz there should be a one click create function and expand function into nodes.

I think the best way for organization would be a 'code collapse box' that could be used in different ways:

https://github.com/godotengine/godot/issues/18267#issuecomment-382490921
This can be expanded for the one unified graph enhancement @reduz mentioned in planning.
The approach is to have one graph. Use the existing custom node functionality but have the graph function collapse its nodes and be able to be callable. We can store the nodes inside of the custom node and expose it as a global library for sharing. The nodes can be collapsed and expanded at any point.

This was discussed in Google Summer of Code 2019, but the schedule was moved back. @swarnimarun Do you want to post your proposal for feedback?

Yep I am planning to finalize a proposal ASAP.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Clearing cache

Hi,
I was close to make myself a plugin like this, but then I found yours and noticed this is exactly what I had in mind. Nice!
One thing I'd suggest is the ability to clear the cache, because for large numbers of files and longer tasks it may nice to have memory freed as the task proceed.
I had two strategies in mind: one would be adding a second, boolean argument to restore, like this:
module.exports.restore = function (store, clear) {
    ...
    cache[store].forEach(function (file) {
        this.push(file);
    }.bind(this));

    if (clear) {
        delete cache[store];
    }
    ...
};

The other approach would be adding a clear method to the plugin, that clears the whole cache (or maybe just a selection of stores):
module.exports.clear = function () {
    var stores;
    if (arguments.length) {
        stores = Array.prototype.slice.call(arguments);
    } else {
        stores = Object.keys(cache);
    }

    stores.forEach(function(store) {
        delete cache[store];
    };
};

What do you think about it?

Hey,
Im happy to accept PRs, your second approach looks good to me.
If you are too busy I can implement it during weekend.
Thanks.

Closed with #7.


bug: ion-button inside ion-list-header is not visible

Bug Report
Ionic version:

[x] 4.x
Current behavior:

I wanted to have a button inside an ion-list-header, ut it doesn't seem to work since is not visible
Expected behavior:

Visible ion-button inside ion-list-header
Steps to reproduce:

Just write the following markup
Related code:
<ion-list>
    <ion-list-header>
        <ion-label>Some Text</ion-label>
        <!-- Button -->
        <ion-button slot="end" size="small" fill="clear" shape="round" color="danger">
            <ion-icon color="danger" name="trash"></ion-icon>
        </ion-button>
    </ion-list-header>
    <!-- Some items -->
    <ion-item>...</ion-item>
</ion-list>

insert short code snippets here

Other information:

Ionic info:

insert the output from ionic info here

Thanks for the issue. ion-list-header does not have slots, so doing slot="end" will cause the button to not show up. I was able to reproduce this on my end, and removing slot="end" fixed the issue.
Here is a CodePen with a working example: https://codepen.io/liamdebeasi/pen/oKNarB


add some additional modules/libraries

add random, network, etc. libraries/modules

I'm sure we'll add it later this year


Exception using word_timestamps=True in model.transcribe

Hi there! I was hoping to use whisper's ability to provide timestamps around the audio events your work captures.
I'm currently getting an exception when I pass through a True word_timestamps value to model.transcribe().
Thanks!
import whisper_at as whisper
model = whisper.load_model("small")
result = model.transcribe(audio_path, at_time_res=10, word_timestamps=True)

Traceback
[/usr/local/lib/python3.10/dist-packages/whisper_at/transcribe.py](https://localhost:8080/#) in transcribe(model, audio, verbose, temperature, compression_ratio_threshold, logprob_threshold, no_speech_threshold, condition_on_previous_text, initial_prompt, word_timestamps, prepend_punctuations, append_punctuations, at_time_res, **decode_options)
    344
    345             if word_timestamps:
--> 346                 add_word_timestamps(
    347                     segments=current_segments,
    348                     model=model,

[/usr/local/lib/python3.10/dist-packages/whisper_at/timing.py](https://localhost:8080/#) in add_word_timestamps(segments, model, tokenizer, mel, num_frames, prepend_punctuations, append_punctuations, **kwargs)
    310
    311     text_tokens = list(itertools.chain.from_iterable(text_tokens_per_segment))
--> 312     alignment = find_alignment(model, tokenizer, text_tokens, mel, num_frames, **kwargs)
    313     merge_punctuations(alignment, prepend_punctuations, append_punctuations)
    314

[/usr/local/lib/python3.10/dist-packages/whisper_at/timing.py](https://localhost:8080/#) in find_alignment(model, tokenizer, text_tokens, mel, num_frames, medfilt_width, qk_scale)
    193
    194     with torch.no_grad():
--> 195         logits = model(mel.unsqueeze(0), tokens.unsqueeze(0))[0]
    196         sampled_logits = logits[len(tokenizer.sot_sequence) :, : tokenizer.eot]
    197         token_probs = sampled_logits.softmax(dim=-1)

[/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py](https://localhost:8080/#) in _call_impl(self, *args, **kwargs)
   1499                 or _global_backward_pre_hooks or _global_backward_hooks
   1500                 or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501             return forward_call(*args, **kwargs)
   1502         # Do not call functions when jit is used
   1503         full_backward_hooks, non_full_backward_hooks = [], []

[/usr/local/lib/python3.10/dist-packages/whisper_at/model.py](https://localhost:8080/#) in forward(self, mel, tokens)
    271         self, mel: torch.Tensor, tokens: torch.Tensor
    272     ) -> Dict[str, torch.Tensor]:
--> 273         return self.decoder(tokens, self.encoder(mel))
    274
    275     @property

[/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py](https://localhost:8080/#) in _call_impl(self, *args, **kwargs)
   1499                 or _global_backward_pre_hooks or _global_backward_hooks
   1500                 or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501             return forward_call(*args, **kwargs)
   1502         # Do not call functions when jit is used
   1503         full_backward_hooks, non_full_backward_hooks = [], []

[/usr/local/lib/python3.10/dist-packages/whisper_at/model.py](https://localhost:8080/#) in forward(self, x, xa, kv_cache)
    210             + self.positional_embedding[offset : offset + x.shape[-1]]
    211         )
--> 212         x = x.to(xa.dtype)
    213
    214         for block in self.blocks:

AttributeError: 'tuple' object has no attribute 'dtype'

hi there,
This seems to be a bug and will be fixed in the next version.
Unfortunately I am working on another deadline and only have time to do this after 8/10.
-Yuan

Hi ,I have faced with the same issue and fixed it,
Just change the line 273 in package/whisper-at/whisper_at/model.py to this:
        return self.decoder(tokens, self.encoder(mel)[0])

This change worked for me!
On Sun, Aug 6, 2023 at 10:06 PM congtuong @.***> wrote:

Hi ,I have faced with the same issue and fixed it,
Just change the line 273 in package/whisper-at/whisper_at/model.py to this:
return self.decoder(tokens, self.encoder(mel)[0])
—
Reply to this email directly, view it on GitHub
https://github.com/YuanGongND/whisper-at/issues/4#issuecomment-1667192618,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM3FEETZON4RGM2TRRVGXTXUBZVXANCNFSM6AAAAAA3AUN2HE
.
You are receiving this because you authored the thread.Message ID:
@.***>

thanks @congtuong and @tallzilla , I will check this and put it in the next version.
-Yuan

thanks! this is fixed in whisper-at==0.5.
-Yuan


Can I copy actual HTML tags and css?

I read all the issues, it seems some people were able to copy HTML but maybe am missing something.
They seems like they wanted to just copy text, I want copy everything,  tags and css inside a specified element.
I used the code found #72 still just copying text without tags.
Please advise.

Was hoping and clipboard.js could support this, never heard got a response. I had to find another solution. I manage to remove the need for clipboard.js all together with a few lines of JS.
Good support for cut and paste using Document.execCommand()
copy/cut : Chrome < 42,	 Firefox  < 41 (41),	IE < 9, Opera < 29,  Safari <10
Here is working code with no dependencies. http://codepen.io/rgfx/pen/GjOmOL
Jquery with pop notification. http://codepen.io/rgfx/pen/yaPWGg
Hope this helps someone.

@rgfx clipboard.js supports copying any kind of string you pass to it.
new Clipboard('.btn', {
    text: function() {
        var htmlBlock = document.querySelector('.yourSelector');
        return htmlBlock.innerHTML;
    }
});

But I'm glad you were able to leverage your own copy to clipboard solution :)

@zenorocha thanks for the response, even it was tad bit late. Learned some more js so it wasn't all that bad.

🎉 🎉 🎉

What if you have multiple code blocks on one page? Do you need to instantiate a new clipboard for each block ('.yourSelector'), ('.yourSelector1')?
new Clipboard('.btn', {
    text: function() {
        var htmlBlock = document.querySelector('.yourSelector');
        return htmlBlock.innerHTML;
    }
});

This just copies the HTML as text, but when you paste it in Chrome it's not formatted as HTML... it's just text.  Is there any way to avoid this?

Maybe the advanced option documentation changed since 2018 but this doesn't seem to answer the question. All the examples seem to copy plain text (not text/html) from what I see.

Maybe the advanced option documentation changed since 2018 but this doesn't seem to answer the question. All the examples seem to copy plain text (not text/html) from what I see.

Hello @Johennes, did you solve this? I also want to copy the html code of an element as rich text, so that I can paste to Gmail web client (in browser).

Maybe the advanced option documentation changed since 2018 but this doesn't seem to answer the question. All the examples seem to copy plain text (not text/html) from what I see.

Hello @Johennes, did you solve this? I also want to copy the html code of an element as rich text, so that I can paste to Gmail web client (in browser).

@Dan-Do I ended up doing something else in the project where I needed this (mailto:me browser extension). I merely needed a nice reader-like view of the content, so I opted for extracting that with Readability and then copied the resulting HTML to the clipboard like this.

@Dan-Do I ended up doing something else in the project where I needed this (mailto:me browser extension). I merely needed a nice reader-like view of the content, so I opted for extracting that with Readability and then copied the resulting HTML to the clipboard like this.


Enable Fast Forward Merge between Staging and Master

Specification
Right now the automatic merge between staging and master occurs through a merge commit.
Having a merge commit in master is not a good idea because it means we have to remember to rebase our staging onto master each time after merge.
Github doesn't support native fast forward merges. https://github.com/github/feedback/discussions/4618
Therefore it has to be done manually within the Gitlab CI/CD.
This means the integration:merge job rather than just triggering auto-merge, it will need to do a proper git merge --ff-only staging.
git checkout master
git merge --ff-only staging
git push upstream master

The only issue right now is that we would have to ensure we check out the master. Which means using git remote add -f upstream ....
Note that gitlab's origin points elsewhere.
Now when doing the git push it will need authentication. I'd prefer not to use SSH keys for this usage, and instead an auth token that is capable of the push.
I believe we should be able to achieve this with our GITHUB_TOKEN being used for HTTP authentication.
Will need to understand how git does HTTP authentication.
Additional context

https://github.com/github/feedback/discussions/4618

Tasks

Find out about HTTP authentication for git
Try using it to do a git push
Make sure you can do a git checkout upstream/master in the CI/CD
Change the integration:merge job to do this, the PR can still be created, it will just be automatically merged once the integration:merge job is pushed up

Note that if we are doing this, perhaps we can also just do tagging directly... and push up those changes, it's kind of complicated since it is concurrent, and those tag commits may be in conflict with new changes. So we leave that till later.

It's dangerous to leave this to manual rebasing, even I forgot that the tag ended up pointing to nowhere cause the rebase occurred after the tagging of npm version.

I believe fast forward merge should be working now.
However the branch protection for commit status must be removed. This is because GH is blocking merges to master until the pipeline finishes. And in this case, the pipeline hasn't finished, while we are doing a git push upstream HEAD:master.
Or the the integration job will need to send in a commit status.  It would require using a curl call. For now let's not bother with this.
In doing this I learned a few things about how remotes work.
The fast forward merge required:

The GIT_DEPTH: 0 to ensure that GL CICD would fetch all commits for the the ref spec, which is basically the staging branch that we are on.
git remote add upstream<EMAIL_ADDRESS>is sufficient
Using git checkout origin/master results in a detached head, as in a local branch isn't created.
Using git merge --ff-only origin/staging will only do fast forward merge if possible, the history must be compatible
Finally git push upstream HEAD:master, the HEAD:master is what allows one to be in the detached head to proceed. The detached HEAD is where the fast forward merge occurred.

In the case of gh pr create, the usage of the --body "" option meant that we weren't relying on the command to figure out what to write in the body. If we used --fill instead, additional requirements are needed.
git remote add upstream ...
git fetch upstream staging master
git checkout -B staging upstream/staging
gh pr create ...

Basically the --fill option requires the source and target branches of the upstream remote to be fetched, AND you need to be on a local branch branch tracking the source upstream branch.
This is alot of requirements just to compute the relevant commit messages, so I'll leave this for posterity but won't implement it.

Furthermore it does make sense to change the default branch to staging.
It is very easy to make a mistake and get new feature branches from master.

As a precaution, master now does have a limitation being:

That is linear history is required. This should prevent any one clicking on the github PR to merge as that would produce a merge commit.


Proofs supports for the mobile

This code is already implemented in the mobile. Will need to revisit for proofs.
Here are changes on Finalize page.
https://github.com/mwcproject/mwc-qt-wallet/issues/548
Here are changes on send page.
https://github.com/mwcproject/mwc-qt-wallet/issues/549

@bayk This was sorted in https://github.com/mwcproject/mwc-qt-wallet/pull/669
Please check it.

There are some UI changes,  I will demo you next week

This was already done and merged into master.

Testing:
Please check for proofs, the mobile app show other party address before finalize the transaction for http (send confirmation step) and files (before finalize step).

Closing this ticket.


es2015 build not working

Angular 8 allows for differential loading - production build outputs 2 bundles:

for browsers supporting only es5
for browser supporting es2015

You can enable it by setting target: "es2015" in tsconfig of the storefront app.
Unfortunately, when running the es2015 version, there displays only the top loader and blank page:

There seems to be a problem with ngrx + angular + es2015. Here is the PR of angular-cli that claims to fix it:
https://github.com/angular/angular-cli/pull/ 14585
The PR claims to fix all those issues:
https://github.com/ngrx/platform/issues/ 1888 (effects not dispatched with build --prod)
https://github.com/ng-packagr/ng-packagr/issues/ 1307 (ES2015 problem since Angular 8 )
https://github.com/angular/angular-cli/issues/ 14613#issuecomment-498960867 (App that has lazy loading modules is not rendering any pages in prod mode after update to angular 8. )


Removing many habitat sup tests

These take way too long and severely impact our ability to test
Retaining a basic smoke test.

Runs in 6-8 minutes instead of 38 minutes.


spec v0.5 ch 4 - end feedback

By chapter
4.5

Typo "details" -> "detailed", and "constrains" -> "constraints"

4.7

I think you probably want a dmcontrol.acksecfault bit, to clear the any/allsecfault bits for selected harts.  Otherwise an S-mode debugger can't ever clear them, since it can't ever execute a successful resethaltreq, reset, or keepalive.

4.8

Register 1 caption should probably just say "new bits in dmstatus" or the like, since it has more than just the *secured bits
Table 8: resize column widths

I think you probably want a dmcontrol.acksecfault bit, to clear the any/allsecfault bits for selected harts. Otherwise an S-mode debugger can't ever clear them, since it can't ever execute a successful resethaltreq, reset, or keepalive.

I am wondering whether there is use case to ack the fault. Because if it is not possible to issue successful operation, it will be fine to always indicate error.  And the error just represent the result of the last operation, I don't think there is ambiguity even if it is not cleared.

fixes applied in #38, closing


add rotates

Did you want the assert_instr tests as well?
had to make a few fixes to get it building

Did you want the assert_instr tests as well?

We can add those later, now that rust has been upgraded to LLVM7, they might work out better.


[Bug]: The title has multiple commit titles instead of one

Opencommit Version
2.4.1
Node Version
v18.17.0
NPM Version
9.6.7
What OS are you seeing the problem on?
Mac
What happened?
The title has multiple commit titles instead of one.
There should also be a an empty line between the title and description, otherwise it looks unprofessional. Also see #239.
Expected Behavior
Opencommit should produce one commit title and description when more than one file is changed.
Should have an empty line between the title and description.
Current Behavior
📝 chore(create-analytics.sh): add script to create analytics docker compose files for each domain
📝 chore(deploy-nuke.sh): add command to stop and remove analytics containers and volumes
📝 chore(deploy.sh): add command to start analytics containers for each domain
📝 feat(docker-compose.analytics.template.yml): add template for analytics docker compose file
📝 chore(docker-compose.backup.yml): update version to "3.8"
📝 feat(docker-compose.test.yml): add docker compose file for testing Traefik setup with custom domain

The "create-analytics.sh" script is added to automate the creation of analytics docker compose files for each domain. The "deploy-nuke.sh" script is updated to include commands to stop and remove analytics containers and volumes. The "deploy.sh" script is updated to include commands to start analytics containers for each domain. Docker compose files are add

Possible Solution
Example of a great Gitmoji commit.
🔧 Optimize Docker configurations and tweaked robots.txt
This commit introduces a set of improvements aimed at fine-tuning our Docker Compose configurations and the WordPress robots.txt file.

The deploy.sh file has been updated to restart the Ofelia service after all other services are up, addressing the limitation of Ofelia's inability to detect new jobs in real-time.

In our Docker Compose files, we have replaced the usage of hardcoded hostnames with the Docker's built-in host-gateway, providing a more dynamic and universal solution.

The rules and labels for Traefik have been adjusted for clarity and efficiency. For instance, the Traefik rule for the phpMyAdmin service has been simplified by assigning it a unique subdomain rather than using a path prefix.

The robots.txt file for WordPress has been tweaked by commenting out the /feed/ disallow rule. This change was made to permit web crawlers to access the site's feed, which may improve its online visibility.

These changes should enhance the overall performance, flexibility, and search engine optimization of our web services.
Steps to Reproduce
Edit multiple files and run oco
Relevant log output
No response

this should be fixed if we improve the GPT prompt with a better example of a great commit (as you shared)
anyone wants to play with the prompt? I may have time next (or next-next) weekend


trying to deploy pip workspace using ubuntu agent  / Mac getting error

Invoke-WebRequest: {"requestId":"7f4ddf80-5570-4657-8220-a45faa8ecff9","errorCode":"MissingDefinitionParts","message":"Neither 'model.bim' nor anything with 'definition/*.tmdl' were found."}
$workspaceName = ""
$pbipPath = "./prj-pbip"
Import-Module "./modules/FabricPS-PBIP" -Force
Set-FabricAuthToken -servicePrincipalId  -servicePrincipalSecret  -tenantId   -reset
$workspaceId = New-FabricWorkspace  -name $workspaceName -skipErrorIfExists
Import-FabricItems -workspaceId $workspaceId -path $pbipPath

https://github.com/EvaluationContext/Analysis-Services/tree/JDuddy/256
Part Name is the issue, need to strip the ""

still getting same error
Invoke-WebRequest: {"requestId":"7b95fe2c-5e6e-4460-b3ca-feaabb2050da","errorCode":"MissingDefinitionParts","message":"Neither 'model.bim' nor anything with 'definition/*.tmdl' were
found."}

You can see the part names like part: /diagramLayout.json. You need to
update TrimStart() to trim both / and \ to deal the both Windows and Linux
On Thu, 13 Jun 2024, 17:55 adsouza75, @.***> wrote:

here is the full error System.Collections.Hashtable
System.Collections.Hashtable System.Collections.Hashtable
part: /definition.pbism
part: /diagramLayout.json
part: /model.bim
Creating a new item
Invoke-WebRequest:
{"requestId":"ab69693b-0cc9-418a-b577-2d7d1771bd25","errorCode":"MissingDefinitionParts","message":"Neither
'model.bim' nor anything with 'definition/*.tmdl' were
found."}
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Analysis-Services/issues/259#issuecomment-2166252200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHN3O7UP3WSZ3NAGD3KMEODZHHFI5AVCNFSM6AAAAABIHU37OKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWGI2TEMRQGA
.
You are receiving this because you commented.Message ID:
@.***>

that done , it import first time successfully , when ran it second time got this error
part: diagramLayout.json
part: model.bim
Item 'Reimbursement Account Insights' of type 'SemanticModel' already exists.
Updating item definition
Waiting for request to complete. Sleeping...
Waiting for request to complete. Sleeping...
Exception: LRO API Error: 'Dataset_Import_FailedToImportDataset' - An error occurred while processing the operation

I in the script I updated $filePath.Replace($itemPathAbs, "").TrimStart("/").Replace("", "/")

looks like correct issue with conflict , as dataset already exist , not sure if there is option to  manage conflict here

Should be able to update. There are different APIs for new items and
updating items. Might be worth looking at the update section of the script
for similar problems
On Thu, 13 Jun 2024, 18:48 adsouza75, @.***> wrote:

looks like correct issue with conflict , as dataset already exist , not
sure if there is option to manage conflict here
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Analysis-Services/issues/259#issuecomment-2166433924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHN3O7RFXEH44DXCW3QBQLTZHHLOBAVCNFSM6AAAAABIHU37OKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWGQZTGOJSGQ
.
You are receiving this because you commented.Message ID:
@.***>

Confirmed windows and Linux, create new fabric items and update item
definitions work for me when deploying this this script when testing
Semantic model and Report
On Thu, 13 Jun 2024, 18:52 Jake Duddy, @.***> wrote:

Should be able to update. There are different APIs for new items and
updating items. Might be worth looking at the update section of the script
for similar problems
On Thu, 13 Jun 2024, 18:48 adsouza75, @.***> wrote:

looks like correct issue with conflict , as dataset already exist , not
sure if there is option to manage conflict here
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Analysis-Services/issues/259#issuecomment-2166433924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHN3O7RFXEH44DXCW3QBQLTZHHLOBAVCNFSM6AAAAABIHU37OKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWGQZTGOJSGQ
.
You are receiving this because you commented.Message ID:
@.***>

IS there any specific I need to do not sure why it not working for me

The only way i got it working in ubuntu (linux) is to replace all ocrurence of .TrimStart("\") with .TrimStart("\").TrimStart("/")


[GLUTEN-7977][VL]fix: Add cstdint include explicitly

What changes were proposed in this pull request?
Add cstdint include explicitly for avoiding compile error with gcc 13.2 on Ubunut 24.04
(Fixes: #7977)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Manual test passed
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@yabinma, thanks for your pr! Please fix code style issue reported by clang-format.

The Velox build error should not be caused by the code change here.  I merged the latest code base and let's run it again. Thanks.