Problem with global auth token in IntelliJ context

Hi,
The IntelliJ HTTP Client plugin keeps client.global env parameters between runs. httpyac doesn't it seems?
We have a setup in our structure where we have a login job setup:
### User Login
POST {{auth0_base_url}}/oauth/token
Content-Type: application/json

{
 "client_id": "{{user_auth_client_id}}",
 "client_secret": "{{user_auth_client_secret}}",
 "grant_type": "http://someauth.com/realm",
 "username": "{{user_auth_username}}",
 "password": "{{user_auth_password}}",
 "audience": "an/audience",
 "realm": "Username-Password-Authentication",
 "scope": "openid"
}
> ./login.js

with a script like this:
client.test('Token retrieved successfully', function () {
  client.assert(response.status === 200, 'Response status is not 200')
  var access = response.body.access_token
  if (access) {
    client.global.set('user_access_token', access)
  }
})

This user_access_token is stored globally in the IntelliJ plugin in a way where this env parameter is available to requests that are executed later.
So, If I run:
### List companies
GET {{my_base_url}}/api/companies
Authorization: Bearer {{user_access_token}}

it will work just fine. But If I do this in httpyac, I get the error:
...ReferenceError: user_access_token is not defined at Object.userJS...

So, my questions are:

Is this intended behaviour?
Do you have a suggestions on how I could solve this scenario for httpyac, without breaking this logic within IntelliJ?

Actually it would have been relatively easy to solve using @import and @ref but unfortunately I accidentally broke the functionality in the last release. Sorry. I will fix it again with the next version.
You need to add a name to your login method:
### User Login
# @name login
POST {{auth0_base_url}}/oauth/token
Content-Type: application/json
...

Then you just have to import and reference the login in the other method.
# @import ./login.http

### List companies
# @forceRef login
GET {{my_base_url}}/api/companies
Authorization: Bearer {{user_access_token}}
...

Normally this should be compatible with Intellij. But please let me know if this is not the case.
And the error is intended. Because all dynamic variables are stored per file. I want to be able to open two http files (in my vscode extension) that are configured against different environments and the variables should not leak to the other system. With explicit @import you disable this protection layer.

But I'm thinking about bypassing it for the Intellij scripts in general, since explicitly a global.set is called. I will have a look at it.

It is now possible to run it without modifying the .http files, but --interactive must be added to the httpyac command to prevent the program from terminating. If the calls are not in one file, they can be loaded at the same time using glob pattern (**/.http).
This procedure is necessary, because I don't want to persist the variables on the hard disk, because they could contain passwords or more.

Thank you so much for your help! The @name/@forceRef works in VSCode. And it doesn't break IntelliJ it seems. I can live with this solution.
The last --interactive fix I guess cannot be transferred to the VSC plugin?

AND: Thank you for sharing this tool with us! It is so cool and a great timesaver... :)

the --interactive fix is not needed in VS Code plugin. The intellij variables are cached as long as my extension is active and therefore it should just work there now. Prerequisite is the version 2.5.0


Test predictions length not exceeding 5

I am trying to train a number plate recognition model. The lengths can be upto 13 characters long. But the test predictions that I see after each epoch has at max 5 characters in it. What am I missing?
config/config.yaml:
defaults:
  - override hydra/job_logging: custom

processing:
  device: cuda
  image_width: 200
  image_height: 60

training:
  lr: 3e-4
  batch_size: 1
  num_workers: 10
  num_epochs: 20
  device: "cuda"

bools:
  DISPLAY_ONLY_WRONG_PREDICTIONS: false
  VIEW_INFERENCE_WHILE_TRAINING: true
  SAVE_CHECKPOINTS: false

paths:
  dataset_dir: ./dataset
  save_model_as: ./logs/crnn.pth

model:
  use_attention: true
  use_ctc: true
  gray_scale: false
  dims: 256

Also, Accuracy and Best Accuracy columns are always 0.0 for some reason.
Note: if I keep anything other than batch size 1 I am gettig the error  "Trying to resize storage that is not resizable" when calling pin_memory() on some zero-dimensional tensors although it looks like it might be a torch version issue Similar issue in ultralytics
Will appreciate any help thank you.

EDIT: I turned of CTC loss and now it's not limiting the number of characters.

Hi! It happens that I made padding automatic for cross entropy and kept it manually for CTC.
Padding is the action of making every training sample the same size (num of characters).
So if you have some plates with 6 chars, other with 8, and at max 13, choose a character to represent empty space (like "$" or "_", anything that is not one of the training characters), and use it to make every plate the same size. Example
before:
adm167
iuw68990ju90k
after:
adm167$$$$$$$
iuw68990ju90k
Then at your predictions just make a replace of "$" to ""(nothing).

Thanks. Added the following lines in utils/data_loading.py
sequence_length=15
for x in targets:
        x.extend(['$']*(sequence_length-len(x)))

Working as intented and seeing longer sequences in subsequent epochs. Thanks for the training advice.


Let a message in the old channel wich says message has been moved

Hello
All is in the title :)
Let a message wich says the message has been moved and the channel where it has been moved.

This feature isn't entirely necessary, as the bot will ping the user who sent the original message when it is moved to the new channel.


[RFE] [MPL Gaming]

Feature Request Outline for ScyllaDB Terraform Provider
Limitations in Current Version of ScyllaDB Terraform Provider:
Currently, while provisioning Scylla clusters using the Terraform provider, customers are facing a limitation: the inability to specify the drive size for the instances.
This limitation restricts the flexibility needed for different use cases, especially where specific storage requirements are essential for performance and capacity planning.
Broader Infrastructure Integration:
The ability to specify drive sizes is crucial for tailored deployments, catering to varying workloads and data volumes.
This feature would enable more precise resource allocation, optimizing cost and performance in diverse scenarios.
Feature Description
Feature Overview:
The proposed feature involves enhancing the ScyllaDB Terraform Provider to allow users to specify the drive size for each instance in a Scylla cluster.
Functionality:
Users should be able to define the drive size as a parameter within their Terraform scripts when setting up ScyllaDB clusters.
The feature should dynamically allocate the specified drive size to each instance in the cluster during the provisioning process.
Benefits:
This feature will provide greater control over resource allocation, enabling optimized storage configurations based on specific workload requirements.
It will enhance cost efficiency by allowing users to allocate storage in accordance with their actual needs, avoiding overprovisioning.

I think the storage is missing in the payload request  of  API itself.
Because i tried creating the schema by cloning the repo. It's showing in plan but not reflecting when I create.
https://cloud.docs.scylladb.com/stable/api.html#tag/Cluster/operation/createCluster
I will create the PR for the same. This needs to be fixed from backend.

Hey @susiesolis @memetics19! Thanks for reporting this, this is already on our radar since some time and we are yet to dedicate it some attention. When I have some news I will update this issue 🤗

Hi Team, Any update? ETA?

@ricardoborenstein Here's is issue number in internal repository for the reference - 10003. The progress is not tracked here.


Use once_cell from async-lock

Resolves #93 by using the OnceCell type found in async-lock. This type allows for blocking operations, just like the once_cell::Lazy type that was used earlier. In addition, async-lock does not bump our MSRV, while once_cell does.
This is a draft PR because smol-rs/async-lock#27 has not yet been released.

Clippy failure is unrelated to this pull request.

Published a new version of async-lock including https://github.com/smol-rs/async-lock/pull/27.

Published a new version of async-lock including smol-rs/async-lock#27.

Alright, I've switched to using the released version of async-lock in this PR, and marked it ready for review.


Fix css highlighting in code

CSS blocks are not highlighted in markdown code blocks or the editor

Seems to be fixed.


Typed property VisibleString::$value must be string, null used

Error I've got:
Typed property Readdle\AppStoreReceiptVerification\ASN1\Universal\VisibleString::$value must be string, null used
I updated today my apple app store notifications api integration including everything related, this lib too.
Now I'm getting api responses with the notification body v2 format:
The incoming webhook message's content (signedPayload) becomes garbage & brokes any futher processing after the base64decode is called on that... it looks like something like this:
"{"alg":"ES256","x5c":["MIIEM

KA=="]}\x1E╚øø¦\x1AYÜXÏ]\x1A[█ò\x1E\\x19HÄêò\x11Tı\x08ï\x08øø¦\x1AYÜXÏ]\x1A[█òURQ\x08Äêì\x19\fÏÿ═╠XïL═Ïì\vMîïN\x18X╠KYìN\fM\x19ÿNM┘\x19Yêï\x08Ö\x18]\x18HÄ×╚ÿØ[Ö\e\x19RY\x08ÄêÜ\x1DKÿ\\x1C\x18\Ö[Ø\e\x1EKÿ\\x1C\x08ï\x08Ö[ØÜ\ø█øY[Ø\x08ÄêöÏ[Ö\x18øÌ\x08ƒK\x08ØÖ\£┌[█êÄêîïî\x08ï\x08£┌Y█ÖY\x11\x18]\x19HÄîM╠Ä\fî══ìLìîƒD"@Æ░RÐÉÖXÚÝñÏÐó-'2x½aâ╗ÀØ░©░Æ±p¤\x15\x030<x\x14Ðâ┬¼~8ÁJØ§£\x01vòNö\x01}█\x7F&qÖÈ"

I've deleted the sensitive parts from the middle. The character decoding brokes at the marked spot: KA=="]}
then the many unicode control chars are starting to appear...
Tried to replace every text modifying function calls with the multibyte variations & tried to convert the string to other charset but didn't had any luck... :/
Maybe I should switch to the JWT based decoder from the other lib https://github.com/readdle/app-store-server-api and also replace the root cert with the newer one?

Hello,
To be honest, I've missed the point. From what you say I see that you're trying to use this library to parse server-to-server notifications from App Store. Is that correct? If yes, then it's misuse of the library, it's not intended to deal with App Store notifications. For such a purpose you have to switch to readdle/app-store-server-api instead.
This library is only intended to parse App Store receipts (those, which are provided by App Store to an app on a device).
Would be great, if you can provide further clarification on what exactly you're doing and which part of the library's behaviour is unexpected.

I'm trying to process both server-device & server-server receipts with this library, I saw there are a solution (AppStoreServerAPIReceiptExtender.php) by using the app-store-server-api too to merge the receipts into one data entity but didn't knew it's a requirement to do that until now... thanks for the heads up! closing the issue!


[Bug] API Link Reference missing for optional properties of a component

Is this a regression?
No
Description
When we create a user-defined data type or interface and reference the inputs as optional, the document references are not linked.
Expected Behaviour: The API references should be able to link to the documentation irrespective of whether the attributes are optional or required.
Suggestion: It is better to check all the edge cases with regard to these scenarios. Thank you :)
My Workspace: https://github.com/akhileshThapliyal/my-ng-doc-app
With Optional Value (Unable to link to AriaAttributes documentation page)

