Redundant separate "CREATE UNIQUE INDEX" for PRIMARY KEY

Description
The ddl generate a redundant separate "CREATE UNIQUE INDEX" for PRIMARY KEY.

DBeaver Version
Community <IP_ADDRESS>307091549
Operating System
windows 10
Database and driver
database: oracle <IP_ADDRESS>.0
driver: com.oracle.database.jdbc:ojdbc8:RELEASE <IP_ADDRESS>
java.version=17.0.6
Steps to reproduce
No response
Additional context
No response

Thank you for bug report.


Strange character appears after the line number

I'm using terminal Vim with iTerm, and just updated to the macOS Sierra. Then I found there's a strange character appears after the line number and before the colon. I can't find anywhere I can customise this thing. I'm using the theme 'molokai' and here's a screenshot.

you will find an explanation in the help.


Unable to locate element on Non angular page with Browser.driver

This way works on a different non-angular page for me before without any wait or sleep.
Basically I have some code looks like this:
            var driver=browser.driver;
            // maximize window
            driver.manage().window().maximize();
	// log in
	browser.get('https://www.qa.dealertrack.com/default1.aspx');
	driver.findElement(by.name('username')).sendKeys(id);
	driver.findElement(by.name('password')).sendKeys(password);
	driver.findElement(by.name('login')).click();

	browser.driver.wait(function() {
						return browser.driver.getCurrentUrl().then(function(url) {
											expect(url).toEqual(
													'https://www.qa.dealertrack.com/default1.aspx');
											console.log('Successfully loged into '
															+ url);
											return url;
										});
					}, timeOut,
					'Expectation error: Timed out waiting for current url');
             driver.findElement(by.id('AppSub')).click();

Element in the last line could not be found. I double checked and I am sure the locator is not wrong.
I have used some way to test other non angular page, everything went smoothly. I don't understand why it does not work on this site.

Please direct general support questions like this one to an appropriate support channel, see https://github.com/angular/protractor/blob/master/CONTRIBUTING.md#questions
Thank you!


Styles Fair search bar and adds search svg

Creates SearchIcon from SVG
Styles Fair SearchLink component

#skip_new_tests

Yikes, snapshot conflicts. Might be easiest just to rm those files and re-record.

@l2succes good now


Update build instructions

Update build instructions
As of #278 CMake was introduced as a method for building Dictu. The README currently only describes how to build using the older makefile, we should extend the README (as well as the site documentation) to show how to build Dictu with CMake as well.

