[SPARK-17796][SQL] Support wildcard character in filename for LOAD DATA LOCAL INPATH

What changes were proposed in this pull request?
Currently, Spark 2.0 raises a input path does not exist AnalysisException if the file name contains '*'. It is misleading since it occurs when there exists some matched files. It was a supported feature in Spark 1.6.2. This PR aims to support wildcard characters in filename for LOAD DATA LOCAL INPATH like Spark 1.6.2.
Reported Error Scenario
scala> sql("CREATE TABLE t(a string)")
res0: org.apache.spark.sql.DataFrame = []

scala> sql("LOAD DATA LOCAL INPATH '/tmp/x*' INTO TABLE t")
org.apache.spark.sql.AnalysisException: LOAD DATA input path does not exist: /tmp/x*;

How was this patch tested?
Pass the Jenkins test with a new test case.

Test build #66437 has started for PR 15376 at commit dfdfc46.

Test build #66437 has finished for PR 15376 at commit dfdfc46.

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

Merged build finished. Test FAILed.

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

The only test failure is irrelevant. Locally, it passed.
[info] - MAP append/extract *** FAILED *** (2 milliseconds)
[info]   java.lang.IllegalArgumentException:

Retest this please.

Test build #66452 has started for PR 15376 at commit dfdfc46.

Test build #66452 has finished for PR 15376 at commit dfdfc46.

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

Merged build finished. Test PASSed.

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

Test build #66618 has started for PR 15376 at commit f328f3a.

Test build #66618 has finished for PR 15376 at commit f328f3a.

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

Merged build finished. Test PASSed.

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

Test build #66842 has started for PR 15376 at commit 982bf3e.

Test build #66842 has finished for PR 15376 at commit 982bf3e.

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

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

Merged build finished. Test PASSed.

Hm, so this did used to work? I always thought inpath was a single file or directory. The Hive docs suggest this is the case, and we've tried to follow Hive I guess.
https://cwiki.apache.org/confluence/display/Hive/LanguageManual+DML

Oh. Indeed. I thought it's a supported way since it works on Hive 1.2.
hive> load data local inpath '/data/t/*.txt' INTO TABLE x;
Loading data to table default.x
Table default.x stats: [numFiles=12, totalSize=613224000]
OK
Time taken: 3.712 seconds

According to your advice and the URL, it seems not a normal or recommended way.
Thank you for @srowen . You prefer close this PR and the issue, do you?

Thank you for review, @srowen . I'll update the PR.
Also, I'll investigate more if there is a reason not to recommend this way.

Test build #67052 has started for PR 15376 at commit 401c4ee.

KafkaSourceSuite failure seems to be irrelevant.
[info] KafkaSourceSuite:
[info] - cannot stop Kafka stream (1 minute, 1 second)
[info] - subscribing topic by name from latest offsets *** FAILED *** (10 seconds, 511 milliseconds)
[info]   The code passed to eventually never returned normally. Attempted 669 times over 10.012014778000001 seconds. Last failure message: assertion failed: Partition [topic-2, 0] metadata not propagated after timeout. (KafkaTestUtils.scala:312)
[info]   org.scalatest.exceptions.TestFailedDueToTimeoutException:

Retest this please.

Test build #67064 has started for PR 15376 at commit 401c4ee.

Test build #67052 has finished for PR 15376 at commit 401c4ee.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #67064 has finished for PR 15376 at commit 401c4ee.

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

Merged build finished. Test PASSed.

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

Test build #67083 has started for PR 15376 at commit c74191b.

Test build #67085 has started for PR 15376 at commit 933ad85.

Test build #67085 has finished for PR 15376 at commit 933ad85.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Retest this please.

Test build #67083 has finished for PR 15376 at commit c74191b.

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

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

Merged build finished. Test PASSed.

Test build #67089 has started for PR 15376 at commit 933ad85.

Test build #67089 has finished for PR 15376 at commit 933ad85.

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

Merged build finished. Test PASSed.

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

Thank you for your review and approval, @srowen !

Merged to master

Thank you, @srowen !


⚠ Bofrost.be has degraded performance

In 183d4bc, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1108 ms

Resolved: Bofrost.be performance has improved in bab119a after 23 minutes.


Qwik docker file

Compiling qwik in development has more than usual deps:

Rust toolchain
Wasm
Several rust cargo bins
Specific node versions

A dockerfile will help our infra team at builder.io and part of the community familiar with rust.
Notice, that docker will NEVER be required. Its just another option to have a fully working environment.

Done!


Dependency Graph not finding dependencies on class libraries when using MacOS

Describe the bug
When using MacOS, the nx dependency graph does not show dependencies on class libraries from apps.
To Reproduce
Steps to reproduce the behavior:

Create workspace
Create console application (nx generate @nx-dotnet/core:app Demo.App)
Create class library (nx generate @nx-dotnet/core:lib Demo.Lib)
Using an IDE, create a class in Demo.Lib and consume it in Demo.App - ensure the IDE adds a project reference from Demo.App to Demo.Lib.
Look at the dependency graph (nx dep-graph)

Expected behavior
The dependency graph should show a dependency on Demo.Lib from Demo.App.
Environment:

OS: MacOS
DotNet SDK: 5.0.100

Additional context
After some investigation, this problem appears to be caused by the fact that the DotNet CLI writes project reference paths into the *.csproj files using backslashes, for example:
  <ItemGroup>
    <ProjectReference Include="..\..\libs\demo.lib\DemoLib.csproj" />
  </ItemGroup>

If you manually edit the *.csproj file and convert the backslashes to forward slashes, then the dependency graph works as expected:
  <ItemGroup>
    <ProjectReference Include="../../libs/demo.lib/DemoLib.csproj" />
  </ItemGroup>

Strangely, if you create a unit test project for the app when prompted by nx, then the dependency from the test project to the app works fine despite the use of backslashes - maybe this works because it doesn't have to go up a folder to the libs folder?:
  <ItemGroup>
    <ProjectReference Include="..\demo.app\DemoApp.csproj" />
  </ItemGroup>

It looks like the DotNet CLI creates project references with backslashes; I wonder whether there's an issue with how resolve is calculating the paths here: https://github.com/nx-dotnet/nx-dotnet/blob/master/packages/utils/src/lib/utility-functions/workspace.ts#L60

Hey @photomoose! Thanks for this issue, excited to see people trying this out on non-windows machines as I am sure that is where the plugin should shine.
I think I know what causes this, will get a fix out soon.

@photomoose Let me know if you are still having issues with this, I don't have a mac available for testing just yet.

@AgentEnder thanks for the quick fix, but unfortunately this didn't actually resolve the issue! I will take a look at it to see what's going on...

Ah, drat. I'll see if I cannot get to the bottom of this later tonight.

I was able to reproduce this via WSL2, so I should be able to publish a fix more confidently.

@AgentEnder I can confirm this now works! Cheers.

Haha! Excellent. Thanks for the confirmation. Anything else bumpy on non-windows machines? I really want to keep mac / linux first class.

Not yet, I haven't really used it in anger yet, I'm still trying to get a feel how this might work with a .NET project / mono-repo...
Our .NET applications run inside K8s, so our build artifacts are Docker images, where the Dockerfile copies the .NET source code into the build container and compiles it into a final image - (which can be achieved by bringing in the nx-docker plugin and adding a docker target to workspace.json) - your dependency fix now ensures the Docker images are built when changes occur.
Building / running tests locally is typically done inside the IDE (i.e. Rider) so we don't really need to invoke nx as part of our local dev workflow. However, on the CI server, we run the tests inside Docker containers, running as part of a Docker Compose network with other dependencies (e.g. Elasticsearch etc. for integration tests) - this allows us to have no dependencies on the build agents, everything is inside containers. I'm trying to think how we might be able to do something with nx here...
I'm also trying to workout whether we should have a single build/deployment pipeline for the mono-repo as a whole or individual pipelines for each service in the mono-repo...