Without Optional Value

Regards,
Akhilesh
Please provide the exception or error you saw
No response
OS
Unix (Linux, macOS, etc.)
Browser
Chrome
Node version
20.9.0
Anything else?
Latest Version: 17.6.14

This is already fixed in NgDoc 18.

This is already fixed in NgDoc 18.

Wow good to know, we are going to have the next release soon :) I will re-test this once 18 is released.

Released in NgDoc 18


Followed instructions but doesn't work

I've followed the instructions in the article but I still can't get my runbooks to authenticate to Exchange online using a managed identity.
Despite assigning Exchange administrator role to the managed identity whenever I try connecting with that managed identity i get:
Failed
UnAuthorized (UnAuthorized)

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

ID: ba626583-a2d3-00ba-6b8e-69e89e7af74c
Version Independent ID: 4da53073-acc6-c813-8a2d-c0200f9494bf
Content: Use Azure managed identities to connect to Exchange Online PowerShell
Content Source: exchange/docs-conceptual/connect-exo-powershell-managed-identity.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Videsh-Lingabavan, this other issue talks about the need to create a new service principal.
Can you confirm or deny?

Hi Chris,
Thanks for replying, using a user managed identity, I tried following to Exchange online:
New-ServicePrincipal -AppId manageduserclientid -ServiceId manageduserobjectid
And
New-ServicePrincipal -AppId manageduserobjectid -ServiceId manageduserclientid
Either I get this when running on Azure with a runbook:
@.***
Or when running on hybrid worker:
@.***
Here is the powershell code I am running:
@.***
Many Thanks,
Videsh Lingabavan
Solutions Architect | Joseph Hage Aaronson LLP
D:  +44 (0)20 7851 8875
M: +44 (0)7809 337 321
T:  +44 (0)20 7851 8888
From: Chris Davis @.>
Sent: Thursday, June 29, 2023 5:38 PM
To: MicrosoftDocs/office-docs-powershell @.>
Cc: Videsh Lingabavan @.>; Mention @.>
Subject: Re: [MicrosoftDocs/office-docs-powershell] Followed instructions but doesn't work (Issue #11008)
@Videsh-Lingabavanhttps://github.com/Videsh-Lingabavan, this other issuehttps://github.com/MicrosoftDocs/office-docs-powershell/issues/10942 talks about the need to create a new service principal.
Can you confirm or deny?

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/office-docs-powershell/issues/11008#issuecomment-1613508129, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASL5TTSJHNX3EUV2E24FIH3XNWVOVANCNFSM6AAAAAAZYPF6YA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>Joseph Hage Aaronson LLP is a limited liability partnership registered in England & Wales with registered number OC382231. The LLP is authorised and regulated by the Solicitors Regulation Authority. A list of members of the LLP is available for inspection at the LLP's registered address: 280 High Holborn, London, WC1V 7EE, United Kingdom. The word "partner" denotes a member of the LLP or any of its associated entities, or a consultant or employee with equivalent standing and qualifications.
This email (and any attachments) is confidential and may also be legally privileged. If you are not the intended recipient, please immediately notify the sender, then delete it from your system. You should not copy it or use it for any purpose, nor disclose its contents to any other person. Although we have taken reasonable precautions to ensure that no viruses are present in this email and any attachments, you should also ensure that this email and any attachments are virus-free before opening as we cannot accept responsibility for any loss or damage arising from their use. Please also note that emails sent or received may be monitored to ensure compliance with the law and with our policies. We will never use email to notify you of a change to any bank account details we have already provided to you. If you receive any email purporting to come from this firm which seeks to do this, please contact us immediately by telephone and do not act on it or reply to it. We cannot accept responsibility for any loss if you do not follow these instructions.

@Videsh-Lingabavan Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Add Metric Buckets for Notification Latency

Description
We wanted to use notification_latency_seconds to indicate whether our alert manager was overly-delayed in sending notifications. The highest configured bucket boundary is 20 seconds, which is lower than we would like. A latency of up to 60 seconds is acceptable to us, but the histogram does not allow for alerting based on that latency threshold.
https://github.com/prometheus/alertmanager/blob/c7097ad76c07c7fc325292718115e3de9d0a125f/notify/notify.go#L309
Want
Add a 60s bucket to the metric.

Alertmanager version: 0.27.0

I think the reason the largest bucket is 20s is because the default peer timeout is 15s. If it takes more than 15s to send a notification then HA starts failing over. The other issue is depending on the Group interval, notifications might never be delivered as each time the Group interval timer elapses any in-flight notifications from the previous timer are canceled. Here is an example of that:
ts=2024-08-27T09:25:00.016Z caller=dispatch.go:166 level=debug component=dispatcher msg="Received alert" alert=[3fff2c2][active]
ts=2024-08-27T09:25:15.017Z caller=dispatch.go:526 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]
ts=2024-08-27T09:25:45.017Z caller=dispatch.go:357 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="webhook/webhook[0]: notify retry canceled after 2 attempts: context deadline exceeded"
ts=2024-08-27T09:25:45.017Z caller=dispatch.go:526 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]
ts=2024-08-27T09:26:15.018Z caller=dispatch.go:357 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="webhook/webhook[0]: notify retry canceled after 2 attempts: context deadline exceeded"
ts=2024-08-27T09:26:15.018Z caller=dispatch.go:526 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]

Given what I've written above, are you 100% sure that this is the case?

A latency of up to 60 seconds is acceptable to us, but the histogram does not allow for alerting based on that latency threshold.

I think the reason the largest bucket is 20s is because the default peer timeout is 15s. If it takes more than 15s to send a notification then HA starts failing over. The other issue is depending on the Group interval, notifications might never be delivered as each time the Group interval timer elapses any in-flight notifications from the previous timer are canceled. Here is an example of that:
ts=2024-08-27T09:25:00.016Z caller=dispatch.go:166 level=debug component=dispatcher msg="Received alert" alert=[3fff2c2][active]
ts=2024-08-27T09:25:15.017Z caller=dispatch.go:526 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]
ts=2024-08-27T09:25:45.017Z caller=dispatch.go:357 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="webhook/webhook[0]: notify retry canceled after 2 attempts: context deadline exceeded"
ts=2024-08-27T09:25:45.017Z caller=dispatch.go:526 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]
ts=2024-08-27T09:26:15.018Z caller=dispatch.go:357 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="webhook/webhook[0]: notify retry canceled after 2 attempts: context deadline exceeded"
ts=2024-08-27T09:26:15.018Z caller=dispatch.go:526 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]

A latency of up to 60 seconds is acceptable to us, but the histogram does not allow for alerting based on that latency threshold.

Given the above, I would recommend decreasing the latency of the service receiving notifications. If that is not possible, you could also consider queueing them in some intermediate service that immediately acknowledges the notification from Alertmanager, and then forwards the notification on to the intended service.

For a little more context, we're running a self-hosted LGTM stack that has experienced some occasional increased alert notification latency. This behavior occurs after a Mimir ingester pod dies, and it looks like traffic continues to be sent from the alertmanager to the dead ingester for up to five minutes until the ingester pod is replaced. During that five minutes, the cortex_alertmanager_notification_latency_seconds_bucket duration increased to around 40 seconds, which is between the 20s and +Inf bucket boundaries.
Decreasing the latency of the receiving service is not really an option here since we do not control the code of the running services and are prevented by license from changing it. Interestingly, I don't see any logs indicating alert failure from the alertmanager container, only increased latency.

Hi! Thanks for the context. I'm not sure I understand though.

from the alertmanager to the dead ingester for up to five minutes until the ingester pod is replaced

You are sending notifications from Alertmanager to the Mimir ingesters? I don't see how that's possible Alertmanager shouldn't be sending any traffic to Mimir ingesters, other than metric scrapes via an agent like Grafana Alloy.
Do you know why the latency of your receiving service is so high? Alertmanager doesn't like receiving services with high latency for the reasons mentioned. You need to be a little bit careful when choosing Group wait and Group intervals in your Alertmanager configuration as otherwise you may not receive notifications, or receive endless duplicates as the notification never succeeds from Alertmanager's perspective.
Onto the chosen buckets, we can maybe add some more, but can you share why you chose 60 and 300 seconds? How does this choice of buckets help other users? 300 seconds seems excessive?

Hi! Thanks for the context. I'm not sure I understand though.

from the alertmanager to the dead ingester for up to five minutes until the ingester pod is replaced

You are sending notifications from Alertmanager to the Mimir ingesters? I don't see how that's possible? Alertmanager shouldn't be sending any traffic to Mimir ingesters, other than metric scrapes via an agent like Grafana Alloy.
Do you know why the latency of your receiving service is so high? Alertmanager doesn't like receiving services with high latency for the reasons mentioned. You need to be a little bit careful when choosing Group wait and Group intervals in your Alertmanager configuration as otherwise you may not receive notifications, or receive endless duplicates as the notification never succeeds from Alertmanager's perspective.
Onto the chosen buckets, we can maybe add some more, but can you share why you chose 60 and 300 seconds? How does this choice of buckets help other users? 300 seconds seems excessive?

Okay, sorry about that. I reviewed my notes again, and I think conflated two different things. We are not sending notifications to ingesters, but we are using the ruler component, which does query the ingester. Because an ingester cycled, the ruler received errors querying the ingesters. There was a period of latency reported on Mimir notifications that began precisely when the ingester went down, and ended precisely as the new ingester became ready.
Does the metric reported also take into account the time for the ruler to query and get a result? If the metric only reports the duration from the alertmanager to the receiving service, then I can understand we would need to introduce a more stable alert receiver, probably a simple queue.
Our group_* settings are the defaults, so there shouldn't be anything to do there.

can you share why you chose 60 and 300 seconds?

We chose those values because 60 seconds is an acceptable delay for a critical notification to be sent, assuming that is an unusual behavior. Typical behavior would expect the latency to be much lower, less than a second. 300 seconds would help distinguish between a high priority issue and a critical one. More than a five minute delay in alerts will start to significantly hamper remediation time and increase the chances of an issue felt by customers.
Is there a better place to continue this conversation? I don't know if it really fits an Alertmanager issue, but I'm fine continuing here if it's not a problem.

If the metric only reports the duration from the alertmanager to the receiving service, then I can understand we would need to introduce a more stable alert receiver, probably a simple queue.

That is correct. The metric just measures the duration of notifications between Alertmanager and receiving service. It doesn't have anything to do with the ruler, whether Prometheus or Mimir.


Reorganize PEtab library

Closes #170
No changes in function, just refactoring.

Codecov Report

Merging #177 into develop will increase coverage by 0.61%.
The diff coverage is 80.16%.

@@             Coverage Diff             @@
##           develop     #177      +/-   ##
===========================================
+ Coverage    71.04%   71.65%   +0.61%
===========================================
  Files           11       16       +5
  Lines         1157     1182      +25
  Branches       270      270
