Q - Identity: Using a custom User type, UserStore, and non-string Id

Hi,
This would be more of a general question about Identity, but I feel like the docs are confusing enough that it should be a real issue. I apologize for the long form. I have a tendency of not being able to stay concise, but I'm trying.
For context (you can skip this)
I have a very simple setup. I want to integrate with an old firebird database that is handled by an old Dephi program. Data is recorded through the Delphi program, and I should build a really basic Webshop around it.
(Later, the whole Delphi program will be converted to ASP.NET, so I want to have some coding done for that too.)
For this I've chosen the ASP.NET Core 3 Web API / NHibernate / Angular stack.
Uphill battle considering 70% of the MS documentation assumes Entity Framework, and the use of templates / generated code, service registration / middleware configuration through undocumented shortcut extensions which configure another 10 service / middleware ...
(Which I, personally despise, as I like to control and know everything my applications do.)
Problem comes, when I want to integrate some very basic stuff by Identity.
I only have a "Users" table (user, pass, email) for now, which I can only modify for critical needs.
Problem
I started here but by the end I realized this is completely useless for me, as it seems to assume an Entity Framework layer which I don't have and don't want.
I went onto this one and this just screams of confusion and contradictions.
My users table only has 3 columns, and the docs in Customize the user class say I can have only an Id and a UserName field. So that would be a check.
The next few lines are already telling me, that because my users have an integer Id, I have to  inherit from IdentityUser to specify that...?
I don't want those extra properties... I don't want to specify TUserLogin, TUserRole, TUserToken and TUserClaim, as I don't need any of those functionalities, at least not yet.
I've read through it all, and I have more problems, but for now, this is by far the biggest one. I just need a simple user class with an integer Id.
For now, I wouldn't even use roles, or claims.
This is all the functionality I'll need for my UserStore;
public class IntegraUserStore : IUserStore<IntegraUser>,
                                  IUserPasswordStore<IntegraUser>,
                                  IUserEmailStore<IntegraUser>,
                                  IQueryableUserStore<IntegraUserStore>

But if I go on like this, the interfaces just require me to have string Ids...
I would like some pointers in the right direction. My goal is an absolute minimal but completely transparent / explicit setup, with only critically needed DB modifications which I can eventually build up to a more "complete" Identity integration.
Thank you in advance!

If you only have 3 columns why use identity at all? It has the assumption you want so much more.
Just do your check, and then use the cookie middleware to write out a secure cookie.

In the end, that's what I did. I'm just using cookie authentication. This Issue on one hand solved itself, on the other, it did not. I still believe it to be a really bad sign that the framework, while providing a lot, is so rigid. It's built on the principles of flexibility, but still some of the most common needs are not supported.
I wanted to use Identity, so that I could build up the implementation as we needed them. The project will grow. Isn't that why supported features can be provided in the form of implemented interfaces?
You can close the issue if you feel like it's not actionable. But I'd love to see some voices about this.


Update e2e with better errors

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Communication Credits unavailable for PAYG Calling Plan

Hi,
In the following scenario, you are unable to purchase Communication Credits due to dependency requirements:

Get a common area phone and associate a user
Buy the license for the common area phone
Assign a PAYG license to the common area phone user
Assign communication credits license to the user -> DOES NOT WORK

Communication Credits only support these 2 dependencies:

Domestic Calling Plan
Audio Conference

Leaving in the dust:

Domestic and International Calling Plan
PAYG Calling plans

Dependency has to be updated to reflect the new calling plans.
Regards,

Hi,
You can close the question / comment. Microsoft has fixed the dependencies
and is now on par with the provided documentation.
Thanks
On Mon, Nov 14, 2022 at 3:54 PM Dario Woitasen @.***>
wrote:

Hi @oragain https://github.com/oragain, can you please tell us to which
article(s) your question applies? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9437#issuecomment-1314365893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEAFUUEW2IVZSJTJOF6FAHTWIKRHTANCNFSM6AAAAAAROBOE5E
.
You are receiving this because you were mentioned.Message ID:
@.***
com>


Panels - nodes that are not movable, holding only one tab at a time

I'd like to use the doc control for side bars and bottom bars on my app, to allow users to drag and drop the tabs between these sections.
I have a middle Node that i'm creating that I'd prefer if users couldn't drag it or drop other tabs from other nodes into this tab.
i've added a function to DockArea that allow me to set that no tab labels should be shown - this isn't particularly useful in my use case, as it really prevents the ability to see more than one tab in any node - or drag any of them anywhere.
I've made a couple of changes to enable this functionality,
i've added a prop on Node::Leaf hide_label that allow one to set if they what that node to hide the labels - if there are no labels showing it prevents the ability to drag the tab in the node, and i've also used the same setting to prevent dropping other tabs into the node.
to enble this on a node, i've create the additional functions split_left_single... which only accepts a single tab and creates the Node:Leaf with hide_label set to true.

Looking through the changes so far I think it would be better if the implementation relied on a new kind of node (perhaps something like Node::Panel or Node::Single) or a reimagining of Node::Leaf. egui_dock hasn't ever really displayed anything that isn't a leaf before, so by introducing multiple simultaneous types of nodes we should ideally make it as "right" as possible before opening that can of worms.
Ideally any sort of functionality you wish to have (such as restricting a node to only one tab) should be reinforced by the Node type, otherwise we might set a president for tacking on new fields onto Node::Leaf endlessly for new types of nodes, which wouldn't be sustainable long term and annoying for future developers.
Im also not fond of the idea of being able to disable all nodes from having their tab bars disabled via DockArea, since it sounds like a backalley way of achieving a typical TopBottomPanel/SidePanel ui in vanilla egui. It could also introduce further confusion for developers who aren't familiar with the bigger picture of the api. So I think it's better if it was left to being decided on a node to node basis only.
However it's decided that this implementation should move forward i would highly recommend considdering how future features may come into play, eg. by generalising DockArea to accept displaying multiple kinds of nodes rather than just Node::Leaf or generalising tabs inside Node::Leaf to allow for multiple tab configurations with good customization.

hey @Adanos020 I can remove the show_label_bar option, don't need it for my use case, just put it in initially to help me follow the code and agree with the comments from @Vickerinox that when turned on it's really just reproducing the native egui functionality.
i'm currently using the leaf_with_single calls to set the additional hide_label prop on the node - which is then used in the show_inside method to prevent showing the label bar on the node.
the split_..._single methods only accept a single Tab rather than an array - to prevent people passing multiple Tabs when creating a node that will not show the title bar and therefore will not allow switching tabs in the node.
happy to explore the other option @Vickerinox of creating a different node type, like Node::Single
would probably then need the four different split methods to generate the nodes, and could add functionality to DockArea for rendering the nodes, to remove changes from the Node::Leaf implementation in the current PR.
let me know if you'd be interested in me moving forward with this other approach?

i'm currently using the leaf_with_single calls to set the additional hide_label prop on the node - which is then used in the show_inside method to prevent showing the label bar on the node.
the split_..._single methods only accept a single Tab rather than an array - to prevent people passing multiple Tabs when creating a node that will not show the title bar and therefore will not allow switching tabs in the node.

Once we switch to the Node::Panel approach, we could still do fine without these methods. Instead, I'd opt for creating an enum like this:
enum InsertTabs<Tab> {
    Panel(Tab),
    Leaf(Vec<Tab>),
}

which could be passed to the split_* methods instead of the Vec<Tab>. This way we could keep the number of methods to a minimum while still ensuring that the correct number of tabs is passed every time.

ok, I'll see what I can do - i'll hopefully get an opportunity to have a look at it over the next couple of weeks.


How to balance the number of quic connections on each thread

Title: How to balance the number of quic connections on each thread
Description:
I have a QUIC Listener as follows:
  - name: listener_5  # listen tcp connection from high_domian get response
    address:
      socket_address:
        protocol: UDP
        address: ip
        port_value: port
    udp_listener_config:
      quic_options:
        quic_protocol_options:
          initial_stream_window_size: 65536
          max_concurrent_streams: 8192
      downstream_socket_config:
        prefer_gro: true  # GRO（Generic Receive Offload）
      udp_packet_packet_writer_config:
        name: envoy.udp_packet_writer.udp_gso_batch_writer
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.udp_packet_writer.v3.UdpGsoBatchWriterFactory
    per_connection_buffer_limit_bytes: 10000000

I found that the number of quic connections per thread was not evenly distributed. Is there a mechanism to distribute it as evenly as possible on each worker thread?

@danzh2010 @RyanTheOptimist for any recommendations

Are you using BPF as recommended here: https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/http/http3#bpf-usage

Yes. I've already used the BPF.

@danzh2010 Do you have any thoughts here?

BPF should have distributed connections evenly. Did you use a customized connection id generator? The default one distribute the connections based on the first 4 bytes of their connection ids.


[Excel] (Custom Functions) Add note on Simple CORS

OPS Build status updates of commit a4fc1aa:
:clock10: Preparing: average preparing time is 44 sec(s)

OPS Build status updates of commit a4fc1aa:
:clock10: Incremental building: average incremental building time is 29 sec(s)

OPS Build status updates of commit a4fc1aa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-runtime.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@mscharlock , @davidchesnut  - please review. if you agree, can we add a note to our tutorial as well?

Also, could we add a note to specify only these types are supported for the XHR requests:
application/x-www-form-urlencoded
multipart/form-data
text/plain

