Use context correctly

name: Pull request
about: Submit changes to the code

Changes proposed by this PR

Use reconciler context in subsequent async calls, all the way to the client.
Remove redundant WithValues from context based logging

Release Note

PR Checklist
Note: Please do not remove items. Mark items as done [x] or use ~strikethrough~ if you believe they are not relevant

~[ ] Linked to a relevant issue. Eg: Fixes #123 or Updates #123~
[x] Removed non-atomic or wip commits
~[ ] Filled in the Release Note section above~
~[ ] Modified the docs to match changes~

yayy nice! that's a lot of changes 😅  thanks for putting the work!
trying to follow the chain of ctx being propagated through the code, one thing that I noticed that seems a little off is how we pass from cmd/main.go down to pkg/root:

here we're passing it via a struct field

https://github.com/vmware-tanzu/cartographer/blob/1bd6074009be49c21f238aa96d54537912a2669e/cmd/cartographer/main.go#L54

here we're consuming from the struct

https://github.com/vmware-tanzu/cartographer/blob/1bd6074009be49c21f238aa96d54537912a2669e/pkg/root/root.go#L112-L113
that's not very usual, and discouraged - the idiomatic way of doing doing would be to pass it via the function instead, being the first argument of the list.
it's not something introduced in this PR, but, sounds somewhat appropriate to patch given the context (no pun intended) 😁  wdyt?
thx!

sorry, one more 😅  we're doing the right thing creating a context with an associated cancel function
https://github.com/vmware-tanzu/cartographer/blob/1bd6074009be49c21f238aa96d54537912a2669e/cmd/cartographer/main.go#L42-L44
but that will not by default lead to a proper smooth termination when a signal is sent (either us sending a SIGINT / SIGTERM to make run, or kubelet trying to stop the pod with SIGINT)

By default, a synchronous signal is converted into a run-time panic. A SIGHUP, SIGINT, or SIGTERM signal causes the program to exit.

