Panagram2

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #1537
Replace issue_no with the issue number which is fixed in this PR
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[ ] My code follows the style guidelines(Clean Code) of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@SrihariTadala comment out the parts which are not a part of the program and add main function

@SrihariTadala not updated

Since no update I am closing it


Add "Dockerize your angular app" option to the setup wizard.

I would like to be prompted in the ngx setup wizard if I would like to "Dockerize" the angular app. This would create docker files for dev and production. In the case of prod; ask which prod web server I would like to use (Apache, nginx etc...). We could have a task (npm start --docker) which builds the image and runs it inside of a docker container mounting the local src, node_module folders so that we can hot-reload changes while writing our app.
I'm submitting a...

[ ] Bug report
[x] Feature request
[ ] Documentation issue or request

Current behaviour

Currently there is no support for dockerizing angular apps.
Expected behaviour

Would like to automate the Dockerzing of angular app for dev and prod purposes.

Sure that would be nice, I think it would be a great fit for an add-on 👍

OK, i'll take it...

@kenan435 Are you thinking of using nginx to host the app container or expressJs, or something else?

Closing this one, as it seems official support via ng CLI is coming: https://github.com/angular/angular-cli/blob/master/docs/design/docker-deploy.md

@sinedied your link is dead. I think, they moved the functionality to ngx-deploy-docker, but I found no documentation about it. Only this commit: https://github.com/angular/angular-cli/commit/245e669d631eabf3476169aa6e9208766503bcd6
Maybe rocket can just add the ngx-deploy-docker?

This might be a good idea, since official support seems to have been dropped.
Are you willing to submit a PR for it?
This could be added to other deployment option, and as ngx-deploy-docker supports ng add it should be a straightforward addition.

I'm discovered ngx-rocket and ngx-deploy-docker today for the first time.  :D
I will try it, but I can't make any promises.


Changing GF in the menu

I Need Help With Changing GF in the intro menu
I need help with changing GF in the intro menu, Thats it, i tried just replacing the gfbumpin.png and .xml but it doesnt work, i only need help with changing it.
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

That's because it's not gfBumpin, its gfDanceTitle.


BUG: fix the repr for Function objects

This also cleans up some of the metaclass stuff for python 3.

ping @cpcloud about the metaclass stuff


wordgrinder: 0.7.2 -> 0.8

Motivation for this change
https://github.com/davidgiven/wordgrinder/releases/tag/0.8
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.


Fixed long press Repeated keys Situation

when a key is Long pressed, it leads to repeated keys. which sounds like a rotating barrel gun

What is taking so long to merge it?


Configure Azure Pipelines

This configuration builds the application and produces builds for both Debug and Release configurations.

 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: billkindle sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Tidy up a couple of small things

🚀 Deployed on https://66283d1367d74f2ce26818c5--alexwlchan.netlify.app


Use swoft-db package out of the Swoft framework

Q
A

Bug report?
no

Feature request?
no

Hi! I'm planing to use https://github.com/swoft-cloud/swoft-db package out the Swoft framework. My question is how and where can I set the configuration? Because right now it uses the app/bean.php when you use the Swoft framework.
Thanks in advance.

hi @ljfreelancer88  swoft-db Maybe it doesn't work correctly from the outside, it depends on swoft's container. Need you to do some renovation


Encoding is not support French text from thread_settings.

I have setup the greeting text in english and french but the problem is some of french characters are not being displayed correctly : é = � , è=� , à = �
controller.api.thread_settings.greeting([{
                "locale": "default",
        "text": "Sent to you."

            }, {
                "locale": "fr_FR",
        "text": "Envoyé à toi ."

        }]);

You french characters are in ISO-8859-1 encoding, try saving your file as UTF-8.

Done. It is working well now. Thanks.


Make drop functions queries pg11 compatible

Related #14657

@Algunenano If you want, you can do the accceptance of this changes, after the CR, with your PG11 part :)


update the library to handle a _Logging event or a Logging event

The _Logging event stream is being renamed to Logging; we should update this library to allow it to handle either name.
cc @bkonyi, fyi

To update, this library currently doesn't handle the _Logging events. We'll just need to re-generate it once the spec documents the Logging event.


Dependency updater

When a package is updated, and other packages in this project depend on it, also update those packages with a patch bump/changeset.
If there are more substantial changes (which require code changes in the dependent package) I would assume the developer would create a changeset in that package with a changeset of its own to give it a bigger bump if necessary. Like: "Minor: Update nodejs-ext with new "feature" function", or "Patch: Update to new extension interface"

Made for Node.js and Ruby packages in 0.5.0. Elixir packages are next.

First add it to the separate Elixir repos and then solving the updater thing later.


Express hangs if no response is sent

If I create a route that gives no response, Express will hang. I understand that this is incorrect code, but as someone who teaches programming, it is a mistake I have seen almost everyone make, and which can be very confusing to debug for a beginner.
To make this much easier for users of the library to debug, it could be logical to give a 500 with the error message "The route callback did not send a response". Honestly, any error message would be more helpful than a freeze while waitng for a response that will never come.
This issue can be trivially reproduced by the following code:
import express from 'express';

let app = express();

app.get('/', (req, res) => {
  console.log("Just about to hang forever");
});

app.listen(5000, () => console.log('started on http://localhost:5000/'));

Attempting to load the URL in a web browser will lead to the server freezing, with no response being given.

Hello, and thank you for the issue. Yes, the Node.js HTTP servr that underlies Express.js does this: your code will need to send a response it it will wait. The reason of course ia how long will Node.js know to wait? If it is immediately after the function above runs, then there would be no way to first retrieve data from a database and then respond with that data.
As far as I know, there is no method to solve this problem apart from including a timeout middleware in your app that will send a timeout when you take too long (and you'll need to know what "too long" is for your given app).
If you know a way that this could be implemented, I would encourage either a pull request here or ideally a pull request to Node.js which is where the behavior originates.

So to check, is Express aware of when we've reached the end of the available callbacks? If so perhaps it could wait until then. That being said I wouldn't regard myself as an expert on JS, given I had no experience with it until 3 months ago when the university course I teach changed over from Python. I would have thought that it'd be relatively easy to tell it a response has been sent and whether or not our app is in a state where it can possibly send a response. If that's not the case, I'll definitely move this issue over to whatever repo is used for the underlying HTTP server that Node uses, since this is a pretty important issue imo.

So to check, is Express aware of when we've reached the end of the available callbacks?

That's the unfortunate part of the async nature of Node.js: it doesn't know when you're done unless you tell it so. Express.js (just like Node.js in the example above) knows it finished called the function, but your function could have scheduled async tasks like db calls, https requests, etc. that will eventually later send a response.
AFAIK there is no method to know that the user's code is still intending to send a response or not at a later time. The Node.js example I posted above demos that as well, as the underlying Node.js HTTP server has the same behavior.
Express.js sits on top of the Node.js HTTP server, which means it inherits all the limitations of that server. It adds some fancy accessors to the request and response objects to make some actions easier (like res.send doing the work of calling res.setHeader, res.write, and res.end for you) and it provides routing (so you can say app.get('/foo', ...) instead of if (req.method === 'GET' && req.url === '/foo') { ... } but that is mostly the extent of what Express.js is doing, if that makes sense. The heavy lifting is actually all done by the Node.js HTTP server: https://nodejs.org/dist/latest-v19.x/docs/api/http.html#class-httpserver
I hope that helps. I'm going to close this because AFAIK it is not possible to implement your ask, at least at the Express.js level, outside of the timeout middleware kind of method. If there is, I would be happy to accept a pull request implementing such a thing, however 👍

Thanks for the clarifications - I'll just implement the timeout middleware.


Deno edge function encounters unexpected bug with supabase-js client in local deployment

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Description
I'm able to deploy a deno edge function locally and run a test without problems using supabase functions serve --env-file ./supabase/functions/.env.local. The supabase-js client library loads fine when I run deno test --allow-all ./tests/jl-call-test.ts --env=./functions/.env.local. However, the functions server shuts down with the following error:
Segmentation fault (core dumped)
error running container: exit 139

which happens at
const client = createClient(
      supabaseUrl,
      supabaseKey, {
        auth: {
          autoRefreshToken: false,
          persistSession: false,
          detectSessionInUrl: false,
        },
        db: {
          schema: 'custom_schema_name'
        }
      })

As I said the above snippet runs perfectly well in my unit test. Running with the --debug flag outputs no extra information.
Weirdly, if I remove the supabase-js client logic, my function is responsive in local development and works as expected. Any idea what might be going on here? Thanks 🫶
System information

OS: Linux Ubuntu 22.04
Version of supabase-js: 2.38.4, but same error occurs in older versions
Version of Deno: 0.192.0, but same error occurs in older versions
Version of Docker: 24.0.4
Version of Supabase 1.110.1

the same error occurs during local development

Same here

maybe on mac i don't know

I'm seeing exact same segfaulting on local deno edge functions on a Macbook M3 Pro


IPEX Duplicate importer V2

Description

Replace importer after importing ipex, then revert it back before converting model.
Simply check to improve performance.

1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] N/A
[ ] Unit test
[ ] Application test
[ ] Document test
[ ] ...

5. New dependencies

[ ] New Python dependencies
- Dependency1
- Dependency2
- ...
[ ] New Java/Scala dependencies and their license
- Dependency1 and license1
- Dependency2 and license2
- ...

Add calling package to avoid error when using intel_extension_for_deepspeed, e.g.,
from intel_extension_for_deepspeed import XPU_Accelerator

Or
import deepspeed

@jason-dai added a control env IPEX_DUPLICATE_CHECKER, default is true. We can turn it off with IPEX_DUPLICATE_CHECKER=1

Failed unit tests are not related to this PR


Conversation with Google TTS and STT

Here's my approach for TTS and STT. It only evaluates what is said after a significant (10dB) volume difference --> no evaluations of background noise. It has a few more tweaks and works better than the other approach in various ways.
The actual connection to the server api still has to be implemented in act().

Yay... perfect =) Merging ^^


