Fix for Error: Error in function call (...) aws_subnet.public is empty tuple (at least in some cases)

Hello,
I created VPC from this module, but I got following error during the development:

Error: Error in function call
│
│   on .terraform/modules/vpc/main.tf line 1025, in resource "aws_nat_gateway" "this":
│ 1025:   subnet_id = element(
│ 1026:     aws_subnet.public[*].id,
│ 1027:     var.single_nat_gateway ? 0 : count.index,
│ 1028:   )
│     ├────────────────
│     │ aws_subnet.public is empty tuple
│     │ count.index is 0
│     │ var.single_nat_gateway is false
│
│ Call to function "element" failed: cannot use element function with an empty list.
╵

I was creating it in us-east-1 region and providing following configuration:
data "aws_availability_zones" "available" {
  state = "available"
}

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "3.14.0"
  # insert the 23 required variables here
  name = "TestVPC"
  cidr = "<IP_ADDRESS>/16"

  azs              = slice(data.aws_availability_zones.available.names, 0, 3)
  private_subnets  = ["<IP_ADDRESS>/22", "<IP_ADDRESS>/22", "<IP_ADDRESS>/22"]
  public_subnets   = ["<IP_ADDRESS>/22", "<IP_ADDRESS>/22", "<IP_ADDRESS>/22"]
  database_subnets = ["<IP_ADDRESS>/22", "<IP_ADDRESS>/22", "<IP_ADDRESS>/22"]

  enable_nat_gateway     = true
  single_nat_gateway     = false
  one_nat_gateway_per_az = true

  public_dedicated_network_acl = true
  # public_inbound_acl_rules     = {}

  private_dedicated_network_acl = true
  # private_inbound_acl_rules     = {}

  enable_vpn_gateway = true

  tags = var.tags
}

The solution was to change:
azs              = slice(data.aws_availability_zones.available.names, 0, 3)
Because, us-east-1 have 6 AZs,I was providing only 3 sets of subsets so terraform couldn't iterate further.
Maybe is worth to consider, changing iteration from AZ to for example number of public subnets. As users we don't have control on number AZs in region, but we know exactly how many of them we want to use to meet project requirements.
Best,
Rafal

the number of AZs is always left up to users

Hello @bryantbiggs,
Size of list with AZ will differ depends on region US-East-1 is 6, US-East-2 is 3, US-West-1 is 2, US-West-2 is 4. So is not totally left up to users, because if I would like to create generic template with 2 sets of subnets with:
data "aws_availability_zones" "available" {
  state = "available"
}

for AZ names listing, in 3 from 4 cases I will receive that error. I have to leverage slicing of list to avoid this.

I have to leverage slicing of list to avoid this

Correct - this is up to users. The module does not make assumptions about how many AZs users are utilizing nor how many subnets per AZ; this is up to users and how they want to design their network


Understand why scripts loaded from binary UPLC have a different address.

The Problem
Scripts loaded from binary UPLC have a different address than scripts loaded from a PlutusTx.compile; because the address
of a script (AFAIK) is the hash of its source, this behavior raises an alarm that serializing then deserializing a script could be changing its source.
The Test
Here is the test that should pass, but was marked as expectedFail until we investigate this issue.
Here's the output of running the test suite as of f0501c3:
  SplitSpec imported from UPLC
    Simple example succeeds:                                OK (0.04s)
    Same address as compiled script:                        FAIL (expected)
      tests/SplitUPLCSpec.hs:90:
      expected: Address {addressCredential = ScriptCredential 7cf78c7583416d52a7cec69455392fe690ed4f02c832f5742eb4be71, addressStakingCredential = Nothing}
       but got: Address {addressCredential = ScriptCredential de83fdcc19466a286e174a03de5ec9228ef642a7ab607ddf32e8f962, addressStakingCredential = Nothing} (expected failure)

Deserializing Scripts
The splitValidator is defined as:
splitValidator = Scripts.mkTypedValidator @Split
                   $$(PlutusTx.compile [||validateSplit||])
                   $$(PlutusTx.compile [||wrap||])

by the definition of mkTypedValidator, is somewhat equivalent to:
sv = Scripts.mkValidatorScript $
       $$(PlutusTx.compile [||wrap||])
       `applyCode` $$(PlutusTx.compile [||validateSplit||])

which, in turn, expands to (through mkValidatorScript)
sv = Validator . fromCompiledCode $
        $$(PlutusTx.compile [||wrap||])
        `applyCode` $$(PlutusTx.compile [||validateSplit||])

Now, expanding fromCompiledCode we have:
sv = Validator . fromPlc . getPlc $
       $$(PlutusTx.compile [||wrap||])
       `applyCode` $$(PlutusTx.compile [||validateSplit||])

Now, because unflat . flat should be equal to Right, I'd have expected Right sv to be equivalent to:
Right sv = fmap (Validator . fromPlc) . unflat . flat . getPlc $
             $$(PlutusTx.compile [||wrap||])
             `applyCode` $$(PlutusTx.compile [||validateSplit||])

Finally, to wrap all of this into a TypedValidator Split, we do:
splitFromBS = unsafeCoerce . fmap (unsafeMkTypedValidator . Validator . fromPlc) . unflat $ splitBS

where splitBS is defined as:
splitBS = flat . getPlc $
  $$(PlutusTx.compile [||wrap||])
  `applyCode` $$(PlutusTx.compile [||validateSplit||])

The Punchline
I'd have expected that splitFromBS == validateSplit, but that is clearly not the case since these scripts end up with different addresses

Can you investigate which of the above steps is not valid? My guess would be that unflat . flat does not roundtrip properly.

I think unflat . flat is no issue:
splitBS :: ByteString
splitBS = flat $ splitPLC

splitPLC =
    PlutusTx.getPlc $
      $$(PlutusTx.compile [||wrap||])
        `PlutusTx.applyCode` $$(PlutusTx.compile [||validateSplit||])
  where
    wrap = Scripts.wrapValidator @SplitDatum @SplitRedeemer

splitPLC' = let Right res = unflat splitBS in sameType splitPLC res
  where
    -- I don't want to write these huge types, let me just convince Haskell its
    -- the same type.
    sameType :: a -> a -> a
    sameType a = id

Then, I load that into GHCi and run:
λ> show splitPLC == show splitPLC'
True

One more edit: can you add a type signature to unsafeCoerce to make sure the coercion is not the issue.

On the call site unsafeCoerce has a type signature, it only really changes the phantom type variable to TypedValidator: https://github.com/tweag/plutus-libs/blob/main/cooked-validators/src/Cooked/MockChain/RawUPLC.hs#L23

Here's another test, not flattening to bytestrings at all:
splitValidator :: Scripts.TypedValidator Split
splitValidator =
  Scripts.mkTypedValidator @Split
    $$(PlutusTx.compile [||validateSplit||])
    $$(PlutusTx.compile [||wrap||])
  where
    wrap = Scripts.wrapValidator @SplitDatum @SplitRedeemer

splitValidator' =
    fromPlc $
      PlutusTx.getPlc $
        $$(PlutusTx.compile [||wrap||])
          `PlutusTx.applyCode` $$(PlutusTx.compile [||validateSplit||])
  where
    wrap = Scripts.wrapValidator @SplitDatum @SplitRedeemer

    fromPlc :: UPLC.Program UPLC.NamedDeBruijn UPLC.DefaultUni UPLC.DefaultFun () -> Script
    fromPlc (UPLC.Program a v t) =
          let nameless = UPLC.termMapNames UPLC.unNameDeBruijn t
           in Script $ UPLC.Program a v nameless

splitAddr = Scripts.validatorAddress splitValidator
splitAddr' = scriptAddress (Validator splitValidator')

Now:
λ> splitAddr
Address {addressCredential = ScriptCredential 7cf78c7583416d52a7cec69455392fe690ed4f02c832f5742eb4be71, addressStakingCredential = Nothing}
λ> splitAddr'
Address {addressCredential = ScriptCredential 906b19c1f7119b7d3c13e1b0fed99a335d179fca8e7d75c711f4be7e, addressStakingCredential = Nothing}

As @sjoerdvisscher pointed out, this could be due to normalization. There's a reference to an nonexistent note in here: https://github.com/input-output-hk/plutus/blob/1f31e640e8a258185db01fa899da63f9018c0e85/plutus-ledger-api/src/Plutus/V1/Ledger/Scripts.hs#L164


additional labels / affinity rules calculated by machine driver

Related issue: https://github.com/harvester/harvester/issues/4588
When a VM set is launched from the Harvester UI, and anti-affinity rules are specified, then Harvester enhances the VM definitions to add additional labels in vm.spec.metadata.labels
      labels:
        harvesterhci.io/vmName: ubuntu-vmset-01
        harvesterhci.io/vmNamePrefix: ubuntu-vmset

The additional vmNamePrefix label is used during defining podAffinity or podAntiAffinity
        podAntiAffinity:
          preferredDuringSchedulingIgnoredDuringExecution:
          - podAffinityTerm:
              labelSelector:
                matchLabels:
                  harvesterhci.io/vmNamePrefix: ubuntu-vmset
              topologyKey: kubernetes.io/hostname
            weight: 1
          requiredDuringSchedulingIgnoredDuringExecution:
          - topologyKey: topology.kubernetes.io/zone
      domain:

Rancher machine driver does not pass this information when provisioning nodes.
We can however evaluate the rancher machineset name from the VM name as it follows the convention clusterName-poolName-generatedID
The PR leverages this logic to add an additional label to VM's
  labels:
    harvesterhci.io/creator: docker-machine-driver-harvester
    harvesterhci.io/machineSetName: affinity2-pool1-0d403cc4
    harvesterhci.io/vmName: affinity2-pool1-0d403cc4-4qzs7

This additional label can then be used to generated more refined anti affinity rules
        podAntiAffinity:
          preferredDuringSchedulingIgnoredDuringExecution:
          - podAffinityTerm:
              labelSelector:
                matchLabels:
                  harvesterhci.io/machineSetName: affinity2-pool1-0d403cc4
              topologyKey: kubernetes.io/hostname
            weight: 1
          requiredDuringSchedulingIgnoredDuringExecution:
          - topologyKey: topology.kubernetes.io/zone

Thank you for the fix! I have some questions, regarding the default affinity and anti-affinity terms. If I understand the changes correctly, it will add a default affinity term for the hypervisors in case there are other affinity terms defined, and the same logic applies for anti-affinity.
I don't see the logical connection of any affinity term and hostname affinity term based on pool. If I set up an affinity term between the VMs, I don't necessarily want the pool members to be on the same hypervisor.
Also, including the generatedID would make it hard (or even impossible) to set up specific anti-affinity terms because the generatedID is only known after the pool is reconfigured (including the affinity term). It would be a race condition since we need to include the generatedID before we actually know it.

