TODO & feature request

Please add features you think necessary in this issue.

[ ] line spacing: waiting for reply from GS.
[ ] lstlisting style for other languages: remove or add?
[ ] signature page: list of TEC names? RPG handbook does not mention this.
[ ] documentation: In README / mythesis / dtx, or all of them?

I have some suggestions for documentation. For the Readme page, we just need to provide a brief introduction of this
project and guide users to the manuals or online docs (so I recommend host an online doc by GitHub pages).
Another thing, it seems the font size of the thesis title can be larger, and we can choose a more beautiful font by default.

I have some suggestions for documentation. For the Readme page, we just need to provide a brief introduction of this
project and guide users to the manuals or online docs (so I recommend host an online doc by GitHub pages).
Another thing, it seems the font size of the thesis title can be larger, and we can choose a more beautiful font by default.

I am working on README and PDF documentations. It is a good idea to host docs on Pages. I can start to work on it after I complete the PDF documentation.
I will have a look at the font size. For font type, I think Times New Roman (on Windows/MacOS) is the most commonly used so I do not want to change default font to another one. But there is, IIRC, a setting allowing users to have full control on customizing the fonts.

I can help work on the Makefile and online documentation.

Hi! I really loved your template. I am thinking on using it for my own thesis on Overleaf. I have a request though: Can it be modified to be in B5 instead of A4?
Thank you in advance.

Hi! I really loved your template. I am thinking on using it for my own thesis on Overleaf. I have a request though: Can it be modified to be in B5 instead of A4? Thank you in advance.

Thank you for your love.
If you want to use b5 instead of a4, you can modify the hkustthesis.cls file, search for a4paper and then replace it with b5paper. This is the easiest method for general users to achieve this.
However, I have not tested b5paper page size, so I am not sure whether there will be new bugs. Please file issues if any. Thanks.

Hi! I really loved your template. I am thinking on using it for my own thesis on Overleaf. I have a request though: Can it be modified to be in B5 instead of A4? Thank you in advance.

Thank you for your love.
If you want to use b5 instead of a4, you can modify the hkustthesis.cls file, search for a4paper and then replace it with b5paper. This is the easiest method for general users to achieve this.
However, I have not tested b5paper page size, so I am not sure whether there will be new bugs. Please file issues if any. Thanks.

Thank you very much for your quick answer. I will try it and test if there is any bug.


Disable camera until use

This is a bug. My camera is on when app is ran.

17fa10b1002bdb695ed6029bed8c3bd2ce7680f2 disabled for now


codegen-kotlin: Don't generate equals/hashCode/toString/copy/:Serializable for abstract classes

Motivation:
Don't generate code that isn't useful.
Changes:

Don't generate equals/hashCode/toString/copy methods for abstract classes.
Don't make abstract classes serializable.

Result:

Less code is generated.
Fixes #713.

Note:
codegen-java does the same for stateless classes, but it also makes them non-instantiable.
codegen-kotlin generates stateless regular classes far less often than codegen-java.

I'd like to refactor KotlinCodeGeneratorTest in the same way I refactored JavaCodeGeneratorTest.
Before I embark on this journey, it would be good to get #710 approved/merged.

Refactored KotlinCodeGeneratorTest in the same way as JavaCodeGeneratorTest in #710.
Ready to review from my side.


Build Error on Linux

When running a build on a Linux (Swift 5.10.1) it fails because Ignite is using URL functions that are not currently available on Linux. E.g.
/home/runner/work/iOSDevHappyHourSite/iOSDevHappyHourSite/.build/checkouts/Ignite/Sources/Ignite/Publishing/PublishingContext.swift:376:21: error: cannot infer contextual base in reference to member 'reverse'
            order: .reverse
                   ~^~~~~~~
/home/runner/work/iOSDevHappyHourSite/iOSDevHappyHourSite/.build/checkouts/Ignite/Sources/Ignite/Publishing/PublishingContext.swift:383:49: error: value of type 'URL' has no member 'appending'
            let destinationURL = buildDirectory.appending(path: site.feedConfiguration.path)
                                 ~~~~~~~~~~~~~~ ^~~~~~~~~
/home/runner/work/iOSDevHappyHourSite/iOSDevHappyHourSite/.build/checkouts/Ignite/Sources/Ignite/Publishing/PublishingContext.swift:384:79: error: cannot infer contextual base in reference to member 'utf8'
            try result.write(to: destinationURL, atomically: true, encoding: .utf8)
                                                                             ~^~~~
/home/runner/work/iOSDevHappyHourSite/iOSDevHappyHourSite/.build/checkouts/Ignite/Sources/Ignite/Publishing/PublishingContext.swift:396:49: error: value of type 'URL' has no member 'appending'
            let destinationURL = buildDirectory.appending(path: "robots.txt")
                                 ~~~~~~~~~~~~~~ ^~~~~~~~~
/home/runner/work/iOSDevHappyHourSite/iOSDevHappyHourSite/.build/checkouts/Ignite/Sources/Ignite/Publishing/PublishingContext.swift:413:51: error: value of type 'URL' has no member 'appending'
        let destinationDirectory = buildDirectory.appending(path: destination)
                                   ~~~~~~~~~~~~~~ ^~~~~~~~~
/home/runner/work/iOSDevHappyHourSite/iOSDevHappyHourSite/.build/checkouts/Ignite/Sources/Ignite/Publishing/PublishingContext.swift:431:49: error: value of type 'URL' has no member 'appending'
            let destinationURL = buildDirectory.appending(path: "js/syntax-highlighting.js")
                                 ~~~~~~~~~~~~~~ ^~~~~~~~~

This is more for tracking and notifying as this should be fixed in Swift 6 (which will require #81 to be solved)

Hello! Yes, I talked to one of the team at the Swift anniversary thing during WWDC, because I was confused and considering going back to the older APIs! I'm looking forward to this one being resolved.

Hello! Yes, I talked to one of the team at the Swift anniversary thing during WWDC, because I was confused and considering going back to the older APIs! I'm looking forward to this one being resolved.

Thanks from Twitter :)

Has anyone seen this "fix itself" with a Swift 6 toolchain on Linux yet?

@jpurnell yes this builds fine on Linux with Swift 6


Teams personal app settings

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
no

What's in this Pull Request?
Sample web part that demonstrates how you can store Teams Personal App Web Part's properties in user's OneDrive.

@AJIXuMuK this looks like it is going to be an awesome sample.
Would you mind making a few tweak before we accept the PR?