(Re-titled this PR using our standard convention.)

OPS Build status updates of commit 487624e:
:clock10: Preparing: average preparing time is 44 sec(s)

OPS Build status updates of commit 487624e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-runtime.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 94b463c:
:clock10: Preparing: average preparing time is 44 sec(s)

OPS Build status updates of commit 94b463c:
:clock10: Incremental building: average incremental building time is 29 sec(s)

OPS Build status updates of commit 94b463c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-runtime.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


tour: Map literals less changes and text details

Between 'map-literals.go' and 'map-literals-continued.go' there are four changes (omit element type, single line the block, add space to align the "Google" element, and remove the terminal comma in the block) which makes it hard for a reader to understand what it is "you can omit" in the 'Map literals continued' section of 'moretypes.article'.
To avoid the three text changes that the article does not talk about, update the before code in 'map-literals.go' to already be formatted on a single line per map entry.
Add code examples in the article explaining the two versions and why the literal element type may be elided.
Fixes golang/tour#1449

With the extra detail in the article I think this also
Fixes golang/tour#174


Rectangle box goes up, and down when I am scrolling on Twitter (canary)

Version 84.0.512.0 (Official build) canary (64-bit)

Thanks for reporting this issue. This repo is used for documentation. If this is still an issue and hasn't been updated in a more recent version of Edge, please report the issue by submitting the issue in Edge - select Alt+Shift+I on Windows. I'll go ahead and close out this issue now. Thanks.


defaultProps is ignored when validating a prop that has sub-methods (e.g. Date)

Issue
It looks like the parser is ignoring the fact that I'm defining a fallback for my optional prop, therefore stating I might call getTime() on an undefined prop.
However, I'm falling back on today's date using defaultProps, which should take care of that.
call of method `props.date.getTime()`. Method cannot be called on possibly undefined value undefined

My real case scenario is using moment.js and .format, but it doesn't matter, as I was able to reproduce with "normal" JS dates as well.
Repro
Reproduced on flow.org/try
Config

App scaffolded with create-react-app
flow-bin v0.62.0
flow-config pristine

p.s. coming from TypeScript, I recently starting exploring flow... I LOVE IT!

This is working as expected, actually. The type of props should be non-optional, and defining the defaultProps field makes the field optional the type of the "config" object. Note the distinction between props and config. Props are what you have in the render method of a react component, while config is the object you pass to createElement (via JSX or not).
Try Flow example

I'm not sure I get it.
If date is not optional, as you defined in your interface, why isn't flow erroring when calling <MySFC />?

In the body of the MySFC function, date will always be present, because it's provided by defaultProps if not explicitly passed in. That's why the field should not be optional in the props type.
Loosely speaking, you have something like this: Props = Config + DefaultProps, where Config is the object that you pass to React.createElement.
Does that make more sense?

Sorry, delete my comment cause I got it after re-reading 😄
I guess my confusion comes from the fact that both TypeScript and PropTypes work in a slightly different way. I could be wrong, but I don't think they make a distinction between Config and Props.
Anyway, got it, thanks!

Yeah, I'm not super familiar with how TypeScript deals with this problem :)


400 Client Error: Bad Request for url https://trial-888888.okta.com/oauth2/v1/token

Device not activated
Your device cannot be activated because of an internal error when running the following command:
./gimme-aws-creds
following is configuration file
okta_org_url = https://trial-888888.okta.com
okta_auth_server = default
client_id = 0.....7
gimme_creds_server = https://trial-888888.okta.com/home/amazon_aws/0....7/272
aws_appname =
aws_rolename =
write_aws_creds = True
cred_profile = role
okta_username =
app_url =
resolve_aws_alias = True
include_path = False
preferred_mfa_type =
remember_device = n
aws_default_duration = 43200
output_format =
force_classic = False
open_browser = True
Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

App Version used:
Environment name and version:
Operating System and version:

Do you see any errors in the Okta logs?  That error message is not getting generated by gimme-aws-creds, so it must be coming back from Okta

Yes, the error in the OKta logs is "[FAILURE]: user_not_assigned

your configuration is incorrect.  You should have:
gimme_creds_server = appurl
and
app_url = https://trial-888888.okta.com/home/amazon_aws/0....7/272
That will request the SAML login for that Okta app after initial authentication


Steamcmd can't locate app build file

When using the app_build parameter, steamcmd will fail with a file not found error regardless of whether the build file exists or not. This is due to the fact that the app build file path is passed to the program in quotation marks, and the quotation marks are part of the argument. Note that this issue isn't present on any of the other path-based arguments.

@TeraV8 thanks for reporting, can you give an example of a workflow file that breaks for you?

https://gist.github.com/TeraV8/310c489a34ecdc026307798272db05ee

Thanks, I'll look into this and fix it. Thanks for the PR, but I just need to do a quick build validation to make sure the changes work even with paths that contain whitespace characters.

I noticed that you use both windows and ubuntu, was it only the ubuntu runner that was failing?

Not sure, the windows job canceled as soon as the ubuntu build failed. It should be the same problem on both though, because the issue is the raw path string received by steamcmd contains the quotation marks

I def use this on my windows machines, but never tested fully on ubuntu. Can you add fail fast false  to your matrix to test it out?

try this:
    uses: buildalon/upload-steam@development
      with:
        username: ${{ secrets.STEAM_USERNAME }}
        config: ${{ secrets.STEAM_CONFIG }}
        app_build: ${{ github.workspace }}/.github/workflows/build.vdf

Updated workflow file: https://gist.github.com/TeraV8/e8a39ae0172792f2b80e3746436c6a7c
It only runs on ubuntu now, but the issue is solved. Haven't tested with spaces in the filename, but I believe that shouldn't be an issue if core.exec escapes it properly.

can you please test for windows as well please?

https://gist.github.com/TeraV8/4528ffdb0880d9aea5e5f5fcee02687a
It works fine when on Windows with a space in the pathname. core.exec adds the quotation marks when necessary, but they seem to become part of the argument when they aren't escaping anything.

Thanks! I'll publish shortly


New version: trinsic.cli version 1.13.0

Pull request has been created with komac v2.1.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @TejasMate,
This manifest adds Apps and Features entries that aren't present in previous PR versions. These entries should be added to the previous versions, or removed from this version.
(Automated response - build 869.)


thunk type annotations are incorrect

We assumed that thunk could take functions that return any kind of Virtual Element. Turns out it needs to be a Node as neither ThunkNode nor TextNode are supported.
Type annotations should reflect that so type checker could catch those errors.

For details see browserhtml/browserhtml#1102

Closing in favor of https://github.com/mozilla/reflex-virtual-dom-driver/issues/10


ImportError: cannot import name 'VideoManager' on v0.6

Follow the Quick Start (Python API)
I  run the scripts in my docker with PySceneDetect version ==v0.6-dev3（installed by pip）
unfortunately, i get this error:
ImportError: cannot import name 'VideoManager' from 'scenedetect'
however, it runs well with command  line:
scenedetect -i video.mp4 detect-content split-video

Hi @LianShuaiLong;
Make sure you are looking at the v0.6 documentation (bottom right should say 'v0.6', not 'latest'):
http://scenedetect.com/projects/Manual/en/v0.6/api.html#quickstart
VideoManager is part of the old API.  Can you share your script?
Thank you!

This is my simple test script:
`from scenedetect import VideoManager
from scenedetect import SceneManager
#for content-aware scene detection
from scenedetect.detector import ContentDetector
import argparse
def parse_parser():
parser=argparse.ArgumentParser()
parser.add_argument('--input_video',type=str,default='video.mp4',help='Video path')
parser.add_argument('--threshold',type=int,default=30,help='Scene detect threshold')
args = parser.parse_args()
return args
def find_scenes(video_path,threshold=30):
#create our video & scene manager, then add the detector
video_manager = VideoManager([video_path])
scene_manager = SceneManager()
scene_manager.add_detector(ContentDetector(theshold=theshold))
#improve processing speed by downscaling before processing
video_manager.set_downscale_factor()

#start the video manager anf perform the scene detection
video_manager.start()
scene_manager.detect_scenes(frame_source=video_manager)

# each returned scene is a tuple of the (start,end) timecode
return scene_manager.get_scene_list()

if name=='main':
args = parse_parser()
video_path = args.input_video
threshold = args.threshold
scene_list = find_scenes(video_path,threshold)
print(scene_list)
`

In v0.6 you can just use the detect function to get a scene list, e.g. instead of
scene_list = find_scenes(video_path, threshold)

You can write:
scene_list = scenedetect.detect(video_path, ContentDetector(threshold=threshold))

See this example for more advanced use.
Thanks!

There is now a backwards compatibility layer in the latest v0.6 release (rc0) which should allow most v0.5.x scripts to work with the new version.  This layer is only temporary and will be removed in a future version, so new code should not use VideoManager anymore.


WIP: Use normalized path as a file name

Hello,
On some artifacts collection (e.g. ChromeExtensions) it's hard to recognise what file was actually collected without looking at items.db contents.
What do you think about the idea that we can use normalised path as a resulting file name?

Hi and thanks for the PR. I'll have a deeper look at this tomorrow and do some tests on Windows.

Turns out, paths are already normalised and don't contain : or \\, so the code could be simplified a lot.

Long paths create issues with the windows internal zip decompression, so I reduced that. I also shortened the directory name lengths. I hope this also solves your issue.

Thanks!


