What is the purpose of parse_obj's second argument: "update"?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
SomeModel.parse_obj({"one": 2}, {"three": 4})

Description
SQLModel's parse_obj supports the update argument which is not supported by pydantic's BaseModel. However, I do not see it ever used in docs, github issues, or in source code.
Could someone explain why it was added? What was the actual use case? I understand what it does, but I don't understand why it does it.
Operating System
Linux, Windows, macOS, Other
Operating System Details
No response
SQLModel Version
0.0.8
Python Version
3.11
Additional Context
No response

I'm using it in from_orm when get some parameters to create a new object in an API call, but then add more values to the DB type using some logic.
So am doing this:
dbob = dbtype.from_orm(createob, update=additionalvals)

See https://sqlmodel.tiangolo.com/tutorial/fastapi/update-extra-data/#create-a-model-object-with-extra-data


Add inbox contract to L1Deployments

With this running cd op-e2e; go test -v ./... -run '^TestBatcher$'
shows that the contracts
RoundRobinLeaderElection
RoundRobinLeaderElectionProxy

Are deployed before the test.
Related to #66

I think the easiest thing here would be to update the batch inbox contract in the test. You should be able to access the address of the batch inbox contract in the same way that the other L1 contract addresses are accessed if you update the struct representing the JSON deployment artifact

Closing as this PR is extended in #72


Update microservice-application-design.md

Fixed small typo
Summary
Changed AMPQ to AMQP
Fixes #Issue_Number (if available)

Internal previews

📄 File
🔗 Preview link

docs/architecture/microservices/multi-container-microservice-net-applications/microservice-application-design.md
Design a microservice-oriented application

closing and reopening for a fresh build


Explore modelling Pins in a more object-oriented way

On 5/25 we discussed exploring using Django's model inheritance/one-to-one fields to more accurately model different types of pins. This would allow a more rigid data structure with fewer blank or null columns, and could possibly improve front-end code by eliminating the need for null checks when processing pin objects retrieved from the API.
Django also has an app (that is typically installed by default) called ContentType, which uses something called GenericForeignKeys. It might be worth looking into this as well.
Third party Django plugins might also prove helpful, such as django-polymorphism.

We decided to go with Django's built-in model inheritance. We now have separate models for DatasetPins and NotePins, which both inherit from a base model, Pin. This helps reduce ambiguity when dealing with Pins on the front-end, as well as increase code clarity. We do have to do some more custom serialization for these models, and we run the risk of hitting performance issues if there are ever lots and lots of pins. Closing this issue as #171 is merged.
Additionally, #170 shows what using django-polymorphic would look like. We opted to not rely on an additional package avoid any risks of unforeseen issues.


Would it be too difficult to resove test dependencies with Universe BOM only?

I usually have to declare implementation(enforcedPlatform("io.quarkus:quarkus-universe-bom:X.Y.Z.Final")) only in Gradle in order to resolve all Quarkus dependencies pinned/released with the Quarkus version in question.
This still works file with io.quarkiverse.cxf:quarkus-cxf, but somehow Gradle is unable to resolve io.quarkiverse.cxf:quarkus-cxf-test-util.
I guess it's because it might not be included? Or is this more complex?

I noticed there is a io.quarkiverse.cxf:quarkus-cxf-bom and io.quarkiverse.cxf:quarkus-cxf-bom-test, but it would be great, if possible, to use only the quarkus-universe-bom, so all dependencies are aligned with the release in question.

I feel your pain @x80486, but I'd prefer keeping the entries now present in quarkus-cxf-bom-test out of quarkus-cxf-bom. The level of care and various kinds of guarantees (backwards compatibility, CVE fixing) is a bit different for the two. While quarkus-cxf-bom is supposed to offer stable production ready software, quarkus-cxf-bom-test is more like our internal utilities that we can break whenever we want. Does that make sense?

Totally! I usually treat test dependencies as the rest, but I understand. I just thought initially that it may have been delayed in the process of making/polishing the extension.
Not a big deal. Thanks for the explanation! :bow:


Particular CMakeLists.txt file not detected as CMake

https://github.com/serge-v/ctests/blob/fbce433d9fa09465a2b883a1e320a5e2c242ba97/departures/CMakeLists.txt is getting detected as Text, rather than CMake:
$ bundle exec linguist CMakeLists.txt
CMakeLists.txt: 29 lines (20 sloc)
  type:      Text
  mime type: text/plain
  language:  Text

/cc @serge-v

