Group Field Level Localisation

Link to reproduction
No response
Describe the Bug
Environment info
payload: '^2.28.0'
npm: '10.8.2',
node: '20.17.0'
Describe the Bug
When trying to set up translatable fields i was getting an issue where data from the translatable fields were not being saved correctly, i was having issues with content being deleted or overwritten. The issue seemed to boil down to setting the localised option on a group field rather than each individual field in that group like so:
{
  name: "attract_title",
  label: "Attract Title",
  type: "group",
  fields: [
    {
      name: 'line_one',
      label: 'First Line',
      type: 'text',
      localized: true,
    },
    {
      name: 'line_two',
      label: 'Second Line',
      type: 'text',
      localized: true,
    }
  ]
}

This field was at the start of the collection and then proceeded to have knock on effects and preventing other fields from saving correctly.
When following the payload docs to set up all the necessary steps i came across this message:
Note:
Enabling localization for field types that support nested fields will automatically create localized "sets" of all fields contained within the field. For example, if you have a page layout using a blocks field type, you have the choice of either localizing the full layout, by enabling localization on the top-level blocks field, or only certain fields within the layout.  - link to doc here
Screen recording of the bug
https://github.com/user-attachments/assets/eb6b4a9f-0183-46cf-8c36-8cedd4f8a54c
To Reproduce
Set up your payload config with localization:
localization: {
    locales: [
      { label: "English", code: "en" },
      { label: "French", code: "fr" },
      { label: "Deutsch", code: "de" },
      { label: "Arabic", code: "ar" },
      { label: "Japanese", code: "ja" },
      { label: "Chinese", code: "zh" },
      { label: "Korean", code: "ko" },
      { label: "Taiwan Chinese", code: "zh-tw" },
    ],
    defaultLocale: "en",
    fallback: true,
  },

next set up a collection
const Campaigns: CollectionConfig = {
  slug: 'campaigns',
  labels: {
    singular: "Campaign",
    plural: "Campaigns",
  },
  admin: {
    useAsTitle: 'title',
    group: "Campaign Information",
    defaultColumns: ['title'],
    hooks: {
      beforeDuplicate: async ({ data }) => {
        console.log(data)
        return {
          ...data,
          title: `${data.title} (Copy)`,
          general: {
            ...data.general,
            linked_stock_levels: []
          }
        }
      },
    },
  },
  fields: [
    {
      name: "title",
      label: "Title",
      type: "text",
    },
    {
      type: 'tabs',
      tabs: [
        {
          name: 'content',
          label: 'Content',
          fields: [
            {
              name: "attract_title",
              label: "Attract Title",
              type: "group",
              localized: true,
              fields: [
                {
                  name: 'line_one',
                  label: 'First Line',
                  type: 'text',
                },
                {
                  name: 'line_two',
                  label: 'Second Line',
                  type: 'text',
                }
              ]
            },
            {
              type: 'row',
              fields: [
                {
                  name: "attract_asset",
                  label: "Attract Asset",
                  type: "json",
                  localized: true,
                  admin: {
                    components: {
                      Field: CloudinaryMediaUpload,
                    },
                    width: "50%",
                  },
                },
                {
                  name: "attract_background_asset",
                  label: "Attract Background Asset",
                  type: "json",
                  admin: {
                    components: {
                      Field: CloudinaryMediaUpload,
                    },
                    width: "50%",
                  },
                },
              ]
            },
            {
              name: 'highlight_color',
              label: 'Highlight Color',
              type: 'text',
              admin: {
                description: 'Please input the hex code for the colour you want to use.'
              }
            },
            {
              type: 'row',
              fields: [
                {
                  name: 'colour_mode',
                  label: 'Set Dark Mode',
                  type: 'checkbox',
                  defaultValue: false,
                },
                {
                  name: 'logo_colour',
                  label: 'Make Logo White',
                  type: 'checkbox',
                  defaultValue: false,
                },
              ]
            },
            {
              name: "no_win_asset",
              label: "No Win Asset",
              type: "json",
              localized: true,
              admin: {
                components: {
                  Field: CloudinaryMediaUpload,
                },
                width: "33.33%",
              },
            },
            {
              name: 'show_disclaimer',
              label: 'Show Disclaimer',
              type: 'checkbox',
              localized: true,
            },
            {
              name: 'disclaimer',
              label: 'Disclaimer',
              type: 'textarea',
              localized: true,
              admin: {
                condition: (data) => {
                  if (data?.content?.show_disclaimer) {
                    return true
                  } else {
                    return false
                  }
                },
              }
            },
            {
              name: 'show_claim_instructions',
              label: 'Show Claim Instructions',
              type: 'checkbox',
              localized: true,
            },
            {
              name: 'claim_instructions',
              label: 'Claim Instructions',
              type: 'textarea',
              localized: true,
              admin: {
                condition: (data) => {
                  if (data.content?.show_claim_instructions) {
                    return true
                  } else {
                    return false
                  }
                },
              }
            },
          ]
        },
      ]
    }
  ],
}

export default Campaigns

Payload Version
2.28.0
Adapters and Plugins
db-postgres v0.8.5

Hey @miller-design, we know this issue and it's already fixed in 3.0. And actually, we got merged this huge PR https://github.com/payloadcms/payload/pull/8468 that ports many bug fixes, including this. I don't have info when we can release that, but it should be soon.

Hey @r1tsuu, I had saw some similar issues relating to V3 just couldn't find anywhere where it was flagged for V2. Thanks for the update and ill keep an eye out for when this PR is released.


Feat: implement react router dom

issue #1

@thezeeshann have you taken pull before raising PR..

@sahilsingh7735 I am a new branch emerging from the master branch.

Can you tell me what is your branch name?

@sahilsingh7735 branch name feature/react-router-dom

Take the latest pull from Master there are some changes in the code and publish it on my repo.

@sahilsingh7735 bro check I published the latest code


Path fix

this not worked for me

Does the opentrade automatically create sqlite.db on opentrade/server/database? On fresh install it did not create sqlite.db on  opentrade/server/database I have to manually create sqlite.db on  opentrade/server/database

The file sqlite.db should creating automatically by OpenTrade script. But the folder opentrade/server/database should creating with "git clone "

Ok


Building with only "accelerate" feature fails

Linker fails with the following message if run cargo test with only the accelerate feature enabled

ld: archive member 'metal.o' not a mach-o file in '/.../mlx-rs/target/debug/build/mlx-sys-e8d54ba65338b1d5/out/libmlx.a'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

This is solved by removing "mlx/mlx/backend/metal/metal.h" from the const FILES_MLX, which contains the files that will be built regardless of any features


Add docker_stack type and provider and swarm_extract_tokens fact

Hello, sorry for the delay.
I leave my last contribution, the provider docker_stack and a facter to extract the swarm tokens with the idea of using them as exported resources, through the puppetDB, etc.
The docker_stack provider allows you to deploy a compose file to v3. To call it, the following syntax would be used:
docker_stack {'path-to-compose-v3.yml':
   ensure    => 'present',
   namestack => 'name-of-the-stack',
}

Facter swarm_extract_tokens extracts in hash format both manager and worker tokens. An output of this facter is as follows:
swarm_extract_tokens => {
  swarm_tokens => {
    manager => "SWMTKN-1-2nv8qxa5npe850c4hx3utvcn7tdmmg08d70yw6nrzlu5et1kil-f2qik9f2ui4ciuvnf6b8uuxs5",
    worker => "SWMTKN-1-2nv8qxa5npe850c4hx3utvcn7tdmmg08d70yw6nrzlu5et1kil-9unklwesgt1njskt7gjovny96"
  }
}

The provider and the facter have been tested in my working environment and put into production. I'm sorry I did not write the rspec tests, but as I already tell you because of my work urgency, I do not have time to do this. If necessary to do the pull request my master branch has the changes. I feel free to clone my changes, add what is missing and do the pull request.
Regards!

@drignoto Could we please break this up into 2 PR's
So one PR for stacks, the other for the swarm token.
Thanks heaps

@scotty-c Ok Scotty, sorry! Close this PR


Add license to package.json

This project includes the license text in the LICENSE file, but it doesn't indicate which license it uses in the package.json metadata, which doesn't play nicely with some tooling.

Merged, thanks!


Integrate CSCC GA API in Forseti

CSCC is moving from Beta to GA. Forseti currently supports Beta API and need to integrate GA in Forseti.

#2680


[RFC FS-1069] Implicit yields (allow dropping yield in list, array, sequence and computation expressions)

This is the implemntation for RFC FS-1069 Implict yields, i.e. the "implicit yields" language suggestion. This allows implicit yield in list, array, sequence and those computation expressions supporting Yield/Combine/Zero/Delay.
See the RFC for notes and discussion

Design notes:

The simplest, most compatible rule seems to be "implicit yields are enabled if there is no explicit 'yield' in the computation expression"
We need to consider if this applied to other computation expressions, or just sequence/list/array expressions.

We need to consider if this applied to other computation expressions, or just sequence/list/array expressions.
I'd be glad if it works in other cases too.

I'd be glad if it worked in other cases too.

I've

added the general implementation for computation expressions
expanded the testing
clarified the proposed rules in this issue (getting it closer to a draft of an RFC)

Current failures:
Conformance\Expressions\DataExpressions\SequenceExpressions (W_IfThenElse01.fs) -- failed
Conformance\Expressions\DataExpressions\SequenceExpressions (W_IfThenElse02.fs) -- failed
Conformance\Expressions\DataExpressions\SequenceExpressions (W_IfThenElse03.fs) -- failed
Warnings (80) -- failed
Warnings (81) -- failed
Warnings (82) -- failed

RFC at https://github.com/fsharp/fslang-design/blob/master/RFCs/FS-1069-implicit-yields.md

This is confusing, at least for a beginner like me. I think when in confusion, make things explicit. Thanks.

Why not enable this for yield! as well? If an expression has type that's implemented seq<'a> where 'a is the (inferred) type of elements of the computation expression, why not just implicitly yield! it?

Why not enable this for yield! as well? If an expression has type that implements seq<'a> where 'a is the (inferred) type of elements of the computation expression, why not just implicitly yield! it?

It's a good question. My intuition is it's just too much, too implicit, both for type inference and for users.  I don't have proof of this though.

Don't think it's a good idea to implicitly mix map and flatmap. Usually I
want to know when flattening is happening. In react view code it's usually
a sign that I want to refactor or that I need to add fable.react's ofList
function that gives better perf
Don Syme<EMAIL_ADDRESS>schrieb am Di., 19. März 2019, 12:31:

Why not enable this for yield! as well? If an expression has type that
implements seq<'a> where 'a is the (inferred) type of elements of the
computation expression, why not just implicitly yield! it?
It's a good question. My intuition is it's just too much, too implicit,
both for type inference and for users. I don't have proof of this though.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/6304#issuecomment-474322078,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNLkLZN-yMAYmx4-7tyzLwQaQPFrEks5vYMqDgaJpZM4baZth
.

Unrelated flaky test failure:
[xUnit.net 00:01:16.59]     FSharp.Core.UnitTests.FSharp_Core.Microsoft_FSharp_Control.AsyncType.TaskAsyncValueException [FAIL]
Failed   FSharp.Core.UnitTests.FSharp_Core.Microsoft_FSharp_Control.AsyncType.TaskAsyncValueException
Error Message:
 System.TimeoutException : The operation has timed out.