dts: rt11xx: Fix invalid SAI4 address

Register address definition for SAI4 was incorrect.
Fixes: #42435
Signed-off-by: David Leach<EMAIL_ADDRESS>

ping @dereksnell


Chat name and icons misaligned in Zen appareance [BUG]

Description
Hi,
When the Zen appareance mode is selected, the chat name and the icons are not aligned anymore in the left menu. Now that the avatar is missing, the icons should also align to the left, as shown below :

It seems the span class shown below should be removed in Zen mode :

However, the result is slighly off because of the first icon padding :

I would suggest also adding a left-padding of 6px to the title above :

Hope this helps. Great tool.
Device and browser
Windows 10, Chrome Browser.
Screenshots and more
No response
Willingness to Contribute

[ ] 🙋♂ Yes, I would like to contribute a fix.

Thanks very much for reporting @jpalbert . This issue has been fixed on the main branch and will be released in the coming days as part of 1.15.0

Ready for release.


power.deep_sleep() without wake up sources does not go to sleep

This might be related to this CODAL issue:

https://github.com/lancaster-university/codal-microbit-v2/issues/203

However, based on the latest conversations with @JohnVidler we were thinking the issue might be in the MakeCode extension instead of CODAL.
However, in MicroPython executing power.deep_sleep() doesn't go to sleep either. Having a brief look the MicroPython code I can't see an obvious reason why it might not go to sleep if it's supported within CODAL itself:
https://github.com/microbit-foundation/micropython-microbit-v2/blob/2233d79b8ab15bb769adaa44d53874ce80ebd3eb/src/codal_port/modpower.c#L54-L109
https://github.com/microbit-foundation/micropython-microbit-v2/blob/6ea476f3ddb4eb7aba13ca264fab9d15abaae3ca/src/codal_app/microbithal.cpp#L130-L136
Example programme to replicate with MicroPython v2.1.0-beta.3:
from microbit import *
import power

while True:
    power.deep_sleep()
    display.show(Image.SURPRISED)
    sleep(500)

This always shows the SURPRISED face and doesn't go to sleep

I think this is a "feature" of the CODAL: if there are no wake up sources configured, and no timeout, then the deepSleep() call returns immediately (presumably to prevent infinite waits that cannot be interrupted, requiring a reflash of the microbit to recover).

We could fix this on the MicroPython side by just calling the CODAL deepSleep(ms, true) with a large timeout, and keep calling it in a loop forever.

This has been changed/fixed in the CODAL side in this PR, so we can just wait for the next tag to get the fix:

https://github.com/lancaster-university/codal-microbit-v2/pull/229

This is resolved in CODAL v0.2.43, we can close it when https://github.com/microbit-foundation/micropython-microbit-v2/pull/132 is merged, or a different commit updates CODAL in the main branch.

CODAL v0.2.43 is updated here.


What is the install command for the plugin

Hi there,
You forgot to write the installation manual in your readme.md file. What's the name of the package? I can't find any by searching Laravel Blade in sublime text

It's pretty clear from the readme with either the automatic install with package control or manual installation. Please see the readme or check the link to package control for installation instructions.

Yes, I can see but when I search it on package control -> install package, it's not showing there. :(


Handling failed async requests

I'm using RESTful API and WebuiPopover with async option, and I have a problem.
Normally, when my item isn't found my API returns HTTP 404 and the following payload:
{
 "error": {
    "code": "GEN-NOT-FOUND",
    "http_code": 404,
    "message": "Resource Not Found"
  }
}

Unfortunately there's no option to handle these requests.
    async: {
        success: function (that, objectResult) {
          console.log(objectResult); //this code is never called
        }
    },

So I quess it would be amazing if there could be a way to handle failed requests. Like by including something opposite to success, ie failed method.
    async: {
        success: function (that, data) {
        }
        failed: function (that, data) {
        }
    },

Nevermind, I see it's already implemented :+1:
Time to update


adding shushi coin 5

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Hello, please help me!! i have had 5 auto merge errors..... what seems to be the problem? I am just trying to add my logo and name to my token...nothing else.

I can help stop wasting your time refer this zakimzf/profile#1


Support for UWP and System XAML

Filing this here to track a dependency on cswinrt from https://github.com/microsoft/react-native-windows/issues/6539
We would like to be able to move away from C# UWP but we still need to target system xaml. So we would like the ability for cswinrt to target UWP apps and system XAML so that we can avoid a whole bunch of issues related to .net native and consuming nugets from C++ apps.
Related: https://github.com/microsoft/react-native-windows/issues/6539

This would require custom type mapping for xaml to map onto inbox System XAML. Right now the custom type mappings go against WinUI.

I'm a little surprised that C#/WinRT doesn't already support system XAML. Just a noob here, but I had assumed that "lifting this projection support out of the compiler and runtime" would also entail decoupling from WinUI. Are there technical reasons why it can't/won't be done?

Also this will unblock the XAML islands on .NET 5+ scenario, without this we'll have to wait for WinUI 3 XAML islands which is "scheduled to a later release".

I have a component that exposes a few platform XAML types. Before I could just use them from C# but now with C#/WinRT, it blacklists all those types from the projection, unlike every other WinRT projection. Even if you reserve certain mappings to Microsoft.UI.Xaml (eg: INotifyPropertyChanged), Windows.UI.Xaml types must be available even if you have Microsoft.UI.Xaml (from WinUI 3) referenced. C#/WinRT shouldn't be framework dependent.

@angelazhangmsft  This issue is hanging on thin air and should be dropped right away.
UWP/CoreWindow was doomed to oblivion a loooong time ago.
As .NET 5 / .NET 6 / .NET 7 and upwards versions themselves will never support it , CsWinRT can't do very much, can it ?

.NET 5 / 6 / 7 won't be compatible with AppContainer and UAP-only environments anytime soon, so UWP is not actually relevant here.
This is mainly for XAML islands.

