magnitude words return 0

thousand
million
billion
if input as single words return zero.

sorry for the delay - I missed this somehow. will look into it

again sorry for the delay, I will be pushing a fix tomorrow. Completely refactored the whole library using ohm.js so now things will work much better

fixed in fc621fed10c9aa2047059da35472de59f48351ee (v1.2.2 on npm) - closing. thanks for the feedback


[Feature] my feature idea after using the project... (be able to modify the notification / webhook from the ui) and also (pause test ...)

General

[X] My feature does not exist in the latest version of MySpeed.
[X] I have checked that my feature has not been suggested by anyone else.

Your Idea
For the message part, be able to modify the content of the message (for example, via Discord to customize it, add other
information, variables, pictures with a small report, etc.). A small JPG with stats could also be included.
I don't know if this already exists, but if the speed drops below a certain amount for too long, send a notification.

For the pause part, add a feature on the main page where we can add other servers.
Include a feature to pause the server directly without having to open its dashboard. Also, add a rust color around it, just like it is
currently. I don't know if this is really clear, but if it's not, I can provide more details.

Hey, sorry for the delay.
For the discord part, you can already set custom messages in the dialog. You can also add a profile picture directly in the Discord webhook settings, but anything beyond that (embeds, attachments) would make the integration menu too bloated and complicated. If you want to implement your own custom logic on what should happen with the data, you would need to create a new webhook and implement the logic on your own.
I'm not sure if I understood your second part correctly, but if you are talking about adding new servers, you can already do that in the menu on the top left. However, you still need to pause them individually by clicking on them since it would cost too many resources if I also asked the pause state of every server in the list every few seconds.
I'm closing this issue for now, but feel free to submit other ideas or re-open this issue if I understood something wrong.


tsp, generate convenient method if it is overload method with non-union types

Currently we disable method generation(including protocol and convenient methods) for overload operations (the ones with @overload decortor). We need to generate convenient method if it is overload method with non-union types
Link: https://github.com/Azure/autorest.java/issues/1958#issuecomment-1562558219

Logic for whether to generate convenient is same as normal operation (that is, currently, no JSON merge path, no multiple content-type; may have no Union after https://github.com/Azure/autorest.java/issues/2160).


opt: avoid cloning Page in locate_child and make PageId::clone cheaper

the Arc clone was 5% of merkle update time.
the default clone implementation for PageId involved into_iter().collect(), when it should've been a memcpy

#431  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite


Race condition mounting configfs

On systems where loading the configfs module (modprobe configfs) automatically mounts /sys/kernel/config, a race condition can occur in the mount_configfs() function in utils.py.
As that function is called immediately after the modprode function, the os.path.ismount check can be called before configfs is finished mounting. The mount command is then run but fails because the mount is busy as it's mounted.
One possible workaround is to make a second os.path.ismount check if the mount command fails, and if it is mounted then we don't raise an exception.
if process.returncode != 0 and not os.path.ismount("/sys/kernel/config"):
raise RTSLibError("Cannot mount configfs")
I submitted a pull request #73 with this workaround. There are other ways this could be handled as well, this is just one fix that worked for me.

PR applied, closing this. Thanks for the report and the fix @colml !


clarify tapetal layer/cell terms in GO

GO terms:
tapetal cell differentiation, GO:0048657
tapetal cell fate specification, GO:0010234
Definitions refer to anther specific tapetum but there are non-anther tapetum terms in PO.
rename as anther wall tapetum cell differentiation,
GO:0048657 and anther wall tapetum cell fate specification, GO:0010234.
In a similar way, the following terms need to be renamed, tapetal layer -> anther wall tapetum
tapetal layer formation, GO:0048656
tapetal layer development, GO:0048658
tapetal layer morphogenesis, GO:0048655
Revisit all definitions to make sure they are in line.
Relevant PO Sourceforge items:
https://sourceforge.net/tracker/index.php?func=detail&aid=3298006&group\_id=76834&atid=835555
https://sourceforge.net/tracker/index.php?func=detail&aid=3609728&group\_id=76834&atid=835555
Reported by: tberardini
Original Ticket: geneontology/ontology-requests/10099

Relevant bits shown:
[Term]
id: GO:0010234
name: anther wall tapetum cell fate specification
intersection_of: GO:0001708 ! cell fate specification
intersection_of: results_in_specification_of PO:0025545 ! anther wall tapetum cell
[Term]
id: GO:0048657
name: anther wall tapetum cell differentiation
intersection_of: GO:0030154 ! cell differentiation
intersection_of: results_in_acquisition_of_features_of PO:0025545 ! anther wall tapetum cell


Date is truncated in Date picker.

Found during accessibility grading.
Tracking internal bug: https://microsoft.visualstudio.com/DefaultCollection/OS/_workitems/edit/40139451
"Users with low vision will not able to consume the content of the button if it is not visible in Text scaling mode."


Config to automatically increment to next major, minor or patch version

I understand how sbt-dynver works and how it comes up with the version string.
However IMHO it would be usefull if there would be a config which increments either the major, minor or patch version, plus ignore the distance.
For example, right now in Play's master branch we are working towards 2.9.0, also we are in the process of moving to GitHub actions and sbt-ci-release and want to publish sonatype snapshots on each push to the master branch.
Right now the version in Play's master branch looks like this:

2.8.1+1506-a4b4198b-SNAPSHOT
2.8.1+1506-a4b4198b+20220214-1249-SNAPSHOT when dirty

(2.8.1 because the 2.8.1 was tagged before we branched off a 2.8.x branch).
So people asked us already why this is 2.8.1... and not 2.9.0-SNAPSHOT.
So actually what we want to publish to the sonatype repo is:

2.9.0-a4b4198b-SNAPSHOT
2.9.0-a4b4198b+20220214-1249-SNAPSHOT when dirty

which makes it clearer these are snapshots of the upcoming 2.9.0 release. I think this is what people are used to.
So actually we would have to remove the distance and increment one of the major/minor/patch parts.
The same for the 2.8.x branch right now, it would be nice to publish 2.8.14-...-SNAPSHOT versions instead of 2.8.13+42-....-SNAPSHOT.
Actually we have a pull request already which achieves that, however I think it would be nice to have this build into sbt-dynver: https://github.com/playframework/playframework/pull/11168

One of the goals of sbt-dynver is not require build changes, just rely on the git metadata.  What you're asking requires making a change to the build from using increaseMinorVersion to increasePatchVersion (or viceversa).
What I suggest instead of tagging the start of the new version series, for instance 2.9.0-M0.

What I suggest instead of tagging the start of the new version series, for instance 2.9.0-M0.

@dwijnand Sorry, Dale, but it's a not good suggestion 😞 Because it's not only about the new version series, it's about all versions. Having count of tags 2 times more than now is too much, IMHO.

One of the goals of sbt-dynver is not require build changes, just rely on the git metadata. What you're asking requires making a change to the build from using increaseMinorVersion to increasePatchVersion (or viceversa).

Yep, but It's a very small one-time change for the versions series branch. I think that @mkurz proposal makes sense. For sbt-dynver it will be just a few helper functions that don't break current behavior and versions semantic but can be useful for someone who wants to use different semantics like in Play 😉 Let's be honest it's default version semantics for Maven repositories 🤷♂ And how Matthias wrote, Play already has the draft of implementation of this semantics and she can stay in Play, but it looks like she can be implemented in sbt-dynver and useful not only for Play.

You're right, it wouldn't be just for new version series, and I can see how once per version series is a lot less work than once per release.
I sympathise with the desire, but I want to be careful adding complexity here.  So I may be convinced that this can be added to sbt-dynver is someone wants to send a PR with the code and README changes for this.

@eed3si9n I think this issue should be re-opened, what do you think (also see last comment where Dale agrees this would make sense if implemented carefully)

Hi, I've been copy-pasting this PlayFramework VersionHelper file between my projects – thanks for that, that helper works exactly how I need it.
Ideally it should be published somewhere to reduce the amount of boilerplate that library authors have to deal with to get the desired behaviour. If adding this version helper to sbt-dynver is controversial, can this perhaps be published as a separate plugin instead? It would still be better than the status quo.

Somehow related, but not really, still linking them here, maybe there is something useful there:

https://github.com/sbt/sbt-release/issues/398
https://github.com/sbt/sbt-release/pull/397
https://github.com/sbt/sbt-release/pull/393


Problem when decreasing the size

I don't understand how should I decrease the overall size of the rating stars?

This issue has been fixed here:
https://github.com/LunarLogic/starability/issues/27#issuecomment-355590299


assignFlair doesn't work with flair_template_id

assignFlair should be replaced with selectFlair for this API call to work

❤


not able to create test user

i had logged in using github but now i want to test chat feature, I only have 1 github accout. So i tried to create test user but when i click ok, nothing happened, why. Should i add twitter auth secret that i can use twitter to create another account.

Hi @tanay-pingalkar Is there a specific error displayed on the BE logs?

Hi @tanay-pingalkar Is there a specific error displayed on the BE logs?

No that's why I am confused

sometimes you need to press it a couple of times

I think your problem comes when calling the api, check for errors in devtools as you might have set the wrong url for it.

@tanay-pingalkar did you export REACT_APP_API_BASE_URL=http://localhost:4001 ?

@tanay-pingalkar did you export REACT_APP_API_BASE_URL=http://localhost:4001 ?

Yup

I think your problem comes when calling the api, check for errors in devtools as you might have set the wrong url for it. Happened to me when writing the local development steps.

I will check

If you're using Docker, the test user won't get created, as that is production.

If you're using Docker, the test user won't get created, as that is production.

No i am not using docker

i am getting 11:16:15.792 [error] calling logger:remove_handler(default) failed: :error {:badmatch, {:error, {:not_found, :default}}} warning otherwise everything is going in good in elixer api

dont know how when i restart the whole setup, i got test user working. Thanks for the help.


Using Spring ShallowEtagFilter with compression filter

When These both filters are used, ShallowETagFilter generates Etag and compare with If-None-Match header value from request. The compression filter appends -gzip to ETag value. ShallowEtagFilter when comparing does not know, it has to append gzip, before comparing. In all cases, it will be 200 rather than being 304.

Changing the order of filters solves the problem. Is there a reason, for appending -gzip at the end?


Feature Request: Dialogs with custom content

Feature Request (hence not using the template)
This is a nearly complete solution for all my dialog needs - except (I think) for the possibility to quickly show an overlay with some custom content, e.g. a ContentView. If you could consider this for a future version, that would be awesome!

I am working on something for v7 (if I ever get around to finishing it that will allow for slightly more extensibility), but I will never make this library take a custom native view as it defeats the purpose of this library.  I also can't integrate a monster like Xamarin forms because this library is used beyond xamarin forms.

Fair enough :)
Thanks for the quick reply, and keep up the good work!


