Incompatible Biome Server: ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

ℹ Rage discovered this running server using an incompatible version of Biome.

Server:
  Version:                      <=10.0.0

What happened?

user1 starts VS Code with remote connection to Ubuntu Server 22.04.3 LTS
user2 starts VS Code with remote connection to Ubuntu Server 22.04.3 LTS
user2 receives permissions error relating to connecting to the the existing LSP

[cli-stderr] data 216
[cli-stderr] end
[cli-stdout] end
[cli] exit 1
[Error - 11:56:39 AM] Biome client: couldn't create connection to server.
Error: Command "/home/user2/.vscode-server/extensions/biomejs.biome-1.0.1-linux-x64/server/biome __print_socket" exited with code 1
Output:
internalError/io ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  × Errors occurred while executing I/O operations.

    Caused by:
      Permission denied (os error 13)

	at getSocket (/home/user2/.vscode-server/extensions/biomejs.biome-1.0.1-linux-x64/out/main.js:19599:11)
	at processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async createMessageTransports (/home/user2/.vscode-server/extensions/biomejs.biome-1.0.1-linux-x64/out/main.js:19614:16)
	at async LanguageClient2.createConnection (/home/user2/.vscode-server/extensions/biomejs.biome-1.0.1-linux-x64/out/main.js:15576:28)
	at async LanguageClient2.start (/home/user2/.vscode-server/extensions/biomejs.biome-1.0.1-linux-x64/out/main.js:15152:30)
	at async activate (/home/user2/.vscode-server/extensions/biomejs.biome-1.0.1-linux-x64/out/main.js:19406:3)
	at async p.n (/home/user2/.vscode-server/bin/6c3e3dba23e8fadc360aed75ce363ba185c49794/out/vs/workbench/api/node/extensionHostProcess.js:103:6206)
	at async p.m (/home/user2/.vscode-server/bin/6c3e3dba23e8fadc360aed75ce363ba185c49794/out/vs/workbench/api/node/extensionHostProcess.js:103:6169)
	at async p.l (/home/user2/.vscode-server/bin/6c3e3dba23e8fadc360aed75ce363ba185c49794/out/vs/workbench/api/node/extensionHostProcess.js:103:5626)
[cli-stdout] close
[cli] close 1
[cli-stderr] close

Expected result
LSP would be started under a different socket/port and connected normally.
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

Forgive me if there is a setting or something I am missing that would properly handle this, but for speed & simplicity purposes another user & myself use an Ubuntu 22.04.3 server as our dev server which appears to be causing issues.
After converting from eslint/prettier I got everything working aside from quirkiness caused by yarn pnp. I can work around that and just keep a binary somewhere consistent, but when the other user logged on to install the biome extension and try out the new tools his errored and doesn't connect.
Based on the error I was guessing this was a socket conflict issue with both LSP's fighting over the same connection.
Let me know if there is anything else that could assist with this issue.

This is a use case that we haven't encountered so far, so this may be a valid issue.

Based on the error I was guessing this was a socket conflict issue with both LSP's fighting over the same connection.

Yes, that's likely the issue. We don't actually check if there's already a connection active. That would require some change, but I ma not an expert, so I can't provide a quick fix at the moment.

The fix will be available in the next release


Aberrations

Added JWSTAberratedPrimary class.
(I accidentally pushed a draft version to dev earlier today so this can directly replace that.)

wait no don't do it yet I need to do more testing but also its 5pm Friday so please bear with me

These commits were merged through a different branch.


Minimum Sellable Quantity

Issue Reference
#9967
Description
Add logic to manage the minimum sellable quantity for a product
How To Test This?
Set a minimum sellable quantity on a product greater than 1 and in the product page and cart page you cannot add quantity less than the value you set in the admin.
Branch Selection

[ ] Target Branch: 2.2

We will come to this pull request in the end, we need to check the flow as well as testing. Marking this PR on hold.


Trigger API

Following this comment.
This is not an issue, but I was just curious about this use case, and how it could potentially be solved with farfetched.
For example, we have page with 5 widgets, each one fetches data, so, we have 5 data requests. All data available only for authenticated user. User logs into application, doing their things, then went to have lunch. After an hour they come back and wants to check data on those widgets. Clicks on a navigation link, page is opened, 5 requests are sent, but OH NO, authentication session has been expired, and all 5 requests have received error 401.
There was a requirement in one of my previous projects, so this is not some fancy user case out of nowhere. And the requirement was following:

All 5 widgets continue to show "loading state"
Sign-in form pops up, like modal window in overlay
When user types their username and password and hits enter, sign-in form closes
And all 5 widgets shows actual data

So, we need to postpone all failed requests somewhere in a internal queue, without showing errors to user, and without notifying widgets, that their requests has received responses (so they will continue to show "loading state" like still waiting for response). Then, after session successfully renewed — take all postponed failed requests out of the internal queue and retry them all. Upon receiving successful  responses widgets will show data. Like nothing happen. From each widget's point of view it was just a loooooong answer for the request. Widgets have no idea, that requests were actually failed and retried.
I'm pretty sure this could be implemented differently, but this is like I did this, literally. It was long ago, I was young, and it was angular.js though :) Maybe you will suggest different approach to implement this kind of requirement.
It could be other cases, like with tokens. For example:
Request receives error 401 in response. This means access token has been expired. We took refresh token, and requests new access token. Upon receiving new access token — retry failed request. All automated and without letting UI to even know there was some trouble.

I've read your case one more time and released that it would be covered by Trigger API. I do not close this issue for now because I do not have clear vision of Trigger API, only general idea of executing Queries after some application-wide events.
Thanks, one more time 💙

related to #182

OTP Confirmation
I have a dynamic config for requests which needs cover by extra auth (MFA)
How it works:

Request sent
Got specific HTTP error (eg. 'OTP_REQUIRED')
Need to show prompt with OTP field to the user
User types code
Request retries with a new header

Note:

OTPGuard can be on every request. And I don't want to check it manually every time
I don`t know which route has this guard. Completely dynamic, and can be changed in runtime (on the backend side)


How to trigger the command palette to appear?

Hi! I just came across this gem and found it wonderful!
I installed your gem on a new rails project using Tailwindcss & Meilisearch. How do you display the command palette? What are the key bindings to toggle it?

I completely forgot to document that. It does not come with any default keybindings, but instead uses the built in keyboard event filter from Stimulus.
I've got it setup like this in our app:
<body class="min-h-full flex bg-zinc-100"
    data-controller="command-palette keyboard-navigation ahoy"
    data-action="keydown.meta+k->command-palette#toggle keydown.ctrl+k->command-palette#toggle keydown.esc->command-palette#hide keydown.down->keyboard-navigation#down keydown.up->keyboard-navigation#up keydown.enter->keyboard-navigation#select">
  ...
</body>

I updated the documentation with some more details about this.


feat(core): allow

Also fix gitattributes :smile:

that's going to let us easily use react-vis in codepen, etc.


Demonstrate usage of getInitialProps in _app.js.

A smol boi demonstrating usage of getInitialProps on an _app.js component with next-urql. This is to clear up some questions on #35.
While al urql queries will get picked up and run by react-ssr-prepass on the initial server render, there is a trick if you have blocking data requirements in a Page component that implements getInitialProps and is wrapped by _app.js. Basically, to ensure that Page's getInitialProps is ran, you have to call NextApp.getInitialProps: https://nextjs.org/docs/advanced-features/custom-app
We do have this in our second example, with the Home component having its own getInitialProps method. In order to have this function run, we need to add in this call to NextApp.getInitialProps.
cc/ @jgoux does this help? Your urql queries should still run fine in v0.3.0 if you're using useQuery or the Query component. This only applies if you're implementing getInitialProps in your Page components.

@parkerziegler I think I wasn't clear sorry. I'm not using getInitialProps at all. Not in my _app.js and not in my pages. I'm just wrapping my _app with withUrqlClient and that's it. But since 0.3.0 and without touching my code, SSR stopped working.

@jgoux thanks! If that's the case then we may have a bug. I can't repro in our local examples, granted those are quite trivial compared to a production app. If you can provide a CodeSandbox that minimally reproduces the issue that'd be super helpful!


Remove condensedinlinepanel integration/recommendation

At the time, adding support for condensedinlinepanel made sense, because it made the UI much cleaner. However, most of the issues raised these days are related to that apps limited support for certain field types, and the app doesn't seem to be maintained any longer, so I don't see these issues being fixed any time soon.

Also, when using condensedinlinepanel the checkbox for Allow sub-menu for this item: does not appear to work, although under water, it does.


Adding cross colo related metrics for the router

Three new metrics for the router:

crossColoRequestCount
crossColoSuccessCount
slippedPutAttemptCount (slippedPutSuccessCount exists already).

Reviewer: Ming, Estimate: 10 min.

LGTM, merging.


[DOCS] OS

Thanks, fixed. Will leave the enums as they're defined now and it doesn't hurt, as discussed. :)

Cherry picked into 3.0.3


M04-LAB10: DevOps Starter cannot be created

Contact Details
<EMAIL_ADDRESS>What happened?
Exercise : 01
Task: 01
Step: 09
Description of issue:
After filling up the form with the described values, the DevOps Starter is not created and no error message is displayed. There are also some limitations in place:
1 - A message is displayed: DevOps starter is being deprecated and will get retired on March 31, 2023.
2 - By default a database server is going to be created in Australia region, but this location is not allowed by the Azure Pass - Sponsorship subscription
Repro steps:

Do all the steps using an Azure Pass - Sponsorship subscription. This step number 9 is not going to do anything.
I tried to change the location to another region like "Central US" - it does not help.

Lab
Lab 10 Creating a release dashboard
Relevant screenshots

Do you want to help us? 👏

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

I tried again after some hours and it was finally built. That must be a kind of a problem on Azure side, as the attached web page was just open in the browser a few hours. Before closing it I tried to click once again the "Create" button and this time it worked. I suggest to close the issue

lab will be replaced, as tool is getting deprecated


Calculate Total Approved Amount correctly

Updated view-claim.js so that totalApproved function is called when page is loaded
Updated view-claim template to append 'approved-amount' class onto claim expense amounts to allow totalApproved function to work correctly on page load

Resolves https://github.com/ministryofjustice/apvs-asynchronous-worker/issues/123

Being dealt with by PR #209: This one has issue of when approved displaying double the amount for the approved cost, and deductions but being handled in other PR


Pretty output for debugging purposes

How about adding a feature for pretty output?

This should be a separate utility in the new mjson-ext package. This way, the implementation doesn't have the "lightness" constraint and can include all sorts of options for indentation, aligning elements on new lines, opening/closing brackets and new lines.


VB-330 Additional visit support

What does this pull request do?
Replaces the visit free text Reasonable Adjustments column with a list of available Support.
What is the intent behind these changes?
To all the UI to record multiple support requirements (Pre-defined & free text) against a visit.

Hi Aled,
Yeah I think now you're onboard it would be a good time for a clean up.
I've given both projects a light refactor for dead code and config but I've
left most of the code as is doing bits and pieces as I work on them.  I
have a list of things which work but are not ideal, maybe for next sprint
we could fix some of the important ones.
Phil
On Fri, 18 Mar 2022 at 09:57, AledWEvans @.***> wrote:

@.**** commented on this pull request.
In
src/main/kotlin/uk/gov/justice/digital/hmpps/visitscheduler/jpa/VisitSupport.kt
https://github.com/ministryofjustice/visit-scheduler/pull/57#discussion_r829850843
:

@@ -0,0 +1,50 @@
+package uk.gov.justice.digital.hmpps.visitscheduler.jpa

+import org.hibernate.Hibernate
+import java.io.Serializable
+import javax.persistence.Column
+import javax.persistence.Embeddable
+import javax.persistence.Entity
+import javax.persistence.Id
+import javax.persistence.JoinColumn
+import javax.persistence.ManyToOne
+import javax.persistence.Table
+
@.***
+data class VisitSupportPk(
I see what you mean, my comment was more about how they were using the JPA
annotations, what they have done is a composite key but a obscure way of
doing it that breaks some OOP rules when could be done using the composite
annotation . I agree we should use use a simple indexing approach with
primary keys and other keys especially when the data set is so small on
this table!
—
Reply to this email directly, view it on GitHub
https://github.com/ministryofjustice/visit-scheduler/pull/57#discussion_r829850843,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWX6SU7ZGTJODOGPLMEIRX3VARHR7ANCNFSM5Q6TPWBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***
com>

awesome, as long as we don't break anything :-)


Update BlockBlobClient.uploadStream to use NodeJS.ReadableStream instead of Readable

Package Name: storage-blob
Package Version: 12
Operating system: mac
[x] nodejs

version: 14

[ ] browser

name/version:

[x] typescript

version: 4

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Due to this method using Readable type rather than NodeJS.ReadableStream type I cannot pass a standard Node.js readable stream. With this change, users can still pass Readable types to the method since it implements NodeJS.ReadableStream.
I previously tried to fix this with https://github.com/Azure/azure-sdk-for-js/pull/11483 but didn't anticipate other methods require the Readable type. Maybe i misunderstand the difference between it and NodeJS.ReadableStream. In addition, maybe I need to do something in my own code.
In my specific instance I'm using Node.js and Busboy which defines its file type as NodeJS.ReadableStream (https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/busboy/index.d.ts#L41)

Did a bit more digging seems like I've been confused by new vs old streams using new Readable().wrap(oldStream) i was able to get everything working correctly and type safe. https://nodejs.org/api/stream.html#stream_readable_wrap_stream
I'll leave this open for others to comment. Maybe we can add something like this to the docs or examples?

Hey, @Ethan-Arrowood
Thanks for reaching us and good to know you have found the right approach. Yes, according to Readable's definition, using wrap is the right approach. This is a valuable topic related to old vs new stream, and might help others as well.
Thanks,
Jiachen

@jiacfan What are the next steps here? Are we looking at updating docs or something else?

Yes, we'll optimize doc.

Close the issue as it's a node.js usage related pattern and welcome feedback if anyone hit similar isse.


Adjust structure of developer guide

Make it clearer when to use each set of instructions.

Same here. I haven't verified for correctness of commands yet.


Safe interface of PyArray

Goal
Add PyArray on numpy crate with basic accessor

[x] Array Creation

[x] new
[x] zeros
[x] arange

[x] Accessor

[x] ndim/dims/shape
[x] data (as_slice{,_mut})

[x] Print elements in array in Rust (for debug)

misc todos

[x] Resolve build missing on Travis CI (fixed by #8 )
[x] Split numpy-test crate into anther PR #7

I really appreciate your work and want to thank you.
I tried to expose PyArray to Python using rust-cpython, just to test it, here is my code:
extern crate numpy;
extern crate numpy_sys;
#[macro_use] extern crate cpython;

use numpy_sys::*;

use numpy::{PyArray};
use cpython::{PyResult, Python};

py_module_initializer!(librust2py, initlibrust2py, PyInit_librust2py, |py, m| {
    try!(m.add(py, "__doc__", "This module is implemented in Rust."));
    try!(m.add(py, "sum_as_string", py_fn!(py, sum_as_string_py(a: i64, b:i64))));
    try!(m.add(py, "get_arr", py_fn!(py, get_arr_py())));
    Ok(())
});

fn sum_as_string_py(_: Python, a:i64, b:i64) -> PyResult<String> {
    Ok(format!("{}", a + b).to_string())
}

fn get_arr_py(py: Python) -> PyResult<PyArray> {
    let arr = PyArray::zeros(py,
                             &[3, 5],
                             ::numpy::NPY_TYPES::NPY_DOUBLE,
                             ::numpy::NPY_ORDER::NPY_CORDER);

    assert!(arr.ndim() == 2);
    assert!(arr.dims() == [3, 5]);
    assert!(arr.strides() == [5 * 8, 8]);

    Ok(arr)
}

Value is received by Python but accessing it causes a segfault although I'm not sure if it was not caused by my mistake in any way.
from target.debug import librust2py
# prints 3 (OK)
print(librust2py.sum_as_string(1, 2))
# segfault
print(librust2py.get_arr())

[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
3

Program received signal SIGSEGV, Segmentation fault.
__memmove_ssse3_back () at ../sysdeps/x86_64/multiarch/memcpy-ssse3-back.S:1574
1574	../sysdeps/x86_64/multiarch/memcpy-ssse3-back.S: No such file or directory.
(gdb) backtrace
#0  __memmove_ssse3_back () at ../sysdeps/x86_64/multiarch/memcpy-ssse3-back.S:1574
#1  0x00007ffff62d399d in memmove (__len=18446744073709551615, __src=0x0, __dest=<optimized out>) at /usr/include/x86_64-linux-gnu/bits/string3.h:59
#2  dump_data (string=string@entry=0x7fffffffd6e0, n=n@entry=0x7fffffffd6e8, max_n=max_n@entry=0x7fffffffd6f0, data=<optimized out>, nd=nd@entry=0,
    dimensions=dimensions@entry=0xaefc40, strides=0xaefc50, self=0x7ffff7e6a9e0) at numpy/core/src/multiarray/arrayobject.c:488
#3  0x00007ffff62d3acb in dump_data (string=string@entry=0x7fffffffd6e0, n=n@entry=0x7fffffffd6e8, max_n=max_n@entry=0x7fffffffd6f0,
    data=0xaab2d0 "", nd=nd@entry=1, dimensions=dimensions@entry=0xaefc38, strides=0xaefc48, self=0x7ffff7e6a9e0)
    at numpy/core/src/multiarray/arrayobject.c:495
#4  0x00007ffff62d3acb in dump_data (string=string@entry=0x7fffffffd6e0, n=n@entry=0x7fffffffd6e8, max_n=max_n@entry=0x7fffffffd6f0,
    data=0xaab2d0 "", nd=<optimized out>, dimensions=0xaefc30, strides=0xaefc40, self=0x7ffff7e6a9e0) at numpy/core/src/multiarray/arrayobject.c:495
#5  0x00007ffff62d3ba4 in array_repr_builtin (self=0x7ffff7e6a9e0, repr=0) at numpy/core/src/multiarray/arrayobject.c:595
#6  0x000000000057751a in PyObject_Str ()
#7  0x00000000005a4cc4 in PyFile_WriteObject ()
#8  0x00000000005c5e93 in ?? ()
#9  0x00000000004e9bc7 in PyCFunction_Call ()
#10 0x0000000000524414 in PyEval_EvalFrameEx ()
#11 0x000000000052d2e3 in ?? ()
#12 0x000000000052dfdf in PyEval_EvalCode ()
#13 0x00000000005fd2c2 in ?? ()
#14 0x00000000005ff76a in PyRun_FileExFlags ()
#15 0x00000000005ff95c in PyRun_SimpleFileExFlags ()
#16 0x000000000063e7d6 in Py_Main ()
#17 0x00000000004cfe41 in main ()

I would appreciate any information that could help me resolve this issue, thank you.

Thanks comment, and testing!
I reproduce your error on my machine while SSSE becomes AVX.
If I omit print, this error disappears. So, this error occurs to print the array at array_repr_builtin in numpy/core/src/multiarray/arrayobject.c:595.
I guess that I have to bind the repr function to for my PyArray, but I am still gathering information.

print(len(librust2py.get_arr())) does indeed print 3 but
for i in arr:
    print(i)

causes the same segfault

for _ in arr:
    pass

does not cause error. So the problem is printing elements.

Confirmed,
for i in arr:
    print(type(i))

causes
Program received signal SIGSEGV, Segmentation fault.
0x00000000005774de in PyObject_Str ()
(gdb) backtrace
#0  0x00000000005774de in PyObject_Str ()
#1  0x00000000005a4cc4 in PyFile_WriteObject ()
#2  0x00000000005c5e93 in ?? ()
#3  0x00000000004e9bc7 in PyCFunction_Call ()
#4  0x0000000000524414 in PyEval_EvalFrameEx ()
#5  0x000000000052d2e3 in ?? ()
#6  0x000000000052dfdf in PyEval_EvalCode ()
#7  0x00000000005fd2c2 in ?? ()
#8  0x00000000005ff76a in PyRun_FileExFlags ()
#9  0x00000000005ff95c in PyRun_SimpleFileExFlags ()
#10 0x000000000063e7d6 in Py_Main ()
#11 0x00000000004cfe41 in main ()

but
arr = librust2py.get_arr()
arr2 = numpy.zeros(1)

for i in arr:
    print(i * arr2)

prints
[ 0.  0.  0.  0.  0.]
[ 0.  0.  0.  0.  0.]
[ 0.  0.  0.  0.  0.]

arr.dtype causes segmentation fault while other operations, e.g. arr.max, numpy.sum(arr) works.
It must be related to the PyArray_Descr pointer in npffi::PyArrayObject.

On IPython
>>> dt = librust2py.get_dtype()
>>> dt.shape
AttributeError: 'numpy.dtype' object has no attribute 'shape'
>>> dt. # press [TAB]
            dt.alignment       dt.char            dt.flags           dt.isbuiltin       dt.kind            dt.names           dt.num             dt.subdtype
            dt.base            dt.descr           dt.hasobject       dt.isnative        dt.metadata        dt.ndim            dt.shape           dt.type
            dt.byteorder       dt.fields          dt.isalignedstruct dt.itemsize        dt.name            dt.newbyteorder    dt.str
>>> dt.shape
()

It seems that the initialization is incomplete :<
We can find information about initialization at How to extend NumPy or Boost.NumPy (a wrapper of NumPy C API in C++)
But this PR is already large, and the initialization will be postponed into the next PR.

There are two dynamics properties

dimensions of array
type of elements in array

These should be static in Rust, while be dynamic in Python.
I plan to use rust-ndarray for typed array interface in Rust. So, the PyArray implemented on this PR will be used to implement DataOwned in rust-ndarray in the future PR.

Exposing rust-ndarray as numpy array in Python (and with zero copy) would be a killer feature.

Good suggestion :+1:
You may be aware that the above segmentation fault bug has been fixed on current master and v0.1.0.
I created an example directory based on your testing code.
https://github.com/termoshtt/rust-numpy/tree/master/example
Thanks 😄

I am watching this repo as number one now on github, thank you :+1:


Artifacts and Modules

download

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

Please create 1 PR per artifact.

For engineering efficiency we require one artifact per PR. Please resubmit accordingly.


Why cancel discovering for internet/smart reader ?

Description
In the Javascript plugin you are able to launch a discovery twice and you don't need to cancel the discovery for Smart reader.
In stripe-terminal-react-native you can't do this because you will be receiving  when calling discoverReaders : {"code": "Busy", "message": "Could not execute discoverReaders because the SDK is busy with another command: discoverReaders."}}
Since we will be receiving only one update from onUpdateDiscoveredReaders, why should we be forced to call cancelDiscovering first if we want to trigger a new search for smart readers ?
Additional context
I've integrated both bluetooth and internet readers in my app, starting by the bluetooth one. And it seems to me, that contrary to the bluetooth reader, the smart one is not really still discovering/searching after receiving the first onUpdateDiscoveredReaders listener update. I'm trying to face some UI problems because when you are discovering with the smart reader and receiving the reply we still display kind of loading style since we are supposed to be in discovering. But to me it is not true.

Hi @melarts, thanks for the inquiry. The RN SDK is a bit of a departure from our JS SDK in that it simply acts as a bridge to our underlying mobile SDKs which do follow this divergent discovery pattern. The reasoning is it attempts to use the same pattern and rails as the discover operation for out bluetooth readers which do continually poll and look for new readers and must be cancelled. We're aware of this integration papercut and have tracked it internally in case we do try to alleviate this in a future breaking SDK upgrade. Thanks for the input!.


No such file or directory build\checkin.proto

Hi,
I'm using electron-push-receiver version 2.0.0.
When I use ipcRenderer.send(START_NOTIFICATION_SERVICE, senderId); in my renderer process, there is an error in my console window says that; PUSH_RECEIVER:::Error while starting the service { Error: ENOENT: no such file or directory, open '<my path>\build\checkin.proto'
What wrong have I done ?
Thank you,

Are you using a bundler like webpack ?

Yes, I'm using web 4.12

@robmontesinos Are u able to solve this ?

Anyone able to solve this

Having the same issue when I tried to use grpc in election with vue & vuetify.
I have tested all of the "webpack" related grpc loaders on the npm and none of them worked, perhaps I've made some mistake which I've not aware....either it shows ".proto file not found", or "Invalid package electon.asar" finally.
I believe the root cause is related to the proto file loading when webpack compiled the project, but even I configured the "CopyPlugin" in my vue.config.js, I still have problem to locate the proto file...
I worked 3 whole days and finally found this article which seems very close to my issue, really appreciate if any one could figure out how to config the CopyPlugin and load it after pack.

I tried this and worked for me
`const CopyWebpackPlugin = require("copy-webpack-plugin");
module.exports = {
/**

This is the main entry point for your application, it's the first file
that runs in the main process.
*/
entry: "./src/index.js",
// Put your normal webpack config below here
module: {
rules: require("./webpack.rules")
},
plugins: [new CopyWebpackPlugin([{ from: "src/grpc/proto", to: "proto" }])],
resolve: {
extensions: [".ts", ".js", ".json"]
}
};`

Having the same issue when I tried to use grpc in election with vue & vuetify.
I have tested all of the "webpack" related grpc loaders on the npm and none of them worked, perhaps I've made some mistake which I've not aware....either it shows ".proto file not found", or "Invalid package electon.asar" finally.
I believe the root cause is related to the proto file loading when webpack compiled the project, but even I configured the "CopyPlugin" in my vue.config.js, I still have problem to locate the proto file...
I worked 3 whole days and finally found this article which seems very close to my issue, really appreciate if any one could figure out how to config the CopyPlugin and load it after pack.

I tried this and worked for me @wanguolin
const CopyWebpackPlugin = require("copy-webpack-plugin");
module.exports = {
  /**
   * This is the main entry point for your application, it's the first file
   * that runs in the main process.
   */
  entry: "./src/index.js",
  // Put your normal webpack config below here
  module: {
    rules: require("./webpack.rules")
  },
  plugins: [new CopyWebpackPlugin([{ from: "src/grpc/proto", to: "proto" }])],
  resolve: {
    extensions: [".ts", ".js", ".json"]
  }
};

while using it I just use it as
import * as grpc from "@grpc/grpc-js";
import * as protoLoader from "@grpc/proto-loader";

const PROTO_PATH = `${__dirname}/proto/core.proto`;
const packageDefinition = protoLoader.loadSync(PROTO_PATH, {
  keepCase: true,
  longs: String,
  enums: String,
  defaults: true,
  oneofs: true
});

const { core } = grpc.loadPackageDefinition(packageDefinition);

export default core;

Thanks for your answer, I put the configuration like yours in rendering process, is shows: "Uncaught Error: Invalid package ...\resources\electron.asar".
Now I've worked around this issue by using 'webpack-proto-loader' in main process with proto files in vue's 'public folder'.

A little update since copy-webpack-plugin has changed:
plugins: [
    new CopyWebpackPlugin({
      patterns: [
        {
         // CHANGE NODE MODULES PATH TO WHERE ELECTRON IS INSTALLED. FOR MONOREPOS IT'S USUALLY ../../node_modules
          from: "./node_modules/**/*.proto",
          to: "[name].[ext]",
        }
      ]
    })
  ],

Keep in mind that

I have used copy-webpack-plugin, still facing the same issue. Below is the code from my webpack file.

For someone struggling when integrating this awesome package with your electron project build with vue-cli-plugin-electron-builder, you need to use custom webpack config build of main process, not renderer process, this is vue-cli-plugin-electron-builder docs about that. You need to use chainWebpackMainProcess
Here is my configuration at vue.config.js:
`
const { defineConfig } = require("@vue/cli-service");
const NodePolyfillPlugin = require("node-polyfill-webpack-plugin");
const CopyWebpackPlugin = require("copy-webpack-plugin");
module.exports = defineConfig({
transpileDependencies: true,
pluginOptions: {
electronBuilder: {
preload: "src/preload.ts",
builderOptions: {
appId: "...",
publish: [
{
provider: "generic",
url: "http://<IP_ADDRESS>:3005/",
},
],
mac: {
category: "public.app-category.business",
target: ["zip", "dmg"],
},
nsis: {
oneClick: true,
perMachine: false,
runAfterFinish: true,
allowElevation: true,
deleteAppDataOnUninstall: true,
menuCategory: true,
},
win: {
target: [
{
target: "nsis",
arch: ["x64"],
},
],
publisherName: "...",
legalTrademarks: "...",
verifyUpdateCodeSignature: false,
},
},
/**
* work around to fix this issue: https://github.com/nklayman/vue-cli-plugin-electron-builder/issues/1647#issuecomment-1019400838
* some resources is defined by url in css file can't be loaded on production build (urls start with app:///)
* docs of package related to this issue: https://nklayman.github.io/vue-cli-plugin-electron-builder/guide/configuration.html#changing-the-file-loading-protocol
* */
customFileProtocol: "./",
externals: ["chokidar"],
chainWebpackMainProcess: (config) => {
// Chain webpack config for electron main process only
config.plugin("copy").use(CopyWebpackPlugin, [
{
patterns: [
{
from: "node_modules/push-receiver/src/mcs.proto",
to: "mcs.proto",
},
{
from: "node_modules/push-receiver/src/gcm/android_checkin.proto",
to: "android_checkin.proto",
},
{
from: "node_modules/push-receiver/src/gcm/checkin.proto",
to: "checkin.proto",
},
],
},
]);
},
},
},
css: {
loaderOptions: {
scss: {
additionalData: @import "@/assets/css/variables.scss";,
},
},
},
configureWebpack: {
// Webpack's configuration applied to web builds and the electron renderer process
plugins: [
// This config was added after installing Trusting social's SDK, Web SDK
new NodePolyfillPlugin(),
],
},
});
`


The app drawer can't be opened under specific conditions.

To reproduce the issue, set OpenLauncher as the default home. Next, click on OpenLauncher in the app drawer to open what I think is a second instance of the app. When in the second instance of the app, if home is pressed the first instance is opened and the app drawer icon doesn't work. The second instance of the app will also show up in the recent app list now and when swiped away will also cause the app drawer in the first instance to break. As far as I can tell, the issue only shows up when OpenLauncher is set as the default home.

OK I will look into the problem

Starting a second instance of a Launcher that is already running doesn't seem like a good idea anyways. In my opinion the icon in the app drawer should open the settings if the launcher is already running.
Or the icon should not even show up in the drawer, only a shortcut to the settings, like for example with Nova Launcher.

Right, that is a good idea

I have encounter the problem. and will fix this

Fixed


test: break up unit tests to support platforms with limited RAM

Summary
Broke up unit tests into smaller parts to better support
devices with limited RAM.  This allows for the creation of
smaller unit test firmware images.

Checklist

[ ] Documentation (if necessary)
[ ] Tests (if necessary)
[ ] Ready to be merged

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

A contributor has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.
Thank you for your contribution!

Your pull request has been merged and marked as tier 4. It will earn you $20 USD.


A question about Python one-click installation script.

He compiles and installs on Azure's VM and is stuck in test_socket, unable to proceed to the next step. Is there a good solution? Or if you have a telegram, I will try to contact you to solve it.

Hello
You can contact me via @ThyCrow
Plus I have not worked with Azure or Google Cloud or AWS and I cannot confirm that the script works on them.


Trying to create a player's inventory throws a NPE

When I use this code to try and create and display a player's inventory, it throws a NPE. Log : http://pastebin.com/EWa9vhui

I might be able to fix the problem when building the inventory.
But displaying will not be possible as mentioned in the comment on the Archetype.
Try using InventoryArchetypes.CHEST instead.


[Feature] The test process takes too long.

What's the feature?
When I inference the test data, the merge process and evaluation process take too long. For example, when I use Rotated Retinanet with Res50 as backbone to test, it will cost half an hour. Please give me some advises.
Any other context?
No response

Hi, I have set the nproc by 24, but it seems like still only use 1 process to evaluate the iou, and it takes a lot of time to test. Could like tell me how you fix this? Thank you very much!

The evaluation process stuck in the IoU computation stage which could get worse when there are dense predictions and instances. We have already organized the codes in a multi-processing manner hence increase the args nproc could alleviate such situation.

Hi, I have set the nproc by 24, but it seems like still only use 1 process to evaluate the iou, and it takes a lot of time to test. Could you like tell me how can fix this? Thank you very much!

The evaluation process stuck in the IoU computation stage which could get worse when there are dense predictions and instances. We have already organized the codes in a multi-processing manner hence increase the args nproc could alleviate such situation.

Hi, I have set the nproc by 24, but it seems like still only use 1 process to evaluate the iou, and it takes a lot of time to test. Could you like tell me how can fix this? Thank you very much!

Hi, I set the iou_thrs by np.array([0.5]), and it does save a lot of time!

Hi, I have changed some code to solve this problem. Hope this can be helpful.#10

Hi, I have changed some code to solve this problem. Hope this can be helpful.
shaunyuan22/SODA-mmrotate#7


feat: Display file message from bot

수천만원에 엑싯하는 사이드 프로젝트가 한둘이 아니다 (feat. 비즈니스 마켓플레이스) (2).webm
Note: empty message above is expected (empty user message was sent before file message)
File viewer will be added in the future (currently no design so I left it as comment instead)
수천만원에 엑싯하는 사이드 프로젝트가 한둘이 아니다 (feat. 비즈니스 마켓플레이스).webm

From the video you attached, I just saw this is being rendering in the image viewer message. Please make sure this is not being rendered before you merge :)


Create some introductory E2E tests

Summary
We should go ahead and add some simple E2E tests that establish how we create them moving forward. As a first pass, it likely makes sense to make a few tests that guarantee the flags being passed are working as intended.

Checkout https://github.com/exdx/coreutils/tree/main/head/tests for an example of an e2e structure for testing a CLI command


Not query gRPC server for offline commands in wallet

Description
Some commands in the wallet, such as "get label," do not need to connect to any server.
We should avoid querying the gRPC server in these commands to improve security and performance.

@themantre we aren't query gRPC servers for these commands right now too.

So, you mean we can close this issue?

Yes, as I can see we get them from valut without any network connection. I tried to find any network connection on those functions but all of them come from local data.

for example to create a new address we don't need to connect to any server. To test it; change the server to invalid addresses (You can change all ports to non-existing one) then try to create a new address.

@themantre OK, let me test this case!

@b00f @themantre
Here is my test result:
Create command:
 ./pactus-wallet create --path=./wallet
✔ Password: |
Confirm password:

Your wallet was successfully created at: /home/kay/code/pactus/build/wallet
Seed phrase: "***"
Please keep your seed in a safe place; if you lose it, you will not be able to restore your wallet.

New Address command:
 ./pactus-wallet address new --path=./wallet
✔ Label: |

pc1zhpfnxwwxsm8m2j82ry3dkcequgxv8gdxxm2ww5

All Address command:
./pactus-wallet address all --path=./wallet

1- pc1zhpfnxwwxsm8m2j82ry3dkcequgxv8gdxxm2ww5

And my wallet servers.json in this build was this:
{
    "mainnet": [
        "localhost:50051"
    ],
    "testnet": [
        "localhost:50052"
    ]
}

I have no local node running for testnet or mainnet. and there was no error. I think we can close this issue.


Autoupdate issue

Hi,
First let me thank you for your work on this docker container. It rocks.
I just noticed that whenever an update is pushed on steam the server is not automatically updated.
To have it updating the server I need to remove lastUpdate file in /conanexiles and restart the container.
Anyone else having this issue ?
Thanks !

@kijdam i am facing this problem too. I don't know why but the lastUpdate check is not working. Maybe anybody has an idea what we could check here. I tried to check the 'lastUpdate' value from public branch via steamcmd and also the 'buildid', but none of the them worked so far.
When you want to trigger the update you only need to clear the file lastUpdate, because the controllerscript is checking for a change of the lastUpdate (= buildid) and a new buildid, if they are not the same, an update will be triggered.
So only do something like: echo > /var/lib/docker/volumes//_data/lastUpdate or echo > /lastUpdate. After this the Update will be triggered without restarting the container.

@alinmear
Thanks for your answer. Okay so I'm quite "happy" you have the issue too.
Thanks, I'll take a bit of time to read your script and try to find a way that the check works automatically.
I don't know what you think about it but I would also add a little piece of code to cp -a Saved dir to Saved.timestamp before the update, because I had some surprises the db being wiped/crashed after an update (including the xxxx_backup.db!)
Sounds good to add this ?

@kijdam you can find the update logic here conanexiles_controller.sh. If you find a good Solution maybe can you make a pull request for this?
Regarding the backup logic before an Update: Sounds good to me. I will open a new issue for this and will implement it.

New approach merge with #5. Let's see, hopefuly it works now.

Seems to work. Thanks alot. Closing this issue ...


Favicon helper

This pull request attempts to fix issue #400.

@joneslee85 I thought it was strange that my rebase didn't have more merge conflicts then it did. Looks like the other person touched different parts of the same file. I have removed the duplicate code and travis should now pass.

@michaeldeol yes, I have merged #408 before you, can you please make another PR with just integration test?


Wrong operator

Hi there,
Could you check this code in RedirectToLogin.razor.
You are checking IsAuthenticated. but when it return true your method redirect to "access-denied" page or am i wrong ?
          if (user.Identity?.IsAuthenticated ?? false)         {             _navigationManager.NavigateTo("/access-denied");             return;         }

Hi,
If the user is already logged in - it is unauthorized to login again.
Hence the redirect.
Thanks!


bond.dll public key token changed

Hi, I occasionally upgraded Bond.Runtime.CSharp from 3.0.7 to latest 7.0.1, I found that Bond.dll public key token changed, it makes assembly binding redirect doesn't work. Any workaround except I have to stay on 3.0.7 (bond.dll v<IP_ADDRESS>) ? Why public key got changed? Thanks.

@koote: I haven't seen an e-mail with details, so I'm going to close this issue for now.


Set the serializer with polymorphic association

Hi,
I would like to know if if is possible to force the serializer with polymorphic relation?
The problem is I used namespace.
I have a 3 models:

Product::Product
Receipt::Receipt
Receipt::Ingredient

each one has it's own serializer:

Product::ProductSerializer
Receipt::ReceiptSerializer
Receipt::IngredientSerializer

A ``Receipt::Ingredienthave anitemwhich can be either aProduct::ProductorReceipt::Receipt`.
The code of my model Receipt::Ingredient:
class Receipt::Ingredient < ApplicationRecord
  belongs_to :item, polymorphic: true
