DHW - Setpoint Hight and low

Hi,
I am looking for information about DHW
I find my hight set point :
https://smart.vaillant.com/mobile/api/v4/facilities/2*************4/systemcontrol/v1/dhw/Control_DHW/hotwater/configuration
{“body”:{“operation_mode”:“AUTO”,“temperature_setpoint”:57.0}
I find my actual T°c :
https://smart.vaillant.com/mobile/api/v4/facilities/2*************4/livereport/v1
{"unit":"°C","name":"Température de l'ECS du ballon","value":54.0,"_id":"DomesticHotWaterTankTemperature"
But I can't find ' low setpoint ' ?
also for ECS,
it is possible to know the times when it works thanks to the calculation with programming schedules
but isn't there an information (state) that indicates it directly?
you talk about it in an excerpt
        !! It could be misleading here, since when heater is not heating,
        target temperature if fixed (35 °C) - The API doesn't allow to change
        this setting. It means if the user wants to change the target
        temperature, it will always be the target temperature when the
        heater is on function. See example below:

        1. Target temperature when heater is off is 35 (this is a fixed
        setting)

I know :

/systemcontrol/v1/dhw/Control_DHW/circulation

/systemcontrol/v1/dhw/Control_DHW/circulation/configuration

/systemcontrol/v1/dhw/Control_DHW/circulation/configuration/timeprogram

/systemcontrol/v1/dhw/Control_DHW/hotwater

/systemcontrol/v1/dhw/Control_DHW/hotwater/configuration

/systemcontrol/v1/dhw/Control_DHW/hotwater/configuration/operation_mode                  Erreur 405

/systemcontrol/v1/dhw/Control_DHW/hotwater/configuration/temperature_setpoint         Erreur 405

/systemcontrol/v1/dhw/Control_DHW/hotwater/configuration/timeprogram                       Erreur 404

MErci pour votre aide :)

Hello,
sorry for the late reply.
There is no way to set low setpoint for hotwater.  Normally, the app of the VRC700 should only allow you to set target temperature from 35 to 70.

Sorry I worded my request badly,
I don't want to ""set"" low/high points ,  I want to " get"  values  high / low and also "get" value if the ECS is in a heating state or standby

Ah I see, there is no perfect way to do it.
There is no way to get it directly. You have to get the timeprogram for the hot water and for each timeframe you have the start time and the target temperature. So based on the current time you can find the right timeframe and you have to target temperature.

I see there is therefore nothing next to the API.
I also calculate like this because (fortunately) I have a simple schedule.
I close ticket. :)
Thank you again and happy holidays to you and your loved ones!


Add credentials: include flag to Fetch template

I am requesting a minor change so that NSwag code generation listens to the WithCredentials property.
I believe below change would to the trick. Is that something that could be added?
FetchClient.liquid:
{%-    if WithCredentials -%}
            credentials: "include"",
{%-    endif -%}

TypeScriptClientTemplateModel.cs
/// <summary>Gets or sets a value indicating whether to set the withCredentials flag.</summary>
public bool WithCredentials => _settings.WithCredentials;

I would like to second this request.
I just spent the last day and a half fighting with CORS in an environment where Windows Authentication is being used.  Part of the fix was manually changing the generated client to include the credential prompt.
The above option handles "Fetch" mode, but it would be nice if the "JQueryPromises" mode (or anything else using jQuery) could be configured to include credentials.  More details are available here:
https://learn.microsoft.com/en-us/aspnet/core/security/cors?view=aspnetcore-6.0#cors-policy-options
The important part is in the sub-section "Credentials in cross-origin requests".

I need to remove my second.  It turns out there is a straight forward way to add in the credentials property without having to hack the generated file.
First, create a file with a base class that contains a transformOptions method:
export abstract class CredentialClientBase {
   protected transformOptions(options: RequestInit) {
      options.credentials = "include";

      return Promise.resolve(options);
   }
}

Next, add some options to the NSwag config file:
{
   "codeGenrators": {
      "openApiToTypeScriptClient": {
         "extensionCode": "credentialClientBase.ts",
         "clientBaseClass": "CredentialClientBase",
         "useTransformOptionsMethod": true
      }
   }
}

This is working great for me.


rocPRIM on NAVI22 = gfx1031 only one test fails - can we fix this?

I added NAVI22 gfx1031 as a device, for a start, I simply copied the sections for gfx1030, compiled it and ran the tests.
Only test 14 failed:
Test results:
[----------] Global test environment tear-down
[==========] 280 tests from 112 test suites ran. (781 ms total)
[  PASSED  ] 272 tests.
[  FAILED  ] 8 tests, listed below:
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/16.ReduceMultiplies, where TypeParam = block_params<__half, __half, 32u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/17.ReduceMultiplies, where TypeParam = block_params<__half, __half, 64u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/18.ReduceMultiplies, where TypeParam = block_params<__half, __half, 128u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/19.ReduceMultiplies, where TypeParam = block_params<__half, __half, 192u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/20.ReduceMultiplies, where TypeParam = block_params<__half, __half, 256u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/21.ReduceMultiplies, where TypeParam = block_params<__half, __half, 129u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/22.ReduceMultiplies, where TypeParam = block_params<__half, __half, 162u>
[  FAILED  ] RocprimBlockReduceSingleValueTestsFloating/23.ReduceMultiplies, where TypeParam = block_params<__half, __half, 255u>
8 FAILED TESTS  	 of Test 14 - rocprim.block_reduce (Failed)
98% tests passed, 1 tests failed out of 51
Label Time Summary:
hip    = 500.81 sec*proc (49 tests)
Total Test time (real) = 502.70 sec
The following tests FAILED:
14 - rocprim.block_reduce (Failed)
Errors while running CTest
Three questions:

Could you fix this or should I contact the people from the gentoo distribution (I am using RHEL9.1 (AlmaLinux 9.1))?
Would you mind, adding the actual GPU name on which the tests run to the terminal output, not just an ID for clarity*?
Are 502.70 sec a reasonable time?

*My problem is, that the ID's (Node numbers) which I set using: export HIP_VISIBLE_DEVICES=0; export ROCR_VISIBLE_DEVICES=0  before running the tests  don't match rocminfo. I compiled rocPrim specifically for gfx1031 using: CXX=/opt/rocm/hip/bin/hipcc cmake -DAMDGPU_TARGETS=gfx1031 ../. but I am still not sure it runs on the dedicated GPU.

Hello,

Could you fix this or should I contact the people from the gentoo distribution (I am using RHEL9.1 (AlmaLinux 9.1))?

Unfortunately I do currently not have access to a system with a gfx1031 and I could not reproduce this on another GPU (which does not run alma). Is there any more information that the tests return? Usually there is a trace with some information, like the nature of the failure (crash/incorrect values, etc).

Would you mind, adding the actual GPU name on which the tests run to the terminal output, not just an ID for clarity*?

The node ID reported in rocminfo is the HSA node ID, which is indeed different from the GPU's index that HIP_VISIBLE_DEVICES and ROCR_VISIBLE_DEVICES etc use. You could use rocm-info to check which GPU is being busy, though im not sure if those indices are guaranteed to overlap with the hip device index. If you really want to be sure, you can set the specific device's UUID that is reported by rocminfo under Uuid: for example ROCR_VISIBLE_DEVICES="GPU-0123456789abcdef" test/rocprim/test_block_reducer.
If you set AMDGPU_TARGETS then the tests will only work for gfx1031, so unless your other gpu is also a gfx1031 which is somehow different than the one you want to run the test on, they would crash with a message that no binary would be available for the target. You can check by setting HIP_VISIBLE_DEVICES to that gpu :)

Are 502.70 sec a reasonable time?

Yes, this seems fine.

You know, perhaps this issue is related to https://github.com/ROCmSoftwarePlatform/rocPRIM/pull/401. It mentions the exact same errors. Can you try it?

#401 is the same error which I get on the rocPRIM-develop branch
The tests with rocPRIM-rocm-5.4.1 are successful:
100% tests passed, 0 tests failed out of 49
Label Time Summary:
hip    = 501.66 sec*proc (47 tests)
Total Test time (real) = 503.60 sec

great! so this issue is resolved?

For me and the time being (using version rocPRIM-rocm-5.4.1) yes, the development version has an issue at the moment

Closing as the issue appears to be resolved.


dont update source

I dont want 200 notsobots.
Also I bet you are reading this.

update source from beyond the grave

you couldve had 200 notsobots regardless... the code is right here on github anyway, so an updated source cant do anything but good

@plethorum Updating the source will only promote more "I made this" scenarios. For someone who wont just copy NSS's code it would be easy to port the source to a more up-to-date library version


input file name in call_stats2surf

"nii.gz" is by default part of the find-command, so if you input a file with nii.gz at the end it will look for a file with nii.gz.nii.gz
could be fixed via sed 's,.nii.gz.nii.gz,.nii.gz,' or if loop

I can also set -i directly to a file, so that the whole -i + -a combination for a single file is avoided

would be most straight forward


Add 2.3 release notes, add 's' to 'Enhancement' header

Signed-off-by: Chris Hesterman<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #260 (95da9d0) into 2.3 (0da978a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.3     #260   +/-   ##
=======================================
  Coverage   51.54%   51.54%
=======================================
  Files         183      183
  Lines        5805     5805
  Branches      888      888
=======================================
  Hits         2992     2992
  Misses       2592     2592
  Partials      221      221

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


ThisAssembly.Git.Commit: Information Mis-match

Hi, i started to use GitInfo for versioning and while checking generated version detail, seems like something is wrong
AssemblyInfo

Git History

Application Detail

Any help would be appreciated, Thanks

Hi, following is the build output related to GitInfo
_GitRoot:
  "C:\Program Files\Git\bin\git.exe" rev-parse --show-toplevel
  The command ""C:\Program Files\Git\bin\git.exe" diff-index --quiet HEAD" exited with code 1.
_GitClearCache:
  Deleting file "obj\x86\Release\GitInfo.cache".
_GitCommit:
  "C:\Program Files\Git\bin\git.exe" log --format=format:%%h -n 1
  "C:\Program Files\Git\bin\git.exe" log --format=format:%%H -n 1
_GitBaseVersionTagExists:
  "C:\Program Files\Git\bin\git.exe" describe --tags --abbrev=0
  The command ""C:\Program Files\Git\bin\git.exe" describe --tags --abbrev=0" exited with code 128.