Can return a failure if the CBD completed Synchronously

You say
> Note that the NTSTATUS value returned by FltPerformAsynchronousIo indicates only whether the I/O operation was successfully initiated.
This is not correct.  I have a dump (if Craig  cares) where a perform has failed STATUS_OPLOCK_NOT_GRANTED and I have gone to free the contex , only to discover that it has been freed by my CallbackRoutine.
Good luck documenting this: "some of the time a failure status means that the request hasn't been sent, but sometimes it has been".

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

ID: ea4590d7-4f8a-c1f5-376a-3af10ed95c40
Version Independent ID: 632bd07c-b98e-7c71-c46c-8e742660b0a9
Content: FltPerformAsynchronousIo function (fltkernel.h) - Windows drivers
Content Source: wdk-ddi-src/content/fltkernel/nf-fltkernel-fltperformasynchronousio.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

@rodwiddowson , thanks for reporting this. I've updated the page to hopefully be more clear about status codes and context freeing (which can totally be done in the callback routine)!

looks fantastic.  Thanks @lorihollasch


Check response before using it

Do we really have to check if response is falsy in shouldRetryRequest? Does this mean there are cases where requestretry (or request) call shouldRetryRequest with a falsy response argument?
If they do so, shouldn't we check for (!response) in getRetryReason and return a proper reason? I mean, response being falsy looks like a case where we want to retry the request to me.
I may misunderstand something here.

Okay, I took a quick look into the requestretry source.
The first thing getRetryReason does is passing response and error to retryrequest.RetryStrategies.HTTPOrNetworkError(). This function checks if response is falsy, so we probably shouldn't add any own checks to that logic.

It needs some logic for this as currently twitch-helix crashes due to this being null and then being accessed.
I'll post the exception if needed currently I'm running off my fork instead of v1.0.0 as it just crashes.

You seem to be right. I now added a truthy check to shouldRetryRequest before calling this.log(). Should fix it.
Sorry for the waiting time.


Azure-NPM: Compatibility with kube-proxy in IPVS mode

What happened:
When NPM is enabled on a cluster with kube-proxy configured for IPVS instead of IPTABLES, e.g.:
Containers:
  kube-proxy:
    Command:
      --proxy-mode=ipvs
      --ipvs-scheduler=lc
      --ipvs-tcp-timeout=900s
      --ipvs-tcpfin-timeout=120s
      --ipvs-udp-timeout=30s

Traffic against a Service IP fails.
What you expected to happen:
Traffic to Services are successful, as would normally be the case with IPTables.
How to reproduce it:
Create a Kubernetes cluster with IPVS. Currently this is (or will shortly be) available on AKS via KubeProxyConfigurationPreview with an ARM call that has the contents:
{
   "apiVersion":"2022-08-02-preview",
   "type":"Microsoft.ContainerService/managedClusters",
   "properties":{
      "networkProfile":{
         "kubeProxyConfig":{
            "enabled":true,
            "mode":"IPVS",
            "ipvsConfig":{
               "scheduler":"LeastConnection",
               "TCPTimeoutSeconds":900,
               "TCPFINTimeoutSeconds":120,
               "UDPTimeoutSeconds":30
            }
         }
      }
   }
}

Otherwise, will need to configure a cluster with manually setting the kube-proxy arguments to the above.
Kubernetes Version:
v1.23.8
Kernel (e.g. uname -a):
5.4.0-1090-azure #95~18.04.1-Ubuntu SMP Sun Aug 14 20:09:27 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days

In progress


Can I detect if a client is disconnected within a State's method?

The page of code shown below will continue the while loop even if the browser is reloaded and disconnected from the client. What are possible ways to detect the disconnection from the client and break the while loop?
import reflex as rx

class State(rx.State):
    @rx.background
    async def loop_function(self):
        while True:
            # I want to terminate this loop when disconnected from the client.
            foo()

@rx.page(on_load=State.loop_function)
def index():
    return rx.text("Hello")

app = rx.App()

If the PR I linked is merged, you will be able to terminate the break out of the loop with the following code :
class State(rx.State):
    @rx.background
    async def loop_function(self):
        while True:
            if self.router.session.client_token not in app.event_namespace.token_to_sid:
                break
            await asyncio.sleep(2)

@rx.page(on_load=State.loop_function)
def index():
    return rx.text("Hello")

The final API might change a little but the logic should be the same.

Related PR: https://github.com/reflex-dev/reflex/pull/2671

Thank you for your great work.


pentacles with bedrock instead of candle

when im trying to do a ritual its say its false and when i'm openning the book it says to place bedrock instead of candle
step 1: i went to dictiobary of spirit
step 2: open Pentacles
step 3:i watched all the pentacles we're needed a bedrock instead of candles
I wanted to have the candle instead

Occultism Version:1.120.0.jar
Minecraft Version:1.20.1
Modpack Link and Version, or list of mods:
alexscaves-1.0.9.jar
balm-forge-1.20.1-7.2.1.jar
blockui-1.20.1-0.0.101-ALPHA.jar
citadel-2.4.9-1.20.1.jar
domum_ornamentum-1.20-1.0.150-BETA-universal.jar
essential_1-3-0-6_forge_1-20-1.jar
iceandfire-2.1.13-1.20.1-beta-2.jar
journeymap-1.20.1-5.9.18-forge.jar
minecolonies-1.20.1-1.1.452-RELEASE.jar
modonomicon-1.20.1-forge-1.60.0.jar
moonlight-1.20-2.10.0-forge.jar
octolib-1.20.1-0.2.jar
relics-1.20.1-0.5.3.jar
sophisticatedbackpacks-1.20.1-<IP_ADDRESS>2.jar
Sophisticated Core
storagedrawers-1.20.1-12.0.2.jar
structurize-1.20.1-1.0.662-BETA.jar
supplementaries-1.20-2.8.0.jar
the_bumblezone-7.2.5+1.20.1-forge.jar
Vampirism-1.20.1-1.10.6.jar
Werewolves-1.20.1-<IP_ADDRESS>.jar
ars_elemental-1.20.1-<IP_ADDRESS>.jar
ars_nouveau-1.20.1-4.9.0-all.jar
ars_ocultas-1.20.1-1.2.1-all.jar
BetterAdvancements-1.20.1-<IP_ADDRESS>.jar
Botania-1.20.1-443-FORGE.jar
Botarium
carryon-forge-1.20.1-<IP_ADDRESS>.jar
cosmeticarmorreworked-1.20.1-v1a.jar
create-1.20.1-0.5.1.f.jar
createoreexcavation-1.20-1.4.2.jar
create-new-age-forge-1.20.1-1.1.2.jar
cristellib-1.1.5-forge.jar
curios-forge-5.7.0+1.20.1.jar (Forge/NeoForge)
fabledweaponry-1.20.1-<IP_ADDRESS>.jar
FerriteCore ((Neo)Forge)
geckolib-forge-1.20.1-4.4.2.jar
Guide-API-VP-1.20.1-2.2.6.jar
jei-1.20.1-forge-<IP_ADDRESS>.jar
JustEnoughResources-1.20.1-<IP_ADDRESS>.jar
more_music_discs_blossom.jar
multipiston-1.20-1.2.31-ALPHA.jar
occultism-1.20.1-1.120.0.jar
oreexcavation-1.13.170.jar
Patchouli-1.20.1-84-FORGE.jar
SmartBrainLib-forge-1.20.1-1.13.jar (Forge/Fabric/Quilt)
ToroHealth-Unofficial-Forge-1.20.1-1.0.0.jar
Towns-and-Towers-1.12-Fabric+Forge.jar
Waystones

i cant do the ritual because its say the pentacle is wrong

You need to update supplementaries. The version you have accidentally deleted the candles tag, hence the weird behaviour of any mod using that tag, including occultism.

You need to update supplementaries. The version you have accidentally deleted the candles tag, hence the weird behaviour of any mod using that tag, including occultism.

thx  for the help


When signing up users, we can only redirect them to the Auth0 login form

Please do not report security vulnerabilities here. The Responsible Disclosure Program details the procedure for disclosing security issues.
Thank you in advance for helping us to improve this library! Your attention to detail here is greatly appreciated and will help us respond as quickly as possible. For general support or usage questions, use the Auth0 Community or Auth0 Support. Finally, to avoid duplicates, please search existing Issues before submitting one here.
By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Describe the problem you'd like to have solved

When signing up users, we can only redirect them to the Auth0 login form

Describe the ideal solution

Redirect users directly to Auth0 Sign up form

Alternatives and current work-arounds

Users signing up have to navigate to the login form, scroll down and click the Sign up button. This is not ideal as we would like to have a sign up and log in button on our website. I checked the https://auth0.github.io/auth0-react/interfaces/redirectloginoptions.html to see if there was an option for 'sign up', but there is not. Please add this or a new method called auth0.signupWithRedirect()

Hey @jgresham, I appreciate that this isn't obvious but you're able to achieve this by setting screen_hint to signup, and your users should see the "sign up" page first.
Note: this assumes that you have configured your Universal Login page to the "New" experience.
ReactDOM.render(
  <Auth0Provider
    domain="YOUR_AUTH0_DOMAIN"
    clientId="YOUR_AUTH0_CLIENT_ID"
    redirectUri={window.location.origin}
    screen_hint="signup"
  >
    <App />
  </Auth0Provider>,
  document.getElementById('app')
);