end

The code of my serializer Receipt::IngredientSerializer:
module Receipt
  class IngredientSerializer
    include FastJsonapi::ObjectSerializer

    belongs_to :item, polymorphic: true

  end
end

When I try to serialize I have an exception NameError (uninitialized constant Receipt::ProductSerializer).

NameError (uninitialized constant Receipt::ProductSerializer)
If that's not a typo then this is your problem. You don't have a Receipt::ProductSerializer. You have a Product::ProductSerializer. To be honest using namespaces that have the same names a classes is gonna cause you problems and you're probably not correctly abstracting things with names like that.
That said, you should be able to pass the serializer option to belongs_to e.g.:
belongs_to :item, polymorphic: true, serializer: Product::ProductSerializer

In that case it's possibly an issue with fast_jsonapi. As a workaround you may be able to do something like ProductSerializer = Product::ProductSerializer inside the IngredientSerializer.

Thanks, that's what I did for now but that's a big hack.
I expect more something like an option serializer for polymorphic association like we can define the record_type.

I'm having a similar issue! I'm using ActiveStorage and want to serialize ActiveStorage::Attachment objects with my AttachmentSerializer which has no prefixed modules.
In my NewsPostSerializer, has_many works! (It helps that my association name gives away the needed serializer and there are no modules)
has_many :attachments, polymorphic: true, if: Proc.new { |post| post.attachments.attached? }