Stack Trace:
   at Microsoft.FSharp.Control.AsyncPrimitives.RunSynchronouslyInAnotherThread[a](CancellationToken token, FSharpAsync`1 computation, FSharpOption`1 timeout)
   at Microsoft.FSharp.Control.FSharpAsync.RunSynchronously[T](FSharpAsync`1 computation, FSharpOption`1 timeout, FSharpOption`1 cancellationToken)
   at FSharp.Core.UnitTests.FSharp_Core.Microsoft_FSharp_Control.AsyncType.TaskAsyncValueException() in D:\a\1\s\tests\FSharp.Core.UnitTests\FSharp.Core\Microsoft.FSharp.Control\AsyncType.fs:line 424

@vasily-kirichenko how would you distinguish what the return type of this is - is it int list or int list list?
let x =
  [
      [ 1 .. 5 ]
  ]

What about these cases
let y =
  [
      [ 1 .. 5 ] // implicitly yield! ? Or is it int list list?
      1 // compiler error?
  ]

let z =
  [
      1
      [ 1 .. 5 ] // compiler error? implicitly "yield!" ?
  ]

On the one hand, I get the idea of making this "simpler" to look at, but on the other hand I find this "implict" yield (and especially yield!) feels to me to be going against the explicit and (very importantly) predictable nature of F#. I feel that this feature would have to be very clearly documented to explain what and why is happening.

Closed in favour of https://github.com/dotnet/fsharp/pull/6806


Remove indentation for bibliography

e.g. relevant for APA

Hi, can you also update the example and pdf? Thanks


Cannot develop locally using Localstack

Is this a bug report?
Yes.
Can you also reproduce the problem with the latest version?
Yes, I can. Here is a link to a sample repo. https://github.com/rabberbock/MassTransitLocalstack/
Environment

Operating system:  Windows 10
Visual Studio version: Visual Studio 2019
Dotnet version: .Net Core 3.0 preview 5

Steps to Reproduce

Download the Sample project from https://github.com/rabberbock/MassTransitLocalstack.
Make sure you have .Net Core 3 preview 5 installed
Have localstack running on your computer. (I ran it via docker and added docker.localhost to the hosts file). I am using the sqs transport. I tried it with go-aws as well and got the same error.
Run the application, and notice the error when trying to execute the handler.

Expected Behavior
It should execute the handler with no errors.
Actual Behavior
Currently message deserialization is broken using localstack to emulate sqs and sns. Running this with a real AWS sqs and sns actually works.
 R-FAULT amazonsqs://us-east-1/my_queue 7db85235-948f-4060-b37f-9568e3e51de3 Value cannot be null.
      Parameter name: source
System.Runtime.Serialization.SerializationException: An exception occurred while deserializing the message envelope ---> System.ArgumentNullException: Value cannot be null.
Parameter name: source
   at System.Linq.ThrowHelper.ThrowArgumentNullException(ExceptionArgument argument)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at MassTransit.Serialization.JsonConsumeContext..ctor(JsonSerializer deserializer, IObjectTypeDeserializer objectTypeDeserializer, ReceiveContext receiveContext, MessageEnvelope envelope)
   at MassTransit.Serialization.JsonMessageDeserializer.MassTransit.IMessageDeserializer.Deserialize(ReceiveContext receiveContext)
   --- End of inner exception stack trace ---
   at MassTransit.Serialization.JsonMessageDeserializer.MassTransit.IMessageDeserializer.Deserialize(ReceiveContext receiveContext)
   at MassTransit.Serialization.SupportedMessageDeserializers.Deserialize(ReceiveContext receiveContext)
   at MassTransit.Pipeline.Filters.DeserializeFilter.Send(ReceiveContext context, IPipe`1 next)
   at GreenPipes.Filters.RescueFilter`2.GreenPipes.IFilter<TContext>.Send(TContext context, IPipe`1 next)
dbug: MassTransit.Messages[0]
      SEND amazonsqs://us-east-1/MassTransit-ReceiveFault 47360000-5d56-0015-5d31-08d6f2863ce7 MassTransit.ReceiveFault
dbug: MassTransit.Messages[0]
      SEND amazonsqs://us-east-1/MassTransit-ReceiveFault 47360000-5d56-0015-3386-08d6f2863ceb MassTransit.ReceiveFault

Reproducible Demo
https://github.com/rabberbock/MassTransitLocalstack.
Thanks for all your help!

I'll be honest, I don't plan to spend any time on this because:

I have no desire to install LocalStack
It works with Amazon SQS/SNS

If anyone else figures it out, great, but I will close this in 30 days if there isn't any activity.

@rabberbock what tag version of localstack are you using?

I tried it with go-aws as well and got the same error.
if you got the same error with go-aws-sdk I would file an issue with localstack repo

@gertvjr I am using localstack 0.8.0. There are more recent updates, but that was the version I was able to get working with an earlier version of masstransit (5.3.1).

I further investigated the issue by debugging it on a local clone of MassTransit.
Here is a sample payload when it is read from the queue
{
  "Message": "{\r\n  \"messageId\": \"47360000-5d56-0015-1f29-08d6f2c35d99\",\r\n  \"conversationId\": \"47360000-5d56-0015-ca5d-08d6f2c35da2\",\r\n  \"sourceAddress\": \"amazonsqs://us-east-1/TempMassTransitQueue?durable=false\",\r\n  \"destinationAddress\": \"amazonsqs://us-east-1/BackgroundTasks-SampleMessage\",\r\n  \"messageType\": [\r\n    \"urn:message:BackgroundTasks:SampleMessage\"\r\n  ],\r\n  \"message\": {\r\n    \"message\": \"Hello World\"\r\n  },\r\n  \"sentTime\": \"2019-06-17T01:30:22.3228002Z\",\r\n  \"headers\": {},\r\n  \"host\": {\r\n    \"machineName\": \"DESKTOP-2HRK6MB\",\r\n    \"processName\": \"BackgroundTasks\",\r\n    \"processId\": 10448,\r\n    \"assembly\": \"BackgroundTasks\",\r\n    \"assemblyVersion\": \"<IP_ADDRESS>\",\r\n    \"frameworkVersion\": \"3.0.0\",\r\n    \"massTransitVersion\": \"<IP_ADDRESS>\",\r\n    \"operatingSystemVersion\": \"Microsoft Windows NT 6.2.9200.0\"\r\n  }\r\n}",
  "Type": "Notification",
  "TopicArn": "arn:aws:sns:us-east-1:123456789012:BackgroundTasks-SampleMessage",
  "MessageAttributes": {
    "Content-Type": {
      "Type": "String",
      "Value": "application/vnd.masstransit+json"
    },
    "MessageId": {
      "Type": "String",
      "Value": "47360000-5d56-0015-1f29-08d6f2c35d99"
    }
  }
}

The message envelope is not a the root level, but it is wrapped in a Message property. So the envelop has many null values, hence it is throwing a null pointer.

Looks like Local Stack isn't properly respecting the RawMessageDelivery = true attribute in the subscription.

localstack has a dependency on moto which does support RawMessageDelivery so I would have a closer look how localstack is handling the inner communication between sns to sqs

@phatboyg @gertjvr Thanks for the suggestions! I will take a look and let you know if I find anything.

@rabberbock any more insight, and can we close the issue?

@gertjvr I have not found any new information. Will post again if I find anything out. We can close for now.

No changes are planned for MassTransit, so closing this issue.

That should be easy change to do, happy for you to send a PR
On Wed, Aug 28, 2019 at 2:47 AM rabberbock<EMAIL_ADDRESS>wrote:

@gertjvr https://github.com/gertjvr @phatboyg
https://github.com/phatboyg I got a chance to look into this some more.
So it is indeed a problem related to RawDelivery. The subscription does
not have raw delivery set properly.
The example JSON on this page,
https://aws.amazon.com/blogs/compute/managing-amazon-sns-subscription-attributes-with-aws-cloudformation/,
uses "RawMessageDelivery": "true" to set the RawMessageDelivery. The
screenshot attached shows that the RawDelivery in C# code is "RawMessageDelivery":
"True". On a real AWS setup this does not seem to make a different, but
on localstack it does.
What are your thoughts on changing
https://github.com/MassTransit/MassTransit/blob/develop/src/MassTransit.AmazonSqsTransport/Topology/Entities/TopicEntity.cs#L61
to TopicSubscriptionAttributes["RawMessageDelivery"] = 'true';. If you
agree to this idea I can open up a pull request with a couple of tests.
[image: image]
https://user-images.githubusercontent.com/3247622/63790571-8b24c280-c8c7-11e9-9d50-cd6428d18b23.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MassTransit/MassTransit/issues/1476?email_source=notifications&email_token=AAM43NRDSQSJZ3HI53JN5QDQGVLANA5CNFSM4HYRXLKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5IMJQA#issuecomment-525386944,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAM43NTQBZT5BRVJZGB5G5DQGVLANANCNFSM4HYRXLKA
.

Was this fixed? I can't find a referencing PR.

@ffMathy It was fixed with #1531. There was another issue fixed on the localstack side https://github.com/localstack/localstack/issues/1516. So it should work with latest localstack release 0.10.5.

I use local stack to test SQS.

Okay thanks guys.
@phatboyg can you show an example of such a test? I am receiving various errors, such as "Name or service not known" etc.
Do you have a dockerfile or docker-compose file showing your setup?

I literally just installed it using pip3 install localstack and ran the unit tests in MassTransit.
Like this one: https://github.com/MassTransit/MassTransit/blob/develop/src/MassTransit.AmazonSqsTransport.Tests/ErrorQueue_Specs.cs

I have always had trouble setting it up with pip on my windows machine. Here is the docker-compose I use.
version: '2.1'

services:

  localstack:
    image: localstack/localstack:0.10.5
    ports:
      - "4575-4576:4575-4576"
      - "8090:${PORT_WEB_UI-8080}"
    environment:
      - SERVICES=sns,sqs
      - DEBUG=1
      #- DATA_DIR=/tmp/localstack/data
      - PORT_WEB_UI=${PORT_WEB_UI- }
      - DOCKER_HOST=unix:///var/run/docker.sock
    volumes:
      - "localstackdata:/tmp/localstack"

volumes:
  localstackdata:

I'm unable to locally test MassTransit with localstack SNS, SQS. It can't connect.
Running the latest localstack with
docker run -it -e SERVICES=sns,sqs -e TEST_AWS_ACCOUNT_ID="000000000000" -e DEFAULT_REGION="us-east-1" -e LOCALSTACK_HOSTNAME="localhost" -e -rm --privileged --name localstack_main -p 4566:4566 -p 4571:4571 -p 8080-8081:8080-8081  -v "/tmp/localstack:/tmp/localstack" -v "/var/run/docker.sock:/var/run/docker.sock" -e DOCKER_HOST="unix:///var/run/docker.sock" -e HOST_TMP_FOLDER="/tmp/localstack" "localstack/localstack"

and I can see the services up and running http://localhost:4566/health
What am I missing? This is my code
var region = "eu-west-3"; // I've tried with localhost, localhost:4566
var accessKey = "test";
var secretKey = "test";
var busControl = Bus.Factory.CreateUsingAmazonSqs(c =>
{
    var hostAddress = new Uri($"amazonsqs://{Region}")
    var hostConfigurator = new AmazonSqsHostConfigurator(hostAddress);
    hostConfigurator.AccessKey(accessKey);
    hostConfigurator.SecretKey(secretKey);
    c.Host(hostConfigurator.Settings);
});