Also, including the generatedID would make it hard (or even impossible) to set up specific anti-affinity terms because the generatedID is only known after the pool is reconfigured (including the affinity term). It would be a race condition since we need to include the generatedID before we actually know it.

Hi @berkitamas, if I understand correctly, this change set a new label without generatedID and add it to anti-affinity, so users don't need to manually set it.
		machineSetSplit := strings.Split(d.MachineName, "-")
		machineSetName := strings.Join(machineSetSplit[:len(machineSetSplit)-1], "-")
		vmBuilder = vmBuilder.Labels(map[string]string{poolNameLabelKey: machineSetName})

@ibrokethecloud, not sure whether the example in your description is wrong. There is generatedID on it.


[bug]: Running out of memory 2.3.1 RC3 - Free memory ON

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
6GB
What happened?
I was trying the weekly context on Discord and after +/-10 images I run out of memory. Changing Model empty the vRam, but doesn't solve the problem.
Also I have the Free vRam memory ON  in the settings.
Screenshots

Additional context
Using imgToimg and ddim
Contact Details
<EMAIL_ADDRESS>

@JPPhoto Could you have a look at this?

I did a workaround. Every time it hit the max, I change something inside the model manager and update the model, purging it.

I'm working on this problem with @JPPhoto . Are you using the stock stable-diffusion-1.5 model, or something else?

Standard sd1.5 diffuser

With higher resolution OOM appears sooner. Current VRAM usage after freeing keeps rising after every generation. It works fine (at least for me) when using safetensor model (VRAM usage after freeing is the same). Using Dreamshaper 3.31 safetensor model.

Here even safetensor model increase memory, but due to the smaller size, it increase less. I tested using a 2GB safetensor model and it increased by 500mb each step

I think that through the efforts of @jpphoto we have fixed the memory
consumption issues. Could you please try with the 2.3.1 release and let me
know if the problem has improved (and hopefully disappeared).
Lincoln
On Fri, Feb 24, 2023 at 6:42 AM Gabriel @.***> wrote:

Here even safetensor model increase memory, but due to the smaller size,
it increase less. I tested using a 2GB safetensor model and it increased by
500mb each step
—
Reply to this email directly, view it on GitHub
https://github.com/invoke-ai/InvokeAI/issues/2786#issuecomment-1443567953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA3EVMFS5HWPEX7HB6BFITWZCNDNANCNFSM6AAAAAAVGKSUKY
.
You are receiving this because you commented.Message ID:
@.***>

Solved

Most excellent! I am closing this issue in that case.


[vmware] Use the New Resource Model for VM operations

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

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

vmware

@nascarsayan Please resolve these CI issues

❌Azure CLI Extensions Breaking Change Test
❌connectedvmware
rule	cmd_name	rule_message	suggest_message
❌ 1007 - ParaRemove	connectedvmware vm delete	cmd connectedvmware vm delete removed parameter retain	please add back parameter retain for cmd connectedvmware vm delete
❌ 1002 - CmdRemove	connectedvmware vm list	cmd connectedvmware vm list removed	please confirm cmd connectedvmware vm list removed

@nascarsayan Please note some breaking changes had been introduced in this PR. Could we use the deprecation process instead of directly breaking customers without giving them any preparation time? For more details about breaking change process, please refer to this doc Implementing Breaking Changes in Azure CLI

❌Azure CLI Extensions Breaking Change Test
❌connectedvmware
rule	cmd_name	rule_message	suggest_message
❌ 1007 - ParaRemove	connectedvmware vm delete	cmd connectedvmware vm delete removed parameter retain	please add back parameter retain for cmd connectedvmware vm delete
❌ 1002 - CmdRemove	connectedvmware vm list	cmd connectedvmware vm list removed	please confirm cmd connectedvmware vm list removed

@nascarsayan Please note some breaking changes had been introduced in this PR. Could we use the deprecation process instead of directly breaking customers without giving them any preparation time? For more details about breaking change process, please refer to this doc Implementing Breaking Changes in Azure CLI

@zhoxing-ms , VMware extension is in preview and never released any stable version. Let us know. For the swagger APIs we reviewed with ARM breaking changes team and got the sign off as offering is in preview.

❌Azure CLI Extensions Breaking Change Test
❌connectedvmware
rule	cmd_name	rule_message	suggest_message
❌ 1007 - ParaRemove	connectedvmware vm delete	cmd connectedvmware vm delete removed parameter retain	please add back parameter retain for cmd connectedvmware vm delete
❌ 1002 - CmdRemove	connectedvmware vm list	cmd connectedvmware vm list removed	please confirm cmd connectedvmware vm list removed

@nascarsayan Please note some breaking changes had been introduced in this PR. Could we use the deprecation process instead of directly breaking customers without giving them any preparation time? For more details about breaking change process, please refer to this doc Implementing Breaking Changes in Azure CLI

@zhoxing-ms , current extension is in preview. We never released any stable version to the customers as the offering is currently preview and not GA'ed yet. We reviewed the same with ARM breaking changes team and got the sign off for the preview refresh. We are also communicating the changes to our preview customers on this change. Let us know if you have any suggestions.

❌Azure CLI Extensions Breaking Change Test
❌connectedvmware
rule	cmd_name	rule_message	suggest_message
❌ 1007 - ParaRemove	connectedvmware vm delete	cmd connectedvmware vm delete removed parameter retain	please add back parameter retain for cmd connectedvmware vm delete
❌ 1002 - CmdRemove	connectedvmware vm list	cmd connectedvmware vm list removed	please confirm cmd connectedvmware vm list removed

@nascarsayan Please note some breaking changes had been introduced in this PR. Could we use the deprecation process instead of directly breaking customers without giving them any preparation time? For more details about breaking change process, please refer to this doc Implementing Breaking Changes in Azure CLI

Have fixed the breaking changes, by deprecating the vm list command and retain parameter.


Purescript-uri dependency updated

Let me know when you want me to check etc, and I'll code review it.

Hi, this last pr is just a dependency update, so there's not much to check if last one was ok :)


Access token missing - Access token is require which is missing && user creation

Hi all!
As my reading, plugNmeet is going to be so helpful for my needs, I also have tried BBB and OpenMeetings, but plugNmeet seems to be much lightweight., thanks for the team...
I am sorry if I am asking basic questions, but I am really new with PlugNmeet. I have run the installation successfully according to https://www.plugnmeet.org/docs/installation article. When I try to connect via my custom URL it gives me the message "Access token missing - Access token is require which is missing". From /opt/plugNmeet/log/plugNmeet.log I am getting {"file":"/go/src/app/cmd/server/main.go:58","func":"main.startServer.func1","level":"info","msg":"exit requested, shutting down signal terminated","time":"2023-02-15T07:11:50Z"}
{"file":"/go/src/app/pkg/models/room_service.go:134","func":"github.com/mynaparrot/plugnmeet-server/pkg/models.(*RoomService).EndRoom","level":"error","msg":"twirp error internal: operation cannot be completed","time":"2023-02-15T08:04:03Z"}
I tried to google, but nothing helped me. Can someone help me from here?
The frontend test URL https://plugnmeet.mydomain.com/login.html is OK. But I don't see a way to invite anyone to that room.
Obviously the URL plugnmeet.mydomain.com is not the real URL I am using.
I also would like to know how to create user accounts.
Thanks in advance

Hello,
You will require to use API to connect with server. You can have a look from here: https://www.plugnmeet.org/docs/intro#sdks--tools
https://www.plugnmeet.org/docs/api/intro
Thanks

Hi @jibon57
Thank you for your response.
Can you guide on how to do that, I didn't get how to set it up
Thanks

Do you mean I will need to have another application to integrate to plugnmeet to run the meetings and creating the users? And I will need to do that via API?

Yes, at present plugNmeet doesn't have any separated management system. You can easily use WordPress, Joomla or Moodle for this. We've ready solution for those CMS. Of course you can use API to implement your own solution.

Many thanks, I am already working on that.


On TextField, inputClassName will overwrite all the input classes

Problem description
You have this usecase: <TextField inputClassName={classes.someRandomClass} />
Steps to reproduce

the inputClassName value will be attached to the inputProps object with the key className =>
https://github.com/callemall/material-ui/blob/v1-beta/src/TextField/TextField.js#L165

as seen here: https://github.com/callemall/material-ui/blob/v1-beta/src/Input/Input.js#L550,
the className key that is passed through inputProps will overwrite all the other classes that are being added to InputComponent

there result is that only the custom class is applied to the InputComponent, breaking the style for the input

a possible solution is to add the className from the input props to inputClassName object
https://github.com/callemall/material-ui/blob/v1-beta/src/Input/Input.js#L499

    const inputClassName = classNames(classes.input, inputProps.className, {
      [classes.inputDisabled]: disabled,
      [classes.inputSingleline]: !multiline,
      [classes.inputMultiline]: multiline,
      [classes.inputDense]: margin === 'dense',
    });

   // delete `className` key from `inputProps` or move `inputProps` as first property on `InputComponent`, to prevent static defined properties to be overwritten

Versions

Material-UI: beta 8

a possible solution is to add the className from the input props to inputClassName object

This sounds like a good idea. You are right, it's broken.


Tiny things.

I noticed some little things that the code can do better?
Maybe I'm wrong.
For the emit part I tried the code below to test:
const util = require('util');
const EventEmitter = require('events');

function MyEventEmitter() {
  EventEmitter.call(this);
}

util.inherits(MyEventEmitter, EventEmitter);

var a = new MyEventEmitter();

a.on('test', (obj) => {
  obj.test = 'something';
});

var obj = {test: ''};
a.emit('test', obj);

console.log(obj);// => {test: 'something'}

Sorry for the multiple commits, this is my first time making a pull request, I have no idea that I should run test on my local machine before I push. I'll do that next time.

Hi — thanks for filing a PR! :)
I must admit I don't quite understand the rationale behind most of these changes. Would you be able to annotate the differences in the diff? If the two are functionally equivalent, is this just a style change? And if so, what pattern did you use to decide how to make the changes?
You can fix your duplicate commits by squashing them into one! I/we generally prefer to accept PRs with clear commit messages, and no duplicate commits. David Walsh has a handy guide here.
Christopher

Hi,
Ah! Somehow I included the last PR after I squashed my duplicate commits. How do I fix this. :-P
I edited my commits so that maybe it make sense now? If not please let me know. Sorry my commits are poorly written, I must do better with my English.
Fan

Hi! So I see 3 changes going on here:

wait to set the main crawl interval untill the next tick. We currently wait for the next tick to make the first crawler request to enable the dev to change settings directly after having started the crawler.
on line 513, resources is used, but that array will always be empty, so we're really looking to use list instead.
in the fetchQueueItem method, the event listeners on clientRequest are registered inside of a process.nextTick call. According to the inline documentation, the function should return the crawler object for chaining, which it currently does not, since the return is being done inside of the process.nextTick call. So you removed the process.nextTick call there as well.