Add icon and images

Hi I would like to know how to add also an icon or an image next to the text that identify the  node.
thanks!

Hi, I'm sorry currently vue-treeselect doesn't support that. But that feature has been planned.

But...but.... the main image shows them?

In fact those are not images bug emojis...

Oh, I'm sorry. I misunderstood you. The new version of vue-treeselect released yesterday supports customizing the option label.
Please check out: https://riophae.github.io/vue-treeselect/#customizing-option-label
Hope this can help you!

Oh, I'm sorry. I misunderstood you. The new version of vue-treeselect released yesterday supports customizing the option label.
Please check out: https://riophae.github.io/vue-treeselect/#customizing-option-label
Hope this can help you!
How can I access some custom properties such as description, url... in option label?


triangle: incorrect test in some tracks

Please check if there's a test that states that a triangle with sides 2, 4, 2 is invalid. The triangle inequality states that for any triangle, the sum of the lengths of any two sides must be greater than or equal to the length of the remaining side. If this doesn't affect this track, go ahead and just close the issue.

There is no 2, 2, 4 but the very similar 1, 1, 2. The tests demand this to be of kind Illogical.

Let's wait for the discussion here to settle: https://github.com/exercism/xerlang/issues/85 then deal with this in all the tracks a week or so from now.

There was a bit of confusion with this issue. I've summarized my current understanding here: https://github.com/exercism/x-common/issues/202
I am closing this issue. If necessary I will open a new issue once we have come to an agreement about how to resolve the inconsistencies and what exact steps to take.


Crash when a PDF tab is open

OS/Platform
macOS
Installed
Compiled from source
Version
121.0.6167.85
Have you tested that this is not an upstream issue or an issue with your configuration?

[X] I have tried reproducing this issue in Chrome and it could not be reproduced there
[X] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[ ] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
Whenever I have a tab with a PDF file, the app crashes.
How to Reproduce?

Open a normal ungoogled-chromium window with multiple tabs (Gmail, Wikipedia, etc)
Open a new tab with a PDF file ( I am currently reading this book -> https://dn790008.ca.archive.org/0/items/TheElementsOfComputingSystems_201408/The Elements of Computing Systems.pdf)
Open an incognito window with any other page (I open YouTube for music)
Use the computer normally but eventually allow the computer to sleep with the PDF file tab open (As if you were reading the PDF file and decided to take a break with the PDF tab on the screen)
When I log back in, ungoogled-chromium crashes (It might not crash on the very first try, but it crashes on the 2nd or 3rd)

Actual behaviour
If I have multiple tabs open on both normal and incognito windows, and I have a tab with a PDF file and allow the computer to go to sleep (auto-lock) on the PDF tab, when I log back in ungoogled-chromium crashes.
Screenshots:

It has happened multiple times, and I can't understand why.
Expected behaviour
It shouldn't crash.
Relevant log output
No response
Additional context
No response

I couldn't do the step "I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there" because I don't understand how to do the "--user-data-dir" part.

How did you install ungoogled chromium? I suspect a contradiction here: «Compiled from source» ←→ «I don't understand how to do the "--user-data-dir" part»

I think that I followed a tutorial on how to install ungoogled chromium. I don't remember what the steps were. I am not very tech-savvy, so I have no idea how to do the third step with the "--user-data-dir" part.

@Cubik65536 Do you encounter such crash on your system?

@Cubik65536 Do you encounter such crash on your system?

No

Sorry I was in a rush, I haven’t experienced anything like this, but I will try to reproduce this later this week just following these instructions.

By the way, @k-afonso can you update to the latest version of Ungoogled Chromium just to make sure this situation still exists? And please send a more detailed information (Mac Model, macOS version) about your device.

I updated to the latest version of Ungoogle Chromium and I repeated the situation. After I logged back in, I managed to still use Chromium for about 3 minutes until it crashed again.
I took some screenshots of the crash report. I don't know if there is any sensible information in the report that I shouldn't share publicly. If you guys think that it is safe to share the full report here, I can make a .txt file and send it.

Sadly I can't duplicate this issue on my side following these steps:

Open some webpages.
Open the PDF
Open an incognito window.
Wait till the Mac sleeps.
Wake the Mac.

Could there be any significant differences between architectures? @k-afonso seems to be using arm.

Could there be any significant differences between architectures? @k-afonso seems to be using arm.

I am testing on an ARM machin.

@Cubik65536 is the computer going to sleep with the PDF page on the screen? As if you were reading and decided to take a break. Sometimes it doesn’t crash right away once you log back in. It can take 1-3 minutes of normal browser use. I can send the full crash report if you guys think that it is safe and it might be useful.

I have the PDF opened… You can send the crash report (it usually doesn’t contain any sensitive info), but I doubt if it will really help in our case (where we basically just add patches to another software).

Crash report:
Report.txt

Yeah I can say it's not very useful...
I'll try to duplicate it again and if it's still not working then I don't know what can we do in a short term...
It might even be a device specific issue, which will be hard to diagnose.

Okay, I understand.
Thank you guys for trying to help.

@PF4Public Hi, I tried reproducing the bug several times and I couldn't. I guess it's fixed in this later version.
Thank you guys for your great help!

The bug seems to be fixed in the most recent version 122.0.6261.111-1.
If anyone experiences the same (or a similar) bug, try updating to the most recent version available.


Secrets Expansion

Resolves #609
This PR implements the following:

Expansion of Secrets and Config constructs
Expansion of LambdaFunction -> Secret edges
Supresses a longstanding error message that occurred when compiling with the --verbose flag (no functional impact)

This change was verified to work with the ts-secrets and go-secrets sample apps.
ts-secrets:

go-secrets

This change has been verified with the following samples:

ts-sequelize (testing configuration of RDS proxy secrets)
ts-secrets (modified to use 2 fs secrets annotations, one of them including multiple secrets)
go-secrets (modified to use 2 config/secrets instead of one)


Buttons in tree jiggles when scrolling horizontally

Tested versions
Godot v4.4.dev7
System information
Fedora Linux 40 (KDE Plasma) on Wayland - X11 display driver, Multi-window
Issue description
Buttons in tree jiggles when scrolling horizontally
https://github.com/user-attachments/assets/7f4627bc-dc0d-4d1b-8a48-023a32606e7e
Steps to reproduce
Scroll horizontally in scene tree
Minimal reproduction project (MRP)
jiggle.zip

Same behavior on v4.4.dev7 on Windows 10 as well.
Also tested on v4.0.RC1 and it's happening there as well.


Is there any validation rule for  ?

Hi all,
I wonder, if there is any Validation rule for select boxes?
Unfortunately the Validation rules for a field bound to a drop down list (i.e.  ) does not work yet. (I double checked the field names spelling several times... )
Should this feature work already?
-I use validateTrigger.change as Trigger
-I bind the selected field as value (value.bind=...)
-Empty Option has the value ""
-Other Options are bound as model (model.bind=...)
Does anyone have a Suggestion for this issues?
Thanks in advance!
Martin

If you have a question, please ask it in gitter. If you have a bug, please use the issue template.


Image upscale pipeline

Add a new pipeline for image up-scaling

[x] Upscale Diffuser
[ ] Integrate with existing pipelines
[ ] UI View for up-scaling

Will be using the following model for development
https://huggingface.co/ssube/stable-diffusion-x4-upscaler-onnx

Upon further investigation into implementation it seems the max sample size for upscale is 128
128 -> 512
For images over 128 we are going to have to split into tiles, upscale and reassemble
512 -> [128, 128, 128, 128] -> [512, 512, 512,  512] -> 2048

there is an upscaler in onnx named SwinIR, which included some models in it like RealSR [best upscaler i ever used] it is a very small onnx model that can be found here:
https://huggingface.co/rocca/swin-ir-onnx/tree/main
it will be great if it be included as another model for ur upscaler

there is an upscaler in onnx named SwinIR, which included some models in it like RealSR [best upscaler i ever used] it is a very small onnx model that can be found here: https://huggingface.co/rocca/swin-ir-onnx/tree/main it will be great if it be included as another model for ur upscaler

Thanks for this link dude.
I have been trying for weeks to get a diffusion upscaler to work, but never got a good result
I tried this one and it worked first time, plus you don't have to tile the images, I can just give it a 512 image and it spits out a 2048
However these type of upscalers do not work like diffusion ones, e.g no steps, prompt needed, so I need to think of a good way to add this to the codebase
I will add the x2 and x4 versions along with smaller ones like Face Upscalers etc

u r welcome, thanks!


fix(functions): allow calling pdf without argument

This pull request updates the pdf function so it can be called without having to specify a view. This usage is currently documented but not supported.
I also added a few tests.

Thanks!


About the best result

" main training at 107K iterations leads to the best result (full training is 110K). "(see https://github.com/hkchengrex/XMem/blob/main/docs/TRAINING.md) Which training process is this sentence refer to, s012 or s03? The iterations of main training is 150K in s2 and 100K in s3 in your code, neither of which is 110K.

You need to add the 10K iterations for finetuning.
https://github.com/hkchengrex/XMem/blob/139a7458b84a4829cb3bc7434cea1e7383eca5f8/util/configuration.py#L73


[modify] mobile converter: thumb url

Coverage decreased (-0.003%) to 84.86% when pulling 49b3816aa8a0f12a681bcd544ea30184b2bd07ba on itowtips:mobile into ba30e623925e5de0ad1736f3d30dc82b5968d929 on shirasagi:master.


Is there a way to control the speed of sliding?

Is there a way to control the speed of sliding?

+1

Would love to be able to control speed- the default speed is pretty fast, especially if you have content for the user to read and have autoplay on.

Not yet. You can add a prop to control the speed of sliding
PRs are welcome :)

+1


CAMEL-19893: Migrate from jackson-module-jsonSchema to jackson-module-jsonSchema-jakarta

https://issues.apache.org/jira/browse/CAMEL-19893

@davsclaus is the impact on public API acceptable for a minor release?

Well this would go to 4.1.0, so it's fine. We need to add a note to the migration guide

There is also this PR
https://github.com/apache/camel/commit/3cae732be89d20e65d776ad72ef452f91654b1d2
I think this one fixes this by doing some pom.xml ant search/replace. But maybe that is no longer needed if this PR fixes this also ?

I think this one fixes this by doing some pom.xml ant search/replace. But maybe that is no longer needed if this PR fixes this also ?

Thanks, I was not aware of that one. I guess we need both.

/component-test camel-salesforce


BranchProtectionRuleEvent Installation field

Although it isn't yet documented at github, the BranchProtectionRuleEvent
includes an installation field which is crucial for letting Apps reacting to it
to authenticate their requests.
This PR adds that field the event type, along with an accessor for it.

I've just added a CLA for this project.

@googlebot rescan

Thank you, @nyarly !
Please run go generate ./... and push (not force-push) the changed files to this PR, and the tests should then pass.
Hopefully @googlebot will also see your signed CLA by then.


Fix export to PDF. Add drag and drop for JSON: inserts pages. Fix unreported issues with overwriting from JSON.

Resolves

https://github.com/cjquines/qboard/issues/39
https://github.com/cjquines/qboard/issues/38

Labels:

documentation
resolve-bug
style
enhancement
refactor

does this not resolve #44

does this not resolve #44

No, pagesJson: any[] is still an arbitrary array.


Zero Clients

Are there any more plans to certify Zero Clients for AVD?
[Enter feedback here]

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

ID: 2790f804-e26d-d830-cfd3-f856fb2f4d45
Version Independent ID: 0049bc46-c7c6-fd43-b9ca-2b8e76c3ef55
Content: Azure Virtual Desktop thin client support - Azure
Content Source: articles/virtual-desktop/user-documentation/linux-overview.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@bhetc There are plans to increase the number of partners in the future to add more clients however we don't have any additional details that we can share for the time being.  We should note that there are some Zero Client options in the thin client device list in the documentation (looking at 10Zig and Dell, for example):
https://docs.microsoft.com/en-us/azure/virtual-desktop/user-documentation/linux-overview#connect-with-your-thin-client-device

@bhetc We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Slow performing /cloudwatch/alarms/all/{region}

User with over 500 alarms in a single region is getting a timeout from the above mentioned API. This API call is used to populate the subscribe-to-alarms dialog. We need to implement paging into this. We also use a tabulator control here, so if a custom API can be created to leverage its built-in paging support, that would be great.

Today I found an expensive filter function that was processing each alarm record before returning the payload the browser. This function was adding about 200-300% more time to the REST API call. Rewriting the filtering function has reduced that time significantly.

With this change, I am able to load 1025 alarms from us-west-2 in about 10s.


@orta -> suppress internal webview zoom

fixes https://github.com/artsy/eigen/issues/392
Previously I thought I'd solved this problem, but I actually hadn't. we were trying to set the web view's scroll view's delegate to two different objects -- the scroll chief to handle scrolling and the vc to handle suppressing zoom.
I also looked at a test that I've seen failing intermittently and re-wrote the whole spec.

Off-topic: Why are ARExternalWebBrowserViewController and ARInternalMobileWebViewController not using the scrollView shortcut?

@alloy -> I think that's pretty new

I'm going to get rid of both of the custom delegates. Each view controller should just be the scroll view's delegate, and in its implementation of scrollviewdidscroll it can pass along that method to the chief it if so chooses. how does that sound?

@orta @alloy i re-wrote this without the delegates, which i think is simpler , easier to read, and makes more sense as

Cool, yeah, this feels much nicer, kudos :+1:

gonna presume travis is still broken.

no idea what's going on with travis

This PR seems to be broken. E.g. this class still says it conforms to the now removed ARScrollNavigationChiefAwareViewController  protocol https://github.com/artsy/eigen/blob/d49c9c95161376d5db8460a1b429f7cdb8f4d69b/Artsy/Classes/View Controllers/ARFairGuideContainerViewController.m#L10

Cool, yeah, this feels much nicer, kudos :+1:

Agreed :+1:


Add -patchup initialization hook

See https://github.com/onaio/superset-patchup/pull/35 which enables this hook
I'm not sure how this stuff gets versioned (again) - the approach here is simple but will break the deployment scripts for branches that don't have the hook in superset-patchup.  Probably pretty easy to not do that and just conditionally include, just let me know.

I'm not sure how this stuff gets versioned (again) - the approach here is simple but will break the deployment scripts for branches that don't have the hook in superset-patchup. Probably pretty easy to not do that and just conditionally include, just let me know.

So usually the Superset playbook defines a patchup version.  I think whoever defines a version that includes this hook is on the line to ensure nothing breaks.  :)

