add rsc syntax

Add syntax file for RouterOS script.
Let me know if there's any issues to fix before merge.

Thanks for adding this filetype.
It looks like you're missing the return command and boolean true/false.
You could also consider using the #!rsc by RouterOS\n shebang line for filetype detection.  That would go in runtime/scripts.vim.

@dkearns , I added the missing keywords.
For the detection I added a regex as well since RouterOS adds a line like this during config export:
# nov/05/2021 22:01:24 by RouterOS 6.41.3

please also update .github/CODEOWNERS

@chrisbra , sure, done.

Would this filetype be better named as routeros?  As far as I can tell it's mostly known as RouterOS Script or sometimes MikroTik Script.

Would this filetype be better named as routeros?

Makes sense, I didn't actually realize it's named like this in other editors.
I think we can change this after we take care of the other PR you sent me to avoid conflicts.

If we are going to use "routeros" we better do this right away, no point in first adding rsc and then renaming.

Yes, we're just working on a couple of changes in zainin's fork under the old name then it'll be updated to the new name before it's offered for merging here.

I think we're done for now, let me know if there's anything else to tweak.

Thanks, I'll include it.  The filetype detection will be a patch, the rest will come with the next runtime file update.


Configure subdomain for Hall

We're evaluating Hall as an alternative for Linen: https://kedro.hall.community/
The tool gives you the possibility of configuring a custom domain:

Would community.kedro.org be a good one? Should we do hall.kedro.org instead? (Hence intent vs product name).
Past evidence: https://linen-slack.kedro.org/ (product name), http://slack.kedro.org (product name).

To clarify, custom subdomains is part of the paid plan

We're not doing this for now.


Use jest-mock-console instead of the withConsole util

@FredyC

Can we perhaps use jest-mock-console instead?

https://github.com/mobxjs/mobx-react/pull/724#discussion_r302284153
@vkrol

It has nothing to do with react-testing-library. I can investigate it in the separate PR, if you don't mind.

https://github.com/mobxjs/mobx-react/pull/724#discussion_r302381652

Looking good, thanks!


Maximum 1000 documents per batch is actually 32000

The documentation here says max 1000 documents per batch. When using the actual API the limit is 32000. And a higher batch size results in better indexing performance. Perhaps the documentation is for a different version of the API?
https://docs.microsoft.com/en-us/azure/search/search-limits-quotas-capacity#queries-per-second-qps

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

ID: 05e60837-26b9-1f5a-7a44-88ae8c9a7bc3
Version Independent ID: eaa6fabe-3d51-d626-3a51-d2e46efca454
Content: Service limits for tiers and skus - Azure Cognitive Search
Content Source: articles/search/search-limits-quotas-capacity.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@danglund Thanks for the feedback! We are currently investigating and will update you shortly.