(from https://pkg.go.dev/os/signal#hdr-Default_behavior_of_signals_in_Go_programs)
i.e., despite passing the context to the right places, we're not being graceful at all.
controller-runtime provides a nice little utility for covering this though: https://github.com/kubernetes-sigs/controller-runtime/blob/4d10a0615b11507451ecb58bfd59f0f6ef313a29/pkg/manager/signals/signal.go#L26-L30
// SetupSignalHandler registers for SIGTERM and SIGINT. A context is returned
// which is canceled on one of these signals. If a second signal is caught, the program
// is terminated with exit code 1.
func SetupSignalHandler() context.Context {

so, doing something like

-       if err = cmd.Execute(); err != nil {
+       if err = cmd.Execute(ctrl.SetupSignalHandler()); err != nil {
                panic(err)
        }
 }

we should see that after sending a signal, it actually exits gracefully:
^C{"level":"info","ts":1637262059.6318526,"logger":"controller.workload","msg":"Shutdown signal received, waiting for all workers to finish"}
{"level":"info","ts":1637262059.63193,"logger":"controller.supply-chain","msg":"Shutdown signal received, waiting for all workers to finish"}
{"level":"info","ts":1637262059.6319268,"logger":"controller.deliverable","msg":"Shutdown signal received, waiting for all workers to finish"}
{"level":"info","ts":1637262059.6318586,"logger":"controller.delivery","msg":"Shutdown signal received, waiting for all workers to finish"}
{"level":"info","ts":1637262059.6319535,"logger":"controller.runnable-service","msg":"Shutdown signal received, waiting for all workers to finish"}
{"level":"info","ts":1637262059.6319625,"logger":"controller.delivery","msg":"All workers finished"}
{"level":"info","ts":1637262059.6319673,"logger":"controller.runnable-service","msg":"All workers finished"}
{"level":"info","ts":1637262059.6319716,"logger":"controller.deliverable","msg":"All workers finished"}
{"level":"info","ts":1637262059.631974,"logger":"controller.supply-chain","msg":"All workers finished"}
{"level":"info","ts":1637262059.6319742,"logger":"controller.workload","msg":"All workers finished"}

(without that context w/ proper signal handling, we don't have the messages above ^)

yayy nice! that's a lot of changes 😅 thanks for putting the work! overall, lgtm!
trying to follow the chain of ctx being propagated through the code, one thing that I noticed that seems a little off is how we pass from cmd/main.go down to pkg/root:

here we're passing it via a struct field

https://github.com/vmware-tanzu/cartographer/blob/1bd6074009be49c21f238aa96d54537912a2669e/cmd/cartographer/main.go#L54

here we're consuming from the struct

https://github.com/vmware-tanzu/cartographer/blob/1bd6074009be49c21f238aa96d54537912a2669e/pkg/root/root.go#L112-L113
that's not very usual, and discouraged - the idiomatic way of doing doing would be to pass it via the function instead, being the first argument of the list.
it's not something introduced in this PR, but, sounds somewhat appropriate to patch given the context (no pun intended) 😁 wdyt?
thx!

I agree but I don't think I want to wait to rebase against main again just to get this work in.

Issue created: https://github.com/vmware-tanzu/cartographer/issues/361


[pipecleaner] Don't assume dummy variables are safe.

What
When shellchecking scripts in the pipelines, pipecleaner inserts dummy
export statements for each of the PARAMS from the pipeline to prevent
shellcheck complaining about undefined variables. Because these were set
to the static string 'DUMMY', shellcheck wouldn't complain if they were
used unquoted in the scripts because it assumed their value was safe.
Inserting a call to date in these prevents shellcheck from assuming
they're safe.
How to review
Code review.
Who can review
Anyone but myself.

Excellent fix, nice work. Will merge when the tests pass.


Publicly publish each stack version

Reverting back to old versions of the stack is a little tricky. In #138 at least there's a URL outside of Buildkite that people can use. But it'd be nicer if there was a public list of releases where you could download the stack JSON file.
One idea would be to have a <major-ver>.<auto-incrementing-number> auto-tagging + release step as part of publishing, so you could go to GitHub releases and find previous releases of the stack. The major version could be checked into the repo, the minor version could just be the Buildkite build number?

I've just implemented this manually for now. You simply tag a release, upload to S3 to the right place, and create a GitHub release.


Secure your application with XSUAA (Forecast App)

Tutorial URL: https://developers.sap.com/tutorials/hxe-aa-forecast-17.html
I believe it should be updated with new HANA Express version.

when we run the html or js app, it will automatically generated service instances
therefore, step 3 and 4 should be updated because it will not use the service instance what we created.

after I added new role with the generated application into the role collection, it worked

-> This is new feature of Hana Express 2.0 SPS03.
Therefore, we don't need to create service but need to assign auto generated application to Role collection manually.

Unfortunately, when I select auto-arima and click the 'Execute' button, it will return error.
I believe we may need new code to fetch csrf token but not sure about it.

#ERROR#/Handler#####k8q7nm48####pgzz6foq7Lr5FIIxIrFBCyODkS-Z9few######k8q7nm48#PLAIN##POST request to /xsjs/pal/auto_arima.xsjs completed with status 403 - The request does not contain a x-csrf-token#
4/7/20 2:02:00.253 PM [RTR] OUT   <IP_ADDRESS> - - to hxehost:51054 "POST /xsjs/pal/auto_arima.xsjs HTTP/1.1" 403 sent 19 in 10 by 01A-5325bc98-16a1-43db-87de-2fb48f4a1a4a
-> I fixed it and put it into my git.
https://github.com/jameslim87/forecast.git

we need to add a empty file to fetch a csrf token.
we need to add a js file for fetching token
index.html should include #2 js file.

i will look into these changes this week. thank you

@jarjarbentley Any updates?

I've made slightly different changes than suggested.  Rather than let the Web IDE create the resources, I kept the flow with creating the service manually. However I changed the resource type to org.cloudfoundry.existing-service in the mta.yaml. This way the Web IDE will not try and create the resources at run any longer.  I think this makes the flow easier for educational purposes since you aren't bouncing back and forth in order to add the generated role to the role collection after the run (and potentially having to log out of our browser in order pickup the role collection change).
I also removed the requirement for csrf in the configuration of the xs-app.json later in the tutorial.


When will FABind+ be released?

The README says soon, any idea when that would be? I'd be really interested in trying it!

Thanks for your interest. We feel sorry for the delay and will provide the code of FABind+ in next few weeks.

Ok good to hear, thank you! Will the weights be released as well?

Sure.

@GriffinYoung
The code, scripts and weights are released today. Check it out!


Small docs updates

Fix links
Integrate Settings File info into Running the ETL docs page
Remove old Settings File from docs (and indexes)
Add some comment headers to the actual deployed settings files to clarify which settings pertain to which script

Note that the docs build is failing because there are still references to the now removed settings page.

Note that the docs build is failing because there are still references to the now removed settings page.

It seems like the error is due to some :download: commands in the data_source pages:


QA: comment incentives [payment-permit-max-price]

TEST
should apply payment-permit-max-price for comment and task creator
PASSED

Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum.

a
b
c
d

/start


skywalking-rocketbot-ui6.6 versioon  why An error was reported by the console when the project started in topology module eg:1. [vuex] unknown action type: rocketTopo/CLEAR_TOPO_INFO 2.Error in beforeDestroy hook: "TypeError: Cannot read property 'runtime' of undefined"

why An error was reported by the console when the project started in topology module
eg:1. [vuex] unknown action type: rocketTopo/CLEAR_TOPO_INFO
2.Error in beforeDestroy hook: "TypeError: Cannot read property 'runtime' of undefined"

Duplicate of https://github.com/apache/skywalking-rocketbot-ui/issues/240


[SPARK-23778][CORE] Avoid unneeded shuffle when union gets an empty RDD

What changes were proposed in this pull request?
When a union is invoked on several RDDs of which one is an empty RDD, the result of the operation is a UnionRDD. This causes an unneeded extra-shuffle when all the other RDDs have the same partitioning.
The PR ignores incoming empty RDDs in the union method.
How was this patch tested?
added UT

Test build #90647 has started for PR 21333 at commit f67a88d.

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/testing-k8s-prb-make-spark-distribution/3236/
Test PASSed.

Test build #90647 has finished for PR 21333 at commit f67a88d.

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/90647/
Test PASSed.

Nice change! I tested this out as well and verified that the shuffle doesn't happen. I did notice that this change wasn't reflected in the dataset API. Is that something that should be addressed in this change?

@varuvish the Dataset API uses sparkContext.union under the hood, so it is addressed as well by the current change.

cc @cloud-fan @JoshRosen

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/testing-k8s-prb-make-spark-distribution-unified/327/
Test PASSed.

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/testing-k8s-prb-make-spark-distribution/4221/
Test PASSed.

Test build #92098 has started for PR 21333 at commit 7f16ea0.

LGTM

Test build #92098 has finished for PR 21333 at commit 7f16ea0.

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/92098/
Test PASSed.

thanks, merging to master!


Manipulate file with > 1GB

I saw this plugin is capable to read and write file with over > 256MB in size. However, both mobile platforms crashed while the size reaches 512MB.
I am developing an app which needs to unzip a zip file with >1GB, is it possible?

Can you provide any error logs? It would be good to find out exactly what caused the crashes.

Sorry that I haven't tried yet. I am just asking because of the README Document stated that

Android will crash as "Quota exceeded[2]" for 512MB
iOS will crash as "WebKit error[2]" for 512MB

I'm sorry, I don't know what will happen with really large files. I would expect write_blob to go into fallback mode if the file is really big. The file will be written (if there is enough space on disk) but it will be written quite slowly.


Pipeline Query Language (querying metrics) Feature Request

Not sure if this is the right place to request a feature, but I am interested to find an open source database to add capabilities in querying metrics with an M3QL/promQL-like language. Would dicedb benefit from this or would that be too out of scope? Under the hood it can be a simple M3QL -> SQL language translation (CTEs would need to be supported). Schema requirements would be up for debate, but there would need to be a timestamp column, value column and a column to support an internal "json" to support metrics tags. Data ingestion would need to be streamed and be immutable. Similar to the leaderboard demo, perhaps a QWATCH can be placed on a metric query to generate a grafana-like timeseries display.
M3QL gives simple ways to express metrics transforms like summarizing bucketed summation over time-windows and finding the percentage of a metric against a time-shifted version of itself. I would love to see it supported outside of Uber in the open source world. All of this is possible in SQL as well, but the syntax becomes too cumbersome for a human to write.
I'm happy to give further ideas/support if there is interest here.

Actually a KV store may make things slightly simpler. I assumed sql db because of the qwatch example but being a redis replacement I should have done more diligence. In this case you don't need a schema; M3DB is a kv store as well. One way to do it is convert ingested metrics into a {m3id, epochMillis} pair as the key and the measurement/count as the value where the m3id is sorted and delimited string for all the tags of the emitted metric. Now querying becomes simple if you know the m3id, the bucketed timerange and have enough compute to perform all the summation operations. The tricky part is that queries will only contain a partial list of tags e.g. matching
fetch name:ingest-event error:true zone:dca* | sum shadow-mode | summarize 5m avg

to match the tags up to all M3IDs

"name=ingest-event,zone=dca1,error=true,shadow-mode=on,env=production"
"name=ingest-event,zone=dca2,error=true,shadow-mode=off,env=staging"
...
Which Uber uses ES for this (which may not be the best solution). OK will block time for later in the week thanks, sounds exciting.


New Term Request: increased hepatocyte number

Please add: increased hepatocyte number
def: greater than normal number of parenchymal liver cells
child under: abnormal hepatocyte morphology
reference: J:90905.
Thanks,
Monika

New term added:
increased hepatocyte number MP:0014038


[FLINK-33126] Fix EventTimeAllWindowCheckpointingITCase jobName typo

What is the purpose of the change
Fix EventTimeAllWindowCheckpointingITCase jobName typo
Brief change log
Adjust the jobName of test in EventTimeAllWindowCheckpointingITCase
Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is a trivial rework

@flinkbot run azure


Wpf controls

Initial implementations of the UiaMeta.find_wrapper method and the ButtonWrapper for UIA (Issue #142)

Current coverage is 96.55%

Merging #148 into UIA will increase coverage by +0.01% as of e550bc1

@@              UIA    #148   diff @@
=====================================
  Files          29      31     +2
  Stmts       17041   17094    +53
  Branches        0       0
  Methods         0       0
=====================================
+ Hit         16453   16505    +52
  Partial         0       0
- Missed        588     589     +1

Review entire Coverage Diff as of e550bc1

Powered by Codecov. Updated on successful CI builds.


Boundwith DAG for handling uploaded CSV files

Fixes #771
TODO:

[x] Add support for email
[x] Unit tests

@jermnelson ran two tests using the same csv (first time the barcode was formatted by excel incorrectly). I can see in the DAG that the email step failed, but the BW relationship was also not created in folio. Second one is https://sul-libsys-airflow-dev.stanford.edu/dags/add_bw_relationships/graph?dag_run_id=manual__2023-12-05T13:52:38.746093+00:00
Let me know if you want the csv.

Merging this PR to allow work on follow-up ticket #821


DirectoryUtils - isEqual to compare directories

First implementation of DirectoryUtils.isEqual() including optional file content compare.

Jira issue for discussion: https://issues.apache.org/jira/browse/IO-532

What is the use case? I can hardly imagine a case, where I would want to compare directories. If I need equality, then I'd rather create a copy in a temporary directory.

Besides, rather than simply returning a boolean value, I'd hope for an event API, like
    public interface DirCompareListener {
        /**
         * Called, if a file has been detected in the first directory, which is missing in the second.
         * In other words, the difference can be remedied by copying the file from the first
         * directory to the second.
         * @param pFirstDir First directory being compared (root directory); the directory, where
         *                               a file has been found, which is not present in the second directory.
         * @param pSecondDir Second directory being compared (root directory); the directory,
         *                               where a file is missing.
         * @param pFile Path pf the additional file, relative to the root directory.
         */
         public void fileAdded(Path pFirstDir, Path pSecondDir, String pFileName);
    }
    /** Method, which compares two directories by iterating over the files in the first directory,
      * checking, whether they are present (and identical) in the second directory, and vice versa.
      * Upon detection of differences, a suitable method of the listener is being invoked.
      */
    public void compareDirectories(Path pFirstDir, Path pSecondDir, DirCompareListener pListener);
    /** Method, which compares two directories by iterating over the files in the first directory,
      * checking, whether they are present (and identical) in the second directory, and vice versa.
      * Upon detection of differences, the value false is being returned, otherwise returns true.
      */
    public boolean compareDirectories(Path pFirstDir, Path pSecondDir) {
        final DirCompareListener listener = new DirCompareListener(){
             public void fileAdded(Path pFirstDir, Path pSecondDir, pFileName) {
                 throw new IllegalPathStateException();
             }
        };
        try {
              compare(pFirstDir, pSecondDir, listener);
              return true;
        } catch (IllegalPathStateException e) {
              return false;
        }
   }

You're right it is probably not a daily use case, but I thought it would be handy to have a routine for this.
For small automated directory comparison the boolean should be OK, of course it can be improved to an API-like routine. The intention was to setup a quick comparison routine as such was not already present...

This would be great to have to do integration testing of libraries generating some file trees.
Is there any plans on merging this PR?

Did you copy from https://stackoverflow.com/questions/14522239/test-two-directory-trees-for-equality ? Are you the author? If not, you should at least link or credit.

Yes I was the author of that SO snippet back in 2017 :). Of course we can link to the original SO reply.
Thx for the review - I'll see when I can clean it up.
Best regards,
Johannes

Please see git master for new APIs PathUtils.
Jira: https://issues.apache.org/jira/browse/IO-648
This can be closed IMO.
Feedback welcome.


Add support for Neural Network graphs

I would like a feature to allow us to show neural network graphs, something akin to this one:
http://hackology.co.uk/wp-content/uploads/2013/08/neuralnetwork.png

Fixed the edge references so this will work well now. An example added to visual tests.


Weather is not displayed

Describe the bug
Weather is not displayed
To Reproduce
Set the weather to be displayed. I made tons of tests. I'm sure that the MQTT message is well formatted and published on my MQTT broker.
Is there something changed with this ? maybe I am missing something...
I tried several weather HA integrations.
Smartphone (please complete the following information):

Device: Samsung GT-P5110
OS: Android 7 Lineage OS 14
Applicaiton version [app > settings > version] v 1.1.2 build0

You need to post your MQTT message or automation that publishes weather.   The weather has changed because now we use the generic weather integration.  But the documentation has been updated with examples.

I am using the the payload_template described in documentation.
{''weather'':{''condition'':''{{states.weather.home.state}}'',''humidity'':{{states.weather.home.attributes.humidity}},''temperature'':{{states.weather.home.attributes.temperature}},''forecast'':{{states.weather.home.attributes.forecast}}}}
And I am using the Meteorologisk institutt (Met.no) integration (I think it the default one).

Have you tried to use the HA dev tools to evaluate this and see that it works?   Also in the MQTT configuration there is a tool to subscribe to the weather to see what is being broadcast.   I don't see any other issues with weather, I am assuming it's on your end perhaps.

I was not able to reproduce this and no others have reported issues with weather.   Maybe bad formatted MQTT message, you can check the logs in the setting section of the app.

i have two tablets running the alarm panel. one shows weather and the other doesnt...

Not seen this issue on any of my test devices.   Check the logs in the settings to see that the tablets are receiving the weather information and that they are subscribed to the correct event in the settings.

I have a similar issue in that 1.0.1 is receiving the weather and sun mqtt just fine, but not receiving 1.1.2.  Maybe it is the topic, but the topic setting screen has changed from 1.01 to 1.1.2 and I'm not clear on which topic setting monitors sun/weather in 1.1.2.

I think the most likely issue is quotes in the MQTT message, which cause parsing errors for the JSON.   If you need help with the sun/weather, check the wiki for examples.

This is my weather YAML for Home Assistant:
alias: MQTT Weather
trigger:
  - minutes: /15
    platform: time_pattern
condition: []
action:
  - data:
      payload_template: >-
        {'weather':{'condition':'{{states.weather.home.state}}','humidity':{{states.weather.home.attributes.humidity}},'temperature':{{states.weather.home.attributes.temperature}},'forecast':{{states.weather.home.attributes.forecast}}}}
      retain: true
      topic: alarmpanel/command
    service: mqtt.publish
mode: single

This is my sun YAML:
alias: MQTT Sun
trigger:
  - minutes: /30
    platform: time_pattern
condition: []
action:
  - data:
      payload_template: '{''sun'':''{{states(''sun.sun'')}}''}'
      retain: true
      topic: alarmpanel/command
    service: mqtt.publish
mode: single

Thanks, I'll give it a go.   Mine had been formatted as such (works with prior version)

id: 486a7b1ec65c4acaa373bbc04e31d657
alias: MQTT Weather
trigger:

minutes: /30
platform: time_pattern
condition: []
action:
data:
payload_template: '{''weather'':{''condition'':''{{states.weather.home.state}}'',''humidity'':{{states.weather.home.attributes.humidity}},''temperature'':{{states.weather.home.attributes.temperature}},''forecast'':{{states.weather.home.attributes.forecast}}}}'
retain: true
topic: alarmpanel1/command
service: mqtt.publish

I tried using your examples and it still doesn't work.   The app does not appear to be subscribing to the topic I have configured for remote command alarmpanel2/command
I am monitoring the topic with MQTTBox and can confirm the topic is receiving the post.   The app does not show any activity for this topic in the log.

Use default client name in settings rather than change to alarmpanel2.  In my example I’m using default settings.


Append to index file occurring at wrong position: API nodeos failure since 1.7.2

We've recently suffered two failures to nodes on EOSIO v1.7.2. In both cases, these nodes are API nodes and handle public traffic, as well as use the history_api plugin. Although the history_api is deprecated, it is likely that the issue will affect other API's as well (such as get_block) so it may be useful to investigate.
The failure scenario begins with looping errors that look like the following:
error 2019-04-23T12:08:52.824 thread-0  http_plugin.cpp:652           handle_exception     ] FC Exception encountered while processing history.get_transaction
debug 2019-04-23T12:08:52.824 thread-0  http_plugin.cpp:653           handle_exception     ] Exception Details: 13 St13__ios_failure: basic_ios::clear: iostream error
rethrow basic_ios::clear: iostream error:
    {"what":"basic_ios::clear: iostream error"}
    thread-0  block_log.cpp:64 check_index_read
rethrow
    {}
    thread-0  block_log.cpp:289 read_block_by_num
    {"block_num":27593175}
    thread-0  controller.cpp:1980 fetch_block_by_number
warn  2019-04-23T12:09:04.963 thread-0  block_log.cpp:64              check_index_read     ] 13 St13__ios_failure: basic_ios::clear: iostream error
rethrow basic_ios::clear: iostream error:
    {"what":"basic_ios::clear: iostream error"}
    thread-0  block_log.cpp:64 check_index_read
