How do I retrieve password?

Hi,
I'm in a scenario where I need to display the password to the user. How do I achieve that? I found no matching option on the UserManager.

You don't. If you can, then it'd not be very secure, would it?

Please refer to the Password Storage Cheat Sheet.
Here's the quoted part you are interested in @weitzhandler :

Leverage an adaptive one-way function

Adaptive one-way functions compute a one-way (irreversible) transform
The rest is also very interesting to read if you are into this. I know @brockallen is :stuck_out_tongue:

Basic principle is... assume somebody is going to download your whole database and have all the time in the world to hack those passwords.
If you are using clear text... you are basically insane.  If you are using encryption with a single key, they only need this to get everything (bad!). If you use hashing, they have to try every password one by one to get them (better). If you use hashing with salting per accounts, then they can't match identical in your database (excellent!).  If you use hashing with a compute factor, then they will have an incredibly hard time finding individual passwords even if they use GPU farms to hack them (best so far!).
Your point shouldn't be in sending the user his password. It should be in providing him a way to reset is password with a link sent to a confirmed email address. No need to send passwords.
But basically... use something somebody else already did.
Don't roll your own

In my scenario, the server doesn't hold very secure information and the customers are senior people most of them without computers or internet access, and they're accessing the system via phone authentication (the password consists of digits only).
One of the requirements of my customer is to provide a back-office that gives complete control over the users and their passwords, so the secretary that manages the user base should be able to reveal users' passwords on demand.
Isn't there a possible way to reverse the password hashing and display it to authorized roles?

Why do you need to reveal the password? Is this so a back-office user can login as an end-user?

No, it's so that when the end user calls up the back office asking "hello what's my password" she can tell her the answer after verifying other identity details.
This is my customer's demand.

Well, that's a tough call how to really design it for that requirement and also have any sort of meaningful security.
To answer your original question, why not replace the password hasher with your own that does a Nop?

Or just have the lady up front reset the password to anything she wants. Unsecure as hell... but hey. If there's nothing to protect.

Yes, especially if they're just 4-digit pins -- that's easily brute forceable. A pin on the web is just keeping honest people honest.

It's not that easy, those people want to KNOW the password THEY have previously chosen and can't remember it.

I didn't actually think it's going to be a real issue, but it's my customer's demand to let the backoffice RETRIEVE the users password without having to reset it.
Isn't there a way to un-hash the password?

Isn't there a way to un-hash the password?

No, and that's by design. If you want it reversible, then you want to encrypt it (but then you have another problem, which is key management).

This is a BAD BAD BAD idea but @weitzhandler You could.... simply add a field to the database to store a copy of the password in however you want it to be. When its created/updated without changes to the identity model at all...  Or better yet explain to the customer that... While his/her company's data in this system does not need to be secured some reason. People often use the same password for other things and his/her practice is compromising them as well.

You can't. You won't ever be able to. We will not support this at all.

@blowdart
Thanks for the update.
I've notified my customer and we agreed there will be a single-click way for the back office to rest password for user.


Add criptio to the apps list

This has a conflict because of #245, could you merge/rebase and resolve it? Thanks!


[fix] (compaction) fix time series compaction policy #38220

Proposed changes
bp #38220

run buildall


How can I implement "Read more" feature for long text in Draftjs?

When the text is too long, I want display only first 30 characters and append a button "see more" into  drafts editor. When the button is clicked, full text is displayed.
Have anyone done that before? Please help. Thank you in advance!

As far as I understand you're using it in readOnly mode to display your content, am I right?
If that's the case: You can use EditorState.createWithContent({contentHere}) and update it when clicked view more to fetch the rest of the data and fill your component with it.

Thank @gabriel-miranda for the suggestion. You're right, I'm using Editor in readOnly mode. To be clearer, my question can be broken down into 2 parts:

How add "see more" link to the end of editor's content state in summary mode?
How to shorten the text while preserving styling?
E.g.
===Full text===
{"entityMap":{},
"blocks":[
{
"key":"vm8k",
"text":"123456789",
"type":"unsettled",
"depth":0,
"inlineStyleRanges":[
{"offset":0,"length":4,"style":"red"},
{"offset":4,"length":5,"style":"blue"}
],
"entityRanges":[],
"data":{}
}
]
}
===Summarized text===
{"entityMap":{},
"blocks":[
{
"key":"vm8k",
"text":"12345",
"type":"unsettled",
"depth":0,
"inlineStyleRanges":[
{"offset":0,"length":4,"style":"red"},
{"offset":4,"length":1,"style":"blue"}
],
"entityRanges":[],
"data":{}
}
]
}

Do you have any idea?

It seems like this isn't a Draft issue, but rather a View issue. I would figure out the space that your content can occupy, and apply a max-height to the editor, as well as a "See more" button with an absolute position to overlay the editor, and then when the "See more" button is clicked just remove the max-height

Looks like this has been resolved so I'm closing it out. Feel free to reopen if that's not the case.

It seems like this isn't a Draft issue, but rather a view issue. I would figure out the space that your content can occupy, and apply a max-height to the editor, as well as a "See more" button with an absolute position to overlay the editor, and then when the "See more" button is clicked just remove the max-height

But if you have an array of articles and only if an article exceeds the defined height you want "see more" to appear below it and toggle it. How can you calculate the height can decide whether it will exceed and should have "see more" or not even before it will render ?

The most logical approach would be the one which will be based on the text size/length of the draft editorState content. Is there any property to capture the size/length of the editorState


Determine how feasible it would be to automagically copy Rmd -> md between repos

Summary
Discussion on basecamp has some more context around this. The idea is we want leanpub and bookdown materials for a course to be separate. We could link out to the bookdown chapters in the leanpub repo, but this may be detrimental to user experience.
Alternatively, I'll look into how feasible/not janky of a solution we can find for copying over Rmd materials that have been changed in the bookdown repo to the leanpub repo in markdown format.
The basic level command might look something like below, where we save a markdown copy of a rendered Rmd. What's not reflected here is the repo transfer part which would need to happen by a github actions and have it file a PR so we don't accidentally overwrite changes without looking at it first.
rmarkdown::render("02-chapter_of_course.Rmd", output_format = "md_document", output_file = file.path("leanpub", "02-chapter_of_course.md"))

This command would obviously need to be run for each notebook so we can have one script that grabs all the Rmds, renders them and saves the markdown.
What I need to test is how well this renders in leanpub with Markua.

Okay. This command for render is a bit closer. It saves the markdown with images links as ![]() as well as keeping the images with it. In this scenario, I'm assuming manuscript folder is what we supply to Leanpub, but I need to have access to Leanpub before I am able to test this out.
rmarkdown::render("02-chapter_of_course.Rmd",
                                  intermediates_dir = "manuscript",
                                  clean = FALSE,
                                  output_format = rmarkdown::output_format(knitr = rmarkdown::knitr_options(opts_chunk = list(fig.path = "/resources/images/")), pandoc = rmarkdown::pandoc_options(to = "markdown")))

Spacing is wonky here.

Next step is figuring out a non-janky way to add in any needed Markua tags like: {width: "70.5%"} and as described in: https://leanpub.com/markua/read#resource-types-and-formats
I think the ones that are easiest to apply universally are the ones that adjust image width.
We can also keep in mind that with github actions plan, we would file a pull request with the conversion done from a bookdown repo, but ultimately we would have the ability to scan through the render and make any minor corrections needed. This would still be easier maintenance than trying to copy over text manually.

For the github actions part of this and copying between repos between these two examples we should be able to get it done:
https://github.com/marketplace/actions/create-pull-request
https://github.com/marketplace/actions/push-a-file-to-another-repository
I think ideally we'd only copy over newly changed files, so we also will need one bit of logic to only copy over those that have changed.
This also might be better served as a manually triggered github action. Say we have a group of changes we'd like to make, we don't need to push changes to Leanpub for every little thing, may be easier to wait and publish a release of sorts.

Next step is figuring out a non-janky way to add in any needed Markua tags like: {width: "70.5%"} and as described in: https://leanpub.com/markua/read#resource-types-and-formats
I think the ones that are easiest to apply universally are the ones that adjust image width.
We can also keep in mind that with github actions plan, we would file a pull request with the conversion done from a bookdown repo, but ultimately we would have the ability to scan through the render and make any minor corrections needed. This would still be easier maintenance than trying to copy over text manually.

Sometime soon @muschellij2 can show you what he did with this.

For the github actions part of this and copying between repos between these two examples we should be able to get it done:
https://github.com/marketplace/actions/create-pull-request
https://github.com/marketplace/actions/push-a-file-to-another-repository
I think ideally we'd only copy over newly changed files, so we also will need one bit of logic to only copy over those that have changed.
This also might be better served as a manually triggered github action. Say we have a group of changes we'd like to make, we don't need to push changes to Leanpub for every little thing, may be easier to wait and publish a release of sorts.
https://github.blog/changelog/2020-07-06-github-actions-manual-triggers-with-workflow_dispatch/#:~:text=You can now create workflows,the workflow is run on.

sounds good

So far with the merging of test PR #13 :

Code and preview of plot look good
The files converted from Rmd don't seem to have any glaring issues at least with the few things in the 02-chapter test. :+1:
I don't see any detectable difference between using the .knit.md or the .utf8.md at least in the context of my little test here.

What I haven't yet addressed is the other image issues (also discussed #9). That will require some tag additions
So far I think this is a feasible plan. I'm going to make up a draft of a markua-rendering script that actually runs the code mentioned in https://github.com/jhudsl/ITCR_Course_Template/issues/12#issuecomment-814281939
What needs to be figured out is how reliably we can add the Markua tags and get these images and videos to render nicely (I'll work on that next -- sounds like @muschellij2 may have done some work on this?). But as far as the rest of the Rmd -> md conversion, I think this is pretty safe to use.

See #19. The material in this convo is now mostly relevant to the Leanpub focused repo. So I'll transfer this issue over there.


Modify Ferret release process to include file browser

Reported by ansley b manke on 5 Jan 2011 20:07 UTC
None
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1783

Modified by ansley.b.manke on 30 Sep 2011 22:34 UTC


Detect missing dependencies

I had an issue where I tried to run Ionide, but nothing was happening in F# files except syntax highlighting. The problem turned out to be that I didn't have this installed:
Visual F# Tools 3.1.2 [http://www.microsoft.com/en-us/download/details.aspx?id=44011]
Is it possible to detect this, and warn the user / suggest the download if it is missing?

I don't really think it's easily possible... but maybe someone else has an idea ;)


Support HTMLTemplateElement.content for #1065

See also #1065
@domenic here is my quick patch for supporting template.content, I'm not sure what's the best approach to hack that things. Please take a look!

