Typo in error when failing to connect to Console

Please provide us with the following information:

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
I recently had a container app where connecting to the console via the portal  failed. The error (below) includes the typo Kubsernetes:
{"Error":{"Code":"ClusterExecEndpointWebSocketConnectionError","Message":"Failed to establish WebSocket connection with the container, error is Failed to establish WebSocket connection with Kubsernetes cluster exec endpoint, error: The server returned status code \u0027500\u0027 when status code \u0027101\u0027 was expected..","Details":null,"Target":null,"AdditionalInfo":null}}

Steps to reproduce

.. With a container that fails to start, attempt to connect to the console via the portal

Expected behavior [What you expected to happen.]
Expect the error to include Failed to establish WebSocket connection with Kubernetes cluster exec endpoint
Actual behavior [What actually happened.]
The actual error includes Failed to establish WebSocket connection with Kubsernetes cluster exec endpoint
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
n/a

Thank you for reporting this! Fix has been rolled out. Please create a new issue if you continue to experience difficulties.


Feat/target_function containing kwargs support

Before submitting

[ㅇ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ㅇ] Did you make sure to update the docs?
[ㅇ] Did you write any new necessary tests?

What does this PR do?
Fixes #5.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Sure !

@jungbaepark Great, thank you!


Bearer_token help!

How do you get the bearer token retrieved from the HQ Trivia API? Do you need to get this before every game or is it the same for each game? Add me on discord if you want to chat there Cheesy #8556. Thanks

Hi @Cheezizy, I recommend you take a look at Charles proxy.
This will allow you to proxy traffic coming from your device, and you'll be able to find the bearer that is sent within each request. There should be plenty of tutorials online. Thanks!


org.owasp:dependency-check-maven

I'm using dependabot to manage dependency vulnerabilities .
Currently, org.owasp:dependency-check-maven is not needed.


Cannot insert entity into DB using attach and EntityState.Added

I am getting following exception when trying to insert entity by attaching and changing its state.
Microsoft.EntityFrameworkCore.DbUpdateException occurred
  HResult=0x80131500
  Message=An error occurred while updating the entries. See the inner exception for details.
  Source=Microsoft.EntityFrameworkCore.Relational
  StackTrace:
   at Microsoft.EntityFrameworkCore.Update.ReaderModificationCommandBatch.Execute(IRelationalConnection connection)
   at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.Execute(Tuple`2 parameters)
   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](Func`2 operation, Func`2 verifySucceeded, TState state)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChanges(IReadOnlyList`1 entriesToSave)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChanges(Boolean acceptAllChangesOnSuccess)
   at Microsoft.EntityFrameworkCore.DbContext.SaveChanges(Boolean acceptAllChangesOnSuccess)
   at EFCoreTest.Program.Main() in C:\Users\Dopare\documents\visual studio 2017\Projects\EFCoreTest\EFCoreTest\Program.cs:line 22

Inner Exception 1:
SqlException: Cannot insert explicit value for identity column in table 'Persons' when IDENTITY_INSERT is set to OFF.

Steps to reproduce
Sample repo
using (var db = new AppDbContext())
{
    var person = new Person
    {
        FirstName = "FirstName",
        LastName = "LastName",
        BirthDate = DateTime.Now
    };

    db.Persons.Attach(person);
    db.Entry(person).State = EntityState.Added;

    db.SaveChanges();
}

Further technical details
EF Core version: 1.1.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10 x64
IDE: (e.g. Visual Studio 2015)

@dopare Attaching the entity means that the entity already exists in the database. This means that no key generation is done--because the key must already have had a value assigned to be saved int the database. On the other hand, if Add is called, then it means that the entity is new and has not yet been saved to the database. In this case key generation will kick in, ultimately using the Identity column in the database
In this case, because the entity was first Attached, and then later changed to Added, it is not getting key generation. However, we are considering changing this so that even if the entity is first Attached, then later when it is made Added it will get the same key generation that would have happened if it was made Added in the first place.


Adding support for variables set for snmp trap mibs #547

Adds support for variables in snmp trap oids. Allows a profile to look like:
traps:
  # Traps from Visual Data Center (VDC), formerly owned by Optimum Path
  - trap_oid: <IP_ADDRESS>.4.1.345<IP_ADDRESS>.6.1
    trap_name: vdcAlarms
    drop_undefined: true
    events:
      - name: vdcAlarmId
        OID: <IP_ADDRESS>.4.1.345<IP_ADDRESS>.<IP_ADDRESS>
      - name: vdcMonAttribWild
        OID: <IP_ADDRESS>.4.1.34510.22.*
      - name: cbgpPeerLastErrorTxt
        OID: <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.1.7.{bgpPeerRemoteAddr:*}
      - name: chsrpTrapVarBing
        OID: <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.5.{ifIndex:1}.{cHsrpGrpTable:1}

Closes #547

Hello @i3149 is this fix already available / released to the final users? Is there any documentation on how to apply it to a existing environment? When looking at the releases page https://github.com/kentik/ktranslate/releases I see the last release was in Dec 2022. Can you please help to clarify that? thanks a lot


Screen share not working

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating System

[ ] macOS
[ ] Windows
[X] Linux

Operating System Version
Ubuntu 22.04
It happens on the web browser too?
No, it just happens on the Desktop app
Rocket.Chat Desktop App Version
3.9.9
Rocket.Chat Server Version
6.5.0-rc.18
Describe the bug

Just showing this loader for the past 20min. I am not being able to share my screen.
How to Reproduce
Just make a video call in any group and try to share screen.
Describe your Expected behavior
I should be able to share my screen.
Anything else
No response
Are you willing to submit a code contribution?

[X] Yes, I am willing to submit a Pull Request!

Please try our latest version 3.9.11.
There is a button on configurations to clean permissions for screen sharing that you can try too:

@jeanfbrito There is only a 3.9.9 version available in the Ubuntu software store. Also when I check your docs it tells me to install using the command sudo snap install rocketchat-desktop. It will also install the same version as 3.9.9. How do I install the version 3.9.11? Never mind I have cloned the repo and running the server using yarn start. The screen sharing works there with a minor issue. I will raise an issue related to it.

Shall we close this issue?

same thing happens on Mac OS, no matter if I install App Store version or via home-brew. Clearing Screen Capture Permissions does not do anything. Manually allowing screen capture in settings also doesn't help. Colleagues on Linux and Windows are experiencing the same issue.

@pitwegner can you share a screen capture of what is happening? Maybe opening a new issue would be good as the author of this one said to closed this one. I need more information to help you.


Update Issue - 15703535

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 77.0

Operating System
Windows 7

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


CI Builds for Debian 12 Failing

Description
CI builds for Debian 12 are consistently failing at the same point.
https://ci.swift.org/job/oss-swift-package-debian-12-aarch64/
Reproduction
N/A
Expected behavior
Successful builds
Environment
Debian 12
Additional information
The issue is being caused by being unable to find the host Swift install for bootstrapping.
See build logs -
-- Warning: {} Host toolchain could not locate a compiler to build swift-driver.
This problem seems to be unique to Debian Docker containers being started via ssh e.g. a Jenkins pipeline build.
When the container is started via ssh the saved env PATH is overwritten.
The solution is to add the PATH to /etc/profile
Added fix here - https://github.com/swiftlang/swift-docker/pull/397

@shahmishal


Fix socket leak in custom IO manager client (python/gevent)

This PR fixes the socket leak observed in the issue https://github.com/grpc/grpc/issues/27407
I've rebuilt the python grpc package with this PR included, and was not able to reproduce the leak.
The PR move the socket->refs decrement operation to the alarm and connect callbacks so it's called twice. If the connection is successful, we don't expect the socket to be destroyed as refs are increased on creation of the custom_tcp_endpoint : https://github.com/grpc/grpc/blob/4c40ee3f78bafcada42b8ac30570a6a327759ace/src/core/lib/iomgr/tcp_custom.cc#L367 and this code should continue tracking the lifecycle of the socket as show in https://github.com/grpc/grpc/blob/4c40ee3f78bafcada42b8ac30570a6a327759ace/src/core/lib/iomgr/tcp_custom.cc#L307
If the connect fails the socket object would be cleaned up (socket->refs would decrease from 2 to 0).
It's also possible to initialise socket->refs to 1 which should fix the issue without any side effect, but IMHO it would make the code slightly less clear, as conceptually we hold two or three references to socket (alarm callback, connect callback and tcp endpoint if the connection succeeds).
As this is my first contribution to the codebase, I wasn't sure how to test this change and whether unit/integration tests were needed. Happy to add some tests if you can provide me a few pointers to the best way of testing this change!
@yashykt from past contributions on the repo, it seems this also falls in the scope of @lidizheng ?

@gnossen a gentle bump on this PR, please let me know if I can do anything to help move it forward 🙏

Thanks a lot! It looks like the ObjC iOS test failure is a timeout, and, I think, should not be related to this PR

Known failure


Add support for BlueMap

As Minecraft Overviewer is in a bit of an unknown state at the moment, it would be good to also support alternatives such as BlueMap.

Bluemap works perfectly with MSCS already, you can place it in your mods directory as any other mod and follow their documentation https://bluemap.bluecolored.de/wiki/


Convert app to sveltekit and port over gallery functionality from webnative-app-template

Since we're only able to use MetaMask for the foreseeable future, we don't really need RainbowKit anymore, which means we can 🪓 React from this app. I'm going to replace it with sveltekit and implement the gallery functionality from webnative-app-template, but using a DDD approach so it's easier for people to remove the gallery code if they don't need it.
The newest version of sveltekit doesn't assume all files in the routes directory are actually routes(the file names need to have a + at the beginning to be interpreted as routes), so I can create a gallery directory in there and use it as the domain for this functionality 👍🏼

https://github.com/webnative-examples/walletauth


