Fix artifact name

Proposed changes
Multiple job runs don't allow artifact uploads in new runner: https://github.com/actions/upload-artifact/issues/478
Used the GITHUB_RUN_ID variable to make artifacts unique
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] I have run make install-tools and have attached any dependency changes to this pull request
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5efc8c2) 66.00% compared to head (057f883) 66.14%.
Report is 5 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #558      +/-   ##
==========================================
+ Coverage   66.00%   66.14%   +0.14%
==========================================
  Files         118      118
  Lines       13468    13461       -7
==========================================
+ Hits         8889     8904      +15
+ Misses       3979     3956      -23
- Partials      600      601       +1

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

Does this mean that there was a name collision?


Nuxt does not register classes in superjson after building

Environment

Operating System: Windows_NT
Node Version:     v20.5.1
Nuxt Version:     3.7.2
CLI Version:      3.8.3
Nitro Version:    -
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      alias, css, build, nitro, vite, typescript, imports, runtimeConfig, modules, authJs, googleFonts, pinia, unocss
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-wn9aqn-sxviyo

Run npm run build
Run npm run start or node .output/server/index.mjs
Observe in Console tab that there is an error TRPCClientError: Unable to transform response from server.

Describe the bug
I want to use trpc with superjson for transforming json data and classes, but it seems like the registered classes work in development but after build, it only runs on one-side (the api endpoint returns the right json + metadata in the network request) but trpc fails to transform it on the server, most likely somehow not registering the classes.
linked discussion in trpc: https://github.com/trpc/trpc/issues/4790
Additional context
No response
Logs
No response

Hey I am running into this as well. Any luck figuring it out?

Would you see if you can reproduce this in pure Nitro (reproduction sandbox), and if so, raise there? 🙏

Im not quite sure how to do that but i'll see what i can do

Closing as I'm pretty sure this is a Nitro-related issue and not a Nuxt one. If it only recurs with Nuxt, let me know and I'll reopen it here.


[v3] Carousel autoplay doesn't work after switching pages with pageTransition

Environment

Operating System: Windows_NT
Node Version:     v18.20.4
Nuxt Version:     3.13.2
CLI Version:      3.15.0
Nitro Version:    2.10.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      default
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Version
v3.0.0-alpha.7
Reproduction

Clone the repository https://github.com/BasoAlter/nuxt-carousel-transition
Run the server, then on the main page has autoplay carousel and a link to other page
Click the other page link. At other page, it has a link to main page, then click it
The autoplay carousel will not work after that and no error log

Description
The autoplay still works when I don't use the pageTransition in nuxt.config.ts. But when I do use it, somehow it doesn't work again after changing the page.
Additional context
No response
Logs
No response

Have you tried, calling the play method yourself?


FSharpType.IsUnion broken

I have the following code which is broken through upgrading:
F#:
type Side =
  | Left
  | Right

C#:
FSharpType.IsUnion(typeof(Side), FSharpOption<BindingFlags>.None)
FSharp.Core 4.0.0: returns true
FSharp.Core 4.2.1: returns false

Closing in favour of https://github.com/Microsoft/visualfsharp/issues/3225


BLE: ESP32C3: legacy advertisement through extended API

I have started development on M5Stamp C3U - based on ESP32-C3. After starting beacon sample I was trying to get an extended advertisement functionality. It turns out that there is no support for extended advertisement. Even I have tried to send raw command to controller (BT_HCI_OP_LE_READ_NUM_ADV_SETS) with responce command disallowed. It looks like the controller blob do not have enabled that feature.
As the ESP32-C3 supports extended advertisement the functionality should be exposed to Zephyr BLE host.
Am I missing any config to enable that one or is there any ongoing work in that area?

Well, it should work. samples/bluetooth/periodic_adv uses extended advertising:
Starting Periodic Advertising Demo
[00:00:00.000,000] <inf> esp32_bt_adapter: esp_bt_controller_init: BT controller compile version [d913766]
[00:00:00.166,000] <inf> bt_hci_core: bt_dev_show_info: Identity: CD:96:80:55:E7:32 (random)
[00:00:00.166,000] <inf> bt_hci_core: bt_dev_show_info: HCI: version 5.0 (0x09) revision 0x0016, manufacturer 0x0060
[00:00:00.167,000] <inf> bt_hci_core: bt_dev_show_info: LMP: version 5.0 (0x09) subver 0x0016
Start Extended Advertising...done.

Does this sample code work as you need? Or you need something else?

Thank you for the response, I had some wrong configs, and the demo you have proposed works.
Have you tried to run legacy advertisements through an extended API? The goal is to run multiple legacy advertising sets independent of the main application.
I have observed that when in advertising parameter options are set as BT_LE_ADV_OPT_CONNECTABLE | BT_LE_ADV_OPT_SCANNABLE then I have got assert and I can not debug:
assert co_utils.c 728, param 000000ff 000000ff
The same situation is every time if there is no BT_LE_ADV_OPT_EXT_ADV option selected.

I'm able to duplicate this same assert, which is from the Espressif binary blob.
A trivial BLE example that does nothing but enable legacy advertising will trigger it, bt_le_adv_start(BT_LE_ADV_CONN_NAME, ad, ARRAY_SIZE(ad), NULL, 0);
If BT_EXT_ADV is disabled, then this works fine.  It Legacy advertises as it should.  But once BT_EXT_ADV is enabled, then the assert is triggered by the bt_le_adv_start() call.
OTOH, using extended advertising (only) does work, i.e. bt_le_ext_adv_create() + bt_le_ext_adv_start().
I'm not aware that enabling extended advertising support in the build config is suppose to make the legacy advertising API unavailable.  It's certainly isn't documented anywhere I can find.  And even legacy advertising should be unavailable, I'd expect the call to bt_le_adv_start() to return -ENOTSUP rather than assert.

This seems to work now. I'm using 2 advertising sets with legacy advertising on the current zephyr main branch. I haven't yet tried it with 3.6.*'s release though.

It does work in 3.7.


Wrapping Select method panics 'expected slice but got interface'

I am writing the following wrapper
func (c *Client) Select (ctx context.Context,dest interface{}, query string, args ...interface{}) error{
	defer c.trackQueryTime(query, args...).Tracef("Query")
	return c.db.SelectContext(ctx, &dest, query, args...)
}

its just a trace log for timing the queries and then passing all args to the inner sqlx Select function
here is the test ..
func TestSelect(t *testing.T) {
	initDataBase(t)
	ctx, client := initTestSqlClient(t)
	pp := []Person{}
	err := client.Select(ctx, &pp, "select Id, Name from Persons")
	assert.NoError(t, err)
}

the result of the test ..
INFO   [0004] initializing new connection pool for sql connStr:[server=localhost;user id=sa;password=password-1234;port=1433;database=Test1;log=1]  server=IL-ARTHURVA-VIMB
TRACE  [0004] Query                                         executionTime=0.0020014 params="[]" query="select Id, Name from Persons" server=IL-ARTHURVA-VIMB
    TestSelect: mssql_test.go:249:
        	Error Trace:	mssql_test.go:249
        	Error:      	Received unexpected error:
        	            	expected slice but got interface
        	Test:       	TestSelect
--- FAIL: TestSelect (4.29s)
FAIL

Process finished with exit code 1

Sorry .. found the issue ..
I was passing here
func (c *Client) Select (ctx context.Context,dest interface{}, query string, args ...interface{}) error{
	defer c.trackQueryTime(query, args...).Tracef("Query")
	return c.db.SelectContext(ctx, &dest, query, args...)
}

return c.db.SelectContext(ctx, &dest, query, args...) the &dest is a pointer to the interface ..
i just needed to pass the dest that pas received since interfaces are pointers themselves .. :\


How to pass audio file and transcribe it

Hi team, I am working on implementing a voice chatbot using RealtimeSTT library. For the speech-to-text part, I am using the RealTimeSTT library. Here, I am attempting to provide an audio file as input and transcribe it. You mentioned that if we don't want to use a microphone, we should set 'set_microphone' to False and provide the audio as a 16-bit PCM chunk to obtain the transcribed text as output.
i have implemented the code as below.
import soundfile as sf
import numpy as np
import json
from scipy.signal import resample
from RealtimeSTT import AudioToTextRecorder

def format_audio(filepath):
# Read the audio file
    data, samplerate = sf.read(filepath)
    print("data------------->",data)
    pcm_16 = np.maximum(-32768, np.minimum(32767, data*32768)).astype(np.int16)
    print("pcm data----------------->",pcm_16)
    # Create metadata
    metadata = {
        "sampleRate": samplerate,
    }
    metadata_json = json.dumps(metadata)
    metadata_bytes = metadata_json.encode('utf-8')

    # Create buffer for metadata length (4 bytes for 32-bit integer)
    metadata_length = len(metadata_bytes).to_bytes(4, byteorder='little')

    # Combine metadata length, metadata, and audio data into a single message
    combined_data = metadata_length + metadata_bytes + pcm_16.tobytes()
    # print("Combined data: ", combined_data)
    return combined_data

def decode_and_resample(
            audio_data,
            original_sample_rate,
            target_sample_rate):

        # Decode 16-bit PCM data to numpy array
        audio_np = np.frombuffer(audio_data, dtype=np.int16)

        # Calculate the number of samples after resampling
        num_original_samples = len(audio_np)
        num_target_samples = int(num_original_samples * target_sample_rate /
                                 original_sample_rate)

        # Resample the audio
        resampled_audio = resample(audio_np, num_target_samples)

        return resampled_audio.astype(np.int16).tobytes()

if __name__ == '__main__':
    combined_data = format_audio('chat1.wav')
    recorder_config = {
        'spinner': False,
        'use_microphone': False,
        'model': 'tiny.en',
        'language': 'en',
        'silero_sensitivity': 0.4,
        'webrtc_sensitivity': 2,
        'post_speech_silence_duration': 0.7,
        'min_length_of_recording': 0,
        'min_gap_between_recordings': 0,
        'enable_realtime_transcription': False,
        'realtime_processing_pause': 0,
        'realtime_model_type': 'tiny.en'
    }

    recorder = AudioToTextRecorder(**recorder_config)
    metadata_length = int.from_bytes(combined_data[:4], byteorder='little')
    metadata_json = combined_data[4:4+metadata_length].decode('utf-8')
    metadata = json.loads(metadata_json)
    sample_rate = metadata['sampleRate']
    chunk = combined_data[4+metadata_length:]
    resampled_chunk = decode_and_resample(chunk, sample_rate, 16000)
    recorder.feed_audio(resampled_chunk)

    # Get the transcribed text
    text = recorder.text()
    print(f"Transcribed text: {text}")

