java.lang.IllegalArgumentException: Cannot copy between a TensorFlowLite tensor with shape [1, 11] and a Java object with shape [1, 11, 4].

I trained a Mobilenet model using transfer learning with Keras. I then converted the keras h5 model to tflite using toco and replaced the mobilnet model and the labels with my labels. I have 11 classes in my model. But, while running, I am getting the above error. Here is the full trace.
E/AndroidRuntime(21599): FATAL EXCEPTION: AsyncTask #2 E/AndroidRuntime(21599): Process: com.example.examplerealtime, PID: 21599 E/AndroidRuntime(21599): java.lang.RuntimeException: An error occurred while executing doInBackground() E/AndroidRuntime(21599):        at android.os.AsyncTask$3.done(AsyncTask.java:354) E/AndroidRuntime(21599):        at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:383) E/AndroidRuntime(21599):        at java.util.concurrent.FutureTask.setException(FutureTask.java:252) E/AndroidRuntime(21599):        at java.util.concurrent.FutureTask.run(FutureTask.java:271) E/AndroidRuntime(21599):        at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:245) E/AndroidRuntime(21599):        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167) E/AndroidRuntime(21599):        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641) E/AndroidRuntime(21599):        at java.lang.Thread.run(Thread.java:764) E/AndroidRuntime(21599): Caused by: java.lang.IllegalArgumentException: Cannot copy between a TensorFlowLite tensor with shape [1, 11] and a Java object with shape [1, 11, 4]. E/AndroidRuntime(21599):        at org.tensorflow.lite.Tensor.throwIfShapeIsIncompatible(Tensor.java:282) E/AndroidRuntime(21599):        at org.tensorflow.lite.Tensor.throwIfDataIsIncompatible(Tensor.java:249) E/AndroidRuntime(21599):        at org.tensorflow.lite.Tensor.copyTo(Tensor.java:141) E/AndroidRuntime(21599):        at org.tensorflow.lite.NativeInterpreterWrapper.run(NativeInterpreterWrapper.java:161) E/AndroidRuntime(21599):        at org.tensorflow.lite.Interpreter.runForMultipleInputsOutputs(Interpreter.java:275) E/AndroidRuntime(21599):        at sq.flutter.tflite.TflitePlugin$RunSSDMobileNet.runTflite(TflitePlugin.java:622) E/AndroidRuntime(21599):        at sq.flutter.tflite.TflitePlugin$TfliteTask.doInBackground(TflitePlugin.java:419) E/AndroidRuntime(21599):        at sq.flutter.tflite.TflitePlugin$TfliteTask.doInBackground(TflitePlugin.java:393) E/AndroidRuntime(21599):        at android.os.AsyncTask$2.call(AsyncTask.java:333) E/AndroidRuntime(21599):        at java.util.concurrent.FutureTask.run(FutureTask.java:266)

Is your model trained with SSD? SSD should have 4 outputs but your model only have one. Rather than detectObjectOnImage can you please try runModelOnImage?
Thanks,
Qian

Hi Qian, the model is trained with Mobilenet. Here is my code
#base_model=MobileNet(weights='imagenet',include_top=False) #imports the mobilenet model and discards the last 1000 neuron layer. base_model = MobileNet(include_top=False, weights="imagenet", input_tensor=Input(shape=(224,224,3)), input_shape=(224,224,3)) x=base_model.output x=GlobalAveragePooling2D()(x) x=Dense(1024,activation='relu')(x) #we add dense layers so that the model can learn more complex functions and classify for better results. x=Dense(1024,activation='relu')(x) #dense layer 2 x=Dense(512,activation='relu')(x) #dense layer 3 preds=Dense(11,activation='softmax')(x) #final layer with softmax activation model=Model(inputs=base_model.input,outputs=preds)   I think you are saying that mobilenet is different from mobilenet ssd. Is that right?

Hi Qian, per you recommendation, I added the following code to my camera.dart
` } else if (widget.model == "SSDMobileNet") {
Tflite.runModelOnFrame(
bytesList: img.planes.map((plane) {
return plane.bytes;
}).toList(),
imageHeight: img.height,
imageWidth: img.width,
numResults: 2,
).then((recognitions) {
int endTime = new DateTime.now().millisecondsSinceEpoch;
print("Detection took ${endTime - startTime}");
            widget.setRecognitions(recognitions, img.height, img.width);

            isDetecting = false;
          });`

But, now, I am getting
java.lang.IllegalArgumentException: Cannot copy between a TensorFlowLite tensor with shape [1, 11] and a Java object with shape [1, 7, 7, 80].

For both Mobilenet and yolo, I get the following error when I replace the existing tflite and labels files with my files.