For future reference, you want to split these three changes into one commit each, and make sure to comment on each change separately so it's clear what the purpose of the change is :) Since you've already edited your commits once, you don't have to split this commit into three, though.


🐛 [firebase_remote_config] NoSuchMethodError Tried calling: *(1000)

Bug report
Describe the bug
NoSuchMethodError

NoSuchMethodError: The method '*' was called on null.
Receiver: null
Tried calling: *(1000)

Caused by line 52 here in remote_config.dart:

Steps to reproduce
I'm afraid I have no idea. This started to show up in the logs recently. Was fine before.
Expected behavior
This exception shouldn't be thrown.

Additional context
Results from: firebase_remote_config: ^0.6.0

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.0.1, on macOS 11.4 20F71 darwin-x64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.1.3)
[✓] Connected device (2 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
flutter
Dart SDK 2.12.0
Flutter SDK 2.0.1
rsc 21.16.0+1

dependencies:
- auto_size_text 2.1.0 [flutter]
- cached_network_image 2.5.1 [flutter flutter_cache_manager octo_image]
- cupertino_icons 1.0.3
- dartz 0.9.2
- dio 3.0.10 [http_parser path]
- email_validator 2.0.1
- equatable 2.0.2 [collection meta]
- file_picker 2.1.7 [flutter flutter_web_plugins flutter_plugin_android_lifecycle plugin_platform_interface]
- firebase_analytics 7.0.1 [meta flutter firebase_core firebase_analytics_web firebase_analytics_platform_interface]
- firebase_core 0.7.0 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- firebase_crashlytics 0.4.0+1 [flutter stack_trace firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface]
- firebase_messaging 8.0.0-dev.15 [meta flutter firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web]
- firebase_remote_config 0.6.0 [flutter firebase_core]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_bloc 7.0.0 [flutter bloc provider]
- flutter_keyboard_visibility 4.0.3 [meta flutter_keyboard_visibility_platform_interface flutter_keyboard_visibility_web flutter]
- flutter_linkify 5.0.2 [flutter linkify]
- flutter_local_notifications 4.0.1+2 [flutter flutter_local_notifications_platform_interface platform timezone]
- flutter_localizations 0.0.0 [flutter intl characters clock collection meta path typed_data vector_math]
- freezed_annotation 0.12.0 [collection json_annotation meta]
- get_it 7.1.3 [async collection]
- graphql_flutter 4.0.0-alpha.3 [graphql gql_exec flutter meta path_provider path connectivity hive]
- image_picker 0.6.7+22 [flutter flutter_plugin_android_lifecycle image_picker_platform_interface]
- implicitly_animated_reorderable_list 0.4.0-dev+1 [flutter async meta]
- intl 0.17.0 [clock path]
- json_annotation 3.1.0
- logger 1.0.0
- mime 0.9.7
- overlay_support 1.2.0-nullsafety.0 [flutter async]
- package_info 0.4.3 [flutter]
- path_drawing 0.4.1+1 [vector_math meta path_parsing flutter]
- permission_handler 5.0.1+1 [flutter meta permission_handler_platform_interface]
- photo_view 0.11.1 [flutter]
- pretty_dio_logger 1.1.1 [dio]
- sentry_flutter 4.0.6 [flutter flutter_web_plugins sentry package_info]
- shared_preferences 0.5.12 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- sprintf 6.0.0
- sqlite3_flutter_libs 0.5.0 [flutter]
- stream_chat_flutter 1.5.6 [flutter stream_chat_flutter_core photo_view rxdart scrollable_positioned_list jiffy flutter_svg flutter_portal cached_network_image shimmer flutter_markdown url_launcher emojis video_player chewie file_picker image_picker flutter_keyboard_visibility video_compress visibility_detector meta lottie substring_highlight flutter_slidable image_gallery_saver share_plus photo_manager ezanimation synchronized characters dio path_provider video_thumbnail]
- streaming_shared_preferences 1.0.2 [flutter shared_preferences meta]
- url_launcher 5.7.10 [flutter url_launcher_platform_interface url_launcher_web url_launcher_linux url_launcher_macos url_launcher_windows]
- webview_flutter 2.0.8 [flutter]

dev dependencies:
- bloc_test 8.0.0 [bloc meta mocktail test]
- build_runner 1.10.9 [args async build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style glob graphs http_multi_server io js logging meta mime path pedantic pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]
- freezed 0.12.7 [analyzer build build_config meta source_gen freezed_annotation]
- json_serializable 3.5.0 [analyzer build build_config json_annotation meta path source_gen]
- mockito 4.1.3 [analyzer build code_builder collection dart_style matcher meta source_gen test_api]

dependency overrides:
- intl 0.17.0 [clock path]

transitive dependencies:
- _fe_analyzer_shared 12.0.0 [meta]
- analyzer 0.40.6 [_fe_analyzer_shared args cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml]
- archive 2.0.13 [crypto args path]
- args 1.6.0
- async 2.5.0 [collection]
- bloc 7.0.0 [meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 1.5.2 [analyzer async convert crypto logging meta path glob]
- build_config 0.4.2 [checked_yaml json_annotation meta path pubspec_parse yaml]
- build_daemon 2.1.4 [built_collection built_value http_multi_server logging pedantic path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 1.4.4 [analyzer build crypto graphs logging meta path package_config pool pub_semver]
- build_runner_core 6.0.3 [async build build_config build_resolvers collection convert crypto glob graphs json_annotation logging meta path package_config pedantic pool timing watcher yaml]
- built_collection 4.3.2 [collection quiver]
- built_value 7.1.0 [built_collection collection fixnum quiver]
- characters 1.1.0
- charcode 1.2.0
- checked_yaml 1.0.2 [json_annotation source_span yaml]
- chewie 1.0.0 [cupertino_icons flutter video_player wakelock]
- cli_util 0.2.0 [path]
- clock 1.1.0
- code_builder 3.4.1 [built_collection built_value collection matcher meta]
- collection 1.15.0
- connectivity 0.4.9+5 [flutter meta connectivity_platform_interface connectivity_macos connectivity_for_web]
- connectivity_for_web 0.3.1+2 [connectivity_platform_interface flutter_web_plugins flutter]
- connectivity_macos 0.1.0+5 [flutter]
- connectivity_platform_interface 1.0.6 [flutter meta plugin_platform_interface]
- convert 2.1.1 [charcode typed_data]
- coverage 0.13.11 [args logging package_config path source_maps stack_trace vm_service]
- crypto 2.1.4 [collection convert typed_data]
- dart_style 1.3.10 [analyzer args path source_span]
- emojis 0.9.3
- ezanimation 0.4.1 [flutter]
- fake_async 1.2.0 [clock collection]
- ffi 0.1.3
- file 5.2.1 [intl meta path]
- firebase 7.3.3 [http http_parser js]
- firebase_analytics_platform_interface 1.0.3 [flutter meta]
- firebase_analytics_web 0.1.1 [flutter flutter_web_plugins firebase firebase_analytics_platform_interface meta]
- firebase_core_platform_interface 3.0.1 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.1+3 [firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_crashlytics_platform_interface 1.1.6 [flutter meta collection firebase_core plugin_platform_interface]
- firebase_messaging_platform_interface 1.0.0-dev.10 [flutter meta firebase_core plugin_platform_interface]
- firebase_messaging_web 0.1.0-dev.6 [flutter flutter_web_plugins meta firebase_core firebase_core_web firebase_messaging_platform_interface js service_worker]
- fixnum 0.10.11
- flutter_blurhash 0.5.0 [flutter meta]
- flutter_cache_manager 2.1.2 [flutter path_provider uuid http path sqflite pedantic clock file rxdart image]
- flutter_keyboard_visibility_platform_interface 1.0.1 [flutter meta plugin_platform_interface]
- flutter_keyboard_visibility_web 1.0.1 [flutter_keyboard_visibility_platform_interface flutter_web_plugins flutter]
- flutter_local_notifications_platform_interface 2.0.0+1 [flutter plugin_platform_interface]
- flutter_markdown 0.5.2 [flutter markdown meta path]
- flutter_plugin_android_lifecycle 1.0.11 [flutter]
- flutter_portal 0.3.0 [flutter]
- flutter_slidable 0.5.7 [flutter]
- flutter_svg 0.19.3 [flutter meta path_drawing vector_math xml]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- glob 1.2.0 [async collection node_io path pedantic string_scanner]
- gql 0.12.5-alpha+1617643196353 [source_span meta collection]
- gql_dedupe_link 1.0.10 [meta gql_exec gql_link async]
- gql_error_link 0.1.1-alpha+1601131172858 [async gql_exec gql_link meta]
- gql_exec 0.2.6-alpha+1617643196368 [gql meta collection]
- gql_http_link 0.2.11-alpha+1599239999284 [meta gql gql_exec gql_link http http_parser]
- gql_link 0.3.2-alpha+1617643196377 [meta gql gql_exec]
- gql_transform_link 0.1.5 [gql_exec gql_link]
- gql_websocket_link 0.1.8-alpha+1612972259626 [gql gql_exec gql_link meta rxdart uuid_enhanced web_socket_channel]
- graphql 4.0.0-alpha.3 [meta path gql gql_exec gql_link gql_http_link gql_websocket_link gql_transform_link gql_error_link gql_dedupe_link hive normalize http collection]
- graphs 0.2.0
- hive 1.4.4+1 [meta crypto]
- http 0.12.2 [http_parser path pedantic]
- http_multi_server 2.2.0 [async]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- image 2.1.19 [archive xml meta]
- image_gallery_saver 1.6.9 [flutter]
- image_picker_platform_interface 1.1.6 [flutter meta http plugin_platform_interface]
- io 0.3.4 [charcode meta path string_scanner]
- jiffy 3.0.1 [intl]
- js 0.6.3
- linkify 4.0.0
- logging 0.11.4
- lottie 0.7.0+1 [archive characters charcode collection flutter logging meta path vector_math]
- markdown 3.0.0 [args charcode meta]
- matcher 0.12.10 [stack_trace]
- meta 1.3.0
- mocktail 0.1.2 [collection matcher test]
- nested 1.0.0 [flutter]
- node_interop 1.1.1 [js]
- node_io 1.1.1 [node_interop path]
- node_preamble 1.4.12
- normalize 0.2.4 [gql meta]
- octo_image 0.3.0 [flutter flutter_blurhash]
- package_config 1.9.3 [path charcode]
- path 1.8.0
- path_parsing 0.1.4 [vector_math meta]
- path_provider 1.6.28 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+4 [flutter]
- path_provider_platform_interface 1.0.3 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+1 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.11.0
- permission_handler_platform_interface 2.0.1 [flutter meta plugin_platform_interface]
- petitparser 3.1.0 [meta]
- photo_manager 1.2.5 [flutter]
- platform 3.0.0
- plugin_platform_interface 1.0.3 [meta]
- pool 1.5.0 [async stack_trace]
- process 3.0.13 [file intl meta path platform]
- provider 5.0.0 [collection flutter nested]
- pub_semver 1.4.4 [collection]
- pubspec_parse 0.1.5 [checked_yaml json_annotation pub_semver yaml]
- quiver 2.1.3 [matcher meta]
- rxdart 0.25.0
- scrollable_positioned_list 0.1.10 [flutter meta]
- sentry 4.0.6 [http meta stack_trace uuid]
- service_worker 0.2.4 [js]
- share_plus 1.2.0 [meta mime flutter share_plus_platform_interface share_plus_linux share_plus_windows share_plus_web]
- share_plus_linux 1.1.0 [share_plus_platform_interface file flutter meta url_launcher]
- share_plus_platform_interface 1.2.0 [flutter meta mime plugin_platform_interface]
- share_plus_web 0.1.0 [share_plus_platform_interface url_launcher flutter flutter_web_plugins meta]
- share_plus_windows 0.1.0 [share_plus_platform_interface flutter meta url_launcher]
- shared_preferences_linux 0.0.2+2 [file flutter meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 0.0.1+10 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 1.0.4 [meta flutter]
- shared_preferences_web 0.1.2+7 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 0.0.1+1 [shared_preferences_platform_interface flutter ffi file meta path path_provider_platform_interface path_provider_windows]
- shelf 0.7.9 [async collection http_parser path stack_trace stream_channel]
- shelf_packages_handler 2.0.0 [path shelf shelf_static]
- shelf_static 0.2.8 [convert http_parser mime path shelf]
- shelf_web_socket 0.2.3 [shelf web_socket_channel stream_channel]
- shimmer 1.1.2 [flutter]
- sky_engine 0.0.99
- source_gen 0.9.7+1 [analyzer async build dart_style glob meta path pedantic source_span]
- source_map_stack_trace 2.1.0 [path stack_trace source_maps]
- source_maps 0.10.10 [source_span]
- source_span 1.8.0 [charcode collection path term_glyph]
- sqflite 1.3.2+4 [flutter sqflite_common path]
- sqflite_common 1.0.3+3 [synchronized path meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- stream_chat 1.5.3 [async collection dio freezed_annotation http_parser json_annotation logging meta mime rxdart uuid web_socket_channel]
- stream_chat_flutter_core 1.5.3 [flutter meta rxdart stream_chat]
- stream_transform 1.2.0
- string_scanner 1.1.0 [charcode source_span]
- substring_highlight 0.1.2 [flutter meta]
- synchronized 2.2.0+2
- term_glyph 1.2.0
- test 1.16.5 [analyzer async boolean_selector coverage http_multi_server io js node_preamble package_config path pedantic pool shelf shelf_packages_handler shelf_static shelf_web_socket source_span stack_trace stream_channel typed_data web_socket_channel webkit_inspection_protocol yaml test_api test_core]
- test_api 0.2.19 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- test_core 0.3.15 [analyzer async args boolean_selector collection coverage glob io meta package_config path pedantic pool source_map_stack_trace source_maps source_span stack_trace stream_channel vm_service yaml matcher test_api]
- timezone 0.6.1 [path]
- timing 0.1.1+2 [json_annotation]
- typed_data 1.3.0 [collection]
- url_launcher_linux 0.0.1+1 [flutter]
- url_launcher_macos 0.0.1+8 [flutter]
- url_launcher_platform_interface 1.0.9 [flutter meta plugin_platform_interface]
- url_launcher_web 0.1.5+1 [url_launcher_platform_interface flutter flutter_web_plugins meta]
- url_launcher_windows 0.0.1+1 [flutter]
- uuid 2.2.2 [crypto convert]
- uuid_enhanced 3.0.2 [crypto convert collection]
- vector_math 2.1.0
- video_compress 2.1.1 [flutter]
- video_player 2.1.10 [flutter flutter_test meta video_player_platform_interface video_player_web]
- video_player_platform_interface 4.1.0 [flutter meta flutter_test]
- video_player_web 2.0.1 [flutter flutter_web_plugins meta video_player_platform_interface]
- video_thumbnail 0.2.5+1 [flutter]
- visibility_detector 0.1.5 [flutter]
- vm_service 4.2.0 [meta]
- wakelock 0.4.0 [flutter meta wakelock_macos wakelock_platform_interface wakelock_web]
- wakelock_macos 0.1.0+1 [flutter flutter_web_plugins wakelock_platform_interface]
- wakelock_platform_interface 0.2.1+1 [flutter meta]
- wakelock_web 0.2.0+1 [flutter flutter_web_plugins js wakelock_platform_interface]
- watcher 0.9.7+15 [async path pedantic]
- web_socket_channel 1.2.0 [async crypto stream_channel]
- webkit_inspection_protocol 0.7.3 [logging]
- win32 1.7.3 [ffi]
- xdg_directories 0.1.2 [meta path process]
- xml 4.5.1 [collection convert meta petitparser]
- yaml 2.2.1 [charcode collection string_scanner source_span]

Hi @rohan20
Please flutter upgrade to the latest stable flutter version and use the latest firebase_remote_config: ^0.11.0.
You can look at the example https://pub.dev/packages/firebase_remote_config/example to see how the methods are now meant to be used.
If the issue persists please provide a minimal complete reproducible code sample.
Thank you

Hi @markusaksli-nc, we have some compatibility issues with higher Flutter versions so it isn't so easy to upgrade. Should I assume from your reply that it wouldn't be possible to fix this for ^0.6.0? 😢

Well the plugin code was updated to null-safety, the API has changed a bit, and the underlying SDK versions have been updated
https://github.com/FirebaseExtended/flutterfire/blob/d35732adc7c9d740271a4ff45e2e3c0d127acf97/packages/firebase_remote_config/firebase_remote_config/lib/src/remote_config.dart#L123-L132
So I'm not sure what the behavior would be on this version. If you mean would we release a fix for a legacy version then sadly the answer is no.
Even if this turns out to be a valid issue on the latest version the only way you would get the fix is by upgrading to a future version anyway.
Flutter 2 and the updated FlutterFire plugins have been stable for a while now so out of curiosity what compatibility issues are you having?

Alright, thank you for the update 👍

Flutter 2 and the updated FlutterFire plugins have been stable for a while now so out of curiosity what compatibility issues are you having?

The last time we upgraded some dependencies, we ran into cyclic dependency issues so we either need to upgrade a bunch of them at once or stay as-is. I'll try that once more with the remote config version that you recommend, hopefully, it'll work.
Thank you once again. This issue can be closed 🚀


when I reset route, all old routes appear my App

const resetAction = NavigationActions.reset({
    index: 0,
    actions: [
        NavigationActions.navigate({routeName: 'Index'})
    ],
    key:null
});
this.state.navigation.dispatch(resetAction);

before I reset, I through some pages, and when I reset toutes, these pages appear in my App until navigate Index
| react-navigation | 1.0.0-beta.11
| react-native     | 0.44

I see some developer take this problem, but looks like have no effect method to solve it

To handle your app's navigation state in redux, you can overwrite reset action.

Is this a bug report or a question?

@spencercarli We are experiencing the same.
We have a flow of multiple screens. On the final screen in the flow we have an action that dispatches a reset. The app navigates to the given route but all other screens in that flow appear in the transition:
NavigationActions.reset({
  index: 0,
  actions: [NavigationActions.navigate({routeName: 'Home'})],
})

closing since I believe this was fixed by https://github.com/react-community/react-navigation/commit/e6505054bd765f0771aec8c328238b0b4d8ddab9, but feel free to reopen if that is not the case.


Support for pngs.

From my duplicate issue:
The mapper expects a provinces.bmp file in the map directory, but new PDX games use provinces.png. This requires converting to BMP, which may lead to a map-staring guy working on an outdated map if he forgets to repeat the process after a game update

This can be closed :)


Mining

Hello, we cannot get the wallet synced up with the node. We see it at block height 17 but the wallet get stuck at height 0.
It seems like the source code here is not fully the same used by the node seeds.kyc.koyotecoin.org
Could you please upload the correct source code thank you

Hello @le2Ks
I see that you are already running some nodes, I am happy for your willingness to help the network, but I ask you to wait for the final stable code ( release ), because I am still doing some version/connection tests with peers and mining, such as you are already running some nodes, they can cause conflicts, so I ask you to wait a little for the official release that will be available soon. 👍

Hello @le2Ks
not yet at the moment.


Allow bosh addons on specific networks

Hi, the addon structure for bosh allows you to specify deployments to which an addon should be installed like this:
include:
  deployments: [dep1, dep2]
  jobs:
  - name: redis
    release: redis-release
  stemcell:
    - os: ubuntu-trusty

On our vsphere-staging and concourse environment we are currently doing some stuff with the networking release (set gateway to public, propagate some routes, close port 22 on <IP_ADDRESS>) which is currently done in the specific deployment manifests. We would be very interested, if the include rules would also allow you to define a inclusion filter, that will install this job on all deployments that are in a specific network like this:
include:
  networks: [elastic]

This is being implemented via PR #1653

PR was merged. will be in v262.


adbc_driver_postgres flatten multi dimensional array in Postgres

What happened?
When querying a multi dimensional array in Postgres the returned result "loose" the dimensions, e. g. a 2 dimension array is returned as 1 dimension array, the query:
select array[[1, 1], [2, 3]]

It should return:
[([[1, 1],[2, 3]],)]

It returns:
[([1, 1, 2, 3],)]

Stack Trace
No response
How can we reproduce the bug?
import adbc_driver_postgresql.dbapi

conn_string = 'postgresql://postgres:XXXX@localhost:5432/postgres'
conn = adbc_driver_postgresql.dbapi.connect(conn_string)

cursor = conn.cursor()
query = 'select array[[1, 1], [2, 3]]'
cursor.execute(query)
cursor.fetchall()

Environment/Setup
macOS
PostgreSQL 15
adbc-driver-postgresql 1.1.0

Thanks for the response, in a ideal world this query:
select array[[1, 1], [2, 3]]
union all
select array[[1, 1]]

would return
[([[1, 1], [2, 3]],), ([[1, 1], [,]],)]

but an warning (or error that can be silenced with a flag) at least can help to debug why the data is different.


Support --coverage for npm start and npm run build (#3257)

UNVERIFIED PROOF OF CONCEPT PATCH
Passing --coverage makes babel-preset-react-app add
babel-plugin-istanbul to support code coverage reports for end-to-end
tests.
I got no feedback in the ticket I filed if this feature was interesting to create-react-app maintainers so I'm providing a sketch of how the code could look in hope of getting some feedback.

Thanks for sending the PR! Please see my reply in https://github.com/facebookincubator/create-react-app/issues/3257#issuecomment-340217459.
We don’t intend to immediately merge this for those reasons so I’ll close. But if more people ask for it in the issue, I’m happy to eventually revisit. Cheers!


Add utility to easily compare 2 soft_coverage_test reports

--reference option of valid.soft_coverage_test can be used to compare with a previous test run report (generated using --gen-reference  option)


Sema: Propagate noescape bit into closure passed to withoutActuallyEscaping [4.2 6/11]

Description: Fixes a Swift 4.2 regression where closure literals passed to withoutActuallyEscaping() were treated as escaping if they contained multiple statements, causing Sema to diagnose member accesses not qualified with self.

Scope of the issue: Source compatibility break for users of withoutActuallyEscaping, reported a couple of times.

Origination: Introduced at some point in the 4.2 cycle after type checker cleanups. The visitor logic around closures is a bit flaky, because multi-statement closures are treated different from single-expression; the latter are effectively part of the expression that contains them, but the former must be visited separately.

Risk: Very low.

Radar: Fixes rdar://problem/40853562.

Tested: Added test case from radar.

Reviewed by: @jckarter (on master)

@swift-ci Please test


Reschedule to import new images on Tuesday in Prod

We discussed on retro to move the stable branches on Monday, instead of
Friday, and release a new version of the service on Tuesday. This way if
something goes wrong we can take care of it during the week, without
ruining someones weekend.
Signed-off-by: Hunor Csomortáni<EMAIL_ADDRESS>

This was already applied to prod.

regate


gpuIndices

Describe the issue:
Environment:

NNI version:2.2
Training service (local|remote|pai|aml|etc):local
Client OS:Windows 10
Python version:3.8.5
PyTorch/TensorFlow version:1.7.1
Is conda/virtualenv/venv used?:conda version 10.2
Is running in Docker?:False

Configuration:

Experiment config (remember to remove secrets!):config.yml
authorName: default
experimentName: test_nni
trialConcurrency: 1
maxExecDuration: 1d
maxTrialNum: 30
trainingServicePlatform: local
searchSpacePath: search_space.json
useAnnotation: false
tuner:
builtinTunerName: TPE
classArgs:
optimize_mode: minimize
trial:
command: python train_nni.py
codeDir: .
gpuNum: 1
localConfig:
useActiveGpu: true
gpuIndices: 3

Search space:
{   "start_lr": {"_type": "choice", "_value": [0.0001, 0.001, 0.01, 0.1]},
"batch_size": {"_type": "choice", "_value": [8, 12, 16, 32, 64, 128]}
}

error:
ERROR: Config V1 validation failed: SchemaError('gpuIndex format error!')

error:
ERROR: Config V1 validation failed: SchemaError('gpuIndex format error!')

https://github.com/microsoft/nni/pull/3540
Adding error msg pr owner in to suggest.

error:
ERROR: Config V1 validation failed: SchemaError('gpuIndex format error!')

#3540
Adding error msg pr owner in to suggest.
Thank you, but I don't quite understand your suggestion. Could you tell me more about it.

error:
ERROR: Config V1 validation failed: SchemaError('gpuIndex format error!')

#3540
Adding error msg pr owner in to suggest.

Thank you, but I don't quite understand your suggestion. Could you tell me more about it

This bug will be fixed in next release.
For now you can walkaround by replacing 3 (int) with '3' (str).

This bug will be fixed in next release.
For now you can walkaround by replacing 3 (int) with '3' (str).

nice， thanks!

Expected to be fixed in v2.3 release next week.

hello @creater-zq, I'm closing this issue as it had been fixed, free feel to reopen it if you still think it an active issue.


net: lib: sockets: Unable to poll file descriptor (number >= 32)

Describe the bug
zsock_select() cannot poll file descriptors if all the file descriptors in readfds or writefds are all greater than or equal to 32.
To Reproduce

Create 32 threads to accept 32 TCP connections, each thread handles one TCP connection. Some of the file descriptor returned by zsock_accept() are >= 32.
In each thread, use zsock_select() to poll read event for the file descriptor. If the file descriptor >= 32, it never becomes ready for reading even there is data received.

Solution
In zsock_select(), when there is no any bit set in the first 32 bits of readfds, we should still add 32 to fd_no before iterating to next bitset, so I add fd_no += 32;  as following (code in red color):

Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK
Commit: abb94b1198ec6ce618f61dff367275f96cad4eca

@chyu313 could you send a PR for this?


Fixing Failed Tests in test_user_schemas.py by Adding Missing Fixtures

The existing tests in test_user_schemas.py were failing due to missing or incomplete data setup for the test cases. Specifically, the following issues were observed:
Missing Test Fixtures: Critical test data (user_base_data, user_create_data, etc.) was not defined, leading to errors during test execution.
Inconsistent UUID Handling: The UserResponse model required a valid UUID, which was not provided in the test setup.
Validation Errors: Tests for email and URL validations were not properly passing due to incomplete or invalid test data.

To address these issues:
Created and integrated missing test fixtures for consistent test data across all tests.
Ensured the user_response_data fixture included a valid UUID.
Validated data with realistic and compliant formats for email, URLs, and UUIDs.
Rerun the tests, verifying that all passed successfully after the fixes.

Description
The existing tests in test_user_schemas.py were failing due to missing or incomplete data setup for the test cases. Specifically, the following issues were observed:
Missing Test Fixtures: Critical test data (user_base_data, user_create_data, etc.) was not defined, leading to errors during test execution.
Inconsistent UUID Handling: The UserResponse model required a valid UUID, which was not provided in the test setup.
Validation Errors: Tests for email and URL validations were not properly passing due to incomplete or invalid test data.
To address these issues:
Created and integrated missing test fixtures for consistent test data across all tests.
Ensured the user_response_data fixture included a valid UUID.
Validated data with realistic and compliant formats for email, URLs, and UUIDs.
Rerun the tests, verifying that all passed successfully after the fixes.
Proposed Changes
Added Test Fixtures:
user_base_data: Basic data for user objects (nickname, email, profile picture URL).
user_create_data: Data for creating new user records, including passwords.
user_update_data: Data for updating user profiles.
user_response_data: Data for user responses, ensuring a valid UUID and other fields.
login_request_data: Data for simulating login requests.
Validated Fixture Data:
Added a valid UUID to user_response_data.
Ensured proper email and URL formats for other fixtures.
Updated Tests:
Parametrized tests for nickname validation with valid and invalid examples.
Enhanced URL validation tests with both valid and invalid scenarios.
Verified all models against the updated fixtures to ensure compatibility and reliability.
Steps to Reproduce the Original Issue
Run pytest tests/test_user_schemas.py without the added fixtures.
Observe errors related to missing or invalid test data.

https://github.com/chandrA957355/homework-10/tree/user_data_missing_fixtures


14: Add meta_analytic_samples wrapper

closes #14

I squashed and merged. If in general you prefer that responsibility to stay with PR creator, pls say!


Issue while bulding KMaxStackDepth

Kindly assist me where is the actual issue?
Using bazel 6.3.2
`Compiling tcmalloc/transfer_cache.cc failed: (Exit 1): gcc failed: error executing command (from target @com_google_tcmalloc//tcmalloc:common)  /bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' ... (remaining 40 arguments skipped)
Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
In file included from external/com_google_tcmalloc/tcmalloc/internal/lifetime_tracker.h:21,
                from external/com_google_tcmalloc/tcmalloc/huge_page_filler.h:33,
                from external/com_google_tcmalloc/tcmalloc/huge_region.h:24,
                from external/com_google_tcmalloc/tcmalloc/huge_page_aware_allocator.h:26,
                from external/com_google_tcmalloc/tcmalloc/page_allocator.h:25,
                from external/com_google_tcmalloc/tcmalloc/static_vars.h:39,
                from external/com_google_tcmalloc/tcmalloc/transfer_cache.cc:35:
external/com_google_tcmalloc/tcmalloc/internal/lifetime_predictions.h:179:20: error: declaration of 'const int tcmalloc::tcmalloc_internal::LifetimeDatabase::kMaxStackDepth' changes meaning of 'kMaxStackDepth' [-Wchanges-meaning]
179 |   static const int kMaxStackDepth = 64;
     |                    ^~~~~~~~~~~~~~
external/com_google_tcmalloc/tcmalloc/internal/lifetime_predictions.h:107:18: note: used here to mean 'constexpr const int tcmalloc::tcmalloc_internal::kMaxStackDepth'
107 |     void* stack_[kMaxStackDepth];
     |                  ^~~~~~~~~~~~~~
In file included from external/com_google_tcmalloc/tcmalloc/common.h:37,
                from external/com_google_tcmalloc/tcmalloc/central_freelist.h:28,
                from external/com_google_tcmalloc/tcmalloc/transfer_cache.h:35,
                from external/com_google_tcmalloc/tcmalloc/transfer_cache.cc:15:
external/com_google_tcmalloc/tcmalloc/internal/logging.h:49:22: note: declared here
  49 | static constexpr int kMaxStackDepth = 64;
     |                      ^~~~~~~~~~~~~~
Target //fleetbench/tcmalloc:empirical_driver failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 17.559s, Critical Path: 4.14s
INFO: 116 processes: 22 internal, 94 processwrapper-sandbox.
FAILED: Build did NOT complete successfully
ERROR: Build failed. Not running target
`

-Wchanges-meaning is a new warning with gcc 13. Currently we have only verified Fleetbench with gcc 12.
However, the code that causes the warning has been removed in the upstream version of tcmalloc.  I've upgraded our tcmalloc dependency to the latest and built with gcc 13 - I think it should be working for you now if you pull latest.
Thanks for reporting it!

This is the new error. DEBUG: /private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/rules_python/python/pip.bzl:47:10: pip_install is deprecated. Please switch to pip_parse. pip_install will be removed in a future release.
INFO: Repository libpfm instantiated at:
Desktop/bechmark_fleetbech/fleetbench/WORKSPACE:110:13: in
Repository rule http_archive defined at:
/private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/bazel_tools/tools/build_defs/repo/http.bzl:372:31: in
WARNING: Download from https://sourceforge.net/projects/perfmon2/files/libpfm4/libpfm-4.11.0.tar.gz/download failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException Checksum was 251a85b3bac687974f360d3796048c20ded3bf0bd69e0d1cfd1db23d013f89ed but wanted 5da5f8872bde14b3634c9688d980f68bda28b510268723cc12973eedbab9fecc
ERROR: An error occurred during the fetch of repository 'libpfm':
Traceback (most recent call last):
File "/private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/bazel_tools/tools/build_defs/repo/http.bzl", line 132, column 45, in _http_archive_impl
download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://sourceforge.net/projects/perfmon2/files/libpfm4/libpfm-4.11.0.tar.gz/download] to /private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/libpfm/temp3458073497591957110/download.tar.gz: Checksum was 251a85b3bac687974f360d3796048c20ded3bf0bd69e0d1cfd1db23d013f89ed but wanted 5da5f8872bde14b3634c9688d980f68bda28b510268723cc12973eedbab9fecc
ERROR: /Desktop/bechmark_fleetbech/fleetbench/WORKSPACE:110:13: fetching http_archive rule //external:libpfm: Traceback (most recent call last):
File "/private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/bazel_tools/tools/build_defs/repo/http.bzl", line 132, column 45, in _http_archive_impl
download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://sourceforge.net/projects/perfmon2/files/libpfm4/libpfm-4.11.0.tar.gz/download] to /private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/libpfm/temp3458073497591957110/download.tar.gz: Checksum was 251a85b3bac687974f360d3796048c20ded3bf0bd69e0d1cfd1db23d013f89ed but wanted 5da5f8872bde14b3634c9688d980f68bda28b510268723cc12973eedbab9fecc
ERROR: /private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/com_google_benchmark/BUILD.bazel:35:11: @com_google_benchmark//:benchmark depends on @libpfm//:libpfm in repository @libpfm which failed to fetch. no such package '@libpfm//': java.io.IOException: Error downloading [https://sourceforge.net/projects/perfmon2/files/libpfm4/libpfm-4.11.0.tar.gz/download] to /private/var/tmp/_bazel_athbha01/d016f079ed8063c5398a942b18f70a7b/external/libpfm/temp3458073497591957110/download.tar.gz: Checksum was 251a85b3bac687974f360d3796048c20ded3bf0bd69e0d1cfd1db23d013f89ed but wanted 5da5f8872bde14b3634c9688d980f68bda28b510268723cc12973eedbab9fecc
ERROR: Analysis of target '//fleetbench/tcmalloc:empirical_driver' failed; build aborted:
INFO: Elapsed time: 335.156s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (75 packages loaded, 703 targets configured)
ERROR: Build failed. Not running target

It looks like Sourceforge is having trouble with downloads.
https://downdetector.com/status/sourceforge/
It should resolve itself soon, hopefully.
A workaround for now is to disable libpfm when you build:
bazel build --define=pfm=0 fleetbench/tcmalloc:empirical_driver


Importing dashboard goes wrong?

Hey, every time i try to import the dashboard in grafana, i get this following error:

Do you guys know what this is and how i can fix it?

what is your grafana version?

Hello. I am using bungeecord-prometheus-exporter for the first time using Grafana v9.3.6 (978237e7cb). I get the same error. Is there any further information I can provide you with?

thanks, @StealWonders, it seems like a common error.
have you already added a prometheus data source in grafana?

updated the dashboard, please try again


Trying to compile in CentOS 7 (want to know what compiler is used at Facebook?)

Issue Description
I am trying to compile and install OpenR in a CentOS 7 docker image.
According to the OpenR github-page ,
this should be supported, but I am running into issues compiling fbthrift.
These issues seem to be caused by a bug in the Red Hat Developer Toolset 7 that I use,
so I was wondering what toolset was used by Facebook when you got it to compile on CentOS 7.
I also tried devtoolset-6, but that doesn't offer proper C++14 support and fails compilation in different ways.
Environment

tag or commit hash on which this occured

commit hash d331eb0e8a8516b3f171ee2f938e8958d51b7c99 , but with folly version bumped to include bugfix for #820 (otherwise it would not compile).

/tmp/openr-centos/build # git diff
diff --git a/build/deps/github_hashes/facebook/folly-rev.txt b/build/deps/github_hashes/facebook/folly-rev.txt
index f535f21..0491c68 100644
--- a/build/deps/github_hashes/facebook/folly-rev.txt
+++ b/build/deps/github_hashes/facebook/folly-rev.txt
@@ -1 +1 @@
-Subproject commit 7153cc6bf1b0f71b80ce4412fbc83163d10c9a2f
+Subproject commit e752dcd00ed4b8ff4d633e7996db027321ac7708

OS version: <e.g. ubuntu-16.04>
CentOS 7,
with devtoolset-7 installed.

Minimal test code / Steps to reproduce the issue
Run attached script,
which is a version of build_openr.sh updated for CentOS 7.
What's the actual result?

[ 31%] Built target compiler_generators
[ 32%] Linking CXX executable ../../bin/thrift1
[ 32%] Building CXX object thrift/lib/cpp/CMakeFiles/security.dir/util/kerberos/Krb5OlderVersionStubs.cpp.o
/jeroend/tmp/openr-centos/build/deps/fbthrift/thrift/lib/cpp/concurrency/TimerManager.cpp: In destructor 'virtual apache::thrift::concurrency::TimerManager::~TimerManager()':
/jeroend/tmp/openr-centos/build/deps/fbthrift/thrift/lib/cpp/concurrency/TimerManager.cpp:158:7: warning: throw will always call terminate() [-Wterminate]
       throw;
       ^~~~~
/jeroend/tmp/openr-centos/build/deps/fbthrift/thrift/lib/cpp/concurrency/TimerManager.cpp:158:7: note: in C++11 destructors default to noexcept
/opt/rh/devtoolset-7/root/usr/lib/gcc/x86_64-redhat-linux/7/libstdc++_nonshared.a(cow-stdexcept.o): In function `std::__sso_string::__sso_string(std::string const&)':
(.text._ZNSt12__sso_stringC2ERKSs+0x8): undefined reference to `std::__sso_string::__sso_string(char const*, unsigned long)'
/opt/rh/devtoolset-7/root/usr/lib/gcc/x86_64-redhat-linux/7/libstdc++_nonshared.a(cow-stdexcept.o): In function `std::_V2::error_category::_M_message(int) const':
(.text._ZNKSt3_V214error_category10_M_messageEi+0x30): undefined reference to `std::__sso_string::__sso_string(char const*, unsigned long)'
/opt/rh/devtoolset-7/root/usr/libexec/gcc/x86_64-redhat-linux/7/ld: ../../bin/thrift1: hidden symbol `_ZNSt12__sso_stringD1Ev' isn't defined
/opt/rh/devtoolset-7/root/usr/libexec/gcc/x86_64-redhat-linux/7/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status
make[2]: *** [thrift/compiler/CMakeFiles/thrift1.dir/build.make:185: bin/thrift1] Error 1
make[1]: *** [CMakeFiles/Makefile2:115: thrift/compiler/CMakeFiles/thrift1.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 33%] Building CXX object thrift/lib/cpp/CMakeFiles/security.dir/util/kerberos/Krb5CredentialsCacheManager.cpp.o
[ 33%] Building CXX object thrift/lib/cpp/CMakeFiles/security.dir/util/kerberos/Krb5CCacheStore.cpp.o
[ 34%] Building CXX object thrift/lib/cpp/CMakeFiles/security.dir/util/kerberos/Krb5Tgts.cpp.o
[ 35%] Building CXX object thrift/lib/cpp/CMakeFiles/security.dir/util/kerberos/FBKrb5GetCreds.cpp.o
[ 36%] Linking CXX shared library ../../../lib/libthrift-core.so
[ 36%] Built target thrift-core
[ 36%] Linking CXX shared library ../../../lib/libserver.so
[ 36%] Built target server
[ 37%] Linking CXX shared library ../../../lib/libconcurrency.so
[ 37%] Built target concurrency
[ 37%] Linking CXX shared library ../../../lib/libsecurity.so
[ 37%] Built target security
make: *** [Makefile:128: all] Error 2

What's the expected result?
A script that I can run that simply works and installs OpenR on CentOS 7 would be best :-),
but any help would be appreciated.

Quick googling releavs that this is bug in devtoolset-7 just like you mentioned.
https://bugzilla.redhat.com/show_bug.cgi?id=1519073
You will need devtoolset-7.1 to get pass this error. You might pull latest devtoolset (from trunk if possible) and compile with it. Here we use CentOS but don't rely on devtoolsets for compilation. Our internal build systems leverages specific gcc-5+ version which supports c++14 very well.

Thanks for the answer.
After you mentioned you use gcc-5 I took a closer look at devtoolset-6 (which uses gcc-6),
and I realized the failure was not due to C++14 but due to an interop issue with folly.
So I opened an issue there,
but for now I managed to work around it, so I can now compile on CentOS 7.
@anyone else reading this looking for the same answer: The solution turned out to be to simply use devtoolset-6, and use the patch here for the folly issue

Thanks @jeroen-dhollander for posting patch for future references. Will go ahead and close the issue then :)


Fixed error anim speed.

Thanks @AlbertoCasasOrtiz it works well. the text of the spinner box though cuts into the slider knob at 90 or above

Thanks @aymanhab . Ithink we should locate it in another place. In the end, controlling the animation is an important feature and people are used to having it available all the time in opensim-gui and opencap. Maybe at the bottom bar, or floating like the camera controls?

Agree, not high priority at the moment.
On Wed, Nov 22, 2023, 12:30 PM Alberto @.***> wrote:

Thanks @aymanhab https://github.com/aymanhab . Ithink we should locate
it in another place. In the end, controlling the animation is an important
feature and people are used to having it available all the time in
opensim-gui and opencap. Maybe at the bottom bar, or floating like the
camera controls?
—
Reply to this email directly, view it on GitHub
https://github.com/opensim-org/opensim-viewer/pull/129#issuecomment-1823466622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6JY4H5PGSFBBL3OA2VPGDYFZOFTAVCNFSM6AAAAAA7VKYGZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRTGQ3DMNRSGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


gpt4all-chat: init at 2.6.2

Source: https://github.com/nomic-ai/gpt4all
Disclaimer: NIH, shamelessly stolen from @polygon's flake at https://github.com/polygon/gpt4all-nix, all the props to him/her! @polygon if you want to claim commit authorship, feel free to amend this PR and do the necessary changes, I'm totally OK with that.
Fix #259798
Tested locally successfully:

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/gpt4all-nix-derivation/27744/22

@GrahamcOfBorg build gpt4all-chat

Marking it as draft, I just noticed that there's a .desktop file in the source, going to add it.

I also noticed that there is an include folder in the outputs. I don't believe that we actually need that one.

I don't really know why CI is not building the new derivation, so I will trigger it from here.
@GrahamcOfBorg build gpt4all-chat

I don't really know why CI is not building the new derivation, so I will trigger it from here.
@GrahamcOfBorg build gpt4all-chat

I don't really know why CI is not building the new derivation, so I will trigger it from here.
@GrahamcOfBorg build gpt4all-chat

Sadly, removing the lib folder broke the app.

Sadly, removing the lib folder broke the app.

Not the whole folder, just the files ending in .a inside the folder. Just gave it a got and seems to work without issues.

That's indeed working when removing lib/*.a files! Committed and pushed.

I don't really know why CI is not building the new derivation, so I will trigger it from here.
@GrahamcOfBorg build gpt4all-chat

In the device selection in cannot choose the gpu in any way, even though vulkan is included in the package. The only option avaliable is the cpu.

In the device selection in cannot choose the gpu in any way, even though vulkan is included in the package. The only option avaliable is the cpu.

Do you have any idea on how to fix that?

In the device selection in cannot choose the gpu in any way, even though vulkan is included in the package. The only option avaliable is the cpu.

Are you running NixOS or Nix on another distribution?

NixOS

In the device selection in cannot choose the gpu in any way, even though vulkan is included in the package. The only option avaliable is the cpu.

Do you have any idea on how to fix that?

Not really

I don't really know why CI is not building the new derivation, so I will trigger it from here.
@GrahamcOfBorg build gpt4all-chat

Hello ! I get this error
ix run github:NixOS/nixpkgs/nixos-unstable#gpt4all-chat
[Warning] (Tue Feb 20 10:41:30 2024): Detected locale "C" with character encoding "ANSI_X3.4-1968", which is not UTF-8.
Qt depends on a UTF-8 locale, and has switched to "C.UTF-8" instead.
If this causes problems, reconfigure your locale. See the locale(1) manual
for more information.
[Debug] (Tue Feb 20 10:41:31 2024): deserializing chats took: 0 ms
[Warning] (Tue Feb 20 10:41:31 2024): Could not connect "org.freedesktop.NetworkManager" to "stateChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMState
[Warning] (Tue Feb 20 10:41:31 2024): Could not connect "org.freedesktop.NetworkManager" to "connectivityChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMConnectivityState
[Warning] (Tue Feb 20 10:41:31 2024): Could not connect "org.freedesktop.NetworkManager" to "deviceTypeChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMDeviceType
[Warning] (Tue Feb 20 10:41:31 2024): Could not connect "org.freedesktop.NetworkManager" to "meteredChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMMetered
[Warning] (Tue Feb 20 10:41:31 2024): qglx_findConfig: Failed to finding matching FBConfig for QSurfaceFormat(version 2.0, options QFlagsQSurfaceFormat::FormatOption(), depthBufferSize -1, redBufferSize 1, greenBufferSize 1, blueBufferSize 1, alphaBufferSize -1, stencilBufferSize -1, samples -1, swapBehavior QSurfaceFormat::SingleBuffer, swapInterval 1, colorSpace QColorSpace(), profile  QSurfaceFormat::NoProfile)
[Warning] (Tue Feb 20 10:41:31 2024): qglx_findConfig: Failed to finding matching FBConfig for QSurfaceFormat(version 2.0, options QFlagsQSurfaceFormat::FormatOption(), depthBufferSize -1, redBufferSize 1, greenBufferSize 1, blueBufferSize 1, alphaBufferSize -1, stencilBufferSize -1, samples -1, swapBehavior QSurfaceFormat::SingleBuffer, swapInterval 1, colorSpace QColorSpace(), profile  QSurfaceFormat::NoProfile)
[Warning] (Tue Feb 20 10:41:31 2024): qglx_findConfig: Failed to finding matching FBConfig for QSurfaceFormat(version 2.0, options QFlagsQSurfaceFormat::FormatOption(), depthBufferSize -1, redBufferSize 1, greenBufferSize 1, blueBufferSize 1, alphaBufferSize -1, stencilBufferSize -1, samples -1, swapBehavior QSurfaceFormat::SingleBuffer, swapInterval 1, colorSpace QColorSpace(), profile  QSurfaceFormat::NoProfile)
[Fatal] (Tue Feb 20 10:41:31 2024): Could not initialize GLX
Aborted
and this one
nix run github:NixOS/nixpkgs/nixpkgs-unstable#gpt4all-chat
[Warning] (Tue Feb 20 10:42:32 2024): Detected locale "C" with character encoding "ANSI_X3.4-1968", which is not UTF-8.
Qt depends on a UTF-8 locale, and has switched to "C.UTF-8" instead.
If this causes problems, reconfigure your locale. See the locale(1) manual
for more information.
[Debug] (Tue Feb 20 10:42:32 2024): deserializing chats took: 1 ms
[Warning] (Tue Feb 20 10:42:32 2024): Could not connect "org.freedesktop.NetworkManager" to "stateChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMState
[Warning] (Tue Feb 20 10:42:32 2024): Could not connect "org.freedesktop.NetworkManager" to "connectivityChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMConnectivityState
[Warning] (Tue Feb 20 10:42:32 2024): Could not connect "org.freedesktop.NetworkManager" to "deviceTypeChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMDeviceType
[Warning] (Tue Feb 20 10:42:32 2024): Could not connect "org.freedesktop.NetworkManager" to "meteredChanged" : Type not registered with QtDBus in parameter list: QNetworkManagerInterface::NMMetered
[Warning] (Tue Feb 20 10:42:32 2024): qglx_findConfig: Failed to finding matching FBConfig for QSurfaceFormat(version 2.0, options QFlagsQSurfaceFormat::FormatOption(), depthBufferSize -1, redBufferSize 1, greenBufferSize 1, blueBufferSize 1, alphaBufferSize -1, stencilBufferSize -1, samples -1, swapBehavior QSurfaceFormat::SingleBuffer, swapInterval 1, colorSpace QColorSpace(), profile  QSurfaceFormat::NoProfile)
[Warning] (Tue Feb 20 10:42:32 2024): qglx_findConfig: Failed to finding matching FBConfig for QSurfaceFormat(version 2.0, options QFlagsQSurfaceFormat::FormatOption(), depthBufferSize -1, redBufferSize 1, greenBufferSize 1, blueBufferSize 1, alphaBufferSize -1, stencilBufferSize -1, samples -1, swapBehavior QSurfaceFormat::SingleBuffer, swapInterval 1, colorSpace QColorSpace(), profile  QSurfaceFormat::NoProfile)
[Warning] (Tue Feb 20 10:42:32 2024): qglx_findConfig: Failed to finding matching FBConfig for QSurfaceFormat(version 2.0, options QFlagsQSurfaceFormat::FormatOption(), depthBufferSize -1, redBufferSize 1, greenBufferSize 1, blueBufferSize 1, alphaBufferSize -1, stencilBufferSize -1, samples -1, swapBehavior QSurfaceFormat::SingleBuffer, swapInterval 1, colorSpace QColorSpace(), profile  QSurfaceFormat::NoProfile)
[Fatal] (Tue Feb 20 10:42:32 2024): Could not initialize GLX
Aborted

The solution was to do:
NIXPKGS_ALLOW_UNFREE=1 nix run --override-input nixpkgs nixpkgs/nixos-unstable --impure github:guibou/nixGL -- nix run nixpkgs#gpt4all-chat

Correct, ack, thanks!


Error: Only local python files are supported: gs://...

I extended the docker image using the recent spark-2.2.0-k8s-0.4.0-bin-2.7.3 release to add the GCS (Google Cloud Storage) connector.
Observed:
It works great for scala jobs / jars with a gs://<bucket>/ prefix - I see it creates the init container and does populate the spark-files from what was already in GCS. However, when I try to submit a python job (or use --py-files), the spark-submit client does not allow the gs:// prefix and refuses the job.
Error: Only local python files are supported: gs://<my_bucket_name>/pi.py
Run with --help for usage help or --verbose for debug output

Expected:
The job to be allowed by spark-submit, the relevant files populated in an initcontainer, and available for the spark-driver-py and spark-executor-py to use successfully.
(FYI To add the GCS connector, I added these lines to spark-base Dockerfile:)
ENV hadoop_ver 2.7.4
# Add Hadoop 2.x native libs
ADD http://www.us.apache.org/dist/hadoop/common/hadoop-${hadoop_ver}/hadoop-${hadoop_ver}.tar.gz /opt/
RUN cd /opt/ && \
    tar xf hadoop-${hadoop_ver}.tar.gz && \
    ln -s hadoop-${hadoop_ver} hadoop

# Add the GCS connector.
ADD https://storage.googleapis.com/hadoop-lib/gcs/gcs-connector-latest-hadoop2.jar ${SPARK_HOME}/jars/

I should note that for the GCS connector, I also had to add some runtime config files (notably core-site.xml, start-common.sh which I merged into the this repo's entrypoint.sh), mostly based on https://github.com/kubernetes-incubator/application-images/tree/master/spark
I also had to add :${SPARK_HOME}/conf to SPARK_CLASSPATH in spark-driver-py and spark-executor-py, for it to pick up the core-site.xml.

cc @liyinan926
This looks similar to your work with GCS.

This should be fixed by adding !isKubernetesCluster to https://github.com/apache-spark-on-k8s/spark/blob/branch-2.2-kubernetes/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala#L328.

BTW: @paulreimer I found that instead of baking in core-site.xml just for configuration (e.g., service account configuration) for the gcs connector, you could pass in the configuration properties using --conf spark.hadoop.[ConfigurationName].

Interesting, it took me so long to figure out to add ${SPARK_HOME}/conf to the SPARK_CLASSPATH, to get it to pick up core-site.xml, but I also tried to set fs.gs.project.id from the command line and couldn't figure it out. Does your suggestion mean I could use --conf spark.hadoop.fs.gs.project.id (e.g. prefix it with spark.hadoop)? That would have saved me a lot of time.
One nice thing about baking it in though, is the start-common.sh script detects the GCE project name and writes the fs.gs.project.id setting in core-site.xml before starting. That way, at least for GCE clusters, you don't need to submit that info to spark-submit and can re-use the same image (it will work automatically for gs:// URIs, then, assuming the GCE nodes have access to the storage bucket).

(I was using only GCE resources, and so allowing "application default credentials" to Just Work, instead of manually specifying service accounts.)

@paulreimer Yes, you can use --conf spark.hadoop.fs.gs.project.id. Spark will peel off the prefix spark.hadoop.

@paulreimer FYI https://github.com/liyinan926/spark-gcp-examples/blob/master/spark-examples/bigquery-wordcount/README.md.

Sounds good, I will need something like that for the non-GCE clusters.
I was unable to build a working distribution with the !isKubernetesCluster change applied (it did build, but the initcontainer doesn't work.
My build also fails for Scala jobs that worked before with my image with the GCS container added (using the 0.4.0 release jars), so something must be wrong with my build environment (I have never built before). I used build/mvn -T4 -DskipTests package, and I noticed that  there are way more jars in the official release tarball than were generated in assembly/target/scala-2.11/jars. I also didn't get a dist tarball at the end of the process, not sure if that is expected.
I would be happy to test updated binaries from a working build, with the !isKubernetesCluster change applied, if anyone else can build them.

Try this build command
./dev/make-distribution.sh --pip --tgz -Pmesos -Pyarn -Pkinesis-asl -Phive -Phive-thriftserver -Pkubernetes -Phadoop-2.7 -Dhadoop.version=2.7.3.

Right on, that command worked for me, and the suggested change also worked! I was able to successfully submit my python job, using gs:// on GCE (without a local copy of the file, and without using the resource-staging-server). I also applied the change to the check for R files in the same place in that file.
Note, I only had to replace the spark-submit client binary from my build, I was able to use my existing images -- based on the official 0.4.0 binaries -- with the GCS connector added. Seems it was really just that client check denying the job, the initcontainer part worked smoothly with a gs:// URI.
Thanks so much, I really appreciate your help, @liyinan926 !

Cool! Can you submit a PR with the change? Thanks!


Update runtime2 todos

Remove or reduce todos that have been addressed by latest pull
request:

error struct instead of error message
validation errors
DSOM "fixed" getter
DRY for duplicate code
count of parserStatements/unparserStatements

May I have a +1 so I can merge this pull request?


Broken README badges

I'm not sure what's up with the elm package version badge, but that needs figured as does the broken build badge but a blocker for that will be #16 to reinstate some actual CI that we can say passed or failed.

fixed!


added WaitingList, QRCodeList, and FacilityList classes

For Lottery.java, it's supposed to handle "Send notifications to entrants of their lottery outcome" and "Handle re-selection when an invitation is declined" too, no? I was also doing Lottery.java since I didn't know you were working on it, and I implemented those responsibilities mentioned in the CRC cards.

For Lottery.java, it's supposed to handle "Send notifications to entrants of their lottery outcome" and "Handle re-selection when an invitation is declined" too, no? I was also doing Lottery.java since I didn't know you were working on it, and I implemented those responsibilities mentioned in the CRC cards.

Oh sorry I was going to implement the rest of the features afterwards. Since you're already done we'll use your implementation.

I can add my Lottery.java but idk if it'll cause problems when trying to merge, but I'll make the request and you and Nathan can both check it over to see if you think it's correct or not

For your WaitingList.java, where do you handle "Support querying for entrant details", which is mentioned in the CRC card?

For your WaitingList.java, where do you handle "Support querying for entrant details", which is mentioned in the CRC card?

I've edited the file

I think the querying looks good, but let's wait to see what Nathan thinks.


UK local numbers treated as possible and invalidly converted to E.164

Country/region affected: GB
Example number(s) affected: 1111
The phone number range(s) to which the issue applies: Numbers below 9 digits
The type of the number(s): Local/ short
Supporting evidence:
http://www.ofcom.org.uk/static/archive/oftel/publications/numbering/2003/num_guide.htm
http://stakeholders.ofcom.org.uk/binaries/telecoms/numbering/Numbering_Plan_July2015.pdf
http://libphonenumber.appspot.com/phonenumberparser?number=1111&country=GB
libphonenumber accepts these extremely short numbers as possible numbers, and outputs them in E.146 format as, for instance,  +441111. However, the normal UK numbering plan does not include numbers this short. There are, admittedly, nationally-dialable short codes for special services, but since some of those are 3-digit numbers (999, 111 etc.) that are not marked as 'possible' I don't think 4-digit numbers should be either.
In particular, the argument for permitting these numbers is probably that they are valid local numbers (UK local numbers go down to 4 digits in length), but in that case it is invalid to produce E.164 format by prepending +44; the area code would need to be present on the number.

isValidNumber is returning false though, it's only isPossibleNumber that is returning true.
Whenever I format any numbers, I'm checking if they are valid first.

Yes, 'valid' returns false - but isPossibleNumber shouldn't be returning 'true' if the base numbering plan doesn't allow the number. As I understand it, the difference between those two functions is that 'valid' will tell you if it's part of an allocated number block, and 'possible' should tell you if it could be allocated as part of the current numbering scheme. 1111 is not a possible number under the current numbering plan.
For context - I am using isPossibleNumber to decide if the numbering plan can be applied to a number, because I would like not to have to wait for an update and apply it to all systems every time a new number block starts to be assigned.
Perhaps my real question here is "why does isPossibleNumber return true, when we have a separate method isPossibleShortNumber?" I'd expect the behaviour here to be false for the former, and true for the latter.

To understand the behavior of functions, please refer to the documentation in the Javadoc.
See "public ValidationResult isPossibleNumberWithReason(PhoneNumber number)" in PhoneNumberUtil.java for example:
/**
   * Check whether a phone number is a possible number. It provides a more lenient check than
   * {@link #isValidNumber} in the following sense:
   *<ol>
   * <li> It only checks the length of phone numbers. In particular, it doesn't check starting
   *      digits of the number.
   * <li> It doesn't attempt to figure out the type of the number, but uses general rules which
   *      applies to all types of phone numbers in a region. Therefore, it is much faster than
   *      isValidNumber.
   * <li> For fixed line numbers, many regions have the concept of area code, which together with
   *      subscriber number constitute the national significant number. It is sometimes okay to dial
   *      the subscriber number only when dialing in the same area. This function will return
   *      true if the subscriber-number-only version is passed in. On the other hand, because
   *      isValidNumber validates using information on both starting digits (for fixed line
   *      numbers, that would most likely be area codes) and length (obviously includes the
   *      length of area codes for fixed line numbers), it will return false for the
   *      subscriber-number-only version.
   * </ol>
   * @param number  the number that needs to be checked
   * @return  a ValidationResult object which indicates whether the number is possible
   */

The library is working as intended here.

Thank you for pointing me at this. I have had some trouble with the code comment, because it appeared to be false in its assertion that "It only checks the length of phone numbers" - e.g. US<PHONE_NUMBER> is rejected whereas<PHONE_NUMBER> is not. I now understand that this is because of the way these numbers are parsed.
I wonder if you could tell me whether it is actually possible to access the lengths used by isValidNumber, without doing the associated digit checking? I can't see a way to get this info out of the metadata. I will ask this question on the discussion group, instead of on this issue, if I am granted access.

Happy to discuss further at libphonenumber-discuss - leaving this here for anyone else who may have read this far.
One-off: Just look at PhoneNumberMetadata.xml.
Programatically: If you parse a metadata file e.g. PhoneNumberMetadataProto_AC as phonemetadata.proto, look at its general_desc.national_number_pattern - it will be e.g. "\d{7,10}" for GB. This isn't limited to length though, sometimes it's a more specific regex like "[1-9]\d{7}" for AM, which still is able to tell you it's 8 digits.


Inverse rotation using EulerAngles

When calling inverseRotate on EulerAngles, the procedure is as follows:

convert EulerAngles to RotationQuaternion
call inverted on RotationQuaternion
call rotate on RotationQuaternion

The question is whether rotating with the inverse (transpose) of a RotationMatrix is not the better way. Therefore, I wrote a little test attached below. The output on my machine (compiled in release mode) is:

running 1000000 rotation tests
timing angle.inverseRotate(vector)
Total time: 498 ms. Average: 0.000498 ms
timing angle.toRotationMatrix().inverted().rotate(vector)
Total time: 150 ms. Average: 0.00015 ms
average error: 3.24446e-16
max error 2.67665e-15 for pair:
euler angles:   3.03138   1.18812 -0.602041
vector to rotate: -0.678126  0.990644 -0.910583
vector inverse rotated (direct): -0.203027  -1.45571 -0.331804
vector inverse rotated (rotation matrix transpose): -0.203027  -1.45571 -0.331804

So it seems numerical precision is pretty similar but runtime is faster. So maybe we should specialize inverseRotate for EulerAngles (both Xyz and Zyx)?
@gehrinch As discussed by phone, here the test.
#include <chrono>
#include <kindr/Core>

int main()
{

	static const size_t nTests = 1000000;
	std::vector<kindr::EulerAnglesXyzD, Eigen::aligned_allocator<kindr::EulerAnglesXyzD> > angles(nTests);

	typedef std::vector<Eigen::Vector3d, Eigen::aligned_allocator<Eigen::Vector3d> > test_vector_t;
	test_vector_t vectors(nTests);
	test_vector_t vectorsRotatedNormal(nTests);
	test_vector_t vectorsRotatedMatrix(nTests);

	std::cout<< "running "<<nTests<<" rotation tests"<<std::endl;

	for (size_t i=0; i<nTests; i++)
	{
		angles[i].setRandom();
		vectors[i].setRandom();
	}

	std::cout << "timing angle.inverseRotate(vector)" << std::endl;
	auto start = std::chrono::high_resolution_clock::now();
	for (size_t i=0; i<nTests; i++)
	{
		vectorsRotatedNormal[i] = angles[i].inverseRotate(vectors[i]);
	}
	auto end = std::chrono::high_resolution_clock::now();
	auto diff = end - start;
	size_t msTotal = std::chrono::duration <double, std::milli> (diff).count();
	std::cout << "Total time: " << msTotal << " ms. Average: " << msTotal/double(nTests) << " ms" << std::endl;

	std::cout << "timing angle.toRotationMatrix().inverted().rotate(vector)" << std::endl;
	start = std::chrono::high_resolution_clock::now();
	for (size_t i=0; i<nTests; i++)
	{
		vectorsRotatedMatrix[i] = kindr::RotationMatrixD(angles[i]).inverted().rotate(vectors[i]);
	}
	end = std::chrono::high_resolution_clock::now();
	diff = end - start;
	msTotal = std::chrono::duration <double, std::milli> (diff).count();
	std::cout << "Total time: " << msTotal << " ms. Average: " << msTotal/double(nTests) << " ms" << std::endl;

	double error = 0.0;
	double maxError = 0.0;
	size_t maxErrorIndex = 0;
	for (size_t i=0; i<nTests; i++)
	{
		double err = (vectorsRotatedNormal[i]-vectorsRotatedMatrix[i]).norm();
		if (err>maxError)
		{
			maxError = err;
			maxErrorIndex = i;
		}
		error += err;
	}
	std::cout << "average error: "<<error/double(nTests)<<std::endl;
	std::cout << "max error "<<maxError<<" for pair: "<<std::endl;
	std::cout << "euler angles: " <<  angles[maxErrorIndex].toImplementation().transpose() << std::endl;
	std::cout << "vector to rotate: "<< vectors[maxErrorIndex].transpose() << std::endl;
	std::cout << "vector inverse rotated (direct): "<< vectorsRotatedNormal[maxErrorIndex].transpose() << std::endl;
	std::cout << "vector inverse rotated (rotation matrix transpose): "<< vectorsRotatedMatrix[maxErrorIndex].transpose() << std::endl;

	}

@neunertm that looks good to me. We should specialize it then.

I can take a look if you want. But if you prefer to do it yourself, I am happy to back off ;).