@moshthepitt @Wambere just pinging, rejiggered some initialization code

@gstuder-ona I think that we should note somewhere that after these changes this role will not work when the Ketchup version is <= 0.1.5

@moshthepitt yep agree, just needed an actual version to be able to do that with :-)  got it now

@moshthepitt Added note to README


add istio e2e tests

This change adds real istio test cases. The test does the following:

Install Istio.
Deploy workloads with istio proxy injected.
Happy path test: read/write works with the proxy injected, with proper pod annotations, and service entry.
Failure test: with the pod or service entry missing, we should see specific errors.

We don't rely on or test the sidecar ordering, so removing the sidecar ordering test.
Docs will be updated in follow-up PRs.

/lgtm


population statistics: allign tabs of course filters

Now this looks horribl. Tabs, such as "all", "passed", ... should start at the same left allignment


Travis-CI support (SOFTWARE-2313)

Since we're limited to 5 concurrent builds and we're kicking so many off, we should make sure that we set the opensciencegrid/osg-test travis-ci limit to 2 builds at a time.

Why the limit?  I think we should just let it go, full speed!

We'd potentially be starving other opensciencegrid Travis-CI builds and since the osg-test builds take a while to run, I'd rather have them just run a little longer.

Ok, seems reasonable.

Why didn't Travis-CI build this pull request?

Maybe because I set up the github/Travis-CI integration after submitting the PR? If it's helpful, here are the results from my fork:
https://travis-ci.org/brianhlin/osg-test

Ah great, now it's building!  I will approve when the build succeeds.


Add a css variable for Emoji height for easier customization

Hmm, shouldn't this actually be in em or % so that it's relative to text size?

Cool, I'll merge it in and play with a relative size to see if it works.


fixes or related to #10 #13 #15 #20 #22 #21

All of these changes are for android

updated documentation to reflect new 'ReactActivity' in MainActivity.java instead of 'Activity'
added documentation instructions to install react-native-fabric on non-Android Studio projects
added logException method for use instead of recordError since the latter doesn't work on android

fixes or related to #10 #13 #15 #20 #22 #21

Looks good, thanks!

I just published 0.2.0 to NPM, thanks for this PR!


runstaskell seems to not use sandbox

$ runstaskell --bootstrap rc-1.15
Writing a default package environment file to
/home/jkarni/Documents/Zalora/runstaskell/.cabal-sandbox/share/x86_64-linux-ghc-7.8.4/runstaskell-0.1/sandboxes/rc-1.15/cabal.sandbox.config
Using an existing sandbox located at
/home/jkarni/Documents/Zalora/runstaskell/.cabal-sandbox/share/x86_64-linux-ghc-7.8.4/runstaskell-0.1/sandboxes/rc-1.15/.cabal-sandbox
--2015-04-19 14:12:11--  http://www.stackage.org/snapshot/lts-1.15/cabal.config
Resolving www.stackage.org (www.stackage.org)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to www.stackage.org (www.stackage.org)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/plain]
Saving to: ‘/home/jkarni/Documents/Zalora/runstaskell/.cabal-sandbox/share/x86_64-linux-ghc-7.8.4/runstaskell-0.1/sandboxes/rc-1.15/cabal.config’

    [  <=>                                                                                ] 30.440       126KB/s   in 0,2s

2015-04-19 14:12:11 (126 KB/s) - ‘/home/jkarni/Documents/Zalora/runstaskell/.cabal-sandbox/share/x86_64-linux-ghc-7.8.4/runstaskell-0.1/sandboxes/rc-1.15/cabal.config’ saved [30440]

Resolving dependencies...
 The sandboxes/rc-1.15 folder does get created, and the cabal.config and cabal.sandbox.config files are generated, and point to the right locations.
Using cabal exec runstaskell ... results in the same error.

I tried to reproduce this but without success. I tried installing runstaskell in a sandbox and put packages that would be overwritten into my user database (~/.ghc). Then I started ./.cabal-sandbox/bin/runstaskell --bootstrap rc-1.15, but that succeeded. @jkarni: Any idea what I could do differenly to reproduce the bug?

We've ran into this on @tuleism's machine. The problem is that the packages that runstaskell wants to install into the sandbox conflict with packages in ghc's global package database (in e.g. /usr/lib).
This is not a problem specific to runstaskell but to cabal sandboxes. I don't know a solution.

This answer recommends using --force-reinstalls, which should be safe: http://stackoverflow.com/questions/21587200/how-to-get-cabal-to-ignore-the-global-package-db-when-using-a-sandbox

Some context:
If you have packages installed globally, unfortunately they will be available to runstaskell. So you will be able to use these packages although other users of runstaskell will not necessarily have those.
With @tuleism's PR it may happen that those additional packages are broken by the packages in your runstaskell sandbox (broken only when the sandbox is used, that is).

Fixed through #6.


Rename projects section in docs

It is basically a section about configuration, we should maybe remove it and put it in Getting started?
It is also incomplete on Forge's side

Hey @piotmag769, can I work on this issue if you're not already working on it?

@ATREAY Sure, assigning you!

Hey @piotmag769, I noticed in the issue description that you mentioned the Forge section on the project config page isn't complete. Would you kindly provide a reference or source from where I can find more details to fix it?

@ATREAY I was thinking about fork configuration in Scarb.toml, I think we could include it in an example and link the page about fork configuration so we don't repeat the same thing two times
https://foundry-rs.github.io/starknet-foundry/testing/fork-testing.html#configure-fork-in-the-scarbtoml

Hey @piotmag769, I've included the necessary updates. Please take a look whenever you have time, and tell me if any changes are needed.


It is seems that the pretrained detector checkpoint is not available

I have downloaded the pretrained detector checkpoint many times, However, the error shows that the archive is corrupted.