You seem to have a few changes to the react-officegraph sample... was that intended?
Would you mind providing a little bit of info on how to deploy/use the solution? (i.e.: the ## Minimal path to awesome section
Can you include a screen shot of the web part?

I look forward to accepting this PR... and you know what question comes next, right?
Would you like to present this sample at an upcoming SPFx community call? We have an opening on June 18th.

Hey @hugoabernier!
I pushed the changes you asked for. Sorry for not providing the image and minimal path to awesome right away.
Regarding changes in react-officegraph - it looks like really old change that lived in my master branch related to line endings.
If it's a an issue, I can re-fork the repo and copy my changes to the new forked repo.

And I'm open for June 18 :)

Thank you @AJIXuMuK, it's always a pleasure to review your samples.
As discussed, I have you scheduled for June 18th. Looking forward to see your demo!


Detail about what the response time measures

https://docs.microsoft.com/en-us/azure/application-insights/app-insights-web-monitor-performance#average-response-time
"Measures the time between a web request entering your application and the response being returned." - need to explicitly call out what this time includes and what it doesn't. (Call out that time spend in IIS queue is not counter in the case of Asp.Net, etc)

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

ID: cca7b295-429c-bd6c-cfe0-8a2f7ad30298
Version Independent ID: c4c072af-b19c-94e2-0d77-9418767abac3
Content: Monitor your app's health and usage with Application Insights
Content Source: articles/application-insights/app-insights-web-monitor-performance.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

Also need to mention about whether the time includes the overhead by the SDK itself.

@cijothomas Thank you for pointing this out.  I'm going to assign this to the document author so this can be updated accordingly.

@kobulloc-MSFT In general Application insights platform metrics don't have a description that I could find.
I think this item should track adding a table with those details.

@yanivlavi we do have this for Azure Monitor supported metrics all up, but the table does get a bit difficult to read and I am not sure how up to date it is at the moment: https://docs.microsoft.com/azure/azure-monitor/platform/metrics-supported.

@mrbullwinkle It's good the know it exists, but Google first page doesn't find it:
https://www.google.com/search?rlz=1C1CHBF_en__821__821&ei=EUBgXITVCMSZ1fAPlZWg2Ak&q=Server+response+time+azure+monitor&oq=Server+response+time+azure+monitor&gs_l=psy-ab.3..33i22i29i30.2374.5812..6013...0.0..0.164.1746.0j14......0....1..gws-wiz.......0i71j0i67j0j0i22i30.CxZkqt6AZN0
I also think paging the tables would make that page more readable.

#please-close


fix(errors): Update deprecated error handling #5

Please see if this is the appropriate fix you desired.

Hi @Nabeelhassan, thanks for the pull request. This fixes the issue for one of the error cases.
Could you implement this for all error handling?

By for all you mean the secret:get command ?

I actually meant across all of the commands. Anywhere within the project that we catch an error.

could you please run/review it ? I hope it is up to the mark.

Looks good. Thanks for this. 👍

Thank you too. stay blessed.


"Custom AMP frontpage" feature works, but doesn't show correct "Title"

Kudos for adding the "Custom AMP frontpage" feature on the latest update.  I tried it out and it works but I notice that the "title" doesn't render correctly.  It still says "Page Not Found."
I tried flushing rewrite rules and clearing cache already.  Here's an image.

Please update to the latest version, it should fix the issue.


Hl-827, HL-816, HL-824: New applicant login page and Accessibility statement

Description :sparkles:

HL-827: New applicant login page style
HL-816: Accessibility Statement page
HL-824: Make PDF links clickable

The accessibility statement is only in Finnish. Login page texts should be translated by a professional later.

Codecov Report

Merging #2111 (3e5fd74) into develop (dc1e3a5) will decrease coverage by 15.71%.
The diff coverage is 3.44%.

:exclamation: Current head 3e5fd74 differs from pull request most recent head 4966ad1. Consider uploading reports for the commit 4966ad1 to get more accurate results

@@             Coverage Diff              @@
##           develop    #2111       +/-   ##
============================================
- Coverage    33.43%   17.73%   -15.71%
============================================
  Files          772      122      -650
  Lines        14886     2899    -11987
  Branches      3137      725     -2412
============================================
- Hits          4977      514     -4463
+ Misses        9421     2292     -7129
+ Partials       488       93      -395

Impacted Files
Coverage Δ

...benefit/applicant/src/components/footer/Footer.tsx
0.00% <0.00%> (ø)

...nefit/applicant/src/components/layout/Layout.sc.ts
0.00% <0.00%> (ø)

...benefit/applicant/src/components/layout/Layout.tsx
0.00% <0.00%> (ø)

...benefit/applicant/src/components/pages/Pages.sc.ts
0.00% <0.00%> (ø)

...t/applicant/src/components/pdfViewer/PdfViewer.tsx
0.00% <0.00%> (ø)

...ontend/benefit/applicant/src/hooks/useUserQuery.ts
0.00% <0.00%> (ø)

frontend/benefit/applicant/src/constants.ts
100.00% <100.00%> (ø)

... and 649 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix: Improve error handling and fix wrong compress dir for windows

Refactored the CompressDir function in the fileutil package for improved error handling and handling of edge cases
Improved error messages for various file operations
Added test file and created tests for the CompressDir function in the fileutil package

Proposed changes
Close https://github.com/moonD4rk/HackBrowserData/issues/366

Checklist

[ ] Pull request is created against the dev branch
[ ] All checks passed (lint, unit, build tests etc.) with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

27 of 40 (67.5%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.6%) to 18.585%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/hack-browser-data/main.go
0
1
0.0%

utils/fileutil/filetutil.go
27
39
69.23%

Files with Coverage Reduction
New Missed Lines
%

utils/fileutil/filetutil.go
1
32.04%

Totals

Change from base Build<PHONE_NUMBER>:
1.6%

Covered Lines:
373

Relevant Lines:
2007

💛  - Coveralls


Fix typos

Hey,
This tiny PR will fix :

1 typo in .github/CONTRIBUTING.md
1 typo in pages/space.js

Bye! :robot:

📝Changed routes:

/space

(view full size)

Commit 8d61ca925366f712a56d4ee7a247e558ce88225c (https://coderplex-git-fork-pgrimaud-master.coderplex.now.sh).


Handle permission denied in geolocation service

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.6
      CPU: x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
      Memory: 29.59 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 10.6.0 - /usr/local/bin/node
      Yarn: 1.7.0 - /usr/local/bin/yarn
      npm: 6.1.0 - /usr/local/bin/npm
      Watchman: 4.7.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 11.3, macOS 10.13, tvOS 11.3, watchOS 4.3
      Android SDK:
        Build Tools: 23.0.1, 23.0.3, 25.0.0, 25.0.2, 25.0.3, 26.0.1, 27.0.3, 28.0.0
        API Levels: 23, 24, 25, 26, 27
    IDEs:
      Android Studio: 3.2 AI-181.55<IP_ADDRESS>56338
      Xcode: 9.3.1/9E501 - /usr/bin/xcodebuild
    npmPackages:
      @storybook/react-native: 3.4.11 => 3.4.11
      @types/react: 16.4.14 => 16.4.14
      @types/react-native: 0.55.22 => 0.55.22
      react: 16.5.2 => 16.5.2
      react-native: 0.57.2 => 0.57.2
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-git-upgrade: 0.2.7

Description
On Android, if the user denies location permissions, the getCurrentPosition function in Geolocation.js does not call the geo_error callback as expected.
As per the docs regarding permissions for android SDK 23+, access must be granted for certain permissions. In the case of the location permission, the RN Geolocation service handles this in the getCurrentPosition function (with some issues). These issues could be avoided by doing the android permission request yourself but it seems like this should either be fixed or removed from the geolocation service code to avoid confusion.
It appears the correct fix would involve moving the android permission check/request code into the android LocationModule.java file, as the rest of the error handling knowledge live within the java code and this is how it is done in iOS.
We have added a temporary fix for this internally by patching getCurrentPosition so it calls the geo_error callback if permission is denied (see below) but as mentioned, this does not seem to be the proper place for this code.
Figured I would open the issue in case anyone else is running into a similar issue, and with the hopes that this will eventually get properly fixed.
export const getCurrentPosition = async function(
  geo_success: Function,
  geo_error?: Function,
  geo_options?: GeoOptions
) {
  invariant(typeof geo_success === 'function', 'Must provide a valid geo_success callback.')
  let hasPermission = true
  // Supports Android's new permission model. For Android older devices,
  // it's always on.
  if (Platform.OS === 'android' && Platform.Version >= 23) {
    hasPermission = await PermissionsAndroid.check(
      PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION
    )
    if (!hasPermission) {
      const status = await PermissionsAndroid.request(
        PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION
      )
      hasPermission = status === PermissionsAndroid.RESULTS.GRANTED

      // the patch:
      if (!hasPermission) {
        geo_error({ code: PERMISSION_DENIED_ERROR_CODE })
      }
    }
  }
  if (hasPermission) {
    RCTLocationObserver.getCurrentPosition(geo_options || {}, geo_success, geo_error || logError)
  }
}

Reproducible Demo
Create a react native app and request location via the Geolocation servie. If location is denied on android, the geo_error callback will not be triggered as it is on iOS.

Working on this

Quick Workaround I'm using:
async function getLocation(onSuccess, onFail) {
    if (Platform.OS === 'android') {
        // https://github.com/facebook/react-native/issues/22535
        const permission = await PermissionsAndroid.request(PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION)

        if (permission === PermissionsAndroid.RESULTS.GRANTED) {
            navigator.geolocation.getCurrentPosition(
                async (position: Position) =>  { onSuccess() },
                async (err: PositionError) => { onFail() }
            )
        } else {
            onFail()
        }
    } else {
        navigator.geolocation.getCurrentPosition(
            async (position: Position) => { onSuccess() },
            async (err: PositionError) => { onFail() }
        )
    }
}

This issue can probably be closed once GeoLocation has been removed from core RN.  The fix for this bug has been filed in the new repository as: https://github.com/react-native-community/react-native-geolocation/pull/1

Thanks @Jyrno42, Looking at your PR now!

Good job. Working for me


Remove C2R requirement step as it appears MSI is supported

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Help me with graph communication calls api

Could someone help with an example on how to call a team contact through api rest? I have seen the documentation of Graph Communication Calls but I do not understand how to do it.

https://docs.microsoft.com/en-us/graph/api/application-post-calls?view=graph-rest-beta&tabs=javascript
@VinodRavichandran @zhengni-msft do you know when we're doing a documentation sync?  Not all the examples in these are correct.

@MiguelAngelSilva are you using an SDK and one of the samples, or are you using REST API directly?

I want to use the Microsoft Graph sdk, but when I read the documentation I can not understand how to create a call that calls a Microsoft Teams contact.

@MiguelAngelSilva It sounds like you are looking for a client SDK.  The API and SDK is meant to support server scenarios as a "Bot".  You need to have a publicly visible callback URI.
Do you require access to the raw media streams?
If you have a Windows Forms Application, you have a stateful application.  You should leverage the stateful SDK:
https://www.nuget.org/packages/Microsoft.Graph.Communications.Calls
Samples can be found here, keep in mind that these are all web applications:
https://github.com/microsoftgraph/microsoft-graph-comms-samples

Thanks @ksikorsk  , are there any explanations about the architecture that you implemented in the examples?

The samples try to stay simple... but they are stateful, so require call affinity.
Some documentation here on the app hosted media samples.
And the specific documentation for AudioVideoPlaybackBot and HueBot.

Closing this issue as the question was too generic, and guidance was provided.

The samples try to stay simple... but they are stateful, so require call affinity.
Some documentation here on the app hosted media samples.
And the specific documentation for AudioVideoPlaybackBot and HueBot.

What kind of project have you created for IncidentBot?
is Asp Net Web Api? Bot Framwork project? I do not know.
And where did the directory structure created in the project come from (Folders: Bot, Controller, Data, Extensions and IncidentManagement)?

Call affinity means that the instance of the application that handles the initial request needs to be the same instance that handles the call throughout the lifetime of the call.  There is affinity between the call and the instance handling the call.  In a multi instance environment (which the samples support), all notifications for the same call need to be routed to the same instance, given that that is where the call state is homed.
Incident bot is an ASP.NET Core Web Application leveraging the Microsoft.Graph calling APIs, and using the Microsoft.Graph.Communications SDK.
The directory structure is a logical grouping of content:
Bot -> The bot protocols for placing/answering calls, checking for participants, playing media, etc...
Controllers -> The Web API component of the sample.  This is how the sample communicates with other services.
Data -> The objects the sample uses for communication using it's rest API (exposed in Controllers)
IncidentManagement -> The business logic of the IncidentBot.

you are the best! @ksikorsk
Thanks! But  how can I use your example to have the bot redirect or transfer a call to a contact? In which class the code should be implemented to transfer the call. Could you help me or give me an example? since in the examples none uses redirection or call transfers.

Hi,
Getting Below error while trying to fetch call details-
{
"error": {
"code": "UnknownError",
"message": "",
"innerError": {
"date": "2020-06-30T12:32:22",
"request-id": "9b8597c1-f6bf-483a-953b-809760bb7e7a"
}
}
}
https://graph.microsoft.com/v1.0/communications/calls/7ce57e74-f13e-4b83-8ab7-08518db159c9
I have provided token in header as well.
Any help would be much appriciated.


chore: fixed dead_code warning for test-only code

Fixed:
warning: associated function `new` is never used
   --> src/client.rs:153:12
    |
152 | impl<InnerClient> ProfilingClient<InnerClient> {
    | ---------------------------------------------- associated function in this implementation
153 |     pub fn new(inner: InnerClient) -> Self {
    |            ^^^
    |
    = note: `#[warn(dead_code)]` on by default

The windows build just failed twice in a row with an error I've never seen before:
ERROR: Analysis of target '//prost:tonic_runtime' failed; build aborted: Failed to clean up module context directory: C:/b/modextwd/rules_rust~~_crate~~~crate (Directory not empty)

Ugh, support for Windows in bazel wold is lacking a lot. Glad it worked on third attempt, I have not seen this issue before yet.


snake case of "ImageURLs" returns "image_ur_ls"

Is it intended to separate an uppercase letter followed by a lower case letter with an underscore?
For example, snakeCase("ImageURLs") returns "image_ur_ls". Shouldn't it return "image_urls"?
Link to try: https://runkit.com/embed/7g2z45mdxmoc

This is currently the intended behavior. You can customize it like mentioned in the README. Imagine the browser APIs such as XMLHttpRequest. It’s unfortunate but impossible to handle every case without a dictionary.


Quiz/Practice toggle not updating.

Pressing "Practice" doesn't change the toggle. It remains on Quiz.

Resolved fixing JQuery issues.


Where does the telize API configuration file go?

Can you please tell me Where the telize API configuration file go? I'm running CentOS 7 and OpenResty/NGINX.

Closing this as it's a distribution specific configuration issue, and has nothing to do with the open source project.


Error when running with no config file specified

I'm trying to set up my *Configuration classes with defaults in the class, so I don't need to tell the app a config file when it's run in development.  However, when I run the app, I get this error:
default configuration has an error:
  * Unrecognized field at: database.healthCheckValidationTimeout
    Did you mean?:
      - logValidationErrors
      - removeAbandonedTimeout
      - validationQueryTimeout
      - initializationQuery
      - validationQuery
        [30 more]

I can see that DataSourceFactory has a getHealthCheckValidationTimeout, method, but I see no matching setter.  From what I see in the code, getValidationQueryTimeout and setValidationQueryTimeout read/write the same field as getHealthCheckValidationTimeout.
I'm unable to find the "default configuration" that specifies database.healthCheckValidationTimeout to attempt changing it to database.validationQueryTimeout and see if that works.
I think this is a bug, but I'm not sure.

@lewisd32 Which version of Dropwizard are you using?

I should have mentioned I was using 0.9.0, but it looks like you figured it out. Thanks! :)

wr


Added support to server side applications and server side example

The support was added by editing the GetModel function so it will pass IJSRuntime from page where Monaco editor was called

Could we get this merged?

This PR does not fix the issue but patches it for just one use-case. Instead of that, I've deprecated the static IJSRuntime instance and added support for server-side apps to provide their own IJSRuntime instances when needed. You can find the fix here : #98


Greenrobot PR -- Ask questions to understand the code better DO NOT MERGE

@Koshkap @alpha4s @FarahAbdalla @ArshadurRahman @Ali100k
Here is how you can ask questions to @Aman-Anas , @habibmostafa , Ahmed Abulila, @yousufbadar , @Github4Atif  and @Arshad-Anas.

Go to https://github.com/raleighmasjid/marvels-ftc/pull/31
Click on "Files changed"
Read the code line by line and make sure you understand it.
Click on plus sign next to the line you don't understand.
Add a comment and tag someone you think knows the answer or don't tag.
Click on "Add single comment".

Resolve conversation if the answer makes sense to decrease clutter.

Resources:
What is roadrunner: https://learnroadrunner.com/#frequently-asked-questions


Error on close

Hi there,
I'm using the plugin in combination with ouibounce (https://github.com/carlsednaoui/ouibounce#cookie-domain).
I'm getting a strange error every time I close the modal:
Uncaught TypeError: Cannot read property 'open' of undefined
I'm using this method to call the modal:
$('[data-remodal-target=modal]').remodal(options).open();

Any idea?
This is the link: http://www.ignitehospitality.com/temp_hosting/dev/
Just hover over the navigation bar when the page is loaded.
Thanks in advance!

You should use: [data-remodal-id=modal]

data-remodal-target is used for the links only.

Thanks @VodkaBears that works but it leaves the hash on the URL so when you click the back button it triggers the modal again even if "hashTracking" is set to false.

Sure? data-remodal-options="hashTracking: false". It is not set on your page.
<div class="remodal" data-remodal-id="modal" data-remodal-options="hashTracking: false">

Options are unneccesary -> $('[data-remodal-id=modal]').remodal().open();.

In your case $('[data-remodal-id=modal]').remodal() returns the modal without the re-creation.

@VodkaBears thanks for the clarification! It works like a charm now!
Great plugin by the way!


AppCenter Implementation

This connect's our Master builds to AppCenter so we can have CI for all merged pull requests.
AppCenter allows for:

Automatic builds
App Signing
App Deployments
App Testing
App Distribution

@IsaacHardy Also, I noticed that the tests failed because something to do with Coveralls. Shrug.


[DOCS] Please, provide a kotlin build.gradle.kts sample in documentation

@dodalovic thanks for reporting!
Unfortunately, I'm a complete noob when it comes to gradle. The gradle plugin was mostly contributed by @rferreira and @edward3h. Maybe they can provide help there.
If not and you find out how to do this with Kotlin DSL, I'd be glad for any update!

The snippet being referred to is not a top level block in the gradle build, which probably contributes to the confusion. I'll see if I can improve the example.

Aww, wow that was quick! Thanks for looking into it @edward3h!


[Docs] Fix typo in testing docs

Took me a good 15 minutes to realise the component had a typo
Changes

Fix a small typo in the camel-testcontainers-junit5 and camel-testcontainers-junit5 module links.

Thanks


Fixed linting error in controller.go

Fixes #93
Signed-off-by: Rupesh Krishna Jha<EMAIL_ADDRESS>

Codecov Report

Merging #141 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #141   +/-   ##
=======================================
  Coverage   55.77%   55.77%
=======================================
  Files          41       41
  Lines        1533     1533
=======================================
  Hits          855      855
  Misses        621      621
  Partials       57       57

Impacted Files
Coverage Δ

cmd/controller/controller.go
18.66% <ø> (ø)
: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 ec007d6...517f9e5. Read the comment docs.

@Akash4927  please review


[Local Testing] SNOW-931294 Support DataFrame.createOrReplaceView and DataFrame.createOrReplaceTempView

Please describe how your code solves the related issue.
This PR enables DataFrame.createOrReplaceView and DataFrame.createOrReplaceTempView by changing the existing TableRegistry class to store a separate map of string to MockExecutionPlan pairs, where the key is the view name (str) and the value is an MockExeuctionPlan instance corresponding to the underlying query of a View . When users fetch from a view using Session.table, we execute the MockExecutionPlan to obtain the results. In this way, changes in dependencies (e.g. sum(*) from a table) will be correctly reflected.

@sfc-gh-stan I have fixed the datatype issue of the group by column by calculating the expression, I think we should be able to merge the PR


OpenIdConnect modifications for K.

Formatting issues put aside (please kill those tabs with fire! :fire:), this PR introduces questionable changes: what's the point of deeply changing the OAuth2 client middleware to use OIDC-specific concepts if you also provide an OpenID Connect client middleware, that doesn't even inherit from the OAuth2 one?

My 2 cents: the different mentions of OpenID Connect types/concepts/logic in the OAuth2 bearer middleware make these changes very confusing. Maybe it'd better to use a neutral name like BearerAuthenticationMiddleware or TokenAuthenticationMiddleware and extract OAuth2/OIDC specific stuff to different projects?

General Bearer token question summarized from earlier discussion:
The prior bearer token middleware was quite simple, just looking at the header and then applying the any interpreter you provided. JWT was plugged into this and AAD and ADFS were layered on top of JWT.
With the new middleware all of these layers have been merged to make things like AAD & ADFS metadata easier to work with.  That's cool, but it's not clear if the the original simple scenario can still be implemented. Can you please provide a sample showing how to use the new middleware without metadata, JWT, etc., but only a custom token reader?
This merger also pulls in several dependencies that I think are inappropriate for the other middleware in this package, the (interactive) generic OAuth middleware used by all of the social auth. I think we'll need to break up the packaging to avoid crossing dependencies. This will require either moving OAuthBearer to its own package, or re-establishing some of the old layering so the added dependencies are pulled in via a higher level package.

Lets move all comments to #112


Fix race condition in config handling

Description of the Change
The CLI encounters race conditions when handling config files if multiple commands are executed in parallel.
Currently, the CLI writes a new config for every command executed due to the code in command_parser.go. However, it also deletes all temporary config files when reading the config, leading to race conditions between multiple processes. This PR introduces two changes to mitigate this problem:

Ignore "File Not Found" errors when attempting to delete a file. This change addresses the issue where multiple processes list and delete temporary files simultaneously.
Only delete temp config files older than 5 minutes. This change resolves the problem where one process creates a temporary config that it later wants to rename, while a second process deletes this temp config in the meantime. Orphan files should be rare, as they are normally pruned by the signal handler, as seen in https://github.com/cloudfoundry/cli/blob/04df8ae6dd68b64c2080c863c1ddf4c3e06abcd6/util/configv3/write_config.go#L63-L66

Why Is This PR Valuable?
Enables usage of the CF CLI in scripts which execute commands in parallel
Applicable Issues
fixes #2232
How Urgent Is The Change?
Medium
Other Relevant Parties
Anyone using the CLI in scripts with parallel execution. Multiple users have encountered this issue, as discussed in #2232.
Related PRs
Will be added later after first feedback (main branch)

Hi @f-blass,
Thank you for your contribution. Looks like some tests need to be addressed as part of this fix. Can you please look into it.

Hi @gururajsh
thank you! Should be fixed now.

Hi @gururajsh
thank you. PRs for v7 and main branch are created and linked.


feat: add support for validating glob patterns

Fixes: https://github.com/tj-actions/changed-files/issues/565


landing page container content not lined up

- the line should full width
- the shapes should also start at the edge of the screen

👋


Update migrate-v1-v2.md

To avoid confusion for users of the document

@deeksha-microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@vhorne - As the listed author, could you review the pull request? Thanks.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix outdated Nerd Font icons

No description provided.

Fixed in 3213d3b, 6c1a745, e235458, 247f7d7, 027f331.


[bug]: shadcn/ui web page breaks when using combobox

Describe the bug
This is an issue that I've found on the web page of the library, I was seeing the "combobox" component and using the different examples. When I reached the "Form" section it prompted a client-side error.
This isn't a problem that affects the use of the framework, but I guess it usefull to have the web page working correctly
Affected component/components
Combobox
How to reproduce

Go to the web page
Access the combobox component
Go to the "Form" example
Click on the component

Codesandbox/StackBlitz link
No response
Logs
97525-01e51dd4c392194a.js:1 TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))
    at Function.from (<anonymous>)
    at O (69016-e46275a046c77977.js:1:8485)
    at M (69016-e46275a046c77977.js:1:7484)
    at 69016-e46275a046c77977.js:1:6256
    at 69016-e46275a046c77977.js:1:14852
    at Map.forEach (<anonymous>)
    at 69016-e46275a046c77977.js:1:14841
    at aI (eb2a38ac-8c1a930f3b668d8a.js:1:72883)
    at aU (eb2a38ac-8c1a930f3b668d8a.js:1:72943)
    at aV (eb2a38ac-8c1a930f3b668d8a.js:1:73382)
window.console.error @ 97525-01e51dd4c392194a.js:1

System Info
Brave browser
Version 1.67.123 Chromium: 126.0.6478.126 (Build oficial) (64 bits)

Also tried with

Firefox
127.0.2 (64-bit)

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

You should wrap the items in command component with CommandList component.

https://github.com/shadcn-ui/ui/pull/4178

#4178

I guess this means that it is known, my intention wasn't to use the component (at least for the moment) but I think it was worth to mention because I don't know if it is actually a problem with the component or just a bug on the web page


publish<EMAIL_ADDRESS>

Requested by: @lforst
Merge target: 0.6.x
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[ ] github
[ ] npm

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

Please do not accept until coordinated with me. This is a backport of a fix.


Supported input file formats?

What are the supported input file formats? PNG only?
No BMP?

Just .PNG - for now. We need to get approval to use any 3rd party code, and as this can be a long/complex process we're sticking with .PNG.


Fix on-heap aux stack allocation

Because stack grows from high address towards low address, the value returned by malloc is the end of the stack, not top of the stack. The top of the stack is the end of the allocated space (i.e. address returned by malloc + cluster size).
#1790

We need to remove these two lines https://github.com/bytecodealliance/wasm-micro-runtime/blob/dev/wasi_threads/samples/wasi-threads/wasm-apps/CMakeLists.txt#L32. I had to put them in this PR since there was a heap corruption problem that was making the pipeline fail.


include links to haproxy base container

It would be helpful to include links to the haproxy base containers.
balenaOS puts the root haproxy configuration files in different locations than some may expect.

That's sensible to have more documentation, @drjasonharrison-vp-eio
In some ways that part is also showing different ways of creating projects (per-architecture / per-device-type Dockerfile). Maybe that's less important, and should just build Haproxy from source? (though the time it takes is a downside for a getting started project, most likely).
Where would people expect the haproxy configuration to be put? Maybe we'll have to adjust the demo project to be more conforming to the best practices of haproxy projects, and any info is appreciated.


Add the membership.getAllByUser() methods to the application & organization models

Change-type: minor

I self-certify!


example error

I'm trying to run the example code on MNE official page, which is a simple code.
But it throws a error, I don't know how to solve it.
This is error code:
ValueError: The number of FixedLocator locations (20), usually from a call to set_ticks, does not match the number of ticklabels (1).
import mne
import numpy as np
import matplotlib.pyplot as plt
data = np.random.randn(5, 1000)
info = mne.create_info(
ch_names=['MEG1', 'MEG2', 'EEG1', 'EEG2', 'EOG'],
ch_types=['grad', 'grad', 'eeg', 'eeg', 'eog'],
sfreq=100
)
custom_raw = mne.io.RawArray(data, info)
scalings = {'eeg': 2, 'grad': 2, 'eog':2}
plt.yticks(['MEG1', 'MEG2', 'EEG1', 'EEG2', 'EOG'],
labels=5)
custom_raw.plot(n_channels=5, scalings=scalings, title='Data from arrays', block=True)

This has already been reported in #8031 - you can use the development version until we make a new release.

This has already been reported in #8031 - you can use the development version until we make a new release.

Thank you for your reply!

@cbrnr
I can't install the dev version of mne.
Could you tell me how to install it?
Here is my install error:
C:\Users\USER>pip install mne==0.21.dev0
ERROR: Could not find a version that satisfies the requirement mne==0.21.dev0 (from versions: 0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.7.1, 0.8, 0.8.1, 0.8.5, 0.8.6, 0.9.0, 0.10.0, 0.11.0, 0.12.0, 0.13, 0.13.1, 0.13.2, 0.14, 0.14.1, 0.15, 0.15.1, 0.15.2, 0.16, 0.16.1, 0.16.2, 0.17.0, 0.17.1, 0.17.2, 0.18.0, 0.18.1, 0.18.2, 0.19.0, 0.19.1, 0.19.2, 0.20.0, 0.20.1, 0.20.2, 0.20.3, 0.20.4, 0.20.5, 0.20.6, 0.20.7)
ERROR: No matching distribution found for mne==0.21.dev0

pip install -U https://github.com/mne-tools/mne-python/archive/master.zip
On Thu, Aug 20, 2020, 9:16 PM 5dh15<EMAIL_ADDRESS>wrote:

@cbrnr https://github.com/cbrnr
I can't install the dev version of mne.
Could you tell me how to install it?
Here is my install error:
C:\Users\USER>pip install mne==0.21.dev0
ERROR: Could not find a version that satisfies the requirement
mne==0.21.dev0 (from versions: 0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.7.1,
0.8, 0.8.1, 0.8.5, 0.8.6, 0.9.0, 0.10.0, 0.11.0, 0.12.0, 0.13, 0.13.1,
0.13.2, 0.14, 0.14.1, 0.15, 0.15.1, 0.15.2, 0.16, 0.16.1, 0.16.2, 0.17.0,
0.17.1, 0.17.2, 0.18.0, 0.18.1, 0.18.2, 0.19.0, 0.19.1, 0.19.2, 0.20.0,
0.20.1, 0.20.2, 0.20.3, 0.20.4, 0.20.5, 0.20.6, 0.20.7)
ERROR: No matching distribution found for mne==0.21.dev0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/8133#issuecomment-677984273,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKTXHNNG2RPOG7HEP7YZODSBXDFJANCNFSM4QGDZT3Q
.

Thank you! Helped me a lot! @bloyl


Fix #1712 - remove reflection from validation code in MutableObjectModel...

...Binder
This change moves [BindingBehavior(...)] and friends into the model
metadata layer, and removes the reflection code from
MutableObjectModelBinder that was looking for them previously.

:watch: and haven't looked in detail at the tests yet

:shipit: after fixing minor issue in test.

@dougbu @harshgMSFT updated

:watch: mostly for test gaps and potential ModelAttributes extension

@dougbu updated - added tests

:shipit:


Sample POCOs

The third item on the Roadmap is to make a separate library of annotated POCOs. I was wondering if anyone has already created any and has submitted them in anyway. I need an X12 835 & 837. As much as I would love to pore over the documentation, it would be neat if that was already done.
So what I am asking is if anyone has already created POCOs for 835/837 and would be interested in sharing. As I'd be interested before I head down the road of writing it myself.
Thanks!

@nycbauer The sample POCOs are here: https://github.com/indice-co/EDI.Net/tree/master/test/indice.Edi.Tests/Models Its not really a library, but it is a collection of models that you can use.

Unfortunately the repository does not appear to contain 835 or 837 as requested by OP. @nycbauer did you have any luck with this?

I am looking for a 277 implementation (goes as a response to the 837 files). I think some more samples would help. I am going to try get this working as it would be great to have this in house ability to parse files.


Command "npm run seed" fails

Nodejs version - v21.1.0
npm version - 10.2.0
OS - Ubuntu
I have followed the process and even created the database but when I run the "npm run seed" command. I get this error:
> seed
> node -r dotenv/config ./scripts/seed.js

node:events:493
      throw er; // Unhandled 'error' event
      ^

AggregateError
    at internalConnectMultiple (node:net:1114:18)
    at internalConnectMultiple (node:net:1177:5)
    at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
    at listOnTimeout (node:internal/timers:575:11)
    at process.processTimers (node:internal/timers:514:7)
Emitted 'error' event on WebSocket instance at:
    at emitErrorAndClose (/home/syprose/projects/learn/nextjs/nextjs-dashboard/node_modules/ws/lib/websocket.js:1016:13)
    at ClientRequest.<anonymous> (/home/syprose/projects/learn/nextjs/nextjs-dashboard/node_modules/ws/lib/websocket.js:864:5)
    at ClientRequest.emit (node:events:515:28)
    at TLSSocket.socketErrorListener (node:_http_client:495:9)
    at TLSSocket.emit (node:events:515:28)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  code: 'ETIMEDOUT',
  [errors]: [
    Error: connect ETIMEDOUT <IP_ADDRESS>:443
        at createConnectionError (node:net:1634:14)
        at Timeout.internalConnectMultipleTimeout (node:net:1685:38)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -110,
      code: 'ETIMEDOUT',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 443
    },
    Error: connect ENETUNREACH 64:ff9b::1717:e509:443 - Local (:::0)
        at internalConnectMultiple (node:net:1176:40)
        at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -101,
      code: 'ENETUNREACH',
      syscall: 'connect',
      address: '64:ff9b::1717:e509',
      port: 443
    },
    Error: connect ETIMEDOUT <IP_ADDRESS>:443
        at createConnectionError (node:net:1634:14)
        at Timeout.internalConnectMultipleTimeout (node:net:1685:38)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -110,
      code: 'ETIMEDOUT',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 443
    },
    Error: connect ENETUNREACH 64:ff9b::2ccf:9495:443 - Local (:::0)
        at internalConnectMultiple (node:net:1176:40)
        at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -101,
      code: 'ENETUNREACH',
      syscall: 'connect',
      address: '64:ff9b::2ccf:9495',
      port: 443
    },
    Error: connect ETIMEDOUT <IP_ADDRESS>:443
        at createConnectionError (node:net:1634:14)
        at Timeout.internalConnectMultipleTimeout (node:net:1685:38)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -110,
      code: 'ETIMEDOUT',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 443
    },
    Error: connect ENETUNREACH 64:ff9b::3414:6b83:443 - Local (:::0)
        at internalConnectMultiple (node:net:1176:40)
        at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -101,
      code: 'ENETUNREACH',
      syscall: 'connect',
      address: '64:ff9b::3414:6b83',
      port: 443
    },
    Error: connect ETIMEDOUT <IP_ADDRESS>:443
        at createConnectionError (node:net:1634:14)
        at Timeout.internalConnectMultipleTimeout (node:net:1685:38)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -110,
      code: 'ETIMEDOUT',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 443
    },
    Error: connect ENETUNREACH 64:ff9b::2cc7:3bac:443 - Local (:::0)
        at internalConnectMultiple (node:net:1176:40)
        at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -101,
      code: 'ENETUNREACH',
      syscall: 'connect',
      address: '64:ff9b::2cc7:3bac',
      port: 443
    },
    Error: connect ETIMEDOUT <IP_ADDRESS>:443
        at createConnectionError (node:net:1634:14)
        at Timeout.internalConnectMultipleTimeout (node:net:1685:38)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -110,
      code: 'ETIMEDOUT',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 443
    },
    Error: connect ENETUNREACH 64:ff9b::22c2:641c:443 - Local (:::0)
        at internalConnectMultiple (node:net:1176:40)
        at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -101,
      code: 'ENETUNREACH',
      syscall: 'connect',
      address: '64:ff9b::22c2:641c',
      port: 443
    },
    Error: connect ETIMEDOUT <IP_ADDRESS>:443
        at createConnectionError (node:net:1634:14)
        at Timeout.internalConnectMultipleTimeout (node:net:1685:38)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -110,
      code: 'ETIMEDOUT',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 443
    },
    Error: connect ENETUNREACH 64:ff9b::641a:7485:443 - Local (:::0)
        at internalConnectMultiple (node:net:1176:40)
        at Timeout.internalConnectMultipleTimeout (node:net:1687:3)
        at listOnTimeout (node:internal/timers:575:11)
        at process.processTimers (node:internal/timers:514:7) {
      errno: -101,
      code: 'ENETUNREACH',
      syscall: 'connect',
      address: '64:ff9b::641a:7485',
      port: 443
    }
  ]
}

