requirejs confusion

Am I correct in saying that graphlib 1.0.1 does not support loading via requirejs, but dagre does?

@cpettitt first of all i want to thank you for such a great library!
I think it's important to add info about how to made this library umd compatible to a wiki.
And also it would be great to add umd build to a dist folder, for example as dist/graphlib.umd.js (ideally with update of bower.json)
Not all people know how to or want run browserify manually to get self contained umd/amd compatible build. Even people who know how to do this would prefer use your umd build rather than make fork just to push their own umd build.
Again - thanks a lot for the library!

(as a complete aside - google is using this library + dagre with TensorFlow:  https://www.tensorflow.org/)

@sprilukin IIRC, the -s graphlib approach would work for all use cases, so could just be the default - I think that's what I ended up doing for dagre. This is just waiting from a PR from an interested enough party who can verify with a UMD loader.
@GordonSmith yeah, I saw that. Pretty cool! Thanks for mentioning it.

Well i've already made my build with -s graphlib and can confirm that it works perfectly in browser with amd.  So we just need to test this in node and in browser without amd. I'll try to do that.
Or how you expect to do such verification?

That part is easy: just run the tests (make test should do it). If you have a patch, I can run the tests and publish a new version.

Well the problem is that i'm not familiar with make,
i just build umd version for myself using direct command:
browserify index.js -o dist/graphlib.umd.js -s graphlib

i just tried to modify browserify targets in makefile so they'll produce same result,
but result is different and tests are failed.
here are my changes in makefile:
diff --git a/Makefile b/Makefile
index f05dd9f..759525b 100644
--- a/Makefile
+++ b/Makefile
@@ -52,14 +52,14 @@ browser-test: $(BUILD_DIR)/$(MOD).js $(BUILD_DIR)/$(MOD).core.js
 bower.json: package.json src/release/make-bower.json.js
 	@src/release/make-bower.json.js > $@

-$(BUILD_DIR)/$(MOD).js: browser.js $(SRC_FILES) | unit-test
-	@$(BROWSERIFY) $< > $@
+$(BUILD_DIR)/$(MOD).js: index.js $(SRC_FILES) | unit-test
+	@$(BROWSERIFY) $< > $@ -s graphlib

 $(BUILD_DIR)/$(MOD).min.js: $(BUILD_DIR)/$(MOD).js
 	@$(UGLIFY) $< --comments '@license' > $@

 $(BUILD_DIR)/$(MOD).core.js: browser.js $(SRC_FILES) | unit-test
-	@$(BROWSERIFY) $< > $@ --no-bundle-external
+	@$(BROWSERIFY) $< > $@ --no-bundle-external -s graphlib

 $(BUILD_DIR)/$(MOD).core.min.js: $(BUILD_DIR)/$(MOD).core.js
 	@$(UGLIFY) $< --comments '@license' > $@

Ok, PR #57 should do the trick, both graphlib and graphlib.core are now amd compatible with this PR,
see this comment for details about using graphlib.core.

Fixed in v2.1.0. Thanks @sprilukin!

Glad i can help!


Roll Engine from b5f5e6332cb4 to 6cb6af354e66 (4 revisions)

https://github.com/flutter/engine/compare/b5f5e6332cb4...6cb6af354e66
2020-06-18<EMAIL_ADDRESS>Roll Skia from c950e05bafcd to 92887b547497 (1 revision) (flutter/engine#19133)
2020-06-18<EMAIL_ADDRESS>Roll Skia from b54946b86d33 to c950e05bafcd (5 revisions) (flutter/engine#19131)
2020-06-18<EMAIL_ADDRESS>Roll Dart SDK from 54481776c96d to a26ad7fb7c9f (6 revisions) (flutter/engine#19130)
2020-06-18<EMAIL_ADDRESS>Word substitutions (flutter/engine#19111)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


feat(tutorial): complete step 5 -hursley

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

@sarahmariemaddox can you put the URL to your deploy app in the description?

updated the package.json

App at http://carbon-tutorial-sarahmariemaddox.eu-gb.mybluemix.net
Looks good! I've recorded your completion. Please make sure you complete the 'wrapping up' stage to get your badge and t-shirt!


added features.xml file for easier karaf deployment

https://jira.duraspace.org/browse/FCREPO-1303
This makes deployment in Karaf significantly easier through the use of a features.xml file

N.B. this is also a necessary first step toward getting a full OSGi-based integration test with pax.exam


createBunWsHandler handleRequest race problem

websocket
-----> open
https://github.com/cah4a/trpc-bun-adapter/blob/4de96b72e099c4d19561651bc43b74c8fc3240ec/src/createBunWSHandler.ts#L79
await ctxPromise; // if this takes some time.
----> message
https://github.com/cah4a/trpc-bun-adapter/blob/4de96b72e099c4d19561651bc43b74c8fc3240ec/src/createBunWSHandler.ts#L290
    const promises = msgs
        .map((raw) =>
            parseTRPCMessage(raw, router._def._config.transformer),
        )
        .map(client.data.handleRequest); // client.data.handleRequest is no ready yet, so there is an error TypeError: Array.prototype.map callback must be a function.

https://github.com/cah4a/trpc-bun-adapter/blob/4de96b72e099c4d19561651bc43b74c8fc3240ec/src/createBunWSHandler.ts#L125
client.data.handleRequest = ... // set  handleRequest

Make a PR https://github.com/cah4a/trpc-bun-adapter/pull/12

Thanks for your contributing!
Packaged version 1.2.0 is available now.
There more fixes, please double check on your case and re-open issue if reproduces.


Create cinder_volume_hds_spec.rb

Support HNAS nfs driver

Supprt HNAS nfs driver

Thank you for contributing to stackforge/puppet-cinder!
stackforge/puppet-cinder uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Add support for dind in multitenant plugin

This change updates the multitenant setup script to support
docker-in-docker deployment, which uses supervisord instead of systemd.

cc: @rajatchopra

Once this merges, can the dind-supporting changes be pulled into origin so that dind deployment can work by default without an openshift-sdn repo?


User is able to delete all agencies (including default agency)-- causing a crash on subsequent login

The user is able to use the Manage Agencies list to delete all agencies, including the default agency.  This causes an ArrayIndexOutOfBoundsException the next time the user logs out and logs back in.

It just happened to me today. Any temporary solution, other than reinstalling (git clone + play dependencies)?


PHPCI Github status checks on PR

I am wondering if it is possible get status checks for a pull request similar to what travis-ci and other services are doing:

This would help a lot since we don't constantly want to monitor the PHPCI server but rather just have our checks show up on pull requests.
Edit
I have noticed in the GithubBuild.php that there is a sendStatusPostback but I don't think anything happens with it curently. Does anybody know how I can enable this?
Seems like this also needs a token to work because it sets an Authorization token header. How can I check if a token has been set or not? Or how can I set up a token for my PHPCI app?

Stephan,
If you look at older pull requests you will notice there used to be a check
provided by the phpcibot user. I think the integration either broke or the
server died.
On Fri, Dec 23, 2016, 7:12 AM Stephan<EMAIL_ADDRESS>wrote:

I am wondering if it is possible get status checks for a pull request
similar to what travis-ci and other services are doing:
[image: status]
https://camo.githubusercontent.com/ec7636809f79c4849297fd869f18312fbe56d7f3/68747470733a2f2f66696c65732e726561646d652e696f2f4a52447543446752675747484447365a714d59775f676974687562312e706e67
This would help a lot since we don't constantly want to monitor the PHPCI
server but rather just have our checks show up on pull requests.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Block8/PHPCI/issues/1288, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPd3QkibkJ74mZqaHWWPPmjPcRBuh3cks5rK7q8gaJpZM4LUxzG
.

--
Sincerely,
Pablo Tejada
From Mobile

You are not making a whole lot of sense. You mean to say that there is all this Github specific logic that needs a token but no documentation on how to add a token or some config option to add your own token?
Things like:
./console phpci:poll-github
Need an access token to work, yet there is no option to add a token. That is besides editing the config.yml manually, which you have to know just by digging through the code.
Even if you set the access token commands like poll-github still fail.

Sorry Stephan i was not clear on my comment. I did not tried to give you
the solution to what you are asking. All am saying is that it has to be
possible some how because i saw the green checks in older pull request in
this repo.
Here as an example PR with a failing status
https://github.com/Block8/PHPCI/pull/1200
On Fri, Dec 23, 2016, 8:05 AM Stephan<EMAIL_ADDRESS>wrote:

You are not making a whole lot of sense. You mean to say that there is all
this Github specific logic that needs a token but no documentation on how
to add a token or some config option to add your own token?
Things like:
./console phpci:poll-github
Need an access token to work, yet there is no option to add a token. That
is besides editing the config.yml manually, which you have to know just by
digging through the code.
Even if you set the access token commands like poll-github still fail.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Block8/PHPCI/issues/1288#issuecomment-268987147, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPd3dTmDnqANWF4XFi0D4cr_4O1Zwgcks5rK8cRgaJpZM4LUxzG
.

--
Sincerely,
Pablo Tejada
From Mobile

I know it is probably possible at the moment since I have found it in the codebase, but it seems utterly broken at the moment. Is that correct?

Yes, at least the integration in this repo is broken. I think is because
the live server that is suppose to run the tests is not running.
In your original post it sounds like you were asking if it was possible. I
am just confirming that have seen the feature been used but i dont know how
it works 😁
To be honest, if your repo is public you better of using travis-ci
On Fri, Dec 23, 2016, 8:27 AM Stephan<EMAIL_ADDRESS>wrote:

I know it is probably possible at the moment since I have found it in the
codebase, but it seems utterly broken at the moment. Is that correct?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Block8/PHPCI/issues/1288#issuecomment-268989561, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPd3ZaNFLqb1-s1u5hcF187lbMsAqBdks5rK8wwgaJpZM4LUxzG
.

--
Sincerely,
Pablo Tejada
From Mobile

Travis-CI is too expensive for private repo's but thanks anyways.

If you figure this out please consider adding a how to guide to the wiki.
On Fri, Dec 23, 2016, 9:09 AM Stephan<EMAIL_ADDRESS>wrote:

Travis-CI is too expensive for private repo's but thanks anyways.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Block8/PHPCI/issues/1288#issuecomment-268995112, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPd3RENRtqsiCHWsakVrl9aDWOPbLUdks5rK9X6gaJpZM4LUxzG
.

--
Sincerely,
Pablo Tejada
From Mobile

It does work, but it is a slight pain in the rear to get working it seems. On a lower level:

Create a OAuth Integration
Update config.yml to have a phpci:github section with the Integration OAuth token and id
Setup a webhook that includes the permissions for pull requests as well as read permissions, etc.

Disclaimer Still working on this, but I do have it at least communicating to a certain degree.


Error: The operation couldn’t be completed. (com.facebook.sdk.core error 3.)

Hi! Great Nativescript plugin! I've managed to follow all guide. But I'm getting this error. Did I miss something? Thanks!

Hi, @bonlemuel. I am sorry for the problem you experienced with nativescript-facebook plugin. Could you share for which platform (Android/iOS) you faced this issue? Also, whether your NativeScript app is Angular or not? Did you manage to build and run any of the demo apps?

It runs on Angular. I'm currently testing it on iOS. But I've managed to solved the issue by adding the code below.
<key>CFBundleURLTypes</key>
	<array>
		<dict>
			<key>CFBundleURLSchemes</key>
			<array>
				<string>fbxxxxxxxxxxxxxxxxx</string>
			</array>
		</dict>
	</array>
	<key>FacebookAppID</key>
		<string>{AppID}</string>
	<key>FacebookDisplayName</key>
		<string>{AppName}</string>
<key>LSApplicationQueriesSchemes</key>
<array>
	<string>fbauth2</string>
	<string>fbapi</string>
	<string>fb-messenger-api</string>
	<string>fbshareextension</string>
</array>

Thank you.

Awesome!


Add basic support for configuring server password

This diff adds basic support for configuring password, in case you need to auth to the server or a bouncer.
It works correctly for authorizing to my bouncer, and doesn't throw any more warnings than current master after make build and make run. One problem however, is that I haven't been able to figure out how to do password masking. I can add that in another request or this one if you feel that it's blocking. (This is the first React app I've looked at, so I'd be grateful for some pointers about how to do that.)

Looks awesome. I'll happily give the pointers.

I think it should work It shows the password field masked now.
Let me know if there is anything else I should do.

Used the interface solution you suggested instead.
Let me know if you would prefer me to squash the commits and push before you merge the request.

No problem. Thanks for the great work! :+1:


ETL::Control::FileDestination does not always generate valid CSV file

The class ETL::Control::FileDestination does not generate a valid CSV file in some cases.  At least not one that could be read by a file source with a :parser[:name] of :delimited.  It appears that FileDestination was originally designed to use FasterCSV to create the output file (based on the method "options"), but it apparently wasn't implemented.
Here is a case which will break under the current implementation:

In one .ctl, read from a database source where the value of a column contains a double quote
Output the results to a file using a file destination, make to set :enclose to """
In another .ctl, read from the file

In this case, a MalformedCSVError exception is raised from FasterCSV when attempting to read from the file in the second .ctl.  Here is the sequence of events that lead to this exception.

When data is read from the database it may also be written to the "source_data" directory, if you examine that file you'll see that embedded double-quotes are escaped by an additional double-quote ("").  This is the correct way to escape embedded double-quotes in a CSV file.
Compare to the file output from step 2 above.  Note that each field is enclosed by double-quotes and the embedded double-quote is escaped with a backslash (rather than another double-quote).
When reading this file with a delimited file source, FasterCSV raises an error because it doesn't understand escaping embedded double-quotes with a backslash.

Because of the above issues, it appears there is no way to create a file in one .ctl and read it in another when embedded quotes are present.
The solution could involve using FasterCSV instead of the manual escaping and enclosing implemented in FileDestination.  It may be wise to use another name, such as CsvDestination to avoid a breaking change.

Closing this - the destination has been merged back into master.


The more and less utilities (pagers) break command-line editing on macOS and Linux

Note:

On macOS 10.14, both more and less produce the symptom, whereas it is only less on Ubuntu 18.04.
Whether or not the PSReadLine module is loaded makes no difference.

It took considerable discipline to refrain from more or less jokes.
Steps to reproduce
On macOS or Ubuntu, run the following and then press q to exit the pager:
# On macOS, `more` is affected too.
1..200 | less

Expected behavior
less should exit, the command prompt should return, and command recall with arrow up should continue to function.
Actual behavior
less exits and the command prompt does return, but command recall is now broken: an A is printed instead.
Environment data
PowerShell Core v7.0.0-preview.3 on macOS 10.14.6
PowerShell Core v7.0.0-preview.3 on Ubuntu 18.04.3 LTS

@iSazonov  Yes; to quote from the OP:

Whether or not the PSReadLine module is loaded makes no difference.

It's natural for people to assume this is PSReadLine-related, even though it isn't, and there's already more information in the linked issue. I suggest we tag this one 'Resolution-External' and keep it open in parallel, for discoverability.

'Resolution-External' will auto closes the issue.

If a native command is last in a pipeline will it rise the issue too?

'Resolution-External' will auto closes the issue.

I guess a closed issue is better than none for discoverability, but it would more make sense to me to keep such issues open until the issues is actually resolved (externally).

If a native command is last in a pipeline will it rise the issue too?

more and less become effective no-ops (buffering changes aside) if you don't use them as the last pipeline segment, so the issue doesn't arise then - but it's  a useless use case.


No Data in Pimatic

Hello,
I use your plugin with this setup:
{
"id": "smartmeter",
"class": "Smartmeterdevice",
"name": "Smartmeter Strom",
"serialport": "/dev/ttyAMA0",
"baudRate" : 9600,
"dataBits" : 7,
"parity" : "even",
"stopBits" : 1,
"flowControl" : true
},
My electric meter gives with cat /dev/ttyAMA0 following information:
/ISk5MT671-0001
1-0:0.0.0*255(331300-5033124)
1-0:1.8.1*255(033238.2842)
1-0:96.5.5*255(80)
0-0:96.1.255*255(39225479)
!
There is only the meter reading 1-0:1.8.1*255(033238.2842) what means 33238.2842 kWh 	interesting.
Now your plugin shows in my Pimatic installation no data.
Hope you can help me.
Best regards
Andre

Hmmm this seems related to this issue : https://github.com/saberone/pimatic-smartmeter/issues/4


Update opportunity-kanban-view.md

Rename: Opportunity sales process (old name)
to new name: Sales Process

Hi @antrung699
Thank you so much for submitting this PR and sorry for the delay in merging it. The PR notification went into an unmonitored folder. The changes will be published shortly.
Thanks!


SettingsUI: keep "page" when changing profile

Description of the new feature/enhancement
I have seven profiles.  Yesterday I was leisurely switching among them in the SettingsUI, essentially comparing them.  It would have been a better experience if the UI stayed on the same "page" ("General", "Appearance", "Advanced") when I switched profiles instead of automatically going to "General".

So for the record, I experimented with this before!
https://github.com/microsoft/terminal/pull/8844#issuecomment-764925226

should going back to "Windows PowerShell" have Appearance selected?

So, I had it that way in 67e38a3, but that actually felt weird. Idk, it was definitely a subjective choice, but it felt like changing profiles should return to the general pivot. Reverting that would be easy. It would I guess also fix the "rename profile returns to the general pivot" problem

idk, I'm conflicted on this. It felt weird before, but it seems like it makes sense on paper. I'll leave it for the team to discuss. The fix is trivial.
<discussion>

In my case, I wanted to compare the appearances of various profiles.  Moving around among the profiles took twice the mouse clicks.  Starting at "General" is arbitrary (might as well start elsewhere) and doesn't seem to serve any purpose.

Team consensus: lets do it.

Suits me.  Watch all the complaints.  😁

Team consensus: lets do it.

Suits me. Watch all the complaints. 😁

Sorry? "Let's do it" is common parlance for "let us implement the thing that has been requested"...

I figured that.  What's your point?  Was I wrong to close the issue?

Yeah! We're gonna use this issue to track completing the work to make the profiles page remember where it was. Why'd you close it? :smile:

I figured a decision was the end.  You don't always use an original suggestion to do the tracking, do you?

You don't always use an original suggestion to do the tracking, do you?

Not always, but like 99.97% of the time we do. We usually only file our own versions for follow-ups. Like, #653 is the thread tracking adding support for quake mode, but #8888 is the thread for "everything related to quake mode after the initial PR". The only exceptions I can think of are #1564 and #5000

AHH! I understand. Sorry! Yeah, Mike's got it here. We love to use the original issue since it lets the suggester know how we're doing, too. :smile:

Then what ... someone comes back here, notes that it has been implemented, and closes it?  I don't see a lot of that but I admit that I'm not paying close attention to many issues.

So, we use these issues (user suggestions, user bug reports, user requests even) as the currency of work tracking in our project. We talk about them all the time -- "let's fix 376" or "9920 is a good idea, let's do it" and then use that issue mostly for the lifetime of the request.
We mention it in any pull request, too:

"Closes #XXXX" or "Fixes #XXXX" are two keywords GitHub picks up to automatically close the related bug when the pull request merges
"Related to #XXXX" adds a link between the bug and the PR so somebody who's interested in how the work is going can follow it.

Once the feature is complete and released, the commit and the changelog entry mentions the pull request number. Our release bot finds all issues "Closed" or "Fixed" by any PR mentioned in the changelog, and tells the original submitter that their submitted issue, request, etc. was addressed in whatever version mentioned it.

I realize that I set a bad precedent with your issues specifically. I've closed them early when I did not agree with them, only to come back and fix the issue anyway. That makes it look like we track the work elsewhere . . . and I'm sorry about that! :smile:

OK, that sounds vaguely familiar.  Thanks for the explanation.


r50_deformable_detr_coco_mot17.pth

Where can I find the pretrained "r50_deformable_detr_coco_mot17.pth"?
In Google Drive, I can only see checkpoints for DanceTrack, SportsMOT, and a generic one

Sorry, I must have missed this. Now it has been uploaded.


Relax diamond restriction on iarange nesting for enumeration

Addresses #1391
This partially relaxes the no-loops restriction on iarange nesting during enumeration. After this PR, there is still a restriction involving iarange nesting, and I believe this is optimal for the complexity class. I am working on a parsimonious statement of this restriction in #1391 .
The change in this PR was basically to compute the parent ordinal independently for each connected component of tensors at each ordinal, rather than compute a single parent for each ordinal.
Tested

[x] refactoring is covered by existing tests
[x] added a regression test for ubersum
[x] added a regression test for TraceEnum_ELBO

Looks great! I checked that this works with one of the dag examples in mcmc.test_valid_models, as well. I would like to dig in a bit more, and understand how the Expected tree-structured iarange nesting.. warning is triggered with this refactor in another pass shortly.

@neerajprad let me first add a test for ubersum("ae,be,abe->", ..., batch_dims="ab") and make sure we can handle this.


[🐞] Qwik playground / tutorial broken

Which component is affected?
Qwik Playground
Describe the bug

Error: Unable to require() path<EMAIL_ADDRESS>from a browser environment. at globalThis.qwikServer.globalThis.require (eval at g (https://qwik.builder.io/repl/repl-sw.js:1:341), <anonymous>:33:11) at eval (eval at g (https://qwik.builder.io/repl/repl-sw.js:1:341), <anonymous>:164:20) at eval (eval at g (https://qwik.builder.io/repl/repl-sw.js:1:341), <anonymous>:783:26) at g (https://qwik.builder.io/repl/repl-sw.js:1:377) at async B (https://qwik.builder.io/repl/repl-sw.js:1:1357) at async F (https://qwik.builder.io/repl/repl-sw.js:12:5469)

(linked https://github.com/BuilderIO/qwik/issues/1037 for x-ref in case somebody googles around)
Reproduction
https://qwik.builder.io/tutorial/welcome/overview/
Steps to reproduce
Just try it 😨
System Info
M1 mac pro / Google Chrome

Additional Information
No response

Hi @ubarbaxor 👋
Thanks for creating this issue 🙏  Imo it looks like a duplicate of https://github.com/BuilderIO/qwik/issues/3206. Hope it's okay, when i close this. If not, pls let me know 👼
Thanks again 😃


chore: Update codegen

This PR is to:

Update codegen

Signed-off-by: cegao<EMAIL_ADDRESS>

Coverage remained the same at 70.839% when pulling 8d6f92916c436adf071fc47b3ab0b22be25f60eb on gaocegege:gen into c895046976f201fd9b1bff49da75ff0c93c4b0f7 on kubeflow:master.

Coverage remained the same at 70.839% when pulling 8d6f92916c436adf071fc47b3ab0b22be25f60eb on gaocegege:gen into c895046976f201fd9b1bff49da75ff0c93c4b0f7 on kubeflow:master.

/assign @terrytangyuan

/assign @terrytangyuan


chore(cfn): restrict System Key by EC

Issue #, if available:
Description of changes:
Squash/merge commit message, if applicable:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This CFN was successfully deployed.


Error 3: Syntax error!

$ liquidsoap -c process.liq
At process.liq, line 8, char 34-36:
def multiband_band(process, from, to, attack, release, ratio, threshold, gain)

Error 3: Syntax error!

char 34-36 corresponds to "to" field and is highlighted in red in output
using it on ubuntu 22.04lts
$ dpkg -l |grep liquid
ii  liquidsoap                           2.0.2-1build2                           amd64        audio streaming language

This needs to be integrated as part of a bigger Liquidsoap script.
Would you be able to give me an example of how you're running it? (removing any sensitive data if necessary)

I'm assuming this is fixed since there's no answer. Closing! Feel free to reopen if there's still an issue.


Update to hie-bios 0.3

Yesterday I released hie-bios-0.3 which includes quite a number of improvements over the previous release. ghcide will need to be updated for the API changes and could take advantage of some new features such as reprorting progress when trying to load a cradle.

https://github.com/digital-asset/ghcide/pull/257 at least makes sure that things compile. Taking advantage of new features will take more time but we can probably at least benefit from some bugfixes.

fixed in #257
We have #258 for the non-cradle and #113 to track the missing features.


Add variants required for square blue icon share links

What / Why

Add variants required for square blue icon share links
Reasoning: https://trello.com/c/LNOqGMgZ/49-build-social-media-links
The existing component styles use position: absolute for the icons, which doesn't work nicely when we need them square. This is because the absolutely positioned icons don't affect the width/height of the DOM, so adding square background styles just ended up in overlapping, fiddly overflows, and made it difficult to align the text in the centre of the icons.
Therefore, I've used display: flex along with inline-block to get the styles we want.
I added a min-width at a certain breakpoint to make the reflow look nicer, but this is specific to the page we're going to put this variation on. (i.e. because we know the social icons / share link text that will be on the page, we can make the shorter sized social links look nicer when they stack next to longer social links by adding a min-width.) Of course that CSS rule is not very helpful if this variation was used on other pages that are using an arbitrary number of social links.

Visual Changes

Multiple examples added, so will need a Percy review.

Thanks for the suggestions @andysellick - It should be ready for another review. I've refactored the implementation completely to try and separate layout and presentation. Hopefully I haven't overdone it, as I've ended up with five different variants. However I couldn't think of an other way of representing all the changes needed without creating the five variants. All of the new variants combine together to give us the icon styles we need for our other project. I've added them as separate commits to try and make it easier to review.
@matthillco Thanks for looking at the Firefox zoom as I didn't check that, I've incorporated your suggested fix into the work 👍 .

For the flexbox layout, I was wondering about this kind of thing...
.gem-c-share-links--flexbox {
  .gem-c-share-links__list {
    display: flex;
    flex-wrap: wrap;

    @include govuk-media-query($until: tablet) {
      flex-direction: column;
    }
  }

  .gem-c-share-links__list-item {
    flex: 1 1 0px;
  }
}

Which gives the below - expands to fill the available width on a wide screen, but allows for overlap with extreme content, then collapsing to a single column on mobile.

Mobile
Tablet
Desktop

@AshGDS I think there might be too many options here now. I'd do one for the layout, and one for the new icon style.

Thanks @andysellick - I have moved back to two variants and pushed a new commit to review.
Thanks for writing that code - on the square icons it behaves a bit differently because my icons were inline-block which meant they influenced the width of the links, and therefore led to scenarios like this:

Therefore I've reverted back to position: absolute to try and better accommodate using your flex options. Using position: absolute which was simpler than I initially thought, apologies. This works well, but when two icons are on a row on their own it will look like the below screenshot - are we OK with this?:

As I'm back using position: absolute for the icons, we can also go back to just using the existing columns: true functionality, which makes it collapse like the non-square icons would. The minor flaw with columns: true is that the text will wrap even if there is still space for the text to the right.

I've left it with the absolute positioned icons, and using the existing columns: true functionality for now as this solution requires the least amount of new code, but let me know if you think I should look at using flex.
Apologies if I'm over complicating this...maybe I'm worrying too much about accommodating the Anything else that might be included that could have quite a long name example, when it will likely never show up in production.

Thanks @matthillco 👍  - it's a good idea, I did a small spike however and space-evenly lead to scenarios like this:

I appreciate your idea was just a starting point though. My original implementation was a more complex iteration of display: flex, but because I had used a min-width on the list elements to make the wrapping of items look nicer, and I wasn't using the existing columns: true functionality, it was the cause of some of the back and forth in the PR.
I think the problem is trying to make elements of varying widths act and wrap perfectly, and therefore any solution we use is going to have a trade off with either alignment or text wrapping.
I agree that position absolute is fiddly, though as it's the existing functionality of the share links I'm more inclined to keep using it just to make this PR as small as possible. I'm also leaning towards just go with the existing functionality of columns for this reason.

Thanks @matthillco 👍 - it's a good idea, I did a small spike however and space-evenly lead to scenarios like this:
I appreciate your idea was just a starting point though. My original implementation was a more complex iteration of display: flex, but because I had used a min-width on the list elements to make the wrapping of items look nicer, and I wasn't using the existing columns: true functionality, it was the cause of some of the back and forth in the PR.
I think the problem is trying to make elements of varying widths act and wrap perfectly, and therefore any solution we use is going to have a trade off with either alignment or text wrapping.
I agree that position absolute is fiddly, though as it's the existing functionality of the share links I'm more inclined to keep using it just to make this PR as small as possible. I'm also leaning towards just go with the existing functionality of columns for this reason.


ChannelInitializer may be invoked multiple times when used with custo…

…m EventExecutor.
Motivation:
The ChannelInitializer may be invoked multipled times when used with a custom EventExecutor as removal operation may be done asynchronously. We need to guard against this.
Modifications:

Change Map to Set which is more correct in terms of how we use it.
Ensure we only modify the internal Set when the handler was removed yet
Add unit test.

Result:
Fixes https://github.com/netty/netty/issues/8616.

You may want to document about this behavior in Javadoc though, e.g. risks of overriding handlerRemoved() and using a certain EventExecutor implementation.

Good idea

Am 05.12.2018 um 10:37 schrieb Trustin Lee<EMAIL_ADDRESS>You may want to document about this behavior in Javadoc though, e.g. risks of overriding handlerRemoved() and using a certain EventExecutor implementation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@trustin actually I think it is fine to not add docs as the same is true for other methods as well.

@ejona86 any other comment ?


Can add spriteFx pipeline to Video game object

Please do not update the README or Change Log, we will do this when we merge your PR.
This PR (delete as applicable)

Adds a new feature

Describe the changes below:
Can add spriteFx pipeline to Video game object.
See this test code, which add Components.FX by Phaser.Class.mixin.

Sure, why not. Should be ok.


Static Web App Tutorial could be improved by including a YAML file section

This page is missing content regarding the creation/modification of the yml file to build your web app. It was confusing to me to go through the quick start and to see the web app in Azure report to me there was no content for it to publish. Incorporating a basic YAML file would assist in making this a better tutorial.

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

ID: 63230b97-2269-3fed-7fca-882bb380c5f6
Version Independent ID: feb2708e-5cea-4ad8-eaaa-f3061e18c11a
Content: Quickstart: Building your first static web app with Azure Static Web Apps using the Azure portal
Content Source: articles/static-web-apps/get-started-portal.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

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

FWIW you'd need to update this page's content and add a YAML file to https://github.com/staticwebdev/angular-basic along with adding an instruction to copy/paste the api token from the Azure Portal to the api token field in the YAML file.

@bh3605 Thanks again for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Consolidate .landr and www folders

We should only use a single folder for assets, dist dir ect.

We should go with www


[WIP] Run functional tests on Travis macOS VM

https://github.com/DevExpress/testcafe/issues/791#issuecomment-254445543


Windows stack canaries obscure dataflow

Here is a simple C program that forces the use of canaries in quux
size_t quux(const char *p) {
    char buf[0x10] = { 0 };
    strcpy(buf, p);

    return strlen(buf); // NOTE: we return a strlen()
}

int main(int argc, char **argv) {
    printf("%d\n", quux(argv[0]));
}

If we compile this as an x64 PE and load this into binja and fix up the names, we get the following:
int64_t quux(char* arg1)
140001011  void var_48
140001011  int64_t rax_1 = *data_14001d018 ^ &var_48
14000101e  void var_28
14000101e  void* rdi = &var_28
140001028  for (int64_t rcx = 0x10; rcx != 0; rcx = rcx - 1)
140001028      *rdi = 0
140001028      rdi = rdi + 1
140001034  strcpy(&var_28, arg1)
14000103e  strlen(&var_28)
140001057  return canary_check(rax_1 ^ &var_48)

int64_t main(int32_t arg1, void* arg2)
14000108e  printf(data_14001d000, quux(*(arg2 + 0)))
140001099  return 0

Note that in quux, at 103e we compute the strlen, and according to hlil, discard it to return the value of the cookie check function. This is obviously inaccurate.
If we look at the asm for that canary_check function, it has a custom calling convention that does not touch rax:
canary_check:
140001130  483b0de1be0100     cmp     rcx, qword [rel data_14001d018]
140001137  f27512             bnd jne 0x14000114c  {data_14001d018}

14000113a  48c1c110           rol     rcx, 0x10
14000113e  66f7c1ffff         test    cx, 0xffff
140001143  f27502             bnd jne 0x140001148

140001146  f2c3               bnd retn  {__return_addr}

140001148  48c1c910           ror     rcx, 0x10

14000114c  e9ab020000         jmp     sub_1400013fc

This problem is even worse in the the arm64 case -- because the canary is installed by a function that also has a custom calling convention, dataflow in and out of the function is lost:
int64_t quux()
140001e94  int64_t var_20 = 0
140001e94  int64_t var_18 = 0
140001ea0  strcpy(&var_20, canary_install())
140001ec4  return canary_check(sx.q(strlen(&var_20)))

Note the lack of function arguments, because the canary_install function clobbers x0. canary_install is implemented as :
canary_install:
140001000  ff4300d1   sub     sp, sp, #0x10
140001004  f1000090   adrp    x17, 0x14001d000
140001008  310a40f9   ldr     x17, [x17, #0x10]  {data_14001d010}
14000100c  f16331cb   sub     x17, sp, x17 {var_10}
140001010  f10700f9   str     x17, [sp, #0x8 {var_8}]
140001014  c0035fd6   ret

While I'm capable of fixing these functions as they pop up, this seems to be a use-case that should be well supported.
I see two possible fixes, though obviously I'm open to others:

Introduce a naked or empty calling convention -- ideally these functions would end up tagged as such and thus not affect the arguments in/out of the functions. I don't have a good sense for other bad side effects this might have, so it's entirely possible its a bad idea.
Platform specific heuristics/fixups -- identify the stack check functions on WIndows PE's and fiddle with the calling convention only on these functions to correct dataflow.

Followup, there are two distinct things happening here.

Input arguments to a function being clobbered when they shouldn't be (the arm64 input case). From talking to Ryan this is due to the stack pointer being adjusted inside the canary install function.

The canary_check function on both platforms is basically this:

def canary_check():
  if canary is good:
      return // without clobbering any registers
  else:
    kill()

def kill():
  TerminateProcess(GetCurrentProcess())

kill is noreturn, however binja cannot determine this with some further enlightenment/heuristics. As a result, kill is considered returnable, and therefor the canary_check function ends up clobbering all the registers that kill clobbers (which includes rax). With kill updated to be noreturn, analysis works as expected.
The easiest solution might just be to have a windows.py or something that looks for TerminateProcess called on the result of GetCurrentProcess and flag those functions as noreturn? This pattern would also apply to RaiseException with certain exception codes.

The bug specifically in this issue is due to noreturn analysis taking precedence over typelib noreturn annotations. Will be fixed in 3.1.3484-dev


Migrate aspnet-mssql sample to .NET 5 🚀

Updated MVC app to .NET 5
Added SQL Server health check as nginx-aspnet-mysql sample
Updated README.md files accordingly

@aiordache please take a look ;)

Hi, @stvansolano. As I check on the Startup.cs. There is no connection to the database, also no DbContext on Controller.
These are just 2 standalone containers without any connection.


SFR-620 header with search

ResearchNow Jira Ticket
Things this PR has changed

Link missing URL error message
Consolidated Header and PageTitle
Added ResearchNow specific SearchBar error state
Made SearchBar follow the A11Y guidelines in Metronome

Front End Review:

[ ] View the example in Storybook
[ ] Check against the metronome page

[ ] note to self: Add a story for AT-264: H2

@helenvholmes
figma:

sb:

Metronome: http://themetronome.co/components/at-264-h2/?fresh=true
I'm thinking this class looks a lot like AT-251: H3 and if it's easy we can include that mixin but if not, throw it on the pile too

I think the description link to the Storybook change is actually http://localhost:9001/?path=/story/headerwithsearch--research-now-search-promo-header.

I think the description link to the Storybook change is actually http://localhost:9001/?path=/story/headerwithsearch--research-now-search-promo-header.

(I don't understand)


Problem with Data From Business Object

Hello ,
I'm having trouble adding data, follow the object's link, before I register.
https://prnt.sc/tma872
After I register I check the DataSources the columns are null
https://prnt.sc/tmaaam
One thing you notice is that the reference property is bringing the data
https://prnt.sc/tmabhr
follows the reporter file, the property remains null, if I try to enable it I get an exception
https://prnt.sc/tmacrs
Thanks good night.

Hello,
Could you make a simple test project that demonstrates the problem?

Hello, I created a simple example based on my routines.
Thank you
TestBusinessObject.zip

Hello,
Thank you for the test project. You should register data in the following way to enable nested data sources:
_report.RegisterData(_objectBusiness, "Order", 3); // where 3 is the max nesting level


Cannot specify the type of an expression involving a Class returned from a function

Bug Report
🔎 Search Terms
path dependent, untypable, nested class type, nested class reference
🕗 Version & Regression Information
No version known which works correctly
⏯ Playground Link
https://www.typescriptlang.org/play?jsx=0&ts=4.2.3#code/MYewdgzgLgBAjjAvDAPAFQHwAoAeAuGNASiQwCgYZgAbAQwghgGEYBvCymAWyRhw8qhIUAE4BXYFBAisJVgF8OiskOjxavMAFMA7jCxwsAIgBMRokVkwA9NZhQAngActMAJYQCcFNBFuwAOYYAHRMKuBqcABGXj6i-kGhmrr6hqbmliS2MACiIiLSZGTZAHIgUK5QABa0sNUe7oy01BAgMLRgjRBirjpu1TBg4DBOINQOXNJOVW7AVCAAJlpFqrAA1rxWiOSCdAzMbALzwuKS0lYKSisR6xrI2npYa7KZNnaOLo0EYaswazEHe4pJ4vKzZPIFERFIA
💻 Code
const q = <T>(x: T) =>
  class C {
    m = x
    constructor() {}
  }

const qa = new (q("2"))() // type is: q<string>.C
const qb: q<string>.C = new (q("2"))() // Error

// Note that this is also an issue with non polymorphic code

const k = () =>
  class C {
    constructor() {}
  }

const ka = new (k())() // type is: C
const kb: C = new (k())() // Error

🙁 Actual behavior
I can't explicitly type the expressions
🙂 Expected behavior
I would expect to be able to type this expression.
Ideally using the signature inferred by the compiler

It's possible to infer the type by doing
const ka = new (k())() // type is: C
const kb: InstanceType<typeof ka> = new (k())()

For non-polymorphic version:
const kb: InstanceType<ReturnType<typeof k>> = new (k())();

Unfortunately it doesn't work in the polymorphic version, because ts doesn't support Generic Values.
There are some workarounds in https://stackoverflow.com/a/59072991/3016883 and https://stackoverflow.com/a/62724646/3016883

const doesn't create a new type; this is the intended behavior. There are many ways to refer to this type depending on what you want to do with it.

@RyanCavanaugh thanks for having taken the time to review this issue;
are all the ways to refer to this type enumerated above or do you know others? (or know where I can find others)
Thanks again!


21: Add clang-format 15

Please verify that I have the correct SHAs. I validated Darwin and Linux but was unable to check Windows.
Also, on the googleapis site, it seems like there are 4 releases per iteration (just going off of release date). Darwin, Windows, and Linux must be 3, but what is the fourth?

Also, on the googleapis site, it seems like there are 4 releases per iteration (just going off of release date). Darwin, Windows, and Linux must be 3, but what is the fourth?

BSD?
I also don't have a Windows machine to validate the Windows SHA.  And it looks like the promised script from the comment never materialized, cc @sanrut01
Also for my info, is this actually the clang-format 15 release, or is it a random revision of LLVM at some point after v15?  My guess is that it's the latter?

Also, on the googleapis site, it seems like there are 4 releases per iteration (just going off of release date). Darwin, Windows, and Linux must be 3, but what is the fourth?

BSD?
I also don't have a Windows machine to validate the Windows SHA. And it looks like the promised script from the comment never materialized, cc @sanrut01
Also for my info, is this actually the clang-format 15 release, or is it a random revision of LLVM at some point after v15? My guess is that it's the latter?

Thanks for the quick merge, good guess with BSD. My validation of the windows version was just to open the file in the viewer, both the sha that I had and the 11.0.0 version had “can’t be run in DOS mode” or something like that at the beginning, so I’m at least fairly confident. Mac and Linux I just validated by downloading all 4 versions from the latest date and seeing which ones ran on my laptop / in docker.
is there a better place to download the files from? I don’t know the story but it seems like if the current site is meant for Google APIs then it’s liable to change. Maybe not though, I didn’t look into it.
if you just google clang-format, 15.0.0 is the latest version in the docs. And when you run the downloaded file, it’s what —version prints out, so it at least exists.

Would you be willing to set up a GitHub actions / circleci pipeline? The free tier is probably sufficient, I’d be willing to write a quick test to validate the project cross-platform

Click buttons is probably all that is needed. Let me take a look and see what I can come up with. I appreciate that you’re very responsive!

Hey here's a possibility - https://github.com/ssciwr/clang-format-wheel provides wheels for clang-format. So you could just have that as a dependency and cover more OSs instead of having to deal with the downloads.
Also it turns out that file (or python-magic, as usable from py) is able to tell you what the executable target is for. I started something to go through the list and autodownload here: https://github.com/tgross35/clang-pre-commit-hooks/blob/0f0286d19ea137e1ca4c05c4df6f3bfb344a9486/clang_sha_getter.py
...But realized there are like quite a few binary types and I don't even know what all the differences are. Hence the reasoning for my suggestion.
00:'ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV)'
01:'ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux)'
02:'ELF 64-bit LSB executable, x86-64, version 1 (SYSV)'
03:'ELF 64-bit LSB shared object, x86-64, version 1 (SYSV)'
04:'Mach-O 64-bit arm64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|WEAK_DEFINES|BINDS_TO_WEAK|PIE|HAS_TLV_DESCRIPTORS>'
05:'Mach-O 64-bit x86_64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|BINDS_TO_WEAK|PIE>'
06:'Mach-O 64-bit x86_64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|WEAK_DEFINES|BINDS_TO_WEAK>'
07:'Mach-O 64-bit x86_64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|WEAK_DEFINES|BINDS_TO_WEAK|PIE>'
08:'Mach-O 64-bit x86_64 executable, flags:<NOUNDEFS|DYLDLINK|TWOLEVEL|WEAK_DEFINES|BINDS_TO_WEAK|PIE|HAS_TLV_DESCRIPTORS>'
09:'Mach-O universal binary with 2 architectures: [x86_64] [arm64]'
10:'PE32 executable (console) Intel 80386, for MS Windows'
11:'PE32+ executable (console) x86-64, for MS Windows'

Hey here's a possibility - https://github.com/ssciwr/clang-format-wheel provides wheels for clang-format. So you could just have that as a dependency and cover more OSs instead of having to deal with the downloads.

From my pov, it feels like an additional point of failure to pull in a package just to pull in a binary that we already (mostly) know how to download.  Also I admit I'm kind of paranoid about the chain of trust here, since we're downloading and running binary code on user systems.  I trust the Chromium build (in part because all Chromium devs have to trust it), but do we want to trust an additional random person?

Well for any particular version (or at least the latest version) there are only 4 options, right? My guess is that the exact way Chromium built clang-format changed over time and that's what you're seeing?

The main kinds are MacOS 64 bit, MacOS arm, MacOS universal, Linux 32 bit, Linux 64 bit, Windows 32 and Windows 64, I'd need to double check which ones are relevant to the latest release. Probably Universal and the 64-bit options though.

From my pov, it feels like an additional point of failure to pull in a package just to pull in a binary that we already (mostly) know how to download. Also I admit I'm kind of paranoid about the chain of trust here, since we're downloading and running binary code on user systems. I trust the Chromium build (in part because all Chromium devs have to trust it), but do we want to trust an additional random person?

Yeah I get that part for sure, just trying to think about the edge cases. At least for the mac binaries, there's a pretty good chance of a relatively common platform mismatch as-is. Maybe we could replicate the logic

Heya, there might actually be an easier option out there. https://clang.llvm.org/docs/ClangFormat.html clang-format includes a script to format only a diff. Might be interesting to look into

Yes, this repo already contains that script: https://github.com/jlebar/pre-commit-hooks/blob/master/git-clang-format


Error on relation column

Hello,
I just started a site with your project.
I created a model (Frequency) with two relations: a Many to Many to link cities and a One to Many to link a radio.
I would like to display the name of the linked radio in the table, but I have the following error.

Return value of Fjord\Crud\CrudColumnBuilder::component() must implement interface Fjord\Contracts\Page\Column, null returned

I put the code concerned below.
app/Models/Frequency.php
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Fjord\Crud\Models\Traits\TrackEdits;

class Frequency extends Model
{
	use TrackEdits;

	/**
	 * Fillable attributes
	 *
	 * @var array
	 */
	protected $fillable = [
		'frequency', 'active', 'radio_id'
	];

	protected $casts = [
		'active' => 'boolean'
	];

	public function cities() {
		return $this->belongsToMany('App\Models\City');
	}

	public function radio() {
		return $this->belongsTo('App\Models\Radio');
	}
}

app/Models/Radio.php
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Fjord\Crud\Models\Traits\TrackEdits;
use Spatie\MediaLibrary\HasMedia as HasMediaContract;
use Fjord\Crud\Models\Traits\HasMedia;
use Fjord\Crud\Models\Traits\Sluggable;

class Radio extends Model implements HasMediaContract
{
	use TrackEdits, HasMedia, Sluggable;

	[...]

	public function frequencies() {
		return $this->hasMany('App\Models\Frequency');
	}
}

fjord/app/Config/Crud/FrequencyConfig.php
<?php

namespace FjordApp\Config\Crud;

use Fjord\Crud\CrudShow;
use Fjord\Crud\CrudIndex;
use Fjord\Crud\Config\CrudConfig;
use Illuminate\Support\Str;

use App\Models\Frequency;
use App\Models\Radio;
use FjordApp\Controllers\Crud\FrequencyController;

class FrequencyConfig extends CrudConfig
{
	[...]

	/**
	 * Build index page.
	 *
	 * @param Fjord\Crud\CrudIndex $page
	 * @return void
	 */
	public function index(CrudIndex $page)
	{
		// Expand html container to full width.
		$page->expand(false);

		$page->table(function ($table) {
			$table->col('Fréquence')->value('{frequency}')->sortBy('frequency');

			$table->relation('Radio')
				->related('radio') // Relation name.
				->value('{name}') // Related attribute to be displayed.
				->routePrefix(
					Crud::config(Radio::class)->route_prefix
				);
		})
			->sortByDefault('id.desc')
			->search('frequency')
			->sortBy([
				'id.desc' => __f('fj.sort_new_to_old'),
				'id.asc' => __f('fj.sort_old_to_new'),
			])
			->width(12);
	}

	[...]
}

I also give you the stack trace of the error.

Can you help me solve the problem?
thank you,
Deniz.

Hello Deniz,
Can I see the corresponding Config?

Sorry, my message went too fast. :)
I modified the first post.
Deniz.

This should fix the bug: https://github.com/aw-studio/laravel-content-administration/commit/d3577723da195cabfd1a341b226aa180f2375752
Additionally I noticed Crud::config is deprecated. Use this instead:
use Fjord\Support\Facades\Config;

Config::get(Radio::class)->route_prefix

The fix worked well for my first bug.
Now I have another error: "Trying to get property 'route_prefix' of non-object".
The Config::get function returns null.
Deniz.

Use Config::get(RadioConfig::class) instead.

Hi,
Sorry to bother you again. I am coming back to you because I have another problem. The code you gave me worked.
On the radio link, his name is missing and the link leads to a 404 page. Without the /edit, the link works.

Thank you,
Deniz.

Hey Deniz, thank you for the feedback. The edit suffixed link was related to an outdated version where the detail page was available through that link.
This should fix the bug https://github.com/aw-studio/laravel-content-administration/commit/84d1a82d70037cb2a0310fe61a394f4e922ab2be

Hello,
I just used your code to fix the links and it worked.
For the display of the name of the relation, I was able to correct it by modifying the code.
In the getColValue function of the TableCol.vue file, the "col" parameter had for value the following character string: "{name}".
I add an if loop on the type of the parameter and put the parameter in the value variable if it's a string.
The code is below.
resources/js/components/Fjord/BaseIndex/TableCol/TableCol.vue
getColValue(col, item) {
	let value = '';
	let col_type = typeof col;

	if (col_type == "object") {
		if (col.value_options) {
			value = col.value_options[item[col.value]];

			if (value === undefined && col.default_value) {
				value = col.default_value;
			}
		} else {
			value = col.value;
		}
	} else if (col_type == "string") {
		value = col;
	}

	// Regex for has {value} pattern.
	if (/{(.*?)}/.test(value)) {
		value = this._format(value, item);
	} else if (item[value] !== undefined && item[value] !== null) {
		value = item[value];
	}

	return this.format(value);
},

I don't know if that can help you.
Regards,
Deniz.

Thanks, that has been fixed as well.


ENH: speedup tests + examples using the get_tris + connectivity

Related to #1503

I'm not sure which examples and tests you have in mind...?

Closing this one since it's really old and I'm not sure we still need it, but @dengemann if you have more specifics feel free to reopen


Map - Put remaining road markings

Arrows, lane bus marking, etc

Road markings of intersections and directions near intersections will be added after fix Issue #84


Implement leaner inline PipeScheduler

Should call the action directly from the Schedule override

Closing - it wasn't the pipe scheduler causing the suspicious dispatching details in the stack


How to mock stuff in external crates?

Hey, cool crate! I'd really like to use it to mock methods in an external crate's struct but I can't really find an example on that. Basically, something like
#[faux::create]
use SomeOtherCrate::SomeStruct;

Is this currently possible?

Unfortunately it is not possible to mock external crates. faux needs to have access to the structure of the struct and its methods and it won't get that from just an import.
If you control the external crate though, you can export the mocks under a feature so other crates can use those mocks.
The general advice for testing is already to not mock what you don't own. My recommendation is that if it is really impossible to use the real version in tests (i.e., using some I/O), then do a thin wrapped around it. This thin wrapped can be mocked and allows you to have structs and methods that are more specific to your domain rather than the more general methods in the crate.

Alright, thanks. After quite a bit of research, I've come to the same conclusion. Though for complex foreign types, it's quite annoying to write a wrapper especially if the type is heavily generic. While not strictly in scope of this issue, do you have some advice on how to more conveniently generate a wrapping derived type for the purposes outlined above?

It depends on your use case, if what you want from it is simple then make the wrapper simple by removing all the generics out of it, or at least the ones that you don't need today.
A benefit of writing this wrapper is that it let's you simplify the external API by making it focus on your need/domain rather than the general need that it was built for. And as a general rule of thumb, even if you think that maybe you will need it to be generic in the future, if it doesn't need to be generic today then don't add that complexity until you need it.

Thanks, we can consider this closed. However, I'd advise to change your readme as currently you mention the ability to mock external methods which doesn't seem to be possible.

In the README I mentioned mocking methods in external modules as you can mock methods of structs from other modules, just not from other crates. I see how this can misleading though, do you have ideas on a better wording?

How about plainly Methods in external modules (though not other crates)?


Fix type mismatch in call to PyUnicode_AsUTF8AndSize

@Mic92 Ping

Is this broken in nixpkgs atm?

Yes it's broken in nixpkgs.


Add PM readiness callback API.

This pull request currently won't build due, and requires the <mptcpd/mptcp_private.h> removal related changes in found in PR #107.


Set-ConsoleColors.ps1 should be able to update shortcuts too

Changing console color palettes for both default process registry settings and shortcuts are important to cover all cases. See https://github.com/lukesampson/concfg for shortcut API calls, maybe.

Console colors scripts have been deprecated; Windows Terminal handles that stuff better.


[router] Reduced allocations in router read path

A lot of multi-key specific state was moved out of VenicePath and into its subclasses, which makes single get queries require less heap space. This includes streaming (chunkedResponse), HAR (helixGroupId and requestId), longTailRetryThresholdMs and responseHeaders.
In addition, all kinds of VenicePaths are also getting their heap size reduced by leveraging shared instances where possible. For example:

The resourceName, storeName and versionNumber properties are replaced by a single reference to a shared instance of StoreVersionName, which is a new class obtained from a NameRepository (same idea as the PubSubTopicRepository).

The smartLongTailRetryEnabled, smartLongTailRetryAbortThresholdMs and longTailRetryThresholdMs properties are replaced by a single pointer to a RouterRetryConfig object, which is a simple facade wrapping the VeniceRouterConfig.

The responseDecompressor is now coming from a map of shared instances in the VenicePathParser.

Miscellaneous:

Various refactorings enable the VenicePath constructors to have fewer params and to make more of the properties final.

Cleaned up generics in the VenicePathParser and ScatterGatherHelper builder.

Deleted VeniceMetricsProvider which can be trivially inlined.

Minor tweaks to the main README page.

How was this PR tested?

CI.
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Carved out a portion of this big-ish PR into this other fairly trivial PR: #1405.

Rebased after merging https://github.com/linkedin/venice/pull/1405 to reduce the PR size, but also added a flaky test fix which still increases it a bit more 😅 ... oh well 🤷

Will be able to further reduce the size of this PR after rebasing on top of #1413.


where are the game assets?

how to download them?

For now you can download them here: https://github.com/kidscancode/Godot3_dodge
But you are right, the current version of the tutorial text is missing a link to the download.

Links are all being left until I'm of the final locations for everything.


Add new QS guide from Cordial.com called "Create a Bidirectional Data Share with Cordial"

In this guide we show how to setup a bidirectional data sharing capability between Cordial and our clients. This consists of two parts: (1) using Snowflake's secure data share feature to make Cordial data available to clients within their Snowflake account and (2) using the client's Snowflake account to ingest data into the Cordial platform using Cordial's automated Data Jobs feature.

I have read Snowflake's CLA and I accept the terms.

@iamontheinet - this looks good to me, the images didn't load locally here, but otherwise the format looked great.


Passed the scheduling argument through the *_generator function.

Previously, the scheduling parameter was not passed to the OrderedEnqueuer, which was needed in training.
Since the scheduling parameter is currently a binary decision, I've also altered the value to a boolean and changed the kwarg to be more descriptive.
It also, seems we may need a shuffle function inside the Sequence call. Currently, only the input is shuffled, whilst this ensures some randomness this could be improved on via a function call.
Also, @fchollet do you remember why in the generators you passed different seeds? - This is also missing from the current code, however, it is trivial to implement.

We already have a shuffle argument in fit. Is this the same thing?

Pretty much, but for Sequence. So every epoch the order of the batches will be shuffled.

1 - if this only affects Sequence, shouldn't it be part of its API, and not part of the *_generator API?
2 - the API should be consistent across the codebase, so we should use shuffle, not ordered

So a property in Sequence that the Enqueuer can poke?

1 - had to pass through the *_generator methods, since the OrderedEnqueuer isn't accessible else where, unless we make users pass the enqueuer into these methods? - the plus is that it will be even more flexible, down side is more work for the user.
2 - I'll alter when I'm back from vacation

@fchollet It keeps timing out on this test: tests/keras/wrappers/scikit_learn_test.py::test_regression_class_build_fn
However, I haven't touched anything related to it. How can I fix it?

@Dref360 Thanks for the suggestion!


Add generated_jit decorator

This adds the @generated_jit decorator as discussed.

How will this functionality interact with serialization?

I should try that. It should work, since the Dispatcher's infrastructure is basically reused, but as long as it's not tested...

Actually, serialization may require a little hackery...

Once we are sure everything is stable after the LLVM 3.7 merge, I think we can merge this.

OK, I think we are ready to merge.


Does the HBase Table needs to be empty before bulk load?

I wonder if this reason cause my failure...

No.


I can't get the plural string.

I am making a pot file in Japanese. And I have translated it into English po file.
pot is here:
# SOME DESCRIPTIVE TITLE.
# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
# This file is distributed under the same license as the PACKAGE package.
# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
#
#, fuzzy
msgid ""
msgstr ""
"Project-Id-Version: PACKAGE VERSION\n"
"Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2016-04-11 18:59+0900\n"
"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
"Language-Team: LANGUAGE<EMAIL_ADDRESS>"Language: ja\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n"

#: Sdx/Validation/StringLength.cs:15
#, csharp-format
msgid "現在{0}文字"
msgid_plural "現在{0}文字"
msgstr[0] ""
msgstr[1] ""

en.po is here:
# SOME DESCRIPTIVE TITLE.
# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
# This file is distributed under the same license as the PACKAGE package.
# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
#
msgid ""
msgstr ""
"Project-Id-Version: \n"
"Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2016-04-12 13:22+0900\n"
"PO-Revision-Date: 2016-04-12 13:23+0900\n"
"Last-Translator: \n"
"Language-Team: \n"
"Language: en\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n"
"Plural-Forms: nplurals=2; plural=(n != 1);\n"
"X-Generator: Poedit 1.8.7\n"

#: Sdx/Validation/StringLength.cs:15
#, csharp-format
msgid "現在{0}文字"
msgid_plural "現在{0}文字"
msgstr[0] "{0} character now"
msgstr[1] "{0} characters now"

c# code is here:
var assembly = Assembly.GetExecutingAssembly();
var stream = assembly.GetManifestResourceStream("Sdx.locale.en.message.mo");

Catalog catalog = new Catalog(new MoLoader(stream));
Console.WriteLog(catalog.GetPluralString("現在{0}文字", "現在{0}文字", 1, 1));
//log is `1 character now`

Console.WriteLog(catalog.GetPluralString("現在{0}文字", "現在{0}文字", 2, 2));
//log is `2 character now`

Two codes return the same string.
What is wrong with my code ?

Oh I'm sorry, I found the answer.I forgot to set the "CultureInfo".
Catalog catalog = new Catalog(new MoLoader(stream), new CultureInfo('en'));
Console.WriteLog(catalog.GetPluralString("現在{0}文字", "現在{0}文字", 1, 1));
//1 character now

Console.WriteLog(catalog.GetPluralString("現在{0}文字", "現在{0}文字", 2, 2));
//`2 characters now

Thanx you.

Glad you've found the solution.
Thank you for using NGettext and good luck with your project. =)


Add Link to Slack

Add link to Slack

I don't know how to generate the slack icon black/white like the other icons @cdbartholomew


🐛 [cloud_firestore_web] The method 'FirebaseFirestoreWeb.runTransaction' has fewer named arguments than those of overridden method 'FirebaseFirestorePlatform.runTransaction'

Bug report
Im trying to build my flutter web app. These are the dependencies I'm using:
  firebase_auth: 3.4.1
  firebase_auth_platform_interface: 6.3.1
  firebase_core: 1.19.1
  firebase_core_web: 1.6.6

But the build is failing and I'm getting this error
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore_web-2.6.9/lib/cloud_firestore_web.dart:101:14: Error: The method 'FirebaseFirestoreWeb.runTransaction' has fewer named arguments than those of overridden method 'FirebaseFirestorePlatform.runTransaction'.
  Future<T?> runTransaction<T>(TransactionHandler<T> transactionHandler,
             ^
../../development/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore_platform_interface-5.7.0/lib/src/platform_interface/platform_interface_firestore.dart:168:14: Context: This is the overridden method ('runTransaction').
  Future<T?> runTransaction<T>(TransactionHandler<T> transactionHandler,

Flutter doctor
Click To Expand
[✓] Flutter (Channel stable, 2.10.0, on macOS 11.6 20G165 darwin-x64, locale en-GB)
    • Flutter version 2.10.0 at /Users/hinnarutaba/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5f105a6ca7 (6 months ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/hinnarutaba/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] VS Code (version 1.67.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (2 available)
    • SM G935F (mobile) • ce011711628f4b2604 • android-arm64  • Android 8.0.0 (API 26)
    • Chrome (web)      • chrome             • web-javascript • Google Chrome 103.0.5060.134

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Same problem here :
/C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore_web-2.6.19/lib/cloud_firestore_web.dart:101:14: Error: The method 'FirebaseFirestoreWeb.runTransaction' has fewer named arguments than those of overridden method 'FirebaseFirestorePlatform.runTransaction'.
Future<T?> runTransaction(TransactionHandler transactionHandler,
Error: The method 'FirebaseFirestoreWeb.runTransaction' doesn't have the named parameter 'maxAttempts' of overridden method 'FirebaseFirestorePlatform.runTransaction'.
Future<T?> runTransaction(TransactionHandler transactionHandler,

@rutaba1
Please upgrade all flutterfire plugins to latest versions, specially, cloud_firestore_web which has been upgraded and that should resolve the issue you are reporting: https://pub.dev/packages/cloud_firestore_web/changelog#281

@darshankawar I don't want to update my flutterfire plugins because of the persistence issue in the auth

what  if i had updated the packages from old to new
cloud_firestore_web: ^2.8.1
cloud_firestore_platform_interface: ^5.7.0
firebase_core: ^1.20.0
firebase_core_web: ^1.7.1
and still got this error :
Error: Assertion failed:
options != null
"FirebaseOptions cannot be null when creating the default app."
at firebase_core_web.FirebaseCoreWeb.new.initializeApp (http://localhost:49827/packages/firebase_core_web/firebase_core_web.dart.lib.js:226:42)
at initializeApp.next ()
it is unable to initialize the firebase now.....................

what if i had updated the packages from old to new
cloud_firestore_web: ^2.8.1 cloud_firestore_platform_interface: ^5.7.0 firebase_core: ^1.20.0 firebase_core_web: ^1.7.1
and still got this error :
Error: Assertion failed: options != null "FirebaseOptions cannot be null when creating the default app." at firebase_core_web.FirebaseCoreWeb.new.initializeApp (http://localhost:49827/packages/firebase_core_web/firebase_core_web.dart.lib.js:226:42) at initializeApp.next ()
it is unable to initialize the firebase now.....................

Passing in the options parameter in the Firebase.initializeApp() solves this issue. However the updated auth dependency will cause persistent login state issue

However the updated auth dependency will cause persistent login state issue

Same as https://github.com/firebase/flutterfire/issues/9228#issuecomment-1195466404

why this is marked as blocked on customer response?


Pull down to refresh

Pull down to refresh in DiscoverFragment Adapter not update .

Because you need to configure the music service API

ok thank


NetBox 4 Compatibility

This PR contains commits required to maintain compatibility with the upcoming NetBox 4 release.
The NetBox DNS release that supports NetBox 4 will no longer be compatible with NetBox 3.
Code changes will be logged hereafter:

Adjusted GitHub workflows to reflect dropped support for Python 3.8 and 3.9
Replaced references to extras.plugins with netbox.plugins
Replaced object references with a reference by object id/pk in cases where a potentially unsaved object is referenced in filter() expressions
Removed the ready() method that accessed the database by calling ContentType.objects.get_for_model() because of a RuntimeWarning during initialisation
Changed the actions attributes in tables to be dictionaries instead of tuples because of NetBox PR #14670
Modified the 'Add Record' button in the base template for the zone detail view so it matches the rendition of the other buttons (using <button> instead of CSS on the <a> element
Adjusted the menu item buttons to the new UI look by removing the colours
Refactored filtersets to become compatible with htmx.ObjectSelectorView, specifically its _get_filterset_class() method.
Added the brief_field attribute to all model serializers
Added description to all models and to brief_fields for all serializers matching NetBox PR #15238
Workaround for NetBox #15351: Remove validators from ZoneSerializer if nested=True
Renamed ContentType to ObjectType and changed parameter name to object_type NetBox PR# 15327
Added tests for all filter sets to make automated testing by the extended NetBox filter set test framework possible NetBox PR #15369
Renamed the object_type attribute for the IPAM Coupling ipaddress_dns_zone_id CF to related_object_type NetBox PR #15366
Added missing filters detected by the new NetBox test functionality from NetBox PR #15369

Thank you for keeping such a detailed log of your changes! This was very helpful as a reference when drafting a migration guide for plugin maintainers. I just submitted a draft PR for it here; any feedback you have would be much appreciated!

Hi @jeremystretch, that was exactly the idea behind it - no need for others to have to find out what might go wrong. I was surprised how many deprecated features I was using :-)


WIN UI 3 Preview 3 XAML parser error using XAMLReader.Load when style is used in the XAML file

Describe the bug
I'm trying to convert a WPF loose XAML application to Win UI 3 preview 3.  I can add the style to the gird and the parsing is successful. If I add a reference to the style I get and XAML parser error. Below is the XAMML file I reading and the code. For  Win UI it looks like a can not use a streamreader as input to the XAMLReader like in WPF. But I found if I read the file, convert it to a string and replace a few characters i can do the read.      Parser has problem when Style="{StaticResource style1}" is added to the XAML file.
       string sFileName = CommonDataPathName + "clearwtest.xaml";
        StreamReader mysr = new StreamReader(sFileName);
        string test = mysr.ReadToEnd();
        test = test.Replace("\r\n", " ");
        test = test.Replace("\\", "");

        object rootObject = XamlReader.Load(test);

<Grid.Resources>

</Grid.Resources>
<Grid.RowDefinitions>

</Grid.RowDefinitions>
<Grid.ColumnDefinitions>

</Grid.ColumnDefinitions>

Microsoft.UI.Xaml.Markup.XamlParseException
HResult=0x802B000A
Message=XAML parsing failed.
Source=WinRT.Runtime
StackTrace:
at WinRT.ExceptionHelpers.g__Throw|20_0(Int32 hr)
at WinRT.ExceptionHelpers.ThrowExceptionForHR(Int32 hr)
at ABI.Microsoft.UI.Xaml.Markup.IXamlReaderStatics.global::Microsoft.UI.Xaml.Markup.IXamlReaderStatics.Load(String xaml)
at Microsoft.UI.Xaml.Markup.XamlReader.Load(String xaml)
at winui4.MainWindow.testloadingXAML() in C:\Users\whitewi\source\repos\winui4\winui4\MainWindow.xaml.cs:line 137
Steps to reproduce the bug
See the above code
Expected behavior
No response
Screenshots
No response
NuGet package version
No response
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
No response
Additional context
No response

@RealTommyKlein and @MikeHillberg FYI

@willwhite2 - does the markup work when placed in a .xaml file instead of using XamlReader.Load?  Also, what element is setting Style="{StaticResource style1}"?  RowDefinitions don't have a Style property and won't work with Styles in general.

Sorry, this is not a bug. I'm use WPF and assumed similar properties in  Win UI. Changed my loose XAML to use a button with a style instead of a style for the rowdefinition. I created a resource dictionary with the style for the button. read the resource dictionary, added it to the application. Then read the new loose XAML file and added it as a child to a control in the compiled XAML and the controls were rendered.
thanks


can't connect

Hi
I have an issue when click on connect from app client, after logging to my email it shows to message to authorize (only the first time) then it shows login successful,but nothing happens in the app.
running Windows 11 X64.
What i have tried :
Add an inbound rule for netbird program(allow all).
I have tried using the command line "netbird up" and this is the output :

C:\Windows\System32>netbird up -F --log-level debug
2023-12-14T11:21:03+01:00 DEBG client/internal/login.go:93: connecting to the Management service https://api.wiretrustee.com:443
2023-12-14T11:21:03+01:00 DEBG client/internal/login.go:63: connected to the Management service https://api.wiretrustee.com:443
2023-12-14T11:21:04+01:00 DEBG client/internal/login.go:72: peer registration required
2023-12-14T11:21:04+01:00 DEBG client/internal/pkce_auth.go:55: connecting to Management Service https://api.wiretrustee.com:443
2023-12-14T11:21:04+01:00 DEBG client/internal/pkce_auth.go:61: connected to the Management service https://api.wiretrustee.com:443
Please do the SSO login in your browser.
If your browser didn't open automatically, use this URL to log in:
https://login.netbird.io/authorize?audience=https%3A%2F%2Fapp.wiretrustee.com%2F&client_id=x3KvnKHEDY2j3b0n0wLq4eu8SiPDKq6o&code_challenge=jwfRSIiFj6LTWyKagVlnvN8R3cauYcFU1Wc3IVhjxhg&code_challenge_method=S256&redirect_uri=http%3A%2F%2Flocalhost%3A53000%2F&response_type=code&scope=openid+profile+email+offline_access+api+email_verified&state=508488a72510b6c5dc27b3a745527abc9338b3c50f93d334
2023-12-14T11:21:08+01:00 DEBG client/internal/login.go:93: connecting to the Management service https://api.wiretrustee.com:443
2023-12-14T11:21:08+01:00 DEBG client/internal/login.go:63: connected to the Management service https://api.wiretrustee.com:443
2023-12-14T11:21:09+01:00 DEBG client/internal/login.go:72: peer registration required
2023-12-14T11:21:09+01:00 DEBG client/internal/login.go:122: sending peer registration request to Management Service
2023-12-14T11:21:09+01:00 ERRO client/internal/login.go:126: failed registering peer rpc error: code = PermissionDenied desc = can't login,00000000-0000-0000-0000-000000000000
2023-12-14T11:21:09+01:00 INFO client/internal/connect.go:47: starting NetBird client version 0.24.4
2023-12-14T11:21:09+01:00 DEBG client/internal/connect.go:102: connecting to the Management service api.wiretrustee.com:443
2023-12-14T11:21:09+01:00 DEBG client/internal/connect.go:110: connected to the Management service api.wiretrustee.com:443
2023-12-14T11:21:09+01:00 DEBG client/internal/connect.go:121: rpc error: code = PermissionDenied desc = peer login has expired, please log in once more
2023-12-14T11:21:09+01:00 DEBG client/internal/connect.go:208: exiting client retry loop due to unrecoverable error: rpc error: code = PermissionDenied desc = peer login has expired, please log in once more
Error: rpc error: code = PermissionDenied desc = peer login has expired, please log in once more

NetBird status -d output:

Daemon status: NeedsLogin
Run UP command to log in with SSO (interactive login):
netbird up
If you are running a self-hosted version and no SSO provider has been configured in your Management Server,
you can use a setup-key:
netbird up --management-url <YOUR_MANAGEMENT_URL> --setup-key <YOUR_SETUP_KEY>
More info: https://docs.netbird.io/how-to/register-machines-using-setup-keys

thank you

Hello @omarbenzekri can you restart the agent with the following command and try again via regular powershell session:
Admin session powershell
netbird service restart

Regular session powershell
netbird up

it didn't work, still having the same issue.
what port is listening to?

Hi @omarbenzekri
I faced the same problem. I deleted the peer with login error and login again. That works for me.

Hi, i had a similar issue, login was expired and i don't know why, but Netbird icons in Desktop and system tray were deleted so to re-login i had to do it from terminal with "netbird down - netbird up", this worked but i'm without icons yet... Does anyone knows what could has been happened?

I answer to myself, it happened that netbird ui at login expiration had been uninstalled (but why???) so i had to reinstall netbird from scratch, all this in Windows 11

I have a similar connection problem. I created a reusable Setup Key for unlimited clients and a 365 day expiration, then used it to connect an OpenWrt server just fine. I then tried to connect a Ubuntu x86_64 desktop machine and got this error:
$ curl -fsSL https://pkgs.netbird.io/install.sh | sh
Installation has been finished.
$ netbird login --setup-key C566D8D5-CBC9-4339-86BB-REDACTED
error: login failed: rpc error: code = PermissionDenied desc = peer login has expired, please log in once more

I tried with sudo and got the same error. I did netbird up and used the provided URL and the web page said it worked:

However, in my Netbird dashboard it doesn't show up and the terminal says Error: waiting sso login failed with: rpc error: code = PermissionDenied desc = can't login
$ netbird status
Daemon status: NeedsLogin

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.4 LTS"

+1 for macos
❯ netbird up -k *****
Error: login failed: rpc error: code = PermissionDenied desc = peer login has expired, please log in once more
❯ netbird version
0.28.7

@kukat it seems like you've previously authenticated using user credentials. you need to either remove the peer and add it again with a setup-key or reauthenticate with the same user from before.

@mrbluecoat, the issue happened because you attempted to log in using a different user from when the peer was registered.


cropper is not a function

Love your extensive documentation - but especially all the functionality built into this cropper!
BUT - for the  life of me I cannot get it to work inside an ASP MVC application!
I've worked iwth java script for years - and typically can take such projects like this and incorporate into my web applications - but not so here...
I've tried everything from the "quick start" suggested - to importing everything from the demo project - being sure to check css and script links etc..... but I STILL can't get it to work - and always (whatever my approach) get the dreaded "cropper is not a function".
Is there any way you you possibly help me? I'd love to use this cropper if at all possible.
Thanks so much!!!
Wayne Fair
////////////////////////////
UPDATE:
I DID get it to work sort of...
But what I have discovered is that if I put any of the method calls inside another function - I still get the "cropper is not a function"
Like:
function btnClicked() {
$().cropper('getCroppedCanvas');
       $().cropper('getCroppedCanvas', {
           width: 160,
           height: 90
       });

    };

I think you don't use it correct.

Without a doubt, you are correct, Chen.
I am much to much of a hack when it comes to java script.
Most stuff works when I use it - but I think my problems with cropper have to do with understanding scope, closures - etc.
I do have it working fully now - and thank you for responding to my queries.
It's a brilliant piece of work - thanks for sharing it with the world :)

Thank you too @hwaynefair.

You could at least have shown what you did to make it work :(

@Solenark Yeah I'm having the same problem. Have you resolved it?
@hwaynefair any insight?

It's been so long since I either figured it out - or (more likely) stumbled into a solution (through trial and error) which I may have never understood...
I think this usually happens when the reference (to the javascript library) in your code is not placed correctly - in the right order, etc.
I also think that the solution may have had something to do with how I was instantiating the local instance of the cropper object.
Sometimes it helps just to start all over with the most simplified project - and try to get the implementation working....
Wish I could be more help - but I am at work.... Let me know if you don't get a break through - maybe I will have time later to go back and dig up my code....
Thanks!
wayne
  From: bobby2525<EMAIL_ADDRESS>
To: fengyuanchen/cropper<EMAIL_ADDRESS>Cc: Wayne Fair<EMAIL_ADDRESS>Sent: Wednesday, February 17, 2016 11:27 AM
Subject: Re: [cropper] cropper is not a function (#364)
@Solenark Yeah I'm having the same problem. Have you resolved it?@hwaynefair any insight?—
Reply to this email directly or view it on GitHub.

So what was the solution? I'm having the same issue. Even with a fresh page with nothing else going on but the image I want to use the plugin on, following the quickstart instructions yields the "cropper is not a function" error. Below is the page in its entirety. If anyone can spot what I'm doing wrong I'd appreciate it.
`

Photo

`

Be sure to download the correct package with npm install cropper.
Please don't use this npm install cropperjs as Cropper.js is not a jQuery plugin.

/headdesk. Yeah, that did it. Thanks.

There are apparently two versions of Cropper - one for jquery and one for non-jquery. The readme says to use bower install cropperjs but doesn't mention the alternate version. Everything worked well for me until I went to use setCropBoxData, then I kept getting not a function errors. Doing bower install cropper and tweaking the script paths got things working again.

Hi am using jquery cropper.js but still am facing the issue .... can you please help me out

.cropper is working nicley in local setup.
but when i use in live than i get to error like this .cropper is not a function error.
So any one have soluction for this error.
Please help me.

It's been so long since I used this cropper and I can't remember what the problem resolution was to get rid of that error. I'm fairly certain it had to do with versioning of JQuery. Sorry I can't be more help @meddev001

i have same issue when i am using requirejs , cropper is undefined.

For CommonJS:

var $ = require('jquery');

require('cropper/dist/cropper.common.js');
// require('cropper/dist/cropper.css'); // requires css loader if you are using webpack

For ES6 Module:

import $ from 'jquery';
import 'cropper/dist/cropper.esm.js';
// import 'cropper/dist/cropper.css'; // requires css loader if you are using webpack

For browser side:

<script src="/path/to/jquery.js"></script><!-- jQuery is required -->
<link  href="/path/to/cropper.css" rel="stylesheet">
<script src="/path/to/cropper.js"></script>


How to custom the encryption key in the "none_tee" demo?

Hi, I'm following this guide to try coco in a none-tee environment.
I guess that :

in the above-mentioned guide, the demo container image(docker.io/katadocker/ccv0-ssh) is encrypted by this key .
And the custom resource(github.com/confidential-containers/operator/config/samples/ccruntime/ssh-demo?ref=<RELEASE_VERSION>) also containers this key?

Am I understanding correctly?
And My question is :
Now I want to encrypt my image with my own key, what should I do to? Do I need to re-generate the custom resource(aka runtime class payload)? If I should ,is there any guide?

Check out this guide for more info about how to encrypt your own images.
When you use the non-tee runtime class you will use the offline_fs_kbc inside the guest. Once you encrypt your image following the above guide, you'll need to modify the initrd that the PodVM boots with to include your new decryption key.

Check out this guide for more info about how to encrypt your own images.
When you use the non-tee runtime class you will use the offline_fs_kbc inside the guest. Once you encrypt your image following the above guide, you'll need to modify the initrd that the PodVM boots with to include your new decryption key.

Hi, @fitzthum  Thanks for your reply.
I have checked this guide.
I don't know how to modify the initrd. Is there any guides?
Here is my qemu command:(I guess I need to modify the /opt/confidential-containers/share/kata-containers/kata-ubuntu-latest.image) ?
/opt/confidential-containers/bin/qemu-system-x86_64 -name sandbox-ed1ca33183178f1ffb682097128b0b64729b8e1544b3c1767236bf3d03fb48bd -uuid 7d830ef5-507b-4e42-a318-132fca446f6b -machine q35,accel=kvm,nvdimm=on -cpu host,pmu=off -qmp unix:/run/vc/vm/ed1ca33183178f1ffb682097128b0b64729b8e1544b3c1767236bf3d03fb48bd/qmp.sock,server=on,wait=off -m 2048M,slots=10,maxmem=8976M -device pci-bridge,bus=pcie.0,id=pci-bridge-0,chassis_nr=1,shpc=off,addr=2,io-reserve=4k,mem-reserve=1m,pref64-reserve=1m -device virtio-serial-pci,disable-modern=true,id=serial0 -device virtconsole,chardev=charconsole0,id=console0 -chardev socket,id=charconsole0,path=/run/vc/vm/ed1ca33183178f1ffb682097128b0b64729b8e1544b3c1767236bf3d03fb48bd/console.sock,server=on,wait=off -device nvdimm,id=nv0,memdev=mem0,unarmed=on -object memory-backend-file,id=mem0,mem-path=/opt/confidential-containers/share/kata-containers/kata-ubuntu-latest.image,size=268435456,readonly=on -device virtio-scsi-pci,id=scsi0,disable-modern=true -object rng-random,id=rng0,filename=/dev/urandom -device virtio-rng-pci,rng=rng0 -device vhost-vsock-pci,disable-modern=true,vhostfd=3,id=vsock-4252177920,guest-cid=4252177920 -chardev socket,id=char-ba71a2fa9f0cedbf,path=/run/vc/vm/ed1ca33183178f1ffb682097128b0b64729b8e1544b3c1767236bf3d03fb48bd/vhost-fs.sock -device vhost-user-fs-pci,chardev=char-ba71a2fa9f0cedbf,tag=kataShared -netdev tap,id=network-0,vhost=on,vhostfds=4,fds=5 -device driver=virtio-net-pci,netdev=network-0,mac=66:5a:84:8b:2c:28,disable-modern=true,mq=on,vectors=4 -rtc base=utc,driftfix=slew,clock=host -global kvm-pit.lost_tick_policy=discard -vga none -no-user-config -nodefaults -nographic --no-reboot -daemonize -object memory-backend-file,id=dimm1,size=2048M,mem-path=/dev/shm,share=on -numa node,memdev=dimm1 -kernel /opt/confidential-containers/share/kata-containers/vmlinux-5.19.2-96 -append tsc=reliable no_timer_check rcupdate.rcu_expedited=1 i8042.direct=1 i8042.dumbkbd=1 i8042.nopnp=1 i8042.noaux=1 noreplace-smp reboot=k cryptomgr.notests net.ifnames=0 pci=lastbus=0 root=/dev/pmem0p1 rootflags=dax,data=ordered,errors=remount-ro ro rootfstype=ext4 console=hvc0 console=hvc1 debug systemd.show_status=true systemd.log_level=debug panic=1 nr_cpus=8 systemd.unit=kata-containers.target systemd.mask=systemd-networkd.service systemd.mask=systemd-networkd.socket scsi_mod.scan=none agent.debug_console agent.debug_console_vport=1026 cc_rootfs_verity.scheme=dm-verity cc_rootfs_verity.hash=7c45fe8a04c18da806b37a2e760966980d487387aa54d177d11cae108bc75d98 agent.enable_signature_verification=false agent.config_file=/etc/agent-config.toml -pidfile /run/vc/vm/ed1ca33183178f1ffb682097128b0b64729b8e1544b3c1767236bf3d03fb48bd/pid -D /run/vc/vm/ed1ca33183178f1ffb682097128b0b64729b8e1544b3c1767236bf3d03fb48bd/qemu.log -smp 1,cores=1,threads=1,sockets=8,maxcpus=8

Hey @gxcuit did you ever figure this out? I missed your reply here but I can give you some more info if you are still working on this.

Hey @gxcuit did you ever figure this out? I missed your reply here but I can give you some more info if you are still working on this.

Hi, @fitzthum  Thanks for your reply. I am still working on this.  I would appreciate it if you give me some advice. Thanks a lot!!

Ok, currently if you are bringing your own encrypted/signed images with the non-tee runtime class (kata or kata-qemu), you will need to update the initrd to include the appropriate secrets.
This should give you a pretty good idea about how to open up one of the images and put something inside.
The path you will need to update to setup a new key is probably /etc/aa-offline_fs_kbc-keys.json.


Proxying a model that contains an event field

I'm doing some initial testing with django-pghistory.
I have tried to create a proxy class to an existing third-party object, but that fails, because that class already has an event foreign key (and most models will have it, it's an event management system).
auditlog.RoomProxyEvent.pgh_obj: (fields.E302) Reverse accessor for 'auditlog.RoomProxyEvent.pgh_obj' clashes with field name 'auditlog.RoomProxy.event'.
	HINT: Rename field 'auditlog.RoomProxy.event', or add/change a related_name argument to the definition for field 'auditlog.RoomProxyEvent.pgh_obj'.

I wonder if there is an easy solution for this. I can live with the fact that the event key is not logged.
I tried
 @pghistory.track(
         pghistory.Snapshot(), exclude=["event"])
 class RoomProxy(Room):
     class Meta:
         proxy=True

but that does not work.

Found the solution myself: this is possible by setting the https://django-pghistory.readthedocs.io/en/2.5.1/event_models.html#pgh-obj-field


unknown symbol "PyInt_AsSsize_t" (Fiddle::DLError)

$ PYCALL_DEBUG_FIND_LIBPYTHON=1 PATH=/tmp/local/bin:$PATH ruby -ve 'require "matplotlib"
ruby 2.7.0dev (2019-09-29T17:56:15Z master 54d49aeb52) [x86_64-linux]
DEBUG(find_libpython) find_libpython(nil)
DEBUG(find_libpython) investigate_python_config("python3")
DEBUG(find_libpython) libs: ["libpython3.7m.so.1.0", "libpython3.7m.so", "libpython3.7m", "libpython3.7", "libpython"]
DEBUG(find_libpython) libpaths: ["/usr/lib", "/lib", "", "/usr", "/usr/lib"]
DEBUG(find_libpython) Unable to find /usr/lib/libpython3.7m.so.1.0
DEBUG(find_libpython) Unable to find /usr/lib/libpython3.7m.so.1.0.so
DEBUG(find_libpython) dlopen("/usr/lib/x86_64-linux-gnu/libpython3.7m.so.1.0") = #<Fiddle::Handle:0x0000563b910297e8>
Traceback (most recent call last):
	2: from -e:1:in `<main>'
	1: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:81:in `require'
/tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:81:in `require': cannot load such file -- matplotlib (LoadError)
	9: from -e:1:in `<main>'
	8: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:34:in `require'
	7: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:157:in `rescue in require'
	6: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:157:in `require'
	5: from /tmp/local/lib/ruby/gems/2.7.0/gems/matplotlib-1.0.0/lib/matplotlib.rb:4:in `<top (required)>'
	4: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall.rb:62:in `import_module'
	3: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall/init.rb:16:in `const_missing'
	2: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall/init.rb:43:in `init'
	1: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:81:in `require'
/tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:81:in `require': cannot load such file -- 2.7/pycall.so (LoadError)
	11: from -e:1:in `<main>'
	10: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:34:in `require'
	 9: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:157:in `rescue in require'
	 8: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:157:in `require'
	 7: from /tmp/local/lib/ruby/gems/2.7.0/gems/matplotlib-1.0.0/lib/matplotlib.rb:4:in `<top (required)>'
	 6: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall.rb:62:in `import_module'
	 5: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall/init.rb:16:in `const_missing'
	 4: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall/init.rb:41:in `init'
	 3: from /tmp/local/lib/ruby/gems/2.7.0/gems/pycall-1.3.0.dev/lib/pycall/init.rb:45:in `rescue in init'
	 2: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:61:in `require'
	 1: from /tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:61:in `require'
/tmp/local/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:61:in `sym': unknown symbol "PyInt_AsSsize_t" (Fiddle::DLError)

It seems that PyInt_AsSsize_t exists only in Python 2: https://docs.python.org/2/c-api/int.html#c.PyInt_AsSsize_t
Python 3 provides PyLong_AsSsize_t: https://docs.python.org/3/c-api/long.html#c.PyLong_AsSsize_t

Thanks for the report.
It seems that the version checking may fail.
I tried to reproduce on my Ubuntu bionic machine, but I couldn't, unfortunately.
mrkn@mrkn-devel:~$ python3.7 -m pip install pillow
Collecting pillow
  Downloading
    100% |████████████████████████████████| 2.1MB 965kB/s
Installing collected packages: pillow
Successfully installed pillow-6.2.0
mrkn@mrkn-devel:~$ PYCALL_DEBUG_FIND_LIBPYTHON=1 irb -rpycall
irb(main):001:0> PyCall.init("python3.7")
DEBUG(find_libpython) find_libpython("python3.7")
DEBUG(find_libpython) investigate_python_config("python3.7")
DEBUG(find_libpython) libs: ["libpython3.7m.so.1.0", "libpython3.7m.so", "libpython3.7m", "libpython3.7", "libpython"]
DEBUG(find_libpython) libpaths: ["/usr/lib", "/usr/lib", "", "/usr", "/usr/lib"]
DEBUG(find_libpython) Unable to find /usr/lib/libpython3.7m.so.1.0
DEBUG(find_libpython) Unable to find /usr/lib/libpython3.7m.so.1.0.so
DEBUG(find_libpython) dlopen("/usr/lib/x86_64-linux-gnu/libpython3.7m.so.1.0") = #<Fiddle::Handle:0x0000560618af6238>
=> true
irb(main):002:0> require "matplotlib"
=> true

What information should I provide?
Or should I create a pull request?

It is great for me if you can create a pull-request by yourself.
PyInt_AsSsize_t is refered at this line, but it's referred as an optional symbol, so it should not raise an error.
What version of fiddle do you use?

I use Fiddle bundled in ruby 2.7.0dev (2019-09-29T17:56:15Z master 54d49aeb52) [x86_64-linux].

OK.
I suspect that the error occurred in fiddle_handle_sym function that is called via rb_funcall at here.  If so, rb_protect at here fails to protect the error but I have no idea the cause of that.
Could you investigate whether my guess is correct?

I've created pull request: #102


release-2.3: update test case for bugzilla 203197

related: #19634
Signed-off-by: Subbarao Meduri<EMAIL_ADDRESS>

/retest-required