but i got response as;
[2024-04-28 11:36:10.671] [ctranslate2] [thread 21416] [warning] The compute type inferred from the saved model is float16, but the target device or backend do not support efficient float16 computation. The model weights have been automatically converted to use the float32 compute type instead.
data-------------> [0.00000000e+00 0.00000000e+00 0.00000000e+00 ... 9.15527344e-05
1.22070312e-04 1.52587891e-04]
pcm data-----------------> [0 0 0 ... 3 4 5]
[2024-04-28 11:36:57.949] [ctranslate2] [thread 17900] [warning] The compute type inferred from the saved model is float16, but the target device or backend do not support efficient float16 computation. The model weights have been automatically converted to use the float32 compute type instead.
RealTimeSTT: root - WARNING - Audio queue size exceeds latency limit. Current size: 104. Discarding old audio chunks.
But after this, I have to get the transcribed text as output, but it is running forever and didn't give any output after that warning message. I don't have any clue to identify what I'm doing wrong. According to my findings, for real-time transcription, it is working fine (when we speak, it continuously transcribes), but when we give an audio file, how do we transcribe it? It would be helpful if you provide a solution for passing the audio file and getting the text from it.

I'm facing a similar issue. I'm trying to figure out how to pass an audio file and receive the corresponding text. On the client side, several processes are being executed to prepare the Blob file, which is then sent to the socket. On the Python socket side, the remaining speech-to-text processing occurs. I'm wondering how to achieve the same result by passing the audio file as a parameter?

Please use the really great faster-whisper library to transcribe an audio file (this lib is just not the right tool for that).
Explanation: RealtimeSTT depends on timing. I you'd want to transcribe an audio file with it, you'd have to feed a chunk then wait for the time it would need to play out the chunk before feeding the next chunk. It would take much, much longer time to process that than with faster-whisper, which also delivers a full transcript, so I suggest using that library which was especially designed for this purpose.
Also text = recorder.text() would only yield you the first detected full sentence. You'd have to call it repeatedly to retrieve the full transcript. Btw use recorder.shutdown() or create recorder with "with" statement (Context Manager) to prevent it running forever.

Please use the really great faster-whisper library to transcribe an audio file (this lib is just not the right tool for that).
Explanation: RealtimeSTT depends on timing. I you'd want to transcribe an audio file with it, you'd have to feed a chunk then wait for the time it would need to play out the chunk before feeding the next chunk. It would take much, much longer time to process that than with faster-whisper, which also delivers a full transcript, so I suggest using that library which was especially designed for this purpose.
Also text = recorder.text() would only yield you the first detected full sentence. You'd have to call it repeatedly to retrieve the full transcript. Btw use recorder.shutdown() or create recorder with "with" statement (Context Manager) to prevent it running forever.

Thankyou, I will check further


Attaching document in chat shows a default name of "Chat_attachement" instead of the document name

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
Preview of the document should show the name of the file.
Actual Result:
Default name is displayed "chat_attachment"
Action Performed:

Login in chat app.
Go to a conversation
Tap on the attachment icon
Tap on document
Select a document.

Workaround:
No need, visual issue.
Platform:
Where is this issue occurring?
Web
iOS
Android ✔
Desktop App
Mobile Web
Version Number: 1.0.41-11
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
View all open jobs on Upwork

From @quinthar https://expensify.slack.com/archives/C01GTK53T8Q/p1620702841036200

tapping + > Add Attachment > Choose Document on Android native brings up a weird preview, but ultimately works.  The preview should show what was picked.

view the job here: https://www.upwork.com/jobs/~0140aec4e5587952a8

I don't have an android device to test with, and this isn't happening on iOS. Issue looks good though so I'm releasing to the pool.

