Hide rich text toolbar until the user focuses the editor

Is your proposal related to a problem?
For pages that have a lot of rich text fields (e.g. successive paragraph blocks), the toolbar can be quite distracting, even with the changes from #4997.
Describe the solution you'd like
From @benenright in https://github.com/wagtail/wagtail/issues/4997#issuecomment-577705230,

Hide the toolbar until user has focused on the field. Ideally we will create a space for it above the field text so that it doesn't jarringly push the content down when it appears on focus.

I think it would also be worth specifying:

This might be better done with opacity so we can animate it easily, and so screen reader users can still go through the toolbar (they can skip it as a single item if desired)
There should be space for the toolbar to appear without reflowing the content (if we hide the toolbar with opacity: 0 this should be very straightforward)
Challenging – end users shouldn’t have to guess where to click to focus the editor – ideally the area where the toolbar normally sits should be clickable so focus moves to the editor.
There’s already a class Draftail-Editor--focus which we can use to implement this.

As of #8806, we now have a floating toolbar, which achieves the same thing as what this issue intended.


Exception from Tracker recompute function: too much recursion  especially in firefox

Thank you very much for this wonderful web app you created.
I do not why iam getting this error in console(client) especially for firefox browser and not getting it  in chrome...
Exception in callback of async function: TypeError: Messages.clearSeen is not a function
at Router._filters.clearSeenMessages (http://localhost:3010/lib/router/filters.js?

Hey, @bogere! Telescope has changed a lot in the past months, and there is a new React-based version, Telescope Nova (intro video).
The Legacy version is no longer supported, I'm closing this issue. I invite you to check the new version, it's awesome!
Happy hacking! 🚀
Ps: by the way, don't hesitate to join the Slack to ask any question, you are welcome! 👍


Added geospatial samples for V3

Pull Request Template
Added in new geospatial samples for v3 following the v2 geospatial samples.
Description
Currently, there exists no new geospatial samples for v3. I added those in with samples of using ST_DISTANCE, ST_WITHIN and ST_ISVALID.
Type of change
Please delete options that are not relevant.

[] New feature (non-breaking change which adds functionality)
[] This change requires a documentation update

Closing issues
To automatically close an issue: closes #IssueNumber

Thank you @StefArroyo


blob.ParseURL() parses wrong for azurite url with custom hostname

Bug Report
Using the blob.ParseURL() in azblob return wrong information if azurite is used with a custom hostname e.g. in a docker compose setup.
Used version:
github.com/Azure/azure-sdk-for-go/sdk/storage/azblob v1.5.0
package main

import (
	"fmt"
	"github.com/Azure/azure-sdk-for-go/sdk/storage/azblob/blob"
)

func main() {
	parts, err := blob.ParseURL("http://azurite:10000/devstoreaccount1/container/test.csv")
	if err != nil {
		fmt.Println("could not parse url")
		return
	}
	fmt.Printf("Host: %s, Container: %s, Blob: %s", parts.Host, parts.ContainerName, parts.BlobName)
}

Outputs:
Host: azurite:10000, Container: devstoreaccount1, Blob: container/test.csv
instead of
Host: azurite:10000, Container: container, Blob: test.csv
This was fixed in the .NET sdk in https://github.com/Azure/azure-sdk-for-net/pull/13124
I believe it is also the root cause for https://github.com/Azure/azure-sdk-for-go/issues/23496

Hi @PaulKC ,
Thank you for reporting this, we are looking into this issue and will get back to you


wheres the code

Hi, thanks for your comment. As we stated on the main page, we will release the code after the CVPR2024 paper submission deadline (Nov. 17th) since we are all busy working for the deadline now. Sorry for the inconvenience.


Does not show swedish translation after upgrading to 1.4.5

After I upgraded GoAccess from 1.4.3 to 1.4.5, it shows GoAccess in english not swedish.

I fixed it my self, I am running goacces as a service in debian. Just adding "export LANG=sv_SE.UTF-8" in the service script.

I fixed it my self, I am running goacces as a service in debian. Just adding "export LANG=sv_SE.UTF-8" in the service script.

Thanks for pointing that out. Feel free to reopen it if needed.

Thanks for pointing that out. Feel free to reopen it if needed.


refactor to correctly use distributed storage

distributed storage is required pass files amount the following LightningWork components:

4 tracer scripts
fiftyone
cvat

using drive.


fix bias param from float to string

As seen in HF docs and in HF repo, the value for bias used for Lora tuning should be a string and not a float.

Thank you Anh!


Ubuntu 18.04

Is ubuntu 18.04 supported and if not when can support be expected?

Grumpy
Once it is a lts it will be supported, I have been running it for months. There are no issues. Vega users will have to run the 4.17 kernel or a 4.15 patched with KFD support. As of march 26 the kfd patches for the Vegas an kfd were accepted into the mainline. Ill already have a ubuntu live 18.04 I have been using for months.
https://github.com/RadeonOpenCompute/ROCm/issues/361
Installer for full desktop or server system Fluxbox xinit and xterm, LXDE. Lubuntu Core with the Midori Browser, Lubuntu (standard desktop), Xubuntu, Xubuntu (standard desktop), Ubuntu MATE,Install Ubuntu (standard desktop),Ubuntu Gnome,Kubuntu,Server
**Real-time Priority running example auto start for cloud and or mining

So, the 4.17 kernel will support Vega + ROCm, contradicting statements on Phoronix?
There was a claim dating from January that it isn't ready yet, repeated just a few days ago:
https://phoronix.com/scan.php?page=news_item&px=AMDKFD-Rest-Of-dGPU-4.17

It would be nice to have some official statement support from AMD for 18.04. I hope the team does try an install and testrun internally.

We are not allowed by our legal team to make these statements.


Support Net8

When migrating to net8, instantiation of the OnPremiseClient class results in:
Could not load type 'System.IdentityModel.Tokens.SecurityKeyType' from assembly 'System.ServiceModel.Security, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

I've built a simple test app on .NET 8 and I can run this library without any changes. Can you share a simple sample project to help me reproduce the error?

My bad, you are right it has nothing to do with .Net 8, but with package incompatibilities: I've migrated to .net 8 and updated all packages at the same time, hence the confusion.
I narrowed my complex solution to a single project in order to identify the incompatibility:
Here's what works:

If I update System.ServiceModel.Http && System.ServiceModel.Security to their latest nuget version:
<ItemGroup>
	<PackageReference Include="System.ServiceModel.Http" Version="6.2.0" />
	<PackageReference Include="System.ServiceModel.Security" Version="6.0.0" />
	<PackageReference Include="Data8.PowerPlatform.Dataverse.Client" Version="2.4.1" />
</ItemGroup>

It then fails to load System.IdentityModel.Tokens.SecurityKeyType.

It seems you're updating some packages but not all. If you also update System.ServiceModel.Federation to 6.2.0 this should resolve the problem.

Excellent, adding the System.ServiceModel.Federation package to my narrowed project solves the problem!
<ItemGroup>
	<PackageReference Include="System.ServiceModel.Federation" Version="6.2.0" />
	<PackageReference Include="System.ServiceModel.Http" Version="6.2.0" />
	<PackageReference Include="System.ServiceModel.Security" Version="6.0.0" />
	<PackageReference Include="Data8.PowerPlatform.Dataverse.Client" Version="2.4.1" />
</ItemGroup>


🔀 :: (#170) 현재 모집 중인 모집의뢰서 목록 API 수정

close #170

Codecov Report

Merging #172 (fff62c5) into main (be42549) will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main     #172      +/-   ##
============================================
+ Coverage     32.32%   32.34%   +0.02%
  Complexity      203      203
============================================
  Files           122      122
  Lines          1420     1419       -1
  Branches         10       10
============================================
  Hits            459      459
+ Misses          960      959       -1
  Partials          1        1

Impacted Files
Coverage Δ

...t/CurrentRecruitmentInfoListForStudentRequest.java
0.00% <0.00%> (ø)

...domain/recruitment/service/RecruitmentService.java
1.42% <0.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


Convert FactoryGirl to FactoryBot

Bring all non-standard references to Factory to FactoryGirl
Convert From FactoryGirl to FactoryBot
Resolve deprecations in prep for FactoryBot 5.0

Thinking on this more. I think it is better to do this the other way. This PR touches more files and would affect almost all other PRs. So it is better to get this one merged first so we don't get stuck adding more and more to it.


Internal IBM link in example App home page

Link referring to an internal IBM link
https://github.com/IBM/wxflows/blob/16b1f6f4ff370ced17ec3002c9a9453cfad5b9ab/examples/rag-question-answer/app/src/App.jsx#L55-L57
May just need to change github.ibm.com to github.com

Looks like commit eed2c03281d3c0e058bc33032ed284c3625b380c reverted this fix here


Buch of errors [Windows 10, Python 3.10]. final_wave = generated_waves[0], IndexError: list index out of range

Hi, I did everything fine and still get errors. I tried reinstalling from zero 3 times, cuda is 11.8 and I have ffmpeg the latest version. I updated pip, I updated Anaconda. My GPU driver is updated. I'm using Windows 10. I also tried to purge the cache without results. I installed Python 3.10 in the environment.
At first, I thought it was a file issue, so I tried a bunch of different allowed formats, but after the second re-installation, I just tried the sample audio you provided and still got exactly the same errors. I installed all using the requirements file. I also tried on my last attempt to move the git folder from admin driver C to regular one E, didn't help. I tried to run it with administration rights and didn't help.
This is all the errors:
"
C:\Users{username}\anaconda3\envs\f5\lib\site-packages\transformers\models\whisper\generation_whisper.py:496: FutureWarning: The input name inputs is deprecated. Please make sure to use input_features instead.
warnings.warn(
You have passed task=transcribe, but also have set forced_decoder_ids to [[1, None], [2, 50359]] which creates a conflict. forced_decoder_ids will be ignored in favor of task=transcribe.
C:\Users{username}\anaconda3\envs\f5\lib\site-packages\transformers\models\whisper\modeling_whisper.py:599: UserWarning: 1Torch was not compiled with flash attention. (Triggered internally at ..\aten\src\ATen\native\transformers\cuda\sdp_utils.cpp:455.)
attn_output = torch.nn.functional.scaled_dot_product_attention(
Passing a tuple of past_key_values is deprecated and will be removed in Transformers v4.43.0. You should pass an instance of EncoderDecoderCache instead, e.g. past_key_values=EncoderDecoderCache.from_legacy_cache(past_key_values).
The attention mask is not set and cannot be inferred from input because pad token is same as eos token. As a consequence, you may observe unexpected behavior. Please pass your input's attention_mask to obtain reliable results.
Traceback (most recent call last):
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\queueing.py", line 536, in process_events
response = await route_utils.call_process_api(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\route_utils.py", line 322, in call_process_api
output = await app.get_blocks().process_api(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\blocks.py", line 1935, in process_api
result = await self.call_function(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\blocks.py", line 1520, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio_backends_asyncio.py", line 2441, in run_sync_in_worker_thread
return await future
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio_backends_asyncio.py", line 943, in run
result = context.run(func, *args)
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\utils.py", line 826, in wrapper
response = f(*args, **kwargs)
File "E:\F5-TTS\gradio_app.py", line 66, in infer
final_wave, final_sample_rate, combined_spectrogram = infer_process(
File "E:\F5-TTS\model\utils_infer.py", line 214, in infer_process
return infer_batch_process(
File "E:\F5-TTS\model\utils_infer.py", line 307, in infer_batch_process
final_wave = generated_waves[0]
IndexError: list index out of range
"
btw "generated_waves" is empty if it's important. I know it because, on my very first attempt, I was trying to use GPT, and it suggested printing "generated_waves" to check if it's really empty, so I just added a print line before the error and yes it was empty.

seems a failure with whisper asr pipeline.
have you tried with input manual the reference text for prompt audio?

seems a failure with whisper asr pipeline. have you tried with input manual the reference text for prompt audio?

Tried now, errors:
"
Traceback (most recent call last):
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\queueing.py", line 536, in process_events
response = await route_utils.call_process_api(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\route_utils.py", line 322, in call_process_api
output = await app.get_blocks().process_api(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\blocks.py", line 1935, in process_api
result = await self.call_function(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\blocks.py", line 1520, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio_backends_asyncio.py", line 2441, in run_sync_in_worker_thread
return await future
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio_backends_asyncio.py", line 943, in run
result = context.run(func, *args)
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\utils.py", line 826, in wrapper
response = f(*args, **kwargs)
File "E:\F5-TTS\gradio_app.py", line 66, in infer
final_wave, final_sample_rate, combined_spectrogram = infer_process(
File "E:\F5-TTS\model\utils_infer.py", line 214, in infer_process
return infer_batch_process(
File "E:\F5-TTS\model\utils_infer.py", line 307, in infer_batch_process
final_wave = generated_waves[0]
IndexError: list index out of range
"
I tried using whisper manually, and it works fine. Gave me the text of the "country.flac" with 100% accuracy.

Hi @kamineko16 some possible solutions:

try lower gradio version if >=5, pip install gradio==4.44.1
check for audio format, should with pcm_s16le e.g.. will have

visuals like this if successfully uploaded
maybe force re-pull the repo?

Hi @SWivid , answers to your suggestions:

It is already 4.44.1, so I guess it installed with the requirements as 4.44.1.

I used the sample from the F5-TTS sample folder, and I guess it is in the right format? However, I tried what you said anyway.
Here are pictures of both:
Original Flac file:

Same audio, but Wav file with pcm_s16le:

Errors:
"
C:\Users{username}\anaconda3\envs\f5\lib\site-packages\transformers\models\whisper\generation_whisper.py:496: FutureWarning: The input name inputs is deprecated. Please make sure to use input_features instead.
warnings.warn(
You have passed task=transcribe, but also have set forced_decoder_ids to [[1, None], [2, 50359]] which creates a conflict. forced_decoder_ids will be ignored in favor of task=transcribe.
C:\Users{username}\anaconda3\envs\f5\lib\site-packages\transformers\models\whisper\modeling_whisper.py:599: UserWarning: 1Torch was not compiled with flash attention. (Triggered internally at ..\aten\src\ATen\native\transformers\cuda\sdp_utils.cpp:455.)
attn_output = torch.nn.functional.scaled_dot_product_attention(
Passing a tuple of past_key_values is deprecated and will be removed in Transformers v4.43.0. You should pass an instance of EncoderDecoderCache instead, e.g. past_key_values=EncoderDecoderCache.from_legacy_cache(past_key_values).
The attention mask is not set and cannot be inferred from input because pad token is same as eos token. As a consequence, you may observe unexpected behavior. Please pass your input's attention_mask to obtain reliable results.
Traceback (most recent call last):
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\queueing.py", line 536, in process_events
response = await route_utils.call_process_api(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\route_utils.py", line 322, in call_process_api
output = await app.get_blocks().process_api(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\blocks.py", line 1935, in process_api
result = await self.call_function(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\blocks.py", line 1520, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio_backends_asyncio.py", line 2441, in run_sync_in_worker_thread
return await future
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\anyio_backends_asyncio.py", line 943, in run
result = context.run(func, *args)
File "C:\Users{username}\anaconda3\envs\f5\lib\site-packages\gradio\utils.py", line 826, in wrapper
response = f(*args, **kwargs)
File "E:\F5-TTS\gradio_app.py", line 66, in infer
final_wave, final_sample_rate, combined_spectrogram = infer_process(
File "E:\F5-TTS\model\utils_infer.py", line 214, in infer_process
return infer_batch_process(
File "E:\F5-TTS\model\utils_infer.py", line 307, in infer_batch_process
final_wave = generated_waves[0]
IndexError: list index out of range
"

I already tried it, when I said I reinstalled 3 times, what I meant was including the repo. I wiped out everything and downloaded everything again from zero 3 times. The last time was on drive E (current drive), in case it's an admin privilege issue, but it didn't help at all.

@kamineko16 it's very weird we haven't met this before, seems you're being the first one encountering this, sadge
will inference-cli.py work from your side?

Hi @kamineko16 , I have no idea of what's going wrong, as I could not reproduce the error.
You could just leave it open for somedays, see if someone also encounter this and are able to fix.

Hi @SWivid

@SWivid huh weird indeed, it's indeed working with inference-cli.py , so now I know everything is basically fine and the issue is with gradio_app.py ? Should I close this case or you want to check it more to understand why it's not working specifically with gradio_app.py ?

No. Problem is with whisper or transformer's pipeline. I have not tested on gradio, but my application is producing same warning. Though it generates output

will close this issue, feel free to open if further questions~

Any fix for this? Facing same issue.

Any fix for this? Facing same issue.

Idk... I used the suggestion here which worked.

@kamineko16 which suggestion? Using CLI?

@kamineko16 which suggestion? Using CLI?

Yep. I also gave gpt the cli code and regular code with the interface and asked it to combine them. GPT surprisingly managed to combine the two, to some degree, so you have cli with interface or whatever, but it worked.


[Bug] Render Issues on Indium 1.0.18

After updating to latest Indium 1.0.18, some chunks and subchunks started to unload or not fully load all layers
Proofs:

Mod List: (With Fabric Loader 0.14.21)

Can you send your latest.log file from when these errors occurred?

Can you send your latest.log file from when these errors occurred?

Sure thing!
https://paste.gg/p/anonymous/d48c761b58a64c8f831289471e3c24a1

This is a problem with Continuity, see https://github.com/PepperCode1/Continuity/issues/253

I have just released Continuity 3.0.0-beta.3 for 1.19.3, 1.19.4, and 1.20.1 that have fixed this incompatibility. Note that the 1.19.4 and 1.20.1 versions require Fabric API >=0.86.0, which is currently the latest version.


undefined result on get-github-info call

Affected Packages
get-github-info

Problem
When configuring changeset, if you pass in the repo name of the wrong format e.g https://github.com/user/repo instead of just user/repo it will cause the call to the Github API to return undefined and therefore causing Cannot read property 'author' of undefined. This error message can be misleading/confusing and so I think we need to handle that and show a nicer message.
There is an actual github action with that error of the repo that I'm working on - https://github.com/sajari/sdk-react/runs/1668436313?check_suite_focus=true

Proposed solution
The solution is to add a more specific/stricter check for request.repo field to make sure that it follows the format (userOrOrg/repoName)

I still have this issue. Is there something to do?

I still have this issue, but i'm using the correct format "user/repo" for the name . Is there something to do?

Can't really recall since it's been so long 😄, is your repo private or public?

My repo is public, My configuration is:
  "changelog": [
    "@changesets/changelog-github",
    { "repo": "equinusocio/scrolles" }
  ],

PS: I add to add a local .env file with the GITHUB_TOKEN and a personal access token.

Ok nevermind, I tried again and it seems to work, probably was my fault on generating the token or miswrite the repo name dunno.


No Release for Node 20 Upgrade

Describe the bug
When we use this github action, we have the following warning: Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/. Node was updated back in November with this commit but no new release was created. Please create a release with that change.
To Reproduce
Steps to reproduce the behavior:
Use the latest version in a github workflow and run that workflow.
          - name: Terraform min/max versions
            id: minMax
            uses<EMAIL_ADDRESS>
Expected behavior
No warning message that Node is outdated

this is now resolved, apologies!


MAINT: AttributeSignal should run subscriptions

Description
Noticed this while making a typhon widget for an AttributeSignal. When you call .put on your AttributeSignalit never runs it's subscriptions. I realize we won't get updated when the underlying attribute gets swapped out but it is nice to get updated when somebody writes using the AttributeSignal interface.
Question
Was also curious how people felt about the ability to add a little metadata to these signals. (I'd do this in a separate PR, but since you have read this far). The idea is that we might want to add a little extra data to .describe but not subclass.
class MyDevice(Device):
        calculated_width = Cpt(AttributeSignal, attr='width_calc', desc={'units': 'mm'})

Looks like caproto / areadetector related test failure, don't think related to this PR

Sorry about that caproto/AD failure... I'm honing in on a fix.


[Draft] react-custom-notion official document

Description of Changes
Here is reference: #25

support 'custom your created block' block

Please add the block you created to replace the 'crate' block.

ex) add react-katex dependency to support equation block
-> add custom solution for rendering 'your custom block'

Review point

ex) Instead of adding the react-katex dependency, should I create a custom solution specifically for this project?
-> Should we integrate an existing solution or use a custom implementation for 'your custom block'?

To reproduce

npm run story:start
click 'your created block' on the Storybook left side panel

Screenshot

A screenshot of your created block being rendered in Storybook.

Review Guide
Reviews are conducted based on priority levels, such as p0, p1, p2, p3, p4, and p5.
p0 ~ p2: If the author decides not to reflect a review for p0 to p2, it signals that a proper discussion with the reviewer is
necessary. It is expected that the review will be resolved either through incorporating the feedback or through further discussion.
p3: indicates that the reviewer has identified a significant issue, but either lacks a clear solution or the comment lacks sufficient context. Further explanation or additional discussion on the reviewer's concerns is needed.
p4, p5: p4 and p5 suggest low priority, and if the author does not deem them important, these comments can be disregarded.

##KR
와우 퀄리티가 장난 아닌데요?? 좋습니다 ㅎㅎㅎ
##EN
you are good!


How to add validations on each step

How to add validations on each step and make user not goto the next step until those validations are fulfilled.
Can you please build or make tutorials for these?

see my answer to the #69 - you can use any standard validation package or form package that provides validators. If you combine it with global state manager, you would have complete state (from all steps) validated and available to submit


inheriting the release profile can break mvn release:perform

After adding the progressive-organization pom as parent, mvn release:perform would fail with
--- maven-gpg-plugin:1.6:sign (default) @ parent ---
[INFO] gpg: no default secret key: No secret key
[INFO] gpg: signing failed: No secret key
[INFO] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-gpg-plugin:1.6:sign (default) on project parent: Exit code: 2 -> [Help 1]
Could
<plugin>
      <groupId>org.apache.maven.plugins</groupId>
            <artifactId>maven-gpg-plugin</artifactId>
            <executions>
              <execution>
                <goals>
                  <goal>sign</goal>
                </goals>
              </execution>
            </executions>
          </plugin>

be changed to add
<inherited>false</inherited>

or what would other ways be to not have the unwanted behaviour?
e.g. we don't use the maven-gpg-plugin in our build.

For the moment I've solved it with
<plugin>
         <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-release-plugin</artifactId>
                <version>2.5.1</version>
                <configuration>
                    <arguments>-Dgpg.skip=true</arguments>
                    <autoVersionSubmodules>true</autoVersionSubmodules>
                </configuration>
    </plugin>

but still thinking that the gpg sign should not be inherited

Do you know if inherited set to false can be overridden to true again? All the consuming projects I am working with are releasing to the Central Repository and need gpg as part of the release... so if I take it out I have to put it back in again.
If you skip it like you do above thats a good idea imho. Just take the version out of the config..

so there is a usecase for inheriting it. skipping it works fine. don't think you can override it to true again. will close this issue.


Few-shot examples with energy conservation are used in prompts when chatting ungrounded with the models

Bug Details
When chatting ungrounded, directly with the model, it will occasionally give responses based on the few-shot examples used for the grounded chats.
Steps To Reproduce

Start an ungrounded chat
Send the prompt "test test"

What is the expected behavior?
The model would typically reply with something like "it appears you are trying to test something. If you give me more context I can help"
Screenshots

Information Assistant details
Please provide the following details. You can simply include a screenshot of your Info panel as well.

GitHub branch: main
main
Version or Latest commit:
v1.1.1 commit e60ed3e6dced43796a96d9b2115b4c283872a0eb
What region is your Azure Open AI Service in?
EastUS
What ChatGPT model are you using?
gpt-4o
model name: (i.e. gpt-3.5-turbo, gpt-4)
gpt-4o
model version: (i.e. 0613)
2024-05-13
What embeddings model are you using?
BAAI/bge-small-env-v1.5

Additional context
Add any other context about the problem here.
If the bug is confirmed, would you be willing to submit a PR?

[X] Yes
[ ] No

@nhwkuhns , I have put in a PR for this. It will get updated in our next release.


To build binding contexts, the ActivatableBindingsProvider must pass parent contexts to the context factory

The class ActivatableBindingsProvider must always pass the parent binding context to the implementation of GetsBindingContext.  This is new required functionality now that #25 has been completed.
In doing this, it should always pass the binding context associated with the parent HTML element for the context which is currently being processed.

No longer required - instead of passing the parent context, the model context is now created externally to the binding context, and that is passed instead.


[Event Request] - New event OnBeforeRecallIssuedBill inside codeunit 12170 "Recall Customer Bill"

Hello, please consider adding the following event to Codeunit 12170 "Recall Customer Bill"
    [IntegrationEvent(false, false)]
    local procedure OnBeforeRecallIssuedBill(var IssuedCustomerBillLine: Record "Issued Customer Bill Line"; var IsHandled: Boolean)
    begin
    end;

and call it inside the "RecallIssuedBill" procedure:
    procedure RecallIssuedBill(var IssuedCustomerBillLine: Record "Issued Customer Bill Line")
    var
        IssuedCustomerBillLine2: Record "Issued Customer Bill Line";
        IssuedCustomerBillHeader: Record "Issued Customer Bill Header";
        BillPostingGroup: Record "Bill Posting Group";
        BalanceAccountNo: Code[20];

        **IsHandled: Boolean;**

    begin

        **IsHandled := false;
        OnBeforeRecallIssuedBill(IssuedCustomerBillLine, IsHandled);
        if IsHandled then
            exit;**

        SalesSetup.Get();
        IssuedCustomerBillHeader.Get(IssuedCustomerBillLine."Customer Bill No.");
        GetBillCode(IssuedCustomerBillHeader."Payment Method Code");
...
...

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52146.


OpenVPN client export tool does not export latest server settings

On OPNsense v16.7.9, generating the OpenVPN client configuration using the export tools does not seems to use the actual/latest server settings.
Steps to reproduce

Create an OpenVPN server.
Export client configuration using the export tool
Change a setting of the OpenVPN server.
Re-export the client configuration using the export tool.

Expected result
The second generated .ovpn file obtained from step 4 should reflect any changes you have performed on step 3.
Actual result
The second generated .ovpn file obtained from step 4 is the same as the one obtained from step 2.
Note
The following scenario where no export is done prior to change server settings works:

Create an OpenVPN server.
Change a setting of the OpenVPN server.
Export the client configuration using the export tool.

Oh boy, the code hands out file names that are always the same, probably not wiping a file somewhere. Which exact export type are you using?

Both (config only and config+secret archive) have the issue.

According to my knowledge of this kind of problem I know why the following is "normal" (considering the issue) but I'll also report the following (in case anyone get the same issue):
After deleting an OpenVPN server and creating a new one: the exported file is the one for the deleted server.

Have you tried another browser? When I change the cipher in a server and export the same user again, all looks good on my end.
To be sure I inspected the code for the "file only" export and it doesn't look like these type of files are synced to disk, so a caching issue on the server end seems very unlikely in this case.

timeout


Add a license

Hi, I'm currently in the process of adding license info to all the games in Erwin's Arduboy repo so people can easily filter out proprietary games. Would you be willing to add an open source license to your project so I can tag it as an open source game in the repo?
Thanks!

Hi Alex,
yes sure. Do you need me to do it? Or can you add it yourself?
Rem.

On Feb 12, 2018, at 4:27 PM, Alex Gleason<EMAIL_ADDRESS>wrote:
Hi, I'm currently in the process of adding license info to all the games in Erwin's Arduboy repo so people can easily filter out proprietary games. Would you be willing to add an open source license to your project so I can tag it as an open source game in the repo?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @rveilleux, if you add a file to the root of this repo (usually called LICENSE) containing the text of the license you'd like to use, I can update Erwin's repo to reflect the license you chose. Here are some common license options: https://choosealicense.com/
Alex

Hello again Alex,
the deed is done: LICENSE.txt added. It is an MIT License.
Thank you!
On Mon, Feb 12, 2018 at 6:20 PM, Alex Gleason<EMAIL_ADDRESS>wrote:

Hi @rveilleux https://github.com/rveilleux, if you add a file to the
root of this repo (usually called LICENSE) containing the text of the
license you'd like to use, I can update Erwin's repo to reflect the license
you chose. Here are some common license options:
https://choosealicense.com/
Alex
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rveilleux/ard-drivin/issues/1#issuecomment-365097054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGYfqSW9WKiXYkfW243pg0uyva54AktSks5tUMclgaJpZM4SC2Xk
.

Awesome, thanks!


ARROW-13594: [CI] Re-enable turbodbc integration builds

@github-actions crossbow submit turbodbc

I tried doing the same here: https://github.com/apache/arrow/pull/12144, but so the issue is that we changed some APIs again, and so the build is still failing (unless that was changed the last days)

Closing in favor of #12144


The embedded video is marked private on this page

[Enter feedback here]

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

ID: 8c161a2e-d9f4-466f-5a63-5e25648a1779
Version Independent ID: 6032f644-b69d-622f-73df-bacebdfc4509
Content: Create a multi-instance Universal Windows App - UWP applications
Content Source: windows-apps-src/launch-resume/multi-instance-uwp.md
Product: uwp
Technology: processes-and-threading
GitHub Login: @alvinashcraft
Microsoft Alias: aashcraft

Thanks for submitting your feedback. I went ahead and created an internal work item to resolve this doc bug. I found the updated embed URL for the video and merged a PR to fix the page. The changes should be live today.
Though the Issue will be marked as Closed, we will monitor any replies that come afterwards so treat this like an open Issue. Many thanks!


Error with Ipopt and specification of lb, ub

[a75be94c] GalacticOptim v2.4.0
[b8f27783] MathOptInterface v1.1.1
[b6b21f68] Ipopt v1.0.2

julia> using GalacticOptim, ForwardDiff

julia> rosenbrock(x, p) =  (p[1] - x[1])^2 + p[2] * (x[2] - x[1]^2)^2
rosenbrock (generic function with 1 method)

julia> x0 = zeros(2)
2-element Vector{Float64}:
 0.0
 0.0

julia> _p  = [1.0, 100.0]
2-element Vector{Float64}:
   1.0
 100.0

julia> f = OptimizationFunction(rosenbrock, GalacticOptim.AutoForwardDiff())
(::OptimizationFunction{true, GalacticOptim.AutoForwardDiff{nothing}, typeof(rosenbrock), Nothing, Nothing, Nothing, Nothing, Nothing, Nothing}) (generic function with 1 method)

julia> prob = GalacticOptim.OptimizationProblem(f, x0, _p, lb = zeros(2), ub=[1.1, 1.1])
OptimizationProblem. In-place: true
u0: 2-element Vector{Float64}:
 0.0
 0.0

julia> using Ipopt

julia> optimizer = Ipopt.Optimizer
Ipopt.Optimizer

julia> # Choose a relaxation method
       opt = GalacticOptim.MOI.OptimizerWithAttributes(Ipopt.Optimizer, "print_level"=>0)
MathOptInterface.OptimizerWithAttributes(Ipopt.Optimizer, Pair{MathOptInterface.AbstractOptimizerAttribute, Any}[MathOptInterface.RawOptimizerAttribute("print_level") => 0])

julia> sol = solve(prob, opt)
ERROR: UndefVarError: SingleVariable not defined
Stacktrace:
 [1] __solve(prob::OptimizationProblem{true, OptimizationFunction{true, GalacticOptim.AutoForwardDiff{nothing}, typeof(rosenbrock), Nothing, Nothing, Nothing, Nothing, Nothing, Nothing}, Vector{Float64}, Vector{Float64}, Vector{Float64}, Nothing, Nothing, Nothing, Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}}}, opt::MathOptInterface.OptimizerWithAttributes; maxiters::Nothing, maxtime::Nothing, abstol::Nothing, reltol::Nothing, kwargs::Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}})
   @ GalacticOptim ~/.julia/packages/GalacticOptim/1ht5p/src/solve/moi.jl:145
 [2] __solve
   @ ~/.julia/packages/GalacticOptim/1ht5p/src/solve/moi.jl:135 [inlined]
 [3] #solve#480
   @ ~/.julia/packages/SciMLBase/GW7GW/src/solve.jl:3 [inlined]
 [4] solve(::OptimizationProblem{true, OptimizationFunction{true, GalacticOptim.AutoForwardDiff{nothing}, typeof(rosenbrock), Nothing, Nothing, Nothing, Nothing, Nothing, Nothing}, Vector{Float64}, Vector{Float64}, Vector{Float64}, Nothing, Nothing, Nothing, Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}}}, ::MathOptInterface.OptimizerWithAttributes)
   @ SciMLBase ~/.julia/packages/SciMLBase/GW7GW/src/solve.jl:3
 [5] top-level scope
   @ REPL[10]:1

I have just registered a new version https://github.com/JuliaRegistries/General/pull/57740, that should probably fix this since the MOI wrapper was recently updated to support v1

Just tried this in the new version and it is working indeed.


Tinkerstool bulk crafting recipe only yields one compressed.

Harrison attempted to craft 500 cyl crystal into 150 crystcyl using the tinkerstool bulk crafting recipe and did not receive the supposed 150 crystcyl.


add parameter constraints to Design constructor, and remove all specified constraints

The current way of giving constraints to the Design violates the DRY principle, since we make the user specify the type of constraint (e.g., StrandConstraint, DomainPairConstraint, ComplexConstraint) when it is created, and then again in the constructor by using a named parameter indicating the type of constraint, e.g.
design = Design(strand_constraints = [sp_constraint1, sp_constraint2],
                domain_pair_constraints = [dp_constraint1, dp_constraint2]),
                complex_constraints = [c_constraint1, c_constraint2])

It would be better for the Design constructor to use the type of each constraint to do this classification itself, rather than forcing the user to do it, so they can simply pass in a list of all the constraints:
design = Design(constraints = [sp_constraint1, sp_constraint2, dp_constraint1, dp_constraint2, c_constraint1, c_constraint2])

Add a constructor to Design, and remove __post_init__, by changing it to __init__, as follows:
def __init__(self, strands: Iterable[Strand[StrandLabel, DomainLabel]], constraints: Iterable[Constraint] = ()):
    self.strands = list(strands)
    for constraint in constraints:
        if isinstance(constraint, DomainConstraint):
            self.domain_constraints.append(constraint)
        elif isinstance(constraint, StrandConstraint):
            self.strand_constraints.append(constraint)
        # ...

    # rest of __post_init__ below here


Please update this for Swift 4 on iOS 11

Thanks for making this, can you update it for swift 4 and iOS 11?

@clipartinc I've updated the code for Swift 4 https://github.com/digitallysavvy/RecordButton

Updated now :) thanks


Browser on smart TV?

Hi,
First of all thaks a lot for sharing your work. I had immediate success running it with a mac mini and a ps4 controller.
I was wondering whether it would be possible to use it on a browser on a smart tv.
If so you could have the teensy just being connected permanently to the back of the TV and make music instead of watching TV or playing games. Even with a treated studio it is quite nice to make some music just using a controller while sitting on the couch.
As I setup everything just yestrerday I was wondering whether it is possible to stream the audio to a connected bluetooth receiver. If not I suppose the audiostream can be routed there inside the OS of the host.
Oh also I was wondering whether there is any other way to get the audio when connected to a mobile running android. Did anyone try to connect devices to the teensys usb host port (the solderpads on top of the mcu)?

Hi, whether it works on a smart TV would come down to what browser is on it. It would need to be based on a relatively up to date version of Chrome. The TV platform would also need to support serial over USB, or if it doesn't (in particular it won't if it is based on Android) then the WebUSB fallback might work.
I'm not sure how the audio works on a system like that, but if your Bluetooth output is set as the default if should work automatically. If the TV platform is Android based though, the audio will probably not work at all, unless Android TV handles things differently from the phone versions.
I think it might be possible to get audio directly off the Teensy if you connect up the right hardware, but it might not be available at all in the headless firmware. You'd have to ask around on Discord for more info.


KEYBOARD - make keyboard.js library working fine + remove CTRL + work only in a context + contextualized shortcuts - test and fix **Win shortcuts

https://github.com/RobertWHurst/KeyboardJS/issues/105#event-818914844


Transcription Failed Error

I get transcription failed error. I have installed the requirements.txt file and entered my API key both with and without the <>'s to test.

The API key should be enclosed in quotations like:
OPENAI_API_KEY = ‘sk-key’
Let me know if that fixes it.
sorry if that wasn’t clear, I’ll update the readme with an example

Unfortunately that did not work. I'm still receiving the same error.
On Sat, Jun 10, 2023 at 8:36 PM Ethan @.***> wrote:

The API key should be enclosed in quotations like:
OPENAI_API_KEY = ‘sk-key’
Let me know if that fixes it.
sorry if that wasn’t clear, I’ll update the readme with an example
—
Reply to this email directly, view it on GitHub
https://github.com/e-johnstonn/SalesCopilot/issues/1#issuecomment-1585874617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4YGQDTTFTXW5RC2CIL6WULXKUHH3ANCNFSM6AAAAAAZB7JJ3E
.
You are receiving this because you authored the thread.Message ID:
@.***>

try running it with the latest version, I just fixed something that should at least make the error message more descriptive and easier to diagnose

Still erroring out unfortunately.

i added error handling for every type of api error, so you should now be able to see exactly what the issue is.

Working now! Wow this is amazing! Great job!

Thanks for your help, appreciate it!
-Jeremi
On Sun, Jun 11, 2023 at 12:28 AM Ethan @.***> wrote:

i added error handling for every type of api error, so you should now be
able to see exactly what the issue is.
—
Reply to this email directly, view it on GitHub
https://github.com/e-johnstonn/SalesCopilot/issues/1#issuecomment-1586008204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4YGQDTL6B2AYJ554SHPYKLXKVCM7ANCNFSM6AAAAAAZB7JJ3E
.
You are receiving this because you authored the thread.Message ID:
@.***>

no problem! my bad with the poor error handling, could've saved us both some time if it was done properly from the start

Haha, no problem! Don’t be so hard on yourself! I can’t express how much
this is going to help me with my daily meetings.
On Sun, Jun 11, 2023 at 12:42 AM Ethan @.***> wrote:

no problem! my bad with the poor error handling, could've saved us both
some time if it was done properly from the start
—
Reply to this email directly, view it on GitHub
https://github.com/e-johnstonn/SalesCopilot/issues/1#issuecomment-1586014778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4YGQDUDSOE2T6D3XHS6JE3XKVECZANCNFSM6AAAAAAZB7JJ3E
.
You are receiving this because you authored the thread.Message ID:
@.***>


JWL Media Window on the main screen

Sometimes when JwlMediaWin applies JW Library Fix, JWL media window appears on the main screen, over the JWL main window.
I've experienced this at KH and couple of times at home. Seems like this is happening only when JWL main window is not maximized. It happens randomly and I haven't been able to see a pattern.
When JWL main window is maximized I haven't seen this happening at all.
The only difference between Home and KH's setup is that I don't use a 3rd screen at home (I don't know if this makes any difference).
I also noticed that when JwlMediaWin applies JW Library Fix, if JWL is not maximized, yeartext flashes once covering JWL main window (on the main screen) and after is displayed on the 2nd screen... but probably this is the normal behavior.
Below is a screenshot with the mention that I've moved a bit JWL media window just to see the main window which is underneath.

The same thing has also happened to me a few times.
Sent with GitHawk

i experienced same issue

I have not been able to reproduce the issue but have guessed at what may be happening and applied a 'fix' based on this assumption!

@AntonyCorbett unfortunately I still have this issue even using the latest update.
I have no idea what could be. At home it's pretty hard to reproduce it (almost impossible since <IP_ADDRESS>) but at KH let's say that there are 50% chances to came across this issue.
The only workaround is to have JWL main window maximized. But some congregations from our hall are using JWL window half screen so this workaround can't be applied all the time.
Do you happen to know if there's a way to enforce JWL main window to be maximized at open?
Thanks for all your help and advices.

@LvDaniel What does the log show when you experience this bug please?

@AntonyCorbett I haven't thought to check the log since the app doesn't crash or something like this. I'll check it tomorrow's evening and I'll let you know.
Another difference I spotted is that at KH I don't get the information message when JwlMediaWin applies the fix (on bottom right). But probably this is from settings.

I don't get the information message when JwlMediaWin applies the fix (on bottom right).

@LvDaniel That's very useful to know. I have attempted another fix (latest pre-release), but I doubt it will work! Your log content will be helpful.

@AntonyCorbett sure, I'll also install the pre-release tomorrow and let you know how it goes. Thanks for your precious help.

@AntonyCorbett I've test this at work also. When using version <IP_ADDRESS> and <IP_ADDRESS> I encounter this issue all the time (I didn't expect that). When using <IP_ADDRESS> everything works fine.
Logs don't give many details but maybe will be helpful for you. Please see them attached.
<IP_ADDRESS>_log-20190214.txt
<IP_ADDRESS>_log-20190214.txt
<IP_ADDRESS>_log-20190214.txt
I'll test tonight <IP_ADDRESS> version at KH and let you know the results.
I'll give you some details about the setup, maybe will be helpful.
At work I have 3 monitors, 2 of them connected to the on-board graphic card (Monitor1 & 2) and one connected to a USB to DVI Adapter (Monitor3). JWL main window is displayed on Monitor1, while Monitor2 is the 2nd display (JWL Media Window).
At KH, 2 monitors are connected to the on-board graphic card, one being the main display and 2nd being the Media display. Third monitor is a portable monitor (Asus MB169B+) which is connected via USB to the PC and it's used for timer (OnlyT).

@LvDaniel Thanks. Look forward to your next report!

@AntonyCorbett I've tested the latest release (<IP_ADDRESS>) last evening at KH and I haven't encounter this issue anymore. Everything works fine now.
Thanks a lot Antony for all your efforts to solve this.

@LvDaniel Excellent - thanks.

@AntonyCorbett I tried to take a look at the code but it looks like you forgot to push the latest commits.

@TizianoNoto you're right, thanks. Now pushed.


Question: does the dx get prs --review flag make sense anymore?

it may make sense to switch this to using is:open is:pr review-requested:<self>


Filename completion

Ok,
I am not happy with the Competer Trait but it works...
If you have any suggestion, just tell me.
Regards.

Are you unhappy with something specific in the Completer Trait?  It seems sufficient to me

I've tried the following signature (&mut self):
fn complete(&mut self, line: &str, pos: usize) -> Result<(usize, Vec<String>)>;

without success.
For example, if you want to make a SQL completer but the schema has changed, you will have to invalidate/mutate the cached database metadata...

The next pull request should be 'incremental history search' (Ctrl-R/S) and may be some 'word' actions (delete previous/next word, ...).
Regards.


aptly graph to support more output options

It would be nice if aptly graph would support output options, e.g. -type=svg and -stdout=false -- so that (as an example) a SVG-format graph could be generated to stdout.
Use case: repo server has a CGI program that generates an up-to-date graph (using the above options) as a response to an HTTP request.
EDIT: Considering discussion, the ability to dump the graph to stdout is probably not needed. Still, the option to generate SVG would be welcome (scalable vector graphic instead of a bitmap is so much nicer when the amount of mirros/repos/snapshots is very high).

Adding here, so that it is not missed: deb package should recommend graphviz package.

Done in master.


Delete feature/experimental/qos/ate_tests/qos_policy_config_test dire…

…ctory
The files have been moved to https://github.com/openconfig/featureprofiles/tree/main/feature/qos/ate_tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.612%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1203

Relevant Lines:
2125

💛  - Coveralls

/gcbrun


Macro with children

Is it possible to update the documentation to show how to write macros with body/children ?
{% macro form(action, class) %}
    <form class="{{ class }}" role="form" action="{{ action }}" novalidate="novalidate">
        {{ body }}
    </form>
{% endmacro %}

Macros do not support inheritance. Either you need to provide the "body" via an argument to the macro or instead of macros use the include tag to include a full fledged template which does support inheritance.
{% include 'child.html' with { 'action' : 'post', 'class' : 'myclass'} %}

and the base form template, form.html:
<form class="{{ class }}" role="form" action="{{ action }}" novalidate="novalidate">
  {{ body }}
</form>

and a child form template, child.html:
{% extends 'form.html' %}
{% block body %}
   This is the body.
{% endblock %}


30  remover dependencia de satooshi php coveralls

Coverage remained the same at 97.893% when pulling<PHONE_NUMBER>26f2363c40171ceb3f454c0ed6cd9c on 30--Remover-dependencia-de-satooshi_php-coveralls into 25517066c938b933286b5c6763c1a533efa6c5d8 on master.


Aligment of the remember me option in login page

Describe the bug
"Remember me" text not align with the check box in login page
To Reproduce
Steps to reproduce the behavior:

Go to https://circuitverse.org/users/sign_in
See the error below the email and password input section.

Expected behavior
It should be perfectly aligned with the checkbox given for it.
Screenshots

Device Information [optional]:
for all devices this issue is visible.
Are you working on this issue? (Yes/No)
No, but I would like to work if assigned to me.

@omthakare16 May I work on this please?

@vedant-z I myself want to work on this issue but I don't have right to assign anyone, @vedant-jain03  can you assign me.

@vedant-jain03  @omthakare16  can i work on this issue???

This issue is duplicate
https://github.com/CircuitVerse/CircuitVerse/issues/3252


update command probe

Add environment and secret usage

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659f89422aebc8322a3566e3--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659fb158c323e149d8e8d633--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659fdd32bd4b0706c59f333a--harness-developer.netlify.app


Serialization is too costy

I noticed that even if I create a NanoCuckooFilter with capacity 1, the serialized version of this object is around 2kb.
I think this happens because of Java's default serialization method, maybe it shouldn't serialize some of NanoCuckooFilter's fields (transient fields)
I want to create an optimized serialization solution, what exactly should I serialize at minimum?
Thanks so much!

I think a lot of that size will be overhead from the serialized Java objects. I'll see what I can do to cut it down.
I can also provide the capability to dump only the internal filter memory, which could then be imported into a new NanoCuckooFilter, as long as the the filters were built with the same settings. This would give the smallest possible file size.

Fixed in 2.0.0. Serialization overhead reduced to about 124 bytes. Also added readMemory and writeMemory methods so internal filter memory can be dumped and reloaded directly.


Skip labelling process if the pull request author is already a repository maintainer

In this case, we can bypass the PR security process since the committer already has write access/

Just the CRDA scan approved and CRDA scan pending  label or other labels too?

the passed/failed labels still make sense to add

Yeah, that makes sense.


Unsafe code requires the `unsafe' command line option to be specified

OmniSharp has errors open my project. I think it has something to do with a plugin using unsafe DLL code:
[ERROR:OmniSharp.MSBuild.MSBuildProjectSystem] Unsafe code requires the `unsafe' command line option to be specified

Any ideas on how to fix this in VSCode?

Currently, Visual Studio Code doesn't have any way that I know of to pass parameters to the compiler (roslyn). I would pass your request on up the Visual Studio Code teams' chain about adding that feature, then I would integrate based on it.
https://visualstudio.uservoice.com/forums/293070-visual-studio-code

Well, normally, the project option <AllowUnsafeBlocks>True</AllowUnsafeBlocks> is supposed to fix this issue. For some some reason, even after manually modifying csprojs, OmniSharp is still reporting those errors.


Update guzzle requirement to allow ^6.5

This fixes a problem on GSG where guzzle is 6.5.5

This was done with #18


Add TailwindCSS MLP example

@kingscott-sugarcrm would be great if you could apply a CSS customization to a Sugar component (perhaps the .test class from the bundle.css), so developers know exactly how Tailwind will help on real-world scenario.
Also add an override of something we ship OOB

I do have a small example in the README for this. Using Tailwind doesn't really change the workflow for overriding any templates we have. @rfernandes-sugarcrm


Private Endpoint Missing

Private Endpoint Resource is not mentioned on this document

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

ID: 374ec8eb-64d4-3766-5c5e-39137a262ffd
Version Independent ID: 3abeeae9-6854-651a-483c-a1673eea9f9f
Content: Resource naming restrictions - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/resource-name-rules.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@wasytb72  Thanks for the feedback! I have fixed the referenced change, and the change should reflect on the doc within 24 hours. We will now proceed to close this thread. If there are further questions regarding this matter, please revert and we will gladly continue the discussion.


Inquiry about threads_api abilities

Hey @noahclark556, thanks for this great work. I would like to confirm something. Can this help me get access to threads videos once provide the video url? I know nothing about Python. Just confirming. Thanks

As of now, media access is not available. Although i do plan to add it in
the near future!
On Wed, Jul 26, 2023 at 11:58 Jexsie @.***> wrote:

Hey @noahclark556 https://github.com/noahclark556, thanks for this
great work. I would like to confirm something. Can this help me get access
to threads videos? I know nothing about python. Just confirming. Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/noahclark556/threads_api/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOWYH5KA2SOSDKO2GPNJ7KLXSE5AZANCNFSM6AAAAAA2YZYFGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

As of now, media access is not available. Although i do plan to add it in the near future!
…
On Wed, Jul 26, 2023 at 11:58 Jexsie @.> wrote: Hey @noahclark556 https://github.com/noahclark556, thanks for this great work. I would like to confirm something. Can this help me get access to threads videos? I know nothing about python. Just confirming. Thanks — Reply to this email directly, view it on GitHub <#4>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWYH5KA2SOSDKO2GPNJ7KLXSE5AZANCNFSM6AAAAAA2YZYFGQ . You are receiving this because you were mentioned.Message ID: @.>

Roger. Thanks anyway. Is there anyway I can jump into now and help you with some tasks? Minor ones ofcourse

Absolutely, adjust as needed and send up a pull request. i’ll review and most likely merge it with main branch!

As of now, media access is not available. Although i do plan to add it in the near future!

Roger. Thanks anyway. Is there anyway I can jump into now and help you with some tasks? Minor ones ofcourse

As of now, media access is not available. Although i do plan to add it in the near future!

Roger. Thanks anyway. Is there anyway I can jump into now and help you with some tasks? Minor ones ofcourse

Absolutely, adjust as needed and send up a pull request. i’ll review and most likely merge it with main branch!


Forgotten Races Boons don't show up in selector

Like the title says i have unlocked the forgotten races salt man boon and yet its not showing up in the boon selector.

Yeah, unfortunately, there's no simple way to automatically get all the boons, so all the available boons have to be added manually. The "boon from last run" option is kind of an intentional fall-back to the default behavior so it always works correctly.
I'll try to look into adding the Forgotten Races boons when I have some time.

Yeah, unfortunately, there's no simple way to automatically get all the boons, so all the available boons have to be added manually. The "boon from last run" option is kind of an intentional fall-back to the default behavior so it always works correctly.
I'll try to look into adding the Forgotten Races boons when I have some time.

Yeah, unfortunately, there's no simple way to automatically get all the boons, so all the available boons have to be added manually. The "boon from last run" option is kind of an intentional fall-back to the default behavior so it always works correctly.
I'll try to look into adding the Forgotten Races boons when I have some time.

Yeah, unfortunately, there's no simple way to automatically get all the boons, so all the available boons have to be added manually. The "boon from last run" option is kind of an intentional fall-back to the default behavior so it always works correctly.
I'll try to look into adding the Forgotten Races boons when I have some time.

The latest version now supports boons from Forgotten Races, Forgotten Diamonds, and Insurrection+ Selection.


Fails to validate for ad-hoc/enterprise distribution

When creating an archive from xcode, and trying to publish for ad-hoc distribution, the archive is rejected by apple with following error:
        {
            code = 0;
            description = "Failed to verify bitcode in sqlitewrapper.framework/sqlitewrapper:\nerror: Bundle only contains bitcode-marker /var/folders/6b/htlj4_ks3xsfg5c7_7wbwntc0000gn/T/IDEDistributionOptionThinning.Sft/Payload/wespr.app/Frameworks/sqlitewrapper.framework/sqlitewrapper (armv7)\n\n";
            info =             {
            };
            level = ERROR;
            type = "malformed-payload";
        }

Apparently, the xcode project must have ENABLE_BITCODE = NO; when building a framework:
https://github.com/EddyVerbruggen/nativescript-plugin-firebase/commit/ceac3076e8466ea2c92ce89455b8c660943d0523

Thanks for letting me know.

@nitrotm  - Can you tell me if you are a paid client or not -- I need to know if this is a priority or something that can wait for a couple weeks for my next release that is currently being worked on.

No, it's a bit pricey for my use-case. Seems to be a pretty easy fix to apply though.

It is a pretty easy fix; but their is time usage on my part for it still, no matter how simple the fix.      Open source doesn't feed my family, and my wife gets very testy with me when she doesn't have money for food.  So, paid stuff always has to have priority.  :grinning:
That was the reason I asked -- I never want an actual customer to feel like my support isn't superb.  Since open source is free -- it has to always take a lesser priority, so that my actual customers do feel the support is superb, because I have the time for their issues.  :grinning:
I'll update this issue when I release a new version in a couple weeks with the fix...   Thank you for letting me know about it!

Sure, I understand and share your concerns ;) Thank you for looking into it when it makes sense to do so.

This plugin seems to work:
https://market.nativescript.org/plugins/nativescript-disable-bitcode

This is no longer an issue in v2.6.0


Fix NPE for when fields are missing in searchNearestVectors

Related to: https://github.com/apache/lucene/pull/13162
Since this is unreleased, no changelog entry is necessary.

Was this behaviour regressed by https://github.com/apache/lucene/pull/13162? I wonder if a test would be warranted to ensure that it does not reoccur.  Also, in some areas fi.getVectorDimension() == 0 was also checked at the same time.

@ChrisHegarty added a test :)


Is it possible to set Port=None as Python?

Hello,
I didn't set a port. Is it possible to set Port=None as Python?
Python example:
https://github.com/qdrant/qdrant-client/issues/394#issuecomment-2008190756
QdrantClient client = new(new Uri("https://url.com"), apiKey: "");
Thank you.
/// <summary>Instantiates a new Qdrant client.</summary>
/// <param name="host">The host to connect to.</param>
/// <param name="port">The port to connect to. Defaults to 6334.</param>
/// <param name="https">Whether to encrypt the connection using HTTPS. Defaults to <c>false</c>.</param>
/// <param name="apiKey">The API key to use.</param>
/// <param name="grpcTimeout">The timeout for gRPC calls to Qdrant; sets the gRPC deadline for all calls.</param>
/// <param name="loggerFactory">A logger factory through which to log messages.</param>
/// <remarks>
/// This type provides higher-level wrappers over the low-level Qdrant gRPC API. If these wrappers aren't
/// sufficient, <see cref="QdrantGrpcClient" /> can be used instead for low-level API access.
/// </remarks>
public QdrantClient(
	string host,
	int port = 6334,
	bool https = false,
	string? apiKey = null,
	TimeSpan grpcTimeout = default,
	ILoggerFactory? loggerFactory = null)
	: this(new UriBuilder(https ? "https" : "http", host, port).Uri, apiKey, grpcTimeout, loggerFactory)
{
}

Hi @fatihyildizhan
Yep, you can use the constructor that takes a Uri. The example is the same as you've specified in the question 🙂
QdrantClient client = new(new Uri("https://url.com"), apiKey: "");


[darwin] Update the NetworkFramework UDP implementation

Problem
The NetworkFramework implementation has been broken by multiple changes.
This PR updates the various #ifdefs flags to make it possible to compile it again. It also adds a chip_system_config_use_network_framework build flag and convert the implementation from .cpp to .mm with ARC enabled.

For reviewers: I tried to git mv src/inet/UDPEndPointImplNetworkFramework.cpp src/inet/UDPEndPointImplNetworkFramework.mm but the change is large enough that git believe this is a new file.


Migrate to esbuild for faster builds & stable dev experience

Description
Migrate to esbuild for faster builds & stable dev experience
Fixes: #389, #340
Checklist

[x] The documentation reflects the changes.
[X] I have manually tested the application to make sure the changes don’t cause any downstream issues, which includes making sure ./orchest status --ext is not reporting failures when Orchest is running.

Wauwie! Great job :smile:
Are the development workflow docs still up to date after these changes?


IOS Background mode: skip forward/back

Describe the bug
While skipping forward and/or backward while the app is backgrounded on notification player, the forward/back button will randomly gray out and loose the ability to skip. There are still tracks available behind/ahead in the queue. Opening up the app and calling skiptonext/skiptoprevious works just fine.
To Reproduce
Have a tracklayer spun up with multiple tracks in queue. Background app. Use notification controls to skip forward/back a few times. Eventually a forward or backward skip button will gray out and not function.
Environment (please complete the following information):
ystem:
OS: macOS 11.6
CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
Memory: 207.29 MB / 32.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.17.6 - /usr/local/bin/node
Yarn: Not Found
npm: 7.23.0 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
Android SDK:
API Levels: 28, 29, 30
Build Tools: 29.0.2, 30.0.2
System Images: android-29 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-30 | Google APIs Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7678000
Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.66.1 => 0.66.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
What react-native-track-player version are you using?
v2.1.1
Are you testing on a real device or in the simulator? Which OS version are you running?
Real device iPhone 7, iPhone 13 pro max, iPad Pro 11 (2018). Issue present on all devices and on both iOS 14 and 15.
Code
I looked through the native code but I'm not that fluent in swift. I wasn't able to locate the offender. If I had to guess for some reason the remote player is thinking that the previous/next track is no longer available in the queue but I couldn't figure out why.

Confirmed present in the example project provided in this repo. I'll be reverting to 1.2.7 for now.

Thanks @Enragedsaturday are you able to investigate and/or propose a fix PR for this?

@bradleyflood I tried to track it down but I'm a humble JS dev and couldn't sift through the swift to figure out the cause.

@Enragedsaturday me too heh. Ok let's hope for an iOS dev. Know anyone? hah

@Enragedsaturday would you be able to test and see if this is an issue in v2.2.0-rc3 (here's the branch).

I just tested this against main and I'm unable to reproduce this behavior. It's possible it was fixed by this upgrade. I did a lot of testing in different scenarios and in all cases I was seeing the behaviour I would expect. I'm going to close this issue as solved. If you're able to create a reproducible example against main I'm happy to reopen.

I've the issue on rc4, so that's not solved yet.

@matteodanelli , as I said I am not able to reproduce it. If you want us to reopen this issue you'll need to create a reproduction based on the example app and send us detailed reproduction steps.


https://viromedia.com/signup/ - NOT WORKING

Unable create account to obtain API KEY. Please look into this?

This is not needed anymore since it's been open source

@7sxzx could you please close the issue when you can ?

Thank you for your response and open sourcing this great product.


Add vaccination data source

Describe the solution you'd like
The source of the vaccination could be put as a link here:

Hello, I want to add source of the vaccination link . please give me this opportunity

Create a pr

Can you describe what will be the use of it?

Can we use 'a' tag for this.?
On Fri, 23 Apr 2021 at 23:21, Shikari Shambu Jr. @.***>
wrote:

Can you describe what will be the use of it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/covid19india/covid19india-react/issues/2417#issuecomment-825820395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMYAJMTUJU4SR5QDVV2I2C3TKGXSVANCNFSM42UGCEFA
.

Can you describe what will be the use of it?

So that we shall be able to directly obtain the data directly from source as well as cross-verify. Also, we can see the first dose and second dose data seperately


Resolve token from session

Instead of leaving the work of setting the default token on the application, let the service provider do it automatically.
Solves #140

I know this isn't the best way to handle it, but I'd like to give a heads up and get your opinion.


[ContextMenu] Scroll to top when on absolute container

Bug report
Current Behavior
I have a context menu inside an absolute container, this container is powered by react-window (long virtual lists).
When I use Portal (attach to document.body) it doesn't show up.
When I don't use a portal it shows up for items before scroll but for items that require scrolling down it scrolls to top when I right click and not show the menu.
Expected behavior
Not to scroll to the top and show the menu there.
Reproducible example
https://user-images.githubusercontent.com/60936/192516916-043eb7d6-ed6a-45be-ae8e-4142d12dd074.mp4
Suggested solution
I suspect this could be popper configuration issue, we have a few poppers given tooltips and menus in our app.
Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-context-menu
1.0.0

React
n/a
17

Browser
Chrome
105

Assistive tech

Node
n/a
16.14.0

npm/yarn
yarn
1.22.19

Operating System
macOS
12.6

Hey @cabello,
You report using version  1.7.0 of @radix-ui/react-context-menu which I don't think exists.
If you're using an older version from 1.0.0 then it's likely by upgrading your issue would be fixed.
If you are indeed using 1.0.0 then it would be very helpful if you could provide a reproduction in a codesandbox.
Thanks!

Hi @benoitgrelard , it was a typo 🙈 I am using 1.0.0.

In this case, a reproduction would be great because at first glance, there's no reason why it shouldn't work in your scenario.


BUG: loffset not applied when using resample with agg() (GH13218)

[x] closes #13218
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

_apply_loffset is called in resampler's downsample and groupby_and_agg methods, defined here and here
When a str is passed into aggregate, the above methods are called here, and consequently the loffset is applied.
However, if anything other than a str is passed into aggregate, then a different path will be taken and the loffset must be applied. So a simple check in aggregate will apply the loffset correctly.
note: my last PR was closed, and I can't reopen it by a force push, hence the new one

Current coverage is 85.24% (diff: 100%)

Merging #14213 into master will increase coverage by <.01%

@@             master     #14213   diff @@
==========================================
  Files           140        140
  Lines         50566      50570     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          43102      43107     +5
+ Misses         7464       7463     -1
  Partials          0          0

Powered by Codecov. Last update f363236...c897609

@wcwagner if you want to update to 0.19.2 and rebase, we can take another look.

can you move release note to 0.20.0

Sorry for the delay, I will update everything tomorrow.

pls add example referenced in #15002 as well (your changes should cover it)

@jreback do you want me to add the example in #15002 into the unittests? I suppose it would be a good idea to do so, seeing as it tests the how parameter (even though it's deprecated).

yep

Seems like resample's loffset tests were removed from TestPeriodIndex and TestTimedeltaIndex. Should I remove my tests from those two as well?

thanks @wcwagner
I reworked this a bit to make it cleaner in https://github.com/pandas-dev/pandas/commit/366241386282272328333256b6e267a68f8133a4
love for you to tackle some more issues!


Yarn or npm for plugin development?

This is a request for decision
Description
Currently the (development) infrastructure of the plugin is aimed to use npm as packaging tool
and contains a npm 5 compatible package-lock.json. To have reproducible PR and implementation verifications, only tools that cope with the existing lock files can and should be used.
We should decide if we stick with npm or move to yarn as development package manager. Currently yarn cannot use package-lock files, so the decision would be an "either or", as otherwise (in case we commit both lockfiles) care has to be taken to keep them synchronized - and that each time anyone submits a PR that changes dependencies, as well as when a new version is published (the version number changes).
Here are some interesting links that might be beneficial for the discussion:
Should I commit yarn.lock and package-lock.json files?
Yarn: Idea: support package-lock.json from npm 5
DOES NPM 5 DEPRECATE YARN?
Yarn determinism
Thanks to @franciscocpg for bringing this up 🥇

Some scenarios
Without cache
yarn
First lets remove cache rm -rf node_modules && yarn cache clean
Then:
Command
time yarn

Result
yarn install v0.27.5
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
Done in 21.05s.
yarn  11,66s user 2,55s system 66% cpu 21,427 total

npm
First lets remove cache rm -rf node_modules && npm cache clean --force
Then:
Command
time npm i

Result
><EMAIL_ADDRESS>postinstall /home/francisco/programs/git/github.com/franciscocpg/serverless-webpack/node_modules/spawn-sync
> node postinstall

><EMAIL_ADDRESS>postinstall /home/francisco/programs/git/github.com/franciscocpg/serverless-webpack/node_modules/serverless
> node ./scripts/postinstall.js

npm WARN<EMAIL_ADDRESS>requires a peer of webpack@* but none was installed.

added 371 packages in 46.809s
npm i  19,69s user 2,44s system 46% cpu 47,428 total

Conclusion
yarn - 21.427s
npm - 47.428s
In this scenario yarn runs about 2.21x faster.
With cache / without node_modules
yarn
First lets remove node_modules rm -rf node_modules
Then:
Command
time yarn

Result
yarn install v0.27.5
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
Done in 4.41s.
yarn  4,72s user 0,96s system 118% cpu 4,782 total

npm
First lets remove node_modules rm -rf node_modules
Then:
Command
time npm i

Result
><EMAIL_ADDRESS>postinstall /home/francisco/programs/git/github.com/franciscocpg/serverless-webpack/node_modules/spawn-sync
> node postinstall

><EMAIL_ADDRESS>postinstall /home/francisco/programs/git/github.com/franciscocpg/serverless-webpack/node_modules/serverless
> node ./scripts/postinstall.js

npm WARN<EMAIL_ADDRESS>requires a peer of webpack@* but none was installed.

added 371 packages in 7.745s
npm i  14,21s user 1,58s system 189% cpu 8,330 total

Conclusion
yarn - 4.782s
npm - 8.330s
In this scenario yarn runs about 1.75x faster.
With cache and node_modules
yarn
Command
time yarn

Result
yarn install v0.27.5
[1/4] Resolving packages...
success Already up-to-date.
Done in 0.37s.
yarn  0,82s user 0,05s system 115% cpu 0,751 total

npm
Command
time npm i

Result
npm WARN<EMAIL_ADDRESS>requires a peer of webpack@* but none was installed.

up to date in 1.642s
npm i  2,39s user 0,11s system 111% cpu 2,226 total

Conclusion
yarn - 0.751s
npm - 2.226s
In this scenario yarn runs about 2.96x faster.
Final Conclusion
So it looks like yarn is about 2x faster than npm in most cases.
Also node community are adopting yarn, eg https://github.com/docker-library/official-images/pull/2705
IMO I see no reason not to adopt yarn.

npm is still too widely used to be abandoned. I would keep both of them for the moment being.

Despite the speed of yarn, the linked articles show that npm 5 is currently more robust regarding dependency stability and integrity - yarn needs the package.json in combination to the yarn.lock to retrieve the dependencies, which might lead to different versions if the package.json changed independently, whereas package-lock.json contains the integral definition of all dependencies (including transient ones).
We could check in a package-lock.json as well as a yarn.lock in parallel (for master I can take care of that after merging PRs that change dependencies and for release preparations), so contributors can use their tool of choice to get the development environment up and running - as long as the package.json is not newer than the yarn.lock file.
As soon as yarn supports package.json files, we could even drop the yarn.lock (I do not really get why yarn does not support it already, because the format is the same as the shrinkwrap ;-) )

@HyperBrain
So we are going to generate and commit them both (package-lock.json and yarn.lock) every time a dependencies change to keep it sync with both yarn and npm?

Sorry for being silent for this issue for a long time now. Bringing the repo and project back on track left not really time to care about these more "uncritical" issues 😃 . I think we can bring that up again soon for further discussion (at least the examples now have current yarn.lock files thanks to @franciscocpg ).

It turned out that having and maintaining both for the main repository adds a lot of overhead. So I'm in favor of keeping npm for the main repo, and yarn locks for the examples.
Any objections?

Nope

I'll close this now :)


Will is forwarded twice (Windows)

When a client loses connection to the server the will for that client is forwarded to subscribed clients twice. This may be expected if QoS was set to 0 or 1, but the will has QoS 2 and the clients are subscribed with QoS 2. In this scenario I would expect the message would be forwarded exactly once to each subscribed client. I cannot reproduce this issue in a linux environment.
server.js
var aedes = require('aedes');
var net = require('net');

var broker = aedes();

var server = net.createServer(broker.handle);

broker.on('publish', function (packet, client) {
    if (packet.topic === 'Will/Example') {
        console.log('Client', client ? client.id : 'internal', 'published', '"' + packet.payload.toString() + '"');
    }
});

broker.on('client', function (client) {
    console.log('Client', client.id, 'connected.');
});

broker.on('clientDisconnect', function (client) {
    console.log('Client', client.id, 'disconnected.');
});

server.listen(1883, function () {
    console.log('MQTT broker listening on port: 1883');
});

client.js
var mqtt = require('mqtt');

var client = mqtt.connect('mqtt://localhost:1883', {
    clientId: process.argv[2],
    will: {
        topic: 'Will/Example',
        qos: 2,
        retain: false,
        payload: 'this is a will for ' + process.argv[2]
    }
});

client.on('error', function (err) {
    console.error(err);
});

client.on('offline', function () {
    console.log('MQTT client went offline.');
});

client.on('close', function () {
    console.log('MQTT client is closed.');
});

client.on('connect', function () {
    console.log('MQTT client connected to broker.');
});

client.on('message', function (topic, payload) {
    console.log(payload.toString());
});

client.subscribe('Will/Example', {qos: 2}, function (err) {
    if (err) {
        console.log('Subscription failed...');
    } else {
        console.log('Subscribed to will.');
    }
});

Output:
C:\Users\jholzinger\WebstormProjects\aedes-will>node server
MQTT broker listening on port: 1883
Client foo connected.
Client bar connected.
Client bar disconnected.
Client bar published "this is a will for bar"
Client bar published "this is a will for bar"

C:\Users\jholzinger\WebstormProjects\aedes-will>node client foo
MQTT client connected to broker.
Subscribed to will.
this is a will for bar
this is a will for bar

C:\Users\jholzinger\WebstormProjects\aedes-will>node client bar
MQTT client connected to broker.
Subscribed to will.
^C

Closing issue as it was resolved in the pull request.


Add OSG Team Members

Adds Showmic and Colby to the OSG Website.

Going to merge this over #319 as it includes both missing members.


Fix long URL for from_clickhouse

i have a  case with very long "in" condition in request.
default url length is 16kb, so, according to  clickhouse documentation about http interface, it is possible to move query to POST body.
I coded this functionality(15kb or more check, just to be sure) and made test for this case.
Welcome for comments

@JLeech sorry for the late response, it looks good to me however has formatting issues. Could You please run flake8 on the changes?


Menu option "Run utPLSQL test" always disabled in Connections navigator tree (at sign)

When then connection name contains at sign ("@"), the "Run utPLSQL test" menu option is always disabled.
The associated connection node is not found and therefore no enable checks occur and the node stays disabled.
As a workaround do not use the at sign ("@") in the connection name.

I can reproduce the issue. This is a bug. Thanks for reporting.

this is a similar bug like in #16. Needs generic solution for "special" characters.


UX Design Updates - OIPC

Title of ticket: UX Design Updates - OIPC
Description
UX Design Updates needed based off od
Dependencies
Are there any dependencies?
DOD

[x] Need module to be able to undo outcome and edit fields
[x] Advanced Search - Each Flag needs to be added under Request Type
[x] Update Ministry Designs based off of new fields
[x] Need to update all with additional Date Field

@m-prodan @liseandtea @sumathi-thirumani-aot @Aman-Hundal @divyav-aot
Updated Designs based off of Discussion at Refinement - reach out if anyone has any questions
OIPC Detail - Single Review - https://jacklyn808742.invisionapp.com/overview/FOI---Intake-ckpq19frj032r013q6gpm8t1m/screens
OIPC Details - Multi Review - https://jacklyn808742.invisionapp.com/overview/FOI---Intake-ckpq19frj032r013q6gpm8t1m/screens
OIPC Outcome Module - https://jacklyn808742.invisionapp.com/overview/FOI---Intake-ckpq19frj032r013q6gpm8t1m/screens
Delete OIPC Module - https://jacklyn808742.invisionapp.com/overview/FOI---Intake-ckpq19frj032r013q6gpm8t1m/screens
Undo OIPC Outcome - https://jacklyn808742.invisionapp.com/overview/FOI---Intake-ckpq19frj032r013q6gpm8t1m/screens
Ministry User OIPC - https://jacklyn808742.invisionapp.com/console/share/3W2O64AP5C/783340486


App Goes Back To Initial Route On Hot Module Reload

Version
Tell us which versions you are using:

react-native-router-flux v3.43.0
react-native v0.55.4

Expected behaviour
Maintain current route after hot module reload occurs.
Actual behaviour
Whenever hot module reload occurs, the app returns to the initial route, this is inconvenient because I have to start navigating all the way deep in to the screen I was on, and in most cases it's just  a small change, this makes development very painstaking, is there something I can do about this?
Steps to reproduce
My Router is connected to redux and I used Actions.create(/**/) to create the scenes.

Use the scene you want to test as initial
Your issue is actually not an issue.. it is a common behaviour and it totally makes sense

I can't agree that it makes sense, navigating all the way back to the screen you're working on because of a small adjustment you made to it isn't convenient, it kills my vibe when I'm working.


Feature: k0s runtime remain work

Describe what this PR does / why we need it
After k0s init step, there are update/join/delete/reset interface which need to implement, this PR finish all k0s cluster install action.
Does this pull request fix one issue?
NONE, but a step for #1399

Codecov Report
Base: 18.24% // Head: 18.24% // No change to project coverage :thumbsup:

Coverage data is based on head (48cf21c) compared to base (b9f928a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1686   +/-   ##
=======================================
  Coverage   18.24%   18.24%
=======================================
  Files          66       66
  Lines        5377     5377
=======================================
  Hits          981      981
  Misses       4272     4272
  Partials      124      124

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

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

I just reviewed the delete_master.go file, and attached several feedbacks. I think you have the same issue in other files, like delete_node.go, join_node.go and so on. Please update the original code all all files carefully. Then you could comment this pr to seek my further review. Thanks. @starComingup 🌹 🐯 🦁

Left some comment, and wait for @starComingup 's feedback and update before merging.

All comment and code updated! Waiting for test.

Thanks greatly for your patience during review. @starComingup I think your work could improve sealer quite a lot. Thanks again.


[Packaging] Support ARM64 on Linux

Description

The long-awaited feature is finally here. The ARM64 build will be released for current supported Linux distribution.
Close #7368
Related issue: #22875

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Support ARM64

/azp run

/azp run

Any timeline for this to be released?

@TravisEz13 On March 07


Features/SMA-81-testing-framework

Description
Added Jest, I highly recommend using it as it's widely used, easy for our team to adopt, and has extensive documentation for quick problem-solving.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Ran existing tests, update tests to cover all packages

lerna bootstrap or yarn and then yarn build is not working for me.


Kernel install folder names should not start with .

The dot (.) gets interpreted on Linux as a hidden folder and it's annoying, since the install location is actually incorrect and I move the folders to Anaconda environment. Them being hidden makes the mv command more wordy.

From #650

When this is fixed, it will impact Papermill support, which will also need to be updated:
https://github.com/nteract/papermill/pull/448
https://github.com/nteract/papermill/pull/451


SBOL Stack issues

-iGEM parts should be compliant (dispalyID)
-iGEM parts have annotation to iGEM entry url
-have support for collections
-have support for filtering by type

updated issues:
-have support for collections
-have support for filtering by type

Done