Understood. I do not have a ton of experience with docker + kubernetes (really painful on windows), so it has not been something I have heavily considered with the plugin.
Local development inside of an IDE like rider or Visual Studio is also something that is not a core focus for the plugin, but is something that we will strive to maintain. If a project is opened up in an IDE, nothing the user does there should break the nx plugin, and everything done there should be accounted for by the plugin in some way (e.g. the sync generator pulls in dependencies added by an ide instead of through the plugin generators).
The build/test/serve executors really shine when using vs-code or any other editor that does not have first class support for .NET. These also really shine in CI for cases w/o kubernetes, as the nx affected ... commands play with the dependency graph.
The end goal of this plugin is to enable cross-language repositories where you can house your typescript front ends + .NET back ends in the same repo, and use the same commands to build and test them. Another part of this is the hopeful integration of typescript class generation from the .NET OpenAPI / swagger docs (#4 )


print docker logs when container fails to start

See #139

Going to close as it has been a while and I'm not sure if this is relevant anymore.


Support 0.55.4

Just copying 0_55_3-post.rb and it works at least for me.

Cool, sounds like a good start to me then

Thanks for merging!


Failed to import another Language_Code.xml for language-specified variables

Expected Behavior
Import another language-specified variables xml
Actual Behavior
I typed:
<import href="Language_Code.xml"/>
but it did not work.
Copy of the error message, log file or stack trace
I also tried:
<include href="Language_Code.xml"/>
or
<import src="Language_Code.xml"/>
or
<include src="Language_Code.xml"/>
however, none of them works. So I am wondering how can I import an existed Language_Code.xml to use the language-specified variables?
Environment

DITA-OT version:
dita-ot-3.7.0
Operating system and version:
Windows
How did you run DITA-OT?
dita command
Transformation type:
PDF2

@Billyshane - I'm not sure I understand what your situation is. Do you have a tiny testcase that illustrates the issue?

@Billyshane Where did you try to use <import href="Language_Code.xml"/>? In which file?

@Billyshane - I'm not sure I understand what your situation is. Do you have a tiny testcase that illustrates the issue?

For example, the variables declared in zh_CN.xml can also be used in zh.xml, so I just want to import zh_CN.xml in zh.xml to publish the map with @xml:lang="zh" in order to use the same variables.

@Billyshane Where did you try to use <import href="Language_Code.xml"/>? In which file?

As I described in the last reply, I tried to use <import href="zh_CN.xml"/> in zh.xml, is it illegal?

Have you read the docs? Adding new strings

Have you read the docs? Adding new strings

Thank you for your guidance, I have made it included.


HTTrack 3.01 support

[ ] different hts-ioinfo.txt format
[ ] split logs structure
[ ] older ndx cache format

Looks like new.txt wasn't generated too. Might have to construct it from a combination of the debug log and new.ndx. :/


Storage and Photo permission types use the same type

Storage and Photo permission types use the same type? Are they both required on Android?
https://github.com/yonahforst/react-native-permissions/blob/master/lib/permissions.android.js#L16
const permissionTypes = {
  location: PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION,
  camera: PermissionsAndroid.PERMISSIONS.CAMERA,
  microphone: PermissionsAndroid.PERMISSIONS.RECORD_AUDIO,
  contacts: PermissionsAndroid.PERMISSIONS.READ_CONTACTS,
  event: PermissionsAndroid.PERMISSIONS.READ_CALENDAR,
  storage: PermissionsAndroid.PERMISSIONS.WRITE_EXTERNAL_STORAGE, // - here
  photo: PermissionsAndroid.PERMISSIONS.WRITE_EXTERNAL_STORAGE, // - here
  callPhone: PermissionsAndroid.PERMISSIONS.CALL_PHONE,
  readSms: PermissionsAndroid.PERMISSIONS.READ_SMS,
  receiveSms: PermissionsAndroid.PERMISSIONS.RECEIVE_SMS,
}

@yonahforst any update on this?

Shouldn't photo be READ_EXTERNAL_STORAGE?

This is intentional. See https://github.com/react-native-community/react-native-permissions/pull/199


SMALLINT Datatype has flaws

ISSUE
MapD's SMALLINT datatype has several flaws upon applied boundary conditions.  To reproduce the issues, run the follow sql statements on the MapDQL cursor:
CREATE TABLE t2(id SMALLINT, val SMALLINT);
INSERT INTO T2 VALUES(1, 0);
INSERT INTO t2 VALUES(2, 32766);
INSERT INTO t2 VALUES(3, 32767);
INSERT INTO t2 VALUES(4, 32768);
INSERT INTO t2 VALUES(5, -32767);
INSERT INTO t2 VALUES(6, -32768);
INSERT INTO t2 VALUES(7, -32769);
INSERT INTO t2 VALUES(8,<PHONE_NUMBER>222222222222);
SELECT * FROM t2;

Results from the SELECT statement
mapdql> SELECT * FROM t2;
id|val
1|0
2|32766
3|32767
4|NULL
5|-32767
6|NULL
7|32767
8|-1

Comparisons of Expected and Actual Results
id: 1 and val: 0
- Expected Results: **0**  = Actual Results: **0**? -> PASS

id: 2 and val: 32766
- Expected Results: **32766**  == Actual Results: **32766?**? -> PASS

id: 3 and val: 32767
- Expected Results: **32767**  == Actual Results: **32767**? -> PASS

id: 4 and val: 32768:
- Expected Results: **Error Msg**  == Actual Results: **NULL**?-> FAIL

id: 5 and val: -32767:
- Expected Results: **-32767**  == Actual Results: **-32767**? -> PASS

id: 6 and val: -32768:
- Expected Results: **-32768**  == Actual Results: **NULL**? -> FAIL

id: 7 and val: -32769:
- Expected Results: **Error Msg**  == Actual Results: **32767**? -> FAIL

id: 8 and val:<PHONE_NUMBER>222222222222
- Expected Results: **Error Msg**  == Actual Results: **-1**? -> FAIL

Explanation
The SMALLINT datatype has 2 bytes in the -32768 to 32767 range.  The way MapD is a detection handling to outside boundaries is incorrect and doesn't throw error exceptions.  Also, Is the -1 result for the large number input part of an error?

Thanks for the detailed report! Yes, we're aware of this internally and we'll work on it very soon.

smallint.txt

@RalphLoen @andrewseidl please take a look into the issue.

@asuhan  Thanks for your reply.  This week is my 2nd week at MapD.

@DennisDawson Can you please add detail of this datatype to www.mapd.com/docs?


Audio looping twice when the game is paused during a marker transition

If the audio loop is going to send a signal that its completed and is stopped by the menu it will continue sending that and then 2 audio loops exist, one is uncontrollable and the other is the normal one. There is no way to stop the oncontrollable one outside from closing the game

fixed now, we dont use a phaser signal anymore to transition from phaser marker to phaser marker. a sound simply starts playing, once it reaches its end it will return to the loop point. so only a loop point needs to be defined.
however now a mode needs to call its sound.step method in update to watch after the looping point.


reason for using custom_mmpkg?

Hello I have a question about using custom_mmpkg.
In src folder, it using 'custom_mmpkg/mmcv'  why embedding this in src instead like 'pip install mmcv'?
because of embedded in src folder I have an issue of conflict mmcv version in another module file in custom_nodes

Can you post the full log? I think I screw up the sys.path somewhere in the code as I made "custom_mmpkg" just for avoiding conflicts lol.

Traceback (most recent call last):
File "/home/workspace/ComfyUI/nodes.py", line 1513, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/home/workspace/ComfyUI/custom_nodes/my_custom_node.py", line 101, in
from mmdet.apis import (inference_detector, init_detector)
File "/usr/local/lib/python3.10/site-packages/mmdet/init.py", line 17, in
and mmcv_version < digit_version(mmcv_maximum_version)),
AssertionError: MMCV==1.3.17 is used but incompatible. Please install mmcv>=2.0.0rc4, <2.1.0.
Cannot import /home/workspace/ComfyUI/custom_nodes/my_custom_node.py module for custom nodes: MMCV==1.3.17 is used but incompatible. Please install mmcv>=2.0.0rc4, <2.1.0.
Error log is like this. It's because my_custom_node.py has import mmcv but it's try to use  src.custom_mmpkg.mmcv
I resolve this issue just remove src/custom_mmpkg/mmcv :)

