[Documentation] Add Table to Display Repository Statistics in README.md

Description
Is there an existing issue for this?

[x] I have searched the existing issues

Issue Description:
The issue involves adding a table to display repository statistics, such as the number of stars, forks, issues, and pull requests, directly in the README.md file. This will help contributors and users track the activity and contributions at a glance.
Steps to Implement:

Create a table in the README.md file displaying the following repository statistics:

Stars
Forks
Open Issues
Open Pull Requests
Closed Pull Requests

Use markdown syntax to include badges or icons for each statistic.
Add dynamic links to the corresponding sections of the repository for easy access (e.g., link to the "Issues" or "Pull Requests" tabs).

Expected Behavior:
The README.md file will contain a neatly formatted table that displays real-time repository statistics, making the repository's activity more visible and engaging for contributors and users.
Screenshots
Screenshot (for reference):

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@BhavaniSankar0107 it is not important please raise a meaningful issue


Integration tests with OAuth2 mock

Description
Changes proposed in this pull request:

Add OAuth2 mock in integration tests

Related issues

LGTM


Create Empty Directory

Hello,
I think create directory is missing. I tried $zipper->folder("folder"); but its not worked because there is no added file. I also tried $zipper->folder("folder")->add(array());.

I used these style but sometimes I should create directory with no file.
$zipper->folder("folder/subfolder")->addString("file.txt", null);


only search require calls in JS files

require('foo') in TypeScript files has no special meaning. Therefore we don't need to look it up and treat it like an import. It used to be in the same function as the lookup of dynamic imports to find both in one full AST walk.

removed useless parameters as the SourceFile was already in scope
split lookup function into two separate functions
only look for require calls in JS files
only look at JSDoc when searching ImportTypeNode in JS files (require will never be in JSDoc anyway)
slightly optimized the regex while keeping it readable

Followup on #28104 /cc @sandersn

This change uncovered some pretty surprising behavior: if you have require('something') in your code (it could even be a local function named require), it would resolve that module like a regular import and include its declaration file into the program.
For the same reason @types/node was implicitly included in the compilation of src/compiler. After this change I had to explicitly reference that declaration file to make it compile again.

@DanielRosenwasser and I had a long discussion about @types/node, since this incorrect behaviour has existed since April 2017 [1]. I think it's OK to take this fix as long as we note that it's a breaking change and improve the "Are you missing node" error.
Specifically,
Change "Try npm i @types/node" to
"Try npm i @types/node and then add node to the types field in your tsconfig."
Adding @DanielRosenwasser to the review for signoff on the error message change.
[1] Note that the behaviour has existed for longer in JS, but is probably a good idea there.

There's also a number of new "Cannot find module 'x'" messages in the prettier user test. Mind checking those out? jake runtests ru=user t=prettier, and then you can open the project in $ts/tests/cases/user/prettier/prettier

There's also a number of new "Cannot find module 'x'" messages in the prettier user test.

There are baseline changes, but they are the same in this branch an in master.

I think it's OK to take this fix as long as we note that it's a breaking change and improve the "Are you missing node" error.

I agree the proposed error message makes more sense, but I don't think it's strictly related to this PR. There are a few more error messages suggesting installing @types/jquery, @types/jest or @types/mocha. Should they be changed, too?
Note that this change will not only break code that implicitly relies on @types/node. It will break compilation of TypeScript files that contain require('foo') where 'foo' resolves to a declaration file containing declarations of ambient modules or global variables which are used in the program without being explicitly referenced (through import, types in tsconfig or a type reference directive).

Dont think its worth taking this at point given people might be depending on this behavior by now.


Setup failed -1

Having these issues with a Arduino Yun.
I followed the info at this post:
https://github.com/aws/aws-iot-device-sdk-arduino-yun/issues/28
I obtained the following in the serial monitor:

AWS IoT SDK Version(dev) 2.2.0-
root@arduino:/#
/bin/ash: 1: not found
/bin/ash: /root: Permission denied
/bin/ash: 1: not found
/bin/ash: /root: Permission denied
/bin/ash: 1: not found
/bin/ash: /root: Permission denied
/bin/ash: 1: not found
/bin/ash: /root: Permission denied
Linux
/bin/ash: cd: can't cde/
python: can't open file 'run.py': [Errno 2] No such file or dir
Setup failed!
-1

Any help would be appreciated as I am new to this...

Also might be helpful...
/bin/ash is a symlink and has 0777 permissions
It points to /bin/busybox
/bin/busybox has 0755 permissions

Found my issue. I had the files in the wrong folder on the Arduino.


Implement scholarships as variables

Trello card
https://trello.com/c/2WHavuTv
Context
We're centralising monetary values into one place so that we only need to update them once for them to filter down to the rest of the site. This PR does that for the PGITT scholarships.
Changes proposed in this pull request

Replace every reference to any scholarship with the corresponding variable, in the correct formatting depending on whether the values are in markdown, erb or YAML files.

Guidance to review
Here's a list of pages where a value has been changed, I recommend ticking each one off as you go:

[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/funding-and-support/scholarships-and-bursaries
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/funding-and-support/scholarships-and-bursaries (funding widget)
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/is-teaching-right-for-me/computing
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/is-teaching-right-for-me/maths
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/is-teaching-right-for-me/physics
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/is-teaching-right-for-me/engineers-teach-physics
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/is-teaching-right-for-me/qualifications-you-need-to-teach
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/landing/how-much-do-teachers-get-paid
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/landing/how-to-become-a-teacher
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/landing/how-to-fund-your-teacher-training
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/non-uk-teachers/fees-and-funding-for-non-uk-trainees
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/non-uk-teachers/get-an-international-relocation-payment
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/non-uk-teachers/train-to-teach-in-england-as-an-international-student
[ ] https://get-into-teaching-app-review-4022.test.teacherservices.cloud/steps-to-become-a-teacher

Pre-election period restrictions

[ ] This change is permitted within the constraints of the pre-election period guidelines

@gemmadallmandfe all looks good to me, the one thing I can't check is 'collage', I can't remember where I look for this!


tutorials updates

Changes in this push

updated or removed internal/deprecated links
modified semantic_parser_onboarding tutorial with more detailed datasets generation instructions
added section for testing models with agent, including note on ground truth

LGTM!


pause on blur, resume on focus

PR Description
On page blur, game will pause. On page focus, game will resume. There are no indication made to the interface (ex. no pause icon or overlay), but it could be a future feature!
Fixes #73
Describe how to test your PR locally

Open index.html locally
Start game
Switch for a tab for a few seconds.
Come back to the tab, the timer should be at the same number it was before switching tabs

@Sarahjdes  Once you resolve the merge conflicts I will approve and merge!

@Alexandria We should be good now!
Also , would you be willing to keep on maintaining this project once Hacktoberfest is over? I'd like to keep on contributing to it, I see a lof of features we could add. I don't get to develop games at work, I'd love to work on something different for fun!

@Alexandria We should be good now!
Also , would you be willing to keep on maintaining this project once Hacktoberfest is over? I'd like to keep on contributing to it, I see a lof of features we could add. I don't get to develop games at work, I'd love to work on something different for fun!

Oh absolutely! I have every intention on keeping this game up as a resource. It will remain open source indefinitely


Emails should be delivered in background

Based on this line, emails seem to be delivered during the cycle of a request.
This means that if an exception occurs, a significant delay will take place while the notification is being sent. Furthermore, if someone hits an exception multiple times there's the possibility of suffering a DDoS while all the emails are being sent.
A suggestion could be sending emails in background, using Bamboo's deliver_later method rather than deliver_now

Nice catch! I hope #26 fixes it


The HTML canvas layer has bugs thats halts the rendering of the badges at correct places

We use cytoscape-layers plugin for rendering the informational badges with each node. We were using version 2.1.0 for a very long time, but since it had some unpredictable bugs and extra amount of re-rendering involved, we had to move to the latest version of this plugin. We had started using v2.3.0, but with the html layer it has got some serious bugs that are blocking the functional properties of our badges. In the code, there has been some calls to variables whose references aren't resolved giving uncaught errors, catching those errors block the behavioural properties of badges.
This is the error we get in production compiled build:

This error is arising from here:
https://github.com/sgratzl/cytoscape.js-layers/blob/74cd9a7beeac847c748c7e1327fc176e7f5cf5b5/src/layers/ABaseLayer.ts#L67-L69
fixing this particular error, the line 69 contains this.update(); which throws further error of update is not a function.
This has been the error with the v2.2.0 also.
The cytoscape badge in our UI looks like this:
on changing the position of node, the position of badge doesnt change and stay at the same place.
To Reproduce

const createBadge = (nodeId, render, cy,) => {
  /** @type {LayersPlugin} */
  const layers = cy.layers();

  layers.renderPerNode(
    layers.append("html"),
    (elem, node) => {
      if (node.data("id") === nodeId) {
        render(node, elem);
      }
    },
  );
};

layers.append("html") creates an html layer and render process has the styling config for it.
On changing position of node, it throws an error as described above.

Expected behavior
On changing the position of the node, the badge should move at the correct place.

Screenshots

Context

Version: 2.3.0 and 2.2.0
Browser: Any

Additional context

Looking over the NPM package download stats, we've been using 2.1.0 for a couple years now and might be your biggest fans, @sgratzl. 😉


Moved some log messages to debug level

Fixes #1518

Changed some messages back to INFO. Not sure if this PR does anything anymore.

@kke it does remove recurring spam.


VS2017 and csproj migration

I have migrated the web solution to VS 2017 and updated as required to get this running locally.
One gotcha was many, many failures when trying to build it the first time. >200
Seems that the easiest way to solve this is to complete clear the old nuget cache at USERPROFILE%/.nuget
Close VS 2017 first and afterwards run a clean and build.
I've upgraded a few package versions as needed. There's still some places we could take newer dependencies but I have left those that I can until a future PR.
I've made the changes on the latest master branch, that is why this includes many commits from other contributors since the vs2017 branch on GitHub was made.
@tonysurma - Could you give this a try locally
@MisterJames - Perhaps you could try it also to make sure this works for more than just me :-)
If it looks good I can make a fresh PR against the master if we want to bring it in there.