But in my DlsClass::GroupSerializer, has_one is a pain. The following will make "type": "one" because ActiveStorage wraps it in an Attached::One object with delegation.
has_one :orientation_pdf, polymorphic: true, if: Proc.new { |group| group.orientation_pdf.attached? }

I solved this by applying a block and directly returning the attachment
has_one :orientation_pdf, polymorphic: true, if: Proc.new { |group| group.orientation_pdf.attached? } do |group|
  group.orientation_pdf.attachment
end

but because the group serializer is in a module DlsClass::GroupSerializer, it is trying to find a serializer DlsClass::AttachmentSerializer and even though in the end I tried explicitly stating the serializer option, this thread makes me think your project is ignoring it because it is a polymorphic association.
has_one :orientation_pdf,
        serializer: AttachmentSerializer,
        polymorphic: true,
        if: Proc.new { |group| group.orientation_pdf.attached? } do |group|
    group.orientation_pdf.attachment
end

So this module guessing for a polymorphic association is a major problem.

Well, in fact because this association is not polymorphic I was wrong. I fixed everything:
class DlsClass::GroupSerializer
  include FastJsonapi::ObjectSerializer

  has_one :orientation_pdf,
          record_type: :attachment,
          serializer: AttachmentSerializer,
          if: Proc.new { |group| group.orientation_pdf.attached? } do |group|
    group.orientation_pdf.attachment
  end