Sorry to ask on an already closed ticket, but it's related.
PS: @phatboyg The link to see the tests seems broken. Found this  not sure if there is a more appropriate one

You can check the options used in the unit tests via docker-compose for localstack that I use to test MassTransit.

Thanks so much. Following with my example, this works fine to connect to localstack as per previous message.
var region = "localhost:4566";
var accessKey = "test";
var secretKey = "test";
var busControl = Bus.Factory.CreateUsingAmazonSqs(c =>
{
    var hostAddress = new Uri($"amazonsqs://{region}")
    var hostConfigurator = new AmazonSqsHostConfigurator(hostAddress);
    hostConfigurator.AccessKey(accessKey);
    hostConfigurator.SecretKey(secretKey);
    hostConfigurator.Config(new AmazonSimpleNotificationServiceConfig {ServiceURL = $"http://{region}"});
    hostConfigurator.Config(new AmazonSQSConfig {ServiceURL = $"http://{region}"});
    c.Host(hostConfigurator.Settings);
});

https://github.com/MassTransit/MassTransit/blob/develop/src/MassTransit.AmazonSqsTransport/Topology/Entities/TopicEntity.cs#L61

saved my day, thanks!


UCL for configuration file

Does Linux have a libucl package? What about Cygwin?

We could also use JSON, which probably has better support under Linux/Cygwin and is better known.

libjson-c is supported bon both Linux and FreeBSD.


withdraw()

