Push workflow

Push workflow to work with Travis based insights-frontend-builder-common scripts

Codecov Report

Merging #1768 (a8f912d) into master (c9bb4ec) will increase coverage by 1.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1768      +/-   ##
==========================================
+ Coverage   77.40%   78.47%   +1.07%
==========================================
  Files         295      295
  Lines        4686     5128     +442
  Branches      902     1181     +279
==========================================
+ Hits         3627     4024     +397
- Misses        960     1082     +122
+ Partials       99       22      -77

Impacted Files
Coverage Δ

src/components/i18n/i18nProvider.tsx
57.14% <0.00%> (-9.53%)
:arrow_down:

src/pages/state/maintenance/maintenanceState.tsx
66.66% <0.00%> (-8.34%)
:arrow_down:

src/store/common.ts
81.81% <0.00%> (-7.08%)
:arrow_down:

...ponents/reports/reportSummary/reportSummaryAlt.tsx
52.94% <0.00%> (-7.06%)
:arrow_down:

src/api/reports/reportUtils.ts
27.77% <0.00%> (-6.71%)
:arrow_down:

src/api/exports/exportUtils.ts
29.16% <0.00%> (-5.84%)
:arrow_down:

src/api/tags/tagUtils.ts
30.00% <0.00%> (-5.30%)
:arrow_down:

src/api/forecasts/forecastUtils.ts
31.25% <0.00%> (-4.47%)
:arrow_down:

src/utils/permissionsComponent.tsx
30.76% <0.00%> (-4.02%)
:arrow_down:

...d/azureCloudDashboard/azureCloudDashboardCommon.ts
33.33% <0.00%> (-3.34%)
:arrow_down:

... and 94 more

Continue to review full report at Codecov.

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


How to show custom image inside bar chart tooltip?

What problem does this feature solve?

What does the proposed API look like?

tooltip.formatter accepts HTML - Demo Code

Thanks for the previous response. Actually i have to render image inside tooltip for react native application so what is the approach?

Maybe others may answer better for react native, I'm not familiar.
But if the issue is about remote vs local (or embedded) images then one could use SVG :
tooltip: { formatter: `<svg width="100px" height="100px"><path d="M 10,30 A 20,20 0,0,1 50,30 A 20,20 0,0,1 90,30 Q 90,60 50,90 Q 10,60 10,30 z" /></svg>` }

I am currently using ECharts for a React Native app.  I need to render a custom tooltip that includes both images and text with various colors and styles. Is it possible to create a custom tooltip in ECharts for React Native?
Additionally, I have attached a demo image for reference.

How the hell I can add image in tooltip in react native ehart in line chat ?
I am fucked here

Thanks for the previous response. Actually i have to render image inside tooltip for react native application so what is the approach?

I am facing this same issue ?


Remove Razor Pages specific things from the MVC (Model-View-Controller) template #10935

Summary of the changes (Less than 80 chars)

Remove RazorPages extension if template is not an Auth one

Addresses #10935
Per @rynowak the RazorPages extensions are needed for auth projects because of the Identity library. If the project generated is not an auth one, the extensions are not needed.

@ryanbrandenburg flaky test failure

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=238057

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2489

Whoops, I'm still catching up on emails, it appears that we're mostly not merging things into master at the moment. But the PR looks good, we'll merge it for Preview8.


Wemo motion sensor not triggering automations anymore after a few hours since version 0.80

Home Assistant release with the issue:

Last working Home Assistant release (if known):
Before 0.80.0
Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

Description of problem:
Since version 0.80.0 -> 0.80.3 the Wemo motion sensor isn't triggering automations nor recorded in "history" a few hours after booting HA (it works only for a few hours).
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:
I checked in the wemo app and the sensor still works when it fails to trigger automations in HA.

After 12h I confirm it is solved in version 0.81.0.