_GitBaseVersionFile:
  Using base version from version file E:\Development\Production-WS\Lab\GitInfo.txt'
  "C:\Program Files\Git\bin\git.exe" log -n 1 --format=format:%%h "E:\Development\Production-WS\Lab\GitInfo.txt"
  "C:\Program Files\Git\bin\git.exe" rev-list --count --full-history "9d10c23"..HEAD "E:\Development\Production-WS\Lab"

Without knowing the contents of the GitInfo.txt file, it's impossible to know if the version is correct or not.
Also, when a file is used, the commits from that file directory are used for commit count. From your  screenshot, it looks like the commit hash is the right one, so I'm not sure what the problem is?
Note: git.exe rev-list --count --full-history "9d10c23"..HEAD. Looks like 9d10c23 was the commit that last touched GitInfo.txt. But again, without knowing what its contents are, hard to know.
I'd guess you have 1.0.0 on it, created on 9d10c23, then added 3 more commits on top, and that's why you're getting 1.0.3. The trailing .0 on the file version is because tools that expect 4 segment Version will default it to zero. Not sure that you used to take that screenshot.
But it seems to work as expected, TBH.

Sorry, i just missed to mention the issue, ThisAssembly.Git.Commit returned 39c3ee6 but it should be 39c3ee68.

it defaults to using seven characters

https://git-scm.com/book/en/v2/Git-Tools-Revision-Selection#Short-SHA-1
That's why you see 7 chars there. You can actually see for yourself by running the build with diagnostics and observing the git commands issued by GitInfo, and then running them yourself in the command line.

Sorry, i am commenting on the close ticket but it seems like tools are using 7-8 chars and guide also suggests Generally, eight to ten characters are more than enough to be unique within a project.
GitHub using 7 chars

GitLab using 8 chars

There should be way to configure this as this was the exact reason due to which i created this ticket because SHA i was seeing in VS and GitLab was different from the Version.
Thanks so much for your reference


Name is not suitable to be abbreviated

VSCode Version: unrelated
OS Version: unrelated

Steps to Reproduce:

Look at program tittle

The issue is the following the abbreviation VS is far too culturally extended that VS stands for versus, not visual studio and hence, me and what I figure is also many other people read Versus Code, this is so ingrained in our culture I am suprised nobody has signaled this before.
I think it is an error to present this in such manner.

We try to use the full name Visual Studio Code wherever possible in the product itself and on the website. In the documentation we try to use Visual Studio Code first, and then we sometimes abbreviate it to "VS Code" after that.
If you find places we use VS instead of Visual Studio please let us know exactly where.


main.yml CI installs from pyproject.toml

https://github.com/TomNicholas/VirtualiZarr/issues/89

This passes - merge?


using find packages instead of hard-coding the submodules.

the way it was before, my system could not find the labels submodule. like this all works for me.

Ah! Glad you were able to figure out what was going on. I left a couple comments on some cleanup and then I'll merge and publish a new version to pypi. Also feel free to add yourself to AUTHORS.rst!

Should I bump the minor revision, if u want to push it to pypi?

I can bump the version. There is another line I wanted to add to the history.

ok, then it's done i think.

Looks good thanks!

Just pushed to pypi. Does this fix the issue in #7?


Using a bash script for auto-conversion of .mp4

I've found a more convenient method for generating .gifs where we can use a shortcut key in VLC to set the starting and ending points.
We can use a simple Bash script that employs ffmpeg and fswatch to automatically detect new recording .mp4 files and convert them to .gif files.

Hi! Thanks for your input, I'll gladly hear more - can you walk me through your solution, as so:

User clicks xxxx button
Something happens
Some other thing happens
...


Device that gets connected to, is unable to check for available messages

Mobile Device Environment
Provide a list of operating systems on which this issue is relevant.

Device: Two Android 12 phones
OS: Android 12

Application Environment
Provide information about your development environment:

React Native version: 0.68.5
RN Bluetooth Classic version: 1.60.0-rc.25

Describe the bug
device1: accept connection
device2: start discovery
device2: finds device1
device2: connects to device1
device1/device2: accept pairing on both devices
device2: check for bonded devices, which returns device1. Then check for available messages from device1 and get no available messages (this is working correctly)
device1: check for bonded devices, which returns device2. Then check for available messages and get an error saying device1 is not connected to device2
Expected behavior
I expect to be able to check for available messages on device1, because it says they are bonded.
Am I missing something here? It seems like things are only working correctly on the device that runs the "connectToDevice", while the device that gets connected to (device1), doesn't work correctly?
Also, why is it that I can only find device1 when it's on the Bluetooth settings screen, but not on the app itself. Is this normal behavior? This means that users have to move away from the app, go to the bluetooth settings, to be discoverable, and once they get connected to, they go back to the app. This seems like too much work?

I suggest you go back to the documentation.
tldr;

Bonded = paired
Connected = open RFCOMM socket

There will never be any communication without being connected.  This seems to be working as designed, I suggest you provide specific code for review if you want more assistance.
With regards to your question on settings screen vs app screen, you need to provide code and/or screenshots for me to give any realistic answers.
The library was designed without phone/phone communication in mind, it was just a happy side effect that it allowed this to work.  The negative is that you still need to go into bluetooth settings to enable broadcast.  The good thing is that this only happens once, once you're paired you're paired going forward.
I believe there is an issue open, and I'll accept a PR, with regards to adding the functionality for turning on broadcasting/discoverability in the library.

Does that mean device1 will also need to do "connectToDevice(device2)"? Even though device2 already did it.

When I say device2 connects to device1, I'm doing this on device2: peer.connect(), where the peer is device1.
That's when I get the notification, on both phones, to pair.
If connect is just for pairing, which method is for actually connecting?

So are the two phones paired or not paired?  You have two options for pairing, personally I would do it in the Android Settings as I don't particularly trust the libraries pairing functionality between phones, I've only ever used it for actual devices.
So assuming the devices are paired, they show up in each others bluetooth list.

On both devices doing a BluetoothClassic.getBondedDevices gets you a list including the other device??
Once you have both devices you would:

On device 1 do BluetoothClassic.accept()
On device 2 do device1.connect() or BluetoothClassic.connectToDevice(device1.address)
both of these should return back on their respective devices.  This should NOT pop up a message asking you to pair, if it does, then you aren't paired, and you can therefore not actually connect.
Once you have the connection you should be able to read and write as you wish.

Ah!
If I pair them manually, by going to the bluetooth settings, on both devices, then do device1.connect() from device2, and then check for available messages on both devices, I no longer get the "not connected to device2" error.
When the devices are not paired, peer.connect() will trigger the pairing, but not the connection (socket), while if I do connect when they are already paired (from the Bluetooth settings screen), then it will actually connect the devices.
Thanks for your help!

Ya, this is a finicky thing in Android where it opens the pairing dialog, but it's not handling the connect correctly (it should fail the connect) but this is deeper in the BluetoothAdapter than I've had time to debug.
Glad it's up and running.


Windows Support

Will there be any windows support version of the library in the future?

Thanks for the question! See #438 for the discussion on this topic.


[Bug] Line limit?

Hello, I have a text file with 13 digits codes separated by line, and in this file there are 59880 lines / codes, the file is approximately 878 KB, however when viewing the file, from line 12079 the codes disappear, I can still select, but it does not appear, I tested it on Notepad++ and the codes normally appeared until 59880.

Hi. This is a known bug related to the underlying RichEditBox control. Also a duplicate of this and this.

Known issue for now.


Clear ENS on Address Election Addition

The field to add an Address to the election does not clear properly on ENS additions.
Inconsistent: when adding an account to an election, the field does not clear automatically and must be deleted character-by-character in order to add a new address properly.

In addition, we should also disable the add user button while ENS is working to resolve the names typed in.


feat: add arrow kotlin dependencies to gradle

Changes included

Added Arrow dependencies to build.gradle.kt

How to use this server inside kotlin-playground

Add kotlin-playground as NPM dependency of your web app: npm install kotlin-playground -S

Configure and use kotlin-playground:

// ES6
import playground from 'kotlin-playground';

const options = {
   server: 'http://localhost:8080' // an instance of kotlin-compiler-server
};

playground('.kotlin-code', options); // attach to all class="kotlin-code" elements

Create some HTML element hosting Kotlin with Arrow code:

<div class="kotlin-code">
   import arrow.core.Either

   val right: Either<String, Int> =
   //sampleStart
   Either.Right(5)
   //sampleEnd
   fun main() {
      println(right)
   }
</div>

For more information check the project's GitHub.

Will merge to new arrow branch.


How to use combination like IC-STN-->CNN-->IC-STN-->CNN type of structure

It should be straightforward -- you could possibly add a CNN on top of the final warped image from the first IC-STN, and the second IC-STN reads from the output of the CNN, and so on.
We didn't experiment with this as it wasn't really the central narrative of the paper, but if you happened to try it out, let me know what you find!


initial B-20229