I will test later today or tomorrow AM

@stevejgordon quick dumb question for me and future generations, why does this PR contain a bunch of commits from other PRs etc.?  I was trying to look at which files had to change and it makes it look like a bunch of unrelated files had to change?
 It's because the vs2017 branch we have here is quite a bit behind the main master branch. However, to ensure I had the most up to date code possible I used the current master when starting my work. Since I've made the PR against the older vs2017 it's showing a bunch of prior commits as changed.
I can make the PR against the main master if that's easier for review?
Not sure if there's a way to catch vs2017 back up to date with master or recreate and I can PR again after that.

@stevejgordon I totally forgot we made that branch, since it appears we can make this work in one pr let's do it off of master.  My local build worked, I just want to verify a deploy can work.  Figure we need to give everyone a bit of notice as well.

@tonysurma I have opened #1950 against master so will close this against the branch


Search in files skips files that havenare open but have not been activated yet

When multiple files are open, a global search will only return matches from files that have been activated since starting vscode (meaning: Clicking on the file's tab to make it visible)
Steps to reproduce:

Open a number of files. For instance by dragging a selection from Windows Explorer
Open the Search window (Ctrl+Shift+F) and search for a text that is in some of the opened files
Notice that search returns only matches from the active file
Activate one of the other files
Repeat step 3, and notice that now results from 2 files are returned

VS Code version: Code 1.67.1 (da15b6fd3ef856477bf6f4fb29ba1b7af717770d, 2022-05-06T12:37:03.389Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10850H CPU @ 2.70GHz (12 x 2712)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.78GB (20.84GB free)

Process Argv
--crash-reporter-id fe6ec923-5563-4e9b-8607-b38137618d79

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

vscode-m3u
af4
1.0.0

vscode-eslint
dba
2.2.2

xml
Dot
2.5.1

azure-pipelines
ms-
1.205.0

csharp
ms-
1.24.4

python
ms-
2022.6.2

azure-account
ms-
0.10.1

powershell
ms-
2022.5.1

vs-keybindings
ms-
0.2.1

log-analysis
Xin
1.1.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
testflightcf:30433671
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebugcf:30475217
vsclangdc:30486549

Maybe you have clicked this?

If you have, you should see this extra message at the top of your results:

And after it a "(disable)" link with which to toggle that button and re-run the search.

@gjsjohnmurray
That options was off. If I turn it on, the search works as expected. But the issue is really something else: With this option turned off, only files that have been activated in the current vscode session are searched. So, initially one 1 file is searched, and after I Alt-Tab to another file and repeat the same search, 2 files are searched. This behavior is inconsistent.
I also noticed that if I open a folder, rather than dragging random files in from Explorer, the search works in all files, at least if the Search only in open editors option is off. With Search only in open editors on, only open files are searched, as expected.

I think I understand what you are reporting.

With no VS Code instances running, open a new VS Code.
From Windows Explorer drag three individual files onto the VS Code window. Each one gets loaded in its own tab.
Use the Search view to find text which occurs in all of the files. The results show the matches correctly.
Close VS Code.
Launch VS Code again. You see tabs for each of the files you previously dragged into VS Code. One of these tabs is active, and the contents of the file are visible.
Use the Search view to find the same text as previously. This time it only reports matches from the file in the active tab.
Click on the next tab and repeat your search. Now it finds matches in this one as well, but still not in the third one.
Click on the third tab and repeat search. Now it finds matches in all three files.

Is that an accurate description of what this issue is about?

Yes, that's pretty much it. You could skip steps 3–5 by dragging multiple files at once from Explorer, in which case only one file becomes active, and search will only see that file. When opening files one by one, each one will be activated already, and search works as it should.
Somewhat related: The Search only in open editors makes sense when a folder is opened in VS Code, but IMO not so much when only separate files are opened. Should I file a new issue about that?

I'm aware of this issue and thought it was covered by an open issue somewhere, but can't find it right now. You're right, files only get searched when they have been loaded in an editor in your scenario, or if they are in a folder that is open in the window.

The Search only in open editors makes sense when a folder is opened in VS Code, but IMO not so much when only separate files are opened. Should I file a new issue about that?

I think in this scenario, the search should return the same results regardless of that option, and I consider that the same issue.

https://github.com/microsoft/vscode/issues/62488


Add plaintext license.

So that we'll have a nice license badge on GitHub! 💄

Thanks for the contribution @kdeldycke !

LGTM

Thanks for the merge! That was fast! 👍


Do you have a plug-in for model encryption?Before generation？

This template is for miscellaneous issues not covered by the other issue categories.
For questions on how to work with TensorFlow, or support for problems that are not verified bugs in TensorFlow, please go to StackOverflow.
If you are reporting a vulnerability, please use the dedicated reporting process.
For high-level discussions about TensorFlow, please post to<EMAIL_ADDRESS>for questions about the development or internal workings of TensorFlow, or if you would like to know how to contribute to TensorFlow, please post to<EMAIL_ADDRESS>

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
Thanks!


Offline transaction generation not working

Hi,
Using either chrome, or firefox (latest versions) on my offline machine, I'm seeing the following error in the console:
"TypeError: Cannot read property 'result' of undefined" Etherwallet-master.js:41964
The "Generate Transaction" button is not enabled and will not generate a transaction, despite filling in all fields. I've tried using both my wallet file and the private key directly.

This bug makes it impossible to generate offline transactions on mobile devices. Please fix!


fix(generator): Connection base-class operations should fail

While we do not test the behavior of the generated "ServiceConnection"
base classes, all operations should fail with kUnimplemented. That
was not the case for paginated calls, which previously returned an
empty sequence instead.

This change is

Can we return this instead:

Done.  PTAL.

Note: Unfortunately the change in google/cloud/internal/pagination_range.h from "needs-override" to "not implemented" was incorrectly grouped in the "regeneration" commit ef7628a.


Getting Duplicate interface definition for class 'AppDelegate' AND Property has a previous declaration errors when adding swiftvalidator delegate method to my uiviewcontroller

im getting odd errors with one of the swiftvalidator delegate methods
i added swift validator to my podfile
podfile looks as follows
platform :ios, '8.0'
use_frameworks!
pod 'FBSDKCoreKit'
pod 'FBSDKLoginKit'
pod 'SwiftValidator', '2.1.1'
i imported swiftvalidator to my LoginViewController (which is a subclass of UIViewController) by declaring import SwiftValidator
my class is declared as
class LoginViewController: UIViewController, UIAlertViewDelegate, UITextFieldDelegate, ValidationDelegate {
I initialised let validator = Validator() at the top level of the class
i added validator.registerField(emailField, errorLabel: emailErrorLabel, rules: [RequiredRule(), EmailRule()]) to viewdidload
i added skeleton of validationSuccessful delegate method as follows
func validationSuccessful() {
// submit the form
println("validation success")
}
but when i add other delegate method ie validationFailed as follows
func validationFailed(errors: [UITextField : ValidationError]){
    println("test")
}

i get the following build errors ...
Getting Duplicate interface definition for class 'AppDelegate' and
Property has a previous declaration
in the apps swift header file ie komunety-Swift.h
if i change the errors parameter in this delegate function ie to func validationFailed(errors: [UITextField : String] the error goes away ... ie it seems to be related to the ValidationError type itself
any ideas on how to fix

Were you able to find a solution to this?

no. I ended up up just copying the framework manually rather that using cocoapods

I just had the same problem, and the problem is because AppDelegate.swift is installed with the pod.

Fixed in 3.0.1


Mapping private properties

The document explains how to use the NotMapped attribute or the .Ignore fluent method to exclude public properties. How do we include (map) private properites?

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

ID: 76c24b57-557f-a6d6-16f1-1da50e4744cd
Version Independent ID: 8bee29c6-58e4-1d07-025e-0bdcead9b5a1
Content: Including & Excluding Properties - EF Core
Content Source: entity-framework/core/modeling/included-properties.md
Product: entity-framework
GitHub Login: @rowanmiller
Microsoft Alias: divega

@sjb-sjb short answer is by using the Property fluent API. But this is obviously missing in the documentation page.

It is worth noting that EF cannot always be informed of collection navigation properties using the Property fluent API. Specifically, the Property method cannot be used to make EF aware of a private navigation whose collection type is a private type derived from ICollection. This is because the Property API does not allow specifying a base class as the type: if you use entityTypeBuilder.Property( typeof(ICollection), propertyName), where the property type is derived from ICollection, then an exception will be thrown saying that the property cannot be added because the property does not have type ICollection. On the other hand if you specify the actual (private) collection type then EF cannot process the type during model construction because it is not a type recognized by EF for navigation collections. The only way to handle this situation, it seems, is to have the collection be public in which case EF notices the ICollection base type and is able to access and use the property.
Also worth noting that specifying the InverseProperty attribute on the navigation collection property does not by itself make the property known to EF.

[Bug] Fluent API to exclude a property.
            modelBuilder.Entity<Province>(entity =>
            {
                entity.HasKey(e => e.Iso)
                    .HasName("PK_Province_Iso");

                entity.Ignore(e => e.CountryCodeNavigation);

                entity.Property(e => e.Iso)
                    .HasMaxLength(4)
                    .IsUnicode(false)
                    .IsFixedLength()
                    .ValueGeneratedNever();

                entity.Property(e => e.Name)
                    .IsRequired()
                    .HasMaxLength(80);

                entity.HasOne(e => e.CountryCodeNavigation)
                    .WithMany(e => e.Provinces)
                    .HasForeignKey(e => e.CountryCode)
                    .HasConstraintName("FK_Province_Country_CountryCode")
                    .OnDelete(DeleteBehavior.Restrict);
            });

If I place the script at the top of the entity group, the property still remains in the data response. If I place it at the bottom of the entity group, it will change the DeleteBehavior.Restrict value to DeleteBehavior.Cascade and also not exclude it from the data response.
Migration data.
            migrationBuilder.CreateTable(
                name: "Provinces",
                columns: table => new
                {
                    Iso = table.Column<string>(unicode: false, fixedLength: true, maxLength: 4, nullable: false),
                    Name = table.Column<string>(maxLength: 80, nullable: false),
                    CountryCode = table.Column<short>(nullable: false)
                },
                constraints: table =>
                {
                    table.PrimaryKey("PK_Province_Iso", x => x.Iso);
                    table.ForeignKey(
                        name: "FK_Provinces_Countries_CountryCode",
                        column: x => x.CountryCode,
                        principalTable: "Countries",
                        principalColumn: "Code",
                        onDelete: ReferentialAction.Cascade);
                });

@Yuant-tobing Both Ignoring CountryCodeNavigation and later configuring as a navigation is not valid. That is, it's either a navigation or it's ignored--it can't be both.
That being said. this issue tracker is for documentation. If you believe that you're hitting a bug, then please file a new issue at https://github.com/aspnet/EntityFrameworkCore/issues and include a small, runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

One additional comment on this, I would love to have a PropertyAttribute(string propertyName), because then when I am processing my class definition using T4 I can easily add this attribute if the access is not public. Adding fluent code to the model builder is a lot harder to do using T4.

Another comment: it seems that a non-public property cannot be used as the Inverse Property for a collection. I would classify it as a bug.

@sjb-sjb If that's the case, that would be a bug. Please file a new issue and post a small, runnable project/solution or complete code listing that demonstrates the behavior you are seeing.

short answer is by using the Property fluent API. But this is obviously missing in the documentation page. Thanks for posting.

I would also prefer to have a MappedAttribute like there is a NotMappedAttribute. I mainly try to configure my whole data model via data annotations. Many things seem to be able only via Fluent API which is IMHO not good. Fluent API is EF specific while Data Annotations can be used by multiple frameworks. I'm having an OData API on top of my data model but need to maintain certain things twice because they are "Fluent API" only (like composite keys and cascade delete configurations).
Currently I'm having a class with some properties as explicit interface implementations and cannot tell EF to map them.

How do you ignore a non primitive type column? It cannot be mapped as a Property.

Make sure to reiterate or make clear the tips described at https://docs.microsoft.com/en-us/ef/core/modeling/constructors#read-only-properties


Stop the pagination page 1 from removing location

When you went to page 1 in the pagination it would clear all url
parameters, which was lazyness on my side when I implemented it,
didn't think of this use case.
Tgis fix ensures that we only remove the pages search param on page 1.

 When changing search type to all from in need with predefined search parameters it just returns all results.


Make recursion and options optional for requireDir

Description :
using recursion and options in requireDir is not always neccessary, but since they are not optional, it is annoying to provide them every time when it's not used
Suggestion :

make recursion false by default
make options  { filterExtension: ['.js'] }

by default, and other options could be undefined

development active on issue#1


.NET SDK does not appear to provide the ability to self-sign a certificate.

@GregDomzalski
When can I expect this feature?

The sample code demonstrates how to build a self-signed cert. It uses the C# CertificateRequest class.
The SDK does not have a class or method to build a self-signed cert, but the CertificateRequest class does. So if you want to build a self-signed cert, use that class.
The only trick is to get the CertificateRequest class to use the YubiKey to sign, instead of software. But that's done using an X509SignatureGenerator. So all you need to do is build one of those. Or use the one in the sample code.
The PIV sample (.../dotnet-enterprisesdk/Yubico.YubiKey/examples/PivSampleCode) includes code that uses the CertificateRequest class to build a self-signed cert, including a "YubiKeySignatureGenerator".

Closing this issue as it looks like there is a path forward with example code.
If you have further questions or problems getting a solution up and running, please feel free to reopen the issue.


WIP: blog architect comparison

comparisons:

normalised state + REST
apollo + graphql
nested state + graphql
normalised state + graphql
nested state + REST

Coverage decreased (-43.2%) to 56.774% when pulling 2e988f4d440ea987366d9287a75dda2d2608e0d8 on restful-normalised into ce07fae826b1a2eb049b574d6861b03741b66c55 on master.

Coverage decreased (-41.6%) to 58.385% when pulling 86d39a55a91b32d743fc80c2eacdbbeb8b82fe02 on restful-normalised into ce07fae826b1a2eb049b574d6861b03741b66c55 on master.

Coverage decreased (-38.9%) to 61.143% when pulling ff0c54c116cfe35069bc61653c0af83192376d46 on restful-normalised into ce07fae826b1a2eb049b574d6861b03741b66c55 on master.

Coverage decreased (-44.3%) to 55.729% when pulling 6f9c883e15195e6459043434a590ae6e448146de on restful-normalised into ce07fae826b1a2eb049b574d6861b03741b66c55 on master.

Coverage decreased (-44.9%) to 55.102% when pulling<PHONE_NUMBER>c0bbb96c9f43e97b2086fa936f2c06 on restful-normalised into ce07fae826b1a2eb049b574d6861b03741b66c55 on master.

Coverage decreased (-44.2%) to 55.779% when pulling 02391c035c4d5d32d3701ad84387b1a34d21f66c on restful-normalised into ce07fae826b1a2eb049b574d6861b03741b66c55 on master.


Better to elaborate more about "-ClientRootCert" parameters.

[Enter feedback here]
Nowadays, every windows machines keep a bunch of clients certificates and I tested by myself (MS FTE alias: gujin) I had to fill out "-ClientRootCert" when I download certificates from Azure Portal.
otherwise, with a p2s client package without "-ClientRootCert" would returns error 798.
After I put -ClientRootCert, we would prompt so that I can pick the right cert.

https://docs.microsoft.com/en-us/azure/vpn-gateway/point-to-site-vpn-client-configuration-radius#certeap
-ClientRootCert is the .cer file that contains the root certificate. It's an optional parameter. If the device that you want to connect from has only one client certificate, you don't have to specify this parameter.

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

ID: a601bb83-ee21-3417-4f71-b63de99ad3f8
Version Independent ID: 75914163-23e9-48f5-fe0a-2996f4a1d84c
Content: Azure VPN Gateway: Create & install VPN client config files - P2S RADIUS connections
Content Source: articles/vpn-gateway/point-to-site-vpn-client-configuration-radius.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@GavinJin8 Thank you for your feedback . We will investigate and update the thread.

@GavinJin8 Thank you for your feedback . We will investigate and update the thread.

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


[Future build] Add language and scope support in the build generator

Issue #702 has asked to add language and scope to csolution which has been done. Unfortunately, cbuildgen does not support this. So the support in the CMSIS-Toolbox is only half-baked. With a partner, I am working on a CMSIS-Pack that requires the "language" attribute being supported throughout the whole tools flow.
As we are planning for the updated build generator tool based on cmake, I ask for adding this to the list of features to be present from the start.

Implemented in https://github.com/Open-CMSIS-Pack/cbuild2cmake/pull/91 and https://github.com/Open-CMSIS-Pack/devtools/pull/1433.
It will be available in the next release CMSIS-Toolbox 2.4.0.


cursor invisible when selecting

is this normal on atom? when selecting, the background is so dark gray that it's very hard to see where the cursor is supposed to be.

Could not replicate this anymore, close for now. Reopen if it reoccurs.


PVA: uischema

We need a uischema for a PVA runtime template that expresses their custom recognizer experience

Update: they don't have a custom recognizer experience, they will be authoring to the LU file format. Closing as not necessary.


Fix script fetching and local path handling in inspect_dataset and inspect_metric

Fix #4348

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

Added back the [:] and a comment to explain why this is needed.


⚠ Samsung has degraded performance

In 5ccf58d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2169 ms

Resolved: Samsung performance has improved in 0e3eacf.


fix zap logger rotation issues

Reason for Change:

This PR is to fix log rotation issue that is crashed after windows nodes are restarted.
Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] relevant PR labels added

Notes:

Can we add more to the description about what the rotation issue is ?


        The returned instance will have attributes args, returncode, stdout and
        stderr. By default, stdout and stderr are not captured, and those attributes
        will be None. Pass stdout=PIPE and/or stderr=PIPE in order to capture them.

        If check is True and the exit code was non-zero, it raises a
        CalledProcessError. The CalledProcessError object will have the return code
        in the returncode attribute, and output & stderr attributes if those streams
        were captured.

        If timeout is given, and the process takes too long, a TimeoutExpired
        exception will be raised.

        There is an optional argument "input", allowing you to
        pass bytes or a string to the subprocess's stdin.  If you use this argument
        you may not also use the Popen constructor's "stdin" argument, as
        it will be used internally.

        By default, all communication is in bytes, and therefore any "input" should
        be bytes, and the stdout and stderr will be bytes. If in text mode, any
        "input" should be a string, and stdout and stderr will be strings decoded
        according to locale encoding, or by "encoding" if set. Text mode is
        triggered by setting any of text, encoding, errors or universal_newlines.

        The other arguments are the same as for the Popen constructor.
        """
        if input is not None:
            if kwargs.get('stdin') is not None:
                raise ValueError('stdin and input arguments may not both be used.')
            kwargs['stdin'] = PIPE

        if capture_output:
            if kwargs.get('stdout') is not None or kwargs.get('stderr') is not None:
                raise ValueError('stdout and stderr arguments may not be used '
                                 'with capture_output.')
            kwargs['stdout'] = PIPE
            kwargs['stderr'] = PIPE

        with Popen(*popenargs, **kwargs) as process:
            try:
                stdout, stderr = process.communicate(input, timeout=timeout)
            except TimeoutExpired as exc:
                process.kill()
                if _mswindows:
                    # Windows accumulates the output in a single blocking
                    # read() call run on child threads, with the timeout
                    # being done in a join() on those threads.  communicate()
                    # _after_ kill() is required to collect that and add it
                    # to the exception.
                    exc.stdout, exc.stderr = process.communicate()
                else:
                    # POSIX _communicate already populated the output so
                    # far into the TimeoutExpired exception.
                    process.wait()
                raise
            except:  # Including KeyboardInterrupt, communicate handled that.
                process.kill()
                # We don't call process.wait() as .__exit__ does that for us.
                raise
            retcode = process.poll()
            if check and retcode:
                raise CalledProcessError(retcode, process.args,
>                                        output=stdout, stderr=stderr)
E               subprocess.CalledProcessError: Command '['/home/runner/work/setuptools/setuptools/.tox/python/bin/python', 'setup.py', 'install', '--prefix', '/tmp/pytest-of-runner/pytest-0/target1']' returned non-zero exit status 2.

/opt/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/subprocess.py:512: CalledProcessError

This issue appears to be due to pypa/packaging#363.

The reason the tests pass locally is because the tests were all skipped due to
SKIPPED [9] setuptools/tests/test_integration.py:36: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1122)>


I can not find opencv-python-master/cv2/version.py

Expected behaviour
I build opencv-python from source and run 'python setup.py bdist_wheel' but got error msgs  below
File "setup.py", line 455, in
main()
File "setup.py", line 62, in main
build_contrib, build_headless, is_CI_build
File "setup.py", line 424, in get_and_set_info
with open(version_file) as fp:
FileNotFoundError: [Errno 2] No such file or directory: 'opencv-python-master/cv2/version.py'

Please use pip . wheel --verbose as described in the readme. Make sure that opencv-python-master is a Git repository. It won't work if you just download the code, you must clone it.
https://github.com/skvark/opencv-python#manual-builds

It is work.
Thank you very much


cointop windows width problem (too narrow)

Hello,
after running cointop, its window is always this narrow (picture below), regardless of my Terminal window width. However, in the official cointop GitHub picture its window is much wider, occupying the entire Terminal window width.
mine:

official cointop GitHub picture:
https://user-images.githubusercontent.com/168240/39569578-7ce9f3b6-4e7a-11e8-82a9-8a18b91b1bd5.png
What could be the problem?

Hi @sinepgnol, can you provide the following details:

your OS
which terminal you're using
how you installed cointop

It would help me in debugging. Thanks

my OS: macOS Catalina 10.15.6
I'm using Apple's default built-in app called Terminal with zsh shell.
I installed cointop via Homebrew

Can't seem to reproduce this. Very strange that it would be cut off like that.


Typo in etcd health check

I'm not an etcd expert but there seems to be a typo in etcd health check:
"path": "/healthz",

@jdef James could you have a look at that?

https://github.com/mesosphere/etcd-mesos/blob/8208cfad1d876bdd73162dbc7993eb2083ffd0b5/scheduler/scheduler.go#L1040
/healthz should be a valid endpoint. is the health check not working for
you?
On Fri, Nov 24, 2017 at 5:53 PM, Tomas Barton<EMAIL_ADDRESS>wrote:

I'm not an etcd expert but there seems to be a typo in etcd health check
https://github.com/mesosphere/universe/blob/version-3.x/repo/packages/E/etcd/1/marathon.json.mustache#L32
:
"path": "/healthz",
@jdef https://github.com/jdef James could you have a look at that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/universe/issues/1543, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPVLJ7t_1JDHdwok4tsabShZ7T44f6pks5s50j3gaJpZM4QqOdY
.

Right, sorry I was looking into etcd source code instead:
https://github.com/coreos/etcd/blob/0dbcd7c1a7321f0fffa4c807d35cd5ca5ed04a00/etcdserver/api/etcdhttp/metrics.go#L51


Sbt Setting for package name

I was looking around for some way to set the package name in the build.sbt file instead of having to rely on the file structure of the templates.  It would be nice if you could set a package prefix that would be appended to the beginning of the file structure name.

I don't really like this idea - since Scala templates are a DSL with a straight forward translation to Scala, we should consider features like this in the same vein as Scala source files, would you also have an sbt setting that prefixes package names from Scala source files?  I don't think so.  Perhaps we could support an @package directive at the start of the file, but I think using the directory structure by default makes sense since matching directory structure to package structure is a convention that's used (and in the case of Java, enforced) for all JVM languages anyway.


farrah/FEQ-2672/deep linking in ios and android

Added iOS and Android deep linking.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

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

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
6391

Relevant Lines:
6952

💛  - Coveralls


[BUG] DeepSpeed Inference example in the tutorial got killed for no reason.

Describe the bug
I am running the tutorial GPT-neo inference example in https://www.deepspeed.ai/tutorials/inference-tutorial/. This is my inference code:
import os
import deepspeed
import torch
from transformers import pipeline

local_rank = int(os.getenv('LOCAL_RANK', '0'))
world_size = int(os.getenv('WORLD_SIZE', '1'))
generator = pipeline('text-generation', model='EleutherAI/gpt-neo-125M',
                     device=local_rank)

generator.model = deepspeed.init_inference(generator.model,
                                           mp_size=world_size,
                                           replace_with_kernel_inject=True
                                            )

string = generator("DeepSpeed is", min_length=50, num_return_sequences=1, max_length=50)
if not torch.distributed.is_initialized() or torch.distributed.get_rank() == 0:
    print(string)

I got no error message and the process get killed for no reason:
Setting ds_accelerator to cuda (auto detect)
[2023-06-08 04:35:52,281] [WARNING] [runner.py:196:fetch_hostfile] Unable to find hostfile, will proceed with training with local resources only.
[2023-06-08 04:35:52,296] [INFO] [runner.py:555:main] cmd = /usr/bin/python -u -m deepspeed.launcher.launch --world_info=eyJsb2NhbGhvc3QiOiBbMCwgMV19 --master_addr=<IP_ADDRESS> --master_port=29500 --enable_each_rank_log=None ds_infer.py
Setting ds_accelerator to cuda (auto detect)
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NV_LIBNCCL_DEV_PACKAGE=libnccl-dev=2.13.4-1+cuda11.7
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NV_LIBNCCL_DEV_PACKAGE_VERSION=2.13.4-1
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NCCL_VERSION=2.13.4-1
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NV_LIBNCCL_DEV_PACKAGE_NAME=libnccl-dev
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NV_LIBNCCL_PACKAGE=libnccl2=2.13.4-1+cuda11.7
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NV_LIBNCCL_PACKAGE_NAME=libnccl2
[2023-06-08 04:35:53,337] [INFO] [launch.py:138:main] 0 NV_LIBNCCL_PACKAGE_VERSION=2.13.4-1
[2023-06-08 04:35:53,337] [INFO] [launch.py:145:main] WORLD INFO DICT: {'localhost': [0, 1]}
[2023-06-08 04:35:53,337] [INFO] [launch.py:151:main] nnodes=1, num_local_procs=2, node_rank=0
[2023-06-08 04:35:53,337] [INFO] [launch.py:162:main] global_rank_mapping=defaultdict(<class 'list'>, {'localhost': [0, 1]})
[2023-06-08 04:35:53,337] [INFO] [launch.py:163:main] dist_world_size=2
[2023-06-08 04:35:53,337] [INFO] [launch.py:165:main] Setting CUDA_VISIBLE_DEVICES=0,1
Setting ds_accelerator to cuda (auto detect)
Setting ds_accelerator to cuda (auto detect)
Xformers is not installed correctly. If you want to use memory_efficient_attention to accelerate training use the following command to install Xformers
pip install xformers.
[2023-06-08 04:35:57,463] [INFO] [logging.py:96:log_dist] [Rank -1] DeepSpeed info: version=0.9.3+4559aa9b, git-hash=4559aa9b, git-branch=HEAD
[2023-06-08 04:35:57,463] [WARNING] [config_utils.py:69:_process_deprecated_field] Config parameter mp_size is deprecated use tensor_parallel.tp_size instead
[2023-06-08 04:35:57,464] [INFO] [logging.py:96:log_dist] [Rank -1] quantize_bits = 8 mlp_extra_grouping = False, quantize_groups = 1
[2023-06-08 04:35:57,466] [WARNING] [comm.py:152:init_deepspeed_backend] NCCL backend in DeepSpeed not yet implemented
[2023-06-08 04:35:57,466] [INFO] [comm.py:594:init_distributed] cdb=None
[2023-06-08 04:35:57,466] [INFO] [comm.py:625:init_distributed] Initializing TorchBackend in DeepSpeed with backend nccl
Xformers is not installed correctly. If you want to use memory_efficient_attention to accelerate training use the following command to install Xformers
pip install xformers.
[2023-06-08 04:35:57,555] [INFO] [logging.py:96:log_dist] [Rank -1] DeepSpeed info: version=0.9.3+4559aa9b, git-hash=4559aa9b, git-branch=HEAD
[2023-06-08 04:35:57,555] [WARNING] [config_utils.py:69:_process_deprecated_field] Config parameter mp_size is deprecated use tensor_parallel.tp_size instead
[2023-06-08 04:35:57,555] [INFO] [logging.py:96:log_dist] [Rank -1] quantize_bits = 8 mlp_extra_grouping = False, quantize_groups = 1
[2023-06-08 04:35:57,558] [WARNING] [comm.py:152:init_deepspeed_backend] NCCL backend in DeepSpeed not yet implemented
[2023-06-08 04:35:57,558] [INFO] [comm.py:594:init_distributed] cdb=None
[2023-06-08 04:35:57,569] [INFO] [logging.py:96:log_dist] [Rank 0] DeepSpeed-Inference config: {'layer_id': 0, 'hidden_size': 768, 'intermediate_size': 3072, 'heads': 12, 'num_hidden_layers': -1, 'dtype': torch.float16, 'pre_layer_norm': True, 'norm_type': <NormType.LayerNorm: 1>, 'local_rank': -1, 'stochastic_mode': False, 'epsilon': 1e-05, 'mp_size': 2, 'scale_attention': True, 'triangular_masking': True, 'local_attention': False, 'window_size': 256, 'rotary_dim': -1, 'rotate_half': False, 'rotate_every_two': True, 'return_tuple': True, 'mlp_after_attn': True, 'mlp_act_func_type': <ActivationFuncType.GELU: 1>, 'specialized_mode': False, 'training_mp_size': 1, 'bigscience_bloom': False, 'max_out_tokens': 1024, 'min_out_tokens': 1, 'scale_attn_by_inverse_layer_idx': False, 'enable_qkv_quantization': False, 'use_mup': False, 'return_single_tuple': False, 'set_empty_params': False, 'transposed_mode': False}
Setting `pad_token_id` to `eos_token_id`:50256 for open-end generation.
------------------------------------------------------
Free memory : 20.773438 (GigaBytes)
Total memory: 23.689514 (GigaBytes)
Requested memory: 0.087891 (GigaBytes)
Setting maximum total tokens (input + output) to 1024
WorkSpace: 0x7f2916000000
------------------------------------------------------
Setting `pad_token_id` to `eos_token_id`:50256 for open-end generation.
[2023-06-08 04:35:59,354] [INFO] [launch.py:314:sigkill_handler] Killing subprocess 2824
[2023-06-08 04:35:59,354] [INFO] [launch.py:314:sigkill_handler] Killing subprocess 2825
[2023-06-08 04:35:59,355] [ERROR] [launch.py:320:sigkill_handler] ['/usr/bin/python', '-u', 'ds_infer.py', '--local_rank=1'] exits with return code = -7

I have double checked that the process is not been killed due to OOM.
To Reproduce
deepspeed --num_gpus 2 ds_infer.py

ds_report output
Setting ds_accelerator to cuda (auto detect)
--------------------------------------------------
DeepSpeed C++/CUDA extension op report
--------------------------------------------------
NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
async_io ............... [YES] ...... [OKAY]
cpu_adagrad ............ [YES] ...... [OKAY]
cpu_adam ............... [YES] ...... [OKAY]
fused_adam ............. [YES] ...... [OKAY]
fused_lamb ............. [YES] ...... [OKAY]
quantizer .............. [YES] ...... [OKAY]
random_ltd ............. [YES] ...... [OKAY]
sparse_attn ............ [YES] ...... [OKAY]
spatial_inference ...... [YES] ...... [OKAY]
transformer ............ [YES] ...... [OKAY]
stochastic_transformer . [YES] ...... [OKAY]
transformer_inference .. [YES] ...... [OKAY]
utils .................. [YES] ...... [OKAY]
--------------------------------------------------
DeepSpeed general environment info:
torch install path ............... ['/usr/local/lib/python3.8/dist-packages/torch']
torch version .................... 1.13.1+cu117
deepspeed install path ........... ['/usr/local/lib/python3.8/dist-packages/deepspeed']
deepspeed info ................... 0.9.3+4559aa9b, 4559aa9b, HEAD
torch cuda version ............... 11.7
torch hip version ................ None
nvcc version ..................... 11.7
deepspeed wheel compiled w. ...... torch 1.13, cuda 11.7

System info (please complete the following information):

Ubuntu 20.04 LTS (in Docker)
2 x RTX 3090 24G
transformers version: 4.31.0.dev0
Python version: 1.13.1
Any other relevant info about your setup

Docker context
I am using the Dockerfile from deepspeed's repo https://github.com/microsoft/DeepSpeed/blob/master/docker/Dockerfile with the base image changed to nvidia/cuda:11.7.0-devel-ubuntu20.04.

Same bug

Same bug

Same bug

same bug

maybe try add shm_size.
refer to this
https://stackoverflow.com/questions/30210362/how-to-increase-the-size-of-the-dev-shm-in-docker-container
or modify shm_size with a exist container.

stop docker
sudo systemctl stop docker
cd to container_path and vim hostconfig
sudo -s
cd /var/lib/docker/container/YOUR_CONTAINER_ID
vim hostconfig.json
modify this ->> "ShmSize": 67108864
to                ->> "ShmSize":<PHONE_NUMBER> (Choose a larger number that you like, I choose 4G)
restart docker service
sudo systemctl restart docker

ref
https://discuss.pytorch.org/t/training-crashes-due-to-insufficient-shared-memory-shm-nn-dataparallel/26396/2

maybe try add shm_size.
refer to this https://stackoverflow.com/questions/30210362/how-to-increase-the-size-of-the-dev-shm-in-docker-container or modify shm_size with a exist container.

stop docker
sudo systemctl stop docker
cd to container_path and vim hostconfig
sudo -s
cd /var/lib/docker/container/YOUR_CONTAINER_ID
vim hostconfig.json
modify this ->> "ShmSize": 67108864
to                ->> "ShmSize":<PHONE_NUMBER> (Choose a larger number that you like, I choose 4G)
restart docker service
sudo systemctl restart docker

ref https://discuss.pytorch.org/t/training-crashes-due-to-insufficient-shared-memory-shm-nn-dataparallel/26396/2

It seems not the reason of shm, my shm is about 100g, but still error when training.

Interesting! Having the same bug for 2 GPU settings, 1 GPU one is working fine for me.

Setting the shm_size to 128GB works for me.
Thanks!


dev.jspm.io is now jspm.dev

see jspm.dev Release

dev.jspm.io will still be available for a further 12 months until June 2021. Upgrading to jspm.dev should be a seamless upgrade path in most cases, with the main user-facing compatibility change being that not all internal subpaths are exposed due to the code splitting optimizations.

Follow the import at the top of mod.ts file
import HandlebarsJS from<EMAIL_ADDRESS>
The source map in external file can be fetched successfully in jspm.dev  while returns 500 in dev.jspm.io.
Therefore it is suggested to use the new URL instead.

somehow the import with dev.jspm.io works again.
closing this PR

maybe I should keep it open because it works sometimes only

Seems @mocoso is switching the source of handlebars to esm.sh but stuck in the build test failure which due to old deno version

#17


enable workbench.editor.revealIfOpen by default

workbench.editor.revealIfOpen is feature that should be enable by default. The current behavior is cause of many griefs, as you can see from a quick search

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Double public and public_collected directories

After building this, and running ./run manage collectstatic I've noticed there are four directories for static assets in the web container (in addition to /app/assets), but only two are used:

/public - always empty
/public_collected - fills with the assets after collectstatic
/app/public - the static assets collectstatic collects
/app/public_collected - always empty

I wonder if this means there are some unnecessary creations in the Dockerfile, but I'm not sure which you intended to be correct? Or have I misunderstood something?

Hi,
The (2) /public*/ directories are there to hold your assets in a non-volume mounted location so they never get overwritten by your volumes. These get populated at build time when you build your Docker image, the /public_collected/ directory gets populated when collectstatic gets run based on having set DEBUG=false in your .env at build time.
At runtime (thanks to https://github.com/nickjj/docker-django-example/blob/main/bin/docker-entrypoint-web) the public_collected/ directory gets copied over to /app/public_collected where your volume can take over to allow you to persist files back to your Docker host and potentially serve these assets from a web server not running in Docker such as nginx.
It is pretty convoluted but I don't see a way around all of this. We have to meet 2 criteria here:

Django does not let you output collectstatic in the same directory where your source is, this means we need 2 sets of public/ directories, I named them accordingly
We need to build both sets of public files in a non-volume mounted directory but also make them available in a volume at runtime in such a way where existing files in the volume on the Docker host at runtime won't overwrite the newly built public files, this means we need to store both sets of public/ directories in 2 locations

If you have a way to meet both of these criteria in a cleaner way I'm all for improving it. I'd love to be able to get rid of this workflow.

Thanks that makes sense. I've only been using this for local development so far, so haven't quite grasped the differences with how it would work in production.
Previously I've only used manage.py runserver during development, not gunicorn, and added "whitenoise.runserver_nostatic" to INSTALLED_APPS if DEBUG=True. Which I think means I don't need to run collectstatic except on the production server (where I wasn't using Docker).
I assume we need to run collectstatic in development because we're using gunicorn?
Now that I'm thinking of using Docker for both dev and production I have this extra level of complication to get my head round I guess!
In my dabblings with Docker and Django I'm always frustrated that dealing with static files ends up being so complicated!

You don't need to run collectstatic in development.
The exact instructions in the readme file combined with the source code in this repo will work out of the box and your assets will get served in development.
The collection process is mainly around digesting the files with an md5 hash, it's aimed at production although there's nothing stopping you from digesting them manually in development. It happens in production by default due to the Dockerfile having a condition to run it when DEBUG=false:
https://github.com/nickjj/docker-django-example/blob/b87f4654da1c4c5b7d1d81fffc42445c51cea9e6/Dockerfile#L64-L66

Sorry, I'd just now realised collectstatic is not run in development and was coming here to correct myself. Apologies; it's hard to follow exactly what's going on.

Found this issue because I was wondering the same @nickjj

We need to build both sets of public files in a non-volume mounted directory but also make them available in a volume at runtime in such a way where existing files in the volume on the Docker host at runtime won't overwrite the newly built public files, this means we need to store both sets of public/ directories in 2 locations

Why is this the case? Why not just build static assets into /app/public (binded to assets/public on the host) for development, and serve from this folder in development? And into /app/public_collected for production?
I don't understand the overwriting files issue, can you please explain a little more? Are you saying because you don't want your production build assets to overwrite the dev build assets? If so, why does it matter? I would imagine that's only an issue if trying to test your production flow locally, but in that scenario does it really matter if we overwrite the files when they can just be recompiled/generated again? And in the case where the prod build is happening on a different machine for deployments you won't have dev files there to overwrite anyway. Let me know if I'm missing something. Thanks.

@nullbio You quoted one of my bullets but the first bullet is:

Django does not let you output collectstatic in the same directory where your source is, this means we need 2 sets of public/ directories, I named them accordingly

We can't collectstatic in public/ because Django prevents this. Now, that was 2 years ago and I'm not sure if anything changed since then but back then if assets got built into public/ then collectstatic couldn't output its files in public/, instead you had to use a separate directory and that's why I chose public_collected/.


(winscp) package failing validation

Expected Behavior
Package passing validation
Current Behavior
Package failing validation:
The ProjectSourceUrl element in the nuspec file should be a valid Url. Please correct this More...

Possible Solution
Fix ProjectSourceUrl or remove it.
Steps to Reproduce (for bugs)
see https://chocolatey.org/packages/winscp/5.17.2

This has been fixed here: https://github.com/chocolatey-community/chocolatey-coreteampackages/commit/4b46e8ebcb8ed9b897ea8ee113eed1dedfc43559
A new version of the package should be pushed out. @majkinetor @AdmiringWorm could one of you do that?

@mkevenaar are there any plans to fix the existing package versions that are failing validation?  Or should those be rejected?

I will reject them. We can always un reject them if required.

We don't generally do that here.

@mkevenaar you had typo here
[AU winscp\5.17.2 winscp.install\5.17.2 winscp.portalbe\5.17.2]

@majkinetor I added another commit to fix that just now.


When the registries are 0 (also for example when you don't have any data, only headers), specify another thing rather than Successfully. eg. Failed to convert...

Solved in commit 6149a96583a3c972389b6735b8be4592d43d0dcf


Pin weasyprint 58

See https://github.com/fdemmer/django-weasyprint/issues/71

Codecov Report

Merging #696 (94b7df5) into main (d06cfc7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #696   +/-   ##
=======================================
  Coverage   91.18%   91.18%
=======================================
  Files          24       24
  Lines        1066     1066
  Branches       55       55
=======================================
  Hits          972      972
  Misses         80       80
  Partials       14       14

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


[CAT-81] support bytestream uploads

Enable the ability to submit bytestreams for upload (and also check in retry improvements)

Can we also add an example for this

Example added


Update UserGuide.md for Track2Gather

This PR updates the UserGuide as per the draft written in our previous meeting.

This PR fixes https://github.com/AY2122S1-CS2103-W14-2/tp/issues/7.


scalyr inside docker for zookeeper.out

I like the idea of pushing zookeeper logs to scalyr, but I don't understand why do you want to install scalyr into docker...

We already have scalyr configured on EC2 instance.
What if one day we will decide to switch from scalyr to some other facility?

So instead of installing scalyr into docker it's much easier and faster just "tailf zookeeer.out". This way it will appear in a application.log and later would be imported by scalyr automatically.
Something like this:
https://github.com/zalando/exhibitor-appliance/pull/10/files

@drummerwolli have a look on that black magic solution for scalyr and logentries integration from @CyberDem0n

i also thought about just tailing the logs into the application logs. this would also make the hosts distinguishable, but i didn't like the idea to scram all kind of different logs together into one log in scalyr. i thought it would be nicer to identify which logs come from which service. imagine having more than two services/logs in one container ...
but sure, the other thing would work too, and is log provider independet, agreed.

Yes, this is nice discussion and very valid topics about the ability to differentiate the logs from different systems.
Maybe there is some nicer way to do it, like using syslogd or something like this to push logs to, that will be delivered to the EC2 level configured logging collector?

internally discussed to rather use tail -f, done with https://github.com/zalando/exhibitor-appliance/commit/c2cb8d8ca104f8e355f7274af4618fb61c4009c5 ... i will close this ...


Homebridge is suddenly coming with errors, used to work fine

Running Homebridge on my Raspberry Pi worked fine, but suddenly comes with an error after a reboot:
What could be wrong? Homebridge seems to be running when asked status, but accessories are not reachable.
Thanx for the help.
Cor
Snipping from log-file:
Homebridge is running on port 51826.
events.js:141
throw er; // Unhandled 'error' event
^
Error: listen EADDRINUSE :::51826
at Object.exports._errnoException (util.js:874:11)
at exports._exceptionWithHostPort (util.js:897:20)
at Server._listen2 (net.js:1234:14)
at listen (net.js:1270:10)
at Server.listen (net.js:1366:5)
at EventedHTTPServer.listen (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/util/eventedhttp.js:58:19)
at HAPServer.listen (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/HAPServer.js:132:20)
at Bridge.Accessory.publish (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/Accessory.js:347:16)
at Server._publish (/usr/local/lib/node_modules/homebridge/lib/server.js:50:16)
at Server. (/usr/local/lib/node_modules/homebridge/lib/server.js:229:14)
at /usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/util/once.js:15:19
at /usr/local/lib/node_modules/homebridge-nest/index.js:225:6
at /usr/local/lib/node_modules/homebridge-nest/node_modules/unofficial-nest-api/index.js:232:17
at IncomingMessage. (/usr/local/lib/node_modules/homebridge-nest/node_modules/unofficial-nest-api/index.js:214:25)
at emitNone (events.js:72:20)
at IncomingMessage.emit (events.js:166:7)
at endReadableNT (_stream_readable.js:903:12)
at doNTCallback2 (node.js:439:9)
at process._tickCallback (node.js:353:17)
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

That means there is something else using the port homebridge wants to use.
Most of the time this is due to accidentally having 2 different homebridge
process running.

AppleTechy: thank you for your reply.
I did a reboot several times, with no result. What can I do?

I solved it: it was an issue of IOS 9.3: I had to reinstall homekit to my iPhone.

Sorry for not responding, I didn't get email notifications till u had figured everything out already. Glad you got it working.


Can Markdown be used for rendering during the final output. Easy to read

Can Markdown be used for rendering during the final output. Easy to read

It should be like this:

Appreciate to AI to teach me how to write this code :). Welcome any suggestions.


show lost tabletIds which do not have any available replica

Add tabletIds without available replica in system info page, to improve the monitoring information.

Compile failed:
[ERROR] /home/palo-ci/teamcity/TeamCity/buildAgent/work/3e3628d3d06a368d/fe/src/main/java/org/apache/doris/catalog/Tablet.java:42:8: 无用导入 - java.util.Collections 。 [UnusedImports]

Done @morningman

HI @spaces-X  the FE code structure has been change, please rebase the code the solve the conflicts.

This is already supported #6045


Remove the 'external' toolchain file

The 'external' toolchain file does nothing but transitively sets
some cmake system variables from values received on the command
line, and forcibly sets the C compiler. However, the same cmake
system variables can be directly set via the command line, together
with the C compiler, and specifying a toolchain is not a must.
Thus, this patch drops the superfluous 'external' toolchain file
and updates cmake-based targets to invoke cmake in a simpler form.
Related changes in this commit:

While updating the cmake invocations, all the command line
arguments have been reviewed and simplified (removed those, which
did not change the defaults).
Removed unnecessary forced C compiler settings from some
toolchain files.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

Related to #2085, follow-up to #2088. The PR is speculative, needs confirmation from ESP8266, Mbed, Particle, RIOT, TizenRT, and Zephyr maintainers + from the original issue's reporter.

@dbatyai Good point. Updated.

I tested the PR, the compiler version is correctly identified, but now the problem is that the arm-none-eabi toolchain fails the working compiler test, due to an undefined reference to _exit. In this case the docs suggest to use the force compiler option. It seems we've come full circle.

Doesn't sound good. But let me ask one more experiment: what if we added -DCMAKE_C_COMPILER_WORKS=TRUE after -DCMAKE_C_COMPILER=...? (Or set(CMAKE_C_COMPILER_WORKS TRUE) in the toolchain files, depending on the target.) Would that still run compiler identification (to properly set version) but disable the checks that fail? (Because, if I see it right, these are two separate steps (the logs have separate lines for: "The C compiler identification is ..." / "Check for working C compiler: ...").
sources:

https://stackoverflow.com/questions/10599038/can-i-skip-cmake-compiler-tests-or-avoid-error-unrecognized-option-rdynamic/30642130#30642130
https://github.com/Kitware/CMake/blob/master/Modules/CMakeTestCCompiler.cmake

I checked, it seems setting CMAKE_C_COMPILER_WORKS solves the issue, the compiler is still identified correctly.

Thanks! I'll update the PR soon.

PR updated (sets CMAKE_C_COMPILER_WORKS for all non-OS-based targets).

I've tested this PR in ESP8266-12E and it works properly. Nice job!

Rebased onto master, which was a good thing to do as the new target build jobs have revealed that a CMAKE_C_COMPILER_WORKS define was missing for Zephyr. Commit fixed, PR updated.


Create CODEOWNERS file to streamline contribution process

A CODEOWNERS file needs to be established for this repository in order to streamline future contributions by the community.

Fixed by #6


allow to define remote repo

don't no why compiling the typescript gives such a big diff 🤷

Line endings probably

Hm the releaseconfig  approach is suboptimal, because it will have different remote name for every dev, should be a cli arg.

also not really sure why the first hypens are needed while testing I hadn't any issues, is it only a windows problem? So basically I tried to stick to your -c option for the config file.

AFAIK they are necessary for options with double dashes. Without the extra --, npm thinks the options are for itself, not the invoked command.


Blocking Uploading of specific File Types

[The content was pretty clear. I have a specific query though. Can this policy be imposed on some users and not be imposed on some users ? or is the policy applied on all users ]

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

ID: 04ad2f16-c861-7a1b-5c1d-e4601e8f3855
Version Independent ID: c47df4cd-f706-d807-5eac-172fba1e500f
Content: Use OneDrive policies to control sync settings - OneDrive
Content Source: OneDrive/use-group-policy.md
Service: one-drive
GitHub Login: @adeejoseph
Microsoft Alias: adjoseph

I know, ther is no policy "Blocking Uploading of specific File types"
I only know "Exclude specific kinds of files beeing uploaded", but this is a Computer Policy, not a user Policy


[help wanted] add docs for volume plugin

Ⅰ. Issue Description
We have implemented the volume plugin, and the volume plugin protocol follows the docker volume plugin protocol:
https://docs.docker.com/engine/extend/plugins_volume/#volumedriver
but the plugin discovery has changed. The pouch discovers the plugin by scanning the plugin directory. There are three kinds of plugin files

.unix  files is unix domain socket, which locate in the /run/pouch/plugins and /run/docker/plugins

.spec files contains the plugin URL, like unix:///run/pouch/plugins/local-persist.sock, which locate  in the /etc/pouch/plugins or /usr/lib/pouch/plugins

.json files contains the json specification for the plugin, which locate  in the /etc/pouch/plugins or /usr/lib/pouch/plugins

Please add the docs for the volume plugin, including the volume plugin protocol and plugin discovery. You can refer the docker docs
https://docs.docker.com/engine/extend/plugins_volume/#volumedriver
https://docs.docker.com/engine/extend/plugin_api/#plugin-discovery
Ⅱ. Describe what happened
Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign


Highlight blank line at the top of PHP file as a syntax error

Forgive me if this is the wrong place to file this bug, but I just spent an hour debugging a PHP program that wasn't working because I had a blank line at the top of the file. Is there a way ale can highlight that as a syntax error?

If php gives you a syntax error for the file, then that could help.
Could you paste the output of :ALEInfo when you have a blank line at the top of a file? There might be something from some program we can parse in there. Or maybe php itself will return some message we could parse, which isn't being captured at the moment.

Current Filetype: php
Available Linters: ['hack', 'langserver', 'php', 'phpcs', 'phpmd', 'phpstan']
  Enabled Linters: ['hack', 'langserver', 'php', 'phpcs', 'phpmd', 'phpstan']
 Linter Variables:
let g:ale_php_langserver_executable = 'php-language-server.php'
let g:ale_php_langserver_use_global = 0
let g:ale_php_phpcs_executable = 'phpcs'
let g:ale_php_phpcs_standard = ''
let g:ale_php_phpcs_use_global = 0
let g:ale_php_phpmd_ruleset = 'cleancode,codesize,controversial,design,naming,unusedcode'
let g:ale_php_phpstan_executable = 'phpstan'
let g:ale_php_phpstan_level = '4'
 Global Variables:
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%s'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_linter_aliases = {}
let g:ale_linters = {}
let g:ale_open_list = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_offset = 1000000
let g:ale_sign_warning = '--'
let g:ale_statusline_format = ['%d error(s)', '%d warning(s)', 'OK']
let g:ale_warn_about_trailing_whitespace = 1
  Command History:
(finished - exit code 0) ['/usr/bin/zsh', '-c', 'php -l -d error_reporting=E_ALL -d display_errors=1 -- < ''/tmp/vnjyLvR/71/db_manager.php''']
<<<OUTPUT STARTS>>>
No syntax errors detected in -
<<<OUTPUT ENDS>>>
(finished - exit code 0) ['/usr/bin/zsh', '-c', 'php -l -d error_reporting=E_ALL -d display_errors=1 -- < ''/tmp/vnjyLvR/72/db_manager.php''']
<<<OUTPUT STARTS>>>
No syntax errors detected in -
<<<OUTPUT ENDS>>>
(finished - exit code 0) ['/usr/bin/zsh', '-c', 'php -l -d error_reporting=E_ALL -d display_errors=1 -- < ''/tmp/vnjyLvR/73/db_manager.php''']
<<<OUTPUT STARTS>>>
No syntax errors detected in -
<<<OUTPUT ENDS>>>
(finished - exit code 0) ['/usr/bin/zsh', '-c', 'php -l -d error_reporting=E_ALL -d display_errors=1 -- < ''/tmp/vnjyLvR/74/db_manager.php''']
<<<OUTPUT STARTS>>>
No syntax errors detected in -
<<<OUTPUT ENDS>>>
(finished - exit code 0) ['/usr/bin/zsh', '-c', 'php -l -d error_reporting=E_ALL -d display_errors=1 -- < ''/tmp/vnjyLvR/75/db_manager.php''']
<<<OUTPUT STARTS>>>
No syntax errors detected in -
<<<OUTPUT ENDS>>>

Php isn't seeing the blank line as a syntax error.

Maybe there is a PHP linter out there which will complain about this.

Is there one you're already aware of that does that? I'll look around, but recommendations are always helpful.

I'm not aware of one myself, but I wouldn't be surprised if there is one.

Strange, for me the PHP linter does give an error about it:

It's up to the tools to report this as an error. Some tool might do that.


Can't issue restart command twice with Review branch

Hi. I'm running the review branch bot. Everything else is working fine so far. But I notice a bug that if I issued the restart command the first time after running the bot, it will restart just fine. But when I issued the restart command again the following error comes up.
!restart   File "run.py", line 314     def pyexec(pycom, *args, pycom2=None):                                   ^ SyntaxError: invalid syntax
I'm hosting the bot using Ubuntu server 16.04.1 VM.
PS: I've tried running the bot under Windows. The second time the restart command issued would cause the bot to stop running.

I can't reproduce this. Can you give some information about the environment you're running the bot on? (system, Python version, etc).

I've been able to reproduce this. Mine outputs the same error in the console after using !restart twice  File "run.py", line 314     def pyexec(pycom, *args, pycom2=None):                                   ^ SyntaxError: invalid syntax
Mine's running on a VPS with Ubuntu Server 16.04.2 and Python 3.5.2.

Can you put the code around three tick marks (```)? It seems that the ^ arrow is pointing to the position of the line that errors
Three tick marks such that

```
(code)
```

Sure thing. sorry about that, I've only recently started using github, didn't know how to do that haha.
File "run.py", line 314
    def pyexec(pycom, *args, pycom2=None):
                                  ^
SyntaxError: invalid syntax

I fixed this problem with some shell scripting
INPUT_STRING=run
while [ "$INPUT_STRING" != "shutdown" ]
do
	python3.5 run.py
	read -p "Type shutdown to quit" -t 5
done #If you want to check erros put the code below this line above this line.
read -p "Press enter to exit" nothing
echo "Hope I work next time"
sleep 5```
change the 5 in ```read -p "Type shutdown to quit" -t 5``` if you want more of a chance to stop it from auto-restarting. If you want to check erros and not auto-restart move ```read -p "Press enter to exit" nothing
echo "Hope I work next time"
sleep 5``` before ```done```

I too get this error after calling !restart twice in a row. Same error message as fofofofofofofofofoXD. I'm also running Ubuntu 16.04 and Python 3.5. I would really like to know how to solve this by improving the python code rather than a shell script

I may have fixed it by changing
def restart(*args):
    pyexec(sys.executable, *args, *sys.argv, pycom2='python')

to
def restart(*args):
    pyexec(sys.executable, *args, *sys.argv, pycom2='python3')

But this takes away from the modular approach I feel like this was going for.

Any word on when this will hit master? We're having the issue pretty consistently after the bot idles and needs to be restarted.

@Manifold0 The review branch is currently a release candidate. So very soon


Added function to retrieve player friends, added function to retrieve profile pictures, fixed some typos

Merging! Thanks 👍 👍 👍


detox api to select photo from photo library ios 11

Description
We've started using Detox for our E2E test which we felt good. But one of the problems that we face now is to select the photo from photo library in ios and for android as well. At first i tried to do for IOS, but got struck at the point when i find that the View hierarchy while debugging in Xcode doesn't return any Views for IOS 11. I happen to encounter the following link which explores the possibility, but didn't answer for IOS 11.
https://stackoverflow.com/questions/48208456/e2e-select-an-image-from-a-uiimagepickercontroller-with-wix-detox
And found these one as well,
https://stackoverflow.com/questions/29490728/how-to-use-uiautomation-to-select-image-from-uiimagepickercontroller?noredirect=1&lq=1
https://stackoverflow.com/questions/46404628/ios11-photo-library-access-is-possible-even-if-settings-are-set-to-never/46594741
I think, It looks like if we have access to PHPhotoLibrary, then our problem will be solved.
If you have any answer for this, Please advise.
Steps to Reproduce

Detox, Node, Device, Xcode and macOS Versions

Detox: 7.1.0
React Native: 0.53.2
Node: 8.9.1
Device: IOS simulator 6, 11.2
Xcode: 9.2
macOS:  10.13.3

Device and verbose Detox logs
--

As I explained in my comment and has been also mentioned elsewhere, UIImagePickerController runs in a different process, thus we cannot control it. PHPhotoLibrary is something completely different and has nothing to do with UIImagePickerController. If you use PHPhotoLibrary to build your own photo picker, you could make it work, but it doesn't help for UIImagePickerController, which most people use.
Your best bet is to mock the image picker API to just return some default image.

@LeoNatan
Thanks for your response. But i've couple of questions to clarify

If I'm mocking. how does that satisfy the condition in terms of flow to select an item from image library and uploading it to wherever i want
Can you pls provide some example to mock image Picker API for IOS?
Please provide some time before closing this ticket, as it didn't exactly resolve my problem.

I'm not sure how leaving it open would help in that case. It's an OS limitation which we cannot overcome, so you have too look at workarounds.
You mock only the image picker API, where your mock returns an image. You can then proceed to do the same logic as you normally do in your app, such as update your own UI or upload. I cannot provide examples, but for React Native apps, we have a section in the documentation, dedicated to mocking. Please see that.

@LeoNatan
Thanks for your quick response. What i meant was, the issue is under discussion and not sure if it is good to close. The reason why i asked was we have a solution working for IOS 10.2 version in one the links provided above. But now we can't use that as the OS version has been upgraded to 11 and the existing solution is not working due to system limitation. So I want to know if you guys are working anything on this?  If not, pls let me know. But anyway i'll give it a shot and get back to you guys in case i end up in trouble. Thanks!!!

As I have said before, there is little to nothing we can do. There are numerous things that Apple has moved out of process for user security reasons (otoh, Safari view controller, contact view controller, image picker view controller, etc.).
The only workaround (I understand it's not a full solution) is to mock this API and not use it when testing.


Patch gRPC-C++ podspec to work with Apple framework

Make gRPC-C++.podspec work with framework by using grpcpp as the framework name and explicitly excluding all files in include/grpc++/.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

This is neat!


The video files are gone  inside of nvidia.app.box.com

The video file https://nvidia.app.box.com/s/ezzw0js1ti555vbn3swsggvepcsh3x7e
mentioned in "perception_docker/README.md" is not available any more.
README.md  line 17
Download videos from https://nvidia.app.box.com/s/ezzw0js1ti555vbn3swsggvepcsh3x7e and place it in the 'videos' directory.

Yes. the same issue I have.

any update?

any update?

Hi @sujitbiswas ,
Any update on this issue?

Any updates on that? I also need the videos..


Discussion: Create a better search experience & knowledgebase

There's currently no way to search the documentation and this is essential for good DX/UX.
Fred: I'd love your thoughts on this one. You mentioned a knowledgebase/the ability to search through that KB so I wonder if a unified search is the answer? Does this knowledgebase live in the docs somewhere or is that a separate site? When you mentioned searching the KB, what sorts of things would get pulled into that search (docs, snippets, repos, blog posts, etc.)?

A basic search has been implemented on the site. It won't scale to thousands of documents very well but it's okay until we figure out how we want to do a unified search. I'll remove this from the launch milestone and add it to a later sprint.


Finish authorization sync job for god's sake

Current state of job requires manual launch & mapping renewal

[ ] Implement mapping renewal as a part of go code
[ ] Set up the whole thing as kubernetes cronjob and add to fleet
[ ] Manage secrets the right way
[ ] Create repo & write pipeline for building and pushing image to registry
Fix known bugs:
[ ] Diff works incorrectly
[ ] Vault doesn't overwrite entities when there is no change (?)

Lords Name in Vain :-1:


kaggle kernels output majedahalrwaily/crime-prediction-using-machine-learning -p /path/to/dest

Hi @devadathnu - are you having trouble with the API?  Please provide a description including detailed repro steps so someone can help.  Thanks!


fromJS fails with observableArray

I added two tests to demonstrate the failure.
Tested in FF 6 and Chrome.
Please contact me if you need more info.

@joefeser: I can confirm there's a bug and I'll be fixing it asap. Again, sorry for the delays.

@joefeser: Your unit test is failing because you did not create the original viewModel using 'ko.mapping.fromJS'. When doing the update, the mapping plugin assumes the second parameter you pass is either an object containing mapping options, or a previously mapped object.
In your case it is therefore using your viewModel as an options object. You can fix this by changing your 'ko.mapping.fromJS' to read:
ko.mapping.fromJS(newData, {}, viewModel);

I am just following the example here http://knockoutjs.com/documentation/plugins-mapping.html.
Are you saying the array now updates correctly if it is the third item as shown in your above example?

Yes, it would.

Bug in my code. issue never existed.


Warning message in console when placing Template

Foundry v11.304
PF2e v5.1.0
Walled Templates v0.6.2
When placing a spell template, I am seeing the following warning in the console.  Template still places and targets correctly.

This has been fixed since posted either by a system or Sequencer update.


Memory management and patterns in ASP.NET Core

Internal review URL  Fixes #15306
Migration from https://github.com/sebastienros/memoryleak/blob/master/README.md
Developers who understand their topic typically write in long sentences. Long sentences don't MT (machine translate). Long sentences are difficult for native speakers to comprehend.  Verify I didn't change the meaning when splitting up long sentences.
Please checkoff the following non-word-smithing changes:

[ ]  Changed links from

old respository https://github.com/aspnet/ResponseCaching/blob/c1cb7576a0b86e32aec990c22df29c780af29ca5
to current https://github.com/aspnet/AspNetCore/blob/v3.0.0/src/Middleware/ResponseCaching/src/Streams/StreamUtilities.cs#L16

[ ] Added: Is not a supported tool, however, it can be used to display memory usage patterns of ASP.NET Core apps.
[ ]

@Maoni0 please review.

@sebastienros  @davidfowl  can you approve or let me know what needs to be fixed?


[SPARK-24428][K8S] Fix unused code

What changes were proposed in this pull request?
Remove code that is misleading and is a leftover from a previous implementation.
How was this patch tested?
Manually.

Test build #91305 has finished for PR 21462 at commit 74e9011.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3706/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91305/
Test PASSed.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3573/

@foxish pls review.

jenkins, ok to test

Test build #91312 has started for PR 21462 at commit 74e9011.

Test build #91312 has finished for PR 21462 at commit 74e9011.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91312/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3709/
Test PASSed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3576/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3576/

Test build #91560 has started for PR 21462 at commit f8635fd.

Test build #91562 has started for PR 21462 at commit 9f1ddf3.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3851/
Test FAILed.

Test build #91560 has finished for PR 21462 at commit f8635fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #91562 has finished for PR 21462 at commit 9f1ddf3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91562/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3853/
Test PASSed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3708/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3708/

@felixcheung @mccheah pls review.

@foxish gentle ping.

@felixcheung gentle ping.

@felixcheung its local.

Thank you @foxish I was also thinking of adding the following:
if [ -n "$HADOOP_CONFIG_URL" ]; then
echo "Setting up hadoop config files...."
mkdir -p /etc/hadoop/conf
wget $HADOOP_CONFIG_URL/core-site.xml -P /etc/hadoop/conf
wget $HADOOP_CONFIG_URL/hdfs-site.xml -P /etc/hadoop/conf
export HADOOP_CONF_DIR=/etc/hadoop/conf
fi
I have already tested it with: org.apache.spark.examples.DFSReadWriteTest
At least we get some basic support for hdfs integration. Are you ok with adding that?

@skonto can we hold off on that until I merge Kerberos support? It is in the works :)

@foxish @felixcheung gentle ping for merge. I replied for mount paths.

ping @mccheah @liyinan926 on https://github.com/apache/spark/pull/21462#discussion_r197909535

LGTM, we can file a separate task to track what happened to secrets mounting without blocking on removing dead code.
Thanks for the fix @skonto.

Merging to master


Update MainApp Version to v1.2

Update version
Rename app

Coverage remained the same at 91.372% when pulling 8441fb1254f7d75aed81e6e102d7abd48ce03499 on pwenzhe:UI into ed1d0665cdbed4ecaeec427921cb6d94f6080fd7 on CS2103AUG2017-T16-B3:master.


Display correct statistics in front-end

Field names for statistics changed since the original implementation, this moves the values
to the correct implementation

LGTM ;)


Example Chip(Chest): Buy & Sell Blocks via Stablecoin

Token supply controlled by the chip. Players receive a single token for each block placed in the chest, and can take a block for each token deposited.

Specialized into settlers-union-bank


Made it work (for me). Beware, never touched Go before

As I never touched Go before I am not sure if that patch is correct. At least it make the bot "work for me".

Not sure if you're still interested in this, but if you want to get this merged, we'll at least need to remove the localized imports.


Migrating to a web-based user interface

I recently came stumbled upon this project
https://nicegui.io/
It allows to create applications based on web technologies, so it is cross-platform by definition and it is less dependent on the target platform when it comes to packaging and deployment. The code seems easier to develop and maintain than that based on PyQT5. It has also many features that would take a lot of work to implement in PyQT5 if one day one wants to do it (like e.g. displaying or interacting with data in the GUI directly).
It has also an integrated testing framework (see #45).
Maybe an idea for easyfermi v3 :)

This should solve also issues like #20 and #57.


Add ability to run non-instrumentation java tests from Android Studio

We need this to work under OS X


dbeaver/dbeaver#16972 fix sqlite recreate ddl

Closes #16972
Fixes issue with incorrect DDL. In the process was found that we have a pretty annoying dependency on the class position in the CHILD_TYPES enum, it may require large refactoring in the future.

Maybe it would be better to override this method for SQLiteTableColumn manager, but then we need to almost copy paste the code from SQLTableColumn manager. So, I'm not sure.

Maybe it would be better to override this method for SQLiteTableColumn manager, but then we need to almost copy paste the code from SQLTableColumn manager. So, I'm not sure.

Not sure, we only need to add extremely minor part to the query, it will be just excessive

These modifications are excessive. Unnecessary abstractions just to avoid copy-pasting only required parts on existing code.

Please create an SQLite-specific implementation of the ColumnModifier for the DEFAULT modifier that will include parentheses.


[3 of 4] Tiers - remove deprecated 'tier' columns

Tidies up the schema cruft left over from previous work.
NOTE: This must not be deployed to any server until this PR and this PR have been released OK.

Looks good! ☀


Warning redux branch

C:\react-universally>npm run development

><EMAIL_ADDRESS>development C:\react-universally
> babel-node tools/development

==> VENDORDLL -> No changes to existing "client" vendor dependencies. Using the existing vendor dll.
==> Creating webpack config for "client" in "development" mode
webpack building...
==> CLIENT -> Building new bundle...
webpack built f62abf9a195547487762 in 2963ms
==> CLIENT -> Running with latest changes.
==> Creating webpack config for "server" in "development" mode
==> SERVER -> Building new bundle...
==> SERVER -> Server running with latest changes.
Server listening on port 1337
==> SERVER -> Error in server execution, check the console for more info.
(node:13816) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): TypeError: Converting circular structure to JSON
==> SERVER -> Error in server execution, check the console for more info.
(node:13816) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

to get this error, go  to http://localhost:1337/posts/3000
Windows 10 x64
C:\react-universally>node -v
v7.4.0
C:\react-universally>npm -v
4.1.2

I thought that the problem solves by adding catch
function fetching(id) {
  return { type: 'FETCHING_POST', payload: id };
}

function fetched(post) {
  return { type: 'FETCHED_POST', payload: post };
}
function fetchFailure(error) {
  return { type: 'FETCH_POST_FAILURE', payload: error };
}

export function fetch(id) {
  return (dispatch, getState, { axios }) => {
    dispatch(fetching(id));

    return axios
      .get(`https://jsonplaceholder.typicode.com/posts/${id}`)
      .then(({ data }) => dispatch(fetched(data)))
      // We use 'react-jobs' to call our actions.  We don't want to return
      // the actual action to the 'react-jobs' withJob as it will cause
      // the data to be serialized into the react-jobs state by the server.
      // As we already have the state in the redux state tree, which is also
      // getting serialized by the server we will just return a simple "true"
      // here to indicate to react-jobs that all is well.
      .then(() => true)
      .catch((error) => dispatch(fetchFailure(error)));
  };
}

but it doesn't help.

Thanks for the report @Kimel1
I will see if I can recreate this, but it won't be until I land v13

Closing this as the redux branch will be overhauled soon.


Processor usage high when alert

When i'm alerted for go back to work, Pomotroid have a big usage iof cpu for one minute.
(about more than 20%)
i have a intel core i7700HQ @2.8ghz, 24g of ram

Not sure what this might be about. Does this spike in CPU usage happen if you have notifications enabled? Do you have this issue with any other Electron applications?

When i have just the gui of electron without notification activated, it take 5 to 11% of cpu !
(my firefox with 250tabs takes no more 5% when i type a text on github)
maybe it's because to electron, but i'm not developper, i didn't know electron before you ask me about it :)

My best guess is an issue with Electron. Closing due to inactivity.

This issue still exist, take 5% of cpu on idle