updated golangci.yml to exclude new errors

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

PTAL @dvyukov

Codecov Report

Merging #1405 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1405   +/-   ##
=======================================
  Coverage   57.82%   57.82%
=======================================
  Files         131      131
  Lines       24849    24849
=======================================
  Hits        14368    14368
  Misses       9812     9812
  Partials      669      669

Continue to review full report at Codecov.

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

Perfect!


Audio not working

Gnar suddenly stops midsong for a few seconds and then plays along the moment he stopped, there is no problem with the internet connection though. And he just left the channel he was playing in even though the song should still be playing and the queue is not empty.

Several times a minute
Gnar absolutely does not want to sing to us :-(

@Avarel help us


ZXHN H267N router error

When trying to decrypt the config.bin for ZXHN H267N, I get the following error:
Traceback (most recent call last):
  File "examples/decode.py", line 75, in <module>
    main()
  File "examples/decode.py", line 63, in main
    infile = zcu.encryption.aes_decrypt(infile, key, is_digi)
  File "/tmp/zte-config-utility/zcu/encryption.py", line 44, in aes_decrypt
    decrypted_data.write(aes_cipher.decrypt(encrypted_data.read()))
  File "/tmp/zte-config-utility/env/lib/python3.8/site-packages/pycryptodomex-3.10.1-py3.8-linux-x86_64.egg/Cryptodome/Cipher/_mode_ecb.py", line 196, in decrypt
    raise ValueError("Data must be aligned to block boundary in ECB mode")

Unfortunately, I'm not sure what could have led to unaligned data. I can send you the config.bin if there's a private way to do that.

It's always useful to know what command you used, too. If you want me to take a look when I have the time, just tell me where I can contact you. I can't guarantee anything and you should know that config.bin has some sensitive info in it.

I ended up finding the password in a forum, but I'd be glad to send you the config file (I think I still have it). You can contact me<EMAIL_ADDRESS>if you like!

Is the one you found listed here?

IIRC no, I remember looking. I can get you the password later if you want to add it.

If it's not there, then that would help, yeah!
It would also help if you could provide the exact signature, along with it.
You can get that by using:
python3 examples/signature.py <your config.bin>

The signature is:
12:24:29 $ python3 signature.py ~/Downloads/Sync/config.bin
ZXHN H267N V1.0

So it looks like the password is $Extra,ODP0ptiOns$987%, but this isn't a password issue, it crashes because of an alignment issue. It doesn't proceed past the first password tried.

I'm fairy certain this is not your config.bin encryption key, but your root user password. If you're using the latest version, it should have said it's the wrong key though, so maybe that's something worth taking a look at. I think I'll just throw you an email for the config.bin, so I can run some tests.

Ah, yes, I thought the two were the same, sorry. I've sent you my config.bin in case it helps, thanks!

Could you try using your router's serial number instead of a key (with the --serial parameter)?
or send it to my email if you want me to try instead


Minor fixes from  (#212)

Hypothesis: Creating this PR will result in EasyCLA running which will allow me to push -f the branch.
(This from #1, where the same happened.)

Hypothesis confirmed. If you push to the branch that the PR is requested against such that there's no diff, the PR somehow auto-merges. I didn't press any merge buttons on this PR, just pushed the commit to main.


Sample needed that shows how to perform advanced clientside caching techniques

We do have example with cookie and HTLM5 LocalStore usage https://github.com/OfficeDev/PnP/tree/master/Samples/Performance.Caching


Doctests

We need doctests working.

This looks like the wanted package: https://github.com/JuliaLang/JuliaDoc/blob/master/juliadoc/jldoctest.py

Reference to #4


[FEATURE] - Support for !Send actors

Feature Description
A way to handle actors containing !Send data.
Motivation
Often when dealing with !Send data you spawn a dedicated thread that owns it and processes commands that comes through channel. When you want to do that while using this library you basically need to forward all messages to that thread, so it's generate boilerplate that shouldn't be needed for that.
Proposed Solution
Relaxing Actor trait bound to not require Send or adding new LocalActor trait that doesn't require it. Adding spawn_local function, similar to other spawn_* functions, that spawns !Send actor.
Alternatives Considered
Adding some macros to reduce boilerplate while using dedicated thread to manage !Send data.

I can definitely see this being useful. I'm wondering how would you intend on using this in your given use case? Would you plan on running the actor in an isolated thread with spawn_in_thread? Or would you make use of tokio's LocalSet for this?

I've been using LocalSet in one of my projects with my naive actor implementation running as task there, and it was fine for my case, so something similar would be enough for me. But I think it would make sens to allow for spawning both blocking and non-blocking !Send actors, as you can do with normal actors.

Great suggestions, I'll try implementing this soon when I get the time, otherwise I'm also open to pull requests


The poodles code review | Jul-31-2020

Beautiful site Carlos!
I hope your problem got solved now that you deleted your dist from your .gitignore.

Yes @Meltrust . That was the error. Now the site is properly deployed. Thanks for your feedback.


UDPRoutes from other namespaces are not getting attached

Hello,
I hit very weird problem today:
When I'm using UDPRoute in app's namespace (and allowed all namespaces on the listener) - the route seems attached in route status, but on Gateway status there are 0 attached routes. My clients also cannot connect to the backend app and are getting permission denied from the stunner.
There's a route:
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: UDPRoute
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: >
      {"apiVersion":"gateway.networking.k8s.io/v1alpha2","kind":"UDPRoute","metadata":{"annotations":{},"labels":{"argocd.argoproj.io/instance":"janus-dev"},"name":"janus-dev","namespace":"dev"},"spec":{"parentRefs":[{"name":"stunner-config","namespace":"stunner"}],"rules":[{"backendRefs":[{"name":"janus-dev","namespace":"dev"}]}]}}
  creationTimestamp: '2023-06-30T01:11:58Z'
  generation: 1
  labels:
    argocd.argoproj.io/instance: janus-dev
  name: janus-dev
  namespace: dev
  resourceVersion: '71179887'
  uid: f86715d8-b32c-4459-ad64-b9b33951239b
spec:
  parentRefs:
    - group: gateway.networking.k8s.io
      kind: Gateway
      name: stunner-config
      namespace: stunner
  rules:
    - backendRefs:
        - group: ''
          kind: Service
          name: janus-dev
          namespace: dev
          weight: 1
status:
  parents:
    - conditions:
        - lastTransitionTime: '2023-06-30T01:16:09Z'
          message: parent accepts the route
          observedGeneration: 1
          reason: Accepted
          status: 'True'
          type: Accepted
        - lastTransitionTime: '2023-06-30T01:16:09Z'
          message: all backend references successfully resolved
          observedGeneration: 1
          reason: ResolvedRefs
          status: 'True'
          type: ResolvedRefs
      controllerName: stunner.l7mp.io/gateway-operator
      parentRef:
        group: gateway.networking.k8s.io
        kind: Gateway
        name: stunner-config
        namespace: stunner

But when I apply similar route to the same namespace as Gateway is - it works just fine.
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: UDPRoute
metadata:
  name: janus-dev
  namespace: stunner
spec:
  parentRefs:
    - name: stunner-config
  rules:
    - backendRefs:
        - name: janus-dev
          namespace: dev

Now traffic is being passed through stunner and gateway shows attachedRoutes: 1 on the listener status.

Thanks for the report, this is indeed a bug. In fact, it is a combination of two things: a somewhat underdocumented STUNner limitation plus an actual bug:

Currently STUNner does not implement cross-namespace Gateway-UDPRoute bindings for simplicity: only UDPRoutes from the same namespace are allowed. This limitation is documented here and here. We didn't think this was an important feature for STUNner, but now that you are reporting we will make sure to fix this (hopefully in the next release). See the new issue here.
The bug is that for some reason the URPRoute seems to misreport the Accepted status as True, even though the Gateway rejected the route due to a cross-namespace binding attempt. This then quite understandably creates the illusion that cross-namespace bindings should work. This should be fixed ASAP, see the new issue here.

Is deploying the Gateway and the UDPRoute into the same namespace an acceptable workaround to you until we fix this? Note that, as another subtle STUNner limitation, currently the UDPRoute can refer to any Service in any namespace (see docs here): to comply  with the Gateway API we would also need to implement ReferenceGrants, but this is also a low-prio item on the TODO list at this point.

Is deploying the Gateway and the UDPRoute into the same namespace an acceptable workaround to you until we fix this?

This would mean I need to allow application helm chart to modify stunner namespace or add UDPRoutes to stunner-config helm chart. It can be temp workaround, but it's not good. Thank you very much for a quick response anyway, at the moment I'll use this workaround, but I'm waiting for this to be solved! :D

I see. We'll prioritize this feature then.
Quick question: do you want full support for ReferenceGrants (ReferenceGrant is  a CRD that you place into a namespace to allow Gateways from other namespaces to accept routes from said namespace or vice versa) or is it enough if we allow UDPRoute bindings from any namespace without restriction?
Anyway, this feature is contingent on another major milestone: maganed dataplane support in the operator. Once that lands, we can easily add support for cross-namespace bindings. Until then, please bear with us. Or better yet: please keep on bugging us frequently on Discord or here so that we do not forget!...:-)

This would be enough: https://gateway-api.sigs.k8s.io/references/spec/#gateway.networking.k8s.io/v1beta1.AllowedRoutes (a field on the listener).
ReferenceGrants seems to be hard to implement and too much husstle for initial support ;)
I don't use discord ;/

This should now fixed as of e770d05 in the gateway-operator repo, can you please test?
The below now works for me perfectly:
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: Gateway
metadata:
  name: stunner-config
  namespace: stunner
spec:
  gatewayClassName: stunner-gatewayclass
  listeners:
    - name: udp-listener
      port: 3478
      protocol: UDP
      allowedRoutes:
        namespaces:
          from: All

apiVersion: gateway.networking.k8s.io/v1alpha2
kind: UDPRoute
metadata:
  name: janus-dev
  namespace: dev
spec:
  parentRefs:
    - name: stunner-config
      namespace: stunner
  rules:
    - backendRefs:
        - name: janus-dev
          namespace: dev

You can also use label selectors to choose the namespaces the gateway would accept routes from:
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: Gateway
metadata:
  name: stunner-config
  namespace: stunner
spec:
  gatewayClassName: stunner-gatewayclass
  listeners:
    - name: udp-listener
      port: 3478
      protocol: UDP
      allowedRoutes:
        namespaces:
          from: Selector
          selector:
            matchLabels:
              udp-gateway: accept

Of course, this requires the target namespace to be labelled with udp-gateway=accept.
Currently this feature is only available on the dev release channel from the stunner/stunner-gateway-operator-dev chart:
helm install stunner-gateway-operator stunner/stunner-gateway-operator-dev --create-namespace --namespace=stunner-system

We hope to put together a new stable release soon.
Please report back your findings.

@FLM210 Can I please ask you to test the dev version to know if it fixes the problem?
I've no time to break my develop infra just to check if it fixes the issue.

@FLM210 Can I please ask you to test the dev version to know if it fixes the problem?
I've no time to break my develop infra just to check if it fixes the issue.

The dev version solved my problem, but there is a small issue with the dev version
https://github.com/l7mp/stunner-gateway-operator/issues/39

@FLM210  Can you check it now?
Do not forget to helm repo update.
helm install stunner-gateway-operator stunner/stunner-gateway-operator-dev --namespace=stunner --create-namespace --set stunnerGatewayOperator.dataplane.mode=managed
(Obviously, the managed dataplane flag is needed if you'd like to skip installing STUNner manually.)

@davidkornel  Now the dev version can run normally

Great, if you face any issues feel free to reopen, until then I'm closing this issue.


Implementing a missed c'tor in CrossCrossCoefficient [cross-cross-dev]

This is a quick fix to address an issue pointed out in issue #1838.

@YohannDudouit, can you also fix MassIntegrator::SetupPA in this PR (see https://github.com/mfem/mfem/issues/1838#issuecomment-720144269)?

@tzanio , we discussed this at the PR meeting.  I went ahead and removed the unimplemented method with @YohannDudouit 's approval.  Thanks, guys!

Merged in next for testing...


[Extension] load only Single page extension on desktop-desktop env

Signed-off-by: Abhishek-kumar09<EMAIL_ADDRESS>Description
This PR checks for user-agent in which the website is running and based on that loads the pages without drawer to navigate away.
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Preview

@Abhishek-kumar09, what you have in the screenshot hits the nail on the head. It's good.

@Abhishek-kumar09 for your reference - https://youtu.be/y_LsCOG-PHE?t=15

I assume you will be giving reference for all your idioms from now on 😸

ha!


Remove deprecations

Closes #1045. Removed all deprecations from the December release and back:
Compiler

Removed all FooType aliases of Foo types.
Removed uri scalar.
Removed Map<K, V> model.
Removed @serviceTitle and @serviceVersion decorators.
Removed getServiceNamespace, getServiceTitle, getServiceVersion, getServiceNamespaceString and setServiceNamespace functions.

Rest

Removed @produces and @consumes decorators.
Removed getProduces and getConsumes functions.

TODO

[ ] Remove @segmentSeparator

emitters option to be removed in March sprint per (https://github.com/microsoft/cadl/issues/1556)

Here's the corresponding cadl-azure PR https://github.com/Azure/cadl-azure/pull/2439

Here's the tracking issue for breaking changes: https://github.com/microsoft/cadl/issues/1564


first commit to new branch

Merge new branch to main

"I have read the CLA Document and I hereby sign the CLA"

recheck


Breaking line indentation: pick a style

Let's agree on a consistent indentation style for line breaking statements including ternary operators, logical operators and string concatenators.
Discussion started in https://github.com/mesosphere/marathon-ui/pull/184/files#diff-ee58f29f51a4e5443773b0fed24eb749R86
For ternaries, we agreed to
return doWeLikeTernaries
  ?
YES

  : null;

But we haven't decided on a similar style for
var extraLongString = "dadadadadadadadadada" +
  "dedede!";
// OR
var extraLongString = "dadadadadadadadadada"
  +  "dedede!";
// ?

and
if (whateverAndEver &&
  alsoThisAndThat &&
  whyNotThose) {
// OR
if (whateverAndEver
  && alsoThisAndThat
  && whyNotThose) {

Should we always go for the leading operator on the new lines for consistency with the ternary syntax?

Pulling in @rcorral @mlunoe @kennyt @jfurrow

For function calls (you are representing declarations here, and I think we should avoid the variable declarations and prefer function declarations) I would like to go with this, as it follows what we do in jsx:

SomeEntity.extraLongFunc(superLongFirstArgument,
  secondArgument,
  thirdArgument,
  fourthArgument
);

I think it makes it more readable and

Did we decide to go with ternary operators?
For string concatenation and boolean operators I'd like us to just be consistent. Some prefer the leading style as it is more apparent what is going on, and I tend to agree.

Re function calls: yep, that was my idea too, first argument inline and following arguments one per line, just like we do in JSX.
About ternaries: we definitely agreed to not agree on anything until the Airbnb styleguide covers them :) But as a matter of fact some of us are writing them, and it makes sense to at least keep a consistent style. We (the HH team) decided for now to only ever use them for short, simple expressions and to indent them as above.
I also tend to like the leading && and || .


[FAILED] Failed to start Docker Storage Setup.

After the intial setup I always see
[FAILED] Failed to start Docker Storage Setup.

in the boot log. However it does not seem to be critical since everything seems to work fine.

Yes. An annoying problem. I wonder why they don't fix it in the docker storag esteup.

Fixed by #14


Command to copy VS Code version/commit and OS version to clipboard

Implemented feature request from issue #67764. The command Alt+Shift+I copies to clipboard the version and OS platform information in the format of issue reporter. Work done with @rthaneda.

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

@RMacfarlane the location where this command is contributed from is not ideal, it should probably come from some issue.ts contribution.

Hey @TejDNagaraja, thanks for your work on this! Unfortunately, we've decided that the original issue is out of scope - we would prefer to avoid introducing another command for something that can already be accomplished. Sorry!


Fetching and displaying user information.

Displaying list of all users for system admin

Adding customer in system admin dashboard

Updating user status

Displaying user data in profile

Fetching user data in account page


remote_directory command for transferring files of size > 500 MB

Hi
I am trying to transfer some files from my workstation to my client. I am using the following resource.
remote_directory '/scratch/ofsaebas/conf' do
  source 'conf'
  owner 'ofsaebas'
  group 'dba'
  mode '0755'
  action :create
end

When I do knife cookbook upload . the command works for directory of lesser size (say 1 MB ?) but I get the following error when the file size is 500 MB or more.
Uploading learnChef      [0.2.0]
ERROR: NoMemoryError: failed to allocate memory

I just want to copy the directory (size: 500 MB) from my workstation to some node.

Perhaps try knife cookbook upload, I'm not sure how smart ChefFS is about using streaming requests for everything, though I remember some recent patch activity around something like that.

I had made a typing error. I am using knife cookbook upload my_cookbook_name

so while that looks like a bug, a better approach is to toss a tarball up into s3 somewhere and then download that with remote_file and extract with an execute resource.  a 500MB directory structure as a remote_directory is most likely not going to make you happy and this is likely just the first issue with it.

Thanks. I will try that.

We are currently experiencing the exact same issue. When using the "knife cookbook upload my_cookbook_name" syntax, in the console output we get ERROR: NoMemoryError: failed to allocate memory but only if the .zip file we are trying to copy is greater than 500mb.
Is there any ETA on getting this bug fixed? If not can someone help by more explicitly explaining what you would need to do to try the upload to S3 and download it from there to the deployment machine. I am an IT person not a dev so this is a bit above my head.

This bug won't get fixed. You absolutely should never put files that size in a cookbook. If you need to download a file, use remote_file against a URL (any url) where that file is stored.


Few questions regarding script

Is it possible to make that script starts from token #0 instead of #1?
If we don't need hidden png/json, can we just leave environmental variable empty/blank?
Also this is my image url in json file: "image":"http:\/\/tv3-app-vg9pw.ondigitalocean.app\/asset\/1.png"
What's up with \/
Also thank you for this script, this is literally a game changer

Is it possible to make that script starts from token 0 instead of 1?

Hi @vespuccieth, I'm about to publish a video that shows how you can customize the app to your needs, you will be able to start from there to change any behavior. But as far as I know the ERC721 standard expects token IDs to start from 1. As a developer I would prefer IDs to start from 0 too, but we have to be compliant to that... 😅
I'm gonna drop the link here as soon as I publish the video. 😉

If we don't need hidden png/json, can we just leave environmental variable empty/blank?

Everything should work as expected, but your hidden metadata (if requested by any marketplace) will be invalid. I highly suggest having the hidden metadata/assets anyway to be fully compliant... but it's up to you.

Also this is my image url in json file: "image":"http://tv3-app-vg9pw.ondigitalocean.app/asset/1.png"
What's up with /

The JSON code you are showing is perfectly fine. I know that syntax is a bit uncommon, but it's correct as per JSON specifications.
You can see it trying to paste the following code on https://jsoneditoronline.org:
{
  "image": "http:\/\/tv3-app-vg9pw.ondigitalocean.app\/asset\/1.png"
}

You will see that the \/ chars will be highlighted as an escaped sequence for /.

Also thank you for this script, this is literally a game changer

Thank you so much, I really appreciate it. 🙏

Great, appreciate it ♥


Markdown code blocks highlighted incorrectly in documentation comments

Not sure if this is related to this plugin, but this has been happening for quite a while now. You can reproduce it by typing the same comment as shown below in Atom.

If that's possible real syntax highlighting in the code block (in this case YAML) would be very nice!

It would be interesting to see how dartdoc treats that documentation - whether it considers it yaml or not. I don't think code fences (triple backquotes) are part of dartdoc. Lines that have 5 leading spaces (1 after the /// comment marker, and 4 to indicate a code section, are treated as Dart source code.
This may be as-designed -

In theory it should support markdown (I've seen it working as well). I
actually didn't know markdown supported code blocks like in ReST, I do
actually like this style.
On Sun, Feb 28, 2016, 21:06 Devon Carew<EMAIL_ADDRESS>wrote:

It would be interesting to see how dartdoc treats that documentation -
whether it considers it yaml or not. I don't think code fences (triple
backquotes) are part of dartdoc. Lines that have 5 leading spaces (1 after
the /// comment marker, and 4 to indicate a code section, are treated as
Dart source code.
This may be as-designed -
—
Reply to this email directly or view it on GitHub
https://github.com/dart-atom/dartlang/issues/811#issuecomment-189935676.


Blob storage issues first pass

ContentType field in AppendBlobAppendBlockOptions is the wrong type.
Need to add a type for binary types
Rename vars assigned by header parsing to the name they have defined in the swagger
XML group fields are being outputted twice each
Uri type not being transformed properly

Nested headers are missing

These issues were resolved in PR: #289

Adding separate issue to track nested headers


Add support for running without systemd

Docker containers can't currently coexist with systemd (it can't be installed in them, they replace it with fakesystemd), so being about to support running without systemd is a requirement of provisioning them. This Ansible role currently uses systemd for at least one task (info below).
The Packer configuration in the upstream ODL Integration is currently hitting an issue because of this. While building Docker images (using under-dev, uncommitted scratch logic), everything works up until a firewalld check that requires systemd.
    docker: PLAY [localhost] **************************************************************
    docker:
    docker: GATHERING FACTS ***************************************************************
    docker: ok: [localhost]
    docker:
    docker: TASK: [dfarrell07.opendaylight | Add ODL yum repo] ****************************
    docker: changed: [localhost]
    docker:
    docker: TASK: [dfarrell07.opendaylight | Install ODL via RPM repo] ********************
    docker: changed: [localhost]
    docker:
    docker: TASK: [dfarrell07.opendaylight | Install ODL via RPM path] ********************
    docker: skipping: [localhost]
    docker:
    docker: TASK: [dfarrell07.opendaylight | Configure ODL Karaf features] ****************
    docker: changed: [localhost]
    docker:
    docker: TASK: [dfarrell07.opendaylight | Configure ODL NB REST port] ******************
    docker: changed: [localhost]
    docker:
    docker: TASK: [dfarrell07.opendaylight | Check if FirewallD service is running] *******
    docker: fatal: [localhost] => error while evaluating conditional: firewalld_status.stdout not in ['unknown', 'active']
    docker:
    docker: FATAL: all hosts have already failed -- aborting
    docker:
    docker: PLAY RECAP ********************************************************************
    docker: to retry, use: --limit @/root/install_odl.retry
    docker:
    docker: localhost                  : ok=5    changed=4    unreachable=1    failed=0
    docker:
==> docker: Killing the container: abb4c1dd9fde122569933f1d9f7db089bec6f1e8626c9e70d2c9b6d0046350ad
Build 'docker' errored: Error executing Ansible: Non-zero exit status: 3

I think the command in this task is causing the problem we're currently hitting
- name: Check if FirewallD service is running
  command: systemctl is-active firewalld
  failed_when: firewalld_status.stdout not in ['unknown', 'active']
  changed_when: false
  register: firewalld_status

Also going to see a problem with starting ODL's systemd service, I strongly suspect.
- name: Start ODL systemd service
  service:
    name=opendaylight
    enabled=yes
    state=started

ef051553f69e434f6b7959e56176875b698770c2, a8ace2b2643ff0b45dc0ead5305f1218342e60e3 and 6a7e02a10e6e943f64b204e4466f82fc96402269 added this.


Re-implementing watch

It seems that the overall cli is nicely formatted, it's mainly a case of removing spaghetti and updating the code base while allowing for future developments. Leaving this as a draft while I work on the rest

Okay so, current developments and errors

Providers have been updated and will autoupdate the same way as dl
Play actually works, hardcoded with mpv right now, going to add an option for adding and changing that
Seem to get a non-type error after it loads one episode, everything still functions but that needs to be fixed. I think all the non-type is doing is kicking us out of the watch environment

If anyone has any optimisation ideas about removing some of the spaghetti in the code base send them my way, I'm focusing on merely getting it running and usable for users at the moment. A refactor is hard when the original code doesn't even work.

Will probably take a good look at it tomorrow.  A big refractor isn't out of the question. Haven't used watch so you'll have to fill me in on the details.
I think watch uses the player folder, but since it's already done in utils I think it's better to use that already finished implementation.

Yeah I'm starting to drag in utils

Okay so now

When you watch a show it will always bring you back to that show, to watch again.

Things to be added

A way to re-enter the list from a show without using CTRL-C
A way to queue multiple episodes in watch (it may already be there, the pasta sauce is covering it )
Improve the docs / tutorial for the use case of watch
Local serialisation of the list into a MAL like format so we can then import and export MAL lists.

For context, this PR is just to regain functionality of watch using the old codebase, it literally wasn't working before.
A rewrite will happen once the watch command is functional again

Download still seems to use the basedownloader instead of aria2
Watch loops the same episode forever until exit.

Yeah downloader hasn't been touched yet
Watch shouldn't loop the episode, it should return back to the play menu

Fixed the weird looping issue. Next it to set up downloader to choose between aria and default, lagrad should be working on this.

It now uses the same downloading system as dl does, including the same config

Very nice, we are ready to merge to fix the spaghetti! And then it's time to refactor it all

Bug: Using back multiple times will make q (quit) not work as intended. If you use back 10 times you'll have to spam q 10 times to actually exit the script.

Bug (ish): you can set episodes_done above title length (also causing some funky behavior).

What the heck tests are you doing to find these bugs :o

Bug: Assuming valid input on set episodes_done causing errors if not int. We'd want at least graceful exit or reprompt.

Bug: invalid provider name errors watch, graceful exit or reprompt would be better

dude the teamwork on this PR is wild, 100% efficiency.

Bug: inconsistent download, anime dl -e 1:3 downloads ep 1-3, while watch downloads 1-2


Fix broken url in docs

Summary
Easy navigation and easy reading for others.
/area documentation
/kind

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rajakavitha1 after the PR has been reviewed.
You can assign the PR to them by writing /assign @rajakavitha1 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rajakavitha1 after the PR has been reviewed.
You can assign the PR to them by writing /assign @rajakavitha1 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md
389710e Merge branch 'fix-url-bug' of https://github.com/saintmalik/falco-website into fix-url-bug

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md
389710e Merge branch 'fix-url-bug' of https://github.com/saintmalik/falco-website into fix-url-bug

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@oke-py , i dont know why the dco check isn't successful, i did sign it from my local repo before pushing, any help?

@oke-py , i dont know why the dco check isn't successful, i did sign it from my local repo before pushing, any help?

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

1a1ecc0 Update content/ja/docs/running.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@saintmalik Actually, I'm not an owner of this repo, and so I cannot give you a good advice...
How about rebasing the commits to single commit?
It would not be a good manner, but I have no idea except for this.

@saintmalik Actually, I'm not an owner of this repo, and so I cannot give you a good advice...
How about rebasing the commits to single commit?
It would not be a good manner, but I have no idea except for this.

@saintmalik Thanks for the PR!!!
To add a 'DCO signoff' to your commit:

Ensure that you have a local copy of your branch by checking out the pull request locally via command line.
 Push the commit with the DCO signoff:
git push --force-with-lease origin <branch_name>

@saintmalik Thanks for the PR!!!
To add a 'DCO signoff' to your commit:

Ensure that you have a local copy of your branch by checking out the pull request locally via command line.
 Push the commit with the DCO signoff:
git push --force-with-lease origin <branch_name>

@saintmalik Actually, I'm not an owner of this repo, and so I cannot give you a good advice...
How about rebasing the commits to single commit?
It would not be a good manner, but I have no idea except for this.

Okay thanks for the tip

@saintmalik Actually, I'm not an owner of this repo, and so I cannot give you a good advice...
How about rebasing the commits to single commit?
It would not be a good manner, but I have no idea except for this.

Okay thanks for the tip

@saintmalik Thanks for the PR!!! 1a1ecc0 Update content/ja/docs/running.md is missing the DCO signoff
To add a 'DCO signoff' to your commit:

Ensure that you have a local copy of your branch by checking out the pull request locally via command line.
 Push the commit with the DCO signoff:
git push --force-with-lease origin <branch_name>

Did all this but its not rectifying

@saintmalik Thanks for the PR!!! 1a1ecc0 Update content/ja/docs/running.md is missing the DCO signoff
To add a 'DCO signoff' to your commit:

Ensure that you have a local copy of your branch by checking out the pull request locally via command line.
 Push the commit with the DCO signoff:
git push --force-with-lease origin <branch_name>

Sorry am closing this also😩, opened another pull here #377, sorry if i might have stressed you out🙏.

@saintmalik Thanks for the PR!!! 1a1ecc0 Update content/ja/docs/running.md is missing the DCO signoff
To add a 'DCO signoff' to your commit:

Ensure that you have a local copy of your branch by checking out the pull request locally via command line.
 Push the commit with the DCO signoff:
git push --force-with-lease origin <branch_name>

Did all this but its not rectifying

@saintmalik Thanks for the PR!!! 1a1ecc0 Update content/ja/docs/running.md is missing the DCO signoff
To add a 'DCO signoff' to your commit:

Ensure that you have a local copy of your branch by checking out the pull request locally via command line.
 Push the commit with the DCO signoff:
git push --force-with-lease origin <branch_name>

Sorry am closing this also😩, opened another pull here #377, sorry if i might have stressed you out🙏.


Add Joe gOHM-FRAX Vault

Vault: 0x42b2713e4eDA3C2275CE28696709A35CeB98A8E4
Strategy: 0xB0225E1c3BEd1aba89fa0c355de3941aC9FD6758
Want: 0x3E6Be71dE004363379d864006AAC37C9F55F8329
PoolId: 34

New vaults need to go on top under BIFI Maxi and any boosts. Can you move below the AVAX boost?

@MirthFutures I updated the order

@Tburm Call Rewards and Rewards Available are broken. Its because you used the wrong strategy. There is trader joe specific strategies.

@MirthFutures I see, I will re-deploy with the TraderJoe strategy.


[5.3] Remove controller inspection from routing

DO NOT MERGE YET
This method has quite a lot of issues and also makes routes definitions confusing as they are not directly referenced in routes file. This sometimes may lead to unexpected route definitions (e. g. if method like getFilesystem is added to parent or trait - developer mistake, but may be unnoticed).
It also may cause security mistakes when middleware is applied on specific methods (e. g. someone adds new method to controller and forgets to add middleware), which arise rarely when routes are defined manually.
And at last - parameters issue, which is hard to fix because of grouping or other unexpected route combinations (See #10774).
To decrease amount of breaks it should be marked as deprecated in next 5.1.x release.
Possible alternative: mark this method as deprecated in master and remove in next major release.

CS fixed in #10778

You can rebase this now. :)

To decrease amount of breaks it should be marked as deprecated in next 5.1.x release.
Possible alternative: mark this method as deprecated in master and remove in next major release.

I think we should deprecate in 5.1 and remove in 5.2.

We can't really mark something as deprecated in 5.1 just a few weeks before release of 5.2. :smile: ... I'm OK with deprecating this in 5.2 and removing in 5.3 though. Agree it is not really a good idea to use it.

:+1:

Can we keep this PR open then, and mark it as 5.3?

Then, this can be merged as soon as we have a 5.2 branch made.

Please could you rebase this, then we can merge this once we have a 5.2 branch.


draft JMC presentation finished

Hopefully everyone is enjoying the weekend and you all won't see this message until after my presentation to the JMC on Monday morning.
However, for the record, I've just uploaded a complete draft beamer-hake-JMC.pdf to "Hake JTC 2018" on Google Drive.
I also committed the changes to make this presentation, along with some minor edits to a few plotting functions that were incorporated into the talk.
I tried to make the edits to the plotting functions in a way that new options were added while maintaining the default behavior as used in the assessment report, but I haven't tried building again to check. If anything breaks, we can always revert to the v1.5 release https://github.com/cgrandin/hake-assessment/releases for building.
I'm hoping to take the day off tomorrow, but will check for any comments on this issue sometime in the evening. If you look at the presentation and anything seems really out of whack, please send me a text message at any time.

This looks good to me Ian.  The only thing I saw was an extra period on bullet four of Page 3.  I got an error when trying to build it, but didn't spend time debugging it for me.  I just used the google drive version for now.  Thanks again for representing the JTC at the JMC!

I should be around and available most of the day today and tomorrow until 1PM so if anything comes up Ian that you need help on (or an extra pair of eyes) just start an issue.

I just commit a new version of the JMC presentation file which removes the extra period (thanks Aaron) and adds back the lines
source("all.r")
load.models.into.parent.env()

which I had commented out to expedite building while working in the R workspace using build.doc().
I'm guessing that having those back will allow it to build for any of you.
The JTC presentation took something like 2 hours, with lots of questions about the maturity and weight-at-age calculations, but no clear conclusions or direction so far. The final presentation (with fixed period) is on the Google Drive in the JMC folder.


Dynamic Lists in a Dynamic UI Element is not performant

Describe the Bug with repro steps
We are trying to generate a UI like:

We have an element rfcName with Dynamic List, Ref: Manifest below:

Where the contents of the elements are fetched by a dynamicInvoke call like:
https://management.azure.com/subscriptions/f34b22a3-2202-4fb1-b040-1332bd928c84/resourceGroups/siddharth-imp/providers/Microsoft.Web/sites/sid-sap-testbox/hostruntime/runtime/webhooks/workflow/api/management/operationGroups/sap/operations/listRfcs/dynamicInvoke?api-version=2018-11-01

The element works just fine when it's a static element, but performance is much slower and the page becomes un-responsive when the element is created through a dynamic call, Here the field rfcName is generate as a result of Dynamic selection of bodyType.
Selecting bodyType as Dynamic makes below call:
curl 'https://management.azure.com/subscriptions/f34b22a3-2202-4fb1-b040-1332bd928c84/resourceGroups/siddharth-imp/providers/Microsoft.Web/sites/sid-sap-testbox/hostruntime/runtime/webhooks/workflow/api/management/operationGroups/sap/operations/getCallRfcInputVarientSchemaSwagger/dynamicInvoke?api-version=2018-11-01' \
  -H 'x-ms-user-agent: LogicAppsDesigner/2.75.0 (host azure-logic-apps-client ; source DesignerStandard)' \
  --data-raw $'{"parameters":{"bodyType":"Dynamic"},...' \
  --compressed

and Response:

Which results in creation of dynamic field rfcName. When is element is top level element, and not a dynamic element, it is able to handle list of RFCs but not in case of Dyanmic element.
What type of Logic App Is this happening in?
Standard (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response
AB#25863872

Just to be clear, you are not seeing dynamic list being populated for rfcName when that is a dynamic property returned by body type dynamic call?

No, we see a REST call to dynamicInvoke fetching all the RFCs. But while populating the UI stops responding, and the Edge browser suggests to kill the page.


Update the "Contribute" and "About" pages

With the conclusion of the NGLP project in Spring 2022, Antleaf is no longer actively maintaining SComCat and their repository is archived; Open Tech Strategies now maintains SComCat on a semi-volunteer basis.

@smpsnr -- I didn't test these edits in a running instance.  If I made any formatting mistakes or anything, please feel free to fix them in the obvious way and deploy the completed changes.


How to use Testmode

How can I use test card in this Lib

by providing the test credentials (from Stripe) like always


Can't access AppData folder

Program will not backup Windows user AppData folder, complaining about not having permission when putting check mark on the group containing AppData. Issue does not happen when trying to backup .\AppData\Roaming.
AppData folder has hidden attribute. .\AppData\Roaming has archive attribute.

Clarification, I'm noticing the statement the MessageBox is making is specifying ".\AppData\Local\Application Data" as not having access.


detect country before number it's fully typed

There should be one or more regexp to find out if a string is the beginning of a vat number for a country
eg: if it starts with FI, it could return that this is an invalid vat number that looks like it's from Finland

Gonna be available in v 2.3.0


[DC-1457] Silence all end user credentials user warnings when using personal credentials to run service account impersonation

Closed since DC-1457 marked as WON'T FIX


Introduce strategic_token_limit to fix Anthropic bug

Fix for https://github.com/assafelovic/gpt-researcher/issues/1022
New config variable smart_token_limit defaults to 4000 to fix ValidationError thrown by ChatAnthropic when max_tokens=none, replacing it with max_tokens=4000.

@assafelovic here's the follow-up PR to https://github.com/assafelovic/gpt-researcher/issues/1022
INFO:     [07:18:11] 🤔 Planning the research strategy and subtasks (this may take a minute)...
Error with strategic LLM: 1 validation error for ChatAnthropic
max_tokens
  Input should be a valid integer [type=int_type, input_value=None, input_type=NoneType]
    For further information visit https://errors.pydantic.dev/2.9/v/int_type. Retrying with max_tokens=4000.
See https://github.com/assafelovic/gpt-researcher/issues/1022
Retrying with max_tokens=4000 successful.


Icons issue after upgarding react native from 0.62 to 0.63.2

In below code you can see Icon used is eye and eye-off but actually app shows cart.Similar is the case for all other ICONS in app. This issue occured after i updated react-native to 0.63 version and also i upgraded nativebase to latest verions.
`InputGroup rounded
error={this.props.loginPassword.length > 0 && !this.props.loginPasswordValid}
success={this.props.loginPassword.length > 0 && this.props.loginPasswordValid}>
<Input value={this.props.loginPassword} placeholder='Passsword' secureTextEntry={this.props.secureTextEntry} onChangeText={(event) => this.props.handleInputChange('loginPassword', event)} />
<Icon active name={this.props.secureTextEntry ? 'eye-off' : 'eye'} onPress={() => this.props.toggleShowPassword()} />
</InputGroup`
Attaching screenshot of mobilescreen

package.json file:
"native-base": "^2.13.13",
"react": "16.13.1",
"react-native": "0.63.2",
"react-native-vector-icons": "^7.0.0"
react-native-config.js
module.exports = {
project: {
ios: {},
android: {},
},
assets: ['./assets/fonts', './assets/icomoon'],
};

Missing Environment Information
Your issue may be missing information about your development environment. You can obtain the missing information by running react-native info in a console.

info Fetching system and libraries information...
System:
OS: Linux 5.4 Ubuntu 20.04 LTS (Focal Fossa)
CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
Memory: 950.75 MB / 15.53 GB
Shell: 5.0.16 - /bin/bash
Binaries:
Node: 10.19.0 - /usr/bin/node
Yarn: Not Found
npm: 6.14.4 - /usr/bin/npm
Watchman: Not Found
SDKs:
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Languages:
Java: 11.0.7 - /usr/bin/javac
Python: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.2 => 0.63.2
npmGlobalPackages:
react-native: Not Found


Add OracleDB to integration tests and documentation

To have more robust tests let's add this DB in: Maybe Oracle

After some researches seems oracle enterprise version is not easy to integrate with.
I am using this oracle-xe now, will report the progress if any updates

Development Note:
To start oracle docker instance on Mac M1/M2 chips

We need to use colima to simulate x86 machine, install colima on your Mac through homebrew
Start colima start --cpu 4 --memory 4 --disk 10 virtual machine environment (This matters, or your container might fail)
To test if oracle container can be run by docker run -d -p 1521:1521 -e ORACLE_PASSWORD=admin123 -e APP_USER=my_user -e APP_USER_PASSWORD=my_password gvenzl/oracle-xe

It's in progress now.
TODO LIST:

[ ] Setup the local running command on docker
[ ] Setup the config testing module in _testing
[ ] Setup the dynamic docker client fetcher by colima or native docker API (Since Mac M1 users need additional setup)
[ ] Add to generous integration testing
[ ] Add to individual integration testing
[ ] Add colima section in developer doc
[ ] Add OracleDB integration doc


[Test] ProtoBuf Update at Wed Oct 21 13:14:57 UTC 2020

@mlenkeit I changed the base branch to master and we get a lot of conflicts. Can you tell from which branch chore/update-protobuf was created? Branches must always be created from master to avoid conflicts with squashed commits.

@fredrb the original base branch poc/central-protobuf-files branched off of master.

@mlenkeit is this still just a test or do we need to really merge it?

I think we can close this one. We already had the protobuf changes merged in #950, #952 and #953. @mlenkeit please confirm.


Can I use it with swift 2.0 ?

Question above :)

I am working on this repository at the moment, but as I note in the README, I am debugging right now. I have got it down to only 8 errors and 2 warnings though.


Added button no longer visible but seems to still exist

Added a button w/ an address that has lots of transactions in it (ecash:qz3yu2m8dzwrw5uc85a5pz78dyxnrvdhf542l5ja56).
No transactions showed up after waiting 20-30 seconds
Eventually it goes missing on the buttons page
Trying to re-add the button results in an error saying that it already exists
System jobs seem to be working hard in the background (in theory to sync the address) but the button is totally missing from the UI

I just tried reproducing this and noticed the automatic sync of addresses was broken.
Opening a new PR that will fix it, should probably fix this too


changed to always disabled, removed unused modal close function

Small fix gary mentioned in default test parameter PR

Codecov Report

Merging #132 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #132   +/-   ##
========================================
  Coverage    64.89%   64.89%
========================================
  Files           95       95
  Lines         2555     2555
  Branches       323      323
========================================
  Hits          1658     1658
  Misses         723      723
  Partials       174      174

Continue to review full report at Codecov.

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


Significant changes in scores with the v0.0.11 physics

Context:
I am running my implemention of the PGA-MAP-Elites (see paper) in brax halfcheetah.
I obtained good and consistent results with the old v0.0.10 physics in halfcheetah (and in all other environments too), ranging from 6000 to 7000 at the very end of the run.
Problem description:
Since the v0.0.11 version, these results have changed a lot, here are some points that I want to mention:

At the beginning of the runs, scores are significantly higher and have more variance. With the old physics, the first scores that I plotted ranged from 2000 to 3000 maximum and never went out of these bounds. With the new physics, starting scores range from 4000 to 7000 (which is more than what I obtained for a full run using old physics). Of course these absolute values are not relevant but the sudden change is.

For a full run, there is way less amplitude in the score evolution. Final scores with new physics are still ranging from 6000 to 7000 but now they start way higher.

I want to emphasize that obtaining 7000 at the very beginning of a run seems problematic since for another seed with the same parameters I could not be reaching 7000 even at the end of the run.

It makes comparisons with old results and brax oriented papers irrelevant, at least in halfcheetah.

Additional details:
All parameters are kept equal for these comparisons, only the physics changes (I made use of legacy_spring=True to switch between versions).
The results reported here take into account multiple seeds for each version, by that I mean that I see a very significant change that cannot be due to statistical noise.

Indeed, the position based dynamics physics introduced in version v0.0.11 are quite different than the old springy physics.  They much more accurately model stiff joints, whereas the legacy_spring=True physics of original brax were significantly more wobbly.
As an engine developer, this is a tricky problem more generally.  When we know there's an area we can improve the physics, we want to be able to make those improvements, and have those improvements reflected in the environments that we support.  At the same time, certain users, especially users in the RL research community, want the physics to be exactly the same all the time because they typically are using these sorts of environments as baselines, and want to be able to compare across papers.
We take the opinionated approach that our environments will always have the best physics we have to offer, but that if users want old behavior, that it be available, hence the legacy_spring=True kwarg.  Alternatively, you can specify which Brax version you're comparing against, which should be an unambiguous indicator of environment version.  Version pinning is probably a good idea anyway when doing comparison experiments.
Stepping back: we made a pretty significant effort to maintain backwards compatibility support in the new version, and tried to loudly inform users about these changes in warning messages.  Is there anything you feel like you cannot do in the new version that you could in the old?  Ideally, these updates are purely additive to engine performance and capability.

Thanks for your answer.
There is nothing we cannot do in the new version that we could do in the old. My concern was mostly regarding RL research and comparability between papers.
As you said there is always the solution of mentioning the Brax version in the paper, but in terms of practical use I'm afraid it will not always be easy to find the exact version used in other works or simply to compare to multiple works that use different versions (if Brax physics change on a regular basis). As a Brax adept I'd like it to become a standard in the RL/Evo community and hope it won't be an obstacle :)
Another point that I want to emphasize is iii. from the original post. If, for any reason, the change in physics make it suddenly very easy (or very difficult) to obtain good results on a given environment, I fear it won't help to discriminate between good and bad research ideas.
Again thanks for answering our concerns, I totally understand your opinionated approach and will stick with legacy_spring for now !

With Brax 0.9.3, PPO can only achieve 3000 with the implementation and hyper-parameters provided in the notebook.

Hi @qlan3
As brax has evolved from version 0.0.11 to now, the physics has once again changed quite a bit, and @cdfreeman-google's comment still holds.  For best reproducibility I would stick to a specific version of Brax and physics backend (generalized, positional, or spring).


stdlib benchmarks should produce reports in a unified format

Currently engine benchmarks and stdlib benchmarks produce different and incompatible outputs.
The former is currently nicely plotted  in https://enso-org.github.io/engine-benchmark-results/.
The task is for the stdlib benchmarks to produce output that is acceptable for the plotted benchmarks and can be tracked continuously.
Came as part of the discussion on https://github.com/enso-org/enso/pull/6534

There is currently no support for parsing XML in our standard library. However we can do polyglot import java javax.xml.* and use the needed classes directly. It shouldn't be that hard to mimic the behavior of BenchmarkRunner and its Report or even call into that class directly.

The work that @Akirathan has done to run libs benchmarks within the JMH supersedes this ticket.

#7101


Why when I am recording spoutInput, the recorded mp4 will be very stuck

I am trying to use the node video in T3 and it will not freeze


[CMDCT-3538] - Ensure all icon buttons are 24x24

Description

Before

After

Related ticket(s)

CMDCT-3538

How to test

Create a workplan
Fill it out
Checkout an initiatives Funding sources page to see the icon in the entity card is 24x24
Checkout that the on mobile the return to dashboard is no longer off screen
Submit the WP
Create a SAR
See that the edit button on the dashboard is now 24x24

Pre-review checklist

[x] I have added thorough tests, if necessary
[x] I have updated relevant documentation, if necessary
[x] I have performed a self-review of my code
[x] I have manually tested this PR in the deployed cloud environment

Pre-merge checklist

Review

[x] Design: This work has been reviewed and approved by design, if necessary
[x] Product: This work has been reviewed and approved by product owner, if necessary

the "leave form" icon on mobile came up as 32x32 the check indicating the funding source was complete came up 16x16
just checking if those are expected

Yes, these are expected :). The funding source isn't an icon button, just an icon. I had the same thought too 😆. The leave form can be 32x32, we're really just making sure the min size is at least 24x24 if its an icon button.


build(ci): echo parameters

Codecov Report

Merging #2358 (0a4f70b) into master (72d195c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #2358   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          221       221
  Lines         4387      4387
  Branches       985       985
=========================================
  Hits          4387      4387

Continue to review full report at Codecov.

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

v13.5.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Addrenv fixes part1

Summary
This removes some incorrect usage of up_addrenv_select/up_addrenv_restore. When an address environment is destroyed, there is no need to map the address environment to MMU. Everything that is freed is either from kernel memory or page pool.
Also, remove the unnecessary detaching of L2 table from L1 table in ARM / arm_addrenv_destroy_region. The region is being destroyed thus the mapping will also be naturally lost.
This is preparation for a much bigger re-factoring of the address environments, especially usage up_addrenv_select/up_addrenv_restore which is prone to system crashes if a context switch or IRQ happens in between.
More about this story can be found from here: https://github.com/apache/nuttx/pull/7966
Impact
Removes redundant code
Testing
icicle:knsh and sabre-6quad:netknsh with qemu

@xiaoxiang781216 I verified with qemu that there is no regression regarding this: https://github.com/apache/nuttx/pull/7498
The reason it crashed before, was the removal of the L1 mapping, which is not necessary and is removed in this PR.
I added an undefined symbol into hello app and ran it, no crash only the error about the missing symbol:
ville@ville-ThinkPad-T14s-Gen-1:~/wpc_toolchain/workspace/nuttx-bare/nuttx$ qemu-system-arm -semihosting -M sabrelite -m 1024 -smp 4 -nographic -kernel ./nuttx
telnetd [0:100]

NuttShell (NSH) NuttX-10.4.0
/system/bin/hello
[    8.610000] elf_relocate: Section 2 reloc 11: Failed to get value of symbol[219]: -2
nsh: /system/bin/hello: command not found
nsh>


Issues with scroll in React-Table in mobile view

Minor CSS issues with scrolling horizontally across table in Main directory page in mobile view

resolved


Can't get handler to send only the first Flapping event

I want to update my handler so that the first time a check becomes "flapping", it sends a notification - but on all subsequent events it filters them out based on the "refresh" value in the json (or the default of 1800 seconds). Once it leaves the flapping state, I want to send another notification to show the transition, and then go back to normal event handling.
The problem that I'm running into is that (as far as I know) there's no state information stored about when a check started flapping, or how long it's been flapping (whether that's an occurrences count or a timestamp). For any given event, there's no way to tell whether this was the first flapping event, or for "create"/"resolve" there's no way to tell whether flapping has just stopped.
@portertech mentioned in IRC that he'd be okay with adding a flapping timestamp to the event context coming from sensu-server, and deleting it once flapping stops. That would make it visible in the event data, the API, and Redis.

If the flapping timestamp matches the event timestamp, then flapping just started.
If the flapping timestamp does not match the event timestamp (and it's action:flapping), then it's an ongoing flapping state.
If the flapping timestamp is passed but the action is "create", flapping has just ended.

Alternatively we could track the occurrences of flapping in Redis and pass that to the handler. That would allow us to use the same logic as regular checks for the refresh interval; when comparing timestamps it's hard to actually tell when you should or should not alert if a check is delayed in execution for some reason.

Any comments on this?
I'm sure it'll take a bit of development. Any suggested workaround in the mean time?

@ghicks-rmn hi there. Sorry for the radio silence on this.
I'm tagging this as a feature as part of an ongoing issues triage. This doesn't mean we've decided to implement it, but it does mean we'll revisit this soon. I'm also noting that this could potentially be related to #1196, which we have flagged for 0.24, so this should at least get some fresh attention in the next few weeks. :smile:


Outputting multi-line text fields in templates.

I think it's about time this be implemented as a feature in the framework core. Apparently there's no native/intuitive method for users to automatically output/formatted TextareaField content on the front-end. Why is that?
For example: http://stackoverflow.com/questions/4917596/is-it-possible-to-run-a-function-on-a-silverstripe-template-variable-to-format-o/14517067#14517067
And solutions like this are needed (this is mine):
class TextFormatterExtension extends Extension {
	/**
	 * Chose 'HTMLOut' instead of 'HTML' because that's too generic, could end up overridden in future versions, and
	 * 'NL2BR' because it also needs to convert HTML entities unlike nl2br() on its own.
	 *
	 * @return string
	 */
	public function HTMLOut() {
		return nl2br(Convert::raw2xml($this->owner->value));
	}
}

It's so simple, I don't see why this can't be done. Let me know if you agree :smile::+1:

We already call nl2br() in DBString::forTemplate() (formerly StringField::forTemplate()), and SSViewer.default_cast is Text, so this should happen automatically.
The fact that it doesn’t is because we don’t enforce default_cast in 3.x (see https://github.com/silverstripe/silverstripe-framework/pull/5653). Could you test on master and see if it handles this correctly?

I'm sorry @kinglozzer I'm a dunce :hankey: I realized that as I was vacillating between text vs. HTML for a field, I ultimately decided upon text, setup the TextareaField in ->getCMSFields() but absentmindedly left the DB field type itself as HTMLText which will obviously will have a ->forTemplate() that doesn't need to convert  newlines. Your response made me look backwards at the code to confirm exactly what my type was here (I was assuming Text, which I don't often use) and ... d'oh.
Place :hand: on :grimacing:


Support building current trace context with scope decorators

Right now, we allow people to override CurrentTraceContext, which is cool. we can do things like switch propagation etc. Would be neat if things like SLF4J could be made additive to whatever the primary context backend choice is. This would prevent people from knowing too much and/or excessive wrapping.
For example, we can provide a bean of type CurrentTraceContext.Builder which has a list builder method for ScopeDecorator. These scope decorators can implement SLF4J, logging or even things like census integration.
For example, we could list injection to create the final CurrentTraceContext:
ex
@Bean CurrentTraceContext currentTraceContext(CurrentTraceContext.Builder builder, List<ScopeDecorator> scopeDecorators) {
  for (ScopeDecorator scopeDecorator : scopeDecorators) {
    builder.addScopeDecorator(scopeDecorator);
  }
  return builder.build();
}

cc @bogdandrutu @anuraaga @trustin @kojilin
See https://github.com/bogdandrutu/oc-spring-boot
See https://github.com/openzipkin/brave/pull/683
See https://github.com/line/armeria/pull/1322

I fully agree here, one of the things that has kept me from implementing baggage in our system is the need for a custom implementation to handle the MDC/Propagation case


Fixes ActionSet not correctly updating its bound

Related Issue
Fixed #3637
Possibly fixed #3784
Description
Updated variable names to better reflect what those variable names do.
Added a layout update logic, so scroll view will have more precise dimension upon rendering.
How Verified
Took snapshots
Before

After

Notice the frame differences for buttons between two images.
Test with other cards for regression checks.
Microsoft Reviewers: Open in CodeFlow

@nesalang heads up -- there's some conflicts here we may need to resolve. Do we have the same conflicts when you port these over to the 1.2.7 cherrypicked PR?


BUG

Program version
3.1.0
Operating System
Windows
Type
Cli
Service
Funimation
Command used
aniDL --service "crunchy" -s %N% -e %E% --skipdl --novids --noaudio --dlsubs ar
Show ID
GRGGCV3PJ
Episode
2
Console Output
=== Multi Downloader NX 3.1.0 ===
[INFO] USER: ******** (@.)
[ERROR] HTTPError 503: Service Unavailable
[ERROR] Body: no healthy upstream
[ERROR] Authentication CMS token failed!
[ERROR] Authentication required!
Press any key to continue . . .
Additional Information
No response

I think there is problem with crunchyroll itself, on twitter People are writing that the app also doesnt work

ya i think their API is down, you can still watch videos if you click on the links from the simulcastcalendar directly however navigating to other seasons etc is down atm..


Hey @sasharevzin Thanks for the PR!


With include_javascripts, is there a way to get just the URL?

I need to use $.getScript, To use getScript I would need just the URL from Jammit. Is this possible?
$.getScript('ajax/test.js', function(data, textStatus){
console.log('Load was performed.');
});
Thanks

Here's what i did:
<%= javascript_path (Jammit.asset_url("desktop", :js)) %>
Assumes you have :desktop JavaScript packages in config/assets.yml
cheers


Cannot read properties of undefined (reading 'start')

Any idea what is causing this?

It isn't related to this package.
Maybe this one: https://www.npmjs.com/package/postcss-sass ?


dev question: how does the crash handler work for MacOSX?

How does BSF catch crashes on MacOSX? I don't find any function like:
catch_mach_exception_raise_state or mach_excServer.c
? :)

It catches signals, see BsUnixCrashHandler.cpp.  It's not my primary OS so I'm not aware of intricacies of its error handling, but if there's a better macOS specific way to do it, I'd consider adding it.

Thanks. I wonder why that this works ;)

Here is a link to how mach exception handler works:
https://www.mikeash.com/pyblog/friday-qa-2013-01-11-mach-exception-handlers.html
On an app i develop i used signals as well, but this doesn't prevent the default crash handler of MacOSX to pop up. So, i wonder why this work on your framework.

Do you know what determines with the apple crash reporter pops up? I'm assuming it then eats the signals? Did you experience issues with bsf where a crash wasn't properly handled?

I didn't not find any issue during debugging of bsf with the crash handler.

It might be nice to support Mach error handling, especially if it provides nicer error messages. Some of the crashes triggered from objC can be confusing. Although it's not a high priority for me.


Single Row Week Calendar Malfunction

IMPORTANT - BEFORE CREATING YOUR ISSUE PLEASE READ THE FOLLOWING:

PLEASE STATE THE VERSION NUMBER you are using. 7.0
Are you using the latest version of JTAppleCalendar? Latest version is currently 7.1.5 No
PROVIDE ENOUGH INPORMATION SO I CAN RECREATE THE PROBLEM.

So my task is pretty straight forward I am trying to create a one week calendar for my event app.  The one week calendar should only show three days at a time. As we scroll to the right or to the left three more should show.
I want my calendar to take something similar to the appearance in this picture

What mine currently looks like is this

Now that I have stated what Im trying to accomplish I can talk about my issues.
I have followed the general tutorial to setup the calendar and even followed the link provided by @patchthecode to setup one week calendar and it seemingly works but here are the current problems.
It should be noted that my one week calendar is constrained inside a collectionviewCell to get that appearance

When I open the app it scrolls to August as it should but it doesn't select any date.
It seems like a date is selected only if I select the date next to it.
When I interact with any part of the calendar it auto-scrolls back to January and I can never get back to August.
There are 5 days appearing and I only want three at a time
Is there a built-in function to get the day of the week for the corresponding day to put it above the current day number.

I have included my code for review. Im sort of at the frustrated stage so I can't really see what's wrong any help is greatly appreciated.
import Foundation
import JTAppleCalendar

class CalendarHeader: UICollectionViewCell {
    let cellID = "cellID"
    let headerID = "headerID"
    let formatter = DateFormatter()

    //day labels for calendar

    let todayLabel : UILabel =  {
        let todayLabel = UILabel()
        todayLabel.text = "Today"
        todayLabel.textAlignment = .center
        todayLabel.font = UIFont(name:"NoirPro-Medium", size: 15)
        todayLabel.adjustsFontForContentSizeCategory = true
        return todayLabel
    }()

    let yearLabel : UILabel =  {
        let yearLabel = UILabel()
        yearLabel.textAlignment = .center
        yearLabel.font = UIFont(name:"NoirPro-Light", size: 15)
        yearLabel.adjustsFontForContentSizeCategory = true
        return yearLabel
    }()
    let monthLabel : UILabel =  {
        let monthLabel = UILabel()
        monthLabel.textAlignment = .center
        monthLabel.font = UIFont(name:"NoirPro-Light", size: 15)
        monthLabel.adjustsFontForContentSizeCategory = true
        return monthLabel
    }()

    //collectionview for calendar view
    let calendarCollectionView: JTAppleCalendarView = {
        let cv = JTAppleCalendarView(frame: .zero)
        cv.scrollDirection = .horizontal
        cv.allowsSelection = true
        cv.allowsMultipleSelection = false
        cv.backgroundColor = UIColor.rgb(red: 255, green: 255, blue: 255)
        cv.minimumInteritemSpacing = 0
        cv.minimumLineSpacing = 0
        cv.scrollingMode = .stopAtEachCalendarFrame
        return cv
    }()

    override init(frame: CGRect) {
        super.init(frame: frame)
        setupViews()
    }

    @objc func setupViews(){
        addSubview(todayLabel)
        addSubview(monthLabel)
        addSubview(yearLabel)
        addSubview(calendarCollectionView)

        calendarCollectionView.scrollToDate( Date(), animateScroll: false )
        calendarCollectionView.selectDates([Date()])
        calendarCollectionView.visibleDates { (visibleDates) in
            self.setupViewsOfCalendar(from: visibleDates)

        }
        calendarCollectionView.isPagingEnabled = true
        calendarCollectionView.calendarDataSource = self
        calendarCollectionView.calendarDelegate = self
        calendarCollectionView.showsHorizontalScrollIndicator = false
        calendarCollectionView.showsVerticalScrollIndicator = false

        todayLabel.snp.makeConstraints { (make) in
            make.left.equalTo(self.snp.left).offset(5)
            make.top.equalTo(self.snp.top).offset(7)
        }
        monthLabel.snp.makeConstraints({ (make) in
            make.left.equalTo(todayLabel.snp.right).offset(5)
            make.top.equalTo(self.snp.top).offset(7)
        })
        yearLabel.snp.makeConstraints({ (make) in
            make.left.equalTo(monthLabel.snp.right).offset(5)
            make.top.equalTo(self.snp.top).offset(7)
        })

        calendarCollectionView.register(CalendarCell.self, forCellWithReuseIdentifier: cellID)
        calendarCollectionView.snp.makeConstraints { (make) in
            make.left.equalTo(yearLabel.snp.right).offset(5)
            make.right.equalTo(self.snp.right)
            make.height.equalTo(self.bounds.height)
        }
    }

    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
}

extension CalendarHeader: JTAppleCalendarViewDataSource {

    func calendar(_ calendar: JTAppleCalendarView, willDisplay cell: JTAppleCell, forItemAt date: Date, cellState: CellState, indexPath: IndexPath) {
        let cell1 = cell as! CalendarCell
        cell1.dayNumberLabel.text = cellState.text
        cell1.dayOfWeekLabel.text = cellState.text
        handleCellTextColor(view: cell1, cellState: cellState)
    }

    func setupViewsOfCalendar(from visibleDates: DateSegmentInfo){
        guard let date = visibleDates.monthDates.first?.date else {
            return
        }
        self.formatter.dateFormat = "yyyy"
        self.yearLabel.text = self.formatter.string(from: date)
        self.formatter.dateFormat = "MMMM"
        self.monthLabel.text = self.formatter.string(from: date)
    }

    func configureCalendar(_ calendar: JTAppleCalendarView) -> ConfigurationParameters {
        let year = Calendar.current.component(.year, from: Date())
        let firstOfYear = Calendar.current.date(from: DateComponents(year: year, month: 1, day: 1))
        let lastOfYear = Calendar.current.date(from: DateComponents(year: year, month: 12, day: 13))
        let parameters = ConfigurationParameters(startDate: firstOfYear!, endDate: lastOfYear!, numberOfRows: 1, calendar: Calendar.current, generateInDates: .forFirstMonthOnly, generateOutDates: .off, firstDayOfWeek: .sunday, hasStrictBoundaries: false)
        return parameters
    }

}

extension CalendarHeader: JTAppleCalendarViewDelegate {

    func handleCellTextColor(view: JTAppleCell, cellState: CellState){
        guard let validCell = view as? CalendarCell else {
            return
        }
        if cellState.isSelected {
            validCell.dayNumberLabel.textColor = UIColor.black
            validCell.dayOfWeekLabel.textColor = UIColor.black

        }else {
            validCell.dayOfWeekLabel.font = UIFont(name:"NoirPro-Light", size: 13)
             validCell.dayOfWeekLabel.textColor = UIColor.rgb(red: 207, green: 207, blue: 207)
             validCell.dayNumberLabel.font = UIFont(name:"NoirPro-Light", size: 13)
            validCell.dayNumberLabel.textColor = UIColor.rgb(red: 207, green: 207, blue: 207)
        }

    }

    func calendar(_ calendar: JTAppleCalendarView, cellForItemAt date: Date, cellState: CellState, indexPath: IndexPath) -> JTAppleCell {
        let myCustomCell = calendar.dequeueReusableCell(withReuseIdentifier: cellID, for: indexPath) as! CalendarCell
        myCustomCell.dayNumberLabel.text = cellState.text
        myCustomCell.dayOfWeekLabel.text = cellState.text
        return myCustomCell
    }

    func calendar(_ calendar: JTAppleCalendarView, didSelectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        guard let validCell = cell as? CalendarCell else {
            return
        }
        handleCellTextColor(view: validCell, cellState: cellState)

    }

    func calendar(_ calendar: JTAppleCalendarView, didDeselectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        guard let validCell = cell as? CalendarCell else {
            return
        }
        handleCellTextColor(view: validCell, cellState: cellState)
    }

    func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {
        setupViewsOfCalendar(from: visibleDates)
    }

}

When I open the app it scrolls to August as it should but it doesn't select any date.
calendarView.selectDate( theDate )

I think I went on the wrong track when you opened multiple issues under one issue.
Its hard to keep track.
is there any part of this problem that still needs fixing?


[AutoPR iotcentral/resource-manager] Update iotcentral.json

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7896

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/9047


--save-dev?

I guess this might be more of an issue for verb, but why is --save-dev the recommended installation? This seems to be more of a --save type of lib

thanks looks like I ran verb on this when the default templates weren't setup right.

fixed. thanks again


Waiter InvalidationCompleted failed: Max attempts exceeded

Deploying StaticSite fails on cache invalidation:
Received response status [FAILED] from custom resource. Message returned: Waiter InvalidationCompleted failed: Max attempts exceeded

We should gracefully handle this error.

Dan https://serverless-stack.slack.com/archives/C01JG3B20RY/p1645060768947149

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Serious slow asp.net web api

https://github.com/aspnet/AspNetCoreModule/issues/87
Running load test on my localhost with visual studio, debug or release barely have any difference
Api which barely do anything, from the template
/api/values

In azure app service S1 Small, only 150-200 requests per sec?

Api which doing connection to sql server express, localhost

azure app service, azure sql S3

@sebastienros

Can you share the application you are testing, I could verify that there is nothing obviously slowing down your requests. Maybe your SQL query is slow, maybe you are using Console logging, maybe something else, but if I can see the code you are stress testing that would be better understand where your issue is.
We are doing extensive perf test on many difference scenarios, see https://aka.ms/aspnet/benchmarks, including database access, and we are confident that it can be very fast. Seeing your code would help excluding some mistakes.

Its ValuesController from web api template
I suspect company network + antivirus slowing down the load test above.

Here I test from home network to azure S1 Small

WebSurge 2 threads
70 requests/sec for ValuesController
WebSurge 4 threads
140 requests/sec for ValuesController
Make a test with database connection
WebSurge 4 threads
40 requests/sec
its no where close TechEmpower benchmark

Can you run your benchmark in azure app service S1 Small?

Can you run your benchmark in azure app service S1 Small?

Are you asking us? We already measure on Azure VMs D3v2. It's not surprising to have such numbers on S1 Small in App Service, it's the smallest machine available, and App Service has some perf impact compared to VMs.

Yes
Is it too much to ask?

Closing because there is no further action planned for this issue.


Set-AtWsTicketCost - Update status

Hi,
Thanks again for the modifications you made to make this work in Azure Automations.
So far it's working great.
Currently i'm working on a Office 365 PowerApp witch is used to let the customers sign for received prodcucts. Every hour the runbooks runs, gets all ticketCosts and saves them to a sharepoint list.
The PowerApp allows me to select the items for the selected customer, and make them sign.

When the customer has signed, I want to update the Quantity and the status.
Updating the quantity works, the status however isn't saved.
My question is, can this be done by the API, or is it just not the way it workes.
Sample code, only saving the quantity:
$test = Get-AtwsTicketCost -id 29689473
$test

$test.UnitQuantity = 2
$test.Status = 5

$test | Set-AtwsTicketCost

Great work! It is nice to see the module in use! :) That is a very nice idea! Would you be able to share some of the steps you have gone through to get this far in a blog post somewhere? Or you could publish it here alongside the module documentation?
Unfortunately the Status field is Read-Only according to the API documentation (page 346) so you cannot update the cost status through the API. At first look it seems to be because the cost status is integrated with procurement.
Are all the costs connected to the same ticket? Maybe you'd be able to solve it with Ticket status? Or a ticket UDF?

Are all the costs connected to the same ticket? Maybe you'd be able to solve it with Ticket status? Or a ticket UDF?
Hmm, I guess i'm out of luck. If I assume there is one item per ticket it still wouldn't give me an option to update the status. The tickets (opportunities) for items waiting to be delivered are already closed.
I've tried looking at workflow rules in AT to see if it's possible to use an UDF as a trigger, but in workflow rules the option to set an item shipped is absent as well.

I think I go for the workaround; when an item is marked as delivered / shipped in the PowerApp, I send a daily mail will all delivered items to the sales manager. They have to manually update the item in the delivery / shipping overview.
Thanks again for your support. I'm thinking about writing a blog to publish some of the code / solutions.

Hmm starting to dislike AutoTask more and more. I don't get why their API is so limited.
We offer some some services, licensed on a per-user base.
Using flow we get the active users from a customer's tennant, write the amount of active users back to SharePoint.
My idea was to run a runbook every month, updating contracts when the amount of active users has been changed. So far so good.

You can do this, but the syntax is not what you expect and the object you need is not ContractServiceUnit. The datamodel is probably quite complex beneath all this. There are no way, neither through the API, nor the GUI to just modify the contract service units directly. But you can add or subtract users (service units) using New-AtwsContractServiceAdjustment
Hope that helps!

You are amazing! Exactly what I was looking for! Now I'm able to automatically adjust the contract items :) Thanks a lot!!


Invalid argument type ImageProxy when trying to use native module and passing the frame as argument🐛

What were you trying to do?
I am trying to create something of my own, i want to get the frames in native java, i am passing the frame as argument to my native code, as i have checked out the the projects built on this project, the type of Frame is ImageProxy in java , i have done the necessary imports, but when i try to trigger the native function, i get error that the argument i passed to my native function is of invalid type
Reproduceable Code
// Java code
 @ReactMethod
    public void AIProcessor(ImageProxy frame) {
        Log.d("AIProcess", "Triggered");
}

// React Native Code
AIProcessorModule.AIProcessor(frame)

What happened instead?
The app crashes and in android studio logcat i found out the error is coming as invalid argument
Relevant log output
No response
Device
Android 12
VisionCamera Version
latest
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Hey! I've rewritten the entire Android codebase of VisionCamera from CameraX to Camera2 in the efforts of ✨ VisionCamera V3.
Note: In this version, I don't use CameraX anymore, so ImageProxy is not used anymore. Now it's just Frame.image :)
I just now completed the Camera2 rewrite and I believe the core structure is running, but there might be some edge cases to iron out. Can you try and test the PR #1674 for me to see if you can still reproduce this issue here?
Here's an instruction on how you can test that: https://github.com/mrousavy/react-native-vision-camera/pull/1674#issuecomment-1684104217
If the issue cannot be reproduced with that version/PR anymore, then hoorayy, I fixed it! 🎉
Otherwise please let me know and I'll keep this issue open to keep track of it.
Thank you!


