Remove unnecessary assert messages from testString in curriculum challenges

This issue will serve as a way to track all the work needed to remove the unnecessary assert messages from the yaml testString.  For all challenges which only have a text for each testString assert (most of the front-end challenges), we can simply remove the message argument, because the text is used for displaying both in the tests frame and in the fake console.
Any PRs created for this issue will only affect English curriculum files and only change two things:

Replace existing text with assert message argument value if the assert message gives a better test message and remove the assert message argument from the testString.
When both the text and the assert message argument values match, just remove the assert message argument from the testString`.

I have created a script which should capture all of the applicable tests.  I will create PRs for only the following sections which are known to not have multiple assert messages in the testString per single text tests.  I will limit the number of files per PR to make review easier on GitHub.
At the time of this issue's creation, there is only one open PR https://github.com/freeCodeCamp/freeCodeCamp/pull/35770 (waiting on final approval before being merged), which will need to have it's testString assert message arguments reviewed/removed.

Responsive Web Design - PRs:  #36405, #36406, #36407
JavaScript Algorithms And Data Structures - PRs:  #36401, #36402, #36403, #36404
Front End Libraries -  PRs: #36408, #36409
Data Visualization - PR #36424
Coding Interview Prep - PRs: #36410, #36412, #36415, #36416, #36417, #36418, #36419, #36420, #36421

Hey @RandellDawson I was just thinking about how best to review these.  The problem being that, with so many, human error is likely to creep in.
As such, would it be possible to group them so that all the challenges with solutions are in one lot of PRs and all those without are in another?  My thinking was just that we could focus our attention on the ones that're not being tested and the ones that are wouldn't need as much scrutiny.
What do you reckon?

Let me see how many challenges which are in these PR does actually do not have solutions (probably a lot of them in the Coding Interview Prep section).  If I update each PR body with a list of PRs without solutions, would that be enough?

@thecodingaviator I went back through and either made a new commit on an existing PR or created a new PR to remove the assert message arguments from the testString which either were surrounded by double quotes or two single quotes.  I think this should take care of 99.99% of the arguments in the curriculum sections which this issue aimed to deal with.

@RandellDawson I think we merge all the PRs in that case because looking for 0.01% of the errors would be like finding a needle in a haystack.
cc: @moT01 @ojeytonwilliams @Manish-Giri

I think the only ones worth looking at all are the ones without solutions.  If they have solutions npm run test will complain if the syntax is wrong or they suddenly start giving the wrong answers.

I think we merge all the PRs in that case because looking for 0.01% of the errors would be like finding a needle in a haystack.

It is not that they are errors.  It is just that they would not be removed (which does not hurt anything).

All PRs related to this issue have been merged now. It's up to us to decide whether to close the issue and keep removing message as we keep coming across them or let it be open.

In one of the PRs in this series, @ojeytonwilliams noticed that during one of my manual removals (not script based) of an assert message argument, I accidentally deleted the 2nd argument of the assert.equals statement.  It was caught by Travis and fixed.  This got us thinking about the fact that not all challenges have solutions and this series of PRs contained affected 181 challenges which do not have solutions and also have assert.equals variants,
I have compiled a list of all 181 challenges and the testStrings which contain an assert.equals variant.   Everyone should review this list of assert statements and if you find any where the 2nd argument was removed by mistake, a PR should be created to fix them.

@thecodingaviator I am not making any further mass update PRs which remove any more of the assert message arguments.  At this point, I think we will remove them if we come across any while making changes to challenges on future PRs.  The main mission (reducing the amount of code in the repo) has been accomplished.
This issue can be closed now.

I had a look at the gist and only found one thing: https://github.com/freeCodeCamp/freeCodeCamp/blob/master/curriculum/challenges/english/08-coding-interview-prep/rosetta-code/gamma-function.english.md still has assert messages.  Everything else looked fine, though I can't be 100% sure I haven't missed something.

@ojeytonwilliams Feel free to create a PR to remove the assert messages from that one challenge.

@ojeytonwilliams Are you ok with me closing this issue now?

@RandellDawson Sure, I think we're in a good enough state now.


[Designer-Publish Adaptive Card Schema]: On activating 'Set due date' control focus does not lands to the 'View' control in forward navigation under 'Publish Adaptive Card Schema' card.

https://microsoft.visualstudio.com/OS/_workitems/edit/37777989

This issue is not repro, hence closing this issue.


Please support iOS 8, I like the project, but I can't use.

Completed in b9e0a58f @meigesir. This should be released today or tomorrow.
Cheers. 🍻


Make NativeLibrary API work on reflection-free mode

In an attempt to solve networking issue https://github.com/dotnet/runtime/issues/33127
My previous PR didn't fully solve it: https://github.com/dotnet/corert/pull/8129
As explained by @MichalStrehovsky on gitter, few more switches, and one change were needed
Here is a sumary

2 new switches that will help with reflection-free mode

<AppContextSwitchOverrides Include="Switch.System.Reflection.Disabled.DoNotThrowForAttributes" />
<AppContextSwitchOverrides Include="Switch.System.Reflection.Disabled.DoNotThrowForGetDllImportSearchPathFlags" />

Made native NativeLibrary API to use reflection callbacks instead

All that make NativeLibrary  to finally work in reflection free mode, as long as the 3 new switches are used, therefore networking is finally working on that mode!

After testing with my own project, i hit this:
Unhandled Exception: EETypeRva:0x003A6140: Reflection_Disabled
StackTrace_AtWordInternal.Reflection.RuntimeTypeInfo.get_Assembly() + 0x6c
StackTrace_AtWordSystem.Net.NameResolutionPal.GetAddrInfoExSupportsOverlapped() + 0x27
StackTrace_AtWordSystem.Net.NameResolutionPal.<EnsureSocketsAreInitialized>g__InitializeSockets|4_0() + 0x60
StackTrace_AtWordSystem.Net.Dns.GetHostName() + 0x14
StackTrace_AtWordSystem.Net.Sockets.Socket.<InitializeSockets>g__InitializeSocketsCore|227_0() + 0x38
StackTrace_AtWordSystem.Net.Sockets.Socket..ctor(AddressFamily, SocketType, ProtocolType) + 0x43
StackTrace_AtWordArc.Shared.Networking.NetClient.netLoop() + 0xa9
StackTrace_AtWordSystem.Threading.Thread.StartThread(IntPtr) + 0x16d
StackTrace_AtWordSystem.Threading.Thread.ThreadEntryPoint(IntPtr) + 0x1a

I will need to do one more thing!

Oh no nevermind, i actually forgot to add the new switches haha :p
It work just fine :D :D

The approach with having assembly just return "I have no attributes" looks better than what I was proposing on gitter (it keeps everything local to the Reflection.Disabled assembly).
Do we need both mechanisms? Maybe the callback is no longer necessary if the old code can simply take the "no attributes" code path?

You are right, it's not needed anymore

Done!


Continue inventory collection on bios config err

What does this PR do
If bios config collection fails don't stop all inventory collection. Fixes https://github.com/metal-toolbox/alloy/issues/92
The HW vendor this change applies to (if applicable)
Supermicro, Asrockrack and Dell
The HW model number, product name this change applies to (if applicable)
N/A
The BMC firmware and/or BIOS versions that this change applies to (if applicable)
N/A
What version of tooling - vendor specific or opensource does this change depend on (if applicable)
N/A
How can this change be tested by a PR reviewer?
Run alloy-inband docker image on a target system

Thanks, the changes seem to work for inventory collection 👍 . Although the BIOS data collection utility on ASRR is missing a kernel module - we'd need this to be built into the inband image
{"component":"collector.inband","err":"error loading asrr bios kernel module: open /opt/asrr/asrdev-5.15.0-60-generic\n.ko: no such file or directory","level":"warning","msg":"error collecting server bios configuration","time":"2023-02-28T12:51:13Z"}


Create mod.xml

Uploading Menu+ V1.0 to catalog

Been switched to draft as @Digixu is still working on this and i don't want it to accidently get pushed.


Add function forkExistingRepoR

Closes: https://github.com/phadej/github/issues/280
Hacked together a little one-off test and it worked as expected.

Took the easy way out and just added a TODO.


New year!

Thank you! But keeping the first year of publication is evidently the right way to do things. (Yes, this conflicts with https://github.com/taptapship/wiredep/commit/92965089dea69307c070124d4cca68ff908c72f3 - I learned this after I did that  :))


Add an API key-based connection class

There should be a basic Octopus server connection class that takes in a server URL and credentials. This will generate a new Faraday base connection that can be used to make API requests in the future.
If we want to add a model class for testing, we should pick something simple and consistent between servers, like Environments. Projects are also a candidate, however I believe that model is slightly more complicated. Tenants are familiar, but they're an optional feature of the server, so they aren't as critical.

Why do we need a base connection class? The end user surely won't be instantiating it, so won't a Octopus Client class just be creating it behind the scenes when it is instantiated?
Basically, what's the added benefit to this being its own class, instead of just being a Faraday connection object?

For extensibility, it's possible that the end user will be instantiating it. I don't have any particular use cases in mind, but I don't see why they wouldn't be allowed to.
My preference for it being its own class that wraps the Faraday connection rather than just a Faraday connection itself is because just using a raw Faraday connection tightly couples the project to the Faraday connection API. Wrapping it allows us to define the interface to make requests.


scores_train/valid take different parameters than custom_scores

custom_scores are defined as
function(target, prediction)
while scores_train and scores_valid are defined as
function(predict, target)
I'm not sure if this was intentional, but it makes it somewhat confusing

I agree it's confusing and it's mostly an accident.  custom_scores was first introduced and uses the scikit-learn metrics interface (an example here), while scores_train and scores_valid use Lasagne's objectives interface where the arguments are predictions, target, so the other way around.


Invalid calculation of MINfloat16/MAXfloat16 in pandas/algos.pyx

The code generated by Cython from pandas/algos.pyx contains an undefined behavior (invalid cast of floating-point number to integer):
easy_install -U Cython
git clone git://github.com/pydata/pandas.git
python setup.py install

The generated code for the line from pandas/algos.pyx
cdef np.float16_t MINfloat16 = np.NINF

in pandas/algos.c contains:
typedef npy_float16 __pyx_t_5numpy_float16_t;
//... more code
__pyx_t_5numpy_float16_t __pyx_t_10;
//... more code
__pyx_t_10 = __pyx_PyFloat_AsFloat(__pyx_t_1);

The file numpy-1.9.2-py2.7-linux-x86_64.egg/numpy/core/include/numpy/npy_common.h
defines npy_float16 as 16-bit integer type (I guess numpy implementation has a custom code that carefully represents these 16 bits as mantiss+exponent). Casting floating-point value -inf to a 16-bit integer type is an undefined behavior according to C standard.
I wasn't able to find any place in pandas code that uses MINfloat16 (and MAXfloat16). Is it safe to just delete them?

can u show an example with the error?
the float16 constants are prob hanging around - as it's not an explicitly supported type (though u can actually pass an array in) - it's just that all of the algos will cast

Unfortunately, I see this error only if I compile generated .c files with clang -fsanitize=float-cast-overflow (http://clang.llvm.org/docs/UsersManual.html#controlling-code-generation). Looks like Cython directives are somewhat limited... Let me know if I should craft reproduction steps.

dupe of #10382


Unable to use Java 8 Time objects in RemoteApplication Events.

I want to use Java 8 Instant objects in my custom RemoteApplicationEvents. I have the jackson-datatype-jsr310 loaded. It fails when the event is processed locally. There is no way to customize the Jackson ObjectMapper in the BusJacksonAutoConfiguration. I demonstrate it with a small sample application: https://github.com/pmoerenhout/spring-cloud-bus-custom-events. Here I overwrite the BusJacksonAutoConfiguration and use a Jackson2ObjectMapperBuilderCustomizer.
Adding this.mapper.findAndRegisterModules(); will use the Jackson JSR-310 module. The other options will allow the timezone information in a ZonedDateTime to pass along. I can make a pull request if needed.

Duplicates #95


Remove ParentData#merge

There aren't any clients anymore.
Fixes #1684

@Hixie

LGTM to the merge removal. I don't think we should make the parent data classes private though. The objects are exposed, they're not internal implementation details.


[Frontend_Web_Development_HTML_CSS_JS] : CSS Classes and IDs

Description 📜

Added the documentation of CSS classes and IDs in the form of markdown file.
Added the separate sub-folder for screenshots, HTML and CSS files.

Fixes #951

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[x] Frontend Dev HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

/assign

/unassign

@code-reaper08 I have modified the file as suggested.
Thank you!

I don't know, but the image is showing below the content in my text-editor (VS-Code).

@2024-SANDHYA have a look here.

Okay, noted @2024-SANDHYA and @code-reaper08 , the alignment is the big issue here, I will work on this problem tonight.
Thank you so much for the review.🙌

I have worked on the alignment of the document @code-reaper08, @2024-SANDHYA. Kindly check.

html class and id selectors has already been merged

html class and id selectors has already been merged

I know that, I am not demanding to merge this PR, I was demanding to just look for the mistakes..... I am aware of the fact it is already merged.


Fix select/checkbox/radio with object

Using objects in select/checkbox/radio was completely broken. It always matched the first object. I removed some weird hacky stuff and used vue:s native v-model binding for those inputs.
Can you please release this and ssr fix soon, so I don't have to run a fork in production.

Good stuff. This will most likely close out the pull request submitted. #356
Will review when I get the chance

Thanks! Will also make a release.


[512] Expose inactive on status via ApplicationAttribute on the Vendor API (V1.6pre)

Context
We developing the next version of the API (1.6).
A draft version (1.6pre) will be passed by the vendors as a way to test the changes on sandbox before we officially release the new version.
Once the vendors have tested and are happy, we will make the release official, at which point we will update the API release notes and official documentation.
Changes proposed in this pull request

Expose inactive to the ApplicationAttribute statuses

Guidance to review

I've set application choice 16 to inactive on the review app.

Use the Vendor API on the review app or locally. In particular, check the status on ApplicationAttribute for an application choice which has a status of inactive.

Repeat for other statuses and ensure the response accurately reflects the database.

Alternatively you could inspect the MultipleApplicationsResponse on the draft API release document i've linked below.

Review the draft release documentation.

Adding a status to the Application will be classed as a breaking change 🫠

@dcyoung-dev see my comment here, this is not a breaking change.


Laravel 11 support

Support for Laravel 11

@tkaratug Any chance you can merge this ?

Hi @JVercout ,
Thank you so much for your contribution and for taking the time to open this PR.
I sincerely apologize for the delay in reviewing and merging your PR. Things got a bit busy on my end, but I’ve now had the chance to review your work, and it looks quite ok. I’m happy to merge this into the project.
Thank you again for your support and understanding.
Best regards,
@tkaratug


GpxMainObject class

I'm missing a class to hold all the information that I can pass in memory from place to place.
Currently the writer is getting a metadata object and three list of features (among other stuff).
It would be nice to have a interface call to the writer that receives an object that hold all the information.
Some thing like:
public class GpxMainObject {
    public GpxMetadata { get; }
    public GpxRoute[] {get;}
    ...

And the writer:
public void Write(..., GpxMainObject gpx, ...)

It makes sense to give, e.g., GpxRoute[] to the consumer from Read, since we can always guarantee that it'll be an array, which gives the consumer maximum flexibility for what they can do with it right away, at no performance penalty.  So yeah, it totally makes sense to have a Read kind of method that gives back a single object that encapsulates all the top-level gpx element content.
On the other hand, it doesn't make quite as much sense for Write to accept the same.  The provider might want to pass in any number of different sequence types that implement IEnumerable<T>, and so if they have, e.g., IEnumerable<GpxRoute> that's being created by a LINQ query, then it would be wasteful to force it to go through a .ToArray() just to meet the demands of the API.
If you think it would help usability significantly, then I wouldn't mind doing something like this:
public static void Write(XmlWriter writer, GpxWriterSettings settings, GpxMainObject gpx)
{
    Write(writer, settings, gpx.Metadata, gpx.Waypoints, gpx.Routes, gpx.Tracks, gpx.Extensions);
}

I never said any running about removing existing calls :-)
The above suggesting is great IMO.


⚠ Bofrost.es has degraded performance

In 21e4359, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1635 ms

Resolved: Bofrost.es performance has improved in a61d6e0 after 6 minutes.


"Hello World" is not mentioned in instructions.

Challenge Understand String Immutability has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

// Setup
var myStr = "Jello World";

// Only change code below this line

myStr = "H"; // Fix Me

@Harendra1993 thanks for the issue. I agree that the challenge is a bit vague but there are two points about this challenge. First, the challenge tests show you that you'll need "Hello World" in the string. Second, recently, this challenge has been updated to be more clear. You can check it out in the beta site. Happy coding!


Fix typo Larcaon to Laracon

Sorry, couldn't help it, my OCD got triggered when scrolling :-)
Awesome resource, btw, thank you.

Haha...thanks


Solearn Token Added with Logo

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Please check if you can approve my PR as well?
https://github.com/solana-labs/token-list/pull/4241

Please check if you can approve my PR as well? #4241

Thank you for the approval. I have looked at your commit and emailed your company through the contact form just now.

Thank you ! Could you please check again?
https://github.com/solana-labs/token-list/pull/4367/files

Thank you ! Could you please check again?
https://github.com/solana-labs/token-list/pull/4367/files

Please standardise the spacing of website, discord and twitter.
Could you approve this please: https://github.com/solana-labs/token-list/pull/4364


docs(readme): typo

Changes Unknown when pulling e65b2403aeb11cb34b4556b8cb6d18f534d048e4 on docs(readme)-typo into ** on master**.


Which release version do you support Swift 5 from ?

Sorry for asking something which has nothing to do with issues.
I just saw 1.2.2: fix warnings on Swift 5 but I'm not sure this means 1.2.2 started to support Swift 5.
I'd like to ask you which version starts to support Swift 5.
Thank you.

Not quite sure about it myself :P. The framework itself should build in Swift 5 (or with Swift 4 compatible mode) from any of the 1.x versions and 1.2.2 just fixed some warnings.
But I personally strongly suggest you choose version 2 if it is a new integration. The new version is much more stable and well-written compared to the original one.


Chris ruptures 101

bringing this across from its former home in [GNS-Science/nshm-nz-opensha
using flake8 to delint
tests live in package\test folder; run with nosetests

Hi @willic3  There are some rough edges, but this stuff is now being used downstream in our opensha experiments, so we should really get it into master next week. We can discuss with Andy on Tuesday.
If you have time to take a look at the tests in this branch e.g. src/eq_fault_geom/geomops/rupture_set/test/test_fault_section.py  they're using some of those techniques I talked about:

using the python unittest base classes (and a test runner e.g nosetests or pytest)
minimalist 'fixtures' where test data is required and/or
'faking' file IO using the io packages. or similar techniques for internet IO
testing positive and negative cases (e.g are exceptions raised)

OK, Chris.  I have some other stuff to get done today but I’m hoping to get to some of this stuff this afternoon.
Cheers,
Charles
Charles Williams I Geodynamic Modeler
GNS Science I Te Pῡ Ao
1 Fairway Drive, Avalon 5010, PO Box 30368, Lower Hutt 5040, New Zealand
Ph 0064-4-570-4566 I Mob 0064-22-350-7326 I Fax 0064-4-570-4600
http://www.gns.cri.nz/ I Email: C.Williams<EMAIL_ADDRESS>On 14/08/2020, at 10:47 AM, Chris B Chamberlain <notifications<EMAIL_ADDRESS>wrote:
Hi @willic3https://github.com/willic3 There are some rough edges, but this stuff is now being used downstream in our opensha experiments, so we should really get it into master next week. We can discuss with Andy on Tuesday.
If you have time to take a look at the tests in this branch e.g. src/eq_fault_geom/geomops/rupture_set/test/test_fault_section.py https://github.com/GNS-Science/eq-fault-geom/pull/7/files#diff-54a1796827b3cdfa709dd9904ed4655b they're using some of those techniques I talked about:

using the python unittest base classes (and a test runner e.g nosetests or pytest)
minimalist 'fixtures' where test data is required and/or
'faking' file IO using the io packages. or similar techniques for internet IO
testing positive and negative cases (e.g are exceptions raised)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/GNS-Science/eq-fault-geom/pull/7#issuecomment-673747035, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABJHOVLKAJXWBPPSTBBY56TSARUQVANCNFSM4PVALQ3A.
Notice: This email and any attachments are confidential and may not be used, published or redistributed without the prior written consent of the Institute of Geological and Nuclear Sciences Limited (GNS Science). If received in error please destroy and immediately notify GNS Science. Do not copy or disclose the contents.

Hey thanks @andy22b  - I'm going to temporarily restore that deleted branch, until I can revise my visualisation stuff to use master...


'QMServicesManager.h' file not found

After latest pod update, i am getting this error
'QMServicesManager.h' file not found
In a swift project and in an objective-c library, i use
#import "QMServicesManager.h"
after latest pod update i started using use_frameworks! but now i get this error
'QMServicesManager.h' file not found
Any idea ?

Hi,
Thank you for reporting the issue. We are working on fix and will post update as soon as possible.

@UtkuDalmaz I was able to fix the issue. We are planning to make a release later this week.

Check out our latest version of QMServices (0.4.4) and QMChatViewController (0.3.10). We have fixed all issues with frameworks pods. Check our updated samples with use_frameworks! pods flag on installation tips: https://github.com/QuickBlox/quickblox-ios-sdk

Still having the problem even though i update the latest

Hey guys any idea on this bug?

Have you tried importing using '<>' ?

it says

Please show your podfile content.

platform :ios, “9.0”
use_frameworks!
target “Jemiyet” do
pod 'SDWebImage'
pod 'SVProgressHUD'
pod 'TWMessageBarManager'
pod 'QMServices'
pod 'Fabric'
pod 'Crashlytics'
pod 'Bolts'
pod 'FBSDKCoreKit'
pod 'FBSDKLoginKit'
pod 'ALCameraViewController', '1.2.5'
pod 'Agrume'
pod 'SCLAlertView', :git => 'https://github.com/vikmeup/SCLAlertView-Swift'
pod 'SwiftyJSON'
pod 'ImageScrollView'
end
post_install do |installer|
installer.pods_project.targets.each do |target|
target.build_configurations.each do |config|
config.build_settings['SWIFT_VERSION'] = '3.0'
end
end
end

Import QMServices like this:
#import "QMServices.h"

Checkout our sample-chat projects to see more in-depth usage.

Doesn't work man, it says : 'QMServices.h' file not found
This happened after the framework update :(

Have you checked our samples projects?

#import <QMServices/QMServicesManager.h> is fixed the problem

And what were you doing before?

I am having Warning
/Users/utkudalmaz/Developer/Jemiyet/Pods/QMServices/QMBaseService/QMBaseService/QMBaseService.h:23:1: Conflicting nullability specifier on return types, 'nullable' conflicts with existing specifier 'nonnull'
just to inform you


opae-c: Add closedir call in build_error_list

build_error_list function calls opendir to walk the sysfs tree looking for
error register sysfs nodes but never closes the directory.
This adds a call to closedir to close the directory fd.

I ran this through valgrind memory check and it shows 2 additional issues with error.c. Are these real issues?
3,552 bytes in 6 blocks are still reachable in loss record 2 of 3
at 0x4C27C23: malloc (vg_replace_malloc.c:299)
by 0x507008C: build_error_list (error.c:286)
by 0x50706FE: build_error_list (error.c:276)
by 0x509092E: token_add (token_list.c:93)
by 0x506C3E3: fpgaEnumerate (enum.c:727)
by 0x4E50F81: common_test::getAllTokens(void**, fpga_objtype, int, unsigned char*) (in /home/lab/workspace/itbarraz/opae/fpga-internal/tests/build/libcommonlib.so)
by 0x4E5132E: common_test::BaseFixture::TestAllFPGA(fpga_objtype, bool, std::function<void ()>, unsigned char*) (in /home/lab/workspace/itbarraz/opae/fpga-internal/tests/build/libcommonlib.so)
by 0x627653: StressLibopaecMmioFCommonHW_04_Test::TestBody() (in /home/lab/workspace/itbarraz/opae/fpga-internal/tests/build/gtapi)
by 0x57644EF: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (in /usr/local/lib/libgtest.so)
by 0x575E141: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (in /usr/local/lib/libgtest.so)
by 0x574326E: testing::Test::Run() (in /usr/local/lib/libgtest.so)
by 0x5743B15: testing::TestInfo::Run() (in /usr/local/lib/libgtest.so)
9,472 bytes in 16 blocks are still reachable in loss record 3 of 3
at 0x4C27C23: malloc (vg_replace_malloc.c:299)
by 0x507008C: build_error_list (error.c:286)
by 0x509092E: token_add (token_list.c:93)
by 0x506C3E3: fpgaEnumerate (enum.c:727)
by 0x4E50F81: common_test::getAllTokens(void**, fpga_objtype, int, unsigned char*) (in /home/lab/workspace/itbarraz/opae/fpga-internal/tests/build/libcommonlib.so)
by 0x4E5132E: common_test::BaseFixture::TestAllFPGA(fpga_objtype, bool, std::function<void ()>, unsigned char*) (in /home/lab/workspace/itbarraz/opae/fpga-internal/tests/build/libcommonlib.so)
by 0x627653: StressLibopaecMmioFCommonHW_04_Test::TestBody() (in /home/lab/workspace/itbarraz/opae/fpga-internal/tests/build/gtapi)
by 0x57644EF: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (in /usr/local/lib/libgtest.so)
by 0x575E141: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (in /usr/local/lib/libgtest.so)
by 0x574326E: testing::Test::Run() (in /usr/local/lib/libgtest.so)
by 0x5743B15: testing::TestInfo::Run() (in /usr/local/lib/libgtest.so)
by 0x57441D9: testing::TestCase::Run() (in /usr/local/lib/libgtest.so)
LEAK SUMMARY:
definitely lost: 0 bytes in 0 blocks
indirectly lost: 0 bytes in 0 blocks
possibly lost: 0 bytes in 0 blocks
still reachable: 15,168 bytes in 26 blocks
suppressed: 0 bytes in 0 blocks

It is a memory leak as token_cleanup is currently disabled.
Should be fixed once we re-enable this function and add it to a library destructor.
https://github.com/OPAE/opae-sdk/blob/e669a34c7ae42460eca3e5e4e46cfdbaaaf619d2/libopae/src/token_list.c#L189-L211


bugfix: 播报当前步骤获取错误

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wheelwang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Where to download the 3d detection label

I cannot find where to download the 3D label here: https://www.boreas.utias.utoronto.ca/#/download.
Could you please provide me with the link?

Have you seen this?
https://github.com/utiasASRL/pyboreas/blob/master/download.md

Thanks. Got it.


Issue federation ID tokens with group claim as an array

What happened?

@enj and I were working on https://github.com/vmware-tanzu/pinniped/pull/322.
We noticed that we issue downstream ID tokens with a groups claim formatted either as an array or a string: https://github.com/vmware-tanzu/pinniped/blob/3f08f2e11e5c0599984f16d6b15c5bc2f364ef98/internal/oidc/callback/callback_handler.go#L275-L276.
We should be consistent about the data type for our downstream groups claim.

What did you expect to happen?

We should be consistent about the data type we use for our downstream groups claim.
We should always use an array.

What is the simplest way to reproduce this behavior?
In what environment did you see this bug?

Pinniped server version: v0.3.0
Pinniped client version: n/a
Pinniped container image (if using a public container image): n/a
Pinniped configuration (what IDP(s) are you using? what downstream credential minting mechanisms are you using?): n/a
Kubernetes version (use kubectl version): n/a
Kubernetes installer & version (e.g., kubeadm version): n/a
Cloud provider or hardware configuration: n/a
OS (e.g: cat /etc/os-release): n/a
Kernel (e.g. uname -a): n/a
Others: n/a

What else is there to know about this bug?

We should also try to cast groupsAsInterface to an []interface{} in the case that the upstream token has been unmarshaled that way.

I'm wondering how many possible scenarios we want to cover with this change.
The groups claim could be anything so it seems difficult to anticipate every possible format we might get from the upstream token.
Currently in our tests we have the following examples:

slice of strings
A string separated by spaces (e.g. "group1 group2")
An integer (results in unprocessable entity)

It seems reasonable to try to parse 2. into a slice, but are there are other possible formats we should be worried about?

I'm wondering how many possible scenarios we want to cover with this change.
The groups claim could be anything so it seems difficult to anticipate every possible format we might get from the upstream token.
Currently in our tests we have the following examples:

slice of strings
A string separated by spaces (e.g. "group1 group2")
An integer (results in unprocessable entity)

It seems reasonable to try to parse 2. into a slice, but are there are other possible formats we should be worried about?

I do not think we should support anything more than string and []string from the upstream IDP group claim.  We should try to parse the input as []interface simply because all the elements may still be a string, i.e. []interface{"group1", "group2"} should work.
For the downstream token, we should guarantee that the groups claim is always a []string.  @mattmoyer seemed to be leaning towards [] over omitting the claim when the user has no groups which I think is reasonable.

I do not think we should support anything more than string and []string from the upstream IDP group claim.  We should try to parse the input as []interface simply because all the elements may still be a string, i.e. []interface{"group1", "group2"} should work.
For the downstream token, we should guarantee that the groups claim is always a []string.  @mattmoyer seemed to be leaning towards [] over omitting the claim when the user has no groups which I think is reasonable.


how to solve it???

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

@ruchiroffical Please visit the Help Room if you need assistance. Happy Coding!


[Website] Align the website design with Figma

Describe your idea
Detox website's Figma: https://www.figma.com/file/QGhhB7djq0OxvLzULhs7u5/Detox

[ ] [Website] Create a kitchen sink page for testing global CSS changes

[ ] example


when would the project be finished?

Hi,
Thanks for implementing UnDeepVO. I wonder when could this system be finished?
Thanks

Please see: #3


type 'ReactIterableAPI' does not conform to protocol 'IterableAuthDelegate'

We're using the React Native SDK and our latest builds are failing with the following error:
❌  (node_modules/@iterable/react-native-sdk/ios/RNIterableAPI/ReactIterableAPI.swift:651:1)

  649 | }
  650 |
> 651 | extension ReactIterableAPI: IterableAuthDelegate {
      | ^ type 'ReactIterableAPI' does not conform to protocol 'IterableAuthDelegate'
  652 |     func onAuthTokenRequested(completion: @escaping AuthTokenRetrievalHandler) {
  653 |         ITBInfo()
  654 |

Perhaps a regression from an earlier issue: https://github.com/Iterable/swift-sdk/issues/616

Why havent we seen this error ?

Thanks for reporting @drewminns!
Would you share Iterable's RN SDK version used and which version was working for your app before?

We're also hitting this, we've fixed by patching the podspec file in the RN SDK:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 7906f59..74506c7 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -24,6 +24,6 @@ Pod::Spec.new do |s|

   s.swift_version = '5.3'

-  s.dependency 'Iterable-iOS-SDK', '~> 6.5.3'
+  s.dependency 'Iterable-iOS-SDK', '6.5.4'
   s.dependency 'React-Core'
 end


frontend: Improve Map performance

This fixes a stack overflow crash #2534 and improves grouping and filtering performance
There's still a bunch of stuff that can be optimized so this is only first step
Testing done:

[x] Tested on KWOK cluster with 20,000 pods

After fixes:
Map loading time: 5s
Filter by error: <1s
Group by node: <1s
Before:
Map loading time: ~3.5min
Filter by error: 50s
Group by node: 9s

Very nice.
Did you look into how it reacts with a fairly high frequency of change? I mean with pods/events being added, rather than with a static set of resources.
(If not, probably something to look at in another PR.)

Did you look into how it reacts with a fairly high frequency of change? I mean with pods/events being added, rather than with a static set of resources. (If not, probably something to look at in another PR.)

Yeah it struggles a bit with frequent changes, but that is another issue

Doh. I think I broke the test plugins CI job with the release of the new headlamp-k8s/eslint-config. It is trying to upgrade to the "latest" headlamp-k8s/eslint-config, where it should be getting the version related to that version of headlamp-plugin.
So you can ignore that test plugins job failure in this PR.

Got it, thanks for the explanation


ALZ - Review of links v2

changing the more info link for a recommendation that contains capacity planning in the content: https://learn.microsoft.com/azure/well-architected/performance-efficiency/capacity-planning

@hgotar1 - I'm pointing this into a working branch to consolidate the three active PRs into one so we can check between them together.

Will proceed with a shorter pull request


KAFKA-8240: Fix NPE in Source.equals()

backport of PR #6589 to 2.2 branch

Call for review @bbejeck @cadonna

SUCCESS
10533 tests run, 5 skipped, 0 failed.
--none--

Java8 never started or never finished... (is in "yellow" pending state for half a day) Java11 passed.
Retest this please.

Failures seem unrelated.
Retest this, please

Merged to 2.2 and cherry-picked to 2.1 branch.


Migration Marketplace PoC to branch

The development of the marketplace has been started as a PoC in https://github.com/HanSolo/Marketplace
We should first of all move the PoC to a branch of this repo to integrate it in the page. Based on this we can start the development of the marketplace.

Adding @gdams and @smlambert
Will ping Gerrit by mail.

This is complete, closing


refactor: unify clients configuration

There are multiple places where we use old client (plugin, sdkv2, tests), and multiple constructors to build it. That adds unnecessary complexity, and complicates the new client integration.

Replaces multiple client constructors with a single approach
Unifies old and new client configuration

tests


Incorrect scope visibility of renaming

If we have few loops with same iterator name, when rename iterator in any of loops we got renaming in all loops, but should rename locally in target loop.
For example we rename first "i1"
var sprites:Array<Sprite> = [];
for (i1 in sprites)
{
    trace(i1);
}

var ints:Array<Sprite> = [];
for (i1 in ints)
{
    trace(i1);
}

and got
var sprites:Array<Sprite> = [];
for (i2 in sprites)
{
    trace(i2);
}

var ints:Array<Sprite> = [];
for (i2 in ints)
{
    trace(i2);
}

but should be renamed only one
var sprites:Array<Sprite> = [];
for (i2 in sprites)
{
    trace(i2);
}

var ints:Array<Sprite> = [];
for (i1 in ints)
{
    trace(i1);
}

Currently FlashDevelop doesn't discern between block level variables when looking for references in Haxe.

Is it possible to fix this?

With enough time it is, of course.

It's in the "really minor annoyances" category for me.

Fixed via #1430

I wouldn't say this issue is fully fixed as it depends on Haxe 3.2+ (although I'm tempted to say 3.4+...). I know some people depend still on previous versions.


Updates billing mode for managed DynamoDB lock table to PAY_PER_REQUEST

Note to other maintainers: See https://github.com/gruntwork-io/terragrunt/pull/934#discussion_r341253107 for further context and confirmation that PAY_PER_REQUEST billing is available in all active regions.


pull request for e2e test at 01/14/2017 16:56:11

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


Bookmark warning

I created a first draft of it. We can obv work on the wording. Let me know what you think about it. My idea was, that it pushes down the Darkmode Switch and  AirSwap Logo, but after being dismissed, those elements return to their normal position.
Also created an alternative grey version which looks a bit more subtle.


Provide API for pip show info by invoking pip as a subprocess (and offer to contribute)

I recently needed to read information from a requirements file and a frozen constraints file (which I did using pip-api and works wonderfully — thanks) and then walk the tree of dependencies to identify installed packages that are directly or implicitly required by those in the requirements file.
Walking through the dependency hierarchy requires information from pip show, so I wrote code to obtain that information by running pip show in a subprocess and parsing its textual output. I think that capability would make a nice addition to the pip-api package.
Is there interest in having me contribute that functionality?
Any preferences regarding what the Python API should look like for this?

Hi @stevecj, thanks for the feature request. TL;DR: yes.
The purpose of this package to providing a drop-in replacement for existing uses of pip's internal API by wrapping command-line calls to pip. It looks like a function that replaces the pip._internal.commands.show.search_packages_info would be the function we'd want to replace by wrapping pip show.
This internal function takes a list of package names like ['six'] and yields a dictionary like:
{
    "name": "six",
    "version": "1.14.0",
    "location": "/Users/dustiningram/.pyenv/versions/3.8.2/lib/python3.8/site-packages",
    "requires": [],
    "required_by": [
        "virtualenv",
        "uncurl",
        "twilio",
        "tweepy",
        "tox",
        "readme-renderer",
        "python-frontmatter",
        "pip-tools",
        "bleach",
    ],
    "installer": "pip",
    "metadata-version": "2.1",
    "summary": "Python 2 and 3 compatibility utilities",
    "home-page": "https://github.com/benjaminp/six",
    "author": "Benjamin Peterson",
    "author-email"<EMAIL_ADDRESS>    "license": "MIT",
    "classifiers": [
        "Development Status :: 5 - Production/Stable",
        "Programming Language :: Python :: 2",
        "Programming Language :: Python :: 3",
        "Intended Audience :: Developers",
        "License :: OSI Approved :: MIT License",
        "Topic :: Software Development :: Libraries",
        "Topic :: Utilities",
    ],
    "files": [
        "__pycache__/six.cpython-38.pyc",
        "six-1.14.0.dist-info/INSTALLER",
        "six-1.14.0.dist-info/LICENSE",
        "six-1.14.0.dist-info/METADATA",
        "six-1.14.0.dist-info/RECORD",
        "six-1.14.0.dist-info/WHEEL",
        "six-1.14.0.dist-info/top_level.txt",
        "six.py",
    ],
}

It looks like we can get all of this if using the following:
$ pip show -f --verbose six
Name: six
Version: 1.14.0
Summary: Python 2 and 3 compatibility utilities
Home-page: https://github.com/benjaminp/six
Author: Benjamin Peterson
Author-email<EMAIL_ADDRESS>License: MIT
Location: /Users/dustiningram/.pyenv/versions/3.8.2/lib/python3.8/site-packages
Requires:
Required-by: virtualenv, uncurl, twilio, tweepy, tox, readme-renderer, python-frontmatter, pip-tools, bleach
Metadata-Version: 2.1
Installer: pip
Classifiers:
  Development Status :: 5 - Production/Stable
  Programming Language :: Python :: 2
  Programming Language :: Python :: 3
  Intended Audience :: Developers
  License :: OSI Approved :: MIT License
  Topic :: Software Development :: Libraries
  Topic :: Utilities
Entry-points:
Files:
  __pycache__/six.cpython-38.pyc
  six-1.14.0.dist-info/INSTALLER
  six-1.14.0.dist-info/LICENSE
  six-1.14.0.dist-info/METADATA
  six-1.14.0.dist-info/RECORD
  six-1.14.0.dist-info/WHEEL
  six-1.14.0.dist-info/top_level.txt
  six.py

Any preferences regarding what the Python API should look like for this?

I think this should mimic search_packages_info as much as possible. I'd imagine the docs for this would look something like:

pip_api.search_packages_info(List[str]: query) -> Iterator[Dict[str, src]]

Takes a list of package names. Yields a dict for each matching package with the following fields:

name (string): The name of the installed distribution
...

Acknowledged. 🙂


[Tree] Improve performance of inserting many items at a time into a tree (nested set)

When inserting a large amount of records, performance on the nested set extensions pretty much tanks.  Understandably, storing large trees inside SQL is going to prove difficult.
I've tried approaching bulk processing like Doctrine suggests, getting a "batch" of objects persisted, then flushing and assembling another batch.  This make for a speedy initial insert, but then it appears that almost for each inserted record, the nested tree code kicks in and runs multiple queries per each item inserted.  What I see a lot of time being spent on is updating things like the right and left values for the tree traversal.
I'm not seeing any documentation that shows how to best handle large inserts.  It's possible that it just is not suggested and the library is not build to handle such use cases.
Is there any way to improve the performance of large inserts?  These large inserts can be governed by a little bit of criteria too (for example, we are adding a whole new section of a tree, we are not adding these large amounts of items randomly in multiple areas of an existing tree).

I guess as an example of what I am thinking about, is it possible to set the tree fields via code and just insert them without triggering the tree extensions.
For example, we do a heavy amount of processing to these items before they go into the system.  Using known information and the structure of the data, we can probably pre-set the level, left, and right values manually and thus don't need to run individual queries to set this information.
If this tree gets inserted as a "sub tree" of another, we may need to run some bulk queries to update the level, left, and right values of other affected nodes via the newly inserted records, but even these would only be a handful of queries I think compared to the hundreds upon hundreds that are currently being run by fixing all the items individually.

Hi, extensions are not simple and made not having performance oriented architecture. They are made for easy usage and rapid development. If at some point you need better performance, your best option is to first of all not to use ORM, because everything around it is slow. Another option:

is to read about nested set trees it is really simple
build your nested set tree in memory (with correct lft rgt values)
then insert all your records using DBAL (for even better performance).

It just would not be wise to even try to provide something like that in extension library, it always boils down to something custom.
If there is not enough memory (means you are persisting other tree node fields with huge data related, nullify these and update later).
In general, I should not be answering these kind of questions here, use stackoverflow if you cannot find a solution to such problem.


Uploads over 7MB keep failing/timing out.

{
  "name": "StorageError",
  "message": "The specified blob or block content is invalid.\nRequestId:a6e20757-c01e-00c8-0f3f-b457fa000000\nTime:2019-01-24T23:54:05.6783332Z",
  "stack": "StorageError: The specified blob or block content is invalid.\nRequestId:a6e20757-c01e-00c8-0f3f-b457fa000000\nTime:2019-01-24T23:54:05.6783332Z\n    at Function.StorageServiceClient._normalizeError (/private/var/folders/_y/v50r58vd4897phjb3nrc_7qm0000gp/T/MicrosoftAzureStorageExplorer 2/Microsoft Azure Storage Explorer.app/Contents/Resources/app/node_modules/azure-storage/lib/common/services/storageserviceclient.js:1198:23)\n    at Object.StorageServiceClient._processResponse (/private/var/folders/_y/v50r58vd4897phjb3nrc_7qm0000gp/T/MicrosoftAzureStorageExplorer 2/Microsoft Azure Storage Explorer.app/Contents/Resources/app/node_modules/azure-storage/lib/common/services/storageserviceclient.js:744:50)\n    at Request.processResponseCallback [as _callback] (/private/var/folders/_y/v50r58vd4897phjb3nrc_7qm0000gp/T/MicrosoftAzureStorageExplorer 2/Microsoft Azure Storage Explorer.app/Contents/Resources/app/node_modules/azure-storage/lib/common/services/storageserviceclient.js:317:37)\n    at Request.self.callback (/private/var/folders/_y/v50r58vd4897phjb3nrc_7qm0000gp/T/MicrosoftAzureStorageExplorer 2/Microsoft Azure Storage Explorer.app/Contents/Resources/app/node_modules/request/request.js:185:22)\n    at emitTwo (events.js:126:13)\n    at Request.emit (events.js:214:7)\n    at Request.<anonymous> (/private/var/folders/_y/v50r58vd4897phjb3nrc_7qm0000gp/T/MicrosoftAzureStorageExplorer 2/Microsoft Azure Storage Explorer.app/Contents/Resources/app/node_modules/request/request.js:1157:10)\n    at emitOne (events.js:116:13)\n    at Request.emit (events.js:211:7)\n    at IncomingMessage.<anonymous> (/private/var/folders/_y/v50r58vd4897phjb3nrc_7qm0000gp/T/MicrosoftAzureStorageExplorer 2/Microsoft Azure Storage Explorer.app/Contents/Resources/app/node_modules/request/request.js:1079:12)",
  "code": "InvalidBlobOrBlock",
  "statusCode": 400,
  "requestId": "a6e20757-c01e-00c8-0f3f-b457fa000000"
}

Closing due to lack of activity. Please comment/reopen if you have more info to share.


Improve support dApp Providers (Bitcoin Provider) (Round 3)

Describe the task
Improve support dApp Providers (Bitcoin Provider) (Round 3)

[ ]  Inscribe (text, images)
[ ] Send inscription
[ ] Update more detailed information of input - output when perform Sign PSBT
Refer from Unisat :

[ ] Categorize information displaying transaction history when perform sign psbt

[ ] Improve UI in case account isn't in the wallet

[ ] Handle case input amount = 0 when perform signTransfer
Actual : Transaction failed

Expect :

[ ] Validate case input amount = 0

[ ]  Improve UI on Transaction history screen :

Disable Transaction hash field
Disable button View on explorer

For OP_RETURN:


Update CoFH Mods

Crashes, http://pastebin.ubuntu.com/11401488/.
As this didn't happen before, I'm going to guess this was caused by updating Forge.

Invalid, switched to NOVA


steps to setup development environment

how to setup the development environment to customized or extend functionality.
Please guide the steps after cloning this repo with some example to change the type script.
So that people can contribute and raise the pull request for bugs and importants.
Development steps and guidelines, pull request standards.
How to setup package folder contains core, formula and react to start and open basic sheet by importing modules, it will help.

To get the basic sheet, just see Development.
If you need local backend storage or want the edit sheet collabrately, you need a mongodb. see backend

@gudipudipradeep Are you working on this project? In this project, packages are managed by Lerna

https://assets.ruilisi.com/W0DUU83Beyfouo=CNLx5VA==
This is my screenshot for you to start the project from git clone to editing sheet

@gudipudipradeep
See this screen shot.
I will show you how to start the project from git clone to edit in sheet.

@jikoczl thanks


Allow silent curl

If you add the -s parameter to curl, you can prevent that the webhook endpoint gets leaked by stdout.
With this PR you'll be able to enable that silent mode by setting silent to true in your workflow.

I created my own release in the meantime. If you want to merge this and release the feature in your own action, I'll reverse my release.

Hi. Thanks for the contribution. I am busy with some big migrations at work, which should conclude over the weekend some time. I'll have a look at these as soon as I have a bit of time, either by end of the weekend or early next week.


[bazel] Convert tock tests to the new rules

Allow the opentitan_test rule to accept a binary that will be
used as the test firmware rather than compiling code from sources.
This can be used to supply a pre-compiled test binary or compile a
binary with rules for a different language (e.g. Rust).
Convert the tock basic_test to use the new rules.

Fixes #19932.


Improve GH workflows/actions

Codecov Report

Merging #29 (d02e032) into develop (aa0a1ef) will not change coverage.
The diff coverage is n/a.

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

Additional details and impacted files
@@             Coverage Diff             @@
##             develop       #29   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity         2         2
===========================================
  Files              1         1
  Lines              4         4
===========================================
  Hits               4         4

Flag
Coverage Δ

functional-tests
100.00% <ø> (ø)

php-8.2
100.00% <ø> (ø)

unit-tests
50.00% <ø> (ø)

Components
Coverage Δ

php-jsonrpc/poc-monorepo-pkg1
100.00% <ø> (ø)

macrorepo
∅ <ø> (∅)


fix rate-limiter compare-lock-compare logic issue

fix invalid condition check; update span after startTime updated

resend after pull


Use calibration with a different RGBD camera

I tried to use your calibration tool with a different RGBD camera, as it is in my point of view actually the best way to calibrate intrinsic and extrinsic parameters with ROS. I remapped all topics to fit, like shown in the rqt_graph attached.
Unfortunately the CV Images do not show up. After a little debugging, I found out, that the callback function of the Subscriber is not called anyway. Any idea why not?

Hi,
the calibration tool is listening to the compressed images, maybe you are not proving them. But you can change the calibration tool to use the raw images. Go to line 154 and change compressed to raw and check if it works.

I am already one step further :D
All compressed images are provided and I checked them all through rviz. Therefore I did change line 154 already. That's the reason why it rqt_graph is showing actually the link from the raw images to the calibration tool. Never the less I added a terminal output in the first line of the callback function, but it is never reached.

Then it could be the synchronizer. Right now ExactTime is used, but if the timestamps of depth and color are not identical, it won't work. Try to change it to ApproximateTime. It is in line 88.

After republishing the Images with a new Timestamp this works for me, thx! Now the IR image is doubled and therefore shrinked to half in width in the CV image viewer, but that seems to be some encoding problems.


Make pkg/config/remote a Go module

What does this PR do?
Make pkg/config/remote a Go module. To make it possible, pkg/util/grpc also
had to be converted into a module.

Motivation
Making it a proper module makes it easier to be reused in other projects.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Is it possible to run the e2e tests before merging?


Resource names not converted to correct format

According to the Data Resource specifications https://specs.frictionlessdata.io/data-resource/#name a resource name:

MUST consist only of lowercase alphanumeric characters plus “.”, “-” and “_”.

Currently, if you provide a CKAN dataset to  ckan_to_frictionless.py with a resource not following the above rule, the name will be written as it is, containing any character and not in lower case.
There are two options to solve the issue:

Just raise an error to the user informing about the problem
Convert the name to a valid one, removing invalid characters and putting in lowercase.

What do you think? @amercader

I think we should follow the spec here and parse incoming CKAN resource names to make them adhere to the spec


How to use diffusers with IPAdapterFaceID and ip_adapter-plus-face together? Thank you! I hope to get your reply.

How to use diffusers with IPAdapterFaceID and ip_adapter-plus-face together? Thank you! I hope to get your reply.

currently, you can use https://github.com/cubiq/ComfyUI_IPAdapter_plus/issues/145#issuecomment-1868261202

@cubiq do you will support it on your diffusers repo

I am very rough to implement multipath ipadater in diffusers, if you need you can contact me.  https://github.com/Daming-TF/Diffusers-For-Multi-IPAdapter


Starlark fn interprets arguments as numbers when possible and reformats them

Expected behavior
Arguments from config map (potentially coming from command line when running imperatively) are passed as is to Starlark code.
Actual behavior
The arguments are interpreted as numbers when this is possible and gets passed this way to the Starlark code. The problem we have run into is that short random hex strings with hexadecimal characters are interpreted as scientific notation, e.g. "9e5" becomes 90000, and then when the fn code inserts that into some resource it gets converted back to a string but as "90000".
Information
Kpt version: v1.0.0-beta.4 (same problem if using v0.39.2)
Steps to reproduce the behavior
$ kpt fn eval . --results-dir=. --image gcr.io/kpt-fn/starlark:v0.2.2 -- \
  source='print(ctx.resource_list["functionConfig"]["data"]["thing"])' thing='9e5'
[RUNNING] "gcr.io/kpt-fn/starlark:v0.2.2"
[PASS] "gcr.io/kpt-fn/starlark:v0.2.2"

For complete results, see results.yaml
$ cat results.yaml
apiVersion: kpt.dev/v1
kind: FunctionResultList
metadata:
  name: fnresults
exitCode: 0
items:
  - image: gcr.io/kpt-fn/starlark:v0.2.2
    stderr: |
      900000.0
    exitCode: 0

Ping @mengqiy since you seem to be working with the Starlark fn

Thanks for reporting the issue! I will look into it.


ValueError("Paths don't have the same drive

Running the demo in the README, error output:
  File "C:\Program Files\Python310\Lib\site-packages\nicegui_toolkit\layout_tool\services\intercept_service.py", line 129, in get_frame_with_file_name
    if _Helper.is_descendant_of(file_path, folder) and (
  File "C:\Program Files\Python310\Lib\site-packages\nicegui_toolkit\layout_tool\services\intercept_service.py", line 114, in is_descendant_of
    common_path = os.path.commonpath([directory, file])
  File "C:\Program Files\Python310\lib\ntpath.py", line 778, in commonpath
    raise ValueError("Paths don't have the same drive")
ValueError: Paths don't have the same drive

When the drive where NiceGUI is located is different from the drive where the program code file is located, an error will occur. This will be fixed in the next version.


Added setter for R and L HeadLightBroken

confusing name for last commit, but setting StartScrollOffset to 0 in a menu means it will begin scrolling at the edge of the items displayed on screen, setting it to 1 and it starts 1 above/below the edge etc


migrating examples to react 18

@tannerlinsley trouble understanding how current changes causing the pr failures. can you help?


Synthesis failed for Google.Cloud.Asset.V1

Hello! Autosynth couldn't regenerate Google.Cloud.Asset.V1. :broken_heart:
Here's the output from running synth.py:
Cloning into 'working_repo'...
Switched to a new branch 'autosynth-Google.Cloud.Asset.V1'
Running synthtool
['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-03-12 08:35:14,286 synthtool > Executing /tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.Asset.V1/synth.py.
Cloning into '/home/kbuilder/.cache/synthtool/gapic-generator-csharp'...

Welcome to .NET Core 3.1!
---------------------
SDK Version: 3.1.102

----------------
Explore documentation: https://aka.ms/dotnet-docs
Report issues and find source on GitHub: https://github.com/dotnet/core
Find out what's new: https://aka.ms/dotnet-whats-new
Learn about the installed HTTPS developer cert: https://aka.ms/aspnet-core-https
Use 'dotnet --help' to see available commands or visit: https://aka.ms/dotnet-cli-docs
Write your first app: https://aka.ms/first-net-core-app
--------------------------------------------------------------------------------------
curl: (35) gnutls_handshake() failed: Error in the pull function.
2020-03-12 08:36:45,721 synthtool > Failed executing /bin/bash generateapis.sh --check_compatibility Google.Cloud.Asset.V1:

None
2020-03-12 08:36:45,722 synthtool > Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.Asset.V1/synth.py", line 20, in <module>
    hide_output = False)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '('/bin/bash', 'generateapis.sh', '--check_compatibility', 'Google.Cloud.Asset.V1')' returned non-zero exit status 1.
Synthesis failed

Google internal developers can see the full log here.

Suspect this is just a transient network error. Other APIs seem to be fine. Closing - we'll see what happens tomorrow.


Need to click twice to select item from slide menu in iOS app

Ionic version:
Ionic Framework Version: 3.0.1
When I click on item from ionic slide menu, it doesn't work on first click
It works on second and sometimes on third click.
The same application is working fine in android
Current behavior:
Slide menu item's click work on second time or third time
Expected behavior:
It should work on first click in iOS app as well
Related code:
In app.html code
<ion-menu id="loggedInMenu" [content]="content">
<ion-item detail-push menuClose (click)="next()">
willy.dee
<ion-item detail-push class="item-height" menuClose *ngFor="let p of loggedInPages" (click)="openPage(p)">
{{p.title}}
</ion-list>

In app.component
export class MyApp {
@ViewChild(Nav) nav: Nav;
rootPage:any = SystemIdentifierPage;
loggedInPages: Array<{ title: string, icon: string, component: any }>;
constructor(platform: Platform, statusBar: StatusBar, splashScreen: SplashScreen,
public menu: MenuController) {
platform.ready().then(() => {
// Okay, so the platform is ready and our plugins are available.
// Here you can do any higher level native things you might need.
statusBar.styleDefault();
splashScreen.hide();
});
this.loggedInPages = [
{ title: 'Home', icon: 'home', component: Home },
{ title: 'Message', icon: 'mail', component: MessagePage },
{ title: 'Entries', icon: 'paper', component: EntriesPage },
{ title: 'Pending Entries', icon: 'document', component: PendingEntriesPage },
{ title: 'Employees', icon: 'people', component: EmployeesPage },
{ title: 'Clients', icon: 'person', component: ClientList },
{ title: 'Search', icon: 'search', component: MySearch },
{ title: 'Logout', icon: 'log-out', component: LoginPage }
];
this.menu.enable(true, 'loggedInMenu');
}

@Gurjit-ONEBCG For elements that are not clickable there is a delay of 300ms for the click event to file.
You will need to make it as a button and add the ion-item property to it.
Refer to this code:
<ion-menu>
  <ion-list>
    <button ion-item (click)="openPage1(p)">Item 1</button>
    <button ion-item (click)="openPage2(p)">Item2</button>
    <button ion-item (click)="openPage3(p)">Item3</button>
  </ion-list>
</ion-menu>

For more info refer to this link: http://ionicframework.com/docs/troubleshooting/#click-delays

Hello @Gurjit-ONEBCG , could you try @chandanch solution and see if that solves the issue for you? Thanks!

Thanks  worked in my case. But can you explain that if there is delay of 300 ms before the click event fire, then why it would not fired on first click even after the delay of 300 ms. But yes it fired when I clicked on second time after the delay of 300 ms

Thanks < button  > tag worked for me

It should probably be mentioned more clearly in the documentation, especially on the menu component that adding the tappable attribute fixes this.  It's too easy to miss in the docs.

Thanks all! @daveshirman ill talk to the docs people about this.


Switch to row instead of trajectory

I think this is a change in a good direction and I'd take it one step further and use "rows" for the row dimension name rather than "traj". IIRC "traj" came from some earlier convention, perhaps a CF-conventions documentation or something like that. I think it's good to use an existing convention if it exists but it should be evaluated against whether it's actually a good choice and whether it's consistent here within CloudDrift.

We could have the default has rows, and set it to traj for the GDP and other datasets we are creating.
Note that I haven't look at the code to see how much work would that required.

We could have the default has rows and set it to traj for the GDP and other datasets we are creating.
Note that I haven't look at the code to see how much work would that required.

Yes, I suggested defaulting to "rows". Datasets can still have any dimension name they want. The change for that should only be in ragged.subset (and corresponding tests) I think.

Yes, I suggested defaulting to "rows". Datasets can still have any dimension name they want. The change for that should only be in ragged.subset (and corresponding tests) I think.

We have it hardcoded here too raggedarray.

Yes, but I think raggedarray.py is only used by GDP adapters.

Actually, @kevinsantana11, also uses it with the Hurdat2 adapter.

I'll resolve the conflicts later.

@kevinsantana11 could you help with this? Not sure how to proceed with the changes to raggedarray.py. What I wanted to add is a mapping GDP_DIMS = {"rows": "traj", "obs": "obs"}, so we can use this to change the name of the dimensions and don't have to write: "traj" or "obs" in the gdp adapters.
But I now see that we have:
DimNames = Literal["traj", "obs"]

@dataclass
class Dim:
    name: DimNames
    size: int

class RaggedArray:
    def __init__(
        self,
        coord_dims: list[tuple[str, Dim]],
        ....

in the raggedarray module..

@philippemiron taking a look into this now. The change I had made to the ragged-array data structure got rid of the ability to modify "traj", "obs" dimensions. I like the changes here though so I'm going to submit a PR to your repo.

Thanks

@philippemiron looks like I should have rebased and then made changes on top of yours rather than merging main into your branch. I'll need to take a look at this later today as some git surgery will be necessary.
I think my approach will be to remove the most recent merge from main, squash all commits here into 1 commit and then rebase main. This way when I rebase, all of the changes are made on-top of the most recent commits and so when fixing the merge conflicts I'll just need to keep the current changes. I'm going to test this out separately in my own branch first before pushing into here. Just wanted to give you a heads up.

I remove the conflict but there is still work to be done. We have two things that are conflicting at the moment.
I had the name_dims = {"rows": "name_use_as_alias", "obs": "name_use_as_alias"}, with a default to simply name_dims = {"rows": "rows", "obs": "obs}, and for example for GDP name_dims = {"rows": "traj", "obs": "obs}.
In raggedarray.py, this was written in the opposite way: name_dims = {"name_use_as_alias": "rows", "name_use_as_alias": "obs"}. I think it makes more sense how I had it, but I might be missing something.
We still need to have something for the coordinates that maps each coordinate to a dimension. And this part could be confusing too.. We could only map to rows and obs here, but then the dimension could be renamed if we supply something to name_dims.
Comments?

I remove the conflict but there is still work to be done. We have two things causing issues.
I had the name_dims = {"rows": "name_use_as_alias", "obs": "name_use_as_alias"}, with a default set to name_dims = {"rows": "rows", "obs": "obs}, and for example for GDP name_dims = {"rows": "traj", "obs": "obs} to rename the "rows" dimension.
In raggedarray.py, this was written in the opposite way: name_dims = {"name_use_as_alias": "rows", "name_use_as_alias": "obs"}. I think it makes more sense how I had it, but I might be missing something.
We still need to have something for the coordinates that maps each coordinate to a dimension. And this part could be confusing too.. We could only map to rows and obs here, but then the dimension could be renamed if we supply something to name_dims.
Comments?

I mainly reversed the relationship as it made certain parts of the code a bit easier to deal with.
In relation to mapping coordinates to dimension, it seems when converting from xarray we can 'detect' the (coord, dim) mapping ref. The current approach is to look at the coordinates dimension and pick the last one.
When it comes to awkward arrays, since their data structure doesn't provide such information we need that to be provided from the user. I think it could become a source of error/confusion for users to have to create maps of alias -> dim and then coord -> alias where they could mistakenly enter the dim instead and it wouldn't be until runtime that they get an error for this.
We could try to catch this preemptively and throw a descriptive error

I think it's good now.

Ok, added my review with a couple of edits and a suggestion above. @kevinsantana11 please review also.

@selipot I'm working on adding the doctest to atleast run for our ragged module, have most of the changes done just have a couple more changes to be made.

We agreed this would be for another PR


Multiple TaggableManager in single model resulting in  Cannot resolve keyword u'None' into field.

Note : Currently using 0.11.1 as 0.11.2 does not support multiple TaggableManager objects in the same model.
environment
$ python --version
Python 2.6.6
$ pip freeze
Django==1.6.1
django-taggit==0.11.1

models.py
from django.db import models

from taggit.managers import TaggableManager
from taggit.models import GenericTaggedItemBase, TagBase

class SecondTag(TagBase):
    pass

class SecondTaggedItem(GenericTaggedItemBase):
    tag = models.ForeignKey(SecondTag)

class MyModel(models.Model):
    first_tag = TaggableManager()
    second_tag = TaggableManager(through=SecondTaggedItem)

Create an save a new MyModel object, then attempt to iterate through second_tag.
>>> a = MyModel()
>>> a.first_tag = 'one,two'
>>> a.second_tag = 'three,four'
>>> a.save()

>>> q = MyModel.objects.get(pk=1)
>>> for f in q.second_tag.iterator(): f
...
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/manager.py", line 181, in iterator
    return self.get_queryset().iterator(*args, **kwargs)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/taggit/managers.py", line 335, in get_query_set
    return self.through.tags_for(self.model, self.instance)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/taggit/models.py", line 153, in tags_for
    return cls.tag_model().objects.filter(**kwargs).distinct()
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/manager.py", line 163, in filter
    return self.get_queryset().filter(*args, **kwargs)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/query.py", line 590, in filter
    return self._filter_or_exclude(False, *args, **kwargs)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/query.py", line 608, in _filter_or_exclude
    clone.query.add_q(Q(*args, **kwargs))
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/sql/query.py", line 1198, in add_q
    clause = self._add_q(where_part, used_aliases)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/sql/query.py", line 1232, in _add_q
    current_negated=current_negated)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/sql/query.py", line 1100, in build_filter
    allow_explicit_fk=True)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/sql/query.py", line 1351, in setup_joins
    names, opts, allow_many, allow_explicit_fk)
  File "/home/jesse/code/blah/ve/lib/python2.6/site-packages/django/db/models/sql/query.py", line 1274, in names_to_path
    "Choices are: %s" % (name, ", ".join(available)))
FieldError: Cannot resolve keyword u'None' into field. Choices are: id, mymodel, name, secondtaggeditem, slug

This still doesn't seem to work =(...

@jesseward Look at this, this helped me solve this: https://groups.google.com/forum/#!topic/django-taggit/5qncw-FpDHQ

TODO:

confirm if still valid
add some test casing on this


Blazor dotnet watch run in devcontainer crashes extension host after saving file

Issue Type: Bug
I am trying to work on my website in a dev container and every time I save a razor file, even if I don't make any changes, VSCode tells me the window has disconnected and I need to reload it. Opening the developer tools shows me the extension host has experienced multiple errors and then says the container is no longer running.
console.ts:137 [Extension Host] Error: stream ended with:4 but wanted:9    at u (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57708)    at c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57889    at s (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5302)    at Socket.<anonymous> (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5472)    at Socket.emit (events.js:327:22)    at endReadableNT (internal/streams/readable.js:1327:12)    at processTicksAndRejections (internal/process/task_queues.js:80:21)
S @ console.ts:137
console.ts:137 [Extension Host] Exception thrown by PacketStream substream end handler Error: stream ended with:4 but wanted:9
    at u (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57708)
    at c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57889
    at s (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5302)
    at Socket.<anonymous> (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5472)
    at Socket.emit (events.js:327:22)
    at endReadableNT (internal/streams/readable.js:1327:12)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)
S @ console.ts:137
console.ts:137 [Extension Host] Error: stream ended with:4 but wanted:9    at u (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57708)    at c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57889    at s (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5302)    at Socket.<anonymous> (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5472)    at Socket.emit (events.js:327:22)    at endReadableNT (internal/streams/readable.js:1327:12)    at processTicksAndRejections (internal/process/task_queues.js:80:21)
S @ console.ts:137
console.ts:137 [Extension Host] Exception thrown by PacketStream substream end handler Error: stream ended with:4 but wanted:9
    at u (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57708)
    at c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57889
    at s (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5302)
    at Socket.<anonymous> (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5472)
    at Socket.emit (events.js:327:22)
    at endReadableNT (internal/streams/readable.js:1327:12)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)
S @ console.ts:137
console.ts:137 [Extension Host] Error: stream ended with:4 but wanted:9    at u (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57708)    at c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:7:57889    at s (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5302)    at Socket.<anonymous> (c:\Users\logan.dam\.vscode\extensions\ms-vscode-remote.remote-containers-0.191.1\dist\extension\extension.js:12:5472)    at Socket.emit (events.js:327:22)    at endReadableNT (internal/streams/readable.js:1327:12)    at processTicksAndRejections (internal/process/task_queues.js:80:21)
S @ console.ts:137
log.ts:301   ERR [remote-connection][Management   ][6e7b6…][reconnect] A RemoteAuthorityResolverError occurred while trying to reconnect. Will give up now! Error:
log.ts:301   ERR Error: Container is no longer running.
    at br.resolveAuthority (extensionHostManager.ts:289)
    at processTicksAndRejections (internal/process/task_queues.js:93)
    at async tl._resolveAuthorityAgain (extensionService.ts:319)
abstractExtensionService.ts:615 Extension host terminated unexpectedly. Code:  0  Signal:  null
_onExtensionHostCrashed @ abstractExtensionService.ts:615
log.ts:301   ERR [remote-connection][ExtensionHost][a0f21…][reconnect] A RemoteAuthorityResolverError occurred while trying to reconnect. Will give up now! Error:
log.ts:301   ERR Error: Container is no longer running.
    at br.resolveAuthority (extensionHostManager.ts:289)
    at processTicksAndRejections (internal/process/task_queues.js:93)
    at async tl._resolveAuthorityAgain (extensionService.ts:319)
console.ts:137 [Extension Host] Shell server terminated (code: 137, signal: null)

Steps to reproduce:

Clone my repo (https://github.com/biltongza/ldam.co.za) and open Header.razor in client/Shared
Open a terminal, cd to client/ and run dotnet watch run
Save the razor file

Expected results: the file saves properly.
Actual results: VSCode window crashes.
VS Code version: Code 1.59.1 (3866c3553be8b268c8a7f8c0482c0c0177aa8bfa, 2021-08-19T11:56:46.957Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No
Fetching remote diagnostics for 'Dev Container: C# (.NET)' failed: Canceled

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.82GB (16.62GB free)

Process Argv
--folder-uri vscode-remote://dev-container%2B633a5c6465765c6c64616d2e636f2e7a61/workspaces/ldam.co.za --crash-reporter-id c2eb1839-92da-4408-96c4-14d4c8c4a2a3

Screen Reader
no

VM
0%

Fetching remote diagnostics for 'Dev Container: C# (.NET)' failed: Canceled
Extensions: none
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspyt653:30270858
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
pythonvspyt639:30300192
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
vscextlang:30333561
binariesv615:30325510
vsccppwt:30329788
pythonvssor306:30344512
bridge0708:30335490
pygetstartedt2:30351387
vstre464cf:30350173
bridge0723:30353136
vsdyn321cf:30354108

Probably an extension issue, however first forwarding to @stuartleeks and feel free close as "caused by extension"


Add internal duk_remove_m2() helper for footprint

Add a helper for duk_remove(ctx, -2) which occurs enough many times internally to benefit from a helper. Footprint saved is around 100 bytes.

x64 defsize is -150 bytes.

This is a much more common idiom than one would intuitively guess.  I find myself doing duk_remove(ctx, -2) a lot too.

The other values in the code base are: 0, -3, -4, and a variable idx_args.
The -2 is related to the idiom of replacing the stack top with another value:
if (!duk_is_string(ctx, -1)) {
    duk_push_string(ctx, "replacement");
    duk_remove(ctx, -2);
}

Maybe there's potential for same aptly named public API function?

Sometimes footprint work is unintuitive: the change to use packed arguments for internal property helper calls and the duk_remove(ctx, -2) shorthand together account for close to 1kB of footprint improvement:

What those changes basically do is remove one argument from some commonly used calls affecting maybe 150-200 call sites. That's enough to gain about 1kB, much more easily than by e.g. optimizing specific built-in objects.
So I'll hunt for common call patterns a bit in the internals, and there may be opportunities for public API calls there too. Code policy check should also point out that a certain idiom (say, duk_remove(ctx, -2)) has a footprint-preferred variant because they're somewhat unintuitive and hard to remember.


[codemod][new] aurelia.js - migration to 2

Migration details
[Link to upgrade guide]

[ ] codemod-enhanceapi - [Codemod Studio Link]
[ ] codemod-attribute - [Codemod Studio Link]
[ ] codemod-compose- [Codemod Studio Link]

To contributors working on this task:

[ ] Framework Expert:  For each codemod use case in this migration, provide the Codemod Studio link (example) with detailed specifications. Include before/after pairs, transformation logic in comments, and edge cases. Even if the transformation is tricky, provide instructions on detecting the patterns. This helps estimate the effort needed and guides developers to the areas requiring manual changes.

[ ] Codemod Creator: Build and publish quality codemods that handle most edge cases. False negatives are acceptable, but there should be no false positives. Even if the transformation is tricky, build a codemod that can detect and add comments to guide developers to the areas needing manual changes.

Questions? -> Community

Closing as stale.


CI break: lintrunner is timing out

MYPY is timing out for some reason. Need investigation. Temporarily disabled in https://github.com/microsoft/onnx-script/pull/525

Exclude generated files and function_libs


Handle connection errors

Hi,
This is a very nice project which I had been trying to use for my project. However, occasionally I hit the following error.
 ('Connection aborted.', ConnectionResetError(54, 'Connection reset by peer'))

Usually, something like this can be handled by backoff, for example, in the Request class from urllib3. Would including this be interesting to you? Happy to contribute towards this.

Hey,
I currently don't have the time to implement this, but am happy to accept a PR :)
I'm not 100% sure how a good way of implementing would be, maybe a variable with a default on the profile class(es).
Just out of interest: With which profile did the error occur?

With which profile did the error occur?

This is with the DBProfile.

I'm not 100% sure how a good way of implementing would be, maybe a variable with a default on the profile class(es).

I haven't looked at detail but I would think an optional parameter with retries and backoff_factor would need to be set somewhere and all requests calls should use this.

Added a PR for this.
https://github.com/FahrplanDatenGarten/pyhafas/pull/31

Now solved in #31, thanks to you!


⚠ DDoS-Relay lin-US-W-lax-01 has degraded performance

In 5645e99, DDoS-Relay lin-US-W-lax-01 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 147 ms

Resolved: DDoS-Relay lin-US-W-lax-01 performance has improved in 299b144 after 20 minutes.


displaying html on the screen via LAN

Do you think it is possible to write an application that will transmit the image over IP to a given screen / monitor?

This project is just a protocol, not framework.


Add getArticleByPath route

I didn't see a route for this API route, maybe it was added recently? https://docs.forem.com/api/#operation/getArticleByPath
Are you open to PRs or have some time to add?

Hi! Good catch. If you could create PR for it, it would be great!


Automated Database Seed for Cypress Testing

Additional Context

In order to support Cypress, additional dev work is required to seed the database for required data.
The final goal is to have a QA DB seeding solution that can be executed in the QA environment directly or in the future in a docker environment.

Acceptance Criteria

[ ] Do research to look for possible best practices on Cypress guidelines or in any other approach used to seed the database;
[ ] Create a solution to execute the DB scripts needed for seeding the data in a test case isolated way. For instance, for a particular test case, the code structure should be easy to understand and must consolidate all the DB changes needed to execute the particular test case.
[ ] Implement one data seed using one of the test scenarios already in place, for instance, a designation agreement.
[ ] Explore how we can run altogether from docker-compose, similar to what is happening in local-api-test that is also running on GitHub actions.

Initial thoughts

Create a new backend app and consume the @sims-db library;
Create service-like classes to execute the DB commands similar to the other services in API/Workers;
Execute all the service methods to seed the DB in a parallel way.

As part of my research, I found the below links
https://learn.cypress.io/advanced-cypress-concepts/database-initialization-and-seeding
which basically list almost all the approach we discussed.
Custom scripts (factories) to generate "test" data are something similar that we are planning to do as part of this ticket with Faker.js.
Below are the methods in cypress with which we execute or run our scrips
https://docs.cypress.io/faq/questions/using-cypress-faq#How-do-I-seed-reset-my-database
Below cypress function will help to execute a system command.

Below cypress function will help Execute code in Node via the task plugin event.

As part of this story we are planning to run / exceute the script via Github actions.

discovery class ref: https://github.com/golevelup/nestjs/blob/master/packages/discovery/src/discovery.service.ts


Set the updateToolbar method of ToolbarPlugin to be private

Summary:
Set the updateToolbar method of ToolbarPlugin to be private

Hi @patelneal, thanks for the PR! Feel free to keep contributing, and let me know if there's anything I can help with.


Why only support JDK8?

I'm using jdk6, and as I add the maven dependency of t-digest library, it throws exception of not supporting jdk version less than jdk8. However, when I copy all the source code into my project and run the code, it seems working well with jdk6. So, is there any reason for why only supporting jdk8? Can I use it under jdk6?

Actually, I think that the guarantee is that it will work with JDK7.
I can't think of anything off-hand that would break on JDK6, but I haven't used that forever. If the unit tests pass, it should be fine.
But why in the world are you using a version that was end-of-life in 20013? Even JDK7 was EOL five years ago!

Actually, I think that the guarantee is that it will work with JDK7.
I can't think of anything off-hand that would break on JDK6, but I haven't used that forever. If the unit tests pass, it should be fine.
But why in the world are you using a version that was end-of-life in 20013? Even JDK7 was EOL five years ago!


Help with using Pointer-Generator for abstractive summarization

❓ Questions and Help
I would like to implement the PG model (as well as other models) but can't find any examples for how to go about this using fairseq.
What have you tried?
I have seen this issue, but wondering how it was resolved (if there is a way that doesn't involve changing the source code)

@griff4692 Any luck getting this to work?

Maybe https://github.com/zhawe01/fairseq-gec will help you.


[Enhance] add a down_cast check for ColumnHelper::as_column

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Add a down_cast check for ColumnHelper::as_column.

@Mergifyio backport branch-2.3

@Mergifyio rebase


Dashboard Overview not loading when a Cluster Member has no access to any namespace

Create a cluster cluster1
Create a second cluster cluster2
As admin assign user1 to both cluster1 and cluster2 as Cluster Member
As user1 In cluster1 create a new project and add a namespace to it.

Default and System projects shouldn't be accessible for a Cluster Member

Leave cluster2 as-is
Try opening the Dashboard from cluster2

GET to https://<rancher_host>/k8s/clusters/c-xxxxx/v1/namespaces returns 405
{"type":"error","links":{},"code":"MethodNotAllowed","message":"Method GET not supported","status":405,"type":"error"}

Browser displays:
HTTP Error 405:
Method GET not supported
Reload

Opening the Dashboard from cluster1 works as it has at least one namespace.

If relevant, the setup was HA Rancher
Rancher version master-head 8bd924599 (04/20/2020)

This appears to be working on latest master-head

Rancher version: master-head (09/14/2020) 0f691dc70f86bbda3d6563af11779300a6191584 HA k8s
Created Downstream DO cluster.
Added a Cluster Member to it with role Standard User
Login with this cluster member.
Try Access Dashboard

As soon as I add a namespace as this standard user I am able to access the Dashboard.

The root cause is that the user does not have permissions in the downstream cluster to list namespaces. This makes sense as they don't have any namespaces to access so the permissions were never created in rancher and pushed downstream to the user cluster. The same behavior is seen if using the kubeconfig for the user and attempting to list namespaces against the downstream cluster using kubectl. @westlywright @vincent99 Can dashboard handle this situation gracefully?

After some more testing, I don't think it will matter if dashboard handles this one situation. The user is still missing permissions in the downstream cluster so they can't even make that first namespace. This issue falls under the ongoing RBAC consideration/work @ibuildthecloud has. Rancher currently does this push reactively to things getting created through it vs being proactive and pushing the perms before a user needs them.

@ibuildthecloud will fix the 405 error on dashboard in this use case. However, dashboard will be blank until cluster member is granted access to a project or creates a project.

This is now working on latest master-head (09/19/2020) bd88c23
But there's a blocker issue #29021

@izaac can we close this and just track the other issue separately? This issue is clearly about dashboard not loading for cluster members, which is fixed, but the other issue is a different problem.

Closing, original reported issue is not reproducible https://github.com/rancher/rancher/issues/28568#issuecomment-695349970


Document inference_service_pb2

inference_worker.py doesn't work without it.
Maybe has to do with gen_inference_service_py_srcs.sh but I can't find any documentation on it.

My issue was failing to activate my virtual env which meant I got python2 instead of python3.
Also do we want generated files checked it?


package.json build script places files incorrectly

From @hut8:
Minor issue: running npm run build, which is:
rm -rf ./dist && NODE_ENV=production ./node_modules/.bin/webpack && cp -r domo/ dist/

results in the dist folder containing a domo folder, which contains the thumbnail and manifest.json. Another script, upload, has:
npm run build && cd dist/ && domo publish && cd ..

Because manifest.json is in dist/domo and not just dist/, that ends up causing this error:
X No manifest.json file in this directory
Please publish from a directory containing a Custom App design manifest.json file

Perhaps the build script could just run cp domo/* dist/ instead?

@hut8
So after looking at the issue a bit, it could be an environment thing that we'll have to fix. What OS are you running/which environment is the npm build command running in?

Hey! Sorry I somehow missed the notification for this. I'm on Linux (Ubuntu Xenial 16.04).

I've created a pull request which has fixed it on our linux machines. Let me know how that works and I'll get it in advanced-sample-app as well.

This can be closed 😄 Thanks for fix.