Triggered auto assignment to @Julesssss (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Confirmed that the issue is occurring on Android.

Triggered auto assignment to @jboniface (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

view the job here: https://www.upwork.com/jobs/~0140aec4e5587952a8

Triggered auto assignment to @chiragsalian (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Interested in picking this up :)

I'm actually not sure why this is not an issue on iOS, but documentation for RNDocumentPicker says that the returned object has name and not fileName, so this function in src/components/AttachmentPicker/index.native.js has a mistake.
function getDataForUpload(fileData) {
    return {
        name: fileData.fileName || 'chat_attachment',
        type: fileData.type,
        uri: fileData.uri,
    };
}

The way I see it, the fix is to do fileData.name in the function above instead of fileData.fileName and after that check if iOS is still ok.

Did a quick check on my local. The proposed solution resolves the issue on Android and doesn't affect iOS.

Nice thats great. So on your local did you change it to,
name: fileData.name || 'chat_attachment',

?
If so I'm wondering why does the change not affect iOS. Mostly asking because then it feels like,
name: fileData.fileName || fileData.name || 'chat_attachment',

could be a safe solution. Let me know your thoughts.

@chiragsalian Yes, that's the change.
The way I see it, since documentation doesn't reference fileName at all, we shouldn't even try to check for it. A hypothetical future developer that's unaware of the current issue we have, would probably waste some time trying to understand the purpose of this check, unless we add some comment that clarifies why we do it.
Overall, I'm against it, the additional check wouldn't solve anything and won't make the app future-proof(in case the object returned by the lib function changes structure) either.
I am actually genuinely curious why it works on iOS without the change.
I assume the previews on iOS are rendered using only URI, and on android the native module requires both valid URI and filename. I'm looking into how that actual result is handled to render the preview, so maybe I get an actual answer instead of these speculations.

Cool yeah, I agree with you. If iOS is not using fileData.fileName, then yes we definitely should not have it there since its useless and confusing 👍
Yup looking forward to your research to finding the actual answer. I see it was originally added here - https://github.com/Expensify/Expensify.cash/pull/463/files#diff-76e298ccb5738caa8083274a1ead20df7abb8848e99e11284063975ce452b701R13
so maybe @Jag96 can shed some light on why he added it as filename. Maybe somewhere along the line we changed from using image picker to document picker and I bet filename was associated with image picker.

Ok, I got it. One could probably write a small novel on why it works on iOS, but i'll try to make it as brief as I can.
So first all, the file data gets passed down to AttachmentView component like this:
<AttachmentView sourceURL={sourceURL} file={this.state.file} />

Please note that in our case sourceURL is the same as file.uri.
The AttachmentView ultimately renders one of three views depending on file type: PDF, image and a general block with clip icon for everything else (you can see it on the screenshot attached to issue description).
Let's talk about PDF case first as that's what I had my initial testing with.
Here is a snippet of code that determines whether to render the file as PDF or not:
    if (Str.isPDF(props.sourceURL)
        || (props.file && Str.isPDF(props.file.name || props.translate('attachmentView.unknownFilename')))) {
        ...

Now the reason why the original issue isn't present on iOS is because of how the Str.isPDF() function is implemented. It basically only checks for .pdf at the end of the URI string. Here are sample URIs for both platforms:

android - content://com.android.providers.downloads.documents/document/msf%3A26
ios - file:///Users/dklymenk/Library/Developer/CoreSimu...2/tmp/com.chat.expensify.chat-Inbox/About_KPI.pdf

So on iOS the file would pass the initial check and it would render as PDF no problem, but on android you can see that it will check the file.name for .pdf instead. And this is why when you submit PDF on android it wouldn't render pdf preview, as second check results in false too.
Now after I discovered how AttachmentView works, I did a test with non-pdf, non-image file and as expected on both android and iOS it produces a chat_attachment label instead of file name. (Just like on the issue description).
So the fix I proposed earlier fixes the pdf display on android and non-pdf, non-image display on both iOS and android.

I noticed that on android pdf preview there is no scrolling. I am not sure if that's expected or not, but definitely worth mentioning here before applying the fix. If it's not expected, it would probably be outside of the scope of this particular issue.
Additionally, one might suggest that the file extension checking is implemented incorrectly as it relies on filename having .pdf in the end. I personally would suggest checking the data under type key instead of or in addition to current checks (depending on how it impacts web client). For pdf file, on both android and ios it returns application/pdf for me.

I noticed that on android pdf preview there is no scrolling. I am not sure if that's expected or not, but definitely worth mentioning here before applying the fix. If it's not expected, it would probably be outside of the scope of this particular issue.

Cool yeah I agree with you that if needed that would be outside the scope of this particular issue. cc @jboniface for awareness that it looks like even if we show the preview on android the scrolling does not work well. would be a different follow up issue though.

Additionally, one might suggest that the file extension checking is implemented incorrectly as it relies on filename having .pdf in the end. I personally would suggest checking the data under type key instead of or in addition to current checks (depending on how it impacts web client). For pdf file, on both android and ios it returns application/pdf for me.

Yeah I agree with you. We'll change this at a later time and not tackle this atm.
Anyway nice, thank you for those findings @dklymenk. Yeah, your proposed solution makes sense to me. Go ahead and create a PR for it.

so maybe @Jag96 can shed some light on why he added it as filename

Originally we were using a different library (image-picker) and that was what worked with that implementation. When we changed to the document picker in this PR I would think it was still working granted the tests there, so perhaps something else went wrong along the way.

Thanks @chiragsalian , I assume I'm meant to wait for proposal approval on upwork before submitting PR. That is according to contributing guide.

Yes, sorry, a little new to upwork 🙂  I've gone ahead and sent a hire request for your proposal on upwork. Let me know if anything else is pending on my side and if not go ahead and work on the PR.

Thanks. Offer accepted. You may assign the issue to me.

done

@chiragsalian Submitted a PR.


Clean up GraphEdit ports and slots

Godot version:

3.1
Issue description:

GraphEdit and GraphNode use "slots" and "ports" interchangeably, including in the exposed API. Can we unify that to just ports? Should be a simple across the board replacement job.

Actually, they are two different things. "Slots" are the nodes you need to add to GraphNode, while "ports" are the actual points on either side that you can connect to. Maybe this could be better clarified, but the term isn't really interchangeable.


Refine isNullsLast and isAsc functions.

With this change, we will not get an error if the ORDER BY expression has an empty operand (which might occur in V2 https://github.com/apache/pinot/issues/11168).

Codecov Report

Merging #11199 (d0d0943) into master (2904f1b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master   #11199   +/-   ##
=======================================
  Coverage    0.11%    0.11%
=======================================
  Files        2224     2224
  Lines      119351   119346    -5
  Branches    18069    18068    -1
=======================================
  Hits          137      137
+ Misses     119194   119189    -5
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
0.00% <0.00%> (ø)

integration1temurin17
?

integration1temurin20
0.00% <0.00%> (ø)

integration2temurin11
?

integration2temurin17
?

integration2temurin20
?

unittests1temurin11
?

unittests1temurin17
0.00% <0.00%> (ø)

unittests1temurin20
?

unittests2temurin11
0.11% <0.00%> (ø)

unittests2temurin17
0.11% <0.00%> (ø)

unittests2temurin20
0.11% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...uest/context/utils/QueryContextConverterUtils.java
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


tweak dependencies

the cats part is needed now for Doodle to be included in the Scala
community build.  depending only on the parts of cats that are actually
needed is also generally good practice.
the ScalaCheck version bump isn't strictly necessary at the moment
but will help ensure community build compatibility moving forward.
the ScalaTest version bump is just for good measure.

Thanks!


CGX swep: problems with bodies from surfaces

In geo.fbd I try to generate bodies by sweeping surfaces, made from lines. Even the surfaces with 5 edges won't sweep to a body although in geo1.fbd this works.
geo.zip

"swep" of a  surface either creates a map-meshable body or fails to generate a body from the generated surfaces. It is still possible to create a free-meshable body using the "body" command.
If "swep" has multiple surfaces to sweep, it stops to generate bodies as soon as it fails to do so for any of the surfaces. Workaround: Sweep the problematic surfaces separately and make the bodies using "body".


Contact you

Hi,
I was looking at your snippet and I was wondering if you have experimented with other kind of data.
As far as I can see this DALI library (I've been trying to find examples) is intented to be used for specific kind of data, right?
Pytorch dataset is flexible as you can code almost everything. Is it possible such flexibility here? is it restricted to premade image/audio/video loader?
As far as I can see it's only possible to use the system as dataloader. Is it possible to make use only of GPU accelerated video/image decoding tools in a normal pytorch dataset class?

This example is purely for image loading. DALI itself has added support for many other datatypes, such as audio. I suggest you check out the official docs, there's some examples in there now for other datatypes


chore(*): update dependencies

I spent a bit of time a few weeks ago submitting some PRs to express-graphql. All of them got merged (except one) so I figured it was time to update the express-graphql dependency and all the others while I was at it.

Looks good to me.


chore: major cleanup

Describe your changes
Based on the commit messages provided, here is a description of the changes made:

Reworking Dockerfile to use multistage build:

The Dockerfile has been reworked to utilize a multistage build approach.
The build process now uses separate images for the build stage and the runtime stage.

Cleanup and update tooling:

The Makefile has been cleaned up, removing unused or outdated variables and task.
The pre-commit hooks have been updated to ensure code quality and adherence to coding standards.

Reducing dependencies and cleaning up code:

The dependency on the Cobra and Viper libraries has been removed.
Parsing of environment variables has been added, which is a replacement for the removed dependencies.

Extending documentation for servers:

The godoc documentation has been extended for both the Provisioner and Identity servers.
This may include additional explanations, comments, and examples to help other developers understand and use these servers more effectively.

Standardizing labels on container:

The commit involves adding labels that conform to the org.opencontainers.image.* standard.
These labels include metadata about the container image, such as its name, version, and other important information, and help maintain consistency and compatibility with various tools and platforms.

Issue ticket number and link
N/A
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.
[ ] CHANGELOG.md has been updated should there be relevant changes in this PR.

I hope I didn't broke any CI, but I tried to maintain all defaults and behaviour of both the driver and Makefile.

Hi @shanduur - thank you for the submission!
I'll review it in the coming days, if not earlier. :)

@shanduur lint failed for some reason - mind checking whether this looks like it's coming from your changes?
https://github.com/s3gw-tech/s3gw-cosi-driver/actions/runs/9042470387/job/24849064117?pr=6#step:4:123
Let me know if not and I'll try figuring it out.

@shanduur merge - thank you for your patience and effort! :)


Animation track keys shift position when selected

Tested versions
v4.3.beta.custom_build [e96ad5af9]
Not happening in 4.3 beta1
System information
Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 (NVIDIA; <IP_ADDRESS>33) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)
Issue description
Animation keys are bugged. I had this situation, where selecting a key would shift it by a constant distance:
https://github.com/godotengine/godot/assets/2223172/192f66db-f9bb-49bf-8560-9d6894201304
Not sure how to reproduce it, but a similar bug is easy to reproduce.
Steps to reproduce

Have an animation with at least 1 key
Right click the key (make sure it wasn't selected before clicking)
Left click next to the key

https://github.com/godotengine/godot/assets/2223172/bf646804-36c0-4e9d-92c3-40ab23222497
Minimal reproduction project (MRP)
N/A

Bisecting points to #92424 as the culprit:

I have investigated and found that the problem is probably due to the fact that InputEventMouseMotion at line 3049 in animation_track_editor.cpp is valid even though it should not be fired when closing a popup. But in BezierEdit, mouse events are not fired when closing a popup.
if (mm.is_valid() && mm->get_button_mask().has_flag(MouseButtonMask::LEFT) && moving_selection_attempt) {
	if (!moving_selection) {
		moving_selection = true;
		emit_signal(SNAME("move_selection_begin"));
	}

	float moving_begin_time = ((moving_selection_mouse_begin_x - timeline->get_name_limit()) / timeline->get_zoom_scale()) + timeline->get_value();
	float new_time = ((mm->get_position().x - timeline->get_name_limit()) / timeline->get_zoom_scale()) + timeline->get_value();
	float delta = new_time - moving_begin_time;
	float snapped_time = editor->snap_time(moving_selection_pivot + delta);

	float offset = 0.0;
	if (abs(editor->get_moving_selection_offset()) > CMP_EPSILON || (snapped_time > moving_selection_pivot && delta > CMP_EPSILON) || (snapped_time < moving_selection_pivot && delta < -CMP_EPSILON)) {
		offset = snapped_time - moving_selection_pivot;
		moving_selection_effective = true;
	}

	emit_signal(SNAME("move_selection"), offset);
}

It seems that https://github.com/godotengine/godot/pull/92424 is causing the unintended firing of mouse events, but I don't have a detailed reason as to why this is happening.
@bruvzg Do you have any idea?

Check if it's fixed by https://github.com/godotengine/godot/pull/93105, might be the same issue.

@bruvzg The issue still occurs in the latest commit after merging it, so it seems to be a different problem.

Seems like mouse move event can be received after closing popup wit
h a click and button can be still pressed, but I do not see anything wrong with it, since mouse is moving and button is pressed.
You can also do the same by opening a popup menu, holding down the left button (while hovering the menu separator) and closing the menu with ESC, the key will be moving until you release the button.


Does the variable name using ref conflict with the registered name of the component

Version
3.2.31
Reproduction link
stackblitz.com
Steps to reproduce
main.ts
createApp(App).component('HelloWorld', HelloWorld).mount('#app');

App.vue

<template>

    <hello-world msg="Hello Vue 3 + TypeScript + Vite" />

</template>

What is expected?
The HelloWorld component can be displayed normally
What is actually happening?
1、The HelloWorld component is not working properly
2、Console error: Cannot read properties of undefined (reading 'el')
3、If the variable name of ref does not use helloWorld, such as helloWorld1, it will behave normally

This is expected:
https://vuejs.org/api/sfc-script-setup.html#using-components


formation control

inav or CF can be attached nRF24L01 as receiver
as I know, it can use in networking, so it will be possible to connect some drone with 1 bridge (hub or router)
so, maybe you interest in formation flight?
point need to be important :
identify of the drone. and CF and iNAV have MSP, its protocol is good and I think just need some header for identify the drone?

Great, thanks for your information


Adding results2

Added filepath printing for graphs and fixed creating filenames (removed whitespaces)

this does not look good :)

You can see - https://github.com/judovana/benchmarks-in-nested-virtualisation-toolchain/commits/main -  4 commit sof this batch are already here. Please check that they are identical. If no, check why, and whyic way is correct. If they are identical, preapre PR just with  https://github.com/judovana/benchmarks-in-nested-virtualisation-toolchain/pull/20/commits/3b368796731037b8fc782f3464c5f6315fc6319e

Fixed via https://github.com/judovana/benchmarks-in-nested-virtualisation-toolchain/pull/22 which is now mrged. TY!


Native Wifi / Wlanapi

MSDN Native Wifi Reference -> Wlanapi.h etc.
Is this missing and wanted?

Is it missing? Yes.
Is it wanted? I don't know who wants it, but I am assuming that you probably want it given that you filed this issue.

If y'all will allow me to chime in, I would love to have this implemented!
I am currently writing an application that aims to use Wifi Direct. I'm willing to contribute some code as needed, though I'm somewhat of a Rust n00b and haven't wrote Windows FFI bindings in Rust before.

@retep998 I want it too. I'm currently evaluating rust as an replacement for an old codebase which makes heavy use of the wlan api.


Accuracy mismatch using classifier.py - test() != validate()

I executed resnet50 with pruning, validation set and test set are the same. But for some reason, for the same weights, validate() and test() returned (similar but) different statistics.
On the last epoch, these are the results I got:
--- validate (epoch=122)-----------
50000 samples (440 per mini-batch)
Epoch: [122][   57/  114]    Loss 0.966480    Top1 76.120415    Top5 92.838915
Epoch: [122][  114/  114]    Loss 0.960678    Top1 76.186000    Top5 92.882000
==> Top1: 76.186    Top5: 92.882    Loss: 0.961
--- test ---------------------
50000 samples (440 per mini-batch)
Test: [   57/  114]    Loss 0.963796    Top1 76.251994    Top5 92.799043
Test: [  114/  114]    Loss 0.962186    Top1 76.186000    Top5 92.882000
==> Top1: 76.186    Top5: 92.882    Loss: 0.962
[checkpoint at: 2019.03.17-121241]

Hi @barrh
Because you expect that the results for test and validate to be equal, I assume that you ran compress_classifier.py with --vs=0.
I'm pretty sure you see these insignificant differences due to the way we accumulate the statistics.  When you use multiple worker threads for the dataloader, the inputs are processed in a non-deterministic order and that may slightly impact the intermediate  statistics.
I.e. this:
Epoch: [122][ 57/ 114] Loss 0.966480 Top1 76.120415 Top5 92.838915
vs. this:
Test: [ 57/ 114] Loss 0.963796 Top1 76.251994 Top5 92.799043
The final statistics are the same except for the thousands digit in the loss (0.961 vs 0.962).  I don't have an immediate explanation, but I'm sure it's also due to the different order inputs are processed.
Cheers
Netsa

Hey,
Thanks for your quick answer.
I close this issue because the difference is indeed insignificant, and evaluating the model with --deterministic provides a consistent result.

Just to clarify, it's not related to multiple workers. The way the data loader is implemented now, it randomizes the order of images in the set each time it's iterated over, regardless of the number of workers. This was done to enable using only a subset of the test/validation set, where we want the subset to be representative AND we want a different subset each time (useful in auto-compression, for example).
So the intermediate stats are expected to be different between different calls in the same run (wouldn't be surprised even if they were significantly different in the first couple of mini-batches). The final stats should be the same, but as Neta said, just the fact the inputs are ordered differently can cause small differences in floating point calculations.


Cannot reference data structure - again?

Now I am again convinced that I have it right, yet, it does not seem to work.
This data structure Místo - zahraniční adresa has https://slovník.gov.cz/legislativní/sbírka/89/2012/pojem/místo in root and I want to reference it.

This data structure has "odesílá dokumenty z místa" leading to Místo:

but the reference does not work:

Then we have "má místo sídla":

that leads to a more generic "Místo" https://slovník.gov.cz/generický/umístění/pojem/místo:

Yet, here it works, even though I would not expect it to:

Perhaps a caching problem? No longer happening with a fresh data structure.

Now it happens with this spec here:

For both cases, there are suitable structures in the reused specs, the first one also working in other places of this structure.


GatewayFilter in wrong scope based on README

moved GatewayFilter out of Filter class to fix https://github.com/rubycas/rubycas-client-rails/issues/4

DoH!


Copy-pasteable Glide instructions

My shell scripts bring all the boys to the yard

Thanks @bspaans for your copy-pastable instructions!


[berkeley] Fix entrypoint.d warning on startup: don't allow hidden scripts, and don't include . and .. as scripts

This PR removes a pattern that would handle (hidden) scripts which start with ., ensuring that all scripts are easily visible with ls. This logic also includes the directories . and .., giving us the annoying message
[ERROR] Entrypoint script /entrypoint.d/.. is not a regular file, ignoring

at every docker startup.

!ci-build-me

!ci-nightly-me

!approved-for-mainnet


Added Program for implementation of Doubly linked list

Description
1.Added this program in C folder
2.Program do all the operation on doubly linked list like insertion and deletion of node
3.Program also perform printing operation on doubly linked list

@afred-khan Can you please review my code and Let me know any correction that is needed to be done and Can you please explain me why this merge request is showing error because all the things seemed good.

@Anupam-Panwar Actually there's a problem with lint_python check, your code is fine!


Support the presence of the extraPages field

See: https://algorand.github.io/js-algorand-sdk/classes/transaction.html#extrapages
When I try to sign a transaction with this field using AlgoSigner (version 1.6.0 of the Chrome extension), I get:
"Creation of ApplTx has extra or invalid fields: extraPages."

The upgrade to using the js sdk 1.10.0 is slated for the 1.6.1 release which includes the extra-pages support. I'll add a ticket to explicitly verify it works for application transactions.

More info on this:
Issues appeared with 1.10.1, so we're updating to sdk-1.9.1 in the meantime for the AlgoSigner-1.6.1 release.
We'll focus on adding support for 1.10.1 on a different release.

I see there is a 1.6.1 release on github :tada:
When is this expected to land in the Chrome web store?

The release is there now, but a user reported a further issue with extra pages that is under investigation

I can confirm that I am still experiencing the same issue with AlgoSigner 1.6.1 in Chrome.

The extraPages field is lost during an internal decodeUnsignedTransaction() to the sdk. An issue has been logged https://github.com/algorand/js-algorand-sdk/issues/418.


R/Python Packages

Would you take a PR for these types of packages?  R's phyloseq package for example.

Sure.  If it is used for microbe analysis, let's put it on the list!


Cygwin install error: error: 'isnan' was not declared in this scope

Dear all,
after struggling with the installation of basemap I decided to post here. I have tried a couple of different 'hacks' found in google but none worked.
Full error:

libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I../../../include -I../../../include/geos -I../../../include -DGEOS_INLINE -pedantic -Wall -ansi -Wno-long-long -ffloat-store -g -O2 -MT IndexedPointInAreaLocator.lo -MD -MP -MF .deps/IndexedPointInAreaLocator.Tpo -c IndexedPointInAreaLocator.cpp  -DDLL_EXPORT -DPIC -o .libs/IndexedPointInAreaLocator.o
In file included from ../../../include/geos/geom/Geometry.h:26:0,
from IndexedPointInAreaLocator.cpp:18:
../../../include/geos/geom/Coordinate.inl: In member function 'bool geos::geom::Coordinate::isNull() const':
../../../include/geos/platform.h:88:27: error: 'isnan' was not declared in this scope
define ISNAN(x) (isnan(x))
                       ^

../../../include/geos/geom/Coordinate.inl:39:10: note: in expansion of macro 'ISNAN'
return (ISNAN(x) && ISNAN(y) && ISNAN(z));
^
../../../include/geos/platform.h:88:27: note: suggested alternative:
define ISNAN(x) (isnan(x))
                       ^

../../../include/geos/geom/Coordinate.inl:39:10: note: in expansion of macro 'ISNAN'
return (ISNAN(x) && ISNAN(y) && ISNAN(z));
^
In file included from ../../../include/geos/geom/Coordinate.inl:23:0,
from ../../../include/geos/geom/Coordinate.h:161,
from ../../../include/geos/geom/Envelope.h:26,
from ../../../include/geos/geom/Geometry.h:28,
from IndexedPointInAreaLocator.cpp:18:
/usr/lib/gcc/x86_64-pc-cygwin/5.4.0/include/c++/cmath:862:5: note:   'std::isnan'
isnan(_Tp __f)
^
In file included from ../../../include/geos/geom/Geometry.h:26:0,
from IndexedPointInAreaLocator.cpp:18:
../../../include/geos/geom/Coordinate.inl: In member function 'bool geos::geom::Coordinate::equals3D(const geos::geom::Coordinate&) const':
../../../include/geos/platform.h:88:27: error: 'isnan' was not declared in this scope
define ISNAN(x) (isnan(x))
                       ^

../../../include/geos/geom/Coordinate.inl:83:21: note: in expansion of macro 'ISNAN'
((z == other.z)||(ISNAN(z) && ISNAN(other.z)));
^
../../../include/geos/platform.h:88:27: note: suggested alternative:
define ISNAN(x) (isnan(x))
                       ^

../../../include/geos/geom/Coordinate.inl:83:21: note: in expansion of macro 'ISNAN'
((z == other.z)||(ISNAN(z) && ISNAN(other.z)));
^
In file included from ../../../include/geos/geom/Coordinate.inl:23:0,
from ../../../include/geos/geom/Coordinate.h:161,
from ../../../include/geos/geom/Envelope.h:26,
from ../../../include/geos/geom/Geometry.h:28,
from IndexedPointInAreaLocator.cpp:18:
/usr/lib/gcc/x86_64-pc-cygwin/5.4.0/include/c++/cmath:862:5: note:   'std::isnan'
isnan(_Tp __f)
^
make[4]: *** [Makefile:373: IndexedPointInAreaLocator.lo] Error 1
make[4]: Leaving directory '/tmp/basemap/geos-3.3.3/src/algorithm/locate'
make[3]: *** [Makefile:392: all-recursive] Error 1
make[3]: Leaving directory '/tmp/basemap/geos-3.3.3/src/algorithm/locate'
make[2]: *** [Makefile:444: all-recursive] Error 1
make[2]: Leaving directory '/tmp/basemap/geos-3.3.3/src/algorithm'
make[1]: *** [Makefile:476: all-recursive] Error 1
make[1]: Leaving directory '/tmp/basemap/geos-3.3.3/src'
make: *** [Makefile:368: all-recursive] Error 1

I am installing it on cygwin 64 bits.
I have also tried to use cygwin's binaries but they give errors on a tested script.
I am not sure of any additional relevant information.
Best regards,

attn: @cgohlke since you are the only one I know that understands Windows-related compile issues.

Well, this is Cygwin, which should behave like Linux. It seems that HAVE_ISNAN is defined even though the Cygwin headers don't have a isnan function. Probably better to install the GEOS library using the Cygwin package management and set the environment variable GEOS_DIR. Or try to patch the platform header to use std::isnan.

I could make it work:
On the original file include/geos/platform.h I changed (starting on line 87):
#if defined(HAVE_ISNAN)
# define ISNAN(x) (isnan(x))
#else
# if defined(_MSC_VER)
#  define ISNAN(x) _isnan(x)
# elif defined(__MINGW32__)
// sandro furieri: sanitizing MinGW32
#  define ISNAN(x) (std::isnan(x))
# elif defined(__OSX__) || defined(__APPLE__)
   // Hack for OS/X <cmath> incorrectly re-defining isnan() into oblivion.
   // It does leave a version in std.
#  define ISNAN(x) (std::isnan(x))
# elif defined(__sun) || defined(__sun__)
#  include <math.h>
#  define ISNAN(x) (::isnan(x))
# endif
#endif

to
# define ISNAN(x) (std::isnan(x))
# include <math.h>
# include <cmath>

Should I submit this bug somewhere else to get it corrected for cygwin users?

Yeah, you should check in with the geos folks to figure out what the
correct patch should be. I am thinking that there is something special
about 64-bit cygwin that is not getting captured by this file. Let me know
what the final patch is, and I'll include it here as well.
On Tue, Aug 23, 2016 at 8:36 AM, fmv1992<EMAIL_ADDRESS>wrote:

I could make it work:
On the original file include/geos/platform.h I changed (starting on line
87):
#if defined(HAVE_ISNAN)
define ISNAN(x) (isnan(x))
#else
if defined(_MSC_VER)
define ISNAN(x) _isnan(x)
elif defined(MINGW32)
// sandro furieri: sanitizing MinGW32
define ISNAN(x) (std::isnan(x))
elif defined(OSX) || defined(APPLE)
// Hack for OS/X  incorrectly re-defining isnan() into oblivion.
// It does leave a version in std.
define ISNAN(x) (std::isnan(x))
elif defined(__sun) || defined(sun)
include <math.h>
define ISNAN(x) (::isnan(x))
endif
#endif
to
define ISNAN(x) (std::isnan(x))
include <math.h>
include
Should I submit this bug somewhere else to get it corrected for cygwin
users?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/matplotlib/basemap/issues/314#issuecomment-241716415,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARy-KcegKC_gr9PUQxO-5ksLZ8Ys4z5ks5qiuk3gaJpZM4JqOCf
.

Looks like this has been reported issue for GEOS https://trac.osgeo.org/geos/ticket/630
I wonder if it has actually be patched in the GEOS repository.  The basemap supplied version of GEOS is a few releases behind current release.

When I replace the line in include/geos/platform.h and re-run the make; make install` command the platform.h file gets overwritten resulting in the same error message.

Ah! Don't replace those lines in the include/geos/platform.h dir, but rather in the basemap install directory ...\basemap-1.0.7\geos-3.3.3\include\geos

Most of my testing for #532 and #582 was done on Cygwin, so I suspect this issue has gone away, one way or another.
As a note: this kind of thing on Cygwin Python packages tends to be a result of
#include <stdio.h>
#include <Python.h>

rather than the other way around (Python.h sets a lot of visibility macros that need to be set before any system header is read to be effective).


Main script (kinomodel.py)

Description
This script requests information from KLIFS and populates a kinase object.
Todos
Notable points that this PR has either accomplished or will accomplish.

need to add functions in the Kinase class to calculate dihedrals and distances
need to write proper tests for this tool

Questions
Status

One suggestion would be to try to make this flexible so that the kinase object can aggregate information about all of the available pdb files for that particular kinase

One suggestion would be to try to make this flexible so that the kinase object can aggregate information about all of the available pdb files for that particular kinase

Thanks Steve!  Actually by querying with one kinase ID you do get all the pdb codes associated with it (each with probably slightly different start and end of the sequence)---but we can definitely gather all these information.

Exactly! Maybe pocket_seq could take a list of lists?

Yes that change should be straight forward.


Improve README.md - mention api_client as false

This PR improves the documentation of the README file and also
includes a mention that the user can configure api_client to false in
order to disable it.
Code blocks are splitted to make easier to read and copy.
Closes https://github.com/swoosh/swoosh/issues/609

Hi @philss , thanks a lot for the improvements!
Didn't realized you were adding mailer generator to Phoenix. That's amazing.


How to use Float when Menu.Button is a Fragment?

Use Version
Use version when question appear:

Headless UI: v1.6.6
Headless UI Float: v0.7.0
Framework: react v17

Describe the question
I'm trying to make a reusable wrapper around the Headless UI's Menu.Button component, but for some reason when I use it, Float doesn't seem able to position the Menu.Items correctly and it just goes to the top left of the screen instead of next to the button itself.
Here's what I'm using:
import { Menu as HeadlessMenu } from '@headlessui/react'

// ....

const Opener: React.FC<OpenerProps> = React.forwardRef<
  HTMLDivElement,
  OpenerProps
>(({ children }, ref) => (
  <HeadlessMenu.Button as={React.Fragment}>
    <children.type {...children.props} ref={ref} />
  </HeadlessMenu.Button>
))

If I change:
<HeadlessMenu.Button as={React.Fragment}>

to:
<HeadlessMenu.Button as="div">

it works, but for my use-case I really want to use a Fragment there instead.
Any thoughts on what I could do to use a Fragment but still get the correct positioning?  I thought by passing the ref down into the children that would do it, but apparently not.
In case it helps, here's what I have for my Float component (again this is inside a custom reusable component):
const Wrapper: React.FC<WrapperProps> = ({ children, placement }) => (
  <HeadlessMenu>
    <Float
      offset={10}
      placement={placement}
      portal
      enter="tw-transition tw-ease-out tw-duration-100"
      enterFrom="tw-transform tw-opacity-0 tw-scale-95"
      enterTo="tw-transform tw-opacity-100 tw-scale-100"
      leave="tw-transition tw-ease-in tw-duration-75"
      leaveFrom="tw-transform tw-opacity-100 tw-scale-100"
      leaveTo="tw-transform tw-opacity-0 tw-scale-95"
    >
      {children}
    </Float>
  </HeadlessMenu>
)

Hi @jarcoal, the ref must be defined on the <Menu.Button>:
const Opener: React.FC<OpenerProps> = React.forwardRef<
  HTMLDivElement,
  OpenerProps
>(({ children }, ref) => (
  <HeadlessMenu.Button as={React.Fragment} ref={ref}>
    <children.type {...children.props} />
  </HeadlessMenu.Button>
))

Thanks for the reply @ycs77 .  I should've mentioned that I tried that and received this Typescript error related to the ref prop:
Type '{ children: ReactElement<any, string | JSXElementConstructor<any>> & ReactNode; as: ExoticComponent<{ children?: ReactNode; }>; ref: ForwardedRef<...>; }' is not assignable to type 'IntrinsicAttributes & Props<ExoticComponent<{ children?: ReactNode; }>, ButtonRenderPropArg, ButtonPropsWeControl>'.
  Property 'ref' does not exist on type 'IntrinsicAttributes & Props<ExoticComponent<{ children?: ReactNode; }>, ButtonRenderPropArg, ButtonPropsWeControl>'.ts(2322)

I'm assuming the type HTMLDivElement needs to be changed to something else, I'm just not sure what it should be...

The button DOM type is HTMLButtonElement, this will be clear the error:
const Opener: React.FC<OpenerProps> = React.forwardRef<
  HTMLButtonElement,
  OpenerProps
>(({ children }, ref) => (
  <HeadlessMenu.Button as={React.Fragment} ref={ref}>
    <children.type {...children.props} />
  </HeadlessMenu.Button>
))

Same error:
Type '{ children: ReactElement<any, string | JSXElementConstructor<any>> & ReactNode; as: ExoticComponent<{ children?: ReactNode; }>; ref: ForwardedRef<...>; }' is not assignable to type 'IntrinsicAttributes & Props<ExoticComponent<{ children?: ReactNode; }>, ButtonRenderPropArg, ButtonPropsWeControl>'.
  Property 'ref' does not exist on type 'IntrinsicAttributes & Props<ExoticComponent<{ children?: ReactNode; }>, ButtonRenderPropArg, ButtonPropsWeControl>'.ts(2322)

Maybe it's an issue with Headless-UI?
For now I've created a work-around, so I'm all set, just wanted to see if there was a quick-fix that I was missing.  Thanks again for your help.


import the required package in templates directly

Sometimes goimports can't fix the imports correctly.
Signed-off-by: spacewander<EMAIL_ADDRESS>Please provide a description of this PR:
When running go run pkg/config/schema/codegen/tools/collections.main.go to generate code, goimports will be invoked to sort and fix the imports. The latest goimports can't fix the cache import in pkg/config/schema/kubeclient/resources.gen.go, which will cause a later compilation error.
The reproducible script can be found at https://github.com/mosn/htnn/actions/runs/8995044785/job/24709403041?pr=526.
The k8s.io/client-go/tools/cache will be missed in the latest generated code:
+ git diff --exit-code
diff --git a/pkg/config/schema/kubeclient/resources.gen.go b/pkg/config/schema/kubeclient/resources.gen.go
index 6bd27b3..b1e0c8d 100755
--- a/pkg/config/schema/kubeclient/resources.gen.go
+++ b/pkg/config/schema/kubeclient/resources.gen.go
@@ -18,7 +18,6 @@ import (
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
 	"k8s.io/apimachinery/pkg/watch"
-	"k8s.io/client-go/tools/cache"
 	sigsk8siogatewayapiapisv1alpha2 "sigs.k8s.io/gateway-api/apis/v1alpha2"
 	sigsk8siogatewayapiapisv1beta1 "sigs.k8s.io/gateway-api/apis/v1beta1"

The solution here is adding the required package directly in the templates, so we don't need to depend on goimports to fix it.

Hi @spacewander. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

/ok-to-test


Add list of changes; up-to-date when added

It is probably good idea to have a place not to notify people of never-forthcoming changes to the codebase. Seems like a CHANGELOG was a must have.
I've added one that is currently up-to-date with the source code.

I would recommend we rename this file to NOCHANGELOG.md, but otherwise, this is a good change.


Lots of product templates sharing same base css file, getting ENAMETOOLONG

I have product templates 1-10. product.liquid, product.1.liquid, product.2.liquid, etc. They all share the same product.css file. This causes webpack to try to create the following file: /dist/assets/vendors@template.collection@template.index@template.page.about@template.product@template.product.1@template.product.10@template.product.2@template.product.3@template.product.4@template.product.5@template.product.6<EMAIL_ADDRESS>which results in a ENAMETOOLONG error.
I tried to edit the prod.config.js file to instruct mini-css-extract-plugin to change the chunkfilename but seems like there's an issue with mini-css-extract-plugin and webpack 5? It results in a "The 'compilation' argument must be an instance of Compilation" error. I see that in the following file here: https://github.com/hayes0724/shopify-packer/blob/956a42dba5152c1676ef4cb3324b42e9992412bf/src/utilities/get-chunk-name.js#L23 you have a reference to ENAMETOOLONG but that's just for JS files, right?
Any way to resolve this? In reality I don't even need product.1.js, product.2.js, product.3.js, etc. They could all share the base product.js entry file, I just need the separate templates so that I can have independent sections for each of the templates. So if there's a way to have them all share the same product.js template that would resolve my problem too.

I resolved this by creating a chunker.js file which is a copy of this package's /src/utilities/get-chunk-name.js file, and modified it to 100 characters instead of 256. I also had to remove the ~ character from the returned filename because apparently shopify says that's an illegal character. Here it is for reference:
const crypto = require('crypto');

module.exports = function (module, chunks, cacheGroup) {
  let containsLayout = false;
  const names = chunks
    .map((chunk) => {
      if (chunk.name.includes('layout.')) {
        containsLayout = true;
      }
      return chunk.name;
    })
    .filter(
      (name) => !containsLayout || (containsLayout && name.includes('layout.'))
    );

  if (!names.every(Boolean)) return;

  names.sort();
  let name =
    (cacheGroup && cacheGroup !== 'default' ? `${cacheGroup}@` : '') +
    names.join('@');

  // Filenames and paths can't be too long otherwise an
  // ENAMETOOLONG error is raised. If the generated name is too
  // long, it is truncated and a hash is appended. The limit has
  // been set to 100 to prevent `[name].[chunkhash].[ext]` from
  // generating a 256+ character string.
  if (name.length > 100) {
    name = `${name.slice(0, 100)}${hashFilename(name)}`;
  }

  /* eslint-disable-next-line consistent-return */
  return name;
};

function hashFilename(name) {
  return crypto.createHash('md4').update(name).digest('hex').slice(0, 8);
}

Then in prod.config.js I added
const chunker = require('./chunker');
const mergeProd = {
module: {
// ...
},
optimization: {
splitChunks: {
chunks: 'all',
name: chunker
},
}
}


python3.5

[ ] model __unicode__ vs __str__
[x] install python3.5 on hyperbola01
[x] deployment with selectable python

Had to install some software for the build to work: https://github.com/yyuu/pyenv/wiki/Common-build-problems
sudo apt-get install make build-essential libssl-dev zlib1g-dev libbz2-dev \
libreadline-dev libsqlite3-dev wget curl llvm libncurses5-dev libncursesw5-dev

Python installation fixed in https://github.com/lopopolo/hyperbola-tools/commit/119f1ec71f55f2f7d136a6b10b8b625b811709bc
Deployment with selectable python fixed in https://github.com/lopopolo/hyperbola-tools/commit/93c6a65114674ce0f23e41560ee6651d6f481a0a

Currently staging is running with py3.

as of https://github.com/lopopolo/hyperbola/commit/22cd9d4815d5f92e479e34b4e442029f22a5bbb9, hyperbola is running with python 3.5.1 in both staging and production.


Fix clash with modules that manage the unit folder too

Other puppet modules in a puppet setup might manage the {unit}.d folder as well. For example puppetlabs/docker does (See https://github.com/puppetlabs/puppetlabs-docker/blob/main/manifests/service.pp).
If one still wants to add limits overrides via this systemd module, puppet will complain about duplicate resource declaration.

@ekohl You would think so but unfortunately in our case it doesn't.
The detail is in the explanation of the function ensure_resource.

If the resource already exists but does not match the specified parameters,
this function will attempt to recreate the resource leading to a duplicate
resource definition error.

You can test yourself:
ensure.pp
file { '/tmp/foo.d':
  ensure => directory
}

ensure_resource('file', '/tmp/foo.d', {
  ensure                  => directory,
  owner                   => 'root',
  group                   => 'root',
  recurse                 => false,
  purge                   => false,
  selinux_ignore_defaults => false,
})

Results in:
$ puppet apply ensure.pp
Error: Evaluation Error: Error while evaluating a Function Call, Duplicate declaration: File[/tmp/foo.d]
is already declared at (file: /tmp/ensure.pp, line: 1); cannot redeclare (file: /tmp/ensure.pp, line: 5)
(file: /tmp/ensure.pp, line: 5, column: 1) on node

Iirc, ensure_resource only works if the parameters are exactly the same in both declarations.

That's why we had to add the if !defined check. For standalone usage inside the systemd module, ensure_resource works like it should. But you will run into puppet failures when the {unit}.d folder is being managed by another puppet module.
@raphink how can I get this change merged?

@hdeheer do you know which parameters don't match?

@ekohl Yes.
For example https://github.com/puppetlabs/puppetlabs-docker/blob/main/manifests/service.pp only uses:

ensure

file { '/etc/systemd/system/docker.service.d':
  ensure => 'directory',
}

But the systemd module manifests/dropin_file.pp  uses:

ensure
owner
group
recurse
purge
selinux_ignore_defaults

Easy fix in our case is to use if !defined.

I still think it's a bad idea. This module depends on certain behavior, such as purge. If that's not specified, you get unpredicatable behavior. Namely, depending on the parse order it could purge unmanaged files or not. That can be a nasty surprise.

I'm also not a fan of this at all. I'd rather contribute to the Docker module to depend on the systemd module.

This should be closed.
Will close after the solstice.

Winter solstice 2023 @traylenator? :D


[BUG] Issue with percolate query transforming documents with object type fields

Mapping
{
  "mappings": {
    "properties": {
      "something.a.b": { "type": "long" }
    }
  }
}

Document that the monitor is not able to recognize but the index ingests correctly based on. the mapping
{
    "something": {
        "a" : {
            "b" : 123
        }
    }
}

Document that the monitor is able to recognize and trigger
{
    "something.a.b" : 123
}

Doc-level monitor spec
{
						"type": "monitor",
						"schema_version": 1,
						"name": "test",
						"monitor_type": "doc_level_monitor",
						"enabled": true,
						"enabled_time":<PHONE_NUMBER>150,
						"schedule": {
							"period": {
								"interval": 1,
								"unit": "MINUTES"
							}
						},
						"inputs": [{
							"doc_level_input": {
								"description": "test",
								"indices": [
									"i3"
								],
								"queries": [{

										"name": "78b3756a-7804-4ef7-8555-7b9024a02d2d",
										"query": "(something.a.b:123)",
										"tags": [
											"low",
											"cloudtrail",
											"attack.exfiltration",
											"attack.t1537"
										]
									}
								]
							}
						}],
						"triggers": [{
							"document_level_trigger": {
								"id": "FY5RL4cBirpptBbLj0ju",
								"name": "CW Trail Alert",
								"severity": "1",
								"condition": {
									"script": {
										"source": "(query[tag=cloudtrail])",
										"lang": "painless"
									}
								}
							}
						}]

					}

We need to consciously handle transformation logic for null values and array type values also

Field transform logic at needs to be fixed
https://github.com/opensearch-project/alerting/blob/3f2e6519792a6162c79481b5575b5625b5f78f20/alerting/src/main/kotlin/org/opensearch/alerting/DocumentLevelMonitorRunner.kt#L563-L575

PR has been merged, but backports failed


Fix PyPi description

I think at some point I broke the description on PyPi.
https://pypi.org/project/mavsdk/

What is broken there? I see the description, at least :thinking:

Huh! It's fixed itself. With 0.9.0 it was broken: https://pypi.org/project/mavsdk/0.9.0/

Good, I don't wanna know :D


Extra features

Hello @GlobalHive
Just came across this plugin trying to make vue-tours work with Vue3.
Happy that I found yours work as a charm.
I aim to use this plugin for creating multi-page onboarding tutorials for my webapp, therefore I need some more utility functions to possibly achieve that.
Requirements:

Being able to run a tour over and over again (I got around that by sending save-to-local-storage="nope" to not store in the local-storage that the tour was completed)
Being able to jump to a certain step (maybe a function called goToStep)
Being able to NOT show neither Next or Finish (this is to prompt the user to do an action like press a button that will lead him to another page and thus continue the tour from there)

Thank you very much in advance :)

Hi @Ktsirakos,
If you don't want to save the tour state, use saveToLocalStorage="custom" or any value other than "step" or "end"
To hide "Back / Next / Finish," customize the "action slot" and leave it empty. Check this link for more details.
I'll add the goToStep  as soon as possible. (Might be added in 1-2 days)
Sincerely,
Sascha
Global Hive

Feature request has been added.
To use it create a ref to the VTour component. Then call the goToStep() function
vTour.value.goToStep(2)


Add Standalone and AMD edition

See also #15.

Oops, coverage was down. I'll update it, soon.

It looks great thanks


Port Wavegen functions

[x] SET_SINE1 (#92)
[x] SET_SINE2 (#92)
[x] SET_WG_PHASE (#95)
[x] LOAD_WAVEFORM1 (#92)
[x] LOAD_WAVEFORM2 (#92)
[x] SET_SQR1 (#92)
[x] SET_SQR2 (#92)
[x] SQR4 (#92)
[x] MAP_REFERENCE (#96)

All tested and implemented.


missing line breaks in functions types without comments

Original reporter: waldmann@
if a function's type contains no embedded annotations,
then it is rendered without any reasonable linebreaks
(it just breaks at the right border) which makes it hard to read.
See function symdiff here
http://dfa.imn.htwk-leipzig.de/doc/autolib-fa-1.1/html/Autolib-NFA-Minus.html
and I attach a screenshot, just to be sure.

Related: #472.


[FEATURE] Add Github link to the site #7

#7
Created a hard-coded GitHub link and placed it with an icon below the sidebar navigation elements.
Please let me know if you would like this to be moved.
Is the plan to have a site config model where this data could be stored rather than hard coding the link ref?

also i believe it should open in new tab rather than closing the current user session. what you think @rupali-codes

I have made the link open a new tab and updated the title to specify that this is an external link to make it as accessible as possible.
In terms of making the icon static, this will require a refactoring of the sidebar which I think is something that should be discussed first. It will need a fixed height and overflow added to statically display link items at the bottom. Are you sure you want to do that?
I am not sure it is a great user experience to have a scrolling sidebar with tons of links and there are not many examples of other sites doing that.

okay, can we put it somewhere else where user can see it without scrolling the page?

@rupali-codes Could you let me know why the build is failing please.

@JoshHargreaves I have resolved that. now its LGTM. What are your thoughts on this @rohitdasu


Refactor error type

Check out the previous commit (204b2fa) to see the user-facing effect. As noted above, the error code functionality is present but disabled via comment. This is for uniformity (the other error types don't yet have codes) and to prevent us from committing to specific error codes / error types right now (in case of renumbering or more cases).


How to render some element as CDATA?

I have feed:
<?xml version="1.0" encoding="UTF-8"?>
<SHOP>
	<SHOPITEM id="392">
		<NAME>Andělika obrovská</NAME>
		<GUID>575c1bb4-710a-11eb-b3db-0cc47a6c9370</GUID>
		<SHORT_DESCRIPTION><![CDATA[
Výška sazenice cca 20 cm
]]></SHORT_DESCRIPTION>
	</SHOPITEM>
</SHOP>

I need to parse it and then render it.
I would like to ask your advice on how to render "SHORT_DESCRIPTION" as CDATA.
$reader = $service->getReader();
$reader->xml(file_get_contents($feedUrl));
$xml = $reader->parse();

//bdump($xml);

$writer = $service->getWriter();
$writer->openMemory();
$writer->setIndent(true);
$writer->startDocument('1.0', 'UTF-8');
$writer->namespaceMap[''] = '';
$writer->write($xml);

//bdump($writer->outputMemory());

Can you please add the code to my sample?

Why do you need it? It's almost always a mistake

Because the "SHORT_DESCRIPTION" element contains HTML code that consists of "<" and ">" characters.
Element must be rendered as "CDATA". It is a mandatory requirement for importing feed in the e-shop platform shoptet.cz.

It's much better to map those to < ad >. If your e-shop platform requires it, they use a broken xml parser which would be very surprising!


"AKA" in documentation

Having read this: https://github.com/ramda/ramda/commit/b7867021a423b0eed0de26add56fe49da3836086
I thought it might be a good idea to add an AKA section, or perhaps a "You may know it as", section to the docs under functions that have multiple names they've been popularized as. I'm totally onboard with having just one name for everything, and also choosing the name thats best for this library vs what it might be called in another library, but it can be quite frustrating when the function you know isn't available under the name you know it as (just the other day I was trying to think "what's the other word for every?"). At least for me, one of the more difficult parts of switching over to this style of programming was the nomenclature (it feels like a lot of times what you want is already implemented, if you just knew what it was called...). If the docs for example had foldl under "reduce", then a quick search would bring it up, you see what its called here, and then use that.

A very good idea.
Adding and formatting a JSDoc tag should be easy.  The trick will be to make it easily searchable in the documentation.

see #1038, closing as duplicate


security: Improve TLS

The implementation of TLS in Skydive is severely lacking and a little bit unorthodox.
Here is an attempt at making the implementation a little more standard and easier to understand.
Agent Does not use TLS unless CA Certificate and Keys are specified
Previously, the logic for checking if the admin wanted TLS was to check whether the analyzer:cert was specified. This is counter-intuitive and bad security practice.
This MR changes this so that the logic is changed.
Concepts regarding certificates and services confusing
Previously, the code used analyzer:cert for server certificates, and agent:cert for client certificates. This gets confusing, since when Skydive runs in "agent" mode, a server is set up. Why would it use the certificates from the Analyzer configuration for that.
This MR proposes to move the specification of certificates to a separate node, like so:
tls:
 client_cert: /pki/client.crt.pem
 client_key: /pki/client.key.pem
 server_cert: /pki/server.crt.pem
 server_key: /pki/server.key.pem
 ca_cert: /pki/ca.crt.pem

When Skydive is started in Analyzer mode, it uses the Server certificates for the service, and uses Client certificates when connecting out to other Agents.
When Skydive is started in Agent mode, it also uses the Server certificates for the Agent service, and uses Client certificates when connecting to the Analyzers.
Inappropriate use of certificate trust
Previously, the examples and implementation did not allow for an administrator to dictate what certificates to trust in a secure way.
This MR proposes to allow the admin to specify tls:ca_cert that is used to verify peers (either from client to server or from server to client).
This way, the admin does not have to "give away the keys to the kingdom" by giving all Agents the certifcate AND key for the issuing CA.
It also matches better with the standard way of implementing PKI, where a Root CA issues a Subordinate CA that issues Certificates. The Root and Subordinate CAs private keys must be kept secret.

Hi Maglo,
Thank you for your contribution.
Thanks to pin point us the inappropriate use of certificate trust in our implementation
Mainly due to the most common fault : a bit of code history, as the connection was only agents -> analyzer at the beginning.
Could you please accept our github invitation (email), aim to run our CI tests on this PR.
If you have 2min could you give us some clue how do you use/have interest in Skydive ?

I’m currently not using it, but I’m considering it as a source of truth regarding networking. Also want to provide ad-hoc inspection capabilities to operators of private clouds or container platforms.

I’m hoping that in the future Skydive will integrate better with physical network equipment via f.ex OSPF, Netflow etc.

@maglo good to known :)
We support LLDP discovery probe from #1344
OSPF and NetFlow/IPFix could be interesting, but NetFlow is pretty limited
You will may be interested by contrib/snort example, basically to map snort alerting on the topology

run python-tests

Great work.
Thanks for the help with CI.


fix: multi fixes (see PR details)

Changes description

fix: Align naming style for resources
fix: Remove hardcoded dimens
fix: Remove hardcoded text appearances
fix: Change parent of modal text style
Context

close #114
close #115
Checklist

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Please, don't request directly from your main!
[x] Check commits & PR names matches our requested structure. It must follow the https://www.conventionalcommits.org pattern.
[x] Check your code additions will fail neither code linting checks.
[x] I have reviewed the submitted code.
[x] I have tested on a phone device/emulator.
[x] I have tested on a tablet device/emulator.
[x] I have tested on a large screen device/emulator.
[x] If it includes design changes, please ask for a review with a core team designer.

Does this introduce a breaking change?

No

Screenshots
Large screen

Example of what we can do in redefining some dimens in the sample project. (This example is not committed, it's just some testing)
Before

After

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/62ac2962b2dd627a/build/f5482a9f-a1f0-4b97-9410-1382b6f0ff4c/artifact/25245a906b6d6726/p/5b7141e3ea0544909baab74ffd1a09e9

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/62ac2962b2dd627a/build/2d3b2c3c-2bd7-4877-a710-b8bad64fcc1a/artifact/1e62de98e5ff9ee9/p/b98b44058844099950096cb97bded885

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/62ac2962b2dd627a/build/866eab02-1ab0-44ae-bee0-e18fa8ac8c19/artifact/01cc131894d8b191/p/ec6c4d2c96d155da5316b345f5581655


As a CIF User, I want the list of CIF operators to be automatically imported from SWRS, so that I don't have to enter all the operators

Description:
An import of the operators needs to be triggered every time the SWRS data is being transformed, and when the application is deployed.
To avoid the issues encountered with CIIP when the data is copied across databases with pg_dump/pg_restore, the connection to the SWRS database should instead be done with a foreign data wrapper. The FDW can be created and removed during the job that imports the list of operators.
Acceptance Criteria:
Given that I am installing or upgrading the cas-cif helm chart
When the database is setup (after the sqitch plan is deployed)
Then a cif_private.import_swrs_operators(host, dbname, port, user password) postgres function is run to import the operators from SWRS
Given that the cif_private.import_swrs_operators function exists
When it is invoked
Then it:

creates a server, user mapping and foreign tables (swrs.organisation, swrs.report) using the postgres_fdw extension
copies the operators from the swrs.organisation table into the cif.operator table
drops the server with cascade

Given that the swrs.organisation foreign table is setup
When operators are imported from it
Then only the latest record for each swrs_organisation_id is imported (most recent reporting year and report submission date)
And only the swrs_organisation_id, business_legal_name and english_trade_name are imported
Given that an operator was previously imported
When importing the operators
Then fields that were edited in the CIF app should not be overwritten (i.e. we need to track editing of the legal and trade names)
Development Checklist:

[x] The helm chart should lookup the GGIRCS DB readonly user credential and copy them into the cas-cif secret, which is updated on every helm upgrade
[x] Add network policy in GGIRCS namespace to allow CIF db ingress
[x] cif.operator has a swrs_organisation_id column
[x] cif.operator has legal_name_updated_by_cif_user and trade_name_updated_by_cif_user boolean columns tracking whether the fields were edited via the CIF app. That could be done via a trigger that is disabled during the import
[x] Meets the DOD

Notes:

Personas:

Program Development Team
Operations Team

Hey team! Please add your planning poker estimate with ZenHub @AlexZorkin @dleard @pbastia

Thank you for the walk through @dleard, much appreciated. I look forward to seeing this in TEST where the data will be more representative. Awesome work.


material who

i am fvoziy i smell bad -luckfire

asked

me

WOOOOOOOOOOO

GET RICK ROLLED LOLOLOL

MATERIAL ME

Material Hoof

hoofer now owns this theme. all your cookies belong to him. you dont get a say in anything.


Compile error: Module not found '@ledgerhq/devices/hid-framing'

Describe the bug
Compiling React app fails due to Ledger dependencies. A module named '@ledgerhq/devices/hid-framing' is not found.
To Reproduce
Steps to reproduce the behavior:

Create test React app
Add Solana-Wallet-Adapter as described in the Readme.md
Compile error yarn build:yarn run v1.22.19
$ react-scripts build
Creating an optimized production build...
Failed to compile.

<EMAIL_ADDRESS>Cannot find module: '@ledgerhq/devices/hid-framing'. Make sure this package is installed.

You can install this package by running: yarn add @ledgerhq/devices/hid-framing.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Expected behavior
The React app should compile without any errors. The Ledger dependency has been moved because, yarn add @ledgerhq/devices/hid-framing turn also into an error:
yarn add v1.22.19
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@ledgerhq%2fdevices/hid-framing: Request \"https://registry.yarnpkg.com/@ledgerhq%2fdevices/hid-framing\" returned a 405".
info If you think this is a bug, please open a bug report with the information provided in "/Users/volker/Projekte/solana/nft-geocaching/react-app/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

Screenshots

Additional context
package.json:
 "dependencies": {
    "@solana-mobile/wallet-adapter-mobile": "^0.0.1-alpha.8",
    "@solana/spl-token": "^0.2.0",
    "@solana/wallet-adapter-base": "^0.9.9",
    "@solana/wallet-adapter-react": "^0.15.8",
    "@solana/wallet-adapter-react-ui": "^0.9.11",
    "@solana/wallet-adapter-wallets": "^0.17.0",
    "@types/react": "^18.0.0",
    "react": "^18.2.0",
    "react-dom": "^18.1.0",
    "react-range": "^1.8.12",
    "react-router-dom": "^6.3.0",
    "react-scripts": "^4.0.3",
    "typescript": "^4.5.5"
 }

Edit: Is already addressed to Ledger: https://github.com/LedgerHQ/ledger-live/issues/763

Based on https://github.com/LedgerHQ/ledger-live/issues/763#issuecomment-1198967469 and subsequent comments it sounds like this is a problem with your build tool. I see you're using react-scripts (aka CRA) 4, which uses Webpack 4, leading to this issue. This is not supported (CRA 5 is required, along with react-app-rewired). Check out the CRA 5 starter project.

Getting some other reports of this happening, reopened

This appears to be an issue with the latest release of the dependencies, see https://github.com/LedgerHQ/ledger-live/issues/763#issuecomment-1205887506
Will fix this by pinning the previous version.


Replace the existing ModelAdmin importer

The existing importer will need to be replaced. This could be as simple as removing the sidebar importer, and adding the GridFieldImporter component to the ModelAdmin GridField.

Introduced in aaa408c3cacb672df0772caf9ce3b66bdc576686


Add support for authenticating with Github Enterprise

Upgrades Authom to use the built-in support for detection of the correct
backend according to the github url provided.

goes with filearts/plunker_api#2

Haven't had a chance to look at this deeply. I notice that you add a new config param for the github url. Can you please take a look if the relevant code-paths will automatically fall back to the current behaviour?

Yes, I if I omit the oauth:github:url parameter it automatically falls back to github.com as the auth provider. Have verified with www and api PRs.

here is the code path this relies on in the updated Authom module.  nconf.get("oauth:github:url") will be null if the option is not provided so the URL will be set to github.com.  I've copied this logic directly into the API project PR since you aren't relying on Authom there.


[proposal] visuals.SpinnerPromise

###*
	# @summary Create a CLI Spinner that spins on a promise
	# @name SpinnerPromise
	# @class
	# @public
	# @memberof visuals
	#
	# @description
	# This function will start a Spinner and stop it when the
	# passed promise is either fulfilled or rejected. The function
	# returns a promise that is is resolved with the passed
	# promise's value.
	#
	# @param {Promise} promise - promise to spin upon
	# @param {String} startMessage - start spinner message
	# @param {String} stopMessage - stop spinner message
	# @fulfil {Object} value - resolved 'promise' value
	# @returns {Promise}
	#
	# @throws Will stop spinner and throw on 'promise' rejection
	#
	# @example
	#  visuals.SpinnerPromise
	#    scanDevicesPromise
	#    "Scanning devices"
	#    "Scanned devices"
	#  .then (devices) ->
	#    console.log devices
	###

@jviotti do you think it'd make sense to have this function PRed here? https://github.com/resin-io-modules/resin-sync/blob/master/lib/utils.coffee#L125-L138

Looks good, however I'd integrate these changes into visuals.Spinner itself rather than creating a new class. What about accepting an optional promise as an argument to the .start() method, and make such method be resolved/rejected based on the input promise, and .stop() automatically when necessary?
Also, the "stop" message should be handled by the client IMHO.
For example:
spinner = new visuals.Spinner("Scanning devices")
spinner.start(scanDevicesPromise).then ->
  console.log('Scanned devices')
  console.log('Spinner also stopped')
.catch ->
  console.log('An error happened')
  console.log('Spinner was also stopped')

Also, the "stop" message should be handled by the client IMHO.

This was one of the things I wanted to avoid actually :P . If the passed promise is resolved, and a 'stopMessage' has been provided, it would save module users from adding an extra .then() method call just to print their success message.
Passing an optional promise in .start() would work, but that would make the API inconsistent: in the synchronous version the .start() will be followed by a matching .stop(), while when a promise is passed .stop() will be implied . Hence the proposal of having a separate class for this feature.

I see, lets added it then, by accepting an options object containing both startMessage and stopMessage so its clear from the user point of view which message corresponds to what.


Bot updates

Description

Test Steps

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run uncrustify

/bot run checks

/bot run formatting


Toggle Sidebar

Maybe I'm blind but I can't find command to toggle sidebar. Minimap already have toggle function/

Hi, thank you for this issue :)
I have just added GonvimSidebarToggle command.
https://github.com/akiyosi/goneovim/actions/runs/2245157612


Following the Getting Started Guide gives me this error

Checklist

[X] I have read and accepted the contributing guidelines.

How did you create the site?
Generated from chirpy-starter
Describe the bug
I followed the chirpy guide, and when I run bash/tools.sh I got this error:
HEAD is now at 20987f2 chore(release): 7.0.1
Submodule 'assets/lib' (https://github.com/cotes2020/chirpy-static-assets.git) registered for path 'assets/lib'
Cloning into '/mnt/c/Users/kamek/OneDrive/Documents/Code/Websites/kamek437.github.io/assets/lib'...
Submodule path 'assets/lib': checked out 'b9c58cf485a7dcbc833e698d67dd1850bdc93eb3'
npm warn deprecated<EMAIL_ADDRESS>Renamed to read-package-up

added 706 packages, and audited 886 packages in 5m

143 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

><EMAIL_ADDRESS>build
> concurrently npm:build:*

/mnt/c/Users/kamek/OneDrive/Documents/Code/Websites/kamek437.github.io/node_modules/yargs/build/index.cjs:1
{const i=new te(t,e,s,de);return Object.defineProperty(i,"argv",{get:()=>i.parse(),enumerable:!0}),i.help(),i.version(),i}),argsert:h,isPromise:f,objFilter:g,parseCommand:o,Parser:fe,processArgv:b,YError:e};module.exports=ue;
                                                                                ^

SyntaxError: Unexpected token ')'
    at wrapSafe (node:internal/modules/cjs/loader:1427:18)
    at Module._compile (node:internal/modules/cjs/loader:1449:20)
    at Module._extensions..js (node:internal/modules/cjs/loader:1588:10)
    at Module.load (node:internal/modules/cjs/loader:1282:32)
    at Module._load (node:internal/modules/cjs/loader:1098:12)
    at TracingChannel.traceSync (node:diagnostics_channel:315:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:215:24)
    at Module.require (node:internal/modules/cjs/loader:1304:12)
    at require (node:internal/modules/helpers:123:16)
    at Object.<anonymous> (/mnt/c/Users/kamek/OneDrive/Documents/Code/Websites/kamek437.github.io/node_modules/yargs/index.cjs:5:30)

Node.js v22.5.1

I am running Arch Linux via WSL2.
ruby -v 3.1.3
node -v Node.js v22.5.1
uname -a Linux Spellbook 6.8.11-locietta-WSL2-xanmod1
Steps To Reproduce
Fork it on github as per guide. git clone your fork of chirpy. cd chirpyfork bash tools/init.sh
Expected Behavior
Site to build and be ready to push to github pages for my site.
Environment

Ruby:
Jekyll:
Chirpy:
ruby 3.3.4 (2024-07-09 revision be1089c8ec) [x86_64-linux]
jekyll 4.3.3
jekyll-theme-chirpy (7.0.1)
node -v v22.5.1

Anything else?
Am I doing something wrong? I really want to use this the design is awesome!

In the beginning, you chose chirpy-starter for "How did you create the site?", but in "Steps To Reproduce", you mentioned using fork. Please do not waste the project maintainers' time and use the Starter to create your site.

I might have missclicked I forked it.


docker sshd cpu 100%

I am suprised that sshd comsumes 100% CPU of my server.

 They are blocked.

Can you explain how this relates to docker? sshd is software installed on your machine, and not installed by docker

@thaJeztah I have never met such a situation before, sshd can run 100% cpu in my machine. I run sshd as  deamon in my docker so that it can keep running after  I logout docker.

@thaJeztah  Two weeks ago, it was normal. But It was stuck when I came back. And other command docker images, docker kill, and docker stop can not be run because they are blocked when I try to stop
my docker.

I run sshd as deamon in my docker so that it can keep running after I logout docker.

Is it running in a container or on the host? We really need more information, because just "sshd uses a lot of CPU", at least;

what platform / distro are you  running docker on
what version of docker do you have installed
how is that sshd running; is it in a container or on the host
steps to reproduce, perhaps log files?

Please keep in mind that the GitHub issue tracker is not intended as a general support forum,
but for reporting bugs and feature requests. For other type of questions, consider using one of;

https://forums.docker.com
the Docker community Slack channel
StackOverflow

@thaJeztah It is running on a container.

The docker is running on ubuntu.

root@iZ2321o1tjkZ:~# uname -a
Linux iZ2321o1tjkZ 3.13.0-32-generic #57-Ubuntu SMP Tue Jul 15 03:51:08 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

version of docker:

root@iZ2321o1tjkZ:~# docker version
Client:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:        Wed Oct 26 21:44:32 2016
OS/Arch:      linux/amd64
Server:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:        Wed Oct 26 21:44:32 2016
OS/Arch:      linux/amd64

sshd is running on docker, I can not run docker ps now. but the last CMD is as following:

CMD ["/root/run.sh"]
run.sh
#!/bin/bash
/usr/sbin/sshd -D
~
4.This is how I run the docker:
docker run -d -p 19701:22 -p 3306-3308:3306-3308 -it

Couple of things I notice;

You're publishing a number of other ports; is something else running in that container, and, are you sure nothing is connected to that sshd possibly causing this issue?
Docker 1.12.3 is not an up to date version of docker; many bugfixes have gone into 1.12.4,.5 and .6, including a security fix for RunC; did you install docker from the ubuntu repository, or using the official binaries from the apt.dockerproject.org repo?
Make sure your kernel is up to date; 3.13.0-32 is an outdated kernel; there has been issues with older kernels that could result in processes hanging (see https://github.com/docker/docker/issues/18180#issuecomment-193708192)

@thaJeztah OK, thanks.

good to hear it's resolved!


Invalid release tag in README: 0.3.13 vs v0.3.13

Seems the latest release is tagged as 0.3.13 when all the documentation states v0.3.13

Fixed. I'll keep the old tag around for now. Good catch :)


sap_hana_install: Support an hdblcm installation check

The resident hdblcm command can be called with --action=check_installation to perform some checking against a property file located in /hana/shared/<SID>/global/hdb/install/support. Its scope is mainly directory and file ownership and permission checking. The role could optionally call this command after the installation, or - by using a tag - just run this check.

Solved in https://github.com/sap-linuxlab/community.sap_install/pull/79 .


Xcode Package Resolution Failed: the package manifest at '/Package.swift' cannot be accessed (/Package.swift doesn't exist in file system)

#Xcode
Version 14.3.1 (14E300c)

Latest release does not contain a /Package.swift file.
https://github.com/madmachineio/mm-sdk/releases/tag/0.5.0


Cannot build on macOS BigSur version11.6

Build Issue
Environment
macOS BigSur version 11.6
Description
I was trying to follow the installation guide.
I can't seem to build it on macOS BigSur.
What I did
First of all, I googled this error.
$ ./meta/bootstrap.sh
[+] Compiling the bootstrap compiler...
ld: dynamic main executables must link with libSystem.dylib for architecture x86_64

https://stackoverflow.com/a/65570573
In accordance with this, fixed.
(meta/bootstrap.sh)
   Darwin)
        cp bootstrap/macos.yasm bootstrap/cupcc.yasm
        yasm -f macho64 -o bootstrap/cupcc.o bootstrap/cupcc.yasm
        ld -o bootstrap/cupcc bootstrap/cupcc.o -macosx_version_min 11.6 -L /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/lib -lSystem
        ;;

Then, I cannnot fix the next error.
$ ./meta/bootstrap.sh
[+] Compiling the bootstrap compiler...
ld: warning: PIE disabled. Absolute addressing (perhaps -mdynamic-no-pic) not allowed in code signed PIE, but used in func_die from bootstrap/cupcc.o. To fix this warning, don't compile with -mdynamic-no-pic or link with -Wl,-no_pie
[+] Creating build/cupcc with bootstrap compiler...
[+] /usr/local/bin/yasm -fmacho64 -o build/cupcc.o build/cupcc.yasm
[+] /usr/bin/ld -o build/cupcc build/cupcc.o -lSystem
ld: library not found for -lSystem
std/common.cup:350:14: Child exited with non-zero status: (1)

I think they changed the default paths ld looks at for newer versions of macOS. I'm still running Catalina because I refuse to upgrade 😅. In any case, you might also need to pass the -L /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/lib flag to ld to get this to work.
Perhaps the easier fix here might be to just use clang for linking as opposed to ld, and it will pass all the correct arguments to the linker call. Changing the linking step to clang XXX.o -o XXX (in both the meta/bootstrap.sh file and compiler/main.cup) seems to do the trick for me, but I'm not sure if that might be different on your version.
Thanks for pointing this out, I didn't know there would be weird stuff with different macOS versions, I'll look into this further, and see if I can push a change to the repo within the next few days that just lets it work out of the box. I think Github Actions has macOS 11, so I might be able to test there.

For the error you have right now at least, since the build/cupcc.o has been generated, you can just link that manually to be able to get an executable (so that you can re-bootstrap your local compiler after changing the command in compiler/main.cup). It's a bit of an annoyance getting the initial workflow up, but after that it's usually fine. You can reach out on Discord if you're still interested in getting it working and want some help: mustafaq#8791

Actually, it was a simple enough fix. I've updated the code to use clang for linking on macOS, and I've also added a test runner for macOS 11 to make sure we don't regress. If you just checkout the newest master, it should work fine now on your machine. Thanks again for taking the time to point out the bug!


Use e.target instead of e.currentTarget

I'm using a validation library which uses $.proxy, and this breaks the payment formattors since they use e.currentTarget. See https://api.jquery.com/event.currentTarget/ for reference.
Is there a particular reason we're using e.currentTarget instead of e.target?

@fishermand46 e.currentTarget is the node for which the handler was attached, not a nested node. For inputs, there probably shouldn't be a difference, but it's not the kind of thing subtle thing we'd like to change at this point if we can avoid it. Can you give a minimal js fiddle example of how this breaks with $.proxy?

Closing for lack of activity. Please recreate with more details if this is still an issue!


add mistral

When running the model according to readme, in docker image amperecomputingai/pytorch:latest, I encounter such an issue:

Click to expand terminal output :)
Loading checkpoint shards: 100%|██████████████████████████████████████████████████████████████████████████████████████████████| 3/3 [00:03<00:00,  1.22s/it]
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.

CAUTION: Remember to compile your model with torch.jit / torch.compile for Ampere optimizations to work.

CAUTION: KernelPageSize is 4 kB, consider using 64k pages.

CAUTION: Transparent_hugepage is set to never. Consider using 'always'.

Intraop parallelism set to 16 threads

Running with PyTorch

  0%|                                                                                | 0/60 [00:00<?, ?it/s]Traceback (most recent call last):
  File "/ampere/aml/natural_language_processing/text_generation/mistral_instruct/run.py", line 53, in <module>
    main()
  File "/ampere/aml/natural_language_processing/text_generation/mistral_instruct/run.py", line 49, in main
    run_pytorch_fp32(**vars(parser.parse()))
  File "/ampere/aml/natural_language_processing/text_generation/mistral_instruct/run.py", line 40, in run_pytorch_fp32
    return run_pytorch(num_runs, timeout, dataset_path, disable_jit_freeze, **kwargs)
  File "/ampere/aml/natural_language_processing/text_generation/mistral_instruct/run.py", line 36, in run_pytorch
    return run_model(run_single_pass, runner, dataset, 1, num_runs, timeout)
  File "/ampere/aml/utils/benchmark.py", line 229, in run_model
    single_pass_func(runner, dataset)
  File "/ampere/aml/natural_language_processing/text_generation/mistral_instruct/run.py", line 15, in run_single_pass
    input_array = dataset.get_input_array()
  File "/ampere/aml/utils/nlp/alpaca_instruct.py", line 42, in get_input_array
    return self.data[self.__current_sample]
KeyError: 0
  0%|                                                                                | 0/60 [00:00<?, ?it/s]

@dkupnicki any updates on the issue Marcel seen?

I don't see this issue when running normally, but I was able to get this error message when I manually corrupted the dataset file. Maybe try redownloading it and see if it helps?

requirements:
transformers==4.36.0
change the alpaca_data.json link to: (or add as an optional)
https://raw.githubusercontent.com/tatsu-lab/stanford_alpaca/main/alpaca_data.json


WIP: remove Loggo.events and change config to kwargs

I also split up the massive test file

Codecov Report

Merging #81 into master will decrease coverage by 1.12%.
The diff coverage is 96.92%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   92.06%   90.94%   -1.13%
==========================================
  Files           2        2
  Lines         315      287      -28
==========================================
- Hits          290      261      -29
- Misses         25       26       +1

Impacted Files
Coverage Δ

loggo/loggo.py
90.87% <96.92%> (-1.14%)
:arrow_down:

Continue to review full report at Codecov.

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


GJSS TEST ISSUE #2

Topic Link

Description

Possible solutions

Additional information

@adobe export issue to Jira project COMDOX as Story