Remove square size from data root tuple root generation

          That's great. Let's do it in a subsequent PR

Originally posted by @cmwaters in https://github.com/celestiaorg/celestia-core/pull/1003#discussion_r1269239232

@cmwaters I believe this is where we should remove the square size from:
https://github.com/celestiaorg/celestia-core/blob/c3ab251659f6fe0f36d10e0dbd14c29a78a85352/proto/tendermint/types/types.proto#L84-L104
However, we still need the data root for the QGB. Is there a different way to get it so that  we also remove it from here and only leave txs?

The data root is actually also part of the header so we can retrieve it from that


SSLError - Exception

Add an exception handler for connection timeout.

Hi,
The exception is handled as expected - an informative log is printed describing the issue, in addition to the trace.
Thanks,
Doron

Well, the problem is not the log. The problem is that the script stops after this error and has to be restarted.


Redirect error on publishSigned

Describe the bug
When I try to run publishSigned, I get the error below:
sbt:JSONoid Discovery> publishSigned
[info] Wrote /home/mmior/apps/jsonoid-discovery/target/scala-2.13/jsonoid-discovery_2.13-0.20.1-0-f8b1bab1-20240620-1043-SNAPSHOT.pom
[info] compiling 1 Scala source to /home/mmior/apps/jsonoid-discovery/target/scala-2.13/classes ...
[warn] multiple main classes detected: run 'show discoveredMainClasses' to see the list
[info] Main Scala API documentation to /home/mmior/apps/jsonoid-discovery/target/scala-2.13/api...
[info] Main Scala API documentation successful.
[error] stack trace is suppressed; run last publishSigned for the full output
[error] (publishSigned) java.net.ProtocolException: Server redirected too many  times (20)
[error] Total time: 10 s, completed Jun 20, 2024 10:46:05 AM

How to reproduce the issue
Describe how to reproduce the issue and show the current configurations.

sbt version: 1.8.2
sbt-sonatype version: 3.10.0
Show your build settings:

$ sbt
> sonatypeProfileName
io.github.dataunitylab
> sonatypeRepository
https://s01.oss.sonatype.org/service/local

GitHub repository: https://github.com/dataunitylab/jsonoid-discovery.git

I'm guessing the whole thing is really sbt/sbt-pgp#182

This seems to have gone away on its own. Either that or I changed something I'm not aware of, but the same credentials that were failing now seem to be working so this is resolved.

@michaelmior I'm on the same boat now. Haven't changed my credentials, just logged into the web interface successfully, yet I'm getting the same redirect error ("java.net.ProtocolException: Server redirected too many  times")
Can you try to recall what changes you may have done? Anything specific to the credentials file, upgrading any plugin, etc? I'm at my wits end :(

@gueniai Unfortunately I can't pinpoint what I changed may have had any impact. When things finally worked, I don't believe there were any code or configuration changes on my end.

same situation here...
@gueniai were you able to solve the problem?


Count(): Parameter must be an array or an object that implements Countable

Hello, @Hussam3bd.
I have a erro:
"Count(): Parameter must be an array or an object that implements Countable" in FirebaseMessage.php line 293.
However in the commit 7bae8c3989ba6bc0228152fe081420d8876af033, this erro has already been fixed.
But I use the composer update or I delete the "vendor" and reinstall the package again and the code does not modify, is there something missing?
Obs: The package code does not change too.

Hi @bognarjunior,
Thank you for informing us, we didn't release the new commit that @Evertt mead
However try now composer update and I hope everything gonna be okay


add template_type in vizjson presenter