The spec definitely requires a lot more details with template elements, especially in regards to cloning so modifying the result of content doesn't modify the template element itself.

Thanks for working on this!
My biggest concern here is that I'm pretty sure template.content and template.firstChild should actually be different. The former should be a document fragment, whereas the latter should be the first node nested under the template tag in the source.
It'd be OK to have an incomplete implementation, but we should at least get that right.
For the tests, could you either find appropriate web platform tests, or create new ones in https://github.com/tmpvar/jsdom/tree/master/test/web-platform-tests/to-upstream ?

Hey @domenic,
Unfortunately, there are lots of tests cannot passed web platform tests, so I leaved some passable tests and updated the patch.

I was thinking the spec [1] is a little bit unclear to me and I spent lots of time doing trial and errors on real browser to figure out real behavior behind that. (Or maybe I missed something from spec?)
[1] https://html.spec.whatwg.org/multipage/scripting.html#template-contents

This is probably ready to pull when I finally get some free time for a jsdom release. But, I'm curious:

What did you find unclear about the spec?
Were you not able to get any of the other tests in https://github.com/w3c/web-platform-tests/tree/master/html/semantics/scripting-1/the-template-element passing?

There are only two additions-to-the-steps-to-clone-a-node tests are able to pass after I checked all tests of the-template-element. Some tests got stuck without any error messages so I guess it probably ran into problem during parsing?
We need to do more works to support correct behavior for template like as only append template's childNodes into template.content in HTML parsing stage.
(I found that it will create a documentFragment object as its first child after HTML parsing)
After that, we can do more improvement :)

About unclear part, for instance:

Let doc be the template element's node document's appropriate template contents owner document.
what's doc mean?
Create a DocumentFragment object whose node document is doc.
Could you give me an code example how to do this? It may help me understand it quickly.

thank!

Descriptions between [1] and [2] are a little bit different. Keyword node document appears in [1] but  ownerDocument appears in [2]. Which one is better?
[1] https://html.spec.whatwg.org/multipage/scripting.html#template-contents
[2] http://www.w3.org/TR/html5/scripting-1.html#the-template-element

Please don't use the outdated W3C fork when implementing jsdom features. Use http://html.spec.whatwg.org/multipage/ instead.

ok I got it, but description of appropriate template contents owner document is same as outdated one...lol
I've updated my patch to move jscs:disable maximumLineLength in index.js.

OK. To create a new Document without a browsing context, you can do new core.HTMLDocument({ parsingMode: "html" }).
But yeah, this is still good to pull in when I have the time to do a release, just clarifying :)

Merged as 86ef3543de49deb21d75e07179baf5a45a56c7c3; look for a 7.0.0 soon with this in it! Thanks so much for contributing; this is a great feature to have.

nice!!


adding extra owners

Adding CSUP reviewers as owners so automation adds them automatically

I rebuilt the actions that use the owners list.


Detail on WeightDrop class _setup() cuDNN RNN weight compacting issue & register_parameter()

Hi there,
cc @Smerity
Thanks for sharing the code first of all. I've been diving into the details and would really appreciate if you could share some insight into WeightDrop class' self._setup() method.
I have 2 questions.

regarding the comment on the cuDNN RNN weight compacting issue, code here. Could anyone expand on what exactly this issue is?

Why does the code delete parameters and registering them again by calling register_parameter()? code here

Thanks.

I share @esvhd's request: I'd love to have more details on this issue.

I can explain part 2 of the question, but would love an explanation of part 1.
Essentially in a forward pass of the network with WeightDrop, there needs to be two separate copies of each weight parameter. The first copy is weight_raw, which is equivalent to the normal weight variable. The second copy is a version of weight_raw, but with dropout applied. It is derived from weight_raw, but has to be named the same as the original weight parameter so that it'll be used in the downstream calculations (eg a linear layer expects a parameter named weight to use in the computation, so the weight-dropped parameter must have this name).
During backpropagation, the gradient is propagated through weight into weight_raw (see #8 for more details) so that weight_raw accumulates the gradient updates and forms the actual copy of the layer weights.

@zplizzi thanks for the clarification.
Let me make sure I got this right. The two separate copies of each weight parameters - the first, un-registered is needed by the forward pass, and the second, registered, is used in training model for applying dropout and computing gradients? During training, the registered weights are updated, then copied to the un-registered version for eval later?
So for the section of setup() code I linked to, it is simply making sure that the weights are registered when the network is initialised? Wouldn't weights such as these be registered by default?
Perhaps what I need to understand better is registered vs un-registered weights in pytorch...
Thanks.

I think you mostly got it right. In the setup() code, it's actually changing the name of the weights - it's moving them from weight to weight_raw. That way there's a place to create a new version of weight (with dropout applied) in the _setweights() method.
I'm not exactly sure the significance of weight_raw being registered with register_parameter() and weight not being registered that way. I suspect that's a way to indicate that the weights should be updated in SGD.

Thanks @zplizzi , very helpful.

@zplizzi If we print the parameters, we can see that the dropped version of the weights are not included. They are only included as an attribute of the model and not their parameters. How does the network know they're the parameters needed to be updated?


Document Instructions for adding AAD Conditional Access Policy Exclusions

💡 Summary
Add instructions to enable users to specify specific user and group exclusions that are documented exemptions to select conditional access policies in the AAD baselines.
Define the yaml structure for CAP exclusions and provide examples.
Motivation and context
CAP exclusions were introduced in the v.0.3 release, but detailed instructions are needed to help users configure them before running the tool.
This would be useful because most tenants are configured to allow CAP exclusions. Previously, the ScubaGear tool flagged as failures any user or group exclusions to these policies. Users need a way to define exemptions, such that their baseline policies are not marked as failures by the ScubaGear tool.
Implementation notes

This feature is implemented using the -ConfigFilePath option and a configuration file using  yaml notation.
To run the tool using this option, type: Invoke-SCuBA -ConfigFilePath .\sample-config-files\aad-config.yaml

Acceptance criteria
How do we know when this work is done?

[ ] Examples for each of the relevant conditional access policies
[ ] Examples of yaml aliases and anchors
[ ] Clear and easy to follow instructions

Didn't realize we had this issue and I made a related issue: #344. Might be best to tackle them both at the same time.

Recommend to add anything missing from 344 to this issue and close 344. They look the same to me, or at least are trying to achieve the same purpose. @schrolla Thoughts?


Unify heaj students & teachers domains

Teachers have @heaj.be
Students have @student.heaj.be
So I renamed student.txt into heaj.txt and move it one level upper.

@AlexisWalravens First stage of verification passed, official domain is heaj.be as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@AlexisWalravens Pull request merged. Thank you.


Minor correction to avoid warning

I'll merge this and we can deal with the build thing separately.


NXP RW6xx: Add additional support to Wakeup from low power modes

This SoC has an external pin interrupt that can be used to wakeup from low power modes.

As this PR stands, it will simply wake the part up- there will be no callback to the application indicating that a level change on pin 24 or 25 caused the interrupt. Personally, I think we should add this support within this PR, but if you'd prefer to enable this more basic functionality in the initial revision I am ok with that as well.

We do not need the notification at this point as we PM mode 3 is not enabled. However I do agree that some form of notification would be required. We should be able to use the GPIO PM suspend and wakeup hooks to notify the GPIO driver. The driver can take appropriate action for pin wakeup.


[WIP] Introduce Puli and PHP-DI configuration file

This PR aims to introduce Puli into this project in order to simplify the installation process even more for users of DependecyInjection libraries that are compatible with Puli.
It also introduce a PHP-DI configuration file as the library will probably soon support Puli (https://github.com/PHP-DI/PHP-DI/pull/360, https://github.com/PHP-DI/Kernel).

IMO, the library should be agnostic of DI containers

The library will stay DI agnostic : it is just a help to setup the library in PHPDI using Puli. It's a bit the equivalent of the Symfony EventDispatcher bridge, and I would like to create a Symfony DI integration as well in the future :) .

Wouldn't be the wiki a better place for this "recipe"? Or maybe the docs?

The useful thing about Puli is it's auto installation of packages with the Composer plugin. When you will install FOSMessage using Composer (composer require ...), if you are using PHP-DI and Puli, it will automatically load the configuration file of this PR so the services will be loaded without any action from you (not even an "include"). It's much more powerful than documentation IMO.
I'm not sure of anything on this level for the moment, it's something quite new (PHP-DI is experimenting Puli). Do you think we should not include the PHP-DI integration for the moment?

It sounds quite useful and I'm aware of PHP-DI's experiments with Puli, but I'm not sure if this should be included directly into this lib.
But toying around with it a little bit can't hurt ;)

In fact, the idea I have behind this would be to support a basic integration of the library into any framework without having to create a link package (bundle for Sf, module for ZF, etc.). I think creating a link package for 3 services and a bit of configuration does not worth it and I would like to try to implement this with Puli :) .
It's very expermiental though, so I probably won't merge it soon (I think only PHP-DI supports Puli forthe moment).

Is there anything that can be done here that would benefit the development of this proposal: https://github.com/FriendsOfSymfony/FOSMessage/issues/19 (Driver for Drupal)? If so, I may have some time to help over the coming weeks. Thanks!

I think we should keep it out of the library for now, until the team working on PSR-11 at least agrees on an approach to the problem

I agree with @stof, there have been quite a lot of work on the container-interop side (PSR-11 but also https://github.com/container-interop in general). We should probably wait for this to be ready.
@mccrodp: is the container accessible from plugins in Drupal 8 or is it only for the core? If you can use it in your plugin, it could be a great opprtunity to implement https://github.com/container-interop/service-provider directly into the library so you don't have to do anything (your plugin would be less coupled to the library internal architecture).

@tgalopin Thanks for your reply. I'm going to do some research into your suggestions and then answer you properly soon, re: Drupal 8.

@tgalopin the @container-interop organization is the sandbox of the PSR-11 workgroup for their experiments.
I would not rely on any of their experiments, as they are just that: experiments which can change at any time

Well ... I think that's what I said isn't it?

Hi everyone! I don't know how I ended up on this issue but that's the magic of the internet I guess :)
That discussion is an interesting use case for https://github.com/container-interop/service-provider indeed (as you mentioned @tgalopin). This experiment is for PSR-11, and it's not stable. However if you have feedback as a potential user of this [not-yet] standard feel free to post it to the project.

@tgalopin the @container-interop organization is the sandbox of the PSR-11 workgroup for their experiments. I would not rely on any of their experiments, as they are just that: experiments which can change at any time

@stof repositories that are tagged with stable version numbers (eg. 1.0.0) are stable though. The only example for now is https://github.com/container-interop/container-interop which indeed started as an experiment, but today it's used as a stable "standard" (not a PSR yet though, but that doesn't prevent it from being useful). However the "service-provider" is still experimental indeed.


ValueError: Assignment map with scope only name bert/position_embeddings should map to scope only bert/embeddings/position_embeddings. Should be 'scope/': 'other_scope/'.

when i try to  do
model_utils.init_bert_checkpoint(init_checkpoint)
I get
ValueError:Assignment map with scope only name bert/position_embeddings should map to scope only bert/embeddings/position_embeddings. Should be 'scope/': 'other_scope/'.
This was working fine earlier ,could this be due to the new changes introduced in texar ,as I have not changed my code .

Hi, we've just updated the TransformerEncoder/Decoder modules.
Please either use the previous Texar version (v0.2.0) to use your BERT code, or use the (slightly) updated bert example with the lastest Texar.

For now I would want to use the previous version as I am in the middle of something very urgent .
Could you please explain how I could get  the previous version .
Because I use colab and session gets reset (every 12 hours ) so I do not have the older version of texar .

Here is the older  version (v0.2)：https://github.com/asyml/texar/releases/tag/v0.2.0

Is v0.2.0 the latest one or v0.1.0?
Should I not need v0.1.0?

I  used the https://github.com/asyml/texar/releases/tag/v0.1.0
However I am still facing the following issue :
_build() got an unexpected keyword argument 'length_penalty'
looks like length_penalty has been removed from even the older version??
I was in the middle of training , and if I remove length_penalty  there is a incompatability of checkpoints.

Pls use v0.2.0, which is an older version than the one in the master branch now.

Thankyou ! It works now:)