end

I guess if you were on the ActiveStorage::Attachment side and wanted to serialize the polymorphic record (record_type, record_id, name) then this would be applicable if modules were involved.


[Feature] Impersonate other accounts for admins

What do you think about https://github.com/404labfr/laravel-impersonate ?
Found via: https://murze.be/2017/02/laravel-package-impersonate-users/

Looks nice, but needs integration into our permissions system so we can assign who can impersonate & so on, not like the default README says through method override. We can use it as a base, and build upon it..

Not a priority, plus third-party package could be used, no need to be re-implemented!


[BUG] The size of CSS file imports are being limited

What happened?
The framework is limiting the size of CSS file imports into Content Script UI files. For example, looking inside the

I didn't see this issue, we are also using tailwindcss.

Getting the same error here, the fonts I import (using data-base64) seems to be way to large (they are variable fonts), so all tailwind classes are not correctly imported in the style tag.


GCC 12.2 changes

Hi,
I tried to make your code run using GCC 12.2 (RISCV target).
It seems that the GCC people changed the internal gcov_info struct - between stamp and filename there is a new additional checksum field.
See here: https://github.com/gcc-mirror/gcc/blob/4d1e4ce986f84a853531ac9c51e37d101e28d15c/libgcc/libgcov.h#L232
After adding the checksum field, the lib worked also using the newer GCC.

Besides having an additional checksum field in the gcov_info struct, the .gcda files also have this checksum added to their header, see this line:
https://github.com/gcc-mirror/gcc/blob/4d1e4ce986f84a853531ac9c51e37d101e28d15c/libgcc/libgcov-driver.c#LL519C50-L519C50

With the modifications pointed out by @drgrandios I managed to make a cross-target GCOV work with GCC14. Given that GCC14 adds actual MC/DC coverage out of the box, this is a high-value ticket.

@drgrandios You are the boss!


chore: bdd-spring-1574929544 to 0.0.1

chore: Promote bdd-spring-1574929544 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


increase min electron version