close template compile issue in embeds described in https://github.com/CartoDB/cartodb/issues/8558
rather than change the default template_type in cartodb.js for the new embeds (which for some reason was underscore, instead of moustache, as in old embeds) we are now exposing the template_type (updated at the time it's changed in the builder) for namedmaps
CR @gfiorav (thanks mate) /cc @alonsogarciapablo

Have you checked that the default case works fine? i.e, when you don't specify a template type?

If you also checked old embeds, than it's 📦 after CI is 🍏
Also, don't forget to purge varnish cache please :)

sure I did! thanks for the thorough check 😅

:clap:


Create a Colab notebook

Provide a Colab notebook for faster implementation.

Do you want a Colab notebook created?

Yes

Do you want any content in it or just empty?

Hi @MauryaRitesh, I am learning/using Colab as part of my Intro to Data Science course in college. I liked the idea behind the project and want to try and contribute to it. I can try and make a Colab notebook for this project as I also think it will lead to smoother usage. Can you assign this task to me?

Hey can you assign me to this issue as well. I do have experience working with Colab notebooks for my own projects.

Please assign this issue to me

@MauryaRitesh does this issue still needs to be worked one...if yes can u assign it to me??


C19 error when using tgpio service

Hello. We are using ROS2 Humble to develop our application on xArm6. The firmware version is 1.12.10, studio version is 1.12.4.
We added two new services in xarm driver_service to call API to control our end effector.

Then, we executed ros2 launch xarm_planner lite6_planner_realmove.launch.py robot_ip:=<IP_ADDRESS> and executed a test script to demonstrate the workflow by calling services:[/xarm_pose_plan, /xarm_joint_plan, /xarm_exec_plan] (moving to a specific position and turning the end effector on/off).

However, after successfully executing the while loop several times, we encountered the C19 error code, causing the robot_state ROS topic to change (mode 1->0; state 2->5). Additionally, the joint_state topic stopped publishing. The xarm6_realmove log shows that the path was successfully planned, but could not be executed. But the end effector services keep working well. Here is the log:

We are sure that this case only occur involve calling tgoio service. Can you please assist us in resolving this issue? Thanks.

@KylecheungKml
Generally, C19 occurs at the end of the communication failure. There are many reasons for the failure. I guess you are timed out here (the default is 5ms). You can set the timeout time through set_tgpio_modbus_timeout.
Once an error occurs, the robotic arm cannot be controlled because the mode and status of the robotic arm have changed. The error must be cleared before the mode and status can be set.

Thank you. The issue was resolved after increasing the tgpio timeout. However, after running the loop for two hours, the log showed the message: [ros2_control_node-6] [INFO] [1684733867.884816963] [UFACTORY.RobotHW]: [<IP_ADDRESS>] Read() returns: 5. As a result, the arm was unable to execute the planned pose. I also noticed that this issue arises when I boot up my computer for the first time and execute the ros2 launch xarm_planner lite6_planner_realmove.launch.py robot_ip:=<IP_ADDRESS>.  But the second-time execution is OK.

@KylecheungKml

Isn’t your robotic arm xArm6? Why do you use the lite6 startup script?
Can you check the SDK version output at startup?
Can't start xarm_planner for the first time to plan and output anything?

@vimior
1, sorry, it is a typo, should be xArm6.
2, SDK version is 1.11.7
3, Can't start xarm_planner problem does not always occur. Terminal keeps output [Read() returns: 5], Rviz2 will show up, but cannot exec the robot arm.

@KylecheungKml
I tried to use the same version to test, but I can't reproduce this situation for the time being. You can leave any abnormal output before and after the earliest output "Read() returns 5". Generally, this sentence is output because the correct response data cannot be obtained when obtaining the current position of the robotic arm.

I have attached two log files, one for abnormal execution during the first run and another for normal execution during the second run. Based on the logs, I guess that starting with a C2 error at the beginning of the execution may cause the problem. However, it's unclear if the Read() returns 5 error that occurs during the two-hour execution also has this same issue.
abnormal.txt
normal.txt

@KylecheungKml
When C2 appears, the emergency stop is usually pressed or the emergency stop IO is triggered. The xarm_planner needs to be initialized when it is started. It cannot be initialized when it is in C2. The subsequent acquisition of the position of the robotic arm will also fail, but generally it will not return 5, I will try to reproduce the "Read() returns 5" phenomenon.

Thank you. We have conducted tests for several weeks, and the issue with the "return 5" problem no longer occurs.


Changed caching strategy

Changed caching strategy in TokenKeyServiceWithCache

@mwdb this belongs to the mail from last week. Please have a final look at it.

@mwdb can you please have a look at it, the details were provided to you via mail.
Thanks!

@hassler-d
please adapt the configuration:

reduce time to 10 minutes

and we have to clarify with Patrick , whether is a delay given, until the new token key is used for token creation...

@hassler-d
please adapt the configuration:

reduce time to 10 minutes

Was already at 10 minutes. However, the comment explaining the value was not correct. I changed that.


Write manual test cases for "QuebecPage"

Please write a manual test case for the following pages, which involves breaking down the functionality of the webpage into specific steps that can be tested.
QuebecPage: https://prayersconnect.com/mosques/search/Montr%25C3%25A9al%252C%2520Quebec%252C%2520Canada/45.45621%2C-73.546783/45.45621%2C-73.546783
Please use this Google sheet for your test cases.
https://docs.google.com/spreadsheets/d/1PIzULJpRSBqpsujibqaAh4GZrrIZb48yqy2q3yJpSBI/edit?usp=sharing

Quebec Page:
https://docs.google.com/spreadsheets/d/1PIzULJpRSBqpsujibqaAh4GZrrIZb48yqy2q3yJpSBI/edit#gid=522322061
all the problematic mosque id number is enlisted with problem and remark in the image review sheet.

Thank you


Indentation/spacing on custom icons

I'm using 16px x 16px icons without spacing on the edges.
Added spacing on the edges will only result in overlapping icons aove the file name or smaller icons.
What's the solution to add spacing between the file name and the icon like in the case of the default Material icons?
users.facade.ts and user.models.ts are custom added icons and they don't have spacing between the file icon and the file name.

Some of the icons have padding, as you have said. They make the icon appear smaller, indeed, but it is not an issue in general, as those icons are simple. But indeed, I see some weird behaviour in your screenshot, as if the icon "suck up" the text. I'll investigate.


Bring the ability to use influxdb (#2388)

RPC-Maas is able to push metrics to influx, we should expose
this facility to our users, which includes our gates.
On top of it, we should bring the facility to deploy influx, if
need be.
RPC-Maas is able to push metrics to influx, we should expose
this facility to our users, which includes our gates.
Note: To include this in a more standardized way, I had to
create scripts/(functions|deploy-rpc-playbooks).sh, and use these
from deploy.sh. Further work can be done to simplify deploy.sh
script to look more like the one on upper branches.
Issue: LA-314
Manual cherry pick from 45046ee09887b4eb3e91e10de9707c9634011b23

run_ansible command not found. Will fix that.

Ready to review, and passed checks!

Yeah, and all the variables too.


Move scipy import in clearsky.py

As discussed in #13.
I briefly looked into adding a real test for this, but it wasn't immediately obvious how to do it. Instead, I ran the test suite on a virtualenv without scipy. Here's the output, as expected:
$ nosetests pvlib/test/test_clearsky.py -v
pvlib.test.test_clearsky.test_ineichen_required ... ERROR
pvlib.test.test_clearsky.test_ineichen_supply_linke ... ok
pvlib.test.test_clearsky.test_ineichen_solpos ... ok
pvlib.test.test_clearsky.test_ineichen_airmass ... ok
pvlib.test.test_clearsky.test_ineichen_keys ... ok
pvlib.test.test_clearsky.test_haurwitz ... ok
pvlib.test.test_clearsky.test_haurwitz_keys ... ok
pvlib.test.test_clearsky.test_disc_keys ... ok
pvlib.test.test_clearsky.test_disc_value ... ok

======================================================================
ERROR: pvlib.test.test_clearsky.test_ineichen_required
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/holmgren/.pyenv/versions/pvlib-3.4.3/lib/python3.4/site-packages/nose/case.py", line 198, in runTest
    self.test(*self.arg)
  File "/Users/holmgren/git_repos/pvlib-python/pvlib/test/test_clearsky.py", line 34, in test_ineichen_required
    clearsky.ineichen(times, tus)
  File "/Users/holmgren/git_repos/pvlib-python/pvlib/clearsky.py", line 134, in ineichen
    'supply your own turbidities.')
nose.proxy.ImportError: The Linke turbidity lookup table requires scipy. You can still use clearsky.ineichen if you supply your own turbidities.
-------------------- >> begin captured logging << --------------------
pvlib: DEBUG: creating Location object
pvlib: DEBUG: using PyEphem to calculate solar position
pvlib: DEBUG: tz_localize to US/Arizona and then tz_convert to UTC
pvlib: DEBUG: irradiance.extraradiation()
pvlib: DEBUG: Calculating ET rad using Spencer method
pvlib: DEBUG: using PyEphem to calculate solar position
pvlib: DEBUG: tz_localize to US/Arizona and then tz_convert to UTC
--------------------- >> end captured logging << ---------------------

----------------------------------------------------------------------
Ran 9 tests in 1.986s

FAILED (errors=1)

Resolved the merge conflict in the whats new file.

@bmu or @Calama-Consulting are we good to go on this one?

Ooops, maybe I was to hasty. When I run the tests without scipy I get for errors:

pvlib.test.test_clearsky.test_ineichen_required ... ERROR
pvlib.test.test_clearsky.test_disc_value ... ok
Failure: ImportError (The Linke turbidity lookup table requires scipy. You can still use    clearsky.ineichen if you supply your own turbidities.) ... ERROR
pvlib.test.test_location.test_location_print_pytz ... ok
Failure: ImportError (The Linke turbidity lookup table requires scipy. You can still use clearsky.ineichen if you supply your own turbidities.) ... ERROR
pvlib.test.test_solarposition.test_calc_time ... ERROR

Not sure, where the errors are thrown in all cases (I don't see any use of ineichen or scipy in Location), I just copied the output of  nosetests -v -w pvlib --with-coverage --cover-package=pvlib.

Ok, that was very confusing but I tracked it down to test_irradiance.py and test_pvsystem.py failing on module import without scipy. They both use clearsky.ineichen to calculate the irradiance data used in later tests. I changed the calls to use a fixed turbidity.
The nosetests log now looks right to me.
I agree that we should skip the tests that require scipy if scipy is not installed, however, I would suggest that we make that a 0.2 milestone.

Ah, not sure what the best git approach is here, but I ended up making a new PR in #34 to resolve this.


langchain[patch],community[minor]: Move some unit tests from langchain to community, use core for fake models

Updates in some tests

https://github.com/langchain-ai/langchain/issues/21080


Add rule: extra-white-lines

Progress of #41
This will report any white lines that are extra.
I have commented out one test case, because there is a bug in HTMLBars. See: tildeio/htmlbars#454

Looking good! Can you add:

Info to the README
Details about the expected config in the inline docs in the rule
add an entry to the default blueprints config file

@martndemus - Ping?

The underlying engine has been migrated to https://github.com/rwjblue/ember-template-lint (only ember-cli specific parts live here now).  Can you please migrate this work over there?


Incorrect Basic Auths Make Postman API Call Fail

Hi,
Here is the collection: https://github.com/CiscoDevNet/dne-dna-code/blob/master/intro-mdp/postman/DevNet Learning Labs- Intro to Model Driven Programmability.postman_collection.json
Every call which uses this auth...
Basic cm9vdDpEX1ZheSFfMTAm
...should replace it with this auth...
Basic ZGV2ZWxvcGVyOmxhc3RvcmFuZ2VyZXN0b3JlYmFsbDg4NzY=
This is based on the newish credentials in this sandbox: https://devnetsandbox.cisco.com/RM/Diagram/Index/27d9747a-db48-4565-8d44-df318fce37ad?diagramType=Topology
I have tested it

These need updated


A list view

A list view for the commits would be nice because it’s sometimes hard to navigate in the graphical interface among the bubbles.

this is a good idea.


Injest ihat figures

References https://www.pivotaltracker.com/story/show/88902590
Take the images returned from iHat and create figures out of them.

:+1:

:+1:


Feature/func

resolving a global ie $x: false !global; then copying it, results in foobar. Need to address when resolution happens when a copy could be possible ie. inside every function


add --dry-run to cli utilities

234a4324fc4abcbfcfba1cd1729ec044938a35b2 adds the ability to turn your context into a non-destructive one by reloading the db into memory.
e45c7b1100f2959a2cb697d8ce4e81f4c151254b adds basic --dry-run functionality into ainclude

[ ] add to arm
[x] add to atags create, atags remove

atags received --dry-run in 8bc794ac8f73cfea8e44bee42269c286e95d0b32.

arm in https://github.com/lun-4/awtfdb/commit/7aa1aa9d417310aabb18ed4a0a1b09b895d0b747.


Windows 8.1 does not support file:///xxx protocol icon paths

When using a file:///xxx protocol path for the icon, it will just fail on Windows 8.1. I do see that documentation always says to use absolute paths and doesn't mention file protocol paths; It was an oversight on my part because I was refactoring an nw.js app using window.Notifcation where file:/// are needed.
Platform test results when using file:///xxx paths (node-notifier@4.5.0):

OSX: Works
Windows 10: Works
Windows 8.1: Does not work

Example:
var notifier = require('node-notifier');

notifier.notify({
  'title': 'Heya',
  'message': 'foo bar',
  'icon': 'file:///C:/some-project/mlm.png'
  //'icon': 'file:///C:\\some-project\\mlm.png'
});

From my knowledge, you're only able to provide the file's absolute path, by means of path, __dirname, and specified other methods. Opening via the file:// protocol is not supported.

We could possibly patch the paths on Windows 8.1 to remove the scheme, to make it work.

@mikaelbr :+1: That is also what I had in mind


New User UX

Create instructions for first time users, so they know where to click and how to use stuff.
Create question symbol buttons that allow the user to get information on various areas of the screen. Modal popups with the required information would serve best in this case.

UX Initial login should encourage the user to click on the menu on the top right.
UX Login dropdown menu should have E-mail signup and try it out menu options.
UX Login modal should have a guest account option.
UX Login modal, upon clicking on the signup button, should populate the signup modal with variables from the Login modal.
UI Login modal's login button should be more prominant than the other buttons to make it look like it's a part of the form.
UI Signup modal's signup button should be more prominant than the other buttons to make it look like it's a part of the form.
UX Alter User Profile modal to have user initial variables be instructive variables
UX Alter User Profile modal to have Contacts and Chatrooms sections declare that the current user has no contacts and/or isn't a member of any chatrooms.
UX On first login, add instructions into the empty pane on how to use this application in the form of chat bubbles. This will be superceded by #20 later on.

done
not needed, done in modal
done
done
done
done
done
done with UX flow advice
partially done - help chatbot manual yet to be written, but functionality exists


Ideas to make pickadate.js supports Islamic calendar?

Hi there,
Anyone have ideas to support Islamic calendar
or
Can you support in the next updates?
moment-hijri.js
https://github.com/xsoh/moment-hijri/blob/master/moment-hijri.js
I think it's feature needed by a large segment.
Thank you,

@MajidCS we do have an arabic translation but you can send in a PR for what you're expecting


ViaPolling should only accept functions

It takes channels and functions now. Receive could use a select with a timeout and remove the need to poll a channel.

There is an interesting case where you don't care if you're getting other data down a channel, so long as you (ViaPolling?) eventually see the correct data.  Maybe Receive should have some passable options, a. la. Dave Cheney's constructors talk.  Something like o.Receive(onpar.Equal(foo),  onpar.After(time.Second), onpar.DiscardFailures), using some kind of type ReceiveOption interface?
Either way, we shouldn't just completely ignore the use case where polling the values on a channel is desired.

The current pattern onpar has taken looks more like:
Expect(t, bar).To(ViaPolling(Chain(
  Receive{Duration:time.Second}, Equal(foo),
)))

So this issue would really change into Receive should have a Duration field and then it would not need to be paired with ViaPolling unless the channel is going to receive a bunch of stuff.

Sure, that makes sense.  If ViaPolling still accepts channels (i.e. it's the underlying matcher's job to determine what's supported and what's not), then we're fine.
It's also worth pointing out that matchers like BeCalled actually take structs and could conceivably be polled, too.  But maybe that should be the job of the matcher, since I can see a lot of situations where ViaPolling on structs could easily encourage data races.

I agree. I would prefer to force BeCalled to do something similar to receive.


Rename .cs file -> The namespace 'Namespace' already contains a definition for 'ClassName' [netcoreapp1.0]

Environment data
dotnet --info output:
.NET Command Line Tools (1.0.0-preview2-003121)

Product Information:
 Version:            1.0.0-preview2-003121
 Commit SHA-1 hash:  1e9d529bc5

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14393
 OS Platform: Windows
 RID:         win10-x64

VS Code version: 1.5.3
C# Extension version: 1.4.1
Steps to reproduce

Open dotnet core solution from base solution directory
Open any .cs file
Rename it
Make a minor, valid change to the code so OmniSharp reevaluates it
Observe as errors appear: Class name has The namespace 'Namespace' already contains a definition for 'ClassName' [netcoreapp1.0].  Methods have Type 'ClassName' already defines a member called 'MethodName' with the same parameter types [netcoreapp1.0]

I had this same issue, see https://github.com/OmniSharp/omnisharp-vscode/issues/803#issuecomment-252280404.

Experienced same issue.

Same error occuring for me - rename a .cs file and get the errors mentioned above.  Need to restart VS Code to fix.

@THuffam
Restarting Omnisharp also works. CTRL + SHIFT + P, then 'Restart Omnisharp'. Faster than restarting VSCode at least 😄

@mausworks  Thanks for sharing! it works!

This is fixed with the latest beta v1.13.0-beta4. You can install it by following the instructions here.

This is fixed only in beta? because I am still facing this issue with v1.13.

Yes, this was fixed in 1.13. @rchande?

@ehajri It should be fixed. Can you provide more information about what's happening to you?

@DustinCampbell, @rchande I removed the extension and it appears to be 1.13.1 now.
It seems to be fixed (or maybe a reboot did it, not sure). I'll update this if it reoccurs.

Thanks for the update @ehajri. I've seen some evidence of strange install issues with VS Code 1.18 that seem to "correct" themselves later. Hopefully, that's all this was.

I'm having this problem using C# extension 1.13.1 (and VSCode 1.19 Insiders) in a particular scenario.
Steps

Rename a class using F2 within the buffer.
Rename the filename using F2 in the explorer to match the new class name.

I get the squiggly lines on the classname for the same error.

Odd, but I experienced this today, too. Restructuring my project directory triggered this. Restarting VSCode fixed it.

@johncrisostomo What extension version are you on?

Had the same issue today.
VS Code info
Version: 1.27.2 (user setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64

OmniSharp
Version: 1.16.1.

Workaround
OmniSharp restart helped.
Case

Create a class description.
Use it in another class.
Use F2 to refactor the class name (from the another class).
Go to Explorer in VS Code and rename file with the class description.
Get the error.

i was experiencing this as well, every time i rename a file the type inside it appears to be duplicated
code --version
1.29.1
bc24f98b5f70467bc689abf41cc5550ca637088e
x64
code --list-extensions --show-versions
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>i also changed
"omnisharp.path": null
to
"omnisharp.path": "latest"
i think i was using a default(older) omnisharp version but this didn't solved the issue
now i've
OmniSharp Version = 1.32.9-beta.10
the issue appears ONLY if i work with vscode workspace,
steps to reproduce:

mkdir test
cd test
dotnet new classlib -f netstandard2.0 -n omnitest
code .
from inside vs code
File -> Save Workspace As…
chose where the test folder is
name the workspace "test.code-workspace"
wait for omnisharp to complete to load and process the project
you should see a file "class1.cs" created by the classlib template by dotnet new
select it and rename it "class2.cs"
you'll see that a red squiggle appear below
public class Class1
problems tab will report
{
"resource": "/c:/temp/omnisharp/omnitest/Class2.cs",
"owner": "csharp",
"severity": 8,
"message": "The namespace 'omnitest' already contains a definition for 'Class1' [omnitest]",
"startLineNumber": 5,
"startColumn": 18,
"endLineNumber": 5,
"endColumn": 24
}
as i said if i open the folder with "code . " instead of using the workspace file the issue didn't appear.
it is strictly related to the use of workspaces.
I hope this help to understand where the issue is, it is really annoying to keep restarting omnisharp

@PhotoAtomic Oh, you might be on to something. If I open a single folder, I can rename without issue, but if I then add another folder (if that has any importance, via the side pane Explorer -> Right-Click -> Add Folder to workspace), then error will appear on renames.

@THuffam
Restarting Omnisharp also works. CTRL + SHIFT + P, then 'Restart Omnisharp'. Faster than restarting VSCode at least

This worked for me.

The same issue here, And restart vs code or Omnisharp doesn't work for me. So odd

The same issue here, And restart vs code or Omnisharp doesn't work for me. So odd

Running a VS Code with admin rights "helps".

3 years later, still have to restart omnisharp after a file rename. Supporting these "advanced" use cases must be real hard.

Original bug is still present. Also reproducible on copying and pasting a file containing a class. And as of today, reproducible on adding a new file and adding a new class.
Looks like latest stable:
extensions\ms-dotnettools.csharp-1.23.0.omnisharp\1.36.1\OmniSharp.exe
Windows 10
VS Code 1.48.0
.net core 3.1 project

The same issue.
code --version
1.65.0
b5205cc8eb4fbaa726835538cd82372cc0222d43
x64
code --list-extensions --show-versions
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 21.04
Release:        21.04
Codename:       hirsute

Same issue... getting tired of omnisharp, many years and no solution. Considering transition my development team to Visual Studio.

Still no solution for this? It happens to this very day.
/.vscode/extensions/ms-dotnettools.csharp-1.25.8-linux-x64/.omnisharp/1.39.6-net6.0/OmniSharp.dll

I downgraded C# add-on to v1.24.1, and it is a workaround.


How to add new icons to features.json?

I want to add new icons to features.json. I tried to add some FontAwesome icons in fa6.min.css but it didn't work. Is it possible to add new ones or can you only use the ones already defined?

If you want to use FontAwesome icons, enable them in config.toml and add the following code to features.json.
{
    "title": "Github",
    "description": "Please fork the Github repository and contribute to this theme!",
    "icon": "fa-brands fa-github",
    "link": "https://github.com/opera7133/tella"
}

If you want to use something other than the default defined icons and FontAwesome icons, we recommend editing the theme or using custom.css.

Thanks a lot. It worked. Have a nice day.


Registering 3d images?

I'm trying to register 3d volumes (about 2048x2048x2048 in size, but for now I'm trying simpler cases). However, even this test case seems to fail (you can't see the plots unfortunately, but you can see from the last value that the registration is way off). It seems that in the registered image, the sphere is located at (17, 32, 26). Am I using SimpleElastix incorrectly?

Feel free to reopen if this is still an issue.


Add improvements on table literal syntax

Purpose

Table literal syntax is improved as follows.

table-type-descriptor := table { column-type-descriptor+ }
column-type-descriptor := primarykey? column-name ;
column-name := identifier

Ex1:
table<Person> t1 = table {
	{ primarykey id, primarykey salary, name, age, married }, [
		 {1, 300.5, "jane",  30, true},
		 {2, 302.5, "anne",  23, false},
		 {3, 320.5, "john",  33, true}
	]
};

Ex2:
Person p1 = { id: 1, age: 30, salary: 300.50, name: "jane", married: true };
Person p2 = { id: 1, age: 30, salary: 300.50, name: "jane", married: true };

table<Person> t1 = table {
	{ primarykey id, salary, name, age, married },
	[p1, p2]
};

:white_check_mark: Build ballerina-lang 25 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/35129e1eaf by @anupama-pathirage)

:white_check_mark: Build ballerina-lang 26 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/b8ead7787c by @anupama-pathirage)

:white_check_mark: Build ballerina-lang 36 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/2e00de60d2 by @anupama-pathirage)