preserve attributes when copying files in rebar_utils:cp_r for unix

Does anyone know why we shell out to cp -R instead of using ec_file:copy/3?
This PR makes it so copying preserves the modified time of the file, this fixes a bug with extra_src where it would recompile all modules in the extra src dirs because the compiler considered them modified.

git blame seems to think that function has been around since 2009. so no good reason probably


AttributeError: 'Session' object has no attribute 'create_client'

This is a new  error for me with the 0.4.2 release (no issue with 0.4.0). Tried on python 3.5.4 and 3.6.8. I get an attribute error when creating an S3FileSystem object as follows:
session = boto3.Session(profile_name=profile_name)
fs = s3fs.S3FileSystem(anon=False, session=session)

Full traceback:
AttributeError                            Traceback (most recent call last)
<ipython-input-5-a2a74472519c> in <module>
----> 1 fs = s3fs.S3FileSystem(anon=False, session=session)

~/bioinf-whole-genome-analysis/.direnv/python-3.5.4/lib/python3.5/site-packages/fsspec/spec.py in __call__(cls, *args, **kwargs)
     52             return cls._cache[token]
     53         else:
---> 54             obj = super().__call__(*args, **kwargs)
     55             # Setting _fs_token here causes some static linters to complain.
     56             obj._fs_token_ = token