I believe async calls require node version 7.6 or higher.  electron 1.8.2 bumped node version to include node ^7.6 (see https://github.com/electron/electron/commit/0330a30fdb405d99cddaf2b0529f9ea02086d949#diff-bcf7bcf05ac59e51b9adfd7394c810c4)
When I tried installing electron-push-receiver and running I was getting unexpected token errors on async calls.

That's weird because I have no issue running it with electron 1.8.1

In the link you shared, you can see that node v7.9+ is available in electron since 1.7.3

No issue on my side with Electron 1.7.6

I was having general instability issues (i.e. windows crashing) with electron 1.8.1 so I've reverted back to electron 1.6.15 (latest stable vs. beta) for my project.  I'll withdraw this pull request, but maybe consider requiring something more stable than 1.8 line unless that's really required?

@CydeSwype Well it seems that latest 1.7 is enough


go 1.21.5

coverage: 78.648% (-0.04%) from 78.687%
when pulling d41836a133b30d85b7786b9b4b66a93c52ba12f8 on dissoupov
into 4e0b00f6fd3aa7fc0c588d9ff0691a657ceeafeb on main.


Android and Debian clients no longer work: Certificate issue

Describe the bug
Any attempt to connect to the Keybase.io API yields:
 ERROR API network error: doRetry failed, attempts: 2, timeout 8m27s, last err: Post "https://api-1.core.keybaseapi.com/_/api/1.0/logdump/send.json": tls: failed to verify certificate: x509: certificate signed by unknown authority
Version on Linux is
6.4.0-20240821175720.3212f60cc5
Additional numbers
The number 345567

This seems magically to have been fixed overnight, without a new client?


[ISSUE #1037]Optimize DefaultMQPushConsumer

Which Issue(s) This PR Fixes(Closes)

Fixes #1037
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


[vcpkg-tool] Support hook script for binary caching

Is your feature request related to a problem? Please describe.
vcpkg currently offers a variety of binary caching solutions, but they do not meet all possibilities. If the protocol (eg. sftp, samba, ftp, etc...) of the storage backend is not supported by vcpkg, then sharing binary caching will be a problem.
Proposed solution
Add upload/download hook script support, and then we can set hook script by env variable.
Suppose we set the following environment variables:
VCPKG_BINARY_SOURCES="clear;hook,/path/to/hook.sh,readwrite"

Then we use the following script to implement WebDAV as the storage backend:
I know that vcpkg already supports HTTP as a binary source, and I use this example just to illustrate my assumption

hook.sh

#!/usr/bin/env bash
set -e

# triplet="xxx"
# name="xxx"
# version="xxx"
# sha="xxx"
# package="/path/to/package.zip"
# action="upload" # or download
#####
# The above variables are provided by vcpkg as environment variables
#####

if [[ "${action}" == "download" ]]; then
    curl -o "${package}" "http://www.xxx.com/{triplet}/{name}/{version}/{sha}.zip"
elif [[ "${action}" == "upload" ]]; then
    curl -T "${package}" "http://www.xxx.com/{triplet}/{name}/{version}/{sha}.zip"
else
    false
fi

I hope vcpkg will consider this and provide a solution that will help small and medium-sized teams build their own private package management center.
At the same time, this has the added benefit, we can strip the existing binary caching upload and download function from the vcpkg-tool and controlling it by an external independent process. This allows us to easily extend the storage backend protocol.
Describe alternatives you've considered
No response
Additional context
No response

What do you propose to report errors and how to report the resulting file name back to vcpkg?

report errors
Maybe simply check exit code, if not 0, then show the stderr.

report the resulting file
I think this is no need, vcpkg should tell the script where to find or save the file.

Maybe simply check exit code, if not 0, then show the stderr.

So if the exit code is not 0 there was an error in the script or the binary is not available?

I think this is no need, vcpkg should tell the script where to find or save the file.

Also possible. For some binary caches like the file binary cache that would result in unnecessarily copies but since there is already a file binary cache source that is probably not an problem.
Yeah I think passing things on the command line is the best. But I would prefer to only provide the sha so that things like the following works:
https://github.com/microsoft/vcpkg/discussions/19091

No stale


Add 3.0 announcement

Final update for the 3.0 announcement on our docs.

coverage: 94.237%. remained the same when pulling ff4d34bb0e26231c234a1ae99e67627de3ad84e0 on feature/3.0-announcement into f2ee19094559f3dc119bb3403452243007031ef3 on main.


Added embedded and missing models for mistrals

The code works.
However, I had to set the type argument of "client" in both mistralEmbedded and mistralModel to Any.
The root cause is that the import statement import('@mistralai/mistralai') must be asynchronous. So, placing this statement directly at the top level of the module isn't possible. As a consequence, I was forced to use the type Any to make typescript happy

Embedding -> You set a test and return the vector
Models -> You get back a response given a prompt.
But yes, Unit tests across plugins would be great


Confirmation should be true by default.

while using --no-interactive mode, the whole process won't start, because the confirm method returns false by default.

@Metrique can this PR be merged, as it's blocking change for deployment automatization.
Thanks

Thanks! Apologies for the delay, I was not getting notifications at all 👎


Write-Error contains bearer access token and is written in the console log

Using EntraExporter and not having sufficient rights, it give an error. This  error is not handled by the module and passed through the console log and we can clearly see the bearer access token (stripped from this capture)

Any way to hive this part of the request? I don't mind having the error but having the access token all over in the logs....even if it's valid only for 1 hour and having limited access, I prefer not having it :)
Thanks !

The skiptoken field wouldn't be an access token, it's just used for pagination in the Graph API to handle large datasets. If you wanted to redact that or something from the output you could always wrap the command in a try/catch and use regex to redact any output that you don't want to see.

See below, I mask the authorization header bearer... its weird on dark mode browser 😅
Plus, I run this inside an azure function so maybe its more verbose than usually.


Add logger integration test

Adding regression test for CLI logging.
This is the last thing to do to close #199

Related: #228

Why would you bump munit to M1?
On Mon 9. Oct 2023 at 08:09, Paweł Prażak @.***> wrote:

@.**** commented on this pull request.
In integration-tests/project.scala
https://github.com/VirtusLab/besom/pull/227#discussion_r1349887303:

-//> using lib "org.scalameta::munit::1.0.0-M1"
+//> using toolkit latest
@lbialy https://github.com/lbialy WDYT?
—
Reply to this email directly, view it on GitHub
https://github.com/VirtusLab/besom/pull/227#discussion_r1349887303, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBVNUSFXWWBQN6POTKPE4DX6OIKDAVCNFSM6AAAAAA5V3RXUWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNRTG43DMNBSG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


RunnoHost Add Clear Terminal Method

I've had a look through the documentation and from what I can see there doesn't seem to be a method to clear the terminal from RunnoHost. I think that this would be a useful method to have access to.

Yeah, that makes sense. For now you could do:
element.interactiveUnsafeCommand('clear', {})


MarGo build failed on go1.7.3

go env
go version go1.7.3 windows/amd64
| GoSublime error: MarGo build failed
|                > This is possibly a bug or miss-configuration of your environment.
|                > For more help, please file an issue with the following build output
|                > at: https://github.com/DisposaBoy/GoSublime/issues/new
|                > or alternatively, you may send an email to<EMAIL_ADDRESS>|                >
|                >
|                > cmd: `['D:\\tools\\go_amd64\\bin\\go.exe', 'build', '-tags', '', '-v', '-o', 'gosublime.margo_r16.07.09-1_go?.exe', 'disposa.blue/cmd/margo']`
|                > stdout: `
|                >
|                > `
|                > stderr: `
|                > net
|                > disposa.blue/margo
|                > disposa.blue/something-borrowed/types
|                > disposa.blue/margo/meth/importpaths
|                > net/textproto
|                > net/http/httptrace
|                > crypto/x509
|                > mime/multipart
|                > crypto/tls
|                > net/http
|                > net/http/pprof
|                > net/rpc
|                > disposa.blue/something-borrowed/gocode
|                > disposa.blue/cmd/margo_pkg
|                > disposa.blue/cmd/margo
|                > go install disposa.blue/cmd/margo: open gosublime.margo_r16.07.09-1_go?.exe: The filename, directory name, or volume label syntax is incorrect.
|                > `
|                > exception: `None`
|   install state: busy
| sublime.version: 3126
| sublime.channel: stable
|       about.ann: a16.07.09-1
|   about.version: r16.07.09-1
|         version: r16.07.09-1
|        platform: windows-x64
|            ~bin: D:\tools\sublime_text\Data\Packages\User\GoSublime\windows-x64\bin
|       margo.exe: ~bin\gosublime.margo_r16.07.09-1_go?.exe (missing)
|          go.exe: D:\tools\go_amd64\bin\go.exe (ok)
|      go.version: go?
|          GOROOT: D:\tools\go_amd64
|          GOPATH: D:\developing\go\mj;
|           GOBIN: (not set) (should usually be `(not set)`)
|       set.shell: []
|       env.shell:
|       shell.cmd: ['C:\\Windows\\system32\\cmd.exe', '/C', '${CMD}']

| GoSublime error: MarGo build failed
|                > This is possibly a bug or miss-configuration of your environment.
|                > For more help, please file an issue with the following build output
|                > at: https://github.com/DisposaBoy/GoSublime/issues/new
|                > or alternatively, you may send an email to<EMAIL_ADDRESS>|                >
|                >
|                > cmd: `['/usr/local/go/bin/go', 'build', '-tags', '', '-v', '-o', 'gosublime.margo_r16.07.09-1_go1.7.3.exe', 'disposa.blue/cmd/margo']`
|                > stdout: `
|                >
|                > `
|                > stderr: `
|                > runtime
|                > # runtime
|                > /usr/local/go/src/runtime/lfstack_amd64.go:16: lfstackPack redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/lfstack_64bit.go:37
|                > /usr/local/go/src/runtime/lfstack_amd64.go:20: lfstackUnpack redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/lfstack_64bit.go:41
|                > /usr/local/go/src/runtime/os_linux.go:27: _FUTEX_WAIT redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:22
|                > /usr/local/go/src/runtime/os_linux.go:28: _FUTEX_WAKE redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:23
|                > /usr/local/go/src/runtime/os_linux.go:36: futexsleep redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:31
|                > /usr/local/go/src/runtime/os_linux.go:67: futexwakeup redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:62
|                > /usr/local/go/src/runtime/os_linux.go:83: getproccount redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:78
|                > /usr/local/go/src/runtime/os_linux.go:109: _CLONE_VM redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:104
|                > /usr/local/go/src/runtime/os_linux.go:110: _CLONE_FS redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:105
|                > /usr/local/go/src/runtime/os_linux.go:111: _CLONE_FILES redeclared in this block
|                > 	previous declaration at /usr/local/go/src/runtime/os1_linux.go:106
|                > /usr/local/go/src/runtime/os_linux.go:111: too many errors
|                > `
|                > exception: `None`
|   install state: busy
| sublime.version: 3126
| sublime.channel: stable
|       about.ann: a16.07.09-1
|   about.version: r16.07.09-1
|         version: r16.07.09-1
|        platform: linux-x64
|            ~bin: ~/.config/sublime-text-3/Packages/User/GoSublime/linux-x64/bin
|       margo.exe: ~bin/gosublime.margo_r16.07.09-1_go1.7.3.exe (missing)
|          go.exe: /usr/local/go/bin/go (ok)
|      go.version: go1.7.3
|          GOROOT: /usr/local/go
|          GOPATH: ~/projects/go
|           GOBIN: (not set) (should usually be `(not set)`)
|       set.shell: []
|       env.shell: /usr/bin/zsh
|       shell.cmd: ['/usr/bin/zsh', '-l', '-c', '${CMD}']

Same on my linux install:
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.1 LTS
Release:	16.04
Codename:	xenial

Each directory listed in GOPATH must have a prescribed structure:
The src directory holds source code.  The path below src
determines the import path or executable name.
The pkg directory holds installed package objects.
As in the Go tree, each target operating system and
architecture pair has its own subdirectory of pkg
(pkg/GOOS_GOARCH).


QueryBuilder::insert() doesn't preserve passed params when building a INSERT INTO … SELECT query

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#15194

Merged. Thank you!


Clear cached sample_result in tracing::Span::set_parent()

Motivation
Fixes https://github.com/tokio-rs/tracing-opentelemetry/issues/150

Solution
Clear cached OtelData.builder.sample_result in tracing::Span::set_parent()
so it is recomputed again after the parent changes.
TODO: needs a repro that we can turn into a test.

Closing in favor of #153, thanks for the effort!


Code snippets review #1: code files

Hey guys sorry it took me so long, here are my thoughts:

There is a difference in lengths in code snippets. Some have 20+ lines, and some 7-8 lines (and in some cases only 3 lines of code). I suggest making them more-or-less similar in length. For example 17+-3 lines of code (or 15+-3 is also good).
The variety of code 'types' and 'styles' is quite good :)

@ehungbu and @jyotsna-penumaka have edited the snippets


Add headings from H1 to H6 in a drop drown menu

[ ] h1

h1

[ ]  h2

h2

[ ]  h3

h3

[ ]  h4

h4

[ ]  h5

h5
Visit #22 for more details

Hello, working on this issue. Kindly assign.

Nyc work !


Encourage sites to configure /etc/ssh/ssh_known_hosts for login and submit hosts?

The LinuxHostAdapter and ssh wrapper features both include setting strict_host_checking: false for when the host is not in the user's ~/.ssh/known_hosts file. It seems that we should at least document, or even provide a helper script, to:

add the hosts OnDemand is configured to ssh to in the /etc/ssh/ssh_known_hosts file
some type of verficiation script to run to output the hosts that need to be added when cluster configs are modified to specify a new login or submit host that is not in /etc/ssh/ssh_known_hosts

I do not know if it is /etc/ssh/ssh_known_hosts is always the path, or if sometime it could be /etc/ssh/known_hosts