hyper is not compatible to GGUF model!!

Bytedances: 8-steps and 16-steps FLUX.1-dev-related LoRAs are available now
I just tried the 8-steps hyper lora to GGUF...seems it's not compatible

i think you have to ask the "hyper profider to support it"

after I upgrade the comfyui and this node, seems the hyper lora is supported
I don't know why, some lora loader node is no effect
and now seems the lora has effect on, but the cmd does not show any log of loading of lora (no matter it's success or not)

is Q lora supported ?@kiron

Q lora

you mean does flux1-dev-bnb-nf4 support LORA?
or it means other thing?
All I heard is the nf4 models do not support lora yet, coz I use GGUF only, I cannot tell if the current situation.

What issues are you seeing? Not sure if relevant but it does say this in the description:

With that set it does sample for me at least.

Yes, the 8 step lora it's working for me, without errors.
The FP8 results are similar to Q4_K_S in terms of quality ( although not as good as a normal 20 step generation ).

I think that's because of this: https://github.com/city96/ComfyUI-GGUF/issues/33

Yes, the 8 step lora it's working for me, without errors.
The FP8 results are similar to Q4_K_S in terms of quality ( although not as good as a normal 20 step generation ).
 I think that's because of this: #33

can you share the workflow with the lora ?

Can you share the workflow with the lora ?

You can use the [ LoraLoaderModelOnly ] node.
The model creator recommends a strength of 0.125
https://huggingface.co/ByteDance/Hyper-SD
The issue with the Q8_0 model was fixed.

Closing since I *think* this should work on latest without issues? Reopen if not.


Dependency updates July 2019

<EMAIL_ADDRESS>update in dependencies - to ensure that the recent cordova-common update, with the recent fs-extra@8 update, will be installed even when the user would upgrade from<EMAIL_ADDRESS>using npm
fs-extra@8 update in devDependencies
nyc@14 update in devDependencies - resolves a non-production npm audit issue

AppVeyor CI build is green for these changes on my fork.
I would normally have proposed these changes in separate PRs, don’t have so much time to do this right now. If approved, I think a merge commit should be used to merge these changes into master.
I hope we can get these changes reviewed, approved, and merged before we make the<EMAIL_ADDRESS>release.

Codecov Report

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

@@          Coverage Diff           @@
##           master    #356   +/-   ##
======================================
  Coverage    73.6%   73.6%
======================================
  Files          17      17
  Lines        2118    2118
======================================
  Hits         1559    1559
  Misses        559     559

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 b236887...87aa340. Read the comment docs.

Used "rebase and merge" to avoid the merge commit.


Move the RAG content image reference for Konflux to the build.args file

Description
Move the RAG content image reference to the build.args file. This file contains reference to the RAG content image as built by Konflux and will be kept up to date by nudges from the lightspeed-rag-content Konflux component.
For Prow CI and local environments, the RAG content image reference used by the service-api image build remains the same and is taken from the Containerfile. The get-rag make target always takes the RAG content image reference from the build.args file.
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue # https://issues.redhat.com/browse/OLS-484
Related PR: https://github.com/openshift/lightspeed-rag-content/pull/67
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

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

Project coverage is 95.61%. Comparing base (9ebbaea) to head (b11adb1).
Report is 11 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1226      +/-   ##
==========================================
- Coverage   95.64%   95.61%   -0.03%
==========================================
  Files          56       56
  Lines        2410     2394      -16
==========================================
- Hits         2305     2289      -16
  Misses        105      105

/retest

/retest

/lgtm

/approve


MudBlazor v8.0.0-preview.4 - DatePicker Issue

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudDatePicker
What happened?
There is an issue how the display for the date picker when opening with year selection. See the differences below:

Expected behavior
Please have a look to the screenshot. Left side is the correct behavior.
Reproduction link
https://dev.mudblazor.com/components/datepicker#different-views
Reproduction steps
Try to open a datepicker on the web page DatePicker in the section Different views.
Relevant log output
No response
Version (bug)
8.0 Preview 4
Version (working)
7.15.0
What browsers are you seeing the problem on?
Firefox, Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi,
Thanks for the report, but this bug report caused me to waste a lot of time due to the inaccurate information provided. It works in 8.0 Preview 4, the issue is that it doesn't work with the latest dev changes (that are also on dev.mudblazor.com).
Please, when submitting a report, double-check your information.
@Anu6is @versile2, this is a regression from https://github.com/MudBlazor/MudBlazor/pull/10215.
Also, very important is that this bug only shows up when you have a big browser size only.

Hi,
Thanks for the report, but this bug report caused me to waste a lot of time due to the inaccurate information provided. It works in 8.0 Preview 4, the issue is that it doesn't work with the latest dev changes (that are also on dev.mudblazor.com).
Please, when submitting a report, double-check your information.
@Anu6is @versile2, this is a regression from #10215. Also, very important is that this bug only shows up when you have a big browser size only.

The website I'm developing runs on version 8.0 Preview 4 and I'm experiencing this problem (see the screenshot below). This is therefore accurate information. In any case, the most important thing is to fix the problem.

The website I'm developing runs on version 8.0 Preview 4 and I'm experiencing this problem

Can you post a small reproduction project?
I can't reproduce it with preview.4

But I can when it contains exactly the https://github.com/MudBlazor/MudBlazor/pull/10215

Could you get somehow a js version from the dev?

Is the issue reporting the position of the picker or the width of the picker.
I know @ScarletKuro  is referring to the width which is a regression. That change was merged after preview-4 though.

Is the issue reporting the position of the picker or the width of the picker. I know @ScarletKuro is referring to the width which is a regression. That change was merged after preview-4 though.

Ah, position. The issue was just "see the diff".
I think this is because of https://github.com/MudBlazor/MudBlazor/pull/10071 @danielchalmers and this is expected according material specs. Pls confirm

@Anu6is @ScarletKuro Here's a little project where there's a problem displaying the date selector. I hope this helps.
BlazorAppbug.zip

@Anu6is @ScarletKuro Here's a little project where there's a problem displaying the date selector. I hope this helps.
BlazorAppbug.zip

You didn't reply on the question. Is your issue originally talking about the popover position or the space area on the left after the scrolling?

The only thing odd I can see in the example is the picker isn't taking up the full width as it's input

@Anu6is @ScarletKuro Here's a little project where there's a problem displaying the date selector. I hope this helps.
BlazorAppbug.zip

You didn't reply on the question. Is your issue originally talking about the popover position or the space area on the left after the scrolling?

I don't know how to answer your question. I posted a screenshot where the problem is displayed on the right in the first post. There is no shadow border anymore and the lower part of the calendar popup is cut off/not displayed. The screenshot below shows the problem reproduced in the project I've provided.

I don't understand your attitude. I feel like you're accusing me of not providing enough information. I'm just doing it to help and I've spent some time providing a project to reproduce the problem as requested... And with the 2 pictures, it is clear there is an issue.

I don't know how to answer your question. I posted a screenshot where the problem is displayed on the right in the first post. There is no shadow border anymore and the lower part of the calendar popup is cut off/not displayed. The screenshot below shows the problem reproduced in the project I've provided.

Thanks for your response. I couldn't figure out the problem, even after running your example, because it seemed fine at first glance. The only difference I noticed was the position, which now makes sense in v8 after this change: https://github.com/MudBlazor/MudBlazor/pull/10071. Now that you've explicitly pointed out the issue, I can see the missing shadow.
However, I would still ask that you be more precise when writing a bug report next time. The bug report needs to be as detailed as possible, because something that may be obvious to you might not be to others. As you can see, three people had trouble understanding the issue at the same time.

Doesn't help we've made a dozen changes around popovers either. I see the missing shadow but I don't recall any of our changes should have affected that. I don't see the cut off though. If there's something I need to review/look at please let me know.

Bisect shows this PR as the culprit for missing shadow https://github.com/MudBlazor/MudBlazor/pull/9993
@danielchalmers

I don't think it's related to popover changes (done by you), at least not the shadow ("not sure about the the lower part of the calendar popup is cut off/not displayed" - quote) since the same problem is present in preview.1. I will run bisect to try find the commit that causing the shadow problem

The cut off part is just a visual effect of the bottom shadow missing I think.

The shadow is being cut off because it's applied inside the picker (wrong way) instead of on the popover itself (correct way).
If we move the mud-elevation-8 class from .mud-popover > .mud-picker-inline-paper > .mud-paper to the .mud-popover it should fix the issue. This should be done on all applicable components now that #9993 is in place.

The shadow is being cut off because it's applied inside the picker (wrong way) instead of on the popover itself (correct way).
If we move the mud-elevation-8 class from .mud-popover > .mud-picker-inline-paper > .mud-paper to the .mud-popover it should fix the issue. This should be done on all applicable components now that #9993 is in place.

Should we revert now the https://github.com/MudBlazor/MudBlazor/pull/9993 or just wait for the fix?

The shadow is being cut off because it's applied inside the picker (wrong way) instead of on the popover itself (correct way).
If we move the mud-elevation-8 class from .mud-popover > .mud-picker-inline-paper > .mud-paper to the .mud-popover it should fix the issue. This should be done on all applicable components now that #9993 is in place.

Should we revert now the #9993 or just wait for the fix?

I wouldn't revert #9993 if it's the correct behavior. I can't make a PR for this right now but it might be as simple as
removing the elevation level from the picker
https://github.com/MudBlazor/MudBlazor/blob/960e7c1d588acf052f0833d2f76bcaf04ac38287/src/MudBlazor/Components/Picker/MudPicker.razor.cs#L40
and adding it to the popover
https://github.com/MudBlazor/MudBlazor/blob/960e7c1d588acf052f0833d2f76bcaf04ac38287/src/MudBlazor/Components/Picker/MudPicker.razor#L63


Remove scenarios that require domain joining

Removing these scenarios in the docs because we're deprecating the domain join scripts and not recommending to customers that they either AD/AAD join or register their lab VMs.

@ntrogh

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ntrogh - for context on these changes, see blog post: https://techcommunity.microsoft.com/t5/azure-lab-services-blog/use-labs-without-registering-joining-to-ad-aad/ba-p/3880261

#sign-off

#sign-off


kangaru: init at 4.3.2

Add new package for https://github.com/gracicot/kangaru.

Kangaru is an inversion of control container for C++11, C++14 and later. It provides many features to automate dependency injection and reduce the amount of wiring boilerplate in your code.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@FliegendeWurst thanks for the review btw other question should i squash these commits together or not?

Yes, they should be squashed.

squashed and rebased to latest master


implement releasing and reclaiming of tasks.

can be merged once the following PRs are merged:

https://github.com/effectai/effect-network/pull/75
https://github.com/effectai/effect-js/pull/54

@laurensV tested ✅  can be merged


cage: <IP_ADDRESS> -> 0.1.3

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

On test:
machine: waiting for alice@machine to appear on screen
machine: performing optical character recognition
machine: sending monitor command: screendump /build/tmp5er4se9f
machine # [   14.433187] show_signal_msg: 93 callbacks suppressed
machine # [   14.433191] .cage-wrapped[753]: segfault at 20 ip 00007f0325aa78c7 sp 00007ffc7af59790 error 4 in libwlroots.so.8[7f0325aa1000+5a000]
machine # [   14.436361] Code: 4f a8 ff ff eb aa 0f 1f 44 00 00 c3 0f 1f 80 00 00 00 00 41 54 49 89 f4 55 31 ed 53 48 89 fb 48 8d 7f 18 48 8d 83 b8 00 00 00 <80> 7f 08 00 75 0d 48 83 3f 00 0f 85 91 00 00 00 48 89 fd 48 83 c7
machine # [   14.360139] systemd[1]: Created slice system-systemd\x2dcoredump.slice.
machine # [   14.366883] systemd[1]: Started Process Core Dump (PID 888/UID 0).
machine # [   14.654872] systemd-coredump[889]: Process 753 (.cage-wrapped) of user 1000 dumped core.

Oof!

The CI test also fails (on both architectures) so it seems reproducible and doesn't only affect your system. Have you tried reproducing the test by running xterm manually or via the Cage module? (Unfortunately it could even depend on the kernel/mesa driver.)

The CI test also fails (on both architectures) so it seems reproducible and doesn't only affect your system. Have you tried reproducing the test by running xterm manually or via the Cage module? (Unfortunately it could even depend on the kernel/mesa driver.)

I have run xterm manually, not yet via the cage module. I'll try that I guess

Result of nixpkgs-review pr 119615 at b1a2ebe7 run on x86_64-linux 1

2 packages built successfully:

cage
wio

I'm also seeing this segfault in the test. It would be useful to have a backtrace.

I was about to take a look at the test but it is already broken since 2021-02-20 (no segfault though): https://hydra.nixos.org/build/141768308
cc @matthewbauer @flokli (test maintainers): Can you take a look?

I'll start bisecting this.

There's also a stacktrace available here: https://github.com/NixOS/nixpkgs/pull/119963#issuecomment-823439106

I gave up trying to bisect this - there was some other issues which made this pretty much impossible to bisect.
It might be related to the WAYLAND_DISPLAY changes - see https://github.com/NixOS/nixpkgs/issues/119445#issuecomment-819766445 for context.

Note https://github.com/NixOS/nixpkgs/pull/121174 - I can't/want to really debug/maintain this without real hardware.

@GrahamcOfBorg test cage
The test should finally succeed now... Seems like it's indeed related to the GPU driver. I guess switching to virtio should be fine (is faster anyway). Needed to rebase as I've fixed the existing test failure in #121190.

Does this also fix the cagebreak module?

Seems like wlroots 0.13 had some change that breaks with -vga std (the default).

Is that something we want to report upstream?

Is that something we want to report upstream?

I also wondered about that but I think we should at least provide a full stack trace and if this only affects that one driver/combination is likely pretty much irrelevant. But could maybe be something to mention on their IRC? Anyway, feel free to contact them.


Many pieces of image slices found after snapshot finished.

Besides the whole screenshots captured by Snapshot, also found many small slices images of respective whole screenshots under the folder /Snapshot.
Are they right? How to avoid these slices generated?

As far as I know, they are only stored if an error occurred when executing UI Automation.


ACCUMULO-4184: Fixed issues with the HostRegexTableLoadBalancer

Balancer now watches for configuration changes so the master does not have to be restarted
Fixed a bug where pools overlapped the same hosts and tablets for those tables were constantly reassigned.
Added a property to control the number of migrations that can be created by this balancer in one pass
Fix issue where matching properties were being parsed as table regex properties

cc @joshelser  @ivakegg


Fix return type of _BaseParameter.call

Question - is this fix correct? :)