Add support for feature/* branches

Summary
Qiskit is going to release the first major version, and post-1.0 development requires more strict stability policy as described in https://github.com/Qiskit/documentation/pull/366. However, we still want a pattern for experimental feature development, which must be agile and can be fail-fast (easily reverted) as we expect to quickly gain practice and confidence during more integration tests. One possible solution is an experimental decorator that emits user warning and hacks API document, but this doesn't fit in with epic level development that would introduce many classes and public methods during development. We may prefer feature branches rather than the decorator in some situation.
Details and comments
This PR lets our CI recognize feature/* branch.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
17 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 87.075%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.42%

crates/qasm2/src/parse.rs
12
97.13%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
60806

Relevant Lines:
69832

💛  - Coveralls


Was Harry Burisch in the Navy?

yes, according to obit Las Vegas Sun [Obituaries for August 10, 2000](https://lasvegassun.com/news/2000/aug/10/obituaries-for-august-10-2000/)


add ArbitraryAttributes parameter (#274)

Finally!
It was missing
Anyway, shall we write at least one unit test for this? for example using BitButton and arbitrary attributes?
@maryamhdr @msynk

Unfortunately, It seems this implementation does not work!
It might be because of the difference between the definition location of the property vs the usage location of it and how the SetParametersAsync is overridden in this project.

let's move writing appropriate unit tests for this feature to another issue.


Validation error calling vkGetPhysicalDeviceSurfaceSupportKHR

I am writing a macOS app and have used the Cube source as an example, however I use the Vulkan SDK instead of using MoltenVK directly and loading them via vkGetInstanceProcAddr() as cube.c does.
I link against libvulkan.dylib and libMoltenVK.dylib and #include <MoltenVK/mvk_vulkan.h> in order to expose the metal extension automatically and to use this to create the surface:
VkMetalSurfaceCreateInfoEXT createInfo = {};
createInfo.sType = VK_STRUCTURE_TYPE_METAL_SURFACE_CREATE_INFO_EXT;
createInfo.pNext = NULL;
createInfo.flags = 0;
createInfo.pLayer = (const CAMetalLayer*)initInfo.windowHandle;

result = vkCreateMetalSurfaceEXT(_instance, &createInfo, NULL, &_surface);

I have enabled validation layers by setting VK_LAYER_PATH in the Xcode scheme (and disabled Sandboxing so the app can access the layers) and it's giving me the following validation error:
Invalid VkSurfaceKHR Object 0x6000029009a0. The Vulkan spec states: surface must be a valid VkSurfaceKHR handle (https://www.khronos.org/registry/vulkan/specs/1.1-extensions/html/vkspec.html#VUID-vkGetPhysicalDeviceSurfaceSupportKHR-surface-parameter)
For this code:
VkBool32 presentSupport = false;
vkGetPhysicalDeviceSurfaceSupportKHR(deviceInfo.device, i, _surface, &presentSupport);

However the surface pointer looks good to me:
(lldb) p _surface
(VkSurfaceKHR) $0 = 0x0000600002914e70
(lldb) p deviceInfo.device
(VkPhysicalDevice) $1 = 0x0000600000245200

Am I doing something fundamentally wrong here?  The code runs fine if I disable validation and the codebase runs fine under Windows/GLFW.
I hacked the Cube sample to also use vkCreateMetalSurfaceEXT() but I don't believe it honours the VK_LAYERS_PATH envvar as it's not using the Vulkan loader and therefore I could not enable the validation layers (I passed --validate to cube_main() and demo->validate is TRUE).
Any help greatly appreciated.

I think the validation layer may not even support VK_EXT_metal_surface. I know the CTS doesn't use that extension--it just uses the old VK_MVK_macos_surface extension.

Hey thanks for the reply.  Are there any disadvantages to using VK_MVK_macos_surface over VK_EXT_metal_surface?

VK_MVK_macos_surface is now deprecated in favor of VK_EXT_metal_surface. So a future version of MoltenVK might rip the old extension out. We just need to implement support for the new extension in the validation layers, and test this in the CTS, and...
But other than that, not really. They largely provide the same functionality.

OK nice one, many thanks.  I'll support both for now.


Change the linux binary to xdg-open

Changelog
Changed the zeal option to xdg-open, which allows zeal on flatpak to pick up the requests.
It also allows another dash-compatible app to run too.
Checklist

[x] Code compiles correctly
[X] Add or update tests, if possible
[ ] All tests passing
[X] Update the README, if necessary
[X] New contributor? added myself via npm run contributor:add

How do I run the tests? :sweat_smile:

@deerawan  we really need this. Could you make a new release with this?


Update DG and parsers

Update find commands parsers, add User Stories and add diagrams.

Codecov Report
Base: 77.65% // Head: 77.73% // Increases project coverage by +0.08% :tada:

Coverage data is based on head (33afdd6) compared to base (86d145f).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #218      +/-   ##
============================================
+ Coverage     77.65%   77.73%   +0.08%
- Complexity      784      790       +6
============================================
  Files           117      117
  Lines          2318     2327       +9
  Branches        288      294       +6
============================================
+ Hits           1800     1809       +9
  Misses          430      430
  Partials         88       88

Impacted Files
Coverage Δ

...va/seedu/hrpro/logic/parser/FindCommandParser.java
100.00% <100.00%> (ø)

...edu/hrpro/logic/parser/FindStaffCommandParser.java
100.00% <100.00%> (ø)

...eedu/hrpro/logic/parser/FindTaskCommandParser.java
100.00% <100.00%> (ø)

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

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


Add support for Windows ARM64

Summary
As mentioned in this cmdstan issue, R and RTools have released previews of their Windows ARM64 versions.
The TBB is the only part of our existing build system that breaks (across CmdStan, Stan, and Math), as far as I can find.
Tests
I've tested that CmdStan builds and runs models with reduce_sum and STAN_THREADS=true using the RTools ARM64 toolchain.
I'm not sure if there's a way to add any CI or similar for this, since GH doesn't have windows arm64 runners yet.
Side Effects
N/A
Release notes
Added support for Windows ARM64 with RTools ARM64 toolchain
Checklist

[x] Copyright holder: Andrew Johnson
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Is there any downstream changes this is based on?
it needs an entry in the STAN_CHANGES file but otherwise looks reasonable

Being able to use cmake would be nice if for no other reason than because it would let us use oneTBB's other improvements like this.
Anyway, I think it's a pretty small set of people who actually have an arm64 Windows machine. As long as these changes don't actively break x86 I think they're totally fine


Fix MemoryError in case of invalid checksum

io.utils.download_file calls itself to retry the download in case checksum verification fails. In case the "true" checksum is incorrect, this leads to an endless recursion stopped eventually by a MemoryError. I suggest to raise a RuntimeError instead of retrying. Alternatively, we could introduce a parameter to specify how many times the download should be retried.

Good idea! Some suggestions:

I'm fine with raising an exception, no need to retry.
I wouldn't include the Raises section here because the error message should be clear already (see also the recommendation on this optional section in the docs).
I'd include the actual and expected checksums in the error message – unless of course the string is too long. WDYT?
Can you add an entry to CHANGELOG.md (I guess under "Fixes")?

I wouldn't include the Raises section here because the error message should be clear already (see also the recommendation on this optional section in the docs).

Okay, I included a sentence in the function description, so one doesn't need to try out how it will crash in case verification fails.

Can you show how an error would look like?

RuntimeError: Checksum mismatch for README.md:
    '06e13a3e2bd156e66c49fdbc10bc78be' (excpected)
    '06e13a3e2bd156e66c49fdbc10bc78bf' (calcuated)

Another typo: calcuated -> calculated 😄

Thanks @hofaflo!


dialog_cmac: update XCVR_TX_SCHED_DELAY

XCVR_TX_SCHED_DELAY is time the device needs to wake up before a
scheduled event occurs. With higher mbox size (1024), CMAC can take
longer to process the scheduled events. Scheduled events are processed
in an interrupt context and this parameter gives enough time to service
the scheduled event.
Signed-off-by: Naveen Kaje<EMAIL_ADDRESS>

I am wondering if we should set this based on the mailbox size. I am not sure how many pther configurable factors influence this number as well. @andrzej-kaczmarek do you have any comments about this?

I updated the change to depend on mailbox size. Not sure if we have to account for C2S or S2C alone is sufficient.

I am wondering if we should set this based on the mailbox size. I am not sure how many pther configurable factors influence this number as well. @andrzej-kaczmarek do you have any comments about this?

I updated the change to depend on mailbox size. Not sure if we have to account for C2S or S2C alone is sufficient.

tbh I do not really see any relation between mbox size and timer interrupt latency.
Larger c2s mailbox means it's less likely that CMAC will spin waiting for free space in mailbox to write data and this would only matter if writes disable interrupts, but iirc they do not.
Larger s2c mailbox means there may be more data for CMAC to read in single interrupt, but it should be enough that LL_TIMER2LLC irq has higher priority than SYS2CMAC, iirc the former has prio 2 while the latter has default prio (lowest).
So do you know what exactly causes delays here?

tbh I do not really see any relation between mbox size and timer interrupt latency.
Larger c2s mailbox means it's less likely that CMAC will spin waiting for free space in mailbox to write data and this would only matter if writes disable interrupts, but iirc they do not.
Larger s2c mailbox means there may be more data for CMAC to read in single interrupt, but it should be enough that LL_TIMER2LLC irq has higher priority than SYS2CMAC, iirc the former has prio 2 while the latter has default prio (lowest).
So do you know what exactly causes delays here?

So the only problem I could think of is reading s2c mailbox in an interrupt context causes some extra delays, although as I wrote above I think it should not really matter since timer interrupt has still higher priority. Anyway, should be really no problem if we just defer reading from that queue to LL task, here are quick patches for core and nimble to try:
https://github.com/andrzej-kaczmarek/apache-mynewt-core/commit/96d0f6e05779d1af27918d68b23ca6b033397c95
https://github.com/andrzej-kaczmarek/apache-mynewt-nimble/commit/665f92130f560f73e6ea7a8f4d5694e358f1d1dc

So the only problem I could think of is reading s2c mailbox in an interrupt context causes some extra delays, although as I wrote above I think it should not really matter since timer interrupt has still higher priority. Anyway, should be really no problem if we just defer reading from that queue to LL task, here are quick patches for core and nimble to try:
https://github.com/andrzej-kaczmarek/apache-mynewt-core/commit/96d0f6e05779d1af27918d68b23ca6b033397c95
https://github.com/andrzej-kaczmarek/apache-mynewt-nimble/commit/665f92130f560f73e6ea7a8f4d5694e358f1d1dc

@andrzej-kaczmarek The reason this change was made was something you had commented on in the past. Ben pulled up a note saying if the mailbox size the scheduling delay should be increased from 250 to 300. This was not done because there was an issue seen in the code. So if you do not think it necessary then we have no issues.

@andrzej-kaczmarek The reason this change was made was something you had commented on in the past. Ben pulled up a note saying if the mailbox size the scheduling delay should be increased from 250 to 300. This was not done because there was an issue seen in the code. So if you do not think it necessary then we have no issues.

One more thing to check: set only c2s mbox size to 1024 and leave s2c on default value. Large c2s mbox size guarantees that data can be pushed from CMAC quickly so it will not waste time spinning and waiting for sys while smaller s2c mbox size limits amount of data coming into CMAC. That should workaround the problem since I am pretty sure writing from CMAC to large mbox does not have any negative impact on performance (should be the opposite), it's just reading that can potentially cause issue.

One more thing to check: set only c2s mbox size to 1024 and leave s2c on default value. Large c2s mbox size guarantees that data can be pushed from CMAC quickly so it will not waste time spinning and waiting for sys while smaller s2c mbox size limits amount of data coming into CMAC. That should workaround the problem since I am pretty sure writing from CMAC to large mbox does not have any negative impact on performance (should be the opposite), it's just reading that can potentially cause issue.

@wes3 so could be that I had a hunch that larger s2c mbox size may introduce some timing issues, but tbh I do not recall what was my reasoning in the past. Anyway after reconsidering it now "from scratch" I think instead of just increasing scheduling delay to another magic number would be better to handle this in a cleaner way - deferring mbox read to task context seems like reasonable solution. I applied both patches to my local tree and will test them for any regressions, will push proper PRs if it's all ok.

@wes3 so could be that I had a hunch that larger s2c mbox size may introduce some timing issues, but tbh I do not recall what was my reasoning in the past. Anyway after reconsidering it now "from scratch" I think instead of just increasing scheduling delay to another magic number would be better to handle this in a cleaner way - deferring mbox read to task context seems like reasonable solution. I applied both patches to my local tree and will test them for any regressions, will push proper PRs if it's all ok.

is this stil needed?

I don't think we need this, there were significant optimizations done in CMAC code that seem to resolve pretty much all known issues, at least those I was aware of.


1677.1 Add ReleaseNotes.md in admiral repo to update the release notes for every release

Shippable/pm/issues/9686

RELEASE_NOTES.md file added in admiral repo. Closing.

RELEASE_NOTES.md file removed in admiral repo. Closing.
releaseNotes are available in https://github.com/Shippable/admiral/tree/master/releaseNotes


Questions on Carla Evaluation

Hi,
I have few questions regarding Carla results.

For the paper, have you trained VAD  on Carla and then reported the evaluation results or used your open loop checkpoint (pretrained on nuscenes) for evaluation.
Can you please share your closed loop evaluation script  (such as carla_agent.py as in stp3) so we can reproduce the closed loop results?
Can we directly use an open loop checkpoint in your evaluation script to see closed loop evaluation results?

I would appreciate your quick response on this matter.
Best,
Burhan

Hi Burhan,
Sorry for the late reply.

We trained VAD on Carla and then reported the evaluation results.
The closed-loop scripts will be open-sourced along with VADv2 as soon as possible.
We haven't tried this, but I think the results won't be satisfactory due to the domain gap.

If you have any other questions, please feel free to discuss them with us.

@byaman14 Hi, we are the authors of Bench2Drive and we would like to introduce that we have implemented VAD (non-official) in CARLA with code, checkpoints, and closed-loop results open sourced. Feel free to try it. https://github.com/Thinklab-SJTU/Bench2DriveZoo/tree/uniad/vad

Hi @jiaxiaosong1002, sounds great, thanks a lot!!

@jiaxiaosong1002 ,hi, do you reproduce the results successfully in carla? It seems that your reported driving score is much lower that the score reported in the paper of VADv2.


Incorporate data from the signup form

#34 and #35 already contributed to that.

In fact, use only that data.


feat: Update onboarding content and help links

Description
Update onboarding content and help links
Task Item
#minor

Coverage increased (+0.02%) to 52.325% when pulling 0158b324b56d22cd7e39791ff60a851335c6bd61 on durnford/feat/onboarding-content into 1503c58be52629e21935dc92c88d58ae6152a1d6 on main.


Parse for JSON?

Hello, i think it is a bit tricky to actually validate json. If i see correctly, the only accepted type for Parse() is map[string]any. Would it be possible to also accept something very common like a json.RawMessage or the aliased byte[]?
Because i want to validate my json but i'm NOT on a http server (which is a common use case) i need to first unmarshal into a struct (so my json: struct tags work as expected) and then pass the struct to zog to validate.
So in summary, it would be nice to have the ability to Parse a struct OR some method to parse directly byte[].

Hey! thanks for creating the issue. There are two things that are planned that I hope will help address this.
The first is that I agree that zog should provide some way of parsing raw json, but I haven't decided on how this will be done yet.
The second is the Validate method which has been planned for a while but I haven't gotten around to releasing it because it causes some pains with the current mental model. But I should be able to release the first working version of this hopefully over these holidays. That way you can just parse the json into your struct then validate it directly


[Bug]: Version 0.8 document parsing error

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
013db
Other environment information
OS：ubuntu 22.04
GPU: RTX A5000
CUDA 12.1

Actual behavior
1.Document parsing failed
2.Login with the initial account(admin@ragflow.io:admin) reported a password error, but the newly registered account can log in normally
Expected behavior
No response
Steps to reproduce
cd docker
sudo docker compose -f docker-compose-base.yml up -d
cd ..
sudo ./entrypoint.sh (The entrypoint.sh file has been modified with relevant configurations）
cd web
npm run dev

Additional information
Docker is running normally：
d93cb611d7ca   redis:7.2.4                                            "docker-entrypoint.s…"   17 hours ago   Up 17 hours             <IP_ADDRESS>:6379->6379/tcp, :::6379->6379/tcp                       ragflow-redis
dc45f60a1afa   mysql:5.7.18                                           "docker-entrypoint.s…"   17 hours ago   Up 17 hours (healthy)   <IP_ADDRESS>:5455->3306/tcp, :::5455->3306/tcp                       ragflow-mysql
5358c6058bf1   quay.io/minio/minio:RELEASE.2023-12-20T01-00-02Z       "/usr/bin/docker-ent…"   17 hours ago   Up 17 hours             <IP_ADDRESS>:9000-9001->9000-9001/tcp, :::9000-9001->9000-9001/tcp   ragflow-minio
44a08c5965e8   docker.elastic.co/elasticsearch/elasticsearch:8.11.3   "/bin/tini -- /usr/l…"   17 hours ago   Up 17 hours (healthy)   9300/tcp, <IP_ADDRESS>:1200->9200/tcp, :::1200->9200/tcp             ragflow-es-01
ES is working normally：
curl -u elastic:infini_rag_flow  -XGET http://<IP_ADDRESS>:1200
{
"name" : "es01",
"cluster_name" : "docker-cluster",
"cluster_uuid" : "O8R09jtqQAeYddcdhvQ_zA",
"version" : {
"number" : "8.11.3",
"build_flavor" : "default",
"build_type" : "docker",
"build_hash" : "64cf052f3b56b1fd4449f5454cb88aca7e739d9a",
"build_date" : "2023-12-08T11:33:53.634979452Z",
"build_snapshot" : false,
"lucene_version" : "9.8.0",
"minimum_wire_compatibility_version" : "7.17.0",
"minimum_index_compatibility_version" : "7.0.0"
},
"tagline" : "You Know, for Search"
}
minio.log:
Fail put eeb3b5f440c211efa0af4ed83c63d3f1/GBT 10184-2015电站锅炉性能试验规程 .pdf: S3 operation failed; code: NoSuchKey, message: Object does not exist, resource: /eeb3b5f440c211efa0af4ed83c63d3f1/GBT%2010184-2015%E7%94%B5%E7%AB%99%E9%94
%85%E7%82%89%E6%80%A7%E8%83%BD%E8%AF%95%E9%AA%8C%E8%A7%84%E7%A8%8B%20.pdf, request_id: 17E1A862DC1DA965, host_id: dd9025bab4ad464b049177c95eb6ebf374d3b3fd1af9251148b658df7ac2e3e8, bucket_name: eeb3b5f440c211efa0af4ed83c63d3f1, object_nam
e: GBT 10184-2015电站锅炉性能试验规程 .pdf
es.log:
ES updateByQuery deleteByQuery: NotFoundError(404, 'index_not_found_exception', 'no such index [ragflow_683eb2ba403d11ef88154ed83c63d3f1]', ragflow_683eb2ba403d11ef88154ed83c63d3f1, index_or_alias)【Q】：{'match': {'doc_id': 'd3303a9a40c
811ef91cd4ed83c63d3f1'}}
ES updateByQuery deleteByQuery: NotFoundError(404, 'index_not_found_exception', 'no such index [ragflow_683eb2ba403d11ef88154ed83c63d3f1]', ragflow_683eb2ba403d11ef88154ed83c63d3f1, index_or_alias)【Q】：{'match': {'doc_id': 'd3303a9a40c
811ef91cd4ed83c63d3f1'}}
ES updateByQuery deleteByQuery: NotFoundError(404, 'index_not_found_exception', 'no such index [ragflow_683eb2ba403d11ef88154ed83c63d3f1]', ragflow_683eb2ba403d11ef88154ed83c63d3f1, index_or_alias)【Q】：{'match': {'doc_id': 'd3303a9a40c
811ef91cd4ed83c63d3f1'}}
cron_logger.log:
An error happened while trying to locate the files on the Hub and we cannot find the appropriate snapshot folder for the specified revision on the local disk. Please check your internet connection and try again.
ERROR.log:
Fail put eeb3b5f440c211efa0af4ed83c63d3f1/GBT 10184-2015电站锅炉性能试验规程 .pdf: S3 operation failed; code: NoSuchKey, message: Object does not exist, resource: /eeb3b5f440c211efa0af4ed83c63d3f1/GBT%2010184-2015%E7%94%B5%E7%AB%99%E9%94
%85%E7%82%89%E6%80%A7%E8%83%BD%E8%AF%95%E9%AA%8C%E8%A7%84%E7%A8%8B%20.pdf, request_id: 17E1A862DC1DA965, host_id: dd9025bab4ad464b049177c95eb6ebf374d3b3fd1af9251148b658df7ac2e3e8, bucket_name: eeb3b5f440c211efa0af4ed83c63d3f1, object_nam
e: GBT 10184-2015电站锅炉性能试验规程 .pdf
ES updateByQuery deleteByQuery: NotFoundError(404, 'index_not_found_exception', 'no such index [ragflow_683eb2ba403d11ef88154ed83c63d3f1]', ragflow_683eb2ba403d11ef88154ed83c63d3f1, index_or_alias)【Q】：{'match': {'doc_id': 'd3303a9a40c
811ef91cd4ed83c63d3f1'}}
ES updateByQuery deleteByQuery: NotFoundError(404, 'index_not_found_exception', 'no such index [ragflow_683eb2ba403d11ef88154ed83c63d3f1]', ragflow_683eb2ba403d11ef88154ed83c63d3f1, index_or_alias)【Q】：{'match': {'doc_id': 'd3303a9a40c
811ef91cd4ed83c63d3f1'}}
ES updateByQuery deleteByQuery: NotFoundError(404, 'index_not_found_exception', 'no such index [ragflow_683eb2ba403d11ef88154ed83c63d3f1]', ragflow_683eb2ba403d11ef88154ed83c63d3f1, index_or_alias)【Q】：{'match': {'doc_id': 'd3303a9a40c
811ef91cd4ed83c63d3f1'}}
An error happened while trying to locate the files on the Hub and we cannot find the appropriate snapshot folder for the specified revision on the local disk. Please check your internet connection and try again.

There is no default account such as<EMAIL_ADDRESS>I guess minio is not connected. You might need to check conf/service_conf.yaml.

1.If the database data is completely cleared during startup, the system will initialize a superuser account, and there will be a prompt in the running log:
【INFO】Super user initialized. email<EMAIL_ADDRESS>password: admin. Changing the password after logining is strongly recomanded.
This account can be found in the database, but cannot be logged in with the given password. I guess the reason is that the encryption algorithm used during initialization and the verification algorithm used during login are not consistent
The account registered through the login page is not a superuser account by default, and I don't know what the difference is between them.
2.The logs of this project are not very user-friendly. After tracking and researching, I found that this error was caused by not adding "HF-INDPOINT= https://hf-mirror.com" in the environment variable.
@KevinHuSh Thank you for your enthusiastic reply

Hello,
I'm having the exact same issue. @charliboy could you please provide more information on what exactly solved the problem? Where exactly did you add "HF-ENDPOINT", I'm finding it in multiple files?


Feature/tilelayer rework

@charlenni This is a change I talked about in this comment here. This will impact your VectorTileLayer.


HIVE-25621: Added HivePrivilegeObjects for the commands Alter table m…

…erge/concatenate

What changes were proposed in this pull request?
Added HivePrivilegeObjects for alter table merge/concatenate commands.

Why are the changes needed?
To prevent  malicious users from doing alter operations on table.

Does this PR introduce any user-facing change?
No

How was this patch tested?
Local machine, remote cluster.

@dengzhhu653 - Can you please review my patch?

@dengzhhu653 - Can you please review my patch again? Thanks

@dengzhhu653 Can you review my patch again?

@dengzhhu653 Can you review my patch again?

leave a minor comments, also the failed tests seem related, others look good to me.

@dengzhhu653 - Tests are green now. Can you do another review here?


fix: time-convert is too new. Use time-to-seconds

time-to-seconds is quite old: C-h f time-to-seconds says:

Probably introduced at or before Emacs version 24.4.

As a bonus, it conveys intent slightly better, as time-to-seconds is a
more descriptive function name.
Closes #19

Thanks!

Thanks! I confirm that this fixes the problem for me.


[#2954] feat(all): add case-sensitive capability

What changes were proposed in this pull request?

support case-sensitive capability for name
Hive catalog use case-insensitive capability

Why are the changes needed?
Fix: #2954
Does this PR introduce any user-facing change?
no
How was this patch tested?
tests added

@mchades One thing I think we can improve is to use wrapper design pattern https://zhuanlan.zhihu.com/p/43368677.
We can have one dispatcher that handles all the capability things, another dispatcher that handles operation logic, one another dispatcher that does the event posting. using wrapper pattern to chain them altogether like using InputStream. With this, all the related logic will only reside in the related class, no need to change the existing a lot. which makes the code easy to understand, can you please think a bit on this?

For example, like adding new CapabilityAppliedFilesetDispatcher, and chain with FilesetEventDispatcher then FilesetOperationDispatcher. Like:
  public NameIdentifier[] listFilesets(Namespace namespace) throws NoSuchSchemaException {
    try {
      // Apply the capabilities to fileset
      // Call `FilesetEventDispatcher`'s listFilesets
      NameIdentifier[] nameIdentifiers = dispatcher.listFilesets(namespace);
      return nameIdentifiers;
    } catch (Exception e) {
      eventBus.dispatchEvent(
          new ListFilesetFailureEvent(PrincipalUtils.getCurrentUserName(), namespace, e));
      throw e;
    }
  }

I'm doing the refactor, but I would name the class xxxStandardizedDispatcher(such as TableStandardizedDispatcher) and it focus on standardize the input and out of the xxxOperations, because some other capabilities, such as managedStorage need to use internally in xxxOperations. WDYT? @jerryshao

OK, I think we can do it as possible as we can.

OK, I think we can do it as possible as we can. But maybe not just standardize the name, others like check nullable, default value, it can also be done in this dispatcher?

yes

@jerryshao Please help review again when you have time, thx

@mchades there're some style issues, please fix them.


Exclude the MISRA Website from CI-CD link verifier checks

This fixes the issue seen in https://github.com/FreeRTOS/FreeRTOS-Kernel/pull/880
Putting the fix here so that all the repos that contain this link will receive this change instead of opening a per repo PR


Prevent possible crash on startup.

We saw crashes on this line, which can only happen if transport_ is bad.
The Transport has a shared pointer to the Endpoints, so the Transport can only go out of scope after the endpoints, not before it. Because the pointer was not initialised it could happen on startup, if transport_ was not a nullptr and not yet assigned to the valid Transport we could interact with it when traces are produced before setTransport is called.
Also updated cbor to include https://github.com/iwanders/cbor/pull/6
@efernandez, @jasonimercer

:+1:


Does This Creates all the containers into the Azure Storage account?

Do this activity creates all the seprate containers into the azure storage account, if we talk about S3 to Azure Storage account data movement?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 218316eb-5f14-3904-e596-c9d1679ad80c
Version Independent ID: f3191220-8c7b-3b6a-7c9c-2b0cc2f4211f
Content: Copy files from multiple containers with Azure Data Factory
Content Source: articles/data-factory/solution-template-copy-files-multiple-containers.md
Service: data-factory
GitHub Login: @dearandyxu
Microsoft Alias: yexu

@Anmolgan81 Thank you for your interest in Azure products and services. We are investigating and will get back to you soon.

@Anmolgan81 Yes, this activity creates all the separate containers into the Azure storage account or Azure Data Lake Store.
As mentioned in the document: The solution template described in this article helps you to copy files from multiple files, containers, or buckets between file stores. For example, maybe you want to migrate your data lake from AWS S3 to Azure Data Lake Store. Or maybe you want to replicate everything from one Azure Blob Storage account to another Azure Blob Storage account. This template is designed for these use cases.
Hope this helps.

@Anmolgan81 We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Fixing NewSession not prompting for unsaved data

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

Recent changes introduced a bug in which pressing New Session did not prompt the user about unsaved data. This bug has been fixed.
after saving, if the Post Weight field is modified, the Save button toggles purple reminding users to save this value.
"File/New" was routed to "clear", I now routed it to "New Session", since we already have "File/Clear"

What issues or discussions does this update address?

resolves #396

Describe the expected change in behavior from the perspective of the experimenter
If you press New Session with unsaved data, you will be prompted to save
Describe the outcome of testing this update on a rig in 447

[ ] tested in 447

@XX-Yin This is ready for review


[Object recolorizer 0.1.3] Issue with Gdevelop version 5.3.198

Is there an existing issue for this?

[X] I have searched the existing issues

Enter the name of the extension
Object recolorizer 0.1.3
Describe the bug of the extension
TypeError: sprite._animations is not iterable
at Object.GDJSInlineCode [as userFunc0xc34160] (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/gdjs-evtsext__recolorizer__spriterecolorizer.js:78:36)
at Object.gdjs.evtsExt__Recolorizer__SpriteRecolorizer.SpriteRecolorizer.onCreatedContext.eventsList0 (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/gdjs-evtsext__recolorizer__spriterecolorizer.js:123:91)
at SpriteRecolorizer.gdjs.evtsExt__Recolorizer__SpriteRecolorizer.SpriteRecolorizer.onCreated (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/gdjs-evtsext__recolorizer__spriterecolorizer.js:192:91)
at r.onCreated (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/runtimeobject.js:1:3072)
at new r (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/spriteruntimeobject.js:1:495)
at createObject (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/RuntimeInstanceContainer.js:1:5517)
at createObjectsFrom (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/RuntimeInstanceContainer.js:1:1755)
at c.loadFromScene (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/runtimescene.js:1:2136)
at r._loadNewScene (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/scenestack.js:1:1589)
at r.push (file:///C:/Users/bclau/AppData/Local/Temp/GDTMP--1/preview/scenestack.js:1:1364)
Steps to reproduce

Use Object recolorizer in the current scene
Launch the game
Game crash using the new gdevelop version 5.3.198

GDevelop platform
Desktop, Web, Mobile
GDevelop version
5.3.198
Platform info

OS (e.g. Windows, Linux, macOS, Android, iOS)

OS Version (e.g. Windows 10, macOS 10.15)

Browser(For Web) (e.g. Chrome, Firefox, Safari)

Device(For Mobile) (e.g. iPhone 12, Samsung Galaxy S21)

Additional context
No response

Thank you for reporting this issue, it's fixed by this PR:

https://github.com/GDevelopApp/GDevelop-extensions/pull/1247


Build 8.0 beta

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Sorry, I accidentally targeted this to main instead of rc.  I'll close it and open a new PR.


refactor(breadbox): Remove catalog nodes

This is the last step in removing catalog nodes from breadbox 🎉
Asana
This ended up being pretty simple since we've already removed all catalog-node-dependent functionality. My process for this last step was to:

Replace all of the CatalogNode references in tests (usually by querying TabularColumn instead), and then confirm the tests still pass
Remove CatalogNode references from the rest of breadbox. At this point, it was mostly just places where CatalogNodes were being inserted or deleted from the database.
Drop the CatalogNode table from the database.

Is this comment still true?
https://github.com/broadinstitute/depmap-portal/blob/f2a8a79aba451297a4f9c50184f3f395c6b070b6/breadbox/breadbox/models/dataset.py#L201-L203

@rcreasi good question! It looks like that is being used more regularly now, so I just updated the comment:
# Denormalized data: this information is also stored in the dataset's corresponding
# feature/sample/index type name column (the information is duplicated here for convenience)
dataset_dimension_type = Column(String, nullable=True)


formatting large files with repetitive statements, i.e. insert

Hi again!
I have some SQL files which contain large number (hunderds or thousands) of INSERT statements. It is not unusual to use SQL Developer to export the content of a table to insert statements. I even have an example of Oracle's demo APEX Interactive Grid app which has a stored procedure with >4000 insert statements. Formatting such sql file doesn't work... SQLcl just hangs. This is not a problem of your tool, but it might be nice if you can add some logic to format.js or to the arbori script to skip such files.
Best regards,
Plamen

Hi @plamen-ppv
Right, there are some SQL constructs which are slow to process and may fail when running from SQLcl. There are two workarounds:
a) use the standalone formatted tvdformat.jar. It's still slow, but it usually finishes (not feasible for interactive usage)
b) exclude the files from processing via the an ignore configuration . I use this approach in various projects where I format the code via pre-commit hook. You could also create a batch-file so you do not have to pass the ignore parameter each time.
I do not see an easy way to identify problematic files. It is not always the size. Also, very large lists (in or type constructors) are problematic, even if the file is small. So using the ignore parameter seems to be the best approach IMO.

If the root cause of the slow Arbori queries cannot be addressed an option could be to define a timeout option with a default value of let's say 10 seconds per file. If the timeout is reached the formatting process for the current file could be aborted.

Adding such timeout option will be the most pragmatic solution! it will cover all cases, regardless of origin.


WIP: RAF tool i18n and configurable county threshold

Adds a language data attribute to the container element in the RAF tool template and updates the RAF JS to include translations that enable a Spanish-language version of the tool. (Note: the JS should be imported from an npm module when PR is ready for final review)
Additions

data-language attribute on container element in RAF tool template to pass the page's language to the tool
New RAF tool struct block with optional county-threshold field
data-county-threshold attribute on container element in RAF tool template to pass the optional county-threshold value to the tool

Changes

Update RAF JS

How to test this PR

Check out branch and run gulp copy & migrations
Replace existing Rental Assistance Finder Tool block on English RAF tool page with new RAF Tool (configurable) block
Navigate to English RAF tool page and verify that English-language tool content displays as expected
In admin, create a new browse page and set page language to Spanish. Add RAF tool (configurable) and verify that tool content displays in Spanish.
Change language of page to anything other than Spanish or English and verify that tool displays in English
Verify that by default, tool shows county dropdown when a state with more than 10 results is selected rather than old threshold of 5 (New Jersey should show county dropdown, Arizona should not)
Change county_threshold value in RAF Tool block and verify that tool uses new threshold value instead of default 10

Screenshots

Notes and todos

This PR includes copied RAFT JS. We will either update this PR or open a new one immediately afterwards that imports the JS from a recently-published npm module.

Checklist

[ ] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing, e.g., "Mega Menu: fix layout bug", or "Docs: Update Docker installation instructions".

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Future todos are captured in comments and/or tickets
[ ] Project documentation has been updated, potentially one or more of:

This repo’s docs (edit the files in the /docs folder) – for basic, close-to-the-code docs on working with this repo
CFGOV/platform wiki on GHE – for internal CFPB developer guidance
CFPB/hubcap wiki on GHE – for internal CFPB design and content guidance

Front-end testing

Browser testing
Visually tested in the following supported browsers:

[x] Firefox
[x] Chrome
[x] Safari
[x] Edge 18 (the last Edge prior to it switching to Chromium)
[x] Internet Explorer 11
[x] iOS
[x] Android

Accessibility

[ ] Keyboard friendly (navigable with tab, space, enter, arrow keys, etc.)
[ ] Screen reader friendly
[ ] Does not introduce new errors or warnings in WAVE

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Does not introduce new lint warnings
[ ] Flexible from small to large screens

A couple notes on my latest commit:

I moved the erap folder inside unprocessed into the apps folder. I'm not sure if there was a reason it was out there on its own, but I figured this was better for folder organization.
Inside that erap folder, I simply added @cfpb/rental-assistance-finder as an npm dependency.
I changed the copy:erap gulp task to simply move the main.js file from inside unprocessed/apps/erap/node_modules/ to the routes folder where it was going before.

If there's any problem with this, let me know, but it seems to be working for me!

Thanks @higs4281 & @mistergone!

Love the translation work, too.


Set Focus in Amount Textbox

For Qtum-Qt v0.18.3, v0.18.2, maybe earlier, on the Send Coins page, after entering the amount to send in the Amount textbox, with the cursor in that textbox, pressing the "Enter" key will change that amount to the "use available balance" amount (Windows) or 0.0 (Linux). If the amount is entered and then clicked away from the Amount textbox, the amount will be shown correctly. Perhaps the focus must be set on the Amount textbox. Repeatable.
This works correctly for the Amount textbox on Send QRC20 tokens page. Pressing the “Enter” key after entering an amount on the Send to Contract Page Optional Amount textbox will highlight the amount but does not validate that amount.
Also, for the Amount textboxes, the increment/decrement control changes the amount of QTUM by 0.001 per click and for QRC20 tokens by 0.1 per click. Would 1.0 per click be more useful?

Is this still an issue?


Added functionality to xPfxImport and fixed issue #17

This pull request contains the following updates:

Added support for CurrentUser as location
Added Ensure parameter, to remove certificates where needed
Added Provider$ as filter in Validate-Thumbprint to resolve #17
Added a ValidationSet to the $Store parameter
Changed xPfxImport Keys to make them more Unique
Test script MSFT_xPfxImport.tests updated

This change is

Hi @DdenBraver, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@DdenBraver hi, please remove [ValidateSet()] for the store. I left it out specifically because you can't determine the set at design time; it's possible to add new stores. As an example, the WebHosting store does not exist until you install IIS.
A Dynamic Parameter could handle this, but since it's a DSC resource that would be useless.

@briantist Thanks for the feedback, I was indeed facing the same issue where I did want to validate the input against something.
What I could do is validate the $certpath exists.
I could place this in the Get-TargetResource (which gets called by Test-TargetResource) to throw an exception if its not available.

@KarolKaczmarek @briantist @TravisEz13
Please let me know how you think about my last comment, so I can change the code accordingly.

@DdenBraver  I agree that you cannot have a static validation of the stores.  Your proposal seems reasonable, but this assumes the resource is not supposed to create the store.  This also seems reasonable.

Reviewed 2 of 4 files at r2, 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

ReadMe.md, line 46 [r3] (raw file):
We need to at least note the breaking change and what the  user needs to change to work with this breaking change

Comments from Reviewable

ReadMe.md, line 46 [r3] (raw file):
I do not think this should be a breaking change, as the default value of the Ensure parameter is Present.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

ReadMe.md, line 46 [r3] (raw file):
'Store' went from optional to being a key.  This is a breaking change.

Comments from Reviewable

ReadMe.md, line 46 [r3] (raw file):
I just updated the readme.md showing this change.

Comments from Reviewable

Review status: 3 of 4 files reviewed at latest revision, 1 unresolved discussion.

ReadMe.md, line 46 [r3] (raw file):
@KarolKaczmarek Is there any special way we want to indicate a change is breaking?

Comments from Reviewable

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Surely it would need to have a major version bump on the next release cycle? And some big/bold in the readme.md!

Review status: all files reviewed at latest revision, 1 unresolved discussion.

ReadMe.md, line 46 [r3] (raw file):
@ddenbraver  I followed up with Karol offline, can you change * Updated xPfxImport Store parameter is now a key value making it mandatory·' to * Breaking Change - Updated xPfxImport Store parameter is now a key value making it mandatory·'

Comments from Reviewable

ReadMe.md, line 46 [r3] (raw file):
Changed as requested.

Comments from Reviewable

Reviewed 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

ReadMe.md, line 46 [r5] (raw file):
unicode character in readme

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

ReadMe.md, line 46 [r5] (raw file):
This is not blocking, consider fixing or filing an issue.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


Style landing page, add router

What was the feature/problem?
{Please write here}
How does this PR address the above feature/problem?
{Please write here}
Check lists (check x in [ ] of list items)

[ ] Test passed
[ ] Coding style (indentation, etc)

Additional Comments (if any)
{Please write here}

Codecov Report

Merging #22 into master will decrease coverage by 0.39%.
The diff coverage is 50%.

@@            Coverage Diff            @@
##           master      #22     +/-   ##
=========================================
- Coverage   73.07%   72.67%   -0.4%
=========================================
  Files          17       17
  Lines         182      183      +1
  Branches       12       12
=========================================
  Hits          133      133
- Misses         43       44      +1
  Partials        6        6

Flag
Coverage Δ

#api
65.64% <ø> (ø)
:arrow_up:

#ui
90.38% <50%> (-1.78%)
:arrow_down:

Impacted Files
Coverage Δ

ui/src/landing/Landing.js
83.33% <ø> (ø)
:arrow_up:

ui/src/app/App.js
100% <ø> (ø)
:arrow_up:

ui/src/Image/ImageButton/ImageButton.js
60% <50%> (-15%)
:arrow_down:

Continue to review full report at Codecov.

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


Reverse proxy load balancer

If I have a stateless service running on 4 nodes and another service uses the reverse proxy to hit the endpoint, then can we safely assume that reverse proxy is doing the load balancer work  and is distributing the load? If not, then what's the alternative to keep the services internal and still have a load balancer

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 31e0a1d2-789d-240f-be0b-892d43f80b66
Version Independent ID: aac58453-4adc-5c04-8067-d7107dc2df45
Content: Azure Service Fabric reverse proxy
Content Source: articles/service-fabric/service-fabric-reverseproxy.md
Service: service-fabric
GitHub Login: @BharatNarasimman
Microsoft Alias: bharatn

Thanks for the feedback! We are currently investigating and will update you shortly.

Reverse proxy would randomly pick one of the stateless instances to forward the request to.

Thanks @BharatNarasimman!
@dixitsuneel let us know if that information helps.

I will close this out but if you feel you need more information please just let me know.

@BharatNarasimman @MicahMcKittrick-MSFT  Thank you so much for the update.
It will be very useful if this information is clearly outlined in the docs for other pople

For sure.
@BharatNarasimman can you see if we can add this info into the doc?

https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reverseproxy
"TargetReplicaSelector This specifies how the target replica or instance should be selected.◦When the target service is stateful, the TargetReplicaSelector can be one of the following: 'PrimaryReplica', 'RandomSecondaryReplica', or 'RandomReplica'. When this parameter is not specified, the default is 'PrimaryReplica'.
◦When the target service is stateless, reverse proxy picks a random instance of the service partition to forward the request to."

How does the Reverse Proxy deal with service names that are overlapping? I have the following two services:
fabric:/FileAndServe.Xpress_DEV1/Web
fabric:/FileAndServe.Xpress_DEV1/Web/Core
Up until.... minutes ago... for a URI like http://CLUSTER:19081/FileAndServe.Xpress_DEV1/Web/Core/api/foo it was picking the second service (Core). Now it's picking the first one. I'm wondering if it was never deterministic, and I simply did something to cause it to pick the wrong order.

It looks like the original issue has been addressed, and since it is > 90 days, I am closing this. Thanks!


Update logic-apps-monitor-your-logic-apps.md

Fixed a minor typo.

@rionam : Thanks for your contribution to the Azure documentation! The author, @jeffhollan, has been notified to review your proposed change.

@rionam Thank you for catching this! @SyntaxC4

#sign-off

@JennieHubbard: I'm sorry - only the author of this article, @jeffhollan, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


v0.4.3 merge

Hi @Rosejoycrocker could you please run tests on your end and approve if all pass?  (PRs cannot be merged into main without review)
I've run tests on my end and they pass but would like confirmation.

Hi @Rosejoycrocker could you please run tests on your end and approve if all pass? (PRs cannot be merged into main without review)
I've run tests on my end and they pass but would like confirmation.

Sure, I'll run them now :)

Hey @ConnectedSystems, all tests passing on my end

I'll just quickly review code changes and then approve

Most if not all changes should have been reviewed before as this is the v0.4.3 branch

Yeah realising I've seen most of this, I'll approve now


Add benchmark for count vs select size

From #1168

We could use rubocop-performance, but probably not until we drop Ruby version conditional branches.


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In 6a5d770, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 16840 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in e864c6b after 1 hour, 42 minutes.


Request for ownership of abandoned dinnerbone/mcstatus

As per discussion near the bottom of https://github.com/pypa/warehouse/issues/1506, I decided to open an issue here to make this request.
I dislike that it has come to this, but I feel it necessary to take action in effort to either grab the attention of the author, or maintain the project myself.  It has been over a year since the last release of mcstatus to pypi.  Attempts to contact the author, Dinnerbone, by raising a PR on GitHub have failed https://github.com/Dinnerbone/mcstatus/pull/56  As did tweeting https://twitter.com/MarkL4YG/status/929362396317175808

Hello @kevinkjt2000, I have sent an email to @dinnerbone regarding your request. Hopefully we will hear from him soon and move forwards with it.

Hello @kevinkjt2000, @Dinnerbone has not responded in a timely fashion and the project is thus considered abandoned. However, before transferring ownership all requirements listed in the corresponding section of PEP 541 need to be satisfied:

[x] The project has been determined abandoned by the rules described above;
[x] The candidate is able to demonstrate their own failed attempts to contact the existing owner;
[ ] The candidate is able to demonstrate that the project suggested to reuse the name already exists and meets notability requirements;
[ ] The candidate is able to demonstrate why a fork under a different name is not an acceptable workaround;
[ ] Download statistics on the Package Index for the existing package indicate project is not being used; and
[ ] The maintainers of the Package Index don't have any additional reservations.

I'm concerned however about the download statistics, pepy.tech shows ~300 downloads a week and almos 2.5k downloads a month which I think we need to respect, which leads me to ask if if you've considered creating a fork under a different name.
Let us know what you think.

Where are notability requirements listed?  This seems vague to me in the PEP 541 document.  I do have a fork that I would like to release, and I even successfully tested publishing to the test pypi (about a year or so ago).
https://github.com/kevinkjt2000/mcstatus
Are pypi maintainers okay with having lots of little forks with alternate names?  To me that seems confusing and chaotic.  In my opinion, packages with zero active authors do not belong on pypi.  If that is not convincing enough, I may consider taking another name, but will likely just withdraw this request instead.
Yes the package is downloaded a fair bit, because it still works (mostly); however, PRs and issues are being ignored by the author.  I would want to leave dinnerbone as a co-owner too, in case he returns (i.e. stops blatantly ignoring his github, twitter, and email).  All I want is the ability to collaborate and have permission to publish a package where the original author is inactive.  I think of this situation as if it was someone from the community stepping in, after someone died, to continue maintenance (and possibly features? not likely, but I would at least review/accept PRs from others instead of ignoring them).

Should I fork, rename, and slowly steal away the users?  Personally, that feels odd to me.

I would favor a fork under the same name. The current project has several issues where PRs are already available.
I would say users benefits from fixes being finally applied after years are bigger than the possible drawbacks given a pledge of the new maintainers to not break existing (API) contracts without proper notice.
Semantic versioning would also help with this.
I would also offer to maintain the project regarding PR review, issues and code updates when required.

Let me ask: Is it possible to convince you of the benefits that everyone would gain from updates to the same package if we explicitly write transition documentation with specific pledges on how the development in the project continues including what will (bug fixes, additional features, ...) and will not (breaking changes) be done ?

Just want to add, I am in full support of this. I use this library and even messaged the author on twitter. He just ignores everything tho :( I do not want to tell every user who uses my project to go into dinnerbones libary and edit bugfixes in ( also, editing in bugfixes for every virtualenv :(). That feels terrible. I personally feel like using the same name would be helpful, since it is referenced a lot, but forces users to use a buggy version (and probably give up whatever they were trying to do, I was close to this too.)

Hey everyone,
I'm sorry for not catching and dealing with this much sooner.
I have no issues with transferring ownership, I (clearly) do not have time to maintain this project myself anymore. To that end, I have added kevinkjt2000 as Owner on pypi.org, and to my github repository. I do not mind, however, if you'd rather fork it and move it somewhere else, either.
Sorry again for not handling this in a nice way.
<3

@Dinnerbone Thank you so much for your response and allowing us to continue development! :raised_hands:

@Dinnerbone Thanks for responding.
@kevinkjt2000 If you're satisfied, could you please close this issue?

Whoa!  Was not expecting this turn of events!  Thanks Dinnerbone!

Thanks again @Dinnerbone !  I was able to release a new version and respond to a large number of the open issues.


refine servicebus session message handler

Description
refine servicebus session message handler

Codecov Report

Merging #639 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #639      +/-   ##
============================================
- Coverage     56.06%   56.01%   -0.06%
  Complexity      748      748
============================================
  Files           179      179
  Lines          3139     3142       +3
  Branches        229      231       +2
============================================
  Hits           1760     1760
- Misses         1244     1247       +3
  Partials        135      135

Impacted Files
Coverage Δ
Complexity Δ

...tegration/servicebus/ServiceBusMessageHandler.java
72% <ø> (ø)
9 <0> (ø)
:arrow_down:

...tion/servicebus/queue/ServiceBusQueueTemplate.java
50% <0%> (-3.2%)
6 <0> (ø)

Continue to review full report at Codecov.

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

@neuqlz
What's the context of this PR?

@neuqlz
What's the context of this PR?

It is related to this issue #594  and this PR #596.
I think there is something to refine, so I create this PR to do it.


about the results

Hi, thanks for your work. Can I ask a question? what's the meaning of the result "test_in     test_out    train_in    train_out"?

"train_in" indicates performance of training dataset, "train_out" indicates performance of validation dataset, and "test_in" indicates performance of test dataset. "test_out" is used for oracle model selection method in DomainBed paper, but it is not used in our work.

"train_in" indicates performance of training dataset (in-domain), "train_out" indicates performance of validation dataset (in-domain), and "test_in" indicates performance of test dataset (out-of-domain). "test_out" is used for oracle model selection method in DomainBed paper, but it is not used in our work.

OK. The explanation is very clear. Thank you for your quick reply.


Please correct any issues or errors

https://github.com/wizardofozzie/blockcypher-python/blob/master/blockcypher/__init__.py

No context/info


Replace empty function parameters with void

from: http://port70.net/~nsz/c/c99/n1256.html#6.11.6

The use of function declarators with empty parentheses (not prototype-format parameter type declarators) is an obsolescent feature.

and this SO answer suggest you should use void in the function parameter definition to explicitly say the function does not take any arguments.
Therefore, should these two functions be changed to use void, as in:
//h3Index.c
int H3_EXPORT(pentagonCount)(void) { return NUM_PENTAGONS; }

//baseCells.c
int H3_EXPORT(res0CellCount)(void) { return NUM_BASE_CELLS; }

Thanks for flagging this. I agree, it looks like they should be defined as (void). I opened a PR, #788 to change that.


csv columns include irrelevant headers

Bug report
__System info:__InfluxDB 1.2.0
Steps to reproduce:

Create some users or retention policies
Use show users with an Accept: text/csv header

Expected behavior: The csv output would mirror the JSON output, containing user and admin columns:
{
  "results": [
    {
      "series": [
        {
          "columns": [
            "user",
            "admin"
          ],
          "values": [
            [
              "john",
              true
            ],
            [
              "steve",
              false
            ]
          ]
        }
      ]
    }
  ]
}

In CSV, I would expect this to be:
user,admin
john,true
steve,false

Actual behavior: The output includes empty name and tags columns:
name,tags,user,admin
,,john,true
,,steve,false

Additional info: This appears to be the case in other statements too, like show measurements. In this case those prefix columns actually conflict with other columns:
name,tags,name,duration,shardGroupDuration,replicaN,default
,,autogen,0s,168h0m0s,replicaN,true
,,7d,168h0m0s,24h0m0s,replicaN,false

I stumbled upon this while implementing csv support in node-influx. There's a script that I use to automatically generate fixtures, you can see the results in this folder.

Hm, it seems to me that this happens because the JSON won't emit a name or tags section if they are empty, but I have it so the CSV always emits those. I could make it so it doesn't omit names or tags when neither of those are present.
It might present some other difficulties though. I think if the name is present we have to include a column for tags (even if there aren't any) mostly because there could be. We can be reasonably certain that if there is no name though that there are no tags and can omit both of them.

It also looks like CSV doesn't work for some outputs. For example, SHOW STATS will do a recoverable panic.


Control Site-Specific Outlining Based on Types with Full-Confidence

Binary Ninja's outlining feature is marvelous, however sometimes there are specific instances where the outlining transform is not optimal and I would like to convert it back into regular assignments.
For Example:

Support was initially added for this in 4.1.5255-dev, and expanded upon in 4.1.5536-dev.
Simply set the destination type to something that is incompatible with the outlined builtin. In the example above, hit 'y' in the UI on the destination variable, and accept it's default type of uint32_t. If you want the outlined version to return, set the type to char var_2c[4]


[Bug]: Invalid request url: /openpose_editor_index?theme=dark

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits of both this extension and the webui

What happened?
I deployed a WebUI on an Alibaba Cloud DSW server. However, when I open the editor, it displays a blank window and shows an invalid address. I'm unsure how to modify this address or which file to modify.
Steps to reproduce the problem

What should have happened?

Commit where the problem happens
webui:
controlnet:
openpose-editor:
What browsers do you use to access the UI ?
Google Chrome
Command Line Arguments
--theme=dark

Console logs
/mnt/workspace/stable-diffusion-webui
Python 3.10.13 (main, Sep 11 2023, 13:44:35) [GCC 11.2.0]
Version: v1.9.0
Commit hash: adadb4e3c7382bf3e4f7519126cd6c70f4f8557b
loading WD14-tagger reqs from /mnt/workspace/stable-diffusion-webui/extensions/stable-diffusion-webui-wd14-tagger/requirements.txt
Checking WD14-tagger requirements.
Launching Web UI with arguments: --xformers --theme=dark
[2024-04-24 18:13:31,183] [INFO] [real_accelerator.py:161:get_accelerator] Setting ds_accelerator to cuda (auto detect)
2024-04-24 18:13:31.648939: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2024-04-24 18:13:31.687591: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-04-24 18:13:32.413393: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Tag Autocomplete: Could not locate model-keyword extension, Lora trigger word completion will be limited to those added through the extra networks menu.
[-] ADetailer: Failed to load model 'face_yolov8n.pt' from huggingface
[-] ADetailer initialized. version: 24.4.2, num models: 4
ControlNet preprocessor location: /mnt/workspace/stable-diffusion-webui/extensions/sd-webui-controlnet/annotator/downloads
2024-04-24 18:13:47,033 - ControlNet - INFO - ControlNet v1.1.445
2024-04-24 18:13:47,243 - ControlNet - INFO - ControlNet v1.1.445
sd-webui-prompt-all-in-one background API service started successfully.
== WD14 tagger /gpu:0, uname_result(system='Linux', node='dsw-362506-5885b645c4-xmwqx', release='4.19.24-7.34.cbp.al7.x86_64', version='#1 SMP Tue Jun 20 06:15:49 UTC 2023', machine='x86_64') ==
Loading weights [cdf971b4a0] from /mnt/workspace/stable-diffusion-webui/models/Stable-diffusion/AWPainting_v1.4.safetensors
2024-04-24 18:13:48,018 - ControlNet - INFO - ControlNet UI callback registered.
Running on local URL:  http://<IP_ADDRESS>:7860
Creating model from config: /mnt/workspace/stable-diffusion-webui/configs/v1-inference.yaml
Applying attention optimization: xformers... done.
Model loaded in 2.4s (load weights from disk: 1.1s, create model: 0.3s, apply weights to model: 0.4s, load textual inversion embeddings: 0.3s, calculate empty prompt: 0.2s).

To create a public link, set `share=True` in `launch()`.
🤯 LobeTheme: Initializing...
Startup time: 29.5s (prepare environment: 6.8s, import torch: 5.5s, import gradio: 1.0s, setup paths: 0.6s, other imports: 0.4s, load scripts: 11.9s, create ui: 1.0s, gradio launch: 2.2s).
2024-04-24 18:16:59,702 - ControlNet - INFO - Preview Resolution = 512
2024-04-24 18:30:28,199 - ControlNet - INFO - Preview Resolution = 512

Browser logs
I don't know where it is
Additional information
No response

我是更改了https://github.com/huchenlei/sd-webui-openpose-editor/blob/main/scripts/openpose_editor.py，里面的<EMAIL_ADDRESS>


[qt5-base:x64-linux] build failure

Host Environment

OS: CentOS 7
Compiler: gcc v9.3.1

To Reproduce
Steps to reproduce the behavior:
./vcpkg install qt5-base:x64-linux
Failure logs
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:142 (message):
Command failed: /opt/rh/devtoolset-9/root/usr/bin/make -j17
Working Directory: /home/aaron.shelley/vcpkg-master/buildtrees/qt5-base/x64-linux-dbg
See logs for more information:
/home/aaron.shelley/vcpkg-master/buildtrees/qt5-base/build-x64-linux-dbg-out.log
/home/aaron.shelley/vcpkg-master/buildtrees/qt5-base/build-x64-linux-dbg-err.log
Call Stack (most recent call first):
ports/qt5-base/cmake/install_qt.cmake:66 (vcpkg_execute_build_process)
ports/qt5-base/portfile.cmake:311 (install_qt)
scripts/ports.cmake:135 (include)
Error: Building package qt5-base:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with ./vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: qt5-base:x64-linux
Vcpkg version: 2020.06.15-unknownhash
Additional context
I have come across this error in an effort to try and find a snapshot of vcpkg that can successfully build ~300 packages I require. I was originally using vcpkg from 08/23/2020 commit but ran into issues with building some packages that were never resolved here. That being said, Qt5 built just fine. I attempted to build my ~300 packages from a 09/24/2020 commit and ran into Qt5 failing to build right away.
Finally, I tried to build Qt5 from vcpkg master branch as of today's date and ran into the exact same build error as described above.
build-x64-linux-dbg-err.log
build-x64-linux-dbg-out.log

cc @Neumann-A

Hi @ASxa86 does this issue still exist?

We haven't been able to repro this; if more information comes up, or this issue appears again, please reopen.


[help w/ form validation] How to remove 'valid' styling

I've asked on IRC, asked on a related issue, but I couldn't get an answer, so I'm asking here.
How do you remove ":valid" styling? I'm trying this:
@import "~bootstrap/scss/functions";
@import "~bootstrap/scss/variables";
@import "~bootstrap/scss/mixins";

$form-validation-states: map-remove($form-validation-states, "valid");

But after rebuilding the assets and reloading the page, I can see it didn't change anything.
This is the documentation page I was reading.
Bootstrap version 4.4.1

Looks like no error in your SCSS and it works correctly: https://repl.it/repls/BreakableMysteriousTrees.
Maybe there is a problem with your build environment.

You're right. It's working now.

Information for future readers with the same problem:
It's a Symfony project with Webpack Encore, and I create styles for each section of the site, like 'admin', 'user' and 'login'. I import Bootstrap in these files and perform variable overrides.
But one of them, app.scss, applies common styling to all sections of the site, and that's where I was trying to override the validation states. It doesn't work like this. I presume it's because we're already importing and overriding variables in the section-specific files, so the changes won't apply here.


Response ToString() overridden

Codecov Report

Merging #46 (17144fb) into master (4e96c00) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   95.83%   95.84%   +0.01%
==========================================
  Files          29       29
  Lines         768      771       +3
  Branches       96       97       +1
==========================================
+ Hits          736      739       +3
  Misses         12       12
  Partials       20       20

Impacted Files
Coverage Δ

SoftAPI/SoftAPIClient/MetaData/Response.cs
93.75% <100.00%> (+1.44%)
:arrow_up:

Continue to review full report at Codecov.

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


How to run the selected text ? and what is the keyboard short cut to do the same?

How to run the selected text (Command)? and what is the keyboard short cut to do the same?

Press F1 and search for that command:

This is a built-in command from VS Code, I don't think there is a shortcut assigned, but you can certainly assign one (I use Cmd + ' on Mac).

There is also documentation on assigning keybindings, HTH: https://code.visualstudio.com/docs/getstarted/keybindings


Clean up predicate ref

Coverage increased (+0.73%) to 57.83% when pulling 52f1efee257c92d95ad07c78d34d01f46e2a7e19 on predicateRef into 60c011908bc5f1676e59652b62fff94c71085435 on v2.


settings_notifications.js: Refactor and save setting on change.

I'm refactoring user notification settings and thought I'd also work on having them save automatically, similarly to user display settings. I had a few questions/ran into a few issues that I tried to solve for:

update_subscription_properties_backend in zerver/views/streams.py wasn't converting the strings 'true' and 'false' (from JS) to Python booleans, thus the changes to streams.py. I wasn't sure the correct way to do this piece.
I wasn't clear on what maybe_bulk_update_stream_notification_settings was doing in settings_notifications.js. I removed it and in manually testing, things seem to still be working, but I might be missing a piece.
Is there a preferred way to make a modal or a status div that sticks to the visible part of the settings container? (Currently, if someone is scrolled to the bottom of the container, the notification that something has been changed isn't in view).

Automated message from Dropbox CLA bot
@sarahcstringer, it looks like you've already signed the Dropbox CLA. Thanks!

Great, thank you for the feedback @timabbott ! I just updated the areas you pointed out.

I'm still working on having notification messages stick to the top of the settings div, but I'll submit that in a separate commit.

I fixed the (really silly) error I was making in settings_notifications.js, so I got rid of the unnecessary changes I'd made to streams and test_subs.
Also, it looks like this addresses issue #3264.

Agreed that this addresses #3264.

Nice, merged, thanks @sarahcstringer!  I marked your second commit as closing #3264 via the commit message (that's how you get GitHub to auto-close issues when something is merged).


Add git polling to jenkins quickstart templates

@EricJizbaMSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Update icon and doc images

Regenerate Doc Images + Icons for R2019b #236
https://www.cyberbotics.com/doc/guide/objects?version=update-icon-and-images

before:

after

before

after

I have improved the script, now 100% of the images are generated by the script without any manual edition. I checked all of them in the doc and couldn't see any issue with the cropping or background removal.

:+1:

👀


Firebase version

Hi,
There is any specific reason to keep the firebase version on 10.x and not to upgrade?

nope


Fix CX warning in OS update context

Description
Remove const qualifier for a CX field - in the context of an OS upgrade app.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[x] Other (for changes that might not fit in any category)

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

Comparison is base (e54fb1b) 69.09% compared to head (70d652b) 69.09%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #461   +/-   ##
=======================================
  Coverage   69.09%   69.09%
=======================================
  Files          11       11
  Lines         877      877
=======================================
  Hits          606      606
  Misses        271      271

Flag
Coverage Δ

unittests
69.09% <ø> (ø)

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

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


ScreenModel working not properly

It's look like ScreenModel logic working wrong. Overriden onDispose method in root screen not called when we leave this  screen.

Hey guys, the voyager owner is very busy and his wife is pregnant. Updates will come soon. Stay up to date.

@adrielcafe @programadorthi Hey. Any news on this?


[Twinkle-0010] Update filter panel

[ ] Implement new design for filter panel;
[ ] Incorporate to composite content panel;
[ ] Redesign filter logic;
[ ] Use hibernate for creating filtering query.


pac powerpages download not working correctly

Hello,
We have upgraded our portal from standard to enhanced. However, when we attempt to download the portal in Visual Studio Code, we encounter an error.
The error message we receive is:
Updating manifest...
Microsoft PowerPlatform CLI
Version: 1.29.11+g9e2b163
Online documentation: https://aka.ms/PowerPlatformCLI
Feedback, Suggestions, Issues: https://github.com/microsoft/powerplatform-build-tools/discussions
Error: Input string was not in a correct format.
After attempting to downgrade the CLI to versions 1.28.3 and 1.24.3, we still face the same error.
We've checked the log file located at C:\Users%username%\AppData\Roaming\Code\User\globalStorage\microsoft-isvexptools.powerplatform-vscode\pac\tools\logs, but we couldn't find any exceptions or errors recorded.
Any advice would be greatly appreciated.

Hi, bumping this! Did you find a reason and a fix for this behaviour? I'm encountering similar problem.

Hi, bumping this! Did you find a reason and a fix for this behaviour? I'm encountering similar problem.

Unfortunately, no. I have tried downgrading version of Power Apps CLI to 1.28.3 and 1.29.*  I have also been in contact with MS support regarding this, but they couldn't find what is causing this issue.

I have this too with the version 1.30.7.
The site seems to get downloaded but it's always outputting this error at the end since a couple of weeks.

Error: Input string was not in a correct format.

@positiveDimension Did you find anything since ?

Hey @marlierdamien, @positiveDimension, @Glideh, @dynam0r, can you try upgrading to PAC v1.32.8 as it has enhancements that should throw more light on this issue.


exploitdb: 2024-06-15 -> 2024-06-27

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/exploitdb/versions.
meta.description for exploitdb is: Archive of public exploits and corresponding vulnerable software
meta.homepage for exploitdb is: https://gitlab.com/exploit-database/exploitdb
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2024-06-27 in filename of file in /nix/store/bvli4ic8s3gryk9613qb6qy4r4qiadmg-exploitdb-2024-06-27

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

exploitdb

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bvli4ic8s3gryk9613qb6qy4r4qiadmg-exploitdb-2024-06-27 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/0215ebf7d92d7372a83bddd64c09e0b68248f7e7#exploitdb

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bvli4ic8s3gryk9613qb6qy4r4qiadmg-exploitdb-2024-06-27
ls -la /nix/store/bvli4ic8s3gryk9613qb6qy4r4qiadmg-exploitdb-2024-06-27/bin

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

1 package built:

exploitdb

Maintainer pings
cc @applePrincess @fabaff for testing.

Result of nixpkgs-review pr 322894 run on x86_64-linux 1

1 package built:

exploitdb


[BUG] App creation & updation fails when instance count is 0

Is there an existing issue for this?

[X] I have searched the existing issues

What version of the Terraform provider are you using?
No response
What version of the Terraform CLI are you using?
1.8.2
What CF API version are you using?
3.159.0
What type of issue are you facing
bug report
Describe the bug
When the app instance count is set to 0, the provider fails and throws the following exception:
Error: Provider produced inconsistent result after apply
│
│ When applying changes to cloudfoundry_app.wow4, provider "provider[\"registry.terraform.io/sap/cloudfoundry\"]"
│ produced an unexpected new value: .processes: planned set element
│ cty.ObjectVal(map[string]cty.Value{"command":cty.NullVal(cty.String), "disk_quota":cty.UnknownVal(cty.String),
│ "health_check_http_endpoint":cty.NullVal(cty.String), "health_check_interval":cty.NullVal(cty.Number),
│ "health_check_invocation_timeout":cty.NullVal(cty.Number), "health_check_type":cty.UnknownVal(cty.String),
│ "instances":cty.NumberIntVal(0), "log_rate_limit_per_second":cty.UnknownVal(cty.String),
│ "memory":cty.UnknownVal(cty.String), "readiness_health_check_http_endpoint":cty.NullVal(cty.String),
│ "readiness_health_check_interval":cty.NullVal(cty.Number),
│ "readiness_health_check_invocation_timeout":cty.NullVal(cty.Number),
│ "readiness_health_check_type":cty.UnknownVal(cty.String), "timeout":cty.NullVal(cty.Number),
│ "type":cty.StringVal("web")}) does not correlate with any element in actual.
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.

During update, the instance count remains same after the provider panics.
If it creating the app for the 1st time, the provider panics, but app gets deployed with 1 instance.
Expected Behavior
The application instance count is set to 0.
Steps To Reproduce
Execute terraform apply
Your Terraform Configuration
resource "cloudfoundry_app" "wow4" {
  name         = "wow4"
  space_name   = data.cloudfoundry_space.space.name
  org_name     = "PerformanceTeamBLR"
  docker_image = "kennethreitz/httpbin"
  routes = [
    {
      route = "wow2.cfapps.sap.hana.ondemand.com"
    }
  ]
  processes = [
    {
      type = "web"
      instances  = 0
    }
  ]
 }

Roles
No response
Add screenshots to help explain your problem
No response
Additional context
This is a valid scenario from the API context. It can set and scale down instances to 0

Closed by #119


CONTRIBUTING.md draft file discussion

So here's a first draft of a proposed CONTRIBUTING.md file. For the most part it's a collection of points on contribution guidelines from other large Github projects, with a few extra bits thrown in.
I'll keep ammending this pull request till we've got it how we want it, then rebase it all before it's merged in.
Suggestions/opinions/feedback welcome :-)
p.s formatted version here: https://github.com/chrismou/radiant-player-mac/blob/contributing/CONTRIBUTING.md

Suggestion for PR review process:
For PR review / merges, I personally am an advocate of having the PR author merge their own PRs (if they have permission), but only after getting a sign-off (LGTM) from one of the project collaborators.  That other collaborator is responsible for code review and is equally responsible for the code change after the LGTM.  And if the PR author merges the PR, then nothing will get merged before it's ready, if the PR wasn't good to go.  Obviously it works a little differently with non-collaborator PRs...

I'm in 2 minds about this. On a busier project, I'd ideally look for a couple of :+1:'s, but as people are busy and things are a bit quiet, for smaller PRs I'd personally be happy with just having one person review it. And yeah, fine with merging in myself, but would be interested to hear @kbhomes's opinion. I can understand if he's want to review all PRs before merging, but then we'd essentially be in the same position we were in before the collaborators were added.
@radiant-player/radiant-player-mac Any other thoughts?

@jacobwgillespie I prefer dropping a :shipit: vs a LGTM :smile:

Yep, :shipit: works :wink:

@chrismou I really appreciate your section on squashing commits. This should keep the history of Radiant relatively clean, and hopefully people will follow the one fix/branch guideline. If there are no objections, I say :shipit:, we should get this in so others know how to properly contribute to Radiant.

@jadengore Cheers dude. Still not sure how to address the bit about merging - the more I think about it, the more I'm finding myself agreeing with @jacobwgillespie about being responsible for your own merges.
Perhaps change that final line to:

Even if you have write access to the repository, do not directly push or merge pull-requests without first allowing others time to review your changes. Ideally you should have at least one :+1:, :shipit: or LGTM before you merge into master. If your changes are substantial or high risk, you may want to wait for more than one. Use your own judgement.

@kbhomes What are your thoughts overall?

I'm going to add my :+1: on that addition @chrismou.
And I'll add my :shipit: on this PR after that addition - we can always change the guidelines later if we need.

Cool, guidelines updated, merging now. Cheers guys!

@chrismou Might be nice to squash this next time, just a suggestion :wink:

I thought I did? Oh god, how did that merge get in there? Ha!
No way I can let that go. Fixing now... :stuck_out_tongue_winking_eye:

I thought I did? Oh god dammit, how did that merge get in there? Ha!
Where's the shame face emoji? :stuck_out_tongue_winking_eye:

OK, I'm going to pretend that the guidelines weren't official until after the merge :wink:

:+1:


Touch not working on Sunton ESP32-2432S028r

Describe the bug
Touch not working on Sunton ESP32-2432S028r
To Reproduce
When I got the display touch was working with the flashed demo.
Flashed esp32-2432s028r_full_4MB_v0.7.0-rc4_f76dee8.bin and configured wifi. Uploaded my page and voila, no touch whatsoever is working. Works on ESP32-8048S050 and ESP32-3248...
Expected behavior
Touch to work.
Screenshots or video
N/A

My bad. I didnt do calibration first :(

No problem. I was about to look into this, but good to hear it worked out.


Given groups=1, weight of size [320, 4, 3, 3], expected input[2, 12, 86, 57] to have 4 channels, but got 12 channels instead

Under the condition of lighting items with foreground and background, only the FBC model can be used. When using this sampler, an error occurs: !!! Exception during processing !!! Given groups=1, weight of size [320, 4, 3, 3], expected input[2, 12, 86, 57] to have 4 channels, but got 12 channels instead. Please help solve this issue, I’ve been trying to figure it out for two days without success.
1.txt
2.txt


Can it be used on tensorflow>= 2.2.0?

Hi, I watched the video and really like your excellent work. When I try to install it, the version error was shown. Is it possible to use it in a higher tensorflow version? Or I have to change my tensorflow version?
Collecting tensorflow>=2.2.0 (from tf-siren)
  Could not find a version that satisfies the requirement tensorflow>=2.2.0 (from tf-siren) (from versions: 0.12.1, 1.0.0, 1.0.1, 1.1.0rc0, 1.1.0rc1, 1.1.0rc2, 1.1.0, 1.2.0rc0, 1.2.0rc1, 1.2.0rc2, 1.2.0, 1.2.1, 1.3.0rc0, 1.3.0rc1, 1.3.0rc2, 1.3.0, 1.4.0rc0, 1.4.0rc1, 1.4.0, 1.4.1, 1.5.0rc0, 1.5.0rc1, 1.5.0, 1.5.1, 1.6.0rc0, 1.6.0rc1, 1.6.0, 1.7.0rc0, 1.7.0rc1, 1.7.0, 1.7.1, 1.8.0rc0, 1.8.0rc1, 1.8.0, 1.9.0rc0, 1.9.0rc1, 1.9.0rc2, 1.9.0, 1.10.0rc0, 1.10.0rc1, 1.10.0, 1.10.1, 1.11.0rc0, 1.11.0rc1, 1.11.0rc2, 1.11.0, 1.12.0rc0, 1.12.0rc1, 1.12.0rc2, 1.12.0, 1.12.2, 1.12.3, 1.13.0rc0, 1.13.0rc1, 1.13.0rc2, 1.13.1, 1.13.2, 1.14.0rc0, 1.14.0rc1, 1.14.0, 2.0.0a0, 2.0.0b0, 2.0.0b1)
No matching distribution found for tensorflow>=2.2.0 (from tf-siren)

Tensorflow 2.0 has been released for the last 6 months, which platform are you on?

Tensorflow 2.0 has been released for the last 6 months, which platform are you on?

Hi, I am using Ubuntu 18.04. I upgraded the pip version, and tf_siren can be installed now. Thanks!


Find IAM Roles for API Gateway lambdas

Need to install each lambda with a Role

Will need to generate roles to have permissions for each resource that matters. For now, just want things to execute correctly.

For now, will require users to specify IAM roles. Can generate them later.


Reading every batch while being written to the server

I was going through this Flight Java Example and realized that reading starts only after all the batches are written. This seems problematic if we are writing a huge amount of data in batches. Ideally we would expect every batch being read as soon as they are written to the server.
What are the changes needed in the example code to achieve this, if that is even possible? Please suggest.

Hi @iamsmkr can you explain? That is not true, the server gets data as soon as the client writes it. The example shows read-after-write because it's from the client's perspective.

@lidavidm Thanks for your reply. I have written a small working example (in Scala) for you to have a look here.
In the Main.scala, I am writing and reading in separate threads. What I would expect is that all batches are read as soon as they are written. However, I could do this only after "completing all sends per batch" i.e., calling listener.completed() after every listener.putNext(), as shown below:
  Future(reader.readMessages(1))

  writer.addToBatch(0, "Shivam")
  writer.addToBatch(1, "Shyam")
  writer.addToBatch(2, "Srinivas")
  writer.sendBatch()
  writer.completeSend()

  Thread.sleep(1000)

  writer.addToBatch(0, "Neha")
  writer.addToBatch(1, "Naveen")
  writer.addToBatch(2, "Nisha")
  writer.sendBatch()
  writer.completeSend()

First read in the following version won't work!
//  Future(reader.readMessages(1))

  writer.addToBatch(0, "Shivam")
  writer.addToBatch(1, "Shyam")
  writer.addToBatch(2, "Srinivas")
  writer.sendBatch()
//  writer.completeSend()
  reader.readMessages()                 // THIS DOESN'T READ THE FIRST BATCH AS EXPECTED!
  println(s"reading A")
  Thread.sleep(5000)

  writer.addToBatch(0, "Neha")
  writer.addToBatch(1, "Naveen")
  writer.addToBatch(2, "Nisha")
  writer.sendBatch()
  writer.completeSend()
  reader.readMessages()

Hope I could explain it better this time. Please suggest what changes could be made to fix this. Thanks!

Well, yes of course: the server implementation isn't actually making the data available until the call finishes:
  override def acceptPut(context: CallContext, flightStream: FlightStream, ackStream: StreamListener[PutResult]): Runnable = {
    val batches = new util.ArrayList[ArrowRecordBatch]()
    () => {
      var rows: Long = 0
      while (flightStream.next()) {
        val unloader = new VectorUnloader(flightStream.getRoot)
        val arb = unloader.getRecordBatch
        batches.add(arb)
        rows = rows + flightStream.getRoot.getRowCount
      }

      val dataset = ArrowFlightDataset(batches, flightStream.getSchema, rows)
      datasets.put(flightStream.getDescriptor, dataset)
      ackStream.onCompleted()
    }
  }

This buffers the data in memory and does not update the member variable datasets until the call completes. How would Flight be able make the data available to the getStream implementation for you?
Flight is just a framework for building services, so if you want to be able to read data while writing, Flight will not get in your way of implementing that, but neither will it do that for you automatically. In this case, you'll have to update datasets as soon as the call starts, mutate the dataset as the call progresses, and have some way of coordinating between acceptPut and getStream so the latter knows when all data has been uploaded. And you'll want to handle the client disconnecting, etc.

Thanks @lidavidm. This clarifies! But I do wonder what APIs to use to achieve this. Is there any example code that I could refer? Any help is much appreciated!

The APIs are not different. This is up to the application. You are synchronizing two threads, that is nothing specific to Arrow or Flight.

@iamsmkr do you have other questions here?

Which file and line are you pointing at? @lidavidm

https://github.com/iamsmkr/arrow-flight-scala/blob/b7ecb25d4e4e772e47c3c6ee0485134580b428df/src/main/scala/com/iamsmkr/arrowflight/ArrowFlightProducer.scala#L19-L33
this just creates a new queue and overwrites the previous one on every iteration - why bother with the queue in the first place?

@lidavidm I am not sure if I understand you. It is creating a new blocking queue only per file descriptor. See this -> https://github.com/iamsmkr/arrow-flight-scala/commit/06482900ebb4ae7fcfbf40be9f4a7dfb751c0cb0
The print statement is called only once per endpoint. If the endpoint is same new messages are added to the same blocking queue.

Ah…I can't deal with the lack of braces, I skipped that line entirely, makes sense


Remove two empty files

This PR removes two empty files from the repo. I'm just starting out with talon; sorry if these empty files actually serve a secret purpose :-)

I'm going to merge this, it seems sensible and those files are certainly doing nothing right now. Maybe they were there as stubs to remind us to do something in future? @knausj85 any idea?

This was probably just an oversight, thanks for deleting them.


jo_0 wants an entrypoint into building a world for V-Sekai

Describe the proposed feature and how it helps to overcome a problem or limitation

jo_0 — Yesterday at 13:41
Where is the best entrypoint for someone interested in building a world for v-sekai?

Other information about your context or usecase.
No response
If known, describe how your proposal will work, with code, pseudo-code, mock-ups, or diagrams
Write a tutorial for jo_0
Anything else
Include pictures.

jo_0's current project is targeting WebXR. There are many resources jo_0 could learn to get existing artwork into Godot but jo_0 wasn't sure what additional work would be required to support V-Sekai.

Closing for now.