See the docs for more info. This param doesn't currently appear in the options for our SDKs, but it is an option accepted by the login endpoint and our SDKs generally allow you to send arbitrary parameters through, in addition to the ones we mention in the SDK docs, like domain and clientId.
Does this help?

Thanks for the response @stevehobbsdev! I would still like to use the SDK to either take the user directly to the sign up form or directly to the login form based on which button they click.
It appears that this works!
auth0.loginWithRedirect({ screen_hint: "signup" })
and auth0.loginWithRedirect({ screen_hint: "login" })

Perfect! Glad you got it working :)

@stevehobbsdev it would be great to put screen_hint in the docs for https://auth0.github.io/auth0-react/interfaces/redirectloginoptions.html . If you point me to a guide to generate docs, I can submit a PR for this.

@jgresham Thanks. This is already something we're discussing internally as part of a bigger thing across a few of our SDKs.


Misleading ssl option description for --serverCert

Describe the bug
in first sentence the option is described with "optional" and "If you want". but later the text changes to "While clientCert and clientKey are not always needed, the serverCert is mandatory for building a connection."
This is a little bit misleading. I think I know what you want to say (namely that if user does not provide it, it needs to be in default paths). but maybe describe better.
   --serverCert=<path to server certificate>
	   Use this optional parameter, if you want to specify a server certificate.
	   If this option is not set, the default-roots will be used.
	   The default location is the system cert path, which can be overridden by
	   gRPC environment GRPC_DEFAULT_SSL_ROOTS_FILE_PATH.
	   While clientCert and clientKey are not always needed, the serverCert
	   is mandatory for building a connection.

Another thing is: Is it really always required? can we not have a server whic only requires client certs and does not authenticate itself?

Regarding the necessity of the serverCert:
For the client's private key as well as the client's certificate the documentation explicitly states, that they may be empty. This is not the case for the server's certificate where the documentation indicates that either the default certificate or a certificate provided by the user has to be used in order to establish a secure connection:
The buffer containing the PEM encoding of the server root certificates. If   this parameter is empty, the default roots will be used.  The default    roots can be overridden using the \a GRPC_DEFAULT_SSL_ROOTS_FILE_PATH    environment variable pointing to a file on the file system containing the    roots.


Mark v1.17.1 in the changelog and release notes.

Description
Mark v1.17.1 in the changelog and release notes.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

🚀


[backport] Reload GPG key for ROCm 4.2 test

Backport of #5636 by @kmaehashi

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 1b079262f407d6fe7ae3567c2bb01d332c778ea3, target branch v9) succeeded!


Typo in Variable name for model: InvoiceLineRetDto

Hello Grey,
I found a typo while working with invoice Line items.
    [XmlElement("DataExtRet")]     public List<DataExtRetDto> DateExtRet { get; set; }
The variable DateExtRet should be renamed to DataExtRet
Thank you <3

Nice catch. I never worked with the DataExtRet fields so I don't know if they even work.
Let me know if you find anything else.
Have a great New Years.
QbModels.1.2.7.zip

Thanks <3. I'll keep you updated if anything come in future


docs: fix vCluster link

Fixes https://github.com/keptn/lifecycle-toolkit/issues/3436

build is failing because of external factors


feat(pod): add fail pod add fail pod experiment

Describe what this PR does / why we need it
add fail pod add fail pod experiment
Does this pull request fix one issue?
Describe how you did it
所选的pod在指定的时间段内将不可用，CR删除之后，将销毁实验容器恢复之前镜像。
Describe how to verify it
Special notes for reviews

@Ghostbaby Thank you for you contribution, please add usage about the feature.


Support text/plain mode?

Any chance to support text/plain mode?

In the demo you can view the raw test by unchecking this box

This sets the component's AutoFormatMarkdown parameter accordingly.
If that's not what you have in mind, could you elaborate ?

OK I think I understood : you want plain text as a language option ?

Yeah if possible. In CodeMirror5  you can set the mode to null or plain/text
Looking at what you wrote I am now thinking so set language to Markdown, switch off the autoformat and disable highlighting. This should do the same thing?
If so, wonder if you change so that you can select plain text and it sets those options for you.

Implemented and deployed ; can you check and either close this issue or comment it ?

Got the following exception
index-C6LAOLAl.js:77011   Error in initializing CodeMirror TypeError: Cannot read properties of null (reading 'extension')     at inner (index-C6LAOLAl.js:2038:31)     at inner (index-C6LAOLAl.js:2022:13)     at inner (index-C6LAOLAl.js:2015:17)     at flatten (index-C6LAOLAl.js:2044:5)     at Configuration.resolve (index-C6LAOLAl.js:1952:25)     at EditorState.create (index-C6LAOLAl.js:2783:43)     at Module.initCodeMirror (index-C6LAOLAl.js:76986:45)

Can you make sure the browser cache is cleaned ?

Just tried again with cache cleared, still the same.

Are you running the example locally or are you on the github actions demo ?
If locally, try running dotnet clean and dotnet build in the CodeMirror6 project .

I am running it all locally. Cloned the git, referenced the project and using in my own projects. I am now going to use your examples project locally. Will get back in a few moments

Ok I have managed to replicate the issue locally in your examples.
Go into Example.razor and change the Language from the default CodeMirrorLanguage.Markdown to CodeMirrorLanguage.PlainText so that it starts up with that language set. Now when I run it I get the same errors.

I also just done this in my own project to confirm it's the initial setting. This works.
`protected override Task OnAfterRenderAsync(bool firstRender)
{
if (_codeMirror is not null)
if (_codeMirror.Language != CodeMirrorLanguage.PlainText)
_codeMirror.Language = CodeMirrorLanguage.PlainText;
return base.OnAfterRenderAsync(firstRender);

} `

I can confirm the issue is resolved. Tested and now using the nuget package :-D

I can confirm the issue is resolved. Tested and now using the nuget package :-D


Support functools.partial for event handlers

Fixes #766

I'm not a super-expert on the best way to do this but this seems reasonable to me.

@richard-to WDYT about this feature / implementation? I think the feature is useful, but the implementation is a little bit hairy. In particular, we need to avoid getting a repr for types that return a memory address (e.g. vanilla Python classes)
class Car:
  def __init__(self, name: str) -> None:
    self.name = name

car = Car(name="a") # repr -> <__main__.Car object at 0x79cfdc146080>

Nice! I remember when I first started using Mesop, I had tried using functools.partial to no avail.
Looking at the code example, it does make the devex a lot nicer than having to pass in a key and then parsing the string.
Question: does this work when doing a dynamic loop? Say I wanted to add an event handler to each box using partial?

Yup. I added an example in the functools_partial example.