@bluelocate I pushed a mass rename from src.custom_mmpkg.mmcv to src.custom_mmpkg.custom_mmcv and src.custom_mmpkg.mmseg to ``src.custom_mmpkg.custom_mmseg`. Can you try it again?

@Fannovel16 Oh it works well :) Thank you for fixing


Fixes issue #752 - DefaultResourceManagerClassLoader isn't reading classes from cache

Good catch, thanks! :)


test: sharding for logs

looks like multiplexed integration test is failing on converage but isn't logged
/build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/_tmp/actions/stdout-19949) exceeds maximum size of --experimental_ui_max_stdouterr_bytes=1048576 bytes; skipping
I'm hoping sharding reduces log size, if not I'll look into tweaking flags.

/wait doesn't actually work yet :-P


Switch to jestjs.io

Summary
This PR switches a few remaining items (like images and links) over to the new jestjs.io domain, and fixes the deploy to not break the CNAME file

Ignoring that last red. Looks unrelated.


CA-254480: XenCenter error message refine for CA-205515

This is another part of pull request on xenserver/xenadmin.
https://github.com/xenserver/xenadmin/pull/1668
Both of them are for a new error message: POOL_AUTH_ENABLE_FAILED_INVALID_ACCOUNT

The SDK messages are autogenerated from the API and they don't need (shouil dnot need) to be added to the SDK, unless we want to override them for some reason (e.g. when they are a C# string format or they have a meaning that makes sense in a C# client, but not in xapi). Now, since xapi has already two different errors, why can't we correct the text there? It may be beneficial for CLI users too.
 error Api_errors.pool_auth_enable_failed_invalid_ou ["host";"message"]
    ~doc:"The pool failed to enable external authentication." ();
error Api_errors.pool_auth_enable_failed_invalid_account ["host";"message"]
    ~doc:"The pool failed to enable external authentication." ();

Hi @kc284
Thanks!
Your comment makes sense. At present, I get to know that I need to change the message in XAPI.
But for the xen-api-sdk, I'm still confused: do I still need to change the message in xen-api-sdk?
I'm asking because, actually, my changing on this message is because I saw another pull request: https://github.com/xapi-project/xen-api-sdk/pull/106
More questions are:

if the message I changed here is supposed to be auto-generated, why is it here as source code?
how will the message I change here be auto-generated? I mean how to sync the two projects, XAPI and xen-api-sdk. I don't know that exactly.

Thanks again!
Ming

The messages in the SDK will be autogenerated from the API when the project is built (instructions in the README). The code that does this is in csharp/friendly_error_names.ml.
The file csharp/FriendlyErrorNames.resx should contain only overrides of the API errors. For example, compare this API error:
error Api_errors.device_detach_rejected [ "type"; "ref"; "msg" ]
~doc:"The VM rejected the attempt to detach the device." ();

with the override:
<data name="DEVICE_DETACH_REJECTED" xml:space="preserve">
    <value>The VM rejected the attempt to detach the device {1} of type {0}. {2}</value>
</data>

This message cannot be corrected in xapi and we need an override because it is a C# string format. However, throughout the years the error list in here has come to contain many more messages from xapi, even those that do not need override (for example, DUPLICATE_MAC_SEED). It could be that PR#106 was not needed either. I suppose we have to refactor the errors one day.
To decide whether an error message needs override, you need to reproduce it with the CLI and the C# client (XenCenter) and if it makes sense for the user in both cases, then you leave it in xapi and you make any corrections in xapi. Now if it only makes sense in the C# client, then you need an override.

Got it. Great elaboration!
So I will cancel this PR, and raise another PR on XAPI.
@kc284 , thanks very much!
Ming


Added support for locale on DatePicker dialog component

Updated components, docs and spec.

This is not a PR to address #623, just a simple way to provide strings to DatePicker to show months and days in other languages (with some predefined languages loaded).

Beautiful work! I think this is very useful and does not create any breaking changes so happy to merge! Thanks 💃 !

This is great. When is this going to be on NPM?

I just realize that action buttons ('Cancel' and 'OK') need to be translated too. I was thinking on inserting it in locale prop too (pro: easy api for Date Picker component) but I think that inserting new properties like okButtonLabel, cancelButtonLabel, dismissButtonLabel... could make sense too (pro: standarization of action buttons). What do you think?

I think they should come from props as you mentioned. In fact, all internationalization in a library such as react-toolbox should be based on that. Anyway there are not a lot of components with text so it should be easy.

While the props option sounds more technically correct, I think that the
labels being attributes on the locale object is more useful, because the
default locale with the string locale prop is simple option that covers 95%
of the use cases. Besides, it would still allow customization through the
object locale prop.
On Aug 30, 2016 11:35, "Javi Velasco"<EMAIL_ADDRESS>wrote:

I think they should come from props as you mentioned. In fact, all
internationalization in a library such as react-toolbox should be based on
that. Anyway there are not a lot of components with text so it should be
easy.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/react-toolbox/react-toolbox/pull/680#issuecomment-243460102,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABfVlJEp1bdLRL6o3eHAAlnMQ_k0ipuIks5qlD-ogaJpZM4JZfTh
.

Yeah maybe. It also would be a cleaner api. More options on this would be great


Remove aws_ring_buffer_allocator_init declaration from public header

Make header not lie about the existence of aws_ring_buffer_allocator_init implementation.
I tried to call this API in my function but got an unresolved symbol.
 nm -gC crt/aws-c-common/libaws-c-common.a | grep aws_ring_buffer_
00000000000043be T aws_ring_buffer_check_atomic_ptr
00000000000043fe T aws_ring_buffer_is_empty
000000000000443f T aws_ring_buffer_is_valid
00000000000003d3 T aws_ring_buffer_acquire
000000000000095e T aws_ring_buffer_acquire_up_to
0000000000001363 T aws_ring_buffer_buf_belongs_to_pool
000000000000035c T aws_ring_buffer_clean_up
0000000000000216 T aws_ring_buffer_init
000000000000125b T aws_ring_buffer_release

And it's not there... looks like the implementation was recently removed but the declaration in the public header is still there.  This PR removes it.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Merged in https://github.com/awslabs/aws-c-common/pull/870


Multi-cluster capability for supporting 2 or more K8s clusters

This requires mostly testing and documentation for the multi-k8s registry code we have.
Use cases covered: https://goo.gl/AQARBs#heading=h.myl1ds62om2j

curious, why did you remove all the blocking sub issues ? conversion to epic ?
but shouldn't they all be in the epic ?
( like https://github.com/istio/istio/issues/626 )

Relevant additional use case
https://github.com/istio/issues/issues/236

Was this idea abandoned or completed?

Completed, see https://istio.io/docs/setup/kubernetes/multicluster-install/


[SPARK-19236][Core] Added createOrReplaceGlobalTempView method

What changes were proposed in this pull request?
Added the createOrReplaceGlobalTempView method for dataset

Can one of the admins verify this patch?

Hi @arman1371, is it still active?

@HyukjinKwon Yes

Merging to master. I will submit a separate PR for resolving the comment. Thanks!

@gatorsmile this didn't run any tests!!!

@gatorsmile, merging this commit ended up breaking tests because this commit fails Scalastyle checks and the blank line actually had trailing whitespace. I'm pushing a hotfix now to address this, but next time please run tests, even for trivial changes.

Sorry, I forgot it.


Add automatic opening of embedded player

Proposed Changes

Add a new option to enable/disable the automatic opening of the embedded player
Adjust the CSS class that is added when the embedded player is created

Expected Behaviour
The embedded player appears opened always.

Previous Behaviour
The player would start hidden and the user would need to click on a button to show it.

Thank you for your contribution. :+1:


pull out python 3.7 support

In May of 2020, we added python 3.7 support since several of the reinforcement learning algorithms we were using did not yet support anything above 3.7. Since then the libraries like RLlib has been updated to support later versions of python. I think it makes sense to drop this now, opening us up to use 3.8 and up specific features.

Nice, sounds good to me!

thanks to @erik-hasse for pointing out that 3.7 was a required test in the branch protection rules so just had to pull that out!


Adding sdf::Error logging to Param

🎉 New feature
Work towards https://github.com/ignitionrobotics/sdformat/issues/820.
Summary
Adding sdf::Errors &_errors as an argument option to any function that does any logging. The user can now choose to call functions with the sdf::Errors argument to retrieve the error messages or call them without it and get the same old behavior.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[x] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Codecov Report

Merging #939 (1dbc80b) into main (6e7336c) will decrease coverage by 0.32%.
The diff coverage is 70.83%.

@@            Coverage Diff             @@
##             main     #939      +/-   ##
==========================================
- Coverage   88.16%   87.83%   -0.33%
==========================================
  Files         100      102       +2
  Lines       14637    14867     +230
==========================================
+ Hits        12905    13059     +154
- Misses       1732     1808      +76

Impacted Files
Coverage Δ

src/Param.cc
84.12% <70.00%> (-7.18%)
:arrow_down:

include/sdf/Param.hh
83.75% <83.33%> (+5.17%)
:arrow_up:

python/src/sdf/pyError.cc
81.53% <0.00%> (ø)

python/src/sdf/_ignition_sdformat_pybind11.cc
100.00% <0.00%> (ø)

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 6e7336c...1dbc80b. Read the comment docs.


[Bug] Cannot run local tests without internet

leetgo debug
Leetgo version info  :
1.4.11
commit: 2656a5ce011a8874d2e04ae6cb4b02263499c595
built at: 2024-11-09T05:14:22Z
goos: darwin
goarch: arm64

Home dir             : /Users/amyr/.config/leetgo
Project root         : /Users/amyr/code/my-sols
Working dir          : /Users/amyr/code/my-sols/python/0001.two-sum
Project config file  : /Users/amyr/code/my-sols/leetgo.yaml
Project configuration:
# Leetgo configuration file, see more at https://github.com/j178/leetgo

# Your name
author: amyrd
# Language of the question description: 'zh' (Simplified Chinese) or 'en' (English).
language: en
code:
  # Language of code generated for questions: go, cpp, python, java...
  # (will be overridden by command line flag -l/--lang).
  lang: python
  # The default template to generate filename (without extension), e.g. {{.Id}}.{{.Slug}}
  # Available attributes: Id, Slug, Title, Difficulty, Lang, SlugIsMeaningful
  # (Most questions have descriptive slugs, but some consist of random characters. The SlugIsMeaningful boolean indicates whether a slug is meaningful.)
  # Available functions: lower, upper, trim, padWithZero, toUnderscore, group.
  filename_template: '{{ .Id | padWithZero 4 }}{{ if .SlugIsMeaningful }}.{{ .Slug }}{{ end }}'
  # Generate question description into a separate question.md file, otherwise it will be embed in the code file.
  separate_description_file: true
  # Default modifiers for all languages.
  modifiers:
    - name: removeUselessComments
  go:
    # Base directory to put generated questions, defaults to the language slug, e.g. go, python, cpp.
    out_dir: go
    # Functions that modify the generated code.
    modifiers:
      - name: removeUselessComments
      - name: changeReceiverName
      - name: addNamedReturn
      - name: addMod
  python3:
    # Base directory to put generated questions, defaults to the language slug, e.g. go, python, cpp.
    out_dir: python
    # Path to the python executable that creates the venv.
    executable: python3
  cpp:
    # Base directory to put generated questions, defaults to the language slug, e.g. go, python, cpp.
    out_dir: cpp
    # C++ compiler
    cxx: g++
    # C++ compiler flags (our Leetcode I/O library implementation requires C++17).
    cxxflags: -O2 -std=c++17
  rust:
    # Base directory to put generated questions, defaults to the language slug, e.g. go, python, cpp.
    out_dir: rust
  java:
    # Base directory to put generated questions, defaults to the language slug, e.g. go, python, cpp.
    out_dir: java
leetcode:
  # LeetCode site, https://leetcode.com or https://leetcode.cn
  site: https://leetcode.com
  # Credentials to access LeetCode.
  credentials:
    # How to provide credentials: browser, cookies, password or none.
    from: cookies    # Browsers to get cookies from: chrome, safari, edge or firefox. If empty, all browsers will be tried. Only used when 'from' is 'browser'.

    browsers: []
contest:
  # Base directory to put generated contest questions.
  out_dir: contest
  # Template to generate filename of the question.
  filename_template: '{{ .ContestShortSlug }}/{{ .Id }}{{ if .SlugIsMeaningful }}.{{ .Slug }}{{ end }}'
  # Open the contest page in browser after generating.
  open_in_browser: true
# Editor settings to open generated files.
editor:
  # Use a predefined editor: vim, vscode, goland
  # Set to 'none' to disable, set to 'custom' to provide your own command and args.
  use: custom
  # Custom command to open files.
  command: "nvim"
  # Arguments to your custom command.
  # String contains {{.CodeFile}}, {{.TestFile}}, {{.DescriptionFile}}, {{.TestCasesFile}} will be replaced with corresponding file path.
  # {{.Folder}} will be substituted with the output directory.
  # {{.Files}} will be substituted with the list of all file paths.
  args: ""

Full configuration   :
author: amyrd
language: en
code:
  lang: python
  filename_template: '{{ .Id | padWithZero 4 }}{{ if .SlugIsMeaningful }}.{{ .Slug }}{{ end }}'
  separate_description_file: true
  modifiers:
    - name: removeUselessComments
  go:
    out_dir: go
    modifiers:
      - name: removeUselessComments
      - name: changeReceiverName
      - name: addNamedReturn
      - name: addMod
  python3:
    out_dir: python
    executable: python3
  cpp:
    out_dir: cpp
    cxx: g++
    cxxflags: -O2 -std=c++17
  rust:
    out_dir: rust
  java:
    out_dir: java
leetcode:
  site: https://leetcode.com
  credentials:
    from: cookies
    browsers: []
contest:
  out_dir: contest
  filename_template: '{{ .ContestShortSlug }}/{{ .Id }}{{ if .SlugIsMeaningful }}.{{ .Slug }}{{ end }}'
  open_in_browser: true
editor:
  use: custom
  command: nvim
  args: ""

Debug log
No response
Description
I honestly don't know if this is a bug or not but I though "local" testing means I can run it even offline. But it only works if I have internet?

Thanks for the report, will take a look.

I reviewed the local test code path and, unfortunately, we can't make local testing work entirely offline. We might need to redefine "local testing" to mean running tests locally with an internet connection.


new devnet token request

Hi
hope your well :)
when requesting devnet tokens loads for ages then this error report see attached but no tokens

Indeed, Devnet has been having issues for a few days

faucet is working again

thank you hope your good :)


[Q] How to support other DL backend like paddle?

I found wandb is efficient and easy to use, and I want to add paddlepaddle to wandb like pytorch/tensorflow does, so how can I contribute that?

Hi @HydrogenSulfate thank you for contacting us and glad to hear you would be interested to contribute integrating wandb with other frameworks. We currently have an integration with PaddleOCR and PaddleDetection implemented in PaddlePaddle. Have you checked the WandbCallbacks in Paddle's repo, would this not work for your use case?
We also have a guide in our Docs page  on how to integrate wandb with other libraries. For contributing to our SDK, please also refer to our guide here. I hope this helps, but please let us know if you had any more questions.

Sure! Thanks @HydrogenSulfate for the update, and please feel free to reach out to us back here if you didn't find the information you were looking for.

I have implemented a easy use of wandb in PaddleScience in that PaddlePaddle/PaddleScience#422, and I am going to add it to docs in wandb like PaddleOCR...

Sure! Thanks @HydrogenSulfate for the update, and please feel free to reach out to us back here if you didn't find the information you were looking for.

I have implemented a easy usage of wandb in PaddleScience in https://github.com/PaddlePaddle/PaddleScience/pull/422 with reference to wandb's document, and I am going to add it to docs in wandb like PaddleOCR...

Great to hear that, thanks so much for the contribution! Please keep us posted here once it's merged.

Great to hear that, thanks so much for the contribution! Please keep us posted here once it's merged.

I have make wandb in PaddleScience：https://paddlescience-docs.readthedocs.io/zh/latest/zh/user_guide/#14-wandb
But i find a problem as descripted in the user guide, wandb seems use Step for default counter which will be added 1 when we call wandb.log, but not every one's code would prepare all stats for logging and call wandb.log onece per step, such as me. I wound call several wandb.log for not all my stats is prepared at the same time. So I note in the guide latter for change the x-axis from Step to step for correct the step number. Is there any good solution for this?

Hey @HydrogenSulfate you can use the commit=False inside your wandb.log calls.
Example:
for _ in range(10):
    # do something
    wandb.log({"fancy_metric": metric1}, commit=False)
    # do something more
    wandb.log({"fancy_metric1": metric2}, commit=False)
    # final do something
    wandb.log({"fancy_metric2": metric3}, commit=True)

commit=True is default so you can choose to not pass it explicitly.

Thanks for replying, I guess it might be not convenient to know when is comes to a final log call. Your code might be somewhat a simple case but a generic case for those git repo which encapsulate common functions into utils. It is hard to know when a log call is a final call except for adding an extra flag argument like commit: bool = False in my fnction.
def scaler(name: str, value: float, step: int, vdl_writer=None, wandb_writer=None):
    if vdl_writer is not None:
        vdl_writer.add_scalar(tag=name, step=step, value=value)
    if wandb_writer is not None:
        wandb_writer.log({"step": step, name: value})

It may still be the best solution with changing axis to step😓


Visual Studio Code installer failing

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

VS Code Version:
OS Version:

Steps to Reproduce:

Go to https://code.visualstudio.com/download
Click on any version's download link
Download times out waiting for response.
Alternatively, go to https://update.code.visualstudio.com/v1_85/win32-x64/stable response is "Cannot GET /v1_85/win32-x64/stable"
I have also tried getting it from the Microsoft store

https://update.code.visualstudio.com/1.85.1/win32-x64/stable


x/build: add LUCI openbsd-riscv64 builder

Following the instructions at Dashboard builders:
hostname openbsd-riscv64-mengzhuo
CSR is openbsd-riscv64-mengzhuo.csr.txt
since Github doesn't seem to allow attaching with the name openbsd-riscv64-mengzhuo.csr origin issue asked for.
Builder info(uname -a):
OpenBSD mars-1.lan 7.4 GENERIC.MP#427 riscv64
cc @4a6f656c

Updates:  openbsd-riscv64-mengzhuo is up and running.
https://chromium-swarm.appspot.com/bot?id=openbsd-riscv64-mengzhuo&s=true

Thanks. From the 5 builds it started during the time it ran, it's possible to see it correctly connected and identified its cipd_platform dimension as openbsd-riscv64, and as my comment above suggested, its CIPD dependencies are not yet in place. So resolving that that is the next step here.

@dmitshur OpenBSD has completed port, but LUCI builder failed to start, do I need to update something?
19057 2024-01-25 09:26:17.097 E: Request to https://chromiumswarm.appspot.com/swarming/api/v1/bot/handshake failed with
HTTP status code 403:
403 Client Error: Forbidden for url: https://chromium-swarm.appspot.com/swarming/api/v1/bot/handshake

I'm not sure why you'd be getting that error now and not before. But last time the builder connected, the error we needed to resolve was:
E: internal failure: Could not resolve version infra/tools/cipd/openbsd-riscv64:git_revision:ec494f363fdfd8cdd5926baad4508d562b7353d4: no such package: infra/tools/cipd/openbsd-riscv64

(Also see the last 2 paragraphs of https://github.com/golang/go/issues/64176#issuecomment-1834401616.)
The next step for that is to get the CIPD dependencies built for GOOS=openbsd GOARCH=riscv64. Right now the LUCI infrastructure is building those dependencies using Go 1.21, which doesn't support openbsd/riscv64 and so it can't work. We either need to wait for it to start using Go 1.23 where that port is added, or create a way for it to use a development version of Go 1.23 sooner, or find some other way without those dependencies being available in CIPD.

@mengzhuo @dmitshur What is the exact plan here? Is there a reason why we're bringing a new builder online, rather than continuing to use the openbsd-riscv64 builder that I already run?

I'm not sure why you'd be getting that error now and not before. But last time the builder connected, the error we needed to resolve was:
E: internal failure: Could not resolve version infra/tools/cipd/openbsd-riscv64:git_revision:ec494f363fdfd8cdd5926baad4508d562b7353d4: no such package: infra/tools/cipd/openbsd-riscv64

(Also see the last 2 paragraphs of #64176 (comment).)
The next step for that is to get the CIPD dependencies built for GOOS=openbsd GOARCH=riscv64. Right now the LUCI infrastructure is building those dependencies using Go 1.21, which doesn't support openbsd/riscv64 and so it can't work. We either need to wait for it to start using Go 1.23 where that port is added, or create a way for it to use a development version of Go 1.23 sooner, or find some other way without those dependencies being available in CIPD.

Unless I'm missing something, this seems like a pretty big hurdle for bringing builders/trybots online for new ports... how does one bring a builder up in order to have it running before merging a port?

@mengzhuo @dmitshur What is the exact plan here? Is there a reason why we're bringing a new builder online, rather than continuing to use the openbsd-riscv64 builder that I already run?

These is an issue (#63471) that requires all builders that should migrate from "go farm" to "LUCI". (https://go.dev/wiki/DashboardBuilders#luci-builders)
I can drop/transfer keys of openbsd-riscv64 LUCI builders if you want maintain it.

@mengzhuo Can you confirm that builds are processed properly if the builder is up?

@mengzhuo Friendly ping on this. Are you planning to continue to work on this builder, or should this be closed in favor of issue #67105?
By now the CIPD dependencies and a Go bootstrap are available for the openbsd/riscv64 port, so it should work if if's brought online again.

@dmitshur sorry for this long halt, I think I can't get enough visionfive2 to run openbsd-riscv64 builder now :(
Let's close it.

No problem, thanks for the update and for working on this. If the situation changes in the future, please feel free to file another new-builder issue. The openbsd-riscv64-mengzhuo-1701367184 certificate is revoked since it's not actively needed now.


⚠ DesiTorrents has degraded performance

In 21a95af, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 3224 ms

Resolved: DesiTorrents performance has improved in 1eb80fe after 7 minutes.


[BUG] Players get "User lacks permission to update ActiveEffect" error message when mooks take damage

Last week we resumed our GURPS game after a hiatus of several months. I updated the GURPS Game Aid to the current release. Now in combat the players get error messages when mook take damage that results in a status change (especially when going from no status to reeling). Here is what shows in the logs:
foundry.js:72076 User Kirsten lacks permission to update ActiveEffect [FmyDLKVaBndg5gr0] in parent ActorDelta [tOGvVkReScjsw36B]
Uncaught (in promise) Error: User Kirsten lacks permission to update ActiveEffect [FmyDLKVaBndg5gr0] in parent ActorDelta [tOGvVkReScjsw36B]
[No packages detected]
at file:///C:/Program%20Files/Foundry%20Virtual%20Tabletop/resources/app/dist/database/backend/server-backend.mjs:1:5101
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async Promise.all (index 0)
at async ServerDatabaseBackend._updateDocuments (file:///C:/Program%20Files/Foundry%20Virtual%20Tabletop/resources/app/dist/database/backend/server-backend.mjs:1:4748)
at async Semaphore._try (file:///C:/Program%20Files/Foundry%20Virtual%20Tabletop/resources/app/common/utils/semaphore.mjs:101:17)

I should add I also get the error with all modules disabled

With a little more experimentation, the players don't get this error if I make sure no tokens are selected by me (the GM) so we can close it

With a little more experimentation, the players don't get this error if I make sure no tokens are selected by me (the GM) so we can close it

That's still so weird.


Importing IP Addresses with DHCP status import as Active

Hello,
With the new status field added for IP Addresses in 1.7.0, I tested to import a range of DHCP addresses:
<IP_ADDRESS>/26,,,DHCP,,,,
<IP_ADDRESS>/26,,,DHCP,,,,
<IP_ADDRESS>/26,,,DHCP,,,,
<IP_ADDRESS>/26,,,DHCP,,,,
<IP_ADDRESS>/26,,,DHCP,,,,
<IP_ADDRESS>/26,,,DHCP,,,,
<IP_ADDRESS>/26,,,DHCP,,,,

It imports my addresses, but does not set their status to "DHCP", instead they are all marked as "Active".
Netbox 1.7.0
ArchLinux
Thanks!

Great, thanks a lot!

Thanks for filing the bug! Will be out in v1.7.1, probably early next week.


Added "Companion tools" section to the docs.

As suggested by @stubailo on #307

@lucasconstantino hey, welcome! I think this kind of description is OK for awesome-graphql, but on this page it would be worthwhile to have 2-3 sentences of what that library is for. I'd specifically mention that it allows combining multiple resolvers and specifying dependencies between them.

Ping! Would love to have this.

Added some code sample and extended the documentation for graphql-resolvers. Any opinions?

@lucasconstantino this is great, thank you!


Slow performance on initial login

On login to FAM, after credentials are entered and navigation to the FAM callback URL starts, there's a long delay before anything is displayed (just see a blank screen).
LIkely cause is that if there is an application selected by the user previously, we try to load the user role assignments before entering the router and the screen stays blank while that loading happens.
Suggest loading in background and not blocking rendering - even if it is just to display a loading screen.

Should be done in #1650


Add Base layout component

Is your feature request related to a problem?
@roiLeo suggested in #1339 many useful layout features.
Basically I copy this layout from a page to page and It would be nice to have a component:
<BaseLayout>
 <my-current-component-that-i-am-working-on />
 </BaseLayout>

Describe the solution you would like
Basically it should be:

    <slot ></slot>

Describe alternatives you have considered
:roc
Screeenshots
Paste your vision

Let's bring it to the nuxt version only


created an install guide for the VSIX file

amended from https://www.stweep.com/documentation/installation-guide-twincat-xae-shell/
I use TcXaeShell and can confirm this works

Hi @chrisbeardy! Thanks for your contribution! Can you add a link to the installation instructions on the README like:

... visual studio extension (TcBlackExtension (Install guide)).


Need to add example and description about adding entitlements

[Enter feedback here]
this doc does not have description about entitlements though we can add entitlement property for adding it to groups.
In example, it should have description like it can add entitlement as well.
{
"schemas": [ "urn:ietf:params:scim:schemas:core:2.0:Group" ],
"displayName": "",
"entitlements": [{"value":""}],
"members": [
{
"value":""
}
]
}

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

ID: ab8fc1cf-2f30-d6fb-36dc-331d646d502e
Version Independent ID: 5d277f96-3795-a8eb-923a-23bb2e6dd9df
Content: SCIM API (Groups) - Azure Databricks - Workspace
Content Source: databricks/dev-tools/api/latest/scim/scim-groups.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@yururin  Thanks for the feedback! I have assigned the issue to the content author to evaluate and add the above details.

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue and apologize for the delayed response. The requested updates have not yet been made. The timeline for resolution varies based on resourcing. We've created an internal work item (DOC-7016) to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close


No auth exception

Hi, after start all looks good
But after time I got error no_auth, check pls
2022-04-11 11:03:09.063 | INFO     | lokbot.farmer:socf_thread:604 - a loop is finished
2022-04-11 11:47:48.145 | DEBUG    | lokbot.client:post:73 - {"url": "quest/list", "data": {}, "elapsed": 0.691461, "res": {"result": false, "err": {"code": "no_auth"}}}
upd. When game is open in browser and window active I dont get error.
Can I close a browser and still successfully using a bot?
Tnx

you need to have the browser open to keep the session active


Demo page is dead.

whenever you open an issue, you should write how to reproduce it.

rocket science has not been mentioned above.
click demo link in readme.md

The Demo page is not available again. Could you please fix this?

fixed


Update animation simulation to fix restart ballistic animation

Same as  #96512
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@Piinks
I tried a new method to solve the problem.
I don't know if this is the right plan, Maybe you could take a look. If you think this solution is feasible, I will complete the documentation and fix the test.
Here's what I think:
In the previous method:

I need to update the BouncingSimulator/ClampingSimulator properties.
I need to call a specific method in ClampingSimulator/BouncingSimulator.
I have to use type checking to get me to call this method.

In the new method:

My purpose is to not restart the sliding animation
Maybe we can use the new Simulator, just make sure it is generated with the initial position and initial velocity.
Then we update the new Simulator to the animation.

@Piinks
Yes, Everything is as expected. I record a video and add some test.
I record clamping performance only. Bouncing performs consistently before and after modification(validate the expected)

Before
After

We could see sliding damping is not as great as before. The animation of clamping is same as without size change.
And after a scroll slows down to a sufficient amount, items appear to "snap" into place. So maybe we should try to reland https://github.com/flutter/flutter/pull/77497 after this PR merge

Same as https://github.com/flutter/flutter/pull/96512

Can you please update the PR description with an actual description of what this is fixing? I tried to follow the chain of links, but it is not clear to me. Is this fixing a perf regression? A fidelity problem? A bit of both?

@MxSoul Do you still have plans to come back to this PR?

@MxSoul I saw that you responded to some of the comments, did you maybe forget to push your updated code to this PR?

Since there hasn't been a response, I am going to close this one for now. If you find the time to work on this again, feel free to reopen this with the feedback from above addressed.


MeasureReporting Refactor

We need to package the measure reporting module for mADX before the closing of the project that is Feb 28th 2022. We have a hard deadline for this. @ellykits had made some refactoring to the reporting module and we can use this to complete the remaining work to do measure reporting based on the $evalute_measure API via PR https://github.com/google/android-fhir/pull/1092 to be merged later today.
This will also be reused for the e-CBIS pieces on demonstrating measure reporting in-app before loading  the required libs for measure reporting for the eCBIS use case which can start off by testing Immunization indicator for BCG
Tasks

Clean up PR #965 , resolve the TODOs and merge it into the main branch
Updated the Gradle file to use the merged PR for measure reporting and not the published Sonatype version

api('org.smartregister:workflow:0.1.0-alpha02-preview-SNAPSHOT') {
transitive = true
exclude group: 'xerces'
exclude group: 'com.github.java-json-tools'
exclude group: 'org.codehaus.woodstox'
exclude group: 'ca.uhn.hapi.fhir'
exclude group: 'org.opencds.cqf.cql'
exclude group: 'com.google.android.fhir', module: 'common'
//        exclude group: 'com.google.android.fhir', module: 'engine'
exclude group: 'org.smartregister', module: 'engine'
}

Add measure libs into assets folder before shipping and packaging mADX work  -
Libs are ~54MB for 13 indicators and should not be pushed to the repo, this should be loaded from HAPI. Each ANC indicator can be downloaded and loaded to the assets folder via https://github.com/WorldHealthOrganization/smart-anc/tree/master/bundles/measure . For each indicator library folder, check for the file  named ANCIND01-bundle.json
Test each indicator measure reporting for ANC Indicator 1-13

Later

Refactor all the work into a reusable reporting module
Sync measure and Libs from HAPI, though for specific use cases it would be advisable to load from assets folder. To be Discussed (TBD)

CC @dubdabasoduba

closed by https://github.com/opensrp/fhircore/pull/1105


Add load commend to CC instruction

Need to load java module in the new environment so as to install pygaggle.

Can I get this PR reviewed?

@ronakice ?

LGTM! Sorry I missed this


PHP Functions Framework

PHP Functions Framework
We should release the Functions Framework for PHP. It will use NGINX.

… but GCP does not sport PHP in Cloud Functions?! 👀

This was fixed with the release of this repo:
https://github.com/GoogleCloudPlatform/functions-framework-php


Typo "** Applies to: **"→"Applies to:"

https://docs.microsoft.com/en-us/skype-sdk/ucwa/callforwardingstate_ref

Docs Build status updates of commit 10350d4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Skype/UCWA/CallForwardingState_ref.md
:warning:Warning

Details

Skype/UCWA/CallForwardingState_ref.md

[Warning] Missing attribute: title. Add a title string to show in search engine results.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


identify calls from cloud users

we're currently logging a True/False variable to identity telemetry events from cloud users. However,  we have no way to identify them, making providing support much harder.
we should log another ID that allows us to uniquely identify them. one option is their user ID or email

note: we're currently logging an email field but it's always none. in the past, we implemented logic to optionally enter a user's email but we removed it.


TimePicker Component

Bug, feature request, or proposal:
Feature request
Am I the only person who thinks that its missing a TimePicker component?

I don't think you're the only person and you're certainly not the first one thinking about this. Please use the search function before issuing duplicate requests. It's being tracked in #5648


OricToken

Blockchain Platform with
SELF-STABILIZING PROPERTY GOVERNANCE
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

UpLoad OricToken


T5 for DST (evaluation results)

I fine-tuned T5 for DST. I got the (predict metrics) after evaluation. Then, I used "merge_predict_res" and  "evaluate_unified_datasets", I got different results.. why?

evaluate_unified_datasets results:

The first result is calculated by dst_metric.py and the second one is by dst/evaluate_unified_datasets.py. The latter assumes the golden state contains all slot-value pairs of involved domains, and ignores the prediction out of this scope.
For example, if a model predicts Hotel-Ticket-Free which is not a valid domain-slot, the latter will ignore it while the former will regard it as an error prediction. In other words, the latter use the golden state as a filter to ignore invalid domain-slot predictions.

Thank you for the clarification.

Regarding the returned state by dst.update() method. The 'system_action': [], 'user_action': [] when/where they need to be filled?

If the DST model is used to build a system agent, the agent should pass the user_action (from NLU) and system_action (from Policy). You can look into the policy model (eg. DDPT) who needs this feature as input

The prediction accuracy is the Joint goal accuracy (JGA), right ?

Yes


Switches from SI to binary prefix.

Summary
Switches from the SI prefix MB to the binary prefix MiB in cases where the underlying value represents 1024^2 bytes.
Ticket Link
n/a
Checklist
n/a
Test Information
n/a

I understand the difference between the two but the result is normally presented to the end user who has no idea and it might be confusing for them, so my question here is why do we want to make this change?

@enahum Disambiguation.

Can we run this by a PM?

Of course.

What is the end user impact? I don't really understand it. Is it that file size is now shown as "50MiB" instead of "50MB"?

@jasonblais Yes MB or megabyte refers to 1,000,000 (mega) bytes whereas MiB mebibyte is the ISO standard for referring to 1,048,576 bytes.

Exactly but my guess is that end users do not really know the difference between the two.
I’m 1/5 on this but is not a bug deal to change it, but perhaps we should change it everywhere in help text and stuff

@enahum I have PRs for the server and the webapp. I can also make the changes to all the docs.

Cool will leave PM to make the decision on this ;)

I'm not sure, I haven't really seeing MiB used anywhere but also don't know enough to make a decision here.
I've deferred to @esethna since it's an end user related change

Agree with JB and EN here. While it may technically be more accurate, I've never seen "MiB" facing the end user in other software and I don't think the difference between MiB and MB is not commonly known. I would prefer that we stick what's standard and well known.


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #7.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #7

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/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.

For the host section of the recipe, you should usually use python {{ python_min }} for the python entry.
For the run section of the recipe, you should usually use python >={{ python_min }} for the python entry.
For the test.requires section of the recipe, you should usually use python {{ python_min }} for the python entry.
If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: passed

Thus the PR was passing and merged! Have a great day!


Gracefully handle zarf shorthand removal

This re-adds some behavior to tell users the zarf zarf-package-thing-arch.tar.zst is no longer valid.

Current experience on master (not yet released)

New behavior


Explore how to maintain several versions of the documentation accesible from the website

https://github.com/dgraph-io/dgraph/tree/master/wiki

https://github.com/dgraph-io/dgraph/tree/master/wiki


How project is graded

I find quite many students confusing about how your project.py file is utilized in grading. To make it clearer, I separate the project.py file and a test.py file. DO NOT change the test.py file.
The principle is:
You can do anything in your project.py file as long as:

on a machine that only installed scikit-learn, pandas, and gensim,
when you execute the test.py file, it correctly outputs the f1 score.

Do we need to use assignment8? Can't we use sklearn.metrics.f1_score?

We have already used the earlier project.py. Now the recent commits on the project in github repo suggests some changes which would be difficult to incorporate as the we have been using the earlier files for the past month.
Please address this concern.

@neha6351 you can use any evaluation. However, when your file is getting graded, I will use the reference answer version of my_evaluation to calculate your f1 score.

@Kruita-05 If you find your project.py file not compatible with the test.py file, that means your initial understanding is incorrect. The requirement of the project has not changed since the beginning.

Again, there is no change to the project. If you correctly implemented the project.py file, you should find it no difference when running with test.py file.


Configurations for running on GPU?

I'm actually trying to run the process on single GPU with 32GB gives me out of memory issue,
If I'm using multiple GPUs, how do I execute the training process? All the jobs in training are inter-related right? Can I assign a different GPU for a different process using CUDA_VISIBLE_DEVICES?
Can I directly use the code for CPU version, or do I have to make changes to execute this on the GPU environment.
Any help is highly appreciated,
@esonghori please let me know.
Thanks,
Mohan.

This is a very late response. I have updated and created and e2e example that runs training on GPU and everything else on CPU.  https://github.com/google-research/circuit_training/tree/main/tools#end-to-end-smoke-test

This is a very late response. I have updated and created and e2e example that runs training on GPU and everything else on CPU. https://github.com/google-research/circuit_training/tree/main/tools#end-to-end-smoke-test

The e2e example for GPU does not work due to the branch version. The ubuuntu_circuit_training script is not updated in r.0.0.3 making not even be ran. From my understanding GPU is only supported on the devel/HEAD branch but then the code doesn't really work for that
I am just trying to get the e2e_smoke test to run using the GPU, perhaps the code requires some tweaks or clarification in that regard.


[WIP] allow env var substibution for Pinot configs

Testing some ideas to make env var substitution work for Pinot configs

Codecov Report

Merging #10785 (8bd2bf3) into master (efb4f46) will decrease coverage by 56.64%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10785       +/-   ##
=============================================
- Coverage     70.32%   13.68%   -56.64%
+ Complexity     5658      439     -5219
=============================================
  Files          2159     2104       -55
  Lines        116028   113572     -2456
  Branches      17563    17278      -285
=============================================
- Hits          81593    15541    -66052
- Misses        28738    96757    +68019
+ Partials       5697     1274     -4423

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
13.68% <0.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...pache/pinot/spi/env/CommonsConfigurationUtils.java
0.00% <0.00%> (-65.39%)
:arrow_down:

...a/org/apache/pinot/spi/env/PinotConfiguration.java
0.00% <0.00%> (-91.77%)
:arrow_down:

... and 1710 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


litclient: add taprpc packages to Registrations

Add the priceoraclerpc, rfqrpc, and the tapchannelrpc JSON callbacks to the litclient's Registrations array. This allows the litclient to use the rpc functions contained in these JSON callbacks.
These registrations are required for apps using the LNC (wasm) to call RPC methods. Without these additional callbacks, the wasm will return a "RPC method not found" error when calling RPCs contained in these packages.

Rebased on the latest 0-19-staging branch (f7d6d72).


Uncontrolled Checkbox Component

Please add an ability to use uncontrolled checkboxes, so that a checkbox can be used with libraries which require it to not have checked prop. For example, 'react-hook-form' with basic register solution.
Current behavior: checkbox checked prop is always set due to the fallback of defaultValue ?? false.

reproduced here https://codesandbox.io/p/sandbox/kind-thunder-q4ntyf?file=%2Fsrc%2FFakeForm.tsx%3A84%2C7

Current behavior: checkbox checked prop is always set due to the fallback of defaultValue ?? false

That's incorrect. defaultValue ?? false is set for the hook's value and checked input's attribute is only set when checked prop is set. See source code here here

I digged the problem. There is no change event on inputs when form is reset. So our styling (custom tick svg) renders stale state. The only way to fix this I see is to not relay on our state but rather on browser's internal state (:checked, :indeterminate CSS selectors) to do our styling.


Upgrade AWS SDK

Upgrade AWS SDK v2 to the latest version (2.20.29 at the time of creating this issue).

Upgrade AWS SDK v1 to the latest version (1.12.432 at the time of creating this issue).

Dependency versions are defined in https://github.com/awspring/spring-cloud-aws/blob/main/spring-cloud-aws-dependencies/pom.xml
The way to implement this change is to bump these versions, and run the build with tests to see if everything is working as expected.

Can I start here?

Sure!

Hi @maciejwalkowiak
After upgrading the AWS SDK's, there is issue in the S3CrtAsyncClientAutoConfigurationTests. The private helper method s3NativeClientConfiguration() which obtained the S3NativeClientConfiguration  from the CopyObjectHelper class is failing with below error.
java.lang.IllegalArgumentException: Could not find field 's3NativeClientConfiguration' on target object [software.amazon.awssdk.services.s3.internal.crt.CopyObjectHelper@4cb40e3b] or target class [class software.amazon.awssdk.services.s3.internal.crt.CopyObjectHelper]

	at org.springframework.test.util.ReflectionTestUtils.getField(ReflectionTestUtils.java:275)
	at org.springframework.test.util.ReflectionTestUtils.getField(ReflectionTestUtils.java:217)
	at io.awspring.cloud.autoconfigure.s3.S3CrtAsyncClientAutoConfigurationTests.s3NativeClientConfiguration(S3CrtAsyncClientAutoConfigurationTests.java:97)
	at io.awspring.cloud.autoconfigure.s3.S3CrtAsyncClientAutoConfigurationTests.lambda$setsPropertiesOnClient$0(S3CrtAsyncClientAutoConfigurationTests.java:87)

But in aws-sdk2 version 2.20.9 the CopyObjectHelper no longer have the instance of S3NativeClientConfiguration.  Below is difference between those two classes.
- public CopyObjectHelper(S3AsyncClient s3AsyncClient, S3NativeClientConfiguration s3NativeClientConfiguration)
+ public CopyObjectHelper(S3AsyncClient s3AsyncClient, long partSizeInBytes)

So we will not be able to get the native configuration and assert the properties which are set after auto configuration.
Can you give some pointer here how we tackle this?
References :
[AWS SDK S3 2.20.9 JavaDoc] (https://www.javadoc.io/doc/software.amazon.awssdk/s3/latest/index.html)
[AWS SDK S3 2.19.25 JavaDoc] (https://www.javadoc.io/doc/software.amazon.awssdk/s3/2.19.25/index.html)
Also Can I raise the draft pull request for this issue?

Not really, i guess it is somewhere else accessible with reflection but i don't know where.

I think AWS SDK v2 (2.20.29) might be unstable. I see the bom has not been updated to the correct level of aws-crt which should have been bumped to 0.21.9 by AWS. I see that 2.20.31 is underway.

@maciejwalkowiak
Yes indeed it was accessible on the SdkAsyncHttpClient on the delegate of the S3AsyncClinet. I have fixed the test. Also upgraded the AWS SDK V2 to 2.20.30 which was release latest version.
@arunkpatra You were saying that AWS SDK V2 (2.20.29) is not stable, where did you refere that from?

@kunalvarpe I was building it today with 2.20.29 and was getting errors while running tests. It appeared that AWS did not update the aws-crt version to latest in the bom. I had to override the version of aws-crt to latest to make things work.


ciscoreputation 2.1.4 - error JSONDecodeError

ciscoreputation reputation IP --tos
ciscoreputation 2.1.4
Traceback (most recent call last):
File "/usr/local/bin/ciscoreputation", line 11, in
load_entry_point('ciscoreputation==2.1.4', 'console_scripts', 'ciscoreputation')()
File "/usr/local/lib/python2.7/dist-packages/ciscoreputation-2.1.4-py2.7.egg/ciscoreputation/init.py", line 4, in main
do_main()
File "/usr/local/lib/python2.7/dist-packages/ciscoreputation-2.1.4-py2.7.egg/ciscoreputation/ciscoreputation.py", line 120, in do_main
data = get_data(arguments[''])
File "/usr/local/lib/python2.7/dist-packages/ciscoreputation-2.1.4-py2.7.egg/ciscoreputation/ciscoreputation.py", line 46, in get_data
'query_entry':search_string
File "/usr/lib/python2.7/dist-packages/requests/models.py", line 897, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python2.7/dist-packages/simplejson/init.py", line 518, in loads
return _default_decoder.decode(s)
File "/usr/lib/python2.7/dist-packages/simplejson/decoder.py", line 370, in decode
obj, end = self.raw_decode(s)
File "/usr/lib/python2.7/dist-packages/simplejson/decoder.py", line 400, in raw_decode
return self.scan_once(s, idx=_w(s, idx).end())
simplejson.errors.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I'm having this issue too with version 2.1.4

This may be because Cisco now asks for a Captcha for non-logged in accounts. You can dump the HTML by doing print(r_details.text) and load this into a browser to see more-or-less what is going on. It is free to create a Cisco account, perhaps there is some way for account holders to bypass the Captcha (with an API key or some such mechanism)?
I would attach an example, but I can't get GitHub to accept the file (even as a .txt, .zip or .gz)


Implemented Data storage for Books and Labels

In this PR we:

Implemented JSON storage for Books
Implemented JSON storage for Labels

STATUS: APPROVED :green_circle: :clap:
Hi @andrianarivo 🙋♂,

Your project is complete!  There is nothing else to say other than... merge it!! Great job 👏
Congratulations! 🎉
TO HIGHLIGHT:

Impeccable sense of Ruby
No offenses in Rubocop

Now you can merge this branch and move on :1st_place_medal:
Cheers and Happy coding!👏👏👏


[CRITICAL] [Advanced Approval Workflows] Adjust  for Approver Selection

Coming from: #wave-control: advanced approval + workflows polish

Hey! I’m Błażej Kustra from Software Mansion, an expert agency, and I’d like to work on this issue!

Monthly Update

I'm checking to see if this issue can be closed. I see the PR had a C+ review on it, but there is no record in this issue of the C+ getting paid.

Next Steps

@tgolen If the C+ needs to be paid, assign this issue to someone from BZ to pay them out. If they don't need to be paid, this issue can be closed.

ETA

Closed by Monday, Sep 3

Monthly Update

I see the C+ was paid https://github.com/Expensify/App/pull/46510#issuecomment-2321953555 so I will close out this issue.


Cleaned up CMakeLists.txt, used OpenCV C++ API for cpp file

This is a clean commit I created out of my port from Opencv3 to OpenCV 4. There are still a few kinks left for a complete port, but I thought I should upstream this portion at least since it still works for opencv3 and changes are unrelated to opencv4

Thanks for the commit! Looks good to me and the CMakeLists is much cleaner now.
Marko


After nan in BO objective all agents fail with: "TypeError: '>' not supported between instances of 'float' and 'str'

wandb, version 0.8.31
Python 3.8.2
Linux
Description
When using method: bayes after some time all my agents stop with a repeated error message of wandb: ERROR  (<Response [500]>) wandb: ERROR Error while calling W&B API: anaconda 400 error: {"code":400,"message":"TypeError: '>' not supported between instances of 'float' and 'str'" }.
After further investigation I am fairly confident that this has to do with one of the runs having a nan as the metric: name: valid_loss value. Why do I believe this?

I was watching a run, and as soon as the reported metric for valid_loss which is my metric has a nan as a final result in one of my runs, then all consecutive agents fail.
A random sweep with the exact same code and wand config (except random instead of bayes) does not have these issues, despite some runs having nan as the result.
When I change my code to log as a final value a large number instead of nan I don't observe this issue.

This should be fixed as of 2 days ago. Please re-open if you see this error agian.


make error

Hi Benedict,
I got an error in compilation process.
make[1]: Entering directory `/gpfs2/projects/gec/tool/assemblathon1/cactus/api'
cp inc/cactus*.h ../lib//
gcc -std=c99 -O2 -g -Wall -Werror --pedantic -funroll-loops -lm  -I ..//../sonLib/lib -I/projects/gec/tool/assemblathon1/local/include -DHAVE_TOKYO_CABINET=1 -I/projects/gec/tool/assemblathon1/local/include -DHAVE_KYOTO_TYCOON=1    -I inc -I ../lib// -c impl/cactus*.c
cc1: warnings being treated as errors
impl/cactusDisk.c:369: error: â€˜getNextStubâ€™ defined but not used
make[1]: *** [../lib//cactusLib.a] Error 1
make[1]: Leaving directory `/gpfs2/projects/gec/tool/assemblathon1/cactus/api'
make: *** [all.api] Error 2

The error may be removed by the definition of getNextStub.
Thank you.
Yun

Fixed on the development branch. Will do a merge with master shortly with some other changes, so consider closed.


Open url in natified app when single instance is enabled and app is minimized to tray

Homework

[x] I looked at nativefier --help and https://github.com/nativefier/nativefier/blob/master/API.md
[x] I searched existing issues, open & closed. Yes, my question is new.
[x] I'm using the latest version available at https://github.com/nativefier/nativefier/releases

Your question
I have natified the google search page (https://www.google.com)
I want to use autohotkey to send selected text as a search query to the natified google search app.
The autohotkey command :
Run, "C:\Windows\System32\Google-win32-x64\Google.exe", https://www.google.com/search?q=%Clipboard%

works with normal browsers but doesn't seem to work with the natified app.
The app opens but just shows the default google home page after that....
Any way to make this possible?
Thanks
Context

Nativefier: 47.1.3
Node.js: 17.9.0
Npm: 8.8.0
OS: windows 11 build 22000.652

Closing as this issue has been solved in the latest version.

Currently you can only send urls to the app if single instance is not enabled.
This is because all the new url's are opened in a new window.
It would be nice if the urls passed on to the app could be opened in the original app window.
Thanks

Currently you can only send external urls to the app if single instance is not enabled. This is because all the new url's are opened in a new window. It would be nice if the urls passed on to the app could be opened in the original app window even when single instance is enabled.

Ping to the contributor who implemented #1418 that shipped in 47.2.0: @spywhere .


Is dashboard also open source?

The dashboard isn't open source for the moment but maybe in the future.


Add --abbrev-ref to get the right branch name of HEAD

Add --abbrev-ref to get the right branch name of HEAD rather than checksum to return back to original branch.
Without it will make merge tool to go to unnamed branch.
Somehow old PR #548 could not be reopen so submit new one. Sorry

Will push this EOD if no objection.

LGTM. Robert also gave his OK in the old PR.
@hsaputra: Can you squash both commits and add a tag like [tools] before merging?

Thanks @uce and @rmetzger. Will squash both commits and add tag before commit.


Type error thrown when schema uses oneOf

schema:

    oneOf:

         - type: array

         - type: string

Will throw
Exception occurred:   File "/Users/maxmurmann/opt/anaconda3/envs/bullhorn-apis/lib/python3.8/site-packages/sphinxcontrib/openapi/openapi30.py", line 282, in _httpresource     type=param['schema']['type'], KeyError: 'type'
whereas I believe this should be a fine formatting for OpenAPI?

This is certainly valid OpenAPI.

This same issue occurs with anyOf, e.g.:
"schema": {
    "anyOf": [
        {
            "type": "string",
            "format": "date"
        },
        {
            "type": "null"
        }
    ],
}

which should also be valid OpenAPI.
Sphinx : 6.2.1
sphinxcontrib-openapi: 0.8.1

Can someone provide a minimal example of a full schema with this used that can be tested against?

This might be related to https://github.com/sphinx-contrib/openapi/issues/96

This issue seems to be related as well to #53, #54, and #113. The two latter also proposed fixing changes, though that was years ago, so I don't know if they can be copy-pasted. I'll definitely try the workaround mentioned in #96, but feel free to check out our current build failure on the PR linked to above where I mentioned this issue and let me know how I could help with a fix.
The issue does seem to lie with entries like {"name": "table", "in": "query", "required": false, "schema": {"anyOf": [{"type": "boolean"}, {"type": "null"}], "default": false, "title": "Table"}} in our openapi-v1.json. They are created automatically, in this case from line 28 of ixmp4/server/rest/iamc/datapoint.py, which type-hints table as bool | None.

If the contents of the log file are of any interest: sphinx-err-dzck2sih.log

With the workaround in #96, the docs "build successfully", but I still see
WARNING: unknown directive or role name: openapi:httpdomain
/home/fridolin/ixmp4/doc/source/devs/ixmp4.server/endpoints.rst:4: ERROR: Error in "openapi:httpdomain" directive:
unknown option: "examples".

.. openapi:httpdomain:: /openapi-v1.json
    :examples:
    :group:

The error can be fixed by removing :examples: and :groups: and using the supported options detailed in issue/96, but this doesn't solve the warning that the directive openapi:httpdomain seems to be unknown.

Resolved via https://github.com/sphinx-contrib/openapi/pull/143


compile svgren with static runtime

Hi there!
I try to compile svgren.lib (or dll) for Win32 with statically linked runtime (/MT flag instead of /MD ).
VSVS 2015 give me a bunch of linking error (see bellow)
Steps to reproduce:

Update to the latest svgren (v. 0.4.43)
Open msvs_solution.sln
Choose v140_Release configuration and x86 platform
Right click on libsvgren -> Properties -> C/C++-> Code Generation -> Runtime Library-> Multi-threaded (/MT)
Rebuild

There are lot of complains about unresolved externals symbols and mismatch detected:
build.log.txt
Properties -> Reference Packages shows that we have choice between static and dynamic for cairo and pixman libs.
Further investigation shows that in despite of these settings all libs are 'multi-threaded dll' linked (they all contain -MD flag inside, instead of -MT).
Earlier I saw in some previous versions of svgren, pugixml package had two selectable options (static / dynamic)
It would be nice if we could choose what runtime to use too.
(I personally prefer static linking)
Many thanks for your work!

I have no idea what DLL/static in Properties -> Reference Packages selection does...
It looks like I just need to add one more configuration to nuget packaging.

I'm not very experienced with windoes RT lib...
Do you know what are the advantages of haveing RT dynamically linked?
Maybe we could have only statically linked RT option?

One of the advantages of dynamic linking I can guess is smaller size of executable.
And it doesn't matter at all in our case.
Moreover, disadvantages are more significant (DLL hell for example).
Having two versions of the linking would be ideal case but for simplicity let's keep one of them.
And as I mentioned above my personal choice is static linking.
Properties -> Reference Packages lets us select desired configuration if more than one exist in the package. See pugixml nuget package for example, it contains both static and dynamic linking libs.
And this options allows us to select desired one.

I will switch all my projects to statically linked RT lib.

I found this article http://www.davidlenihan.com/2008/01/choosing_the_correct_cc_runtim.html
and it actually has a good point about how to select which runtime to use.
Looks like it is better to use dynamic runtime to avoid mixing runtime libs. Most of the libraries used dynamic runtime, including opengl32.dll, so switching to static runtime is not a good idea,
That means that this issue is to be dropped.
@Papirosnik do you have any other reasoning why we need static runtime linkage?

Closing this as there is not enough need in supporting static runtime for now.


Unity Builder Serial Activation Limit Hit

Bug description
It seems a non-free license is being used to activate Unity in the tests which is causing the activation limit to be hit due to the number of parallel runs.
How to reproduce

Run Linux, mac, and windows builds simultaneously.

Expected behavior
No activation errors.
Additional details
Can the repo switch to a free license so these limits aren't hit? Otherwise will need a way to ensure the parallel run count is kept lower.
CC: @davidmfinol @GabLeRoux

We'll create a new free account and use that. Planning to use<EMAIL_ADDRESS>as the email address and just waiting on some setup for it.

Worker for forwarding email into our maintainers discord channel is completed.
Unfortunately I'm struggling to get it deployed due to a bug in Cloudflare.
In the meantime I've forwarded emails to me and I'll paste them into the email channel manually until that's resolved.

I've set<EMAIL_ADDRESS>along with the corresponding UNITY_PASSWORD for the free Unity account.
I've also updated the UNITY_LICENSE.
However, I'm not sure how to set the UNITY_SERIAL.
I can only find the Serial key at https://id.unity.com/en/subscriptions if I pay for a PLUS or PRO license, but we see this issue if we use a paid account.
After looking through our discord logs, I'm pretty sure I had set UNITY_SERIAL before, but I'm still not sure about how did we have this working before?

If you look in the ulf, unbase64 developer data

Re: Worker. This is now also completed.

Thanks @AndrewKahr , I've now updated UNITY_SERIAL  as well, so now we just need to test and confirm that it's all set correctly now.

I've got #511 ready for merging 😀

This should be fixed now, right?
I see some windows and mac builds still failed, but that's unrelated right?

Yeah the failed runs are just hangs so this is done.


xfreerdp-cli does not fall back to software rendering on some VAAPI errors

My video card (Radeon 6350, Linux radeon kernel driver) only supports up to 2048x1152 for h264 VAAPI decoding, so when I connect to a 1920x1200 display, the VAAPI frame setup fails
To Reproduce
Steps to reproduce the behavior:

With a Radeon 6350, connect to a remote (I used freerdp-shadow-cli on the remonte) with more than 1152 horizontal lines
The initial image renders, but screen never updates (clicking/typing has an effect on the remote computer)
console shows repeated error messages like:[h264 @ 0x7fe7c805e1c0] Failed setup for format vaapi_vld: hwaccel initialisation returned error.
[h264 @ 0x7fe7c805e1c0] decode_slice_header error
[h264 @ 0x7fe7c805e1c0] Hardware does not support image size 1920x1200 (constraints: width 0-2048 height 0-1152).

Setting LIBVA_DRIVER_NAME to something bogus is a workaround; it causes VAAPI to fail to initialize and xfreerdp falls back on software rendering

Expected behavior
xfreerdp-cli should fall back on software rendering
Application details

This is FreeRDP version 2.7.0 (2.7.0)
`xfreerdp /v:host -grab-keyboard
Output of xfreerdp /buildconfig:This is FreeRDP version 2.7.0 (2.7.0)
Build configuration: BUILD_TESTING=ON BUILTIN_CHANNELS=ON HAVE_AIO_H=1 HAVE_EXECINFO_H=1 HAVE_FCNTL_H=1 HAVE_GETLOGIN_R=1 HAVE_GETPWUID_R=1 HAVE_INTTYPES_H=1 HAVE_JOURNALD_H=TRUE HAVE_MATH_C99_LONG_DOUBLE=1 HAVE_POLL_H=1 HAVE_PTHREAD_MUTEX_TIMEDLOCK=ON HAVE_PTHREAD_MUTEX_TIMEDLOCK_LIBS= HAVE_PTHREAD_MUTEX_TIMEDLOCK_SYMBOL=1 HAVE_SYSLOG_H=1 HAVE_SYS_EVENTFD_H=1 HAVE_SYS_FILIO_H= HAVE_SYS_MODEM_H= HAVE_SYS_SELECT_H=1 HAVE_SYS_SOCKIO_H= HAVE_SYS_STRTIO_H= HAVE_SYS_TIMERFD_H=1 HAVE_TM_GMTOFF=1 HAVE_UNISTD_H=1 HAVE_XI_TOUCH_CLASS=1 WITH_ALSA=ON WITH_CAIRO=ON WITH_CCACHE=ON WITH_CHANNELS=ON WITH_CLANG_FORMAT=ON WITH_CLIENT=ON WITH_CLIENT_AVAILABLE=1 WITH_CLIENT_CHANNELS=ON WITH_CLIENT_CHANNELS_AVAILABLE=1 WITH_CLIENT_COMMON=ON WITH_CLIENT_INTERFACE=OFF WITH_CUNIT=ON WITH_CUPS=ON WITH_DEBUG_ALL=OFF WITH_DEBUG_CAPABILITIES=OFF WITH_DEBUG_CERTIFICATE=OFF WITH_DEBUG_CHANNELS=OFF WITH_DEBUG_CLIPRDR=OFF WITH_DEBUG_DVC=OFF WITH_DEBUG_KBD=OFF WITH_DEBUG_LICENSE=OFF WITH_DEBUG_MUTEX=OFF WITH_DEBUG_NEGO=OFF WITH_DEBUG_NLA=OFF WITH_DEBUG_NTLM=OFF WITH_DEBUG_RAIL=OFF WITH_DEBUG_RDP=OFF WITH_DEBUG_RDPDR=OFF WITH_DEBUG_RDPEI=OFF WITH_DEBUG_RDPGFX=OFF WITH_DEBUG_REDIR=OFF WITH_DEBUG_RFX=OFF WITH_DEBUG_RINGBUFFER=OFF WITH_DEBUG_SCARD=OFF WITH_DEBUG_SND=OFF WITH_DEBUG_SVC=OFF WITH_DEBUG_SYMBOLS=OFF WITH_DEBUG_THREADS=OFF WITH_DEBUG_TIMEZONE=OFF WITH_DEBUG_TRANSPORT=OFF WITH_DEBUG_TSG=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF_AVAILABLE=0 WITH_DEBUG_URBDRC=OFF WITH_DEBUG_WND=OFF WITH_DEBUG_X11=OFF WITH_DEBUG_X11_CLIPRDR=OFF WITH_DEBUG_X11_LOCAL_MOVESIZE=OFF WITH_DEBUG_XV=OFF WITH_DSP_EXPERIMENTAL=OFF WITH_DSP_FFMPEG=OFF WITH_EVENTFD_READ_WRITE=1 WITH_FAAC=OFF WITH_FAAD2=OFF WITH_FFMPEG=TRUE WITH_FFMPEG=TRUE WITH_GFX_H264=ON WITH_GPROF=OFF WITH_GSM=OFF WITH_GSSAPI=OFF WITH_ICU=OFF WITH_IPP=OFF WITH_JPEG=ON WITH_LAME=OFF WITH_LIBRARY_VERSIONING=ON WITH_LIBSYSTEMD=ON WITH_MACAUDIO=OFF WITH_MACAUDIO=OFF WITH_MACAUDIO_AVAILABLE=0 WITH_MANPAGES=ON WITH_MBEDTLS=OFF WITH_OPENCL=OFF WITH_OPENH264=OFF WITH_OPENSLES=OFF WITH_OPENSSL=ON WITH_OSS=OFF WITH_PAM=ON WITH_PCSC=ON WITH_PROFILER=OFF WITH_PROXY=ON WITH_PROXY_MODULES=OFF WITH_PULSE=ON WITH_SAMPLE=OFF WITH_SANITIZE_ADDRESS=OFF WITH_SANITIZE_ADDRESS_AVAILABLE=1 WITH_SANITIZE_MEMORY=OFF WITH_SANITIZE_MEMORY_AVAILABLE=1 WITH_SANITIZE_THREAD=OFF WITH_SANITIZE_THREAD_AVAILABLE=1 WITH_SERVER=ON WITH_SERVER_CHANNELS=ON WITH_SERVER_INTERFACE=ON WITH_SHADOW=ON WITH_SMARTCARD_INSPECT=OFF WITH_SOXR=OFF WITH_SSE2=ON WITH_SWSCALE=OFF WITH_THIRD_PARTY=OFF WITH_VAAPI=ON WITH_VALGRIND_MEMCHECK=OFF WITH_VALGRIND_MEMCHECK_AVAILABLE=1 WITH_VERBOSE_WINPR_ASSERT=ON WITH_WAYLAND=ON WITH_WINPR_TOOLS=ON WITH_X11=ON WITH_XCURSOR=ON WITH_XDAMAGE=ON WITH_XEXT=ON WITH_XFIXES=ON WITH_XI=ON WITH_XINERAMA=ON WITH_XKBFILE=ON WITH_XRANDR=ON WITH_XRENDER=ON WITH_XSHM=ON WITH_XTEST=ON WITH_XV=ON WITH_ZLIB=ON
Build type:          Release
CFLAGS:               -fPIC -Wall -Wno-unused-result -Wno-unused-but-set-variable -Wno-deprecated-declarations -Wimplicit-function-declaration -Wredundant-decls -fno-omit-frame-pointer -DWINPR_DLL
Compiler:            GNU, 11.3.0
Target architecture: x64

Connecting to xfreerdp on remote side
log output

Environment (please complete the following information):

OS: Linux
Version/Distribution: NixOS 22.05
Architecture: amd64

Some time ago I did some changes that were allowing the window size to be bigger than the actual physical screen and I hit the same bug. We shall definitely test the capabilities of the VAAPI driver before using it...

all for adding these checks, so who is willing to do it? (e.g. pull requests welcome)
for the time being the feature is still marked experimental so better not use it in production


Reduce school overview memory usage further

Notes
This PR builds on top of #1445 and follows a similar methodology.
When I looked at the MemoryProfiler output for the school overview page, I found another suspicious thing: among the "Retained String Report" output were strings like "999-999-9999 C-Mom" and<EMAIL_ADDRESS>These were suspicious because they are fake generated student contact info. The school overview never surfaces contact info!
Using #select to pull only the necessary fields for students brought the retained memory down (measuring warm path to warm path):
master (current app state):
Total allocated: 3785005 bytes (35109 objects)
Total retained:  443397 bytes (3354 objects)

with student fields #selected in students_query_helper:
Total allocated: 3168397 bytes (29097 objects)
Total retained:  376418 bytes (2603 objects)

Combined with #1445, this change brings retained memory down from the baseline by about 15%.
We are still querying the un-#selected student from the database to get discipline_incidents_count, absences_count, and tardies_count. I think that's the reason we still have some<EMAIL_ADDRESS>showing up in the memory profiling report (but much fewer than in the baseline report).
Checklist

[x] Manually QA visiting School Overview Page
[x] Manually QA running rake school_overview precompute job

Deployed to prod ✅
Ran heroku run rake precompute:school_overview ✅
Manually QA'ed school overview dashboard ✅


How to change the index.html being loaded in Heroku

I have successfully deployed my react app to my heroku server, however, I am getting a blank white screen.
After doing some research, apparently it is because the index.html file is being loaded.
I went to the index.html file and added text and redeployed it and the text showed.
Is it possible for any method to load the App.js instead of the index.html?
Please could I have help on displaying my app and not just the index.html file?
Thank you.

If you're trying to deploy the react app. Can't you run yarn build and use Heroku to host the build directory ( with build/index.html being the entry point)?
Is that not what you want?


Missing required python module: xcbgen

Build Process
yay -Syu
aur/polybar  3.4.1-1 -> 3.4.1-3
Build Log
Build Log
CMakeOutput.log
CMakeError.log
Environment:

Distro (with Version):Arch Linux x86_64(5.4.6-arch3-1)

Duplicate of #1882

You need to make sure you use system python when building, not pyenv. See the wiki.
Basically run pyenv local system before building or build in a clean chroot


fix: adopt new libsinsp logger

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:
This adopts the new logger from libs.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: therealbobo
Once this PR has been reviewed and has the lgtm label, please assign incertum for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: b3600a1ec89c5741f8e364c9d949ff4a092e8feb

/hold

before this, we need to bump Falco to the latest libs commit

/milestone 0.38.0

Could we take care of this next? To make Falco compatible with current libs master?

New changes are detected. LGTM label has been removed.

Engine checksum must be updated ;)