so the title be modified to reflect XAML Island only. (Support for UWP and System XAML for XAML Islands.
<RE> well contrary to MS marketing, AppContainer is not tied to any Window specific tech (CoreWindow aka UWP)</RE>

I don't think that it is necessary, you can actually create UWP apps with .NET 6, however the incompatibility with System XAML in stock C#/WinRT blocks that.

That sounds like great news, I am always baffled by the lack of XAML islands support in .NET 5/6.

@gus33000's fork of CsWinRT is at https://hub.fastgit.xyz/gus33000/CsWinRT/tree/wux.

There are some roadblocks at the moment for having both Windows.UI.Xaml and Microsoft.UI.Xaml support in CsWinRT, which is why I haven't tried to do a PR yet for this. Mainly you have interop classes for .NET/WinRT that can currently only be applied to Windows.UI.Xaml or Microsoft.UI.Xaml. (As an example, CollectionChanged) It would be a bit messy at the moment to have support for both in the code. Maybe someone has an idea in order to make this a bit better?

There are some roadblocks at the moment for having both Windows.UI.Xaml and Microsoft.UI.Xaml support in CsWinRT, which is why I haven't tried to do a PR yet for this. Mainly you have interop classes for .NET/WinRT that can currently only be applied to Windows.UI.Xaml or Microsoft.UI.Xaml. (As an example, CollectionChanged) It would be a bit messy at the moment to have support for both in the code. Maybe someone has an idea in order to make this a bit better?

Is there any specific reason why WinUI 3 needs C#/WinRT coupling? If no then I think the best approach is to just remove that coupling.

@gus33000 Most of the types eg: CollectionChanged are very similar across both WinUI 2.x and WinUI 3 so those can be projected as the same type while other types will be projected separately.

This should definitely be addressed because .net core 3.1 is going out of support by the end of the year and I doubt that winui 3 will be a good replacement for it by that time.

C#/WinRT 1.6.1 should already support building projections for system XAML, however building projections for libraries like the XamlApplication Toolkit or WinUI 2 causes the "class not registered" exception.

Mainly you have interop classes for .NET/WinRT that can currently only be applied to Windows.UI.Xaml or Microsoft.UI.Xaml
I guess it should be fine to split CsWinRT into two libraries right? CsWinRT for the core WinRT projection, and CsWinRT.WinUI for XAML related stuff (with separated assembly for system XAML and WinUI 3 XAML).
Now with trimming working the worry of yet another DLL is not that relevant anymore.

Maybe someone has an idea in order to make this a bit better?

How about adding a new property (i.e. CsWinRTXamlNamespace), like C++/WinRT is currently doing?
Of course, the property would be set to Microsoft.UI.Xaml by default.

How much code in WinRT.Runtime is dependent on the XAML implementation? Can we generate entire classes like C++/WinRT does and eliminate the issue all together?

It powers some XAML types, as well as properly managing some error codes.

If someone wants to try this out, #1421 adds System XAML support to CsWinRT, and Windows.UI.Xaml projections 🙂


2.6 ?

https://github.com/doctrine/doctrine2
refer to the 2.6 branch for the current stable

great! they started use php 7.4 it's version don't use in production eat! very professional. my project is down. my work was stopped. thanks a lot.

2.6 is allowed by the contraint (^2.5.11 allows 2.6)


Refresh browser occur error: "Error in event handler for (unknown): TypeError: Cannot read property 'data' of null",

I follow the react-router examples huge-apps to write a sample example. But occur a error every refresh the browser.the error tree:

my code:

react version:^0.13.3
react-router： 1.0.0-rc1
How can I fix this error?
thanks

The error is caused by a chrome extension (json-handle). Try again without it.

@taurose .Thanks. It works fine after remove json-handle.


Update to jupytelite.github that has a new xarray version

The xarray version on jupyterlite.github.io is 2023.12.0.

addresses #679
@jhamman @andersy005 I'm currently working with folks at the scientific python developer summit trying to get these Vercel previews to work, I'm not sure who currently has permissions - I requested to be added

thank you for this addition, @thomasjpfan!

addresses #679
@jhamman @andersy005 I'm currently working with folks at the scientific python developer summit trying to get these Vercel previews to work, I'm not sure who currently has permissions - I requested to be added

@scottyhq, i approved the deployment.
regarding the addition to the team, unfortunately we are on a free tier plan and adding new members isn;t free

unfortunately we are on a free tier plan and adding new members isn;t free

Bummer. Well all good. who currently has access @andersy005 ? just you? I'm not sure why it's once again asking for authorization? But this seems good to merge from my point of view!


use helm for installing cert-manager

close #22

I think that most valuable is version not the url of cert-manager repository. So added version, repo name and repo url.


Improve GAN Network

Hello, let me just say I am a big fan of the code in this repo. I noticed though that the reference implementation you use for the GAN example is from November of 2015. I'd like to suggest that the code be improved by utilizing the latest techniques from a recent OpenAI paper. The code can be found here and is already in Tensorflow, so it shouldn't be too hard to import. It would be a welcome addition since the main reason I use this project is for the very efficient GAN implementation. From what I've read from skimming over the paper, OpenAI modifies the loss function for the generator and discriminator to make them converge faster.

I have implemented these changes in my version:

mini-batch discrimination
switched the labels 0 <-> 1
label-smoothing

However, I did not observe any (dramatic) changes in the examples, did you run some tests?
But they have the correct log-likehood term in InfoGAN :wink:  I don't know what @ppwwyyxx's plans are. But the next example I planned to move to tensorpack is InfoGAN with a distribution-class for different codes.

@PatWie I've noticed some difference on my own dataset with improved GAN. It seemed to help with my dataset in terms of yielding qualitatively better results at earlier epochs on their implementation. I maybe mistaken but isn't one important part of the paper you are missing is feature matching in section 3.1? Or have you called it something else? It also sounds like it will fix the issue you are having currently where the GAN is unstable. That sounds like possibly the most important improvement in the paper, but I could be mistaken.

About improved GAN, it interests me as well because it introduces a lot of new ways of training a model, so it would be good to implement them to test whether the design of tensorpack is really flexible.
About InfoGAN, @PatWie could you elaborate what do you mean by "correct log-likelihood"? I used OpenAI code as a reference when I wrote the examples but maybe I missed something.
Also it would be nice to have distribution classes like the original code.

correct log-likelihood
I didn't saw the sign.

Another issue with the wait infogan is written is that it's not easily configurable for nonmnist datasets. It would be nice to make it more flexible for other datasets. Other infogan repos even let you set the discriminator and generator via the command line. Expanding infogan to run on different datasets would really also show if tensorpack is flexible. Just a thought. Using infogan on the celebA dataset for instance would be a good exercise.
Any updates on the improved GAN? Was it included in #107 or was that just the groundwork? Looking forward to trying out the improvements.

You can simply change the one line which reads the mnist dataset to two other lines which reads the CelebA dataset.
But you'll certainly need to change your architecture and it's impossible to do it automatically.
I would refuse to yse command line to define complicated things like a network, because it only makes things less flexible.

My colleague just recommended a paper on GANs: https://github.com/martinarjovsky/WassersteinGAN
Supposedly this paper uses earth mover's distance to "virtually guarantee" that GANs will converge.  Would love to see this implemented in Tensorpack. @ppwwyyxx @PatWie

So they claim, they solve all issues with GAN :wink: .
As far as I can tell the differences are:

good weight initialization
get all weights from discriminator (alias critic), only know a hacky solution. Maybe @ppwwyyxx has an elegant way.
add clipping (trivial)
run several optimization steps of the discriminator (currently ugly hack)

I just came up a few lines, but I have to read the paper in detail.

So aside from the reference implementation, I found an implementation that can use Tensorflow as backend. (It seems like it might be using a feed_dict or something because I get relatively low GPU utilization), and even one in Chainer.
The fact that this GAN is already implemented in so many frameworks less than a week after release bodes really well. Also I have tested at least one of the implementation and while it was unoptimized, I was impressed how quickly the loss dropped (even if the epochs did take a while).  So far it seems that the might have actually solved (most if not all) of the issues which would be really impressive!

It is not my primary focus but I really like to see a working wGAN as well.

I think this is out-dated.

Agreed. Amazing how fast the GAN research has been in recent months. This repo has some of the best most flexible GAN implementations out there. Keep up the good work. 👍


Unable to use WebSocket provider with Remix SSR

Faced a problem trying to use WebSocket provider (web & node) with Remix.
Steps to reproduce:

Create a basic Remix app using npx create-remix@latest
Import provider function

import {getWsProvider} from "polkadot-api/ws-provider/web";

Create PolkadotClient at default page

const client = createClient(withPolkadotSdkCompat(getWsProvider('wss://westend-rpc.dwellir.com')));

Run npm run dev

Web WS provider throws error:
4:16:19 PM [vite] Internal server error: WebSocket is not defined
      at file:///Users/yaroslav/Documents/Work/remix-papi/node_modules/@polkadot-api/ws-provider/dist/web/esm/web.mjs:4:45
      at ModuleJob.run (node:internal/modules/esm/module_job:218:25)
      at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
      at async nodeImport (file:///Users/yaroslav/Documents/Work/remix-papi/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:53045:15)
      at async ssrImport (file:///Users/yaroslav/Documents/Work/remix-papi/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:52903:16)
      at async eval (/Users/yaroslav/Documents/Work/remix-papi/app/routes/_index.tsx:6:31)
      at async instantiateModule (file:///Users/yaroslav/Documents/Work/remix-papi/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:52961:5)

Error for node provider:

Same steps but provider is for node

import {getWsProvider} from "polkadot-api/ws-provider/node";

Node WS provider throws error:

Hi! Thanks for the report! This is indeed an issue with SSR using the same code for server (nodeJS) and web bundles, and WebSocket not being implemented natively for NodeJS <22.
There are two solutions to this problem.

Use nodeJS 22, and import from the web export. It'll work out of the box. From that version on, NodeJS implements the WebSocket natively. NodeJS 22 will become LTS in 2 weeks, so it wouldn't be that much of a problem.
If you want to use NodeJS 20 (or anCreate a provider.ts file with the following code. vite will take care of it and strip the unused branches, so no increase in bundles 👍🏻

import { getWsProvider as serverProvider } from "polkadot-api/ws-provider/node";
import { getWsProvider as clientProvider } from "polkadot-api/ws-provider/web";

// eslint-disable-next-line @typescript-eslint/no-explicit-any
export const getWsProvider: typeof clientProvider = (...args: [any]) => {
  if (import.meta.env.SSR) return serverProvider(...args);
  else return clientProvider(...args);
};

And when creating the client, instead of importing getWsProvider from our export, just import it from that file.
Feel free to comment anything else and we will reopen, I think this should solve it!


pnpm7 filter by directory issue

Same as #1920, but with pnpm 7
ok looks like with yarn3 it needs single quotation marks in when defining script --filter='./apps/*'
Originally posted by @gbkwiatt in https://github.com/vercel/turbo/issues/1920#issuecomment-1243856267

@AntoineKM Does the quoting the filter not work here? Are there docs that need updating?
If you could provide the error message you're getting that would be helpful

Hi @chris-olszewski, thanks for the answer, I will give you more details but yes I think the doc needs a little update!
I just followed the following instructions on the kitchen repo (https://github.com/tonightpass/kitchen):
[Image]
https://turbo.build/repo/docs/core-concepts/monorepos/filtering#filter-by-directory

First try without the quotes (fail):
Input:
turbo run build --filter=./examples/*
Output:
...\tonightpass\kitchen> pnpm build:examples

> @tonightpass/root@ build:examples ...\tonightpass\kitchen
> turbo run build --filter=./examples/*

No matches found: "--filter=./examples/*"
 ELIFECYCLE  Command failed with exit code 1.

Second try with quotes (work):
Input:
turbo run build --filter='./examples/*'
Output:
...\tonightpass\kitchen> pnpm build:examples

> @tonightpass/root@ build:examples ...\tonightpass\kitchen
> turbo run build --filter='./examples/*'

• Packages in scope: expo-typescript, next-typescript
• Running build in 2 packages
• Remote caching disabled

...classic logs...

 Tasks:    2 successful, 2 total
Cached:    0 cached, 2 total
  Time:    1m45.257s

⠏ ...writing to cache...  [3s]

but before using pnpm on kitchen I was using yarn and I think this command was working without quotes (to be checked by your side, and thanks again for your attention on this issue)

Thanks for reporting this. Did a quick check and this is pnpm intercepting the --filter argument before it gets sent to us. I'll update the docs to quote the filter to avoid future confusion.


I want to install and run two instances of this project on Debian 11, but it seems that only one can run at a time. Does anyone know what to do?

I want to install and run two instances of this project on Debian 11 in two separate directories, but it seems that only one can be run at a time. Does anyone know what to do?

I suspect it's either the duplicated container name or port used.


Add ProgrammingExercise by ProjectKey endpoint

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[ ] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) and too complex database calls.
[ ] I strictly followed the principle of data economy for all database calls.
[ ] I strictly followed the server coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[ ] I documented the Java code using JavaDoc style.

Client

[ ] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
[ ] I strictly followed the principle of data economy for all client-server REST calls.
[ ] I strictly followed the client coding and design guidelines.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).
[ ] I tested all changes and their related features with all corresponding user types on a test server configured with Gitlab and Jenkins.

Description
Adds an endpoint that fetches a programming exercise by its projectKey
Steps for Testing

Either code reviews or testing with Postman
Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Performance Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Summary by CodeRabbit

New Features

Introduced a new endpoint to query programming exercises by project key, enhancing access to specific exercises for users with student roles.

Bug Fixes

Improved error handling by returning a 404 Not Found response when a programming exercise is not found.

@EnforceAtLeastStudent

should be replaced by AtLeastStudentInExercise but this has to be implemented without the macro, because no exerciseID is passed

Looks good, but you also should add some simple server tests.

good point thanks


Typecasting / Searching Non-Text (sort of) Columns

I use the active record extension to save IP Objects as text strings, however they are encoded, so are not available to search (for example an IP Address of <IP_ADDRESS> is held in the database as 4^0a01010a
The ActiveRecord process serializes them to or from the database, however, I cannot search for 10.1.1  as the representation is 0a0101 (in that case). This creates an IP Object, which has the to_s to turn it back to a string for display purposes
Is it possible to either use any ActiveRecord serializer functionailty, or  specify the CAST or retrieval method for the field rather than going 'direct' to the database

@msimkins you can override the following methods inside your datatable class to achieve the desired functionality:

search_condition method (defined here: https://github.com/antillas21/ajax-datatables-rails/blob/master/lib/ajax-datatables-rails/base.rb#L116-L123), you can take a look at how the final search condition is built at the new_search_condition method if you're using the latest version (0.3.0) or deprecated_search_condition if you're using any other version. Actually, I just noticed I forgot to refactor this, :facepalm:
the search_condition method, (by means of passing the bill to any of the other two methods listed above) uses the typecast method (defined here: https://github.com/antillas21/ajax-datatables-rails/blob/master/lib/ajax-datatables-rails/base.rb#L148-L154), so you have two points where you can introduce your changes.

If you go a full ActiveRecord way, perhaps you can lean on Arel (as we currently do), just making a prevision for an edge case.
Looking forward to how you address this, and if you'd be interested in contributing back your solution :smile:


Space in Payload is getting replaced with +

Hi wagiro,
Space in Payload is getting replaced with +
Please can you suggest how we can prevent it. We need Space in payload.
Thanks in advance.
Thanks and Regards,
Karibasavaraj K

Hi @havocuser!
You can replace space with whatever you want on "Match and replace" feature. For example, replace space by %20, etc
Regards.

Hi wagiro,
Tried the same.
We are not able to get space in payload. But we get %20 characters in payload.
Please can you suggest.
Thanks in advance.
Thanks and Regards,
Karibasavaraj K


Component for token gating via POAP

Description
This is a simple drop-in component for potentially "token gating" features within the app behind a POAP NFT (https://poap.xyz/), similar to ConnectWalletToast
The idea is that we can have certain features on the development server that are available early for engaged community members, to increase community engagement.
Working with the community / marketing folks, we'll have our own POAP that can be awarded during community calls, on site conferences, etc.
The purpose of this PR is to open discussion about this idea.  If anyone has other comments, questions, or concerns with this approach, feel free to bring them up here!
Notes

Please test with a wallet that both has the POAP and does not have the POAP.
The Sign up button will link to a typeform or Google form to allow users to input an email or wallet address to get into the list of folks we award the POAP.

Issue (if applicable)
N/A
Testing
The POAP used in this PR my own POAP. Single use mint links here:

http://POAP.xyz/claim/3ym2a4
http://POAP.xyz/claim/k1n0az
http://POAP.xyz/claim/pdwgzb
http://POAP.xyz/claim/0ofu9c
http://POAP.xyz/claim/v67sx6

If we decide to go ahead with implementing this in some way, we will work with design to get a proper POAP image and copy.

Closing this PR and will discuss new features with product / design / marketing going forward.


Expose the state of the system on core crashes

This is independent of the other issues, and is a UI feature request.
It seems like a lot of user confusion comes up because you aren't following good design practices, namely exposing the state of the system. If a process crashes and you still expose all features in the UI that link into that process, you're intentionally harming the user experience and the perceived credibility of the product. If I click on something and nothing happens, it's really bad. If I can't click on it because it's grayed out with the error The JavaScript engine has crashed, it's still bad, but now I know what's going on, and I have more confidence in the software. Right now my mental model of vscode is...well...not great, because about half of the UI doesn't work and I don't know why. At least if I know it's from a process crash I can say "oh, it's not that 50% of this editor doesn't work, it's that one core process doesn't" which gives me more faith in the system as a whole.

@DelvarWorld could you please be more specific as to what is not working for you. I'm asking for real examples and reproducible steps here. Thanks.

Can't you just induce a crash manually? Either way take your pick: https://github.com/Microsoft/vscode/issues?utf8=✓&q=author%3ADelvarWorld

@weinand the current issue we are tracking down is #2054.

@DelvarWorld we are moving from running language services in web workers into separate processes. I've checked and there we indicate the state when the server crashes to the user (also support to restart).
Given the move of the JS webworker into a server, we will no longer invest a lot into the webworker based language services.

@egamma since you seem to oversee the individual issues behind this feature request, I'll assign it to you.

More can be done on this issue, but I close this issue for now.


Mixins are not being generated

Hi,
I'm opening this issue because I am unable to use your (awesome!) tool. I also want to ask some questions regarding the behavior of the generator.
My situation
I have a Flutter Project with flavors. Here's my project structure.
- lib
  - init.dart  // here's the actual runApp(...);
  - main_development.dart  // here' a void main() {...}
  - main_staging.dart  // here' a void main() {...}
  - main_production.dart  // here' a void main() {...}

 All they do is, then, call init() inside init.dart. Formally, that's my true entry point.
The documentation says that the build file should be configured with one or more entry points; now, I am unsure of what to write in there, but since all the above files contain a main I'm currently structuring my build.yaml file like the following:
targets:
  $default:
    builders:
      dart_mappable_builder:
        generate_for:
          - lib/main_development.dart
          - lib/main_staging.dart
          - lib/main_production.dart

Note. The bug I'm about to show happens even if I configure just init.dart as an entry point.
The bug
While the files are being generated, no mixins are generated when I try to use an abstract interface and its subclasses with mappable. I followed (I think closely) both the documentation and the example in your repo (the "polymorphic" one).
My project is complicated, but let's say I have the following two classes:
// base.dart
import 'package:dart_mappable/dart_mappable.dart';

import '../../main_development.mapper.g.dart';

@MappableClass()
abstract class Base with BaseMixin, Mappable {
  const Base();

  String get id;
}

and
// a.dart
import 'package:dart_mappable/dart_mappable.dart';

// ignore: unused_import
import '../../main_development.mapper.g.dart';
import 'base.dart';

@MappableClass()
class A extends Base with AMixin {
  const A({required this.id});

  @override
  final String id;
}

The compiler indeed complains that no AMixin nor BaseMixin are present in the generated file. Why? What am I missing here?
Two Questions
Also, I wanted to ask:

why does this package generate a single big singleton file? Is there any design choice? I am used to Freezed, in which the files are directly generated in the same folder (or it is configurable) of the blueprints
why should there be an "entry point" explicitly declared? What is that for? I just need some models to be generated, why should my main file be involved?

Thanks in advance.

What exact version are you using? There are a lot of changes regarding this in the current prerelease.
The single entry point is more a necessity than a choice caused by how the package has to work wir more complex class structures. The only impoetant thing for the entry-point is that it (transitively) imports all your annotated models, it does not have to be the main file. I tried to get around this but haven't found the right solution for it yet.

The version being used is the latest marked as stable.
dependencies:
  dart_mappable: ^1.2.0

dev_dependencies:
  dart_mappable_builder: ^1.2.0

The only important thing for the entry-point is that it (transitively) imports all your annotated models

Ok, therefore my init.dart will do, as everything begins there - practically speaking. Before init(), there's just a simple assignment.
// main_flavor1.dart
import 'flavors.dart';
import 'init.dart';

void main() {
  flavor = Flavor.dev;
  init();
}

And then my init.dart file is:
import 'package:flutter/material.dart';
import 'package:hooks_riverpod/hooks_riverpod.dart';  // not related, but it's just there

import 'app.dart';

Future<void> init() async {
  runApp(const ProviderScope(child: App()));
}

Now, my build.yaml has been changed like so:
targets:
  $default:
    builders:
      dart_mappable_builder:
        generate_for:
          - lib/init.dart

No Mappers are generated at all. I just tried to recreate the problem, let me update a repo I've been using for these experiments.

@schultek
Here's the repo with the reproducible problem.

In this code the main does not import your models. At least a and b have to be imported for the generation to work.

Hold on, does this mean that in init.dart I have to import all the generated files, even though I don't use em there?
Or did I misunderstood this package usage, namely should I give an entry point (in build.yaml) for each model I create (e.g. in that repo define lib/models/base.dart` as an entry point)?

I updated the repo with your suggestion, now I am using A and B in my main.dart file, but everything's basically broken in the  generated file. No errors or warnings are shown in build_runner logs.

So yes, the entrypoint you specify in build.yaml should import all models that you want to use with the package. But the important bit is that transitive imports are also fine, meaning if main imports x, x import y and y imports your model that also works.
But in practice this shouldn't be a limitation or require extra setup, since you probably use your models somewhere in your app, so all models are sooner or later imported by the app entrypoint. Therefore your build entrypoint often is just your main.dart.
Hope that clears things up.

Yes, that clarifies it, thanks. In a real scenario, even if I'm just testing something out, that something will be used somewhere in my application.
Still, after importing the classes, everything's broke now (see last comment). The codegen tool doesn't behave.

First, there are no mixins generated for each class (AMixin, BaseMixin), only the general Mappable mixin.
More importantly, the error here is that the package internally has a BaseMapper class which conflicts with the Base model when generating the code. This is a bug of the package I didn't notice before. It basically means that currently there can not be a model named Base.

Thanks for your assistance. That's great to know 👍🏼 I won't call my classes Base in a real-world scenario, so I guess it's not that much of a problem.
I am including AMixin and MyBaseMixin (I changed the name of the class, now) to follow up with your example.

The repository on github currently is the prerelease, not the stable version from pub.
The stable version can be found here:  https://github.com/schultek/dart_mappable/tree/1b001716a8886de394cd0d427982537d375376e6

I see. so the 2.0 version will have a slight change on the APIs.
Thank you again for your time. Should I leave this open to track that small bug about Base?

No problem, I will close this then.
Also thanks for the sponsorship! 💙

You're solving a problem Freezed isn't able to solve right now. You 110% deserve this (I'm looking forward for QOL updates 👀).
Good luck!


Broken link on page

Hello,
please note that the link in this sentence is broken: "The ability to read secrets from the KeyVault, see [Service Principal] (#create-service-principal-and-security-group) section for more details."
Kind regards,
Anne H., QC

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

ID: 5054258a-0c6b-9ac9-15d1-2bfdc2cbd716
Version Independent ID: 605a6e2d-1429-343d-a31e-53d1358573ff
Content: Azure Operator Nexus: How to run Instance Readiness Testing
Content Source: articles/operator-nexus/howto-run-instance-readiness-testing.md
Service: azure-operator-nexus
GitHub Login: @lesage-oded
Microsoft Alias: odedlesage

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

This has been fixed in https://github.com/MicrosoftDocs/azure-docs-pr/pull/273586

#sign-off
#please-close

@ManoharLakkoju-MSFT this issue has been resolved, not sure how to close it

@QueryCoordinator
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


[BUG] couriers list page got blank

Describe the bug
got blank due to avatar data is not consistent
To Reproduce
Steps to reproduce the behavior:

Go to 'https://example.refine.dev/couriers?current=1&pageSize=10&sort[]=id&order[]=desc'
see an error

Expected behavior
better handling when showing avatar data

Thank you! @sprabowo


ashuffle: 3.12.3 -> 3.12.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/joshkunz/ashuffle/releases.
meta.description for ashuffle is: "Automatic library-wide shuffle for mpd"
meta.homepage for ashuffle is: "https://github.com/joshkunz/ashuffle"
meta.changelog for ashuffle is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/r5x75jdsamg8r5n126fv31iqp7g1lpbd-ashuffle-3.12.5/bin/ashuffle passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 3.12.5 with grep in /nix/store/r5x75jdsamg8r5n126fv31iqp7g1lpbd-ashuffle-3.12.5
found 3.12.5 in filename of file in /nix/store/r5x75jdsamg8r5n126fv31iqp7g1lpbd-ashuffle-3.12.5

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ashuffle
docbookrx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r5x75jdsamg8r5n126fv31iqp7g1lpbd-ashuffle-3.12.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ashuffle https://github.com/r-ryantm/nixpkgs/archive/e92dec78b35074df4d489aa64135b7403879262b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r5x75jdsamg8r5n126fv31iqp7g1lpbd-ashuffle-3.12.5
ls -la /nix/store/r5x75jdsamg8r5n126fv31iqp7g1lpbd-ashuffle-3.12.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ashuffle

Maintainer pings
cc @tcbravo for testing.

Result of nixpkgs-review pr 148810 run on x86_64-linux 1

1 package built:

ashuffle


[vcpkg-tool] Allow overriding FETCHCONTENT_FULLY_DISCONNECTED

Describe the pull request

What does your PR fix?
Fixes #28386
Alternative to #28483

We can't simply remove FETCHCONTENT_FULLY_DISCONNECTED because this would lead to bugs for users that want to build on isolated machines. Therefore, we have to set this variable in vcpkg_configure_cmake() as well. Since using vcpkg_configure_cmake() is deprecated, it shouldn't break ports in private registries. If a user still needs FETCHCONTENT_FULLY_DISCONNECTED set to OFF, they can override it via OPTIONS.

While vcpkg_configure_cmake() is deprecated, that does not mean we can break existing ports. We must take #28483, which reverts the change only for vcpkg_configure_cmake().
However, we absolutely should support the override for vcpkg_cmake_configure() as you've done in this PR. I think @dg0yt's suggested improvement is best -- we need to move the vcpkg "preset" options above the user's supplied OPTIONS.

VCPKG_CMAKE_CONFIGURE_OPTIONS is designed to override all settings. So I don't get why this is even needed.

VCPKG_CMAKE_CONFIGURE_OPTIONS is designed to override all settings. So I don't get why this is even needed.

IMO it is generally bad to override anything which is already passed explicitly in OPTIONS. Because this is the primary interface for this purpose.
VCPKG_CMAKE_CONFIGURE_OPTIONS is nice to have for use in triplet files.

We must take #28483, which reverts the change only for vcpkg_configure_cmake().

Why do we have to take #28483 before merging this PR? This would introduce a bug (see PR description above) until this PR gets merged.

@Thomas1664 Could you please handle the conflicts?

Thanks for your fix :). I merged Robert's change which had only baseline issues, merged that with this, and committed (as this has a green run).

Thanks for your fix :). I merged Robert's change which had only baseline issues, merged that with this, and committed (as this has a green run).

@BillyONeal Now FETCHCONTENT_FULLY_DISCONNECTED is not enabled in vcpkg_configure_cmake. What's wrong with setting it on this one as well? one can override it just like with vcpkg_cmake_configure.

What's wrong with setting it on this one as well?

It breaks backwards compatibility with old port versions.


Missing required dependency in setup.py

The dependency psutil is missing from the list of required installs in setup.py causing it to throw an exception when heiko is installed/built with pip install . or pip install heiko.

I can fix this


SAA-303 refactoring schedule and slot creation and also some other minor refactorings.

Codecov Report
Base: 68.03% // Head: 69.14% // Increases project coverage by +1.11% :tada:

Coverage data is based on head (43e1360) compared to base (85c4c21).
Patch coverage: 91.23% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #159      +/-   ##
============================================
+ Coverage     68.03%   69.14%   +1.11%
- Complexity      633      675      +42
============================================
  Files           140      142       +2
  Lines          3103     3260     +157
  Branches        109      120      +11
============================================
+ Hits           2111     2254     +143
- Misses          927      930       +3
- Partials         65       76      +11

Impacted Files
Coverage Δ

...gementapi/entity/ActivityScheduleEntityListener.kt
0.00% <0.00%> (-27.28%)
:arrow_down:

...sactivitiesmanagementapi/entity/DailyStatistics.kt
0.00% <0.00%> (ø)

...ppsactivitiesmanagementapi/entity/EventConsumed.kt
0.00% <0.00%> (ø)

...sactivitiesmanagementapi/entity/PrisonerHistory.kt
0.00% <0.00%> (ø)

...hmppsactivitiesmanagementapi/entity/ActivityPay.kt
61.11% <50.00%> (ø)

...mentapi/service/ActivityScheduleCreationService.kt
81.39% <81.39%> (ø)

...activitiesmanagementapi/service/ActivityService.kt
88.88% <87.50%> (-3.42%)
:arrow_down:

...vitiesmanagementapi/entity/ActivityScheduleSlot.kt
89.47% <87.87%> (+6.61%)
:arrow_up:

...activitiesmanagementapi/util/TransformFunctions.kt
76.32% <94.44%> (+0.06%)
:arrow_up:

...ps/hmppsactivitiesmanagementapi/entity/Activity.kt
87.36% <94.73%> (+4.55%)
:arrow_up:

... and 31 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


PMM-11323 added conf file and meta_metrics tests

mysql conf file: https://github.com/Percona-Lab/pmm-submodules/actions/runs/4473983285

mysql conf file: https://github.com/Percona-Lab/pmm-submodules/actions/runs/4473983285
mongo telemetry: https://pmm.cd.percona.com/job/pmm2-ui-tests/31380/

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests-matrix/detail/pmm2-ui-tests-matrix/1878/pipeline/26/
Includes failed test which has no changes at all ... should not be impacted
https://pmm.cd.percona.com/job/pmm2-ui-tests/31435/testReport/(root)/MongoDB Collectors Parameters and Flags tests/PMM_T1280_Verify_that_pmm_admin_inventory_add_agent_mongodb_exporter_with___log_level_flag_adds_MongoDB_exporter_with_corresponding_log_levelPMM_T1282__PMM_T1284__PMM_T1291_Verify_that_pmm_admin_inventory_add_agent_node_exporter_with___log_level_flag_adds_Node_exporter_with_corresponding_log_level__not_ui_pipeline__mongodb_exporter__exporters/

https://pmm.cd.percona.com/job/pmm2-ui-tests-matrix/1887/


Remove feature flag

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
2021.07
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
2021.07
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[x] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@ruowan Remove the properties due to the changes for onboarding diagnostic logs. It's about to be used. but monitoring team ask us to change the diagnostic log config , then config diagnostic log config need to change and cause the properties get unused. I can comfirm that no customer depends on the properties. And the service is public preview, I wonder if I can make the change in the existing api version?

Assign to oncall reviewer @zhenglaizhang

@ruowan Removing the properties due to the changes for onboarding diagnostic logs. It's about to be used. but monitoring team ask us to change the diagnostic log config , then config diagnostic log config need to change and cause the properties get unused. I can comfirm that no customer depends on the properties. And the service is public preview, I wonder if I can make the change in the existing api version?

Hi @wanlwanl you need to go through the breaking change review process to get approval by following https://github.com/Azure/azure-rest-api-specs/pull/15105#issuecomment-875388869.
And as you are not adding new API version but just remove some properties, so there is no need to do ARM review, So I just removed the WaitForARMFeedback label

@ArcturusZhang pls help check go SDK breaking change.
@msyyc pls help check python SDK breaking change.

I will approve after the breaking change review board does.

I create another one by adding new api version for breaking change https://github.com/Azure/azure-rest-api-specs/pull/15406


Terra autostake broken on newer versions

On the newer versions of the autostake script it says: No Authz support
Even though it has been working and still works on older versions,
I think because these lines from cosmos-directory chains/chainMonitor.js
      try { await got.get(restUrl + 'cosmos/authz/v1beta1/grants') } catch (error) {
        if (error.response.statusCode === 400) {
          data.authz = true
        } else if(error.response.statusCode === 501) {
          data.authz = false
        }
      }

Don't account for when public endpoints are unavailable?
Maybe we could add an override here in restake in the src/networks.local.json?

Sorry about that - will have been broken since last night. cosmos.directory now provides the Authz status as you saw, and because Terra RPC have been down it hasn't had a chance to set it to true. I've just manually updated so it'll solve your local restaking.

Thank you!


Nested TabControl raises SelectionChanged event in a wrong tab control

Describe the bug
Nested TabControl raises a SelectionChanged event in wrong tab control
To Reproduce
Steps to reproduce the behavior:
Details.

Create two tab controls:

Parent tab control with SelectionChanged event handler.
A child tab control inside the parent tab control item. The child tab control has no event handlers.

Switch tab in the child tab control

Expected behavior
Nothing will happen because child tab control has not event handlers
Actual behavior
The parent tab control event handler gets fired.
Screenshots

https://user-images.githubusercontent.com/618115/159928506-77870e54-3643-4b1a-bac9-9a03758d602b.mp4
Desktop (please complete the following information):

OS: Mac
Version 0.10.13

The problem is likely because the SelectionChanged event in SelectingItemsControl is a routed event that is bubbling up from the child into the parent. Since this event is literally the exact same event it will appear as if the parent is actually raising it when in fact it is bubbling from the child.
Can you show the code on how you are subscribing to the event just to make sure there is no confusion? It might also help to look at this in DevTools.
Otherwise, I think someone on the core team needs to reply on how to fix this. To me it looks architecture related.

I already shared link with a sample project with the core team.

Can you show the code on how you are subscribing to the event just to make sure there is no confusion?

Yes, sure.
MainWindow.xaml
<Window xmlns="https://github.com/avaloniaui" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" x:Class="AvaloniaTest.MainWindow" Title="AvaloniaTest">
      <DockPanel>
        <Button Content="ShowDialog" Click="OnShowDialog"/>
        <TabControl x:Name="TabControl" />
      </DockPanel>
</Window>

MainWindow.cs
    public partial class MainWindow : Window
    {
        private TabControl _tabControl;
        public MainWindow()
        {
            InitializeComponent();

            _tabControl = this.FindControl<TabControl>("TabControl");
            _tabControl.SelectionChanged += _parentTabControl_SelectionChanged; // here is the subscription point

            var tabItem = new TabItem();
            tabItem.Header = "Item1";
            var ctrl = new MyUserControl();
            tabItem.Content = ctrl;
            _tabControl.Items = new [] { tabItem, new TabItem { Header = "Title2"} };
        }

        private async void _parentTabControl_SelectionChanged(object sender, SelectionChangedEventArgs e)
        {
            var state = e.Source == sender ? "🟢" : "🔴";
            Console.WriteLine($"{DateTime.Now} {state} _parentTabControl_SelectionChanged");
        }
...
    }

MyUserControl.xaml which contains the nested tab control which doesn't have a subscription
<UserControl xmlns="https://github.com/avaloniaui"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:avaloniaTest="clr-namespace:AvaloniaTest"
             x:Class="AvaloniaTest.MyUserControl">
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="*" MinHeight="110"/>
            <RowDefinition Height="1"/>
            <RowDefinition Height="*" MinHeight="110"/>
            <RowDefinition Height="*" MinHeight="110"/>
        </Grid.RowDefinitions>

      ...

        <TabControl Grid.Row="3" x:Name="TabControlWithoutEventHandler">
          <TabItem Header="TabItemWithoutEventHandler1">
            <TextBlock Text="Switching that tab must not fire any event"/>
          </TabItem>
          <TabItem Header="TabItemWithoutEventHandler1">
            <TextBlock Text="Switching that tab must not fire any event"/>
          </TabItem>
        </TabControl>
    </Grid>
</UserControl>

MyUserControl.cs
public partial class MyUserControl : UserControl
{
    public MyUserControl()
    {
        InitializeComponent();
    }

    private void InitializeComponent()
    {
        AvaloniaXamlLoader.Load(this);
    }
}


Merged reads restricted to length of the read pairs?

Am i missing something obvious? For example if I have the following read configurations across a 400 bp fragment:
1-------------------------------400 : insert fragment

1-------------------------250             : read 1 (fwd)
                 150-------------------400 : read 2 (rev)

where running fastp --merged_out I get fastq records ONLY spanning positions 1-250. Is this the intended behavior? Is there a way to output the complete 400 bp span where obviously the middle 150-250 contain corrections and the flanks are the single read information?
In the case above an example command is:
fastp -i R1.fq.gz -I R2.fq.gz --merged_out merged.fq --length_required 100 --average_qual 30 --adapter_fasta adapters.fa --merge --overlap_len_require 75 --overlap_diff_limit 30 --overlap_diff_percent_limit 50

Hey I see similar things. Wondering if you got a response @robinfchan ?
I get output reads with merged_150_0 in the header. Which means the reads were not merged I guess?
I can't figure out a reason why the sequence of the merged read is missing, it is present when merging with flash.
I disabled any quality filtering etc.
fastp --in1 test_PE_R1.fastq.gz --in2 test_PE_R2.fastq.gz --merge --qualified_quality_phred 1 --disable_trim_poly_g --disable_adapter_trimming --disable_quality_filtering --disable_length_filtering --dont_eval_duplication --out1 fastp.notCombined_1.fastq.gz --out2 fastp.notCombined_2.fastq.gz --merged_out fastp.merged.fastq.gz --overlap_len_require 10
Example input reads:
test_PE_R1.fastq.gz
@A00121:521:H5CFGDSX3:2:2150:11550:26475 1:N:0:CGCTCATT
GCGGATCCTGACCATGTACGATTGACTATATCTTGGAAAGTTGGAAGCTTCGCAGGCATGGATCCTGACCATGTACGATTGACTAGTGCGTTCTTACCTTGCTACTTTCTAAGTTTCCAACAAGCTTTCGTTCGATGCTGCAACGTGCTA
+
FFFFFFFFFFFFFFFFFFFFF:FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF:F,F:FFFF::F:FFFFFFFFFF,FFFF,F:FFFFFFFF,FFF:FF,FFFFFF:F:FFFFFFF:FFFFF:FFFFF:FFF,FFFFFFFFF:

test_PE_R2.fastq.gz
@A00121:521:H5CFGDSX3:2:2150:11550:26475 2:N:0:CGCTCATT
CCTAGTCAGTACGCATTAGCACGTTGCAGCATCGAACGAAAGCTTGTTGGAAACTTAGAAAGTAGCAAGGTAAGAACGCACTAGTCAATCGTACATGGTCAGGATCCATGCCTGCGAAGCTTCCAACTTTCCAAGATATAGTCAATCGTA
+
FFFFFFF:FFFFFFFFFFFFFFFFFFFFF,:FFFFFFF,F,FFFFFFF:,FFFF:FFFFF,F:,:F,FFFF:FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF:F:FFFFFFFFFF,FFFFFFFFFFFFFFFF:,FFFF,FFF::F

output of fastp:
@A00121:521:H5CFGDSX3:2:2150:11550:26475 1:N:0:CGCTCATT merged_150_0
GCGGATCCTGACCATGTACGATTGACTATATCTTGGAAAGTTGGAAGCTTCGCAGGCATGGATCCTGACCATGTACGATTGACTAGTGCGTTCTTACCTTGCTACTATCTAAGTTTCCAACAAGCTTTCGTTCGATGCTTCAACGTGCTA
+
FFFFFFFFFFFFFFFFFFFFF:FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF:F,F:FFFF::F:FFFFFFFFFF,FFFF,F:FFFFFFFF,FFF:FFFFFFFFF:F:FFFFFFF:FFFFF:FFFFF:FFFFFFFFFFFFF:

Output of flash:
@A00121:521:H5CFGDSX3:2:2150:11550:26475 1:N:0:CGCTCATT
GCGGATCCTGACCATGTACGATTGACTATATCTTGGAAAGTTGGAAGCTTCGCAGGCATGGATCCTGACCATGTACGATTGACTAGTGCGTTCTTACCTTGCTACTTTCTAAGTTTCCAACAAGCTTTCGTTCGATGCTGCAACGTGCTAATGCGTACTGACTAGG
+
FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF,FFF:FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF:FFFFFFFFFFFFFFFFFFFFFF:FFFFFFF

@HenriettaHolze -- no you're the only reply haha
We ended up just using fastp for trimming and filtering and sticking with flash for PE merging for the time being


⚠ Bofrost.be has degraded performance

In 1e85292, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1825 ms

Resolved: Bofrost.be performance has improved in 3263f04 after 304 days, 3 hours, 10 minutes.


Release 1.10

Hi Team
Forgive me if I am jumping in out of turn here but I am making extensive use of this library at the moment.
The Async connect in the latest release is super useful so thanks!
I picked up that the exponential backoff params are still being set:
// See https://cloud.google.com/iot/docs/how-tos/exponential-backoff
    if (this->__backoff__ < this->__minbackoff__) {
      this->__backoff__ = this->__minbackoff__;
    }
    this->__backoff__ = (this->__backoff__ * this->__factor__) + random(this->__jitter__);
    if (this->__backoff__ > this->__max_backoff__) {
      this->__backoff__ = this->__max_backoff__;
    }

Is there a reason why as they do not seem(?) to be used when using the async connect.
I have wrapped this and implemented my own exponential backoff. It may be nice to integrate something similar:

    // Do nothing if already connected.
    if (mqtt_client_->connected()) {
        return;
    }

    // Init the backoff index.
    static int backoff_index = 0;
    // Init the last connect time.
    static unsigned long last_connect_t_ms = 0;
    // Init backoff delay.
    static uint32_t backoff_ms = 0;

    // Return if delay has not expired.
    if ((millis() - last_connect_t_ms) < backoff_ms) {
        return;
    }

    if(backoff_index > CONNECT_CERT_SWAP_BACKOFF_COUNT){
        log_w("Switch to primary/backup root certificate.");
        net_client_->setCACert(BACKUP_ROOT_CERT);
        // TODO save cert option to preferences.
    }

    // Set the connect ping.
    last_connect_t_ms = millis();

    // Attempt connect.
    mqtt_->mqttConnectAsync();

    // If connection fails delay before trying again.
    if (mqtt_client_->connected()) {

        // Reset the backoff index.
        backoff_index = 0;
        // Reset the last ping time.
        last_connect_t_ms = 0;
        // Reset backoff delay.
        backoff_ms = 0;

    } else {

        // Compute the backoff delay.
        backoff_ms = min(int(pow(2, backoff_index++)) * 1000 + int(random(1000)), CONNECT_MAX_BACKOFF_MS);

        // Log the delay.
        log_w("Failed to connect google. Trying again after %dms", backoff_ms);
    }

I have also started work on the certificate switchover to the backup. A feature request I added a ticket for.


Install dir is incorrect for ARM builds

Install dir is incorrect for ARM builds. Builds are put in x86 dir instead of ARM. The x86 dir in the image below should be ARM.

I've updated master-merge-cmake with a fix. However mind that observed behaviour is actually expected per existing OpenCV logic. Please see Overview and Summary for details.
Overview
The default behaviour for 'stock' configuration is to put binaries into
   install/${OpenCV_ARCH}/${OpenCV_RUNTIME}/

which  for example turns out to be
 "x86/vc12/"

for the case that you're

building with x86 CMake binary
using "VS 12 2013" as generator.

I'm passing an additional parameter from the build script and using it to define the binaries install folder. For the following command to generate the project files
cmake -G "Visual Studio 12 2013" -DCMAKE_SYSTEM_NAME:String=WindowsPhone -DCMAKE_SYSTEM_VERSION:String=8.1 D:\projects\OpenCV\MSOpencv-master-merge-cmake -DCMAKE_VS_EFFECTIVE_PLATFORMS:String="ARM"

and after (1) building the project with BUILD_ALL and (2) building INSTALL sub-project you'd get binaries in
   "install\ARM"

Considerations
My concern is that this changes the default approach to installing the binaries that OCV devs are using and potentially break wiring for existing projects if they depend on these binaries when building for platforms other that WinRT.
Mind that the origin for this issue is that currently official CMake does not allow generating a single project for several architectures. This is the only reason we have to have extra directory level, it shouldn't be there in the first place. Instead  we should have been switching it from within IDE.

On a separate note there was an issue for MS CMake that has been resolved and allowed multiarchitecture generation. You can even try it out (binaries are referenced in the ticket). However this has not yet been contributed to official CMake codebase and is not available with the latest ditsribution.

Summary
I see three possible solutions:

Use the scheme above for winrt only
Make paths look like  "install/x86/vc12/arm
Leave it as it was before my latest change

I'd vote for (3) since:

The observed behaviour is actually expected per existing OpenCV logic
(1) and (2) will both break existing 'stock' OCV pattern with (1) being a little less destructive compared to (2)
The only reason we want this change now is to change the way sample apps that we provide reference these libraries and sources. There is no obligation or convention on the location of the binaries we must follow here. Morever this will require developer to make an additional step by first building the solution with BUILD_ALL and then with INSTALL.

CC Gilles @khouzam

@mkostin  If the generator is  "Visual Studio 12 2013 ARM" (2012) then the output dir should be ARM. If the generator is  "Visual Studio 12 2013" then the output dir should be x86.

I'm still looking at it and apparently my first conclusion is wrongIssue originates from OpenCV_ARCH being set wrong and current fix is valid. I'd only add OpenCV_Runtime part to make it fully compatible with vanilla pattern.

Found it, OpenCVConfig.cmake doesn't provide ARM option for architecture.

Can you update it?

I will, need to figure out value options for CMAKE_CL_64 first. Also I've found another place with identical code in OpenCVDetectCXXCompiler.cmake.

Ok, so CMAKE_CL_64 is obviously 0/false for ARM. The missing part is a way to detect current architecture is ARM. The cleanes path would be using a floag from CMake, otherwise stick to current solution with extra command line parameter. I'll investigate the first option.

Latest commit fixes the issue.

Referencing impovement commit: 6414980a6e1819a0e39b350ce9266cb7fbdcf337


[INFRA-1196] Add tests for update-site redirects

@rtyler, It is incomplete and new tests can be added but it is enough to reproduce the problem.

It seems that PR building is not configured so you will have to either trust me it works or grant me more permissions on ci.jenkins.io ...

Why would you need to remove readme and LICENSE?

Dammit, it should not have been filled against master - it needs to go to update-site branch. GitHub does not seem to permit PRs to branches that does not exist and I do not have the commit access to push directly.
@rtyler

INFRA-1196 FTR

A test like this will notify us of problems, but only after the problematic site is live. I think we should explore making the build for backend-update-center2 itself perform checks like this, so bad configurations never get deployed. Would I suppose mean running a test container with Apache and sending sample requests to it.

That would be better, no question about it. I lean toward getting this meged in the meantime ...

@rtyler, can we move on with this?


Add supported OCP version detection

The bundle-dockerfile script invocation was changed to match how the OCP
version info script is called.

LGTM


multisig 2-of-2, how to create a blank check transaction?

Hello,
I have a 2-of-2 multisig address and would like to create a blank check transaction with SIGHASH Single with one private key.
Is there anyone here who can tell me or guide me on how to create a a blank check with SIGHASH Single?

Any ideas?

You want to use SIGHASH_NONE mixed with SIGHASH_ANYONECANPAY

Create a transaction using SIGHASH_NONE and SIGHASH_ANYONECANPAY
I think Coinbin will break if you don't add an output, so just make a random one and send it all.
Sign it with 2 keys of 2.
Send the transaction to whoever you want to use it.
They will add any utxo they own with their key as the second input and change the output to send to their own address and account for fees and sign it using SIGHASH_ALL (the default normal one) (This can not be done in the coinbin UI currently, but if you modify the raw transaction manually, it should work.)
The final signed transaction will be one input from your 2of2 signed using NONE and ANYONECANPAY, and one input from the person claiming your check using ALL (default sighash) and sending the outputs to themselves. Then they broadcast it.

Note: If you broadcast the transaction on step 4 it will be valid, and if it is confirmed in a block it will be set in stone... but anyone who sees the transaction could change the outputs and it would still be valid. Though unless you opted in RBF it wouldn't propagate.

@dabura667 thx for the step by step description of how to do it.
But cannot I use SiGHASH single with a blank check instead ?
For example I want user1 sign it priv1key, the amount should be minimal since it will be a blank check. (Could the amount be set to zero here ? And the I use an address which the user 1 holds.
User2 will be the one who will add new outputs and sign  the RAW hex which is signed by user1.
User2 will sign it with SIGHASH ALL and will broadcast it to the network.
Please correct me if this is correct  way of thinking or if there is a better method to do this?
Regards