B-20229
Summary
Update FRA weekly effective period
Background
the app pulls the diesel prices every day looking for newly published prices
EIA publishes new prices generally every Monday, before this change they would just be effective until the next publication.
This change makes it so that when new diesel prices are published they are only valid from tueday( or the day after they are published0 through the following Monday
to test

run make tasks_save_ghc_fuel_price_data to get the latest pricing data for your test
create a new move
approve move
access prime and create a payment request for fuel surcharge
login as TIO - review the payment request
validate that it selected the correct price based on the actual pickup date being between the effective and End dates in the DB GHC_DIESEL_FUEL_PRICES

may want to manually enter a few extra rows verify it works and making the actual pickup date land on the effecting and end dates

Warnings

:warning:
Please include a description of your PR changes.

  Generated by :no_entry_sign: dangerJS against ecb7a33573465ef1af38e5506e4ba099a0e2b0be

I had a pickup date 2 days after the latest ghc_diesel_fuel_prices. So my pick-up date was 06-12-2024 and in the DB the latest record had an effective date of 2024-06-04 and an end date of 2024-06-10. This records fuel_price_in_milicents was selected even though the pick up date was after its end date. It looks like in this case it picked the latest. Is that expected? Also how do we get these updated prices? If June 12th is 2 days from now but there is no ghc_diesel_fuel_prices with that date in its range what is that a likely situation and what do we expect to happen in that case?

I had a pickup date 2 days after the latest ghc_diesel_fuel_prices. So my pick-up date was 06-12-2024 and in the DB the latest record had an effective date of 2024-06-04 and an end date of 2024-06-10. This records fuel_price_in_milicents was selected even though the pick up date was after its end date. It looks like in this case it picked the latest. Is that expected? Also how do we get these updated prices? If June 12th is 2 days from now but there is no ghc_diesel_fuel_prices with that date in its range what is that a likely situation and what do we expect to happen in that case?

Yes- this is what it should do-
if there isn't a newly published rate- then it should grab the last published

I changed the pickup date to 06/02/2024 and it chose the ghc fuel price for date range 2020-07-21 - 2020-07-27. Not sure If thats expected.


GH: Stream receiver doesn't like a valid stream

I'm facing a problem with this stream: https://speckle.xyz/streams/0355f60d50/branches/02_base_model
The receiver stream doesn't like it.
I tried already with a new component, a new file... same error:

If I stream the last commit url of 02_base_model stream, I actually get another commit.. of another branch.... and empty!

Currently have the master at the 2.1.17 release.

I tried to push again on the branch test001 the same data, and it actually do the same...

Hi @iltabe!! As usual, thanks for reporting this (and for pushing the Speckle connectors to its limits!!). We've managed to track down the issue and we'll discuss this afternoon how best to fix this.
FYI, the error is not related with the stream url itself, but rather with the amount of objects it contains and how we're sending the request to fetch them :)

Did I pushed too many objects? ahah 🙈

Hey @AlanRynne,
This issue is quite essential for me to fix. Any workaround I can apply myself to be able proceed?

We've prioritized this and should get a fix in place today/tomorrow!

Hey @iltabe, as @teocomi very well pointed out... we're merging this right now and looking to release all affected connectors.
If you're building your own connectors, the latest master should do the trick

It worked! Nice job!


Nav check

This check is placed to check:
If the user installs the demo operator using the RED UI and then runs 1-click install script (externalized), in order to avoid conflict we just check if it already exists.

Stashing this one away since we don't need it right now.


Processing 'xx' failed.

When there is an error in a tt file, the error message in the title is all I get. It would be nice to have the full error message, if possible.
Currently you have to right-click on the offending file and invoke the custom build tool manually.

It also appears as if the transformations happens before the DLL or its dependencies are built, which means that a template cannot depend on classes in the current or another assembly. Is that correct?

It also appears as if the transformations happens before the DLL or its dependencies are built, which means that a template cannot depend on classes in the current or another assembly. Is that correct?

Yes, that is correct. You can't depend on classes in the current assembly. Maybe you could depend on classes in another assembly of the same solution if you fiddle around with the target dependencies in the .csproj files (see BuildTasks.TextTemplating.targets), but I haven’t' tried that.
You CAN definitely depend on  assemblies built outside of the current solution.

Fixed in NuGet package version 1.1.0


Link in Social Networks

Link in some social networks, such as twitter and tumblr and things!

Finished via b048c455dc5a20b8bb2e4b1d82e4debb8b3f6577


Us94 implement soft delete

Reviewed. LGTM, only the column order in the new index should be changed, so that it would be used by elasticsearch_updater queries.

Looks good to merge. Will do it on Monday morning, because if something breaks now, I won't be able to fix it.


Dropdown and its items have undefined react ids

.1l.$undefined-dropdown-list.$HJgjJEsPY-item-2-C0

I think was fixed more than a year ago by this commit. https://github.com/salesforce/design-system-react/commit/cba54b86e00dfd142470597ad8acee3775454835


TextInput and TouchableOpacity not working on android device

Description
Text input and TouchableOpacity not working on Android device
if I don't use navigation everything works fine, but when I use navigation to navigate through another screen TextInput, and TouchableOpacity don't work or react slowly
React Native Version
0.72.1
Output of npx react-native info
System:
OS: macOS 13.3
CPU: (8) arm64 Apple M2
Memory: 114.63 MB / 16.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 18.16.1
path: /opt/homebrew/opt/node@18/bin/node
Yarn:
version: 1.22.19
path: /opt/homebrew/bin/yarn
npm:
version: 9.5.1
path: /opt/homebrew/opt/node@18/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.12.1
path: /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 22.4
- iOS 16.4
- macOS 13.3
- tvOS 16.4
- watchOS 9.4
Android SDK:
API Levels:
- "28"
- "29"
- "30"
- "31"
- "32"
- "33"
Build Tools:
- 28.0.3
- 29.0.2
- 30.0.2
- 30.0.3
- 31.0.0
- 32.0.0
- 33.0.0
System Images:
- android-30 | Google APIs ARM 64 v8a
- android-30 | Google Play ARM 64 v8a
- android-31 | Google APIs ARM 64 v8a
- android-31 | Google Play ARM 64 v8a
Android NDK: 22.1.7171670
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9514443
Xcode:
version: 14.3.1/14E300c
path: /usr/bin/xcodebuild
Languages:
Java:
version: 11.0.19
path: /opt/homebrew/opt/openjdk@11/bin/javac
Ruby:
version: 3.2.2
path: /opt/homebrew/opt/ruby/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.72.1
wanted: 0.72.1
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: true
iOS:
hermesEnabled: true
newArchEnabled: false
Steps to reproduce
navigation (useNavigation) -> not working or slow
Snack, code example, screenshot, or link to a repository
same issue https://stackoverflow.com/questions/76484108/react-native-text-input-and-touchable-opacity-not-working-on-android-device

That code works at rn 0.71.12

Do you still experience this issue?
I have four years of experience maintaining facebook/react-native and I specialize in the Text and TextInput components. I currently have 58 facebook/react-native PRs.
If you still experience this issue, I will prepare a patched release with the fix.
Thanks a lot

Any suggestions? I had the same problem


Excluding Goland idea based IDE meta files

Incomplete .gitignore.
Please may you accept the additional ignore line.

This should be in your ~/.gitignore


update the checks interface readme to include the vulnerability check

update the checks interface readme to include the vulnerability check

/cc @listx @prathak07

/uncc

/lgtm
/approve


The model Landord could not be resolved from the registry plugin.

I have a User model (show below). It hasOne Tenant and hasOne Landlord (any given user could be a registered tenant, landlord or both). I also have a Users route that includes the User model.
For some reason that I can't seem to run down, I can reference the Tenant model as user.related('tenant') just fine, but if I flip that to user.related('landlord') I get the error above. What am I doing wrong here?
//
// In routes/users.js
//
var User = require('../models/user');

exports.doSomething = function(req, res) {
    return User.forge({ email: req.body.email }).fetch()
        .then(function(user) {
            console.log(user.related('tenant'));  // Works. Returns an object structure.
            console.log(user.related('landlord'));  // Doesn't work. "The model Landord could not be resolved from the registry plugin."
        });
};

//
// In models/app.js
// This is an application-level base model that domain models extend
//
var App = Bookshelf.Model.extend({
    // Functionality shared across all models
});

Bookshelf.plugin('registry');
App.model = Bookshelf.model.bind(Bookshelf);
module.exports = App;

//
// In models/user.js
//
var App = require('./app');
var User = App.extend({
    tableName: 'users',

    tenant: function() {
        return this.hasOne('Tenant');
    },

    landlord: function() {
        return this.hasOne('Landord');
    },
});

module.exports = App.model('User', User);

There's no way this would work without defining the 'Tenant' model somewhere (i.e.: App.model('Tenant', ???)). Wherever you are doing that you forgot to do the same with the Landlord model.

@ricardograca : Thanks. I didn't include them because I think they're exactly the same, but here are the Tenant and Landlord model files:
// models/tenant.js
var App = require('./app');

var Tenant = App.extend({
    tableName: 'tenants',

    user: function() {
        return this.belongsTo('User');
    },
});

module.exports = App.model('Tenant', Tenant);

// models/landlord.js
var App = require('./app');

var Landlord = App.extend({
    tableName: 'landlords',

    user: function() {
        return this.belongsTo('User');
    },
});

module.exports = App.model('Landlord', Landlord);

Are you requiring the Landlord and Tenant models somewhere in your code before trying to access them in other models (in this case the User model)?

No. The routes/users.js file only includes models/user.js and the User model itself includes neither the Tenant nor the Landlord model.

Then I'm not sure how the related('tenant') is even working in the first place :confused: The bottom line is that you must require those models somewhere before they can be used.

I'm still thoroughly stumped by this. Now I'm getting the same error message when attempting to fetch a user { withRelated: ['tenant', 'landlord']}. I'm using the Registry plugin and I've tried:

In models/user.js, including code to require('./models/(landlord|tenant)') as the plugin documentation seems to indicate.
In models/user.js, commenting out the code to require('./models/(landlord|tenant)') as mentioned in this issue comment

There is clearing something that I'm horribly, horribly misunderstanding here, but I'm feeling beyond lost. The Tenant and Landlord models are absolutely  identical and yet 1 works and the other does not. Where have I gone so completely, off the charts wrong here?

Code please!

 This is the users route as well as the user, landlord and tenant models.
https://gist.github.com/robwilkerson/62e4201705cd688ac80f
As always, thanks for having a look.

Everything looks okay to me. Next step would be publishing a full-on cloneable repo that uses in-memory SQLite so we can test the same thing.
That would mean:
require('knex')({
  client: 'sqlite3',
  connection: {
    filename: ':memory:'
  }
})

It'll take me some time to put that together. In the meantime, I've updated my gist to include my App model. As far as I know or can tell, these are the only files in play that would have anything to do with an error like this. I'll try to reproduce the issue in a simplified repo as soon as I can.

Aw, s**t.  It was a freaking typo.
Not only is it in my code, but I copied and pasted the error message directly from my console and the same typo is right in the subject of this issue. I missed this all over the place. My apologies, guys.
https://gist.github.com/robwilkerson/62e4201705cd688ac80f#file-user-js-L31

It happens :) You know, I read your updates and looked at the title many
times and still didn't notice the LanDORD. I'm sure Ben also didn't catch
it :D
Em 15/04/2015 19:38, "Rob Wilkerson"<EMAIL_ADDRESS>escreveu:

Aw, s**t.  It
was a freaking typo.
Not only is it in my code, but I copied and pasted the error message
directly from my console and the same typo is right in the subject of this
issue. I missed this all over the place. My apologies, guys.
https://gist.github.com/robwilkerson/62e4201705cd688ac80f#file-user-js-L31
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/bookshelf/issues/691#issuecomment-93525902.

What was the typo?

I can't remember 2 hours ago, much less 2 years ago, but based on @ricardogama 's comment, I can only assume that I misspelled "landlord". That was the name of a model I was working with on that project. 🙂


chore: bump ape core version

Bump Ape core version 0.2.1

Duplicated/issue with missing env variable


GROOVY-8289: STC and default value in ctor is causing debugging error

PR based on Eric's suggested fix in the issue. Still needs a test.

Test added.


[PT-BR] TypeScript with React

Fiquei em dúvida se teria que traduzir os nomes de funções e variáveis também...
@khaosdoctor, @alvarocamillont, @danilofuchs e @orta