warn  2019-04-23T12:09:04.963 thread-0  block_log.cpp:289             read_block_by_num    ] 13 St13__ios_failure: basic_ios::clear: iostream error
rethrow basic_ios::clear: iostream error:
    {"what":"basic_ios::clear: iostream error"}
    thread-0  block_log.cpp:64 check_index_read
error 2019-04-23T12:09:04.963 thread-0  http_plugin.cpp:652           handle_exception     ] FC Exception encountered while processing history.get_transaction
debug 2019-04-23T12:09:04.963 thread-0  http_plugin.cpp:653           handle_exception     ] Exception Details: 13 St13__ios_failure: basic_ios::clear: iostream error
rethrow basic_ios::clear: iostream error:
    {"what":"basic_ios::clear: iostream error"}
    thread-0  block_log.cpp:64 check_index_read
rethrow
    {}
    thread-0  block_log.cpp:289 read_block_by_num
    {"block_num":27593175}
    thread-0  controller.cpp:1980 fetch_block_by_number

When shutting down nodeos, the following message appears:
warn  2019-04-23T13:42:40.117 thread-0  block_log.cpp:214             append               ] 3190002 block_log_append_fail: fail to append block to the block log
Append to index file occuring at wrong position.
    {"position":"18446744073709551615","expected":435561376}
    thread-0  block_log.cpp:202 append
warn  2019-04-23T13:42:40.487 thread-0  block_log.cpp:214             append               ] 3190002 block_log_append_fail: fail to append block to the block log
Append to index file occuring at wrong position.
    {"position":"18446744073709551615","expected":435561376}
    thread-0  block_log.cpp:202 append
terminate called after throwing an instance of 'eosio::chain::block_log_append_fail'

It is not possible to re-launch nodeos without a replay after this failure, as the database dirty flag is set.
This error is very curious: it is new since 1.7.2, and it appears very unlikely -- it happened on a node with high traffic, and on a node with very little traffic, but not on several other nodes with higher and lower traffic, so it is not clear if it is related to API throughput.
Some thoughts on cause:

It may be possible that a specific API call, reading/addressing the block log, is interfering with writing.
It may be a threading issue.
It is unlikely to be a memory corruption issue as it happened twice to different nodes, with ECC memory.

What is the value of ulimit -n ?

1024 -- the default soft limit. True for all servers, those that failed and those that have not.

That should be increased to something like 65536 as 1024 is too low and can starve nodeos of needed file descriptors.

We'll try that, and report back if the issue still occurs.

It is possible that 1.7.2 is more acceptable to this starvation as it handles requests quicker which allows it to consume more file descriptors.

I ran out of file descriptors on 1.7.2 earlier this week when my disk I/O went through the roof due to some other unrelated activity and everything slowed to a crawl.... nodeos wouldn't shutdown and had to restore from backup.  (no history or anything fancy enabled).

Fixed by #7199

This is fixed in 1.7.3. Please open new issue if you see this in 1.7.3+

experienced same issue in 1.7.3-1

@applesauc3 Can you provide your log?

info  2019-05-07T19:28:15.501 thread-0  producer_plugin.cpp:1597      produce_block        ] Produced block 000a1f5ef7b47ed4... #663390 @ 2019-05-07T19:28:15.500 signed by eosio [tr
xs: 0, lib: 663389, confirmed: 0]
warn  2019-05-07T19:28:16.002 thread-0  block_log.cpp:205             append               ] 13 St13__ios_failure: basic_ios::clear: iostream error
rethrow basic_ios::clear: iostream error:
    {"what":"basic_ios::clear: iostream error"}
    thread-0  block_log.cpp:205 append
warn  2019-05-07T19:28:16.002 thread-0  producer_plugin.cpp:1542      maybe_produce_block  ] 13 St13__ios_failure: basic_ios::clear: iostream error
rethrow basic_ios::clear: iostream error:
    {"what":"basic_ios::clear: iostream error"}
    thread-0  block_log.cpp:205 append
warn  2019-05-07T19:28:16.500 thread-0  producer_plugin.cpp:1542      maybe_produce_block  ] 3020000 fork_database_exception: Fork database exception
committed block did not become the new head in fork database
    {}
    thread-0  controller.cpp:683 commit_block
warn  2019-05-07T19:28:16.500 thread-0  producer_plugin.cpp:1131      start_block          ] 3060000 database_exception: Database exception
db revision is not on par with head block
    {"db.revision()":663390,"controller_head_block":663391,"fork_db_head_block":663391}
    thread-0  controller.cpp:1109 start_block
error 2019-05-07T19:28:16.500 thread-0  producer_plugin.cpp:1429      schedule_production_ ] Failed to start a pending block, will try again later
warn  2019-05-07T19:28:16.551 thread-0  producer_plugin.cpp:1131      start_block          ] 3060000 database_exception: Database exception
db revision is not on par with head block
    {"db.revision()":663390,"controller_head_block":663391,"fork_db_head_block":663391}
    thread-0  controller.cpp:1109 start_block

producing blocks and running state history plugin, ulimit -n 1048576

attempting to reproduce but nothing yet. was looking through the change and the error and hunch is something around https://github.com/EOSIO/eos/pull/7199/files#diff-5fe104cf373746bc2b06b9f5d57b1aa9R182
also curious why reopen is kinda hacky close open close open with new RW mode https://github.com/EOSIO/eos/pull/7199/files#diff-5fe104cf373746bc2b06b9f5d57b1aa9R56

The "hacky" close open close open came up in peer review as well. It is there in case the file doesn't exist, the first one will create it while the second one would not. That is only called on startup and a corner case on shutdown.

lsof did not indicate any runaway file descriptors

I've only seen that error when out of file descriptors, but you may have hit something else.


autoscaling slaves resets users

autoscaling works great, but when a new slave is added, it slows down the RPS curve, because slaves are reset - wouldn't it be better if the slaves would not go all the day down to 0?

I agree. The problem is this line:
https://github.com/locustio/locust/blob/013d64ac6de8e4b6f07fc2b704eeea55601e9858/locust/runners.py#L336
I believe that it might be completely unnecessary, and that we can simply fix the problem by removing that line. Did a quick test that seems to support this, but will look a little deeper into it.

I believe that it might be completely unnecessary, and that we can simply fix the problem by removing that line.

That wasn't really true, but the problem have now been solved in c16dc71

works now, thanks

@heyman
I'm still seeing this in version 0.13.1.