:white_check_mark: Build ballerina-lang 251 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/a9d6db48f9 by @anupama-pathirage)

:white_check_mark: Build ballerina-lang 282 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/9b465ae2f3 by @anupama-pathirage)

:white_check_mark: Build ballerina-lang 310 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/3bec6f2671 by @anupama-pathirage)


Please specify which awscli version is required

When download dependencies, please specify which awscli version is required.
In my case, I have already installed to a newer version and it's not compatible with the KFD requirements.
Please, provide an automatic and indipendent way to download awscli when download dependencies

The version check has been addressed in the linked PR, we should also document that the awscli is needed but not managed by furyctl

the aws version check is in place, closing


Request Identity Header

per #584
I also rejiggered some of the header sections to try to make them more consistently in the same spot for each operation entry.

Actually, closing this and going to do #585 first


Release Google.Cloud.Iot.V1 version 1.2.0

Changes in this release:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Created release for Google.Cloud.Iot.V1-1.2.0

Looks like it actually succeeded - the library has been released.


Update to editor settings dll

Overview

Removed some unused methods and cleaned up the class a bit.
Enables shared depth buffer for Windows Mixed Reality in the project by default.

Changes

Fixes: #3158
Fixes: #1939
[x] Fixes: #3259

@wassx I didn't intend to try and fix #3183 with this change, but can you test it for me?

Hi Stephen, I won’t be able to test this.
@davidkline-ms could you or Kurtis test this or have someone follow up?

I'll test today. Thx!

Sry, the issue is still there: 'sharedassets0.assets' is corrupted!

Yep, deleted Library and my build directories, so unity reloaded everything

StabilizationPlaneModifier

We don't have that in vNEXT