Hi! I uses the previous Texar version (v0.2.0) but it still appears to have this problem. Could any other solutions provided? Thank you!

Can you post the error msg?


Feature/routing lohmann

[ ] closes #xxx
[ ] tests passed
[ ] new tests added
[ ] science test figures
[ ] ran uncrustify prior to final commit
[ ] ReleaseNotes entry

@wietsefranssen - can you provide some context for this? I don't know what this PR is trying to do. Is this different from PR #751?

No! This is the same, sorry! I'll close it!


UTXO balances have a quantity of 0 after detaching

I have this asset PEPEFAIR that was previously attached to a utxo.
 ~/ curl 'https://dev.counterparty.io:4000/v2/addresses/bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp/balances/PEPEFAIR?verbose=true' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1002  100  1002    0     0   1875      0 --:--:-- --:--:-- --:--:--  1879
{
  "result": [
    {
      "address": null,
      "asset": "PEPEFAIR",
      "quantity": 0,
      "utxo": "62577e84c39908b3adbadffa946b15bccbda7182f27c1cb5ab1efa8fa55fa046:0",
      "utxo_address": "bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp",
      "asset_info": {
        "asset_longname": null,
        "description": "https://jlbmoope6h7l77q57j6ij7az35ekslty4lzhqyxdv425pqqffira.arweave.net/SsLHOeTx_r_-Hfp8hPwZ30ipLnji8nhi468118IFKiI/PEPEF.json",
        "issuer": "1LEVY62hNa2qQybRFWLb2h9sdPPexnDcR",
        "divisible": true,
        "locked": true
      },
      "quantity_normalized": "0.00000000"
    },
    {
      "address": "bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp",
      "asset": "PEPEFAIR",
      "quantity":<PHONE_NUMBER>00,
      "utxo": null,
      "utxo_address": null,
      "asset_info": {
        "asset_longname": null,
        "description": "https://jlbmoope6h7l77q57j6ij7az35ekslty4lzhqyxdv425pqqffira.arweave.net/SsLHOeTx_r_-Hfp8hPwZ30ipLnji8nhi468118IFKiI/PEPEF.json",
        "issuer": "1LEVY62hNa2qQybRFWLb2h9sdPPexnDcR",
        "divisible": true,
        "locked": true
      },
      "quantity_normalized": "9986.00000000"
    }
  ],
  "next_cursor": null,
  "result_count": 2
}

you can see that I've detached it from the utxo:
 ~/ curl 'https://dev.counterparty.io:4000/v2/addresses/events?verbose=true&addresses=bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp&limit=1000&event_name=DETACH_FROM_UTXO' | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1830  100  1830    0     0   5447      0 --:--:-- --:--:-- --:--:--  5446
{
  "result": [
    {
      "event_index": 18831294,
      "event": "DETACH_FROM_UTXO",
      "params": {
        "asset": "PEPEFAIR",
        "block_index": 871815,
        "destination": "bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp",
        "fee_paid": 0,
        "msg_index": 0,
        "quantity":<PHONE_NUMBER>00,
        "source": "62577e84c39908b3adbadffa946b15bccbda7182f27c1cb5ab1efa8fa55fa046:0",
        "status": "valid",
        "tx_hash": "b463d69483ba5a4dc94006c66be904a9e6611b5ab000ca25bb7ab7928af9060a",
        "tx_index": 2970965,
        "block_time":<PHONE_NUMBER>,
        "asset_info": {
          "asset_longname": null,
          "description": "https://jlbmoope6h7l77q57j6ij7az35ekslty4lzhqyxdv425pqqffira.arweave.net/SsLHOeTx_r_-Hfp8hPwZ30ipLnji8nhi468118IFKiI/PEPEF.json",
          "issuer": "1LEVY62hNa2qQybRFWLb2h9sdPPexnDcR",
          "divisible": true,
          "locked": true
        },
        "quantity_normalized": "1000.00000000",
        "fee_paid_normalized": "0.00000000"
      },
      "tx_hash": "b463d69483ba5a4dc94006c66be904a9e6611b5ab000ca25bb7ab7928af9060a",
      "block_index": 871815,
      "block_time":<PHONE_NUMBER>
    },
    {
      "event_index": 18831290,
      "event": "DETACH_FROM_UTXO",
      "params": {
        "asset": "PEPECASH",
        "block_index": 871815,
        "destination": "bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp",
        "fee_paid": 0,
        "msg_index": 0,
        "quantity":<PHONE_NUMBER>00,
        "source": "28a8cf210d58fcb2fabb8a62ba79d5492e490322e2d820ec6a31a5361a391a5b:0",
        "status": "valid",
        "tx_hash": "8daf023934c21364e6543ba2323977c7f54b6b1a5fdec15c7a189d6c030b539c",
        "tx_index": 2970964,
        "block_time":<PHONE_NUMBER>,
        "asset_info": {
          "asset_longname": null,
          "description": "http://rarepepedirectory.com/json/pc.json",
          "issuer": "1GQhaWqejcGJ4GhQar7SjcCfadxvf5DNBD",
          "divisible": true,
          "locked": true
        },
        "quantity_normalized": "1000.00000000",
        "fee_paid_normalized": "0.00000000"
      },
      "tx_hash": "8daf023934c21364e6543ba2323977c7f54b6b1a5fdec15c7a189d6c030b539c",
      "block_index": 871815,
      "block_time":<PHONE_NUMBER>
    }
  ],
  "next_cursor": 18340473,
  "result_count": 1565
}

but it looks like from the balances call, it is still "attached" with a quantity 0, causing my utxos to be unavailable. This could definitely account for some of the signing errors we've been seeing (utxos surprisingly being unavailable)
    {
      "address": null,
      "asset": "PEPEFAIR",
      "quantity": 0,
      "utxo": "62577e84c39908b3adbadffa946b15bccbda7182f27c1cb5ab1efa8fa55fa046:0",
      "utxo_address": "bc1q0eapk4tyqa7r2vcta6z6v2mgnqcux3kfkmurzp",
      "asset_info": {
        "asset_longname": null,
        "description": "https://jlbmoope6h7l77q57j6ij7az35ekslty4lzhqyxdv425pqqffira.arweave.net/SsLHOeTx_r_-Hfp8hPwZ30ipLnji8nhi468118IFKiI/PEPEF.json",
        "issuer": "1LEVY62hNa2qQybRFWLb2h9sdPPexnDcR",
        "divisible": true,
        "locked": true
      },
      "quantity_normalized": "0.00000000"
    }

it's unclear what the expected behavior is. I assume we should mimic what we do for regular balances.
the important thing is that once the balance is zero, the associated utxo shouldn't be excluded from a tx input set

fixed here https://github.com/CounterpartyXCP/counterparty-core/pull/2768/commits/6ee2b048291b976142061f1302db6ddd523c31e5

and also here https://github.com/CounterpartyXCP/counterparty-core/pull/2781/commits/b63f471f6d204bb6c4eede3de3a15f34d9ad3b2d


Make it so when calling the function guildID and userID don't have to be Strings

Make it so when calling the function guildID and userID don't have to be Strings

Ahhh, ok. I was wondering if there was a reason for them being strings, I felt that it was weird because it is technically an integer but you made us put them in as strings


Add nanvar

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.15%. Comparing base (1aeccc2) to head (08da92a).
Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
+ Coverage   85.60%   86.15%   +0.55%
==========================================
  Files          10       10
  Lines         125      130       +5
  Branches        8        8
==========================================
+ Hits          107      112       +5
  Misses         18       18

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adapt to coq/coq#17576 (declare_variable takes typing flags argument)

Upstream PR merged, this can be merged.


Error handling on proxy error

What is the graceful way to handle proxy errors?
Currently what happens if the server that should be proxied to fails, is that the application crashes with a:
.../node_modules/http-server/node_modules/http-proxy/lib/http-proxy/index.js:119
    throw err;
          ^
Error: connect ECONNREFUSED
    at errnoException (net.js:901:11)
    at Object.afterConnect [as oncomplete] (net.js:892:19)

The proxy object is not available. So I can not add an error handler. Something like the following would be very usefull:
  if (typeof options.proxy === 'string') {
    var proxy = httpProxy.createProxyServer({});
    before.push(function (req, res) {
      proxy.web(req, res, {
        target: options.proxy,
        changeOrigin: true
      });
    });
    // NEW CODE HERE
    if (options.proxyErrorHandler) {
      proxy.on('error', options.proxyErrorHandler);
    }
    // END NEW CODE HERE
  }

This is just an idea. But custom handling of errors in the proxy would be more useful to me, then an exception. :)

+1 !

I also met this error when my target server was starting and not yet ready to be used.
When ECONNREFUSED  happens, instead of killing the http-server process, it is possible to simply return "503 Service Unavailable" to the client, keeping the http-server service alive?
Or at least, can be helpful to provide an option to disable this default behavior (exit process), returning 503 http code instead.

