kernel will die with empty inputs

providing empty qrel/run file leads to kernel crash!
qrel = {}
run = {}
evaluator = pytrec_eval.RelevanceEvaluator(qrel, pytrec_eval.supported_measures)
results = evaluator.evaluate(run)

It should be catched in anyway.

This should be fixed in here: https://github.com/seanmacavaney/pytrec_eval/blob/master/tests/pytrec_eval_tests.py#L224


Add all reserved words for the languages.

For example for Go
(https://golang.org/ref/spec#Keywords)

This is partially fixed, need to figure out which functions are being used in the stack implementation and also count them as reserved words.


resume training

Hello,
I am trying to resume training for celeb dataset from your check point, I changed the inpainting_celebahq.json  like the instructions.
  "path": { //set every part file path
        "base_dir": "experiments", // base path for all log except resume_state
        "code": "code", // code backup
        "tb_logger": "tb_logger", // path of tensorboard logger
        "results": "results",
        "checkpoint": "checkpoint",
        "resume_state": "experiments/train_inpainting_celebahq_221006_180531/checkpoint/200",
        "resume_state": "200"
        // "resume_state": null // ex: 100, loading .state  and .pth from given epoch and iteration
    },

I inserted the 200.state and 200_Network.pth in the "200" folder,  but after running the command
 python run.py -p train -c config/inpainting_celebahq.json
The training doesn't start. I don't even get an error, I only get Close the Tensorboard SummaryWriter. in the ouput.
What is the correct way of resuming the training.

Sorry for this error, and you can comment out the try in https://github.com/Janspiry/Palette-Image-to-Image-Diffusion-Models/blob/136b29f58d0af6e5db9f3655d2891f5a855fcdaa/run.py#L56 to get the exact traceback info.


Input Filter Behavior Option/Toggle

Take Apigility out of the equation for a moment...
Default Input Filter Behavior:
    $inputFilter = new InputFilter();
    $inputFilter->add(array(
                'name' => 'foo',
                'required'=>true,
                'allow_empty'=>false,
                'filters'=>array(
                    array('name'=>'StripNewLines'),
                    array('name'=>'StripTags'),
                    array('name'=>'StringTrim'),
                ),
                'validators'=>array(
                    array('name'=>'StringLength',
                    'options'=>array(
                        'min'=>3,
                        ),
                    ),
                ),
            )
        );

       $inputFilter->setData(array('foo'=>' bar<a>', 'baz'=>'boo'));

       $inputFilter->isValid();  // returns true
       $inputFilter->getValues(); // returns array('foo'=>'bar')

Even though we are passing a value for a field that was not defined, the input filter is valid and returns only the data that it has instructions for. This is expected behavior.
Now, let's say you compose the same input filter via Apigility for an API endpoint and submit the same array of data. The API will respond with a status 400 (Bad Request) with the error message of: Unrecognized Field "baz"
I understand the API throwing this error; I get it. However, here is a use case for having the API endpoint not throw that error and simply ignore the undefined fields just like the default behavior of the input filter:
Two API endpoints that represent two database tables:
/artists  (artists table:  id, artists_name)
/albums (albums table: id, artist_id, album_name)
When retrieving an album through /album/:id, you may want to include additional fields in the response via the way of a JOIN:
{
  id:1,
  artist_id:12,
  album:"Best Hits",
  artist_name:"Dull Needle & The Statics",
}

It is much more efficient to include the artist's name in the album request instead of making two API calls.
Ok, so what does this GET response have to do with input filters?
We have several apps built using Angular.js since it is great at two way data binding and we can easily push the modified data models up to the server (PATCH, PUT) when the user edits a view.  With the current behavior, we would have to clone the data model and filter it in the angular controller, then push it up to the API. In the example above, if you were to leave the model untouched, you would get a 400 response. If a new joined field was added to the response payload you would also have to update the js controller to filter that out for PATCH and PUT operations.
Had the API acted exactly as the InputFilter example, nothing would have to be done on the view layer or in the angular controllers. We could confidently know that extraneous data would be ignored and removed via way of the input filter. We could pass complex relational data via GET and not have to replicate filtering logic in the JS or view side of things for POST, PUT, PATCH.
There should be an option on how to handle this use case. At the very least, it should be up to the developer to decide wether the extra data constitutes an error or not.
Sorry for the length; just tried to explain it clearly.

This repository has been closed and moved to laminas-api-tools/api-tools; a new issue has been opened at https://github.com/laminas-api-tools/api-tools/issues/69.


OpenIdConnectProtocolValidator should provide for validation of id_token_signed_response_alg

See oidc spec https://openid.net/specs/openid-connect-core-1_0.html#IDTokenValidation
We should be able to check this during OpenIdConnectProtocolValidation. Perhaps, acceptable algorithms property on the context passed to validate.
Assuming that discovery was used, there isn't much of an attack here.
See: Laura Rodriguez comment here:
https://github.com/dotnet/docs/issues/5321

Users can specify acceptable algorithms with this change: https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/1413


Error validating acs response

I'm adding OneLogin authentication to a Django rest application.  In local development, things are working fine.  When I try to deploy into our kubernetes cluster, I get
Element '{http://www.w3.org/2001/XMLSchema}element', attribute 'ref': The QName value '{urn:oasis:names:tc:SAML:2.0:assertion}Issuer' does not resolve to a(n) element declaration.

When I decode the response, the issuer matches what is in the settings file.  I believe the OneLogin configuration that responds to localhost and to the cluster are generally the same (just the SP server name is different).
Is there something I'm missing?  Thanks!

Never saw that before, is your kubernete cluster reading properly the xsd files to validate the XML.
Are the XML the same on local and kubernete (maybe a parse issue)?


Use Maven build tool

Use Maven build tool.
benefits:

dependency management and versioning
able to upload to maven central repository
instead of providing a jar file in your github repo, those who use maven can automatically download the jar file by providing a dependency

for example in my github repo util-classes you can use the jar file by providing the dependency in the pom.xml file
<dependencies>
    <!-- other dependencies are there -->
    <dependency>
        <groupId>io.github.pitzzahh</groupId>
        <artifactId>util-classes</artifactId>
        <version>1.0.4</version>
    </dependency>
    <!-- other dependencies are there -->
</dependencies>

If you know maven
follow the instructions here
instructions

If you know maven follow the instructions here instructions

oh, cge check ko, thank you

PR requested: https://github.com/soybean15/CScreen/pull/3#issue-1410508045


Closing command sign creation without selecting a location causes problems.

Using CommandSigns 2.1.0 on Paper 1.14.2.
Type /commandsign create
Select "done" without selecting a block.
Try to view the broken sign with /commandsign list
Invalid command sign is saved despite having no location data and throws an error if it should appear in /commandsign list.
Error Log - https://pastebin.com/raw/U07VzqP4
Invalid Sign - https://pastebin.com/raw/fwMkc1fp

I'll check that after work.
You shouldn't be able to create a command sign without a block selected

Fix in version 2.0.1 (for Spigot 1.13) and 2.1.1 (for Spigot 1.14)

Yeap, that certainly stops it. Thanks.
Maybe an option to cancel creation would be nice. But beyond the scope of this ticket :)


Bug: Visual IC boxes disappear

There are green (monomorphic) yellow (polymorphic) red (megamorphic) indicators that show up in the code editor. However, this only shows up the first time a log is loaded. It disappears when reloading, or when changing the file in the editor.
To fix this I've been restarting vscode and then loading the log again.

I've seen this happen sometimes as well. IC indicators sometimes seem to fail when there are a large number of editor decorations to apply. I've set a limit at 10,000 decorations, but that number may actually be too high.
So far it seems to be a VS Code issue, but if you have a specific repro I can use, I can verify you aren't seeing some other issue.


only deduplicate files in ls if they have the same type

this PR fixes a bug in the file browser's ls function where if folders and files held the same name, one of them would be deduplicated and removed from the file list. ls now checks not only for equivalent filenames, but also equivalent types.

Could you provide an example? Is it possible to have a file named 'asdf' and a folder also named 'asdf'? That strikes me as something we should fix.

if you had a file named asdf and a folder named asdf, which one appeared in the file list would be whichever appeared in the un-parsed list first. This PR fixed that and was merged


Performance & SW: FirebaseError: Performance: Custom metric name _fp is invalid

Version info

Angular: 8.3.0
Firebase: 6.4.1
AngularFire: 5.2.1
How to reproduce these conditions
Steps to set up and reproduce