===========================================
+ Hits           822      847      +25
  Misses         286      286
  Partials        49       49

Impacted Files
Coverage Δ

petab/core.py
90.9% <100%> (+10.9%)
:arrow_up:

petab/sbml.py
69.62% <100%> (+3.79%)
:arrow_up:

petab/parameter_mapping.py
80.46% <100%> (-0.16%)
:arrow_down:

petab/conditions.py
47.82% <47.82%> (ø)

petab/lint.py
51.41% <66.66%> (-1.66%)
:arrow_down:

petab/problem.py
71.28% <71.28%> (ø)

petab/measurements.py
80% <80%> (ø)

petab/sampling.py
89.18% <89.18%> (ø)

petab/parameters.py
91.54% <91.54%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Should this be included here or separately?

Somewhat out of scope, but those were kind of simple.

and adding new or changing stuff in a separate merge.

Agreed.

@dweindl could you shortly say what has changed, or been added, beyond the refactoring?

One moved existing functions to different places (and adapted imports). No changes in implementation. The last two commits were added documentation and format fixes for sphinx.


Conflict experienced

[Enter feedback here]
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"Ext_VMExtensionProvisioningError","message":"There was an error while installing SqlIaasExtension on the virtual machine. Please completely uninstall and reinstall the extension."}]}

I got this error while trying to create the container
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 995d55a9-b9de-2173-ff45-212ce288ddb3
Version Independent ID: 110dacb3-e317-6b0c-a574-2f5bf4b93630
Content: Create SQL Server on a Windows virtual machine in the Azure portal - SQL Server on Azure VMs
Content Source: azure-sql/virtual-machines/windows/sql-vm-create-portal-quickstart.md
Service: virtual-machines-sql
Sub-service: deployment
GitHub Login: @bluefooted
Microsoft Alias: pamela

@synenet Thank you for your feedback!  We are reviewing the documentation and will get back to you shortly.

@synenet It appears that this is an intermittent issue which can be resolved by redeploying the VM (you can see my deployment from today below).  If you consistently run into this error, please let us know which image you are using, what region you are deploying to, and what VM size you are using to help us track down the cause of the issue.

@synenet We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Hi,
I am consistently facing this issue.
Region: central india
image: Free SQL server license: SQL server 2022 developer on windows server 2022
Vm: B1 free tier
Disks: 8gb ssd (data) + 8gb ssd(log+tempdb)


ESLint improvement

As discussed, I have removed my custom ESLint config (@mephisto5558/eslint-config) from your codebase and are instead using the plugin's recommened rules.
I also removed @stylistic/eslint-plugin so no more style-only errors shall be reported.
I disabled and modified some rules to match the current style of your codebase.
You now should only see ESLint errors/warnings on things that acutally have an impact on the code, e.g. unused variables, missing parentheses, etc.

Is this getting merged anytime soon?


[gui error report] Error: ipfs daemon failed to start and produced no output (see error.log

no issue i use different hardware

@wansingcheng is this something you need assistance with or are you good to go? Can you provide more information about what you were doing when you received this error?


Give warning if it looks like the $schema is incorrect for the resources being used

... For example, if trying to use a resourceGroup resource inside a deploymentParameters.json# deployment.

Here's another example:
{
    "$schema": "https://schema.management.azure.com/schemas/2018-05-01/subscriptionDeploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "resources": [
      {
        "name": "name",
        "type": "Microsoft.Subscription/aliases",
        "apiVersion": "2020-09-01",
        "properties": {
          "workLoad": "workLoad",
          "displayName": "displayName",
          "billingScope": "billingScope",
          "managementGroupId": "[tenantResourceId('Microsoft.Management/managementGroups/', parameters('targetManagementGroupId'))]"
        }
      }
    ]
  }

... and this error (from #1060):

Template validation failed: The template resource 'mgname-test' at line '14' and column '5' is not valid: The language expression property 'location' doesn't exist, available properties are 'name, properties'.. Please see https://aka.ms/arm-template-expressions for usage details.


Add Events Button to Navbar in About.html

When you go to about us page, the "events" button disappears from the navbar. Make the Button visible and functional. Clicking on the button should redirect the user to the events page.

hey I would like to work on this can you assign me this issue

@Arthy22 done


Constructive comments

Thanks for VimTutor sequel!
I've just completed it and kept track of things that were confusing or awkward to me.
I've cleared all the following doubts already, but recording them here in case you find them useful.
The points below are meant as constructive suggestions. Can submit PRs, should you decide to greenlight any.
Introduction

could explain how to get/run vimtutor

Lesson 10.1

paragraph motions (used in V}), first non-blank ^, reselect Visual area gv are neither introduced nor explained. In particular, coming from vimtutor where all g commands introduced where shorthands for "go", gv took me a bit by surprise.
was confused by =, not understanding it expected a motion command afterwards (and therefore not understanding the difference with ==)

Lesson 10.2

:norm is not explained anywhere, and mysterious

Lesson 10.3

first time verbs made of multiple keystrokes (e.g. gu, gU) are encountered across lessons 1--10, made mentally parsing the commands confusing at first.
I would also have preferred being given generic commands in the form e.g. gu{motion} like :help does rather than specific instances, i.e. guw

Lesson 11

search and replace are already covered by lesson 4.4 of vimtutor

Lesson 13

could explicitly say what nnoremap does (for example, that it creates a mapping for normal mode). This is one of the few case in which :help does not help much
in the vimscript example, got confused by why variable names had a prefix, e.g. a:spell

Lesson 15.2

was confused re: what point (5) asked me to do. I had already learned how to create macros in Lesson 12; I guess the message here was that registers can contain anything, macros included, and that in fact recording a macro uses that particular register. What I attempted to do instead was to yank the characters qmcwHogwarts<Esc>jq into register "m and then execute them as a macro XD

Thank you for the feedback! These are all good things to consider for the next release. Just to touch on a few of your points:

Improving the explanations and command specifications are spot on and will definitely be priority for the next release(s)
vimtutor installation was opted to not be covered as the sequel assumes that users have already gone through the vimtutor program
while search and replace was covered in the original program, I wanted to highlight the different options that are available for search and replace (for example with confirmations), which I believe the original did not cover

Thanks for replying :)
Feel free to close this if there's no immediate next action!