(All terminal commands are shown assuming the current directory is Dictu's.)
Author's Note
None of what I wrote here is intended to be put into the Readme or documentation, rather it is meant to be an accessible demonstration of the CMake CLI in the context of Dictu for those who wish to write the documentation. A lot of what is written here is written in more detail than is necessary.
This also ended up being way longer than I intended it to be. Sorry about that.
Prologue
Building using CMake can be somewhat odd due to the fact that CMake is not a build file but rather a build file generator. This means its a two steps process: generate the build file and then use the build file. It is also further complicated by the fact that the CLI is not the only way to use CMake. On Windows, there is also CMake-GUI. Many IDEs also have support for CMake, with interaction methods differing from the CLI. Personally, I use VSCode with CMake-Tools. However, for simplicities sake I will just focus on the CLI.
Generating the build file
Basic Building
The first step is generating build files. To do so is fairly simple:
cmake ./

However, that clutters the main folder. If you want to send the new files into a build folder you can do this:
cmake -S ./ -B ./build

I wholeheartedly recommend doing it this way. Depending of the chosen generator, there may be numerous files and folders which are generated. It also makes it more difficult to remove the generated files if needed. Rather than just deleting the build folder, you have to pick and choose individual files.
Options
To disable HTTPS and Linenoise, you do this:
<Previously mentioned setup> -D DISABLE_HTTP=TRUE -D DISABLE_LINENOISE=TRUE

If you want to ensure it is compiled as debug:
<Previously mentioned setup> -D CMAKE_BUILD_TYPE=DEBUG

Or if you want it to be release:
<Previously mentioned setup> -D CMAKE_BUILD_TYPE=RELEASE

Generators
If you want to specify which generator file is generated, you can do so like so:
<Previously mentioned setup> -G "<generator>"

Where <generator> is a valid generator listed here: https://cmake.org/cmake/help/latest/manual/cmake-generators.7.html#manual:cmake-generators(7)
CMakeCache
When CMake generates a build file, it also generates a cache in the build directory. The cache saves the above settings to allow easier building for future builds.
cmake -D DISABLE_HTTP=TRUE -D DISABLE_LINENOISE=TRUE -D CMAKE_BUILD_TYPE=DEBUG -G "MinGW Makefiles" -S ./ -B ./build
cmake -S ./ -B ./build

If the first command is run before the second command, the two commands are equivalent. It also allows for changing of individual variables:
cmake -D DISABLE_HTTP=TRUE -D DISABLE_LINENOISE=TRUE -D CMAKE_BUILD_TYPE=DEBUG -G "MinGW Makefiles" -S ./ -B ./build
cmake -D CMAKE_BUILD_TYPE=RELEASE -S ./ -B ./build

This first builds a debug build and then a release build. Although I didn't specify the most of the variables in the second call, they remain the same as what they were in the first call.
CMake caches also allow simpler directing of the CMake program. Rather than specifying the source directory and the build directory, you can simply specify the build directory.
cmake -D DISABLE_HTTP=TRUE -D DISABLE_LINENOISE=TRUE -D CMAKE_BUILD_TYPE=DEBUG -G "MinGW Makefiles" -S ./ -B ./build
cmake -D CMAKE_BUILD_TYPE=RELEASE ./build

I specify both in the first call, but only specify the build directory in the second.
There is no way to directly remove the Cache, however if you sent the build files to their own directory you can simply delete the directory.
Building the build file
After you do that, you build the new build files. You can do so manually (For instance, if you generated a makefile, you can manually use make). Or, you can let CMake automatically dispatch to the correct program. If you sent the build files into the same directory:
cmake --build ./

If you sent it to the build directory:
cmake --build ./build

It is important to note that the executable may be in different places based upon which generator you used. When I use Visual Studio, the executable is in ./build/Debug/Dictu.exe for debug builds. When I use MinGW-Makefiles it goes to ./build/Dictu.exe no matter what. If you are unsure of where the executable may have ended up, I recommend looking around the build folder for it.
Epilogue
While I tend to use CMake-Tools rather than the CLI, my setup roughly matches this:
cmake -D DISABLE_HTTP=TRUE -D DISABLE_LINENOISE=TRUE -D CMAKE_BUILD_TYPE=<build type> -G "MinGW Makefiles" -S ./ -B ./build
cmake --build ./build

After the first call to CMake, due to the aforementioned Cmake caches, it would be more similar to this:
cmake -D CMAKE_BUILD_TYPE=<build type> ./build
cmake --build ./build

Or even this if the build type is the same as last time:
cmake ./build
cmake --build ./build

Author's Note Part 2
After the first build it is possible to mix the generation and the building into one command. Lets say we have generated release version and built it, but now we want to build a debug version. We can run this:
cmake --build ./build --config CMAKE_BUILD_TYPE=DEBUG

However, this can be problematic. If a new file is added to the source directory, this command will error.
Let's say I have added two files: header.h and header.c. header.h defines foo and header.c implements foo. If I include header.h in main.c and call foo and then run the above command, I get an error stating foo was undefined. It works fine if I instead run this:
cmake ./build -D CMAKE_BUILD_TYPE=DEBUG
cmake --build ./build

The usage of glob in the CMakeLists requires CMake to recheck all source files whenever the generator file is regenerated, however the mixed version of the command does not do this check. If we ever move away from glob and to manual defining of compiled files (See Author's Note Part 3), this will be less of an issue. It may still give odd error messages if you forget to modify the CMakeLists, but after modifying the CMakeLists, you can run the mixed command without concern.
It is possible to use the smaller version for when you haven't added or removed any files and then use the two separate commands for when you have. But if you do mention the combined version, it is also important to mention that it doesn't always work.
Author's Note Part 3
This Author's Note has far less to do with using the CMake CLI and far more with our current CMakeLists.
For the sake of maintainability and similarity to the Makefile, I used glob for file searching in the CMakeLists. This has consequences covered in Author's Note Part 2. I don't have a strong opinion on glob vs manual definitions, however I do hope to clarify how manual definitions could work. Initially, this might seem like it would clutter the CMakeLists to a high degree. But by using add_subdirectory and target_sources, we can split the logic and the files. We can keep the logic in the main CMakeLists which lives in the main folder and then leave the file collection to sub-CMakeLists in c, c/datatypes and c/optionals. Then, if you add a file to c/optionals, you only have to modify c/optionals/CMakeLists.txt.
I created an example of this technique here: https://github.com/TheGreatCookieMachine/Sub-CMakeLists-Example
See also
https://cmake.org/cmake/help/latest/manual/cmake.1.html
https://cmake.org/cmake/help/latest/manual/cmake-generators.7.html#manual:cmake-generators(7)
https://cmake.org/runningcmake/ (However this seems to be woefully outdated.)
https://crascit.com/2016/01/31/enhanced-source-file-handling-with-target_sources/ (The source used for most of Author's Note Part 3)

What a fantastic collection of information! Absolutely no need to apologise for the length of the text, it was a great read; obviously as you say probably a bit too much for the README however it gives more than enough to be able to cherry pick the important parts, so thank you very much!


update from original master

When submitting a pull request, make sure you do the following:

[ ] submit the PR to the dev branch on the main repo, not the master branch
[ ] include a brief description what the PR is for in the subject
[ ] in the body, explain what the PR addresses
[ ] in the body, reference the issue that it closes by number in the format Closes #000.
[ ] squash multiple commits into one and use good commit messages (https://github.com/ngOfficeUIFabric/ng-officeuifabric/blob/master/docs/guides/CODING.md#commit-messages)

Please ensure you have followed the guidelines outlined in the docs: https://github.com/ngOfficeUIFabric/ng-officeuifabric/blob/master/docs/guides/PULL-REQUESTS.md
The branch manager can handle breaking up / squashing commits and updating commit messages if you don't do it, but this will delay your PR from being merged most likely.

done


fix: remove index.js in components

Updated imports for context and base.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: faf55f1ee888cecdbaec5922fb1e024ae63bba28

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: faf55f1ee888cecdbaec5922fb1e024ae63bba28

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Code match is to external code which re-uses Comcast code i.e. not a problem

Commit: faf55f1ee888cecdbaec5922fb1e024ae63bba28

Sorry, realized halfway through that you were just updating Base and context, this looks great, thanks!

No Problem, but I think I can update imports of other components as well(your comments) in the same PR.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: f3202cc411181d084daf600213a14ff0d5e28674

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: 5eef604f579325722c455ad1944713a5ee5f4f1e

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: 19db21f24113f0926e76380c669ab97f2b13d3a9

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: a538f6d495754ede7c9a9569f0e8583ec80517d0

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 2 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-UI-Components/132/rdkcentral/Lightning-UI-Components

Commit: a538f6d495754ede7c9a9569f0e8583ec80517d0

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: One match is not real, the other is definitely Metrological code

Commit: a538f6d495754ede7c9a9569f0e8583ec80517d0

@lightningjs/mixins

I

looks good! only file im seeing that needs a fix is withLayout.mdx - not only are it's docs referencing index.js, they're also referencing @lightningjs/mixins which certainly isn't a valid import 😬

Thanks for catching this, thinking to change import { withLayout } from '@lightningjs/ui-components'?

:tada: This PR is included in version @lightningjs/ui-components-v2.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Creating window from inside another window

Hello,
It seems it is impossible to focus a window that is created from inside a previous window, as the index of the previous window will always be higher than the newly created window.
This is a serious issue as it doesn't allow to create a modal window with children windows.
Even if you use the .focus() method on the child window right after creating it, it doesn't work.
Would it be possible to change this behaviour?
Thanks.

@vpetit-reimagine It seems if you put a 1 millisecond timeout on the focus of the newly opened modal it works, not ideal but it's a workaround for now.
function openNewWindow () {
  var newWin = new WinBox('Second Window')

  setTimeout(function () {
    newWin.focus()
  }, 1)
}

window.addEventListener('load', function () {
  new WinBox('First Window', {
    html: '<button type="button" onclick="openNewWindow()">Open New Window</button>'
  })
})

I found another solution and just do event.stopPropagation(); on all buttons of a popup (that creates new popups).
Still not perfect, but I think it's pretty decent


hovering effect added

hovering effect added in download file button in editor section in doc2pen

please merge the pr

Check the deploy preview here please. It is still not fixed

please check now


Laravel

Problem description

Problem location

I saw a problem on the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix

Thanks for the sharing. Closing in favor of https://github.com/docker/docker.github.io/issues/12719


Vendor Load UI: Manual upload interface

An interface that allows a tech services staff member to manually upload a file, for a vendor, select the data import profile, processing dag, and processing schedule is required to allow for "upload only vendors" and situations where data has to manually be altered in order to pass data import.
This interface/modal should be available at:

The vendor level, allowing selection of data import profile, processing dag, and dag schedule

The vendor interface level, defaulting to the configured values for the data import profile, processing dag, and dag schedule but allowing selection to override.

[ ] Add a link to the upload page (or open a modal) from the vendor page:

Notes:

Processing date is optional. If not selected saving this page should immediately instantiate the selected processing dag with config.
Processing date should be a date picker of some kind, if selected it should instantiate the selected processing dag with a start date of that selected.
The marc configurations should be optional

Is this done?  Or are there more bits?
I've used the file upload a few times for testing ftp stuff

@aaron-collier Is the implementation good enough to close this ticket?

Yes - this seems good, and the upload-only interface work/design will come shortly.


Maven Package Creation - v2.1.18

Bump Selenium version '4.8.2' -> '4.8.3'.
Maven package creation - v2.1.18.

Codecov Report
Patch and project coverage have no change.

Comparison is base (f930456) 98.55% compared to head (a4d6fad) 98.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##               main      #48   +/-   ##
=========================================
  Coverage     98.55%   98.55%
  Complexity       20       20
=========================================
  Files             1        1
  Lines            69       69
  Branches          6        6
=========================================
  Hits             68       68
  Misses            1        1

Impacted Files
Coverage Δ

...java/io/github/abhinavminhas/ShadowRootAssist.java
98.55% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Game-oriented networking library wrapping around quinn naming

I was suggested to open an issue about this to open up discussion.
I'm writing a library on top of quinn that provides basic game networking functionality, similar to libraries such as RakNet, but accessible from a synchronous game loop. I'm figuring this library may be useful to other people writing networked games in the Rust ecosystem, and benefit from contributions and code reviews. That's why I'm aiming at publishing this (right now fairly small) library to crates.io.
I'm hoping to call this library "harlequinn", and I wanted to open up discussion on if anyone has objection to this, because of the naming including "quinn".

No objection from me. @djc, @stammw?

I don't have any objection. Have fun !

Sounds good to me. Thanks for asking!


update to use crossbeam-channel 0.5

update dependency

Thanks! I'll merge once CI passes (could be a while, unfortunately)


[ASCII-1321] Ensure agent startup requests order

What does this PR do?

Ensure the agent waits at least a minute since startup to start sending the metadata payloads.
 I've removed the code to support PriorityProviders
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Ensure agent metrics have tags are present from the beginning.
You can add system tags into the datadog configuration in the tags . We can add custom tags in the datadog.yaml file
Ex. tags: ["foo:bar", "hello:world"]
To test that metrics do not have N/A tags. Run the agent with some custom tags.

Ensure your agent shows in the UI under the infrastructure list. (The host might take a few minutes to show up, that is expected)

Go to the metrics explored and validate that when querying for our custom tags we do not get nil values

💡 In the screenshot below we can compare two agent running. One with the fix the other running from main.

We can see the one in main we see metrics with tags N/A

/merge


Window stays grayed out after opening a link

Version: 1.45.1
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-17T22:31:53.325Z
Electron: 7.1.14
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.6.14-arch1-1
Steps to Reproduce:

Open link which opens modal window (or whatever it is)
Sometimes the background stays grayed out even after the action is taken

Opening Help -> About  makes it go away.

Does this issue occur when all extensions are disabled?: Yes

This is the intended behaviour when you open an external link. You can configure the trusted domains from this dialog.

@egamma I think you misinterpreted what I thought was the issue. I meant that when I click the open button and the modal window itself disappears, the background stays the same (grayed out/dimmed) until I open About modal. Then it works as intended even when I open link and click the actual open button.


server: remove number of compressed packets from diagnostics

These numbers are not particularly useful and are not available in upstream sysinfo (because they are Linux-only, as far as I can tell).
PTAL @lonng @overvenus

/merge


agent卸载后加载报异常

What do you want to ask?
插件通过attach方式加载后，通过attach的方式先卸载，再进行加载会报异常DupServiceException。

@zhzx2162626  Which plugin did you install and uninstall?  Please provide the specific error logs for us to locate the problem.

代码片段
...
private String install = "command=INSTALL-PLUGINS:qoc-uni-plugin";
private String uninstall = "command=UNINSTALL-PLUGINS:qoc-uni-plugin";
public void exec(string cmd){
....
VirtualMachine vm = VirtualMachine.attach(pid);
vm.loadAgent("/myagent.jar", cmd);
vm.detach();
}
public void reload(){
exec("uninstall");
exec("install");
}
调用 reload之后发生如下异常

@zhzx2162626 The possible cause is that the LocalMockHttpService is not properly stopped during the uninstallation. As a result, this problem occurs when the plug-in is installed for the second time. You can check whether the logs generated during the uninstallation contain any exception information. Alternatively, run the debug command to check whether LocalMockHttpService is properly stopped during uninstallation and whether LocalMockHttpService is removed from ServiceManager after being stopped.

经过debug我感觉发现问题了，是因为在卸载插件的时候只有plugin里的服务的实现类被卸载了，但是接口类没有卸载；在重新安装插件的时候因为接口类仍然在service的map里导致冲突，不知道这是否一个bug；
然后我解决的方法是卸载的时候直接卸载agent而不是卸载插件，然后再重新安装agent，不知道是否可以？

You can directly uninstall the Agent. After the Agent is uninstalled, the map of the service is cleared.
Another thing I want to ask：
In which module is your interface class defined?  Is it in the plugin module?  What class loader do you use to load your interface classes?

@zhzx2162626 The next version fixes the problem that PluginClassLoader can be loaded into the host class, which is not the same as your problem.

I have reproduce it using 1.2.0 and it has been fixed in 1.2.1 and newer version. So you can update your sermant version


Feature: Control charge current limit

Thanks to @magior  for the hard work

Something seems to go wrong if no charge current limitation is selected.
2023-07-27 19:41:36,384 Invalid charge current limit received: 0
2023-07-27 19:41:36,384 Refreshing vehicle status succeeded...
2023-07-27 19:41:36,385 'handle_vehicle_XXXXXXXXXXXXXXXXXX' task crashed with an exception
Traceback (most recent call last):
  File "/home/tosate/git/saic-python-mqtt-gateway/mqtt_gateway.py", line 110, in handle_vehicle
    self.ha_discovery.publish_ha_discovery_messages()
  File "/home/tosate/git/saic-python-mqtt-gateway/home_assistant_discovery.py", line 22, in publish_ha_discovery_messages
    [m.get_limit() for m in ChargeCurrentLimitCode], icon='mdi:current-ac')
  File "/home/tosate/git/saic-python-mqtt-gateway/home_assistant_discovery.py", line 22, in <listcomp>
    [m.get_limit() for m in ChargeCurrentLimitCode], icon='mdi:current-ac')
  File "/home/tosate/git/saic-python-mqtt-gateway/venv/lib/python3.10/site-packages/saic_ismart_client/common_model.py", line 84, in get_limit
raise ValueError(f'Unknown charge current limit code: {self}')
ValueError: Unknown charge current limit code: ChargeCurrentLimitCode.C_IGNORE

Process finished with exit code 255

Should ChargeCurrentLimitCode.get_limit() also return the value Max for ChargeCurrentLimitCode.C_IGNORE?

Yes, maybe this could be a good default to start.
Could you describe the situation that led you to receive 0 as charge current limit?
In my tests, my MG4 car never returned 0.

Yes, maybe this could be a good default to start.
Could you describe the situation that led you to receive 0 as charge current limit? In my tests, my MG4 car never returned 0.

That's a bug in my HA integration, we should never be able to set IGNORE from there.

@tosate How about we postpone this for 0.4.0?
We can use that as a release for the "Charging management 2.0", i.e.

AC Current Limit
Scheduled charging configuration via the gatway and HA (I've got a PoC ready)
Dynamic polling during charging

Okay, we can do that

So, this PR is foreseen for 0.4.0. I will update the README tomorow and then publish 0.3.0 as it is now.

I still have errors when I run this code. For my ZS the charge status provides:
charge_mgmt_data.bmsAltngChrgCrntDspCmd = 0
This leads to a call of update_chage_current_limit()with parameter ChargeCurrentLimitationCode.C_IGNORE
Finally this leads to a ValueError in VehicleState.update_charge_current_limit() because the charge_current_limit is never None, but has an unexpected value.
I have added a condition to the if statement in VehicleHandler.handle_mqqt_command()

You find this in branch 0.4.0-rc

I still have errors when I run this code. For my ZS the charge status provides: charge_mgmt_data.bmsAltngChrgCrntDspCmd = 0
This leads to a call of update_chage_current_limit()with parameter ChargeCurrentLimitationCode.C_IGNORE
Finally this leads to a ValueError in VehicleState.update_charge_current_limit() because the charge_current_limit is never None, but has an unexpected value.
I have added a condition to the if statement in VehicleHandler.handle_mqqt_command()

Can you provide a dump of the configuration topic for your car? The raw one

You mean this:
name:Tire,code:J17,value:1;name:Airbag,code:Q00,value:1;name:FAirbag,code:Q01,value:0;name:AirbagS,code:Q09,value:0;name:Sunroof,code:S35,value:3;name:Remote control,code:S61,value:1;name:AC,code:T11,value:1;name:EPS,code:EPS,value:1;name:SA64,code:SA64,value:0111110000000000001000000100101000000010100000000000000000000110;name:BONNUT,code:BONNUT,value:1;name:DOOR,code:DOOR,value:1111;name:BOOT,code:BOOT,value:1;name:ENGINE,code:ENGINE,value:1;name:EV,code:EV,value:0;name:HeatedSeat,code:HeatedSeat,value:0;name:KEYPOS,code:KEYPOS,value:1;name:ENERGY,code:ENERGY,value:1;name:BATTERY,code:BATTERY,value:1;name:INTEMP,code:INTEMP,value:1;name:EXTEMP,code:EXTEMP,value:1;name:WINDOW,code:WINDOW,value:0000;name:LRD,code:LRD,value:0;name:BTKEY,code:BTKEY,value:1;name:Battery,code:BType,value:2

AC current? I thought this is about charge current.

Yeah A/C as in alternate current. Only thing I can see is a battery of type 2 while mine is of type 1. But I'm quite sure that's related to the battery chemistry

I found it: Gateway Charging refresh period


pvrtc alpa decode bug

I have a problem when using pvrtc decompression in bgfx. When I am using pvrtc textures with gles2 extension the texture alpha looks fine, but when it is decompressed with decodeBlockPtc14A it is wrong.
RGB channels are ok.
See attached image:

The right picture shows my attempt to fix it with modifying
https://github.com/bkaradzic/bgfx/blob/master/src/image.cpp#L2250
and https://github.com/bkaradzic/bgfx/blob/master/src/image.cpp#L2268 to
*_a += 255 * _factor;
The results seems better but far from perfect...
Here is our pvr with alpha. ( the images above only shows a circle/star part of the whole decal)
decals.zip

Are you using PVRTexTool?
Also can you try with this texture:
https://github.com/bkaradzic/bgfx/blob/master/examples/assets/textures/alphatest.png

I am using PVRTexTool for compression.
Thanks for the fix. It is much better, but there are still slight differences.
I have checked the rgb channels more thoroughly to see if this is some kind of precision issue, but I haven't found any difference. So rgb looks exactly the same.

I think I found the solution. It seems the punchtrough value was incorrect.
This https://github.com/bkaradzic/bgfx/blob/master/src/image.cpp#L2298
should be this: const bool punchthrough = !!(bc[4] & 1);
And the change (https://github.com/bkaradzic/bgfx/commit/ea67fa406f198657181d61e453ff15f78293a4c6)
at this line (https://github.com/bkaradzic/bgfx/blob/master/src/image.cpp#L2115) should be reverted.
The original 0,0 for alpha was good. It caused problem because of the invalid punchtrough values.
I compared multiple image with this fix. I have only found slight intensity differences. In rgb it is hardly noticable, In alpha it is more visible. It seems like the rounding or something like this is done differently?  I have attached the result of conversion from ptc1a -> rgba8 with pvrtextool and bgfx.
decomp.ZIP


Update deprecated dependencies

Is your feature request related to a problem? Please describe.
Kinda of a bug, kinda feature request.
When build a docker container with docker build -t rethink:dev ., the following pops out of the terminal.
 Check these notes: https://bit.ly/2ZEqIau

Describe the solution you'd like
Upgrading deprecated dependencies.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Fixed the build. Took a while to make it=)

On branch rethink/code-clean I still get the following
Sending build context to Docker daemon  2.225MB
Step 1/12 : FROM node:gallium-alpine as build
 ---> 59b389513e8a
Step 2/12 : WORKDIR /app
 ---> Using cache
 ---> 137b9d332858
Step 3/12 : COPY package.json package-lock.json ./
 ---> Using cache
 ---> 7de29d33acb2
Step 4/12 : RUN npm i
 ---> Running in 8443aa9d175d
npm WARN deprecated<EMAIL_ADDRESS>Deprecated due to bug in CI build https://github.com/visionmedia/superagent/pull/1677\#issuecomment-1081361876
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.

added 1371 packages, and audited 1372 packages in 57s

160 packages are looking for funding
  run `npm fund` for details

1 high severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.
npm notice
npm notice New minor version of npm available! 8.5.0 -> 8.7.0
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.7.0>
npm notice Run `npm install -g<EMAIL_ADDRESS>to update!
npm notice
Removing intermediate container 8443aa9d175d
 ---> a6e77a3aa552
Step 5/12 : COPY . .
 ---> 420d29068c58
Step 6/12 : RUN npm run build
 ---> Running in 697f5440c5e5

><EMAIL_ADDRESS>prebuild
> rimraf build

><EMAIL_ADDRESS>build
> cross-env NODE_ENV=production webpack

asset server.js 3.92 KiB [emitted] [minimized] (name: server)
orphan modules 8.21 KiB [orphan] 19 modules
./src/main.ts + 19 modules 8.69 KiB [not cacheable] [built] [code generated]
webpack 5.72.0 compiled successfully in 3896 ms

asset app.585919a8edc80a64e256.js 4.32 MiB [emitted] [immutable] [minimized] [big] (name: main) 1 related asset
asset favicon.ico 5.55 KiB [emitted] [from: static/favicon.ico] [copied]
asset index.html 584 bytes [emitted]
orphan modules 1.86 MiB [orphan] 822 modules
runtime modules 2.03 KiB 5 modules
cacheable modules 1.64 MiB
  modules by path ../../node_modules/ 1.27 MiB 159 modules
  modules by path ./src/ 381 KiB
    modules by path ./src/*.tsx 291 KiB
      ./src/index.tsx + 72 modules 284 KiB [built] [code generated]
      ./src/data-explorer.tsx 6.82 KiB [built] [code generated]
    modules by path ./src/*.ts 87.4 KiB
      ./src/socket.ts + 24 modules 85.8 KiB [built] [code generated]
      ./src/requests.ts 1.64 KiB [built] [code generated]
    ./src/top-bar/data-hooks.ts 2.07 KiB [built] [code generated]

WARNING in<EMAIL_ADDRESS>1:2296-2297
export 'nameToIdDefault' (imported as 't') was not found in '@reatom/core' (possible exports: addToSetsMap, callSafety, createAtom, createReatomError, createStore, createTemplateCache, createTransaction, defaultStore, delFromSetsMap, getState, isAction, isActionCreator, isAtom, isFunction, isObject, isString, noop, pushUnique)
 @ ./src/store.ts 2:0-53
 @ ./src/app.tsx 15:0-32 96:15-20
 @ ./src/index.tsx 3:0-28 5:35-38

WARNING in<EMAIL_ADDRESS>1:2766-2767
export 'initAction' (imported as 'r') was not found in '@reatom/core' (possible exports: addToSetsMap, callSafety, createAtom, createReatomError, createStore, createTemplateCache, createTransaction, defaultStore, delFromSetsMap, getState, isAction, isActionCreator, isAtom, isFunction, isObject, isString, noop, pushUnique)
 @ ./src/store.ts 2:0-53
 @ ./src/app.tsx 15:0-32 96:15-20
 @ ./src/index.tsx 3:0-28 5:35-38

WARNING in<EMAIL_ADDRESS>1:2840-2846
export 'initAction' (imported as 'r') was not found in '@reatom/core' (possible exports: addToSetsMap, callSafety, createAtom, createReatomError, createStore, createTemplateCache, createTransaction, defaultStore, delFromSetsMap, getState, isAction, isActionCreator, isAtom, isFunction, isObject, isString, noop, pushUnique)
 @ ./src/store.ts 2:0-53
 @ ./src/app.tsx 15:0-32 96:15-20
 @ ./src/index.tsx 3:0-28 5:35-38

WARNING in asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
This can impact web performance.
Assets:
  app.585919a8edc80a64e256.js (4.32 MiB)

WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
Entrypoints:
  main (4.32 MiB)
      app.585919a8edc80a64e256.js

WARNING in webpack performance recommendations:
You can limit the size of your bundles by using import() or require.ensure to lazy load some parts of your application.
For more info visit https://webpack.js.org/guides/code-splitting/

webpack 5.72.0 compiled with 6 warnings in 21860 ms
Removing intermediate container 697f5440c5e5
 ---> 5690460cda24
Step 7/12 : FROM node:alpine as release
 ---> 9e17f47b0a78
Step 8/12 : COPY package.json package-lock.json ./
 ---> aaf5db2baebc
Step 9/12 : RUN npm i --only=prod
 ---> Running in<PHONE_NUMBER>7a
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '>=12.x <=16.x' },
npm WARN EBADENGINE   current: { node: 'v17.1.0', npm: '8.1.2' }
npm WARN EBADENGINE }

added 200 packages, and audited 201 packages in 21s

25 packages are looking for funding
  run `npm fund` for details

1 high severity vulnerability

To address all issues, run:
  npm audit fix

Run `npm audit` for details.
Removing intermediate container<PHONE_NUMBER>7a
 ---> eee6f5247b30
Step 10/12 : COPY --from=build /app/build ./build
 ---> 3e1f176b430b
Step 11/12 : CMD ["npm", "start"]
 ---> Running in c993f2df5f12
Removing intermediate container c993f2df5f12
 ---> 33995c652560
Step 12/12 : EXPOSE 3000
 ---> Running in 3c5488bf544f
Removing intermediate container 3c5488bf544f
 ---> f8b7047096a5
Successfully built f8b7047096a5
Successfully tagged rethink:dev

Is it on the main branch?

code cleaning breaks heavily, I wanted to migrate on new mui version, but thats didn't go fine
I think it is better to stick to this version, do the useful stuff and then try to migrate again
But if someone is capable to finish the job- it ld be cool

The project has moved far away since issue opened. Closing


Update whitequark ref and ruby version for Ruby 3.1 support

Co-authored-by: Alexandre Terrasa<EMAIL_ADDRESS>First changing the script, subsequent pull-requests will bring the imported tests and related parser changes.
Motivation
First step in supporting new grammar in Ruby 3.1
Test plan
See included automated tests.
@Morriar

That's a good point, I wasn't sure if the script was used on an ongoing basis. We followed the previous way of splitting the work into smaller PRs: https://github.com/sorbet/sorbet/pull/4238 by starting from this change.
I think we will end up having 4-5 parser change PRs each supporting a different feature. Do you want to combine them into 1 PR alongside this change?


Score value returned in search

I'd like to see a feature whereby I could extract the score value from the search results in an easier fashion. At the moment the only way (as far as I know) to retrieve the score value for documents is to add the .withOptions({ debug: true }) method to qb.where().
Imagine a scenario where we have the following code:
var search = function(req, res) {
  var searchQuery = req.params.searchQuery;
  db.documents.query(
    qb.where(
      qb.collection('character'),
      qb.parsedFrom(searchQuery)
    ).withOptions({ debug: true })
  ).result().then(function(response) {
    console.log(response);
    res.json(response);
  }).catch(function(error) {
    console.log(error);
  });
};

Let's assume that executing a the following search 'term1 AND term2' would produce two results. The response in the code above would now have 3 objects. 1 object containing debug information about the search (including the score in the results array) and the other 2 objects would contain information about the actual documents (including their content). There's no easy way to display a property of these documents along with their score values at the moment.
Probably the best way to do this - albeit I'm not sure if it's possible at all - would be to have an extra options for withOptions categories such as: .withOptions({ categories: ['content', 'score'] }) which could produce:
{ uri: '/character/lukeskywalker.json',
    category: 'content',
    format: 'json',
    contentType: 'application/json',
    contentLength: '837',
    score: 123456,
    content:
     { name: 'Luke Skywalker' }
}

Reassessment of the document envelope might be best to take up as part of a general comparison with JSearch after 8.0-4

Are there any updates regarding this now that 8.0-4 and JSearch are both available? I also have a few other suggestions for search which I'm happy to discuss.

Just checking in to see if this will be added anytime soon.

I am going through my old open issues - should I close this one off or would there be any potential updates to this?

It's still a good request.  Basically, the idea is to have a knob to put the metadata in the descriptor rather than have an initial summary object.

Closing the issue to keep API stable.


introduce cmake option ONNX_MLIR_DECOMP_ONNX_CONVTRANSPOSE

This is needed for Groq to disable this.

@sorenlassen Does the Groq have own implementation or rewriting for ConvTranspose op?

@sorenlassen Does the Groq have own implementation or rewriting for ConvTranspose op?

yes, we lower it to Groq's internal dialect
we don't support ONNXReverseSequenceOp at the moment so we can't compare the performance of your decomposition with Groq's lowering

Jenkins Linux s390x Build #10208 [push] introduce cmake option O... started at 05:20

Jenkins Linux ppc64le Build #9261 [push] introduce cmake option O... started at 05:23

Jenkins Linux amd64 Build #10197 [push] introduce cmake option O... started at 04:21

Jenkins Linux amd64 Build #10197 [push] introduce cmake option O... passed after 1 hr 25 min

Jenkins Linux s390x Build #10208 [push] introduce cmake option O... passed after 1 hr 36 min

It's unfortunate to have this PR merged in hurry. It's quite weird to deal with a specific operation using this way, saying using CMake. We should discuss a better solution that uses MLIR infrastructure.

Jenkins Linux ppc64le Build #9261 [push] introduce cmake option O... passed after 2 hr 3 min

Hey @tungld,
Sorry for rushing this into the codebase. The AMD folk have kindly opened an issue where we can discuss a robust way of solving the decomposition issue https://github.com/onnx/onnx-mlir/issues/2105.
We were hoping to be able to use a short term fix while we figured out the best way to solve the problem in the long run. Sorry for not giving you enough time to review the PR.
I sent you a slack message as well if you want to discuss this with me further.

@philass thanks for the information! Got it.


Inconsistent return value for PUT on action between Swagger and implementation

According to the Swagger file, when PUT is called on an action it returns an ItemId.
see: https://github.com/openwhisk/openwhisk/blob/master/core/controller/src/resources/whiskswagger.json#L285
In practice if the action already exist it seems to fail (i.e. not update possible) and if the action does not already exist it creates it (expected) but does not return an itemId but the full action.

Actually, this is true for the following requests :

updateAction
updateTrigger
updateRule
updatePackages
setRuleState

Also, return type is defined as "DeletedItem" for DELETE requests but this model is not defined in the Swagger file (it only has a description), while the API is returning the deleted object.

Cross linking to #270.


v1.9.0-rc-1 compute node OOM with ch-benchmark 5 MVs through PG/MySQL CDC

Describe the bug
https://buildkite.com/risingwave-test/ch-benchmark-pg-cdc-shared-source/builds/55#018f749e-d8ac-4e2a-a905-25038438f9f0
https://buildkite.com/risingwave-test/ch-benchmark-mysql-cdc-shared-source/builds/66#018f74a0-1ec8-4a62-8b65-61fda243cad7
https://grafana.test.risingwave-cloud.xyz/d/EpkBw5W4k/risingwave-dev-dashboard?from=1715648701806&orgId=1&to=1715655520239&var-component=All&var-datasource=P2453400D1763B4D9&var-instance=benchmark-risingwave&var-namespace=tpc-20240514-010410&var-pod=All&var-table=All
Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
No response
The version of RisingWave
v1.9.0-rc-1
Additional context
No response

The amplification is high.

The memtable spilling does not happen too frequently.
Is memtable spilling expected to solve this kind of high-amplification case?

But I think we need to sort this out
as one of our users did encounter bad performance when the amplification was high
High amplification cases do exist


RtContainer.java:22-24: Once we have the CI environment...

The puzzle 46-21e68db8 from #46 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/ca2e586aa4c4550784c893f3ad205def5014f0d1/src/main/java/com/amihaiemil/docker/RtContainer.java#L22-L24
The puzzle was created by amihaiemil on 19-Mar-18.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@amihaiemil/z please, pay attention to this issue

The job #58 assigned to @amihaiemil/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this

Job gh:amihaiemil/docker-java-api#58 already assigned to @amihaiemil, can't assign to @amihaiemil

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @amihaiemil/z

Order was finished: +30 point(s) just awarded to @amihaiemil/z

The job #58 is now out of scope


Is this script compatible on Windows?

When I try to install this on window, I get following error
ValueError: path '/etc/bash_completion.d/' cannot be absolute
This results in Error code 1
The path seems to indicate that this is probably meant for linux based system. Am I correct?

I have the same issue. s4cmd.py is in lib/site-packages, just not installed in the right location.


Add go1.12 to Travis CI

This adds the newly released go1.12 to the Travis build.
Do we still intend to support go1.10 or would you like that removed @miekg?

Codecov Report

Merging #915 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #915   +/-   ##
=======================================
  Coverage   55.01%   55.01%
=======================================
  Files          41       41
  Lines       10057    10057
=======================================
  Hits         5533     5533
  Misses       3499     3499
  Partials     1025     1025

Impacted Files
Coverage Δ

server.go
66.03% <0%> (ø)
:arrow_up:

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 164b22e...1415e9d. Read the comment docs.


Added ambit check to OpGetScale and OpSetScale (fixes #507)

Added ambit check to OpGetScale and OpSetScale (fixes #507)

no changing a players ambit anywhere 😔


feat(sheets-ui): undo set cols visible should scroll to cols

close https://github.com/dream-num/univer-pro/issues/2104

close #xxx

Pull Request Checklist

[x] Related tickets or issues have been linked in the PR description (or missing issue).
[x] Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
[x] Unit tests have been added for the changes (if applicable).
[x] Breaking changes have been documented (or no breaking changes introduced in this PR).

@wzhudev 隐藏行后取消定位到对应的行
2024_9_7 11_02_54 video.webm
2024_9_7 11_02_54 commands.json

Made it work for rows too. Please verify that. @zhaolixin7

Made it work for rows too. Please verify that. @zhaolixin7

@wzhudev 行列都不行了
2024_9_7 17_26_19 video.webm
2024_9_7 17_13_55 commands.json

Made it work for rows too. Please verify that. @zhaolixin7

@wzhudev 行列都不行了 2024_9_7 17_26_19 video.webm 2024_9_7 17_13_55 commands.json

或者 @oumomomo 修改一下对 bug 的描述？这样我好知道到底哪些路径上应该跳转到被展开的位置


Start 0.18.0 development cycle

Please fix the test

Please fix the test

Done.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1543   +/-   ##
=========================================
  Coverage          ?   82.93%
  Complexity        ?     1658
=========================================
  Files             ?      251
  Lines             ?     7719
  Branches          ?      754
=========================================
  Hits              ?     6402
  Misses            ?     1097
  Partials          ?      220

Impacted Files
Coverage Δ
Complexity Δ

...ensus/exporter/trace/ocagent/OcAgentNodeUtils.java
74.62% <ø> (ø)
12 <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 ffd2176...4ab9866. Read the comment docs.


Python 3 support

I have a library that depends on pybabel-json and while porting it to Python 3 I wanted to see if yours is compatible with Python 3 but running the tests results in:
$ python __init__.py
E
======================================================================
ERROR: test_number_of_occurences (__main__.TestExtraction)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "__init__.py", line 15, in setUp
    with open(os.path.join(os.path.dirname(__file__),'messages.pot'),'r') as output:
FileNotFoundError: [Errno 2] No such file or directory: 'messages.pot'

----------------------------------------------------------------------
Ran 1 test in 0.582s

FAILED (errors=1)

Using Python 3.4.3 on osx 10.7.5.

Got it, the path to babel.cfg was screwed, fixed that, PR will follow. Also will try to see if I can fix the obvious Py3 incompatibilities.

@tigrawap Here's the pull request.

Merged and closed, thanks


Catalina 3

The pipeline main estructure is created. Still other details but I will be improving in more short issues.


Move database to a separate service

We should move the data storage layer elsewhere, outside of the purview of the app. This way, data can be updated independent of the website.

hello 👋
i took a stab at this, and ended up with a simple API that exposes an endpoint that supports badge lookup and name search (similar to what is currently implemented in the app).
examples:
https://spd-lookup.herokuapp.com/officer?first_name=adr*
https://spd-lookup.herokuapp.com/officer?badge=6666
this layer is hosted on heroku, source code is here
hope this helps; but if not, i certainly had fun building it :)

This is awesome! Definitely along the lines of what we were thinking (TBH my first thought was just a DB somewhere that the service could connect to :joy: , this is much better). Let's talk offline about connection details & deployment!


[Suggestion] Add command to move tab to another window

Chrome lets me move a tab to another window by either dragging it to that window or right-clicking on the tab, selecting "Move Tab to Another Window", and selecting the appropriate window there.
I'd like an omni command to move the active tab to another window.

+1


Support Pulsar proxy from C++/Python client library

Motivation
Currently the C++ client lib (and by transition also the Python lib) is not following the redirection instruction to force the client to re-connect always through the Pulsar proxy.

Created #1167


Connect-VMNetworkAdapter : No network adapter was found with the given criteria (Is this ok?)

PS C:\Users\User\OSX-Hyper-V-main\dist\Scripts> .\create-virtual-machine.ps1 -name "Ventura" -version 13.7.2 -cpu 2 -ram 3 -size 128
Connect-VMNetworkAdapter : No network adapter was found with the given criteria.
At C:\Users\User\OSX-Hyper-V-main\dist\Scripts\create-virtual-machine.ps1:31 char:1

Connect-VMNetworkAdapter -VMName "$name" -Name "$($networkAdapter.nam

  + CategoryInfo          : ObjectNotFound: (:) [Connect-VMNetworkAdapter], VirtualizationException
  + FullyQualifiedErrorId : ObjectNotFound,Microsoft.HyperV.PowerShell.Commands.ConnectVMNetworkAdapter

Downloading 696-17987...
Saving http://oscdn.apple.com/content/downloads/13/14/072-44289/nz731hqcunm3nmfr0beuwwpw254laqxzck/RecoveryImage/BaseSystem.chunklist to C:\Users\User\OSX-Hyper-V-main\dist\com.apple.recovery.boot\BaseSystem.chunklist...
0.0/0.0 MB |=======================================| 100.0% downloaded
Download complete!
Saving http://oscdn.apple.com/content/downloads/13/14/072-44289/nz731hqcunm3nmfr0beuwwpw254laqxzck/RecoveryImage/BaseSystem.dmg to C:\Users\User\OSX-Hyper-V-main\dist\com.apple.recovery.boot\BaseSystem.dmg...
381.0/828.7 MB |=================                      | 46.0% downloaded

Either the network adapter 'device' populated in the VM default doesn't exist or there doesn't exist a 'default switch' type.
https://github.com/Qonfused/OSX-Hyper-V/blob/7f57012ca2b43c094149a1ce3e2b9ce31c298ea6/scripts/lib/create-virtual-machine.ps1#L29-L31
Just to check what caused the issue, could you return the outputs for Get-VMNetworkAdapter -VMName "Ventura" and Get-VMSwitch -Name "Default Switch"?
Since the rest of the VM appears to have been created without error, you can also go into the VM settings and manually change the virtual switch type for the network adapter:


fix(full-screen): dom fullscreen status text

General

✏ Mark the necessary items without changing the structure of the PR template.

[ ] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[x] My code follows the style guidelines of this project
[x] Is the code format correct
[x] Is the git submission information standard?
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

你这个提交记录好混乱

你这个提交记录好混乱

@likui628  没有新开分支 (•ิ_•ิ)


3.0-python3.9-core-tools not available to download

Is the image mcr.microsoft.com/azure-functions/python/3.0-python3.9-core-tools available to download? I can't seems to be able to use it. Here, for example, it is not listed: https://mcrflowprodcentralus.data.mcr.microsoft.com/mcrprod/azure-functions/python?P1=1631616167&P2=1&P3=1&P4=etS4ZUbzMrMeSv8lGYZp3X7kAkJMENBrrS6MMTlKwW4%3D&se=2021-09-14T10%3A42%3A47Z&sig=sqcaSdqtC6pFnoRuvOEVppU5FkAZMaozb1lK40m%2F7xs%3D&sp=r&sr=b&sv=2015-02-21
EDIT. If I use the docker file here I get the error: E: Unable to locate package azure-functions-core-tools-3

Are you able to use python3.8 image instead?
mcr.microsoft.com/azure-functions/python:3.0-python3.8-core-tools
Looks like only python3.8 core tools image is being published now
https://github.com/Azure/azure-functions-docker/blob/dev/host/3.0/core-tools-publish.yml#L265

Are you able to use python3.8 image instead?
mcr.microsoft.com/azure-functions/python:3.0-python3.8-core-tools
Looks like only python3.8 core tools image is being published now
https://github.com/Azure/azure-functions-docker/blob/dev/host/3.0/core-tools-publish.yml#L265

I can use 3.8 but I kind of need 3.9. I managed to compile it myself from the dockerfile posted in this repo with some minor modifications


Added International University in Germany Bruchsal

@lepasq lib/domains/de/iu-bruchsal: Your request has an incorrect file path format, please ensure the request follows     the repository rules set out at the bottom of this page and change     it accordingly or start a new one.     If you are unable to make a pull request with the instructions provided, please send an email to<EMAIL_ADDRESS>mentioning the URL of this request in the message.

@lepasq lib/domains/de/iu-bruchsal: Your request has an incorrect file path format, please ensure the request follows     the repository rules set out at the bottom of this page and change     it accordingly or start a new one.     If you are unable to make a pull request with the instructions provided, please send an email to<EMAIL_ADDRESS>mentioning the URL of this request in the message.

@lepasq lib/domains/de/iu-bruchsal: Your request has an incorrect file path format, please ensure the request follows     the repository rules set out at the bottom of this page and change     it accordingly or start a new one.     If you are unable to make a pull request with the instructions provided, please send an email to<EMAIL_ADDRESS>mentioning the URL of this request in the message.

@lepasq You might miss my previous comment. Please react.

@lepasq http://i-u.de/ says that the school is permanently closed. This domain does not belong to a university, nor to a college. I have to close the request without merging. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request. Shall you have any questions or concerns regarding the issue, please email us at<EMAIL_ADDRESS>


How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: https://github.com/drowe67/freedv-gui/releases
Publisher: freedv.org
Package Name: FreeDV
Description: Open Source Amateur Digital Voice mode for HF radio.
Package Version: 1.8.4
Installer URL: https://github.com/drowe67/freedv-gui/releases/download/v1.8.4/FreeDV-1.8.4-win64.exe

Note: latest package version is v1.8.12

I still create with this version now, you could create an issue to require to update it later


Byebug 9 and 10 stopping in internal Rails code instead of where I add the byebug call

Problem description
Any version of byebug > 8.
Rails 5.2
Ruby 2.5.1
When I add a byebug call, it stops in a (seemingly) random context that's not where I put the call. For example, I've gotten into ActiveSupport before. I'm unable to access local variables/methods/etc, making debugging impossible.
Expected behavior
Byebug stops in the context where I add the call.
Actual behavior
Byebug stops in some other context.
Steps to reproduce the problem
For me at least it's just adding a byebug call when on a version other than 8. If there's something particular I can provide to help please let me know.

This is due to bootsnap being integrated in Rails 5.2.
See https://github.com/deivid-rodriguez/byebug/issues/452

Thanks for the link. I couldn't quite tell from the discussion, is it planned to be fixed or is the fix just to change those bootsnap settings?

Try the bootsnap settings. It wasn't enought for me but apparently was good enough for others. Please report back if you still have issues with the bootsnap settings, this way I won't be the only one with that problem :-)

Something about those settings changes the loading order (or something) of my app code and my app won't even boot due to missing classes.


Feature request: Using json files for storing messages

Thank you guys for this awesome module.
Using json files for messages is a standard in i18n.
Another important feature is loading just the active language messages.

Hi @mahmoudmy !
There should be no problem using JSON files to store your translations.
vueI18n: {
  messages: { en: require('./lang/en-US.json') }
}

Loading active language only seems to be a most wanted feature, I'd really like to implement it but I'm not sure how to approach it yet and did not have much time to think about it, I hope I'll be able to get to it soon. Any help or ideas are very welcome :)
Thanks!


Difficulty destroying

I'm using your tool with https://github.com/roboll/helmfile and can successfully start up my terraform from a GIT repo. When I try to update or destroy, however, I seem to be having some problems. It doesn't seem to check GIT sources for updates if I do an apply and the runner pod seems to stay around if I do a destroy. I'm running the service in its own namespace and trying to delete that seems to hang in a "terminating" state. The result is that I basically have to recreate the cluster to get it to see updates. Any thoughts?

I did try editing the config map and changing action to "abort" makes the runner shut down but switching back to "apply" doesn't seem to do anything.

Helmfile is a great tool, I'm not sure it's related to the problem. I'm going to assume this would also happen when running helm delete on the terraform resource as well.
As far as the behavior you described, I'd like to clarify to get a clear picture.
Does the terraform resource in k8s get deleted?
i should have started with this question, what are the applyOnCreate, applyOnUpdate, applyOnDelete, and ignoreDelete options set as?

Also, what version of tfo are you running?

It does run a destroy pod and it runs to completion and stays there as "complete". Destroying leaves the runner around and doing a "kubectl delete pod" respawns it without a new config. The apply settings are:
applyOnCreate: true
applyOnUpdate: true
applyOnDelete: true
ignoreDelete: false

This on K3D and K3S by the way.

Does kubectl get terraform still have the resource there?
Just in case you missed my previous comment, what version of terraform-operator are you running?

After a destroy? It does not immediately go away but does after a bit. The pod remains running.
I'm running latest 0.3.8.
My terraform file originally used SSH agent but now I'm migrating it to use private key secrets and a lot of my sessions end with a stuck terraform waiting for an SSH agent that isn't going to show up. Does your tool rely on terraform runs always finishing?

I finished updating my terraform to not use local subdirectories. I can confirm that things are much more well behaved if apply runs to completion. Trying to destroy a stuck terraform seems problematic.

Thank you for keeping this ticket updated with your findings. I appreciate the feedback.

Does your tool rely on terraform runs always finishing?

There is an "soft" order of running apply and delete. In general, it is important to wait until the terraform apply completes before running destroy. This is especially important when the user is not using a “locking” backend.
If the user is using a “locking” backend, when the user deletes the terraform resource while apply is still running, the terraform destroy command will be blocked from running until it can obtain the backend lock. In essence, the terraform apply must release the lock. It is not up to this project, terraform-operator, to handle locks.
Now there is a caveat that both terraform apply and terraform destroy only retry 10 times to get started. 10 is an arbitrary number in run.sh.  In any case, I don’t think that the “destroy” pod should remain running so I will look into that.


This warning is important?

First Reload Config
2017-08-19 15:02:32: 15:02:32 ** Warning:   wfilter: No accessibility access to app RealtimeBoard Helper (no watcher pid)
2017-08-19 15:02:32: 15:02:32 ** Warning:   wfilter: No accessibility access to app Code Helper (no watcher pid)
2017-08-19 15:02:32: 15:02:32 ** Warning:   wfilter: No accessibility access to app Franz Helper (no watcher pid)
2017-08-19 15:02:32: 15:02:32 ** Warning:   wfilter: No accessibility access to app WhatsApp Helper (no watcher pid)

after restart the Mac:
2017-08-19 15:22:29: *** ERROR: ...on.app/Contents/Resources/extensions/hs/fnutils/init.lua:87: bad argument #1 to 'for iterator' (table expected, got nil)
stack traceback:
	[C]: in function 'next'
	...on.app/Contents/Resources/extensions/hs/fnutils/init.lua:87: in function 'hs.fnutils.each'
	/Users/micheldiz/.hammerspoon/utils/notify/init.lua:5: in function 'utils.notify.start'
	/Users/micheldiz/.hammerspoon/init.lua:48: in local 'fn'
	...on.app/Contents/Resources/extensions/hs/fnutils/init.lua:87: in function 'hs.fnutils.each'
	/Users/micheldiz/.hammerspoon/init.lua:43: in main chunk
	[C]: in function 'xpcall'
	...app/Contents/Resources/extensions/hs/_coresetup/init.lua:561: in function 'hs._coresetup.setup'
	(...tail calls...)
2017-08-19 15:22:34: 15:22:34 ** Warning:   wfilter: No accessibility access to app manager-osx (no watcher pid)
2017-08-19 15:22:34: 15:22:34 ** Warning:   wfilter: No accessibility access to app Code Helper (no watcher pid)
2017-08-19 15:22:34: 15:22:34 ** Warning:   wfilter: No accessibility access to app RealtimeBoard Helper (no watcher pid)
2017-08-19 15:22:34: 15:22:34 ** Warning:   wfilter: No accessibility access to app Franz Helper (no watcher pid)
2017-08-19 15:22:34: 15:22:34 ** Warning:   wfilter: No accessibility access to app WhatsApp Helper (no watcher pid)
2017-08-19 15:23:35: 15:23:35 ** Warning:   wfilter: Application Manager (354) is not registered yet

I'm looking for a solution to the annoying problem of Chrome not being in the Spaces that I determine. It loads all windows in the same space 1. I usually use more than 10 separate spaces on orders. Like research, study, reading, videos, folders, more research and etc. But Chrome is such annoying with this positioning thing.

First of all, my dotfiles are mainly for reference/inspiration, not for drop-in-and-use kind of thing :) This repo is also very outdated at the moment, I'll push update in next few weeks once I settle with what I'm working on right now.
First question (the error) — I think you've modified main init.lua a bit, and the error looks like you're trying to use notify module (via notify.start()), but have nothing in notify.enabled?
Second thing — nothing in my dotfiles really solves this, and this will be tricky to do (but should be possible!). You'd probably want to write a script that starts chrome, creates its windows and then moves them to spaces using https://github.com/asmagill/hs._asm.undocumented.spaces

My issue is something related to this https://bugs.chromium.org/p/chromium/issues/detail?id=74812
In fact it is this problem itself. They're working on it, but I'm surfing around to find other temporary solutions. Anyway, thanks for answering.
Best Reggards.


Question: Is there any way to know when a schema will have a fixed size and what size it will be?

We have specific optimizations in place where a Fixed Size serializable schema will use a different type of tree. Schemas are (in this case, but not required to be) known at compile time. Is there any way to know if say a type {T} will is fixed size and a bounded size for it.
This property is obviously false when dealing with Dictionaries, Strings, byte arrays, blobs, etc.

This can only be answered for a pair (schema, protocol). For example in Compact Binary almost all types are variable length, the only exceptions are bool, float and double and 8-bit integers. In Fast Binary all integer are fixed size. Furthermore most protocols omit optional fields that are set to default value.
There is no such API.

Can be done from outside of Bond internals? (If there is no such API).
Is there any intention to provide such an API?
If possible how?

The rules for determining if a schema is const length for a given protocol would have to hardcoded per protocol. There is no way to determine it programmatically. One could in theory imagine some way to encoding this as protocol metadata but I think it would only make sense to add if Bond itself had some feature taking advantage of fixed size fields.
As for determining what the size of a fixed length schema is, the easiest way would be to serialize an instance (e.g. default instance) to a stream that just counts bytes. We have a stream like that implemented in C++ and use it to calculate lengths in the first pass of Compact Binary v2 serialization.

Yes, I was actually thinking into implementing our own SimpleFastProtocol disallowing variable size types altogether; where we can estimate that without much problems. The solution used in C++ would also work, we use counting streams ourselves too.


Properly truncate system notices

As noted in #11658 the system notifications list will always suffix
system notices with ... even when the notice is longer than 120
characters.
Instead we should use .text.truncate to auto truncate and make the
notices clickable to view their details.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Also clicking on empty space does nothing, only clicking on actual text or icon shows up details modal.

ping LG-TM


Bug: ERROR: Service 'dashboard' failed to build : Build failed

What are you trying to achieve?
Fail when docker-compose build saleor platform on Fedora 37
Steps to reproduce the problem
git clone https://github.com/saleor/saleor-platform.git --recursive --jobs 3
cd saleor-platform
docker-compose build
What did you expect to happen?
Compilation successful.
Logs
#27 37.08
#27 37.08 ><EMAIL_ADDRESS>build
#27 37.08 > cross-env NODE_OPTIONS=--max_old_space_size=16384 vite build
#27 37.08
#27 40.25 [log] Enabling service worker...
#27 40.50 [log] vite v3.2.4 building for production...
#27 40.63 [log] transforming...

executor failed running [/bin/sh -c npm run build]: exit code: 1
ERROR: Service 'dashboard' failed to build : Build failed
Environment
Saleor version: 3.10.0
OS and version: Fedora 37 - 8GB RAM - Intel Core i5-4200U CPU @ 1.60GHz

I have the same issue. But it solved when I rebuild after  the server add  Memory to 8GB and Swap to 8 GB.

I have the same issue. But it solved when I rebuild after the server add Memory to 8GB and Swap to 8 GB.

I've tried it and unfortunately it doesn't work...

Anybody have any ideas? I've tried everything and I can't solve it...


external packages in packages.yaml recipe are not added to the generated spack.yaml

I am trying to create a recipe for the AMD stack on Hohgant. A working version of the generated spack.yaml would be:
spack:
  include:
  - compilers.yaml
  - config.yaml
  view: false
  concretizer:
    unify: true
    reuse: false
  specs:
  - kokkos+rocm std=17 amdgpu_target=gfx90a<EMAIL_ADDRESS><EMAIL_ADDRESS>  - kokkos-kernels
  - cray-mpich-binary+rocm<EMAIL_ADDRESS>  -<EMAIL_ADDRESS><EMAIL_ADDRESS>  - fftw +mpi
  - hdf5 +mpi
  - openblas
  - boost
  packages:
    all:
      variants: std=17 amdgpu_target=gfx90a amdgpu_target_sram_ecc=gfx90a target=zen3
      compiler: [gcc@11]
    mpi:
      require: cray-mpich-binary
    hip:
      buildable: false
      externals:
      - spec<EMAIL_ADDRESS>        prefix: /opt/rocm
    rocm-cmake:
      buildable: false
      externals:
      - spec<EMAIL_ADDRESS>        prefix: /opt/rocm/
    rocminfo:
      buildable: false
      externals:
      - spec<EMAIL_ADDRESS>
        prefix: /opt/rocm/
    rocprim:
      buildable: false
      externals:
      - spec<EMAIL_ADDRESS>
        prefix: /opt/rocm/rocprim
    llvm-amdgpu:
      buildable: false
      externals:
      - spec<EMAIL_ADDRESS>
        prefix: /opt/rocm
    hsa-rocr-dev:
      buildable: false
      externals:
      - spec<EMAIL_ADDRESS>
        prefix: /opt/rocm

Manually modifying the generated packages/gcc-env/spack.yaml file to the above model I am able to build a working AMD stack and create the squash-fs file. The rest follows the receipe in test/base-amdgpu.
However I was not able to correctly modify the recipe to generate the right content as described above. In this config file I am using external packages and I tried to put them into different positions into the packages.yaml file of the recipe but I never got it correctly and they always got discarded. The file that I ended up creating is the following:
packages:
    gcc-env:
      compiler:
          toolchain: gcc
          spec: gcc@11
      unify: true
      specs:
      - kokkos+rocm std=17 amdgpu_target=gfx90a<EMAIL_ADDRESS><EMAIL_ADDRESS>      - kokkos-kernels
      - cray-mpich-binary+rocm<EMAIL_ADDRESS>      - fftw +mpi
      - hdf5 +mpi
      - openblas
      - boost
      mpi:
        spec: cray-mpich-binary
        gpu: rocm
      hip:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>          prefix: /opt/rocm
      rocm-cmake:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>          prefix: /opt/rocm/
      rocminfo:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>          prefix: /opt/rocm/
      rocprim:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>
          prefix: /opt/rocm/rocprim
      llvm-amdgpu:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>
          prefix: /opt/rocm
      hsa-rocr-dev:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>
          prefix: /opt/rocm
    tools:
      compiler:
          toolchain: gcc
          spec: gcc@11
      unify: true
      specs:
      - cmake
      -<EMAIL_ADDRESS>
      - py-numpy

It discarded all the HIP related external packages together with the +rocm specification in the cray-mpich-binary package
even though the following was set:  gpu: rocm. Same if you use gpu:cuda.
Where would be the correct place for adding those external packages for a correct generation?

The packages.yaml file in the recipe is not the same packages.yaml provided
The following in your packages.yaml will be ignored
      hip:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>          prefix: /opt/rocm
      rocm-cmake:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>          prefix: /opt/rocm/
      rocminfo:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>          prefix: /opt/rocm/
      rocprim:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>
          prefix: /opt/rocm/rocprim
      llvm-amdgpu:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>
          prefix: /opt/rocm
      hsa-rocr-dev:
        buildable: false
        externals:
        - spec<EMAIL_ADDRESS>
          prefix: /opt/rocm

There is a fix to better test yaml files for valid input in #17.
The (slightly hacky) way to fix this at the moment is to edit the packages.py for hohgant.
That said, the solution of using ROCM installed as part of the CPE isn't ideal, as we want to be able to use the spack stacks on systems that don't have CPE installed. The following tool could be used to build a separate, stand alone ROCM: https://github.com/PawseySC/rocm-from-source


Revert "made Queue.hashCode transient"

Reverts javaslang/javaslang#534
actually me can't make Queue.hashCode transient. Wonder why this works for List.Cons!?

mmm... List.Cons has a SerializationProxy, but Queue is not


feat(backend): backupserver添加默认存储类型

关联研发信息
dev_branch: feat/helmchart_backup/#2414
target_branch: v1.3.0


Low level API for creating new delta tables

Description
Adding the ability to create a new Delta table from scratch
Use Case
Create a delta table outside of spark

I have started some work on this

Thanks @Smurphy000 ! I have assigned the issue to you :)

Hello, I see the issue have been closed, however I don't see the PR. What is the status of this subject?
Br

@Bernolt https://github.com/delta-io/delta-rs/pull/342


Latest everything, Forge, SpongeForge and Latch and I am crashing with the following error in console.

01.08 13:12:59 [Multicraft] Server stopped 01.08 13:12:59 [Multicraft] Server shut down 01.08 13:12:59 [Server] INFO at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_72] 01.08 13:12:59 [Server] INFO at org.spongepowered.mod.plugin.SpongeModPluginContainer.constructMod(SpongeModPluginContainer.java:296) ~[spongeforge-1.8.9-1890-4.2.0-BETA-1625.jar:1.8.9-1890-4.2.0-BETA-1625] 01.08 13:12:59 [Server] INFO at com.google.inject.internal.InjectorImpl.getInstance(InjectorImpl.java:1051) ~[spongeforge-1.8.9-1890-4.2.0-BETA-1625.jar:1.8.9-1890-4.2.0-BETA-1625] 01.08 13:12:59 [Server] INFO at com.google.inject.internal.InjectorImpl$2.get(InjectorImpl.java:1025) ~[spongeforge-1.8.9-1890-4.2.0-BETA-1625.jar:1.8.9-1890-4.2.0-BETA-1625] 01.08 13:12:59 [Server] INFO 1 error 01.08 13:12:59 [Server] INFO while locating com.ichorcommunity.latch.Latch 01.08 13:12:59 [Server] INFO at org.spongepowered.common.guice.SpongePluginGuiceModule.configure(SpongePluginGuiceModule.java:68) 01.08 13:12:59 [Server] INFO at com.ichorcommunity.latch.Latch.<init>(Latch.java:81) 01.08 13:12:59 [Server] INFO 1) Error injecting constructor, java.util.NoSuchElementException: No value present 01.08 13:12:59 [Server] INFO com.google.inject.ProvisionException: Unable to provision, see the following errors: 01.08 13:12:59 [Server] Server thread/ERROR [FML]: Caught exception from latch 01.08 13:12:59 [Server] Server thread/ERROR [FML]: The following problems were captured during this phase

This is because you are running 1.8.9, this plugin only supports Sponge API 5.0.0 which is for Minecraft 1.10.2. For now you would need to switch to 1.10.2, we'll consider a downgraded version but it is not likely.

Ok, I guess until then we will just stick with Keys.
Jason Brown
phone:<PHONE_NUMBER>
address: 42 North Ridge Dr Adairsville, GA 30103
site: jsnbrown.com
email<EMAIL_ADDRESS>https://www.facebook.com/jsnbrown
https://twitter.com/jsnbrown1989
https://www.instagram.com/jsnbrown1989/
On Mon, Aug 1, 2016 at 9:33 PM, Parker<EMAIL_ADDRESS>wrote:

This is because you are running 1.8.9, this plugin only supports Sponge
API 5.0.0 which is for Minecraft 1.10.2. For now you would need to switch
to 1.10.2, we'll consider a downgraded version but it is not likely.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IchorPowered/Latch/issues/2#issuecomment-236765604,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEIiFfKQiVRuVDDdgiFBvwE2b-GlGgkMks5qbp5ggaJpZM4JaHWW
.

We're going to release a backwards compatible version, but have some issues to fix in the main branch first. Watch this issue for updates.

Sorry that we never got to this, have been busy. I hope to update Latch soon, but for now support will remain on the latest API of Sponge.


Remove mention of bank account on error 400

We only have payouts through PayPal right now, so in order to remove confusion we shouldn’t tell people to attach a bank account when they can’t.  https://github.com/gratipay/gratipay.com/issues/3654

@mattbk The *.po translation files are updated automatically underneath release.sh. We shouldn't modify them directly here in this PR.

Okay, time to learn something new then.

Can I change the *.spt files, or does that mess with what messages the translation files are matched to?
I should be able to undo the changes I made to the *.po files and keep it all in the same PR, right?

Can I change the *.spt files, or does that mess with what messages the translation files are matched to?

You should only change the *.spt files (not the *.po files). During deployment, we extract strings from the *.spt files themselves into i18n/core.po, upload that to Transifex, and then download completed
*.po files from Transifex. This all happens in these Makefile targets. Bottom line: change *.spt files, not *.po files.

I should be able to undo the changes I made to the *.po files and keep it all in the same PR, right?

Yes! Easiest would be to manually edit the *.po files, commit that change, and then push it up to the same branch (error-400-bank-account). How's your git-fu? Want/need more specific pointers than that? :)

I replaced the *.po files from master but left the *.spt files modified.  I think.

!m @mattbk

@mattbk Looks like we've got tests for those error messages. Wanna try updating those as well? :)

I think it worked?

You win! :-)

!m @mattbk


Don't count acts or scene in an appendix

For example, this play provides the structure of the play in the appendix, see here.
In the metadata file, this play ends up having 10 acts (instead of just 5).

The appendix for the play in question should be moved out of the body element and into a back element, see https://tei-c.org/release/doc/tei-p5-doc/en/html/ref-back.html.

Thanks, that makes sense, will do that.

Ok done, see here. But this didn't solve the problem, we're still at 10 acts for "Perikles" in the metadata table.


Data API - /account/balance

what /account/balance needs to include the following information

unlocked balance
total balance (including both unlocked balance and locked balance)
vesting schedule
account nonce

why this is essential for our Custody team to be able to show the client their account details.
Note: we will followup with where exactly we want to place these information. But we will be leveraging SubAccounts to expose these information in the API
References: MsigGetAvailableBalance returns the unlocked balance of a Msig, the actual “balance” of the Msig actor is the total balance (including locked). There isn’t an endpoint to get the actual vesting schedule, but StateReadState will get you the StartEpoch and UnlockDuration (it’s always linear vesting).

I have drafted up some of the potential requests to the account balance endpoint

request to get the LockedBalance of a msig account**

{
    "network_identifier": {
        "blockchain": "bitcoin",
        "network": "mainnet"
    },
    "account_identifier": {
        "address": "0x3a065000ab4183c6bf581dc1e55a605455fc6d61",
        "sub_account": {
            "address": "LockedBalance",
            "metadata": {}
        },
        "metadata": {}
    }
}
request to get the TotalBalance of a msig account or a regular account
{
    "network_identifier": {
        "blockchain": "bitcoin",
        "network": "mainnet"
    },
    "account_identifier": {
        "address": "0x3a065000ab4183c6bf581dc1e55a605455fc6d61",
        "sub_account": {
            "address": "TotalBalance",
            "metadata": {}
        },
        "metadata": {}
    }
}

request to get the Unlocked balance of a msig account

{
    "network_identifier": {
        "blockchain": "bitcoin"
        "network": "mainnet"
    },
    "account_identifier": {
        "address": "0x3a065000ab4183c6bf581dc1e55a605455fc6d61",
        "sub_account": {
            "address": "UnlockedBalance",
            "metadata": {}
        },
        "metadata": {}
    }
}

rosetta cli command
rosetta-cli view:account '{"address":"t12ixisglleldv5nicf6z7vutaeavdn4hcj4cv73i"}'

log
2020/07/22 23:10:06 Primary Network: {
 "blockchain": "Filecoin",
 "network": "testnet"
}
2020/07/22 23:10:06 account { "address": "t12ixisglleldv5nicf6z7vutaeavdn4hcj4cv73i"} fetch error: BlockIdentifier is nil


When using integer scaling, resizing the window to be smaller than content_scale_size (the original viewport size) will clip the contents.

Godot version
master
System information
Windows
Issue description
Follow-up to https://github.com/godotengine/godot/pull/81012#pullrequestreview-1596954975.
This causes contents of the viewport to be clipped out of the window, which is not desirable, and also causes a problem with the implemented Window.move_to_center() when used with embedded windows.
Steps to reproduce
Create a 2D scene, set the window stretch mode to canvas_items and stretch scale mode to integer in the project settings, and resize the window to be smaller than the original viewport size. Result demonstrated in linked comment.
Minimal reproduction project
In linked comment.

In my opinion, the reasonable solution is to force the window minimum size to be equal to content_scale_size while using integer scaling, since integer scaling is supposed to be used with small content sizes anyway.

Perhaps we could note in the docs that with integer scaling it is recommended to set a minimum window size?


[oc_erchef] Make the _status endpoints health check timeout configurable.

http://wilson.ci.chef.co/job/chef-server-12-trigger-ad_hoc/98/

We typically avoid putting ChangeLog-Entry:  in the first line and rather do it in the body of the commit message.  However, we aren't using omnibus's change log generator anymore so you can omit it entirely.

Gone from my commit, and the PR title!

I also tested this with ec-metal and some load. When the timeout is lower, we get _status failures but no failed client requests. I'm not sure exactly where the ceiling is on this, probably not much more than a couple thousand ms, but making it configurable should help high load customers reduce false failures from the _status endpoint. I think the default as configured was good for most Chef server users.


Fix typo in Babel Loader for Hot-Reload

Brings the correct query so react-transform can hot-reload components properly.

Thanks so much for sticking with it and catching that bug, so much appreciated! :heart:


Added Cricket Provider

Added a new Cricket Provider.

Codecov Report

Merging #417 (520afff) into main (afe9f7b) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #417      +/-   ##
============================================
- Coverage     93.91%   93.85%   -0.07%
- Complexity     1985     1989       +4
============================================
  Files           218      219       +1
  Lines          3996     4003       +7
  Branches        384      384
============================================
+ Hits           3753     3757       +4
- Misses          146      151       +5
+ Partials         97       95       -2

Impacted Files
Coverage Δ

.../main/java/net/datafaker/service/files/EnFile.java
93.33% <ø> (ø)

src/main/java/net/datafaker/sport/Cricket.java
100.00% <100.00%> (ø)

.../main/java/net/datafaker/sport/SportProviders.java
100.00% <100.00%> (ø)

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

.../main/java/net/datafaker/service/FakerContext.java
84.61% <0.00%> (-2.57%)
:arrow_down:

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

Thank You @snuyanzin for the review. I have committed suggested changes. Actually, the reference was taken from FootballTest which has public test methods.

Thanks @ChiragMangukia for this PR, looks great!

@ChiragMangukia thanks for letting us know about the FootballTest. I've made all the test methods a bit more consistent now: https://github.com/datafaker-net/datafaker/pull/418


48 support for temporal mentions and normalizer

This PR fixes #48.
Description of changes
Huggingface has pretrained models to detect Temporal mentions in a chunk of text, which can be leveraged in a processor for our use case.
Timexy Normalizer (based on spacy): To incorporate different types of date formats and ensure consistency, etc.
Possible influences of this PR.
Provides temporal detection and normalization
Test Conducted
Test on two temporal detection and normalization unit tests

I see changes from #36 in this PR, You should pull the latest changes from master.

@Piyush13y can you help review this ? Thanks

@nikhilranjan7 Can you pull from master and resolve these conflicts? Also, we should add the test case which works with both the Normaliser and the Temporal mention tagger, so we can close this PR.


Show the impact of an expense on the active user's balance

Show the impact of an expense on the active user's balance. Relates to #131.

Note that this branch builds upon #138 as I use that for my development environment. Once the PR is in a finished state and approved, I can rebase on main.

This is necessary

Thanks @dcbr this is great! I just moved the balance on a different line (which looks better especially on mobile).


Remove DS_Store from github and add it to gitignore

.DS_Store shouldn't be stored on Github.

Fixed in #14


Update Manifest: updated manifest version for th-ch.YouTubeMusic 1.13.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest ?
[x] Have you tested your manifest locally with winget install --manifest ?
[x] Does your manifest conform to the 1.5 schema?

Note:  is the name of the directory containing the manifest you're submitting.

This PR updates manifest version 1.13.0 for the th-ch.YouTubeMusic package. Note: currently, versions newer than 1.20.0 are not being recognized as newer by winget, due to these versions being prefixed with 'v'.
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:


Add a function to allow users to delete unwanted deadlines and add new ones by themselves.

Basic implementation: 538f7b6


Browserify tests

While project uses browserify, it is not set up for testing.
There is a need to plug it into karma.conf.js so that we could use require() etc.

This duplicates #43


Add specs for Saxophone

Please remove the header generated by the IDE. Also, please open your PR on the fb-all-instruments branch (base).


Retain IP does not work on rollback

Rancher Version: master 11/06
Steps to Reproduce:

Create a service where retainIP is on
Upgrade the service
Rollback the service

Results: The IP of the original container changes on restart.  Looks like the original container is started back up before the new one is stopped.  Which is causing the new IP.
Expected:  The IP should not change.  Should remove new container before restarting old one.

Version - master 11/09
Verified fixed


Implement RSA_PSS_RSAE

Please note that while we are transitioning from travis-ci to AWS CodeBuld, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available): https://github.com/awslabs/s2n/issues/1504
Description of changes:

Add a "sig_alg" argument to the sign/verify functions to support variations like PSS
Add a pkey argument to s2n_pkey
Try to limit "#define RSA_PSS_SUPPORTED" usage by defining stub functions + "s2n_is_rsa_pss_supported()".
Do not send PSS signature schemes if PSS is unsupported

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

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

1. would it potentially break usages if these api were used outside of s2n

No. The structure is opaque, so customers can pass it around but not access its members.

what would you think about having the signature algorithm as part of the pkey, so sign() and verify() retains it's signature eg.

Because of how we use pkeys, we really don't know what specific pkey/sig_alg we're using anywhere. So we couldn't choose not to call your "s2n_pkey_sig_alg" function, except in a few specific pkey-internal methods like the key verification. We'd always have to call "s2n_pkey_sig_alg" before we called "s2n_pkey_sign." I would argue that makes splitting the two apis more clunky, since forgetting the "s2n_pkey_sig_alg" could break future code and would always be necessary. It would also require a new field on the pkey, which would also just be used by rsa.


Stop ignoring the W291 ruff rule
Motivation

https://docs.astral.sh/ruff/rules/trailing-whitespace/

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


ner_crf does not respect regex

Rasa NLU version: 0.13.8
Operating system (windows, osx, ...): MacOS High Sierra 10.13.4
Content of model configuration file:
language: "nl"

pipeline: "spacy_sklearn"

Issue:
When specifying a regex for a custom entity called 'psp' in the NLU training data, the regex is not correctly applied.
From nlu.md:
## intent: inform
- de psp pm is [0123456789](psp)
- de PSP PM is [6541323543](psp)
- [8547132541](psp)
- [9574657674](psp)
- [7527214272](psp)
- [5725275247](psp)
- [7427427527](psp)
- [8352407275](psp)
- [1757427558](psp)
- [7571227575](psp)

## regex:psp
- ^[0-9]{10}$

The following regex ^[0-9]{10}$ should only accept 10 digit numbers, but when it's shorter or longer, it's also mapped to the custom entity.
It's picked up by ner_crf.
From the logs:
entities '[{u'extractor': u'ner_crf', u'confidence': 0.8953123891331077, u'end': 5, u'value': u'12345', u'entity': 'psp', u'start': 0}]'
entities '[{u'extractor': u'ner_crf', u'confidence': 0.8953123891331077, u'end': 22, u'value': u'6516486464648468867461'

it does, though regex features are just features. As in, the features are used to train the entity recognizer, but it's not a strict rule for extracting the entity. If you'd like to make sure you only use numbers with 10 digits, then you can either use the FormAction or a custom action in Rasa Core


configloader package

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

158 of 207 (76.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-12.6%) to 82.524%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/configloader/configloader.go
158
207
76.33%

Totals

Change from base Build<PHONE_NUMBER>:
-12.6%

Covered Lines:
255

Relevant Lines:
309

💛  - Coveralls


Add FORK NEXT forks and Sepolia Suggestion

Change Description
Adds the EIP-2124 FORK_NEXT upgrade section and suggests block 1735371 for Sepolia.
This block number was chosen to happen near slot 417120, which is expected on 8/17/2022 at 12:24:00 PM UTC.
Block Number Rationale

Current slot: 223758
Slots until 417120:<PHONE_NUMBER>58 = 193362
Blocks per slot: 0.949 (caused by 100/1973 Sepolia validator keys lost)
Blocks until slot 417120: 193362*0.949 = 183500
EL block number at slot 223758: 1551617
Block height around slot 417120: 1551617+183500 = 1735117
Closest palindrome: 1735371 🎉

Cute Animal Picture

Codecov Report

Merging #564 (96a7604) into master (b70bc37) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #564   +/-   ##
=======================================
  Coverage   77.82%   77.82%
=======================================
  Files         333      333
  Lines       19219    19219
=======================================
  Hits        14958    14958
  Misses       4261     4261

Flag
Coverage Δ

unittests
77.82% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

🐕


graalvm-ce: 21.2.0 -> 21.3.0

Motivation for this change

Drop graalvm8 since it was removed by upstream
Add update.sh script to make it easier to generate hashes for all   platforms
Fix GraalPython, broken since #141825 (sorry)
Small refactorings and fixes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Tested build on linux and that also LGTM

I just realized that the update.sh script may generate unnecessary downloads of big chunks of tarballs because it doesn't check first if there is an update.
Let me fix this first.

I just realized that the update.sh script may generate unnecessary downloads of big chunks of tarballs because it doesn't check first if there is an update.
Let me fix this first.

Fixed.


Some great spells don't give Blind Diversion advancement

Versions
Minecraft: 1.18.2
Hex: Commit 6568168
Paucal: Commit 1f1dd45
Forge: 40.0.24
Issue
Green Sun's Zenith, Red Sun's Zenith, Black Sun's Zenith, Blue Sun's Zenith, White Sun's Zenith, Summon Greater Sentinel, and Flay Mind do not give the Blind Diversion advancement when casted successfully.
Create Lava, Summon Rain, Dispel Rain, Summon Lightning, Flight, and Greater Teleport give the Blind Diversion advancement when casted successfully.

Good catch!


Docs don't do syntax highlighting

Describe the bug
On our doc pages, even though I'm doing code fences and declare the language, the proper css to highlight the syntax isn't being applied to make the code look pretty.
To Reproduce
https://porter.sh/docs/
Expected behavior
Pretty code on our doc pages, such as yaml or go snippets being highlighted. Whatever we do shouldn't make the console/bash look awful.

@carolynvs I've got a fix for this in PR above


box breaks smart quotes

Description
Because of the #1716 I had to make a ubox (https://github.com/typst/typst/issues/1716#issuecomment-1690842054). But when I try to use smart quotes which are near the box they break:
#let underline-thickness = 0.5pt
#let underline-offset = 1pt

// underline box (can't use #underline(h(1fr)) in v0.8)
#let ubox(..box-args) = box(
  width: 1fr,
  stroke: (bottom: underline-thickness), // Same as underline
  outset: (bottom: underline-offset), // Same as underline
  ..box-args,
)
#set underline(stroke: underline-thickness, offset: underline-offset)

#let text = [ "this" ]
#let text = [ "this"]
#let text = ["this"]

#{
  ubox(width: 1em)
  underline(emph(text))
  ubox(width: 1em)
}

#let text = [ "text" ]
// #let text = [ "text"]
// #let text = ["text"]

Adding a pair of whitespaces is an easy workaround, but they do widen the overall length, which is very bad, if precision is the key. Thankfully, I didn't have (yet) cases where the precision (of underline length) is mission-critical.
Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

This is happening because boxes with non-fr widths are being treated as containing only an object replacement character (U+FFFC) in the typst::library::par::collect function, which handles smart quotes. Perhaps we could try inspecting the contents of the box for the purposes of the smart quote algorithm; any thoughts about this?
For some reason, boxes with fr widths are treated as whitespace when processing the last seen character but as an object replacement character for peeking.

This is now fixed! I tried compiling with the new version and without the "workaround spaces" after reading the new changelog:

Implemented a new smart quote algorithm, fixing various bugs where smart quotes weren't all that smart


Update Base64 variable names for better readability

Follow-up PR: https://github.com/facebookincubator/velox/pull/10371
The following changes are done in this PR.

Variable names are updated for better readability.
Function declarations in Base64.h are reordered for better maintainability.

@Joe-Abraham Would you rebase?

@Joe-Abraham Would you rebase?

@mbasmanova I have rebased with master


Fix replace-namespaced-keywords

Avoid infinite loop when key is not present.

Thanks for the PR! However, the latest commits in master are pretty experimental, and I've been working on other designs that might make your fix redundant, unfortunately.


Fix in Success message Url

For success message "This obj was added/changed successfully"
The url was unnecessarily? quoted, which resulted in a not working url.
Thank you for this compact super benefecial app !

Hey,
sorry, for the late reply, I've been away on vacation. Thanks for this, I'll merge it and make a new release ASAP.