Node package updates, passing tests

Hey @glennreyes :) Thanks for the dependencies bump!

Hmm, seeing some regressions with this PR.
Tests are failing for me with:
RangeError: Maximum call stack size exceeded
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:44:35)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
...

Looks like a Webpack issue reported here.
Also the local demo app doesn't seem to update the "Output" preview after language or text changes. Only the initial preview.
Any insight?

I just published v1.0.1 to node after manually testing for regressions. It's unfortunate that the issue between highlight.js and Webpack cause tests to fail but I have too much on my plate at the moment to dig into that further. Would welcome help in that area if you're willing, @glennreyes :)

Great, thanks @bvaughn! The test fails could have something to do with "highlight.js": "^9.3.0" as some class referencing got messed up and .hljs-variabledisappeared somehow. I would revert the changes for that back to 8.9.1 until we have a clean way to update that 👍

Unfortunately the issue occurs for 8.9.1 and 9.3.0 so I went ahead and used the latest release. (You can see on isagalaev/highlight.js/issues/895 that the issue was first encountered for version 8.8.0 I think)

Works everyfine on my machine (with node v6.0.1), hm:
07 05 2016 18:12:47.729:INFO [karma]: Karma v0.13.22 server started at http://localhost:9876/
07 05 2016 18:12:47.735:INFO [launcher]: Starting browser PhantomJS2
07 05 2016 18:12:48.247:INFO [PhantomJS 2.1.1 (Mac OS X 0.0.0)]: Connected on socket /#-_QOJ44wjbDgQoWVAAAA with id 16103533
PhantomJS 2.1.1 (Mac OS X 0.0.0): Executed 1 of 1 SUCCESS (0.006 secs / 0.002 secs)

Did some updates on my fork, probably you can check if its good with that version:
https://github.com/glennreyes/react-highlight.js

Heh, sorry, hadn't pushed my lint-fixed master yet :)
Let me try again with a fresh install then and see.

Let me know if you want another PR ;)

Not sure what's up. Just did a fresh install with "highlight.js": "^8.9.1" and npm test fails for me:
ERROR in ./~/highlight.js/lib/languages/autoit.js
Module parse failed: /Users/brianvaughn/Documents/git/react-highlight.js/node_modules/highlight.js/lib/languages/autoit.js Maximum call stack size exceeded
You may need an appropriate loader to handle this file type.
RangeError: Maximum call stack size exceeded
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:44:35)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
    at Parser.evaluateExpression (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:787:20)
    at Parser.<anonymous> (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/webpack/lib/Parser.js:66:16)
    at Parser.applyPluginsBailResult (/Users/brianvaughn/Documents/git/react-highlight.js/node_modules/tapable/lib/Tapable.js:48:50)
 @ ./~/highlight.js/lib/index.js 13:32-61
Child html-webpack-plugin for "index.html":

For what it's worth, the Webpack issue does seem to hint at the bug being nondeterministic:

Likewise, we have this problem building vector-im. We have to keep building until we get lucky. It sometimes takes 10 or more runs until it works. Not fun.

So it could just be that you're getting luckier than I am with your test runs.

I ran it 9 times just now and the 9th worked.

Hm, interesting. No idea at the moment 😅 Which node version are you running?

Have tried 5.3.0 and 5.9.1.

Those are the 2 I've been using with other projects. I'll give 6.1.0 a spin and see if results are any different.

Cha-ching. 6.1.0 seems to be much less flaky.

We should consider adding CI like Travis into this module the keep every desired node versions on track 😎

Yeah, that would be worth doing. Until now I didn't realize anyone but me was using this particular module so I haven't given it much love. ;)

I've been looking for some cool highlight syntax module for react and that one was the most funny one to hack 😎

Interesting to note that tests failed for Node 5.3.0 and 5.9.1 with highlight.js 8.9.1 and 9.3.0. Tests pass for Node 6.1.0 with both versions.

You sure? It's failing at the moment:
PhantomJS 2.1.1 (Mac OS X 0.0.0) Highlight should render content with basic markup FAILED
	expected '' to contain 'hljs'
	assert
	contain

That looks now wrong to me imo:
expect(highlightDOMNode.className).to.contain('hljs')

Sorry for the ambiguity. I'm talking about my local git repo. I haven't pushed everything.

(I'm in a Starbucks this morning with a slow wifi.) Just pushed.

Nice, thanks! 😎 Hope the coffee was great

Having issues now using this module:

Try to fix the react refs

Sorry but I'm not seeing this error locally or on the demo site: http://bvaughn.github.io/react-highlight.js/.
If you can reproduce it in something that you can share with me (show me) then file a bug and I can take a look at it.

PS. Not to point out the obvious, but do you have multiple copies of React loaded?

@bvaughn It occurs only when you import this module from another project. We should point the "main" property in package.json directly to the React component instead to dist/main.js

Ah, the problem is that package is pointing at a UMD build huh. That's not new with this release.

Filed issue #3 for this. Will try to get to it at some point. Busy this afternoon working on an upcoming react-virtualized release though.

Ah cool, thanks! But we've set libraryTarget: 'commonjs2' as output in webpack config, right?

Sorry, I misspoke. I was doing too many things at once over the weekend trying to finish up react-virtualized 7 and doing a js-search release also. I believe the problem is that React isn't configured as an external and so Webpack is dumping it into the dist as well.

Just published 1.0.2 with React and ReactDOM added to Webpack externals so they didn't get inlined. Give it another spin.

Wow that sounds awesome! I'll give it a spin this evening.


Frontpage

@jancborchardt @karlitschek took a while and it is hackish but here's a start.
open questions:

I feel it is pretty small now, I think that some more details about what this ownCloud thing is would fit just fine below the 'who uses ownCloud' part.

[x] add nice images for above the three text items
[ ] add nice image as background.
[ ] if that image has no screenshot of ownCloud in action, add one somewhere.

I like it.
A few things:

background photo is obviously a placeholder only
I think the question if we use a blue navigation header or leave it transparent depends a bit on the background photo that we choose. Let's wait for that and see what looks better.
Let's leave the social stuff (forum, twitter, ..) as it is for now. Separate discussion.
We can add some icons/images/illustrations above the three boxes. But please keep it small. I think the focus should be on the main photo and the text.
We need to add the hosted aspect. Let's change the middle box to "ownCloud keeps your files secure and private - you decide where you want to host your data and which server you want to use."

The rest is great!

Made changes. More spacing, white background. The three feature items all have a link (with a subtle mouse-over effect of dark blue and no underline) going to respectively /contribute, /???? and /features. Yes, those are not great matches. We ought to add a page on privacy and security to our site and change the features and contribute page. Right now, features essentially is "more information about ownCloud" and I contemplate linking the button in the top teaser there with a 'learn more' instead of to install.

More feedback:

the navigation header being transparent is good, and should stay like that
change the bright blue button color to the ownCloud blue #1d2d44
move the monthly newsletter button on the same line as the social g+, fb, twitter, rss icons
roughly double the size (or 150% at least) of text and button in the main area (slogan, text below, action button) as currently they float lost in space
lose the dot at the end
add »calendars« between files and contacts in the second sentence
a bit more vertical whitespace between each of the sections below (3 boxes, customers etc)
I’m fine with leaving the social section for now, but please let’s cut it to 2 twitter posts, and 5 each of forum and news. In the future I would even cut the forum so we can focus on promo here – twitter and blogs
make the customer logos just a bit smaller, more horizontal whitespace between them, and add ~3 more so they fill the same width as the 3 boxes above.
add exactly 1 link on the bottom of each of the 3 boxes.

first box: »Join our community« to https://owncloud.org/contribute/
second box: »Install your own« to https://owncloud.org/install/#instructions-server
third box: »Browse apps« to https://apps.owncloud.com/index.php?xsortmode=high&page=0&xcontentmode=920x922x923x924 – we really need to redesign this and integrate it into owncloud.org

we need somewhere on the frontpage where we put a screenshot still – nothing is more frustrating than a nice frontpage which doesn’t actually show you how the hell the product looks like. ;) I’d say between the customers and social stuff. Not sure about the exact layout yet.

I did add some more customers. The customer logo's are either this size, or miniscule. That is the nature of the layouting using bootstrap and I'm not sure if I really can change that without truly advanced hackery. Help welcome...
About the linking - the entire thing now links to it on mouse-over, I guess you prefer explicit links, that's easy.
contribute - sure.
that skips the option to choose a provider (!) and thus locks a large part of our userbase out. Let's just link to owncloud.org/install instead.
I don't feel good at all about liking to apps.owncloud.com - not only because it doesn't look great, but we should FIRST tell them what awesome features we have already, and capabilities like external storage (!) encryption, versioning and all the other awesome isn't even explained at all yet. I think most of those are FAR more interesting for most users than the apps, so I'd rather first send them to owncloud.org/features before the app store. We could, instead, add a prominent link to the app store on the bottom of the feature list on owncloud.org/features.
screenshot - I agree. Could be in the main picture, a phone with ownCloud Mobile lying on the table, a laptop (Frank's Macbook, whoah) with the web UI fullscreen... Would work for me.

[ ] I'd like to add three more customers to make it an even 12 (that way it looks best in terms of responsiveness) and I'd like to ask for suggestions on the talk ML @karlitschek

Add customers:

[ ] Investment Planning Council
[ ] Joy Global
[ ] Health Management Associates

@karlitschek as far as I am concerned - the background is now the main thing still open.

@jospoortvliet can you post a new screenshot? Some details again:

[ ] the second line (below the main sentence) should be a single line. To achieve that, convert the two »and« into »&«. If it still exceeds the width, make the font size a tiny bit smaller until it’s only one line.
[ ] the Download button on the top right should also have the background color #1d2d44, just like the main button
[ ] »The latest Twitter/Forum/News« should not be bold. Also, is this really Open Sans?
[ ] the headers for »The latest …« should be center aligned and use the same heading / text sizes as the 3-block of »open community, privacy first, capable cloud« headers
[ ] the headers should be simpler: »Community«, »Privacy«, »Flexibility« maybe?
[ ] typo: »… all you need to live your digital life«

made the 2nd line a bit shorter but - if it goes to two lines or not depends on the width of the screen ;-)
done
everything should be Open Sans... Can't speak for the actual feed.
done
done
done