I looked into sending PRs, but my preferred way to amend the tutorial would be to add two new lessons before the current 10, one with advanced navigation including }, {, W, B, E, ^, a second with advanced selection V, ^v, gv (since I realized that V is also never covered, neither here nor in the original vimtutor).
This would require shifting all lesson numbers and feels a little too intrusive, but I can go ahead if ok with you. Otherwise, it's my suggestion to you for next version 😄

Sorry for the late response. I think instead of two new lessons it would be better to have shorter "asides" on these topics. This would keep the lesson numbers the same. Additionally, I think keeping it to 8 lessons is good for keeping the time to complete the tutor reasonable.


only show flavor above 0.1u

About the PR
if a pill has say 0.01u carbon you could taste it
this pr fixes and only lets you taste it if its above 0.1u in the solution being ingested
Media
less than 0.1u sugar

does not taste sweet!

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
no cl no fun

Can this be configurable so that certain flavors have a higher or lower threshold for being noticed? Something like oil should be immediately tasted whereas some other flavors might be more subtle.

ok ill add thing to reagent prototype


CORE-9058 Group approval rules API

Introduces API for management of group approval rules, which determine whether a membership request will require manual approval. Includes 3 new endpoints for adding, retrieving and deleting group approval rules.
https://r3-cev.atlassian.net/browse/CORE-9058
[This change has also been tested manually]

Next time we do manual testing, can we attach some screenshots to the ticket? Or just describe it in a very short section what scenarios you followed, etc.


Remove deprecated and warn users for MySQL/MariaDB

remove deprecated code previously announced
add note in the documentation for MySQL/MariaDB support
disable tests for MySQL/MariaDB

@MVrachev we need to reapprove this before #606


[Improvement][API] The relevant configuration parameters of the zookeeper service are missing a Unit of measurement

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
缺少计量单位：

Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I want to apply it


Integrate with ePMC and PubMed Central to get papers

Add as a /use endpoint

This probably supersedes #70 so I will close that one.

I believe we can close this because we already check these sources. Re-open if there is something extra you wanted it to do that I am missing.


This somehow broke deconstructing crates with a screwdriver.

crates now have to be welded to be deconstructed so its likely a construction change that broke it


Multiple fixes

Fixed product naming
Fixed UI changes
Fixed some typos
Fixed markdown issues
Corrected names so they are now consistent with MCT PPTx and MS Learn titles / nomenclature
Every module affected.

Reviewing suggested changes

Any updates?

We're almost at the end of January. I'm curious as to what was stopping the PR be merged when first submitted almost 2 months ago?

@WigF1, thank you for pointing out all this issues.  We had made a push of some of the same changes we also identified with the product name change back on December 16th.  That said, you found many other issues that we had not identified, thank you for that, so since a merge was no longer possible, we went ahead and made those additional changes you suggested manually to 13 of the 26 md files.  I made a note about that on each of those files.  I will go ahead and close this request.  Again thank you for all your help.  And thank you for your patience.


Vertical scrolling through view controllers

Is it possible to vertical scroll through all the view controllers?

Yes it could be possible but you'll have to deal with gestures already present on controller sheets representations.
Put the expand sheet gesture only on the header and then put the GDSheetController view instance inside a UIScrollView.
Or feel free to fork the project, add it directly to the GDSheetController component and proprose a PullRequest I'll be happy to merge it.

Perhaps I am doing something wrong, but here is how I'm adding the GDSheetController to a scrollview:
in GDSheetController.m
- (id)initWithControllers:(NSArray *)arrayOfControllers
                  options:(NSDictionary *)options
{
    self = [super init];
    if(self)
    {
        //Set view controller frame correctly
        self.view.frame = self.view.bounds;
        // Adding a scrollview
        self.contentView = [[UIScrollView alloc] initWithFrame:self.view.frame];
        [self.view addSubview:self.contentView];
        self.contentView.contentSize = CGSizeMake(self.view.frame.size.width, 5000); //this must be the appropriate size!

        if(options)
        {
            self.controllerOptions = [options mutableCopy];
        }

        [self commonInitGDSheetController];
        [self addControllerSheets:arrayOfControllers];
    }
    return self;
}

and later on...
- (void)addControllerSheets:(NSArray*)arrayOfControllers
{
    NSUInteger idx = 0;

    self.numberOfSheets = [arrayOfControllers count];

    for(UIViewController *vc in arrayOfControllers)
    {
        UIViewController *embedded = vc;
        UIViewController *preview  = nil;

        if([vc isKindOfClass:[GDEmbeddedControllers class]])
        {
            embedded = [(GDEmbeddedControllers*)vc embeddedController];
            preview  = [(GDEmbeddedControllers*)vc previewController];
        }

        embedded.sheetController = self;

        GDSheetView *view = nil;

        if(preview)
        {
            view = [[GDSheetView alloc] initWithEmbeddedController:embedded
                                                 previewController:preview
                                                   sheetController:self];
        }
        else
        {
            view = [[GDSheetView alloc] initWithEmbeddedController:embedded
                                                   sheetController:self];
        }

        view.delegate = self;

        [self.sheetControllers addObject:view];

        view.defaultTopInSuperview  = [self topOriginOfSheetAtIndex:idx];
        view.top                    = view.defaultTopInSuperview;
        view.currentScalingFactor   = [self scaleFactorOfSheetAtIndex:idx];

        if(preview) [self addChildViewController:preview];
        [self addChildViewController:embedded];
        [self.contentView addSubview:view];
        [self.contentView sizeToFit];
        if(preview) [preview didMoveToParentViewController:self];
        [embedded didMoveToParentViewController:self];

        idx++;
    }
}

I am trying to accomplish something like the android recent applications menu in Lollipop
I am not sure what I am doing wrong in the scroll view. Can you help me a bit with the scroll view?
The gestures I have already fixed to my needs.


Visualize max WLs and WL timesteps from NC9 Nov 2021 ADCIRC run in SMS

Summary presentation with model run background, measured water levels from Yacht Basin, and modeled maximum water levels. Timing of peak water levels does not seem to match between ADCIRC and measured values
20220608_MaxEle&Fort63.pptx

CD: To investigate your questions, we will want to look at time series of the water levels and winds at observation stations, to see how the quality of the input tides and atmospheric forcing.  I checked the NAM site, and I am confident that their atmospheric products are relative to UTC.  But they are low resolution -- (about 0.1 degrees in space, 3 hr in time), so it is possible that they are not representing (or 'smearing') an atmospheric driver.

CD: In the meantime, if you have time to process the bad simulation, then I will be curious to see time series comparisons for the winds -- the error in the tidal forcing should not affect the winds.

Thomas, I put an updated set of model results on the cluster here:
/share/jcdietri/For/Thomas/220610-NC9-Nov2021/
Can you please (a) copy these files into your archive, and then (b) investigate whether they are a better match to the extreme water levels during Nov 2021?

What datum are the modeled water levels relative to?

ADCIRC misses long term drivers like the gulf stream. May have to adjust for this, eg use a constant offset

CB Nodes used for WL timeseries interpolation circled in blue. Not convinced that these are the 3 closest nodes to the sensor (circle in red), but they are in the right general area


pull in main to facilitate gz namespcae updates

Hi @adityapande-1995,
Targeting your PR to pull in main with recent namespace changes and resolve conflicts. I also took the liberty and updating batt specific files with the gz namespace in place of the deprecated ignition namespace. Should be ready to merge with main with these updates.
Thanks,
Ben

The YoYo, PitchMass, DepthVBS, and PitchDepthVBS tests are failing, but I'm not sure how they can be related to this PR.

Hmm... interesting. I'll try to reproduce on my local

I was able to reproduce the YoYo test error locally with this repo on main

test_yoyo_circle_mission passes with my local build, so perhaps an issue with the Docker image I created? I'll look into that next.

It is also failing on the DVL PR now : https://github.com/osrf/lrauv/pull/212 cc @hidmic

The CI is failing due to LRAUV State msg ABI difference on the lrauv app side. I'l go ahead and merge this one into the battery msg branch.


Cannot open era5-lan nc file.

Error code is Error: Netcdf(2).
I tried open with options but no luck, and I can read it from python.
Below is the nc file(zipped to upload to github) I want to read.
2009.01-04.nc.zip

I am afraid I can't replicate this, I can read this using the example ncdump in this repo. Can you read other netCDF files using the crate? Could you share any details on what os/hdf5/netcdf you have installed?

I am afraid I can't replicate this, I can read this using the example ncdump in this repo. Can you read other netCDF files using the crate? Could you share any details on what os/hdf5/netcdf you have installed?

Thanks, I use the static feature (netcdf = { version = "0.8", features = ["static"] }).
And netcdf cannot read those nc file, which works on other nc files not from era5.
I tested on Windows (with msys2) and it fails, but at my linux machine with the same Cargo.toml, it can read it.
I'll test more.

@magnusuMET
it is the path string encoding problem, I guess.
I test under rust 1.74.1 and with msys2 installed.
stable-x86_64-pc-windows-gnu (default)
rustc 1.74.1 (a28077b28 2023-12-04)

main.rs
use netcdf;
fn main() {
    let file = "℃.nc";
    let _ = netcdf::open(file);
}

Then I modified source of netcdf (file.rs) to print output of get_ffi_from_path.
In Windows:
println!("{:?}", f.as_bytes_with_nul());

In Linux:
println!("{:?}", f);

When file = "℃.nc", output of windows(Failed to read) is [226, 132, 131, 46, 110, 99], output of linux(Read successfully) is [226, 132, 131, 46, 110, 99, 0]
When file = "1.nc", output of windows(Read successfully) is [49, 46, 110, 99], output of linux(Read successfully) is [49, 46, 110, 99, 0]
I searched in Unidata's repo and found: Error opening netcdf in path with "special" characters. But I still dont know how to fix it.

Some special handling of "weird" filenames is included, but for linux only since that is what I have access to. I see python has some encoding handling in [0], maybe we need to add something similar?
[0] https://github.com/Unidata/netcdf4-python/blob/7ead17a38f0c7ae690c90389d46ef2784064e915/src/netCDF4/_netCDF4.pyx#L2369

But what does sys.getfilesystemencoding() return on Windows? ascii? utf-16?

But what does sys.getfilesystemencoding() return on Windows? ascii? utf-16?

output of sys.getfilesystemencoding() on Windows is 'utf-8'.


is there a standardised way to interact with LLMs

Hi.
has anyone come up with a unified, standard API to talk to all LLMs? (LLaMA, Bard, OpenAI etc)? if not, it would be helpful to start defining that as a community and build the compatibility layers for that. keen to hear your thoughts.

OpenAI API is the de facto standard in this area. That's why I start this project to call custom LLM in OpenAI way. Any suggestion?

I agree that currently OpenAI is considered the standard. However, I don't think OpenAI is committed to keep the API specification intact for the foreseeable future.
My suggestion is to start a community-led group to codify a standard, and start building a compatibility shim between major models (OpenAI, LLaMA etc) according to said model. We can always start with OpenAI's reference and build on top of it.

Agree, that would be very useful😎

cool :) so how should we start this? need to start a group of some sort and start exchanging ideas on what it would look like.

sounds good. I can help with Bard.


[INFINITY-1703] Track TaskIPs so that latest IP of a task is always available.

Often we need the IP of a task.
However, sometimes TaskStatuses we receive do NOT have the IP.

@benclarkwood:  Both DefaultStateStore in storeStatus and DefaultScheduler now have "find the a matching TaskID" logic.  This should be factored out.

retest this please

(There is a new bug in Marathon behavior that #1100 fixed)

🤔 Well, the thing I thought I fixed is broken. So... I guess I didn't fix it yet.

retest this please

retest this please

retest this please (ccm)

Is that two CCMs in a row?

yep! it was [2/2] failed or similar.

failed on frameworks.cassandra.tests.test_recovery.test_node_replace_replaces_node (apparently the github status update itself failed). flakes with that test may soon be fixed via #1107 ?

Am hopeful that #1107 will have helped... Retrying.

retest this please


remove prod

Docs Build status updates of commit 6229e4a:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 6229e4a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

DocsLegal/docfx.json
:white_check_mark:Succeeded

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


Python dynamic module loading

I'm not sure if this is the appropriate place to ask, but I'm wondering why the Python runtime spawns a subprocess instead of dynamically importing the function.
Did you decide one is better than the other for Python?

hi @johnhoffman sorry for the late reply - if you're still curious there are two version of the python runtime python3action and actionloop-python-v3.7. The former does what you're suggesting and the latter spawns a process. We have normalized many of the runtimes on the latter mechanism because it allows us to reuse a runtime proxy implemented in go. Performance benchamrks also showed this is still faster than the intra process function execution.


provide downloadable javascript builds

Imported from Google Code issue #263 created by cberzan on 2013-04-18T02:40:12.000Z:

Please provide the JavaScript version of libphonenumber as a download. (I only see Java versions on the downloads page.)
Currently there is too much friction in building the JavaScript version by hand. See, for example, https://code.google.com/p/libphonenumber/issues/detail?id=262.

Duplicate of https://github.com/googlei18n/libphonenumber/issues/683


Certeju De Sus, Hunedoara - Is not written correctly

ENVIROMENT: ui.orase
URL: https://peviitor-ro.github.io/ui.orase/
Browser: Chrome
Device: Laptop
OS: Windows 7
STEPS TO REPRODUCE:

Open URL in browser
Type "Certeju De Sus" in the search bar
Select Certeju De Sus, Hunedoara from the list

EXPECTED RESULTS:
The name of the locality written in the search bar is written correctly based off the law nr. 167/1929
https://legislatie.just.ro/Public/DetaliiDocument/189
ACTUAL RESULTS:
The name of the locality written in the search bar is not written correctly based off the law nr. 167/1929

Fixed! Poti retesta


haproxy zombie processes

#245
What do you think of the implementation? Are there any risks with this approach?
How would you test the reap functionality?

I added a test and, unless I misunderstood this change, it failed. Can you please take a look at https://github.com/vfarcic/docker-flow-proxy/blob/master/integration_tests/integration_swarm_test.go#L130 and check whether reaping does not work or I made a wrong test.

Thanks for the test. It looks like a test which should validate the functionality. Sadly I still can't get the integration test running on a mac. Do you have any experience with that?

I'm running the tests locally on my Docker for Mac. Did you follow http://proxy.dockerflow.com/feedback-and-contribution/#the-complete-cycle-unit-build-staging ?

First of all, I'm sorry for not responding to this thread earlier. I somehow missed to follow up.
A few days ago @carlhals submitted PR #328 and it passed all the tests.


Issues with download_udpos

Hi,
I'm currently running the download script for XTREME. I'm running into some issues with the downloading and preprocessing of the UD data, and wanted to check if some of these are an issue with my setup or an issue with the provided code.

The script uses the third party ud-conversion-tools file $REPO/third_party/ud-conversion-tools/conllu_to_conll.py. However, the script contains the line
from lib.conll import CoNLLReader
whereas the lib folder from ud-conversion-tools has not been included in the $REPO/third_party/ud-conversion-tools folder. I was able to get around this by separately git cloning from https://github.com/coastalcph/ud-conversion-tools and adding that to my PYTHONPATH
After correcting for the above, it looks like a good number of the preprocessing commands for UD are able to work, but a small number still run into some errors (or warnings). Are these to be expected? (These are just messages I grabbed during my run)

Case 1.
python /mypath/xtreme/third_party/ud-conversion-tools/conllu_to_conll.py /mypath/xtreme/download//udpos-tmp/ud-treebanks-v2.5/UD_Dutch-Alpino/nl_alpino-ud
-train.conllu /mypath/xtreme/download//udpos-tmp/conll//nl//nl_alpino-ud-trai
n.conll --lang nl --replace_subtokens_with_fused_forms --print_fused_forms
Traceback (most recent call last):
  File "/mypath/xtreme/third_party/ud-conversion-tools/conllu_to_conll.py", l
ine 53, in <module>
    main()
  File "/mypath/xtreme/third_party/ud-conversion-tools/conllu_to_conll.py", l
ine 41, in main
    orig_treebank = cio.read_conll_u(args.input)#, args.keep_fused_forms, args.lang, POSRANKPRECEDENC
EDICT)
  File "/mypath/xtreme/ud-conversion-tools/lib/conll.py", line 350, in read_conll_