Thanks for your reply. Could you help me to figure it out when I run ./script/pretrain_detector.sh?
THCudaCheck FAIL file=/opt/conda/conda-bld/pytorch_1512387374934/work/torch/lib/THC/generic/THCTensorMathPairwise.cu line=21 error=48 : no kernel image is available for execution on the device
Traceback (most recent call last):
File "models/train_detector.py", line 209, in
rez = train_epoch(epoch)
File "models/train_detector.py", line 65, in train_epoch
tr.append(train_batch(batch))
File "models/train_detector.py", line 98, in train_batch
result = detector[b]
File "/home/sim/Desktop/neural-motifs-master/lib/object_detector.py", line 417, in getitem
outputs = nn.parallel.parallel_apply(replicas, [batch[i] for i in range(self.num_gpus)])
File "/home/sim/anaconda3/envs/torch3.0/lib/python3.6/site-packages/torch/nn/parallel/parallel_apply.py", line 67, in parallel_apply
raise output
File "/home/sim/anaconda3/envs/torch3.0/lib/python3.6/site-packages/torch/nn/parallel/parallel_apply.py", line 42, in _worker
output = module(*input, **kwargs)
File "/home/sim/anaconda3/envs/torch3.0/lib/python3.6/site-packages/torch/nn/modules/module.py", line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sim/Desktop/neural-motifs-master/lib/object_detector.py", line 297, in forward
gt_classes, gt_rels, train_anchor_inds, proposals=proposals)
File "/home/sim/Desktop/neural-motifs-master/lib/object_detector.py", line 272, in get_boxes
return fn(*args, **kwargs)
File "/home/sim/Desktop/neural-motifs-master/lib/object_detector.py", line 157, in rpn_boxes
post_nms_topn=2000 if self.training and self.mode == 'rpntrain' else 1000,
File "/home/sim/Desktop/neural-motifs-master/lib/object_detector.py", line 597, in roi_proposals
pre_nms_topn=pre_nms_topn, post_nms_topn=post_nms_topn)
File "/home/sim/Desktop/neural-motifs-master/lib/object_detector.py", line 607, in filter_roi_proposals
nms_thresh=nms_thresh,
File "/home/sim/Desktop/neural-motifs-master/lib/fpn/nms/functions/nms.py", line 24, in apply_nms
keep.append(keep_im + s)
File "/home/sim/anaconda3/envs/torch3.0/lib/python3.6/site-packages/torch/tensor.py", line 293, in add
return self.add(other)
RuntimeError: cuda runtime error (48) : no kernel image is available for execution on the device at /opt/conda/conda-bld/pytorch_1512387374934/work/torch/lib/THC/generic/THCTensorMathPairwise.cu:21

I have found the solution in the closed issue.

我在已解决的问题中找到了解决方案。

how can you solute this problem？

我在已解决的问题中找到了解决方案。

how can you solute this problem？


Fixed point additions and subtractions

This line of code
#ex(0) = -1.0+.#ex(0)
does not work (it does nothing)
While this other
#ex(0) = #ex(0)-.1.0
works as expected

To be redesigned in another version.


rnote: 0.7.1 -> 0.8.2

Description of changes
Diff: https://github.com/flxzt/rnote/compare/v0.7.1...0.8.2
Changelog: https://github.com/flxzt/rnote/releases/tag/v0.8.2

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

closes https://github.com/NixOS/nixpkgs/pull/251391

Result of nixpkgs-review pr 263494 run on x86_64-linux 1

1 package built:

rnote

is staging or staging-next the correct branch?

is staging or staging-next the correct branch?

We can merge this into master once https://github.com/NixOS/nixpkgs/pull/263535 is merged.


Heroku buildpack

Hi,
I'm owner of https://github.com/mthenw/heroku-buildpack-vertx. Are you interested in hosting it as a official buildpack? Of course some work is required to update packages to latest version. I'm also not sure if that kind of buildpack is required, maybe the one handling maven (or gradle) is enough for developers using vertx.

Hi Maciej,
Could you explain a bit about what hosting as an official buildpack would involve?

Buildpack is a set of three shell scripts (detect, compile, release). When you want to deploy application, heroku runs detect script from every buildpack. If it returns true, that means that this buildpack should take care of compilation and release. The only thing that is required for making buildpack usable is bumping versions of vert.x (https://github.com/mthenw/heroku-buildpack-vertx/blob/master/bin/compile#L8) and sometimes Java.
When I was using vert.x for my on purpose I was bumping version quite often. Now I'm not sure if anyone is using my buildpack (but based on number of forks, looks like it's useful). Making it official probably should cause that it would be more popular I could spend more time on it.
One thing to figure out is how buildpack should decide that some application is compatible with it. I was assuming that repository has server.js and mod.json. Of course it's not good solution as it doesn't support other languages (e.g. nodejs buildpack checks if there is package.json).
One more thing, when I wrote "hosting" I only meant making a repository under vert-x organization. Noting more is required.

I would be happy for this to become official. I assume this is only for vert.x 2 at the moment?
It would be good to get something working for the imminent Vert.x 3.
The vert-x organisation in GitHub is for Vert.x 2, the vert-x3 org is for Vert.x 3.

Great. I will prepare working buildpack for version 3.

I've updated 2.x buildpack to Java 1.8 and Vert.x 2.1.5. I've also prepared one for v3-milestone3 but looks like it's not working (https://github.com/mthenw/heroku-buildpack-vertx/issues/1). I will try to fix that later today or tomorrow.
BTW, you would rather there be one repo for that (one branch for v2, one for v3) or two separate repositories?

Hi Maciej - I think it would be better to have a separate repo for V3

Well, as Heroku started supporting docker images and vert.x has official docker images buildpack is no longer needed.


[capture] Add FPGA mode for AES

On the FPGA, the AES core automatically raises the SCA trigger we are using for capturing traces. However, on the chip this functionality is disabled.
This PR transmits the fpga_mode bit to the code running on the FPGA or the chip. Based on this bit, the target code either manually raises the trigger or lets the AES core handle automatically the trigger.
The target code is available in https://github.com/lowRISC/opentitan/pull/21741 and the binary
sca_ujson_fpga_cw310 was created using this PR and the command:
./bazelisk.sh build //sw/device/tests/crypto/cryptotest/firmware:firmware

Thanks @nasahlpa, lgtm, one small q please: Any chance we can avoid this case distinction? Is it the same trigger GPIO pin? Or can we just leave the SW raising its trigger while the HW controls a different faster one?

Thanks for the review! As mentioned in #21784 this is currently not possible but we could do it in the future.


Dense graphs can be a problem?

Expected behavior and actual behavior
Steps to reproduce the problem
Important Questions
Runninng Mode

[ ] Embedded, using PLOCAL access mode
[ ] Embedded, using MEMORY access mode
[+ ] Remote

Misc

[+ ] I have a distributed setup with multiple servers. How many?
[ ] I'm using the Enterprise Edition

OrientDB Version

[ ] v2.0.x - Please specify last number:
[ ] v2.1.x - Please specify last number:
[+ ] v2.2.x - Please specify last number:

Operating System

[+ ] Linux
[ ] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Java Version

[ ] 6
[ ] 7
[+ ] 8

Question?
https://www.quora.com/Is-there-any-reason-not-to-use-OrientDB
Dense graphs can be a problem?
Is this right or already solved by a new version?

Hi,
This is the key part.

vertices with millions of edges

These vertices with millions of edges are considered supernodes and they do cause problems for the reasons the reply author mentioned. However, this isn't an ODB problem. It is a general issue with all graph databases.
Scott

Hi @lucevers
Supernodes can create some problems with flat traversal (without conditions), just because of the number of nodes/edges you visit.
In v.2.2 we solved this problem with pattern matching. Consider this scenario:

A and B are supernodes with millions of edges
you want to know if there is an edge that connects A and B

If you have an index on the edge out and in properties, the following query will use it and will be extremely fast:
MATCH
{class:A, as:a, where:(condition for node A)} -EdgeClass-> {class:B, as:b, where:(condition for node B)}
RETURN a, b
Of course the index is used also if the pattern is in more complex query.
The index is used only in case of supernodes, if you have only a few incoming/outgoing edges OrientDB will use normal index-free adjacency, so you will have the best performance for each use case
Thanks
Luigi

Thanks!


Add CI script and ansible role for occm functional test

What this PR does / why we need it:
The test-infra change has been merged in
https://github.com/kubernetes/test-infra/pull/23328

The script ci-occm-e2e.sh is invoked by a Prow CI pod, with some
environemnts passed.
The primary task of the script is to provision a GCP VM and run
ansible playbook on the VM.
The ansible playbook is installing openstack devstack and set up
a k8s cluster (using k3s) and private docker registry.

This is the first step to migrate CPO CI from openlab to kubernetes
prow, no function tests are actually running yet.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

/cc @jichenjc @ramineni

/test openstack-cloud-controller-manager-e2e-test

/test openstack-cloud-controller-manager-e2e-test

/approve
I think we should merge this and use real test to enhance later on..

/approve
I think we should merge this and use real test to enhance later on..

I was hoping to see some logs for the job, but I didn't see any output in https://testgrid.k8s.io/provider-openstack-openstack-cloud-controller-manager, do you have any experience for the Prow CI setting?

I was hoping to see some logs for the job, but I didn't see any output in https://testgrid.k8s.io/provider-openstack-openstack-cloud-controller-manager, do you have any experience for the Prow CI setting?

I personally didn't pay attention before
I think you might want to check cluster-api-openstack repo, I can check capi logs very efficiently
I will also take a look soon

/approve
I think we should merge this and use real test to enhance later on..

I was hoping to see some logs for the job, but I didn't see any output in https://testgrid.k8s.io/provider-openstack-openstack-cloud-controller-manager, do you have any experience for the Prow CI setting?

@lingxiankong One question , I see you have added the job seperately under release-master-presubmits , what does that mean. Cant be directly added to cloud-provider-openstack-config.yaml itself?

@lingxiankong Does always_run shouldnt be true here? https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes/cloud-provider-openstack/release-master-presubmits.yaml#L11 ,

@lingxiankong One question , I see you have added the job seperately under release-master-presubmits , what does that mean. Cant be directly added to cloud-provider-openstack-config.yaml itself?

I was intentionally using separte files for different purposes, e.g. release-master-presubmits , release-master-postsubmits, release-v1.22-presubmits, etc.

I see the job itself is not triggered, not the dashboard issue I suppose

No, I was not saying it's dashboard issue, I'm just wondering why the job didn't run.

@lingxiankong Does always_run shouldnt be true here? https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes/cloud-provider-openstack/release-master-presubmits.yaml#L11 ,

No, the job is specifically for openstack-cloud-controller-manager, it should run only when the related files are changed or manually triggered. That's what I understand.

/test openstack-cloud-controller-manager-e2e-test

/test all

@lingxiankong Does always_run shouldnt be true here? https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes/cloud-provider-openstack/release-master-presubmits.yaml#L11 ,

No, the job is specifically for openstack-cloud-controller-manager, it should run only when the related files are changed or manually triggered. That's what I understand.

Got it. Looks like manual triggering not happening though.
One more question , why we have branch as main not master https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes/cloud-provider-openstack/release-master-presubmits.yaml#L13  . I see this has master https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes/cloud-provider-openstack/cloud-provider-openstack-config.yaml#L6

/test openstack-cloud-controller-manager-e2e-test

/test openstack-cloud-controller-manager-e2e-test