Any update on this? I have a similar issue (already posted here http://stackoverflow.com/questions/38795695/http-proxy-handling-econnrefused-and-econnreset before I found this).

Need this too. Any updates?

Any updates?


Stylable per-page dropdown/select template

Your default template uses a plain old select dropdown for the per-page selection. I'm working on using, as a start, angular-ui's bootstrap dropdown button. Any advice on connecting models and clicks to influence the pagination directives scoped variables?

You could try attaching a modified template to the directive. The default template is https://github.com/begriffs/angular-paginate-anything/blob/master/src/paginate-anything.html
Try copying the template to a new name, modifying it, and setting the directive's templateUrl attribute to point at the file.
The way it works right now is that by default it uses src/paginate-anything.html - https://github.com/begriffs/angular-paginate-anything/blob/master/dist/paginate-anything-tpls.js#L66-L68
And dist/angular-paginate-anything-tpls.js bundles html into this default template - https://github.com/begriffs/angular-paginate-anything/blob/master/dist/paginate-anything-tpls.js#L357-L364
But if you override templateUrl then it will use your own html template wherein you're free write custom angular code for a new dropdown etc.

Hey, thanks for the quick response. I apologize for not being clearer. I have already created a local template and the directive is successfully using it. My question was really directed towards how to get my dropdown button to use/modify the directive scopes values (perPage).
If I replace the select, for example, with a simple . Where would this function belong? In the directive params I set perPage attribute to a scoped variable in my controller. That works. but I can't seem to update that from within the template, ie. updatePP().
Does that make more sense?

I think you're right that you have to create a function in your controller $scope for an ng-click, as demonstrated here http://www.infragistics.com/community/blogs/dhananjay_kumar/archive/2015/06/29/how-to-work-with-the-bootstrap-dropdown-in-angularjs.aspx
Does your example work if you put pp in your controller rather than in the template code? Perhaps in the template you could then say <button ng-click="$parent.pp(100)">

I'll keep working on it, thanks.
On Thu, Jan 7, 2016 at 1:53 PM Joe Nelson<EMAIL_ADDRESS>wrote:

I think you're right that you have to create a function in your controller
$scope for an ng-click, as demonstrated here
http://www.infragistics.com/community/blogs/dhananjay_kumar/archive/2015/06/29/how-to-work-with-the-bootstrap-dropdown-in-angularjs.aspx
Does your example work if you put pp in your controller rather than in
the template code? Perhaps in the template you could then say
—
Reply to this email directly or view it on GitHub
https://github.com/begriffs/angular-paginate-anything/issues/88#issuecomment-169772669
.


[rom_ext] fix cert update

If a DICE certificate must be updated, the entire page must be erased before writing, since flash bits can only transition from 1 --> 0.

CI failures are unrelated to this PR.


feat(pose_initilizer): support gnss/imu pose estimator

Description
This is a modification of pose_initilizer to use GNSS/IMU pose estimtator.
When parameter ndt_enabled is false, LocalizationTriggerModule without ndt_trigger_ is activated.
Related discussions are here.
https://github.com/autowarefoundation/autoware.universe/pull/2848#discussion_r1104149473
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

This implementation was not adopted due to cyclomatic complexity
https://github.com/autowarefoundation/autoware.universe/pull/2904/commits/afdadb325b2ed66af12a4c788312162858c150f1

@YamatoAndo
Hello. I have extended pose_initilizer for gnss/imu pose estimator, please review.

@rsasaki0109
Is the copyright 2022 intentional? It is recommended to use the same year if it is almost the same as the original file, and 2023 if there is a lot of new code.

Just conforming to localization_trigger_module.cpp, no deeper meaning. I think your suggestion is good and I will modify it.

Is the copyright 2022 intentional? It is recommended to use the same year if it is almost the same as the original file, and 2023 if there is a lot of new code.

Sorry, this was a comment on the new file. Existing files will retain their original publication year. So please don't change ekf_localization_trigger_module.hpp.

@YamatoAndo @KeisukeShima
I would appreciate a review when you are free.

@YamatoAndo @Igata-ctrl
I don't have write permission, so could you please merge them?


Cant escape # signs

When trying to run crackmapexec rdp if the password has a # sign in it the module wont work
crackmapexec rdp <IP_ADDRESS> -u 'uname' -p 'pwd#' --nla-screenshot --screenshot
RDP         ~~~~~~~~~~~~~~~~~~ (domain:~~~~~~~~~~) (nla:True)
RDP         ~~~~~~~~~~~~~~~~~~~
Traceback (most recent call last):
  File "/usr/bin/crackmapexec", line 8, in <module>
    sys.exit(main())
  File "/usr/lib/python3/dist-packages/cme/crackmapexec.py", line 257, in main
    asyncio.run(
  File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()
  File "/usr/lib/python3/dist-packages/cme/crackmapexec.py", line 105, in start_threadpool
    await asyncio.gather(*jobs)
  File "/usr/lib/python3/dist-packages/cme/crackmapexec.py", line 69, in run_protocol
    await asyncio.wait_for(
  File "/usr/lib/python3.10/asyncio/tasks.py", line 408, in wait_for
    return await fut
  File "/usr/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/lib/python3/dist-packages/cme/protocols/rdp.py", line 59, in __init__
    connection.__init__(self, args, db, host)
  File "/usr/lib/python3/dist-packages/cme/connection.py", line 65, in __init__
    self.proto_flow()
  File "/usr/lib/python3/dist-packages/cme/protocols/rdp.py", line 91, in proto_flow
    self.call_cmd_args()
  File "/usr/lib/python3/dist-packages/cme/connection.py", line 112, in call_cmd_args
    r = getattr(self, k)()
  File "/usr/lib/python3/dist-packages/cme/protocols/rdp.py", line 242, in screenshot
    asyncio.run(self.screen())
  File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()
  File "/usr/lib/python3/dist-packages/cme/protocols/rdp.py", line 232, in screen
    await self.connect_rdp(self.url)
  File "/usr/lib/python3/dist-packages/cme/protocols/rdp.py", line 152, in connect_rdp
    connectionfactory = RDPConnectionFactory.from_url(url, self.iosettings)
  File "/usr/lib/python3/dist-packages/aardwolf/commons/factory.py", line 68, in from_url
    target = RDPTarget.from_url(connection_url)
  File "/usr/lib/python3/dist-packages/aardwolf/commons/target.py", line 78, in from_url
    if url_e.port:
  File "/usr/lib/python3.10/urllib/parse.py", line 177, in port
    raise ValueError(f"Port could not be cast to integer value as {port!r}")
ValueError: Port could not be cast to integer value as 'pwd'

I know the account works, I can do it manually - I know the script works with passwords that dont include a #. I've tried single quotes, double quotes, escape chars, -p=, etc. to no avail
CME 5.4.0

Can confirm cme is dying when a # is in the password. The reason is urlparse uses # as seperator for "fragments", therefore everything brokes apart. I will open an issue on the aardwolf repo.

I'm on it :)

Hello,
Thanks for the issue, it is now fixed on the last public release of CrackMapExec https://github.com/mpgn/CrackMapExec v6.0.0
Regards,
mpgn


Cross-platform problem with path separator

When publishing assets on Windows and pulling them into Linux, the asset directory structure is not properly preserved. Instead after unzipping, assets include the path with backward slashes in the filename (e.g. assets\css\bootstrap.min.css).
I assume that happens because directory separators are hard-coded as forward slash (/), but should use the PHP global constant DIRECTORY_SEPARATOR to avoid cross-platform issues with path names.

Ah, yeah that would be a really good change, thanks for the heads up @io238 - I'll try and get this updated for you as soon as possible.

I went a bit into the details of this. Seems to be an inconsistency how the Symfony Finder class returns path names.
See also:
https://github.com/symfony/symfony/issues/15472
https://www.php.net/manual/en/ziparchive.addfile.php
I've added a pull request with a fix for this.

Thanks so much @io238 - sorry for the slow reply to this PR, been very busy. I'm going to be looking into this tomorrow for you, and thank you for the PR!

@Sammyjo20 from our Twitter chat, looks like @io238 did the work already and has a PR. Tested and works like a charm on Windows.

This has now been merged, I'll tag a release for it now @alexjustesen 🤘


Account + profile page

I plan on adding some auto tests for the user deletion stuff

@danielliu04 one issue with the current profile page (not account), is that achievements will always show the logged in user's achievements, instead of the profile owner,
Can you make the achievement table more flexible in that regard? Note that we access profiles with username and not user_id


Change the date only icon by setting the hidden field.

In version 3, I could change the date to the hidden field and so call the function and changed the page. In the new version 4, I can not change the date, do not open the calendar.
My code is in https://jsfiddle.net/pedrosalpr/prsodvv6/1/  that worked in version 3. How can I change the code to work to work in version 4?
I've tried several times, but all failed.

your fiddle throws console errors. Can you reproduce your issue with v3 or v4 that doesn't get console errors?

I updated the fiddle, creating several fiddles with different libs:
moment (2.9.0) and datetimepicker (3.1.3) - (Runs)
http://jsfiddle.net/pedrosalpr/prsodvv6/2/
moment (2.10.3) and datetimepicker (3.1.3) - (Runs)
http://jsfiddle.net/pedrosalpr/prsodvv6/3/
moment (2.10.3) and datetimepicker (04.07.14) - (Error - It does not show the calendar)
http://jsfiddle.net/pedrosalpr/prsodvv6/4/


The "only finland" filter needs be done properly with "[x] finland [x] other countries"

Corrected, and tested to work.


adding scalar software to blogs category

Thank you, @alexandarp! Before I merge it, would you mind adding yourself to the AUTHORS file?

@vmbrasseur I added myself in a previous PR :)

Ah! So you did! Apologies for missing that.
:heart: :heart: :heart: for yet another contribution. Thank you!


feat: add basic statement converter for influxql raw query

Which issue does this PR close?
Part of #698
Rationale for this change
See #698

What changes are included in this PR?

Are there any user-facing changes?
None.

How does this change test
Test by ut.

Codecov Report

Merging #703 (eea59fe) into main (f79890b) will decrease coverage by 0.21%.
The diff coverage is 55.70%.

:exclamation: Current head eea59fe differs from pull request most recent head f84ade2. Consider uploading reports for the commit f84ade2 to get more accurate results

@@            Coverage Diff             @@
##             main     #703      +/-   ##
==========================================
- Coverage   67.55%   67.35%   -0.21%
==========================================
  Files         281      287       +6
  Lines       44343    45088     +745
==========================================
+ Hits        29956    30369     +413
- Misses      14387    14719     +332

Impacted Files
Coverage Δ

server/src/grpc/storage_service/prom_query.rs
54.28% <0.00%> (-0.29%)
:arrow_down:

sql/src/frontend.rs
0.00% <0.00%> (ø)

sql/src/influxql/planner.rs
0.00% <0.00%> (ø)

sql/src/influxql/util.rs
0.00% <0.00%> (ø)

sql/src/lib.rs
100.00% <ø> (ø)

sql/src/planner.rs
91.91% <25.00%> (-0.48%)
:arrow_down:

sql/src/influxql/stmt_converter.rs
57.40% <57.40%> (ø)

sql/src/influxql/stmt_rewriter.rs
68.19% <68.19%> (ø)

sql/src/influxql/mod.rs
75.00% <75.00%> (ø)

sql/src/influxql/test_util.rs
94.73% <94.73%> (ø)

... and 3 more

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


Teach sync test to prevent forks harder

Delivers https://www.pivotaltracker.com/story/show/160892233
Please refer to commit messages for details.

I will open new PR with cleaner history.


render Id as attribute in svg items

Hi,
Could you add the id of the task, switch, any element in general to the group or rect of the svg items as an attribute? This way we can target it with CSS, thanks!

Hello @MrMoronIV!
Currently, the designer is adding a type of a step to the class list (sqd-type-{type}). Do you need an id, or a task type?

I need the actual internal id of the task etc. on the svg element :)

The 0.19.4 version now generates the data-step-id attribute for root <g> elements of step components. Example CSS selector:
.sqd-workspace-canvas g[data-step-id="6cc1da8be15b9c3173a2effc09705c2e"] rect {fill: red !important;}


Create a Book class and Label class

Hi @KareemWilson  and @Olivier-Kango ,🙃
On this branch I:

Create a Book class in a separate .rb file.
Create a Label class with an association to the Item class (in a separate .rb file).
Implement methods:

[x] add_item method in the Label class

should take an instance of the Item class as an input
should add the input item to the collection of items
should add self as a property of the item object (by using the correct setter from the item object)

[x] can_be_archived?() in the Book class

should override the method from the parent class
should return true if parent's method returns true OR if cover_state equals to "bad"
otherwise, it should return false

[x] Add unit tests for all implemented methods.

Hi @DaveZag ,
Status: Changes Required ♻
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Good Highlights✔

Linter checks are passing
Good commit messages
Adhered to Gitflow

Required Changes ♻
It seems that your Pull request is different from the work you did.
I think you worked on

[ ] MusicAlbum class and
[ ] Genre
But I see in your pull request Create a Book class and Label class.
I suggest just writing a Pull Request which corresponds exactly with the code you wrote. Then your project will be perfect.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Yeah, that one was a mistake. I've fixed it


Nonexistent functions of AudioServer

Godot version:
3.0.2_stable
OS/device including version:
Linux Mint 18.3 x64
Issue description:
Can't call methods (also autocompletion doesn't show these methods):
	musicLevel = AudioServer.get_stream_global_volume_scale()
	effectsLevel = AudioServer.get_fx_global_volume_scale()