Hey @keveleigh do you know anyone who can tell us exactly what the right thing is to do when Shared Depth Buffer is enabled for RS4? Should we disable  StabilizationPlaneModifier or set it's SetStabilizationPlane field to false? Or maybe something else?

StabilizationPlaneModifier

We don't have that in vNEXT

How does the stabilization plane get set in vNEXT? Is RS4 or later required for vNEXT?

RS4 or later required for vNEXT

That is correct. We designed it with the assumption that the stabilization plane will not be modified by the app.

Okay. I can approve the current fix assuming that only RS4+ will be used.

Although I think you're right Jarrod, we should call out that we're setting the stabilization plane in the dialog.
Let me make that change before you approve.

Done :)


Add invulnerabilty destructor effect without ability to kill enemy

This is not the actual commit.


MAHOUT-1949 [WIP] Docker Based ITs

[x] Create Docker Images with Maven of Spark and Hadoop Env
[x] mount project directory in docker
[ ] UPDATE CI ENV to utilize dockers
[ ] A script which runs through an exhaustive list of tests (cli drivers/examples/etc)
[ ] A way to tell weather those tests passed or failed (checking the output?)
[ ] A way to fail the build if if the examples/etc fail. (no idea how this works, I've always tried to make build successful, never tried to fail one).

closed JIRA


sudo make install report errors in Ubuntu

can you retry it looks like cache issue, I pushed the changes explicitly again in #23

should be closed


fix: remove typescript v2 causing flaky tests

Description

Test to generate bindings with unity and typescript can be flaky due to the support of ByteArray that is not fully updated yet.
This PR disable for now one test that will be introduced again once bindings are updated.
The reason seemed to be typescript v2 has it's not currently maintain to follow SDK updates.
Related issue

Tests

[ ] Yes
[x] No, because they aren't needed
[ ] No, because I need help

Added to documentation?

[ ] README.md
[ ] Dojo Book
[x] No documentation needed

Checklist

[x] I've formatted my code (scripts/prettier.sh, scripts/rust_fmt.sh, scripts/cairo_fmt.sh)
[x] I've linted my code (scripts/clippy.sh, scripts/docs.sh)
[x] I've commented my code
[ ] I've requested a review after addressing the comments

To investigate further, not seems to be typescript v2 all the time.


Update README.md

Description:
Added my profile link in Contribiutors section.

🎉 Thanks for your contribution! Your PR has been successfully merged.


added story reporter

I have added story reporter. In our case in IDE we prefer to run stories with scenario reporter (step level) but on CI our preference is to run stories with story reporter. Story reporter can be simply enabled via environment property -Dreport.storyLevel=true.
Also i have replaced custom logger implementation to standard slf4j logger.
I think story level reporter can be useful also for other users from community, so please accept this PR.

Coverage decreased (-8.9%) to 80.603% when pulling 86cacf6bbedf44e25af85d5cb48d9b95e081e850 on mbocek:master into cf36a73000e5b9038fdcbcdaa79668405d000bea on codecentric:master.


What are NCBITaxon_Union:* classes?

In GO Plus (Base) there are plenty of classes like
http://purl.obolibrary.org/obo/NCBITaxon_Union_0000030.
Where would I get these classes from? Which ontology? @balhoff

These should be added to go-base, since GO is is the only place they come from. These terms are similar to GOCHE in that they're a rogue OBO prefix used by GO. You can get a pre-classified version from here: http://purl.obolibrary.org/obo/go/imports/go-taxon-groupings.owl, which is probably fine as long as NCBITaxon doesn't conflict.

@balhoff Is this documented anywhere?


feat(CategoryTheory): Braided one-side Rigid categories are (two-side) Rigid

This proves that Braided one-side Rigid categories are Rigid.

The proof I formalised is at MO.
Please help with naming lemmas and theorems, as well as instructing on where to place these things. In particular, I am not sure whether the "ExactPairing X Y => ExactPairing Y X" part as an instance would cause infinite loops in the typeclass inferences, and also whether the "BraidedCategory + LeftRigidCategory => RigidCategory" would cause typeclass diamonds.
Also this resolves a TODO item in FdRep, namely to prove that FdRep k G is a Rigid category. I will remove the item in another PR, since I am working on finishing them all :) Here is verification code:
import Mathlib.CategoryTheory.Monoidal.Rigid.Braided
import Mathlib.Algebra.Category.FGModuleCat.Basic

open CategoryTheory Category

variable (k : Type*) [Field k]

#synth RigidCategory (FGModuleCat k)

Update: I changed my mind lol

ExactPairing X Y => ExactPairing Y X should be fine.
General with instance loops or diamonds, be bold. Try things out and if you get into trouble retreat. :-)

I renamed exactPairing_braided to exactPairing_swap since the definition is already under CategoryTheory.BraidedCategory namespace, but I still think it's a bad name :P

I'm happy with this now, but you need to git merge origin/master and resolve the merge conflict.

should work now, thanks!

bors merge

bors merge

Thank you!


add pullrequest vim syntax.

See #1916. (discussion starts here)
This should install fine on MacOS and Linux. Windows install steps still need to be added. I don't own a windows machine so any tips there would be appreciated.
One thing I'm noticing right now as I write this is that on MacOS the pullrequest template doesn't render with any whitespace between the title line and the scissors line, so the syntax currently causes the first line of the scissors line to be highlighted with an error group.
I think it would be best to keep that way and instead just put some whitespace in the template if it doesn't already exist.
Let me know if you need anything else. Thanks!

Thank you; this looks great!

@dsifford awesome!
Will this require an update to the homebrew formula for hub to ensure the syntax gets installed? It doesn't appear that the homebrew formula runs install.sh so I think this will need duplicated over there.

Or maybe not? Looks like it symlinked correctly?
$ ls -al /usr/local/share/vim/vimfiles
lrwxr-xr-x  1 jasonkarns  admin    48B Dec 21 09:23 /usr/local/share/vim/vimfiles@ -> ../../Cellar/hub/HEAD-cc38879/share/vim/vimfiles

@jasonkarns The homebrew formula runs make install.


Fix issue if multiple headers are returned

Context
If multiple headers are returned the and CURLOPT_HEADER is set to false the first header will be removed but the response contains all other headers.
What has been done

Removes all headers expect the last one.

How to test

The behavior can be discovered by a curl request which returns multiple headers.

Hello @GM-Alex
Thanks for contributing! Could you please rebase your PR and also provide some tests for this issue?


ci(github): update goreleaser job

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.51%. Comparing base (735ed3f) to head (4d6fe92).
 Consider uploading reports for the commit 890664d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   65.51%   65.51%
=======================================
  Files           1        1
  Lines          29       29
=======================================
  Hits           19       19
  Misses         10       10

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[HOLD for payment 2024-12-25] [$125] Copilot - Selection checkmark is displayed an access level option that is not verified yet

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.57-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail:  N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go Settings > Security > Add copilot
Select a user > Select 'Limited' for access level > Add copilot > Verify code
Click on the added copilot > Change access level > Select 'Full'
Click on RHP back button > Select 'Full' access again
Click on the added copilot again

Expected Result:
On Step 4, checkmark should be displayed on 'Limited' option, and user should be navigated to magic code page on clicking 'Full' option since the access level is Limited as we see it in step 5
Actual Result:
On step 4, checkmark is displayed on 'Full' option, and RHP is closed on clicking 'Full' option
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/60d592e2-5765-4016-a1cd-385744808937
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021856451982020785346
Upwork Job ID:<PHONE_NUMBER>020785346
Last Price Increase: 2024-12-05
Automatic offers:

nyomanjyotisa | Contributor | 105213634

Issue OwnerCurrent Issue Owner: @alitoshmatov

@alitoshmatov we have three proposals for review. Can you please take a look at your earliest convenience? Thanks!

@daledah Thank you for your proposal, please provide some explanations next time. Your code changes is not solving the issue completely. Checkmark is still being updated to Full option

@nyomanjyotisa Thank you for your proposal, Your RCA is correct and solution is working

@alitoshmatov looks like there was a follow-up comment here

@alitoshmatov bump on the comment follow-up above. Thanks!

@youssef-lr I think we can continue with selected proposal, waiting for your approval: https://github.com/Expensify/App/issues/51970#issuecomment-2481317927

Proposal LGTM. Given the very simple change I'll adjust the price.

just dropping an update that the PR is actively being reviewed — great movement so far!

@nyomanjyotisa and @alitoshmatov, I will be OOO starting this afternoon (December 20) and will be returning January 6th. A handful of folks on the BZ team will be online for a few days in between the 25th and the 1st, but we'll be operating with a skeleton crew. I will be issuing my payments when I return on January 6th. If you would like the payment issued sooner, please post this issue in #expensify-open-source and someone on the team will jump in.
Thank you!

BugZero Checklist:

[x] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[x] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[x] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[x] 3d. QA
[ ] 3z. Other:

[x] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: https://github.com/Expensify/App/pull/47984/files#r1899186243

[x] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion: No discussion needed

[x] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

No regression test needed

Looks like automation failed here, completed the checklist. Only handling payment is left


Question: How to run it?

Hi,
I have to build it via $ gulp, that will launch the Chrome Browser, but the VSC debugger get's not attached. So if I start debugging, a second Chrome windows will be launched (and I have to close the first one).
Is this the right way or should I do something else?

I think those steps should do the trick:

Open the generator folder in vscode
Go to your terminal
Type npm install
Type bower install
Run the build task (CTRL + B) and wait until the browser sync has been started (may take a while). This will open your default browser (which explains your first open of chrome).
Check in the debug section, which launch task is selected. It should be firefox, change it to chrome
Press the run button or F5, which will bring up chrome (again) and attach the debugger.

Hope it works, I think I made a typo in some of the examples. Let me know if it does not work. I am currently on vacation and do not have my dev machine with me, so I cannot test it right now.

And the generator is not really a generator, sorry. I started this project, but it is abandoned. Please use the declarations from here or dt. I am currently working on a generator with a modular ES6 syntax, which uses a custom typescript transpiler to convert to ui5 javascript style.