/test openstack-cloud-controller-manager-e2e-test


paper: RQs

research questions for paper

PR-Codex overview
The focus of this PR is to enhance the research method section by incorporating state-of-art transformer models for text classification in the study of classifying GitHub repositories as real or sample.
Detailed summary

Added state-of-art transformer models for text classification
Revised research questions for clarity and specificity
Expanded on the methodology for dataset preparation and labeling
Mentioned the use of Random-Forest algorithm and text transformers for classification
Removed redundant information from the section.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

migrated to Overleaf, paper will be pushed to repo after it will be approved


I2C Overhaul, Dependency Removal, and Peripheral Manager Redux

I2C Overhaul
This moves configuration of I2C controller blocks to a global scope, instead of a per-addon scope.

All functionality currently using I2C (Display, ADS1219, and Wii) has been updated to only require a controller block option, and in certain cases, a device address (Wii extension addresses are not selectable so it is exempt).
If no I2C block has been enabled and configured, I2C functionality will be disabled and affected functionality will display a message.

Dependency Removal
This also removes a dependency on the BitBangI2C library that existed in name only and wrapped pico-sdk functionality.  It has been replaced with a library developed for the project called PicoPeripherals that will handle core peripheral functionality (initialization, reads, writes, etc.).
Peripheral Manager
To round out the changes to I2C, a Peripheral Manager has been introduced.  This manager was written to leverage on-chip peripheral devices as needs arise.  In addition to I2C, SPI has been added to this manager, but no functionality currently leverages it.

Peripheral Manager handles the global configuration scope of I2C and SPI devices, and initializes them if the respective devices have been enabled.
Each device is initialized as a singleton, so there can only ever be reference to one interface per controller block.  With I2C in mind, this reduces the need to initialize devices or change options, which could disrupt functionality.

Since I2C and SPI access are now encapsulated, it would also make sense to take this opportunity to allow proper peripheral sharing across cores. Adding a recursive lock and methods to allow exclusive access for multiple read/write transactions (e.g. i2c.grab() and i2c.release()) would probably be enough.

@dogtopus I do agree, but the first iteration of this is to get the new structure in place, and make sure that it doesn't break things in the process.  Optimizations will come in a later PR, as there's some work that I would like to do to the Display functionality to clean things up.  Right now, I'm assessing options for that part, which could mean ditching the OneBitDisplay library entirely, but it's a bit too early for me to tell.  Display is the culprit for a lot of our I2C collisions, and likely the reason why things do not play nice across cores.


Add new ispc

New version of ispc, plus flag for intel syntax: https://groups.google.com/forum/#!topic/ispc-users/zPz_1ppC-dQ

Hey Matt, we're coming up on a new release of ISPC and I'd really love to get this addressed.  The new version of ISPC will probably drop in the next week or so.  Myself and Jon Kennedy (also Intel) recently plugged CE in our ISPC 2019 GDC talk, so I'd love to get both versions up.

The intel support and 1.10.0 are now live. Once the new compiler is ready let me know!

Excellent thanks Matt!  I’ll create a new issue when the next version is released!

On Apr 6, 2019, at 10:11 AM, Matt Godbolt<EMAIL_ADDRESS>wrote:
The intel support and 1.10.0 are now live. Once the new compiler is ready let me know!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Fab; I'll close this one off and await the new issue :)

Cheers!


Multiple URL's  WebMapServiceImageryProvider

Hello,
I have only found a few old posts of this question, and no solutions. So I apologize if this has been resolved already.
I have two sets of imagery data, and I'm trying to use the addImageryProvider() method, but it doesn't seem to be working.
I added one set of tiles directly to the instance of the Cesuim.Viewer imageryProvider options, and I got a blue globe from that. so it appears to be a step closer.
how can I add both sets directly to the view instance? for example:

imageryProvider: Cesium.createTileMapServiceImageryProvider({
    url: ['url1', 'url2']
    layers: ['layer1', 'places1']
    ...
)}

As of right now, I cannot get anything to load with my current code:
this.viewer = new Cesium.Viewer('cesiumContainer2',
      baseLayerPicker: false,
      geocoder: false,
      animation: false,
      fullscreenButton: false,
      vrButton: false,
      homeButton: true,
      infoBox: false,
      sceneModePicker: true,
      selectionIndicator: false,
      timeline: false,
      navigationHelpButton: true,
      navigationInstructionsInitiallyVisible: true

      // affects how multiple objects appear when on top of each other
      // orderIndependentTranslucency: false
    });

    const layers = this.viewer.scene.globe.imageryLayers;
    const imagery = new Cesium.WebMapServiceImageryProvider({
      url: 'http://reference2:15001/geowebcache/service/wms',
      layers: 'imagery',
      minimumLevel: 1,
      maximumLevel: 19,
      tileWidth: 512,
      tilesHeight: 512
    });

    layers.addImageryProvider(imagery);

    const places = new Cesium.WebMapServiceImageryProvider({
      url: 'http://reference2:15001/geowebcache/service/wms',
      layers: 'places',
      minimumLevel: 1,
      maximumLevel: 19,
      tileWidth: 512,
      tileHeight: 512
    });

    layers.addImageryProvider(places);

When I add the var imagery settings directly to the instance like this, i get a blue globe:
imageryProvider: Cesium.createTileMapServiceImageryProvider({
      url: 'http://reference2:15001/geowebcache/service/wms',
      layers: 'imagery',
      minimumLevel: 1,
      maximumLevel: 19,
      tileWidth: 512,
      tilesHeight: 512
}),
...

You might find this tutorial helpful: https://cesiumjs.org/tutorials/Imagery-Layers-Tutorial/
I don't see any problems with your code.  The only other problem I can think of is that there may be a CORS issue.  Make sure your imagery server has CORS enabled. https://enable-cors.org/
If you're still having trouble please ask questions like this on our forum: https://groups.google.com/forum/?hl=en#!forum/cesium-dev
More people see posts there so it is more likely someone will be able to help.  We use GitHub exclusively for bug tracking and planning new features.

CORS issue, thanks @hpinkos


[Drawer] Focus when open

When you click the button to toggle the left nav, the nav should receive focus so that keyboard and screen reader users are placed there.

I'm really new to material-ui, but even when I focus the menu I'm not seeing that it supports any sort of keyboard interaction. What should I be expecting to see here?

@ryanflorence The old menus are deprecated and the new menus can be accessed through keyboard. So closing this issue!

i don't understand why this issue was closed. the reason given:

the new menus can be accessed through keyboard

is not a solution to this issue. the drawer still does not receive focus when opened.


Move babel config to babelrc

Move babel config .babelrc to allow node-crc to be imported into react-native projects. For a bit more context: https://github.com/facebook/react-native/issues/4062
Since you use the files declaration already we don't need to set that up or ignore the babelrc in npmignore.

Thanks!


Add LinkedList and interface

Add Singly-Linked-List Data Structure with time-space complexity.
If implemented with a vector or binary tree, becomes a HashMap with hash function f(x) = x.

Thanks @varunjm , unsure how I missed that.

Some changes are still left to be made. Please do so when you get the time.

template definitions need to be defined in the header not in the cpp file.

Hello. Would I be able to take over making changes to the linked list since it seems this is a dead PR.

@slimchahla Yes, you can.

Closes #34


Feature Request: make compatable with loopyLoop.js

This extension is very cool, and would make spicetify's Loopy Loop (a-b repeat) extension potentially easier to use thanks to this extensions visual feedback on song segments.
Understandable if it's too much of a task tho!

Hey there, im having the same issue but then with the tool "Section marker"
https://github.com/Aimarekin/Aimarekins-Spicetify-Extensions/tree/main/section-marker
The main issue is that Spicetify-waveform removes the old progressbar with its own, which is fine.
But perhaps you could simply opacity 0 the old progress bar, and put your own bar next to it.
But keep their structure and their current/total time things.
I understand that making your extension work with other external extensions is not your n1 priority,
but would be really cool if you could take a look at this :)

Just adding my voice to the a-b repeat feature. In combination with the waveform view this would be super useful for learning songs. Maybe the markers could even snap to transients in the waveform.

I would love to make these existing extensions compatible but with the changes I have in mind for future versions it makes more sense for me to stick to the current route of replacing the entire seekbar. I am already looking into adding the same sort of functionally as section markers and loop functionality to the waveform. The next version will also have a larger RGB type waveform highlighting lows/mids/highs.


please fix, NPE @getCurrentItem

I guess you've already finished NPE in method 'getCurrentItem' in UltraViewPagerView.
but you didn't update for users.
plz update library ver for us!!!
Thanx :)

Can you provide the related issue or more detail about NPE?
And do not guess, plz verify it using source code. I will release new version if it was fixed.

it's exactly same #48 issue.
I think your team already fix it.
but current library version is not updated from ver <IP_ADDRESS> to ver <IP_ADDRESS>.
so your work is not affected by code.
please recheck codes #48 issues.
i wish update library version in gradle.
if i miss some point, let me know about it!

I have released <IP_ADDRESS> and it is available on jCenter and mavenCenter

oh... I shoud have read README in detail...
Sorry, and Thank you 👍


Can not log in keybase.

Hello, I created account on keybase.io, then to create a key I must be logged in.
I successfully logged in, but when I were specifying device name, I made typo.
So I decided to do "keybase deprovision".
Now when I trying to log in, keybase gives error:
▶ WARNING Error running Bug 3964 repairman: lksec load: no device id set, thus can't fetch server half
▶ WARNING In Bug 3964 repair: lksec load: no device id set, thus can't fetch server half
▶ ERROR eldest called on user with existing eldest KID
How to solve it?
my log id: 84911bc15eab0e802f637c1c

Thanks for the report, CC @maxtaco

My guess is that deprovisioning killed this user's last key
@oconnor663 can you look into this?
On Sun, Dec 11, 2016 at 10:16 AM Chris Ball<EMAIL_ADDRESS>wrote:

Thanks for the report, CC @maxtaco https://github.com/maxtaco
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/5074#issuecomment-266287759,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_3QCGhAxnPcWSn7LiGjxQaqVBZ7Tks5rHBPKgaJpZM4LJ7g8
.

Yes, that's what's happened. https://keybase.io/doitcreative/sigchain
@DoITCreative I'm sorry for not having a better error message for you there. (I've filed an internal issue for us.) What's happening is that there are no keys on your account at all, and because your account hasn't been "hard reset" you're getting an error when your new login attempts to add one. What you need to do is go on the website and do the reset here:

That would normally force you to redo all the social proofs you've done, which is one reason we don't do it automatically, but if your case (if I found the right account above) it looks like there's nothing to redo?