ng new FirebasePerfErr --skip-tests
ng add @angular/fire
ng add @angular/pwa
Add firebase config object to your environment.ts file
app.module.ts :

  imports: [
    BrowserModule,
    AngularFireModule.initializeApp(environment.firebase),
    AngularFirePerformanceModule,

ng serve

Debug output
Uncaught FirebaseError: Performance: Custom metric name _fp is invalid (performance/invalide custom metric name).
    at<EMAIL_ADDRESS>(http://localhost:4200/firebase-performance.js:2155:33)
    at<EMAIL_ADDRESS>(http://localhost:4200/firebase-performance.js:2140:18)
    at<EMAIL_ADDRESS>(http://localhost:4200/firebase-performance.js:2252:23)
    at setupOobTraces (http://localhost:4200/firebase-performance.js:2368:15)
    at http://localhost:4200/firebase-performance.js:2334:37
    at ZoneDelegate.invokeTask (http://localhost:4200/polyfills.js:3626:31)
    at Zone.runTask (http://localhost:4200/polyfills.js:3403:47)
    at invokeTask (http://localhost:4200/polyfills.js:3700:34)
    at ZoneTask.invoke (http://localhost:4200/polyfills.js:3689:48)
    at timer (http://localhost:4200/polyfills.js:5885:29)

Expected behavior
With Angular 8.2.2 and Firebase 6.3.5 (& @angular/fire 5.2.1) I did not have this issue.
Actual behavior
Getting error:  FirebaseError: Performance: Custom metric name _fp is invalid

Seems fixed with Firebase 6.4.2

Thanks for reporting the issue. Please update to the latest version. It should be fixed as of this pr:
https://github.com/firebase/firebase-js-sdk/pull/2110


Compaction today uses a Avro file scan to read all the records

It should use the logVersion and logOffset in HoodieDeltaWriteStat - from the delta commits to read all the records (avro log files can have corrupted blocks)

This requires #149 to be done first


Feature/179 join via chat

Introduced JoinGameHelper as general utility for joining games from UI side.
Reworked password dialog
Connected GameTilesController and GamesTableController to new JoinGameHelper
Added tests for JoinGameHelper and GameTileController
Small changes for better test

thxalot!


Cartesian product

Description
Add documentation for the cartesian product operator which optimizes some queries when matching nodes that have no relationship.
Pull request type
Please check what kind of PR this is:

[X] Fix or improvement of an existing page

Related PRs and issues
PR this doc page is related to:
https://github.com/memgraph/memgraph/pull/1193
Checklist:

[X] Check all content with Grammarly
[X] Perform a self-review of my code
[X] Make corresponding changes to the rest of the documentation (consult with the DX team)
[X] The build passes locally
[X] My changes generate no new warnings or errors
[ ] Add a corresponding label
[ ] If release-related, add a product and version label
[ ] If release-related, add release note on product PR

@Josipmrden - can you please merge into the memgraph-2-12 branch? thank you!


BugFix: Fix Be crash while doing schema change

Add a check for consistency between the nullable of the new column and the old column when doing schema_change
If nullable is inconsistent, new column can not be assigned as old column directly
close https://github.com/StarRocks/starrocks/issues/1296

please add test case

https://github.com/StarRocks/starrocks/blob/b5e98d83d5c855a761b1f95fffcafcccf23301c9/be/src/storage/vectorized/schema_change.cpp#L340
Can append_column achieve this function?

https://github.com/StarRocks/starrocks/blob/b5e98d83d5c855a761b1f95fffcafcccf23301c9/be/src/storage/vectorized/schema_change.cpp#L352
check thre return ptr of slice.data = reinterpret_cast<char*>(mem_pool->allocate(slice.size));

https://github.com/StarRocks/starrocks/blob/b5e98d83d5c855a761b1f95fffcafcccf23301c9/be/src/storage/vectorized/schema_change.cpp#L352
check thre return ptr of slice.data = reinterpret_cast<char*>(mem_pool->allocate(slice.size));

fix in https://github.com/StarRocks/starrocks/pull/1297/commits/fb63cfe122c9c9e9e0f17da6c108800515589c60

https://github.com/StarRocks/starrocks/blob/b5e98d83d5c855a761b1f95fffcafcccf23301c9/be/src/storage/vectorized/schema_change.cpp#L340
Can append_column achieve this function?

if column type is OLAP_FIELD_TYPE_CHAR, the size of slice should be reset to length of new column, otherwise append_column can replace append_datum. PTAL https://github.com/StarRocks/starrocks/commit/fb63cfe122c9c9e9e0f17da6c108800515589c60


Update README.md

Closes #397

@aditigopalan do you also want to review and let us know if you have any questions or feedback on the updated processes outlined in the readme?

@adamjtaylor @aclayton555 looks good to me, thanks so much for updating!


Use helmet (or something similar) by default

Feature request
Is your feature request related to a problem? Please describe.
I was removing my custom server to make my website's fully serverless , and I ended up removing the helmet package in the process. This kind of sucks, because it adds a lot of cool security features.
Describe the solution you'd like
I'd love to have all of these headers applied by default, by Next, or at the very least a simple way to apply them without creating a custom server.
Describe alternatives you've considered
I can re-create a custom server, but I'd rather not.

Would https://github.com/zeit/next.js/issues/7208 help you in achieving this?

You can add it in getInitialProps of _document 👍

It would definitely help - great feature btw @timneutkens - but why can't these headers / best practices be the default as well?

Basically all cases require extensive configuration of headers.

Hm, well I guess some of them do. But setting them up with the approach in #7208 would definitely help :)
I guess this can be closed then. Looking forward to see your RFC become a reality!


Support for cjs and jest

Works for https://github.com/stackframe-projects/pgmock/issues/6, https://github.com/stackframe-projects/pgmock/issues/11

+1 This has solved the jest problems. It runs great. Can we get it merged?

@N2D4 Hi, could you review this request?

I would love to see this merged so I could give it a shot!

@N2D4 @andymitchell @cgatian As a temporary solution, I published @zfben/pgmock that includes updates. And it will be deprecated after this request merged.

@N2D4 @andymitchell @cgatian As a temporary solution, I published @zfben/pgmock that includes updates. And it will be deprecated after this request merged.

Awesome. I'll give it a try!

Thanks @zfben !

Sorry for having been slow with this stuff, and thank you so much for the contribution!


20161019-RESTful APIs

Thoughts? I'd really enjoy a session on make files.

@mobiusklein @bcow feel free to add more information as needed

from @mobiusklein
As I understood it, you wanted to me to talk about how to talk to REST APIs out in the wild. I can do that readily using several resources I use:
GlyTouCan - A database of glycan structures with a generic REST JSON API and a SPARQL Endpoint.
UniProt - Each protein or gene can be accessed with REST, and retrieved in several generic formats including JSON, XML, and RDF, as well as more domain specific flat files. UniProt also has a SPARQL Endpoint.
Entrez - The "Web Service" side of GenBank, the National Center for Biotechnology Information has some REST components and some non-REST components which could be useful for demonstrating why REST is a good thing. They provide data in XML, their own flavor of flat files, JSON, and ASN.1.
I have written clients for the first two, and kind of wrote a half-baked client for the 3rd because I was told I couldn't use BioPython, whose client is better. I do this work in Python, but I can show how parts could be done with cURL and grep or R's analogs.
Format Glossary
XML, short for eXtensible Markup Language
The format of choice for enterprise software and hierarchical data. Can encode many relationships in a variety of ways without any consistency and is considered nearly illegible to human beings. Despite that, it was the gold standard in data exchange on the web during the 80s and 90s. It still is used often today, but JSON has supplanted a large market share.
JSON
JavaScript Object Notation. What the "cool kids" are using these days for data exchange because its easy, quick, and expressive. Serializes complex, arbitrarily nested data structures of key-value pairs (objects), arrays, numbers, and strings, and has one of the most optimized parsers on the planet.
RDF, short for Resource Description Framework
A notion for representing semantic networks in one of many different formats, including XML. Semantic Web technology is roughly 10 years old, but has just started to really see serious use in Life Sciences. It allows you to encode arbitrary first order predicate logic in an expressive fashion. This is a W3C specified format and supports "distributed knowledge bases", meaning that it does not assume that all resources reside in the same system, so all resources must be fully qualified URIs (universal resource indicators, a superset of URL)
SPARQL, recursively SPARQL Protocol and RDF Query Language
A query language (like SQL) for querying Semantic Web networks. Rather than write a million REST services to wrap individual SQL queries on your server, you instead expose a SPARQL Endpoint and allow the user to run arbitrarily complex queries they make for themselves. This is better than just providing an SQL Endpoint since most SQL vendors don't provide a way to prevent destructive operations from being executed, or for protecting certain tables from being accessed. Since all you need to transfer to the server is your query, a SPARQL Endpoint satisfies the definition of a REST API.
ASN.1, short for Abstract Syntax Notation 1
If you've ever used a device with multiple communicating microprocessors made in Germany, chances are those processors are communicating in ASN.1. It has more in common with serialization frameworks like Protobuf and Avro than it does with XML or JSON, and parsers are hard to write or expensive to buy because of the complexity therein.

How much does the audience know about SQL, JavaScript, and XML? These are probably things I take for granted that I need to gauge how much to not treat these as good things to make comparisons with.

I also created a etherpad for the session at:
https://public.etherpad-mozilla.org/p/BU-RESTful-APIs

@bcow and @mobiusklein - how are you feeling about Wednesday?

I'm still up for presenting how to use REST APIs. I've put together  a few slides, but after that I'll be switching primarily to demonstrative live-coding.
There will be an interactive step since I'll be showcasing a Swagger document, a tool that helps people test out REST APIs in the browser and shows you the equivalent curl command.
Any idea about the audience's familiarity with the topics I mentioned above. If not, I'll try to avoid comparing and contrasting things with them.

not sure about everyone's familiarity, you might have to poll the room and adjust from there.  if there's anything ppl should do before the event (sign up / download anything) you can just  add it to the etherpad.

Sounds good - I’ll probably have one or two slides as well and a web demo.
I know SQL pretty well but I would not assume that everyone from Earth &
Environment has SQL experience.
On Tue, Oct 18, 2016 at 4:27 PM Tom Hohenstein<EMAIL_ADDRESS>wrote:

not sure about everyone's familiarity, you might have to poll the room and
adjust from there. if there's anything ppl should do before the event (sign
up / download anything) you can just add it to the etherpad.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bulib/studyGroup/issues/67#issuecomment-254629055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFP6a2ue3PrvgB7v6x_-rQdl7Px2AWKRks5q1SuRgaJpZM4KKFTX
.

great session!
counting: 8 ppl


[KYUUBI #5210]Cancel operation will cause the log file node to leak

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?

Codecov Report

Merging #5211 (34dcc57) into master (b1eef09) will not change coverage.
Report is 2 commits behind head on master.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master   #5211   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         589     589
  Lines       33238   33238
  Branches     4387    4387
======================================
  Misses      33238   33238

Files Changed
Coverage Δ

.../org/apache/kyuubi/operation/KyuubiOperation.scala
0.00% <0.00%> (ø)

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

thanks, merged to master/1.8/1.7


[Fix] Cannot read properties of undefined (reading 'country_code')

Fixes https://github.com/nchaulet/node-geocoder/issues/350.
I would really appreciate if this could be merged in the near future, as it's affecting the uptime of my app.
Usage (the try/catch doesn't help):
  await geocoder
    .geocode(privateListingRes.address)
    .then((entries) => {
      const entry = entries[0];
      if (entry?.latitude && entry.longitude) {
        latitude = entry.latitude;
        longitude = entry.longitude;
      }
    })
    .catch(() => undefined);

Error:
TypeError: Cannot read properties of undefined (reading 'country_code')
    at OpenStreetMapGeocoder._formatResult (/home/ec2-user/superset/node_modules/.pnpm/node-geocoder@4.2.0/node_modules/node-geocoder/lib/geocoder/openstreetmapgeocoder.js:71:38)
    at /home/ec2-user/superset/node_modules/.pnpm/node-geocoder@4.2.0/node_modules/node-geocoder/lib/geocoder/openstreetmapgeocoder.js:58:34
    at tryCatcher (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/util.js:16:23)
    at Promise.successAdapter [as _fulfillmentHandler0] (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/nodeify.js:23:30)
    at Promise._settlePromise (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/promise.js:601:21)
    at Promise._settlePromise0 (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/promise.js:649:10)
    at Promise._settlePromises (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/promise.js:729:18)
    at _drainQueueStep (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/async.js:93:12)
    at _drainQueue (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/async.js:86:9)
    at Async._drainQueues (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/async.js:102:5)
    at Immediate.Async.drainQueues (/home/ec2-user/superset/node_modules/.pnpm/bluebird@3.7.2/node_modules/bluebird/js/release/async.js:15:14)
    at processImmediate (node:internal/timers:466:21)

@nchaulet please take a look!

Not sure why the tests were failing but I updated w/ the code I used in my patch.


fix: RN 75 build errors

Please merge this PR.
It contains a simple one-line change that resolves an issue with the new React Native versions.
We prefer not to create a fork for such a minor change.

I'll take a deeper look at this PR once our office is back open after the holidays. I want a deeper understanding of why type casting is necessary here.


add mock eth1 to quickstart

quickstart should not use a real eth1, but create / use a mocked eth1

https://github.com/ethereum/eth2.0-pm/pull/72/files


THIS "QUICKSTART" CODE THROWS "ModuleNotFound: Azure" EXCEPTION WHEN running first few lines !!!

SUCCESSFULLY COPIED/PASTED/EXECUTED the following cell in a JupyterNotebook page in a VSCode project:
pip install azure-cognitiveservices-vision-customvision
AFTER THAT I IMMEDIATELY GOT EXCEPTIONS WHEN COPYING/PASTING/EXECUTING the next line of code:
from azure.cognitiveservices.vision.customvision.training import CustomVisionTrainingClient
ERROR DISPLAYED IN VSCODE:
ModuleNotFoundError                       Traceback (most recent call last)
in
----> 1 from azure.cognitiveservices.vision.customvision.training import CustomVisionTrainingClient
ModuleNotFoundError: No module named 'azure'

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

ID: 4f6c3f20-25a4-bc19-0cb6-377eff01b00e
Version Independent ID: 02d76c7b-cffa-c94f-2298-ba7e506d4f79
Content: Quickstart: Image classification with Custom Vision client library - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/quickstarts/image-classification.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

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

@markvogt Thanks, we are able to install successfully in the Jupyter notebook. Please find the snapshot for the same.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Add commands for importing and renaming worlds

I've had this code sitting on my system for a long time.  It seems like it could be useful.
I would just like to get at least one other person to test it before I merge it.


Move out search analytics script to own repo

This script is easier changeable if it lives in the search-analytics
repo.
Added here: https://github.com/alphagov/search-analytics/pull/7
https://trello.com/c/kAm186hM

👍

I like small Jenkins jobs :+1:


Use namespaced class names for CSS classes on the host element

Hey all.
I've spent a few hours debugging why we're not seeing the notifications for old versions and PR builds with the new Notification add-on. It turns out that the element is there, but Bootstrap (which our theme uses) hides it because it has the toast class:

See the opacity: 0; at the bottom right of the screenshot. That comes from Bootstrap, and it requires toast to be initialised via JS (hence the initial state has opacity: 0;). You can check here: https://sphinx-wagtail-theme--295.org.readthedocs.build/en/295/
Debugging this was quite hard because the notification immediately disappears after 5s. I had to overwrite the JS and increase the timeout to investigate this.
Is there a reason why the custom element needs to rely on CSS classes in the host element? Could you perhaps use something else, e.g. data attributes?
Or, if you want to use CSS classes, I think they should be namespaced, e.g. prefixed with rtd- so that it doesn't conflict with the theme's CSS (and can still be overridden if that's the desired goal).
Thanks!
P.s. love the addons – DocDiff in particular is really awesome.

Thanks for opening this issue. I understand this should not happen since the notification is a Web Component with its shadow DOM is closed. I will need to take a deeper look to understand what's going on.

Is there a reason why the custom element needs to rely on CSS classes in the host element? Could you perhaps use something else, e.g. data attributes?

@agjohnson shouldn't the web component ignore this external modification? Is it maybe because we left the shadow DOM open somehow?

I can't see the notification on this example, so can't tell you what is happening yet.
But yes, the elements we inject are inside a shadow DOM and should not inherit CSS styling from the document root DOM. We do allow variable customization of some of the CSS properties, all of those variables are prefixed with --readthedocs-....
My guess is there is something else happening here.

@agjohnson Demo'd the issue in #416.
https://readthedocs-addons--416.org.readthedocs.build/
https://github.com/user-attachments/assets/60a80c53-673f-4ece-a213-b9aaa9c5d1c3

@humitos FYI web components don't fully guarantee that styles from the parent DOM won't affect the shadow DOM's contents. Inherited styles, e.g. font-weight, opacity, display, text-align etc. can be inherited by the shadow DOM. Ref: https://open-wc.org/guides/knowledge/styling/styles-piercing-shadow-dom/
You should be able to fix this by adding the following to your web components' styles:
:host {
  /* Reset all CSS properties */
  all: initial;
}

This is a separate issue from the one I described here, however. The above reset will not fix the clashing CSS class issue, because CSS classes have higher specificity. The root cause is still the same though (I think), so this can be fixed if you use namespaced CSS classes for your web component. Or, better yet, don't rely on CSS classes being applied on the host element itself.

I'll just note here that the CSS classes are not an important part of the implementation, though they seemed like an okay pattern originally. I'm feeling like this is a pattern we should just move away from entirely. Prefixing the CSS classes does work, but prefixing and data attributes are both older patterns for what web components give us.
I'd rather we use element attributes here, as class="toast" does have meaning at the code level, outside CSS styles. I'm describing <readthedocs-notification variant="toast"> or similar which shouldn't greatly affect our style rules.

Yep, I think that makes sense! I looked into the code a bit and I thought the CSS classes weren't necessary, but I wanted to make sure.
And yes, custom attributes would definitely be a great solution, since it's a custom element. I noticed that the EthicalAds widget is not a custom element yet (still a div), and it also uses the similar raised class like the custom elements, hence I suggested data attributes in case you wanted to keep it consistent among the other addons. If that consistency isn't important, then I think custom attributes are the way to go.

That's good insight, that's actually exactly where this pattern originated from. When we started the EA client, we weren't yet invested into web components, and so the patterns there are much different. If I had to restart that library today, it 100% would use a web components and similar approach as to the elements here.

I understand the path forward here is:

change class=toast by variant=toast
change all CSS queries where .toast by [variant=toast]

@agjohnson what's the correct way to set variant=toast by default when it's not explicitly defined in the HTML code?

Note that properties (the DOM object's property, accessed by .propertyName) and attributes (the HTML attribute in the DOM tree) are different. In Lit's default behaviour, while attributes are observed (i.e. changing an attribute also updates the property), properties are not reflected (i.e. changing a property's value does not update the attribute).
If you intend to use the custom attributes in CSS, you'll probably also want to make the properties reflective. See: https://lit.dev/docs/components/properties/#reflected-attributes
I don't think I found a component that uses reflect: true in any of the addons.
Ignore me if you already know this 🙂

We haven't needed to use reflection yet, but this is indeed a place we'd need to. Any CSS selector that relies on a value we manipulate -- setting a default value or changing the value for some reason -- should be using property reflection.


add link 2 amazing grandpa dave slides from 2017-08-15 PWA talk wow

this is how amazing this change looks do u love my cascading style sheet skills find me on linkedin

/cc @davatron5000

lookin fresh! thank u bb
http://austinjavascript.com/posts/meetups/2016/08/09/


How do I tell Youtube-dl that I want it to download in the highest resolution available?

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question
I am not very knowledgeable about coding or anything computer related so I apologize if I am missing something obvious. Youtube-dl is not automatically downloading the highest resolution of the videos that are available. The videos I am trying to download are available in 1080p (and can be downloaded in that resolution with other downloaders), but will only download in 720p with Youtube-dl (and sometimes even lower resolution).
I am using a .bat file to use Youtube-dl. The text in that .bat file is copied below. I tried using the "-f" command that was recommended in the faq, but that just caused everything to fail completely.
youtube-dl.exe -i --download-archive downloaded.txt -f best --add-metadata --embed-subs --embed-thumbnail --all-subs -o "Downloads%%(uploader)s%%(upload_date)s-%%(title)s-%%(id)s.%%(ext)s" -a VideosToDownload.txt

install ffmpeg.

Well, if you only take the res into acc, you won't get the best one, instead ytdl uses other means to decide on the best, which seems to cause a mismatch every so often.
But apart from that, ytdl defaults to asking the website for the separate best vid and best audio, otherwise it'll take the best pre-existing combination. You already told it to get the best combination, which is actually undesirable in the case of YT since if the vid has a high enough res (dunno what the criteria for audio is) it'll be split into separate vid and audio (I suspect they separate them in all cases).
^ So, take out the -f best flag. If you need to be more specific in the case of some content you can. But remember to designate the folder of FFmpeg so you can use the bestvideo+bestaudio !
Remember to read the readme.
Examples:
Audio only, or the best combination: --format "bestaudio/best"
Best vid+audio, going from 720p down to 240p, then to 1080p, then to whatever other res is available, then to the best combination, if such options aren't available:
--format "bestvideo[height=720]+bestaudio/bestvideo[height=480]+bestaudio/bestvideo[height=360]+bestaudio/bestvideo[height=240]+bestaudio+bestaudio/bestvideo+bestaudio/best"

The above {c,s}ould be optimized into:
--format "(bestvideo[height=720]/bestvideo[height=480]/bestvideo[height=360]/bestvideo[height=240]/bestvideo)+bestaudio/best"


Uploaded image don't use all the available space when using init-max-area="true"

I've been trying to use ui-cropper to crop and upload an image. To do this, I created a div to show the uploaded image and the crop tool, using the following code:

    <ui-cropper
        image="headerImage | ngfDataUrl"
        area-type="rectangle"
        ng-init="croppedDataUrl=''"
        aspect-ratio="2.6"
        result-image="croppedDataUrl"
        result-image-size="{w:960,h:370}"
        init-max-area="true">
    </ui-cropper>

But when I upload an image, the image is not using the entire space of the ui-crop area, like this:

What can it be?

I am not using this feature, but from what i have seen in my tests, you have to add something in the css. In near feature i will checkup codepens and add one with this feature.

ping

ping

Hey @CrackerakiUA, any codepen where we could look for a solution to this?

i will try to make it this evening.

Please checkup this LINK and leave some comments.

I have the same behavior. I tried all options but no success. Canvas inside croparea is always 300 pw width and 200 px height. How could I change this ?

For some reason, width or height never seen to exceed 300px when I use this library.

i was having same issue.
after i added reference of  ui-cropper.css then canvas worked perfectly.

@smohammedyasin Thanks buddy, that did the trick.

Solved i guess.


Add symbolic Gaussian Square Echo to Qiskit Pulse symbolic_pulses library

Summary
Adding the implementation of a symbolic Gaussian Square Echo pulse
Details and comments
This is a first implementation of a symbolic version of Gaussian square echo pulse. Tests and documentation to be added. I have left the definition of the gap between echo pulses as it was but we may want to look at it more closely. Usually, the two echo pulses are separated by the duration of a single qubit gate which allows to echo the control qubit.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: mamico:x: miamicomamico seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I'm glad to see we have other pulse shapes in Qiskit. If this is going to be backend-supported, you may also want to update
qobj converter,
https://github.com/Qiskit/qiskit-terra/blob/f2befd10b1aba6d9369daf4465cffdf75c7da576/qiskit/qobj/converters/pulse_instruction.py#L36-L46
and qpy loader
https://github.com/Qiskit/qiskit-terra/blob/f2befd10b1aba6d9369daf4465cffdf75c7da576/qiskit/qpy/binary_io/schedules.py#L94-L106

I think I have made the changes to those modules to support the GaussianSquareEcho pulse. I have a question though. I see that all the pulses have a corresponding equivalent in the continuous and discrete modules. Should I implement a gaussian square echo pulse there as well?

Implementation of waveform counterpart, i.e. continuous and discrete, is not mandatory, because we can get the same array with SymbolicPulse.get_waveform method (but nothing stops you to implement).

Picking this up again since this PR was merged recently. As it is now, GaussianSquareEcho only contains the Gaussian square pulses without the single qubit gates for the echo. There is a delay between the Gaussian Square pulses with positive and negative amplitudes where the second single qubit gate for the echo would fit. I wonder if we should include these single qubit gates or not. If not, how would we add the single qubit gate needed for the echo in the delay between the Gaussian squares?


add ICCV 2023 paper

Paper name/title: MRN: Multiplexed Routing Network for Incremental Multilingual Text Recognition
Paper link: https://arxiv.org/abs/2305.14758
Code link: https://github.com/simplify23/MRN
Blog in Chinese: https://zhuanlan.zhihu.com/p/643948935
keywords: OCR, Incremental learning

Hello! Could you add our work?
Paper name/title: Self-supervised Character-to-Character Distillation for Text Recognition
Paper link: https://openaccess.thecvf.com/content/ICCV2023/papers/Guan_Self-Supervised_Character-to-Character_Distillation_for_Text_Recognition_ICCV_2023_paper.pdf
Code link: https://github.com/TongkunGuan/CCD
Keywords: OCR, SSL


Uncaught TypeError: Cannot read property '$element' of undefined

Hi, I found this error while I was testing the drag and drop option of angular-tree-dnd. After that, I saw that in your demo happens the same. Could you fix that? I need to implement this option on my project asap. Thanks.

The same issue happened to me when you try to drag an element form sub level to the root


CSVData shouldn't combine all records -> single sample, default 1

Allows the user to set how many rows are combined together when reading a CSVData with data format records.
Necessary change with update of examples.
filepath = "dataprofiler/tests/data/csv/diamonds.csv"

options = {'data_format': 'records', 'record_samples_per_line': 1}
data = dp.Data(filepath, options=options)
data.data[:5]

with new code (default value 1):
['carat,cut,color,clarity,depth,table,price,x,y,z',
 '0.23,Ideal,E,SI2,61.5,55,326,3.95,3.98,2.43',
 '0.21,Premium,E,SI1,59.8,61,326,3.89,3.84,2.31',
 '0.23,Good,E,VS1,56.9,65,327,4.05,4.07,2.31',
 '0.29,Premium,I,VS2,62.4,58,334,4.2,4.23,2.63']

previous code:
['carat,cut,color,clarity,depth,table,price,x,y,z\n0.23,Ideal,E,SI2,61.5,55,326,3.95,3.98,2.43\n0.21,Premium,E,SI1,59.8,61,326,3.89,3.84,2.31\n0.23,Good,E,VS1,56.9,65,327,4.05,4.07,2.31\n0.29,Premium,I,VS2,62.4,58,334,4.2,4.23,2.63\n0.31,Good,J,SI2,63.3,58,335,4.34,4.35,2.75\n0.24,Very Good,J,VVS2,62.8,57,336,3.94,3.96,2.48\n0.24,Very Good,I,VVS1,62.3,57,336,3.95,3.98,2.47\n0.26,Very Good,H,SI1,61.9,55,337,4.07,4.11,2.53\n0.22,Fair,E,VS2,65.1,61,337,3.87,3.78,2.49\n0.23,Very Good,H,VS1,59.4,61,338,4,4.05,2.39\n0.3,Good,J,SI1,64,55,339,4.25,4.28,2.73\n0.23,Ideal,J,VS1,62.8,56,340,3.93,3.9,2.46\n0.22,Premium,F,SI1,60.4,61,342,3.88,3.84,2.33\n0.31,Ideal,J,SI2,62.2,54,344,4.35,4.37,2.71\n0.2,Premium,E,SI2,60.2,62,345,3.79,3.75,2.27\n0.32,Premium,E,I1,60.9,58,345,4.38,4.42,2.68\n0.3,Ideal,I,SI2,62,54,348,4.31,4.34,2.68\n0.3,Good,J,SI1,63.4,54,351,4.23,4.29,2.7\n0.3,Good,J,SI1,63.8,56,351,4.23,4.26,2.71\n0.3,Very Good,J,SI1,62.7,59,351,4.21,4.27, ........
.
.
.
]

I think this needs updating in a comment to CSVData
https://capitalone.github.io/DataProfiler/docs/0.4.4/html/dataprofiler.data_readers.csv_data.html


chore(website): create index page for node runner manuals

https://discord.com/channels/984015101017346058/1117285651394600960/1132172169430978670
Think I covered the request here, idea being to create a landing page for the node runner manuals that can be more easily linked to.

Looks like someone else got to this in https://github.com/taikoxyz/taiko-mono/pull/14243, up to reviewers which implementation makes more sense


Ideas for using fontmake with Noto

In order to give users more flexibility, it could be interesting to add more options to fontmake. Some useful options could be:

Pair fontmake with scale_ufo.py or scale_font.py to output 2000 UPM fonts from 1000 UPM sources;

Add multiple combining options to fontmake CLI to be able to:

merge writing systems (eg. Noto Naskh Arabic + Noto Sans Latin)
megamerge everything sans into 1 sans family (non cjk)
subset character sets (eg. Noto Latin without IPA and Cyrillic and Greek)
rename binary fonts with custom font names
build Variable Font (possibly slim range, with fontTools.varLib.instancer)
add secure set (currencies, punctuations, basic Latin, figures…)
swap tab and prop figures
swap I J alt
and finally build Static Fonts with some of this previous options

@jbmorizot please could you summarize the conclusions of the call today where this was discussed? :)

It appears that the best way to handle different fonts building (merged, renamed, subsetted, etc.) is to plug existing tools and libraries, and not update fontmake for these specific uses.
That said, to deal with the needs of Noto, the following things could be done:
• Scale_ufo/scale_fonts could be implemented in ufo2ft.
• The best way to handle merging may be at binary level, with the fontTools merger (that could be improved, and is the most interresting tool to build upon).
• Same goes for the megamerge, subsetting, renaming.
• If merging has to be done at sources (= ufo) level, the right way would be to remove duplicated glyphs by using objects provided by ElementTree for updating the .fea file.
One important point is that merging takes longer that subsetting. So a good workaround could be to have a (several) megamerged font(s) and subset it (them) on demand to provide custom fonts.


[CHANGELOG] diem-core-v1.2.0 release

diem-core-v1.2.0 release

/land

:broken_heart: Test Failed - ci-test-success

/land

:broken_heart: Test Failed - ci-test-success

/land


Jekyll image: Fix build failures

https://github.com/devcontainers/images/actions/runs/5247340000/jobs/9477366918
> ERROR:  Error installing github-pages:

Failures to the jekyll image are irrelevant to this change, see [here](https://github.com/devcontainers/images/actions/runs/5247090532/jobs/9477149465?pr=601#step:3:839)

Will open a separate issue to track this.

Originally posted by @samruddhikhandale in https://github.com/devcontainers/images/issues/601#issuecomment-1587881039

Opened - https://github.com/ntkme/sass-embedded-host-ruby/issues/130
Temporary workaround - https://github.com/devcontainers/images/pull/604


[BUG] install_deps on macOS failed

Describe the bug
I try to build GraphScope on macOs in standalone mode, and I install dependencies, but it failed when installing zetcd.
To Reproduce
./install_deps.sh --dev --cn
Screenshots
[2022-06-01T07:20:56+0800]: Installing zetcd. go: found github.com/etcd-io/zetcd/cmd/zetcd in github.com/etcd-io/zetcd v0.0.5 go: finding module for package github.com/coreos/zetcd go: finding module for package golang.org/x/net/context go: finding module for package github.com/coreos/etcd/pkg/transport go: finding module for package github.com/coreos/etcd/clientv3 go: finding module for package github.com/coreos/zetcd/zk go: finding module for package github.com/coreos/zetcd/version go: downloading github.com/coreos/etcd v0.9.4 go: finding module for package github.com/coreos/zetcd/xchk go: found github.com/coreos/zetcd in github.com/coreos/zetcd v0.0.5 go: found github.com/coreos/zetcd/version in github.com/coreos/zetcd v0.0.5 go: found github.com/coreos/zetcd/xchk in github.com/coreos/zetcd v0.0.5 go: found github.com/coreos/zetcd/zk in github.com/coreos/zetcd v0.0.5 go: found golang.org/x/net/context in golang.org/x/net v0.0.0-20220531201128-c960675eff93 go: finding module for package github.com/coreos/etcd/pkg/transport go: finding module for package github.com/coreos/etcd/clientv3 go: finding module for package github.com/coreos/etcd/clientv3/concurrency go: finding module for package github.com/golang/glog go: found github.com/golang/glog in github.com/golang/glog v1.0.0 go: finding module for package github.com/coreos/etcd/pkg/transport go: finding module for package github.com/coreos/etcd/clientv3 go: finding module for package github.com/coreos/etcd/clientv3/concurrency go: finding module for package github.com/coreos/etcd/pkg/transport go: finding module for package github.com/coreos/etcd/clientv3 go: finding module for package github.com/coreos/etcd/clientv3/concurrency go: finding module for package github.com/coreos/etcd/pkg/transport go: finding module for package github.com/coreos/etcd/clientv3 go: finding module for package github.com/coreos/etcd/clientv3/concurrency ../../../go/pkg/mod/github.com/etcd-io/zetcd@v0.0.5/cmd/zetcd/zetcd.go:27:2: reading github.com/coreos/etcd/go.mod at revision v0.9.4: unknown revision v0.9.4 ../../../go/pkg/mod/github.com/coreos/zetcd@v0.0.5/zketcd.go:27:2: reading github.com/coreos/etcd/go.mod at revision v0.9.4: unknown revision v0.9.4 ../../../go/pkg/mod/github.com/etcd-io/zetcd@v0.0.5/cmd/zetcd/zetcd.go:28:2: reading github.com/coreos/etcd/go.mod at revision v0.9.4: unknown revision v0.9.4
Environment (please complete the following information):

GraphScope version: main
OS: MacOS
Version 12.3.1
Kubernetes Version : none
go version: 1.15.14

Additional context
I looked up the repository under CoreOs, it doesn't contain etcd, which may cause the error.
Maybe it's a problem for zetcd, but it seems stop maintenance for a long time.

Thanks for the bug report, can you try to
export GO111MODULE="auto"
before running the script.

it works, thanks a lot.

Thanks for reporting!
It will be fixed with a following PR.


SPD defund

Defund SPD, Michigan

Pr seems to have all history.

I need help to get the PR request label


State Preservation

Figure out how to preserve the state of each component when navigating to a new page

Should be allowed to press the back button and be brought back to the previous page in the state just before the user left it.  For instance, if a user searches "Android" and then views a specific book, when they press back they should see the text "Android" in the search box and all search results beneath it.

Placeing on hold.  This feature can easily be done using Redux which is in the next Udacity module


Move git-specific repo_url logic into git plugin

The implementation of the repo_url helper method had some special logic that was Git-specific. It looked for the presence of :git_http_username and :git_http_password and used these to mutate the original :repo_url.
Since we eventually want to move all SCM implementations out of core and into separate plugins, I have moved this code into the Git plugin.
Also add a unit test (this feature was previously untested).
Fixes #1715.

@leehambley @will-in-wi any strong opinions on this PR or #1858? I'll probably merge them next week.

LGTM!


[BUG]: The events of the PaypalButtons component don't catch the updated external variables

Is there an existing issue for this?

[X] I have searched the existing issues.

🐞 Describe the Bug
The events of the PaypalButtons component miss the changes of the variables outside of the event, for example in the createOrder event I'd like to use a state outside of the event, like a state value, when the state value change, inside the event has not detected this change and use the previous value of the state.
😕 Current Behavior
The external values of the variables, states, or props values  used  in the events like createOrder are kept
🤔 Expected Behavior
The external values of the variables, states, or props values  used  in the events like createOrder have  the updated value
🔬 Minimal Reproduction
I created a [codesandbox](https://codesandbox.io/s/paypal-bug-test-kdckvc?file=/src/App.js:726-909) with the example of the issue

In the example, we can find two buttons, a regular button, and the Paypal button, there is a state with a boolean value, it's named test.

The regular button changes the state and there is a useEffect that shows the test state value when its value changes. you can see it on the console

If we click on the Paypal button the test value is the previous one, inside the event the value never changed

🌍 Environment
| Software         | Version(s) |
| ---------------- | ---------- |
| react-paypal-js  |     7.8.9  |
| Browser          |     any    |
| Operating System |     any    |

Relevant log output
No response
Code of Conduct

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

➕ Anything else?
No response

I found the solution. I figured out the prop forceReRender

For things like amount, you can useRef and reference amountRef.current in the createOrder function closure and it will stay up-to-date as well. Frankly, this needs a better callout in the docs!


[Bug]: when running img2img error

What happened?
when use img2img, error cameout. Although it can be carried out normally, but every step will prompt an error, what is the problem?
Colab cell output
0% 0/61 [00:00<?, ?it/s]Error executing callback cfg_denoised_callback for /content/drive/MyDrive/stable-diffusion-webui-colab/stable-diffusion-webui/extensions/stable-diffusion-webui-two-shot/scripts/two_shot.py
Traceback (most recent call last):
  File "/content/drive/MyDrive/stable-diffusion-webui-colab/stable-diffusion-webui/modules/script_callbacks.py", line 175, in cfg_denoised_callback
    c.callback(params)
  File "/content/drive/MyDrive/stable-diffusion-webui-colab/stable-diffusion-webui/extensions/stable-diffusion-webui-two-shot/scripts/two_shot.py", line 506, in denoised_callback
    if self.enabled and params.sampling_step < self.end_at_step:
TypeError: '<' not supported between instances of 'int' and 'str'

Which colab and model(s) were you using when the error occurred?
https://colab.research.google.com/github/camenduru/stable-diffusion-webui-colab/blob/drive/run.ipynb
Which Public WebUI Colab URL were you using when the error occurred?
gradio.live
If you used HiRes mode when the error occurred, please provide the Hires info
No response

It would have been interesting to have your prompt and two-shot settings, because apparently, it's a two-shot bug. It tries to compare if the parameter sampling_step is inferior to the parameter end_at_step. The problem is one is an integer (Real number like 27) and the other one is a string (a following of characters like this "27"). Therefore it can't compare the numbers, because one of them is a text, it's like trying to do if 27 < magma which doesn't makes sense for it.


Confusing wrong results of tl.sum

I am running a simple sample. I load two D(=64) elements from memory and I want to calculate the dot product as follows. However, I find that tl.sum returns confusing wrong results.
 d_offsets = tl.arange(0, BLOCK_SIZE_D)
i_ptrs = i_start_ptr
og_ptrs = og_start_ptr + d_offsets
ag_ptrs = ag_start_ptr
 i = tl.load(i_ptrs)
w_ptrs = w_ptr + i*stride_wm + d_offsets
og = tl.load(og_ptrs)
w = tl.load(w_ptrs)
a_grad = tl.sum(og * w, axis=0) #### here tl.sum returns confusing and buggy results
tl.atomic_add(ag_ptrs, a_grad)
For example, og is：
tensor([ 1.6916, -2.5078, -0.0033, -0.0592, -0.3384, -0.7056, -0.1013,  1.6383,         -1.3259,  0.3731, -0.6718,  2.0669,  0.0321, -0.4966,  0.4508, -1.4682,         -2.6196,  0.4923,  1.1060,  0.6657,  0.7153,  0.4815,  0.9671, -0.1847,         -1.7997, -0.6160, -0.6303, -1.2660,  0.5478,  0.2037, -0.9250, -0.8854,          2.0756, -0.7088, -1.6158,  2.3636,  0.4683,  0.9704, -0.5309,  0.9216,         -0.3219, -2.2406, -0.8657,  0.2703,  2.3221, -0.2172, -0.9697,  0.4703,         -1.1627, -0.6571,  0.0471, -0.6415, -1.6859,  0.3609,  0.6448, -1.7089,          1.5976, -1.0887,  0.6811, -1.8891,  0.7437, -1.6817, -0.4980,  0.2190],        device='cuda:0')
and w is:
tensor([-0.6188, -0.9096, -0.9603, -2.2475, -0.8947,  0.5095, -0.4408, -1.2311,          1.1195, -1.1724, -0.0710, -0.3652,  0.9777,  2.3192,  1.6981,  0.4374,          1.1976, -0.6147,  1.1315,  0.0893,  1.5470, -0.0380,  1.0146,  1.5697,         -0.2959, -0.0326,  0.1034, -0.4230,  0.9715, -0.3151,  1.0171, -0.2515,         -0.1225,  0.4729, -0.9266, -0.4360, -0.5116, -1.6078,  0.3006, -0.2831,         -0.3081,  0.0149, -0.6390,  0.8726, -0.3815,  0.6331, -0.2095, -0.7015,          1.6649, -0.7005,  2.1741,  0.3759,  0.7326,  0.6088,  0.1208,  1.3127,          0.5000, -0.1982,  0.8699,  0.7272, -0.4705,  0.6378,  2.2552, -0.0503],        device='cuda:0')
However, the result is -15.1852 while the true value is -13.6172. I am running on RTX 3090, cuda 11.3 and triton 1.1.0.

@ptillet , I have tried to change the num_warps and it works. However, I met another problem. If I want to achieve a loop in the kernel, I found the results of tl.sum is still wrong even I have changed the num_warps.

Should be fixed. Please reopen with a minimal repro if it still doesn't work.


Drive letters are not consistent on Windows

I see  both c:\\ and C:\\, which i'm sure screws up the cache and makes everything load twice. Do we normalise them? If so, to which? FWIW I see c:\\ as the initial load, seemingly coming out of VS Code, and then C:\\ for our recursive imports.
I'm seeing this when trying to load a session by creating a separate session for each file, which I hope will all resolve to the same session. The difference between drive letters stops that happening.

We’ve never normalised drive letters. The only normalisation that we perform at the moment is normalising file path slashes.

It seems there are some nuances to doing it there: https://github.com/digital-asset/ghcide/blob/master/src/Development/IDE/Types/Location.hs#L54-L56
I can't imagine it's not causing us issues somewhere, perhaps double loading on Windows.

That code explicitly does not normalise drive letters. IIRC the issue was that if we do try to normalize them without normalizing them in haskell-lsp we get VFS lookup failures. So this needs to change in both NormalizedUri in haskell-lsp and here or we’ll run into issues.

Did this get fixed in #152 ? Or are there more issues here?

This should be fixed by the haskell-lsp upgrade which fully normalises file paths and #266 on our side.


Changes to support Node 16

Changes to support Node16

[x] add V2 for emailReportTask
[x] V2 should support Node16
[x] Add pipeline to automate deployment
restructure, rename manifest file
update required pacakges
rename V2 extension
rename to V1, add new package commands
restructure extension, add webpack to package application
resturcure, setup packagin for dev and release
add initial pipeline to build, package and deploy emailReporting extension

@madkoo there's a lot of added intermediary files that I don't think should be committed. Can you clean that up so it's easier to do the review?

@MOlausson , indeed included files that shouldn't have gone in. Cleaned up now.


Graph in sidebar does not load in SAFARI

Describe the bug
Graph in sidebar does not load in SAFARI 17 though Chrome and FF work.
Operating System
SAFARI 17 (186<IP_ADDRESS>.22) + MacOS Ventura
To Reproduce
Steps to reproduce the behavior:
Open graph sidebar, see no graph
Screenshots

IMPORTANT: Error Log

Can you check if you have hardware acceleration turned off in Safari? I think the error "unable to find a suitable renderer" is the actual problem.

Unfortunately, as of macOS Catalina (version 10.15), Hardware acceleration is automatically enabled, and there is no way turn this on and off. What now? :-/

I am not sure, I do not have a Mac to test this with and I don't see anyone else having this problem. Have you tried other devices other than yours?

I tried it on an INTEL Mac too with the same effect in SAFARI. On both machines Chrome works though, so I will necessarily stick with that.
Thanks!

Can you try this again using the latest version 1.8.0?
And can you also verify whether this webpage is able to load the graph view or not?
https://notes.nathang.dev/


[enhancement]: hardware minimums document

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
I would love to see documentation for invokeai detailing hardware minimums for the models it supports.  I have an RTX 3060 which has three times the ram (12GB) compared to the minimum specified in the invokeai readme (4GB), but I still get a cuda out of memory error for the 2.1-768 model.  Initially I thought it might be an invokeai bug, but all the other models run fine.  So now I think that it's telling the truth, and that my card really ISN'T good enough for the 768 generations.
A potentially useful bit of additional information is that I'm not doing inpainting or img2img or other stuff where it could be having a problem because I'm trying to give it an image that's too big -- it gets this problem when I'm trying to do pure prompt based generation.
Alternatives
I guess one alternative would be to continue to use the single hardware minimum specification in the main readme but drop support for models that require more than that (or increase the minimum to match whichever model is the hungriest, but that has the downside of losing a bunch of potential users who don't have good enough hardware for the hungriest model).
Aditional Content
No response

Strange I have the 3060 12gb too but cant replicate cuda out of memory error on any output size with 2.1 768 even on 2048x2048.
I tried 960x1280 with hires fix, used 71% vram and takes about 30sec on 40 steps to make the output.

Strange I have the 3060 12gb too but cant replicate cuda out of memory error on any output size with 2.1 768 even on 2048x2048.
I tried 960x1280 with hires fix, used 71% vram and takes about 25sec on 40 steps to make the output.
Honestly there is so many models out there that give better outputs then sd 2.1 I rarely use it.
Edit: I did just remember I am using xformers so it might be making a difference.

what is xformes?

what is xformes?
It give better performance and less memory usage on cuda based gpus,

Install instructions here
https://github.com/invoke-ai/InvokeAI/blob/main/docs/installation/070_INSTALL_XFORMERS.md

what is xformes?

It give better performance and less memory usage on cuda based gpus
follow the pip Install instructions here if you wish to try it out https://invoke-ai.github.io/InvokeAI/installation/070_INSTALL_XFORMERS/?h=xformers

I followed the steps and it works now, thanks!  In another comment you mentioned that sd 2.1 isn't generally your choice.  What would you recommend as a better model or models?  Or do you use fine tuned stuff which would mean the answer is "depends on what you're trying to make"?

I followed the steps and it works now, thanks! In another comment you mentioned that sd 2.1 isn't generally your choice. What would you recommend as a better model or models? Or do you use fine tuned stuff which would mean the answer is "depends on what you're trying to make"?

Yes it is a depends on what you want from it situation
At the moment I have been trying out the Protogen models https://huggingface.co/darkstorm2150/
I am Finding I am getting the results want from the Protogen Eclipse model (https://huggingface.co/darkstorm2150/Protogen_Eclipse_Official_Release)

vs SD 2.1 768

same seed/steps/prompt at 768x768


Add Qiskit Pulse Benchmarks

Summary
Partially fixes #794.
Schedule construction benchmarks:

with parameters being the:

[x] number of unique pulses
[x] number of channels
[x] ~number of uses of each pulse~

options:

pulse types:

[x] Parametric
[x] Sample

Building methods:

[x] Append instruction
[x] Insert instruction right to left
[x] Convert the instruction to a schedule and then take the union of all schedules.

Details and comments

I understand the the pulse api is in flux and currently being refactored. But what is concerning me here is that the main value of asv is showing performance over time as we make changes. If/when we need to update the benchmark code to adapt to deprecation removals in the future that will invalidate the old results, not only in spirit (by changing the measurement) but the benchmarks are all actually implicitly versioned with the hash of the code. So changing the code treats the benchmark as a new version that can't be compared to the old one. The question I have is while we probably should update the api here to be the non-deprecated version so we can measure consistent results moving forward, is this new api going to stay around for longer than 3 months? Rebuilding the historical data set every time there is an api change won't scale (back-filling old data takes a very long time)

The core content of the tests will remain relevant over time, I imagine this revision of the pulse API should stay current for > 3 months even as we move to the new builder interface which will sit on top of the IR these tests use.

Ok, sounds good then let's update the API usage in the benchmarks here to avoid the deprecation warnings and then this is good to go from my perspective.

Lint error
************* Module test.benchmarks.pulse.schedule_to_instruction_conversion
test/benchmarks/pulse/schedule_to_instruction_conversion.py:20:0: E0611: No name 'Play' in module 'qiskit.pulse' (no-name-in-module)
************* Module test.benchmarks.pulse.schedule_construction
test/benchmarks/pulse/schedule_construction.py:19:0: E0611: No name 'Play' in module 'qiskit.pulse' (no-name-in-module)

Will be fixed after Qiskit Terra release.


An option to allow a task to inherit stdin

I'd like to propose a way to allow a task to inherit stdin.
The motivation is that in a watch & nodemon setup the watch-type tasks don't need any input, but it would be nice to be able to pass the rs command to nodemon.
Task options
Apart from providing a task name, there would be an option to provide an array with the name and the options object (similar to grunt-browserify):
{
  tasks: ['watch', ['nodemon', { <options> }]]
}

This shouldn't break anything existing today.
The inheritStdin option
This should be a Boolean-type value (defaults to false).
If true, then process.stdin would be provided as the stdin for the child process instead of 'ignore'.

Not something I'm interested in. Maybe try asking for it on https://github.com/iammerrick/grunt-parallel.
Although, sounds like something that should be fixed in nodemon. Maybe do an issue/PR there about ability to pass the rs thing non-interactively through a CLI flag.

I understand that you are not interested in this, just want to clear up what rs does - this is a runtime command for restarting whatever nodemon is taking care of. From nodemon docs:

Whilst nodemon is running, if you need to manually restart your application, instead of stopping and restart nodemon, you can simply type rs with a carriage return, and nodemon will restart your process.

@sindresorhus Hi Sindre, would you mind clarifying why you wouldn't be interested in this kind of option?
I have the exact same use case as @fatfisz: nodemon+watch as tasks being run concurrently. Sometimes it is useful to manually trigger a nodemon restart by typing rs. When running nodemon by itself this works fine, when running under grunt-concurrent it does not receive stdin.
This cannot be solved by passing a flag to nodemon as it needs to be trigger-able at runtime.
The solution is passing 'inherit' as the fd 0 option.
If 'inherit' isn't the best choice as the default option, then it would be super awesome if this was configurable similar to logConcurrentOutput. I can get a PR up if you'd be willing to take a look.

@zpnk From my understanding (although fairly limited), would a better solution be to pass this option individually for each task, in case they have different interpretations for the same flag? E.g
tasks: [{ name: 'watch', inherit: false }, { name: 'nodemon', inherit:  true }]


Update README.md

@phillipgibson - please use the "Squash and Merge" option and follow the commit style guideline on future PRs.


Ineffective input method

On Linux
Failure of Chinese input method，Input method switching without response.Can you help me? Thanks.

OK！The problem has been solved. #146 Thanks @JordanMartinez very much.


dotnet new is not installing the latest version of this starter project referencing aspnet core 2.1

per your readme, I ran this command to install this template:
dotnet new -i aspnetcore-vuejs
when i created a new template from that, i noticed that the aspnet core version was still 2.0.3 and not 2.1.x. Did you deploy any of the latest changes so the dotnet new command pulls the newer version.
This is what dotnet new installed
Installing aspnetcore-vuejs 1.0.1.
thoughts?

@mika76 Hey Mladen, can you add me to the nuget package owner list so I can update it as well, looks like you're the only Owner of it at the moment.

I have permissions now let me try and update the package today!

Fantastic. Good luck!
On Tue, 21 Aug 2018 at 16:32, Mark Pieszak<EMAIL_ADDRESS>wrote:

I have permissions now let me try and update the package today!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarkPieszak/aspnetcore-Vue-starter/issues/102#issuecomment-414695075,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAN_v27GCWcPDedGdD79rpkKj2Gg8PAiks5uTBn5gaJpZM4V-OZ1
.

Looks like it's still pulling old version template when running dotnet command. Have you manage to update nugget package?

Sorry for the delay! Just published it now, it looks like it's validating so hopefully 1.0.2 will be up there soon with the latest & greatest from the Repo here. :+1:
https://www.nuget.org/packages/aspnetcore-vuejs/
@adyra @kirk-quinbar


multiple currencies and disabled

GetCandy version: 2.0-beta15

Expected Behaviour:
when disable currency, it should be disabled and won't be required when filling product info
Actual Behaviour:
validation fails when save product for disabled currency

Steps To Reproduce:

more than 1 currency
disable 1 currency
fill prices except disabled currency

@glennjacobs mentioned on discord this could be:

on intention to avoid missing prices when re-enable the currency
for controlling currencies on frontend

for 1. i think it's valid but also at the same time the same issue will happen for adding new currency to the store. for 2., maybe a clearer label needed

@wychoong I think this one can be simply solved by just updating getCurrenciesProperty to check only if enabled
Looks like the hasPriceValidationRules depends on this, so I believe thats the only place that needs changing 👍
I have way too many open PR's so will let you submit this one 😂
HasPrices.php line 205
/**
 * Returns the currencies computed property.
 *
 * @return \Illuminate\Database\Eloquent\Collection
 */
public function getCurrenciesProperty()
{
    return Currency::whereEnabled(true)->get();
}

@adam-code-labx yea, depends which behavior should be implemented

Well in terms of controlling currencies on the front-end. The livewire currencies property will not impact this.
So I see 2 different behaviour options for the hub based on one of more currencies disabled.

We exclude the disabled ones from the dropdown and do not allow to enter the price.
Include all currencies allow price to be filled but only validate enabled currency pricing.

I would suggest in the getcandy-hub/system.php config file an array for different behaviours then to set a default.
The developer can then choose the behaviour suited to their/client use case.
I plan to make similar suggestions for the addresses behaviour, however addresses will require to run migrations.
Which could simply check the behaviour config and only run based on the required behaviour.

2. Include all currencies allow price to be filled but only validate enabled currency pricing.

I think this would be great. However this doesn’t address the initial intention for current behavior is to avoid missing prices when re-enable a currency.
What I can think of now is to have a list to show products with missing prices, and only allow enabling currency when all products have the said currency’s price. And since new currency added is disabled by default, this could be good preventive measure for all scenarios can think of now
@glennjacobs what do you think?

Based on your solution, have raised a PR #1268
It fixed this issue along with another issue that I faced in #1267

Jumping in here as this topic is active again.
The original purpose for currencies being enabled or not was for storefront display. Perhaps a currency's exchange rate isn't so great and the store owner wants to stop offering it for the moment.
Regardless of whether a currency is enabled or not, we always want to capture the pricing information in the hub. Otherwise, if a currency is re-enabled we don't have the pricing data for the store.
So the issue comes, I think, when a store is initially installed and the developer wants a different currency, e.g. EUR, or when a store decides it no longer wants to offer a currency and never will going forward.
The first is fairly easy I guess, ensure all data that uses the currency is removed and then remove the currency.
The latter is more problematic, as we don't want to have to keep inputting prices, as we no longer want to support that currency, but we do have historic orders using it. Perhaps we should introduce soft deletes for this purpose?
We also have the same pricing data issue when a new currency is added. Perhaps #383 could assist with this to create initial prices for the new currency?
I also agree that the UI in the hub probably needs better descriptions to avoid confusion.
I would be interested in people's thoughts on this.

a potential solution is to only validate enabled currency. When enabling a currency, it will check if all products are set with the currency, else it will not be able to enable. and a list of products with missing price will be displayed.
this way it:

only validate enabled currency
disable currency without soft delete
can enable only if all products* with currency
new currency added is disabled by default / or option to auto convert with exchange rate

if this is the way, we can then review the UI

Right, so when creating a new currency it would have to be disabled by default (assuming we have products in the database). But the currency would still show in the hub when disabled (just not validated), so if we want to remove a currency completely we'd need a background task most likely.

closing this issue as moving to 1.0 and no longer facing the original issue.
further handling of enable/disable currency as discussed here should be reviewed and open a new issue for v1.0


[AREA]: Control vendor managed inventory and consignment inventory

Contact details
<EMAIL_ADDRESS>Organization type
Microsoft employee
End-to-end business process
Inventory to deliver
Specify the business process area name for the article.
Control vendor managed inventory and consignment inventory
Enter any additional comments or information you want us to know about this business process area.
Looking for help on the business process area.
Specify the date you expect the article to be completed and ready for review.
09/30/2023
Code of Conduct

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

Consolidated under area 'Maintain inventory levels'


worker: RollingUpdate, readinessProbe and volumeClaimTemplates

needs https://github.com/packit-service/packit-service/pull/107

ready for review


Several modifications to clean-up / enhance the sample:

Fixed an initialization issue with GVRCylinderScneObject.
Removed the code for rotating scene objects as it seems superfluous
to what this sample intends to demonstrate.
Attached the GVRCameraSceneObject to camera rig instead of main scene
so that it can move with user's head motion.
Enabled VRMode for the camera so that 60 fps camera preview can be used.

GearVRf-DCO-1.0-Signed-off-by: Gaurav Srivastava
<EMAIL_ADDRESS>

Probably in this pull request, you are using tabs-only spacing policy instead of spaces only in formatter. I think you need to change it in Eclipse->Preference.(Sorry I don't have Eclipse in hands so that I can't provide detailed steps for you but I think you can check the settings in gearvrf.org.)

Looks good to me. Haven't tried it though.

sample doesn't compile.  still has the mActivity argument in the constructor for GVRCameraSceneObject

@thomasflynn, not sure if you have the updated code. I don't see any Activity argument in the constructor and it compiles fine for me. The only redundant thing is 'import android.app.Activity' which I will remove in the next commit but it should not break compilation.


ImportError: cannot import name 'jsx'

Hi, I'm VERY new to python but have fallen in love w/ pandas.  As such I'm trying to use pyxley for some of my projects.  Quick question....when I run the datatables example, and set up bower, I get an error b/c the example doesn't have a bower file.  If I borrow bower.json from another folder, when I run the app.py, I get an import error, " cannot import name 'jsx'"
What am I doing wrong?

I forgot to update the readme in that folder. Follow the set up in one of the other examples.
Like metricsgraphics.

Thanks for the setup advice...another quick and very newb question....Having an issue w/ babel loader....am I not installing the right webpack and/or correct version of babel-loader?  I've read that this could be a config issue..
ERROR in multi main
Module not found: Error: Cannot resolve module 'babel-loader' in C:\Users\phill\pxy\examples\plotly
@ multi main


Add idle to cpus result object [Accuracy not verified]

Hi,
I happen to be using your module and needed the idle field for each CPU.
So I thought I'd better implement it first rather than asking for the feature.
I have made the necessary changes but I am not familiar with the formula on how to get the accurate calculation of the idle reading.
Please review the added lines when you have a minute.
Many thanks :-)

Please disregard this PR, please review #29 instead.
Thanks


Phone Donation: If "save/submit" request fails, we need to generate a new recaptcha code

While testing over a slow/unreliable connection, I frequently encountered failed save requests. This is not a problem with the application, but rather a problem with my internet connection, however...
After a failed request, the page still uses the same recaptcha code with the AJAX submit request. This is rejected by the server.
So, when a save request fails over AJAX, we still need to ask the the user to re-do the captcha before submission, with a new code, otherwise the request will also be rejected the next time they try to submit.
I hope that makes sense! Thanks

@topherwhite cannot reproduce such issue, but I've added recaptcha reset if error response returned. Check please.


Add radio buttons to id: What the defendant looks like and id: More about what the defendant looks like

Should be radio buttons instead of checkboxes for fields where you only want one selected

My colleagues say to allow multiple choices under Race. We can also add a text box to collect Other. Let's say max of 12 characters. If you create that, I'll add a field to the law enforcement PDF. -Kris

Should I remove "Select one." for race then? I will also add a text box

It's just contradictory to say "Select one" and then have checkboxes where user can select every choice. If one of them should try be "Select one" they should be radio buttons

Remove "Select one" for race, at text field to collect Other with max of 12 characters. When. complete, ask Kris to add field to law enforcement PDF


NuKeeper throws UriFormatException on execution.

🐛 Bug Report
Since latest release the Azure DevOps extension outputs following exception when running NuKeeper:
Unhandled Exception: System.UriFormatException: Invalid URI: The URI is empty.
   at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
   at System.Uri..ctor(String uriString)
   at NuKeeper.Gitea.GiteaSettingsReader.CanRead(Uri repositoryUri) in /home/vsts/work/r1/a/drop/NuKeeper.Gitea/GiteaSettingsReader.cs:line 36
   at NuKeeper.Commands.RepositoryCommand.PopulateSettings(SettingsContainer settings) in /home/vsts/work/r1/a/drop/NuKeeper/Commands/RepositoryCommand.cs:line 51
   at NuKeeper.Commands.CommandBase.OnExecute() in /home/vsts/work/r1/a/drop/NuKeeper/Commands/CommandBase.cs:line 88
   at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.InvokeAsync(MethodInfo method, Object instance, Object[] arguments)
   at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.OnExecute(ConventionContext context)
   at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.<>c__DisplayClass0_0.<<Apply>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at McMaster.Extensions.CommandLineUtils.CommandLineApplication.<>c__DisplayClass142_0.<OnExecute>b__0()
   at NuKeeper.Program.Main(String[] args) in /home/vsts/work/r1/a/drop/NuKeeper/Program.cs:line 33

Could have something to do with recently added support for Gitea, looking at the exception?
I posted this here since I suspect this will also fail from a manual NuKeeper run.
Expected behavior
Package being updated.
Reproduction steps
Run NuKeeper from Azure DevOps extension with following params:
-v detailed -c Major -i (MSTest|Microsoft.NET.Test.Sdk|Microsoft.TestPlatform.TestHost) -n
Configuration
Version: 0.20.2
Platform if applicable:

[x] :hammer_and_wrench: NuKeeper CLI
[ ] :sparkles: Github
[x] :robot: AzureDevops
[ ] :checkered_flag: Bitbucket
[ ] :earth_americas: Gitlab
[ ] :tv: Gitea
[ ] :whale: Docker

That's bad could you have a look @sharpsteff?

yes

Not sure if it's from your PR. Could also be from the github target branch PR.

at NuKeeper.Gitea.GiteaSettingsReader.CanRead(Uri repositoryUri) in /home/vsts/work/r1/a/drop/NuKeeper.Gitea/GiteaSettingsReader.cs:line 36
I catched only HttpRequestException in the CanRead.
Did not think about UriFormatExceptions

Mm this is also about the azuredevops extension, so it's probably in the something else then gitea

could reproduce the exception with a little unit test
```
///
/// Test for #739
///
[Test]
public void CanRead_NoException_OnBadUri()
{
Assert.DoesNotThrow(() => _giteaSettingsReader.CanRead(new Uri("")));
}

Good, can you make a PR for that?

yes on the way

i like the automatic link to my commit

Awesome, thanks for the quick help. Your pr and #744 should fix this issue.

yes think so, too.

Fix is released(https://github.com/NuKeeperDotNet/NuKeeper/releases) version 0.20.3 should not be available. Can you confirm that it work @MaxMommersteeg?

Sure, testing in a minute.

The execution failed again, but because of a different reason. Currently on mobile and not able to copy the exact exception message. Maybe tomorrow. This exact issue is fixed.

Seems to work fine with me including the new formating

Getting following output:
[command]C:\agents\agent1\_work\_tool\dncs\2.2.202\x64\dotnet.exe C:\agents\agent1\_work\_temp\nukeeper\tools\netcoreapp2.1\any\NuKeeper.dll repo C:\agents\agent1\_work\10\s ******** --targetBranch origin/master -v detailed -c Major -i (MSTest|Microsoft.NET.Test.Sdk|Microsoft.TestPlatform.TestHost) -n
Matched uri 'https://fudura.visualstudio.com/' to collaboration platform 'AzureDevOps'
FindPushFork. Fork Mode is SingleRepositoryOnly
2019-04-20T07:33:35Z: Started
Attempting delete of folder C:\agents\agent1\_work\_temp\NuKeeper\package
Deleted folder C:\agents\agent1\_work\_temp\NuKeeper\package
Attempting delete of folder C:\agents\agent1\_work\_temp\NuKeeper\tools

Unhandled Exception: System.UnauthorizedAccessException: Access to the path 'C:\agents\agent1\_work\_temp\NuKeeper\tools\netcoreapp2.1\any\LibGit2Sharp.dll' is denied.
   at System.IO.FileSystem.DeleteFile(String fullPath)
   at System.IO.File.Delete(String path)
   at NuKeeper.Inspection.Files.Folder.DeleteDirectoryInternal(String targetDir) in /home/vsts/work/r1/a/drop/NuKeeper.Inspection/Files/Folder.cs:line 68
   at NuKeeper.Inspection.Files.Folder.DeleteDirectoryInternal(String targetDir) in /home/vsts/work/r1/a/drop/NuKeeper.Inspection/Files/Folder.cs:line 74
   at NuKeeper.Inspection.Files.Folder.DeleteDirectoryInternal(String targetDir) in /home/vsts/work/r1/a/drop/NuKeeper.Inspection/Files/Folder.cs:line 74
   at NuKeeper.Inspection.Files.Folder.TryDelete() in /home/vsts/work/r1/a/drop/NuKeeper.Inspection/Files/Folder.cs:line 45
   at NuKeeper.Inspection.Files.FolderFactory.DeleteExistingTempDirs() in /home/vsts/work/r1/a/drop/NuKeeper.Inspection/Files/FolderFactory.cs:line 44
   at NuKeeper.Collaboration.CollaborationEngine.Run(SettingsContainer settings) in /home/vsts/work/r1/a/drop/NuKeeper/Collaboration/CollaborationEngine.cs:line 35
   at NuKeeper.Commands.CollaborationPlatformCommand.Run(SettingsContainer settings) in /home/vsts/work/r1/a/drop/NuKeeper/Commands/CollaborationPlatformCommand.cs:line 116
   at NuKeeper.Commands.CommandBase.OnExecute() in /home/vsts/work/r1/a/drop/NuKeeper/Commands/CommandBase.cs:line 96
   at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.InvokeAsync(MethodInfo method, Object instance, Object[] arguments)
   at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.OnExecute(ConventionContext context)
   at McMaster.Extensions.CommandLineUtils.Conventions.ExecuteMethodConvention.<>c__DisplayClass0_0.<<Apply>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at McMaster.Extensions.CommandLineUtils.CommandLineApplication.<>c__DisplayClass142_0.<OnExecute>b__0()
   at NuKeeper.Program.Main(String[] args) in /home/vsts/work/r1/a/drop/NuKeeper/Program.cs:line 33
##[debug]task result: Failed
##[error]Error: The process 'C:\agents\agent1\_work\_tool\dncs\2.2.202\x64\dotnet.exe' failed with exit code<PHONE_NUMBER>

Not sure why this fails, since I didn't make any changes to NuKeeper nor authentication or authorization areas.

Can you open a new one with that info? This might be confusing for one people.

Sure, will do tomorrow

Created new issue.


Feature/replicas

Describe your changes
Enable hot standby PostgreSQL replicas.
Notes:

This still needs UI support from the Console
I've not tuned any GUCs for PostgreSQL to make this work better
Safekeeper has gotten a tweak in which WAL is sent and how: It now sends zero-ed data from the start of the timeline's first segment up to the first byte of the timeline to be compatible with normal PostgreSQL WAL streaming.
This includes the commits of #3714

TODO:

[ ] check that we can replicate an instance that receives a pg_regress workload
[ ] handle Hot Standby StandbyReply messages correctly (c.q. https://github.com/neondatabase/neon/blob/main/safekeeper/src/send_wal.rs#L309-L315)
[ ] Code cleanup

Issue ticket number and link
https://github.com/neondatabase/neon/issues/769
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[x] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.
    Neon, and Neon's flavor of PostgreSQL, now support physical replicas.
XXXX: This doens't mean cloud users can use this immediately, see the first note as well.

Please add some regression tests, something under test_regress/regress/


Button style props.

Is there any api to modify that? ( i was looking for editing font size )
Thank you

Is there any api to modify that? ( i was looking for editing font size )
Thank you

Now this function is not supported. You can change it by modifying MenuButton.tsx

Thank you for your responds


Kafka is not coming up

Hi, I am using Docker Toolbox for windows and trying to bring Single Zookeeper and Single Broker up.
I used the following command from the source folder
'docker-compose -f zk-single-kafka-single.yml up
with which i can see the servers are up but when i go to source folder by opening another docker window and try to create a topic it says 'Kafka-Topics' is a bad command. Not sure why?
My Folder structure
Here are my configuration files i am

using which you have provided in the Udemy course. From this location only i used this command 'C:\Program Files\Docker Toolbox\kafka-stack-docker-compose-master'. Please let me know what i am doing wrong.

Just add the kafka-topics.bat (you're on windows) to your path and use that

Just add the kafka-topics.bat (you're on windows) to your path and use that

Thanks Stephane for the response
I tried doing that also. The server comes up with this command 'docker-compose -f zk-single-kafka-single.yml up'
Now the problem is with create and other kafka commands... I have basically done the following after starting the server

Tried to create topic using 'kafka-topics.bat --zookeeper <IP_ADDRESS>:2181 --create --topic test --partitions 3 --replication-factor 1' command

Also tried using 'kafka-topics.bat --zookeeper <IP_ADDRESS>:2181 --create --topic test --partitions 3 --replication-factor 1'

In both of these cases i get the same message command not found.  It looks strange to me


Suggestions for why parallelshell breaks node-notifier?

I've been able to get multiple people on multiple operating systems (OSX and Windows) to reproduce some unexpected behavior with the following package:
{
  "dependencies": {
    "node-notifier": "^4.5.0",
    "parallelshell": "^2.0.0"
  },
  "scripts": {
    "works": "npm run notify",
    "broken": "parallelshell \"npm run notify\"",
    "notify": "notify -t \"Some Title\" -m \"Some Message\" || echo 'failed'"
  }
}

For some reason when notify is called behind something run with parallelshell it will just hang. I'd love to try and figure out how I can get this unstuck but I'm not sure if it is a problem with parallelshell or node-notifier. :( Any ideas?
I've also opened an issue on parallelshel here: keithamus/parallelshell#52

Hi. Thanks for the detailed and thorough issue! It seems to be an issue with the node-notifier bin using process.stdin.on('readable', fn) to listen for when the stdin stream is ready to send information, but when using parallelshell it never becomes readable. This might be an issue in how parallelshell spawns the process and what streams it uses when spawning. I don't fully know the details of parallelshell.
Let's wait until we get a response in your other issue to see what the maintainer says about it, if that works for you?

@mikaelbr works for me. :) however, i see that parallelshell is now marked as not maintained. :( A friend of mine (@silentworks) has been helping me through this and I'm going to try using another (similar) library to parallelshell to see if that unsticks me. :)
I figured it had something to do with spawning/exec/child process something-or-another but I'm not familiar enough with process-level stuff (at list with node) to even start tracking this theory down. Thanks for looking into it! If I find a solution using another library I'll update this ticket. It might be worth adding to a FAQ or on the README of notifier in case other people run into this. If you have a preference on that let me know and I'll see if I can contribute a note to the docs.

Contribution is very much welcome. If you feel like helping out, you can add a section to the Common Issues.

@mikaelbr ok, great. fwiw, while working on this problem yesterday i was able to find a workaround by replacing parallelshell with npm-run-all. however, at that point i found that it was still broken in the case that i was using nodemon in another part of my build process (for watching). I've updated my little sample to show all of the cases that work, don't work, or work in ways that are not super nice (watch works but is super slow at detecting filesystem changes compared to nodemon).
{
  "dependencies": {
    "node-notifier": "4.3.1",
    "nodemon": "^1.9.1",
    "npm-run-all": "^1.6.0",
    "parallelshell": "^2.0.0",
    "watch": "^0.17.1"
  },
  "scripts": {
    "notify": "notify -t \"Some Title\" -m \"Some Message\" || echo 'failed'",
    "works-simple": "npm run notify",
    "works-npm-run-all": "npm-run-all --parallel notify",
    "works-watch--but-slow": "watch 'npm run notify' .",
    "broken-parallelshell": "parallelshell 'npm run notify'",
    "broken-nodemon": "nodemon -q -w . --exec 'npm run notify'"
  }
}

My guess is that nodemon is doing something similar to parallelshell in how it spawns processes while npm-run-all and watch are doing something that is more compatible with notify.
I had hoped that watch would work but the delay is suboptimal and a downgrade from the previous speed/performance we were seeing with nodemon. The whole notification stuff was supposed to aid things and not slow stuff down so I think I'll just have to ditch notify for awhile.
If you find a fix for this and end up closing this issue I'll know I can come back and hopefully get going more quickly than this pass. :) I've spent a lot of time on this so far (more time than I'd like to admit) and think I need to move on quickly. :)
I think it makes sense to leave this issue open as an unconfirmed bug until such time as you confirm it (one way or another) or fix it (yay!) so I won't close it myself. If you decide to close it for now that is fine with me. :)
I'll still try to get something into common issues over the next day or two if I get some free time.
Thanks for your help and looking into it!

I'll look into this some more. Maybe I'll find a better way to support pipe functionality that won't break this behaviour.

@mikaelbr Thanks for taking a crack at fixing this. I'm still not seeing it working, though. I updated my sample package.json to include 4.6.0 and the two broken scripts (broken-parallelshell and broken-nodemon) still fail even using 4.6.0.
{
  "dependencies": {
    "node-notifier": "4.6.0",
    "nodemon": "^1.9.1",
    "npm-run-all": "^1.6.0",
    "parallelshell": "^2.0.0",
    "watch": "^0.17.1"
  },
  "scripts": {
    "notify": "notify -t \"Some Title\" -m \"Some Message\" || echo 'failed'",
    "works-simple": "npm run notify",
    "works-npm-run-all": "npm-run-all --parallel notify",
    "works-watch--but-slow": "watch 'npm run notify' .",
    "broken-parallelshell": "parallelshell 'npm run notify'",
    "broken-nodemon": "nodemon -q -w . --exec 'npm run notify'"
  }
}

Hm. That's weird. I got all of those examples working when developing and testing out. But there are some issues here now. I might have messed it up. Will look more into this as soon as I can.

@mikaelbr No rush. I do appreciate you taking the time to look into it at all. :) I'll try to test it again as soon as I see you've made a change but it took me almost two weeks last time.

Closed in favour of https://github.com/mikaelbr/node-notifier-cli/issues/1 as CLI has moved to own project.

FWIW, I'm having a similar issue (my setup is a bit convoluted because it uses npm-watch which uses nodemon under the hood and I have a bunch of && and ||, but I think the workaround that worked for me was to add stdin:
{
  "scripts": {
    "fail": "echo | notify -t 'Warning' -m 'Build failed.'",
    ...
  }
  ...
}

Here is the entire scripts:
  "scripts": {
    "dev": "npm-watch",
    "run_dev": "npm run build && npm run chrome || npm run fail",
    "build": "tsc && vite build",
    "chrome": "open -a 'Google Chrome' http://reload.extensions",
    "fail": "echo | notify -t 'Warning' -m 'Build failed.' && false"
  },