Hi @apazureck. I just saw your July comments on Slack and I'm bringing the matter here just because @github  is the only social that does not suck.

Would be nice to see some coordination on the typescript topic, as there are multiple TS definitions out there and it's getting more and more messy.
I am just trying to use the typescript compiler API to get an ES6/AMD syntax style in ts and native UI5 code.

And I am not sure how they want to do that. Adjust their module system? Imports in Typescript would be handled like import { Button, Table } from "sap/m";, in UI5 you would have to write import * as Button from "sap/m/Button"; import * as Table from "sap/m/Table";.

But that's just cleaner/better syntax. And async and await is just working "perfectly" in TS, as they introduced a ES5 shim in TS version 2.1 or so. That was a real relief.

And going to ES6 syntax without decreasing performance on some places would be really challenging. They would need to provide some wrappers or just put out UI5 2.0. But let's see where this is going...

I frankly don't know what - apart Dummheitkonditioniertweltanschauung - is preventing the Sole contributor At Project  UI5 to spend a couple of days of a few of their resources in creating an automatic emitter of TS bindings from documentation itself.
It would be a marketing winning edge as well, as it could foster adoption in teams where the web is seen as evil.
Nur Gott weiß warum nicht.
--R

Oh I have done that, but I wanted to have a better syntax (ES6 syntax). And you could do much more than that, Generics for models, events, etc.
But I have a guess: SAP is distributing their own dev environment, so they are not interested in anything else, which will not make them money. Why use crappy slow and buggy software (yes, I have used their designer...), if you have a fast and lightweight open source solution? And it's a big, German company, so they tend to have each decision go through multiple committees until it passes. If you have a look at the starting point of some typescript projects, which was around October / November last year, I am not surprised of the lack of attendance.
And I think 90% of the customers using SAP UI are SAP customers anyway and, therefore, have their IDE and stuff and support, etc.
But I think it would be cheaper and easier for SAP to make offers to some devs already on that topic to make them a typescript generator instead of letting someone start from scratch. I think the DT declarations are from some company, which SAP could easily buy, I guess ;)
BTW @SunboX: Did my instructions work?

100% agree. I even ported the crappy slow IDE to work on my Linux machine to show it to developers that ask "Is there an official tool to write UI5 apps?" -- reactions are very funny. Next time that I set up a UI5 intro in some company I'll make a video of the devs seeing it for the first time and post it to youtube :grinning:
Fun (and sad as well) is that interest in UI5+TypeScript started to rise in 2014:
https://github.com/SAP/openui5/issues/88
By that time UI5 had all the aces in its hand to become the framework, since Angular was pestilent (still it is crappy but TS made it win the race, and other frameworks are building on top of it: https://github.com/Teradata/covalent ).
Now, alternatives abound. Personally I'm leaning toward adopting TS  with bare HTML5 + CSS + some components, since all you need in enterprise apps is:

a calendar
a currency formatted input box (still broken many ways in sap.m.*)
datagrids, datagrids, datagrids (sap.m.Table, sap.ui.table.Table really? btw they have different datamodel bindings)
charts and graphs (shooting on UI5 charting features  is as easy as shooting on the Red Cross but hey, it's still 100 points).
security (yes, I found some serious bugs regarding this)

Still I hope one day UI5 will be great again.
--R

Hi,
i tried to use this definitions, because I want to test building ui5 Apps with typescript. I just installed npm and bower and compile tsc -w, but if start index.html in VSCode with F5 I just see an empty page and if I refresh I'm getting following error:

What can I miss? I set index.html at starting point in launch.json:

I think you need run the gulp task instead of manually running tsc. Did you use ctrl+shift+b for the build task? It may not work anymore, as vscode changed the layout. In you out folder is no resources folder, thus the gulp installui5resources did not run.


[feature] default language keys when missing in a split_export_files

Hi and thanks a lot for this great and useful package.
It could be nice to have an option that make generation of split_export_files merging default language keys when they are missing in a language file.
For example with those languages keys in resources/langs :

en [default]

foo
bar

fr

foo

we could navigate in language "fr" and displaying the "en [default]" value in place of the key path.
Also it make Javascript translation works like the Php Laravel side with a default language value when a language missing a key.
I Hope to be clear enough ... I'm not sure :-)

I'am glad using this package, And we also finding a solution for this in our project. Really appreciate if you guys can add this ASAP. Thanks

This package is not under active development anymore, at least not by me.
Feel free to submit a PR implementing the feature you need, though. I will handle the PR, we just need some community work here 😉

@andywer made a pull with a change check that out thanks.
https://github.com/andywer/laravel-js-localization/pull/62


SPEC-4 endorsement for SciPy

The discussion on SciPy's mailing list was opened here<EMAIL_ADDRESS>Lazy consensus applying, that means we are endorsing. I will send another email to SciPy's list with a week's additional RFC.

Thank you all for the thumbs up and Ralf for your message. Let's merge then.


ScottPlot 5: Add xgfs_normal to palettes

Originally posted by @bclehmann in https://github.com/ScottPlot/ScottPlot/pull/2161#pullrequestreview-1135272992

Never mind, ScottPlot 4 palettes are automatically availabe in ScottPlot 5 through some csproj trickery.


Materialize executor unexpectedly compact the UPDATE and DELETE operations from CDC

Describe the bug
Materialize executor unexpectedly compact the UPDATE and DELETE operations of same pk from CDC
Error message/log
The chunk trace of the upstream UPDATE and DELETE operations.

2023-11-09T19:15:45.28921+08:00 DEBUG actor{otel.name="Actor 6" actor_id=6 prev_epoch=5393176590614528 curr_epoch=5393176656216064}:executor{otel.name="DmlExecutor 600000001 (actor 6)" actor_id=6}:executor{otel>
+---+-----+------+-----------------------+
|   | id  | name | description           |
+---+-----+------+-----------------------+
| + | 101 | RW   | Small 2-wheel scooter |
| - | 101 | RW   | Small 2-wheel scooter |
+---+-----+------+-----------------------+
 cardinality=2 capacity=2
2023-11-09T19:15:45.289386+08:00 DEBUG actor{otel.name="Actor 6" actor_id=6 prev_epoch=5393176590614528 curr_epoch=5393176656216064}:executor{otel.name="DmlExecutor 600000001 (actor 6)" actor_id=6}: events::str>
+---+-----+------+-----------------------+
|   | id  | name | description           |
+---+-----+------+-----------------------+
| + | 101 | RW   | Small 2-wheel scooter |
| - | 101 | RW   | Small 2-wheel scooter |
+---+-----+------+-----------------------+
 cardinality=2 capacity=2
2023-11-09T19:15:45.289835+08:00 DEBUG actor{otel.name="Actor 2" actor_id=2 prev_epoch=5393176590614528 curr_epoch=5393176656216064}:executor{otel.name="Materialize 200000003 (actor 2)" actor_id=2}:executor{ote>
+---+--------------------+----------------------+-----------------------------+
|   | mytest.products.id | mytest.products.name | mytest.products.description |
+---+--------------------+----------------------+-----------------------------+
| + | 101                | RW                   | Small 2-wheel scooter       |
| - | 101                | RW                   | Small 2-wheel scooter       |
+---+--------------------+----------------------+-----------------------------+

To Reproduce

create a table in mysql and insert one row

mysql> select * from products;
+-----+---------+-----------------------+
| id  | name    | description           |
+-----+---------+-----------------------+
| 101 | scooter | Small 2-wheel scooter |
+-----+---------+-----------------------+

Ingest the table into RW

create table products ( id INT,
 name STRING,
 description STRING,
 PRIMARY KEY (id)
) with (
 connector = 'mysql-cdc',
 hostname = 'localhost',
 port = '8306',
 username = 'dbz',
 password = '123456',
 database.name = 'mytest',
 table.name = 'products',
 server.id = '5085'
);

Shutdown the cluster
Execute following commands in mysql

UPDATE products SET name='RW' where id=101;
DELETE from products where id=101;

Restart the cluster
Select the table in RW

The expected result is the row should be deleted from RW, but it still exist.
Expected behavior
The expected result is the row should be deleted from RW, but it still exist.
How did you deploy RisingWave?
No response
The version of RisingWave
No response
Additional context
No response

cc @st1page

This is really a severe fault. Chaos unit test might be a good-to-have if code review cannot be enforced.

It is because we use a MaterializeBuffer to compact the input chunk and because the upsert stream does not obey the associative property, the compact operation with the retractable stream rules is illegal. e.g.
INSERT 1, DELETE 1 can be compacted into No-op. but UPSERT 1, DELETE 1 can not because we can not know if the record that should be deleted is inserted by the upsert operation.
I think we should remove the MaterializeBuffer structure but just leave a HashSet to help to get all the keys in the streamChunk and fetch them into the cache. And then we need to do all real operations in the state cache.

closed by https://github.com/risingwavelabs/risingwave/pull/13351


[SPARK-36447][SQL] Avoid inlining non-deterministic With-CTEs

What changes were proposed in this pull request?
This PR fixes an existing correctness issue where a non-deterministic With-CTE can be executed multiple times producing different results, by deferring the inline of With-CTE to after the analysis stage. This fix also provides the future opportunity of performance improvement by executing deterministic With-CTEs only once in some circumstances.
The major changes include:

Added new With-CTE logical nodes: CTERelationDef, CTERelationRef, WithCTE. Each CTERelationDef has a unique ID and the mapping between CTE def and CTE ref is based on IDs rather than names. WithCTE is a resolved version of With, only that: 1) WithCTE is a multi-children logical node so that most logical rules can automatically apply to CTE defs; 2) In the main query and each subquery, there can only be at most one WithCTE, which means nested With-CTEs are combined.
Changed CTESubstitution rule so that if NOT in legacy mode, CTE defs will not be inlined immediately, but rather transformed into a CTERelationRef per reference.
Added new With-CTE rules: 1) ResolveWithCTE - to update CTERelationRefs with resolved output from corresponding CTERelationDefs; 2) InlineCTE - to inline deterministic CTEs or non-deterministic CTEs with only ONE reference; 3) UpdateCTERelationStats - to update stats for CTERelationRefs that are not inlined.
Added a CTE physical planning strategy to plan CTERelationRefs as an independent shuffle with round-robin partitioning so that such CTEs will only be materialized once and different references will later be a shuffle reuse.