I found the documentation that says max 1000 per batch and added feedback directly in the article.
Dan
From: SnehaAgrawal-MSFT<EMAIL_ADDRESS>Sent: fredag 28. februar 2020 10:38
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: danglund<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Maximum 1000 documents per batch is actually 32000 (#49139)
@danglundhttps://github.com/danglund Thanks for the feedback! We are currently investigating and will update you shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/49139?email_source=notifications&email_token=ADR22IZ7PCCVV27MR4NGF5LRFDLPJA5CNFSM4K5MPC5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENH4S3I#issuecomment-592431469, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADR22I2ZJMDV72SH6JLOY73RFDLPJANCNFSM4K5MPC5A.

@danglund @SnehaAgrawal-MSFT  Several customers have noticed the additional capacity, but the product group does not want to change the documented limits at this time.  They are not sure the 32K batch size will be supported moving forward.

Ok, that’s good to know. I will make sure to make this configurable in our application with a note that it might change.
In my experience the larger batch size resulted in better performance.
Dan
From: Heidi Steen<EMAIL_ADDRESS>Sent: fredag 28. februar 2020 16:32
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: danglund<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Maximum 1000 documents per batch is actually 32000 (#49139)
@danglundhttps://github.com/danglund @SnehaAgrawal-MSFThttps://github.com/SnehaAgrawal-MSFT Several customers have noticed the additional capacity, but the product group does not want to change the documented limits at this time. They are not sure the 32K batch size will be supported moving forward.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/49139?email_source=notifications&email_token=ADR22I5STEWTGQYKHVJKPGLRFEU6VA5CNFSM4K5MPC5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENI5E6Y#issuecomment-592564859, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADR22I2LLOH6BVXN3TQEBLDRFEU6VANCNFSM4K5MPC5A.

@danglund We will now close this out but if you feel you need more information please just let us know.


Implement all methods for sending messages [1]

What does this PR do?
This PR contains implementation for all of the Viber methods used for sending messages. And also some of the Telegram methods for sending messages. It also contains the implementation for the custom keyboard functionality for both messangers.
The PR fulfills these requirements:

[x] Tests for the proposed changes have been added/updated.
[x] The documentation has been updated according to the introduced/changed components.
[x] ApexDoc comments have been attached to all global classes, fields, properties and methods.

Codecov Report
Patch coverage: 90.48% and project coverage change: +1.23 :tada:

Comparison is base (6a70db5) 90.33% compared to head (04451eb) 91.56%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #16      +/-   ##
==========================================
+ Coverage   90.33%   91.56%   +1.23%
==========================================
  Files          22       52      +30
  Lines         424     1209     +785
==========================================
+ Hits          383     1107     +724
- Misses         41      102      +61

Flag
Coverage Δ

Apex
91.56% <90.48%> (+1.23%)
:arrow_up:

unittests
91.56% <90.48%> (+1.23%)
:arrow_up:

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

Impacted Files
Coverage Δ

...es/services/sender/TelegramSendKeyboardOptions.cls
27.18% <27.18%> (ø)

...n/telegram/classes/services/TelegramBotService.cls
83.33% <83.33%> (-16.67%)
:arrow_down:

src/main/default/classes/enums/BotEnumType.cls
85.18% <85.18%> (ø)

...elegram/classes/entities/TelegramBotChatEntity.cls
85.71% <85.71%> (ø)

src/main/default/classes/models/BotModel.cls
87.50% <87.50%> (ø)

.../viber/classes/services/ViberBotCalloutService.cls
87.50% <87.50%> (ø)

src/main/default/classes/entities/ValueMap.cls
93.75% <88.88%> (+14.06%)
:arrow_up:

...ram/classes/services/TelegramBotWebhookService.cls
88.88% <88.88%> (ø)

...rc/main/viber/classes/services/ViberBotService.cls
88.88% <88.88%> (ø)

.../classes/entities/TelegramBotUpdateEventEntity.cls
92.30% <92.30%> (ø)

... and 40 more

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

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


Books DB

Project title
Setting and Getting Books
Project Description
I wish to create a smart contract using which we will be able to store books with their title, author's name and id and then later on we can get the books using its id.
Domain

[x] Solidity

Are you contributing under any open-source program ?

[x] Hacktoberfest 2022

please add the relevant name of the issue . write full form for Db also .
Thank u @vryan-06

Done

Great start  working on this issue .
@vryan-06


Support forwarding of IEvent from Marten in addition to the T

Would be great to add support for handling via the interface of a type, instead of just the concrete type.
public class MartenEventsConsumer : IMartenEventsConsumer
{
    private readonly IServiceProvider _serviceProvider;

    public MartenEventsConsumer(IMessageBus bus)
      => _bus = bus;

    public async Task ConsumeAsync(IReadOnlyList<StreamAction> streamActions)
      => foreach (var @event in streamActions.SelectMany(streamAction => streamAction.Events))
              _bus.Send(@event) ==> // this doesn't work
<EMAIL_ADDRESS>==> // this does
}

public class ElasticEventHandler
{
    public async Task Handle(IEvent<MyEvent> message)
        => // doesn't get here;

    public async Task Handle(MyEvent message)
        => // this works;
}

I think this is turning into a bigger post-1.0 story about optimizing the event publishing and this is going to go into 1.1. Also probably gets done at the same time as the daemon subscription integration
Ideas:
Optimized strategy per event type like:

internal interface IMartenEventPublisher
{
    ValueTask PublishAsync(IEnumerable<IEvent> events, IMessageBus bus);
}

That would already pre-routed, so that the real thing is either a nullo, do nothing, or publishes immediately through MessageRoute and bypasses a lot of the existing MessageBus.PublishAsync() method.

Also, introduce the idea of event filtering.

Transformers to other message shapes, including using IEvent as either the destination or the transformation source

Type filtering off of the EventForwardingToWolverine syntax

Test Cases

[ ] Can still automatically forward messages from IEvent to T
[ ] Directly handle IEvent<T>
[ ] Custom transformation of IEvent<T> to a different type


Rename RegexComponent.Output

This renames RegexComponent.Output to RegexOutput, so that it is less likely to collide with other associated type that String, etc might conform to.

@swift-ci Please test


Dask Error: distributed.core - ERROR - addresses should be strings or tuples, got None

The default to pandas test has been failing recently on Dask with Windows. I have searched and the errors thrown do not appear to be known issues at the moment.
Link to test failure: https://github.com/modin-project/modin/runs/862907959?check_suite_focus=true

Traceback (most recent call last):
  File "C:\hostedtoolcache\windows\Python\3.7.8\x64\lib\site-packages\distributed\utils.py", line 656, in log_errors
    yield
  File "C:\hostedtoolcache\windows\Python\3.7.8\x64\lib\site-packages\distributed\scheduler.py", line 2208, in remove_worker
    address = self.coerce_address(address)
  File "C:\hostedtoolcache\windows\Python\3.7.8\x64\lib\site-packages\distributed\scheduler.py", line 4946, in coerce_address
    raise TypeError("addresses should be strings or tuples, got %r" % (addr,))
TypeError: addresses should be strings or tuples, got None
distributed.core - ERROR - addresses should be strings or tuples, got None
Traceback (most recent call last):
  File "C:\hostedtoolcache\windows\Python\3.7.8\x64\lib\site-packages\distributed\core.py", line 513, in handle_comm
    result = await result

Issue was not resolved with #1723

It was found out, that Modin CI fails with the latest versions of Dask and Distributed packages, so versions of these packages are downgraded to the previous versions for now.


Need for a "Dev mode" TLS free option

Hi there.  It would be very handy if EventStreams could be installed in a "Dev mode" that didn't require all the certificates and the TLS and the whatnot.  People kicking tires on the product want to be up and running in minutes, and although I realize security is very important, it can slow people down which causes a drop-off in enthusiasm.  A switch on the helm chart to enable or disable TLS would be awesome.
Additionally, when using the beta, I had to connect to the brokers via their internal container addresses.  I actually really liked this, as I had my own containers which could connect to Kafka without having to go out through the external proxy.  It would be very nice to bring this feature back.

We are actually working on just such a facility at the moment, hopefully for inclusion in the fixpack at the end of this month.

Awesome!  The first thing or the second thing?

Sorry, I was referring to the first item.
Re. the 2nd one: no, sorry, we intend to keep all access via the proxy. Firstly, there are cross-cutting concerns and features that need (or will need) to be handled in one place. But also I suspect that ICP itself does some networking cleverness so your Kafka clients won't actually go outside the cluster when accessing the proxy nodeport anyway.


Does the CVE-2022-22965 Vulnerability Affect spring-cloud-circuitbreaker？

A Spring MVC or Spring WebFlux application running on JDK 9+ may be vulnerable to remote code execution (RCE) via data binding. The specific exploit requires the application to run on Tomcat as a WAR deployment. If the application is deployed as a Spring Boot executable jar, i.e. the default, it is not vulnerable to the exploit. However, the nature of the vulnerability is more general, and there may be other ways to exploit it.

See
https://spring.io/blog/2022/04/01/spring-framework-rce-mitigation-alternative
https://spring.io/blog/2022/03/31/spring-boot-2-6-6-available-now
https://spring.io/blog/2022/03/31/spring-boot-2-5-12-available-now


Add autoSelect labels to more ServingRuntime formats

Some formats like onnx did not have an autoSelect-able runtime assigned, so using an InferenceService with modelFormat: onnx would show no runtime found to support predictor with model type.... This enables autoSelect on formats I thought needed them.

/retest

/lgtm
/approve


cant use startVideoRecording on Android 13

environment:
Windows
Android 13 actual device
@viro-community/react-viro": "^2.41.1", "react-native": "0.72.6",
Infinix Zero 30
    const handlePressRecordVideo = async () => {
      if (recording) {
        try {
          const result = await ref.current?._stopVideoRecording();
          setRecording(false);
          console.log("[254]", result);
        } catch (e) {
          console.error("[254]", e);
        }
      } else {
        try {
          setRecording(true);
          const result = await ref.current?._startVideoRecording(
            "test",
            true,
            (e) => {
              // https://viro-community.readme.io/docs/viroconstants#section-video-recording-screenshot-errors
              console.error("[254]", e);
            }
          );
          console.log("[254]", result);
        } catch (e) {
          console.error("[254]", e);
        }
      }
    }

it returns {"errorCode": 1, "success": false, "url": null}
This is working fine with android 12 with these permissions
await request('android.permission.WRITE_EXTERNAL_STORAGE'); `
what do I need to do in order to make this _startVideoRecording work with Android 13? or just store the recording anywhere on the phone which I can access

@saadxprt Pls Check the below Link for Android 13 Permissions Changelogs and Select the Suitable Answer for your Problem
Android 13 Permissions

@saadxprt Pls Check the below Link for Android 13 Permissions Changelogs and Select the Suitable Answer for your Problem
Android 13 Permissions

i did try to incorporate the appropriate permissions as per many answers, didnt work for me

Someone solved this ?

Hi I think that I have the solution to this bug, basically the viro library internally is reviewing the permissions but on sdk above to 33 we need to ask for other permissions types, so, I fixed recently if you create your own react viro version and make a local installation
After you clone the rect viro repository (https://github.com/viromedia/viro) you will need to add the follow lines inside the:
android -> viro_bridge -> java -> src -> module -> ARSceneNavigatorModule.java
search the line 331:
if (audioAndRecordingPerm){
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
                reactActivity.requestPermissions(new String[]{
                        Manifest.permission.READ_MEDIA_IMAGES,
                        Manifest.permission.READ_MEDIA_AUDIO,
                        Manifest.permission.READ_MEDIA_VIDEO,
                        Manifest.permission.RECORD_AUDIO
                }, PERMISSION_REQ_CODE_AUDIO, listener);
            } else {
                reactActivity.requestPermissions(new String[]{Manifest.permission.WRITE_EXTERNAL_STORAGE,
                        Manifest.permission.RECORD_AUDIO}, PERMISSION_REQ_CODE_AUDIO, listener);
            }
        } else {
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
                reactActivity.requestPermissions(new String[]{
                        Manifest.permission.READ_MEDIA_IMAGES,
                        Manifest.permission.READ_MEDIA_AUDIO,
                        Manifest.permission.READ_MEDIA_VIDEO,
                        Manifest.permission.RECORD_AUDIO
                }, PERMISSION_REQ_CODE_AUDIO, listener);
            } else {
                reactActivity.requestPermissions(new String[]{Manifest.permission.WRITE_EXTERNAL_STORAGE},
                        PERMISSION_REQ_CODE_STORAGE, listener);
            }
        }

       private static boolean hasAudioAndRecordingPermissions(Context context) {
              if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
                  boolean hasReadMediaImagesPermissions = ContextCompat.checkSelfPermission(context, "android.permission.READ_MEDIA_IMAGES") == 0;
                  boolean hasReadMediaAudioPermissions = ContextCompat.checkSelfPermission(context, "android.permission.READ_MEDIA_AUDIO") == 0;
                  boolean hasReadMediaVideoPermissions = ContextCompat.checkSelfPermission(context, "android.permission.READ_MEDIA_VIDEO") == 0;

                  return hasReadMediaImagesPermissions && hasReadMediaAudioPermissions && hasReadMediaVideoPermissions;
              } else {
                  boolean hasRecordPermissions = ContextCompat.checkSelfPermission(context, "android.permission.RECORD_AUDIO") == 0;
                  boolean hasExternalStoragePerm = ContextCompat.checkSelfPermission(context, "android.permission.WRITE_EXTERNAL_STORAGE") == 0;
                  return hasRecordPermissions && hasExternalStoragePerm;
              }
          }

Basically that's the issue because I tested this and the recording it's working properly, I would help you

Any fix for this in the lib please

For android 14 or latter. You can just follow the @fernandoamz for the viro repo. Also, you need to fix the permission in the virocore repository. After you fix the permission issue, you need to rebuild the .aar file. Then you need to rebuild the package.
To re build the .aar file you can just follow the readme instruction in virocore repository.
To rebuild the react viro package just follow the documentation in react vision documentation site.
If you just build the project for experimental you can just force the permission check to returning true. 😁

@naufanituharish I can't build the local version after following the instructions on react vision documentation site

Could you share the error message?

Hi I think that I have the solution to this bug, basically the viro library internally is reviewing the permissions but on sdk above to 33 we need to ask for other permissions types, so, I fixed recently if you create your own react viro version and make a local installation
After you clone the rect viro repository (https://github.com/viromedia/viro) you will need to add the follow lines inside the:
android -> viro_bridge -> java -> src -> module -> ARSceneNavigatorModule.java
search the line 331:
if (audioAndRecordingPerm){
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
                reactActivity.requestPermissions(new String[]{
                        Manifest.permission.READ_MEDIA_IMAGES,
                        Manifest.permission.READ_MEDIA_AUDIO,
                        Manifest.permission.READ_MEDIA_VIDEO,
                        Manifest.permission.RECORD_AUDIO
                }, PERMISSION_REQ_CODE_AUDIO, listener);
            } else {
                reactActivity.requestPermissions(new String[]{Manifest.permission.WRITE_EXTERNAL_STORAGE,
                        Manifest.permission.RECORD_AUDIO}, PERMISSION_REQ_CODE_AUDIO, listener);
            }
        } else {
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
                reactActivity.requestPermissions(new String[]{
                        Manifest.permission.READ_MEDIA_IMAGES,
                        Manifest.permission.READ_MEDIA_AUDIO,
                        Manifest.permission.READ_MEDIA_VIDEO,
                        Manifest.permission.RECORD_AUDIO
                }, PERMISSION_REQ_CODE_AUDIO, listener);
            } else {
                reactActivity.requestPermissions(new String[]{Manifest.permission.WRITE_EXTERNAL_STORAGE},
                        PERMISSION_REQ_CODE_STORAGE, listener);
            }
        }

       private static boolean hasAudioAndRecordingPermissions(Context context) {
              if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
                  boolean hasReadMediaImagesPermissions = ContextCompat.checkSelfPermission(context, "android.permission.READ_MEDIA_IMAGES") == 0;
                  boolean hasReadMediaAudioPermissions = ContextCompat.checkSelfPermission(context, "android.permission.READ_MEDIA_AUDIO") == 0;
                  boolean hasReadMediaVideoPermissions = ContextCompat.checkSelfPermission(context, "android.permission.READ_MEDIA_VIDEO") == 0;

                  return hasReadMediaImagesPermissions && hasReadMediaAudioPermissions && hasReadMediaVideoPermissions;
              } else {
                  boolean hasRecordPermissions = ContextCompat.checkSelfPermission(context, "android.permission.RECORD_AUDIO") == 0;
                  boolean hasExternalStoragePerm = ContextCompat.checkSelfPermission(context, "android.permission.WRITE_EXTERNAL_STORAGE") == 0;
                  return hasRecordPermissions && hasExternalStoragePerm;
              }
          }

Basically that's the issue because it is reviewing internally the permissions before record I tested this and the recording it's working properly, I would help you. Would be great if some contributor could add this changes and thanks to @doranteseduardo he explain me how I can make my local viro version

Can you please share intructions here how to build this locally so I can use it with my project?

Any one can tell how to install viro for local ( with changing some part of code  as @saadxprt is doing ) step by step. Because the guide link in the repo are broken and instruction is not clear too.


The file path to a BoundFile object should be easily accessible

Improvement Description
Make BoundFile filepaths easily accessible.
Current Behavior
Their doesn't seem to be any easy way to access the path to a BoundFile object. Their repr just shows them as a Python object, and their name attribute isn't a filepath either.
Proposed Behavior
Create an easy way to access the path to a BoundFile
Question

Are there any technical limitations restricting the ability to make this a thing?

Comments

Probably just do this via the exposure of a new path parameter.

Is this issue confusing BoundFile with TextFileFormat/BinaryFileFormat? BoundFile already does have a view method, while the unbound formats (TextFileFormat and BinaryFileFormat) don't. That means you can't call view on a TextFileFormat unless it is a member of a DirectoryFormat (and thus, is a BoundFile).

This functionality does exist, currently - does this meet your needs, @Oddant1?
Using this Artifact as input.
import qiime2
from q2_types.per_sample_sequences import SingleLanePerSampleSingleEndFastqDirFmt

x = qiime2.Artifact.load('demux.qza').view(SingleLanePerSampleSingleEndFastqDirFmt)
print(x.manifest.path_maker())

Which returns:
/tmp/matthew/qiime2-archive-owop1d70/4dbc0b6f-0b9e-4449-a350-c415f3e30be7/data/MANIFEST

@thermokarst I could have sworn we talked about the path_maker and it did something else, but that being the case I'll get rid of this.


[DNM] Test workflow for zip_test

Codecov Report
Attention: 21 lines in your changes are missing coverage. Please review.

Comparison is base (1c6f02c) 97.74% compared to head (0e1928e) 97.36%.

:exclamation: Current head 0e1928e differs from pull request most recent head cf92d7b. Consider uploading reports for the commit cf92d7b to get more accurate results

Files
Patch %
Lines

...ources/OpenCombine/Publishers/Publishers.Zip.swift
89.50%
21 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
- Coverage   97.74%   97.36%   -0.38%
==========================================
  Files         106      105       -1
  Lines        7633     7640       +7
==========================================
- Hits         7461     7439      -22
- Misses        172      201      +29

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


Update to support newer Three.js API and Blender-exported models

Thanks for making this module! Saved me a ton of headaches. 😁
Seems like it needs a bit of updates to support the newest THREE.js, and I noticed there were a couple bugs that prevented me from loading a Blender-exported X3D file.
I've tested these changes against a couple of my own models, but I'd welcome further scrutiny.

Thanks for the PR! Since we don't have any tests in this repo I'm a bit unsure about merging stuff but you changes seem small and well documented. I also ran the example and it works with your PR so I'll go ahead and merge this.

Published as 0.4.0.


Add an alternative locust client

Ⅰ. What this PR does / why we need it
Add locust client as an alternative, support both gRPC and HTTP

[pkg/client_locust/image.jpg] if this is used for demo, pls rmv it to the demo folder
[pkg/client_locust] pls provide docs(e.g., readme) and demo here, showing how to use the new feature
the pull request tittle: Add an alternative locust client. pls provide detailed description for this pull request: the motivation, new features provided, etc

I am not sure whether should I remove [pkg/client_locust/image.jpg].  It's just like the file [pkg/client/image.jpg] and will be used by the demo client.
README.md is added now, new features are as follows:

simpler test file needed
both gRPC and HTTP supported
load test params could be specified in yaml
prometheus metrics export supported

I got it, changing it now
let me know what I should further change then~ thanks for your patience.


README.md example doesn't work anymore.

Using this as compared to the example in the README.md file I still get some errors.
{-# LANGUAGE OverloadedStrings, RecordWildCards #-}

import Data.Text
import Pipes

-- import Language.Discord -- Not included with the package anymore.
import Network.Discord

reply :: Message -> Text -> Effect DiscordM ()
reply Message{messageChannel=chan} cont = fetch' $ CreateMessage chan cont

main :: IO ()
main = runBot "TOKEN" $ do
  with ReadyEvent $ \(Init v u _ _ _) ->
    liftIO . putStrLn $ "Connected to gateway v" ++ show v ++ " as user " ++ show u

  with MessageCreateEvent $<EMAIL_ADDRESS>-> do
    when ("Ping" `isPrefixOf` messageContent && (not . userIsBot $ messageAuthor)) $
      reply msg "Pong!"

and the error
tak@moon ~/dev/projects/sodimm/dimmy % sb                                      :(
dimmy-0.1.0: build (exe)
Preprocessing executable 'dimmy' for dimmy-0.1.0...
[1 of 1] Compiling Main             ( Dimmy.lhs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/dimmy/dimmy-tmp/Main.o )

/home/tak/dev/projects/sodimm/dimmy/Dimmy.lhs:9:45: error:
    • No instance for (DoFetch (Maybe Embed -> ChannelRequest Message))
        arising from a use of ‘fetch'’
        (maybe you haven't applied a function to enough arguments?)
    • In the expression: fetch' $ CreateMessage chan cont
      In an equation for ‘reply’:
          reply (Message {messageChannel = chan}) cont
            = fetch' $ CreateMessage chan cont

/home/tak/dev/projects/sodimm/dimmy/Dimmy.lhs:12:17: error:
    • No instance for (Data.String.IsString Auth)
        arising from the literal ‘"TOKEN"’
    • In the first argument of ‘runBot’, namely ‘"TOKEN"’
      In the expression: runBot "TOKEN"
      In the expression:
        runBot "TOKEN"
        $ do { with ReadyEvent
               $ \ (Init v u _ _ _)
                   -> liftIO . putStrLn
                      $ "Connected to gateway v" ++ show v ++ " as user " ++ show u;
               with MessageCreateEvent $ \ msg@(Message {..}) -> do { ... } }

--  While building package dimmy-0.1.0 using:
      /home/tak/.stack/setup-exe-cache/x86_64-linux/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.0.2 --builddir=.stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS> build exe:dimmy --ghc-options " -ddump-hi -ddump-to-file"
    Process exited with code: ExitFailure 1

I'll play around with it and submit a PR if I figure it out, but it may just be a simple fix.

The fixed example is in examples/pingpong.hs, I just forgot to move it over.


Update README.md of yahoo_answers_topics dataset

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks, @borgr. We have removed all dataset scripts from this repo. Subsequent PRs should be opened directly on the Hugging Face Hub.

Do you mean to edit through "edit dataset card" button? because it just leads to a broken page...
https://huggingface.co/datasets/yahoo_answers_topics

https://github.com/huggingface/datasets/tree/main/datasets/yahoo_answers_topics


GUI: TFTP Code update: Basic variable definition

Looks like this is not needed, we have base variables are defined in https://github.com/openbmc/openbmc-test-automation/blob/633b9ce244f7aa0840e99a89668dc0b0d03aa4d9/lib/resource.txt
Will use the same.

GUI: There is no separate PNOR / BMC file name. Will use ${IMAGE_FILE_PATH}  and ${TFTP_SERVER}  parameters.


0 5 3 sam

For some reason these tests run reaaaaly slowly.
But they run.
@liutuo368 this will probably give us a good indication of whats required for the django 4.2 upgrade.


I am interested in using your game on my website. Could you add a license file like the MIT one in this pull request so I know how you want your software used?

Feel free to add changes or suggest changes! Thank you.


permissions all messed up out of nowhere.

hi, so i'm not sure how to use verbose since that's what someone said earlier but let me explain my issue further
i know not all of this is relevant to luckperms but humor me since you might be abe to see a correlation that i cant.
i noticed after coming online today that 1 my worldguard had just gone and evaporated
the plugin was still there but the regions were gone
2 the npc's that i had made last night were gone, the ones from before remained (as a sidenote, we set up the worldguard 2 days ago on this particular server and a rollback of the server itself doesnt seem plausible because all the kingdoms (its a kingdoms server) and their progression were untouched
and i only mention these 2 issues that i have fixed because maybe you fine folk can see a correlation that i cant but besides that the issue that does relate to luckperms is the fact that my groups still exist, suffixes, prefixes all this is still set.
the people that were op are op etc, BUT me and others have lost the ability to /fly, /fly others, go into gamemode etcetera
its like the permissions just flipped
the people that shouldnt be able to fly were flying and the ones that should werent
causing a fair bit of mayhem and eventual bans sadly
i tried to fix it myself by manually readding the permissions to the groups, this did not work at all
i'm in the owner group and despite giving myself essentials.fly or essentials.fly.use, essentials.fly.* or essentials.flyy.others i still could not fly
i did manage to get my creative to work
and for the default roles to no longer be able to use gamemode
but only by taking it away user per user from them
and not by groups

Sounds like something happened to your database, causing it to drop luckperms data, worldguard data etc. Not really a luckperms issue

I think @codingJWilliams 's suggestion is most likely what happened, but I'm just guessing too. :(
Without knowing specifics about individual broken permission assignments, etc, there's not really much we can do to help you. :/


Invite new staff user scroll is broken

Issue Summary
Hi. I am just starting with ghost and playing around the dashboard. I have discovered that when you want to invite new staff members, the scroll behaviour of the modal/popup is broken - not scrolling at all.
Steps to Reproduce
Open Settings -> Staff -> Invite people

Ghost Version
4.32.2
Node.js Version
v16.13.2
How did you install Ghost?
I have followed https://ghost.org/docs/install/local/. macOS Sierra
Database type
SQLite3
Browser & OS version
Google chrome 97.0.4692.71, mocOS Sierra
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Hi. I have found the problem and fixed it locally. I will have a look this week into the contribution guidelines and raise a PR.

Hey @zsid, the issue has just been fixed in this commit – thanks a lot for creating the issue and helping out!


feat: add miner mode change by method

PR Type
Enhancement, Bug fix

Description

Implemented a new RPC method stratus_change_miner_mode to allow dynamic changes to the miner mode.
Refactored miner mode handling to use RwLock<MinerMode> for thread-safe access.
Added comprehensive error handling for poisoned locks throughout the codebase.
Introduced new error types in StratusError for miner mode conflicts, changes, and related operations.
Implemented interval miner shutdown management with new global state variables and functions.
Updated executor and miner configuration to work with the new miner mode management system.
Improved consensus-related error handling and checks in various parts of the code.
Added safeguards to prevent miner mode changes when transactions are enabled or when there are pending transactions.

Changes walkthrough 📝
Relevant filesEnhancement

executor.rsRefactor miner mode handling in executor

src/eth/executor/executor.rs
Improved handling of miner mode changes  Added error handling for poisoned locks  Simplified logic for miner lock acquisition

+16/-8

miner.rsImprove miner mode management and error handling

src/eth/miner/miner.rs
Changed mode field to use RwLock  Added error handling for poisoned locks  Implemented new should_shutdown function  Updated interval miner spawning logic

+43/-18

miner_config.rsEnhance miner configuration and mode handling

src/eth/miner/miner_config.rs
Added error handling for poisoned locks  Updated miner mode checking logic  Made MinerMode derive PartialEq and serde::Deserialize

+12/-3

stratus_error.rsAdd new error types for miner and consensus operations

src/eth/primitives/stratus_error.rs
Added new error types for miner mode conflicts and changes  Introduced errors for pending transactions and consensus states

+27/-0

rpc_server.rsImplement miner mode change RPC and improve error handling

src/eth/rpc/rpc_server.rs
Implemented stratus_change_miner_mode RPC method  Added error handling for poisoned locks  Updated consensus and miner mode change logic

+108/-4

globals.rsAdd interval miner shutdown management

src/globals.rs
Added INTERVAL_MINER_SHUTDOWN atomic boolean  Implemented functions to manage interval miner shutdown state  Updated GlobalState struct and JSON representation

+34/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/describe

/describe

/describe


Develop

Updated docs/use_cases///*.py files with links in the mouse-over section.

This is a pull request from develop into master_v3.0. @TaraJensen, did you do the work in a feature branch? The pull request should go from the feature branch (on the right) to the develop branch (on the left).


Need a nodes list to continue syncing my wallet please

Hello,
I need a nodes list to continue syncing my wallet please.
Thank you :)

<IP_ADDRESS>:8333
<IP_ADDRESS>:8333
<IP_ADDRESS>:8333

root:~# nslookup ip.ub.com
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53
Non-authoritative answer:
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>
Name:   ip.ub.com
Address: <IP_ADDRESS>


Logging doesn't seem to do anything

I am following https://github.com/VirtusLab/besom/blob/develop/examples/aws-webserver/Main.scala
It works just fine, with the exception of logging. I am not well versed in Pulumi, but none of log.warn, log.info or log.error has any effect on the output at all. I even tried with pulumi up -v=10 --logtostderr, but still with no luck.
To be more precise, I am using the code from the mentioned example, with slight modification in the main for:
for {
    server: ec2.Instance <- server
    _                    <- sshKey
    _                    <- keyPair
    _                    <- log.warn("Anything WARN")
    _                    <- log.error("Anything ERROR")
    _                    <- p"Connect to SSH the first time: pulumi stack output privateKey --show-secrets > key_rsa && chmod 400 key_rsa && ssh -i key_rsa ${userName}@${server.publicIp}".map(log.info(_))
    _                    <- log.info("Connect to SSH: ssh -i key_rsa ec2-user@$(pulumi stack output publicIp)")
    _                    <- log.info("Connect to HTTP: open http://$(pulumi stack output publicHostName)")
  } yield Pulumi.exports(
    publicKey = publicKey,
    privateKey = privateKey,
    publicIp = server.publicIp,
    publicHostName = server.publicDns,
  )

So that rules out the possibility of unused Output

That's sadly true in 0.1.0. There's a very tricky bug related to dispatch on overridden methods in BesomLogger implementation. That logger sends all logs to two sinks, a grpc based Pulumi engine one and a normal file based one. Due to aforementioned bug in 0.1.0 only sink to receive logs is the file one. This is fixed in develop but not released yet. You can enable file logging though using PULUMI_ENABLE_TRACE_LOGGING_TO_FILE env variable set to true.
Sorry!

Thanks for the update!

Release of 0.2.0 that publishes fix for this problem is now underway. Closing the issue.


Looking for maintainers

@danethurber just added me as a collaborators to this project
I think it will be better if I'm not the only one making the decisions on this project! :)
So if you are ready to spend some time maintaining this project, ping me here
ping @jordansexton @gmac @andreicek @tikotzky @martynovs @nickescallon @modosc  @iliakan @a-x- @davidmpaz @Lobstrosity @weaverryan @craigmichaelmartin @wtgtybhertgeghgtwtg @orbiteleven

I have a few ideas that could help, sure, but how did my name come up?  I don't think I've ever touched this project.
Did you confuse my name for somebody else's?  Happens all the time.

https://github.com/danethurber/webpack-manifest-plugin/pull/32

@wtgtybhertgeghgtwtg I pinged everybody who contributed / opened a PR

I can definitely spend some time reviewing PRs, but as I am not an expert in how WebPack works under the hood this would be mostly general logic checking/analyzing + code style reviews

+1, Me too
Let's try

I'm quite familiar with webpack, not that much with the history of webpack-manifest-plugin
So I guess with the three of us we have a good mixed team ;)
I will ask Dane to add you as Collaborator, the only rule is don't push to master changes impacting the plugin's logic

I'm closing this now, but feel free to reach out here if you are willing to help ;)

I don't mind helping out
On Mon, Jul 10, 2017 at 8:18 AM Thomas Sileghem<EMAIL_ADDRESS>wrote:

Closed #49
https://github.com/danethurber/webpack-manifest-plugin/issues/49.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/danethurber/webpack-manifest-plugin/issues/49#event-1156675216,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAsYuLYGBpJ2dDyCPK95Cz9lhqDn-Cbgks5sMhaRgaJpZM4OMaqm
.

@gmac definitely, I think you can help on 2 and 3 of #51

@gmac @a-x- @zuzusik I created a slack channel here: https://webpack.slack.com/messages/C6CCZ0DSS
Anyone else welcome of course ;)

Can you create an invite to<EMAIL_ADDRESS>

I asked @d3viant0ne to add all of you and @danethurber on slack

Alright, so I'm created a gitter instead: https://gitter.im/webpack-manifest-plugin#


Typesafety

Not sure if this is a good place to ask questions, but here it is: Does the library provide some typesafety when working with units? I've done some small tests using the library but both speed and acceleration seem to be of type Expression :
val someSpeed: Expression = 2.m/s
val someAcceleration: Expression = 2.m/(s*s)

So it seems I can not use the library to provide a type-safe speed argument for a car:
data class Speed(val kmh: Number) // TODO Can we use si-units lib here instead in a typesafe manner?
data class Car(val speed: Speed)

Can someone clarify how to use the API correctly

You can define your own units that have nothing to do with SI units, as was done with artificial Unit Thing. (See [here](https://github.com/sheepdreamofandroids/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonMain/kotlin/eu/sirotin/kotunil/specialunits/Thing.kt
https://github.com/vsirotin/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonMain/kotlin/eu/sirotin/kotunil/specialunits/Thing.kt) and [here](https://github.com/sheepdreamofandroids/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonTest/kotlin/eu/sirotin/kotunil/specialunits/ThingTest.kt
https://github.com/vsirotin/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonTest/kotlin/eu/sirotin/kotunil/specialunits/ThingTest.kt) ).
You can define your own derived units as shown below. See also https://github.com/sheepdreamofandroids/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonTest/kotlin/eu/sirotin/kotunil/physics/KmH.kt
https://github.com/vsirotin/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonTest/kotlin/eu/sirotin/kotunil/physics/KmH.kt and https://github.com/sheepdreamofandroids/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonTest/kotlin/eu/sirotin/kotunil/physics/SpeedTest.kt
https://github.com/vsirotin/si-units/blob/6283b7edc3b39e1275f7a7111ecc709e8ae5c402/kotunil/src/commonTest/kotlin/eu/sirotin/kotunil/physics/SpeedTest.kt.

private val formula =  km/h

//Unit for speed in km/h.
class KmH(value: Number): DerivedUnit(value, formula)

 //Unit 1 km/h.
val kmh = KmH(1.0)

// Transforms number in km/h.
val Number.kmh : KmH
    get() = KmH(this.toDouble())

Test:

internal class Car(val midSpeed: KmH){
    fun distanceForHours(duration: Second): Expression = (this.midSpeed*duration)
}

internal class SpeedTest {

    private val porsche = Car(200.kmh)

    @Test
    fun testDistance() {

        val d = porsche.distanceForHours(1.5.h)
        assertEquals(300.km, d)
        assertEquals(300_000.0, d.value)
        assertEquals("m", d.unitSymbols())
        assertEquals("300000.0 m", d.toString())
     }

    @Test
    fun testTypes() {

        assertEquals(porsche.midSpeed, 200*kmh)

    }
}

I hope, it can help you.

Thanks for the example. I think I stated my question not well. I struggle that in
fun distanceForHours(duration: Second): Expression = (this.midSpeed*duration)

the return type is not something like Distance or Length which is the correct measurement when multiplying a speed with a duration.
When having a library called si-units I'd think/hope it to be more strongly typed. So more technically, when overloading operators, it should take the SI system into account to define correct return types.

Thanks for the kind explanation/clarification, @vsirotin.
I initially misread the API intent. I believed, similar to kotlin.time.Duration, the API would provide types for measurements (mass, length, luminous intensity). It seemed to me that unit and measurement were intertwined. Specifically, I thought that the product of speed and duration would result in a type like Length, which could be expressed in meters (someLength.inMeters). I understand that expressing this in a type system, such as Kotlin's, is challenging and currently out of scope.
Please excuse my lack of expertise here, in case you had addressed this question already above.

If you need it very much and you are ready to participate in it, we can think about creating a library on the basis of KotUniL for kinematics with objects like coordinate, velocity, acceleration, angular velocity, angular acceleration, etc.

I hope, the issue can be closed now.

Yes thanks for the clarification and kind support.


License

Would you add a license, please? See http://choosealicense.com/ for more information.

Thanks for the reminder, I'm reviewing the options at the moment !

Now licensed under GPLv2, closing the issue

GPLv3 is probably a better choice.

AGPLv3 seems more relaxed, which I prefer. So I switched to it, thanks for the suggestion.
@Calinou : any reason you particularly care about this ? Do you need to reuse the code/program somewhere ?

@Calinou : any reason you particularly care about this ? Do you need to reuse the code/program somewhere ?

Not particularly – but GPLv3 and AGPLv3 work better to defend user freedom than GPLv2 does.

Ok, thanks for the advice !
I've never carefully thought about licensing, so I'm still a bit lost. But AGPLv3 should be OK.


setAttribute does not accept key as integer

Laravel-mongodb Version: 10.25.2
PHP Version: 8.2.11
Database Driver & Version: 4

Description:
When triyng to set a key of object as integer on update it throws a type error
Steps to reproduce
Set a key as integer on updateOrCreate
Model::updateOrCreate(['_id' => 1], [ 1 => "value"])
Actual behaviour
It throws an error in MongoDB\Laravel\Eloquent\Model::216
str_contains(): Argument #1 ($haystack) must be of type string, int given
My sugesstion is change the code from
if (str_contains($key, '.')) {
to
if (is_string($key) && str_contains($key, '.')) {

Strictly speaking, keys have to be strings, but I think the handling of invalid keys could be better (e.g. by casting integers to strings). Care to create a pull request with a test case?


Add form tag field option (for v2 branch)

I've added an override option for the tag-slot @ multiple-select. For example: I've had the need to trim the labels of the selected values as I was using indents to a page-tree dropdown (indents per level).
See: Multiple select
Not sure if' it's wanted, PR is here in any case :)
PS. I'll add a PR for master (Nova v4) as well, but I haven't upgraded yet, so that PR is not tested yet.

Hey @gavro! Thanks a ton for the PR. I released it in version 2.4.2 for Nova 3 as well now. Good luck!


Issue

** removed **

Hello, and thank you for your issue. It seems you may be trying to report some kind of security-related issue. Please try and report it privately if possible before posting publicly. You can find the expressjs project policy at https://expressjs.com/en/resources/contributing.html#security-policies-and-procedures
You are welcome to email it to me.


RFC: Deprecate type aliases in std::os::*::raw

Deprecate type aliases and structs in std::os::$platform::raw in favor of
trait-based accessors which return Rust types rather than the equivalent C type
aliases.

Rendered

:bell: This RFC is now entering its week-long final comment period :bell:

The libs team discussed this today and the conclusion was that there is not a clear enough path moving forward that doesn't involve theoretical breakage. We decided to leave this in FCP while I gather some statistics and flesh out a more concrete plan.

Ok, I've done some analysis of what this change might have. The numbers here were gathered against these changes to the standard library, which I believe fully implement this RFC (enabling us to move to LFS on Linux at the same time).
A crater report reports four regressions, three of which were spurious. I have submitted a PR for the other regression. Specifically, this regression involved code that looked like:
libc::foo(..., value as std::os::raw::off_t)

In other words, the regression came from the reliance that the types in std agreed with those in libc, which this RFC would be breaking.
I then compiled Servo against a standard library with the patches above applied. With the fs2-rs changes Servo compiled for both x86_64-unknown-linux-gnu and arm-linux-androideabi with no other modifications necessary.
Finally, I did a manual survey of all code on crates.io for usage of MetadataExt or the std::os::*::raw types. The investigation revealed two cases which would break

fs2-0.2.2 - visible breakage

passed a std off_t into a libc off_t
fixed in https://github.com/danburkert/fs2-rs/pull/2

gphoto2-sys-0.1.1 - silent breakage

uses time_t in FFI
fixed in https://github.com/dcuddeback/gphoto2-sys/pull/1 (now merged)

The following usage was all observed but would not break from the change in this RFC (confirmed from the crater run)

cargo-extras-0.3.0

casts .mtime() to u64 immediately

cargo-script-0.1.4

casts .mtime() to u64 immediately

devicemapper-0.3.2

mode() & constant == constant
assumes rdev is u64 (ok)

filetime-0.1.9

immediately casts
only user of as_raw_stat

fs2-0.2.2

immediately casts

lalrpop-0.9.0

compares mtime() ot mtime()

lalrpop-snap-0.9.0

compares mtime() ot mtime()

logwatcher-0.1.0

assumes ino() returns u64 (ok)

pnacl-build-helper-1.4.10

immediately casts mtime()

psutil-1.0.0

reads uid/gid

systemd-crontab-generator-1.0.0-rc6

reads uid
casts mtime

tango-0.3.3

casts mtime immediately
prints mtime

utime-0.1.3

casts atime/mtime immediately
equates atime/mtime with constant

walkdir-0.1.5

compares dev/ino with another dev/ino

keyutils-0.2.0

uses uid/gid

procinfo-0.1.1

uses uid/gid/pid

tar-0.4.0

creates permissions from a mode_t

My conclusion from this is that we can likely take the path set forth in this RFC (aka merge the two commits above at the same time). In summary, they:

Deprecate all raw types that aren't in std::os::raw
Expand the MetadataExt trait on all platforms for method accessors of all fields. The fields now returned widened types which are the same across platforms (consistency across platforms is not required, however, it's just convenient)
[breaking] Change the definition of all std::os::*::raw type aliases to correspond to the newly widened types that are being returned on each platform.
[breaking] Change the definition of std::os::*::raw::stat on Linux to match the LFS definitions rather than the standard ones.

The breaking changes are specifically breaking if you assume std/libc or std/gcc agree on the types, which from the above analysis looks like is limited to just a few crates (both of which have PRs to be updated).

We discussed this during libs team triage today, and the conclusion was that given the limited fall out we should push forward with this. Thanks again for the discussion everyone!

:tada:

I’m not sure what’s the procedure for this but, is it possible to plan to remove/forbid calling e.g. MetadataExt::as_raw_stat() in a future edition?
I messed around with the code to be able to use statx (to get creation time on Linux) instead of stat64 but I’m stuck because of this API…
https://github.com/rust-lang/rust/pull/61386

The standard library cannot change in this way between editions; we can deprecate things but not remove them.

On Oct 3, 2019, at 7:19 PM, Mélanie Chauvel (ariasuni<EMAIL_ADDRESS>wrote:

I’m not sure what’s the procedure for this but, is it possible to plan to remove/forbid calling e.g. MetadataExt::as_raw_stat() in a future edition?
I messed around with the code to be able to use statx (to get creation time on Linux) instead of stat64 but I’m stuck because of this API…
rust-lang/rust#61386
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


✨ feat(Complete JavaScript): All JS concepts

Added Complete Module for Almost complete JavaScript.

Topics added:

Values and Variables- Premitive data types- Numbers
How JavaScript works?
Premitive data types- Boolean, Null, Undefined, String
Control and logic flow- if/else, switch, ternary operator, logical operators etc.
Arrays, array methods
Objects in JavaScript
Loops in JavaScript
Functions in JavaScript
Scope,Function expressions, Higher Order functions, callbacks, hoisting
Array callback methods- filter, forEach, reduce, map, some, every etc.
Default Parameters, Spreads,Rest Parameters, Destructuring
Object Methods and the "This" Keyword
Document Object Model
Communicating with Events
Asynchronous JavaScript, Callbacks and Promises
Making http requests: Fetch, Axios
Async and Await
Prototypes, Classes, & the new operator
JavaScript with Node.js

@microsoft-github-policy-service agree [company="default"]

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

Thank you for your suggestion to add these JS examples and lesson! We are currently not expanding this to any new lessons and projects. But stay tuned for when we open contributions for additional lessons, and example web apps to build. For now, sharing your creations within Discussions would be amazing!


Add support for TestNG

@dragisak, could you share how you would do it? Would help others in the future.

For a record, I was able to create a trait in Groovy, which just needs implementing.
trait HttpMock {
    WireMockServer wireMockServer
    WireMock wireMock

    @BeforeMethod
    void start(){
        wireMockServer = new WireMockServer(wireMockConfig().port(httpPort));
        wireMockServer.start();
        WireMock.configureFor("localhost", httpPort);
        wireMock = new WireMock("localhost", httpPort);
    }
    @AfterMethod
    void stop(){
        wireMockServer.stop();
    }
    abstract int getHttpPort()
}

A blast from the past. :-)
It's been a couple of years since I last did TestNG. Completely forgot the context of my question here. Thanks for posting the solution !


trailing slash removed when navigating

Describe the bug
I would like my path to end up with a trailing slash.
Currently with these lines, svelte kit always remove it : https://github.com/sveltejs/kit/blob/84e9023c06dc3579972f93e0d5cf1f2481aa3be4/packages/kit/src/runtime/client/router.js#L185-L188
Expected behavior
This behavior should at least be configurable.
Having a trailing slash is important to me as ipfs gateways append it anyway and so without being able to keep the trailing slash, there is a different behaviour between SPA navigation and navigation through request.
Plus it cause issue to calculate relative path as without slash the resource is now considered to be in the parent folder
I think trailing slash are more accurate in that regard since on a static website, the corresponding underlying index.html file is in its own folder.
Severity
This is a blocker for me that deploy website on ipfs
I am working on a ipfs adapter to support ipfs properly and I am currently stuck with this.

The adapater repo is here : https://github.com/wighawag/sveltejs-adapater-ipfs
See readme for what it actually does : https://github.com/wighawag/sveltejs-adapater-ipfs/blob/main/README.md
The test app is here :  (branch : adpater-ipfs) can be found here : https://github.com/wighawag/svelte-kit-ipfs-example/tree/adapter-ipfs
With the current modif it works but ideally I would not need to replace generated code as this is brittle

previous discussion here, which lead to #267. from there:

Starting with no slashes for the beta and waiting until we get a use case for something else sounds like a sensible plan to me

Sounds like we have a use case!
What we don't have is a sense of how this should be configured. Is an app-wide setting sufficient, or does it need to be per-page? What sort of thing did you have in mind?

Hi @Rich-Harris thanks for the link to past discussion

What we don't have is a sense of how this should be configured. Is an app-wide setting sufficient, or does it need to be per-page?

For my use case (ipfs hosting), an app-wide config would be sufficient
ipfs gateway force the trailing slash when an index.html is detected and used.
The trailing slash make it easier to compute relative path too

Another use case is SEO, the same route with slash and without would be indexed as two different pages by a crawlbot, therefore deciding to redirect to either or makes sense.

FWIW regarding relative paths, I borrowed/tweaked this url building from routify's helpers
to better handle them
import { derived, get } from 'svelte/store';
import { page } from '$app/stores';
import { goto as gotoApp } from '$app/navigation';

export function url(path: string) {
  const $page = get(page);

  if (path.match(/^\.\.?\//)) {
    // Relative path (starts with `./` or `../`)
    let [, breadcrumbs, relativePath] = path.match(/^([\.\/]+)(.*)/);
    let dir = $page.path.replace(/\/$/, '');
    const traverse = breadcrumbs.match(/\.\.\//g) || [];
    // if this is a page, we want to traverse one step back to its folder
    traverse.forEach(() => (dir = dir.replace(/\/[^\/]+\/?$/, '')));
    path = `${dir}/${relativePath}`.replace(/\/$/, '');
    path = path || '/'; // empty means root
    console.groupEnd();
  } else if (path.match(/^\//)) {
    // Absolute path (starts with `/`)
    return path;
  } else {
    // Unknown
    return path;
  }

  return path;
}

export function goto(path: string) {
  const newPath = url(path);
  return gotoApp(newPath);
}

Examples:
<a href={url(`./${item.Id}`)}>Child / Subitem path
<a href={url(`../${key}`)}>Sibling path
Parent path (back to list, etc)
Grandparent path

What's wrong with just using relative paths?

Another use case to to not remove the trailing spaces, is how Netlify handles URLs. URLs without a trailing spaces will automatically get redirected (301) to the one with a trailing slash, as you can see in the below screenshot. This is impacting performance a little bit, which I measured with web.dev.

(FYI, I am using the static adapter to deploy to Netlify).

Netlify redirects tries the exact path in the request and if that is not available it will redirect to trailing slash or non-trailing slash whichever one is available.

I've been looking into SvelteKit routing deeper recently for https://github.com/sveltejs/kit/pull/943.
And I think I understand some of the reasoning behind the current design. Making routes/category/index.svelte responsible for /category achieves the following:

have $layout.svelte specific to that path
have hooks specific to that path

Because there's no other way to scope those two features down to a specific path if we're using filesystem based routing.
The trailing slash and non-trailing slash URLs can exist at the same time and support existing features in a couple of different scenarios. E.g.

Using $index.svelte file for trailing slash:

routes/category/index.svelte → /category
routes/category/$index.svelte → /category/

The problem with this approach is that /category and /category/ can't have different hooks and layouts.

Using $index directory for trailing slash

routes/category/index.svelte → /category
routes/category/$index/index.svelte → /category/

This might be a little confusing at first glance but allows scoping of layout and hooks.

Using $ directory for trailing slash

routes/category/index.svelte → /category
routes/$category/index.svelte → /category/

I think this probably the cleanest. We could also go with # as prefix for directory name.
I think any of these can be implemented without breaking existing behavior. I'm willing to put in the work and implement this since we have a need for this kind of routing for a website we want to port to SlvelteKit. But I'd like to get a go ahead from SvelteKit authors and maintainers first.

@RaeesBhatti It is an interesting approach to allow for both.
I still think we need a way to have that configurable globally though and let user use normal folder name if desired. This is especially important for ipfs gateway which do not allow for path that do not ends with slash. They get redirected to path with a trailing slash. It would be confusing to force user to use $foldername.
Your proposal should be an opt-in option that let users use both trailing slash and non-trailing slash if they need it, so that use of trailing slash can continue to use the normal folder name.

@Conduitry could you chime in with your opinion on this!
I personally prefer option 3. It can be documented in the routing section to quell any confusion.

Another vote and use case for not stripping trailing slashes: write a blog inside of SvelteKit, use MDsvex for posts, and attempt to link to a resource in the same directory.
Write the post /my-first-article/index.svx. Reference a file inside the same directory via ![useful-alt-text](./my-image.png) and the link will be broken.  You are instead forced to write ![useful-alt-text](./my-first-article/my-image.png) and hope the directory never gets renamed/moved.  I know referencing images via imports is generally preferred, but that's not really ideal for content authoring and vite handles img tags just fine on its own.

This may be a more serious issue. It turns out that Netlify redirects to the trailing slash version and SvelteKit redirects to the version without trailing slash so you get an infinite redirect: https://github.com/sveltejs/kit/issues/1346

I actually have all the optimizations by Netlify disabled, so I would say not related to that.

trailingSlash: "ignore" works for all pages and lets Netlify handle the trailing slashes. <- this is what you want.

@mhatvan Are you referring to the "Pretty URLs" option under Deploy Settings > Asset optimization?

Interesting — I would have regarded that as a bug, not a feature. The fact that they're technically separate URLs has always felt like an accident of history; I can't imagine a situation in which you'd want to have /about and /about/ in the same app. Is there a reason to have that?

file browsing in browser,... if it's ending with symbol / it's folder, if not, it's file.

file browsing in browser (for example for cloud storages or repos, or torrent trackers),... if it's ending with symbol / it's folder, if not, it's file. That's used pretty often this way.

But in that case you're not going to have a new .svelte file for each file or directory. You're going to have a single .svelte file that acts as the file browser. That .svelte file can dynamically show different contents based on the URL including whether it has a slash or not. You wouldn't need any support in SvelteKit's file system router for that

@Rich-Harris Thanks for implementing trailingSlash: "never" | "always" | "ignore". I was wondering if it could somehow be possible to set this flag on a page basis? I imagine there are quite some websites which haven't implemented a consistent trailing slash strategy and just use a mix of everything.
I could imagine either exporting it in

What do you think?

@bdadam is it entirely necessary? I believe having an inconsistent behavior here here is an anti-pattern. Also with trailingSlash: ignore you can probably let your webserver handle all the required behavior by the rules you define.
Although SvelteKit is aiming to support services like Vercel or Netlify, where the webserver config is not available, I there are many other uses for the Kit that might not even involve aforementioned services (for instance waiting to be able to finally use things like https://github.com/svelte-add/tailwindcss in production env), but maybe I am wrong here.

file browsing in browser (for example for cloud storages or repos, or torrent trackers),... if it's ending with symbol / it's folder, if not, it's file. That's used pretty often this way.

But in that case you're not going to have a new .svelte file for each file or directory. You're going to have a single .svelte file that acts as the file browser. That .svelte file can dynamically show different contents based on the URL including whether it has a slash or not. You wouldn't need any support in SvelteKit's file system router for that

Agree, if we are talking about routes/*.svelte files,... but what if we are talking about routes/*.js (endpoint) files, when programmer write logic in it, that if there is "/" at the end of URL, it will provide content of folder, but if not, then content of file?

Agree, if we are talking about routes/.svelte files,... but what if we are talking about routes/.js (endpoint) files, when programmer write logic in it, that if there is "/" at the end of URL, it will provide content of folder, but if not, then content of file?

You can still do that today in SvelteKit it you'd like

I just filed an issue in the static adapter which IMHO would be easier to tackle if the file path determined what the output filename is, e.g. routes/hello.svelte => hello.html, routes/hello.html.svelte => hello.html.html, routes/hello/index.svelte => hello/index.html.

FYI: relative paths within a file like /xyz/index.svelte pointing to /subfolder/123 do not translate to /xyz/subfolder/123 for me (using ordinary  links). very weird and not what i was expecting. https://github.com/sveltejs/kit/issues/733#issuecomment-810177995 solution did however solve it. just leaving this here as i am not sure if this is a bug or just how svelte works.

Use case: Hosting on AWS S3.


Create rest methods

Add to the SkypeBot remaining methods from Skype Bot API for Messaging to send messages to the skype

Create

[ ] AttachmentMessage
[ ] ContactRelationUpdate
[ ] ConversationUpdate

Create base class Message with field activity. Based on value of field create a list of concrete inheritors

Created REST methods based on Swagger API


profiling and optimizing startup (#180886818)

Pull Request Description
Implement #180886818:

add profiling to GUI init

Important Notes

Checklist
Please include the following checklist in your PR:

[ ] The documentation has been updated if necessary.
[ ] All code conforms to the Scala, Java, and Rust style guides.
[ ] All documentation and configuration conforms to the markdown and YAML style guides.
[ ] All code has been tested where possible.

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

I found a better way to do this than my initial plan: I moved awaiting MSDF into start_and_forget, so that start_and_forget can await controller init and Application init in parallel internally. This will not affect the many test cases that involve Initializer, since the only interface change is moving the boundary between entry_point_ide and start_and_forget.


VlanInterfaceStatus obsolete

The ONF modelling for the VLAN interface does not contain any Status class.
The corresponding path needs to be deleted from the modelling.

VlanInterfaceStatus entirely removed from specification.


Add a toggle for interval matrix indexing #411

Added radio buttons for "Interval indexing": 0-indexing (default) and 1-indexing.

Missing the actual ref that would be manipulated by the v-model
Commits should be squashed into one
The value should be persisted in local storage

The commit message should end with ref #445.

Or do we actually have duplicate issues? #411 is just fine then.


Handling iso_level_x_LATCH

Hello,
Following my comment in this issue, I am trying to get keyd working with my dead key.
To recap everything, when I try to use keyd, it works mostly fine for the alpha layers and altGr (iso_level_3_switch),
but I have a deak key (let’s call it 1dk)  that is technically an iso_level_5_latch (one-shot modifier) to make diacritics like è é à ù ê î û ß ç or typographic characters like ’ or « —», « – ». This key is located on the [N] position on a QWERTY layout.
With keyd, AltGr functions normally, as expected. The 1dk also works, except —as far as tested— if a key has an overloadt. This prevents me from having mod-taps or layer-taps on some keys like [sdf] or space, which are of course the most convenient.
A workaround was proposed, but I either did not understand it correctly, or it did not work.
I tried two things :

setting up a virtual layer identical key

n = oneshotm(dk)

[dk]
e = e
a = a

making a one-shot layer to overide the 1dk

n = oneshot(dk)
[dk]
e = è
i = é
u = ù
  = ’

But I think I do not know how to call this dead key since it has no output…
I hope the issue is clear and the title explicit ; I am no developper myself, so i can be slow to understand… bear with me.
PS : not sure this is needed, but here is my layout :
the ** key is the iso_level_5_latch aka 1dk
how to read

┆ 2  6 ┃
┆ 1  5 ┃

1 : level 1 / normal
2 : level 2 / shift
5 : level 5 / iso_level_5
6 : level 6 / iso_level_5 shift

base: |
  ╭╌╌╌╌╌┰─────┬─────┬─────┬─────┬─────┰─────┬─────┬─────┬─────┬─────┰╌╌╌╌╌┬╌╌╌╌╌╮
  ┆ ¬   ┃ # ¡ │ « “ │ » ” │ ( „ │ )   ┃ @   │ + ÷ │ ^*ˇ │ / | │ * × ┃*µ   ┆ § ¶ ┆
  ┆ ~*~ ┃ 1 # │ 2 < │ 3 > │ 4 ( │ 5 ) ┃ 6 @ │ 7 ± │ 8*^ │ 9 \ │ 0 ° ┃ = ≠ ┆ % ‰ ┆
  ╰╌╌╌╌╌╂─────┼─────┼─────┼─────┼─────╂─────┼─────┼─────┼─────┼─────╂╌╌╌╌╌┼╌╌╌╌╌┤
  ·     ┃ B   │ M   │ P   │ O   │ W   ┃ Z   │ V   │ D   │ L   │ J   ┃ {   ┆ }*¤ ┆
  ·     ┃   â │   û │   î │   ô │   œ ┃     │   ŭ │     │     │     ┃ [   ┆ ] ¢ ┆
  ·     ┠─────┼─────┼─────┼─────┼─────╂─────┼─────┼─────┼─────┼─────╂╌╌╌╌╌┼╌╌╌╌╌┤
  ·     ┃ A   │ U   │ I   │ E   │ ; Ê ┃ C   │ T   │ S   │ R   │ N   ┃ "   ┆ & £ ┆
  ·     ┃   à │   ù │   é │   è │ , ê ┃   ç │   ™ │   ß │   ® │   ñ ┃ ' ` ┆ $ € ┆
  ╭╌╌╌╌╌╂─────┼─────┼─────┼─────┼─────╂─────┼─────┼─────┼─────┼─────╂╌╌╌╌╌┴╌╌╌╌╌╯
  ┆ \ ¦ ┃ ! – │ Y   │ X   │ : · │ K   ┃ ?

No worries at all!

This key is located on the [N] position on a QWERTY layout.

I didn't get this the first time - so it is actually not AltGr that latches Level5, but the N key?
The important thing to remember here is that there are many points at which you can modify the behaviour of your keyboard. Keyd works at a rather low level --- it captures your keypresses and presents them through a 'new' virtual keyboard --- but you can also reinterpret keys at the display server level (this is where the ISO_Level_X keys reside) or at the application level.
It sounds like you've changed the interpretation of keys at the display server level and also wish do so some remapping on keyd's level, which gets confusing fast, but it can probably be done. However, if it's possible to do everything on the same level, I do recommend that instead! Perhaps you don't need keyd.
I suggested the following strategy in the linked issue:
[main]
n = oneshotm(dk, n)

[dk]
e = e
a = a

That is, n will still emit n (because that is needed for the display server to engage the ISO_Level5_Latch), but it will also (for the next keypress) enter a layer in which e and a are not overloaded.
I think that should work, but it was based on a guess of how your config is set up. If it doesn't work, can you tell me what you've used to set up the dead key; what your keyd config is; what the output of keyd monitor is when you try to use your 1dk with keyd; and also what the output of keyd monitor is when you use it without keyd running?

As an aside, the oneshotm(dk) in your attempted configs isn't a valid binding, because it's missing a macro as the second argument. e = è et al are valid bindings, but do require that you've set up the compose key as detailed in the manual.   = ’ is not a valid binding; you probably mean space = ’

Closing due to inactivity.


Build warnings with clang-cl on Windows.

Attempting to build with clang on Windows (via clang-cl) fails due to this issue with POSIX names. I believe the solution is either to try to disable the relevant warnings in as targeted a way as possible, or wrap the affected functions (e.g., strdup is where I'm hitting failures at the moment) in something with platform-specific implementations that call the _-prefixed versions on Windows.
This is related to https://github.com/flutter/flutter/issues/16256

/cc @ZichangG

I'll have one or more patches shortly to post for discussion. There were some other things caught by clang, at least one of which look like a real bug.

https://dart-review.googlesource.com/c/sdk/+/134502 has the first part, which is the fixes that aren't really specific to supporting clang, but rather general issues found when compiling with clang, so I expect to be uncontroversial.
The next part will require some small GN changes, but I'm still adjusting it to make those changes minimal.

https://dart-review.googlesource.com/c/sdk/+/134781 has the second part. With those two changes I'm able to build Flutter using clang on Windows.
There's a larger question of whether Dart's buildroot should support this directly, and whether it should be part of CI.

@a-siva Could you recommend a reviewer for the second patch above? It's the only remaining source-level blocker for building the Flutter engine with clang.

Siva is out for vacation.
@zanderso Do you know the right person to review?

Ah, sorry, I left a comment, but I don't know who should review. Maybe @rmacnak-google ?

Ping again on the review above.

Another issue that only showed up in certain build configurations: https://dart-review.googlesource.com/c/sdk/+/144520
That's a hard blocker for us switching to clang (unlike the previous CL that's still unreviewed, which we were able to work around), so if someone could route it to a reviewer that would be great. It's also a potentially serious bug for non-clang, FWIW, and apparently we've just been lucky so far.

The warning suppression CL above is abandoned in favor of fixing the calls to uses non-deprecated versions:

https://dart-review.googlesource.com/c/sdk/+/151394
https://dart-review.googlesource.com/c/sdk/+/151362

(Not exhaustive; more minor cleanup to follow)

Alternately, https://dart-review.googlesource.com/c/sdk/+/142704 will fix everything. (It turned out I happened to come back to this when it was already in progress on the Dart team side.)

The PR above didn't quite fix everything, it looks like. Here's what we're seeing in a recent Flutter build:
ANGLE roll is finally in. A bot build log is now a much more reasonable 6k lines and 500kb.
The remaining warnings from the run with the ANGLE fixes are:
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime/bin/gen_snapshot.cc(632,26): warning: 'strdup' is deprecated: The POSIX name for this item is deprecated. Instead, use the ISO C and C++ conformant name: _strdup. See online help for details. [-Wdeprecated-declarations]
../../third_party/dart/runtime/bin/gen_snapshot.cc(641,15): warning: 'strdup' is deprecated: The POSIX name for this item is deprecated. Instead, use the ISO C and C++ conformant name: _strdup. See online help for details. [-Wdeprecated-declarations]
2 warnings generated.
../../third_party/dart/runtime/bin/gen_snapshot.cc(632,26): warning: 'strdup' is deprecated: The POSIX name for this item is deprecated. Instead, use the ISO C and C++ conformant name: _strdup. See online help for details. [-Wdeprecated-declarations]
../../third_party/dart/runtime/bin/gen_snapshot.cc(641,15): warning: 'strdup' is deprecated: The POSIX name for this item is deprecated. Instead, use the ISO C and C++ conformant name: _strdup. See online help for details. [-Wdeprecated-declarations]
2 warnings generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.
../../third_party/dart/runtime\vm/compiler/asm_intrinsifier.h(37,16): warning: unqualified friend declaration referring to type outside of the nearest enclosing namespace is a Microsoft extension; add a nested name specifier [-Wmicrosoft-unqualified-friend]
1 warning generated.

So it looks like there's one namespace bug, and a couple of places where strdup isn't using the new wrapper.

@stuartmorgan @a-siva is this issue still valid?

I checked the latest version of gen_snapst.cc and I see that strdup has been replaced with Utils::StrDup so that warning should be accounted for.
I believe you fixed the other warning from asm_intrinsifier.h here https://dart-review.googlesource.com/c/sdk/+/162901

The issue could be closed I suppose.

Yep, and I sent one more strdup cleanup today which is merged:
https://dart-review.googlesource.com/c/sdk/+/242386


Upgrade from 22.1.9 to 22.1.10 fails because module Jinja2 is missing.

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

On upgrade from .9 to .10, error about missing module Jinja2 is displayed. Configd appears to fail on start.
To Reproduce
Steps to reproduce the behavior:

Upgrade 22.1.9 system to 22.1.10
Observe errors in the console while the upgrade is running

Expected behavior
Error free upgrade, no daemon failures.
Additional context
Except from the console while running update:
Stopping configd...done
Starting configd.
Traceback (most recent call last):
File "/usr/local/opnsense/service/configd.py", line 43, in <module>
import modules.processhandler
File "/usr/local/opnsense/service/modules/processhandler.py", line 43, in <module>
from . import ph_inline_actions, syslog_error, syslog_info, syslog_notice, singleton
File "/usr/local/opnsense/service/modules/ph_inline_actions.py", line 32, in <module>
from . import template
File "/usr/local/opnsense/service/modules/template.py", line 43, in <module>
import jinja2
ModuleNotFoundError: No module named 'jinja2'
/usr/local/etc/rc.d/configd: WARNING: failed to start configd
Reloading plugin configuration
Configuring system logging...done.
Reloading template OPNsense/Syslog: configd socket missing (@/var/run/configd.socket)
Reloading template OPNsense/ddclient: configd socket missing (@/var/run/configd.socket)
pkg-static: POST-INSTALL script failed
[44/45] Upgrading opnsense from 22.1.9_1 to 22.1.10...
[44/45] Extracting opnsense-22.1.10: .......... done
configd not running? (check /var/run/configd.pid).

Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 22.1.10 (amd64, OpenSSL).
Discovered on PC Engines APU4, reproduced in a Xen VM

A reboot fixed it, this update probably needs to force a reboot after finishing.

Not according to my testing. Things can always happen though.

We've had multiple reports, none of those fatal so I don't think this has to be kept open...


Adding code from first design session

Took the code from the first design session and moved all classes to their own files.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Chris Albert seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@chris-albert Looks great! Just sign the CLA and I'll approve/merge.

@jdegoes signed

Hurray! 🎉


assembly - deduplicate problem when building app

Hi,
I'm not very familiar with scala ecosystem, and I have a problem when building this app.
After cloning a repo I'm running:
sbt assembly
and I get the following exception:
java.lang.RuntimeException: deduplicate: different file contents found in the following:
/home/rzukow/projects/KafkaOffsetMonitor/lib_managed/jars/org.eclipse.jetty/jetty-continuation/jetty-continuation-8.1.13.v20130916.jar:about.html
/home/rzukow/projects/KafkaOffsetMonitor/lib_managed/jars/org.eclipse.jetty/jetty-http/jetty-http-8.1.13.v20130916.jar:about.html
/home/rzukow/projects/KafkaOffsetMonitor/lib_managed/jars/org.eclipse.jetty/jetty-io/jetty-io-8.1.13.v20130916.jar:about.html
and so on... (the list is quite long).
Am I doing something wrong?

Fixed in #57


Retrieve actual calibration records

We should add support for retrieval of Dexcom's calibration records (slope, intercept, etc.) so we can properly use the raw filtered and unfiltered records. Currently, openaps use cgm calibrations does the same thing as meter_data. We should instead make it pull the actual calibration records, with slope, intercept, etc.

Shipped. :ship:


Understand menssages obtained when running the simple example (libj2v8_linux_x86_64.so)

Dear maintainers,
I am able to run the simple example in the README.md but I got the following messages that I do not know what to they mean:
16:01:56.415 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Could not initialize guru.nidi.graphviz.engine.GraphvizCmdLineEngine
guru.nidi.graphviz.engine.GraphvizException: dot command not found
16:01:56.435 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Starting V8 runtime...
Java HotSpot(TM) 64-Bit Server VM warning: You have loaded library /home/hlfernandez/libj2v8_linux_x86_64.so which might have disabled stack guard. The VM will try to fix the stack guard now.
It's highly recommended that you fix the library with 'execstack -c <libfile>', or link it with '-z noexecstack'.
16:01:56.653 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Started V8 runtime. Initializing graphviz...
16:01:57.158 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Initialized graphviz.

As the message says, the file libj2v8_linux_x86_64.so is created in my home every time I run the code. What do this mean? Is there a way to get rid of this?
Thank you very much in advance,
Hugo.

Hola Hugo
The library has 3 ways to process dot files:

with the ‚dot‘ command if you have graphviz installed on your machine
using javascript on a v8 engine, which is extracted in your home directory
using javascript on java’s own javascript engine

The messages come from the first and second option.
You can customize this by using Graphviz.useEngine().
But usually using V8 is the best option. (no need to install anything, faster than java’s own javascript)
I created two new issues due to your mail:
https://github.com/nidi3/graphviz-java/issues/76 https://github.com/nidi3/graphviz-java/issues/76
https://github.com/nidi3/graphviz-java/issues/77 https://github.com/nidi3/graphviz-java/issues/77
thanks and cheers
nidi

Am 03.08.2018 um 16:18 schrieb Hugo López-Fernández<EMAIL_ADDRESS>Dear maintainers,
I am able to run the simple example in the README.md but I got the following messages that I do not know what to they mean:
16:01:56.415 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Could not initialize guru.nidi.graphviz.engine.GraphvizCmdLineEngine
guru.nidi.graphviz.engine.GraphvizException: dot command not found
16:01:56.435 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Starting V8 runtime...
Java HotSpot(TM) 64-Bit Server VM warning: You have loaded library /home/hlfernandez/libj2v8_linux_x86_64.so which might have disabled stack guard. The VM will try to fix the stack guard now.
It's highly recommended that you fix the library with 'execstack -c ', or link it with '-z noexecstack'.
16:01:56.653 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Started V8 runtime. Initializing graphviz...
16:01:57.158 [main] INFO guru.nidi.graphviz.engine.AbstractGraphvizEngine - Initialized graphviz.
As the message says, the file libj2v8_linux_x86_64.so is created in my home every time I run the code. What do this mean? Is there a way to get rid of this?
Thank you very much in advance,
Hugo.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/nidi3/graphviz-java/issues/75, or mute the thread https://github.com/notifications/unsubscribe-auth/AAoyiZZbPC8PoX0EdSxn5qOwEHHMtgVlks5uNFusgaJpZM4VuDK4.

Thank you @nidi3,
I have tried using "Graphviz.useEngine(new GraphvizJdkEngine());" and it works fine, without throwing any error message (although it is slower as you said)
I have seen that you have already closed the other two issues and pushed two new commits, so I am looking forward to try the 0.5.4 release with the V8 engine.
Cheers,
Hugo.

I had to reopen #76 but it should still work ok.
0.5.4 is released and will be available soon.

Hello @nidi3
I am getting no error while running the the simple graph on java using Graphviz.useEngine(new GraphvizJdkEngine());But graph is not showing or opening it just ends the process with 0.
Can you please help me

hi @tamir123
what code do you run?
normally a png file is created, but not opened/shown.


linode-cli: 5.8.2 -> 5.9.0

linode-cli: 5.8.2 -> 5.9.0

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137846 run on x86_64-linux 1

1 package built:

linode-cli


ICodecAPI is not available via strmif.h on UWPs

icodecapi.h should be used on UWP apps

@LoadLibrary : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for your contribution. Can you provide more details and context for this change? I tested some code in a C++/WinRT project, and it looks like I can compile code that declares an ICodecAPI pointer by referencing the strmif.h header file.

Since I cannot reproduce this issue, I am closing this pull request for now. Please reopen it if you have more context and information you can provide to support this change.


update torch one more time

torch 1.12.1

[X] * 220: yolov5 v6.1 contains bug which incompatible with torch1.11+, upgrade ultralitics version
[X] * 219: changed default input names during ONNX conversion, pinned specific names for avoiding this issue in future
[x] * 217: bug in pytorch onnx instance norm export, fixed in 1.13
[ ]  * 102: moved geffnet to 1.0.2 version which contains fix for issue with wrong imports, but introduce unexportable silu

torch 1.13.0:

[X] 211: model unreshapable after tracing, fixed on OMZ side: https://github.com/openvinotoolkit/open_model_zoo/pull/3623

ported to https://github.com/openvinotoolkit/openvino_notebooks/pull/673


make the bucket argument required

Added an argument to make -b / --bucket required for enable_flow_logs, along with a workaround to make argparse not show it as optional.

thank you @mikeurbanski1


QuantumCircuit.measure_active gives a creg the name "measure" that is a reserved QASM word

Environment

Qiskit Terra version: 0.20
Python version: 3.7
Operating system: osx

What is happening?
https://github.com/Qiskit/qiskit-terra/issues/4023 found an issue that QuantumCircuit.measure_all creates a ClassicalRegister called measure, which is a reserved keyword in OpenQASM2+3. This was changed to meas for measure_active in #4079 , and addressed somewhat more generally for OpenQASM3 export in #7744 .
How can we reproduce the issue?
qc = qk.QuantumCircuit(3)
qc.x([0,1,2])
qc.measure_active()

QuantumCircuit.from_qasm_str(qc.qasm())

raises
QasmError: "Expected an ID, received 'measure'"

What should happen?
It would be good to:

have consistent behavior between measure_all and measure_active,
possibly provide an option for the user to provide the name of the ClassicalRegister to be added,
work around so that the above doesn't fail for QASM 2 export + import.

Any suggestions?
No response

We've run into this issue as well:
import qiskit
import qiskit_experiments.library.quantum_volume as qv
from qiskit.compiler import transpile

rand_circs = qv.QuantumVolume(qubits=[0,1,2]).circuits()

t_rand_circ = transpile(rand_circs[0], basis_gates=['u1','u2','u3','cx'])
print(t_rand_circ.qasm())
# prints
# OPENQASM 2.0;
# include "qelib1.inc";
# qreg q[2];
# creg measure[2];
# u3(0.95275496,-0.40048302,2.3396091) q[0];
# u3(1.4710132,1.4238801,-0.9022253) q[1];
# cx q[0],q[1];
# u3(0.7076726,-pi/2,-pi/2) q[0];
# u3(2.1567055,2.5482935,0.68681294) q[1];
# cx q[0],q[1];
# u3(0.28381753,-pi,pi/2) q[0];
# u3(1.0477967,0.64780817,-2.5581967) q[1];
# cx q[0],q[1];
# u3(1.4541338,-0.35391366,-1.7202243) q[0];
# u3(1.9771718,0.44431931,-0.27642371) q[1];
# barrier q[0],q[1];
# measure q[0] -> measure[0];
# measure q[1] -> measure[1];

qiskit.QuantumCircuit.from_qasm_str(t_rand_circ.qasm()) #raises qiskit.qasm.exceptions.QasmError: "Expected an ID, received 'measure'"

Thanks for bringing this up again.  This one particular case should hopefully be able to be handled by a simple "escape name" type of function in the exporter, as opposed to the OpenQASM 3.0 exporter's symbol-table version.  I'm happy to accept/help with PRs to fix this, though.

"escape name" type of function in the exporter

What's an "escape name"? Is it basically a way to tell the exporter to write _measure (with underscore) instead of the existing measure?

Yeah, essentially.  We already do some of this, if a gate claims its name is something like BLAH!£%& - that's not a valid OQ 2 identifier because of the leading capital letter and the symbols, so we "escape" it to gate_BLAH____.  That same function just needs a bit more logic to ensure it recognises that the OQ 2 reserved keywords are also invalid identifiers.

Fixed by #9100


chore(travis): use test matrix

Result in some parallel environment testing ;)
https://travis-ci.org/ModuleLoader/es6-module-loader/builds/48717383

Nice!


Query Debugger

Description

Checklist

[ ] For UI or styling changes, I have added a screenshot or gif showing before & after
[ ] I have added a changeset
[ ] I have added to the docs where applicable
[ ] I have added to the VS Code extension where applicable

@mcrascal where/when do we want to land this?


Get rid of building=detached setting?

It's not really clear how different it is from building=house. It only creates confusion. Looking at http://taghistory.raifer.tech/ it got pushed two times quite inorganically.

Yes I did see on the discussion pages for building=house and
building=detached that some people think the overly-specific house tags should be deprecated, and I think they are confusing too.
However I'm not really comfortable removing a preset that's used around 900,000 times, so the right way forward would be to achieve consensus on the tagging@ mailing list and start a project to replace all those tags with better ones.

Can it be made kind of read-only as in "recognize, but don't suggest"?

Can it be made kind of read-only so iD recognizes it, but doesn't suggest it?

It could, but without real consensus about its removal it's likely someone else would come here and open an issue to have it put back.


issues/125

Thank you for contributing to naz.
Every contribution to naz is important.
Contributor offers to license certain software (a “Contribution” or multiple
“Contributions”) to naz, and naz agrees to accept said Contributions,
under the terms of the MIT License.
Contributor understands and agrees that naz shall have the irrevocable and perpetual right to make
and distribute copies of any Contribution, as well as to create and distribute collective works and
derivative works of any Contribution, under the MIT License.
Now,
What(What have you changed?)

Why(Why did you change it?)

Fixes: https://github.com/komuw/naz/issues/125

References:

this wont work without a lot of headaches, https://blog.miguelgrinberg.com/post/unit-testing-asyncio-code


Refine and augment Kotlin DSL Primer user manual chapter

This PR includes a set of changes around IDE support and working with containers.

@bamboo, this is ready for another round


⚠ Jenkins Plugins Artifactory has degraded performance

In 3197d3b, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 2528 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 7a78b58 after 1 hour, 58 minutes.


Fix Peers List for iOS SDK v4

PeersListView, developed for iOS v3.x, used DittoPeer.address to display
connected peer data which broke with iOS v4.0.0, where DittoConnection
peer1 and peer2 properties were changed to the Swift Data type.  This commit
refactors Peers List feature to display local peer connections and list
remote peers with the v4.x DittoSwift package.

A note on all the commits. I started with the v4 branch and rebased onto main, which has had some updates since v4 was created. So all the previous commits in v4 were committed again in this branch in the rebase. I then made the updates fixing the peers list code for SDK v4.
When I created the PR, the branch to merge into was set to main automatically and I didn't notice until later. When I changed the merge target to v4, two conflicts arose: the Xcode Package.resolved file deep in the xcshareddata  folder, which is of little consequence, but the Presence Viewer javascript file was different. I pushed a new commit resolving both of those to the v4 branch versions.

@magicwave Is it hard to do git cherry-pick to create a new PR?

@magicwave Is it hard to do git cherry-pick to create a new PR?
I don't think it would be hard. Cherry-pick would essentially be doing the rebase manually, although there would be greater risk of missing something, I think.

Are all these changes meant to be merged into v4?

Closing in favor of #56


Influx-Capacitor is not compatible yet with InfluxDB 0.11

I would recommend to add a line in the InfluxCapacitor description that it is not yet compatible with InfluxDB 0.11.
The problem lies in InfluxDB.Net that is very specific about InfluxDB version
Please see https://github.com/ziyasal/InfluxDB.Net/blob/master/InfluxDB.Net/Client/InfluxDbClientAutoVersion.cs#L24
Cheers!

You are righ. Also I will contribute to InfluxDB.Net to have it support new versions.

Interestingly enough, I have a single installation of Influx-Capacitor v<IP_ADDRESS> (that for some reason has not been auto-updated) which appears to work with InfluxDB 0.11, while all my v<IP_ADDRESS> installations do not.

Version 0.11.x should work from version <IP_ADDRESS> (http://influx-capacitor.com/Home/Download). I have yet not confirmed that it it so, but the packages are updated anyway. Will test before I create a new official package.

Same problem here, InfluxDB in 0.12 and the release <IP_ADDRESS> of Influx-Capacitor  doesn't work (Could not etablish a connection to the database).

Working on version 0.12 now,..

Just pushed version 1.0.18 with support for influxdb version 0.11 and 0.12.


Clickin X does not deselect option from list

Hi I am having a plroble with django auto complete light.
Any searchbox with an option list that I create can not be deselected.

We're going to need a lot more context description than that