~/bioinf-whole-genome-analysis/.direnv/python-3.5.4/lib/python3.5/site-packages/s3fs/core.py in __init__(self, anon, key, secret, token, use_ssl, client_kwargs, requester_pays, default_block_size, default_fill_cache, default_cache_type, version_aware, config_kwargs, s3_additional_kwargs, session, username, password, **kwargs)
    185         self.s3_additional_kwargs = s3_additional_kwargs or {}
    186         self.use_ssl = use_ssl
--> 187         self.s3 = self.connect()
    188         self._kwargs_helper = ParamKwargsHelper(self.s3)
    189         super().__init__(**super_kwargs)

~/bioinf-whole-genome-analysis/.direnv/python-3.5.4/lib/python3.5/site-packages/s3fs/core.py in connect(self, refresh)
    290         else:
    291             conf = Config(**config_kwargs)
--> 292             self.s3 = self.session.create_client('s3', aws_access_key_id=self.key, aws_secret_access_key=self.secret, aws_session_token=self.token, config=conf, use_ssl=ssl,
    293                                         **self.client_kwargs)
    294         return self.s3

AttributeError: 'Session' object has no attribute 'create_client'

It should have been more widely documented: s3fs now depends upon botocore rather than boto3. This was done in #277 in order to allow compatibility with async backends. Therefore, session should be a botocore.session.Session (which is stated in the docstring, but is a change from the previous requirement).