Oconnor663, thanks for review, but I do not have option "reset my keys & start from scratch", because I have not added any keys yet. And to add them I must be logged in.

Also, when I tryed curl method to add key, it gives following error:
Code: 908
Name: KEY_NEED_REVOKE
Description: You need to delete your active key before uploading a new one

But there is no option to delete any keys.

Oh no, you're totally right, if you haven't uploaded a PGP key before then you're in a state where "reset" doesn't show up, and it becomes unfixable.  I'll contact you over email.

Ok, thanks ;-)

It's working now, thanks, oconnor663!

@oconnor663 I'm in the exact same situation, I revoked my only key in an attempt to rename my device and adding a new PGP key doesn't seem to work. Unless there's now another way to fix this that I'm missing, could you manually reset my account?
keybase.io/kevin3black

@kevin3-black done, apologies for the wait.

Hey, connor. I'm in the exact same situation as these two... Unfortunately, there's no way back into the account. :\
keybase.io/oldrego
If you could help with this, I'd be thankful.

Heyya, Connor. I'm actually in the same boat as the people before me, with no access to my account. :\
keybase.io/oldrego
If you could help reset it, I'd be much obliged.

@Oldrego I'll email you on the side.

@oconnor663 I've managed to do the same thing as all the others before me. No PGP and no keys remaining. Could you help me out?
keybase.io/chrisclarke
Thanks in advance

@oconnor663 I also locked myself out while playing around and setting up my account today.
https://keybase.io/efritzsche
Can you do a reset? I would appreciate it.

@efritzsche done.

Everything is working now, thank you very much :)


Cannot compile in Raspberry(arm64)

Compiling socket2 v0.4.1
error: attributes are not yet allowed on if expressions
--> /home/pi/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.1/src/sys/unix.rs:878:5
|
878 |     #[cfg(not(any(target_os = "haiku", target_os = "openbsd")))]
|     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
error: aborting due to previous error
error: could not compile socket2.

What rustc are you using? We require at least rustc 1.46.

@lams2135 I think if you update your rustc version this should be resolved. I'm closing this for now, but if its still an issue please reopen it.

thank you @Thomasdezeeuw , I updated rustc then resolve this issue.


[SPARK-29158][SQL] Expose SerializableConfiguration for DataSource V2 developers

What changes were proposed in this pull request?
Currently the SerializableConfiguration, which makes the Hadoop configuration serializable is private. This makes it public, with a developer annotation.
Why are the changes needed?
Many data source depend on the Hadoop configuration which may have specific components on the driver. Inside of Spark's own DataSourceV2 implementations this is frequently used (Parquet, Json, Orc, etc.)
Does this PR introduce any user-facing change?
This provides a new developer API.
How was this patch tested?
No new tests are added as this only exposes a previously developed & thoroughly used + tested component.

cc @dbtsai who I was talking about this with.

Test build #110949 has started for PR 25838 at commit 76b9f96.

Merged build finished. Test PASSed.

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

I guess it's fine.

Test build #110949 has finished for PR 25838 at commit 76b9f96.

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

Merged build finished. Test PASSed.

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

+1 to this as it is. I think developer API is more appropriate. I'm not against unstable if that's what it takes to get this in, but it seems unlikely that this is actually unstable.

I've marked it as unstable, I doubt we'll change it but I don't feel strongly about that.
I've also added a test to make sure it's callable from Java.
If no one objects I'll merge this tomorrow.

Test build #111017 has started for PR 25838 at commit 9f1f561.

Merged build finished. Test PASSed.

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

Test build #111017 has finished for PR 25838 at commit 9f1f561.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SerializableConfigurationSuite

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

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

Test build #111036 has started for PR 25838 at commit 9f1f561.

Test build #111036 has finished for PR 25838 at commit 9f1f561.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SerializableConfigurationSuite

Merged build finished. Test PASSed.

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

Merged to master.


Huge FPS loss when in water

Mod loader
Forge
Minecraft version
1.20.1
Woodwalkers version
walkers-5.2
CraftedCore version
craftedcore-5.5
Modloader version
47.3.7
The latest.log file
latest.log
Issue description
I am hosting a survival world where i'm morphed as a Skeleton and  my friend is a human.
Whenever I get into a pool of water, >everyone starts to lag<. (and by that i mean we both go from a solid 120+ FPS to 1 - 3)
What's also curious is that it happens only when someone is morphed in survival and is in water. (not even swimming, just standing there does it) It also depends on where you're looking at, whenever I look up/at the sky or go back into Creative Mode, our FPS seem to be fine again.
So far I've tested only as a Creeper, Zombie and a Skeleton in an ocean and river and this has happened everytime.
Here's an example of me switching back and forth into creative and survival while in water
https://github.com/user-attachments/assets/235f75c3-0930-494e-982e-7340eb4bfb45
And here's me showing how looking up stops the lag
https://github.com/user-attachments/assets/b94af588-b573-4f82-945f-4d5ed16a4e35
Steps to reproduce
In a world, morph into a mob (for ex: Skeleton, Zombie, Creeper)
Go into Survival Mode
Jump in a large pool of water (lake, ocean, river....)
Swim, (Underwater or not, doesn't really matter) move your camera around
Other
No response

Can you test it again with only  woodwalkers & CraftedCore installed?
If it works, it's a mod incompatibility. Since I don't know what mods you installed, you'll have to manually go through the mods and sort the unrelated ones out (so only woodwalkers & the mod it doesn't work with remain).

It is indeed a mod incompatibility, after further testing it seems that this bug occurs only when "Siren Head: The Arrival" is installed, I've only tested with Siren_head v1.1.4-1.20.1, not the other mod versions

Alright, since "Siren Head: The Arrival" is closed source, I can't create a PR or something similiar to fix it.
You might report it to them, it's probably an issue in their AI logic.


tools/magit: map zt/zb motions in magit-mode

Hi!
This PR adds zt and zb motions to magit-mode since these two keybinds aren't mapped to anything by default, and zz motion already exists. I've been using the following snippet in my config for a while:
(map! :map magit-mode-map
  :nv "zt" #'evil-scroll-line-to-top
  :nv "zb" #'evil-scroll-line-to-bottom)

Thanks!


Fix MSVC linker warning for tests and samples - LNK4098: defautlib 'LIBCMTD' conflicts with use of other libs.

Fixing the MSVC specific warning when linking tests and samples (due to use of dependencies like cmocka, libcurl, etc.):
You can see the warning in CI legs like Validate Win_x86_with_samples_and_unit_test, during the cmake build step. For example:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=473026&view=logs&jobId=a129effc-2dd1-54d1-fb5a-ad7bdc0e851d&j=7031c8ed-765e-5300-045b-7ee6bc7c031d&t=d4aab572-434d-5ae2-f6a1-d1992a17c385

LINK : warning LNK4098: defaultlib 'LIBCMTD' conflicts with use of other libs; use /NODEFAULTLIB:library [D:\a\1\s\build\sdk\tests\iot\common\az_iot_common_test.vcxproj]
az_iot_common_test.vcxproj -> D:\a\1\s\build\sdk\tests\iot\common\Debug\az_iot_common_test.exe

See https://docs.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-warning-lnk4098?view=vs-2019 for more info.

Please look into trying to put single change to eng/cmake/global_compile_options.txt so it could affect all cmakes at once.

I couldn't get the different suggested alternatives to work. @antkmsft  and @vhvb1989 if you have any other alternatives for a cleaner way to skip such warnings (that reduces the amount of duplication), please let me know or feel free to give it a shot yourself. For now, I'll merge this as is, to get a warning free cmake build on Windows :)

cc @danieljurek to see if we can update the CI pipelines to catch such warnings as errors.


Introduce DI to Beef

This issue is to collaborate on making changes to Beef to introduce Dependency Injection (DI) as presented by @edjo23 within his POC. This POC also has suggestions around changing Beef to leverage Handlebars.NET which will managed separately to this change.
This is an initial pass of DI to be used for the generated components only at this stage, being the following:

XxxController - constructor change for IXxxManager.
XxxManager - constructor changes for IXxxDataSvc.
XxxDataSvc - constructor changes for IXxxData, will no longer be static, and will introduce new IXxxDataSvc.

As identified by @edjo23 there can only be a single public constructor, so there needs to be a way to configure this, so the following attributes will be added to the Entity XML:

ControllerConstructor="Public|Private|Protected" - (defaults to Public).
ManagerConstructor="Public|Private|Protected" - (defaults to Public).
DataSvcConstructor="Public|Private|Protected" - (defaults to Public).
DataConstructor="Public|Private|Protected" - (defaults to Public).

During discussions @edjo23 suggested a further option to limit the output of the generated extensions code and have the developers explicitly opt-in where required to reduce unnecessary code bloat. This will be all or nothing for the class, not per operation. The following attributes will be added to the Entity XML for those classes that support extensions:

ManagerExtensions="true|false" - defaults to false.
DataSvcsExtensions="true|false" - defaults to false.
DataExtensions="true|false" - defaults to false.

There will be a new ServiceCollectionExtensions class generated for (within the appropriate solution hierarchy) to invoke the IServiceCollection.AddTransient as follows:

Api - ./Controllers/ServiceCollectionExtensions.cs - method AddGeneratedControllerServices
Business - ./Generated/ServiceCollectionExtensions.cs - method AddGeneratedManagerServices
Business - ./DataSvcs/Generated/ServiceCollectionExtensions.cs - method AddGeneratedDataSvcsServices
Business - ./Data/Generated/ServiceCollectionExtensions.cs - method AddGeneratedDataServices

Additional changes to be considered:

The Api changes above need to occur for the gRPC components also.
The API startup.cs will need to be refactored to support DI (including solution templates).
The testing and mocking will need to be revisited to work with DI.

Please provide feedback and thoughts. A new branch will be created next week to begin work.
Thanks...

Branch issue-61-introduce-di has been created. A number of the capabilities identified above have been introduced:

Code-generation DI support for XxxController, XxxManager, XxxDataSvc (instance), XxxData, and Grpc XxxService.
Code-generation support for the constructor (per Entity XML config).
Code-generation support for the extensions (per Entity XML config).
IDatabase, IEfDb, ICosmosDb and IOData interfaces added to enable DI for these into the XxxData (removed references to the static Default as a result).
The API startup.cs has been updated to enable.

I am super happy with how quickly and easily this has come together so far; and again a huge thanks to @edjo23 for providing the guidance and POC.
Remaining work:

Add DI support to the AgentTester and the internal TestServer (where the IServiceProvider is managed).
Review Agent and underlying ServiceAgent for DI support (considerations for different HttpClient, etc.).
Review the ReferenceData implementation to determine changes for DI (firstly assess effort).
Add DI support for Functions; specifically the EventSubscriber which is intended to access an XxxManager.