Here's your latest screenshot. Any more changes?
BTW as part of this change, all pages on the site will go to white background and the buttons which had white backgrounds go to light gray so as to not look bad.

@jospoortvliet Nice. Just one question: Is this the final background image? Frank mentioned this one was only temporary, and it makes the subtitle difficult to read.

@danimo yeah it's a temp pic, Frank promised to make a nice one.

made the 2nd line a bit shorter but - if it goes to two lines or not depends on the width of the screen ;-)

@jospoortvliet I know, but it should look fine as long as the width of the screen is wider than the main site container. ;)

[ ] actually for the background color of the dark blue buttons, use #35537a which is a slightly lighter and friendlier blue and also corresponds with the primary action button color in ownCloud itself
[ ] the button background for the formerly white buttons should be rgba(240,240,240,.9), with a border of 1px solid rgba(240,240,240,.9). Just like in the ownCloud web interface.
[ ] can you reduce the size of the TU Berlin logo a bit? Currently it sticks out a lot.
[ ] we should really add a screenshot or basically the image which is on the frontpage now, between the org customers and the social stuff

That blue is SO MUCH BETTER!
Yes, much nicer!
Done. Also reduced CERN and Uni of Hawaii, now it all looks much nicer.
I could slap it in the middle but that's weird. Why don't we, first, try to see if we can have this in the big background image? putting devices on the table which show the user interface should not be hard...

putting devices on the table which show the user interface should not be hard...

Yeah, works for me. That’s a good combination and keeps the content focused.
Can you take a new screenshot then? It seems that everything is fine except the image then, but just to make sure. :) Also cc @owncloud/designers for review.

@jancborchardt we would need a photo first... Eg something like the many pics here: https://placeit.net/stages/ipad-and-iphone-6-mockup-at-a-corporate-office-a5489 and then put the ownCloud UI on there. I can gimp stuff in, easy enough, but we'd need a base pic. @karlitschek promised to make one - that's what we wait for, then. Or we buy a stock one, those on the site I just linked to are USD 8-15 so not particularly expensive...

How about giving it all a little space to breath:

@Henni good call, a bit less space would be even better maybe. Also I guess it should have the same width as the rest of the page so it doesn’t overflow on other widths.
Maybe it’s best to only put 5 organizations per line? Then we can have the proper space. @jospoortvliet? :)

@jancborchardt bootstrap is limited to 2-4-6-12 rows. Well, you can have other numbers of rows, but then it doesn't get more spacing, just margins left and right... At least as far as I can figure that's how it works. getbootstrap.com/css/#grid

Looked around for stock images that look kind of like what we're looking for. Or at least, what I am thinking off. They all have the problem that they have whites in the wrong places.
We will need a picture with an area in the middle which is NOT white (empty table?) and the devices have to be on the sides somehow as the top also needs a dark background.
The text shadows help a bit but not enough...

@jospoortvliet the second screenshot would work if you blur the background a bit more (or make it darker) and put a gradient from dark to transparent behind where the navigation is.

That is, a very slight gradient, like Youtube use for their video controls.

Guys. I'm working on a picture. But I need a few day. Relax please :-)

@karlitschek but we're so anxious to see what you come up with ;-)
/me waits impatiently

Last two commits implements most changes asked for in #780

Here's the new image. I've cropped/resized it to 1280x720 and worked it in the gimp jpeg export to 150kb. That's a bit much still but if I go lower it really gets ugly imho with the jpeg artifacts. What do you think?

Hmm. The text should stay in the dark area for readability. Not sure we can move stuff around to make this happen. @jancborchardt What do you think?

How about loading different images depending on screen size? This way mobile devices don't have to download the large image file and could be served a smaller size instead.
This would be quite easy to implement, as this image is included via CSS.
Because of this we don't have to use any new fancy picture element, but can simply apply media-queries.
I would suggest adding media-queries for:

mobile (maybe iPhone6)
tablet (iPad)
desktop (1280x720 or 1920x1080)
Retina

@Henni good suggestion. Let me give it a try. I'll have to make different versions of the image but I've already been playing with it and I think I can ;-)

I think the text overlapping the image is fine since the background is still pretty dark there. Cropping or linebreaking would look strange there.
And wow, that image looks really good! Great color of the table, fits ownCloud. And the red/orange accent color draws attention to the actual ownCloud screenshot/interface.

Maybe just moving the image a tad to the left (like 50px) would already give the main tagline more breathing space.

I've done some CSS magic. Here the page on a few different sizes:
~1440p:

~1080p:

~900p:

~720p:

I think that that should be fine for desktops.
But mobile is another story. I tested on my Android and it wasn't pretty:

After some custom css and creative image editing, this is kind'a the best I can do on mobile. The left represent what you seen on portrait, the right is landscape on my 1080P 4.7" Android phone (no screenshot visible, unfortunately)

This is AFTER creating a version of the image which is quite distorted for the portrait view to ensure it would still show something... I know, you don't really see the distortion because I used liquid rescale plugin in Gimp extensively.
Ok, I edited the CSS to show this on landscape on my Android (quite nice, I think):

Real look on the phone (scrolled down a little):

Honestly, I think we need a new photo specifically for mobile. More space to the top, items closer to each other on the bottom, at least one phone almost centered so a what ownCloud looks like is always visible.
But I wouldn't oppose already going live this way.
Thanks to @camilasan for some of her css magic and cleaning up stuff.

Oh and note, to save bandwidth, it loads different images for mobile, 720p, 1080p and 1440p.

@jospoortvliet nice! Just one thing: When the second sentence breaks, it should break so the second line is »from any device, on your terms« to have each row make sense on its own. Also, the line-height should be increased a little – line-height: 120% or so, if that’s too little 130 maybe.

@jancborchardt linehight I can do but no idea how to change the breaking of the line... I can try to fine-tune the sizes but I bet that it will look just enough different on different devices that it doesn't really solve the problem. Any suggestions?

@jospoortvliet crazy concept: put the second row in a <span style="white-space:nowrap;">. Then if any word of that part of the sentence wraps, the whole part will wrap.

@jancborchardt I was faster ;-)
Having fixed this last thing, are you :+1: this?

@jospoortvliet I also thought about that wrapping technique, but the most important is actually that it properly splits into 2 big parts when the size is just slightly smaller. Can you add my span thingy on top? Then I’m all :+1:’s up in here.

@jancborchardt I did that in the commit I pushed. I did do it a little more granularly, tested it - this works really nicely, only wrapping in an annoying (but unavoidable) way when the window is real small. You will be able to test on staging. I'll try to get this merged today so you can give it a try.

It's on http://staging.owncloud.org - feedback welcome.

@jospoortvliet nice! :) 2 things:

[ ] on the narrowest mobile view where there are just 2 rows of customer references, we should cut down the number of logos to 4 or 6 max. Otherwise it takes too long to scroll by.
[ ] the Android screenshot mockup on the right can be improved: 1) use latest Android, not Holo 2) Use a more engaging screen than sharing settings, like a grid view of photos 3) Remove all the notifications in the header bar.

I just want to thank you all for a very nice look and the work you've done here.

Very nice! The only feedback I have is that on 1080x1920, the white ownCloud logo on the top drowns in the grey-white newspaper:

Not sure if this can be fixed e.g. by a CSS hack showing a different logo that contrasts better.

@jospoortvliet did you see my feedback? :) When do we push this live btw? We can always iterate further as soon as it’s live.

@jancborchardt I will do the screenshot thing tomorrow and see if I can fix the thing @danimo mentioned. Then if @karlitschek agrees, I will deploy.

yes. let's push this out and do incremental improvements

Aaaand... it went live here: https://github.com/owncloud/owncloud.org/pull/800 ;-)


Getting error when trying to run with Expo and react native

I'm getting the following error, any idea what is causing it?
`Invariant Violation: Tried to register two views with the same name RNGestureHandlerButton
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:95:4 in reportException
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:141:19 in handleException
at node_modules/react-native/Libraries/Core/setUpErrorHandling.js:24:6 in handleError
at node_modules/@react-native/polyfills/error-guard.js:49:36 in ErrorUtils.reportFatalError
at node_modules/metro-runtime/src/polyfills/require.js:203:6 in guardedLoadModule
at http://<IP_ADDRESS>:19000/node_modules/expo/AppEntry.bundle?platform=ios&dev=true&hot=false&strict=false&minify=false:196673:3 in global code
Invariant Violation: "main" has not been registered. This can happen if:

Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called.
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:95:4 in reportException
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:141:19 in handleException
at node_modules/react-native/Libraries/Core/setUpErrorHandling.js:24:6 in handleError
at node_modules/@react-native/polyfills/error-guard.js:49:36 in ErrorUtils.reportFatalError`

Hello @minifisk, sorry for the late reply. I'm currently unable to replicate this issue. Can you provide me with some more info about the setup?
In particular I'd like to know:

If you are already using react-native-gesture-handler in your project. In that case, which version?
What Expo SDK version are you using?

It would be great if you could setup a snack on Expo.dev where the error is reproducible. This would be of great help for understanding the issue better.
Thanks.

closed due to non reproducibility

Having the same issue with "expo": "~48.0.6" and no other react-native-gesture-handler dependency in the project


TIM Italy and TIM Brasil contributions proposal