u
    token_dict = {key: conv_fn(val) for (key, conv_fn), val in zip(self.CONLL_U_COLUMNS, parts)}
  File "/mypath/xtreme/ud-conversion-tools/lib/conll.py", line 350, in <dictcomp>
    token_dict = {key: conv_fn(val) for (key, conv_fn), val in zip(self.CONLL_U_COLUMNS, parts)}
  File "/mypath/xtreme/ud-conversion-tools/lib/conll.py", line 26, in parse_deps
    return [(int(pair[0]), pair[1]) for pair in dep_pairs]
  File "/mypath/xtreme/ud-conversion-tools/lib/conll.py", line 26, in <listcomp>
    return [(int(pair[0]), pair[1]) for pair in dep_pairs]
ValueError: invalid literal for int() with base 10: '5.1'

Case 2.
Not a tree after fused-form heuristics: غزة 15 - 8 ( اف ب ) - حذرت الجبهة الشعبية لتحرير فلسطين وحزب
الخلاص الوطني ، الاسلامي القريب من حركة حماس ، من اية محاولات او اف منه الى وكالة فرانس برس الى " ضرو
رة الحفاظ على المصداقية في هذا الخصوص والا فان الدولة ستتحول الى ورقة استهلاكية تستخدم في المناسبات "
 .

Case 3.
Traceback (most recent call last):
  File "/mypath/xtreme/third_party/ud-conversion-tools/conllu_to_conll.py", l
ine 53, in <module>
    main()
  File "/mypath/xtreme/third_party/ud-conversion-tools/conllu_to_conll.py", l
ine 48, in main
    s.filter_sentence_content(args.replace_subtokens_with_fused_forms, args.lang, current_pos_precede
nce_list,args.remove_node_properties,args.remove_deprel_suffixes,args.remove_arabic_diacritics)
  File "/mypath/xtreme/ud-conversion-tools/lib/conll.py", line 219, in filter_sent
ence_content
    self._keep_fused_form(posPreferenceDict)
  File "/mypath/xtreme/ud-conversion-tools/lib/conll.py", line 179, in _keep_fused
_form
    deprel = self[localhead][ext_dep]["deprel"]
KeyError: 3

Thanks!

Hi @zphang
Thanks a lot for pointing out the issues w/ detailed cases!

The .gitignore file made me miss the lib folder. I've just uploaded my modified conll.py file. For the particular error in your case1, there are a very small number of words in some files that have non-integer indexes. So I filtered them out by:
https://github.com/JunjieHu/xtreme/blob/develop/third_party/ud-conversion-tools/lib/conll.py#L28

That warming is because the heuristic conversion breaks down a single tree structure for the sentence. Since we are doing mostly on the POS tagging task, that should be fine. I also commented that warming.
https://github.com/JunjieHu/xtreme/blob/develop/third_party/ud-conversion-tools/lib/conll.py#L229

If you use my uploaded file, there should not be such errors. I just test the download script one more time in a fresh new machine.


create or new results in "Error: user aborted"

Summary
Attempting to run gopass create or gopass new results in the error Error: user aborted
Steps To Reproduce
Simply run gopass create or gopass new
➜ gopass create

Error: user aborted
➜ gopass new

Error: user aborted
➜

Expected behavior
Either proper functionality, or a better error message if the syntax of the create/new command was wrong.
Environment

OS: Ubuntu 18.04
OS version: Linux labs-319 4.15.0-43-generic #46-Ubuntu SMP Thu Dec 6 14:45:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
gopass Version:

gopass 1.8.4 (47d1c43d 2019-01-02 14:30:40) go1.10.7 linux amd64
<root>     -  gpg 2.2.3 - noop 0.0.0 -   fs 0.1.0
Available Crypto Backends: gpgcli, openpgp, plain, xc
Available RCS Backends: gitcli, noop
Available Storage Backends: fs, inmem

Installation method: using docker

Additional context
N/A

Thanks for reporting this issue.
This does not happen when I try to reproduce it.
Maybe the curses UI isn't working properly in Docker? Could you try to run it outside of docker, please?
But the error message should definitely be better.

Sorry I haven't had a chance to update on this. Unfortunately, my current setup is such that I can only run this in a docker container. I'll close this for now, and reopen in future if I manage to reproduce it outside of docker.


Keep Allow_Administrator_account_lockout out of schemas

We need to make sure "Allow administrator Account lockout" never ends up in the schema, since it's missing from the baseline local configuration manager. To generate the resources, some information about this type of account policy setting, but we don't want it to end up in the actual schema. 1.2.3 is specific to Win 11 and it may not be the same recommendation # in the future. This was added to $script:AccountPolicySettings = @{} in CISDSCResourceGeneration.psm1 to fix the resource generation issue with Win 11.
if($ExcludeList -notcontains '1.2.3' -and $LevelOne){
AccountPolicy "1.2.3 - (L1) Ensure Allow Administrator account lockout is set to Enabled" {
Allow_Administrator_account_lockout = 1
Name = 'Allow_Administrator_account_lockout'
}
}

Per our offline discussion:
This is a newer account policy that is missing from the pinned version of SecurityPolicyDSC. It might be supported in the latest release but that module doesn't seem to be in active development anymore. There is no stable release past what we have pinned just a 3yr old release candidate for v3.
We're not the only ones missing the support.
https://github.com/dsccommunity/SecurityPolicyDsc/issues/193


No option to turn off sound for notifications

First up: thank you so much for this app, it's incredibly useful!
It would be nice to be able to turn off the notification sound, but there is no Play sound for notification option in the notification preferences panel:

This is how it normally looks:

Strange, could you include what macos version you are on?
I have the option on MacOS 13.3.1 (22E261)


Clarify documentation for complex.rst - Ensuring Object Creation

Updated documentation per issue #279
Updated complex.rst Ensuring Object Creation section to make the behavior clearer. Explicitly calling out which method raises the error.

#closes #279


Set from_email as string instead of DjangoSetting object

Problem
gqlauth/models.py --> class UserStatus --> def send(...):
    ... previous code ...

    return send_mail(
            subject=_subject,
            from_email=app_settings.EMAIL_FROM,
            message=message,
            html_message=html_message,
            recipient_list=(recipient_list or [getattr(self.user, USER_MODEL.EMAIL_FIELD)]),
            fail_silently=False,
        )

The send_mail method sets from_email to be an object of class DjangoSetting (This is set in another file where the app_settings.EMAIL_FROM is pointing to)
This causes a problem with 3rd party email backend which expects to get the from_email as a string.
Solution proposal
modify the line from_email=app_settings.EMAIL_FROM to from_email=app_settings.EMAIL_FROM.value
The above solution fixes the issue at least in my case. Is there any reason why we should not use this solution?

Is there any reason why we should not use this solution?

Yes, users wouldn't know that GqlAuthDjangoSettings is expecting a DjangoSetting also you are not allowed to access the settings object until all apps are ready.
This abstraction is required in order to retrieve other django settings after the django settings object has been defined. If you have another solution I would love to hear.

BTW I can't understand how does a 3rd app gets our  DjangoSetting object?

I am using anymail as an email backend in my Django project. When I try to send password reset email with gqlauth, the send_email (gqlauth/models.py line 50) gets called with attribute from_email=app_settings.EMAIL_FROM which is the same as GqlAuthSettings.EMAIL_FROM=DjangoSetting("DEFAULT_FROM_EMAIL"). This is where the DjangoSetting object is sent outside of gqlauth's scope.
This send_email function call is then passed through django/core/mail to anymail's email handler. The anymail's email handler tries to handle the from_email field, but it cannot read the value because it did not expect to get a DjangoSetting object in the from_email field.

I can see two possible solutions for this case:

Change the gqlauth/models.py line 52, where the values for an email message are sent for an email handler (email backend):

from_email=app_settings.EMAIL_FROM, #OLD
from_email=app_settings.EMAIL_FROM.value #NEW

(not optimal solution IMO) Add a str method for DjangoSetting class:

    def __str__(self) -> str:
        return str(self.value)

and also make some changes for anymail package so that it accepts also an objects in case where the object has a str method. I would not like to need to change two packages for just one simple use case :D
So, the solution 1 is not going to change the GqlDjangoSettings at all. The GqlDjangoSettings would still have the DjangoSetting object in use.

@kurttsam Can you checkout into #391 and see if it is fixed?

Thanks, works fine!

feel free to close this if it is fixed.


[Monitor] Revised Alert Rule Commands

Closes #3135. Closes #3100. Closes #3059. Fixes #3052.
cc/ @jefffall

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[X] Each command and parameter has a meaningful description.
[X] Each new command has a test.

(see Authoring Command Modules)

monitor alert create
Command
    az monitor alert create: Create a metric-based alert rule.

Arguments
    --condition         [Required]: The condition expression upon which to trigger the rule.
        Usage:   --condition "METRIC {>,>=,<,<=} THRESHOLD {avg,min,max,total,last} PERIOD"
        Example: --condition "Percentage CPU > 60 avg 1h30m".
    --name -n           [Required]: Name of the alert rule.
    --resource-group -g [Required]: Name of resource group. You can configure the default group
                                    using `az configure --defaults group=<name>`.
    --description                 : Free-text description of the rule. Defaults to the condition
                                    expression.
    --disabled                    : Create the rule in a disabled state.  Allowed values: false,
                                    true.
    --location -l                 : Location. You can configure the default location using `az
                                    configure --defaults location=<location>`.
    --tags                        : Space separated tags in 'key[=value]' format. Use "" to clear
                                    existing tags.

