Quick question

Hi. Thank you very much for sharing your code. I have 1 clarification doubt - I was confused whether we also need to backpropagate through the Sinkhorn algorithm. The repository code doesn't do that. Is this due to the Envelope theorem used in the papers [1] & [2]

Hi,
Good question ! The answer is : we don't need to do that ! Why ? Thanks to Danskin's theorem ! (https://en.wikipedia.org/wiki/Danskin's_theorem).
The entropic-regularized UOT has a unique solution, so we have a gradient with respect to the ground cost C. Furthermore, we have an unbiased estimator of the expectation of minibatch OT, so we can exchange gradients and expectations. Thus minimizing our empirical estimator leads to the minimum of the whole problem. This is justified with our second theorem.
Note that your suggestion would work. This was used to get a differentiable loss with a fixed budget of Sinkhorn iteration. But with a small budget you do not get the optimal solution, that is why I prefer using Danskin theorem. This is also what the original deepjdot algorithm did https://github.com/bbdamodaran/deepJDOT

Thanks a lot for the detailed & quick response.


Add border support feature (issue #32)

Hello,
Here is a PR for adding border support to the excel dart package.

@amigne Could you also add a few tests to make sure that this feature doesn't break in the future?
You can just drop a sample file in the test folder and test out the major new functions.

@FauconSpartiate Sure, I'll add tests. Good idea!
At the same time, I was thinking to change a little bit the code. I was not very happy to let the users deal with the BorderSet class. I'll take the opportunity to write these tests to make BorderSet a private class (_BorderSet) and to hide it in the parser and the save part of code. This should make the border feature easier to use for the users.

Sounds good, don't forget to update the documentation accordingly 👍

@FauconSpartiate Sure, I'll add tests. Good idea!
At the same time, I was thinking to change a little bit the code. I was not very happy to let the users deal with the BorderSet class. I'll take the opportunity to write these tests to make BorderSet a private class (_BorderSet) and to hide it in the parser and the save part of code. This should make the border feature easier to use for the users.

Change the code if it simplifies the things and makes life of devs easier. Do your magic @amigne .
Thanks in advance.

Re-request for review when you had done the changes....


Support Apple sign-in

Want to support Apple sign-in, as App Store audit rules require apps to implement Apple sign-in if they need a third-party login
@wuweilaiya

已支持


Expose HttpRequestMessage object to PowerShell function

In C# HTTP trigger, the dev can use req (HttpRequestMessage object) to handle HTTP request - e.g. to get client certificate by calling req.GetClientCertificate().
But for PowerShell HTTP trigger, the req is an string object, there is no way for dev to deal with HTTP request message.

While you are right that all aspects of the HttpRequestMessage may not be available, many of the important request details are available as environment variables, including:

http method via REQ_METHOD
headers via REQ_HEADERS_XXX
query params via REQ_QUERY_XXX
route params via REQ_PARAMS_XXX

Correct, we have requirements to retrieve the client certificate from the HttpRequestsMessage. But, the PowerShell script hides the object of HttpRequestsMessage, so that there is no way to get the client certificate.

@mathewc , any updates on this?

Addressing this won't be a quick fix for us unfortunately. The issue is that certificates and other "advanced" request properties aren't easily mappable to environment vars like all the other request properties above. We can't currently just make the entire .NET HttpRequestMessage available to your code - that's not the current execution model.
Can you provide an example of the Powershell code you'd want to be able to write? How would you access/use this cert?


Fix do_outdated() when pip freeze is blank

If no packages are installed, results contains a single empty string. This causes problems in the subsequent call to convert_deps_from_pip. Therefore filter out empty strings from results to avoid this.
Fixes #1699

✨🍰✨


Add rules to simplify zipWith and zipWith3

Thanks, good catch!
Hmm - I see that we don't suggest (\x y -> (x, y)) becomes (,). Usually when a rule can be achieved in a few independent steps, and each of those steps is "obviously good", I won't put the composition in as a rule. But here we're not suggesting the reduction of a tuple. That seems like a mistake, so I'd be inclined to add that rule instead. Seem reasonable?

Yeah, that makes total sense.


FileNotFoundError: [Errno 2]

Hi!!
I get:
FileNotFoundError: [Errno 2] No such file or directory: '/etc/docker/dockerfile_php_7'
seems like the path in the socker-compose.yml is wrong
It say:
dockerfile: /etc/docker/dockerfile_php_7
Should say:
dockerfile: ./etc/docker/dockerfile_php_7

Hi @joaquin-becerra  good catch there was a PR raised by @lemil that I just merged and addresses this issue. https://github.com/wdekkers/docker-php7-httpd-apache2-mysql/pull/2


Pull in upstream updates

For future reference what I've done (in an attempt to simplify the process as much as possible) is to checkout into the upstream main branch, create the maxb/pull-upstream branch from there and then git rebase origin/main (where origin is this/Invisv-Privacy/client-sdk-js)
This allows us to just replay all of the commits from upstream on top of our HEAD all at once, which I find much more reasonable to consider/reason w/.

Vite

iiiiiinteresting! I'll admit I'm not familiar with Vite. Would you mind opening a separate PR with the changes you're suggesting and we can review it there? Thanks!!!!


insert data other language(farsi)

Issue Type: Bug
Hi
At the time of entering information in Farsi language in INSERT Command in an unorderd way
Azure Data Studio version: azuredatastudio 1.34.0 (83a4316cf89cbf961a0f09a72c861379cfd01fa9, 2021-12-15T20:50:08.229Z)
OS version: Darwin x64 22.2.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
16.00GB (0.01GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

agent
Mic
0.49.0

arc
Mic
0.11.0

azcli
Mic
0.3.0

azuredatastudio-postgresql
Mic
0.2.7

dacpac
Mic
1.13.0

import
Mic
1.5.4

profiler
Mic
0.12.2

schema-compare
Mic
1.12.0

sql-database-projects
Mic
0.14.1

azdata-sanddance
msr
4.0.3

sql-prompt
Red
0.2.11

(1 theme extensions excluded)

@hamidrasouli2951 thanks for opening this issue.  I'm not able to see in the screenshot the bug your pointing out.  It looks like there is an unexpected N' before the integers in the second column and no trailing ' in the final string column.  But I'm not sure what you're trying to do.

@hamidrasouli2951
Gentle reminder, we're waiting for your response.

Closing as not enough information available to investigate issue.


build failing on android

Expected Behaviour
Actual Behaviour
when adding the plugin to the build, the build fails with error
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'android'.
> Could not resolve all dependencies for configuration ':_debugCompile'.
   > Could not find any matches for com.android.support:support-v4:+ as no versions of com.android.support:support-v4 are available.
     Searched in the following locations:
         https://repo1.maven.org/maven2/com/android/support/support-v4/maven-metadata.xml
         https://repo1.maven.org/maven2/com/android/support/support-v4/
         https://jcenter.bintray.com/com/android/support/support-v4/maven-metadata.xml
         https://jcenter.bintray.com/com/android/support/support-v4/
         file:/private/tmp/xy/xxyy/platforms/android/libs/
     Required by:
         :android:unspecified

the build is completed correctly with no problems, when the plugin is not added
it also worked with the plugin added a month ago
the plugin is the only one I am adding
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
<EMAIL_ADDRESS>(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Not relevant, problem occurs in the compile step
Cordova CLI version and cordova platform version
cordova --version: 6.3.1
cordova platforms<EMAIL_ADDRESS>
Plugin version
cordova plugin version | grep phonegap-plugin-barcodescanner: 6.0.3

Sample Code that illustrates the problem
Logs taken while reproducing problem
No additional logs

any luck @redon? I'm having the same issue with very similar cordova/plugin versions

to get around this, I removed this line from my -barcodescanner.gradle file:
-    compile 'com.android.support:support-v4:+'

(no idea whether that will impact the behavior of the plugin)

that
http://stackoverflow.com/questions/33185086/failed-to-resolve-com-android-supportsupport-v423-0-0
helped me
I think especially «Had to add "Android Support Repository" from SDK Manager.»
So the problem is not with the barcodescanner-plugin itself but with the installed SDKs
I therefore close this issue


BZ & N2O decomposition

About the PR

Nitrous oxide now decomposes into oxygen and nitrogen when the mixture temperature is 1400K or above.
Atmoses will be able to produce BZ gas - a potent hallucinogenic that also puts slimes into sleep and kills your brain.
Synthesized using nitrous oxide and plasma with 1:3 ratio. Also decomposes N2O mid-process.
When inhaled deals 2.5 cellular and 2.5 poison damage, also gives you rainbows & drunk effects, puts slimes to sleep.
Why / Balance

SS13 content, also as a cheaper and more dangerous alternative to Frezon (as a drug).
Technical details

I've violently copied reactions code from /tg station, so it just works. 🤷♂
Also the toxic damage is there to just crit people.
Media

BZ synthesis
https://github.com/space-wizards/space-station-14/assets/113782077/6159a76f-7a5c-4c41-b2f2-83246371da8f
BZ effects when inhaled
https://github.com/space-wizards/space-station-14/assets/113782077/833c0969-7ef7-444e-a90d-0344d07751e6
BZ effects on slimes
https://github.com/space-wizards/space-station-14/assets/113782077/5f0e4e12-e6d6-4213-a4d1-527fe5f882fd
Nitrous oxide decomposition
https://github.com/space-wizards/space-station-14/assets/113782077/0c125217-2c7d-4206-a971-ca50074c3876

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

TODO:

[ ] Balance.

Changelog

add: Added BZ. BZ is a hallucinogenic gas that is more powerful and cheaper to make than frezon. It also kills your brain and puts slimes to sleep, which may be useful when there's a slime outbreak.
add: BZ can be made by mixing nitrous oxide and plasma. Also decomposes nitrous oxide into oxygen and nitrogen.
add: Nitrous oxide can be decomposed into oxygen and nitrogen when heated to 1400K and above.

BZ has nearly the same effects as frezon, as in the damage and hallucinogenic properties

BZ has nearly the same effects as frezon, as in the damage and hallucinogenic properties

chinese frezon

BZ has nearly the same effects as frezon, as in the damage and hallucinogenic properties

i want BZ to be the chinese frezon - easier to make, more powerful and deadly.
deadly because then you would consider twice before inhaling it, giving off a better effect since it's more dangerous than frezon.
also dealing with slimes as a bonus, since n2o can't put them to sleep 😀

You could make humans cough when inhaling. Makes a lot of sense and a nice touch.
Does the max temperature 100000 have a reason in n2o decomposition? Instead of no limit.
And if its easier to make, maybe it should be a little less deadly? Like just the toxin, and maybe something like asphyxiation. Cellular is pretty bad.

You could make humans cough when inhaling.
maybe it should be a little less deadly? Like just the toxin, and maybe something like asphyxiation. Cellular is pretty bad.

Yeah makes sense

Does the max temperature 100000 have a reason in n2o decomposition? Instead of no limit. And if its easier to make, maybe it should be a little less deadly? Like just the toxin, and maybe something like asphyxiation. Cellular is pretty bad.

I copied it from /tg station, I guess they put it as high as possible to remove the limit

cl doesnt work and should probably be condensed

cl doesnt work and should probably be condensed

what's cl?

changelog
you are missing - before each add and the :cl: emoji before them all
also nobody reading allat

damn

Hi there, thanks for the PR. This sounds fun and interesting and we can all help take a look at balance once the build is working. Would you be willing to split up the nitrous oxide decomposition into a separate PR? That way we can get that in without having it blocked by BZ balance discussions.

not a fan of bz conceptually, its effects are already done better by other gases

Would you be willing to split up the nitrous oxide decomposition into a separate PR?

Sorry, i'm not really strong in git, idk how to make commits switch branches, and there's constants for both n2o n bz defined in a single commit, which i also dunno how to split.

oh nice, it now works

so?

so?

Could you respond to @mirrorcult 's comments? Particularly those about BZ not being distinct enough from frezon to warrant being its own gas. Can you think of other things that BZ could do/be used in so that it is different from frezon?

Can you think of other things that BZ could do/be used in so that it is different from frezon?
It already puts slimes to sleep, which no other gas does, and makes people cough 🤷♂
In ss13 it's also used in freon formation instead of tritium. (actually, lemme make that rq)

could also be used in healium production (https://tgstation13.org/wiki/Guide_to_Atmospherics#Healium)

As you've probably noticed, the functions of BZ have been sort of taken over by tritium and frezon here. I don't think a direct copy would be received well here.
I do like the nitrous decomposition reaction, which is why I asked if you would be willing to split it out into a separate PR or make this PR primarily about nitrous. Perhaps you could also consider adding a nitrous synthesis reaction involving miasma and oxygen forming nitrous and water?

which is why I asked if you would be willing to split it out into a separate PR or make this PR primarily about nitrous

Figured it out and moved to a separate pr https://github.com/space-wizards/space-station-14/pull/22738

nitrous synthesis reaction involving miasma and oxygen forming nitrous and water

Or use BZ 😆

I don't really see any logic in using miasma and oxygen to form n2o n water.
BZ afaik in ss13 is used as a catalyst or such to produce other gases, so it'd be weird to use anything else, might be just me 🤷♂

Because if you assume miasma is something vaguely like ammonia:

Which is a reaction that happens to really exist, without any catalysts.
It's probably going to take more discussion about what would make BZ different from the gases that already exist. But the nitrous changes are very welcome since it would make it synthesizable from existing gases.

idk who would actually collect miasma in a tank and use it to produce n2o and water vapor, would probably lead to mass grave miasma factories, but sure why not 😆

involving miasma and oxygen forming nitrous and water

come to think of it, wouldn't it react automatically with the air and form vapor bath sleeping houses after somebody dies?

I think a minimum temperature requirement for this reaction to happen should alleviate this issue.

updated pr. review, any1?

Sorry for the delay getting feedback to you on this PR. There still remains the issue that BZ gas isn't distinct enough from the gases that already exist to justify adding a new gas. Perhaps you would be willing to discuss this more on Discord for how to proceed?

hmu

Thank you for your PR. Unfortunately, I'm opting to close it because as discussed here and on Discord, BZ is not distinct enough from gases that already exist in game, e.g. frezon. Please feel free to re-open if you want to discuss other changes that could make it a good candidate to add to the upstream game.


Add support for multi-core

So far Mirage only supports single core system. However, most of the interesting use cases we have in mind target multi-core platforms (such as the boards we have in the lab). Multi-core support is one of the main goal we should aim for in the near term.
There are still open questions, but here are some of the elements that should be taken care of:

[x] Understanding the boot process on RISC-V (do secondary harts need to be started manually? What is the entry vector?)
[x] Synchronization barriers to park secondary harts until initialization on the main hart.
[x] Prepare multiple stacks (can we prevent stack overflows? maybe by locking a PMP entry?)
[x] Create one vCPU per physical hart.
[x] How do we start secondary vCPUs?

We now have multi-core support on QEMU since #164
The support is still in-progress on bare metal.


Apache setting is wrong

Description
According to docu this should be the correct config for Apache 2.4.6 and later:
RequestHeader set "X-Forwarded-Proto" expr=%{REQUEST_SCHEME}s
When I access with https:// the X-Forwarded-Proto header is set to httpss - which looks wrong.
When I access with http:// the X-Forwarded-Proto header is set to https - which IS wrong.
The X-Forwarded-Proto header is for identifying the protocol (HTTP or HTTPS) that a client used to connect.
When I access with http:// I access with http, not https. There should not be a trailing s.
Which makes this setting wrong.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/host-and-deploy/linux-apache?view=aspnetcore-8.0#configure-apache
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/host-and-deploy/linux-apache.md
Document ID
c22ca7a0-d1d4-200d-fc36-a9798d350c6e
Article author
@Rick-Anderson

Thanks for reporting this. Any chance you could PR this? At the top right of the article, select the Edit pen icon:

Select the pen icon again.  Edit, then Save.

I could do this, but I am alerted about this error.
Why should someone write Note: Apache versions before 2.4.6 don't require the RequestHeader set contain the trailing s?
Is it the other way around? Maybe before 2.4.6 it was required.
Should I drop this before 2.4.6?


Added typescript types file (.d.ts), so lib can be used in typescript projects

Added a basic .d.ts file, so this library can be used in typescript projects.

Added a line in .gitignore to prevent unnecessary Mac ".DS_Store" files from getting committed.

Actually, this PR isn't enough to do the job; closing it.


bug when the phone is turned off in the app with the component

Hello, I found a bug.. When the phone is turned off in the application and then the phone is turned on again then the collapsing component no longer works, is there any way to solve this? I add a video of the problem: https://streamable.com/7e3g8
Regards,

Hi, thanks for report, have you experience this issue even with a production build? which react-native version are you using?

Hi Cesar, thats happen in react-native 0.55.0.. and also happen with '--variant=release'.
Is there a suggestion of compatibility with some version of react-native?
Thanks

I haven't tried new version from react-native yet, so i presume that new versions "just works".
Have you tried call the redraw() function from the AppBarLayout ref on componentDidMount or when the app came back from the background (by AppState)?

I have try
<AppBarLayout ref={(ref) => { this.appBar = ref }}>

`
_handleAppStateChange = (nextAppState) => {
if (this.state.appState.match(/inactive|background/) && nextAppState === 'active') {
this.appBar.redraw();
}
this.setState({appState: nextAppState});
};
componentDidMount() {
AppState.addEventListener('change', this._handleAppStateChange);
}
componentWillUnmount() {
AppState.removeEventListener('change', this._handleAppStateChange);
}
<AppBarLayout ref={(ref) => { this.appBar = ref }}>

`
but without luck :(

I have noted that redraw() some times work and some times not

Interesting, i will take a look at it as soon as i get some time this week

Thank you! I appreciate that very much!

Hi, sorry for delayed response, i've fully reproduced your issue on my simulator, and i have tested your code and worked, i tested multiple times and never had any single attempt that didn't worked.
Here's a demonstration

Or maybe i am testing wrong? any idea?
I also don't think this is the best solution at all, it's actually really hard to understand how native modules behave in certain situations, i will still think in some pleasant solution.

try it in on real device you will see the problem

@magestein Hello, how did you solve it? I'm just like you, but I happened when the application was started, not when the application was waked up. #21


How to setTimeout() for query?

Hi,
A query that usually takes 2 seconds, sometimes goes beyong the default 10 seconds.
Is there any way to increase timeout?
Please help
Thank you

Hi @klukiyan . I guess this can help => https://salesforce.stackexchange.com/questions/99075/my-soql-select-is-timedout-sometimes
AFAIK, there is no way to increase limits. But you can refactor to limit the rows amount per query, thus reducing the amount of time needed to scan and return.

Hi @davispeixoto
It didn't help.
The default api time-out is 10 seconds. Salesforce accepts timeout header parameter that can be up to 120 seconds.
I've seen examples with Java and C#. I've seen one with PHP. But it was plain php. I'm trying find a place where your package is merging with the toolkit to inject this command. It looks somehting like this;
$sfdc->setTimeout( 2*60*1000);

Can you help me to find where I could fit it? Where is the raw header constructed?
Thank you. And thank you for a great package!

@klukiyan , thx for the compliments about the package.
Can you provide me a link to the documentation you mentioned? I'd like all links php, java and c#...

@davispeixoto ,
I will try to find it in my browser history. I was googling for 3 hours yesterday before coming here with questions. It will take me some time to find those links that contained PHP codes samples.
Here are few that make remarks about 10 and 120 seconds for start:

https://developer.salesforce.com/forums/?id=906F00000009ABuIAM

https://salesforce.stackexchange.com/questions/99977/read-timeout-error-while-making-an-api-inbound-call-to-salesforce

The error that I'm getting is Error Fetching Http Headers.
The scheduled hourly job on web server will get it everyhour, until I try to execute some simpler query, and after that it sometimes start to work again. Usually the query takes 2.5 seconds and returns 400 rows.
I don't know how to optimize it further:
SELECT Credit_Note__r.Name, Credit_Note__r.Type__c, Credit_Note__r.Expected_credit_note_date__c, Credit_Note__r.Settlement_Type__c, Credit_Note__r.Budget_Status__c, Credit_Note__r.Status__c, Credit_Note__r.CreatedDate, Credit_Note__r.Completed_Date__c, Credit_Note__r.Posting_Date_SAP__c, Credit_Note__r.Invoice_Number__c, Credit_Note__r.Cancellation_Correction__c, Credit_Note__r.Modification_Cancellation_Remarks__c, Credit_Note__r.Reposting__c, Credit_Note__r.Order_Type__c, Credit_Note__r.SAP_ID_Credit_Note__c, Credit_Note__r.Account_SAP_SO__c, Credit_Note__r.Plant__c, Credit_Note__r.Storage_location__c, Credit_Note__r.VAT_exception__c, Credit_Note__r.Billing_Heading_Text__c, Credit_Note__r.Do_Not_Send_Credit_Note_to_Customer__c, Credit_Note__r.Customer_s_Invoice_Number__c, Credit_Note__r.Comments__c, Credit_Note__r.Credit_note_Payment_Term__r.Name, Amount__c, Billing_item_text__c, Condition_Type_I__c, Condition_Type_Picklist__c, CurrencyIsoCode, Financial_Material_Reporting__c, Name, Order_Reason_Code__c, Order_Type__c, Original_invoice_number__c, Price_Difference__c, Product_for_SAP_Batch_Upload__c, Quantity_Difference__c, Quantity__c, Rebate_Amount__c, Stock_Compensation__c, Credit_Note__r.Current_Approver__c,  Division__c
    FROM Credit_Note_Products__c
    WHERE (Credit_Note__r.LastModifiedDate >= YESTERDAY OR LastModifiedDate >= YESTERDAY) AND Modification_Age__c < 0.5

I'll come back with more precise links later

I see a few problems around here...
First, please bear in mind that the library I allowed here is for the SOAP API, not to REST API (or any other ones... Salesforce have a lot of them actually, like, data api, bulk api, UI, visualization api, analytics api...). The current library is for SOAP API, and you sent references for the REST API.
Second, I'm not seeing you limited your query using explicit LIMIT clause. This can lead to full table scans, thus making a query to run longer and longer as your records amount increase.
With that said, I didn't see any way to change the time limit for expiration for a REQUEST/API CALL, only for the SESSION you open (which usually closes after 2 hours).
Now getting into your specific error, It feels like it is all about your php settings. It seems that your wsdl/soap client configuration is closing the sockets before getting a reply. You can try changing this on your side.
Follow a few references about it.
https://stackoverflow.com/questions/9403486/error-fetching-http-headers-in-soapclient
http://php.blogaboutwhatever.com/2011/11/error-fetching-http-headers/
https://stackoverflow.com/questions/4824799/soapfault-exception-http-error-fetching-http-headers

I'm gonna close this issue as this is not specifically related to the package, but rather to implementation practices and/or server setup.

@davispeixoto Thank you very much.
I've increased the default socket timeout and will see if failure happens again.
Thank you for your help

thx for your feedback, @klukiyan


npm test throws errors

Hey,
I'm trying to run npm test on my Mac Host but it throws errors.
#npm test

><EMAIL_ADDRESS>test /Users/alexander/Code/foundation-emails
> cd testing && npm start

><EMAIL_ADDRESS>start /Users/alexander/Code/foundation-emails/testing
> gulp

[22:08:23] Requiring external module babel-register
[22:08:23] Using gulpfile ~/Code/foundation-emails/testing/gulpfile.babel.js
[22:08:23] Starting 'default'...
[22:08:23] Starting 'build'...
[22:08:23] Starting 'clean'...
[22:08:23] Finished 'clean' after 2.22 ms
[22:08:23] Starting 'copy'...
[22:08:23] Finished 'copy' after 12 ms
[22:08:23] Starting 'pages'...
[22:08:24] 'pages' errored after
[22:08:24] TypeError: (0 , _inky2.default) is not a function
    at pages (gulpfile.babel.js:35:11)
    at bound (domain.js:287:14)
    at runBound (domain.js:300:12)
    at asyncRunner (/Users/alexander/Code/foundation-emails/testing/node_modules/async-done/index.js:36:18)
    at nextTickCallbackWith0Args (node.js:452:9)
    at process._tickDomainCallback (node.js:422:13)
[22:08:24] 'build' errored after
[22:08:24] 'default' errored after

npm ERR! Darwin 15.2.0
npm ERR! argv "/usr/local/Cellar/node/5.4.1_1/bin/node" "/Users/alexander/.npm-packages/bin/npm" "start"
npm ERR! node v5.4.1
npm ERR! npm  v3.5.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `gulp`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'gulp'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the foundation-emails-template package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs foundation-emails-template
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls foundation-emails-template
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/alexander/Code/foundation-emails/testing/npm-debug.log
npm ERR! Test failed.  See above for more details.

Following versions are installed:
#node -v
v5.4.1
#npm -v
3.5.3

Any ideas?
Thanks

Could you try setting up our new template stack and see how it works for you? The errors you're getting are related to the Inky dependency not being up to date, and this template has been configured to use the latest version of everything.
It's the same as the one in the testing/ folder, but standalone.


clean up registry

Make sure all these following Definitions can really work.

add knative as workload and add componentDefinition for it.
fix knative auto scale trait, make it real work

apiVersion: core.oam.dev/v1beta1
kind: TraitDefinition
metadata:
  annotations:
    definition.oam.dev/description: "Auto scale for knative serving"
  name: knative-autoscale
  namespace: vela-system
spec:
  appliesToWorkloads:
    - knative-serving # this should be some knative like workload
  schematic:
    cue:
      template: |-
        import "encoding/json"
        patch: {
          metadata: annotations: {
              "my.autoscale.ann": json.Marshal({
                  "minReplicas": parameter.min
                  "maxReplicas": parameter.max
              })
          }
        }
        parameter: {
          min: *1 | int
          max: *3 | int
        }

fix expose trait and add into kubevela chart as inner trait

apiVersion: core.oam.dev/v1beta1
kind: TraitDefinition
metadata:
  name: expose
  namespace: vela-system
spec:
  schematic:
    cue:
      template: |-
        parameter: [string]: int

        outputs: {
          for k, v in parameter {
              "\(k)": {
                  apiVersion: "v1"
                  kind:       "Service"
                  spec: {
                      selector:
                          app: context.name
                      ports: [{
                          port:       v
                          targetPort: v
                      }]
                  }
              }
          }
        }

Delete the code of PodDisruptionBudgetTrait and write a CUE based trait instead, put it into registry. https://kubernetes.io/docs/tasks/run-application/configure-pdb/

add expose trait done see:https://github.com/oam-dev/kubevela/pull/1678


Fix skl nan tag.

Closes https://github.com/dmlc/xgboost/issues/5401 .

Weird that the Python test passes on Linux but fails on Windows ...


Errors when integrating this package into Xamarin Forms application

Hi Kazo0,
I've been having all sorts of problem integrating this framework into a Xamarin Forms application.
I tried to integrate into both Android and iOS, but then gave up because I received all kinds of build (java) errors.
Do you have a Xamarin.Forms sample project that shows how the GoogleMediaFramework-Xamarin can be integrated?  Can be really basic just to point me in the correct direction.
I have been experimenting with upgrading the Google packages in your Android samples. I managed to get it working, but noticed that I cannot upgrade the Xamarin GooglePlayService packages to anything higher than <IP_ADDRESS>.  Attempting to do this results in build errors.  Any idea why this is?  Ideally the framework would use later packages.
If you'd like to see whether my upgrade is worth including, I can create a fork/pull request.
thanks

I'm also trying to use this component in a Xamarin Forms application. I was able to get it working by creating an Android PageRenderer, which requires launching a new activity in order to use the player. I would like to override it as an Android ViewRenderer instead, so I can drop it in an existing page as an element. Do you have any examples laying around that describe that? I noticed the player has a .SetFullscreenCallback method, but I'm not sure how to assign that from inside a ViewRenderer.
Thanks!


Workshop to setup spam trap

Link to the document: https://docs.google.com/document/d/133woZJaz9rMC526vP0LTxmGuAbBSBiHSi4tK-EKvuzw/edit

done; closing the task.


add more debug info to gdal source

 more gdal_source debug prints

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 18 (61.11%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 85.314%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

operators/src/source/gdal_source.rs
11
18
61.11%

Files with Coverage Reduction
New Missed Lines
%

operators/src/source/gdal_source.rs
1
86.05%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
27471

Relevant Lines:
32200

💛  - Coveralls

bors merge


Exception: Failed to load font with url https://fonts.gstatic.com (google_fonts)

Package
google_fonts 6.1.0 (https://pub.dev/packages/google_fonts/versions/6.1.0)
Existing issue?

[X] I checked the existing issues

What happened?
I'm getting some error on Android via Firabase Crashlytics. Here is log output. I encountered this error once in my device. If this error occurs, Flutter loads default font in Android app and continue without crashing. But anyway I'm getting errors and maybe this will help to fix bugs. Error has different variants.
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Failed to load font with url https://fonts.gstatic.com/s/a/e549403e4a007249673d041c55e7205d754210d79b962001d95a8313191aa3c2.ttf: HandshakeException: Handshake error in client (OS Error:
	CERTIFICATE_VERIFY_FAILED: Hostname mismatch(handshake.cc:393))
       at ._httpFetchFontAndSaveToDevice(google_fonts_base.dart:263)
       at .loadFontIfNecessary(google_fonts_base.dart:180)
       at googleFontsTextStyle.<fn>(google_fonts_base.dart:117)

Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Failed to load font with url https://fonts.gstatic.com/s/a/e549403e4a007249673d041c55e7205d754210d79b962001d95a8313191aa3c2.ttf: ClientException with SocketException: Failed host lookup: 'fonts.gstatic.com' (OS Error: No address associated with hostname, errno = 7), uri=https://fonts.gstatic.com/s/a/e549403e4a007249673d041c55e7205d754210d79b962001d95a8313191aa3c2.ttf
       at ._httpFetchFontAndSaveToDevice(google_fonts_base.dart:263)
       at .loadFontIfNecessary(google_fonts_base.dart:180)
       at googleFontsTextStyle.<fn>(google_fonts_base.dart:117)

Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Failed to load font with url https://fonts.gstatic.com/s/a/e549403e4a007249673d041c55e7205d754210d79b962001d95a8313191aa3c2.ttf: ClientException with SocketException: Connection reset by peer (OS Error: Connection reset by peer, errno = 104), address = fonts.gstatic.com, port = 44498, uri=https://fonts.gstatic.com/s/a/e549403e4a007249673d041c55e7205d754210d79b962001d95a8313191aa3c2.ttf
       at ._httpFetchFontAndSaveToDevice(google_fonts_base.dart:263)
       at .loadFontIfNecessary(google_fonts_base.dart:180)
       at googleFontsTextStyle.<fn>(google_fonts_base.dart:117)

Have a nice day
Relevant log output
[√] Flutter (Channel stable, 3.16.0, on Microsoft Windows [Version 10.0.22621.2861], locale en-US)
    • Flutter version 3.16.0 on channel stable at C:\FlutterSDK
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (7 weeks ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\AndroidSDK
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = C:\AndroidSDK
    • ANDROID_SDK_ROOT = C:\AndroidSDK
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Build Tools 2019 16.11.25)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
    • Visual Studio Build Tools 2019 version 16.11.33423.256
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] VS Code (version 1.79.2)
    • VS Code at C:\Users\Aykut\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.22621.2861]
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 117.0.5938.134
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 120.0.2210.91

[√] Network resources
    • All expected network resources are available.

• No issues found!

I'm still getting a lot of error. Is there any update to fix this issue? I found similar issues https://stackoverflow.com/questions/61124989/error-google-fonts-was-unable-to-load-font but without conclusive answer.

Same issue: https://github.com/material-foundation/flutter-packages/issues/537

https://github.com/material-foundation/flutter-packages/issues/521

https://github.com/material-foundation/flutter-packages/issues/507

@rodydavis

I got the same issue on Android device reported by Firebase crashlytics

Sdk_gphone64_arm64: 75%
Galaxy S20 5G: 25%

I think, the solution is already documented here.

The google_fonts package will automatically use matching font files in your pubspec.yaml's assets (rather than fetching them at runtime via HTTP). Once you've settled on the fonts you want to use:

This is not a solution, just another option for fetching fonts. I'm going to create a pinned issue for this

Closing in favor of #560

same issue


Add nperiods property to Driver

This will just add the possibility to configure the number of periods used by the underlying soundcard. This can be especially helpfull when tailoring performance on jittery systems.

Hi there! Looks great, can you just add tests? I want to keep coverage at 100%.

Hi there! Looks great, can you just add tests? I want to keep coverage at 100%.

hmm. jack2 dummy driver - used in the tests - doesn't allow you to change nperiods JackDummyDriver. Adding tests for nperiods would then involve either using alsa -driver or change the JackDummyDriver

OK, got it. I will see what can be done once I’ll be in front of computer 😃

I don't want to add alsa driver in CI, so let's just special case this param 😃

Just released v0.2.0—it includes this PR :)


Add --remove-previous-comments functionality

This PR adds --remove-previous-comments CLI flag. The flag can be used in order to get the new danger comments in the tail of the PR and remove previous comments, so the PR looks bot so spammy.
Closes #948

--remove-previous-comments flag work incorrect on gitlab-ci, all comments will be removed and no comment displayed.
--new-comment flag still update previous comment instead of creating new one.
danger version is 5.6.7

@znacloud  you're welcome to take a look and fix this 👍

the problem must be here: https://github.com/JoeSSS/danger/blob/3bda96eb104a50f0f255ba6b1a66b07d788162f6/lib/danger/request_sources/gitlab.rb#L131
the logic should be decoupled from comment posting.


Enable Admins to add other Admins

and see who the other Admins are.

@jeanwalshie you can actually already do this in the Admin section.
You should be able to see them at https://adoptadrain.sfwater.org/admin/user?model_name=user&utf8=✓&f[admin][26241][v]=true&query= but more generally you can filter users by many different filters on that page (the like has them filtered by Admin=true.

To add a new admin, find their user in the users list (or have them register first if they haven't), and then click

perfect, got it! sorry, i should have been able to figure this out myself. still familiarizing myself with the admin area. thank you, @jszwedko!!!


Searching users to add as contributors does not work in IE11

If you create a Web App resource, then try to add contributors, you'll find that the user search feature doesn't work like other browsers.  Namely, the search doesn't work at all.
013.zip

This bug was reintroduced recently, but caused by another issue:

I will be addressing these.

This has been fixed and can be tested at https://beta.hydroshare.org


Improve article 1906.03291

Exact location of issue
Sadly, this is in the abstract so it's tough to pinpoint it with an anchor.
Example:

The authors section, for instance with ID id1.1.id1

Problem details
Well, it seems the authors section has been parsed wrongly, see the image below:

Thank you for reporting this @mrshu I think our neurips_2019.sty needs a small fix.


10 steps for creating your first pull request.

I hope the following screenshots will help with your first pull request.
Firstly create your account on github and goto twindle repository page here
Now you can follow the following steps.
Step 1

Step 2

Step 3

Step 4

Step 5

Step 6

Step 7

Step 8

Step 9

Step10

@vijaya-bhaskar can you add some different colour in the background of text then it will be more easier to read the comments you added 😄

I agree @pranavgoel29 but I cant add background color to the text now :)
But you can click on the image to open in a new window which would be clear.
Next time I will take that into consideration. Thank you.

@vijaya-bhaskar ok, no problem. 😄

I think it's awesome and well explained! I did a video for it too, I'll post it here
Points covered:

Create a reverse PR (from the twindle repo to yours, so you can get the updates) via Github
Creat a file via Github
Create PR (from your repo to twindle)

These steps are recommended only for the first pull request, since the task is only about creating one file without conflict.

Thanks, it was very helpful

Thank you so much for this kind of tutorial! It will be quite useful!

This is brilliant. Thank you so much for all these! Now I will be more confident to participate.

woow I love it marvelous work. it is well explained. thank you

Awesome Vijaya. Great step by step instructions.

this was really helpful, keep up with the good work team

@vijaya-bhaskar 💯❤

This was referenced in the FAQ document and updated in the PR #89  ✅

Please close this issue if completed

This has been added into the website at https://twindle-co.github.io/twindle/docs/articles/creating-first-pull-request.html and in the repo at /docs/articles/creating-first-pull-request.md.


Fix the PDB for the Gardener API server to allow roll outs

How to categorize this PR?

/area control-plane ops-productivity
/kind bug
What this PR does / why we need it:
Fix the PDB for the Gardener API server to allow roll outs
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Fix a bug in the PodDisruptionBudget of the Gardener API server that was not allowing maintenance operations with the hosting cluster when the HVPA is enabled the replicas are set to 1.

cc @mliepold

/assign @timebertt

/needs cherry-pick

/cherry-pick release-v1.44

/cherrypick release-v1.43
/cherrypick release-v1.42

/cherrypick release-v1.42

Just for reference, this issue was triggered by #5000, before that the PDB was configured only when replicaCount>=1 no matter the of the HVPA enablement.


add ability to jump to addressing errors in json-ld (3 pts)

when the overview page displays a set of errors I should be able to jump to and address them in a context appropriate specific way.

+1
and errors only show up on the overview page, but it would be useful if they are displayed in the step-tab where they occur.
for example here, id expect to see the error under "RECORDSETS" tab but it is not diplayed there


Support vykea furniture

Also added support for maxing fam weight as a function of a 2crs pref.

I haven't tested the new modifier. The other changes look OK.


Make favicon configurable

Currently favicon is hardcoded. Would be nice to make favicon link configurable (plus provide some guidance on how to do an svg one in the docs!)
Acceptance

[ ] Favicon configurable from the config
[ ] Docs for this e.g. docs/favicon

Tasks

[ ] Get favicon href from config in components/Layouts.jsx
[ ] Add docs

Still the case and quite easy to fix.


Add Sinatra::JSONRequest module with body_as_json helper

We've found this useful in dealing with json request bodies; it abstracts away the "is there a non-nil/non-empty body?  Is it valid JSON?" and returns the parsed body.

Hello! Since there hasn't been any discussion on this feature I'm going to close.
If anyone can suggest a nice API to handle JSON body for the request, I'd be welcome to discuss.
Thank you for your contribution!


pytest-cpp: handle exceptions

https://github.com/pytest-dev/pytest-cpp/pull/117

we will just wait until it's merged upstream


[feature_request]:

What is missing in the project?
I just got aware of your theme, great work!
I realized that your theme

is not a hugo module yet.
is based on bulma framework, which is pulled in as git submodule.

Solution?
My proposal is to transform the theme to a hugo module. Then you can pull in bulma framework as hugo module, which makes update of this dependency even easier.
A while ago, I authored a pull request that did exactly that for the hugo-fresh theme (which is based on bulma too). The theme author accepted my PR and we got it merged. Let me know if you are interested in receiving a similar PR for your shadocs theme. I'm happy to step in and assist you here!

Hi @deining ,
Indeed I never took time to look at that, but it seems more easier for both contributors and users...could be very useful.
I will be glad to add it in the theme, so don't hesitate to push a PR .
Thanks a lot 😊


Add bb --version

Problem
The bb binary doesn't currently report its version, which adds friction to checking of outdatedness / compatibility.
Happy case
Add a bb --version command that prints the bb binary's version, e.g.:
$ bb --version
bb 0.7.2 (git version hash: ...)

Extended discussion
An even happier case could be having the command prints Nargo version(s) tested compatible in the CI as well, e.g.:
$ bb --version
bb 0.7.2 (git version hash: ...)
Compatible with nargo 0.11.1 (git version hash: ...)

Not sure how much additional works has to be poured in to make this happen however (e.g. do we already have such compatibility CI tests?), so noting this as an extended discussion where we can revisit in additional new issue(s).

Duplicate of https://github.com/AztecProtocol/aztec-packages/issues/1728.


Patch for desktop-gtk3 outdated

The build failed with:
Pulling desktop-gtk3
Cloning into '/home/ubuntu/gimp-snap/parts/desktop-gtk3/src'...
remote: Enumerating objects: 28, done.
remote: Counting objects: 100% (28/28), done.
remote: Compressing objects: 100% (20/20), done.
remote: Total 1284 (delta 8), reused 17 (delta 7), pack-reused 1256
Receiving objects: 100% (1284/1284), 229.43 KiB | 596.00 KiB/s, done.
Resolving deltas: 100% (663/663), done.
patching file gtk/launcher-specific
Reversed (or previously applied) patch detected!  Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file gtk/launcher-specific.rej
Failed to run 'override-pull': Exit code was 1.

The new launcher-specific code:
if [ "$USE_gtk3" = true ]; then
  if [ "$wayland_available" = true ]; then
    export GDK_BACKEND="wayland"
    export CLUTTER_BACKEND="wayland"
    # Does not hurt to specify this as well, just in case
    export QT_QPA_PLATFORM=wayland-egl
  fi
  append_dir GTK_PATH $RUNTIME/usr/lib/$ARCH/gtk-3.0
else
  [ "$wayland_available" = true ] && echo "Warning: GTK2 does not support Wayland!"
  append_dir GTK_PATH $RUNTIME/usr/lib/$ARCH/gtk-2.0
fi

I've already fixed this locally, but haven't uploaded here yet.

https://github.com/diddlesnaps/gimp/blob/core18/patches/desktop-launch.patch has the latest patch (note it got renamed and has some extra things in it)


/usr/local/bin/raspibolt and https://bitnodes.earn.com/api/v1/nodes/me-${public_port}

Noticed today that the raspibolt script was saying:
Public  Not reachable
Found out why.
https://bitnodes.earn.com/api/v1/nodes/me-${public_port} was returning:
{"detail":"Request was throttled."}
Considering that raspibolt runs rarely (only on login), the service can not be throttling on requests/hour.  Maybe the total number of requests from my IP address is the problem.
Looking for an alternative service ... but no luck just yet.

Good catch, might be the cause for some of the issues raised here. Do you know of a different method to check the public availability of the Bitcoin node?

Sorry no.
Tried without success to fine an alternative service.

Hi @robclark56, I tried to figure out how this should work. As long as I use my own ip address, netcat always return no value. Which means 'you are public', I guess?

Am I missing something?

nc -z IP PORT returns:

0 if success (ie PORT on IP is open)
Non-Zero if failure (ie Port is not open)

Here are my examples (8333 is open, 12345 is not)
admin ~  ฿  nc -z  <IP_ADDRESS> 8333
admin ~  ฿  echo $?
0
admin ~  ฿  nc -z  <IP_ADDRESS> 12345
[..... long wait ....]
admin ~  ฿  echo $?
1

And here it is verbose
admin ~  ฿  nc -vz  <IP_ADDRESS> 8333
Connection to <IP_ADDRESS> 8333 port [tcp/*] succeeded!
admin ~  ฿  nc -vz  <IP_ADDRESS> 12345
nc: connect to <IP_ADDRESS> port 12345 (tcp) failed: Connection timed out

But you indicate you get 'success' for all ports. That should not happen.
Try the verbose (-zv) option on a closed port.

Ok, works great! Was not aware that the return value needs to be printed...

Any special reason you went with the == operator? I get the following error message:

./20-raspibolt-welcome: 117: [: 0: unexpected operator
When using the normal = operator, it works fine.

I think it was because I updated my raspibolt script to bash instead of /bin/sh
#!/bin/bash
In bash you check equality of STRINGS with ==.
And I converted $public_check to a string as follows: "$public_check"  (added quotes)
But by all means do what works. I am not an expert on shell scripting at all.
:-)

Updated the welcome script, thanks @robclark56!
PR https://github.com/Stadicus/guides/pull/121

Hello!
I have some issues checking the public availability of my node (on testnet) with nc -vz IP PORT. As my node try to access my public IP address from behind a NAT, he got confused. This is a classic networking problem.
Reading this blog post System: Accessing Public IP address from behind NAT I found what I first thought was a solution, which is adding a rule in iptables to convert my public IP to my local IP.
$ sudo iptables -A INPUT -p tcp --src <IP_ADDRESS> --dport 22 -j ACCEPT
$ sudo iptables -L -t nat
Chain OUTPUT (policy ACCEPT)
target       prot    opt          source                 destination
DNAT       all        --            thundroid            <IP_ADDRESS>            to:<IP_ADDRESS>

It seems to work but in fact it is not checking anything because the request never really leave my local network. So not a proper solution.
It seems that this nc call works seamlessly for some people but I can't figure out why it doesn't work for me. Any ideas? Thank you.


Update CI, move to GH actions

Testing on both pip and conda is deliberate to ensure that the application works when installed via either method, as advertised in the README.


declaring type alias causes unexpected  error

Roblox LSP doens't support full Luau syntax yet.
To use it, you have to install Selene too.
Read more here: https://devforum.roblox.com/t/roblox-lsp-full-intellisense-for-roblox-and-luau/717745

sorry


patroni db templates

Also some more points for next story:

Do we want to integrate creating Patroni as part of Jenkins job that creates hnsesb application resources?
Should we document the steps to run this template? Its minor only 2 commands but will that info be handy?

Also some more points for next story:

Do we want to integrate creating Patroni as part of Jenkins job that creates hnsesb application resources?
Should we document the steps to run this template? Its minor only 2 commands but will that info be handy?

Documentation added to confluence.
Agreed yes we should automated this in some capacity. Though I don't know if we need patroni for things like feature branches?


Albumartist

The backend sets whether 'album' or 'albumartist' is used (CLI flag). The frontend gets this flag via a new 'Config' message so it can use the correct column in the playlist tab (the frontend now also gets the MPD URL, since why not).
Currently there is a new additional tab 'AlbumArtist' in the GUI. The idea is that this goes back to three tabs ("Playlist | Files | Artists") and the frontend shows either the artists or the albumartists tree when the "artist" tab is selected, depending on the config.
The frontend now keeps track of three view trees: files, artists, and albumartist. This way things will work nicely when the backend is restared with a new config.
The two commits are likely best read standalone.

needs robx/elm-edn#11


how to prevent running muli-instance the same tauriApp

double click TauriApp icon will casue muli-instance running the same App, how to prevent it

I suppose this is more related to tauri and perhaps tao.
Please open an issue in tauri and follow the feature request template.


Output of -version flag is 'Unknown' when building from source

Description of the problem or steps to reproduce
Using:
go get -u github.com/zyedidia/micro/...
micro -version

Gives this:
Version: Unknown
Commit hash: Unknown
Compiled on Unknown

Specifications
You can use micro -version to get the commit hash. <- Not really possible :)
Commit hash: Unknown
OS: Kubuntu 16.04 with go version go1.6.2 linux/amd64
Terminal: Konsole

when micro is build without making use of the Makefile it does this.

Go does not allow build hooks when using go build or go get. If you want to build with the version information you have to use the makefile (make install). This is simply a limitation of Go.


API Changes: LWM2M

Task Description

[x] Do not use typedefs.
[x] Document what's a TLV
[x] Rename sol_lwm2m_client_info_get_sms() to sol_lwm2m_client_info_get_sms_number()
[x] Rename sol_lwm2m_notify_observers() to sol_lwm2m_client_notify()
[x] Rename sol_lwm2m_send_update() to sol_lwm2m_client_send_update()
[x] sol_lwm2m_tlv_get_bytes() should receive a struct sol_buffer as argument
[x] Rename sol_lwm2m_tlv_array_clear() to sol_lwm2m_tlv_list_clear()
[x] Rename sol_lwm2m_tlv_to_bool(), sol_lwm2m_tlv_to_int()... and so on to
sol_lwm2m_tlv_get_bool(), sol_lwm2m_tlv_get_int() ...
[x] Rename sol_lwm2m_server_management_write() to sol_lwm2m_server_write()
[x] Rename sol_lwm2m_server_management_read() to sol_lwm2m_server_read()
[x] Rename sol_lwm2m_server_management_delete to sol_lwm2m_server_delete_object_instance()
[x] Rename sol_lwm2m_server_management_execute() to sol_lwm2m_server_execute_resource()
[x] Rename sol_lwm2m_server_management_create to sol_lwm2m_server_create_object_instance()

Dependencies
None.

I believe this is closed, isn't it? :-)

@bsmelo Indeed. Closing


javlibrary.com doesn't work. It was working before.

I reninstalled manytimes and still doesn't work. javlibrary.com was working before now i can't open without vpn.
Please help.

RTFM: https://github.com/ValdikSS/GoodbyeDPI/wiki/How-to-make-traffic-dump-with-Wireshark

Doesn't seem to be a server-related issue. Works for me currently.


fix application scope for mocha acceptance tests

@adamjmcgrath FYI

tests are failing because it runs on node 0.12

lgtm


Not connected!

I use nodejs start the bleno, and it is ok, and then run downloaded code and it is always show "label" and seems not connected, any help?
Thanks.

@mmrobert, example works fine for me


Lock down branches / update settings for repo

I locked down branches.  I have not played with the "merge" settings as of yet since Rebecca is not here.

Changed settings to "squash and merge".  May need to get used to how this looks, but after much reading this feels like the best:

Rebase and merge - avoids a single "merge" commit which is ugly; however, every commit gets laid down in master, including interim commits addressing comments.  This makes for a really ugly commit history
Squash and merge - squashes all commits to a single commit.  Loses interim commits, but keeps the commit history relevant.  We have to be dilligent that the single commit that is merged is well documented.


Error when call RetinaFace.detect_faces

Traceback (most recent call last):
File "", line 1, in
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/retinaface/RetinaFace.py", line 90, in detect_faces
net_out = model(im_tensor)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/tensorflow/python/util/traceback_utils.py", line 153, in error_handler
raise e.with_traceback(filtered_tb) from None
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/tensorflow/python/eager/execute.py", line 55, in quick_execute
inputs, attrs, num_outputs)
tensorflow.python.framework.errors_impl.InternalError: Graph execution error:
Detected at node 'model/bn_data/FusedBatchNormV3' defined at (most recent call last):
File "", line 1, in
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/retinaface/RetinaFace.py", line 90, in detect_faces
net_out = model(im_tensor)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/engine/base_layer.py", line 982, in error_handler
def call(self, *args, **kwargs):
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/engine/base_layer.py", line 987, in error_handler
**kwargs: Keyword arguments to be passed to self.call.
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/engine/base_layer.py", line 1096, in call
outputs = call_fn(inputs, *args, **kwargs)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/utils/traceback_utils.py", line 92, in error_handler
return fn(*args, **kwargs)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/engine/functional.py", line 452, in call
inputs, training=training, mask=mask)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/engine/functional.py", line 589, in _run_internal_graph
outputs = node.layer(*args, **kwargs)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/utils/traceback_utils.py", line 64, in error_handler
return fn(*args, **kwargs)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/engine/base_layer.py", line 1096, in call
outputs = call_fn(inputs, *args, **kwargs)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/utils/traceback_utils.py", line 92, in error_handler
return fn(*args, **kwargs)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/layers/normalization/batch_normalization.py", line 767, in call
outputs = self._fused_batch_norm(inputs, training=training)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/layers/normalization/batch_normalization.py", line 624, in _fused_batch_norm
training, train_op, _fused_batch_norm_inference)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/utils/control_flow_util.py", line 106, in smart_cond
pred, true_fn=true_fn, false_fn=false_fn, name=name)
File "/home/ubuntu/anaconda3/envs/dev/lib/python3.7/site-packages/keras/layers/normalization/batch_normalization.py", line 613, in _fused_batch_norm_inference
data_format=self._data_format)
Node: 'model/bn_data/FusedBatchNormV3'
cuDNN launch failure : input shape ([1,3,1024,1222])
[[{{node model/bn_data/FusedBatchNormV3}}]] [Op:__inference_function_7605]

it seems an environmental problem
please re-try for the libraries with version "opencv-python==3.4.4", "tensorflow==1.9.0"

closed due to inactivity


Add babelrc

docs

@thejameskyle, I think I'm gonna close this as wontfix for now. I'm a fan of using the babelrc to configure stuff, but atm it would just be an empty file. I think most people won't need to configure babel, so it would just be a file they would delete.

The file wouldn't actually be empty, it'd be:
{
  "blacklist": ["useStrict"]
}

There's also the problem that if someone wanted to configure babel at all they'd have to update it in four places:

https://github.com/babel/babel-library-boilerplate/blob/master/gulpfile.js#L82
https://github.com/babel/babel-library-boilerplate/blob/master/gulpfile.js#L101
https://github.com/babel/babel-library-boilerplate/blob/master/gulpfile.js#L118
https://github.com/babel/babel-library-boilerplate/blob/master/gulpfile.js#L136

:+1: k – will add


feat: added unit test coverage make targets

Description
feat: added unit test coverage make targets
Verification Steps

Run make test/output/coverage/report without coverage.out file present in the root dir and confirm that coverage.out file not found was displayed
Run make test/html/coverage/report without coverage.out file present in the root dir and confirm that coverage.out file not found was displayed
Run make test and confirm that cover.out is generated and processed at the end of the test execution (certain files are removed from the coverage report (e.g. mocked files...))
Run make test/output/coverage/report with coverage.out file present in the root dir and confirm that coverage results are displayed in the terminal
Run make test/html/coverage/report with coverage.out file present in the root dir and confirm that coverage results are displayed in the browser

Checklist (Definition of Done)

[x] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has created for changes required on the client side

@VanillaSpoon ping.. want to have a look / review at this?

Looks good, thanks for the clear verification steps. Nice to see the implementation too :)

@machi1990 - can I get this PR approved given that it was verified?


Import ded98a003d: fix crash; added isValid check to pose (by Caitlyn)

This is an automated proposal to look at a commit made by Caitlyn and import it into Overte

Commit:
ded98a003d86d1658130ff812c1ab5a2aa9bbaf9

Author:
Caitlyn

Date:
Tue, 25 Feb 2020 00:46

fix crash; added isValid check to pose

Stats:

Filename
Stats
Lines
Added
Removed
Lines in blame

libraries/controllers/src/controllers/Pose.cpp

15
10
5
10

libraries/controllers/src/controllers/impl/endpoints/ScriptEndpoint.cpp

1
1
0
1

2 files
-
16
11
5
11

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
If the commit is not useful, tag it with "Tivoli: Discard", and close it.
If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.
You can cherry-pick this issue with this command:
git cherry-pick ded98a003d86d1658130ff812c1ab5a2aa9bbaf9

This looks reasonable, code needs looking into


fix & modify: fix pbft.list and removeCount, modify README.md

Codecov Report

Merging #241 into dev-1.5 will decrease coverage by <.01%.
The diff coverage is 15.78%.

@@             Coverage Diff             @@
##           dev-1.5     #241      +/-   ##
===========================================
- Coverage    14.14%   14.13%   -0.01%
===========================================
  Files          280      280
  Lines        19132    19139       +7
===========================================
  Hits          2706     2706
- Misses       16426    16433       +7

Impacted Files
Coverage Δ

libconsole/ConsoleServer.h
0% <ø> (ø)
:arrow_up:

libconsole/ConsoleServer.cpp
0% <0%> (ø)
:arrow_up:

unittest/libstorage/test_MemoryDB.cpp
97.3% <100%> (ø)
:arrow_up:

unittest/libprecompiled/test_DBPrecompiled.cpp
97.05% <100%> (ø)
:arrow_up:

libstorage/MemoryDB.cpp
95.96% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


tls: assorted checks - v1

Add checks for TLS missing fields from JSON schema or from the output (subjectaltname was missing from logs, certificate and chain were missing from the schema)
Ticket
If your pull request is related to a Suricata ticket, please provide
the full URL to the ticket here so this pull request can monitor
changes to the ticket status:
Redmine ticket:

https://redmine.openinfosecfoundation.org/issues/7287
https://redmine.openinfosecfoundation.org/issues/7332

Merged in #2099, thanks!


Add details to section on new things

In #9 I have added various things to the paper and it's (in very much a draft form that could/should still be ripped a part) a first step. I've put in some very loose details about some of the implications. This can be added to but at the same time, I'm aware that we might not want to put that much in this paper, @marcharper could you take a look and A) let me know what you think or B) have a go at it?
Basically, I think it would be great if Marc took a look before I kept going.

I've added some inline commands. Everything looks correct to me otherwise. :+1:


Something wrong with partial rotation

In the example torus, the result is wrong if one make the angle of secondary rotation more than π and less than 2π.
I think this occurs by the convergence of Newton method, however, perhaps there is a bug in the primitive modeling algorithm.

This issue is resolved in truck-modeling v0.1.1.


Replicating the results of the pre-trained models.

Thanks for releasing this amazing work!
However, I cannot replicate the results of the pre-trained models using the provided code.
The results after training with the provided code are

RVQ Reconstruction:

FID: 0.032, conf. 0.000
Diversity: 9.676, conf. 0.054
TOP1: 0.507, conf. 0.003, TOP2. 0.697, conf. 0.002, TOP3. 0.794, conf. 0.002
Matching: 3.024, conf. 0.009
MAE:0.042, conf.0.000

Text2motion Generation

FID: 0.148, conf. 0.006
Diversity: 9.576, conf. 0.079
TOP1: 0.504, conf. 0.003, TOP2. 0.699, conf. 0.003, TOP3. 0.795, conf. 0.002
Matching: 3.062, conf. 0.010
Multimodality:1.327, conf.0.044

May I ask where is the problem? Many thanks!

I just use the provided code and commands without any change.
vq training: opt.txt
mtrans training: opt.txt
rtrans training: opt.txt

I just use the provided code and commands without any change.
vq training: opt.txt
mtrans training: opt.txt
rtrans training: opt.txt
vq test: rvq_nq6.log
mtrans test: evaluation.log

Hi wenhao, thanks for your info.
I've checked the configures and corrected the scripts in our README. We used --gamma 0.05  to train the rvq. I just re-trained the rvq and got the following results:

Hope you will find it useful.

I just use the provided code and commands without any change.
vq training: opt.txt
mtrans training: opt.txt
rtrans training: opt.txt
vq test: rvq_nq6.log
mtrans test: evaluation.log

Hi wenhao, thanks for your info.
I've checked the configures and corrected the scripts in our README. We used --gamma 0.05 to train the rvq. I just re-trained the rvq and got the following results:

Hope you will find it useful.

Hi, after setting --gamma 0.05 I got a worse result:

I meet the same problem and get the same results with @weihaosky .
Have you solved the problem？ Thanks.

I meet the same problem and get the same results with @weihaosky . Have you solved the problem？ Thanks.

No. I still cannot replicate the results.

Have you solved the problem now? @weihaosky @aszxnm
Thanks.

the same for me...

I meet the same problem and get the same results with @weihaosky . Have you solved the problem？ Thanks.

No. I still cannot replicate the results.

The same for me

I just use the provided code and commands without any change.
vq training: opt.txt
mtrans training: opt.txt
rtrans training: opt.txt
vq test: rvq_nq6.log
mtrans test: evaluation.log

Hi wenhao, thanks for your info.
I've checked the configures and corrected the scripts in our README. We used --gamma 0.05 to train the rvq. I just re-trained the rvq and got the following results:

Hope you will find it useful.

Hi, after setting --gamma 0.05 I got a worse result:
net_best_fid.tar final result, epoch 36
        FID: 0.040, conf. 0.000
        Diversity: 9.598, conf. 0.097
        TOP1: 0.506, conf. 0.002, TOP2. 0.696, conf. 0.002, TOP3. 0.793, conf. 0.002
        Matching: 3.027, conf. 0.008
        MAE:0.039, conf.0.000

Have you solved the problem now? https://github.com/weihaosky.
Thanks.

I just use the provided code and commands without any change.
vq training: opt.txt
mtrans training: opt.txt
rtrans training: opt.txt
vq test: rvq_nq6.log
mtrans test: evaluation.log

Hi wenhao, thanks for your info.
I've checked the configures and corrected the scripts in our README. We used --gamma 0.05 to train the rvq. I just re-trained the rvq and got the following results:

Hope you will find it useful.

Hi, after setting --gamma 0.05 I got a worse result:
net_best_fid.tar final result, epoch 36
        FID: 0.040, conf. 0.000
        Diversity: 9.598, conf. 0.097
        TOP1: 0.506, conf. 0.002, TOP2. 0.696, conf. 0.002, TOP3. 0.793, conf. 0.002
        Matching: 3.027, conf. 0.008
        MAE:0.039, conf.0.000

Have you solved the problem now? @weihaosky @aszxnm
Thanks.

I just use the provided code and commands without any change.
vq training: opt.txt
mtrans training: opt.txt
rtrans training: opt.txt
vq test: rvq_nq6.log
mtrans test: evaluation.log

Hi wenhao, thanks for your info.
I've checked the configures and corrected the scripts in our README. We used --gamma 0.05 to train the rvq. I just re-trained the rvq and got the following results:

Hope you will find it useful.

Hi, after setting --gamma 0.05 I got a worse result:
net_best_fid.tar final result, epoch 36
        FID: 0.040, conf. 0.000
        Diversity: 9.598, conf. 0.097
        TOP1: 0.506, conf. 0.002, TOP2. 0.696, conf. 0.002, TOP3. 0.793, conf. 0.002
        Matching: 3.027, conf. 0.008
        MAE:0.039, conf.0.000

Have you solved the problem now? @weihaosky @aszxnm
Thanks.

No. I still cannot replicate the results.

Thank you all for your attempts to replicate the results. I just got some time to re-train the masked-transformer and res-transformer using our released code. Here are some results for your reference:

I used the rvq checkpoint I obtained here https://github.com/EricGuo5513/momask-codes/issues/27#issuecomment-1956264373. I used the following scripts to train the m-trans and r-trans:
python train_t2m_transformer.py --name mtrans_replicate --gpu_id 1 --dataset_name t2m --batch_size 64 --vq_name rvq_replicate

python train_res_transformer.py --name rtrans_replicate --gpu_id 2 --dataset_name t2m --batch_size 64 --vq_name rvq_replicate --cond_drop_prob 0.2 --share_weight

evaluation script:
python eval_t2m_trans_res.py --res_name rtrans_replicate --dataset_name t2m --name mtrans_replicate --gpu_id 1 --cond_scale 4 --time_steps 10 --ext evaluation_replicate

The above results are obtained from these scripts without any modification to this code base. The replicate experiments were done on a single RTX 2080 Ti GPU, torch==1.7.1. For the processed dataset cloned from the original HumanML3D project, please send inquiry to<EMAIL_ADDRESS>or<EMAIL_ADDRESS>

https://github.com/EricGuo5513/momask-codes/issues/27#issuecomment-2185009113
Does anybody knows the problem?

The same for me, and another confusing problem is the MPJPE of the reconstruction setting on the kit dataset:
The command I run is python eval_t2m_vq.py --gpu_id 0 --name rvq_nq6_dc512_nc512_noshare_qdp0.2_k --dataset_name kit --ext rvq_nq6, and I used the default pretrained checkpoint.
Similarly, when I test it on the Humanml dataset, the results is :
It is obvious that the MPJPE results on these two datasets differ by several orders of magnitude. Does anyone else have the same problem?

The scale is different. Check the Mean.py files.

OK，I will check it! Thank you!

https://github.com/EricGuo5513/momask-codes/issues/27#issuecomment-2198380894
Any other suggestions? Thanks!

I re-download the KIT-ML dataset from Link directly. While it still remains this problem. Maybe it is not the mean problem？Thank you in advance! @Murrol

Scale difference means that the scale of AMASS(HumanML3D) and kit data is different. Take an example, inch vs cm.
"the MPJPE results on these two datasets differ by several orders of magnitude" should not be an issue.

I re-download the KIT-ML dataset from Link directly. While it still remains this problem. Maybe it is not the mean problem？Thank you in advance! @Murrol

Scale difference means that the scale of AMASS(HumanML3D) and kit data is different. Take an example, inch vs cm.
"the MPJPE results on these two datasets differ by several orders of magnitude" should not be an issue.

Oh, thanks! I checked the range of  means from these two  datasets, the mean of KIT datasets  is three orders of magnitude higher than that of HumanML3D on average. Thank you! @Murrol

Thank you all for your attempts to replicate the results. I just got some time to re-train the masked-transformer and res-transformer using our released code. Here are some results for your reference:

I used the rvq checkpoint I obtained here #27 (comment). I used the following scripts to train the m-trans and r-trans:
python train_t2m_transformer.py --name mtrans_replicate --gpu_id 1 --dataset_name t2m --batch_size 64 --vq_name rvq_replicate

python train_res_transformer.py --name rtrans_replicate --gpu_id 2 --dataset_name t2m --batch_size 64 --vq_name rvq_replicate --cond_drop_prob 0.2 --share_weight

evaluation script:
python eval_t2m_trans_res.py --res_name rtrans_replicate --dataset_name t2m --name mtrans_replicate --gpu_id 1 --cond_scale 4 --time_steps 10 --ext evaluation_replicate

The above results are obtained from these scripts without any modification to this code base. The replicate experiments were done on a single RTX 2080 Ti GPU, torch==1.7.1. For the processed dataset cloned from the original HumanML3D project, please send inquiry to<EMAIL_ADDRESS>or<EMAIL_ADDRESS>
Hello, I have some doubts regarding the reproduction of results. Currently, the minimum FID value I am getting while replicating the results on t2m is 0.059, and other metrics are normal. However, after adding certain modules, the FID value is 0.047, but in your paper, the FID value is 0.045. I would like to know if this is an improvement? I am a bit unclear whether my modifications are effective, or if, in fact, my reproduced results are not correct.

My rvq replicated FID is also 0.032, has it been solved?

My rvq replicated FID is also 0.032, has it been solved?

Same question, my rvq replicated FID is similar, (0.33).
Using this rvq and given terminal command here to train mask transformer and residual transformer leads me to a generation fid only around (0.182).
But if  I use the given vq checkpoint to train both transformers, I can achieve a more similar result here
Hope we can have a solution soon since rvq quality matters a lot for generation here.


Push globus status into Influx database

https://terramonitor.workbench.terraref.org/status
Every 30 seconds, hit this endpoint and insert into InfluxDB. We can do this in python and have some code snippets we can use.

Example influx code: https://github.com/terraref/terrautils/blob/master/terrautils/influx.py

use requests to GET the url: https://terramonitor.workbench.terraref.org/status

write_points to influx for each of the categories returned in JSON

to get time for database:
curr_time = datetime.datetime.now().strftime('%Y-%m-%dT%H:%M:%S')
completed_ts = int(parse(curr_time).strftime('%s'))*1000000000

I use "time": str(datetime.datetime.utcnow()) (see https://opensource.ncsa.illinois.edu/bitbucket/snippets/04d9dae144f14925a64e49036ed38848)

Docker image built and deployed for next planting.


Remove Kyma upgrade capability

Description
This ticket is an outcome of the TASK.
AC

[ ] 1) Remove /upgrade/kyma endpoint
[ ] 2) Remove KymaOrchestrationProcessingQueue with it's steps
[ ] 3) Remove KymaVersion

@KsaweryZietara what will happen now if someone will upgrade Kyma clusters right now? I expect nothing should happen with success.

Kyma upgrade works, but nothing will be done.


gradient accumulation

what should i do to achieve gradient accumulation. thanks very much

Could you be more specifc? You want to forward multiple times and then do backpropagation once?

@hellock you are right. I want to forward multiple times for the purpose of increasing batchsize

Here is where backpropagation is done, and you can modify it as you want.


Support Playwright 1.33 & 1.34

Playwright 1.33 introduced significant changes to the inner workings of its native reporters.
As reported by @yaffol on the Serenity/JS Community Chat, this means that using the Playwrigt HTML reporter version 1.33 and above with @serenity-js/playwright-test 3.2.1 and below results in the following error:
Error in reporter RangeError: Invalid time value
    at Date.toISOString (<anonymous>)
    at RawReporter._serializeStep (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:184:33)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:176:56
    at Array.map (<anonymous>)
    at RawReporter._serializeResult (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:176:39)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:164:43
    at Array.map (<anonymous>)
    at RawReporter._serializeTest (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:164:29)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:149:40
    at Array.map (<anonymous>)
    at RawReporter._serializeSuite (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:149:26)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:148:42
    at Array.map (<anonymous>)
    at RawReporter._serializeSuite (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:148:28)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:148:42
    at Array.map (<anonymous>)
    at RawReporter._serializeSuite (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:148:28)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:104:21
    at Array.map (<anonymous>)
    at RawReporter.generateProjectReport (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/raw.js:103:28)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/html.js:98:34
    at Array.map (<anonymous>)
    at HtmlReporter.onEnd (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/html.js:96:35)
    at /Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/multiplexer.js:104:115
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Multiplexer.onExit (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/reporters/multiplexer.js:102:45)
    at Runner.runAllTests (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/runner/runner.js:75:5)
    at runTests (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/cli.js:135:138)
    at ji.<anonymous> (/Users/pvale/sc/mycrossref/node_modules/@playwright/test/lib/cli.js:48:7)

Playwright 1.33 has also introduced test step reporting in UI mode and the trace viewer that Serenity/JS could leverage:
https://www.youtube.com/watch?v=JeFD6rqDbBo

Closed by #1716, released in Serenity/JS 3.3.0 🎉


Package overwrites previous changes (force push) when a new update is released

What happened? What is the problem?
Packit overwrites everything in rawhide-update-pull_from_upstream when a new update is released. This happened with Bottles recently. Version 51.16 was released. It didn't build since upstream introduced some breaking changes. I pushed a commit on top of Packit's PR to fix that and allow the package to build. After that, I allow some time for others to review.
Shortly after 51.16 upstream released 51.17 fixing a few bugs introduced in 51.16. That resulted in Packit overwriting everything updating straight from 51.15 to 51.17. The history can still be seen in PR #15, but all changes are gone, overwritten.
What did you expect to happen?
Packit should be smart about pushing changes to its own active PR. Even without changes pushed by packagers to Packit's branch, Packit could just add a commit on top, instead of replacing the entire branch.
While I think not overwriting should be the default, at the very least there should be an option to disable the current behavior. I'd rather apply another commit myself updating to a newer release than have Packit overwrite my changes. Something like PR #16 is what I would expect.
Example URL(s)
Packit's PR: https://src.fedoraproject.org/rpms/bottles/pull-request/15
My PR with the changes I previously pushed to Packit's PR: https://src.fedoraproject.org/rpms/bottles/pull-request/16
Steps to reproduce
No response
Workaround

[ ] There is an existing workaround that can be used until this issue is fixed.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

On the standup we discussed we should make this behaviour configurable. The default is something we will probably still discuss and maybe ask other maintainers about, for now, we lean towards letting the current behaviour default.


update vscode 1.17.0 CPU usage at 100%  rg.exe  bug?

update vscode 1.17.0 CPU usage at 100%  rg.exe  bug?

Your report isn't very clear (not sure what rg.exe is), but on Ubuntu, I'm getting 100% cpu spikes just from typing.
I'm desperately looking for a way to download the version previous to this one.

file location  \Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg.exe

run as administrator the problem still exists.

Could you check what the command arguments are?

Also: Does this happen immediately after opening the window or only after using Ctrl + P to show QuickOpen?

If my project exists node_modules folder will appear cpu 100% of the situation

https://code.visualstudio.com/updates/v1_16
click update on left, the top of each page has download links for vs code

Hi, the same bug occours on my Surface Pro3.
Env
OS: win64
VSCode Version: 1.17.0
locale: Chinese
How to reproduce
Openning a npm package containing a node_modules folder with VSCode 1.17.0 and doing nothing else ，the CPU  usage of rg.exe will quickly grows to 80% above .
If I choose to wait for a min , the rg.exe will exit silently and the usage of CPU will fall down to 3% .

Everyone seeing this: Is this right after starting the application or only after using Ctrl + P (Cmd + P on Mac) to open a file?
Could you also check what command parameters rg is running with?

@chrmarti  Hi . sorry for the late reply.

Is this right after starting the application or only after using Ctrl + P (Cmd + P on Mac) to open a file?

It's right that the CPU usage grows quickly to 100%  after starting the vscode ,  but I can't reproduce the error when openning a file using Ctrl+P.

what command parameters rg is running with

"c:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg" --files --hidden --case-sensitive -g *.csproj -g *.sln -g *.csx -g **/*.csproj -g **/*.sln -g **/*.csx --no-ignore --follow -- .

the Surface Pro3  OS: Microsoft Windows [版本 10.0.15063] x64
however , when I used my desktop in my office in the morning today,I could not reproduce the same bug on my win7 x64 desktop : win7 professional x64 SP1（ 6.1.7601 ）
After going home , I come back to my Surface Pro3 and try to disable extenstions to see whether it is caused by some extension. Here's my result:
with ms-vscode.csharp installed , the rg.exe will takes a huge amount of CPU usage.
however , if I disable the ms-vscode.csharp extention ,opening a npm package containing node_modules folder will not cause the bug.
@617450941 @cipchk  do you get the ms-vscode.csharp installed?

My cpu no matter how long it is maintained at 100% in exchange for the old version did not appear this situation

@newbienewbie
thx, can be confirmed ms-vscode.csharp extension that will cause rg.exe of CPU usage, fixed this to disable it.
but not all projects, have to meet the following qualifications:

large composite project (includes FE & C#)
has node_modules

Could you run the following command in Powershell from your workspace folder and let me know what the output is?
(Measure-Command { & 'C:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg.exe' --files --no-ignore --follow --hidden -- . | Measure-Object –Line | Out-Default }).TotalSeconds

Also, what is the output of the following when run in Powershell while the rg.exe are running?
(Get-WmiObject Win32_Process -Filter "name = 'rg.exe'").CommandLine

@cipchk I don't have my surface pro3 at hand , so I can't test the command now .
While on the desktop in my office , the first command will issue a lot of errors (os error 3) that complain about “File system loop found” and  “path not founding ”.
To get a complete output  , you can redirect the stderr to a file like this :
(Measure-Command { & 'C:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg.exe' --files --no-ignore --follow --hidden -- . | Measure-Object –Line | Out-Default }).TotalSeconds 2>out.txt

can you load the complete out.txt ?
for the second command :

(Get-WmiObject Win32_Process -Filter "name = 'rg.exe'").CommandLine

the output is :
"C:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg.exe" --files --no-ignore --follow -

Is that the same to your output ?

@newbienewbie Yes.
my steps:

run (Get-WmiObject Win32_Process -Filter "name = 'rg.exe'").CommandLine
run (Measure-Command { & 'C:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg.exe' --files --no-ignore --follow --hidden -- . | Measure-Object –Line | Out-Default }).TotalSeconds 2>out.txt

I confirmed that the run second command will 100% cpu!

so, the out.txt content.

Could you rerun the Measure-Command... command without --follow, that will help us understand if performance would be better when not following symlinks?
Please run the Get-WmiObject command while VSCode is showing the problem (not when you run the other command, I was imprecise in my instructions).

@chrmarti There will be no re.exe 100% CPU usage if without --follow.
the all output content:
PS D:\psdm-FE> (Measure-Command { & 'C:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\r
g.exe' --files --no-ignore --hidden -- . | Measure-Object –Line | Out-Default }).TotalSeconds

Lines Words Characters Property
----- ----- ---------- --------
84062

1.217515

the --follow parameter into a key problem?

@cipchk Which version of npm did you use to install the node_modules? I'm trying to reproduce the problem.
I'm not sure yet if removing --follow will be the best solution in your case.

my env:

npm: 5.3.0
node: v8.5.0
system: win10 64bit
about vscode:

Version 1.17.0
Commit be377c0faf7574a59f84940f593a6849f12e4de7
Date 2017-10-04T23:45:46.804Z
Shell 1.7.7
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

and, the D:\psdm-FE\package.json content:
{
  "name": "ps-h5",
  "version": "1.0.0",
  "author": "@cipchk",
  "dependencies": {},
  "devDependencies": {
    "babel-preset-es2015": "6.24.1",
    "event-stream": "3.3.2",
    "gulp": "3.9.1",
    "gulp-angular-htmlify": "2.3.0",
    "gulp-babel": "6.1.2",
    "gulp-bom": "1.0.0",
    "gulp-buffer": "0.0.2",
    "gulp-changed": "3.1.0",
    "gulp-clean-css": "3.5.0",
    "gulp-concat": "2.6.0",
    "gulp-if": "2.0.2",
    "gulp-inject": "4.2.0",
    "gulp-livereload": "3.8.0",
    "gulp-minify-html": "1.0.4",
    "gulp-modify-file": "^1.0.0",
    "gulp-ng-annotate": "2.0.0",
    "gulp-ng-html2js": "0.2.0",
    "gulp-rename": "1.2.2",
    "gulp-sass": "3.1.0",
    "gulp-sequence": "^0.4.6",
    "gulp-sync": "0.1.4",
    "gulp-uglify": "3.0.0",
    "gulp-util": "^3.0.6",
    "gulp-wrap": "0.13.0",
    "lodash": "4.17.4",
    "map-stream": "0.0.7",
    "merge-stream": "1.0.1",
    "merge2": "1.0.2",
    "path": "0.12.7",
    "rev-hash": "^2.0.0",
    "run-sequence": "^2.2.0",
    "sort-keys": "^2.0.0",
    "through2": "2.0.0",
    "vinyl-file": "^3.0.0"
  }
}

All of the above.

@chrmarti
I found reason, GOD!
Use the cnpm install command to install the package will have rg.exe 100% CPU usage, but works with npm install.
I think it's a file name problem, eg: gulp package should correspond to gulp fold, but cnpm actually use shortcuts to _gulp@3.9.1@gulp fold.
I don't know the question of cnpm or vscode.

I can reproduce it with cnpm. Thanks!

Looks like you could use cnpm install --by=npm to avoid the symlinks and work around the problem if you have to use cnpm.

@chrmarti @cipchk @fengmk2 @dead-horse
use npm install

use cnpm install

Why?
I watched their folder contents。
npm
674  projects

cnpm
1488  projects

look thes：cnpm

It has a lot of shortcuts，Don't know whether I can optimize performance from here；

hi guys, cnpm use symbol link to speed up installation like pnpm, so can you found a way to fix this issue not by using cnpm install --by=npm ?

@dead-horse   Thanks！

Tracking this on ripgrep's side with https://github.com/BurntSushi/ripgrep/issues/633.

Same for me on Ubuntu, 100% CPU usage spikes just from typing, file search is painfully slow

@Pacu2 Could you open a new issue using Help > Report Issues?

so what is the the best way to solve this problem? just simply use cnpm install --by=npm?

@gy18505 Yes, for now cnpm install --by=npm seems to be the simplest workaround.

Same on MacOS using SSHFS/OSXFUSE mounted folder.

The upcoming 1.18 release has a new setting "search.followSymlinks" that you can set to false as an alternative workaround for this issue. (From #37000.)

下载 vscode1.18以上版本  Documentation for Visual Studio Code 1.18
安装完成后 文件>设置>首选项     搜索设置 search.followSymlinks ：false；

I am having this same issue.  The command line looks like this:

"c:\Program Files\Microsoft VS Code\resources\app\node_modules\vscode-ripgrep\bin\rg" --files --hidden --case-sensitive -g /*.csproj -g /*.sln -g /*.csx -g /*.cake -g **/*.csproj -g **/*.sln -g **/*.csx -g **/*.cake --no-ignore --follow --quiet -- .

This operation is completely inappropriate in my environment. By design, the node_modules subfolder of my project contains thousands of symlinks into thousands of nested folders.  It would never make sense for a tool to do a brute force search of this subfolder without an explicit request from the developer.
In my repro, I did not perform any searches using the search tab.  I simply launch VS Code and it immediately spawns rg.exe.
Has anyone figured out why VS Code is doing this?  It is it a plugin?  Is it a misguided feature of VS Code itself?

Workaround:  Deleting rg.exe seems to eliminate the problem without any apparent loss of functionality in VS Code.

I am running into the same probelm: the second I press ctrl+c it starts eating cpu.
What's cnpm? I just opened a directory using VSCode and have no idea what cnpm is.

@pgonzal Deleting rg.exe will break quick open feature.

Setting search.followSymLinks to false doesn't help.

I also have the issue of VScode running at 100% cpu when doing a file search.

I also have the issue of VScode running at 100% cpu when doing a file search.

@BurntSushi is this the same issue that you fixed in BurntSushi/ripgrep#633?  If so does VS Code just need to upgrade?

@pgonzal I think unfortunately there isn't a simple answer to that question. I guess the closest answer is "wontfix" or more like "unable to fix." There is just a fundamental limit on how fast a tool can process a directory tree containing 2 million entries. You'll need to read my comment to understand this issue from ripgrep's end. (Others are reporting issues even when symlinks are disabled, but I need a reproducible case outside of VS code on my end.)

I think I found a workaround: replacing the rg.exe binary by version 0.6.0 inside VS Code resources directory seems to have fixed the problem.

@warpdesign Now that would be interesting. Can anyone provide the exact command being run and a corpus to reproduce it with?

What might explain the renewed interest here: We updated to ripgrep 0.7.1 for the VS Code 1.20 released last week. VS Code 1.19 was based on 0.6.0.

@pgonzal The command arguments you posted indicate that the file search was triggered by an extension for C# that would activate for certain file types. Could you disable that extension and see if that works around the problem?

@pgonzal One other thing to try first would be to make sure you have search.followSymlinks set to false in your settings. You command line arguments show --follow which is added to follow symlinks.

@warpdesign Could you try running the original rg with the same command line arguments you posted from your project folder? Does that stop or slow down at some point? Any errors being shown?

This issue is for a specific issue affecting people using cnpm/pnpm. The fix for that will probably come with https://github.com/Microsoft/vscode/issues/34711 and with extensions not making wide-ranging searches over all of node_modules.
@warpdesign and anyone not using those tools, please open a new issue with specific repro steps. Also if you put old rg.exe into insiders 1.21, it will break, so make sure that search/quickopen works at all.

rg.exe is using 100% disk space also. Please let me know if there is any alternative, or is this is going to fix soon?

@shrivatsahosabettu AIUI, that's not disk space, but disk bandwidth. And rg.exe is only using a paltry 0.4MB/s in your screenshot.

I think we have addressed this issue with QuickOpen and text search when there are a lot of symlinks (cnpm/pnpm) in the folder being searched by adding the setting search.followSymlinks. The workspaceContains trigger will be covered with #34711.
For any remaining issues, especially regressions between releases, please open a new issue so we can follow up with you on the details. Thanks.

@chrmarti I think you are closing the issue too quickly: as I said above, adding search.followSymlinks: false doesn't fix the bug for me. And I don't have that much symlinks.
Also, I need to be able to search through symlinks.

@burntsushi. In the screen print only 2 items are running as of now. But some more are running below that also. One more thing I noticed is this will run for first 5-6 min of my VS Code launch where the disk will become 100%, after 5-6 min all the rg.exe will disappear and disk size will come down to 36-40.

@warpdesign Let's continue the investigation in a new GitHub issue. It sounds like your problem is not the same we discussed and fixed here.
When you open a new issue could also do the following and include the result: Try running the original rg with the same command line arguments you posted above in your project folder? Does that stop or slow down at some point? Any errors being shown?
Thanks.

@shrivatsahosabettu If this reproduces also with the setting "search.followSymlinks": false, please open a new issue so we can follow up with you on the details. Thanks.


Fix issue when tbrom settings.json file has no namedselection entry

Codecov Report

Merging #99 (69d18db) into main (d8a1a7c) will decrease coverage by 0.24%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
- Coverage   88.62%   88.38%   -0.24%
==========================================
  Files          11       11
  Lines        2294     2291       -3
==========================================
- Hits         2033     2025       -8
- Misses        261      266       +5


[v1.6] wip run with race detector

This change is

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please


💡 Alternative Design Idea

So I just had an idea that I might try out, but I figured I'd run by you to see what you think.
What if we just used rustdoc to generate the HTML file for the lib.rs file, and then parsed the HTML to modify the links to point to docs.rs. We might not even have to convert it to markdown because markdown does allow HTML inside it, as long as it doesn't do anything spooky.
Then we are depending on the reliability of rustdoc to actually generate the documentation and the links, knowing things like what's a struct, what's a method, and producing the proper struct.MyStruct.html style links instead of us having to use the search feature to find items.

I just verified that rust-analyzer is indeed a valid language server protocol ( LSP ) server which uses a JSON RPC over stdin and stdout. We should be able to interact with it simply by running the rust-analyzer command off of the system and sending/receiving JSON from it. I haven't tried it yet, but I think that we'd be able to use the workspace/symbol request and use the symbolKind field to get the info we need to construct accurate docs.rs links.


shouldMinify ignores @shield

hello,
i found a problem with this script.
i use defender for laravel and it uses @shield and @endshield
but the shouldMinify ignores this and prints the @shield as text on the web page.
what do i need to add to shouldMinify to fix this problem?

Sounds like this other package is doing something wrong to cause this bug. Please take it up with them.