This all sounds extremely fragile.  I would recommend using these SSH flags:
-oStrictHostKeyChecking=no -oUserKnownHostsFile=/dev/null -oLogLevel=ERROR

We use these on clusters for both clustershell and pdsh, so that the only thing looked at for SSH authorization is /etc/ssh/ssh_known_hosts.  The default also is /etc/ssh/ssh_known_hosts but that value can be changed.
I would not recommend adding code to OnDemand to touch the global known hosts file. I think we just need to document the need keep that file up-to-date.  This is something a site's automation should be handling, NOT OnDemand.  Also for clusters it's usually common for sites to have a single host key for the entire cluster and use aliases to match all hosts...this is alias for Owens:
owens-rw01.ten.osc.edu,o0*,owens-rw*,owens-login*,owens.hpc.osc.edu,owens.osc.edu,10.2.*,!10.2.200.*,ib-o0*,ib-owens-rw*,ib-owens-login*,10.22.5?.*,10.22.6?.*

We would be imposing ourselves a huge burden to try and meet all use cases of SSH known hosts if we start touching the file.  I think we need to leave it to sites to handle this otherwise it will become something we will never stop updating to catch up to all the ways sites use SSH host keys.  So I think documentation is all we need to do.

Also it's really important to use UserKnownHostsFile=/dev/null because if a host key changes it will prevent entries not managed by automation from causing conflicts. It's best not to read user's known hosts file to keep things simple and also prevents the user's file from having new entries automatically added and thus relying only on the global known hosts file.

Is there any security concern for the Linux Host Adapter and the ssh wrapper for submitting jobs to both default to StrictHostKeyChecking=no? I had assumed that this would be a type of "insecure default" so should be opt-in i.e. manually specify strict_host_checking=false in the cluster config, with it defaulting to true.
Also it seems you are suggesting that even if strict_host_checking=true we would still do UserKnownHostsFile=/dev/null and expect the global known hosts file to be properly configured. Is that correct?

Yes, setting StrictHostKeyChecking=no is technically an insecure default so maybe we need to provide a means to allow sites to specify their own SSH flags with some sane defaults provided like UserKnownHostsFile=/dev/null.
Yes we need UserKnownHostsFile=/dev/null otherwise every time a user logs in with SSH for first time the host key will get added to ~/.ssh/known_hosts and also that file will be checked for matching host key.  If the host key is changed and the host's old key exists in ~/.ssh/known_hosts it will cause errors.
One thing I vaguely recall is if you set UserKnownHostsFile=/dev/null then every SSH login will prompt for yes/no to accept unless you set StrictHostKeyChecking=no so it might be the case that we need to document the "insecure" default and make sites aware of this.  If we are concerned about a potentially insecure default then we would need a way to allow sites to provide SSH flags and we would need to document the recommended "insecure" defaults.  FWIW I believe that anyone putting their hosts into LinuxHostAdapter will trust those hosts so StrictHostKeyChecking=no is not necessarily insecure as it's not likely a site is running OnDemand at SiteA and then setting up LHA to SSH to a external entity's site. I like to think that most security comes down to common sense and while at first glance StrictHostKeyChecking=no seems insecure, in practice the way it's used with LHA is not insecure.

Testing from my mac, if I do
ssh -oUserKnownHostsFile=/dev/null -oStrictHostKeyChecking=yes owens.osc.edu

or
ssh -oUserKnownHostsFile=/dev/null owens.osc.edu

After doing
mv .ssh/known_hosts .ssh/known_hosts.bak
sudo cp .ssh/known_hosts.bak /etc/ssh/known_hosts

So if the global /etc/ssh/known_hosts is configured, we can set UserKnownHostsFile=/dev/null and have strict host checking enabled as a secure default.

I'd recommend testing on Linux, the behavior maybe different.  If the entry is in /etc/ssh/ssh_known_hosts and you do /dev/null for the user file, I think you are correct that no need to add StrictHostKeyChecking...now that I think about it more I do recall that's the behavior I see when adding new hosts at OSC and once I add the host to bastion SSH known hosts the user /dev/null is fine and doesn't prompt yes/no.

The ssh man pages on Owens are the same as on my Mac in this regard so I’d be surprised if the behavior differed. But will check

Testing on Fedora Linux, the command ssh -oUserKnownHostsFile=/dev/null -oStrictHostKeyChecking=yes owens.osc.edu worked as intended


coverage: fix issue with skip and tarpaulin

Looking at https://github.com/xd009642/tarpaulin/issues/756 we could fix coverage generation with the following changes
In local the following commands work, so it should on CI too:
$ cargo install cargo-tarpaulin --version 0.18.0-alpha3
$ cargo tarpaulin -o Html

thanks, this should fix the build until the bitvec situation improves


Please explain WHY

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4cb129d2-9594-bf0c-5946-982f0ed0d888
Version Independent ID: 6522b038-6853-3552-55c8-f174cf56573d
Content: Enable per-user Multi-Factor Authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-userstates.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@mikew1927 Thank you for your query. We will review this request and update the thread.

@mikew1927 This thread seems duplicate of #56930 . I will be closing this thread and we will work on the other issue with detailed description. Please let us know in case otherwise.
Thank you .


multiple where conditions

Is it possible to make multiple where conditions?
var q = db.from('people').where('age', '=', 25);
i want to get people who are age 25 and who live in state = ny

Hello there,
thank you for this library! In my case I'm struggling with multiple where conditions with OR operator. Is there any way to apply two where conditions with an OR operator without having to do two queries? I'm trying to do the following query:
WHERE (timestart >= X OR timeend >= X) AND timestart <= Y
If I chain .where functions they always use AND operator.
Thanks!

Multiple equal-AND over multiple OR query is possible.
Multiple keyrange-AND is not possible. You will have to use on memory filter for one of the range query.

Hello @yathit,
I also want to combine multiple where clauses but https://dev.yathit.com/ydn-db/nosql-query.html seems to be unavailable.
Do I need to add an index for all my field names which are used in the where clauses?

Yes, all field in where clause mutual be indexed.
It was old url. All pages are in documation section on the site.
On Feb 1, 2016 6:51 AM, "Benny Neugebauer"<EMAIL_ADDRESS>wrote:

Hello @yathit https://github.com/yathit,
I also want to combine multiple where clauses but
https://dev.yathit.com/ydn-db/nosql-query.html seems to be unavailable.
Do I need to add an index for all my field names which are used in the
where clauses?
—
Reply to this email directly or view it on GitHub
https://github.com/yathit/ydn-db/issues/17#issuecomment-177646430.

Thanks for your fast reply.
I created indexes for name, age and name, age but I still get the error: db.from(...).where(...).where(...).list.
Do you see what I am doing wrong?
var schema = {
  stores: [
    {
      name: 'users',
      autoIncrement: true,
      indexes: [
        {
          name: 'name',
          keyPath: 'name'
        },
        {
          name: 'age',
          keyPath: 'age'
        },
        {
          name: 'name, age',
          keyPath: ['name', 'age']
        }
      ]
    }
  ]
};

var db = new ydn.db.Storage('my-db', schema, {mechanisms: ['indexeddb']});

db.clear();

var values = [
  {"name": "Lara", "age": 45, "registered": "2015-12-21T10:14:59.661Z"}
  , {"name": "Sarah", "age": 25, "registered": "2016-01-12T09:00:32.736Z"}
  , {"name": "Benny", "age": 28, "registered": "2015-12-21T09:55:17.201Z"}
  , {"name": "John", "age": 39, "registered": "2015-12-18T11:14:50.426Z"}
  , {"name": "Benny", "age": 30, "registered": "2014-12-18T11:14:50.426Z"}
];

for (var i = 0; i < values.length; i++) {
  db.put('users', values[i]);
}

var query =
  db
  .from('users')
  .where('name', '=', 'Benny')
  .where('age', '<', 30)
  .list()
  .done(function (records) {
    records.forEach(function (record) {
      console.log('Result', record);
    });
  });

Good news!
I solved the db.from(...).where(...).where(...).listissue by using a ydn.db custom build which has the cur module. But now my db query does not give me any result but it is expected to find 1 record. :cry:

That function is not finished yet. The multiple where clause should generate query as follow:
query = db.from('users').where('name, age', '>', ['Benny', 30']);

Thanks for your answer. I changed the following:
.where('name', '=', 'Benny')
.where('age', '<', 30)

->
.where('name, age', '<', ['Benny', 30])

These changes cause now this error: Uncaught ConstraintError: Require index "name, age" not found in store "users".
But I have there is an index name, age which is defined as:
{
  name: 'name, age',
  keyPath: ['name', 'age']
}

Also... Shouldn't there be two operators for the multiple where clause? Because I need = and <.

All look good. Please post to jsbin.

Thanks for helping me!
I made a Fiddle: https://jsfiddle.net/bennyn/ynchm1tw/
If you click on "Run" you can see the output in your Browser's JavaScript console.

Definitely a bug.
current use https://jsfiddle.net/ynchm1tw/2/

Thanks for helping out!
This works:
var iterator =
  ydn.db.IndexValueIterator
  .where('users', 'name, age', '>', ['Benny'], '<', ['Benny', 30]);

var query =
  db
  .values(iterator)
  .done(function (records) {
    records.forEach(function (record) {
      console.log('Result', record);
    });
  });

As a note, none of my application use query api and the bug fix will be slow.


members of @doNotStore tagged libraries are not reliably inheriting the annotation

In some tests internally, the following pattern is not reliably working:
@doNotStore
library strings;

import 'package:meta/meta.dart';

String get someString => 'string';

import 'strings.dart' as strings;

class C {
   String title(Object o) => strings.someString; //<= should show warning
}

What is the intended semantics of applying the annotation to an entire library? According to the documentation (which might need to be updated):

The annotation can also be applied to a class to implicitly annotate all of the valid members of the class, or applied to a library to annotate all of the valid members of the library, including classes.

That implies, to me, that applying the annotation to the library implicitly applies it to the class C, which implicitly applies it to the method C.title, so it seems like it ought to be valid to return the value of the getter.

applying the annotation to the library implicitly applies it to the class C

Sorry, I think my example wasn't clear.  C is in a different library from strings.  (I'll add some details.)

Unless I'm misunderstanding the desired semantics, I'd expect to see a warning on C.title above.

Me too, given that they're in different libraries.


TextOverflow not working as expected in Text widget

Steps to reproduce
Run the following app (also visible at http://dartpad.dev/0c6794bd5daf972aff91724efee85dd9):
import 'package:flutter/widgets.dart';

void main() {
  runApp(BugReport());
}

class BugReport extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Directionality(
      textDirection: TextDirection.ltr,
      child: Align(
        alignment: Alignment.center,
        child: SizedBox(
          width: 150,
          child: ColoredBox(
            color: const Color(0xff666666),
            child: Text('onelongword anotherreallylongword', maxLines: 1, overflow: TextOverflow.visible),
          ),
        ),
      ),
    );
  }
}