TIM Italy and TIM Brasil reviews points and contributions proposal:
Suggest to converge on a YAML from previous proposal A+B+C:
A) Proposal by TEF (Telefonica Proposal #3)
B) Proposal by DT (Initial content for Number Verify #2)
C) Proposal initially by TIM Brasil (IP and Port header input parameters)
Review points include in new YAML file version regarding path /authorize:
IP and Port as optional input header parameters
oAuth2 already mapped in DT (Initial content for Number Verify #2)
Error code handling according to CAMARA others API (e.g., the Traffic Influence API)

Hello @paulocesardiaslima.
Please find my comments here:
https://github.com/camaraproject/NumberVerification/pull/2#issuecomment-1321737568
We have been discussing in several meetings about what the API should include and the idea is to have an independent simple API with specific resources (and NOT general resourses such as /userinfo that offers MC) where the authentication/authorization and getting the token will be handled outside of the API and the API just handles the verification part. The goal is not to define an authentication flow but a verification API.
Please check out the comments in this PR. From Telefonica we're against duplicating MC solution. It doesn't follow the commonalities decisions and it's against the whole idea of CAMARA.


getKeyBIP44(..) for bitcoin coinType is incorrect

Describe the bug
getKeyBIP44(..) for a .bitcoin coinType is incorrect and is using the BIP88 purpose instead.
To Reproduce
let wallet = HDWallet(mnemonic: "swarm swap wagon attitude roof inquiry all file reform nose put census", passphrase: "")
let coin: CoinType = .bitcoin

print(wallet.getKey(coin: coin, derivationPath: "m/44'/0'/0'/0/0").data.hexString)
// CORRECT: 8b1ddc92d9efff6dfc80b848c585258afb23098918cfe10a5f5545e7ec515a89

let key = wallet.getKeyBIP44(coin: coin, account: 0, change: 0, address: 0)
print(key.data.hexString)
// INCORRECT: 55a726045fc154d91e6b94b0f990b85322ec93cadce843cfb7b266c0da1d2a46

// This looks more like getKeyBIP44 actually is getKeyBIP84
print(wallet.getKey(coin: coin, derivationPath: "m/84'/0'/0'/0/0").data.hexString)
// 55a726045fc154d91e6b94b0f990b85322ec93cadce843cfb7b266c0da1d2a46

Expected behavior
getKeyBIP44() should return the private key using Bip44 purpose, not Bip84 purpose.

it's not a real bug, the API name is a bit misleading, it gets purpose from coins.json which is 84. we will improve it later

A bit misleading indeed! Ok, how should we rename it?

getDerivedKey


Web Socket connection getting closed

We are using istio 1.9.3 , We enabled debug logs for istio-proxy sidecar  whenever after new listener config load connections are getting drained.
After the below log we see websocket disconnect
021-06-23T10:54:05.796256Z debug envoy config removing draining listener: name=<IP_ADDRESS>_4194, hash=7697804453189935468, address=<IP_ADDRESS>:4194
got postStop: 561548d5-f98f-46df-bf20-077f3f3b6fce, cleaning up registry...notifs-85dcb6d677-gwfngJun 23rd, 16:24:06.357

Changed KeepAliveMaxServerConnectionAge to 24h , that's solved the issue. Just wanted is there any alternative way or side effects of this change?

@anannaya how long did the connection remain open for before being closed?


Support for node and edge sets

Addresses #6.
I know there is already a PR open for this, but I think this is a less redundant implementation for the node sets. I noticed that the tocellsets method is quite general in terms of dim. Supplying the global node number from gmsh directly gives the proper node set definition. So the function can be reused.
Some questions regarding Ferrite grids:

What are vertex sets? I noticed them in the documentation. I guess this is just a different name and they are interchangeable with node sets.

I assume node sets can be created independent of the dimension in gmsh. They always exist and are of dim=0.

 So, for dim=2, they are equal to node sets, so I would ignore them in this case and only extract edge sets for dim=3.

[x] Support node sets

[ ] Support edge sets for (dim == 3)

[ ] Add test cases

Not sure how to deal with tonodes now returning two arguments. It makes sense to do it because toelements does the same, but it breaks compatibility.

Recently we have changed the definitions of edges and faces in Ferrite, see description [...]

So, what's your opinion on downward compatibility here? Is it sufficient to just support the latest definitions of Ferrite?

I think it's sufficient to use the latest definition and make a compat entry for a new FerriteGmsh version that it only supports 1.0 in the new release

Two things I'm currently uncertain of.

How to treat 1-dim physical groups for 3d geometries? They can be defined in gmsh and are edgesets in Ferrite. However, edgesets can't be stored in the grid. Should they be returned eg. by the togrid function as an extra return value?
How to deal with nodes vs. vertices? Every vertex is a node but not every node a vertex. We could keep the information twice. So each 0-dim physical group exists as a node and a vertex set (if applicable). Alternatively, everything that is already in a vertex set is not required to be stored in a nodeset. Not sure what the better approach is regarding the intended usage of nodesets in Ferrite.

How to treat 1-dim physical groups for 3d geometries? They can be defined in gmsh and are edgesets in Ferrite. However, edgesets can't be stored in the grid. Should they be returned eg. by the togrid function as an extra return value?

I'd suggest the following: either we add a docstring to todimEntitysets that explains how one can use it for extracting edgesets or we include a separate function that returns a Set{EdgeIndex}. If that exceeds the scope of the PR then I'll open a issue with these things. After https://github.com/Ferrite-FEM/Ferrite.jl/pull/914 edgesets were removed and it's unsure if they will return in Ferrite.Grid. So introducing a changing interface seems odd to me and either we don't offer this functionality or we go the route with a little bit of additional effort with a separate function

How to deal with nodes vs. vertices? Every vertex is a node but not every node a vertex. We could keep the information twice. So each 0-dim physical group exists as a node and a vertex set (if applicable). Alternatively, everything that is already in a vertex set is not required to be stored in a nodeset. Not sure what the better approach is regarding the intended usage of nodesets in Ferrite.

I think if you query 0 dimensional entities from GMSH you always get a nodeset, right? I'd just include it for now as a nodeset, not sure what the use case of vertexsets would be in that case.

@koehlerson @KnutAM, have you guys had a chance to review this?


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

@mergify merge


Add links for 'Transform and apply a function' in APIs with minor corrections

Codecov Report

Merging #1448 into master will decrease coverage by 2.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1448      +/-   ##
==========================================
- Coverage   94.00%   91.81%   -2.20%
==========================================
  Files          36       36
  Lines        8346     8345       -1
==========================================
- Hits         7846     7662     -184
- Misses        500      683     +183

Impacted Files
Coverage Δ

databricks/koalas/frame.py
92.31% <ø> (-3.26%)
:arrow_down:

databricks/koalas/series.py
96.24% <ø> (-0.12%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.32% <0.00%> (-70.28%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50.00% <0.00%> (-50.00%)
:arrow_down:

databricks/koalas/__init__.py
78.84% <0.00%> (-13.47%)
:arrow_down:

databricks/conftest.py
88.67% <0.00%> (-7.55%)
:arrow_down:

databricks/koalas/namespace.py
84.40% <0.00%> (-1.62%)
:arrow_down:

databricks/koalas/utils.py
95.43% <0.00%> (-1.02%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0.00%> (-0.96%)
:arrow_down:

... and 4 more

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 8337888...8ba7908. Read the comment docs.


train/valid/test split for MagnaTagATune

Hi,
thanks for sharing the pretrained models!
I am currently trying to figure out which train/valid/test split you used. You linked to this repo where I can't find any information about splits. They only mention a split of 13:1:3 which I first thought would be the folders but there are 16 folders and not 17 ...
Did you process the tags as proposed there?
I found this split but there are only 15244 train samples and you mentioned that your version of the dataset contains ~19000 train samples.
I assume you downloaded the audio and csv with tags from here.
I hope you can shed some light on my confusion :) Thanks!
Best regards,
Verena

In the musicnn-training repository you have code to create the MTT partition: https://github.com/jordipons/musicnn-training/blob/master/aux/mtt/partition_gt.py (line 24).
It is important to know that the MTT partition we use is different from the one Jongpil Lee used: https://github.com/jongpillee/music_dataset_split/tree/master/MTAT_split. I clarify this issue in our ISMIR 2018 paper: http://ismir2018.ircam.fr/doc/pdfs/191_Paper.pdf (section 5.2).
And yes! I downloaded the audio from here: http://mirg.city.ac.uk/codeapps/the-magnatagatune-dataset


Generating Eclipse project files with Maven fails due to missing dependencies

Following the build instructions in README.md, generating the Eclipse project files with Maven fails.
Executing mvn eclipse:eclipse produces the following error:
[ERROR] Failed to execute goal on project edu.kit.iti.formal.pse.worthwhile: Could not resolve dependencies for project edu.kit.iti.formal.pse.worthwhile:edu.kit.iti.formal.pse.worthwhile:eclipse-plugin:0.1:  Failure to find  edu.kit.iti.formal.pse.worthwhile:edu.kit.iti.formal.pse.worthwhile.model:jar:0.1 in http://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
Full output of the maven run can be found at http://pastebin.com/yfrKLQza

I think the problem is that maven-eclipse-plugin, responsible for generating the .project files does not use the maven-tycho-plugin dependency resolution mechanism and therefore tries to use a jar from the local repository instead.
I've decided to adopt a quickfix for this chicken-egg problem (no Eclipse project without successful build of artifacts, no model build without Eclipse project to run the genmodel-Workflow). The generated source files will be added to the repository, making all artifacts buildable without an Eclipse environment.


:heavy_plus_sign: Add selected theme as docs dependency

Try to get failing docs running.
Theme dependency is missing.

The website will only be rebuild once we merge... If there is another error it might take a while to update everything.


[refactor] destroying process

Context
See #167. And now, Segmentation Fault occurs when destroying texture.
Summary
destroy resources (e.g. client, backend, texture) before destroying server.
How to check behavior
start zen and exit, check no error occured. Leave comments if you expect to destroy something before exiting.

When I terminate zen without starting any client, it causes a segmentation fault. Please let me know if you can't reproduce this.
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7dcceaf in wl_list_remove () from /usr/lib/libwayland-server.so.0
#0  0x00007ffff7dcceaf in wl_list_remove () at /usr/lib/libwayland-server.so.0
#1  0x000055555555ec6c in zn_scene_destroy (self=0x5555556e12d0) at ../zen/scene/scene.c:278
#2  0x000055555555c6ec in zn_server_destroy (self=0x55555558a580) at ../zen/server.c:356

That change will solve #153


restyle the respository section in the review page, and possibly the details page.

a couple things we should change:

the width of the column containing "repositories" should never be so narrow that it needs a second line.


Automate population of publicationsInfo and "Published in DCP"

Background
The publicationsInfo field was added to the project metadata for use in the project catalogue so that we can add information about e.g. a publications journal title. The wrangling status "Published in DCP" was added so that the catalogue could filter by those in the DCP. These fields were populated by a script that looked up the information from euro PMC and azul, repectively. The intention was to run this script periodically via a cron job or whenever a new project was created, this was never done. So now any new projects in the catalogue will not have the journal title or potentially the correct wrangling status.
Task

[x] Automate the population of "Published in DCP" in the wrangling field. Cron job with the same schedule as DCP release schedule?
[x] Automate populations of publicationsInfo with cron job
[x] Allow for bulk updating of wrangling field and publicationsInfo without cron job
[x] This relates to #363 and we may need to take that into account
[ ] Use script to bulk update prod and staging

maybe task but maybe separate ticket

[ ] Use the UI and the DOI service to automatically populate publicationsInfo with journalTitle in the background so that any new projects will have the correct info
[ ] Add a UI element for adding journal title to ingest UI and have it populate publicationsInfo

Relates to #426
We need to introduce and populate a date field in publicationsInfo in order to filter by publication date

comments from Alegría will be addressed today by @jacobwindsor


KAFKA-10209 fix flaky connect_rest_test.py

There are two new configs introduced by 371f14c3c12d2e341ac96bd52393b43a10acfa84 and 1c4eb1a5757df611735cfac9b709e0d80d0da4b3 so we have to update the expected configs in connect_rest_test.py
issue: https://issues.apache.org/jira/browse/KAFKA-10209
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[false]
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[true]
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[true]

they are unrelated to this PR and traced by #8963

ping for reviewing this straightforward PR :)

What I don't like too much about this test is that it fails on optional configs. But on the other hand I guess it's good to have a test for a complete listing, it's just that this is failing after a merge. Thanks for the fix. I confirmed by running this system test locally. Unit/integration tests are not relevant here.

Also, note this is not flaky. It's failing consistently.

Merged to trunk and cherry-picked to 2.6

Also, note this is not flaky. It's failing consistently.

Got it. I will give more accurate title next time!


Continuation section too long

I get this error when try to compile a simple script to open a chrome window.

Hello,
I am unable to reproduce the error. If you are using compression, try turning that off and try again. Otherwise please also provide your AHK and Windows versions.

This was an oversight from my side, i tried to save ahk the file by right click and save it rather than opening it and copy the contents and save it.
Thats why it added additional HTML code to the file which failed to run. You can completely delete this report, as there is no actual issue. Sorry about the trouble.


Use H, not escape, to go to the Home view.

The escape key has always been a dangerous choice
for going to the home view.  People tend to hit
escape aggressively as basically a no-op key, but the
behavior of going to your Home view can be very disorienting
for folks.
The key has not only led to usability problems, but it
also complicates the codepath for dealing with Escape (already
a complicated problem), and it has led to bugs in the past.
Using H will definitely be an adjustment for some users,
but we are starting to position the interleaved view as
an advanced feature, and this is a good trial of that
concept.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

I think before we can merge changes that de-emphasize the home view, we need to make advancing the pointer less dependent on the home view.  In particular, update_message_flags should advance the pointer when you mark messages ahead of the pointer as read, which would be the sane replacement for the "advance pointer when hitting escape to unnarrow after reading things" logic we currently have on the frontend.  I think it's probably high time to do that change, regardless, since it'll address other bugs as well.

Oops, misclicked; didn't mean to close.

Changed title to have "wip" in it.

Heads up @showell, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the upstream/master branch and resolve your pull request's merge conflicts accordingly.

I think @rishig's work on renaming "Home" to "All messages" is probably the right path forward here; we can pull up this PR later if we need it.


Download URL is broken in brew

Hi,
I don't know if it's the right the place but download link in brew is broken. I got the error message below:
 ~/ brew reinstall openfoam2206
==> Downloading https://github.com/gerlero/openfoam-app/releases/download/v1.8.1/openfoam22
curl: (22) The requested URL returned error: 404

Error: Download failed on Cask 'openfoam2206' with message: Download failed: https://github.com/gerlero/openfoam-app/releases/download/v1.8.1/openfoam2206-app-homebrew-.zip

Good catch. Cask openfoam2112 is broken in the same way. Brew's suggested syntax update in #37 and #38 got it wrong, so that the arch variable is not actually set, leading to a bad URL. The right way to define arch in a cask seems to be as in https://github.com/Homebrew/homebrew-cask/pull/129480.

Should be fixed now (a brew update might be needed to get the new definitions). Thank you @volkan-a!


Position problems

Hi, I have a few question :
1- Integral sup and sub elements are not correctly positioned for my case. Or wide hat too close to its content. Is there an easy way to change positions?
2- Is it possible to add new symbols by using unicode and svg path? If it is how can I do that?
3- For example, I need double brackets (unicode: U+301A). How can I add it and how can I make it responsive to its content?
Thnx for your time.

Could you post a screenshot?

@ali-turan could you open separate issues for these?

I'm not sure we have an easy workflow for how to add symbols to fonts, though it's an area we've discussed.
Is there a LaTeX equivalent of double brackets that you're trying to achieve? If so, we'd probably consider it for the main KaTeX distribution.

@ali-turan
1 - You might be able to tweak the CSS manually, but there isn't a easy way right now.  Having a screenshot though will help us determine if the spacing you're seeing is a bug or not.
2 - It isn't possible at the moment.  We do you use SVG for stretchy accents, but regular symbols are all rendered from fonts and changing that system to also all SVG would big a significant amount of work.
3 - Adding specific glyphs from a package is definitely more doable.
@edemaine to answer your question there are two options:

\llbracket and \rrbracket from the stmaryrd package
\textlbrackdbl and \textrbrackdbl from the textcomp package

It might make sense to add these glyphs to our blackboard font file.
I found the answer in this awesome post https://tex.stackexchange.com/questions/81785/latex-code-for-white-square-brackets-and.  I love that it used dextexify to figure which packages the symbols were in.

I added the double-bracket feature request as a separate issue: https://github.com/KaTeX/katex-fonts/issues/46
I'm going to close this issue for now, but feel free to provide more information and we can try to help.


Piece detection not working

Hi, not sure if this is still a side project or not.  I'm trying to build a little bot that will let me play an over-the-board game with chess.com bots or my friends using raspberry pi parts.
I got your code working on my macbook but the detection doesn't seem to be working for the pieces.  It looks like it draws the board corners and box right, but the pieces don't get detected at all.  Anyone else run into that?  No errors, just a warning about the tensor model but doesn't seem like that would cause the problem: WARNING:tensorflow:No training configuration found in save file, so the model was not compiled. Compile it manually.

I've tested both with my own and with the included test images.
Lee
On Mon, Oct 30, 2023 at 2:22 PM aelmiger @.***> wrote:

What images are you testing with?
—
Reply to this email directly, view it on GitHub
https://github.com/aelmiger/chessboard2fen/issues/9#issuecomment-1785802909,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDDTZPU4KK3FVSHSUCK2TYB7V6HAVCNFSM6AAAAAA6VDQH56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVHAYDEOJQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Do you have any more information regarding what is not working exactly?

I'm not really sure how to troubleshoot this issue.  The bounding box
appears on the board outline from the corner detection step, so I'm
confident the packages are installed right.  But none of the test images
show bounding boxes on the pieces, and the generated chess board image is
always the starting position.  I don't know if it's a problem with the
chess piece model or something like that?  I confirmed the filename is
right and there's no error messages in the output terminal.
Any pointers on things to try?
On Mon, Oct 30, 2023 at 2:35 PM aelmiger @.***> wrote:

Do you have any more information regarding what is not working exactly?
—
Reply to this email directly, view it on GitHub
https://github.com/aelmiger/chessboard2fen/issues/9#issuecomment-1785822545,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDDT7EITY2CHG7I6J3LTTYB7XPJAVCNFSM6AAAAAA6VDQH56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVHAZDENJUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

After a lot of experimentation with this and several other projects what I'm learning is that these models will only work reliably with the exact chess set they were trained on.  I'd be willing to just buy the set you used for training if you can point me at the online store you got it from?  My focus is more on building a companion bot to play an over-the-board game against so I'm trying to avoid having to take the time right now to train my own model if I can avoid it.

Yes, that is correct. The CNN was only trained with one specific board and will most likely fail on other boards without additional training data.
Unfortunately I cannot find out which chessboard I used for training.

Ok, thank you for the time and responses in any case.  I don't know
anything about how to train these kinds of models so I'll go start doing
some research.  Cheers!
On Tue, Oct 31, 2023 at 10:58 AM aelmiger @.***> wrote:

Yes, that is correct. The CNN was only trained with one specific board and
will most likely fail on other boards without additional training data.
Unfortunately I cannot find out which chessboard I used for training.
—
Reply to this email directly, view it on GitHub
https://github.com/aelmiger/chessboard2fen/issues/9#issuecomment-1787390501,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDDT6KL2LNPOUJM5DNDNLYCEG2BAVCNFSM6AAAAAA6VDQH56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBXGM4TANJQGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


支持多条语句的执行

seata能否类似支持如下语句的执行：
<delete id="deleteById" parameterType="java.lang.String">       delete from a where id=#{id};       delete from b where id=#{id}; </delete>

not supported yet


about.md updated - Changed double byte numbers to Single.

thanks @Hideaki1. I made them consistently double byte because I thought it was more "Japanese" to do so. :-)


Combine Resource and Resources using class methods.

It's possible to combine the Resource and Resources classes into a single class by refactoring all existing Resources methods as @classmethod implementations. This change requires defining session and url as class variables, which impacts the architecture of the Client class. Currently, these are instance variables, which are set on object initialization. This ensures that multiple clients can be defined for different Connect servers. Class variables, on the other hand, are scoped to the class, not the instance. Therefore, each client must define separate classes at runtime to ensure that the resources used by the client are scoped to the proper Connect server. Here is a simple example showing how to define classes at runtime with different instance variables:  https://gist.github.com/tdstein/f8326a7f63562652a98ca7958c31f813

After further exploration, we discovered that this pattern introduces too many oddities. Foremost is handling multiple Client instances correctly. Using class variables to manage the url and session instances becomes overly complex.


Idea: Allow custom URL paths for Self Hosted

For users who have a self hosted version of Gitlab it would be great to be able to customise the URL endpoint so that we can access our repositories.

Hey @dwhoban thanks for raising this! We've heard this request from some other GitLab customers as well and it is on our backlog but candidly I do not have an ETA for delivering it.
We're accepting pull requests if you'd be interested in contributing! 😄 We could add an optional setting on the admin page to specify a self-hosted endpoint. The egress in the app's manifest would need to account for this URL as well to allow the outbound calls from the Atlassian Forge platform. If you'd be interested in taking this on I think we could work together and help you out! Another option would be to fork the repo and create your own private version of the app that talks to your instance.
Cheers!

@seajoshc just coming back round to this one - trying to get a local environment running with the instructions and getting a bunch of Forge and React errors

I'm curious if this is on any roadmap. Does anyone from atlassian or compass know if/when a self-managed GitLab integration will be available?


[WIP] Uses custom domain instead of ingress for grafana and prometheus

See #77

This should allow the domain to be set using the deployment variable DOMAIN_NAME
This also eliminates the use of INGRESS_IP in the manifest files and in prombench commands

I've tested this once on whether all the manifest files get applied accordingly. however, I didn't check if there is any change in grafanas behavior. I'll test it soon.

is this still WIP?

I tested this e2e, working as expected.

This also will need docker image update for prombench, I think we can push it to 2.0.1 itself(the last one)?

Thanks.
yes feel free to push to 2.0.1
at some point we need to push these new changes to the production Prow server.


Added a note in layer 2 mode concepts page on comparison with other a…

…pproaches like VRRP

What this PR does / why we need it:
This PR adds a note comparing metallb's layer2 mode with other similar solutions such as VRRP and KeepAliveD. This was the first open question I had when looking at metallb for the first time and I expect others will have this question as well so would be good to add this note to make this point clear and also shed a bit more insight on the architecture of metallb. In future, we can add further architecture documentation for clarity of both layer 2 and bgp modes as appropriate so that users can quickly compare with other similar solutions.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Special notes for your reviewer:
This was briefly discussed on the metallb slack channel with Dave Anderson

@danderson or others PTAL


Rd 11270 add prisma support

:tada: This PR is included in version 1.31.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Windows builds fail for PHP 8.1

Describe the bug
Windows builds are currently failing for the PHP 8.1 nightly builds at the setup PHP step with this library. The following error is thrown:

It seems a non existing PHP 8.1.0RC2-dev version is used even though that's not been released yet. We're at PHP 8.1.0 Beta3 right now: https://www.php.net/archive/2021.php#2021-08-19-1
You can see the builds fail here: https://github.com/laravel/framework/pull/38404/checks?check_run_id=3493365859
Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
Microsoft Windows Server 2019
PHP versions
PHP 8.1 nightly builds
To Reproduce
This part of our GitHub Actions setup: https://github.com/laravel/framework/blob/fa1c2f51a69cbe759ffcd16ad8250ef045a6537b/.github/workflows/tests.yml#L89-L118
Expected behavior
I expect PHP 8.1 Beta 3 to be loaded but it tries to setup a non existing PHP 8.1.0RC2-dev
Additional context
I discovered this while working on a PR to fix PHP 8.1 issues in the Laravel framework's 8.x branch: https://github.com/laravel/framework/pull/38404
Are you willing to submit a PR?
I'm afraid I don't know enough about this library to submit a PR.

@driesvints No worries, Till there is a fix in powershell-phpmanager, I have built PHP 8.1 with the PHP version patched to 8.1.0-dev, which is parsed correctly. So, it should work again.

Seems like @mlocati is already on it 🙂

I should have fixed the issue. Please keep me updated.

@mlocati Thanks, works now.


Panic goroutine stack exceeds<PHONE_NUMBER>-byte limit

Some of my TF plans fail with this stacktrace.

@Vad1mo thanks for opening up the issue! Would you be able to provide a redacted output of that aws_dynamodb_table.dynamodb-table block that is currently causing the error? I think I know what's causing it and I'll put a fix later today but I'd like to confirm I fixed the issue you're seeing.

@Vad1mo I believe this is fixed in master now (be3f5758831413a15bafa8902ff161d25af96d52). Can you confirm that is the case. If it's not, I'll reopen this issue.

thx, for the quick fix. it works now!


Fixes i.e., set calldata in correct representation

feat: export Validate()
fix: add safety check
fix: Save calldata in byte-level representation

Merging it to proceed with testing.


Enable runtime-tokio for quinn

Without enabling runtime-tokio the HTTP/3 backend will not work at all.
Considering reqwest only support tokio, this should be appropriate.
I'm surprised this was missed, or was this intentional?

It wasn't intentional. Does our http3 CI job not fail? How do we make it check this?

It's because we don't actually execute anything in CI, we just run compile checks:
https://github.com/daxpedda/reqwest/blob/abb729fc3d68ff1446b697e468ab1181c686489d/.github/workflows/ci.yml#L207-L208
To test this in CI we could potentially run the h3_simple example, which currently fails without this PR.

Yes, it guaranteed to fail at runtime on any connection attempt with HTTP/3.
Running the h3_simple will output the following:
Fetching "https://hyper.rs"...
thread 'main' panicked at 'unable to create QUIC endpoint: Custom { kind: Other, error: "no async runtime found" }', src/async_impl/h3_client/connect.rs:41:58

Ok, got it. Running something in CI could be a good idea, though I'd probably lean towards it should be connecting to a server in the same process, so that CI doesn't depend on network access and a remote server that we cannot control.
Either way, I can merge this without that.

Was just looking at the tests, it's using hyper to spawn servers to connect to. Unfortunately hyper doesn't support HTTP/3 yet: https://github.com/hyperium/hyper/issues/1818.
So I guess when it does, we can add feature = "http3" to the test suite.


[Feature-10273][Datasource] Add dataSource for AWS Athena

Purpose of the pull request
add dataSource for AWS Athena
close  #10273

Brief change log

Add the ATHENA backend
Add the ATHENA test case
Add the ATHENA frontend

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #11157 (ef44aa8) into dev (764a243) will decrease coverage by 0.10%.
The diff coverage is 60.00%.

@@             Coverage Diff              @@
##                dev   #11157      +/-   ##
============================================
- Coverage     40.39%   40.28%   -0.11%
- Complexity     4877     4940      +63
============================================
  Files           950      988      +38
  Lines         37186    37590     +404
  Branches       4079     4128      +49
============================================
+ Hits          15021    15145     +124
- Misses        20642    20909     +267
- Partials       1523     1536      +13

Impacted Files
Coverage Δ

...ce/api/datasource/AbstractDataSourceProcessor.java
17.39% <0.00%> (ø)

...gin/datasource/athena/AthenaDataSourceChannel.java
0.00% <0.00%> (ø)

...asource/athena/AthenaDataSourceChannelFactory.java
0.00% <0.00%> (ø)

...ugin/datasource/athena/AthenaDataSourceClient.java
0.00% <0.00%> (ø)

.../org/apache/dolphinscheduler/spi/enums/DbType.java
0.00% <0.00%> (ø)

...g/apache/dolphinscheduler/spi/utils/Constants.java
0.00% <ø> (ø)

...asource/athena/param/AthenaDataSourceParamDTO.java
66.66% <66.66%> (ø)

...source/athena/param/AthenaDataSourceProcessor.java
69.38% <69.38%> (ø)

...datasource/athena/param/AthenaConnectionParam.java
80.00% <80.00%> (ø)

...erver/master/processor/queue/TaskEventService.java
69.64% <0.00%> (-10.72%)
:arrow_down:

... and 51 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

Hi @guodongym , Thx for contributing this great feature! It seems there is no related docs for this PR, could you please add the docs? Thx again.

Yes I can

Hi @guodongym , Thx for contributing this great feature! It seems there is no related docs for this PR, could you please add the docs? Thx again.

The document has been added

SonarCloud Quality Gate failed.
  21 Bugs   9 Vulnerabilities   37 Security Hotspots   2101 Code Smells
 37.9% Coverage  2.4% Duplication

I found the sonar result is wrong, seems checked all the code, how can we retry this check again?

SonarCloud Quality Gate failed.
  21 Bugs   9 Vulnerabilities   37 Security Hotspots   2101 Code Smells
 37.9% Coverage  2.4% Duplication

I found the sonar result is wrong, seems checked all the code, how can we retry this check again?

Done

@songjianet @Amy0104 @devosend do you have time to take a look the FE part?

I add missing doc side bar in 8def3b2

And will merge PR when it pass


Fix Dapr annotations in Kubernetes deployment file

Fix Dapr annotations to test Dapr Workflow with Dapr V.1.0

cc: @yaron2

Thanks @jthin


Running Run-TestsInBcContainer with -testSuite defining a new test suite does not execute the tests

Issue
When running Run-TestsInBcContainer with -testSuite defining a new test suite does not execute the tests, but only creates the new test suite.
Scripts used to run the test in container container and cause the issue
# container name and version
$containerName = 'd365bc18WA'

# set credentials for using with Windows
$login = 'xxx'
$password = 'yyy'
$securepassword = (ConvertTo-SecureString -String $password -AsPlainText -Force)
$credential = New-Object System.Management.Automation.PSCredential($login, $securepassword)

Run-TestsInBcContainer -containerName $containerName `
                       -credential $credential `
                       -extensionId 'e26890f8-fafe-49c6-8951-2c1457921f9b' `
                       -detailed `
                       -testSuite $containerName `
                       -testPage 130455 `
                       -testRunnerCodeunitId 130451

Expected behavior
Tests are executed with the first call to Run-TestsInBcContainer that creates a new test suite.

testSuite and extensionId are mutually exclusive.
extensionId will run all tests in an extension
testSuite will run all tests in a pre-defined test suite.

OK, tested and works indeed when only using one of them. No way of having your script telling me that it's either one or the other?

My script should tell me what parameters you submit? :-)
Not sure I understand your suggestion?