Cheers...

Hi @chullybun, I have started looking at the integration tests and how services could be replaced by mocks. You may be way ahead of me but I have updated my POC with an initial concept. It is quite basic and will need some refinement but I thought I'd share.
Thanks

Hey @edjo23, thanks for the updated POC - I was able to follow along and grok the approach. In playing with it on the Beef solution proper the ReferenceData with its internal statics just blew up 😢 I think we both expected that. Anyways, this just brought forward the effort to add DI to ReferenceData, which again was not as hard as I had expected. To still enable the richness in the entities I had to add the IServiceProvider to the ExecutionContext - which means I can get access to the configured ReferenceData by its interface (new). I looked at the HttpContext and they have a similar pattern of keeping a reference to IServiceProvider so I don't feel dirty doing this. Will sync once I get the solution building again :-)

Hi @chullybun, I've made another update to the testing stuff in the POC to use ASPNET "options" (ie) for injecting parameters into service agents.
Options support the concept of named options allowing for a single type to be used and injected across multiple services. So this allows for one 'ServiceAgentOptions' type to be defined and used across all service agents. The service agents lookup the options based on a name.
I've updated the AgentTesterFactory to use a ServiceCollection to configure and build the service agent. So if you are interested in how the options work have a look at the AgentTesterFactory.Create method.
HTH

Hi @chullybun, I've made a further change to add custom configuration to the tests.

Hi @edjo23, thanks I had a look. One thing, in using the shared WebApplicationFactory I was expecting it to behave similar to the TestServer we had previously in that it could support multiple concurrent tests (using [Parallelizable(ParallelScope.Children)]). But the behaviour is the tests appear to be single-threaded (through the HttpClient) as all tests seem to complete in the same time, but not quickly. Are you aware of how to support parallelism?

Hi @edjo23, good progress. Now have all my intra-domain tests passing. Hey one thing that would be super helpful would be to get an ILogger that writes to the NUnit TestContext.Out. Is this something you could whip up :-) Cheers...

Hi @chullybun, that's awesome. I'll have a look at that logger for you.
With regards to the parallel testing. I had a look, it only appears that the first "batch" of tests have some sort of locking/waiting on a resource, subsequent tests seem to be fine. Still looking into trying to find out what the locking/waiting is to see if it could be primed in the one time set up.