Action Arguments
    --action -a                   : Add an action to fire when the alert is triggered.
        Usage:   --add-action TYPE KEY [ARG ...]
        Email:   --add-action email<EMAIL_ADDRESS><EMAIL_ADDRESS>        Webhook: --add-action webhook https://www.contoso.com/alert apiKey=value
        Webhook: --add-action webhook https://www.contoso.com/alert?apiKey=value
        To specify multiple actions, add multiple --add-action ARGS entries.
    --email-service-owners        : Email the service owners if an alert is triggered.  Allowed
                                    values: false, true.

Target Resource Arguments
    --target            [Required]: Name or ID of the target resource.
    --target-namespace            : Target resource provider namespace.
    --target-parent               : Target resource parent path, if applicable.
    --target-type                 : Target resource type. Can also accept namespace/type format (Ex:
                                    'Microsoft.Compute/virtualMachines)').

Examples
    Create a High CPU-based alert on a VM with no actions.
        az vm alert rule create -n rule1 -g <RG> --target <VM ID> --condition "Percentage CPU > 90
        avg 5m"

    Create a High CPU-based alert on a VM with email and webhook actions.
        az vm alert rule create -n rule1 -g <RG> --target <VM ID> \
            --condition "Percentage CPU > 90 avg 5m" \
            --action email<EMAIL_ADDRESS><EMAIL_ADDRESS>--email-service-owners \
            --action webhook https://www.contoso.com/alerts?type=HighCPU \
            --action webhook https://alerts.contoso.com apiKey=<KEY> type=HighCPU

monitor alert update
Command
    az monitor alert update: Update a metric-based alert rule.
        To specify the condition:
            --condition METRIC {>,>=,<,<=} THRESHOLD {avg,min,max,total,last} PERIOD.

Arguments
    --description         : Free-text description of the rule.
    --enabled
    --tags                : Space separated tags in 'key[=value]' format. Use "" to clear existing
                            tags.

Resource Id Arguments
    --ids                 : One or more resource IDs (space delimited). If provided, no other
                            'Resource Id' arguments should be specified.
    --name -n             : Name of the alert rule.
    --resource-group -g   : Name of resource group. You can configure the default group using `az
                            configure --defaults group=<name>`.

Action Arguments
    --add-action -a       : Add an action to fire when the alert is triggered.
        Usage:   --add-action TYPE KEY [ARG ...]
        Email:   --add-action email<EMAIL_ADDRESS><EMAIL_ADDRESS>        Webhook: --add-action webhook https://www.contoso.com/alert apiKey=value
        Webhook: --add-action webhook https://www.contoso.com/alert?apiKey=value
        To specify multiple actions, add multiple --add-action ARGS entries.
    --email-service-owners: Email the service owners if an alert is triggered.  Allowed values:
                            false, true.
    --remove-action -r    : Remove one or more actions.
        Usage:   --remove-action TYPE KEY [KEY ...]
        Email:   --remove-action email<EMAIL_ADDRESS><EMAIL_ADDRESS>        Webhook: --remove-action webhook https://contoso.com/alert https://alerts.contoso.com.

Condition Arguments
    --aggregation         : Type of aggregation to apply based on --period.  Allowed values: avg,
                            last, max, min, total.
    --condition           : The condition expression upon which to trigger the rule.
        Usage:   --condition "METRIC {>,>=,<,<=} THRESHOLD {avg,min,max,total,last} PERIOD"
        Example: --condition "Percentage CPU > 60 avg 1h30m".
    --metric              : Name of the metric to base the rule on.  Values from: az monitor metrics
                            list-definitions.
    --operator            : How to compare the metric against the threshold.  Allowed values: <, <=,
                            >, >=.
    --period              : Time span over which to apply --aggregation, in nDnHnMnS shorthand or
                            full ISO8601 format.
    --threshold           : Numeric threshold at which to trigger the alert.

Target Resource Arguments
    --target              : ID of the resource to target for the alert rule.
    --target-namespace    : Target resource provider namespace.
    --target-parent       : Target resource parent path, if applicable.
    --target-type         : Target resource type. Can also accept namespace/type format (Ex:
                            'Microsoft.Compute/virtualMachines)').

Codecov Report

Merging #3373 into master will increase coverage by 0.14%.
The diff coverage is 85.43%.

@@            Coverage Diff            @@
##           master   #3373      +/-   ##
=========================================
+ Coverage   70.66%   70.8%   +0.14%
=========================================
  Files         391     392       +1
  Lines       25255   25442     +187
  Branches     3835    3875      +40
=========================================
+ Hits        17847   18015     +168
- Misses       6272    6276       +4
- Partials     1136    1151      +15

Impacted Files
Coverage Δ

...ure-cli-core/azure/cli/core/commands/parameters.py
72.47% <100%> (ø)
:arrow_up:

...onitor/azure/cli/command_modules/monitor/params.py
100% <100%> (ø)
:arrow_up:

...itor/azure/cli/command_modules/monitor/commands.py
100% <100%> (ø)
:arrow_up:

...or/azure/cli/command_modules/monitor/validators.py
48.38% <100%> (+34.87%)
:arrow_up:

...-monitor/azure/cli/command_modules/monitor/help.py
100% <100%> (ø)
:arrow_up:

...ure/cli/command_modules/monitor/_client_factory.py
77.27% <66.66%> (+35.6%)
:arrow_up:

...onitor/azure/cli/command_modules/monitor/custom.py
71.91% <75%> (+4.63%)
:arrow_up:

...nitor/azure/cli/command_modules/monitor/actions.py
79.1% <79.1%> (ø)

Continue to review full report at Codecov.

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

@jefffall this change will be included in the next release (End of May).  Thanks for your feedback!


[AutoPR machinelearningservices/resource-manager] typo: machinelearningservices/resource-manager/Microsoft.MachineLearningServices

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4706

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/717


Add 13 tickets plus update .gitignore

This adds 13 feature requests I've opened with Mosyle.
Please let me know if the formatting looks OK.
It also includes updates to the .gitignore file in the repo.

Fabulous stuff. Adding now


Pre-trained waveglow not producing results like samples

I've been playing around with tacotron and it's such a great piece of work but I'm guessing I'm missing something because I can't get it to produce similar results as demonstrated with https://github.com/NVIDIA/waveglow.
I'm using pre-trained waveglow_old for testing but I'm getting a lot of artifacts.
Here's a sample of what I'm getting https://picosong.com/wJMxP/

What parameter sigma did you use to produce this sample?

@rafaelvalle 0.666. It's pretty much exactly as in https://github.com/NVIDIA/tacotron2/blob/master/inference.ipynb.

What's your Taco2 val loss?

I know this is incredibly unhelpful but I don't know because I submitted this 8 days ago and colab resets itself day after so I lost everything I had done.
However I didn't change anything from inference.ipynb at the end and I was still getting the result I linked above. So any ideas how I could improve the results? I looked at hparams but without understand the code, the options are a bit of blackbox knobs ;)

You will need to anneal the learning rate during training when the validation loss starts to plateau otherwise you will not reach lower validation losses than having a fixed learning rate.


TAS: Add sanity e2e test for JobSet

What would you like to be cleaned:
To add a basic sanity e2e test for JobSet as we have for Job.
Why is this needed:
To make sure the integration for the JobSet CRD works e2e.

/cc @mbobrovskyi @tenzen-y

/assign


Sample link is invalid

The link for "NumericUpDown Custom Control with Theme and UI Automation Support Sample" (https://go.microsoft.com/fwlink/?LinkID=160025) is invalid.

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

ID: f082db11-a47c-222d-e0d4-112b9033249b
Version Independent ID: 401d14cf-63e7-7fd1-618a-a7aca0f205cb
Content: UI Automation of a WPF Custom Control
Content Source: docs/framework/wpf/controls/ui-automation-of-a-wpf-custom-control.md
Product: dotnet-framework
Technology: dotnet-wpf
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

I'm not familiar with that topic, so not sure if This sample is the one meant by that link.

It's a version of it, @Youssef1313. Thanks for finding it!

@rpetrusha, Thanks for confirming. I've opened https://github.com/dotnet/docs/pull/14188 to fix it.

Thanks @Youssef1313. I've also updated the internal fwlink to point to the same location.

Closing this one as fixed as seen on the following commit:
https://github.com/dotnet/docs/commit/240050fd40de6381f109d82788d6bf3dea5aeeeb#diff-8ad728aa3a36c439a0448df0c0182ad5


Login: should arrive on account landing page

Reproduce:

Open login page https://<IP_ADDRESS>:8443/posters/login
Provide valid credentials
Submit

Issue: User gets sent to homepage, only sees green success message on top. Since the login doesn't work via modal or flyout, I expected to arrive on the account landing page here.

Issue remains for poster version 2.1.


Slow on large documents

When editing larger documents it struggles a bit. A possible solution would be to implement a diff algorithm.

How large is your document, and how complex is your behavior? Can you give repro steps?
Since we use persistent data and aggressively avoid re-rendering, the size of the document should generally not be a problem unless there is a bottleneck somewhere.

Also, some more details on the structure of your document would help pinpoint any issues. Do you have especially long paragraphs?

good

Sorry that I haven't posted an example yet, attempting to figure out exactly the root cause so that I can avoid giving extra work. Once I can pin point i will create a paste bin with an example.

Awesome, thanks!

I will close the issue for the time being, since I identified that the bottleneck was on integration layer not on draftjs iteself.

Cool, thanks for following up.

Creating new content with 9999 "hello" words in both Draft homepage and Facebook Notes, then editing text. The slow performance appear.
I tried with contenteditable, it's okay when editing.
How can I void this? :smile:


Image verification links wrong

The links to CoreOS Image Signing Key - https://coreos.com/security/image-signing-key in os/verify-images.md page are getting a 404:

Whoops, those resources should not have been removed.  We'll get them restored today.  Thanks for the report!

This is fixed now.  Thanks again for reporting.


Added management command to change zone_length of a pzone

New

set_pzone_zone_length management command added that allows you to change the length of a pzone

@vforgione here's the moved and updated code if you can review.

:+1:


Fresh up our deps

This PR will keep our boilerplate still up to date.

@strues thanks

@diondirza Thanks for this! Curious if you tried upgrading to Webpack 3 as well as part of this update?


Typo in Readme regarding AL 2.0

Apache Software License 2.0.
should be
Apache License 2.0.
[The License is not restricted to software]

Appears reasonable. I think @hyandell can give a more thorough answer.

See http://www.apache.org/licenses/#2.0

Resolved :)


Update minimum rust compiler version

Update README to include MRCV, not sure what is the minimum right now but it should be 1.51. Please do test it before updating the README

Confirmed 1.50.0 and below will produce error:
➜  justbus-rs git:(master) cargo build
error: failed to download `actix-http v3.0.0-beta.10`

Caused by:
  unable to get packages from source

Caused by:
  failed to parse manifest at `/home/ruqqq/.cargo/registry/src/github.com-1ecc6299db9ec823/actix-http-3.0.0-beta.10/Cargo.toml`