Thank you. Knowing that is very helpful and the change works perfectly!


Extend AD forest recovery guide with steps for gMSA

Step 4, 10 and 11 in the initial recovery page focus on resetting possibly compromised passwords for accounts, AD-trusts, KDC and DCs during AD forest recovery.
All gMSA accounts have passwords calculated using a static KDC Root Key. This page should include the necessary steps and/or link to a guide with step by step-instructions to reset this as well. A new KDC root key has to be generated and all gMSA accounts reestablished since you can't migrate existing accounts to the new root key AFAIK.
Should the old root key be deleted and if so, how?

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

ID: cfa1b96d-309f-ec35-6365-48baaf7fcb25
Version Independent ID: d3fbd66c-0abf-a078-faf0-d725b2f2ba04
Content: AD Forest Recovery - Perform initial recovery
Content Source: WindowsServerDocs/identity/ad-ds/manage/AD-Forest-Recovery-Perform-initial-recovery.md
Product: windows-server
Technology: ad-ds
GitHub Login: @iainfoulds
Microsoft Alias: daveba

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


etcd-kubernetes-resources-count-exporter

Issue: https://github.com/giantswarm/giantswarm/issues/25482

This PR:

adds/changes/removes etc

Testing
Description on how default-apps-cloud-director can be tested.