New changes are detected. LGTM label has been removed.

@therealbobo the checksum change must also include a version bump

LGTM label has been added.  Git tree hash: e2fcbd020628790f6971e359b3b6415703233420

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, incertum, jasondellaluce, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,incertum,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, incertum, jasondellaluce, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,incertum,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


ISPN-9754 Cache size method should consult Shared cache stores when not

async

Directly ask loader for size when a shared store

https://issues.jboss.org/browse/ISPN-9754

To be honest a large chunk of this PR is just noise around making the AccessMode a predicate to make some methods a bit more flexible.
The real meat is contained in:

Actual interceptor changes - https://github.com/infinispan/infinispan/pull/6449/files#diff-d70f5361c5448faaa6e2d954a02b42bbR284
Size method via Predicate - https://github.com/infinispan/infinispan/pull/6449/files#diff-d93c51f709416559fc199f2b1687e2f7R774
Explicit test - https://github.com/infinispan/infinispan/pull/6449/files#diff-75800b15db75d9791a8459795a87b74fR106

Actually this is broken when running in a tx. We could only read the underlying entries in the tx and manually add/subtract. This would still prevent size being an N operation with shared store. This needs to be redone.

@wburns NullPointerException on HotRodUpgradeWithStoreTest

Thanks! Actually I forgot to add a check for the flag skip cache loader as
well that needs to be added.
On Wed, Nov 21, 2018, 5:07 AM Ryan Emerson<EMAIL_ADDRESS>wrote:

@ryanemerson requested changes on this pull request.
@wburns https://github.com/wburns Overall appraoch LGTM, but there is a
NullPointerException on HotRodUpgradeWithStoreTest that needs fixing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/infinispan/infinispan/pull/6449#pullrequestreview-177160391,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnj73Oa5dTEhktm-h_ZErbec4MZ9juvks5uxSX1gaJpZM4Yr0N1
.

@ryanemerson weird... the NPE is from something that would have happened before even. Somehow the loaders collection and config collection are out of sync despite holding the read lock for the stores mutex.

Aha, wow okay we have a pretty big bug with disabling a store...
https://github.com/infinispan/infinispan/blob/master/core/src/main/java/org/infinispan/persistence/manager/PersistenceManagerImpl.java#L1186
If you notice it always removes the loader from the status. It should only remove it if the loader matched the type.

It seems this bug is in 9.3, 9.4 and 10.0. Crept in with the store availability changes.

I created https://issues.jboss.org/browse/ISPN-9773 to fix the NPE.

Rebased on the other PR to fix the disable store NPE.


Streams Trait

The eloquent trait for bolting on Streams entry model behavior. This will lightly hook into the model layer for attribute casting and expanding.

Done


Research ways to test the parachain locally

[x] Determine the easiest way to deploy the parachain locally for testing purposes (may require a relay chain)
[x] Update readme with the instructions
[x] Research how to perform a runtime upgrade on a parachain
[x] Research if there's a way to transition of a normal runtime to a parachain one.

Transition from solo chain to a parachain: https://docs.substrate.io/reference/how-to-guides/parachains/convert-a-solo-chain/