A current limitation is that With-CTEs mixed with SQL commands or DMLs will still go through the old inline code path because of our non-standard language specs and not-unified command/DML interfaces.
Why are the changes needed?
This is a correctness issue. Non-deterministic CTEs should produce the same output regardless of how many times it is referenced/used in query, while under the current implementation there is no such guarantee and would lead to incorrect query results.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Added UTs.

Test build #142172 has started for PR 33671 at commit 4f24104.

Test build #142175 has started for PR 33671 at commit 4d08307.

Test build #142172 has finished for PR 33671 at commit 4f24104.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UnresolvedWith(
case class CTERelationDef(child: LogicalPlan, id: Long = CTERelationDef.newId) extends UnaryNode
case class CTERelationRef(
case class WithCTE(plan: LogicalPlan, cteDefs: Seq[CTERelationDef]) extends LogicalPlan

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

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46684/

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

cc @peter-toth @cloud-fan @sigmod

Test build #142176 has started for PR 33671 at commit 7ab352a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46687/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46687/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46688/

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

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46688/

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

Test build #142175 has finished for PR 33671 at commit 4d08307.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #142176 has finished for PR 33671 at commit 7ab352a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Thanks @maryannxue for pinging me. Unfortunately, I can take a closer look at this PR only the week after next...

Never mind, @peter-toth! Hopefully the added logical nodes make sense to you. We can always improve the implementation later on.

Test build #142229 has started for PR 33671 at commit dd4fc28.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46742/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46742/

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

Test build #142277 has started for PR 33671 at commit 2c30d4a.

Test build #142278 has started for PR 33671 at commit 7443fb9.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46786/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46785/

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

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

Test build #142283 has started for PR 33671 at commit 92262a2.

Test build #142277 has finished for PR 33671 at commit 2c30d4a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46790/

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

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46790/

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

Test build #142278 has finished for PR 33671 at commit 7443fb9.

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

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

Test build #142283 has finished for PR 33671 at commit 92262a2.

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

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

Test build #142349 has started for PR 33671 at commit 09c1c27.

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

Test build #142349 has finished for PR 33671 at commit 09c1c27.

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

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

Test build #142386 has finished for PR 33671 at commit 3746d76.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46894/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46894/

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

Test build #142393 has started for PR 33671 at commit 4cc52f7.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46899/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46899/

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

Test build #142393 has finished for PR 33671 at commit 4cc52f7.

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

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

thanks, merging to master/3.2 (since it's a correctness fix)!

Late LGTM. Just a very minor comment.

@maryannxue  I find after this change, the eventually expanded query now contains duplicated expression Ids.
is this expected? I am thinking of the assumption expression id would be unique globally.


Refactored and updated doclet, Included documentation and sample project

Performed some refactoring to better organize the code.
Updated and included missing javadocs.
Enabled the doclet to accept default command line arguments given to the javadoc tool (initially only the -d, that defines the output directory)
Provided a complete README that describes the doclet, its goal and how to use it by directly calling the javadoc tool or through maven.
Included a pre-configured sample maven project.

Fantastic @manoelcampos, ill review soon.

Pretty good stuff.  Love the new example project.

I'm wondering if it's possible to used asciidoc include tag inside the README.
I tried but it doesn't work and I haven't found any workaround.

Further, I don't why it ignores the outputDirectory tag inside the pom.xml. Giving a different directory just works with the snippet below:
<additionalparam>-d "${project.build.directory}/site/asciidocs"</additionalparam>

I think Asciidoctor on github is limited, so imports may not work within a project.

I think it's done. If there is anything else, please let me know.

Fantastic work @manoelcampos, thanks again!


How should I use trained weights to infer my own images

Hello, now that I have completed setting up the environment and downloading the training weights, I would like to infer my own images without configuring the validation and training sets. What should I do.

follow the huggingface demo with App_main.py


(question) Can I create a drawer with navio?

Hey guys. Awesome library!
I was wondering whether there's a way right now to create a drawer navigator with navio. And if not, if theres a suggested way to integrate navio with @react-navigation/drawer to accomplish that
Thanks!

Hi @lakardion! Thank you!
Unfortunately, Navio doesn't provide a drawer functionality at the moment :( but as the library is at the very beginning stage, many functionality will be added. And drawer (@react-navigation/drawer) is actually the first on the todo list 😄 Could you share the details of your use case of a drawer component? Because I've used it a few times only (as I think it's not good UX), so more examples would be helpful
That's the way I'm thinking to build it:
const navio = Navio.build({
  screens: {...},
  stacks: {...},
  tabs: {...},
  drawer: {
    stack: <one from stacks>,
    content?: <custom drawer content>,
    options?: <drawer options>,
    // maybe something else
  },
})

I think it should be enough to achieve at least basic drawer functionality. Also, drawer navigator can be inside other navigators, so this case also should be handled.
Then, I am keeping the issue open until drawer functionality is not done.

Cool, I was also hoping for the drawer to be implemented.  My use case is that I want to add a Hamburger menu upper left, and be able to open a left navigation menu, and switch to different stacks easily.  What you have drawn up in your pseudo-code would suit my needs nicely.  Thanks!

Hi @seanadkinson! Thanks for the use case. Then, I will implement as the pseudo-code states and it would be great if you could test it once it's implemented. I'm just very busy last months with the project but will dedicate some time for open source very soon.

Hey @lakardion @seanadkinson! Drawer support has been added in v0.0.2 and added to readme. Let me know how it works in your cases!


Support relative paths for embedded images

GitHub lets you use relative links when referring to images (and any other files) in the repo, e.g.
![Company logo](distribution/media/logo.png)

It would be nice if quickmd supported this. Ideally it would look for the file starting at the root of the current Git repo, but for a start using the directory of the current file would be good enough.
I can try opening a PR for this if it's not too hard to implement.

Currently, it's possible to render a file-relative image using ![Company logo](./distribution/media/logo.png) -- that gets parsed right here: https://github.com/AndrewRadev/quickmd/blob/7c1f2ee71d64f3b3a6461cff249fc8f5ba79043a/src/markdown.rs#L55-L57
I'd be happy to implement rendering your example as well. In fact, I think I considered it, but I wasn't sure if it was a file path or a domain name. Now that I'm testing, though, it seems it only parses URLs if they start with http:// or https://, so maybe it'd be quite simple to get this done by changing the .starts_with? with a check for not starting with http:// or with https://. Or maybe even .\w+:// for stuff like ftp:// or file://.

Ideally it would look for the file starting at the root of the current Git repo,

According to this doc, I think repo-root-relative files start with /? https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#images
This would also be fine to implement, and pretty useful -- the code could check for a .git file or directory upwards, and if it exists, check if that file exists. But maybe both of these could be done separately.

I can try opening a PR for this if it's not too hard to implement.

I think it should be relatively easy to use the regex crate to check that the "url" in that code doesn't start with a protocol and ://. There's also a test that would need to be tweaked and maybe extended: https://github.com/AndrewRadev/quickmd/blob/7c1f2ee71d64f3b3a6461cff249fc8f5ba79043a/tests/test_markdown.rs#L32-L50
If you don't have the time/energy or Rust experience, I can take care of it as well -- let me know :+1:.


Deprecate exclusivePrefixSum, accumulate, adjacentDiffernce, iota and provide KokkosExt ones

ArborX::exclusivePrefixSum -> ArborX::Details::KokkosExt::exclusive_scan
ArborX::accumulate -> ArborX::Details::KokkosExt::reduce
ArborX::adjacentDifference -> ArborX::Details::KokkosExt::adjacent_difference
ArborX::iota -> ArborX::Details::KokkosExt::iota
Things I'm not sure about:

Should be do KokkosExt::StdAlgorithms namespace?
Should iota be in StdAlgorithms?

The goal of this and #998 is that ArborX_DetailsUtils.hpp is pretty clean post removal of deprecations, and nothing in ArborX:: namespace.

Why did you not introduce the initial value trailing argument with exclusive_scan if the intent is to align with standard parallel algorithms?

Right now, the only thing I've done is moving things around. I haven't done the alignment with the standard algorithms in this PR.

Things I'm not sure about:

Should be do KokkosExt::StdAlgorithms namespace?
Should iota be in StdAlgorithms?

I don't think either of these are necessary (but it's easy to change since we don't care about backward-compatibility in that namespace).


Consider UIAppFonts in info.plist

I think starting in ios3.2 you can use the UIAppFonts info.plist value. I.e.

Applications that want to use custom fonts can now include those fonts in their application bundle and register those fonts with the system by including the UIAppFonts key in their Info.plist file. The value of this key is an array of strings identifying the font files in the application’s bundle. When the system sees the key, it loads the specified fonts and makes them available to the application.

although the ios 4.0 (and 3.2) release notes mention

CGFontCreateWithFontName can hang in some circumstances when using the UIAppFonts key in the Info.plist.

But because this isn't explicitly  used (you're using CTFont...), you can probably get away with the easier UIAppFonts fix.

Good idea. Done!


Candidate for 0.4

Contains multiple fixes and enhancements, like updates to the UNs and the SafeConnect code.

There has been a lot of changes and fixes so I would like to make a new release of attestation.jar
... Although I would like to wait with pushing the release until after DevCon of course :)

Note that blockchain integration tests have been disabled to ensure that tests works, until the integration tests get updated to use current test-net


Docker Environment to Quickstart

There should be a docker setup to quickstart the testing. Any developer/tester just have to pull the docker, mount the Karate scripts and run the tests. This approach will help to run tests on clusters, hence finishing the tests faster.
I would like to contribute this feature.

closing due to inactivity


Cancel deployments of the same block in case the block finished with FAILURE or CANCELLED status

What this PR does / why we need it:
In case some running deployments are in the same block with a FAILURE or CANCELLED deployment, we should stop those deployments too. By this PR, I added a method to check whether a deployment should be stopped or not when its piped sync the on processing deployment status via ReportDeploymentPlanned and ReportDeploymentStatusChanged, at that point of time, deployment is still on processing and if the current block which contains that deployment finished with FAILURE | CANCELLED status, we will stop the current deployment too by sending CANCEL_DEPLOYMENT command so that the handling piped will be aware and stop its deployment.
Note: I don't use the same method (add cancel flag to those gRPC responses) as we do with InChainDeploymentPlannable since

This time, the gRPC which be updated is called once, not intervally as check plannable RPC.
I want to keep the deployment handling mechanism on piped side simple (reuse the cancelling by command flow in planner and scheduler codebase)

Which issue(s) this PR fixes:
Fixes #2888
Does this PR introduce a user-facing change?:

NONE

This way is only checking when the deployment has just been transited to a new status so the cancellation could be delayed.
But I think that is fine at this time.
We can invest to have a more precise way in the future.
/lgtm

Nice. Keep it up.
/approve


messagelist: new app

Displays messages inline as a single list.
Still plenty of room for improvement, but I think it's good enough for general release now.
Maybe hold off merging this until the app loader is smart enough to detect+handle messagegui conflicts? (forum discussion) I'm in no hurry to get this merged, and you'll be the one getting the bug reports 😉

:) Thanks! Yes, I'll try and get that added to the app loader this week sometime