Yes, the limitation of needing a stable repr is interesting. How much of a drawback do you think that will be in practice? I think it may be slightly annoying, but the improvement in devex is worth it I think (granted I haven't tested this out with some of the uses I have for this yet)

I think it's not too bad since this should cover most of the cases. It may be a little hard to understand for Mesop app developers, but I think we can work on improving the error message + docs if this becomes a common issue.
Thanks for the feedback, I'll merge this in.


Configuration property "local-dns-names" is not defined

Error: Configuration property "local-dns-names" is not defined
at Config.get (/app/node_modules/config/lib/config.js:203:11)
at Object.srf.use [as handle] (/app/app.js:39:23)
at next (/app/node_modules/drachtio-srf/lib/proto.js:200:15)
at Object.handle (/app/node_modules/drachtio-srf/lib/srf.js:113:7)
at next (/app/node_modules/drachtio-srf/lib/proto.js:200:15)
at Function.app.handle (/app/node_modules/drachtio-srf/lib/proto.js:209:3)
at DrachtioAgent.app [as puntUpTheMiddleware] (/app/node_modules/drachtio-srf/lib/connect.js:17:9)
at DrachtioAgent._onMsg (/app/node_modules/drachtio-srf/lib/drachtio-agent.js:595:18)
at emitTwo (events.js:126:13)
at WireProtocol.emit (events.js:214:7)

I figured that out ... in /config/default.json.example variable is called local-dns-name meanwhile code is basically trying to find local-dns-names

ok thanks.  I've not spent that much time on this project, so let me know if you hit any other issues and I will try to clean them up

If you are going to use this, could you update to the latest commit, as I checked in some updates and fixed at least one other bug you would probably encounter


is_transparent_passthrough_allowed always returns false

In the "parent" class "ProxyClientSession.h" the function "is_transparent_passthrough_allowed()" always return false (hard-coded). The inherited class "Http1ClientSession.h" contains internal variable "f_transparent_passthrough" but doesn't implement the "is_transparent_passthrough_allowed()" function to return this its value.
My commit, just implement the function in the inherited class.

[approve ci]

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/32/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1600/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1494/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/164/ for details.

Looks good.

Linux build successful! See https://ci.trafficserver.apache.org/job/RAT-github/9/ for details.

Why wasn't this merged? @SolidWallOfCode Ping?

I have no objections, looks good to me.


wait_for_edge event-limit (RuntimeError: Error #3 waiting for edge)

There seems to be a limit to how many wait_for_edge-events can be processed.
I use a circuit that switches the input on and off around 60 times per second.
Everything is working until I hit a reproducable limit of 1017 events.

The used code:

import RPi.GPIO as GPIO
import time

GPIO.setmode(GPIO.BOARD)
channel = 7
avg = 0
i = 0
starttime = time.time()
print "starttime: " + str(starttime)

GPIO.setup(channel, GPIO.IN, pull_up_down=GPIO.PUD_UP)
while True:
    try:
        GPIO.wait_for_edge(channel, GPIO.RISING)
        print GPIO.input(channel)
        t = time.time()
        starttime = t - starttime
        avg = (avg + starttime) / 2
        print "elapsed: " + str(starttime)
        print "avg: " + str(int(avg*1000))
        print i
        i += 1
        starttime = t
    except KeyboardInterrupt:
        GPIO.cleanup()

After holding the button on my circuit for a while the process exits with this
errormessage:
...
elapsed: 0.0232310295105

avg: 23

1016

1

elapsed: 0.0233290195465

avg: 23

1017

Traceback (most recent call last):

  File "input.py", line 17, in <module>
RuntimeError: Error #3 waiting for edge

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 22 Apr 2013 at 8:07

Merged into: #38

I have the same issue in 0.7.1a4


legacy: BibAuthorList removal

INCOMPATIBLE Removes legacy BibAuthorList module together with ported
'authorlist' module.  (addresses #3233)

Signed-off-by: Leonardo Rossi<EMAIL_ADDRESS>

@jirikuncar
git grep -i authorlist
invenio/legacy/oaiharvest/utils.py:def create_authorlist_ticket(matching_fields, identifier, queue):
invenio/legacy/oaiharvest/utils.py:    in extracted authors from collaboration authorlists.
invenio/legacy/oaiharvest/utils.py:Harvested record with identifier %(ident)s has had its authorlist extracted and contains some UNDEFINED affiliations.
invenio/legacy/websession/templates.py:                if action == "runauthorlist":
invenio/legacy/websession/templates.py:                    tmp_out += """
    %s""" % (CFG_SITE_URL, _("Run Author List Manager"))
invenio/modules/access/local_config.py:             ('authorlistusers', 'Users who can user Authorlist tools', 'deny all'),
invenio/modules/access/local_config.py:               ('runauthorlist', 'run Authorlist tools', '', 'no'),
invenio/modules/access/local_config.py:             ('authorlistusers', 'runauthorlist', {}),
invenio/modules/oaiharvester/tasks/postprocess.py:    "<collaborationauthorlist.*?>.*?</collaborationauthorlist>", re.DOTALL)
invenio/modules/oaiharvester/tasks/postprocess.py:    """Perform the special authorlist extraction step.
invenio/modules/oaiharvester/tasks/postprocess.py:            ) or "authorlist2marcxml.xsl"
invenio/modules/oaiharvester/tasks/postprocess.py:            authorlist_record = create_records(authors)
invenio/modules/oaiharvester/tasks/postprocess.py:            if len(authorlist_record) == 1:
invenio/modules/oaiharvester/tasks/postprocess.py:                if authorlist_record[0][0] is None:
invenio/modules/oaiharvester/tasks/postprocess.py:                    eng.log.error("Error parsing authorlist record for id: %s" % (
invenio/modules/oaiharvester/tasks/postprocess.py:                authorlist_record = authorlist_record[0][0]
invenio/modules/oaiharvester/tasks/postprocess.py:            translate_fieldvalues_from_latex(authorlist_record, '100', code='a')
invenio/modules/oaiharvester/tasks/postprocess.py:            translate_fieldvalues_from_latex(authorlist_record, '700', code='a')
invenio/modules/oaiharvester/tasks/postprocess.py:                          + record_xml_output(authorlist_record) + '</collection>'
invenio/modules/upgrader/upgrades/invenio_2013_09_26_webauthorlist.py:    return "Tables for new webauthorlist module"

I need to remove something from them connected with authorlist?

All except invenio/modules/upgrader/upgrades/invenio_2013_09_26_webauthorlist.py

@hackreak @jirikuncar Can you undo all the changes done to legacy oaiharvest and oaiharvester in this PR? You see, those parts have nothing to do with the authorlist module that is removed. Its something else. (I won't be back until Wednesday to fix it myself ;-) ). Thanks!

@jalavik I'm sorry about that. I have reverted it in 78fa942.

@jirikuncar No worries. Thanks for the revert!


Ycmd server timeout

My config.
(require 'ycmd)
(add-hook 'after-init-hook #'global-ycmd-mode)
(setq ycmd-toggle-force-semantic-completion t)
(set-variable 'ycmd-server-command
              '("python" "D:/appdata/emacs/extraBin/ycmd/ycmd"))
(defun ycmd-setup-completion-at-point-function ()
  "Setup `completion-at-point-functions' for `ycmd-mode'."
  (add-hook 'completion-at-point-functions
            #'ycmd-complete-at-point nil :local))
(add-hook 'ycmd-mode #'ycmd-setup-completion-at-point-function)
(require 'company-ycmd)
(company-ycmd-setup)
(require 'flycheck-ycmd)
(flycheck-ycmd-setup)
(when (not (display-graphic-p))
  (setq flycheck-indication-mode nil))
(require 'ycmd-eldoc)
(add-hook 'ycmd-mode-hook 'ycmd-eldoc-setup)
(setq request-message-level -1)

init output.
File mode specification error: (error "ERROR: Ycmd server timeout")
You should have a section marked ";;; Commentary:"
The first line should be of the form: ";;; package --- Summary"
REQUEST [error] Callback is not called when stopping process! Explicitly calling from timer.
REQUEST [error] Error (timeout) while connecting to http://<IP_ADDRESS>:53841/shutdown. [2 times]
REQUEST [error] Error (error) while connecting to http://<IP_ADDRESS>:53841/ready. [5 times]
Ycmd server killed
ycmd--wait-until-server-ready: ERROR: Ycmd server timeout

HI, can you please tell me how to get those init output. My Message only outputs "server killed; timeout", not helpful at all.

@Congee Hi. I got this from *Messages*.

@moyotar Thank you. It's weid that my Message doesn't show anything about this. Had to debug the emacs-ycmd source code.


Not able to see logs when i click on step name

Not able to see logs when i click on step name

that is not a supported feature at this time


chore: remove @prismicio/client as a dependency

Types of changes

[ ] Chore (a non-breaking change which is related to package maintenance)
[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This PR moves @prismicio/client from dependencies to devDependencies.
Important: @prismicio/client remains a peer dependency.
Thanks to @samuelhorn and @lihbr for diagnosing the issue!
Checklist:

[ ] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

🐭

Codecov Report

Merging #49 (99e5902) into main (3a6d1d8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #49   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          716       716
  Branches        43        43
=========================================
  Hits           716       716

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


Fix issues in last GameControllerDB sync

We don't support 'misc1' as an output string yet (seems used for PS5 controller).
Yet another broken CSV line with 'CO.,LTD' (will send fix upstream. Edit: https://github.com/gabomdq/SDL_GameControllerDB/pull/426).

@madmiraal Not sure what this misc1 is but it seems to be for a new type of button on the PS5 controller, I guess we need to add support for it somehow? Maybe we can find Godot users with a PS5 to test. Before this PR the last sync would give:
ERROR: misc1:b12
Unrecognised output string: misc1
   at: parse_mapping (core/input/input.cpp:1245)
ERROR: misc1:b13
Unrecognised output string: misc1
   at: parse_mapping (core/input/input.cpp:1245)
ERROR: misc1:b13
Unrecognised output string: misc1
   at: parse_mapping (core/input/input.cpp:1245)

misc1 is the mic button in this case, it differs per controller. see sdl source for a few more new fields (misc2 as well and paddles iirc)

See http://hg.libsdl.org/SDL/rev/56b3a5c20eec and http://hg.libsdl.org/SDL/rev/8fe13a9ef824

It's been changed since 56b3a5c20eec which added aux1, aux2, aux3, aux4.
https://hg.libsdl.org/SDL/rev/e5e22d42f2e2 replaced aux1, aux2, aux3, aux4 with misc1, paddle1, paddle2, paddle3 and paddle4.
https://hg.libsdl.org/SDL/rev/0be349835d8e added touchpad.

@offalynne Maybe the header of gamecontrollerdb.txt should be changed to say:
# Game Controller DB for SDL in 2.0.14 format

Since the new identifiers were added in 2.0.14, so I assume they wouldn't work on earlier SDL versions or other engines that implement the same format from an earlier SDL version.


Incorrect parsing JSDocSeeTag with url string only

Bug Report

🔎 Search Terms
ts.JSDocSeeTag
🕗 Version & Regression Information
Typescript 4.7.3
⏯ Playground Link
https://www.typescriptlang.org/play?#code/JYOwLgpgTgZghgYwgAgJLIN4FgBQuCQ+A9AFQkH7InICCyAzsAA4A2KTUA9k9GAJ4NgAcxBwwAVygQKVGSQAC9CCgAWYME3oAuIkTgAjBADoAHnwBecohQ7ctyOCD4BuXBVwBfXEA
💻 Code
interface I {
/**
* A siple property signature
*
*@see https://abc.xyz
*/
prop: any;
}
🙁 Actual behavior
tag.name = https
tag.comment = ://abc.xyz
🙂 Expected behavior
tag.name = undefined
tag.comment = https://abc.xyz

The syntax of @see is
@see [ { ] QualifiedName [ } ] Comments
Given that many people put urls after @see, the language service displays unresolved identifiers named 'https' like a url. Other consumers of the compiler API can do the same if they want, or do even more post-processing, eg supporting markdown

@sandersn I think you're wrong though? TSDoc and JSDoc both say @see should support links, possibly via the @link tag.

This syntax works in Typescript, and matches the documentation of JSDoc and TSDoc: /** @see {@link https://abc.xyz} */


toolz not installed with dask

After I installed dask via Pip, I get No module named 'toolz'
Traceback (most recent call last):
  File "/home/niklas/Desktop/norbert/nodepy_modules/.pip/lib/python3.5/site-packages/dask/delayed.py", line 10, in <module>
    from cytoolz import unique, curry, first
ImportError: No module named 'cytoolz'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/nodepy", line 11, in <module>
    load_entry_point('node.py-3.5', 'console_scripts', 'nodepy')()
  File "/home/niklas/Desktop/nodepy/nodepy.py", line 827, in main
    module.exec_()
  File "/home/niklas/Desktop/nodepy/nodepy.py", line 153, in exec_
    exec(self.code, vars(self.namespace))
  File "/home/niklas/Desktop/norbert/backend/post.py", line 9, in <module>
    from dask import delayed
  File "/home/niklas/Desktop/norbert/nodepy_modules/.pip/lib/python3.5/site-packages/dask/delayed.py", line 12, in <module>
    from toolz import unique, curry, first
ImportError: No module named 'toolz'

Dask 0.14.1

See notes in http://dask.pydata.org/en/latest/install.html#pip

Missed that portion. Thanks

No worries.  Thanks for raising a public issue.  Hopefully this helps
people in the future who run into your situation.
On Tue, Apr 4, 2017 at 8:17 AM, Niklas Rosenstein<EMAIL_ADDRESS>wrote:

Missed that portion. Thanks
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2159#issuecomment-291482044, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszDZ4NQ_WGDv9xOnbTQnFlavWILB7ks5rsjTmgaJpZM4Myt8b
.


Documentation: move all the Aquilon documentation in a single repository

Currently, Aquilon documentation is spread between this repository (README.md describing the installation) and the web site (describing Aquilon DB configuration). I think that we should have on the documentation on one side, either this repo or the web site repo. I see pro and cons for both solutions:

Pro (this repo):

Having it colocated with the code increases the chance to have it up-to-date

Cons (this repo):

This information is mainly maintained by the community, not Aquilon developers
The web site allows better rendering thanks to Jekyll (and its CSS)
There is a complex workflow to merge this repo contents with the repo used by the main developers (outside GH)

So I would probably support moving everything to the web site.
In addition, after everything has been moved into one repo, I suggest that we process it  as part of the Quattor release documentation that we put on ReadTheDocs.

Note that the answer to this question may impact #56.

+1 to this - I think it makes a lot of sense.


Switch keyword has no auto-completion

VSCode Version: 1.17.2
OS Version: Windows 10

Steps to Reproduce:
class Foo {
        public function new() {
                var i : Int = 0;
                switch(i) {
                     case truc:
                }
        }
}

When I write "swi" VsCode show me all functions beginning by these letters, but not the keyword switch.

Please file this issue against the c# extension: https://github.com/OmniSharp/omnisharp-vscode/issues/new


Build the "Create an account" flow UI - error flow

Continues #1233

[x] Write error messages for pages with text inputs
[x] Build out the error states

Figma designs

Tagging myself to take initial swing at error messages

I took a swing at these here: https://github.com/CDCgov/prime-simplereport/pull/1484/commits/235a9fe700d854ed9df9e82865cd390da7f1d150 @vz3

@maya did you still need me for this one? Was OOO and now back!

@Rebecca-LM I think we're good! If you'd like to check what we came up with, see my linked message right above yours.

@maya looks good! Made one suggestion as a comment to a line in #1484


litex_vexriscv: k_busy_wait() never returns if called with interrupts locked

Describe the bug
k_busy_wait() never returns if called with interrupts locked on the litex_vexriscv platform, which uses litex_timer system timer.
For more details, refer to the issue #23519.

cc @mateusz-holenko

@mateusz-holenko do you have plans to address this?

@carlescufi  - yes, we are planning to look at this soon.

Has this been addressed?

We are currrently working on this. It turned out that the timer in LiteX is very simple and implementing a proper k_busy_wait() (allowing to wait for periods longer than 10ms) would require either a custom not-trivial implementation or extending the capabilities of HW (since it's a soft IP core anyway).
We decided to go with the second option and are now adding a register allowing to read the total number of timer ticks from reset to the timer itself. Once it's done and tested we will update the driver to use this in k_cycle_get32() function.


Unattend description is not correct

In the help of tfsconfig /unattend is mentioned that multiple bindings can be configured.
I was not able to get it working, each binding for its own was working, joined with ";" it didn´t worked

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

ID: 4b10e093-64cf-198b-182a-42ff5e3c000b
Version Independent ID: d24b7453-3139-f691-4597-8c4c9f8015d3
Content: Configure with TFSConfig - Azure DevOps Server & TFS
Content Source: https://mseng.visualstudio.com/DefaultCollection/TechnicalContent/_git/tfs-docs-pr?path=%2fdocs%2fcommand-line%2ftfsconfig-cmd.md&version=GBlive&_a=contents
Product: devops-server
Git Login: aaronhallberg
Microsoft Alias: aaronha

I thought it strange that a ";" is used as a comment, so I tried using a "," in the configuration ini file and this created the additional bindings.

I can confirm that what @powersparks says is still correct in 2020.0.1. I just ran into this issue today.


Set peerready and available conditions to true when no VRGs and there…

… are available clusters

Elaborating a use-case that @BenamarMk and me discussed where the force status update will not happen, and in such cases a user has to modify the drpc spec to the new required action, as follows:
DRPC from a backup has the following spec:
Failover:
	action: Failover
	preferredCluster: east
	failoverCluster: west

Hub and west went down, requiring a standby hub on east where the above DRPC is restored without any status fields
In this situation, we will not be able to  read any VRG from west and the current failover target is west
As a result the force status update will not be triggered as vrgs == 0 is true, but spec.failoverCluster != failedClusterToQuery

The above case is not handled and requires user intervention to proceed with failing over the workload to the surviving east cluster


schemaPath in ajv.errors is often incorrect for errors inside $ref'd schema

What version of Ajv are you using? Does the issue happen if you use the latest version?
4.11.8 and 5.1.5. Yes.
Ajv options object
Only { v5: true } is needed (and only for version 4), in order to have $ref.
JSON Schema
Schemas with a $ref that doesn't contain # or that ends in #/.
Sample data
Data that makes validation fail because of the $ref'd schema.
Your code
Minimal example:
var options = { v5: true };
var ajv = require('ajv')(options);
ajv.addSchema([
  { id: '/a', type: 'integer' },
  { id: '/b', items: { $ref: '/a' } }
]);
var data = [true]; // is not valid

Validation result, data AFTER validation, error messages
ajv.validate('/b', data);  // false
ajv.errors[0].schemaPath;  // '/a/type', but should be '/a#/type'

What results did you expect?
For that example, the correct schema path is '/a#/type'.
A longer example illustrates what happens, and points to a workaround and maybe a correct fix:
ajv = require('ajv')({ v5: true });
ajv.addSchema([
  { id: '/a', type: 'integer' },
  { id: '/b', items: { $ref: '/a' } },
  { id: '/c', items: { $ref: '/a#' } },
  { id: '/d', items: { $ref: '/a#/' } }
]);
ajv.validate('/b', [4]);  // true
ajv.validate('/c', [4]);  // true
ajv.validate('/d', [4]);  // true
ajv.validate('/b', [true]);  // false
ajv.errors[0].schemaPath;    // '/a/type'
ajv.validate('/c', [true]);  // false
ajv.errors[0].schemaPath;    // '/a#/type'
ajv.validate('/d', [true]);  // false
ajv.errors[0].schemaPath;    // '/a#//type'

A workaround consists of including an empty fragment (as in { "$ref": "/a#" }) when writing the schema. It is not a normalized id, but apparently that doesn't hurt anything.
As to fixing it...
I think this is controlled here https://github.com/epoberezkin/ajv/blob/master/lib/dot/ref.jst#L44, but I'm not sure it's the only place. If this guess is correct, then the fix could be as simple as replacing that line with $it.errSchemaPath = $schema.indexOf('#') != -1 ? $schema : $schema + '#'.
That doesn't address the #/ case in schema "/d". As far as I can tell, https://github.com/epoberezkin/ajv/blob/master/lib/dot/ref.jst#L26 calls https://github.com/epoberezkin/ajv/blob/fde7030a19833273b08e8de879cb2bf149adaf2f/lib/compile/index.js#L170, which in turn calls https://github.com/epoberezkin/ajv/blob/master/lib/compile/resolve.js#L225, which does eliminate terminal # or #/. (For example, it.resolve.url('/b', '/a#') == '/a' and it.resolve.url('/b', '/a#/') == '/a', while it.resolve.url('/b', '/a#something') == '/a#something'.) Maybe there is some practical way of taking advantage of getting the normalized schema id from the $refVal in L26? (Normalizing it again seems wasteful...)
Are you going to resolve the issue?
It would be cool, but I'm not at all certain I can understand all the ramifications. Specifically, I can't figure what should happen in the non-inlined case (L51 onward), and I can't follow the workings of the refVals (to address also the case with #/ ending) nor whether the change would have negative impact elsewhere. But maybe with some advice I can navigate it?

@jboavida well spotted. It's a known issue (https://github.com/epoberezkin/ajv/issues/273) and it's a tricky one indeed. I should've written down the reasons I decided to leave it as is, there were some complications around it. I'll look into it.

I'm getting a similar problem but without any '#' or multiple schemas. I've a fairly large amount of definitions, some of them cross-referencing each other. One particular instance is a case where I have an object that can be one of two referenced definitions.
One of these definitions ('function') requires two properties, "func" and "params", while the other ('connection') requires "location" and "type".
The object being validated has only "params" defined, ie. it is lacking the "func" property. But as this is "oneOf" situation, I get errors from both. I get "missingParameter: 'location'" with schemaPath "#/definitions/connection/required" from 'connection but for the equivalent 'function' while I do get "missingParameter: 'func'", the schemaPath is "#/required"
Any further information that you need, I'll be happy to provide to the best of my abilities.

@aapoalas thank you. Some simplified example of your issue would be helpful too, in runkit.com if possible

Horrah, I actually managed to reproduce this! I was kind of expecting that to not happen with a simpler example but here it is: https://runkit.com/aapoalas/5941134ab79c0b00117f3137
The schema itself is still not too simple, about 60 lines, but the gist of it should be pretty understandable.
If you have any further questions, just ask.

I am trying to use the schema as below and getting the error as "can't resolve reference device from id #"
schema:
"device" : {
"id" : "device",
"type": "object",
"additionalProperties": false,
"properties": {
"deviceId": {
"type": 'string',
"maxLength": 50
}
}
},
"deviceData" : {
"id" : "deviceData",
"type": "object",
"additionalProperties": false,
"properties": {
"deviceData": {
"type": 'object',
"items": {
"$ref": "device"
}
}
}
}
Could you please tell me what part I am missing here?

@mythili007 it’s unrelated to this issue, you reference is incorrect. You may need #/device - but it’s not clear from a partial sample. Please refer to the docs or JSON schema spec or submit a question to stackoverflow.

I ran into this today with version 6.12.0, and have a much shorter, simpler RunKit example: https://runkit.com/5e68dad1e356eb0014ed2786/5e68dae0e356eb0014ed27af

ajv.errors[0] has a schemaPath of #/required, which should be #/definitions/A/required.
ajv.errors[1] has a correct schemaPath of #/definitions/B/required.

Removing the $ref in A (commenting out line 12) gives the correct schemaPath.


Update READ ME

Hey guys we need to make sure to update the readme so that it reflects the stage we are at on our package. We cant just tell them to pip install the package just yet so I am going to make an update to inform people how to run it using poetry.

Just add the test instruction to README as well.


BE AS CLEAR AND CONCISE AS POSSIBLE.
describe the bug
Something Wrong with the window button
expected behaviour
why is this a bug - what should be happening?
screenshots

platform
Windows 10 2004
Surface pro 3

@Septem-YDL are you behind a proxy/vpn by any chance (e.g. a school network)?

this looks like a duplicate of https://github.com/dragonwocky/notion-enhancer/issues/66

Yes，i was behind a vpn.
today it runs normally 😂


save_settings doesn't store the selected topic

When I close the plugin and restart it, it should re-open and re-subscribe to the previously used topic (and transport).
Currently, it seems the saved topic is one index further up.

Thanks for reporting, I can confirm this is indeed happening.


Error printed when calling tf.data.experimental.load

System information

OS Platform:  WSL2 Linux Ubuntu 20.04
TensorFlow version: 2.5.0-rc1
Python version: 3.9
CUDA/cuDNN version: 11.2
GPU model and memory: RTX 3090

Describe the current behavior
When calling  tf.data.experimental.load the bellow error is printed, however, the dataset appears to load without issue
Unimplemented: Cannot merge options for dataset of type LoadDataset, because the dataset does not implement 'InputDatasets'.
Describe the expected behavior
No error message should be printed
Standalone code to reproduce the issue
import tensorflow as tf

ds = tf.data.Dataset.from_tensor_slices([1, 2, 3])
for i in ds:
    print(i)

tf.data.experimental.save(ds, "/mnt/d/load_issue.dataset", compression="GZIP")
load_ds = tf.data.experimental.load("/mnt/d/load_issue.dataset", compression="GZIP")

for e in load_ds:
    print(e)

@bhack, ahh looks like It's fixed in the nightly builds. I will close the issue!

Did the fix actually land in the TensorFlow 2.5.0 stable?
I'm still experiencing this error message with TensorFlow. It looks like the commit solving the problem missed the v2.5.0 release and landed in v2.6.0-rc0.
Would it be possible to cherry-pick the commit into a patch release for the 2.5.x branch?

Patch releases are only for security issues.


Fix case consistency and 'till'

Signed-off-by: John Howard<EMAIL_ADDRESS>Fixed some case consistencies on output messages, and corrected Polling till temp... to Polling until...

Hi @jhowardmsft, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (John Howard (VM)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Not sure why CI is failing, or if it's usually out on this repo. The command "eval go get -t -v ./..." failed. Retrying, 3 of 3.. These PR only has text changes.

LGTM.
There's probably a dependency issues with the azure-sdk-for-go that's causing the failure.  I'll investigate.


Mac installation -- missing directories

Hi!  I'm trying to build freebayes on a Mac, OS X 10.10.2 (and also tried on a machine with 10.10.1).
after "sudo make install",
I get the error:
cp bin/freebayes bin/bamleftalign /usr/local/bin/
cp: bin/freebayes is a directory (not copied).
cp: bin/bamleftalign: No such file or directory
make: *** [install] Error 1
bin is completely empty.  To install the freebayes folder, I used git:
git clone --recursive git://github.com/ekg/freebayes.git
I didn't see any errors during the cloning.
Is there anything I can do to fix it?
Thanks,
Kim

I would just "make" and then copy the files in the bin folder to where you
want them to be (perhaps /usr/local/bin, it depends on your system).
On Fri, Mar 6, 2015 at 5:10 PM, kkwinner<EMAIL_ADDRESS>wrote:

Hi! I'm trying to build freebayes on a Mac, OS X 10.10.2 (and also tried
on a machine with 10.10.1).
after "sudo make install",
I get the error:
cp bin/freebayes bin/bamleftalign /usr/local/bin/
cp: bin/freebayes is a directory (not copied).
cp: bin/bamleftalign: No such file or directory
make: *** [install] Error 1
bin is completely empty. To install the freebayes folder, I used git:
git clone --recursive git://github.com/ekg/freebayes.git
I didn't see any errors during the cloning.
Is there anything I can do to fix it?
Thanks,
Kim
—
Reply to this email directly or view it on GitHub
https://github.com/ekg/freebayes/issues/151.

@ekg perhaps just modify the Makefile
OLD
install:
        cp bin/freebayes bin/bamleftalign /usr/local/bin/

NEW
install: bin/freebayes bin/bamleftalign
        install $^ /usr/local/bin/


Lightgbm does not work on GPU?

Description
It's really hard to install a gpu version.
"Please recompile with CMake option -DUSE_CUDAP=1"
??? What is CUDAP?
cmake -DUSE_CUDAP=1 ..
but still got the same error.
Reproducible example
import lightgbm as lgb
import numpy as np

X = np.random.rand(500, 10)
y = np.random.randint(2, size=500)
dataset = lgb.Dataset(X, label=y)
# params = {'device': 'gpu'}
params = {'device': 'cuda'}
gbm = lgb.train(params, dataset)

Environment info
centos 7 (in docker)
LightGBM version or commit hash:
master branch (4.1?)
Command(s) you used to install LightGBM
cmake -DUSE_CUDAP=1 ..
sh ./build-python.sh install --precompile
# the command above upgrades numpy and lightgbm will fails to import (numpy.float error). why do you upgrade that then?
pip3 install numpy==1.20

Additional Comments

LightGBM does not work on GPU

That is absolutely not true. There are 2 GPU-accelerated builds of LightGBM:

"device": "gpu" (-DUSE_GPUE=1) = OpenCL-based implementation targeting a wide range of GPUs from multiple manufacturers
"device": "cuda" (-DUSE_CUDA=1) = CUDA kernel targeting NVIDIA GPUs

To install the Python package using these builds, see the guide in https://github.com/microsoft/LightGBM/blob/master/python-package/README.rst.

What is CUDAP?

Thanks for pointing this out! That's a typo.
I found that error message using GitHub search: https://github.com/search?q=repo%3Amicrosoft%2FLightGBM+"Please+recompile+with+CMake+option+-DUSE_CUDAP%3D1"&type=code
https://github.com/microsoft/LightGBM/blob/18dbd65e57995618ee2a8b1f7e4cb0df1f9c6333/src/treelearner/cuda/cuda_single_gpu_tree_learner.hpp#L158
It should be -DUSE_CUDA (no P). We'd be grateful if you'd open a pull request fixing that. Are you interested in contributing?

Thanks I will try USE_CUDA later.
I just compiled with opencl. It tells "LightGBMError: No OpenCL device found", which clinfo founds nvidia cuda.

clinfo
Number of platforms                               1
  Platform Name                                   NVIDIA CUDA
  Platform Vendor                                 NVIDIA Corporation
  Platform Version                                OpenCL 3.0 CUDA 11.8.87
  Platform Profile                                FULL_PROFILE
  Platform Extensions                             cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_fp64 cl_khr_3d_image_writes cl_khr_byte_addressable_store cl_khr_icd cl_khr_gl_sharing cl_nv_compiler_options cl_nv_device_attribute_query cl_nv_pragma_unroll cl_nv_copy_opts cl_nv_create_buffer cl_khr_int64_base_atomics cl_khr_int64_extended_atomics cl_khr_device_uuid cl_khr_pci_bus_info cl_khr_external_semaphore cl_khr_external_memory cl_khr_external_semaphore_opaque_fd cl_khr_external_memory_opaque_fd
  Platform Host timer resolution                  0ns
  Platform Extensions function suffix             NV

  Platform Name                                   NVIDIA CUDA
Number of devices                                 1
  Device Name                                     NVIDIA GeForce GTX 1080 Ti

No OpenCL device found

Please see the advice in https://github.com/microsoft/LightGBM/issues/5914#issuecomment-1713129547.

Thanks, this works. Will use USE_CUDA much faster than USE_GPU for nvidia GPU?
And this is another typo DUSE_GPU :)

another typo

ha sorry about that! Thanks, just edited

USE_CUDA much faster than USE_GPU for nvidia GPU?

Yes. The "device": "cuda" build should be faster, as:

it handles more operations on the GPU (i.e. not falling back to CPU) than the "device": "gpu" version
it was written much more recently, and takes advantage of newer versions of CUDA which have performance enhancements for newer NVIDIA GPUs

If you have an NVIDIA GPU, use the USE_CUDA version.

this works

great, we'll close this issue

great thanks

Can lightgbm use multi gpu cards? I have 4 cards, seems it uses only one.

Not yet, that's being tracked in #5076.


调度模式建议增加相关的文字说明和解释

Describe the bug
A clear and concise description of what the bug is.

调度模式建议增加相关的文字说明和解释
目前不知道“聚合“和”动态权重“ 是什么效果
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Noted, thanks. Will improve soon.

@SAMZONG we need to add more text about those parameters about kairship.

Great idea, we will add more descriptions in future.

已优化，点击问号可查看解释：


Caching for read only files

What is your current rclone version (output from rclone version)?
$ rclone version
rclone v1.47.0
- os/arch: darwin/amd64
- go version: go1.12.4

What problem are you are trying to solve?
rclone does not currently cache read-only files which means it always retrieve chunks from the backend even if it has retrieved them previously. It seems like performance could be significantly improved for read-only files if they were able to be cached locally.
How do you think rclone should be changed to solve that?
rclone should cache read-only files like it currently caches read-write files.

Sorry, I just discovered the cache backend that addresses this request. Please close.

@mcafaro - you can just hit the close button at the bottom as you have the ability to close it yourself.


Is there a way to encrypt for multiple users from command line?

I want to mimic what kbfs does to a shared private folder, but encrypt the file to be stored elsewhere, so that it can be decrypted by multiple keybase users.

Yes, see keybase encrypt --help.  It's not exactly the same as KBFS though, since it won't re-encrypt the file when the users add new devices later.

I tried that, and it implies that you can, but when I do keybase encrypt name1,name2,name3 -i filename.txt I get an error saying Unexpected OR operator, which suggests you can't do multiple names.

@Engelberg I think it's keybase encrypt -i filename.txt name1 name2 name3.

@Engelberg: did @cjb's solution resolve your issue?

Yes. Thanks.


possibility to call module from registry with version stored in variable or param

Is your feature request related to a problem? Please describe.
Now it is not possible to call a module from registry with specified version in a variable
var version = '0.0.1'
module mod_webapp 'br/CoreModules:mod_webapp:${version}' = {
  name: 'deployment-mod_webapp'
...
}

Describe the solution you'd like
I would like to store the module version in the variable or param.

I thought we had another issue about this, but I can't find it. This is feasible with variables, but not with parameters, since parameter values are not known until deployment and that could change the module code that needs to be validated.

Related: #413

I also have a use case where the path to the ACR and the version of the Bicep module need to be defined as parameters that are leveraged by an Azure DevOps pipeline when deploying/updating infrastructure.

Specifying the name of the ACR as a parameter would be indeed valuable as you wouldn't need to update every template when you migrate to another container registry.

Specifying the name of the ACR as a parameter would be indeed valuable as you wouldn't need to update every template when you migrate to another container registry.

@jhueppauff - specifically for this case, we would recommend adding an alias to bicepconfig, and if you ever change the ACR URL, you would only need to change the value in bicepconfig:
https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/bicep-config-modules#aliases-for-modules

Any news about this feature request plz ? I still have "Error BCP092: String interpolation is not supported in file paths."

could this please be reopened? I would really like this enhancement as we can tokenize our paths.

Re-opening and renaming to cover variables specifically, since that's much easier than params.

We are a large team of developers in my company that would appreciate this feature immensely as we address different versions of bicep modules hosted in an ACR. Pretty please, save us :)

This would be very useful.  I have a UAT and a Production container registry for modules, and being able to set the path would be useful


PMM-10307 advisor filtering test

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests/detail/pmm2-ui-tests/20790/pipeline/


Changes to test content from the Java CQL engine, based on the 3.0.0 …

…release

Since this is more or less the "initial PR", so we should just pull this, right?

Well, these are the tests that are in the Java regression suite, and this PR is the difference between what is published in the 1.5.2 version of the specification (what's currently in the main branch). So long as we all agree with the changes made in the Java engine's suite, then yes.


Lift up dialect specific methods in the CreateTableBuilder

Ref:

https://github.com/knex/knex/issues/3478#issuecomment-546629137
https://github.com/knex/knex/issues/3478#issuecomment-546631483

@lorefnon
 2) Integration Tests

       oracle | oracledb

         Joins

           accepts a callback as the second argument for advanced joins:

     select * from "accounts" left join "test_table_two" on "accounts"."id" = "test_table_two"."account_id" or "accounts"."email" = "test_table_two"."details" order by "accounts"."id" asc - ORA-00932: inconsistent datatypes: expected - got CLOB

  Error: ORA-00932: inconsistent datatypes: expected - got CLOB

      at Client_Oracledb._query (lib/dialects/oracledb/index.js:246:19)

      at Client_Oracledb.query (lib/client.js:165:17)

      at Runner.query (lib/runner.js:135:36)

      at /home/travis/build/knex/knex/lib/runner.js:39:23

  From previous event:

      at Runner.run (lib/runner.js:25:16)

      at Builder.Target.then (lib/interface.js:14:43)

I think this is a new failure. Can you check it out?

@kibertoad yeah, this broke due to an accidental change. Fixed now.


Renaming a blob displays the default account tier after changing the blob tier

Storage Explorer Version: 1.1.0
Platform: Windows/Linux/Mac
Architecture: ia32/x64
Build Number: 20180516.1
Commit: e0a60be3
Regression From: Not a regression
Steps to Reproduce:

Launch Storage Explorer and Select one GPv2/Blob account.

Set the default Access Tier for the account as 'Cool'.
Open one blob container -> Upload one block blob named 'test.txt' -> Make sure the Access Tier of the blob is 'Cool(inferred)'.
Change the blob access tier to 'Hot'.
Rename the blob.

Expected Experience:
 The blob tier still displays as 'Hot' after renaming successfully.
Actual Experience:
The blob tier displays as the default account tier 'Cool(inferred)'.

Since a rename is actually creating a new file, the value doesn't carry over.

I think this is more of a service ask, since we're doing a server side copy.


Fix header layout to allow flexible header sizes + some refactoring

For #4800

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[X] Title summarizes what is changing
[X] Has a news entry file (remember to thank yourself!)
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

If this makes it easier to look at here is a refactor of how I changed the top level panels in a jsfiddle. Can be a bit easier to see how the header and content panels are sizing this way.
https://jsfiddle.net/jua8b0yq/


Re-org includes

to #329

looks good, however a bit difficult to review 8k lines

note that include ZABAPGIT_OBJECT has to be deleted manually after pulling these changes

looks good, however a bit difficult to review 8k lines

Yeap, sorry for that. Changing includes assumes that :( I hope my description in 329 hwlped somewhat

note that include ZABAPGIT_OBJECT has to be deleted manually after pulling these changes

you mean on target systems ? In my it was deleted.

changing includes: yeah, there is no way around it
target systems: yes, abapGit does not delete obsolete objects automatically


Replication between 2 databases stuck

I am replicating documents between 2 databases on the same server, it has been stuck on 46% for the last few days
Current Behavior
Replication does not progress it will stay on the same amount of checkpointed source sequence
Steps to Reproduce (for bugs)
I have deleted the document from the database and re-entered it, it will stay in triggered state with no errors in the logs. Checkpointed source sequence 257043, current source sequence 557413, progress 46%. I have restarted the service and server multiple time it would just go up  to the same state it was.
Your Environment

Version used: 1.6.1
Browser Name and version: Chrome
Operating System and version (desktop or mobile): Windows Server
Link to your project:

Hi @rynobrits1 , please provide any logs you may have that can help diagnose the problem.

Closing this issues because there won‘t be any more CouchDB 1.x releases. If you experience this issue on current 2.x releases, please re-open it with additional information.


Support transifex

Note that there is still some work we need to do before we can actually
enable translations:

Replace UPPERCASE strings with Normal Capitalization and then do the
uppercasing within the code.
Remove new lines (\n) from UI strings where they don't make sense and
make sure the views still fit properly.
For example, wifi_dialog_message_help breaks in the middle of a
sentence. It's okay to have newlines if they're breaks that should be
there in any language and don't depend on knowing the width of the
dialog.

JIRA: MA-312

@hanningni please review. cc @rohan-dhamal-clarice @shahidtamboli

Looks good :+1:


Testcafe is taking screenshot on catched expect Errors

What is your Test Scenario?

We sometimes have to catch the error thrown by testcafe.expect, for example when we want to verify a process involving an asynchronous backend task. In that case we try to validate the existance of an element on the page and if not found, catch the Error, wait some time, reload the page and assert again. This works regarding the test result, so at the end if the element is found eventually, even though some of the testcafe.expects did not succeed, the test is green, which is also our expectation.
We have also activated the screenshot-option "takeOnFails". To our surprise, enabling this option leads to screenshots being taken, even if the exception thrown by the testcafe.expect call is catched.
If our approach is not optimal regarding catching the Exception, please guide us to a way that is more appropriate.
What is the Current behavior?
a catched testcafe.expect Exception leads to a screenshot being taken if the option "takeOnFails" is true
What is the Expected behavior?
screenshots are only taken for failed tests
Your Environment details:

testcafe version:                    1.8.2
node.js version:                     10.17.0
browser name and version:  tested in chrome latest version
platform and version:           happened under windows, linux and osx

Also, please note that the screenshot behavior is documented as:
Specifies if screenshots should be taken automatically when a test fails.
So a failed(and caught) expecation does not lead to a failed test, therefore the documentation also does not match the current implementation

Hi @ferencbeutel4711
Thank you for your input. We need additional time to research your case and find the most suitable solution. We will update this issue when we have any news.

+1
This enforce me to continue using older testcafe version (1.8).
@miherlosev can you share any estimation for this to be fixed?

I'm unable to provide you with any time frame as any personal estimate may be misleading. Once we get any results, we will update this thread.

This is super annoying! Makes it hard to find the real screenshots since so many fake (from non failing tests) ones are created.

I apologize for the inconvenience caused by this bug. I increased the bug level, so we will give it a higher priority when planning the next development sprint.
However, if the issue is caused by a failed assertion that checks for element existence, you can use the Selector.exists property instead of suppressing the assertion error. @cattermo, @Shreky could you please share your error suppressing cases as well?
For the team:
The issue can be reproduced with the following test:
CODE
It is caused by the explicit screenshot capture call, which didn't exist in previous versions:
https://github.com/DevExpress/testcafe/blame/5ee0c0d9b620d542746d2fbc752fd9736b46a17e/src/test-run/index.js

I apologize for the inconvenience caused by this bug. I increased the bug level, so we will give it a higher priority when planning the next development sprint.
However, if the issue is caused by a failed assertion that checks for element existence, you can use the Selector.exists property instead of suppressing the assertion error. @cattermo, @Shreky could you please share your error suppressing cases as well?
For the team:
The issue can be reproduced with the following test:
import fs from 'fs';

fixture `Fixture`;

test('Test', async t => {
    if(!fs.existsSync('screenshots'))
        fs.mkdirSync('screenshots');

    try {
        await t.expect(false).ok();
    }
    catch (e) {
    }

    await t.expect(fs.readdirSync('screenshots').length).eql(0);
});

It is caused by the explicit screenshot capture call, which didn't exist in previous versions:
https://github.com/DevExpress/testcafe/blame/5ee0c0d9b620d542746d2fbc752fd9736b46a17e/src/test-run/index.js

I ended up rebuilding the test so no error will be thrown but I don't know if the new solution will be stable. I couldn't get it to wait in the same way.
My use case is a button that might be disabled but might also loose the disabled attribute if we wait a little. If it keeps the disabled attribute it should fail unless some other condition is true (which is why we caught the error previously). Now I check for the disabled attribute instead but the hasAttribute will not wait so it's not the same as previously.

@cattermo
Thank you for the additional information. Your scenario is clear. So, as @AndreyBelym mentioned, we increased the bug's priority level. This means that we'll prioritize the issue in our next development sprints.

FYI, this is also an issue when skipjserrors is set to true. If a test passes without any errors, a screenshot still gets generated if there are browser console errors. Additionally, when a test does fail, sometimes it only includes the screenshot taken when a js error occurs, not when the assertion fails. This makes it very difficult to troubleshoot test failures.

@Makavelic
I was not able to reproduce the issue. When the --skip-js-errors option is enabled, TestCafe does not make screenshots in my example. Here is my HTML and test code:
HTML:
<!DOCTYPE html>
<html>
<head>
</head>
<body>
<button>click me</button>

<script>
    document.querySelector('button').addEventListener('click', () => {
        throw new Error('error');
    });
</script>
</body>
</html>

Test code:
fixture('screehshots and js errors')
    .page `..\\pages\\index.html`;

test('Log in to Paypal', async t => {
    await t.click('button');
});

I use the following command to run TestCafe: testcafe chrome test.js --skip-js-errors -s takeOnFails=true
Could you please modify my example to reproduce the issue?

Additionally, when a test does fail, sometimes it only includes the screenshot taken when a js error occurs, not when the assertion fails
This behavior seems correct. If the page has JS error and there is no --skip-js-error option, the test will fail at this stage since the page state is already incorrect.

The issue only occurs when you navigate away from the original page. For example:
test1.html
`

click me
test2

`
test2.html
`

new page

`
test
`import { t } from "testcafe";
fixture("errorFixture")
.page("test1.html")
test("testError", async () => {
await t.click("a");
await t.wait(3000);
});

@Makavelic
I was still unable to reproduce the issue following your instructions. If I enable the --skip-js-errors option, the screenshots are not taken. If I do not use the --skip-js-errors option, the screenshot of the first page is taken because the error occurs inside the 'DOMContentLoaded' event handler of the first page.
I am using the following command to run tests: testcafe chrome test.js --skip-js-errors -s takeOnFails=true
My TestCafe version is 1.9.4.
Could you please specify which command you are using to run tests? Please also specify the browser, OS, and the TestCafe version.
If you have some additional instructions on how to reproduce the issue, please share them with us.

@AlexKamaev, I'm using a custom runner, here are the relevant parts:
.screenshots({
                takeOnFails: true,
                path: paramMap.get(ParamNames.PathToScreenshots),
                pathPattern: '${TEST}/${DATE}_${TIME}.png'})
            .run({skipJsErrors: true});

Testcafe: 1.9.4
Chrome: 86
Screenshots are taken on all of our tests even when they do not fail. This did not occur in earlier versions of testcafe.

@Makavelic
I created a sample based on your runner and your test files. However, I was not able to reproduce the issue. Here is my project: https://gist.github.com/AlexKamaev/24bd42cc8a844d7f7c3f3acfb0f6c1ea
Please specify how I should modify it to reproduce the issue.

Unfortunately, there's not much we can say here without a reproducible example. As for screenshots that are being overwritten, you can try to add the ${FILE_INDEX} placeholder to your "pathPattern" option and see if it helps.

Unfortunately, there's not much we can say here without a reproducible example. As for screenshots that are being overwritten, you can try to add the ${FILE_INDEX} placeholder to your "pathPattern" option and see if it helps.

Is there a way to get more logging from testcafe to hopefully gather more info that would be useful? For whatever reason, testcafe takes a screenshot after logging into the website its testing.

There is a guide in our docs, which describes how to debug TestCafe tests: Debug. Manual step-by-step execution should help to identify a particular test action or assertion after which the screenshot is taken.

Hi folks,
The problem occurs because you use an assertion for checking element visibility or existence.
It's incorrect. Instead of that you need to use the Selector.visible, Selector.exists, or something else depending on your scenario.
Also do not catch assertion errors during test execution. It's not a recommended practice. If the test logic depends on some Selector values, retrieve them in the test and use them later.
Also, we will add a note about capturing screenshots for caught assertion errors.
If you have some problems getting rid of caught assertion errors, please describe them in separate issues. We will investigate them separately.

Hi
issue is closed but I am
Getting screenshot while test is passed . I am using Testcafe 2.5 version . Every time after login it takes screen shot and test passed . When same test failed due to selector got changed it never takes the screenshot of new page .it always showing same screenshot.
{
takeOnFails: true,
path: paramMap.get(ParamNames.PathToScreenshots),
pathPattern: '${TEST}/${DATE}_${TIME}.png'})
.run({skipJsErrors: true});
Any help will
Save me

Hi @testcafe-HK ,
It is difficult to determine the cause of the issue you encountered without being able to reproduce it locally. If I understand correctly, it is not related to the current topic. Please create a separate GitHub issue (Bug Report) and share a simple working project with exact steps to reproduce the issue.


Add CUDA::cudart_static to the link line for libcufilejni

Signed-off-by: Jim Brennan<EMAIL_ADDRESS>When compiling with nvcomp libraries that link with the static cudart, the ai.rapids.cudf.CuFileTest started to fail with the following dump:
/usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>2.b09-1.el7_9.x86_64/jre/bin/java: symbol lookup error: /tmp/cufilejni2743598080003698811.so: undefined symbol: cudaGetLastError
Robert Maynard suggested the fix.

build

build

build

build


[spreadsheet] Resize handles are hidden when sheet not attended

@thure is this a problem? wouldn't it not be possible to resize without attending anyways?

I think rather #8214 should replace this.


Manage blob replication levels

Some light refactoring, and then adding two queue tasks. One replicates a blob to the target, and the other removes the blob from a target. Use the first to replicate blobs whenever they are created to config.BLOB_REPLICATION_FACTOR nodes. The deletion task isn't used yet.

One thing which worries me about managing blob replicatation down is that we don't current have any locking on blobs anywhere in the code base. I worry about a blob which has been fetched to a node to start an instance, but the instance has not yet started, and then the cleaner removes the blob causing the instance start to fail. I think we need to do a bit of a bench audit to ensure we have the right locks around blobs.

Although maybe that's not true because an instance in state INITIAL is counted as a user of the blob in the Blob.instances() code.

I am running this branch on the CI cluster to shake out any weirdness, but it seems to mostly work without smoke coming out.

I'm going to convert this one to a draft and try and split out logical steps now that it works into separate PRs.

This is now the remainder of that series after the other things had been teased out.


docs: DoubleMLEstimator document and sample notebook

Related Issues/PRs
https://github.com/microsoft/SynapseML/pull/1715
What changes are proposed in this pull request?
DoubleDMLEstimator document

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md

link
:warning:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

@memoryz

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
78
100
69
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
87
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
87
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
87
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Also theres still a few issues in acrolinx that are pretty simple to fix

nit: theres still a few issues in acrolinx that are pretty simple to fix. LMK if you cant access the scorecard above

The oldest one acrolinx link to this pull request URL, I think the lastest scorecard passed as it shows "successfully processed".

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
87
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
100
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
100
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Codecov Report

Merging #1730 (75aed02) into master (94ed685) will decrease coverage by 61.06%.
The diff coverage is n/a.

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

/azp run

/azp run

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Word andPhrases Score
CorrectnessScore
Scorecard
Processed

website/docs/features/causal_inference/about.md
100
100
100
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

/azp run