Well, if these two parameters are mutually exclusive couldn't I be notified on that they exclude each other and that it doesn't makes sense to select both? This would have prevented this issue, right?

Got it:-)

Thanx

Shipped in 2.0.16


Surprise Trip Example: troubles with the installation

I'm trying to use CrewAI on a Ubuntu 22.04 virtual machine with python 3.10.12
The installation works fine with the default project using Open AI Key and Serper Key.
Now I'd like to try to use the Surprise Trip example: I've cloned the project

Trying the command poetry lock (on my virtual machine the poetry version is 1.112 ...), the error is

Any suggestions will be appreciated.
Thank you in advance!!

Can you try deleting the lock file and then retrying


add support for multipart/form-data POST,PUT,PATCH similar to httpbin's

i think the output matches httpbin's behavior pretty closely for multipart/form-data requests. supports both regular form-data and file uploads.

That's a great contribution! Thank you very much @jordan0day!
The build is failing for other reasons, I will just drop support to older Elixir versions.

Also thanks for the extensive unit tests! :+1:

Updated to use comprehensions in a couple of places -- both where you suggested in the parse_content_disposition_header fn and in the main post_multipart header, removing the filter and reducer anonymous fn's.

Thanks for the update. I just commented a small thing. Again, thank you for the contribution :+1:

Ah yes, of course, good catch. Fixed.


Fix npm installation

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
Fix npm installation for client docker image
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2806 (a093c9a) into main (8fe3c58) will decrease coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2806      +/-   ##
==========================================
- Coverage   94.15%   93.91%   -0.24%
==========================================
  Files         157      157
  Lines        4242     4242
==========================================
- Hits         3994     3984      -10
- Misses        248      258      +10

Impacted Files
Coverage Δ

src/system/temporary_directory.py
64.44% <0.00%> (-28.89%)
:arrow_down:

src/system/os.py
93.75% <0.00%> (-6.25%)
:arrow_down:

src/system/process.py
100.00% <0.00%> (+6.45%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fix issue 693 (end-to-end test cannot handle svrl:active-pattern/attribute(document) if it's zero-length)

Fixs #693
Commits

50fe689658683c65a4ad5a7baf282ce73f6b2c0c adds an end-to-end test which produces svrl:active-pattern/@document[. = '']. This test terminates the entire end-to-end test. (Travis)
db4aee76bd0c028d6555ab75fbb93d23d1c170fe fixes the report XML normalizer to ignore the zero-length attribute.

Merging this because

This is blocking #665
Both the failure and the fix are demonstrated on CI
No XSpec code is touched


How to set up Registry Workers behind an HTTP_PROXY

Hi there,
Our host changed to a server behind an HTTP_PROXY... The workers can no longer access the Internet. Here's an example of a build that got stuck:

What's the proper way to setup the Workers to properly download the images whose instructions include external access?
thanks!

Our host changed to a server behind an HTTP_PROXY... The workers can no longer access the Internet.

Can you further elaborate on this? Thanks

Hi @jzelinskie...
Summary

I can successfully build images on my local desktop with both intranet and Internet connection without HTTP_PROXY.

On hosts under restricted environments, we enforce the use of an HTTP_PROXY server and, therefore, we need a way to expose the HTTP_PROXY and NO_PROXY environment variables to specify our needs.

May be related to the https://github.com/docker/docker/issues/4962

The Dockerfile
It contains the instruction to update and add a new RPM repo.
FROM dockerhub.corp.intuit.net/intuit/rhel
MAINTAINER<EMAIL_ADDRESS>RUN yum update -y && yum clean all
RUN rpm -Uvh http://dl.fedoraproject.org/pub/epel/7/x86_64/e/epel-release-7-5.noarch.rpm

Local Builds without HTTP_PROXY
Running a build locally from my desktop, which has full connectivity to both our intranet and the external Internet, I can successfully build an image from this Dockerfile:
$ docker build --no-cache -t base .Sending build context to Docker daemon 2.186 MB
Sending build context to Docker daemon
Step 0 : FROM dockerhub.corp.intuit.net/intuit/rhel
 ---> 6d294a800cf7
Step 1 : MAINTAINER<EMAIL_ADDRESS> ---> Running in 255fcc231c0b
 ---> 87819d821dfc
Removing intermediate container 255fcc231c0b
Step 2 : ADD ./redhat-yum/pki /etc/pki
 ---> 5469583fcaf1
Removing intermediate container c021a4a208eb
Step 3 : ADD ./redhat-yum/redhat.repo /etc/yum.repos.d/redhat.repo
 ---> 11ae2f77d885
Removing intermediate container 36dd7ebac5ea
Step 4 : RUN yum update -y && yum clean all
 ---> Running in c775bb561774
Resolving Dependencies
--> Running transaction check
---> Package ca-certificates.noarch 0:2013.1.95-71.el7 will be updated
---> Package ca-certificates.noarch 0:2014.1.98-70.0.el7_0 will be an update
---> Package openssl-libs.x86_64 1:1.0.1e-34.el7_0.3 will be updated
...
...
---> Package openssl-libs.x86_64 1:1.0.1e-34.el7_0.7 will be an update
---> Package rpm-libs.x86_64 0:4.11.1-18.el7_0 will be an update
---> Package rpm-python.x86_64 0:4.11.1-16.el7 will be updated
---> Package rpm-python.x86_64 0:4.11.1-18.el7_0 will be an update
---> Package tzdata.noarch 0:2014d-1.el7_0 will be updated
---> Package tzdata.noarch 0:2015a-1.el7 will be an update
--> Finished Dependency Resolution

Dependencies Resolved

================================================================================
 Package             Arch    Version                  Repository           Size
================================================================================
Updating:
 ca-certificates     noarch  2014.1.98-70.0.el7_0     rhel-7-server-rpms  387 k
 glibc               x86_64  2.17-55.el7_0.5          rhel-7-server-rpms  3.6 M
 glibc-common        x86_64  2.17-55.el7_0.5          rhel-7-server-rpms   11 M
 libdb               x86_64  5.3.21-17.el7_0.1        rhel-7-server-rpms  718 k
 libdb-utils         x86_64  5.3.21-17.el7_0.1        rhel-7-server-rpms  101 k
 libgcc              x86_64  4.8.2-16.2.el7_0         rhel-7-server-rpms   86 k
 libstdc++           x86_64  4.8.2-16.2.el7_0         rhel-7-server-rpms  288 k
 libtasn1            x86_64  3.3-5.el7_0              rhel-7-server-rpms  316 k
 libxml2             x86_64  2.9.1-5.el7_0.1          rhel-7-server-rpms  664 k
 libxml2-python      x86_64  2.9.1-5.el7_0.1          rhel-7-server-rpms  244 k
 nss                 x86_64  <IP_ADDRESS>-2.el7_0         rhel-7-server-rpms  826 k
 nss-softokn         x86_64  <IP_ADDRESS>-1.el

Restricted servers running behind an HTTP_PROXY
Since our Enterprise Docker Registry workers run behind our firewall, they require an HTTP_PROXY. Our Docker setup includes the HTTP_PROXY and NO_PROXY environment variables to the Docker Daemon in order to be able to pull images from outside during "run" and "pull".
# /etc/sysconfig/docker
#
# Other arguments to pass to the docker daemon process
# These will be parsed by the sysv initscript and appended
# to the arguments list passed to docker -d
export HTTP_PROXY=http://proxy02.ie.intuit.net:80
export no_proxy=".intuit.net, .intuit.com, 10.*.*.*, localhost, <IP_ADDRESS>"
other_args="-b=bridge0"

However, the process during the build seems to not inherit the environment variables, which may be related to https://github.com/docker/docker/issues/4962. Which leads to the errors from the Workers not being able run the build process because of the lack of connectivity and the need of the HTTP_PROXY.
I thought about adding it to the Dockerfile, but it conflicts for other environments... So I found this project as a possible workaround, but I can't run privileged containers.
https://github.com/wtsi-hgi/docker-proxify
Questions

Is the Docker Workers running in Docker themselves. Should we inject the environment variables on them?

Do they run "Inception" mode? Docker inside Docker to build? If so, can we instruct the worker command with the variables?

Closing due to inactivity.


If condition inside a while loop - no while loop, use for loop and break

I could not find the problem in these conditions if inside an elementary while loop ...
I would appreciate your help
:(
https://www.google.com
cols = 3
n = 1
max = 10
while  (n < max)
{
validate = 'Row: [' + n + ']:' + cols + ' column. Not Equal'
if (n == cols)
{
validate = 'Row: [' + n + ']:' + cols + ' column. OK'
}
echo validate
if ( n == 5 )
{
  n = n + 3
}
else
{
  n = n + 1
}

}
_D:\tagui\src>tagui samples\easy chrome
START - automation started - Sun Nov 17 2019 17:30:44 GMT-0300 (Hora verano Suda
m├®rica Pac├¡fico)
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
Row: [1]:3 column. Not Equal
....
....

There is no while loop in TagUI - https://github.com/kelaberetiv/TagUI#conditions-examples
It is not possible to implement due to technical limitation in CasperJS component when I designed an auto-wait feature as part of all automation steps. Use a for loop with break instead.
for n from 1 to 9
{
if n equals to 4
continue
echo n

if n equals to 7
    break

}


Cannot use keybaord shortcut to copy/paste from extension page

Search for an extension (e.g., "Custom CSS and JS Loader")
Attempt to copy the config information to put in settings.json via Cmd+C
Notice that it doesn't work (Cmd+V does not yield those contents)
Select that text and use Edit>Copy (in menu bar) to copy
Notice that your clipboard now has those contents

/duplicate of #92545


Add obofoundry topic to repo metadata

The OBO Foundry has recently created a topics page at https://github.com/topics/obofoundry. It would be great if you could add the obofoundry topic to your repository, since it's currently listed as active in the OBO Foundry. If you're not sure how to do that, follow the instructions here.   The main issue for this task is at https://github.com/OBOFoundry/OBOFoundry.github.io/issues/1538 in case you want some more context or to join the larger discussion.

Hi @yongqunh, this is an automated message. On June 8th, 2021 (almost 2 years ago),        I posted this issue using another automated script in hopes that someone would        be responsive on this repository and then be able to add a tag to this repository's        metadata. This is regarding https://github.com/OBOFoundry/OBOFoundry.github.io/issues/1538.
Because it's been so long without addressing the issue, I am concerned        that this repository is no longer responsive/being maintained. Can you please        let me know if that's the case? In a few months,        if this follow-up message goes unanswered, I will recommend to        the OBO Foundry to change the status of this ontology to either "inactive"        or "orphaned", meaning that the nominal contact person is no longer responsive        and/or the repository is no longer maintained.
Given one of the OBO Foundry's principles is about responsiveness        (see https://obofoundry.org/principles/fp-020-responsiveness.html), it is        important that we are transparent to the community about the status of each        resource so potential users can make the best choice of which resources to        use.
Finally, please let me know if there are any questions about how to add the        "obofoundry" topic tag to this repository.

Hi @cthoyt The topic tag is now added. Sorry for the delay. Thanks!


Publish a new version of Microsoft.AspNetCore.ProtectedBrowserStorage

As noted in https://github.com/aspnet/AspNetCore/issues/13093, using this package with 3.0 RTM results in an error due to binary breaking changes in late 3.0 previews. We should publish a newer build of it compiled against 3.0 RTM.

Hi, when will this be resolved? It's just a simple build process

A new version that's compiled against 3.0.0 RTM is available on NuGet.org now: https://www.nuget.org/packages/Microsoft.AspNetCore.ProtectedBrowserStorage/0.1.0-alpha.19521.1


Bump pynacl version

Hi,
We are currently building multi-architecture images of our internal project and we are using this library which has a dependency on pynacl.
The current version of PyNaCl is set to 1.3.0. We were wondering if it was possible to bump this version to 1.5.0 which supports ARM64 (macOS).
Thanks!

Hi,
If you install this with Pip, it should use the setup.py and not the requirements.txt to resolve its dependencies, and your project should end up with PyNaCl==1.5.0 . Are you installing with pip install pusher ?
I've tried touching the requirements.txt in the past and its quite outdated. For now I'm using pip install . if I'm working on the library itself.

Please reopen if this is still an issue for you, I'm going to close this.


c8y devicemanagement certificates create: default to common name for the cert name

It should be possible to upload a trusted certificate without specifying the --name. The default --name should be the Common Name (CN) of the given certificate.
c8y devicemanagement certificates create --file ./tmp/tedge-certificate.pem

This is slightly more difficult to implement as first thought. It requires a new mechanism to allow setting two fields from a single input, whilst still allowing the user to override the name with an explicit value.


Add a DP Section with few basic beginner level problems to get started...

Hi @aaryahjolia, I'm well versed with Data Structures and Algorithms in C++, Python, Java and JavaScript and I'd like to work on this issue, can you kindly assign this to me?

Hi @aaryahjolia , I am a GSSoC'23 contributor and would love to work on this issue.I have solved around 900 questions on leetcode.


Disable System Accounts. Fix #6

This PR disables sets the login-shell of system-accounts to nologin aswell as it removes the password.

removes does not do what you think it does. See here:
a filename, when it does not exist, this step will not be run.
This means, if the file is not present, the task will not be run. Similary the creates-option. If the file exists, this step will not be run (the opposite).

@rndmh3ro as always great work


Spams the app open endlessly

running the exe will spam the app open endlessly

TaskbarGroups.zip
Try this release with the .exe alone
I may have identified a potential cause for why it would be looping like that.
Place this into its own folder as it would generate the normal folders as usual.
Sorry for taking so long. Had to take a step back due to personal work and school.

I have the same problem

@aemdca4 Did you try the build above?


Limited to top 10 Instances for a given metric

If a given metric has more than 10 unique instances, only 10 are returned, since the &$top={$top} parameter is not configurable.

Azure Monitor or Application Insights?

Azure Monitor or Application Insights?

Azure Monitor


Hi,
thanks a lot for just the greatest HN TUI! fantastic work! The only thing is lacking and, IMHO, it's a major issue (but very easy to add): the loading progress visualisation.

Or if by some reason there is one and I'm missing it... Just let me know :)
Thanks!

Hi @ink-splatters, thank you for your kind words.
Regarding the loading progress visualisation, are you referring the time when loading stories, comments, etc?
Currently there is a simple loading screen (not a progress bar) for those:
https://user-images.githubusercontent.com/40011582/141149270-00ec8d8a-83d9-4744-be5b-22eed95dce16.mov
One reason that I don't use progress bar is because it's hard to estimate the time needed to finish an API call to the Hacker News server.

@aome510 thank you very much for getting back to me. It's sort of a shame I didn't notice it, maybe I need to check if I'm blue color - blind:)
but just in case if this feedback helps, you may want to make the progress bar to stand out more, e.g. by making it thicker :) just saying :)
Regarding the API call time: can't you rely on Content-Length? or it's not guaranteed to present? (you could then update the status every time you receive a new buffer).

but just in case if this feedback helps, you may want to make the progress bar to stand out more, e.g. by making it thicker :) just saying :)