@astafan8 I believe this is correct. I fixed the issues that came up as a result


forcing.sources["MarrmotForcing"] doesn't work with dict for forcing_info

When running the demo notebooks (tested for both M01 and M14), the use of a dict "forcing_info" does not update / influence the forcing_file mentioned in the dict. In the notebook in this repo, the output of print(forcing) is also very different.
No errors are thrown however.
See screenshot and compare to https://github.com/eWaterCycle/ewatercycle-marrmot/blob/main/docs/MarrmotM01.ipynb

Hi Rolf, thanks for raising this issue. The forcing file is now directly passed as a keyword argument. I have updated the notebook in #4 to fix this.


[Obsolete] MONDO:0006315 neoplastic medium-sized B-lymphocyte with basophilic cytoplasm

Mondo term (ID and Label)
MONDO:0006315 neoplastic medium-sized B-lymphocyte with basophilic cytoplasm
Reason for deprecation
this is a cell type and not a disease
Suggested term to consider
NCIT:C37005
cc @zoependlington

related to #2095

On Thu, Oct 8, 2020 at 12:10 PM Nicole Vasilevsky<EMAIL_ADDRESS>wrote:

related to #2095
https://github.com/monarch-initiative/mondo/issues/2095

is it?

yes, MONDO_0006315 had an invalid def


Bootstrap 3.3.5 Panel class is not available in v4-alpha?

why panel class is not available in bootstrap v4-alpha?

Panels, thumbnails and wells have been dropped in favour of cards, see: http://v4-alpha.getbootstrap.com/components/card/

See also http://v4-alpha.getbootstrap.com/migration/#components


mbedtls_pk_parse_keyfile returned -0x3D20，cert and key is check in openssl no problem but now branch failed

Answers checklist

[x] I have read the documentation ESP-AT Programming Guide and the issue is not addressed there.
[x] I have used the latest released firmware or have updated my ESP-AT branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

AT+GMR
AT+GMR
AT version:<IP_ADDRESS>(ec2dec2 - ESP32C3 - Jul 28 2023 07:04:22)
SDK version:v5.0.2-376-g24b9d38a24-dirty
compile time(6118fc22):Nov 25 2023 22:59:36
Bin version:v<IP_ADDRESS>(MINI-1)
ESP-AT Firmware Source
3.Compile use branch master and release/v<IP_ADDRESS> alway the same result but for release/v<IP_ADDRESS> is ok
Hardware Information
ESP32C3-MINI-1
Power Supply used
USB
What is the expected behavior?
right cert and key(EC) pass
What is the actual behavior?
right cert and key(EC) failed
Probability of recurrence
Must appear
AT+SYSRAM?
AT+SYSRAM?
+SYSRAM:126120,97748
OK
Steps to reproduce
just add a EC cert and key and ca to customized_partitions\raw_data\mqtt_ca
and run connect
AT command port output
Common sanity checks
MBEDTLS_ECP_TYPE_SHORT_WEIERSTRASS
ecp_mul_comb_after_precomp start
********ecp_mul_comb_after_precomp:1
********ecp_mul_comb_after_precomp:2
********ecp_mul_comb_after_precomp:3
********ecp_mul_comb_core:1
********ecp_mul_comb_core:2
********ecp_randomize_jac,ret:-32
mbedtls_ecp_mul,ret:-32
pk_parse_key_sec1_der ret:-0x3D20
E (11102) esp-tls-mbedtls: mbedtls_pk_parse_keyfile returned -0x3D20
E (11109) esp-tls-mbedtls: Failed to set client pki context
E (11115) esp-tls-mbedtls: Failed to set client configurations, returned [0x8019] (UNKNOWN ERROR)
E (11126) esp-tls: create_ssl_handle failed
E (11130) esp-tls: Failed to open new connection
E (11135) transport_base: Failed to open a new connection
E (11145) mqtt_client: Error transport connect
MQTT ERROR
MQTT DISCONNECTED

AT log port output
ready
WIFI CONNECTED
WIFI GOT IP
AT+MQTTUSERCFG=0,5,"venus&/MCHJ23926DNND1010003/TLS/AUK1_MQTT_HMAC_SHA1/1700794165000/AUTH","/venus/MCHJ23926DNND1010003","",0,0,""

OK
AT+MQTTCONN=0,"auk-iot.test.zservey.com",1883,1
+MQTTDISCONNECTED:0

ERROR

More Information.
No response

@sjWang It seems that there is an issue with parsing the private key, returning an incorrect private key format errno. I would like to look into this issue if you can provide your EC certificate and private key, and ca.

@sjWang It seems that there is an issue with parsing the private key, returning an incorrect private key format errno. I would like to look into this issue if you can provide your EC certificate and private key, and ca.

already find problem:
1、aes_ctx not seed，because of mbedtls3.x interface change。