Result:
Invalid call. Nonexistent function 'get_stream_global_volume_scale' in base AudioServer.
What I'm doing wrong? Thanks.

Those functions don't exist anymore (used to exist pre 3.0), check the AudioServer doc: http://docs.godotengine.org/en/3.0/classes/class_audioserver.html

@marcelofg55 thanks for answer.
but, how I can mute all sound?

You can use set_bus_mute or set_bus_volume_db. If you want different levels for music and SFX create two buses so you can control their volumes separately.

@marcelofg55 thanks!


Double SG values at same timestamp

Once in a while, I get double SG values for some period of time when extracting sensor events; this may be from one sample to some 5-10 samples. The attached plot from my web application shows this behaviour. Any clue? I do admit that I have not thoroughly traced the origin of these extra data, but I will do that.
My best theory about the reason for these branches is the pump losing contact with the sensor transmitter for a short time. The transmitter stores the SG-values, and once contact with the pump is established, the pump can read out the saved SG values. These values supposedly have an offset time with respect to some other point. In the current code this offset time is added to the time of the reference point, whereas they should have been subtracted.
This is becoming an increasing problem, since we observe that the frequency of the pump losing signal from the transmitter is increasing, and the durations longer. It can get quite difficult to grasp the  plot sometimes.

P.S.: This issue was originally posted in the @paulokow repository.

I believe the solution must lie in the handling of timestamps for nested sensor readings events at https://github.com/pazaan/decoding-contour-next-link/blob/b0198b721958230dc38a7899209b7de637f83c03/pump_history_parser.py#L399
My hypothesis is, that for a period where the pump could not get signal from the sensor tranmitter, and for which the data generated will be later read out, a separate SensorGlucoseReadingsEvent will be generated. The timestamp for this, aggregate, event will be at the end of the dead period of communication. The timestamps of the nested events have to be offset back in time, rather than, for a regular SensorGlucoseReadingsEvent, forward in time from the aggregate event timestamp.
If this is so, how to distinguish the two different types of SensorGlucoseReadingsEvent. I will try to investigate this.

@mortlind hey, we already handle this as part of the android uploader. Here's the latest code for this:
        // validate that this contains a new SGV record
        if (pumpRecord.isCgmActive()) {
            if (pumpRecord.getEventRTC() - pumpRecord.getCgmRTC() > (POLL_PERIOD_MS + (POLL_GRACE_PERIOD_MS / 2)) / 1000)
                pumpRecord.setOldSgvWhenNewExpected(true);
            else if (!pumpRecord.isCgmWarmUp() && pumpRecord.getSgv() > 0 &&
                    activePump.getPumpHistory().where().equalTo("cgmRTC", pumpRecord.getCgmRTC()).findAll().size() == 0)
                pumpRecord.setValidSGV(true);
        }

Sometimes the pump will get a SGV late and it will be available this poll period and so a retry is done after 60-90 secs.

@Pogman Thanks, but where do you have the code. Could you paste a reference? I do not find the same code or interface here, e.g. the method .isCgmActive is not to be found. If that method should be applicable in the current code base, it must be some property which can be parsed from the payload belonging to a SensorGlucoseReadingsEvent object. Rigth?

That code is not online, probably will have a new branch up in a few days.
When we poll we get the pump time and the the pump status. The pump time RTC is used as the RTC for this event and if the difference from the event and the CGM RTC is too big we've gotten an old SGV from the pump.

Code from master that handles the old sgv from pump:
https://github.com/pazaan/600SeriesAndroidUploader/blob/d582d467fa67d9062a1227e3e2ba8b77068f4d83/app/src/main/java/info/nightscout/android/medtronic/service/MedtronicCnlIntentService.java#L303

@Pogman It seems that you are talking about extracting status? The symptom that I get arises when extracting sensor history data from the pump. Do you think that we are talking about two different issues here?

Ha maybe... ;)
BTW I've worked extensively with history and it's always precise

Hi @mortlind, can you tell if the issue is visible in the dump you provided some time ago
Otherwise I'd need a new dump to be able to help on the issue (or wait 3 weeks, my son gets CGM provisioned in mid Dec).
It may be that I ported something badly from Javascript back then :/

@paulokow Thanks, I will make a new dump, and send it here. I will take a look at it too :-)

@paulokow I looked a bit into the code now. If I dump data, the allNestedEvents method of SensorGlucoseReadingsEvent-objects have already been called, and the SensorGlucoseReadingsEvent-objects will be missing from the dumped data, replaced by the wrongly timestamped SensorGlucoseReadingEvent-objects. So we will have to modify the code to inspect the SensorGlucoseReadingsEvent-objects. But in the current code, the part that is parsed on the eventData for the  SensorGlucoseReadingsEvent-objects, there is no clue to distinguishing the direction of time offsets. We will have to inspect the 600SeriesAndroidUploader-project for that. Agreed?