Whoops, I also just realized I haven't properly tested uiRemove. As in: fast loading seems to work fine, but I haven't checked for memory leaks yet, and might have forgotten some timeouts.
(This is as much a reminder to myself, as a warning that you might want to wait a bit with merging this ;-)

Ok, I've just added the code to check for clashes to the development loader - if you get a second please could you check it works ok for messagelist?

The loader seems to work fine, thanks!
messagelist is leaking memory though :-(

Couldn't figure out the memory leak yet, so I converted it to draft.
I'll have another look tomorrow, if I still can't find it I'll just disable the fast clock loading for now.

Great - thanks!


Formatting if optional question is empty

Describe the bug
If you skip through an optional question in one program, the "previously answered" text in the next program on the summary view is on top of the edit button.
To Reproduce
Steps to reproduce the behavior:

Create 2 programs with the same optional question
(On desktop) apply to the first program and skip through the optional question without answering it
Click apply to another program
Apply to the program with the same optional question
When getting to the main page see formatting issue (also not sure if "last answered" should even show if its empty)

When clicking continue, the optional question is navigated to again which is expected, so the potential issue is only isolated to the initial review screen.
Expected behavior
"Last Answered" text should not be over edit and maybe shouldn't show when question wasn't answered?

Spoke to the team and the intended behavior is for "Last answered" not to show when the question hasn't been answered (even if its been seen)


Git Flow pull of main hotfix into develop

Git Flow pull of main hotfix into develop

I'll close this as #585 is the only one that'll work (yay weird conflicts!)


Vanilla CMake packages - correct CMAKE_PREFIX_PATH in dependencies

I am trying to use colcon with vanilla CMake (without cmake-ament and ROS).
In a minimal example I have two packages, where package B contains an executable "app" which depends on "lib_a" from package A:
ws
|-package_a
|--include/lib_a.h
|--src/lib_a.cc
|--CMakeLists.txt
|--colcon.pkg
|
|-package_b
|--src/app.cc    (uses lib_a)
|--CMakeLists.txt    (with FindPackage(LibA REQUIRED))
|--colcon.pkg    (with "build-dependencies": ["package_a"])
CMakeLists.txt from "package_a" exports and installs a target "pkg_a::lib_a" in share/cmake/LibAConfig.cmake and installs the include folder and the lib.
When I build and install "package_a" manually with CMake in the first step and in a second step build "package_b" manually with CMake passing CMAKE_PREFIX_PATH to LibAConfig.cmake from step 1 everything works like expected.
I tried to build both packages automatically with colcon but failed.
The dependency graph (colcon graph) looks correct, but apparently when colcon calls CMake to configure and build package_b, it doesn't pass the CMAKE_PREFIX_PATH pointing to install/package_a/share/cmake.
The install folder after calling colcon build looks like this:
ws
|-install
|--package_a
|---include
|----lib_a.h
|---lib
|----liblib_a.a
|---share
|----package_a
|-----hock
|------...
|-----package.bash
|-----...
|----cmake
|-----LibAConfig.cmake
|----colcon-core
|------packages
|-------package_a
|--local_setup.bash
|--....
From the documentation it is not clear how I can tell colcon to add the correct paths to the CMAKE_PREFIX_PATH from its dependencies.
Does colcon crawl the folders for "*Config.cmake" on itself files or do I have to give them explicitly somewhere?

when colcon calls CMake to configure and build package_b, it doesn't pass the CMAKE_PREFIX_PATH pointing to install/package_a/share/cmake.

colcon prepends the "install prefix" of package_a when downstream packages like package_b are processed. That install prefix is install/package_a. You can double check that by outputting the environment variable in the CMake code of package_b.
The CMake documentation describes in which subdirectories of the provided prefix paths it searches for CMake config files. share/cmake is not one of them.

Thanks a lot. That solved my problem.


Question: Aliasing weave IP on host

First, sorry to ask a question in your issues, I looked on the web site and did not find a link to a mailing list. I'm using weave and have a case where I have a non-Docker process and I want to allocate an IP for it on the network and assign it a DNS entry. The following worked:
ifconfig weave:0 inet <weave_ip_plus_one> netmask <netmask_of_weave_if> broadcast <broadcast_of_weave_ip> mtu 1376
weave dns-add <ip> -h <fqdn>
With that I am able to ssh to this host from another host on the weave network. I tried several things using the weave cli but was unable to just allocate an IP without a process. Will this work in the long term? Is this the "correct" way to do this?

Sounds quite like what we call "service import"

@bboreham - Thanks for the response. Re-reading that section makes a little more sense now. I have a good number of http-based services that are not in Docker that our docker applications call out to. The example in the 'import services' section seems like I would still have all of my services bound to the host weave IP, but on a different port. I'm attempting to get out of having to maintain a bunch of port mappings (serviceA -> port1, serviceB -> port2, etc) and instead just use DNS addresses (and the default port - 80) for the different services. I think if WeaveDNS supported SRV addresses that might solve my issue. Absent that, I came up with the solution above to:

create a virtual IP on the weave network
have my service bind to that virtual IP
register that IP in WeaveDNS with a service name
This seems to work, but I'm looking for some confirmation that the virtual IPs defined on the host won't break weave.

Generally Weave Net works at Layer 2, i.e. it doesn't even know what you are doing with IP addresses.
So I can't think of anything immediately that would break, but it's impossible to be sure without trying it.
Do let us know how you get on.


JavaDoc update for MvpBasePresenter

Please have a look @dimsuz #43
I have also added this to FAQ section
http://hannesdorfmann.com/mosby/summary/

Nice and detailed explanation, thank you!
I find that documentation is Mosby is wonderful and having it helps a lot!

Thanks!


Improve docker CI: push vX.Y tag (without patch) to DockerHub (for v0.28.0)

Bringing a commit from main (5ae5b06) into release-v0.28.0 to have this change already applied for v0.28.0
Following the one merged on main: https://github.com/meilisearch/meilisearch/pull/2521

bors merge


cli/release: adding 'release update' to allow tweaking existing releases

Signed-off-by: Nathaniel Eliot<EMAIL_ADDRESS>Fixes #1535

The commit message prefix should be cli: not cli/release:
The code needs a gofmt -s -w.

https://ci.flynn.io/builds/20160120020445-6ffdea3e failed for no obvious reason: I suspect something is flaky with CI again. Trying again, but my luck has been poor thus far . . .

https://ci.flynn.io/builds/20160120043412-81d544ea didn't fare any better. Giving up for tonight.

https://ci.flynn.io/builds/20160120145313-b0fee0bc also bombed, for similar reasons that don't appear to have anything to do with this. I'm done trying until those have been resolved.

Added tests: https://ci.flynn.io/builds/20160121224149-7a787677

Well, it builds cleanly on my local machine, at any rate. @titanous - let me know if you need any further revisions.

Okay, code should be complete, and all tests pass locally. Builds keep failing for unrelated reasons. Let me know if more tweaks are needed.

LGTM


Extend Azure pipeline template with release pipelines and release deployments

This PR is adding additional capabilities for the Azure template:

Deployment to controlled test environments
Leverage Azure environments to document which build is deployed in which environment
Automated computation of release tags, and automated tagging of release candidates and the production release

@M-DLB , please see the updates in https://github.com/IBM/dbb/pull/208/commits/cd1898f420c94a4aa42a536eb0082d3646253256


Enhanced logic to prevent analytics failure when manifests not found

LGTM. Can merge.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f551f20). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage          ?   57.31%
  Complexity        ?       43
=========================================
  Files             ?        9
  Lines             ?      335
  Branches          ?       45
=========================================
  Hits              ?      192
  Misses            ?      108
  Partials          ?       35

Impacted Files
Coverage Δ
Complexity Δ

...jenkins/plugins/bayesian/BayesianAnalysisStep.java
61.7% <0%> (ø)
6 <0> (?)

Continue to review full report at Codecov.

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

@yzainee Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-21


CI input tool should also send text

After selecting a chant using the CI input tool, the edit form should get not only the selected Cantus ID, but also the full text from CantusIndex. Currently this is left blank, so you have to copy it from elsewhere.

Huh - I'd expect it to populate the genre etc. as well. Something is broken here, surely!