[ ] fresh install works
[ ] upgrade from previous version works

Checklist

[x] Update changelog in CHANGELOG.md, check all commits are in it before release (renovate included).
[x] Make sure values.yaml and values.schema.json are valid.

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites


Chrome 55+ unprefixed clip-path

https://blog.chromium.org/2016/10/chrome-55-beta-input-handling.html:

CSS Clipping Path properties no longer require the webkit prefix.

Questions:

What about Opera (41)?
Should we update http://caniuse.com/#feat=css-masks?

Thanks! Yes, I'll update Opera too, I don't think I've ever seen a CSS change in Blink that didn't make it to Opera.
What kind of change would you feel is needed for css-masks?

I see, will keep that in mind for the future!
I thought maybe they removed the prefix for css-masks too (as clip-path is listed as a subfeature), but just checked and they didn't, so no change needed.


[SUPPORT] New to Contributing

I am new to contributing and come from a Pre-Sales Engineering background.  I want to start getting my hands on OpenAI contributions and creating exciting new products/features/prototypes in the OpenAI space.  Any suggestions, guidance, steps, etc. are greatly appreciated.
Cheers!

Heya, welcome to the OpenAI community!
For help with OpenAI services it might be best to check out the cookbook itself to get started. It's full of practical examples and use cases that can help you understand the possibilities. The API documentation is also a great resource to familiarize yourself with the APIs 👍
For help with contributing on GitHub, maybe check out this guide
Mind if I ask what specifically you're trying to do?