Expected behavior
You expect something like the following result:

Actual behavior
This is what's rendered -- the second word doesn't show up at all, despite the overflow being set to visible:

Related
If you change the overflow to TextOverflow.clip, then you expect the following output:

But again, the second word is completely omitted:

Other info
In the latter case, if clipping at word breaks is actually working as intended, then I think we should add a new TextOverflow enum value that allows the caller to clip in the middle of a word.  The first case above seems unlikely that it's working as intended.

Setting softWrap: false achieved the desired behavior.
This should be documented better.  Currently, the Text.overflow property only says "How visual overflow should be handled."  If the overflow behavior is affected by the softWrap property, the interplay should be documented (in the properties and also the Text constructor arguments), and there should be breadcrumbs from one to the other.

Does anyone start to deal with this? I wanna do it.

@tvolkert
I set the attribute sofWrap: false of Text class as you said and it worked with TextOverflow.visible.
But it shows no any text with 'TextOverflow.clip` property.
Is this expected work?

@broccolism here's what I used:
Text(
  'insert text here',
  maxLines: 1,  // Don't wrap at all
  softWrap: false,  // Don't wrap at soft breaks
  overflow: TextOverflow.clip,  // Clip the overflow
);

This can be seen at http://dartpad.dev/4afce9459e04be1bfc5524fad8eb35df


Add documentation

Still on progress.

:+1: nice

Well, it's just the beginning. A lot remains here.

@gkoz: I think I'm done here. Can you just take a quick look before I merge please ?

It's ready (I added ignore). I'll update cairotest too.

My head hurts after thinking about the GObject type system internals =)

No problem. ;-)
I saw the question about adding custom widget. Do you think we'll add something which will make it possible ? For me, it seems very unlikely. We didn't take this path from the beginning after all....

It could be an interesting challenge and maybe even Rust-C interoperability case study. But I haven't seen a strong enough need for this yet.
It's fair to want to have custom widget types. Take that Clock example. You have some internal fields and you want to access them from a signal handler. Currently widget signal handlers take Widget as a receiver. If they could take that custom type instead, they'd avoid some Rc dances. It could also be considered lunacy to make a sandwich of glib-allocated buffers that hold Rust data that is accessed by Rust code called from C callbacks. Too bad that absent stable Rust ABI we pretty much can't have a native toolkit.

Hum... It'd be interesting to give some feedback to rust core developers in order to make it possible if not yet in nightly.


Angular Directive for Visualsearch

Is there any directive available for visualsearch that i can use in my Angular project?

Take a look at: https://github.com/documentcloud/visualsearch/issues/101#issuecomment-75980661


[feature] Create subdomains on Cloudflare

For Linode provider we seem to support auto creation of subdomain if it does not exist (mentioned in the docs and implemented here), however for Cloudflare I currently get:
level=error msg=NXDOMAIN

Error goes away once I manually create the A record from Cloudflare > DNS UI, but I would like to manage everything as code.
Would be cool having the same behavior as for Linode for Cloudflare as well.
It would also be nice to have the non-existent subdomain in the error message so it saves one round of checking back in the code/configs which subdomain failed.

As side note, thanks for this amazing tool :heart:

Thanks for the suggestion, the providers are implemented by different contributors, so the behaviors may be different from each other.
I'll look into the API docs of Cloudflare and try to create the subdomains instead of throwing out the errors.

Thanks a lot for implementing so fast! Works from my side :sparkles:
However, it creates the record with proxying disabled (I see it's the default when calling the API), thus the IP gets exposed, and I am still required going to the Cloudflare UI to enable record by record. It would be awesome if we had the option to instruct "proxied":true


minor changes on readme file again

 All committers of the pull request should sign our Contributor License Agreement in order to get your pull request merged.

Coverage remained the same at 95.39% when pulling fae1fa5ab567d535dcaa86fe31077648abf72748 on pomidoro:master into 2bb5f8772d6cc8a75c6b1d980d174636382aa1e9 on cla-assistant:master.


Stream log output from composer commands

Summary

Leverage the packit scribe.LogEmitter writer to stream composer command logs in real-time.
The outcome will be logs that look like this:
[builder] Paketo Buildpack for Composer Install 1.2.3
[builder]   Executing build process
[builder]   Building new layer /layers/paketo-buildpacks_composer-install/composer-packages
[builder]   Running 'composer config autoloader-suffix PaketoDefaultAutoloaderSuffix'
[builder]   Running 'composer install --no-progress --no-dev'
[builder]       Installing dependencies from lock file
[builder]       Verifying lock file contents can be installed on current platform.
[builder]       Warning: The lock file is not up to date with the latest changes in composer.json. You may be getting outdated dependencies. It is recommended that you run `compo
ser update` or `composer update <package name>`.
[builder]       Package operations: 6 installs, 0 updates, 0 removals
[builder]         - Downloading symfony/polyfill-php80 (v1.25.0)
[builder]         - Downloading symfony/polyfill-mbstring (v1.25.0)
...
[builder]       Generating autoload files
[builder]       > php -r 'foreach (get_loaded_extensions() as $ext) { print("PostInstall [$ext]\n"); }'
[builder]       PostInstall [Core]
[builder]       PostInstall [date]
[builder]       PostInstall [Phar]
[builder]   Copying from /workspace/vendor => to /layers/paketo-buildpacks_composer-install/composer-packages/vendor
[builder]       Completed in 3.267s
[builder]
[builder]   Generating SBOM for /layers/paketo-buildpacks_composer-install/composer-packages
[builder]       Completed in 13ms

Use Cases

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

because we are using the check-platform-reqs log output to parse which extensions we found and create the "composer-extensions.ini", I wasn't able to add the live log streaming to that step. It will still print the logs afterwards with the list of extensions found.

If you wanted, you could use io.TeeReader to send the logs to a buffer as well as stdout (or the scribe.Logger) - that way you can get both.

@robdimsdale for some reason I wasn't able to successfully use io.TeeReader, I think I got messed up with the readers/writers.
Instead, I achieved similar behaviour by using io.MultiWriter. Let me know what you think, or if there's a straightforward way to use io.TeeReader, and if that's preferred over the multi writer.

Oh, I probably got them backwards. MultiWriter sounds entirely plausible and consistent with what I was thinking of. Glad it's working!


feat: add a button to the popup to open the report generator panel

The tool works as expected but should only apply to the hazards app.

I believe I was able to resolve this!


about sample process

Hello, thanks for your good work!
I notice that you use DDPM in the code, which is slower than DDIM. I am curious is it possible to use the DDIM sample and add self attention mask to the DDIM process? Thanks

Hi, apologies for the delayed response.
Yes, it's possible to use the DDIM sampler with a few straightforward changes. I'll be working on this, but it might take some time. Alternatively, you could use the diffuser's implementation, which supports Stable Diffusion and any scheduler.
Thanks for your interest!

As a quick update, you can now use SAG with DDIM sampler in this repo. For usage, please refer to the updated README!

Very nice work! Can SAG be used for EulerDiscreteScheduler? It seems that the implementation with diffusers still doesn't work properly with Euler sampler


Australia East Availability Zone not shown on map at https://azure.microsoft.com/en-us/global-infrastructure/regions/

Describe the bug
According to https://docs.microsoft.com/en-au/azure/availability-zones/az-region, the Australia East region has availability zones, but that is not shown on the map at https://azure.microsoft.com/en-us/global-infrastructure/regions/

The documentation mentions that Availability Zone is available in Au East but provisioning of VMs still do not work. Can you please provide an ETA of when it will be GA?

Now fixed on the new page at https://azure.microsoft.com/en-au/global-infrastructure/geographies/


Does it work to run MLAPI in two different projects?

Hi, I had been trying to use HLAPI to implement object sharing between a server and client which are in two different projects. Got to know that this is not possible in HLAPI. I was successful in using LLAPI to implement message exchanges between the server and the client in the two different projects. Was wondering if MLAPI would be possible to be run on two different projects for doing the object sharing? Thanks!!

As stated in the Readme:

Supports separate Unity projects crosstalking

Answer is yes. However, certain features have some requirements. SyncVars require that the variable names are the same. Scene management requires that the Scene's have the same names.

Sorry for missing that and thanks a lot for developing this! Will give it a try today.

No problem. If you have any issues. Let me know.