Nice one, I think this is the first portuguese translation in the playground, so tyou'll need to make a pt/sections.json  like this one: https://github.com/microsoft/TypeScript-Website/blob/v2/packages/playground-examples/copy/es/sections.json

@orta! Just did it :)

'brigados!
Just need someone to give an OK on the language and we're good to go - thanks!

@orta Does translating the file name implies in any problems? I just saw that the file name was translated to PT but I'm not sure what it implies to the project as a whole. I probably think it should stay in English.

From the language POV, I agree with the changes proposed by @danilofuchs :)

Yeah, the filename definitely needs to stay the same, good spot @khaosdoctor  - I actually don't have a way to provide different display titles for the samples ATM, will have a think about how/if I can do that

@helcioItiyama Can you please revert the filename back to the English title?

@khaosdoctor done :)

@helcioItiyama I also think you're gonna need to apply @danilofuchs's changes by hand, if you haven't done that already

haha, so much rebasing and fixes - thanks a lot @helcioItiyama !

@orta no worries hehe

BTW, here it is: https://www.staging-typescript.org/pt/play/?jsx=2&esModuleInterop=true&q=334#example/typescript-with-react

Awesome! I'll do my best to keep helping!


Test with latest version of copier

We want to test with the latest version of copier.
This project is not meant to be packaged, and we just need to declare test dependencies, so we don't really need a packaging tool for that. So I simplify things a little bit by removing the need for poetry.
A couple of tests need updating for compatibility with the latest version of copier.
Also run tests with python 3.8 and 3.10, and stop testing with 3.6 which is EOL.

Eh eh, I was expecting that :)
The problem with Poetry, in my view, is that it does not follow python packaging interoperability standards and has its very how way of doing things. I reckon it was pioneering some solutions when the standards were lagging behind, but now there are better standard based solutions.
Anyway, I don't want to enter a tool war.
Would you mind, then, fixing the tests here, and make them run with the latest version of copier?
I could not do it myself with Poetry (probably because I lack Poetry knowledge).
And we are blocked on that for preparing the 16.0 branches.

Besides, https://github.com/dependabot has builtin support for updating both pyproject.toml and poetry.lock files. We can drop a configuration file, or enable it in repo settings, and get update PRs automatically.

BTW, there is no need pinning dependencies here. On the contrary. We want to test with the latest versions all the time, because that is what users are likely to have when they pipx run copier.

The problem is that this is fixing nothing, but it's introducing new problems.
Keep in mind that you would have a broken Copier 6.2 installation without all of this I'm saying (thanks to dependabot and poetry I detected this incompatibility before the release https://github.com/copier-org/copier/pull/785). Even using pipx. Also, main branch here would be red instead of green if we merged this before. Random CI breakages help nobody.
However, if you prefer another tool that gets the same benefits, I'm not against that.
Let me do the update.

To test with multiple copier version we should rather add them to the test matrix, no?

Yes, but that's not really needed. The template can just support one copier version.

Check out https://github.com/OCA/oca-addons-repo-template/pull/159, let's see if that works.

The problem is that this is fixing nothing, but it's introducing new problems.

I want to apologize. Later, while developing #159, I realized this was actually fixing something: adding support for latest copier, and allowing it to coexist with older python versions that couldn't execute copier but had to be installed to execute old hooks. I also realized these words sounded more rude in the comment than in my head. So sorry friend! 😓
I still think the poetry approach is better, due to the reasons already explained. But the rest of the change was actually needed and good, so thanks for your patience, cooperation and dedication. ❤
FWIW I'm using these days precommix, which puts an end to pre-commit surprises. I'd love to see it in use in OCA, but I know it can be scary for some people. It really takes the pre-commit experience to the next level because with it, if pre-commit works today, it will work forever.

No worries, thanks for fixing the CI.


⚠ Bofrost.nl has degraded performance

In 35b538b, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1862 ms

Resolved: Bofrost.nl performance has improved in 907614d.


[Bug] Tenant Login dropping tenant domain and login with super tenant domain

Description
When logging in to the publisher and dev portals using the tenant admin login after creating a tenant it logins as the super tenant admin.
Steps to Reproduce

create an API using the super tenant admin user (admin:admin)
create a new tenant using the carbon console. (tenant1, admin1@tenant1.com:admin1)
login with the new tenant admin.(admin1@tenant1.com:admin1)
the APIs published by the admin user is visible to the tenant admin user.
new apis created will belong to the super tenant domain. (refer AM_API table ORGANIZATION column)

Affected Component
APIM
Version
4.3.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
possibly introduced by :
https://github.com/wso2/api-manager/issues/1664
https://github.com/wso2/carbon-apimgt/pull/11976
Suggested Labels
No response

fixed with https://github.com/wso2/carbon-apimgt/pull/12281 and https://github.com/wso2-extensions/apim-km-wso2is/pull/124


slow paste with spinner

I have default paste settings on in insiders, but see a spinner every once in a while, for particular snippets of code
https://github.com/user-attachments/assets/59d36cd4-4cd8-4a00-9d60-aff32c7d86ac

This is expected in some cases as we need to see if there are imports to add
What code are you trying to paste?

I was pasting the import * as DOM ... line from collapsedCellOutput.ts into a new file.


Incorrect indentation with multiple elvis operators and multiline statements

Current behavior:
val foo =
    existingFoo
        ?: fooRepository.findExisting(
            bar = "baz",
        )
            ?: Foo(
            bar = "baz",
        )

Expected behavior:
val foo =
    existingFoo
        ?: fooRepository.findExisting(
            bar = "baz",
        )
            ?: Foo(
                bar = "baz",
            )

Using comma here to force multiline statements, but the same behavior occurs with long statements without trailing comma.

I think this was fixed by https://github.com/facebook/ktfmt/pull/416

Closed since now the mentioned code formats to:
val foo =
    existingFoo
        ?: fooRepository.findExisting(
            bar = "baz",
        )
        ?: Foo(
            bar = "baz",
        )


Use forwarding references in string/split.hpp

On compilers that support C++11, this allows both lvalues and rvalues to be used as inputs to the split(), find_all(), and ifind_all() functions.
For example, given a function get_string() that returns a std::string, this allows you to write:
boost::split(result, get_string(), boost::is_any_of(" "));

I'm somewhat surprised that this hasn't been done before, so if I'm missing something, then do let me know. I've hardly done a comprehensive investigation on where else these changes could be made in boost::algorithm, so I'm also happy to expand this PR if necessary.

Tests?

Whoops, looks like I saw the tests/ directory, but missed the string/tests/ directory, so I naïvely assumed there weren't any existing tests for split and co. I'll push some new lvalue tests now.

Now that tests have been added, is there anything else you'd like me to do to advance this PR, @mclow?

Sorry to bump this again, but it'd be really nice if this usability improvement could be merged at some point 🙂 If there's anything you'd like me to do to aid that, e.g. seeing where else this change could be helpful, or rebasing/squashing commits, then I'd be very happy to do so!

Just saw this tonight. I'll see if I can get to it this weekend.


Revisions status isn't correct

Description
I have a resource where revisions are enabled. There IS a saved version in the db, then I made a small change and clicked Saved Changes.
I see this:

I don't believe it should say This is the published version. In Statamic collections it has this:

Steps to reproduce

enable revisions in a resource
in an existing model (i.e. not a new one), edit something
Click Save changes

Environment
Environment
Application Name: Zakat Foundation of America
Laravel Version: 11.14.0
PHP Version: 8.3.8
Composer Version: 2.7.7
Environment: local
Debug Mode: ENABLED
URL: zakat.test
Maintenance Mode: OFF
Timezone: America/Los_Angeles
Locale: en
Cache
Config: NOT CACHED
Events: NOT CACHED
Routes: NOT CACHED
Views: CACHED
Drivers
Broadcasting: null
Cache: file
Database: mysql
Logs: stack / daily
Mail: smtp
Queue: sync
Session: file
Livewire
Livewire: v3.5.2
Statamic
Addons: 13
Sites: 3 (zakat.org (EN), zakat.org (AR), zfinstitute.com (ZFI))
Stache Watcher: Disabled
Static Caching: Disabled
Version: 5.12.0 PRO
Statamic Addons
aryehraber/statamic-captcha: 1.12.0
edalzell/forma: 3.0.1
jacksleight/statamic-bard-mutator: 2.3.1
jacksleight/statamic-distill: 0.7.2
jonassiewertsen/statamic-livewire: 3.6.0
statamic-rad-pack/runway: dev-fix/has-many-revisions
statamic/seo-pro: 6.0.3
transformstudios/events: 5.4.3
transformstudios/front: 3.0.1
transformstudios/github: dev-main
transformstudios/neon: dev-feature/ai-updates
transformstudios/review: 5.0
transformstudios/simple: 5.1.10

Hmm, I can't seem to reproduce this on my end 🤔
"This is the published version" only shows for me before I make any changes or after I hit publish on my changes, the same behaviour as on core.
https://github.com/statamic-rad-pack/runway/assets/19637309/f5067d8e-89a6-4c1b-8f11-2f9ecd21b086
When I edit a model with no previous revisions, I see "Model has not been published" up until the revision is published.

Make a change, click Save Changes, then reload the page.

Oh, I see it now, thanks!


[bug] - FullNode storing duplicate certs in process_certificate

to reproduce the bug, just do a transfer-coin and query the sui_getTransactionsFromAddress RPC endpoint, it will return duplicated transaction hash, the problem is probably in process_certificate the same cert is processes multiple times and the same entry is written into the indexes table.

this is fixed


feat: export extension_utils

Summary
This PR exposes extension_utils from lib/private/extension_utils.bzl.
I tend to copy this kind of functionality from extension to extension, so it would be really helpful to have this great method available in the public API.
I hope this is fine and other users might benefit from it, too :smile:
Changes are visible to end-users: yes

Searched for relevant documentation and updated as needed: yes
Breaking change (forces users to change their own code or config): no

Test plan
Do we need tests?

My concern with making this public was that I don't know if it's high enough fidelity and stable enough that we want to support the current iteration. I'll take another look through it..
fyi @alexeagle

Yeah, I don't think I want to make extension_utils public in it's current form. The toolchain_repos_bfs macro finds the toolchain version closest to the root but the right solution is probably to find the highest version in the dependency graph, allowing the root to override it.

thanks for the feedback. @kormide I can understand this decision. Currently I include the macro like this. I hope this is fine for you.


Copy over installation scripts and Docker config

Closes #9.