Logs look OK. Master:
[2019-11-18 16:26:23,697] locust-opa-master-787476b668-mlrz9/INFO/locust.main: Starting web monitor at *:8089
[2019-11-18 16:26:23,698] locust-opa-master-787476b668-mlrz9/INFO/locust.main: Starting Locust 0.13.1
[2019-11-18 16:26:53,182] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-kmhlw_aebbf1d9b85e4bc4b5d18431d9a4c0d3' reported as ready. Currently 1 clients ready to swarm.
[2019-11-18 16:34:47,209] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 1 ready clients
[2019-11-18 16:36:19,065] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-phgt9_47a162891b3a448ea366205c37607157' reported as ready. Currently 2 clients ready to swarm.
[2019-11-18 16:36:19,065] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 2 ready clients
[2019-11-18 16:38:20,539] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-4kfkf_a2f31c4d7f7d405a95b5ca4d3d8c3d36' reported as ready. Currently 3 clients ready to swarm.
[2019-11-18 16:38:20,539] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 3 ready clients
[2019-11-18 16:41:22,522] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-vvjb4_a0b8d33b274e40b58ff4a8225370dd8b' reported as ready. Currently 4 clients ready to swarm.
[2019-11-18 16:41:22,523] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 4 ready clients
[2019-11-18 16:41:22,731] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-scbd6_065d9980a1ee4629b15b5b79aaed6ff2' reported as ready. Currently 5 clients ready to swarm.
[2019-11-18 16:41:22,731] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 5 ready clients
[2019-11-18 16:44:24,490] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-zfz9d_306a9987e14042efab8aec45720acf28' reported as ready. Currently 6 clients ready to swarm.
[2019-11-18 16:44:24,491] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 6 ready clients
[2019-11-18 16:44:24,622] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-cddmn_06ff9d1f91334e809499cf522bd838cc' reported as ready. Currently 7 clients ready to swarm.
[2019-11-18 16:44:24,623] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 7 ready clients
[2019-11-18 16:47:26,348] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-x95nc_6172fcc3741b4408aa6db4c5b933eee3' reported as ready. Currently 8 clients ready to swarm.
[2019-11-18 16:47:26,348] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 8 ready clients
[2019-11-18 16:47:26,567] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Client 'locust-opa-slave-86cf6df9fb-6fgrl_6c203c03e497472289ed644e2e7ff09f' reported as ready. Currently 9 clients ready to swarm.
[2019-11-18 16:47:26,568] locust-opa-master-787476b668-mlrz9/INFO/locust.runners: Sending hatch jobs to 9 ready clients

Slaves look like this:
[2019-11-18 16:26:36,988] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.main: Starting Locust 0.13.1
[2019-11-18 16:34:47,210] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Hatching and swarming 1000 clients at the rate 0.5 clients/s...
[2019-11-18 16:36:19,066] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 500 locusts
[2019-11-18 16:38:20,540] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 167 locusts
[2019-11-18 16:41:22,525] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 83 locusts
[2019-11-18 16:41:22,732] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 50 locusts
[2019-11-18 16:44:24,492] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 33 locusts
[2019-11-18 16:44:24,626] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 24 locusts
[2019-11-18 16:47:26,352] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 18 locusts
[2019-11-18 16:47:26,572] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 14 locusts
[2019-11-18 16:50:28,707] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 11 locusts
[2019-11-18 16:50:28,766] locust-opa-slave-86cf6df9fb-kmhlw/INFO/locust.runners: Killing 9 locusts

@heyman can we reopen this issue?


Camera message docs (WIP)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

wrong upstream, my bad


Feature: add new nodes to the center of the visible area of the graph

Currently the nodes are added somewhere to the top left corner so the user has to move them quite a bit

see #70 there's probably some more to it than simply centering new nodes.

i think this is now done.


Solving Visa Raw Connection issue with weird addresses like GPIB over LAN

driver for Tinybuilt now compatible with ecpy (term character issues solved)
driver for Anapico sources now exploit the new firmware and set the timeout before reconnection to 0 (avoid loosing connection for good in case of an error)
driver for the R&S ZVA24 vector network analyzer

Why did you close ?!? Pushing to the branch is sufficient.


fix host value on swagger.json

https://maproulette.org/assets/swagger.json is returning host as "localhost:9000" instead of maproulette.org

thanks!


latest version not working

[ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: MissingPluginException(No implementation found for method ssid on channel eng.smaho.com/esptouch_plugin/example)
E/flutter (23722): #0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:156:7)
E/flutter (23722):
E/flutter (23722): #1      ssid (package:join_kit/wifi_info.dart:9:10)
E/flutter (23722):
E/flutter (23722): #2      _MyAppState.fetchWifiInfo (package:join_kit/main.dart:104:20)
E/flutter (23722):

======== Exception caught by services library ======================================================
The following MissingPluginException was thrown while activating platform stream on channel eng.smaho.com/esptouch_plugin/results:
MissingPluginException(No implementation found for method listen on channel eng.smaho.com/esptouch_plugin/results)
When the exception was thrown, this was the stack:
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:156:7)

#1      EventChannel.receiveBroadcastStream. (package:flutter/src/services/platform_channel.dart:545:9)

I also have had trouble getting the ssid using the custom flutter channel when on ios. Not sure if this is even implemented?
But as the developer mentioned, probably better to use a network_info specific library for getting the built in ssid. Those libraries are going to be updated and built just for this purpose.
I am using https://pub.dev/packages/network_info_plus this one for filling the ssid.

======== Exception caught by services library ====================================================== The following MissingPluginException was thrown while activating platform stream on channel eng.smaho.com/esptouch_plugin/results: MissingPluginException(No implementation found for method listen on channel eng.smaho.com/esptouch_plugin/results)
When the exception was thrown, this was the stack: #0 MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:156:7) #1 EventChannel.receiveBroadcastStream. (package:flutter/src/services/platform_channel.dart:545:9)

Can you tell me how to resolved this?