Hi @chullybun, I've created the logger in my POC but it is not working as I hoped. Not sure why but TestContext.CurrentContext.Test.ID is different in the test, and in my controller where I have some logging. The result is the logs from the test framework show, but logs from the SUT don't show. Not sure if you wanted to try and log everything, or just stuff directly related to the test (I'm guessing everything).
The code is here if you wanted to try and add it to this repo to see how it goes - https://github.com/edjo23/CodeGen.POC/blob/master/Demo.Service.IntTest/NUnitTestContextLogger.cs

Hi @edjo23. There could be a delay related to the caching of reference data; however, I would expect similar behaviour when using the TestServer? I think we just continue supporting Server and Waf as is. Thanks for the Logger I will have a go at integrating and see where I end up (and yes was hoping everywhere). I sync'd up all my changes earlier - all API-related tests are passing - yay! I will start to look at DI in the functions and related event processing. Cheers...

Hi @edjo23. I have the solution running and testing end-to-end. Just the Beef.Template.Solution remaining to be fixed up; this is purely cosmetic. I have also updated all the version numbers to 4.0.1 to recognize the significance of the changes. It is in a state now for you to properly review; to make sure we are meeting all the goals we set out in the beginning. As an FYI, the Factory class has been removed from the solution as it is no longer being referenced - that is a positive sign!
There are two main objects not instantiated by DI, being Validators and Mappers. I am in two minds with these, especially the latter. For me they are somewhat internal to the Manager and Data objects. They are declared as static as these is a runtime cost to instantiate, and they could just as easily be code that would be compiled anyway - the question is whether they would ever need to be mocked? I cannot think of a scenario? Thoughts??
Cheers 🍺

Hi @chullybun, that is fantastic. I'll review it and let you know how I go. I was thinking I might also upgrade some of our components that we have and run their integration tests as well.
With regards to Validators and Mappers ... I agree with you that they are somewhat internal and I can't think of a scenario either where you would want to mock them.

Hi @edjo23, some more changes made and ready, specifically around testing (improvements/ease of use) resulting in less breaking changes hopefully. Also, as an FYI the DataContextScope has been deprecated; connection management now exclusively within DatabaseBase (much simpler) - being a connection per instance - which should be scoped per request (the Startup uses AddBeefDatabaseServices to ensure correct configuration). How is your validating going?
Also, I will probably publish the v4.0.1 packages later this week - you'll need to let folks know not to use yet :-)

Hi @chullybun, I spent some time last night reviewing the changes, got through most of it, so far nothing has jumped out. My plan was to try and get this into some of our existing components so that I can run our existing integration tests. I was going to ask if it would be possible for you to publish pre-release versions of the packages, this would make that much easier for me to do 😄.

Hi @chullybun, I worked out an easy way to deploy pre-release versions of the packages in our stack, so you can ignore that request.

Hi @chullybun, I got one of our smaller solutions cut over to use the new DI changes. It took a lot longer than I thought, mainly due to a bunch of our stuff also needing to be updated. I had planned on getting another larger solutions or two cut over, but ran out of time. Everything still looks good, although I hit a few small issues but managed to work around them.
We have an operation that is a custom, manager only, operation. The XML is defined as:
<Entity  Name="EventData" ...>
    <Operation  Name="Publish" ... ManagerCustom="true" ExcludeData="true" ExcludeIData="true" ExcludeDataSvc="true" ExcludeIDataSvc="true">
    </Operation>
</Entity>

... but an IEventDataDataSvc.cs is still created, although not the other files. The ServiceCollectionExtension has this code:
return services.AddScoped<IEventDataDataSvc, EventDataDataSvc>();

... which won't compile as the EventDataDataSvc class has not been generated. The work was easy though, just creating the missing (empty) class.
With regards to testing I couldn't workout how to use the UsingAgentTesterServer<TStartup> with a custom configuration, so had to roll my own base class with this setup (since we have a custom configuration setup):
protected IConfiguration Configuration { get; private set; }
protected AgentTesterServer<Startup> AgentTester { get; private set; }

[OneTimeSetUp]
public void CreateAgentTester()
{
    Configuration = ...;
    AgentTester = Beef.Test.NUnit.AgentTester.CreateServer<Startup>(environment: "IntegrationTest", config: Configuration);
}

We also have some tests that make multiple calls via the AgentTester in each test method, e.g. maybe a GET then a PUT, etc. Those tests were failing with this exception:
Message:
    System.ArgumentException : Value is immutable; cannot be changed once already set to a value.
  Stack Trace:
    ExecutionContext.set_CorrelationId(String value)
    AgentTestCore`1.ctor(AgentTesterBase agentTesterBase, String username, Object args)
    AgentTestBase`1.ctor(AgentTesterBase agentTesterBase, String username, Object args)
    AgentTest`3.ctor(AgentTesterBase agentTesterBase, String username, Object args)
    AgentTesterServer`1.Test[TAgent,TValue](String username, Object args)

As a workaround I added AgentTester.PrepareExecutionContext(username); between the calls to the AgentTester.
I'll try to get another larger solution updated tomorrow which will have a more comprehensive set of tests, I'll let you know how I go. I also haven't tested any function apps yet (e.g. Event Hub processors), although we largely have our own set of base libraries for those.
Cheers

Hi @edjo23, I've fixed the code-gen where XxxManager was expecting IXxxDataSvc where none generated. I could not reproduce issue with ExcludeIDataSvc; it functioned as expected for me?
I had not made any provisions within UsingAgentTesterServer to support custom configuration. Given your usage do you have a suggested implementation, or is it easier for you to have your own customised version? If you offer something similar to UsingAgentTesterServer make sure you implement IUsingAgentTesterServer as the TestSetUpAttribute looks for that and invokes PrepareExecutionContext automatically before running the test method. I also have tests with multiple calls to AgentTester, these do not require PrepareExecutionContext?
I will also publish version 4.0.1-alpha to nuget later today (as I also need for another project I am working on).
Cheers...

Thanks @chullybun, I'll take a look at IUsingAgentTesterServer and get back to you.
I'm still trying to get our core multi-tenanted API up and running. It is requiring more effort that I anticipated, there is tons of plumbing code that has been defined as statics in our start up, with lots of new'ed up managers mixed in there for testing auth tokens against tenants and users. So just trying to untangle that.
I hit another couple of smaller issues that I managed to work around, I'll share them tomorrow when I get a chance to write them up.

The issues I had were:
With this config:
<Entity Name="Tenant" Text="Tenant" ExcludeData="true" ExcludeIData="true" ExcludeDataSvc="true" ExcludeIDataSvc="true" WebApiRoutePrefix="api/v1/tenant">
    ...
</Entity>
<Entity Name="User" Text="User" Implements="IETag, IChangeLog, IUser" Collection="true" CollectionResult="true" WebApiRoutePrefix="api/v1/sec/users" AutoImplement="Database" DatabaseSchema="Sec">
    ...
</Entity>

The DataSvc ServiceCollectionExtensions code that is generated doesn't compile.
        public static IServiceCollection AddGeneratedDataSvcServices(this IServiceCollection services)
        {
                           .AddScoped<IUserDataSvc, UserDataSvc>()
                           .AddScoped<IOrgUnitDataSvc, OrgUnitDataSvc>()
                           .AddScoped<IOrgUnitOpeningHoursDataSvc, OrgUnitOpeningHoursDataSvc>()
                           .AddScoped<IUserProfileDataSvc, UserProfileDataSvc>();
        }

I think this is due to the first entity in the code gen xml having ExcludeIDataSvc="true". Swapping the entity definitions around so the the first item does not have ExcludeIDataSvc="true" works around the issue.
Another issue was with one (and only one, the last one) of the generated Data access classes injecting in a database parameter of type "Database" not "IDatabase". I didn't track down the root cause but I think the issue is to do with this line - https://github.com/Avanade/Beef/blob/f013fb9c8215d539c0cbdd848b03c3d968eb14c2/tools/Beef.CodeGen.Core/Templates/EntityData_cs.xml#L631
Setting DatabaseName="IDatabase" on the root CodeGeneration xml element worked around the issue.
Also was wondering if the GetAllTypes method generated in the ReferenceData entity class could be made static. We use this list of types to mock a reference data provider for unit testing. Previously we used a new'ed up ReferenceDataProvider to get to the property, now with the new constructor for DI this is a bit messier. For now I have just created a class in the test library that inherits the abstract ReferenceData entity class so that I can get to the property, but it would be great not to need to do that.
Cheers
https://github.com/Avanade/Beef/blob/f013fb9c8215d539c0cbdd848b03c3d968eb14c2/samples/Demo/Beef.Demo.Common/Entities/Generated/ReferenceData.cs#L37

Hi @edjo23, I made some changes that I hope will resolve your first two issues - I was unable to replicate behaviour but believe I found the offending code. GetAllTypes is now static. I have published new -alpha-20200805 nuget packages. Let me know how you get on. Cheers...

Hi @chullybun, as it turns out I can't use IUsingAgentTesterServer as it is internal. I tried to replicate what the attribute does in my base class, using my own attribute for simple metadata and a [SetUp] method. This sort of worked but I couldn't call TestSetUp.InvokeRegisteredSetUp as it is also internal. In the end I had to use reflection to invoke it.
public abstract class AgentTesterServerTestBase
{
    protected IConfiguration Configuration { get; private set; }
    protected AgentTesterServer<Startup> AgentTester { get; private set; }

    [OneTimeSetUp]
    public void CreateAgentTester()
    {
        TestSetUp.Reset();
        typeof(TestSetUp).GetMethod("InvokeRegisteredSetUp", BindingFlags.Static | BindingFlags.NonPublic).Invoke(null, null);

        Configuration = ...;
        AgentTester = Beef.Test.NUnit.AgentTester.CreateServer<Startup>(environment: "IntegrationTest", config: Configuration);
        AgentTester.RegisterBeforeRequest(BeforeRequest);
    }

    [SetUp]
    public void Setup()
    {
        var attribute = GetType().GetMethod(TestContext.CurrentContext.Test.MethodName)?.GetCustomAttribute<AgentTesterAttribute>();
        if (attribute != null)
            AgentTester.PrepareExecutionContext(TestSetUp.ConvertUsername(attribute.TestUser), AgentTesterExtensions.GetTenant(attribute.Country));
        else
            AgentTester.PrepareExecutionContext();
    }

    protected virtual void BeforeRequest(HttpRequestMessage request)
    {
        ...
    }
}

The next issue I hit was that my ExecutionContext was being "lost". I tracked that down to the constructor of the  AgentTestCore<TStartup> class.
https://github.com/Avanade/Beef/blob/d418f77abb2d3057da661756108592b7ca5ead54/tools/Beef.Test.NUnit/Tests/AgentTestCore.cs#L38
I think that line should be if (username != null || !ExecutionContext.HasCurrent) ... I'm not passing a username as I have an ExecutionContext.
Also I have some other code gen issues with Managers that I'm still looking into, seems to be related to using ManagerCustom="true".
    <Operation Name="GetDetails" OperationType="Get" WebApiRoute="details" ReturnType="Tenant" ManagerCustom="true"/>

Code is generating as:
        public Task<Tenant?> GetDetailsAsync()
        {
            return ManagerInvoker.Current.InvokeAsync(this, async () =>
            {
                ExecutionContext.Current.OperationType = OperationType.Read;
                var __result = await GetDetailsOnImplementationAsync().ConfigureAwait(false);
            });
        }

I'm having to manually add return __result; to get this to compile.
Also:
    <Operation Name="GetByUsername" OperationType="Custom" ReturnType="User?" WebApiRoute="username/{username}" WebApiMethod="HttpGet" WebApiAlternateStatus="NotFound" AutoImplement="None">
      <Parameter Name="Username" Type="string" />
    </Operation>

Is generating as:
        public Task<User?> GetByUsernameAsync(string? username)
        {
            return ManagerInvoker.Current.InvokeAsync(this, async () =>
            {
                ExecutionContext.Current.OperationType = OperationType.Unspecified;
                Cleaner.CleanUp(username);
                MultiValidator.Create()
                    .Run().ThrowOnError();

                return Cleaner.Clean(await _dataService.GetByUsernameAsync(username).ConfigureAwait(false));
                return Cleaner.Clean(__result);
            });
        }

With two return statements.
When I get some time I'll try and create an xml for you to repo the code gen issues.
Cheers 🍺

Hi @chullybun, to reproduce the issue I mentioned above with the code gen'ed DataSvc ServiceCollectionExtensions not compiling; add the following entity as the first entity in the code gen xml.
  <Entity Name="Tenant" Text="Tenant" ExcludeData="true" ExcludeIData="true" ExcludeDataSvc="true" ExcludeIDataSvc="true" WebApiRoutePrefix="api/v1/tenant">
    <Property Name="Id" Type="Guid" UniqueKey="true" DataName="TenantId"/>
    <Property Name="Name" Type="string" DataName="Name"/>

    <Operation Name="GetDetails" OperationType="Get" WebApiRoute="details" ReturnType="Tenant"  />
  </Entity>

Also if you add the xml below (which is slightly different to the previous xml) to the end of the code gen xml, it should repro the other issue I was having.
  <Entity Name="Tenant" Text="Tenant" ExcludeData="true" ExcludeIData="true" ExcludeDataSvc="true" ExcludeIDataSvc="true" WebApiRoutePrefix="api/v1/tenant">
    <Property Name="Id" Type="Guid" UniqueKey="true" DataName="TenantId"/>
    <Property Name="Name" Type="string" DataName="Name"/>

    <Operation Name="GetDetails" OperationType="Get" WebApiRoute="details" ReturnType="Tenant" ManagerCustom="true"  />
  </Entity>

Generated code:
public Task<Tenant?> GetDetailsAsync()
{
    return ManagerInvoker.Current.InvokeAsync(this, async () =>
    {
        ExecutionContext.Current.OperationType = OperationType.Read;
        var __result = await GetDetailsOnImplementationAsync().ConfigureAwait(false);
    });
}

Hi @edjo23, I have renamed IUsingAgentTesterServer to ITestSetupPrepareExecutionContext and made public, so that this can be used by anyone. I have reviewed where ExecutionContext.Reset was being invoked and adjusted where applicable; i.e. should not be being called as often; was perhaps being invoked a little too enthusiastically :-O.
The Manager code generation has been reviewed. The first scenario you presented (with no ManagerCustom="true") was working as expected, although ExcludeDataSvc and ExcludeIDataSvc were selected, as the Manager was not custom it still will execute via the interface - it was expecting the developer to have implemented (would therefore exist, just not being automatically generated by Beef). The second scenario with ManagerCustom="true" was outputting incorrectly and has been fixed.
Also, not sure if you had referenced, but Entity.ControllerConstructor code-gen attribute has been renamed to Entity.WebApiConstructor for consistency with other attributes.
I have published new -alpha.20200811 nuget packages.

Thanks for the changes @chullybun.
I'm still having issues with the ExecutionContext username being reset when calling the AgentTesterServer.Test method.
My tests look like this (where AgentTester is an AgentTesterService instance setup in the class).
 [Test]
 [AgentTester(TestUser.Admin)]
 public void GetUserSuccessful()
 {
    // Note: ExecutionContext is set up by the SetUp method in the base class (uses the NUnit [SetUp] attribute)

    var user = GetUser();

    AgentTester.Test<UserAgent, User>()
        .IgnoreChangeLog()
        .IgnoreETag()
        .ExpectStatusCode(HttpStatusCode.OK)
        .ExpectValue((t) => user, "Permissions")
        .Run((a) => a.GetAsync(user.Id));
}

The parameter docs on the AgentTesterService.Test method indicates that passing null for username should use the ExecutionContext.Current.Username, but that isn't happening. I think the changes made to the AgentTestCore constructor aren't quite right as they don't handle username being passed as null.
Cheers

Hi @edjo23 - I have attempted to replicate with no luck. I created a couple of tests to validate and they passed. I assume the [AgentTester] attribute is custom as I had renamed in Beef to [TestSetUp]. I did another NuGet publish, not that there should be anything in there specifically correcting the issue.
I am planning on doing a 4.1.0 release/publish proper tomorrow, including merge into master. So you will need to let folks know not to use until you are ready.
Cheers...

Hi @chullybun, thanks. I tried to replicate in the Beef.Demo solution, but couldn't. Went back to my stuff and re-checked, and it was working as expected! ... Not sure what happened, I swear I had the debugger on it and it was returning the username as Anonymous 😳. Sorry was wasting your time.
The other code gen changes have fixed most of my code gen issues (even one you couldn't reproduce), the only one left is in an area that I'm refactoring anyhow so I think it will go away. If not it is easily worked around.
I've got our integration tests running now and nothing more is jumping out.
Looking forward to the release 😄.

Hi @edjo23, all good - sh!t happens 💩 Also, thanks a lot for the assistance along the way, would never have got here as quickly or thoughtfully without your help 👊

Closing as committed and published as v4.1.1.


CCD not enabled for angular movement

Godot 4.03
Godot-Joilt 0.1.0-rc3
Windows 10
Intel(R) Core(TM) i7-7700K CPU
Nvidia 2080 Super
You can find the whole discussion here https://github.com/godot-jolt/godot-jolt/discussions/376.
But to sum up the issue, when an object is rotating and not moving, its linear velocity is null and CCD is not enabled.
That can lead to tunelling effect :
https://github.com/godot-jolt/godot-jolt/assets/11818237/7bfd2937-2127-40fe-9136-755273714587
https://github.com/godot-jolt/godot-jolt/assets/11818237/e59fdd7b-eb21-4d3b-b9bc-6ecd0cf80cba
Here is the MRP : Use your left mouse click to place the ball, space bar to trigger the rotation
angular-ccd-godot-jolt.zip

So, as mentioned in the discussion linked above, CCD is not something I have much experience with, but I did some more digging and found that Jolt actually mentions this exact case in its documentation about CCD, or what it calls LinearCast motion quality:

Fast rotating long objects are also to be avoided, as the LinearCast motion quality will fully rotate the object at the beginning of the time step and from that orientation perform the CastShape, there is a chance that the object misses a collision because it rotated through it.

With this image next to it:

I apologize for not catching that sooner. It's been a while since I looked through the documentation.
With that in mind, I can't really in good conscience do anything but close this as wontfix.
As for workarounds, I think your best bet is to just bump up the "Physics Steps per Second" setting. The only other solution I can think of would be to make the flippers kinematic and completely engineer the collision detection/response between the flipper and the ball yourself, but I personally wouldn't even know where to start with that.
Thank you for taking the time to create the issue/MRP though, as well as all the other issues. It's much appreciated.

Just for future references, I've managed to prevent the tunneling, keeping a decent collision behaviour and staying at 120 physics frames per second by exagerating the collision shape

Of course this is not a definitive shape, but it gives a direction to work on