Node.js v21.1.0
`

I have a same problem, when I try to run this command I get the next message

Faced with same issue as @shamov102  on Windows 11, Node 20.9.0LTS, NPM- 10.1.0

I had similar error, but not this one. It was a web socket error. What I have done is, go to vercel dashboard->storage->settings->Reset Credentials and reset your credentials. Then copy the new credentials to your .env file. Then it gets connected.

@A-Ostrovnyy I had same error. Check in your .env if you have all values filled I had:
POSTGRES_USER=""
POSTGRES_PASSWORD=""
after refresh on Vercel site  it suddenly gave me those values, then it worked

I think this step is a little early in the process, but have you deployed your app to Vercel yet?
It could be that you're having a firewall issue. If it also fails when deployed to Vercel, it likely means something else is the matter.

Thank you for the feedback! I have been seeing an issue where not all of the credentials are being copied over correctly from the clipboard in the Vercel dashboard. Please make sure that all the environment variable have values in your .env file. You should have the following:
POSTGRES_URL=
POSTGRES_PRISMA_URL=
POSTGRES_URL_NON_POOLING=
POSTGRES_USER=
POSTGRES_HOST=
POSTGRES_PASSWORD=
POSTGRES_DATABASE=

This should fix the issue :) I have added a note to the course for troubleshooting this


Create pull_request_template.md

Pull request template

LGTM. @daniekpo, what do you think?

LGTM. @daniekpo, what do you think?

Looks good to me.


[WIP] #3312 | Large Pulse backend mock

Summary
Create a large (100Q) Pulse backend mock and/or create a util method or a script to generate reasonable Pulse-enabled backend
Issue https://github.com/Qiskit/qiskit-terra/issues/3312
Details and comments
The pulse module of Qiskit has yet to be benchmarked. The purpose of benchmarking is to provide a deterministic way to assess performance of software over time. There is room for improvement in the performance of pulse operations; building benchmarks will allow us to quantify progress and justify code refactors.

Team, I need some input from you 😄
I'm trying to write tests for generated by builder backends (create circuit and run it on the backend.)
It goes as usual:
desired_vector = [1 / math.sqrt(2), 0, 0, 1 / math.sqrt(2)]
qr = QuantumRegister(2, "qr")
qc = QuantumCircuit(qr)
qc.initialize(desired_vector, [qr[0], qr[1]])

then I build a backend and try to transpile cirquit, create schedule and assemble qobj:
FakeOpenPulse10Q = FakeBackendBuilder("Tashkent", n_qubits=10).build()
backend = FakeOpenPulse10Q()

experiments = transpile(qc, backend=backend)
experiments = schedule(circuits=experiments, backend=backend)
qobj = assemble(experiments, backend=backend)

and everything works but to finish logical sequence for execute I need to run backend.run(qobj).
And here execution fails due to pickle error:
_pickle.PicklingError: Can't pickle <class 'abc.FakeOpenPulse10Q'>: attribute lookup FakeOpenPulse10Q on abc failed

Same thing happens when I try to use FakeOpenPulse2Q (existing) backend. But in case of existing backend it failed due to inaability to pickle local object:
AttributeError: Can't pickle local object 'QobjToInstructionConverter.convert_frame_change.<locals>.gen_fc_sched'

This can be solved (I assume) by using __setstate__ and __getstate__ to manually pickle-unpickle objects.
Anyhow both of this errors occurs in run function, where in FakeBackend we are using FakeJob which uses futures.ProcessPoolExecutor(). And after 15 seconds of googling I noticed that dynamic classes has issue with pickling (which is happening while using process pools).
I think, I can create other FakeJob which will not use pools and use it in generated fake backends run method. 🤔 Maybe it's 2am and I need to sleep a little to came up with better solution 😄
Any suggestions how to overcome this?)
P.S
Is is ok that I always mention your names in PR discussions @taalexander @lcapelluto ? It probably creates a lot of notifications in your accounts. Sorry for that. I can just leave a comments and wait 😄

It's totally ok to ping me 😄. I was tried to run the sample code
import math

from qiskit import QuantumRegister, QuantumCircuit, transpile, schedule, assemble
from qiskit.test.mock.utils.fake_backend_builder import FakeBackendBuilder

desired_vector = [1 / math.sqrt(2), 0, 0, 1 / math.sqrt(2)]
qr = QuantumRegister(2, "qr")
qc = QuantumCircuit(qr)
qc.initialize(desired_vector, [qr[0], qr[1]])

FakeOpenPulse10Q = FakeBackendBuilder("Tashkent", n_qubits=10).build()
backend = FakeOpenPulse10Q()
experiments = transpile(qc, backend=backend)
experiments = schedule(circuits=experiments, backend=backend)
qobj = assemble(experiments, backend=backend)

but am getting the error
TranspilerError: "The 'layout' must be full (with ancilla)."
If I look at the coupling_map, it includes too many qubits.
>>> backend.configuration().coupling_map
[[0, 4], [1, 2], [2, 3], [2, 6], [3, 4], [4, 8], [5, 6], [6, 7], [6, 10], [7, 8], [8, 12], [9, 10]]

The backend is missing a parameter rep_times I think.

I can, however, recreate your error with
import math

from qiskit import QuantumRegister, QuantumCircuit, transpile, schedule, assemble
from qiskit.test.mock.utils.fake_backend_builder import FakeBackendBuilder
from qiskit.test.mock import FakeOpenPulse2Q

desired_vector = [1 / math.sqrt(2), 0, 0, 1 / math.sqrt(2)]
qr = QuantumRegister(2, "qr")
qc = QuantumCircuit(qr)
qc.initialize(desired_vector, [qr[0], qr[1]])

FakeOpenPulse10Q = FakeBackendBuilder("Tashkent", n_qubits=4, coupling_map=[[0, 1], [1,2], [2,3]]).build()
#backend = FakeOpenPulse2Q()
backend = FakeOpenPulse10Q()
experiments = transpile(qc, backend=backend)
experiments = schedule(circuits=experiments, backend=backend)
qobj = assemble(experiments, backend=backend, rep_time=100)

job = backend.run(qobj)
result = job.result()

It seems like this is trying to pickle the backend object in multiprocessing which may not be possible. I'm not sure why this is necessary perhaps @mtreinish has an idea? Maybe the problem is that run_job for the future is a class method?

I think this method is broken and untested.

Did merging in master help?

Fake job changed. So, no it's using Aer simulator and reads hamiltonian. Now I need to generate Hamiltonian correctly and continue with tests. But all of your comments above correct and it helped :)
I'll get back tonight if I'll not be able to run it today.

@taalexander I'm trying to run
        self.backend = FakeBackendBuilder("Tashkent", n_qubits=4).build()
        ...
        desired_vector = [1 / math.sqrt(2), 0, 0, 1 / math.sqrt(2)]
        qr = QuantumRegister(2, "qr")
        cr = ClassicalRegister(2, 'cr')
        qc = QuantumCircuit(qr, cr)
        qc.initialize(desired_vector, [qr[0], qr[1]])
        qc.measure(qr[0], cr[0])
        qc.measure(qr[1], cr[1])

        experiments = transpile(qc, backend=self.backend)
        experiments = schedule(circuits=experiments, backend=self.backend)
        qobj = assemble(experiments, backend=self.backend)

        job = self.backend.run(qobj)

which is code from here -> https://github.com/Qiskit/qiskit-terra/pull/3797/commits/7fe4bd9ca91d27c256572ec6b655dcbb38549a44#diff-5b5c1ce1e7ad1fef068081a327804e4aR31
and I'm getting
qiskit-terra/lib/python3.7/site-packages/qiskit/providers/aer/pulse/qobj/digest.py", line 323, in build_pulse_arrays
    oplist_to_array(pulse['samples'], pulses, pulses_idx[ind - 1])
TypeError: Argument 'A' has incorrect type (expected list, got numpy.ndarray)

which is so funny because that is the only change which is still not included into Aer release 😆
here is the change which will fix error above https://github.com/Qiskit/qiskit-aer/compare/0.4.1...stable/0.4#diff-efd70aeb09d7cc715321c96acb7a7af0R324
Should I wait till release of Aer?

I just talked to Matthew and release of Aer will be within 2 weeks. Meanwhile I'll clean the code up and finish all remaining todos.

Ok, I've tried to write save and load methods, but faced a problem with to_dict methods. Apparently, to_dict is not converting all fields of BackendProperties correctly therefore json.dump raises error TypeError: keys must be str, int, float, bool or None, not tuple. I notices there were recent changes in this methods, maybe it's bug somewhere, because I remember it was working before I updated master.
I removed those methods for now and probably we need to create an issue for BackendProperties to_dict method.
I'll leave it here for now :)
    def save(self, folder: str):
        """Saves backend configuration files to specified folder."""
        with open('{0}/props.json'.format(folder), 'w') as file:
            json.dump(self.build_props().to_dict(), file, indent=4, sort_keys=True)

        with open('{0}/conf.json'.format(folder), 'w') as file:
            json.dump(self.build_conf().to_dict(), file, indent=4, sort_keys=True)

        with open('{0}/defs.json'.format(folder), 'w') as file:
            json.dump(self.build_defaults().to_dict(), file,
                      indent=4, sort_keys=True,
                      default=lambda o: '')

    @classmethod
    def load(cls, folder: str):
        """Loads saved backend configuration from specified folder."""
        with open('{0}/props.json'.format(folder), 'r') as file:
            props = BackendProperties.from_dict(json.load(file))

        with open('{0}/conf.json'.format(folder), 'r') as file:
            conf = PulseBackendConfiguration.from_dict(json.load(file))

        with open('{0}/defs.json'.format(folder), 'r') as file:
            defs = PulseDefaults.from_dict(json.load(file))

        backend = FakeBackend(conf)
        backend.defaults = defs
        backend.properties = props
        return backend

Ok, that makes sense. Thanks for the update 😄

Shoot! I was not quick enough removing WIP and Luciano marked this PR as draft 😄
@taalexander @lcapelluto  Anything else I need to consider here?

@mtreinish  and @lcapelluto have your concerns been addressed by the very patient @IceKhan13  😄?

I actually have one small request. Other backends have this format: backend = Something(). I think introducing FakeBackendBuilder().build() doesn't really gain us anything, since we can't change parameters after initializing and running build, right? What do you think @IceKhan13 , @taalexander ?

Are you suggesting that the builder be made just the backend class that is to be initialized?

@lcapelluto @taalexander
I guess if we want to remove build, we should rename class to something like ConfigurableBackend and inherit from FakeBackend class like 2q and 3q examples we have.

ping :)

@IceKhan13 I like the ConfigurableBackend idea. I think it's more inline with what this accomplishes: as it is, everything is provided at init, and the result object can't be used until build is called to get a backend.

@IceKhan13 Can the name be changed to ConfigurableFakeBackend? We may have real configurable backends in the future.

Hello, we had a similar project to this one titled: open pulse mock backend for spin based qubits from qiskit camp Africa 2019, the mock qubits where associated with the 13-C nuclear spins coupled to NV centres in a diamond. Based on the computational ability of carbon atoms, it was my believe that these carbon atoms could be used as fake backends that will participate as reactive groups in drug design to combat the covid19 pandemic. If we treat the electron spin as a quantum bus, we can adjust the states that correspond to the syndrome measurements and thus the 13-C  nuclear spins provide a platform to design the proper chemistry that will help mitigate the syndrome. There is a lot of research on the development of viral nanoparticles as a template for organising materials at the nano scales and I think that it is time to consolidate these efforts.

I think it is fine to still have an example backend for an NV center system. Would it be possible to use @IceKhan13's work for this and choose the right parameters? Maybe some slight modifications would be required?

Yes, with 100 qubits we can realise a 100*100 tabular qubit ordering, how do I become part of this discussion so that I may be able to present my attempts?

If you open up a draft PR with your attempts we can comment directly on it 😄


Print multiple errors

In many cases will be great when assertion prints all found errors in json. For that option library need to use tv4.validateMultiple method.
Is there any workaround to do this?

This has been resolved in version 1.4.0, which was just published on NPM. See (README)[https://github.com/chaijs/chai-json-schema/#non-standard-tv4-properties] for instructions how to implement.

Thanks for your help @Niklv !


[WTD] PowerBI Sample Inclusion Articles

In support of #1045 , here is a list of articles with line numbers of where non gender neutral language is present.

[ ] sample-customer-profitability.md Line   #19
[ ] sample-customer-profitability.md Line   #61
[ ] sample-customer-profitability.md Line   #76
[ ] sample-customer-profitability.md Line   #82
[ ] sample-customer-profitability.md Line   #97
[ ] sample-customer-profitability.md Line   #125
[ ] sample-customer-profitability.md Line   #126
[ ] sample-customer-profitability.md Line   #129
[ ] sample-customer-profitability.md Line   #133
[ ] sample-customer-profitability.md Line   #134
[ ] sample-customer-profitability.md Line   #135
[ ] sample-customer-profitability.md Line   #136
[ ] sample-customer-profitability.md Line   #141
[ ] sample-customer-profitability.md Line   #149
[ ] sample-customer-profitability.md Line   #150
[ ] sample-customer-profitability.md Line   #151
[ ] sample-customer-profitability.md Line   #152
[ ] sample-opportunity-analysis.md Line   #23
[ ] sample-opportunity-analysis.md Line   #67
[ ] sample-opportunity-analysis.md Line   #112

Fixed in https://github.com/MicrosoftDocs/powerbi-docs-pr/pull/2311


GEODE-10053: Use stored DistributionConfig to create SSL configurations

While creating SSLConfig no new DistributionConfigImpl is created.
Now, the DistributionConfig is taken from the InternalLocator class.

For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

they were typos and they have been fixed


Output cache: Fixing a wrong timestamp. Credits to khoggatt (CodeIgniter forum).

The timestamp of the created cache file should be taken, instead of the cache directory.
Reported at CodeIgniter forum: http://forum.codeigniter.com/thread-61534.html

A changelog entry is required for this one.

Done.


update revision of azure-webapp-maven-plugin

@andxu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Sorry, please ignore this PR since it is in wrong place


Add some non-autoclosure placeholder overload

This should allow to "wrongly" define the placeholder of an unimplemented () -> A as () -> A. More explicitly:
struct S {
  var a: () -> String
}
S(a: unimplemented(placeholder: "X")) // and
S(a: unimplemented(placeholder: { "X" } ))

would behave the same. Right now the second line triggers a failure when we try to legitimately assign its value in a withDependencies block.
Hopefully this overloads doesn't create ambiguities. It also seems that this is the only one that needs to be implemented (AFAIK, @autoclosure excludes async and throws).
See https://pointfreecommunity.slack.com/archives/C04L2D0MNJH/p1676449096114709

@tgrapperon Now that we have @DependencyEndpoint and @DependencyClient, I think we can consider unimplemented to be deprecated. Thanks for the PR, though I think we can all be happy that we can eliminate many of these overloads some day soon :smile:


Doesn't show the input area for some plugins

Hi,
I'm using Emmet, AdvancedNewFile, etc, which open their own inputs near the status bar, like to create a new file via ANF, pressing Win + Alt + N or to wrap a peice of HTML in another tag via EMMET, pressing Ctrl + Shift + G.
These open up an input area down below near the status bar (just like Sublime's built-in Ctrl+I or Ctrl+F do for searches)
This theme doesn't show anything. just a fat block of that spacegray-blue chrome colour. There is no indication that an input is opened, and when any text is typed, nothing shows up.
See these screenshots for reference:
Material Spacegray

Material Spacegray with input (bad)

Compare that with Sublime's default theme:
Default theme with input (good):

Thanks Kumar, I'll look into it.

Hi Kumar,
Did you restart Sublime when you switched themes? I am not getting the same issue that you are.
This is what happens when I use AdvancedNewFile with the theme:

@saadq yes, i've restarted sublime multiple times... nothing of note seems to be there in the console either.

did you see this : https://cloud.githubusercontent.com/assets/474329/8178894/a0dd09c0-1412-11e5-8ecf-f7f9ade439ae.gif

@gj1118 thanks for that... Earlier from the gif it's not apparent what's that going to do... the user is just resizing the pane...
I realize now that perhaps an Autosize is not being triggered by the theme in Sublime (it's a wontfix from the material-theme's creator See #67)
So, we have to manually resize the bottom sidebar input area to trigger the recalculation, which shows the input.

@kumarharsh, did resizing the input area fix your issue?

yes it did actually.


Display closing price instead liquidation price once known

The liquidation price becomes irrelevant once we have a closing price (i.e. once we are closing the cfd), so we display the closing price then.
Since we did not previously expose the closing price in the projection we do now.

bors r+


Semantics of use item is not preserved

The following is a valid module in Rust 2018, and rustc 1.33.0 happily accepts it:
use ::std::marker::PhantomData;

mod std {}

struct S(PhantomData<i32>);

If I run rustfmt on this module, however, it turns the import statement into use std::marker::PhantomData;, dropping the initial ::. This creates ambiguity and causes a compile error.
In general this problem can arise whenever there is a local module whose name conflicts with a crate name.

You need to tell rustfmt that you are using edition 2018.


Support Trailing Commas

Fixes #7279
When TS compile function parameter list, checkForTrailingComma do't be used In parser.ts. So, I change check.ts and related tests.

Just as a note, we need to check whether the services layer, and the compiler itself, still resolves overloads correctly.

This should be addressed by #8942.


passes.hpp file extension

All other header files in the include directory use .h while include/coreir-pass/passes.hpp uses a .hpp extension. I propose to switch this file to passes.h for consistency.

Because there are a mix of c and c++ files in this repo now, maybe it would be better to name any C headers with .h and any C++ headers with .hpp?

I think in general that should be the case, but for anything in include/* we should name it as .h so that we can include them in the c api.
In the Readme, we have a well defined way of adding libraries so that we can use it in both the C api and the c++ api.
Passes should probably do something similar.

This should be fixed by pull request #67


Create functionality for subcommands and bash completion for those

docker run y
docker build x
git rev-parse
git merge
git checkout
etc.

subcommands can be handled by the user, but still, how to get bash completion for them? git has bash completion for git merge/git checkout/git commit, etc.

https://github.com/git/git/blob/master/contrib/completion/git-completion.bash


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 995 ms

Resolved: Bofrost.fr performance has improved in ad7d560 after 9 minutes.


panic: runtime error: slice bounds out of range

$:~/Downloads/Sia-v1.0.0-darwin-amd64 $ ./siad
Loading...
(1/7) Loading gateway...
(2/7) Loading consensus...
(3/7) Loading transaction pool...
(4/7) Loading wallet...
(5/7) Loading miner...
(6/7) Loading host...
(7/7) Loading renter...
Finished loading in 0.013999521000000001 seconds
panic: runtime error: slice bounds out of range

goroutine 19 [running]:
panic(0x6cb560, 0xc82000a0d0)
	/usr/local/go/src/runtime/panic.go:464 +0x3e6
github.com/NebulousLabs/Sia/modules/gateway.myExternalIP(0x0, 0x0, 0x0, 0x0)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/gateway/upnp.go:38 +0x489
github.com/NebulousLabs/Sia/modules/gateway.(*Gateway).threadedLearnHostname(0xc8201881a0)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/gateway/upnp.go:62 +0x113
created by github.com/NebulousLabs/Sia/modules/gateway.New
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/gateway/gateway.go:157 +0xa35

My guess is that the call to http://myexternalip.com/raw returned an empty resp string. Then buf[:n-1] in the return statement at the end of myExternalIP would have a negative index.

Thanks for reporting this! I've made a PR that fixes it. Does this happen every time you start siad or was this a one off error?

@VoidingWarranties It happened every time

@hosiawak should be fixed now. If you recompile the master branch and try running siad again can you post the gateway's log? <sia-dir>/gateway/gateway.log
It's possible that myexternalip.com is returning an empty body. This isn't an issue for the gateway but would be problematic for the host module.

$ cat bin/gateway/gateway.log
2016/07/01 05:51:16.772478 gateway.go:105: STARTUP: Logging has started.
2016/07/01 05:51:16.773093 gateway.go:152: INFO: gateway created, started logging
2016/07/01 05:51:20.984222 upnp.go:100: WARN: could not automatically forward port 9981: no UPnP-enabled devices found
2016/07/01 05:51:21.881555 upnp.go:68: WARN: failed to discover external IP: myexternalip.com returned a 0 length IP address

$ curl -v http://myexternalip.com/
*   Trying <IP_ADDRESS>...
* Connected to myexternalip.com (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: myexternalip.com
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Fri, 01 Jul 2016 05:55:04 GMT
< Server: Apache
< X-Frame-Options: SAMEORIGIN
< Last-Modified: Tue, 26 Jan 2016 13:18:14 GMT
< Accept-Ranges: bytes
< Content-Length: 0
< Connection: close
< Content-Type: text/html; charset=UTF-8
<
* Closing connection 0

Something isn't right. Does using https in the curl request change anything? Otherwise we could add another fallback like https://api.ipify.org


Unexpected area results from polygons with holes

QGIS and geoprocessing return very different areas for the intersection between rocky_shores and mpa_test
Archive.zip

Resolved by regenerating package-lock.json and node_modules directory in the geoprocessing project it was occurring.  Can only guess there was some issue with a dependency or an older version the lockfile was retaining.


Recommendations section for Disk Cache Settings incorrect

On this page, there is a section headed "Disk Cache Settings".  Right below, there is a subsection headed "Recommendations" which reads...

To maximize the throughput, we recommend that you start with ReadOnly for host caching
whenever possible. For Premium Storage, keep in mind that you must disable the "barriers"
when you mount the file system with the ReadOnly options. Update the /etc/fstab file with
the UUID to the disks.

...which is then followed by a screen shot.
Only the first sentence of these recommendations is correct, and the second and third sentences are incorrect.  Instead, this section should read as follows...

To maximize the throughput, we recommend that you start with ReadOnly for host caching.
Oracle datafiles, tempfiles, controlfiles, BFILEs, spfiles, files for external tables,
block change tracking files, and flashback log files benefit from ReadOnly host
caching being enabled in the underlying Azure managed disk.  These database files
should be considered the working set of the database files.
However, sequential I/O to the Oracle online redo log files do not benefit from any
form of caching, and can even negate all performance benefits of host caching as well.
If there is a great deal of redo information being generated by the database, then it is
important to isolate the online redo log files away from the other Oracle database
files, to their own separate Azure managed disk with host caching disabled by setting
to None.  It is also preferable for the archived redo log files and RMAN
backupsets to be isolated together with the online redo log files, because all of
these files can be considered the recovery set of database files.

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

ID: 9d2baad7-03e3-a26c-f831-900ab3e3ae38
Version Independent ID: d5f1631a-39ca-bcd7-eade-c90b608d3e70
Content: Design and implement an Oracle database on Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/oracle/oracle-design.md
Service: virtual-machines-linux
Sub-service: workloads
GitHub Login: @Dbakevlar
Microsoft Alias: kegorman

@tigormanmsft
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@tigormanmsft
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@Dbakevlar
Can you please check and add your comments on this doc update request as applicable.

@Dbakevlar
Can you please check and add your comments on this doc update request as applicable.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Duplicate config.go files

We now have crypto/config.go and backends/config.go with the exact same contents.
Why?

For flexibility; If/when we add additional backends, for example, we may find it necessary to add attributes to the backends.config. Same goes for the encryption clients. I do not believe those distinct packages should share configuration.


Issue DAPABORT at startup and after receiving WAIT response from CMSI…

…S-DAP
The ADIv5 spec states that DAPABORT should be issued after receiving WAIT responses over an extended period. Generally a WAIT response from a CMSIS-DAP probe means that the probe has timed out after receiving multiple WAITs from the target, so it would be appropriate to issue a DAPABORT.
Do the same at startup in case the DP was left in a bad state from a previous tool run.

Yes, probe-rs was trying to communicate with a powered down AP leading to a WAIT response which needed to be cleared. (Generally it seems that we don't handle very well the case where parts of the target are powered down -- I'll try to start a discussion about this.)


Zoom resets after label visibility changes

Describe the bug
For large graphs labels are sometimes hidden and when zooming in they start to appear. That triggers redraw and zoom resets. Labels get hidden again resulting in a looping zoom behavior.
Maybe there is another reason underlying large graphs zooming in, - labels changing visibility is the one i noticed,
Steps to Reproduce the Bug or Issue

Goto https://reagraph.dev/?path=/story/demos-use-cases--mitre-all-techniques
Start zoomed out and slowly zoom in. View will reset as soon as labels start to appear

Expected behavior
Smooth zoom in / zoom out
Screenshots or Videos
No response
Platform

Reagraph Version: 4.15.26
Windows
Browser: Chrome

Your Example Website or App
No response
Additional context
No response

I encountered a similar issue when zooming in, hovering over a node to highlight it and then the zoom/pan will get reset.
A very similar thing is happening as well when zooming and panning without hovering anything. Then, the view gets centered without resetting the zoom (can be related to the labels as you mentioned).


Error on generate entities

Hello,
I installed and followed the first part about authentication.
However during the second part on the generator, I get an error while trying to use
php bin/console wandi:easy-admin-plus:generator:generate
The error:

In PropertyTypeHelper.php line 179:
Notice: Undefined index: EasyAdminType

Thanks by advance for your help !
Cheers
JB.

Hello,
Thank you for reporting this issue, it's patched 0.6.3
kasou


Add quest extension

Description
Hey 👋🏻
Quest is a simple extension that allows you to create and work on checklists. Based on The Checklist Manifesto you know checklists can save lives and make you're more productive by improving the quality of repetitive tasks. Quest lets you create, edit and delete checklists (quests). You can easily start working on them and keep track of the progress.
How it works

Call Show Quests.
Add a Quest (Checklist) by pressing CMD+N.
Give it a name and press CMD+Enter to add tasks. Finally add the quest by pressing CMD+K → Create Quest.
Now select the Quest and press Enter to start working on it. Check off tasks by pressing Enter.
Once you're done with a Quest, export a markdown report by pressing CMD+T → Export Markdown.

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Using npm run publish somehow used an old fork of mine. I'll close this PR and create a new one after deleting all existing forks.


Output for enums looks a bit wonky

Hello again,
The generated documentation for enums seems overly verbose:

Other than that, things look great!

Hello @lensbart  and thank you (again!) for using SpectaQL!
If it's not too much trouble, can you provide me with an example JSON or SDL of your Region enum so that I can reproduce this on my side?
Thanks!

Nothing out of the ordinary!
enum Region {
  BRUSSEL
  VLAANDEREN
  WALLONIE
}

Updated to look a bit nicer, and published in version 0.1.0...thanks!


Return the actual values for complexity rules

Expected Behavior
I'd like to get the actual values for the ComplexMethod, LongMethod and ComplexCondition rules in the message from Detekt.
Current Behavior
At this moment, Detekt returns messages without actual values for metrics.
For the rule ComplexMethod it returns a message without a value:
The function main appears to be too complex.
For the rule LongMethod it returns only the threshold (but not the actual value):
The function main is too long. The maximum length is 60.
For the rule ComplexCondition, it returns neither actual nor threshold values:
'This condition is too complex.'
At the same time, standard linters for other languages often returns these value.
For example, checkstyle does it for all the metrics:

Cyclomatic Complexity is 15 (max allowed is 14).
NCSS for this method is 120 (max allowed is 100).
Boolean expression complexity is 6 (max allowed is 4).

Context
We build an education platform and analyze learner's solutions for our code challenges. We parse all the output from linters and take all the values to produce some general feedback for their code. In addition, learners can use the values to simplify their code. If the actual value is displayed it's easy to see how much more a learner needs to simplify the functions. This is especially helpful for beginners in Kotlin.

Good idea! Please go ahead.
May I ask you to submit a PR with your suggestions, so that we can review and merge it later?
It better highlights the changes, when one sees the actual code.
The following lines need to be adapted (including tests):
https://github.com/arturbosch/detekt/blob/21ea09012e72e8f7cfd08ac0e275bfee5484ae5e/detekt-rules/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/ComplexCondition.kt#L80-L83
https://github.com/arturbosch/detekt/blob/21ea09012e72e8f7cfd08ac0e275bfee5484ae5e/detekt-rules/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/LongMethod.kt#L50-L53

@schalkms yes, I can. I will find some time for that tomorrow


CNF-12656: Handle SIGTERM for stateroot setup job

This PR aims to handle the case when Stateroot setup job may be in-progress and an unexpected  SIGTERM is received (e.g moving to Idle while Prep in progress).
The solution is broken into to two parts:
k8s:

The max wait time before a SIGKILL is sent is determined with TerminationGracePeriodSeconds. This is now set to 60 secs (default 30)
PreStop lifecycle handler was investigated but not useful in our case. Generally it's used to delay the container from getting SIGTERM and in the meantime k8s does other housekeeping tasks (e.g de-registering Ingress) to avoid any race conditions.

Handler in the code:

Every app is expected to have it's own signal handler. In this case if seed image is present we allow the rest of the stateroot setup to go through by sleeping at most time set with TerminationGracePeriodSeconds before shutting down.

/cc @donpenney @jc-rh

/unhold

/retest

/test ibu-e2e-flow


add check preventing partial EC xrefs on leaf terms

@pgaudet @sjm41 should we allow partial ECs on leaf terms if they are marked as broadMatch?

Yes we need them.


Set log code when closing half open connections.

Since we now disallow half open connections for HTTP1/TLS we need
to set the log code when we close these connections so that
the 'crc' code is ERR_CLIENT_ABORT rather than ERR_UNKNOWN.

@zwoop -- Would like to request back-port to 7.1.x also since the corresponding functional change in PR #4213 is also marked for back-port to 7.1.x.

@pbchou Sure. Can you make a PR against the 7.1.x branch as well please?

@zwoop PR is #4359 for back-port to 7.1.x. Thanks.


mixed precision training(fp16) support

Does following https://www.tensorflow.org/guide/mixed_precision help?

I changed some code slightly to adapt to mixed precision training，but it's easy to have OOM problem (the batch_size is same as FP32 training)..

That can happen for a lot of reasons. Maybe the variant of the model that you're using is too large to fit on the GPU.
But the base variant should work okay.

FP32 can train normally, but change to FP16 will OOM(keep anything same as FP32, just add mixed precision training). Is this normal? FP16 should use less memory.

Without taking a look at the full code, I can't comment on this. Unfortunately, I don't have the bandwidth to review code now.

I made few changes to the code， just add "tf.cast" to some layer to adapt the mixed precision training.

OK. Thank you anyway!


Refactor client display

Describe the feature request
Now we use foam tree as us data display. But i create a new project squarified is same as foam tree and the size can reduce 200kb. I'm not sure if it's a good idea.

@SukkaW CC

https://airbnb.io/visx/treemap
What about this then? But we might have to bring other packages for tooltip.

We can implement tooltip by ourselves by just getting the coordinates of the rect. By the way. squarified provide these functions.

I think squarified can replace @foam/tree now although they don't seem like same. ( The theme system will be updated later but i think the current default rainbow color is enough.


Split code

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

195 of 334 (58.38%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-10.8%) to 38.71%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/operations/mod.rs
92
101
91.09%

src/app/key_binding.rs
7
28
25.0%

src/event/key.rs
50
80
62.5%

src/app/models.rs
46
84
54.76%

src/main.rs
0
41
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main.rs
1
0.65%

Totals

Change from base Build<PHONE_NUMBER>:
-10.8%

Covered Lines:
196

Relevant Lines:
360

💛  - Coveralls


Handled error on empty reponse body

Now when receiving an empty response from Chino.io, execution will continue (still the error will be notified)

Repository migrated to https://gitlab.com/chinoio-public/sdk-chino-nodejs


rust format buffer now saves position across multiple frames

This solves issue https://github.com/rust-lang/rust-mode/issues/347 by looping over every frame and then looping over the windows in each frame.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @brotzeit (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.


Fix proxy cache key

The current proxy_cache_key is proxy_cache_key "$http_authorization$cookie_nginxauth"; - this means that the basic auth header will be stored as a key in the cache file. As the auth header is base64 encoded and can be easily decoded, this mechanism can be used to harvest passwords. This PR fixes that by using $remote_user in the proxy_cache_key thereby making the key unique and achieving the purpose of not use the password.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aliasgarjh - can you please review and update?

@aliasgarjh - reminder - can you check and comment on whether you can merge this? Letting users inadvertently harvest passwords can cause issues if an enterprise uses this - even for a PoC.


FileGetContents retriever : stream context unused

Context resource (see $context) must be passed to file_get_contents()

Closing due to age. Please re-open if necessary.


How to record a program's window instead of full screen

Capturing windows in windows has always been a complex problem that can't be handled simply.
The difficulty with capturing windows is that capturing self-rendering windows, such as games or browser pages, requires all sorts of special handling, and it's not a simple thing to do.
As far as I know, the current project is unlikely to support your feature request anytime soon. If you have a need to capture windows, I'd recommend using the core of obs (libobs) to do it, but obs is not a Rust project and the quality of the obs code isn't great, but at the moment using libobs is the best way to work with all types of windows.

The best thing you could do at the time is to make sure the window size wouldn't change and use the Window struct for the capture settings target.


Btree

Closes #2361. Benchmarks are there, but they're showing different dynamics for different tests which is very annoying, there is no clear winner, so this can't be merged right away. We can of course try separating things that need to be sorted from things that don't in the MemoryStorage itself, but that seems like a big kludge.

Obsoleted by #2364.


Enable iOS plugin

Add iOS dart plugin with static libraries compilation
Related with Destiny issue 196

Code Review Checklist (to be filled out by reviewer)

[ ] Description accurately reflects what changes are being made.
[ ] Description explains why the changes are being made (or references an issue containing one).
[ ] The PR appropriately sized.
[ ] New code has enough tests.
[ ] New code has enough documentation to answer "how do I use it?" and "what does it do?".
[ ] Existing documentation is up-to-date if impacted.

Also updated submodule to latest w-w version as there was fix for hints.


Why performance of openssl1.1.1k is lower than openssl1.0?

Hi, my service use openssl1.0 and openssl1.1.1k seperately and use same cipher(TLS_RSA_WITH_AES_256_GCM_SHA384).
The test method is receiving https request and send 200 ok immediately using libevent, why performance of the version using openssl1.1.1k is very low? Btw, the tcp connection is short.

By analyzing captured packet, i found the Ack answered time is different.
The openssl1.0 result is:

The openssl1.1.1k result is:


readme: update for new algorithm, add db table, add primer

@erezsh I've re-worked the README:

Brief summary of how it works in intro
Add table with performance and support and sample connection strings (for ease of use)
Re-work the technical explanation for the new algorithm and diagrams

Would love your feedback.

I think you get too quickly into specifics. Instead, overviews should start abstract, and when possible, get into specifics layer by layer.
Personally, it would be much easier for me to read a technical analysis of the algorithm and the main features, than a story full of examples where I have to read several pages to get the whole picture.
Also, I'm not sure about the table you put. Did we test snowflake more than Redshift? I'm not aware of it.

I think you get too quickly into specifics. Instead, overviews should start abstract, and when possible, get into specifics layer by layer.

The introduction at the top of the README is the abstract overview. The technical explanation is exactly how I've explained it to users, and it seems to resonate very well. In my experience, example-walkthroughs like this are very powerful, and most users glance over/don't read thinks that are highly abstract.
We need an outsider to judge though, as that's your insider opinion against mine :) I've asked in the channel.

Also, I'm not sure about the table you put. Did we test snowflake more than Redshift? I'm not aware of it.

Yep. One company we ran it with did Postgres <-> Snowflake for a sizable table.

Don't get me wrong, I think the walkthrough is great. I should have led with that. But I think it should come after a more brief and technical explanation, which I think is valuable to have for the more technical audience.

I think your approach leading with the abstract followed by an example and setup instructions is good. When I open up a README my first two questions are 1. what does this do? and 2. how do I use it?

Don't get me wrong, I think the walkthrough is great. I should have led with that. But I think it should come after a more brief and technical explanation, which I think is valuable to have for the more technical audience.

But wouldn't this duplicate the intro, which I think kinda does that? 👂🏻 I can write a paragraph to briefly explain it again if you prefer!

@sirupsen I was also talking about the overview. I think the original overview, while for sure it needs improvement, was much clearer and more informative.

@erezsh updated according to our session. Let me know what you think

@erezsh fixed it up based on your review, let me know if you think this is ready to go:)

Okay

@sirupsen Icon isn't showing

I guess the Emoji isn't in Windows yet
I don't have a great other suggestion...
🖇
🧳
📑


AMQ-8987 EncryptableLDAPLoginModule support wider password encryption…

… schemes
AMQ-8987
Description of changes:
Added support for wider password encryption schemes for EncryptableLDAPLoginModule. Similar to https://github.com/apache/activemq/pull/520, set IV generator for PBE-AES-based algorithms.

LGTM, I'm merging.