Use the network_info_plus to get the WiFi SSID (it's the WiFi name).


Allow more options when automating Security System accessory

Creating an automation for the Security System accessory exposes three options:

Triggers
Disarms
Arms

There should be two additional options provided, to correspond with the Alarm.com API:

Arms (stay)
Arms (away)

They should be added because the Alarm.com API provides this nuance for arming an alarm system. It's common to most alarm systems, really. "Arms (stay)" means that the perimeter of the system is armed, but motion sensors are not; this is common for people who are staying home to sleep, for example, and want the perimeter sensors armed, but do not want to trigger the motion sensors in the middle of night if they walk around or inevitably move in bed. "Arms (away)" means that all sensors are armed.
Exposing this nuance in Home's automation section means that a user can create an automation for turning off all the lights when "Arms (away)" is selected. Currently, selecting "Arms" handles both the "stay" and "away" situations. That makes creating such an automation ineffective, as users who are arming with "stay" probably do not want to turn all the lights off, while users arming with "away" almost certainly do.
Can these provided options be modified, or are they up to Apple's team developing HomeKit and the Home app for the Security System accessory?

You'll notice that this plugin and the Home app support both Armed states. In the Home app, viewing the device will give you options to set Away (Armed Away) and Night (Armed Stay) states.

The issue you're describing is with the automation capabilities of Home and/or HomeKit.

The issue you're describing is with the automation capabilities of Home and/or HomeKit.

I know. That's the issue I described. That's why I opened the ticket. I have no issue setting those nuanced states outside of creating an automation. I notice you marked this ticket closed but also "not possible yet." We'll have to wait and see what Apple exposes in future updates :+1:

@danemacmillan With all due respect, open a ticket with Apple. This is not an issue with my plugin. It works as intended.

@danemacmillan

Can these provided options be modified, or are they up to Apple's team developing HomeKit and the Home app for the Security System accessory?

@bryanbartow

The issue you're describing is with the automation capabilities of Home and/or HomeKit.

Yes, that's exactly how I described it--as likely on their end. My ticket wanted confirmation on this very point. You misread. I was not blaming your code for Apple's shortcomings. I was looking to confirm my suspicion that it is likely on their end and therefore their responsibility.
@danemacmillan

We'll have to wait and see what Apple exposes in future updates.

@bryanbartow

This is not an issue with my plugin.

Yes, we both agree.

Thanks. If Apple ever supports this, I'll re-open the ticket and consider it a feature request.


"TextCompositionLayer.swift" used twice IOS

Hello, I have a problem with iOS building.

info :0: error: filename "TextCompositionLayer.swift" used twice: '/Users/XXX/Documents/React-Native/PROJECT/node_modules/lottie-ios/lottie-swift/src/Private/LayerContainers/CompLayers/TextCompositionLayer.swift' and '/Users/XXX/Documents/React-Native/PROJECT/node_modules/lottie-ios/lottie-swift/src/Private/LayerContainers/CompLayers/TextCompositionLayer.swift'
info :0: note: filenames are used to distinguish private declarations with the same name
info Command /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swiftc failed with exit code 1

Lottie-react-native @ 3.0.3
React-native @ 0.59.8

FIXED!
Problem comes through the installation. No need to write npm i --save<EMAIL_ADDRESS>because npm i --save lottie-react-native package has lottie-ios package inside node_modules. Therefore, when you write react-native link lottie-ios it ruins and duplicates everything.
So, the installation must be (for 0.59):

Remove from node_modules / unlink all lottie files;
Write npm i --save lottie-react-native;
Go to project/node_modules/lottie-react-native/node_modules;
Cut lottie-ios folder and paste it into the project node_modules folder near lottie-react-native;
Write
react-native link lottie-ios react-native link lottie-react-native

That's it! :)

I'm glad for you :)
What about 0.60> ? There's no react-native link, I just add the packages with yarn, pod istall, and no way to build the project.. so sad.

@brascene are you having any issues? There's no react-native link needed in RN 0.60

@brascene Just try to do all steps, except fifth. React-Native 0.60 has auto-linking.

Thank you guys for quick support, and yeah @emilioicai I have issue that I cannot build iOS project after following simple installation guide:

yarn add lottie-react-native,
yarn add<EMAIL_ADDRESS>pod install

There was like kilometer long error like nothing is working xD I've tried to delete pods, and node modules etc. but no progress still. When I run the app from terminal with react-native run-ios command I get that long error, and when I run the app from XCode directly, there're two errors, one of them was fatal error: module map file... etc..
Currently (after deleting and installing again, clearing derived data, pods, and node modules), now I have 100 errors (all complaining about Undefined symbol and many warnings related with Could not find autolinked library).
@LPranulis
lottie-ios@  node_module is already inside project's node_modules, so all your steps are covered with above two commands..
I have no idea what happend the first time I added this to my project, now I can't run it even without lottie..

@brascene Not related to auto linking at all but did you try adding a swift file as suggested here? https://stackoverflow.com/a/56176956/3265176

I'll give it a try in a moment

OMG it's working :)
This is what magically make it work:
$(TOOLCHAIN_DIR)/usr/lib/swift/$(PLATFORM_NAME)
I hope Android won't have similar issue..
Thanks again for very quick support! And thanks for Lottie!

there's something in lottie-ios project under the project.pbxproj file where
on line 639 and line 676 TextCompositionLayer.swift is included twice

@ganoch Have the same error, and how do i fix this now? :(

@LPranulis i don't have anything inside lottie-react-native's node_modules, it's just an empty folder. Im using RN 0.60.4 and no cocoapods. Any help?

I made it work, it took me so many hours but finally. Had to add some stuff on react-native-lottie

OMG it's working :) #513 This is what magically make it work: $(TOOLCHAIN_DIR)/usr/lib/swift/$(PLATFORM_NAME)
I hope Android won't have similar issue.. Thanks again for very quick support! And thanks for Lottie!

Where should I add that??


Update readme.java.md with new tag

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

@ChenTanyi  Can I know what is the reason to have this change? Looks the Java SDK auto gen is fail and the change you did is for Java

@akning-ms The java generation error is due to the swagger doesn't fit with java sdk generation in https://github.com/Azure/azure-rest-api-specs/pull/7699. The service team doesn't add java generation in that PR, so I update the java generation here.
I have asked the service team to fix the generation error. @pratimaupadhyay02 Could you update the swagger here in order to fix the java generation error, like we talked in email?

/openapi sdkautomation

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?


ERROR Error: [@ant-design/icons-angular]: the icon arrow-left-o does not exist or is not registered.

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start?embed=1&file=package.json
Steps to reproduce
根据官网快速上手一步步创建，在使用图标的时候错误。
What is expected?
能够显示图标
What is actually happening?
ERROR Error: [@ant-design/icons-angular]: the icon arrow-left-o does not exist or is not registered.

Environment
Info

ng-zorro-antd
7.3.2

Browser
Chrome

原因：
{
"glob": "**/*",
"input": "./node_modules/@ant-design/icons-angular/src/inline-svg/",
"output": "/assets/"
}
版本7.3.2install后node_moudle没有/@ant-design/icons-angular/src/inline-svg/目录

重新安装即可解决。

@BruneXX Please read the doc https://ng.ant.design/components/icon/en#static-loading-and-dynamic-loading. You need to register icons (import icon resources in another word). The error is quite explicit IMHO.

PR is welcome! @BruneXX

Hi! I received this error when running tests. Icons is dynamically linked under "ant" namespace. Can you help me?

@olegdeezus In test environments, I strongly recommend you to write a IconTestingModule and provide all icons in the module. And import that module when you are configuraing tests.
You can use this as a reference. https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/components/icon/testing/nz-icon-test.module.ts
Hope this helps.

@wendzhue Thank you!

Hi,
Thanks for the answers above,
I am Using Angular Universal and I should mention that for my case it seems the error was displayed but the svgs were displayed as  HTTP remote calls was loading them. It seems the error showing on the console is therefore a false alarm, since its not aware that they are dynamically loaded.
Now its quite strange to wire my mind to ignore the error messages displayed

Hi,
Thanks for the answers above,
I am Using Angular Universal and I should mention that for my case it seems the error was displayed but the svgs were displayed as HTTP remote calls was loading them. It seems the error showing on the console is therefore a false alarm, since its not aware that they are dynamically loaded.
Now its quite strange to wire my mind to ignore the error messages displayed

I'm facing the same issue. Is there any solution to get rid of those ugly error messages on the console?

Hi,
Thanks for the answers above,
I am Using Angular Universal and I should mention that for my case it seems the error was displayed but the svgs were displayed as HTTP remote calls was loading them. It seems the error showing on the console is therefore a false alarm, since its not aware that they are dynamically loaded.
Now its quite strange to wire my mind to ignore the error messages displayed

I am facing the same in Angular Universal.

重新安装即可解决。

Hope you are here to help me?
File : package.json
`  "dependencies": {
"@angular/animations": "~10.0.2",

"@angular/common": @ #"~10.0.2",

"@angular/compiler": "~10.0.2",

"@angular/core": "~10.0.2",

"@angular/forms": "~10.0.2",

"@angular/platform-browser": "~10.0.2",

"@angular/platform-browser-dynamic": "~10.0.2",

"@angular/router": "~10.0.2",

"@ant-design/icons": "^4.2.2",

"@ant-design/icons-angular": "^10.0.1",

"ng-zorro-antd": "^10.0.2",

"rxjs": "~6.5.5",

"tslib": "^2.0.0",

"zone.js": "~0.10.3"`

When I want to add an icon :

It runs but

Error:
http://localhost:4200/assets/outline/menu-fold.svg 404 (Not Found)
ERROR Error: [@ant-design/icons-angular]:the icon menu-fold-o does not exist or is not registered.
Thanks you

Fix this here!
https://giai-ma.blogspot.com/2021/05/ant-designicons-angular-icon-reload-o.html

This issue solved with installing @ant-design/icons-angular and add " {
"glob": "**/*",
"input": "node_modules/@ant-design/icons-angular/src/inline-svg/",
"output": "/assets/"
}"
in angualr.json.
Thank you

In my particular case if I use:  the following error arise:
Can't bind to 'nzType' since it isn't a known property of 'i'
but when I'm using  the specific error displayed here appears:
ERROR Error: [@ant-design/icons-angular]: the icon menu-fold-o does not exist or is not registered.
I've extracted that from the documentation page and nothing of above tried seems to work.

Please add import { NzButtonModule } from 'ng-zorro-antd/button'; to app.module and then add NzButtonModule to the list of imports.
That should do the trick.

In my particular case if I use:  the following error arise:
Can't bind to 'nzType' since it isn't a known property of 'i'
but when I'm using  the specific error displayed here appears:
ERROR Error: [@ant-design/icons-angular]: the icon menu-fold-o does not exist or is not registered.
I've extracted that from the documentation page and nothing of above tried seems to work.

I have come across the same problem as you. I did as the documention says, but it didn't work.
Can you help me ? thank you

我也遇到过同样的问题，删除node_modules之后重新下载依赖之后就好了，仅供参考

@wendzhue Thank you

Hi all,
I m facing the same issue when I try to run ng test.
I've tried all the solutions above but I still cannot run ng test without getting many errors saying the following message:
ERROR Error: [@ant-design/icons-angular]: the icon arrow-left-o does not exist or is not registered.
My version of ng-zorro-antd is 13.0.0

As wzhudev says above
1：Create a new file . eg: import { NzIconTestModule } from<EMAIL_ADDRESS>You can use this as a reference. https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/components/icon/testing/nz-icon-test.module.ts
2: Introduce it in your test file eg: beforeEach(async () => { await TestBed.configureTestingModule({ imports: [ NzIconTestModule //here
now , is OK！

Thanks chenweihan.
I already implemented that but nothing has changed.

@dimitriadisat
please check it again, because this file is the file in the official test case.
https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/components/icon/testing/nz-icon-test.module.ts.

This issue solved with installing @ant-design/icons-angular and add " { "glob": "**/*", "input": "node_modules/@ant-design/icons-angular/src/inline-svg/", "output": "/assets/" }" in angualr.json. Thank you

thank you this is working

Create a new icons-provider in the directory at the same level as app, and type the icons you need, and then introduce app.config.ts
import {EnvironmentProviders, importProvidersFrom} from '@angular/core';
import {
  BookOutline,
  EnterOutline,
  LockOutline,
  LoginOutline,
  MenuFoldOutline,
  MenuUnfoldOutline,
  PlusOutline,
  RollbackOutline,
  SearchOutline,
  UserOutline
} from '@ant-design/icons-angular/icons';
import {NzIconModule} from 'ng-zorro-antd/icon';

const icons = [PlusOutline, UserOutline, BookOutline, LoginOutline, MenuFoldOutline, MenuUnfoldOutline,
  EnterOutline, LockOutline, SearchOutline, RollbackOutline];

export function provideNzIcons(): EnvironmentProviders {
  return importProvidersFrom(NzIconModule.forRoot(icons));
}


Glitter App Pages

Fixes #69
See integration.rst for instructions on how this works.

Nudging @tomkins


add src directory and add.py

I've added a source folder ("src") with two new python files. One of them returns x + y with x being the first number and y defined as 100.
Suggest we merge it to the base.

Hi Cris - Do you want me to add as the reviewer for this change so I can review and provide approval.

Merged with main


Asyncio implementation

Experimental stage.
Comments welcomed.

Very nice! Reviewing today :)

This seems to work pretty well! I've merged it in. Sorry for delay, I kept running into issues building uWSGI with asyncio plugin on Mac OS X and putting things off :)


Clarify Disk encryption support

If I get this section correct, only ADE PMK is supported. DES PMK is not supported.

Learn Build status updates of commit 4e1064d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/defender-for-cloud/concept-agentless-data-collection.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 9ad9bc9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/defender-for-cloud/concept-agentless-data-collection.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@bmansheim

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

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

Confirmed by Matan Shabtay
#sign-off


Subnet warning incorrect.

For some subnets, even though the TV and PC are on the same subnet - LGTV Companion warns that they are not.
Here's an example:
PC is <IP_ADDRESS>/22
LGTV is <IP_ADDRESS>/22

For the above subnet:
Network Address/CIDR: <IP_ADDRESS>/22
Subnet Mask: <IP_ADDRESS>
Valid Host IP rage: <IP_ADDRESS> - <IP_ADDRESS>
Broadcast Address: <IP_ADDRESS>

I don't know if it's related or not, but scan has never worked for me in such a configuration either. Manual configuration seems to work fine though.

Hi @individualist , thanks for the bug report. I will review the code

This should be fixed in v3.1.9, Issue closed

Hi, @JPersson77
Unfortunately this is still happening for me for subnet <IP_ADDRESS> (22)
I could be mistaken, but I think the issue may be that during the conversion, the CIDR is being incremented on all bits, but should only increment for bits that are on...
https://github.com/JPersson77/LGTVCompanion/blob/83073254c70344adb692f335fcff47db902ccb53/Common/Common.cpp#L152-L155
e.g.
if ((m & 0x01) == 0x01) CIDR++

This seems to produce CIDR=22, as I expected. Otherwise it produces CIDR=24.
As an aside, this tool is great, thanks for creating it: it really increases quality of life when using a LG OLED.

Thanks! Appreciate the detailed response again. Reopening the issue to try again

Should be fixed in the next version. Please confirm

Closing due to inactivity. Please feel free to reopen as needed


[BUG] Augmented projection does not supply types

Is there an existing issue for this?

[X] I have searched the existing issues

Nature of Your Project
JavaScript, TypeScript
Current Behavior
Currently, when I extend an external service with a custom association, it is not included in the .d.ts files generated by cds-typer. It seems that the select on the projection is not even taken into account by cds-typer.
Expected Behavior
I expect that when I either restrict a service projection with a select or extend it with new fields/expands, it should be considered by cds-typer.
Steps To Reproduce
One can reproduce the behaviour with the ext-service-consume-ui branch in the Risk Management Sample:

Clone the sample branch and add cds-typer:

git clone --branch ext-service-consume-ui https://github.com/SAP-samples/cloud-cap-risk-management.git
cd cloud-cap-risk-management
cds add typer
npm i

add an association to the Suppliers entity (it is not necessary, because the select will also not be generated):

entity Suppliers as projection on bupa.A_BusinessPartner {
  key BusinessPartner          as ID,
      BusinessPartnerFullName  as fullName,
      BusinessPartnerIsBlocked as isBlocked,
+      to_Risks : Association to many Risks on to_Risks.supplier = $self
}

generate types

npx @cap-js/cds-typer '*' --outputDirectory @cds-models

Check the generated Types

export function _SupplierAspect<TBase extends new (...args: any[]) => object>(Base: TBase) {
  return class Supplier extends Base {
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        BusinessPartner?: string;
    /**
    * Gives an alphanumeric key, which clearly identifies the customer or vendor in the SAP system.
    */
        Customer?: string;
    /**
    * Specifies an alphanumeric key that uniquely identifies the supplier in the SAP system.
    */
        Supplier?: string;
    /**
    * Key for academic title.
    *
    * You can define a key for an academic title in Customizing.
    */
        AcademicTitle?: string;
    /**
    * You can use authorization groups to stipulate which business partners a user is allowed to process.
    *
    * Use the following authorization object:'Business partners: authorization groups' (B_BUPA_GRP).The system only checks this authorization if you made an entry in the "Authorization group" field for the business partner. Otherwise, any user may process the business partner.
    */
        AuthorizationGroup?: string;
    /**
    * Category under which a business partner is classified.
    *
    * You can distinguish between the following business partner categories:OrganizationNatural personGroup of natural persons or organizationsThe processing screens for the business partner categories are set up differently.So, for example, the screen for an organization contains the field Legal form, but this is not needed in the screen for a natural person.
    */
        BusinessPartnerCategory?: string;
        BusinessPartnerFullName?: string;
    /**
    * Classification assigned when creating a business partner.
    *
    * Assign each business partner to a grouping when you create the business partner. The grouping determines the number range. You cannot change the assignment afterwards.You can define the groupings, their descriptions, the associated number range and other attributes in Customizing.You can define standard groupings for the internal and the external number assignment.For each grouping create a number range.When you create a business partner, the entry in the grouping field determines whether and how an entry is made in the business partner number field.
    */
        BusinessPartnerGrouping?: string;
        BusinessPartnerName?: string;
        BusinessPartnerUUID?: string;
    /**
    * Correspondence language (written) for business partners in the 'Person' category. Maintain the correspondence language for business partners in the 'Organization' and 'Group' category with the address (communication).
    *
    * When transferring data (direct input), make sure that for a'Person', the field 'LANGU_CORR' and for an'Organization' or "Group" the field 'LANGU'has an entry.
    */
        CorrespondenceLanguage?: string;
        CreatedByUser?: string;
        CreationDate?: string;
        CreationTime?: string;
        FirstName?: string;
    /**
    * Key for form of address text.
    *
    * You can also define a form of address text in Customizing.The form of address text can be maintained in different languages.
    */
        FormOfAddress?: string;
    /**
    * An industry sector is the term used to classify a company according to its main business activity.
    *
    * You can assign an industry sector to business partners in the category 'Organization'RetailBanksServicesIndustryHealth servicePublic sectorMedia
    */
        Industry?: string;
    /**
    * Here you enter the first 7 digits of the international location number.
    *
    * The International Location Number (ILN) is assigned (in Germany by the Centrale for Coorganisation GmbH)) when a company is founded. It consists of 13 digits, the last digit being the check digit. There are two categories of location numbers:Participants who only need an ILN to cleary and unmistakably identify themselves for communication with the business partner are given a category 1 ILN. This cannot be used to identify articles by means of EAN.Participants who wish to assign the location numbers for their own enterprise areas are given a category 2 ILN. For a category 2 ILN, digits 1 to 7 are described as basis number. This is used as basis for the creation of article numbers (EAN).
    */
        InternationalLocationNumber1?: string;
    /**
    * Here, you enter digits 8-12 of the 13-digit international location number.
    *
    * The international location number (ILN) is assigned when establishing a company (by the "Zentrale für Coorganisation GmbH" in Germany). It consists of 13 digits, the last of which is the check digit. There are two types of international location numbers:Subscribers who only need one ILN to identify themselves in communication with the business partner are given an ILN of type 1. These cannot be used for identifying articles by means of EAN.Subscribers who need to assign location numbers for their own company areas are given an ILN of type 2. Positions 1 through 7 of the ILN type 2 are known as the basis number. This basis number forms the basis for article numbers (EAN).
    */
        InternationalLocationNumber2?: string;
        IsFemale?: boolean;
        IsMale?: boolean;
    /**
    * Indicator through which a distinction between natural and legal persons can be made during tax reporting.
    *
    * Is used in Italy and Mexico ,among other countries.Brasil: If the indicator is not set, 'CGC' is relevant in tax number 1. If the indicator is set, 'CPF' is relevant in tax number 2.Colombia: In the case of some natural persons, the NIT number does not have a check digit. In this case you should set this indicator and the check is deactivated.
    */
        IsNaturalPerson?: string;
        IsSexUnknown?: boolean;
        GenderCodeName?: string;
    /**
    * Language for verbal communication with a business partner.
    *
    * This language may differ from the language(s) defined for written correspondence.
    */
        Language?: string;
        LastChangeDate?: string;
        LastChangeTime?: string;
        LastChangedByUser?: string;
        LastName?: string;
    /**
    * Denotes certain legal norms that are of significance for the organization of a company.
    *
    * For business partners in the category "Organization", you can state the legal form of the company. This is for information purposes only.Stock corporation (AG in Germany)Limited liability company (GmbH in Germany)
    */
        LegalForm?: string;
    /**
    * First name field for business partners in the Organization category.
    */
        OrganizationBPName1?: string;
    /**
    * Second name field for business partners in the Organization category.
    */
        OrganizationBPName2?: string;
    /**
    * Third name field for business partners in the Organization category.
    */
        OrganizationBPName3?: string;
    /**
    * Fourth name field for business partners in the Organization category.
    */
        OrganizationBPName4?: string;
    /**
    * Indicates the official registration of a company in the Commercial Register.
    *
    * If a company is not officially registered in the Commercial Register, it has to use some type of text addition, such as foundation pending, when referring to the legal form.
    */
        OrganizationFoundationDate?: string;
    /**
    * Term for the end of bankruptcy proceedings.
    *
    * This date also indicates that the company no longer exists.
    */
        OrganizationLiquidationDate?: string;
    /**
    * Denotes the term that you define for a business partner, and via which you can restrict the search for a business partner in the business partner search or in the locator.
    */
        SearchTerm1?: string;
    /**
    * Denotes the term that you define for a business partner, and via which you can restrict the search for a business partner in the business partner search or in the locator.
    */
        SearchTerm2?: string;
        AdditionalLastName?: string;
        BirthDate?: string;
    /**
    * If the Business partner's actual birthdate is unknown, a partial entry can be made or read. This field then indicates which part of the birthdate is unknown. Below are some examples (Dates are specified in format MM/DD/YYYY):
    *
    * 01/01/0000 => 01/01/1880 Year unknown00/01/1911 => 01/01/1911 Month unknown05/00/1911 => 05/01/1911 Day unknown05/05/1911 => 05/05/1911 Fully known00/00/1911 => 05/05/1911 Day & Month unknown00/05/0000 => 05/05/1911 Month & Year unknown05/00/0000 => 05/05/1911 Day & Year unknown00/00/0000=> 05/05/1911 Fully unknown'Date of Birth: Status' field determines the unknown parts of birthdate.Enter a valid date under Birthdate. Future dates are not accepted. If any part/s of the date is/are unknown, select the nearest date.If year is unknown, select any random valid year.
    */
        BusinessPartnerBirthDateStatus?: string;
        BusinessPartnerBirthplaceName?: string;
        BusinessPartnerDeathDate?: string;
    /**
    * If the business partner is blocked centrally, certain activities cannot be executed.
    */
        BusinessPartnerIsBlocked?: boolean;
    /**
    * You can use the business partner type to group business partners according to your own criteria in Customizing (IMG).
    *
    * In Customizing you can show or hide fields for data entry, depending on the requirements of the relevant business partner type.Select a business partner type. You can obtain help by pressing the F4 key.
    */
        BusinessPartnerType?: string;
        ETag?: string;
    /**
    * First name field for business partners in the Group category.
    */
        GroupBusinessPartnerName1?: string;
    /**
    * Second name field for business partners in the Group category.
    */
        GroupBusinessPartnerName2?: string;
    /**
    * Internal key for identifying the address for communication data that spans all addresses in Business Partner.
    *
    * For more information on the significance and usage of the address number, see the documentation for Business Address Services (BAS).
    */
        IndependentAddressID?: string;
    /**
    * The check digit is derived from a special check digit procedure from digits of the previous international location numbers. In this way, you can check whether the ILN entered is actually valid.
    */
        InternationalLocationNumber3?: string;
        MiddleName?: string;
    /**
    * The country/region for the name format rule and the name format rule key together uniquely identify a formatting rule.
    *
    * A country/region can have several formats which correspond to different rules. You need formatting rules to describe the composition of a person's name.
    */
        NameCountry?: string;
    /**
    * See Name format.
    */
        NameFormat?: string;
    /**
    * States the complete name of a person.
    *
    * The complete name is generally generated and saved by the Business Address Services (BAS) according to country-specific rules from the individual name components (without the form of address).If, during the formatting of an address, you want to use an alternative name, you can manually format the alternative name here.
    */
        PersonFullName?: string;
    /**
    * Internal key for identifying a person in Business Address Services.
    *
    * For more information about the meaning and use of the person number and Business Address Services concepts, see the function group SZA0 documentation.
    */
        PersonNumber?: string;
    /**
    * Establishes if the business partner is meant to be archived.
    *
    * If the indicator is set, the relevant business partner can be archived from the view of the business partner administration.If the check of the data to be archived shows, for example, that there are still active business transactions the archiving of the business partner data is prevented even if the indicator is set.If the indicator is not set, the business partner will not be taken into consideration during archiving.
    */
        IsMarkedForArchiving?: boolean;
    /**
    * Business partner number from an external system or a legacy system.
    *
    * If the current business partner is known under a different number in an external system, you can store this number here for information purposes.Direct input gives you the option of maintaining a business partner by specifying the external business partner number. If you maintain business partner data in your legacy system, you can transmit changes made to business partners to the SAP system without having to know the SAP business partner number in the legacy system.
    */
        BusinessPartnerIDByExtSystem?: string;
    /**
    * Company ID standard for the whole group.
    */
        TradingPartner?: string;
    /**
    * Please add  an ON condition
    */
        to_BuPaIdentification?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BuPaIdentification_>;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BuPaIdentification_BusinessPartner?: string;
    /**
    * A document (such as an ID card or driver's license) or an entry in a system of records (such as a commercial register) whose key can be stored as an attribute for a business partner.
    *
    * The identification type is for classifying identification numbers.You can define the identification types and their descriptions in Customizing.You can also specify for which business partner category an ID type should be valid.If necessary, assign the identification type to an Identification Category.You can only assign one identification type to an identification category.
    */
        to_BuPaIdentification_BPIdentificationType?: string;
    /**
    * Number that serves to identify a business partner, such as driver's license, or ID card number.
    */
        to_BuPaIdentification_BPIdentificationNumber?: string;
    /**
    * Please add  an ON condition
    */
        to_BuPaIndustry?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BuPaIndustry_>;
    /**
    * Describes an industry.
    *
    * An industry is a classification of companies according to their main business activity. For example, you can use Commerce, Banking, Services, Industry, Healthcare, Public Sector, Media, and so on, as industries.You can define industries along with their descriptions in Customizing.Assign the industry key to an industry key system.
    */
        to_BuPaIndustry_IndustrySector?: string;
    /**
    * Serves to combine and categorize several industries into a group.
    *
    * You can create different industry systems, each with its own catalog of industries, whereby an industry can be assigned to several industry systems.You have to select one industry system as the standard industry system. This is then automatically displayed in the initial screen for the maintenance of industry data.You can define an industry system along with its description in Customizing. You can assign several industry systems to a business partner.If you choose the button All Industry Systems, you can access all the industry systems defined in the Customizing using the input help.
    */
        to_BuPaIndustry_IndustrySystemType?: string;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BuPaIndustry_BusinessPartner?: string;
    /**
    * Please add  an ON condition
    */
        to_BusinessPartnerAddress?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BusinessPartnerAddress_>;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BusinessPartnerAddress_BusinessPartner?: string;
    /**
    * Internal key for identifying a Business Address Services address.
    *
    * For more information about the meaning and use of the address number and the Business Address Services concepts, see the function group SZA0 documentation.
    */
        to_BusinessPartnerAddress_AddressID?: string;
    /**
    * Please add  an ON condition
    */
        to_BusinessPartnerBank?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BusinessPartnerBank_>;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BusinessPartnerBank_BusinessPartner?: string;
    /**
    * Key identifying a business partner's bank details.
    *
    * Enter a bank details ID for each separate set of bank details for a business partner.Business Partner: H. MillerBD-ID Fin.institution Acct no. 0001 Chemical Bank, NYC<PHONE_NUMBER>2 Chemical Bank, NYC<PHONE_NUMBER>3 First Bank of Pittsburgh ...Business partner: T.Wolsey and Co.BD-ID Fin.institution Acct no.GIR0 Citibank, Charleston ...GIR1 Chemical Bank, NYC ...
    */
        to_BusinessPartnerBank_BankIdentification?: string;
    /**
    * Please add  an ON condition
    */
        to_BusinessPartnerContact?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BusinessPartnerContact_>;
    /**
    * The business partner relationship number is an internal number that identifies the business partner relationship set.
    */
        to_BusinessPartnerContact_RelationshipNumber?: string;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BusinessPartnerContact_BusinessPartnerCompany?: string;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BusinessPartnerContact_BusinessPartnerPerson?: string;
        to_BusinessPartnerContact_ValidityEndDate?: string;
    /**
    * Please add  an ON condition
    */
        to_BusinessPartnerRole?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BusinessPartnerRole_>;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BusinessPartnerRole_BusinessPartner?: string;
    /**
    * Function that a business partner takes on, depending on a business transaction.
    *
    * You can define business partner roles along with their attributes in Customizing.You can create an alphanumeric, 6-digit key for the BP role. You can also choose a title as the short form and a description as the long form for the role text.Screen control in the dialog takes place by assigning a BP view.A program can access specific business partner roles for a business partner using thebusiness partner role category . The role categories are also in the TB003 table.
    */
        to_BusinessPartnerRole_BusinessPartnerRole?: string;
    /**
    * Please add  an ON condition
    */
        to_BusinessPartnerTax?: __.Association.to.many<_API_BUSINESS_PARTNER.A_BusinessPartnerTaxNumber_>;
    /**
    * Key identifying a business partner in the SAP system. The key is unique within a client.
    */
        to_BusinessPartnerTax_BusinessPartner?: string;
    /**
    * Specifies the tax number category.
    */
        to_BusinessPartnerTax_BPTaxType?: string;
    /**
    * Please add  an ON condition
    */
        to_Customer?: __.Association.to<_API_BUSINESS_PARTNER.A_Customer>;
    /**
    * Gives an alphanumeric key, which clearly identifies the customer or vendor in the SAP system.
    */
        to_Customer_Customer?: string;
    /**
    * Please add  an ON condition
    */
        to_Supplier?: __.Association.to<_API_BUSINESS_PARTNER.A_Supplier>;
    /**
    * Specifies an alphanumeric key that uniquely identifies the supplier in the SAP system.
    */
        to_Supplier_Supplier?: string;
      static actions: {
    }
  };
}
export class Supplier extends _SupplierAspect(__.Entity) {}

Environment
- **OS**:
- **Node**: v18.18.0
- **npm**: 9.8.1
- **cds-typer**: 0.11.1
- **cds**: 6.8.4

Repository Containing a Minimal Reproducible Example
No response
Anything else?
No response

As I'm debugging my way through the source code, maybe the problem is that cds-typer is working with the xtended falvour of CSN instead of the inferred, as stated in the visitor.js:visitDefinitions()?
Maybe I am on the wrong track here, but I wanted to add this to my request. Below you find a Screenshot with the difference between inferred and xtended CSN for the Risk Management Sample from above.
If I can support here, please let me know
BR Tom

Hi, i am currently trying to solve a problem similar to yours, but i have as select from with joins.

but looking at solution with providing additional arguments on runtime

and propagate to compile csn

try test, maybe its better for us to expand this part with arguments?

Hi naferok,
I tried your suggestion and it looks good. As I suspected, I need the inferred-flavor in my current project.
Your suggestion to pass the flavor through sounds reasonable to me, but I can't evaluate whether this might cause problems elsewhere.
I will continue follow the idea of the inferred flavor.
BR Tom

Hi naferok,
I've implemented the proposed changes in a forked repo, which is currently passing all the tests. Please feel free to have a look into it, so we can eventually open a PR for this.
BR Tom


Add exception handling for versioning and locking

I'll merge this manually.

Turns out, there's nothing to do. The current code already contains all fixes.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104440

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/117182


Why do you use PDateTime in the sample code?

your readme https://github.com/urasandesu/Prig/blob/master/README.md uses PDateTime which is not anywhere in the scope - this is your custom class i believe... - https://gist.github.com/urasandesu/8751367
having that, what is the meaning of example???:
[Test]
public void IsNowLunchBreak_should_return_false_when_11_oclock()   {
// IndirectionsContext can minimize the influence of the API replacement.
using (new IndirectionsContext())
{
// Arrange
// Replace DateTime.Now body. Hereafter, DateTime.Now will return only 2013/12/13 11:00:00.
PDateTime.NowGet().Body = () => new DateTime(2013, 12, 13, 11, 00, 00);
            // Act
            var result = LifeInfo.IsNowLunchBreak();

            // Assert
            Assert.IsFalse(result);
        }
    }

PDateTime is a Prig Type, so it will be generated automatically based on Stub Settings File(in QUICK TOUR case, the settings is in mscorlib.v4.0.30319.v<IP_ADDRESS>.prig). If you can't find the type, the auto generation has possibly failed.
Was the Step 3 success?

For your information, the gist is just a memo for the design of Prig Type.


Remove check for dotnet.exe from startvs.cmd.

This check is not helpful. At points in time where we are running against stable bits (like 8.0.100 RTM) the .\restore.cmd script won't actually install a repo local copy of .NET, and so this path will never exist. We may as well ditch the check.

The check guards against attempts to start VS without restoring the toolset first. The change also presumes .NET 8 is installed globally, which is not necessarily the correct assumption.

Unfortunately restore.cmd assumes that if it is installed globally it doesn't need to install it locally. Which means right now, with 8.0.100 being what we are using in the repo, startvs.cmd is broken. If we are going to have a check it needs to be able to work regardless of where we are in the release lifecycle.


Load more Posts doesn't work or is super slow

Describe the bug
Load more Posts doesn't work or is super slow
To Reproduce
Scroll down on the Feed and try to load more posts
Screenshots
see bottom of the screenshot attached
Smartphone (please complete the following information):
Android
Vivo V9

fixed with the new version


Implement children for org transform

The current implementation of the org transform supports parent, but does not implement mapping of children orgs.
The requirements call for:
Children == $.children (array of strings, identifiers for each child), mapped to OBO.BFO_0000051 for each child identifier as a child organization URI

Lets come to some understanding why/if we need bi-directional association before we start on this.

I'm good with that.

Not currently required.


keygen output

It might be a good idea to generate output for the gobl keygen command that indicates what files have been generated, and where.
Does it make sense to write message to STDERR incase STDOUT is being used to pipe a generated key (perhaps?) A -v tag for verbose output might be a good idea.

I think this type of thing requires some thought, as there are several ways to do something like this.
For this particular feature, what probably makes sense to me is outputting helpful info to STDOUT except when STDOUT is being used as the output location of the key (in which case the helpful output isn't helpful anyway), or when quiet mode (i.e. --quiet?) is on (i.e for scripting purposes).
I expect there are output situations not covered by that, though.

I think for this application where STDOUT is likely to be used for data output its acceptable to use STDERR for all logging, as a rule. Sending to STDERR requires more effort, hence why it's not so common, but I don't think it's bad practice.
If a logging library is used, then it should be easy to add a --quiet option too.

The problem with sending non-errors to STDERR is it makes error handling/detection in scripts very.

It occurred to me that perhaps an alternate solution would be to always generate JSON output and make sure that the exit status code is non-zero. If you're worried about script handling, this would solve the problem and provide a universal interface with comparable results in both CLI and HTTP.
Argument errors, i.e. stuff caught by cobra, should be sent to STDERR as this clearly reflects invalid usage and you wouldn't want those to be scriptable.


Restore working Crowdin scripts

Summary
@SimenB @merceyz the change made here:
https://github.com/jest-website-migration/jest/commit/fd34cd0cf64f0f1d0b0750400a65914c65b65df8
It broke Crowdin translation download (didn't notice)
Crowdin is quite sensitive to the path from which you call its cli, and it seems using the : and global yarn script does not run from the place where the cli is declared, unlike yarn workspace.
Test plan
Netlify preview works

Codecov Report

Merging #11082 (cebd69d) into jest-website-v2 (2fd0ce1) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           jest-website-v2   #11082   +/-   ##
================================================
  Coverage            64.22%   64.22%
================================================
  Files                  305      305
  Lines                13255    13255
  Branches              3235     3235
================================================
  Hits                  8513     8513
  Misses                4050     4050
  Partials               692      692

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 2fd0ce1...cebd69d. Read the comment docs.

Some $pwd thing? interesting. Would love @arcanis and @merceyz 👀 on this

$pwd ?
the crowdin cli should be run from the root so I think we can use yarn workspace or yarn .. (as --cwd apparently does not work?)

Right, I meant cwd (my thought process was cwd is "run pwd in a terminal" which prints cwd, thus the confusion).
My question is - why is the working directory different between yarn workspace @jest/monorepo crowdin:upload and yarn workspace crowdin:upload if any scripts with a : in it is supposed to be global?

I can double check tomorrow, but I'm fairly sure it's not the case - colon-scripts and workspace <script> should have the same effect 🤔

You are right @arcanis my mistake, I had the crowdin:upload script declared in both the monorepo + subpackage 😅


remove /server_api proxy, It's doesn't work!

I want change the edp webserver to express, so I was removed '/server_api' proxy, and change the url to 'http://vop.baidu.com/server_api', but It doesn't work...
console throwError:

XMLHttpRequest cannot load http://vop.baidu.com/server_api. Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://<IP_ADDRESS>:8848' is therefore not allowed access.
index.html:73 Error: {"type":"unknow"}(anonymous function) @ index.html:73

The RESTful API (http://vop.baidu.com/server_api) didn't support across-domain access yet, so it will work fine if you use Express to make a proxy API for it just like the demo.


latest 1.5.10 AppImage version have no desktop integration pop-up when first launch and there is no icon-tray

Describe the bug
as title.
Expected behavior
previous 1.4.1 have this pop when opening the app.
Environment:

OS & Version: [Linux]
Version: [linux-mint 19.3]
Motrix Version: [1.5.10]
Installation package type: [AppImage]

edited:added icon-tray issue

I think this not a bug : |

Desktop Integration
Since electron-builder 21 desktop integration is not a part of produced AppImage file.
AppImageLauncher is the recommended way to integrate AppImages.

https://www.electron.build/configuration/appimage

okay so I should used AppImageLauncher.
how about there is no icon tray when closing the it does not stay in background like previous version

ail-cli integrate 'Motrix-1.5.10.AppImage'

https://github.com/agalwood/Motrix/issues/583#issuecomment-620323913

#583 (comment)

this for for icon-tray which I already have and enabled but only problem was that is needed to be re-order to be viewed also I see the colored version not (black/white) is that okay also already integrated the app using AppImageLauncher.
thanks for your support.

Due to negligent testing, the white tray icon under Linux is slightly larger : |
If you want to change back to the colorful tray icon, you can manually modify the tray-theme in user.json to 'colorful'.
The path of user.json is the same as system.json, please refer to https://github.com/agalwood/Motrix/issues/138#issuecomment-473632340

Due to negligent testing, the white tray icon under Linux is slightly larger : |
yes it larger than colored version.
(off-topic)any thought about making browser extension or system integration in the future (know that there are multiple extension and one of them specific for mortix but don't work will in linux)?

If I want to develop a Motrix browser extension with better experience, it is equivalent to developing a simple version of the Motrix Web UI. I haven't planned a development plan yet, so wait a while.

也是linux mint 19.3
用的不是applmage版本 而是deb安装的  也没有托盘图标 也要装AppImageLauncher吗？

@k3bc999 https://github.com/electron/electron/issues/21445#issuecomment-565710027 试试看

Cinnamon/Mint: "System Settings" -> "General" -> "Enable support for indicators" flag needs to be enabled. Some issues still possible, see linuxmint/cinnamon#9176.

@k3bc999 electron/electron#21445 (comment) 试试看

Cinnamon/Mint: "System Settings" -> "General" -> "Enable support for indicators" flag needs to be enabled. Some issues still possible, see linuxmint/cinnamon#9176.

用的是mate桌面 我找找解决思路去

https://github.com/electron/electron/pull/23674
看来等待electron更新是比较好的选择


no match found

Hello
when I run this command to install V2 beta :
cordova plugin add https://github.com/mapsplugin/cordova-plugin-googlemaps#multiple_maps --variable API_KEY_
FOR_ANDROID="AIz" --variable API_KEY_FOR_IOS="AIz"

it gives me this output error:
no matches found: https://github.com/mapsplugin/cordova-plugin-googlemaps#multiple_maps

The issue of plugin installing is your problem depends on your environment.
Unfortunately I can not help you at all. Too less information.

still this is a visible problem.. i'm working on ionic3 now
ionic info:
cli packages: (/Users/user/project/node_modules)

    @ionic/cli-utils  : 1.9.1
    ionic (Ionic CLI) : 3.9.1

global packages:

    Cordova CLI : 7.0.1

local packages:

    @ionic/app-scripts : 2.1.3
    Cordova Platforms  : browser 4.1.0
    Ionic Framework    : ionic-angular 3.6.0

System:

    ios-deploy : 1.9.0
    ios-sim    : 5.0.13
    Node       : v8.2.1
    npm        : 5.3.0
    OS         : macOS Sierra
    Xcode      : Xcode 8.0 Build version 8A218a

Remove the lines of cordova-plugin-googlemaps from config.xml, package.json and package-lock.json files.
Or you should create an empty project, then try it.
This is completely your environment problem.

thanks, that worked with me.
but now:

$> chmod +x platforms/android/gradlew

even aftre running the command above, the problem still there.
note:
i'm using firebase plugins
here's a peak of dependancies in build.gradle:
compile "com.google.firebase:firebase-core:+"
    compile "com.google.firebase:firebase-messaging:+"
    compile "com.google.firebase:firebase-crash:+"
    compile "com.google.firebase:firebase-config:+"
    compile "com.google.android.gms:play-services-maps:+"
    compile "com.google.android.gms:play-services-location:+"

@wf9a5m75

I found that this problem caused by fcm plugin
thanks again
@wf9a5m75


Intermittent issue in Safari browser when I open a Tearsheet containing 2 tabs

What package(s) are you using?

[x] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive)
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description

Sometimes an intermittent  issue pops up in Safari browser when I open a Tearsheet with 2 tabs  as shown in the screenshot below :

Is this issue related to a specific component?
Tearsheet component having tabs inside. I have attached code sandbox link to test.

What browser are you working in?
Safari 16.3

What version of the @carbon/ibm-products (or @carbon/ibm-cloud-cognitive)
2.0.0-rc.6

package are you using?

"dependencies": {
    "@carbon/elements": "^10",
    "@carbon/ibm-products": "2.0.0-rc.6",
    "@carbon/icons-react": "11.13.0",
    "@carbon/react": "1.18.0",
    "carbon-components": "10.58.6",
    "carbon-components-react": "8.34.0",
    "react": "^17",
    "react-dom": "^17",
    "react-scripts": "^5",
    "sass": "latest"
  }

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Steps to reproduce the issue

Step one
Step two
Step three
etc.

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/proud-frost-5zvthy?file=/src/Example/Example.js

Additional information

Screenshots or code
Notes - It is an intermittent issue. I couldn't replicate it every time when I open the <Tearsheet/>

@leenu-r Wanted to check if you have still been seeing this issue.
Your dependencies also show you are using both Carbon 10 and Carbon 11 versions!
@carbon/ibm-products from version 2 should be used with @carbon/react, but most likely you do not need @carbon/elements or carbon-components@10 unless you are working on an incremental migration.