Regarding the API call time: can't you rely on Content-Length? or it's not guaranteed to present? (you could then update the status every time you receive a new buffer).

Great! Thanks for the feedback. I'll look into those and see what I could do.


Add LINE messenger in-app ad servers

a.line.me
ad-center.line.me
lap-click.tr.line.me
lt-click.tr.line.me
obs.line-scdn.net
optout-api.tr.line.me
sch.line.me
tr.line.me

Hi Nicolas @nicolasmaia.
And we would block these domains because...?
I need to see a clear case for this.

They serve in-app ads.

I'm sorry Nicolas @nicolasmaia that's not enough justification to move me to deplatform a bunch of domains.
Closing.

Wait, do I need to prove it? How would one do that?


snap: Builds started to fail

https://github.com/parca-dev/parca-agent/actions/runs/3497876186/jobs/5857631546
The 'snap' directory is meant specifically for snapcraft, but it contains
the following non-snapcraft-related paths:
- README.md

This is unsupported and may cause unexpected behavior. If you must store
these files within the 'snap' directory, move them to 'snap/local'
which is ignored by snapcraft.
Launching instance...
Traceback (most recent call last):
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/bases/buildd.py", line 515, in _setup_apt
    executor.execute_run(
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/lxd/lxd_instance.py", line 289, in execute_run
    return self.lxc.exec(
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/lxd/lxc.py", line 329, in exec
    return runner(final_cmd, **kwargs)  # pylint: disable=subprocess-run-check
  File "/snap/snapcraft/8567/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['lxc', '--project', 'snapcraft', 'exec', 'local:snapcraft-parca-agent-on-amd64-for-amd64-10422[21](https://github.com/parca-dev/parca-agent/actions/runs/3497876186/jobs/5857631546#step:6:22)', '--', 'env', 'PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin', 'SNAPCRAFT_MANAGED_MODE=1', 'apt-get', 'update']' returned non-zero exit status 100.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/lxd/lxd_provider.py", line 126, in launched_environment
    instance = launch(
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/lxd/launcher.py", line 212, in launch
    base_configuration.setup(executor=instance)
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/bases/buildd.py", line 351, in setup
    self._setup_apt(executor=executor, deadline=deadline)
  File "/snap/snapcraft/8567/lib/python3.8/site-packages/craft_providers/bases/buildd.py", line 521, in _setup_apt
    raise BaseConfigurationError(
craft_providers.bases.errors.BaseConfigurationError: Failed to update apt cache.
* Command that failed: 'lxc --project snapcraft exec local:snapcraft-parca-agent-on-amd64-for-amd64-104[22](https://github.com/parca-dev/parca-agent/actions/runs/3497876186/jobs/5857631546#step:6:23)21 -- env PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin SNAPCRAFT_MANAGED_MODE=1 apt-get update'
* Command exit code: 100

Any ideas? Anyone?
cc @jnsgruk (Let me know if you don't want to get pinged ☺ )

Hmm. That's odd. I've had issues with 22.04 on runners where docker is also installed - LXD and Docker sometimes fight over iptables and LXD containers end up with no outbound internet access.
AFK today but happy to take a look on Monday! It might work better if the runner is on Ubuntu Focal as a starter, and I could follow up with a change to make out functional on Jammy

Yeh reading some GitHub issues, it looks like ubuntu-latest only started meaning 22.04 quite recently so we might have been bitten by the shift in underlying version


How can I add MathJax support to my site using the Ananke theme?

The Hugo documentation says to put
<script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-AMS-MML_HTMLorMML">
</script>

in a partial layout that will appear on every page that you need the mathjax, but I can't figure out how to make this repository a submodule and at the same time add mathjax support.

Hey, @jasikpark You can duplicate this file into the root layouts folder of your project and add the script there. https://github.com/budparr/gohugo-theme-ananke/blob/master/layouts/partials/site-scripts.html

You mean put it in /layouts? Thanks!
--
Caleb Jasik
<EMAIL_ADDRESS>https://jasik.xyz
281.614.9841
On September 3, 2018 at 10:43:44 AM, Bud Parr<EMAIL_ADDRESS>wrote:
Closed #129.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

yep

But it needs to be along the same path, so
current:
/themes/gohugo-theme-ananke/layouts/partials/site-scripts.html
new:
/layouts/partials/site-scripts.html

ah, that's what I was wondering.

{{ $script := .Site.Data.webpack_assets.app }}
{{ with $script.js }}
  <script src="{{ relURL (printf "%s%s" "dist/" .) }}"></script>
  <script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-AMS-MML_HTMLorMML">
</script>
{{ end }}

Hey, @jasikpark You can duplicate this file into the root layouts folder of your project and add the script there. https://github.com/budparr/gohugo-theme-ananke/blob/master/layouts/partials/site-scripts.html

This link seems to be broken, can you please resolve the problem in accordance with newer version

I'd like to echo the support for updated MathJax documentation with this theme.

@benslack19  adding the link to mathjax in footer.html worked for me. What is the issue you are facing?

I appreciate your response @KunikaValecha. The version of the ananke theme I just downloaded didn't have a footer.html file but it had asite-footer.html file in the partials folder. These are the different things I've tried but each didn't render the MathJax text correctly.

Put the MathJax script snippet at the bottom of the site-footer.html file in the partials folder of the theme (themes/ananke/layouts/partials)
Copying the site-footer.html file with the MathJax script snippet and placing a copy in my site's /layouts/partials (I'm just playing around with the quick start version.)
Renaming site-footer.html to footer.html.

Do you have a repo with the footer.html file that I can look at?

@benslack19 you are in right direction. I also added the link in site-footer.html. You won't require any further step that you have done. Just add the link to site-footer.html and you are good to go.

Thank you so much @KunikaValecha. I'm getting closer after realizing my initial error. I was trying different things, but I was using an old markdown file that only contained inline MathJax examples. I see that it works when it's not inline.
On that note, did you try and have success with the solution for handling inline code and mathjax underscores (subscripts)?

For subscript I did it, \(word_{subscript}\). What do you mean by inline? Inserting mathematical sign in the line of text?

@benslack19 writing any code in \( math \) will keep math in text. PS, that's a double backslash.

Ah okay thank you @KunikaValecha and thank you for clarifying the double backslash is used there. I was using a single dollar sign for my in-line math (which was working on my Jekyll-based site) but I see here that that's now discouraged practice. Thanks for your help!

This doesn't seem to work anymore:
https://github.com/acgetchell/blog/blob/master/layouts/partials/site-scripts.html


Can't create new user cert

Hello!

root@gritsenko ~ # docker run -v $OVPN_DATA:/etc/openvpn --rm -it kylemanna/openvpn easyrsa build-client-full kutakov nopass
Easy-RSA error:
EASYRSA_PKI does not exist (perhaps you need to run init-pki)?
Expected to find the EASYRSA_PKI at: /etc/openvpn/pki
Run easyrsa without commands for usage and command help.

my bad

@degritsenko
How did you solve that?

I got this same error, It was because I was following
https://hub.docker.com/r/kylemanna/openvpn/
Guide, and did not run
docker run -v $OVPN_DATA:/etc/openvpn --log-driver=none --rm -it kylemanna/openvpn ovpn_initpki
The "My Bad" from degritsenko was probably because of a missed command.
I am posting this because - It may help anyone else who falls into this trap !


Repo prefix is not being respected in union queries

Environment

Elixir version: 1.7.3 (compiled with Erlang/OTP 21)
Database and version: (PostgreSQL) 10.6
Ecto version: 3.1.5
Database adapter and version: postgrex 0.14.3
Operating system: MacOS 10.14.4

Current behavior
Repo prefix is not being respected in union queries. e.g. given a query like:
from c in query,
    union: ^union_query

the prefix will be applied to the query portion but not to the union_query portion.
It works if I pass in a prefix to each join in union_query but this is only appropriate for compile time strings and my prefix is dynamic.
Expected behavior
I expect it to prefix all of the tables in union subqueries with the prefix I specify in my Repo.all().

Closing in favor of #3039.


storage error halts VM creation

WARNING: Since the VM is created using premium storage or managed disk, existing standard storage account, tqcrg1diag, is used for boot diagnostics.
New-AzureRmVM : Changing property 'osDisk.osType' is not allowed.
ErrorCode: PropertyChangeNotAllowed
ErrorMessage: Changing property 'osDisk.osType' is not allowed.
ErrorTarget: osDisk.osType
StatusCode: 409
ReasonPhrase: Conflict
OperationID : a9cfa972-d083-4f4b-8373-b073cee5f14e
At line:61 char:5

New-AzureRmVM `

~~~~~~~~~~~~~~~

CategoryInfo          : CloseError: (:) [New-AzureRmVM], ComputeCloudException
FullyQualifiedErrorId : Microsoft.Azure.Commands.Compute.NewAzureVMCommand

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

ID: 7eed234f-9a8d-4c1b-2368-fe4c8fbe8299
Version Independent ID: 876980d5-e42c-b070-c0a6-73ed3f2c91d1
Content: Change a VMs availability set
Content Source: articles/virtual-machines/windows/change-availability-set.md
Service: virtual-machines-windows
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

Thanks for the feedback! We are currently investigating and will update you shortly.

@zr-msft can you take a look at this one? I ran through the doc and confirm I get the same error. Using managed premium disks.
Using something like https://blogs.technet.microsoft.com/mckittrick/how-to-rebuild-from-an-existing-managed-disk/ works but it is not as fluent as following this doc.

adding @cynthn

@MicahMcKittrick-MSFT where does it fail? On the final VM creation?
@harrisryan93 thanks for the feedback! I'll try to get to this soon, but it may take a couple of days.

@cynthn yes on the VM creation
Recreate the VM
New-AzureRmVM `
   -ResourceGroupName $resourceGroup `
   -Location $originalVM.Location `
   -VM $newVM `
   -DisableBginfoExtension

if you check the variables after each step there are no errors. Just seems to throw it when actually trying to create the VM and use that existing VHD.

@MicahMcKittrick-MSFT Okay, I might quickly try just creating using "-attach" with another VHD to see if it is a wider issue with creating from an existing VHD. Thanks!

@MicahMcKittrick-MSFT can you reassign this to me? Thanks!

This script is for window machine only. Dont use it with linux machine

This issue still exists.  Is there any fix in sight?  Soon?

@ctimon Do you know how the original VM was created? Was it created by attaching an existing VHD to a VM?
@MicahMcKittrick-MSFT This sounds suspiciously like the other issues we have seen where the initial VM was created in a way where it is missing the pieces of information that are in the VM configuration. Do you happen to remember those issues so we can trace who responded to those?

@cynthn I do remember those but have no idea where the issues are at this point. I will check my emails and see if I can track it down.

Inline….
From: Cynthia Nottingham<EMAIL_ADDRESS>Sent: Thursday, August 22, 2019 1:34 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Chuck Timon<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] storage error halts VM creation (#20344)
@ctimonhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fctimon&data=02|01|ctimon%40microsoft.com|e17e9bb3390644d2169d08d72726e85a|72f988bf86f141af91ab2d7cd011db47|1|0|637020920364376381&sdata=aISMRj8DKJaCKZSUjsfc5UGyuftvtULNF%2FbsXo6oU%2Bs%3D&reserved=0 Do you know how the original VM was created? Was it created by attaching an existing VHD to a VM?
I created it from scratch with no pre-existing resources in the RG.  I am testing in my own MAG account.    I went ahead and converted the script back to AzureRM module and that worked for me.
@MicahMcKittrick-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicahMcKittrick-MSFT&data=02|01|ctimon%40microsoft.com|e17e9bb3390644d2169d08d72726e85a|72f988bf86f141af91ab2d7cd011db47|1|0|637020920364386368&sdata=MnvXFFRfXEoLxUB8ZE5xXXW7hndEwhaN8WGTDJPTng0%3D&reserved=0 This sounds suspiciously like the other issues we have seen where the initial VM was created in a way where it is missing the pieces of information that are in the VM configuration. Do you happen to remember those issues so we can trace who responded to those?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F20344%3Femail_source%3Dnotifications%26email_token%3DAFKXKE4SAXBQCFZ3GQ35SBTQF3EYHA5CNFSM4GJPF3I2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD452K6Y%23issuecomment-524002683&data=02|01|ctimon%40microsoft.com|e17e9bb3390644d2169d08d72726e85a|72f988bf86f141af91ab2d7cd011db47|1|0|637020920364386368&sdata=5j3xTIavtfXM%2BAsU4W5FQ%2FHOsTyWi%2B43%2F80x8KhMhZI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFKXKE4KSQG4FGOZY443XIDQF3EYHANCNFSM4GJPF3IQ&data=02|01|ctimon%40microsoft.com|e17e9bb3390644d2169d08d72726e85a|72f988bf86f141af91ab2d7cd011db47|1|0|637020920364396368&sdata=M%2BU7bVrPMqfXPTQULqqQl%2Frgxyo6%2FpQ822389UH%2Bo9I%3D&reserved=0.

We are currently closing out long standing backlog in order to better determine which issues are still valid and which have been addressed. If you are still seeing a specific problem with this document please open a new issue so we can properly assign the feedback to the correct author and get it addressed ASAP.
#please-close


Fiber local / Thread local ?

Ruby have thread_local, as well as C, Java, Rust, Python and many others but Go and maybe some others.
Here is a situation require fiber_local:
class MyContext
#...
end
fiber_local current_context
def do_something_with_context(&block)
    current_context = # ...
    yield
    current_context = nil
end

Of course with ... yield is a way to do this, but for more complicated case:
do_something_with_context do
    a = MyClass.new
    a.do_some_require_contenxt
end

Here, do_some_require_contenxt can not get the context instance but send as an argument. It makes hundreds of methods must have a verbose argument context, at least in my code.
Just like global variable, I think fiber local have the same significance.
And just a hash in the Fiber and related instance methods could implenment this. For the consideration of the performance, compilation-time defined NamedTuple may be better. But thread local is always slow in most of lanugage, no mater how it's implenmented -- even in C it's a performance problem -- so if it cannot be fast but have a necessary reason to exist, we could never solve the performance. In fact, I think it should only be used in performance-non-sensitive case, where a Hash is enough, and even a SortedArray for less memoy usage.
I would be willing to contribute code if the feature were accepted.

@xqyww123 fork unshares all the memory so everything will be independent after a fork.

@RX14 but fiber. I mean create a new thread or fiber but process.

@xqyww123 You can reopen Fiber and add instance variables to it, to get fiber-local variables.

I'm interested in fiber locals sometimes, but either a Hash(Fiber, T) (only some fibers need a value) or reopening Fiber (if most fibers will need it) has been enough.

Maybe, it's another def_delegators.
Yes, Hash(Fiber, T) in module could also prevent name confliction.


Support for Python with Durable Functions?

Hi,
Is there an ETA on support for Python with Azure Durable Functions?

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

ID: b934ad1d-aabb-707d-d296-02cfbb22ec5c
Version Independent ID: 5d8f8cfb-be79-fac8-62ef-3a396a3be3e8
Content: Durable Functions Overview - Azure
Content Source: articles/azure-functions/durable/durable-functions-overview.md
Service: azure-functions
GitHub Login: @cgillum
Microsoft Alias: cgillum

@markl11 Thank you for your query. We will engage the author to find more information on this .

@cgillum Could you please review this request and help on the same ?
Thank you.

You can use this GitHub thread to track our progress (and provide feedback!) on Durable Functions for Python: https://github.com/Azure/azure-functions-python-worker/issues/227#issuecomment-566654804.
As stated in the comment by @anthonychu, we're targeting early 2020 for the initial preview.

It seems that it is available now: https://github.com/Azure/azure-functions-durable-python Please update this document accordingly.

The docs are being updated. While the Durable Functions Python repo is public and we're accepting contributions, we're not quite ready for a public preview. Look for an announcement in the next month or so.

Durable Functions for Python is now GA. https://azure.microsoft.com/updates/python-durable-functions-support-in-azure-functions-is-now-generally-available-2/
#please-close


Small fix in example

Must be:
return File(bytes, "application/vnd.ms-excel", "MySpreadsheet.xlsx");

Either I'm blind or that's exactly what is already in the readme. What is the issue?

Oh wait, I see it. The "s" and "l" are transposed in "xlsx". Good catch!


Cole/m1 support

LGTM. If we move forward with the Dockerfile changes then maybe you should just add the Docker Desktop settings to the Testing with Docker section and get rid of the rest.