@jwallwork23 do I recall correctly that you wanted to copy over something else here? Or is it ready for review as it is?

@jwallwork23 do I recall correctly that you wanted to copy over something else here? Or is it ready for review as it is?

Apologies, done now. Ready for review. Linked PR: https://github.com/mesh-adaptation/animate/pull/104.


trust: check for --insecure-skip-verify

Fix: #1285

thanks, lgtm

#1287 still?

#1287 still?

Yes...
But this is not related to this PR. Merging.


Fix bug in event filtering for image events

Fix bug with tests, plus make event handling simpler and more robust generally to reduce possible states with optional event ids.

Codecov Report

Merging #148 (f1fdc54) into main (01165fb) will decrease coverage by 0.18%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             main     #148      +/-   ##
==========================================
- Coverage   92.19%   92.00%   -0.19%
==========================================
  Files          16       16
  Lines        1769     1777       +8
==========================================
+ Hits         1631     1635       +4
- Misses        138      142       +4

Impacted Files
Coverage Δ

google_nest_sdm/event_media.py
95.91% <88.46%> (-0.65%)
:arrow_down:

google_nest_sdm/event.py
97.76% <100.00%> (-0.38%)
:arrow_down:

google_nest_sdm/google_nest_subscriber.py
62.33% <0.00%> (-0.17%)
:arrow_down:

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 01165fb...f1fdc54. Read the comment docs.


Optimize ping() method

ping() method in ManagementCommands
https://github.com/valkey-io/valkey-py/blob/c490780a1d7b629317e5c0d976ec13dd64de8e55/valkey/commands/core.py#L1203-L1209
when i use ping() method in class ManagementCommands, i have some questions：

it does't support message param, but ping() in class Pubsub does, why?
https://github.com/valkey-io/valkey-py/blob/c490780a1d7b629317e5c0d976ec13dd64de8e55/valkey/client.py#L1079-L1084
ping() method has kwargs to allow extra options, but does this method really need it? if you add some extra options, it will raise a exception because of the callback in _ValkeyCallbacks

>>> from valkey import Valkey
>>> r = Valkey(host="<IP_ADDRESS>", port=6888, password='xxxxx')
>>> r.ping()
True
>>> r.ping("hello")
Traceback (most recent call last):
  File "<python-input-3>", line 1, in <module>
    r.ping("hello")
    ~~~~~~^^^^^^^^^