2、v<IP_ADDRESS> pass because NULL default seed, code below:
#if !defined(MBEDTLS_ECP_NO_INTERNAL_RNG)
if( f_rng == NULL )
{
/* Adjust pointers */
f_rng = &ecp_drbg_random;
#if defined(MBEDTLS_ECP_RESTARTABLE)
if( rs_ctx != NULL && rs_ctx->rsm != NULL )
p_rng = &rs_ctx->rsm->drbg_ctx;
else
#endif
p_rng = &drbg_ctx;
    /* Initialize internal DRBG if necessary */

#if defined(MBEDTLS_ECP_RESTARTABLE)
if( rs_ctx == NULL || rs_ctx->rsm == NULL ||
rs_ctx->rsm->drbg_seeded == 0 )
#endif
{
const size_t m_len = ( grp->nbits + 7 ) / 8;
MBEDTLS_MPI_CHK( ecp_drbg_seed( p_rng, m, m_len ) );//seed here
}
#if defined(MBEDTLS_ECP_RESTARTABLE)
if( rs_ctx != NULL && rs_ctx->rsm != NULL )
rs_ctx->rsm->drbg_seeded = 1;
#endif
mbedtls_printf("%s f_rng reset and seed:%ld\r\n",func,grp->nbits);
}
#endif /* !MBEDTLS_ECP_NO_INTERNAL_RNG */
now,temporary modified and pass code :
esp_err_t esp_create_mbedtls_handle(const char *hostname, size_t hostlen, const void *cfg, esp_tls_t *tls)
{
assert(cfg != NULL);
assert(tls != NULL);
int ret;
esp_err_t esp_ret = ESP_FAIL;
tls->server_fd.fd = tls->sockfd;
mbedtls_ssl_init(&tls->ssl);
mbedtls_ctr_drbg_init(&tls->ctr_drbg);
mbedtls_ssl_config_init(&tls->conf);
mbedtls_entropy_init(&tls->entropy);
// fix: add seed after mbedtls_ctr_drbg_init
if ((ret = mbedtls_ctr_drbg_seed(&tls->ctr_drbg,
                                 mbedtls_entropy_func, &tls->entropy, NULL, 0)) != 0) {
    ESP_LOGE(TAG, "mbedtls_ctr_drbg_seed returned -0x%04X", -ret);
    mbedtls_print_error_msg(ret);
    ESP_INT_EVENT_TRACKER_CAPTURE(tls->error_handle, ESP_TLS_ERR_TYPE_MBEDTLS, -ret);
    esp_ret = ESP_ERR_MBEDTLS_CTR_DRBG_SEED_FAILED;
    goto exit;
}

@sjWang 非常感谢您的反馈！
个人基本认可上面的做法，应该是先调用 mbedtls_ctr_drbg_seed(), 再调用 set_client_config(), 我会和负责这部分的同事进一步确认后，再修复。
再次感谢！

@sjWang 此问题已在 master 上修复，请验证。如有问题，请 reopen 一下。

@sjWang 此问题已在 master 上修复，请验证。如有问题，请 reopen 一下。

ok，thanks

Hello @sjWang ! 我们想加一些 testcases 来测试此类私钥，您方便告知一下，这种私钥是如何生成的么？  :pray:
我们尝试了一些 openssl ecparam 命令，似乎都无法复现之前的问题。


Keys MAX_FOOD_LEVEL, MAX_HEALTH_LEVEL, MAX_SATURATION_LEVEL have wrong value

They are defaulting to their max values instead of the default specified

Format:
player.get(Keys.HEALTH).get : player.get(Keys.MAX_HEALTH).get()
etc
etc

It seems this was an issue with my player's data file or something along those lines.


Git sparse checkout

Makes it possible to only checkout a part of a git repository using the src keyword in the manifest.
Solves #21

Currently not workings for tags

Currently not workings for tags


Update Policy documentation to support UAMI and new remediation properties

Step #1

@kenieva : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Do not merge, I will need to add more edits...

#hold-off

@kenieva : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kenieva @meghanagupta95 @DCtheGeek - I think that with a new image file and formatting changes, this update will need to be moved to a new pull request in the private repository, for quality validations. The PR review team can help with that, or one or you might prefer to do it. Thanks!

@kenieva : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kenieva : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver thank you for that information! Once @eladperets  and @meghanagupta95  approve this I will be happy to reach out for further validations.

@ktoliver can you provide more information on how to moved this  to a new pull request in the private repository, for quality validations? thanks.

@kenieva Yes, I'll send you email that has the information. Thanks.

@kenieva Would you like me to move the commits to a new pull request in the private repo? Your association with the commits will remain intact; the PR would be created on my fork of the repo.

@ktoliver Yes that works!

This update has been moved to a new pull request in the private repo. I've asked the content team to review the new PR. We'll close this PR. Thanks, all.


comment BSD/GNU utils differences

Example of different behaviors:

[ ] find
[ ] mktemp
[ ] sed
[ ] ...

@aureliojargas would like to help here?

Yep, here is a good source: https://github.com/funcoeszz/funcoeszz/wiki/Portabilidade


Pagination SEO issue: canonical url on paginated pages does not work

Discussed in https://github.com/h-enk/doks/discussions/344

Originally posted by krispkrisp June 17, 2021
On pages with numbers: ../page/2/ ...
it shows the canonical for the main taxonomy page, without the page number this is a problem for indexing.
As described here:
https://www.searchenginejournal.com/technical-seo/pagination/
this is a general issue with hugo described here:
https://github.com/gohugoio/hugo/issues/4507
In DOKS pagination and canonicalization is done in seo.html in partials/head
However, this:
`{{ if $.Scratch.Get "paginator" }}

  {{ if .Paginator.HasPrev -}}

  {{ end -}}
  {{ if .Paginator.HasNext -}}

  {{ end -}}
{{ else -}}

{{ end -}}`
does not work
on pagination pages the ELSE is executed, and no prev/next and canonical is done
way around the Hugo problem with canonical for pagination pages is described here:
https://moonbooth.com/hugo/seo/#canonicals

Fixed w/ a1dce26c2450359e02743ac6289f53179971e929


Response body should be null in a HEAD response

Currently, if a server returns a Content-Length in the headers of a HEAD request matching the size of the GET entity (as it is supposed to), OkHttp will think that the response has a body to be read. When attempting to read this body, an exception is thrown because only 0 bytes are read instead of whatever the content length is: Content-Length and stream length disagree. OkHttp should just report a null body, since the Content-Length info is still available in the headers.
Here's a test case for what I expect AND what actually happens: https://gist.github.com/kenzierocks/6f061563cbcaafc05fe469a494ab45d6

The spec allows a body in a HEAD response. An empty response is much more
appropriate, although if the content length is 0 I agree we should actively
close the underlying source.
On Thu, Sep 22, 2016, 10:37 PM Kenzie Togami<EMAIL_ADDRESS>wrote:

Currently, if a server returns a Content-Length in the headers of a HEAD
request matching the size of the GET entity (as it is supposed to), OkHttp
will think that the response has a body to be read. When attempting to read
this body, an exception is thrown because only 0 bytes are read instead of
whatever the content length is: Content-Length and stream length disagree.
OkHttp should just report a null body, since the Content-Length info is
still available in the headers.
Here's a test case for what I expect AND what actually happens:
https://gist.github.com/kenzierocks/6f061563cbcaafc05fe469a494ab45d6
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/2879, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEEEctiTXodqWTjy7s3FTZJ06BvPx55ks5qsztqgaJpZM4KEjoA
.

An empty response would also make sense.

I added a test. The empty body is working as designed.

Fantastic, thanks!


AppView and Menubar (stubs) Added

Created an AppView class (in app_view.js) next to app_controller.js to deal with all view-related functions for main application. Currently, AppView initializes the menubar that has the basic operations necessary for navigating the application.
These are mostly stubs -- there are no actions associated with most of the buttons, but I thought I should make a pull request so that other people can start linking actions to the menubar.

This change is

Review status: 0 of 5 files reviewed at latest revision, 10 unresolved discussions, some commit checks failed.

begin.html, line 1 at r3 (raw file):

<!DOCTYPE html>

Attaching screenshots can be helpful when you're creating new pages or layouts.  (I have no idea what this will look like.)

src/app_view.js, line 87 at r3 (raw file):

    let help = new nw.Menu();

    // FILE > NEW dropdown.

I would break this into helper functions by the categories that you've already called out in these comments.  There's too much going on in this function.

src/app_view.js, line 207 at r3 (raw file):

   */
  addMenuAction(menu, path, fcn) {
    // TODO(celinechoo): Complete addMenuAction() to simplify adding actionable

Instead of TODO(name), do TODO(#number), and create a corresponding issue on github.  That lets you see all of your issues without reading through code, and also means someone else can solve the problem for you if they're motivated.

src/app_view.js, line 220 at r3 (raw file):

   * Helper function to add menu items to the main menubar.
   *
   * @param {!Menu} menu Menu object from NW.js to which we are adding an element.

Is the full type just Menu?  Usually it's part of a namespace (e.g. Blockly.Menu).

src/app_view.js, line 223 at r3 (raw file):

   * @param {string} name Name of element being added to menu.
   * @param {!Menu} subMenu Menu object to be added if the added element will also
   * have its own sub-dropdown. Optional (if unused, will be null).

Nit: four more spaces at the beginning of this line, and for two lines below.
Lines that continue an annotation always have a four-space additional indent.

src/app_view.js, line 259 at r3 (raw file):

        menuItem = menuItem[pathToMenuItem[i]];
      } catch(e) {
        console.log(e);

Will there be enough information in the log for you to debug?  If not, add a message with some context.
Is this a recoverable error?  Or does it break the whole application?

Comments from Reviewable

lgtm overall.  Make sure you squash when merging.

Reviewed 2 of 11 files at r3, 3 of 3 files at r4.
Review status: all files reviewed at latest revision, 11 unresolved discussions.

src/app_view.js, line 30 at r4 (raw file):

/**
 * @fileoverview The AppView Class deals with the visual parts of the main

Nit: fileoverview comes before the requires: https://github.com/google/blockly-devtools/blob/master/src/block_library_storage.js#L21
The comment here would describe the class.  Sometimes that will be similar to the fileoverview text, but not always.

Comments from Reviewable

Review status: 3 of 4 files reviewed at latest revision, 11 unresolved discussions.

src/app_view.js, line 51 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
Fine for now, but as I look at these two, I'm wondering if these will be dependent on the project configuration. One has to configure the locations of each platform before one can save it. A project may only have iOS configured, so does it make sense to have a save item for the others? Maybe the others are just disabled, or maybe they trigger the project configuration screen.
Also, It might read cleaner to make "Save" the menu group, and add an "All" at the top.

I consolidated Save into one submenu, and left the four options for now.

src/app_view.js, line 62 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
There is enough nuance to iOS and Android project setup, I don't think we want to take these on.  Let's just leave a stub for web only.

Done.

src/app_view.js, line 87 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
I would break this into helper functions by the categories that you've already called out in these comments.  There's too much going on in this function.

Done! Created a simpler menubar init function.

src/app_view.js, line 89 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
The duplication of the menu hierarchy here and above bothers me.  There is an opportunity for the two to get out of sync. In this first one item, you've specified 'New Project' three times: once above, once to the left of the assignment, and again as a parameter.
Consider adjusting your addMenuAction helper function to actually use path as a paththat will take in a menu path (string array), and find or generate any necessary menuTree hierarchy that doesn't already exist. Something like...
function addMenuAction(pathToItem, clickHandler) { ... }

addMenuAction(['File', 'New', 'Project'], () => { ... })

Then, there is only one place to edit it each path.

I removed the addMenuAction function because I realized my implementation of the menubar init was very disorganized.

src/app_view.js, line 207 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Instead of TODO(name), do TODO(#number), and create a corresponding issue on github.  That lets you see all of your issues without reading through code, and also means someone else can solve the problem for you if they're motivated.

Noted, thanks!

src/app_view.js, line 220 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Is the full type just Menu?  Usually it's part of a namespace (e.g. Blockly.Menu).

Done.

src/app_view.js, line 223 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Nit: four more spaces at the beginning of this line, and for two lines below.
Lines that continue an annotation always have a four-space additional indent.

Done.

src/app_view.js, line 259 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Will there be enough information in the log for you to debug?  If not, add a message with some context.
Is this a recoverable error?  Or does it break the whole application?

Removed these lines, but will note the next time I use try/catch.

src/app_view.js, line 30 at r4 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Nit: fileoverview comes before the requires: https://github.com/google/blockly-devtools/blob/master/src/block_library_storage.js#L21
The comment here would describe the class.  Sometimes that will be similar to the fileoverview text, but not always.

Done.

begin.html, line 1 at r3 (raw file):
Previously, rachel-fenichel (Rachel Fenichel) wrote…
Attaching screenshots can be helpful when you're creating new pages or layouts.  (I have no idea what this will look like.)

I didn't mean to add this page as part of this pull request. Bad oversight on my part.

begin.html, line 9 at r3 (raw file):
Previously, AnmAtAnm (Andrew n marshall) wrote…
I don't think this window will need prettify.

Bad oversight on my part -- this shouldn't be part of this pull request.

Comments from Reviewable


[AIRFLOW-5601] Use built-in pagination mechanism in MLEngine hook

Make sure you have checked all steps below.
Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.
In case you are proposing a fundamental code change, you need to create an Airflow Improvement Proposal (AIP).
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does
If you implement backwards incompatible changes, please leave a note in the Updating.md so we can assign it to a appropriate release

Codecov Report

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

@@            Coverage Diff            @@
##             master     #366   +/-   ##
=========================================
  Coverage          ?   80.05%
=========================================
  Files             ?      610
  Lines             ?    35260
  Branches          ?        0
=========================================
  Hits              ?    28229
  Misses            ?     7031
  Partials          ?        0

Impacted Files
Coverage Δ

airflow/gcp/hooks/mlengine.py
21.36% <0%> (ø)

Continue to review full report at Codecov.

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


Typo in env var:  JWT_PUBKICKEY_LOCATION

https://github.com/rht-labs/lodestar-backend/blob/e3016409adb67c53132b7f4e1de81c4f6ff3efa1/src/main/resources/application.properties#L18
Nothing wrong from a functional perspective -- just would be a bit more clear to get this fixed up in future releases

fixed with commit 0912ed5582c03259386d7eb51baaa4b452401654


Load PMID to HPO via mesh into database table + add PMID column to sentences_input

@ajratner Is there any documentation on how to "recreate" the current 50/50 split sentences_input table? I'm just trying to run ./run.sh /dfs/scratch0/ajratner/dd_raw_docs/PLoS_ONE/PLoS_ONE_2006_Dec_20_1\(1\)_e100.nxml sentences_input new on one of my databases, but I get:
Using bazaar installation at /lfs/local/0/jbirgmei/bazaar
Parsing XML docs
Running NLP preprocessing
NO MATCH
parallelism = 80
batch-size  = 1000
/lfs/local/0/jbirgmei/bazaar/parser/run.sh -i json -k item_id -v content
/lfs/local/0/jbirgmei/bazaar/parser/run.sh: line 5: /lfs/local/0/jbirgmei/bazaar/parser/target/start: No such file or directory
Adding to existing table sentences_input in db
Copying to sentences_input table in database genomics_jbirgmei_50k
cat: xml_parsed.json.split/*.parsed: No such file or directory
To clean up, run: rm -rf xml_parsed.json*

@ajratner where the important part is that it doesn't find one of these scripts, and nothing happens. And I think even if it worked, I wouldn't know how to recreate the current sentences_input table.

@Colossus which run.sh script is this (abs path)?

@Colossus I think you need to build the bazaar/parser code, see the README.md in bazaar/parser

@Colossus Then to recreate the 50/50 split table I just used some SQL, not sure if I wrote it down anywhere but I just took all the WHERE doc_id NOT LIKE '%one%' and then the same number of the complement set...

@ajratner it was the run.sh in /lfs/raiders2/0/jbirgmei/dd-genomics/parser, but you're right, I need to run setup.sh in the bazaar/parser directory

Changing this objective on this, have a much better and easier solution

@ajratner It turns out, for obvious reasons, that of the 16,917,165 lines in onto/data/hpo_to_pmid_via_mesh.tsv, we're using only those for which we can also map the PMID to a DOI (our documents key). By simply pre-joining hpo_to_pmid_via_mesh.tsv with the DOI mapping, the size of the file is reduced to 0.3%, or about 60,000 lines, which easily fit into memory. The runtime of the pheno candidate extractor is roughly halved, the results are exactly the same. Please execute:
cd onto/data
sort -k2,2 hpo_to_pmid_via_mesh.tsv > hpo_to_pmid_via_mesh_sortk2.tsv
sort -k2,2 plos_doi_to_pmid.tsv > plos_doi_to_pmid_sortk2.tsv
join -1 2 -2 2 hpo_to_pmid_via_mesh_sortk2.tsv plos_doi_to_pmid_sortk2.tsv | awk '{print $2"\t"$1}' > hpo_to_pmid_via_mesh_with_doi.tsv

To make my (uncalled for :) debut on the forum - JB, you can bias the non Plos one towards more caluable ploses. Plos genetics, maybe biology maybe comp bio - you can look at the OMIM distribution.
-Gill
On Jul 21, 2015, at 4:15 PM, Alex Ratner<EMAIL_ADDRESS>wrote:

@Colossus Then to recreate the 50/50 split table I just used some SQL, not sure if I wrote it down anywhere but I just took all the WHERE doc_id NOT LIKE '%one%' and then the same number of the complement set...
—
Reply to this email directly or view it on GitHub.

@gbgbg OK, we can definitely do that. But we're not using the current dataset for anything serious anyways. When we're extracting for Aaron, we're gonna use all available data anyways.


Integrate new TimeSeriesWidget from lib

An example of the new widget inside sample-app-2, made by Sergio
This can't be effectively merged (with CI running well), until a new carto-react version is released, including the new widget

Should we consider this integration also part of the delivery, @borja-munoz ?
If that is the case then a bit of time should be dedicated to the design of the widget position, also in mobile view

@Clebal Not sure if it's something about the specific widget configuration in Collisions, but changing Speed looks like it has no effect (linked to your current PR in lib, as it is right now). Could you check it please?

Should we consider this integration also part of the delivery, @borja-munoz ?
If that is the case then a bit of time should be dedicated to the design of the widget position, also in mobile view (and also what is the widget behaviour in a smaller width -left panel, like other widgets)

Yes, this should be part of the delivery, and I agree we should give some love to the layout. As per the widget behavior, I think it is resized to the width of the container so it should work in the left panel, but we need to test that the different labels and controls are correctly displayed.


Auto create a URL to file an issue when encountering an error

Is your feature request related to a problem? Please describe.
When exceptions in Bandit occur as a result of bugs, the easier we make it to get bugs reported, the better.
Describe the solution you'd like
So when an exception is encountered in Bandit, instead of just logging the issue, provide a custom URL to the user
for them to open an issue on GitHub with much of the relevant information already populated.
See https://docs.github.com/en/issues/tracking-your-work-with-issues/creating-an-issue
Describe alternatives you've considered
n/a
Additional context
n/a

This is a good idea! @ericwb
I would happy to work on this feature if you can provide some more info on where to start! Thanks!


Ws symbols

rebased on master

Rebase on master of https://github.com/rescript-lang/rescript-vscode/pull/510

linux: 30s until fail
macOS: 40s until fail
win: 57s until fail

On my dev machine time make -j test from scratch.
Master: about 12s.
PR: under 10s.


set group display name; close #80

@patflynn @loosebazooka

PTAL. This actually pulled out another issue in the tests. All commits are now squashed.

LGTM


TD-834: Fixes cascade with user interaction transiency in session

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 90.205%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
4522

Relevant Lines:
5013

💛  - Coveralls


feat: add babel parser and traverse and migrate some rules

Summary
Adds babel parser and traverse packages for AST parsing for some rules.
Description

rule functionSize refactored with babel
rule parameterCount refactored with babel
rule computedSideEffects refactored with babel
Extract some reusable blocks to /helpers

Related Issues
Supports #368
Fixes #365 #364 #431
Type of Change

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)

Screenshots (if applicable)
N/A

not ready to merge, there is a weird bug when running the cli tests

not ready to merge, there is a weird bug when running the cli tests

It seems to be something with the traverse logic because if functionSize, parameterCount and computedSideEffects rules are commented, the error dissapears.
There is a file inside the project which is causing that error when walking through it... I'm still debugging 👀

@David-Pena there are too many conflicts to fix manually. Can you just delete and regenrate the lock file?

Oh no worries, I will. I'll be sitting in front of this issue today 🫡

@David-Pena can you check the failing tests and update yarn.lock?

I'm back 👨🏼💻

did the tests passed on your end?

did the tests passed on your end?

Still debugging...
I'm getting an error traverse is not a function but I'm pretty sure I'm importing it correctly, so I doing a bit of research

Ah ok. I thought it is ready to merge and then I saw the failing tests.
No worries, take your time.
It is ok if it is ready only next year.


 Maybe text overflow issue.

Thanks for your report! Maybe we could just get rid of those lines, they don't add much.


Another two Smart Select + Popup issues

What you did
I put a smart select inside an initialized popup.
Expected Behavior
The pop up opens normally, and when user clicks the back button from navbar i was expecting the smart select to get closed.
Actual Behavior
1- The smart select AND the popup together get closed.
2- If you try again and open the popup, then the smart select don't work anymore (not clickable).
JsFiddle Example here: https://jsfiddle.net/tinyCoder32/L7cpjuvq/

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Background-Color

Hi, great theme! I use it at beck24.be - however, I can't seem to be able to change the background color. Any idea why?
And, just out of curiosity: which font is used? a google font? or system fonts? if google fonts, I need to include that into my data protection page.
I did follow the explained way: https://github.com/Track3/hermit/issues/58 but nothing did change. Neither if I change the colors in themes/assets/scss/_predefined.scss not if I copy the whole path to root/assets/scss/_predefined.scss
What did I do wrong? All I want to do is exchange the grey thru a darker blue #084B8A and (for the menue) #0B3861

you may want to take a look at https://github.com/Track3/hermit/issues/99#issuecomment-646126134


test: Add gitlab api test

Fixes #855

Proposed Changes

...
...
...

Readiness Checklist
Author/Contributor

[x] If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

[x] Label as either enhancement, bug, documentation or dependencies
[x] Verify design and implementation

通过命令行的方式传参

https://doc.hutool.cn/pages/AsymmetricCrypto/#使用

RSA 非对称加密 将私钥加密，公钥解密


not able to install with Python3 (brombo/galgebra#37)

@Foadsf commented on Apr 26, 2018

I tried to install using python3 setgapth.py  command but I get the error:

File "setgapth.py", line 32
print 'os name:',os.name
^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print('os name:',os.name)?

@hugohadfield commented on Apr 26, 2018

Hi @Foadsf this repo is python2 only, if you want to help getting it into python3 I am working on that over here:
https://github.com/hugohadfield/galgebra
With the goal of merging it back later
(👍: Foadsf, waldyrious, utensil, mevangelista-alvarado)

@mevangelista-alvarado commented on Jan 15

Hello @Foadsf for more information about galgebra you can reading https://github.com/brombo/galgebra/blob/master/doc/GAlgebraPrimer.pdf
(👍: Foadsf)

From http://webcache.googleusercontent.com/search?q=cache:https://github.com/brombo/galgebra/issues/37

@foadsf, closing this since setgapth.py is gone and we now just use setuptools


Added a check against cdn.id when updating DS with existing SSL keys

What does this PR (Pull Request) do?

[x] This PR fixes #5478

Which Traffic Control components are affected by this PR?

Traffic Ops

What is the best way to verify this PR?

Create a DS
Generate SSL keys (If using TP, go to mange SSL keys in a DS and then click on generate SSK keys OR use SSL Keys generate API)
Edit DS using PUT API (/api/3.0/deliveryservices/) and change the cdn id to a different value.
You should see an error as follows:

{
   "alerts":[{
          "text":"delivery service has ssl keys that cannot be automatically changed, therefore CDN and routing name are immutable",
          "level":"error"
 }]}

and a 400 HTTP status code
If this is a bug fix, what versions of Traffic Control are affected?

master (456f7620)

The following criteria are ALL met by this PR

[x] This PR includes tests
[x] have explained why documentation is unnecessary
[x] This PR does not includes an update to CHANGELOG.md as it is not necessary
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

API Tests already present. No documentation change.

Tested this in CiaB. I noticed an issue:
Consider a DS that has SSL keys associated with it.
If I try to update this DS with something that has a different CDN ID, but no CDN Name or Routing Name, it gives me the success message, which it shouldn't. I think it has got to do with the way you are handling the conditional statements in deliveryservices.go.
I would do something like this:
Check for ds.CDNId to not be nil
Now, put all the other conditionals under if hasSSLKeys {...}
Inside this conditional, you can check for the old and new CDN IDs to be equal
Next, you would check for if the ds.RoutingName is nil or not, and if it isn't, proceed to check if its equal to the old routing name(this check goes within the if hasSSLKeys conditional)
Next, you would check for if the ds.CDNName is nil or not, and if it isn't, proceed to check if its equal to the old CDN name (this check goes within the if hasSSLKeys conditional)

Tested this in CiaB. I noticed an issue:
Consider a DS that has SSL keys associated with it.
If I try to update this DS with something that has a different CDN ID, but no CDN Name or Routing Name, it gives me the success message, which it shouldn't. I think it has got to do with the way you are handling the conditional statements in deliveryservices.go.
I would do something like this:
Check for ds.CDNId to not be nil
Now, put all the other conditionals under if hasSSLKeys {...}
Inside this conditional, you can check for the old and new CDN IDs to be equal
Next, you would check for if the ds.RoutingName is nil or not, and if it isn't, proceed to check if its equal to the old routing name(this check goes within the if hasSSLKeys conditional)
Next, you would check for if the ds.CDNName is nil or not, and if it isn't, proceed to check if its equal to the old CDN name (this check goes within the if hasSSLKeys conditional)


Missing Schema and Hreflang Tag

Hi @mhaack
Please look into this and fix-
Hreflang Tag missing, blog, event and app note schema are also missing.
Hreflang tag excluded for these sections like Resources and Blogs, news, and events.
For reference-
Blog- https://live-md30.pantheonsite.io/lab-notes/clone-screening/role-of-crispr-in-microbiome-engineering
Event- https://live-md30.pantheonsite.io/events/slas-2023-building-biology-in-3d-symposium
App Note- https://live-md30.pantheonsite.io/en/assets/app-note/dd/img/ai-enabled-phenotypic-analysis-of-label-free-intestinal-organoids

@RajMankotia Regarding your comment to
Hreflang tag excluded for these sections like Resources and Blogs, news, and events.
I understand the Hreflang tag is missing for these sections.
Please identify the exact types where Hreflang tag should be added.

Hi @lydiapuric
If you check the old website - https://live-md30.pantheonsite.io/
Hreflang was implemented in these sections--  Product Pages, Applications, Technology, Video Gallery, Lab Automation pages, Contact Us Page, About Us Page, Leadership Page, and Customer Breakthrough

Thanks @RajMankotia.
Could you please verify in the meantime the fix for the schemas mentioned in this issue?

Blog: https://1105-schema--moleculardevices--hlxsites.hlx.page/lab-notes/clone-screening/role-of-crispr-in-microbiome-engineering
Event: https://1105-schema--moleculardevices--hlxsites.hlx.page/events/slas-2023-building-biology-in-3d-symposium
App Note: https://1105-schema--moleculardevices--hlxsites.hlx.page/en/assets/app-note/dd/img/ai-enabled-phenotypic-analysis-of-label-free-intestinal-organoids

Hi @lydiapuric
The blog and App Note schema looks good but for the Event please check it again.

Thanks for quick response, @RajMankotia ! Regarding the Event schema: The only difference I see is the date format for startDate and endDate, the schema otherwise matches the original one. Is there another issue in the event schema I am missing?

@RajMankotia Could you please verify the Hreflang tags? Thanks!
Some example pages I used for verification:

Homepage: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/
Product: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/products/microplate-readers/multi-mode-readers/spectramax-i3x-readers
Lab Automation: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/service-support/lab-automation-solutions/lab-automation-for-high-throughput-clone-screening
Application: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/applications/monoclonality
Technology: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/technology/luminescence
Customer Breakthrough: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/en/assets/customer-breakthrough/dd/img/university-of-sheffield
Video Gallery: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/video-gallery/flipr-system
Contact Us: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/contact?region=americas
About Us: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/about-us
Leadership: https://1105-hreflang--moleculardevices--hlxsites.hlx.page/leadership
Event (no Hreftag!): https://1105-hreflang--moleculardevices--hlxsites.hlx.page/events/slas-2023-building-biology-in-3d-symposium

Hi @lydiapuric,
It shows the same Home Page Hreflang tags for all pages. All pages have their own URL with countries.
Here are eg.-
for page- https://live-md30.pantheonsite.io/products/microplate-readers/multi-mode-readers/spectramax-i3x-readers

--
  |
  |
  |
  |
  |
  |

and it is different for this page- https://live-md30.pantheonsite.io/applications/cell-viability-proliferation-cytotoxicity-assays

--
  |
  |
  |
  |
  |
  |

I think this example will be helpful.

@RajMankotia Can you please recheck, fix is done.

@lydiapuric
Yes, now looks good. Thanks

Thank you for quick verification!


Fix narrow width audio results

Fixes

Fixes #1278  by @zackkrida
Description

The fix in this PR is actually just two lines, flipping one boolean and switching an and to an or.
The rest is all tests and test support.
Testing Instructions

Checkout the snapshots and also run the app locally by checking out the branch and running pnpm dev. Visit the audio results page, both via client side navigation and via SSR with and without a mobile UA and verify that the correct layout and size is used for every situation. The snapshots should help with that as well.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@obulat Interesting, I didn't notice that.
Using onMounted in the audio track doesn't seem to change anything, indeed, if you check the props being passed it is passing the correct size to the layout component (s in this case). It appears to be some kind of issue where the layout isn't re-rendering even though the prop value changes.
This commit (https://github.com/WordPress/openverse-frontend/pull/1281/commits/d5a965632a56c2bb974da8054b4336ae6dca0edd) fixed it :shrug:


TPCDS query 05/66/84 with hive external table return error

Steps to reproduce the behavior (Required)

testing environment:
qa_test_master
use hive_ex_tpcds_1t_orc;

perform tpcds query 84:
select  c_customer_id as customer_id
, coalesce(c_last_name,'') || ', ' || coalesce(c_first_name,'') as customername
from customer
,customer_address
,customer_demographics
,household_demographics
,income_band
,store_returns
where ca_city	        =  'Edgewood'
and c_current_addr_sk = ca_address_sk
and ib_lower_bound   >=  38128
and ib_upper_bound   <=  38128 + 50000
and ib_income_band_sk = hd_income_band_sk
and cd_demo_sk = c_current_cdemo_sk
and hd_demo_sk = c_current_hdemo_sk
and sr_cdemo_sk = cd_demo_sk
order by c_customer_id
limit 100;

Expected behavior (Required)
Returns the correct result set
Real behavior (Required)
ERROR 1064 (HY000): Operand '(coalesce(default_cluster:test_hive_hdfs_orc_tpcds1t_1636615764417.customer.c_last_name, '')) OR (', ')' part of predicate 'coalesce(default_cluster:test_hive_hdfs_orc_tpcds1t_1636615764417.customer.c_last_name, '')' should return type 'BOOLEAN' but returns type 'varchar(-1)'.
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()
mysql> select current_version();
+-------------------+
| current_version() |
+-------------------+
| UNKNOWN bfe6488   |
+-------------------+
1 row in set (0.01 sec)

Please set sqlMode =2 and try again.


Makes Mod Is Compatible With BE

Hello, Ferlern. I opened this issue because i want this mod is Compatible with BE. please fix it Fastly. i proud your mod 👍🏽
Tempesyad05

Hello. https://github.com/Ferlern/extended-UI/tree/BE-build
Main branch still for v6 & v7 alpha. You will have to manually download from here.
I want to warn you right away: I fixed only critical problems. Some features may not work / work incorrectly.
If you find a new critical bug due to new BE changes, please let me know. I don't watch them.
But please remember that the BE architecture is still subject to change, I don't want to waste too much time. I will create a full and stable version as soon as v7 is released.

@Ferlern I think it has already been released)


Add Gymnasium Otterndorf

@onesimpledot  I am sorry for the delay with processing your request. Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!

No problem, and thanks a lot!


Tabris failed to open on ios simulator

Problem description
The app failed to open when launching it to ios simulator
Expected behavior
should run
Environment

Tabris.js version: 3.9.1
Device: Simulator (iPhone 13 Pro)
OS:  iOS 15.5

Code snippet
Okay , i built the app using below command
tabris run ios --debug --cordova-build-config=/path/to/my/build.json --target="iPhone-13-Pro"
it build and signed ,
i am currently using Mac OS Sonoma with Xcode 14 and has iphonesdk17 installed on it and per checking on the logs
it builds using iphonesdk17 i don't know if it does matter because the simulator iPhone-13-Pro is running on iOS 15.5 sdk
it builds successfully but failed to launch from the simulator ,
below is the logs i received
device.name: iPhone 13 Pro
device.runtime: iOS 15.5
device.id: D00847F9-5C23-47A2-8797-979D4B9F91B0
[ios-sim] Simulator already running.
[ios-sim] An error was encountered processing the command (domain=IXUserPresentableErrorDomain, code=4):
Simulator device returned an error for the requested operation.
This app needs to be updated by the developer to work on this version of iOS.
Failed to find matching arch for input file: /Users/xxx/Library/Developer/CoreSimulator/Devices/D00847F9-5C23-47A2-8797-979D4B9F91B0/data/Library/Caches/com.apple.mobile.installd.staging/temp.1rQd8A/extracted/MyApp.app/MyApp
Underlying error (domain=IXUserPresentableErrorDomain, code=4):
        “My App” Needs To Be Updated
        This app needs to be updated by the developer to work on this version of iOS.
[ios-sim] An error was encountered processing the command (domain=FBSOpenApplicationServiceErrorDomain, code=1):
Simulator device returned an error for the requested operation.
[ios-sim] The request was denied by service delegate (SBMainWorkspace) for reason: NotFound ("Application "com.domain.myapp" is unknown to FrontBoard").
Underlying error (domain=FBSOpenApplicationServiceErrorDomain, code=1):
        The request to open "com.domain.myapp" failed.
        The request was denied by service delegate (SBMainWorkspace) for reason: NotFound ("Application "com.domain.myapp" is unknown to FrontBoard").
[ios-sim] logPath: /Users/xxx/tabris_proj/build/cordova/platforms/ios/cordova/console.log
Command finished with error code 0: /Users/xxx/tabris_proj/build/cordova/platforms/ios/cordova/node_modules/ios-sim/bin/ios-sim launch,/Users/xxx/tabris_proj/build/cordova/platforms/ios/build/emulator/MyApp.app,--devicetypeid,com.apple.CoreSimulator.SimDeviceType.iPhone-13-Pro,--log,/Users/xxx/tabris_proj/build/cordova/platforms/ios/cordova/console.log,--exit
Simulator successfully started via `ios-sim`.

does tabris xframework support x86 / x86_64 ?

@karolszafranski can you please respond to this issue ?


Serve Over HTTPS With SSL

I have been using this software for a number of years but with customization's for my own setup and have decided to contribute some back. This pull request will add the ability for the garage controller software to serve the web application with SSL for secure communication using HTTPS.
To maintain simplicity, this provides the option to switch HTTPS on. It does not handle communication on both HTTP and HTTPS ports at the same time. The end user must choose HTTP or HTTPS.

This looks good.  Thanks.  Could I as you to alter line 267 of controller.py so that instead of calling:
if not self.config['config']['use_https']:
It will first check to make sure that 'use_https' is defined in the config?  That will give us a little backwards compatibility.
Something like
if not 'use_https' in self.config['config'] or not self.config['config']['use_https']:

Thanks for the feedback. I have added compatibility checking for the config option as requested and tested.

The way I've achieved HTTPS access was by fronting the python HTTP server with nginx in HTTPS only. This also allows you to use an oauth proxy for authentication via Google or other authentication provider.


Add more metrics to flow and job executions.

New metrics:
-flow success meter
-flow killed meter
-flow killing meter
-flow time to kill histogram
-flow time to start histogram
-job success meter
-job fail meter
-job killed meter

Pull Request Test Coverage Report for Build 5164

57 of 59 (96.61%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 40.646%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/FlowRunner.java
30
31
96.77%

azkaban-exec-server/src/main/java/azkaban/execapp/FlowRunnerManager.java
0
1
0.0%

Totals

Change from base Build 5162:
0.09%

Covered Lines:
13031

Relevant Lines:
32060

💛  - Coveralls


Improve Deprecation Guidelines more

Which issue does this PR close?
Follow on to https://github.com/apache/datafusion/pull/13701
Rationale for this change
I missed one of @findepi 's suggestions on https://github.com/apache/datafusion/pull/13701#discussion_r1877921868, so let's fix that here.
Also there was another good idea:

https://github.com/apache/datafusion/pull/13701#discussion_r1884581982

What changes are included in this PR?
Improve the guidelines to be clearer and more concise
Are these changes tested?
CI
Are there any user-facing changes?

Thanks for the review @findepi


Too many info for「呼」

Found 4 etymologies

While debugging the issue a breakpoint was set inside EtymologyRunnable, but after resume it seemed like it's task was already hijacked and updateUi() was issued for a different Fragment.
May be related to #22.

Not an issue, there are several etymologies possible for a kanji.


[Soneium] Add support for Soneium Testnet

If this is your first time integrating, the following information is required

A short introduction: Sony Block Solutions Labs launches this Ethereum layer-2 solution to empower builders and drive Web3 adoption.
Document: https://soneium.org/en/docs/
Social media links (telegram, discord, website)

If you want to integrate a chain, you would need to provide

Name of the blockchain: Soneium Minato
Type of the blockchain: EVM
Blockchain providers: https://rpc.minato.soneium.org/
Metadata of the blockchain: https://docs.soneium.org/docs/builders/overview
Logo of the blockchain: updating
Explorer: https://explorer-testnet.soneium.org/

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Resolved in https://github.com/Koniverse/SubWallet-Extension/issues/3518


Provide onDidHighlght hook

fix #12
To create minimap plugin without mess
Things to consider

Performance impact by creating decoration for whole buffer
Update spec

ServiceProvider(=quick-highlight)

package.json

  "providedServices": {
    "quick-highlight": {
      "description": "quick-highlight events",
      "versions": {
        "1.0.0": "provideQuickHighlight"
      }
    }
  },

ServiceConsumer: 3rd party package which use quick-highlight service

package.json

  "consumedServices": {
    "quick-highlight": {
      "versions": {
        "1.0.0": "consumeQuickHighlight"
      }
    }

code to use service

  consumeQuickHighlight(quickHighlightService) {
    const boundHandler = this.onChangedHighlights.bind(this)
    this.subscriptions.add(quickHighlightService.onDidChangeHighlight(boundHandler))
  },

Hi t9md, is there any blocker on this, that I could help with ?

No if you agree with that I will merge it.

I'm happy with the exposed service, thanks 👍
I will publish the minimap integration on apm when this is merge an published too.

Close in favor of #17.
And will send PR to you minimap-plugin soon.


test suite

Add some more tests and update the bench-file.
Nothing much, but it should be enough to get information
about the performance of the interpeter.

This is part of the way to the target, but it's not as much as I want it to be. I would love to see some more tests.


Overwrite original document by default

My expectation when using this utility is that I will be unlocking my original document. Instead of creating a new document in the "unlocked" directory, the default behavior should be to overwrite the original document.
To handle scenarios in which the original document should be maintained, I would also suggest adding some sort of "--preserve" flag to keep the original documents around.

Hmm... This was a consideration of mine when designing the tool, however I was hesitant to make it destructive by default and decided an unlocked copy was the way to go.
I do like the idea of introducing a flag to allow the user to decide though. As I'm pretty sold on keeping it non-destructive by default this would look something like "--overwrite". I'll introduce this the next time I work on the project.

Hmm... This was a consideration of mine when designing the tool, however I was hesitant to make it destructive by default and decided an unlocked copy was the way to go.
I do like the idea of introducing a flag to allow the user to decide though. As I'm pretty sold on keeping it non-destructive by default this would look something like "--overwrite". I'll introduce this the next time I work on the project.


#1699 make executor filters configurable

This basically implements what's suggested in #1699 (using proposal 1)
Please see individual commits to see what's changed.
Added unit tests.

If it is not clear to you yet, I am interested in retiring this entire feature and replace it with random assignment and leverage flow controls at the individual executor level.

I'm sry but I don't see the logical transition here. What's the answer you are expecting? What's the reason for "retiring this entire feature"? After all you have not yet proven to me that random assignment is better than current design after I typed this much. I guess I still don't understand why we keep talking about random assignment as that's more of an alternative to comparator in current design not filter.
Users should have the right to choose what they want. If you want to rip off all the code and replace it with something else that's ok but that should be on the 4.x road map.

I don't honestly care about what random assignment is. Maybe that's better. But my point is we should not be randomly taking features away in a public software project.

@HappyRay the PR to make job retry when memory is not enough: https://github.com/azkaban/azkaban/pull/937

Hmm after checking that PR if I understand your proposal correctly, you want to let scheduler send jobs to executor regardless, and let executor determine when to actually execute it based on certain conditions.
This might be fine in certain scenarios but I don't think that is right in general. Think of a scenario where executors run long-running jobs. If new jobs get blindly dispatched to busy executors, they will be stuck there forever until other jobs finish, which might take hours even days. That will severely delay the new jobs, whereas in the current model, they will be queued on the web node side, and I can easily mitigate the congestion by adding more executor nodes.
If you want to let executor do the job, the only right way I can think of is changing the push model to pull model. That means jobs will only get picked up by an executor node if it is capable of handling it (or it thinks so), otherwise it will stay in a queue. However that's a fundamental change and I don't think it can handle more sophisticated scheduling logic.
Again, I think users should have the right to choose what they want. That is one of the fundamental principles of building any successful OSS projects.

I think it seems like we are mixing two separate issues:
Issue 1:  Unless there is at least 6 GB memory free on executor web server is not dispatching executions to this executor and 6 GB is not configurable - this creates issues in smaller VM instances.
(Which this PR attempts to solve)
Issue 2: Dispatching logic in itself may not be optimal, and perhaps random assignment is one possible solution
(Which this PR is not trying to solve)
I think #2 is a broad issue - and needs more discussion. I have created Issue #1739 to track this.
I do not see why we should not fix issue 1 right now unless we have an alternative easier fix available.


refactor: redo binary codec types to simplify

High Level Overview of Change
This PR:

Fixes a typing issue with the binary parser that was caused by a circular dependency (which we now know how to resolve)
Renames SerializedDict -> STObject and SerializedList -> STArray (to match rippled names)
Refactors the list of types in FieldInstance to pull directly from xrpl.core.binarycodec.types, so that doesn't need to be edited every time a new type is added

There are no feature changes here; this is just an internal refactor.
Context of Change
I was adding some new types to the binary codec and thought that needing to add them in 2 places was unnecessary. I fixed a few other things to make the binary codecs more analogous as well.
Type of Change

[x] Refactor (non-breaking change that only restructures code)

Test Plan
CI passes. Tests that test the internal objects were edited to use the new type names (the methods exposed externally have not changed at all).

@mvadari Would you consider minor version bump?

@mvadari Would you consider minor version bump?

There's literally no feature change - this PR is just refactors. No one should be using this library at the depth where I changed things.
We'll have to do a minor version bump for #397 anyways, so it can be rolled into that.


"Nearby Place found" despite already Nearby upload

Via Nearby, I took a pic of place1
While entering title, it asked me whether itis a pic of place2:

I said no, obviously.
Sad result: pic is linked from no Wikidata item.
https://commons.m.wikimedia.org/wiki/File%3A日本福音ルーテル社団.jpg

@nicolas-raoul Can I work in this
?

@madhurgupta10 Sure, thanks! :-)

Just to make sure I was explicit:
When I come from Nearby, I already have a Wikidata item QID number associated, so the app should not display any "Nearby Place Found". Actually it should not even perform the latitude/longitude retrieval and Wikidata request, as they take time for nothing in that case.
:-)

I agree, the solution would be to simply disable the search if the user is doing a direct Nearby upload.

The issue is still present, I uploaded the picture below via Nearby, and P18 did not get filled: https://commons.wikimedia.org/wiki/File:Embassy_of_Slovakia,_Tokyo_2.jpg

Actually my last 5 Nearby uploads have all failed to get P18, this bug is particularly penalizing in dense cities.

@madhurgupta10 I can take this issue if your are busy

@macgills Sure, thank you!


Power not working?

When applying a preset with finitialdriveforce it doesn't work until I go into vMenu and toggle the engine upgrade. Can be any level of engine.
Ive reinstalled this on a server with no other resources and still the same issue.
Ive also been on servers where I can apply my preset without this issue.

This seems one of the things that should be handled by the fivem API themselves, the extra-natives to edit handling might require a check and/or more documentation

This seems one of the things that should be handled by the fivem API themselves, the extra-natives to edit handling might require a check and/or more documentation

This seems one of the things that should be handled by the fivem API themselves, the extra-natives to edit handling might require a check and/or more documentation

No, it's from your handling editor script. FiveM api has nothing to do with this, I made my own handling editor and it works without any issues. It's very good overally, but it has some bugs you still haven't bothered to fix after a whole year. Not only finitialdriveforce does not apply, but fInitialDriveMaxFlatVel as well. You must use "SetVehicleEnginePowerMultiplier" after fInitialDriveMaxFlatVel and "SetVehicleMod" after fInitialDriveForce for those values to apply in-game.
P.S. I enjoy your work a lot, keep creating!


Rti merge

@YaphetKG I tested the changes and it works for me on my side... let me know when you wanna work on testing these changes on RENCI infrastructure


New owner requires refresh before it shows up on landing page

If manage access is used to add a new owner to a resource, then the manage access panel is closed, the new owner does not show up on the landing page until the landing page is refreshed.  Consider having close manage access trigger a page refresh so that actions in manage access are shown to have taken effect.

I am able to reproduce this on www. It isn't a regression. @dtarb Are you aiming to get this fixed for 1.19?

@horsburgh There are two scopes of refresh. One is only refreshing the fields on the resource landing page that are affected by the sharing panel. The second is the entire resource landing page. I think we should the first one. Using ajax call to update just portion of the page. I checked the hsapi, I did spot an REST API to do that. @Maurier can shed light on this.

I agree with @engineSound in that we should update the items via an ajax call rather than refreshing the page. We already have a response set up for this action, it would be a matter of adding this data and use it to update the UI.

I agree that this is not a regression.  It does not have to be fixed for 1.19.  I am happy for the solution to be whatever is best.  "consider trigger page refresh" was one suggestion, but if there is a better way to do it, I am happy for you to do that.

We should aim for zero page reload once the user is in resource edit mode. There are still couple of use cases where the page elements don't get updated until a manual page reload is done. @Maurier I can help on backend code changes to avoid any need for page reload.

@engineSound is this deprecated by new Landing page?

@cuahsimarko It is the current behavior of the new landing page.

@dtarb I'm unable to reproduce this problem. When I add an owner, I'm required to update the Quota holder, then remove myself as owner, at which time I get a warning dialog. Upon accepting consequences of removing myself as owner (assigning the new owner), it takes me to the My Resources View and the resource no longer appears in that view.

@cuahsimarko I do not think your attempt followed what I did.  I did not describe doing anything about quota holder.  Closing based on no reading the issue properly does not seem warranted.  This may not be a high priority issue, but please look at it to get an understanding of what it actually is.

changed owner is not reflected on page.zip

The way this was addressed in https://github.com/hydroshare/hydroshare/pull/3390, in my opinion, is inefficient. Pressing the button to dismiss the window now refreshes the page. This unnecessarily introduces yet another page reload, which we are trying to get rid of.
This could have been implemented better by updating the UI in the AJAX response that was already set up.