Board Review: Introducing  Application Insights OpenTelemtery distro

Contacts and Timeline

Service team responsible for the client library: Azure Monitor (ApplicationInsights)
Main contacts: Matthew McCleary, Liudmila Molkova (@lmolkova), Leighton Chen (@lzchen)
Expected stable release date for this library: April 2023 (TBD)

About the Service

Link to documentation introducing/describing the service: https://learn.microsoft.com/en-us/azure/azure-monitor/app/app-insights-overview
Link to the service REST APIs: N/A
Is the goal to release a Public Preview, Private Preview, or GA?  Public preview and then GA

About the client library
Azure Monitor is working on OpenTelemtery distros -  uber-packages that bring and configure all users need to instrument their applications (OpenTelemetry, instrumentation libraries, and exporter) to send all kinds of telemetry to ApplicationInsights.
The plan is that only Python will have green-field SDK that's based on vanilla OpenTelemetry. Other languages will evolve existing Application Insights SDKs adding new OTel-based experiences to them for the time being.

Name of client library:

Python: azure-monitor-applicationinsights-opentelemetry
Future (not creating green-filled packages for now, listing to demonstrate the principle):

.NET: Azure.Monitor.ApplicationInsights.OpenTelemetry.AspNetCore or  Azure.Monitor.ApplicationInsights.OpenTelemetry.WorkerService

Link to library reference documentation:

https://github.com/microsoft/ApplicationInsights-Python/tree/main/azure-monitor-opentelemetry-distro

Is there an existing SDK library? If yes, provide link:

https://github.com/microsoft/ApplicationInsights-Python/tree/main/azure-monitor-opentelemetry-distro

Step 1: Champion Scenarios

Minimal distro configuration for traces, metrics, and logs

TODO

configuring tracing

https://github.com/microsoft/ApplicationInsights-Python/blob/main/azure-monitor-opentelemetry-distro/samples/tracing/simple.py
TBD: sampling?

configuring metrics

TODO

configuring logs

https://github.com/microsoft/ApplicationInsights-Python/blob/main/azure-monitor-opentelemetry-distro/samples/logging/simple.py

TODO: API view
Step 2: Quickstart Samples (Optional)
from azure.monitor.opentelemetry.distro import configure_azure_monitor
from opentelemetry import trace

configure_azure_monitor(
    connection_string="<your-connection-string>",
    service_name="foo_service",
    tracing_export_interval_millis=15000,
    logging_level=WARN
)

logger = getLogger(__name__)
tracer = trace.get_tracer(__name__)

with tracer.start_as_current_span("hello"):
    print("Hello, World!")
    logger.info("Correlated info log")
    logger.warning("Correlated warning log")
    logger.error("Correlated error log")

closed in favor #5423

@lmolkova
Should I refer to this issue or https://github.com/Azure/azure-sdk/issues/5423 ? Can this issue be closed?


update composer.json to support php7.3 and 8.x

Minimum requirement of laravel 8 is php 7.3 or 8.0

Please accept this pull request

Oh yes, nvm
I dont know that laravel 9 requires php 8.x
Thank you


Use relative href to stay on production site

These links will take the user off of the production site. Using relative links like these should fix it.

Hi @brockfanning , there's a conflict with this PR.  not sure if it is due to recent changes I've made to the file.  Appreciate your review.
Thanks!
Dawn

@dawncomer I think I resolved the conflicts. This should make those links work as expected, whether on the staging or production site.


Home page functions clean

Here is the new functions but not with the crazy commit history.

This looks good except for at 768px there is a gap on the right side of the window. Can you see if the Functions component is causing that?


Unable To Scan Barcode through deployed app