java.lang.IllegalArgumentException: Cannot convert between a TensorFlowLite tensor with type UINT8 and a Java object of type [[F (which is compatible with the TensorFlowLite type FLOAT32).

Hi @shaqian any thoughts into this? Basically, I am trying to understand how I can use my transfer learning Keras model with this framework.

Hi @aakashganga,
Can you use tfLite.getOutputTensor(0).shape() to get the shape of output tensor of your model? For image classification, the output size should be the number of classes, so if you trained 11 classes, the shape should be [1, 11] not [1, 7, 7, 80].
The IllegalArgumentException would happen when date type doesn't match. I don't know if it's happening with input tensor or output tensor. Can you share the call stack?
Thanks,
Qian

I'm archiving this thread. Feel free to reopen if you have further questions.
Thanks,
Qian


Make header compatible with the collapsible menu view

Summary
This fixes issue #192.
There currently is no max-width property set on the start or end elements for the Header.  This allows the side elements to be 100% of their content.  When these elements are super big, this can cause any text in the middle to not be shown or even push the content off to the right.  Therefore, a max-width is needed for these areas.
I used a max-width of 40% for the side elements.  This will allow for the case when they are both extremely large, there will still be 20% of the content available for the text in the middle.
One caveat to this that I was not able to find a solution for is for when the collapsible menu view is first rendering, it renders correctly, if you resize it down, it collapses correctly, but if you resize it back larger, it cannot uncollapse the elements from the dropdown.  Could not find a flex-box combination that would solve this.
Additional Details
This is how the the header works today with large side elements.

By using the max-width attribute, this is how it appears with this PR.

Is there anything we can do about the large button overflow issue?

@mjhenkes For the large button, that would be a separate issue to have the button text truncate rather than wrap at the white-space.  This is how our terra-buttons work today so the fix would need to be applied there.

Another option would be allowing the header to grow.

@mjhenkes set a min-height on the header to allow for it to grow vertically if its content is larger.

@kschuste make sure you bet this with ux. The implications deserve discussion.

Getting input from a few others in UX to make final determination.

Recap from yesterday - new requirements are to expand the height to show a full button if text wraps to a second (or more) lines, also new change is to convert the truncation of the title (center area) to always wrap instead of truncate, and height of header would grow as well to show full text.
Still to be determined -- the appropriate widths of the regions (40% etc.) - still investigating and will work out an answer.

Is this what we would expect?

OR

Should this test be updated to show height adjustment? Or is this needed anymore since we removed no wrap with normal?
https://terra-clinical-deployed-pr-203.herokuapp.com/static/#/tests/header-tests/long-title

@emilyrohrbough update the test to make it more pleasing to look at, the text wasn't wrapping at it was all one-word so was being truncated based off of word-wrap: normal rules; therefore, updated it to be break-word so even a really long word will wrap.

It looks like the left and right content is no longer being vertically centered.

updated to vertically center the left and right contents, I think it looked like it was being centered vertically before, but we never allowed the header to expand vertically until now

Closing this PR until can find a 1 size fits all solution


Update README.md

Added prefix to page location to fix issue 404.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #55 (7a55939) into main (4d707e9) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #55   +/-   ##
=======================================
  Coverage   24.80%   24.80%
=======================================
  Files          24       24
  Lines        4076     4076
=======================================
  Hits         1011     1011
  Misses       3008     3008
  Partials       57       57

@ytwang0320 Please sign the cla.


[Environments reliability] Hyper-V Configuration error: Unable to perform the requested operation

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Attempt to configure a Win 10/11 Hyper-V VM
Expected result
User is able to configure Hyper-V Win 10/11 VM. Or receives better error messaging.
Actual result
The user receives the display error: "Unable to perform the requested operation. Check the Dev Home Hyper-V Extension's log file:  for more information".
The diagnosticText simply states: “Exception of type HyperVExtension.HostGuestCommunication”
looking at the configuration code this one may likely be due to the extension not being able to start the VM. We'll need to update the error messaging since all the other operations e.g shutting down, terminating, pausing all have the same error message if something unexpected occurs.
Included System Information
No response
Included Extensions Information
No response

There is diagnostic text in the event saying, "Exception of type 'HyperVExtension.HostGuestCommunication.HResultException' was thrown.". That exception is used when processing results or progress from the guest OS. It's not clear how that message gets into the event if the error is coming from Start VM operation.

A repro that simulates this error: Create VM that asks for a lot of memory so it can't be started. Try to configure it.
This error happens when something goes wrong with Hyper-V manager or PowerShell runspace used to send command to Hyper-V manager. We'll get more data in future telemetry.
This is no nesassaraly Dev Home's bug, but we might be able to mitigate some of the issues.


x/website: anchor links don't seem to work properly anymore

I think this is a regression from when golang.org was replaced by go.dev in the past week or so.
If I go to https://go.dev/ref/spec#Order_of_evaluation on either Firefox 94.0.2 or Chrome 96.0.4664.45, the anchor link doesn't seem to be working properly, as I don't see the heading I linked to unless I scroll back up.
I imagine this might be related to the banners at the top, as they are what is blocking the view.

Here is roughly what I expected to see:

cc @dmitshur

CC @jamalc, @rsc.

Thanks. The problem is the floating header. It seems OK to me for now to just unfloat the header.


feat(SideNav): 4 Levels deep, non-leaf onClick support

…actions on non leaf node clicks
Closes #
Summary
This change adds support for menu hierarchies that are more than 2 levels deep. This change also adds support for click events on non-leaf menu nodes.
Change List (commits, features, bugs, etc)
N/A
Acceptance Test (how to verify the PR)
Create a menu hierarchy that is more than 2 levels deep such as:
const links = [
{
isEnabled: true,
icon: Dashboard,
metaData: {
label: 'Dashboards',
href: 'https://google.com',
element: 'a',
target: '_blank',
},
linkContent: 'Dashboards',
childContent: [
{
metaData: {
label: 'Link 1',
title: 'Link 1',
onClick: ()=>{alert('Link 1 click')},
element: 'button',
},
linkContent: 'Link 1',
childContent: [
{
metaData: {
label: 'Link 2',
title: 'Link 2',
onClick: ()=>{alert('Link 2 click')},
element: 'button',
},
linkContent: 'Link 2',
childContent: [
{
metaData: {
label: 'Link 3',
title: 'Link 3',
onClick: ()=>{alert('Link 3 click')},
element: 'button',
},
content: 'Link 3',
},
]
},
]
},
],
},
];
Verify that the menu hierarchy is honored by the Navigation. Verify that the onClick functions defined in non-leaf menu nodes are executed.

@ScottHale we'll also need at least one commit message to follow the commit format guidelines like
feat(sidenav): support nested clickable links

Without it the CI server won't know to automatically publish this PR. feat increases the package version by 1 minor version, fix increases the package version by 1 patch version.

After linting I decided to go a different route that doesn't violate the rules of hooks. I'll open a new PR for those changes.


Exampels

Hello. Have you tried using recognition on more complex examples, for example, on scanned passports?

Closing this issue as it was diverging from the original topic, and full natural image support is sadly being postponed most likely until v3 to get the main OCR component working properly and stable first.
Thanks for your interest in the OCR @teemiik, hopefully in the future it can fully support natural images.


Restrict input to items available in dropdown

Hi,
This is not an issue really, just to see if this feature is available. Bdw great work!
I'd like the user to input text from what is available in the dropdown. Anything else would be removed. Would this be possible please? So, say I start typing and what I type isn't available in the dropdown.. could I have the text removed from the input automatically?
Thanks,

@vlazar, I think we've received this request often enough that we need to support this by default somehow (perhaps if you link to a <select> this is what happens?).
In the meantime, we should have an advanced example for how to do it right now.

Related #16789


Detect Firefox on android

Fixes #11

@kshyju Would you care to take a look at this? I'm not sure if you received a notification for this pull request :) Kind regards, Jorn


Usability issue: selecting semaphore in the dashboard should disable the options related to thread and vice versa

@chrisgray
I have found a usability issue in the breakerbox dashboard:
Selecting semaphore in the dashboard should disable the options related to thread and vice versa.
Is this the expected use case? This can be confusing. I can fix this in case this is not expected.

I agree this would be a nice addition! As well as enforcing the user to select one of the possible execution isolation strategy options.


Show user location and custom Pin are not working in UWP

I have implemented the Xamarin.Forms.GoogleMaps for Cross Platform Project.
If I enable the  IsShowingUser = true in UWP, Then I got crash. I had following Issue details
(It works fine in Android)

Exception = {"Object reference not set to an instance of an object."}
at Xamarin.Forms.GoogleMaps.UWP.MapRenderer.LoadUserPosition(Geocoordinate userCoordinate, Boolean center) at Xamarin.Forms.GoogleMaps.UWP.MapRenderer.d__11.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Xamarin.Forms.GoogleMaps.UWP.MapRenderer.d__0.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.b__6_0(Object state) at System.Threading.WinRTSynchronizationContext.Invoker.InvokeCore()

The Custom Pin option is not working in UWP project. I have used the same code for android it works.

kindly identify and fix these issues.

Could you please tell me which version of Xamarin.Forms.GoogleMaps do you use?
I will try it if I am able to induce.

Xamarin.Forms.GoogleMaps 1.7.0

For me it works, try to use beta version 4
Custom pins for UWP is not implemented I want to implement it, but I dont know how right now.

Thanks for your suggestion, Still the issue exist.

@JKennedy24 ,

Could you reproduce (and fix) it?
(Sorry I don't have real windows phone device.)

When trying to use UserLocation I get:
Access is denied.
Your App does not have permission to access location data. Make sure you have defined ID_CAP_LOCATION in the application manifest and that on your phone, you have turned on location by checking Settings > Location.
I will try have a look into this when I can.
@amay077  I dont have windows phone either I am using UWP application on Windows 10, works quite well
Custom pushpins have been submitted as a pull request here: https://github.com/amay077/Xamarin.Forms.GoogleMaps/pull/143

I added "location" to device capability in UWP Sample app.
https://github.com/amay077/Xamarin.Forms.GoogleMaps/commit/8a2373dd3982a472d91f87f117fea5a080df96ef
This is works fine in my Win10 desktop.
Can I close this?

I close it because no longer response.


Instructions for running tutorials on k8s missing

at https://egeria.odpi.org/open-metadata-resources/open-metadata-labs/ there is a link to running the tutorials on k8s. However the link points to a more general page which addresses running on docker-compose, but not kubernetes.
needs correcting ...

Made k8s/compose distinct, and made improvements/corrections.


Add validation to selected columns

Add validation to the columns selected. For example, the label column for value prediction should be a number.

See error experience improvement epic. https://github.com/dotnet/machinelearning-modelbuilder/issues/1378


i am getting below error when i run this command : flutter pub run klutter:producer install=platform

i am following tutorial by klutter ( https://buijs.dev/klutter-2/)
KLUTTER: Received invalid command.
 Did you follow the tutorial literally or did you use newer dependency versions?
Which command gave this error?

i followed each and every step of the tutorial , every step successfully ran but struck at step 3 could you please update the tutorial or suggest me some blogs or tutorials would be a great help i am trying this since 2 days
thank you

For now you can check the cookbook examples. I am refactoring the project and will update the tutorials when done, but because this is just a side project it takes a while...

thank you @buijs-dev  , if you need any help from my side, i am ready to help you , this project will  become big for sure

@vishwaprojects 2024.1.2 is released. Could you have a look at the cookbook examples and run them locally or try to create a new project using the Intellij/Android Studio plugin?

definitely been waiting for your release @buijs-dev

i cloned the hello_world i tried to run intiallly it prompted me to add the below thing in gradle
lintOptions {                                                                                │
│     checkReleaseBuilds false                                                                                │
│   }
so i added this in app level build gradle then i tried to run it i got the below error
You are applying Flutter's app_plugin_loader Gradle plugin imperatively using the apply script method, which is deprecated and will be removed in a future release. Migrate to applying Gradle plugins with the declarative plugins block: https://flutter.dev/go/flutter-gradle-plugin-apply
FAILURE: Build failed with an exception.

Where:
Build file 'C:\Users\vishwateja\Desktop\flutter_vs\klutter-cookbook\hello_world\example\android\app\build.gradle' line: 32

What went wrong:
Could not compile build file 'C:\Users\vishwateja\Desktop\flutter_vs\klutter-cookbook\hello_world\example\android\app\build.gradle'.

startup failed:
build file 'C:\Users\vishwateja\Desktop\flutter_vs\klutter-cookbook\hello_world\example\android\app\build.gradle': 32: Unexpected character: '?' @ line 32, column 161.
?
^

1 error

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1s
Running Gradle task 'assembleDebug'...                           2,665ms
Error: Gradle task assembleDebug failed with exit code 1

./gradlew klutterGetKradle
this command is giving the below error

What went wrong:
A problem occurred evaluating root project 'hello_world'.

Failed to apply plugin 'com.android.internal.library'.
Android Gradle plugin requires Java 17 to run. You are currently using Java 11.
Your current JDK is located in C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot
You can try some of the following options:
- changing the IDE settings.
- changing the JAVA_HOME environment variable.
- changing org.gradle.java.home in gradle.properties.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

solutions tried (from stack overflow )
changed my java home var into java 17
changed to java 17 in my build tools
no use same error

C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot

If you're still seeing the same error it's likely it's still relying on JDK 11. Have you tried removing it?


Add support for version flag to audittool

How do you know which version of audittool you're running?  Right now on Linux you have to run the following to know which version you're running: update-alternatives --display audittool.sh
On Macs it's not all that clear how to determine the version you're running.
For the next release of audittool it would be great to add a --version flag to make it easy to determine the version you're running.

Not only display, but log.

https://itaffinity.wordpress.com/2014/02/25/maven-create-a-simple-build-number/

Clsed in PR #153


Fix success endpoint for @once DAGs

@mistercrunch
This ended up being a better solution than the discussed one since I use the dates variable below in other ways as well.

fyi @artwr

Muy bien.


Introduce "purge-tables" subcommand

Includes also some cleanup and a small refactoring.

This will need a bit of rebase (mainly in the unit tests) once #24 is merged.

Works nicely, just please adapt the documentation.

Works nicely, just please adapt the documentation.

done


hunspell: support empty dictionaries, adapt to the hunspell/C++ repo changes

Pater, would you mark the milestone these changes go into? If they're not going to be backported to 9x, then only main (10.0.0) would be the right one to select.

@dweiss I've scheduled them for 10 for now, thanks! Actually, if there happens to be a new major release from the 9x branch, I'd like to backport all recent Hunspell changes there. Do you have any concerns about that?

No, absolutely - I think these changes are great and everyone on 9x would benefit from them. We should probably add a note to migration.txt but otherwise I'm more than fine for everything to land on 9x as well!

What's migration.txt? I see no such file in the repository :(

Sorry - typed from memory, here it is:
https://github.com/apache/lucene/blob/main/lucene/MIGRATE.md

Aha! What do you think should be added there? I'd expect the APIs to stay the same, maybe with more of them available in 10.

Sorry if I lost track among all these patches - I thought there would be user facing changes as well. If there are none - no need to add anything.

There are some, but they're just API additions which should be backward-compatible.


Incorrect wording on <IP_ADDRESS>.c diagnostic

From j1939-84 created by jacobbosch-cummins: Equipment-and-Tool-Institute/j1939-84#1160
The tool correctly diagnoses a failure on <IP_ADDRESS>.c (MIL was on due to an active emissions related fault), but the wording for the diagnostic is inverted.  The description should indicate the failure is caused by the MIL being on (or flashing).  Diagnostic is correctly described in the J1939-84 document.
Start Test 7.2 - DM23: Emission related previously active DTCs
15:31:59.5799 Destination Specific DM23 Request to Engine #1 (0)
15:31:59.5809 18EA00F9 [3] B5 FD 00 (TX)
15:31:59.5909 18FDB500 [8] 43 FF B9 04 04 01 FF FF
DM23 from Engine #1 (0): MIL: on, RSL: off, AWL: off, PL: not supported
DTC 1209:4 - Engine Exhaust Pressure 1, Voltage Below Normal, Or Shorted To Low Source - 1 times
FAIL: <IP_ADDRESS>.b - OBD ECU Engine #1 (0) reported a different DTCs from the DM12 DTCs
FAIL: <IP_ADDRESS>.c - OBD ECU Engine #1 (0) reported MIL off and not flashing

Cross check artifact text then tag as bug.
43 in byte one is Mil On and not flashing.


Client Credentials Configured with No Scopes

Repro:

Register a client credentials client with no allowed scopes
Request a token
Client Authentication is Successful but a "Token Issued Failure" event is generated with an "invalid_scope" error

Should client credential clients be allowed to get access tokens if they do not have any allowed scopes?
I have some APIs that just need a valid access token and don't care about the aud or scope claims. It's easy enough to work around by creating and assigning a dummy scope to the client for these scenarios.

A scope is mandatory in our implementation.


Keyboard layouts setup

Hi there! While checking the default config files I've stumbled upon /usr/share/sway/scripts/setkeyboard.sh, a script that is supposed to grab xkb settings from X11 and apply them to Sway, for compatibility purposes as far as I understand. The idea is neat, especially for newbies in Wayland, though I have some suggestions and I think this is a good topic for discussion.

The script doesn't parse xkb_options which is used mainly to setup a hotkey to switch layouts, without it the user would have to dive into sway configs anyway.
That said, I think this behavior can be very confusing and non-educating for some users, for example users may assume that Sway is compatible with XKB, and, therefore, with all the tools made for XKB. A better solution might be to add documentation (or a link to ArchWiki) to educate users on how to setup keyboard layout through ~/.config/sway/config.d/10-layout.conf, for example

For documentation purposes, I will leave my own config for layouts which I placed at ~/.config/sway/config.d/10-layout.conf. It adds two layouts, Caps Lock is the shortcut to switch between them (to use Caps, you have to press Shift+Caps Lock). Caps LED is used as an additional indicator for a layout.
input * {
    xkb_layout us,ru
    xkb_options grp:caps_toggle,grp_led:caps
    xkb_capslock disable
}

i really like this capslock-setting and am wondering if we shouldn't add that to our default. wdyt?
isn't both possible?

This might be a counter-intuitive experience for many users, and it only makes sense to someone who uses 2 layouts or more (in case of 3+ I don't think the Caps LED setting will make a lot of sense), but yeah, it would be cool to introduce this feature to the multilingual users. But for that we have to make a separate UI for configuring layouts and stop relying on X11. Using environmental variables such as XKB_DEFAULT_LAYOUT, XKB_DEFAULT_VARIANT and XKB_DEFAULT_OPTIONS might be an option, but it has to be set before sway launches, doesn't look good to me.

I brought up this issue exactly because I had problems with it, so yeah, isn't possible unless we either check if X11 config isn't default or introduce a special file in ~/.config to set up layouts and will check it's existence, and do nothing in case it exists.

The problem I had: I didn't use X for anything for a long time, so when I transferred my keyboard settings from my old config and reloaded Sway, I was surprised that it didn't work. Apparently because there are no layout settings for X11 (on my machine), the script pulled up defaults from X11, which is US layout only, and on reload all my changes were dead. I had to delete the script from my machine so I could setup keyboard within sway.
It's a good question if we shall prioritize the "idiomatic" way to do this (via Sway config) or the "friction-less experience" (the X11 compatibility), but it may be confusing for someone who already tried Sway and understood it cannot be controlled via XKB. It is unlikely that Calamares will gain support for setting up layouts in any Wayland compositor, so it's up to you.
P.S Honestly I think that someone who decided to try a distribution with Window Manager should be teached on how to set it up, after all many Manjaro users end up trying Arch, where no such conveniences exist, it will be nice if they will be ready for this.

Yeah, I'm a bit out of ideas here. We had a lot of people complaining about a lot of settings from calamares having no effect (https://github.com/Manjaro-Sway/manjaro-sway/issues/36), but all your arguments basically apply to me too, but didn't come into effect (I have some per keyboard configurations that take precedence). One "middle ground" could be, that we add a file check for ~/.config/sway/config.d/10-layout.conf and if its given, we don't run the keyboard scipt. Otherwise I'd be in favor of removing it again or adding it on the live image only.

@FallenChromium would running this on the live-iso be fine with you? if yes, I think the PRs I opened should cover most bases.

Yep, running it on live-cd sounds good to me! Though I'd clarify the documentation on setting the keyboard.
First, it looks like a necessary step that you have to apply the same way it's presented. So I'd paraphrase it like 'If you use any keyboard layout other than US, add your keyboard settings, for example 'something-something'. It will save people from accidentally making their keyboard German :D
For 2 and more layouts it will be nice to add a link to ArchWiki right to the point about setting keyboard layout on first boot, so people can get more knowledge on how to set it up. Actually, it might be better to rewrite the example from echo to "paste this to the file", it's worse than a one-liner in terms of convenience, but it's more understandable.
Btw, is "swaymsg" part supposed to be in echo "swaymsg input type:keyboard xkb_layout de" > ~/.config/sway/config.d/01-keyboard.conf? Looks like a typo to me.
With all that said, thanks for ironing out one more pain point from Manjaro Sway! I love using it so far, it's a way more polished experience than I had on bare Sway with bare Arch, I was too lazy to set it up properly tbh 🤣

Btw, is "swaymsg" part supposed to be in echo "swaymsg input type:keyboard xkb_layout de" > ~/.config/sway/config.d/01-keyboard.conf?

oh. copy pasta.


cla-bot does not seem to work with Github Merge queue feature + required action

It looks like if you have the

merge queue feature enabled for your repo
cla-bot configured
add the verification/cla-signed action as required

Then once a PR is placed in the merge queue the verification/cla-signed action that is required never runs/responds. As a result the PR hangs in the merge queue until we reach the maximum time for an action to respond. At that point then PR gets removed from the merge queue and does not get merged.
Not sure if there is a workaround here. The verification/cla-signed "action" is part of the cla-bot and I am not sure I can "wrap it" in another action that calls it and skips when in a merge queue. Any other ideas for a work around?

Hi @therapon , sorry for the very late reply; as you may have already noticed, the project is not active, and it's probably going to be archived soon (the github repository will stay, but in read-only state); we used to run the cla-bot at FINOS, but we've switched to https://easycla.lfx.linuxfoundation.org a few years back; for this reason, we've discontinued our support to the project.
That said, if you are interested to lead this project, I'd be happy to keep it running and jump on a quick call to explain you more what this entitles.
cc @ColinEberhardt , who is the initial contributor of the codebase and the current lead maintainer.


Does not show users

I wanted to add a user to my lock, when I went to do so, no users were listed.  I attempted to add the new user anyway, it worked (for the new user anymway) I believe over write mode was on, so it deleted all my existing (and working) codes.  What's odd, is within a short period of time, all my old codes worked and the new code stopped working.
I have since attempted to re-add all the new codes picking a unique slot for each user code adding as well the new user, the users themselves still do not show up when in the app.  Overly just want it to work, I want to be able to delete a user after creation and I want to add them remotely if possible.

Running into the same issue. Nothing is being populated in the "installed" part of the app (e.g., locks, users). I'm running iOS with SmartThings Classic app, a SmartThings V3 Hub v3, and a Kwikset 914 z-wave.

Same problem here.  I just followed the directions to install the SmartApp and Device Handler.  Added a lock and users.  When I go back into the SmartApp nothing shows up under installed.  I also can no longer see status or control my Schlage Connect lock via the new SmartThings app.  It seems to have broken connectivity there.


How to connect the battery to “ESP32 S3 Sense”? Thank you!

How to connect the battery to “ESP32 S3 Sense”? Thank you!

See this image:

for detail info, refer to : [https://wiki.seeedstudio.com/xiao_esp32s3_getting_started/](this link)

请看这张图片：
有关详细信息，请参阅：https://wiki.seeedstudio.com/xiao_esp32s3_getting_started/

After connecting the battery in this way, how to charge the battery?


Support for distinct aggregations

This is part 2 of making TPC-H 16 work.
Currently, our aggregation code does not support aggregate expressions that have isDistinct set to true. This PR intends to add support for that by doing two things:

Serializing an is_distinct field for aggregate expressions and implementing a set-based tracker in C++ that updates rows according to the aggregate expression if is_distinct is false or if the given value was not already seen.
In Scala, checking if an aggregate operation has any aggregate expressions with isDistinct = true. If this is the case, then we need to perform a global sort before the partial aggregation. This is to make sure that the same aggregate values are not distributed across different partitions and are not counted more than once.

See https://github.com/mc2-project/opaque/pull/163


HH : MM : AM/PM time select, in 3 select boxes?

I can't believe there is no information about this in google at all, let alone just in simpleform. I get why the default for time_select is to output in military time (most of the world uses this). But, pass in the ampm option and it is formatted like: HH PM : MM, with the AM/PM included in the first select box.
This bewilders me as to why someone would program it like that. I've never seen this, ever, at least not in the USA. Is this a normal design pattern in other countries? It certainly is not in the USA.
I just want my time select to be like HH (blank padded, like 3 or 12, not 03) : MM : AM/PM - in 3 separate select boxes.

But I rest my case as to the rails core - back to simpleform: Is there a way to output a time select in that format?

I was able to accomplish this manually (without simpleform) with the following code (if the form object is 'derp'):

	<select class="hour form-control" name="derp[hour]">
		<option value="1">1</option>
		<option value="2">2</option>
		<option value="3">3</option>
		<option value="4">4</option>
		<option value="5">5</option>
		<option value="6">6</option>
		<option value="7">7</option>
		<option value="8">8</option>
		<option value="9">9</option>
		<option value="10">10</option>
		<option value="11">11</option>
		<option value="12">12</option>
	</select>
	:
	<select class="minute form-control" name="derp[minute]" >
		<option value="00">00</option>
		<option value="15">15</option>
		<option value="30">30</option>
		<option value="45">45</option>
	</select>
	<select class="meridiem form-control" name="derp[meridiem]" >
		<option value="AM">AM</option>
		<option value="PM">PM</option>
	</select>

Then in the derp model:
attr_accessor :hour, :minute, :meridiem

after_validation :parse_time

def parse_time
	if hour and minute and meridiem
		self.time = DateTime.parse("#{hour}:#{minute}#{meridiem}")
	end
end

I have to make the values stick on an edit still.

Thanks for posting your solution and saving me some time.  It's absolutely crazy the way ampm: true behaves.  The time input should be fully customizable in Rails, just like the date is.


Dependency issue, possibly because ansible-galaxy is downloading v1.1.0?

I installed this role via a line in a requirements.yml:
- src: Stouts.wale

which appears to have installed v1.1.0:
ansible-galaxy install -r requirements.yml
....
- downloading role 'wale', owned by Stouts
- downloading role from https://github.com/Stouts/Stouts.wale/archive/1.1.0.tar.gz
- extracting Stouts.wale to .galaxy/Stouts.wale
- Stouts.wale was installed successfully

Now when I run the role on Ubuntu Trusty I get the following error:
TASK [Stouts.wale : Check backup-list] *****************************************
fatal: [vsdemo1]: FAILED! => {"changed": false, "cmd": ["/etc/wale/wale.sh", "backup-list"], "delta": "0:00:00.065290", "end": "2016-04-21 09:09:44.950490", "failed": true, "rc": 1, "start": "2016-04-21 09:09:44.885200", "stderr": "Traceback (most recent call last):\n  File \"/usr/local/bin/wal-e\", line 5, in <module>\n    from pkg_resources import load_entry_point\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 2749, in <module>\n    working_set = WorkingSet._build_master()\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 446, in _build_master\n    return cls._build_from_requirements(__requires__)\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 459, in _build_from_requirements\n    dists = ws.resolve(reqs, Environment())\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 628, in resolve\n    raise DistributionNotFound(req)\npkg_resources.DistributionNotFound: requests>=2.8.1,!=2.9.0", "stdout": "", "stdout_lines": [], "warnings": []}

formatted error:
Traceback (most recent call last):
  File "/usr/local/bin/wal-e", line 5, in <module>
    from pkg_resources import load_entry_point
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2749, in <module>
    working_set = WorkingSet._build_master()
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 446, in _build_master
    return cls._build_from_requirements(__requires__)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 459, in _build_from_requirements
    dists = ws.resolve(reqs, Environment())
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 628, in resolve
    raise DistributionNotFound(req)
pkg_resources.DistributionNotFound: requests>=2.8.1,!=2.9.0"

So the error I believe may be the same as #11 however it appears this role is @ v1.1.3.  Does the Ansible galaxy index need to be updated? https://galaxy.ansible.com/Stouts/wale/

I just changed the role dependency to point to this github repo and re-installed v1.1.3 and re-ran the role.  Still appears to fail with the same error.  I do see dependencies installed but not request:
TASK [Stouts.wale : Install requirements - pt. 1] ******************************
ok: [vsdemo1] => (item=[u'cron', u'libffi-dev', u'lzop', u'pv', u'python-dev', u'python-lxml', u'python-pip'])

TASK [Stouts.wale : Install requirements - pt. 2] ******************************
ok: [vsdemo1] => (item=six)
ok: [vsdemo1] => (item=wal-e)

TASK [Stouts.wale : Install requirements - pt. 2] ******************************
skipping: [vsdemo1] => (item=six)
skipping: [vsdemo1] => (item=wal-e)

....

TASK [Stouts.wale : Check backup-list] *****************************************
fatal: [vsdemo1]: FAILED! => {"changed": false, "cmd": ["/etc/wale/wale.sh", "backup-list"], "delta": "0:00:00.062012", "end": "2016-04-21 09:22:25.714262", "failed": true, "rc": 1, "start": "2016-04-21 09:22:25.652250", "stderr": "Traceback (most recent call last):\n  File \"/usr/local/bin/wal-e\", line 5, in <module>\n    from pkg_resources import load_entry_point\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 2749, in <module>\n    working_set = WorkingSet._build_master()\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 446, in _build_master\n    return cls._build_from_requirements(__requires__)\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 459, in _build_from_requirements\n    dists = ws.resolve(reqs, Environment())\n  File \"/usr/lib/python2.7/dist-packages/pkg_resources.py\", line 628, in resolve\n    raise DistributionNotFound(req)\npkg_resources.DistributionNotFound: requests>=2.8.1,!=2.9.0", "stdout": "", "stdout_lines": [], "warnings": []}

Also tried setting wale_latest: true, saw the six dependency change but still same error.


Remove validation in QueryBuilder (or add a way to disable it)

QueryBuilder performs preliminary checks against the schema as the query is being built. I've run into various cases where this is really annoying and counter-productive.
For example create() queries fail with IllegalArgumentException if the element already exists.

Let's say I'm handling a user request to create a keyspace with specific replication options, but if it already exists with different options I want to fail and show the differences. I can't use ifNotExist() (it doesn't take options into account), so I need to execute a regular create() and catch the error. But IllegalArgumentException is very generic, I'll have to rely on some brittle message parsing. I would much rather deal with the persistence backend's AlreadyExistsException. And in fact I'll have to deal with that exception as well: there could easily be a race condition where the keyspace gets created after the query builder's check (especially if I don't use the built query immediately). So now I have to catch two exceptions.
Another example is if I'm doing a CQL migration. I know the target data model, and if a table doesn't match my strategy is to drop and recreate it. It might look like I can build the queries as I'm iterating the schema, and execute them all at the end. But in fact that doesn't work: the create() query will fail because the table already exists.

Another problem is that some of those checks are wrong. For example in BuiltUpdate:
    Preconditions.checkArgument(
        !rowsUpdated.isRanges(), "Invalid WHERE clause for UPDATE: cannot UPDATE a range of rows");

Range updates are valid in CQL.
QueryBuilder should focus on its primary responsibility: building queries. There might be a few runtime checks for syntactic errors that can't be enforced by the DSL. But semantic checks are the persistence backend's job.
IMHO those checks should be removed. If this is too aggressive, we could also add an option to disable them: dataStore.queryBuilder(boolean strict).

Related: Column.Type.fromCqlDefinitionOf() checks that UDTs exist in the keyspace. There are legitimate case where we might want to parse types that will only exist in the future, for example a tool that generates a schema according to a model.

I addressed fromCqlDefinitionOf  in ##679.

No plans to address (partly due to SGv2 refactoring removing schema connection for most builders), closing.


Flow

Let's discuss the flow process based on live examples.
The code may not be production-ready, of course, and without tests, but it should be sufficient for manual testing. Unfortunately, I haven't worked with the Telegram bot API for the last two years, so I may not be able to cover all cases. I'm asking for help from anyone who wants to contribute. Just pull my branch, try to describe your own flows, and provide feedback.
Anyone who wants to help can take a look at flow_simple_manual_testing/main.go, please. I've written two examples there, so you can test and modify as you want.
Updated: Please do not attempt to use my bot API token; I have already revoked it.

I will soon add some details for the "short" features.

Hey, first of all, thanks for putting in the effort; state-machine behaviour is one aspect of the API that we had discussed time and time again with very little to show in terms of implementation. Well, I have built quite a few bots over the years, and indeed employed various approaches to state-management. Flow at the time was my best attempt at "generalising" that experience, yet unfortunately it coincided with big changes in my life, that is to say I haven't touched Telegram bots in a long time now.
My money would be on @demget to provide most accurate feedback as he's much more knowledgable than me, however I'll do my best to point out some things about your implemnetation along the way.

I've completed some refactoring. Firstly, I updated the API, and now it looks like:
b := &tele.Bot{}
sendUserMessage := func(message string) func(flow.State) error {
	return func(state flow.State) error {
		return state.Read(flow.StateContextKey).(tele.Context).Reply(message)
	}
}
stepCompletedLogging := func(state flow.State, step *flow.Step) error {
	log.Println("Step completed")

	return nil
}
// Configure flow bus
flowBus := flow.NewBus(5 * time.Minute)
// Handle any text by flow bus
b.Handle(tele.OnText, flowBus.Handle)
// First flow
var email, password string
b.Handle("/start", flowBus.Flow(
	flow.New().
		Next(
			flow.NewStep(sendUserMessage("Enter email:")).
				Validate(nonEmptyValidator).
				Assign(TextAssigner(&email)).
				Then(stepCompletedLogging),
		).
		Next(
			flow.NewStep(sendUserMessage("Enter password:")).
				Assign(TextAssigner(&password)),
		).
		Then(func(state flow.State) error {
			log.Println("Steps are completed!")

			return state.Read(flow.StateContextKey).(tele.Context).Reply("Done")
		}),
))

Secondly, I had to make [State] an interface as you suggested. It's a good point.
Now, I'm considering using 'then' for the step. I believe we need to provide some DTO/interface for the completed step's information. With that opportunity, we can create a generalized solution (logging/metrics, etc.).

I suppose we can begin discussing the code now. The main area is ready, particularly the API (I don't plan to change it at the moment). Additionally, I've added metadata information for the flow/steps (the last one is a bit lacking, but...).
There are a few features that I need to implement, but we can review them later since they don't require changes to the code base; these features are just supplementary.
Idle flows: Naturally, we need to terminate idle flows after a user timeout and provide control over that to the user. I intend to achieve this by introducing a new [MetaDataFailureStage]. Since we already know the last step that the user completed, this shouldn't pose a problem.
We need to discuss the behavior of [assigner] and [validation]. I'm confident that there's a good opportunity here, and we need to develop some useful implementations. At the very least, we should validate prompts by type (text/media/etc.), implement a letters range validator, and for the assigner, a basic one that fills the user variable.
The final API appears as follows:
b := &tele.Bot{}
sendUserMessage := func(message string) func(flow.State) error {
	return func(state flow.State) error {
		return state.Read(flow.StateContextKey).(tele.Context).Reply(message)
	}
}
loggingEachStep := func(state flow.State, metadata flow.StepMetaData) {
	log.Println(fmt.Sprintf("Step completed [%d]", metadata.Step))
}

flowBus := flow.NewBus(b, 5*time.Minute)
b.Handle(tele.OnText, flowBus.Handle)

var email, password string
err := flowBus.Flow(
	"/start",
	flow.New().
		Next(flow.NewStep(sendUserMessage("Enter email:")).Assign(TextAssigner(&email))).
		Next(flow.NewStep(sendUserMessage("Enter password:")).Assign(TextAssigner(&password))).
		Then(registerUser).
		Catch(flowFailed),
)

@shindakioku

Then(registerUser).

How registerUser will have access to the variables email and password?
will this approach force me to use handlers as anonymous functions?


Support for Indiegala

Added in https://github.com/gogcom/galaxy-integrations-python-api/commit/8d6ec500f97e74efd709902cad2447dafc1b7397

Added in 0.49 8d6ec500f97e74efd709902cad2447dafc1b7397


IBM

This pull request adds the IBM feature to MFC. It supports:

2D and 3D IBM
A variety of IBM geometries
Support for multiphase flow and bubbles

@anshgupta1234 (Warning: use at your own risk) To fix conflicts due to formatting, I would suggest:
$ git branch ibm-backup origin/ibm
$ git remote add upstream git@github.com:henryleberre/MFC.git
$ git fetch upstream
$ git checkout upstream/master -- mfc.sh toolchain/indenter.py toolchain/bootstrap/format.sh
$ ./mfc.sh format
$ git stash -- mfc.sh toolchain
$ git add src
$ git commit -m temp
$ git rebase -i origin/master
... squash commits here ...
$ git rebase upstream/master
... hope there are only a few conflicts ...
$ git push -f

@anshgupta1234 you'll need to grab the latest toolchain formatting files to re-format your code (this is why there are so many merge errors). @henryleberre's instructions for this are above.

@sbryngelson I believe your #314 PR does not account for when files are added or removed in PRs (that is, when the file list of both branches does not match).

@sbryngelson I believe your #314 PR does not account for when files are added or removed in PRs (that is, when the file list of both branches does not match).

You're correct. I tested this when I was making the PR, but I guess I didn't consider some edge case. I will look into this, but it need not hold up this PR.

Is this PR "ready for review," or @anshgupta1234, do you need to sort out some other things first?

I believe it should be good to go!

Pasting some discussion I had on Slack with @anshgupta1234 here for "permanence":

512x256x256 3D IBM vs No IBM comparison test. Viscosity on, ran for 100 timesteps on GPUs w/ 4 ranks
Without IBM: 1.125714475909869 sec/timestep
With IBM:       1.144588934878508 sec/timestep
= 1.67666% increase

It’s GPU w/ 4 ranks

Regarding "correctness," @anshgupta1234  has verified that the theoretical bow shock angles are correct at a specific Re and Ma for flow past a cylinder.
Re = 6.5 x 10^5
Density = 0.2199 kg / m^3
Velocity: 527.2 m/s
Sphere Diameter (used as characteristic length): 10 cm = 0.1 m

Example overlay of MFC and ref paper simulations (of S. Sarkar UCSD).

@wilfonba and @henryleberre can you review this PR for obvious opportunities for refactoring? at present it adds many lines that seem redundant. obviously any bugs you spot should be flagged as well.

I made a few comments above. One way to remove a few hundred lines is to generalize the buffer communication subroutines so that only one is needed rather than two. I know this works with my surface tension work, but I'm not sure how well it would work with IBM communication. Also note that the total additions are closer to 5400 once you ignore the eigen solver module. I didn't see any obvious other ways to par down the number of additions significantly.

I made a few comments above. One way to remove a few hundred lines is to generalize the buffer communication subroutines so that only one is needed rather than two. I know this works with my surface tension work, but I'm not sure how well it would work with IBM communication. Also note that the total additions are closer to 5400 once you ignore the eigen solver module. I didn't see any obvious other ways to par down the number of additions significantly.

Got it. @anshgupta1234 can you have a look at the above? @wilfonba can you link your abstraction for MPI?
Also, the eigensolver module was associated with another problem which I just resolved via a commit. That was a good catch, I had to rewrite that module before @anshgupta1234's fork in order to remove goto statements for formatting purposes.

I haven't actually implemented a more general communication routine myself yet. I still have two separate but very similar subroutines. The data structures and types are almost identical in my case, so I can see how it would be done pretty easily. I just noticed that the IMB code is communicating integer values rather than floats. this would make a more general routine less straightforward.

@anshgupta1234 delete all of your case.py and README files in the tests/ directory. This is an update we made and they are no longer included in version control.

I haven't actually implemented a more general communication routine myself yet. I still have two separate but very similar subroutines. The data structures and types are almost identical in my case, so I can see how it would be done pretty easily. I just noticed that the IMB code is communicating integer values rather than floats. this would make a more general routine less straightforward.

Ok - understood. Let's keep this refactor for another time then, and do all communication refactoring at the same time (or at least most of it).
@anshgupta1234 can you confirm that both slip and no-slip work (for inviscid and viscous cases, respectively)? I think you showed me it did, but I cannot fully remember and at some point in time there was a bug. I guess the main one I'm thinking of is 3D+no-slip.

Hi everyone! I'll address the comments in the PR shortly. As for the no-slip case, although it does work qualitatively and quantitatively for 2D IBM, it's a little weirder for 3D IBM. @sbryngelson you may remember from our conversation that the no-slip case returned a realistic result with the correct bow shock but the flow behind the sphere did not match the paper I was referencing.
I discussed the case file with @anandrdbz, fixed the Reynolds number, and everything else pretty much matched the paper. However, the flow was still different.

@anshgupta1234 I'm aware of this case but suspect it to be due to differences in inflow (turbulent versus steady) or grid resolution, given that all the other features match so well. What about the slip case in 2D/3D?

@anshgupta1234 I'm aware of this case but suspect it to be due to differences in inflow (turbulent versus steady) or grid resolution, given that all the other features match so well. What about the slip case in 2D/3D?

@sbryngelson Yes, those do match up!

Looks good. Thanks @anshgupta1234. I think some minor cleanup is needed like what @wilfonba mentioned, and then we can merge.

@anshgupta1234 sorry to drag this out.

You have a couple of new tests golden files that don't have goldenfile_metadata. That metadata file should have been autogenerated when creating the goldenfile. Can you recreate/commit those by just regenerating the goldenfile?
Does IBM work with the 6-equation model, model_eqns = 3? This would be a quite useful feature if true. If it doesn't work, can you look at what is needed to add support? The velocities are all the same with the 6-equation model. This will be quite useful for our multiphase efforts. If it doesn't work but seems like a lot of effort to add, let me know.

@sbryngelson No worries, that's my bad! I'll push  it right now. And as for the 6-equation model, yes, that does work. @anandrdbz famously said,  "it only took like 4 lines of code" (or something like that)  to do it 😂

@sbryngelson I'm a bit unfamiliar with the 6-equation model, so correct me if I'm wrong; if we simply change model_eqns to 3 in examples/2D_ibm_multiphase, that would do the trick, right? As far as just having a 6-equation model test case.

@sbryngelson I'm a bit unfamiliar with the 6-equation model, so correct me if I'm wrong; if we simply change model_eqns to 3 in examples/2D_ibm_multiphase, that would do the trick, right? As far as just having a 6-equation model test case.

Umm I think so. See this case for a reference: https://github.com/MFlowCode/MFC/blob/master/examples/3D_sphbubcollapse/case.py

@henryleberre this is "ready" to merge pending your review. I see the test suite GPU run failed, but I think you fixed this in one of our recent PRs. Please confirm/deny that.

The latest autobenchmark @anshgupta1234 @henryleberre  suggests that your new code is a factor of 1/5-times slower than the current master branch for a 3D shock droplet problem:
  Case              Pre Process   Simulation   Post Process
 ───────────────────────────────────────────────────────────
  3D_shockdroplet         1.00x        0.80x          1.00x

This could well be a limitation of the test itself. Perhaps it needs to be re-run or ran for longer times.

@sbryngelson Ah, wait--do I need to make a separate PR to MFlowCode/MFC and then rebase this  branch?

@sbryngelson Ah, wait--do I need to make a separate PR to MFlowCode/MFC and then rebase this branch?

Potentially. I don't 100% understand it. @henryleberre would be more confident about the answer.

New GPU bench result:
mfc: OK > (venv) Entered the Python 3.9.12 virtual environment (>= 3.8).

      .=++*:          -+*+=.        |<EMAIL_ADDRESS>[Linux]
     :+   -*-        ==   =* .      | ----------------------------------------------------------
   :*+      ==      ++    .+-       | --jobs 1
  :*##-.....:*+   .#%+++=--+=:::.   | --mpi
  -=-++-======#=--**+++==+*++=::-:. | --no-gpu
 .:++=----------====+*= ==..:%..... | --no-debug
  .:-=++++===--==+=-+=   +.  :=     | --targets pre_process, simulation, and post_process
  +#=::::::::=%=. -+:    =+   *:    | ----------------------------------------------------------
 .*=-=*=..    :=+*+:      -...--    | $ ./mfc.sh (build, run, test, clean, count, packer) --help

 Comparing Bencharks: master/bench-gpu.yaml is x times slower than pr/bench-gpu.yaml.

  Case              Pre Process   Simulation   Post Process
 ───────────────────────────────────────────────────────────
  3D_shockdroplet         1.00x        0.21x          1.00x

mfc: (venv) Exiting the Python virtual environment.

This is crazy! I'll look into it.

Actually @sbryngelson , there is an alternative theory I just realized. I updated the benchmark to go from taking 100 to 500 time steps. So if it's running the simulation for 500 time steps on this PR but only for 100 on master, then that could be the issue!!

@anshgupta1234 yes this is surely it. can you post a screenshot of the nsight sys profile from the GUI of an IBM simulation without the openacc statements in that loop?

Latest benchmark btw

Interesting, I did the profiling on the 2D ibm case and the IB section took so little time even without the ACC loop.

@anshgupta1234 Can you post the profile file so I can look at it? What I really want to see is if the IBM loop ends up as a kernel on the device or not.

Looks good. It turns out that the IBM part of the profiled simulations was so short that it wasn't visible on the benchmarks, so removing the ACC loop and performing it on the host didn't result in a meaningful slowdown.


python312Packages.equinox: 0.11.3 -> 0.11.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.equinox is: A JAX library based around a simple idea: represent parameterised functions (such as neural networks) as PyTrees
meta.homepage for python312Packages.equinox is: https://github.com/patrick-kidger/equinox
meta.changelog for python312Packages.equinox is: https://github.com/patrick-kidger/equinox/releases/tag/v0.11.4
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.11.4 with grep in /nix/store/na0bys36i63zzyjlaqx8bnpxvl4xffnk-python3.12-equinox-0.11.4
found 0.11.4 in filename of file in /nix/store/na0bys36i63zzyjlaqx8bnpxvl4xffnk-python3.12-equinox-0.11.4

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.equinox
python312Packages.equinox

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/na0bys36i63zzyjlaqx8bnpxvl4xffnk-python3.12-equinox-0.11.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.equinox https://github.com/r-ryantm/nixpkgs/archive/7f0edf51e7916064a206b1959d7ef509d4ec9d29.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7f0edf51e7916064a206b1959d7ef509d4ec9d29#python312Packages.equinox

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/na0bys36i63zzyjlaqx8bnpxvl4xffnk-python3.12-equinox-0.11.4
ls -la /nix/store/na0bys36i63zzyjlaqx8bnpxvl4xffnk-python3.12-equinox-0.11.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

4 packages built:

python311Packages.equinox
python311Packages.equinox.dist
python312Packages.equinox
python312Packages.equinox.dist

Maintainer pings
cc @GaetanLepage for testing.


[(LIVE.FREE)] UFC Fight Night Edmonton Brandon Moreno vs Amir Albazi Live Streams Free On Tv Channel 2 NOV 2024

UFC Fight Night schedule, predictions, main event start time, live streaming: How to watch Brandon Moreno vs Amir Albazi?
UFC Fight Night is scheduled to take place at Rogers Place, Edmonton in Canada. UFC Fight Night main event is scheduled to start at 8 pm (ET) on November 2. UFC Fight Night will be live streamed on ESPN+.
Brandon Moreno will take on Amir Albazi at Flyweight category. Erin Blanchfield will fight Rose Namajunas in Women's flyweight category. Jhonata Diaz will have duel with Derrick Lewis in Heavyweight category. Caio Machado will fight Brendson Ribeiro in Light heavyweight category.
Marc-Andre Barriault will fight Dustin Stoltzfus in Middleweight category. Mike Mallott will take on Trevin Giles in Lightweight.
The co-main event is also a flyweight battle. Namajunas, a former two-time champion at strawweight, has already scored unanimous decision victories over Amanda Ribas and Tracy Cortez this year. Blanchfield is coming off a loss to Manon Fiorot in March, her first setback in seven UFC fights, but still poses a significant test for Namajunas.
Both fighters are trying to position themselves as the next contender behind Fiorot for a shot at title holder Valentina Shevchenko. Blanchfield, who is still only 25, is the -140 favorite and has been backed by 56 percent of the bets. However, the better-known Namajunas has been supported by 59 percent of the money.
Derrick Lewis vs. Jhonata Diniz
Diniz is a graduate of Dana White's Contender Series, and can continue his rise up the rankings with a win over the veteran Lewis. Lewis ran his record of UFC knockouts to 15 with a third-round stoppage of Rodrigo Nascimento in May. However, he hasn't won consecutive fights in three years, while Diniz has kicked off his promotion career with wins over Austen Lane and Karl Williams.
Lewis has been a popular underdog, drawing 69 percent of the bets and 78 percent of the money to upset Diniz (-165).
Also Read : If Harris loses U.S elections, Democrats have already made up their mind on who to blame and why
Caio Machado vs. Brendson Ribeiro
Both fighters were part of the DWCS last year, with Machado moving down to light heavyweight after dropping his first two UFC appearances at heavyweight. Ribeiro has likewise found the going rough early in his UFC career, losing to Zhang Mingling in his debut and most recently Magomed Gadzhiyasulov.
The public has been nearly split on which fighter will earn his first UFC victory on Saturday night. Ribeiro has garnered 53 percent of the total bets at +125, while Machado has been supported by 54 percent of the money as the -150 favorite.
Marc-Andre Barriault vs. Dustin Stoltzfus
These middleweights are both looking to rebound from losses. Barriault was stopped by Joe Pyfer in September for his second consecutive defeat, while Stoltfus was dropped by Brunno Ferreira his last time out.
Barriault has seen lopsided action with 88 percent of the money backing him as the heavy -200 favorite. However, Stoltzfus has drawn 55 percent of the total bets at +165.
Mike Malott vs. Trevin Giles
The main card opens with a welterweight fight between Malott and Giles. Malott is coming off a third-round stoppage loss to Neil Magny in January, while Giles has lost his past two fights. He has lost three consecutive against DWCS alums, and gets a shot to break that streak Saturday night.
Also Read : U.S elections: Who will Nikki Haley voters vote for - Trump or Harris? Suspense begins
Malott opened as the -275 favorite, but those odds have shortened even further to -350 as he has been backed by 77 percent of the money to beat Giles, who has lengthened from +210 to +260.
FAQs
Q1. When is UFC Fight Night?
A1. UFC Fight Night main event is scheduled to start at 8 pm (ET) on November 2.
Q2. Where can we watch UFC Fight Night?
A2.


No recepient after message sent

I know it would be a extremely stupid question, and it is probably my fault, as a newbie for libp2p, but I didn't get any help from neither GhatGPT nor StackOverflow, so I hope I can find any help here.
I merged the piece of example code showed in the Usage section of README.md into the example libp2p-in-the-browser of the js-libp2p repo, and started a dev server. However, I simply didn't find any message published from or to other "peers"(In my case, browser tabs), and when I entered await libp2p.services.pubsub.publish('test', new TextEncoder().encode('This is a test message.')) in the browser console, it returned a {recipients: Array(0)}. If that is my fault, please tell me how should I fix it properly.
Below is the code pieces related to Pubsub services:
import { gossipsub } from '@chainsafe/libp2p-gossipsub'

const libp2p=await createLibp2p({
//<other parts not shown here for brevity>
    services: {
        pubsub: gossipsub({ allowPublishToZeroPeers: true });
    }
})
  libp2p.addEventListener('peer:connect', (evt) => {
    const peerId = evt.detail
        const connections = libp2p.getConnections(evt.detail)
        log(`Connected to ${peerId.toString()}`)
        connections.forEach((conn) => {
            libp2p.services.pubsub.addPeer(conn.remotePeer, conn.stat.direction, conn.remoteAddr)
            libp2p.services.pubsub.subscribe('test')
            libp2p.services.pubsub.publish('test', new TextEncoder().encode('This is a test message.'))
        })
  })

Uhh, I realized what I have missed. Sorry for a useless issue.

@hutianyu2006 can you share what you missed? Will be helpful for people in the future with the same issue.

Currently I can't get my computer, so I can only recite it based on my memory lmao😂
The event listener should look like this:
libp2p.addEventListener('peer:connect', (evt) => {
            const peerId = evt.detail
            log(`Connected to ${peerId.toString()}`)
            libp2p.services.pubsub.connect(evt.detail)
            libp2p.services.pubsub.subscribe('test')
            libp2p.services.pubsub.publish('test', new TextEncoder().encode('This is a test message.'))
        })
  })

which means I simply changed the addPeer()  function into a connect() function.
By the way, there is a tiny question: Is it possible to add the libp2p.services.pubsub.connect(peerId) function into the example code of usage section? Otherwise that will cause confusing errors to newbies.


Added northern to cfg file.

@NikitaGandhi I assume the test and build check failure is from the prod_af root cfg file which is drawing on all crop mask layers even though the new layer hasn't been indexed into prod yet. Can we ignore it?

@eefaye updated prod inventory to make test pass. it won't impact production ows until we index.


Announcement block: Order of listing and dates of event items

This affects LIS, PB and SB.
Expectation from the announcement block as it has been in used in LIS & PB in classic sites(configurable by curator in Drupal block):
-- Events such as meetings, conferences relevant to the community are announced here.
-- The date of the future event is important than the date on which this event was created by the curator
-- The order of listing is chronological based on the future date of occurrence of the event. Example: the next immediate event may be listed first before an event in the a more distant future.
Current behaviour:
-- Behaves like a blog post as opposed to event announcements.
-- The date on which the _posts file is created affects the chronological order of listing of the event.
-- This results in a listing of events that is independent of (chronologically mixed) the actual event dates.
-- The announcement page at https://www.legumeinfo.org/announcements/ is also affected listing the events in a mixed chronological(event occurrence) order.  UX is affected because users have to manually sort out when would be the next couple of events or look for the date of past events.
-- Curator cannot configure in which order the events are to be listed.
-- (ADVANTAGE) Future events can be listed to appear in a future date and not necessarily now (like a blog post).
(https://github.com/legumeinfo/jekyll-legumeinfo/tree/main/announcements/_posts)
DEVELOPERS NEED
(I think @alancleary)

Developer(s) want to know how others(LIS, PeanutBase and SoyBase) feel about the use of this piece of utility and its behavior: as in the past classic site like event listings vs. the current Jekyll behavior like blog posts.
They will code the 'common theme and layout' according to inputs and opinions, and need.
Should they create another utility called Events?

Once there's consensus from the stakeholders, I can change this in the theme and update the starter site. Someone will need to port it to LIS, though, since it has been severed from the theme. I probably won't be able to get to it until the week of January 16th.

If the need of the individual Jekyll sites differ (I don't know if it should!), then some provision should be made to list them in the required order (for example in the index.html file) taking the dates from the individually posted files (the .md file within _posts/).  Currently it is:
{% for post in site.categories.announcements %}  # I am interpreting(not sure if this is correct) this is the order in the dir listing.

Per discussion in LIS-PB meeting, Jan-04:
The events should appear in order of occurrence date, immediate events at top and later events below.  The theme code will need change and @alancleary will attempt the needed modifications when convenient.
PeanutBase will use the changes (@svengato) once this is achieved at LIS.
https://github.com/PeanutBase/jekyll-peanutbase/issues/16

@sdash-github I'm going to try and take care of this today. This is what I plan to implement:

Change the name of "Announcements" to "Events". The existing "News" item should be used for announcements moving forward.
Add an optional end_data variable to the event front matter that allows events to occur over multiple days, e.g. a conference
Change the event ordering so that they are shown in chronological order, i.e. events that are currently happening will be first followed by the next upcoming event after that, and so on. Events that have already occurred will be shown in a separate "past events" section on the events page and omitted from the events widget on the main page.
Adding a date to the filename and in the front matter have the same effect (I'm not sure what happens when you use both...). I'll document that the preferred method is to add the date in the front matter so if there's an end date it'll be in the same place as the (start) date.

Let me know if I missed anything or if there's any errors.

I think that should do the job. Thanks.

Commit legumeinfo/jekyll-theme-legumeinfo@69e8ab0e1160ff9fed0d6eee29e918590cbab8f3 in the events branch repurposed (i.e. renamed) the announcements content for events. Announcements should now be made using the news content. New event-specific ordering and display pages still need to be implemented.

Commit legumeinfo/jekyll-theme-legumeinfo@188838b1996eaa0efea7de5488e3deec2cdc4bd4 added code that correctly orders the events and keeps their lists up to date using JavaScript so the site doesn't have to be rebuilt for the upcoming and past events lists to be correct.

done


[WFLY-11730] Do not record Galleon provisioned state and log provisioning time

https://issues.jboss.org/browse/WFLY-11730

@jfdenise Does this have any negative effect on your demos?
@yersan FYI, in case this affects eap-ization.

Thanks for the FYI @bstansberry , I was waiting for this issue; it will affect positively to our product branches since it will resolve https://issues.jboss.org/browse/JBEAP-16213 as not needed at all

@bstansberry, no impact. We provision the server using Galleon CLI, in this case the .galleon directory is generated. This directory is interesting to get information on what has been installed.


Releasing hermetic_cc_toolchain v3.0.1

https://github.com/bazelbuild/bazel/issues/21469
https://github.com/bazelbuild/bazel-central-registry/pull/1519


Send newsletter by name

Q
A

Branch
7.0x for features

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
yes

License
MIT

The Newsletter Specific Sending feature is a new addition to our Chameleon application that allows you to send a specific newsletter by its name. This feature simplifies the process of sending targeted newsletters to your subscribers.
Execute the Command:
 php app/console chameleon_system:newsletter:send-newsletter "Test Newsletter"

@kzorluoglu  could you create a PR for 7.1.x please?
Or we need a cherry pick for this PR.


Update Tensorflow version in Dockerfile

Updating Tensorflow version to v. 1.0.0

Thanks @raykendo


Week five scheduler

Implement week five scheduler with optional attendees and optional extra coding challenge. Add tests for each case

I didn't see test cases for the optional extra coding challenge. Could you please add some?

Fix changes, add tests for optional extra coding challenge

Fix suggested changes


Are you going to update it?

Thanks.

Hey @aleixsr. Have you noticed and issues with the app?

Well, really on evening, when status bar changes to darker colours is difficult to check ms.

@aleixsr Thank you for reporting this. I am on vacation right now but I can work on fixing that when I get back.

@aleixsr can you also please attach a screenshot of the issue for reference? Thanks

Sorry it was a mistake.

Sorry, I finally found what I was asking for, is it possible that ping bars appear also in white like other icons?

I have the same problem too, but after some random clicks, it goes back to white.


Changed "prepare data on the fly" functionality

Motivation and context

Add ability to manually prepare meta information and upload this meta information along with a video to reduce a task creation time.
Fix issue #2112
How has this been tested?

Manual tests, unit test
Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG file
[x] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

I need to discuss some moments with @azhavoro

Coverage decreased (-0.4%) to 64.98% when pulling c88f6450e3423e8551c2f56203676d0b950a3a65 on mk/fix_cache into 7456a1c6c57c5ee15e517a568b90d30d95fce8d7 on develop.

@Marishka17  @azhavoro
Is the PR ready and works? Can we merge it?

@Marishka17
Please, solve appeared merge conflicts


Certain base36 number literals being parsed as instructions

For example, when trying to use the T instruction with a length of 16 (g), the g is being parsed as the instruction instead, causing all the values originally locked inside the track to start executing on their own as well. However, this doesn't happen when creating the g using an Add instruction on the line above.
Example code:
0gTCDEFGHIJKLMNOPQR
...................

If you are using the builds from Itch.io, I just pushed a new version where the lowercase g doesn't override passive mode. If you're working from the repo, just pull again :)
Let me know if that works for you.

ah okay, thanks!


Correct the spelling of CocoaPods in README

This pull requests corrects the spelling of CocoaPods 🤓
https://github.com/CocoaPods/shared_resources/tree/master/media
One day I’ll make a bot that looks through the READMEs of all Pods, looks to see if it uses “Cocoapods” and PRs “CocoaPods” :D— Ørta (@orta) February 10, 2016

Current coverage is 100.00%

Merging #1 into master will not affect coverage as of 19619a5

@@            master      #1   diff @@
======================================
  Files            1       1
  Stmts          105      84    -21
  Branches         0       0
  Methods
======================================
- Hit            105      84    -21
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 19619a5

Powered by Codecov. Updated on successful CI builds.


When selecting datetime range, selection disappearing

When user selecting by mouse range of two dates and selection goes out of chart, then selection immediately is disappearing.
I think something could be done with on mouse event 'mouseup'.
Highcharts js plugin don't have this problem, it keeps selection until user releasing mouse button and event 'mouseup' is fired.
Thanks.

Thanks for reporting.
The issue is already reported on #578 and will be fixed soon.
Closing as duplicate of #578


Bring C-AD UI improvements to H-AD

C-AD has got some really nice UI improvements recently, especially thanks to @seansfkelley.
If these changes could be brought to H-AD (whatever is applicable) it will a huge improvement on the current UI.

Related: https://github.com/History-AutoDelete/History-AutoDelete/issues/24.


CameraDisplay visualisation inaccurate for an offset NominalFrame

Describe the bug
When using CameraDisplay to visualise an image in a NominalFrame that has an origin anywhere but at the exact centre of the camera, pixel positions seem generally correct but their sizes are greatly increased to the point they begin obscuring each other.
To Reproduce
Here's a relatively tame example of a telescope pointing at 10° altitude and trying to visualise in a NominalFrame with origin at 0°.
import astropy.units as u
from astropy.coordinates import SkyCoord, AltAz
from ctapipe.coordinates import CameraFrame, TelescopeFrame, NominalFrame
from ctapipe.image.toymodel import Gaussian
from ctapipe.instrument import CameraGeometry
from ctapipe.visualization import CameraDisplay
from matplotlib import pyplot as plt

camGeom = CameraGeometry.from_name('FlashCam')

# Set up example physical parameters on the CameraGeometry
focal = 16*u.m
pointing = SkyCoord(alt = 10*u.deg, az = 0*u.deg, frame = AltAz())
camGeom.frame = CameraFrame(focal_length = focal, telescope_pointing = pointing)

# Image data
r = camGeom.guess_radius()
model = Gaussian(x = 0.25*r, y = 0.75*r, width = 0.05*r, length = 0.1*r, psi = '10d')
_,image,_ = model.generate_image(camGeom, intensity = 5000)

# Camera geometry into alternate coordinate systems
horizon = SkyCoord(alt = 0*u.deg, az = 0*u.deg, frame = AltAz())
camGeom_t = camGeom.transform_to(TelescopeFrame(telescope_pointing = pointing))
camGeom_n = camGeom.transform_to(NominalFrame(origin = horizon))

fig,(ax1,ax2) = plt.subplots(1,2)
fig.set_size_inches(16,6)
camDisp_t = CameraDisplay(camGeom_t, ax = ax1, image = image)
camDisp_n = CameraDisplay(camGeom_n, ax = ax2, image = image)
plt.show()

Expected behaviour
Obviously I do expect there to be a distortion of the image as the pointing direction of the telescope gets further away from the origin (the horizon in this case) due to the projection to 2D, but I had anticipated that this would be correctly visualised with pixels changing their shapes. Instead, they seem to stay as a fixed shape and their sizes become wildly incorrect.
Supporting information
ctapipe v0.15.0
Additional context
I assume visualisation of NominalFrames like this wasn't really foreseen, which I understand. But I was trying to visualise the data in this frame to confirm that the NominalFrame transform was actually doing what I expected. For reasons like this I think it will happen that users attempt to visualise such frames. If meaningful calculation of the projected pixel shapes isn't possible, perhaps ensuring they at least don't blow up in size would be enough to make the visualisation useful.

Obviously I do expect there to be a distortion of the image as the pointing direction of the telescope gets further away from the origin (the horizon in this case) due to the projection to 2D, but I had anticipated that this would be correctly visualised with pixels changing their shapes. Instead, they seem to stay as a fixed shape and their sizes become wildly incorrect.

The reason is that when transforming to an angle based frame from the distance based frame we scale the pixel area by scaling without checking for translation.
Will try to fix!

@vmharvey mmh. Indeed.. That should probably not happen

Just don't try to make this kind of plot at an offset of +/-90 degrees in elevation ;-)  Then you'll really run into problems

@kosack I think this is not about the spreading (which is indeed correct) but about the slide rotation of the individual pixels

Ah, yes that does seem to be a bug. Was not obvious until I zoomed in.

Ah, yes that does seem to be a bug. Was not obvious until I zoomed in.

I think I know where this comes from, let me check...

I was correct, this should fix it: https://github.com/cta-observatory/ctapipe/pull/2037
Side note: I am not completely sure that when transforming between TelescopeFrame and NominalFrame, scaling the pixel area is the correct thing to do, since the fov of a pixel should not get larger or smaller.
So I guess we should add a check for if we switch units and only then do the scaling? But that would also not work for transforming from TelescopeFrame to NominalFrame...
Also, for these high offset versions, the pixel rotation needs to per pixel I guess...
So I'd probably say that at the moment the CameraDisplay (or also the transformations) are not really intended to work for large rotations.

Fixed in #2037


Markdown: Removed/replaced closing tags that were never opened.

Would be great to have a 0.3.2 release with these changes as it breaks my documentation. ;-)

It's already released as 0.3.2! Thanks!

Thank you!


cadence-1.0 upgrade

Hello @joshuahannan Please take a review too:
In-Scope:

cadence/contracts/*
cadence/scripts/*
cadence/transactions/*

Thanks!


Improvements to dev proxy

Prefer RAZZLE_INTERNAL_API_PATH over RAZZLE_API_PATH for devProxyToApiPath. This makes more sense (the proxy target is accessed from the Volto server side) and it makes the dev proxy work without additional configuration when running Volto in a Docker container RAZZLE_API_INTERNAL_API_PATH is already set correctly.
Keep the URL protocol from the API path in the virtual hosting path passed to the backend, instead of hardcoding http. This lets the backend generate correct URLs when the backend is served over https.
Use a new environment variable, RAZZLE_DEV_PROXY_INSECURE, to control whether the proxy checks certificates on the backend.
Always log where API paths are proxied on startup, not only in development mode.
In the APIResourceWithAuth helper, always use devProxyToApiPath, not only in development mode. It's a potentially confusing inconsistency.

With these changes, it's possible to run Volto with a remote backend served over https. For example:
RAZZLE_INTERNAL_API_PATH=https://demo.plone.org RAZZLE_PROXY_REWRITE_TARGET=/++api++ RAZZLE_DEV_PROXY_INSECURE=1 yarn start

Note: I still need to check if there are any implications for the docs.

@davisagli seamless traefik docker config:
  routers:
    frontend:
      rule: "Host(`localhost`)"
      service: frontend
    backend:
      rule: "Host(`localhost`) && PathPrefix(`/++api++`)"
      service: backend
      middlewares:
        - backend

  middlewares:
    backend:
      replacePathRegex:
        regex: "^/\\+\\+api\\+\\+($|/.*)"
        replacement: "/VirtualHostBase/http/localhost/plone/++api++/VirtualHostRoot$1"

  services:
    frontend:
      loadBalancer:
        servers:
          - url: "http://host.docker.internal:3000"
    backend:
      loadBalancer:
        servers:
          - url: "http://host.docker.internal:55001"

has localhost on it, could be the problem?

works like a charm... We should document it properly.


Test for stats versioning

testing for #17549

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "testForStatsVersioning" git@github.com:donhcd/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@calavera the tests you requested. not sure what's going on with the windows tests?

cool, thank you!

I guess the windows servers are on weekend mode already.

LGTM

LGTM


Is scanner set to the ScannerFactory obsolete

I am using third party code that uses: ScannerFactory.setScanner( ... );
If my reading of the recent SwaggerContextService is correct the getScanner() functions returns

SwaggerScannerLocator.getInstance().getScanner(scannerIdKey) if the result is not null
otherwise ScannerFactory.getScanner();

The getScanner function of the SwaggerScannerLocator is implemented this way:
    public Scanner getScanner(String id) {
        Scanner value = map.get(id);
        if (value != null) {
            return value;
        }
        return new DefaultJaxrsScanner();
    }

so it returns never null.
Isn't the usage of the ScannerFactory then obsolete?

I have compared v1.5.5 to v1.5.7:
The previous call to the factory has been changed:
<EMAIL_ADDRESS>The getScanner function uses the ScannerFactory only if the value is null.
<EMAIL_ADDRESS>But SwaggerScannerLocator will create a default scanner if there is no one defined and never returns null.
<EMAIL_ADDRESS>

Please give details of what you expect.

Hm, I could try to rephrase my question:
The scanner set by the ScannerFactory.setScanner method is not (has not been on v1.5.7) used anymore (see explanation in the previous two comments).
So, code that relies on the fact the the scanner set by setScanner is used if there is no one for the given scanner id has been broken with the linked changes.
If the scanner set by the resp. method is not used and the behaviour has been changed, too, wouldn't it be better to remove the function so third party code knows, that this method to force a special default scanner does not work anymore?


[INLONG-hello][Sort] Optimize the definition of enumeration variables…

… and remove the semicolon
Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes #XYZ

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

@YaziLee thanks for your contribution, please create an issue to describe the problem.

please change the title [INLONG-hello] to inlong-issueNumber


Physics interpolation (3D): Issue when setting the position of a Rigidbody or Kinematicbody

Godot version
Godot 3.5 beta 2
System information
Ubuntu 21.10, GLES3
Issue description
When I instance a Rigidbody or Kinematicbody and immediately set the position right after, the object blinks for at least 1 frame before moving to the desired position.
This is not an issue when Physics Interpolation is disabled (Project Settings -> Physics -> Common).
Steps to reproduce

Instance a Rigidbody or Kinematicbody and add it as a child node. Either call_deferred("add_child", body) or add_child(body) - doesn't matter.
Set the position using body.global_transform.origin = Vector3(5, 0, 0)

Minimal reproduction project
Run the game and left-click your mouse.
Godot-3.5.beta2.PhyInt.zip

See the in progress docs:
https://github.com/lawnjelly/Misc/blob/master/FTIDocs/FTI.md
Specifically this section:
https://github.com/lawnjelly/Misc/blob/master/FTIDocs/FTI.md#call-reset_physics_interpolation-when-teleporting-objects-large-distances
This is expected :+1: , this is one of the gotchas of using interpolation, you need to call reset_physics_interpolation() after moving to a new location (teleport) rather than regular movement, in order to prevent this "streaking".

By the way, you can avoid the deferring and the yielding by setting the correct position from the start, like this:
func _input(event: InputEvent) -> void:
    if event is InputEventMouseButton:
        if event.button_index == BUTTON_LEFT and event.pressed:
            var sphere = _sphere.instance()
            sphere.translation = to_local(Vector3(5, 0, 0))
            add_child(sphere)
            sphere.reset_physics_interpolation()

The "difficult" part of this alternative is using local coordinates instead of global coordinates. In my opinion it's conceptually easier than deferring and yielding. Moreover, it ensures that the object in the physics server is created with the correct values on the first frame. @lawnjelly Why is the reset needed in this case as well (in 3.5 beta 2)?

@lawnjelly Why is the reset needed in this case as well (in 3.5 beta 2)?

It could either be some transform being set at the origin in the client code, or even just the previous transform in the physics interpolation code is unset on the first tick. So calling the reset_physics_interpolation() is always best practice I think.

Hm, I'm not sure if this is the right issue to append my problem to, but from my initial search it seems like it.
I am unsure if it is a problem from my side but it seems that adding nodes to the tree in general seems to interpolate from their "original" translation.
So:
var new_thing = Thing.instance()
new_thing.translation = Vector3(300,0,0)
add_child(new_thing)

would interpolate from Vector3.ZERO.
To have the behaviour that I guess most users would expect, I need to do:
var new_thing = Thing.instance()
new_thing.translation = Vector3(300,0,0)
add_child(new_thing)
new_thing.reset_physics_interpolation()

Is that correct?

To have the behaviour that I guess most users would expect, I need to do:
Is that correct?

Yes, this is correct, adding a new object to the scene (in a location other than the origin) is one of the cases where a reset_physics_interpolation() is needed.
I think at some point I did consider automatically doing a reset_physics_interpolation() when adding to the scene tree, but there are so many cases where this would break down, it might be more confusing than helpful.
In short any time you are instantaneously moving to a new location (teleport, often referred to), including adding to the scene tree you should call reset_physics_interpolation() to prevent "streaking".

@lawnjelly there is no equivalent that I see for Godot 4.0 that is for calling reset_physics_interpolation(), how would I do this in 4.0?

@lawnjelly there is no equivalent that I see for Godot 4.0 that is for calling reset_physics_interpolation(), how would I do this in 4.0?

Physics interpolation isn't implemented yet in 4.x.

Closing, cause lawnjelly answered https://github.com/godotengine/godot/issues/59173#issuecomment-1068391400
For 4.x see https://github.com/godotengine/godot/pull/92391


Don't instruct LLM to use the LIMIT clause, which is incompatible with SQL Server

The current prompt specifically instructs the LLM to use the LIMIT clause. This will cause issues with MS SQL Server, which uses SELECT TOP instead of LIMIT. The generated SQL will use LIMIT; the instruction to "always limit... using the LIMIT clause" seems to override the "create a syntactically correct mssql query to run" portion. Reported here: https://github.com/hwchase17/langchain/issues/1103#issuecomment-1441144224
I don't have access to a SQL Server instance to test, but removing that part of the prompt in OpenAI Playground results in the correct SELECT TOP syntax, whereas keeping it in results in the LIMIT clause, even when instructing it to generate syntactically correct mssql. It's also still correctly using LIMIT in my MariaDB database. I think in this case we can assume that the model will select the appropriate method based on the dialect specified.
In general, it would be nice to be able to test a suite of SQL dialects for things like dialect-specific syntax and other issues we've run into in the past, but I'm not quite sure how to best approach that yet.

I am still seeing the same issue exist in .103 version. here is the issue for MSSQL. It uses LIMIT instead of TOP
Find the restaurant owners (owner_name) that own one or multiple restaurants in the city with the number of restaurants (num_restaurants) they own. Find the first top 10 owners ordered by descending order using the number of restaurants.
SQLQuery:
SELECT owner_name, COUNT(*) AS num_restaurants
FROM businesses
GROUP BY owner_name
ORDER BY num_restaurants DESC
LIMIT 10;


Don't use assert in examples

assert doesn't evaluate its argument in a non-debug build, so the examples are misleading.

Fixed by #14.


Clean up image API code paths

Most commits have individual descriptions, but at a high level I'm essentially getting rid of the complex mutually-recursing-across-processes behavior previously present in image.js. I've moved the part that actually calls into ImageMagick into its own file, so instead of commands calling magick.run which calls the image API which calls magick.run again (or magick.run calling itself in a worker thread if the image API is disabled), magick.run and the image API both delegate to the new "call ImageMagick" function.
I've also gotten rid of some duplicate code for handling GIFs, removed some dead code relating to image types (it can be re-added later in a much cleaner way if needed), and fixed a bug where GIF-only commands would throw an internal error instead of displaying the intended "that isn't a GIF!" message.

When testing this out on my instance, something happened where it pulled the previous image instead of the current one when running the motivate command. No idea how these changes could have caused that (or if they even did) but I'll look into it.

When testing this out on my instance, something happened where it pulled the previous image instead of the current one when running the motivate command. No idea how these changes could have caused that (or if they even did) but I'll look into it.

Never mind, it seems to be an issue with the dev bot as well.

Never mind, it seems to be an issue with the dev bot as well.

Looks pretty good.

Looks pretty good.


Update to use ActivityStreams

Related to: https://jira.duraspace.org/browse/FCREPO-2602

Create resource
String Property - org.fcrepo.jms.eventType | https://www.w3.org/ns/activitystreams#Create,https://www.w3.org/ns/activitystreams#Update

Delete resource
String Property - org.fcrepo.jms.eventType | https://www.w3.org/ns/activitystreams#Delete

Looks good 👍


add token logo Mars

Dear Trust wallet Team,
kindly assist me to add logo for Mars token.
Contract: 0x165cb5cd6c56c5f1e224a9a2da750621929f4419
Symbol: MARS
Decimal: 18
Thanks and best regards.


Release 0.1.0 version?

There are a few reasons for this bump:

It allows consumers to depend on ember-app-scheduler via ^ (caret) dependency. Which means they can automatically get patch version bumps if we have bugs that need fixing.
It signifies that a breaking change has occurred (talking about dropping support for Node < 4 in my last PR).

Any objections?

Done and published 0.1.0 to npm.

Thank you!


Can't scroll to hash twice

I'm using the method explained in issue #373, including the 0 timeout, to scroll to hashes, and it does work fine if routing happens, but if the destination URL including hash is the same as current, then it seems as though no routing happens and therefore Router.after is never called. To reproduce, click a link that scrolls to a hash. Then scroll away from that spot and click a link that scrolls to the same hash. It doesn't scroll back.
@booshtukka @tmeasday, do you have a solution for this?
Also, this seems to me like something that should be default behavior. Couldn't iron-router set up this hook by default? Maybe with an option that allows you to specify a scrollto offset in case there's a fixed header.

@aldeed Thanks for submitting these bugs and the associated MeteorPads. I'm having the same problems in my iron router app where the back button won't return (when trying to return to an original link on the same page) and then the original link also no longer works).  I installed iron-router-autoscroll, but that didn't solve these problems either.
Did you ever find a solution? I can't believe this doesn't work. Links are the foundation of the web.

Any update on this issue? I'm still facing it.