Bug: Download functionality  not working on bulk data files page

Test Case
Tester: Maegan Wood
Test Case: TC800
Steps to Reproduce
Download not working on the bulk data files page

Navigate to the bulk data files page
Select a file
The download functionality remains disabled and a user can not download a file

Dev Notes

Will need to go in after Camel Casing is fixed and EDR & XML files are migrated from SFTP to S3 to ensure that nothing broken on the UI

Actual Results
Requirements
User Story
Acceptance Criteria

Filename and File size has been fixed but the download needs more development

@kstetsyuk TC800


Ft User should be able to start/create a new game 157801295

What does this PR do?
Implements User should be able to start or create a game
Description of Task to be completed?

Add a modal that prevents user from getting into the game screen immediately
Add a new endpoint /api/game/:id/start to create a game and /api/game/save to save the game
Create a game model
Connect angular to consume the newly created api endpoints

How should this be manually tested?

A modal should pop up when the user clicks on play game
When the game is started a new game should be created in the database
At the end of the game the winner and players of the game are recorded

Any background context you want to provide?
What are the relevant pivotal tracker stories?
157801295
Screenshots (if appropriate)
Questions:
none

Pull Request Test Coverage Report for Build 82

13 of 40 (32.5%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 51.213%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/models/game.js
6
7
85.71%

app/controllers/users.js
0
2
0.0%

app/controllers/game.js
4
28
14.29%

Totals

Change from base Build 63:
-1.3%

Covered Lines:
527

Relevant Lines:
867

💛  - Coveralls


added compact benchmarking scripts

Codecov Report

Merging #28 (9d9e035) into main (4c6fa51) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
- Coverage   96.58%   96.52%   -0.06%
==========================================
  Files          15       15
  Lines        1784     1784
==========================================
- Hits         1723     1722       -1
- Misses         61       62       +1

Impacted Files
Coverage Δ

src/rcache/__init__.py
89.83% <0.00%> (-0.17%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Fix docstring in display() function


Bug when click "View Explorer" in testnet

When i click to view a NFT in testnet it always redirect to mainet

Hey, where did you click at? Suiet wallet? Or other website? I assume this issue isn't related to our wallet kit.


Resolving "realm-cocoa"

I use the tool swift package that comes with xcode to integrate, but it gets stuck in Resolving "realm-cocoa". How can I solve this?

That step is where spm downloads the repository from github.


The search for constructs does not work

Describe the bug
The search for constructs does not seem to work.
To Reproduce
Go to tab Search and enter the fully-qualified name of a Java class that you know is present in one of the application's dependencies.
Expected behavior
When hitting the button Search constructs, the corresponding archive (dependency) should show up.
Vulas version
3.0.17 (4e5e240c6f04dc7054343cdcf854c033e56d43a4)

Right now, the frontend receives a 404 when the following is called:
https://<host>/apps/undefined/apps/<group>/<artifact>/<version>/search?searchString=org.apache.struts.action.ActionServlet.initOther()&wildcardSearch=false
The correct call would be https://<host>/backend/apps/<group>/<artifact>/<version>/search?searchString=org.apache.struts.action.ActionServlet.initOther()&wildcardSearch=false, with the space token specified in X-Vulas-Space header.

Closed by https://github.com/SAP/vulnerability-assessment-tool/pull/192


Consider gluing extra bytes onto fuzz_skrifa_outline corpus to use to populate OutlineRequest

Ref #973. Currently we just take a cross of likely options, the fuzzer can't control the specific options via the incoming data.
Naively it would be nice if the fuzz_target could have multiple inputs, such as corpus + additional Arbitrary (https://github.com/rust-fuzz/libfuzzer/issues/119).

Per @garretrieger HarfBuzz just takes the end of the file as options. This probably also works, relying on the fuzzer to generate extra bytes. My inclination is to glue more on regardless, just to minimize interference with the bytes that are likely to "look like" a font.


Whatsapp template flow

Library name and version
Azure.Communication.Messages 1.1.0
Query/Question
Hello team,
How can I implement WhatsApp template flow with the WhatsAppMessage implementation SDK?
WhatsApp recently introduced some interesting forms within its application. https://developers.facebook.com/docs/whatsapp/flows/gettingstarted
I want to know how I can implement these using the existing SDK or how I could use the SDK’s connection to call WhatsApp’s API directly and implement it.
I don’t want to stop using the Azure SDK since our code is implemented with these libraries.
Also, is it possible for me to gain access to these libraries? I could make the necessary changes. This is something that interests me and is a WIN2WIN!
Thanks in advance
Environment
No response

This is and example of the current response.

This means that if I can’t get the call to work using the SDK, I’ll have to stop using it. However, the idea is to keep using it, as it is extremely useful.
I reiterate my willingness to make the change myself.

@CJavierSaldana, Thank you for your valuable suggestions and feedback. We are currently working on incorporating this feature, along with other new features introduced by WhatsApp, into our server-side application. Once deployed, we will update our SDK to support these new features as well. While we do not have a specific release date for this update, we are tentatively targeting the first quarter of 2025.

Hi @arifibrahim4. Thank you for letting me know that the team are working on that.
Is it possible to use the same connection provided by Azure and WhatsApp to directly access the WhatsApp API?
My intention is to leverage our existing clients’ connection without needing to disconnect from Azure.


More tolerance for lower case responses

Allows the user to type the DYM option in lowercase.

Thanks @HenryJammes - merged now!


[Feature] need to collect FAQ issues to official website faq.md docs

there are some FAQ question on the issue list, and need to be collected to official website
the steps:
1、get the faq list:  https://github.com/apache/incubator-dolphinscheduler/issues?q=is%3Aissue+label%3AFAQ+
2、then tidying up the doc to faq.md on the website: https://github.com/apache/incubator-dolphinscheduler-website , last , submit a PR

I will finish this.


QUIC client fuzzer hang

OSSfuzz have found a new hang in the QUIC client fuzzer. Fuzzer file attached.
clusterfuzz-testcase-minimized-quic-client-6245605980372992.gz

It doesn't hang, it's just really slow.

This is related to PING frames and congestion control again.
We have determined that we need to send a PING frame but congestion control is preventing us from doing so. So we keep going around in a loop trying to send one. Each time around we do update the ping_deadline - but we only increment it by 1ms. This is because the peer has told us to use a max_idle_timeout of 2ms:
https://github.com/openssl/openssl/blob/09298141592c579504966f1907a44cb95f37cc6e/ssl/quic/quic_channel.c#L3443-L3454
Because of the excessively small max_idle_timeout value you might expect that the very small increments in ping_deadline wouldn't matter because we would also close the connection due to an idle timeout very quickly. But actually we ignore excessively small idle timeouts requested by the peer:
https://github.com/openssl/openssl/blob/09298141592c579504966f1907a44cb95f37cc6e/ssl/quic/quic_channel.c#L3421-L3432

Possible fix in #22533

Fix was merged.


Demo throws

FYI your live demo throws errors:

audio_renderer.js:42 DOMException: Unsupported configuration. Check isConfigSupported() prior to calling configure().
error @ audio_renderer.js:42
simple_video_player.html:1 Uncaught DOMException: Failed to execute 'configure' on 'VideoDecoder': H.264 decoding is not supported.
    at VideoRenderer.initialize (https://wc-talk.netlify.app/video_renderer.js:37:18)
    at async Promise.all (index 1)
    at async https://wc-talk.netlify.app/simple_video_player.html:43:3
audio_renderer.js:211 Uncaught (in promise) DOMException: Failed to execute 'decode' on 'AudioDecoder': Cannot call 'decode' on a closed codec.
    at AudioRenderer.fillDataBufferInternal (https://wc-talk.netlify.app/audio_renderer.js:211:20)
    at async AudioRenderer.fillDataBuffer (https://wc-talk.netlify.app/audio_renderer.js:166:5)

Just to confirm, sounds like you're using a chromium build that doesn't support H264? Agree, this could be handled in a more user-friendly way (probably just showing a "can't play" type message on  the page).

Yes. Or decode the H264 (by any means, on all OS's). Both Chromium and Firefox can play Matroska files with Opus and VP8, VP9 - though only when the MIME type or file extension is changed, e.g., https://bugzilla.mozilla.org/show_bug.cgi?id=1422891. User and developer options can be improved re media codec and container handling.

How does netlify.app get involved with GitHub pages deployment?
Makes it difficult to test when the branch deployed does not get updated when changes are made.
Should probably do something like
if (!AudioDecoder.isConfigSupported(config)) {
  // reject Promise
}

Right now
console.assert(AudioDecoder.isConfigSupported(config));
is useless when the next line is
this.decoder.configure(config);
anyway, irrespective of the result of the previous line.


Add testing

Add testing in preparation for external contributions

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c2afd77). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #7   +/-   ##
=========================================
  Coverage          ?   97.14%
=========================================
  Files             ?        1
  Lines             ?       35
  Branches          ?        7
=========================================
  Hits              ?       34
  Misses            ?        1
  Partials          ?        0

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


fix: update dashboard

I don't see any changes from commit beeaf3a in the main branch

I got a warning in console about a list not having a key. I think it has to do with grade.tsx:41?

@fvcci The key prop thing we can add to bug fix list. I'm not sure what happened to the commit you mentioned, but since its related to application submissions which are now closes, is that a priority to fix in this PR or can we delay that?

@fvcci The key prop thing we can add to bug fix list. I'm not sure what happened to the commit you mentioned, but since its related to application submissions which are now closes, is that a priority to fix in this PR or can we delay that?

what is a quote reply 😩. Since applications are closed I suppose it's not that necessary? I'll add them as issues on linear. One outstanding issue though would be disabling the discord button from dashboard.

Does this PR not do that?


[Compute] vm create: New parameter --count

Description

az vm create --count, create multiple VMs in one command.
Testing Guide

az vm create -g fy -n vm --image centos --count 3

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

/azp run

/azp run

/azp run