if(participantRecord.whitelisted == true) : using require() will make the code cleaner (currently it reverts at the end of the function).
(https://github.com/lukso-network/rICO-smart-contracts/blob/master/contracts/ReversibleICO.sol#L705)

if(remainingTokenAmount > 0), maybe we should return instead: if  (remainingTokenAmount == 0) to make it more explicit.
( https://github.com/lukso-network/rICO-smart-contracts/blob/master/contracts/ReversibleICO.sol#L723)

assign  getCurrentStage() in the loop (https://github.com/lukso-network/rICO-smart-contracts/blob/master/contracts/ReversibleICO.sol#L735)

possible duplicate line participantRecord.withdrawnETH += returnETHAmount;
lines 794, 803

#1 - ok, require can work
#2 - ok
#3 - if done in the for() it will remove a stack item, that's cool.
#4 - yes, bad merge on my part.

please make a PR


Extend config extensions doc with binding source generator info

https://devblogs.microsoft.com/dotnet/announcing-dotnet-8-preview-6/#configuration-binding-source-generator-improvements
https://learn.microsoft.com/en-us/dotnet/core/extensions/configuration
https://github.com/dotnet/docs/blob/b8f805b78dc90ca6339d18d0d404a3fff9c01453/docs/core/extensions/configuration.md
ASP.NET https://github.com/dotnet/AspNetCore.Docs/issues/30215

@jkotas in https://github.com/dotnet/runtime/issues/79527#issuecomment-1706986512.
Also, we should have this source generator documented at https://learn.microsoft.com/en-us/dotnet/core/extensions/configuration. Once the documentation is available, please update this link to point to it.

@IEvangelist @gewarren is it possible to prioritize this for .NET 9?  Seems we still don't have good conceptual docs for the Configuration source generator.

Yes, I'll commit to making that happen for .NET 9 for sure. I've been buried with .NET Aspire docs work lately, and haven't had lots of time to work on .NET docs for a while now.


NoMethodError (undefined method `body=' for CypherNode 2093<PHONE_NUMBER>2020):Neo4j::Server::CypherNode):

After:
 q = @res.questionbyid(params[:id])

[...]
 def questionbyid(id)
     self.query_as(:a).match('a-[r:aquestions]-n').where(n: {uuid: id}).pluck(:n).first
 end

Also a possibly related error (???) -
  TypeError (no implicit conversion of Symbol into Integer):
    app/models/resource.rb:968:in `block in as_json'
    app/models/resource.rb:968:in `map'
    app/models/resource.rb:968:in `as_json'

Possibly another class mismatch?

You're getting a CypherNode because it's having trouble finding the mode
that's responsible for that combination of labels. Weird. If you call
:wrapper on the CypherNode, what does it return?
On Tuesday, August 25, 2015, djvs<EMAIL_ADDRESS>wrote:

Also a possibly related error (???) -
TypeError (no implicit conversion of Symbol into Integer):
app/models/resource.rb:968:in block in as_json'     app/models/resource.rb:968:in map'
app/models/resource.rb:968:in `as_json'
Possibly another class mismatch?
—
Reply to this email directly or view it on GitHub
https://github.com/neo4jrb/neo4j/issues/943#issuecomment-134806127.

Might have been an error in my code (commas left over from a hash...) testing...

Hmm yes...so I guess if you define a model's attribute to be the model itself... or possibly some hash... it will cause that...

Can you elaborate and give an example?
On Tuesday, August 25, 2015, djvs<EMAIL_ADDRESS>wrote:

Hmm yes...so I guess if you define a model's attribute to be the model
itself... or possibly some hash... it will cause that...
—
Reply to this email directly or view it on GitHub
https://github.com/neo4jrb/neo4j/issues/943#issuecomment-134806601.

Well, I closed the file now, but it looked something like this:
q.explanation = params[:explanation],
q.version = 0,
q.answers = answers
q.save

This would mess something up, then the next time the whole Rails action was called (I think), it would give the CypherNode error.  Removing the commas makes the problem vanish.

A very unusual  error so maybe not of interest.

I take that back.  It's still doing it.

puts q.wrapper.inspect gives:
CypherNode 2170<PHONE_NUMBER>4900)

Yes, the two bugs I reported are related.  This:
obj[:questions] = self.questions.map{|x|puts x.inspect + " XXX"; x.as_json("teacher")}

Will give this:
CypherNode 2093<PHONE_NUMBER>6840) XXX
Completed 500 Internal Server Error in 19ms

TypeError (no implicit conversion of Symbol into Integer):

I just released 5.1.5 to fix this.

You can reopen this issue and/or ping me on Gitter if you're still having trouble.


Changes to public composables

Describe the PR
Lot's of changes to composables.
Small replication
A small replication or video walkthrough can help demonstrate the changes made. This is optional, but can help observe the intended changes. A mentioned issue that contains a replication also works.
PR checklist

What kind of change does this PR introduce? (check at least one)

[x] Bugfix :bug: - fix(...)
[x] Feature - feat(...)
[ ] ARIA accessibility - fix(...)
[ ] Documentation update - docs(...)
[ ] Other (please describe)

The PR fulfills these requirements:

[x] Pull request title and all commits follow the Conventional Commits convention or has an override in this pull request body This is very important, as the CHANGELOG is generated from these messages, and determines the next version type. Pull requests that do not follow conventional commits or do not have an override will be denied

@VividLemon @dwgray any ideas why the docs build hangs forever?

@VividLemon @dwgray any ideas why the docs build hangs forever?

We'll figure it out after I'm done reviewing. Still not done yet, but I've submitted the first set of changes I'd like to see

I wonder if it could be preferable to combine useModal into useModalController. But maybe not, they serve different purposes 💭 plus the naming of methods would seem like it would cause confusion

I was thinking of creating a useToggle that can control all the useShowHide components, that would make useModal kinda reduntant. Thoughts?
The documentation build was due to setInterval in script setup. Which is weird because I didn't add them 😄
I'm a little under the weather at the moment, so might not continue this right away.

@VividLemon I'm thinking of removing the support for "component" their slots don't work at the moment, we need to support all the slots for that to work, now we just render what ever is in modal.slots for example. I couldn't find a way to list components available slots, other that just putting them to a variable, so I did the current version.
Only thing I think it's useful for is to have a custom default for the bmodal and you don't want to list them in the function args.

How do you share many same props between two instances of show?
const mySharedProps = computed(() => ({ foo: bar.value, buzz: aef.value }))
show({ ...mySharedProps.value, a: 1 })
show({ ...mySharedProps.value, b: 2 })

This would required show(computed(() => ({ ...  Is this valid syntax? Ideally both show({ title: ref() }) and show(computed(() => ({ title: ref() }))) would be valid

They should all work, I haven't tested all. But the { var: ref() } part is only typed for title and body at the moment and it goest through toValue inside the watch. Maybe I'll loop all the props to be able to be refs... I'll test next week

How do you share many same props between two instances of show?
const mySharedProps = computed(() => ({ foo: bar.value, buzz: aef.value }))
show({ ...mySharedProps.value, a: 1 })
show({ ...mySharedProps.value, b: 2 })

This would required show(computed(() => ({ ...  Is this valid syntax? Ideally both show({ title: ref() }) and show(computed(() => ({ title: ref() }))) would be valid

They should all work, I haven't tested all. But the { var: ref() } part is only typed for title and body at the moment and it goest through toValue inside the watch. Maybe I'll loop all the props to be able to be refs... I'll test next week

show(ref({ foo: 'bar' })
show({foo: ref('bar') })
show(() => ({ foo: props.bar }))
show(computed(() => ({ ...computedValue.value }))

I think for simplicity just the root should be MaybeRef show({foo: ref('bar') }) Might be too much overhead.
How this would work with values to be updated, I'm not sure yet.
MaybeRefOrGetter has the limitation of it's implicitly readonly. So we should probably do MaybeRef with toRef so if it's not a ref we can mutate the modelValue, and when it is, we can still update the modelValue

MaybeRefOrGetter<T = any> = MaybeRef | ComputedRef | (() => T)
it's not readonly

export type MaybeRefOrGetter<T = any> = MaybeRef<T> | ComputedRef<T> | (() => T);

it's not readonly

ComputedRefs will be. Which means if someone passes a computed, it breaks. MaybeRef is better in this case


Dragging to do AreaSelection and releasing mouse on StoryItem selects item

Version: 0.3.2
MouseClick on a child StoryItem of ActiveStoryItem in edit mode should wrap it with the AreaSelector so that it can be moved/resized
However dragging on an area to define a new AreaSelection should result in changing the area to the bounds of the child item upon which MouseUp event occured which seems to be occuring now

this is useful when item is outside its parent (dragging from inside the parent and onto the item to select it instead of making a bit selection to contain it), but is problematic when one has an item above another item (being children of same parent - say wanting to define hotspot areas over some other item) and one wants to select one of the hotspots by dragging an area around it. With the area being over a bigger item that is underneath, that item results in being wrapped with the area selector. But when dragging the area selector (currently) moves the top item in the stack, that is the item we did want to select. So this gets confusing

seems the drag selection edge circle and release mouse button action does a mousepress event or mouseclick or something (wonder if it's a but in Delphi FMX), so the item upon which the mouse is released gets the automatic fitted-to-borders selector action executed (discarding our selection area action)


fixes HHH-8805-BUG

https://hibernate.atlassian.net/browse/HHH-8805?focusedCommentId=103026

Substituted by this PR which includes tests as well.


Enable Codeblock Copying

Write a function so that the contents of a codeblock get copied to the clipboard

This looks promising:
https://codyhouse.co/ds/components/info/copy-to-clipboard


Just a small patch fixing RSS reader time format

Fix UTC parsing

Actually, UTC is not in the standard, UT should be used instead, but some feeds might use UTC anyway
reference: https://datatracker.ietf.org/doc/html/rfc822#section-5.1


HEDA error when approving new accounts

Hi, we have an approval process setup on accounts.  We create them in one temporary record type and convert them to the official record type when approved.  This worked until the May HEDA update.  Now we are getting the following error when trying to approve accounts.  Any thoughts?
Thanks!!
Error thrown when approving the creation of a new account.
Validation Errors While Saving Record(s)
There were custom validation error(s) encountered while saving the affected record(s). The first validation error encountered was "Apex trigger hed.TDTM_Account caused an unexpected exception, contact your administrator: hed.TDTM_Account: execution of AfterUpdate caused by: System.NullPointerException: Attempt to de-reference a null object: Class.hed.AFFL_MultiRecordTypeMapper.getKeyAfflFieldByAccRecordType: line 139, column 1".

HI @jdaze10 - Please move this question to the HEDA User Group in the Power of US Hub so we can address your question in the community. Here is a little background on how Issues works in Github so you understand why I am asking you to move this discussion to the PUB:
Issues in Github should only be created if you've encountered a bug with a Release. Errors, like what you've encountered, should always be posted in the PUB and reported to salesforce customer support for resolution. However, if the error you encountered is understood by you and you know how to fix it you could submit a Pull Request to address the fix in code.
You can find me in the PUB so feel free to reach out to me there if you need additional information on the use of Github. I'm always willing to talk about it :)
Thank you!

Thanks I did create a case to resolve as well.

Thank you @jdaze10! I am going to look for your PUB post in just a few minutes. I'll be closing this Issue out now.


rememberSnapperFlingBehavior for scrollable Row

We have horizontally scrollable container/content. It contains about 1 to 3 (at most) containers. The design requires to have the boxes the same height, so we use intrinsic height sizing (and of course, the width is also the same). Using intrinsic is forbidden in LazyRow, so we are currently using normal scrollable Row.

It could be probably done via custom SnapperLayoutInfo, yet it seems like enormous and non trivial work/amount of code.
Do you have any idea how to make it easier?
Solution I'd like
I am not much sure, maybe something like this?
val state = rememberScrollState()
val flingBehavior = rememberSnapperFlingBehavior(state, itemWidth = 200.dp)
Row(
    Modifier
        .horizontalScroll(
            state = state,
            flingBehavior = flingBehavior,
        )
)

Alternatives I've considered

having custom snapping code (what we have now)
implementing SnapperLayoutInfo

flingBehaviors have no control over the layout of items, only the flinging.
I can imagine there being some wrapper of Lazy* and Snapper which achieve this? This sounds very similar to what Accompanist Pager achieves, and that is heavily reliant on Snapper.
I'm going to mark this as closed and I don't this is in the realm of Snapper.

Thx for the input, I'll explore Pager a bit more.

flingBehaviors have no control over the layout of items, only the flinging.

Now I see, that may be misunderstanding, I do not wanna have modify my layouting, the passed itemWidth was just stupid idea to let snapper know on what offsets those items are.

Ah, sounds like you want snapOffsetForItem? https://chrisbanes.github.io/snapper/usage/#snap-positions

Well yeah, the API seems open enough, yet I opened this issue to avoid implementing SnapperLayoutInfo for a scrollable Row.

@hrach Did you end up with anything here? I'm faced with the same situation and considered making a non-lazy SnapperLayoutInfo but If you have found a better way I'm all ears 👂

There's no generic way to do this as Row or Column do not expose any layout info as state.
The SnapperLayoutInfo interface is there for this exact use case, but you'll need to implement the layout info above, together with the ScrollState.

I logged this Feature Request https://issuetracker.google.com/issues/228336552


Implemented a simple repr method for fit result

A simple implementation for the __repr__ such that for any fitResult we can simply print:
fitRes = fit(...)

print(fitRes)

Additionally, the class FitResult now also stores the prior (required in the printing)
I commented out the automatic serialization of the FitResult in fit(...). This should be done by the user if desired.
Any additions/improvements can be added later I think unless you find a fundamental flaw.

Yes, that comes with another patch. Currently working on it.
Should we merge?


Fix display overlap with images

The graphic and your photo overlap in odd ways in the mobile version of the site.

I fiddled with it so it doesn't overlap, wanted feedback before I closed this issue. I just adjusted the size of the image file, and I'm not sure if it will look the same on other sites. I wish there was  a better way to set a responsive back-ground image. Tried a couple of thing with media query and couldn't figure anything out.


WIP: Avoid apostrophe warning for possessive form of filenames / paths / URLs

I didn't find a SourceNode type or similar, so I'm not sure how the predicate should at L163 should look. My only idea would be to execute the code on everything, see this commit. I don't have the time to look into all these issues further, I'm sorry, because:

Don't usually work with JavaScript, so I have no functioning test workflow set up.
Not sure how & where exactly a test that Node.js\' does not trigger any warning should be implemented.

Thanks for getting this started! See 4d5346a for what I meant :)


generator: Generate the example widget/config survey in the CI

With #794 we have a working generator project (ie it produces compilable code, even though the project itself is not complete and has no actual frontend or backend application).
To make sure updates in evolution do not break the generated code, it should be run during CI. Here are the steps to run the generator:

Run in an environment with python enabled and poetry (see https://github.com/chairemobilite/evolution/tree/main/packages/evolution-generator#how-to-run step 1 for instructions to install poetry if necessary)
Install the poetry dependencies from the packages/evolution-generator directory: cd packages/evolution-generator && poetry install

Now, from the root of the project:

Run the generator: yarn generateSurvey:example
Make sure the evolution code is compiled before hand: yarn compile
Compile the generated code: yarn workspace demo_generator run compile:example

Ok, I'll check that once #794 is merged


Registering functions with multiple plugins

When using host functions and multiple plugins, I run into the following error:
extism.extism.Error: Function cannot be registered with multiple different Plugins

This happens in the following situation:
...

@extism.host_fn(name="do_something", namespace="app")
def do_something(): # some implementation

with extism.Plugin(...) as plugin:
  fn_with_import = lambda: plugin.call(...)

with extism.Plugin(...) as plugin:
  fn_without_import = lambda: plugin.call(...)

What works is this:
...

with extism.Plugin(...) as plugin:
  fn_without_import = lambda: plugin.call(...)

@extism.host_fn(name="do_something", namespace="app")
def do_something(): # some implementation

with extism.Plugin(...) as plugin:
  fn_with_import = lambda: plugin.call(...)

I was unable to find a way to scope host functions to their plugins. If this is possible, it'd be great to have documentation around it, if not, then it'd be great to have the functionality :)

Hey @fliepeltje - this is interesting... just to confirm, this is running from a Python SDK, not compiling code using the Python PDK to wasm.
assuming it is running in Python via the SDK, @zshipko or @chrisdickinson may have some ideas here!

It is with code compiled from the PDK.
I've got a plugin that looks something like this:
@extism.import_fn("app", "kv_store")
def kv_store(input: str): ...

@extism.plugin_fn
def do_something():
    kv_store(...)

Then I have another plugin without any import, both are compiled to wasm and via the sdk used as described above.

Hey! I think this is because we cut a new release of extism_sys before releasing the necessary changes on python-sdk, and python-sdk was configured to use a >= range on that dep (D'oh, apologies.)
I intend to release a new extism_sys and extism release today that should clear this up. Apologies for the trouble. I'll report back once I've published!

@chrisdickinson has this been published?


Pricefx Connector: Field Mapping Multi-Level

The field mappings does not take into account the 2nd level when considering data objects that have subsets. e.g. Product Extensions will map from any of the specific Product Extension tables rather than the specific one being used.

Fixed this. However, further work to improve the mapping is required. Not all fields are mapped back and forth as it stands.
Further research into the pfx api required.


Disconnect history for tests

Problem Statement
When running tests, you don't need the post_save and post_delete signals unless you're testing actual Simple History functionality.
Describe the solution you'd like
A documented way to turn off Simple History for all tests and then turn it back on for certain tests.

I don't see it documented anywhere but you can set SIMPLE_HISTORY_ENABLED = False in your Django settings for the tests.
Then both post_save and post_delete will be skipped. See https://github.com/jazzband/django-simple-history/blob/master/simple_history/models.py#L538

I'm assuming the reply from @duebbert resolves this issue, so closing. Please reopen if that's not the case 🙂


Simple case class derivation doesn't work with circe 2.11 - 8

I know it's been asked multiple times but for me simpler example throws compile error
object TestJsonLibs extends App {
      import io.circe.generic.auto._, io.circe.syntax._
      import io.circe.parser
      import io.circe.parser._
      import io.circe.{ Decoder, Encoder }
      import io.circe.generic.semiauto._
      import io.circe.generic.semiauto.deriveDecoder

      case class Staff(name: String)

      implicit val kcmDecoder = deriveDecoder[Staff] //compile error
}

Compile error:
could not find Lazy implicit value of type io.circe.generic.decoding.DerivedDecoder[com.mypckg.TestJsonLibs.Staff]

Closing since I can't reproduce this with circe 0.11.1 or either Scala 2.11 or 2.12. Feel free to open with more info, e.g.:

What version of circe are you using?
By "circe 2.11 - 8" do you mean "Scala 2.11.8"?
Can you minimize your imports?
Can you put an explicit return type on kcmDecoder, since inference for implicit definitions is often buggy or weird?

I updated case description with versions. I tried explicit return type as well. It still fails. It's silly but its happening. I found other library that works for me. thanks for help.


No catalog Vendor in my project in Yii 2.0.8

What steps will reproduce the problem?
What is the expected result?
What do you get instead?
Additional info

Q
A

Yii version

PHP version

Operating system

After setup Yii 2.0.8 No catalog in ny project in Yii2, Php version 5.5.30 in easyPhp, Operating sysstem Windows 8.1 x64! Plis, help me.

No token given, aborting. can also add it manually later by using .. Could not fetch https:/api.github.com/repos/ .... create a GitHub OAuth token to go over the API raye limit...
What I must will be? Can help me, please.

What is rate Linit?

rate limit?


[Config] Add import Dialog

This will add import Dialog for webview
ONE-vscode-DCO-1.0-Signed-off-by: sunjin an<EMAIL_ADDRESS>

From Issue #58
From Draft #117


questions

I am really appreciate about your work, I have a few questions

Is there an  special correlation between the final skeleton mesh and  point cloud? how can I get the original  point cloud from the skeleton mesh ?
how do you get skeleton meshs from a sequence in Skeleonization with consistent correspondence of More Applications? predict skeleton mesh per frame or other ways?

Hey!

There are two methods. (1) In MeshUtil.py, we have a function named "rand_sample_points_on_skeleton_mesh", and you can use it to densely sample a large quantity of 3D spheres on a skeletal mesh to approximate the skeletal mesh (see line 62-63 in test.py). Then you can use a virtual scanner (https://github.com/wang-ps/O-CNN/tree/master/virtual_scanner) to extract the surface points. This method is simple but time-consuming.
(2) Given a skeletal triangle with spheres centered at its vertices, you can calculate the two common tangent triangles to approximate the surface. For edge segments, it is similar, while you need to calculate a cone. Just like the figure below.

You just need to predict the skeletal mesh for each frame and the network will learn the correspondence automatically, e.g., the location of the predicted k-th vertex is almost consistent for all the inputs. Note to use this property, your input sequences should be aligned first.


num_batches_per_epoch not well accounted for by gluonts.torch.model.deepar.DeepAREstimator

Description
During each epoch, validation starts at the num_batches_per_epoch specified in the DeepAREstimator constructor call, but after this, training exhausts the whole training dataset, while validation runs concurrently.
Maybe I am not using the API correctly, or something is wrong with the gluonts version I'm using?
To Reproduce
Please check out this minimal example
Error message or code output
Before num_batches_per_epoch is reached, training proceeds as expected:
141 K     Trainable params
0         Non-trainable params
141 K     Total params
0.564     Total estimated model params size (MB)
Epoch 0: : 24it [00:01, 23.54it/s, loss=7.04, v_num=35]

After it is reached (50 in that case), training and validation run concurrently:
141 K     Trainable params
0         Non-trainable params
141 K     Total params
0.564     Total estimated model params size (MB)
Epoch 0: : 169it [00:08, 19.96it/s, loss=6.34, v_num=37]
Validation DataLoader 0: : 119it [00:05, 20.99it/s]

Environment

Operating system: Linux
Python version: 3.7.10
GluonTS version: v0.11.7
MXNet version: 1.9.1

@pbruneau I simplified your example by removing num_workers:
from gluonts.dataset.repository.datasets import get_dataset
from gluonts.torch.model.deepar import DeepAREstimator

dataset = get_dataset("electricity")
context_length = 2 * 7 * 24
prediction_length = dataset.metadata.prediction_length

model = DeepAREstimator(
    prediction_length=prediction_length,
    context_length=context_length,
    freq='1H',
    batch_size=32,
    num_batches_per_epoch=2,
)

predictor = model.train(
    dataset.train,
    dataset.test,
)

The validation data consists of 2247 series in this case, which are 71 batches of size 32; after the num_batches_per_epoch=2 batches are consumed for training, 71 are accounted for in the validation logging line, adding up to 73. I think the first logging line is just counting and timing the total number of batches processed.
It's left to see whether

logging is working as expected (this is handled by PyTorch Lightning, but the estimator class may be using it wrong)
num_workers is working as expected

Thanks for the follow-up @lostella, this is not the expected behaviour but at least you explain the numbers!
Following up on this, I tried to run the same example, just adding num_workers=2
The number of training steps looks right (i.e. validation starts after num_batches_per_epoch steps), but the number of validation steps is doubled... Maybe something with the slight differences between create_training_data_loader and create_training_data_loader? Looking at the code, I did not see anything obvious...

this is not the expected behaviour but at least you explain the numbers!

You mean logging should not present information that way?
In the num_workers=2 case, it seems like the two worker processes are consuming the whole set of validation batches each? If so, this is something that needs to be looked into

You mean logging should not present information that way?

Well, I would expect it to behave like the default gluonts progress bar (and like the progress bars with TF, for example), with distinct progress for train and validation. But I agree this is minor (and possibly tied to PyTorch Lightning as you mentioned).

In the num_workers=2 case, it seems like the two worker processes are consuming the whole set of validation batches each? If so, this is something that needs to be looked into

Maybe so, just that the count is aggregated in a single bar,
that is, with num_workers=2, the validation counter goes up to 142. Behind the scenes, indeed this could be that each worker consumes the whole validation set.

The issue of worker processes independently consuming the entire dataset affects not only validation, but also training set as well. This seems to be well documented here. While #2615 solves this issue, in my experiments where data loading is the major bottleneck, this impacts performance severely. Another more performant fix is to implement this selection logic in the underlying map-style gluonts dataset (arrow table, dictionary, etc.).

Yes, I think #2615 has the drawback that workers will still process the elements they skip (so all lazily evaluated transformation will still take time and slow the worker down)

The issue of worker processes independently consuming the entire dataset affects not only validation, but also training set as well. This seems to be well documented here. While #2615 solves this issue, in my experiments where data loading is the major bottleneck, this impacts performance severely. Another more performant fix is to implement this selection logic in the underlying map-style gluonts dataset (arrow table, dictionary, etc.).

Yes, you right. It better to use  map-style if it possible but if you use ItarableDataset class it means you can't. But the sistem should work predictable. Because today I have 52 000 timeseries and 10 workers and batch is 100 elements, approximatly one epoch takes 52 bathes but in practice I will cover all data (one epoc) in 520 iterations it looks dramatikal and unpredictable. It can be better to add shuffle for train and presplit for validation and test but it's still difficult for streaming data. This commit solves problem in general and another solution can be done fore specific tasks.

Agreed, I do feel that #2615 should be rolled out as a quick fix, since this is essentially quite a critical bug -- if you use 10 workers, you will see 10 of the same batches in a row during training. A more performant fix for map datasets can be made later.

I think this can be closed by #2628.
@pbruneau Can you verify if the behaviour is as expected on dev?

I have just checked, the numbers displayed look right, thanks! I'm closing the issue then.
I have a subsidiary question though; considering the example code from @lostella above:
from gluonts.dataset.repository.datasets import get_dataset
from gluonts.torch.model.deepar import DeepAREstimator

dataset = get_dataset("electricity")
context_length = 2 * 7 * 24
prediction_length = dataset.metadata.prediction_length

model = DeepAREstimator(
    prediction_length=prediction_length,
    context_length=context_length,
    freq='1H',
    batch_size=32,
    num_batches_per_epoch=2,
)

predictor = model.train(
    dataset.train,
    dataset.test,
)

If I change num_workers  in the model.train call, training seems to be faster indeed according to my monitor. But if I try to use my GPU with these lines juste before model.train:
model.trainer_kwargs={
    'gpus': -1 if torch.cuda.is_available() else None,
}

Having num_workers > 1 does now not seem to bring any benefit. Having several workers should be the way to increase GPU occupancy, right? Maybe there is an additional step to effectively do so?

We've removed num_workers from the train, so my assumption is that it should have no effect. We probably should add a warning if arguments are passed that are not used.
We want to re-add parallel dataloading in the future, but do it correctly that time. There is some work to be done to shard the underlying data correctly first.

All right, in my old version num_workers made a difference, but indeed since I pulled in dev it is not the case any more. Thanks for the anwser @jaheba!


How to configure the agent in build.sbt

Hello I would like to know how to configure the otel.javaagent.extensions in build.sbt using https://github.com/sbt/sbt-javaagent

Thanks for the answer.


Segmentation Fault in s3 getObject when timeouts are set in client config

Confirm by changing [ ] to [x] below to ensure that it's a bug:

[x] I've gone though Developer Guide and API reference
[x] I've searched for previous similar issues and didn't find any solution

Describe the bug
There is a crash that happens 100% of the time (not sporadic) when long timeouts are used for connectTimeoutMs and requestTimeoutMs, for an Aws::S3::Model::GetObjectRequest and an Aws::S3::S3Client. This is using default SDKOptions at Init time. When these two settings are not used, no crash occurs.
SDK version number
1.8.8
Platform/OS/Hardware/Device
OS: Linux
Arch: x86-64
Code
    Aws::SDKOptions options;
    Aws::InitAPI(options);
    {
        const Aws::String bucket_name = "newcmbucket";
        const Aws::String object_name = "functest/csv/ds_airline_1/air1.csv";
        if (!getObject(object_name, bucket_name))
        {
            return;
        }

and the the getObject local function is
bool Some::getObject(const Aws::String& objectKey,
                           const Aws::String& fromBucket)
{
    Aws::Client::ClientConfiguration clientConfig;
    clientConfig.scheme = Aws::Http::Scheme::HTTPS;
    clientConfig.region = Aws::Region::US_WEST_2;
    clientConfig.connectTimeoutMs = 100000; // if this is removed no crash occurs
    clientConfig.requestTimeoutMs = 600000; // if this is removed no crash occurs
    Aws::S3::S3Client s3_client(clientConfig);
    Aws::S3::Model::GetObjectRequest object_request;
    object_request.SetBucket(fromBucket);
    object_request.SetKey(objectKey);
    Aws::S3::Model::GetObjectOutcome get_object_outcome =
            s3_client.GetObject(object_request); // crash happens here.

Stack Trace
Received fatal signal 11 (Segmentation fault).
 Cause:
   No memory mapped at address [0x00007FC800000017].
 Crashing thread: someThread
 Registers:
    RIP:  [0x000055DEB5DF526F] std::_Rb_tree<Str, std::pair<String const, String>, std::_Select1st<std::pair<String const, String> >, std::less<String>, std::allocator<std::pair<String const, String> > >::_M_erase(std::_Rb_tree_node<std::pair<String const, String> >*)   15 (food   0x196B26F)
    RDI:  [0x00007FC7715F3880]
    RSI:  [0x00007FC7FFFFFFFF]
    RBP:  [0x00007FC7715F3600]
    RSP:  [0x00007FC7715F35C0]
    RAX:  [0x000055DEB8CBD210]
    RBX:  [0x00007FC7FFFFFFFF]
    RCX:  [0x00007FC7FFFFFFFF]
    RDX:  [0x000055DEB8D3A3F0]
    R8:  [0x00007FC7715F3B00]
    R9:  [0x00007FC779200080]
    R10:  [0x0000000000000092]
    R11:  [0x00007FC779756AA0]
    R12:  [0x00007FC7715F3880]
    R13:  [0x00007FC7715F5090]
    R14:  [0x00007FC7715F5420]
    R15:  [0x00007FC7715F5990]
    EFL:  [0x0000000000010206]
    TRAPNO:  [0x000000000000000E]
    ERR:  [0x0000000000000004]
    CSGSFS:  [0x0000000000000033]
    OLDMASK:  [0x0000000000000000]

 OS: Linux
 Arch: x86-64

 Backtrace (PIC build):
  [0x000055DEB5DF526F] std::_Rb_tree<Str, std::pair<String const, String>, std::_Select1st<std::pair<String const, String> >, std::less<String>, std::allocator<std::pair<String const, String> > >::_M_erase(std::_Rb_tree_node<std::pair<String const, String> >*)   15 (food   0x196B26F)
  [0x000055DEB69C8BBF] DateParser::~DateParser()   79 (food   0x253EBBF)
  [0x000055DEB7D20464] RFC822DateParser::~RFC822DateParser()   42 (food   0x3896464)
  [0x000055DEB7D1E910] Aws::Utils::DateTime::ConvertTimestampStringToTimePoint(char const*, Aws::Utils::DateFormat)   284 (food   0x3894910)
  [0x000055DEB7D1DD58] Aws::Utils::DateTime::DateTime(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, Aws::Utils::DateFormat)   72 (food   0x3893D58)
  [0x000055DEB7B1224C] Aws::S3::Model::GetObjectResult::operator=(Aws::AmazonWebServiceResult<Aws::Utils::Stream::ResponseStream>&&)   1100 (food   0x368824C)
  [0x000055DEB7B11DFD] Aws::S3::Model::GetObjectResult::GetObjectResult(Aws::AmazonWebServiceResult<Aws::Utils::Stream::ResponseStream>&&)   577 (food   0x3687DFD)
  [0x000055DEB7C28082] Aws::Utils::Outcome<Aws::S3::Model::GetObjectResult, Aws::S3::S3Error>::Outcome<Aws::AmazonWebServiceResult<Aws::Utils::Stream::ResponseStream>, Aws::Client::AWSError<Aws::Client::CoreErrors>, 0>(Aws::Utils::Outcome<Aws::AmazonWebServiceResult<Aws::Utils::Stream::ResponseStream>, Aws::Client::AWSError<Aws::Client::CoreErrors> >&&)   44 (food   0x379E082)
  [0x000055DEB7AD87BA] Aws::S3::S3Client::GetObject(Aws::S3::Model::GetObjectRequest const&) const   1332 (food   0x364E7BA)
  [0x000055DEB735B784] SomeThread::generateResults()   292 (food   0x2ED1784)
  [0x000055DEB5DC7AB5] SomeThread::main()   85 (food   0x193DAB5)
  [0x000055DEB6A9E551] Thread::callMain(void*)   129 (food   0x2614551)
  [0x00007FC77A56FE25] ? (libpthread.so.0   0x7E25)
  [0x00007FC7796C8BAD] clone   109 (libc.so.6   0xFEBAD)

** Build Options **
BUILD_SHARED_LIBS=OFF
Primarily, I don't need to use timeouts so theres no pressing issue, but I am curious why a crash would occur based on a timeout? Are multiple threads use to process a single request for a get object? Whats the general recommendation for setting timeouts in a client config ?

Looks like this was actually due to an equivalent class named DateParser in my code. Wrapping some of the AWS SDK class in DateTimeCommon.cpp in an anonymous namespace solved this, ill put out a PR for this soon on this repo to fix it for others.

@jweinst1 ,
Thanks for updating on this,glad it worked out for you and sorry for the lack of response, really appreciate you coming back to close it.


FXI: add pypdf2

Installed manually at the beamline for now (xf18id-ws2), needs to be deployed automatically during the next deployment.

Thanks for merging, @danielballan!
@gmysage, this will be deployed in the next cycle automatically.


Web - Profile - Upload/Remove popup displayed covering the avatar image

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

Action Performed:

Go to https://staging.expensify.cash
Log in with expensifail account
Click on Picture and select Profile
Click on Pen icon

Expected Result:
Upload/Remove popup should not be covering the picture
Actual Result:
Upload/Remove popup is covering the picture
Workaround:
Unknown
Platform:
Where is this issue occurring?
Web ✔
iOS
Android
Desktop App
Mobile Web
Version Number:
1.0.77-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

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

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

Duplicate of #3895

Closing in favour of https://github.com/Expensify/Expensify.cash/issues/3895


Improving the keyboard controls.

I made a reddit comment that goes into full detail THERE.

Love it!
Personally I would only use +/- because they're on the UX ...but I'm not invested so have at 'er!


[BUG] iOS 18 Chinese rendering bug

Description
I've experiencing this regression on iOS 18 when drawing the degree symbol °.
iOS 17.5

iOS 18.0

Code
Simple project to reproduce: https://github.com/cho-trackman/SkiaIos18Bug
Expected Behavior
A normal looking degree symbol (°). It works on all iOS versions prior to 18. It also works in Japanese and Korean on iOS 18.
Actual Behavior
You'll see different character than a degree symbol.
Version of SkiaSharp
2.88.8 (Current)
Last Known Good Version of SkiaSharp
Other (Please indicate in the description)
IDE / Editor
Other (Please indicate in the description)
Platform / Operating System
iOS
Platform / Operating System Version
iOS/iPadOS 18.0
Devices
No response
Relevant Screenshots
No response
Relevant Log Output
No response
Code of Conduct

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

I've just added a CoreGraphics part to test repo (https://github.com/cho-trackman/SkiaIos18Bug), but also realized that maybe it should have been done in CoreText to reflect how Skia is drawing (I don't know).

Sample project updated with CoreText as well.
iOS 17.5

iOS 18.0.1

Can you try the latest preview 3.118? I wonder if there has been an update in the engine.

If you ask me, what is to investigate here is how the PingFang SC font changed, the one shipped with iOS 18.
The fact that UILabel/Core-libs-whatever render that symbol means absolutely nothing in that regard, personally I would expect them to find the appropriate font for every glyph inside the text to render. While in the repro we look for appropriate font for the symbol 单 only.
An example of this is when you pass an emojy to a system label and specify a font where it doesn't exist. A normal behavior for such control would be to find an installed font where this glyph is present and render it. If you do same with skia low-level DrawText you won't get it rendered correctly if it's missing. It's by design, it's okay, low-level renderers dont have to provide hi-level logic for you.
So one probability is if that glyph is missing from PingFang SC shipped with iOS 18 this issue is to be closed.
To be able to render complex text with emojies and such glyphs without worrying about what font to use when glyph is missing some would use RichTextKit, some would use DrawnUI SkiaMarkdownLabel see example.

Just tested on 3.118.0-preview.1.2 and the issue is still there. I will look into your other suggestions and come back with more info.

Also, what happens if you remove the Chinese character? The character might be causing an issue somehow.

The degree char gets drawn as below, removing it removes the rectangle with the circle inside.

So one probability is if that glyph is missing from PingFang SC shipped with iOS 18 this issue is to be closed.

Don't mix up Unicode and ASCII.

I just stumbled upon this thread https://forums.developer.apple.com/forums/thread/758189.
An Apple dev claims:

PingFang.ttc is no longer in /System/Library/Fonts/Core. iOS 18 has a new Chinese UI font.

That is crazy news to me :O

This must be the issue:

https://forums.developer.apple.com/forums/thread/758189?answerId=811700022#811700022


Use single shared random string generation function

Replace 3 functions that do the same with 1 shared one
Output only alphanumerical for URL compatibilty
Seed the math RNG on program start

Replaces: #15536

The previous generate.GetRandomString did not seed the RNG. It was only seeded (repeatedely)  if random avatars were generated, so this might also be a security improvement because go docs say:

If Seed is not called, the generator behaves as if seeded by Seed(1)

I think we need to check whether we actually need to be using a cryptographic random number for these things. I suspect that in almost all of these cases this should be cryptographic.

Indeed, I think it's used for tokens and stuff so crypto/rand should be preferred.

Yeah I think it's best to change the function to use crypto/rand.
We're not using these random numbers as throwaways for an algorithm - they need to be cryptographically random

Yeah, I guess we can likely just use the crypto function in all places, generating short strings is usually fast enough even for cases where security is no concern.

Move to crypto/rand done.
I also found that modules/context/secret.go was unused and a 99% copy of modules/secret/secret.go, so I deleted it.


Source code searching

Not really an issue but a question/idea. Currently in RDi, we use iSphere to search source code. Is there something similar to that in Code for i, or plans for something like that. I work in a lot of older COBOL programs and the like, and the source code search is often valuable in finding where a program is called from and various other quick search needs.
Thanks,
AJ

⚠ REMOVE THIS LINE AND ANY SENSITIVE INFORMATION BELOW! ⚠

Context
Version

Code for IBM i version
2.10.4

Visual Studio Code version
1.89.1

Operating System
win32_x64

Active extensions
COBOL (cobol): 10.5.29
Code Runner (code-runner): 0.12.2
Code Spell Checker (code-spell-checker): 3.0.1
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.5.0
CodeTour (codetour): 0.0.59
Db2 for IBM i (vscode-db2i): 1.0.0
ES7+ React/Redux/React-Native snippets (es7-react-js-snippets): 4.4.3
ESLint (vscode-eslint): 2.4.4
EditorConfig for VS Code (EditorConfig): 0.16.4
Emmet (emmet): 1.0.0
Error Lens (errorlens): 3.18.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
Git Project Manager (git-project-manager): 1.8.2
GitHub (github): 0.0.1
GitHub Authentication (github-authentication): 0.0.2
GitLens — Git supercharged (gitlens): 15.0.4
IBM i Debug (ibmidebug): 1.0.0
JSON Language Features (json-language-features): 1.0.0
Live Server (LiveServer): 5.7.9
Live Share (vsliveshare): 1.0.5918
Material Icon Theme (material-icon-theme): 5.3.0
Merge Conflict (merge-conflict): 1.0.0
Microsoft Account (microsoft-authentication): 0.0.1
Node Debug Auto-attach (debug-auto-launch): 1.0.0
Prettier - Code formatter (prettier-vscode): 10.4.0
TODO Highlight (vscode-todo-highlight): 1.0.5
Tabnine AI (tabnine-vscode): 3.104.0
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0

Remote system

Setting
Value

IBM i OS
n/a

Tech Refresh
n/a

CCSID Origin
65535

Runtime CCSID
65535

Default CCSID
37

SQL
Enabled

Source dates
Disabled

Enabled features

/QOpenSys/pkgs/bin
/usr/bin
/QSYS.lib/ILEDITOR.lib

/QSYS.LIB
/QIBM/ProdData/IBMiDebugService/bin

bash
attr
GETNEWLIBL.PGM
ILEDITOR.GETMBRINFO
QZDFMDB2.PGM
startDebugService.sh

git
iconv

grep
setccsid

ls

md5sum

sort

stat

tar

Shell env

Variants
{
  "american": "#@$",
  "local": "#@$"
}

Errors
[
  {
    "command": "/QOpenSys/usr/bin/qsh",
    "code": 1,
    "stderr": "CPF2111:  Library ILEDITOR already exists.",
    "cwd": "/home/ITAJ"
  },
  {
    "command": "export BUILDLIB=\"BC001\" && export CURLIB=\"BC001\" && export USERNAME=\"ITAJ\" && export HOST=\"BOONEDEV\" && export HOME=\"/home/ITAJ\" && export WORKDIR=\"/home/ITAJ\" && export LIBLS=\"TAATOOL QGPL QGDDM QTEMP RVILIB RVICUST MMAIL BC001 ITA05\" && env",
    "code": 1,
    "stderr": "bsh: BUILDLIB=BC001: is not an identifier",
    "cwd": "/home/ITAJ"
  },
  {
    "command": "export BUILDLIB=\"BC001\" && export CURLIB=\"BC001\" && export USERNAME=\"ITAJ\" && export HOST=\"BOONEDEV\" && export HOME=\"/home/ITAJ\" && export WORKDIR=\"/home/ITAJ\" && export LIBLS=\"TAATOOL QGPL QGDDM QTEMP RVILIB RVICUST MMAIL BC001 ITA05\" && env",
    "code": 1,
    "stderr": "bsh: BUILDLIB=BC001: is not an identifier",
    "cwd": "/home/ITAJ"
  }
]

Could this be what  you're looking for?

Eh, not exactly, with the RDi iSphere source file search you can specify the LIB you want to search in, or search all libs. That will only look for references within that source member.

For example,

This will search all libraries and all source members for a value


Make database selection dropdown list wider

SQL Operations Studio Version: sqlops 0.23.6 (8a3011756318a2ded1e6e84332230a2b2bf9c544, 2017-11-06T21:51:43.232Z)

Make the database selection dropdown list wider in SQL scripts tabs.
Long database name are being truncated.

It seems to me that it's not a good idea to make a drop-down list wider, it looks not so good
I propose to do with long names in the same way on the server panel

Crop long names and add dots to the end, and when navigating with a mouse, show a pop-up message with a full name


Trying to install on ubuntu 16.04 version with dotnet core version 2.1.816

Below is the error I get when trying to install,
" dotnet tool install --global dotnet-certificate-tool
error NU1202: Package dotnet-certificate-tool 2.0.6 is not compatible with netcoreapp2.1 (.NETCoreApp,Version=v2.1) / any. Package dotnet-certificate-tool 2.0.6 supports:
error NU1202:   - net50 (.NETFramework,Version=v5.0) / any
error NU1202:   - netcoreapp3.1 (.NETCoreApp,Version=v3.1) / any
The tool package could not be restored.
Tool 'dotnet-certificate-tool' failed to install. This failure may have been caused by:

You are attempting to install a preview release and did not use the --version option to specify the version.
A package by this name was found, but it was not a .NET Core tool.
The required NuGet feed cannot be accessed, perhaps because of an Internet connection problem.
You mistyped the name of the tool. "

i am trying to dotnet a .dll file to check if the application runs but having below error,
Unhandled Exception: System.Security.Cryptography.CryptographicException: Unix LocalMachine X509Store is limited to the Root and CertificateAuthority stores. ---> System.PlatformNotSupportedException: Unix LocalMachine X509Store is limited to the Root and CertificateAuthority stores.
Please suggest if there is any solution.

I have installed dotnet 2.1 runtime and now i find below error
Welcome to .NET Core!
Learn more about .NET Core @ https://aka.ms/dotnet-docs. Use dotnet --help to see available commands or go to https://aka.ms/dotnet-cli-docs.
Telemetry
The .NET Core tools collect usage data in order to improve your experience. The data is anonymous and does not include command-line arguments. The data is collected by Microsoft and shared with the community.
You can opt out of telemetry by setting a DOTNET_CLI_TELEMETRY_OPTOUT environment variable to 1 using your favorite shell.
You can read more about .NET Core tools telemetry @ https://aka.ms/dotnet-cli-telemetry.
No executable found matching command "dotnet-tool"

Got it no further issues. downloaded both .net core SDK and runtime to 3.1, issue solved


Color code on windows.

Hi, many thanks for great work. pyllamacpp is my favorite llama.cpp binding, and I love using it on my Mac. But on Windows, I see that 'color code' doesn't seem to be working, like
"You: What's the point of Zen Buddhism ?
AI: Zen Buddhism is a branch of Mahayana Buddhism that emphasizes the attainment of enlightenment through meditation and the experience of moment-to-moment awareness. The ultimate goal of Zen Buddhism is to realize one's true nature or[...."
Any idea how to fix this, please ?

Hi @kdsull,
You are welcome, glad you found the project useful.
Thank you for reporting this issue, I didn't know Windows does not support color codes!
I wanted the package to be dependency free, that's why I didn't want to use another package for coloring.
To fix this just go to this line and remove the colors.
Or you can just use WSL on windows, it should work I guess.
If you find any issues doing that, maybe I will add another option to the cli for you, to prevent using colors ?

Aha.. Yes, WSL seems to be the right answer. I need to use docker anyways, so installed WSL2, and it works fine.
Tricky part was how to find model files on Windows file system, but '/mnt/e/models/Wizard~' did the trick, and allocating more memory to wsl by making a '.wslconfig' file with 'memory=26GB' option.
Out of my curiosity, you heard about 'ggcc' for Falcon 40B model, and any idea how to apply this ? I'm addicted to pyllamacpp, and want no other implementations... I mean, it's so easy and robust !!  Cheers.

I only use Linux, so I don't know the details of how things work in Windows to be honest :sweat_smile:  .. but 'Glad it worked, WSL should be a Linux machine at the end of the day.
About 'ggcc', are you talking about this ?
I didn't know about it until now.  it seems really interesting as llama.cpp does not support Flacon yet afaik.
I wish I could integrate it in pyllamacpp, but I think I might run into several conflicts as both llama.cpp and gllm.cpp depend on a modified version of ggml.
But I will look into it when I have some time.
Many Thanks to you as well :), @kdsull

Yup, looks like 'ggllm.cpp' is the one. I heard about it from TheBloke card on huggingface, whose huge ggml works make me busy running on your pyllamacpp ! I wish you could find some time to implement it, coz this Falcon stuff seems pretty hot recently. In the meantime, I'll have to try running ggllm.cpp, and study your code. Many many domos again !

@kdsull, as you are really interested in using it, I got some time and decided to implement it for you. Hope it will be useful.
Luckily it is a fork of llama.cpp, so I made a unified backend around both models, and you can run ggcc models as well as other ggml models with the same old pyllmacpp API (I tried it with falcon and Wizard-Vicuna from TheBloke and they both work)
You can find the implementation in the ggllm branch.
clone and install or just go with pip install git+..
(if you are on windows don't forget to use WSL :))
give it a try and let me know how it goes ?


fix worker paths in parcel

I just tried this instructions and I found this small detail for they to work perfectly

Interesting. I'm using a Windows machine so the shell script wasn't an option. I used the following with Parcel and the script isn't needed:
self.MonacoEnvironment = {
  getWorker(_moduleId: string, label: string) {
    switch (label) {
      case "json":
        return new Worker("../node_modules/monaco-editor/esm/vs/language/json/json.worker.js");
      case "css":
        return new Worker("../node_modules/monaco-editor/esm/vs/language/css/css.worker.js");
      case "html":
        return new Worker("../node_modules/monaco-editor/esm/vs/language/html/html.worker.js");
      case "typescript":
      case "javascript":
        return new Worker("../node_modules/monaco-editor/esm/vs/language/typescript/ts.worker.js");
      default:
        return new Worker("../node_modules/monaco-editor/esm/vs/editor/editor.worker.js");
    }
  }
};

:+1: Thank you!


add daily build

Signed-off-by: Nobuhiro MIKI<EMAIL_ADDRESS>

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

Coverage data is based on head (60c2572) compared to base (0c264dc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #20   +/-   ##
=======================================
  Coverage   63.39%   63.39%
=======================================
  Files          10       10
  Lines         224      224
=======================================
  Hits          142      142
  Misses         82       82

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

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


cmd/govim: expose gopls "find all references"

Off the back of https://go-review.googlesource.com/c/tools/+/181122

This was implemented in #291


Code Sample: Adding Flutter to an app with FlutterFragment

Publish a code sample that demonstrates #21848

After this was filed (but some time ago), FlutterFragment arrived in a sample demonstrating how to use multiple Flutter engines in a single Android app:
https://github.com/flutter/samples/blob/main/add_to_app/multiple_flutters/multiple_flutters_android/app/src/main/java/dev/flutter/multipleflutters/DoubleFlutterActivity.kt
This issue should have been closed at that time, so I'm going to go ahead and close it now.


chore: Added 3 VSCode extensions to the extensions category

Related Issue
Closes #1509
chore : Added 3 VSCode extensions to the extensions category
Description
This PR adds 3 useful VSCode extensions enhancing productivity and optimizing the coding experience. 🚀
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@shreyT19 Tailwind CSS IntelliSense for VSCode the title can have atmost 2 words

@shreyT19 Tailwind CSS IntelliSense for VSCode the title can have atmost 2 words

So. Should I rename it to "TailwindCSS IntelliSense" then  ?. Can we create a new category called "vscode extensions" if it's possible.

@shreyT19 yes give a title which is smaller


Remove Login request page

I followed the steps mentioned here: https://github.com/jackdh/RasaTalk/issues/7#issuecomment-412575343
But still I get redirected immediately to the login-page. How can I remove this since I cannot register...

Hey @ctrado18 is there a particular reason you want to remove the login / authentication?
Can you try pulling the latest version and if you are using MongoDB add in the new environmental fields for MONGOUSER / MONGOPASS ?

Any luck @ctrado18 ?

I'm going to close this issue now as it's been a week @ctrado18 with no reply. Thanks!


Timeout extensions

Continuation of what's going on here. This is how I envisioned integrating the extensions into celluloid. Main changes are a few Exceptions, method signatures and the fact that the main timeout and sleep logic is directly inside the Celluloid module, instead of inside the actor.
Main discussion topics:

I think that the Actor should drop the timeout and sleep instance methods. I mean, they should be using timeout and sleep directly, which will revert to the handlers inside its thread. This will not be a breaking change for the ones who have been using timeout and sleep inside the actor scope as if they were a private method, but it would break such usages as Thread.current[:celluloid_actor].timeout(duration)...:
I think that one should drop Celluloid::Task::TimeoutError in favour of Timeout::Error. I mean, one should use the same exceptions as other potential libraries expect (Net::SSH, Net::HTTP, etc, they all work with it). I think one did it because of the ResumableError class, but since this has become a module, one could "extend" the Timeout::Error instance before resuming the task (see how this was solved under Celluloid.timeout).
A somewhat elephant in the room is Celluloid.sleep argument handling. I couldn't really mimic the Kernel.sleep method signature, as you can call sleep without arguments. The question here is, should one allow celluloid actors to sleep forever? Does the timers gem support this feature? Do you think this is a relevant use-case? I think one should at least throw a relevant exception for the case in which no argument is passed to a sleep evaluated inside an actor.

Need to get timeout-extensions working on Celluloid master first

@tarcieri , I've noticed the celluloid refactorings recently. Are the timeout extensions being considered for any of the planned minor releases?

@TiagoCardoso1983 given how invasive they are, I think I'd prefer to keep them out of core if possible. Perhaps this patch could be further modularized so that less of it depends on changes to Celluloid.

I would argue that these changes are not that invasive, but correct some fundamental flaws to the "time" methods. For instance, Celluloid.timeout currently takes one argument instead of two, like ruby's Timeout::timeout. Besides that I don't see what you mean by invasive.

Here is the all necessary monkey-patching needed for the extensions to work:
module Celluloid
  class Thread < ::Thread
    def initialize(*)
      self.timeout_handler = Celluloid.method(:timeout).to_proc
      self.sleep_handler = Celluloid::method(:sleep).to_proc
      super
    end
  end

  def self.timeout(duration, klass = Task::TimeoutError)
    bt = caller
    task = Task.current
    timers = Thread.current[:celluloid_actor].timers
    timer = timers.after(duration) do
      exception = klass.new("execution expired")
      exception.set_backtrace bt
      task.resume exception
    end
    yield
  ensure
    timer.cancel if timer
  end

  class Actor
    def timeout(*args)
      Celluloid::timeout(*args) { yield }
    end
    private :timeout
  end

  remove_const(:ResumableError)

  module ResumableError ; end

  class Task
    remove_const(:TerminatedError)
    remove_const(:TimeoutError)
    class TerminatedError < Celluloid::Error; include ResumableError ;end
    class TimeoutError < Celluloid::Error; include ResumableError ;end
  end

end

I could indeed puts this into a timeout-extensions celluloid script, but this looks like API which can be subject to change, and actually everything that timeout-extensions needs to work are those patches in the Thread class. The rest of it are just problems I had while using Celluloid::IO with some classes which call timeout with 2 arguments.

@digitalextremist could you review this? This was under discussion with @tarcieri , but was forgotten in the whole 0.17 migration and was a bit in the limbo. Nevertheless, this addresses the timeout issue discussed under the sidekiq thread (Namely that plain ruby timeout calls break actors).
There is currently one failing spec, and this has to due with the (again, discussed somewhere else) fact that TaskTimeout exceptions now inherit from Exception and can't be caught by the RSpec raise_error call (and again, this could also be the initial point for a rediscussion on that design decision, as I'm not particularly a fan of it).

@TiagoCardoso1983 no problem. Will review.

What are the performance implications of this additional layer of indirection? RubyDNS uses timeouts a lot in performance sensitive code for handling the (rare) case where the network doesn't respond in time. The overhead for timeouts that don't fire and are cancelled needs to be minimised at the expense of timeouts that do eventually fire, IMHO.

@ioquatix I'm not sure that I understood. This pull request doesn't change the way celluloid works, it just "moves" the timeout BL to the Celluloid module. Why? Because it makes it easier to use here.
I would argue that network not respond in time is not rare, and in the context of celluloid, you'd need to prevent the usage of the ruby timeout/sleep,  which will block the actor message-processing. In the context of celluloid-io, which can be used to patch stdlib and other network libraries, which use Timeout.timeout extensively, that's even more important.

Okay I will review.

As you may have seen by now, this was applied to an ancient version of celluloid. It has been scattered across subgems as of 0.17, which makes this harder to track. What concerns celluloid, the parts I'd like to maintain in the pull request are migrating the BL of timeout to Celluloid module, where Celluloid.sleep is. I guess, since they're timer based tasks, it makes sense to have them close to each other and publicly acessible.
The point of overhead you might have noticed comes from the thread pool spawner. timeout-extensions defines two internal accessor methods to keep pointers to an alternative thread-local timeout/sleep implementation (it doesn't need to be celluloid-specific). Therefore, on thread spawing, Celluloid timeout/sleep are passed to the thread. timeout-extensions proceeds to "monkey-patch" Timeout.timeout and Kernel.sleep (when the thread has the method locally-assigned, it calls it).
This timeout/sleep overwrites have been discussed ages ago with @tarcieri . He was never fond of "monkey-patching" Kernel and Timeout.timeout (who is?), rather wanted to "push" the community to adopt the approach. I decided not to wait and have been using it in production in my company internal projects without significant issues (main issues: 1) Celluloid::Task::TimeoutError inherits from Exception instead of StandardError/Timeout::Error, makes rescu'ing a bitx; and 2) every call to sleep/timeout will have timeout-extensions in the stack trace, prompting users to single it out whenever they have issues with timers, although the ruby timeout/sleep are ultimately executed).

I completely agree with you on all points. We will try to resolve these issues for 0.18.0 including merging your changes.

No worries, I just wanted to give you the history of this issue.

@HoneyryderChuck not sure if you're interested in picking this up. Many of the subgems are now unified again, which should make this change easier.

@tarcieri haven't been using celluloid for a while, so this stopped becoming a priority for me. Besides that, I think the PR is sort of complete from my side, so it's a matter of it being worthy of being merged or not. I'm actually surprised there are no conflicts, given how much has changed...


MdCheckBox - error when [(ngModel)] on md-checkbox inside ngFor

<tr *ngFor="let alarm of alarms">
  <md-checkbox [(ngModel)]="alarm.isActive"></md-checkbox>
</tr>

throws an error in the console:

"Uncaught (in promise): TypeError: Cannot set property 'stack' of undefined
TypeError: Cannot set property 'stack' of undefined
at SyntaxError.set [as stack] (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :1628:63)
at assignAll (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :704:29)
at SyntaxError.ZoneAwareError (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :775:16)
at SyntaxError.BaseError [as constructor] (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :1590:18)
at new SyntaxError (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :1793:18)
at TemplateParser.parse (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :11210:21)
at JitCompiler._compileTemplate (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :27480:70)
at eval (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :27363:64)
at Set.forEach (native)
at JitCompiler._compileComponents (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :27363:21)
at createResult (eval at  (file:///C:/projects/school/dist/vendor.js:17:2), :27246:21)
at ZoneDelegate.invoke (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :242:26)
at Zone.run (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :113:43)
at eval (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :535:57)
at ZoneDelegate.invokeTask (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :275:35)
at Zone.runTask (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :151:47)
at drainMicroTaskQueue (eval at  (file:///C:/projects/school/dist/polyfills.js:1744:2), :433:35)"

Looks like an error with<EMAIL_ADDRESS>(see: https://github.com/angular/angular/search?q=zone.js+0.7.5&type=Issues)
npm install<EMAIL_ADDRESS>resolved similar issue for me

then it would throw the following error

(program):405 Unhandled Promise rejection: Template parse errors:
Can't bind to 'ngModel' since it isn't a known property of 'md-checkbox'.

If 'md-checkbox' is an Angular component and it has 'ngModel' input, then verify that it is part of this module.
If 'md-checkbox' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.

Seems to work for me: http://plnkr.co/edit/AcJc12SViI5VGKMyLgVj?p=preview

my mistake. forgot FormsModule ..


Need to update conda-forge recipe

conda-forge version is 1.7.1 which encounters the following error in proteus_notebooks/Tutorials/Tutorial_1/Introduction.ipynb
# scotch has been compiled with SCOTCH_PTHREAD
# so MPI_Init() must be called with MPI_THREAD_MULTIPLE, or MPI_Init_thread() must be called.
# at import time, mpi4py initializes the MPI execution environment calling MPI_Init_thread().
# mpi4py must be imported before petsc4py otherwise petsc4py initializes MPI with MPI_Init().

need to remove .h5 references in setup.py file

conda-forge version is 1.7.1 which encounters the following error in proteus_notebooks/Tutorials/Tutorial_1/Introduction.ipynb
# scotch has been compiled with SCOTCH_PTHREAD
# so MPI_Init() must be called with MPI_THREAD_MULTIPLE, or MPI_Init_thread() must be called.
# at import time, mpi4py initializes the MPI execution environment calling MPI_Init_thread().
# mpi4py must be imported before petsc4py otherwise petsc4py initializes MPI with MPI_Init().

This can be solved by importing mpi4py first thing, see #1047.

@davidbrochart Yes this was resolved already with the previous release. If I recall correctly, you had run into the problem earlier and resolved it on the master branch, but it was not included in one of the releases. conda-forge/proteus-feedstock#18 resolved this issue.


Get rid of FullTrust for ContextMenu and AppLaunch handlers

Let's do it in a progressive way.
As a beginning, aiming at ContextMenu and AppLaunch.
Opened an issue for traking: #9858

@hez2010 this is great! Have you noticed any performance improvements?

Have you noticed any performance improvements?

The loading speed of context menu is faster, though not that obvious. Also it doesn't require async loading, so the context menu will be fully loaded before showing.

so the context menu will be fully loaded before showing.

I'll have to try this out but the current behavior of dispalying the menu before it finished loading was done on purpose since it there was often a two or three second delay.

there was often a two or three second delay.

I also noticed it. I'm making changes to make the context menu be loaded in the background.

I'm making changes to make the context menu be loaded in the background.

Done.

Added a commit to cancel context menu loading eagerly to avoid duplicated entries in the context menu being added if user quickly right-clicks twice.

I think this PR is ready for merge.
@yaichenbaum @lukeblevins PTAL.

It seems it does not show all the menu items, e.g with a zip file 7zip menu is not shown.


It seems that the batchsize matters much for beit?

I am trying to reproduce the results of Beit of base.

I used 4 DGX-2 boxes for 22k pretraining and intermediate finetuning. And I do not change the batchsize.
Then I used 2 DGX-2 boxes for finetuing on 1k with batchsize 128.
I obtained 83.4; 83.5 top1 acc for beit_base_patch16_224_pt22k and beit_base_patch16_224_pt22k_ft22k.
Later, I change batchsize to be 32 with only one DGX-2 box for beit_base_patch16_224_pt22k_ft22k training, it delivers 84.6.
For 384*384 of beit_base_patch16_224_pt22k_ft22k, the performance is 86.2.
I have several questions:

Do I need to keep the same total batchsize no matter how many gpus I used ?
I disabled the deepspeed to avoid errors, does that caused the performance degradation ?
Does the total batchsize matters a lot when doing pretraining ? Because the pretraining on 22k is very time-comsuming and I need to use 4 or 6 DGX-2 boxes to make the training  time bearable for me.
Could you give some advices for pretraining/finetuning of the changes on super-parameters?

Many thanks in advance!

Hi @Zhaoyi-Yan ,
Q1: I recommend ensuring that the total batch size is kept the same and that the same learning rate is used to ensure reproducibility.
Q2: Deepspeed can speech up the training by using APEX O2 instead of the APEX O1 in torch.amp, more details can be found at this URL.
Q3 & Q4: We didn't try to adjust the learning rate in pretraining because it is costly and time-consuming you mentioned. It is relatively easy and fast to adjust the learning rate in the finetuning stage. We follow BERT to try several different learning rates in fine-tuning stage, as shown in Appendix H of our paper. It will need to reduce the learning rate and increase the layer decay when fine-tuning the models intermediate fine-tuned on ImageNet-22k. The more fine-tuning scripts can be found at this URL.


Install-to-internal-storage script broken on arch

It looks like the arch installer uses the same script as ubuntu for install-to-internal-storage, so it exits with apt not found

the script works fine otherwise, just needs some extra stuff to build some aur stuff from source :/

something like this

git clone https://aur.archlinux.org/flashrom-git.git
git clone https://aur.archlinux.org/trousers.git
git clone https://aur.archlinux.org/vboot-utils.git

cd ./flashrom-git  && makepkg -cis
cd ../trousers     && makepkg -cis
cd ../vboot-utils  && makepkg -cis

pacman -Syu rsync curl

cd ../
rm -rf ./flashrom-git ./trousers ./vboot-utils

I've updated the script to include this in the latest commit. It'll be included in new builds, but you can download the script from bin  and make it executable to replicate this.


Update da.yml

Additions, corrections, consistence

This file was updated on https://www.localeapp.com/projects/377 , so there are conflicts now. Please review to see if there is still anything to update.

I looked through the translations and merged. Changed one of the translations in patch-1 based and kept the rest in favor of the ones from localeapp as I think they are more correct and matches the English versions better

Tak!


Update known_good.json

Rollback MoltenVK to 1.2.9 from 1.2.10 due to stability issues

CI Vulkan-Tools build queued with queue ID 222259.

CI Vulkan-Tools build queued with queue ID 222259.

CI Vulkan-Tools build # 1492 running.

CI Vulkan-Tools build # 1492 passed.


Fixing Bug https://jira.sonarsource.com/browse/SVS-60: Rename the "Sy…

…nc" context command on the SonarQube project in Team Explorer to "Update"

LGTM

Rebased and pushed.


[Problem] Insertion Sort implementation

Insertion Sort
For given array A[12, 13, 8, 2, 16, 5, 0, 1, 4] implement the insertion sort algorithm.

can I work on this?

Yes, no problem.
On Sat, Aug 13, 2022, 9:38 AM ATHUL SURESH @.***> wrote:

can I work on this?
—
Reply to this email directly, view it on GitHub
https://github.com/ademclk/callofcode/issues/4#issuecomment-1213838077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTBEJ737JSYST575QELNTDVY47H5ANCNFSM56NQZIVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

C++ implementation done. Waiting for more programming languages.


[expo-zustand-styledcomponents] Avoid page re-render

Description

We need to find a way to ensure the main page doesn’t need to re-fetch everything we visit as it has been visited before. There is probably a condition making it re-render.

I noticed this happens only on the web but on an actual mobile device, it's okay.


[dashboard] Refresh Notebooks

Opening this issue will trigger GitHub Actions to refresh all notebooks.
cc: @pratapvardhan

Only I can trigger this at the moment ...


Merge pull request #96 from MicrosoftDocs/master

sync branches

:warning: Validation status: warnings
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.