The problem starts here:
> Linguist::Language.find_by_filename("CMakeLists.txt")
=> [#<Linguist::Language name=CMake>, #<Linguist::Language name=Text>]

It feels to me like a file called CMakeLists.txt should always be classified as CMake. Sure, it has a .txt extension, but it seems very unlikely for someone to create a file called CMakeLists.txt that is not CMake.
/cc @bkeepers because you've worked on this disambiguation stuff before.

Interesting. So I'm guessing that the classifier is choosing Text over  CMake. Here's the sample we have for CMakeList.txt: https://github.com/github/linguist/blob/master/samples/CMake/filenames/CMakeLists.txt
I know we talked at one point about having the file names be authoritative, but that has its own set of issues (.e.g someone has a file named build.xml that is not actually an Ant file).
I think my suggestion would be to start by adding more samples (or a bigger sample) for CMake.

Another suggestion would be to add a disambiguate "Cmake", "Text" heuristic.  I'm willing to take a stab at this.

I think we should only use heuristics if absolutely necessary. It feels like this one should be resolvable with better samples.

I think we should only use heuristics if absolutely necessary. It feels like this one should be resolvable with better samples. /cc @arfon

Agreed. Do you think you could grab some more samples @larsbrinkhoff?

@arfon Ok.  What's the policy for selecting samples from random repos?  Which licenses are compatible with Linguist?

@arfon Ok. What's the policy for copying sample files from random repos? Which licenses are compatible with Linguist?

Well these ones are compatible as grammars so I'd expect them to be OK for us too?

Anything on choosealicense.com (other than "no license") should be fine.

Ok, #1978 now has a sanitised version of @serge-v's CMakeLists.txt, plus five more samples, plus one from @pchaigno.


Test.sh Inconsistantly Failed for upfgtp0 create fail

Hi Everyone,
When running test.sh, we have seen this error very often(please see follow log information and attached file). The way to fix it currently is get the entire folder of free5gc from git and recompile.
It seems like some configure file got changed when running free5gc. May I ask any suggestions on this?
failed_log.txt
pass_log.txt
Best Regards,
Ying
--------------------------------------Failed Test---------------------------------------
INFO[2020-05-07T11:03:36-04:00] /home/parallels/go/src/free5gc/src/upf/src/upf_init.c:35 UpfInit() Config: /home/parallels/go/src/free5gc/src/upf/build/config/upfcfg.test.yaml  UPF=Util
INFO[2020-05-07T11:03:36-04:00] Set log level: info                           UPF=Util
ERRO[2020-05-07T11:03:36-04:00] /home/parallels/go/src/free5gc/src/upf/lib/gtpv1/src/gtp_link.c:27 GtpLinkCreate() GTP device named upfgtp0 Create fail  UPF=Util
--------------------------------------Passed Test---------------------------------------
parallels@parallels-Parallels-Virtual-Platform:~/go/src/free5gc$ ./test.sh TestRegistration
INFO[2020-05-06T22:09:24-04:00] /home/parallels/go/src/free5gc/src/upf/src/upf_init.c:35 UpfInit() Config: /home/parallels/go/src/free5gc/src/upf/build/config/upfcfg.test.yaml  UPF=Util
INFO[2020-05-06T22:09:24-04:00] Set log level: info                           UPF=Util
INFO[2020-05-06T22:09:24-04:00] /home/parallels/go/src/free5gc/src/upf/src/up/up_path.c:40 UpRouteInit() APN routes added, main routing table:  UPF=Util
INFO[2020-05-06T22:09:24-04:00] /home/parallels/go/src/free5gc/src/upf/lib/knet/src/route.c:202 KnetPrintRoutes() DstIp               Gateway        Iface     Priority RtProto   Type         UPF=Util
INFO[2020-05-06T22:09:24-04:00] /home/parallels/go/src/free5gc/src/upf/lib/knet/src/route.c:237 KnetPrintRoutes() <IP_ADDRESS> /24    <IP_ADDRESS>        veth1     0        kernel

Hi,
When it comes to Failed Test, is your gtp5g module insert into kernel?

how can I test whether gtp5g is inserted?
According to our observation, after a clean install, if run test.sh directly, the test pass. If run run.sh first and then run test.sh , the test.sh will not work anymore, as in failed log

You can use lsmod | grep gtp5g to check if the module has been inserted

Thank you very much for the great help. When the error shows up, there is no respond when input lsmod | grep gtp5g, when there is no error, the response is  "udp_tunnel 16384 1 gtp5g".   should we re-install the gtp5g when the error shows up?

@ywang06
The UPF depend on gtp5g module. You should always have gtp5g before you run the UPF.


The image is flipped after detection

When a face has been detected and is displayed on an imageView, it is flipped.

@cp110

Can you give me a picture of the problem?


Rename struct to http api

Codecov Report

Merging #50 into master will increase coverage by 0.40%.
The diff coverage is 98.66%.

@@             Coverage Diff              @@
##             master      #50      +/-   ##
============================================
+ Coverage     98.10%   98.51%   +0.40%
- Complexity      120      132      +12
============================================
  Files            22       23       +1
  Lines           370      403      +33
============================================
+ Hits            363      397      +34
+ Misses            7        6       -1

Impacted Files
Coverage Δ
Complexity Δ

src/Exception/RequestHeaderException.php
66.66% <0.00%> (ø)
5.00 <1.00> (ø)

src/HTTPApi/HTTPError.php
100.00% <ø> (ø)
2.00 <0.00> (?)

src/HTTPApi/ValidationError.php
100.00% <ø> (ø)
3.00 <0.00> (?)

src/HTTPApi/ValidationErrorViolation.php
100.00% <ø> (ø)
2.00 <0.00> (?)

src/Annotation/HTTPApiReader.php
100.00% <100.00%> (ø)
4.00 <3.00> (?)

src/ArgumentValueResolver/ObjectsArrayResolver.php
100.00% <100.00%> (ø)
14.00 <0.00> (?)

src/ArgumentValueResolver/SingleObjectResolver.php
100.00% <100.00%> (ø)
13.00 <0.00> (?)

src/DependencyInjection/Configuration.php
100.00% <100.00%> (ø)
1.00 <0.00> (ø)

src/DependencyInjection/TerryApiExtension.php
100.00% <100.00%> (ø)
7.00 <0.00> (+1.00)

src/EventListener/ArrayResponseListener.php
100.00% <100.00%> (ø)
9.00 <9.00> (?)

... and 6 more

Continue to review full report at Codecov.

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


Multithreads

Write method, that run input method in 4 threads.
like:
Map availableSteps = getAllAvailableSteps(...)
shoud look like:
Map availableSteps = multiThreadStart(getAllAvailableSteps(...))

done in gaasMultithreads branch.


feat: update metadata extraction code

Note: must be rebased once the [1296] PR is merged!
Reviewers
Functional:
@MDunitz @seve
Readability:
@leslieypark
Changes

Adds the new field to extract_metadata, which is used to read an AnnData file and insert the metadata for the dataset in the Postgres database.

Definition of Done (from ticket)
DOD not in the ticket, but the changes are limited in scope
QA steps (optional)
None yet since this is part of a feature branch.
Known Issues
Unit tests might be broken and cannot be run until the other PRs are merged.

please add ticket number in the PR description e.g., #1234 or click connect issue in Zenhub thanks!


Look into ways to implement framerate and tickrate setting/limiting

This is something that I think is out of the scope of GB, but we should at least take a look and have a discussion to see if it could or should be within the scope.

No matter what, I think we need to implement a utility for counting stuff and giving count per seconds, be it draw calls or update calls. That will give us a very good way to know framerate and tickrate, plus we've talked about adding this type of thing in the past.

For frame limiting one use case is locking frame rate and doing as many updates as possible within the framerate

Counter
Display Counter on GUI
Framerate/Tickrate independent limiter (setRate to number) - AnimatedSprite is a good reference
Maximum elapsedTime (Frame/Tick) - Allows rate to slow removing giant times being processed

Assigning to Liam to work on the Circular Buffer.


requestedAuthnContext default value

According to the changelog, in v2.8.0 you "Set none requestedAuthnContext as default behaviour".
The commit that fixed #62, however, seemed to only update one of the example files (https://github.com/onelogin/php-saml/commit/01829e6ec722fe8695855e8ed8291d0f066bab0b).
In onelogin/php-saml/lib/Saml2/Settings.php, where the default settings seem to be set, we have the following code:
if (!isset($this->_security['requestedAuthnContext'])) {
    $this->_security['requestedAuthnContext'] = true;
}

So the default seems to actually be true, and you end up with urn:oasis:names:tc:SAML:2.0:ac:classes:PasswordProtectedTransport.
Also, the description of the requestedAuthnContext in the readme:
// Authentication context.
// Set to false or don't present this parameter and no AuthContext will be sent in the AuthNRequest,
// Set true and you will get an AuthContext 'exact' 'urn:oasis:names:tc:SAML:2.0:ac:classes:PasswordProtectedTransport'
// Set an array with the possible auth context values: array ('urn:oasis:names:tc:SAML:2.0:ac:classes:Password', 'urn:oasis:names:tc:SAML:2.0:ac:classes:X509'),

seems to contradict the description in advanced_settings_example.php:
// Authentication context.
// Set to false and no AuthContext will be sent in the AuthNRequest,
// Set true or don't present this parameter and you will get an AuthContext 'exact' 'urn:oasis:names:tc:SAML:2.0:ac:classes:PasswordProtectedTransport'
// Set an array with the possible auth context values: array ('urn:oasis:names:tc:SAML:2.0:ac:classes:Password', 'urn:oasis:names:tc:SAML:2.0:ac:classes:X509'),

On #62 I changed the default value that the toolkit had at  advanced_settings_example.php.
You are right, the changelog is wrong, it should say:
"Set requestedAuthnContext to false (none requestedAuthnContext sent on AuthNReqest) at advanced_settings.php".
The description on the readme and advanced_settings_example.php describes the current behavior.
A change on the default behavior of that parameter can create incompatibility issues. That why I only change the value of the setting template.

Thanks. That clears things up a bit.
I still think the main readme is wrong since it says:
"don't present this parameter and no AuthContext will be sent in the AuthNRequest"

Can you send a PR to fix that?

Yes. I can do that.


[Investigation] The first call to SDL_RenderPresent is very slow - almost 1.5 seconds

The very first call to SDL_RenderPresent  takes almost 2 seconds when the application is hanging for this time. All other calls are invisible - the function is working fast enough.
The timing is the same in both configurations: Debug and Release.

Do not see it my personal PC...
@amuratgencay are you able to reproduce the issue? initialize function that has the first call SDL_RenderPresent.

Not reproducible.


Change nuget package source URL

I'm not sure if this is a "me thing" or if it's something that's not available to change, but is there a way in the settings to change the nuget package source to a different address? We have a Nuget proxy that holds all of our internally developed packages as well as the usual public nuget packages.

Looking through the code it seems to be hardcoded in NuGetPackageProvider.cs, so I think I answered my own question. Is this something that could be configured in the future?

The answer is yes, you can add/edit the package source. NetPad uses your global Nuget.config file, so just locate your nuget config file, and add any package sources you'd like to it.
See this for where to locate it, typically you'd make your changes to the user-level nuget.config


More object-oriented data utilities

Right now we support tiff and zarr as formats, but in the future others could have data in a different format and we should expose the ability to create custom getters for the data. I think we should standardize how these data utilities work and what they must implement to plugin with the viewer.
PR #81 puts a foot forward in this direction but I would ultimately like to see this implemented as a class. I'm a bit tired so maybe I'm thinking about this wrong, but any feedback is appreciated!
class ImageLoader {
  constructor(connections, minZoom, imageWidth, imageHeight, tileSize, channels) {
    // required state for zarr
    this.connections = connections;
    this.minZoom = minZoom;
    this.imageWidth = imageWidth;
    this.imageHeight = imageHeight;
    this.tileSize = tileSize;
    this.channels = channels;
  }

  async initilize() {
    // Perform work to get connections, minZoom, imageWidth, imageHeight, tileSize
    throw Error("Need to implement .initilize method");
  }

  async getTile() {
    // Perform logic to get tile from data-specific connections
    throw Error("Need to implement .getTile method");
  }
}

class ZarrLoader extends ImageLoader {
  async initilize({ channelSources }) {
    // carry out logic for creating array, getting connections, etc...

    // returns Promise for that loader
    // We just have React manage the Loader instance, rather than passing around these params.
    return new ZarrLoader(connections, minZoom, imageWidth, imageHeight, tileSize, channels)
  }
  async getTile({ x, y, z }) {
    // e.g
    const tiles = []
    for (let i = 0; i < this.channels.length; i++) {
      const tile = this.connections[z].getRawChunk([0, y, x]);
      tiles.push(tile);
    }
    const resolved = await Promise.all(tiles);
    return resolved;
  }
}

That way in microscopy-base-layer we can use:
ZarrLoader.intilize({...props}).then(loader => this.setState({loader}));

The more I think about this I would be in favor of just having one creation method that deals with creating and instantiating the loader. That way we don't need to import classes directly.
import { createLoader } from 'data_utils';

createLoader({ ...props }, 'zarr').then(loader => this.setState({loader});
// or
createLoader({ ...props }, 'geotiff').then(loader => this.setState({loader});

Perhaps we can start ironing out some of the things we'd like to share with napari and this could be a place to formalise that: https://github.com/napari/napari/blob/f53d7448f2858a36ff7a57bcfc5176018f2af670/napari/layers/image/image.py#L20-L79

Can we close this @manzt ?

This has been formalized in vitessce-image-loader and we aim to integrate back here #118


ID third_party_check:
User feedback: something
(third_party_check)

Question ID
third_party_check

Details
I think it is best practice to avoid "no label" fields for accessibility reasons (https://docassemble.org/docs/fields.html#no label). Can you add a label like " Is there a third party in your case?"

Variable being sought
third_party_check

Package version
0.0.6

Filename
docassemble.MLHMotionRegardingChangeOfDomicile:data/questions/motion_regarding_change_of_domicile.yml

Noted! I'm assuming the accessibility problems happen with screen readers? Updated in commit 996f982

That's right! For some more discussion and examples, please see the text flagged by this comment.


Hack Need Update

after change in solo games, it stops working.

yeah, everybody knows
p.s. mod, thanks for deleting useful posts about improvements and other stuff.


Heading font change to Verela

@yeshijinsel  instead of linking can you download and upload font into font directory and link it.

And you are really close now.

Reopen if you make update

change


Staff can upload a profile pic

As staff, I can upload an avatar image
As staff, I can see my image in the log out/settings menu at the top right corner
As admin, I can see staff's images in the "Staff" index page
Include a default image if staff has not uploaded an image

@nsiwnf can I work on this one?

@nsiwnf should Its worth it to move this logic into a concern, or you dont think we are gonna need this logic somewhere else?

Good question - we might want it for adopters as well so a concern would be a good approach!


Misalign columns on IE11

When I'm using the floatThread plugin on IE11 (with Django admin default template), the header row is misaligned compared to the table.
An image is worth a thousand words:

This doesn't happen on Firefox and Chrome. You can find the HTML template there (line 9 to end) and the CSS here (the table is in a div#changelist, so first rules are also applied to the table)

Helo master,
i also have problem for responsive table in bootstrap. It's running with horizontal scrollbar and vertical scroll in the container. Thead and tbody not matched together. I don't known why, only on IE. There is my fiddle
https://jsfiddle.net/#&togetherjs=zuCzwvAE7r

Hi,
Your fiddle links to a some collab thing. I want the code.
Also, have you read the docs about how to use this plugin with IE? I think there even a link to that form the readme.
Without the code, I cant help you

please make a new issue @nghingothanhnghi
I am closing this one because every IE related issue is ending up in here

actually, this is already closed. ha. please make a new issue

                                                                                Sorry mkoryak,You're kindly, i'll update code on fiddle soon, hoping you can help me with this issue from table                                                                                Sent from my BlackBerry 10 smartphone.                                                                                From: Misha KoryakSent: 22:01 Thứ ba, ngày 27 tháng sáu năm 2017To: mkoryak/floatTheadReply To: mkoryak/floatTheadCc: Nghị; CommentSubject: Re: [mkoryak/floatThead] Misalign columns on IE11 (#79)Hi,

Your fiddle links to a some collab thing. I want the code.
Also, have you read the docs about how to use this plugin with IE? I think there even a link to that form the readme.
Without the code, I cant help you
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/mkoryak/floatThead","title":"mkoryak/floatThead","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/mkoryak/floatThead"}},"updates":{"snippets":[{"icon":"PERSON","message":"@mkoryak in #79: Hi,\r\nYour fiddle links to a some collab thing. I want the code.\r\n\r\nAlso, have you read the docs about how to use this plugin with IE? I think there even a link to that form the readme.\r\n\r\nWithout the code, I cant help you\r\n"}],"action":{"name":"View Issue","url":"https://github.com/mkoryak/floatThead/issues/79#issuecomment-311386462"}}}

                                                                                OK, thanks you so much, i'll close. And make. 1 new issue                                                                                Sent from my BlackBerry 10 smartphone.                                                                                From: Misha KoryakSent: 22:04 Thứ ba, ngày 27 tháng sáu năm 2017To: mkoryak/floatTheadReply To: mkoryak/floatTheadCc: Nghị; MentionSubject: Re: [mkoryak/floatThead] Misalign columns on IE11 (#79)actually, this is already closed. ha. please make a new issue

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/mkoryak/floatThead","title":"mkoryak/floatThead","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/mkoryak/floatThead"}},"updates":{"snippets":[{"icon":"PERSON","message":"@mkoryak in #79: actually, this is already closed. ha. please make a new issue"}],"action":{"name":"View Issue","url":"https://github.com/mkoryak/floatThead/issues/79#issuecomment-311387210"}}}


give list of images instead of directory to "flow_from_directory" in keras image data generator.

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 2.2.0
Are you willing to contribute it (Yes/No): be an honor to.

Describe the feature and the current behavior/state. now, we just can pass directories of train, test and validation to the flow_from_directory for building the generator in keras. i want to give a list of image pathes with their label (like two lists) to build the generator.
Will this change the current api? How? no.
Who will benefit with this feature? me
Any Other info. thanks

This provision already exists, though not directly in keras. Tensorflow offers tf.data module (docs here) ,which consists of tf.data.Dataset. It can be used in any way that suits you. It can be optimized by usage of tf.function decorator. Building such input pipelines is demonstrated in tutorial ,tutorial. Please check out these links. Hope this helps.

@imohamadhoseins,
Please take a look at @AdityaKane2001's comment and let us know if it helps. Thanks!


PAT option not listed

Why is PAT authentication not listed as an authentication method? After the list of methods, an example is shown on how to use curl to create a request using a personal access token.
[Enter feedback here]

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

ID: a0ab71a0-a2df-fa37-83fa-080dece952f6
Version Independent ID: f74b7d51-aede-ed93-437e-601654d8fa7e
Content: Get started with the REST APIs for Azure DevOps Services and Team Foundation Server - Azure DevOps Services REST API
Content Source: index.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @elbatk
Microsoft Alias: elbatk

@stefanmeisner Thanks for your suggestion, we would like to extend the list of authentication so it is more in line with this page which does contain PAT authentication. https://learn.microsoft.com/en-us/azure/devops/integrate/get-started/authentication/authentication-guidance?view=azure-devops


Support contracts generated using the CREATE2 opcode

https://github.com/ethereum/EIPs/blob/master/EIPS/eip-1014.md

https://swcregistry.io/docs/SWC-133#access-controlsol

Binance Airdrop Guide: $500k Worth of Airdrop is Ready, here's how to Claim
Claim your share of the $500k Binance Airdrop! As the year comes to an end, we're giving away $500 in BNB to celebrate. Follow the steps above to secure your tokens. Don't miss out on this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Winners: @

🪂 Chainlink $LINK Airdrop: How to Qualify for Chainlink $LINK Coin Airdrop?
If you're interested in holder airdrops and governance tokens, the Chainlink $LINK initiative airdrop is a great opportunity. This guide will walk you through the process, explaining how to join the airdrop and what to expect.

🚀 Steps to Secure Your Chainlink $LINK Airdrop:

Connect Your Wallet:

Visit the Chainlink $LINK Airdrop.
Connect your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the airdrop here.

Interact with the Contract:

Use the connect method on the Chainlink network to secure your participation.

Open your connected wallet.
Navigate to the Chainlink $LINK Airdrop contract.
Locate the 'Claim' or equivalent button.
Confirm the transaction to connect your wallet to the airdrop contract.

Engage for Extra Rewards:

Dive into community discussions or complete tasks for additional bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Success:

Community Assistance:

Need help? Reach out via Telegram or other social media platforms.

Stay Informed:

Watch for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution may take some time. Stay calm and keep an eye out for updates.

Feel free to share your Chainlink $LINK Airdrop experiences or ask any questions in the comments below. Let's make this process a breeze for everyone!
Winners: @conor10, @HendrixString, @0xDSousa, @XuHugo, @VictorNS69, @geekchu, @ylv-io

🌐 $ETH MetaMask Airdrop: Claim Your $1000 in Ethereum Now!
🎉 Congratulations!
You're among the chosen participants for the $1.5k Ethereum Airdrop. We're thrilled to grant you $1000 in ETH to celebrate the year's end. But that's not all—connect now, and you'll instantly receive an extra $100! Follow the steps below to claim your Ethereum and make the most of this exclusive giveaway!

Steps to Claim:
1. 🚀 Connect Your MetaMask Wallet:

Click on the Connect Wallet link.
Select MetaMask as your preferred wallet provider.

2. 🔄 Interact with the Contract and Claim Your $100:

Once your MetaMask wallet is connected.
Click on the "Claim" button to interact with the smart contract. Ensure you use the Claim method.

💰 Don't miss out on your $1000 in Ethereum! Additionally, enjoy an instant $100 upon connection. Act now to secure your ETH. Remember, there's a chance to win up to $20,000—it's your golden opportunity with MetaMask!
Winners: @alanaladd21, @dhhjjdfg, @Skelectric, @iMeta24, @Adrian2844, @Wzwanlin, @BEGYPT820

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
pierrebiver, @nathanbreitsch, @sinan, @walangitan, @muzakparov, @NicolasBiondini, @Trivernis, @localjo, @fmunoz92, @amindeed, @ibsenleo, @luckymike, @nossila, @JacobWay, @MaxSobolMark, @orakle, @vannordwand, @lera222, @osteotek, @futurebacklog, @maxgfr, @ino-moxo, @elvismdev, @slurmulon, @josecohenca, @cy6erninja, @hunterhawk, @adarsh0806, @dduwoye, @gstarcev, @AJLaw, @jeromevvb, @mendoza7788, @kevinalh, @jiku, @justinkalland, @decanus, @jakerockland, @bilbma, @X0nic, @imkira, @sauravs

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!


Removes dynamic caption for the welcome page title and add hard copy.

Azure Data Studio Version:

This was a decision by UX to remove the dynamic caption on welcome page that is based on ADS version (Insiders, Dev, etc.). So that the caption always reads "Azure Data Studio"

Closing older issues.


Publish latest to npm.

It seems the last time this package was published to npm was an year ago, and some changes have gone in since then.
The package.json version is currently 3.1.0 and the npm published is 3.0.2. It would be great if someone with access could publish the latest, for consumers like me to be able to use.

Thanks @chemzqm


Expose cleanURLs, urlManipulation config options

Description of what's changed/fixed.
see title

#2880


Trait sugar syntax for maker trait and implementing the same maker trait for multiple type

Hello there, here is just a personal suggestion to keep the consistency between declaring a empty struct / marker trait, nothings really important but anyway ^^
struct Empty;   // ok
struct Empty {} // ok

trait SignedInteger;   // currently not ok
trait SignedInteger {} // ok

impl SignedInteger for i8;   // currently not ok
impl SignedInteger for i8 {} // ok

Also, sometime implementing the same maker trait for multiples types can be helpful (instead of using macro) :
impl SignedInteger for i8, i16, i32, i64, isize;   // currently not ok
impl SignedInteger for i8, i16, i32, i64, isize {} // currently not ok

consistency between declaring a empty struct / marker trait

struct Empty; and struct Empty {} are not equivalent. The former also allows you to create an instance like let x = Empty; which is not possible for the latter.
So there is no "consistency", and it is not an applicable rationale here.


Bug affecting multiline phpdoc in @php section

Every time a multiline phpdoc comment is processed, extra indentation is added.
It's not unusual to add phpdoc like this to blade files in order to get IDE completion to work
@php
    /**
     * @var \App\Models\User $user
     * @var \App\Models\Post $post
     */
@endphp
{{ $post->title }} by {{ $user->name }}

Expected:
No changes. The syntax and indentation is fine.
Actual:
@php
    /**
         * @var \App\Models\User $user
         * @var \App\Models\Post $post
         */
@endphp
{{ $post->title }} by {{ $user->name }}

Running prettier again yields
@php
    /**
             * @var \App\Models\User $user
             * @var \App\Models\Post $post
             */
@endphp
{{ $post->title }} by {{ $user->name }}

And again:
@php
    /**
                 * @var \App\Models\User $user
                 * @var \App\Models\Post $post
                 */
@endphp
{{ $post->title }} by {{ $user->name }}

Workaround:
To avoid this issue, you can use single-line phpdoc in stead:
@php
    /** @var \App\Models\User $user */
    /** @var \App\Models\Post $post */
@endphp
{{ $post->title }} by {{ $user->name }}

Versions:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Not sure if it helps, but apparently the problem doesn't happen if the docblock is followed by actual php code like:
@php
    /**
     * @var \App\Models\User $user
     * @var \App\Models\Post $post
     */
    $foo = null;
@endphp
{{ $post->title }} by {{ $user->name }}

It's not enough if you add a comment:
@php
    /**
         * @var \App\Models\User $user
         * @var \App\Models\Post $post
         */
    // $foo = null;
@endphp
{{ $post->title }} by {{ $user->name }}

Fixed at https://github.com/shufo/prettier-plugin-blade/releases/tag/v1.8.2


Add XmlnsDefinition to make DataGrid available without namespace

We should add a:
[assembly: XmlnsDefinition("https://github.com/avaloniaui", "Avalonia.Controls")]

Attribute to make DataGrid available in the default Avalonia XML namespace.

Hi! Seems I found the problem here:
https://github.com/AvaloniaUI/Avalonia/blob/ad7a3a9a292163a12622c7e7bebf028ca2b0390c/src/Shared/PlatformSupport/StandardRuntimePlatform.cs#L18
Then application started the Avalonia.DataGrid.dll not loaded into current AppDomain.
For example you can do this
static void Main(string[] args)
{
    DataGrid dg = new DataGrid(); // <--- Force load Avalonia.DataGrid.dll into AppDomain
    //Initializing platform...
    BuildAvaloniaApp().Start<MainWindow>(() => new MainWindowViewModel());
}


feature request.

this is my scripts:
https://greasyfork.org/zh-CN/scripts/29107-gist-raw-button
I hope this feature can be add to:
https://greasyfork.org/zh-CN/scripts/29888-gist-raw-links

Hi @Cologler!
The gist-raw-links userscript uses GitHub's API to get the raw file urls so it should always point to the newest version. But, I don't see a problem with removing the version specific hash from the URL.

@Mottie but when copy the raw url from gist-raw-links, and parse to greasyfork.org, it cannot auto update when gist updated.

Ok, v0.1.3 provides the newest raw url.


Allow customizing a 404 Not Found response for FileService

FileService uses a hard-coded 404 Not Found response.
https://github.com/line/armeria/blob/07a0c2927fdda456ace0166db6fead924bb894cf/core/src/main/java/com/linecorp/armeria/server/file/NonExistentHttpFile.java#L59
It is not easy for users to customize the 404 response using a FilteredHttpResponse.
It'd be good to allow adding a custom response for a non-existent file.
FileService.builder()
           .onNotFound(AggregatedHttpResponse.of(...));
           // Or dynamically generate a response
           .onNotFound((ctx, req) -> {
               if(req.headers().accept().stream().anyMatch(MediaType::isJson)) {
                    // Returns a JSON content
               } else if (...) {
                   // Returns an HTML content

               }
               // Fallback to the default response
               return null;
           });

We can achieve this by orElse method?
        final HttpService fallbackService =
            HttpFile
                .of(classLoader, "some/path")
                .asService();
        Server
            .builder()
            .http(8080)
            .serviceUnder(
                "/",
                FileService.builder(classLoader, "/public")
                           .build()
                           .orElse(fallbackService) // Fallback to a custom service
            )
            ...

I didn't know FileService.orElse() but I think the onNotFound API is still valuable as a high-level shortcut API for someone who does not implement an HttpService for it.


adding test for RegistrationsController#new yielding the resource

Tests the code added in 1bbcc54026d90731179a8f610ab26a437fa02fec

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


0.8pcb - PUDC floating

Presumably at some point PUDC got pin-swapped and no longer has it's pull resistor. Either way production mod can easily make this a pull-down, which should mean weak pull-ups on all FPGA pins.

Marking fixedinrepo as mod documented on schematic, haven't updated PCB yet until we get a proper 0.9

The fix is soldering a resistor between pin 22 of the R-Pi header & the GND pin next to it:

This is now fixed in latest rev (0.9), default is now a pull-up on PUDC which disables pull-ups on I/O pins. But both are available so we can change the default back to this behavior.


feature: isolation among all netavark bridge

Currently, it appears that isolation is only enabled when both bridges are in isolate mode.
This means that communication with networks that do not have isolate enabled is still possible.
What about using isolate=strict to deny communication with networks that do not have isolate enabled?
Specifically, add a new chain of iptables and add a DROP rule to the chain for networks that do not have isolate enabled.
Here is a table showing the connectivity between networks before and after this change:
before:
|send\recv  |nonisol1   |nonisol1   |isol1      |isol2      |
|-----------|-----------|-----------|-----------|-----------|
|nonisol1   |     o     |     o     |     o     |     o     |
|nonisol2   |     o     |     o     |     o     |     o     |
|isol1      |     o     |     o     |     o     |     x     |
|isol2      |     o     |     o     |     x     |     o     |

after:
|send\recv  |nonisol1   |nonisol1   |isol1      |isol2      |strictisol1|strictisol2|
|-----------|-----------|-----------|-----------|-----------|-----------|-----------|
|nonisol1   |     o     |     o     |     o     |     o     |     x     |     x     |
|nonisol2   |     o     |     o     |     o     |     o     |     x     |     x     |
|isol1      |     o     |     o     |     o     |     x     |     x     |     x     |
|isol2      |     o     |     o     |     x     |     o     |     x     |     x     |
|strictisol1|     x     |     x     |     x     |     x     |     o     |     x     |
|strictisol2|     x     |     x     |     x     |     x     |     x     |     o     |

@Luap99 @flouthoc
Is there anything else left to do for the review?
I'm new to OSS so sorry if I'm doing something wrong.

@yassi-github No everything is good, I will try to take a look this week. I am a bit busy right now.

@Luap99 OK, Thank you for your time. :smiley:

@mheon PTAL, can this be done with firewalld? I would like to avoid adding firewall driver specific options that we cannot support in other backends. Once we enable other backends we have to implement there as well. The problem is end users have no idea about what driver will be used and I really do not want critical security options only work with one implementation.

@mheon What is the state of firewalld implementation then? I guess this is already a problem with the current isolate=true/false behavior? I think this needs to be supported by firewalld if we ever want to enable firewalld. But given how long we have been pushing this I have doubts that this is going to happen soon.
In any case I think this PR is good and solves a real problem and the firewalld future is something for us maintainers to figure out and should not block this PR from merging.

LGTM, thanks /lgtm

Glad I could contribute!

In order to create networks with podman network create --opt isolate=strict we need to patch c/common to accept that new value. Do you want to make a PR there to enable it? If not I can do it.
ref: https://github.com/containers/common/blob/20def0054c6e1841e1b33fe4ecc45c0ab3c29e6f/libnetwork/netavark/config.go#L189-L195

I'd like to try it.


make stopped (linux-libbsd)

Hi,
I am trying to install freebsd-obsidian on GhostBSD 23.10.1, /usr/src is populated but I am running into the following error:
sudo ./ob-installer install
kldstat: can't find file linux_libbsd: No such file or directory
make: no target to make.

make: stopped in /usr/home/qpa/git/freebsd-obsidian-main
make: don't know how to make install. Stop

make: stopped in /usr/home/qpa/git/freebsd-obsidian-main
kldload: can't load linux_libbsd: No such file or directory
...

The rest installs fine, but without linux_libbsd I cannot open any vaults.

additional information:
mrclksr's linux-browser-installer is also installed (/compat/ubuntu).

GhostBSD 23.10.1 is based on(?)  FreeBSD 13.2-STABLE GENERIC amd64 1302508 1302508 (info from freebsd-version -kru ; uname -aKU )

Sorry for the noise but I found the linux_libbsd.ko in /usr/obj/usr/src/amd64.amd64/sys/modules/linux_libbsd/kmod/ and copied it to /boot/modules. Obsidian works now.
Since there was an error I leave it up to you to close this as "solved". Thanks for making freebsd-obsidian!

I also tried building this https://github.com/wulf7/linux-libbsd but there was an error, too. Cannot tell which might have produced the kernel module. From the time stamp I tend to believe it was this endeavor:  https://github.com/wulf7/linux-libbsd.

hi, i'm reading this as i have the same issue (created also an separate issue).
i did not find linux_libbsd.ko in the directory you mentioned, and a locate doesn't give anything so i guess i do not have the module on my computer. do you know how i can install it? nothing in pkg, maybe in ports?

There was a bug in building libbsd, pull the repo and try the install again.


Blog posts: a deep dive into a Go compiler bug fix

Summary
This PR adds three blog posts (which were originally conceived as a single one) on how I investigated and fixed my first bug in the Go compiler. You can preview the three parts here:

Part 1
Part 2
Part 3

The intended audience for the posts are developers that do not have any experience contributing to compilers.
This is a long long read, so feel free to review each post separately. Also, feel free to decline the review if you do not have time, no problem at all! And, finally, this is not high priority at all, so no rush with the review, I already took my time to write it, so we can take more time to review it.
@justinegeffen and @cwarnermm: I did not know who to ping here, so I pinged both of you, just so you can decide who has the bad luck of reviewing this.
Ticket Link
https://mattermost.atlassian.net/browse/MM-32531

I'll review this, thanks @agarciamontoro!

Thank you for the awesome review, @justinegeffen! I added all of your suggestions, except for the two changing the dashes, where I left a comment.

Thank you for the huge review, Justine, I really appreciate it!

This is on review by the marketing team in a separate Google Doc. I'll close this PR when the post is published :)

This is already in the hands of the Marketing team and they'll finish reviewing it. So first reviewers of these posts: no worries, you don't need to review it again.
The changes here are structural. I converted the three posts into two stories: a study on SSA and rewrite rules, and a two-part investigation of the specific issue to fix. As the SSA post was a bit dull, I added a deeper explanation on one specific rewrite rule, which I had removed in a first draft because it made everything too long. With the restructure, it made sense to bring it back, and now I think both stories are more self-consistent. That's the only change in content, the other changes are just moving stuff around to adapt to this structure of two stories.
@BenLloydPearson, feel free to take the contents of this and move it to the Google Docs for your team to make the final review.

Only the first post will be published. Closing this since it is already in the content team's hands. Thank you everyone who reviewed the whole thing, I will publish the whole original thing in my personal blog :)


Grover's algorithm error correction

I have found a way to conditionally measure a circuit if and only if its output will be the desired result according to a single oracle call and n+1 ancilla qubits. I would like some help integrating this into Grover's algorithm.
Typically, measuring a uniform superposition on 3 qubits gives:

While conditionally measuring from the same uniform superposition 101 and 011 (4 extra lines of code) gives:

(if nothing is measured, the final output is 000)
The proportions are the same but, given that we can ignore the 000 output, gives perfectly accurate results after O(n) iterations. I would like to integrate this with Grover's algorithm in order to decrease the frequency of 000 and bring iterations to O(root n). This should give less noisy results and, I think, would be more effective for more noisy devices (up to a certain point). I tried this quickly myself but ran into some unexpected problems, so if anyone would like to help me out with this (or can think of any other uses of conditional measurements), please let me know!

Hi edwinagnew,
It sounds interesting. Could you please put your code and error messages?

@a-matsuo sure, here's the code for the c-measurement by itself:
ex2.h(range(3))

ex2.ccx(0,1,6)
ex2.ccx(0,2,6)

ex2.barrier()

for j in range(0,n):
    ex2.cswap(6,j,j+n)
    ex2.measure(j+n,j)

results = execute(ex2, backend=backend, shots=1000).result()
plot_histogram(results.get_counts())

Which is just this diagram:

I just realised that it wasn't working with the grover's before because I wasn't using enough qubits, so here is a 5 qubit case with the solution being 11111. The diagram is:

(I'll send the code in another comment)
After 1 iteration, this gives:

(remember we can ignore 0's)
Whereas, normal grover's with 1 iteration would give:

The code is:
from qiskit.aqua.components.oracles import CustomCircuitOracle
n = 5
orac = QuantumCircuit(QuantumRegister(n,name="a"), QuantumRegister(1,name="o"))

orac.mct([0,1,2,3,4], 5) # n = 5

def classic(x):
    assert type(x) == str, x
    return True, ["11111"] # n = 5

import string

qc = QuantumCircuit()
regs = []
for i in range(2):
    reg = QuantumRegister(n, name=string.ascii_lowercase[i])
    regs.append(reg)
    qc.add_register(reg)

reg = QuantumRegister(1, name="o")
regs.append(reg)
qc.add_register(reg)

qc.barrier()

o = CustomCircuitOracle(variable_register=regs[0], output_register=regs[2], circuit=orac, evaluate_classically_callback=classic)

grov = Grover(o, num_iterations=1)

qc += grov.construct_circuit()

qc.add_register(ClassicalRegister(n))
qc.add_register(QuantumRegister(1, name='cm'))

qc.mct([0,1,2,3,4], 14) #oracle on other qubit n = 5

for j in range(0,n):
    qc.cswap(qc.num_qubits-1,j,j+n)
    qc.measure(j+n,j)

from qiskit import execute
backend = BasicAer.get_backend('qasm_simulator')

results = execute(qc, backend=backend, shots=1000).result()
print(results.get_counts())
plot_histogram(results.get_counts())


chore: Update windows-sys

Hey, looks like this hasn't been release to crates.io? https://crates.io/crates/snapbox/0.4.4/dependencies
Can you please release a new version of snapbox? Thanks!

Released


CreateTearsheet: Support for pre-create pattern

What will this achieve?
Proposing an enhancement to the CreateTearsheet pattern that would add more flexibility to the steps within a CreateTearsheet. This is to allow alternative patterns to leverage this component.
Please allow:

Steps that don't have/hide the influencer
This is to allow for the "pre-create" pattern described below. The first "step" is to decide what steps the user might follow in after clicking the "next" button. At this stage there are no "steps" so ideally we should be able to hide the influencer. This step would also be hidden from the influencer if it is shown.

Steps that are added and removed based on previous steps
Depending on the user action on a step, additional steps might be required for the user. In the "pre-create" scenario below we need the user to be able to select between X amount of paths to take to complete the action, and based on their choice display the correct group of steps.

How will success be measured?
Specific scenario - Pre-create
I'll try and describe what we're trying to achieve at a high level to try and give a picture of what we need the CreateTearsheet to support.
When using CreateTearsheet I would like the ability to choose between a selection of "step groups" or "paths". This might be so the user can complete similar actions through the same flow. Eg: Create message queue might have different steps depending on the queue provider.
The user would first select what task/flow they want to complete and then when clicking next be shown the steps for the flow. If the user clicks the wrong path they should be able to return to the "pre-create" step using the secondary/back button.
The influencer should not show when the user is selecting what path to take but should show when they're completing a path.

Additional information

Thanks @matthewgallo!! 🎉  We'll be trying it out soon 😃


Custom holder isn't working.

Precheck

[x] Do a quick search and make sure the bug has not been reported yet.
[x] Please disclose security vulnerabilities privately at<EMAIL_ADDRESS>[x] Finally, be nice and have fun!.

Description
The "tooltip" symbol doesn't show when selecting text/words.
Environment

Editor.js version:
editorjs@latest
Plugins you use with their versions:

<script src="https://cdn.jsdelivr.net/npm/@editorjs/editorjs@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/editorjs-undo@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/header@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/quote@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/warning@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/nested-list@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/checklist@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/simple-image@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/link@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/code@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/raw@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/marker@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/inline-code@latest"></script>
<script src="https://cdn.jsdelivr.net/npm/@editorjs/underline@latest"></script>
<script<EMAIL_ADDRESS><script src="https://cdn.jsdelivr.net/npm/editorjs-toggle-block"></script>
<script src="https://cdn.jsdelivr.net/npm/editorjs-tooltip"></script>
<script src="script.js"></script>

tools: {
            header: Header,
            quote: Quote,
            warning: Warning,
            list: {
                class: NestedList,
                config: {
                    defaultStyle: 'unordered'
                }
            },
            checklist: Checklist,
            image: SimpleImage,
            code: CodeTool,
            raw: RawTool,
            marker: Marker,
            inlineCode: InlineCode,
            underline: Underline,
            toggle: {
                class: ToggleBlock,
                inlineToolbar: true
            },
            tooltip: {
                class: Tooltip,
                config: {
                    location: "left",
                    underline: true,
                    highlightColor: "#FFEFD5",
                    backgroundColor: "#154360",
                    textColor: "#FDFEFE",
                }
            }
        },

Device, Browser, OS:
Desktop PC, Brave 1.49.120, Windows10

Current behavior
It doesn't show any errors but the "tooltip" symbol isn't showing in the inline bar.
Expected behavior
See the "tooltip" symbol in the inline bar after selecting text/words.
Additional context
When I change the holder ID in HTML to "editorjs" it works, the ID before was "editor" and it didn't work.

Hello @4e576rt8uh9ij9okp, thanks for reporting this issue!
It's now fixed and available on v1.1.15 :tada:
Also don't forget to add the holder to the tooltip configuration to set the custom holder ID as follows:
tooltip: {
  class: Tooltip,
  config: {
    location: "left",
    highlightColor: "#FFEFD5",
    underline: true,
    backgroundColor: "#154360",
    textColor: "#FDFEFE",
    holder: "customHolderID", // Don't forget to add this
  },
},

Also, I'll provide you a code-sandbox to test it out: https://codesandbox.io/s/gifted-johnson-k74eh8?file=/src/App.js


Add Encrypt Text Hash Options

It would be great to have a choice of secure encryption options.  MD5, published in 1992, is no longer considered secure and SHA-1 has been deprecated by Microsoft.  SHA-2 (256/512) and SHA-3(256/512) would be great to used to secure data in D365.   In fact, to meet some US government requirements, a company must use a specific encryption method like SHA-2. I would keep the MD5 as an option, since existing data would need to be upgraded to a secure method of encryption.
Here is some info on SHA-3 hash:
https://keccak.team/index.html

Hi ArtKarp,
i'll think about it for next releases...
Thanks for your feedback.
Demian

Hi ArtKArp, you have it now. I just added in my last version, the encription to SHA512.
download the lasat version: <IP_ADDRESS>
regards

I also updated the documentation:
https://github.com/demianrasko/Dynamics-365-Workflow-Tools/blob/master/docs/Encrypt Text.md
thanks


triton always reports >0 register spills for kernels using sin/cos

We found that triton always reports >0 register spills for kernels using sin/cos. This is annoying since torchinductor leverages register spills information to skip suboptimal tuning configs.
Here is a simple repro:
    def test_sin(self):
        """
        A test to show triton always report >0 register spills for sin/cos.
        Output:
          n_regs 24, n_spills 8
        """
        @triton.jit
        def sin_kernel(in_ptr, out_ptr, numel, XBLOCK: tl.constexpr):
            xindex = tl.program_id(0) * XBLOCK + tl.arange(0, XBLOCK)
            xmask = xindex < numel
            t0 = tl.load(in_ptr + xindex, xmask)
            t1 = tl.sin(t0)
            tl.store(out_ptr + xindex, t1, xmask)

        n_elements = 128 * 1024 * 1024
        x = torch.randn(n_elements).cuda()
        y = torch.empty(n_elements).cuda()
        grid = lambda meta: (triton.cdiv(n_elements, meta['XBLOCK']),)
        sin_kernel[grid](x, y, n_elements, XBLOCK=1024)
        assert torch.allclose(x.sin(), y)

We need make 1 line change in triton code here to print register spill information:
diff --git a/python/triton/compiler/compiler.py b/python/triton/compiler/compiler.py
index 8c1dacbe1..bdf44b2d7 100644
--- a/python/triton/compiler/compiler.py
+++ b/python/triton/compiler/compiler.py
@@ -547,6 +547,7 @@ class CompiledKernel:
         self.n_regs = n_regs
         self.cu_module = mod
         self.cu_function = func
+        print(f"n_regs {n_regs}, n_spills {n_spills}")

     def __getattribute__(self, name):
         if name == 'c_wrapper':

(NOTE, I did this temporary change in triton just to make the repro easier, in inductor, we call triton.compile directly so the spill information is available in inductor. Here is the inductor code pointer: https://github.com/pytorch/pytorch/blob/28f43c767c0de48885f8de285ca4af4424d28f29/torch/_inductor/triton_heuristics.py#L172 )
The repro will output:
n_regs 24, n_spills 8

The weird register spill only happens for sin/cos. exp/log/add/mul etc. are fine.
It's possible that triton just report the register spill information back from cuda tools. But we want to understand more where the register spills come from.

Hmm yeah this doesn't really seem like a Triton issue. But I think if you look at the SASS you'll be able to see where the spills are happening.

It's likely a register spill caused by nv_sin when linking the libdevice code.
In this case, if we reduce the number of elements per thread, we might be able to get rid of the spill.


Perform sanity checks and output the results of the sanity checks

If input is State Root → Confirm whether JSON state matches the expected state root
If input is Genesis File → Identifying which state and sanity checks pass and which ones fail

Not relevant as mainnet has launched.


golangci-lint should be in parity with CAPI

Describe the solution you'd like
There are some linters that are present in CAPI but not in CAP-BYOH. We could add those missing linters in CAP-BYOH too to make the contributor experience more similar.
Anything else you would like to add:
Missing linters in CAPG:

asciicheck
containedctx
exportloopref
gci
godot
gofmt
goimports(commented out)
ifshort
importas
nilerr
prealloc
predeclared
revive
thelper
unparam(commented out)

Environment:

Cluster-api-provider-bringyourownhost version:
Kubernetes version: (use kubectl version --short):
OS (e.g. from /etc/os-release):

/assign


fix: wrong bran name

What does this change?
fix: wrong bran name

Hey @wasabeef,
Why do you consider this a fix? It's breaking backward compatibility since the User-Agent header is now prefixed with Apple, Google, etc. instead of the app's name.
Before:

User-Agent: SampleApp/1.0.0 (iOS 17.2; iPhone; Simulator; x86)

After:

User-Agent: Apple/1.0.0 (iOS 17.2; iPhone; Simulator; x86)


ionic build does not work if app.module.ts does not import some page or component

From @ceeliro on October 26, 2016 11:51
Description:
"ionic build" does not work if app.module.ts does not import some page or component in the src folder
What behavior are you expecting?
build works OK if the app.module.ts does not even import some page or component contained in the src folder as the i.e the command "ionic serve" does
Steps to reproduce:

install ionic: npm install ionic cordova
create sample project: ionic start MySampleProject --v2
run ionic build. Then it works
remove for example AboutPage and all references to it but not the component itself
run ionic build. Then fails

Running 'build:before' npm script before build

> ionic-hello-world@ build C:\Users\Carlos.Liro\Desktop\IonicReleaseProject
> ionic-app-scripts build

[13:42:08]  ionic-app-scripts 0.0.36
[13:42:08]  build prod started ...
[13:42:08]  clean started ...
[13:42:08]  clean finished in 6 ms
[13:42:08]  copy started ...
[13:42:08]  ngc started ...
[13:42:08]  copy finished in 119 ms
[13:42:09]  lint started ...
[13:42:13]  lint finished in 4.42 s
[13:42:30]  Error: Cannot determine the module for component AboutPage!
[13:42:30]  ngc failed
[13:42:30]  ionic-app-script task: "build"
[13:42:30]  Error: Error

npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js"
 "run" "build"
npm ERR! node v6.4.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR! ionic-hello-world@ build: `ionic-app-scripts build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the ionic-hello-world@ build script 'ionic-app-scripts build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ionic-hello-world package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ionic-app-scripts build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ionic-hello-world
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ionic-hello-world
npm ERR! There is likely additional logging output above.

Post the output of ionic info below please
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.4
Ionic App Lib Version: 2.1.2
Ionic App Scripts Version: 0.0.36
OS: Windows 10 Pro
Node Version: v6.4.0
Copied from original issue: driftyco/ionic-cli#1592

@ceeliro,
This seems like the expected behavior to me. What am I missing here? I'm going to close this, I'll re-open if needed.
Thanks,
Dan

Hi @danbucholtz!
Just to clarify. So You mean that if I create new component but still in development and I don't want to add it to the app.module.ts to be load then I can not build the whole thing?
From my point of view and developing/testing view this is and unwanted behaviour due to if I have many components in development that I don't want to still include in the build process then I cannot build to test with those that I want to include and I have to remove from the project those others that I don't want to include.
Another issue related to this if that then if I do back up of the app.module.ts to app.module.bk.ts to have a clean/save app.module.ts with just components that I want to use then I cannot either build the thing very likely because of the same issue with building
Anyway up to you and your team!
Kindest regards and thanks for the time taken with this!
Carlos Liro

https://github.com/driftyco/ionic/pull/8921/files << Here is your fix ceeliro!

i'm still having the same issue...
npm ERR! Darwin 16.0.0
npm ERR! argv "/usr/local/Cellar/node/6.5.0/bin/node" "/usr/local/bin/npm" "run" "build"
npm ERR! node v6.5.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR! ionic-hello-world@ build: ionic-app-scripts build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the ionic-hello-world@ build script 'ionic-app-scripts build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ionic-hello-world package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ionic-app-scripts build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ionic-hello-world
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ionic-hello-world
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Applications/AMPPS/www/TypeScript/ionic2/gcmFinal/npm-debug.log
Caught exception:
undefined


Capture probe requests

Hy...Is there any way to capture probe requests in windows in python without using wireshark or any other software?
Kindly help.

Hi, I'm not a windows user myself, so I don't personally have experience with these tools, but I know Scapy is a popular python packet capture library that I assume is cross platform. Most packet capture tools are built on top of libpcap, which has a python package and maybe windows support through WinPcap. A friend from Thotcon also wrote trackerjacker, which is a python library that may be helpful for you depending on what you are working on. Sorry I can't be of more help specifically. If anyone else has experience doing packet capture on Windows feel free to chime in here.


Collapse express and enterprise bootstrap/roles

This PR collapses the st2express and st2enterprise profiles into a single bootstrap.
This new feature installs all the packs that are typically deployed on an enterprise instance and makes them default for express as well.
This change requires a user to enter an Enterprise Key in either the answers file (to be documented) or the GUI installer in order to retrieve any of the other enterprise bits.

once this merges lands, I'll update stackstorm.com/install.sh to point to ....bootstrap-st2

/cc @DoriftoShoes Would you also put some eyes not only on this change, but anywhere else you think things might be needed. I think I got everything, but it could use a another good comb.
Also, testing. Needs that too. :)

@dzimine cool! That's good hygiene, but shouldn't be needed immediately. The bootstrap-st2express script now symlinks to bootstrap-st2. We'll get benefit as soon as this merges.

Also, before this merges, Atlas vagrant images need to be rebuilt or vagrant will fail. Starting that now.

I thought agreed to do this post 1.1.0 to reduce the number of changes. Something changed or did I misunderstand?

@manasdk that's fine. The PR can still sit here. However/whenever we actually release. These are the bits

I guess the question is more about what are we doing - merging this PR or not?

@manasdk I think the PR still needs some testing and :+1: from someone other than myself.
I'm going to test it tomorrow, and if it works, and I have a :+1:, I'll merge.. The request I made was that we'd get the CI matrix stabilized before making smaller changes like this. I don't really have a problem merging this.
This is quite literally one of the last things on my list. I still have a few more things to do as well, so :clock1: will decide if this makes it or not.

This looks good to me.  I do believe we should install both auth backends for enterprise...hell unless we make the installer more intelligent we should probably enable PAM by default on both.

This looks good to me. I do believe we should install both auth backends for enterprise...hell unless we make the installer more intelligent we should probably enable PAM by default on both.

Yep that's the approach we should go with, for now, imo.
Incrementally and later on we can update installer to support selecting and configuring LDAP backend if the license key is present...

I merged into master into this branch and everything seemed to have passed with the new script/bootstrap-st2enterprise script.
What's out plan for this PR? Do we plan to update it so pam is used by default in st2enterprise as well and merge it into master?

@Kami yup. that's the plan. we have CI passing now, so it's possible to ensure that we don't muck things up too terribly now. Going to work on this this AM

LGTM! Merge when ready.


species positive array for new forms in v2

[x] Figure out where it gets populated from in v2
[x] Figure out where it should be in v2 and put it there
[x] Populates as expected

Found the change in activity reducer. V1 has it updating the entire activity, V2 just the form_data. Curious if more data is needed and if we can update the entire activity rather than just the form_data