[WIP] Bond parameter interpolation based on Wiberg bond orders

[ ] Update docstrings/documentation, if applicable

[ ] Update changelog

[ ] Add a test force field with a <Bonds version="0.3" potential="harmonic" fractional_bondorder_method="Wiberg" fractional_bondorder_interpolation="linear"> element

[ ] Add functionality to use user-specified WBO instead of default (e.g. AM1)

[ ] Fractional bond order getter from molecule, and/or individual bonds? Already present for Bond objects. Data type return for molecule level getter?

[ ] How to handle Wiberg values outside interpolation k_bondorder1-->k_bondorder2 interval?

[ ] Add tests for fractional bond orders in the interval, and out of the interval.

[ ] How to implement functionality in label_parameters, e.g. return the functional form or do an explicit evaluation (calculating WBOs if not present).

[ ] How to/Should we allow mixed sections, where nonfractional and fractional are used concurrently

[ ] Make sure simple garbage inputs do not pass tests:
- [ ] k_bo1 and k_bo3 but not k_bo2

[ ] Ensure that bond object order is consistent between isomorphic molecules during parameterization (needs a working isomorphic function e.g. from Josh Horton's #472 PR)

Codecov Report

Merging #501 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

I'm taking over this effort and will open a different PR for it.


Add CODEOWNERS file to the repository.

Also, should we add --reviewer to the PR creation?

It's better to use the CODEOWNERS feature in the downstream repo to add the relevant reviewers for the changed files.
Originally posted by @behnazh in https://github.com/jenstroeger/python-package-template/issues/482#issuecomment-1493439563

PR https://github.com/jenstroeger/python-package-template/pull/511, commit https://github.com/jenstroeger/python-package-template/commit/5d0be9212cf3515cb910cc39ba23d44c50926244.


"Prompt" is actually a adaptor

Thank you for your question, this is one implementation of the adaptor. But we think this can be seen as a way to learn the prompt from the features. So we name it "Prompt".  May the related paper "Prompt-Matched Semantic Segmentation" can support this idea.


Custom header when export to PDF

I've upgraded to sponsor version, in order to export MD to PDF, and I'm trying to add a custom HTML header to each page, but I can't figure out how to do it.
Can you provide any example on how to do it?
Thanks!!

I am also wondering this. There is already the possibility to add text in the header. But text has no spacing to the content, so it is not useful. Please provide an example of how to create a functional header and footer. Thank you!

would like to have the feature as well.


Can't use https proxy for webdriver on Tomcat

Hi there
I can use https proxy in local environment with the IE webdriver  by just copying the readme code and add the .cer.But when I deploy the same code on tomcat,the webdriver can't view https sites.

sorry，perhaps I need to install the cer on tomcat

Perhaps it's the tomcat's problem.When I switch to jetty,it works fine.

This does appear to be tomcat-specific. The stack trace shows:
Caused by: java.lang.ClassNotFoundException: org.apache.tomcat.jni.CertificateRequestedCallback
I'm not sure what tomcat is doing to interfere with Netty's SslContextBuilder there. I don't use tomcat so I don't know any details, but if you are still having the issue and can dig up some more information, feel free to reopen.


Real Time System Latency

Hi,
This is very nice and seems that very powerfull. I want to use this library on my realtime motion simulation project. I am currently processing commands from pc and generating pulses with arduino with interrupt timers, like you. But I dont have acceleration / decc routines, and I am very happy to found your library. Also nano/uno is not enought mcu power for 4 stepper. So ESP32 is also very vital for me.
My question is latency. I am asking only latency of library after run command. On docs, I saw 4ms for ATMEGA and 10ms for ESP, but I am not sure.
Thanks for help.

The implementation on avr and esp32 are quite different, so I focus only on esp32. As I have no specific test case for this, my answer is based on design only.
Shortest latency is for this case:

stepper is NOT running
auto enable is not active

Then the time till the first step comes out is:

execution of move/moveTo, which call _calculate_move()
execution of _calculate_move(), which sets the variables for the cyclicTask
The next cyclicTask (period Time 10ms), will inits hardware registers to output the first step
The hardware will output first step after the time t0. This t0 equals the slowest step frequency at the beginning of the ramp

Example: If the step frequency at the beginning of the ramp is 10Hz, then the latency would be worst case: 10ms+100ms+several us=110ms and best case 100ms.
If the step frequency at the beginning of the ramp is 1kHz, then the latency would be worst case: 10ms+1ms+several us=11ms and best case 1,0xx ms
Hope this helps

Very clear. Thanks again.

In case the stepper is already running (queue contains commands), then a change in commands could take for slow stepper movements even seconds. Reason is - as mentioned under ISSUES, that the queue is always filled completely and the esp32 queue is rather deep (32 entries). Currently I am thinking about an implementation to fill the queue only for approx 20ms as lookahead. Just this is a bit complicated for implementation, that’s why not yet done.

If I am wrong, please correct me. When I send new position command while previous command is not completed, I will wait queue to complete ? Is that possible to clear queue and re calculate route instantly ?

if next move/moveTo changes direction, then you need to wait till the motor stops. Initiating a stop can be done with stopMove(). Current implementation of stopMove() is just a moveTo(end_pos) with end_pos being the position, which is reached with immediately starting deceleration.
If changes shall be done in the same direction as the motor is already running, then the result depends on the command. A moveTo will replace the end position of the current running movement. A move will be added to the end position of the current running movement. Any move/moveTo will consider any updates in speed or deceleration.

and I think, the moveTo for a running motor has the risk of a race condition in case of a running motor....
Explanation: the current end position of a command queue is read and used for calculation. If between getting the end position and update of the control variables the task/interrupt routine gets executed and commands are added, then the end position may be missed.
Sounds like a bug, which I need to look in further.

Thanks for your time. I am watching your project and I will write my results.

After some more analysis and tests by injecting virtual interrupts during testing, I could not found any bug in regard to wrong end position and rereading the code, confirms this.
Nevertheless, those tests revealed a possible divide by zero error, which is quick fixed now until a better rework is done. So it was work the exercise.
For this comment, I think this issue can be closed, because I reopened it due to a possible bug in move/moveTo, while there isn’t.


bigdataviewer-image-loaders dependency

@NicoKiaru I may want to have    as a dependency of mobie-io.
Do you know whether I have to upload all those dependencies to the MoBIE update site or could I omit some of them?
		<dependency>
			<groupId>com.zeroc</groupId>
			<artifactId>glacier2</artifactId>
			<version>${glacier2.version}</version>
		</dependency>
		<dependency>
			<groupId>com.zeroc</groupId>
			<artifactId>ice</artifactId>
			<version>${ice.version}</version>
		</dependency>
		<dependency>
			<groupId>org.openmicroscopy</groupId>
			<artifactId>ome-xml</artifactId>
		</dependency>
		<dependency>
			<groupId>org.openmicroscopy</groupId>
			<artifactId>omero-blitz</artifactId>
			<version>${omero-blitz.version}</version>
			<exclusions>
				<exclusion>  <!-- declare the exclusion here -->
					<groupId>net.sourceforge.findbugs</groupId>
					<artifactId>annotations</artifactId>
				</exclusion>
			</exclusions>
		</dependency>
		<dependency>
			<groupId>org.openmicroscopy</groupId>
			<artifactId>omero-gateway</artifactId>
			<version>${omero-gateway.version}</version>
		</dependency>
		<dependency>
			<groupId>org.openmicroscopy</groupId>
			<artifactId>omero-model</artifactId>
		</dependency>

What we gain from this is that we can open anything that is supported by Bio-Formats as SpimData.
    public AbstractSpimData< ? > openSpimData(String imagePath, ImageDataFormat imageDataFormat) throws UnsupportedOperationException, SpimDataException {
        switch (imageDataFormat) {
            case BioFormats:
                return openWithBioFormats(imagePath);

Hi tischi,
Normally you just need to add bigdataviewer-image-loaders (v0.4.3 on the 15th Feb 2023) dependency. The rest comes with transitive dependencies.

Do you know whether I have to upload all those dependencies to the MoBIE update site or could I omit some of them?

So you should be able to omit all of them.
Which version did you try ?
HOWEVER, nothing works currently on the update sites. I have several repos to migrate at the same time on the PTBIOP update site. My target for the 'big update' is in two weeks.
Right now you can test and see if everything works ok in the IDE, but then we need to synchronize the Fiji release.
Side note: you can open any bio-formats supported file, as well as dataset from OMERO, and also QuPath projects. All lazy loaded, with multi-resolution support and with 8 bits unsigned, 16 bits unsigned, float and rgb pixel type support.

I know that those are transitive dependencies, but I think I still have to upload them to the MoBIE update site.
And I think if @tinenvez wants to use mobie-io in Mastodon then I think he also would have to add them to their update site.
If I only upload bigdataviewer-image-loader we are getting within Fiji, e.g.:
Caused by: java.lang.NoClassDefFoundError: omero/model/enums/UnitsLength     at ch.epfl.biop.bdv.img.opener.OpenerSettings.<init>(OpenerSettings.java:78)

version 0.4.0

Yes, so this issue is fixed is 0.4.3

Now if you activate the OMERO-5.5-5.6 update site this should be ok as well

With 0.4.3: no need to have the OMERO update site enabled if you stick to Bio-Formats.

 Will fix in 0.4.4

Meanwhile I can confirm that from 0.4.3 on one only needs to ship bigdataviewer-image-loaders on the update site. And maybe, if one does not use the Bio-Formats features of mobie-io one does not even need to ship that...

Ok, the 0.4.4 arrives in 2 minutes

@tinevez I think with those changes

Just uploading mobie-io without any new additional dependencies should work for you.
If you also want to open Bio-Formats file formats as SpimData, just additionally uploading bigdataviewer-image-loaders should do the job.


Missing items on installation

While building and installing Arm NN, I have the following items which are not installed from cmake build system:

ArmnnConverter
ArmnnQuantizer
libarmnnQuantizer.so*

I think this should be installed.

And libarmnnQuantizer.so is not versioned.

Hi @ggardet @MatthewARM ,
Any update on this? How do i create the ArmnnConverter and ArmnnQuantizer?

How do i create the ArmnnConverter and ArmnnQuantizer?

They are built automatically when building ArmNN.

Hi @ggardet,
Yes, those are built with ArmNN I could not locate those earlier inside the docker. Thanks for the reply anyway.

Hi @ggardet,
The ArmnnQuantizer and libarmnnQuantizer.so have since been deprecated and removed; however I've added a patch to install ArmnnConverter here:
https://review.mlplatform.org/c/ml/armnn/+/5639
Regards,
Francis.

Going to close as the patch has been merged, please reopen if you have any issues.
Thanks,
Francis.


MClimate webhook template

Summary

...
Changes

...
...

Notes for Reviewers

...
Checklist

[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[x] Compatibility: The changes are backwards compatible, they don't break existing deployments.
[x] Testing: The changes are tested.
[x] Documentation: Relevant documentation is added or updated.

@MClimate can you please elaborate on using the API key for authorization? Which API key is this? I suppose it is from the MClimate app, but where can a user find it, can you provide some info on that? We're working on adding the documentation for this integration https://github.com/TheThingsIndustries/lorawan-stack-docs/pull/668

Hello Nejra,
We used the key in a previous demo version. We no longer need it, so we
will remove it now. Thank you for asking us about this and sorry for the
inconvenience.
Have a great weekend!


gateway: Epochs wrapping over too fast

Currently, the frame sequence number if 3 bytes long, i.e. epochs wrap over every ~17m packets. At >10GB speed that doesn't amount to much time (single seconds).
We need at least 4 bytes for the sequence number.

At 1Tbps, that's ~200B pps. With a 6B sequence number, that wraps every 40 hours. I propose 8 bytes for the sequence number, which means it will wrap after 299 years. That should be future-safe :) In order to accommodate the larger sequence number (and the larger epoch number for #2950), we should move to a 16B sig header.
Proposal:
   0B       1        2        3        4        5        6        7
   +--------+--------+--------+--------+--------+--------+--------+--------+
   | Sig Ver| Sess Id|      Index      |               Epoch               |
   +--------+--------+--------+--------+--------+--------+--------+--------+
   |                               Sequence number                         |
   +--------+--------+--------+--------+--------+--------+--------+--------+

We gain a 1 byte field to specify the sig framing protocol version, which would allow us to do transitions in the future.

Let's adopt the proposal in the previous comment with the following modifications:

rename "epoch" to "stream ID"
limit it to 20 bits so that it has the same size as flow ID
keep remaining 12 bits reserved

Additionally, let's remove packet sizes in the payload. Packet size can be inferred from the encapsulated IP header. Not having size field makes implementation of encoding with gather/scatter arrays easier.
Also, let's require that first 40 bytes of the packet (if available) are never split. That would mean that entire static part of the IP header (whether IPv4 or IPv6) can be always read in one go and doesn't have to be reassembled. It also makes encoding implementation rather simple (if there's less than 40 remaning bytes, don't even try to pack next packet into the frame).

Implemented according to kormat's proposal.

Implemented according to kormat's proposal.


EC2 instances created but not added to ECS cluster

Summary
Ran terraform apply on complete-ecs/main.tf to create ECS cluster. Confirmed on AWS console the ECS cluster gets created. However, created EC2 instances are not added to cluster.

Steps to reproduce:

Set following values in complete-ecs/main.tf

provider "aws" {
  region = "us-west-2"
}

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "~> 2.0"

  name = local.name

  cidr = "<IP_ADDRESS>/16"

  azs             = ["us-west-2a", "us-west-2b"]
  private_subnets = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]
  public_subnets  = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

  enable_nat_gateway = false # this is faster, but should be "true" for real

  tags = {
    Environment = local.environment
    Name        = local.name
  }
}

  # Auto scaling group
  asg_name                  = local.ec2_resources_name
  vpc_zone_identifier       = module.vpc.private_subnets
  health_check_type         = "EC2"
  min_size                  = 1
  max_size                  = 3
  desired_capacity          = 1
  wait_for_capacity_timeout = 0

Ran terraform init, plan, and apply

You need to tell EC2 instances in user-data that they belong to ECS cluster. Use user_data script like this:
#!/bin/bash
{
  echo "ECS_CLUSTER=${cluster_name}"
} >> /etc/ecs/ecs.config

See Launching an Amazon ECS Container Instance, Step 6g for more.

I had the same problem. Running the script didn't solve the problem. how do i know if the sh script is ran?

@guohuang  I had the same issue did you figure out the solution?

@cmosguy no, i gave up. i use the generic terraform resources to solve the problem.

@antonbabenko I believe it's because of changes required by Amazon Linux 2, see similar issue  in another repo
https://github.com/arminc/terraform-ecs/issues/24

Nevermind, seems like this changes were enough
https://github.com/terraform-aws-modules/terraform-aws-ecs/pull/9


Release artifact for release v1.2.6

Release artifact for release v1.2.6
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/hold

/lgtm

/unhold


Cleaning up MA APIs before GA

Removal of the word "Anomaly" from (get|list|create|deelte|update)AnomalyAlertConfig() APIs.
Rename of MetricAnomalyAlertConfiguration to MetricAlertConfiguration.
Rename DetectionConditionsOperator to DetectionConditionOperator
Rename AnomalyAlertConfiguration.dimensionsToSplitAlert property to dimensionsToSplitAlert
Adding [get|set]adminEmail() to Hook
Removal of add** methods (that adds to list properties) from couple of models

/check-enforcer evaluate

/azp run java - metricsadvisor - ci

/azp run java - metricsadvisor - ci

/azp run java - metricsadvisor - ci

/azp run java - metricsadvisor - ci


Multitouch support

Contact Details
<EMAIL_ADDRESS>What is your request?
Currently only a single touch event works at one time for events, it would be nice if all available touch events would be handled.
tsParticles Version
2.5.1
Which library are you using?
React.js (react-particles, react-tsparticles, react-particles-js)
Code of Conduct

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

Closes this for using a more detailed issue #4965


Added checking of some array indexes because of errors thrown on PHP 7

lti base class ensures that HTTP_X_FORWARDED_PROTO index exists before using it. Note: In PHP 7.0 on nginx it was throwing issues. game.php ensure the difficulty exists in the custom launch attributes before using it

I'm going to close this as the example tool has moved to its own repo


add condition to pass the check of postal code for countries that don…

address: https://nanopay.atlassian.net/browse/NP-3151

If we are missing regions from countries we support, we should add them. Along with zip code validations.

If we are missing regions from countries we support, we should add them. Along with zip code validations.

@zahnno I agree.  @minsunny can you create a ticket to capturing all the missing regions and code validations.  Then work on it slowly (lower priority, as time permits), country by country.

@zahnno I agree.  @minsunny can you create a ticket to capturing all the missing regions and code validations.  Then work on it slowly (lower priority, as time permits), country by country.


sprout-ssh

I recommend we close this out in favor of https://github.com/pivotal-sprout/sprout-ssh/issues/2


Nulls values

How do I put null in the network model field?
I have a model like this
@JsonApi(type = "notes")
data class Note(
var name: String? = null,
var description: String? = null,
var note: String? = null,
@field:Json(name = "created-at")
var createdAt: Date? = null
}
I want to patch the null value in the createdAt field to the server, but this field is ignored and it simply does not exist in the body.
{"data":{"type":"note","id":"168356"}}
How can I send null?

How do I put null in the network model field?
I have a model like this
@JsonApi(type = "notes")
data class Note(
var name: String? = null,
var description: String? = null,
var note: String? = null,
@field:Json(name = "created-at")
var createdAt: Date? = null
}
I want to patch the null value in the createdAt field to the server, but this field is ignored and it simply does not exist in the body.
{"data":{"type":"note","id":"168356"}}
How can I send null?


Account for at least once message delivery

What is this PR trying to achieve?
I've seen this transformer test behave unreliably even though it is isolated. It appears that elasticmq (the container providing FakeSQS) adheres to the same "at least once" delivery guarantee that SQS does. As a consequences messages may be delivered more than once (and this behaviour is expected) so the tests need to account for that.
This could also explain reliability issues elsewhere!
Who is this change for?
🎲 👾
Have the following been considered/are they needed?

[ ] Deployed new versions
[ ] Run terraform apply.

Rather than fixing this one test (I've seen this issue elsewhere too), could we add another method to our SNS fixture that solves the problem at its core?
listUniqueMessagesReceivedFromSNS?

@alexwlchan I think it might be difficult to differentiate between situations where messages are the same because they should be or because of the multiple delivery.
I think I prefer it being explicitly accounted for in the tests.


Add PostCSS GTK

I love how I was able to do something crazy like this, and then get full source map support for free

Added, thanks!


Can not exit when there is more then one window

Introduced in https://github.com/rust-windowing/winit/commit/5a6cfc314ea4890d79eefb01ef94bd46ca7858cf there is a check that if there is more than one window it does not exit an event loop:
https://github.com/rust-windowing/winit/blob/66c117e599209ca1fb95219669ed161c1d3cc3ed/src/platform_impl/macos/app_state.rs#L348

we patched winit, but should create an issue in the upstream repo


Device perf fails to collect on ttnn models

Circular dependencies between tt_lib and ttnn with the decorator.py need to be fixed before updating the run_performance.sh and including the device perf metrics on the CI pipeline.

This issue was resolved merged to main.


Incorrect value_key for select options: change content_path to path

I used the config-migrator to transfer my forestry settings to my new cloudcannon site. I had a select field that populated a list of pages from a collection and would enter the path as the
page front matter:
---
title: American Trilogy
featured: true
spirit: _spirits/chapmans-apple-brandy.md
ingredients:
- 1 oz Chapmans Apple Brandy
- 1 oz Purpose Rye
- 1 brown sugar cube
- 3 dashes orange bitters
image: "/img/american-trilogy-cocktail.jpg"
---

forestry settings for select field:
fields:
- name: spirit
  type: select
  default: []
  config:
    required: false
    options: []
    source:
      type: pages
      section: spirits
      file:
      path:

cloudcannon configs for select field:
_inputs:
$.spirit:
  type: select
  comment: >-
    Select a spirit from the website. This list is populated by spirit
    pages added to the website.
  options:
    values: collections.spirits
    value_key: content_path

According to the docs, valid values for the value_key object are: 'id', 'uuid', 'path', 'title', 'name'.  content_path should be changes to path in order to make the field functional.

I just ran into this issue again and found my own issue! Change content_path to path.

Changed content_path to path
Add cocktails scenario to tests using this repo.
Built and tested select inputs on CloudCannon - they are now being populated by the collection using the value_key of path


Shiftx2 got sent to "pre" label

I haven't been keeping up with this, but I would hope that new recipes would be put in the main label by default.
cc #466
Why did this get sent to the "pre" label?

cc #445

I'm manually changing the label from pre to main on each of the bz2 files

Tagging @marscher : Any idea what happened here?  Did you make some changes to the build script that may have caused this?

I think so. The check for a package to go to "pre" is a setuptools related version class attribute name "pre-release", which in this case is true for a version number tagged "A" for alpha. So this is actually expected behaviour for this kind over version number scheme. How do we want to tag releases to be public or restrict them to testing community?
This question arises because we somehow need to automatize how to treat pre-releases.

Ok, shiftx2 doesn't follow pep440 at all, so we'll just have to keep an eye out in this instance whenever we try to rebuild it (which shouldn't be often)
An alternative would be to give the conda package a different, pep440 compliant version string that differs from the upstream number

We could add a blacklist file for non pep440 compliant packages.


fix: module doc should scan _meta.json

Summary
Related Issue

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!canary

:D more test cases to avoid `small errors big mistakes


zplug superset / lazy load / freeze

Problem Summary
why does zplug have to get initialized on each shell init?
Since the changes to the cache I cannot load the cache directly anymore because it depends on zplug functions itself, I made it work one more time by bypassing __zplug::io::file::load but now things changed again and seems the cache is hooked even deeper into zplug itself (?)
Is it possible to decouple functions needed for the cache to load from the zplug initialization and create a superset of zplug functions that load the cache on init, and then lazy load zplug when called from the interactive shell? Or maybe a client/daemon but that would be overkill i guess

There is no reaction so I'll close once. Feel free to re-open if there is a problem and ask us.

This request is similar to what I'm trying to accomplish via #381. The idea is to decouple the load-from-cache logic in order to speed up loading. Maybe zplug can provide a way to do this within its code base, or if not, maybe it can just provide a way to export the cache so that a different loader can take over. Then zplug can be loaded lazily – when needed to manage configuration, installation, updating, and so on. Where zplug shines brightest is its powerful and expressive configuration.
I have an implementation of a streamlined cache loader, which needs some work, but illustrates the concept @untoreh is describing. It'd be even better if zplug shipped with something like this, which could be run without loading all of zplug.
https://github.com/paulmelnikow/prezto/blob/zplug/modules/zplug/init.zsh
Despite the name, it's not specific to Prezto.


Add flake8-eradicate

This plugin might have been useful in the past as I forgot to uncomment some code, I guess it is worth adding it :)
https://github.com/sobolevn/flake8-eradicate

Can I give it a try? So I can familiarize with the the repo. 😃

@flpStrri sure! feel free to ping me on discord if you need any help!

I simply installed it and it is returning 1 when calling flake8. Great. But I think it is marking as an error some comments that were not supposed to be marked, like this:

But it works. 😄

@flpStrri nice, can you send a PR?

#443 should have closed this, right?

Yes 😊 I'll close this now :)


[Zoom] Not accepting child component

First of all, Thanks for giving this beautiful framework. It has always been a great experience to work with Material-UI.
Today, I updated my project to @material-ui/core@next. Successfully removed all deprecated props. But, failed to solve this issue. Prior to v4 this was working.

[ x ] This is not a v0.x issue.
[ x ] I have searched the issues of this repository and believe that this is not a duplicate.

// Input Field component
const InputField = ({
    field,
    form: { touched, errors },
    ...props
}) => {
    const error = errors[field.name];
    return (
        <TextField
            variant="outlined"
            margin="dense"
            error={touched && error ? true : false}
            helperText={touched && error ? `- ${error}` : null}
            fullWidth
            {...field}
            {...props}
        />
    );
};

Expected Behavior 🤔
Zoom component is not accepting a <Field/> component from Formik as child.
import { Field } from 'formik';

<Zoom in={show}>
         <Field
                 name="title"
                 type="text"
                 variant="outlined"
                 margin="dense"
                 component={InputField}
          />
</Zoom>

Current Behavior 😯
Zoom should be given a <div/> to wrap the <Field/> component
import { Field } from 'formik';

<Zoom in={show}>
           <div>
                  <Field
                            name="title"
                            type="text"
                            variant="outlined"
                            margin="dense"
                            component={InputField}
                   />
           </div>
</Zoom>

Error
I am getting this error in the console.
Warning: Function components cannot be given refs. Attempts to access this ref will fail. Did you mean to use React.forwardRef()?

Check the render method of `Transition`.

Context 🔦
I am trying to animate fields/buttons in form whenever the form appears in the DOM.
Your Environment 🌎

Tech
Version

Material-UI
^4.0.0-beta.0

React
^16.8.6

Browser
Chromium: 74.0.3729.108

Fun Fact: This is my first contribution 😋

@eps1lon Would this change work for you? It doesn't break any test but solves the warning here.
--- a/packages/material-ui/src/utils/reactHelpers.js
+++ b/packages/material-ui/src/utils/reactHelpers.js
@@ -18,11 +18,13 @@ export function useForkRef(refA, refB) {
    * This means react will call the old forkRef with `null` and the new forkRef
    * with the ref. Cleanup naturally emerges from this behavior
    */
-  return React.useCallback(
-    refValue => {
-      setRef(refA, refValue);
-      setRef(refB, refValue);
-    },
-    [refA, refB],
-  );
+  return React.useMemo(() => {
+    if (refA || refB) {
+      return refValue => {
+        setRef(refA, refValue);
+        setRef(refB, refValue);
+      };
+    }
+    return null;
+  }, [refA, refB]);
 }

It seems we won't fix this here: #15519. I believe Formik should forward the ref. I have shared a workaround: https://github.com/jaredpalmer/formik/pull/478#issuecomment-487742488.

Is this really a fix or just a workaround?

@vkasraj #15526 should actually fix this.

@vkasraj #15526 should actually fix this.

Waiting for next release 😋

i have some bug in 4.11.2


Put absolute nix path into lorri binary on compilation

I see no good reason why we shouldn’t compile a fixed path to nix into lorri, since we are using nix to build it anyway.
Especially with services, users report errors that stem from a missing nix in the environment. This would fix that problem for everyone.
Disadvantage: Using a different nix requires a recompile with an overwritten pkgs, but I don’t think that’s a big problem in practice. We could document it and make it easy to do.

I don't think we should do that, this makes using lorri with another custom nix variant harder than it should be.
We should instead show a good error message if nix wasn't found in $PATH, and ensure the NixOS module-provided lorri systemd user units already adds the nix from nix.package to it.

You are right I think.
We are working on generating better error messages when something in the environment is missing, https://github.com/target/lorri/pull/174 was merged and https://github.com/target/lorri/pull/199 adds a specialized exit code for these (as far as the code is aware of them).

So this can be closed?


Use sslv23 constant in newer versions of python

Changes Title (replace this with a logical title for your changes)
Description
Replace this with the PR description (mention the changes you have made, why
you have made them, provide some background and any references to the provider
documentation if needed, etc.).
For more information on contributing, please see Contributing
section of our documentation.
Status
Replace this: describe the PR status. Examples:

work in progress
done, ready for review

Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

:+1: I agree something needs to be updated with the SSL settings for newer versions of Python.
Might want to consider how 2.7.13 and 3.5.4 just changed the way SSL settings are applied (see ssl.PROTOCOL_TLS) due to OpenSSL deprecating version specific protocols.

:+1: thanks for the contribution @MattG401

The merge conflict here is complex i think these changes are quite old @Kami things have changed since they were authored. Namely https://github.com/apache/libcloud/pull/808
This PR needs to be rebased before it can be merged. thanks


Common cross-libraries token architecture

Description
There's a need for a preliminary study before implementing tokens in the library. While the currently described tokens are satisfactory on the design side, there remains a fundamental disparity between Figma's capability to handle and define tokens, and the implementation methods available on the development side.
In essence, Figma’s token mechanism requires designers to pre-create a set of tokens that may not be inherently useful for the core part of our tech libraries.
On the technical side, it is important to define what is available by default:

for developers using the Orange theme (default)
what can be overridden for country-specific themes, and how overrides will function for B-Brand themes

This must be considered at different levels of the token stack, including primitive raw values, semantic tokens, and component tokens.
Our objective is to establish a common cross-library architecture to ensure uniformity in terminology and implementation.
The result of this study must be mentioned in this issue before implementing it in this library
Note: Closing this issue will unlock all the "Tokens: *" issues in this repository
Study

[ ] Study the limitations and the way it could be done on Android

Technical details
Android architecture
The following diagram corresponds to the architecture envisaged on the Android side. Most of it has already been implemented in the ODS project.

Everything in ouds-android is maintained by the Android Core Team.
An app using OUDS Android MUST have a dependency to the core and a dependency to the used theme. It CAN have a dependency to one or several modules provided by ouds-android.
The core module contains basic components that can be used by the app and the OudsTheme which MUST be used in the Compose app to apply the chosen theme.
The OudsTheme uses the theme-contract module to provide semantic tokens for colors, typography, spacings... And it also provides components tokens.
The theme-contract is an interface module. Each created theme MUST implement OudsCustomTheme interface.
Some properties of this interface are mandatory. This is the case of the colors which corresponds to the Material 3 semantic colors tokens. So each theme MUST provide these colors.
Some properties like components tokens have default values directly set in the theme-contract that CAN be overridden or not by a theme.
Each theme like theme-white-label or theme-orange contains their raw values and links them to the semantic tokens defined in the theme-contract.
Each theme CAN also override components tokens by linking them to semantic tokens (and raw values?).
UC1: Developer using the Orange theme (default)

Tokens
Rights

Components tokens
READ thanks to OudsTheme

Semantic tokens
READ thanks to OudsTheme

Raw primitive tokens
None

Developer can't access raw primitive tokens which are in the theme-orange and not exposed.
UC2: Developer creating a completely new theme
To create a new theme the developer MUST create a new module which depends on theme-contract and implements OudsCustomTheme interface.

Tokens
Rights

Components tokens
WRITE by overriding components tokens through OudsCustomTheme interfaceREAD thanks to OudsTheme

Semantic tokens
WRITE semantic tokens by implementing OudsCustomTheme interfaceREAD thanks to OudsTheme

Raw primitive tokens
CREATE inside the created themeWRITE inside the created themeREAD inside the created theme

Note that, on theme creation, overriding components tokens is not mandatory cause theme-contract uses semantic tokens set by the theme as components tokens default values.
UC3: Developer extending an existing theme
As an example, a developer wants to create a theme very close to the Orange theme with some small modifications.
He extends theme-orange and takes advantage of everything already set in this theme. Then, he adjusts the theme as he want.
Rights are similar to UC2.

Tokens
Rights

Components tokens
WRITE by overriding components tokens through OudsCustomTheme interfaceREAD thanks to OudsTheme

Semantic tokens
WRITE semantic tokens by implementing OudsCustomTheme interfaceREAD thanks to OudsTheme

Raw primitive tokens
CREATE inside the created themeWRITE inside the created themeREAD inside the created theme

UC4: Developer needs to create a new semantic token
The creation of a new semantic token requires the intervention of the OUDS Android Core Team on the theme-contract and eventually on the core.
It must be the subject of a detailed written request explaining the use of this future semantic token.

Does the question of demo app be treated in this issue? In ODS demo guidelines we needed to display semantic tokens and their values...


Mod failing after upgrade to version 0.4.12A

I do not currently plan to support Experimental branch updates with the mod. Only when the mod goes to stable will a patch be developed


Slow remove of icon on the top bar

Hey, this is an awesome library I loved it :).
I've added the StackNavigator with a TabNavigator to my app. I got some kind of weird stuff happening. I add an icon on the top right for send me to my create post screen. This one works perfectly etc. But if I change route with the tab and show a route who no supposed to have this icon on the right I got a slow icon remove action. I try to make it null and nothing change. I try also to add the other icon and this last one come on top of my old one.
Do this is normal ? Do I need to change something in my component render ?
This is a GIF I make with this error.

This is my HomeScreen
static navigationOptions = {
    header: ({ navigate }) => {
      const style = { backgroundColor: Colors.redColor };

      const right = (
        <View>
          <Button
            transparent
            onPress={() => navigate('AddMeetup')}
          >
            <Icon
              name="md-add-circle"
              style={{
                fontSize: 30,
                color: Colors.whiteColor
              }}
            />
          </Button>
        </View>
      );

      return { style, right };
    },

This is my ProfileScreen
static navigationOptions = {
    header: {
      style: { backgroundColor: Colors.redColor }
    },
    tabBar: {
      icon: ({ tintColor }) => (
        <MaterialIcons
          name="account-circle"
          size={25}
          color={tintColor}
        />
      )
    }
  }

This is my HomeNavigator who is the tab navigator
import { TabNavigator } from 'react-navigation';
import Colors from '../../constants/Colors';
import {
  HomeScreen,
  NotificationsScreen,
  ProfileScreen
} from '../screens';

export default TabNavigator({
  Home: {
    screen: HomeScreen
  },
  Notifications: {
    screen: NotificationsScreen
  },
  ProfileScreen: {
    screen: ProfileScreen
  }
}, {
  swipeEnabled: false,
  animationEnabled: true,
  tabBarPosition: 'bottom',
  tabBarOptions: {
    showLabel: false,
    showIcon: true,
    inactiveTintColor: Colors.blackBlueColor,
    activeTintColor: Colors.redColor,
    pressColor: Colors.redColor,
    indicatorStyle: { backgroundColor: Colors.redColor },
    style: {
      backgroundColor: Colors.whiteColor
    }
  }
});

My main Navigator
import { StackNavigator } from 'react-navigation';
import { CreateMeetupScreen } from '../screens';
import HomeNavigator from './HomeNavigator';

export default StackNavigator({
  Home: { screen: HomeNavigator },
  AddMeetup: { screen: CreateMeetupScreen }
}, {
  mode: 'modal'
});

Hi @EQuimper thank you for your question.
The reason why this is happening is because animationEnabled: true is set in your TabNavigatorConfig. When you change tabs, the animation starts, which animates the tab's screen from left to right (and vice versa). When the animation is "done," the header will swap to the next screen's header. I bet that if you set animationEnabled: false, your problem goes away. I think this is probably a bug with react-native-tab-view (a dependency of react-navigation https://github.com/react-native-community/react-native-tab-view).
This is not the intended behavior, so this is most likely a bug that should be investigated.

I have this same issue where the title in the header only changes after a noticeable delay when switching tabs.
Interestingly this doesn't happen when navigating to a different tab from the containing DrawerNavigator despite the animation still happening. I suspect this is due to the nested nature of the navigators.

Yeah when you disable animationEnabled for the navigator thoses things do not happen. I have the same for a header that is only visible in one tab and the header is slide in very slowly when the animation is enabled.
It doesn`t look so smooth without the animation, but it is a fast workaround.

I believe this is now gone in master since https://github.com/react-native-community/react-native-tab-view/releases/tag/v0.0.60 contains:

Tapping on tabs will now update the state first before starting the transition (earlier it updated the state after finishing the transition)

I can confirm this is still happening.

This is not the intended behaviors, so this is most likely a bug that should be investigated.

The behaviour changed in latest version, and tapping a tab first updates the state before starting the animation. But this was never a bug.
The reason it was this way is performance. When you update the state and things re-render, the animation can get laggy when it starts. This was especially noticeable because of the JS based animation used by default. Now that we use ViewPagerAndroid/ScrollView, the lag is less noticeable.
Performance reasons aside, a basic principle of the tab view is that when you are using animated tabs, you can have everything animated so that it feels natural. The library makes it easy to do that. Unfortunately, it's not possible with React Navigation's current set of APIs.
Now regardless when the state updates, showing/hiding things controlled by state feels abrupt when all other transitions are animated. Also it's not possible to update the state beforehand when swiping because we don't know what will be the next page until you lift your finger. And even if you lift the finger, you can still swipe again in quick succession. So it's impossible to actually know the next page until the animation completes.
It's simple to workaround for now, make the headers part of individual tab screens.

It's simple to workaround for now, make the headers part of individual tab screens.

@satya164 How could it be solved in this case, to get eliminate the delay in the tapping tabs and in the finger transition? #1230

@agarciaguillo you can make the header part of individual tabs. i.e. stack inside tab rather than tab inside stack.
I think we eventually need to figure out a way to expose animations to the parent navigator. But it's a larger discussion.

@satya164 That way:
const Tabs1 = StackNavigator({
	One: { screen: One, navigationOptions: { title: 'One', header: { title: 'First' } } },
	OneB: { screen: OneB, navigationOptions: { title: 'OneB' } }
});

const Tabs2 = StackNavigator({
	Two: { screen: Two, navigationOptions: { title: 'Two', header: { title: 'Second' } } },
	TwoB: { screen: TwoB, navigationOptions: { title: 'TwoB' } }
});

const Tabs3 = StackNavigator({
	Three: { screen: Three, navigationOptions: { title: 'Three', header: { title: 'Third' } } }
});

const Tabs4 = StackNavigator({
	Four: { screen: Four, navigationOptions: { title: 'Four', header: { title: 'Fourth' } } }
});

const RouterComponent = TabNavigator({
	Tabs1: { screen: Tabs1 },
	Tabs2: { screen: Tabs2 },
	Tabs3: { screen: Tabs3 },
	Tabs4: { screen: Tabs4 }
});

This happens:

The Tab Bar is above off the header, and I want the header to be at the top. The reason of #1230 comes from #981

@agarciaguillo There's one way to achieve what you want, but it involves making your own header, you can do something like this:
tabBarComponent: (props) => (
   <View>
     <TabViewHeader {...props} />
     <TabBar {...props}
   </View>
)

The code for your header can be something like the following:
function TabViewHeader(props) {
  const {
    position,
    navigation,
  } = this.props;
  const { routes } = navigation.state;
  // Prepend '-1', so there are always at least 2 items in inputRange
  const inputRange = [-1, ...routes.map((x: *, i: number) => i)];
  return (
    <View style={styles.header}>
      {routes.map((route, index) => {
        const focused = index === navigation.state.index;
        const scene = { route, index, focused };
        const outputRange = inputRange.map(inputIndex =>
            inputIndex === index
              ? 1
              : 0,
        );
        const opacity = position.interpolate({
          inputRange,
          outputRange,
        });
        return (
          <Animated.Text style={[styles.title, { opacity }]}>
            {this.props.getLabel(scene)}
          </Animated.Text>
        );
      })}
    </View>
  );
}

This code is overly simplistic and I haven't tested it, but it should give you an idea.


How can I refresh the Grid data in the dialog?

How can I refresh the Grid data in the dialog?
I use method in dialog to refresh the Grid data.
AdmMailGrid.prototype.refresh();
But it didn't work.

I use
$(".refresh-button").click();

It works!Thank you very much!

As an alternative if you don't have the refresh button.
In your dialog.ts
Serenity.SubDialogHelper.triggerDataChange(this);
Best Regards


Add the ability to use a static key with [FromState]

[FromState("foo", "bar")] assumes that you are going to look up the key bar in the route values to find the state store key to look up.
We should also provide the option to use this with a static key. If you want to use modeling binding but the key you want to look up is static, there's no good way to do this with [FromState].

@halspang - let's come up with a design before coding this up


Create an abstract class in Knora-base ontology

Allow the creation of abstract classes in  knora-base ontology, for modelling purposes, so that such an abstract class is not displayed as a regular class in the Salsah GUI.
Abstract classes will be used to deal with use cases where we want to use the same property for two different resources while still complying with knora-base:objectClassConstraint and knora-base:subjectClassConstraint.
For example a Commentable abstract resource class is created, for resources that can have a comment.   Commentable has property has comment . The property has comment has knora-base:subjectClassConstraint on Commentable.
We also create Article and Book classes, both being subclasses of Commentable. Thus, Article and Book classes inherit has comment property from their mother-class.
But Commentable is a class that is not meant to be seen by GUI users. At least, it is not as a resource a user should see in the window "add resource".

I think both classes and properties should have an optional isAbstract annotation property, which would tell Knora not to instantiate them directly.
I think we could also write GraphDB consistency rules to check this.
Currently some of this is hard-coded. There's a list of abstract resource classes in OntologyConstants.KnoraBase.AbstractResourceClasses, and the ontology responder also has some code that knows about abstract classes and properties.

So this would be like an abstract class or an interface in Java which cannot be instantiated itself (only by subclassing it)?

Yes, that's the idea.

I think it is a good idea to put that information in the ontology. Then the scala code wouldn't have to be adapted once a new abstract resource class is added. We could simply ignore these resources (evaluating isAbstract) in the sparql queries used by ontology responder.

Yes, and in the Scala code, we could also check query results to see if we got an abstract class or property (by asking the ontology responder), and if so, we could throw a nice exception.

So in the end when some tries to instantiate an abstract resource class, the ontology responder would not find anything about it?
E.g., someone wants to create a resource of an abstract type, and the resources responder would ask the ontology responder about the cardinalities of this class. Then the ontology responder would not find this class because isAbstract is true and thus it was ignored.

I think the ontology responder should know about all the abstract classes and properties. When the resources responder asks for information about a resource class, the ontology responder could check whether it's an abstract class, and if so, it could throw a special exception like AbstractClassException, which would be propagated back to the resources responder. That would be better than NotFoundException.

Right, that seems fine to me.
More generally, could we aim at making all knora-base resource classes abstract? To  my understanding, entities defined in knora-base are very generic so they do not "mean" anything.
Still we can make ontologies defining basic concepts that can be reused among different projects (like for the bibliography).
I am thinking about creating a base set of standoff tags in a special ontology and move them out of knora-base. Only the the very basic concepts of standoff would remain there (StandoffTagV1 and the data type standoff tags).

More generally, could we aim at making all knora-base resource classes abstract? To my understanding, entities defined in knora-base are very generic so they do not "mean" anything.

We have to decide which ones should be abstract: #138.

Still we can make ontologies defining basic concepts that can be reused among different projects (like for the bibliography).

Yes, after they go through some kind of standardisation process.

I am thinking about creating a base set of standoff tags (like for the TEI) in a special ontology and move them out of knora-base. Only the the very basic concepts of standoff would remain in knora-base (StandoffTagV1 and the data type standoff tags).

Sounds OK to me.

I think we might also be able to make GraphDB consistency rules like this:
Consistency: direct_use_of_abstract_class
    i <rdf:type> t [Context <onto:explicit>]
    t <knora-base:isAbstract> true
    ------------------------------------

Consistency: direct_use_of_abstract_property
    i p j [Context <onto:explicit>]
    p <knora-base:isAbstract> true
    ------------------------------------

Ok, I moved all non system standoff information to a dedicated ontology: https://github.com/dhlab-basel/Knora/pull/319/commits/35a603f288dbe00acf6678b4076bf62088169758.
I first made a few mistakes when copying the ontology information to another file because I forgot to change some of the : prefixes to knora-base.
We need to find a way to check an ontology for unresolvable references. Moreover, I could instantiate standoff tags that were unknown to the application because I did not load standoff-onto.ttl in the tests yet.

We need to find a way to check an ontology for unresolvable references

Check ontologies for validity: #286

This issue has been inactive for a significant time, so we’re closing it for now. If it’s still relevant, feel free to reopen.


Update nf-winbase-getfirmwaretype.md

FirmwareType is an output parameter to retrieves the result, right?

@Karl-Bridge-Microsoft Could you please take a look? I believe it's wrong.

Hi @RatinCN - SAL in definition specified as:
WINBASEAPI
BOOL
WINAPI
GetFirmwareType (
Inout PFIRMWARE_TYPE FirmwareType
);


⚠ DesiTorrents has degraded performance

In ed1bec2, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 2539 ms

Resolved: DesiTorrents performance has improved in 6104a21.


Feature: Striped Tables

Describe the feature in detail (code, mocks, or screenshots encouraged)
It's quite common to striped/zebra table rows, it would be nice if this was built into this component.
What type of pull request would this be?
None
Provide relevant links or additional information.
No response

This project tries to stay aligned with the original, if table variations are added to that, then we can revisit, but for now this is unplanned as it can easily be added by passing a different class for the background color into the even/odd rows of your table in your local component.
When considering things like this, I think about how much of a PITA is this to implement for the dev/user locally, if it's simple, then adding it just adds bloat to the project that everyone who adds the component may not ever want/need.


events_pi concat logic

originally shows up here: https://github.com/jllanfranchi/pisa/blob/37f168b2d98bd51d8ab90a66c06a80e1a179ef40/pisa/core/events_pi.py#L309-L315
where the np.append was being done, but that will operate on a dict but you can't np.concat dicts. I think whoever wrote this got confused (and confused me) by iterating over keys, values and calling them subkey, subval. Should iterate over key, val and then within each val iterate over subkey, subval and then the arrays live under there and so concatenation can be done.
In the latest commit, this now shows up at
https://github.com/jllanfranchi/pisa/blob/d356c92b9c96495cb3e0044ca44f277540b76697/pisa/core/events_pi.py#L413-L414
and
https://github.com/jllanfranchi/pisa/blob/d356c92b9c96495cb3e0044ca44f277540b76697/pisa/core/events_pi.py#L431-L434

I believe this bug currently affects

https://github.com/icecube/pisa/blob/master/pisa/core/events_pi.py#L654

can we ever even reach this part of the code given that output_data is created as an empty dict a few lines above?

https://github.com/icecube/pisa/blob/master/pisa/core/events_pi.py#L672

looks like we should iterate over keys in sub_data in addition (which could be something like 'true_energy' or 'weighted_aeff')

https://github.com/icecube/pisa/blob/master/pisa/core/events_pi.py#L700

It doesn't seem to affect anyone in practice, though? Perhaps just assert that the output_data doesn't already contain a given key (describing (anti)neutrino flavour and interaction type) as a quick fix until someone really needs the concatenating logic?


:bug:x.x.o: Fix cache

Codecov Report

Merging #2201 (5480df4) into main (78da4c8) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##              main     #2201   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          127       130    +3
  Branches        14        14
=========================================
+ Hits           127       130    +3

Files Changed
Coverage Δ

...i/domain/item/edit/presenter/hook/itemEdit.hook.ts
100.00% <100.00%> (ø)

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


Authentication issue when archiving

As Twitter no longer allows anonymous API access, you'll need to enter the auth details of any account to use for archiving. These will be stored in /Users/abhinav/.ebooksrc if you need to change them later.

How do you work around this?

Have to enter the same keys as in your bots.rb the first time you run archive. Misinterpreted this as having to authenticate the user whose tweets you're trying to archive.


Plugin error after all tests are run

Describe the bug
After running all the tests using pytest, a report file is generated, but a plugin error is thrown in certain cases, preventing the fault localization from working.
To Reproduce

Download source of httpie 2.6.0
Open as PyCharm project
Install all required packages in the used venv
Click 'CharmFL', then select Run

Expected behavior
CharmFL runs without an error, and test metrics are viewable in the IDE.
Screenshots

Desktop (please complete the following information):

OS: Debian 10
PyCharm version: 2021.2
CharmFL version: v1.0.4

Additional context

Problem occurs with any combination of CharmFL settings.
There are less test entries in the generated report than in the IDE when running pytest manually.
report.xlsx

This is no longer an issue, since we do not use the openpyxl anymore


[Bug]: Same UI crash bug as listed by other users but on different OS

Operating System
Windows 10
BG3 Mod Manager Version
<IP_ADDRESS>
BG3 Game Version
Full release, Patch 7
Bug Summary
This bug has been posted by someone on Windows 11, in July.
I am experiencing the same crash, and I'm submitting the log below:
On opening the manager, this crash message is displayed:

the log it generates is this:
release_08-09-2024_17-04-41.log
The most recent post of this bug is listed here https://github.com/LaughingLeader/BG3ModManager/issues/344
This bug was listed as not replicable last year, but as it's persisted clearly something is happening, no logs were posted in the tickets I saw, so I hope this helps a bit :)
Links
No response

I am having the exact same problem, only happening on the patch 7 update the previous versions were fine.


Unexpected token '.'

Hi,
I was searching about for a way to avoid the SSR issue using react and AEM, so nextjs seems a good way to fix it.
I found your repository and I'm trying to create a little POC, using this template, to see if it will work for me.
I had some issues with aem-react-editable-components with this template, just following the tutorial.
So, following the issue report template:
Expected Behavior
Following the readme, I ran:
npm run dev

and the example page should be shown, with the text component (attached below)
Current Behavior
An aem-react-editable-components error is shown
Reproduce Scenario (including but not limited to)
I just followed the readme tutorial
Platform and Version
Latest AEM SDK version
Latest Wknd Version
Node v12.17.0 (NPM 6.14.4)
Windows 10
Logs taken while reproducing problem

Also, follow attached the error.log from the "npm run dev"
error.log

Hi!
It's working! It was the node version, I tried with v14.20.0, and worked!


Towards v5.1

update docs with a getting-started page
closes #3748 by updating and simplifying the iOS instructions
drop Docker image for ARMv7 - the base image is deprecated, and ARMv7 is only the "old" RaspberryPis, which are becoming less and less popular.

We have so far kept everyone from @mitmproxy/devs out of the changelog. Do we want to continue doing that, or shall we just name authors for everything? I'm really indifferent on that, just pointing out the change. :-)
Huge 👍 on the ARMv7 images, they greatly add onto CI time.

Yes - I thought a bit about mentioning everyone or just non-core-team... I have no strong feelings either. I Looked at the changelog and it felt odd that a few items had names, and other didn't.
Ideally for the next release, I would like to try and keep the changelog "always up2date", instead of having to review 100s of PRs and commits before the next releases.


Naming in article

Giving pvc name ("azurefile") the same as pv name is misunderstood the context of this example.

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

ID: bdb3a253-ff94-1b97-62c9-e21153598452
Version Independent ID: b95ca0b2-e809-a149-dec7-a15ac2146ee1
Content: Create a persistent volume with Azure Files in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/azure-csi-files-storage-provision.md
Service: container-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@PedroEss
Thanks for your feedback! We will investigate and update as appropriate.

@PedroEss
Could you please send the screenshot you exactly facing the issue in the doc.


Tapo C100 Support

Hello,
i saw smth about C100 in source code... Is it supportted, does it have rstp and other functions like c200?
Thanks!

I don't own c100 but people did report success with c100.
Try it out and let me know if there are any issues!


bridge hotunplug, e2e flake: test may miss the stage when VMI interface Absent

What this PR does / why we need it:
Test [sig-network] nic-hotunplug a stopped VM cannot be subject to **hot** unplug, but will mutate the template.Spec on behalf of the user may be flaky and sometimes have a panic.
The test is waiting for Absent interface to appear in the VMI spec after it was aded to the VM template spec.
However, since the Absent interfaces are eventually removed from the spec, a condition waiting for the VMI interface to be Absent may be flaky.
Fixing the test to wait for the VMI interface to be Absent or to be removed from the spec.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test pull-kubevirt-e2e-k8s-1.26-sig-network-multus-v4

/cherry-pick release-1.0

/kind flake


Question

Can i remove something from my saved hotbars using nbtexplorer? I saved one of antvenom's stacked chests there and now opening the menu crashes the game

Yes.
You can view the data for your saved hotbars in the hotbar.nbt file in your .minecraft folder.


Question: What drives up the value of a "waifu" ?

Had alot of questions on my server that's using Nadeko.
What is it that drives up a particular persons "value" , when trying to .claim them?
Can you disclose the formula ?
Thanks for the ongoing support.

✔ smart
✔ nice personality
✔ pretty
✔ loli
Other attributes may differ according to taste, but these ones are objective, and are crucial for a top-quality expensive waifu :^)

:*)


BE crashed when running a long term sql

Steps to reproduce the behavior (Required)
OS: centos7.4
MEM: 32G
CPU: 16C
set global parallel_fragment_exec_instance_num = 4;
set global exec_mem_limit =<PHONE_NUMBER>;
Running SQL:
select
i_item_id
,i_item_desc
,s_store_id
,s_store_name
,stddev_samp(ss_quantity)        as store_sales_quantity
,stddev_samp(sr_return_quantity) as store_returns_quantity
,stddev_samp(cs_quantity)        as catalog_sales_quantity
from
store_sales
,store_returns
,catalog_sales
,date_dim             d1
,date_dim             d2
,date_dim             d3
,store
,item
where
d1.d_moy               = 4
and d1.d_year              = 1998
and d1.d_date_sk           = ss_sold_date_sk
and i_item_sk              = ss_item_sk
and s_store_sk             = ss_store_sk
and ss_customer_sk         = sr_customer_sk
and ss_item_sk             = sr_item_sk
and ss_ticket_number       = sr_ticket_number
and sr_returned_date_sk    = d2.d_date_sk
and d2.d_moy               between 4 and  4 + 3
and d2.d_year              = 1998
and sr_customer_sk         = cs_bill_customer_sk
and sr_item_sk             = cs_item_sk
and cs_sold_date_sk        = d3.d_date_sk
and d3.d_year              in (1998,1998+1,1998+2)
group by
i_item_id
,i_item_desc
,s_store_id
,s_store_name
order by
i_item_id
,i_item_desc
,s_store_id
,s_store_name
limit 100;
Expected behavior (Required)
Timeout or normally end
Real behavior (Required)
BE Crashed,
StarRocks version (Required)
Version: 1.18.2
be.out
start time: Thu Sep 16 10:39:14 CST 2021
I0100 00:00:00.000000 13257 vlog_is_on.cc:192] RAW: Set VLOG level for "*" to 10
terminate called recursively
terminate called recursively
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1631762146" if you are using GNU date ***
PC: @     0x7f95efdba387 __GI_raise
*** SIGABRT (@0x33c9) received by PID 13257 (TID 0x7f95b3f8a700) from PID 13257; stack trace: ***
@          0x3093812 google::(anonymous namespace)::FailureSignalHandler()
@     0x7f95f1b2f630 (unknown)
@     0x7f95efdba387 __GI_raise
@     0x7f95efdbba78 __GI_abort
@          0x13ca533 _ZN9__gnu_cxx27__verbose_terminate_handlerEv.cold
@          0x3c29f36 __cxxabiv1::__terminate()
@          0x3c29fa1 std::terminate()
@          0x3c2a0f5 __cxa_throw
@          0x13c9412 _ZN12_GLOBAL__N_110handle_oomEPFPvS0_ES0_bb.cold
@          0x3cdaa7e tcmalloc::allocate_full_cpp_throw_oom()
@          0x14ff02b doris::vectorized::FixedLengthColumnBase<>::append()
@          0x1519476 doris::vectorized::NullableColumn::append()
@          0x25fb6d2 doris::vectorized::CrossJoinNode::_build()
@          0x25fdc64 doris::vectorized::CrossJoinNode::open()
@          0x2509335 doris::vectorized::HashJoinNode::open()
@          0x2509335 doris::vectorized::HashJoinNode::open()
@          0x2509335 doris::vectorized::HashJoinNode::open()
@          0x2509335 doris::vectorized::HashJoinNode::open()
@          0x2509335 doris::vectorized::HashJoinNode::open()
@          0x2509335 doris::vectorized::HashJoinNode::open()
@          0x24c434d doris::vectorized::AggregateStreamingNode::open()
@          0x1f3f0e8 doris::PlanFragmentExecutor::_open_internal_vectorized()
@          0x1f40751 doris::PlanFragmentExecutor::open()
@          0x1edb5af doris::FragmentExecState::execute()
@          0x1edcba0 doris::FragmentMgr::exec_actual()
@          0x1ee293b std::_Function_handler<>::_M_invoke()
@          0x2026389 doris::ThreadPool::dispatch_thread()
@          0x2034a05 doris::Thread::supervise_thread()
@     0x7f95f1b27ea5 start_thread
@     0x7f95efe829fd __clone

It is OOM.
You can set echo 1 > /proc/sys/vm/overcommit_memory to allow overcommit.

We plan to use ‘try catch’ to prevent this std::vector rash from being unable to apply for virtual memory.
Current can echo 1 > /proc/sys/vm/overcommit_memory to allow overcommit virtual memory

Yes, I have reduce the parameter of exec_mem_limit to avoid OOM.


GHES: Rename any branch of an existing repository

Summary
Many communities, both on GitHub and in the wider Git community, are considering renaming the default branch name of their repository from master. We're making some updates to GitHub to make that process easier.
How will it work?
You'll be able to rename any branch, including the default branch, of an existing repository. The rename feature will take care of retargeting open pull requests and open releases, and will show your contributors a message to help them update their local copy of the repository after the rename.

Learn more at github/renaming

https://github.blog/changelog/2021-01-19-support-for-renaming-an-existing-branch/
Shipped with GHES 3.1


Automatically switch chain to base

What changed? Why?
Notes to reviewers
How has it been tested?

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

Review Error for omkarb @ 2024-07-26 13:26:38 UTC
User failed mfa authentication, see go/mfa-help


Installation steps MySQL 8.0.13

Could you please let us know the steps for below configuration:
MySQL 8.0.13 (older version MySQL not working macOS Mojave)
shiro.ini
ds = com.mysql.cj.jdbc.MysqlDataSource
ds.serverName = localhost
ds.user = root
ds.password = root
ds.databaseName = datagen
application properties:
Persistance Details
jdbc.driverClassName = com.mysql.cj.jdbc.MysqlDataSource
jdbc.url = jdbc:mysql://localhost:3306/datagen
jdbc.username =root
jdbc.password =root

Hi Mishra,
We tried with the latest mysql version 8.0.16 and successfully logged into the application and able to generate data. Requesting to verify the configurations again.
Following are the configuration changes done for the latest mysql version check.
Maven dependency
<dependency>     <groupId>mysql</groupId>     <artifactId>mysql-connector-java</artifactId>     <version>8.0.16</version> </dependency>
application.properties
jdbc.driverClassName = com.mysql.cj.jdbc.MysqlDataSource
shiro.ini
ds = com.mysql.cj.jdbc.MysqlDataSource

Needed to use below URL:
http://localhost:1400/datarapid-1.0/index.html
http://localhost:1400/datarapid-1.0/ -- Old (not working)


MI - Deployment Topics

This Epic is to track the issues related to deployment topics for the Micro Integrator. See the attached issues for details on individual topics.

Done


iOS 12 issue in variable.swift file lookup function

facing above screenshot issue in iOS 12 ,  Xcode 10 GM,
I have installed 0.12.1 version of Stencil project.

@mkBajaj can you provide an example project? I can't reproduce this locally.

Closing this due to inactivity.
@mkBajaj please reopen this if you have more information. Also please first test using the latest Stencil release.


NEW: use hashed keys by default in the @cached decorator

make_default_key returns a key of variable length which depends on stringified representation of the arguments. That may be undesirable, and it wouldn't hurt to hash the representation by default.

Making assumptions here, and also nit picking a bit, but the first usage of a library like this one would be developing locally right? To try out the API and understand some of the concepts behind it, the make_default_key is a better option for this scenario, as you might be checking keys directly on your redis process/container running locally and I believe a more human readable format is preferable.
I think it's nice to have the hashed option available as an example of how you could customize the way keys are made, but I'm not sure if I would have it as the default. What do you think?

Yeah, you make a good point.
Think, I'll amend it: just add the function, update the docs and leave the defaults


Constrain commands to the Redis server to the bare minimum: AUTH, GET, SET

In the server configuration we use, the Redis user used to read and write the ctcache stuff has only minimal permissions. By default redis-py always sends some CLIENT SETINFO ... commands to the server. In our case this results in errors. I think these commands can be avoid in case of ctcache.
Of course, this could be made configurable, but I guess it's favorable to not overwhelm the user by configuration options. Therefore, I think this would make a good default.

Merged, thanks for the PR


Add --dart-define option support to build aar command

Description
Add --dart-define option support to flutter build aar command
Related Issues
https://github.com/flutter/flutter/issues/71827

@googlebot I signed it!

Sorry for late, I will add some test code for this PR on this weekend.


Unrelated picture in document

Under the Govern section an unrelated picture snuck in.
| Cost Management discipline template | Define your policy statements and design guidance  the cloud governance in your organization. Focuses on cost management. |

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

ID: 82d6acd7-52c2-0fcd-77ad-a30e96ecbf45
Version Independent ID: af35206e-2b05-0ae7-8ca9-6e43330e96e6
Content: Tools and templates - Cloud Adoption Framework
Content Source: docs/resources/tools-templates.md
Service: cloud-adoption-framework
Sub-service: general
GitHub Login: @JanetCThomas
Microsoft Alias: janet

Image was removed by @janicericketts


Example for GetAccountHierarchy?

I have a manager account and I stumbled on this old c# example.
https://stackoverflow.com/questions/13832008/google-adwords-api-c-sharp-client-library
Is this still possible in the current API? If so, could you add an updated example please.

Yes, this is possible using CustomerManagerLinkService, but I don't have a code example yet. Let me try putting together one.

That would be awesome. Thanks @AnashOommen

Thanks @AnashOommen . we see the ListAccessibleCustomers  method is available but also need to get the account hierarchy to display accounts in a structured way.

Folks,
The basic logic right now is,

If your login email has access to multiple customers, then use
ListAccessibleCustomers to get all such customers. Pick one of those
accounts as your LoginCustomerId. For most users, there will be only one
such customer, so you can skip this step and use LoginCustomerId rightaway.

You retrieve CustomerClient to get all customer ids under a manager
account.

Call CustomerService.get for each such customer id.

This is not very efficient for larger account trees, so we are working on
making that process faster and more efficient for common use cases.
Tx
Anash
*From: *bejan<EMAIL_ADDRESS>*Date: *Sat, May 11, 2019 at 8:38 PM
*To: *googleads/google-ads-dotnet
*Cc: *Anash P. Oommen, Mention
Thanks @AnashOommen https://github.com/AnashOommen . we see the
ListAccessibleCustomers

https://developers.google.com/google-ads/api/docs/account-management/listing-accounts
method is available but also need to get the account hierarchy to display
accounts in a structured way.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googleads/google-ads-dotnet/issues/24#issuecomment-491554524,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCOJCBVZD2W52CFJOOFP4DPU5RIXANCNFSM4GKG4XEQ
.

--
Anash P. Oommen | Developer Programs Engineer |<EMAIL_ADDRESS>|
+1-650-440-2854

This worked for me along with the customer_client_link resource to build a link list and derive the hierarchy. Thanks again, @AnashOommen

Would love to see some code samples 😏

When using the GetCustomerClientRequest for the CustomerClientService what does the part_1 represent in the resource name?
var request = new GetCustomerClientRequest()             {                 ResourceName = "customers/{customer_id}/customerClients/{part_1}"             };

When using the ListAccessibleCustomers call it only returns the customer id of the manager account I have set as the LoginCustomerId in the header for auth.
I have looked through the docs on page https://developers.google.com/google-ads/api/reference/rpc/google.ads.googleads.v1.services#customerservice and I am not getting the specified behavior.
When calling the getCustomer endpoint for the managing account, same as the LoginCustomerId in the auth header it returns the data and confirms that it is the managing account. Am I missing something here, I don't understand why ListAccessibleCustomers isn't returning all the managed account resource names?

@bbatton Yes, only the manager and maybe some non-MCC linked accounts are returned by ListAccessibleCustomers. Next we need to get the customer list of that manager using the customer_client resource, then call GetCustomer for each of these customers, for example -
            var customerServiceClient = adsClient.GetService(Services.V1.CustomerService);
            var adServiceClient = adsClient.GetService(Services.V1.GoogleAdsService);
            var customerIdList = customerServiceClient.ListAccessibleCustomers().Select(s => s.Replace("customers/", ""));

            foreach (var customerId in customerIdList) {

                //get customer list for this customer
                adServiceClient.ServiceContext.Client.Config.LoginCustomerId = customerId;
                var query = @"SELECT customer_client.resource_name, customer_client.client_customer, customer_client.level, customer_client.hidden FROM customer_client WHERE customer_client.hidden = FALSE";

                var customerClientResults = adServiceClient.Search(customerId, query);

                foreach (var customer in customerClientResults) {

                    var clientCustomerId = customer.CustomerClient.ClientCustomer.Replace("customers/", "");

                    try {
                        Thread.Sleep(500);//otherwise error, too many API calls/minute; we need a better way!

                        //get this customer data
                        var customerData = customerServiceClient.GetCustomer(ResourceNames.Customer(long.Parse(clientCustomerId)));

Complete, see here.


pre-commit, erd, and initial terraform script

This looks good, any questions or things for me to check specifically?

I added pip-tools/pip-compile to the pre-commit, do you think it's worthwhile or is that overkill?

Also some things to keep into account with this:

those are good considerations. I think I'm going to remove it.


add transparency/opacity property for items in User Interface

I think that adding a property for the transparency/opacity for some widget would be useful. Googling around a found a few solution like this video (note that it has more then 13.000 views so I guess that many people wanted to do this) or this article.
Button, check box, label, slider, switch ecc should have this property

@ewpatton I could take this issue up and implement the property. A few concerns, however:

What should we do with the existing selection of the opacity property for the color picker in Designer? Should we then only make the opacity property as a block-only property, since we may already use the color picker in Designer to select transparent colors?
Should we implement this for all of the AndroidVisibleComponents, or are there cases where the opacity property would not make sense? (I figure this could be fixed later on by property hiding for the individual components)
Are we concerned with browser alpha value support with regards to Designer? If Internet Edge is ever officially supported, there are some issues there with regards to the alpha color channel.

@lightingft

What should we do with the existing selection of the opacity property for the color picker in Designer? Should we then only make the opacity property as a block-only property, since we may already use the color picker in Designer to select transparent colors?

We could have it so that if they pick a color with an alpha we update the Opacity property accordingly. When opening the picker, set the alpha to the value of Opacity.

Should we implement this for all of the AndroidVisibleComponents, or are there cases where the opacity property would not make sense? (I figure this could be fixed later on by property hiding for the individual components)

It probably makes sense for most AndroidVisibleComponents except for WebViewer, Map, and VideoPlayer. Basically, any component that has a BackgroundColor property should probably have an Opacity property as well.

Should we implement this for all of the AndroidVisibleComponents, or are there cases where the opacity property would not make sense? (I figure this could be fixed later on by property hiding for the individual components)

The designer will only ever be an approximation, so if some browsers struggle with applying opacity I think that shouldn't be a blocker here. Edge appears to be moving to using Chromium, so I expect this will be even less of a problem.

@ewpatton Cheers for the answers. I agree to having the Opacity color update with the color picker as well. Then I suppose we can have both the BackgroundColor and the opacity property in the Designer properties.
Another point of interest is whether the value should be in percentage (integer from 0 to 100), or in an alpha value format (integer from 0 to 255)? I would personally think that having the value in percentage would be somewhat more user friendly for the general user. Thoughts on this?


habitat: 0.90.6 -> 1.1.652, fix build, unpin openssl

Diff: https://github.com/habitat-sh/habitat/compare/0.90.6...1.6.652
Changelog: https://github.com/habitat-sh/habitat/blob/1.6.652/CHANGELOG.md
Description of changes
ZHF https://github.com/NixOS/nixpkgs/issues/230712
unpin openssl https://github.com/NixOS/nixpkgs/issues/210452
closes #203070

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, 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/)
23.05 Release Notes (or backporting 22.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.

@ofborg eval


HDDS-6814. ozone s3 getsecret default --om-service-id

What changes were proposed in this pull request?
For ozone S3 commands, instead of having --om-service-id as a mandatory option, use the serviceID from the config file as the default if there is only one. Otherwise, Make the option a compulsory one.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6814
How was this patch tested?
Added a Robot test for the change.

@Tejaskriya Thanks for updating patch, LGTM +1

Thanks @Tejaskriya for the patch, @ashishkumar50, @LZD-PratyushBhatt for the review.


Mystery 'Unexpected token {' issue

I'm trying to get the update a collection: I've mongoexport'd the data to a file like so:
{"__v":0,"_id":{"$oid":"5549475f0b1fb36c1e205664"},"description":"Box of eggs","imageURL":"http://www.beingabusinesscelebrity.com/wp-content/uploads/2013/06/Sometimes-it-looks-like-an-eggbox.jpg","name":"Eggs","price":1,"quantity":"6"}
{"__v":0,"_id":{"$oid":"55494a6d0b1fb36c1e205665"},"description":"Beef Burgers","imageURL":"http://www.roamingroosters.co.uk/images/uploads/grass_fed_beef/roaming-roosters-beef-burgers.jpg","name":"Burgers","price":1.95,"quantity":"2"}
etc....

seeds.json looks like:
{         "undefined": "localhost/kitchenapp",         "dev": "localhost/kitchenapp-dev",         "prod": "localhost/kitchenapp"     }
I run seed from the command line and get:
Jamess-MacBook-Pro:ka2 jamessherry$ seed
[Error: Module version mismatch. Expected 46, got 14.]
js-bson: Failed to load c++ bson extension, using pure JS version
Seeding files from directory /Users/jamessherry/sites/ka2/seeds
----------------------
Seeding collection ingredients
[SyntaxError: /Users/jamessherry/sites/ka2/seeds/ingredients.json: Unexpected token {]

Has anyone encountered this?
Seed.json is generated to begin as a config object, and even if you turn ingredients.json into a JSON collection (as opposed to what mongoexport outputs) it still throws the same error...

Can you post your ingredients.json file?

Or can you start with a very simple file and see if that seeds. Then keep adding more data until you narrow it down to some specific data?

Hi @toymachiner62 , The first snippet of code is the first few lines of the ingredients.json
If I take it down to one object in that file, I get:
[Error: Seed file recipes does not start with an Array]
[Error: key $oid must not start with '$']

Does this means that I can't mongoexport into a file and then use that file with this package? Would I have to reformat it's output somehow?

Yes so you have 2 issues:

Your json file needs to be an array so with your example content, if you just put them in an array like this:

[
  {"__v":0,"_id":{"$oid":"5549475f0b1fb36c1e205664"},"description":"Box of eggs","imageURL":"http://www.beingabusinesscelebrity.com/wp-content/uploads/2013/06/Sometimes-it-looks-like-an-eggbox.jpg","name":"Eggs","price":1,"quantity":"6"}
  {"__v":0,"_id":{"$oid":"55494a6d0b1fb36c1e205665"},"description":"Beef Burgers","imageURL":"http://www.roamingroosters.co.uk/images/uploads/grass_fed_beef/roaming-roosters-beef-burgers.jpg","name":"Burgers","price":1.95,"quantity":"2"}
]

It will solve your first error.

I'm guessing that you cannot insert into mongo json that contains a $ because mongo uses them for it's functions. I'm not 100% sure on that, but that's my gut feeling. You can verify by trying to manually insert something into mongo that contains a $.


choreTroubleshooting the cause of slow CI execution

Proposed changes
Just CI execution.
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.05 seconds
stream load tsv:          505 seconds loaded<PHONE_NUMBER>9 Bytes, about 141 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.4 seconds inserted 10000000 Rows, about 340K ops/s
storage size:<PHONE_NUMBER>7 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.15 seconds
stream load tsv:          536 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.7 seconds inserted 10000000 Rows, about 336K ops/s
storage size:<PHONE_NUMBER>4 Bytes


rpm missing requirement on cffi package

Describe the bug

To Reproduce
Install azure-cli-2.7.0-1.el7.x86_64.rpm and the jq rpm in a fedora 32 container. Try to execute the following steps to create a storage container:
# log in
az login

az_region="eastus2"
az_resource_group="foo-20200608"
az_storage_account="foostorage20200608"
az_container="foo-storage-container-20200608"

# Create resource group
az group create -l "${az_region}" -n "${az_resource_group}"
# Create storage account for uploading FCOS image
az storage account create -g "${az_resource_group}" -n "${az_storage_account}"
# Retrieve connection string for storage account
cs=$(az storage account show-connection-string -n "${az_storage_account}" | jq -r .connectionString)
# Create storage container for uploading FCOS image
az storage container create --connection-string "${cs}" -n "${az_container}"

It fails with:
The command failed with an unexpected error. Here is the traceback:

'NoneType' object has no attribute '__name__'
Traceback (most recent call last):
  File "/usr/lib64/az/lib/python3.6/site-packages/knack/cli.py", line 215, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 654, in execute
    raise ex
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 718, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 709, in _run_job
    cmd_copy.exception_handler(ex)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/__init__.py", line 257, in new_handler
    raise ex
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 688, in _run_job
    result = cmd_copy(params)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 325, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/__init__.py", line 545, in default_command_handler
    return op(**command_args)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/operations/blob.py", line 40, in create_container
    client = blob_data_service_factory(cmd.cli_ctx, kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/_client_factory.py", line 80, in blob_data_service_factory
    return generic_data_service_factory(cli_ctx, blob_service, kwargs.pop('account_name', None),
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/_client_factory.py", line 36, in generic_data_service_factory
    return get_storage_data_service_client(cli_ctx, service, name, key, connection_string, sas_token,
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/_client_factory.py", line 27, in get_storage_data_service_client
    return get_data_service_client(cli_ctx, service, name, key, connection_string, sas_token,
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/client_factory.py", line 178, in get_data_service_client
    logger.debug('Getting data service client service_type=%s', service_type.__name__)
AttributeError: 'NoneType' object has no attribute '__name__'

This happens because some modules are dynamically loaded and if their deps aren't met
they silently fail. If I change these lines to add checked=False then I see:
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/sdkutil.py", line 35, in get_blob_service_by_type
    print(get_sdk(cli_ctx, ResourceType.DATA_STORAGE, 'BlockBlobService', mod='blob', checked=False))
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/profiles/__init__.py", line 90, in get_sdk
    return _sdk_get_versioned_sdk(cli_ctx.cloud.profile, resource_type, *attr_args, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/profiles/_shared.py", line 528, in get_versioned_sdk
    loaded_obj = _get_attr(sdk_path, mod_attr_path, checked)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/profiles/_shared.py", line 492, in _get_attr
    raise ex
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/profiles/_shared.py", line 483, in _get_attr
    op = import_module(full_mod_path)
  File "/usr/lib64/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/multiapi/storage/v2018_11_09/blob/__init__.py", line 6, in <module>
    from .appendblobservice import AppendBlobService
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/multiapi/storage/v2018_11_09/blob/appendblobservice.py", line 30, in <module>
    from ._deserialization import (
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/multiapi/storage/v2018_11_09/blob/_deserialization.py", line 41, in <module>
    from ._encryption import _decrypt_blob
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/multiapi/storage/v2018_11_09/blob/_encryption.py", line 13, in <module>
    from cryptography.hazmat.primitives.padding import PKCS7
  File "/usr/lib64/az/lib/python3.6/site-packages/cryptography/hazmat/primitives/padding.py", line 13, in <module>
    from cryptography.hazmat.bindings._padding import lib
ModuleNotFoundError: No module named '_cffi_backend'

A sudo dnf install python3-cffi fixes the problem. I think we should add the python3-cffi rpm to our list of deps for the rpm and also maybe we should detect this error for dynamically loaded modules better in the future (should we make checked=False the default)?
Also is there anyone who would want to submit azure-cli for inclusion in Fedora by default? I can help mentor someone if they would like to do that. It would make it much easier for new users to get started with the CLI.
Expected behavior
No errors.
Environment summary
RPM

hi @houk-ms could you pls have a look? thanks

@dustymabe Thanks for your reporting and investigation. As you have found, this is an OS specific issue, which we recorded in the troubleshooting before a better solution is decided.
Your suggestion about dynamically loading modules and better deal with the checked keywarg is useful. We will investigate it and see if we can find a better way.

Close the issue now.


Soundcloud Links Have Problem to Play

The first time I used the bot can play the music from soundcloud's links. However, after a few months later, I used the bot to play the soundcloud, and I unable to do it, but not for youtube's links.

Most likely caused by https://github.com/sedmelluq/lavaplayer/issues/651

fixed by #255


RichCommand: Glyph + ContextMenu + EmptyRecycleBinAction

Resolved / Related Issues

Usage of glyphs in IAction. This simplifies the use of icons with the FontIcon and ColoredIcon properties of IRichCommand.
Simplifies the creation of context menus. It is possible to pass a RichCommand to ContextMenuFlyoutItemViewModel to initialize it.
Added RichCommand EmptyRecycleBin, which uses glyphs. This command is used everywhere, including the sidebar. Old commands are deleted. This new action does not yet use the IsExecutable property. This requires the context class and will be the subject of a future pr.
A behavior has changed slightly. In the context menu of the RecycleBin page, the RecycleBin command is no longer active if the RecycleBin is empty. This seemed like a bug to me.
A bug has been fixed in the parser of HotKey (not yet used).
The next step is to add the context. This is the last part of the command base. Then, it will be necessary to convert the current commands (actions + use in the views).

Validation
How did you test these changes?

[ ] Built and ran the app
[ ] Tested the changes for accessibility

Can you add support for OpacityIcon? We're currently only using them in the toolbar but we're going to want to use them in the context menu as well.

I haven't looked at this class but probably it's possible. I will watch for the next iteration.

@cinqmilleans please fill out validation steps and add any other steps you used. This helps us know what to test for when reviewing the PR.


Fix assertion order for tests which include a description string

description strings are the last argument to wru, the first argument is the assertion being made
This should have some failing tests as I know this polyfill does not stop implicit string coercion of symbols.

I'd really like to swap the test runner for tap or something similar that the other es-shims projects use

Yes, we should absolutely switch to tape. I’d be happy to review a PR that made that conversion :-)

This has been made obsolete by #32, thanks!


More frame events

add the code to fire frame info events back in
move the code to fire events from a closure into a method (work around https://github.com/dart-lang/sdk/issues/30522)

@Hixie, fingers crossed, I think this is ready to land


Missing 'adf.structures.base?

In the wiki it says:
Add the dependency to your module:
angular.module('sample', ['adf', 'adf.structures.base']);
but 'adf.structures.base' module does not exist in the dist nor src versions of the code, am I missing something?

adf-structures-base is a separate dependency you need to reference in addition to angular-dashboard-framework.  If you are using bower to install it, specify adf-structures-base as the dependency.  Then you can reference the adf.structures.base angular module.

I'm new to bower, can you show me the command to define it as a dependency?

If you are using bower, you can install it with the following command:
bower install adf-structures-base --save

But bower isn't necessary, you can also just download and include angular-structures-base.js or  angular-structures-base.min.js from the dist files.

Thank you.


Push to rubygems?

perfdashboard doesn't seem to be available on rubygems.org

btw. homepage is not good URL
ERROR:  While executing gem ... (Gem::InvalidSpecificationException)
    "TODO"

It was renamed and pushed to rubygems.org :)


versions github / npm different?

github / package.json "version": "1.2.0"
npm 1.2.1
Thanks!

Looks like I forgot to push to commit that changed the package.json file for version 1.2.1. But the source code is all the same. Anyhow, this is fixed now in 1.3.0.


adding supported/unsupported platforms to README

#1430

Changes Unknown when pulling 64fe94bbcb9597c7e65e88c4820864d6392fe94f on jabubake:readme_update into ** on GoogleCloudPlatform:master**.

Coverage remained the same at 80.907% when pulling c4a45044ede5864345dc4a3db4635be8568b6aab on jabubake:readme_update into baf7c229f5ec32e8240350a48319354fa62ea533 on GoogleCloudPlatform:master.


Action.reset()

Version
v3 is SLOW for Android right now (because React Native ExperimentalNavigation is slow), see #355, no need to create new issue!
Tell us which versions you are using (react native 0.22 is required for v3!):

react-native-router-flux v3.22.23
react-native v0.24.1

Expected behaviour
Reset the scenes
Actual behaviour
function undefined
Steps to reproduce

So I wrote this into my branch, but recently stopped using it in favor of Actions.scene({type: 'reset'}) - would that work for you? If not, I can submit a PR.

Also, reset will not work for 0.24 (see #525)

I'll close this issue since 0.26 will be landed in few days.
If you still get this issue with 0.26.0 (both updated react-native and react-native-router-flux), please reopen it.


chore(NODE-5776): track aws credential as a devDep in addtion to peerDep

Description
What is changing?
Track aws sdk in dev deps and skip aws auth tests that do not work without SDK
Is there new documentation needed for these changes?
No
What is the motivation for this change?
This is to control which version we test with so we can remove vulnerable versions from our matrix
Double check the following

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: type(NODE-xxxx)[!]: description

Example: feat(NODE-1234)!: rewriting everything in coffeescript

[x] Changes are covered by tests
[x] New TODOs have a related JIRA ticket

Is there any AC for this PR since it is a sub-task of NODE-5710? The PR looks good in isolation.