that's my configuration :
Quagga.init({
        inputStream : {
            numOfWorkers: (navigator.hardwareConcurrency ? navigator.hardwareConcurrency : 4),
            name : "Live",
            type : "LiveStream",
            target: document.querySelector('#scannerOutput'),
            constraints: {
                width: {min: 1024},
                height: {min: 480},
                facingMode: "environment",
                aspectRatio: {min: 1, max: 100}
            }
        },
        locator: {
            patchSize: "medium",
            halfSample: true
        },
        decoder : {
            readers : [
                "ean_reader",
                "code_128_reader"
            ]
        },
        debug: {
            showCanvas: true,
            showPatches: true,
            showFoundPatches: true,
            showSkeleton: true,
            showLabels: true,
            showPatchLabels: true,
            showRemainingPatchLabels: true,
            boxFromPatches: {
                showTransformed: true,
                showTransformedBox: true,
                showBB: true
            }
        }
    }, function(err) {
        if (err) {
            Quagga.stop();
            console.log(err);
            return
        }
        console.log("Initialization finished. Ready to start");
        Quagga.start();
 Quagga.onDetected((data)=>{
            const result = document.querySelector('#scannerValue');
            result.value = data.codeResult.code;
            Quagga.stop();
            alert("detected Product with barcode : "+data.codeResult.code);
        });

I did setup nginx with self signed certificate locally. it's working very well with no issues at all , but when I did deploy it to my VPS and trying to access it through the domain it's showing live stream but it's unable to detect barcode ( I am using same setup for both environments)

Don't ask for an aspectRatio up to 100, iOS devices will actually give it to you, and it doesn't work. :-D
Whether you run it from a local server, or a remote server, so long as it's https, it shouldn't be any different.  Lack of https would prevent the camera from being accessible, so if the livestream is working, but you're not able to scan, then there must be some difference between what you're doing with the working one and the non-working one -- in both cases, it's just shipping the code to run to the device.. so if one works and one doesn't, it pretty much has to be shipping different code?


Configurable retries and trace

I see you already fixed the "infinite retry" bug and reduced the retries from 10 to 3, but i still find that excessive.  Particularly the retries on a succesful response when .Errors collection has items in it.  Eg if I request a User Story and that US doesnt exist, the error "item not found" is sent back, and this framework then retries to get that missing item multiple times (when it will never be there).
I also find the exponential back off logic a little annoying since 10 retries meant it would sleep for 55 seconds!  Even with 3 retries you are sleeping for 3+2+1 seconds.
Not wanting to change the default behaviour, ive just added the ability to pass in the number of attempts desired, so in our case we can set things up to do no retries :+1:
I also found the diagnostic trace output very noisy so i tidied up that code and added a TraceInfo enum flags that can be used to specify the trace info to contain Body, Headers, Cookies (or nothing) as required

wow, awesome.  i was surprised the retry code was in there at all- i used to contribute to this project a couple years ago but in the meantime had been doing other things, so it took a little bit to get my head around everything in its current state.
this looks great.  i'm going to try to get this integrated and republished soon.

yeah we have been using a local fork of this library for some months, and assumed from the state of the code that this wasnt really being maintained anymore.  I had made various changes and setup a build/test pipeline in teamcity, publishing to a private nuget server.
When i saw you close the issue about nuget package (but saw that the dependencies etc werent fixed up), I thought I could tidy up some of my changes and send them back as PR's.
If you want to add me as an admin collaborator I could look at setting up AppVeyor/TravisCI and nuget publishing etc if you're keen.

I'll look into adding you as a contributor.  I'm in, but I just need to run that through legal or something on our end...  It would be awesome to have Travis or the like wired up and auto nuget publishes as well...


(feature) expansion panel: secondary content & description alignment

Bug, feature request, or proposal:
Feature.
What is the expected behavior?
Secondary content is supported by expansion panel header.
The descriptions should line up.
GUIDELINES
What is the current behavior?
Expansion panel has a fixed height and line height. Secondary content is not supported.
The descriptions don't line up.
What are the steps to reproduce?
ROUGH EXAMPLE
What is the use-case or motivation for changing an existing behavior?
Support secondary content.
Align descriptions in a column.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/material 2.0.0-beta.6-*

Maybe adding flex-basis: 0 to mat-expansion-panel-header-title and mat-expansion-panel-header-description could fix the alignment.

Using flex layout and assigning fxFlex="50%" to the title fixes the alignment issue. However, still cant display secondary content, as of beta 8.

I guess the problem here is with expansionHeight trigger for MdExpansionPanelHeader, which sets fixed line-height and height styles. I tried to override them using css !important flag and was quite surprised to find out it doesn't work during animation.

@dmitrysteblyuk, if you're using an  try this:
<md-accordion class="my-headers-align">

in the component css class:
.my-headers-align .mat-expansion-panel-header-title,
.my-headers-align .mat-expansion-panel-header-description {
  flex-basis: 0;
}

If you're using just the , use my-headers-align class directly in it.
Or, to set it globally, just ommit the .my-headers-align part in styles.css:
.mat-expansion-panel-header-title,
.mat-expansion-panel-header-description {
  flex-basis: 0;
}

Closing this since I believe it's something the application can do with CSS.

Well, maybe the application can do this with CSS, but the current version of the spec does codify some of the spacing involved, so I would make the argument that full support from the component for multi-part headers would make spec compliance more likely.
The weird part is that it specifies the space between "parts" of the header, but doesn't specify how the parts are defined.  Clearly the block width of the first and third examples (with two and three blocks per header, respectively) is not the native / inline width of the text itself, but it's also not divided evenly into halves or thirds.  I guess if the application has to decide on some fixed rule for how the headers get divided, and the spec has nothing to say about it, maybe there's nothing the component can do?

Yeah, this is a case where the spec's under-specification leads me to believe the examples aren't intended to be normative.


Miscellaneous UI improvements: reduce wallet creation QR code size, improve OTP box visual and input user experience, overall page responsiveness and card width cap, fix visual defects (e.g. side bar not expanding to full height), improve copy-writing and message prompts, and others

https://github.com/polymorpher/one-wallet/compare/d8a91c058571c3c6f549646e4621143f84456601...5685dcbd678b802c69f37c6a6c7dd93f4006d59f


Added missing link for Verifier service.

Added link

@JordanDoerksen thank you very much for catching and fixing this! Much appreciated!


Verify Windows support

We may have windows users, lots of them. Our builds currently run for linux and osx only. We should verify that emukit works fine on windows, and see if we can have a Travis build for that.

Accidentally, TravisCI announced support for windows builds last month: https://blog.travis-ci.com/2018-10-11-windows-early-release

Some updates here.
First, I should have read the full announcement. "The Windows build environment launches with support for Node.js, Rust, and Bash languages.". So build isn't happening just yet.
Second, I have run "pip install emukit" on some random Windows laptop, and GPy installation failed miserably because a compiler was missing.


Implement debug_access_token() instance function

https://developers.facebook.com/docs/facebook-login/access-tokens#debug

These changes do not pass PEP8.
You should use get_app_access_token to retrieve the app access token.
This entire function looks like it could be implemented using get_object.

Also, since this is a new feature, you should add documentation and tests.

Github doesn't notify participants if you add new commits, so I missed your changes. In the future, you might want to make a new comment here each time you change something.
I don't really understand the changes made in c4a01e8f45774302020c35fa01ee8326d045265f. Can you explain why they were necessary?

Hi,
Sorry about the sloppy/cryptic nature of the commits - this was for use in a personal project and after testing to make sure everything worked I left the documentation somewhat neglected.
Basically, debug_access_token is best off as an external wrapper function because the GraphAPI class includes access_token in the call to the Graph API. However, Facebook actually expects the access_token to be your app's access token, not the access token you're trying to debug/validate.
I don't have time right now to do a PEP8 cleanup and write tests, but if you really want those I will get around to it eventually.

Added a bit of documentation and fixed the PEP8 errors.

It would appear that the API call is primarily intended for client-side use cases that need to perform an action on behalf of the app. Facebook doesn't make any note about preferring the API call over the concatenation, so I think it's fine to leave it as is - if things change it would be trivial to update. Since we already ask for the app id and secret (and debug access token is only intended to be called server-side) it's more appropriate anyway to use the concatenation method instead of incurring the latency of an additional HTTP request and API call.

Facebook doesn't make any note about preferring the API call over the concatenation method, so I think it's fine to leave it as is - if Facebook changes this in the future it would be trivial to update.

https://developers.facebook.com/docs/facebook-login/access-tokens#apptokens only references the API call and does not mention concatentation at all. I haven't seen anything official from Facebook that suggests concatenation is okay. Is there documentation or other resources that I am not aware of?

It's right in the "App Token" section:
There is another method to make calls to the Graph API that doesn't require
using a generated app token. You can just pass your app id and app secret
as the access_token parameter when you make a call:
http://graph.facebook.com/endpoint?key=value&access_token=app_id|app_secret
The choice to use a generated access token vs. this method depends on where
you hide your app secret.
On Mar 29, 2015 11:18 PM, "Martey Dodoo"<EMAIL_ADDRESS>wrote:

Facebook doesn't make any note about preferring the API call over the
concatenation method, so I think it's fine to leave it as is - if Facebook
changes this in the future it would be trivial to update.
https://developers.facebook.com/docs/facebook-login/access-tokens#apptokens
only references the API call and does not mention concatentation at all. I
haven't seen anything official from Facebook that suggests
concatenation is okay. Is there documentation or other resources that I am
not aware of?
—
Reply to this email directly or view it on GitHub
https://github.com/pythonforfacebook/facebook-sdk/pull/199#issuecomment-87528394
.

It's right in the "App Token" section.

I am super disappointed in myself, since I read over that section twice and completely missed that.
I'll probably make minor changes to documentation and will try to add tests before merging, but I am happy with this as it currently stands.

Haha, no problem. Glad we're on the same page. It seems that writing a test
would require an actual access token and app credentials. Not sure how you
want to handle that so I'll leave that to you :)
On Mar 29, 2015 11:50 PM, "Martey Dodoo"<EMAIL_ADDRESS>wrote:

It's right in the "App Token" section.
I am super disappointed in myself, since I read over that section twice
and completely missed that.
I'll probably make minor changes to documentation and will try to add
tests before merging, but I am happy with this as it currently stands.
—
Reply to this email directly or view it on GitHub
https://github.com/pythonforfacebook/facebook-sdk/pull/199#issuecomment-87534606
.

Hi, just wondering if this could be merged soon  - beats having to maintain my own fork. Thanks!


Important note regarding the use of other software

Hi there! 🙂
I want to start by acknowledging the effort and enthusiasm you've shown in contributing to the open-source community through your projects. It's always great to see passionate developers engaging in collaborative development!
I recently took a look at the DevWinUI and HandyControls repositories, and I noticed a few areas where some improvements might be necessary to align with open-source best practices and licensing requirements. I wanted to share these observations with the intention of helping all of us maintain the high standards of our community 😁

The logo currently used in devwinui closely resembles Microsoft's WinUI logo. As this is a trademarked and automatically copyrighted design, it might be helpful to consider creating a unique logo or obtaining proper permissions from Microsoft to use theirs.

It looks like a significant portion of the control implementations may have originated from projects licensed under the MIT License. It’s important to ensure that proper attributions are provided wherever necessary. Including original copyright notices and permission details can clear up usage rights and honour the original developers’ contributions - it doesn't exactly look like the ThirdPartyNotices file in the root directory is anywhere near enough 😅

(more about HandyControls) There appear to be instances where code from other repositories, like HandyControl, is being used. Please remember that blatantly copying something (even licensed under MIT license) without attribution and attempting to pass it off as yours (aside from being just wrong) also goes against various morally asserted copyright conventions.

I believe making these changes will not only help avoid potential legal issues but also enhance the credibility and impact of your projects.
Thank you for your understanding and for your contributions to the community. 🚀

Hi @Lamparter
Thank you for reaching out and for your thoughtful feedback. I greatly appreciate the time and effort you've put into reviewing my projects and providing constructive suggestions.

1.Logo Usage
You’re absolutely right about the resemblance of the DevWinUI logo to Microsoft's WinUI logo. As a solo developer without access to design resources, I initially used it for simplicity. However, I understand the importance of originality and compliance with trademark policies. I will prioritize creating a unique logo in the future to address this issue.

2.Licensing and Attribution
I’ve made efforts to include license headers in the files where applicable, but I admit that some instances might have been overlooked. I’ll ensure this is reviewed thoroughly, and proper attributions are added wherever necessary. As for the ThirdPartyNotices , I’ve recently started building that section, and I’ll continue to refine it to make it more comprehensive.

3.Code Usage from HandyControls
Regarding HandyControls, I’d like to clarify that this project is a fork of the Chinese version of HandyControl. I actively contributed to that repository for years and created this fork with @Nabian's personal permission. Since porting the WPF version to WinUI involved significant changes, I’m curious if it’s still necessary to reference the original repository. I want to ensure I handle this correctly while maintaining proper credit.

Thank you once again for highlighting these areas. Your insights are invaluable, and I’ll work on addressing these points to enhance the quality and compliance of my projects. If you have any further suggestions or resources that might help, please feel free to share.

Thanks for the update bea03a1!
Just wanted to make sure you are aware that the LGPL-3.0 license requires that the library/app where it is consumes must also be licensed under the LGPL or regular GPL.
Also, since a lot of code appears to be from WCT and other providers, I strongly recommend you move the notice in the README to nearer the top. Thank you for your amazing co-operation and I hope to see what you come up with in DevWinUI!

Thanks for the update bea03a1!
Just wanted to make sure you are aware that the LGPL-3.0 license requires that the library/app where it is consumes must also be licensed under the LGPL or regular GPL.
Also, since a lot of code appears to be from WCT and other providers, I strongly recommend you move the notice in the README to nearer the top. Thank you for your amazing co-operation and I hope to see what you come up with in DevWinUI!

Thank you
I Moved to the top README.
I don't know about that. It seems I need to talk to ikas-mc.


Make engagement listing advanced search mobile friendly

Issue #: https://github.com/bcgov/met-public/issues/1520
Description of changes:

Refactor grid layout to make it friendly to different screen sizes

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #2249 (e3f9e43) into main (c98b0ac) will increase coverage by 0.23%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##             main    #2249      +/-   ##
==========================================
+ Coverage   68.48%   68.72%   +0.23%
==========================================
  Files         473      472       -1
  Lines       15572    15498      -74
  Branches     1206     1176      -30
==========================================
- Hits        10665    10651      -14
+ Misses       4704     4645      -59
+ Partials      203      202       -1

Flag
Coverage Δ

metweb
61.91% <89.47%> (+0.37%)
:arrow_up:

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

Files Changed
Coverage Δ

...agement/listing/AdvancedSearch/SearchComponent.tsx
83.87% <88.88%> (+2.05%)
:arrow_up:

...et-web/src/components/engagement/listing/index.tsx
72.72% <100.00%> (+0.61%)
:arrow_up:


Deleting keys while saving issue

calling "save_to_json()" more than once results in a key error.

Minimum code to reproduce bug
from ngclearn import Context
from ngclearn.components import StaticSynapse

with Context("issue19") as model:
    StaticSynapse("l1", shape=(50, 100))

model.save_to_json(directory=".", model_name="atp1")
model.save_to_json(directory=".", model_name="atp2")


First negative number in sliding windows of size k in Python

💥 Proposal
Return the first negative number in the sliding windows of size k. If not present, return 0.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign

Closing this due to inactivity. Thank you for your valuable time.😊


banned-from docs is confusing

banned-from
A list of modules that are allowed to be imported from

I read banned-from = ["A"] as "ban from A import ... statements". However, that doesn't match the description here.

A list of modules that are allowed to be imported from

Is there a missing "not" here or am I misunderstanding something?
Link to relevant option: https://beta.ruff.rs/docs/settings/#flake8-import-conventions-banned-from

Yeah I think you're right, thank you -- I'll revise...


Expand metadata to include fields for mission and project

Additions that fit the specification
Project

[ ] Description

CO Debeque Landslide Pilot Project
codqlspp
Damage assessment of landslides created from
flooding
1

Mission

[ ] Description

Debeque Landslide CO 201306
June 2013 flights
NA
NA

There are a number of additional fields being requested but they do not fit the specification. We need to clarify if we want to add non-standard fields, strictly adhere to the specification, or work on changing the specification.


Point values overlap when drawing two line charts together.

Hi
We are using this library to display two line charts in single view with values being displayed on each plotting points. The issue we are facing is when the Y-axis values in both the chart are close to each other, the values are overlapped. Is there any solution to solve this issue, i.e. values be drawn separate in this case, or we can display only single value and hide other closest value when overlapped.
Thank you for the chart library and hoping to have a solution to his soon.

@kevinzhow
@sanandrea
@MrWooJ
@dullgrass
@Noreaster76
@jackyzonewen
@emmaviolet
@farshidce
I am also facing same issue..Can someone will give solution for this ASAP?

is this a regression or did you have this issue before as well ?
will you also be able to post a screenshot ? It would be easier if you can post the dataset so that i can repro and see if i can push a fix

I hope I could get you fine. Your issue makes us step through more over dynamic aspects of this Library, For now, I assume 2 approaches:

Preparing an option for end-user developer which by setting it, Library will re-calculate Y-Axis (and/or X-Axis) points and this time disregards writing some of Y-Axis (and/or X-Axis) points on its particular vertical/horizontal line.
Or we can step more dynamically, by letting library itself calculates and consider such these situations and make the best decisions for showing either Y-Axis points or X-Axis points perfectly.

my branch is a bit ahead of the master so the behavior could be slightly different but as you can see in the screenshot there are two values either overlapping or very close to each other ( where blue and purple intersect) and i don't see the issue you mentioned


Include "GitHub Usage"

Addresses https://github.com/nuget/nugetgallery/issues/7361
/cc @mogah I don't know why, but I can't list you as a reviewer...

Docs Build status updates of commit 5d93394:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Finding-and-Choosing-Packages.md
:white_check_mark:Succeeded
View

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.

Docs Build status updates of commit 4b141bc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Finding-and-Choosing-Packages.md
:white_check_mark:Succeeded
View

docs/consume-packages/media/GitHub-Usage.png
:white_check_mark:Succeeded
View

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.

Docs Build status updates of commit 375da6b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Finding-and-Choosing-Packages.md
:white_check_mark:Succeeded
View

docs/consume-packages/media/GitHub-Usage.png
:white_check_mark:Succeeded
View

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.

Docs Build status updates of commit 98a4e74:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Finding-and-Choosing-Packages.md
:white_check_mark:Succeeded
View

docs/consume-packages/media/GitHub-Usage.png
:white_check_mark:Succeeded
View

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.


[NEW CONTENT TYPE REQUEST] Valve Material

What type of file would you like magika to detect?
Valve Texture Format (.vtf)
What software can create/open these files?
vtex, included with any valve published Source Engine game.
VTFEdit
VTF Edit Revitalized
VTF Editor
Where can these files be found?
Many Source Engine games. Convert TGA files to VTF using the above tools. Use GCFScape to extract even more VTFs from 'dir.vpk' files, and sometimes BSP files too.
If possible, please provide a specification for this file type.
Additional context
https://developer.valvesoftware.com/wiki/VTF_(Valve_Texture_Format)

Closing, see https://github.com/google/magika/issues/152 for context. Thanks!


createDefaultUsers=False breaks creating databases from console

OrientDB Version, operating system, or hardware.

v2.0 SNAPSHOT[ ] - .18[ ] .17[ ] .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.1 SNAPSHOT[ ] - .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.2 SNAPSHOT[ ] - .7[x]

Operating System

[x] Linux
[x] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Expected behavior and actual behavior
security.json specifies:
"server": {
    "createDefaultUsers": false
}

Expected behavior:
orientdb> create database remote:localhost:2424/test_db <user> <password> memory graph

successfully creates a database on the remote OrientDB server.
Actual
orientdb> create database remote:localhost:2424/test_db <user> <password> memory graph

Creating database [remote:localhost:2424/test_db] using the storage type [memory]...
Connecting to database [remote:localhost:2424/test_db] with user 'admin'...

Error: com.orientechnologies.orient.core.exception.OSecurityAccessException: User or password not valid for database: 'test_db'
        DB name="test_db"
        DB name="test_db"

Steps to reproduce the problem
Edit the default security.json to set createDefaultUsers to true and restart the server. Run the above command in the OrientDB console to try to create a new database, and observe it not working.

The console "create database" command has been updated in 2.2.8-snapshot and develop to connect to the newly created database using the same server credentials (instead of assuming the admin/admin credentials are available) used to create the database.

Thanks!


Create retirement-of-azure-percept-dk.md topic

Learn Build status updates of commit 7e9cc48:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/99456/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Multiple docs with same title breaks search

Summary

Every cloud integration has a section called creating webhooks, and this confuses search. IDEM with MQTT
Possible solutions

Rename the sections to include the parent section name (i.e creating a webhook for Akenza)
Linking to one generic section

Can you do this yourself and submit a Pull Request?

@nejraselimovic putting you in charge

Also beyond the integrations, this applies to Basic Station, see scrshot. And there are probably more generic concepts with multiple bespoke implementations.

I would go for renaming the sections

I'm not sure how to approach this. I was thinking that the most easy way was to just rename the sections, but look at the following example.

If you look at the last part Migrating to The Things Stack, clicking the first result leads here which is fine, but the second results leads here which is not fine, and as far as I can see the problem there isn't the section name. I don't have a clue how this search works, so I'm not sure if renaming sections would resolve this issue entirely.

I'm not sure how to approach this. I was thinking that the most easy way was to just rename the sections, but look at the following example.

If you look at the last part Migrating to The Things Stack, clicking the first result leads here which is fine, but the second results leads here which is not fine, and as far as I can see the problem there isn't the section name. I don't have a clue how this search works, so I'm not sure if renaming sections would resolve this issue entirely.

Renaming the sections is a good start, let's do what we can to fix that. The Migrating issue is weird, I'll message Algolia about it.

I'm not sure how to approach this. I was thinking that the most easy way was to just rename the sections, but look at the following example.

If you look at the last part Migrating to The Things Stack, clicking the first result leads here which is fine, but the second results leads here which is not fine, and as far as I can see the problem there isn't the section name. I don't have a clue how this search works, so I'm not sure if renaming sections would resolve this issue entirely.

Renaming the sections is a good start, let's do what we can to fix that. The Migrating issue is weird, I'll message Algolia about it.

Deal, I'm on it tommorow


Using the term App Passwords creates a security risk

Steps to Reproduce
Using the term "App Password" creates room for ill-informed users to accidentally compromise their account password.
Steps to Reproduce:

Third Party application asks user to provide an App Password.
User, who is not knowledgeable in internet security and unfamiliar with App Passwords in Bluesky, inputs their account password.
Account Password is now compromised.

Alternatively,

Malicious Third Party application asks user for their Account Password.
User, who is not knowledgeable in internet security and unfamiliar with App Passwords in Bluesky, checks to see if it is safe to provide their password.
User see results saying App Passwords are safe to give, but due to their unfamiliarity consider App Passwords in this context to mean their Account Password.
User provides their Account Password to Malicious application.

Solution:
Call it something else like "Personal Access Token"
Attachments
No response
What platform(s) does this occur on?
iOS, Android, Web (Desktop), Web (Mobile)
Device Info
No response
What version of the app are you using?
Build version: 1.92.1; Bundle info: 89cff31 (prod); Bundle date: 24101819; Platform: web
Additional Information
No response

FYI according to the OAuth Roadmap, App Passwords will be "described as deprecated" soon (in the Transition Phase).


Add optional custom values to Gradle Build Scans

This PR introduces 3 main changes:

Extracts all utility methods into a separate Utils class for both the Gradle Plugin and Maven Extension
Adds the option for jps and ps output to be added as custom values. These are enabled via system props.
Allows the Test task configuration to not be added as custom values. This is disabled via system prop.

Nice refactoring. My feedback is only of minor nature.

Nice refactoring. My feedback is only of minor nature.


⚠ Cioos National CKAN has degraded performance

In 56f633d, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 5284 ms

Resolved: Cioos National CKAN performance has improved in f786df5 after 5 minutes.


Is there any way to show overlay with the compiled stacktrace while sourcemap is being fetched?

Right now, there is quite a noticeable delay in overlay showing up because it waits until sourcemap is fetched and parsed. Can I somehow show the compiled version of stacktrace in the overlay until that process is done, so that the overlay opens up immediately on error? Just like redbox-react.

@neoziro hi, could you please help me here?

I noticed that too, but unfortunately I don't have any solution for it...

Outdated, feel free to open a new issue it if you meet that bug.


Coast Mask

Is your feature request related to a problem? Please describe.
Calculate the coastal mask of the file
Describe the solution you'd like
Click a button and the coastal mask is added to the datafile
Describe alternatives you've considered
Use fortran code

This has been integrated in #94


fix read json bug and reconnect time

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #812 into stable will increase coverage by <.01%.
The diff coverage is 43.47%.

@@            Coverage Diff             @@
##           stable     #812      +/-   ##
==========================================
+ Coverage   55.43%   55.44%   +<.01%
==========================================
  Files         232      232
  Lines       23668    23683      +15
==========================================
+ Hits        13121    13130       +9
- Misses       9342     9350       +8
+ Partials     1205     1203       -2

Impacted Files
Coverage Δ

pkg/xds/v2/adssubscriber.go
0% <0%> (ø)
:arrow_up:

pkg/api/v2/types.go
72.9% <71.42%> (+0.71%)
:arrow_up:

pkg/config/config.go
56.25% <71.42%> (+2.91%)
:arrow_up:

pkg/module/http2/transport.go
78.3% <0%> (-0.08%)
:arrow_down:

pkg/module/http2/server.go
81.85% <0%> (+0.14%)
:arrow_up:

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 3e35bab...ca89c91. Read the comment docs.


build fpga example fails

When I try to bulild the fpga example with the suggested command:
fusesoc --cores-root=. run --target=synth --setup --build lowrisc:ibex:top_artya7 --part xc7a35ticsg324-1L

I get the following output:
Failed to resolve dependencies for lowrisc:ibex:top_artya7:0.1

I installed the python-requirements.txt in a venv. Which additional steps are needed to build the example?
Where can I find some instructions on how to use the Ibex core in a Vivado project? A tcl script that generates a ready to use project would be very helpful. Or a list with all the files needed. I tried it for a few hours but couldn't resolve the dependencies and gave up. :(

I was able to generate the hardware and led.vmem.
In order to simulate the led example I had to add the path to the file manually in prim_ram_1p.sv by changing
parameter      MemInitFile     = ""
to
parameter      MemInitFile     = <path to led.vmem>
Why is the SRAM_INIT_FILE parmeter not used in this example?

SRAM_INIT_FILE doesn't exist any more, MemInitFile is the right way of passing a memory file. I filed https://github.com/lowRISC/ibex/issues/956 to get that sorted.


Support unstorage as source

Using unjs/unstorage (new raw utils) we can unblock many features:

Platform agnostic file system paths
Built-in caching support
Multi content sources with multiple endpoints
More secure base
Cache/Store metadata

Adding more unstorage integrations will progressively happen.


Cannot find the config file for awq when load llm with LLaMA-2 + AWQ

Hi，Does anyone know the reason for this?
when I call awq as per the example, I get the error:
File "/quantization/vllm/vllm/model_executor/weight_utils.py", line 108, in get_quant_config
raise ValueError(f"Cannot find the config file for {quantization}")
ValueError: Cannot find the config file for awq.
vllm version: 0.2.1  (commit 651c614)
my code:
llm = LLM(model="./models/llama-2-7b-hf",quantization="awq",dtype="half")

Hi @xianwujie, vLLM assumes that the model weights are already stored in the quantized format and the model directory contains a config file for the quantization method. You can use a tool like AutoAWQ to quantize your fine-tuned model.


Bump coverage a tiny amount, and fix a genuine bug

(the bug is that spi_devices.py wasn't importing InputDeviceError)

Hmmm, Travis is telling me that the tests all pass on Py2.x and Py3.x but fail on pypy and pypy3 with:
GPIOPinInUse: pin 4 is already in use by <gpiozero.InputDevice object on pin GPIO4, pull_up=None, is_active=False>

Any ideas @waveform80 ?

Is this the same GPIO pin in use thing we saw in #680?

@tuftii It's similar, but I don't think it's the same thing. IIRC the bug you found affected all versions of python, where the bug here is only affecting pypy.

@lurch yup - the issue is the pypy GC is too slow at destroying the first InputDevice on GPIO4. It's fine in other tests where "with" is used as the context manager will explicitly close the device (no waiting for the GC). But the first test in test_input_invalid_external_pull_configuration is constructing a bare InputDevice outside a "with" section (which is entirely correct BTW, but in this case it'll cause this issue with pypy). I've just pushed a simple fix which is to separate it into two tests; the mock_factory cleanup will explicitly close all attached devices which should deal with the issue.


'Player' object has no attribute 'register_properties_handler'

Hi,
Nice project.
For some reason I can run example to set properties change handler.
mp.player.register_properties_handler(handle_properties_changes)
AttributeError: 'Player' object has no attribute 'register_properties_handler'

[]'s
Hugo

Hi @hugosenari,
The method register_properties_handler currently is available only in dev version.
You can install dev version from repository using pip: pip install git+https://github.com/wistful/pympris.git


[Improvement][Flink]: Sink to Arctic with dynamically nodes scale-in/out

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
Arctic nodes num can be configured in table properties, and it may be changed in any time.
Support changing it dynamically would bring a better experience for user.
How should we improve?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Subtasks
No response
Code of Conduct

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

@zstraw #2 maybe useful


cannot connect to X server

How to solve this problem? thanks very much
startThreads creating 1 threads.
starting thread 0
started thread 0
argc=2
argv[0] = --unused
argv[1] = --start_demo_name=Physics Server
ExampleBrowserThreadFunc started
X11 functions dynamically loaded using dlopen/dlsym OK!
No protocol specified
cannot connect to X server

Hi, did you run it on a remote server? Can you try on a local computer and see if it works?


[web][html] Image flickering while scrolling.

When using it on iOS, images are flickering while scrolling.
Found this on 1.26 and up.
[✓] Flutter (Channel dev, 1.26.0-17.1.pre, on macOS 11.2 20D64 darwin-x64, locale en-US)
• Flutter version 1.26.0-17.1.pre at /usr/local/Caskroom/flutter/latest/flutter
• Framework revision c40e5294dd (3 days ago), 2021-01-29 08:44:18 -0800
• Engine revision 5d3477eab4
• Dart version 2.12.0 (build 2.12.0-259.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/alonhorovitz/Library/Android/sdk
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.4, Build version 12D4e
• CocoaPods version 1.10.0
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] VS Code (version 1.52.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.19.0
[✓] Connected device (1 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.96
• No issues found!

Hi @Bilonik
Can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample?
Thank you


Unsafe legacy lifecycles

getting this when using this extension witn rn 0.58.3
Warning: Unsafe legacy lifecycles will not be called for components using new component APIs.
Thumbnail uses getDerivedStateFromProps() but also contains the following legacy lifecycles:
UNSAFE_componentWillUpdate
The above lifecycles should be removed. Learn more about this warning here:
https://fb.me/react-async-component-lifecycle-hooks

17.0: Remove componentWillMount, componentWillReceiveProps, and componentWillUpdate . (Only the new “UNSAFE_” lifecycle names will work from this point forward.)


Move community operator to unrestricted namespace

Managed OpenShift clusters might restrict the ability to create/modify
namespaces starting with openshift-, and resources inside them. The
gatekeeper operator should be able to run in any namespace, so it would
be good to create it in a namespace where there won't be any issues.
Refs:

https://github.com/open-cluster-management/backlog/issues/13759

Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/hold

/lgtm

/unhold


lua: allow using moonjit

Description:
moonjit (https://github.com/moonjit/moonjit) is a continuation of
LuaJIT development, which, as of late, has, unfortunately, come to
a stall.
In comparison to LuaJIT, moonjit, among other things, supports
s390x architecture as well as some of the Lua 5.2 language
features.
With this patch, moonjit can be enabled using
--//source/extensions/filters/common/lua:moonjit=1
bazel flag.
Internally, moonjit integration is based on luajit integration.
Some of the fragments thereof (e.g. luajit.patch) were copy-pasted
with minimal modifications. While arguably ugly, this allows
updating LuaJIT and moonjit independently.
Signed-off-by: Ilya Leoshkevich<EMAIL_ADDRESS>Risk Level: Low
Testing: Run tests with --//source/extensions/filters/common/lua:moonjit=1 flag
Docs Changes: None
Release Notes: None

Why not just use a more recent snapshot of luajit instead, which has support for both IBM architectures?

@yselkowitz since this moonjit is an alternative, we can always update luajit "version".

@dio done.
@yselkowitz where can I get it? I checked master and v2.1 branches of https://luajit.org/git/luajit-2.0.git, and s390x support is not there.

The OpenResty fork of luajit2 has support for both Power LE and Z in their latest snapshot:
https://github.com/openresty/luajit2

@yselkowitz what looks interesting about OpenResty fork, is that moonjit maintainer also thinks it's the way forward for the Lua community (https://github.com/openresty/luajit2/issues/80). Unfortunately, the merger discussion has somewhat stalled. What also makes it harder for us to quickly evaluate OpenResty fork, is that their test suite is much smaller than that of moonjit. So we would rather go forward with moonjit for the time being, if possible.


Fix lower section area 375

Updated the Lower Section by adding comments. I also changed the location of " /div"s in a few areas so that the List of pages and Code Editors lined up nicely.

Looks great :)


issue with unit_check_wrapper

First of all: thanks for this great toolbox !
Unfortunately I have encountered an error with the unit_check_wrapper in poppy 0.5.0. If I try to run the unit tests or the example scripts the following error (or similar) occurs:
"Only dimensionless scalar quantities can be converted to Python scalars" or
" 'm' (length) and '' (dimensionless) are not convertible"
I've tried this on a fresh installed system with Python 2.7 x64, Python 3.5 x64 and Python 3.5 x32 (Anaconda distributions). The same for the included ipython notebook files.
With poppy 0.4.1 the error doesn't occur.
What am I doing wrong ? Thanks in advance.
Sven

Thanks for the report! I'll look into this for you. Can you come up with a minimal test case? Does it happen any time a quantity is passed through that wrapper, or only with certain virtual optics or functions?

As a data point, I created a fresh Python 3.5 conda env and installed POPPY 0.5.0 from source. (There was an issue with a distutils import on install, which was unrelated.)
Running this example from the notebooks/POPPY Examples.ipynb file with and without the * u.m included seems to work.
from astropy import units as u

osys = poppy.OpticalSystem()
osys.add_pupil( poppy.CircularAperture(radius=3 * u.m))    # pupil radius in meters
osys.add_detector(pixelscale=0.010, fov_arcsec=5.0)  # image plane coordinates in arcseconds

psf = osys.calc_psf(2e-6)                            # wavelength in microns
poppy.display_psf(psf, title='The Airy Function')

Thanks for the fast response. This example file works for me too. The problem occurs with the MatrixFTCoronagraph.ipynb example, more precisely during the execution of the following call:
"annFPM_mft_psf, annFPM_mft_interm = matrixFTcoron_annFPM_osys.calcPSF(wavelen, display_intermediates=True, return_intermediates=True)"
The error also occurs if I run the tests:
import poppy
poppy.test()

Okay, there might be more than one bug here. (Haven't had the chance to run the test suite yet.)
Looks like in https://github.com/mperrin/poppy/blob/master/poppy/poppy_core.py#L2083 there's an attempt to format a unit-having quantity as a simple float, which is incorrect. It's done correctly elsewhere, so I'm going to read through and see how we missed that one.
If you want a quick fix, replace that line with just title = None in your copy. The only thing you'll miss is the note in the plot title about which wavelength was last propagated.

Great,
your workaround does the trick.
Thanks a lot


Create a libray of QML components

Description
To go fast, currently we don't have a library of components that reuse across the code base. This needs to change. If a component is being reused, effort should be to create a component and make It generic enough to be reused.
Acceptance criteria

Component should be generic
Component should simplify the code base

Difficulty

Medium

Pull request : https://github.com/kounkou/Hedgehog/pull/98 helps solve the issue


Calendar selection widget

[x] Single date
[ ] Date range
[x] Minimal/Maximal date limitation
[x] Multiselect - see mega example

old version https://paste.ubuntu.com/p/zHwXGdgyqG/

https://paste.ubuntu.com/p/pW3ms49Nbr/

Idea 1: translatable labels for a Calendar widget. I think it can be implemented with a gettext package.
The structure would then look like this:
aiogram_dialog/
docs/
example/
translations/
├─ de/
│  ├─ calendar.po
├─ en/
│  ├─ calendar.po
├─ fr/
│  ├─ calendar.po
├─ ru/
│  ├─ calendar.po

And in translations/en/calendar.po (for example) we have strings:
msgid "Mon"
msgstr ""

msgid "Tue"
msgstr ""

msgid "Wed"
msgstr ""

msgid "Thu"
msgstr ""

msgid "Fri"
msgstr ""

msgid "Sat"
msgstr ""

msgid "Sun"
msgstr ""

Translation to ru locale:
msgid "Пн"
msgstr ""

msgid "Вт"
msgstr ""

msgid "Ср"
msgstr ""

msgid "Чт"
msgstr ""

msgid "Пт"
msgstr ""

msgid "Сб"
msgstr ""

msgid "Вс"
msgstr ""

Idea 2: it would be nice, if we can start from a specific date.
class Calendar(Keyboard, ABC):
    def __init__(self,
                 id: str,
                 start_date: Union[datetime] = None
                 on_click: Union[OnDateSelected, WidgetEventProcessor, None] = None,
                 when: Union[str, Callable] = None,
                 ):
        super().__init__(id, when)
        self.on_click = ensure_event_processor(on_click)

        ...

window_reservation_at = Window(
    Format(_("Choose the reservation date")),
    Calendar(id='reservation_at',  start_date: datetime.now(), on_click=reservation_at_handler),
    Row(cancel_button),
    state=UserBotTableReservationSG.reservation_at,
)

Translations should done using babel package or calendar python module with proper locale set. You do not need to create your own translations.
Not sure wht is the meaning of start_date. Is it default date, minimal date or what?

Not sure wht is the meaning of start_date. Is it default date, minimal date or what?

Yes. If a minimal date 20.09.2021, we can only select the date, that is greater than 20.09.2021 :)

The current view of the Calendar widget is:

I suggest splitting the calendar into several widgets:

1st lvl:

ZoomOut - Raises the zoom level by one from the current level, starting at level 1 and up to level 3.
Week - Displays the days of the week and allows them to be positioned either above, below, or not displayed at all.
Month or Calendar - Displays the days of the month, and on click, returns a datetime.date object in the on_click func.
PrevMonth and NextMonth - No comments.

2nd lvl:

Months - Allows the user to remove any months.

3rd lvl:

Years - Can be customized to allow the user to specify range, such as passing range(2000, 2024), or a selector that will get the range from the getter.

While this will kill backward compatibility, but we can it since we are still in beta.

Implemented highly customzable widget in #265
Though selection is not yet supported.

Added an example for multiselect - https://github.com/Tishka17/aiogram_dialog/blob/develop/example/mega/bot_dialogs/calendar.py#L58
Date range selection looks very complicated from UX side, so probably won't be implemented.


Error 400

Kindly see if there's need for an update to this library.
I am getting an error 400 while trying to create a subaccount.
sa, err := r.Paystack.SubAccount.Create(&paystack.SubAccount{
			BusinessName:        *data.Name,
			SettlementBank:      *data.Bank,
			AccountNumber:       *data.AccountNumber,
			PercentageCharge:    PercentageCharge,
			SettlementSchedule:  SettlementSchedule,
			PrimaryContactEmail: *data.Email,
			Metadata:            metadata,
		})

if err != nil {
    return nil, err
}

{\"code\":400,\"details\":{},\"url\":{\"Scheme\":\"https\",\"Opaque\":\"\",\"User\":null,\"Host\":\"api.paystack.co\",\"Path\":\"/subaccount\",\"RawPath\":\"\",\"ForceQuery\":false,\"RawQuery\":\"\",\"Fragment\":\"\"},\"header\":{\"Access-Control-Allow-Origin\":[\"*\"],\"Cf-Cache-Status\":[\"DYNAMIC\"],\"Cf-Ray\":[\"548097858dd2cdb7-CDG\"],\"Content-Length\":[\"64\"],\"Content-Type\":[\"application/json; charset=utf-8\"],\"Date\":[\"Fri, 20 Dec 2019 09:29:26 GMT\"],\"Etag\":[\"W/\\\"40-IhinKFxsDy9hOhy0AlnrOg\\\"\"],\"Expect-Ct\":[\"max-age=604800, report-uri=\\\"https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct\\\"\"],\"Server\":[\"cloudflare\"],\"Set-Cookie\":[\"__cfduid=d966225626d511709c7deeaa7f9fc4a0c1576834166; expires=Sun, 19-Jan-20 09:29:26 GMT; path=/; domain=.paystack.co; HttpOnly; SameSite=Lax; Secure\",\"sails.sid=s%3AbLrbTdaDSSzE3xP9RhdNBkJkz68T0hrU.ElX48nd0jxKE%2BTOfOyxbNvv6ILMTTgxlgJcRlPZatdI; Path=/; HttpOnly; Secure\"],\"Strict-Transport-Security\":[\"max-age=15552000; includeSubDomains; preload\"],\"Vary\":[\"X-HTTP-Method-Override, Accept-Encoding\"],\"X-Content-Type-Options\":[\"nosniff\"],\"X-Powered-By\":[\"Sails \\u003csailsjs.org\\u003e\"]}}

Nevermind, I figured that paystack servers may be rejecting my SSL certificate.

@darmie how did you solve this?

@darmie how did you solve this?

@ozombo i figured that I couldn't handle Paystack golang API from inside a containerized application, due to certificates validation reasons...
This is a very old issue and I don't remember how I worked around it.

@ozombo i figured that I couldn't handle Paystack golang API from inside a containerized application, due to certificates validation reasons...
This is a very old issue and I don't remember how I worked around it.

@darmie mine occurred with charge creation and after lots of hair pulling I figured out it was some issues with the card struct. Made a pr for it tho

@darmie mine occurred with charge creation and after lots of hair pulling I figured out it was some issues with the card struct. Made a pr for it tho

yet to be reviewed


[Q] Is it possible to run "atom" command from termination terminal?

I would like to be able do the following

I would like to run the 'atom' command to open (and then edit) files from the termination terminal.
Most commands I have tried in the termination terminal works, but specifically those starting with 'atom' give the error message: "need to run as root or suid"
Running "sudo atom ..." yields the error message "sudo: effective uid is not 0, is /usr/bin/sudo on a file system with the 'nosuid' option set or an NFS file system without root privileges?"
When running these commands in an external terminal window (gnome terminal) they run exactly as you would expect.
Is this intended behaviour? How can I give sufficient privileges to the termination terminal? Elsewise, is there any similar terminal packages that allows this?

Additional context

Looks like you are running into this error: https://github.com/bus-stop/terminus/issues/84

Yes. But why does atom need sudo privileges in the first place? It does not need sudo to be called with the recular terminal.

Not sure. I am on windows and it seems to work fine for me.

Yes. But why does atom need sudo privileges in the first place? It does not need sudo to be called with the recular terminal.

Its an electron 5.0.13 issue, a fix has been merged into atom and will be available in 1.52.0-beta0 and later releases.

Aight, thanks for the replies!


Is it possible for  use findOne (id ) or findOne (slug) with locale parameters for support strapi i18n

Hi ,  I am facing an issue for get a single page or single type item from strapi collection type . i can not get a  required item or post from strapi by  findOne (slug , { _locale:i18n.locale})  for _slug.vue page , that's make my whole project stop  there . since it's required a locale parameter  to fetch other language for the item or post by slug . any idea  ,how to achieve that or there is any other alternative option for this module rather than use graphql or apollo .
thanks in advance
Kevin

https://strapi.io/documentation/developer-docs/latest/development/plugins/i18n.html#usage-with-strapi-content-api

Closing this as query params for findOne have been added in https://github.com/nuxt-community/strapi-module/pull/167 and released in https://github.com/nuxt-community/strapi-module/releases/tag/v0.3.2.


Mark other_selected if Other selected

Fixes #2846

Added proto in this PR
https://github.com/google/ground-platform/pull/2107

@shobhitagarwal1612  PTAL?

I don't think we need the other variable as mentioned here
https://github.com/google/ground-android/issues/2846#issuecomment-2498445587
Please let me know if I am missing anything here.

Please update unit tests and PR description as well

Waiting for web pr https://github.com/google/ground-platform/pull/2107 to get in so for the proto variable

Please add some test coverage to SubmissionMutationConverterTest


fix(client): set fcc-sound value in local storage after user data is fetched

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #46372
Closes #48107

This is first method of solving issue
After user has logged in successfully I'm setting fcc-sound value in local storage after user data is fetched in fetchSessionUser.
As playTone grabs fcc-sound value from local storage to play audio

Please take a moment to give a name to the PR that follows the guidelines: https://contribute.freecodecamp.org/#/how-to-open-a-pull-request?id=prepare-a-good-pr-title

Please take a moment to give a name to the PR that follows the guidelines: https://contribute.freecodecamp.org/#/how-to-open-a-pull-request?id=prepare-a-good-pr-title

I've updated pr title.

This PR need NOT be reviewed any further until we have achieved agreement on #48107

Hey @nayabatir1,
Are you still working on this? Do you need help?

Hey @Sembauke,
I already did all the changes. If I missed something do let me know.


UHF-7955: Run mobile menu fallback through ::filterLanguages menu manipulator

How to install

composer require drupal/helfi_navigation:dev-UHF-7955
drush cr

How to test

See https://helsinkisolutionoffice.atlassian.net/browse/UHF-7955

Codecov Report

Merging #37 (ffb00d3) into main (29d0fc8) will decrease coverage by 0.64%.
The diff coverage is 22.22%.

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

@@             Coverage Diff              @@
##               main      #37      +/-   ##
============================================
- Coverage     70.66%   70.03%   -0.64%
- Complexity      173      174       +1
============================================
  Files            19       19
  Lines           658      664       +6
============================================
  Hits            465      465
- Misses          193      199       +6

Impacted Files
Coverage Δ

src/Plugin/Block/MenuBlockBase.php
22.22% <ø> (ø)

src/Plugin/Block/MobileMenuFallbackBlock.php
7.14% <22.22%> (-0.32%)
:arrow_down:

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


Make TrieLookupTable destruction non-recursive

Commit Message: Make TrieLookupTable destruction non-recursive
Additional Description: Apparently someone might feed a 20000-character long key into this trie, for example in a fuzz test, but never a 40000-character string because that would be ridiculous, so I have to do this or we'd have to revert back to the version that was worse in every other way, because that absolutely makes sense, and we definitely should not have someone who is in charge of the ridiculous test fix it.

/assign @jmarantz

@ravenblackx I think the original code was buggy also; we shouldn't be recursing through so many levels. Envoy might need to run in an environment with smaller stacks than our test machines as well, so it's good we are sorting this out.

Yes, that's why I'm disgruntled for this to be operating on a "fix our external test or we'll revert your change even though the actual problem was pre-existing" basis. Feels like a certain company is exerting undue power here - if someone else's PR broke one of my external-to-envoy tests based on a problem the PR didn't introduce, I'm pretty sure I wouldn't be able to threaten to revert their PR if they don't immediately address a pre-existing problem.

Maybe prefer https://github.com/envoyproxy/envoy/pull/34849 instead since it fixes the stack overflow destructor and further improves performance.

That one looks good; should we submit this first though while the other gets reviewed? Or close this PR?

We should close this PR if we can get agreement that that one is preferred.

Actually, yeah, let's just close this one, because I don't want to make friend accessors for members of private internal data structures, so this one is stalled anyway. The other version doesn't have that problem, and is simpler, and is more performant, and uses less RAM, and cleans up the dependency mess a bit, so I can't think of any valid reason for this PR to be preferred.


Updating thumbs.json and ips.json.

I found that when a page's title is changed (the page's slug is also changed), data in thumbs.json and ips.json becomes out of sync.
For example, say an article has title "This rocks", and thumbs.jon and ips.json have entries for "this-rocks". Then the article's title is changed to "This sucks", then this article's thumb ratings data is no longer available, since thumbs.jon and ips.json don't have entries for "this-sucks".

Being similar, I also report issue number #5 to this issue, which I have closed.
Therefore, when you change the page's slug there is no more connection with the thumbs.
To solve I can intercept the event onAdminAfterSave and provide for the modification, if any, of the slug of the page: that is when, in Admin, you change the page's title and the resulting page's slug, I can intervene to change even thumbs.json and ips.json.
But this does not solve the situation you have proposed, that is when changes are made outside of Grav


Hangfire.Mongo Doesn't Honor Worker Count when Spawning New Jobs

Hi,
Hangfire v 1.7.20
Hangfire.Mongo v 0.7.20
Fresh install.
Hangfire client running as a Console App. Configured to allow for 8 workers via
bjo.WorkerCount = 8
Configuration:
GlobalConfiguration.Configuration.UseMongoStorage(hfcn, hfdb, New MongoStorageOptions With {.Prefix = "hfx", .CheckConnection = False, .MigrationOptions = New MongoMigrationOptions With {.Strategy = MongoMigrationStrategy.Migrate}})
A series of jobs ran overnight, and when I looked this morning, there were over 1,800 jobs that were running, and it pegged CPU at 100% and made the system inoperable.  I first noticed this behavior a few days ago when I migrated to a new processing server. I don't know if this had been happening with the old server - I honestly never paid much attention.
When I noticed this behavior, I moved to the Redis storage engine - to see if that's the default behavior - or if it was something related to the Hangfire.Mongo package.  The Redis.Pro package FAITHFULLY honors the worker count directive. I never had more than 8 jobs running.
Please advise.

@du8die,
Thank you for posting.
Hangfire.Mongo does handle scheduling or spin up threads or jobs. It's only the storage it handles. The Hangfire.Core lib handles scheduling.
Im not sure how it can start more than the given worker count.
Can you somehow confirm it actually starts more workers than the intended and it's not a reporting issue of sorts?


Update to HTTPS urls in BintrayRepositoriesExtension

Hi,
this PR updates two URLs to make use of HTTPS. This seems especially important for  http://jcenter.bintray.com as this seems to be effectively turned off.
Cheers,
Christoph

👍


Multiple nodes, minikube and calico

Is there an example to run calico and minikube in a multiple node setup?
Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Calico version
Orchestrator version (e.g. kubernetes, mesos, rkt):
Operating System and version:
Link to your project (optional):

@verizonold thanks for asking! I don't think such a guide exists today, but it should follow the standard hosted install procedure: https://docs.projectcalico.org/v3.0/getting-started/kubernetes/installation/hosted/hosted
If you manage to get it working, we'd love to hear about it! If you hit any issues, please open them here and we can help figure it out.


Use exporters in ORTTrainer

What does this PR do?
ORTTrainer uses optimum.exporters.onnx
Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@michaelbenayoun @JingyaHuang rebased the PR. Let me know if it looks good and we can merge.


[FRG] Change the order of alignment options

So, this is a minor thing, but it always is a gotcha for us if we're moving quickly.
The Alignment dropdown options are listed as
Left
Right
Center

But our brains always think it should be
Left
Center
Right

Not a huge issue, but might be a nice QoL thing.

We will get this in the next version...  :grinning:

Fixed in v1.3.5 - In addition, a "None" has been added to the list as the default; If you put "Left", "Center" or "Right" it will override the X coordinate, so "None" allows the X coord to position items in a specific x coordinate.


Add some good default stories

Add some getting started stories for setup it quickly.

 Better templates for getstorybook.


⚠ Bofrost.fr has degraded performance

In e37ae32, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1912 ms

Resolved: Bofrost.fr performance has improved in 2029710.


[URGENT] Fix for cluster issue BZ-1283953

BZ-1283953: Cluster failing to cleanup the global lock properly. After investigating, it was noticed that some FileSystem ("system") were proxyed, and the CDI proxy doesn't include other possible interfaces other than that been injected... so when tested instance of FileSystemId it always failed.
So.. in order to avoid such issue, the solution is to avoid use directly the file system and resolve it using the fs.getRootDirs().interator().next().getFileSystem(); with this we assure that we have access the real FileSystem instance.
Additional PR:
https://github.com/droolsjbpm/guvnor/pull/199

+1 to merge.

Thanks!
0.7.x: 4dbd922dc942b0801d0f1bae0ab19fbae009a595
master: 0d9af94fd979a9eaeadc7a9c11e036d83acafa79 (did that by mistake, but since it belongs there anyway I will leave it. We don't need another PR for master here but for Guvnor).


CO-7031 CO-7050 : Updating right_aws gem for aws cookbook

Please add the appropriate [Risk Level label][1] and make sure you've followed the [code review guidelines][2]!
JIRA
Complete the URLs for the main JIRA ticket and propagation subtask.

Main JIRA ticket
JIRA propagation subtask

Code reviewers

[x] @nikhilcoupa
[ ] @marutinandanpandya
[ ] @alokdnb
[x] @coupacooke
[ ] @charchitcoupa

Please list your code reviewers here. Use a [GitHub Task List][3] and [mention][4] the reviewers you want to review.
Summary of issue
Referring to new gem file which has fix for retrying on rate limits exceeded.
Summary of change
Referring to new gem file which has fix for retrying on rate limits exceeded.
Testing approach

[x] Please refer ticket for more details.

lgtm

[lockit v4.6.0] merged via Lockit CLI on 2018-08-26 16:38:05.448930091 -0700 PDT m=+4.047346837


get_dict_for_symbol() does not work if symbol name starts with "^"

I notice that get_dict_for_symbol() does not work for symbols that start with "^" and "~'. An example would be the yahoo symbol for S&P 500 "^GSPC".
This will be a problem for most yahoo stock indices.
Thanks for your work. It is useful for Amibroker users.

I implemented the fix. However, the automatic build chain is currently broken.
Therefore, no new version was deployed on pypi.
Please use the git repo and try it.
Would be nice if you can close the ticket if it works for you!

I implemented the fix. However, the automatic build chain is currently broken. Therefore, no new version was deployed on pypi. Please use the git repo and try it. Would be nice if you can close the ticket if it works for you!

Thank you for the quick action. I have verified that the fix works. I will close the ticket.


Scrape user information

A minimal set of features from a user:

[x] Basic profile information (username, join date, watching/completed/... stats)

[x] Username
[x] Join date
[x] Last online date
[x] Anime Stats: #Watching, #Completed, #On-Hold, #Dropped, #Plan to Watch

[x] Anime list

[x] Name
[x] Progress/Progress category
[x] Score
[x] Started watching date
[x] Finished watching date
[x] Watch duration calculation (#13)

[x] Handle no access to anime list
[x] ?Anime updates history? (#11)

Watch duration and anime updates (accessible from the user profile page) is being pushed out to another release as they are non-essential for data analysis.

Awaiting release.


Decompression module

I was faced with the problem in framing module.
Function in read_frame checking RSVs states, but it do not consider that the RSV1 is compression bit RFC 7692
This code solved my problem with deflate decomression.
But it has no support dynamic "client_max_window_bits" (in my example it 15).
framing.py
@asyncio.coroutine
def read_frame(reader, mask, *, max_size=None):
    """
    Read a WebSocket frame and return a :class:`Frame` object.

    ``reader`` is a coroutine taking an integer argument and reading exactly
    this number of bytes, unless the end of file is reached.

    ``mask`` is a :class:`bool` telling whether the frame should be masked
    i.e. whether the read happens on the server side.

    If ``max_size`` is set and the payload exceeds this size in bytes,
    :exc:`~websockets.exceptions.PayloadTooBig` is raised.

    This function validates the frame before returning it and raises
    :exc:`~websockets.exceptions.WebSocketProtocolError` if it contains
    incorrect values.

    """
    # Read the header
    data = yield from reader(2)
    head1, head2 = struct.unpack('!BB', data)
    fin = bool(head1 & 0b10000000)
    decompressor = None
    if head1 & 0b01110000:
        if head1 & 0b00110000:
            raise WebSocketProtocolError("Reserved bits must be 0")
        else:
            decompressor = zlib.decompressobj(-15)
    opcode = head1 & 0b00001111
    if bool(head2 & 0b10000000) != mask:
        raise WebSocketProtocolError("Incorrect masking")
    length = head2 & 0b01111111
    if length == 126:
        data = yield from reader(2)
        length, = struct.unpack('!H', data)
    elif length == 127:
        data = yield from reader(8)
        length, = struct.unpack('!Q', data)
    if max_size is not None and length > max_size:
        raise PayloadTooBig("Payload exceeds limit "
                            "({} > {} bytes)".format(length, max_size))
    if mask:
        mask_bits = yield from reader(4)

    # Read the data
    data = yield from reader(length)
    if decompressor:
        data = decompressor.decompress(data)
    if mask:
        data = bytes(b ^ mask_bits[i % 4] for i, b in enumerate(data))
    frame = Frame(fin, opcode, data)
    check_frame(frame)
    return frame

I think this is covered by https://github.com/aaugustin/websockets/issues/93.
I'd like to provide proper APIs for supporting extensions, not just to hardcode this one in the middle of the protocol implementation.

Closing a a duplicate of #93. See also #178.


After SASS integration - Errors and Reloading browser

After following the SASS Support Integration instructions, I get a lot of (stupid) errors when building the project :

Everything was fine before integrating Sass...
I get errors like:
file should end with a newline
trailing whitespace
and more...
Furthermore, when I make changes In any component and save them, I get :
[BS] Reloading Browsers...
And it stays so forever. It doesn't reload.
What can I do?

Let just be rational, and think that this are not programming language (Javascript) conventions,

@katalonne The errors refer to have nothing to do with you enabling SASS because they are pointing towards your Angular2 service files. I.e. they are TSLint errors. The tslint config followings the Angular2 Style Guide as close as possible.
If you so desire to have a different configurations, feel free to edit the tslint.json file which you can find in the root directory of the project.

Dude, This errors showed up after I integrated sass!!!


Connection refused error

Hi,
I am using this library for web-socket communication and able to connect with server but getting following error very frequently :
2019-07-21 19:57:30.042600+0530 Nero[2814:34333] [] nw_socket_handle_socket_event [C2034.1:1] Socket SO_ERROR [61: Connection refused]
2019-07-21 19:57:30.043500+0530 Nero[2814:34332] [] nw_connection_get_connected_socket [C2034] Client called nw_connection_get_connected_socket on unconnected nw_connection
2019-07-21 19:57:30.043989+0530 Nero[2814:34332] TCP Conn 0x600001af2280 Failed : error 0:61 [61]
websocket is disconnected: Optional("The operation couldn’t be completed. Connection refused")
Can you please look into this and let me know the solution to over the issue.
Thank you

This is going to be an issue with generic socket available and not specific to Starscream. It could be lot of different issues that are difficult to pinpoint without more knowledge into your exact setup. unfortunately that is the most help I can provide through an issue like this. Best of luck!

I'm in the process of reworking the WebSocket Transport layer for SwiftPhoenixClient and am trying to get the client's retry logic working with Starscream 4.0.4 (I dont believe it was working using Starscream 3.x so thats not related) and running into an issue with Starscream's behavior here.
My scenario is this. The Socket attempts to open a connection but the server is down or (more likely) the phone loses connection for a bit. I have logic to retry on a timed backoff however that logic is never called because the WebSocketDelegate does not inform the Client that this occurred.
I implemented the same behavior using URLSessionWebSocketTask by implementing urlSession(_:task:didCompleteWithError:) which is called, with an error when the connection was refused. Would it be possible to provide a WebSocketEvent that matches this behavior?
FWIW, here is the code.


patch: Add GHA Built Test Deploy workflows

Adds build deploy test workflow

[x] Create PR with all devices supported
[x] Set deploy environment to balena-cloud.com
[ ] Disable the associated multi-job and device jobs in Jenkins
[ ] All required devices are passing on GHA + doesn't require a rebase
[ ] Approved by the reviewer (policy-bot passed)
[ ] Update branch protection to require the GHA workflows
[x] Remove Jenkins build and deploy workflow (this PR)

@klutchell to make it explicit that worker_type is actually testbot?

@klutchell to make it explicit that worker_type is actually testbot?

Yes, though it's not required I find it helpful at the repo level to quickly see what the worker type is rather than read the comment and deduce it.


gdrive-linux-rpi caps speed at 2MBps (16Mbps)

While I am able to download at 50Mbps through my browser and close to 65Mbps through desktop (ubuntu - gdrive-linux-x64), the download rate stays at 16Mbps in gdrive-linux-rpi. I have tried with multiple connections.  Is this an intentional limitation?

I don't see the purpose and benefits of intentionally limiting the speed for a particular platform? If you're using the vanilla version of the pi (i.e. you did not do any mods to the networking on the pi), it's more likely that you've hit some sort of hardware speed cap on the pi itself.

True. I assumed it might've been done in consideration to the hardware
capabilities of Pi. All other downloads happen in full speed through wget &
curl.
I'm running the latest Raspbian Jessie Lite on RPi2. Ethernet connection.
Is there a way I can find out some debug info on this? A verbose mode or a
log file that I could post here?
On Jan 19, 2017 7:51 AM, "chuanzgh"<EMAIL_ADDRESS>wrote:

I don't see the purpose and benefits of intentionally limiting the speed
for a particular platform? If you're using the vanilla version of the pi
(i.e. you did not do any mods to the networking on the pi), it's more
likely that you've hit some sort of hardware speed cap on the pi itself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prasmussen/gdrive/issues/250#issuecomment-273663623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXaukUpfoteqv3FOD_B9uCfb6ZACGx-ks5rTsiTgaJpZM4LnNj-
.

Sadly no.
On Sun, Jul 1, 2018 at 1:55 PM, pvlvsk<EMAIL_ADDRESS>wrote:

I'm having the same problem.. @mccbala https://github.com/mccbala do
you have a solution by now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prasmussen/gdrive/issues/250#issuecomment-401592221,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXaujpCa8K7m9WZbsIO1dbCAb-h1tI6ks5uCIdggaJpZM4LnNj-
.


router-link failed

Why does router link go to other Vue components , the page content remains the same

The reason lies in    index.html =>  , This life is equal to the whole situation


Adding the ability to pick the label applied to an automatic reference

This changes the syntax for {{?RFCXXXX}} and {{!RFCBLAH}} to allow
the author to specify a name for the reference.  This is added after a
space, as in {{?RFC5389 STUN}}.  This affects the label used if symbolic
references are enabled, e.g., [STUN].  This is especially handy for DOI
references, which I just added because it keeps the symbolic reference
tidy.  For example, {{?DOI.10.1145/1282427.1282421 SST}}, which is a
reference we are using in QUIC.
The first half of the reference only needs to be used once.  You can refer
to {{? SST}} many times as long as there is a single definition that
has the full reference.  This was close to free, and may serve to replace
the abbreviation support for many.
I have basically no testing on <iref>.
This allowed me to consolidate the places where you were testing for
labels that started with a digit.  That too was necessary to have a
single location where the DOI reference was mangled.

@cabo, do you have any opinions on this feature?  I would like to be able to use DOIs for a reference in QUIC, but this is blocking it.  I could contribute a much smaller patch that just added DOI support, but this is much nicer.

Yes, I do.  It's about time kramdown-rfc grows this feature!
There are a couple of things I think need to be done before we can mainline this.  This week I have a heavy week of teaching, so I hope I can provide feedback next week.

Long week.
There are two things I'd like to obtain to be able to mainline this:

This feature needs to be available from the YAML header, not just from the YAML header autogeneration ({{?...}})

spaces in markup elements are icky as they turn into line breaks when reformatting source.

To address the latter, I would probably go for a syntax such as {{?XMPP=I-D.ietf-xmpp-3920bis}} -- this can then simply be referenced as {{XMPP}} in the rest of the document.
To address the former, I'd probably add an attribute "bib" or some such:
normative:
  XMPP:
    bib: I-D.ietf-xmpp-3920bis

that of course defaults to the anchor name.
(Now, that attribute could also provide a way to go beyond the current mapping in XML_RESOURCE_ORG_MAP.)

Yeah, long week is an understatement.  Thanks for looking at this.
I've rebased and changed to your recommended syntax (I wasn't happy with the original, so that was a welcome suggestion).  I decided to use the same syntax for the YAML part as well.  That also makes it easier to tell if you have a real definition for the reference.  You can see from the examples what this looks like.

Oh, one thing I realized after I finished up yesterday is that there is perhaps another alternative to this that is superior, though I won't have time to implement this.
I chose to do this:
informative:
  LOSS-PROBE=I-D.dukkipati-tcpm-tcp-loss-probe:

But this might also work:
informative:
  LOSS-PROBE: I-D.dukkipati-tcpm-tcp-loss-probe

The former is nice in that the syntax is the same throughout, but I had to jump through some hoops to get it to work cleanly.  The latter might be superior implementation-wise.  I can implement it if you would prefer that.

That place is already taken for the local label:
RFC7252: coap

is a shortcut for
RFC7252:
  -: coap

which is the thing that lets you reference RFC7252 as {{-coap}} -- a nice way if you don't want to change internal references each time a document transitions from individual to WG document to RFC, without exposing that internal name in the actual draft.
What I would like to add is the bib thing I had in the above example, what you called href in your code.

Assuming it is OK to use a hash key of "=" for that (we could also use "bib" or "href"), this could become:
informative:
  LOSS-PROBE:
    =: I-D.dukkipati-tcpm-tcp-loss-probe

(and you could even add a "-" to that as well, all though swapping out both anchor and local name doesn't seem to make much sense).
I have some more teaching to do tomorrow, but probably can throw all this together from your code over the weekend.
Other things i'd like to have:

being able to be explicit about the bibxmlX (i.e., not infer that path name component, but write it down if that helps).  We would still cache everything in one directory. (.refcache or ~/.cache/xml2rfc).

maybe:
    =: bibxml3/I-D.dukkipati-tcpm-tcp-loss-probe

(Hmm, I forgot the other one.)

Of course, I could hack it that if the "local name" contains a dot, it is really a bib name.  Hmm, does not work for RFC that well, but even that could be special-cased.  Maybe too much magic.
normative:
  TCP: RFC0793
  RFC7252: coap
  URIDESIGN:
    =: RFC7320
    -: lawn
...

{{TCP}} ... {{-coap}} ... {{-lawn}}

➔ [TCP].... [RFC7252] ... [URIDESIGN]
(Hmm.)

The one thing that stopped me in me track yesterday was that I don't know what to do for stand_alone: false (i.e., where entity references are generated).  Clearly, label picking cannot be supported with that setting.  For a while, I tried coming up with a way a label picking document at least compiles, even if the label picks are ignored.  But it seems, without invasive changes in alias handling, this cannot be done.  If that is so, so be it (in that case, we'll just need an error message).

Yeah, standalone: false is a trap.  I've learned that giving people options is only good if they need those options.  I'm struggling to find a reason that standalone: false is worthwhile here, other than the fact that you can "build" offline.  Your new caching options are superior in that regard, of course.

stand_alone: false is part of the "You can take your data elsewhere" promise of kramdown-rfc.
If you are ever fed up with authoring in markdown, just make this setting, process one last time, and you have a fully editable XML again that automatically updates draft versions from the XML.
So if there has been some label picking, I think it is better to generate a stand_alone: false XML that requires some manual touch-up than to give up.  So I would turn the error into a warning.

I think I have cherry-picked all I needed from this PR in 1.1.0 -- please tell me if I missed any functionality.  Thank you!


Nanopore medaka processing fails with error if model name, not model file, provided

Description of the bug
When a medaka model is specified by model name, not path to model file, viralrecon will fail with an error. The error (see below) seems to come from this line of code.
When a model filename is passed (I extracted the models from the Singularity image) the command runs to completion.
Command used and terminal output
$ nextflow run nf-core/viralrecon --input samplesheet.csv --platform nanopore --genome MN908947.3 --primer_set_version 3 --fastq_dir fastq_pass --artic_minion_caller medaka --artic_minion_medaka_model r941_min_high_g360
...
Error executing process > 'NFCORE_VIRALRECON:NANOPORE:ARTIC_MINION (Sample_01)'

Caused by:
  Argument of `file` function cannot be empty

Source block:
  def args = task.ext.args   ?: ''
  prefix   = task.ext.prefix ?: "${meta.id}"
  def version  = scheme_version.toString().toLowerCase().replaceAll('v','')
  def fast5    = fast5_dir ? "--fast5-directory $fast5_dir"             : ""
  def summary  = sequencing_summary ? "--sequencing-summary $sequencing_summary" : ""
  def model    = ""
  if (args.tokenize().contains('--medaka')) {
          fast5   = ""
          summary = ""
          model = file(medaka_model).exists() ? "--medaka-model ./$medaka_model" : "--medaka-model $medaka_model"
      }
  def hd5_plugin_path = task.ext.hd5_plugin_path ? "export HDF5_PLUGIN_PATH=" + task.ext.hd5_plugin_path : "export HDF5_PLUGIN_PATH=/usr/local/lib/python3.6/site-packages/ont_fast5_api/vbz_plugin"
  """
      $hd5_plugin_path

      artic \\
          minion \\
          $args \\
          --threads $task.cpus \\
          --read-file $fastq \\
          --scheme-directory ./primer-schemes \\
          --scheme-version $version \\
          $model \\
          $fast5 \\
          $summary \\
          $scheme \\
          $prefix

      cat <<-END_VERSIONS > versions.yml
      "${task.process}":
          artic: \$(artic --version 2>&1 | sed 's/^.*artic //; s/ .*\$//')
      END_VERSIONS
      """

Relevant files
No response
System information
Nextflow version: 22.01.0-edge
Hardware: HPC
Executor: slurm
Container engine: Singularity
OS: Ubuntu 18.04
Version of nf-core/viralrecon: 2.3.1 (97bebf8fe1)

Thanks for reporting @pvanheus ! This will be fixed when #282 is merged into dev. It was a bit of an edge case bug because artic minion can accept a model as either a file or a string and if the model is a file then we need to stage it properly as an input. Things broke when I tidied up the module to remove any params definitions but I have added a CI test for this now so we should be able to pick it up in the future!
Be great if you are able to test once #282 is merged:
nextflow pull nf-core/viralrecon

nextflow run nf-core/viralrecon \
    --input samplesheet.csv \
    --platform nanopore \
    --genome MN908947.3 \
    --primer_set_version 3 \
    --fastq_dir fastq_pass \
    --artic_minion_caller medaka \
    --artic_minion_medaka_model r941_min_high_g360 \
    -r dev

I will create a patch release when you have confirmed.
Have a good weekend!


Cannot download from deb

Problem statement
Added repository to the list but could not finish apt update because of the repository, it gives the error: E: The repository 'https://dl.bintray.com/go-swagger/goswagger-debian ubuntu Release' is not signed.
Steps to reproduce
echo "deb https://dl.bintray.com/go-swagger/goswagger-debian ubuntu main" | sudo tee -a /etc/apt/sources.list
sudo apt update

Environment
OS:   Ubuntu 18.04

Hi, I got the same issue. Did you solve it?

Hi, I got the same issue if I add the debian path to my /et/apt/sources.list.
When I do apt update, I encounter the following error:
E: The repository 'https://dl.bintray.com/go-swagger/goswagger-debian ubuntu Release' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
There should be a way to sign the debian package, which is missing!

Duplicates #1812


Script error running open_pdks's sky130.lydrc from klayout

It's unclear if this should work, but since it seems to work for at least one other person I'm going to assume it should.
When I try to run the sky130.lydrc script that ships with open_pdks, klayout throws up this error:

If I click 'ok' or 'ignore', I get this error:

I get the same behaviour if invoking from the menu or from the IDE (once I've fiddled with it enough that the right script runs).
sky130.lydrc.zip

@growly :  How about the (installed) file $PDK_ROOT/sky130A/libs.tech/klayout/tech/sky130A.lyt?  I sort of gave up on the file kept in open_pdks because nobody would claim responsibility for maintaining it.  So now instead I pull https://github.com/mabrains/sky130_klayout_pdk and install that, because at least there's somebody taking responsibility for it.

$PDK_ROOT/sky130A/libs.tech/klayout/tech/sky130A.lyt works for importing the tech into klayout but doesn't include DRC, does it?
Thanks, the mabrains repo just points to the efabless mpw precheck drc script so I'll focus on that.


Test: Git - Support SSH keys with a passphrase

Refs: https://github.com/microsoft/vscode/issues/13680

[ ] macOS @paulacamargo25
[ ] linux @roblourens
[ ] windows @aiday-mar

Complexity: 4
Create Issue

This milestone we have added support for using SSH keys with a passphrase when they are not added to the ssh-agent.
Prerequisites

Ideally you would want to follow these steps on a machine that does not yet have any SSH keys locally
Create a new SSH key with a passphrase. Note that GitHub has different steps based on platforms.
⚠  Please do not go through the steps to add the newly created SSH key to the ssh-agent.
Add the newly created SSH key to your GitHub profile.
Test the newly created SSH key

Testplan

Launch VS Code Insiders
Navigate to a private repository on GitHub, expand the "Code" button and grab the SSH clone URL
Open the integrated terminal, create a new folder and clone the repository using the SSH clone URL

Confirm that you are prompted for a passphrase in the terminal

Open the cloned repository in VS Code Insiders
Make a change to a file and commit the changes
Use the command palette to invoke the "Git: Push" command

Confirm that a quick input is being shown to enter the passphrase

Enter the passphrase and hit Enter

Confirm that the change is being pushed to the remote repository

Make a change to a file and commit the changes
Use the command palette to invoke the "Git: Push" command

Confirm that a quick input is being shown to enter the passphrase

Press Escape without entering the passphrase

Confirm that the push does not occur and no errors are being shown

sorry, Im not familiar with vscode sprints or something like that. when can we expect this feature to go live?

@4c74356b41, this feature will be included in the next Stable release that is planned around October 5th.


Haskell mode REPL hanging

Hi Purcell,
I found an issue  in haskell mode. When I use C-c, C-l to load file to RPEL, the REPl will freeze.
The detailed steps are as below:

Load a haskell file into a new GHCi session using C-c C-l
Use the REPL to evaluate functions, it worked well for this moment
Switch back to the source code  using eitherC-c C-zor C-x o
Make some changes, or  add more functions and save source with C-x, C-s.
Use C-c C-l to load file again, and use C-x, o to switch to the REPL
And then REPl hangs , C-q can exit the hanging, but need to restart emacs to restore normal.

My system info:
Window 10 pro 1903
emacs-26.2-x86_64
I tried to disabled  flycheck and company , the issue still exist.  But I tried to install Haskell-mode in my personal emacs config , there is no this issue.
I use M-x  toggle-debug-on-quit  to enable the debug mode and get the infomation as below picture(you can get details form the file attached):

debug log
I also searched issues from Haskell -mode  of github and goole.
I just found a similar issue but no finall solution.
https://github.com/haskell/haskell-mode/issues/447
So can you give me some advice or directions , how I can dig for it?
Thank you in advance.

This is a Windows 10 specific issue, I have been move to linux and Mac osx, and everything works well now.


Binary Search in golang

This issue is about golang

[ ] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[x] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[X] I will send my PR only after I'm assigned to this issue by any maintainer

[X] Assign this issue to me (I want to work on this)

[X] I ensure that I am not already assigned to 2 or more issues

@Ayush-Mahajan Can you Assign this issue to me? and add tag hacktoberfest?


Integrate llvm/llvm-project@27ac46e6bea2

Updated to llvm/llvm-project@27ac46e6bea2

Used LLVM MathExtras.h to replace MLIR one
Updated applySignatureConversion usage

Updated to openxla/stablehlo@dd48ec5

chlo.minimum_broadcast_shapes op was removed
https://github.com/openxla/stablehlo/pull/2287
chlo.dynamic_reshape op was removed
https://github.com/openxla/stablehlo/pull/2286
Added batching dims to scatter dims
https://github.com/openxla/stablehlo/pull/2259

Updated to llvm/torch-mlir@77d7f64

This seems like a fairly likely candidate for the source of dispatch count changes: https://github.com/iree-org/llvm-project/commit/7ef83f5561b34ca07fdef23ca2b3c01c583dbbf5
Especially because the changes are being observed in data tiling enabled benchmarks. cc @Max191

We need to look at the regressions in number of dispatches. I can help (but not today).

@MaheshRavishankar are you blocking the integrate for this or would you look at it in a follow up since Quinn has explained the reason for the difference?

We need to look at the regressions in number of dispatches. I can help (but not today).

@MaheshRavishankar are you blocking the integrate for this or would you look at it in a follow up since Quinn has explained the possible reason for the difference?

Could you try reverting that locally to see if that is the issue. Then we can decide what to do next

We need to look at the regressions in number of dispatches. I can help (but not today).

@MaheshRavishankar are you blocking the integrate for this or would you look at it in a follow up since Quinn has explained the possible reason for the difference?

Could you try reverting that locally to see if that is the issue. Then we can decide what to do next

@MaheshRavishankar PTAL at the benchmark comment now, the bot has edited it and it seems the dispatch number regression is gone with the revert.


Add support for H2 database

For simple setups and just to get started quickly, we should support a file-based H2 database. We could store the file under $OSIAM_HOME/data/osiam.db (or whatever file extension H2 uses). The H2 database would also be used in the default configuration, so that one doesn't even need to configure anything to get started with OSIAM. This could also be helpful for the Docker image. I'd like to see this in 3.0, as it's fully aligned with one of the goals of OSIAM 3.0: making setup less complicated.

I agree, it's something that we should provide with 3.0

Blocked by #172