Hey - the 600Series history handling code is not online yet but maybe this will be of some use. It's just the code I used to make sure I was seeing the right output:
            } else if (eventType == EventType.SENSOR_GLUCOSE_READINGS_EXTENDED) {
                int minutesBetweenReadings = read8toUInt(eventData, index + 0x0B);
                int numberOfReadings = read8toUInt(eventData, index + 0x0C);
                int predictedSg = read16BEtoUInt(eventData, index + 0x0D);
                result += " Min:" + minutesBetweenReadings + " Num:" + numberOfReadings + " SGP:" + predictedSg;

                int pos = index + 15;
                for (int i = 0; i < numberOfReadings; i++) {

                    Date sgtimestamp = MessageUtils.decodeDateTime(eventRTC & 0xFFFFFFFFL - (i * minutesBetweenReadings * 60), eventOFFSET);

                    int sg = read16BEtoUInt(eventData, pos) & 0x03FF;
                    double isig = read16BEtoUInt(eventData, pos + 2) / 100.0;
                    double vctr = (((eventData[pos] >> 2 & 3) << 8) | eventData[pos + 4] & 0x000000FF) / 100.0;
                    double rateOfChange = read16BEtoInt(eventData, pos + 5) / 100.0;
                    int sensorStatus = read8toUInt(eventData, pos + 7);
                    int readingStatus = read8toUInt(eventData, pos + 8);

                    boolean backfilledData = (readingStatus & 1) == 1;
                    boolean settingsChanged = (readingStatus & 2) == 2;
                    boolean noisyData = sensorStatus == 1;
                    boolean discardData = sensorStatus == 2;
                    boolean sensorError = sensorStatus == 3;

                    int sensorException = 0;
                    int sensorEx = 0;

                    if (sg > 0x1FF) {
                        sensorException = sg & 0x00FF;
                        sg = 0;
                        result += "\n! " + sgtimestamp;
                    } else {
                        result += "\n* " + sgtimestamp;
                    }

                    result += " SGV:" + sg + " ISIG:" + isig + " VCTR:" + vctr + " ROC:" + rateOfChange + " STAT:" + readingStatus
                            + " BF:" + backfilledData + " SC:" + settingsChanged + " NS:" + noisyData + " DD:" + discardData + " SE:" + sensorError + " Exception:" + sensorException + "/" + sensorEx;

                    pos += 9;
                }

@Pogman OK. I thought you meant that it was not distributed yet. I have looked for the snippet you sent in the 600SeriesAndroidUploader-project. But you mean that it is not in the repo? I see there a very different timestamp computation. I will look into the present code, and see if the necessary resources are in place to make the same computation here. Thanks!

Sorry for the confusion. The 600series code that's in the repo only handles the time and status messages from the pump. The history handling and the posted snippet are not yet in the online repo.

@Pogman Can you post more of the code? Specifically I need to be able to find out how eventRTC and eventOFFSET are computed. I suppose eventRTC is the same as in the present code called self.timestamp at https://github.com/pazaan/decoding-contour-next-link/blob/b0198b721958230dc38a7899209b7de637f83c03/pump_history_parser.py#L399

code is up on my repo now: https://github.com/Pogman/640gAndroidUploader/tree/dev-history

@Pogman Thanks. I will try to understand the code and time computation.

Fixed in b7cd28b
The problem was with my porting. The single events are stored in the reverse order and one has to deduct time for each event (what actually makes a lot of sense, as it's impossible to see the events in the future!).
My code was adding the events forward, that's why these were overlapping with following events.

@paulokow I have not investigated thoroughly, but I was searching for a distinction between SensorGlucoseReadingsEventwhich where collected during connectivity and after a period of disconnectivity. I interpret your comment as follows: Under normal conditions, the single sensor events are contained in a SensorGlucoseReadingsEvent alone, i.e. cardinality 1. For these single readings recorded during connection between the pump and transmitter, there is no difference in the order they are traversed and since the index * timebetweenmeasurement is always zero, it does not matter. When the pump has missed events for a period, it gets these missed events from the transmitter in a burst inside a SensorGlucoseReadingsEvent, and now you corrected the arithmetics for calculating the timestamps on the series of missed events. Is that correctly understood?


Change editor font to system default

...from monospace.
We're not doing block-level stuff yet, and when we do, we're not going to do it with monspacing. The monospace font just makes the editor feel... programmery.

Closed by https://github.com/subconsciousnetwork/subconscious/pull/1000


[Leanplum syncResourcesAsync:YES] - Causes UI to freeze

It appears this method, [Leanplum syncResourcesAsync:YES], is not truly Asynchronous.
When I have this method in place, and load my app, I discovered the UI freezes for a couple of seconds while LeanPlum is making some sort of connection/request.
To Repro:

Launch app with above method in place.
Interact with UI for a few seconds (Dragging up and down a TableView is a good example)
Notice that the UI freezes for a couple seconds, then unfreezes.
Remove [Leanplum syncResourcesAsync:YES], then relaunch app and repeat UI interactions.
Notice no UI freeze.

Thanks for reporting this. This issue was fixed in version 1.2.20.


the turn server of webrtcweb.com is not stable

thanks for muaz's turn server.but the server is not stable.i try five times but only one time succeed.i test chatting in which one is symetric nat type and another is port restricted cone nat.

@muaz-khan I just had the situation with my own coturn installation, that the server bandwidth usage increased (checked with nload) up to 2Mbit/s - the onNewStream event fired, but the stream wasn't playing. So the data was transfered over turn, but the stream never started to play.


Add seed orders for WETH

These orders can be used to mind and close the seed short tokens

part of dydxprotocol/Brokerage-Issues#52

Pull Request Test Coverage Report for Build 6366

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 6360:
0.0%

Covered Lines:
1200

Relevant Lines:
1200

💛  - Coveralls


Project Dead?

This project appears to be dead.

The last commit is over a year ago.
Issues and even small bugfix PRs have gotten no response.
I've reached out to the maintainer via email and got no response.

I'm considering forking and maintaining a fork with bugfixes from open PRs but I'll give the maintainer another 30 days to respond first as I don't want to unnecessarily split the community.

I have decided that I will maintain a fork. I've reserved a crate name and am ready to go. I'll publish the info if we don't get a response in 7 more days (or if we get a confirmation of unmaintained status).


[Feature Request]: custom CSS transformer support

What problem does this feature solve?
Vite allows the use of a custom CSS transformer (only lightningcss as of right now), and having that functionality in Farm would be great.
What does the proposed API look like?
In farm.config.ts:
	css: {
		transformer: "lightningcss",
	},

Farm is based on plugins, so all transformers should be implemented as a plugin, we won't support css.transformer, instead, we'll provide a plugin-lightningcss later

Could https://github.com/lawrencecchen/vite-plugin-lightningcss be used as a base?

Yes, we'll provide a pure rust lightningcss plugin later


how customize them??

Make sure to add all the information needed to understand the bug so that someone can help. If the info is missing we'll add the 'Needs more information' label and close the issue until there is enough information.

[x] Provide a minimal code snippet, please make sure it is well formatted.
[x] Provide a stackblitz demo / stackblitz example that reproduces the bug.
[ ] Provide screenshots where appropriate
[ ] What's the version of Angular you're using?
[ ] Does this occur on specific browser?

Are you asking about how to change its style? #44


disable apotheosis cane

Apotheosis Sugar cane is interacting poorly with Inspirations sugar cane, allowing seeds to be duplicated while forcefully replacing inspirations sugar can with apotheosis' sugar cane.  The seeds are not used in any recipes, but are Thaumcraft aspected by the stack and can be used in the Composter. Between the two, Inspirations sugar cane is more like other crops and would be preferable to keep if possible.


CI: Improve on the overlap between "on PR" and "nightly" CI runs

Currently our ci.yml (on PR) based and ci_cron.yml (nightly) based PR runs present us with some problems.

The nightly check redundantly repeats some of the PR checks while also doing additional things
The nightly check does not run until the night, and sometimes–especially in the case of doing CI work–it needs to run earlier to help with PR approvals

@tannalynn threw out the following idea:

I wonder if there is a way to make the ci_cron run on PR but only if its own file was changed? Since that’s usually when we want to run it on pr. Should be able to use the same kind of strategy. Although then we’d have some double runs with the pr ci, so then maybe NOT have pr ci run if ci_cron was changed?

and @fallwith threw out the following idea:

ensure that there is no overlap between ci and ci_cron (they run completely different combos)
nightly runs both parts
pr runs only one, unless one of the yaml files themselves changed

We've been iterating on the CI since this issue was originally created and we're happy with the current state of things.


proxy_* params were removed from class apt

Add them to PPA since they were being used there, and add a placeholder
example for setting up the proxy files.

Mmm, we actually have no tests covering this change.


Add a new success event please for when the player is ready

Currently WordPress defines its own success callback, unfortunately! There is no way to override it.
For my custom WP plugin I really need a way to know when a mediaelement.js driven player is ready or not. An event bound to the mejs global would be great. Something like:
mejs.trigger('success', player);

like that I can attach a callback in my WP plugin code:
mejs.on('success', function(player) { ... });

that would be awesome and solve lots of my problems!

@binarykitchen Closing old issues. If you'd like to reopen this, let us know. Thanks!


fix: adapt the change of spliting aws-secret into 2 secrets

rh-pre-commit.version: 2.1.0
rh-pre-commit.check-secrets: ENABLED
Description
This PR relies on PR https://github.com/konflux-ci/konflux-qe-definitions/pull/29
Issue ticket number and link
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest


Change order of commands in Makefile

It would be helpful if you would remove the 1 Approval rule, thanks

can you image adopting conventional commits?
https://www.conventionalcommits.org/en/v1.0.0/
adopting your commits is something i usually do with your MRs - maybe rather a spleen of mine, but i like to have it in the git history


docs: fix syntax error for createAction with TypeScript

What docs page needs to be fixed?

Section: recipes
Page: UsageWithTypescript

What is the problem?
The example was incorrect as the type of createAction is number but the payload is of type string.
What changes does this PR make to fix the problem?
Payload is changed to be a number :-)

Thanks!


Argo server is breaking intermittently

Hi,
We deployed argo 2.6.2 and its working fine. But we have to run "argo server" command in the background in order to see the argo server UI. sometimes the background process is breaking and its causing issues with argo servers and we are unable to access the UI as well as our job submission is getting crippled. I deployed argo server using the following instruction from official argo github:
https://github.com/argoproj/argo/tree/master/manifests/base/argo-server
Is there anyway I can have "argo server" service running continuously without initiating it manually through command line in my shell like "argo server"?
argo: v2.6.2
BuildDate: 2020-03-12T16:04:25Z
GitCommit: be0a0bb
GitTreeState: clean
GitTag: v2.6.2
GoVersion: go1.13.4
Compiler: gc
Platform: linux/amd64

Kubernetes version:
Client Version: v1.18.2
Server Version: v1.17.1

Thank you and appreciate any help or inputs with this issue?
Regards,
CS

How is this any different than https://github.com/argoproj/argo/issues/2945?

Apologies. I saw your comment there. thank you. will give a try.

Closed as duplicate of #2945


fix sr_id for geometry while updating record

Thanks for this awesome package. It is completely working fine when we create a record. But I got one minor error when we try to update a record. Actually, it's not getting proper sr_id in a query.
Here is the migration that I have created:
Schema::create('locations', function (Blueprint $table) {
    $table->increments('id');
    $table->string('name');
    $table->point('location', 'GEOMETRY', 27700);
    $table->timestamps();
});

This is my Model:
class Location extends Model
{
    use PostgisTrait;

    protected $fillable = [
        'name'
    ];

    protected $postgisFields = [
        'location'
    ];

    protected $postgisTypes = [
        'location' => [
            'geomtype' => 'geometry',
            'srid' => 27700
        ]
    ];
}

And this is how I'm creating and updating record.
$location = new \App\Location();
$location->name = "Location1";
$location->location = new Point(37.422009, -122.084047);;
$location->save();

$createdLocation = \App\Location::find($location->id);
$createdLocation->location = new Point(37.422009, -123.084047);
$createdLocation->save();

dd($createdLocation->toArray());

so while save function, I am getting following error:
SQLSTATE[42804]: Datatype mismatch: 7 ERROR:  column "location" is of type geometry but expression is of type geography
LINE 1: update "locations" set "location" = public.ST_GeogFromText('...
                                            ^
HINT:  You will need to rewrite or cast the expression. (SQL: update "locations" set "location" = public.ST_GeogFromText('POINT(-123.084047 37.422009)'), "updated_at" = 2017-09-16 14:03:47 where "id" = 1)

Coverage increased (+0.6%) to 84.444% when pulling 8696d9da6a023586c953afa685e709849dd32ff5 on InfyOmLabs:master into 278a6c1ae6eb7e4a9438ea62939633cc9fc41aab on njbarrett:master.

Hey @mitulgolakiya  thanks for the PR.
It looks like the code needs to handle a null value - have a look at the travis build log.
Thanks!

Coverage increased (+0.5%) to 84.543% when pulling e465a407309ac46a020606b5d35b92466c403c09 on InfyOmLabs:master into b751206154ad06cfea2e8791815c4e03ea593cb2 on njbarrett:master.


setDisabledOptions removes current selection

Context
Multiple virtual selects for same set of "things", only allowing a "thing" in one select. So when a value changes in a selects, that value is disabled from other selects. Selecting single value.
Current behavior
Previous selection is lost, even if its not in the disabled list.
Expected behavior
Disabled status of the elements is updated, selection is preserved.

@kurkle, thanks for letting me know it. I will check and fix them in the future release.


Add io_uring transport

This can be enabled with Armeria flags if desired, but we need to have the class loaded.

IO Uring is so cool! I'd seen some performance benchmarks and it looked amaze


Add mising node sdk example

Description
It would be nice to have an example which illustrates how to to use the hyperledger client (https://github.com/hyperledger/fabric/tree/master/sdk/node) to build a basic node.js app.

@novusopt - that is the plan for sure.  the sdk is still under major development, hence no examples at this point

think that this can be closed as there's an example in the docs.


Update to use template files

Pull Request (PR) description

Update appveyor.yml to use the default template.
Added default template files .gitattributes, and .gitignore, and
.vscode folder.

This Pull Request (PR) fixes the following issues

Task list

[x] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #229 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #229   +/-   ##
===================================
  Coverage    82%    82%
===================================
  Files        15     15
  Lines      1333   1333
  Branches     10     10
===================================
  Hits       1106   1106
  Misses      217    217
  Partials     10     10


Assets stack to deploy ECR

Description
CDK is being used to deploy roles, policies and public ecr repositories
Issues Resolved
https://github.com/opensearch-project/opensearch-ci/issues/52
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Currently facing issues with docker on jenkins deployed with opensearch-ci. I am therefore yet to test out these changes by running a job

Please fix the failing tests

Fixed and added new test cases as I marked this ready for review

Thats fine, the user can manually delete it but here is a use case that if we delete the stack by mistake or something
happens, we want to be retain any ECR repo since those are public. That should never be taken down since it's one of the
release channel now.

The default is Retain

This repository is not dev let's be clear on that. :) We should assume each repository/code is production grade and contribute accordingly.

What should be the default value for this?

Thats fine, the user can manually delete it but here is a use case that if we delete the stack by mistake or something
happens, we want to be retain any ECR repo since those are public. That should never be taken down since it's one of the
release channel now.

The default is Retain. The public repos will be created with Retain. This overrides the default.

We do not want to override it. Keep the default.

Thats fine, the user can manually delete it but here is a use case that if we delete the stack by mistake or something
happens, we want to be retain any ECR repo since those are public. That should never be taken down since it's one of the
release channel now.

The default is Retain. The public repos will be created with Retain. This overrides the default.

We do not want to override it. Keep the default.

I still think we shouldn't. Let's agree to disagree

Thats fine, the user can manually delete it but here is a use case that if we delete the stack by mistake or something
happens, we want to be retain any ECR repo since those are public. That should never be taken down since it's one of the
release channel now.

The default is Retain. The public repos will be created with Retain. This overrides the default.

We do not want to override it. Keep the default.

I still think we shouldn't. It's a lot of work to delete the repository every time with every redeployment for the stack. Again, this is not what is being used, or will be used for production. Let's agree to disagree

I have strong opinion about this since assets are very crucial and are release platform we do not want accidental deletions to affect the assets. Right now we even have S3 bucket that is retained after each deployment but that is fine. https://github.com/opensearch-project/opensearch-ci/blob/main/lib/auditing/ci-audit-logging.ts#L16
We can get more input from @peterzhuamazon @zelinh @bbarani

Thats fine, the user can manually delete it but here is a use case that if we delete the stack by mistake or something
happens, we want to be retain any ECR repo since those are public. That should never be taken down since it's one of the
release channel now.

The default is Retain. The public repos will be created with Retain. This overrides the default.

We do not want to override it. Keep the default.

I still think we shouldn't. It's a lot of work to delete the repository every time with every redeployment for the stack. Again, this is not what is being used, or will be used for production. Let's agree to disagree

I have strong opinion about this since assets are very crucial and are release platform we do not want accidental deletions to affect the assets. Right now we even have S3 bucket that is retained after each deployment but that is fine as well since we want the audit to be there even if the stack is deployed. https://github.com/opensearch-project/opensearch-ci/blob/main/lib/auditing/ci-audit-logging.ts#L16 We can get more input from @peterzhuamazon @zelinh @bbarani

Update: @abhinavGupta16 is updating readme file stating the consequences of dev stack.


Tx-queue MQTT topic

Hi,
I would like to be able to track wether a downlink message has been removed from the queue and is actually transmitted and confirmed by the mote. Is it possible to add a MQTT topic that tracks this queue?

It would be possible to publish an "event" to MQTT whenever a message is removed from the queue (either because it is sent or when it expires). The second part (confirmation by the device) is something different: Confirmed Downlink.
I have created two separate issues for these features:

#158 for supporting confirmed downlink
#159 for supporting more events


make-prng does not exist

The link https://github.com/sharplispers/ironclad/blob/master/make-prng , that occurs in the readme multiple times, does not exist (any more).
Was it meant to link to https://github.com/sharplispers/ironclad/blob/master/src/prng/prng.lisp ?

The links for function names (like make-prng) are supposed to go the place in the README file where the function is documented. But it looks like GitHub's HTML renderer for README.org is generating wrong links.
However, the links work correctly in the doc/ironclad.html file generated using emacs.


Issue with Max_depth in tfdf

    df_and_nn_model = tfdf.keras.GradientBoostedTreesModel(preprocessing=regmodel_wo_head,
                                                task=tfdf.keras.Task.REGRESSION,
                                                num_trees=500,
                                                max_depth=2,
                                                max_num_nodes=-1,
                                                min_examples=5,
                                                validation_ratio=0.2,
                                                subsample=0.9,
                                                early_stopping='MIN_LOSS_FINAL',
                                                shrinkage=0.001)

and after

df_and_nn_model.compile(metrics=[tf.keras.metrics.RootMeanSquaredError()])
with sys_pipes():
    df_and_nn_model.fit(train_dataset, validation_data=val_dataset)

[INFO kernel.cc:772] Configure learner
[FATAL hyper_parameters.cc:49] Already consumed hyper-parameter "max_depth".

This was working yesterday morning but i made updates on kaggle and it throws this exception, i have no idea what it means.

Hi,
Thanks for the bug report :).
The wrong error message is displayed. Instead, the message should read TensorFlow: INVALID_ARGUMENT: "max_num_nodes" is only available with "growing_strategy=BEST_FIRST_GLOBAL". [Op:SimpleMLModelTrainer].
Regarding the (correct) error message, you should remove the line max_num_nodes=-1 (or replace it with max_num_nodes=None).

Cleanup: This has been fixed a long time ago, closing.


Fix Preview Link In Docs

Summary

Other Information

Thanks for the fix @jess! Keep 'em coming 😄


Update Angular SSR docs to match updated Angular 6 template

The current docs are at https://docs.microsoft.com/en-us/aspnet/core/client-side/spa/angular?view=aspnetcore-2.1&tabs=visual-studio#server-side-rendering
Discussed with @ryanbrandenburg

This issue was moved to aspnet/Docs#8139


OpenID Connect backend won't login new user to Zulip using Keycloak, returns "You need an invite" page

Description
After installing clean Zulip Server 7.4 on Ubuntu Server 22.04.3 LTS, and configuring OIDC backend, I can't login to realm. Keycloak redirects me to "You need an invitation to join this organization" page, after successful login.
Steps to reproduce:
Our example hostname: zulipkc, <IP_ADDRESS>
1. Install Ubuntu Server 22.04.3 LTS, update and upgrade packages
*after clean install*
sudo apt update
sudo apt upgrade -y

2. Install SDKMAN, Java 17 for Keycloak server
*as root*
apt install -y zip unzip
curl -s "https://get.sdkman.io" | bash
source "/root/.sdkman/bin/sdkman-init.sh"
sdk install java 17.0.9-zulu

3. Install and run Keycloak 22.0.4
As per https://www.keycloak.org/getting-started/getting-started-zip and https://www.keycloak.org/docs/latest/server_admin/#creating-the-account-remotely
*as root*
wget https://github.com/keycloak/keycloak/releases/download/22.0.4/keycloak-22.0.4.zip
unzip keycloak-22.0.4.zip
KEYCLOAK_ADMIN=admin KEYCLOAK_ADMIN_PASSWORD=admin ./keycloak-22.0.4/bin/kc.sh start-dev

4. Configure new realm, OpenID client on Keycloak

As per https://zulip.readthedocs.io/en/latest/production/authentication-methods.html#openid-connect

5. Create user in our realm, set password

6. Install Zulip Server 7.4
As per https://zulip.readthedocs.io/en/latest/production/install.html
*as root*
cd $(mktemp -d)
curl -fLO https://download.zulip.com/server/zulip-server-latest.tar.gz
tar -xf zulip-server-latest.tar.gz
./zulip-server-7.4/scripts/setup/install --hostname=zulipkc<EMAIL_ADDRESS>--self-signed-cert

7. Stop the server, disable HTTP Proxy, configure FAKE_EMAIL_DOMAIN
I'm disabling HTTP Proxy due to error in Smokescreen, that does not allow Zulip to connect to Keycloak in local network: https://github.com/zulip/zulip/issues/25488#issuecomment-1776106397
*as root*
root@zulipkc:/tmp/tmp.v7DKbzzbxw# supervisorctl stop all

root@zulipkc:/tmp/tmp.v7DKbzzbxw# cat /etc/zulip/zulip.conf
[machine]
puppet_classes = zulip::profile::standalone
deploy_type = production

[postgresql]
version = 15

[http_proxy]
host =

root@zulipkc:/tmp/tmp.v7DKbzzbxw# /home/zulip/deployments/current/scripts/zulip-puppet-apply

root@zulipkc:/tmp/tmp.v7DKbzzbxw# grep "FAKE_EMAIL_DOMAIN" /etc/zulip/settings.py
## set FAKE_EMAIL_DOMAIN below to a domain that Zulip can use when
FAKE_EMAIL_DOMAIN = "fake-domain.example.com"

root@zulipkc:/tmp/tmp.v7DKbzzbxw# supervisorctl restart all

8. Generate new realm
*as zulip*
./deployments/current/manage.py generate_realm_creation_link

9. Stop server, configure OIDC backend, start the server
*as root*
supervisorctl stop all

/etc/zulip/settings.py:
[...]
AUTHENTICATION_BACKENDS: Tuple[str, ...] = (
    "zproject.backends.EmailAuthBackend",  # Email and password; just requires SMTP setup
    # "zproject.backends.GoogleAuthBackend",  # Google auth, setup below
    # "zproject.backends.GitHubAuthBackend",  # GitHub auth, setup below
    # "zproject.backends.GitLabAuthBackend",  # GitLab auth, setup below
    # "zproject.backends.AzureADAuthBackend",  # Microsoft Azure Active Directory auth, setup below
    # "zproject.backends.AppleAuthBackend",  # Apple auth, setup below
    # "zproject.backends.SAMLAuthBackend",  # SAML, setup below
    # "zproject.backends.ZulipLDAPAuthBackend",  # LDAP, setup below
    # "zproject.backends.ZulipRemoteUserBackend",  # Local SSO, setup docs on readthedocs
    "zproject.backends.GenericOpenIdConnectBackend",  # Generic OIDC integration, setup below
)
[...]
SOCIAL_AUTH_OIDC_ENABLED_IDPS: Dict[str, Any] = {
    ## This field (example: "idp_name") may appear in URLs during
    ## authentication, but is otherwise not user-visible.
    "idp_name": {
        ## The base path to the provider's OIDC API. Zulip fetches the
        ## IdP's configuration from the discovery endpoint, which will be
        ## "{oidc_url}/.well-known/openid-configuration".
        "oidc_url": "http://zulipkc:8080/realms/zuliprealm",
        ## The display name, used for "Log in with <display name>" buttons.
        "display_name": "ZulipKeycloak",
        ## Optional: URL of an icon to decorate "Log in with <display name>" buttons.
        "display_icon": None,
        ## The client_id and secret provided by your OIDC IdP. To keep
        ## settings.py free of secrets, the get_secret call below
        ## reads the secret with the specified name from zulip-secrets.conf.
        "client_id": "zulipoidc",
        #"secret": get_secret("social_auth_oidc_secret"),
        "secret": "z1I0RxwirBQS8xaBZmtQbW4t3V0vBvwn",
        ## Determines whether "Log in with OIDC" will automatically
        ## register a new account if one does not already exist. By
        ## default, Zulip asks the user whether they want to create an
        ## account or try to log in again using another method.
        "auto_signup": True,
    }
}
[...]

*as root*
supervisorctl start all

10. Login, using Keycloak user

11. The bug

Zulip Server and web app version:

[ ] Zulip Cloud (*.zulipchat.com)
[x] Zulip Server 7.0+
[ ] Zulip Server 6.0+
[ ] Zulip Server 5.0 or older
[ ] Other or not sure

Thanks for the detailed report! I started a thread in the Zulip development community.

Thank You! Just tested, everything now works as intended, after toggling Invitations are required for joining this organization.
Found in section: https://zulip.com/help/restrict-account-creation#set-whether-invitations-are-required-to-join


locally tested only

Changes since last PR:
add IFF to glossary at the request of @dothebart
support passing PHRASE(...) value as an array

@vasiliy-arangodb : can you please resolve the conflicts?

@vasiliy-arangodb can you fix the merge conflicts

@vasiliy-arangodb can you please select a useful title

@fceller

can you fix the merge conflicts
committed


Module_Initializers

This pull request has been deployed to Vercel.

Latest commit:
96aac8b

✅ Preview:
https://sui-bootcamp-2024-oc7xtj69n-vbi-academy.vercel.app

🔍 Inspect:
https://vercel.com/vbi-academy/sui-bootcamp-2024/AJD977BqwReeLLZ7Pv8dpwNWpzxP

View Workflow Logs


[KeyVault] Fix flaky tests around the getDeleted* tests

The issue is likely to be that we weren't waiting for both entities to be deleted before we moved on.

/azp run js - keyvault-secrets - tests

/azp run js - keyvault-keys - tests

The test that failed failed because of a connection error 🤔

I can't reproduce the build issue locally.

Merging for now. I'll check how the automatic builds go.


❓ Preview stretched in my app but not in the example on same device

Question
I don't understand why my camera preview is stretched in my Android app but not with your example on my tablet device...
What I tried
I use the same CameraPage.tsx file in my app like in your example.
The format below props given to the camera component is the same in my app than in your example
{
"format":{
"autoFocusSystem":"contrast-detection",
"fieldOfView":85.2381275912114,
"maxFps":30,
"maxISO":1600,
"maxZoom":4,
"minFps":1,
"minISO":100,
"photoHeight":1600,
"photoWidth":2560,
"pixelFormats":[
"yuv",
"native"
],
"supportsDepthCapture":false,
"supportsPhotoHdr":false,
"supportsVideoHdr":false,
"videoHeight":960,
"videoStabilizationModes":[
"off",
"off"
],
"videoWidth":1440
}
}
VisionCamera Version
3.6.17
Additional information

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

#2142 Maybe the same issue

I found why the preview is stretched on my app.
This is because my app is locked in landscape
I have android:screenOrientation="landscape" in the AndroidManifest.xml file
So How can I solve my problem with the landscape orientation ?

Hey! Does this fix your issue maybe? https://github.com/mrousavy/react-native-vision-camera/pull/2377 (still testing)

Hey!
After 8 hours of debugging, I finally found the culprit! I fixed the preview stretching issue in this PR: https://github.com/mrousavy/react-native-vision-camera/pull/2377
If you appreciate my dedication and free support, please consider 💖 sponsoring me on GitHub 💖 so I can keep providing fixes and building out new features for my open-source libraries in my free time.

Hello, I've tested the 3.7.1 and i've still the issue.
In the version 3.7.0, I've made a patch.
I've added the below line in PreviewView.kt file
private val aspectRatio: Float get() = size.width.toFloat() / size.height.toFloat()
and replace the method onMeasure by below lines
@SuppressLint("DrawAllocation")
override fun onMeasure(widthMeasureSpec: Int, heightMeasureSpec: Int) {
super.onMeasure(widthMeasureSpec, heightMeasureSpec)
val viewWidth = MeasureSpec.getSize(widthMeasureSpec)
val viewHeight = MeasureSpec.getSize(heightMeasureSpec)
Log.i(TAG, "PreviewView onMeasure($viewWidth, $viewHeight)")

val newWidth: Int
val newHeight: Int

val actualRatio = if (viewWidth > viewHeight) aspectRatio else 1f / aspectRatio
if (viewWidth < viewHeight * actualRatio) {
    newHeight = viewHeight
    newWidth = (viewHeight * actualRatio).roundToInt()
} else {
    newWidth = viewWidth
    newHeight = (viewWidth / actualRatio).roundToInt()
}

Log.d(TAG, "Measured dimensions set: $newWidth x $newHeight")
setMeasuredDimension(newWidth, newHeight)

}

Created a PR to fix this issue once and for all: https://github.com/mrousavy/react-native-vision-camera/pull/2519 💪🚀
Thanks to everyone who sponsored me on GitHub to fix this bug!! 💖
Try the PR and see if it works (by checking out that branch and running the example app)

Hello @mrousavy, i've tested the 3.9.0-beta.2 and there's still de issue.
You can see below the preview scene and how the photo taken looks like

Is there a release planned soon that fixes this issue ?

Hello @mrousavy, i've tested the 3.9.0-beta.6 version, i have these log in my Logcat

I suppose this is why my imge is zoomed... I hope it can help you to fix the issue.

@Flocurry are you in landscape mode or portrait

I'm in landscape mode

Yea okay, only portrait works atm. See https://github.com/mrousavy/react-native-vision-camera/issues/1891 for more details.

Hello Mark.
Is it normal that the 4.0.0-beta.4 fix my issue ?
With this version, sometimes I have the below error when I open the camera
'Camera.onError(session/invalid-output-configuration): Failed to configure the Camera Session because the output/stream configurations are invalid!', { [session/invalid-output-configuration: Failed to configure the Camera Session because the output/stream configurations are invalid!]
name: 'session/invalid-output-configuration',
_code: 'session/invalid-output-configuration',
_message: 'Failed to configure the Camera Session because the output/stream configurations are invalid!',
_cause:
{ stacktrace: 'java.util.concurrent.TimeoutException: Future[androidx.camera.core.impl.utils.futures.ListFuture@1f7eb6a] is not done within 5000 ms.\n\tat androidx.camera.core.impl.utils.futures.Futures.lambda$makeTimeoutFuture$1(Futures.java:439)\n\tat androidx.camera.core.impl.utils.futures.Futures$$ExternalSyntheticLambda0.call(Unknown Source:6)\n\tat androidx.camera.core.impl.utils.executor.HandlerScheduledExecutorService$HandlerScheduledFuture.run(HandlerScheduledExecutorService.java:245)\n\tat android.os.Handler.handleCallback(Handler.java:938)\n\tat android.os.Handler.dispatchMessage(Handler.java:99)\n\tat android.os.Looper.loopOnce(Looper.java:226)\n\tat android.os.Looper.loop(Looper.java:313)\n\tat android.os.HandlerThread.run(HandlerThread.java:67)\n',
message: 'Future[androidx.camera.core.impl.utils.futures.ListFuture@1f7eb6a] is not done within 5000 ms.' } }
2024-03-04 16:52:20.201  1374-26404 Camera2-Fr...cessorBase cameraserver                         E  FrameProcessorBase: Error waiting for new frames: Connection timed out (-110)
and this error disappear if i go back and reopen the camera.

Hello @mrousavy, the above issue persists with the 4.0.0-beta.6 version

Which issue? Preview stretching or output configuration error?
I cannot reproduce the output configuration error anymore in the latest beta.

output configuration error


Include while (true) lines in coverage

Right now, flacoco does not consider while loop conditions that have a constant value true in the coverage computation.
Example:
package fr.spoonlabs.FLtest1;

public class Calculator {

	public Calculator() {
	}

	public int calculate(String op, int op1, int op2) {

                while(true) {
                    if (op.equals("+")) {
                            return op1 + op2;
                    } else if (op.equals("-")) {
                            return op1 - op2;
                    } else if (op.equals("*")) {
                            return op1 / op2;//buggy
                    } else if (op.equals("/")) {
                            return op1 / op2;
                    } else if (op.equals("%")) {
                            return op1 % op2;
                    }
                    throw new UnsupportedOperationException(op);
                }

	}
}

Output:
fr/spoonlabs/FLtest1/Calculator@-@16,1.0
fr/spoonlabs/FLtest1/Calculator@-@15,0.7071067811865475
fr/spoonlabs/FLtest1/Calculator@-@13,0.5773502691896258
fr/spoonlabs/FLtest1/Calculator@-@11,0.5
fr/spoonlabs/FLtest1/Calculator@-@12,0.0
fr/spoonlabs/FLtest1/Calculator@-@14,0.0
fr/spoonlabs/FLtest1/Calculator@-@17,0.0
fr/spoonlabs/FLtest1/Calculator@-@18,0.0

while(true) { is on line 10.
Another example where this happens is https://github.com/SpoonLabs/astor/tree/master/examples/quixbugscompiled/detect_cycle

In https://github.com/SpoonLabs/astor/tree/master/examples/quixbugscompiled/breadth_first_search it is included

Hi @andre15silva
I could be the case that we have a mistake in our test data. Which is the project example related to  Calculator  that you test?
I see that the in some example projects that use Calculator, the while(true) does not exist.
Example:
https://github.com/SpoonLabs/flacoco/blob/master/examples/exampleFL1/FLtest1/src/main/java/fr/spoonlabs/FLtest1/Calculator.java
https://github.com/SpoonLabs/flacoco/blob/master/examples/exampleFL3/FLtest1/src/main/java/fr/spoonlabs/FLtest1/Calculator.java

Hi @martinezmatias
This is a new test case based on exampleFL1, sorry for not specifying it.
I haven't added it since it won't work, but I'll open a PR in a second so you can see.

You can see https://github.com/SpoonLabs/flacoco/pull/69 now

Perfect. I see now that in the PR the example has the while(true)