TypeError: ManagementCommands.ping() takes 1 positional argument but 2 were given
>>> r.ping(message="hello")
Traceback (most recent call last):
  File "<python-input-4>", line 1, in <module>
    r.ping(message="hello")
    ~~~~~~^^^^^^^^^^^^^^^^^
  File "/root/.pyenv/versions/env-3.13/lib/python3.13/site-packages/valkey/commands/core.py", line 1211, in ping
    return self.execute_command("PING", **kwargs)
           ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^
  File "/root/.pyenv/versions/env-3.13/lib/python3.13/site-packages/valkey/client.py", line 570, in execute_command
    response = conn.retry.call_with_retry(
        lambda: self._send_command_parse_response(
    ...<2 lines>...
        lambda error: self._disconnect_raise(conn, error),
    )
  File "/root/.pyenv/versions/env-3.13/lib/python3.13/site-packages/valkey/retry.py", line 62, in call_with_retry
    return do()
  File "/root/.pyenv/versions/env-3.13/lib/python3.13/site-packages/valkey/client.py", line 571, in <lambda>
    lambda: self._send_command_parse_response(
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
        conn, command_name, *args, **options
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ),
    ^
  File "/root/.pyenv/versions/env-3.13/lib/python3.13/site-packages/valkey/client.py", line 543, in _send_command_parse_response
    return self.parse_response(conn, command_name, **options)
           ~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.pyenv/versions/env-3.13/lib/python3.13/site-packages/valkey/client.py", line 600, in parse_response
    return self.response_callbacks[command_name](response, **options)
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^
TypeError: <lambda>() got an unexpected keyword argument 'message'
>>>

Is there any historical reasons to do so? Now i do some changes for ping:

ping():

    # in class ManagementCommands
    def ping(self, message=None, **kwargs) -> ResponseT:
        """
        Ping the Valkey server

        For more information see https://valkey.io/commands/ping
        """
        args = ["PING", message] if message is not None else ["PING"]
        return self.execute_command(*args, message=message)

add new callback for ping command:

# import safe_str
from valkey.utils import str_if_bytes, safe_str

def parse_ping(response, **options):
    response = str_if_bytes(response)
    message = "PONG" if options.get("message") is None else options.get("message")
    return response == safe_str(message)

_ValkeyCallbacks = {
    # ...
    # "PING": lambda r: str_if_bytes(r) == "PONG",
    "PING": parse_ping,

    # ...
}

then we can try the new ping() method:
>>> from valkey import Valkey
>>> r = Valkey(host="<IP_ADDRESS>", port=6888, password='xxxxx')
>>> r.ping()
True
>>> r.ping(0)
True
>>> r.ping("hello")
True
>>> r.ping(xxx=0)
True
>>> r.ping("abc", xxx=0)
True

maybe ping() method just use to test valkey server is alive or not, so there is no need to add extra message,  but why use **kwargs to receive extra params since the default callback will occurs exception?

Hi!
In general, we don't have a lot of insight about why certain things are done in a certain way, because we are not the original authors of most of the code in valkey-py. Your guess is as good as mine here.
Your proposal makes sense to me.
But I'd rewrite
 message = "PONG" if options.get("message") is None else options.get("message")

as
 message = options.get("message", "PONG")

Would you like to make a PR?

Hi! In general, we don't have a lot of insight about why certain things are done in a certain way, because we are not the original authors of most of the code in valkey-py. Your guess is as good as mine here.
Your proposal makes sense to me.
But I'd rewrite
 message = "PONG" if options.get("message") is None else options.get("message")

as
 message = options.get("message", "PONG")

there is a problem using message = options.get("message", "PONG"), because options.get("message") will get None when message is None, it will not be "PONG":
>>> data = {"message": None}
>>> data.get("message", "PONG") == None
True

Would you like to make a PR?

i will make a pr, thanks for your reply,  i'll continue to fix real bugs in valkey-py and try to make it better. now i only read valkey's command methods, so my opinion may not be right or even wrong, but i will think more before asking questions.

it was my fault to think that this issue was easy to do, i didn't understand the code well enough. adding a message that would affect the callback of the cluster connection in ping. Although I found the callback code , I thought it was unnecessary to modify both, so I closed my pr. I will test more before creating a pr in the future. thanks again  @aiven-sal

No problem!


Fix: Camunda 7 mapper & element template versioning

Issue

https://github.com/FlowSquad/miranum/issues/102
https://github.com/FlowSquad/miranum/issues/100

Description
Fixing these two issues from above and making sure that expressive names are used in the simple example.
Checklist

[x] Code is easy to understand
No obvious errors or bugs -> There is a bug with the message correlation where I need help on
[x]CI/CD Pipelines did run successfully
[x]Tests were implemented
[x]Documentation was created

@rami-nk or @dominikhorn93
I am facing problems when running message correlation in the simple-example process. I am very uncertain about the cause of it. Can you give it a try to see whether the exception persist on your machine?
Thanks a lot in advance!

Codecov Report
Patch coverage has no change and project coverage change: +0.20 :tada:

Comparison is base (465065d) 19.91% compared to head (2897f9a) 20.12%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #103      +/-   ##
============================================
+ Coverage     19.91%   20.12%   +0.20%
- Complexity       34       35       +1
============================================
  Files            78       78
  Lines           492      492
  Branches         10       10
============================================
+ Hits             98       99       +1
+ Misses          391      390       -1
  Partials          3        3

Impacted Files
Coverage Δ

...m/connect/adapter/in/c7/worker/Camunda7Mapper.java
79.41% <0.00%> (+2.94%)
:arrow_up:

...xample/application/port/in/SendMessageCommand.java
0.00% <0.00%> (ø)

...mple/application/service/FireAndForgetService.java
0.00% <0.00%> (ø)

...xample/application/service/SendMessageService.java
0.00% <0.00%> (ø)

...n/miranum/integrations/example/domain/Message.java
0.00% <ø> (ø)

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.

@rami-nk or @dominikhorn93 I am facing problems when running message correlation in the simple-example process. I am very uncertain about the cause of it. Can you give it a try to see whether the exception persist on your machine?
Thanks a lot in advance!

When trying to run it I can't start the process. I notices that you are using variables for the message and the key inside the element template but we are not starting the process from extern in this example at least. But after I changed it and tried to start a process I got the MessageCorrelationMismatch exception. I played a bit around and noticed, that this exception only occurs when we set the correlation key. If you omit the correlation key here, there is a new exception which finds the message but to correlate to, but at this stage there are more than one process it wants to correlate to and thats why it fails. You can set a flag in the messageCorrelationDto to correlate a message to all processes that have that message available. After trying that it works fine.
The problem seems to be, that there is no key present to correlate to. I mean the key is present when sending the messageCorrelationDto but it is clearly missing on the receiver side, because we only specify a message for the intermediate catching event. Hope this makes sense. I'm not sure what to do about this. Maybe just send the businesskey or something else to correlate to? Not sure why we need a correlation key here if it is not clearly set on the receiver side. But please correct me if I'm missing anything @dominikhorn93


misa.B behavior

The description of misa says:

When a standard extension is disabled by clearing its bit in misa, the instructions and CSRs defined or modified by the extension revert to their defined or reserved behaviors as if the extension is not implemented.

This B extension says:

When misa.B is 0, it indicates that the implementation may not support one or more of the Zba, Zbb, or Zbs extensions.

If software writes 0 to misa.B, what can it expect to happen?  Are all 3 of those extensions disabled or is it implementation-defined which subset are disabled?  The above seems to indicate the latter, though I would expect that if software is writing 0 then it would want to disable everything, or at least have some way of knowing what is disabled.
@ved-rivos

misa.B=1 indicates Zba, Zbb, and Zbs are available.
misa.B=0, one or more of these may not be available.
If misa.B can be written, and is set to 0, then what subset of extensions gets disabled is implementation-defined.

If misa.B can be written, and is set to 0, then what subset of extensions gets disabled is implementation-defined.

That's what I thought.  Yuck.
In a bigger picture way, I don't understand the motivation for this extension.  If we want "consistency and conciseness across toolchains" then why don't we also have a grouping for Zicbom_Zicbop_Zicboz which are also mandated by platforms?  Or V_Zvfhmin_Zvbb_Zvkt?  Or any number of other groups?  Shouldn't we consistently want consistency?

The bit manipulation extension was intended to be designated B and its presence reported through misa.B.  Clearing B may not lead to Zba/Zbb/Zbs being disabled as that is the backward compatible behavior. This extension is remedial in nature and should have actuall y happened as part of bit manip ratification.

In the days of bitmanip 0.93 and earlier there was a plan to have a B extension which was Zba, Zbb, Zbs, Zbc, Zbp, Zbe, Zbf, and Zbm.  But the latter 4 extensions were dropped and the plan for what was "informally" B was abandoned two years ago with https://github.com/riscv/riscv-isa-manual/commit/c1c77c4902d5e7c58613d725d09d66a2a743da1c

What were formerly informally referred to as the B and J extensions
are instead going to become multiple smaller and disjoint extensions.
Since it appears unlikely that B and J will ultimately have any
meaning, simply mark these misa bits as "reserved", rather than
"tentatively reserved for X".

It sounds like either this change was done in error or the plan has changed back again (or this is being done two years later to accommodate some people who jumped the gun on assuming a B extension like people jumped the gun with vector 0.7.1).  And the definition now also doesn't include Zbc so it doesn't match the past plan (so it wouldn't match the expectation of people who made premature assumptions).
Overall, I think that we should be moving away from relying on misa rather than adding more things to it.  It made sense when there might have only been <26 extensions but we're already far, far beyond that point.  Will the J extension be resurrected similarly?

There is no plan to do anything for J. The B is remedial in nature. The Zbc not being included is intentional since the carry less multiplication is not expected to be baseline bit manipulation capability needed by all implementation and especially ones that support Zvbc.

OK.  I just wanted to provide some pushback on why this is even useful at all and I wanted to point out that if misa.B is writable then writing it to 0 is of dubious value since you don't really know what's going to happen.  For the latter, the same could be said of misa.M and misa.A now that Zmmul and Zalrsc/Zaamo exist (or will soon), but that could not have been foreseen back when the M and A extensions were defined (unlike with B where we defined subsets before the superset).


Terraform Script not working

Hi Team,
I have added my aws access key & secret keys in github actions and tried to deploy AWSGoat with the terraform script. But I am facing the below error. Can someone help me in fixing this issue asap.

Thank you :)

^^ Facing the same issue. Can someone help.

Hi @hehacks , we checked the deployment it is working successfully as expected. The error may be caused by some preexisting resources in your AWS Account, the error should be shown in the Terraform Plan step, we can better debug your issue if you share the Terraform Plan step output from the Job output.


(partial) msvc support

some changes to support compilation with msvc120
I used .h and .cc files directly without cmake.
also line 2374 is probably wrong
so this is just the first step to support msvc.

Coverage remained the same at 100.0% when pulling 90865ccd20b25379e8aa1f102790e4b0cb808237 on Zoadian:master into c4d74856bbfc99ce788bebfac981377f5cfc6352 on nlohmann:master.

Hi @Zoadian, sorry for not answering earlier. Though I like the idea of having a Windows version, I am currently working on new code that uses even more C++11 features. When I look at your code, I see that you mostly use the preprocessor to exclude those parts that are not yet supported by Visual Studio. I think I would rather wait until Visual Studio has true C++11 support rather than to fragmentize the project like this - in the end, I am using this project to learn the new C++11 features.
I'm going to close the pull request here, but I would be happy if you would host a Windows-compatible version yourself. I would link to it in the README. You might also want to get in touch with @kirkshoop. Kirk opened a similar pull request #29 earlier.

Oh, I forgot. The error you mentioned is indeed a mistake. I will fix it soonish.


Add scipy to requirements.txt for r-extras docker

Submission type

[ ] This submission adds a new dataset
[ ] This submission adds a new method
[ ] This submission adds a new metric
[ ] This submission adds a new task
[ ] This submission adds a new Docker image
[ ] This submission fixes a bug (link to related issue: )
[ ] This submission adds a new feature not listed above

Testing

[ ] This submission was written on a forked copy of openproblems
[ ] Nextflow test pipeline is passing on this base branch of this pull
request (include link to passed test on NF Tower found in GitHub Actions summary: )
[ ] If this pull request is not ready for review (including passing the Nextflow test
pipeline), I will open this PR as a draft (click on the down arrow next to the
"Create Pull Request" button)

Submission guidelines

[ ] This submission follows the guidelines in our
Contributing document
[ ] I have checked to ensure there aren't other open Pull Requests for the
same update/change

PR review checklist
This PR will be evaluated on the basis of the following checks:

[ ] The task addresses a valid open problem in single-cell analysis
[ ] The latest version of master is merged and tested
[ ] The methods/metrics are imported to __init__.py and were tested in the pipeline
[ ] Method and metric decorators are annotated with paper title, year, author, code
version, and date
[ ] The README gives an outline of the methods, metrics and datasets in the folder
[ ] The README provides a satisfactory task explanation (for new tasks)
[ ] The sample test data is appropriate to test implementation of all methods and
metrics (for new tasks)

why do we need scipy in the r docker?


EL-1320: Part 3 - Speed up run_specsquick, tidy-up of docker file

Jira ticket
What changed and why

Tidy-up of docker file as part of this PR.

Guidance to review

Additional tidy-up

Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing
Branch is generally up to date with main Github - definitely no conflicts
No unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
PR description says what changed and why, with a link to the JIRA story.
Diff has been checked for unexpected changes being included.
Commit messages say why the change was made.

Not needed


Support builtins in @overload

Feature request

High-level extension APIs such as @overload should support builtin operators (e.g. setitem, in). Feature parity with low-level extension APIs in general is desirable.

#2297 is a step in that direction. This would allow to @overload anything that has a corresponding function in the operator module by overloading that function.
This PR is an attempt at the implementation. However, I am not sure the approach taken is the best choice. Required more invasive changes than I initially thought. Maybe starting at the other end by modifying  numba.templates.builtin_registry  and/or @overload such that @overload could act on builtin operators would have been smarter.
Also, the PR does not cover getitem, setitem etc,  yet (see numba.utils.OPERATORS_TO_BUILTINS from the PR for what is moved to operator module functions already).


FloatingActionButtonMenu with different icon

Is it possible to create a FloatingActionButtonMenu item with an icon that isn't a plus. Specifically I was hoping to use a vertical ellipsis like the one that would normally be seen in the ActionBar. I have tried looking through the source, but with no luck. Any help is appreciated!

:+1: Need this api, both from xml and programmatically

I would also like this

Duplicate of #56

This should work
   floatingActionMenu.setIconAnimated(false);

   floatingActionMenu.setOnMenuToggleListener(new FloatingActionMenu.OnMenuToggleListener() {
        @Override
        public void onMenuToggle(boolean opened) {
            if (opened)
            {
            		floatingActionMenu.getMenuIconView().invalidateDrawable
            		(
      		        floatingActionMenu.getMenuIconView().getDrawable()
      		        );
    			    floatingActionMenu.getMenuIconView().setImageBitmap(
            			BitmapFactory.decodeResource(
                    			getResources(), R.drawable.<YOUR_DRAWABLE_ID>
            					)
    				);
            }

            else
            {
            	floatingActionMenu.getMenuIconView().invalidateDrawable
            		(
      		        floatingActionMenu.getMenuIconView().getDrawable()
      		        );
    			    floatingActionMenu.getMenuIconView().setImageBitmap(
            			BitmapFactory.decodeResource(
                    			getResources(), R.drawable.<YOUR_DRAWABLE_ID>
            					)
    				);

            }

        }
    });


bug: Not view ETH balance but yes transactions

Bug Report
The balance ETH faucet does not appear after making the claim, it was with the Chrome browser and Metamask.
You can look at my account and see how I have created an erc-20 and it has been minted, but my ETH balance is 0.
Account
SC erc20 NAI

Also in my MM both tokens come out to 0, it is the same account and rpc that I used to deploy.

Steps to reproduce:

Add the automatic RPC from the button and manually checked with the doc
Add code access and address de ETH.
Overcome the captcha (pop op 0.05ETH received)
Create SC simple Wizard OZ and deploy and REMIX.
Mint NAI in Remix (Explorer Good with NAI total supply, mint...)

thanks for your issue @Nadai2010, we are looking into it

@Nadai2010 after investigation, we were out of Eth on the Faucet account! We funded it again and raised an issue for this


[ENH] Modelling seasonality with fourier terms

Is your feature request related to a problem? Please describe.
Seasonality can be modelled by using a matrix of fourier terms as exogenous data. The process is described here: https://robjhyndman.com/hyndsight/longseasonality/ this can be useful to include multiple seasonality in ARIMA models, where sarima is very slow to fit.
Describe the solution you'd like
A function that takes as input a DateTimeIndex, the fourier order desired and which seasonalities to include and outputs a matrix of fourier terms that can model seasonality effects when included as exogenous data.
If this feature already exists please let me know, I couldn't find anything upon a quick search.

that is this, no?
https://github.com/alan-turing-institute/sktime/pull/3374
should be in the next release
That's sktime, the time from issue to solution is negative :-)

@ltsaprounis, did we perhaps forget to add this to the API docs?

oh, no, it's there, but it's not on stable, only on latest docs

Good to know! Thanks @fkiraly, this is the feature I was looking for so I will close the issue.


Fix: Vertically Center Ratio Object Content

Update the ratio object to vertically align content by default. Prevents instances where nested image content alignment mis-matches with background container (ie. average background color / placeholder image doesn't match with the actual image itself).
Thanks @mikemai2awesome for pointing out this one to me!

Added to release notes for the next release: https://github.com/bolt-design-system/bolt/releases/tag/untagged-666ca5fe7284fcbbee93


Allow to provide defaultValue to hamt.modify and hamt.modifyHash

Provide any given defaultValue, via either hamt.modify or hamt.modifyHash, to function f if entry by key, key, doesn't exist.
Note: This changes the semantics of calling the transform function with no arguments when an entry doesn't exist with a given key.
This aligns with notSetValue semantics of Immutable.js. e.g. http://facebook.github.io/immutable-js/docs/#/Map

I have a branch with this change: https://github.com/dashed/hamt/tree/defaultValue
If interested in this change, I can flesh out with more tests and create a PR.

This sounds interesting but do I have a few thoughts on it.
modify is currently called with zero arguments in the case where m does not exist. For example:
var h = hamt.empty;

var f = function(x) {
    return arguments.length === 0 ? 0 : x + 1;
};

h = h.modify('key', f)  // { key: 0 }
h = h.modify('key', f)  // { key: 1 }

This is not perfect, not least because the safest approach outlined here does not work with lambda functions, but this is useful behavior for some cases because a single function handles everything (this is what get uses to do inserts).
I'm not entirely opposed to adding a defaultValue argument, however, the new behavior should only apply when a defaultValue argument is present. The argument order would probably be better expressed as:
export const modify = (f, key, defaultValue, map) => {
    if (arguments.length >= 4) {
       // new, defaultValue behavior
    } else {
       // current behavior
    }
};

As for the implementation, I believe it may be cleaner to handle defaultValue within the modify API and not spread the concept of defaultValues into the internal  _modify implementations.
Let me know what you think.

... however, the new behavior should only apply when a defaultValue argument is present.

I think we can compromise on this; I created a preliminary unfinished PR to demonstrate this https://github.com/mattbierner/hamt/pull/19


Move chain sync-to-genesis to participating stage

This PR modifies the chain sync-to-genesis process to run once the node has transitioned to participating mode.
Some details/fixes worth highlighting are:

an immediate switch block after an emergency upgrade is now signed by the validators for the current era rather than the previous era.
if the last switch block before the last emergency upgrade is used as the trusted block (either explicitly via the trusted hash config option, or implicitly by a node started with its storage having that as its highest block), fast sync is now continued after the commit_upgrade has been performed.
the restriction on using only non-joiners when fetching during chain sync has been limited to trie-fetching only.  Restricting for all fetched data types was problematic as, when the network is going through an upgrade, all nodes can be joiners concurrently, meaning no data can be fetched.
the decision as to whether to commit upgrade, commit genesis or fast-sync is now solely made in the run_fast_sync_task
the metrics chain_sync_emergency_restart_duration_seconds and chain_sync_upgrade_duration_seconds have been removed as I don't believe they were particularly helpful

Closes #2487.

bors merge

bors merge

bors merge

bors merge


Couldn't stop the toggl timer

Hey Ben, first of all, thanks for the automation script!
I installed the latest release and I am able to start the timer.
the icon and 'working-on' context showed up. but I can't stop the timer, no response whatsoever on OF and toggl side both.

I did more tests,  my previous  problem statement is not accurate. apology for that.
for some reasons, omnifocus is not responding to my action (start/stop timer), i had to restart omnifocus all the time.
Steps:
Start timer in OF,  can't stop it,
restart OF, then I am able to stop it. but I can't start timer again.
restart OF, able to start timer

never mind, my computer error.
apology for the noise. & Thanks a lot!


Change the information appearing in a question when CP is hidden

Current behavior
When the CP is hidden, either because of user settings or because of the hidden period at the beginning of the question, my own forecasts don't appear anywhere besides the prediction interface.
Feed:

Question page:

Expected/desired behavior
I would prefer if my current forecasts were visible from the feed, so that I can easily 1) know which questions I've forecasted on, 2) see what my forecasts are and if I no longer agree with them, without needing to open one tab per question.

I don't care about the number of the forecasters at all or the date that much and divergence shouldn't be showing anyway given that the CP is hidden. I only want the number 86% to appear anywhere in this box.
I would also like to see my past forecasts on the question page so that I know how I ended up where I am, when was the last time I updated, and by how much, without needing to either reveal the community prediction or wait for it to be revealed. Both the dots and the number below are essential.

Ready for testing on dev

Thank you for implementing this! A few comments:
It would be nice if I could see the numbers of my predictions when mouseovering the graph in multiple choice and group questions, which I'm not able to do right now:

For group and multiple choice questions I should be seeing the numbers of my predictions, besides the graph, from the main feed:

The CP being hidden from all should function basically the same as me turning it off from my settings (besides the "reveal temporarily" button ofc). Right now, when the CP is hidden I see what I was seeing before:

No graph when I open the question or in the main feed
No way to tell I've predicted on the question from my main feed

Very minor, but when I haven't predicted on a question and I have the CP hidden I'm shown an empty graph in the main feed:

It'd probably be nicer if I wasn't.

Is this the right place to add that we should remove "Forecast data empty" from question tiles on feeds?

Looks like that was fixed for upcoming questions but not hidden questions, agree that should be fixed! Either with a nicer message about the hidden period or showing nothing there. https://github.com/Metaculus/metaculus/issues/738

There are two more things that could be improved when the user has hidden the CP:

Mouseovering the graph should show the user's forecasts (I mean the actual numbers)

All questions should show the user prediction from the main feed, right now group and multiple choice questions don't

The second bullet point is indeed more of a design question, and I leave it to @aseckin: do we show the number of forecasters and user predictions in the feed tile for groups and multiple choice questions, and if so where. My two cents is that it would be nice, but not worth making the tiles too ugly.
The first bullet point is simpler: when the user hides the CP, and they hover over the timeline on the question page, they should see the popup with their predictions, but the CP blanked, like in the table below.

current
desired

The popup works!
But, I noticed the timeline is still hidden on groups that have a fan graph, and on conditionals. Example group here and conditional here for me (105951):

@SylvainChevalier pushed a fix to dev

LGTM! Timelines showing for fan graphs and conditionals with my forecasts. Looks like it's already on prod.


Incorrect checkbox checked state

In short, some of the checkboxes in a list are being rendered as checked, despite the fact that their checked attribute has been set to False in Elm.
Steps to reproduce:

Have a list of records with checked : Bool attribute.
Split the list using List.filter .checked and List.filter (not << .checked) before rendering. ( List.sortBy (\i -> if i.checked then 0 else 1) works as well).
Concatenate the checked and unchecked items together to form a new list. (to get checked items in the front of the unchecked)
Map over the results list in a view function and render them in a list with checkboxes for each of the record. (with state of the checked attribute in the checkbox dependent on the .checked record value).
Click on a couple of the checkboxes.

Expected behavior:
Clicked-on checkbox should become checked and move to the top of the list.
Other unchecked items should be placed under it.
Actual behavior:
Clicked-on record updates its .checked attribute, changes its checkbox state to checked and moves to the top.
But after it's been moved to the top, other checkbox input element also acquires checked state. (despite the fact that its corresponding record have checked = False).
Relevant code snippet:
import Html exposing (Html, div, input, text, ul, li, label)
import Html.Attributes as A exposing (class, property, classList)
import Html.Events exposing (onCheck, onClick, targetChecked, on, onWithOptions)
import Json.Decode as Json

type alias Model =
  List { name : String, checked : Bool }

type Msg
  = Toggle String Bool

main : Program Never Model Msg
main =
  Html.program
    { init = init
    , view = view
    , update = update
    , subscriptions = (\_ -> Sub.none)
    }

init : ( Model, Cmd Msg )
init =
  ( [ { name = "Apple", checked = False }
    , { name = "Orange", checked = False }
    , { name = "Grapes", checked = False }
    , { name = "Bananas", checked = True }
    , { name = "Mangos", checked = False }
    ]
  , Cmd.none
  )

update : Msg -> Model -> ( Model, Cmd Msg )
update msg model =
  case msg of
    Toggle name checked ->
      let
        toggle item =
          if item.name == name then
            { item | checked = checked }
          else
            item
      in
        ( List.map toggle model
        , Cmd.none
        )

view : Model -> Html Msg
view list =
  let
    checked =
      list
        |> List.filter .checked

    unchecked =
      list
        |> List.filter (not << .checked)

    sorted =
      checked ++ unchecked
  in
    div []
      [ ul [] (List.map listItem sorted)
      ]

listItem item =
  let
    name =
      case item.checked of
        True ->
          item.name ++ " checked!"

        False ->
          item.name

    toggle checked =
      Toggle item.name checked
  in
    li []
      [ input
          [ A.type_ "checkbox"
          , A.checked item.checked
          , A.value item.name
          , A.name "fruit"
          , A.id item.name
          , onCheck toggle
          ]
          []
      , label
          [ A.for item.name ]
          [ text name ]
      ]

I tried to always set A.checked to False, but inputs are still being rendered as checked.
Also I've tried to use onWithOptions with preventDefault = True, but it didn't work either.
--
Browsers: tested in Safari 10, Chrome 55, Firefox 50.
Elm: standalone Elm 0.18.0 and http://elm-lang.org/try

Looks like you need to use the functionality of Html.Keyed.
If the description/API there is not clear, ask on the mailing list.

Note that the documentation in Html.Keyed only speaks about optimization and efficiency. But that is actually only part of the story. The use of Html.Keyed is also needed for correctness (instead of efficiency), exactly in cases like yours. That's been discussed a lot on the mailing list. I don't know why the documentation does not mention it. (I've brought it up as a problem about the documentation before.)

@jvoigtlaender Thank you very much! I had no idea that Html.Keyed even existed.
Maybe Elm should put a warning when you try to use ordinary html functions in such cases? Because it's a very weird bug that is hard to google about.


Wrong state of SquashedTable in SHOW PROCESSLIST

6639/113 doesn't look correct:
Command: query
   Time: 2373
  State: SquashedTable(ref_commits, commits, commit_files)(6639/113), blobs(0/113)
   Info: SELECT DISTINCT LANGUAGE(cf.file_path, b.blob_content) AS lang
FROM blobs b
NATURAL JOIN commit_files cf
NATURAL JOIN commits c
NATURAL JOIN ref_commits r
WHERE r.ref_name = 'HEAD'
AND r.history_index = 0
AND is_binary(b.blob_content) = false
AND cf.file_path NOT REGEXP '^vendor.*'
AND lang != ""

$ ./gitbase version
gitbase (v0.18.0-beta.3) - build 11-27-2018_11_31_37

Easily reproducible for me. Tried with empty everything.
Use the script above to get repositories.
Then:
mysql> select count(*) from repositories;
+----------+
| COUNT(*) |
+----------+
|      100 |
+----------+
1 row in set (0.01 sec)

mysql> SELECT DISTINCT LANGUAGE(cf.file_path, b.blob_content) AS lang
    -> FROM blobs b
    -> NATURAL JOIN commit_files cf
    -> NATURAL JOIN commits c
    -> NATURAL JOIN ref_commits r
    -> WHERE r.ref_name = 'HEAD'
    -> AND r.history_index = 0
    -> AND is_binary(b.blob_content) = false
    -> AND cf.file_path NOT REGEXP '^vendor.*'
    -> AND lang != ""
    -> ;

PROCESSLIST shows 299/100.
mysql> SHOW PROCESSLIST \G;
*************************** 1. row ***************************
     Id: 3
   User: root
   Host: <IP_ADDRESS>:52412
     db: gitbase
Command: query
   Time: 25
  State: SquashedTable(ref_commits, commits, commit_files)(299/100), blobs(0/100)
   Info: SELECT DISTINCT LANGUAGE(cf.file_path, b.blob_content) AS lang
FROM blobs b
NATURAL JOIN commit_files cf
NATURAL JOIN commits c
NATURAL JOIN ref_commits r
WHERE r.ref_name = 'HEAD'
AND r.history_index = 0
AND is_binary(b.blob_content) = false
AND cf.file_path NOT REGEXP '^vendor.*'
AND lang != ""
*************************** 2. row ***************************
     Id: 6
   User: root
   Host: <IP_ADDRESS>:52412
     db: gitbase
Command: query
   Time: 0
  State: running
   Info: SHOW PROCESSLIST
2 rows in set (0.00 sec)

ERROR:
No query specified

After few minutes 1399/100:
*************************** 2. row ***************************
     Id: 3
   User: root
   Host: <IP_ADDRESS>:52412
     db: gitbase
Command: query
   Time: 125
  State: SquashedTable(ref_commits, commits, commit_files)(1399/100), blobs(0/100)
   Info: SELECT DISTINCT LANGUAGE(cf.file_path, b.blob_content) AS lang
FROM blobs b
NATURAL JOIN commit_files cf
NATURAL JOIN commits c
NATURAL JOIN ref_commits r
WHERE r.ref_name = 'HEAD'
AND r.history_index = 0
AND is_binary(b.blob_content) = false
AND cf.file_path NOT REGEXP '^vendor.*'
AND lang != ""

The PR that enables auto in memory joins on small tables will mitigate this to a certain extent, but I don't think this can be solved in any way as long as we do multi pass on joins. There's just no way to know ahead of time the maximum number. It's a very confusing thing, but not one we can solve to the best of my knowledge.
Perhaps we should think of a way to display progress that will not incur in this. What if instead of showing X/Y we show just the number of partitions iterated? You will not know when it will end, but it serves the purpose of knowing if it's stuck and where it's stuck more or less. If we keep it this way, 90% of the time (when it's not in-memory) it's not gonna work.

IMO, any changing counter which can show some progress would be good enough (and maybe less confusing as proportion)


Relative Path Produce undefined control sequneces under windows

https://github.com/nschloe/matplotlib2tikz/blob/73d5db7361f8dd6af787ba6801bf140c4524b766/matplotlib2tikz/files.py#L42
Using os.path.join, under windows use the path separator "" when this appear in the latex output its treated as a control sequence, which normally produces the Tex error "! Undefined control sequence."
Under latex the path should always use "/" as the separator.
You can use posixpath.join under windows still, which should fix the problem.

@glenflet Can you check that #246 solves this issue ? I currently don't have a Windows machine for testing.


nameclash with 'python-gitlab'

there's another project that provides access to the gitlab API for Python, named python-gitlab
https://github.com/python-gitlab/python-gitlab

both this project and the other one use the same import name
import gitlab

unfortunately, the two projects have a different API...
I don't really know nor care which of the two projects has a deeper claim on the name, but i'd suggest the two projects coordinate the namespace (most likely renaming this).

i've also opened a ticket on the "other" project: https://github.com/python-gitlab/python-gitlab/issues/385

@umlaeute I'm with @gpocentek. This is a bad situation for both libs. I'm finding it really hard to get time for this project as is.
If we change the namespace it will negatively affect a Debian package as well as everyone that currently uses the lib. It would be like Debian changing the python3-gitlab package from this one to python-gitlab. It would affect businesses in a negative way that use the package/lib for there project.
Side note. I would do it if this project was just starting out.


Carthage BUILD FAILED

Xcode: Version 8.2.1 (8C1002)
macOS: 10.12.2 (16C67)
Carthage: 0.18.1
Swift: 3.0
use carthage update --no-use-binaries --platform ios
The following is an error
/Users/Broccoli/Desktop/Demo/Carrot/Carthage/Checkouts/NEKit/src/RawSocket/NWUDPSocket.swift:58:25: error: value of optional type '[Data]?' not unwrapped; did you mean to use '!' or '?'?
A shell task (/usr/bin/xcrun xcodebuild -project /Users/Broccoli/Desktop/Demo/Carrot/Carthage/Checkouts/NEKit/NEKit.xcodeproj -scheme NEKit-iOS -configuration Release -sdk iphoneos ONLY_ACTIVE_ARCH=NO BITCODE_GENERATION_MODE=bitcode CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES clean build) failed with exit code 65:
** BUILD FAILED **

The following build commands failed:
	CompileSwift normal arm64
	CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
(2 failures)

Have you tried master branch?

@xiao99xiao yes

@xiao99xiao Just at the moment, I used my colleague's mac build sucessed

Make sure you are using the latest Xcode

Why was this closed @zhuhaow ?
@broccolii did you have a solution? I seem to be getting the same error. Using the latest Xcode - 8.2.1
/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaAsyncSocket/Source/RunLoop/AsyncSocket.m:4282:38: warning: 'AsyncSocket' is deprecated: The RunLoop versions of CocoaAsyncSocket are deprecated and will be removed in a future release. Please migrate to GCDAsyncSocket. [-Wdeprecated-declarations]
/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaAsyncSocket/Source/GCD/GCDAsyncSocket.m:1985:37: warning: multiple methods named 'initWithDelegate:delegateQueue:socketQueue:' found [-Wstrict-selector-match]
/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaAsyncSocket/Source/RunLoop/AsyncUdpSocket.m:195:17: warning: Implementing deprecated class [-Wdeprecated-implementations]
/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaAsyncSocket/Source/RunLoop/AsyncUdpSocket.m:2307:3: warning: 'AsyncUdpSocket' is deprecated: The RunLoop versions of CocoaAsyncSocket are deprecated and will be removed in a future release. Please migrate to GCDAsyncUdpSocket. [-Wdeprecated-declarations]
/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaAsyncSocket/Source/RunLoop/AsyncUdpSocket.m:2307:41: warning: 'AsyncUdpSocket' is deprecated: The RunLoop versions of CocoaAsyncSocket are deprecated and will be removed in a future release. Please migrate to GCDAsyncUdpSocket. [-Wdeprecated-declarations]
<module-includes>:1:1: warning: umbrella header for module 'CocoaLumberjack' does not include header '/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaLumberjack/Classes/CLI/CLIColor.h'
<module-includes>:1:1: warning: umbrella header for module 'CocoaLumberjack' does not include header '/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaLumberjack/Classes/CLI/CLIColor.h'
<module-includes>:1:1: warning: umbrella header for module 'CocoaLumberjack' does not include header '/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaLumberjack/Classes/CLI/CLIColor.h'
<module-includes>:1:1: warning: umbrella header for module 'CocoaLumberjack' does not include header '/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaLumberjack/Classes/CLI/CLIColor.h'
<module-includes>:1:1: warning: umbrella header for module 'CocoaLumberjack' does not include header '/Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/CocoaLumberjack/Classes/CLI/CLIColor.h'
warning: SwiftLint not installed, download from https://github.com/realm/SwiftLint
<module-includes>:1:1: warning: umbrella header for module 'CocoaLumberjack' does not include header 'CLIColor.h'
/Users/jorda** BUILD FAILED **

The following build commands failed:
        CompileSwift normal x86_64
        CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
n/Desktop/testingNEKit/Carthage/Checkouts/NEKit/src/RawSocket/NWUDPSocket.swift:58:25: error: value of optional type '[Data]?' not unwrapped; did you mean to use '!' or '?'?
A shell task (/usr/bin/xcrun xcodebuild -project /Users/jordan/Desktop/testingNEKit/Carthage/Checkouts/NEKit/NEKit.xcodeproj -scheme NEKit-macOS -configuration Release ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES clean build) failed with exit code 65:
** BUILD FAILED **

The following build commands failed:
        CompileSwift normal x86_64
        Compile

Try the latest version, 0.11.1

@zhuhaow thanks, that seemed to fix it!


allure version 2.0.0 onwards does not generate full spec test report

Hi,
I am using allure-report to generate my test report. In my framework we are using wdio-allure-report package. Automation script are running parallel on multiple browser like, Safari, Chrome, Firefox etc.  Till the allure-commandline version 1.5.0 it generate the consolidated report for all browser like - capabilities: chrome, FF etc. Lets say I have 10 test cases running on three browser simultaneously, I am getting of 30 test case result as consolidated  one till the allure-commandline version 1.5.0. in the allure index.html page. But  after upgrading to the<EMAIL_ADDRESS>and till the 2.3.3, it does not generate  the consolidated reports of all browser where as it only shows one browser execution report which contains results of only 10 test cases instead of 30 test cases.
Also another issue is .. in order to generate the allure report, the allure-commandline version 1.5.0 and above  requires to install allure-js-commons": 1.2.1 version.
Could you please check and resolve this issue.

Any update on this? I have the same problem.

Thats is a common issue with test environments. Such parameters are not included to historyId (such attribute used for history & retries). You can find missed tests at retries tab.
As workaround you need to provide such variables as parameters to the report (by adding them to each test). BTW not sure is it possible via wdio reporter api.
We are going to solve the problem introducing environments feature https://github.com/allure-framework/allure2/issues/130


RunProxy Uses Default Timeout

From @Tratcher on Thursday, March 30, 2017 9:41:02 PM
From @AceCoderLaura on March 31, 2017 3:56
When using a proxy to redirect calls from ASP to my WCF service, timeouts occur. I suspect that the proxy's HttpClient is using the default request timeout. There needs to be a way to set the timeout.
app.MapWhen(c => IsWCFCall(c), builder =>
{
    builder.RunProxy(new ProxyOptions { Scheme = "http", Host = "localhost", Port = XivicConfiguration.WCFPort.ToString() });
});

Apologies if this is the wrong repository.
Copied from original issue: aspnet/Home#1981
Copied from original issue: aspnet/Proxy#58

From @AceCoderLaura on Thursday, April 6, 2017 4:45:03 PM
Update: Setting the back channel message handler and setting the cancellation token to always be not cancelled fixes the problem. Is there a less hacky way to stop whatever is cancelling the connection?
    internal class ProxyMessageHandler : HttpClientHandler
    {
        protected async override Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
        {
            return await base.SendAsync(request, new CancellationToken(false));
        }
    }

From @AceCoderLaura on Thursday, April 6, 2017 6:05:02 PM
There seems to be a "WebSocketKeepAliveInterval" property but it's not in the latest NuGet package. When's the next release?

From @AceCoderLaura on Thursday, April 6, 2017 7:06:34 PM
I ended up making my own proxy middleware and setting the timeout on the httpclient to a larger timespan. There really should to be a way to set it on the out-of-the-box proxy if there isn't already.