Caused by:
  feature `resolver` is required

  this Cargo does not support nightly features, but if you
  switch to nightly channel you can add
  `cargo-features = ["resolver"]` to enable this feature

Will update README.


ANC Subs Visit - Next Steps - Continued Illness [6h]

Issue
Since this is a subsequent visit, a patient could have been diagnosed with an illness in their first visit, been prescribed medication, and then still come up positive in a lab or hit the markers for the same illness in the subsequent visit. In this scenario, the next step will be to refer them to the hospital.
(First Visit) Diagnoses with Illness ---> (Next Visit) Diagnosed with same illness = refer to hospital for further evaluation.
EXCEPTIONS to this rule are:

HIV (See #264 for more information)
Hypertension (See #256 for more information)
Diabetes (waiting on information from Valens)
Anemia (See #265 for more information)
Syphilis (See #266 for more information)

To Do:

Create logic for next steps for second diagnosis of same illness

@jeanette-pelizzon
About Since this is a subsequent visit, a patient could have been diagnosed with an illness in their first visit, been prescribed medication, and then still come up positive in a lab or hit the markers for the same illness in the subsequent visit:

We need a list of all possible illnesses that meet this criteria. Note that there are illnesses that get diagnosed only on subsequent visit, and patient is prescribed medication. Should they be included as well?

The condition is that patients was diagnosed and got medications at previous visit? What do we do in case patient was diagnosed at first visit, got medication, then was diagnosed again on second visit, and was sent to hospital, and then was diagnosed again on third visit? Not sure this can happen though (need an answer to 1 before we can anlayse this).

It is everything except those listed in the exceptions in the main issue
If they are sent to the hospital and then come back and still have the issue I would assume we send them back to the hospital as it means the health center isn't equipped to handle the issue at that level. Tagging @leowea here in case we don't get to this in our meeting.

@jeanette-pelizzon
Another question - do we apply this logic only for illnesses that can be diagnosed at first nurse encouter, or, both first and subsequent encounters.

Analysis:
Diagnoses that are treated with medication on initial nurse encounter:

Hypertension
Malaria
Syphilis
HIV
Anemia

Diagnoses that are treated with medication on subsequent nurse encounter:

Lower Urinary Tract Infection
Candidiasis
Possible Gonorrhea
Possible Trichomonas or Bacterial Vaginosis
Heartburn

Exceptions listed above are:

Hypertension
Syphilis
HIV
Anemia

Heartburn also does not require special handling, because it's initial requirements already have it included:

That leaves us with:

Malaria
Lower Urinary Tract Infection
Candidiasis
Possible Gonorrhea
Possible Trichomonas or Bacterial Vaginosis

@jeanette-pelizzon some of these diagnoses also trigger Health education.
Do we need health educarion here, in addition to referral to hospital?

@anvmn no we do not need health education if we are referring them to the hospital

https://nutrition-ihangane.pantheonsite.io/
QA Review
Checked and approved:

When symptoms don't change, next steps includes referral to hospital.


Error building on Android: No file or variants found for asset: packages/twitch_chat/.env.

Ran into this error trying to build for Android on a Mac today:
Error detected in pubspec.yaml:
No file or variants found for asset: packages/twitch_chat/.env.

This asset was included from package twitch_chat.
Target debug_android_application failed: Exception: Failed to bundle asset files.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/path/to/flutter/bin/flutter'' finished with non-zero exit value 1

To fix it I had to manually create a .env file in the pub cache at /Users/<username>/.pub-cache/hosted/pub.dev/twitch_chat-0.0.4
Apparently the .env file is no longer included in your package when installing from pub.dev

Adding some info...
According to https://dart.dev/tools/pub/publishing#what-files-are-published, hidden files will not be published.
From the list of items that will not be published:
Any hidden files or directories—that is, files with names that begin with dot (.)

Sorry for missing the issue. Thanks, I will remove the needs of the .env for the testing environnment in the next update. The workaround for now would be to install the package like this in your project:
twitch_chat:
    git:
        url: https://github.com/LezdCS/twitch_chat.git

Should be fixed on

twitch_chat: ^1.0.0


"WinRT WiFiDirect class" need a way(API) to retrieve the publisher's AutoGO's SSID

I am building a WIFIDirect APP with WinRT in C++. I can start an Auto GO(not legacy softAP mode) without problem and I can see the SSID by doing WIFI Scan with my phone. I want send this Auto GO's SSID to another device through BT or BLE(not just by eyes), but I can't find an API to query this local Auto GO's SSID. I wonder if this API is exist in WRT or not. If yes, what it is. If not, is there any plan MS will provide it?

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

ID: 8f123dfa-22cf-e183-1bbf-5ae9f5c5f3f3
Version Independent ID: fe306b23-cf52-6a5b-1b05-b4bbe48505c1
Content: Wi-Fi Direct - Windows drivers
Content Source: windows-driver-docs-pr/partnerapps/wi-fi-direct.md
Product: windows-hardware
Technology: partnerapps
GitHub Login: @barrygolden
Microsoft Alias: bagold

Thanks for filing this issue, @yanfangliu-google. The guidance you received from @stevewhims under the duplicate issue you filed in the winrt-api repo (https://github.com/MicrosoftDocs/winrt-api/issues/2235) is correct. I see already made an entry on the Microsoft Q & A site:
https://docs.microsoft.com/en-us/answers/questions/885084/34wi-fi-direct-sample34-need-a-wayapi-to-retrieve.html
I'll close this issue as you have already engaged the community on Microsoft Q & A.


[AMP Components in Firefox][NF] [ComboBox] UI>Drop down arrow button is truncated

Steps to reproduce:
1.Launch FireFox
2.Go to https://ampbyexample.com/
3.Click on the hamburger menu
4.Tap on "Advanced"
5.Tap on ComboBox
a.scroll to the example form
b. do next step 6.
6.Tap "View Demo" button
Observer result: drop down arrow is truncated in portrait mode
Expected result: display properly
ENVIRONMENT DETAILS:
-FireFox 57.0.4
-Device Model= SM-G900H
-Android OS version = MMB29K.G900HXXS1CQD5
-Device Serial number= 4d0092b14eb721a7
Attachment:

Cannot reproduce this


FreeBSD support

I can help create a FreeBSD port, but I need to know how are you creating the Linux binaries.

(I am not affiliated with the project.) The binaries are built using GraalVM, which unfortunately does not support FreeBSD.

@dbohdan Thanks! Hopefully someone will rewrite this in Rust / Go. #43


Git flow not working Intellij - No Git Flow

I'm submitting a ...

[X] bug report
[ ] feature request
[ ] puppy => You're not submitting a puppy. I already have one and he's adorable

What is the current behavior?

I am working with Android Studio 3.3.1 (Latest)
I downloaded and installed git-flow from the plugin repository. I have done git-flow init -f and multiple other manipulations (like, reinstall everything, remove git project, checkout again, init git flow before, etc...)
Each time the git-flow shows No Gitflow and I am not able to use it in Android studio...

What is the expected behavior?

I should have the possibility to use the context menu.

Please tell me about your environment:

Gitflow4idea version: 0.6.7

Gitflow version: 0.4.1

Android Studio version

Android Studio 3.3.1
Build #AI-182.5<IP_ADDRESS>64788, built on January 29, 2019
JRE: 1.8.0_152-release-1248-b01 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.14.3

Same problem for me but with PHPstorm.

Same problem with intelliJ 183. also if i type git flow init -f

Same but I have a work around with a precondition.
Run git flow version. If it says 0.4.1 then it means you have the vanilla installed (which is last updated in 2012). You must delete that and install https://github.com/petervanderdoes/gitflow-avh instead. When I did that, all worked

@Sam-Kruglov  is correct. You're using nvie/gitflow rather than gitflow-avh. The former is dead and hasn't been updated in years. Please upgrade to AVH Gitflow

I have the same problem, but after having installed gitflow-avh the git flow version command still outputs:
Failed to create /user.slice/user-1000.slice/session-2.scope/init.scope control group: Permission denied
Failed to allocate manager object: Permission denied
0.4.2-pre

Do you know what is the problem?

Still actual problem. Git flow plugin not working - "No Gitflow"
git flow version - 1.8.0 (AVH Edition)
pycharm 2019.1.3
gitflow plugun 0.6.8
"git flow init -f" - does not fix the problem

I think I had the same problem and I solved it.
The Context
Existing Eclipse project imported as maven ptoject. (gitflow used on Eclipse)
several local branch following standard gitflow patterns present in local repository.
Once the plugin installed in IDEA, 'No Gitflow' was visible on bottom tool bar.
Attempt 1
Clic on the toolbar and select  'Init Repo'.
On the "Option for gitflow init window" I left the checkbox "use non-default configuration" unchecked.
Click ok.
=> message "Initialized gitflow in repo FOO" however still "No Gitflow" in the toolbar
Attempt 2
Open terminal and type command git flow init -f
=> no error however still "No Gitflow"
Attempt 3
Clic on the toolbar and select  'Init Repo'.
On the "Option for gitflow init window" I checked the checkbox "use non-default configuration" and left the field as is.
Clic Ok.
=> Error because "develop" was filled for both "production" and "next release" fields
Filled "master" for production
=> Success ! it works now.
Hope this helps.

Sorry. Problem is solved.
Just update git-flow to version 1.12.3.

Sorry. Problem is solved.
Just update git-flow to version 1.12.3.

I have the latest release of gitflow, clion, and gitflow plugin, and it does not work: the already initialized repo is not recognized by the plugin and if I initialize it using clion, the plugin still displays "no gitflow"...

Also getting the same issue in IntelliJ, have updated to git-flow-avh 1.12.3 and git-flow integration plugin to 0.6.8. Have uninstalled git-flow integration and re-installed. Have done "Init Repo" and "git flow init -f" in every combination I can think of. But the same result - "No Gitflow".

I think I had the same problem and I solved it.
The Context
Existing Eclipse project imported as maven ptoject. (gitflow used on Eclipse)
several local branch following standard gitflow patterns present in local repository.
Once the plugin installed in IDEA, 'No Gitflow' was visible on bottom tool bar.
Attempt 1
Clic on the toolbar and select 'Init Repo'.
On the "Option for gitflow init window" I left the checkbox "use non-default configuration" unchecked.
Click ok.
=> message "Initialized gitflow in repo FOO" however still "No Gitflow" in the toolbar
Attempt 2
Open terminal and type command git flow init -f
=> no error however still "No Gitflow"
Attempt 3
Clic on the toolbar and select 'Init Repo'.
On the "Option for gitflow init window" I checked the checkbox "use non-default configuration" and left the field as is.
Clic Ok.
=> Error because "develop" was filled for both "production" and "next release" fields
Filled "master" for production
=> Success ! it works now.
Hope this helps.

It's the only way and the exact order that worked for me to make plugin 0.7.0 work on MacOS with 1.12.3 AVH git flow!

@ivanpadavan: This has also worked for me on fedora 30. How-ever Attempt 3 was not necessary.
Thank you for this.
Regards Markus

I have the same problem, here is my solution:

Upgrade to gitflow-avh.
Open the ./.git/config file and remove all paragraphs like [gitflow "*"].
Click on the toolbar and select 'Init Repo', then click OK.

==> It works!

thanks bro


Change default metadata endpoint handling

Description
Change the handling the /metadata endpoint so the data returned is of the same content as the /meta-data EC2 style endpoint.
Why is this needed
Hegel is meant as a replacement for what is used in EM Production today (Kant). The response from /metadata and /meta-data Kant returns is the same content. Its also arguably confusing to have 2 similarly named endpoints return very different data. This confusion is in part due to not having an Instance object yet expecting instance data somewhere. This should be cleaned up/made to make sense by https://github.com/tinkerbell/proposals/pull/25.
How Has This Been Tested?
Unit tests still pass.
How are existing users impacted? What migration steps/scripts do we need?
This is a bc-break. If any users were using the /metadata endpoint in its default configuration their workflow would be broken. I'd like to wait and see if thats the case before prematurely having any workarounds.

With this change, would it make sense that we add in a CUSTOM_ENDPOINTS: '{"/metadata":".metadata"}' in tink's docker-compose file? (I think James @splaspood also mentioned this earlier but I could be wrong.)
I understand the reasoning behind this change but I'm still a little e_e.
But with this^ we can omit the envvar when we bring up hegel, but for the open source community who (I assume) mostly use the docker compose file, they can "default" to "/metadata": ".metadata"? What do you think?

Yeah that is a good option. I kind of would rather merge this as is though and see if there are any problems with it. I'm not sure if any workflows are even using hegel/metadata (https://deploy-preview-162--suspicious-jackson-6c9a73.netlify.app/blog/2020/10/provisioning-flatcar-container-linux-with-tinkerbell/ just embeds the ignition straight into the template). So I'd rather err on the side of /metadata is same as /meta-data unless we need to come up with a work around.

alrighty @kdeng3849 I think everything should all set now.


Made prom and health server url configurable

Make address field as configurable for prom server.
Make health address as configurable similar to port.

Fixes #349
Signed-off-by: Kuldeep Singh Pal<EMAIL_ADDRESS>

Codecov Report

Merging #351 (49433ba) into main (cba2541) will decrease coverage by 0.01%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #351      +/-   ##
==========================================
- Coverage   68.89%   68.88%   -0.02%
==========================================
  Files          87       87
  Lines        5064     5065       +1
==========================================
  Hits         3489     3489
- Misses       1355     1356       +1
  Partials      220      220

Flag
Coverage Δ

unittests
68.88% <75.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/flowlogs-pipeline/main.go
0.00% <0.00%> (ø)

pkg/api/encode_prom.go
100.00% <ø> (ø)

pkg/config/config.go
50.00% <ø> (ø)

pkg/confgen/flowlogs2metrics_config.go
68.00% <100.00%> (+0.43%)
:arrow_up:

pkg/operational/health.go
100.00% <100.00%> (ø)

pkg/pipeline/encode/encode_prom.go
77.27% <100.00%> (ø)

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


Postgres DB Error Upon Module Creation

Describe the bug
I'm getting the error:
pq: zero-length delimited identifier at or near """"

The full log error from the DB is
corteza_db_1        | 2022-10-13 07:59:35.860 UTC [52654] ERROR:  zero-length delimited identifier at or near """" at character 28
corteza_db_1        | 2022-10-13 07:59:35.860 UTC [52654] STATEMENT:  CREATE TABLE IF NOT EXISTS "" (
corteza_db_1        |     "Phone" TEXT NOT NULL
corteza_db_1        |   , "LastOutreachAttemptStatus" TEXT NOT NULL
corteza_db_1        |   , "BusinessName" TEXT NOT NULL
corteza_db_1        |   , "id" BIGINT NOT NULL
corteza_db_1        |   , "rel_module" BIGINT NOT NULL
corteza_db_1        |   , "deleted_by" BIGINT     NULL
corteza_db_1        |   , "rel_namespace" BIGINT NOT NULL
corteza_db_1        |   , "revision" BIGINT NOT NULL
corteza_db_1        |   , "meta" JSONB NOT NULL
corteza_db_1        |   , "owned_by" BIGINT NOT NULL
corteza_db_1        |   , "created_at" TIMESTAMP(0) NOT NULL
corteza_db_1        |   , "created_by" BIGINT NOT NULL
corteza_db_1        |   , "updated_at" TIMESTAMP(0)     NULL
corteza_db_1        |   , "updated_by" BIGINT     NULL
corteza_db_1        |   , "deleted_at" TIMESTAMP(0)     NULL
corteza_db_1        |   )

To Reproduce
Leave handle as an empty value
Expected behavior
Client side exception. Currently I have to go into the DB and do module cleanup in order to fix.

@skamensky can you maybe address this while solving the whole schema management thing?

@darh ,
Sure. I'll add a fix for this. It looks like this is using the handle as the DB table name (I didn't verify by digging into the code).
We have two options:

Require a handle for a physical table
Use the ID as the table name

If it comes down between a choice of those two things, which do you prefer?

@darh ,
I can no longer reproduce this. It seems to have gotten fixed somewhere along the way.
I think this can be closed.


Navigation in notification adaptive card to item level

Hi Alex,
we are running the scenario for approval of Purchase requisition at item level. We can send notification from event-mesh with configure in notificationConfig.json as:
{
  "BusinessObjectName": "PR",
  "fields": {
    "ReceiverEmailList": "USER_EMAIL",
    "ObjectId": "PR_ID",
    "ActionId": "TASK_ID",
    "Description": "DESCRIPTION",
    "CreatedBy": "CREATED_BY",
    "Others": { "Creation Date": "CREATED_ON", "Priority": "PRIORITY" }
  },
  "DetailScreenConfig": "/frontend/simpleConfig/S4HanaOnPrem/Destination/PurchaseRequisitionItem.json",
  "origSysManagementAppURL": "https://s4hana-onprem-internal-host-ip:port/sap/bc/ui2/flp#PurchaseOrder-manage&/C_PurchaseOrderTP(PurchaseOrder='objectId',DraftUUID=guid'00000000-0000-0000-0000-000000000000',IsActiveEntity=true)"
}

Content of PurchaseRequisitionItem.json:
{
  "pageType": "Object",
  "system": "S4HanaOnPrem",
  "interface": "Destination",
  "businessObject": "PurchaseRequisition/PurchaseRequisitionItem",
  "title": "Purchase Requisition Item",
  "components": [
    {
      "type": "PropertyGrid",
      "properties": [
        {
          "content": "Purchase Requisition ID",
          "key": "PurchaseRequisitionItem"
        },
        {
          "content": "Purchase Requisition Item Category",
          "key": "PurchaseRequisitionItemCategory"
        },
        {
          "content": "Purchase Requisition Item Text",
          "key": "PurchaseRequisitionItemText"
        },
        {
          "content": "Purchasing Group",
          "key": "PurchasingGroup"
        }
      ]
    }
  ]
}

But the page loaded blank. Is it possible to perform such requirement in current version. Our expectation is:

we can directly jump to detail item page as when View detail on notification card like the sample below:

choose View Detail lead us directly to Item page:

Thank you,
Thien

Hi Alex,
The correct question is how we pass the OData service entity set with 2 keys and map to the Object page.
We can list the item table

Navigate to the next page shows blank because the URL cannot build, the URL in the log is
teams-backend.cfapps.us10-001.hana.ondemand.com - [2023-06-23T07:00:37.612251210Z] "GET /gateway/S4HanaOnPrem/Destination/PurchaseRequisitionItem/10 HTTP/2.0" 404

And the page keeps loading.

Hi Thien,
According to the S/4HANA OnPrem Purchase Requisition API document, it needs PurchaseRequisitionID and PurchaseRequisitionItemID to fetch PR item by ID, so this is the reason why you get a 404 when you trying to get PR item only by PR Item ID.
I would suggest you look into the Bridge Framework backend source code, backend/router/S4HanaOnPrem/OnPremDestinationRouter.js file, when you get a chance. Currently we have method to fetching the Business Object by one ID (line No. 22), and fetching children Business Object by partner's business object ID and its own ID.
So with the current Bridge Framework backend structure, I would say OData service entity set with 2 keys is possible, as long as these 2 keys are partner business object key, and children business object key. Fetching the business object with 2 keys at the same level that is currently not support yet, we will enhance it in the next version.
Best Regards,
Weikun Liu

Thanks, Weikun. Then, we may try a custom OData service where we can build the combination of keys. Looking forward for the next release.
All the best,
Thien


Refactor configuration

This nests service-related configurations into separate segments of config.yaml.
Furthermore, model configuration has been unified via ConfigModel and defaults from ctransformer have been integrated into it.
However, not all parameters are passed to piplines or model creation workflows. You need to adjust that yourself whenever you need additional control over model parameters.
You can/must set the context_length in config.yaml or via environment variables or .env explicitly if you want to edit it.
It is not calculated from chunk_size and max_new_tokens any longer. This did probably more harm than good anyway.
Config parameters have to passed to the service constructor now to decrease module coupling.
config.qa.embedding.db_path is now optional. If not set, no database will be loaded and stored. When a document is added via add_file an in-memory vector store will be created.

I put all prompts in separate files (see prompts directory). You can either pass text or path as a prompt parameter:
model:
  prompt:
    text: This is my prompt with {context}!
    path: prompts/hello.txt

If text is set, it will be used. If path is set and text is not (or text == ""), it will attempt to load the file at path and raise an error if this is not possible. If none is set, a PromptConfig will be initialised with text = "". If prompt is omitted, it will nevertheless be initialised (with an empty text string).

i uncommented db_path in config.yaml which will now load the vector store again. Just in case @Depo14 relied on it.


Add cursive-markup to readme

The cursive-markup crate provides a view that can render HTML or other markup.  This patch adds it to the list of third-party views in the readme.

Looks great! Just a small typo and it'll be good to go


Import original celeryutils code from edx-platform.

TNL-6454
Subtask of TNL-6294.

Make configuration work in freestanding context.
Convert tests to idiomatic py.test tests
Clean up docs and formatting

Reviewers

[ ] @nasthagiri
[ ] @efischer19

@nasthagiri @efischer19 I upgraded travis to test against both celery 3 and 4 (and only 4 against later django versions because I don't want to exponentially blow out test configurations, and I'm optimistic that we won't be on celery 3 for that long).   Latest version is RFR.

@efischer19 Click on "Show all checks" below, then select your favorite travis run.